Add space to change [] into checkboxes

Description

Fixes a small issue in the discussion template where [] are rendered as text instead of checkboxes due to a missing space in between them.
Example

[] This ([]) isn't a checkbox
[ ] This ([ ]) is!

Checklist

[x] My pull request has a descriptive title (not a vague title like Update index.md).
[x] My pull request targets the default branch of the repository (main) or a parent feature branch.
[x] My commit messages follow best practices.
[x] My code follows the established code style of the repository.
[ ] I added or updated tests for the changes I made (if applicable).
[ ] I added or updated documentation (if applicable).
[ ] I tried running the project locally and verified that there are no visible errors.

Developer Certificate of Origin

Developer Certificate of Origin
Developer Certificate of Origin
Version 1.1

Copyright (C) 2004, 2006 The Linux Foundation and its contributors.
1 Letterman Drive
Suite D4700
San Francisco, CA, 94129

Everyone is permitted to copy and distribute verbatim copies of this
license document, but changing it is not allowed.

Developer's Certificate of Origin 1.1

By making a contribution to this project, I certify that:

(a) The contribution was created in whole or in part by me and I
    have the right to submit it under the open source license
    indicated in the file; or

(b) The contribution is based upon previous work that, to the best
    of my knowledge, is covered under an appropriate open source
    license and I have the right under that license to submit that
    work with modifications, whether created in whole or in part
    by me, under the same open source license (unless I am
    permitted to submit under a different license), as indicated
    in the file; or

(c) The contribution was provided directly to me by some other
    person who certified (a), (b) or (c) and I have not modified
    it.

(d) I understand and agree that this project and the contribution
    are public and that a record of the contribution (including all
    personal information I submit with it, including my sign-off) is
    maintained indefinitely and may be redistributed consistent with
    this project or the open source license(s) involved.

Fixing at the upstream source of the file: https://github.com/WordPress/openverse/pull/302


ExpectedConditions.elementToBeClickable doesn't account for other elements covering the target

In the following:
var element = $('#div');
browser.wait(protractor.ExpectedConditions.elementToBeClickable(element), 5000, 'Element not clickable');
element.click();

If $('#div') is covered by another element element.click() results in a selenium error:

unknown error: Element is not clickable at point (10, 10). Other element would receive the click: ...

Even through browser.wait(protractor.ExpectedConditions.elementToBeClickable(element), 5000, 'Element not clickable') does not result in an error.

I definitely see this behavior as well and want to see this fixed.
Currently, I use EC.and(EC.elementToBeClickable(element),
EC.invisibleOf(overlayElement)) as a workaround.
On Wed, May 13, 2015 at 9:05 AM, hgted<EMAIL_ADDRESS>wrote:

In the following:
var element = $('#div');
browser.wait(protractor.ExpectedConditions.elementToBeClickable(element), 5000, 'Element not clickable');
element.click();
If $('#div') is covered by another element element.click() results in a
selenium error:
unknown error: Element is not clickable at point (10, 10). Other element
would receive the click: ...
Even through browser.wait(protractor.ExpectedConditions.elementToBeClickable(element),
5000, 'Element not clickable') does not result in an error.
—
Reply to this email directly or view it on GitHub
https://github.com/angular/protractor/issues/2139.

Correct, right now elementToBeClickable just checks that it's visible and enabled.

I don't think there is anything that can be done here unless webdriver exposes an API that returns all elements for a particular coordinate. Also, for reference, the behavior is consistent with Java and Python's expected condition libraries (i.e. https://code.google.com/p/selenium/source/browse/java/client/src/org/openqa/selenium/support/ui/ExpectedConditions.java)

We would address this using document.elementFromPoint and element pretty easily actually.  The consistency issue is real though

Hi there, I ran into the same issue. How about adding something like elementNotToBeCovered? That way the functionality would be easily available without creating inconsistencies :-)

@hankduan thoughts on @dbartholomae's suggestion?

Closing this issue in favor of the more specific issue https://github.com/angular/protractor/issues/2313

I have wrote small check utility method, keep in mind it will click on element immediately when it become clickable:
import { ElementFinder, promise } from 'protractor';

 export let testHelpers = {
   isClickable(el: ElementFinder): promise.Promise<boolean> {
     return new promise.Promise(resolve => {
       let interval = setInterval(() => {
         el.click().then(() => {
           clearInterval(interval);
           setTimeout(() => {
             resolve(true);
           }, 500);
         }, () => { });
       }, 100);
     });
   }
 }

In your test code:
```
import { testHelpers } from '../src/core/e2e/helpers';
describe('App', () => {
  it('should do something', () {
    let btn = $('.cls');
    browser.wait(testHelpers.isClickable(btn), 3000);
  });
});

@alllx Out of curiousity, why are you resolving inside the setTimeout? I think you're solution will work for what I'm trying to do with a slight modification. Just curious why you are resolving the promise this way before I implement.

Hi @amilbeck sorry, for now, I don't remember there was some delay needed, was tested only in Chrome. Maybe now in newer versions, it is don't needed.

Failed: Cannot click on element
Build info: version: '3.141.59', revision: 'e82be7d358', time: '2018-11-14T08:25:53'
System info: host: 'DESKTOP-RPOSEHB', ip: '<IP_ADDRESS>', os.name: 'Windows 10', os.arch: 'amd64', os.version: '10.0', java.version: '11.0.2'
Driver info: driver.version: unknown
Stack:
MoveTargetOutOfBoundsError: Cannot click on element
who can help me


Logout without any activity?

λ python like_hashtags.py streetphotography
2019-09-06 00:25:24,696 - INFO - Instabot Started
We need to create a text file 'secret.txt' where we will store your login and password from Instagram.
Don't worry. It will be stored locally.
Enter your login: xxxxxxxxx
Enter your password: (it will not be shown due to security reasons - just start typing and press Enter)
Password:
Do you want to add another account? (y/n) n
Which account do you want to use? (Type number)
1: xxxxxxxxx
0: add another account.
-1: delete all accounts.
1

It stuck here without any error.

TroubleShoot :
1)make sure your script isn't scheduled
2) CTRL + C stops the script , then try to rerun

CTRL + C makes a false logout request to IG, so it's more safe way to stop the script, than ending a session.
It could be this function has been taken out or you should modify it yourself.
Example:
bot = Bot(do_logout=True)
https://github.com/instabotai/instabotai/blob/master/run.py#L22

After doing all the exercises still on same place.
E:\python\instabotai>python run.py -u xxxx-p xxx
E:\python\instabotai>

Finally solved by uninstalling python 3.7 and installing python 3.6

Problem should be solved and not closed with a "workaround".


e2e training or 2stage training?

I am wondering whether e2e training is the best way.
If I train global net to a steady stage and then I fix global net and start to train refine net, what the result would be like? Have you tried this approach and what about the results.

At first we trained

thanks for your comment


VCrayApp.bat does not need to pause after return from App execution

We want a pause if there is no App execution and VCsplice is not "+" and we want the user to see where we ended up.
Any fail automatically stops everything.

Corrected in VCrayApp.bat 0.0.36.  There is no need to pause on any successful exit.


Adds iOS target

Fix #24
I just created a new target with its own test target and "redirected" the same tests to run for the mobile architecture.
A couple of conditional target imports in 3 files and the project now supports iOS framework with tests passing :)

@esttorhe I've checked your PR and it looks fine but I suggest you change both target names to follow some pattern eg.

XcodeServerSDK - OS X
XcodeServerSDK - OS X Tests
XcodeServerSDK - iOS
XcodeServerSDK - iOS Tests

Also I've noticed that Tests for new iOS target are marked as red in Product directory. You know something about this and where to fix this?
When you fix these things I think this PR will be ready to merge from my POV.
I've also checked - your PR will allow us to use Playground inside project so we can interact with SDK. Everything in that matter is configured as it should be and is described in #26.

Good job! 😉 👌

Here's the list of fixes:

Renamed targets as suggested on your comment
Removed Info.plist for iOS target as its not needed (using same as the OSX target)
Renamed product names

Regarding your question of the test target in red; basically Xcode is a «String based» IDE, now that both targets share the same product names Xcode will search for the same (in this case OSX one) and that's why you see both iOS and OSX targets in black (because both products are pointing to the same artifact) but the test target had different names and Xcode was looking for the iOS test product in the same folder as the OSX ones failing to find it. ¯_(ツ)_/¯

Awesome, thanks @esttorhe and @cojoj. I have some slight edits for this but I'll merge this first and push those changes to the original repo's swift-2 branch right after.


More link shape properties

References

This is a draft PR, working through a way to add an array behavior property.

Resolves #60 by adding curvature and line dashes to LinkShape.
Code changes

[x] add curvature to LinkShape
[x] add line_dash to LinkShape

[x] disable this behavior in 3D (it is not supported)

[x] add coercion to array for line_dash data
[x] update notebooks

[x] fixed a small issue with node selection in Behaviors.ipynb

[x] update docs
[x] add tests

[x] str inputs
[x] int inputs
[x] float inputs
[x] Column inputs
[x] Nunjucks inputs
[x] LinkShape.color
[x] LinkShape.curvature
[x] LinkShape.line_dash
[x] LinkShape.width

[x] Interaction with selection

[x] need to make sure LinkSelection is included before LinkShape to show selection styling
[x] LinkSelection works but it is not tested in conjunction with LinkShape
[x] future work would be to figure out a way to make default_curvature=None to leave the link with its non-selected curvature

User-facing changes

Two new properties on LinkShape:

LinkShape.curvature:

0.0 produces a straight link (default)
1.0 circular link (i.e., radius of curvature = 0.5 * link length)
negative numbers mirror the link

LinkShape.line_dashes:

Define the line dashes using an array of numbers, reference

Backwards-incompatible changes

No backwards incompatible changes (knock on wood)

Curvature is no sweat. 2197523bf5659958a6a1ab767ccbbc0147eaaa70 adds curvature.
As a number:

As a nunjucks template:

line dash seems to work as well
c82314a

3D has rotation for links (spin around the axis between source and target), not sure it is worth it to bring that in or not, but it could make sense as dash_line only works on 2D.

but it could make sense as dash_line only works on 2D.

Again, not blocking, but should probably filter that out of the 3d demo.

but it could make sense as dash_line only works on 2D.

Again, not blocking, but should probably filter that out of the 3d demo.

Good call, fixed in da96020


MOBILE-823 login: Add site first implementation

Added a first implementation of the add site feature
Issue: https://tracker.moodle.org/browse/MOBILE-823

Noting observations as I look through the patch:

No need to add 'ctrl' to controller/*.js files
You surely have written that before but we agreed to use 'BlahController' rather than 'BlahCtrl'
I think we could use $mmConfig directly rather than using it in a 'resolve', and the $mmConfig.initConfig could be set in .run() of its module.
Why do we need a specific 'retry' parameter to getUserToken? That method should not build a URL IMO.
Not too sure what kind of key that is getItem('service'+siteurl);
Why do we prioritise HTTP over HTTPS in determineService?

I'm conscious that most of these are coming from the original app, I'm just asking.


[FEATURE] Allow output names for wayland monitors

Description of the requested feature
Essentially this would allow someone to assign a window to a display by inputting the name of the monitor(e.g. "eDP-1") instead of the monitor number.
Proposed configuration syntax
The syntax for this would look like this
:monitor "eDP-1"

Additional context
This is basically the same as PR #522 but for Wayland.

As discussed previously on that very PR, this is really hard to do properly since GTK doesn't actually expose that info. I'll have to look if things have changed on the gtk-rs side and come back with an update.

@viandoxdev are you still working on this?
I made a quick lookup and
    let monitor: gdk::Monitor = monitors.item(0).unwrap().downcast().unwrap();
    println!("Monitor Name: {:?}", monitor.description());

prints "Monitor Name: Some("AOC 2369M AJDG89A003953 (HDMI-A-1)")"
which you could get the name from

otherwise, i would have time next week to make a pr

@viandoxdev are you still working on this?
I made a quick lookup and
    let monitor: gdk::Monitor = monitors.item(0).unwrap().downcast().unwrap();
    println!("Monitor Name: {:?}", monitor.description());

prints "Monitor Name: Some("AOC 2369M AJDG89A003953 (HDMI-A-1)")"
which you could get the name from

otherwise, i would have time next week to make a pr

I'm not particularly interested in implementing this so don't hesitate to make a PR. I'm not convinced you can reliably extract the name from that description (if that's what you intend to do), but it would be great if you could manage, good luck

Still working on it, but i am quite busy. Might take two weeks.

Did a little more digging and GDK gets its information from https://wayland.app/protocols/xdg-output-unstable-v1#zxdg_output_v1:event:description whose output differs by compositor. In gtk4, it would be possible to get this information from wl_output, so,
tldr: not really possible at the moment without very janky solutions.

should be fixed by 510b824e7545a7c98e050522a7bc93d884c53432 which is indeed very janky


Implement case sensitive property matching

This PR aims to address #815, by implementing a new serializer option, named PropertyCaseSensitivityHandling:
public enum PropertyCaseSensitivityHandling
{
    // Case-insensitive matching (default).
    CaseInsensitive = 0,

    // Case-sensitive matching.
    CaseSensitive = 1
}

To preserve continuity with prior implementations, default is set to CaseInsensitive.
With the new behaviour provided by CaseSensitive, we can now have differentiation and validation for casing, which is required by several API specs (Binance & xAPI, to name a few examples).
As this behaviour has an extremely lightweight implementation (only one extra check on each Property matching), it does what it says on the tin, and only that.
Most of the matching with convention-following PascalCase C# properties will fail if you don't add [JsonProperty("name")] to each property, or write some esoteric conversion between JSON standard camelCase (or whichever casing is used) and C# Properties.

A few footnotes...
This PR was inspired by the stale #601, where I factually stripped the Casing implementation to form my own.
Furthermore, as this is my first ever PR to Newtonsoft.Json (hello guys!), I might've inadvertently broken a few long-standing conventions along the way, so any feedback will be most appreciated.

Really glad to see a PR about that. We had to fork the repo to make these changes. It would be nice to see them integrated in Newtonsoft


Change export html format

Here is the new design for our document. You can contribute to this under twindle-cli
Figma
Figma will help you with CSS and measurements.
https://www.figma.com/file/34YX1uk6ZVhQWVzLyjzUdG/Twindle?node-id=0%3A1
Guidelines

There is absolutely no need to stick to measurement
But, please stick to:

alignment
colors
font
font weight

These are the mockups. Comment below for any change/enhancement/whatever 😛
PS: There is an image in middle which didn't get exported.

This is a good idea, although for the docs, which we are sending to tidible, only need a proper font size, and spacing. Just though it was worth mentioning.
good designs 👍🏻👍🏻

Agree with @tolgaerdonmez

@JoelVinayKumar We need only white background, as our primary target is Kindle.
It would be great if you could recommend some minimalistic designs keeping the same layout as below

You can follow this design. Just remove Twindle thread as heading

The same design I gave can be made monochrome as well

I think we need to add a logo and stylised font, to show off a little branding. Can you propose minimalist design based on that?
A little screenshot with an SVG of logo or something would do.

maybe something on the corners ?

I'm thinking a logo, and the text Twindle alongside it at the top. Nothing too grand, just some simple branding


Additional Generic Curtain Motor Using Tuya Board  ( _TZE200_nogaemzt, TS0601 )

Additional Motor looks same as others different manufacturer like https://www.zigbee2mqtt.io/devices/TS0601_cover.html renamed to be more generic and added additional input cluster missing from original.
Fully tested with my device and working fine.

I have another new device not in the list so will add that and resubmit.


Chakra failed with error C4596 when build with permissive- + MSVC on windows.

Chakra failed with error C4596 when build with permissive-  by msvc on Windows, I use latest source on master branch. Could you please help take a look at this? this issue involved by commit https://github.com/Microsoft/ChakraCore/commit/ff4d7e93aea9b8d2d5c6d1ff020def2a0055ee16
We reported multi similar issues last year, some fixed some are still occurred.
The issue in https://github.com/Microsoft/ChakraCore/blob/master/lib/Common/Core/ConfigParser.h#L43
https://github.com/Microsoft/ChakraCore/blob/master/lib/Common/Core/ConfigParser.h#L44
You can repro this issue as the steps below:
1.git clone -c core.autocrlf=true https://github.com/microsoft/ChakraCore D:\Chakra\src
2.Open a clean x86 prompt (C:\windows\syswow64\cmd.exe) and browse to D:\Chakra\src
3.set CL=-wd4471 /permissive-
4.msbuild /m /p:Platform=x86 /p:Configuration=Test /p:WindowsTargetPlatformVersion=10.0.16299.0 Build\Chakra.Core.sln /t:Rebuild
Error info:
D:\Chakra\src\lib\Common\Core/ConfigParser.h(43): error C4596: 'SetConfigStringFromRegistry': illegal qualified name in member declaration
D:\Chakra\src\lib\Common\Core/ConfigParser.h(44): error C4596: 'ReadRegistryString': illegal qualified name in member declaration

Similar issue that involved by https://github.com/Microsoft/ChakraCore/commit/9c8394ff8971fd7ad4bf60a4104fe66f0307cf21
Failures:
d:\chakra\src\lib\backend\IRBuilder.h(325): error C4596: 'GetEnvironmentOperand': illegal qualified name in member declaration
To fix this issue, please remove the 'IRBuilder::'

Similar issue in d:\chakra\src\lib\backend\lower.h:
d:\chakra\src\lib\backend\lower.h(365,48): error C4596:  'HoistIndirOffset': illegal qualified name in member declaration
d:\chakra\src\lib\backend\lower.h(366,53): error C4596:  'HoistIndirOffsetAsAdd': illegal qualified name in member declaration
d:\chakra\src\lib\backend\lower.h(367,56): error C4596:  'HoistIndirIndexOpndAsAdd': illegal qualified name in member declaration
d:\chakra\src\lib\backend\lower.h(368,46): error C4596:     static IR::Instr *Lowerer::HoistSymOffset(IR::Instr *instr, IR::SymOpnd *symOpnd, RegNum baseReg, uint32 offset, RegNum regNum);
d:\chakra\src\lib\backend\lower.h(369,51): error C4596:  'HoistSymOffsetAsAdd': illegal qualified name in member declaration

Any updates?


fixes issue #274 - can this get a review???????????????????????????

fixing an issue that prevented orders from being created because of how pydantic was being called
fixes #274
❯ python
Python 3.10.11 (main, Apr  7 2023, 07:24:53) [Clang 14.0.0 (clang-14<IP_ADDRESS>)] on darwin
Type "help", "copyright", "credits" or "license" for more information.
>>> from alpaca.broker.requests import OrderRequest
>>> from alpaca.common.enums import SupportedCurrencies
>>> OrderRequest(symbol="AAPL", side="buy", type="limit", time_in_force="day", qty=1, currency=SupportedCurrencies.USD)
{   'client_order_id': None,
    'commission': None,
    'currency': <SupportedCurrencies.USD: 'USD'>,
    'extended_hours': None,
    'notional': None,
    'order_class': None,
    'qty': 1.0,
    'side': <OrderSide.BUY: 'buy'>,
    'stop_loss': None,
    'symbol': 'AAPL',
    'take_profit': None,
    'time_in_force': <TimeInForce.DAY: 'day'>,
    'type': <OrderType.LIMIT: 'limit'>}
>>> OrderRequest(symbol="AAPL", side="buy", type="limit", time_in_force="day", qty=1, currency=SupportedCurrencies.EUR)
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "pydantic/main.py", line 342, in pydantic.main.BaseModel.__init__
pydantic.error_wrappers.ValidationError: 1 validation error for OrderRequest
__root__
  Order type must be OrderType.MARKET if the order is in a local currency. (type=value_error)
>>> OrderRequest(symbol="AAPL", side="buy", type="market", time_in_force="day", qty=1, currency=SupportedCurrencies.EUR)
{   'client_order_id': None,
    'commission': None,
    'currency': <SupportedCurrencies.EUR: 'EUR'>,
    'extended_hours': None,
    'notional': None,
    'order_class': None,
    'qty': 1.0,
    'side': <OrderSide.BUY: 'buy'>,
    'stop_loss': None,
    'symbol': 'AAPL',
    'take_profit': None,
    'time_in_force': <TimeInForce.DAY: 'day'>,
    'type': <OrderType.MARKET: 'market'>}
>>>

@haxdds Mind taking a look at this? I'm surprised more people aren't calling this out...

Sorry it took so long to get around. The past few months we've had limited resources dedicated to maintanence of the repo. The changes look good. I'm merging it into a staging branch so we can see how it interacts with our migration to pydantic v2 before we merge it into master.


Added combo box to config

Made a combo so that the user could select the config without needing to know the name of his existing configs.

good changes, thanks for your contribution!


[FEATURE] gui for team list

What is the feature request for?
like when someone type /team list it shows gui of team lists and click to join the team
Relevant links
No response
Additional Information
No response
Feature requests are not going to be implemented by booksaw, instead you will have to find another developer to make a pull request to implement the feature.

[X] I understand

Duplicate of #547

Closing, as this is a duplicate ticket


Example docs aren't qualifying Point3d

Current KittyCAD API schema includes a type Point3d. For some reason, the code generation from openapitor is generating examples incorrectly. Note that in the example below, the Point3d type isn't being used like other types are.
use std::str::FromStr;
async fn example_modeling_cmd() -> anyhow::Result<()> {
    let client = kittycad::Client::new_from_env();
    let result: serde_json::Value = client
        .modeling()
        .cmd(&kittycad::types::ModelingCmdReq {
            cmd: kittycad::types::ModelingCmd::AddLine {
                from: Point3D {
                    x: 3.14 as f64,
                    y: 3.14 as f64,
                    z: 3.14 as f64,
                },
                to: Point3D {
                    x: 3.14 as f64,
                    y: 3.14 as f64,
                    z: 3.14 as f64,
                },
            },
            cmd_id: uuid::Uuid::from_str("d9797f8d-9ad6-4e08-90d7-2ec17e13471c")?,
            file_id: "some-string".to_string(),
        })
        .await?;
    println!("{:?}", result);
    Ok(())
}

This example won't compile, so cargo test fails. It suggests:
error[E0422]: cannot find struct, variant or union type `Point3D` in this scope
  --> src/modeling.rs:32:21
   |
16 |                 to: Point3D {
   |                     ^^^^^^^ not found in this scope
   |
help: consider importing this struct
   |
2  | use kittycad::types::Point3D;
   |

We should fix openapitor so that its generated examples properly qualify the Point3d name. @jessfraz says this is related to the in_crate: bool parameter.

Fixed by https://github.com/KittyCAD/kittycad.rs/pull/187


connection issue

Description
The connection breaks very easily as following
requests.exceptions.ConnectionError: HTTPSConnectionPool(host='qfe.nas.nasa.gov', port=443): Max retries exceeded with url: /sapi/problems/062013ca-8f32-499c-8498-dddd01398ca2/ (Caused by NewConnectionError('<urllib3.connection.VerifiedHTTPSConnection object at 0x114771320>: Failed to establish a new connection: [Errno 60] Operation timed out',))
To Reproduce
Just run several connections simultaneously or run a three hour simulation.
Expected behavior
Keep connected.
Environment:

OS: [MacOS]
Python version: [e.g. 3.7.0]

Additional context
In the Python 2 version SAPI2, I did not experience such a problem. This seems coming from Ocean Tool.

This looks like network issues on NASA site.
Can you try running the same program using D-Wave cloud API?

Closing this issue due to inactivity.


Wall time auto-resubmit not working

Hi :)
I fear the wall time auto-resubmit is not working for me.
I'm using this submit script:
#SBATCH --job-name=NORA_test_MNIST
#SBATCH --ntasks=1
#SBATCH --cpus-per-task=1
#SBATCH --mem=12000
#SBATCH --gres=gpu:1
### #SBATCH --mem-per-gpu=11000
#SBATCH -o /home/%u/%x-%j-on-%N.out
#SBATCH -e /home/%u/%x-%j-on-%N.err
#SBATCH --mail-type=ALL

#Timelimit format: "hours:minutes:seconds" -- max is 24h
#SBATCH --time=00:15:00

#Resubmit (provided by pytorch lightning)
#SBATCH --signal=SIGUSR1@90

echo "Your job is running on" $(hostname)
python3 mnist_model.py

mnist_model.py is the model from https://towardsdatascience.com/from-pytorch-to-pytorch-lightning-a-gentle-introduction-b371b7caaf09
Pytorch Lightning 1.2.4 is installed on the cluster.
In the error file I get:
TPU available: None, using: 0 TPU cores
2021-03-22 11:19:52.797360: I tensorflow/stream_executor/platform/default/dso_loader.cc:49] Successfully opened dynamic library libcudart.so.11.0
/home/gourmelon/.local/lib/python3.8/site-packages/pytorch_lightning/utilities/distributed.py:51: UserWarning: GPU available but not used. Set the --gpus flag when calling the script.
  warnings.warn(*args, **kwargs)
Set SLURM handle signals.

  | Name    | Type   | Params
-----------------------------------
0 | layer_1 | Linear | 100 K
1 | layer_2 | Linear | 33.0 K
2 | layer_3 | Linear | 2.6 K
-----------------------------------
136 K     Trainable params
0         Non-trainable params
136 K     Total params
0.544     Total estimated model params size (MB)
slurmstepd-lme171: error: *** JOB 656086 ON lme171 CANCELLED AT 2021-03-22T11:34:49 DUE TO TIME LIMIT ***

In the out file I simply see the progress of training.
I also assured that the SIGUSR1 is actually sent using this script:
#!/bin/bash
#
#SBATCH --job-name=pers_job_debug
#SBATCH --dependency=singleton
#SBATCH --signal=B:SIGUSR1@90
#SBATCH --time=00:05:00
#SBATCH -o /cluster/%u/%x-%j-on-%N.out
#SBATCH -e /cluster/%u/%x-%j-on-%N.err
# catch the SIGUSR1 signal
_resubmit() {
    ## Resubmit the job for the next execution
    echo "$(date): job $SLURM_JOBID received SIGUSR1 at $(date), re-submitting"
    sbatch $0
}
trap _resubmit SIGUSR1

echo "$(date): job $SLURM_JOBID starting on $SLURM_NODELIST"
while true; do
    echo "$(date): normal execution"
    sleep 60
done

This script gives me:
Mon 22 Mar 2021 11:35:52 AM CET: normal execution
Mon 22 Mar 2021 11:36:52 AM CET: normal execution
Mon 22 Mar 2021 11:37:52 AM CET: normal execution
Mon 22 Mar 2021 11:38:52 AM CET: normal execution
Mon 22 Mar 2021 11:39:52 AM CET: job 656090 received SIGUSR1 at Mon 22 Mar 2021 11:39:52 AM CET, re-submitting
Submitted batch job 656092
Mon 22 Mar 2021 11:39:52 AM CET: normal execution

Any help is highly appreciated! Please also tell me if the example is not minimal enough (this is my first minimal example :D) or if I should try something else first.

Same problem here with a recent version of pytorch lightning. Are there any updates?

Any updates??

No :( I just made a work around where I specify how many epochs to train (= roughly how many will get done before my job is killed), then save a checkpoint and resubmit...

I am also still having this issue. Basically _slurm_sigusr_handler_fn (https://github.com/Lightning-AI/lightning/blob/049f6c576ad32f4878ffadc24d2b03b329dd9b53/src/lightning/pytorch/trainer/connectors/signal_connector.py#LL69C27-L69C27) never gets called for me.
@Borda any ideas?

Same issue here. Has anyone found a workaround aside from capping the epochs or slicing the training set?


修复中文文档中，关于节点驱逐速率的错误翻译

原来的翻译有问题，翻译成了“节点unhealthy后的策略改变”，实际是“zone的状态（或者zone的异常节点比例）变成unhealthy后，节点驱逐速率的改变

Thanks for the PR
Please write the PR title in English, even if the target language is Chinese.

Thanks for your contribution. However, seems this change is not very necessary, so this PR will be closed. Feel free to re-open it if you provide more improvements.
/close


Add ChenYi015 as spark-operator approvers

🛑 Important:
Please open an issue to discuss significant work before you start. We appreciate your contributions and don't want your efforts to go to waste!
For guidelines on how to contribute, please review the CONTRIBUTING.md document.
Purpose of this PR
Provide a clear and concise description of the changes. Explain the motivation behind these changes and link to relevant issues or discussions.
Proposed changes:

Add myself @ChenYi015 as spark-operator approvers

Change Category
Indicate the type of change by marking the applicable boxes:

[ ] Bugfix (non-breaking change which fixes an issue)
[ ] Feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that could affect existing functionality)
[ ] Documentation update

Rationale

I am currently a member of Kubeflow organization.
Contributed PRs: https://github.com/kubeflow/spark-operator/pulls?q=is%3Apr+author%3AChenYi015
Checklist
Before submitting your PR, please review the following:

[ ] I have conducted a self-review of my own code.
[ ] I have updated documentation accordingly.
[ ] I have added tests that prove my changes are effective or that my feature works.
[ ] Existing unit tests pass locally with my changes.

Additional Notes

/assign @yuchaoran2011


Add a way to configure retention

Prometheus retention is configured via flag and not config, and thus there isn't a way right now to say "keep N weeks/months of data" without customizing the image. It would be great if we could somehow supply the retention (or even other flags) for use with prometheus.

Happy to make a PR if folks have a good direction here. I don't really want to add knobs for every possible flag, so whatever makes most sense to make this flexible works for me.

the image is meant for testing - but I'll happily accept a PR

If you want to have more advanced configs, you can do so by mounting config files like here:
https://github.com/grafana/docker-otel-lgtm/blob/2476e74e4980568d1003a526a8b2d887b7e8aff0/examples/java/json-logging-logback/k8s/lgtm.yaml


Expandable / collapsible row and Pagination

Hi all,
thanks for your great job.
I have a problem with pagination on expandable row. I see the first 20 items of "child" table, but if i click to view the second page, i see "No result found.".
Can you help me?
Thanks!

Anyone have the same problem?
Thanks!

Not happening to me.
How to reproduce this issue?

Is there anybody get this issue? I will close this issue until there's someone reply a confirmed bug n find a way to reproduce this issue..


Upstream Several Bug Fixes

This is a fairly large PR which fixes core issues with offer handling and Jenkins slave lifecycle.  The version number should be changed to whatever is deemed appropriate.  This has been tested on internal production systems.
Bug Fixes

Introduce a proper offer processing thread and make it fault tolerant
Fix threads waiting for Jenkins computers not dying on task failure

NOTE
There are some indenting / whitespace changes.  Use this link to eliminate the whitespace diff.

@Vlatombe @jglick Do you know who the appropriate people are to have a look at these changes?

@maselvaraj: What version number would be appropriate here?


{ preserveSnapshot: true } results in 'undefined' values

How to reproduce these conditions
Steps to set up and reproduce
Follow the docs 3-retrieving-data-as-lists example to the bottom of the page with the preserveSnapshot option.
The example works fine when we leave out the preserveSnapshot option.
** Screenshots **

Expected behavior
Display values of each key.
No error in console.
Actual behavior
Displays "undefined".
Console error.
Debug output
** Errors in the JavaScript console **
core.es5.js:3025 Angular is running in the development mode. Call enableProdMode() to enable the production mode.
AppComponent.html:4 ERROR TypeError: Cannot read property 'set' of undefined
at Object.isFirebaseRef [as h] (utils.js:13)
at FirebaseListObservable.checkOperationCases (firebase_list_observable.js:59)
at FirebaseListObservable.update (firebase_list_observable.js:36)
at AppComponent.webpackJsonp.116.AppComponent.updateItem (app.component.ts:18)
at Object.eval [as handleEvent] (AppComponent.html:4)
at handleEvent (core.es5.js:11826)
at callWithDebugContext (core.es5.js:13041)
at Object.debugHandleEvent [as handleEvent] (core.es5.js:12629)
at dispatchEvent (core.es5.js:8777)
at core.es5.js:9369
View_AppComponent_1 @ AppComponent.html:4
DebugContext.logError @ core.es5.js:12981
ErrorHandler.handleError @ core.es5.js:1144
(anonymous) @ core.es5.js:9373
(anonymous) @ platform-browser.es5.js:2683
ZoneDelegate.invokeTask @ zone.js:414
onInvokeTask @ core.es5.js:4117
ZoneDelegate.invokeTask @ zone.js:413
Zone.runTask @ zone.js:181
ZoneTask.invoke @ zone.js:476
AppComponent.html:4 ERROR CONTEXT DebugContext_ {view: Object, nodeIndex: 4, nodeDef: Object, elDef: Object, elView: Object}
View_AppComponent_1 @ AppComponent.html:4
DebugContext_.logError @ core.es5.js:12981
ErrorHandler.handleError @ core.es5.js:1149
(anonymous) @ core.es5.js:9373
(anonymous) @ platform-browser.es5.js:2683
ZoneDelegate.invokeTask @ zone.js:414
onInvokeTask @ core.es5.js:4117
ZoneDelegate.invokeTask @ zone.js:413
Zone.runTask @ zone.js:181
ZoneTask.invoke @ zone.js:476
Version info
"@angular/common": "^4.0.0",
"firebase": "^3.9.0",
"angularfire2": "^4.0.0-rc.0",
Win10Ult64
Chrome Version 60.0.3094.0 (Official Build) canary (64-bit)

@leotm Can you create a plnkr of this problem so it's easier to debug?

@leotm Plunkr will help but my immediate guess is that you forgot to do .val(). When you use preserveSnapshot option the returned values of objects needs to be accessed with .val(). E.g. data.val().name instead of data.name.

Going to close for now, if you can repro after @elishaterada's suggestion please toss on plnker and reopen.

So just spent the past couple hours trying to get it working in Plunker.
JUST spotted your Plunker Template.
I'll be back.

Here's the plnkr.
Still getting undefined. However, correct output in the console.
All looks fine in Realtime Firebase.
Also can't figure why deleting an item, deletes them all 🤔
(Btw, I don't have permish to reopen the issue)


Option to kill process of host application if not close on close OR wait until it closed

My programm need a bit longer to close and that occus that the updater try to overwrite the file of the running process which is closing. This cause that a error window opens that tell me that the updater wan't able to overwrite the specific file. When I click to close the updater tries again.
But this is not realy nice for the end-user. He see the error and make a bug report to me.
How would be a option for the developer who can setup the behavior for what happens if the host application doesn't close after the updater told the process to close?
E.g. setup by changing a property "BehaviorWhenHostApplicationDoesntClose" on UpdateManager of type of an Enum with this values: ShowErrorDialog, KillProcess, WaitForProcessHasClosed

Hi,
I can try to implement a code in the update installer that will wait until the host application has closed.
Let's see, if that can help.

You may test the newest release to see, if it works now.

I got the new version and will test it next week when I am at home. I will notify you if it works now.


Majority read/write concern is hard-coded in some places and non-configurable.

Describe the bug
In this commit: https://github.com/cloudyrock/mongock/commit/3f4fb2e1816196fafd80563b7ea28984c2d661ff
Majority read and write concern is being hard-coded.  It makes sense as a default option, but for some pre-3.6 Mongo instances, majority read concern is not enabled by default (or perhaps it could be turned off on purpose)
Is it okay to allow the default to be overridden?
To Reproduce
Run pre-3.6 mongo without --enableMajorityReadConcern.  Migrations cannot be run in this case.
Expected behavior
Configurable read/write concern
Logs
N/A
Screenshots
N/A
Additional context
N/A

Good point, thanks. We'll improve this and will notify you

Just encountered this trying to use 4.3.2 when using https://github.com/bwaldvogel/mongo-java-server which is used for integration tests and some local development instead of a real mongo instance. Would love to have this configurable!

Hello @abobwhite , can you explain a bit more? I don't fully understand your point. I see that can be a useful tool for integration tests, but I am not sure why is related to this issue. Thanks!

Sure @dieppa. When running a local config using embedded mongo or in our integration tests, the spring boot application full initializes and executes the mongock changelogs. This is a good way to test our changelogs before hitting a real DB. Unfortunately that mongo-java-server doesn't support the setting of the "ReadConcern" that's happening in Mongock. It works with Mongock BOM 4.1.19 but not with any 4.3.x.

Thanks @abobwhite  and @scottfrazer . Jus to keep you updated, we are already working on this. We expect to have a release at the end of this week.
Thanks

Hello @scottfrazer  and @abobwhite ,
sorry for the delay. This is already done in the version 4.3.5, please could you try. We haven't still document it but meanwhile, I paste a piece of configuration that illustrates how to do it.
mongock:
  change-logs-scan-package:
    - com.github.cloudyrock.mongock.changelogs.client
  trackIgnored: true
  mongo-db:
    write-concern:
      w: majority
      w-timeout-ms: 300000
      journal: false
    read-concern: majority
    read-preference: primary

As it has the spring configuration processor, the IDE will probably suggest the values. The only that is free text is mongo-db.write-concern.w, which should be any value that MongoDB accepts for it.
Any question, please, let us know

seems to work, thanks!

@abobwhite, did you get this working with mongo-java-server? I am using version 1.41.0 and still falling in the same issue with different values of the read-concern property.

Version 1.41.0 of Mongock?


Changes in scoped styles not applied until Vite reload. Vite 4.x

In Vite 4.x, using a <style scoped> block inside a layout SFC file will not have its classes or styles applied until restarting the dev server (reloading is not sufficient).
To reproduce:

Start the dev server npm run dev.
Add a <style scoped> block to an SFC used as a layout. Add a class and apply it to an element in the layout's template.
Load a page showing the layout. The style is not applied.
Restart the dev server. The style is now applied.

I did not test this on Vite 3.x, so I'm unsure whether its an incompatibility with the new major version or some other type of regression.

I met the same issue

PR welcomed :)

I cannot duplicate in Vitesse, with Vite 5.0.5
https://github.com/antfu/vitesse
If you find an issue with that template and Vite 5, please open a new issue


It is working?

Hi,
Is this repository working with current versions of react-native/android/ios?
Thanks

Works with ios I can confirm, but having issues with android.

Is there any reason why you wouldn't use the native upload with xmlhttprequest + FormData? http://doochik.com/2015/11/27/FormData-in-React-Native.html

Could this lib or xmlhttprequest + FormData works with part of file upload?
I need to separate file on some parts and than concat on server side


Fixed the Carousel navigation buttons and the text

Fix Issue #191
This PR fixes the issue where the "Previous" and "Next" buttons were overlapping the carousel margins and ensures proper spacing between the buttons and the page edges.
Changes made:

Adjusted CSS for the button margin and background.
Replaced circular button backgrounds with rounded rectangles to fit the "Previous" and "Next" text.
Conditional rendering of Carousel.Caption to show only on the second slide.
Fixed duplicate arrow signs appearing before the "Previous" and "Next" text.

Testing:

Tested on both light and dark images.
Buttons now properly spaced from the margin and display correctly.
Verified on desktop and mobile views.

https://github.com/user-attachments/assets/5dbfe7cc-31b4-4755-b51e-3e6ab84b5a49

Hello @18Prachi , there are some conflicts please do solve and commit your changes

Still there are some conflicts, please do have a look

Can you please explain what conflicts are there as I am new to open
contributions and am not aware of these scenarios.
On Tue, Oct 15, 2024 at 11:02 PM Soham Adhyapak @.***>
wrote:

Still there are some conflicts, please do have a look
—
Reply to this email directly, view it on GitHub
https://github.com/soham0005/ElectroKart/pull/214#issuecomment-2414616441,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A6IEZFQ2QOYGZ2K4OPNYMJLZ3VGT3AVCNFSM6AAAAABP66W6L2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIMJUGYYTMNBUGE
.
You are receiving this because you were assigned.Message ID:
@.***>

connect on discord, i'll explain what conflicts you are facing

Sure, can you share your id
On Tue, Oct 15, 2024 at 11:47 PM Soham Adhyapak @.***>
wrote:

connect on discord, i'll explain what conflicts you are facing
—
Reply to this email directly, view it on GitHub
https://github.com/soham0005/ElectroKart/pull/214#issuecomment-2414706280,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A6IEZFTEAJRQWQJYLGJRJ73Z3VL4DAVCNFSM6AAAAABP66W6L2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIMJUG4YDMMRYGA
.
You are receiving this because you were assigned.Message ID:
@.***>

soham.215

I have made the changes, you can check now.
On Wed, Oct 16, 2024 at 12:54 AM Soham Adhyapak @.***>
wrote:

soham.215
—
Reply to this email directly, view it on GitHub
https://github.com/soham0005/ElectroKart/pull/214#issuecomment-2414831657,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A6IEZFW2IZCSVRAIDTMSU63Z3VTW3AVCNFSM6AAAAABP66W6L2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIMJUHAZTCNRVG4
.
You are receiving this because you were assigned.Message ID:
@.***>

Still  not solved

@18Prachi Have you figured out the solution/


Set default ResourceRequirements on all resources under OCS

[x] Ceph

[x] Mons
[x] OSDs
[x] Mgrs
[x] MDSs
[x] RGW?

[ ] NooBaa

[x] Core
[ ] Mongo?

Ref: https://url.corp.redhat.com/9d41deb

/cc @jarrpa @obnoxxx All tests pass now.


Make RUMMAGER_INDEX=all refer to all indexes

Currently, it only refers to all indexes holding content.  This is
frequently confusing people.  Instead, make RUMMAGER_INDEX=all refer to
all indexes used by rummager.
This means that auxiliary indexes are also migrated when a
deploy:migrations is done - currently, they're not being.  This means
that if we changed the schemas of the auxiliary indexes, they wouldn't
be updated automatically, which also seems undesirable.

LGTM, though I'll leave it to someone more familiar with Rummager to merge.


Can't run sample usage to get data assets

Package Name: azure-core
Package Version: 1.27.1
Operating System:Linux
Python Version: 3.10

Describe the bug
I used the code given by azure ml to use to get my data asset. But I got an error :

AttributeError                            Traceback (most recent call last)
/home/azureuser/cloudfiles/code/Users/lucas.colas/segmentation-of-windows-and-doors/detection/train_model.ipynb Cell 8 line 1
8 workspace = Workspace(subscription_id, resource_group, workspace_name)
10 dataset = Dataset.get_by_name(workspace, name='object_detection_images')
---> 11 dataset.download(target_path='.', overwrite=False)
File /anaconda/envs/azureml_py310_sdkv2/lib/python3.10/site-packages/azureml/data/_loggerfactory.py:132, in track..monitor..wrapper(*args, **kwargs)
130 with _LoggerFactory.track_activity(logger, func.name, activity_type, custom_dimensions) as al:
131     try:
--> 132         return func(*args, **kwargs)
133     except Exception as e:
134         if hasattr(al, 'activity_info') and hasattr(e, 'error_code'):
File /anaconda/envs/azureml_py310_sdkv2/lib/python3.10/site-packages/azureml/data/file_dataset.py:155, in FileDataset.download(self, target_path, overwrite, ignore_not_found)
150 download_list = None
151 if not is_empty and not overwrite:
152     # need to fail if destination is not empty and download would have collisions
153     download_list = [
154         os.path.abspath(os.path.join(target_path, '.' + p))
--> 155         for p in self._to_path(activity='download.to_path')
156     ]
158     for p in download_list:
159         # encode p to avoid UnicodeEncodeError from os.path.exists
160         if os.path.exists(_encode_if_needed(p)):
...
--> 677     (prefix_path, _) = dataflow._find_path_prefix()
678     portable_path = 'Portable Path'
679     get_portable_path = dataprep().api.functions.get_portable_path
AttributeError: 'Dataflow' object has no attribute '_find_path_prefix'

To Reproduce
Steps to reproduce the behavior:

Create a data asset (URI_Folder) for images.
Go to consume and use the code :

Expected behavior
I should be able to download the data asset.

@LucasColas thanks for your issue. @azureml-github will take a look and respond as soon as possible.

Good. Thank you.

Thank you for reporting this. As it stands today the only type of V2 data asset that is supported in V1 SDK is MLTable.
With currently released code you have two options:

Use V2 sdk to download data
Create an MLtable pointing to the same data and register it as MLTable asset, then load it by name in V1 SDK.

I have added code to support both UriFolder and UriFile data assets in V1 SDK, so it should be released with the next azureml-sdk version (1.56 most likely)
Hope this helps

Thanks @anliakho2 , @LucasColas I will update next azureml-sdk release version 1.56.0

Hi @LucasColas ,
we released SDK 1.56.0 in pypi.org could you please validate and confirm?


Class Container should not be forced

The class container should be optional. This prevents customization of the object, rather it should be given an ID and we can implement any styling we wish.

I would totally be ok with making the container class an option.


[Branch-2.10][Cherry-pick] tidy update subscriptions dispatcher rate-limiter

Motivation

Cherry-pick https://github.com/apache/pulsar/pull/15789  to branch-2.10

Modifications

Because PR-15789 is modified base  on other PR, so the following PR  also need cherry-pick

https://github.com/apache/pulsar/pull/15789
https://github.com/apache/pulsar/pull/15604
https://github.com/apache/pulsar/pull/15315
https://github.com/apache/pulsar/pull/15457
https://github.com/apache/pulsar/pull/14890
https://github.com/apache/pulsar/pull/14909
https://github.com/apache/pulsar/pull/14387
https://github.com/apache/pulsar/pull/14342
https://github.com/apache/pulsar/pull/14038

Verifying this change

[x] Make sure that the change passes the CI checks.

Documentation
Check the box below or label this PR directly.
Need to update docs?

[ ] doc-required
(Your PR needs to update docs and you will update later)

[x] doc-not-needed
(Please explain why)

[ ] doc
(Your PR contains doc changes)

[ ] doc-complete
(Docs have been already added)

/pulsarbot run-failure-checks


[Feature request] Check if certain option is set on each form

I'd like to check if the option csrf_protection is set on each form explicitly, to make sure I haven't forgot one.
Ondřej Mirtes says it's possible https://github.com/phpstan/phpstan-symfony/issues/343#issuecomment-1501111078 but I don't know how to implement such a rule.
With Symplify, the form options to look for could be configured as arguments.
P.S.: I know that form options can be configured globally, but I'd like to make sure that my forms work independently of global settings.

Hi, thanks for suggestion, but it's not the right fit here.
This rather seems like a custom rule for your rule set :+1:


[Gally]: master <- dev

Automatically created by Git-Ally

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Authorization code

How to get authorization code not by input method but by a function???

https://adw0rd.github.io/instagrapi/usage-guide/interactions.html

Actually I am creating a program that interacts with other app and user gives id and password there . Now if Instagram asks for verification code I want to ask user to give code in that app and then enter it but this package receives code by input prompt

then you need this
https://adw0rd.github.io/instagrapi/usage-guide/challenge_resolver.html


Promote incubating ignoring of empty directories

Summary

[ ] @IgnoreEmptyDirectories
[ ] TaskInputFilePropertyBuilder.ignoreEmptyDirectories()
[ ] TaskInputFilePropertyBuilder.ignoreEmptyDirectories(boolean)
[ ] PropertiesFileNormalization
[ ] RuntimeClasspathNormalization.properties()

Checklist

[ ] Validate whether we should de-incubate the API in the 7.0 release.

If the removal is not possible, create a follow-up issue and link it in the overview spreadsheet

[ ] Update release notes (add API to promoted features)
[ ] Check User Manual (it might mention that the API is still incubating)

Think about updating snippets and samples to use this API

[ ] Deprecate existing API that is replaced by the new one

If it's not immediately possible, create a follow-up issue with a milestone (e.g. 7.1 RC1 or 8.0 RC1)

[ ] Check Incubation Report on TeamCity (example)
[ ] Mark the story as done in the overview spreadsheet.

@gradle/execution this API is quite new (6.8). Do we want to deincubate in 7.0? If so I can take it.

@gradle/execution this API is quite new (6.8). Do we want to deincubate in 7.0? If so I can take it.

I don't have any objection to de-incubating this.  Certainly for @IgnoreEmptyDirectories.  I'm a little less confident about properties normalization as it has more moving parts, but I'm also not opposed.

I don't have any objection to de-incubating this.  Certainly for @IgnoreEmptyDirectories.  I'm a little less confident about properties normalization as it has more moving parts, but I'm also not opposed.


Make action only comment on Pull Request if there are failures, errors or more skips

The action should allow to configure it to only comment on the PR if there are any useful information in the test results, i.e. there exist failures, errors or the number of skipped tests increase w.r.t. to the earlier commit. The first run compares to the base.

I used Github notification and would like to only receive the notification when there is test fails

I would find this very helpful still: we run a matrix of 20+ test runs per PR, so this gets out of hand very quickly.

Can you upload test results from each of these 20+ runs and run the publish action once on all test result artifacts / files? If not, what prevents you from that?

I used Github notification and would like to only receive the notification when there is test fails

No notification could also mean that publish check did not run. But from the check that the publish action itself succeeded you see tests have been analysed. If that action fails, you would get notified for that. So I think it is safe to disable the comment in that case. Will look into this.

This has just been released. Enjoy!


Input and Solutions for Day 1 from ylks

Code in separate PR since the introduced bazel change is currently breaking CI

See https://github.com/Moelf/AoC2023/tree/feature/bazel


CSS theme doesn't seem to work properly on Firefox 116???

Here's an image of what is happening when i use the css you provided, i did apply the following in about config and set the boolean to true:
toolkit.legacyUserProfileCustomizations.stylesheets =>true
layout.css.has-selector.enabled =>true
svg.context-properties.content.enabled =>true
userChrome.Tabs.Option4.Enabled =>true
userChrome.DarkTheme.TabFrameType.Border.Enabled =>true
userChrome.DarkTheme.TabFrameType.Shadow.Enabled =>true
userChrome.DarkTheme.TabFrameColor.Black.Enabled =>true
userChrome.DarkTheme.TabFrameSaturation.Medium.Enabled =>true

Firefox Version: 116.0.2
OS: Windows 11 22H2 Insider Build
Proof:
toolkit.legacyUserProfileCustomizations.stylesheets =>true

toolkit.legacyUserProfileCustomizations.stylesheets =>false

idk if this helps but i stored the chrome files here:

You must have installed the v1.6.117 Beta style. It uses completely new CSS properties. Therefore, it does not work on Firefox 116 and below. Download the style from the releases section.

Are you using v1.6.117 Beta? If yes, then replace it with the release one (v1.6.116) or activate the layout.css.nesting.enabled key. However, I do not know how it is implemented in the 116 version of the browser.

I was having the same problem but this solved it. Thank you!

Are you using v1.6.117 Beta? If yes, then replace it with the release one (v1.6.116) or activate the layout.css.nesting.enabled key. However, I do not know how it is implemented in the 116 version of the browser.

Activating layout.css.nesting.enabled worked for me! Thanks for the help!


No such file or directory ct2ct

There is an issue when the folder is created at line 53: the folder is named month_day and in line 63 there is a lookup for a folder named year_month_day

There is an issue when the folder is created at line 53: the folder is named month_day and in line 63 there is a lookup for a folder named year_month_day

Fixed.

There is an issue when the folder is created at line 53: the folder is named month_day and in line 63 there is a lookup for a folder named year_month_day

Hi @MassimilianoLuca , thanks for pointing it out. To fix this issue, you can simply edit the line 53 to:
if os.path.exists(f"{output_folder}/{scale}/{year}{month}{day}/") == False:
os.mkdir(f"{output_folder}/{scale}/{year}{month}{day}/")
Sorry, our repository now is facing a problem regards its data size. We will fix it as soon as possible.


Option to disable cursor animation when jumping to/from command line

Just tried out Neovide today; it's awesome! I'm really fond of the cursor animation, however I find the jumps to and from the command line quite jarring. For example I have <C-s> mapped to :wa<cr> and this pretty much results in a streak of light across my screen whenever I hit save. (along with other mapped commands)
Cheers!

It might be neat to have a sort of shrink-bubble-pop animation as the cursor disappears.

It might be neat to have a sort of shrink-bubble-pop animation as the cursor disappears.

I like this idea haha. Would require some pretty significant refactoring to work though as the current system assumes a single cursor.

I like this idea haha. Would require some pretty significant refactoring to work though as the current system assumes a single cursor.


Server crashes when using async with ServerStreaming

I'm trying to implement a server, which spawns an asynchronous action.
I then call send function from within this async block.
This results in the server crashing with various errors:
E0511 07:02:51.828270196   10996 server.cc:201]              assertion failed: queue.Pop() == nullptr
Aborted (core dumped)

double free or corruption (out)
Aborted (core dumped)

double free or corruption (!prev)
Segmentation fault (core dumped)

corrupted size vs. prev_size while consolidating
Aborted (core dumped)

The server implementation looks like this:
startSubscriptionHandler ::
  ServerRequest
    'ServerStreaming
    Api.StartStreamRequest
    Api.StreamNotification ->
  IO
    (ServerResponse 'ServerStreaming Api.StreamNotification)
startSubscriptionHandler (ServerWriterRequest _metadata Api.StartStreamRequest send) = do
  reponseCode <- do
    async $
      forM_ [1 :: Word32 .. 1000000] $ \i -> do
        res <- send $ Api.StreamNotification i
        case res of
          Left err -> fail $ "error: " <> show err
          Right () -> pure ()
    pure StatusOk
  pure $ ServerWriterResponse mempty reponseCode mempty

main :: IO ()
main = do
  Api.dataApiServer
    ( Api.DataApi
        startSubscriptionHandler
    )
    defaultServiceOptions

Am I doing something wrong? Or is the use async in this context not supported?

This looks wrong to me. My recollection is that sending the response finishes the interaction -- so you can only send the response after you finish sending the streaming results.

Thank you, this seems to have fixed to problem.


Add Localization Support

I wanted to see how this package could be extended for easy localization. What I did for right now is I just created a bunch of hacks for digit mapping to test and see whether it works. If you look at this, this, and this, what I've done is I've basically added an extra option to the \begin{document} declaration called language, and that basically determines how the section numbers/digits are localized.
But I'm trying to wrap my head around how I can make it easier to localize further. Any thoughts?

I think that what you are possibly hinting at relates to having language-dependent styling. In your case (second file modified), it's for applying it to numbering (digits), but there are other things where the styles could have to depend on language.
For instance,

In French, footnote call references are usually preceded by a thin space

That is

English would do this:  Some sentence.² “Some quote.”² (i.e. footnote calls after punctuations, no space before the raised number)
French would do this: Une phrase¤². « Une citation¤² ». (i.e. footnote call before punctuations, but thin space before the raised number, here represented as a ¤ symbol for showing where it should be).

For a simpler example of language-dependent styles: in French usually, paragraphs are always indented, even after section titles.

Moreover, styles can be overridden -- I am not satisfied yet with the current API, but my idea was to have defaults in Lua (where you made a modification) and a way for providing user-overrides (e.g. currently with \style:define pseudo-commands, but I am even less sure the API is sound there).
So rather than imposing it to the class, my current line of thinking is rather than this should be some kind of on-demand styling, triggering a variation of the styles (at least those where this is needed)
For reference, see also https://github.com/Omikhleia/markdown.sile/discussions/34#discussioncomment-4246305 (a discussion on styling Markdown, but it relates to the resilient.book class eventually). Pushing the idea further in the direction drafted there, I think one possibility is to extract style definition wholly from class and packages in some easier format (e.g. raw Lua tables, or in SIL syntax, XML or even YAML/JSON), and let end-users run with their "customized" version.

(Huh, that was a the long answer)
For now, a simpler way with the current (unsatisfying to my eyes) SIL-style API would be not to modify the class but just re-define the styles you need in your document. E.g. (untested)
\begin[class=resilient.book]{document}
\style:redefine[name=sectioning-chapter, as=_original-chapter, inherit=true]{
  \sectioning[display=Gonm]
}% and likewise for all styles needing it
% That is, just redefining them to inherit from the original styles, but overriding some of their properties
...
\end{document}

I think that what you are possibly hinting at relates to having language-dependent styling. In your case (second file modified), it's for applying it to numbering (digits), but there are other things where the styles could have to depend on language.
For instance,
* In French, footnote call references are _usually_ preceded by a thin space

  * That is

    * English would do this:  Some sentence.² “Some quote.”² (i.e. footnote calls after punctuations, no space before the raised number)
    * French would do this: Une phrase¤². « Une citation¤² ». (i.e. footnote call before punctuations, but thin space before the raised number, here represented as a ¤ symbol for showing where it should be).

* For a simpler example of language-dependent styles: in French usually, paragraphs are always indented, even after section titles.

Moreover, styles can be overridden -- I am not satisfied yet with the current API, but my idea was to have defaults in Lua (where you made a modification) and a way for providing user-overrides (e.g. currently with \style:define pseudo-commands, but I am even less sure the API is sound there).
So rather than imposing it to the class, my current line of thinking is rather than this should be some kind of on-demand styling, triggering a variation of the styles (at least those where this is needed)
For reference, see also Omikhleia/markdown.sile#34 (comment) (a discussion on styling Markdown, but it relates to the resilient.book class eventually). Pushing the idea further in the direction drafted there, I think one possibility is to extract style definition wholly from class and packages in some easier format (e.g. raw Lua tables, or in SIL syntax, XML or even YAML/JSON), and let end-users run with their "customized" version.

This makes sense, I'm gonna play around with some of these options (in particular the SIL syntax or YAML). I also noticed that we could also leverage fluent-lua to define language based configurations for things like Table of Contents, Chapter, which I think would be necessary to accommodate languages that aren't necessarily like English. I just wanted to get your thoughts on that as well.

I just wanted to get your thoughts on that as well.

On tables of contents: I do not think the \tableofcontents command (or friends such as the \fancytableofcontents from my fancytoc package) shall have a title. The user should provide it (as an explicit unnumbered chapter, or anything he/she wants there, e.g. some text and pictures possibly). In real books, remembering old days with LaTeX, I used to keep changing whatever the default title was. Why "Table of Contents"? Could be "Contents", "Summary", "Topics" etc. -- Or even nothing at all, after all the readers know how to recognize a ToC when they see one).

On "Chapter", "Appendix", etc. my current implement pre-dates the introduction of fluent in SILE. I have not made my mind yet about it.

When I proposed adding all usual "terms" from LaTeX/Babel, only a subset was included, so basically the current default i18n files in SILE do not have "part", "appendix", "glossary", etc.
It's not a secret that I am not fond of what it does and how it does it. I might change my mind when I revisit and rework the "resilient" styling paradigm (so that rework is still lagging behind schedule).

It's not a secret that I am not fond of what it does and how it does it. I might change my mind when I revisit and rework the "resilient" styling paradigm (though that rework is still lagging behind schedule).

Are you not fond of your current implementation or fluent-lua's implementation. Also, I'd like to help either reworking the "resilient" styling paradigm or at least going with my own implementation of this. What's a good place to start?

Are you not fond of your current implementation or fluent-lua's implementation.

That comment applied to fluent-lua as currently implemented and used in SILE core distribution.
(But I am not that fond either of my SIL commands for styles (\style:define etc.) and their use at random places in a document. See below)

Also, I'd like to help either reworking the "resilient" styling paradigm or at least going with my own implementation of this. What's a good place to start?

You might read the documentation of the resilient.styles and resilient.sectioning packages - from their sources, or including them back in the manual (I commented them out for now) ; or in an older PDF from the SILE 0.12 version of these packages ; it might be (very) slightly outdated, but at least you have their the concepts: what I had in mind and how it started.
I globally like the styling paradigm. I used it a lot in my first published book made with SILE, and I am using it too my (work-in-progress) next book project. What I don't like very much is the SIL-syntax for declaring/updating styles.

What exactly don't you like about fluent-lua?
Also I'll play around with this and see where I go with this. If I come across anything I'll let you know. How exactly do I change the title for the "Table of Contents"?

What exactly don't you like about fluent-lua?

I don't think it solves any problem that cannot be solved in a better way.

How exactly do I change the title for the "Table of Contents"?

The resilient classes use their own table of contents implementation, which doesn't have such a title (as opposed to SILE's core distribution implementation). So there's nothing to change here, you add whatever you want in your document (as I tried to explain above, but perhaps I am misunderstanding the question?)

Also how exactly would I change display of the page number (for example, how would i do the digit mapping for page numbers?)

Er. If not mistaken, I understand you are might be quite new to SILE, and I might be losing you with the specifics of the resilient classes, the styling paradigm, etc.
So let's start with the basics - pure plain SILE, nothing fancy:
\begin{document}
\set-counter[id=folio, display=roman]% switch folio numbering to roman numbers

Lorem ipsum

\end{document}

With styles, one wouldn't have to call that command, but instead would declare something appropriate in the corresponding style specification. The envisioned way will be made clearer when I tackle working on #12. (TBH, folio numbering will not be the most exciting part, I'm afraid :smile_cat: )

I don't think it solves any problem that cannot be solved in a better way.

Interesting, could you give me an example as to how it would be solved in a better way

The resilient classes use their own table of contents implementation, which doesn't have such a title (as opposed to SILE's core distribution implementation). So there's nothing to change here, you add whatever you want in your document (as I tried to explain above, but perhaps I am misunderstanding the question?)

You're right, that's exactly what I was getting at: how would you go about doing that using resilient classes?

Er. If not mistaken, I understand you are might be quite new to SILE, and I might be losing you with the specifics of the resilient classes, the styling paradigm, etc.

You're right, I'm relatively new to SILE (I've read much of the documentation and I'm getting a better feel for how SILE works), but I'm familiar with how to change the folio counters, which I've gotten working. I was more asking along the lines of how that would work using resilient styles for consistency purposes, but I figured you might be getting around to working on that as well.

was more asking along the lines of how that would work using resilient styles for consistency purposes,

In my "in-progess" resilient v2 styling attempt, I have a "style definition file" beside my master document (so if my master document is, say, toto.sil, then I have a file toto.rsty, which I can edit and tune separately). Regarding folio (page numbers) styling, here is my (current) setup:
folio-even:
  # called by classes.resilient.book
  inherit: "folio-base"
  style:
    paragraph:
      align: "left"

folio-odd:
  # called by classes.resilient.book
  inherit: "folio-base"
  style:
    paragraph:
      align: "right"

folio-base:
  style:
    font:
      size: "0.9em"
      features: "+onum"
    numbering:
      display: "arabic"

This is experimental might fall short when I introduce frontmatter / mainmatter / backmatter distinctions, but I am not yet there.

Is this loaded into your class through lyaml?

I'll have a look at it and provide my feedback on it when I get a chance to. I've been flooded with work so it might take me a little while.

How exactly do I change the title for the "Table of Contents"?

This was my dirty and quick hack:
\ftl{tableofcontents-title = Nội Dung}
\tableofcontents

The outcome looks quite decent.

Recap before closing this issue.
I am going to proceed merging #13 this week-end or the next, so as to proceed to other features depending on it, in order to reach the expected release milestone
After it's done:

In Resilient, \tableofcontents does not generate title (by design).
In Resilient, styles in YAML cover both pure styling (font styles, indents, etc.) and prefixed/suffixed texts (such as "Chapter N"). So fluent / ftl are useless in that scheme and won't be used. Everything is done via a separate style file, so that projects can tune their own styles distinctively at convenience (and keep their styling file, archive it alongside the content, etc.).
In Resilient, number styles also cover the possibility to use other numbering system, etc.

Feedback on the system will still be welcome, obviously but in dedicated issues once it's made available.


Add a link to Wikipedia for Luhn's Algorithm

What do you want to see?
Add a link to the Wikipedia page for Luhn's algorithm on the page. The URL is https://en.wikipedia.org/wiki/Luhn_algorithm
Code of Conduct

[X] I agree to follow this project's Code of Conduct

HI, can I work on this issue? Thanks!

Could I work on this?

Can I work on this?

Can I work on this...?

i want to do this

Hello can I do this?

i want to work on this

can i work on this

can i work on this?

hey can I work on this?


S3 Permissions

I am currently using S3 with gaufertte to store files. I was looking into this bundle but noticed it does not support the ability to set the read permissions of a file on a S3.
Is there a workaround for this?

Is there a workaround for this?

I don't think there is one yet. I'll label it as feature request and try to implement it in v2, which is currently under development. Thanks for the suggestion.


WIP: Added basic structure for load testing

ECOM-3483

Added basic structure for load testing of credential service
Added load tests for list endpoint of credential service.

@kashifch Please review this.

I think @benpatterson could help with the review

@jimabramson Can you please review this.

@tasawernawaz it seems like all the JWT-related stuff could be moved to a shared place and reused in both credentials & programs, as well as various other packages, but this is optional.

@jimabramson At this point I am not sure, mainly because we have different keys for programs as well as for credentials.

@tasawernawaz the basic structure looks on track.  I'm mainly interested in where you intend to take things from here.

:+1:

:+1:


APM for 'serialize command' step

Opening up this PR for discussion, not meant to be the final version.
What?
This changes creates three new APM events, related to the serializeCommand operation.
Why?
The CommandXEvents are very good for high level visibility but lack details. As an engineer that has developed with node.js + MongoDB for a while, I've usually found myself wanting to separate:

MongoDB server/DB side command execution time
MongoDB client side command execution time delays, especially related to serialization

Item #2 can be particularly useful to identify either updates/inserts of large document sets (which are bad because serialization time blocks the event loop) and large result sets from queries (not in this PR).
This is what the events look like when logged to the console:
succeeded CommandSucceededEvent {
  duration: 1.702143,
  commandName: 'endSessions',
  reply: { ok: 1 },
  requestId: 4,
  connectionId: 'localhost:27017',
  metadata:
   CommandMetadata {
     bsonSerialization:
      BSONSerializationData {
        type: 'bsonSerialization',
        buffersLength: 141,
        duration: 0.091494 } } }

In this case as a developer I could consider the duration and buffersLength to be fairly high. This can then be either sent to metrics, logs and even tracing services with the appropriate context tracking.
Also, if this is useful I'd like to explore doing the same thing for "deserialization time", aiming to expose timing and size metrics about query results, e.g. knowing about find queries that returns > 1MB of size and block the event loop while deserializing such a big chunk of data.

@mbroadst @daprahamian any feedback on this one?

Hey @dschenkelman,
Sorry this is taking so long. We're currently swamped with feature requirements, so it will be a few more weeks before we can circle back to this.
-Dan

Hey @dschenkelman,
Sorry this is taking so long. We're currently swamped with feature requirements, so it will be a few more weeks before we can circle back to this.
-Dan

I understand. Any chance that in the meantime I can get a couple of pointers on how to add similar instrumentation but for query result serialization instead of commands?
I'd love to take a stab at that 😄

@daprahamian hey 👋, happy new year! any news on this one? anything I can do to help get it merged?

@dschenkelman I am no longer a maintainer on the project, pinging @mbroadst.

Hey @dschenkelman, sorry the PR was autoclosed! I deleted that branch which is what I think forced the PR shut. If you can rebase against 3.5 or 3.6, then we can see about modifying your code for the 4.0 release. We had a lull there where I was the only developer, but now have a sizeable team and can investigate bigger changes like this again!


Typo in readme fixed

Hi @MoskitoHero, Barley looks promising. I've found one small typo.

Hello @vlado, thank you for this, have a nice day !


chore: remove 'active' from validator status type

https://github.com/ChainSafe/lodestar/pull/7143#pullrequestreview-2360968001

:tada: This PR is included in v1.23.0 :tada:


Allow to add response headers in IHttpResultSerializer

Is your feature request related to a problem?
In some cases i want to set custom http status and add header to response. IHttpResultSerializer allow to modify response status but unable to add header. At this time a workaround is to inject HttpContextAccessor to IHttpResultSerializer implementation and add header on HttpContext.Response. IMO it is a code smell because method GetStatusCode has side effect.
The solution you'd like
Add a method IReadOnlyDictionary<string, StringValues> GetResponseHeaders(IExecutionResult result) to IHttpResultSerializer and add its result to HttpContext.Response.Headers.
Another solution would be to allow to access HttpContext.Response in IHttpResultSerializer.
Product
Hot Chocolate

You can now override the method:
    protected virtual void OnWriteResponseHeaders(
        IQueryResult responseStream,
        FormatInfo format,
        IHeaderDictionary headers)
    {
    }


Update build.md

Update build instructions for Library on Windows
Update build instructions for Example on Windows

FWIW I think we should instead update https://dronecore.gitbooks.io/dronecore-guide/content/en/contributing/build.html and link to it everywhere (ie delete this file)
@julianoes

Ok, we can delete this file. We don't want to have it double I agree.

OK, closing then. @Rjasuja this info is now in the guide. I'll delete this file and redirect the readme.


HomeDeviceQoD r1.2 release review (M4)

Release PR or issue to review
https://github.com/camaraproject/HomeDevicesQoD/pull/70
Review actions
Assign this issue to yourself and follow the below list.
For any review action, review the file(s) in the issue/PR listed above.
Put comments in the above issue or PR if they concern non-changed files/text.
Put a short summary of the main review result here into the review issue.

[x] API definition files (YAML) (check version in info & servers objects)
[x] test file(s) availability
[x] changelog updated
[x] readme updated (enforce correct release number / API version naming)
[x] API readiness checklist(s)

Release actions
Assign this issue to yourself or another RM team member and follow the below list.
When done, tick the box in this issue.

[x] API release tracker available on wiki
[x] Review comments addressed (by Sub Project)
[x] Release PR approved (on behalf of Release Management)
[x] PR merged (by Sub Project codeowner)
[x] Release created within GitHub (by Sub Project codeowner)
[x] Release Tracker updated (with creation date of the release and the release tag link)

Additional comments

Release PR or issue to review
https://github.com/camaraproject/HomeDevicesQoD/pull/70
Review actions
Assign this issue to yourself and follow the below list.
For any review action, review the file(s) in the issue/PR listed above.
Put comments in the above issue or PR if they concern non-changed files/text.
Put a short summary of the main review result here into the review issue.

[x] API definition files (YAML) (check version in info & servers objects)
[ ]  test file(s) availability
[x]  changelog updated
[x]  readme updated (enforce correct release number / API version naming)
[ ]  API readiness checklist(s)

Release actions
Assign this issue to yourself or another RM team member and follow the below list.
When done, tick the box in this issue.

[ ]  API release tracker available on wiki
[ ]  Review comments addressed (by Sub Project)
[ ]  Release PR approved (on behalf of Release Management)
[ ]  PR merged (by Sub Project codeowner)
[ ]  Release created within GitHub (by Sub Project codeowner)
[ ]  Release Tracker updated (with creation date of the release and the release tag link)

Additional comments
PR #70 fixes #69

All tasks done. Thanks to @bigludo7 for the RM review ! This issue can be closed.


Create daki.is-cool.dev.json

Requirements

[x] You have completed your website.
[x] The website is reachable.
[x] The CNAME record doesn't contain https:// or /.
[x] There is sufficient information at the owner field.
[x] There is no NS Records (Enforced as of Sepetember 4th, 2024)

Description

For my discord bot dashboard
Link to Website

https://dakii.micoo.xyz/

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


ng generate component (ViewEncapsulation) missing from import

Bug Report or Feature Request (mark with an x)
- [x ] bug report -> please search issues before submitting

Versions.
@angular/cli: 1.5.0
@angular-devkit/build-optimizer: 0.0.32
@angular-devkit/core: 0.0.20
@angular-devkit/schematics: 0.0.35
@ngtools/json-schema: 1.1.0
@ngtools/webpack: 1.8.0
@schematics/angular: 0.1.1
typescript: 2.4.2
webpack: 3.8.1
Repro steps.
ng g component somename
This adds a line:
encapsulation: ViewEncapsulation.None
but does not add the import
import {Component, OnInit, ViewEncapsulation} from '@angular/core';
instead adds
import {Component, OnInit} from '@angular/core';

Duplicate

This should not have the encapsulation: ViewEncapsulation.None at all by defualt

Can this be disabled? I don't want any of my components to not have encapsulation. I'm not sure why this was added by default, it's a bad practice to disable encapsulation anyway.


The software hangs indefinitely without any information about what's happening

Description of the issue
When trying to submit a proposal through the OZ Foundry upgrade plugin, forge hangs indefinitely trying to deploy a contract, after looking over what was being executed it seems, that it hangs with the defender-deploy-client-cli as shown in the screenshot below

Code used for upgradeProposal
// solhint-disable-next-line no-console
contract Upgrade0000 is Script {
    function run() public {
        Options memory opts;
        // Set a Random salt
        opts.defender.salt = bytes32(uint256(0x12345678)); // abritrary salt
        ProposeUpgradeResponse memory response = Defender.proposeUpgrade(
            0xXXXX, "RegistryAccess.sol", opts
        );
        console.log("Proposal id", response.proposalId);
        console.log("Url", response.url);
    }
}

Setup
-> foundry.toml updated according to upgrades requirement
-> .env updated with Defender Key, secret and network ("sepolia")
Command run
forge clean && forge script Upgrade0000 --rpc-url https://ethereum-sepolia.publicnode.com -vvv

Trying to run that manually it shows this:

...

Thanks for reporting this.
If your deployment environment for that network is configured in Defender to deploy using a Relayer, it should deploy without hanging.
If your deployment environment is using an EOA or Safe to deploy, then see this note in the docs:

If using an EOA or Safe to deploy, you must submit the pending deployments in Defender while the script is running. The script waits for each deployment to complete before it continues.

As you observed, the Foundry plugin does not give any indication that it is waiting, and unfortunately I'm not aware of any way for it to print a message to indicate this.  If we use console.log in Forge scripts to attempt to print a status message, the message is not visible to the user until after the script completes (which it won't in this scenario!).  If you have any suggestions on how this could be improved, I'd be happy to consider.


Chart improvements

@akloeckner

This doesn't look intuitive to me:

This is an entity that was only in 'off' state for the whole period of shown history. And it looks like it was on  to me, if you don't read into labels.

This doesn't look intuitive to me:

It could be completed in the yAxis' label computation for things like on/off, home/not_home etc.

This doesn't look intuitive to me:

It could be completed in the yAxis' label computation for things like on/off, home/not_home etc.

Can you think of a specific solution?
I wouldn't want to hardcode possible values because, in theory, the value could be any string.
It could be that a line graph is just not a good fit for state values like those. But Chat.js doesn't seem to have a more appropriate chart for this case, I think.

We could hardcore just a few common values. Suppose, e.g., if any in [on, off] is in the state history, then extend labels by the rest in [on, off]. Same for other sets of states, we know of. But it will probably stay incomplete forever.
Maybe there is at least some comprehensive list of binary states. Such as here: https://www.home-assistant.io/integrations/binary_sensor/. Although I think, that specific list is (1) a bit overkill and (2) not complete, I'm missing home e.g...

It could be that a line graph is just not a good fit for state values like those.

I'm also not very happy with the graphs for that in HA itself. Short changes can only be seen in the tooltip, if at all...

We could hardcore just a few common values. Suppose, e.g., if any in [on, off] is in the state history, then extend labels by the rest in [on, off]. Same for other sets of states, we know of. But it will probably stay incomplete forever.

Handling most common states like that would possibly be worth it although I'm usually not a fan of such half-baked solutions. States can be anything really, for example for media devices it can be "on, off, playing, paused, unavailable". For Sun state it can be "Below/Above horizon", and the list goes on...

It could be that a line graph is just not a good fit for state values like those.

I'm also not very happy with the graphs for that in HA itself. Short changes can only be seen in the tooltip, if at all...

That's true. It's not ideal for things like door sensors where it can be "on" only for a couple of seconds a day...
I won't do anyting about that issue then. Current solution works, even if confusing sometimes.

In case you missed my edit: Maybe a good and simple solution for your specific problem would be to have the single y label be centred vertically in the graph? Not sure how to do it though.

In case you missed my edit: Maybe a good and simple solution for your specific problem would be to have the single y label be centred vertically in the graph? Not sure how to do it though.

Could do that by adding an empty label both before and after existing label, if only one present.
But I think I will actually add handling for the on/off case at least. It's the most important one really.

Hmm. I'm not familiar with code comments as it seems. I put one on your last commit.

I just opened #294. If you can think of a generic solution to add those directives, maybe you can incorporate the fix here...

And why not add an else branch to include the bogus two empty labels before and after? You'd have to move the sort operation up then, too.

Did as you suggested.

BTW. There seems to be no need to provide yAxes.labels when type is linear. Chart.js figures out the scale automatically from available data. Do you remember if you saw some case when it's needed or you just did it because it seemed to make sense @akloeckner ?

I believe, I did it, because I didn't know how to conditionally add the field to the struct only for categorial data.

What a great effort of both of you guys! Well done!

I'm ready with the changes. Feel free to look through and point out if anything looks wrong.
One thing that I'm not certain about is changing the position of the close button for all popups. I haven't tested all those popup cases apart from the history popup itself.

I was also looking into supporting multiple entities in one chart, but that is proving a bit difficult, even with same-unit entities.

I was also looking into supporting multiple entities in one chart, but that is proving a bit difficult, even with same-unit entities.

Yes... The x labels are the problem. The datasets can be overridden. But that is some bigger effort, I believe.

Feel free to look through and point out if anything looks wrong.

I'll check again in the coming days. But I think I'm up to date and I haven't seen anything disturbing so far.

:+1:


find-file a directory into a buffer rather than throwing an error

Modify find-file so that when given a directory, it loads the file and directory names contained in it into a buffer for reading rather than throwing an error.

This code went into Med, so I am closing this pull request.  Thank you.


Consider exporting Message.settled field

In addition, we should consider removing Message.SendSettled, making that an optional property on Sender.Send().
Update related documentation.

At present, we don't export this field and state that you MUST call one of the message settlement APIs IFF the sender isn't in SSMSettled.  This simplifies the pattern for receiving messages, i.e. our settlement APIs perform this check on their behalf.  However, it prevents callers from being able to write this code themselves which could be considered "magic".


Use JWK for signing authentication token

FEATURE REQUEST

Is there an open issue addressing this request? If it does, please add a "+1" reaction to the
existing issue, otherwise proceed to step 2.

Describe the feature you are requesting, as well as the possible use case(s) for it.

Currently authentication tokens are simply being signed using a secret, tokens should be possible to sign using JWK, that would add to flexibility to chose between different algorithms for signing and it can make authorization tokens exchanged with other applications, with exchange of JWK issued authorization token can be verified in other applications.

Indicate the importance of this feature to you (must-have, should-have, nice-to-have).

must-have

@mteodor  There is similar proposal https://github.com/mainflux/mainflux/issues/1672 ,

Closing in favor of #1672.


Migration not found on migrate:refresh with --path option

I am getting "Migration not found" when running migrate:refresh with --path option.
Seems that the script tries to rollback all migrations, not only specified ones.
php artisan migrate:refresh --path=packages/package/src/database/migrations
Migration not found: 2016_06_01_000005_create_oauth_personal_access_clients_table
Migration not found: 2016_06_01_000004_create_oauth_clients_table
Migration not found: 2016_06_01_000003_create_oauth_refresh_tokens_table
Migration not found: 2016_06_01_000002_create_oauth_access_tokens_table
Migration not found: 2016_06_01_000001_create_oauth_auth_codes_table
Migration not found: 2014_10_12_100000_create_password_resets_table
Migration not found: 2014_10_12_000000_create_users_table

The final result is actually fine. It refreshes the specified folder migrations.

i'm getting this same error, when trying to drop a column

If you run a migration from a path and then; run php artisan migrate:rollback or something which calls Migrator::rollbackMigrations method like migrate:refresh; the same error will be noted.
e.g.:
❯ art migrate --path=database/migrations/foo
❯ art migrate:rollback
Migration not found: 2018_10_23_104449_foo

Storing the path of the migration files can change this behavior. But is this what Laravel should do?

@dorin-niscu you need to use
php artisan migrate:refresh --path=vendor/laravel/passport/database/migrations

@erdemkeren you didn't specify the path with the rollback command.

@driesvints Indeed. I was talking about persisting the path of the migration files to migrations table.

@erdemkeren that's not something we're going to change. If the location of the migrations would change you'd have to update the values in the database as well.

Hello everyone !
The post is a year old, but I found a solution with your help.
To remove this error, go to your DB and drop the column.
after you have to make a
php artisan migrete

You must delete the content of the migration table in the database


Bad meta description in docsite

If you paste a link into e.g. Teams, or if you look at search results, it displays the page title and then a description, like so:

The issue is that it says "Description will go into a meta tag in" - that comes from index.tsx:
<Layout
title={Hello from ${siteConfig.title}}
description="Description will go into a meta tag in ">
Note that the "" part isn't displayed because it's an empty HTML tag.

Thank you @LeviAdamMiller


CI and build/test/dist tooling

So it appears we've been added to The GitHub Actions Beta program. This has the potential to better serve multi-arch CI than current travis/appveyor integrations. Any thoughts? I've set up a demo action that simply runs cmake/make with defaults.

Sounds promising. I'm in favor of anything that can make the CI process simpler and more transparent. Perhaps the next step is to extend the demo action to run one or more of the API tests in the tests  folder to see how the process works before moving on to running the regression tests which seem to have more moving parts to it.


feat: signing for tags

I haven't been able to get serde to work with the enums.
Added unit tests.
Signed-off-by: Richard Zak<EMAIL_ADDRESS>

@rjzak I moved the JWS types to a new crate here #39 .

Pushed https://github.com/profianinc/drawbridge/pull/38/commits/69727e146c1835a6c2953aaf8f5d4a872ae9e2e1, which should increase the test coverage a bit and show that data model does not seem to be correct.
First, you need to rebase the branch on latest main - some type definitions changed, namely the hash format.
The payload field within JsonWebSignature enum must either be parsed from base64-encoded string to an Entry or treated as opaque vector of bytes. I do not know what the required semantics here are, but if you do want to parse it - you should implement a Deserialize for some wrapper type around Entry to deserialize it from base64-encoded string. I changed it to Vec<u8> for the precise reason that current implementation does not do any attempts to decode the actual entry.
Please start with rebasing the PR and making sure that the entry format is consistent with latest main

I fixed the accidental removal of Roman's tests, adapted new Jws and related data-types, but still can't get the last test signed::tests::tag_value_signatures_from_request  to validate. I think the internal Jws structure validates the signature, which is likely incorrect, and I'm not sure how to coerce Rust to reveal what it should be (but did figure out how to overflow the stack).


"Barcea" and "Podoleni" are listed in the drop-down menu

Description:
"Barcea" and "Podoleni" are listed in the drop-down menu alongside its corresponding county and commune, according to the law
Precondition:
The website is up an running.

Step 3
Type  again "Barcea" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Barcea, GALAȚI (Barcea)".

Step 2
Press "x" button.

Expected results
The location was deleted from the search bar.


log warning: failed to decode

Any idea what's this warning is?    "msg":"failed to decode", "reason":"consumer_protocol_version",
I see about 2 warns/min similar this one:
{
   "level":"warn",
   "ts":1513803928.0648108,
   "msg":"failed to decode",
   "type":"burrow-web",
   "coordinator":"consumer",
   "class":"kafka",
   "name":"myconsumer",
   "offset_topic":"__consumer_offsets",
   "offset_partition":5,
   "offset_offset":4897430226,
   "message_type":"metadata",
   "group":"consumer_group_etl3",
   "protocol_type":"consumer",
   "generation":283366,
   "protocol":"range",
   "leader":"consumer_group_etl3-0661bc66-73aa-4243-a84e-239dc8a8d51b",
   "reason":"consumer_protocol_version",
   "stream":"stdout",
   "application":"kafka"
}

I am running burrow commit#: https://github.com/linkedin/Burrow/commit/3a4372f5c80ee8426dc85cf3edd1cea735dba119
burrow configuration:
[zookeeper]
servers=["zk-0:2181","zk-1:2181","zk-2:2181"]
timeout=6
root-path="/burrow/notifier"

[client-profile.myburrow_client]
kafka-version="<IP_ADDRESS>"
client-id="burrow_client"

[httpserver.mylistener]
address=":8000"
timeout=300

[storage.mystorage]
class-name="inmemory"

[cluster.cluster1]
class-name="kafka"
servers=["kafka-0:9092","kafka-1:9092","kafka-2:9092","kafka-3:9092","kafka-4:9092"]

[consumer.myconsumer]
class-name="kafka"
cluster="cluster1"
servers=["kafka-0:9092","kafka-1:9092","kafka-2:9092","kafka-3:9092","kafka-4:9092"]
client-profile="myburrow_client"
group-blacklist="^(console-consumer-).*$"

burrow is still working and reporting metrics but this warn is logged continuously as approx. 2 times/min

The failed to decode warning is because Burrow was not able to properly decode a message in the __consumer_offsets topic. The rest of those fields give you information about which message it was, and how far it got in decoding. So offset<PHONE_NUMBER> in __consumer_offsets:5 had a problem. That's a group metadata message, and the reason field indicates that the consumer_protocol_version either wasn't an int16, or it wasn't 0 (which is the only version that Burrow understands).
Now, as far as why, this could be a couple things (since you're seeing it repeatedly, and not as a one-off). There may be a new consumer_protocol version from open source Kafka that we haven't picked up yet. That will require writing some code and tests to detect it. Or it could be you have a consumer that's using something custom.


[Task]: Use published resources for app ImportExport across all the flows - Git, FileExport, Templates

Is there an existing issue for this?

[X] I have searched the existing issues

SubTasks

[ ] https://github.com/appsmithorg/appsmith/issues/22434

No response

We're going to implement this https://github.com/appsmithorg/appsmith/issues/24313
Closing this issue.


Custom darkstone vengence tools

Add an alternative recipe for darkstone tools that would give looting 5 and vengence 2 by replacing the sticks with undead sticks (the ones from the evilcraft tree), replacing the darkstone with diamonds and 3 blood shards to empty spaces in the top two rows.

I'm pretty sure the vengeance pickaxe, in recent versions, specifically prevents you from removing the vengeance enchantment. Do we know if we can do the same thing with other tools?

I don't think so but if we can't, we can give it Fortune 3 instead of 5.


Make GPU CI tests preemptible on Beaker

As of https://github.com/allenai/beaker-run-action/pull/37 this should work seamlessly, and will make us better Beaker citizens.

I don't understand how this can happen, but it seems this makes GPU tests fail?

Ugh, this is a bug. I'm going to close until I fix this on beaker-run-action.


Result of external vulnerability audit

DO NOT PROVIDE ANY DETAILS HERE. Please privately report to https://github.com/louislam/uptime-kuma/security/advisories/new.
Why need this issue? It is because GitHub Advisory do not send a notification to @louislam, it is a workaround to do so.
Your GitHub Advisory URL: https://github.com/louislam/uptime-kuma/security/advisories/GHSA-v7cj-r4v4-rjg7

Thank you very, very much!


Remove scheduler and queue apps from research manifest

Context
We don't run these for the research app as we can manually load registers when we need them.
Changes proposed in this pull request
Remove scheduler and queue apps from research manifest.
Guidance to review

@michaelabenyohai they are referenced in the deploy script https://github.com/openregister/registers-frontend/blob/539e4521ee96b001fcdbc40de888e9ad8699e101/bin/deploy#L5

@dankmitchell I hadn't realised. Do we use this script when deploying to the research environment though? I thought it was only used by Travis?

That's true actually. Manual deploys will be fine


Allow space to be a shortcut for Peek or toggle Preview

Description of the new feature / enhancement
Just to provide parity with macOS finder and because I don't think there's a use of space in Windows Explorer allow use to use  space  as the shortcut for Peek
Scenario when this would be used?
When I am on explorer and I have a file selected.
Supporting information
No response

/dup #26143


Fix the ios-webview.js test

It was broken because the test WebView app automatically adds
"http://" to the address bar when it is selected. The ios-webiew.js
test tried to type "http://www.google.com" into the address bar,
resulting in trying go to "http://http://www.google.com" which caused
the test to fail.
Fix:  Type "www.google.com" into the address bar instead of
"http://www.google.com"

Awesome, thanks :)