[Bug] Helm install Skywalking 9.1.0,Could not initialize Elasticsearch.

Search before asking

[X] I had searched in the issues and found no similar issues.

Apache SkyWalking Component
OAP server (apache/skywalking)
What happened
I successfully initialized it once, and then manually deleted the index of elasticsearch. If I want to install it again, I get an error.
Helm install Skywalking 9.1.0,Could not initialize Elasticsearch.
helm install skywalking skywalking -n skywalking
--set oap.image.tag=9.1.0
--set oap.storageType=elasticsearch
--set ui.image.tag=9.1.0
--set ui.service.type=NodePort
--set ui.service.externalPort=80
--set ui.service.internalPorte=8080
--set ui.service.nodePort=30008
--set elasticsearch.enabled=false
--set elasticsearch.config.host=<IP_ADDRESS>
--set elasticsearch.config.port.http=9200
this is error skywalking oap log
2022-08-18 08:58:47,941 com.linecorp.armeria.common.Flags 1464 [main] INFO  [] - com.linecorp.armeria.dumpOpenSslInfo: false (default)
2022-08-18 08:58:48,041 com.linecorp.armeria.internal.common.JavaVersionSpecific 45 [main] INFO  [] - Using the APIs optimized for: Java 9+
2022-08-18 08:58:48,255 org.apache.skywalking.library.elasticsearch.ElasticSearch 150 [armeria-common-worker-epoll-2-1] INFO  [] - ElasticSearch version is: ElasticSearch 7.4
2022-08-18 08:58:48,269 org.apache.skywalking.oap.server.core.storage.model.ModelInstaller 43 [main] INFO  [] - table: alarm_record does not exist. OAP is running in 'no-init' mode, waiting... retry 3s later.
2022-08-18 08:58:51,289 org.apache.skywalking.oap.server.core.storage.model.ModelInstaller 43 [main] INFO  [] - table: alarm_record does not exist. OAP is running in 'no-init' mode, waiting... retry 3s later.
2022-08-18 08:58:54,314 org.apache.skywalking.oap.server.core.storage.model.ModelInstaller 43 [main] INFO  [] - table: alarm_record does not exist. OAP is running in 'no-init' mode, waiting... retry 3s later.
2022-08-18 08:58:57,333 org.apache.skywalking.oap.server.core.storage.model.ModelInstaller 43 [main] INFO  [] - table: alarm_record does not exist. OAP is running in 'no-init' mode, waiting... retry 3s later.
2022-08-18 08:59:00,350 org.apache.skywalking.oap.server.core.storage.model.ModelInstaller 43 [main] INFO  [] - table: alarm_record does not exist. OAP is running in 'no-init' mode, waiting... retry 3s later.
2022-08-18 08:59:03,367 org.apache.skywalking.oap.server.core.storage.model.ModelInstaller 43 [main] INFO  [] - table: alarm_record does not exist. OAP is running in 'no-init' mode, waiting... retry 3s later.
2022-08-18 08:59:06,377 org.apache.skywalking.oap.server.core.storage.model.ModelInstaller 43 [main] INFO  [] - table: alarm_record does not exist. OAP is running in 'no-init' mode, waiting... retry 3s later.
2022-08-18 08:59:09,390 org.apache.skywalking.oap.server.core.storage.model.ModelInstaller 43 [main] INFO  [] - table: alarm_record does not exist. OAP is running in 'no-init' mode, waiting... retry 3s later.
2022-08-18 08:59:12,403 org.apache.skywalking.oap.server.core.storage.model.ModelInstaller 43 [main] INFO  [] - table: alarm_record does not exist. OAP is running in 'no-init' mode, waiting... retry 3s later.
2022-08-18 08:59:15,419 org.apache.skywalking.oap.server.core.storage.model.ModelInstaller 43 [main] INFO  [] - table: alarm_record does not exist. OAP is running in 'no-init' mode, waiting... retry 3s later.
2022-08-18 08:59:18,434 org.apache.skywalking.oap.server.core.storage.model.ModelInstaller 43 [main] INFO  [] - table: alarm_record does not exist. OAP is running in 'no-init' mode, waiting... retry 3s later.
2022-08-18 08:59:21,450 org.apache.skywalking.oap.server.core.storage.model.ModelInstaller 43 [main] INFO  [] - table: alarm_record does not exist. OAP is running in 'no-init' mode, waiting... retry 3s later.
2022-08-18 08:59:24,455 org.apache.skywalking.oap.server.core.storage.model.ModelInstaller 43 [main] INFO  [] - table: alarm_record does not exist. OAP is running in 'no-init' mode, waiting... retry 3s later.
2022-08-18 08:59:27,462 org.apache.skywalking.oap.server.core.storage.model.ModelInstaller 43 [main] INFO  [] - table: alarm_record does not exist. OAP is running in 'no-init' mode, waiting... retry 3s later.
2022-08-18 08:59:30,479 org.apache.skywalking.oap.server.core.storage.model.ModelInstaller 43 [main] INFO  [] - table: alarm_record does not exist. OAP is running in 'no-init' mode, waiting... retry 3s later.
2022-08-18 08:59:33,486 org.apache.skywalking.oap.server.core.storage.model.ModelInstaller 43 [main] INFO  [] - table: alarm_record does not exist. OAP is running in 'no-init' mode, waiting... retry 3s later.
2022-08-18 08:59:36,501 org.apache.skywalking.oap.server.core.storage.model.ModelInstaller 43 [main] INFO  [] - table: alarm_record does not exist. OAP is running in 'no-init' mode, waiting... retry 3s later.
2022-08-18 08:59:39,527 org.apache.skywalking.oap.server.core.storage.model.ModelInstaller 43 [main] INFO  [] - table: alarm_record does not exist. OAP is running in 'no-init' mode, waiting... retry 3s later.
What you expected to happen
Able to successfully initialize the index in elasticsearch
How to reproduce
helm install skywalking skywalking -n skywalking
--set oap.image.tag=9.1.0
--set oap.storageType=elasticsearch
--set ui.image.tag=9.1.0
--set ui.service.type=NodePort
--set ui.service.externalPort=80
--set ui.service.internalPorte=8080
--set ui.service.nodePort=30008
--set elasticsearch.enabled=false
--set elasticsearch.config.host=<IP_ADDRESS>
--set elasticsearch.config.port.http=9200
Anything else
No response
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

The running demo is on top of a new ElasticSearch and initialized by the OAP automatically.
Re-run OAP init-container if you use helm.
This is not a bug.


Request: firebase_storage to support metadata

I could use support reading/writing metadata, especially custom metadata field.

flutter/plugins#553 is addressing that

Seems the PR was abandoned

Should be added by https://github.com/flutter/plugins/pull/556/files


Documentation on how handle a click on a cluster

First of all thanks for this wonderful work, very useful.
I'd like to to something when a user clicks on a marker cluster and this one expands. I tried to add @click like this: <l-marker-cluster-group @click="testclic">, but it didn't work. I saw you documented "handleevent" in the Playground but it would be great to have @click and why not @hover, for example.
Thanks in advance!

 Here is a list of all available events you can listen to:

As you can see for the marker cluster there is a markercluster click event which you can listen to with @markerclusterclick="yourHandler
The @click listener listen to the marker clicks itself. I know from Vue side this looks a little bit counter intuitive. But the markercluster plugin (on which this plugin for Vue is based of) defines these event naming conventions.
Hope this will help you.
Best regards
Veit

If I got you correctly, you wanna catch the marker cluster click event. Here is a list of all available events you can listen to:

As you can see for the marker cluster there is a markercluster click event which you can listen to with @markerclusterclick="yourHandler
The @click listener listen to the marker clicks itself. I know from Vue side this looks a little bit counter intuitive. But the markercluster plugin (on which this plugin for Vue is based of) defines these event naming conventions.
Hope this will help you.
Best regards
Veit
@djcaesar9114 Do you still have any problems? Otherwise I would mark this issue as resolved and close it.

@djcaesar9114 because I don't get a response from you, I expect the problem was solved. Closing this ticket now.

Wow, it's impossible to take holidays on Github! ^_^
Anyway, I saw it was possible. It's just not documented enough (for me, of course) in the examples, I'll take a look when I'm back to see if I manage to handle the click.
Thanks for your answer!

No holidays are impossible here :-P
You are welcome. Feel free to improve the documentation to make it cleaner. Using Typescript should show the events you can listen to.


Revert "bugfix: Don't return disabled users on GetUser call"

If you think https://github.com/cs3org/reva/pull/4430 is too much for now. We can also revert the original PR for now.
This reverts commit 9b7ec1e4afc754b9ce39cd053aa793febec478a7.
This needs some more work we need to allow admins to lookup disabled users.

Hm, adding a changelog for reverting an  unreleased bugfix doesn't really make sense, does it? It will just confuse people IMO.


Update index.md

small grammar fix

@dlech : Thanks for your contribution! The author(s) have been notified to review your proposed change.


Document Module

Since the site is built with MkDocs mkdocs-material, do you accept improvement PRs? I'm familiar with it. I also have some Flax and TensorFlow docs experience. Awesome work on elegy

Hey @8bitmp3!  For sure, documentation is one of our weakest points right now after the recent change. I am in the process of creating some guides in #156.


Extending ChatGPTConversation

Hi there 👋
Firstly, thanks for the neat wrapper :) I've forked it and added it as a submodule to my project DialogueDreamland and it works nicely. ❤
I don't want to modify your code, and I want to be able to always sync the latest changes, so I want to please ask if you would consider a change.
I plan on using different instances of ChatGPTConversation, so I want to be able to set them up during runtime (instead of defining them in the Editor). I've created my own custom class that extends ChatGPTConversation which allows me to set it up and check if it is ready. It looks like this:
public class CustomChatGPT : ChatGPTConversation
{
    public bool Ready => _selectedModel != null;
    public void SetupGPT(string apiKey, string chatBotName, string initialPrompt, int maxTokens, float temperature)
    {
        _apiKey = apiKey;
        _chatbotName = chatBotName;
        _initialPrompt = initialPrompt;
        _maxTokens = maxTokens;
        _temperature = temperature;
    }
}

CustomChatGPT.cs
This method requires me to change those variables to protected though. I'm not sure if this is the best approach, but any variation that allows me to do this would be great.
Please let me know what you think 🙏

Hi there :-)
Absolutely a must-have feature for this wrapper, I'll work on it tonight.

Alright, so a couple of things:
-The new ChatGPT API does not take the temperature or max_tokens properties, those only exist in the wrapper to support the older GPT completion models, so if you're using ChatGPT you don't need to worry about either of those.
-Even if you are using the older GPT models, I don't think it would be useful to change either of those things at runtime. Those probably should just be set-and-forgets on your prefabs.
-The _chatbotName variable also only applies to the older models. As far as I can tell I have to use 'role':'assistant' in my requests for ChatGPT's response messages, so there's no point in adding a name (You'll want to tell the model what it's name is in the _initialPrompt).
-The API key will not change at runtime, so you don't need to worry about that either.
All of this to say, _initialPrompt is the only thing that needs to be changed at runtime. The initial prompt sets the entire context for the conversation, and also determines what character ChatGPT will be playing and what personality it has.
I just added a ResetChat method to the ChatGPTConversation script that takes a string as an argument and makes a new blank conversation and sets the _initialPrompt to the string argument.
public void ResetChat(string initialPrompt) {
    switch (_model) {
        case Model.ChatGPT:
            _chat = new Chat(initialPrompt);
            break;
        default:
            _prompt = new Prompt(_chatbotName, initialPrompt);
            break;
        }
}

Essentially, just subscribe that method to whatever event that is setting up your conversation at runtime and it should be good to go. I added demo buttons to the UI as well.
Let me know if this is sufficient for what you had in mind, or if you require something else :-)

Thank you. I've updated mine to use the ResetChat and got rid of max_tokens and temperature, and it works well.
This is probably a bit of an edge case, but I'm going to have players enter their own API keys before they can start playing (so that everything isn't tied to my account). I've remedied it with a one-liner which I don't mind maintaining in my forked repo.
protected void SetApiKey(string apiKey) => _apiKey = apiKey;
Thanks for responding so quickly 🙌 I'll let you know if I encounter more struggles.
Cheers 🍻


app: Avatar falls on naruto run

The naruto run seems to be working without any physics, it works for flat surfaces. On areas with slopes such as shadows scn's dunes, The avatar falls down the floor.
Steps to reproduce:
-Switch to any scn with slopes(shadows)
-press shift twice facing towards the dunes
https://user-images.githubusercontent.com/35926530/145467024-74124115-aaf7-4298-8d9d-5320e9271d8a.mp4

Was fixed with https://github.com/webaverse/app/issues/1610


How to pass parameter to main.yml from marathon container?

I have marathon container on ubuntu 14.04. and I have ansible-marathon role in configuration file. In main.yml file of ansible-marathon role I need some parameters for
command: and volume: modules.  So my question is how we can pass them to the container.? For the broader view.
this is yml file what values should come in the highlighted place.

name: pull latest version of marathon container
shell: "docker pull  {{mesosphere}}//marathon:{{V0.15.3}}"
tags : install
name: starting marathon
docker:
image: {{mesosphere}}//marathon:{{V0.15.3}}
name : marathon
command: "                 "
state : reloaded
restart_policy:always
hostname: "{{ansible_eth0.ipv4.address}}"
volumes: "{{     }}"

tags:

config
restart

Several ways to do this:

use the file /etc/default/marathon/marathon.defaults
use environment variables
change the script that starts marathon.

For details see here: https://mesosphere.github.io/marathon/docs/command-line-flags.html


NSFileManager.copyItemAtPathToPathError crashes after copying the file...

I currently haven't setup a station to compile the IOS runtimes; so I can't try the master branch yet; but this issue does effect version 1.01 (earliest I had) to 1.12 exactly the same way.    It copies the file and crashes.   The Command line just says "Session Ended without errors", the actual logs show
com.apple.CoreSimulator.SimDevice.F11B2A14-9647-4D61-95AE-516456A95CF1.launchd_sim27211: Service exited due to signal: Bus error: 10
SpringBoard[27224]: Application 'UIKitApplication:org.nativescript.Hello[0xcc16]' crashed.
The actual code I am using is:
var fileManager = NSFileManager.defaultManager();
var destPath = fs.knownFolders.documents().path + '/Test.sqlite';
var srcPath = fs.knownFolders.currentApp().path + '/Test.sqlite';
return fileManager.copyItemAtPathToPathError(srcPath, destPath, null);

Please note it doesn't matter if I create a interop variable for the Error.      Also note; it only crashes if it copies the file; if the file is already present (so it would then fail to copy) then it actually returns "false" and the program continues fine.

I have managed to reproduce the issue with the following snippet and will take a look at it:
var fs = require('file-system');

var file = fs.knownFolders.currentApp().getFile('Test.sqlite');
console.log(file.path);

var srcPath = fs.path.join(fs.knownFolders.currentApp().path, 'Test.sqlite');
console.log(srcPath);

var destPath = fs.path.join(fs.knownFolders.documents().path, 'Test.sqlite');
console.log(destPath);

var fileManager = NSFileManager.defaultManager();
var errorRef = new interop.Reference();
if (!fileManager.copyItemAtPathToPathError(srcPath, destPath, errorRef)) {
    console.error(errorRef.value);
    return;
}

console.log('Successfully copied file');

This issue is caused by a bug in our metadata generator. We were trying to guess (with the help of some heuristics - read more here) which method from the iOS SDK returns retained object and save this information in metadata. Our guessing was not quite right in some rare cases. This is the case with copyItemAtPath:toPath:error: method. We assume that the method returns retained object because its selector starts with copy. As part of another feature for v1.2 we changed the way of checking which method returns retained object (now we ask Clang to tell us), so in v1.2 the issue is fixed. Here is the commit that fixes it https://github.com/NativeScript/ios-metadata-generator/commit/aedeaf7ca021bb9d8aa526d5f3a766e3755a4575.

Awesome.  I started getting eveything on my mac to build the latest.  But it is good to know that it is fixed in the next version.

where knownFolders.documents() is supposed to be located and it doesn’t tell me in the documentation.  What path is that? Where are these files? Please let me know

@srnaik here is the definition of documents(). On iOS it returns NSFileManager.defaultManager().URLsForDirectoryInDomains(NSSearchPathDirectory.DocumentDirectory, NSSearchPathDomainMask.UserDomainMask).
And here is the Android implementation.

Thank you @ivanbuhov , that was very helpful !


problem in Base2306

          problem in Base2306

Cannot change translation for each - one should be "learn" and second one should be "learning" i cannot achieve this, because line 224 doesnt change anything and line 219 changes both

Originally posted by @AwesomeFart in https://github.com/LorisYounger/VPet/issues/267#issuecomment-1758108711

Because they are same in chinese, both are study.
I will make it different in next update, you can skip it


Feat/create event

a new endpoint to create webinar / inlive event
changelogs :

Added new component for event related interaction

Date Picker using react-datepicker
Time Picker
Image Cropper using react-cropper

new page to create event

require sign in to access the page
uses compressor-js  before uploading the image to server
added new column to the event table

updated_at
is_published
thumbnail_url

only event owner can access the event if event is not yet published

uses custom server with express to host the event thumbnail
updated next config & dockerfile to build app using custom server

next doesn't perform build with standalone

@GagahGK For some reason, I cannot get env INLIVE_HUB_API_KEY value after building and running the custom server through docker. This caused the website cannot create a new room when running through docker. Can you check this?

@GagahGK For some reason, I cannot get env INLIVE_HUB_API_KEY value after building and running the custom server through docker. This caused the website couldn't create a new room when running through docker in my side. Can you check this? This doesn't happen in the main branch version. Probably because the main branch is using standalone output.

I think can solve this by building the code using the standalone option, previously there was a warning and I'm not sure how to handle it, turns out I just need to replace the provided server.js file with the custom server.js
but still the node_modules need to be copied since the standalone didn't provide express library

@faiq-naufal sorry for taking a while to confirm case, but I've tried to use both standalone and non standalone build through docker using docker-compose the server able to create the meeting room successfully
have you make sure your API key is not expired ?

@GagahGK  Ah sorry, I'm not talking about running with docker compose. I'm talking about building the image and running the image manually like in this steps. https://github.com/inlivedev/inlive-room#running-with-docker. The environment variables when using custom server are not included if I run using those steps. In the main branch, I still get the environment variables when running with those steps.

@faiq-naufal
it looks like that the default server.js uses the generated .env and loaded it during runtime, but I think it won't be a problem since the image will be loaded into a kubernetes pod. we can update the config file in kubernetes to load the env variable.
but due to this to run in local now we need to pass a environment argument to load a variable
docker run -p 3000:3000 -e VARIABLE=value inlive-room:latest

@GagahGK I see then, I think it's fine if this is not an issue with the deployment.

@GagahGK I simplified the server.ts logic on 76863eb. No need to throw an error when envs are not set and give them default value instead. I also added env to the deployment workflow fefc272.

@faiq-naufal thankyou, I've updated the code with your requested changes :

removing the console logging
limit access to the create event page
added comments to unimplemented updateDraft() function
reverting the typescript version and
making the icon scalable

@GagahGK currently, there is no navigation link to the event create page /event/create. I think this is fine for now considering there are more features such as event list, update event that haven't been developed. But, I want to hear your opinion about this. It seems, mas Ashar also hasn't built the UI for the navigation link to the event create page.

@GagahGK currently, there is no navigation link to the event create page /event/create. I think not having navigation link is fine for now considering there are more features such as event list, update event that haven't been finished. But, I want to hear your thought about this. It seems, mas Ashar also hasn't built the UI for the navigation link to the event create page.

@faiq-naufal I also noticed that and have asked him previously and haven't receive any update, I think we should do a follow up.
but I think, it was mean to be navigated through the My Events page as seen on the figma

@GagahGK Ah yeah, that's correct. We need to finish the event list page first and from there user can create a new event.

@GagahGK Ah yeah, that's correct. We need to finish the "My Events" page first and from there user can create a new event.

@faiq-naufal it'll be delivered with the next branch alongside with the update & delete event feature


tests: make test-deploy-ad-server idempotent

Signed-off-by: Michael Adam<EMAIL_ADDRESS>

@phlogistonjohn - sorry, couldn't resist, despite our earlier discussion.
This deployment should go into the golang tests.
But I will migrate these things over to the samba-container repo soon...


Checkout fish shell

Fish shell looks super nice, but I would like to know more about the implications of setting it as a default shell.

Going to stay with zsh, as some plugins give me similar features.


New release, pls

https://chrome.google.com/webstore/detail/terra-station-wallet/aiifbnbfobpmeekipheeijimdpnlpgpp
Version 3.0.2
Updated 28 May 2022
https://github.com/terra-money/station-extension/releases
Version 2.5.0
Updated 11 Feb 2022
Update, pls, at github.
Thx

Hello @Kwaskoff A SupportTicket has been opened for you with SupportTicket number TB7833HH. You should talk to a customer service representative on the [LIVECHAT PAGE]. So that your transaction and issues can be looked into. Thank you. [CLICK HERE]

https://github.com/terra-money/station-extension/releases/tag/3.0.8


Subcommand overrides option value set in parent command with default value

Hi. We are currently not using inherited options and because we are using Picocli through Quarkus we are sharing options between command and subcommands by making them a @Singleton (which should be equivalent to having them on a static field but then they can be injected).So we have something like this:
@Singleton
public class MyOption {
    @Option(names = "-m", defaultValue = "default", description = "My option")
    String my;
}

then both the parent and the subcommand have the following:
@Mixin
@Inject
MyOption myOption;

When sub command is called with the option specified in the parent, the option value is overwritten with the default:
command -m bla subcommand # Value of MyOption.my is 'default' instead of `bla`

I don't know if this affects inherited options or other ways of sharing (e.g. static fields), but I guess in this case Picocli is only checking if the options are defined on the children before setting the default value. Could Picocli also check if the value is already set and if it is don't set the default value?

Apologies for my late reply; this slipped off my inbox horizon...
That is a tricky one...
Picocli parses the parameters one by one, so first -m and bla (resulting in MyOption.my having the value "bla"), then subcommand, and when no further parameters are detected, picocli will assign default values for the options of subcommand. This last step is where MyOption.my is reset to the default value.
One idea is to implement the -m option as am @Option-annotated setter method instead of a field. That would allow you to implement the logic you requested:

Could Picocli also check if the value is already set and if it is don't set the default value?

@Singleton
public class MyOption {
    String my;
    @Option(names = "-m", defaultValue = "default", description = "My option")
    void setMy(String newValue) {
        if (my != null && !"default".equals(newValue)) {
            this.my = newValue;
        }
    }
}

I see... I thought that using the setter could be a workaround, but it's a bit annoying, since there's only one place where we need a default value and it would be inconsistent with the rest of the code that doesn't use setters...
Would this also happen with inherited options? If not I might look into switching to them...

If there is only one place where you need special behaviour, then doesn't it make sense that the code for that one place looks different...? 😅
I think using a setter for this is fine. 👍
I will go ahead and close this ticket, but feel free to let me know if it needs to be reopened.

If there is only one place where you need special behaviour, then doesn't it make sense that the code for that one place looks different...?

Well it's about cognitive load, having to add comments to explain the behavior very precisely, etc. because deviating from the norm usually means tricky edge cases, for example, with your setter if the user is explicitly setting the default value in the children, it will be ignored... Is that the right thing to do? Should we give an error? So, lots of question I'd rather not deal with...
I understand our usage with Quarkus might not be directly supported, so closing is fine, but could you let me know if you think this should work properly with inherited options? If it doesn't, then I think there's still a bug, but it could be tracked in a new issue.


E2E: Add script to change webhook certificate when it's expired

To add admission webhook we need to create certificate for it and also run gql-controller with this certificate. So we need some script that replaces the certificate when it's expired or some other approach to add certificate.

Irrelevant for new apollo-based version


[Regression] Randomize options are not working.

Describe the bug
Randomize options are not working.
To Reproduce
Steps to reproduce the behavior. If applicable please include the smallest possible XLSForm or XForm (as zip or link).

Create a xls form, add choices questions, here is a sample xls form: Randomize_Select.xlsx.
Set to randomize options.
Publish and observe that the choices are not randomized.

Expected behavior
Randomize options can work.
Screenshots

Sure, I will create a PR for this issue.

@nanzhang01 sorry, this skipped my inbox :grimacing:
I don't recall how XLSForm converts XLS into XPath queries, but if it's requesting ORDERED_NODE_ITERATOR_TYPE or ORDERED_NODE_SNAPSHOT_TYPE with the randomize=true parameter then I'd say the bug is on that side.

@MartijnR is it possible this is enketo-core's responsibility?
I'm not really sure what I'm looking at, but maybe it always requests ORDERED_NODE_SNAPSHOT_TYPE?  See: https://github.com/enketo/enketo-core/blob/3e9d2ea59a090ca2ed39566e4a7a562ef79118a3/src/js/form-model.js#L1282
I can't see an obvious way to change this result type...

Ah I see. Oddly enketo-xpathjs seemed to be buggy there and returned a shuffled result for ORDERED_NODE_SNAPSHOT_TYPE. Indeed Enketo Core will always request the ordered result.
I agree that fix in ORXE would be hacky. Let me look into what potential issues could arise if Enketo Core always requested UNORDERED_NODE_SNAPSHOT_TYPE instead.

I think the Enketo Core can just request unordered results. If there are any consequences, they'll likely be less serious than the current situation. Will move this issue.


GitHub Integration: Don't commit block pngs to repo

When integrating a blocks project with GitHub, we add some pngs of the blocks to the repo (showing either the entire canvas or just the most recent diff). They look nice in the README, but are a source of problems for collaboration via GitHub. These are binary files, which make them a guaranteed source of merge conflicts when more than one user is committing to the repo.
I also removed the build status badge because it's never worked for me. If you know what the url should be, please let me know and I'll update it.
Test target for microbit: https://makecode.microbit.org/app/7881dff3cd82ba086e640657975c2cfb0eb3d4d9-df8e864a28
Test target for Arcade: https://arcade.makecode.com/app/d6e06a69c459bfeb811db0abe049789288cf62f8-58b4692010
Fixes https://github.com/microsoft/pxt-microbit/issues/5075

LGTM. It sounded like a hood idea but removing this is probably best.

And just to clarify, the block diffs still show up in makecode's github UI, we're just not committing them as files to the repo.


Glow on hover

Glow on hover buttons added

Closing this PR due to inactivity and the contributor's work being way behind the new updates. Thank you.


Foreign function gives segmentation fault only when called from SML/NJ

Version
110.99.4 (Latest)
Operating System

[ ] Any
[X] Linux
[ ] macOS
[ ] Windows
[ ] Other Unix

OS Version
Linux fedora 6.2.15-100.fc36.x86_64 #1 SMP PREEMPT_DYNAMIC Thu May 11 16:51:53 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux
Processor

[ ] Any
[ ] Arm (using Rosetta)
[ ] PowerPC
[ ] Sparc
[X] x86 (32-bit)
[ ] x86-64 (64-bit)
[ ] Other

System Component
Foreign-Function Interface (FFI)
Severity
Major
Description
Using 32-bit SML/NJ, which is required for the FFI, we have found that a foreign function called from SML/NJ gives a segmentation fault but works fine when called otherwise, e.g. via the MLton FFI.
Transcript
[user@hostname sml]$ sml -32
Standard ML of New Jersey (32-bit) v110.99.4 [built: Fri Jan 26 09:05:35 2024]
- CM.make "z3.cm";
[autoloading]
[library $smlnj/cm/cm.cm is stable]
[library $smlnj/internal/cm-sig-lib.cm is stable]
[library $/pgraph.cm is stable]
[library $smlnj/internal/srcpath-lib.cm is stable]
[library $SMLNJ-BASIS/basis.cm is stable]
[library $SMLNJ-BASIS/(basis.cm):basis-common.cm is stable]
[autoloading done]
[scanning z3.cm]
[library $c/c.cm is stable]
[scanning (z3.cm):z3-ffi.cm]
[attempting to load plugin $/make-tool.cm]
[library $/make-tool.cm is stable]
[library $smlnj/internal/cm-lib.cm is stable]
$Execute: required privileges are:
  cm-init
[library $smlnj/cm/tools.cm is stable]
[plugin $/make-tool.cm loaded successfully]
[make -f Makefile FFI/z3.cm SMLNJ_BINDIR=/opt/smlnj/smlnj-110.99.4/bin FFI/z3.cm]
make: 'FFI/z3.cm' is up to date.
make: 'FFI/z3.cm' is up to date.
[scanning (z3.cm):(z3-ffi.cm):FFI/z3.cm]
[library $c/internals/c-int.cm is stable]
[library $c/memory/memory.cm is stable]
...
$Execute: required privileges are:
  c-int
  primitive
[New bindings added.]
val it = true : bool
- Z3.Config.config ();
/opt/smlnj/smlnj-110.99.4/bin/sml: Fatal error -- bogus fault not in ML: pc = 0xf242eb62, sig = 11

Expected Behavior
No segmentation fault should occur.
Steps to Reproduce
Briefly, as I have already investigated this:

Build a 32-bit release version of Z3, say 4.8.12.
Create bindings using ML-NLFFI.
Call Z3_mk_config.

Additional Information
The segmentation fault occurs when the following MOVAPS instruction (indicated by =>) is executed:
(gdb) disass
Dump of assembler code for function _ZN15prime_generator22process_next_k_numbersEy:
   ...
   0xf071eb5f <+479>:	mov    %ebx,-0x7c(%ebp)
=> 0xf071eb62 <+482>:	movaps %xmm0,-0x68(%ebp)
   0xf071eb66 <+486>:	movd   %xmm0,-0x70(%ebp)
   ...

This MOVAPS copies %xmm0 to an address on the stack given by -0x68(%ebp) and requires the address to be aligned on a 16-byte boundary.  However we have
(gdb) print $ebp
$3 = (void *) 0xffffa900

which means that -0x68(%ebp) is not aligned on a 16-byte boundary.  The next instruction causes the processor to generate a general-protection exception (#GP), which the OS traps:
(gdb) ni

Program received signal SIGSEGV, Segmentation fault.
0xf071eb62 in prime_generator::process_next_k_numbers(unsigned long long) () from /tmp/test_z3_ffi/z3-z3-4.8.12/build32/libz3.so

-0x68(%ebp) is misaligned here because esp on entry to the function Z3_mk_config from SML/NJ is not aligned to a 16-byte boundary.  Restarting and breaking just inside Z3_mk_config we have:
Breakpoint 1, 0xf1b420d4 in Z3_mk_config () from /tmp/test_z3_ffi/z3-z3-4.8.12/build32/libz3.so
(gdb) disass
Dump of assembler code for function Z3_mk_config:
   0xf1b420d0 <+0>:	push   %ebp
   0xf1b420d1 <+1>:	mov    %esp,%ebp
   0xf1b420d3 <+3>:	push   %edi
=> 0xf1b420d4 <+4>:	call   0xf06b16d6 <__x86.get_pc_thunk.di>
   ...

(gdb) print $ebp
$1 = (void *) 0xffffa9d0
(gdb) print $esp
$2 = (void *) 0xffffa9cc

The value of esp before

push   %edi is 0xffffa9d0 (equal to the value of ebp);
push   %ebp is 0xffffa9d4;
the function is called (which pushes eip) is therefore 0xffffa9d8.

So it appears that SML/NJ is calling Z3_mk_config with esp equal to 0xffffa9d8 which is not aligned on a 16-byte boundary.  This is misaligned by 8 bytes, the same amount that -0x68(%ebp) was misaligned in the MOVAPS instruction.
GCC introduced a 16-byte alignment requirement for the x86 ABI some years ago presumably to allow faster SSE instructions that require alignment to a 16-byte boundary.  See this useful summary for more background on this requirement.
Email address
<EMAIL_ADDRESS>

I am able to work around this issue by specifying -march=i686 (in addition to -m32) when building Z3.

I'll take a look at it.  It may be easy to fix, since I believe that we already require 16-byte alignment for 32-bit macOS (see base/compiler/TopLevel/backend/x86-ccall.sml).

Fixed in 110.99.5.


tflite_flutter not found in .pub-cache directory

As per the installation instruction, we need to locate this directory
~/.pub-cache/hosted/pub.dartlang.org/tflite_flutter-<plugin-version>/
but even after I do flutter pub get after adding the dependency to pubspec file, this directory is not generated.
I tried doing
pod install
pod install --repo-update
flutter pub cache repair

I even tried to manually add
pod 'TensorFlowLiteC', '~> 2.2.0', in Podfile
But nothing seems to help.
As no one seems to have reported this issue earlier, am I missing something silly in this installation?

Having the same issue myself. I am only using android for development as of now. Any help would be appreciated.

As per the installation instruction, we need to locate this directory
~/.pub-cache/hosted/pub.dartlang.org/tflite_flutter-<plugin-version>/
but even after I do flutter pub get after adding the dependency to pubspec file, this directory is not generated.
I tried doing
pod install
pod install --repo-update
flutter pub cache repair

I even tried to manually add
pod 'TensorFlowLiteC', '~> 2.2.0', in Podfile
But nothing seems to help.
As no one seems to have reported this issue earlier, am I missing something silly in this installation?

Run --> flutter pub cache add tflite_flutter --version "0.9.0"

That also didn't work. The problem was that we were navigating to the folder from command line.
~/.pub-cache/hosted/pub.dartlang.org/tflite_flutter-<plugin-version>/ this command returns pub cache but not the one inside flutter. When I navigated to the directory path above from the flutter installation folder, tflite_flutter was there.

dart pub cache add tflite_flutter --version "0.9.0"

That also didn't work. The problem was that we were navigating to the folder from command line.
~/.pub-cache/hosted/pub.dartlang.org/tflite_flutter-<plugin-version>/ this command returns pub cache but not the one inside flutter. When I navigated to the directory path above from the flutter installation folder, tflite_flutter was there.

I also got stuck on this issue.  I think it deserves an update on README.

any updates about this issue? i also cant find tflite_flutter folder in that path

i fixed it!!, so all you need is re-instal the dart and flutter library from mac and then check again
~/.pub-cache/hosted/pub.dartlang.org/
the folder of tflute_flutter will be there as soon as you do flutter pub get on your flutter project

dart pub cache add tflite_flutter --version "0.9.0"

This worked for me! Thanks :D


complex indices for materials in write_sparameters_lumerical

Is your feature request related to a problem? Please describe.
Cannot use (n,k) Materials in Lumerical in the current state
Describe the solution you'd like
Enable passing a complex index to write_sparameters_lumerical (thru material_name_to_lumerical). This could be done by adding the following snippet to handle the complex index:
elif isinstance(material, complex):
            mat = s.addmaterial("(n,k) Material")
            s.setmaterial(mat, "Refractive Index", material.real)
            s.setmaterial(mat, "Imaginary Refractive Index", material.imag)
            s.setnamed(layername, "material", mat)

Note that setting the material in lumerical depending whether a str, int or float is used is repeated twice, once for the cladding and once for the layers. It might make sense to refactor that into a function.

Hi @Jan-David-Black ,
yes, that's a great idea, thank you for sharing,
How does this new branch work for you?
https://github.com/gdsfactory/gdsfactory/tree/559
I made a new function set_material() as you suggested

Looks Great! I am sorry I only throw stuff in here and don't just open a pull request, with everything set up properly. At some point I will find the time configure a "build environment" so that I can contribute more efficiently!

What was the problem with using complex instead of a tuple of floats?

now it can do both tuples and complex refractive index

now it can do both tuples and complex refractive index

ah understood :)


field shape wrong for native sequence types

Please complete:

Pydantic version import pydantic; print(pydantic.VERSION): master

While working on #1111 I noticed that field.shape is not set correctly for native sequence types:
from pydantic import BaseModel

class Model(BaseModel):
    a: list
    b: set
    c: frozenset
    d: dict
    e: tuple

for name, field in Model.__fields__.items():
    debug(name, field, field.shape)

"""
test.py:13 <module>
    name: 'a' (str) len=1
    field: ModelField(
        name='a',
        type=list,
        required=True,
    ) (ModelField)
    field.shape: 1 (int)
test.py:13 <module>
    name: 'b' (str) len=1
    field: ModelField(
        name='b',
        type=set,
        required=True,
    ) (ModelField)
    field.shape: 1 (int)
test.py:13 <module>
    name: 'c' (str) len=1
    field: ModelField(
        name='c',
        type=frozenset,
        required=True,
    ) (ModelField)
    field.shape: 1 (int)
test.py:13 <module>
    name: 'd' (str) len=1
    field: ModelField(
        name='d',
        type=dict,
        required=True,
    ) (ModelField)
    field.shape: 1 (int)
test.py:13 <module>
    name: 'e' (str) len=1
    field: ModelField(
        name='e',
        type=tuple,
        required=True,
    ) (ModelField)
    field.shape: 1 (int)
"""

Currently the only known consequence of this is that the schema will be wrong, I guess per_item also won't work.
Should be possible to fix without breaking other functionality.

@samuelcolvin I would like to work on the issue. As I have understood that field.shape is not having the correct value as the type defined in the model.

Great, yes please. Good luck!

@samuelcolvin I found the issue. In pydantic/fields.py:ModelField._type_analysis() where the field.shape  field get assign depending on the field type. getattr(self.type_, '__origin__', None) is returning None so the function is getting returned from there without assigning the value to field.shape variable.
https://github.com/samuelcolvin/pydantic/blob/e65d11240a6febcd569ad44a61e2374eb00b9ba6/pydantic/fields.py#L355-L358

It looks like if you just convert the various checks from lines ~382 - ~420 into a separate function, you could call that function on self.type_ when origin is None, and otherwise on origin, and get the right behavior.
@kousikmitra Would you like to submit a PR fixing this?

@dmontagu Yes i will open a PR. Thanks

The field.shape property is now removed in v2 and there is nothing similar, so I'm going to close this issue.


Common base class for extensibility and reduced boilerplate

This introduces two changes:

Make the paltas model classes (SourceBase, LOSBase, etc.) subclasses of a new BaseComponent class. This base class takes care of the common init, update, and parameter checking that was previously duplicated across components.
Refactor ConfigModel.get_lenstronomy_models_kwargs to remove the mostly-duplicated code for initializing and drawing from the different models. In particular, paltas classes now have a draw method that calls methods like add_lenses or add_sources from a common LenstronomyInputs object that is passed around. Besides cleaning things up, this will allow us to add features like painting galaxies onto subhalos.

I put the new base class in a top-level core.py. If you like that, maybe we can move the ConfigHandler stuff there too. We could also squirrel away the base class in an obscure utils file.
One remaining eyesore is the lens light handling. Perhaps we could implement lens light by adding a source model at the lens redshift, instead of collecting separate buckets of lenstronomy arguments? This would require more changes in the test suite though.
The tests will fail until https://github.com/swagnercarena/paltas/pull/38 (or at least https://github.com/swagnercarena/paltas/commit/94bfd3d83c8d3bf650dc53ffbcf093224edbcdd2) is merged and we rebase.

I experimented with a wrapper to convert Sources to lens light classes here: https://github.com/JelleAalbers/paltas/commit/9d1cee2102e73edbb156e7d3f5a4dd2d9c411583 (+ a few bugfixes in later commits). Let me know if you'd like something like that here (or in a future PR), I'm on the fence as to whether it actually made things clearer or not.

Hi Jelle, what is the status of this pull request? Are you waiting for another review on the changes?


Improve parameter handling take 2

Ping

cc @samdutton maybe you could take a look?

LGTM from a quick look (about to get on a plane).

Closing


Error Duplicate Column Name 'state_id'

Hi, i'm trying to install your plugin in my october cms. But, i got these error. Could you tell me how to fix it?

I got the same error.

The issue is that the Responsiv.Pay plugin requires Rainlab.UserPlus which has these same columns so just comment out state_id, city_id, phone in add_profile_columns_to_users_table.php until this is fixed

Same here

remove user plus plugin to resovel issue.

Hi guys, I think this problem is caused by RainLab.UserPlus plugin dependency. I've already fixed it on migration by checking the columns before creating them. Thanks for your feedback!


Scripts fail to load with a parse error on exported projects if it uses editor classes

Tested versions
v4.3.dev.custom_build [2042420bd]
System information
Windows 11
Issue description
Tools scripts using an editor class such as EditorInterface will fail to load with a parse error on exported projects.
@tool
extends Node3D

func _init() -> void:
	if Engine.is_editor_hint():
		var editor_filesystem = EditorInterface.get_resource_filesystem()
		if not editor_filesystem.resources_reimported.is_connected(_on_resource_reimport):
			editor_filesystem.resources_reimported.connect(_on_resource_reimport)

func _on_resource_reimport(resources:PackedStringArray):
	pass

Output of debug template console wrapper:
SCRIPT ERROR: Parse Error: Identifier "EditorInterface" not declared in the current scope.
          at: GDScript::reload (res://node_3d.gd:6)
ERROR: Failed to load script "res://node_3d.gd" with error "Parse error".
   at: ResourceFormatLoaderGDScript::load (modules\gdscript\gdscript.cpp:2894)

Well, it makes sense. Editor classes aren't included in distribution builds but there is no conditional compiling in GDScript so tool scripts with game and editor code can't be used at all in a project.
Workarounds:

Reject types, return to dynamic: var editor_interface = Engine.get_singleton("EditorInterface").
Distribute the source project with godot.exe.

Steps to reproduce
Export a project that uses the script above with a debug template and run the .console.exe version.
Minimal reproduction project (MRP)
N/A

Related:

https://github.com/godotengine/godot-proposals/issues/7375
https://github.com/godotengine/godot-proposals/discussions/4023

Reject types, return to dynamic: var editor_interface = Engine.get_singleton("EditorInterface").

IMO this is the way. You can do this indirectly for partial safety:
func _ready():
	if Engine.is_editor_hint():
		var editor = load("res://some_script_using_editor_interface.gd")
		editor.connect_reimported(_on_resource_reimport)

where the loaded script does what you quoted above. It can provide a type-safe interface with only the script in the scene doing unsafe calls.

It can provide a type-safe interface with only the script in the scene doing unsafe calls.

This will not work with static typing, only with dynamic typing (if you use Engine.get_singleton() and so on for all editor classes).
As a workaround, I would suggest my plugin gdscript-preprocessor (sorry for the self-promotion).
We could try to solve this out of the box by adding an exception for editor classes when analyzing/compiling the script. However, at runtime this should still give an error if execution reaches this point. So I think that this is a rather complex solution, which also adds inconsistency.

This will not work with static typing, only with dynamic typing (if you use Engine.get_singleton() and so on for all editor classes).

No I mean this (for some_script_using_editor_interface.gd from my example):
func connect_reimported(callback: Callable):
	var editor_filesystem := EditorInterface.get_resource_filesystem()
	if not editor_filesystem.resources_reimported.is_connected(callback):
		editor_filesystem.resources_reimported.connect(callback)

This code is type-safe, but it won't cause problems, because it's loaded using load(). The parent script does a dynamic call on this method. Hence I said "partially safe".

Also, this isn't mentioned in the documentation and works when running from the editor leaving it to be discovery possible late in development only after exporting.

Also related: #56798 (I just remembered)

Also: https://github.com/godotengine/godot/issues/86013

Got this happening with me on exported games on 4.3.
The first workaround suggested by @ydeltastar , worked so far.

Just discovered this. I have tool scripts in my game that do stuff in the editor, and already filter their actions based on the editor hint. We made EditorInterface static #75694 to make tool scripts easier. But to break an exported game because a script mentions, but doesn't use the class isn't good.
What about replacing editor classes with dummy classes?
How about recognizing a code block that starts with if Engine.is_editor_hint(): and not including it in the export?
What about giving us preprocessor defines that demark editor only code, to replace if Engine.is_editor_hint() entirely?

Python probably replaces it with a placeholder or an error opcode.

 Therefore, any non-existent identifiers do not cause a runtime error until execution reaches them.
GDScript combines the qualities of both dynamic and static typing. GDScript uses type hints and implicitly inferred types for static analysis and optimizations, and reports non-existent identifiers (but this does not apply to attribute/index access). Changing this behavior may be undesirable because it helps users find errors faster.
You can use Engine.get_singleton(&"EditorInterface") instead of EditorInterface, as documented in the description of the new @export_tool_button annotation. Similar techniques can be used for other cases besides accessing editor singletons.
But I agree that this is a pretty important point that can hinder users who want to have different code for different platforms, operating systems, distribution stores and other types of engine builds and custom feature tags. I think there are the following options to solve this problem:

Introduce a conditional compilation system in the form of preprocessor directives or another mechanism. This is quite complicated, but could potentially be useful for other scenarios.
Change the analyzer and compiler behavior only on export and in the exported project. Instead of a compile-time error, generate a "get global" opcode.
Replace the error about a non-existent identifier with a warning, which is treated as an error by default (for compatibility). I think this option is most consistent with the principles of GDScript.


🐛 fix: corrects the condition to fetch various resources

Description
This PR fixes the logic to fetch packages, devices and deployments based on a name, uuid and pattern. Release v7.0.0 has a bug that assumes all names to be patterns. For example, look at the following output
→ rio device list
Device ID                             Name          Status
------------------------------------  ------------  ----------
e93aaabf-0521-421a-bbb6-7249321e91c5  amit-testing  ONLINE
496c6c55-094f-4284-be01-5d9ca283d1c7  biswa1        ONLINE
005e3b19-929b-40ec-9230-6c624fc342c6  pallab        REGISTERED
f62a18a8-27c6-4b30-8463-93fb089a412d  pallab01      REGISTERED
6bddf8ff-dd85-4236-94e1-ee84fdc8959c  pallab011     REGISTERED

→ rio device delete pallab
Name       Device ID                             Status
---------  ------------------------------------  ----------
pallab011  6bddf8ff-dd85-4236-94e1-ee84fdc8959c  REGISTERED
pallab01   f62a18a8-27c6-4b30-8463-93fb089a412d  REGISTERED
pallab     005e3b19-929b-40ec-9230-6c624fc342c6  REGISTERED

Do you want to delete above device(s)? [Y/n]: n
Aborted!

This PR fixes that. Please look at the
→ pipenv run python -m riocli device delete pallab
Name    Device ID                             Status
------  ------------------------------------  ----------
pallab  005e3b19-929b-40ec-9230-6c624fc342c6  REGISTERED

Do you want to delete above device(s)? [Y/n]: n
Aborted!

→ pipenv run python -m riocli device delete pallab*
Name       Device ID                             Status
---------  ------------------------------------  ----------
pallab011  6bddf8ff-dd85-4236-94e1-ee84fdc8959c  REGISTERED
pallab01   f62a18a8-27c6-4b30-8463-93fb089a412d  REGISTERED
pallab     005e3b19-929b-40ec-9230-6c624fc342c6  REGISTERED

Do you want to delete above device(s)? [Y/n]: n
Aborted!

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


add device_uri for found network printers (attempt 2)

from https://github.com/OpenPrinting/cups/pull/397
i add "title" and i see tooltip if mouse over printer name.

Hi @Pro-pra ,
thank you for the PR! I've verified that I can see device uri once I have my mouse cursor above the entry for network printers.
It would be great if you squashed your commits in the meantime.
LGTM. @michaelrsweet wdyt?

I don't know how to do it now

@Pro-pra you can find SHA ID of the last commit which is not yours (here 411b6136f450a5 - you can check by git log) and then:
$ git reset --soft 411b6136f450a5
$ git commit -a
$ git push --force

In the commit step you will choose a commit message which covers all the changes you've done.

and open new pull request?
I think that a lot of PR is not needed for such a minor problem. Sorry.

No - there's no need for a new PR... otherwise I would've said it.

The PR pushed as commit https://github.com/OpenPrinting/cups/commit/23cef1c1374accc5d256b00b8afddffe645e4586 . Thank you for the PR!


False-positive on a rule that targets "chrome-extension://${id}"

@@||evernote.com^$domain=pioclpoplcdbaefihamjohnefbikjilc

@ameshkov We already have a basic domain validator in AGTree
I recommend improving this utility in the following aspects (new methods):

validation of domains without wildcards, like wildcard TLD
add method for IPs

@ameshkov We already have a basic domain validator in AGTree
I recommend improving this utility in the following aspects (new methods):

validation of domains without wildcards (like wildcard TLD) and check known TLDs
add method for IPs

We may can import known TLDs to a Set from here

I actually specifically don't use a normal validation, it's okay to check invalid domain names too as the tool then will suggest to remove them.
We just need to exclude some things like IP addresses when it does not make sense to check them.

IP addresses when it does not make sense to check them

at least ping check can be performed. No response = dead.
Also response can be checked response code:
curl -I -s -X GET http://<IP_ADDRESS>/ | grep -i "HTTP/1.1 301"
=> HTTP/1.1 301 Moved Permanently / 302. But not HTTP/1.1 403 Forbidden.

curl -I -s -X GET http://<IP_ADDRESS>/ | grep -i "HTTP/1.1"
=> HTTP/1.1 301 Moved Permanently / 302. But not HTTP/1.1 403 Forbidden.

There're more cases than this one.
First of all, there could be $network rules that can be for different ports/protocols.
Also, even when we deal with basic rules, it could be HTTPS like https://<IP_ADDRESS>.

But not HTTP/1.1 403 Forbidden

This is a working IP address, what's wrong with it.
It could be answering 403 to one request and respond with 200 OK to a different one for instance.


MatInput displays white as black

Reproduction
Use StackBlitz to reproduce your issue: https://stackblitz.com/fork/components-issue

Steps to reproduce:

Add a MatInput directive to an input within a mat-form-field
Give it type color
Do not enter a value for color

Expected Behavior
What behavior were you expecting to see?

Actual Behavior
What behavior did you actually see?
Angular Material component displays white as black i.e. if you don't enter a value for the form field the Material input will show black; however, when you actually submit the form, the real value will be white. This is true even after opening the colour picker, leaving the value as #000000 (black) and hitting OK without changing anything else in the color picker.
Environment

Angular: 7.1
CDK/Material: 7.1
Browser(s): Chrome, Firefox, MS Edge
Operating System (e.g. Windows, macOS, Ubuntu): Windows 10

appears to be fixed https://stackblitz.com/edit/components-issue-5fpk3o?file=src/app/example-component.html


[requires.io] dependency update on master branch

This is an automatically generated PR from requires.io.

I think this fails due to psutil 2.x being installed while we rely on 1.x. Syntax is completely different.
We use 1.x because that is what debian is shipped with.

We could probably consider the other requirements though.

I'll look into writing some filters for the requirements.txt file which should stop requires.io to open PRs with broken deps.


Density & Stacks - animation during a filter

Cubes should "fall" to the domain line. Need to add a Vega production rule for Y when cube is not selected.

Fixed in beta version https://sanddance.js.org/app-3.0.0-beta.1/


Add doc and example tests

readme.md
and the examples folder contain a lot of code that is currently not checked automatically.
This will become stale quickly. Bad!
Catch errors in these scripts in automated tests. Will need some sort of extraction of python code for readme.

Fixed in version 3


[Bug] ss: unrecognized option '--no-header'

I executed curvead deploy command and network check failed, reported the following error:
    command: sudo ss --listening --no-header --tcp '( sport = :2380 )'
    output: ss: unrecognized option '--no-header'

ss version in my server is
$ ss -v
ss utility, iproute2-ss130716

Hello, I will fix this issue.


No changing working directory if not needing

#1154  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @CatChen and the rest of your teammates on  Graphite


Authors Page not displaying new author (3rd)

Hi Ed,
could you please assist me in fixing this issue.
We are currently two authors and a third one is joining. We added him as a staff but he is not showing up next to us.
Thanks and best,
buarle

Hi,
He needs to be the author of a post or a page in order to appear in the Authors page.
Cheers.

Thanks that worked! It might be worth a note in the wiki.
Best regards!


rpc: fix regression in SendToRouteSync payment hash parsing

A recent change in this area of the codebase refactored some code to
make way for the deprecation of the streaming multi-route SendToRoute
method. Along the way, it removed the parsing of the PaymentHashString
field for the sync SendToRouteSync which is meant to be used via the
REST interface. In this commit, we fix this by populating both
PaymentHash and PaymentHashString. This is OK, as
extractPaymentIntent will check both values to extract the hex-encoded
payment hash if it's specified.

This is why github should start to show code only-moved 🙄


OpenCV Put Text

Hello,
I am having some issues writing words on top of the image. I am using the Hand Gesture recognition calculator to get an output stream of words.
constexpr char aslLettersTag[] = "ASL";
and then I added this line in ::mediapipe::Status HandGestureRecognitionCalculator::GetContract
if (cc->Outputs().HasTag("ASL")) {
cc->Outputs().Tag(aslLettersTag).Setstd::string();
}
and this line in ::mediapipe::Status HandGestureRecognitionCalculator::Process(
CalculatorContext *cc)
cc->Outputs()
.Tag(aslLettersTag)
.Add("Hello!!!!!!!!!", cc->InputTimestamp());
return ::mediapipe::OkStatus();
I then send this output stream as an input stream for the Open CV put text calculator.
However I get this error:
I0320 15:27:42.768482 205182400 demo_run_graph_main.cc:58] Initialize the calculator graph.
I0320 15:27:42.771833 205182400 demo_run_graph_main.cc:62] Initialize the camera or load the video.
I0320 15:27:44.733304 205182400 demo_run_graph_main.cc:83] Start running the calculator graph.
I0320 15:27:44.733888 205182400 demo_run_graph_main.cc:88] Start grabbing and processing frames.
I0320 15:27:44.977525 224604160 hand-gesture-recognition-calculator.cc:102] No Hand Detected
I0320 15:27:44.977602 227287040 hand-gesture-recognition-calculator.cc:102] No Hand Detected
I0320 15:27:45.244771 227287040 hand-gesture-recognition-calculator.cc:273] Finger States: 01101
I0320 15:27:45.244791 227287040 hand-gesture-recognition-calculator.cc:274] ___
I0320 15:27:45.244801 228896768 hand-gesture-recognition-calculator.cc:273] Finger States: 11101
I0320 15:27:45.244812 228896768 hand-gesture-recognition-calculator.cc:274] ___
F0320 15:27:45.244814 228896768 collection.h:53] Failed to get tag "ASL" index 0
hand_tracking_cpu(54746,0x70000d8c2000) malloc: *** error for object 0x109c8fec2: pointer being freed was not allocated
hand_tracking_cpu(54746,0x70000d8c2000) malloc: *** set a breakpoint in malloc_error_break to debug
Abort trap: 6
Anyone know how to fix this issue?

Seems like the graph config has some problem. You need to specify an output stream with tag "ASL" in the node of HandGestureRecognitionCalculator in your graph config.  For example,
node {
  calculator:HandGestureRecognitionCalculator
  input_stream: ...
  output_stream: ...
  output_stream: "ASL:asl_stream"
}

The current situation seems to be

The GetContract() function doesn't set up the "ASL" output stream for you because you ask it to do this only if the output stream with tag "ASL" is specified in the graph.
The process() function assumes that output stream with tag "ASL" always exists and it outputs something to that stream.

I believe I did that in render_cpu.pbtxt
I am trying to use this calculator: handgesture
and get the strings from here and input it in this calculator opencvputtext
I fixed some other issues however now I am getting this error
F0320 21:04:00.196940 17125376 collection.h:53] Failed to get tag "ASL" index 0
*** Check failure stack trace: ***
@        0x10c5c58ff  google::LogMessageFatal::~LogMessageFatal()
@        0x10c5c2019  google::LogMessageFatal::~LogMessageFatal()
@        0x10c07ac82  mediapipe::internal::CollectionErrorHandlerFatal<>::GetFallback()
@        0x10c3810bc  mediapipe::HandGestureRecognitionCalculator::Process()
@        0x10c3c0e07  mediapipe::CalculatorNode::ProcessNode()
@        0x10c3b679b  mediapipe::internal::SchedulerQueue::RunCalculatorNode()
@        0x10c3b6262  mediapipe::internal::SchedulerQueue::RunNextTask()
@        0x10c3ba149  mediapipe::ThreadPool::RunWorker()
@        0x10c3b9b45  mediapipe::ThreadPool::WorkerThread::ThreadBody()
@     0x7fff753f72eb  _pthread_body
@     0x7fff753fa249  _pthread_start
Abort trap: 6
Still seems like the same one but it has a different error message.

I think I realized that problem is that the  render_cpu.pbtxt is not recognizing the new nodes I have created while it runs. However I am still unsure...

It's because there is a commented line for the output_stream:"ASL:asl_letters" in the modified hand_landmark_cpu.pbtxt graph.
To fix this, either uncomment that line if appropriate, or add the following to your calculator's Process() function:
if (cc->Outputs().HasTag("ASL")) {
   cc->Outputs().Tag("ASL").AddPacket(
          MakePacket<std::string>(asl_letter)
              .At(cc->InputTimestamp()));

}

That builds the demo for me. Nice work! :)

Thank you. Now it works however the text still doesn't come on to the screen. I realized it is because the nodes I have created:
node {
calculator: "HandGestureRecognitionCalculator"
input_stream: "NORM_LANDMARKS:landmarks"
input_stream: "NORM_RECT:rect"
output_stream: "ASL:text_to_put"
}
node {
calculator: "OpenCvPutTextCalculator"
input_stream: "text_to_put"
output_stream: "out_image_frames"
}
Draws annotations and overlays them on top of the input images.
node {
calculator: "AnnotationOverlayCalculator"
input_stream: "INPUT_FRAME:input_image"
input_stream: "detection_render_data"
input_stream: "landmark_render_data"
input_stream: "rect_render_data"
output_stream: "OUTPUT_FRAME:output_image"
}
The output stream from OpenCvPutTextCalculator should be the input stream for AnnotationOverlayCalculator. However, if I try to pass it in, the program breaks. Is there any way to change the output stream to render data type and add it as an input stream for AnnotationOverlayCalculator or is there an easier way?

Yes, instead of using an OpenCvPutTextCalculator, use a simple StringToRenderDataCalculator that generates a new RenderData annotation object for an std::string.
I wrote a quick StringToRenderDataCalculator that you can try out. Copy it to the hand-gesture-recognition directory, and add the following changes to the renderer_cpu.pbtxt:

Comment out/Remove the OpenCvPutTextCalculator node.
Add the following node:node {
  calculator: "StringToRenderDataCalculator"
  input_stream: "TEXT:text_to_put"
  output_stream: "RENDER_DATA:text_render_data"
}

Add the text_render_data as an input stream to the AnnotationOverlayCalculator node:node {
  calculator: "AnnotationOverlayCalculator"
  input_stream: "INPUT_FRAME:input_image"
  input_stream: "detection_render_data"
  input_stream: "landmark_render_data"
  input_stream: "rect_render_data"
  input_stream: "text_render_data"
  output_stream: "OUTPUT_FRAME:output_image"
}

Change the output of the hand-gesture-recognition-calculator.cc to output a string containing one of the detected letters instead of the string "Hello!".
Rebuild the app via
bazel build -c opt --define MEDIAPIPE_DISABLE_GPU=1 \
    mediapipe/examples/desktop/hand_tracking:hand_tracking_cpu

and run it using
GLOG_logtostderr=1 bazel-bin/mediapipe/examples/desktop/hand_tracking/hand_tracking_cpu \
    --calculator_graph_config_file=mediapipe/graphs/hand_tracking/hand_tracking_desktop_live.pbtx

Here is a screenshot of one output frame I got using your graph:

Hi,
So I tried implementing your changes however I get this error:
GLOG_logtostderr=1 bazel-bin/mediapipe/examples/desktop/hand_tracking/hand_tracking_cpu     --calculator_graph_config_file=mediapipe/graphs/hand_tracking/hand_tracking_desktop_live.pbtx
E0321 14:32:09.723388 122648000 demo_run_graph_main.cc:163] Failed to run the graph: ; Can't find file: mediapipe/graphs/hand_tracking/hand_tracking_desktop_live.pbtx
Any ideas how to fix this?

There was a typo, fixed it, it should end with pbtxt...

Awesome thanks it works! Should this also work on mobile applications as well?

Actually I tried building the android application however it keeps crashing when I add these changes to the gpu files. Is there something extra I have to add?

Actually nevermind I made a mistake it works! Thank you very much !!

Grea! Closing the issue. Feel free to open if you have more questions.


rollup is broken

it renames fetch to fetch$1, which breaks everything, and there is no way to make it not do that from the looks of it, so either rollup gets taken out (I'd be all for that) or the fetch code is made a LOT more complex just to deal with rollup's insistence that a not-protected name fetch is a protected name.

Does this mean lib-font.js currently doesn't have any of the new code, as it can't be rolled up? Any way to use the latest code with the latest master/npm package?

lib-font.js does, there just isn't a dist dir anymore for the moment for prebundled legacy use

Is this still relevant? 🤔

not as far as I'm concerned.


add maven-central badge

Coverage decreased (-0.003%) to 86.657% when pulling 0337aa3a705685af5fb0a873c1ff146836d6c03e on bwaldvogel:master into 32c4d8a04244fd54f6872d12d10628283e6738e9 on JSQLParser:master.

I accidentally used squash and merge. Sorry. Therefore your banch was not merged but a new commit of your changes was produced and merged. You should update to the main branch and not reuse your "change branch".
thx for using JsqlParser

Thanks, nevermind ;)


Not able to access application using Secure Connection HTTPS from outside, HTTP is working fine, Using ISTIO

General information

OS: Windows
Hypervisor: Hyper-V
Did you run crc setup before starting it (Yes/No)? Yes
Running CRC on: Laptop

CRC version
# Put `crc version` output here
CRC version: 2.23.0+ddcfe8
OpenShift version: 4.13.3
Podman version: 4.4.4

## CRC status
```bash
# Put `crc status --log-level debug` output here
DEBU CRC version: 2.23.0+ddcfe8
DEBU OpenShift version: 4.13.3
DEBU Podman version: 4.4.4
DEBU Running 'crc status'
CRC VM:          Running
OpenShift:       Running (v4.13.3)
RAM Usage:       12.71GB of 16.77GB
Disk Usage:      26.35GB of 32.68GB (Inside the CRC VM)
Cache Usage:     21.32GB

## CRC config
```bash
# Put `crc config view` output here
- consent-telemetry       : no
- cpus                             : 8
- memory                       : 16384

## Host Operating System
```bash
# Put the output of `cat /etc/os-release` in case of Linux
# put the output of `sw_vers` in case of Mac
# Put the output of `systeminfo` in case of Windows
Host Name:
OS Name:                   Microsoft Windows 10 Enterprise
OS Version:                10.0.19045 N/A Build 19045
OS Manufacturer:           Microsoft Corporation
OS Configuration:          Standalone Workstation
OS Build Type:             Multiprocessor Free
Registered Owner:
Registered Organization:
Product ID:                00330-80000-00000-AA761
Original Install Date:     05/07/2023, 17:21:39
System Boot Time:          14/08/2023, 08:28:00
System Manufacturer:       HP
System Model:              HP ZBook Firefly 16 inch G9 Mobile Workstation PC
System Type:               x64-based PC
Processor(s):              1 Processor(s) Installed.
                           [01]: Intel64 Family 6 Model 154 Stepping 3 GenuineIntel ~1800 Mhz
BIOS Version:              HP U71 Ver. 01.08.01, 18/07/2023
Windows Directory:         C:\windows
System Directory:          C:\windows\system32
Boot Device:               \Device\HarddiskVolume1
System Locale:             en-us;English (United States)
Input Locale:              en-gb;English (United Kingdom)
Time Zone:                 (UTC+00:00) Dublin, Edinburgh, Lisbon, London
Total Physical Memory:     32,400 MB
Available Physical Memory: 4,083 MB
Virtual Memory: Max Size:  42,640 MB
Virtual Memory: Available: 8,014 MB
Virtual Memory: In Use:    34,626 MB
Page File Location(s):     C:\pagefile.sys
Domain:                    WORKGROUP
Logon Server:              \\GB-5CG30917V4
Hotfix(s):                 9 Hotfix(s) Installed.
                           [01]: KB5027122
                           [02]: KB5012170
                           [03]: KB5015684
                           [04]: KB5026037
                           [05]: KB5028166
                           [06]: KB5014032
                           [07]: KB5020372
                           [08]: KB5026879
                           [09]: KB5028318
Network Card(s):           7 NIC(s) Installed.
                           [01]: Intel(R) Wi-Fi 6E AX211 160MHz
                                 Connection Name: Wi-Fi
                                 Status:          Media disconnected
                           [02]: Bluetooth Device (Personal Area Network)
                                 Connection Name: Bluetooth Network Connection
                                 Status:          Media disconnected
                           [03]: Lenovo USB Ethernet
                                 Connection Name: Ethernet 2
                                 DHCP Enabled:    Yes
                                 DHCP Server:     <IP_ADDRESS>
                                 IP address(es)
                                 [01]: <IP_ADDRESS>
                                 [02]: fe80::2eb:fe4f:af21:a2e6
                           [04]: Hyper-V Virtual Ethernet Adapter
                                 Connection Name: vEthernet (Default Switch)
                                 DHCP Enabled:    No
                                 IP address(es)
                                 [01]: <IP_ADDRESS>
                                 [02]: fe80::2b12:abb9:bd42:3885
                           [05]: Wintun Userspace Tunnel
                                 Connection Name: Appgate SDP
                                 DHCP Enabled:    No
                                 IP address(es)
                                 [01]: <IP_ADDRESS>
                           [06]: Zscaler Network Adapter <IP_ADDRESS>
                                 Connection Name: Ethernet 4
                                 Status:          Media disconnected
                           [07]: Hyper-V Virtual Ethernet Adapter
                                 Connection Name: vEthernet (WSL)
                                 DHCP Enabled:    No
                                 IP address(es)
                                 [01]: <IP_ADDRESS>
                                 [02]: fe80::9e42:1841:1177:5553
Hyper-V Requirements:      A hypervisor has been detected. Features required for Hyper-V will not be displayed.

### Steps to reproduce

  1.  Create HTTPs gateway
  2. Create HTTPS VS
  3. Test Https application link - https://[bookinfo-bookinfo-gatewayhttps-684888c0ebb17f37-istio-system.apps-crc.testing](https://bookinfo-bookinfo-gatewayhttps-684888c0ebb17f37-istio-system.apps-crc.testing/productpage)/productpage
  4.

### Expected
Application page

The Comedy of Errors
Summary: [Wikipedia Summary](https://en.wikipedia.org/wiki/The_Comedy_of_Errors): The Comedy of Errors is one of William Shakespeare's early plays. It is his shortest and one of his most farcical comedies, with a major part of the humour coming from slapstick and mistaken identity, in addition to puns and word play.

Book Details
Type:
paperback
Pages:
200
Publisher:
PublisherA
Language:
English
ISBN-10:
1234567890
ISBN-13:
123-1234567890
Book Reviews
An extremely entertaining play by Shakespeare. The slapstick humour is refreshing!

Reviewer1

Absolutely fun and entertaining. The play lacks thematic depth when compared to other plays by Shakespeare.

Reviewer2

Reviews served by:
reviews-v3-55f9d7445c-8pfvk

### Actual

Hmmm… can't reach this pageIt looks like bookinfo-bookinfo-gatewayhttps-684888c0ebb17f37-istio-system.apps-crc.testing closed the connection
Try:

Checking the connection
[Checking the proxy and the firewall](chrome-error://chromewebdata/#buttons)
[Running Windows Network Diagnostics](javascript:diagnoseErrors())
ERR_CONNECTION_CLOSED

### Logs

Before gather the logs try following if that fix your issue
```bash
$ crc delete -f
$ crc cleanup
$ crc setup
$ crc start --log-level debug
``
<script src="https://gist.github.com/santoshnt1/6d2ad1f9cdf444cc5b6a86c4e1014853.js"></script>
Please consider posting the output of `crc start --log-level debug`  on http://gist.github.com/ and post the link in the issue.

Do you get the same 'connection closed' error when using curl to access https://bookinfo-bookinfo-gatewayhttps-684888c0ebb17f37-istio-system.apps-crc.testing/productpage? I'm wondering if this could be an issue with the self-signed certificate crc is using for its cluster.

By default http based route is created, if you want to have different way like https then you need to do some manual steps as part of route config https://docs.openshift.com/container-platform/4.13/networking/routes/route-configuration.html mentions different type of route configurations in OpenShift.

By default http based route is created, if you want to have different way like https then you need to do some manual steps as part of route config https://docs.openshift.com/container-platform/4.13/networking/routes/route-configuration.html mentions different type of route configurations in OpenShift.

Hi Praveen, the solution worked for me. Thanks for the help Many Thanks Santosh Kumar


make @firebase/app a devDependency

It's a sibling PR to https://github.com/firebase/firebase-js-sdk/pull/1916

@hiranya911 This PR is now ready to be merged.


add dispenser

Closes #90

waiting for a new ocean.js, in order to have tests as well

I tried running it locally with Barge but I got an error, Let me know if I did anything wrong, I ran:
npm install
export ADDRESS_FILE="${HOME}/.ocean/ocean-contracts/artifacts/address.json"
npm run bargesetup
npm run create:local-barge
npm run deploy:local-barge

Barge is running in a separate terminal with ./start_ocean.sh --with-thegraph

FIxed now.
@jamiehewitt15 there was one step missing, exporting the address file (it was updated in main).

FIxed now.
@jamiehewitt15 there was one step missing, exporting the address file (it was updated in main). You are missing npm run codegen as well.
Need to fix the tests

Thanks, I'll test it again

Seems to work well now. By the way, I added the npm run codegen command to the readme in the instructions for running it on Barge

It seems like the tests are failing slightly randomly, I have run them a couple of times locally and they pass sometimes but not all the time. The sleep time may need to be increased

@jamiehewitt15 yep, it depends on what resources the graph has. I increased all sleep times, hopefully 3 seconds is enough

Ready from my point of view. @alexcos20 remove the WIP label if no more work is needed here.

Seems ok to me too, I have approved