get-health-check reports 1 second invocation timeout when the default is used

Please fill out the issue checklist below and provide ALL the requested information.

[x] I reviewed open and closed github issues that may be related to my problem.
[x] I tried updating to the latest version of the CF CLI to see if it fixed my problem.
[x] I attempted to run the command with CF_TRACE=1 to help debug the issue.
[x] I am reporting a bug that others will be able to reproduce.

Describe the bug and the command you saw an issue with
While debugging why an application was being restarted by CF continuously we looked into the get-health-check of the application. The invocation timeout was listed as 1 and we believe this is wrong.
What happened
We do not set any healtcheck timeouts on our apps and rely on the default provided by CAPI which should be 60s default_health_check_timeout. Running the get-health-check actually returns 1s.
Expected behavior
We expected the command to return the correct value.
Exact Steps To Reproduce
Steps to reproduce the behavior; include the exact CLI commands and verbose output:

cf push <APP> an application without specifying the healthcheck timeout.
Run cf get-health-check <APP>

Provide more context

platform and shell details ( e.g. Mac OS X 10.11 iTerm): Ubuntu 22.04.1 LTS
version of the CLI you are running: cf version 8.4.0+b4c083e.2022-06-02
version of the CC API Release you are on: cf-for-k8s v5.7.1 and cf-deployment 21.4.0

**Possible code bug
This line in the CLI code when invocation timeout is set to 0 defaults the output of the command to 1, this seems like the problem: https://github.com/cloudfoundry/cli/blob/d3bb1ecfc94ee12432f8defb30ab9459e83cc727/command/v7/get_health_check_command.go#L65
Notes regarding V6 and V7 CLI support:

V6:

Minimum supported version of CF Deployment: v7.0.0 (CAPI Release: 1.74.0 (APIs 2.128.0 and 3.63.0))
Maximum supported version of CF Deployment: v13.4.0 (CAPI Release: 1.94.0 (APIs 2.149.0 and 3.84.0))

V7:

Minimum supported version of CF Deployment: v13.5.0 (CAPI Release: 1.95.0 (APIs 2.150.0 and 3.85.0))

Hi @mymasse. CLI assigns the value 1 when a 0/null value is returned from cloud controller. Not sure why the default 60s is not being sent to CLI from CC. Please feel free to open an issue with CAPI.

Even return 0/null would be better than returning 1, at least it would point out that it's not set. I agree that CAPI should probably return the default value but I still think the CLI should return something else than 1 in this since it still needs to support older CAPI release.

There are two timeouts for healthcheck. One is the default_health_check_timeout which has default of 60s as mentioned here. This is the time the health_check waits for the app to come up. The other one is health_check invocation_timeout which has a default of 1s as mentioned here. This is the time the health_check endpoint is polled to see if the app is responding or not until the defualt_time_out is elapsed. So for ex, health_check is performed for an app A every 1s upto 60s.
The reason CAPI is sending null is to differentiate if a value is set or not by the user. This has been the logic for over 6years. So supported older version of CAPI should be good.


HELP! Wrong static file path with koa-static

Document struct:
static/img/1.jpg
src/view
src/routes/index.js
Static file:
app.use(staticServe('static/'));
Router path:
router.get('/aaa/bbb', () => {
blabla...
ctx.render(.......)
})
But, I get a wrong path result on front end:
'http://localhost:8888/aaa/1.jpg' 404
HELP!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!

Well, it was resolved. Please use '/css/index.css' on IMG or LINK elements. Don't lose '/' as begain.


Authentication with MSAL Angular Access Token

Hello,
I've set up a secure connection with Darkloop (DarkLoop.Azure.Functions.Authorization.Isolated). I have an Angular frontend and users connect with their Microsoft account (accounts have roles). Once connected I have the access token and the token id. When I make requests to the backend, the access token is added to the header. Once the request reaches the backend I get an error, I think I may have made a mistake in the configuration?
I'm in .NET 8 isolated with Azure Functions v4
In the idToken I can see my user's roles [‘Admin’], in the access token they're not, which is where I can't make the link with RequireRole().
services.AddFunctionsAuthentication(JwtBearerDefaults.AuthenticationScheme)
            .AddJwtFunctionsBearer(options =>
            {
                options.Authority = "https://login.microsoftonline.com/tenantId";
                options.Audience = "audience";
            });
        services.AddFunctionsAuthorization(options =>
        {
            options.AddPolicy("OnlyAdmins", policy => policy.RequireRole(RoleConstant.GlobalAdmin));
        });

[16:21:02 INF] Failed to validate the token.
[2024-09-06T14:21:02.724Z] Microsoft.IdentityModel.Tokens.SecurityTokenInvalidSignatureException: IDX10511: Signature validation failed. Keys tried: 'Microsoft.IdentityModel.Tokens.X509SecurityKey, KeyId: 'H9nj5AOSswMphg1SFx7jaV-lB9w', InternalId: 'H9nj5AOSswMphg1SFx7jaV-lB9w'. , KeyId: H9nj5AOSswMphg1SFx7jaV-lB9w
[2024-09-06T14:21:02.724Z] '.
[2024-09-06T14:21:02.724Z] Number of keys in TokenValidationParameters: '0'.
[2024-09-06T14:21:02.724Z] Number of keys in Configuration: '12'.
[2024-09-06T14:21:02.724Z] Matched key was in 'Configuration'.
[2024-09-06T14:21:02.724Z] kid: 'H9nj5AOSswMphg1SFx7jaV-lB9w'.
[2024-09-06T14:21:02.724Z] Exceptions caught:
[2024-09-06T14:21:02.724Z]  '[PII of type 'System.String' is hidden. For more details, see https://aka.ms/IdentityModel/PII.]'.
[2024-09-06T14:21:02.724Z] token: '[PII of type 'Microsoft.IdentityModel.JsonWebTokens.JsonWebToken' is hidden. For more details, see https://aka.ms/IdentityModel/PII.]'. See https://aka.ms/IDX10511 for details.
[2024-09-06T14:21:02.724Z]    at Microsoft.IdentityModel.JsonWebTokens.JsonWebTokenHandler.ValidateSignature(JsonWebToken jwtToken, TokenValidationParameters validationParameters, BaseConfiguration configuration)
[2024-09-06T14:21:02.724Z]    at Microsoft.IdentityModel.JsonWebTokens.JsonWebTokenHandler.ValidateSignatureAndIssuerSecurityKey(JsonWebToken jsonWebToken, TokenValidationParameters validationParameters, BaseConfiguration configuration)
[2024-09-06T14:21:02.724Z]    at Microsoft.IdentityModel.JsonWebTokens.JsonWebTokenHandler.ValidateJWSAsync(JsonWebToken jsonWebToken, TokenValidationParameters validationParameters, BaseConfiguration configuration)

Thanks :)

Hi @Tweentyy, do you have an app registration for each application or are you using the same app registration for both?
Based on your comments above I imagine you have different registrations due to the fact that the id token is the one carrying roles. Have you configured the registration for functions to return roles for users?
Can you elaborate a little more on your app registration setup in EntraID. The ideal situation for functions as they are seen as backend APIs is to consume access tokens.

Hello, thank you for your reply. I'm not available next week, but I'm already starting to make enquiries.
I'll get back to you the following week as soon as I'm available. I didn't take care of this part

@Tweentyy - Just curious if you are using Static Web Apps in Azure to host the front-end Angular application and if you are also using the built-in Static Web Apps - Azure Function for making calls to the back-end (hence the reason for using library).
If so, what I have recently discovered is that Azure replaces the "Authorization" header bearer token from browser request (the one you see in Chrome Dev Tools or Fiddler) going to back-end with their own Proxy bearer token before it actually makes it to your Function App endpoints. See this Issue where @artmasa helped me setup my app to use a custom 'Authorization' header (X-Custom-Authorization). #57
To confirm the bearer token is being replaced or not, I would recommend logging out the Authorization header in Function App log and compare with what is showing in Browser Dev Tools Header and see if it's being replaced.
Paste your bearer toke in this site to parse and see contents: https://jwt.ms/
NOTE: If this is the issue, I used an custom HttpInterceptor in Angular to automatically clone the Authorization header in browser to my own custom. Then use the Program.cs configuration from my Issue posted above to map over this custom header to be the one the Authorize attribute uses.
import { Injectable } from '@angular/core';
import { Router } from '@angular/router';
import { HttpInterceptor, HttpRequest, HttpHandler, HttpEvent, HttpResponse } from '@angular/common/http';
import { Observable } from 'rxjs';

@Injectable()
export class CustomAuthInterceptor implements HttpInterceptor {
  constructor(private router: Router) {

  }

  intercept(req: HttpRequest<any>, next: HttpHandler): Observable<HttpEvent<any>> {

    // IMPORTANT: If this app is deployed as Static Web App and we
    // want server-side resources to have access to the real bearer token
    // for the authenticated user we need to store the authorization bearer token in
    // our own custom request header because a proxy may be placed in between
    // this client app and the back-end Function App API (depending on setup) and the
    // proxy injects/overrides the current authorization header with own bearer token.
    const authHeader = req.headers.get('Authorization');
    if (authHeader) {

      // Clone the request and set the custom header
      const customReq = req.clone({
        setHeaders: {
          'X-Custom-Authorization': authHeader,
        },
      });
    }

    // If no authorization header is found, pass the original request
    return next.handle(req);
  }
}

Also, wanted to make sure that based on your code sample posted. Have you tried this authority instead?
https://login.microsoftonline.com/common/v2.0 <-- if your App Registration is setup for v2 tokens
or
https://login.microsoftonline.com/common
I just went through this setup because my app is setup a Multitenant in the App Registration (I use /organizations though because I'm not supporting MS Accounts). Here is my latest Issue I just got working where I needed to support multiple tenants. #68

Hello,
I see that the access token sent is different and a proxy seems to be applied. My access token still doesn't contain the roles in the claims, I think I've got a scope or permission problem instead.

Hi @Tweentyy, do you have an app registration for each application or are you using the same app registration for both? Based on your comments above I imagine you have different registrations due to the fact that the id token is the one carrying roles. Have you configured the registration for functions to return roles for users? Can you elaborate a little more on your app registration setup in EntraID. The ideal situation for functions as they are seen as backend APIs is to consume access tokens.

Hello,
From what I can see, I have an App Registrations that manages all the roles. For example, I have the "Test.Admin" role in App roles. It is used as SSO.
On the App Registrations side, which manages the frontend, there is no special configuration.
Lastly, Azure Functions seem to be a simple Resource with no specific configuration required.
On my side, on the frontend with the MSAL Angular module, the user logs in on the SSO side (I get the roles from the ID token but not the access token).
In the Azure Functions code, I make the connection with the SSO as shown above in the code extract
I read in the Microsoft documentation that the Apps had the roles in the ID token whereas the APIs had them in the Access Token.
When I send the token, I send an access token of type "App", so it doesn't contain the roles because they are in the ID token.

Hello @artmasa,
I was able to solve my problem, in fact all I had to do was create a scope on my App Registration SSO and associate the permission with it in API Permission and everything worked :)

Glad to hear @Tweentyy!

Glad to hear @Tweentyy!

I have one last question. I want to create 2 policies, one that checks if the user have role A and another that checks if I have role B.
I've created them, no problem.
On an Authorize, is it possible to check if one of the two policies is true?

Okay thanks!


Shared team database between csirt and www

Each team member has a page
This page now shows the pasts on www.divd.nl and csirt.divd.nl
Reasearchers are also credited

@Lennaert89 Wat denk jij hiervan?


How to extract camera position and rotation relative to model?

My goal is to set camera in my unity scene so it would see the model like camera sees the person in video.
I was able to extract camera values for each frame.
In the article was said that it is weak-perspective camera Π = [s, tx, ty], but I'm konfused how to use those values to set my camera position and rotation in scene.

In the weak perspective camera model, scale = focal_length / Z_avg.
Thus, you can approximate the location of the person in the 3D scene by translating the vertices by [tx, ty, f/s]. In practice, I've found that f=1 works reasonably well.
Best,
Jason

Sorry for respondending a closed issue.
But I want to ask what's the units of tx, ty? Is it pixel or meter?


Preventing text files from being saved

I have not figured yet how to prevent the text files from being saved along the images. Maybe we could have a "--no-text-file" function as well?
Thanks

117124d adds the option --no-captions which prevents Instaloader from saving the caption text files. However, it is still in the experimental branch v3-dev.
$ pip3 install --upgrade git+https://github.com/Thammus/instaloader@v3-dev
$ instaloader --no-captions ...

I personally do not consider this an important feature. On modern filesystems, the caption files usually need 0 bytes of space, since most of them can be inlined into their inode. Furthermore, in case they are absolutely not desired, a simple rm *.txt would match and delete them.
Happy instaloading!
Alex

In windows, I have been searching and deleting 70,000 text files a day takes a lot of resources and a few minutes. Thanks for implementing.


grammar scope name for cpp is 'cpp' (at least on new atom versions)

this fixes the issue i previously reported yesterday!

thanks for your contribution.


CNV input format

Dear Prof. Li,
I got some error message when simulating CNV use VarBen, could you please help me ?
the command:
python VarBen-master/bin/sveditor.py --svfile mut.list --bamfile mybam.bam --reffasta hg19.fasta --readlength 150 -o varben_out --alignerIndex hg19.fasta --seqer illumina --aligner bwa
the input sv file:
chr1    110883028       110884090       cnv     2.5     gain
chr1    16258586        16259761        cnv     2       loss
and I got these error:
['chr1', '110883028', '110884090', 'cnv', '2.5', 'gain']
['chr1', '16258586', '16259761', 'cnv', '2', 'loss']
Traceback (most recent call last):   File "/mnt/cfs/project/test_freshman/liaoxinhui/3.20190624_bio_performance/software/VarBen-master/bin/../varben/deal_sv/checkSVInput.py", line 40, in check_sv     raise Exception("cnv is not in correct format") Exception: cnv is not in correct format
('cnv is not in correct format',) ('chr1\t16258586\t16259761\tcnv\t2\tloss', 'cnv is not in correct format')
It seems that the 'AF' column in input file is uncorrected, so how to set 'AF' column ? Was the 'AF' column mean copies of gain/loss ?
Thanks a lot!
Best,
Gerde Liao

Hi Gerde,
As shown in the source code, https://github.com/nccl-jmli/VarBen/blob/master/varben/deal_sv/checkSVInput.py#L36
The copy number of CNV for loss type is less or equal to 1.
Thank you for contacting us, we will fix the wrong description in the README file.

Hi lijia,
  Thanks for your reply !
  So the 'AF' represent the 'copy ratio', am I right ?
  And I have another question here, what's the difference between 'dup/del' and 'gain/loss' in SV simualtion ?
  Thanks much!

Best,
Gerde

Hi Gerde,
Yes, 'AF' represents to 'copy ratio'.
At here, we introduce a type of simple CNV, when modifying the BAM file, we don't create any 'breakpoint' at the start and end site. That means we just add or reduce the read depth of the target region.
Best,
Lijia

Hi lijia,
Thanks a lot! It is much more clear for me now.
VarBen is an efficiency variant simulation tool, thanks for your developments.
I will close this issue.
Best,
Gerde


Add setter configurable only fior benchmarking or tests

This PR adds a setter for BlockNumberProvider if compiling with runtime-benchmarks features. This allows to write benchmarking that is independent to the BlockProvider used, as it allows to set the desired mocked block number.
This is a problem we have encountered in https://github.com/PureStake/crowdloan-rewards/pull/44 (cc @JoshOrndorff), where we want to write benchmark code that is agnostic of the block provider. But to benchmark correctly one of the extrinsics, we need to specifically set the block number in the block provider.

Our crowdloan rewards pallet which @girazoki linked is responsible for paying tokens to a fixed set of users (crowdloan participants in our case). The tokens are not paid all at once, rather there is a linear vesting schedule. The pallet is generic over what block number is used for the vesting schedule. Concretely we imagine users either vesting according to their own chains native block number, or the relay chain's block number.
The worst-case weight for claiming a reward happens when the block number used for vesting is non-zero. (Because if it is zero, the amount due is zero and no transfer is actually made.) In our mock runtime we use the native block number from frame_system and it is trivial to set the block number with the set_block_number helper method. This same practice is well precedented throughout frame benchmarks. For example here.
In the Moonbeam runtime, we use the relay chain block number for vesting. We would like our benchmarks to be re-useable over both block number configurations, thus we need a way to set the block number through the trait.

In many ways, this is a follow up to https://github.com/paritytech/substrate/pull/9209


Added NOT operators to MongoLite's $in and $has

I added $nin for 'not in' and $nhas for 'not has'.

Thanks! Could you also please create a pull request for the main MongoLite repo here: https://github.com/aheinze/MongoLite


Error during DOI resolution

local variable 'extent' referenced before assignment

Caused by https://github.com/ietf-ribose/bibxml-service/pull/215.

Fixed in https://github.com/ietf-ribose/bibxml-service/commit/f04f6c97eea7f1d231542ae694d0359e9e5889e4.
@stefanomunarini note that error. The code in PR was likely not tested, make sure to do that (in this case, just requesting a DOI was enough to get the error).


Fix saas PR check

Add force to enable rewriting credentials file.

+1. As we are cloning git into subdirectories and thus only those subdirectories are wiped (by git plugin), the jenkins-env.json file remains from old builds in the root of the workspace. Adding --force should fix this issue.


Terminal command decorations: When I move the terminal to a side panel, the previous decorations stop working.

Testing #143433

Run command a decoration is created, when you click it show the options ro run again or copy the output.
Move terminal to a side panel.
Move back to the previous panel.
Clicking on the decoration now doesn't show anything, or do anything.

I believe this is now fixed in the latest insiders. Can you pls confirm?


Minor discrepancy in 'discoverable' option default value of register()

I want to express my appreciation for your ongoing work on this library. It's been incredibly helpful, and you've been very responsive. I'm sorry to keep bringing up minor issues, I am migrating to V2 and run into things.
I've noticed a small inconsistency between the documentation and the implementation of the 'discoverable' option in the register() function:

The documentation states the default value is 'preferred'.
The code doesn't set a default value, which could potentially lead to undefined behavior.

Current implementation in client.ts:
authenticatorSelection: {
    userVerification: options.userVerification,
    authenticatorAttachment: getAuthAttachment(options.hints),
    residentKey: options.discoverable,
    requireResidentKey: (options.discoverable === 'required')
}

Suggestion for alignment (if this matches the intended behavior):
authenticatorSelection: {
    userVerification: options.userVerification,
    authenticatorAttachment: getAuthAttachment(options.hints),
    residentKey: options.discoverable ?? 'preferred', // add default fallback
    requireResidentKey: (options.discoverable === 'required')
}

Alternatively, the documentation could be updated if the current implementation is intentional.

I'm sorry, but just want confirm this because I have different behavior with the passkey ceremony when I set discoverable: "preferred" explicitly rather than omitting it and relying on it to default,
MDN

If omitted, residentKey defaults to "required" if requireResidentKey is true, otherwise the default value is "discouraged".

Based on the implementation, if discoverable is omitted, wouldn't it default to "discouraged"?

Reopening for verification


The text embedding of the CLIP should be pooled (The shape should be 1*768)

The text embedding of the CLIP should be pooled, but the code is now not pooled, the shape of it is still 77*768？

conditioner.py:
self.output_key = "pooler_output" if self.is_clip else "last_hidden_state"


[JavsScript ] Required Input validation not working properly on "Input Form with Labels" sample

Platform
What platform is your issue or question related to? (Delete other platforms).

[X] JavaScript

Details
http://adaptivecards-ci.azurewebsites.net/samples/InputFormWithLabels.html
Loaded this card on the website, and the Name and Email fields are marked required, but input validation did not fire. Possibly because they are created with data binding expressions?

@matthidinger the regex seems to be busted for that sample for starters; I was just on it as well!

Here's the fixes to the regex for basic email + name formats:


Include  sympy.stats?

Would it be possible to extend support to cover https://docs.sympy.org/latest/modules/stats.html/?
SymPy.jl is fantastic! Many thanks!

You can already access it with a command like:
julia> SymPy.PyCall.pyimport_conda("sympy.stats", "sympy")

Then you can access the functions with the sympy.stats prefix:
julia> Z = sympy.stats.Normal("Z", 0, 2)
Z

julia> sympy.stats.E(Z)
0

If you wanted to make  PR where a module gives more direct access the function names, that would be helpful.

Many thanks this is perfect! It's signifigantly above my Julia grade to submit such a PR, but for the archives might note that the below worked perfectly for me.
I was mucked up on the syntax on how to call the SymPy module.
using SymPy, PyCall;
SymPy.PyCall.pyimport_conda("sympy.stats", "sympy")
E(x) = sympy.stats.E((x))
sample(x) = sympy.stats.sample(x)
Normal(x,y,z) = sympy.stats.Normal(x,y,z)
Z = Normal("Z", 0, 2)
E(Z)

Again, sincere thanks!

Any plan to add this in SymPy.jl by default?

It depends on what you mean by this.
We could include just the lines
using SymPy, PyCall;
SymPy.PyCall.pyimport_conda("sympy.stats", "sympy")

easily enough.
The import_from could be added to automatically export the methods. But I wouldn't make that a default as it enlarges the namespace too much. We could make a module in SymPy that would basically include these lines:
module Stats

SymPy.PyCall.pyimport_conda("sympy.stats", "sympy")
import_from(sympy.stats)
end

Then users could opt in if they chose through using SymPy.Stats. Adding documentation would be nice as some things don't quite work (e.g. if Z is an RV, the Z>6 isn't going to work, as in SymPy.jl that returns a boolean, but Gt(Z,6) does.
If you wanted to make a PR, I'd be quite willing to merge something like that in.


[SPARK-2945][YARN][Doc]add doc for spark.executor.instances

https://issues.apache.org/jira/browse/SPARK-2945
spark.executor.instances works. As this JIRA recommended, we should add docs for this common config.

Test build #26652 has started for   PR 4335 at commit 46c40c4.

This patch merges cleanly.

Test build #26652 has finished for   PR 4335 at commit 46c40c4.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/26652/
Test PASSed.

Sorry for deleting the branch by mistake. The new PR is here: https://github.com/apache/spark/pull/4350


Python: ModuleNotFoundError: No module named 'semantic_kernel.skill', Python, AzureOpenAI

Describe the bug
While trying to run the code for creating basic agents in Python using Semantic Kernel, it is giving ModuleNotFoundError: No module named 'semantic_kernel.skill'. Facing the same error for semantic_kernel.skill_definition as well.
Referred to 4777 but the hyperlink to the "Example" provided in that issue is giving 404 error.
Can you please help me. This is a blocker for me. As per my knowledge, there is no executable code for Agents using Semantic Kernel in Python. Please share the resources if you have any.
To Reproduce : Execute the below code:
import os
import semantic_kernel as sk
from semantic_kernel.connectors.ai.open_ai import AzureTextCompletion
from semantic_kernel.skill import Skill
from semantic_kernel.skill import sk_function
from azure.core.credentials import AzureKeyCredential
from azure.search.documents import SearchClient
from azure.search.documents.models import QueryType

# Initialize the Semantic Kernel
kernel = sk.Kernel()

# Register Azure OpenAI text completion service
azure_text_completion = AzureTextCompletion(
    endpoint=AZURE_OPENAI_ENDPOINT,
    api_key=AZURE_OPENAI_API_KEY,
    deployment_name=DEPLOYMENT_NAME
)
kernel.register_service("text-completion", azure_text_completion)

# Initialize Azure Cognitive Search client
search_client = SearchClient(
    endpoint=AZURE_SEARCH_ENDPOINT,
    index_name=AZURE_SEARCH_INDEX_NAME,
    credential=AzureKeyCredential(AZURE_SEARCH_API_KEY)
)

# Define a simple skill
class GreetingSkill(Skill):
    @sk_function
    def say_hello(self, name: str) -> str:
        return f"Hello, {name}!"

# Define a search skill
class SearchSkill(Skill):
    @sk_function
    def search_documents(self, query: str) -> str:
        results = search_client.search(search_text=query, query_type=QueryType.SIMPLE)
        response = [doc['content'] for doc in results]
        return '\n'.join(response)

# Register the skills with the kernel
greeting_skill = GreetingSkill()
search_skill = SearchSkill()

kernel.register_skill("GreetingSkill", greeting_skill)
kernel.register_skill("SearchSkill", search_skill)

# Define a function to interact with the AI agent
def interact_with_agent(kernel, skill_name, function_name, **kwargs):
    result = kernel.execute(skill_name, function_name, **kwargs)
    return result

# Example interaction with the AI agent
result = interact_with_agent(kernel, "GreetingSkill", "say_hello", name="Alice")
print(result)  # Output: Hello, Alice!

# Example search interaction with the AI agent
search_result = interact_with_agent(kernel, "SearchSkill", "search_documents", query="Azure Cognitive Services")
print(search_result)  # Output: Search results related to "Azure Cognitive Services"

Expected behavior
Error should be resolved.
Screenshots

Platform

OS: [e.g. Windows, Mac] - Windows
IDE: [e.g. Visual Studio, VS Code] - VS Code
Language: [e.g. C#, Python] - Python
Source: [e.g. NuGet package version 0.1.0, pip package version 0.1.0, main branch of repository] - Python Website

@rakeshmothukuru1 please make sure your code and the package are up to date, skills were renamed to plugins a long time ago... The samples in the repo are up to date, will close this issue, please recreate one if you can't get it running!


Notify routing table updates to replication manager and query the latest leader replica set from Routing table snapshot

Notify the routing table updates in Helix dynamic cluster change handler to Replication Manager. On receiving this notification, replication Manager will:

Re-query the latest set of remote leader replicas for the leader partitions on local node from the routing table snapshot and update its internal map.
Assign the new remote leader replicas to replica threads (will be handled in future PRs)
Remove old leader replicas to replica threads (will be handled in future PRs)

Codecov Report

Merging #1518 into master will increase coverage by 0.06%.
The diff coverage is 94.73%.

@@             Coverage Diff              @@
##             master    #1518      +/-   ##
============================================
+ Coverage     73.73%   73.80%   +0.06%
- Complexity     8047     8090      +43
============================================
  Files           564      567       +3
  Lines         44191    44403     +212
  Branches       5578     5615      +37
============================================
+ Hits          32583    32770     +187
- Misses         9969     9990      +21
- Partials       1639     1643       +4

Impacted Files
Coverage Δ
Complexity Δ

...a/com/github/ambry/clustermap/ClusterMapUtils.java
93.06% <0.00%> (-2.32%)
48.00 <0.00> (ø)

...a/com/github/ambry/clustermap/MockPartitionId.java
96.59% <83.33%> (-0.91%)
35.00 <2.00> (+3.00)
:arrow_down:

...m/github/ambry/replication/ReplicationManager.java
92.30% <97.82%> (+1.26%)
18.00 <0.00> (ø)

...hub/ambry/clustermap/ClusterMapChangeListener.java
100.00% <100.00%> (ø)
1.00 <1.00> (?)

.../ambry/clustermap/DynamicClusterChangeHandler.java
91.15% <100.00%> (+0.10%)
63.00 <0.00> (+1.00)

...ub/ambry/network/SocketRequestResponseChannel.java
85.93% <0.00%> (-9.38%)
10.00% <0.00%> (-2.00%)

...java/com/github/ambry/store/CompactionManager.java
89.88% <0.00%> (-2.39%)
25.00% <0.00%> (ø%)

...va/com/github/ambry/replication/ReplicaThread.java
85.47% <0.00%> (-2.20%)
106.00% <0.00%> (-2.00%)

...va/com/github/ambry/rest/RestServiceErrorCode.java
84.61% <0.00%> (-1.88%)
6.00% <0.00%> (ø%)

... and 46 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 257050c...c343a16. Read the comment docs.


[Doc]: some Chinese files do not exist

https://github.com/apache/apisix-website/blob/60244433b27b8374dd8485d809e91bc848f1de3a/website/i18n/zh/docusaurus-plugin-content-blog/2021/10/13/celebrating-300-contributors-of-apisix.md?plain=1#L51
This link https://apisix.apache.org/zh/docs/general/join does not exists.

I found that redirect can still work in the https://deploy-preview-986--apache-apisix.netlify.app/docs/general/community, but there will be an error that the link does not exist when building. So I did not add them in https://github.com/apache/apisix-website/pull/986.
When the website is updated normally, I will observe whether it can redirect and close this issue.

LGTM


Enforce and abide by PoET key block claim limit

Introduces the following changes:

PoET consensus state statistics are now reset whenever a non-PoET block is encountered in the chain and the sealed signup data is removed from the consensus state as that needs to be maintained on a per-PoET public key basis.
A utility class to manage and persist locally on a validator key state for its PoET key.  This includes the sealed signup data associated with it as well as whether or not it has been refreshed because the block claim limit has been reached with that key.
The PoET genesis CLI and block publisher have been changed to use the PoET key state store for sealed signup data.
A utility class to query PoET configuration settings.
The PoET block verifier now enforces the key block claim limit when determining the validity of a block.
The PoET block publisher abides by the key block claim limit when being asked to initialize a block.  If the limit has been reached with its current key, it will fail to initialize the block and if appropriate will create and register new signup information.

To test locally:
cd /project/sawtooth-core/integration/sawtooth_integration/docker
In poet-smoke.yaml, after the line:
-k /etc/sawtooth/keys/validator.wif \
Add:
sawtooth.consensus.algorithm=poet \
sawtooth.poet.target_wait_time=5 \
sawtooth.poet.initial_wait_time=0 \
sawtooth.poet.key_block_claim_limit=2 \

This should result in a bunch of blocks and validators having to refresh after successfully claiming 2 blocks.

Ooops.  Inadvertently screwed something up when I rebased before pushing PR.  Let's try that again.

Tested locally.

Rebasing before merge.


WIP on upgrade testing

IDEA: https://docs.google.com/document/d/1AEjG78Kvgu_sLG4OXSJxy6SXPcaM2IX_DGyn_GBC9hQ/edit?usp=sharing

[x] recording works (ProgramTest runs against the cloud)
[x] full upgrade test works (ProgramTest runs against the cloud, but does stack import from pre-recorded V-prev state)
[x] unit testing works without the cloud

[ ] however replaying Diff methods is not quite accurate and can generate false positives

[ ] hook into CI

My next steps is to add an rds.Instance configuration to confirm this can detect the regression on rds.Instance, re-record and try to port to master branch representing v6.

Closing in favor of https://github.com/pulumi/pulumi-aws/pull/2714 that supersedes this work.


Scala Native 0.5 Support

Do you have any plans to support Scala Native 0.5? Thanks in advance for your time. Cheers, Objektwerks! :)

I do have! But many dependencies are not released yet, so for now, we can't. I think I will also rewrite it in a Cats Effect first way, when now Cats Effect is, somehow a second-class citizen.

That's great news! I'm not a Cats Effect user. So if you can also accommodate us simple folk, that would be great! :) Thanks!

SNUnit 0.10.0 was released for Scala Native 0.5.
Cats Effect support was stopped until they update to Scala Native 0.5 as well.
SNUnit 0.10.1 was just tagged with support for com-lihaoyi/cask.


Update dashboards and grafana

Updated dashboards to be compatible with newer versions of grafana, as well as updated docker-compose with the current latest version.
I am unable to test the metrics that depend on tesla inverters/powerwall plus, but everything else seems to be working as expected

First observations -

The 9.x version menus are different so it will require an update to the README, setup.sh and upgrade.sh instructions for users.

It seems the Power Meter panels have lost their color and font size - I'll need to figure out how to bring this back:

Old:

New:

This is great work @techlover1 !  I'm testing it now and want to run it for a while before we merge this.  We also still need to fix the "Current State" panel. I see you added updates, but it doesn't seem to fix the clipping.  I'm see what I can figure out as well.

Some screenshots...

Powerwall+

I had not yet fixed the current state panel. Turns out it just needed to be a little taller

Nice!  That did fix it!  @mcbirse how does this look for your setup?

Under the Powerwall+ section, there are a few errors:

String Voltage: The "A string" is missing due to a typo on Voltage in the query.
String Current:  Has the wrong unit, it should be ampere (A).
String Power: Has the wrong unit, it should be wattage (W).

Merge complete.  Thank you @techlover1 !


pending: analyze story events

data should be collected from next update onwards, but interface is not implemented yet

minimal console interface already gives good feeling for the data


Template Literal Delimiters do not appear in the token list

Ported from https://code.google.com/p/esprima/issues/detail?id=632
What steps will reproduce the problem?

Go here: http://esprima.googlecode.com/git-history/harmony/demo/parse.html?code=var foo %3D [`bar %24{baz}`]%3B
Click on the Tokens tab of the output

What is the expected output?
To see the ` token in the list of tokens.
What do you see instead?
They were not present.
What version of the product are you using? On what operating system? Which
browser?
the harmony branch

Fixed by 5a99bb005322407e54142932fea0c868d70fe743


Fixed issue #9194

Issue Reference
Fixed issue #9194

@shivendra-webkul
Checked and Found that the issue is not fixed yet, status is not changing to Inactive. Please check the video for reference.
Video
https://github.com/bagisto/bagisto/assets/92721837/23775bf3-d342-44d7-aba1-2a28fcbc9633

@shivendra-webkul
Checked and Found that the issue has been fixed and working fine. Please check the video for reference.
Video
https://github.com/bagisto/bagisto/assets/92721837/f37d6e1f-9cd4-4e61-9c87-9b885ec2c2ff


Updating README.md

There is a scope of improvement  in the README.md file and i would like to update it.
For Eg:  "please see the xmonad website" can be replaced with "please refer to the xmonad website" .
Such changes would make the file more convenient and edged to the readers and contributors.
i would like to contribute to its betterment so please can you assign this issue to me.

On Tue, Oct 05 2021 03:47, Shubham Swapneel Shah wrote:

For Eg:  "please see the xmonad website" can be replaced with "please
refer to the xmonad website" .

Would that really be an improvement?  This seems like a "mere"
reformulation, which doesn't necessarily add value.  I certainly agree
that the README could use an overhaul, but I'm not sure that that would
be it.

You could open a PR at anytime, especially if the changes aren't that big.
Regarding the README, I think it's better to keep the contrib README small and refer to the README of the core repository, so I'd suggest to make the improvements there


Property 'nav' does not exist on type 'AnyPage'.

Short description of the problem:
Error: Property 'nav' does not exist on type 'AnyPage'.
What behavior are you expecting?
How can I get rid of this error on all code editors and IDEs as well as on compile and ionic serve?
Steps to reproduce:

just generate a page
run ionic serve
you will see the error of the nav but the nav is working

ERROR in [default] /.../app/pages/home-page/home-page.ts:15:9
Property 'nav' does not exist on type 'HomePage'.

Problem with generated code:
constructor(@Inject(NavController) nav) {
    this.nav = nav;//this line throw errors on all Editors and IDEs and on Terminal
 }

Ionic Version: 2.0.0-beta.2
Browser & Operating System: iOS / Android / Chrome
Terminal
Run ionic info from terminal/cmd prompt:
Cordova CLI: 6.0.0
Ionic Version: 2.0.0-beta.2
Ionic CLI Version: 2.0.0-beta.19
Ionic App Lib Version: 2.0.0-beta.9
ios-deploy version: 1.8.5
ios-sim version: 5.0.6
OS: Mac OS X El Capitan
Node Version: v4.3.2
Xcode version: Xcode 7.2.1 Build version 7C1002

I think what you're seeing is a warning from the linter. If so you have a few options, amongst them you could just write:
constructor(private nav : NavController) {}

this way "this.nav" will be available in the class automagically (it creates a field under the hood).

same thing, I have tried it and the error still occur tho

I have added the following outside of the constructor and above then it fixed the problem:
private nav:NavController = null;

would this work?
`
import {Page, NavController} from 'ionic-angular';
import {Page1} from '../page1/page1';
@Page({
templateUrl: 'build/pages/tabs/tabs.html'
})
export class TabsPage {
constructor() {
console.log(NavController)
this.nav = NavController;
this.tab1Root = Page1;
}
}
`

would this work? without having the "navController" in the constructor function..
`
import {Page, NavController} from 'ionic-angular';
import {Page1} from '../page1/page1';
@Page({
templateUrl: 'build/pages/tabs/tabs.html'
})
export class TabsPage {
constructor() {
this.nav = NavController;
console.log(NavController)
this.tab1Root = Page1;
}
}
`

The way to get rid of the errors on Atom or other IDEs is to declare the nav variable:
export class YourPage {

  private nav:NavController = null; // please add this line as default generated page plus the --ts option on command line.

  static get parameters() {
    return [[NavController]];
  }

  constructor(nav:NavController) {
    this.nav = nav;
  }

Hello! As it looks like you found a solution to this issue I will be closing this issue, thanks again!


Using ERC20Upgradeable instead of staticcall

Using staticcall:
·-------------------------------------------------|---------------------------|--------------|-----------------------------·
|              Solc version: 0.8.18               ·  Optimizer enabled: true  ·  Runs: 1000  ·  Block limit: 30000000 gas  │
··················································|···························|··············|······························
|  Methods                                                                                │
·····························|····················|·············|·············|··············|···············|··············
|  Contract                  ·  Method            ·  Min        ·  Max        ·  Avg         ·  # calls      ·  usd (avg)  │
·····························|····················|·············|·············|··············|···············|··············
|  Deployments                                    ·                                          ·  % of limit   ·             │
··················································|·············|·············|··············|···············|··············
|  StableAsset                                    ·          -  ·          -  ·     5056027  ·       16.9 %  ·          -  │
··················································|·············|·············|··············|···············|··············

Using ERC20Upgradeable:
·-------------------------------------------------|---------------------------|--------------|-----------------------------·
|              Solc version: 0.8.18               ·  Optimizer enabled: true  ·  Runs: 1000  ·  Block limit: 30000000 gas  │
··················································|···························|··············|······························
|  Methods                                                                                │
·····························|····················|·············|·············|··············|···············|··············
|  Contract                  ·  Method            ·  Min        ·  Max        ·  Avg         ·  # calls      ·  usd (avg)  │
·····························|····················|·············|·············|··············|···············|··············
|  Deployments                                    ·                                          ·  % of limit   ·             │
··················································|·············|·············|··············|···············|··············
|  StableAsset                                    ·          -  ·          -  ·     5052873  ·       16.8 %  ·          -  │
··················································|·············|·············|··············|···············|··············

Codecov Report

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

:exclamation: Current head 18625a5 differs from pull request most recent head df4b170. Consider uploading reports for the commit df4b170 to get more accurate results

: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.
@@          Coverage Diff           @@
##             main     #10   +/-   ##
======================================
  Coverage        ?   0.56%
======================================
  Files           ?      11
  Lines           ?    1056
  Branches        ?     259
======================================
  Hits            ?       6
  Misses          ?    1049
  Partials        ?       1

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


Use CNFS instead of SPIFFS

I would totally understand if this is a

Turns out SPIFFS loads were slowed down by USB for some reason. This is no longer critical this year, but I am in favor of moving to this system next year.

I won't delete the branch.


Use the correct document_id when indexing docs

Currently this indexer assumes that a document's _id and link are the same. This causes tag updates for the contacts app to fail because they aren't the same (the _id has no leading slash).
Best reviewed per-commit, as some refactoring was necessary to prevent this from happening again.
https://trello.com/c/lRnxI2x5 https://trello.com/c/afa0Mfa4

Would it be better to fix this by making the contacts app send data with a consistent value for the link and _id?  (Following the convention is for other apps, which is to make the _id and link both have a leading slash.)
In order to migrate to this, you could make rummager add a leading slash to any supplied document with an "_id" field with no leading slash (unless it started with http: or https:) when indexing; then the nightly index migration would update all the _id fields to have a leading slash, and you could update contacts accordingly at a later deploy.
It's probably best to try and use the _id field as the unique identifier, since that's the only reason we have it!

The reason I don't want to fix this in contacts-admin is that I think the migration process might be error prone, and we done gain that much. If we start indexing from the message queue we will remove the indexing code from contacts-admin anyway, removing the problem entirely. We'll probably use content_id for the new ids there.

It looks good to me given your concerns and explanation.

Minor comment, but happy to merge.


Now the bootstrap server send end-of-boostrap message

Following last spec revision, when the bootstrap is done we send a POST /bs to the device

integrated in master (commits e353e9dd8f6333e2f893a118bb22cf3b0636a535 and e1bca39b313719c7d260fd15a8c9e9623c93c693)


Add robots.txt for SEO

Goes in the static directory.

https://github.com/rancher/rancher-docs/blob/main/static/robots.txt can be used as reference.


[Serverless Component] - Deploy fails when Lambda configured with a VPC

Describe the bug
When a Lambda has been added to a VPC, the deployment fails.
To Reproduce
Steps to reproduce the behavior:

Deploy a Next.js app with serverless-next.js component
Open the AWS console, and manually attach a VPC to one of the new Lambdas
Re-deploy
See error:

error:
  InvalidLambdaFunctionAssociation: The function cannot have a VPC configuration. Function:[my lambda arn]
    at Request.extractError (/Users/data/.serverless/components/registry/npm/serverless-next.js@1.3.0/node_modules/aws-sdk/lib/protocol/rest_xml.js:53:29)
    at Request.callListeners (/Users/data/.serverless/components/registry/npm/serverless-next.js@1.3.0/node_modules/aws-sdk/lib/sequential_executor.js:106:20)
    at Request.emit (/Users/data/.serverless/components/registry/npm/serverless-next.js@1.3.0/node_modules/aws-sdk/lib/sequential_executor.js:78:10)
    at Request.emit (/Users/data/.serverless/components/registry/npm/serverless-next.js@1.3.0/node_modules/aws-sdk/lib/request.js:683:14)
    at Request.transition (/Users/data/.serverless/components/registry/npm/serverless-next.js@1.3.0/node_modules/aws-sdk/lib/request.js:22:10)
    at AcceptorStateMachine.runTo (/Users/data/.serverless/components/registry/npm/serverless-next.js@1.3.0/node_modules/aws-sdk/lib/state_machine.js:14:12)
    at /Users/data/.serverless/components/registry/npm/serverless-next.js@1.3.0/node_modules/aws-sdk/lib/state_machine.js:26:10
    at Request.<anonymous> (/Users/data/.serverless/components/registry/npm/serverless-next.js@1.3.0/node_modules/aws-sdk/lib/request.js:38:9)
    at Request.<anonymous> (/Users/data/.serverless/components/registry/npm/serverless-next.js@1.3.0/node_modules/aws-sdk/lib/request.js:685:12)
    at Request.callListeners (/Users/data/.serverless/components/registry/npm/serverless-next.js@1.3.0/node_modules/aws-sdk/lib/sequential_executor.js:116:18) {
  message: 'The function cannot have a VPC configuration. Function: ' +
    '[my lambda arn]',
  code: 'InvalidLambdaFunctionAssociation',
  time: 2019-10-05T16:31:19.994Z,
  statusCode: 400,
  retryable: false,
  retryDelay: 38.413945280304794

Desktop (please complete the following information):

OS: macOS 10.14
Version<EMAIL_ADDRESS>
Expected behavior
I'm not sure - I'm guessing this is an issue with serverless or aws-sdk, but I wanted to start here, in case there was anything I could do. Perhaps this library could support specifying a VPC to deploy the Lambdas into?

@dtkelch As you probably suspected this is a limitation on AWS itself, not the next component. Lambda@Edge doesn't support associating with a VPC.
A VPC lives in a region, Lambda@Edge is distributed globally. Having said that AWS has been making progress on interconnecting VPCs globally so wouldn't be surprised if what you need is supported in the future.

@danielcondemarin Ah, I didn't realize that was the case - I've never used the Lamba@Edge pattern, but your explanation makes perfect sense.
Thank you!


chore: remove remnants of C++ format

We can remove pp_expr as well after removing the old compiler


compile error

Describe your problem here.
source/Main.hx:77: characters 19-29 : Missing ;

Are you modding a build from source or with Lua?
Source
What is your build target?
Windows x64
Did you edit anything in this build? If so, mention or summarize your changes.
no

add ;

Try this:
if (zoom == -1) {
    var ratioX:Int = gameWidth;
    var ratioY:Int = gameHeight;
    zoom = Math.min(ratioX, ratioY);
    gameWidth = Math.ceil(stageWidth / zoom);
    gameHeight = Math.ceil(stageHeight / zoom);
}

Obviously I don't know what the rest of your code is like and this is just a guess based on the little bit you provided.

that made it worse

Try this:
if (zoom == -1) {
    var ratioX:Int = gameWidth;
    var ratioY:Int = gameHeight;
    zoom = Math.min(ratioX, ratioY);
    gameWidth = Math.ceil(stageWidth / zoom);
    gameHeight = Math.ceil(stageHeight / zoom);
}

Obviously I don't know what the rest of your code is like and this is just a guess based on the little bit you provided.

that made it worse

well as stated, I don't know what the file looks like, so I just did a guess

https://cdn.discordapp.com/attachments/1128181185546297366/1128181289577611396/Main.hx

Our code (mine is slightly different due to the way the variables are set up) is actually very similar, I can't tell what the issue is.

Left is my code, right is yours


APIError(code=-1013): Filter failure: MIN_NOTIONAL

Hello, running the bot and when it finally identified a volatile coin and proceeded to buy, the code failed with:
APIError(code=-1013): Filter failure: MIN_NOTIONAL
I'm using the latest code (as of 5/26/2021) and using Binance.US (AMERICAN_USER = True).
Below is some of the log entries when the issue occurred:
[2021-05-26 10:10:39] Working...Session profit:0.00% Est:$0.00
[2021-05-26 10:10:39] OXTUSDT has gained 5.466% within the last 3 minutes, calculating volume in USDT
[2021-05-26 10:10:39] Preparing to buy 19.67 OXTUSDT
[2021-05-26 10:10:39] APIError(code=-1013): Filter failure: MIN_NOTIONAL
[2021-05-26 10:10:57] Working...Session profit:0.00% Est:$0.00

Read the faq and wiki, this has been answered x times before.


Do not reset correctly set _vectorSize to an incorrect value.

@toregge PR

Fixes a regression introduced in #29845


Use interface lambda hinting in WHAM DF cleaning

The problems in #549 were two-fold: both were sampling, but for different reasons. First, sometimes interfaces didn't overlap because no path from the inner interface crossed the outer interface. Second, if 100% of paths in an outer interface went well past the interface boundary, the automatic identification of the interface boundary failed.
Both can be solved by more sampling. The second one, however, could also use the new InterfaceSet object to hint what the interface values are.
That's what this code implements. If the interface lambdas are defined in the interface set, then we use that information. If not, we fall back on the old approach.
This should help tests pass some, although the first problem remains.

Ready for review!

I guess we will need to discuss the actual creation of the table used for WHAM. It seems very complicated somehow.
But this looks good, so I will merge it now.


Browser window inconsistent state with respect to FullScreen API

Just a bit of background to set the stage: I use a webview in my app to embed YouTube videos.
I compiled Electron from master on the 3rd of June and the fullscreen button in the embedded YouTube player was working very well without any extra code on my part.
Now that 0.27.3 has shipped the fullscreen button in the embedded player no longer works. I think it may be because of this commit: https://github.com/atom/electron/commit/8f9a109fa67356e6d1e9329ab573b6c6d3a5b052
Are app writers supposed to do something special to allow fullscreen now?

I'm a dork! It would have helped if I just took 10 minutes to read through the patch and then look at the docs for webview. I'd have noticed enter-html-full-screen, leave-html-full-screen. I added some handlers to my webview and then call setFullScreen accordingly.

There is some foo going on with respect to leaving fullscreen. I'm not going to reopen this just yet but leaving fullscreen while actually leaving fullscreen leaves the YouTube player in a state that it still thinks it's in fullscreen. This still may be an issue with the patch mentioned above because it was working prior to the code changes.

I'm reopening this because I think there is still something wrong with fullscreen API changes that happened in https://github.com/atom/electron/commit/8f9a109fa67356e6d1e9329ab573b6c6d3a5b052. I'll provide an example before the night is over.

Okay here is a small example that demonstrates the browser window inconsistent state after entering and leaving fullscreen.
When running this click the fullscreen button on the embedded YouTube player. You'll go into fullscreen as you'd expect. Click the fullscreen button again and you'll leave fullscreen. Try clicking the fullscreen button again the fullscreen events have stopped propagating.
One thing to keep in mind is that this worked as expected just a few days ago when I built from master with absolutely no extra code, "It Just Worked(tm)" out of the box. This does not work in 0.27.2.
The main.js and package.json are standard text book, nothing special so I won't post them here.
<!DOCTYPE html>
<html>

  <head>
    <title>Webview Fullscreen Test</title>
    <style>
      body {
        margin:0;
        overflow: hidden;
      }
      #webview {
        position:absolute;
        width: 100%;
        height: 100%;
        overflow: hidden;
      }
    </style>
  </head>

  <body>
    <webview id="webview" src="http://www.youtube.com/embed/GxfwZMdSt4w"></webview>

    <script type="text/javascript">
      var remote = require('remote');
      var browser = remote.getCurrentWindow();
      browser.openDevTools();

      var webview = document.getElementById("webview");
      webview.addEventListener("enter-html-full-screen", function() {
        browser.setFullScreen(true);
      });
      webview.addEventListener("leave-html-full-screen", function() {
        browser.setFullScreen(false);
      });
    </script>
  </body>

</html>

I compiled from master to test this out and it works very well. Thank you!


packet: support creating a builder from an existing packet

Codecov Report
Attention: 12 lines in your changes are missing coverage. Please review.

Comparison is base (54a7016) 88.76% compared to head (2713f81) 85.76%.

Files
Patch %
Lines

src/builder.rs
0.00%
12 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main       #8      +/-   ##
==========================================
- Coverage   88.76%   85.76%   -3.01%
==========================================
  Files           2        2
  Lines         276      302      +26
==========================================
+ Hits          245      259      +14
- Misses         31       43      +12

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


Use jobset and replicated job names as selectors for headless svc

During manual testing with a distributed pytorch training workload I discovered a bug (pods could not reach each other over the network via hostnames). Upon investigation I realize the generated subdomain name assigned to pods and name of the headless service selecting those pods did not match.
In addition, in order for this headless service to select pods from all jobs within a replicated job (as was the goal of #42) we need to update the service's label selectors to use jobset name and replicated job name labels, rather than just a single job-name label.

/label tide/merge-method-squash


Add color to the instructions after install.

Verification

[X] This issue's title and/or description do not reference a single formula e.g. brew install wget. If they do, open an issue at https://github.com/Homebrew/homebrew-core/issues/new/choose instead.

Provide a detailed description of the proposed feature
The current instructions are look like this. It's hard to recognize post instructions, because it's color is same as aboves.

If instruction has color, it's easy to recognize.

What is the motivation for the feature?
Sometime miss the instructions. So maybe it has color it will be more recognizable.
How will the feature be relevant to at least 90% of Homebrew users?
Most of user see the colored text, so it can be helpful.
What alternatives to the feature have been considered?
Automatically setup instructions. But homebrew consider many environments, so it is hard to make it. (probably)

so it is hard to make it. (probably)

This would be relatively easy for autogenerated instructions but much harder for updating all those inside formulae themselves.
I'm also unsure whether colour is the best fit here compared to e.g. bold/underline/whitespace changes.
@Homebrew/maintainers thoughts on this?

Personally I think that the instructions are the important part, not the commands themselves like they are emphasised here. I do think that colouring the output can be useful.

Thanks for the share context. I was thinking of adding colors, but the comments make me think there are other ways to do it.

Add bold

Maybe cause problem ANSI color codes.

Add emoji

Given that I mostly see colors, bolds, and emoji characters in the output, I'm guessing that if we can't use colors and bolds (the ANSI color codes), we can use emoji, but I'm not sure how much it will catch people's eyes.
e.g.
🚧 If you need to have ... first in your PATH, run:

Something like colouring would be nice, we are pretty much used to reading bland lines of code and personally I don't mind it, but considering accessibility, it's always nice to have a cherry on the cake.
Coloring according to WCAG standards would be definitely be a welcomed step.

Given that I mostly see colors, bolds, and emoji characters in the output, I'm guessing that if we can't use colors and bolds (the ANSI color codes), we can use emoji, but I'm not sure how much it will catch people's eyes.

Colors and bolding are fine. I don't feel strongly about it as long as we have clear guidelines. Even better if we could move the formatting logic outside of the cask/formula, as ANSI escape codes are difficult to read and would make our jobs more difficult.
However, I am strongly against the usage of emojis in any stanzas, unless we make them respect HOMEBREW_NO_EMOJI.

We seem to have no real agreement of what/how this should be done so: passing on this, sorry.
Will review PRs from anyone but a note of warning that they may not be merged and will require further discussion.


Links in the Hub are broken

Although metadata is correctly displayed in the Hub, opening links does not work as expected. For example I clicked on the URL link of the SQS Trigger at https://developer.fermyon.com/hub/preview/plugin_sqs_trigger
Instead of opening the link directly, the following is rendered

Environment
os: latest stable MacOS
browsers:

latest Google Chrome
latest Brave
latest Safari

This applies to "URL" and "Submitted By"


No response recieved from ticker

hi,
I am trying to implement the ticker (websocket client) but I am getting no response from the ticker.
(this might be due to the fact that I am trying this on a weekend :))
My code snippet:
function startWebSocket (api_key, user_id, public_token){
		console.log("-----inside function startWebSocket------");
		check(api_key,String);
		check(user_id, String);
		check(public_token, String);
		let KiteTicker = require("kiteconnect").KiteTicker;
		let ticker = new KiteTicker(api_key, user_id, public_token);
		console.log("-----Getting ticker started ------");
		console.log(ticker);
		console.log("-----starting the Ticker ------");
		ticker.connect();
		console.log("Ticker connected: "+ticker.connected());
		ticker.on("tick", logTick);
		ticker.on("connect", onTickerSubscribe);
		ticker.on("unsubscribe", onTickerUnSubscribe);
		ticker.on("disconnect", onTickerDisconnet);
		function logTick(ticks) {
			console.log("-----recieved quote ------");
			console.log("Ticks", ticks);
		}

		function onTickerSubscribe() {
			console.log("Ticker connected: "+ticker.connected());
			console.log("-----Subscribing to WS ------");
			let items = [738561,11654402,884737];
			ticker.subscribe(items);
			ticker.setMode(ticker.modeLTP, items);
		}

		function onTickerUnSubscribe(){
			console.log("-----Ticker Unsubscribed ------");
			console.log("Ticker connected: "+ticker.connected());
		}
		function onTickerDisconnet () {
			console.log("-----Ticker Disconnected ------");
			console.log("Ticker connected: "+ticker.connected());
		}
	}

The logs from the server:
I20170218-07:42:01.999(5.5)? ------REcieved the response from Zerodha server--------
I20170218-07:42:02.005(5.5)? -----inside function startWebSocket------
I20170218-07:42:02.007(5.5)? -----Getting ticker started ------
I20170218-07:42:02.019(5.5)? { modeFull: 'full',
I20170218-07:42:02.019(5.5)?   modeQuote: 'quote',
I20170218-07:42:02.020(5.5)?   modeLTP: 'ltp',
I20170218-07:42:02.021(5.5)?   autoReconnect: [Function],
I20170218-07:42:02.021(5.5)?   connect: [Function],
I20170218-07:42:02.022(5.5)?   disconnect: [Function],
I20170218-07:42:02.022(5.5)?   connected: [Function],
I20170218-07:42:02.022(5.5)?   on: [Function],
I20170218-07:42:02.023(5.5)?   subscribe: [Function],
I20170218-07:42:02.024(5.5)?   unsubscribe: [Function],
I20170218-07:42:02.025(5.5)?   setMode: [Function] }
I20170218-07:42:02.027(5.5)? -----starting the Ticker ------
I20170218-07:42:02.028(5.5)? Ticker connected: false
I20170218-07:42:03.174(5.5)? Ticker connected: true
I20170218-07:42:03.175(5.5)? -----Subscribing to WS ------
I20170218-07:42:33.173(5.5)? -----Ticker Disconnected ------
I20170218-07:42:33.174(5.5)? Ticker connected: false
I20170218-07:42:38.199(5.5)? -----Ticker Disconnected ------
I20170218-07:42:38.200(5.5)? Ticker connected: false

Will appreciate any help.
REgards
ARsh

I have tested this several time and couldn't reproduce the issue. You might have tried when we flushed our ticker data in backend on weekend. Please try again on a normal market day and you should be able to get ticks.


Use OSCM_ prefix for groups

When creating and geting a group from the Identity server, the OSCM_ prefix should be used to identitfy the OSCM related groups on the OIDC provider.

done with fix https://github.com/servicecatalog/oscm-commons/pull/23


[Question] What is the difference between FlashAttention and Memory Efficient Attention in xformers?

The paper I recently read, SELF-ATTENTION DOES NOT NEED O(n2) MEMORY, uses kernel fusion to overcome large GPU memory usage (to be more specifically, it avoids keeping the huge activation values of S and P during forward).
So I want to know if this is very similar to what you do?

We included a detailed comparison in appendix B5 of the paper.

Thanks, I will check it .

By the way, the block size for Query is min(M / 4 * d, d), why do we have to calculate a minimum? Can't it just be M / 4 * d?

We want S (size block_q * block_k) to fit in SRAM of size M. We choose block_k = M/(4 * d) so we want block_q <= O(d).
We also want one query block (size block_q * d) to fit in SRAM, so block_q <= O(M / d).
As a result, we want block_q <= O(min(d, M / d)).
We just pick the constant 4 so that the proof works out.
In practice the block sizes are picked to make implementation easier (e.g. always multiple of 16, always power of 2, etc.).

OK, thanks.

$$ B_r = \frac{M}{4d} $$, is this constant 4 mannualy specified? isn't it because of four tiled parts: Q_i, K_j, V_j, O_i ? so multiply it with four?

$$ B_r = \frac{M}{4d} $$, is this constant 4 mannualy specified? isn't it because of four tiled parts: Q_i, K_j, V_j, O_i ? so multiply it with four?

@tridao hi, glad to have your clue

Yes, that's right.

Thanks for confirming. @tridao


WIP Linux support

Adds Linux support, only thing I need still need to figure out is why the image is clipped (any ideas?).
Also, don't merge yet since I'd like to rebase this PR on top of the macOS PR after that is merged [DONE], and make the shader selection a bit cleaner [DONE].

also fix a couple of tabs-vs-spaces issues in the CMakeLists.txt file (there were a couple of spaces in the Mac specific parts which I converted to tabs now).
added a GLSL shader pair
changed the shader selection to use the SOKOL_* backend defines instead of the 'raw' platform defines
added Linux to the CI build matrix

NOTE to self:

[ ] update README.md mentioning Linux and build instructions (dev-packages)

Ah... Linux missing in CI, one sec.
Thanks for merging the other PRs, I'll continue with the Linux PR tomorrow then :)

Ok, I think the Linux changes are ok now, I'll log off for today and look over the changes tomorrow again, and also check that I didn't break anything on Windows and Mac.

Too late! I just merged it.

Lol! Ok thanks :)
I guess it's fine, only the readme is missing Linux info. I can take care of that tomorrow. Have a good evening :)

only the readme is missing Linux info.
Too late for that too, since I just updated it :) Much thanks for the cleanups and the Linux support! And for Sokol in general.


Do not convert HTML to strnig

We can remove this: https://github.com/jillix/bind/blob/4794a6fc4c86b1a19a4763feeea2a781ac0eacc2/bind.js#L205-L209
This will allow us to pass jQuery objects instead of simple HTML as string.

Can you please give me a Bind use case?

If the template has attached event handlers, then they will be lost when converting the jQuery object to string.


i want to Add a silent music which plays after the demo video is over and it plays

please assign to me to complete level one

@Dhruvi-tech  Join the discord server send the music if it gets approved you can start working on this issue

okay


use CRDs for storage instead of etcd directly

Important: Join our slack channel for immediate
support and an opportunity to ask questions. Please also have a look at our
FAQ for frequently asked questions first.
If you open a GitHub issue, please follow these guidelines:

It must be a bug or a feature request.
The form below must be filled out.

Bug reports
Title
Choose a descriptive title that summarizes the symptom of the bug
General Information

Cilium version (run cilium version)
Kernel version (run uname -a)
Orchestration system version in use (e.g. kubectl version, Mesos, ...)
Link to relevant artifacts (policies, deployments scripts, ...)
Upload a system dump (run curl -sLO releases.cilium.io/tools/cluster-diagnosis.zip && python cluster-diagnosis.zip sysdump and then attach the generated zip file)

How to reproduce the issue

instruction 1
instruction 2

Feature Requests
Title
Choose a descriptive title, e.g. "Support delivering pizzas"
Description of requested functionality
Paragraph goes here
Example use cases
Use case description goes here
List of sub tasks

[ ] Task 1
[ ] Task 2
[ ] Task 3

@colemickens no worries. Let me know if this solves will solve problem https://github.com/cilium/cilium/issues/4314

I had seen that. It seems to have more dependencies and operational
concerns than using CRDs as a data store, like some other
kubernetes-adjacent projects do. I'm also not aware of how much data Cilium
needs to store, though.  (I inquired on Slack as well...)
On Tue, Jul 3, 2018, 11:58 PM André Martins<EMAIL_ADDRESS>wrote:

@colemickens https://github.com/colemickens no worries. Let me know if
this solves will solve problem #4314
https://github.com/cilium/cilium/issues/4314
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/cilium/cilium/issues/4726#issuecomment-402382588, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAT9dHVPN0rq7NzddsdiwAITrsEInCcEks5uDGeKgaJpZM4VB3fH
.


[WIP] Akka streams documentation skeleton

Added introduction index
Added Quick start guide index
Added Reactive Tweets index

Do not merge this PR yet.
This PR is a WIP made for visibility purposes, and to provide an anchor point for discussions on this part of the documentation.

This PR links to: https://github.com/akkadotnet/getakka.net/issues/154

So the skeleton stands. I've gone and added the introduction page already. Next I could use some feedback.
I already decided we are going to port all of the conceptual documentation. Because they often contain code-snippets, and porting those pages gives us the opportunity to illustrate the code with .net based samples. Which is better for our intended audience.
Which means a lot of copy paste, in which we translate the .NET specific bits.
Are there any subjects/components of the docs which should not be ported? Because they are not, and will not, be ported to Akka.NET ?
I know that the akka IO port is lagging behind and needs more work before it actually starts working. So that part of the docs are probably going to be a while.  But other then that ?
/cc @akkadotnet/core

Thanks for the feedback guys. I've incorporated them in the PR. Ill document your remarks in https://github.com/akkadotnet/getakka.net/issues/154 to help keep track of why some subjects are not ported.


fix(js_formatter): fix indention when huggable expressions break in arrow chains

Summary
I'm not sure if this came from some of the other fixes I'd made to arrow chains or was pre-existing and only came up because of those other fixes, but currently when a call argument layout with an arrow chain ends up using the middle variant (i.e., the signature chain is printed on a single line, but the body of the chain breaks), and the body is a huggable expression (i.e., an array, block body, jsx, etc.)), it gets printed with an extra level of indention:
somePromise.then(({reallyLongArguments}) => (makeTheChainFullyBreak) => [dontIndentTwice]);

// Biome Output
somePromise.then(({ reallyLongArguments }) => (makeTheChainFullyBreak) => [
    dontIndentTwice,
  ]);

// Prettier Output
somePromise.then(({ reallyLongArguments }) => (makeTheChainFullyBreak) => [
  dontIndentTwice,
]);

This PR ensures that there is only a single level of indention for the tail body, matching Prettier's output.
Test Plan
There weren't any existing tests that hit this case, so I added them to the arrow/currying.js spec test.

Actually this is seemingly incorrect and causes block function bodies to dedent too much? Not sure how that isn't being caught by prettier tests, but I'll need to fix that before this moves forward.

Okay the newer commits have updated this to properly match Prettier's logic.
The difference is that we were pre-emptively applying indent_if_group_breaks to all bodies, but we only want to do that if the chain is a grouped call argument, since that's the only time it'll need the extra level. When it's fully broken out or fully collapsed, it'll have the appropriate indention naturally.

Feel free to merge once the conflicts resolved :)


How to set value by web driver?

Question here:
How do you set value by web drivers (e.g. protractor) while doing automation tests?
something like element().sendKeys("31/03/2018 10:13 am")
Thanks

@ariesxje I am sorry but I cannot help you there. We are not doing regression, just unit tests.
I just know that there needs to make full cycle of date selection to apply new date. Or use a keyboard mode and add new value to the hhtml input element


Update redirection file

OPS Build status updates of commit 704a92e:
:clock10: Preparing: average preparing time is 43 sec(s)

OPS Build status updates of commit 704a92e:
:clock10: Incremental building: average incremental building time is 20 sec(s)

OPS Build status updates of commit 704a92e:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

.openpublishing.redirection.json
:white_check_mark:Succeeded

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.


Manifest merger failed

Hello im trying to get the latest version of the library into my project "implementation 'petrov.kristiyan:colorpicker-library:1.1.10' but i keep getting this error after syncing greadle
Manifest merger failed : Attribute application@appComponentFactory value=(android.support.v4.app.CoreComponentFactory) from [com.android.support:support-compat:28.0.0] AndroidManifest.xml:22:18-91
	is also present at [androidx.core:core:1.0.0] AndroidManifest.xml:22:18-86 value=(androidx.core.app.CoreComponentFactory).
	Suggestion: add 'tools:replace="android:appComponentFactory"' to <application> element at AndroidManifest.xml:17:5-103:19 to override.

enable androidx


Nix-prefetch-hg command does not work on darwin

myhost:rhodecode-momentum-fork anderson$ nix-prefetch-hg <url>
abort: could not find web.cacerts: /etc/ssl/certs/ca-bundle.crt
@garbas

i guess you can override web.certs in hgrc file like explained here:
https://mercurial.selenic.com/wiki/CACertificates

but it would be nice to provide some default that "just" works like we did for
git via GIT_SSL_CAINFO.
anybody has any idea how this works in marcurial?
--
Rok Garbas - http://www.garbas.si

Just ran into the same issue. Giving this a quick try. Found this in the wrapping script:
export HOME=:

This might make it difficult to use a setting from my ~/.hgrc, not sure yet though.

I guess this will be fixed by cd52c044568bdf1108428698048a9af92dc0b625 and ffd0539eba473348f752fa8ab1f1f55388634f8f
They seem to change from a hardcoded /etc/ssl/... to something based on cacerts in the nix-store.

Found a workaround:
HGRCPATH=~/.hgrc nix-prefetch-hg

Although I think that it is better to craft a special .hgrc for this case which has only the fingerprint in it. I am afraid that otherwise some settings might influence the result and thus lead to an unexpected result.

Just finished build of nix-prefetch-scripts on latest master, had to cherry-pick the commit 616ed6ee152ac16dd617fbbab2ee275e2e36d401 on top of it. Then it worked, so I guess we will see this soon fixed. :-)

@johbo looks like this has been fixed. please reopen ticket if this still persists afterwards.


Prod

Coverage increased (+0.3%) to 31.281% when pulling f41e0b48443c51210f8fc5d84c960b7c56aeb63d on prod into a253bbfa2142edb1e5d2e837810ee08ba6e09644 on master.


Add migration remarks to code docs

Description

How Has This Been Tested?

Type of Changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation (fix or improvement to the website or code docs)

Checklist

[ ] The PR is submitted to the correct branch (dev).
[ ] My code follows the code style of this project.
[ ] I've added relevant tests.

@ScarletKuro @henon @jperson2000 What do you think about including migration notes directly in the code like this to ease transition from v6?

Sure, why not.

Thanks


phpQuery missing from composer.json

I pulled down the latest and got this error when I tried to edit a page:
Class 'phpQuery' not found
Entering this command fixed it:
composer require electrolinux/phpquery
I think you will need to do that to add that package to composer.json

I am in the process of removing phpQuery in favor of simple_html_dom as the latter provides a few more useful attribute functions and also is much less destructive to the format of the document.  I am guessing there is a reference to phpQuery still hanging around causing the error.  Are you getting a specific line/file with that error?

/app/Http/Controllers/EditController.php on line 53
I did a search through the codebase for the 'phpQuery' and found that it also appears in the file /app/Respond/Models/Setting.php

Thanks!  I will update this in the next push.

This should be fixed in the latest update.


Merge stable in mater

Other PRs:

[x] https://github.com/dlang/dmd/pull/13279 (contains a better description)
[x] https://github.com/dlang/druntime/pull/3615

Thanks @MoonlightSentinel!


[FEAT]: AI generated thread titles

What would you like to see?
Currently when a new thread is created, the title on the left panel is the beginning of the first sentence. It would be useful if the LLM chosen for the workspace is used to generate a more descriptive title, similar to how the major platforms handle this.
A simple method would be to use an {action} {thing} approach.
For example, if I was asking how to build a brick wall, I might write "How do I build a brick wall?" in a thread.
If this was converted to {action} {thing} it would be:
{Building} {a brick wall}
And the prompt to the LLM could be.
Based on the user's initial input, concisely create a title for this conversation in the format of: {action} {thing}. For example 'building' (action) 'a brick wall' (thing)

Referencing this comment as to why this is currently not the case
https://github.com/Mintplex-Labs/anything-llm/issues/2262#issuecomment-2344463292

@timothycarambat I get the thinking behind it, but I'd disagree that it's a minor UX issue.
I tend to start all my chats with variants on the theme of 'please can you...', so all the threads start with the same text.
Perhaps it's something that can be a toggled feature? For me, the latency isn't noticeable because I'm using frontier models on desktop. I'm also focused on the chat so I'm not waiting to see when the thread updates.
If I adjusted the pull request to make it an optional feature would that make more sense?


[feature request] More granular control over sync parameters.

Currently, there are 2 ways the client tries to get back in sync:

rewind on major desync
slow down on minor desync

however, it is not defined what "major" and "minor" actually is.
It would be nice if we could control these parameters to control at which time-difference the programs start rewinding.
It would also be nice to be able to control the slowdown speed in these scenarios (maybe even in intervals depending on how big the time-difference is.

Hi. Firstly, how regularly is Syncplay actually rewinding or slowing down on desync for you? Has it ever been out of sync in a noticeable way?
For me it is only once in a blue moon, so it might indicate something is wrong, e.g.:

Someone has a really flakey Internet connection
Someone is trying to watch videos that is beyond the graphical capabilities of their computer
You are trying to get too many people into a regular room, and should be using a managed room
You are trying to do something which Syncplay was not designed to do, e.g. using Syncplay for public displays

Secondly, you can some of the settings as per the guide at https://syncplay.pl/guide/client/ by modifying the values (in decimal seconds) in the [client_settings] group of the syncplay.ini file, e.g.:

slowdownthreshold
rewindthreshold
fastforwardthreshold

If you want to change the slowdown rate then you will have to modify the constants.py file, so not so easy if you're not a Python developer. The relevant constant is SLOWDOWN_RATE. A different (hacky) way to do it would be to intercept the slowdown commands in the mpv or VLC interfaces to make 0.95 be converted to a different number.

In my case it's definitely the first itemon the list
we're usually only in a room with 2 or 3 people, 1 of my friends has sometimes unreliable internet.
Rewinds usually happen 1 to 3 times while watching a movie.
But when it happens it seems to rewind every 5 seconds for about a minute or so.
I wasn't aware there was a configuration to change these thresholds. I will definitely try to change those.
I'm a python developer, So changing the Slowdown rate should be easy.
Maybe I could also help to add these options to the GUI? Although I'm not really familiar with QT.
Anyway, thanks for the valuable info.

Those options are intentionally omitted from GUI, though the relevant discussion happened years ago. The conclusion was that tuning those is surprisingly complicated and can easily make players behave erratically, fall into back-and-forth loops with other users in the same room etc. Current values were tested with 3g GSM connections and worked fine there so we called it a day.
As it stands right now I don't think adding those options to GUI is desirable because they are more likely to cause trouble than help.

If Syncplay ever gets stuck slowing down or rewinding then pause, wait 5 seconds, and resume. Also, if your friend's connection is too unreliable then one option is to set up a managed room and have everyone but them be a room operator.

Yeah I definitely understand the reason for not wanting it in the UI.
Seems like there are a few other reasons we figured out after a while

we were streaming video + using syncplay at the same time.
connecting VLC to a chromecast pauzes local playback for 10-30 seconds, without triggering the pause/unpause in syncplay, causing the stream to jump back a few times for all the other viewers.


[Connection Issue]: [Docker] Unable to connect to home assistant ... despite correct ip and token

Version
3.0.0-alpha.54
Matter Controller
Google Home
Things you tried

Set up docker-compose as below
Set up token and set correct ip
Already added bridge to google home
No entity is pulled from home assistant due to connection issue
Using labels for entities but 0 entities recognized
Set up docker-daemon with exact setting as in README-link supposed
Deleted slash at the end of the HAMH_HOME_ASSISTANT_URL variable

docker-compose:
services:
  homeassistant:
    container_name: homeassistant
    image: "ghcr.io/home-assistant/home-assistant:stable"
    volumes:
      - /var/home/core/dvol/homeassistant_config:/config:Z
      - /etc/localtime:/etc/localtime:roZ
      - /run/dbus:/run/dbus:roZ
    restart: unless-stopped
    privileged: true
    network_mode: host
  homeassistant-matter-hub:
    container_name: homeassistant-matter-hub
    image: ghcr.io/t0bst4r/home-assistant-matter-hub:latest
    volumes:
      - /var/home/core/dvol/homeassistant-matter-hub_data:/data:Z
    restart: unless-stopped
    privileged: true
    network_mode: host
    environment:
      HAMH_HOME_ASSISTANT_URL: "http://<IP_ADDRESS>:8123"
      HAMH_HOME_ASSISTANT_ACCESS_TOKEN: "xxxxxxxxxxxxxxxxxx"
      HAMH_LOG_LEVEL: "info"
      HAMH_WEB_PORT: "8482"
    depends_on:
      homeassistant:
        condition: service_started
        restart: true

Relevant log output
[ 2024-12-13T13:13:20.403Z ] [ ERROR ] [ Matter / Logger                                    ]: Unhandled error detected: Unable to connect to home assistant with url: http://<IP_ADDRESS>:8123

  at _HomeAssistantClient.parseError (file:///usr/local/lib/node_modules/home-assistant-matter-hub/dist/backend/cli.js:776:14)

  at file:///usr/local/lib/node_modules/home-assistant-matter-hub/dist/backend/cli.js:768:18

  at process.processTicksAndRejections (node:internal/process/task_queues:95:5)

  at async _HomeAssistantClient.initialize (file:///usr/local/lib/node_modules/home-assistant-matter-hub/dist/backend/cli.js:762:23)

file:///usr/local/lib/node_modules/home-assistant-matter-hub/dist/backend/cli.js:776

      return new Error(

             ^

Error: Unable to connect to home assistant with url: http://<IP_ADDRESS>:8123

    at _HomeAssistantClient.parseError (file:///usr/local/lib/node_modules/home-assistant-matter-hub/dist/backend/cli.js:776:14)

    at file:///usr/local/lib/node_modules/home-assistant-matter-hub/dist/backend/cli.js:768:18

    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)

    at async _HomeAssistantClient.initialize (file:///usr/local/lib/node_modules/home-assistant-matter-hub/dist/backend/cli.js:762:23)

Node.js v20.18.1

[ 2024-12-13T13:13:21.634Z ] [ ERROR ] [ Matter / Logger                                    ]: Unhandled error detected: Unable to connect to home assistant with url: http://<IP_ADDRESS>:8123

  at _HomeAssistantClient.parseError (file:///usr/local/lib/node_modules/home-assistant-matter-hub/dist/backend/cli.js:776:14)

  at file:///usr/local/lib/node_modules/home-assistant-matter-hub/dist/backend/cli.js:768:18

  at process.processTicksAndRejections (node:internal/process/task_queues:95:5)

  at async _HomeAssistantClient.initialize (file:///usr/local/lib/node_modules/home-assistant-matter-hub/dist/backend/cli.js:762:23)

file:///usr/local/lib/node_modules/home-assistant-matter-hub/dist/backend/cli.js:776

      return new Error(

             ^

Error: Unable to connect to home assistant with url: http://<IP_ADDRESS>:8123

    at _HomeAssistantClient.parseError (file:///usr/local/lib/node_modules/home-assistant-matter-hub/dist/backend/cli.js:776:14)

    at file:///usr/local/lib/node_modules/home-assistant-matter-hub/dist/backend/cli.js:768:18

    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)

    at async _HomeAssistantClient.initialize (file:///usr/local/lib/node_modules/home-assistant-matter-hub/dist/backend/cli.js:762:23)

Node.js v20.18.1

### What deployment method do you use?

Docker

### Did you already check the following?

- [ ] Does your network include any VLAN setup?
- [ ] Do you have any special firewall rules / services in place?
- [ ] Is IPv6 enabled in your LOCAL network?
- [X] If you are using the docker deployment: Is IPv6 enabled within your docker daemon config?
- [X] If you are using the docker deployment: Do you run the container with host network?

### Documentation & Issues

- [X] I have reviewed the [documentation](https://github.com/t0bst4r/home-assistant-matter-hub/blob/main/packages/documentation/README.md) and the linked troubleshooting guide.
- [X] I have searched the [issues](https://github.com/t0bst4r/home-assistant-matter-hub/issues) for a similar problem.

Sorry,
right after testing it multiple times it seemed to work
I re-started everything several times but no chance
Right now it did
Feel free to close this topic if everything is fine for you

Was home assistant restarted when HAMH started?
HA needs to be up, when HAMH starts.

Yes kind of
I did multiple restarts whilst HA was constantly running without restarts but in the end it seems to be running now
To guarantee this startup order my current try is to use some kind of health check via docker compose
If you have any better solutions feel free to add them
Thank you for your great work, this matter hub is so awesome
services:
  homeassistant:
    container_name: homeassistant
    image: "ghcr.io/home-assistant/home-assistant:stable"
    volumes:
      - /var/home/core/dvol/homeassistant_config:/config:Z
      - /etc/localtime:/etc/localtime:roZ
      - /run/dbus:/run/dbus:roZ
    restart: unless-stopped
    privileged: true
    network_mode: host
    healthcheck:
      test: "curl --fail http://<IP_ADDRESS>:8123/ || exit 1"
  homeassistant-matter-hub:
    container_name: homeassistant-matter-hub
    image: ghcr.io/t0bst4r/home-assistant-matter-hub:latest
    volumes:
      - /var/home/core/dvol/homeassistant-matter-hub_data:/data:Z
    restart: unless-stopped
    privileged: true
    network_mode: host
    environment:
      HAMH_HOME_ASSISTANT_URL: "http://<IP_ADDRESS>:8123"
      HAMH_HOME_ASSISTANT_ACCESS_TOKEN: "xxxxxx"
      HAMH_LOG_LEVEL: "info"
      HAMH_WEB_PORT: "8482"
    depends_on:
      homeassistant:
        condition: service_healthy
        restart: true

OK first system restart and the above shown healthcheck does not work
I think HA needs more time, therefor i need some other solution to further delay the start of homeassistant-matter-hub


README is not clear about docker config

Hi,
I give a try to your project but don't understand if I need ton indicate the name of a docker image or a docker container in the config file.

change php_image_name_here to the name of your project's PHP Docker image (i.e. an image that contains a php binary executable from CLI)

then

make sure the container specified in the configuration is up and running

I got it working by specifiying an image name.
But question, how do you handle multiple containers running at the same time and using the same image ?

That's currently not handled. I can add support for using container names instead of image names.

Thanks I pushed #4 related to this issue. Maybe you could have a look at it?


S3 bucket for country data is not accessible

When following the instructions on the README.md and after installing python dependencies and running streamlit run corona-calculator.py locally, the app fails with
KeyError: 'AWSAccessKeyId'
Traceback:
  File "/Users/struong/project/corona-calculator/.venv/lib/python3.7/site-packages/streamlit/ScriptRunner.py", line 314, in _run_script
    exec(code, module.__dict__)
  File "/Users/struong/project/corona-calculator/corona-calculator.py", line 6, in <module>
    import models
  File "/Users/struong/project/corona-calculator/models.py", line 5, in <module>
    import data.constants as constants
  File "/Users/struong/project/corona-calculator/data/constants.py", line 14, in <module>
    from s3_utils import download_file, S3_DISEASE_DATA_OBJ_NAME
  File "/Users/struong/project/corona-calculator/s3_utils.py", line 10, in <module>
    _S3_ACCESS_KEY = os.environ["AWSAccessKeyId"].replace("\r", "")
  File "/usr/local/bin/../Cellar/python/3.7.5/bin/../Frameworks/Python.framework/Versions/3.7/lib/python3.7/os.py", line 679, in __getitem__
    raise KeyError(key) from None

Is there a way we can make it run locally? with and/or without access to the S3 bucket coronavirus-calculator-data.

Is it possible to have that bucket public?

Hey! Very good question. @psteeves can you chip in on this?
I don't think we can make S3 public, but we should add functionality to mock it locally, I think.

I see two options:

Add a csv version of the data to the repo we can fallback on if the credentials are not available (kinda hacky)
Add a read-only S3 access key to the repo (better option IMO). Users would not be able to update the data (requires write access) but would be able to pull the latest data.

I'd prefer to simulate S3 locally for fear of leaking S3 credentials which, even if RO, might lead to spamming and DOS.

@syltruong Fixed by https://github.com/archydeberker/corona-calculator/pull/70
Will deploy to prod in the next half hour.


pulley: Implement simd extractlane CLIF lowering

Gets a number of *.wast tests passing.

Currently built on https://github.com/bytecodealliance/wasmtime/pull/9842


No 'Access-Control-Allow-Origin' header is present on the requested resource.

Hi everyone,
I just tried to use the webchat just as it is described here by Juste.
I can see a 500 code in Chrome's console saying Access to XMLHttpRequest at 'http://localhost:5005/socket.io/?EIO=3&transport=polling&t=MYei2_H' from origin 'null' has been blocked by CORS policy: No 'Access-Control-Allow-Origin' header is present on the requested resource.
Also python is facing me with an AttributeError:
<IP_ADDRESS> - - [2019-02-01 11:55:46] "GET /socket.io/?EIO=3&transport=polling&t=MYeiq3t HTTP/1.1" 500 161 0.003995
Traceback (most recent call last):
  File "C:\Users\niklas.fallik\AppData\Local\Continuum\anaconda3\envs\py36\lib\site-packages\gevent\pywsgi.py", line 976, in handle_one_response
    self.run_application()
  File "C:\Users\niklas.fallik\AppData\Local\Continuum\anaconda3\envs\py36\lib\site-packages\gevent\pywsgi.py", line 923, in run_application
    self.result = self.application(self.environ, self.start_response)
  File "C:\Users\niklas.fallik\AppData\Local\Continuum\anaconda3\envs\py36\lib\site-packages\flask\app.py", line 2309, in __call__
    return self.wsgi_app(environ, start_response)
  File "C:\Users\niklas.fallik\AppData\Local\Continuum\anaconda3\envs\py36\lib\site-packages\engineio\middleware.py", line 59, in __call__
    return self.engineio_app.handle_request(environ, start_response)
  File "C:\Users\niklas.fallik\AppData\Local\Continuum\anaconda3\envs\py36\lib\site-packages\socketio\server.py", line 360, in handle_request
    return self.eio.handle_request(environ, start_response)
  File "C:\Users\niklas.fallik\AppData\Local\Continuum\anaconda3\envs\py36\lib\site-packages\engineio\server.py", line 333, in handle_request
    transport, b64)
  File "C:\Users\niklas.fallik\AppData\Local\Continuum\anaconda3\envs\py36\lib\site-packages\engineio\server.py", line 454, in _handle_connect
    s = socket.Socket(self, sid)
  File "C:\Users\niklas.fallik\AppData\Local\Continuum\anaconda3\envs\py36\lib\site-packages\engineio\socket.py", line 17, in __init__
    self.queue = self.server.create_queue()
  File "C:\Users\niklas.fallik\AppData\Local\Continuum\anaconda3\envs\py36\lib\site-packages\engineio\server.py", line 430, in create_queue
    sys.modules[queue.__class__.__module__], 'Empty')
AttributeError: 'gevent._queue.JoinableQueue' object has no attribute 'Empty'
2019-02-01T10:55:53Z {'REMOTE_ADDR': '<IP_ADDRESS>', 'REMOTE_PORT': '57190', 'HTTP_HOST': 'localhost:5005', (hidden keys: 23)} failed with AttributeError

The "REMOTE_PORT" is changing once a while.
I used the html-code provided in dev/src/index.html. Chrome shows an empty page.
What am I missing?
Regards,
Nick

Try to use https by using HTTPS=true in your npm script

Until now I have not done anything but cloning the repo. Can you tell me exactly where to add this?

Hi, is there any update on this issue? I'm facing same issue.
"Access to XMLHttpRequest at 'http://localhost:5500/socket.io/?EIO=3&transport=polling&t=MpNmxKe' from origin 'http://localhost:3000' has been blocked by CORS policy: No 'Access-Control-Allow-Origin' header is present on the requested resource."

I think this issue is not resolved by the commit above (https://github.com/CamaraDosDeputados/rasa-webchat/commit/b9cecd53e9ad7783d40217db4af6672c97539e73)
The commit only appended an image alt text, nothing to do with the CORS issue.
Try to add --cors "*" when starting the rasa socket service.
For your reference, I use the following command:
rasa run --credentials ./credentials.yml \
    -m ./models/my-model.tar.gz \
    -p 5500 \
    --cors "*"


Script failure

the action stopped working like 2 week ago
this is the error it's been giving:

 click to see logs
>>> Run npm start

><EMAIL_ADDRESS>start /home/runner/work/lang-box/lang-box
> node --experimental-modules --es-module-specifier-resolution=node src/index

ignoreLangs: [ 'yaml', 'json', 'svg', 'xml', 'toml', 'markdown' ]
username is RyanNorooz.

1 events fetched.
2 commits fetched.

run > git checkout --orphan temp && git rm -rf . && rm -rf *
exited with code 0
run > echo "*.* linguist-detectable" > .gitattributes
run > git config user.name "dummy" && git config user.email<EMAIL_ADDRESS>exited with code 0
exited with code 0
run > git add . && git commit -m "dummy"
exited with code 0
run > github-linguist --breakdown --json
exited with code 0

Error: Validation Failed
    at ApiClient.fetch (file:///home/runner/work/lang-box/lang-box/src/api.js:20:13)
    at processTicksAndRejections (internal/process/task_queues.js:[9](https://github.com/RyanNorooz/lang-box/runs/5392042580?check_suite_focus=true#step:9:9)5:5)
    at async file:///home/runner/work/lang-box/lang-box/src/index.js:[11](https://github.com/RyanNorooz/lang-box/runs/5392042580?check_suite_focus=true#step:9:11)6:5
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>start: `node --experimental-modules --es-module-specifier-resolution=node src/index`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>start script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
npm WARN Local package.json exists, but node_modules missing, did you mean to install?

npm ERR!

Hi, are your secret tokens still alive?

Hello. I also had the same problem and tried to find the cause.
My guess is that if I don't write any code within the date range that the action collects, there is nothing to output and an error occurs.
After committing any modifications, performing the action again worked successfully.

My guess is that if I don't write any code within the date range that the action collects, there is nothing to output and an error occurs.
After committing any modifications, performing the action again worked successfully.

I can confirm the issue was that nothing was committed in the past month (approximately) and this caused the error.
Simply committing to your repositories fixes the issue.