depotdownloader: 2.7.3 -> 2.7.4

2.7.{2,3} are actually unfit for purpose within a nix sandbox (https://github.com/SteamRE/DepotDownloader/issues/566#issuecomment-2496258753), so it would be nice to get this into 24.11 too, otherwise that's a regression between releases.

Things done

Built on platform(s)

[X] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[X] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[X] Tested basic functionality of all binary files (usually in ./result/bin/)
25.05 Release Notes (or backporting 24.11 and 25.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[X] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-already-reviewed/2617/2134


Always show "Real Name" instead of account name (S)

Summary
As a DAO user I would like to always see the users "real name" (not the telos account name) consistent throughout the DAO
AC

[ ] Always show the "real name" instead of the telos account name (e.g. show Irving Ulises Puga Gutiérrez instead of @pugaknzero15) in the following places:
[ ] https://dao.hypha.earth/hypha/proposals/ (keep the style light grey and small font)
[ ] https://dao.hypha.earth/hypha/ "New Member Widget"

@pugakn i still see the telos name on the proposal list cards:

https://dao-dev.hypha.earth/hypha/proposals/


Refactor/circular loader

Remove nprogress package
Refactor SuspenseLoader component to Loader
Add next routing loading component to app root

Closes #144

@heronlancellot can you review pls

hey @diegosano  thanks for your contribution ! gj


[bug] array.length is not supported

- var lessons = ["jade","js"]
if lessons
    if lessons.length>2
        p more than 2: #{lessons.join(',')}
    else if lessons.length >1
        p more than 1: #{lessons.join(',')}
    else
        p no1 lessons:#{lessons.length}
else
    p no2 lessons

output
no1 lessons:

currently only jexl syntax is supported. which is similar to javascript. in a future version you may have the possibility to switch to nashorn or graalvm. but thats currently very experimental and much slower than jexl.


Deallocate tensors after each iteration

I am curious if it is sufficient to do this after every epoch? Or perhaps in another process? I don't think it will be ~that~ expensive to do deallocations, but I'd be interested to see the performance impact

I am curious if it is sufficient to do this after every epoch?

I would say no. Given swapping from backend_deallocate to garbage_collect by itself on every iteration made a difference in using less memory, waiting for the epoch will probably use too much memory.


There appears to be an error in symbol creation in the GOlr loader

Taken from another discussion, @cmungall writes:
The ... issue is a golr loading bug and could be assigned to @hdietze -- gene_association.goa_human uses SAA1 for P0DJI8, whereas for whatever reason it is ending up as SAA1_HUMAN in golr. See:
http://amigo.geneontology.org/amigo/gene_product/UniProtKB:P0DJI8
Assuming that this is not just a data problem, the expected fix would be that the symbol for UniProtKB:P0DJI8 would be "SAA1" instead of "SAA1_HUMAN".

Removed @hdietze as this is likely my code doing something odd.

Pinging on @cmungall.

This looks incredibly similar to the labels used by PAINT. Which method is
being called, that is which method of Bioentity ?
On Wed, May 6, 2015 at 4:32 PM, kltm<EMAIL_ADDRESS>wrote:

Pinging on @cmungall https://github.com/cmungall.
—
Reply to this email directly or view it on GitHub
https://github.com/geneontology/amigo/issues/196#issuecomment-99643617.

zgrep -c P0DJI8 *.gz
...
gene_association.goa_human.gz:43
gene_association.goa_ref_human.gz:43
gene_association.goa_uniprot_noiea.gz:41
gene_association.mgi.gz:6
gene_association.reactome.gz:17

Looking, is seems reactome uses the SAA1_HUMAN symbol. Who wins?

This is looking rather data. However, it would be nice to make sure that whatever symbol gets left out of becoming the "one" should be added to the synonyms list.

Who knows, but the information seems to be "correct" now in AmiGO production, so chalking it up to data and closing.


Add test.go

Other areas to explore:

Follow symlink, and explore other file types
Explore Go templates for the templating
Explore Goroutines (get contents of dirs in parallel, and then combine the results)
Explore the built in data structures
Explore channels, to combine the output of the Goroutines in #3.

Why is this repo using the continuous-integration/symfony/pr check?

Rename the file, because it is conventional to have files containing tests to end in _test in Golang. The name test.go is misleading.

By Islam:
Sounds good, there is another thing I noticed, is loadPage() doesnt return an error, only returns a page, hmmm, I would say it could be renamed to loadFile msln it returns a file or and error, and in the handler ubdo what u need with the error, that way u can support json or xml later on
Right now its tightly coupled to html, and there is no way to know that an error happend
Same for other functions they are returning pages, and no way to know about errors, I would separate the logic from the presentation layer, presentation could be “html, json, xml ... etc”


unsigned trx

Hi Guys
I have question regarding create trx:
when I want to make unsigned trx with 'methods.balances.transferKeepAlive' I need some property like genesisHash,
metadataRpc ,... that I should fetch them from node and it takes time, This time may be longer than 5 seconds and we can not send the transaction in the current block, how can we make sure that the transaction will send in the current block.
thanks in advance

Hey @regazeri,
Question: What is the idea behind including a transaction at the block it was constructed at? Also, what are you using the fetch the information, and are you querying a public node or local one?
The idea being that a transaction is valid for future blocks that contain the same runtimeVersion, so you shouldn't have to get it in the same block. When submitting a transaction there is also no guarantee that the block author will include it at that block, or that it will gossiped to the block author in time.

thanks for your comment, it was helpful.


glass should support multiple s in a single document

https://github.com/glass-lang/glass/commit/7ba2033fdeff1e89b6ffcba914d05c9c91d37314


Corrected descriptions of network endpoints of JSON RPC

Note: I moved the "Node Status" section to be the first since it is the most useful one

@thisisjoshford Wow, thanks for the review and merge! That was blazing fast! :zap:


fix dead link to token whitepaper

Hi,
When reading the doc i notice this link was not working.
I m not sure if it is temporary or if the whitepapers have been moved from "https://files.avalabs.org/" to "https://www.avalabs.org/whitepapers"
I did not use the direct link to the token whitepaper "https://assets.website-files.com/5d80307810123f5ffbb34d6e/6008d7bc56430d6b8792b8d1_Avalanche Native Token Dynamics.pdf" because it is an external domain and looks like a randomly generated link that is probably regularly changing to avoid scraping (my guess) so i rather use the link sending to all whitepaper page.
Hope it helps other people. Charles.

adding some screenshot for reference:

@charly37 please merge the latest from master into your branch and resolve any conflicts. Thank you! 🎩

Closing in favor of https://github.com/ava-labs/avalanche-docs/pull/502. Thanks!


Manage the alignment

Summary
The component must take care of alignment atributes.
See also Babaylon doc.
Things to implement
horizontalAlignment

[ ] HORIZONTAL_ALIGNMENT_LEFT
[ ] HORIZONTAL_ALIGNMENT_CENTER
[ ] HORIZONTAL_ALIGNMENT_RIGHT

verticalAlignment

[ ] VERTICAL_ALIGNMENT_LEFT
[ ] VERTICAL_ALIGNMENT_CENTER
[ ] VERTICAL_ALIGNMENT_RIGHT

Alignment is already take in count:

From this local PG:
sample(2).zip


Filter measurements with the available parameters

@Ndacyayisenga-droid why do you not merge this one?

@Ndacyayisenga-droid why do you not merge this one?

A few adjustments I want to make to the frontend code then we get it merged!


RPi Desktop version for Pi4 emulation?

Hey there,
Thank you for all the hard work you put into this and the documentation.
I'm trying to emulate my RPi4 on my windows 10 desktop and am wondering if is there a .iso version of this that I could run as a virtual machine using Hyper-V or VirtualBox? Should I try to do it the qemu way or find a way to convert your RPi_64OS_DNN.img to .iso then mount use virtualbox (worried the difference in arm and amd architectures for the compiled software)?

I'm sorry @Bukkster, there is no ISO version you can run on Windows computer.
The only way I can think of is by manually installing different frameworks and software.

Dang that's a bummer, thanks for the quick response!


Design document for the Amazon AWS SNS connector

Description:
$subject

The design document can be found at
module-ballerinax-aws.sns - Research & Design


fix(protocol-designer): auto-generate trashBin for flex if no pipetting commands exist

closes AUTH-267
Overview
In PD in production, if you create a Flex protocol with no pipetting steps and reupload it to the app, it does not auto generate a trash bin for you. Technically there is a way around it by adding a trash bin entity. But its a weird state because return tip and no trash entity is not supported in PD.
Ot-2 does not have this issue because it always ensures a hardcoded fixed trash is generated
So this PR auto-generates a trash bin for a flex if there is none when importing a protocol.
Test Plan
I created a test case to test all scenarios, so it should work as expected.
However, you can create a flex protocol and add a pause step. Download the protocol and reimport it back to PD. See that the trash bin is generated.
Changelog

create util for getting an unoccupied slot, taking into account modules, labwares, move labware commands, waste chute, and staging area slots
add test

Review requests
see test plan
Risk assessment
low

question
when I added waste chute, export and import the protocol, trash bin is generated but waste chute is gone.
Is this expected behavior?

question
when I added waste chute, export and import the protocol, trash bin is generated but waste chute is gone. Is this expected behavior?

@koji oops that is definitely a bug! Thanks for catching it.

@koji actually, if you did not add any waste chute commands, it is not a bug. But if you did add waste chute commands, it is a bug.

question
when I added waste chute, export and import the protocol, trash bin is generated but waste chute is gone. Is this expected behavior?

@koji oops that is definitely a bug! Thanks for catching it.

@koji actually, if you did not add any waste chute commands, it is not a bug since we remove any staging area slot and waste chute that is not used in the protocol. But if you did add waste chute commands, it is a bug.

@jerader I added waste chute from the initial setup wizard.

one more question
I compared two json.
left: exported json - no trash bin
right: import the left json and re-export (trash bin with auto generated)
making pipetteTiprackAssignments item nested (adding []) is an expected behavior?

left
no-trashbin.json
right
no-trashbin (1).json

one more question
I compared two json. left: exported json - no trash bin right: import the left json and re-export (trash bin with auto generated)
making pipetteTiprackAssignments item nested (adding []) is an expected behavior?
left no-trashbin.json
right no-trashbin (1).json

@koji  oh i think what's happening is when you reimport, it goes through the 8.1 migration because the version when you export is 8.0.3 and not 8.1.0 - this is a current limitation with PD where the versioning doesn't update until i push a release tag 😦 so for now we have to manually change the version before we reimport.

question
when I added waste chute, export and import the protocol, trash bin is generated but waste chute is gone. Is this expected behavior?

@koji oops that is definitely a bug! Thanks for catching it.

@koji actually, if you did not add any waste chute commands, it is not a bug since we remove any staging area slot and waste chute that is not used in the protocol. But if you did add waste chute commands, it is a bug.

@jerader I added waste chute from the initial setup wizard.

@koji got it, this sounds like expected behavior. if you didn't add any step using the waste chute then it isn't being used in the protocol and won't generate upon reimport or even if you upload it to the app.


Remove use of depricated sudo module

Update scripts for ansible 2.x as outlined here:
http://docs.ansible.com/ansible/become.html
become: yes
become_method: sudo

@mark-cooper updated based on your feedback to remove sudo references.


Offboard control: Support straight actuator pass-through

We should add a board config which just takes in RC and sensors and publishes actuator controls straight.

We have this now for Snapdragon.


Unique path patterns

Hi @tomnomnom,
The tool is great and very helpful.
I have one feature request for consideration of future releases. Frequently the source are wayback urls, which have many similar unique urls but don't make a difference for security testing.
For example, I would only want to include one of the following two urls:
https://www.example.com/path?param1=a&param2=b&param3=c
https://www.example.com/path?param1=c&param2=d&param3=e

However, the following is to be included:
https://www.example.com/path?param1=a&param2=b
https://www.example.com/path?param4=a&param5=b&param6=c
https://www.example.com/path2?param1=a&param2=b&param3=c
https://www.example2.com/path?param1=a&param2=b&param3=c

That is, only unique combinations of domain, path, and keys are to be included (not values of keys). This will help to reduce processing time of subsequent testing steps.
Does this make sense? Please advise. Thanks.

That does make sense! This is actually possible already with the qsreplace tool, by supplying the -a flag with no parameter (see here for an example.
Part of me has considered rolling qsreplace's functionality into unfurl, so I will give that some more thought. Thanks!


Use more types at API boundaries

This is a placeholder issue for adding more types at API boundaries (return types of methods, parameter types for methods, type parameters, ...).

Closed with #780 and #771.


Update GitHub Desktop supported editors

Why:
The list of supported editors by GitHub Desktop is out of date.
What's being changed:
The list with editors is updated with the latest supported editors, based on https://github.com/desktop/desktop/blob/development/docs/technical/editor-integration.md.
Check off the following:

[x] I have reviewed my changes in staging (look for "Automatically generated comment" and click Modified to view your latest changes).
[x] For content changes, I have completed the self-review checklist.

Writer impact (This section is for GitHub staff members only):

[ ] This pull request impacts the contribution experience

[ ] I have added the 'writer impact' label
[ ] I have added a description and/or a video demo of the changes below (e.g. a "before and after video")

I'll review this!


update cask in readme: squirrel-for-simulator

update cask in readme: squirrel-for-simulator

Thank you!


WriteFile doesn't truncate file

In case the new content is smaller than previous, you can have relicas in written file, for exemple:
old content: "Hello World"
write: "test"
new content: "testo World"

I had a very specific use case for this library for which it was ok but you are correct. Will be fixed.

I had a very specific use case for this library for which it was ok but you are correct. Will be fixed.

Can you share your testing code?
use tfio::*;
use std::io;

fn main() -> io::Result<()> {
	let temp_dir = "./tmp";
	let mut tr = Transaction::new()
			.create_file("./foo.txt")
			.write_file("./foo.txt", temp_dir, b"Hello World".to_vec())
			.write_file("./foo.txt", temp_dir, b"test".to_vec());

	if let Err(e) = tr.execute() {
		eprintln!("{:?}", e);

		if let Err(ee) = tr.rollback() {
			panic!("Error during transaction: {}", ee);
		}
	}

	Ok(())
}

and checked foo.txt
$ cat foo.txt
testo World

Can you share your testing code?
use tfio::*;
use std::io;

fn main() -> io::Result<()> {
	let temp_dir = "./tmp";
	let mut tr = Transaction::new()
			.create_file("./foo.txt")
			.write_file("./foo.txt", temp_dir, b"Hello World".to_vec())
			.write_file("./foo.txt", temp_dir, b"test".to_vec());

	if let Err(e) = tr.execute() {
		eprintln!("{:?}", e);

		if let Err(ee) = tr.rollback() {
			panic!("Error during transaction: {}", ee);
		}
	}

	Ok(())
}

and checked foo.txt
$ cat foo.txt
testo World

Yup sure, almost the same code as yours, but with new lines.
use tfio::*;
use std::io;

fn main() -> io::Result<()> {
    let temp_dir = "./tmp";
    let mut tr = Transaction::new()
            .create_file("./foo.txt")
            .write_file("./foo.txt", temp_dir, b"Hello World !\n".to_vec())
            .write_file("./foo.txt", temp_dir, b"test\n".to_vec());

    if let Err(e) = tr.execute() {
        eprintln!("{:?}", e);

        if let Err(ee) = tr.rollback() {
            panic!("Error during transaction: {}", ee);
        }
    }

    Ok(())
}

$ cat foo.txt
test
 World !

Yup sure, almost the same code as yours, but with new lines.
use tfio::*;
use std::io;

fn main() -> io::Result<()> {
    let temp_dir = "./tmp";
    let mut tr = Transaction::new()
            .create_file("./foo.txt")
            .write_file("./foo.txt", temp_dir, b"Hello World !\n".to_vec())
            .write_file("./foo.txt", temp_dir, b"test\n".to_vec());

    if let Err(e) = tr.execute() {
        eprintln!("{:?}", e);

        if let Err(ee) = tr.rollback() {
            panic!("Error during transaction: {}", ee);
        }
    }

    Ok(())
}

$ cat foo.txt
test
 World !

Looks OK to me. What is your desired output?

Looks OK to me. What is your desired output?

Oh yes sorry, I mesread your comment, I did expect that write_file overwrite the entire file...
Maybe a better behaviour is to let the API user to decide or not to truncate the file ?

Oh yes sorry, I mesread your comment, I did expect that write_file overwrite the entire file...
Maybe a better behaviour is to let the API user to decide or not to truncate the file ?

That can maybe go under a new Operation as truncate_file I suppose

That can maybe go under a new Operation as truncate_file I suppose

Maybe yes, or maybe write_file operation could take another argument to specify how to open the file ?
Both are good for me.
But I think there is a refacto to do, indeed the code for WriteFile and AppendFile is almost the same, they only differ in the file OpenOptions.
For a new operation TruncateFile the code will again be the same...
Maybe we can only have an operation WriteFile that takes a enum like this to open the file ?
enum WriteMode {
    Truncate,
    Append,
    AtBegin,
}

Maybe yes, or maybe write_file operation could take another argument to specify how to open the file ?
Both are good for me.
But I think there is a refacto to do, indeed the code for WriteFile and AppendFile is almost the same, they only differ in the file OpenOptions.
For a new operation TruncateFile the code will again be the same...
Maybe we can only have an operation WriteFile that takes a enum like this to open the file ?
enum WriteMode {
    Truncate,
    Append,
    AtBegin,
}

Makes sense.

Makes sense.


Rename IconButtonComponent into LinkComponent and made icon optional

I think the "Få hjelp til frakt" button actually has an icon (the little van).

@ulrikus It’s just an example :)


Update definition.md

Made the language uniform
Summary
Describe your changes here.
Fixes #Issue_Number (if available)

Looks good. Thanks. Please merge


!!!DO NOT MERGE!!!

@kashif-m Can we close this in favor of #1665 or do we need this?

@kashif-m Can we close this in favor of #1665 or do we need this?

We can close this.


Made default and collation attributes work better with attribute subtypes

Fixed the code that deals with the DefaultAttribute and the CollationAttribute to allow subtypes of them to be used,
Also added some tests to demonstrate that it works and provide some motivation for why this is a useful feature.

Hi man, thanks for the recent fixes.
This looks a bit messy, is it ready for review?

Hi I'm likely to be using it on a project soon and I want to fix some of
the problems before I use it in anger.
The code is ready for review but I'm just starting with git so the commits
are messy. I can clean them up first if you want.
On Sun, 1 Mar 2015 11:29 Øystein Krog<EMAIL_ADDRESS>wrote:

Hi man, thanks for the recent fixes.
This looks a bit messy, is it ready for review?
—
Reply to this email directly or view it on GitHub
https://github.com/oysteinkrog/SQLite.Net-PCL/pull/131#issuecomment-76592916
.

I would appreciate that, I'm trying to keep things as clean as possible.

I'm closing this one and have made another one.


MAP-1459 5 ReportService Scenario include Statements and Form report body

As this scenario wasn't tested covered previously,
RequestParam(name = "includeStatements" &  "includeStatements"
This method wasn't hit with this PR is covered
getReportDetailByOffenderNumber(offenderNumber).map { it.toDto(includeStatements, includeFormResponse) }
before

after

this commit has been added in other Pull Request so I close it

this commit has been added in other Pull Request so I close it


Use actual sampler settings from glTF file

At the moment we don't read or use any sampler settings.
Mentioned in #18.

Time for a new label, perhaps...


How to customize the name of PDF reporter

#Hello,
I have seen the issue of "how to customize the format of PDF reporter"(https://github.com/volkanceylan/Serenity/issues/2139 ),I tried it, and it worked.
But I don't know how to customize the name of PDF reporter.I didn't find the field of interface.
Could you tell me how to do it?
Thank you very much!

You mean the name of the output report for when exporting the pdf?
https://github.com/volkanceylan/Serenity/wiki/Adding-PDF-Export

I 'm so sorry that I didn't explain it clearly. I know how to set up the common  output PDF of the Button.
In fact, I mean this one :
 Serene7.Common.ReportHelper.execute({                     reportKey: 'Nomination.NominateionLetterReport',                     download: true,                     params: {                         NominationId: item.NominationId                     }                 });

But,I didn't find the field of interface in ReportHelper.cs.
 export interface ReportExecuteOptions {         reportKey: string;         download?: boolean;         extension?: 'pdf' | 'htm' | 'html' | 'xlsx' | 'docx';         getParams?: () => any;         params?: { [key: string]: any }         target?: string;     }

In ReportController.cs inc Common/Reporting there is this code
        private ActionResult PrepareFileResult(IReport report, string ext, bool download,
            byte[] renderedBytes, ReportRegistry.Report reportInfo)
        {
            string fileDownloadName;
            var customFileName = report as ICustomFileName;
            if (customFileName != null)
                fileDownloadName = customFileName.GetFileName();
            else
                fileDownloadName = (reportInfo.Title ?? reportInfo.Key) + "_" +
                    DateTime.Now.ToString("yyyyMMdd_HHss");

            fileDownloadName += "." + ext;

         [...]

So, I not try, but if you implement ICustomFileName in your Report class and override GetFileName() you can set the output file name.

Thank you for your answer. That's exactly what I want. I just don't know where this interface is .
Thanks very much


[fire_core] Flutter app won't build appbundle/apk or run on android

Basically my app works perfectly on iOS and it builds but for Android it won't work i tried anything and i don't know what to do anymore.
Here is my "flutter doctor -v" :
[✓] Flutter (Channel unknown, v1.9.1+hotfix.5, on Mac OS X 10.15 19A602, locale en-FR)
    • Flutter version 1.9.1+hotfix.5 at /Users/axel/flutter
    • Framework revision 1aedbb1835 (6 days ago), 2019-10-17 08:37:27 -0700
    • Engine revision b863200c37
    • Dart version 2.5.0

[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.1)
    • Android SDK at /Users/axel/Library/Android/sdk
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-29, build-tools 29.0.1
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b49-5587405)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 11.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 11.1, Build version 11A1027
    • CocoaPods version 1.8.4

[✓] Android Studio (version 3.5)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 40.2.2
    • Dart plugin version 191.8593
    • Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b49-5587405)

[✓] VS Code (version 1.39.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.5.1

[✓] Connected device (1 available)
    • Android SDK built for x86 • emulator-5554 • android-x86 • Android 10 (API 29) (emulator)

• No issues found!

Then here is "my flutter run -v" logs  :
[+4191 ms] > Task :app:transformDexArchiveWithExternalLibsDexMergerForDebug FAILED
[        ]          *********************************************************
[        ] WARNING: This version of firebase_core will break your Android build if it or its dependencies aren't compatible with AndroidX.
[        ]          See https://goo.gl/CP92wY for more information on the problem and how to fix it.
[        ]          This warning prints for all Android build failures. The real root cause of the error may be unrelated.
[        ]          *********************************************************
[        ] 336 actionable tasks: 7 executed, 329 up-to-date
[   +1 ms] D8: Cannot fit requested classes in a single dex file (# methods: 74630 > 65536)
[ +533 ms] Running Gradle task 'assembleDebug'... (completed in 26.5s)
[   +7 ms] "flutter run" took 80,576ms.
Gradle task assembleDebug failed with exit code 1

#0      throwToolExit (package:flutter_tools/src/base/common.dart:28:3)
#1      _buildGradleProjectV2 (package:flutter_tools/src/android/gradle.dart:751:5)
<asynchronous suspension>
#2      buildGradleProject (package:flutter_tools/src/android/gradle.dart:494:14)
<asynchronous suspension>
#3      buildApk (package:flutter_tools/src/android/apk.dart:34:9)
<asynchronous suspension>
#4      AndroidDevice.startApp (package:flutter_tools/src/android/android_device.dart:486:13)
<asynchronous suspension>
#5      FlutterDevice.runHot (package:flutter_tools/src/resident_runner.dart:378:54)
<asynchronous suspension>
#6      HotRunner.run (package:flutter_tools/src/run_hot.dart:254:39)
<asynchronous suspension>
#7      RunCommand.runCommand (package:flutter_tools/src/commands/run.dart:472:37)
<asynchronous suspension>
#8      FlutterCommand.verifyThenRunCommand (package:flutter_tools/src/runner/flutter_command.dart:490:18)
<asynchronous suspension>
#9      FlutterCommand.run.<anonymous closure> (package:flutter_tools/src/runner/flutter_command.dart:407:33)
<asynchronous suspension>
#10     AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:154:29)
<asynchronous suspension>
#11     _rootRun (dart:async/zone.dart:1124:13)
#12     _CustomZone.run (dart:async/zone.dart:1021:19)
#13     _runZoned (dart:async/zone.dart:1516:10)
#14     runZoned (dart:async/zone.dart:1463:12)
#15     AppContext.run (package:flutter_tools/src/base/context.dart:153:18)
<asynchronous suspension>
#16     FlutterCommand.run (package:flutter_tools/src/runner/flutter_command.dart:397:20)
#17     CommandRunner.runCommand (package:args/command_runner.dart:197:27)
<asynchronous suspension>
#18     FlutterCommandRunner.runCommand.<anonymous closure> (package:flutter_tools/src/runner/flutter_command_runner.dart:402:21)
<asynchronous suspension>
#19     AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:154:29)
<asynchronous suspension>
#20     _rootRun (dart:async/zone.dart:1124:13)
#21     _CustomZone.run (dart:async/zone.dart:1021:19)
#22     _runZoned (dart:async/zone.dart:1516:10)
#23     runZoned (dart:async/zone.dart:1463:12)
#24     AppContext.run (package:flutter_tools/src/base/context.dart:153:18)
<asynchronous suspension>
#25     FlutterCommandRunner.runCommand (package:flutter_tools/src/runner/flutter_command_runner.dart:356:19)
<asynchronous suspension>
#26     CommandRunner.run.<anonymous closure> (package:args/command_runner.dart:112:25)
#27     new Future.sync (dart:async/future.dart:224:31)
#28     CommandRunner.run (package:args/command_runner.dart:112:14)
#29     FlutterCommandRunner.run (package:flutter_tools/src/runner/flutter_command_runner.dart:242:18)
#30     run.<anonymous closure>.<anonymous closure> (package:flutter_tools/runner.dart:63:22)
<asynchronous suspension>
#31     _rootRun (dart:async/zone.dart:1124:13)
#32     _CustomZone.run (dart:async/zone.dart:1021:19)
#33     _runZoned (dart:async/zone.dart:1516:10)
#34     runZoned (dart:async/zone.dart:1500:12)
#35     run.<anonymous closure> (package:flutter_tools/runner.dart:61:18)
<asynchronous suspension>
#36     AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:154:29)
<asynchronous suspension>
#37     _rootRun (dart:async/zone.dart:1124:13)
#38     _CustomZone.run (dart:async/zone.dart:1021:19)
#39     _runZoned (dart:async/zone.dart:1516:10)
#40     runZoned (dart:async/zone.dart:1463:12)
#41     AppContext.run (package:flutter_tools/src/base/context.dart:153:18)
<asynchronous suspension>
#42     runInContext (package:flutter_tools/src/context_runner.dart:59:24)
<asynchronous suspension>
#43     run (package:flutter_tools/runner.dart:50:10)
#44     main (package:flutter_tools/executable.dart:65:9)
<asynchronous suspension>
#45     main (file:///Users/axel/flutter/packages/flutter_tools/bin/flutter_tools.dart:8:3)
#46     _startIsolate.<anonymous closure> (dart:isolate-patch/isolate_patch.dart:303:32)
#47     _RawReceivePortImpl._handleMessage (dart:isolate-patch/isolate_patch.dart:172:12)

and finally there is my pubspec.yaml :
name: [secret]
description: [secret]

# The following defines the version and build number for your application.
# A version number is three numbers separated by dots, like 1.2.43
# followed by an optional build number separated by a +.
# Both the version and the builder number may be overridden in flutter
# build by specifying --build-name and --build-number, respectively.
# In Android, build-name is used as versionName while build-number used as versionCode.
# Read more about Android versioning at https://developer.android.com/studio/publish/versioning
# In iOS, build-name is used as CFBundleShortVersionString while build-number used as CFBundleVersion.
# Read more about iOS versioning at
# https://developer.apple.com/library/archive/documentation/General/Reference/InfoPlistKeyReference/Articles/CoreFoundationKeys.html
version: 0.1.43

environment:
  sdk: ">=2.1.0 <3.0.0"

dependencies:
  flutter:
    sdk: flutter
  flutter_localizations:
    sdk: flutter

  # The following adds the Cupertino Icons font to your application.
  # Use with the CupertinoIcons class for iOS style icons.
  cupertino_icons: ^0.1.2

  shared_preferences: ^0.5.2
  cached_network_image: ^1.1.1
  flutter_svg: ^0.13.1
  rflutter_alert: ^1.0.3
  flutter_bloc: ^0.11.1
  equatable: ^0.1.6
  flutter_secure_storage: ^3.2.1
  local_auth: ^0.6.0+1
  http_auth: ^0.2.5
  firebase_core: ^0.4.0+9
  firebase_messaging: ^5.1.6
  validate: ^1.7.0
  kiwi: ^0.1.0
  image_picker: ^0.6.1
  flutter_crashlytics: ^1.0.0
  flutter_sticky_header: ^0.4.0
  dio: ^2.1.0
  auto_size_text: ^1.1.1
  flutter_masked_text: ^0.8.0
  permission_handler: '^3.2.0'
  path_provider: ^0.5.0+1
  flutter_email_sender: ^2.0.0
  flutter_swiper: ^1.1.6
  contacts_service: ^0.2.8
  geolocator: ^5.1.1+1
  google_maps_flutter: ^0.5.20+1
  url_launcher: ^5.0.2
  qr_mobile_vision: ^0.2.2
  qr_flutter: ^2.0.0+51
  csv: ^4.0.3
  share_extend: ^1.0.8
  country_pickers: ^1.1.0
  diacritic: ^0.1.1
  recase: ^2.0.1
  keyboard_visibility: ^0.5.6

  flutter_worldpay:
    path: ../flutter_worldpay/

dev_dependencies:
  flutter_test:
    sdk: flutter

# For information on the generic Dart part of this file, see the
# following page: https://www.dartlang.org/tools/pub/pubspec

# The following section is specific to Flutter.
flutter:

  # The following line ensures that the Material Icons font is
  # included with your application, so that you can use the icons in
  # the material Icons class.
  uses-material-design: true

  # To add assets to your application, add an assets section, like this:
  assets:
   - assets/images/
   - assets/icons/
  #  - images/a_dot_ham.jpeg

  # An image asset can refer to one or more resolution-specific "variants", see
  # https://flutter.io/assets-and-images/#resolution-aware.

  # For details regarding adding assets from package dependencies, see
  # https://flutter.io/assets-and-images/#from-packages

  # To add custom fonts to your application, add a fonts section here,
  # in this "flutter" section. Each entry in this list should have a
  # "family" key with the font family name, and a "fonts" key with a
  # list giving the asset and other descriptors for the font. For
  # example:
  # fonts:
  #   - family: Schyler
  #     fonts:
  #       - asset: fonts/Schyler-Regular.ttf
  #       - asset: fonts/Schyler-Italic.ttf
  #         style: italic
  #   - family: Trajan Pro
  #     fonts:
  #       - asset: fonts/TrajanPro.ttf
  #       - asset: fonts/TrajanPro_Bold.ttf
  #         weight: 700
  #
  # For details regarding fonts from package dependencies,
  # see https://flutter.io/custom-fonts/#from-packages
  fonts:
    - family: Lato
      fonts:
        - asset: assets/fonts/Lato.ttf
        - asset: assets/fonts/Lato-Light.ttf
          weight: 300
        - asset: assets/fonts/Lato-Bold.ttf
          weight: 700
    - family: Nunito
      fonts:
        - asset: assets/fonts/Nunito.ttf
        - asset: assets/fonts/Nunito-Light.ttf
          weight: 300
        - asset: assets/fonts/Nunito-Bold.ttf
          weight: 700

I forgot to say that it happens since I updated android Studio to 3.5, before that I could run the app on android but still not build it.

Hi @agnoam
I see there's an open issue addressing the case you described; even though it might refers to a different FlutterFire plugin
it describes the same Dex related  D8: Cannot fit requested classes in a single dex file.
Please follow up on that issue,
I'm closing the current one as duplicate.
If you disagree please write in the comments
and I will reopen it.
Thank you

@iapicca I updated this post I don't have the D8 Error message anymore but the same fire_core problem
I'd like it to be reopen !


FEAT Implemented flights screen

Breaking implementation of base flights screen, project structure updates, and quality of code updates

[x] Defined base flight data map
[x] Implemented base flights screen
[x] Project structure updates and quality of code updates
[x] Bug-fixes

Breaking implementation of viewmodels and other screens, I'll still have to update the flights list map in another PR. All in all, a lot of changes and modifications, most of which change core functionality.


Outputs a log of updated features

Generates a json file listing updates made during a run of updated-bcd. Primarily for further use in BCD update automation.
We're initially just capturing the BCD path, browser and updated support statements in the log file, but there's a lot of additional data from the UpdateLog object that we can add as needed.
cc @teoli2003 @foolip
Closes #946
Sample output from a recent run:
[
  {
    "browser": "firefox_android",
    "path": "api.AudioProcessingEvent",
    "statements": [
      {
        "version_added": false
      }
    ]
  },
  {
    "browser": "firefox_android",
    "path": "api.AudioProcessingEvent.inputBuffer",
    "statements": [
      {
        "version_added": false
      }
    ]
  },
  {
    "browser": "firefox_android",
    "path": "api.AudioProcessingEvent.outputBuffer",
    "statements": [
      {
        "version_added": false
      }
    ]
  },
  ...
]

Looks good for me too!


Webpack throwing error on build

Do you want to request a feature or report a bug?
Bug
What is the current behavior?
Cannot use the library
If the current behavior is a bug, please provide the steps to reproduce and if possible a minimal demo of the problem via https://codesandbox.io/ or similar.
Import a module like Either or Option
When I load the page I get this error:
Uncaught Error: Cannot find module './ChainRec'
at webpackEmptyContext (app.js:11)
at vendors.js:9477
at vendors.js:9465
at Object../node_modules/fp-ts/lib/Either.js (vendors.js:9474)
at webpack_require (bootstrap:781)
at fn (bootstrap:149)
at Object. (app.js:5064)
at Object../src/middlewaire/airlineMiddleware.ts (app.js:5100)
at webpack_require (bootstrap:781)
at fn (bootstrap:149)
With these warnings:
./node_modules/fp-ts/lib/Either.js 14:24-31
Critical dependency: require function is used in a way in which dependencies cannot be statically extracted
./node_modules/fp-ts/lib/ChainRec.js 3:24-31
Critical dependency: require function is used in a way in which dependencies cannot be statically extracted
./node_modules/fp-ts/lib/function.js 3:24-31
Critical dependency: require function is used in a way in which dependencies cannot be statically extracted
./node_modules/fp-ts/lib/Setoid.js 3:24-31
Critical dependency: require function is used in a way in which dependencies cannot be statically extracted
What is the expected behavior?
I don't get a build error
Which versions of fp-ts, and which browser and OS are affected by this issue? Did this work in previous versions of fp-ts?
fp-ts: 1.14.0
typescript: 3.2.2
browsers affected: Chrome and Firefox
OS: Windows

We are experiencing the same issue with v1.14.0.

@TheWix I think that switching to UMD might be the culprit.
I'll revert that change and release a patch.
/cc @jjoekoullas

@TheWix @mavax patch released, could you please try it out?

@gcanti It works with v1.14.1, thanks!

Confirmed. Thanks @gcanti!

Nice, thanks for the report / feedback, closing.


Feature/collectd addon

[x] Installs cleanly on a fresh build of most recent master branch
[ ] Upgrades cleanly from the most recent release
[x] Updates documentation relevant to the changes
[ ] Rebases cleanly onto the latest master

Testing in progress, I just wanted to get this out here so we can have a discussion: this PR removes logstash's collectd forwarding behavior. Motivation is on 05c49f1. Does anyone have feedback of a better way to do this?

Unfortunately, this fails with our Zookeeper checks (which check for /etc/collectd.d/zookeeper.conf).

Looks good otherwise, the addon playbook installs correctly and all services remain healthy. We'll just have to remove that distributive check, which will be part of the distributive refactor that's on its way.

@siddharthist is there a way we could remove that check now? Or move it to the collectd checks right away?

It is installed via the monolithic distributive-rpm. If you merge my PRs on mantl-packaging, I can put the new distributive PR in right away, we can merge that and then this.
Or, we could remove distributive.yml from the zookeeper role altogether in the meantime.

Rebased onto latest master

This will be unblocked by #1268, which doesn't check for /etc/collectd.d/zookeeper.conf.

now that #1268 is in, we can probably get this merged as soon as the merge conflicts are resolved

Great, I’ll work on that today!

this passes most checks but fails on DO for some reason, for different failure scenarios. Weird.

rebased

@BrianHicks We've seen the most recent DO failure in CI builds before, is that what you were referring to? Is this ready to merge after we fix those variables that @ryane pointed out?

@BrianHicks are the mesos_agent_present mesos_master_present variables being used anywhere?

$ ag mesos_agent_present
roles/collectd/tasks/mesos-agent.yml
5:  register: mesos_agent_present

$ ag mesos_master_present
roles/collectd/tasks/mesos-master.yml
5:  register: mesos_master_present

so... no? Do you reckon they ought to be removed?

I kinda think so. they don't break anything but just so there is no confusion...

I changed that, squashed the commit, and rebased. If tests pass, I'll merge.

Well, they all failed, but not for related reasons.

tested successfully on gce


Upgrade blackbox exporter

How to categorize this PR?

/area monitoring
/kind enhancement
What this PR does / why we need it:
Upgrade blackbox exporter version.
Special notes for your reviewer:
/cc @istvanballok
Release note:

Upgrade blackbox exporter version to v0.22.0

/test pull-gardener-e2e-kind


KafkaAdmin inside KafkaTemplate only gets the BOOTSTRAP_SERVER_CONFIG from producerFactory

I encountered this issue in version 3.2.3, and it is also present in version 3.3.0-M3.
The afterSingletonsInstantiated method in KafkaTemplate retrieves the default KafkaAdmin instance (if it has not been overridden) and configures it using the producerFactory property BOOTSTRAP_SERVERS_CONFIG. However, we experienced an error when trying to send messages to Kafka due to the SECURITY_PROTOCOL_CONFIG, which defaults to PLAINTEXT instead of SSL, as required by our configuration. Below is the error we encountered while attempting to send a message:

"stack_trace":"java.util.concurrent.TimeoutException: null
  java.base/java.util.concurrent.CompletableFuture.timedGet(CompletableFuture.java:1960)
  java.base/java.util.concurrent.CompletableFuture.get(CompletableFuture.java:2095)
  org.apache.kafka.common.internals.KafkaFutureImpl.get(KafkaFutureImpl.java:180)
  org.springframework.kafka.core.KafkaAdmin.clusterId(KafkaAdmin.java:351)
  org.springframework.kafka.core.KafkaTemplate.clusterId(KafkaTemplate.java:530)
  org.springframework.kafka.support.micrometer.KafkaRecordSenderContext.<init>(KafkaRecordSenderContext.java:48)
  org.springframework.kafka.core.KafkaTemplate.lambda$observeSend$3(KafkaTemplate.java:788)
  io.micrometer.observation.Observation.createNotStarted(Observation.java:172)
  io.micrometer.observation.docs.ObservationDocumentation.observation(ObservationDocumentation.java:188)
  org.springframework.kafka.core.KafkaTemplate.observeSend(KafkaTemplate.java:786)
  org.springframework.kafka.core.KafkaTemplate.send(KafkaTemplate.java:596)

Steps to Reproduce

Configure the KafkaTemplate bean using the KafkaProducerFactory and set ObservationEnabled to true.
Attempt to send a message to Kafka.

Expected Behavior
The SECURITY_PROTOCOL_CONFIG should be retrieved from the producerFactory configuration, just like BOOTSTRAP_SERVERS_CONFIG.
Workaround
We resolved this issue by overriding the default KafkaAdmin with our custom configuration. However, since this bean is used internally by KafkaTemplate, we believe that SECURITY_PROTOCOL_CONFIG should be handled in the same way as BOOTSTRAP_SERVERS_CONFIG within the KafkaTemplate configuration.

Closed due to duplicated https://github.com/spring-projects/spring-kafka/issues/2780


fix(build): update imports to use web export from ao-sdk

:tada: This PR is included in version 1.0.2-alpha.4 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Issue #687: Add OwnedDrawData for deep cloning DrawData

Introduce an OwnedDrawData struct that holds a cheaply-cloned ImDrawData structure that has been allocated and cloned on the heap. Designed for cases where you need to keep hold of the draw data so it can be evaluated at a later point in time (for instance, in multithreaded renderers where rendering the previous frame is overlapped with updating the next frame). It leverages ImDrawList::CloneOutput for cloning the underlying draw lists, and all allocations are made via ImGui, which has a nice consequence of them being tracked in the IO object's allocation metrics. The OwnedDrawData struct provides an accessor to the underlying DrawData struct, so that it can still be passed to renderers without modifying them.
As discussed in Issue #687

Looks great, thanks!

Excellent, thank you!


RequireJS 2.1.15

Type: Missing
Summary:
RequireJS 2.1.15
Details:
NuGet license field links to MIT
GitHub license is MIT: https://github.com/requirejs/requirejs/blob/master/LICENSE
Resolution:
MIT
Affected definitions:

RequireJS 2.1.15

This version was dual-licensed at the time it was published: https://github.com/requirejs/requirejs/blob/2.1.15/LICENSE.
Updated to BSD-3-Clause OR MIT.


fix: update package.json to fix tsc

[ ] Tests written and linted ℹ
[ ] Documentation written in Wiki/README
[x] Commit history is tidy & follows Contributing guidelines ℹ

What this does
add tslib and fix tsc/typescript versions so build will succeed

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


TypeAnnotationsScanner only sees nested classes

So as a test to see if Reflections will work for us, I made an annotation of the simplest kind:
@Retention(RetentionPolicy.RUNTIME)
public @interface TestAnnotation
{
    public String value();
}

and used it in one of our existing classes, plus a quick added nested class:
@TestAnnotation("foo foo foo")
public class ActualFunctionalityClass
{
    @TestAnnotation("bar bar bar")
    class Thing {}

    .......
}

All of that gets built into a JAR on the local filesystem.  Now I'm trying to use Reflections 0.9.9-RC1 to examine that JAR file:
Path jar_file = ... code elsewhere that sets absolute pathname ...;

// This works:
URL url = jar_file.toUri().toURL();
// so does this:
//URL url = new URL("jar:file:" + jar_file.toString() + "!/");

Reflections r = new ConfigurationBuilder()
    //.addClassLoader(URLClassLoader.newInstance(new URL[]{url}))
    .setUrls(url)
    .setScanners(
        new FieldAnnotationsScanner(),
        new SubTypesScanner(false),
        new TypeAnnotationsScanner(),
        new TypeElementsScanner()
        )
    .build();

Multimap<String,String> mmap = r.getStore().get(TypeAnnotationsScanner.class);
for (String key : mmap.keySet()) {
    System.err.format("key %s:%n", key);
    for (String value : mmap.get(key)) {
        System.err.format("   -> %s%n", value);
    }
}

(The setUrls() call seems to be sufficient to get the JAR scanned; adding the classloader to look at the JAR that way doesn't make a difference as far as I can tell.  If I'm missing something crucial please let me know.)
So I got a dump of lots of expected things like
key java.lang.FunctionalInterface:
    -> ourproject.SomeClass$SomeInterfaceTaggedAsFunctional
    -> ourproject.OtherClass$SomeOtherInterfaceTaggedAsFunctional

as well as
key java.lang.annotation.Retention:
    -> ourproject.TestAnnotation
key ourproject.TestAnnotation:
    -> ourproject.ActualFunctionalityClass$Thing

But I also expected to see TestAnnotation mapping to ActualFunctionalityClass itself, in addition to the nested Thing class.
Is there something I'm missing here as to how the annotations are mapped?
(Eventually I'm going to be doing the scanning via getTypesAnnotatedWith rather than trawling through the store, but I wanted to try some simple cases first.)

as tested, this should work and return both upper and internal classes. please use latest version. provide a reproducible test/jar if issue still happens.


style: update coding style by php-cs-fixer v3.43.0

Description
Follow-up #8368

update php-cs-fixer
update coding style

Checklist:

[x] Securely signed commits
[] Component(s) with PHPDoc blocks, only if necessary or adds value
[] Unit testing, with >80% coverage
[x] User guide updated
[x] Conforms to style guide

This should apply to https://github.com/CodeIgniter/coding-standard

Added in https://github.com/CodeIgniter/coding-standard/commit/5c11c9b4f78e6c28d81ad9890e3fe585d95ba662


Examples for Three

there is a category for them in the presets. we can improve this later, eg. by creating an 'examples' user


Addition of Geosilhouettes

Thought it would be best if I gave you the opportunity to comment on this as I work @knaaptime.
Todo:

All this code is held together by running gdf.fillna(0) before running the clusterer, need to find solution that handles NaN values and keeps the index the same length as the dataframe.
Handle input when an aspatial model is passed to a spatial diagnostic

Currently throws what might be a cryptic error to most users 'NoneType' object has no attribute 'sparse', referring to the absence of a W object

Figure out why these functions return 3 axes when I think they should be returning 2. The third is dtype=object

cool, lets talk about this a bit during the meeting on tuesday. It'll be a little easier to describe what i mean about stashing the index so we dont have to fillna
Also i think it would be better to switch the ordering of the plotting nomenclature, e.g. instead of ***_plot(...), i think it would be better to do plot_***(...)

Handle input when an aspatial model is passed to a spatial diagnostic
Currently throws what might be a cryptic error to most users 'NoneType' object has no attribute 'sparse', referring to the absence of a W object

one thing we could do is add a little metadata attribute to the ModelResults class that stores whether the model is spatial or aspatial. That way we can add a simple check if users pass the wrong type of model to the wrong diagnostic

awesome! looks like this has everything we talked about yesterday. Everything working as expected?

As far as I can tell everything works as expected, and at this point I would designate this ready for review. However, this is such a big PR (+500 lines), and with your understanding of these statistics being better than mine, I would encourage you to test it out yourself before merging.
Let me know if these needs anymore changes.


Fixing a typo (-react-commons)

ozp-center, ozp-backend, and ozp-react-commons all have this typo and a PR to match. (Civilian vs Civillian)
@mannyrivera2010 @lsemesky @mleeBoeing @J-Fid

:+1:


Panasonic viera fix

Description:
Removed the "return False" from the setup_platform function after trying to connect to the Panasonic TV. This is to allow the the TV to be added even when it is turned off.
The other way I was thinking this could be done is to remove the try/except connection attempt and just do a validate_config check.
Related issue (if applicable): fixes #4147

Now we might as well remove the whole try…catch block as it serves no purpose ?

Right, I agree. I have removed the try/except block. Also noticed and removed a validate_config line which should not have been there.


Application access token generation

Description:
Step 9 of https://docs.wso2.com/display/OB140/Account+and+Transaction+API+v3.1.1 needs further clarification on why/how to generate the client assertion and also provide sample values for the json payload.
Suggested Labels:

Suggested Assignees:

Affected Product Version:
1.4.0
OS, DB, other environment details and versions:
Steps to reproduce:
Related Issues:

[ ] clarification on why/how to generate the client assertion
[ ] provide sample values for client assertion json payload


Thnking about names

"Single File Components" is the official name for the .vue component system, but it is not very googlable. Fixing that is a different conversation.
I'm wondering if a different name for this is due though:
How about a repo rename to:

RonnieSan/vuecompose.js
RonnieSan/DotVueToJs
RonnieSan/.vue.js
RonnieSan/.vue2js

And for the gen'd file?  How about a file suffix:

my-component.vue.js

(.vue)
dist/my-component.js

Do you think people can find your repo, if the've a haze recollection of the author?  I'm thinking they'll just google for "vue browser component" and we don't know how your repo will rank with that search term in a few months time.

I think giving it a branded name would be worse, since unless you know the branded name, you would be less likely to find it.  I want to keep it simple and descriptive, so maybe something along the lines of "vue-standalone-sfc" or "vue-browser-sfc" would make more sense?

I'm for that sure, it's more googlable.
Did you know that if you rename a GH repo, then incoming URLs to the old name are redirected to the new URL as you would hope?  That's even true of people pushing to the old repo via git@ssh :)


ServiceWorker behavior?

In your research have you explored the behavior when combined with ServiceWorkers? Possibly because it could remove the need for a server altogether by "spoofing" URLs under a specific prefix, but also because it can reduce round trip to the server by generating the response within the worker itself locally.

In your research have you explored the behavior when combined with ServiceWorkers? Possibly because it could remove the need for a server altogether by "spoofing" URLs under a specific prefix, but also because it can reduce round trip to the server by generating the response within the worker itself locally.

Could you elaborate on that idea? I don't understand exactly what you are referring to tbh.


Provide feedback to users about email delivery issues

Once https://github.com/pypa/warehouse/pull/3580 is merged, Warehouse will begin tracking delivery issues for email addresses we have on file, and can ultimately un-verify a particular email address due to those reasons. We'll want to expose those reasons to end users so that they understand what has occurred to their email addresses on file.
This gives us the following states an email address can be in:

Unverified because the email has had a hard failure during delivery.
Unverified because the email had too many transient delivery failures, and we've given up sending to that address and we're treating it as a hard failure.
Unverified because the user has marked an email from this address as spam, and their ISP or email provider has forwarded that complaint on to us.
Unverified because it is brand new and has yet to be verified:
Verified, but we've been having transient issues sending email to this address, if they don't go away this will be upgraded to the (2) case.
Verified, no issues.

I can handle wiring this up in the code once we have the design side sorted out for how it should be implemented.

@nlhkabu is this something you could work on, and is it amenable to work at the sprint at EuroPython? @yeraydiazdiaz do you have any thoughts?
Relevant to finishing #3632.

maybe we could use tooltips for this feedback? so it doesn't interfere with the action column


*: move ioload to tikv_util

Signed-off-by: Ryan Leung<EMAIL_ADDRESS>
What is changed and how it works?

Issue Number: Ref #13433.
What's Changed:
This PR moves ioload.rs to tikv_util.

Release note
None

/merge


Foreign Index Constraint Support

This commit supports foreign indexes so that foreign indexes can be added and dropped from tables. Since neither operation is destructive the AddConstraint and DropConstraint structs return false when their Unsafe() method is called.
The only thing to be aware of is, adding a foreign constraint to an existing column with data will NOT produce an error if any of the data does not match the referenced table. This is because (I assume) foreign key checks are temporarily disabled.
Evan I would be interested in your feedback on this change and whether you think my simple implementation will work in most use cases OR if there should be additional systems in place to safely facilitate foreign key constraints.

Thank you for the excellent pull request! It will be wonderful to have support for foreign key constraints in Skeema -- this is really a great contribution, and organizationally it fits in well with the existing code.
I've reviewed the code and performed some manual testing. This comment will look long but I promise it's nothing too major :)
Dependency vendoring: Some of Skeema's functionality is split into a couple separate github repos, for modularity purposes. In the main skeema repo, these are duplicated as frozen dependencies in the vendor dir, which I use the third-party godep tool to maintain.
The schema introspection and diff logic is actually in Go La Tengo, so this pull request (PR) should really happen there first. And then once merged to master there, we can do a separate PR here in skeema, which just commits the effect of godep update github.com/skeema/tengo.
I've started a new "foreign-keys" branch of tengo so that we can collaboratively develop. Can you please make a new PR in that repo, with that branch as the merge target (rather than master being the target)? That way I can merge your PR into the foreign-keys branch, and then add some additional commits as needed, without affecting master yet (since we're still testing edge cases).
Unit tests: Godep doesn't copy *_test.go files into the vendor subdir, so these aren't visible in the skeema repo, but there are some tests in the tengo repo. They're fairly basic and messy, and they don't include any actual integration testing against a copy of mysql -- they're more to confirm that the structs behave correctly, and that changes to the structs generate the expected DDL statements.
For this PR, to maintain parity with the existing tests, table_test.go should probably get a new test method TestTableAlterAddOrDropConstraint. For ease of use, this might also involve adding some new helper methods to tengo_test.go, to easily create test tables with foreign keys. Given the messy state of tengo's tests, I'm happy to tackle adding the new tests on my end (by adding more commits once your code gets merged into tengo foreign-keys branch), but please let me know if you'd prefer to write them on your end instead :)
Possible edge case bugs: Just found a couple things so far:

It seems MySQL skips outputting RESTRICT clauses (e.g. "ON UPDATE RESTRICT"). So for example if you add a foreign key that has an ON DELETE clause but no ON UPDATE, MySQL's SHOW CREATE TABLE won't also include the default of ON UPDATE RESTRICT, but the code here will. That then causes Skeema to incorrectly flag the table as using unsupported features, since its opinion of what SHOW CREATE TABLE should output differs from MySQL's actual output (this is basically a safety check in Skeema). Should be fixable with some extra conditionals in Constraint.Definition.

I suspect the current code won't handle cross-database foreign keys. In other words, a foreign key on a table in schema A that references a table in schema B. Probably Constraint just needs a notion of a ReferencedTableSchema field, and then some corresponding changes to the information_schema query.

Let me know if you need help with these or if you'd prefer that I handle on my end, happy to go about this either way.
Cleanups: there are a few debug comments remaining, if you could please remove. Also a few style tweaks to be consistent with previous code, but those ones I'll do on my end in a separate commit since they're just my own personal style nitpicks.
Future considerations on my end:

As you noted, currently skeema uses foreign_key_checks=0. This is so that it doesn't have to figure out table creation ordering dependencies when operating in _skeema_tmp. This also allows mysql 5.6 online DDL to be used for adding foreign keys. I'll need to give more thought into which situations now shouldn't have this be set.
Similarly I'll also still need to think through whether adding a FK should be considered an "unsafe" operation. Probably can just stay a TODO comment for now :)
Interaction with pt-online-schema-change: This will just end up being noted as complex in the README for now, since it's a tricky issue. pt-osc's docs explain its handling of foreign keys if you're curious.
It might be better if Constraint stored pointers to referenced tables, cols, etc instead of using string names. But this can be a bit annoying since it requires more data structures to look up things efficiently, without any immediate functional benefit. Fine to stay strings for now, just a heads-up that I may change it in the future.

Think that's everything. Please let me know if any questions, I'm very happy to help. I'm also totally open to taking on all, most, or none of the above tweaks myself, depending on whatever you prefer. Thanks again!

Hi Evan.
Thanks for the feedback. I have altered the code to support cross database constraints and also made it work for the restrict clause as well. I forked tengo, and made PR to tengo/foreign-keys.
I haven't made the tests for foreign keys but I will take a look this afternoon and see how difficult it is. In which case I will probably update my PR. Just thought I'd give you a look in anyway if you're up.
Thanks for the opportunity to develop this tool!
Chris

Hi Evan
I started afresh, clone skeema again.
To get the dependencies all synced up I ran:
dep ensure
dep ensure -update github.com/skeema/tengo

A lot of changes occurred to the vendor/ directory. I hope this is right. I have never used dep before, just npm. In npm typically the node_modules/ directory is not commit to the git, but it appears that in dep the vendor/ directory is. Let me know if this is right!
Chris

Looks good, thanks Chris! I'll merge momentarily.
The only odd thing in this PR is that it seems to drop a few symlinks in the vendored docker dependency. But that might just be due to a difference in git version or dep version or filesystem/OS/whatever. It seems totally harmless in any case, so no worries!
With dep the common practice seems to differ between repos that have binaries vs repos that are just libraries. For binaries, it's good to commit everything (vendor/, Gopkg.lock, Gopkg.toml), so that builds are reproducible using the exact same versions of all dependencies. This also ensures anyone obtaining the project with just go get can build with the correct versions of dependencies without needing dep. (Even though dep is a semi-official Golang tool, it isn't built into the rest of the tooling like go get, and unfortunately it seems the state of dependency tracking in Golang is still completely in flux)
For libraries the practice I've seen is just to commit Gopkg.toml, so that the library can express any constraints on its own dependencies without causing conflicts on whatever binary is importing it. So that's what github.com/skeema/tengo does.

Hmm, it looks like Github isn't tying that commit to your GitHub account. I think it just bases it on committer email address, so if you add that acs.org.au address to your account and verify it, that might retroactively fix it. Not positive though.
I'll add you to the contributor list in skeema's readme in any case :)

Hi Evan
That's okay - this is due to my old work email still being on my computer. I don't work there any more so I probably shouldn't stick their email address on my github :(
But that's fine add me to the contributors group, no problem


No created_by data in the qna.yaml file

At present the outputted qna.yaml file from knowledge submission has a blank created_by field. This should be populated with the username of the person submitted or they could override and add another name for submission.
Acceptance Criteria:

The created_by section of the qna.yaml file should be populated.

I recently updated the qa.ui.instructlab.ai, and i don't see this issue in this version. It's been address with knowledge/skill edit refactoring patches. Here is the example from the latest version running in QA
https://github.com/instructlab-public/test-taxonomy/pull/78/files

@aevo98765 closing this issue, but if you see the issue again with the latest code, please feel free to re-open the issue.


KAFKA-14114: Adding Metadata Log Processing Error Related Metrics

This PR adds in 3 metrics as described in KIP-859

kafka.server:type=broker-metadata-metrics,name=metadata-load-error-count
kafka.server:type=broker-metadata-metrics,name=metadata-apply-error-count
kafka.controller:type=KafkaController,name=MetadataErrorCount

Thanks for the PR, @niket-goel. Looks like this needs to be rebased on trunk.
This needs to be integrated with the fault handler work. Specifically, the fault handlers need to increment the metrics, rather than doing it directly. This will allow juinit tests to monitor the fault handlers and make sure they are not triggering.
I was going to make a PR against your PR, but it ended up being too much work. The big hassle is the need to pull the metrics objects (BrokerMetrics, ControllerMetrics) up into KafkaRaftServer. Not conceptually difficult but a lot of churn.
This is the general idea, though:
   import BrokerMetadataPublisher._
@@ -259,9 +262,8 @@ class BrokerMetadataPublisher(conf: KafkaConfig,
       }
       publishedOffsetAtomic.set(newImage.highestOffsetAndEpoch().offset)
     } catch {
-      case t: Throwable => error(s"Error publishing broker metadata at $highestOffsetAndEpoch", t)
-        brokerMetrics.metadataApplyErrorCount.getAndIncrement()
-        throw t
+      case t: Throwable =>
+        metadataPublishingFaultHandler.handleFault("Error publishing broker metadata at $highestOffsetAndEpoch", t)
     } finally {
       _firstPublish = false
     }

diff --git a/metadata/src/main/java/org/apache/kafka/metadata/fault/MetadataFaultHandler.java b/metadata/src/main/java/org/apache/kafka/metadata/fault/MetadataFaultHandler.java
index e9f71b80e6..7ccade98f9 100644
--- a/metadata/src/main/java/org/apache/kafka/metadata/fault/MetadataFaultHandler.java
+++ b/metadata/src/main/java/org/apache/kafka/metadata/fault/MetadataFaultHandler.java
@@ -27,10 +27,26 @@ import org.slf4j.LoggerFactory;
  */
 public class MetadataFaultHandler implements FaultHandler {
     private static final Logger log = LoggerFactory.getLogger(MetadataFaultHandler.class);
+    private final String prefix;
+    private final Runnable action;
+
+    public MetadataFaultHandler(
+        String prefix,
+        Runnable action
+    ) {
+        this.prefix = prefix;
+        this.action = action;
+    }

     @Override
     public void handleFault(String failureMessage, Throwable cause) {
+        failureMessage = prefix + ": " + failureMessage;
         FaultHandler.logFailureMessage(log, failureMessage, cause);
+        try {
+            action.run();
+        } catch (Throwable e) {
+            log.error("Failed to run MetadataFaultHandler action.", e);
+        }
         throw new MetadataFaultException("Encountered metadata fault: " + failureMessage, cause);
     }
 }

diff --git a/core/src/main/scala/kafka/server/KafkaRaftServer.scala b/core/src/main/scala/kafka/server/KafkaRaftServer.scala
index e7cf8f8f1f..bf3f5e828e 100644
--- a/core/src/main/scala/kafka/server/KafkaRaftServer.scala
+++ b/core/src/main/scala/kafka/server/KafkaRaftServer.scala
@@ -83,6 +83,8 @@ class KafkaRaftServer(
   )

   private val broker: Option[BrokerServer] = if (config.processRoles.contains(BrokerRole)) {
+
+
     Some(new BrokerServer(
       config,
       metaProps,
@@ -91,7 +93,11 @@ class KafkaRaftServer(
       metrics,
       threadNamePrefix,
       offlineDirs,
-      controllerQuorumVotersFuture
+      controllerQuorumVotersFuture,
+      new MetadataFaultHandler("Unable to load metadata",
+        () => brokerMetrics.metadataLoadErrorCount.getAndIncrement()),
+    new MetadataFaultHandler("Unable to publish metadata",
+        () => brokerMetrics.metadataApplyErrorCount.getAndIncrement())
     ))
   } else {

We also need to integrate this into the test harness so that if the fault handler is called, the test is failed.
This is how we did it in QuorumTestHarness.scala:
  @AfterEach
  def tearDown(): Unit = {
    Exit.resetExitProcedure()
    Exit.resetHaltProcedure()
    if (implementation != null) {
      implementation.shutdown()
    }
    System.clearProperty(JaasUtils.JAVA_LOGIN_CONFIG_PARAM)
    Configuration.setConfiguration(null)
    faultHandler.maybeRethrowFirstException() <==== HERE
  }
``

and in `KafkaClusterTestKit.java`:

@Override
public void close() throws Exception {

...
metadataFaultHandler.maybeRethrowFirstException();
fatalFaultHandler.maybeRethrowFirstException();
}

In these junit tests, it's probably OK to use the same `metadataFaultHandler` object for all the brokers, by default. We don't expect ANY broker to have a metadata fault, after all. If a test wants to do something exotic we can accommodate that in a custom way.

Thanks for the comments @cmccabe. I have rebased and integrated the metrics with the new Fault handling framework you added. Could you please take another look.

https://github.com/apache/kafka/pull/12496 Contains all changes in this PR and some more additional improvements. We will continue with that PR for this Improvement.

Merged from command line with the changes we discussed. Thanks, @niket-goel


fix: TypeError: can't convert BigInt to number

After the following
$ npx create-react-app my-safe-app --template @safe-global/cra-template-safe-app
$ cd my-safe-app
$ npm i
$ npm run build

The build would display a blank page with TypeError: can't convert BigInt to number error in console.
I applied this solution and it fixed the issue.

the CLA link is broken 😅

I have read the CLA Document and I hereby sign the CLA

recheck

Added the changeset 👍 , anything else?


nit: fix grammar in generator deprecation warning

This fixes the tense of will been to will be

@juliangruber : sorry about that, I missed the test case somehow. Just updated the PR

Current coverage is 58.85% (diff: 100%)

Merging #834 into v2.x will decrease coverage by 41.14%

@@           v2.x       #834   diff @@
======================================
  Files         4          4
  Lines       418        418
  Methods      80         80
  Messages      0          0
  Branches    100        100
======================================
- Hits        418        246   -172
- Misses        0        172   +172
  Partials      0          0

Powered by Codecov. Last update ce78786...a02db35

cool, thank you!


💣 dont lint and test on readme change 💣

What is this PR About?
dont lint and tests chart when only the README.md has changed
we dont do markdown llinting (yet) !
cc: @redhat-cop/day-in-the-life

/lgtm
/approve


Fix import in translation status script

What kind of changes does this PR include?

Something else!

Description
In #954 I extracted a function from the translation status checker to a separate module so I could re-use it in the page generator. I messed up the updated import inside the translation status script, leaving out the .mjs extension which is required.
This PR fixes that.

LGTM!


Add --exact flag

Description of changes:
When the user calls kani with a command cargo kani --harness module::check_blah --exact and they want to specify which harness to match, using --exact, they can prevent multiple harnesses from being run. The user will need to specify the harness name by using the fully qualified name as well.
Resolved issues:
Resolves #2285
Call-outs:
If the user adds --exact and they want to run kani on multiple harnesses, they will need to provide the specific name for every harness.
Testing:

How is this change tested? Unit and regression change

Is this a refactor change? no

Checklist

[x] Each commit message has a non-empty body, explaining why the change was made
[x] Methods or procedures are documented
[x] Regression or unit tests are included, or existing tests cover the modified code
[x] My PR is restricted to a single feature or bugfix

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 and MIT licenses.

Thanks @jaisnan! I was expecting to see at least one test with two or more harnesses where the name of one harness is a substring of the other, and --exact --harness is used for selecting the one that is a substring while making sure the other one is not selected.

I think the test tests/ui/exact-harness/some_matching_harnesses/subset.rs was design to test that, but I don't think it is quite testing it yet.

I do wonder if we should fail in the case where an exact match was not found. I don't think rust tests fail, but they also don't fail when a filter has no match.
@adpaco-aws might have an opinion about this one.

Yes, I do. We should fail in that case. In addition to "catch" mistakes from users, it'll also be helpful to detect issues with target selection on the VSCode extension.

@celinval I've added the error when an exact match is not found.


Add xbox button trigger (use undocumented code in winapi) to enable camera w/ controller

See: https://forums.tigsource.com/index.php?topic=26792.0
The way it should work: when pressing the xbox button on the controller, the camera should enable, the game should pause and input from the controller should block. This is close to how an in-game photomode would work. You can then go move the camera around right away by a single button press.
Might be best to simply use the WM_INPUT route, as we already hook raw input. Not sure what key is detected when pressing that button, but it's easy to find out.

Really doesn't work. Can't add this, as the undocumented hack is for xinput 1_3 and most games use 9.1.


Add initial version of cyw43xxx wifi driver

Added initial version of Infineon cyw43xxx Wifi driver
Enable WIFI feature on cy8cproto-062-4343w
Added wifi station sample

@npal-cy please fix the revision field in west.yml

@npal-cy please update to SHA

Not sure why BabbleSim Tests / bsim-test failed..

Not sure why BabbleSim Tests / bsim-test failed..

after rebase it passed.

@tbursztyka can we merge this PR?

@tbursztyka, we are ready for merge. Please review and let us know if you have any remaining feedback.
Thanks in advance,
Ian

After moving driver to hal_infineon i get error in CI:
modules/hal_infineon/Kconfig:79: '/whd-zephyr-integration/Kconfig.cyw43xxx' not found (in 'source "$(ZEPHYR_HAL_INFINEON_MODULE_DIR)/whd-zephyr-integration/Kconfig.cyw43xxx"'). Check that environment variables are set correctly (e.g. $srctree, which is set to '/home/runner/work/zephyr/zephyr'). Also note that unset environment variables expand to the empty string

I do not see such behavior locally.  Maybe somebody has any ideas...why $(ZEPHYR_HAL_INFINEON_MODULE_DIR) is not available from Kconfig?

After moving driver to hal_infineon i get error in CI:
modules/hal_infineon/Kconfig:79: '/whd-zephyr-integration/Kconfig.cyw43xxx' not found (in 'source "$(ZEPHYR_HAL_INFINEON_MODULE_DIR)/whd-zephyr-integration/Kconfig.cyw43xxx"'). Check that environment variables are set correctly (e.g. $srctree, which is set to '/home/runner/work/zephyr/zephyr'). Also note that unset environment variables expand to the empty string

I do not see such behavior locally. Maybe somebody has any ideas...why $(ZEPHYR_HAL_INFINEON_MODULE_DIR) is not available from Kconfig?

Yeah you cannot directly include module Kconfig fragments from the upstream repo, that is because all includes are always parsed regardless of what modules are available, and
source "$(ZEPHYR_HAL_INFINEON_MODULE_DIR)/whd-zephyr-integration/Kconfig.cyw43xxx"
cannot work if the hal is not there.
The fix is to provide a Kconfig file in the module instead of using kconfig-ext: True, see https://github.com/zephyrproject-rtos/lvgl/blob/zephyr/zephyr/module.yml#L5 for example.
You can test locally by running the check compliance on your machine, something like ./scripts/ci/check_compliance.py -c HEAD^^..HEAD.

After moving driver to hal_infineon i get error in CI:
modules/hal_infineon/Kconfig:79: '/whd-zephyr-integration/Kconfig.cyw43xxx' not found (in 'source "$(ZEPHYR_HAL_INFINEON_MODULE_DIR)/whd-zephyr-integration/Kconfig.cyw43xxx"'). Check that environment variables are set correctly (e.g. $srctree, which is set to '/home/runner/work/zephyr/zephyr'). Also note that unset environment variables expand to the empty string

I do not see such behavior locally. Maybe somebody has any ideas...why $(ZEPHYR_HAL_INFINEON_MODULE_DIR) is not available from Kconfig?

Yeah you cannot directly include module Kconfig fragments from the upstream repo, that is because all includes are always processed regardless of what modules are available, and
source "$(ZEPHYR_HAL_INFINEON_MODULE_DIR)/whd-zephyr-integration/Kconfig.cyw43xxx"
cannot work if the hal is not there.
The fix is to provide a Kconfig file in the module instead of using kconfig-ext: True, see https://github.com/zephyrproject-rtos/lvgl/blob/zephyr/zephyr/module.yml#L5 for example.
You can test locally by running the check compliance on your machine, something like ./scripts/ci/check_compliance.py -c HEAD^^..HEAD.

Thanks @fabiobaltieri for suggestion. I got more problems after remove  kconfig-ext: True. Looks like /modules/hal_infineon/Kconfig become unavailable. For now added modules/hal_infineon/whd-zephyr-integration/Kconfig.cyw43xxx in Zephyr repo.

Thanks @fabiobaltieri for suggestion. I got more problems after remove kconfig-ext: True. Looks like /modules/hal_infineon/Kconfig become unavailable. For now added modules/hal_infineon/whd-zephyr-integration/Kconfig.cyw43xxx in Zephyr repo.

Yeah either do that or include modules/hal_infineon/Kconfig from https://github.com/zephyrproject-rtos/zephyr/blob/main/modules/Kconfig.infineon

@jukkar please take a look

Close this PR. Host agnostic version of driver under review: https://github.com/zephyrproject-rtos/zephyr/pull/63721


adds query rescoring

https://www.elastic.co/guide/en/elasticsearch/reference/8.13/filter-search-results.html#query-rescorer

https://github.com/jillesvangurp/kt-search/pull/134/files#diff-a4db7e6283ba73279198bd117cb095caed7d9d83aedd3935fe7046cc88801d30 has to be applied first to succeed the tests

Hey, thanks for your pullrequests, I merged your other changes. So, you should be able to update this branch and get this one fixed.

Hey, just updated from the master branch and all tests work fine.
Thank you!

Great, thanks for all the work you are putting in. I'll cut a release soon.


minor: Abstract Getting Started intro partials

Removing the partials for Introduction in this PR since they are no longer needed. I have created an issue #2322 to migrate the Intel-nuc note to a new home now :tada:
Signed-off-by: Vipul Gupta (@vipulgupta2048) <EMAIL_ADDRESS>

Please make sure to read the CONTRIBUTING document before opening the PR for relevant information on contributing to the documentation. Thanks!

@balena-ci retest

@balena-ci rebase

@balena-ci I self-certify!


Added missing drawables for weather forecast

Description
App crashing on opening weather forecast
Fixes #(issue)
Fixed crash happening on opening weather forecast by adding drawables
Type of change
Just put an x in the [] which are valid.

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

How Has This Been Tested?
Please describe the tests that you ran to verify your changes.

[ ] ./gradlew assembleDebug assembleRelease
[ ] ./gradlew checkstyle

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings

@aman0408 Great!

Can you also add the  check (if drawable exists) where the app was crashing
Also, these images (which you just added) show up as unused drawables in Android studio. Is there a way to disable the warning for these in future so that these doesnt get removes again? (This particular issue can be solved in another PR/issue also)

@aman0408 Great!

Can you also add the  check (if drawable exists) where the app was crashing
Also, these images (which you just added) show up as unused drawables in Android studio. Is there a way to disable the warning for these in future so that these doesnt get removes again? (This particular issue can be solved in another PR/issue also)

Okay i will add the if condition and create another request. About the second part I'll figure something out.

You dont have to create another request. You can add the changes in this PR only.

I didn't know I could update files in the same request. Anyway, I have added the if condition and pushed again on my branch. Is there anything else I need to do to update the PR?

About the unused resources part, we could create a dummy file which references these resources so they're not marked as unused


Docker example

I, and probably others as well, would greatly appreciate having some documentation and an example Dockerfile for running sqlmesh on container-based cloud hosts, such as Google Cloud Run. To make it even more convenient, it would be wonderful if you could maintain a public DockerHub image that we can easily make reference to.

Yeah I agree there is value in offering this. It will be something we look into in the upcoming weeks.

@eakmanrq / @gunar, I created a branch that provides a Dockerfile, plus basic usage documentation in the README.
See here:  https://github.com/jmarch/sqlmesh/tree/add-docker-support
My branch also has a base docker-compose.yml file that can be customized by the end user (or enhanced in the branch) to include additional databases for testing. I only put postgres and mssql example databases in the config to show how this is done.
The additional database service containers are listed with profiles, such that they are optional when running docker-compose up. This means by default compose creates a local sqlmesh docker network with the base sqlmesh container, then allows for starting up particular databases that one wishes to test.
After I created these assets, I found the existing Dockerfile and docker-compose.yml files within the web directory of the main TobikoData/sqlmesh project. Note: there's no documentation around these at all. I found them after combing through the Makefile.
These existing web docker files seem primarily for testing the web UI. They leave a lot to be desired.
For example, the base image has no editor installed (no vim or nano, at least), so testing out the sqlmesh CLI is just about impossible (unless you want to like 'echo' text and redirect to your config file). Furthermore, it doesn't have sudo or sudo access for the end user to install missing packages. And, it's missing some of the sub-modules of sqlmesh (like sqlmesh[mssql]). So even if the end user chooses to use the web UI for all their editing, they are out of luck if they stray from what the base includes. Without sudo, they just can't resolve any issues by attaching to their container.
My base image solves these problems, but may not be as secure with the sudo access by default. That's something to consider if this were adopted.
Since I install all the dialects, the image balloons to I believe roughly 1.74GB. I tested all this last week, but moved on to other things, so forgive me if that's off a bit.
Anyway... no pull request, yet, as I was told none of this work was needed.

@tobymao, can you please clarify was closing this issue means? Does it mean that you won't be providing a Docker example, or that Docker won't be a supported deployment option at all in the foreseeable future?

it’s not a priority for us to add to add a docker example. we’d suggest tobiko cloud as the recommended way to do a deployment but you’re free of course to use sqlmesh in docker as you see fit
https://sqlmesh.readthedocs.io/en/stable/cloud/cloud_index/


Replace 'libpsrpclient' with 'psrp' package

The package libpsrpclient was replaced by psrp in #3271. However this was missed when moving from project.json to MSBuild. This PR makes this update.

:shipit:


Add session key obtaining guide to README

This commit adds a detailed instruction set to the README file, guiding users on how to obtain their session key from Claude.ai. This key is crucial for synchronization between local files and Claude.ai projects.

I've reviewed the changes in PR #14 and noticed a couple of issues:

Duplication of Documentation: The documentation added duplicates what is already present when executing claudesync api login claude.ai. This redundancy should be avoided.

Provider-Specific Information: The login procedure described is specific to the claude.ai provider. Since this tool is envisioned as multi-provider, we should avoid provider-specific details in the main documentation.

valid points closing :)


Release as single YAML

Which Issue(s) this Pull Request Fixes

Resolves #277, resolves #278, may be related to #271
Release Note

NONE

Notes for Reviewers

Enabled the database deployment by default for development, as the Helm chart is now development oriented I think it makes sense.
Moved POSTGRES_URL to the secret, should I move the port too?
Moved the testSA to its own file on the test folder so its easier to toggle.
Removed braces rules: they are not needed anymore and they introduce artifacts in the resulting release YAML.
Introduced the creation of a namespace for production, so we don't need to ask users to kubectl create ns kubearchive

@rh-hemartin
About

Moved POSTGRES_URL to the secret, should I move the port too?
I searching for references about this, and I've found that it's recommended to have in the secret the DB url, DB user, DB pass and the DB name, but nothing about the port.


No instructions for Linux systems for updating bootloader (release 1.7.1)

The readme says that the bootloader must be updated first before updating the firmware, but it only gives directions for EvalTool which requires Windows. How do I update the bootloader/firmware using Linux (CLTool?)?

I've found a couple ways to do this in Linux.
One is running EvalTool.exe through wine.  It seems to run fine, but I can't say with complete confidence that it will work as intended (running into some issues that may or may not be related).
The other is implementing your own code to call bootloadUpdateBootloader() from the SDK (found in inertiaSenseBootloader.c/h).
It currently isn't implemented in CLTool, although it could be in the future.

Bump

In the latest version of the SDK there is a new function bootloadUpdateBootloader() that can be used to update the bootloader as mentioned by @orrinjelo. There is a known issue with this function when used with Linux embedded systems which can prevent the bootloader update to complete properly. We are currently working on a fix as well as implementing a way to update the bootloader within the CLTool.

Latest cltool allows users to update bootloader using -ub flag.


Update CMakeLists.txt

Novice to C totally, embarrassed to admit that not switching to the build directory tripped me up. Compile seems to run, but no executable gets produced if still in examples directory.

Oops, yeah, that's confusing.  Thanks for the PR :)


Ship identification example

Simple workflow for ship identification. Also want to show catalog and parallelization features. For the webinar will need to skip over some parts, 10 minutes are pretty short.
If available in time would also include price estimation.

LGTM! Nice and clean. Love the plots next to one another.
A minor cosmetic suggestion: The identified ships can be in green so that they be easier for visual message conveying!

Thanks for the suggestion. Had it green earlier, but it does not stand out on the turquoise waters. Tried a number of colors, orange worked best in the end.

LGTM! Nice and clean. Love the plots next to one another.
A minor cosmetic suggestion: The identified ships can be in green so that they be easier for visual message conveying!

Thanks for the suggestion. Had it green earlier, but it does not stand out on the turquoise waters. Tried a number of colors, orange worked best in the end.

ahh okay!

LGTM.

Heading "Download selected SPOT images for aois" seems outdated since this section runs the whole pipeline.
Need to add the example as a chapter entry in mkdcos.yml, analouge to the other examples.

Thanks. Did both.


[PE-D][Tester C] Integration between stalls and reviews is not the most intuitive

As a user wanting to make use of FoodWhere (FW) to manage food reviews, the GUI can get really cluttered really quickly, especially when there is multiple reviews for a single stall. Furthermore, with almost 2 of each typology of commands, 1 for each list, it can get quite confusing for the user.
Perhaps you could consider just having the stalls as the main display, then for each review added to the stall, it can be hidden until the user decides to (maybe) click into the specific stall, then the reviews for that stall only will pop up.
Just a suggestion after playing around the app.

Labels: type.FeatureFlaw severity.Medium
original: Samsation/ped#6

Thank you for the suggestion. We will take into consideration after v1.4, however for now, this will be the intended functionality for FoodWhere.


beautify progress bar in tqdm

replace tqdm(enumerate) by enumerate(tqdm)

Very good PR, this will make tqdm more elegant. The Easyedit team appreciates your willingness to invest your time and energy in helping us advance this project together. Your contributions are a valuable asset to our toolkit and a perfect example of the open-source spirit.
Thanks again. 😊


ActionView::Template::Error: Invalid CSS after

thanks for your work! I got this error when i sending an email, could you give me some help?
ActionView::Template::Error: Invalid CSS after "...ings-ratios: (1": expected ")", was ": 2.25, 2: 2, 3..."

thanks again

@genkin-he I know this was a long time ago, but I'm getting the same error and I was hoping you had a solution.

Nevermind, I figured it out. My version of sass-rails was too old.


Discussions tab does not exist

Describe the bug
The referenced discussions tab goes to a github 404
Your Example Website or App
Not needed
Steps to Reproduce the Bug or Issue
Just click on the links in the issue template
Expected behavior
To have some discussion 😁
Screenshots or Videos
No response
Platform

OS: [e.g. macOS, Windows, Linux]
Browser: [e.g. Chrome, Safari, Firefox]
Version: [e.g. 91.1]

Additional context
No response

Oh I see we copied the Solid issue template. Hmm.. need to decide whether this repo should have its own discussions forum. It might be easier to keep things centralized. Not sure yet.

I see you already enabled them. Thank you!


feedback

Related Issue
#1033
Description
double clicking the icon feedback page will pop out, I have added a modal feedback form which will take star rating as well and save the feedback form and submit it successfully.
Type of PR

[x] Feature enhancement

Screenshots / videos (if applicable)
https://github.com/arghadipmanna101/Flipkart_Clone/assets/158484284/a245f862-e2d6-41ac-a082-12c607245040
Checklist:

[x] I have performed a self-review of my code
[x] I have read and followed the Contribution Guidelines.
[x] I have tested the changes thoroughly before submitting this pull request.
[x] I have provided relevant issue numbers, screenshots, and videos after making the changes.
[x] I have commented my code, particularly in hard-to-understand areas.

Additional context:
[Include any additional information or context that might be helpful for reviewers.]

@arghadipmanna101 sir please preview this pull request

@arghadipmanna101  Please review the PR and merge this pull request.

There have some conflict issue check it once.
This is not perfect place to add this , plz replace it :

okay I am making the changes


Summit page and it's components

Essence of this PR
This PR fulfills the project requirement which is to build a responsive and interactive conference page that captures a similar look from the original work yet embraces some finessing and uniqueness like image and text differences to my work .
What does this PR feature?

It satisfies the project requirement to make a responsive web page
It also accomplishes the task to render the speakers dynamically in the event of unavoidable changes prior to the summit date.
It also renders a dual page feature i.e the home page and the about page
It helps users have a more friendly experience via a slide menu functionality for easy page navigation
it also satisfies 100% user accessibility for users with disabilities by making the images displayed have a well-descriptive alternative text so they can flow with what is documented on this page.

Technologies this project features

Html5
CSS3
javaScript

How to use this project

Clone this repository via this link Web_Summit
Paste this in your GitBash
Open with your favorite text editor
Go live with your desired browser.

Project documentation
I added to this project all the features that this project contains and how to use them best. If you want to see the video here is the link https://www.loom.com/share/9d435bf40d9341b4983d790bb545fecc

Thank you @KayLemba  for your professional approach toward my studies here in this program. It has spurred my quest for further learning. I will graciously put all you've said to practice and hope for a brighter future in my endeavor. Thanks


Add an About/citation section and fix navbar weirdness

This will address #4

what do you think of this now @cansavvy ?

This is good enough for now.


⬆🔒 upgrade remaining dependencies

Attempt to upgrade dependencies listed in https://github.com/NUMde/compass-numapp-web/issues/15
❗ untested, no local backend set up to test anymore. Build works and UI seems to work as expected. cc @mbastian93
Note: To upgrade @d4l/web-components-library to v3, package @stencil/router would have to be replaced with https://github.com/ionic-team/stencil-router-v2. However, development on the v2 router was recently dropped for lack of maintainers. The v2 router itself works just fine, has a very small and clean code base and provides plenty of improvements over the v1 router (it will also remove a lot of warnings in the DevTools console currently coming from the v1 version), that's why D4L's web components library decided to upgrade to it despite it never having officially reached a final version.

It seems like the used version stencil is not compatible with the used version of jest; running npm test results in
[56:58.4]  @stencil/core
[56:58.7]  v2.17.0 🚂

[ ERROR ]  Please install supported versions of dev dependencies with either npm or yarn.
           npm install --save-dev<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>

Thank you, I have now downgraded the jest dependencies accordingly


Add combinators on Semigroup, Monoid, Aggregator to reverse order.

see comment here:
https://github.com/twitter/algebird/pull/739#discussion_r343814345
reversing the order of semigroup preserves it being a semigroup, as does putting a constant element between every pair. This could be a combinator on semigroup that we could propagate up to other Semigroup related abstractions, e.g. Aggregator, Fold, Scan.

Hi @johnynek just saw your change @ https://github.com/typelevel/cats/pull/3279 ... I think it will be handy for this one.