Is it safe to have Wqkv be float16?

I notice that the QKV matrix is in fp16. Normally, I just use torch.cuda.amp.autocast to handle converting fp32 operations into fp16, so I'm not actually sure if:

autocast converts the QKV weight matrix into fp16
If the above is false, will the results be very different for fp16 attention vs fp32 attention / is something I need to take into consideration when training.

I'm asking, because I implemented fp16 attention to make a comparison against flash attention (to determine if the results are similar), and I got NaNs, which makes me wonder if fp16 attention is safe:
class Attention(nn.Module):
    def __init__(self, dim: int, heads: int):
        super().__init__()
        self.heads = heads
        head_dim = dim // heads
        self.scale = head_dim**-0.5

        self.softmax= nn.Softmax(dim=-1)
        self.to_qkv = nn.Linear(dim, dim * 3, bias=False, dtype=torch.float16)

    def forward(self, x):
        b, n_patches, dim = x.shape
        qkv = self.to_qkv(x).chunk(3, dim=-1)

        assert qkv[0].shape == x.shape

        q, k, v = map(lambda t: rearrange(t, "b n (h d) -> b h n d", h=self.heads), qkv)

        assert q.shape == (b, self.heads, n_patches, dim // self.heads)

        attn_matrix = (q @ k.transpose(-1, -2)) * self.scale

        assert attn_matrix.shape[-1] == attn_matrix.shape[-2]
        assert attn_matrix.shape[-1] == n_patches

        normalized_scores = self.softmax(attn_matrix)

        reaveraged_values = normalized_scores @ v
        reaveraged_values = rearrange(reaveraged_values, "b h n d -> b n (h d)")

        return reaveraged_values

Edit: Up-casting before applying attention does help, but I still have:

max diff = 0.03125
mean diff = 0.0001729

With FlashAttention we've trained BERT & GPT2 in fp16/bf16 just fine.
We have tests the check the numerical error of FlashAttention and a reference implementation, and the numerical error is similar to the Pytorch implementation.
Standard attention fp16 may or may not be safe, depends on how large the inputs (q, k, v) are and how you're implementing it. One place where it might NaN out is (Q @ K.T), which might produce entries larger than 64k (limit of fp16), before multiplying by self.scale. You can multiply Q by self.scale first before multiply by K.T.
There are several places you can read about numerical stability of attention.

With FlashAttention we've trained BERT & GPT2 in fp16/bf16 just fine. We have tests the check the numerical error of FlashAttention and a reference implementation, and the numerical error is similar to the Pytorch implementation.
Standard attention fp16 may or may not be safe, depends on how large the inputs (q, k, v) are and how you're implementing it. One place where it might NaN out is (Q @ K.T), which might produce entries larger than 64k (limit of fp16), before multiplying by self.scale. You can multiply Q by self.scale first before multiply by K.T. There are several places you can read about numerical stability of attention.

Thanks for the reference. Do you have any tips on how to integrate this with Pytorch's amp? It seems like flash attention wants the QKV matrix to be in fp16. However, when training with Pytorch this causes the exception: "Attempting to unscale FP16 gradients.", which makes sense because I guess Pytorch expects the model's weights to be in FP32 (or something like that?)

You don't have to do any manual casting if you're running with Pytorch AMP.
The matrix W_qkv is in fp32 but AMP will cast the output of the matmul (qkv) to fp16/bf16, which then get passed to FlashAttention.

Amazing! got it working and it managed to 10x my batch size, which is super useful since I'm doing contrastive loss. Thanks!


Allow customizing time/date picker entry modes via QuestionnaireTheme

Allows setting showDatePicker and showTimePicker's initialEntryMode property from QuestionnaireTheme.

@hanny-ph Something I noticed just now... apparently there's a bug with Flutter's time picker in input mode:
https://github.com/flutter/flutter/issues/85527
Apparently if the hour format is set to 24h according to the locale, input validation fails for hours over 12. In dial mode it seems to work properly though.
In that regard, these changes are not so useful, since the timepicker itself may not be usable at all in input mode :S.
Not sure what would be the best course of action here to be honest. The Flutter issue has been open since 2021, and there's doesn't seem to be any fix in the short term. Maybe we can try forcing the time to 12h am/pm format when on input mode, but that sounds a bit odd as well.

Apparently if the hour format is set to 24h according to the locale, input validation fails for hours over 12. Moreover, if I set an hour less than 12 in input mode, it sets the time as if it was pm (e.g. 11:00 becomes 23:00). In dial mode it seems to work properly though.

@hanny-ph I've implemented a workaround for the above issue in the meantime. I'm not very proud of it, but it appears to work both when the time format of the locale is set to either 12h or 24h on my side at least.
Please let me know if this works on your side or if you can think of a better approach for it.


MAP function  needs improvement

Open Event Android app's MAP function
In this function, zoom in and out should be improved. Zoom in should be increased

But how do you propose that? OSM doesn't support this.

Zoom was implement in app in meantime.


Minor helm, microk8s improvements

Improve helm integration:

Split out debug into debug.enabled and debug.verbose
Split out containerRuntime into containerRuntime.integration and containerRuntime.socketPath

Fix microk8s guide with helm.

This change is

test-me-please

Coverage increased (+0.04%) to 44.162% when pulling<PHONE_NUMBER>30c37100623246ff4946f64c05bafa on joestringer:submit/helm-fixes into ce845e71f78110174b72dbb15d6426a7acc29050 on cilium:master.

test-me-please


Align problem-report schema with current RFC

Codecov Report

Merging #285 into master will decrease coverage by 0.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #285      +/-   ##
==========================================
- Coverage   81.61%   81.59%   -0.02%
==========================================
  Files         232      232
  Lines       10769    10759      -10
==========================================
- Hits         8789     8779      -10
  Misses       1980     1980

Per Andrew, this is likely a breaking change that needs to be managed appropriately.

Out of date

Out of date


gcsimui: Add version 0.3.3

Closes #XXXX

Relates to #XXXX

[x] I have read the Contributing Guide.

/verify

Upstream issue, new version Windows build failed


rui-button doesn't handle long names

Currently rui-button has a width of 120px coming from .inline-group button. This causes larger names like "Redox Patient Search" to overflow.
Changing width to min-width should solve the issue

RWARRR! Thanks for posting, @ajkitson. Ich Bin RedoxBot-- nice to meet you!
It has been 60 days since there have been any updates or new comments on this page.       If this issue has not been resolved and needs to be re-opened, please take a minute to re-open it. Report problems with the bot to @nwhatt


[no-UKI] Make ISO images hybrid and with GPT to improve compatability

This allows to boot the ISO with different combination of HW for UKI: https://github.com/kairos-io/enki/pull/107 - we should have the same for non-UKI mode, for instance when booting with UEFI but with grub.
See also: https://github.com/kairos-io/kairos/issues/2547

Seems that the actual issue was not that the USB is not recognized, but the usb boots and it gets a grub shell because it cannot load the default grub.cfg for efi.
This seems to be a side issue of how efi booting works differently from livecd or usb in grub.
Looks like when booting from livecd, the efi artifacts are loaded from the efi image file BUT the livecd is mounted as root for grub.
When booting from usb it seems like the efi artifacts are loaded from the efi image AND the efi image is mounted as root for grub.
In both cases, grub is hardcoded to search for the grub config in /EFI/boot/grub.cfg (so alongside the grubx64.efi file)
When we build the ISO we only copy those artifacts on the ISO root, so you got the /EFI/boot/grub.cfg directly accessible when booting from livecd.
But when booting from usb the livecd is ignored and only the contents of the uefi.img are in the root, so /EFI/boot/grub.cfg does not exists.
This is fixed by copying it to both places so it works on both boot methods.


spring_stream::handler::Handler<_> not satisfied for stream listener fn

I am trying to test out the spring_stream feature but am running into an issue where the Handler trait is not implemented:
rustc: 1.83.0
OS: Fedora Linux 41
rustc: the trait bound `fn(spring_web::extractor::Json<Payload>) -> impl std::future::Future<Output = ()> {listen_topic_do_something}: spring_stream::handler::Handler<_>` is not satisfied
the trait `spring_stream::handler::Handler<_>` is not implemented for fn item `fn(spring_web::extractor::Json<Payload>) -> impl std::future::Future<Output = ()> {listen_topic_do_something}`

# app.toml
[web]
port = 8081              # Port number to bind
connect_info = false     # Whether to use client connection information, default false
graceful = true          # Whether to enable graceful shutdown, default false

[web.middlewares]
logger = { enable = true, level = "info" }

[logger]
enable = true            # Whether to enable the log function, which is enabled by default
pretty_backtrace = false # Whether to print stack information, which is disabled by default.
level = "info"           # The default log level is info
format = "pretty"       # The log format supports compact, pretty, and json.

[sqlx]
uri = "sqlite://db/sqlite.db?mode=rwc"

[stream]
uri = "file://./stream"

[stream.file]
connect = { create_file = "CreateIfNotExists" }

# Cargo.toml
[package]
name = "spring_rs_api"
version = "0.1.0"
edition = "2021"

[dependencies]
serde = { version = "1.0.216", features = ["derive"] }
serde_json = "1.0.134"
spring = "0.3.1"
spring-sqlx = { version = "0.3.0", features = ["sqlite"] }
spring-stream = { version = "0.3.0", features = ["json", "file"] }
spring-web = "0.3.0"
tokio = { version = "1.42.0", features = ["rt-multi-thread", "macros"] }

[profile.release]
lto = "fat"
opt-level = "z"
debug = 0
overflow-checks = false
panic = "abort"
codegen-units = 1

// My Code example
use serde::{Deserialize, Serialize};
use serde_json::json;
use spring::{auto_config, tracing, App};
use spring_sqlx::SqlxPlugin;
use spring_stream::file::{AutoStreamReset, FileConsumerOptions};
use spring_stream::handler::TypedConsumer;
use spring_stream::{stream_listener, Consumers, Producer, StreamPlugin};
use spring_web::axum::Json;
use spring_web::error::Result;
use spring_web::extractor::Component;
use spring_web::get;
use spring_web::{axum::response::IntoResponse, WebConfigurator, WebPlugin};

#[derive(Debug, Serialize, Deserialize)]
pub struct Payload {
    success: bool,
    msg: String,
}

// Main function entry
#[auto_config(WebConfigurator)]
#[tokio::main]
async fn main() {
    App::new()
        .add_plugin(StreamPlugin)
        .add_plugin(SqlxPlugin)
        .add_plugin(WebPlugin)
        .add_consumer(consumers())
        .run()
        .await
}

#[get("/api/streamer")]
async fn hello_world(Component(producer): Component<Producer>) -> Result<impl IntoResponse> {
    let json = json!({
        "success": true,
        "msg": "This is a test"
    });

    let resp = producer
        .send_json("the_topic", json)
        .await
        .expect("Failed to send stream");

    let seq = resp.sequence();

    Ok(Json(json!({ "seq": seq })))
}

fn consumers() -> Consumers {
    Consumers::new().typed_consumer(listen_topic_do_something)
}

// Error Occurs Here v
#[stream_listener(
    "the_topic",
    file_consumer_options = fill_file_consumer_options
)]
async fn listen_topic_do_something(Json(payload): Json<Payload>) {
    tracing::info!("{:#?}", payload);
}
// Error Occurs Here ^

fn fill_file_consumer_options(opts: &mut FileConsumerOptions) {
    opts.set_auto_stream_reset(AutoStreamReset::Earliest);
}

There are two easily confused extrators: spring_stream::extractor::Json & spring_web::extractor::Json. The former should be used in spring_stream.

I see, thanks for the clarification.


Why model is wrapped to be tuple in nyuv2/trainer.py line 37

The code is resulting error on line 38 (model = model.to(device)), with, AttributeError: 'tuple' object has no attribute 'to'. This is a valid error. Just unwrapping, and making model = SegNet() enables the code to run smoothly.
My question is, why model is wrapped to be a tuple. Any requirements down the line?
https://github.com/AvivSham/auxinash/blob/f97bbd1817659ff58cf85c9d79ed3f60b3a460ce/experiments/nyuv2/trainer.py#L37

Thank you for spotting this issue.
This is a typo mistake, I fixed it.


adds metadata to thrift_hsha_test tests

Quick little improvement -- adds some Doxygen metadata to the thrift HSHA tests.

You can feel 100% free to just push these kind of commits. +1 though.

You can feel 100% free to just push these kind of commits.

Personally, I like having reviews even for the smallest commits -- by definition, I don't know when I'm going to screw up. That said, if it's just noise for you, I can stop; just let me know
I can push this up if you'd prefer I not generate merge commit noise. Do you care about that?

But since you're explicitly asking for review on this, could we standardize the value for @jira_ticket? In some places we use CASSANDRA-XYZ and in others, we use XYZ. I'd prefer to use the former, since the html docs we generate with Doxygen will be colocated with docs from other softwares who use other JIRA's. Additionally, we might want to link python driver tickets (unlikely, but possible).

Awesome, thanks. Force pushed with @jira_ticket CASSANDRA-XYZ, merging.


PostgreSQL/CockroachDB connection string

https://github.com/prisma/prisma/issues/10805 added another field options that can be set and should be documented. Details about how this works in https://github.com/prisma/quaint/issues/338 and via the links in there.

👋 we're cleaning up our backlog of issues and are closing everything older than two years. If you still feel that this issue is relevant, please feel free to re-open and our team will take a look. Thank you!


refactor(multiple): deprecate HighContrastModeDetector

Completely deprecate HighContrastModeDetector and use forced-colors
media query instead.
This is still a draft because every component needs to be smoke tested. that's because switching to forced-colors changes the specificity and could cause issues.
High contrast mode is a Windows feature but testing can be done on other platforms on Firefox and Chrome.
https://developer.chrome.com/blog/new-in-devtools-98/#forced-colors

This is working now, and it's ready for review.
I tested this on native Window HCM with chrome. I tested every component's dev-app overview page, including legacy components with high contrast dark. For components that use SVGs, I also tested in high contrast light.

Chrome Version 108.0.5359.72 (Official Build) (64-bit)
Windows 10 Enterprise Version 21H2

cc @crisbeto

Converting back to a draft while we get consensus on what the API should be.


How to do something like "urxvt.letterSpace: -1"?

You patch it in if you want it.

Actually, you know what, adjusting cwscale in the config.h probably would do what you want.


bibfish: Citekey 'le2024wellposedness' was not found in ['bib.bib']

I have main.tex and bib.bib. I run bibfish main.tex bib.bib references.bib in Anaconda Prompt. Then I got an error
(base) C:\Users\Akira>cd C:\Users\Akira\Downloads\Compressed\PhD slides

(base) C:\Users\Akira\Downloads\Compressed\PhD slides>bibfish main.tex bib.bib references.bib
bibfish: Citekey 'le2024wellposedness' was not found in ['bib.bib']

My files are
PhD slides.zip
Could you have a check on this issue?

The problem happens because the entry le2024wellposedness has incorrectly-formatted line %pubstate = {under review}.


Please fix the docs re: visibility

Hello,
The current docs are confusing on the topic of visibility !
Head on over to http://foundation.zurb.com/sites/docs/visibility.html and you get told:

Depreciation Notice: From v6.5.x, we are moving visibility classes to Prototype specific mode and thus visibility classes will be disabled by default. You can re-enable it though, with a simple @include.

Ok, great, so we head on over to http://foundation.zurb.com/sites/docs/prototyping-utilities.html and we get told:

Prototyping classes like these below should only be used for prototyping tasks. Also if you are using Sass, we encourage you to use Prototyping Mixins instead.

and

Not all projects require Prototyping Utilities and adding utility classes like these increase your CSS file size especially if you're not using all of them. For these reasons Prototype mode is disabled by default.

and

For display: none use .hide. or Foundation's Visibility Classes

So the documentation is both confusing ("use prototyping... but don't use prototyping") and in an infinite loop ("for none or hide, use visibility classes" ... whose doc page then points you back to prototyping !).
So, please guys, make your minds up.   Is Visibility going or not ?
If it is going then please clarify the current preferred way to do visibility in prototyping (and if that is the case, perhaps tweak the wording so that prototyping classes sound less like something you would only use for prototyping, but actually have real-life uses like visibility).
If visibility is not going anywhere, then fix the wording on the visibility page.
Thank you. ;-)

I think it makes sense that we don't depreciate visibility classes!
Moreover, I think we need a separate Utility section...
Float classes, and text alignment should be moved though

Please check #10632
BTW I was the only one who added dep notice to visibility classes at first place ... but I have to say I agree with you!

I guess this is already solved as the PR is merged or is something missing or some more feedback needed here?

This is still very confusing. Are visibility classes being moved to prototype class and if so why?

@Bakuryu. No, visibility classes are not moved to prototype classes. The warning is incorrect and I am looking at why it is still visible.

I found why this notice is still there.
https://github.com/zurb/foundation-sites/pull/10632 did removed this notice, merged in september 2017 in https://github.com/zurb/foundation-sites/commit/647a42f28ca6d7b080c37992499727c7716a046d on develop...
But a change has been made on the same spot in August 2017 in https://github.com/zurb/foundation-sites/pull/10519 on master, and master got merged into develop this April (https://github.com/zurb/foundation-sites/pull/11175#issue-181668644).
My bad, I made some mistake in conflict resolution and did not removed the notice while merging.


Added information about the website in the about section in basil.html

Fixed issue #3

Hi @ali-alzaiat ,
Thanks for contributing to this repo man ,but somebody had already pushed those changes previously .
so a new conflict was arising in between the files.
Im regretting for this inconvinence caused to you , so are you willing to change the about section in neem.html and aloevera.html .  I would create an issue and assign it to only  you ,so  are you willing to accept it.

just give one to two lines of info on about section of these pages.
and also i created an issue for you so please message me to assign that issue as soon as possible so that i could assign it to you.

Thank you for the offer, but I'm afraid I won't be able to accept the assignment to change the about section in neem.html and aloevera.html files at this time.


how can we make row separator full width ? (separatorInset = .zero)

hi there, thanks for the great control.
how can we make the row separator, go full width ?

You can override customCellConfiguration to adjust cell seperatorInset.
dropDown.customCellConfiguration = { (index, item, cell) in
      cell.separatorInset = .zero
      cell.layoutMargins = .zero
}


OSGi Feature details page shows sections with no data

Feature details page shows section headers and table headers even when there is no data to show and it looks a bit ugly. Those sections could be hidden when they have data.

Fixed.


Add nebular footer demo

Please read and mark the following check list before creating a pull request (check one with "x"):

[x] I read and followed the CONTRIBUTING.md guide.
[x] I read and followed the New Feature Checklist guide.

Short description of what this resolves:
I added Made with Nebular with a link who redirect on Nebular home page (https://akveo.github.io/nebular/#/home?utm_source=ngx-admin&utm_medium=footer_link) in the footer of demo.
It's like this but only on demo branch : (https://github.com/akveo/ngx-admin/pull/1531)
@nnixaa It's ok to you?

@Pterrat it's great, but it is still on master

@nnixaa Sorry but I can't change my base branch in this PR? akveo:master is disabled
https://github.com/blog/2224-change-the-base-branch-of-a-pull-request
Should I create a new PR again?

@nnixaa done :)

@Pterrat now the PR has a lot of 3rd party changes for some reason, you probably rebased it on master or something?

Done it here https://github.com/akveo/ngx-admin/commit/4050525f97aeb5f3f84d7d0d5f27cc4e11d1b89b. Thanks @Pterrat!


reorder compiler options to respect -Wno-sign-compare

-closes #3867

Do we want to remove the sign compare warnings?

I'm not sure if it actually catches real problems.

Rebased and applied.


Please add this in app's features list.

Sir did you knew that your Image to PDF silently compresses pdf and makes the size of the PDF smaller.
PROOF
I took 3 jpg pictures all of size 3 mb approx. and 3*3=9 mb
Then when the PDF was made I was shocked the PDF was only of 4 mb and it had no quality loss.
And this feature was not even listed in features list.
How was this thing done?
Best compressing I have ever seen in any pdf app.

Thank you for the appreciation!


Override native default label if config server default label is not null

Fixes #1184

Codecov Report

Merging #1185 into 2.0.x will decrease coverage by 0.11%.
The diff coverage is 100%.

@@             Coverage Diff              @@
##              2.0.x    #1185      +/-   ##
============================================
- Coverage     78.19%   78.07%   -0.12%
- Complexity      947      949       +2
============================================
  Files           113      113
  Lines          3352     3353       +1
  Branches        461      462       +1
============================================
- Hits           2621     2618       -3
- Misses          561      567       +6
+ Partials        170      168       -2

Impacted Files
Coverage Δ
Complexity Δ

...nvironment/NativeEnvironmentRepositoryFactory.java
100% <100%> (ø)
3 <0> (+1)
:arrow_up:

...fig/client/ConfigServicePropertySourceLocator.java
65.95% <0%> (-3.55%)
25% <0%> (ø)

...rk/cloud/config/client/ConfigClientProperties.java
89.65% <0%> (-1.73%)
40% <0%> (-1%)

...erver/environment/NativeEnvironmentRepository.java
84.89% <0%> (+0.71%)
44% <0%> (+1%)
:arrow_up:

...erver/environment/NativeEnvironmentProperties.java
66.66% <0%> (+8.33%)
9% <0%> (+1%)
:arrow_up:

Continue to review full report at Codecov.

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


Switch to a different 2d framework

Upon a little investigation, it appears that the main Slick2D development is stale (no changes since mid-2013 on the development branch).
This means that if there are any issues with Slick2D (e.g. #9), they won't ever be fixed unless we create our own fork and try to fix it ourselves (and after briefly glancing at the Slick2D repo - just no).
It would definitely be worth our while to port to another framework. A few of the others I've seen also abstract the differences between the desktop and other devices (e.g. Android, iOS), which means making the game available on those platforms would be a piece of cake (pretty much just a few extra lines).
There is a page on the LWJGL wiki naming a few 2D libraries that are based off of LWJGL (or have multiple possible backends, one of which being LWJGL).
I want GPRG to be the best it can be - and I don't think that it can be that while it is stuck on an old framework.
Yes, it would take a fair amount of work to perform the refactor (a milestone of its own), but I seriously think that it would be worth it.

I'm making the executive decision here - we are switching to LibGDX.
This will push all 0.x milestones back by one, and the port will be published under 0.3.0.


Support for NUnit4

Description
Added support for NUnit4. I know it's in the 5.0 branch, but maybe we could do this one first?

Copied the NUnit3 projects
Modified package references
Changed the assert syntax in the test project
Changed namespaces and other NUnit3 things accordingly

Closed issues
None
Checklist

[x] Reviewed the contribution guidelines
[x] Linked the issue(s) the PR closes
[x] Implemented automated tests and checked coverage
[x] Provided inline documentation comments for new public API
[x] Ran the full solution build and validation locally

@dotnet-policy-service agree


isOpfsSupported breaks if source is transformed

isOpfsSupported converts a function to a string to create a JS source string https://github.com/subframe7536/sqlite-wasm/blob/08c341a7af1ea2e2cd0db979c781dcf0df00f13f/src/utils.ts#L41-L46 which is then loaded into a worker.
If this code is run through certain code transforms (e.g. tsc with target of es2015), the inner function's string representation can become invalid worker code. In my project, inner.toString() === 'e', so the resulting worker source was (e)().then(postMessage) (and e's definition is outside of the worker) – attempting to use isOpfsSupported raises an error because e.then is not defined.
I think with target = es2015, any async function would become invalid: demo.

This is an issue for me because I'm using your dialect-wasqlite-worker (which is otherwise great, ty), which has an unskippable check to isOpfsSupported when trying to use OPFS.
I don't need a fix urgently because I'm probably just going to patch dialect-wasqlite-worker to unblock myself (I can assume OPFS is available in my app), but thought I'd bring this up as a potential pitfall.

Thanks for reporting! I will fix it later

Maybe this will works: link

Release in v0.5.0-beta.2

sorry for silence on my end - I applied that change but couldn't figure out if it fixed my problem. (I wasn't able to create a minimal repro.) Thanks for addressing though!

Release in v0.5.0


Netflix WidevineCdm

I can make a package with WidevineCdm?

I too would like to know if Widevine is supported.

This is a duplicate of #113! It's not supported now, but feel free to submit a pull request!

@LutzPS  @Silent-Hunter  It has been fixed in WebCatalog webcatalog/desktop#4

I'm new with nativefier and I had the same problem with  Spotify, browsing the issue I think it was fixed but is have to be approved before going in the main branch: https://github.com/jiahaog/nativefier/pull/440
Sorry I'm a newbie with github technical terms,
Regards :octocat:


rm test_server http support, tests are now only https

Pros:

slightly faster test
simpler test framework
Cons:
skipping the HTTP server case in test

Codecov Report
Base: 59.32% // Head: 59.07% // Decreases project coverage by -0.25% :warning:

Coverage data is based on head (1c83f5e) compared to base (e13f49c).
Patch coverage: 96.42% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##              dev     #216      +/-   ##
==========================================
- Coverage   59.32%   59.07%   -0.26%
==========================================
  Files          28       28
  Lines        3995     3990       -5
==========================================
- Hits         2370     2357      -13
- Misses       1625     1633       +8

Impacted Files
Coverage Δ

lib/src/lightclient/test_server.rs
75.09% <96.42%> (-0.68%)
:arrow_down:

lib/src/grpc_connector.rs
72.83% <0.00%> (-4.94%)
:arrow_down:

lib/src/lightclient/checkpoints.rs
66.66% <0.00%> (-4.77%)
:arrow_down:

lib/src/wallet_internal_memo_handling.rs
65.30% <0.00%> (-1.37%)
:arrow_down:

lib/src/wallet.rs
50.16% <0.00%> (-0.17%)
:arrow_down:

lib/src/lightclient.rs
75.49% <0.00%> (-0.14%)
:arrow_down:

lib/src/blaze/fetch_full_transaction.rs
73.50% <0.00%> (ø)

lib/src/wallet/transactions.rs
39.28% <0.00%> (+0.21%)
:arrow_up:

lib/src/wallet/keys/unified.rs
51.69% <0.00%> (+0.41%)
:arrow_up:

lib/src/blaze/test_utils.rs
89.30% <0.00%> (+0.55%)
:arrow_up:

... and 2 more

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

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


Latest release cannot start VM

I just updated to the latest kube-solo and when attempting to bring up the VM, I get this:
reating 20GB sparse disk (QCow2)...
/Applications/Kube-Solo.app/Contents/Resources/functions.sh: line 152: /Users/notyou/bin/qcow-tool: No such file or directory
-
Created  Data disk

Starting VM ...

/Applications/Kube-Solo.app/Contents/Resources/functions.sh: line 195: /Users/notyou/bin/corectl: No such file or directory

VM has not booted, please check '~/kube-solo/logs/vm_up.log' and report the problem !!!

I've updated to the latest corectl as well but nothing works.

hmm, this is something todo with corectl.app, as it copies it's files to ~/bin folder.
kube-solo app does not have those files by itself anymore

I think my corectl.app was borked.  I updated it too and when it started, it never prompted me for my password so I assume it just wasn't running.

phew

It was weird because starting the newly updated corectl.app didn't prompt an error but once I saw that its status wasn't Server is running, I got the idea that maybe it was borked.  Force quit and restart seems to have fixed it.

Same error here, I've reinstalled corectl.app but ~/bin/ is always empty.


Docstring formatting standard

Migrated from GitLab: https://gitlab.com/meltano/sdk/-/issues/146
Originally created by @aaronsteers on 2021-06-12 18:11:56

Now that docs are autogenerating, we should align on a docstring format. The three most common appear to be:

sphinx
google
numpy

I'm personally inclined towards the google syntax for readability and simplicity. It should also be noted that since type hinting is pervasive already (and should be considered required in this project), we should intentionally avoid redundant (and therefor potentially divergent) typing of args within docstrings.

View 19 previous comments from the original issue on GitLab


request: Working with Angular 6 datepicker control

Hi
I would like to automate selecting of a date via a 3rd party DatePicker control in Angular 6.
Would you be able to assist with a Code Snippet to get me started?
I've tried the following, but for some reason, on debug, the control comes back as null, like it cannot assign the variables correctly:
        [ControlDefinition("div", ContainingClass = "ngx-datepicker-calendar-container", ComponentTypeName = "datepicker")]
        public class CalenderCtrl : Control<TOwner>
        {
            [FindByClass("topbar-title")]
            public Clickable<TOwner> YearDropDown { get; private set; }
        }

Details:
plugin: ng2-datepicker: 3.1.1
any help would be greatly appreciated!

Date pickers with calendars are always tricky. It took some time to implement it. I used https://ng2-datepicker.jankuri.com/ sample for testing. Here are the working sources of NG2DatePicker<TOwner> control for ng2-datepicker:
NG2DatePicker`1.cs
using System;
using Atata;

namespace AtataSamples.NG2DatePicker
{
    [ControlDefinition(ContainingClass = "ngx-datepicker-container", ComponentTypeName = "date picker")]
    [Format("MMM d, yyyy")]
    [Culture("en-US")]
    [ControlFinding(typeof(FindFirstAttribute))]
    public class NG2DatePicker<TOwner> : EditableField<DateTime?, TOwner>
        where TOwner : PageObject<TOwner>
    {
        [FindFirst]
        [Name("Associated")]
        [Format(null)]
        [TraceLog]
        private TextInput<TOwner> AssociatedInput { get; set; }

        [FindFirst]
        [Name("Associated")]
        [TraceLog]
        private CalendarContainer Calendar { get; set; }

        protected override DateTime? GetValue()
        {
            string valueAsString = AssociatedInput.Value;
            return ConvertStringToValue(valueAsString);
        }

        protected override void SetValue(DateTime? value)
        {
            if (Value != value)
            {
                if (value == null)
                    throw new ArgumentException("Control does not allow to set null value.", nameof(value));

                if (!Calendar.Exists(SearchOptions.SafelyAtOnce()))
                    AssociatedInput.Click();

                var (selectedMonth, selectedYear) = Calendar.GetSelectedPeriod();

                if (selectedYear != value.Value.Year)
                    Calendar.SetYear(value.Value.Year);

                if (selectedMonth != value.Value.Month)
                    Calendar.SetMonth(value.Value.Month);

                Calendar.SetDay(value.Value.Day);
            }
        }

        [ControlDefinition(ContainingClass = "ngx-datepicker-calendar-container", ComponentTypeName = "calendar")]
        private class CalendarContainer : Control<TOwner>
        {
            [FindByClass("topbar-title")]
            [TraceLog]
            private Text<TOwner> Title { get; set; }

            [ControlDefinition("span[@class='day-unit']", ComponentTypeName = "day")]
            [TraceLog]
            private ControlList<Content<int, TOwner>, TOwner> Days { get; set; }

            [ControlDefinition("span[@class='year-unit']", ComponentTypeName = "year", Visibility = Visibility.Any)]
            [ContentSource(ContentSource.TextContent)]
            [TraceLog]
            private ControlList<Content<int, TOwner>, TOwner> Years { get; set; }

            [FindByIndex(0)]
            [TraceLog]
            private MonthNavigationButton PreviousMonth { get; set; }

            [FindByIndex(1)]
            [TraceLog]
            private MonthNavigationButton NextMonth { get; set; }

            public (int Month, int Year) GetSelectedPeriod()
            {
                string selectedMothYear = Title.Value;
                DateTime date = DateTime.ParseExact(selectedMothYear, "MMMM yyyy", Metadata.GetCulture());
                return (date.Month, date.Year);
            }

            public void SetYear(int year)
            {
                Title.Click();

                var yearItem = Years[x => x == year];
                yearItem.ScrollTo();
                yearItem.Click();
            }

            public void SetMonth(int month)
            {
                int monthShift = month - GetSelectedPeriod().Month;

                var changeMonthButton = monthShift > 0 ? NextMonth : PreviousMonth;

                for (int i = 0; i < Math.Abs(monthShift); i++)
                    changeMonthButton.Click();
            }

            public void SetDay(int day)
            {
                Days[x => x == day].Click();
            }

            [ControlDefinition("*[name()='svg']", ComponentTypeName = "button")]
            private class MonthNavigationButton : Control<TOwner>
            {
            }
        }
    }
}

For Testing
SomePage.cs
using Atata;

namespace AtataSamples.NG2DatePicker
{
    using _ = SomePage;

    public class SomePage : Page<_>
    {
        public NG2DatePicker<_> SomeDate { get; private set; }
    }
}

SomeTests.cs
using System;
using Atata;
using NUnit.Framework;

namespace AtataSamples.NG2DatePicker
{
    public class SomeTests : UITestFixture
    {
        [Test]
        public void TestNG2DatePicker()
        {
            DateTime[] dates =
            {
                DateTime.Today,
                DateTime.Today.AddYears(1),
                DateTime.Today.AddYears(-20),
                DateTime.Today.AddYears(15),
                new DateTime(2017, 5, 2),
                new DateTime(2017, 12, 31),
                new DateTime(2000, 8, 8)
            };

            var page = Go.To<SomePage>(url: "https://ng2-datepicker.jankuri.com/").
                SomeDate.Should.Equal(DateTime.Today);

            foreach (var date in dates)
            {
                page.SomeDate.Set(date);
                page.SomeDate.Should.Equal(date);
            }
        }
    }
}

Info Log
Sample of INFO log messages:
2018-09-06 01:38:03.2335 INFO Starting: Set "Sep 6, 1998" to "Some Date" date picker
2018-09-06 01:38:04.2018 INFO Finished: Set "Sep 6, 1998" to "Some Date" date picker (0.968s)
2018-09-06 01:38:04.2018 INFO Starting: Verify "Some Date" date picker value should equal "Sep 6, 1998"
2018-09-06 01:38:04.2328 INFO Finished: Verify "Some Date" date picker value should equal "Sep 6, 1998" (0.030s)
2018-09-06 01:38:04.2328 INFO Starting: Set "Sep 6, 2033" to "Some Date" date picker
2018-09-06 01:38:05.4054 INFO Finished: Set "Sep 6, 2033" to "Some Date" date picker (1.172s)

Trace Log
2018-09-06 01:44:53.3786 INFO Starting: Set "May 2, 2017" to "Some Date" date picker
2018-09-06 01:44:53.4326 TRACE Starting: Click "Some Date" date picker's "Associated" text input
2018-09-06 01:44:53.4865 TRACE Finished: Click "Some Date" date picker's "Associated" text input (0.053s)
2018-09-06 01:44:53.5595 TRACE Starting: Click "Some Date" date picker's "Associated" calendar's "Title" element
2018-09-06 01:44:53.6393 TRACE Finished: Click "Some Date" date picker's "Associated" calendar's "Title" element (0.079s)
2018-09-06 01:44:53.6403 TRACE Starting: Scroll to "Some Date" date picker's "Associated" calendar's "x == 2017" year
2018-09-06 01:44:53.8181 TRACE Finished: Scroll to "Some Date" date picker's "Associated" calendar's "x == 2017" year (0.177s)
2018-09-06 01:44:53.8191 TRACE Starting: Click "Some Date" date picker's "Associated" calendar's "x == 2017" year
2018-09-06 01:44:53.9671 TRACE Finished: Click "Some Date" date picker's "Associated" calendar's "x == 2017" year (0.147s)
2018-09-06 01:44:54.0200 TRACE Starting: Click "Some Date" date picker's "Associated" calendar's "Previous Month" button
2018-09-06 01:44:54.1090 TRACE Finished: Click "Some Date" date picker's "Associated" calendar's "Previous Month" button (0.088s)
2018-09-06 01:44:54.1090 TRACE Starting: Click "Some Date" date picker's "Associated" calendar's "Previous Month" button
2018-09-06 01:44:54.1730 TRACE Finished: Click "Some Date" date picker's "Associated" calendar's "Previous Month" button (0.063s)
2018-09-06 01:44:54.1730 TRACE Starting: Click "Some Date" date picker's "Associated" calendar's "Previous Month" button
2018-09-06 01:44:54.2389 TRACE Finished: Click "Some Date" date picker's "Associated" calendar's "Previous Month" button (0.065s)
2018-09-06 01:44:54.2389 TRACE Starting: Click "Some Date" date picker's "Associated" calendar's "Previous Month" button
2018-09-06 01:44:54.3009 TRACE Finished: Click "Some Date" date picker's "Associated" calendar's "Previous Month" button (0.061s)
2018-09-06 01:44:54.3019 TRACE Starting: Click "Some Date" date picker's "Associated" calendar's "x == 2" day
2018-09-06 01:44:54.7656 TRACE Finished: Click "Some Date" date picker's "Associated" calendar's "x == 2" day (0.463s)
2018-09-06 01:44:54.7666 INFO Finished: Set "May 2, 2017" to "Some Date" date picker (1.388s)

Feel free to ask if something is not clear in control implementation as it's a bit complex. Try it out and please let me know if it works for you.

You sir, are awesome!
Thank you so much for this.
Will be trying it out today and will feedback!

This worked like a charm! Thank you again!

Always welcome!


add basic background music controls

This adds a new timeline piece allowing to set background music.
Music crossfade on change, and fade out on stop.
To stop music, simply put a BackgroundMusic node without a file selected.
Closes #117

I have a few notes, but if you don't want to work on those I can merge and I'll fix later today:

The text on the timeline editor button should have two spaces at the start (amazing padding styling, I know) so it should be:
··Background Music instead of Background Music.
The theme should be the same color as the Audio Event to avoid the rainbow effect, but I don't really mind the difference since it is subtle.
When adding this kind of features it would be nice to add to the changelog so we don't forget to report what new features the plugin has.
The empty state of the event looks different to the empty state of the Audio Event

So ideally both should say "None" or none of them should have it. Both should have the delete icon.

The text on the timeline editor button should have two spaces at the start (amazing padding styling, I know) so it should be:
··Background Music instead of Background Music.

Fixed!

The theme should be the same color as the Audio Event to avoid the rainbow effect, but I don't really mind the difference since it is subtle.

I didn't want to use the exact same color to prevent errors, as the icon is already the same. I took the base color for Audio Events, but slightly darker to show it is a background sound, not a foreground sound. But yeah I can use the same color if you want.

When adding this kind of features it would be nice to add to the changelog so we don't forget to report what new features the plugin has.

True, it's easy to forget!

The empty state of the event looks different to the empty state of the Audio Event

This is because they behave differently. An empty audio event does nothing, while an empty background event stops the music. So there is no need to clear an audio event's track, but it might be useful for background. The ui is not that clear about this but I don't know how to explain it simply.

The empty state of the event looks different to the empty state of the Audio Event

i implemented the same behavior in audio event and background music: an empty audio event stops the previous one. So both foreground and background events have text explaining their behavior when no track is selected. What do you think ?

That looks good! I wasn't able to test with audio yet, but I'll merge and test later 👍🏻


need to document minimum required ffmpeg version

Problem
It's clear that spotDL is incompatible with certain ffmpeg versions. Users would benefit from this knowledge, and codifying it would help the product team.
This issue tracks writing and publishing documentation which explains (and tracks) which version of ffmpeg is required.
Solution
Add documentation.

From memory, theres somewhere where we wanted to make spotDL exit and not run if the correct FFmpeg version(s) are not installed. Can't find it rn as an issue/PR tho.

Found it at #1165. Will close this one in favour of that.
Seems you were the original poster as well aha

This is a bug about the documentation. Without documentation about which version of ffmpeg is required, users have no idea what to install.
#1165 addresses the issue of discovering an incompatibility (and any other error) at runtime. It does not teach the user how to setup spotDL. It won't tell the user which version is needed ... because nobody seems to specifically know.
This issue should be re-opened so that the documentation issue can be addressed, since it is orthogonal to the code issue.


Since v9.0.0 let blocks are identified with type: 'Identifier' instead of type: 'Keyword'

Since version 9.0.0 (the bug is not present in<EMAIL_ADDRESS>and below) the let blocks have type Identifier in the AST. const instead have the correct Keyword type.
This breaks some eslint rules like newline-after-var (that expects a newline after a chunk of variable definitions) as it does not subsequent let definitions as variable declarations.
this means that, for example,
const var1 = 1;
const var2 = 1;
const var3 = 1;

it's not an error but that
let variable1 = 1;
let variable2 = 1;
let variable3 = 1;

is !

The newline-after-var rule has been deprecated. Closing this issue


chore: promote gcp to stable

Promoting GCP to stable and trying new commands on the PR

/ok-to-test-managed sha=38ccd31

/ok-to-test sha=38ccd31

Nice :) seems commands woked like they should

We don´t need to merge this one yet, just wanted a new PR with new stuff to test the new ok-to-test command

@moolen do we wanna merge this one? Do you see something missing?

/approve

/merge


Docs 402 "Payment required" Error

Describe the bug
The documentation is not reachable.
To Reproduce
open https://www.bazzite.com/docs/nuxt-optimized-images/

Hello @fvonellerts. Thanks for opening the issue and Sorry about that, It's now fixed 🤓.

@juliomrqz What can I say, that was fast ;) Thanks!


Release google-cloud-java v0.88.0

This pull request was generated using releasetool.
04-17-2019 11:22 PDT
BigQuery

Added listPartitions. (#4923)
Fix close write channel (#4924)

Compute

Regenerate compute client (#4909)

Contributions

cloud-storage-contrib-nio: Add support for newFileChannel (#4718)

Core

Optimize Date.parseDate(String) (#4920)

Data Catalog

Initial Generation of data catalog (#4957)

Datastore

Adding support to custom ports on LocalDatastoreHelper (#4933) (#4935)

Dialogflow

Regenerate dialogflow client (#4928)

Firestore

Prep version numbers for Firestore GA. (#4968)
Regenerate firestore client (#4963)

KMS

Regenerate kms client (#4954)

Language

Regenerate language client (#4921)

Pub/Sub

Encapsulating a notion of "message batch" in pub/sub (#4951)
More Pubsub cleanup (#4948)
Cleaning up deprecated pubsub code. (#4949)
Minor cleanup in pubsub (#4945)
PubSub subscriber cleanup (#4947)
Change MessageDispatcher to be synchronous instead of asynchronous. (#4916)

Redis

Regenerate redis client (#4910)

Scheduler

Regenerate scheduler client (#4911)

Security Center

Regenerate securitycenter client (#4937)
Regenerate securitycenter client (#4929)
CSCC Snippets (#4917)

Spanner

Refactor SpannerImpl - Move SessionImpl to separate file (#4895)
Refactor SpannerImpl - Move TransactionRunnerImpl to separate file (#4896)
refactor SpannerImpl: move PartitionedDmlTransaction to separate file (#4894)
Refactor SpannerImpl - Move InstanceAdminClient to separate file (#4893)
Refactor SpannerImpl - Move DatabaseAdminClientImpl to separate file (#4892)
Refactor SpannerImpl - Move AbstractReadContext to separate file (#4890)
Refactor SpannerImpl - Move AbstractResultSet to separate file (#4891)

Speech

Prep version numbers for Speech GA. (#4970)

Storage

Fix manage resumeable signedURL uploads. (#4874)
Un-Ignore testRotateFromCustomerEncryptionToKmsKeyWithCustomerEncryption (#4936)

Tasks

Regenerate tasks client (#4912)

Trace

Regenerate trace client (#4931)

Dependencies

Upgrade gax to 1.44.0. (#4966)

Documentation

Use splitindex for google-api-grpc javadoc. (#4952)

Thanks @chingor13! Updated the comment.

Release is at https://github.com/googleapis/google-cloud-java/releases/tag/v0.88.0


Requested Deep Nodes

Hello,
For later when we are using deep, these open source tools are good
https://github.com/charlesangus/DeepC

These are compiled for Nuke versions lower than we are using (15).
https://github.com/charlesangus/DeepC/releases/tag/v0.5.6
Do we want to try these if they work or do we want to compile the whole set again?

We should put in an issue on his repo because he made some fixes for 14 and people have commented. Unless you want to compile them. Whatever is easier. I think this is low prio. I doubt we'll be doing any deep stuff anytime soon.

We can close this for now. Tackle it after we have some deep outputs we rendered to test with.


[Code Addition Request]: Steganography using Open Cv & Matplot lib

Have you completed your first issue?

[X] I have completed my first issue

Guidelines

[X] I have read the guidelines
[ ] I have the link to my latest merged PR

Latest Merged PR Link
N/A
Project Description
Hidding one secret image inside a image. Embedding secret image to two lsb of cover image.
It's has also MSE, PSNR statistical analysis of the output for understanding the correctness of the model.

Full Name
Suvadip Sana
Participant Role
gssoc-extd, hacktoberfest-accepted

@Suvadip-sana raise 1 issue at a time.


Backwards compatability break

Version 10.7.1 introduced a breaking change to the JsonSchemaValidator class in a patch (non-breaking) version update. It replaced the constructor public JsonSchemaValidator(params IFormatValidator[] customValidators) with public JsonSchemaValidator(JsonSchemaValidatorSettings settings). This has broken a lot of our integrations with packages like Confluent.SchemaRegistry.Serdes.Json.

@RicoSuter @drinehim #1520 solves only constructor not the validate methods:
System.MissingMethodException: Method not found:
System.Collections.Generic.ICollection`1<NJsonSchema.Validation.ValidationError>NJsonSchema.JsonSchema.Validate(System.String, NJsonSchema.Validation.FormatValidators.IFormatValidator[])'.

Before 10.7.1:
public ICollection Validate(JToken token, params IFormatValidator[] customValidators);
After:
public ICollection Validate(string jsonData, JsonSchemaValidatorSettings settings = null);
Schema.Validate(strJson) will fail at runtime if a newer package version is included somewhere else.

Thanks for pointing that out @bruegth just created PR #1557

Just ensure you use the same versions,
current releasable version will be v11 and it will anyway breaking.


Patch on multi table combiner and test case

Description

Motivation and Context

How has this been tested?

Types of changes

[ ] Maintenance (no change in code, maintain the project's CI, docs, etc.)
[ ] 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)

Checklist:

[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.

Codecov Report
Attention: 12 lines in your changes are missing coverage. Please review.

Comparison is base (9b4c683) 75.87% compared to head (94c3c70) 75.93%.

Files
Patch %
Lines

sdgx/data_models/multi_table_combiner.py
0.00%
10 Missing :warning:

sdgx/data_models/relationship.py
0.00%
2 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #89      +/-   ##
==========================================
+ Coverage   75.87%   75.93%   +0.05%
==========================================
  Files          58       58
  Lines        2230     2227       -3
==========================================
- Hits         1692     1691       -1
+ Misses        538      536       -2

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


UPS Power up PI issue

Hi, is there any method to control ups power up PI only when the battery charging level is above protection voltage/ empty voltage?

same issue with me :(
https://github.com/geeekpi/upsplus/issues/115

Hi Jiksaw0221,
It can not to control UPS Power up Pi due to the chipset on board is like a power bank, it will up once the UPS is power up.


Update DG with effort appendix

Add effort appendix to the DG.
Fix broken links.

Codecov Report

Merging #188 (0345de0) into master (e3eed85) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #188   +/-   ##
=========================================
  Coverage     75.06%   75.06%
  Complexity      671      671
=========================================
  Files           100      100
  Lines          2049     2049
  Branches        243      243
=========================================
  Hits           1538     1538
  Misses          410      410
  Partials        101      101

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


risc-v/mmu: Fix L3 mappings for kernel, and mpfs protected mode users…

…pace
Summary
The L3 mapping function was just way too simplistic. Depending on memory configuration it either works or not.
Noticed that with icicle:pnsh the software crashes due to instruction page fault, reason is the map_region() implementation that does not work for regions that are not aligned to 2MB (the L2 page size).
Implemented an extremely simplistic page table allocator for the L3 references, that should once and for all get rid of the L3 mapping issue.
NOTE: gran_alloc() cannot be used at this point, it is too early for it.
Impact
Fixes full system crash if the kernel memory regions (or user memory regions in protected mode) are selected so that they
are not aligned to 2MB boundary.
Testing
icicle:pnsh / icicle:knsh

@masayuki2009 I know you use the qemu-rv / rv-virt target, I cannot test / verify that because I don't have the environment set up. If it breaks can you please inform me and I'll fix it.

@pussuw
Let me try this PR later.

@pussuw Let me try this PR later.

Sounds good!

@pussuw
I confirmed that qv-virt:knsh64 works without a side effect.

@pussuw I confirmed that qv-virt:knsh64 works without a side effect.

Excellent, thank you!


Replace all build-ecs commands by sdk-commands build

For consistency

closed in https://github.com/decentraland/js-sdk-toolchain/commit/054d424646fc6082e7a29ebf4cc5903454205935


Run postBuild script after installing app environment.

After pondering over this for a little while, I'd like to withdraw this PR and put more thought into this, because we can't guarantee that changes made by these scripts are isolated to the app environment.

Replaced by #98 .


Requirements to use M365 Lighthouse

Is it possible for an ordinary company to use Microsoft 365 Lighthouse to manage their multiple tenants?
Or is this service only for MSSPs?

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

ID: b4b28125-703d-234a-b52f-2a46de32ffa6
Version Independent ID: 55d48817-9185-2fd9-55cf-cea275666fbe
Content: Requirements for Microsoft 365 Lighthouse - Microsoft 365 Lighthouse
Content Source: microsoft-365/lighthouse/m365-lighthouse-requirements.md
Service: microsoft-365-lighthouse
GitHub Login: @SKjerland
Microsoft Alias: sharik

@ShawnHag Thank you for your feedback. Currently, it is only available for  Managed Service Providers (MSPs). Note that even
MSPs must be enrolled in the Cloud Solution Provider (CSP) program as an Indirect Reseller or Direct Bill partner to use Lighthouse.
Hope this helps!
Thanks
Sri

@msbemba Thank you very much for the contribution and sharing this explanation. @ShawnHag Hope this comment is helpful for you. If you see a documentation update is required, please feel free to open an issue for the same. We proceed here to close it. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


feat: ledger blind signing

resolves #33

verify:

tx link exactly from flow on screens above

@dj8yfo Is there a chance to improve the error message for old Ledger App?


Play

What do you want to do with Clappr?
I want to play the livestream file. I add this m3u8 link “ https://i-am-so-cool.licenses4.me/cdn/premium31/chunks.m3u8?md5=Njgc81Udw63VE0wHYJXncg&expires=1646751049” into the source as the following.

What have you tried so far?
But the player didn’t play the file. Why? How can I fix it? My blogger site is https://www.football-live.co.uk/?m=1

Hello @heinhtwem! Could you share more information regarding this issue? I fear it may be related to your CDN.
I tried loading the livestream source you shared on my machine, and I'm receiving a 403 error when a request to https://uacdn.ga/ is made by your CDN. You can check that by your browser's Network tab.
Furthermore, I did let Clappr try to load the stream for a few more minutes. It did occasionally play for a few seconds, but it quickly stopped after that.

it is because this person is trying to use someone elses stream lol


Fix crash if the len was < maxSetStringLen

If the len was less than the max length, there was a panic
Signed-off-by: Flavio Crisciani<EMAIL_ADDRESS>

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@c2080b3). Click here to learn what that means.
The diff coverage is 0%.

@@            Coverage Diff            @@
##             master    #2096   +/-   ##
=========================================
  Coverage          ?   40.38%
=========================================
  Files             ?      139
  Lines             ?    22356
  Branches          ?        0
=========================================
  Hits              ?     9029
  Misses            ?    12000
  Partials          ?     1327

Impacted Files
Coverage Δ

service_common.go
6.97% <0%> (ø)

Continue to review full report at Codecov.

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

Comment says "If the len was less than the max length, there was a panic". But in general there will be panic only if we exceed max length. In code also you check if length > maxLength ....
I Hope this is just typo

@selansen no that is the bug in fact. take a look at https://play.golang.com/p/J2TltYR77p1

now I see it..My bad..Thanks.


Change some checks to acommodate for clients where ICMP is disabled by firewall

When testing for connectivity Use NetCat instead of ping to test BMC and NTP connection, since some firewalls might block ICMP.
On the ohter hand the "bmc_ip" variable should not be required for vendor: PXE

Theres another ping for testing network connectiveity after appling the network config. Could you update this one as well.
https://github.com/redhat-partner-solutions/crucible/blob/main/roles/apply_nmstate/tasks/main.yml#L40

Theres another ping for testing network connectiveity after appling the network config. Could you update this one as well. https://github.com/redhat-partner-solutions/crucible/blob/main/roles/apply_nmstate/tasks/main.yml#L40

@nocturnalastro in this case it is OK to have a ping because that IP address does not correspond to any type of service so I wouldn't know to which port NetCat should probe. Besides it is configurable to any host the user chooses.


New Crowdin updates

These files are likely horribly out of date. en.json should match translation.json but differs vastly and is causing recurring problems on production for users of the sheet as a result. I'm not sure what CrowdIn is doing here, but "failing" certainly seems like the first blush diagnosis.


Syntax error when using variable

I have checked the following:

[X] I use the newest version of bruno.
[X] I've searched existing issues and found nothing related to my issue.

Describe the bug

[optional] Create any runtime variable in a "post response" script
Create a new query, switch to tab "Variables"
Define a new variable that simply uses the runtime variable:

{
  "lastTrackingWorkdayId": {{lastTrackingWorkdayId}}
}

=> Find out, that the variable usage is producing a syntax error

[optional] Run the query & find out that the variable is still working.

.bru file to reproduce the bug
No response
Screenshots/Live demo link

Hey @ckilb,
Thanks for raising this issue.
I reproduced it and found its bug. We're working on it.


Follow player

Allow to follow a player; if a player joins a game, the user joins the same game.

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

maybe we should form groups instead and it opens a separate chat channel for them

Groups (or as they're commonly called, "Parties") would be nice, however they'd need server-side support. Following a player is almost the same and we can do it client-only (for now). People can always create new chat channels, if desired.

Softly is working on the matchmaker, maybe we can work with him to create a grouping system that will work in both custom games alongside the matchmaker http://forums.faforever.com/forums/viewtopic.php?f=45&t=11400

Good thought, however you can't really prevent it since the information which game you're in is always shared.

I think this can be closed. TMM is going to solve this. As for custom games: if a friend joins a game you get a notification and can join that game either by clicking on the notification or rightclicking in the chat tab. i regularly play with a friend and dont feel like this is a problem right now and even less when TMM is expanded to more than 2v2s.

I agree


Robotium can't click on TextViews if dialog is opened multiple times

From<EMAIL_ADDRESS>on August 08, 2011 05:51:35
In my app I have a ListView which can be filled through a dialog which contains another ListView with TextViews nested in LinearLayouts. By clicking on any of these TextViews they're added to the (main) List and the dialog is closed. But there's a strange issue where Robotium can't click on the TextViews if the dialog is opened multiple times: What steps will reproduce the problem? 1. Open dialog
2. Click on TextView (item is added, dialog is closed)
3. Open dialog again
4. Attempt to click on TextView again --> Robotium can't "see" or click the TextView. What is the expected output? What do you see instead? solo.clickOnText produces the following error trace:
junit.framework.AssertionFailedError: The text: Item 2 is not found!
at com.jayway.android.robotium.solo.Clicker.clickOnText(Clicker.java:282)
at com.jayway.android.robotium.solo.Solo.clickOnText(Solo.java:830)
at foo.bar.test.AddItemsTest.addItemClickOnText(AddItemsTest.java:68)
at foo.bar.test.AddItemsTest.testAddItemsClickOnText(AddItemsTest.java:74)
[...]
The text is clearly visible on the screen and running the app and manually clicking on the items works perfectly fine all the time. What version of the product are you using? On what operating system? Robotium 2.4. with both Android 2.1 and 2.2 on emulator and HTC Desire Please provide any additional information below. I've done a little debugging and traversed the ListView manually. Turns out the ListView contains all the TextViews (and I can access them too), but getCurrentTextViews doesn't return them all the second time the dialog is opened. When I traverse the ListView manually to get the View I want to click on and use solo.clickOnView nothing happens. No click, but no exception or failed assertion either.
I've created a (somewhat) minimal example of the problem and attached it. Just run the test project to see the problem.
Attachment: ListExample.zip
Original issue: http://code.google.com/p/robotium/issues/detail?id=136

From<EMAIL_ADDRESS>on August 08, 2011 22:36:06
Thanks for this. The issue will be analyzed and if possible fixed before the next release.
Summary: Robotium can't click on TextViews if dialog is opened multiple times
Status: Accepted

From<EMAIL_ADDRESS>on August 08, 2011 23:01:25
Owner<EMAIL_ADDRESS>

From<EMAIL_ADDRESS>on August 31, 2011 13:10:15
Unfortunately this is a platform issue and not a bug in Robotium.
Status: Invalid


Respect ruleProcessor option for "sql" format in formatQuery

Currently the ruleProcessor option is ignored for SQL-based formats.
For the "sql" format, substitute the default rule processing logic with a custom function when one is provided with the ruleProcessor option.
Do not use a custom ruleProcessor for "parameterized" and "parameterized_named" (yet). Possibly the subject of an upcoming issue.

@lionturtle99 I've got this working in #488. You can play around with the latest build from that branch here or one of the other links from https://github.com/react-querybuilder/react-querybuilder/pull/488#issuecomment-1467196220.
The docs have been updated too, but only the "Next" version for now. I'll update the v6 docs once v6.1.0 is released. Preview of the new "Export" docs is here (that link will go straight to the "Rule processor" section).

This functionality is included in v6.1.0.


fix(groomer profile update bug)

Description
the groomer profile would not update to reflect edits until hard refresh, this issue has been
resolved
Type

[ ] This is a bugfix.
[ ] This is a new feature.
[ ] This is part of a feature.

Organization

[ ] Have you linked this pull request to a Trello card?
[ ] Have you named the branch in the format of feature/describe_feature_or_change OR bug/describe_bug_this_is_fixing

Review

[ ] Is this a work in progress?
[ ] Is this ready to be reviewed?
[ ] Have you added two reviewers to this pull request?

 groomer profile should display changes on update without hard refresh


Bug that node="[object Object]" is specified when customizing markdown

Which package has an issue?
@yamada-ui/markdown
Package Version
1.0.41
Description
Customizing the markdown as follows will result in the image.

const components = {
  h2: (props) => (
    <Heading
      size="2xl"
      bgGradient="linear(to-l, #f37bdf, #59a9e1)"
      bgClip="text"
      isTruncated
      {...props}
    />
  ),
}

return <Markdown components={components}>{README}</Markdown>

Link to Reproduction
https://yamada-ui.com/components/data-display/markdown#using-custom-components
Steps to Reproduce

Go to '...'
Click on '...'
Scroll down to '...'
See error

Browser
None
Operating System

[ ] macOS
[ ] Windows
[ ] Linux
[ ] iPadOS
[ ] iOS
[ ] Android

Do you know how to fix the issue?
None
Are you willing to participate in fixing this issue and create a pull request with the fix?
None
Additional Information
No response

@koralle
Do you know anything about this?

@illionillion
Thanks for reporting! I'll take a look at it myself.


Fix package shadow by parameter name

Description
Fix package shadow by parameter name
Type of change

Refactor (refactoring code, removing useless files)

Testing steps
Done on CI
Checklist

[ ] make before_commit passes
[ ] updated documentation wherever necessary
[ ] added or modified tests if necessary
[ ] updated schemas and validators in insights-data-schemas in case of input/output change

Codecov Report
Patch coverage: 100.00% and no project coverage change.

Comparison is base (96c9358) 67.87% compared to head (b955636) 67.87%.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #1044   +/-   ##
=======================================
  Coverage   67.87%   67.87%
=======================================
  Files          24       24
  Lines        3331     3331
=======================================
  Hits         2261     2261
  Misses        827      827
  Partials      243      243

Impacted Files
Coverage Δ

server/server.go
60.70% <100.00%> (ø)

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


EIP-681 QR and Payment Link

This PR adds a new Send/Receive switch.

In Receive mode, an EIP-681 QR code is generated, with the chain/token/address/amount info.
Clicking on the QR copies the payment link, which can be sent over to a friend, e.g. requesting 10USD on polygon.
Scanning the QR or pasting the payment link switches to the requested chain/token and populates the toAddress and the amount.
https://github.com/scaffold-eth/punk-wallet/assets/1397179/5794e621-8ba3-4a18-8cf9-31450d42f864
As the QR codes use EIP-681, we can use other wallets like MetaMask mobile to scan/generate payments.
This is an example for scanning a MetaMask payment QR for 0.1 OP token.
https://github.com/scaffold-eth/punk-wallet/assets/1397179/ab332468-bad8-4844-977a-4d94a0eccee2
Demo Site:
https://erratic-trick.surge.sh/

cc @agocsvince


Add tests for readcoda and compose functions

Addresses #10.
Adds the jura dataset and test sets for functions readcoda and compose.
Slight modifications to readcoda and compose functions are introduced for compatibility with the Vector{Symbol} cols dispatch type. An additional dispatch method is introduced for Array{String, 1} cols.

@mralbu what we can do also is consider using DataDeps.jl to download the dataset on the fly during tests.
I quickly checked the link of the dataset, and it seems to be released under a GPL licence, unfortunately. I don't know to what extent this is an actual issue given that we only intend to consume the data though. Do you have an idea if it would be fine? Meanwhile, I will ask people in the community to see if someone has experience with this.

Codecov Report

Merging #11 into master will increase coverage by 9.53%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master      #11      +/-   ##
==========================================
+ Coverage   72.41%   81.94%   +9.53%
==========================================
  Files           4        3       -1
  Lines          87       72      -15
==========================================
- Hits           63       59       -4
+ Misses         24       13      -11

Impacted Files
Coverage Δ

src/utils.jl
100.00% <100.00%> (+100.00%)
:arrow_up:

src/composition.jl
62.85% <0.00%> (-2.15%)
:arrow_down:

src/transforms.jl
100.00% <0.00%> (ø)

src/CoDa.jl

Continue to review full report at Codecov.

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

Closing in favor or #12


Tests without containers should run without errors on Windows, Mac, Linux ?

Currently when mvn clean install runs, there are errors during test run on my Windows system, all apply to the ApplicationContext (I'm thinking it's because of the jdbc.manager object is just completely empty because there are no test DB's or test containers that are running.)
There's about 20,000 lines that get produced from the Spring framework factories for each test.  V E R B O S E.
com.zaxxer.hikari.HikariDataSource       : HikariPool-1 - Starting...
2024-11-03T18:34:17.201+08:00  WARN 16896 --- [           main] ConfigServletWebServerApplicationContext : Exception encountered during context initialization - cancelling refresh attempt: org.springframework.beans.factory.BeanCreationException: Error creating bean with name 'dataSource' defined in class path resource [com/homihq/db2rest/jdbc/config/JdbcConfiguration.class]: Failed to instantiate [javax.sql.DataSource]: Factory method 'dataSource' threw exception with message: Failed to get driver instance for jdbcUrl=${DB_URL}

It would be MUCH better to fail very early and instead to log something to the console quickly if there's no DB metadata or config is empty or whatever and tell the user something helpful instead.  Like we couldn't find all the requirements needed in order to properly run tests (DB's, required config variables needed for tests, etc).  Perhaps something simple like this, but whatever the real requirements are in order to run the tests:
 System.err.println("In order to run tests for DB2Rest, you need to have a database connection " +
                "or ensure that Docker is started so that our test containers " +
                "can be automatically started for you.");

The only thing to do is to check if Docker demon is running.  No other
change is required.
Developers experienced will figure out less experience ones can ask
questions on Discord.
On Sun, Nov 3, 2024 at 8:02 AM Thad Guidry @.***> wrote:

Currently when mvn clean install runs, there are errors during test run
on my Windows system, all apply to the ApplicationContext (I'm thinking
it's because of the jdbc.manager object is just completely empty because
there are no test DB's or test containers that are running.)
There's about 20,000 lines that get produced from the Spring framework
factories for each test. V E R B O S E.
com.zaxxer.hikari.HikariDataSource       : HikariPool-1 - Starting...
2024-11-03T18:34:17.201+08:00  WARN 16896 --- [           main] ConfigServletWebServerApplicationContext : Exception encountered during context initialization - cancelling refresh attempt: org.springframework.beans.factory.BeanCreationException: Error creating bean with name 'dataSource' defined in class path resource [com/homihq/db2rest/jdbc/config/JdbcConfiguration.class]: Failed to instantiate [javax.sql.DataSource]: Factory method 'dataSource' threw exception with message: Failed to get driver instance for jdbcUrl=${DB_URL}
It would be MUCH better to fail very early and instead to log something to
the console quickly if there's no DB metadata or config is empty or
whatever and tell the user something helpful instead. Like we couldn't find
all the requirements needed in order to properly run tests (DB's, required
config variables needed for tests, etc). Perhaps something simple like
this, but whatever the real requirements are in order to run the tests:
System.err.println("In order to run tests for DB2Rest, you need to have a database connection " +
"or ensure that Docker is started so that our test containers " +
"can be automatically started for you.");
—
Reply to this email directly, view it on GitHub
https://github.com/9tigerio/db2rest/issues/757, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAISPRBCFJUD6CEVOUO4FWDZ6YUFVAVCNFSM6AAAAABRCYNHT2VHI2DSMVQWIX3LMV43ASLTON2WKOZSGYZTCMRSHA3TKNY
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

--
Kind Regards... Dhrubo
http://www.linkedin.com/in/dhrubo

There are no tests without containers. All tests run in test container.
Lets close this issue.


Sign in page is not obviously also a sign up page

Change text to make it clear the user can sign in OR sign up, immediately. Get rid of the small print, make it more obvious. This is on the /account page when not logged in.

You've seen the copy changes for this right?
sent from my phone, please excuse any typos.
On 11 Aug 2016 1:02 pm, "markmacgillivray"<EMAIL_ADDRESS>wrote:

Change text to make it clear the user can sign in OR sign up, immediately.
Get rid of the small print, make it more obvious. This is on the /account
page when not logged in.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/OAButton/backend/issues/209, or mute the thread
https://github.com/notifications/unsubscribe-auth/AEuWZwvA84b4QQn4UazGIdzhm5hwLXmpks5qew9ngaJpZM4JiDR1
.

Somewhere yes. But also a note to remind me of the particular issue Steve
has with it, for making it obvious for people signing up when installing
the plugin.
Mark
On 11 Aug 2016 13:51, "Joseph McArthur"<EMAIL_ADDRESS>wrote:

You've seen the copy changes for this right?
sent from my phone, please excuse any typos.
On 11 Aug 2016 1:02 pm, "markmacgillivray"<EMAIL_ADDRESS>wrote:

Change text to make it clear the user can sign in OR sign up,
immediately.
Get rid of the small print, make it more obvious. This is on the /account
page when not logged in.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/OAButton/backend/issues/209, or mute the thread
<https://github.com/notifications/unsubscribe-auth/
AEuWZwvA84b4QQn4UazGIdzhm5hwLXmpks5qew9ngaJpZM4JiDR1>
.

—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/OAButton/backend/issues/209#issuecomment-239151539,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAuXCJEuOFOjaql1Ew8Bx0q7MBf8ogiHks5qexq5gaJpZM4JiDR1
.

New site style fixes this


Add URLQueryItem variants to URL query parameter extensions

🚀
Fixes #1036
Checklist

[ ] I checked the Contributing Guidelines before creating this request.
[x] New extensions are written in Swift 5.0.
[x] New extensions support iOS 10.0+ / tvOS 9.0+ / macOS 10.10+ / watchOS 2.0+, or use @available if not.
[x] I have added tests for new extensions, and they passed.
[x] All extensions have a clear comments explaining their functionality, all parameters and return type in English.
[x] All extensions are declared as public.
[x] I have added a changelog entry describing my changes.

          1 Message

:book:
Thank you for submitting a pull request to SwifterSwift. The team will review your submission as soon as possible.

  Generated by :no_entry_sign: Danger

@guykogus @LucianoPAlmeida  There is a bug in changes when dealing with URLs initialized by URL(string:relativeTo:), as calling URLComponents(url: self, resolvingAgainstBaseURL: false) will cause a lost of base url. We should need to make resolvingAgainstBaseURL = true to make nothing got lost here.
Could you please update your fixes? Thank you.


➕  (deps) NICE-94 virtua (books|music) [b]

Help with rendering potentially long lists for Books & Music.

This pull request has been deployed to Vercel.

Latest commit:
3967801

✅ Preview:
https://jeromefitzgerald-k2uer66m4-jerome5.vercel.app

🔍 Inspect:
https://vercel.com/jerome/jeromefitzgerald/G4mPP4pbHyTan8mEErNgqwh4AEhg

View Workflow Logs


makeLoadViolation created for every file

When running config-lint for terraform I currently get these for everything
  {
    "AssertionMessage": "At 6:10: Unknown token: 6:10 IDENT var.tags",
    "Category": "load",
    "CreatedAt": "2020-04-17T14:09:09Z",
    "Filename": "config.tf",
    "LineNumber": 0,
    "ResourceID": "config.tf",
    "ResourceType": "file",
    "RuleID": "FILE_LOAD",
    "RuleMessage": "Unable to load file",
    "Status": "FAILURE"
  },

running strace shows
epoll_ctl(4, EPOLL_CTL_DEL, 3, 0xc0001fe5ec) = -1 EPERM (Operation not permitted)
for every file that is raised, which is reasonable since epoll can not be run on normal files.
Any clue what is going on here? I'm on void linux (musl).

ok, I have to correct myself. Seems like I was on an old release (0.44). Updated to 1.0 now and running into this issue:
$ config-lint -terraform .
Failed to load built-in rules for Terraform: File or directory doesnt exist

seems to try to open something in /home/runner ? what should be in there?

this seems like the build is broken somehow tbh -  I see that /home/runner only shows up in your github workflows. I assume that shouldn't be happening.

definitely tries to open some files in /home/runner/work/config-lint/config-lint/cli/assets/terraform

i'm trying to build it locally but still have some trouble

Replicated
~/config-lint/example-files/config# config-lint -debug -terraform
Looking for file terraform in Box: {./assets /home/runner/work/config-lint/config-lint/cli map[] map[]}
New Box: {./assets/terraform /home/runner/work/config-lint/config-lint/cli map[] map[]}
Failed to load built-in rules for Terraform: File or directory doesnt exist

so building it locally seems to work; will try once a new release is out if my initial issue is gone. If not I guess it's likely due to musl?

I'm working on a fix, it appears to be a relative pathing issue when referencing files in the packr box.
Thanks for raising the issue, we'll need to add better smoke tests as well.

sure, thx for the fast response; sh*t happens :) Trying it currently and for now it's really impressive :+1: Unlike tflint it actually finds something in our modules, which is great.

@kmonihen
im still seeing this in 1.1.0
config-lint -terraform main.tf -debug
Failed to load built-in rules for Terraform: File or directory doesnt exist
config-lint --version
1.1.0

@CubicNitrous
Please try with 1.2.0, the 1.1.0 release was cut without the fix

@kmonihen looks good in 1.2.0 thanks for the quick fix!

@AFriemann This should be fixed in 1.2.0. Thanks again for brining it to our attention.


CMD and PS shells complain of an error if installtion intentionally omitted support for even one ESP32 target (IDFGH-8879)

Answers checklist.

[X] I have read the documentation ESP-IDF Programming Guide and the issue is not addressed there.
[X] I have updated my IDF branch (master or release) to the latest version and checked that the issue is present there.
[X] I have searched the issue tracker for a similar issue and not found a similar issue.

IDF version.
v5.0
Operating System used.
Windows
How did you build your project?
Command line with idf.py
If you are using Windows, please specify command line type.
CMD
What is the expected behavior?
Using Python in %IDF_TOOLS_PATH%\python_env\idf5.0_py3.8_env\Scripts\
Python 3.8.7
Using Git in %IDF_TOOLS_PATH%\tools\idf-git\2.34.2\cmd\
git version 2.34.1.windows.1
Checking Python compatibility
Setting IDF_PATH: %IDF_TOOLS_PATH%\esp-idf-v5.0
Adding ESP-IDF tools to PATH...
%IDF_TOOLS_PATH%\tools\riscv32-esp-elf-gdb\11.2_20220823\riscv32-esp-elf-gdb\bin
%IDF_TOOLS_PATH%\tools\riscv32-esp-elf\esp-2022r1-11.2.0\riscv32-esp-elf\bin
%IDF_TOOLS_PATH%\tools\esp32ulp-elf\2.35_20220830\esp32ulp-elf\bin
%IDF_TOOLS_PATH%\tools\cmake\3.24.0\bin
%IDF_TOOLS_PATH%\tools\openocd-esp32\v0.11.0-esp32-20221026\openocd-esp32\bin
%IDF_TOOLS_PATH%\tools\ninja\1.10.2
%IDF_TOOLS_PATH%\tools\idf-exe\1.0.3
%IDF_TOOLS_PATH%\tools\ccache\4.6.2\ccache-4.6.2-windows-x86_64
%IDF_TOOLS_PATH%\tools\tools\dfu-util\0.9\dfu-util-0.9-win64
%IDF_TOOLS_PATH%\esp-idf-v5.0\tools
Checking if Python packages are up to date...
Constraint file: %IDF_TOOLS_PATH%\espidf.constraints.v5.0.txt
Requirement files:

%IDF_TOOLS_PATH%\tools\requirements\requirements.core.txt
Python being checked: %IDF_TOOLS_PATH%\python_env\idf5.0_py3.8_env\Scripts\python.exe
Python requirements are satisfied.

Detected installed tools that are not currently used by active ESP-IDF version.
For removing dfu-util, esp32ulp-elf, riscv32-esp-elf, riscv32-esp-elf-gdb, use command 'python.exe %IDF_TOOLS_PATH%\esp-idf-v5.0\tools\idf_tools.py uninstall'
For free up even more space, remove installation packages of those tools. Use option 'python.exe %IDF_TOOLS_PATH%\esp-idf-v5.0\tools\idf_tools.py uninstall --remove-archives'.
Done! You can now compile ESP-IDF projects.

Using Python in %IDF_TOOLS_PATH%\python_env\idf5.0_py3.8_env\Scripts\
Python 3.8.7
Using Git in %IDF_TOOLS_PATH%\tools\idf-git\2.34.2\cmd\
git version 2.34.1.windows.1
Checking Python compatibility
Setting IDF_PATH: %IDF_TOOLS_PATH%\esp-idf-v5.0
Adding ESP-IDF tools to PATH...
ERROR: tool xtensa-esp-elf-gdb has no installed versions. Please run '%IDF_TOOLS_PATH%\esp-idf-v5.0\install.bat' to install it.
ERROR: tool xtensa-esp32-elf has no installed versions. Please run '%IDF_TOOLS_PATH%\esp-idf-v5.0\install.bat' to install it.
ERROR: tool xtensa-esp32s2-elf has no installed versions. Please run '%IDF_TOOLS_PATH%\esp-idf-v5.0\install.bat' to install it.
ERROR: tool xtensa-esp32s3-elf has no installed versions. Please run '%USERPROFILE%\esp-idf-v5.0\install.bat' to install it.
ERROR: tool esp32ulp-elf has no installed versions. Please run '%IDF_TOOLS_PATH%\esp-idf-v5.0\install.bat' to install it.
ERROR: tool dfu-util has no installed versions. Please run '%IDF_TOOLS_PATH%\esp-idf-v5.0\install.bat' to install it.
%IDF_TOOLS_PATH%\esp-idf-v5.0> git describe --tag
v5.0
%IDF_TOOLS_PATH%\esp-idf-v5.0>
Steps to reproduce.

Install ESP-IDF v5.0 using esp-idf-tools-setup-online-2.16.exe on Windows 10 without including support for every single ESP32 target, regardless if you're ever going to develop for them.
Look at the ERRORs appear when ever a CMD and PS shell is opened, including at the end of the ESP-IDF installation.

Build or installation Logs.
No response
More Information.
It shouldn't be flagged as an error if support for one or more target(s) was intentionally selectively omitted during installation.

Hi @JBHunterI. Could you please try to delete idf-env.json from %IDF_TOOLS_PATH and re-run the installer? Does this fix your issue?
Otherwise please inspect your environment variables and clean everything (ESP-IDF-related) from there before running the installer.

No, deleting %IDF_TOOLS_PATH\idf-env.json & re-running the universal online installer on a new PC that has never had the ESP-IDF installed on it (so there were no ESP* or IDF* variables in the environment prior to this installation attempt), pointing it to the existing %IDF_PATH% to avoid downloading & extracting that again, & selecting the exact same installation options again (only interested in the ESP32-C3 tools) does not get rid of the error messages after the "Adding ESP-IDF tools to PATH..." message in the new CMD & PS shells. Both still complain about xtensa-esp*, esp32ulp-elf, & dfu-util files not being installed.  I don't understand how you can't reproduce this with the v5.0 latest stable release.

What version of ESP-IDF are you using? The release/v5.0 of this Github repo? Or the tagged v5.0 version? Please make sure that it is not one of the betas or earlier. Because I think this issue has already been fixed earlier.
BTW, a new version of the installer (2.17) has just been released at https://dl.espressif.com/dl/esp-idf/. This is the first release since the final version of v5.0 of ESP-IDF has been released.

As the 'v5.0' output of my 'git describe --tag' command in the header of this issue shows, I was using the pre-release of v5.0 and updated to v5.0-rc1 before, but I waited for the official release of v5.0 to submit this issue. I don't know if it's the Github repo or the tagged version, but it's whatever the Universal Installer v2.16 installs when I select the very first/top-most version offered:

Are you suggesting this problem will disappear if I install ESP-IDF v5.0 (release version) using the Universal Installer v2.17?  BTW, what' the heck is the difference between "v5.0 (release version)" and "release/v5.0 (release branch)" five lines below, & which one should I select?  Why does this have to be so confusing? Can you not reproduce this issue with v2.16. or v2.17 already?

Thanks, @igrr. I've read that already, but my point was that the Universal Online Installer doesn't state it but, now from my experience, it would download & extract the "v5.0 (release version)" at the top of its list from an archived/zipped copy of the official Release v5.0, & I'm guessing "release/v5.0 (release branch)" near the bottom of its list would get a copy of the same thing but from a tag from the GitHub repo. All that is just not clear, at least to me, from the installer's list. Let me know if that's incorrect.

I think the end result when installing these two versions would be:

If you choose "v5.0 (release version)", you will get a copy of IDF at v5.0 tag. Indeed, it practically gets downloaded as an archived copy of the repo, with submodules included.
If you choose "v5.0 (release branch)", you will get a copy of latest IDF release/5.0 branch. Since v5.0 release, there have been more commits to release/v5.0 branch, so that branch has more features and fixes than the v5.0 release tag. Eventually, there will be a v5.0.1 bugfix release on that branch. Then the installer will start offering "v5.0.1 (release version)" instead of "v5.0 (release version)" as the first choice. The point is, the release branch most of the time is ahead of the release tag. So it is more recent and has more features (+) but unlike the tags, it hasn't been subject to release testing by our QA (-).

Practically, when you choose the "release branch" option, the installer will download the archive with the latest tag on that branch (v5.0 in this case) and then update it to the tip of the release branch. This is purely a download speed optimization, though; the end result is the same as if you would do git clone --branch release/v5.0 --recursive https://github.com/espressif/esp-idf.git.

Thanks for clarifying that, @igrr. Maybe if the installer used the word "stable" in its list, as does the list at the top-left of the online ESP-IDF Programming Guide, it might make it clearer that the we're talking about a fixed point in the product's release (like an archive or a specific 'stable' tag), whereas a branch implies the possibility of getting something that might be changing with time.

FYI, getting back to my original question for this thread, I tried using the Universal Installer v2.17, after uninstalling the "v5.0 (release version)" installed with the Universal Installer v2.16 & manually getting rid of any remaining files/folders/environment variables left behind. Here is what I entered as the installtion configuration:

I also compared the %IDF_TOOLS_PATH%\releases\esp-idf-v5.0.zip from both installer versions: they were identical.
Not sure if dfu-util is needed by the ESP32-C3, but do we agree that an installation configuration that excludes support for some ESP32 targets should not cause errors?

but do we agree that an installation configuration that excludes support for some ESP32 targets should not cause errors?

Nobody denies that. I was collecting more information so we could properly understand the issue and reproduce it.

Hi @JBHunterI
I am sorry for having this issue.
The fix for now is as follows:

Open the ESP-IDF 5.0 CMD
You'll see the same error as before. Copy the command the ERROR tells you. Running this command would install all the tools for all esp targets. Since you want to use only esp32c3, place esp32c3 at the end of the command.

Your fixing command according to the screenshots you provided should look like this: C:\App\ESP\IDF\esp-idf-v5.0\install.bat esp32c3
It seems that we have a problem with the Universal Installer, when parsing the user's choice of targets to the configuration file.
We will fix it in the next release.
Please try this, and let me know if it works for you.

Problem is caused by idf_tools.py which is not flipping the slashes in the --idf-path parameter passed from the installer. The value is then stored in the JSON in incorrect format.
After discussion with @mfialaf we switched slashes to backslashes in the installer: https://github.com/espressif/idf-installer/releases/tag/online-2.18-beta-01

Thank you @georgik and @mfialaf for your help!

Meanwhile we discovered some other issue. Released new build https://github.com/espressif/idf-installer/releases/tag/online-2.18-beta-03
@mfialaf confirmed that it works well.

I had to go back to ESP-IDF v4.4.3 because the Featured FreeRTOS IoT Integration on ESP32-C3 RISC-V MCU won't seem to support ESP-IDF v5.0 for a while (see this response in its issue #16) but I installed v4.4.3 using your Online Installer v2.18-beta3 mentioned above and it seemed to not complain when it only installed the ESP32-C3 RISC-V tools I requested. My system path only contains entries for that, 'where git python' reports the expected 3 paths, 'python --version', 'git --version', and 'git describe --tag' correctly responds with "v4.4.3".

If I read the last comments correctly then this has been resolved the the new version of the installer.


add wJf to MeshData

Closed in favor of https://github.com/jlchan/StartUpDG.jl/issues/58

Removed from #58, but this shouldn't be necessary anymore for cut cell meshes anyways.


DayLightSavingTimeOffset

How can I get dstOffset?
there have an example...
https://developers.google.com/maps/documentation/timezone/intro

This question is not related to this project.
Suggest you to read the documentation of jodatime.


Remove dynamic evaluation to access globalThis in "lite" mode

[...] in<EMAIL_ADDRESS>I recently introduced a "lite" mode that drops the internal Map/Set polyfill, which is accessed via an alternative import of import "reflect-metadata/lite";. I could see modifying "lite" mode to elide dynamic evaluation with the expectation that any necessary polyfills are already installed.
Originally posted by @rbuckton in https://github.com/rbuckton/reflect-metadata/issues/141#issuecomment-1854436740

Fixed by #149


pull request for e2e test at 01/14/2017 17:26:12

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


'open' doesn't work for some apps

this is my config:
# Applications

cmd + shift - 0 : open /Applications/GitKraken.app
cmd + shift - 1 : open /Users/nshv/Repos/neovide/target/release/bundle/osx/Neovide.app
cmd + shift - 2 : open /Applications/Safari.app
cmd + shift - 3 : open /Applications/Discord.app
cmd + shift - 4 : open /Applications/Telegram.app
cmd + shift - 5 : open /Applications/Twitter.app

cmd + shift - e : open /System/Library/CoreServices/Finder.app
cmd + shift - j : open /Users/nshv/Library/Application Support/JetBrains/Toolbox/apps/AppCode/ch-0/222.4167.29/AppCode.app
cmd + shift - h : open /Users/nshv/Library/Application Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/213.7172.25.2113.9014738/Android Studio.app
cmd + shift - b : open /Applications/Books.app
cmd + shift - x : open /Applications/Xcode.app
cmd + shift - d : open /Applications/Dash.app

# ~/~/~/~/~/~/~/~/~/~/~/~/~/~/~/~/~/~/~/~/~/~/~/~/~/~/~/~/~/~/~/~/~/~/~/~/~/~/~/~/~/~/~/~/~/~/~/~/~/~/

# Commands

cmd + shift - f9 : command /Applications/Vpnify.app/Contents/MacOS/CLI/vpnify connect -c hkg
cmd + shift - f6 : command /Applications/Vpnify.app/Contents/MacOS/CLI/vpnify disconnect

ctrl + shift + alt - r : command /opt/homebrew/bin/brew services restart yabai
cmd + ctrl - l : command /opt/homebrew/bin/yabai -m window --focus east
cmd + ctrl - h : command /opt/homebrew/bin/yabai -m window --focus west
cmd + ctrl - j : command /opt/homebrew/bin/yabai -m window --focus south
cmd + ctrl - k : command /opt/homebrew/bin/yabai -m window --focus north
ctrl + shift - l : command /opt/homebrew/bin/yabai -m window --swap east
ctrl + shift - h : command /opt/homebrew/bin/yabai -m window --swap west
ctrl + shift - j : command /opt/homebrew/bin/yabai -m window --swap south
ctrl + shift - k : command /opt/homebrew/bin/yabai -m window --swap north
cmd + shift - return : command /opt/homebrew/bin/yabai -m window --toggle zoom-fullscreen

# ~/~/~/~/~/~/~/~/~/~/~/~/~/~/~/~/~/~/~/~/~/~/~/~/~/~/~/~/~/~/~/~/~/~/~/~/~/~/~/~/~/~/~/~/~/~/~/~/~/~/

'open' doesn't work for several apps such as Books, Android Studio, AppCode...
I tried to use different shortcut - same. Is there any solution?

Let's try with open -a Books

Let's try with open -a Books

tried before - doesn't work

tried before - doesn't work

Did you use the full path open -a /Applications/Books or just open -a Books?
Have you tried to run a which Books to confirm that the Books application is actually located where you think it is?


Fix cython_inline and locals on Python 3.13

The difference is that locals is now a proxy object rather than a dict, and it has stricter rules for equality than dict does.
Fix for some of #6251

CPython issue https://github.com/python/cpython/issues/120906 (but I think we should just make the change and not worry too much about demanding fixes from CPython)

It makes no sense for Python to reject subtypes. That's an obvious bug with respect to normal language behaviour.
I'm fine with this work around. The comment that you added has a few typos, though.

3.0.x commit d68ccaf9c


Update to Bevy 0.10

In Bevy we usually chain App commands like app.init_resource().add_system(...), but I followed your style.

Looks like CI have a configuration issue. Fixed it.

Thanks, looks great.
Also thanks for all the versions update and the CI fix.
I will try to publish the new version tomorrow


-force flag does not work

Hi @themalkolm,
It would be great if you could implement the -force flag.
Thanks
Nabil

@kadimi could you elaborate a little? What do you expect to happen when -force flag is provided?
Lets define what it should do together and I will implement it if it helps you. Deal?
-force - Forces a builder to run when artifacts from a previous build prevent a build from running. The exact behavior of a forced build is left to the builder. In general, a builder supporting the forced build will remove the artifacts from the previous build. This will allow the user to repeat a build without having to manually clean these artifacts beforehand.

Thanks for your response, my build look like this:
rm ./output- -fr && packer build template.json

I was hoping I could like suggested by the error message :
packer build -force template.json

Here is a failed build output:

vagrant output will be in this color.

    vagrant: (vagrant) Builder source_path ...
    vagrant: (vagrant) Builder source_path: /home/n/.vagrant.d/boxes/debian-VAGRANTSLASH-jessie64/8.7.0/virtualbox/box.ovf
    vagrant: (vagrant) Builder type ...
    vagrant: (vagrant) Builder type: virtualbox-ovf
    vagrant: (vagrant) Builder ...
    vagrant: (vagrant) Builder: OK
    vagrant: (vagrant) Builder prepare ...
    vagrant: (vagrant) Builder prepare: OK
    vagrant: (vagrant) Builder run ...
==> vagrant: Output directory exists: output-
==> vagrant:
==> vagrant: Use the force flag to delete it prior to building.
Build 'vagrant' errored: Output directory exists: output-

Use the force flag to delete it prior to building.

==> Some builds didn't complete successfully and had errors:
--> vagrant: Output directory exists: output-

Use the force flag to delete it prior to building.

==> Builds finished but no artifacts were created.

@kadimi sure, will do. I will make a patch release for packer-0.12.2. Does it work for you?

@themalkolm That would be awesome!

@kadimi https://github.com/themalkolm/packer-builder-vagrant/releases/tag/v0.1.3
Now -force (and -debug) are now passed down to the wrapped builder. So your use case now should work.

So your use case now should work.

It surely does, thanks @themalkolm.


Feedback from msk#7744

From msk :
Hello

sorry I mesed up this was an accident

how do I close it?

someone has to run sudo feedback close 16 I think