SegmentedButton Not Vertically Aligned Alongside Other Controls

URL (minimal example if possible): https://jsbin.com/cejocaz/edit?output
What is the expected result?

What happens instead?

Any other information? (attach screenshot if possible)
When placed alongside other user input controls, only the SegmentedButton looks vertically misaligned. It seems to be missing some top and bottom paddings (0.25rem) in contrast to the adjacent controls.

@boghyon Thank you for reporting the issue. It is logged for internal check with ticket #1780488929
Updates will be posted here for anyone interested
BR


Typed property error on 'id' when using alias instead of identifier

Detailed description
I encoutered this bug when creating a new TemplatedMailMessage and using the 'alias' directive instead of using an identifier. My code looks like this:
$message = (new TemplatedMailMessage())
            ->from(
                config('mail.from.address'),
                config('mail.from.name'))
            ->alias($this->templateId)
            ->include($this->templateModel($notifiable));

In the 2.x versions of your package this worked. But in the 3.x version it throws this error:
Error: Typed property CraigPaul\Mail\TemplatedMailMessage::$id must not be accessed before initialization in /.../vendor/coconutcraig/laravel-postmark/src/TemplatedMailMessage.php:27
However, when adding the "identifier" with value '0' it does work:
$message = (new TemplatedMailMessage())
            ->from(
                config('mail.from.address'),
                config('mail.from.name'))
            ->identifier(0)
            ->alias($this->templateId)
            ->include($this->templateModel($notifiable));

Possible implementation
Change the id property on the TemplatedMailMessage class to this:
protected ?int $id = null;
Your environment

Laravel Postmark Version: 3.0.1
Laravel Version: 9.1.0
PHP Version: 8.0

Hey @leeuweriq,
Thanks for catching this. I'll have a fix out in just a little bit


MWPW-135361 Gen Cache Update

Resolves : MWPW-135361

Codecov Report

Merging #342 (8ecf167) into stage (b111a60) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##            stage     #342   +/-   ##
=======================================
  Coverage   94.22%   94.22%
=======================================
  Files          26       26
  Lines        2649     2650    +1
=======================================
+ Hits         2496     2497    +1
  Misses        153      153

Files Changed
Coverage Δ

.../blocks/dc-converter-widget/dc-converter-widget.js
99.01% <100.00%> (+<0.01%)
:arrow_up:

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


Fix MRU sort order after loading vim session

Loading a session which includes hidden or unloaded buffers breaks the
MRUList sort order in BufExplorer, because only the visible buffers are
added to the MRUList and the MRUCmp() function doesn't deal with buffer
numbers not found in the MRUList, which causes visible buffers included
in the MRUList to be listed after other buffers not yet in the MRUList.
Fix this by simply resetting the MRUList after loading a session so all
listed buffers are included.
To allow for wiping out buffers when switching between sessions, which
is done by sessionman.vim and possibly other plugins and scripts, also
filter MRUList when resetting to exclude buffers that no longer exist.

Thanks for merging 👍


Monitor.Models.LogToMetricAction not available in nuget Package Microsoft.Azure.Management.Monitor 0.20.1-preview

The issue is related to the Swagger spec for ScheduledQueryRules API
https://github.com/Azure/azure-rest-api-specs/blob/master/specification/monitor/resource-manager/microsoft.insights/stable/2018-04-16/scheduledQueryRule_API.json
One of the class "LogToMetricAction" does not exist in the 0.20.1-preview version of Nuget package Microsoft.Azure.Management.Monitor. The LogToMetricAction.cs should have got generated https://github.com/Azure/azure-sdk-for-net/tree/psSdkJson6/src/SDKs/Monitor/Management.Monitor/Generated/Models
Locally building using generate.cmd generates this class.

Thank you for reporting the issue. Azure Monitor Alerts team is investigating the report and we'll revert back to you shortly.

@snehithm - can you please assign this to @abver

SDK is automatically generated from swagger spec. This needs to be investigated from SDK team. @msvijayn please assign accordingly.

@snehithm - can you please reassign this to Azure SDK team based on comment from @abver .
I believe owners for Azure .NET SDK are @shahabhijeet and @kurtzeborn

@shahabhijeet - assigning the issue to you as indicated by @abver the issue is apparently emanating by SDK team.

Closing this out as this refers to a legacy version of the library and there has been no recent engagement.  Please feel free to reopen or file a new issue if this remains a problem with the current version of the library.


new CLI for triggers create/delete/update/list and new IT tests

Issue Ref: [Issue number related to this PR or None]
Description:
[PR Description]
TODOs:

[x] Ready to review
[x] Automated Tests
[x] Docs

Main Trigger sub-command
kubeless trigger --help
trigger command allows user to create, list, update, delete triggers running on Kubeless

Usage:
  kubeless trigger SUBCOMMAND [flags]
  kubeless trigger [command]

Available Commands:
  cronjob     cronjob trigger specific operations
  http        http trigger specific operations
  kafka       kafka trigger specific operations

Flags:
  -h, --help   help for trigger

Use "kubeless trigger [command] --help" for more information about a command.

cronjob  sub-command
kubeless trigger cronjob --help
cronjob trigger command allows user to create, list, update, delete cronjob triggers running on Kubeless

Usage:
  kubeless trigger cronjob SUBCOMMAND [flags]
  kubeless trigger cronjob [command]

Available Commands:
  create      Create a cron job trigger
  delete      delete a cronjob trigger from Kubeless
  list        list all Cronjob triggers deployed to Kubeless
  update      Update a cron job trigger

Flags:
  -h, --help   help for cronjob

Use "kubeless trigger cronjob [command] --help" for more information about a command.

http  sub-command
kubeless trigger http --help
http trigger command allows user to create, list, update, delete http triggers running on Kubeless

Usage:
  kubeless trigger http SUBCOMMAND [flags]
  kubeless trigger http [command]

Available Commands:
  create      Create a http trigger
  delete      Delete a HTTP trigger
  list        list all HTTP triggers deployed to Kubeless
  update      Update a http trigger

Flags:
  -h, --help   help for http

Use "kubeless trigger http [command] --help" for more information about a command.

kafka  sub-command
 kubeless trigger kafka --help
kafka trigger command allows user to create, list, update, delete kafka triggers running on Kubeless

Usage:
  kubeless trigger kafka SUBCOMMAND [flags]
  kubeless trigger kafka [command]

Available Commands:
  create      Create a Kafka trigger
  delete      Delete a Kafka trigger
  list        list all Kafka triggers deployed to Kubeless
  update      Update a Kafka trigger

Flags:
  -h, --help   help for kafka

Use "kubeless trigger kafka [command] --help" for more information about a command.

I will also try consoldate kubeless route command into kubeless trigger http also add support for path added into http trigger spec.

Merging this PR now.


Fix component ssr

With this fix, I can use component approach with ssr, no need for directive approach.
Before this fix even dist/ssr.js failed to me because of window.
I had not realized why there is window.swiper, so I just guarded window usage with simple typeof window check, but looks like it should work even if we remove window.Swiper entirely and just make const Swiper = _Swiper ?
Tests failed because of prettier 1.13.0, which is dependency of vue-loader, I can add prettier 1.12.0 to package.json if u want, so test will work
Last test "Multi edirot directive instance" is failing, but it was failing before my change too

https://github.com/NightCatSama/vue-slider-component/issues/114#issuecomment-353827607
checkout this dfeinberg5 did the real magic, it saved my day after I had tried everything.

Would be great to get this merged. The current SSR situation is that the plugin doesn't support SSR (it just turns it off)

Mergeeeeeeeeeeeeeeeee

Running into the same issue, decided to change the library.

Looks like this project needs a new maintainer. I've since switched to Svelte + Sapper instead of Vue + Nuxt, but I use https://glidejs.com/ which should be fairly easy to integrate into a Nuxt app too. Give it a go if you're stuck.

Still no update?

I had the same problem, but solved with mode: 'client' option. Nothing changed with example code for ssr but just add it code below.
plugins: [{ src: '~/plugins/swiper', mode: 'client' }],

then everything works fine.
Check it the official document. https://nuxtjs.org/guide/plugins

@Seonghui yes - making the component client only always works, but this is different to creating an SSR compatible component.

@antony I was able to fix SSR issue without changes to the library (although this MR works perfect):
new webpack.ProvidePlugin({
    'window.Swiper': 'swiper/dist/js/swiper.js',
}),

But I am not 100% sure how that plays together with bundle splitting and effective bundle size minimizing. I guess it just adds the swiper everywhere.

@Oleg-Arkhipov how exactly did you implement this without changing package files?
I added the below in nuxt config but when I enable ssr in plugin I still get window is not defined:
import webpack from 'webpack'
module.exports = {
    build: {
        plugins: [
            new webpack.ProvidePlugin({
                'window.Swiper': 'swiper/dist/js/swiper.js'

@gianniskarmas yeah I think that's equivalent to just including the script tag in your <head>.

@gianniskarmas I added only ProvidePlugin, nothing more. It makes window.Swiper defined and accessible. But I do not use Nuxt, so your problem probably has something to do with it.

@antony if that becomes a problem anyone willing to use this library can create a fork with this MR merged and use it in their projects until @surmon-china fixes it.

If you don't use SSR none of this is a problem anyway. I would suggest that anybody wanting a more permanent fix forks and maintains this library as it appears to be abandoned.

@antony I use SSR and love my images to be in <img> tags in initial response, instead of emptiness. Too bad @surmon-china abandoned merging that when his library otherwise works really well, just small  fix required.


Update grid.js

Double arrow functions for easier reading

Hey, thanks for your interest. It looks like you still need to learn Vue and arrow functions a bit more 😛 . The arrows functions are binding the context, therefore the this changes. This is important for computed properties. Also, with arrows function you can omit the return in one-liners:
(param) => param.toUpperCase()
// equivalent to
(param) => {
  return param.toUpperCase()
}

About the examples, they're kept simple on purpose, to make it easy to understand them for any js developer, even if they still don't know about arrow functions. Cheers

Oops... Sorry


fix(safari): config export displays json in browser

Fix #652
The trick is to use a nonsensical mime type to keep Safari from rendering it.

:tada: This PR is included in version 6.40.2 :tada:
The release is available on:

GitHub release
v6.40.2

Your semantic-release bot :package::rocket:


Is SLIP header standard ? Not defined in RFC 1055

Thank you for your work on this lib.
I'm surprised to see the header added at the beginning of the frame. For me, the frame starts only with an END character followed by the data (see https://tools.ietf.org/html/rfc1055).
Am I missing something?

With this patch, lib is conform with RFC 1055

Hey @adetante, thanks for opening this issue and making a patch!
You're totally right that the header is incorrect currently and your fix looks good to me.
Mind opening a PR so I can merge your patch?

Thanks @mvertescher for your quick answer!
Here is the PIR https://github.com/mvertescher/serial-line-ip-rs/pull/6


Back to results doesn't work in staging + prod

What
See Slack convo
"Back to results" on work pages always displays as "Back to search" (without the previously selected filters/pagination/any query element) as it can't detect a link in the Search Context.
It works locally, so I wonder if it has to do with redirections?
Where

https://wellcomecollection.org/search/works
Enter keyword, change page, add filters
Click on a work item
Observe "Back to search" hasn't retained any filtering, sorting or keywords and defaults to the main search page.

@AMBacelar as I'm off next week I won't be able to keep an eye on this. When it's been deployed please let Jenn know it's been fixed.

I don't think this is redirection related – all the redirection logic lives in CloudFront, and you can rule out CloudFront as a culprit by making requests directly to the EC2 load balancer at http://prod-1800364202.eu-west-1.elb.amazonaws.com/search/works?query=fish
graph LR
    C["CloudFront (inc. redirections)"] --> ALB[EC2 Load Balancer]
    ALB --> ECS["ECS service (aka app running in Docker)"]

That's still dropping the search context, so the issue is happening pre-CloudFront/pre-redirections.

Fixed!


Wizard redesign - Checkboxes and radio buttons with optional description and price (without design update)

winning submission from challenge 30083737 - Topcoder Connect - Wizard redesign - Controls new design

fix to format prices with comma

This PR implements a part of form redesign which is now in https://github.com/appirio-tech/connect-app/tree/feature/form-redesign
After this PR is merged, we have to update dependency in connect app branch feature/form-redesign, to point back to appirio-tech/react-components#feature/connectv2 instead of maxceem/react-components#feature/connectv2-form-redesign

@maxceem can we merge this PR safely without affecting behaviour of current components? I mean if we merge it today and do a production deploy afterwards, would it cause any troubles with the way the form elements are rendered currently on production?

@vikasrohit We only added new features but didn't break any existent features so this PR should be safe.
But if we are not going to merge feature/form-redesign to dev and master today, we can skip merging this PR today. And merge it after release. Even though this PR should be safe, it would be better if we test it for some time on dev if possible.

Okay. I would merge it after today's release. Thanks for the update.

merging the PR @maxceem @RishiRajSahu


special issue for 65535

From here
Hi 65535

First of all, you're claiming 5 to 7 mb of data. What are the files or directories. Some files it might be "cleaning" (by removing them) might already have been vacuumed clean, eg sqlite files. Or it could be detecting extension data. CCleaner does detect SiteSecurityServiceState.txt (it lists the items as cookies) - just get a portable version, so you can throw it away afterwards (portableapps.com), because since Avast took it over it's a bit of a PoS (piece of s**t). When you run CCleaner (and you could use it to just clean Firefox in these test runs), then you can click on details afterwards, and it will list the files and sizes. This is the sort of information that is needed to work out what this 5-7 Mbs is made of.
SiteSecurityServiceState.txt - just edit it empty, then deny permissions on it. HSTS is not that big a deal IMO. You have a preload built in of around 50K domains, and HTTPS Everywhere adds another 125K. And the web is increasingly moving to HTTPS, but still .. the default in browsers is HTTP because of backwards compatibility - here's a little read for you: https://www.eff.org/deeplinks/2018/12/how-https-everywhere-keeps-protecting-users-increasingly-encrypted-web
But that file is a few Kbs, not 5 to 7 Mbs.
Safe Browsing used to use google cookies, stored in a separate jar, that you can't touch within Firefox. I've never had one (and yes I do use SB, just not the real time binary checks) so I can't vouch if any cleaner sees them. But SB v4 doesn't even use cookies any more: I have tested this and also been told directly by this guy who built it: https://feeding.cloud.geek.nz/posts/how-safe-browsing-works-in-firefox/ (article is 99% up to date AFAIK). So if you have a SB cookie in a SB cookie jar, and you get rid of it, then you won't get any back.
But a cookie is minuscule. Getting back to persistent local data. Firefox does clear everything when you tell it to. I've tested it all. But of course, and I'm struggling here to think of anything other than HSTS, not everything is listed in the UI.
Note: there is difference between Ctrl-Shift-Del (clear history panel) and On Close. In the clear history panel, you can choose a time range, or "everything" which means no time range. IDB cannot be cleared by time range. So always use "everything" when doing a manual clean (as compared to when you quit Firefox).

"Cookies" clears cookies.
"Browsing and Download History" clears what it says, and also clears session restore (all files in your /sessionstore-backups/ directory, including previous and temp upgrade ones), it does this even if you have Firefox set to restore pages on startup (because hello, SR contains your history including some "undo close" tab and window history). Session restore is used in session and removed on close if you do NOT have Firefox set to restore pages on start (but only removed the recovery file, not previous etc ones).
"Offline Website Data" clears appCache, localStorage, Service Worker cache, and QuotaManager (IndexedDB, asm-cache)
"Form & Search History" clears what it says - formhistory.sqlite
"Cache" clears disk cache, which I don't use. Memory cache would be released back to the OS.
"Active logins" is not what it sounds like. It is not your logins to websites or passwords etc, it refers to FTPs etc - https://en.wikipedia.org/wiki/Basic_access_authentication
"Site Preferences" clear all your Firefox exceptions (site permissions such as allow notifications/geo/camera etc, zoom settings, cookies preferences such as allow for session/block if they differ from your default) - permissions.sqlite

some locations where WEB data is stored

cookies : profile\cookies.sqlite
localStorage : profile\webappsstore.sqlite
indexedDB : profile\storage\default
appCache : profile\OfflineCache

Look, there's a lot more to it, and Next Gen Local Storage has landed in Nightly, and for the last several releases there has been a new "Cookies & Site Data" interface in Options. Cookies actually control access to localStorage and IDB (you block a cookie, and the site cannot use that persistent storage). And if you delete a cookie it should delete related persistent storage. Stay away from from cookie extensions - they cannot clear your IDB entries by host, and you will end up with ORPHANED data, and Firefox might not know they're there to clear (not 100% sure on this, its not something I need to test). On Close should use a different mechanism because it doesn't care about selective sanitizing. You can disable appCache (Firfoxe blocks it on HTTPS, and it's basically deprecated in the wild now). And you can control all web workers using uMatrix - that is default deny, whitelist - then clear them on close.

Also, what the heck is up with the FF51 fixation. There was a bug that Firefox didn't clear IDB on close, that was fixed ages ago - maybe it was in FF52 it got fixed.
I will assume you are using ESR60 or FF64+. While I can answer some questions, I'm not going to troubleshoot an outdated version.

Privacy Badger focuses on 3rd party trackers only. On PB Repo you find the legitimate google.ad & google.ae in these files: multiDomainFirstParties.js and manifest.json
PB has a big list of 1st party scripts that are whitelisted. By searching for google.ad or google.ae on bgp.he.net you get a list of the systems that may respond to those URL queries (just an example of potential multiplexing by big data centers).
Also see: https://github.com/EFForg/privacybadger/issues/222

If 65535 turns up and needs more answers, I'll re-open


⚠ ClamAV has degraded performance

In ca7e9ed, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 69 ms

Resolved: ClamAV performance has improved in 7b2aa13 after 5 minutes.


Programmable filters

I've always wished for there to be a lag-free way to interact with item Vaults, extracting specific items you might need and what not. Currently there's no easy way to automate getting a specific item out of a vault.
I think, a good additional would be a new filter printer block.
1: Place the "Filter Printer" facing towards any create block that can have a filter applied to it
2: Right click on the filter printer with any source that can return a list of items
3: Attach display link to the CC:C source block
4: Whenever the "Filter Printer" gets updated with a new list, it plays a short animation and edits the filter applied to the block it's pointing towards.
Another way this could be done is via a remote "smart filter" and "smart filter router" that each have channels assigned to it (similarly to redstone links)
Smart filter would be the sturdy sheet version of the filter, while the smart filter router would be another CC peripheral block. It can be interacted with by the player to assign it's channel (similarly to redstone links). Via it's peripheral code you could then assign the filter properties. (Allow/deny list, attribute list etc.)
While this solution would be more elegant, it's also very likely Create simply just doesn't allow this to be done.
Finally, the last way i believe this could be done is by adding a new sturdy sheet funnel/chute. Just like the smart filter idea, they'd connect to their own alternative of "smart filter router". Set the funnel/chute channel, do the same for the router and via a peripheral control what the funnels/chutes should let in/out!

Hm. I think I really gotta have to take a look at Create filters. This is something that I heard people wanting us to implement over and over again.
I don't want to over complicate things with block-chains of peripheral blocks that eventually give you an in/output.
I like the idea of our own Smart filter or funnel/chute block. Not sure how robust the filter item would be to configure within any way (like linking to a peripheral), so ideally I imagine a funnel/chute that acts as a peripheral too, sending events for items that want to pass over and a functions to tell it what to do with it. It would be even better if it can act on its own after configuring it, but not sure how to do this right now.
That all is just brainstorming me.
I will keep this issue open for now. I don't know when I will take some time to take a closer look at this.
Thanks for contributing here with your idea however, I really appreciate it. ^^


Client blinks

When cursor moves fast like on constant holding down key I see how xi-term blinks, like redrawing or something. Checked in Alacritty and xfce4-terminal. Is it a config issue or a bug?

It is a bug. It because currently we are rendering every line in the view on every frame

@bytebuddha, there is a fix in #90, could you take a look?

Hi folks,
#90 doesn't fix this problem. It only presents an optimization that reduces stress on buffer writing.
The bug is still present (my case: xterm on linux).
A bit of research: The screen always flickers when the user makes a few force-redraws in rapid succession (eg when the cursor is moved down with a down arrow key). To make this flickering happen, you don't even need to run xi-term. Consider the following simple console application which prints out the file and redraws everything when the user presses any key:
use std::env;
use std::fs;
use std::io::{Read,Write};
use termios::*;
fn main() {
    let args: Vec<String> = env::args().collect();

    if args.len() != 2 {
        eprintln!("Usage: {} FILENAME", args[0]);
        return;
    }

    let filename = &args[1];
    let contents = fs::read_to_string(filename).unwrap();

    // disable terminal buffering so that keypress is read() immediately
    let mut termios_info = Termios::from_fd(0).unwrap();
    tcgetattr(0, &mut termios_info);
    termios_info.c_lflag &= !ICANON;
    termios_info.c_cc[VMIN] = 1;
    termios_info.c_cc[VTIME] = 0;
    tcsetattr(0, TCSANOW, &termios_info);

    let stdin = std::io::stdin();
    let mut handle = stdin.lock();
    let key = 0u8;

    while true {
        // any key = redraw
        // Ctrl+C = exit
        handle.read(&mut[key]);

        write!(std::io::stdout(), "{}", contents);
    }
}

Hit or hold down any key and observe the screen flickering.
Suggestion: Reopen the bug; Write some code that creates a simple tracker of the start and end of the currently rendered LineCache chunk. If an event does not take us out of this chunk, redraw only the cursor.

@fominok can you please comment on this? The blinking is definitely present.
I confirm that this is happening due to frequent re-rendering of all window lines. I wrote a view.rs hack that skips render_lines() if linecache didn't change and that fixed the blinking. It's not a proper fix; I was using a full backup of the visible lines and then a linewise comparison between the backup and the current linecache slice, which is highly ineffective. Currently the LineCache is simply not able of notifying its users of any atomic change in its contents, so my opinion is that we need a "RenderCache" instead of "LineCache". This "RenderCache" should be aware both of the incoming vector of ops from xi-core and of the rendering, and call render_lines only for the modified lines.
Having that said, also pls consider xi-editor/xi-editor#1228: it seems that xi-core even cannot properly notify the client of a cursor move that does not modify the lines.
Side question: Is anyone ever using xi-core on a daily basis? I'm pretty much surprised that no one is complaining on blinking or on crashing on files as small as 65kb (#112), given the fact that those bugs hit you in the face as soon as you start editing real files with xi-term.

As you rightly said, my fix was an optimization to move from "annoying" to "ok for now" state of blinking.
Personally I'm waiting for Xi SDK to reuse its components for standalone editors as the protocol cannot fit anything and something like modal editing needs to be implemented on the client-side, or there is a need to introduce something like middleware. Answering your last question: this prevents me from using xi-term.
Also, I think you have experience with this and you can propose the right changes to Xi-core.

Have a reply from the xi-core devs, see the talk on the xi-core issue page.
Bottom line: this may become a todo item for xi-core in future, but for the time being, xrl / xi-term would better rely on the protocol as it is.
That said, this problem can be solved on the client (xrl::LineCache side) by introducing a line-level cache invalidation (= a "dirty" bit attached to every cache line), so that the cache could announce that only the given (few) lines were changed and need a redraw.
My understanding is that there is an ongoing work on turning xrl::LineCache into a hash map so this "cache invalidation" objective should better be put on hold until this hash map thing is complete.


Appear files in an editor

Add feature to open files in a Tmux-based IDE. Pretty jank right now, but this branch can make it so double-clicked code opens in a Vim in a Tmux, with enough system setup.
Demo:

Jank setup instructions:

install neovim
install python plugin support for Neovim.
brew install python3
pip3 install neovim neovim-remote
maybe, unsure if needed: run nvim and call :UpdateRemotePlugins
check out this branch, run bundle install then run bundle exec rake install to install the alpha version. You might have to do some shenanigans to make /usr/local/bin/appear --version return 2.0.0.a
put this in your zshrc/bashrc, so that each nvim you spawn has a unique socket (or, just about): export NVIM_LISTEN_ADDRESS="$HOME/.vim/sockets/vim-zsh-$$.sock"
open tools/unix-dropper.applescript with Script Editor, then Export it as an application.  Like this:
finally: set files to open with unix-dropper.app!
If you need to customize the path to the Appear executable, or want to install this alpha version in a different way, just double-click on unix-dropper.app and adjust the script as necessary:

@reissbaker this PR is extensive: I basically rewrote the Tmux library and way Mac revealers are handled. I'd like to wait for a review, but at 1200 lines of (sometimes gonzo) code this is a huge chunk.
The classes that I'm actually interested in using in Airlab are all the Appear::Util classes: https://github.com/airbnb/appear/tree/jake--appear-editor/lib/appear/util
Specifically CommandBuilder and Memoizer


Game Crash on World Load

Game Crash on World Load
Whenever I load a world in a dev environment while working on my mod (it uses the V3 API), I recieve the following crash: https://controlc.com/ee04be1d

Reproduction Steps
Unfortunately, as my mod is not production ready as of yet, I am not ready to share the source code. I don't know what causes the crash

System Info
OS: Bedrock Linux
CC Version:  2.5.2
Modules in use: Base, Chunk
MC Version: 1.16.2
All other details can be extracted from the crash report

Oh wow that's a dumb bug. Thanks for reporting.


Cannot uninstall 'Debugger for Java' extension. 'Test Runner for Java' extension depends on this.

Internal: Cannot uninstall 'Debugger for Java' extension. 'Test Runner for Java' extension depends on this.
at e.$kp.J in src/vs/platform/extensionManagement/common/abstractExtensionManagementService.ts:606:62

Go to Errors Site

@sandy081 we should handle this properly in the UI, if it is already then the error shouldn't be getting sent to telemetry as it's expected.

One of the most frequent errors:
https://errors.code.visualstudio.com/card?ch=74f6148eb9ea00507ec113ec51c489d6ffb4b771&bH=523f415a-5fd5-f1e1-6304-65a44d9a672d
https://errors.code.visualstudio.com/card?ch=74f6148eb9ea00507ec113ec51c489d6ffb4b771&bH=59515349-e646-a122-6d56-6b24347b5b33

I use special Error for extension management errors and is sent across processes.
https://github.com/microsoft/vscode/blob/eb24c3f99c45bae0e5f4773b3e90a3c63a78940d/src/vs/platform/extensionManagement/common/extensionManagement.ts#L422
I cannot change this to ErrorNoTelemetry as it might break the users of this error. I expect all callers should be handling this error and do not understand who is calling this from the given stack.
Hence closing this.


[Localization] Wrong title for confirm button in Spanish

System Information (Required)

SDK Language: NODE
SDK Version: 3.8.3
Development Environment: OTHER

Issue Description
Prompt.confirm with listStyle BotBuilder.ListStyle.button is returning through directline v3 buttons with titles "si" and "no". It should be "sí" with an accent in the letter i.
Code Example
    let options: BotBuilder.IPromptOptions = {
        listStyle: BotBuilder.ListStyle.button,
        maxRetries: 3
    };
    BotBuilder.Prompts.confirm(session, messageText, options);

Expected Behavior
I would expect that this would result in an activity with the buttons 'sí' and 'no'
...
 "inputHint": "expectingInput",
      "attachments": [
        {
          "contentType": "application/vnd.microsoft.card.hero",
          "content": {
            "buttons": [
              {
                "type": "imBack",
                "title": "sí",
                "value": "sí"
              },
              {
                "type": "imBack",
                "title": "no",
                "value": "no"
              }
            ]
          }
        }
      ],
...

Actual Results
The actual response contains the button with 'si' without accent (which is incorrect in Spanish).
 "inputHint": "expectingInput",
      "attachments": [
        {
          "contentType": "application/vnd.microsoft.card.hero",
          "content": {
            "buttons": [
              {
                "type": "imBack",
                "title": "si",
                "value": "si"
              },
              {
                "type": "imBack",
                "title": "no",
                "value": "no"
              }
            ]
          }
        }
      ],

Same issue here, it would be nice if the framework would recognize several variations of yes: yep, yeah, y, and in Spanish too: sí, si, sip, s.

@vianasw can you post your code for the Hero Card?

Upon further review, it looks like there is a typo in the Spanish localization default phrases where "sí" is listed as "si" (incorrect).
I'll put together a PR with a fix.

Follow progress in #2967


Chore/backport 329 330 331

关联的Issues (Related Issues)

close
改动 (Proposed Changes)

附注 (Additional Info)

自查清单 (Checklist)
在你认为本 PR 满足被审阅的标准之前，需要确保 / Before you mark the PR ready for review, please make sure that:

[ ] 符合Venus项目管理规范中关于PR的相关标准 / The PR follows the PR standards set out in the Venus project management guidelines
[ ] 具有清晰明确的commit message / All commits have a clear commit message.
[ ] 包含相关的的测试用例或者不需要新增测试用例 / This PR has tests for new functionality or change in behaviour or not need to add new tests.
[ ] 包含相关的的指南以及文档或者不需要新增文档 / This PR has updated usage guidelines and documentation or not need
[ ] 通过必要的检查项 / All checks are green

Codecov Report

:exclamation: No coverage uploaded for pull request base (release/v1.10@61d6c29). Click here to learn what that means.
The diff coverage is n/a.

Additional details and impacted files

@@               Coverage Diff                @@
##             release/v1.10     #332   +/-   ##
================================================
  Coverage                 ?   54.54%
================================================
  Files                    ?       69
  Lines                    ?     7229
  Branches                 ?        0
================================================
  Hits                     ?     3943
  Misses                   ?     2961
  Partials                 ?      325

Flag
Coverage Δ

unittests
54.54% <0.00%> (?)

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


filetransfers: FI's require FileHeader modification on upload

What were you trying to do?
There's an FI that seems to require setting ImmediateOrigin to<PHONE_NUMBER> prior to upload otherwise the file will be rejected. The problem is paygate needs this value to be properly set so it can merge transfers until it uploads each file.
What did you expect to see?
Uploads should "just work" with every ODFI.

Solved with the Gateway config.


Security analyse to implement recommendations

3-rd party security tests for Android wallet

Please after the task Adjust program Proguard. #138  will be completed

Result:
According to the report, application security is rated as «High», because no extremely dangerous vulnerabilities in sources and application behavior was found.
There are some recommendations to implement

Do not disable logs - Approved by Product


question and add features

add that you can set multiple languages (translations please do them with " deepl" not google translator which sucks).
if I download the "everything" tool from the script is it installed without bloatware, without trelemetry , with marketing purpose disabled and then just the program without things that can compromise privacy or anything else?

I'll see about adding languages in the future, but I want to get it to a less active development state before doing so.
As far as everything this is installed with default settings from winget. I'd assume telemetry is enabled.


NIFI-6905 Prevent non-primary nodes to connect Twitter API when runni…

…ng in primary node only mode.
Description of PR
GetTwitter processor, configured to run on primary node only, initializes connection to Twitter API from every NiFi cluster node, even on non-primary nodes.
Prevent non-primary nodes to connect Twitter API when running in primary node only mode. This Fixes bug NIFI-6905:
I updated the GetTwitter processor onScheduled() method to only create a clientBuilder without connecting it to the Twitter API.
Connection is now initialized by the onTrigger() method when it needs it (in primary node only mode onTrigger() never run on non primary nodes....).
Added onPrimaryNodeChange() to close connection on PRIMARY_NODE_REVOKED events.
For all changes:

[X] Is there a JIRA ticket associated with this PR? Is it referenced
in the commit message?

[X] Does your PR title start with NIFI-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.

[X] Has your PR been rebased against the latest commit within the target branch (typically master)?

[X] Is your initial contribution a single, squashed commit? Additional commits in response to PR reviewer feedback should be made on this branch and pushed to allow change tracking. Do not squash or use --force when pushing to allow for clean monitoring of changes.

For code changes:

[ ] Have you ensured that the full suite of tests is executed via mvn -Pcontrib-check clean install at the root nifi folder?
[ ] Have you written or updated unit tests to verify your changes?
[X] Have you verified that the full build is successful on both JDK 8 and JDK 11?
[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[ ] If applicable, have you updated the LICENSE file, including the main LICENSE file under nifi-assembly?
[ ] If applicable, have you updated the NOTICE file, including the main NOTICE file found under nifi-assembly?
[ ] If adding new Properties, have you added .displayName in addition to .name (programmatic access) for each of the new properties?

For documentation related changes:

[ ] Have you ensured that format looks appropriate for the output in which it is rendered?

Note:
Please ensure that once the PR is submitted, you check travis-ci for build issues and submit an update to your PR as soon as possible.

Thanks @kourge-ch ! Was able to verify that everything is working as expected and code review looks good. Thanks for fixing that! +1 merged to master.


Create a new PJRT API for registering custom calls and use that instead of xla_client.register_custom_call_target

In order to run JAX programs with custom calls on other compiler/runtime backends (eg. OpenXLA/IREE), the custom call implementations have to be registered with the backend. PJRT is meant to be the shim-layer between frontends like JAX and backends like XLA and OpenXLA. Right now custom calls are registered only on the XLA backend using xla_client.register_custom_call_target. It would be ideal to create a new register_custom_call_target in the PJRT API and allow the PJRT clients (such as https://github.com/openxla/openxla-pjrt-plugin) to implement the registration. JAX calls pjrt.register_custom_call_target to register the custom call implementations.

Indeed.
Currently JAX has three in-tree backends, and on each the behavior is:

CPU, where a registration is a (name, function pointer) pair, where the function pointer is a CPU kernel per https://www.tensorflow.org/xla/custom_call
GPU, where a registration is a (name, function pointer) pair, where the function pointer is to a CPU function that launches GPU kernels per https://www.tensorflow.org/xla/custom_call
TPU, where you can't register anything yourself at the moment.

However, I note that XLA:current also recently grew a richer FFI interface, courtesy of @ezhulenev . Here what you pass is not a single pointer, but it is instead a richer API: https://github.com/openxla/xla/blob/7f8cfc91dae856f01fa70760f03b01f51b3afe98/xla/runtime/ffi/ffi_c_api.h
JAX does not yet use this, but plans to do so.
I'm unsure whether there is value in trying to come up with a backend-independent API for registering custom kernels at all! Is there enough in common that it is worth it? Perhaps each plugin should have a separate API  for registering custom kernels.

There are two levels of backends here:
a) CPU/GPU/TPU (let's call it the hardware backend)
b) The framework that compiles/jits the code (let's call it the framework backend. e.g. IREE, XLA)
Having a framework-independent API but one that potentially depends on the hardware backend is useful, for instance, to switch between IREE and XLA frameworks.
(Indeed the two types of backends are being conflated, for instance, in the openxla-pjrt-plugin, where JAX_PLATFORM=iree_cpu runs the IREE framework with the CPU backend which JAX_PLATFORM=cpu runs the XLA framework with the CPU backend.)

The approach we've taken so far for our other plugins is that each backend will define its own API for registering custom calls. I'm not sure there is a sensible abstraction for registering custom calls that makes sense between different hardware and software stacks!


allow ol-dev-versions as peer dependency

Following the convention of ol-mapbox-style, this should allow projects to use the very latest dev-versions of ol, which could include critical features.

Hmm, this doesn't indicate which the minimum version of ol is required though, so not ideal...

This is true, generally these can be found in the README, unfortunately I am not aware of an alternative.
I have to admit that this not ideal, I hope there is a real solution for this issue from the side of npm in the near future. For now this is a judgment call about what is more important, knowing the minimum version when looking into the package.json-file, or allowing to use the latest features and fixes from ol.
Personally, i think that very few users will use this package alongside a legacy version of ol <7.5.2, while having to use the dev-Version because of some critical new feature/fix is quite common.

@m-mohr Other 3rd party libraries for ol do the same, and I'm doing it also in ol-mapbox-style. Of course there's a trade-off between having the minimum compatible version in package.json and convenience of working with dev versions of OpenLayers, but after talking to @tschaub about his experiences I decided to go with "*".

Alright. @RobertOrthofer Could you please add the minimum required version to the README instead?

@m-mohr I have added it to the README and made a small FAQ entry, if that's ok

Thanks

Thanks, merged.


File load error

when loading shop I get this FILE LOAD ERROR can someone please help me?

Had to fully redownload the game on steam to fix


fixed weight location for ft conversion

I already fixed this + some others in my followup pr: https://github.com/mosaicml/llm-foundry/pull/229

@data42lgtm : Please use the one from https://github.com/mosaicml/llm-foundry/pull/229. It will be merged in soon.

Closing as this was completed in #229


shm api update from FreeBSD 13

r? @Amanieu
(rust-highfive has picked a reviewer for you, use r? to override)

@bors r+

:pushpin: Commit 0a57541a7101bd58051b1d193e5a71981715bacb has been approved by Amanieu

:hourglass: Testing commit 0a57541a7101bd58051b1d193e5a71981715bacb with merge df679e80756d9a71e9a7b0b298c0c45ba7b70083...

:sunny: Test successful - checks-actions, checks-cirrus-freebsd-11, checks-cirrus-freebsd-12, checks-cirrus-freebsd-13
Approved by: Amanieu
Pushing df679e80756d9a71e9a7b0b298c0c45ba7b70083 to master...


bug: List table view in Jupyter doesn't show anything

Description
When working through the user study in vanilla, local Jupyter, the list table response wasn't rendered.

@kerinin can you verify this is still a problem? The latest release shows an empty rendering.

This is no longer an issue in the latest version of the Kaskada client. Verified in the ecommerce jupyterhub example. Closing for now, if this is still an issue please re-open.


Fix issue with GDScript binary tokens being non-deterministic

Fixes #96854.
This fixes the issue of GDScript files exported with "Binary tokens" or "Compressed binary tokens" having different contents (and thus hash) with most exports, despite the corresponding GDScript file being identical.
This PR addresses this by simply writing zeroes to the gap at bytes 13-16, and marking the gap as reserved unused, in order to preserve backwards compatibility.
If backwards compatibility isn't deemed important I can of course just shift everything after the 12th byte down 4 bytes instead.

Thanks!

Cherry-picked for 4.3.1.


An Ordered Matrix

Before jumping into Pr first comment for assign.
Problem statement : Create an ordered 2D array (matrix). A matrix is ordered if its (0, 0) element is 1, its (0, 1) element is 2, and so on. Your function needs to create an a × b matrix where a is the first argument and b is the second.
Probelm Link :  https://edabit.com/challenge/qbpHHZYQrJnNYjtjn
Choose one Cateorgy - Medium
question category :  Arrays, Math

Before submitting a PR please Check some details.

[ ] fileName formet should be in camelcase
example- returnTheNextNumber.py
[ ]  Choose label for Problem , example- "good first issue", "easy"
[ ]  In file there should be problem statement and link to problem

/* Problem-Task : This program will add two numbers
 * Problem Link : https://edabit.com/challenge/SFzHtm63XT6EYNHWY
*/
int main() {
    int bla;
    return 0;
}

This repository is vaild for HacktoberFest2020
Read before making PR - How to contribute on Github
Good Luck.

@aayushi-droid  Please Assign this issue to me :)

@kushalbargoti  please assign me this

Hey! @Vrindagupta6828  you can choose any problem from edabit then create an issue and get it assigned and work on it.

I would love to work on this project

@Vrindagupta6828 Please assign this task to me
Thanks!

@kushalbargoti when will these issues be assigned?

I would like to contribute please assign this to me.

@aayushi-droid  Done✅

Problem statement : Create a function to find only the root value of x in any quadratic equation ax^2 + bx + c.
Probelm Link : https://edabit.com/challenge/nasKYub6qEAfQcFuy
Choose one Cateorgy - Medium
question category : Conditions , Math, Algebra
Can i do this in C++?


enhancement Provide default environment variables if it is unset

Proposed changes
Issue Number: close #xxx
from #19041
BE with java support needs some environment variables to start up and we have set them up in the start scripts (such as start_be.sh). However, if we debug BE directly, these environment variables are not set by default. For the convenience, we should provide default environment variables under this scenario.
Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run buildall

run external

run buildall


[BUG] Requesting metadata update due to unknown leader topics  When configured multi advertised listeners include ipv6

When Configured multi advertised listeners such as kop_ipv4://ipv4:9092,kop_vpc_ipv6://[ipv6]:9093 ,  Message Sended by ipv6 listener  has a result with exception:
 DEBUG [Producer clientId=console-producer] Requesting metadata update due to unknown leader topics from the batched records: [test001] (org.apache.kafka.clients.producer.internals.Sender)
But it is normal with ipv4 listener. It is inconsistent when request the metadata use ipv6 listener。 Metadata Response as below
brokers=[
  MetadataResponseBroker(_nodeId=673416243_,  host='2409:8cxxxx', port=9093, rack=null),
  MetadataResponseBroker(**_nodeId=1634887049_**, host='2409:8cxxxx', port=9093, rack=null),
  MetadataResponseBroker(_nodeId=1568042142_, host='2409:8cxxxx', port=9093, rack=null),
  MetadataResponseBroker(_nodeId=1098472548_, host='2409:8cxxxxx', port=9093, rack=null)],

topics=[MetadataResponseTopic(errorCode=0,
  name='test001',
  topicId=AAAAAAAAAAAAAAAAAAAAAA,
  isInternal=false,
  partitions=[
    MetadataResponsePartition(errorCode=0, partitionIndex=0, _**leaderId=967928506**_, leaderEpoch=-1, replicaNodes=[967928506], isrNodes=[967928506], offlineReplicas=[]),
    MetadataResponsePartition(errorCode=0, partitionIndex=1, _**leaderId=967928506**,_ leaderEpoch=-1, replicaNodes=[967928506], isrNodes=[967928506], offlineReplicas=[]),
    MetadataResponsePartition(errorCode=0, partitionIndex=2, _**leaderId=378746841**_, leaderEpoch=-1, replicaNodes=[378746841], isrNodes=[378746841], offlineReplicas=[]),
    MetadataResponsePartition(errorCode=0, partitionIndex=3, _**leaderId=698533689**_, leaderEpoch=-1, replicaNodes=[698533689], isrNodes=[698533689], offlineReplicas=[]),
    MetadataResponsePartition(errorCode=0, partitionIndex=4, _**leaderId=967928506**,_ leaderEpoch=-1, replicaNodes=[967928506], isrNodes=[967928506], offlineReplicas=[]),
    MetadataResponsePartition(errorCode=0, partitionIndex=5, _**leaderId=967928506**,_ leaderEpoch=-1, replicaNodes=[967928506], isrNodes=[967928506], offlineReplicas=[])],
}

The Partition isrNodes(leadeId) isn`t in  the brokers list

When looking up partitionMetadata for topic , the nodeId is computed according the InetSocketAddress
newPartitionMetadata(topic, newNode(inetSocketAddress))
But Node Set in metacache is computed by advertisedlisteners's Info.
  # Map<String, Set<Node>> kopNodesMap = getNodes(kopBrokerStrs);
   # kopBrokerStrs <= "kop_v4://<IP_ADDRESS>:9092,kop_v6://[2409:6c85:ab34:1d3::1e]:9093"

This may lead to inconsistencies, V6 hoststring from InetSocketAddress is 128bits,
2409:6c85:ab34:1d3::1e  => 2409:6c85:ab34:1d3:0:0:0:1e


(1216) Show basic information and user list for workflow managers

This changes the link to an assessment on the Workflow Manager's view to show basic information about an assessment, who it is allocated, as well as fetching the list of users who can be allocated to the assessment. The next PR will do the actual allocation, but I thought it'd be better to break the featue up into two PRs
Screenshot

 Allow a workflow manager to reallocate an assessment


Divisonal tracking reporting for ministries

As a pilot ministry coordinator
I want divisional tracking on my reporting dashboard
so that I can see how divisions are responding

Assumptions & Scope
What are the assumptions for this story?
Our system does not have consults currently.
Graphs that default by ministry become by division when filter is applied.
Ministry coordinator wants to extract data to provide to ministry division.
Require an ETL from the new app into the reporting warehouse
ETL will run on OpenShift - new Python framework
Divisional Tracking:
Filter by division
Filter by request state
Filter by divisional stage
Due dates for 3 stages (Harms, Ministry Sign Off, Records Gathering)
Average numbers of days for each stage by Division and Ministry Total
Average number of days per state by Division and Ministry Total
What is IN scope?
Divisional tracking for pilot ministries.
What is NOT in scope?
Divisional tracking for non-pilot ministries.
Acceptance Criteria
Scenario 1: xxxxxx

GIVEN the ministry coordinator is on Redash
WHEN the ministry coordinator selects the ministry reporting dashboard
THEN the coordinator graphs for Percentage On-Time by Ministry, Total Closed by Ministry, Total Active by Ministry, Average Processing Days by Ministry, Average Days Overdue by Ministry, Average Days by Request States, Average Days by Divisional Stages, Number of Processing Pages by Ministry, No Records Response by Ministry
AND filters for End FYr, End FQtr, By Month, By Week, Request Type (General, Personal), Request Status (Active, Overdue, Closed), Ministry (4 for Economy Sector), Division, Applicant Type, Ministry Assigned To (i.e., coordinators)

Scenario 2: xxxxxx

GIVEN ... (you or your condition)
WHEN ... (what you do)
THEN ... (what you see)

Scenario 3: xxxxxx
...
Dependencies? What is the impact of this dependency? (If so, link dependency in the ticket, make it visible in a team´s backlog)
Validation Rules? (If yes, list here)
Design
@xxx - please link the Design here
Definition of Ready

[ ] Is there a well articulated User Story?
[ ] Is there Acceptance Criteria that covers all scenarios (happy/sad paths)?
[ ] If there is a user interface, is there a design?
[ ] Does the user story need user research/validation?
[ ] Does this User Story needs stakeholder approval?
[ ] Design / Solution accepted by Product Owner
[ ] Is this user story small enough to be completed in a Sprint? Should it be split?
[ ] Are the dependencies known/ understood? (technical, business, regulatory/policy)
[ ] Has the story been estimated?

Definition of Done

[ ] Passes developer unit tests
[ ] Passes peer code review
[ ] If there's a user interface, passes UX assurance
[ ] Passes QA of Acceptance Criteria with verification in Dev and Test
[ ] Confirm Test cases built and succeeding
[ ] No regression test failures
[ ] Test coverage acceptable by Product Owner
[ ] Ticket ready to be merged to master or story branch
[ ] Developer to list Config changes/ Update documents and designs
[ ] Can be demoed in Sprint Review
[ ] Tagged as part of a Release
[ ] Feature flagged if required
[ ] Change Management activities done?

@m-prodan. @JHarrietha-AOT, did I miss thing anything from the session with the economy sector?

nkan-aot, please review and comment.

features we need to add -

category by division
category by Request States
Number of Processing Pages by Ministry
No Records Response by Ministry
Filter by division, analyst

@nkan-aot is this a duplicate ticket? Were all these changes covered in #2379?

Duplicate


[Flink-12164][tests] Harden JobMasterTest#testJobFailureWhenTaskExecutorHeartbeatTimeout

What is the purpose of the change

Harden unstable case JobMasterTest#testJobFailureWhenTaskExecutorHeartbeatTimeout

Brief change log

Introduce factory pattern for HeartbeatMonitor. So that we could inject codes for testing.
Enrich TestingHeartbeatServices by supporting manually triggering timeout
Use TestingHeartbeatServices to trigger timeout manually for JobMasterTest#testJobFailureWhenTaskExecutorHeartbeatTimeout instead of the heartbeat service with a short heartbeat timeout threshold

Verifying this change

This change is about testing
Verify the change by re-running the relevant cases

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): no
The public API, i.e., is any changed class annotated with @Public(Evolving): no
The serializers: no
The runtime per-record code paths (performance sensitive): no
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: no
The S3 file system connector: no

Documentation

Does this pull request introduce a new feature? no
If yes, how is the feature documented? not applicable

@flinkbot attention @tillrohrmann

@tillrohrmann, Thanks for detailed reviewing. It helps a lot.
I have just updated the PR :)

Oops! The Travis building failed due to some python relevant issues. Will rebase master and try again.


[ConstraintSystem] Detect and diagnoses use of members with mutating getters in key path

Such use is currently not allowed so variables or subscripts with
mutating getters should be rejected and diagnosed.
struct S {
  var foo: Int {
    mutating get { return 42 }
  }

  subscript(_: Int) -> Bool {
    mutating get { return false }
  }
}

_ = \S.foo
_ = \S.[0]

Resolves: rdar://problem/49413561

@swift-ci please test

@swift-ci please test source compatibility

Build failed
Swift Test Linux Platform
Git Sha - eaf1dc0012cf6f97761d6e69cc149051aa867997

@swift-ci please test Linux platform

Build failed
Swift Test Linux Platform
Git Sha - eaf1dc0012cf6f97761d6e69cc149051aa867997

@swift-ci please test Linux platform

Build failed
Swift Test Linux Platform
Git Sha - eaf1dc0012cf6f97761d6e69cc149051aa867997

@swift-ci please test Linux platform


AToken Test Harness Setup

We want to implement Flexible Voting for ATokens inside the Aave system. To do so, we'll need a test harness that:

Forks mainnet to interact with the Aave protocol
Extends/implements an AToken for a mock ERC20Votes GOV token
Adds the new GOV AToken market to Aave
Can supply, borrow, liquidate, etc... for said market

So that I keep all of this straight in my head, this is the usecase we're targeting:

there is an ERC20, call it GOV
GOV is the governance token issued by Protocol, which inherits from GovernorCountingFractional
I deposit some GOV into Aave and receive aGOV in return
aGOV implements FractionalPool-like functionality
as proposals are made to Protocol, I can express my voting preference on them directly to aGOV
aGOV will then cast fractionally weighted votes to Protocol according to the expressed voting preferences of its holders


Change protocol to https and port to 443

Landed in https://github.com/metarhia/metarhia.com/commit/e1f435a54271b0027593c0a3f8b2570d3c7385cc


Gradient path width changes with min/max map zoom

It appears that when decreasing the min map zoom from eg 3 to 1 the width of a gradient path increases on the map.

the scale used to render the object is depending of the projectionZoom option. By default this option is set to a function that returns a value depending on the minZoon and maxZoom: https://github.com/manubb/Leaflet.PixiOverlay/blob/master/L.PixiOverlay.js#L36
Fixing the projectionZoom to a constant value should do the trick


New Chat - Icon isn't displayed for the new chat

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Go to https://staging.new.expensify.com
Log in with expensifail
Click on New Chat
Search for a user that you don't have any messages with and open the chat.

Expected Result:
Icon is displayed for new chat
Actual Result:
Icon is not displayed for new chat
Workaround:
Unknown
Platform:

Where is this issue occurring?

Web
iOS
Android
Desktop App
Mobile Web

Version Number: <IP_ADDRESS>
Reproducible in staging?: Yes
Reproducible in production?:  Yes
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
Expensify/Expensify Issue URL:

Issue reported by: Applause
Slack conversation:
View all open jobs on GitHub

Triggered auto assignment to @neil-marcellini (Engineering), see https://stackoverflow.com/c/expensify/questions/4319 for more details.

I was unable to reproduce this on staging or production when logging in as<EMAIL_ADDRESS>See the video below for my attempt to reproduce this on production. @kbecciv can you try with a different account?
https://user-images.githubusercontent.com/26260477/151073405-ad829085-da18-4d85-8db0-690709340093.mov

@mvtglobally have you been able to reproduce this?

@neil-marcellini it seems the issue is still repro on Windows. Mac user was not able to repro
https://utest-dl.s3.amazonaws.com/testRunStepMinAttachmentsResult/17466509/Gravar__360.mp4?AWSAccessKeyId=AKIAJ2UIWMJ2OMC3UCQQ&Expires=1675232108&Signature=j4uxDJWBM2UnRHkpA1%2Bjz%2BQk60U%3D

This is also reproducible on mac.
cc: @neil-marcellini @mvtglobally

To repro this we need to add a new user.

To clarify, do you mean type a completely new username into the new chat input? Or do you mean create a new user and then type their name into the new chat input?

@thesahindia To clarify, do you mean type a completely new username into the new chat input?

Yes we just need to type a new username

I was unable to reproduce on my dev environment on all platforms, even when typing a new username into the input. I was also unable to reproduce it on web production. I'm going to un-assign and let someone work on this who is able to reproduce it.

I was also unable to reproduce.

Issue not reproducible during KI retests. (First week)

Issue not reproducible during KI retests. (Second week)

Issue not reproducible during KI retests. (Third week) Closing


Xamarin.Mac Support

Describe the solution you'd like
App Center available for Xamarin.Mac applications.
Describe alternatives you've considered
Xamarin Insights supported this. When it was terminated, App Center was touted as the replacement and would support the same platforms.
I don't believe there is any other Xamarin supporting tool for insights or crash reporting without a custom in-house solution.
Additional context
App Center website has indication that not all insights platforms are supported. https://docs.microsoft.com/en-us/appcenter/migration/xamarin-insights/
https://github.com/Microsoft/AppCenter-SDK-DotNet/issues/529
https://github.com/Microsoft/AppCenter-SDK-DotNet/issues/615

Are there any updates to this feature request or do we have to integrate using the Swift Mac app logic inside of a Xamarin Mac / Visual Studio for Mac app.  Seems backwards.

Been waiting patiently for this feature for a long time now.

Think you'll be waiting, patiently or otherwise, for a long time... Don't seem a priority for them.

In full transparency @projectgoav, you're totally correct. We've been focused on drilling deeper into our mobile platforms and adding more Windows support lately based on the large amount of feedback we receive. We absolutely are tracking interest here, but we're stuck working a bit higher up the Github list for now.

We absolutely are tracking interest here, but we're stuck working a bit higher up the Github list for now.

I think interest is the wrong word here.
Xamarin Insights users were told the service was being turned off and to migrate to App Center. "Feature Parity" was quoted and visible on both Insights and AppCenter websites.
Insights is no longer available and yet not everything we could do with Insights is available with App Centre....

I would like to add that I've considered AppCenter a few times over the last year or so for Analytics and Crash Reporting since Insights was closed down. I have ditched the idea each time because of lack of Xamarin.Mac support. I come back here every now and then to see what's going on Xamarin.Mac wise and each time I see that it is still not supported with no confidence that anything is likely to change in the near future.
Unfortunately Xamarin.Mac has always been the achilles heel of the Xamarin platform and each time development effort is vastly increased when trying to make solutions work on the Xamarin.Mac platform.
This is clearly the case here too, it's disappointing to see that we still struggle to support the Mac platform as well as we could.
I wanted to be vocal about my disappointment and I'm sure there are a lot of other folks who are not being so vocal that are frustrated by this too. For a framework that is being touted as multi-platform not supporting such a key platform is a pretty strange move.

I think I read somewhere that there is a feature request that can be "voted on".. if that's the case could someone forward me a link so I can vote?

I understand there is a "workaround" contributed to by a community member and a native OSX/ObjC option, does anyone have a reliable implementation of that live and working in the wild?

@KeithBoynton I think you vote here by pressing thumbs up button.

Perfect, thank you @ivanicin

I'm with @KeithBoynton, App Center is great but multiple times I've been in a position where a Mac project in my x-platform Xamarin solution is either left behind or a different choice has to be made for reporting errors/events. Given that both Xamarin and App Center are under the MS umbrella, and the fact that the bindings would likely be similar to the iOS bindings project, from the outside looking in it seems like a natural next step. There's already support for native MacOS platforms why not Xamarin-Mac?

I also see a community member @equinox2k put in some effort with this pull request
https://github.com/microsoft/appcenter-sdk-dotnet/pull/658
Which was just closed down.. what's going on?

If you look through the repsonses linked at the bottom of my original post it's clear that Xamarin.Mac is not a priority for them.
I requested the feature almost 2 years ago now and nothing has been done about it so I think it's a lost cause at this point :(

Yep I feel your pain, it's been like that with Xamarin.Mac for years in most areas, without the legendary support of @chamons who knows where we'd be.
I find it particuarly perplexing given that Native MacOS is supported so struggle to understand why we can't just have a wrapper around that.

Please do not close this issue, bot 🤖

Bump

On behalf of @Suplanus and friends, stay alive

2 and a half years on a Mac still not properly supported on a "cross platform" framework. Is macOS really that much of an unpopular platform?

Hi folks, glad to inform that we are currently working on supporting Xamarin.Mac platform. You can track the progress here: https://github.com/microsoft/appcenter-sdk-dotnet/pull/1586.
The PR is based on @nor0x's implementation, thanks a lot to him for his great work.
The Xamarin.Mac support is expected to be available in the next release this quarter.

Hi there! The feature has been released in the 4.5.0 version of App Center SDK.


AttributeError: 'list' object has no attribute 'encode'

Describe the bug
When I was trying to visualize on compare_classifiers_predictions_distribution, I got this error, saying
AttributeError: 'list' object has no attribute 'encode'
Steps to reproduce the behavior:
Run this command:
ludwig visualize --visualization compare_classifiers_predictions_distribution --ground_truth path/to/train.hdf5 --test_statistics results_1/test_statistics.json --predictions path/to/results_0
Please provide code, yaml definition file and a sample of data in order to entirely reproduce the issue.
The YAML file for the model
input_features:
-
name: data
type: image
encoder: resnet
preprocessing:
image:
in_memory: true
resize_method: interpolate
height: 640
width: 480
num_channels: 1
output_features:
-
name: emotions
type: category
training:
batch_size: 16
epochs: 50
early_stop: 5
learning_rate: 0.001
Expected behavior
the compare_classifiers_predictions_distribution plot to be visualized.
Screenshots

Environment (please complete the following information):

OS: iOS
Version 10.14.5
Python version 3.7.3
Ludwig version 0.2

Additional context
I had all the necessary files as described in the visualization page here https://uber.github.io/ludwig/user_guide/#compare_classifiers_predictions_distribution
but somehow this error persists.

@rthiruv-lab58 I think you're missing one more argument: field. Could you please check if that's the issue?

@rthiruv-lab58 I think you're missing one more argument: field. Could you please check if that's the issue?

I did not use the field parameter, because for "compare_classifiers_multiclass_multimetric"  visualization, It worked without the field parameters.
I thought it (field) had a default value, which it clearly does not have. What threw me off, was the encoding format, which I was not able to track back to the h5py script.

right. yeah, the error message wasn't really helpful. Did that resolve the issue?

right. yeah, the error message wasn't really helpful. Did that resolve the issue?

Hello, sorry for the late reply. I was looking through the documentation, but i could not understand how to generate the field value (for the ground truth) or if it is generated during the process of training, where it is stored. Any insights on that will be useful.

The field is just the name of the column you are predicting.

@w4nderlust thank you for the response. I understand what the field is now.
When i was trying to visualize with the field argument, the following was printed:
[2 6 1 4 5 3 7 8 0]
A list with 9 elements. I saw the sample image in the documentation having 10(0-9) elements  So when i ran the visualization for compare_classifiers_predictions_distribution,
The list index out of error arises which makes sense because the prediction is on 8 classes and not 9. I wonder if the 'UNK' class prediction causes the index to be thrown off.
for the prediction argument i pointed the numpy array for predictions stored in results_0
the snippet for #Draw polygon representing values
for i, alg_predictions in enumerate(predictions):
draw_polygon(alg_predictions, algorithms[i], colors[i])
Screenshot of the new error.

Can you please provide a minimal reproducible example?
The predictions from the model have one more class that the ones in the ground truth because of the UNK, but that shouldn't be a problem.
As a reference i would suggest you to look at the test test_visualisation_cmp_classifiers_predictions_distribution_output_saved in tests/integration_tests/test-visualization.py

Closing due to staleness, feel free to reopen if needed.


EXEC warning SQL73162 --> Option TrustServerCertificate is missing

I run the following:
dotnet publish ./myproj.csproj \
/p:TargetServerName=<IP_ADDRESS> \
/p:TargetUser=MYUSER \
/p:TargetDatabaseName=MYDATABASE \
/p:Encrypt=False

It fails showing:
EXEC : warning SQL73162: Changes to connection setting default values were incorporated in a recent release.
More information is available at https://aka.ms/dacfx-connection

The link provided talks about certificates in the connection

When I run the deploy from Visual Studio 2022, it works. The connection string looks like this:
Data Source=MYSERVER;Persist Security Info=True;User ID=MYUSER;Pooling=False;MultipleActiveResultSets=False;Connect Timeout=60;Encrypt=False;TrustServerCertificate=False

As you can see, the last option is TrustServerCertificate set to False;
So, Encrypt=False seems to be not enough

I suggest you just install the global tool Microsoft.Sqlpackage and then use the full fledged sqlpackage command

Thanks for your response @ErikEJ
Just to let you know, the database I was targeting was running SQL Server 2008 R1.
I needed to downgrade the SSL threshold as mentioned here:
Microsoft.Data.SqlClient.SqlException
But even with that change, I couldn't deploy successfully because SqlClient needs SQLServer 2008 R3.. so.. never mind.


[Bug] MediaElement does nothing on UWP

Description
MediaElement works on Android but does nothing on UWP
Stack Trace

Link to Reproduction Sample

Steps to Reproduce

Create a Xamarin project with Android, iOS and UWP
Add Xamarin.CommunityToolkit (2.0.2) Nugut package to them all
Add MediaElement to the main page of the shared project:
<xct:MediaElement Source="https://sec.ch9.ms/ch9/5d93/a1eab4bf-3288-4faf-81c4-294402a85d93/XamarinShow_mid.mp4" ShowsPlaybackControls="True" AutoPlay="True"/>
"Set as Startup Project" to UWP one, nothing happens (if I do so with Android one, the video will be played well)

Expected Behavior
Actual Behavior
Basic Information

Version with issue: 2.0.2
Last known good version:
IDE: Visual Studio 2022 Community 17.2.3
Platform Target Frameworks:

UWP: Microsoft.NETCore.UniversalWindowsPlatform 6.2.14
Xamarin Forms: <IP_ADDRESS>8
Xamarin.Essentials: 1.7.3

Workaround

Reproduction imagery

Also encountering this.


Unable to access URL using push_to_git_remote

Issue Description

Initially, I was getting the below :
/Library/Ruby/Gems/2.6.0/gems/fastlane-2.186.0/fastlane_core/lib/fastlane_core/ui/interface.rb:153:in `shell_error!': [!] Exit status of command 'git push origin Development:Development --tags' was 1 instead of 0. (FastlaneCore::Interface::FastlaneShellError)
error: src refspec Development does not match any
error: failed to push some refs to '/Users/user/Development/Personal/Project/APP'

But, I was able to fix this after following the instructions mentioned https://github.com/fastlane/fastlane/issues/7588#issuecomment-274462956
This worked locally but It's not working when I run the CI via Gitlab. I get the below error :
[23:29:26]: Exit status of command 'git push origin HEAD:Development --tags --force' was 128 instead of 0.
2583 remote: You are not allowed to upload code.
2584 fatal: unable to access 'URL': The requested URL returned an error: 403

I presume there is some issue with authentication, but I'm not sure how I can inject details in https://docs.fastlane.tools/actions/push_to_git_remote/#push_to_git_remote to fix it.

Environment
🚫 fastlane environment 🚫
Stack

Key
Value

OS
10.15.7

Ruby
2.7.2

Bundler?
true

Git
git version 2.24.3 (Apple Git-128)

Installation Source
~/.rvm/gems/ruby-2.7.2/bin/fastlane

Host
Mac OS X 10.15.7 (19H1030)

Ruby Lib Dir
~/.rvm/rubies/ruby-2.7.2/lib

OpenSSL Version
OpenSSL 1.1.1i  8 Dec 2020

Is contained
false

Is homebrew
false

Is installed via Fabric.app
false

Xcode Path
/Applications/Xcode.app/Contents/Developer/

Xcode Version
12.4

Swift Version
5.3.2

System Locale

Error

No Locale with UTF8 found 🚫

fastlane files:
`./Fastfile`
# This file contains the fastlane.tools configuration
# You can find the documentation at https://docs.fastlane.tools
#
# For a list of all available actions, check out
#
#     https://docs.fastlane.tools/actions
#
# For a list of all available plugins, check out
#
#     https://docs.fastlane.tools/plugins/available-plugins
#

# Uncomment the line if you want fastlane to automatically update itself
# update_fastlane

fastlane_require 'dotenv'
before_all do
  Dotenv.overload '.env.secret'
    puts Dotenv.overload
end

default_platform(:ios)
platform :ios do
  desc "Build Application"
  lane :build do
    clear_derived_data
    clean_build_artifacts
     cocoapods(use_bundle_exec: false)
     gym(scheme: ENV['SCHEME'], skip_archive: true)

   end

 lane: deploy do

    clear_derived_data
    clean_build_artifacts
    cocoapods(use_bundle_exec: false)
    gym(scheme: ENV['SCHEME'])

    api_key = app_store_connect_api_key(
        key_id: ENV['KEY'],
        issuer_id: ENV['ISSUE'],
        key_filepath: "AuthKey_XXX.p8",
       duration: 1200, # optional (maximum 1200)
       in_house: false # optional but may be required if using match/sigh
    )

    pilot(api_key: api_key,
       distribute_external: true,
       skip_submission: true,
       notify_external_testers: false,
       skip_waiting_for_build_processing: false,
       groups: ["HelloWorld Beta Testers"], # Maybe we should create groups
       changelog: File.read("changelog.txt")
    )

    # push_to_git_remote(remote: "origin", remote_branch: ENV['BRANCH'])

    upload_symbols_to_crashlytics

push_to_git_remote(
      remote: "origin",         # optional, default: "origin"
      local_branch: "HEAD",  # optional, aliased by "branch", default: "master"
      remote_branch: ENV['BRANCH'], # optional, default is set to local_branch
      force: true,    # optional, default: false
    )
   end

   lane :versioning do
     increment_build_number
     commit_version_bump(
           message: 'Build Version Bump by fastlane',
           force: true,
           xcodeproj: "./XXX.xcodeproj"  # optional, if you have multiple Xcode project files, you must specify your main project here
         )

      build_number = Actions.lane_context[Actions::SharedValues::BUILD_NUMBER]
      add_git_tag(tag: "testflight-#{build_number}-#{ENV['TAG']}")
   end

  end

`./Appfile`
# app_identifier("[[APP_IDENTIFIER]]") # The bundle identifier of your app
# apple_id("[[APPLE_ID]]") # Your Apple email address

# For more information about the Appfile, see:
#     https://docs.fastlane.tools/advanced/#appfile

fastlane gems

Gem
Version
Update-Status

fastlane
2.185.1
🚫 Update available

Loaded fastlane plugins:
No plugins Loaded
Loaded gems

Gem
Version

did_you_mean
1.4.0

executable-hooks
1.6.1

bundler-unload
1.0.2

rubygems-bundler
1.4.5

bundler
2.2.20

rake
13.0.3

CFPropertyList
3.0.3

concurrent-ruby
1.1.9

i18n
1.8.10

minitest
5.14.4

thread_safe
0.3.6

tzinfo
1.2.9

activesupport
5.2.6

public_suffix
4.0.6

addressable
2.7.0

httpclient
2.8.3

json
2.5.1

algoliasearch
1.27.5

artifactory
3.0.15

atomos
0.1.3

aws-eventstream
1.1.1

aws-partitions
1.469.0

aws-sigv4
1.2.3

jmespath
1.4.0

aws-sdk-core
3.114.3

aws-sdk-kms
1.44.0

aws-sdk-s3
1.96.1

babosa
1.0.4

claide
1.0.3

fuzzy_match
2.0.4

nap
1.1.0

netrc
0.11.0

ffi
1.15.3

ethon
0.14.0

typhoeus
1.4.0

cocoapods-core
1.10.1

cocoapods-deintegrate
1.0.4

cocoapods-downloader
1.4.0

cocoapods-plugins
1.0.0

cocoapods-search
1.0.0

cocoapods-trunk
1.5.0

cocoapods-try
1.2.0

colored2
3.1.2

escape
0.0.4

fourflusher
2.3.1

gh_inspector
1.1.3

molinillo
0.6.6

ruby-macho
1.4.0

nanaimo
0.3.0

xcodeproj
1.19.0

cocoapods
1.10.1

colored
1.2

highline
2.0.3

commander
4.6.0

declarative
0.0.20

digest-crc
0.6.3

unf_ext
<IP_ADDRESS>

unf
0.1.4

domain_name
0.5.20190701

dotenv
2.7.6

emoji_regex
3.2.2

excon
0.82.0

faraday-em_http
1.0.0

faraday-em_synchrony
1.0.0

faraday-excon
1.1.0

faraday-net_http
1.0.1

faraday-net_http_persistent
1.1.0

multipart-post
2.0.0

ruby2_keywords
0.0.4

faraday
1.4.2

http-cookie
1.0.4

faraday-cookie_jar
0.0.7

faraday_middleware
1.0.0

fastimage
2.2.4

jwt
2.2.3

memoist
0.16.2

multi_json
1.15.0

os
1.1.1

signet
0.15.0

googleauth
0.16.2

mini_mime
1.1.0

trailblazer-option
0.1.1

uber
0.1.0

representable
3.1.1

retriable
3.1.2

rexml
3.2.5

webrick
1.7.0

google-apis-core
0.3.0

google-apis-androidpublisher_v3
0.6.0

google-apis-playcustomapp_v1
0.3.0

google-apis-iamcredentials_v1
0.4.0

google-apis-storage_v1
0.4.0

google-cloud-env
1.5.0

google-cloud-errors
1.1.0

google-cloud-core
1.6.0

google-cloud-storage
1.31.1

mini_magick
4.11.0

naturally
2.2.1

plist
3.6.0

rubyzip
2.3.0

security
0.1.3

simctl
1.6.8

terminal-notifier
2.0.0

unicode-display_width
1.7.0

terminal-table
1.8.0

tty-screen
0.8.1

tty-cursor
0.7.1

tty-spinner
0.9.3

word_wrap
1.0.0

rouge
2.0.7

xcpretty
0.3.0

xcpretty-travis-formatter
1.0.1

generated on: 2021-07-13

@jbatra-umeey Hi, have you tried to git push from terminal? What was the result?

Yes, the following commands work fine when tried locally

sh('git', 'pull')
sh('git', 'push')

Also, the below command works as mentioned in https://github.com/fastlane/fastlane/issues/7588#issuecomment-274462956
push_to_git_remote(
      remote: "origin",         # optional, default: "origin"
      local_branch: "HEAD",  # optional, aliased by "branch", default: "master"
      remote_branch: "development", # optional, default is set to local_branch
      force: true,    # optional, default: false
      tags: true     # optional, default: true
    )

Do I need to add some git config for this to work?

I meant running git push command in terminal, not from fastlane or a ruby script to see that it's possible to push and that the repository rights are fine.
Is ENV['BRANCH'] defined properly? Your first call resulted in git push origin HEAD:Development and you say that if you hardcode remote_branch to development it works.

@lucgrabowski
Yes, when I try git push origin HEAD: Development, it works fine. For this run, via Fastlane I had to add additional params in
push_to_git_remote as mentioned https://github.com/fastlane/fastlane/issues/19088#issuecomment-878910111
Below the stack trace from Gitlab
 [23:07:56]: $ git push origin HEAD:Development --tags --force
 [23:07:58]: ▸ remote: You are not allowed to upload code.
 [23:07:58]: ▸ fatal: unable to access 'URL': The requested URL returned error: 403
 +---------------------------+------------------------------------------------------------------------------------------------------+
 |                                                           Lane Context                                                           |
 +---------------------------+------------------------------------------------------------------------------------------------------+
 | ENVIRONMENT               | STAG                                                                                |
 | DEFAULT_PLATFORM          | ios                                                                                |
 | PLATFORM_NAME             | ios                                                                                |
 | LANE_NAME                 | ios deploy                                                                                |
 | SIGH_PROFILE_PATHS        |                                                                                |
 | DSYM_PATHS                |                                                                                |
 | DSYM_LATEST_UPLOADED_DATE |                                                                                |
 | IPA_OUTPUT_PATH           | /Users/user/builds/K4rAvYXo/0/xx/ttap_app/XX.ipa                                    |
 | XCODEBUILD_ARCHIVE        | /Users/user/Library/Developer/Xcode/Archives/2021-07-12/XX 2021-07-12 22.47.32.xcarchive |
 | DSYM_OUTPUT_PATH          | /Users/user/builds/K4rAvYXo/0/xx/ttap_app/XX.app.dSYM.zip                           |
 +---------------------------+------------------------------------------------------------------------------------------------------+
 [23:07:58]: Exit status of command 'git push origin HEAD:Development --tags --force' was 128 instead of 0.
 remote: You are not allowed to upload code.
 fatal: unable to access URL': The requested URL returned error: 403
 +------+---------------------------+-------------+
 |                fastlane summary                |
 +------+---------------------------+-------------+
 | Step | Action                    | Time (in s) |
 +------+---------------------------+-------------+
 | 1    | default_platform          | 0           |
 | 2    | clear_derived_data        | 22          |
 | 3    | clean_build_artifacts     | 0           |
 | 4    | cocoapods                 | 15          |
 | 5    | gym                       | 426         |
 | 6    | app_store_connect_api_key | 0           |
 | 7    | pilot                     | 810         |
 | 💥   | push_to_git_remote        | 2           |
 +------+---------------------------+-------------+
 [23:07:58]: fastlane finished with errors
 Looking for related GitHub issues on fastlane/fastlane...
 ➡  silent option is not available in ipa block
     https://github.com/fastlane/fastlane/issues/403 [closed] 4 💬
     22 Sep 2016
 ➡  Adds more options to the `slack` command
     https://github.com/fastlane/fastlane/pull/128 [closed] 3 💬
     04 Feb 2017
 🔗  You can ⌘ + double-click on links to open them directly in your browser.
 #######################################################################

From the logs, it seems the branch name is passed correctly.

If the website is down, there's not much you can do until it comes back up. Check back after a while to see if anything changes. If you know the site is back up but you still can't access it, clear your browser cache and then try again.

@Gillon6 @lucarge
I tried to add the script in gitlab.yml and I get the below error
Executing "step_script" stage of the job script
00:01
 $ unset cd
 $ git remote set-url origin<EMAIL_ADDRESS> $ cd fastlane
 $ git push
 fatal: You are not currently on a branch.
 To push the history leading to the current (detached HEAD)
 state now, use
     git push origin HEAD:<name-of-remote-branch>
Cleaning up file based variables
00:02
 ERROR: Job failed: exit status 1


Test android-native performance.

@andpor Android-native implementation timing for 20k - 20200ms. Old implementation timing 13000ms. May be problem in verbose mode sqlite-native-driver. How to disable verbose mode ?
11-17 12:20:43.383 28403-28502/com.reactapp V/sqlg: sqlc_st_column_name 0xb3ac4928 1
11-17 12:20:43.383 28403-28502/com.reactapp V/sqlg: sqlc_st_column_type 0xb3ac4928 1
11-17 12:20:43.383 28403-28502/com.reactapp V/sqlg: sqlc_st_column_name 0xb3ac4928 3

@andpor you will need a new version of the Android sqlite-connector.jar and native-driver*.so libraries. I will publish them soon and let you know.

ok. Hopefully the new libs will do the trick.

@andpor I just published an update to Cordova-sqlite-storage with the new versions of sqlite-connector.jar and the native libraries. You should take them at the same time since there is now internal testing to make sure they do not go out of sync.

@brodybits should have it updated in my repo within next few hours

@barbarosh @brodybits - latest native libs + new connector are in the repo. Alex could you please try them out?

@brodybits  @andpor I will make test new library during 1 hour.

@andpor @brodybits It work, but performance did not increased. Now speed 13000ms for 20k.

@barbarosh So it takes 13 seconds...is this pretty much the same result you got using the plain Java plugin to begin with? I would have thought going native would dramatically increase the speed..which does not seem to be the case. Are you seeing excessive logging still or anything else which could be contributing to this? ...and are you sure you are using the native and not the original by some odd chance :)
@brodybits Chris - any thoughts ?

@barbarosh Alex - what are the details of your test case - can you provide?

@andpor Yes it is the same result as the original plug-in, and I connect to the project folder "android-native", I noticed that in the verbose output to log many values by native lib.
Test case details:

I have table with 20k rows and autoindex
Table have uuid, name, type, template, icon, images, subtitle - all columns text type
SQL query "SELECT * FROM items"
code

return this.databaseObject.transaction((tx)=> {
          tx.executeSql(sql, params).then(([tx,results]) => {
            //console.log(results)
            if (results.rowsAffected > 0) {
              queryResult = true // TEMP  After insert etc TODO: Fix this
            } else {
              var len = results.rows.length;
              for (let i = 0; i < len; i++) {
                queryResult.push(results.rows.item(i))
              }
            }
          })
        })

This criteria identical  and to tests on IOS

@barbarosh do you say that the "android-native" version continues to log extra verbose output after the most recent fix? Can you post some of the output if so?

@brodybits @andpor After the last update they seem to have disappeared. I now no way to verify but most likely yes gone, because the duration of the test back to the start mark. Within 4 hours I tested it.

Maybe you can try a native app only running these queries and see how performance is...is it isolated to the db interface or realized in react native ? Perhaps you can open an issue for performance on cordova-sqlite-storage?

@brodybits Verbose information now have disappeared. Thanks.
@andpor I tried find place for check timing by debug code of android-native and set breakpoints but her don`t stoped  and I investigate android-native code and found
        DBRunner(final String dbname, JSONObject options, CallbackContext cbc) {
            this.dbname = dbname;
            this.createFromAssets = options.has("createFromResource");
            this.oldImpl = options.has("androidOldDatabaseImplementation");
            Log.v(SQLitePlugin.class.getSimpleName(), "Android db implementation: " + (oldImpl ? "OLD" : "sqlite4java (NDK)"));
            this.bugWorkaround = this.oldImpl && options.has("androidBugWorkaround");
            if (this.bugWorkaround)
                Log.v(SQLitePlugin.class.getSimpleName(), "Android db closing/locking workaround applied");

            this.q = new LinkedBlockingQueue<DBQuery>();
            this.openCbc = cbc;
        }

May be runned old implementation in new module ? Because old timing.

do you see sqlite4java (NDK) or OLD string in the debug output ? it would run old impl only if you ask for androidOldDatabaseImplementation in params I think...

@barbarosh Alex, I think I figured it out. It seems that if you include a project as a module in another project, the parent project does not import the jniLibs correctly from the child project - this is where the native libs are. I thought it would be automatic but it does not like like it is the case.
I am using the latest Android studio.
Can you try the following:
Copy the jniLibs from android-native project dir (src/main), to the same location in the main project. It should be under src/main of the main project app directory in project structure.

@barbarosh Alex, is think I found the issue. The android-native works fine as stand-along project but when included as module the native libs do not get included for some reason form current location. I reorganized the project and added an enhancement to build.gradle to fix this. Tested on both Genymotion simulator and device and see that native library is definitely used.
Also, the ':app' is not needed when you are adding the react-native-sqlite to parent project. I removed it from instructions.
You are looking for the following debug log stmt to verify native is used:
11-19 04:15:20.174 6783-6849/com.citi.cira.author D/ReactNativeJS: 'OPEN database: test.db'
11-19 04:15:20.186 6783-6849/com.citi.cira.author D/ReactNativeJS: 'new transaction is waiting for open operation'
11-19 04:15:20.193 6783-6850/com.citi.cira.author V/SQLitePlugin: Android db implementation: sqlite4java (NDK)

@andpor I check this today.

@andpor After your commits: for 20k timing 12600ms

not a huge difference...but not much I can do about it myself I am afraid...we can test whether it is caused react native overhead or internal SQLPlugin perf issue..

I do not have time to test, it is possible to check up on the weekend.

@barbarosh Alex - not entirely sure what's going on on your side but I wrote a testcase with 20k rows which I then executed on a Genymotion simulator using basic non-native Android implementation, not even a device. I think timing is quite good. All in all, fetching 20k rows took ~ 4200 milliseconds.
Here is my table definition and timing results. I am sure the timing is going to be much better on a device.
Now I did once small improvement to the code I have to admit... :) Although I am not yet sure if that was the culprit
    tx.executeSql('CREATE TABLE IF NOT EXISTS User( '
        + 'name VARCHAR(55), '
        + 'country VARCHAR(55), '
        + 'id VARCHAR(10) PRIMARY KEY NOT NULL, '
        + 'email VARCHAR(30) );', []);

queryUser(tx) {
    console.log("Executing sql...");
    tx.executeSql('SELECT * from User', [],this.querySuccess,this.errorCB);
},

querySuccess(tx,results) {
    this.state.progress.push("Query completed");
    var len = results.rows.length;
    console.log(`rows retrieved: ${len}`)
    for (let i = 0; i < len; i++) {
        let row = results.rows.item(i);
    }
},

11-28 12:39:11.996 4086-9001/com.awesomeproject D/ReactNativeJS: 'SQLite.backgroundExecuteSqlBatch({"dbargs":{"dbname":"Test.db"},"executes":[{"qid":1111,"sql":"BEGIN","params":[]},{"qid":1111,"sql":"SELECT * from User","params":[]}]})'
11-28 12:39:12.125 4086-9005/com.awesomeproject W/CursorWindow: Window is full: requested allocation 48 bytes, free space 17 bytes, window size 2097152 bytes
11-28 12:39:12.438 4086-4098/com.awesomeproject I/art: Background sticky concurrent mark sweep GC freed 160895(4MB) AllocSpace objects, 0(0B) LOS objects, 6% free, 43MB/46MB, paused 3.372ms total 109.697ms
11-28 12:39:12.744 4086-4093/com.awesomeproject W/art: Suspending all threads took: 28.277ms
11-28 12:39:12.820 4086-4098/com.awesomeproject I/art: Background partial concurrent mark sweep GC freed 400565(13MB) AllocSpace objects, 0(0B) LOS objects, 31% free, 34MB/50MB, paused 3.633ms total 214.935ms
11-28 12:39:16.256 4086-9001/com.awesomeproject D/ReactNativeJS: 'rows retrieved: 20000'
11-28 12:39:16.286 4086-9001/com.awesomeproject D/ReactNativeJS: 'SQLite.backgroundExecuteSqlBatch({"dbargs":{"dbname":"Test.db"},"executes":[{"qid":1111,"sql":"COMMIT","params":[]}]})'

@barbarosh
The fix I applied seems imemtarial. These are the runtimes using the current codebase for pure Android plugin ~ 4500 millis. I am closing this issue.
11-28 12:57:38.672 16872-16987/com.awesomeproject D/ReactNativeJS: 'SQLite.backgroundExecuteSqlBatch({"dbargs":{"dbname":"Test.db"},"executes":[{"qid":1111,"sql":"BEGIN","params":[]},{"qid":1111,"sql":"SELECT * from User","params":[]}]})'
11-28 12:57:38.894 16872-17217/com.awesomeproject W/CursorWindow: Window is full: requested allocation 48 bytes, free space 17 bytes, window size 2097152 bytes
11-28 12:57:43.334 16872-16987/com.awesomeproject D/ReactNativeJS: 'rows retrieved: 20000'
11-28 12:57:43.359 16872-16987/com.awesomeproject D/ReactNativeJS: 'SQLite.backgroundExecuteSqlBatch({"dbargs":{"dbname":"Test.db"},"executes":[{"qid":1111,"sql":"COMMIT","params":[]}]})'

Running on device S5 Mini ...weird as it is slower than the simulator :) Overall still I think it is quite good..
 D/SensorManager: unregisterListener ::

@barbarosh - I eliminated all JSON conversions in the Android module which should improve the performance quite significantly. Would you be able to run your tests on the branch that has the changes?

@andpor Thanks for work! ) But I dont have access to the project which uses this module, so I cant test this branch. Sorry!


test-branch/execute-job/quoted-authorization/63ad5577-a292-46cf-95e9-b0abd17c4349

PR for testing 8fdc2ee5ff7661d88be756ab19160096d8d7b2f5

/canonical/self-hosted-runners/run-workflows 8fdc2ee5ff7661d88be756ab19160096d8d7b2f5


fix: Return shallow copy of Metadata

Description
Noticed that a test would fail under parallel execution: https://github.com/amp-labs/connectors/actions/runs/12360650495/job/34496076253?pr=1220#step:5:1728
Fix
Located an issue shared by connectors that create *common.ListObjectMetadataResult relying on shared static schema. A copy of the map is now used to construct object metadata.

#1221  👈 (View in Graphite)
main

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

I am not sure why the linter started complaining. Running locally the correct version without any reported issues.
golangci-lint --version
golangci-lint has version v1.62.0 built with go1.23.3 from (unknown, modified: ?, mod sum: "h1:/G0g+bi1BhmGJqLdNQkKBWjcim8HjOPc4tsKuHDOhcI=") on (unknown)

Tried to download tar with this program and it also doesn't give an error (I ran it with cleared cache).
Linter logs (screenshot below)

@eberle1080

There must be a github bug. Either that or your branch is way out of date. Check .golangci.yml in the root directory, it's already set to colored-line-number.

FWIW I'm suddenly having failures as well, unrelated to my PR. See https://github.com/amp-labs/connectors/pull/1222. Like you, I can't repro these issues locally.

FWIW I'm suddenly having failures as well, unrelated to my PR. See #1222. Like you, I can't repro these issues locally.

Ok, so I added nolinter on the line that Github action is suggesting (I usually overlook this, but it was right there, however locally I dont get this message and the code of issue is prexisting).

Anyway, this PR has a valid lint action now. Once this is merged all the test issues that you have on your PR and also on main would disappear as I fixed the bug. (The implecations of this bug I assume are not critical, but it was wrong nonetheless).
@eberle1080


Add the sanity check for docker image

We'd better add some sanity checks in:
https://github.com/apache/kvrocks/blob/7f0c5156493dc3601bbc6f2157f9fa1d8462ea51/.github/workflows/kvrocks.yaml#L292-L309
Originally posted by @tisonkun in https://github.com/apache/kvrocks/issues/1784#issuecomment-1735890155
As mentioned in the comments, we at least check if the docker listening port can be accessed from outside.

How about using the following commands? @git-hulk
id="$(docker run --rm -d -p 6666:6666 kvrocks:ci)"
sleep 1m # waiting for the periodic container health check

if [ "$(docker inspect --format='{{.State.Health.Status}}' $id)" != "healthy" ]; then
  echo "Container is not healthy."
  exit 1
fi

if [ "$(ss --listening --no-header --tcp '( sport = :6666 )')" == "" ]; then
  echo "The container listening port can not be accessed from outside."
  exit 1
fi

docker stop $id

@jyf111 Looks good, thank you.


Add typings to package.json

"typings" key is required in package.json so that tsc knows where to find the typings for this module

Good call thanks !


No Space in Device for server in a VM using Hyper-V in windows

I faced this issue when I'm trying to install the server side into VM created by Hyper-V
The error message is attached below
failed to register layer: Error processing tar file(exit status 1): write /usr/local/lib/python3.6/dist-packages/tensorflow_core/include/google/protobuf/util/internal/expecting_objectwriter.h: no space left on device
I adjust the hard disk maximum size to be 50 Gb , but it seems like it has to do something with a volume is full.  It is a new created VM so nothing is added besides the requirement for the server side.
Any idea or suggestion how to solve that, thank  you

Hi @AbdelsalamHaa, thank you for Trying out BrainFrame!
Because this repository is for the BrainFrame Client specifically, and this appears to be more of a server question, I went ahead and created a forum post with your question here.
Please feel free to follow up on the forum post I created.


intermittent failure in test_ping

Failed in CI, here.
Error output:
 >           assert (pings_from_scada + pings_from_atn) >= exp_pings_nominal, err_str
E           AssertionError: pings_from_atn: 2
E             messages_from_atn: 12
E             pings_from_scada: 2
E             messages_from_scada: 20
E             exp_pings_nominal: 5.0
E
E           assert (2 + 2) >= 5.0

tests/test_proactor/test_proactor_comm.py:924: AssertionError

This looks like an ill-controlled test; it appears more messages were sent during the wait than expected, so less pings were necessary.

Also here  in gridworks-proactor