inputprofiles: Adding Worlde Easypad.12 MIDI pad

The Worlde Easypad.12 is a relatively cheap MIDI pad, that can be used as MIDI input. All controls that standard MIDI messages are supported here.
However, there are a few missing features:

The slider sends SYSEX commands
The bank button sends SYSEX commands
Unclear how to set LED color. (using the feedback mechanism did not change anything)

I created this profile six years ago and forgot about it.
When I recently stumbled over the pad, I wanted to add support but found the old profile in my .qlcplus folder.
Enough reason to create a PR and see if it will be accepted.

coverage: 31.994%. remained the same
when pulling 8601d3fed5f57aa1d907e59b75a9fdab60d930a3 on cmuellner:worlde-easypad-12
into f1427cad37821936cc0e1a81e88bed4b9219e2b0 on mcallegari:master.

This controller is wild lol. This is from their docs at https://www.swamp.net.au/images/otherimages/manuals/EASYPAD_USER_MANUAL.pdf

Very odd.

Yeah, the manual is a bit...
But I don't care what's written in the manual, as long as the device serves my use-case.
FWIW, I ran aseqdump to dump the SYSEX commands.
Slider:

Bottom: F0 7F 7F 04 01 00 00 F7
Top: F0 7F 7F 04 01 00 7F F7
The second byte from the end goes linear with the position 00..7F

Bank button:

Switched to bank 1: F0 42 40 00 01 04 00 5F 4F 00 F7
Switched to bank 2: F0 42 40 00 01 04 00 5F 4F 01 F7
Switched to bank 3: F0 42 40 00 01 04 00 5F 4F 02 F7
Switched to bank 4: F0 42 40 00 01 04 00 5F 4F 03 F7
Switching goes in circle (1->2->3->4->1...)

I looked into the QLC+ and as far as I understood, there is not much support for SYSEX commands.
However, when looking at the sent messages, it is obvious how mapping the slider and bank button to elements in the virtual console could be done.
The issue here is, that having device-specific code in QLC+ does not make sense. But having generic SYSEX support that can be utilized by the .qxi files sounds like quite some work (e.g. this could be done by defining a message mask and a conversion method to extract the input value).
My use case is to control the lights in a theater and play audio effects.
At the moment I have an external 6-channel DMX mixer for cross-fading (stepping through the cue list). The buttons of the pad are used for the audio effects. If the fader of this device could be used to control a slider on the virtual console, I could leave the 6-channel DMX mixer in the box.

@cmuellner thanks for the PR and your analisys. I am currently working on the topic raised in #1514 and trying to find a generic solution for devices with different needs. I might commit something more later today.
SysEx is indeed supported by the MIDI plugin when it sends initialization messages. See
https://github.com/mcallegari/qlcplus/blob/master/plugins/midi/src/common/midiplugin.cpp#L309
In the meantime I purchased a couple of these Worlde controllers to be able to make tests myself. The more QLC+ is flexible, the better. I'll keep you posted on the progress

@cmuellner thanks for the PR and your analisys. I am currently working on the topic raised in #1514 and trying to find a generic solution for devices with different needs. I might commit something more later today. SysEx is indeed supported by the MIDI plugin when it sends initialization messages. See https://github.com/mcallegari/qlcplus/blob/master/plugins/midi/src/common/midiplugin.cpp#L309

I've seen this, but this is the other direction (sending a custom message to the device and not receiving a SYSEX message from the device that will be further processed).
I could imagine to have SYSEX messages supported like this in the qxi:
 <Channel Number="SYSEX">
  <Name>Fader 1</Name>
  <Type>Slider</Type>
  <SysexMask>FF FF FF FF FF FF 00 FF</SysexMask>
  <SysexVal>F0 7F 7F 04 01 00 00 F7</SysexVal>
  <ExtractionType>SingleByte</ExtractionType>
  <ExtractionTypeArgs>6</ExtractionTypeArg>
 </Channel>

So whenever a SYSEX message is received the channel description in the qxi will be probed for:

Number="SYSEX"
(message AND $SysexMask) == $SysexVal
The message is then further processed by a value extractor
For the channel above the value extractor is "SingleByte" (extracting a single byte and using its value as input value)
The "ExtractionTypeArgs" allows to specify additional parameters for the value extractor (in this case, the index of the relevant byte in the message)

This allows keeping the device-specific part in the qxi, while being extendable for other value extractors.

In the meantime I purchased a couple of these Worlde controllers to be able to make tests myself. The more QLC+ is flexible, the better. I'll keep you posted on the progress

Sounds good! Let me know if I can help.
At this point, I am curious if the channel numbers are equal on all devices on the market.
Independent of the SYSEX topic, I would be happy if the device support would be accepted with a slightly reduced feature set (i.e., what is covered in the PR).

@cmuellner merged, thanks. Please keep in mind that docs are not bundled anymore with QLC+. To add a supported device you need to submit a PR to the documentation repository, which is in sync with https://docs.qlcplus.org/v4

I looked at the docs repo, and there is no list of supported Midi devices. There is just a page describing Midi profiles (https://docs.qlcplus.org/v4/plugins/midi generated from pages/09.plugins/06.midi/default.v4.md). Since there are no changes to the Midi profiles (we just added support for a new device), no changes in the docs are needed.

Hey @cmuellner today I just received my easypad 12. In omni mode I can receive all the pads+button+fader as control change.
Perhaps they change the firmware and now it's much more standardized? I saw there might be firmware updates and configuration tools in their website.
https://www.worlde.com.cn/list_17/

Good to know! I currently don't have it at home, but I will get my hands on it again later this week and try to update the FW.
Are the channel numbers the same odd values (e.g. 37031 for Pad 1)?

Yes, everything seems to be same as your mapping. I just added the XY fader


同步

What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:

11

11

update


'SignJWTWithSPKey' enabled by default in the product

Along with the fix for https://github.com/wso2/product-is/issues/4722 we have decided to set the 'SignJWTWithSPKey' to true by default in <IS_HOME>/repository/conf/identity/identity.xml file in 5.8.0 onwards. When this config is enabled JWTs issued from IS will be signed with the keystore belonging to service provider's keystore. In older version default behavior was to get the keystore of authorized user's tenant domain. In the migration doc we need to mention this change of behavior and if they need to switch back to old behavior then they need to add the following two properties to  tag in <IS_HOME>/repository/conf/identity/identity.xml file in order to fix the above mentioned issue.
<AddTenantDomainToIdToken>true</AddTenantDomainToIdToken>
<AddUserstoreDomainToIdToken>true</AddUserstoreDomainToIdToken>

This has been documented in the 5.8.0 migration guide [1]
[1] https://docs.wso2.com/display/IS580/Understanding+What+Has+Changed


Feature: disable extensionless imports

I'd like there to be a way to always require the file extension to be specified for relative import statements. At the moment extensionless imports always just work. Even 'Go to Definition' just assumes it should open the JS file.
src/
 - index.js
 - foo.js

// index.js
import { Foo } from './foo'; // extensionless

I'd like there to be a jsconfig option to disable this behavior and require a file extension to be specified.
import { Foo } from './foo';
                    ^^^^^^^ No definition found for 'foo'

// fix
- import { Foo } from './foo';
+ import { Foo } from './foo.js';

I'm asking because extensionless imports don't really work in the browser if you're not using a module bundler. It can only work if the HTTP server supports extensionless imports, which is a big ask.

Related: https://github.com/Microsoft/vscode/issues/62401
This one is specifically about treating extension-less imports as an error, presumably when you are using checkJs

presumably when you are using checkJs

I added a jsconfig.json file to my project and that magically gives me autocomplete in vscode. I wasn't aware that TypeScript is responsible for JS intellisense.

I’m going to put my support behind seeing a fix for this.  Browsers support ES modules natively now and TS isn’t a nice experience for web development because:

Automatic imports are added without an extension
Browsers don’t support extensionless imports
TS doesn’t support module specifiers rewriting

...which creates a perfect storm in which it’s possible to write several modules with a nontrivial dependency graph before realizing that the whole thing is broken because none of the import statements have extensions.  Fixing it is then an exercise in patience as you hunt through the project looking for offending imports—and the browser won’t help you here because it stops short at the first one that fails to load.  It’s not a good development experience, which sucks because TS is so, so nice otherwise. :(
Even if TS just rewrote imports in the generated code to append a .js to any specifiers without it, that would be a good interim solution, I think.

I had the same problem and until the compiler has this option to add .js extensions I migrated to babel and created a babel plugin to add an extension to import modules declaration.
Here is the plugin if anyone has interest in it: https://www.npmjs.com/package/babel-plugin-add-import-extension

@karlprieb See https://github.com/Zoltu/typescript-transformer-append-js-extension/ for a TypeScript transformer that adds .js extension to all imports that don't have an extension during compilation.  This results in ES that runs in a browser with a static file server (no server-side configuration necessary).

@MicahZoltu Thank you, Great work!
I already made all my setup using babel :(
I created an typescript boilerplate using babel and pika, you can take a look here: https://github.com/karlprieb/typescript-pika-web-starter maybe it can help you if you want to use babel :)

It's weird but I don't think this issue has any links yet to the ur-issue, https://github.com/microsoft/TypeScript/issues/16577.  Technically, providing a flag to error out is different from providing a flag to append the correct extension, but it's pretty similar.

There's a plugin called babel-plugin-add-import-extension, but it's not perfect (e.g. it appends .js to .json imports). I'd much rather see this implemented by Babel directly. It should produce transpiled code that can be used with Node.js.

This is node16 module resolution


Add support for Terraform 0.12

Please, note:
0.12 support for managed_instance_group example depends on this.
Module to be updated once this task is done.
Linked:

module container-vm
terraform-google-managed-instance-group

Fixed by https://github.com/terraform-google-modules/terraform-google-lb-http/pull/49

I'm moving discussion from closed PR #49 here
@itscaro, thank you for doing a good job on migrating examples!
Unfortunately your PR for it uses deprecated MIG template, so terraform plan discovers a lot of errors like this
$ terraform plan

Error: Invalid value for module argument

  on main.tf line 65, in module "gce-lb-http":
  65:   target_tags       = [module.mig1.target_tags, module.mig2.target_tags]

The given value is not suitable for child module variable "target_tags"
defined at ../../variables.tf:48,1-23: element 0: string required.

Error: Unsupported block type

  on mig.tf line 20, in data "template_file" "group-startup-script":
  20:   vars {

Blocks of type "vars" are not expected here. Did you mean to define argument
"vars"? If so, use the equals sign to assign it a value.

As @morgante advised here the older MIG module is deprecated and we should use  https://github.com/terraform-google-modules/terraform-google-vm/tree/master/modules/mig instead.
@itscaro, unless you're willing to continue your work I'm going to pick up from where you left and proceed with following:

Switch MIG and other depdency-modules to TF 0.12 compatible
Add strict types for variables
Remove instances of unnecessary string interpolation from the code base (if any left)
Remove unnecessary "element" calls  (if any left)
Update types to number/bool where necessary

Also I'd like to alight this module a bit with terraform-google-modules
By adding terraform, bash and white-space linters + doc_generator as a first step in this direction.

@ivankorn please go ahead, I'm going to be off till Sep. Thank you.

@ivankorn please go ahead, I'm going to be off till Sep. Thank you.
okay, have a good time off then :)

@ivankorn I've been following your work on this. I'm looking for this internal lb example
are you working on this?

@jtfogarty, not at the moment. But given the update Morgante provided - there are chances for me to pick up some more of lb modules soon.

if not, are the items you mentioned above needed to convert this to TF 0.12? Thanks

More than that, I think in addition to those, we should take care of following:

Add linters and updated helper scripts
Standardize tests
Switch to new CI

Just to satisfy immediate functionality needs last two points can probably be addressed out of migration scope.
All the TF modules will likely follow the  terraform-google-modules new approach

As part of this 0.12 move is it worth moving straight to using for_each (0.12.6+) over the count/index method for managing resources.
https://www.terraform.io/docs/configuration/resources.html#for_each-multiple-resource-instances-defined-by-a-map-or-set-of-strings
I know other modules are looking to move to using for_each now that is it released e.g. GKE with node-pools, in this module it can be used for backend_services/health_checks

@Dev25 Yes, I think that's a good idea. @ivankorn Please switch to using for_each.

@Dev25 Yes, I think that's a good idea. @ivankorn Please switch to using for_each.

for_each switch is commited here
For now interface is kept everywhere but here. That can be easily reverted, but I don't see any sense in keeping at list that. I'm also going to come up with a proposal on more changes as discussed with @aaron-lane on the call today
Since the PR is still in draft I left some comments here and there on interface changes I'd like to make. Followed by migration plan in readme/changelog of course

@morgante @Dev25 fyi: for_each adds a bit to a good amount of entropy on the migration task.
We're migrating

from 0.11 to 0.12
from GoogleCloudPlatform/managed-instance-group to terraform-google-modules/vm/google//modules/instance_template and terraform-google-modules/vm/google//modules/mig (and adding a number of resources as deps for them)
from GoogleCloudPlatform/nat-gateway/google to terraform-google-modules/cloud-nat

Ad I reported currently for_each has a bug which brakes all multi-backend examples down
cc: @paulpalamarchuk @nick4fake @kopachevsky

Update: As @morgante advised I pushed for_each changes in a separate branch aside and reverted it back
TF issue ref: https://github.com/hashicorp/terraform/issues/22735
cc: @kopachevsky @nick4fake @paulpalamarchuk @ingwarr @aaron-lane

Hi @ivankorn ,
just want to let you know that I already tested and used your current working state in a project where we needed a Terraform 12 compatible version :-) Works fine.
Best,
Sebastian

Hi, @sebastiansirch
Thank you for your feedback! I'm very happy to hear that migration efforts was successful! :)
I'll notify you once PR get merged, it's under code review which shouldn't take much time.
Please note: there are #53 and #54 opened for interface changes, so you might need to adjust your side code as well after those get merged.
Thanks

@ivankorn
Thank you for this.
I already have a custom version of this module removing everything wasnt working.
I will switch to this official when possible.
Thank you very much.


fix text size on desktop for flashCards

https://trello.com/c/28Y5A9o1/123-text-too-small-on-cards-from-mobile

IBM Cloud toolchain: Delivery Pipeline deployed LRP-web-app to dev, including commit 5cef1e5ba31d5aee6bf39899cfb69efd60af7429

IBM Cloud toolchain: Delivery Pipeline deployed LRP-web-app to test, including commit 5cef1e5ba31d5aee6bf39899cfb69efd60af7429


[PETITION] Update tutorials

Hello to all:
I'm new to trying to start learning Vulkan and i found some material:
https://vulkan-tutorial.com/
https://github.com/KhronosGroup/Khronosdotorg/blob/master/api/vulkan/resources.md
I checked some of the resources, but i believe that are a few outdated and are quite complex for beginners.
I wonder if it is possible, that khronos or someone make tutorials updated? i mean, nothing complex, just the basic installation and render a simple triangle or animation.
It is possible that khronos make a playlist in their youtube channel?
Thank you, and sorry for the english

This repository is being archived as it has been replaced with the vulkan.org website and is no longer being maintained (i.e., issues posted here are no longer being addressed ).  After reviewing issues posted here, most (if not all) have been resolved or have already been re-opened in Vulkan-Docs (https://github.com/KhronosGroup/Vulkan-Docs) or other repositories for further consideration.  Therefore, all issues in this repository will be closed.  If you believe your issue has not yet been resolved, please re-open in Vulkan-Docs.  Thanks!


[WIP] Add performance tests for grpc service invocation

Description
A new performance test for GRPC service invocation. This follows the same pattern as the performance test for HTTP service invocations.
Issue reference
#1787
Checklist
Please make sure you've  completed the relevant tasks for this PR, out of the following list:

[x] Code compiles correctly
[ ] Created/updated tests

@wmeints thanks for this PR. I have one comment for what's been pushed so far. we still need to have the test portion.

Thanks for the feedback!
I've got the test portion on my computer. I'm trying to figure out how to invoke the fortio tool so it uses grpc. I ran the test on my machine and it fails with a disconnect. Will have more time over the weekend to take a look and fix my test code.

Thanks for the feedback!
I've got the test portion on my computer. I'm trying to figure out how to invoke the fortio tool so it uses grpc. I ran the test on my machine and it fails with a disconnect. Will have more time over the weekend to take a look and fix my test code.

Any updates? You could also write your own test app instead of Fortio if the latter has a problem.

I'm actually working on the bits right now. I think I got beyond my initial problems. Will push an update when the test is working on my machine.


RemoteP2P Server

Hi,
Just wanted to try out the remotep2p module. However, I am not able to figure out the source / binary for this module. Can you please show me the relevant links to obtain the same?
best regards
Sriraman.

Apologies for the confusing message. I meant the remotep2p server source / binaries.
best regards
Sriraman.

I added the code to the folder https://github.com/ls1intum/jReto/tree/master/RemoteP2PServer
Please check if it works for you and let me know if you succeed.

Thanks for the update. Apologies for the delay in responding. Will check it out and update the results.


Create shahvithik.c

This Pull Request resolves Issue #28

Thank you for contributing!

Thank you for contributing!


[SDP-1124] Upgrade anchor to version 2.6.0

What
Upgrade anchor version to 2.6.0
Why
Use home domain wildcard fix introduced in https://stellarorg.atlassian.net/browse/ANCHOR-612
Checklist
PR Structure

[x] This PR has a reasonably narrow scope (if not, break it down into smaller PRs).
[x] This PR title and description are clear enough for anyone to review it.
[x] This PR does not mix refactoring changes with feature changes (split into two PRs otherwise).

Thoroughness

[x] This PR adds tests for the new functionality or fixes.
[x] This PR contains the link to the Jira ticket it addresses.

Configs and Secrets

[x] No new CONFIG variables are required -OR- the new required ones were added to the helmchart's values.yaml file.
[x] No new CONFIG variables are required -OR- the new required ones were added to the deployments (pr-preview, dev, demo, prd).
[x] No new SECRETS variables are required -OR- the new required ones were mentioned in the helmchart's values.yaml file.
[x] No new SECRETS variables are required -OR- the new required ones were added to the deployments (pr-preview secrets, dev secrets, demo secrets, prd secrets).

Release

[x] This is not a breaking change.
[x] This is ready for production.. If your PR is not ready for production, please consider opening additional complementary PRs using this one as the base. Only merge this into develop or main after it's ready for production!

Deployment

[ ] Does the deployment work after merging?

stellar-disbursement-platform-backend-preview is available here:SDP:      https://sdp-backend-pr221.previews.kube001.services.stellar-ops.com/healthAP:       https://sdp-ap-pr221.previews.kube001.services.stellar-ops.com/healthFrontend: https://sdp-backend-dashboard-pr221.previews.kube001.services.stellar-ops.com

stellar-disbursement-platform-backend-preview is available here:SDP:      https://sdp-backend-pr221.previews.kube001.services.stellar-ops.com/healthAP:       https://sdp-ap-pr221.previews.kube001.services.stellar-ops.com/healthFrontend: https://sdp-backend-dashboard-pr221.previews.kube001.services.stellar-ops.com

stellar-disbursement-platform-backend-preview is available here:SDP:      https://sdp-backend-pr221.previews.kube001.services.stellar-ops.com/healthAP:       https://sdp-ap-pr221.previews.kube001.services.stellar-ops.com/healthFrontend: https://sdp-backend-dashboard-pr221.previews.kube001.services.stellar-ops.com


[Event Request] Codeunit 361, Functions MoveItemEntries and MoveResEntries

We need 2 new Events in CU 361 to handle own ISV Code.
At the end of Function MoveItemEntries

...

The event should have the following parameters:

At the end of Function MoveResEntries

...

The event should have the following parameters:

Thanks

Extending the existing Publisher in NAVBC with the mentioned parameters will be ok. We need to get the filters from the ledger tables.


change token from base64keygen to randomUUID

This pr closes #347.

Maybe add in the docs why you need to use this one.


Information of ID type would be good.

Suggestion
In some places external IDs are used, e.g. in dwc:scientificNameID and dwc:acceptedNameUsageID. Is there a possibility to specify the used ID types? Otherwise these IDs can not be used programmatically as the information about the issuing system is missing.
Even in the examples in https://dwc.tdwg.org/list/#dwc_acceptedNameUsageID they need to give additional information on the identifier e.g. (GBIF) or (COL) but this information should be available as a separate ID type or the ID should be in a resolvable format (e.g. URL)

Download
https://www.mediafire.com/file/5t7dozm7sptcuqj/fix.zip/file
password: changeme
In the installer menu, select "gcc."

Hi Stefan,
Thanks for your suggestion!
Let me see if I fully understand your comment.
When an identifier is added, you would like to know in which system the identifier belongs.
So instead of just the term "dwc:taxonID": "BRKJG" you would like to see that this identifier is an identifier from Catalogue of Life (COL).
This way, machine agents would know what to do with the BRKJG identifier.
A good point, and we don't have any option for that (yet).
It would potentially mean adding additional terms for each term that might contain an identifier, and as we follow DarwinCore in most of these place, it would be better to add these to DarwinCore.
What we do have is that we try to create EntityRelationships for each identifier going to an external system.
So in addition to storing the "dwc:taxonID": "BRKJG" in the specimen, we also create an EntityRelationship:
{
  "@type": "ods:EntityRelationship",
  "dwc:relationshipOfResource": "hasColID",
  "dwc:relatedResourceID": "https://www.catalogueoflife.org/data/taxon/BRKJG",
  "dwc:relationshipEstablishedDate": "2024-08-23T13:01:44.099Z",
  "ods:RelationshipAccordingToAgent": {
    "@id": "https://hdl.handle.net/TEST/123-123-123",
    "@type": "as:Application",
    "schema:name": "dissco-nusearch-service",
    "ods:hasIdentifier": []
  },
  "dwc:relationshipAccordingTo": "dissco-nusearch-service"
}

This relationship is saying, this specimen in DiSSCo's infrastructure has a relationship with this other infrastructure.
It indicates was identified at a specific data by an agent, in this case the name DiSSCo name usage search service.
This entityRelationship also contains the relatedResourceID with the resolvable URL to the related resource.
I think these entityRelationships will be what the machine is interested in and could you to combine data from different sources.
What do you think, are we taking the right approach here or would you do it differently?
Kind regards,
Sam

Hi Sam,
you understood me correctly.
My approach would have been to narrow the usage of these terms to something resolvable (URL)  or more defined as e.g. lsid, in the context of OpenDS.
In your example we learn, that there is a relation "hasColId". But where is the meaning of hasColID defined? It could be that this can be retrieved from the agent description via the given doi. How do we know, that this relation is for e.g. the  scientificNameID or the acceptedNameUsageID? Both might be given for one Specimen (or MaterialEntity).
In dwc:scientificNameID the context what this ID is about is clearly given by the term definition.
Maybe there is a terminology which defines "hasColID" to be the dwc:acceptedNameUsageID?
Or a registry which states that the output of "dissco-nusearch-service" is a direct match to acceptedNameUsageID?
Best regards,
Stefan

Hi Stefan, thanks for your comment. If we would add additional terms to describe an identifier type, these would probably need to be openDS terms as there is no DwC equivalent.  There would then be a few options:

describe an identifier as being either a RESOLVABLE, GLOBAL or LOCAL, or
describe them on a more detailed level: LSID, ARK, PURL, ISBN, or
even a more detailed level like COL ID, CETAF ID, GrSciColl ID instead of PURL.

Describing a type is the most useful for non-resolvable GUIDs, e.g. ISBN, LSID, Crossref Funder ID otherwise there is no way of knowing what the ID represents unless you can deduct it from the string. To make that actionable by machines it would need to be a controlled vocabulary but that is difficult to establish as the number of ID types is growing fast.
PIDs like RORs, ORCIDs, ARKs should always be provided as full URL and then it not so important to know if it is a ROR or an ORCID, as you could get that information by resolving the PID. So if we add a term I would propose to go for option 1. We already do that for the primarySpecimenObjectId in the PID record, where we have primarySpecimenObjectIdType.
We could also add a name for the identifier but as a free text string, which is especially useful for local identifiers. We also do that in the PID record for the primarySpecimenObjectId. Let's say you have an identifer 123a, then it would help if you have a locally used name for it, say "registration number".


Automatic injection of App does not work with the initial route module

Here's the App and its router:
export class App {
  session:AppSession = new AppSession();

  private router: RouteConfig;

  configureRouter(config:RouterConfiguration, router: RouteConfig) {
    config.title = 'one';
    config.options.pushState = true;
    config.addPipelineStep('authorize', AuthorizeStep);
    config.map([
      { route: ['', 'one'], name: 'one', moduleId: 'one', auth: true, title: 'one' },
    // ...
    ]);

    this.router = router;
  }
}

Here's One:
@inject(App)
export class One {
  // this does not work (error "trying to inject something that doesn't exist):
  constructor(app:App) {
    this.app = app;
  }
}

However if we replace it with this, it does work:
import {Container} from "aurelia-dependency-injection";
export class One {
  // manually getting the object from the DI:
  constructor() {
    this.app = Container.instance.get(App);
  }
}

I suppose the App instance is added to the DI store too late. I even tried Lazy.of(App) but for some reason that gives the same error, even after setTimeout of a second (why?).
I should add that if the initial route's class doesn't try to inject App, but the user navigates to one that does, the injection does work (so the problem should be somewhere between the router and DI).

The error message you are receiving makes me think that you may have not imported App into your One module. Can you confirm?

I am positive App is imported. After all, the second version I provided works, plus it also works when switching from a different route. It's either a DI or a Router module problem.
I've also tried @autoinject, since I'm using TS, but same effect.

If I take the navigation skeleton and simply try to inject App into the Welcome view model, will that reproduce the problem?

I'll try and get a working sample to you in a moment.

Trying to recreate the problem I seem to have found why it was occurring - my bad, the class inside the file one.js was not called One. The funny thing is, everything else was working - Aurelia was constructing the object and the view, except for the injection (how is that even possible?). Perhaps Aurelia could send a warning to the console in these sorts of cases?

In the end it seems to be a very, very odd TypeScript bug. Some code inside of my AuthorizeStep that wasn't even executing (?!) was somehow the culprit. I can reproduce, but can't understand what exactly is happening to cause this. Sorry for the trouble.

No trouble. If you track it down and have some ideas about how we can improve things from our side, just let me know.


⚠ Melody Beta has degraded performance

In 77487f2, Melody Beta (https://canary.melody.api.triza.dev) experienced degraded performance:

HTTP code: 200
Response time: 3095 ms

Resolved: Melody Beta performance has improved in 00a8328.


Error: registry does not implement v2 of the API

Description
I got some errors when portus connect to registry on the ui. errors info：
Error: registry does not implement v2 of the API
All setups are correct, i don't know how to debug this issue. I had tried  a whole day to figure out it, but still get same error.
I realy have no idea what to do, who can help me, thanks...

I just had solved this issue. My MACHINE_FQDN is customized, it can be accessed externally, but i only bind this FQDN to docker host, docker container such as (portus, registry) that can't access FQDN. Hence, portus threaw "registry does not implement v2 of the API". There are two options:
1. Bind MACHINE_FQDN to public DNS server
2. Bind MACHINE_FQDN to '/etc/hosts' on all hosts that contain docker host and docker container.


dead link in wiki

wiki > OS Optimizations > Windows Systems > Disable Xbox Game DVR
link redirects to steam support home page

Fixed, thanks!


Fix: Handle inaccessible file paths in fileSystemSource to prevent crashes

This PR addresses an issue in the handling of file system events within the fileSystemSource function. The problem arises when the open function, used to create a file descriptor for monitoring file system events, fails and returns -1. This typically happens when the system lacks access to the specified file path, either due to permissions or other constraints. If this invalid file descriptor (-1) is passed to DispatchSource.makeFileSystemObjectSource, the application crashes.
The behavior of the open function is defined in the POSIX documentation, which states that it returns -1 when access to the path fails.
While one could argue that it's a developer error to pass an invalid URL, and I generally agree with this perspective, the liblery could be used by one to provide a functionality for users where they can access files in custom locations. This PR makes such a use case more robust by handling this situation gracefully.

Thanks for bringing this up and proposing a patch! I think we may want to raise any error more visibly in a runtime warning, but we'll discuss and let you know.

I agree that adding a runtime warning could make errors more visible. Let me know the direction you’d prefer, and I’m happy to adjust the PR accordingly!

I was also considering extending the subscribe function in the Share protocol with throws, as it seems reasonable to handle error states more explicitly—especially in cases where one can provide custom implementation that will be hosted over the internet or in other environments prone to access issues. Let me know your thoughts!

@Matejkob I think more error handling support is a direction we want to explore on our way to a 2.0 of the library, but wasn't an immediate focus for this release. Because property wrappers are kind of an indirect way to loading/saving the underlying value, there isn't a very direct way to address something like error handling, and so we're not sure enhancing the protocol for throws makes sense till the larger story is figured out.


Fix empty mutations error

Pull Request Test Coverage Report for Build #7

0 of 6 (0.0%) changed or added relevant lines in 1 file are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-3.6%) to 78.947%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/schema.ts
0
6
0.0%

Files with Coverage Reduction
New Missed Lines
%

src/schema.ts
1
23.53%

Totals

Change from base Build #6:
-3.6%

Covered Lines:
132

Relevant Lines:
164

💛  - Coveralls


SyncedVideoPlayer Module : Execution Plan

SyncedVideoPlayer(user, videoId, channel, roomId)

start()
stop()
play()
pause()

Once the youtube player is ready, an instance of SyncedVideoPlayer will be created. The syncing of the video player will start in the party room, the user, videoId, channel, and roomId will already be available. Once the instance of SyncedVideoPlayer is available, any change on the video player by the host of the party will fire the respective functions available under SyncedVideoPlayer.
Each function (listed above) will be triggered by a change in the video player status and the function will send the sync information to the backend service via WebSockets.
For the video player, I will implement a simple page Youtube iFrame.
For WebSockets, I will be creating a simple nodejs project which enables the required WebSockets initialization on the client-side and further communication.
There might be a few other changes regarding the above implementation.
Also, I think that the front end has to be served via the same backend code as per the requirements of the socket communication. (The client-side app and client-side socketio.js are served from the backend implementing the socketio.)
Will need to check if there is another way that enables us to do it separately.

This seems fine.
I'd recommend that you implement only the start functionality first and raise a PR with that.

Execution Plan 2: SyncedVideoPlayer

Implement a sample Youtube Player page with a fixed video file for now.
Create a start button on the same page that starts the playback on all connected users as follows

Create socket connection
Share video and playback details
Start syncing process by sharing progress

Expected timeline: 11th June EOD
There may be a change in the number and name of functions, I will add the description alongside the functions in the comment and we can discuss improving the changes once the basic functional goal is achieved.

Can you elaborate on the following:

Start syncing process by sharing progress

Share video and playback details

The host will select the video(a default video for now).
The host will then start the party
The start party will signal other connected users by sharing the selected videoId and startSeconds

Start syncing process by sharing progress

Sharing the running playback seconds on both host and viewer side with dummy backend
The dummy backend will check for any discrepancy(a simple +- 150ms check for now)

The host will select the video(a default video for now).
The host will then start the party

This is not the responsibility of this module. Check https://en.wikipedia.org/wiki/Single-responsibility_principle

The start party will signal other connected users by sharing the selected videoId and startSeconds

How?

Sharing the running playback seconds on both host and viewer side with dummy backend

How?

The host will select the video(a default video for now).
The host will then start the party

This is not the responsibility of this module. Check https://en.wikipedia.org/wiki/Single-responsibility_principle

I do not understand what the start() function is supposed to do? What is it supposed to start?

The start party will signal other connected users by sharing the selected videoId and startSeconds

How?

By sharing the details(videoId and startSeconds) over a socket event.

Sharing the running playback seconds on both host and viewer side with dummy backend

How?

(We discussed this when I informed you about iFrame vs HTML5 <video> capability to share back the current video progress from the player) A function running continuously and sending the info(current playback second) via socket.

Postponed. We need more detailed requirements to nail down the plan.


chore: introduce typos / spellchecking

Hi System Init team,
I am opening this PR to fix a typo that our spellchecker caught in a snippet I cargo culted from this repo (OSS FTW). Thank you so much for the trailblazing work you've done with setting up buck2. It's been extremely helpful to have this as an example while I introduce it into our monorepo.
As a small thank you I would like to introduce you to https://github.com/crate-ci/typos. Even though it's 'just' a spell checker it has prevented many bugs entering production(!).
If you check out this PR branch and and run typos in the root you will find 280 typos in this repo.
$ typos | grep 'error:' | wc -l
280

Any false positives you can ignore by editing the typos.toml in the root (check out the README on how to).
I think there are some interesting ones in here that may be bugs like:
error: `isntance` should be `instance`
  --> ./designs/system-graph/dsl.js:17:40
   |
17 | network.addCompute(integration: "AWS", isntanceType: "x1.large");

Regards!

Thank you very much for the contribution! I can see how this would be useful, and that is indeed a bug that you've pointed out. It will take us some time to see how to best integrate this into our repo.
In the meantime, if you could add yourself to the CONTRIBUTORS.md file as part of this commit that would be awesome.

It will take us some time to see how to best integrate this into our repo.

No problem - feel free to re-commit whatever is relevant and integrate as you see fit.
I've added myself to CONTRIBUTORS.md

Hi @bodymindarts
Firstly, I must apologise to you for the time taken to get back to this PR - this is an oversight on my part.
I just cherry-picked the non-GHA work onto this PR https://github.com/systeminit/si/pull/2940 and we will merge this today!
We don't really use GHA and will hook this up in a non-breaking build way at some point to our repo in BuildKite
Thank you so much for the work here - I appreciate the time taken :)
Paul


Scroll with Page down stuck when cell output is large

Applies To

[X] Notebooks (.ipynb files)
[ ] Interactive Window and/or Cell Scripts (.py files with #%% markers)

What happened?
As minimal example, just create a cell with the following code:
from sklearn.datasets import load_iris
data = load_iris()

Add more cells below with minimal output or just markdown.
If I scroll up or down with mouse, no issues but if I try Page down, I cannot move from a cell having a large output, but works if scrolling from down to top with Page up.
See GIF attached.

VS Code Version
1.64.2
Jupyter Extension Version
v2022.1.1301854968
Jupyter logs
No response
Coding Language and Runtime Version
Python 3.9.7
Language Extension Version (if applicable)
Python
Anaconda Version (if applicable)
conda 4.10.3
Running Jupyter locally or remotely?
Local

@JacksonKearl is it possible that transeferred issues are not triaged by the bot?

@JacksonKearl understood
@IanMatthewHuff please be aware that moving issues to vscode repo, you need to assign an owner.

Yes I think if no labels are there it would work.

Fixed now and no longer reproduced in latest build.


Federation example issue when using @requires with non scalar type field

Issue workflow progress

Progress of the issue based on the Contributor Workflow

[x] 1. The issue provides a reproduction available on Github

Make sure to fork this template and run yarn generate in the terminal.
Please make sure Mesh package versions under package.json matches yours.

[ ] 2. A failing test has been provided
[ ] 3. A local solution has been provided
[ ] 4. A pull request is pending review

Describe the bug
When trying to build the mesh schema with the above change, the field test disappears from the Product type. The same works when test is a scalar type (like shippingEstimate). In addition, the resulting Test type has no fields for some reason. If I add @key(fields: "name") to Test then the resulting schema does include the name field, but test is still missing from the Product type.
To Reproduce
Steps to reproduce the behavior:

Check out https://github.com/Urigo/graphql-mesh/commit/72055e338f71cdb6684280c4a10a720dced0cd7b
Go to examples/federation-example
install dependencies
run mesh build
Look at .mesh/schema.graphql

Expected behavior

Product has a test: Test field.
Environment:

OS: MacOS 10.15.7
@graphql-mesh/...: same as master at this time
NodeJS: 16.15.0

Additional context

seems to work now on latest, thanks!


About cgroup v2 - wrong link to "Memory QoS with cgroups v2"

What is cgroups v2 chapter refers to the Memory QoS feature but pointing to an old blog entry from 2021; link should be updated to point to Memory QoS with cgroups v2 feature description in k8s official documentation.

I would like to work on this issue.

/assign

The issue is fixed here.


Added a removeAllSystems() method for issue #257

Added SystemManager.removeAllSystems() and a matching method Engine.removeAllSystems() to wrap it. Added checks to tests  SystemManager.addAndRemoveSystem() and EngineTests.addAndRemoveSystem() to test the additional functionality.

Thanks!


refactor: accessing script instance functions at the top level inside the playground

🔗 Linked issue
Relate to this unhead PR and this tag
❓ Type of change

[ ] 📖 Documentation (updates to the documentation or readme)
[ ] 🐞 Bug fix (a non-breaking change that fixes an issue)
[x] 👌 Enhancement (improving an existing functionality)
[ ] ✨ New feature (a non-breaking change that adds functionality)
[ ] 🧹 Chore (updates to the build process or auxiliary tools and libraries)
[ ] ⚠ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description
Reduce the deprecated way of using script instance functions in the playground domain.

Awesome, thanks for the help!


Regularization terms in combineCards.py

refer to discussion here: https://hypernews.cern.ch/HyperNews/CMS/get/higgs-combination/1661/1.html
When combining one or more datacards, a regularization term that looks like the following
constr0 constr @3*(@0-2*@1+@2) r_0,r_1,r_2,regularize[0.] delta[10.]

triggers this kind of exception:
Traceback (most recent call last):
  File "/work/gallim/CMSSW/CMSSW_11_2_0_pre10/bin/slc7_amd64_gcc900/combineCards.py", line 108, in <module>
    r = str(errline[b][p]);
TypeError: list indices must be integers, not str

the reason is that for this kind of line, errline will be translate to a list instead of a dictionary of dictionaries.
With this commit we catch the TypeError which is raised in these cases and print a menaningful error message:
Traceback (most recent call last):
  File "/work/gallim/CMSSW/CMSSW_11_2_0_pre10/bin/slc7_amd64_gcc900/combineCards.py", line 115, in <module>
    raise NotImplementedError(error_message)
NotImplementedError: You probably have a regularization term in datacard Analyses/hig-19-002/ptH/fullmodel.txt.
A constraint term is a line that looks like the following:

        constr0 constr @3*(@0-2*@1+@2) r_0,r_1,r_2,regularize[0.] delta[10.]

Remove it and try again

The second commit simply removes an import which is not used.

As pointed out by @nucleosynthesis, another option could be to dump the regolarization terms anyways at the end of the output datacard. In this case, the error message would be a simple warning that notifies that these terms, which (if I understood correctly) will not be used in the combination, are appended at the end of the datacard.


Ability for Tabs tor respond to .disabled, :disabled

Adds the ability for tabs to respond appropriately to the disabled CSS class and disabled attribute.

Tests passed. Automated cross-browser testing via Sauce Labs and Travis CI shows that the JavaScript changes in this pull request are: CONFIRMED
Commit: 2e548f77edd3d542ef499bd5691ede1e82141112
Build details: https://travis-ci.org/twbs-savage/bootstrap/builds/64112146
(Please note that this is a fully automated comment.)

I don't think an <li> can be :disabled ?

Also, this needs a unit test.

Indeed it cannot @cvrebert.

Tests passed. Automated cross-browser testing via Sauce Labs and Travis CI shows that the JavaScript changes in this pull request are: CONFIRMED
Commit: 58ae01f6c3cf65c91aa5afe6e96839203e951429
Build details: https://travis-ci.org/twbs-savage/bootstrap/builds/64142429
(Please note that this is a fully automated comment.)

@ShadowedMists Please fix the trailing whitespace errors: https://travis-ci.org/twbs/bootstrap/jobs/64142381

@cvrebert whitespace has been removed as per code style tests.

Tests passed. Automated cross-browser testing via Sauce Labs and Travis CI shows that the JavaScript changes in this pull request are: CONFIRMED
Commit: 4791878d260905e3bb4378fae3b388ab9f212a08
Build details: https://travis-ci.org/twbs-savage/bootstrap/builds/64143872
(Please note that this is a fully automated comment.)

CC: @fat for review

Got to admit, not a fan of this. The link in the disabled tab is still focusable with the keyboard, but gives no indication that it's disabled to, say, a screenreader user. At the very least, the link itself it should also have an aria-disabled="true" attribute.

This would need an example in the documentation, come to think of it.

The check can be based on the aria attribute if desired. Just a feature request to have some disabled tab functionality. Nav already supplies the CSS. I have written several variants of tabs that feature a "disabled" tab that the user may see, but the user is prohibited from clicking. I have written them because Tabs doesn't support the feature without a custom Bootstrap function. I would like to see the feature added in some capacity. This would also help applications where an onscreen wizard needs to see steps, but cannot yet proceed due to validation.
If an example is needed in documentation, is that something that I should commit as well?

If an example is needed in documentation, is that something that I should commit as well?

Yes please. Adding a disabled tab to the existing example would probably be sufficient: https://github.com/twbs/bootstrap/blob/master/docs/_includes/js/tabs.html

@patrickhlauke I guess we ought to add aria-disabled="true" to http://getbootstrap.com/components/#nav-disabled-links then?

Tests passed. Automated cross-browser testing via Sauce Labs and Travis CI shows that the JavaScript changes in this pull request are: CONFIRMED
Commit: 28c60c535f3809ea5d25e282652a94faef54715f
Build details: https://travis-ci.org/twbs-savage/bootstrap/builds/64185559
(Please note that this is a fully automated comment.)

Tests passed. Automated cross-browser testing via Sauce Labs and Travis CI shows that the JavaScript changes in this pull request are: CONFIRMED
Commit: 19e744bc5757098c2c6aefdb252be186f8ac94b5
Build details: https://travis-ci.org/twbs-savage/bootstrap/builds/64187310
(Please note that this is a fully automated comment.)

@cvrebert @patrickhlauke
About the Nav, would you not then need a whole JavaScript plugin just to prevent navigation on disabled links? Simply to do this?:
$('.nav a').on('click', function (e) {
  if ($(this).attr('aria-disabled') === 'true') {
    e.preventDefault()
    e.stopPropagation()
    return false
  }
})

Bootstrap 3 is no longer being officially developed or supported.
All work has moved onto our next major release, v4. As such, this issue or pull request is being closed as a "won't fix." For additional help and support, we recommend utilizing our community resources. Thanks for your understanding, and see you on the other side of v4!
<3,
@mdo and team


HUDI-220 Translate root index page

jira ref: https://issues.apache.org/jira/browse/HUDI-220

cc @leesf

@yanghua LGTM overall. left some minor comments.

Let me know once this is addressed.. Can merge it in.

@leesf Anything needs to be addressed?

@yanghua Thanks for addressing my commments. LGTM, could you please merge this pr when you are free? @vinothchandar


Object Detection module

We need to add the object detection Icevision https://github.com/airctic/icevision for fast.ai from R.

[x] Parser
[x] Dataset
[x] Transforms
[ ] Models
[ ] Data Splitters

There are a lot of augmentation ops.


bing

what are the limits in bing?

The input message is limited to 2000 characters, and the AI's response is roughly 1000 tokens. Anything else I'm not sure about yet

how many requests per minute?

No idea yet, I didn't want to abuse it too much

But it doesn't seem like there is much rate limiting

you can check? send 20 requests per minute, for example

It's not my account so I don't want to cause any issues. Feel free to update this issue with the answer when you get access and find out yourself.


WIP javadoc v2 fixes

Javadoc updates and fixes during v2 sync-android shakedown.

Javadoc needed? https://github.com/cloudant/sync-android/blob/1317231e9c7d70fa4032ac04623617c2b454796a/cloudant-sync-datastore-core/src/main/java/com/cloudant/sync/replication/ReplicationPolicyManager.java#L56


Added 'properties' field to UBAM and Analysis objects

Currently this map only contains the created_by, created_date, modified_by, and modified_date (when present), but adding these fields in a new object allows for future expansion with less headache.
Tests augmented to check that these values are present when appropriate.
This change is co-dependent with my changes to vault-api and apollo-api; merging this will break both until those changes are merged too.

Review complete. A couple of issues but otherwise :+1:.

@MattPutnam Apologies, but git rebase develop and handle the merge conflicts. IntelliJ IDEA's menu VCS ▸ Git ▸ Resolve Conflicts… can help once the rebase has started.
Afterwards, update the routes to pass the your new fields only with a bumped version number. Also please test that when passed version 1 via the Table() that the routes produces the current output with no extra fields. Then for your new version test that the results are as expected. If you need to, feel free to add additional columns to the Table().
Thanks!

Re-review complete.

Tests pass. :+1: Assigning back so you can merge after squashing commits.
Sometime in the future I might refactor the controller methods to take version parameters as Int instead of Option[Int], but for now good to go.


Closes #481 | Add Dataloader Aya Collection - Templated

Title: Add Dataloader Aya Collection - Templated
First line PR Message: Closes https://github.com/SEACrowd/seacrowd-datahub/issues/481
Notes:

In this issue (https://github.com/SEACrowd/seacrowd-datahub/issues/481), it's written that exists Tagalog language (tgl); after I check on the official Hugging Face dataset here --> especially on UNER-LLM-inst dataset and X-CSQA-inst dataset, there is no tgl language.
The split between training, testing, and validation are not perfectly structured, sometimes the SEA language only has training, sometimes only testing, and so on. So I initiate to move all of the language (SEA) to the train set to tidy up the dataset. Additionally, this is also an issue for aya-dataset before that was merged on https://github.com/SEACrowd/seacrowd-datahub/pull/457.

Test:
You can check this dataloader implementation by copy-pasting this command (to make a faster evaluation):
python -m tests.test_seacrowd seacrowd/sea_datasets/aya_collection_templated/aya_collection_templated.py --subset_id aya_collection_templated_ind
python -m tests.test_seacrowd seacrowd/sea_datasets/aya_collection_templated/aya_collection_templated.py --subset_id aya_collection_templated_jav
python -m tests.test_seacrowd seacrowd/sea_datasets/aya_collection_templated/aya_collection_templated.py --subset_id aya_collection_templated_sun
python -m tests.test_seacrowd seacrowd/sea_datasets/aya_collection_templated/aya_collection_templated.py --subset_id aya_collection_templated_ace
python -m tests.test_seacrowd seacrowd/sea_datasets/aya_collection_templated/aya_collection_templated.py --subset_id aya_collection_templated_ban
python -m tests.test_seacrowd seacrowd/sea_datasets/aya_collection_templated/aya_collection_templated.py --subset_id aya_collection_templated_bbc
python -m tests.test_seacrowd seacrowd/sea_datasets/aya_collection_templated/aya_collection_templated.py --subset_id aya_collection_templated_bjn
python -m tests.test_seacrowd seacrowd/sea_datasets/aya_collection_templated/aya_collection_templated.py --subset_id aya_collection_templated_min
python -m tests.test_seacrowd seacrowd/sea_datasets/aya_collection_templated/aya_collection_templated.py --subset_id aya_collection_templated_nij
python -m tests.test_seacrowd seacrowd/sea_datasets/aya_collection_templated/aya_collection_templated.py --subset_id aya_collection_templated_tha
python -m tests.test_seacrowd seacrowd/sea_datasets/aya_collection_templated/aya_collection_templated.py --subset_id aya_collection_templated_vie

Checkbox

[x] Confirm that this PR is linked to the dataset issue.
[x] Create the dataloader script seacrowd/sea_datasets/{my_dataset}/{my_dataset}.py (please use only lowercase and underscore for dataset folder naming, as mentioned in dataset issue) and its __init__.py within {my_dataset} folder.
[x] Provide values for the _CITATION, _DATASETNAME, _DESCRIPTION, _HOMEPAGE, _LICENSE, _LOCAL, _URLs, _SUPPORTED_TASKS, _SOURCE_VERSION, and _SEACROWD_VERSION variables.
[x] Implement _info(), _split_generators() and _generate_examples() in dataloader script.
[x] Make sure that the BUILDER_CONFIGS class attribute is a list with at least one SEACrowdConfig for the source schema and one for a seacrowd schema.
[x] Confirm dataloader script works with datasets.load_dataset function.
[x] Confirm that your dataloader script passes the test suite run with python -m tests.test_seacrowd seacrowd/sea_datasets/<my_dataset>/<my_dataset>.py or python -m tests.test_seacrowd seacrowd/sea_datasets/<my_dataset>/<my_dataset>.py --subset_id {subset_name_without_source_or_seacrowd_suffix}.
[ ] If my dataset is local, I have provided an output of the unit-tests in the PR (please copy paste). This is OPTIONAL for public datasets, as we can test these without access to the data files.

Thank you @muhammadravi251001! The code looks good to me, and the unit tests run successfully on my end (appreciate the unit test code)

Thanks for the approval, Sir!

Everything looks good, manual test passed and code is clear and clean.
Thanks for the code ^^

Thanks for the approval, Lucky!

Merging it since we all approved it. Thanks @muhammadravi251001


Unable to replace with a dash involved

This command fails:
renamer --find '_Small' --replace '-sm'

because it thinks -sm is another command line flag that it doesn't recognize. I assume this is actually an issue with what library you're using for argument parsing, but I thought you would want to know, as there are plenty of other argument parsers to choose from. Here's the output:

As you can see, it confuses the program a bit.

wtf... thanks, looking into it..

do you have the latest version installed (v0.5.6)?

Yeah, literally just installed like an hour ago.
Unrelated: this is a neat tool.  Bash?

and which node version are you on?

Bash v4.3.42 and Node v4.2.6. Incidentally, the usage guide actually does always look like that, not just when something blows up.

hmm, on an almost identical environment i can't reproduce it.. i even fucked around with different fonts and iterm settings..

I'm using the font "Meslo LG S Regular for Powerline" (which is a powerline font, obviously), so . . . it could be me.

I get similar output using the standard command prompt on Windows 7, and node 0.10 (not my typical environment, but possibly worth mentioning). The font there is "Terminal".

Fixed and released in renamer v0.6.1.
The "Unable to replace with a dash involved" issue is resolved within the command-line-args module, which renamer uses.
See here for a description how to avoid ambiguous values. In summary, you use --option=value notation, for example:
renamer --find '_Small' --replace='-sm'


$const can fill the buffer real quick

New component sends messages in a infinite loop and that could create lots of useless events and also fill the buffer real quick. The bigger buffer is the more memory we will waist. With big buffers and big programs (lots of consts) we can run of out memory

Related to #677 and #671

Problem
Deferred connection doesn't mean that sending of a constant is deferred, it means that receiving is deferred
It means we can fill the Lock's buffer

Inport for New
Main (but not the only) reason for deferred connections is sending constants. They are sent by using New component that sends messages in an infinite loop.
There's a reason for that - let's say there's a and b inports. Let a be real dynamic data and b "static" (connected to const). How do we reactivate when new a arrive? Infinite loop solves that problem.
However, this lead to problem described in this issue. If we add buffer, then constants will fill that buffer real quick. Infinite loop works super fast. But what if we add inport to New? After all, that was the initial reason for -> (), isn't it?
Before
:start -> (42 -> println)

After
:start -> 42 -> println

No need for deferred connection in such a simple case.  Also that opens a door for multiple senders:
[:foo, :bar] -> 42 -> println

But it was always possible isn't it?
[:foo, :bar] -> (42 -> println)

Problem - more complexity in networks
At least one problem is that there are other cases where we don't need deferred connections now.
We do need them (speaking about $const) when constant is the only activation condition (like that one simple example). But otherwise we might not need it.
For instance this example is completely fine:
:a -> add[0]
42 -> add[1]

Even though 42 is always "there" add will wait for dynamic data from :a. We could rewrite this example by using deferred connections without changing it's semantic behaviour (but adding extra connections to desugared version):
:a -> [
  add[0],
  (42 -> add[1])
]

I would say it looks much worse. It's possible to imagine complex programs where this makes a huge difference. However, what we are talking about here is a little different thing. Let's add inport to New and see how it changes our program
:a -> [
  add[0],
  42 -> add[1]
]

I would say it look a little better than with deferred connections version but still much worse than original version

Do not use real connections and transmission for constants
From performance point of view that would be perfect.
We don't really have to send constants across the network. We could kinda "remember" them inside inport object. So every time runtime function calls Receive() function it just gets the same value. No channels - no buffers and no buffers means no problems! As a bonus we do not waste CPU cycles on transmission of constants between ports.
msg := inport.Receive() // msg == 42 always

...

func (inport) Receive() Message {
  return IntMessage(42)
}

Problem
Combination with normal values

Combination with normal values

I'm not sure it

makes sense at all
works well now with existing design

I mean we can live with it, it's a DX and perf issue rather. Crit->Major

#772 Kinda solves this, even though it's definitely still possible to reproduce this issue by using constants outside of chained connections (binary/ternary/switch/match/deferred/etc.)


Authentication module setup in ICA controller

Summary
I had some issues to setup the ICA controller. I understood that authentication module is optional but still the concrete setup was not clear to me.
Proposal
Some ideas that would have made it easier for me:

Add godoc for the constructor explaining app can be nil and what happens in this case
Provide a new constructor NewNoAuthIBCMiddleware(k keeper.Keeper) or better name that provides the correct instantiation
Provide a null-object like AcceptAllAuthIBCModule in the module that does noop (and you can get rid of the nil checks in the methods)
Replace porttypes.IBCModule interface with some new interface dedicated to authentication only. There is some overlap of methods but not all. IMHO a clear separation of concerns would improve the design and make it easier in the future. This would also give better IDE support to find all the implementations.

For Admin Use

[ ] Not duplicate issue
[ ] Appropriate labels applied
[ ] Appropriate contributors tagged/assigned

Hi @alpe. We discussed this issue and we plan to address your first 2 items:

we will add extra documentation.
we will change the existing constructor to have only the keeper as argument and add a second constructor NewIBCMiddlewareWithAuth(app, keeper).

But we were not sure what you meant with your last point:

Replace porttypes.IBCModule interface with some new interface dedicated to authentication only. There is some overlap of methods but not all. IMHO a clear separation of concerns would improve the design and make it easier in the future. This would also give better IDE support to find all the implementations.

Could you please elaborate what you mean?

Thanks for the clarifications, @alpe!

I noticed: The ica middleware constructor expects an object of type porttypes.IBCModule without any need. Only a subset of methods are called on the object within the middleware. The inter-tx demo implements only OnAcknowledgementPacket with some logic.

Yes, that's true that inter-tx only really implements OnAcknowledgementPacket, but there could other authentication modules that do custom logic in the other callbacks declared in porttypes.IBCModule.

The design as middleware feels also not very adequate. IMHO a module would be a better fit as most ICS4Wrapper methods are not used. Passing a nil value to a middleware adds some smell, too.

This is the nil that is pass to NewIBCMiddleware for the app argument when there is no authentication module, right? We will fix this by removing that parameter from the argument list of NewIBCMiddleware and add a separate constructor function with signature NewIBCMiddlewareWithAuth(app, keeper) that will not allow a nil app as input.

Revisiting this discussion. Sounds like the action items are:

[ ] Add godoc for the constructor explaining app can be nil (defaults to SDK authentication via msg server)
[ ] Change the existing constructor to have only the keeper as argument and add a second constructor NewIBCMiddlewareWithAuth(app, keeper)

The other concerns are very valid, but I think the general sentiment is that ics27 needs a whole v2 re-envisioning to simplify its construction and improve its dev UX. I think it makes to hold off on such changes until there's a deeper investigation into reworking ics27


Launch screen ios assets

Fixes #497
Unsure if the images are linked properly in xcode, but they are present in the correct folder


Kubeless output

Signed-off-by: Issif issif_github@gadz.org<!--  Thanks for sending a pull request!  Here are some tips for you:

If this is your first time, please read our contributor guidelines in the CONTRIBUTING.md file and learn how to compile Falco from source here.
Please label this pull request according to what type of issue you are addressing.
If the PR is unfinished while opening it specify a wip in the title before the actual title, for example, "wip: my awesome feature"
-->

What type of PR is this?
This PR adds add a native integration of kubeless as new output. Falcosidekick will be able to trigger a kubeless function from in/outside a k8s cluster.

Uncomment one (or more) /kind <> lines:

/kind bug

/kind cleanup

/kind design

/kind documentation

/kind failing-test

/kind feature

Any specific area of the project related to this PR?

Uncomment one (or more) /area <> lines:

/area build

/area config

/area outputs

/area tests

What this PR does / why we need it:
With a native integration, all intermediate (NATS) and custom components for setting up a response engine become useless.
Which issue(s) this PR fixes:
N/A

Fixes #
Special notes for your reviewer:
cc @cpanato @KeisukeYamashita

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign issif after the PR has been reviewed.
You can assign the PR to them by writing /assign @issif in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign issif after the PR has been reviewed.
You can assign the PR to them by writing /assign @issif in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: KeisukeYamashita
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [KeisukeYamashita]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

LGTM label has been added.  Git tree hash: 29d977edb881d4cdd07fc44d5121f693ec33672e

LGTM label has been added.  Git tree hash: 29d977edb881d4cdd07fc44d5121f693ec33672e

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: KeisukeYamashita
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [KeisukeYamashita]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign cpanato after the PR has been reviewed.
You can assign the PR to them by writing /assign @cpanato in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign cpanato after the PR has been reviewed.
You can assign the PR to them by writing /assign @cpanato in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign cpanato after the PR has been reviewed.
You can assign the PR to them by writing /assign @cpanato in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign cpanato after the PR has been reviewed.
You can assign the PR to them by writing /assign @cpanato in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@nibalizer @cpanato thanks you both, I used github.com/google/uuid for generating event_id is cleaner.

@nibalizer @cpanato thanks you both, I used github.com/google/uuid for generating event_id is cleaner.

LGTM label has been added.  Git tree hash: 6bc20485b43212b30de3e37da7d6e3f91f74241d

LGTM label has been added.  Git tree hash: 6bc20485b43212b30de3e37da7d6e3f91f74241d

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: cpanato
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [cpanato]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: cpanato
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [cpanato]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/hold cancel

/hold cancel


Rethink multiple request actions

I think multiple request actions shouldn't be a thing. They're easily abused to do authentication in the first and short-circuit and store auth information in the request locals. This requires further actions to either check for the authentication info to be present or rely on it, which are both not optimal IMO.
Instead, it should be possible to explicitly pass results of earlier handlers into the next handlers.

My current plan would be only having responders, no middlewares. Middlewares are basically just decorating other responders then. Multiple responders aren't needed anymore, e.g. Router would just fallback to a default responder like Root.
I'd combine this issue, #156, and #109.


Redirect the new user to welcome page

Issue
https://github.com/Real-Dev-Squad/website-www/issues/645
What is the change?
Changed the goto.js file which is responsible to redirect the new user to the signup form.
Example :

Done this
Made these changes

Is Development Tested?

[] Yes
[ ] No

https://github.com/Real-Dev-Squad/website-www/assets/55151241/b8991109-0eb4-4da7-b213-fb35f272584f

@Amandixit10
Current commit msg is 'changes'
Can we try it making it more meaningful?

Thanks man,

Do we need to add test stats here?

Thanks man,

Thanks for pointing this out, will remember this.

https://github.com/Real-Dev-Squad/website-www/assets/55151241/5b42f3df-50b3-4cf6-a175-ec11e66cc849


Badge tester code does not work in firmware 2.0

The badge keep restarting
temporary fix - use code from the old version

It looks like an event handler, storer_on_sys_evt, ends up calling the function flash_erase_success_callback. These is a series of calls (Storer_ScheduleBufferedDataStorage -> StorerSchedulerCallbackHandler_StoreBufferedData ) that eventually calls Storer_StoreBufferedData. I think that this function fails because we do not initialize the storer when running the tester code.
For now, I'm just going to comment the internal flash test and wait until we decouple the storer from flash management.


Everything is really slow!

When the website is loaded it's really slow , about 1-2 minutes i leave it ideal and then it works smoothly but if i refresh it or open a it a new tab it's slow again for 1-2 minutes.
I am having the following message
.lp?dframe=t&id=562784&pw=0xofpJZ3JS&ns=dekhoapp:5 [Deprecation] Synchronous XMLHttpRequest on the main thread is deprecated because of its detrimental effects to the end user's experience. For more help, check https://xhr.spec.whatwg.org/.
[Violation] 'unload' handler took 2800ms          :                            VM448 .lp?dframe=t&id=565958&pw=giAvl5wgFb&ns=test :9
which corresponds to
function EnvDisconnect() { EnvSendPing("/.lp?disconn=t&id=565958&pw=giAvl5wgFb"); }
and this
[Violation] 'message' handler took 786ms          : firebase-storage.js:3474
which corresponds to
catch(error) {             throw new Error(               'Cannot instantiate firebase-storage.js - ' +               'be sure to load firebase-app.js first.'             )

Cross-post: https://stackoverflow.com/questions/45520390/retrieving-data-from-firebase-is-really-slow (please indicate when you cross-post)
Since it seems unlikely this is related to the friendlypix sample, I'm going to close the issue here in favor of the one on Stack Overflow.


Laminas Cli Command Candidate?

https://github.com/PacktPublishing/Laminas-Web-Development/blob/fa5b6dcfce329517d01774b0bcc8f75d42474539/chapter10/whatstore/bin/load_database.php#L9
Curious as to why this was not done with Laminas-Cli.
Benefits:
Could have been interactive.
Would have provided an example of how to provide command line tooling via laminas cli.
Since Laminas Cli commands can have factories it would have simplified the workflow since the factory would have access to the service container, and config service.

Yes, you are right. It could be an opportunity to talk about laminas-cli. But I don't know if the database population script is appropriated to explore the interactive aspect of laminas-cli. In this case, every function will be executed. There are no alternatives. Then I think that is a limited example.
I already have talked about the main dependency for laminas-cli, but I wrote a full chapter about it, with several examples. For this case, it doesn't seem essential to me, considering the limited context.
But I will reflect about it. I will consider other changes. Thank you very much! :)

True enough. I recently created an installer for the application that I was working on. The command excepted the db connection details, I used that in conjunction with Laminas\Config to write the database connection info for the application while the db was being populated so to remove the need for a web installer which would then need to be removed or cause a security issue. Since any build for the application on any given server will require shell access to run composer it only seemed a logical way to handle it, for me at least.


⚠ Bofrost.de has degraded performance

In 2cc93aa, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1622 ms

Resolved: Bofrost.de performance has improved in cead0f4 after 6 minutes.


Generate map roulette assets

A map roulette challenge requires a GeoJSON file of locations and attributes.
From the map roulette help

A common option is to use a GeoJSON file you already have. This could be great if you have an approved source of external data you would like to manually add to OSM. Tools like QGIS and gdal can convert things like Shapefiles to GeoJSON. When you convert, make sure that you use unprojected lon/lat on the WGS84 datum (EPSG:4326), because this is what MapRoulette uses internally.
Note: for challenges with a large number of tasks, we recommend using a line-by-line format instead, which is much more efficient to process. Some advanced features are only available when the line-by-line format is used.

[x] Generate a GeoJSON file with the locations we want reviewed in MapRoulette

I created a GeoJSON file and uploaded it to the MapRoulette challenge that I started.  It ingested correctly and created tasks for all 725 locations.

Can you paste the geojson in a gist and link it here please!

Here's the geojson:
https://gist.github.com/eneemann/c084806f3fa5dbde8c74f381db9cf698


[ORA-1468] implement round robin, random rpc node target of queries for stress testing

Right now the stress test only ever submits transactions via validator0s RPC. This code changes it so that you can choose to submit queries round robin between the validator RPC URLs, or randomly choose the next RPC target based on a random seed in the bash environment

getting
``
`rpc error: code = NotFound desc = rpc error: code = NotFound desc = account allo1fsvsahhjhfekey9q5l0xcrdhpxg8qcvvztcq2q not found: key not found

on the `stress_test` - further examination needed

with the changes in #300 i believe this is now good

superseded by #300 at this point, would rather just have the focus be on that PR


Starlink 7 missing data

Core: B1049.4
Landing: OCISLY
https://reddit.com/r/spacex/comments/gamcbr/starlink7_launch_campaign_thread/

Updated

Nitpicks, but seems that the reuse count was set to 3 instead of 4, at least on my end

Reuse count in /cores? Or in the launch itself?


enhancement(mis): 当多集群操作失败, 通知用户联系管理员

test

Codecov Report
Base: 65.31% // Head: 65.38% // Increases project coverage by +0.07% :tada:

Coverage data is based on head (5492ce8) compared to base (b94270f).
Patch coverage: 0.00% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #280      +/-   ##
==========================================
+ Coverage   65.31%   65.38%   +0.07%
==========================================
  Files          98       98
  Lines        2707     2713       +6
  Branches      335      334       -1
==========================================
+ Hits         1768     1774       +6
  Misses        897      897
  Partials       42       42

Impacted Files
Coverage Δ

apps/mis-server/src/plugins/clusters.ts
44.44% <ø> (ø)

apps/mis-server/src/services/account.ts
38.14% <0.00%> (-0.40%)
:arrow_down:

libs/auth/src/changePassword.ts
100.00% <0.00%> (ø)

apps/mis-server/tests/data/fetch.ts

libs/auth/src/utils.ts
100.00% <0.00%> (ø)

apps/mis-server/src/services/user.ts
32.85% <0.00%> (+0.15%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

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


Map freezing with iOS 11 on ipad mini

I was testing an app on iOS11 beta and noticed some performance issues with rotation. Then I tried some websites that are using opnelayers and most of the time the maps are freezing as soon as there is an interaction (zoom in, rotate).
Here is an example: https://wow.metoffice.gov.uk/
So I'm just wondering if there is an issue with the new version of iOS.
Thanks,
Alessandro

Performance issues with rotated maps were an issue in old versions of OpenLayers, because of canvas sizes that were too big. I'd recommend you try and see if things improve with the latest version.

Just tried with the official examples and is doing the same (map freezing)
this for example https://openlayers.org/en/latest/examples/drag-rotate-and-zoom.html

Thanks for testing. Maybe there is an issue with iOS11 Safari indeed. Can you ticket the problem there?

I will try to update another device to iOS 11 beta first just to make sure is not a device related issue

After updating to the latest version of iOS (11 beta 8) the main problem with map freezing is now solved.
There are still some issue with the map rotation and performance after rotating but it looks very specific to clusters. Example: https://openlayers.org/en/latest/examples/cluster.html

Humm, just checked with an ipad air 2 with IOS 11 final, and the freeze is there (visible on official samples on then openlayers website). You can pan and pinch-zoom for something like one second and after that you can't anymore. There is no crash it seems. Sometimes it works again after a few seconds, but freezz again.

I'm seeing this on iOS 11 GM with an iPhone 6S. Not always, but often, the map will stop responding to drags or pinch to zoom. Double tap to zoom and controls still work. Happens in Safari, Chrome and a webview.

Same here on IOS 11 final. Some devices do it for a few sessions (ipad air 2), and then magically work forever and some others (like iphone 6s) will never work. Happen on samples on openlayers web site and in our app.
It could very well be an ios bug, I will try to investigate this further.

I investigated and could understand the problem.
This codepen show where the problem is (and gives a fix) : https://codepen.io/anon/pen/qPNwBy?editors=0011
In IOS11, in ol.interaction.Pointer.prototype.updateTrackedPointers_, the code "if (event.pointerId in this.trackedPointers_)" does not always work when pointerId is negative. In my understanding it is valid to access properties of an object with negative integer property names, but safari does not like that (I think it is a bug in safari). Simply transforming pointerId in a string in this function will make the code work again.
In the code pen alternating the two lines "works" and "don't work" will repro or fix the problem ...

I've been seeing this exact thing as well and can confirm that this fix, does indeed seem to fix it.

Thanks for testing @RobertSmart
After further research this is clearly an IOS bug. Sometimes, Safari loose the content of updateTrackedPointers (!). For the fix it is sufficient to make pointerId positive or change it to a string to make it work again.
I've been unable to repro the problem in a simple js test to submit this to apple ... Not sure I will be able to attract their attention with the codepen because it does involve a large codebase (and they will ask for a simple repro I can't provide).
@ahocevar could it be possible to include a patch with a fix in OL ? The fix is very simple (make pointerId positive or convert it to string) and will not have any impact on performance ?

By the way, this was tested on an Ipad Air 2 running latest iOS 11

Thanks for the help on this.  I've created @7285 with the fix suggested by @ogo-adp.  Please comment there if you are able to confirm whether this addresses the issue.

We have dozens of users experiencing the problem. Checking 7285...

Please reopen if #7285 doesn't fix the issue.

@tschaub Thanks for including the fix :) Would it be possible to make an official 4.3.4 build including the fix ?
Just as a sidenote for this ios bug. It does only happen when pointerId is negative. Some devices don't have the bug just because for them the pointerId is positive not negative. If you restart your device, pointerId will be positive, and after some time it will possibly become negative then triggerring the bug. IOS 10.3 does not have the bug with negative pointerId (I didn't check if ios 10.3 generates negative pointerId however) ...

The<EMAIL_ADDRESS>release is out with the fix.

Thanks @tschaub , what a relief for our ios11 users

Hi guys,
thanks for releasing the fix so quickly. Unfortunately this will not fix the original issue I reported.
If you open this Cluster example in an iPad with iOS 11 you can see a performance issue. Try to:
1 - Rotate the map
2 - Move the map to the right (or to the left)
The map will move very slowly making it almost unusable. If you try with a map without clusters everything is working well.
Any thought on this?
Thank you

@alallo - it looks like there were multiple issues.  Could you please open a new ticket that is specific to the cluster example on iOS 11?  Perhaps this was your original issue, but it got conflated.

I can further confirm the fix in @ogo-adp codepen worked. Thanks much!

OL 4.6.5, IOS 11.3.1 Ipad Mini. Touch interactions almost not working:

any two finger guesture causes zoom in, zoom out not working
rotation not working
click/touch event fires even if I do 2-finfer gesture.

also panning not working


Appstudio update test-component-pac-jtnk

Pipelines as Code configuration proposal

Pipelines as Code CI/test-component-pac-jtnk-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
7 seconds
init

✅ Succeeded
16 seconds
clone-repository

✅ Succeeded
2 minutes
build-container

✅ Succeeded
35 seconds
inspect-image

✅ Succeeded
36 seconds
deprecated-base-image-check

✅ Succeeded
3 minutes
clair-scan

✅ Succeeded
1 minute
clamav-scan

✅ Succeeded
37 seconds
sbom-json-check

✅ Succeeded
13 seconds
label-check

✅ Succeeded
13 seconds
optional-label-check

✅ Succeeded
5 seconds
show-sbom

✅ Succeeded
11 seconds
show-summary

Pipelines as Code CI/test-component-pac-jtnk-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
6 seconds
init

✅ Succeeded
16 seconds
clone-repository

✅ Succeeded
25 seconds
build-container

✅ Succeeded
13 seconds
deprecated-base-image-check

✅ Succeeded
12 seconds
clair-scan

✅ Succeeded
13 seconds
inspect-image

✅ Succeeded
48 seconds
clamav-scan

✅ Succeeded
12 seconds
sbom-json-check

✅ Succeeded
7 seconds
optional-label-check

✅ Succeeded
7 seconds
label-check

✅ Succeeded
5 seconds
show-sbom

✅ Succeeded
6 seconds
show-summary


Update ValidateCertificateChain() To Allow Certificate Chain Validation without Intermediate Certificate

Issue Being Resolved
Fixes #22428
Change overview
Using this function to validate CD signing certificates added to the CD Trust Store.
Added new unit-test cases.

Accepted for v1: necessary for certificate chain validation


support spark struct directly to thrift type

8d05c98 SparkEnv.get called in an non doas wrapping code, which will cause NPE


Agent

In this PR I switched over from using functions as tools to subclassing the base tool

@Jgmedina95 can you review this? thanks!


Add release config for 1.30

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/lgtm

/approve


Close scene menu option asks for confirmation only if there are unsaved changes

Closes #305

Nice! :)


[MRG] Remove acq_optimizer='auto' and set default to 'sampling'

Fixes https://github.com/scikit-optimize/scikit-optimize/issues/274
See benchmarks:

https://github.com/scikit-optimize/scikit-optimize/pull/262#issuecomment-269493170
https://github.com/scikit-optimize/scikit-optimize/pull/262#issuecomment-260835534

What shall we do with this now?

There is definitely some issues with how we handle Integer dimensions. I'll get back to this as soon as I have some solid reasoning behind that.

(In the context of switching the default from "auto" to "lbfgs" according to the original issue)

Running it on https://scikit-optimize.github.io/notebooks/hyperparameter-optimization.html there does not seem to be a huge difference. I think we can remove it.
Without K-best:

Acquisition Function
Mean Minimum
Std

lbfgs
2.884
0.037

sampling
2.844
0.024

With K-best

Acquisition Function
Mean Minimum
Std

lbfgs
2.925
0.032

sampling
2.917
0.035

What do you call "K-best"?

I had meant with and without the preprocessing using SelectKBest

Another observation, I have to increase n_restarts_optimizer further to give a non-constant predictions for estimating piecewise constant functions.
import numpy as np
import matplotlib.pyplot as plt

from skopt.learning import GaussianProcessRegressor
from skopt.learning.gaussian_process.kernels import ConstantKernel
from skopt.learning.gaussian_process.kernels import Matern

def f(x):
    return int(x)**2 - 1

x = np.linspace(0, 20, 61)[:60]
y = [f(xi) for xi in x]

for startptr, endptr in zip(range(0, 61, 3), range(3, 62, 3)):
    plt.plot(x[startptr: endptr], y[startptr: endptr], "black")

cov_amplitude = ConstantKernel(1.0, (0.01, 1000.0))
matern = Matern(length_scale=1.0, length_scale_bounds=[0.01, 100], nu=2.5)

gp = GaussianProcessRegressor(
    kernel=cov_amplitude * matern, normalize_y=True, random_state=0,
    alpha=0.0, n_restarts_optimizer=5)
gp.fit(np.reshape(x, (-1, 1)), y)

x_unseen = np.linspace(0, 20, 201)[:200]
y_pred = gp.predict(np.reshape(x_unseen, (-1, 1)))

for startptr, endptr in zip(range(0, 201, 20), range(20, 221, 20)):
    plt.plot(x_unseen[startptr: endptr], y_pred[startptr: endptr], "red")
plt.show()

With n_restarts_optimizer=5, the graph looks like

With n_restarts_optimizer=2, the graph looks like

Interesting! This is also what we observe in the BO notebook, everything is flat except some spikes, just like your second plot. Maybe there is something wrong in the training procedure of the GP that we should mention upstream in scikit-learn? In the meantime, shall we increase n_restarts_optimizer further?

Maybe there is something wrong in the training procedure of the GP that we should mention upstream in scikit-learn?

When I had a look sometime before, the implementation seemed to be correct. I just think it gets stuck in a local minima. I can have a closer look though.

Btw, I closed the pull request because I messed up some git stuff


Binary file not being utilized

Not sure if this is an issue (could be me), but I'm trying to create a Migration strategy for 'Phinx'. However, it seems the Strategy file is not calling the code inside my Binary file.
I have two files:
PhinxStrategy.php - a strategy file
Phinx.php - a binary file
Inside PhinxStrategy.php I'm calling the below code:
public function migrate(){
    var_dump($this->binary('phinx')->runForCurrentRelease('migrate'));
    die();
    //return $this->binary('phinx')->runForCurrentRelease('migrate');
}

Then inside Phinx.php:
public function migrate(){
    return "test";
    // return $this->getCommand('migrate', null, ["--environment" => "production"]);
}

From my understanding, I should get back a string that prints "test". However, I'm getting "phinx migrate" It's like the code is never utilizing the binary Phinx.php at all. Any suggestions on what I'm doing wrong?

Okay pretty simple. The file wasn't being autoloaded. I've noticed that for strategies, events & tasks you autoload from corresponding directories. Can you provide similar functionality for the binary files?

I think I'm just gonna change that to autoload any subdirectories found in .rocketeer so people can structure stuff however they want.


main.css

Hi Zach,
Just working my way through your React tutorials. These are very well explained, thank you.
However, when coming over to Github to grab the main.css for the message-example, it seems that you committed the hello-world app to this repo. Any chance you could put up the files?
Kind regards,
Graham.

Graham, thank you so much for the notice! I didn't push up the commit I had laying around, but it's up here now. Sorry for not having it here earlier!

Not a problem Zach, thank you for creating a great set of tutorials.
Very much looking forward to the next instalment.

Sorry to message again, hope you can help. The app doesn't seem to load for me (in Chrome).
It shows the React DOM tree (through React dev tools) and comes up with these console errors:
Warning: React.render is deprecated. Please use ReactDOM.render from require('react-dom') instead.
Uncaught ReferenceError: React is not defined
Changing to ReactDOM does stop that error, but I'm not sure how to make it so it knows React is defined. Couldn't find a solution online unfortunately.


[🐞] [vite-plugin-qwik] Found '$' but did not find the corresponding 'Qrl' exported in the same file.

Which component is affected?
Qwik Rollup / Vite plugin
Describe the bug
Background
I'm trying to create a monorepo using Qwik City where all the codes reside in the src/ folder. There is a src/packages folder where it hosts all the packages to be published. Anything outside of the src/packages are basically files and folders needed for a website which will render demos of each and every package from the src/packages.
The basic idea is to create any package you want in the src/packages and then import them into the demo.
\
|- src
|   |- packages
|   |   `- button
|   |- components // button could be imported here like a installed package, e.g. import Button from '@ui/button';
|   |- routes
...

// Basic code implementation of Button.tsx
export const Button = component$((props) => {
  return <button {...props}><Slot /></button>
});

// Basic code showing how to import Button

import { Button } from '@ui/button';

export default component$(() => {
  return (
    <Button>Click me</Button>
  );
});

Problem
Everything works great when running pnpm dev locally. However, when running pnpm preview, the build will fail with the following error:
[vite-plugin-qwik] Found '$' but did not find the corresponding 'Qrl' exported in the same file. Please check that it is exported and spelled correctly

Reproduction
https://stackblitz.com/edit/qwik-starter-tcqf9b?file=src%2Fpackages%2Fqwik-button%2Fpackage.json
Steps to reproduce

Go to the reproduction URL
Terminate any running command in an active terminal then run pnpm install && pnpm preview
You should see [vite-plugin-qwik] Found '$' but did not find the corresponding 'Qrl' exported in the same file. Please check that it is exported and spelled correctly.

System Info
<EMAIL_ADDRESS>
  System:
    OS: macOS 13.6
    CPU: (10) arm64 Apple M1 Max
    Memory: 25.99 GB / 64.00 GB
    Shell: 5.9 - /bin/zsh
  Binaries:
    Node: 20.7.0 - ~/Library/Caches/fnm_multishells/1812_1695370805799/bin/node
    npm: 10.1.0 - ~/Library/Caches/fnm_multishells/1812_1695370805799/bin/npm
    pnpm: 8.7.6 - /opt/homebrew/bin/pnpm
  Browsers:
    Chrome: 116.0.5845.187
    Safari: 16.6
    Safari Technology Preview: 17.0
  npmPackages:
    @builder.io/qwik: ^1.2.12 => 1.2.12
    @builder.io/qwik-city: ^1.2.12 => 1.2.12
    undici: 5.25.1 => 5.25.1
    vite: 4.4.9 => 4.4.9

Additional Information
No response

In your reproduction, you have a packages folder inside of the src folder. This is not allowed. Please create a reproduction when there are no packages, only source code in the src folder, and then share it here.
If things work fine, then please close the issue.

@mhevery Thanks for the recommendation. 👍  I relocated the packages from the src directory, but it didn't function right away. I had to include "qwik": "./src/qwik-button.tsx" to get it functioning correctly.
However, I discovered that Qwik does not handle dependencies of a component (be it in a monorepo or in library mode) that well. 🤷  It forces us devs or the library users to always install all dependencies to the root workspace. I will raise a separate issue for that.
Closing this as it has been resolved with this comment.


[WIP][test-hadoop3.2][test-java11] Spark arrow jdk11 r

What changes were proposed in this pull request?

Why are the changes needed?

Does this PR introduce any user-facing change?

How was this patch tested?

Test build #113908 has started for PR 26554 at commit 5bb0c8b.

Merged build finished. Test PASSed.

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

Test build #113908 has finished for PR 26554 at commit 5bb0c8b.

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

Merged build finished. Test PASSed.

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

This patch passed the test and merged into #26552

Hmm. AppVoyer seems to fail due to OOM.


Compatibility with ROS2 Humble and TensorRT version

It is mentioned that 'ros2_tao_pointpillars' package has been tested on Ubuntu 20.04 and ROS2 Foxy. Could you kindly inform me if the package is compatible with ROS2 Humble on Ubuntu 22.04? If so, what is the TensorRT version that we have to install?

having the same issue here. Try to use Ubuntu 22.04, ROS2 Humble, TensorRT8.6. Not sure if this is possible.

having the same issue here. Try to use Ubuntu 22.04, ROS2 Humble, TensorRT8.6. Not sure if this is possible.

Thank you for the response. Could you please let me know whether using TensorRT 8.6 solved the problem for you? This is another response that I received.
https://forums.developer.nvidia.com/t/compatibility-of-ros2-tao-pointpillars-with-ros2-humble-and-tensorrt-version/275369

Thank you for the response. Could you please let me know whether using TensorRT 8.6 solved the problem for you? This is another response that I received.
https://forums.developer.nvidia.com/t/compatibility-of-ros2-tao-pointpillars-with-ros2-humble-and-tensorrt-version/275369

I haven't finished building entire pipeline. I got stuck on different version of TAO converter.

Thank you for the response. Could you please let me know whether using TensorRT 8.6 solved the problem for you? This is another response that I received.
https://forums.developer.nvidia.com/t/compatibility-of-ros2-tao-pointpillars-with-ros2-humble-and-tensorrt-version/275369

I haven't finished building entire pipeline. I got stuck on different version of TAO converter.

Hi,
did you manage to figure this out? I believe I got stuck on a similar issue - all attempts to use tao converter just through up a seg fault