Consolidating vps and bms projects

Added vps and bms go project structure
under /src/github.com/cloudfoundry-community
#133573407

Hi dr.max @maximilien ,
I plan to consolidate the projects as below, please help to take a review if this works for you. Thank you.

Move vps and bms source code to {project}/src/github.com/cloudfoundry-community in release repo. All of the server&release code will be tracked here. (no submodules or new repositories.)

Create bin/jobs/packages/etc. for vps using current release structure. (in progress)

Temporarily keep all of the ruby bms server and release code until the new go bms server is ready for use.

/cc @mattcui @jianqiu

OK @zhanggbj agree. Merging so we can move forward on this ASAP.


Remove CCT testsuite

We stoped using CCT since Cloud 8 and for older releases it creates
random failures. As nobody maintaines or fixes the tool let's remove it
as the value is limited.

@aspiers This was decided over a year ago by the QE team during the Cloud 8 interlock preparations. The tools for testing are documented in the testing plan and CCT was specifically removed. I don't think this is the right time or place to restart the discussion as for me CCT is dead since Cloud 8. As this creates now random Cloud 7 tests I just do the necessary duty to remove it. Please remember the decision to stop using it was not taken by me.

Exactly, thanks @jdsn.  Transparency is key for healthy team collaboration.

I wrote a mail to the devel mailing list to clarify the situation.

@rsalevsky commented on 5 Oct 2018, 12:50 BST:

I wrote a mail to the devel mailing list to clarify the situation.

Thanks a lot for doing that!  I replied there - would be good to get a bit more clarity around the team's position and strategy regarding BDD before we progress on this PR.

@jdsn I'm not sure but I think #2838 already achieved the effect of 1.  @rsalevsky can you confirm?  What about for C8?  Although yes, removing it from the default step expansions would be a more intuitive approach.
Regarding 2. I think this would be a mistake given that people have already stated on the mailing list that they are still using it.

@aspiers yes, https://github.com/SUSE-Cloud/automation/pull/2838 solves the 1st issue, but its kind of hidden, because cct is still there and the step will be executed but then do nothing because the ISO is not defined.
So yes, I also would have preferred the removal from the step lists. But its solved for now.
Regarding 2. I agree, that would have been the subject of the discussion in the new PR. But we could have decided on both topics individually without blocking the CI.
@rsalevsky
I think we can close this PR for now as the issue is solved differently.
Feel free to rebase, change, split, ... after the discussion on the mailinglist.


Make the universe cache update timeout configurable

Right now Cosmos only updates the local package cache at most once a minute. This should be configurable.

No activity in a while. Closing. We are going with a different solution.


'viewContainerRef' does not exist

Hi, I am trying to use the modal component included in ng2-bootstrap , I followed the steps of the documentation to make it work and in fact works, but i am getting error TS2339: Property 'viewContainerRef' does not exist on making impossible for me to build the app.
I am using Angular 2 RC3 (last week RC2 and the problem was the same) and Webpack.
The root component has the following code
import { Component, ViewContainerRef } from '@angular/core';
import { ROUTER_DIRECTIVES } from '@angular/router';
import { HomeComponent } from './home';
import { DashboardComponent } from './dashboard';
import { UserLoginComponent } from './user-login';
/*
 * App Component
 * Top Level Component
 */
 @Component({
   selector: 'spr-app',
   directives: [ ROUTER_DIRECTIVES, UserLoginComponent],
   template: require('./app.component.html'),
 })
 export class AppComponent {
     public constructor(viewContainerRef:ViewContainerRef) {
    // You need this small hack in order to catch application root view container ref
    this.viewContainerRef = viewContainerRef;
  }
 }

and the component who uses the modal
import { Component, OnInit } from '@angular/core';
import {CORE_DIRECTIVES} from '@angular/common';
import {MODAL_DIRECTVES, BS_VIEW_PROVIDERS} from 'ng2-bootstrap/ng2-bootstrap';
import { User } from '../authentication/user';
import { AuthenticationService } from '../authentication/authentication.service';

@Component({
    selector: 'spr-login',
    template: require('./user-login.component.html'),
    directives: [MODAL_DIRECTVES, CORE_DIRECTIVES],
    viewProviders: [BS_VIEW_PROVIDERS],
    providers: [AuthenticationService]
})
export class UserLoginComponent implements OnInit {
    error: string;
    token: string = '';
    user: User = {
        username: '',
        password: ''
    };

    constructor(private authenticationService: AuthenticationService) { };

    ngOnInit() {}

    login() {
        this.authenticationService.login(this.user)
            .then(token => this.token = token)
            .catch(error => this.error = error);
    }
}

Thanks for your support , you are doing an amazing job

You can define that property on your AppComponent class, it should still work. Something like: private viewContainerRef: ViewContainerRef;

@adrianfaciu Works like a charm. Thanks!


Implement Spark Frontend

Spark Frontend that converts spark programs in to IR.
Must support multiple languages (Java, Scala..)
Add sample Spark applications

https://github.com/snuspl/coral/blob/d25517237e468628259f78a51643545a8d93a5af/src/main/java/edu/snu/vortex/examples/spark/LocalALS.scala
https://github.com/snuspl/coral/blob/d25517237e468628259f78a51643545a8d93a5af/src/main/java/edu/snu/vortex/examples/spark/SparkALS.scala
https://github.com/snuspl/coral/blob/d25517237e468628259f78a51643545a8d93a5af/src/main/java/edu/snu/vortex/examples/spark/SparkPi.scala
These files can be used for #412

https://issues.apache.org/jira/projects/NEMO/issues/NEMO-12?filter=allopenissues


[WIP] Add missing docs for some of our components

SDK-751

Coverage remained the same at 98.523% when pulling 54333ef02e62cffb1387ba5fa0ab371db1279c23 on bartvde:missing-jsdoc into 447e5599a7058f59bfe3f9f0703ade39b0e572a2 on boundlessgeo:master.

Coverage remained the same at 98.523% when pulling 411d64fe9ea11285d3ef189dc29ad6551a87779d on bartvde:missing-jsdoc into 447e5599a7058f59bfe3f9f0703ade39b0e572a2 on boundlessgeo:master.

Coverage remained the same at 98.523% when pulling a01cdbfc938e0282fc0d2fd1393d6138f08e21ec on bartvde:missing-jsdoc into 447e5599a7058f59bfe3f9f0703ade39b0e572a2 on boundlessgeo:master.

Coverage remained the same at 98.523% when pulling b32fb20bddb4262a19ae778f31dfa0ba680a57e3 on bartvde:missing-jsdoc into 447e5599a7058f59bfe3f9f0703ade39b0e572a2 on boundlessgeo:master.

Coverage remained the same at 98.523% when pulling b32fb20bddb4262a19ae778f31dfa0ba680a57e3 on bartvde:missing-jsdoc into 447e5599a7058f59bfe3f9f0703ade39b0e572a2 on boundlessgeo:master.


TappedRoutedEventArgs.GetPosition is inaccurate compared to PointerRoutedEventArgs.GetCurrentPoint.Position

Describe the bug
TappedRoutedEventArgs.GetPosition can be 1 or 2 physical display pixels different from PointerRoutedEventArgs.GetCurrentPoint.Position (e.g. if handling PointerPressed PointerReleased vs Tapped). This is awkward when providing pixel-perfect drawing or similar, especially as you may use PointerRoutedEventArgs.GetCurrentPoint.Position during PointerMoved to show a preview of the coordinates and TappedRoutedEventArgs.GetPosition to return the coordinates when the Tapped event fires. This necessitates a clumsy workaround whereby caching the value of PointerRoutedEventArgs.GetCurrentPoint.Position from either the PoitnerPressed or PointerReleased event is needed which can then be used in the Tapped event. (I assume that PointerRoutedEventArgs.GetCurrentPoint.Position is more accurate).
In case it is relevant, I'm using 125% display scaling.
Related discussion
There are various other shortcomings of the tapped event, some of which are as follows. (I can open a separate issue for these if needed).

There is no way to tell which button was pressed, so one has to rely on the same workaround describe above, caching values from PointerReleased.
There is no satisfactory way to handle taps in quick succession. The double-tapped event will fire instead. IsDoubleTappedEnabled only prevents an element being the original source of a double tap, so doesn't fix the problem unless applied to all primitives. Handling the DoubleTapped event instead/as well is not equivalent since it fires on the second mouse down not mouse up. The lack of a way to handle taps in quick succession is a fundamental problem.

I would suggest a revamp of the Tapped event, or an alternative event if backwards compatibility is a problem, which addresses all these issues.
Version Info
Not using WinUI.

Windows 10 version
Saw the problem?

November 2019 Update (18363)
Yes

Device form factor
Saw the problem?

Desktop
Yes

This needs checking in WinUI3.

@benstevens48 , thank you for reporting this issue. Unfortunately we are not currently investigating WinUI2 issues that are not business or security critical (more on the policy here). If the issue reproes in WinUI3, please comment here and we'll investigate.


Impossible to compile

I've downloaded the source code, but I can't compile it. I think that I'm making some mistake but I don't know what it is.
for instance this error : \biggy-master\src\Biggy.Core\bin\Debug\Biggy.dll
Any help?
Thanks a lot!!!

You may need to enable Nuget package restore.
That error doesn't tell us enough-that just means the Biggy sol couldn't compile because of some other error further upstream, most likely due to missing Nuget packages.
Also, if you are not using the azure bits, you might want to exclude that project, and comment out the Azure test files.

@pascualito2000  Did you get this to build on your machine?

I can recreate this one, there are some doggy references in the project files - I have just made a pull request (i hope - first one ever) that should fix this

Hi xivSolutions and thank you for your answer
Yes I've tried to compile it in my machine
I've configured to restore Nuget packages.
I'm still having the messages errors
for example:
cannot load the metadata file.... biggy-master\src\Biggy.Core\bin\Debug\Biggy.dll
I know that I'm making some mistakes, but I can't find out what it is....
I attach an image with references problems. I can't load these packages

Thanks a lot !!!
Javier.

Hi All -
Thanks for your feedback and patience on this. I am unfortunately on the road at the moment, but I will try to see what's up with this asap.
@Lucas-S  - Thanks for the PR - I don't want to just merge it without seeing what the source of the issue was. Appreciate your patience and work on the issue, and will merge as soon as possible. :-)
@pascualito2000 - The two Mono dependencies don;t matter unless you are on Mono, but the other two look like an issue with Nuget.
Can you try pulling down @Lucas-S changes, and see if this fixes the problem?
Sorry to do this this way, but...I'm at a funeral far from home atm.

@pascualito2000 Try now. @Lucas-S PR should fix this. Not sure where those odd paths came from, possibly my Mono implementation?


pkg.NewFuncDecl

Codecov Report

Merging #79 (b607a73) into main (384bdc4) will increase coverage by 0.06%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main      #79      +/-   ##
==========================================
+ Coverage   93.55%   93.61%   +0.06%
==========================================
  Files          13       13
  Lines        5396     5403       +7
==========================================
+ Hits         5048     5058      +10
+ Misses        267      265       -2
+ Partials       81       80       -1

Impacted Files
Coverage Δ

codebuild.go
93.97% <100.00%> (ø)

func.go
97.15% <100.00%> (+1.56%)
:arrow_up:

Continue to review full report at Codecov.

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


add more detail about aux_data in pytree docs

fixes #8726

Ping on this - it would be a nice addition to the docs


Fields which have multiple values

Hi all,
this mapping working fine for me, but i have list and there are some columns in this list contain multiple values and here, i can't retrieving data from the these columns .
any advice please @fixer-m

Hi, thanks for your feedback!
Unfortunately this project kinda frozen, because I don't have access to Sharepoint Server anymore, so I can't support this project right now, sorry. Yes, multiple values support not implemented.


Sensor reading values not imported intoHA

Hi all,
After I got the HAOS-SBFspot connected to MQTT (ticket #92 ), I am now faced with the next problem:
all values are read from the inverter, but the publishing to the queue seems not to work. All MQTT sensor values are "Unknown".
The SBFspot does not show an error (except for the MySQL which I indeed did configure with dummy data).
In the debug info from MQTT, the HAOS-SBFspot entity, I do see the HAOS-SBFspot topics, but "most recently received messages" is "0" for all.
Relevant part of the log from HAOS-SBFspot:
[2024-01-24 14:20:38.124] Error: Can't open MySQL db [SBFspot] : Unknown server host 'core-mariadb' (-5)
MQTT: Publishing (homeassistant/sbfspot_{plantname}/sbfspot_{serial}) "PrgVersion": "3.9.7","Plantname": "sma","Timestamp": "2024-01-24T14:20:38+0100","SunRise": "2024-01-24T08:30:00+0100","SunSet": "2024-01-24T17:16:00+0100","InvSerial": XXXX,"InvName": "SN: XXXX","InvTime": "2024-01-24T14:21:43+0100","InvStatus": "Ok","InvSwVer": "02.08.01.R","InvClass": "Solar Inverters","InvType": "SB 3000TL-20","InvTemperature": 36.020,"InvGridRelay": "Closed","EToday": 2.278,"ETotal": 40861.886,"GridFreq": 49.990,"PACTot": 598.000,"PAC1": 602.000,"UAC1": 228.020,"IAC1": 2.640,"OperTm": 53938.480,"FeedTm": 51766.522,"PDCTot": 630.000,"UDC1": 368.830,"UDC2": 0.000,"IDC1": 1.707,"IDC2": 0.000,"PDC1": 630.000,"PDC2": 0.000,"BTSignal": 73.333,"InvWakeupTm": "2024-01-24T08:48:49+0100","InvSleepTm": "2024-01-24T14:21:48+0100"
Client null sending CONNECT
Client null received CONNACK (0)
Client null sending PUBLISH (d0, q0, r0, m1, 'homeassistant/sbfspot_sma/sbfspot_XXXX', ... (763 bytes))
Client null sending DISCONNECT
********************
* ArchiveDayData() *
********************
startTime: 24/01/2024 00:00:00
Wed Jan 24 14:20:39 2024: INFO: Done

Thanks,
Stefaan

It sounds like your MQTT topics might not be matching.
homeassistant/sbfspot_sma/sbfspot_XXXX

That topic that the addon is posting to the MQTT server needs to be the same when Home Assistant is listening to the MQTT server.
It is probably just a case of deleting the sensors from Home Assistant and re-adding with the addon. They may have picked a different topic the first time they were created when the MQTT connection wasn't working in issue #92
You can use the listen to a topic in the MQTT integration to confirm it is being received in Home Assistant, then you will need to configure your sensors again.
Settings > Devices & Services > MQTT > configure > listen to a topic

You could also use program like "MQTT explorer" to connect and troubleshoot, which is available as a windows download or a Home Assistant addon.
MQTT explorer addon HA community forum

Hi all,
When removing all the sensors, HAOS-SBFspot adds them again on starting up. What I noticed is that these are added without the serial number:

Since these are created prior to connecting, I thought that this was normal.
I changed the MQTT topic configured in HAOS-SBFspot to include the serial number "hardcoded": Chagne from homeassistant/sbfspot_{plantname}/sbfspot_{serial} to homeassistant/sbfspot_{plantname}/sbfspot_2100XXXXXX
When deleting the sensors again, HAOS-SBFspot still generated config messages without the serial number.
So, next try was to remove the serial number in HAOS-SBFspot from the topic name:
homeassistant/sbfspot_{plantname}/sbfspot_
And now I do have results in the sensors.
So, no clue what the actual issue is right now. It seems logic that the serial number is unknown at the time of creation of the topics (since sbfspot was not yet started), but hardcoding the serial number in the topic did not help either.
Stefaan

Run a purge sensors
Or at worst case copy your options (in yaml mode is easy for copy paste) and uninstall /reinstall the addon.
When the addon starts trying to create the sensors the first time it creates a file with the serial number it receives via mqtt from SBFspot. It gets abit stuck if it creates that file incorrectly.
You can't hard code the serial in the mqtt topic in options. That is is one of the sacrifices that had to be made to get the addon options to convert to something that sbfspot could use.

The purge sensors did not solve it.
Uninstall & reinstall did. Now I have the serial number as part of the topic.
Thanks!
Closing the issue.


[SEDONA-498] Add ST_BestSRID

Did you read the Contributor Guide?

Yes, I have read Contributor Rules and Contributor Development Guide

Is this PR related to a JIRA ticket?

Yes, the URL of the associated JIRA ticket is https://issues.apache.org/jira/browse/SEDONA-498. The PR name follows the format [SEDONA-XXX] my subject.

What changes were proposed in this PR?
This PR introduces ST_BestSRID, which estimates the most appropriate SRID for a given geometry, based on its spatial extent and location.

For geometries in the Arctic or Antarctic regions, the Lambert Azimuthal Equal Area projection is used.
For geometries that fit within a single UTM zone and do not cross the International Date Line, a corresponding UTM SRID is chosen.
In cases where none of the above conditions are met, or for geometries that cross the International Date Line, the function defaults to the Mercator projection.

How was this patch tested?

Passes new and existing tests

Did this PR include necessary documentation updates?

Yes, I am adding a new API. I am using the current SNAPSHOT version number in since vX.Y.Z format.

@prantogg Please add Snowflake doc

Snowflake functions of this PR are being tested in a separate PR: https://github.com/wherobots/sedona-snowflake-tester/pull/11

Snowflake test failed: https://github.com/wherobots/sedona-snowflake-tester/actions/runs/7984544420/job/21801519880?pr=11


Event-Subscription for Azure Functions does not accept azurefunction as an option

If the issue is to do with Azure CLI 2.0 in-particular, create an issue here at Azure/azure-cli

Extension name (the extension in question)
az eventgrid event-subscription create
Description of issue (in as much detail as possible)
The example and the parameter for azurefunction as a parameter to endpoint-type fails with :
az eventgrid event-subscription create: 'azurefunction' is not a valid value for '--endpoint-type'. See 'az eventgrid event-subscription create --help'.

I have tested 2.3.1, 2.4.0, 2.5.1

eventgrid

Hello @ebrucucen  Can you please let me know if you are using CLI Extension or mainstream CLI?
The azurefunction as destination is currently available in CLI extension and we are in process of adding it to mainstream cli.
To get this functionality, please do the following.

Remove any eventgrid extension.

Install latest eventgrid extension.

Make sure the eventgrid extension's version is 0.4.8

ashraf@Azure:~$ az extension list
[
{
"experimental": false,
"extensionType": "whl",
"name": "eventgrid",
"preview": true,
"version": "0.4.8"
}
]

azurefunction is part of event subscription create command.

ashraf@Azure:~$ az eventgrid event-subscription create -h
The behavior of this command has been altered by the following extension: eventgrid
Command
az eventgrid event-subscription create : Create a new event subscription.
Arguments
--endpoint                                     : Endpoint where EventGrid should deliver events
matching this event subscription. For webhook
endpoint type, this should be the corresponding
webhook URL. For other endpoint types, this
should be the Azure resource identifier of the
endpoint.
--endpoint-type                                : The type of the destination endpoint.  Allowed
values: azurefunction, eventhub,
hybridconnection, servicebusqueue,
servicebustopic, storagequeue, webhook.
Default: webhook.

Please try the latest extension per comment above and let us know if you are having any issues.

Yes, this is perfect! Can we update the pages as it does not mention the requirement/installation of the extension.
https://docs.microsoft.com/en-us/cli/azure/ext/eventgrid/eventgrid/event-subscription?view=azure-cli-latest
Thank you!

Hello @ebrucucen ..Happy that your issue is resolved. We are closing the issue but if you still need anything, please feel free to reopen it.
As for your suggestion for adding the extension installation info to the event-subscription page, it will not be appropriate to add this info in the command help page. however, we have a new feature work that is coming soon where there will be no need to install any extension for event grid soon. Hope this helps.
thanks


Add how to debug section

Fixes #125
cc @jhamman.

Thanks for the review @jhamman, I've taken your comments into account.

Thanks @guillaumeeb!


implemented hierarchy checks for kick , ban ,role commands

and , removed the default value None for member arguments , i mean let the error handler deal with it shrugs

perhaps we can add a mention instead of name?

perhaps we can add a mention instead of name?

Where though?
If you mean at the "Unable to add role to ..."


Plugins that using Dataview

Discussed in https://github.com/blacksmithgu/obsidian-dataview/discussions/891

Originally posted by ooker777 February 24, 2022
I guess we may want to add a page in the doc for a list of plugins using Dataview.

Breadcrumbs
MetaEdit

This would be very useful


[Spike] API Documentation

Spike

Open API for documentation
Swagger for API interactions

For Service layer (via Spring) - see https://www.baeldung.com/spring-rest-openapi-documentation

Done - see https://github.com/credbull/credbull-app


interactive bots: Create GitHub issue bot.

This bot uses a github account owned by me using an alternate email address. It will post to the fredfishgames/zulip repository at the moment, which can be found here It will also push issues to the issues stream, followed by a tick or a cross depending on whether it was succesfully sent to github.

Automated message from Dropbox CLA bot
@fredfishgames, it looks like you've already signed the Dropbox CLA. Thanks!

Ok, so it looks like I need to store the oauth token as an environment variable, as as soon as I upload the token to github, it is deleted, however, environment variables are machine specific. Therefore, should I simply encrypt the token so that github can't detect it, so that it can be used. The token is unable to do any real damage, as it only allows access to specific github functions.

Ok, so i think i'll upload the key to google drive, then download it in the script. This will mean we can update it in the future if the key gets reset

@fredfishgames, awesome effort 👍. Just left some feedback, mostly dealing with the style. I think you would want just a single commit.

Thanks for the feedback!
I'll get working on these changes later today.

Oh, forgot to add in settings
Where is the settings file?

Ok, so: I've cleaned everything up, put all settings in ./zuliprc-local, and the key is now in dev-secrets.conf. However, both these files are only accessible on my remote instance. However, the code does work, but I am unable to update dev-secrets.conf, as it is not on github. So, hopefully you can see that my code works, but I if you would like proof, I can send some screenshots over. It might also work if you connect to my developer instance of zulip http://fredfishgames.zulipdev.org:9991/, and if you look at my github issue page https://github.com/fredfishgames/zulip/issues, you should be able to test it out using @issue. I will try to keep the script running.
I think I have fixed all the styling issues, and I have implemented all the features you have suggested.
Thanks,
Freddie

The idea of the bot is that particular Zulip Admin can configure (or not) particular bot. So we don't need your config file - we should do it ourselves.

ok thanks! I've just been running lint test, and fixed certain styling errors, and I'll do all your suggestions now

Ok, just fixed and cleaned up everything and it finally passed the lint test!

And additional comments ;-) Maybe something new for you ;-)

Added a comment for every line, hopefully thats enough :)

Oh, sorry, didn't see those earlier comments!

Hello, I've fixed all compatibility issues, and any errors that exist I've fixed, but it's not picking that up...

Hello, I've fixed all compatibility issues, and any errors that exist I've fixed, but it's not picking that up...

You mean the code is not updating on GitHub?

@fredfishgames, your code is beginning to look top-notch :). I think you just need to take care of the configuration file and some minor style issues and you are good to go.

@fredfishgames, this is mostly done. Some minor feedback though.

Hi @fredfishgames. Thanks for making all the necessary changes. Your code looks good to merge.
I would like to request @timabbott to see if this can now be merged. Thanks!

@reyha Have you had a chance to verify the code in your local dev environment?

Thanks!
I really appreciate all the help and feedback!
I didn't really know python before this task, let alone how to make web requests, and I have learned so much.
I'll be doing the github.py task next, so that it is easy for anyone to interface with the api without doing it directly.

Merged!  Thanks, @fredfishgames!


fix flake8 W292 for automatically generated files

I updated the scripts/write_complete_data.py to append a newline at the end of the automatically generated files to follow pep8.
I also updated the pytest.ini

Nice!


RFC: Make vue and svelte dependencies optional

Now, if I use React, i need to install @vue/compiler-dom, @vue/language-core and svelte2tsx, even though it isn't used
I think, it should be optional and live in separate plugins, for example @gql.tada/vue and @gql.tada/svelte and users could install it separately

The goal of the project is partially to simplify tooling. As such, there are certaind dev-time dependencies that will be installed as part of the dependencies. Ultimately, you don't install these dependencies directly, so the reason of "I need to install" this is not a valid one to request this, sorry 😅❤
What I would accept is install time or size, however, typescript is a much higher ratio of install size than any of the other dependencies.
Don't get me wrong, I do find it regrettable that to get the transpilers to TS to work for the CLI, that these dependencies pull in more transitive dependencies. However, the only solution imho for that is to create separate automatically installed plugins, if anything.
But as it stands, this is as intended, and asking users to install more dependencies on demand, without this being automated is a documentation/discoverability nightmare

We've gone several releases with fixes to the Vue/Svelte support and consider it pretty stable now. The only issues we've seen were breaking changes from the Vue package we've used, which is fine. While going through some work to make sure this feature is stable, I ultimately changed my mind on this problem.
We now have a pretty solid documentation experience and unified package install that streamlined the installation process. A change like this can still communicate issues to users of Vue/Svelte proactively, and hence, we can split out Vue and Svelte support into separate packages.
Whether we need any way to auto-install them is a problem that should be addressed retroactively. So, the PR linked splits Vue and Svelte support out into their respective, separate packages


Fix: User is not set admin when creating a new organization

The user is not set as the admin of the organization when he is creating a new one. So, he cannot add the new users and edit the details of the organization. Setting 'is_admin' as True solves the problem.
Cheers

Thanks for the pull request @DESHRAJ but until the errors are taken care of I can't consider accepting this pull request, and I don't have the time or inclination right now to clean it up myself.


Use libsodium.js if available

Please describe the changes this PR makes and why it should be merged:
Support the Emscripten version of libsodium: libsodium.js. Mainly to avoid the long compilation time of sodium.
Semantic versioning classification:

[ ] This PR changes the library's interface (methods or parameters added)

[ ] This PR includes breaking changes (methods removed or renamed, parameters moved or removed)

[ ] This PR only includes non-code changes, like changes to documentation, README, etc.

you need to put in the readme that it would support this module, and add it as a peer dep

Fixed, but apparently the README had CRLF instead of LF line endings. Should I revert it?

@skiptirengu no

Thanks for the contribution! 🎉


Pruning of old commits / revisions

As far as I understand, litetree works by keeping copies of all versions of any changed database pages.  But that also means that the total storage requirement grows over time, particularly if the database is changed very often (even if the database itself does not grow), right?  Is there a way to tell litetree that some commit / old version is no longer needed and can be discarded (like checkpointing in SQLite's WAL mode)?
Particularly for blockchain applications (which seems to be the intent of litetree) this can be useful:  In order to handle chain reorgs, it is necessary to keep (persistent) snapshots of the last couple of states.  But as more and more blocks are added on top of an old one, it can be pruned eventually to save space.  (In some sense, revisions in litetree are like undo data in Bitcoin Core, which can (optionally) be pruned for old blocks.)
Perhaps my understanding is also completely wrong or this is already supported.  If so, please let me know.

Hi Daniel,
I was researching exactly on this (merging commits) at this week.
Yes, it will be available on LiteTree. It may be released probably at this month.

@kroggen:  Ok cool, thanks for the confirmation!  I believe this can be a very powerful feature (particularly for blockchain projects as ours and yours).

Hi Daniel!
The support for merging commits is available in the merge-commits branch.
If you find any bug, please report it as a new issue.

Thanks, good to know!


fix(verde): serde macro features and locations

The #[verde::storage] macro incorrectly uses #[cfg(feature = "serde")] on the derives for Serialize and Deserialize, causing them not be applied, as that searches in the end-user crate and not verde itself. This changes it to conditionally apply the derive from verde's side.

Use #[serde(crate = "::verde::serde")] so that downstream crates do not necessarily need serde in their dependency lists.

certified cute


Add eslint & flow config

Essential Tools to start:

[ ] React
[ ] Flow
[ ] Eslint + Plugins
[ ] Prettier
[ ] Enzyme
[ ] Webpack


Add exceptions for host port and host networking

Towards https://github.com/giantswarm/giantswarm/issues/26746, https://github.com/giantswarm/giantswarm/issues/22657
Adds a Kyverno PolicyException allowing chart-operator to use host networking and a host port, which are necessary during the cluster bootstrap phase.
Checklist

[x] Update changelog in CHANGELOG.md.

Looks like this repo doesn't have any values schema and the action hadn't run for quite a while before this PR. I'd prefer to let the honey-est of badgers sort that out unless they have strong feelings about it

Tested on a WC on gremlin


Implement new OSC Signup Form

[ ] Copy existing form from ocf-host-application
[ ] Create additional information fields
[ ] Collect data from repository
Define abstract data-type to be attached to collective.data. Generic interface so we can later connect with GitLab and other different providers.

[ ] Is Fork
[ ] Has an SPDX compliante license
[ ] Recent activity
[ ] Is owned by org
[ ] Number of contributors

[ ] Pre-fill information that we already fetched from the repository
? Should the user be able to edit this data?
? What is Usage Link?

[ ] Repo URL
[ ] License
[ ] Commit Log
[ ] Usage Link

[ ] Style based on Figma designs https://www.figma.com/file/KinuveYf4WUlRIXzMTnmIsmO/[OC-Design]-Lab-🧪?node-id=16097%3A234377
[ ] Tech-debt: Fetch information from GitLab
[ ] Attach repository data to application.customData.

@iamronen @BenJam
Should the user be able to edit the data we automatically got from their repository? (Repository URL, License, etc...)
Also, what is the Usage Link?

To my understanding the auto-extraction from the repository is an assistive service so editing should be possible. Perhaps, if edited, there could be an indicator for the fiscal host admin? but I will defer this to @BenJam and @poohlaga

? Should the user be able to edit this data?

the idea is that we avoid having to review, so if we detect good data then we shouldn't allow the user to edit as that would mean we would need to review.

? What is Usage Link?

Something like this https://github.com/babel/babel/network/dependents though we need to keep it platform independent.

@BenJam I'm considering not allowing the edit and seeing how it goes. @poohlaga also reported people were editing to impersonate different projects and fraud payments.


Optimize rendering by rendering directly a DOM fragment

Currently, you are rendering HTML text, which you then insert into DOM. Maybe it would be faster if you would render directly a DOM fragment offline and then insert that at once? It should be pretty easy to define a visitor similar to this one, just that it is directly creating DOM elements.

The only issue is that I do not see an easy way to insert it through a helper because HTML.Raw with not work anymore. Maybe it is easier then to do something like {{> nrr narrgs 'template' .}} and then nrr helper returns a dummy template, which in its onRendered simply inserts the DOM fragment.


add a option for callback for list of recognised words in the API

I suggest adding an option in the api for the function to recieve the list of recoginsed phrases so that one can write custom logic with it

Hi @sktguha
Just so I can understand better, can you give an example of what the function will receive and how you would use it?

yes actually looking at it a second time it looks a bit complicated
perhaps it is possible to register an callback function to annyang as
[code]
annyang.addEventListener('recognised', function(arr, listOfCommands){
//so arr contains the list of recognised phrases by the speech recognisation engine
// listOfCommands contain the list of commands added upto now to annyang
});
[/code]

Can you give an example of how you would use this and what for?

well for example if I want to use a custom logic for triggering the actions. for example if I want to trigger the action even if the user has spoken, say the original phrase was "the expressway is to the left" but the user has spoken "the highway is to the left".
Also sometimes the user speaks the entire phrase correctly but it is recognised as seperate parts
for example "Hello how are you" is interpreted as "Hello how" and "are you"
I believe it can be done currently by turning debug mode on and overriding console.log function  but I guess that isn't the best way.
Also on that note I wonder if it possible to provide such a callback function so that say the application can display to the user the currently recognised phrases. Infact I needed to do this for my current application. for now I override the console.log function so that I could diplay to the user the phrases currently recognised by the engine so that the user does not become frustrated.

Hi,
Any updates ?

I just release v2.0.0 last night, inspired by your ideas.
One of the  features added, is that callbacks you register may now return with a list of recognized phrases.
For example:
annyang.addCallback('resultNoMatch', function(phrases) {
  console.log('No command matched. Possible sentences said:');
  console.log(phrases);
});

annyang.addCallback('result', function(phrases) {
  console.log('Speech recognized. Possible sentences said:');
  console.log(phrases);
});

annyang.addCallback('resultMatch', function(said, commandMatched, phrases) {
  console.log('Command matched');
  console.log('The phrase the user said that matched a command');
  console.log(said);
  console.log('The command that was matched');
  console.log(commandMatched);
  console.log('An array of possible alternative phrases the user might\'ve said');
  console.log(phrases);
});

Version 2.0.0 also added support for regular expressions in the commands, so you can do something like this:
var commands = {
  'the :road is to the :direction': {'regexp': /^the (?:highway|expressway) is to the (right|left)$/, 'callback': driveFunction}
}

You can read the full release notes here: https://github.com/TalAter/annyang/releases/tag/v2.0.0

Hi,
Wow that is really great . Indeed the callback system seems superb. Thanks you very much for this.
Indeed anyang as a library is pretty good. I was able to start very fast with the demos on the homepage. I will try to contribute to the project if I get the time

Thanks. :+1:
I would love to see what you build.

Oh sure.
https://github.com/sktguha/speechrecog
It is actually a simple demo of speech recognition based on iron man's video
The user has to speak the required words in place of iron man. They are shown instructions as speak "jarvis you there".
The video pauses at the points where iron man speaks, and on user speaking the required phrase, (in place of iron man) the video skips to the part where jarvis responds.
As of now there is a small bug. So if I seek to the point in the video, the video restarts intead of playing from the specified position( using vid.currentTime = POS). So I am thinking of using youtube API instead.
Also regarding unit testing annyang, issue https://github.com/TalAter/annyang/issues/91 ,  I suggest you to take a look at sinonJs (www.sinonjs.org). I have used it only for few toy examples but I think it is worth checking out

Very cool idea! Would love to see it again when the restart issue is fixed.
As for Sinon... Yes, I've looked at that for https://github.com/TalAter/Carmen but I ended up using something called Georilla (which is the equivalant of what's needed for testing annyang, only for Geolocation testing). When I'm able to pursue writing tests for annyang, I'll look into that and Sinon again.


RuntimeError: Given groups=1, weight of size [64, 3, 7, 7], expected input[1, 1000, 512, 512] to have 3 channels, but got 1000 channels instead

RuntimeError: Given groups=1, weight of size [64, 3, 7, 7], expected input[1, 1000, 512, 512] to have 3 channels, but got 1000 channels instead

The length of train set is: 206
206 iterations per epoch. 20600 max iterations
0%|                                         | 0/100 [02:11<?, ?it/s]
Traceback (most recent call last):
File "E:/AI/SUnetOK/TransUnet/TransUNet-main/train.py", line 93, in
trainer[dataset_name](args, net, snapshot_path)
File "E:\AI\SUnetOK\TransUnet\TransUNet-main\trainer.py", line 55, in trainer_synapse
outputs = model(image_batch)
File "D:\InstallAPP\Anocanda\lib\site-packages\torch\nn\modules\module.py", line 727, in _call_impl
result = self.forward(*input, **kwargs)
File "E:\AI\SUnetOK\TransUnet\TransUNet-main\networks\vit_seg_modeling.py", line 388, in forward
x, attn_weights, features = self.transformer(x)  # (B, n_patch, hidden)
File "D:\InstallAPP\Anocanda\lib\site-packages\torch\nn\modules\module.py", line 727, in _call_impl
result = self.forward(*input, **kwargs)
File "E:\AI\SUnetOK\TransUnet\TransUNet-main\networks\vit_seg_modeling.py", line 254, in forward
embedding_output, features = self.embeddings(input_ids)
File "D:\InstallAPP\Anocanda\lib\site-packages\torch\nn\modules\module.py", line 727, in _call_impl
result = self.forward(*input, **kwargs)
File "E:\AI\SUnetOK\TransUnet\TransUNet-main\networks\vit_seg_modeling.py", line 156, in forward
x, features = self.hybrid_model(x)
File "D:\InstallAPP\Anocanda\lib\site-packages\torch\nn\modules\module.py", line 727, in _call_impl
result = self.forward(*input, **kwargs)
File "E:\AI\SUnetOK\TransUnet\TransUNet-main\networks\vit_seg_modeling_resnet_skip.py", line 145, in forward
x = self.root(x)
File "D:\InstallAPP\Anocanda\lib\site-packages\torch\nn\modules\module.py", line 727, in _call_impl
result = self.forward(*input, **kwargs)
File "D:\InstallAPP\Anocanda\lib\site-packages\torch\nn\modules\container.py", line 117, in forward
input = module(input)
File "D:\InstallAPP\Anocanda\lib\site-packages\torch\nn\modules\module.py", line 727, in _call_impl
result = self.forward(*input, **kwargs)
File "E:\AI\SUnetOK\TransUnet\TransUNet-main\networks\vit_seg_modeling_resnet_skip.py", line 25, in forward
self.dilation, self.groups)
RuntimeError: Given groups=1, weight of size [64, 3, 7, 7], expected input[1, 1000, 512, 512] to have 3 channels, but got 1000 channels instead
Process finished with exit code 1

RuntimeError: Given groups=1, weight of size [64, 3, 7, 7], expected input[1, 1000, 512, 512] to have 3 channels, but got 1000 channels instead

Sorry to bother you. Could you be so kind to share the training and test data? My email is<EMAIL_ADDRESS>

I also encountered this error, have you solved it yet?


Redirect /login to /dashboard if logged in

[X] I have run rspec and corrected all errors
[X] I have run rubocop and corrected all errors
[X] I have tested this change locally and tried to launch the server as well as access a project, and with that at least one build
[X] If there is an existing issue, make sure to add Fixes ... as part of the PR body to reference the issue you're solving

Fixes https://github.com/fastlane/ci/issues/934

@nakhbari oops sry. any chance we run this on ci as well?

@powerivq we tried, but because we currently do not cleanup the files created during a build the disk space ended up being completely used up. Maybe we can look into it again when we get that feature fixed. https://github.com/fastlane/ci/issues/950

@nakhbari Oh gotcha. Thanks for the fix btw.
@nakhbari @taquitos Do you think it's a good idea to run tests on circle as well? I feel that extra coverage won't hurt at the very least.


DirName's separators and their length are known at compile time

Thanks for the patch! What's the motivation here? Does it improve generated code?

Well, I guess it's not going to make a huge difference but as I spotted it while I was having a look at the code, I thought I'd report it, just in case.
Actually, it improves the object size (from 116k to 110k) and reduces the number of instructions taken by the MakeDirs method from 260 to 160.


Wines Problem

💥 Proposal
Given n wines in a row, with integers denoting the cost of each wine respectively.
Each year you can sell the first or the last wine in the row.
Let the initial profits from the wines be P1, P2, P3…Pn.
In the Yth year, the profit from the ith wine will be Y*P[i].
The goal is to calculate the maximum profit that can be earned by selling all the wines.
Have you read the Contributing Guidelines on Pull Requests?
Yes
I want to work on this in C++

/assign


Invalid character. SearchResult«Share»

The » character is causing me problems with this package:
The error message looks like:
Invalid character. (65:30)

export interface SearchResult«Share» {

Seems like the Typescript doesn't like the » symbol.

@elie222 yes TypeScript doesn’t allow ». How are you using it in Swagger? And why that character? We could possibly add support if I got a usage example of both your Swagger definition and the TypeScript code you’d like it to output.

Honestly I don’t know much about the code that generated it. I ended up replacing that character with underscores and then running it through the script. Thanks for the help
Best,
Eliezer
On Thu, Jan 2 2020 at 5:46 PM, <<EMAIL_ADDRESS>> wrote:

@elie222 ( https://github.com/elie222 ) yes TypeScript doesn’t allow ».
How are you using it in Swagger? And why that character? We could possibly
add support if I got a usage example of both your Swagger definition and
the TypeScript code you’d like it to output.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub (
https://github.com/manifoldco/swagger-to-ts/issues/86?email_source=notifications&email_token=AAXSQX7QDIEJ7ZLGWUKJMJLQ3YD6RA5CNFSM4JGKKK3KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEH6UQCI#issuecomment-570247177
) , or unsubscribe (
https://github.com/notifications/unsubscribe-auth/AAXSQXYYFD6MYSM7CRYDVI3Q3YD6RANCNFSM4JGKKK3A
).

OK. I’m going to close the issue then but if you ever run across another weird character, just provide a snippet of the Swagger definition, and what TypeScript code you’d like it to generate, and I’d be happy to re-open this issue.


use safe-buffer

@jprichardson could you add me to the contributors list to publish this?
Also,  maybe minimize the number of active contributors...

Done

Published as v1.0.6


Update showcase link of More Resource

Updated the link showcase in More Resource section.
fix for #1724

Thanks for spotting this!

Always welcome 😸


Drop duplicate Helm values from values.yaml

Fixes #4441

Drop duplicate PodSecurityContext usage
Drop duplicate ServiceAccount usage

I think we just merged a PR that fixes the e2e tests so maybe you can rebase onto main? I would do it but I'm not sure if it will let me 🤔

@jsjoeio is this ready for merge? I don't want main getting to far out of sync and having to do a fat rebase again 😅

Yes! Thanks for the bump! Merging :D


Check for qualname presence in Constant.repr

Stumbled upon this code assumption of the __qualname__ attribute being
present for all Constant values. Sadly, this is not always implemented in the
API of libraries. For example, NumPy's unfunc do not have a __qualname__, as
discussed in https://github.com/numpy/numpy/issues/4952. I noticed that other
parts of the NeuroLang code do check for the presence of the __qualname__
attribute before accessing it. For example, in the Expression class
    @property
    def __type_repr__(self):
        if (
            hasattr(self.type, '__qualname__') and
            not hasattr(self.type, '__args__')
        ):
            return self.type.__qualname__
        else:
            return repr(self.type)

Good catch. Funny that all python 3.4 all functions should have it and python produces automatically. 🤷 https://www.python.org/dev/peps/pep-3155/
Will merge when tests pass


refactor(api): trying to debug retrieval of results of simulation runs

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


Non-numeric prefixes stripped from tags to check versions

          1 Error

:no_entry_sign:
Please include a CHANGELOG entry.

  Generated by :no_entry_sign: Danger

Codecov Report

Merging #413 into master will increase coverage by 0.09%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #413      +/-   ##
==========================================
+ Coverage   67.14%   67.24%   +0.09%
==========================================
  Files         148      148
  Lines        8724     8723       -1
==========================================
+ Hits         5858     5866       +8
+ Misses       2408     2401       -7
+ Partials      458      456       -2

Impacted Files
Coverage Δ

ext/git/context.go
90.47% <100%> (+6.1%)
:arrow_up:

lib/source_context.go
88% <100%> (+7.23%)
:arrow_up:

lib/build_config.go
96.07% <100%> (+0.07%)
:arrow_up:

lib/auto_resolver.go
88.05% <0%> (-1.5%)
:arrow_down:

ext/git/client.go
72.44% <0%> (+3.06%)
:arrow_up:

Continue to review full report at Codecov.

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


adds the service urls for the new regions

PR summary

Adds in two new service urls (eu-de and eu-gb).
PR Checklist
Please make sure that your PR fulfills the following requirements:

[x] The commit message follows the Angular Commit Message Guidelines.
[x] Tests for the changes have been added (for bug fixes / features)
[x] Docs have been added / updated (for bug fixes / features)

Current vs new behavior

Does this PR introduce a breaking change?

[ ] Yes
[x] No

Other information

Vans-MacBook-Pro:platform-services-go-sdk vbui$ go test github.com/IBM/platform-services-go-sdk/atrackerv1  -tags="integration" -v
=== RUN   TestAtrackerV1
Running Suite: AtrackerV1 Suite
===============================
Random Seed:<PHONE_NUMBER>
Will run 129 of 129 specs

•••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••Service URL: https://us-south.atracker.cloud.ibm.com
••••••••••••••••••••••••••••••••••••••
Ran 129 of 129 Specs in 12.459 seconds
SUCCESS! -- 129 Passed | 0 Failed | 0 Pending | 0 Skipped
--- PASS: TestAtrackerV1 (12.48s)
PASS
ok      github.com/IBM/platform-services-go-sdk/atrackerv1      12.968s

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

@van-chau-bui-ibm question... did you make the change to the generated service code manually, or did you re-generate the service from the API definition after the API was updated to define the additional regions?
If it's the former, we really should not be making manual changes to the generated code.  It's much better to first update the API, then just re-gen the code using that.
cc: @yingwang-us

Closing, regenerated in https://github.com/IBM/platform-services-go-sdk/pull/192 and this is a duplicate


First node doesn't get a textbox when the mouse is on the item

When loading the sample file, the first node in the json file doesn't get a textbox when the mouse is on the item like all others.
I did some tests with other json files.
When the nodes are reordered, still only the first node of the file gets the problem.

Having the file "vis.css" fixes the problem ...


feat: pluggable matcher

Background
Original issue: #2132. In short, the performance of adding a large number of routes is slow, and is particularly problematic with SSR.
This PR explores the idea of a 'pluggable matcher'. This allows the matcher to be passed into the router, rather than being created internally. The potential benefits are twofold:

In an SSR scenario, the matcher can be created upfront, rather than processing the individual routes for each request.
Custom matchers can be used, potentially supporting different features or faster implementations.

The code presented here is a long way from being merged, but hopefully it'll work as a starting point to discuss how to proceed.
SSR usage
To get the benefits with SSR, we'd need to replace this:
const router = createRouter({
  history,
  routes: [/* ... */],
  strict,
  sensitive
})

with this:
// Do this once
const matcher = createDefaultMatcher({
  routes: [/* ... */],
  strict,
  sensitive
})

// Do this for each request
const router = createRouterWithMatcher({
  history,
  matcher: matcher.clone()
})

The clone() call creates a copy of the matcher, allowing for more routes to be added or removed without impacting the original matcher.
The performance of this is really good. Calling clone() is very fast and we skip all the overhead of processing the individual routes.
Custom matchers
This is trickier.
The usage is similar to the previous example, but with a custom implementation of matcher.
TypeScript isn't my strong suit, so perhaps this isn't as difficult as it seemed, but I found trying to get the types working as I wanted really difficult. The code in this PR is much less ambitious than some of my other attempts, but hopefully it'll still illustrate the difficulties.
For example, I tried writing a custom matcher that only supported the options path, name, component and components. You can see that at:

SimpleRouterMatcher

If you search through the code for Pain point, you'll find various points where I had to do things that didn't really make sense in the context of that matcher implementation. Some of those may be necessary for other reasons, but in many cases I was adding things just to appease TS. For example, the score and re properties are completely redundant, but the types force them to be included.
I did experiment with changing those types, adding generics all over the place, but it got a bit out of hand and I decided to aim a little lower for this initial PR. Instead, I chose to focus just on the route config.
As SimpleRouterMatcher only supports 4 options for the route, I wanted the types to reflect that. So SimpleRoute is a type that defines that format, with the matcher having type:
interface SimpleRouterMatcher extends GenericRouterMatcher<SimpleRoute> {}

You'll find RC used a generic in various places in the code. RC stands for 'route config'.
This then has an impact on the type of the addRoute() method of the router itself. New routes need to be added using the SimpleRoute format. e.g.:
const matcher = createSimpleRouterMatcher({
  routes: [
    {
      path: '/',
      component: HomeView
    }
  ]
})

const router = createRouterWithMatcher({ history, matcher })

// The type here is 'SimpleRoute', so options like 'strict' aren't available
router.addRoute({
  path: '/about',
  component: AboutView
})

This almost works. The major problem is $router and useRouter(), neither of which take account of the generic.
Other problems
The options property of the Router type is a bit fiddly. It needs to take account of the options passed to createRouterWithMatcher(), which aren't the same as createRouter().
Next steps
If we just want the performance benefits for SSR usage, we don't need support for custom matchers. That would dodge most of the type problems.
The custom matcher feature may tie into #2148. That PR is currently implemented as changes to the default matcher, but it could instead be implemented as a separate matcher implementation.
If the type problems can be overcome, I think we'd also need to consider adding helpers to make custom matchers easier to write. I found myself duplicating a lot of logic from the default matcher when I was writing SimpleRouterMatcher. If we do choose to pursue this further, I recommend anyone wanting to give feedback to attempt writing a custom matcher. I found going through that process quite enlightening.
Depending on what direction we choose to go in, we should also consider an RFC. At the very least we'd need some feedback from the SSR meta-frameworks about whether this is workable for them.

Codecov Report
Attention: Patch coverage is 83.33333% with 1 lines in your changes are missing coverage. Please review.

Project coverage is 90.93%. Comparing base (089378b) to head (be657db).
Report is 3 commits behind head on main.

Files
Patch %
Lines

packages/router/src/matcher/index.ts
83.33%
1 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2166      +/-   ##
==========================================
- Coverage   91.01%   90.93%   -0.09%
==========================================
  Files          24       24
  Lines        1135     1136       +1
  Branches      351      351
==========================================
  Hits         1033     1033
- Misses         63       64       +1
  Partials       39       39

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


Compute descriptive statistics for string type

treat as category if num of unique values <= 40
compute numerical stats over string length otherwise

Codecov Report
Patch coverage: 82.53% and project coverage change: +1.70% :tada:

Comparison is base (4771637) 90.51% compared to head (3cc88d4) 92.22%.
Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1816      +/-   ##
==========================================
+ Coverage   90.51%   92.22%   +1.70%
==========================================
  Files         222       83     -139
  Lines       13697     5955    -7742
==========================================
- Hits        12398     5492    -6906
+ Misses       1299      463     -836

Flag
Coverage Δ

jobs_cache_maintenance
?

jobs_mongodb_migration
?

libs_libcommon
?

services_admin
?

services_api
?

services_rows
?

services_search
?

services_sse-api
?

services_worker
92.22% <82.53%> (-0.20%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Changed
Coverage Δ

...s/job_runners/split/test_descriptive_statistics.py
91.39% <73.33%> (-6.46%)
:arrow_down:

...worker/job_runners/split/descriptive_statistics.py
92.11% <88.00%> (-1.78%)
:arrow_down:

services/worker/tests/fixtures/datasets.py
100.00% <100.00%> (ø)

... and 139 files with indirect coverage changes

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

also i realized why it would be helpful to have feature as another level of granularity - the way it works now we will have to recompute all the datasets, just to add one feature. any ideas how to mitigate this? @severo @AndreaFrancis
@lhoestq suggested to update instead of rewrite the result if it exists in cache but force recomputing also must be supported

also i realized why it would be helpful to have feature as another level of granularity - the way it works now we will have to recompute all the datasets, just to add one feature. any ideas how to mitigate this? @severo @AndreaFrancis

yes, we could add the column/feature level to dataset/config/split. I think we had some discussion already, but couldn't find it. Maybe create a dedicate issue for that.

@lhoestq suggested to update instead of rewrite the result if it exists in cache but force recomputing also must be supported

hmmm, due to the imbalance between strings and other types, I would say the overhead will not be significant on this PR (we will recompute all the datasets anyway)

hmmm, due to the imbalance between strings and other types, I would say the overhead will not be significant on this PR (we will recompute all the datasets anyway)

Yea now it will compute stats for literally all the datasets ! 🤯


Slides height is set to 240px when page loads

This is a (multiple allowed):

[x] bug

[ ] enhancement

[ ] feature-discussion (RFC)

Swiper Version: 3.4.1

Platform/Target and Browser Versions: macOS, Chrome

What you did
Following the vertical slides + responsive / fullscreen demo i created a simple html structure with swiper and initialized it. When the page first loads slides have a height of 240px. When i resize the window height is set to make slides occupy the fulle screen (as i want).
JS:
var mySwiper = new Swiper('.swiper-container', {
        // Optional parameters
        direction: 'vertical',
        loop: false,
        initialSlide: 0,
        speed: 800,
        mousewheelControl: true,
    });

HTML:
.swiper-container
    .swiper-wrapper
        .swiper-slide
            h1 slide 1
        .swiper-slide
            h1 slide 2
        .swiper-slide
            h1 slide 3

CSS:
.swiper-container {
    width : 100%;
    height: 100%;
}

Expected Behavior
Slides should occupy the full screen.
Actual Behavior
Slides are initialized with 240px height.

Would be good to see live link or JSFiddle with the issue. But looks like error on your side with layout

Issue is closed because of outdated/irrelevant/not actual/needed more information/inactivity.
If this issue is still actual and reproducible for latest version of Swiper, please create new issue and fill the issue template correctly:

Clearly describe the issue including steps to reproduce when it is a bug.
Make sure you fill in the earliest version that you know has the issue.
Provide live link or JSFiddle/Codepen or website with issue


[HUDI-3021] Fix HoodieAppendHandle#appendDataAndDeleteBlocks writer o…

…bject occurred NPE
Tips

Thank you very much for contributing to Apache Hudi.
Please review https://hudi.apache.org/contribute/how-to-contribute before opening a pull request.

What is the purpose of the pull request
(For example: This pull request adds quick-start document.)
Brief change log
(for example:)

Modify AnnotationLocation checkstyle rule in checkstyle.xml

Verify this pull request
(Please pick either of the following options)
This pull request is a trivial rework / code cleanup without any test coverage.
(or)
This pull request is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:
(example:)

Added integration tests for end-to-end.
Added HoodieClientWriteTest to verify the change.
Manually verified the change by running a job locally.

Committer checklist

[ ] Has a corresponding JIRA in PR title & commit

[ ] Commit message is descriptive of the change

[ ] CI is green

[ ] Necessary doc changes done or have another open PR

[ ] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.

jump to https://github.com/apache/hudi/pull/4329


Javascript Label bad colors on Editor

The colors are failing with the label statement

VSCode Version: 1.23.0
OS Version: Windows 10

Steps to Reproduce:

Create a Javascript File
Paste following code

class Test {
  constructor() {
    hola: {
      const x = 1;
      console.log('bad highlight');
      break hola;
    }
  }
}

Does this issue occur when all extensions are disabled?: Yes

This issue was moved to Microsoft/TypeScript-TmLanguage#610


CompositionFailedException (Microsoft.ApplicationInsights)

In https://github.com/pharring/EditorGuidelines/issues/21#issuecomment-868253883 @prodigio writes:
Suddenly I encountered the same exception. On my side everytime I open up VS 2019 Version 16.9.6 and EditorGuidelines 2.2.7.
Microsoft.VisualStudio.ComponentModelHost.VsShellComponentModelHost

A MEF Component threw an exception at runtime: Microsoft.VisualStudio.Composition.CompositionFailedException: An exception was thrown while initializing part "EditorGuidelines.ColumnGuideAdornmentFactory". --->

System.IO.FileNotFoundException: Could not load file or assembly 'Microsoft.ApplicationInsights, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=31bf3856ad364e35' or one of its dependencies. The system cannot find the file specified.

at EditorGuidelines.ColumnGuideAdornmentFactory.OnImportsSatisfied()

--- End of inner exception stack trace ---

at Microsoft.VisualStudio.Composition.RuntimeExportProviderFactory.RuntimeExportProvider.RuntimePartLifecycleTracker.InvokeOnImportsSatisfied()

at Microsoft.VisualStudio.Composition.ExportProvider.PartLifecycleTracker.NotifyTransitiveImportsSatisfied()

at Microsoft.VisualStudio.Composition.ExportProvider.PartLifecycleTracker.MoveNext(PartLifecycleState nextState)

at Microsoft.VisualStudio.Composition.ExportProvider.PartLifecycleTracker.MoveToState(PartLifecycleState requiredState)

at Microsoft.VisualStudio.Composition.ExportProvider.PartLifecycleTracker.GetValueReadyToExpose()

at Microsoft.VisualStudio.Composition.RuntimeExportProviderFactory.RuntimeExportProvider.<>c__DisplayClass15_0.<GetExportedValueHelper>b__0()

I checked the released VSIX and the Microsoft.ApplicationInsights assembly is there, so it's not immediately obvious why we'd get this failure.

I can't reproduce this (although I have 16.10.2). The assembly is present in the VSIX, so that shouldn't be an issue.
@Prodigio could you try uninstalling and re-installing the extension?

It seems a simple re-install fixed the problem. Thanks anyway!

Thanks for confirming. Good to know there's a quick workaround even if we don't know the root cause.


[Test Spark RC] Upgrade Spark version to use 3.5.2 RC2

Run CI jobs with Spark 3.5.2 RC2

Hi @vkorukanti.
Spark 3.5.2 (RC3) is polling. Please help Spark community to verify it.
Thanks,
Kent Yao

The Spark master failure is unrelated to this PR.


Implemented Hive registration steps in Hive distcp.

@ydai1124 can you review?

@zliu41 can you review?

@ydai1124 fixed comments, and did some changes to code.

@zliu41 Addressed comments.

LGTM other than one more comment.

LGTM


success event cannot be catched when use download manager

I added an option when success to popup a message when it's success download.
But it only popup when download by the browser and not popup when I use internet download manager.
Is there anything wrong?

Yeah an internet download manager probably wouldn't work with this. I doubt more than .01% of your clientele use one of those (and besides if they do they can likely expect weird behavior on many websites). This won't and can't be supported.


MQTT climate platform [continuation of #8750]

Description:
First of all: This is based on PR #8750 by @clkao who agreed that I finish this. I mainly added further tests, fixed some bugs and added a way of receiving the current temperature via MQTT. This PR description is also mainly copied from there.

The platform currently works in optimistic mode, which means it does not obtain states from MQTT topics, but it sends and remembers control commands.
It uses a sensor or a MQTT topic to obtain the current temperature.
Related issue (if applicable): replaces #8750
Pull request in home-assistant.github.io with documentation (if applicable): home-assistant/home-assistant.github.io#3448
Example entry for configuration.yaml (if applicable):
climate:
  - platform: mqtt
    name: Study
    target_sensor: sensor.study_temperature

Checklist:
If user exposed functionality or configuration variables are added/changed:

[x] Documentation added/updated in home-assistant.github.io

If the code does not interact with devices:

[x] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[x] Tests have been added to verify that the new code works.

@clkao What was your intention behind the 'power' mode MQTT topic? It seems to me like it's just sending off / on based on whether the operation mode is off / something else. It feels somehow strange to have a separate MQTT topic for this. All other things that are sent / received are properties of a climate device (operation mode, target temperature, swing mode, …), but the 'power mode' seems to be special to the MQTT climate platform. Would it be okay for you if I removed it?

@tinloaf please see the actual use case, although the example is not using the power state channel.
Some AC controls have a separate power state (as toggle) in addition to fan modes, although we can maintain state and decide what code combinations to send in the climate component.  That is, you might need to send "power on" + "mode medium" if the current state is "off", but just "power off" if current state is "on".  You might always send "power on" + "mode", but in some cases "power on" and "power off" are the same code (toggle) so this wouldn't work.
With the optional power state channel defined, it is easier to glue the IR control services.  Given that the mqtt component is probably used as an intermediate, it's probably best to provide this flexibility.

@clkao Okay, since it doesn't really hurt I'll leave it in. There can be no 'state topic' however since it would be unclear what operation mode to enter when the power is switched back on via the state topic.


Dump.OneBodyWorld unit test fails.

As noticed by Megaxela, the Dump.OneBodyWorld unit test fails under linux gcc builds with the following result:
gtest-port.cc:616: Failure
Value of: is_valid_
  Actual: false
Expected: true
Regular expression "Vec2 g\(0\.000000000000000e\+00f, -9\.800000190734863e\+00f\);
m_world->SetGravity\(g\);
Body\*\* bodies = \(Body\*\*\)alloc\(1 \* sizeof\(Body\*\)\);
{
  BodyDef bd;
  bd\.type = BodyType\(0\);
  bd\.position = Vec2\(0\.000000000000000e\+00f, 0\.000000000000000e\+00f\);
  bd\.angle = 0\.000000000000000e\+00f;
  bd\.linearVelocity = Vec2\(0\.000000000000000e\+00f, 0\.000000000000000e\+00f);
  bd\.angularVelocity = 0\.000000000000000e\+00f;
  bd\.linearDamping = 0\.000000000000000e\+00f;
  bd\.angularDamping = 0\.000000000000000e\+00f;
  bd\.allowSleep = bool\(1\);
  bd\.awake = bool\(1\);
  bd\.fixedRotation = bool\(0\);
  bd\.bullet = bool\(1\);
  bd\.enabled = bool\(1\);
  bodies\[0\] = m_world->CreateBody\(bd\);

}
Joint\*\* joints = \(Joint\*\*\)alloc\(0 \* sizeof\(Joint\*\)\);
free\(joints\);
free\(bodies\);
joints = nullptr;
bodies = nullptr;
" is not a valid POSIX Extended regular expression.
/home/megaxela/Development/Build/Box2D/Box2D/UnitTests/Dump.cpp:121: Failure
Death test: { open_stderr_as_stdout(); World world; world.CreateBody(); Dump(world); exit(0); }
    Result: died but not with expected error.
  Expected: Vec2 g\(0\.000000000000000e\+00f, -9\.800000190734863e\+00f\);
m_world->SetGravity\(g\);
Body\*\* bodies = \(Body\*\*\)alloc\(1 \* sizeof\(Body\*\)\);
{
  BodyDef bd;
  bd\.type = BodyType\(0\);
  bd\.position = Vec2\(0\.000000000000000e\+00f, 0\.000000000000000e\+00f\);
  bd\.angle = 0\.000000000000000e\+00f;
  bd\.linearVelocity = Vec2\(0\.000000000000000e\+00f, 0\.000000000000000e\+00f);
  bd\.angularVelocity = 0\.000000000000000e\+00f;
  bd\.linearDamping = 0\.000000000000000e\+00f;
  bd\.angularDamping = 0\.000000000000000e\+00f;
  bd\.allowSleep = bool\(1\);
  bd\.awake = bool\(1\);
  bd\.fixedRotation = bool\(0\);
  bd\.bullet = bool\(1\);
  bd\.enabled = bool\(1\);
  bodies\[0\] = m_world->CreateBody\(bd\);

}
Joint\*\* joints = \(Joint\*\*\)alloc\(0 \* sizeof\(Joint\*\)\);
free\(joints\);
free\(bodies\);
joints = nullptr;
bodies = nullptr;

Actual msg:
[  DEATH   ] Vec2 g(0.000000000000000e+00f, -9.800000190734863e+00f);
[  DEATH   ] m_world->SetGravity(g);
[  DEATH   ] Body** bodies = (Body**)alloc(1 * sizeof(Body*));
[  DEATH   ] {
[  DEATH   ]   BodyDef bd;
[  DEATH   ]   bd.type = BodyType(0);
[  DEATH   ]   bd.position = Vec2(0.000000000000000e+00f, 0.000000000000000e+00f);
[  DEATH   ]   bd.angle = 0.000000000000000e+00f;
[  DEATH   ]   bd.linearVelocity = Vec2(0.000000000000000e+00f, 0.000000000000000e+00f);
[  DEATH   ]   bd.angularVelocity = 0.000000000000000e+00f;
[  DEATH   ]   bd.linearDamping = 0.000000000000000e+00f;
[  DEATH   ]   bd.angularDamping = 0.000000000000000e+00f;
[  DEATH   ]   bd.allowSleep = bool(1);
[  DEATH   ]   bd.awake = bool(0);
[  DEATH   ]   bd.fixedRotation = bool(0);
[  DEATH   ]   bd.bullet = bool(1);
[  DEATH   ]   bd.enabled = bool(1);
[  DEATH   ]   bodies[0] = m_world->CreateBody(bd);
[  DEATH   ]
[  DEATH   ] }
[  DEATH   ] Joint** joints = (Joint**)alloc(0 * sizeof(Joint*));
[  DEATH   ] free(joints);
[  DEATH   ] free(bodies);
[  DEATH   ] joints = nullptr;
[  DEATH   ] bodies = nullptr;
[  DEATH   ]

Note that this test failure is due to the unit test code not being up-to-date with the implementation. It is not OS dependent. The implementation also has the problem that it's not up-to-date with respect to its purpose either.

I've decided to just ifdef out the Dump unit test code for now. That's been committed now. So this test shouldn't fail anymore. Wasn't a failure in the actual dumping code as much as the unit test code not having been updated to keep up.


Add documentation for migrating from SharedPreferences

This will be fix #46

Current coverage is 100% (diff: 100%)

Merging #79 into master will not change coverage

@@           master   #79   diff @@
===================================
  Files          22    22
  Lines         656   656
  Methods         0     0
  Messages        0     0
  Branches       97    97
===================================
  Hits          656   656
  Misses          0     0
  Partials        0     0

Powered by Codecov. Last update cc42885...b62110a


Fix the overlap between submission cells in contact diary

Description
fix the overlap between submission cells
Link to Jira
https://jira-ibs.wbs.net.sap/browse/EXPOSUREAPP-14479
Screenshots
before

after

May I ask in which situation two entries are shown? If the user warns twice on the same day?

May I ask in which situation two entries are shown? If the user warns twice on the same day?

Edge case, only relevant for testers. Should not happen in ideal case.

May I ask in which situation two entries are shown? If the user warns twice on the same day?

@Ein-Tim exactly like what Naveed said, normally we have a condition that a certain time has to pass between submissions and more that one submission in the same day is logically not possible but we can disable these checks sometimes for testing and this is where we found the bug


Unable deserialize Protobuf POJO with Spring Boot

I've the following configuration:
spring.kafka.consumer.value-deserializer=com.amazonaws.services.schemaregistry.deserializers.GlueSchemaRegistryKafkaDeserializer
spring.kafka.properties.dataFormat=PROTOBUF
spring.kafka.properties.protobufMessageType=POJO

  @KafkaListener(groupId = "test", topics = "test")
  void consume(final Timestamp payload)  {
  }

However, it looks like the descriptor based approach isn't detecting the class to to be com.google.protobuf.Timestamp; instead it thinks the class is com.google.protobuf.Any. I get the following error:

org.springframework.messaging.converter.MessageConversionException: Cannot convert from [com.google.protobuf.Any] to [com.google.protobuf.Timestamp]

I do prefer this detection approach over having a configuration (i.e. (KafkaProtobufDeserializerConfig.SPECIFIC_PROTOBUF_VALUE_TYPE in Confluent SerDes).

Debugging further inside ProtobufDeserializer#deserialize, it looks like the schema and schemaDefinition are both correctly pointing to com.google.protobuf.Timestamp.


Add physical storage usage and number of blobs in BlobStoreStats

Add physical storage usage and number of blobs in the BlobStoreStats.
We don't return these two new info right now, since we don't want to change the interface. But this PR would expose them in the test cases so we can test them.
Also adding this to the ScanResult in the BlobStoreStats.

Codecov Report

Merging #1948 (fb83bf1) into master (7fe53a1) will decrease coverage by 58.30%.
The diff coverage is 92.79%.

@@              Coverage Diff              @@
##             master    #1948       +/-   ##
=============================================
- Coverage     73.78%   15.47%   -58.31%
+ Complexity     9688     2302     -7386
=============================================
  Files           694      694
  Lines         53450    53516       +66
  Branches       6654     6658        +4
=============================================
- Hits          39437     8284    -31153
- Misses        11928    44750    +32822
+ Partials       2085      482     -1603

Impacted Files
Coverage Δ

...in/java/com/github/ambry/store/BlobStoreStats.java
84.30% <91.39%> (+0.28%)
:arrow_up:

...rc/main/java/com/github/ambry/store/BlobStore.java
82.74% <100.00%> (-3.04%)
:arrow_down:

.../main/java/com/github/ambry/store/ScanResults.java
96.93% <100.00%> (+0.55%)
:arrow_up:

.../src/main/java/com/github/ambry/router/Router.java
0.00% <0.00%> (-100.00%)
:arrow_down:

.../src/main/java/com/github/ambry/store/Message.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...rc/main/java/com/github/ambry/named/PutResult.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...rc/main/java/com/github/ambry/quota/QuotaMode.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...rc/main/java/com/github/ambry/quota/QuotaName.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...rc/main/java/com/github/ambry/rest/RestMethod.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...c/main/java/com/github/ambry/config/KMSConfig.java
0.00% <0.00%> (-100.00%)
:arrow_down:

... and 548 more

Continue to review full report at Codecov.

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


Refactor extraction & remove eslint-plugin-html dependency

The plugin seems to work only when eslint-plugin-html is enabled too. And in a tricky way. So I removed this dependency and do some refactoring.

@joehybird Thanks for this input.
This removes the capability to handle multi-line templates, which Tornado supports. Could you please revert that change?

Ok, I would never guessed that tornado was supported too :) Documentation needs to be improved on this point. And maybe separate detemplatize in different functions for each template language could be great

Noted. There aren't that many template languages, actually, and Tornado's escaping is mostly a superset of Django's. So the earlier code would have worked both for Tornado and Django.
Would you like to revise this PR to just handle the eslint-plugin-html issue?

Noted. There aren't that many template languages, actually, and Tornado's escaping is mostly a superset of Django's. So the earlier code would have worked both for Tornado and Django.
Would you like to revise this PR to just handle the eslint-plugin-html issue?

The current extract function will not work without eslint-plugin-html because it keeps script tags

Where is this at.  It would be a shame for this to bit-rot further.


:sparkles: WIP: new APILifecycle CRD

Summary
Bare minimum implementation of the APILifecycle controller.
Related issue(s)
Fixes #

@kcp-dev/kcp-maintainers  do we still want to do this?


i do not see the drafts in the CMS

Hi,
First of all thank you for this extension.
I installed it but in the CMS I get only the published records i do not see the drafts in the gridfield any idea on what i'm doing wrong?
Thanks
Rob

Hi Rob, do the draft records show up when you switch the site to viewing draft (tack ?stage=Stage on the end of the URL)? If so, this is a known issue that comes from SilverStripe's Versioned data extension modifying all database queries based on a global "reading mode" state by default.
Depending on the context, the solution differs:

If you have control over the query, you can alter the behaviour of Versioned with dataquery parameters to always get draft and live records:
VersionedFoo::get()
  ->setDataQueryParam(['Versioned.stage' => 'Stage']);

If you can't modify the query, you can use VersionedReadingMode from this module to change and restore the global reading mode:
VersionedReadingMode::setStageReadingMode();

// ... code that runs queries

VersionedReadingMode::restoreOriginalReadingMode();

If you need a model admin to manage versioned dataobjects, you can use VersionedModelAdmin from this module in place of ModelAdmin

Thank you that worked great!, i did not know that the ?stage=Stage worked in the admin.
I used the
VersionedFoo::getList()
->setDataQueryParam(['Versioned.stage' => 'Stage']);
and it works like a charm.
Thank you so much


Modified compiler_public_path to undefined

I suggest to change the default value of compiler_public_path from '/' to undefined. Current value doesn't work when proxy sets prefix.
Current version creates the line below. The src is an absolute path, so it will not work if prefix like /deploy is set by proxy.
<script src="/app.c3e1585f9dcb449f54f3.js"></script>

Proxy expects http:///deploy/app.c3e1585f9dcb449f54f3.js, but the src is  http:///app.c3e1585f9dcb449f54f3.js.
After changing compiler_public_path to undefined, the line becomes a relative path, which works well with proxy prefix.
<script src="app.c3e1585f9dcb449f54f3.js"></script>

Hey there, just woke up; is this related to https://github.com/davezuko/react-redux-starter-kit/pull/371? I still have to re-read both PR's but it seems so.

https://github.com/davezuko/react-redux-starter-kit/pull/370 has been merged; let me know if it doesn't solve the same problem.

https://github.com/davezuko/react-redux-starter-kit/pull/370 has been merged; let me know if it doesn't solve the same problem.

Yes, #370 was designed to fix just that. I didn't consider undefined as a possible initial value as I didn't know if it might lead to some bad side effects. Is there some advantage to that over empty string?

Yes, #370 was designed to fix just that. I didn't consider undefined as a possible initial value as I didn't know if it might lead to some bad side effects. Is there some advantage to that over empty string?

Thanks, #370 fixed my issue. Empty string is perfectly fine and that has the same effect as undefined because webpack internally uses publicPath option as below.
var publicPath = outputOptions.publicPath || "";

https://github.com/webpack/webpack/blob/9f440e30ecac00bfc27b91d372a969d3414d194c/lib/TemplatedPathPlugin.js


Input focus not working cordova-plugin-ionic-webview ***

Related to cordova-plugin-ionic-webview
Hi,
After iOS 12.2 update when I tap into any input field, the cursor (focus) dont appear on the input. No matter I tap into it, I cant start typing.
If I press the down arrow on the acessory bar, it changes between the input elements and focus occurs.
If I remove cordova-plugin-webview from the project, things work normally, without any problem.
Problem seens to be this one: apache/cordova-plugin-wkwebview-engine#71
I tried to install cordova-plugin-wkwebview-inputfocusfix and also added but did not work.
Any clue?
thanks

Hi there,
I am going to close this as this is not an Ionic Framework bug. Additionally, it looks like this bug has been fixed in cordova-plugin-ionic-webview, so you may just need to update the webview plugin.
Thanks!


Fixed next step page stylings

Description
Fixed next step page stylings
Fixes # (issue)
#228
How to test?
npm run dev
Checklist

[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] This PR is ready to be merged and not breaking any other functionality

Try to be consistent with the units you use, you can read more about units in MDN. Overall it is best to use relative units rather than absolute units. Try to keep consistency for the things that serve the same purpose, so fx. distances, widths, font-sizes etc.
The top and left properties and translate one serve a similar purpose, so you are moving the elements in the X and Y direction with both properties, plus in some cases, you add an extra distance with the margin. Do you need to do this?
On line 60 and 61 an image is being modified on width and height, just choose one. It is best practice, otherwise you might end up distorting the image.

Hi Valentina, Thank you so much for your review. What you have mentioned, I also agree with you. If I have to consider all these changes, I have to change most of the next step page CSS. For that, I have to discuss with the person who actually did the whole page. Because Currently I changed only the button styles and gave some responsiveness. Please kindly let me know What should I do now?


Fix deprecated react lifecycle methods

This runs npx react-codemod rename-unsafe-lifecycles to replace unsafe methods

Can a new release of react-redux-form be pushed to npm please?


Updated package list for Konflux

Description
Updated package list for Konflux
Type of change

[ ] Refactor
[ ] New feature
[ ] Bug fix
[ ] CVE fix
[ ] Optimization
[ ] Documentation Update
[ ] Configuration Update
[ ] Bump-up dependent library
[ ] Bump-up library or tool used for development (does not change the final image)
[ ] CI configuration change
[x] Konflux configuration change

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 73.09%. Comparing base (010acff) to head (bdeec75).
Report is 13 commits behind head on main.

Additional details and impacted files

@@             Coverage Diff             @@
##             main    #1961       +/-   ##
===========================================
- Coverage   96.84%   73.09%   -23.76%
===========================================
  Files          69       67        -2
  Lines        2883     2880        -3
===========================================
- Hits         2792     2105      -687
- Misses         91      775      +684

see 38 files with indirect coverage changes

/test unit

/approve
/override "Red Hat Konflux / ols-enterprise-contract / lightspeed-service"

/hold

/lgtm

/unhold


add circuit breaker

Signed-off-by: Yaliang Wu<EMAIL_ADDRESS>Description
Add circuit breaker to protect cluster not overloaded by ML tasks.
Issues Resolved
#141
Check List

[x] New functionality includes testing.

[x] All tests pass

[x] New functionality has been documented.

[x] New functionality has javadoc added

[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Approved with several question/comments. Some other questions:

this would need us to run a load test to exhaust the nodes to test the breakers.
what is the strategy to overcome the problem when the service is over loaded? Does the customer have to scale up cluster to resolve it?
Should we sent memory/CPU/Disks usage metrics to somewhere (Pmet or cloudwatch) to monitor the services? Maybe we just rely on the EC2 metrics.

Prefer to test this part in UT. Will add memory threshold cluster setting to make IT test easier later.
If overloaded, customer can scale up cluster or run less train/predict tasks. Will make MAX_ML_TASK_PER_NODE as cluster setting as well, so user can tune it to run less less tasks per node.
Correct, we just rely on node level metrics.


Improve Python bindings framework

What
Craft a pybind11 wrapper that solves various annoyances.
Why
This was spawned from https://github.com/OpenAssetIO/OpenAssetIO/pull/1144#discussion_r1360801884 and subsequent conversations.
pybind11 has (arguably) some non-sensible defaults and is missing useful functionality:

We shouldn't need our PyRetainingSharedPtr to keep Python objects alive.
.none(false) should be the default for pointer-like types.
GIL release vs. hold should be explicitly defined when binding a method.
Argument names should be enforced to match the C++ argument names.
Binding all overloads of a single function requires a lot of boilerplate.

pybind11 has a memory leak when assigning default values for arguments, in some cases.

Closing due to contention + scope being larger than there is potentially time for in the not too distant


Remove doker env variables

What is being addressed
Some deploy.sh scripts used docker registry environment variable which aren't really needed.
How is this addressed

Remove obsolete variables

/test

/test

/test


Bump controller-runtime to 0.15.2

See
https://github.com/kubernetes-sigs/controller-runtime/releases/tag/v0.15.2 for details.

We only need this in Admiral, the release process will take care of the other projects.


docs: Update README.MD

i almost passed:
{ lazy = ...  } instead of assets = { lazy = ...  } (just for clarity)

Could you please edit the commit message to follow the style of conventional commits

ah yes

5c25186

5c25186

whatever happens,

tis' is the greatest feelin'


refactor typings for readability

This should make the typings more readable for someone just diving into this lib's typings, as we already discussed on twitter :)

@markerikson what's your status on this? From my side this would be ready if you don't have any additional comments.

Okay :) Just wanted to make sure this didn't get buried ^^

Sure, let's do this.


[release/5.0] source-build tarball includes prebuilt nuget packages in arcadeRequired dir

Building a tarball from the 5.0 branch results in a few prebuilts present in the build:
$ tar tf dotnet-9c4e5de-original.tar.gz | grep -E '(\.tar\.gz|\.nupkg|\.zip)$'
dotnet-9c4e5de-original/.dotnet/templates/5.0.0/microsoft.dotnet.test.projecttemplates.<IP_ADDRESS>.2-beta4.20420.1.nupkg
dotnet-9c4e5de-original/.dotnet/templates/5.0.0/microsoft.dotnet.web.itemtemplates.5.0.0.nupkg
dotnet-9c4e5de-original/.dotnet/templates/5.0.0/microsoft.dotnet.web.projecttemplates.<IP_ADDRESS>.0.nupkg
dotnet-9c4e5de-original/.dotnet/templates/5.0.0/microsoft.dotnet.common.projecttemplates.<IP_ADDRESS>.0.nupkg
dotnet-9c4e5de-original/.dotnet/templates/5.0.0/microsoft.dotnet.web.spa.projecttemplates.<IP_ADDRESS>.0.nupkg
dotnet-9c4e5de-original/.dotnet/templates/5.0.0/nunit3.dotnetnew.template.1.8.1.nupkg
dotnet-9c4e5de-original/.dotnet/templates/5.0.0/microsoft.dotnet.common.itemtemplates.5.0.0.nupkg
dotnet-9c4e5de-original/packages/arcadeRequired/nuget.versioning.4.4.0.nupkg
dotnet-9c4e5de-original/packages/arcadeRequired/system.memory.4.5.3.nupkg
dotnet-9c4e5de-original/packages/archive/Private.SourceBuild.ReferencePackages.1.0.0-beta.20607.1.tar.gz
dotnet-9c4e5de-original/packages/archive/Private.SourceBuilt.Artifacts.0.1.0-dev.tar.gz

The SDK  at .dotnet/ makes sense. As does the ReferencePackages and SourceBuilt.Artifacts tarballs.
But what are the arcadeRequired packages? They don't seem to be necessary: if i rm them before building the tarball, the build still works. Can they be removed?

Looks like prebuilt tracking wasn't catching these:
https://github.com/dotnet/source-build/blob/9c4e5dee8af0c4d091b32417d6a20225abc3fe3f/eng/SourceBuild.Tarball.targets#L253-L268
I think that chunk can be removed altogether.

Done by https://github.com/dotnet/source-build/pull/1953. Thanks. 🙂


Is there a way to prioritize by whole file name?

Hello, I really like this extension by the same reasons you created it :), between the alternative it matches my workflow the best by far.
There is a case where I think zf could improve, I set the following file tree to illustrate it. For example, searching for app.py will sort test_app.py first. Is there a way to prioritize by whole file name in that case?
.
├── myownmod
│   └── custom
│       └── app.py
└── tests
    └── test_app.py

or
./myownmod/custom/app.py
./tests/test_app.py

I see two possibilities here:

If the query string matches the filename exactly, then that item is given a score bonus and will be ranked higher in the results

It might be reasonable to add a bonus score always. For example
test_app.py and app.py would both match the query app; however, app.py matches on 50% of chars, and test_app.py only matches on 27% of chars. So both could be given a bonus, but the bonus would be relative to the percent matched. This would be nice so you don't have to type the exact filename each time.

I will experiment with this and report back here. There is always a pretty simple workaround for now, just add "c" to the query string in this example and the correct path will match.

Thank you for a such quick and thoughtful response.

(...)
What happens here is that both strings rank the same, but ./tests/test_app.py is shorter than ./myownmod/custom/app.py so test_app.py sorts first.
(...)
I will experiment with this and report back here.  (...)

I suspected it was something like that. I appreciate the job your are doing.  I don't know much about sorting, and nothing about zig nor performance optimization, but if I could help with testing or something just ping me.

There is always a pretty simple workaround for now, just add "c" to the query string in this example and the correct path will match. So search for "app c"

This is interesting, for sure I'm gonna try this tomorrow on my real env. Thank you!

@alefpereira sorry for the wait! I implemented both 1 and 2. You said you would be willing to test it, how is the easiest for you? If you have zig and can compile zf, just switch to the feat-3-name-coverage-priority branch for testing. Otherwise I can compile libzf for you to test with the telescope extension.

@alefpereira I have included this fix in the latest updates to https://github.com/natecraddock/telescope-zf-native.nvim. Let me know if that helped!


Update Microsoft.OneDriveInsiders to 21.205.1003.0002

null
Microsoft Reviewers: Open in CodeFlow

cc @OfficialEsco @jedieaston: you missed this one.


"Page Not Found" error page generated when it is not justifed

Describe the bug
When I navigate through the generated Web site (dumped from my vault), and I hit a "back" button on my browser, I get a "Page Not Found" error page instead of the generated HTML file. Unfortunately, it does not happen all the time (it would be too easy if it did... 😀
Operating System
MacOS 14.2.1 as well as IOS 17.2.1.
I opened the generated Web site on Chromium based browser (in my case Arc), on Firefox, as well as Safari. For Safari that was both on the Mac an on the iPhone. The behaviour is identical.
In all cases I see the html file coming up on the screen and then the screen is 'exchanged' to the error page. I tried to chase down the bug and got to the point that, in the lib/scripts/webpage.js file, where I found the trace of this generated page lined to a timeout function.
To make what follows clearer: my setup is that I dump my vault to a directory on my machine, which I then reach via a server running on it and serving the site on localhost:8001/LocalData/Obsidian.
Looking at the console.log output in the developer's console of chromium (see the screen dump below), what I see is the message:
Page not found: /LocalData/Obsidian/index.html

which refers to the name of the file (which exists) and which I used to navigate from. However, the line above shows:
GET http://localhost:8001/LocalData/Obsidian/Obsidian/index.html webpage.js:1
404 (Not found)

where the URL for the GET is indeed wrong. Ie, the script seems to want to generate a fetch with a URL that (I presume) is constructed, but the construction is buggy.
That is as far as I got... maybe it helps locating the bug.
Here is the screen dump:

To Reproduce
I pretty much described what I did to get to the error, see also the screen dump above.

Hi thanks, this is a duplicate of issue #295. I have already fixed this in development and it will be fixed in the next release :)

Oh, rubbish, I forgot the basic steps in these circumstances, namely check the other issues. Basic github behaviour... My apologies.
I am happy it is fixed already, though.
Thanks for your work, @KosmosisDire.


Handshake failed: no matching client->server cipher

ssh client version: 1.7.1
vscode version: 1.48.2
Windows version:  windows 10 Enterprise LTSC
Ssh client cannot connect to remote server. Vscode reports "Handshake failed: no matching client->server cipher" error.

Maybe many problems can lead to this error. You probably solved some of them.
I updated ssh client's version to 1.7.6, but this error still remains

Since 1.7.7, Support special connect cipher.

Since 1.7.7, Support special connect cipher.

可以用了，谢谢。niubility

可以用了，谢谢。niubility


arm64 plugin

please update arm64 plugin

This now pertains to airpeer. https://www.github.com/adrenak/airpeer


Track RX/TX bytes in the Libreswan cable driver

Signed-off-by: Stephen Kitt<EMAIL_ADDRESS>

@mangelajo could you re-review this? I’ve tried to address your comments.

@mangelajo could you re-review this? I’ve tried to address your comments.

@mangelajo ping?


The alignments of the buttons are not correct

Describe the bug
In reports section, click on any reports. The buttons seems to be out of place on the left hand side.
To Reproduce
Steps to reproduce the behavior:

Go to 'Menu'
Click on 'Reports->Route'
Check the Show,export and email report buttons.

Expected behavior
The whole group of buttons is not inside/outside the div that contains device and period.
Screenshots

Additional context
This problem is there in all the reports.

It's already fixed.

When it was fixed. Why I don't see it. This package was installed 2 days back only.

Are you building from master?

We downloaded and self hosted. Didn't build from master