Move background to React component (#314)

Hey,
I really like this project and felt like contributing something to get a feel for the codebase. I tried to follow your existing codestyle, but feel free to ask me to change anything/everything. I tried to add the Background component to the RootComponent, but that didn't really work out too well since the text editor needs to be "in between" the background and the rest of the React UI in terms of DOM nodes. I couldn't really figure out how to do it differently, but I'm not really a React expert yet.
Simon

We have a linter that checks for style errors as part of the build process.  Your commit failed for these (minor) issues:
oni/browser/src/UI/Reducer.ts[30, 32]: trailing whitespace
oni/browser/src/UI/Reducer.ts[32, 58]: Missing trailing comma

Oops, I missed that. Fixed.

Thanks for the kind words on the project! The change looks great to me.

I tried to add the Background component to the RootComponent, but that didn't really work out too well since the text editor needs to be "in between" the background and the rest of the React UI in terms of DOM nodes

Yes, this is definitely painful. I like the solution you chose for now - just having ReactDOM.render directly to another element. I'm working on another PR - #303 - which refactors the UI structure a bit, so we end up with something like this:
export class RootComponent extends React.Component<IRootComponentProps, void> {

    public render(): JSX.Element {

        return

                <EditorHost editor={this.props.editor} />
                <Cursor />
                <CursorLine lineType={"line"} />
                <CursorLine lineType={"column"} />
 ...

                <StatusBar />

Your solution here will fit nicely with that PR because we can just pick up your  component and place it in the right level there.

Looks great, nice work & thanks for the contribution @FrigoEU ! I'll merge this in.


No Option to Set Required Domain Name

I tried to use this module to create an ElasticSearch cluster. I did not specify a domain name and the domain name is not a required parameter (dns_zone_id), but the plan command still failed with the following message:
Error: module.elasticsearch.aws_elasticsearch_domain.default: "domain_name" must start with a lowercase alphabet and be at least 3 and no more than 28 characters long. Valid characters are a-z (lowercase letters), 0-9, and - (hyphen).

It does not even work after setting dns_zone_id and kibana_subdomain_name values. I still get the same error message.

@Jeeppler thank you for testing.
Please take a look how we deploy the module in real applications:
https://github.com/cloudposse/terraform-root-modules/blob/master/aws/backing-services/elasticsearch.tf
So domain_name is different from dns_zone_id.
domain_name is the name of the Elasticsearch cluster, and it's required.
https://github.com/cloudposse/terraform-aws-elasticsearch/blob/master/main.tf#L61
We use the label module to uniquely name all resources:
https://github.com/cloudposse/terraform-aws-elasticsearch/blob/master/main.tf#L1
dns_zone_id is the ID of Route53 DNS zone which you can specify if you want to create friendly subdomain names for the Elasticsearch cluster itself and for Kibana (leave it empty if you don't want to create any Route53 subdomains).
https://github.com/cloudposse/terraform-aws-elasticsearch/blob/master/main.tf#L129
https://github.com/cloudposse/terraform-aws-elasticsearch/blob/master/main.tf#L140

More specifically, the namespace, stage and name variables must also satisfy these requirements so that the generated domain name is valid.

@Jeeppler feel free to find us at https://slack.cloudposse.com if you're still not able to get it working.

The problem is that elasticsearch domain should be
module.elasticsearch.aws_elasticsearch_domain.default: "domain_name" must start with a lowercase alphabet and be at least 3 and no more than 28 characters long. Valid characters are a-z (lowercase letters), 0-9, and - (hyphen).

We need option to reduce at least name
I'm about to create PR for that

@Jeeppler  did it work for you?

@aknysh I did not continue using this module. I never tried it further.


Migrate to new _rules API

Description
For inspect policies. We should use the newer API that scales better and work in global.
We can also discuss design a little if we review this as some outputs are a bit better.
https://github.com/kumahq/kuma/blob/bf05cb9456e0456e70ed8769dc88c18ad75e903c/api/openapi/specs/api.yaml#L20-L47

Some more notes on this:

Show the policy entries above the "Rules" heading only in non-global mode. They already don’t make sense in global mode and are being phased out. But for standalone/zone mode, they still are of some value.
Replace the rule entries below the "Rules" heading with new content based on the new _rules API:

Group rules in three subsections: Proxy rules, To rules, and From rules
Group from rules by their inbound first and then by policy type (as we do for the current rule entries)
Group proxy rules and to rules by policy type
Matchers behavior:

Empty array means everything
Matchers are always and-chained (i.e. &&)
The representation for a couple of matchers should use the form !kuma.io/service:foo && !kuma.io/service:bar && kuma.io/service:baz


[3] Upgrade Opensearch app version to 2.4.1

What Chart should be upgraded?
I propose that we should upgrade Opensearch and dashboards to version 2.4
Helm chart opensearch-2.9.1 and opensearch-dashboards-2.7.0 defaults to app version 2.4.1
Why should this Chart be upgraded?

[ ] We need to perform a security upgrade.
[ ] We need it for a new feature: which feature (link to blocked issue).
[ ] We need to keep up to prevent future issues.
[x] Other: They have introduced IMO two new features that could be helpful

New functionality in OpenSearch Dashboards lets you restore data from existing snapshots. Previously, restoring snapshots required using the restore API.
[experimental] With searchable snapshots, you can now search indexes that are stored as snapshots within remote repositories without needing to download all of the index data ahead of time.
support for multiple authentication options

Acceptance criteria

I checked the migration of the new Chart:

[ ] I upgraded a Chart and determined that no migration steps are needed.
[ ] I upgraded a Chart and added migration steps.

[ ] I tested the functionality provided by the new Chart (e.g., Auth flow, Grafana dashboards, etc.)

check the changelog, but try to update the chart and app to a newer version v2.5+ (maybe the last one if it's stable enough)


Controller not displaying image form s3 bucket

Laravel Version: 5.6
PHP Version: 7.2.10

Description:
I want to display an image from my amazon s3 bucket directly in the users browser from my controller. For this purpose I'm using Laravel File Storage . From the documentation it seems like my code should work but in my browser it displays nothing. It also doesn't show any errors, only a black screen in the browser.
Here is my controller
 class S3ImageController extends Controller
{
    public function imageUrl($make, $model, $year) {

         $stock = DB::table('stock_car_images')
                ->where(compact('make', 'model', 'year'))
                ->first();

         if (!empty($stock) && Storage::disk('s3')->url($stock->image)) {
            return response(Storage::disk('s3')->get($stock->image))->header('Content-Type',
           'image/jpeg');
          }
          else {
               return 'No Image';
       }
   }
}

Am i missing something or not?

Am i missing something or not?

Yes:

This repository is only for reporting bugs or problems. If you need help, see: https://github.com/laravel/laravel#learning-laravel

I guess this is a bug or a problem

Can you first please try one of the following support channels? If you can actually identify this as a bug, feel free to report back.

Laracasts Forums
Laravel.io Forums
Discord
Larachat
IRC
StackOverflow


ci: build and run tests on windows

As some of the tests are interactive on windows they aren't build and run at the moment.

Enabled since a while now. Closing.


test: [M3-7071] - Add Cypress integration tests for NodeBalancer Create DC-specific pricing

Description 📝
Add new cypress tests for NodeBalancer creation tiered pricing.
Major Changes 🔄

Add e2e tests to verify the DC pricing information in NodeBalancer creation flow.

How to test 🧪
yarn cy:run -s "cypress/e2e/core/nodebalancers/smoke-create-nodebal.spec.ts"

It would be nice to test the submission of multiple nodes and multiple configs since this form is complex. It will help us when refactoring it from a class component.

@abailly-akamai Fair, but this sounds like a good candidate for a follow up ticket to me, because it extends beyond DC-specific pricing. I think we can merge this as is, and get something in the backlog for further NodeBalancer Create test coverage.

@cliu-akamai - Thanks for your work on this! I'm going to go ahead and merge since this test passes in CI and locally for at least two of us. (Not quite sure what @hana-linode was experiencing, but can't replicate it.) I'll then merge develop into #9660 and make the adjustment we need in that PR to avoid a test failure.


fix nightly docs build

I believe it was broken by changes in #1450
Bring back all the extensions that that commit removed.
Towards #1463

Just started a dry run of the docs workflow: https://github.com/opendp/opendp/actions/runs/8554825873


Create temp

bla-bla-bla

this message was sent automatically.

this message was sent automatically.

@intellij-rust/technical-writers, seems like it's time to write the documentation!


Replace names size and indexes

          +1 for `grid_size` and for `bands`!

Originally posted by @rhugonnet in https://github.com/GlacioHack/geoutils/issues/440#issuecomment-1904244023
Following #440, it was decided to rename size (in reproject and proximity) by grid_size and indexes (in load and split_bands) by bands, therefore completely reverting back what was done in previous PRs. 🤦♂

The new aspects being:

The argument names grid_size, size and dst_size were all used before, recently changed to size, so we're making it all consistent but also clearer with grid_size because size is such a common Python name,
We would only replace indexes into bands for the argument names of the two functions mentioned above, and not for internal (non-public) variables, to stay consistent with Rasterio internally.

If that's what you also had in mind.
So we're not completely reverting! :stuck_out_tongue_winking_eye:

You're right, I was a bit exaggerating with the term "completely reverting" ! I could not find a funny way to describe our indecision 😉


Added fetch_audio_file method

On my spotify-based application, I don't want just want to just playback the audio stream, but also to fetch it for playing back later (offline).
In order to do so, I have split the flow of load_track in half:

The part responsible for fetching and decrypting the audio file is now on get_track_file, and is shared with the implementation of fetch_audio_file
The part that skips the first few bytes of the file (Ogg headers?) and creates a decoder continues implemented within load_track.

Actually, I wanted to open this PR on the upstream project :man_facepalming:


[CRUD] MSP

Implement the subj. group of methods according to the spec

MSP combined with the legal_entity


Doesnt work, even though recent has never been tested, waste of people time, very poor

Code Doesnt work, even though recent has never been tested, waste of people time, very poor

Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: 85af7a99-672c-0438-dcc2-e8edfaf70c54
Version Independent ID: 85af7a99-672c-0438-dcc2-e8edfaf70c54
Content: Use ASP.NET Core SignalR with Blazor
Content Source: aspnetcore/blazor/tutorials/signalr-blazor.md
Product: aspnet-core
Technology: aspnetcore-tutorials
GitHub Login: @guardrex
Microsoft Alias: riande

Hello @weedylan1 ... Yes, I'm not sure if it was tested for 7.0, and I do see that the SDK prereq should be updated. However, "it doesn't work" doesn't give me very much to go on: Which .NET version are you using ... 7.0? Which hosting model ... Blazor Server? What is the error/behavior?

It works here @weedylan1 (7.0, Blazor Server) ...

I'll close because your report doesn't include enough information to remark on. However, I would like to fix the SDK remark. This issue will close automatically when the PR merges in a few minutes.

Hi Luke, the code under Blazor Webassembly is incorrect, the Program.cs is
not correct for WebAssembly.
the Server code also has 2 issues, firstly the SignalR imports, which
someone experienced should be able to work out.
the main issue on the Blazor Server code is in the OnInitializedAsync()
method, that code will fail because of the StateHasChanged call.
this will return.
{"The current thread is not associated with the Dispatcher. Use
InvokeAsync() to switch execution to the Dispatcher when triggering
rendering or component state."}
change it to something like
protected override async Task OnInitializedAsync()
{
if (hubConnection == null)
{
hubConnection = new HubConnectionBuilder()
.WithUrl(Navigation.ToAbsoluteUri("/chathub"))
.Build();
        hubConnection.On<string, string>("ReceiveMessage", (user,

message) =>
{
var encodedMsg = $"{user}: {message}";
messages.Add(encodedMsg);
try
{
InvokeAsync(Refresh);
}
catch (Exception ex)
{
Console.WriteLine(ex);
}
});
        await hubConnection.StartAsync();
    }
}
private async Task Refresh()
{
    StateHasChanged();
}

Please modify the example and update the WebAssembly version especially.
Kind Regards
Dan
On Wed, 3 May 2023 at 11:32, Luke Latham @.***> wrote:

Closed #29154 https://github.com/dotnet/AspNetCore.Docs/issues/29154 as
completed via #29155
https://github.com/dotnet/AspNetCore.Docs/pull/29155.
—
Reply to this email directly, view it on GitHub
https://github.com/dotnet/AspNetCore.Docs/issues/29154#event-9153963827,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A7M4PAXWRCY7KPRXHML5H7TXEIX3XANCNFSM6AAAAAAXUFQZ5I
.
You are receiving this because you were mentioned.Message ID:
@.***>

Can you assist with this part of the issue?

No. I don't know blazor very well.
@MackinnonBuck maybe?

@MackinnonBuck ... If you want to work an issue on the PU repo for whatever the problem is, I can work on the Program.cs versioning problem off of this issue. @weedylan1 can open a PU issue for the error ...

The current thread is not associated with the Dispatcher. Use InvokeAsync() to switch execution to the Dispatcher when triggering rendering or component state.

I don't get that error here with the code that we're showing. The sample app runs fine AFAICT.

@weedylan1 ... I just checked and the Program.cs content is correct ...
https://github.com/dotnet/aspnetcore/blob/main/src/ProjectTemplates/Web.ProjectTemplates/content/ComponentsWebAssembly-CSharp/Server/Program.cs
Do you mean that you think the entire file is shown? That's not the case per the article text. It leads into the code with (emphasis added) ...

In Program.cs ...

This code is only meant to show the relevant part of Program.cs where the updates are applied. The text instructions are to that effect, namely to add the Response Compression Middleware at the top of the processing pipeline and where to add the hub. I feel good about leaving this content in its current form. If I receive more feedback asking for a change, I'll try a version that refers to the locations of added lines by their adjacent lines.
WRT your "imports" remark, I don't have anything actionable at this point. You didn't explain what was confusing. The packages and namespaces are called out, and there are no changes to the app's default _Imports.razor file.

@guardrex Thanks for looking into this! I think it's fine if we close this out, since there doesn't seem to be a problem either in the docs or in the product.

cc: @bradygaster ... I just noticed that that error reported here by @weedylan1 for the Blazor-SignalR tutorial is related to something we discussed a while back on ...
https://github.com/dotnet/AspNetCore.Docs/issues/25584
... that led to wrapping the call to StateHasChanged in InvokeAsync.
@weedylan1 is reporting the persistence of the error that you called out ...

The current thread is not associated with the Dispatcher. Use InvokeAsync() to switch execution to the Dispatcher when triggering rendering or component state.

... but I can't reproduce it here. The code for the example can be seen at ...
https://github.com/dotnet/blazor-samples/blob/main/7.0/BlazorServerSignalRApp/Pages/Index.razor#L47
I'm going to leave this issue closed for the time being, and I'll re-open it if you let me know that additional investigation/work should take place.

Why on Hub ( /ChatHub ) the value of Context.USer.Identity is always be null on Server mode, i want to make private chatt app that target to specified user account, it's work on Web Assembly with Persistent Authentication State, but on Blazor server the value of user identity is always null, and no chance to send the message directly to specified user account

@MmMapIoSpace ... We don't work on closed issues/PRs. You should open a new issue, and it sounds like in this case that you need some product support. Open an issue for the product unit at ...
https://github.com/dotnet/aspnetcore/issues
They'll probably need a minimal GitHub repro project cross-linked in your issue so that they can see your code.
Please add the following to the bottom of your opening comment so that I can follow the discussion:
cc: @guardrex

If it turns out that we're missing some documentation, I'll open a new documentation issue to address it after I see how your support issue goes.


Unauthorized to create new websites

Describe the Bug
I've been running Umami for some months already in Kubernetes. I'm not sure after which update, but lost the ability to create new websites. Whenever I try, either as a regular user or as admin, I get Unauthorized.

Database
PostgreSQL
Relevant log output
I can't see anything related to this in the logs.

Which Umami version are you using? (if relevant)
v2.13.2, v2.12.1, v2.11.3, v2.10.2
Which browser are you using? (if relevant)
Chrome
How are you deploying your application? (if relevant)
Docker (Kubernetes) https://github.com/christianhuth/helm-charts/tree/main/charts/umami

Today I wiped my database thinking the issue could be related to a broken migration.
Even from scratch I'm not able to create new websites.
The only difference in my setup is that I'm using Postgres with pgpool.
Maybe the latest versions of Umami have issues with this?

I'm having the same issue, freshly deployed on Kubernetes. Logs say
2024-11-12T16:33:28.011Z umami:middleware useAuth: User not authorized
But basically no team or website can be created.

Same here! and others having same issue: https://github.com/umami-software/umami/issues/3000

The same 😞  Deployed with the helm chart

Is this still happening in the latest version?

Is this still happening in the latest version?

Yes, tested on v2.14.0

Must be something specific to Kubernetes. I can run the it from source and docker just fine.

I can confirm must be something with the helm chart - redeployed in Kubernetes without the helm chart and it works.


Make Texts Attribute Again

In vanilla, texts (long elaborate human texts, not only character fields) are model classes that are linked to entities. It was designed like this out of necessity because the entities in vanilla were inflexible and various projects needed various different texts for entities like works.
In the rdf architecture I propose to remove this design and attach texts as attributes to entities directly, because assuming either possible requirements:
requirement A: given an ontology where a text is an attribute of an entity (or somewhat closely dependent), this would be better reflected by designing texts as attributes in the respective apis-ontology.
requirement B: given an ontology where a text is an independent entity, this can be equivally well reflected in an apis-ontology, where we would just define a model class that only has a text as attribute and the model class acts as its carrier.
Since both requirements can be fullfilled by attaching texts as attributes I see no benefit in keeping it the vanilla way.
Additionally within vanilla a text-type is an instance of a model class TextType. However semantically a text-type is rather a class, hence this design creates semantic frictions.
If we attach texts as attributes to entities, this would have side-effects on the apis-highlighter module as it is wired onto the current Text model classes. However the resulting refactoring should be of acceptable work-load.

This could touch upon highlighter refactoring: https://github.com/acdh-oeaw/apis-core-rdf/issues/75

Relevant TODO in code: https://github.com/acdh-oeaw/apis-core-rdf/blob/eb0d4bc80635d55e9ba7aa000ff614a0759c0f3a/apis_core/apis_metainfo/models.py#L417

Text has been removed, highlighter has been replaced by highlighter-ng, ontologies can now put custom non enitity models in their models.py...
I'm therefore closing this


Logo to main page

Have the Horreum logo on the top left, and clicking it will bring you to the main page

The same will happen when you click Tests (that's the main page anyway).

Done.


When scaling.webhook is enabled webhook-test paths are returning 404

ingress:
  enabled: true
  ...
scaling:
  enabled: true
  webhook:
    enabled: true

I have the configuration above for webhooks. When I deploy n8n and trigger a webhook based on test-url, I always get 404. I changed the ingress definition of path:/webhook-test/ to use main n8n pod it works as expected.
My guess is that setting https://github.com/n8n-io/n8n/blob/master/packages/cli/src/config/schema.ts#L718 (N8N_DISABLE_PRODUCTION_MAIN_PROCESS) to true only affects the production url, so the test url should still point to main n8n

Hmm, what do you suggest could be a solution.

I'm not 100% sure but I feel like webhook-test part of Ingress should point to main n8n process

N8N_DISABLE_PRODUCTION_MAIN_PROCESS

I think the -test is missing here:
https://github.com/8gears/n8n-helm-chart/blob/master/templates/service.webhooks.yaml

That looks ok...
https://github.com/8gears/n8n-helm-chart/blob/4f70095930313ef2bd1dfd05de570afe91175ce4/templates/ingress.yaml#L56


Automatic cache busting?

Does SWA do any automatic cache busting on deploy for the client-side app - or is this still something we have to add into the client for each individual app?

Static Web Apps uses e-tags for client side caching. You can see these headers coming back on the responses. When you update your site, we will change these and any client that supports e-tag caching will invalidate the cache automatically:
Response headers:
etag: "65982876"


Wrong typing for rich_text

The generated types for a rich_text field should be an array? The above works but is giving errors.

Yes, there was a bug in the latest version. I've updated the library types from the latest Notion types and unfortunately their types file had a error. They specified RichTextItemResponse as type for rich text instead of RichTextItemResponse[] 🤦
I've published a fix.
Can you please update to the latest version of notion-ts-client (0.2.10) ?

Started from a clean Next install and unfortunately still get the issue:

Did you run npx notion-ts-client@latest generate?
Also, check the file notion-api.types.ts in the generated SDK folder under core/types and see that the type TitlePropertyItemObjectResponse is correct.

On my end, I'm still seeing RichTextItemResponse instead of RichTextItemResponse[], even after using npx notion-ts-client@latest generate.

On my end, I'm still seeing RichTextItemResponse instead of RichTextItemResponse[], even after using npx notion-ts-client@latest generate.

Fixed on my side!


4.0 prevents logs from showing up in Console.app

Any NSLog that's called by my tweak (inside Safari) no longer shows up in Console.app since updating to libFLEX to 4.0.
After disabling the FLEX(all) dylib for Safari inside Choicy, it works fine again.
I'm compiling my tweak with a 13.0 sdk using theos-next.

This is a sad side effect of how we got logs to show back up in FLEX. You can see them in the System Logs screen. If you need to see your log messages in Console.app, disable FLEX, or use os_log directly in your tweaks instead of NSLog.

This is a sad side effect of how we got logs to show back up in FLEX. You can see them in the System Logs screen

Every app always show loading, no actual output here

@mlyxshi iOS version and FLEX loader and version?

@mlyxshi iOS version and FLEX loader and version?

[] unc0ver Version: 4.3.1
[] Machine Name: iPhone11,6
[] Model Name: D331pAP
[] Kernel Version: Darwin Kernel Version 19.2.0: Mon Nov  4 17:46:11 PST 2019; root:xnu-6153.60.66~39/RELEASE_ARM64_T8020
[] Kernel Page Size: 0x4000
[] System Version: iOS 13.3 (Stable) (Build: 17C54)
libFlEX  4.0.0
FLEXing 1.2.0

@NSExceptional
unc0ver Version: 4.3.1
Machine Name: iPhone11,6
Model Name: D331pAP
Kernel Version: Darwin Kernel Version 19.2.0: Mon Nov 4 17:46:11 PST 2019; root:xnu-6153.60.66~39/RELEASE_ARM64_T8020
Kernel Page Size: 0x4000
System Version: iOS 13.3 (Stable) (Build: 17C54)
libFlEX 4.0.0
FLEXing 1.2.0


refactor(checks): migrate AWS ecr, efs and eks to Rego

Related PRs:

https://github.com/aquasecurity/trivy-checks/pull/174

Closed if favor of https://github.com/aquasecurity/trivy-checks/pull/229


replace vcpkg-cpprestsdk with a +ARM64 -ssl/boost +UWP version

This new cpprestsdk package, 2.10.14, switches us to the app CRT.
cpprestsdk turns fof a bunch of boost and openssl dependencies when it's
built for the Windows Store subplatform, so we got a bunch of stuff for
free.
Incidentally, I fixed #2338 the real/correct way -- the build rules in
the package now make sure they're not using the system vcpkg root.
PR Checklist

[x] 1. Closes a bug, but not a tracked one
[ ] 2
[ ] 3
[ ] 4
[x] 5

Validation
build/run

/azp run


Improve icon consistency

enables correct default icon on Android (non round) (release & debug)
provides round icons (release & debug)
makes the iOS app use the same icon

(#2993 is the previous PR on this subject)

Thanks again @borisyankov for your work on this!
The iOS commit looks great and I'd be glad to merge all those changes -- but I would like to record what that source image was. :smile: So that we don't re-create for our future selves / successors the same kind of messy situation we've found ourselves given by the past.
I think you mentioned a Sketch file that you made -- would you add that to the iOS commit, or as a separate commit?
On the Android side, I'm still not convinced by the switch from a round to a square icon. I would be glad to take a commit that fixed the debug icon, though; and I don't have strong preferences on exactly what that would look like.


With plugin 1.08 payments are not synchronized to WAWI

We got a order right now, payment with twint, in checkout backend the payment has been successfull:

The order has been created in the shop and it is also synchronized to the shop, but it is not marked as paid in WAWI and also not in shop:

Regards

hi @PAO1908CH , thank you for the feedback, we will have a look

@PAO1908CH , this only happens with twint or same with credit card ?

Unfortunately I can not test that at the moment

@PAO1908CH , this only happens with twint or same with credit card ?

It happens also with Credit Card, tested right now.

hi @PAO1908CH , could you please confirm if with the latest release this issue is solved ?

Negative, with 1.10 payments are still not synchronized from Plugin to the shop and as it is not set as paid in the shop, it's also not set as paid in the wawi. I think the problem is with the interface Plugin => Shop, where the payment is not set as received, as when I check the orders in shop backend, the order is here, but not marked as paid.

Hello @PAO1908CH ,
I'm thinking why this issue still exists. Perhaps there are several ways to mark a payment as completed? Could you make a video demonstrating the issue, or would you be available for a video call? I would like to reproduce it and fix it.

Sure, Im here till 1800 CET.
Do you want to Call me today?
Regards
edgaraswallee @.***> schrieb am Fr., 22. Dez. 2023,
16:03:

Hello @PAO1908CH https://github.com/PAO1908CH ,
I'm thinking why this issue still exists. Perhaps there are several ways
to mark a payment as completed? Could you make a video demonstrating the
issue, or would you be available for a video call? I would like to
reproduce it and fix it.
—
Reply to this email directly, view it on GitHub
https://github.com/pfpayments/jtl-5/issues/12#issuecomment-1867788669,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BDVLGBRCWPZUDJ4OVEVWXILYKWOKNAVCNFSM6AAAAABA4Z3RHCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNRXG44DQNRWHE
.
You are receiving this because you were mentioned.Message ID:
@.***>

Let's meet 1720 CET here?
https://meet.google.com/uts-sqck-nxr?authuser=0

@PAO1908CH maybe are you available right now for a call?

yes, +41763875055

https://meet.google.com/ohw-vqdu-bdp?authuser=0

Hello, @PAO1908CH , new release has been made, can you please check if you issues are fixed?


ligand is outside the grid box

Hello,
I'm a bit new to AutoDock-Vina and all this in-silco screen process so if my questions are stupid.
I'm trying to calculate the energy of docking a molecule to a protein,
I tried using the 1iep receptor and 1iep ligand that was given in the python example.
after that, I tried using a different molecule of my choosing that has the following SMILES string: C[C@@]1(c2cc(NC(=O)c3ccc(C#N)cn3)ccc2F)CSCC(N)=N1
I converted the molecule from smi file to pdb using obabel and the following command
obabel -i smi ./mol.smi -O ./mol.pdb --gen3d
after that I used the mk_prepare_ligand on that file to create the pdbqt file using the following command
python3 mk_prepare_ligand.py -i ./mol.pdb -o ./mol.pdbqt
then I tried running the python example again but with my molecule and I got the following error:
>>> from vina import Vina
>>> v = Vina(sf_name='vina')
>>> v.set_receptor('./1iep_receptor.pdbqt')
>>> v.set_ligand_from_file('./mol.pdbqt')
>>> v.compute_vina_maps(center=[15.190, 53.903, 16.917], box_size=[20, 20, 20])
Computing Vina grid ... done.
>>> energy = v.score()
ERROR: The ligand is outside the grid box. Increase the size of the grid box or center it accordingly around the ligand.

I tried looking it up, but I couldn't find any solution
so my questions are the following:

what does exactly vina_maps mean, is it the location on the protein/receptor itself the docking occurs?
is the vina_map depend on the protein/receptor itself or on both protein/receptor and the ligand/molecule?
if I want to use a different protein/receptor in future experiments, how exactly do I compute those vina maps?
did I convert the molecule to ligand pdbqt file correctly?
where I can learn more about molecular docking

thank you and sorry if my question are bad

I got the same error when calling score() method.
Here is what I did:

Run docking with vina normally.
Extract one of the docked model in and save in .pdbqt (using openbabel).
Import vina in Jupyter notebook (and in CLI python) -> load the same receptor and box dimension parameters used in docking experiment -> call compute_vina_maps() -> call score() -> get the same error.
I visualized the box, receptor, and docked ligand with Pymol and was pretty certain that the ligand was completely inside the box.

@eyalmazuz you probably want to run v.dock() before v.score(). Since openbabel is unaware of the receptor, the ligand may even be very far from the receptor. The docs may be helpful: https://autodock-vina.readthedocs.io/en/latest/index.html
@lnguyen21 can you send the input files so I can reproduce the issue? I docked a molecule and then rescored the first pose and it worked fine.

Hi @diogomart running v.dock() before v.score() worked.
but I still don't understand something if you could elaborate a little bit more
I was experimenting with different protein than the one mentioned at the beginning of the issue
I'm using 2b7a which is a JAK2 protein, and I took the smiles of Ruxolitinib which is a known approved treatment that suppresses this protein.
initially following: https://github.com/luost26/3D-Generative-SBDD/blob/main/utils/docking.py
I was exporting the vina_maps by calculating the means of the ligand (Ruxolitinib) the same they do there
using the following rdkit code:
pos = ligand_rdmol.GetConformer(0).GetPositions()
center = (pos.max(0) + pos.min(0)) / 2

but after using software like autodocktools, I got a different vina_map than the one generated from the ligand
more specifically a vina_maps that is super similar to a one that is calculate as if I replace the ligand with the protein in the rdkit calculation
i.e.:
protein = Chem.MolFromPDBFile('./2b7a.pdb')
pos = protein.GetConformer(0).GetPositions()
center = (pos.max(0) + pos.min(0)) / 2

using this ligand and protein I was certain it would work like the python tutorial cause it's a drug that supposes to bind to said protein but I got the same errors as my first attempts, after several searches and tutorials
I tried using the executable and not the python package and surprisingly it worked and I got some results back
I created a simple config script and just ran the following command: vina --receptor 2b7a.pdbqt --ligand ruxolitinib.pdbqt --config config.txt
this both worked with autodock vina and quick-vina
so my questions are as follow and I hope you could answer them:

is there a difference between calculating the vina_map based on the protein and based on the ligand when calculating for the docking? which one is right?

is there a different way that is the more correct to calculate the vina_maps? what does vina maps even mean? I hope you can explain or give a reference for me to read

why calling v.dock() before v.score() fixes the problem, what it exactly do? why doesn't score work without it?

why does using the executable work with the same vina_maps and not when using the python script? does the executable just run docking (equivalent to v.dock()) automatically?

where I can learn more about molecular docking?

sorry for the long comment and for my questions if they are super simple

1 The maps consist of precalculated interactions with the protein. Therefore, the maps must be positioned relative to the protein, not the ligand. Centering the maps on the ligand makes sense only if the ligand is already in the desired binding site of the protein, which is not the case when generating coordinates from smiles strings.
2 The best is to actually look at the protein and the grid box using a molecular visualizer, and define the dimensions and location of the maps manually. The python molecular viewer (PMV) from MGLTools can do this. Also Pymol, VMD, and Chimera all have plugins for displaying grid boxes..
3 v.score() does not move the ligand around, it just calculates the energy for the coordinates that the ligand already has, while v.dock() docks the ligand by optimizing its position and conformation to produce the best score.
4 yes the executable docks by default unless the --score_only or --local_only are used. Not sure what you mean by "the executable work with the same vina_maps".
5 By reading papers and documentation of docking software.

@eyalmazuz you probably want to run v.dock() before v.score(). Since openbabel is unaware of the receptor, the ligand may even be very far from the receptor. The docs may be helpful: https://autodock-vina.readthedocs.io/en/latest/index.html
@lnguyen21 can you send the input files so I can reproduce the issue? I docked a molecule and then rescored the first pose and it worked fine.
@diogomart: I already deleted the files that raised the error, but I have tested on a different receptor last weekend and don't see the error. There must be a difference between the two runs that I missed. But anyway, I reconfirm that it works now - apologize for the panic.
My best guess is that it might has something to do with a flexible side-chain - the openbabel might have extracted one side-chain that I was not seeing it in Pymol.

Thanks for confirming it works as expected :+1:


Simple examples components are good. Documentation is complicated un-related to real tasks.

Simple examples components are good. Documentation is complicated un-related to real tasks. For example:
if this is the task:
#Importing data
data = pd.read_csv('/home/bb00745059/working_directory/Projects_try/Customer_Complaint/Data/FINAL_COMPLAINTS_DATASET(120k Rows).csv', usecols=['complaintid', 'product', 'consumer_complaint_narrative', 'company'])
data.head()
Then how to write it in kubeflow component? Which variables to use - simple or artifacts? And then show it by example component code on it and then using it within pipeline. No where I could find that. It is so confusing.
This will be really helpful.

This is not the right place to ask for help, please reach out to the maintainers of the distribution you are using.
/close


fix(provision): prevent volume deletion if snapshot exists

Why is this PR required? What issue does it fix?:
When an LVM volume  with active snapshot is deleted, both volume and snapshot is removed from the machine, but the resources corresponding to the snapshot still exists in the cluster. This stale snapshot resources will cause the controller to error out when we try to delete these resources as the actual snapshot does not exist.
What this PR does?:

error out the DeleteVolume() controller gRPC call if the volume has active snapshots

Does this PR require any upgrade changes?:
No
If the changes in this PR are manually verified, list down the scenarios covered::
Any additional information for your reviewer? :
Mention if this PR is part of any design or a continuation of previous PRs
Checklist:

[ ] Fixes #
[x] PR Title follows the convention of  <type>(<scope>): <subject>
[x] Has the change log section been updated?
[ ] Commit has unit tests
[x] Commit has integration tests
[ ] (Optional) Are upgrade changes included in this PR? If not, mention the issue/PR to track:
[ ] (Optional) If documentation changes are required, which issue on https://github.com/openebs/openebs-docs is used to track them:

@pawanpraka1  I have added the check for snapshot in the controller DeleteVolume(), but the PVC is getting deleted even though DeleteVolune() errors out. How can we prevent the PVC deletion?

@akhilerm, yes, csi-provisioner does not honour the error returned from delete volume call. Let us put this question in the CSI channel. Here what we can do is, we can error out the delete request at the agent side.

what we can do is, we can error out the delete request at the agent side.

@pawanpraka1 If we error out at agent side, will it prevent the PVC deletion?

If we error out at agent side, will it prevent the PVC deletion?

no, PVC will be deleted any way. Here we can only control whether the actual volume will be deleted or not at the agent side.


VesselViewerRPM update needed

RasterPropMonitor had to update its handling of its render texture target by calling GL.Clear every frame.  This has had the unfortunate side-effect of causing VV RPM to flicker, since it only draws every few frames by default.  I tried reverting the RPM change, but this causes other menus to show garbage, so VV is going to need to make some changes if it's going to support the reduce update rate in RPM.  I can provide some ideas if you're interested.

Alternatively, I can provide a pull request with the changes, but that might take me a few days.

I've got a code change on RasterPropMonitor that will fix this issue, so feel free to close this out.


[beta] RFC - homepage and top nav

RFC - adds a more traditional homepage with a topbar nav.
Preview

I know ya'll want to stick it to Big Monitor. But can we limit the search bar so that I don't have to move my neck by more than 10° to see the relevant stuff? 😆
old:

new:

But can we limit the search bar so that I don't have to move my neck by more than 10° to see the relevant stuff? 😆

yea yea, this is super drafty for now :) we'll fix that
@harish-sethuraman no need to review, this is a draft and there'll be a lot of dirty code here until we start actually converging on something


Refactor contract names

Names must store some logic
CyberСoin => ticket.sol CyberCore => event.sol

Pull request #10 was merged


version 3.5.0   Deoldify not working

latest v3.5 DDColor working Deoldify   Options gray marked not working .

Are you using the last version of Hybrid 2<IP_ADDRESS> ?

yes. two latest verstions used (Hybrid and plugins). ddcolour working. but deoldify not respond  and all options gay colured

This version is using the Presets to simplify its usage. In this case you are using the Preset "fast" and all the filter parameters are selected automatically. To enable full editing of filter parameters you need to select the Preset "custom".

ok i will try

It is not possible to define Presets working for everyone. In any case using the "custom" preset is is possible to manually change all the filter parameters. I consider the issue closed.


brew update is not needed any more

Starting from August 2016 it's no longer necessary to run brew update. Homebrew auto-updates before installs: Homebrew/brew#679
brew doctor before:
[-] iOS toolchain - develop for iOS devices (Xcode 8.2.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 8.2.1, Build version 8C1002
    ✗ ios-deploy not installed. To install:
        brew update
        brew install ios-deploy
    ✗ CocoaPods not installed.
        CocoaPods is used to retrieve the iOS platform side's plugin code that responds to your plugin usage on the Dart side.
        Without resolving iOS dependencies with CocoaPods, plugins will not work on iOS.
        For more info, see https://flutter.io/platform-plugins
      To install:
        brew update
        brew install cocoapods
        pod setup

brew doctor after:
[-] iOS toolchain - develop for iOS devices (Xcode 8.2.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 8.2.1, Build version 8C1002
    ✗ Your Mac needs to enabled for developer mode before using Xcode for the first time.
      Run 'sudo DevToolsSecurity -enable' or open Xcode
    ✗ ios-deploy not installed. To install:
        brew install ios-deploy
    ✗ CocoaPods not installed.
        CocoaPods is used to retrieve the iOS platform side's plugin code that responds to your plugin usage on the Dart side.
        Without resolving iOS dependencies with CocoaPods, plugins will not work on iOS.
        For more info, see https://flutter.io/platform-plugins
      To install:
        brew install cocoapods
        pod setup

@googlebot I signed it!

Landing. Thanks for the contribution!


Add GA4 analytics

Updating the site to Google Analytics 4 per CDT request.

We do have basic GA tracking on Drought, but nothing special beyond that. No events.


feat(client): ts-migrate header components

Checklist:

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the main branch of freeCodeCamp.
[x] I have tested these changes either locally on my machine, or GitPod.

Related to #42256

Converted .js files to .tsx and created required interface for props of component using the types specified in the prop-types.
Removed prop-types
Added es-lint and ts ignore comments to make the code work with minimal changes.
tested client folder on local machine
eslint ignore comments can be removed in future by creating strict props type and passing them correctly from the parent component.

@ojeytonwilliams  Can you please review this PR.

@ojeytonwilliams  thanks for the review! I'll work on the changes 👍🏼

Hey, I have converted the file names to kebab-case in the commit 'resolving eslint error' , sorry for the absurd commit message.

I hope you don't mind @akshatgarg12, but I made a few changes to try and get this ready to be merged today. It should be good to go. Thanks for contributing to freeCodeCamp 🎉


zooming out on graph goes wonky when the mouse leaves the original bounding box of all the nodes

you get a cookie for using gifs

excellent
On Tue, Jul 7, 2020 at 5:18 PM Chris Gardner<EMAIL_ADDRESS>wrote:

you get a cookie for using gifs
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/chris-gardner/usd-dependency-graph/issues/14#issuecomment-654650176,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALIB23BL352354A6NQJFPFLR2LD5JANCNFSM4OSP2T6A
.

🍪


Update Subgraph Addresses

Describe the bug
update weightedpoolfactory/weightedpool2tokenfactory/vault addresses and re-deploy subgraph

Closed by commit 53c8...69e2


Custom SSL connection?

Hi
Thanks for the awesome library, just wondering if it is possible to create API for tweaking HTTPUrlConnection for self signed certificate.
Currently it failed to read any ssl sites when sitting behind proxy.
Cheers,

I have added a proxy support to Jsoup 1.8.4.
It can refer to https://github.com/E-lai/jsoup-proxy

Thanks, I will have a look at it !

(Closing out old, dormant bugs. If you are still impacted by this, please reopen & vote.)


WIP: overhaul API

The new API returns simpler Headers, arranged along the lines of what we originally agreed on ("Headers are the RunStart, Event Descriptor, and Run Stop"), leaving user-friendly representations and custom rearrangements to be implemented at a higher level. One such implementation is album.
The full API is now simpler:

DataBroker[] for slicing by scan ID(s) or recency
DataBroker() for building queries from keyword arguments
get_events  return events generator
get_table  return DataFrame
Header, vastly simplified: it is merely Document with a dedicated constructor that accepts a Run Start Document

Everything is else gone. The nice pretty-printing logic will live on elsewhere. The old DataBroker.find_headers and StepScan are still there for now, but they issue warnings.

@ericdill @tacaswell I know we had agreed we could call it fetch_events and fetch_table, but I have a new reason for preferring get_*: in database lingo (c.f. sqlite) fetch means "load the data", as in fetch_one vs. fetch_all. In this case, get_events does not actually load the data, and that's an important distinction. Perhaps the name should have changed back when we made it a generator. Acceptable?

but get_table does load the data.
It also looks like that the fetch* methods are on the cursor, not the main db so :+1: on renaming to avoid a subtle head-space name clash.


DJ008: Should it also work for tolomea/django-auto-prefetch?

DJ008 - Django model without __str__ method
django-auto-prefetch uses auto_prefetch.Model instead of models.Model so the rule does not trigger. Should this be changed or is there a way to make it work with auto_prefetch.Model via pyproject.toml?
What the rule does
Checks that a __str__ method is defined in Django models.

Example
from django.db import models

class Book(models.Model): # Will trigger DJ008
    author = models.ForeignKey("Author", on_delete=models.CASCADE)

    class Meta:
        verbose_name = "Book"

import auto_prefetch
from django.db import models

class Book(auto_prefetch.Model): # Will not trigger DJ008
    author = auto_prefetch.ForeignKey("Author", on_delete=models.CASCADE)

    class Meta(auto_prefetch.Model.Meta):
        verbose_name = "Book"

Keywords: django-auto-prefetch, DJ008 and django-model-without-dunder-str.
Thanks, TheLovinator.

Yeah, our Django rules are very limited at the moment until we finish migrating Ruff to a multifile-capable analysis (which we're working on). The underlying issue is the same as for https://github.com/astral-sh/ruff/issues/12865: Ruff isn't capable of reasoning about imports from other modules.
A setting could be a short-term workaround that I'm open considering.


Spine integration no longer rendering on iOS 12 and some devices (iPhone 5S)

Forum: https://forum.playcanvas.com/t/spine-demo-forked-not-working-in-ipad-air-first-generation/27975
Project: https://playcanvas.com/project/783555/overview/spine-example-38
There's no error besides the one below:

Last working version was engine 1.54.1
https://launch.playcanvas.com/1128118?debug=true&use_local_engine=https://code.playcanvas.com/playcanvas-1.54.1.js

Note: Renders correctly on iOS 15

I tried Mac OS Webgl1 just in case it's Webgl1 issue, and that renders fine on Chrome and Safari.

iPhone 6 with iOS 12.5.6:

Doesn't work on iPhone 5S on 12.4.5. Will try upgrading to 12.4.6

iPhone 5S on 12.4.6 doesn't render the spine model/animation. Renaming ticket.

As iPhone 5S only supports WebGL1, it's not likely we'll investigate this, and so I'm closing the ticket.


Bug when sending Bytes32 (hex form)

Hello eventeum people,
The smart contract is like that:

I need to keep a bytes32 number into foo.
Right case:
If the number is created from a string like:
let hex = web3.utils.stringToHex("Nick") // => "0x4e69636b00000000000000000000000000000000000000000000000000000000"
send (hex)
Then from Kafka we will get this:

This string ("Nick") can be convert back to our original hex number by:
web3.utils.stringToHex(msg.details.nonIndexedParameters[0].value))
Wrong case:
If the number is just a random like:
"0x177336d08543ac4b000000000000000000000000000000000000000000000000"
On Kafka we will get an unreadable string, which CAN NOT convert back to our original number
web3.utils.stringToHex(msg.details.nonIndexedParameters[0].value))

As you see in the picture:
It's is not "177336d08543ac4b"
Think that the error is when you convert from bytes32 to string and broadcast to kafka
Sincerely,
Quang, VU

Hey, thanks for reporting this.  I've fixed the issue and it will be included within the next release.

Hi, well done (y) . It's good to hear that. I'll close the issue.


Interaction model builder

Hi,
I am interested in automatically build the full interaction model. I.e. combining all the speech assets into a single JSON file. I believe this could be useful to automatically configure the players, artists, albums and genres. So far I have a prototype working for artists, albums and genres but needs a lot of manual intervention.
The process is simple. Edit the config.js file. Run the tool that generates the speech.JSON. The tool extracts all the albums, artists etc from your squeezebox server. The speech.JSON is deployed to Alexa and job done. Shame the speech.JSON file cannot be automatically deployed using something like Claudia.
This work also resolve the case sensitive issue with the Squeezebox server.
Reason why I am posting here is two fold. One to let you know what I am doing and secondly to make sure no one else has already done the same thing (that would save me time).
Cheers,
Geoff

I have completed this work and it works better than the what is here but without what is here I could not have done what I have built.

I hope to be getting back into this soon, thanks for the work and update @GeoffAtHome!

Next thing I want to try is to turn Alexa into a Squeezebox. I.e. serve the music directly to Alexa. This should be possible just need to understand how to registered a speakerless Squeezebox.....

Hello @GeoffAtHome!  Do you have any kind of a procedure written for generating the speech assets from the Squeezebox server?  I'd like to try this out.  Thanks!

Hi,
I do it is checked into GitHub on my branch (if that’s the right term). It enumerate artists, albums and genres. I think it works OK for me. Just built on the other work that has gone before.
Cheers,
Geoff
From: Mike<EMAIL_ADDRESS>Sent: 23 June 2017 20:35
To: lordpengwin/muzak<EMAIL_ADDRESS>Cc: GeoffAtHome<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [lordpengwin/muzak] Interaction model builder (#14)
Hello @GeoffAtHome https://github.com/geoffathome ! Do you have any kind of a procedure written for generating the speech assets from the Squeezebox server? I'd like to try this out. Thanks!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/lordpengwin/muzak/issues/14#issuecomment-310754718 , or mute the thread https://github.com/notifications/unsubscribe-auth/AGttWE_kYG8q857DtFthxmKzYtUmUFldks5sHBN_gaJpZM4NPRKP .  https://github.com/notifications/beacon/AGttWOr_uBMT5nnogPQlXfPJBNH9KPXCks5sHBN_gaJpZM4NPRKP.gif

Great.  Do you have any plans to incorporate that functionality from your branch back into the main trunk?  If so do you have any kind of timeframe?  Thanks again!

If I knew how to then I would….
From: Mike<EMAIL_ADDRESS>Sent: 27 June 2017 18:52
To: lordpengwin/muzak<EMAIL_ADDRESS>Cc: GeoffAtHome<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [lordpengwin/muzak] Interaction model builder (#14)
Great. Do you have any plans to incorporate that functionality from your branch back into the main trunk? If so do you have any kind of timeframe? Thanks again!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/lordpengwin/muzak/issues/14#issuecomment-311435038 , or mute the thread https://github.com/notifications/unsubscribe-auth/AGttWFgfdKY1BlDe3rGB4vFsinCmgv5qks5sIUFQgaJpZM4NPRKP .  https://github.com/notifications/beacon/AGttWEtQl3ML9gRFM6pLt_3QjfiNDwN_ks5sIUFQgaJpZM4NPRKP.gif

@GeoffAtHome is your fork working with native playback commands from Amazon, without having to say "Alex, Tell Muzak"?

As it enumerates all the albums and artist I can say
“Play Pink Floyd” or “Play Dark Side Of the Moon” or “Play Dark Side of the moon by Pink Floyd”.
What I cannot say is “Alexa ask Muzak to play Pink Floyd”.  This may be because I have as the invocation name “ask muzak”. I have also set the global fields to for “Audio Player” to yes.
I think, if I get round to the next part, which is to use Alexa as a headless Squeezebox, then it may work better with Alexa playing the music. Something to try in the future…..
From: glacket<EMAIL_ADDRESS>Sent: 04 July 2017 17:27
To: lordpengwin/muzak<EMAIL_ADDRESS>Cc: GeoffAtHome<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [lordpengwin/muzak] Interaction model builder (#14)
@GeoffAtHome https://github.com/geoffathome  is your fork working with native playback commands from Amazon, without having to say "Alex, Tell Muzak"?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/lordpengwin/muzak/issues/14#issuecomment-312915204 , or mute the thread https://github.com/notifications/unsubscribe-auth/AGttWNjgz-DN-KHLTUWhmSseduddeSE_ks5sKmfogaJpZM4NPRKP .  https://github.com/notifications/beacon/AGttWL-L9Mf4cA9lmaKVYiB2l1yR0Tu5ks5sKmfogaJpZM4NPRKP.gif

@GeoffAtHome can you specify a player? As in "alexa, play pink Floyd in kitchen" or "alexa, play music in kitchen", "alexa, stop music in kitchen".

Again, the skill need to be activated before the second command is given. Does “Alexa, play music in the kitchen” work for you?
From: glacket<EMAIL_ADDRESS>Sent: 04 July 2017 18:03
To: lordpengwin/muzak<EMAIL_ADDRESS>Cc: GeoffAtHome<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [lordpengwin/muzak] Interaction model builder (#14)
@GeoffAtHome https://github.com/geoffathome  can you specify a player? As in "alexa, play pink Floyd in kitchen" or "alexa, play music in kitchen", "alexa, stop music in kitchen".
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/lordpengwin/muzak/issues/14#issuecomment-312920331 , or mute the thread https://github.com/notifications/unsubscribe-auth/AGttWE8q4q4H6u-pNL4-e63HSFOnk8Kuks5sKnBAgaJpZM4NPRKP .  https://github.com/notifications/beacon/AGttWNzS7yF8Fn9ek5vwhNwMP7lfP968ks5sKnBAgaJpZM4NPRKP.gif

@GeoffAtHome I guess I'm confused. I thought your fork of the skill was activated natively by saying "alexa, play pink Floyd", when your fork is configured as the native music playback in AWS. I have 12 rooms with speakers in the ceiling connected to 12 usb soundcards running on a pc running 12 instances of squeezelite. Currently, I have to say "alexa, open muzak" which will activate the current skill fork. Alexa replies "squeezebox control started". Then I can say "play music in kitchen" "lower volume in kitchen". I was hoping with native playback functionality, I would no longer have to say "alexa, open muzak", or "alexa tell muzal to......." .

@GeoffAtHome - In README.md, under section "Publish the skill (manual)", is the zip file creation command incomplete for your new speech asset model generation updates?
From: zip -r muzak.zip muzak.js config.js node_modules
To:  zip -r muzak.zip muzak.js config.js album.js artist.js genre.js node_modules

@MikeDeSantis  Look in the speechAssets folder. The old format is ALBUM.txt, ARTIST.txt, GENRE.txt, PLAYERS.txt and utterences.txt. The intents are in intents.json. The new format combines these into speechAssets.json. The format is correct but last time I tried using this it failed to build hence creating both versions.
It is far easier to upload a single file rather that the six individual. I think Amazon are looking for a method to automate the publishing of these files which would be great - maybe this is available now as it has been a while since I have looked into this.

Thanks @GeoffAtHome, I totally agree with the speechAssets.json consolidation.  Some improvements for this file output would be to:

put empty sample elements in areas like Increase/DecreaseVolume for Amazon's syntax compatibility
keep the players.txt file in use to then populate element types-PLAYERS-values each time speechAssets is regenerated.
  "types": [  {    "name": "PLAYERS",    "values": [      {        "name": {          "value": "Kitchen"        }      },      {        "name": {          "value": "Master Bedroom"        }      }, ...

I bring up the files to include in the zip because of muzak.js lines 14-18:
var config = require('./config'); var albums = require('./album.js'); var artists = require('./artist.js'); var genres = require('./genre.js'); var info = { Album: albums, Artist: artists, Genre: genres };

I have moved the list of players into the config file as this should be the only file that needs to be customised. create-assets has been updated to support the latest changes to Amazon's skill builder beta. This now works with album, artist, genre and track.
note the "invocationName" in the default-assets.js file. This needs to be moved into the config file.

It looks like all the look-up maybe able to be done by the service which will simplify lambda function. See https://developer.amazon.com/docs/custom-skills/define-synonyms-and-ids-for-slot-type-values-entity-resolution.html for details.


Apotheosis Compatibility

Absolutely love your mod, but unfortunately I also can't play without Apotheosis. The tree chopping simply doesn't work when I have https://github.com/Shadows-of-Fire mod active. I'll cross post there as well, because I can't for the life of me figure it out. BTW I also can't seem to get KUBEJS to play nicely with your mod either. I'm trying to get a right click function to work on stripped logs for making planks, but no bother I'm finding a workaround so don't  bother making a new issue out of it. I'm just saying this.

Hi! I know the mods were compatible at some point, TreeChop even had special support for Apotheosis enchantments. What version of each mod are you using? And what version of Minecraft? If you can upload a log file (latest.log, etc.) that would be very helpful.
I'm not sure how KUBEJS works, but the stripped state for treechop's chopped logs is stored in the block entity's NBT data. Someone else was using KUBEJS in #262, there might be something there that'll help you.


hitArea on masked DisplayObjectContainer not affecting children

I'm working on a scrollable panel where I've got a var scrollable.area = new PIXI.DisplayObjectContainer();  I've applied a simple rectangular mask to it scrollable.area.mask = myMask;
scrollable.area contains a list of interactive buttonMode sprites as its children.
The mask works great for rendering, but I am having troubles setting the hitArea correctly.  So even though the child sprites are outside of the masked area, mouse events are still registering on them.
I thought that setting scrollable.area.hitArea = new PIXI.rectangle(0, 0, maskWidth, maskHeight); would do the trick... but no dice.  The child sprites are still capturing the events.

Same bug for me... we cant interact with elements behind the mask. And for me, I can click through the mask...

that's not how hitArea works. Currently you cant create mask for interactionManager at all. You only can specify hitAreas for interactive objects that will receive events. There's no bubbling or proxying or other stuff. There's no clipping. We need to rewrite interactionManager to fix all those problems.

I didnt want to open yet another hitArea Issue.
hitArea fails to get updated on custom classes extending PIXI.Sprite. even with the interactive flag on I can still access objects behind the sprite. It works well if I manually update the hitArea to be the same size of the children inside my sprite.

This issue has been addressed in v4 :+1:


Iterating mutiple times over a generator

Hello, following code crashes with a SIGSEGV:
        auto answer = []() -> cppcoro::generator<int> {
            co_yield 42;
        }();
        for (auto i : answer) {
            std::cout << i << "\n";
        }
        for (auto i : answer) {
            std::cout << i << "\n";
        }

I guess that iterating twice over a generator is not supported.
I am not saying it should be.
But if it is not, it is probably worth being documented, and I could not find anything on that.
My setup : Fedora 33, GCC 10, thus with patches from https://github.com/dutow/cppcoro.git
I could also reproduce with clang here https://godbolt.org/z/51fxb1

Shouldn't you have to call the lambda to create the generator?
for (auto i : answer()) { ... }

Yes it would fix, but that's the point I think is worth being documented.
It's not obvious that answer is pointing to dangling data after enumeration, and it goes against intuition on C++ objects lifetime.
Furthermore, if dangling status is "by design and not a bug", forcing the user to std::move(answer) might make sense (possibly by constraining begin to "this &&" ?).

Ah I didn't see the lambda already being executed on definition, pay no mind to my comment; you're right


WIP: Extr0py/217 inc buffer updates

This results in much improved / faster auto completion for the typescript language service. Re-parsing the entire file for each keystroke is non-trivial...


Request to add the 'PublishAllPorts' option to the ContainerOptionsBuilder

I am using dockertest-rs in my integration tests. After trying to manage locks and host:port mappings, the maintainer pointed out to me that docker now has a 'PublishAllPorts' option available. This option will assign an available host port at random for each exposed port on the container. The ContainerOptionsBuilder does not currently support this option.
Documented Option, search for 'PublishAllPorts'

I would like to add this to the ContainerOptionsBuilder by introducing code such as
    pub fn publish_all_ports(
        &mut self,
    ) -> &mut Self {
        self.params.insert("HostConfig.PublishAllPorts", json!(true));
        self
    }

    /// Test container option PublishAllPorts
    #[test]
    fn container_options_publish_all_ports() {
        let options = ContainerOptionsBuilder::new("test_image")
            .publish_all_ports()
            .build();

        assert_eq!(
            r#"{"HostConfig":{"PublishAllPorts":true},"Image":"test_image"}"#,
            options.serialize().unwrap()
        );
    }

Looks great @tylerhjones


SubViewport not refreshing properly in-editor when reducing its size

Tested versions

Reproducible in 4.2.1-stable and 4.1.1-stable

System information
Godot v4.2.1.stable - Windows 10.0.19045 - Vulkan (Mobile) - dedicated NVIDIA GeForce RTX 3080 (NVIDIA; <IP_ADDRESS>23) - AMD Ryzen 9 5900X 12-Core Processor (24 Threads)
Issue description
When reducing the size of a SubViewport in the editor, the viewport will show an incorrect stretched image (the SubViewport at the new size stretched to fit the previous size). Toggling the SubViewport Container visibility off and on fixes it.
https://github.com/godotengine/godot/assets/35150208/752fc63a-0392-4126-a0aa-c46438a9e200
Steps to reproduce
Steps for the MRP:

Open sub_viewport_container.tscn
Select the SubViewport and change its size X from 500 to 400
Notice the stretched image
Toggle the SubViewport Container visibility off and on again
Notice the image is showing correctly now.

Minimal reproduction project (MRP)
_viewport_bug.zip

Fixed in #86302

Hold on now.
This is desired behavior in my case. I want the subviewport to be stretched to the containers size if it is smaller. That's a feature.

@TheDuriel this cannot be a feature if hiding and showing it causes to render differently.

If you want this effect and in a consistent way, use Size 2D Override for the size of the viewport, and check Size 2D Override Stretch. Afaik, thats the way to achieve smaller viewport stretching the rendered image to the containers size.

~~Which is why I am wondering if there hasn't been a different "issue" discovered. I'm doing some checks on 4.x and on 3.x for a moment to confirm what I expect to happen in this scenario.
I do acknowledge that hiding and revealing the viewport does force a redraw and thus a change in what is being displayed. But that isn't on its own able to tell us which scenario is wrong. ~~
Could this actually be a symptom of the container not shrinking down when the viewport does? It's not certain what should happen in that case. We're in undefined behavior land now, and the proposed PR does not address that, since it forces a redraw but not a resort?
If you make a container at 100x100, and a viewport at 200x200, the container becomes 200x200. Then when you change the viewport to 100x100, the container remains at 200x200 so long as it does not get sorted by a parent container or shown/hidden.
Should the fix then not be, to queue a sorting of the container, which will in turn cause a redrawing? This then gives the correct result regardless of the uncertainty.

It's not certain what should happen in that case. We're in undefined behavior land now

The docs states SubViewportContainer will use the combined size of the SubViewports as minimum size, not as its size, so i believe the behaviour is the intended. This way you can have it work correctly with Expand i believe. It would be also quite problematic if you resize the container manually, then resize the subviewport, and it would bring down the container size again. So again, i believe this is the intended behaviour (i may be wrong).

Additionally I wish to point out that the stretch shrink setting, does not allow the use of precise values

If i understood you correctly, you want the subviewport rendered image be always at a resolution, and stretched to cover the entire viewport container. If so, you dont need stretch shrink at all. You just need the settings i mentioned above.

If i understood you correctly, you want the subviewport rendered image be always at a resolution, and stretched to cover the entire viewport container. If so, you dont need stretch shrink at all. You just need the settings i mentioned above.

I just tried and it was difficult to tell which combination of settings would actually achieve this. Especially with the inconsistency created by resizing the container/viewport. It's a mess.

So lets focus on the thing at hand: ViewportContainers will display different images depending on whether the viewport has been "redrawn". I will note that this isn't limited to static images. The viewport is still rendering frames fine.
I suggest that an attempt is made to queue a sorting of the container when the viewport changes size. Which in turn will cause the viewport to redraw. Achieving the same effect, but also respecting the fact that: Control nodes assume the size of their contents.

You can still achieve the stretching effect shown in the bug report video using the Size 2D Override property.

I believe size is intended to be the actual size of the Viewport, not the target size of the final image. Using the Size 2D Override will give you the same result predictably. Depending on the SubViewportContainer not redrawing is unreliable, as there are various actions that can trigger a redraw such as resizing the window, reparenting an ancestor, etc.


Fix runQuickstartsFromSource.bat

Commit 35912dd2e55bf70fe94604edf3ed87b69121d8bd updated .sh version
of the script to run from ".." but ran the .bat version of the script from ".", which
leads to "mvnw.cmd" not being found. Because of this, the quickstart launcher does
not build or launch in Windows using runQuickstartsFromSource.bat.
After this fix, the quickstart launcher does build and launch. However,
there is still an issue where the quickstarts do not launch on Windows, complaining
"mvnw" not being found (despite the path in the error existing). This issue would need
to be fixed in the Java files.
In regards to the many cd commands: Windows does not revert the
working directory to it previous state after running commands. So
mvnw -f .. changes the working directory back to the parent,
so we need to back into build again. And then an extra cd to get
out of quickstarts-showcase.

JIRA

Referenced pull requests

How to retest this PR or trigger a specific build:

for a pull request build please add comment: Jenkins retest this
for a full downstream build please add the label run_fdb
for a specific pull request build please add comment: Jenkins (re)run [optaplanner-quickstarts] tests
for a Quarkus LTS check please add comment: Jenkins run LTS
for a specific Quarkus LTS check please add comment: Jenkins (re)run [optaplanner-quickstarts] LTS
for a Native check please add comment: Jenkins run native
for a specific Native LTS check please add comment: Jenkins (re)run [optaplanner-quickstarts] native

Update: it works on both Linux and Windows, for both the .sh and .bat variants.

The PR successfully fixes the issue on my Windows machine.

To verify: it works when run from distribution ZIP

Chris will verify if it works when using run from source still too.
If that's ok, then it's good to merge.

I verified it works when run from distribution ZIP + when it run from source on both Windows and Linux

@ge0ffrey or @triceo please merge when you have the chance


License

Hey this is super rad. Can you add an Open Source license to this? This website can help you pick one. I need one to be able to use this and reference it in my work at IBM.
Thanks!

Added an MIT license file. I did not get a notification, sorry for the huge delay!


List of commands that don't work here

alias: alias say-hi [] { echo 'Hello!' }; say-hi  => error: Failed to spawn process
autoenv: might be possible to emulate with #3, but seems like overkill
cal, date: panicked at 'not implemented', ~/.cargo/registry/src/github.com-1ecc6299db9ec823/time-0.1.43/src/sys.rs:101:9
cd, cp, du, enter, exit: depends on #3
clear is meaningless in the browser
config: not supported in wasm; not sure how useful this really is though
debug: not sure how this works, since all the examples are using FS. Simply running debug outputs nothing.
describe: similar to debug, it partially outputs ansi color codes, e.g. for echo '[{"name":"Bob","age":25},{"name":"Fred","age":35}]' | from json | describe it outputs this:

empty?: not sure how to use this, since neither inline docs nor the website have examples (no docs for the command there at all)
history: error: Could not open history - maybe we can create that file with the vfs and update it with each command that the user runs?
kill: not supported in wasm; could emulate processes as web workers, list them with ps and kill them.
(to be continued)

I wonder if we can fix the alias one to work. Looks like the alias isn't set at the end of ; but we could think about making that possible:
20/07/2020 06:10:24> alias say-hi [] { echo 'Hello!' }; say-hi
error: Command not found
  ┌─ shell:1:36
  │
1 │ alias say-hi [] { echo 'Hello!' }; say-hi
  │                                    ^^^^^^ command not found

@jzaefferer regarding char newline question. This demonstrates it echo $(char prompt) $(char newline) $(char prompt). We can do things like this for the prompt such as echo [ $(ansi gb) $(pwd) $(ansi reset) "(" $(ansi cb) $(do -i { git rev-parse --abbrev-ref HEAD | trim }) $(ansi reset) ")" $(char newline) $(ansi yb) $(date --format "%m/%d/%Y %I:%M:%S%.3f %p" --raw) $(ansi reset) "> " ] | str collect"

@jzaefferer Also, here's how to use use ansi echo [$(ansi red) Hello] | str collect.

@fdncred thanks, those are good examples! Can we add those to the usage examples for those commands?

ansi #2230,  char #2231

@fdncred @jonathandturner I've finished the list! Overall looking pretty good.
I would appreciate some help with #12.
PS: I deleted the now-irrelevant commends about ansi and `char.


parents_depth option not honoured when querying

This isn't entirely clear from the documentation, but when specifying a parents_depth of 1, I was expecting to see a maximum of one prefix returned. Below code shows the prefix search returning three prefixes regardless of the parents_depth value.
>>> prefix = "<IP_ADDRESS>"
>>> query = {
...     'operator' : 'contains',
...     'val1' : 'prefix',
...     'val2' : prefix
... }
>>> options = {
...     'parents_depth': 1
... }
>>> ret = Prefix.search(query, options)['result']
>>> ret
[<pynipap.Prefix instance at 0x105e7fea8>, <pynipap.Prefix instance at 0x105e34200>, <pynipap.Prefix instance at 0x105ec6518>]
>>> for prefix in ret:
...     print prefix.prefix
...
<IP_ADDRESS>/8
<IP_ADDRESS>/16
<IP_ADDRESS>/24
>>> options = {
...     'parents_depth': 0
... }
>>> ret = Prefix.search(query, options)['result']
>>> ret
[<pynipap.Prefix instance at 0x105e7ff38>, <pynipap.Prefix instance at 0x105ec6638>, <pynipap.Prefix instance at 0x105ec67e8>]
>>> for prefix in ret:
...     print prefix.prefix
...
<IP_ADDRESS>/8
<IP_ADDRESS>/16
<IP_ADDRESS>/24
>>> options = {
...     'parents_depth': -1
... }
>>> ret = Prefix.search(query, options)['result']
>>> ret
[<pynipap.Prefix instance at 0x105e04b90>, <pynipap.Prefix instance at 0x105e340e0>, <pynipap.Prefix instance at 0x105ec6488>]
>>> for prefix in ret:
...     print prefix.prefix
...
<IP_ADDRESS>/8
<IP_ADDRESS>/16
<IP_ADDRESS>/24

Ok, so the first part of this is to understand the data we get back. In the result set we have direct matches, which are entries that directly match the search query. Most search engines only return direct matches but in NIPAP we also include other prefixes to give "context" to the user. The parent prefixes are the best example of this, as it allows us to see where a prefix belongs in the larger structure.
The parents_depth affects the indirect result, i.e. not direct matches. The search query you have is for prefixes that "contains" <IP_ADDRESS>. All of the results you get back are direct matches so parents_depth have no effect.
Try doing an equal search on "<IP_ADDRESS>/24" and change the parents_depth and I think you should see the expected result.

Also, you can see in the web UI that direct matches are black while prefixes included for context are grey. In pynipap you have a match attribute to differentiate them.


Add BUCK2 docs file (ENG-1968)

Add BUCK2 docs file for getting started with buck2, grokking high level
concepts and how to use buck2 and BUCK files.
The document contains a link to the "RUNNING_RUST_TESTS" document, which
describes similar, but more pointed, functionality.
view the rendered file here

bors merge

bors cacnel

bors cancel

Welp


countDownLatch await blocks the thread forever if error callback is executed instead of subscribe callback in Java SDK example

successfulCompletionLatch.countDown() is not called in error callback which results await call to be blocked forever if error is thrown. Please correct the java example.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 239e3b2d-4a09-c6a1-ab36-70b60a18f8b5
Version Independent ID: 10a323f6-5a10-8b38-6339-48ee7a4c54b3
Content: Register and use stored procedures, triggers, and user-defined functions in Azure Cosmos DB SDKs
Content Source: articles/cosmos-db/how-to-use-stored-procedures-triggers-udfs.md
Service: cosmos-db
GitHub Login: @markjbrown
Microsoft Alias: mjbrown

Thanks Bargavi. Will take a look at this.


Audio Volume & Quality - for singing & music purpose

Hi guys,
I'm doing a react native app for singing & music purpose.
I have some troubles with audio volume and audio quality. Any suggestion for me to resolve it?
Thanks & Best Regards,
Thai

Used incall-manager to high up the volume first. Seem okay.


Fixes to the design document

@seisman I'm going to merge this the way it is right now because I wanted to do some updates as well (include a nod to NSF, for one).

@leouieda Feel free to merge it.


Describe transformer command

Add new cli command: trcli describe transformer --type sql
This command describe how to use specific transformer + list all available transformers:
./binaries/main describe transformer --type sql

  ## Clickhouse transformer

  Based on Clickhouse Local
  https://clickhouse.com/docs/en/operations/utilities/clickhouse-local/

  This tool accept clickhouse SQL dialect and allow to produce SQL-like in memory
  data transformation.

  Image: diagram → https://jing.yandex-team.ru/files/tserakhau/ch_local_trans.svg

  Source table inside CH Local named as  table , clickhouse table structure mimic
  source table structure.

  Since each source change item (row) contains extra metadata we must match source
  and target data together. There for each row must have a key defined. All of
  this key should be uniq in every batch (for this we call collapse function). If
  we can't match source keys with transformed data we will mark such row as
  errored.

   Example Config

    tables:
        include_tables:
        - '"public"."included_data"'
        exclude_tables:
        - '"public"."excluded_data"'
    query: |
        select
                *
        from table

To make configs more user friendly query become a yaml filed, instead of json-baked string field.

:shipit:


Feature maps from 1st layer of each discriminator not included

I noticed that when saving feature maps for the GAN loss there is the condition if i > 0, that means that the feature maps of the first convolutional layer are not considered. Is this an optimization trick? Does the training work better this way?

Good question, I would also like to know the answer!


Visual Finder bug when switching to light mode

Hi,
I really like your app, it's very handy for me. But I just encountered a bug in Finder. Here's what happened:

Switched to dark mode
Turned off MacBook Pro
Turned it on
Then when I went into the finder some of my folders still had the dark mode look (see here: https://www.dropbox.com/s/wimxfa1l4udnpbb/Bildschirmfoto 2020-03-04 um 11.28.15.png?dl=0)
A MacBook restart fixed this

Maybe you could have a look into this issue. That would be awesome :-)
p.s. Just a small feature suggestion: I would find it so great if you implemented an option to always launch in light mode.

Update: I just noticed that when switching to dark mode (even using macOS' switch) the buggy folders actually stay in LIGHT mode. Maybe there's a macOS bug going on ;-)

This does appear to be a bug with macOS, not Nightfall.


Basic implementation of regexp generator

this is not a regexp generator

ops, I should put as a string generator

string-generator is already used on npm :(

which name we should use?

👍  this project sounds funny, what is the use of it?


[BUG] PLACEHOLDER_CONF 'default_plugins' functionality removed.

Description
We have recently migrated to Django-cms 4, and the placeholder conf settings for default_plugins seem to have been completely removed.
When looking at the source code, and comparing this with older projects running on CMS 3 it seems to have been removed.
We have also found a git commit where the code has been removed:
https://github.com/django-cms/django-cms/commit/eef5cbbfe3b4875b9b00db60611171d700c29ced
Despite it seemingly being removed, docs still have a reference to the PLACEHOLDER_CONF.default_plugins:
https://docs.django-cms.org/en/latest/reference/configuration.html#cms-placeholder-conf
Steps to reproduce

Upgrade to DjangoCMS 4
Create PLACEHOLDER_CONF with 'default_plugins' key.
Nothing happens when rendering the placeholder

Expected behaviour
Fill the page with default plugins when the model is first created or the page is first rendered.
Actual behaviour
Nothing, as the code is removed.
Screenshots
When searching dependencies in DjangoCMS 3

When searching dependencies in DjangoCMS 4

Additional information (CMS/Python/Django versions)
DjangoCMS version is 4.1.0
Do you want to help fix this issue?

[ ] Yes, I want to help fix this issue and I will join the channel #pr-reviews on the Discord Server to confirm with the community that a PR is welcome.
[x] No, I only want to report the issue.

@Ceesvanmook I created a PR in #7919. Can you do me a favor and install it locally for testing:
pip install git+https://github.com/fsbraun/django-cms@feat/default-plugins

Let me know any feedback!

@fsbraun thank you! I've installed it locally in my latest project and have done some testing with multiple models and placeholder configurations. On first glance, everything seems to be working as intended! I'll let you know if there are any updates on the matter.