feat(pdk): increase the precision of JSON number encoding from 14 to 16 decimals

Summary
Currently, when Kong is serializing JSON, it can only encode numbers with a precision of up to 14 decimals. For numbers with a precision exceeding 14 decimals, kong will output them using scientific notation, resulting in a loss of decimal precision. This also causes an issue where KIC cannot use integers exceeding 2^46 bits to configure the priority attribute of route.
Now, we are increasing this limitation to 16 decimals. With this change, we can safely store integers up to 2^53 bits without losing precision.
Checklist

[X] The Pull Request has tests
[X] A changelog file has been created under changelog/unreleased/kong or skip-changelog label added on PR if changelog is unnecessary. README.md
[ ] There is a user-facing docs PR against https://github.com/Kong/docs.konghq.com - PUT DOCS PR HERE

Issue reference

Fix FTI-5515

as a remember: if some else that use require("cjson.safe").new() or require("cjson").new(), the encode_number_precision(16) would be invalid
releated: #11389

How are we going to solve or mitigate the concern from @tzssangglass ? Is it a bug in cjson package?

Is it a bug in cjson package?

I don't think so. cjson.new is actually an instantiated copy of a separate Lua CJSON module. The new module has a separate persistent encoding buffer and default settings.
I think we can wrap the cjson.new method in the global patch and set cjson.encode_number_precision(16) by default for instances initialized through it, and that should solve the problem.

I think we can wrap the cjson.new method in the global patch and set cjson.encode_number_precision(16) by default for instances initialized through it, and that should solve the problem.

work for me, but it is a break change.

Didn't read up on the specific case here, but we ran into something similar before, see https://github.com/Tieske/lua-resty-ljsonschema/pull/21 by @vm-001
Just leaving it here as info for whomever wants to double check.

Just leaving it here as info for whomever wants to double check.

The PR https://github.com/Tieske/lua-resty-ljsonschema/pull/21 fixed loss precision by string.format, and does not related to cjson lib.


fix: init from dash using explicit usage model for standard accounts

What this PR solves / how to test:
Fixes a bug where users with the standard usage model would have an explicit usage model defined in their wrangler.toml when they initialized a Worker from dash. Testing can be done by building and then running wrangler init --from-dash [worker] -y --no-delegate-c3 on an account that has opted into the new standard pricing and ensuring there is no usage model defined in the config.
Author has addressed the following:

Tests

[x] Included
[ ] Not necessary because:

Changeset (Changeset guidelines)

[x] Included
[ ] Not necessary because:

Associated docs

[ ] Issue(s)/PR(s):
[x] Not necessary because: this is a bug fix to match existing defined behavior

Note for PR author:
We want to celebrate and highlight awesome PR review! If you think this PR received a particularly high-caliber review, please assign it the label highlight pr review so future reviewers can take inspiration and learn from it.

Hey! 👋 Thanks for opening this PR! I see you noticed there's already an in-flight PR that fixes this behaviour in a slightly different way. Could we close this PR and discuss which solution to use on #4470 instead, or does this need to go in sooner?

We were going to initially do that, but with Samuel being OOO we wanted to ensure that this'd be in the next release. I'm open to using that PR if we can be sure it'll go out in the next release, but product decided we should play it safe with a smaller bugfix PR.


test(robot): add test backup listing for large volumes and many backups

Which issue(s) this PR fixes:
longhorn/longhorn#8355
What this PR does / why we need it:
Implement manual test case Test backup listing S3/NFS in robot.
Special notes for your reviewer:
Additional documentation or context

https://longhorn.github.io/longhorn-tests/manual/pre-release/stress/backup-listing/
https://github.com/longhorn/longhorn/issues/8355
PR test log :

log_20240617_case2.tar.gz

log_20240823.tar.gz

log_20240926.tar.gz

Here's how to execute test case: ./run.sh -t "Backup listing of volume bigger than 200 Gi" -v RETRY_COUNT:5400

@roger-ryao Please let the team know if this PR is ready for review. Thank you!

@roger-ryao Please let the team know if this PR is ready for review. Thank you!

Hi @yangchiu
The PR is ready for review.

The PR is ready for review.

Hi @longhorn/qa
The PR is ready for review.

This pull request is now in conflict. Could you fix it @roger-ryao? 🙏

Done


Add subtitle to _layouts/post.html

Currently post layout markdown only includes title - also include subtitle in _layouts/post.html. And modify accordingly the first blog post (book review of Blood, Sweat & Pixels book).

Done


Auto-completion/type hints for pymongo/motor

Environment data

Language Server version: Pylance=v2021.4.1 (pyright 6410a8a2), Python=v2021.5.756443519-dev
OS and version: macOS Big Sur / Darwin arm64 20.3.0
Python version: 3.8

Expected behaviour
Auto-completion and type inference should work.
Actual behaviour
No auto-completion and type hints are shown.
Code Snippet / Additional information
Install motor with pip install motor.
from motor.motor_asyncio import AsyncIOMotorClient

CONN_URI = "mongodb://root:123456@localhost:27017"

client = AsyncIOMotorClient(CONN_URI, tz_aware=True)
db = client.get_database("dbber")
sc_sessions = db.get_collection("sc_sessions")

Can you please specify which completions are missing? Please include a screenshot. Thank you

This library appears to be doing something very funky (this "metaprogramming" thing that dynamically builds classes):
from . import core, motor_gridfs
from .frameworks import asyncio as asyncio_framework
from .metaprogramming import create_class_with_framework

__all__ = ['AsyncIOMotorClient','AsyncIOMotorClientEncryption']

def create_asyncio_class(cls):
    return create_class_with_framework(cls, asyncio_framework,
                                       'motor.motor_asyncio')

AsyncIOMotorClient = create_asyncio_class(core.AgnosticClient)

It's likely that they'd benefit from some type annotations or stubs to describe this behavior, as it's a bit too dynamic for us to handle.

@jakebailey That looks very "funky" indeed. I guess this issue belongs upstream then. Thank You!


Update dev dependency yarn

@mikeharder if you rebase on develop, tests should go green for you now

All checks passed in DevOps, but for some reason the GitHub status was not updated


menu entry swapper: add Tempoross leave swap

After subduing Tempoross, to save some time, most players choose to "Leave" instantly by interacting with one of four NPCs in the cove. I believe swapping the default "Talk-to" with "Leave" would be a minor QOL improvement for the boss.
Some self-asked QA questions:
Is there a risk that players will leave by accident before the fight is over?
Nope, because the option to leave early is called "Forfeit", which is not being swapped. So throughout the fight, the left-click option for each NPC is going to be "Talk-to".
Should this swap be enabled by default?
In my opinion, yes. I believe in 99% or even more cases players just want to leave the cove once the fight is finished. These NPCs are available outside of the cove should anyone want to learn how to play. There's also plenty of time to talk to the NPCs during the fight.
Should the swap be limited just to the 4 NPCs, or should it be a general thing?
I think limiting it makes it safer to enable by default. Although I'm not aware of any other NPCs with a "Talk-to" and "Leave" interactions combo. As of now, the name of the config item reflects the fact that it is limited to Tempoross ("Tempoross Leave"). If it's made more generic, it should probably be called just "Leave".
How was it tested?
I've manually tested it during a few Tempoross encounters, with both captains and first mates NPCs. ~Also added a unit test, not sure if it adds value though, as it is quite basic.~ The unit test is gone, after discussing on Discord.

There is an exiting quick-leave option, shouldn't this just use that?

I can think of a few reasons why perhaps not:

Tempoross leave is not an Object swap, nor it is a Quick-X action.
Similar case: NPC swap Start-minigame for Pyramid Plunder is not grouped with Object swap Quick Pass/Open/Start/Travel.
Quick-Leave setting is OFF by default (see my prev comment), and perhaps stays OFF for a reason (it can cause you to exit Pyramid Plunder early on a missclick).

I can think of a few reasons why perhaps not:

Tempoross leave is not an Object swap, nor it is a Quick-X action.
Similar case: NPC swap Start-minigame for Pyramid Plunder is not grouped with Object swap Quick Pass/Open/Start/Travel.
Quick-Leave setting is OFF by default (see my prev comment), and perhaps stays OFF for a reason (it can cause you to exit Pyramid Plunder early on a missclick).

Agreed it's better to make this it's own swap entry. Hope we can get this in soon :)


Document elemental operator crds chart

Closes #140

@kkaempf I think it's best if we just update the quickstart reference for the development version right now. That should also be fixed for Stable since it's currently not working.
Regarding your suggestion, I was already thinking to make some more explicit document regarding different channels, so that it's more explicit to the end user. I just miss some good classification/definition for the channels, but I can already open a new issue so that we can discuss that there.

@fgiudici since you improved this same documentation in your other pull request, I'm going to close this one and also close the associated issue. Is that good to you?
I think your changes already mention the CRDs separate chart.

@fgiudici since you improved this same documentation in your other pull request, I'm going to close this one and also close the associated issue. Is that good to you?
I think your changes already mention the CRDs separate chart.

Hey @anmazzotti !
right, that would work for me, thank you 😃


Add MQTT Discovery Mode for Home Assistant configuration?

Last weekend I installed the AquaMQTT but for the life of my I could not get it to create entities in Home Assistant. I presumed it would create them automatically as other MQTT solution did that by default.
Finally I found a French post somewhere that linked to the yaml file needed to configure Home Assistant here: https://github.com/tspopp/AquaMQTT/blob/main/aquamqtt.yaml
It would be nice if the MQTT Discovery Mode was added so that this device really becomes plug and play. I can understand this might be a bit of work, so maybe a link to the yaml file in a readme somewhere with the note that it's needed to configure Home Assistant would be nice as well.

Fully agree, this part is lacking information. Actually my main goal was getting the software on the Arduino done right and I did not expect an increasing user base. With the ongoing cozytouch backend issues and the compatibility to the Atlantic Explorer there are more people trying AquaMQTT than I've ever expected. In the past I pretty much drew a line as soon as the data was given to the MQTT broker. I did spent very little time in creating a configuration files for backends such as homeassistant or iobroker. Actually it was @scoudibou who helped and provided the mentioned configuration file :+1:
I previously did not know about the Discovery Mode from HomeAssistant, but it sounds fun to implement. I might gonna look into this. Thanks for the suggestion :)


Dev/golink-crud

Demo: https://youtu.be/bNkMKYXjTYw
GoLink CRUD features implemented. Every test case is shown in the demo.
Features:

GoLinks:

Title
URL
Description
Pinned
Officer (Currently no way to access them until we have more authentication, although the function to do so is created so it can be done as soon as possible).

Creating GoLinks

All titles are input validated to be lowercase with dashes (ex: the-sse-website instead of TheSseWebsite)

Editing all aspects of a GoLink
Deleting GoLinks

Things to note:

If all GoLinks are deleted, there is a perpetual loading screen. That is because the loading screen that appears is based on if the GoLinkData received has a length of zero. The create button still appears though. I figured this would be an alright thing to do since we will never have zero GoLinks once the site is released.

Looks great!! Thanks Christian for creating issues for the perpetual loading and for creating go links with no URLs


Brew not installing on M1

I've tried installing Brew following official steps and it works. Our bootstrap scripts does some weird stuff and it fails.
--> Installing from sentry Brewfile
Error: Cannot install in Homebrew on ARM processor in Intel default prefix (/usr/local)!
Please create a new installation in /opt/homebrew using one of the
"Alternative Installs" from:
  https://docs.brew.sh/Installation
You can migrate your previously installed formula list with:
  brew bundle dump
Installing pyenv has failed!
Error: Cannot install in Homebrew on ARM processor in Intel default prefix (/usr/local)!
Please create a new installation in /opt/homebrew using one of the
"Alternative Installs" from:
  https://docs.brew.sh/Installation
You can migrate your previously installed formula list with:
  brew bundle dump
Installing openssl has failed!
Error: Cannot install in Homebrew on ARM processor in Intel default prefix (/usr/local)!
Please create a new installation in /opt/homebrew using one of the
"Alternative Installs" from:
  https://docs.brew.sh/Installation
You can migrate your previously installed formula list with:
  brew bundle dump
Installing readline has failed!
Error: Cannot install in Homebrew on ARM processor in Intel default prefix (/usr/local)!
Please create a new installation in /opt/homebrew using one of the
"Alternative Installs" from:
  https://docs.brew.sh/Installation
You can migrate your previously installed formula list with:
  brew bundle dump
Installing watchman has failed!
Error: Cannot install in Homebrew on ARM processor in Intel default prefix (/usr/local)!
Please create a new installation in /opt/homebrew using one of the
"Alternative Installs" from:
  https://docs.brew.sh/Installation
You can migrate your previously installed formula list with:
  brew bundle dump
Installing pkgconfig has failed!
Error: Cannot install in Homebrew on ARM processor in Intel default prefix (/usr/local)!
Please create a new installation in /opt/homebrew using one of the
"Alternative Installs" from:
  https://docs.brew.sh/Installation
You can migrate your previously installed formula list with:
  brew bundle dump
Installing libxslt has failed!
Error: Cannot install in Homebrew on ARM processor in Intel default prefix (/usr/local)!
Please create a new installation in /opt/homebrew using one of the
"Alternative Installs" from:
  https://docs.brew.sh/Installation
You can migrate your previously installed formula list with:
  brew bundle dump
Installing libxmlsec1 has failed!
Error: Cannot install in Homebrew on ARM processor in Intel default prefix (/usr/local)!
Please create a new installation in /opt/homebrew using one of the
"Alternative Installs" from:
  https://docs.brew.sh/Installation
You can migrate your previously installed formula list with:
  brew bundle dump
Installing geoip has failed!
Error: Cannot install in Homebrew on ARM processor in Intel default prefix (/usr/local)!
Please create a new installation in /opt/homebrew using one of the
"Alternative Installs" from:
  https://docs.brew.sh/Installation
You can migrate your previously installed formula list with:
  brew bundle dump
Installing librdkafka has failed!
Using homebrew/cask
Installing chromedriver
Installing docker
Homebrew Bundle failed! 9 Brewfile dependencies failed to install.

==> Next steps:
- Add Homebrew to your PATH in /Users/armenzg/.zprofile:
    echo 'eval "$(/opt/homebrew/bin/brew shellenv)"' >> /Users/armenzg/.zprofile
    eval "$(/opt/homebrew/bin/brew shellenv)"

This got fixed with #32


Missed wake call using .shared()

This is the reproduction:
use std::{
    io::Write,
    pin::Pin,
    task::{Context, Poll},
};

use futures::prelude::*;

pub async fn yield_now() {
    /// Yield implementation
    struct YieldNow {
        yielded: bool,
    }

    impl Future for YieldNow {
        type Output = ();

        fn poll(mut self: Pin<&mut Self>, cx: &mut Context<'_>) -> Poll<()> {
            if self.yielded {
                return Poll::Ready(());
            }

            self.yielded = true;
            cx.waker().wake_by_ref();
            Poll::Pending
        }
    }

    YieldNow { yielded: false }.await
}

#[tokio::main]
async fn main() {
    for _ in 0..200 {
        print!(".");
        std::io::stdout().flush().unwrap();
        for _ in 0..10000 {
            test().await;
        }
    }
    println!();
}

async fn test() {
    let f1 = yield_now().shared();
    let f2 = f1.clone();
    let x1 = tokio::spawn(async move {
        f1.now_or_never();
    });
    let x2 = tokio::spawn(async move {
        f2.await;
    });
    x1.await.ok();
    x2.await.ok();
}

It manages to hang up eventually on most runs, both in debug and release mode.
The example is crafted to expose a weakness spotted during code review of .shared(). The problematic part is here:
https://github.com/rust-lang/futures-rs/blob/aa1f5c74251bb494d1bc7a016381c76f909316bc/futures-util/src/future/future/shared.rs#L288-L293
This code is simply insufficient to ensure the current context’s waker is actually ever woken up. The current context’s waker was stored into the inner slab previously in this line
https://github.com/rust-lang/futures-rs/blob/aa1f5c74251bb494d1bc7a016381c76f909316bc/futures-util/src/future/future/shared.rs#L277
However, a concurrently running poll could have already been within the call to the inner poll before the new waker got registered, i.e. here:
https://github.com/rust-lang/futures-rs/blob/aa1f5c74251bb494d1bc7a016381c76f909316bc/futures-util/src/future/future/shared.rs#L328
As soon as the call to poll on the inner future starts, it could already be calling the waker, thus doing the waking before the new waker that we care about was registered and put into the slab in the first place, because the waking could happen before the state is put back into IDLE.
The waking could happen before the state is put back into IDLE in particular because it is okay for a .poll call to do the waking immediately, before it returns, i.e the way the yield_now function in the example always does (copied from the old yield_now implementation of tokio).

Some initial thought for a potential fix: Add some additional (3 states) state enum next to the Slab in the to the Mutex inside the wakers field.
Right before calling the inner future’s poll method, set this state from the “default” state to “inside inner poll”. The Notifier::wake_by_ref implementation would (operate as it already does and additionally) inspect this state, and if it is in “inside inner poll” state change it to “woken while inside inner poll”.
Then after the call to the inner future’s poll method, if the poll result is Pending then

first lock the Mutex
next change the notifier state from POLLING to IDLE
then look into the Mutex contents and change the state back to “default”

if the state was in “woken while inside inner poll”, awake all (new) wakers immediately (there will only be those wakers left that arrived during the poll operation, anyways

finally, unlock the Mutex

This way, a thread that observes the POLLING state and returns can know that their record_waker call accessed the Mutex before the currently polling state re-locks the Mutex to handle wakes-during-polling, so their registered waker is never ignored.

In more detail:
The previously problematic setting was when a record_waker call happens while the inner .poll call is already in progress and after this call already executed its guaranteed eventual wakeup; and when after such a record_waker call, POLLING state belonging to the same poll to the inner future is still observed. Otherwise there is no problem because either there would already have been a subsequent poll that does consider the registered waker, or we would be doing such a poll ourselves (or the future could be finished).
This setting means that first the polling thread records “inside inner poll” state, and then the poll happening during execution of the inner poll will put it into “woken while inside inner poll” state. Then the problematic thread calls record_waker, adding a new waker to the wakers with the “woken while inside inner poll” state. Afterwards this thread observes POLLING, so that this record_waker is known to happen before the change from POLLING to IDLE in the idle thread. This POLLING to IDLE change happens while the mutex is being held that was also accessed by record_waker, so that mutex acquiry also happened after the call to record_waker. Hence the polling thread will observe both the “woken while inside inner poll” and a list of wakers including the newly registered one, which can then be awoken.

I guess there’s some leeway implementation on the point whether a Notifier::wake_by_ref while in (one of) the “inside inner poll” state(s) should perhaps only update the state (to “woken while inside inner poll”) and not do any waking yet, as any awoken task would run into a currently-POLLING state anyways, and would then need to be re-awoken soon after.
Perhaps more importantly, I have not thought about panic case(s) yet. Those would presumably also need to be handled somewhat properly.

Trying to start an implementation of a fix, I came across a design question where I’m not sure what the best solution should be.
If the poll method panics, then the caller of the Shared instances currently being polled will get that panic propagated properly, but all other clones of the Shared will potentially never learn about the panic, since – as far as I’m aware – a panicking poll does give no promises as to whether or not the waker in the context you gave it will actually ever be awoken.
The possible solution is to make sure to wake all other wakers in case of panic, so they have a chance to learn of the panicking poll themselves. (From their point of view, they get awoken, then call poll and then will get a panic themself, which is the desired behavior, AFAICT, unlike the current approach of potentially just leaving them unawoken indefinitely without ever experiencing any panicking poll call themself.)
Now the problem with this approach: Calling a bunch of wakers in a panic case could be done through code in a Drop implementation, with the effect that if any of the wakers panic on the .wake call, that’s a double-panic / abort. So the question thus is whether we can simply assume that reasonable wakers never panic, so the possibility of abort would be considered “fine”. Or alternatively, I guess, it could be a reasonable approach to use catch_unwind instead of destructors, and that way a panicking waker would not result in immediate abort; but if panicking wakers are a thing to consider, there’s follow-up questions such as: If one waker panics, do we still need to awake all subsequent wakers in the Slab?
So really TL;DR, how bad are missed wake-ups in cases where panics happened somewhere (I suppose bad enough that they do warrant a fix), and how seriously should the case of panicking wakers be considered (I’m unsure on this, and interested in hearing about guidelines or prior art on the question of panicking wakers).

Wanted to pop in and say thank you for working on this issue. I went down a rabbithole today debugging an application that uses Shared dropping futures on the ground. This appears to be the root cause for my bug.

Thanks for working on this!

how bad are missed wake-ups in cases where panics happened somewhere?

It is not ideal, but fine. (IMO, a potential abort is bad anyway.)

guidelines or prior art on the question of panicking wakers

I do not know of any prior art or guidelines for this particular case, but https://github.com/tokio-rs/tokio/pull/3689 is somewhat relevant.

any update or workaround for this issue?


feat: add support for Nextcloud 28

Nextcloud 28 has made a fair few changes and some of these have resulted in new false positives. Most of these false positives are at PL-2 or higher or they they affect features that will only be used on occasion.

@EsadCetiner What is the purpose of splitting exclusions into rules 9508311, 9508312, 9508313 and 9508314? Seems like all of them are matching the same URLs.

@azurit Those rule exclude rules for individual token used in Nextcloud's markdown text editor. All of these rule exclusions could be done in one rule, but then you won't be able to easily implement character whitelisting (which is currently the case).

@azurit everything good to merge?

@theseion I'm trying to use the REQUEST_BODY_LENGTH variable to check if there's no response body then disable rule 200002 since that's causing the XML parser to fail, but the linter doesn't think that's a valid variable. It's supported on both ModSec v2/v3 and Coraza so I don't know why it's marked as invalid.

Wow, wouldn't have expected that. But your right, there's a mistake in the parser model.
I've opened an issue: https://github.com/coreruleset/secrules_parsing/issues/70.
@airween, any idea how to fix that one? I wasn't able to.

There is still a workflow issue, but I'm afraid that's an ftw problem. @theseion could you take a look at it?

Yes, I know. I was waiting for your last PR to the parser to be merged and a new version to be published.

All the checks are good now. @airween, you need to approve the changes, then we can merge.


Control access to Zstid CSRs

Add a bit in xstateen0 CSRs to control access to new CSRs used by the Zstid extension.
Fixes https://github.com/riscv/riscv-cheri/issues/333

if you allocate bit 28 then there'll be a lot more time before it gets overwritten, but they do seem to be allocating from the top so I guess it's ok

I don't think it's clear enough from this change explicitly which bit controls which CSR in which privilege mode. It seems to suggest that if mstateen0.TID=0 this means that only M-level access can access stid, and if sstateen0.TID=0 then U-level access cannot access utid. And if the TID bits are 1, then the read/write-permissions described in the Zstid CSR tables apply. Is that correct? This could be clearer.
Also does this change now mean Zstid is dependent on the Sxstateen extensions being enabled too? Or is possible to implement Zstid without Sxstateen? This could also be clearer.


Serving/opening more than one html files at the same time

I practically face the following problem: Very often, I need to change to a different html page among a number of them.
The first solution is to start separately the server for each page, but it is not so elegant and sophisticated at all!
The second and obvious solution is, every time, to (un)comment the relevant pages in contentServer.index and restart the server. It's very boring!
The third solution is to put all pages as links inside a main page with the rest of the files and call them from there. It's Ok but not the ideal!
The fourth solution is contentServer.index can also accept an array of pages. In this case, for each html page server creates a list of watched files. When a watched file changes, server finds the corresponding page and reload it. If the file is common to more than one html pages than all relevant pages are reloaded. Does it make sense?
In addition, server would automatically open only the first page or all of them found in the array, one by one in different tabs or windows.
What do you think?
Tia

You can do it in v0.20.1. Do it like this
reboost.js
const { start } = require('reboost');

// Base options for every content server
const baseOpt = {}

start({
  contentServer: [
    {
      ...baseOpt,
      name: 'server-one',
      root: './dir1',
      index: 'index.html'
    },
    {
      ...baseOpt,
      name: 'server-two',
      root: './dir2',
      index: 'index2.html'
    }
  ],
  // ...other configurations
})

Hope it helps you.


Fixes for CASSANDRA-10461

As discussed on CASSANDRA-10461, to better debug disk space issues such as [this] (http://cassci.datastax.com/job/cassandra-3.0_dtest/248/testReport/offline_tools_test/TestOfflineTools/sstablelevelreset_test/), I've changed sstablelevelreset_test to first look at the exit code and stderr, and only afterwards validate stdout.
I've also improved splitting of stdout into lines for SSTableUtilTest since it passes locally but fails on Jenkins due to an extra line; so my best guess is a line ending problem.

Looks good, +1.

I swapped the lines checking the error code back to the end since the error code is less helpful than the error message, merging now thanks.


Run db migrations in prod after enumerize PR gets deployed

User.each { |u| u.rename(_role: 'role')}
Patient.each { |pt| pt.rename(_line: 'line')}
Patient.each { |pt| pt.rename(_voicemail_preference: 'voicemail_preference')}

see #1106
tasks:

[x] run in dcaf
[x] run in baltimore
[x] reseed sandbox

Resolved between 11pm and 113pm 2017-07-11 after merging #1106


Cache balances when processing epoch

New balances array inside EpochProcess
Populate it in processRewardsAndPenalties
Use it in processFinalUpdates
Save 165ms with over 110000 validators (same validator number in Pyrmont and Medalla)

waiting for #1799

ready for review


Added Flycheck documentation

[x] It targets the develop branch
[x] I've searched for similar pull requests and found nothing
[x] This change is NOT in Doom's do-not-PR list: https://doomemacs.org/d/do-not-pr
[x] If I've bumped any packages, I've done so according to https://doomemacs.org/d/how2bump
[x] I've linked any relevant issues and PRs below
[x] All my commit messages are descriptive and distinct

Relevant to #1166
Adds documentation for the flycheck module. I was not sure what to put for the since option, so I left that as ???. If anyone knows what should be placed here, tell me and I'll fix it.

put #+SINCE:  v3.0.0, since that is the next version of doom that will come out

I fixed the date and since fields. Is there anything else that needs to be done before this can be merged?

Looks good to me, thanks!


✨ Add Alert When Apply Fails on Main

👀 Purpose

To increase the visibility of failed-to-apply jobs on the main branch
To minimise the time that the deploy pipeline is blocked

✅ Definition of Done

[ ] A slack alert is generated when there is a failure in applying terraform

📓 Notes

There seems to already be a slack webhook URL in the repository secrets, but we may want to create a new app for this - to ensure all the working group has access to it

🆙 Update

New Slack App created https://api.slack.com/apps/A07LVM0CHA8 - this is to enable alerts to be sent programmatically to the #aws-root-account Slack channel
Everyone from the working group has been given access to the Slack App

👀 Remaining Work

[ ] Incoming webhook approved by Slack Admin (https://api.slack.com/apps/A07LVM0CHA8/incoming-webhooks?)
[ ] Integrate alerts into aws-root-account CICD pipelines

📝 Notes

NA

🆙 Update

Alert has been added to all terraform pipelines when they fail and will send alerts to #aws-root-account. Alert has been tested and an example can be found here

👀 Remaining Work

None - just have to double check it fully works when we have an apply fail 🥳 will close this ticket now


#Add Github Greetings Bot

closes #11
The YML file has been reviewed, tested with duplicate branches, Pull Requests and workflows were successful. Triggered greetings for PR and received emails.
Please review and share the feedback.

@Harish-2003 please review and share feedback. Thank you!

Will it works every time when people raise an issue raise an pr?

The checks have Bern failed kindly check

@Harish-2003 I have modified the script. I had added additional messages for  additional actions but now I understood it needs seperate workflows. Please review.


Nick: Watch thelogopalace.com

Nick requests the watch of the watch_keyword thelogopalace\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

thelogopalace\.com has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by cigien in Charcoal HQ


fix(DataStore): Owner and Group Combined @auth

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Related:
https://github.com/aws-amplify/amplify-android/pull/860


Sort values descending, nulls last

Hi there!
At the moment the default of this script is sort the values in ascending order with nulls last. But when I try to sort the values in descending order, the nulls values appear on the top. Is there a way to move the null values to the bottom (and also keeping nulls last when sorting in ascending order)?
It should be around lines 492-518 in jstable.js, but it's hard to understand what's happening there exactly.
Thanks!

Hey @hv92,
your feature request is currently not implemented.
As you already found out the sorting is done in the mentionen rows. Therefore the JavaScript sorting (https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Array/sort#description) with a custom sorting algorithm is used.
The custom sorting algorithm removes some possible prefixes/suffixes to enable sorting of additional values.
Additionally empty values are sorted to the top.
Feel free to submit a pull request with an option to inverse this behaviour.


enhancement(remove enzyme): from button tests

The type of this PR is: Enhancement

This PR resolves []
Description
After @gkartalis' PR to add testing library, in KS we decided to change one tests file to it.
Done with @MounirDhahri, @lordkiz, @stasspodarik, @gkartalis.

PR Checklist (tick all before merging)

[x] I have included screenshots or videos to illustrate my changes, or I have not changed anything that impacts the UI.
[x] I have tested my changes on iOS and Android.
[x] I have added tests for my changes, or my changes don't require testing, or I have included a link to a separate Jira ticket covering the tests.
[x] I have added a feature flag, or my changes don't require a feature flag. (How do I add one?)
[x] I have documented any follow-up work that this PR will require, or it does not require any.
[x] I have added an app state migration, or my changes do not require one. (What are migrations?)
[x] I have added a changelog entry below or my changes do not require one.

Changelog updates
Changelog updates

Cross-platform user-facing changes

iOS user-facing changes

Android user-facing changes

Dev changes

remove enzyme from one file - cx

This PR contains the following changes:

Dev changes (remove enzyme from one file - cx)

  Generated by :no_entry_sign: dangerJS against 712df464b991127692627f385889d9f024474464

closing this. we have already done this in another pr.


Running a subcommand without the actual subcommand crashes

If you have subcommands like !a b, !a c, !a d, just trying to call !a crashes msgroom-nightly. Here's the error from toB:
<toBPath>\node_modules\msgroom-nightly\dist\index.js:281
                    if (key.toLowerCase() == command.toLowerCase()) {
                                                     ^

TypeError: Cannot read properties of undefined (reading 'toLowerCase')
    at Client.getCommand (<toBPath>\node_modules\msgroom-nightly\dist\index.js:281:54)
    at Client.processCommands (<toBPath>\node_modules\msgroom-nightly\dist\index.js:317:42)
    at Socket.<anonymous> (<toBPath>\node_modules\msgroom-nightly\dist\index.js:136:27)
    at Emitter.emit (<toBPath>\node_modules\@socket.io\component-emitter\index.js:143:20)
    at Socket.emitEvent (<toBPath>\node_modules\socket.io-client\build\cjs\socket.js:519:20)
    at Socket.onevent (<toBPath>\node_modules\socket.io-client\build\cjs\socket.js:506:18)
    at Socket.onpacket (<toBPath>\node_modules\socket.io-client\build\cjs\socket.js:474:22)
    at Emitter.emit (<toBPath>\node_modules\@socket.io\component-emitter\index.js:143:20)
    at <toBPath>\node_modules\socket.io-client\build\cjs\manager.js:237:18
    at process.processTicksAndRejections (node:internal/process/task_queues:81:21)

Node.js v18.16.0

Does a.undefined exist?

What do you mean? I have not specified an undefined property on the object.

When you execute !a, it will actually call !a undefined

Every commandMap is expected to have this property (except client.commands).
I need to add some checks for this maybe.

I tested your code and you need to add the undefined property to make it stop crashing, like this:

Okay, thank you, I wasn't aware of this - in earlier versions, it would just not do anything.

The undefined commands are a giant mess, my apologies for that.
#43 will fix this, and I'll add a note somewhere to check if the handler property actually exists, incase someone would forget somehow. I'll use this issue as that note.

This confusion will not happen anymore with the new metadata for (sub)commands system, see #43.


Not getting active parameter highlight

Hello, thanks for the plugin.
I just tried it and I can't seem to get active parameter highlight.
LspSignatureActiveParameter is set in my colorscheme but it doesn't seem to be applied.
Also, does the signature get automatically updated to highlight the next parameter, when a parameter is typed? Or do we have to use the keymaps for that? (Asking since I couldn't discern that since there is no highlighting)
Any ideas?
Neovim
NVIM v0.10.0-dev-2737+g00b7428b3
Build type: Release
LuaJIT 2.1.1710088188

i'm also not getting the parameter highlight
NVIM v0.9.5
Build type: Release
LuaJIT 2.1.1710088188

this workaround helped me. you can use your highlight group of choice of course. add this anytime after the colorscheme is loaded -
      local labelhl = vim.api.nvim_get_hl_by_name('Function', true)
      vim.api.nvim_set_hl(0,
        "LspSignatureActiveParameter",
        { fg = labelhl.foreground, italic = true, bold = true })

@tsakirist - A PR was recently raised and merged that makes it easier to set the highlight parameter for LspSignatureActiveParameter when using this plugin.
That said, I'm not sure why it wasn't working for you before. You should be able to just set the highlight as @fahadbr mentioned, without any issue. I don't know how your colourscheme does it, but it could be setting it incorrectly, causing the highlighting to be "unset".

I am running into the same issue

Confirmed the highlight is set:

I also got this issue.
The LspSignatureActiveParameter highlight group is set, but apparently not being used.
instead :Inspect shows @markup.raw.block.markdown.
This is on nvim 0.10.0 with the default colorscheme.

#45 - This now fixes the issue that the above likely points to. The label of the signature help was being incorrectly identified for highlighting, despite correctly setting the highlight group.


Category.create NoMethodError: undefined method `nan?' for 2:Fixnum

ruby-1.9.2-p290 :022 > Category.create
=> #<Category _id: 4eac08548524884a43000008, _type: nil, lft: 1, rgt: 2, depth: 0, parent_id: nil>
ruby-1.9.2-p290 :023 > Category.create
NoMethodError: undefined method nan?' for 2:Fixnum         from /home/proton/projects/webra/bundle/ruby/1.9.1/gems/mongoid-2.2.3/lib/mongoid/contexts/mongo.rb:374:in grouped'
from /home/proton/projects/webra/bundle/ruby/1.9.1/gems/mongoid-2.2.3/lib/mongoid/contexts/mongo.rb:250:in max'         from /home/proton/projects/webra/bundle/ruby/1.9.1/gems/mongoid-2.2.3/lib/mongoid/criteria.rb:43:in max'
from /home/proton/projects/webra/bundle/ruby/1.9.1/gems/mongoid_nested_set-0.1.3/lib/mongoid_nested_set/update.rb:68:in set_default_left_and_right'         from /home/proton/projects/webra/bundle/ruby/1.9.1/gems/activesupport-3.1.1/lib/active_support/callbacks.rb:395:in _run_create_callbacks'
from /home/proton/projects/webra/bundle/ruby/1.9.1/gems/activesupport-3.1.1/lib/active_support/callbacks.rb:81:in run_callbacks'         from /home/proton/projects/webra/bundle/ruby/1.9.1/gems/mongoid-2.2.3/lib/mongoid/persistence/insertion.rb:25:in block (2 levels) in prepare'
from /home/proton/projects/webra/bundle/ruby/1.9.1/gems/activesupport-3.1.1/lib/active_support/callbacks.rb:399:in _run_save_callbacks'         from /home/proton/projects/webra/bundle/ruby/1.9.1/gems/activesupport-3.1.1/lib/active_support/callbacks.rb:81:in run_callbacks'
from /home/proton/projects/webra/bundle/ruby/1.9.1/gems/mongoid-2.2.3/lib/mongoid/persistence/insertion.rb:24:in block in prepare'         from /home/proton/projects/webra/bundle/ruby/1.9.1/gems/mongoid-2.2.3/lib/mongoid/persistence/insertion.rb:22:in tap'
from /home/proton/projects/webra/bundle/ruby/1.9.1/gems/mongoid-2.2.3/lib/mongoid/persistence/insertion.rb:22:in prepare'         from /home/proton/projects/webra/bundle/ruby/1.9.1/gems/mongoid-2.2.3/lib/mongoid/persistence/operations/insert.rb:26:in persist'
from /home/proton/projects/webra/bundle/ruby/1.9.1/gems/mongoid-2.2.3/lib/mongoid/persistence.rb:44:in insert'         from /home/proton/projects/webra/bundle/ruby/1.9.1/gems/mongoid-2.2.3/lib/mongoid/persistence.rb:149:in upsert'
from /home/proton/projects/webra/bundle/ruby/1.9.1/gems/mongoid-2.2.3/lib/mongoid/persistence.rb:170:in block (2 levels) in create'         from /home/proton/projects/webra/bundle/ruby/1.9.1/gems/mongoid-2.2.3/lib/mongoid/persistence.rb:170:in tap'
from /home/proton/projects/webra/bundle/ruby/1.9.1/gems/mongoid-2.2.3/lib/mongoid/persistence.rb:170:in block in create'         from /home/proton/projects/webra/bundle/ruby/1.9.1/gems/mongoid-2.2.3/lib/mongoid/persistence.rb:273:in creating'
from /home/proton/projects/webra/bundle/ruby/1.9.1/gems/mongoid-2.2.3/lib/mongoid/persistence.rb:169:in create'         from (irb):23         from /home/proton/projects/webra/bundle/ruby/1.9.1/gems/railties-3.1.1/lib/rails/commands/console.rb:45:in start'
from /home/proton/projects/webra/bundle/ruby/1.9.1/gems/railties-3.1.1/lib/rails/commands/console.rb:8:in start'         from /home/proton/projects/webra/bundle/ruby/1.9.1/gems/railties-3.1.1/lib/rails/commands.rb:40:in <top (required)>'
from script/rails:7:in require'         from script/rails:7:in 'ruby-1.9.2-p290 :024 >

mongoid (2.2.3)
bson (1.4.1)

Problem found in lib/mongoid_nested_set/update.rb:68
maxright = nested_set_scope.max(right_field_name) || 0


failed to bundle and package solution by error message "TypeError: Cannot read property 'readCommentedJsonFile' of undefined"

PS E:\spfx\githubdemos\GDPR\sp-dev-gdpr-activity-hub\GDPRStarterKit> gulp bundle --ship
Build target: SHIP
TypeError: Cannot read property 'readCommentedJsonFile' of undefined
TypeError: Cannot read property 'readCommentedJsonFile' of undefined
[14:04:52] Error - Unknown
undefined
[14:04:52] ==================[ Finished ]==================
[14:04:53] Project gdpr-starter-kit version: 0.0.1
[14:04:53] Build tools version: 2.10.0
[14:04:53] Node version: v6.11.1
[14:04:53] Total duration: 2.08 s
[14:04:53] Task errors: 2
PS E:\spfx\githubdemos\GDPR\sp-dev-gdpr-activity-hub\GDPRStarterKit> gulp package-solution --ship
Build target: SHIP
TypeError: Cannot read property 'readCommentedJsonFile' of undefined
TypeError: Cannot read property 'readCommentedJsonFile' of undefined
[14:23:17] Error - Unknown
undefined
[14:23:18] ==================[ Finished ]==================
[14:23:18] Project gdpr-starter-kit version: 0.0.1
[14:23:18] Build tools version: 2.10.0
[14:23:18] Node version: v6.11.1
[14:23:18] Total duration: 2.31 s
[14:23:18] Task errors: 2

Has todo with the dev. environment. goto gdrpstarterkit and execute

npm install --save

Hi Hans,
Thanks you very much for your great help indeed.
But, I did run this command before I'm trying to package it.
I run the command again, looks like it will skip it(only two warning) as it
have installed,
PS E:\spfx\githubdemos\GDPR\sp-dev-gdpr-activity-hub\GDPRStarterKit> npm
install --save
npm WARN optional SKIPPING OPTIONAL DEPENDENCY<EMAIL_ADDRESS>(node_modules\chokidar\node_modules\fsevents):
npm WARN notsup SKIPPING OPTIONAL DEPENDENCY: Unsupported platform for
<EMAIL_ADDRESS>wanted {"os":"darwin","arch":"any
"} (current: {"os":"win32","arch":"x64"})
Thanks,
Abraham Cheng
On 4 September 2017 at 14:40, Hans van der Meer<EMAIL_ADDRESS>wrote:

Has todo with the dev. environment. goto gdrpstarterkit and execute
npm install --save
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/SharePoint/sp-dev-gdpr-activity-hub/issues/21#issuecomment-326878147,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AJFOZPHiu9jrytqKrSdKmM47ybEqKD4Qks5se5tagaJpZM4PLgEo
.

download the source code again, then run the command " npm
install --save", then package command works fine


Add "Download File" Keyword

Originally submitted to Google Code by Andreas.EbbertKarroum on 13 Jul 2010

Add a "Download File" keyword to the library. Description in the codecentric blog:
http://blog.codecentric.de/en/2010/07/file-downloads-with-selenium-mission-impossible

Originally submitted to Google Code by spielman... on 19 Sep 2010

Pekka stated in the comments of above blog post:
"Very nice!
Instead of using wget you could have implemented a simple keyword with Python using urllib and/or urllib2 modules. This would remove the dependency to wget and thus make the approach platform independent. Should we actually add Download file keyword to SeleniumLibrary itself?"
@Andreas: I propose that we look at implementing this with urllib(2).
My 2 cents:

I don't really see "Download File" in the context of the Selenium Library, because it doesn't use Selenium functionality, except for the cookie part. Also, using wget somehow works around the browser, and thus works around Selenium.

On the other hand, moving it to Robot Framework itself appears impossible to me, because in the context in which Andreas developed this, you do need the cookie in order to get the file.

Putting a single keyword in its own library seems to be too much overhead, doesn't it?

I do think that the idea is interesting and that the need for such a keyword arises on a regular basis. Any ideas where this might go, if it were to be incorporated somewhere in RF or the associated libs?

Originally submitted to Google Code by @pekkaklarck on 20 Sep 2010

This functionality could be in a separate HTTP library. Implementing one using Python's httplib and urllib modules wouldn't be too hard technically. The biggest task would probably be deciding what keywords to include, how they should be used, and documenting everything thoroughly. In this case the SeleniumLibrary should get a keyword for getting cookie value (unless it already has).
I don't think adding this keyword into SeleniumLibrary would harm either, though.

Originally submitted to Google Code by Jameson.... on 2 Mar 2011

Here's a vote for just sticking it into SeleniumLibrary. I understand that that's not conceptually cleanest, but functionality matters more than that. It's something people would use, almost always from a context where SeleniumLibrary is already alive, so just use it there.
(And even if the goal is to find the "right, clean place" to put this function, I think that hacking it into selenium library and then waiting for someone to object is a better first step than dithering about it here. Forgiveness, not permission.)

Originally submitted to Google Code by mangaroo on 27 Sep 2011

I noticed a post in the referenced blog mentioning that "certain?" cookies were not accessible to Selenium, hence user had to use Selenium RC driver's capturenetworktraffic option enabled to sniff out the session cookie to use to then get the file. If we do plan to implement this issue, then need to look into this cookie thing first.
Also, use of httplib or urllib is best over wget for a solution that will work over Selenium Grid. Most Selenium workarounds like the GUI-based ones don't take into account Selenium Grid support (or they can't architecturally at present).

Originally submitted to Google Code by @pekkaklarck on 4 May 2012

Selenium RC has been deprecated and developing this library is thus ending. Selenium 2 (a.k.a. WebDriver) and Selenium2Library are the way forward. If you feel this issue is important, consider submitting it to the issue tracer of Selenenium2Library project.


Downloading precomputed meshes from sharded multi level mesh sources fails

Using meshparty.trimesh_io._download_meshes_thread_precomputed fails when the source is a cloudvolume ShardedMultiLevelPrecomputedMeshSource since that source's get method does not take a remove_duplicate_vertices kwarg.
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "<env>/lib/python3.9/site-packages/meshparty/trimesh_io.py", line 458, in download_meshes
    mu.multiprocess_func(_download_meshes_thread,
  File "<env>/lib/python3.9/site-packages/multiwrapper/multiprocessing_utils.py", line 64, in multiprocess_func
    result.append(func(p))
  File "<env>/lib/python3.9/site-packages/meshparty/trimesh_io.py", line 366, in _download_meshes_thread_precomputed
    cv_meshes = cv.mesh.get(
TypeError: get() got an unexpected keyword argument 'remove_duplicate_vertices'

I'm not sure if this belongs as an issue in cloud-volume to make their API consistent across sources or here! A simple fix would be a try-except with a warning that the kwarg is being dropped.

I am facing the same issue. Has anyone found a workaround?
Thanks a lot,
J

I pushed a fix to this just now, 1.16.12 should now have this fix.


Change i to im?

What do we think about changing the name of the imaginary unit from i to im? It's much less likely that someone will choose im for any other use.

Capital I is no better. I'll leave it as i then.


workflow generated site does not match

The site generated by the workflow is different from what is on simul's domain.
On first glance: Missing icon chapter 6, top tabs are not centers and size differently.

I've downloaded and extracted the site.tar.gz from the v0.0.1 tag (ff42e5e) and opened the pages in Chrome, Firefox and Edge. The website is rendering fine in all three browsers. I'm not sure what could be causing the issue here.

Either way it's now resolved.


Cannot send email - email setup on server

The values that goes into the .envrc should not start with a leading "export". That is used to set the variable on the command line. I removed those and it ran fine. I verified with python that the environment variables were set, but the send_email function will not work. I gathers all the email content, also verified, but the email will not be sent. It might be the choice of SMTP-server that I use, a domeneshop-server with SSL. I have tested that connetion elsewhere and it works.
What setup for email have you tested with? Any tips?
export BASE_URL=https://example.com
export<EMAIL_ADDRESS>export EMAIL_SENDER_PASSWORD=example
export EMAIL_SENDER_HOST=smtp.example.com
export DISCLAIMER='This is a disclaimer'

Found out why. I enclosed some of the functions in mailer.py with try..except and it revealed that I had to use a sender address with domain name when logging into the SMTP server. Now, I use a smtp server that does not use the email address as the username, so I added a EMAIL_SENDER_USERNAME variable, added it to envrc and docker-compose.yaml, and changed the mailer to log in with that instead, but kept the _ADDRESS for sender email purposes.
this solved the issue :-)


Testing flameeeeeeeee 🔥 for ex19 y2021

Let's try to figure out why this solution is so SLOOOOOOOOOOOW 🐌

Should we close this one?


change to throw error if directory of dictionary file is invalid

This update force program to throw error when the user give invalid parameter of dictionary files.
Now, the program executes endless loop when cannot open dictionary files.

Coverage remained the same when pulling 2197eafa8d284aeec16f5450c2a51b4e67a67db2 on tako-black:throwFileOpenError into 4ed2bc8b0b26fdbd790df53614322d1beec2027f on takuyaa:master.


Support for redirects and rewrites

Is your feature request related to a problem? Please describe.
I would like to merge existing projects to netlify, But we already are using Nextjs default rewrites and redirects..is it on the roadmap to support them? or is it always going to be a netlify redirect / rewrites features.
Describe the solution you'd like
Use NextJs features (redirects, rewrites, image etc) without any platform specific behavior
Describe alternatives you've considered
No real alternative.
Additional context
Right now we self host our applications. We would just keep doing that or consider some other alternative that supports nextjs as is (like vercel or others) although we already pay for Netlify (for other projects where we are super happy with it), we would just like to have it all in one place.

hey @joaogarin! i see you also commented on https://github.com/netlify/netlify-plugin-nextjs/issues/150. will close this as a duplicate and keep in mind your ask for this feature. given finn's comment, i'm not sure at the moment how we'd be able to provide full support.


"writing is not permitted" error on native iOS when writing to the characteristic on iPhone X

When I try to write [0x01] on iPhone X to the characteristic on a BLE device I got a platform error, which says: "writing is not permitted". It works well on any Android phone. It also works on iPhone 6S (iOS 13.3), but on iPhone X it doesn't. I traced this error up to rejectForFlutterResult method:

When debugging in Android studio I catch the following error:

BleError (Error code: 401, ATT error code: 3, iOS error code: null, Android error code: null, reason: Writing is not permitted.,

I reproduced the same characteristic write sequence in the Nordic nRF Connect app, and everything worked ok (on the same iPhone X). Do you know why this error occurs?
P.S. Thanks for this library, it's the most stable BLE library for the Flutter out there yet :)

Same code works for iPhone 6s with iOS 13.3, but does not work for iPhone X, yes? What's the system version on X? Could  you check the permission flags on that characteristic (isWritableWithResponse, isWritableWithoutResponse)?

Yes, the same code works on iPhone 6s (iOS 13.3) and any Android device, but doesn't on iPhone X (iOS 13.4). The characteristic is "WRITE_NO_RESPONSE". I use the following code:
Future<Characteristic> writeRfidControl(Peripheral peripheral, List<int> intList) async {
  return await peripheral.writeCharacteristic(
      StargateAttributes.RFID_SERVICE, StargateAttributes.CONTROL, toBytes(intList), false);
}

Here's nRF connect screenshot:

Are you using the newest version of the library?

Yes, I am using the latest version (flutter_ble_lib: ^2.2.3). I'd just tested develop branch ref, but still, iPhone X can't write the characteristic :(

Are both iPhones running the same OS?
The error occurs in step 4 or 5 of your stacktrace (safeWriteCharacteristicForDevice or subscribe). The one mentioned in the selected object leads to Multiplatform BLE Adapter and somewhere here: https://github.com/Polidea/MultiPlatformBleAdapter/blob/master/iOS/RxBluetoothKit/RxCBPeripheral.swift#L306
Could you try setting logs to verbose in FlutterBleLib and see if there's any more useful information there?
I don't have access to iPhone X for the foreseeable future, so you'll have to help me if you want it fixed. ;)

Of course, I will try help :)
iPhone 6s currently runs on iOS 13.4.1, while iPhone X is on iOS 13.4. I will have access to the iPhone 6S in a day or two so I can debug app personally on it and see if the issue lies in the iPhone, in iOS version or in something else

Start with verbose logs, if possible.

That's all I managed to track in Xcode, all the rest is leading to your RxBluetooth library. Unfortunately, I can't send you more data from our app logs. Also, the same codebase suddenly stopped working on iPhone 6S. I am trying to figure out why.

Hi, @austbot! Thanks for following up! I think system might cache attributes of a peripheral, so perhaps something changed in the firmware prior to this?
There was an issue fixed in 2.2.3 where withResponse flag was ignored on iOS (always had the same value, true I think), but it was fixed in 2.2.4.
Let me know if the issue occurs again, closing this for now.

I'm having this exactly issue on version 2.2.4


Shortcut Guide is not working on v0.16.1

Shortcut Guide is not working on v0.16.1

Hi @stcodebase
what happens after you long press the WIN key and then release it? Does the Start Menu show?

@stcodebase, do you have any other apps like AutoHotkey or something that could be remapping keys?

+1 no AHK installed.

+1 also no Keyboard Interfering Apps Installed

@lucifer @nailujx86
same question that I asked to @stcodebase: what happens after you long press the WIN key and then release it? Does the Start Menu show?
Also, are you running any program with administrator privileges or do you have UAC turned off?

@enricogior In fact, the Start Menu showed in that case.
However after enabling running powertoys in elevated mode the Shortcut Guide started working again.

@lucifer / @nailujx86 would it be possible to pop on a Teams call where we could share a screen to see installed programs / maybe what is running?

Thanks guys. I think it ok now. I reinstall the app, it works.


Convert all schematics to Ki-Cad

All schematics need to pivot from Fritzing to Ki-Cad as Fritzing is killing me with its routing bugs which I'm often not catching. 🤬

Oh dude, totally just found a few errors in my breadboard:

these are a lifesaver -- but I'm still buying kaldek a brewski.. rage against the ratgdo, if I could buy one I would. just release the schematics dude :[
kudos to you man! doing great work here --

to add, I'll help with drawing out some of the stuff soon as I get mine going...

I just started a PR with my stab at a KiCAD rendition.
https://github.com/Kaldek/rat-ratgdo/pull/13

Accepted pull request to add the Ki-Cad files as they are in their own folder structure.  Time for me to open/review these in Ki-Cad, and then we can close this issue.

@bjhiltbrand let's remove all of the power supply circuitry from the design; I was a bit cart-before-the-horse on this and it turns out there's not enough power available.  Other than that these look like we can use as the basic schematic for the repo.

Sounds good. I have removed those components, you want me to do another PR?

Sounds good. I have removed those components, you want me to do another PR?

Yep, hit it.

@bjhiltbrand I'm going to learn how to use Ki-Cad shortly (promise!) but can you remove J2 from the schematic?   Unless it's there for other reasons of course.

@Kaldek, I put the 2 terminal blocks on there to give people options. In KiCAD, I usually design the schematic first, and then have the program put the component footprints from the schematic on the PCB view when developing a PCB. So, both of these terminal blocks link pins together, so that when someone builds this board, they can choose to use a terminal block with a 5mm pitch between pins (typical screw terminal), or a 2.5mm pitch (typical phoenix or dupont connector). I can remove one of the terminals from the schematic, but leave the footprint on the PCB design, if that would help clear things up?

I can remove one of the terminals from the schematic, but leave the footprint on the PCB design, if that would help clear things up?

Yes it's probably worth doing that given there's already been some struggles with folks getting their prototypes up.  If needed we can work on basic vs. advanced schematics for the different audiences.   I honestly expected only the super technical folks to even consider trying this but I guess the backlog of ratgdo boards is making people desperate.

Alright. I removed the terminal block from the schematic, but not from the board design. I also changed the folder structure to have a self-contained kicad project for each variation on the board (D1 Mini - ESP32 vs. ESP8266). I have a couple more variations for bare ESP 8266 (12F) modules and ESP32 (WROOM) modules, but they're not ready to merge yet. I need to brush up one what pull-up/pull-down resistors I need and where they need to be placed first.

Please shared the esp32 version when you have a chance!

Please shared the esp32 version when you have a chance!

It's in PR right now. Once that's approved, it will be rolled in to the project. I think the pins are correct, but I'm not positive. I left the traces in place between the ESP8266 and ESP32 boards, so in theory, you should be able to use the same board for whichever D1Mini you choose. If the pin assignments need to be different, we will need to change that in the schematic.

Closing as these now exist and can be improved/developed as needed.


feat: Add ticket chart analytics

Fixes #261

Add chart view for ticket analytics
Add chart mutator for handling analytics logic

This is a mock representation. Need to discuss the analysis implementation @mejariamol

@iamareebjamal first will have to decide what data will be shown in the chart. that is what will be on y axis.

This is per day ticket sale analytics. Y axis is amount of tickets bought and X axis is the date when ticket was bought

I think we should have a separate chart for paid tickets with total earn on the Y axis. What do you say? As for paid tickets amount will be more informative that just a number. @iamareebjamal

That will be implemented in Tickets Page #267

Okay. So for implementation of sales versus time, we will need a sorted Ticket list with time of purchase
It can be achieved by sorting Attendee list with Order using completedAt attribute.  and creating sorted ticket list by iterating through these sorted attendees.

That was what I was thinking. Instead of iterating through tickets, we should iterate through orders. Because orders are less, and are done on a single date. What do you think

We will need:

ticket type( from Ticket)
completed at (from order)
quantity (from order)

Just have to think how can we get this info sorted with completedAt

We don't need to sort at all, this will be handled by the charting library

Ok, so what values does charting library take?

It takes a point (X, Y) which in our case will be (purchased, date)

but order and ticket are related by an attendee. How can we get ticket type by iterating through orders?

order will have one to many relationship with attendees

Yeah. then it is better to iterate through orders.

Once we should check complexities for both approaches

Orders = m
attendees = n
iterating through to orders will have m X n
but through attendees it will have just n
Is is correct? :smile:

No, because we have to get the order from attendees as well, and orders will be repeated, so 20 attendees will have the same order
And if we parse by order, then total attendees will be parsed only for their count and type, but it'll be done in 1 pass only

@mejariamol Please review

@mejariamol Can you please check now. I accidentally used Java 8 forEach construct

@iamareebjamal Some suggestions:

please use same colors in the chart and for progress bars.
Currently, it is not clear what actually the chart shows. Please add required info at axes.
please take care of extremity:

what will be shown when no tickets are sold

Same colors are used in both cases.
Axes will not give an immersive experience, and tooltips will handle the data information. The chart is a standard sales/time which is quite known form of analytic chart.
About extremities, I don't think the app will crash, just show an empty chart. I will check it

Axes will not give an immersive experience, and tooltips will handle the data information. The chart is a standard sales/time which is quite known form of analytic chart.

but at least it should show values at axes. :smile:

@mejariamol Imagine an event with 1000 tickets. How will the axes look then? We don't have logic to dynamically show the appropriate amount of labels on the axis. Probably in future
About the colors, I have used primary colors for lines and for dots, I have used the light color. The shade is used in progress bars to show complete incomplete data. What else should be done here?

We don't have logic to dynamically show the appropriate amount of labels on the axis.

For now, let's keep the number of labels at axes constant. Their values will be assigned dynamically. For example, 4 labels on the Y axis and 3 labels on the X axis.
Please try if we can set logic to assign these 7 variables dynamically. Because showing chart without any value at axes is really pointless.

@iamareebjamal is it possible now to switch chart library to PhilJay/MPAndroidChart. It is well documented. what do you say? I shall help in switching.

@mejariamol I don't know what such features it brings which WilliamCharts doesn't. I recommend you to check out its sample app https://play.google.com/store/apps/details?id=com.db.williamchartdemo&hl=en

And about the axes. I am not saying that we shouldn't add it. It was never my implication. The chart is far from complete, but if we set out to make it perfect in this PR only, it'll take forever. Issues should be such that they are solved in a day. This already took a lot of time. There are several things remaining on it and should be solved as other issues, so that we can independently and iteratively develop it.

I don't know what such features it brings which WilliamCharts doesn't. I recommend you to check out its sample app https://play.google.com/store/apps/details?id=com.db.williamchartdemo&hl=en

yeah. right. WilliamCharts also provides good doc. :+1:

@iamareebjamal what do you say about PhilJay/MPAndroidChart? shall I send PR to your repo? Or will you implement as most of the functions are similar? So it will be easier for you.

I'll do it :+1:

@mejariamol Please review now

Please resolve conflicts.


fix: include stats for all columns (#1223)

Description
This is a proposal for how #1223 could be fixed.
Related Issue(s)

fixes #1223

Documentation
The current implementation excludes all columns that lack statistical information. The proposed fix will generate information for all columns, with missing statistical values being replaced by 'null' values. However, it is unclear if this is the correct behavior since the stats_as_batch function lacks documentation.

Thanks for working on this.
I'm not sure, but I'm thinking for columns that don't have any statistics, we should either:

exclude them from the output
use NullArray for their stats, to minimize memory overhead.

My 2 cents would be to exclude them, since delta allows for a config how many columns to collect metrics for (which delta-rs does not yet honor :D), my expectation would be to only get metrics for these columns. This defaults to the first 32 columns.
https://learn.microsoft.com/en-us/azure/databricks/delta/table-properties (delta.dataSkippingNumIndexedCols)

Updated the PR to not include cols without any stats.
code to verify:
import pyspark
from delta import *

builder = pyspark.sql.SparkSession.builder.appName("MyApp") \
    .config("spark.sql.extensions", "io.delta.sql.DeltaSparkSessionExtension") \
    .config("spark.sql.catalog.spark_catalog", "org.apache.spark.sql.delta.catalog.DeltaCatalog")

spark = configure_spark_with_delta_pip(builder).getOrCreate()

location = "/tmp/delta/people10m"

DeltaTable.createIfNotExists(spark).addColumn("firstName", "STRING").addColumn("lastName", "STRING").addColumn("gender", "STRING").property("description", "table with people data").location(location).execute()

# create stats for firstName, lastName
spark.sql(f"ALTER TABLE delta.`{location}` SET TBLPROPERTIES(delta.dataSkippingNumIndexedCols = 2);")

columns = ["firstName", "lastName", "gender"]

data = [("Maria", "Dimas", "female"), ("John", "Francis", "male"), ("Frank", "Sinatra", "male")]
rdd = spark.sparkContext.parallelize(data)
df = rdd.toDF(columns)
df.coalesce(1).write.format("delta").mode("append").save(location)

# This adds a new update to the table, but this time all three columns will have stats.
spark.sql(f"ALTER TABLE delta.`{location}` SET TBLPROPERTIES(delta.dataSkippingNumIndexedCols = 3);")
data = [("Martin", "Johnson", "male"), ("Victoria", "Neal", "female")]
rdd = spark.sparkContext.parallelize(data)
df = rdd.toDF(columns)
df.coalesce(1).write.format("delta").mode("append").save(location)

#[tokio::main(flavor = "current_thread")]
async fn main() -> Result<(), deltalake::DeltaTableError> {
    let table_path = "/tmp/delta/people10m";
    let table = deltalake::open_table(table_path).await?;
    println!("{table}");
    let acts = table.get_state().add_actions_table(true).unwrap();
    dbg!(acts);
    Ok(())
}


Use of coerce needs constructors in scope

coerce is Magic and needs constructors in scope.

So now we have three kinds of declarations that need constructors in scope: foreign imports, standalone deriving and declarations using coerce. We should solve all three in a unified way.

b5cb4faf6c3f800bd64fe7050feda268c4b8e59d creates usages of constructors when the declarations is standalone deriving or mentions unsafe coerce. What remains to be done is use the same mechanism to import constructors for foreign imports.


Accept header could not be properly parsed because of a strict matching process

It is asking for accept header on all routes
local.ERROR: 400 - exception 'Symfony\Component\HttpKernel\Exception\BadRequestHttpException' with message 'Accept header could not be properly parsed because of a strict matching process.' in /home2/bhuvamka/public_html/apcivilsupplies.com/protected/vendor/dingo/api/src/Http/Parser/Accept.php:66

If you've got strict mode enabled then you won't be able to access your API via your web browser. You'll need to pass a valid accept header through either Postman or by using cURL from your console.

Yes, I know! But my normal Routes are not working. "Accept header could not be properly parsed", even on webpages too.
$api = app('api.router');
$api->version('v1',['namespace' => 'App\Http\Controllers', 'protected' => true, 'prefix' => 'api'], function ($api) {
$api->post('oxybiometric', 'Restcontroller@index');
});

Ah I see! Roger, I shall re-open and investigate then.

Can you run composer update and confirm all is well?

It's working perfect. Thank You

When do we expect Facade and stable version?

Both hopefully soon. Still ironing out a few bugs as people stress test more things.


The MuiThemeProvider doesn't apply its style to FormControl

In the docs an example demonstrates how I can style a TextField with a MuiThemeProvider. However, this doesn't apply to a FormControl, although they have the same general appearance. Could you please add MuiThemeProvider support for the FormControl?

@SamueleDassatti I don't understand. What are you trying to do?

https://material-ui-next.com/demos/text-fields/
Under the section "customized inputs" it says that if I wrap a TextField in a MuiThemeProvider I can use it to change its color. However this works only with TextFields but not with FormControls.

@SamueleDassatti The FormControl doesn't carry any color. Why should it work?

I meant the Input inside the FormControl, sorry

It's working too:
import React from "react";
import { MuiThemeProvider, createMuiTheme } from "material-ui/styles";
import Input from "material-ui/Input";
import green from "material-ui/colors/green";

const theme = createMuiTheme({
  palette: {
    primary: green
  }
});

function CustomizedInputs() {
  return (
    <MuiThemeProvider theme={theme}>
      <Input label="MuiThemeProvider" id="mui-theme-provider-input" />
    </MuiThemeProvider>
  );
}

export default CustomizedInputs;

https://codesandbox.io/s/vnjyx03675


Ability to create create customer UTO extensions

https://github.com/astronomer/astro-sdk/pull/1778#discussion_r1111100966

Should be created in astro-sdk


[SIP 44] Proposal for Dart Module implementation

Please make sure you are familiar with the SIP process documented
(here)[https://github.com/SvayamLabs/superset-ui-number-Format]
[SIP 44 ] Proposal for Dart Module implementation
Motivation
My Google-fu is strong.
Description of the problem to be solved.
the reason for the migration
1) typescript chunk size which makes the browser not responding whenthe data is large
2)Flutter is multiplatform (native )  we can port to android devices
3) some package deprecation leads  tools  as incapable
4)flutter widget have native performance which reduces  the browser hangover
Proposed Change
the migration of typescripts to flutter Language:
the syntax of typescript and dart language are similar in variable declaration and functional s declarations and OOPs concepts , the process of converting is easily carryout
the difference in between the both the language is vary only in the export and import process
New or Changed Public Interfaces
AS a native interface we can  create a more animative dashboard which can be competetive with other products like tableau,PowerBi
New dependencies
dart version > 2.0
flutter version > 1.17
Migration Plan and Compatibility
formatting modules ,connection modules =======>>  flutter Packages
Superchart , tooltips ,ui-plugins modules  =======>> flutter widgets
my plan is implementing our charting tool  in a stable products like flutter.
Flutter has variety of widgets
which are suitable  for common 3 interfaces which are
1)Android
2)Ios
3)web Applications
Cupertino (iOS-style) widgets
https://flutter.dev/docs/development/ui/widgets
AS a initial step i  created a module based on the @Superset-ui/Number-Format
the github link of my dart coded package
https://github.com/SvayamLabs/superset-ui-number-Format

https://github.com/Vino2530/dart.git

Do you have specific data on how typescript impacts chunk size and browser performance?

@suddjian

when i refer the chunck size of the superset  it looks pretty bigger when data flows
and the major aim of this Proposal is implementing the superset across all platforms
1)Desktop applications
a)linux
b)mac
c)windows
2)Mobile application
a)ios
b)android
3)web application
flutter having such ability of working across multi platforms compared to other languages
https://github.com/google/flutter-desktop-embedding

-1
I think moving from typescript to dart is a bad idea

-1 - we already have multiple frontend migrations in process and I don't think changing languages is prudent at this time.

I think that based on the sentiment in the thread, the timeline since last comment, and the lack of a VOTE thread, we'll call this closed. If it needs further attention or you want to push it forward for an official vote, we can re-open this. Thank you for the discussion/ideas!

I am really happy to see that there is a proposal at least. I understand this thread is dead, but... can I vote for this?
I use flutter since the beginning and find it a really strong framework. Is it possible to contribute?


[vcpkg] Reset to known-good build scripts

Should fix #10847.

@timautry @Farwaykorse could you build these and see if these actually do fix your issue?

I replaced the

toolsrc/CMakeLists.txt,

scripts/bootstrap.sh with the ones here

and renamed toolsrc/cmake/utilities.cmake to utilities.cmake.txt

reran the ./bootstrap-vcpkg.sh with the following errors about VCPKG_DISABLE_METRICS in the attached:
bootstrap-vcpkg.log
CMakeError.log
CMakeOutput.log

I can help as I am able off and on as I may need to rest due to my disability or do other household chores, etc.

@timautry this should actually work -.- I'm sorry

The latest change works... Yeah :)
bootstrap-vcpkg.log

Cloned in a new directory; checked-out fix-bootstrap; ran ./bootstrap-vcpkg.sh
54ec9b26cb74aa656b07742e7f28a01be580fb42 builds and links.

@Farwaykorse, @timautry can you also try the following?
git clone https://github.com/strega-nil/vcpkg
cd vcpkg
git checkout new-buildsystem
./boostrap-vcpkg.sh

?

I noticed the current HEAD of vcpkg CMake output Line 12
-- Detecting how to use the C++ filesystem library - <experimental/filesystem>

But when I run the functions in an other CMake project it shows:
-- Detecting how to use the C++ filesystem library - <filesystem>

So I tried changing the CMake version in the vcpkgTools.xml, to CMake v3.17.1.
This worked!
Changing it back to v3.14.0, again resulted in it trying to use <experimental/filesystem> and a failed build.

@Farwaykorse, @timautry can you also try the following?
git clone https://github.com/strega-nil/vcpkg
cd vcpkg
git checkout new-buildsystem
./boostrap-vcpkg.sh

?

This one builds. ffac3774dbdd16e8421924d45d7009d5ce572ec9

I created new instance vcpkg.devl from above instructions and it compiled and linked fine.
bootstrap-vcpkg-new-buildsystem.log

The detection to use <filesystem> works properly for CMake 3.15.0 and newer.
Since there is already a PR to update CMake to 3.17.1 #10767 this should be fixed soon.
@timautry can you try if it works for your issue too?
git clone https://github.com/Farwaykorse/vcpkg
cd vcpkg
git checkout update_cmake_linux
./boostrap-vcpkg.sh

@Farwaykorse I think the issue with openSUSE is unrelated -- it looks like the issue was attempting to link with a G++ that was different from the compiler.

@Farwaykorse - I still get a failed to link using the version you have provided using
bootstrap-vcpkg.cmake.test.log
cmake 3.17.1
attached

Superseded by #10867


Unexport var git.GlobalCommandArgs

as title

Codecov Report

Merging #18376 (4c52119) into main (3349fd8) will increase coverage by 0.02%.
The diff coverage is 82.75%.

@@            Coverage Diff             @@
##             main   #18376      +/-   ##
==========================================
+ Coverage   46.03%   46.06%   +0.02%
==========================================
  Files         840      840
  Lines       92856    92868      +12
==========================================
+ Hits        42746    42775      +29
+ Misses      43323    43307      -16
+ Partials     6787     6786       -1

Impacted Files
Coverage Δ

routers/install/install.go
0.56% <0.00%> (ø)

modules/git/command.go
89.14% <75.00%> (-1.61%)
:arrow_down:

modules/git/git.go
39.59% <80.00%> (ø)

modules/git/commit.go
61.15% <100.00%> (ø)

modules/git/diff.go
80.64% <100.00%> (ø)

modules/git/lfs.go
53.84% <100.00%> (ø)

modules/git/repo.go
69.37% <100.00%> (+0.38%)
:arrow_up:

routers/web/repo/http.go
55.83% <100.00%> (ø)

services/gitdiff/gitdiff.go
73.59% <100.00%> (ø)

models/unit/unit.go
48.45% <0.00%> (-2.07%)
:arrow_down:

... and 10 more

Continue to review full report at Codecov.

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


[release-1.23] Backport fixes/bumps from master

Proposed Changes
Backport bugfixes from master for the August release cycle:

93ca992cdb5be94b65747ce0bca41270183c7e5d - https://github.com/k3s-io/k3s/pull/5773
1674b9d6408dd4227a714506296cbe79c249a68d - https://github.com/k3s-io/k3s/pull/5882
bd5fdfce330fee140936e1ce4ba2453544a036bd - https://github.com/k3s-io/k3s/pull/5851
84fb8787f25304cecb6505ebedb654a29c5bf6d8 5eaa0a94226e850993f9c532ced93bd4249b7332 - https://github.com/k3s-io/k3s/pull/5894
049004455ab2807b2c37654e965a4b43ad7db695 - https://github.com/k3s-io/k3s/pull/5896
13af0b1d884e6c31f581fbec2cd5d4f8181de778 - https://github.com/k3s-io/k3s/pull/5906
43508341c14e776c8cdea7479f21a75245f000d8 - https://github.com/k3s-io/k3s/pull/5928

Types of Changes
backports
Verification
See linked issues
Testing

Linked Issues

TBD

User-Facing Change

Bumped minio-go to v7.0.33. This adds support for IMDSv2 credentials.
When set, the agent-token value is now written to `$datadir/server/agent-token`, in the same manner as the default (server) token is written to `$datadir/server/token`
Updated dynamiclistener to fix a regression that prevented certificate renewal from working properly.
The configured service CIDR is now passed to the Kubernetes controller-manager via the `--service-cluster-ip-range` flag. Previously this value was only passed to the apiserver.
Fixed a regression that caused systemd cgroup driver autoconfiguration to fail on server nodes.
The initial health-check time for the etcd datastore has been raised from 10 to 30 seconds.
Bumped rootlesskit to v1.0.1

Further Comments

s390x builder is out of disk space; merging


Store the pull secret once for all

crc should not ask for the pull secret every time. It should store it, either in ~/.crc or in a secure place (https://github.com/zalando/go-keyring for instance).

This is what we had before but then in case of pull secret expire then how do we handle it. Current pull secret validation code only check if input is a valid json schema with some property but we don't have any api to check if it is a expired cert or not.

This is what we had before but then in case of pull secret expire then how do we handle it. Current pull secret validation code only check if input is a valid json schema with some property but we don't have any api to check if it is a expired cert or not.

Included in 1.22.0


Heatmap demo: Can't get vertical scroll working

Library information:

Version: 2.6.0
Heatmap Compose

Question:
I just migrated my heatmap chart to your formidable library. Following your heatmap demo I got everything working quite quickly, the last thing missing is that I somehow can't get the Box() holding the heatmap to scroll vertically.
My implementation is very similar to your demo, means, a user can click on a day to show the content of that day at the bottom of the screen. In portrait mode there is sufficient space, in landscape however, things get cut off or won't show at all.

I tried adding Modifier.verticalScroll(rememberScrollState()) to the top level Box and also the column below, but somehow nothing seems to work?
Box(modifier = Modifier.safeDrawingPadding()) {
            Column(modifier = Modifier.fillMaxSize()) {
                val state = rememberHeatMapCalendarState(
                    startMonth = startDate.yearMonth,
                    endMonth = endDate.yearMonth,
                    firstVisibleMonth = endDate.yearMonth,
                    firstDayOfWeek = firstDayOfWeekFromLocale(),
                )
                HeatMapCalendar(
                …

Anyone got an idea what might go wrong here?

Repeated the button and put Example 6 in a verticalScroll modifier, and it worked without issues. Are you sure you are setting the modifier to the correct layout?
https://github.com/user-attachments/assets/a27cbab9-4194-44fa-b1a0-a62bf7fcebef


Include inode in file fingerprints

Closes #24.

thanks... pypy3 benchmarks failing is unrelated
Run asv run HEAD^-1
  asv run HEAD^-1
  shell: /usr/bin/bash -e {0}
  env:
    pythonLocation: /opt/hostedtoolcache/PyPy/3.6.12/x64/bin
    FSCACHER_BENCH_TMPDIRS: .:/tmp/nfs:/tmp/vfat
· No executable found for python 3.6
· No environments selected
Error: Process completed with exit code 1.

and I wonder if we should just kill that (pypy3 for 3.6)?

thanks... pypy3 benchmarks failing is unrelated
Run asv run HEAD^-1
  asv run HEAD^-1
  shell: /usr/bin/bash -e {0}
  env:
    pythonLocation: /opt/hostedtoolcache/PyPy/3.6.12/x64/bin
    FSCACHER_BENCH_TMPDIRS: .:/tmp/nfs:/tmp/vfat
· No executable found for python 3.6
· No environments selected
Error: Process completed with exit code 1.

and I wonder if we should just kill that (pypy3 for 3.6)?


add option to skip waiting for the updated nuget feed at the end of pack/promote/demote

this is helpful in scenarios where the nuget feed isn't directly updated by snapx
for example with sleet (https://github.com/emgarten/Sleet)
What does the pull request do?
add the option to skip waiting for the updated nuget feed at the end
What is the current behavior?
waiting for the updated nuget feed in pack, promote, demote
What is the updated/expected behavior with this PR?
adds the command line option to skip the waiting
Checklist

[ ] Added unit tests (if possible)?
[ ] Added XML documentation to any related classes?

@hutterm LGTM! Thank you for your contribution 🥇


Test: TerminalOptions.icon API

Refs #120538

[ ] macOS
[ ] linux
[ ] windows

Complexity: 3
Authors: @tyriar

Using the extension-terminal-sample, create a terminal with TerminalOptions including an icon (a codicon id). Verify that this icon is used when you create that terminal and have terminal.integrated.showTabs: true.
Make sure you've enabled proposed api

Oh this one actually isn't ready for testing, we're going to change it to take ThemeIcons/uris


libphal: Fix SPR, XIR, GPR sequence in SBE dump

The sequence of adding SPR, XIR, and GPR in to the SBE dump is different from other service processors, making it same to use a common parser for dump content
Signed-off-by: Dhruvaraj Subhashchandran<EMAIL_ADDRESS>

Merged.


No reason to tap caskroom

Following a change in caskroom from december (https://github.com/caskroom/homebrew-cask#important-december-2015-update-homebrew-cask-will-now-be-kept-up-to-date-together-with-homebrew-see-15381-for-details-if-you-havent-yet-run-brew-uninstall---force-brew-cask-brew-update-to-switch-to-the-new-system) caskroom/cask should no longer be tapped.

Oh neat, thanks.


feat: add mtime glob struct

This PR adds a struct that helps with the computation of the newest modification time for a file. This PR
also cleans some of the used functions and adds documentation for everything

Can you elaborate a little bit wether this also changes the behavior / hashes for the task cache?

@wolfv That is current a seperate implemention. Would be good to merge these at some point. I looked into reusing that initially but there are some performance issues with the method we use in the task hash. E.g. the glob /etc/bin/which* should be very fast but the implementation used by the task hash will search all files on your harddrive. This is a known issue that has been open for years. wax doesn't have this issue. Its also use by other crates like turborepo for instance.

ok, that's a bit annoying but yeah, ideally we can then re-use the new one in the task cache later on.


Complete Scrapping of Gutenberg

Description
Currently Gutenberg website has around 72K books. This issue is to scrape all the books
Expected Behavior
gutenberg_bibliographic_records.json should have around 72 K records
Current Behavior
Today gutenberg_bibliographic_records.json has around 13K records
Possible Solution (optional)
Run Gutenberg scrapper till we scrape all the records

Any new commers can take up this.

What is the issue @vigneshRamanathan3105 ?

We have no issue @neokd .
we only need to run the scrapper till we scrape all the books. Which might take several hours (due to huge no. of books)
As of now the latest book is John's Lily

Thanks for your contribution @VigneshRamanathan101 We'll try scrapping more books


Update emulationstation2.pot

hi,
the pot file is an automatically updated file.
for batocera translations, go here : https://translations.batocera.org/

hi, the pot file is an automatically updated file. for batocera translations, go here : https://translations.batocera.org/

Thank you for your strong support!
Previously I had committed to github twice and it was reflected. After being informed, I am registering according to the translation site's procedures.
I think my ID (DDinghoya) should be registered as a translator on https://translations.batocera.org/
Can you decide? Or is there another way?

yes, for which language ?

yes, for which language ?

Korean on translation site. :)


Bump bindings and Mina repo for lookup gate fix

This PR updates the bindings from https://github.com/MinaProtocol/mina/pull/15450 to fix https://github.com/o1-labs/o1js/issues/1374.

I took the liberty of merging #1284 here - the PR that originally exposed the bug - to verify that this works on the o1js side now. Also addressed my own comments on that PR :D


A scoring variable allows score in pld-test to be customized

Also used in pld-grid-search.

Coverage remained the same at 100.0% when pulling edd9212b8d4e5ebe466898078afb4eaec4159cfc on feature/pld-test-scoring into dd6839ad4d9f13604ec6bd48a03609df9f43cd17 on develop.


feat: add estimation fallback

adds a fallback estimation if NoEstimateAvailable is received from the stacks node
this is closer to legacy behavior and once existed in a way (before transaction estimates were a thing and everything was a transfer estimate)

Codecov Report

Merging #1422 (5dbcd63) into master (272028e) will increase coverage by 0.26%.
The diff coverage is 74.19%.

@@            Coverage Diff             @@
##           master    #1422      +/-   ##
==========================================
+ Coverage   63.49%   63.75%   +0.26%
==========================================
  Files         116      116
  Lines        8500     8498       -2
  Branches     1875     1895      +20
==========================================
+ Hits         5397     5418      +21
+ Misses       2974     2953      -21
+ Partials      129      127       -2

Impacted Files
Coverage Δ

packages/transactions/src/errors.ts
83.33% <60.00%> (+17.61%)
:arrow_up:

packages/transactions/src/builders.ts
77.66% <80.95%> (+4.52%)
:arrow_up:

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


A problem about fine-tuning for resnet

Hi, I wonder why there are so many steps  to finetune resnet?

What do you mean so many steps?

In tensorflow-cnn-finetune/resnet/examples.sh:
These steps:
python finetune.py
--learning_rate "0.00001"
--train_layers "fc"
python finetune.py
--learning_rate "0.00001"
--train_layers "fc,scale5/block3"
python finetune.py
--learning_rate "0.00001"
--train_layers "fc,scale5/block3,scale5/block2"
python finetune.py
--learning_rate "0.00001"
--train_layers "fc,scale5/block3,scale5/block2,scale5/block1"
python finetune.py
--learning_rate "0.00001"
--multi_scale "225,256"
--train_layers "fc,scale5"
python finetune.py
--learning_rate "0.00001"
--multi_scale "225,256"
--train_layers "fc,scale5,scale4/block6"
python finetune.py
--learning_rate "0.00001"
--multi_scale "225,256"
--train_layers "fc,scale5,scale4/block6,scale4/block5"

These are 7 different individual finetuning examples, they all finetune different layers as you can see. They are not sequential steps or something like that. You gotta call just once python finetune.py with your desired parameters.

I see. Thanks!