The website https://unpkg.com/ does not work with cookies disabled

The website https://unpkg.com/ does not work with cookies disabled, despite none of the functions of the website requiring the use of cookies. It it just a wall of text.
Steps to reproduce
Disable cookies and visit https://unpkg.com/
Expected outcome
Something is shown.
Actual outcome
The page is blank.

This is what the console shows:
react-dom.production.min.js:117 DOMException: Failed to read the 'localStorage' property from 'Window': Access is denied for this document.
    at ea (https://unpkg.com/:14:504)
    at Td (https://unpkg.com/react-dom@16.8.6/umd/react-dom.production.min.js:82:11)
    at hi (https://unpkg.com/react-dom@16.8.6/umd/react-dom.production.min.js:102:385)
    at Qg (https://unpkg.com/react-dom@16.8.6/umd/react-dom.production.min.js:144:293)
    at Rg (https://unpkg.com/react-dom@16.8.6/umd/react-dom.production.min.js:145:168)
    at Sc (https://unpkg.com/react-dom@16.8.6/umd/react-dom.production.min.js:158:109)
    at Z (https://unpkg.com/react-dom@16.8.6/umd/react-dom.production.min.js:156:492)
    at Kc (https://unpkg.com/react-dom@16.8.6/umd/react-dom.production.min.js:155:69)
    at ya (https://unpkg.com/react-dom@16.8.6/umd/react-dom.production.min.js:153:185)
    at bh (https://unpkg.com/react-dom@16.8.6/umd/react-dom.production.min.js:160:255)
Bg @ react-dom.production.min.js:117
react-dom.production.min.js:157 Uncaught DOMException: Failed to read the 'localStorage' property from 'Window': Access is denied for this document.
    at ea (https://unpkg.com/:14:504)
    at Td (https://unpkg.com/react-dom@16.8.6/umd/react-dom.production.min.js:82:11)
    at hi (https://unpkg.com/react-dom@16.8.6/umd/react-dom.production.min.js:102:385)
    at Qg (https://unpkg.com/react-dom@16.8.6/umd/react-dom.production.min.js:144:293)
    at Rg (https://unpkg.com/react-dom@16.8.6/umd/react-dom.production.min.js:145:168)
    at Sc (https://unpkg.com/react-dom@16.8.6/umd/react-dom.production.min.js:158:109)
    at Z (https://unpkg.com/react-dom@16.8.6/umd/react-dom.production.min.js:156:492)
    at Kc (https://unpkg.com/react-dom@16.8.6/umd/react-dom.production.min.js:155:69)
    at ya (https://unpkg.com/react-dom@16.8.6/umd/react-dom.production.min.js:153:185)
    at bh (https://unpkg.com/react-dom@16.8.6/umd/react-dom.production.min.js:160:255)

Technically, it's the lack of localStorage rather than the lack of cookies that breaks the website. In either case, I see no reason why it would be required to show a few paragraphs of text. At the very least it should show a friendly error message instead of nothing at all.


[ReleaseNotes] add new changes 2.3.0

New lines in the Release Notes file. Do not hesitate if you want to add something, and/or you have a better way to describe a note.
:m:

Done, i removed "... many other enhancements and bug fixes."


Remove feature flag for create

Issue #, if available:
Description of changes:
Remove feature flag for create
Testing (if applicable):
Documentation added/planned (if applicable):
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/lgtm
/approve
/woof


Update the sidebar selectors on navigator

After navigating to a view through the invocation of the navigator.navigateTo() method (or after a direct navigation through the browser url), the corresponding navigation item in the sidebar drawer should be marked as selected.

Will investigate this one aswell before next release. Fix should be in today.

It should now work with any modifications necessarcy

Unfortunately the Upload does not seem to work currently I'll try tomorrow again

Fix is in!

Sorry but uploading Updates and making them available currenty does not seem to be possible


support index null check

issue: #57

Any changes needed in Milvus?

/lgtm

Any changes needed in Milvus?

prove latter.


Updated luhn checksum operation to work with different bases from 2-36.

Updated luhn checksum operation to work with different bases from 2-36 according to Luhn mod N algorithm.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Access denied

I have downloaded repository, compiled using catkin_make , and connected a cheap USB-UART module to check how it works.
when i used command:
$ rosrun serial serial_example -e|/dev/ttyUSB0 9600 TestString
i got:
bash: /dev/ttyUSB0: Permission denied
I belong to dialout group, and don't know what else I can do.
(The same problem appears when i try to check this library on RaspberryPi with ROS.)

check here: https://websistent.com/fix-serial-port-permission-denied-errors-linux/

I'm going to close this because it is old, but please comment and I can reopen if needed.


alternativeAdvertisingName documentation does not specify maximum length

By examining the bootloader code I've found the maximum length is 20 bytes, but the DFU library documentation does not provide this information and the error logs that occur are misleading.
The errors are:
nRFDFU: Error 13: The value's length is invalid.
nRFDFU: Disconnected
nRFDFU: Buttonless service not configured, see: https://devzone.nordicsemi.com/f/nordic-q-a/59881/advertising-rename-feature-not-working/243566#243566. To workaround, disable alternative advertising name.
DFU error occurred (305): Writing to characteristic failed

While it does indicate the error code received was regarding the value's length, the following errors indicate there is a different problem.

Thank you for reporting this issue. I'll fix it in the next release.


Error adding package to unity.

I opened the package manager, and then clicked + > Add from git url... and then pasted this: https://github.com/tom-weiland/RiptideNetworking.git?path=/UnityPackage It then gives me this error: "[Package Manager Window] Error adding package: https://github.com/tom-weiland/RiptideNetworking.git?path=/UnityPackage." please help

Do you have git installed on your computer? I believe it's required for the Unity Package Manager to be able to install packages via git.
If that doesn't help, you may want to ask on the Unity forums. Considering I'm able to install the package with that url and no one else has reported problems, this seems to be more of a Unity issue on your end than anything else.


Question: How to include related entities

I can write a LINQ query like:
Organizations
.Where(organization => organization.Name == "SomeName")
.AsNoTracking()
.Include("PhoneNumbers")
.Include("Addresses")
and the result will include the collections of phone numbers and addresses.
I can't get this to work using the Dynamic LINQ.  How do I get the related entities to load with Dynamic LINQ?

I figured it out.

This can be closed.


Fix nested enumeration for pybind11

For a nested enumeration, pybind11 takes the parent object (e.g., class), but current chimera implementation and pybind11 template doesn't allow that.

Fixed by https://github.com/personalrobotics/chimera/pull/207/files#diff-0e534e89d173e06af38417fa188dd8b3


0.12.2 - Fix #8335 and #8336 - Snowflake lineage and sampling

Describe your changes :
Cherry pick #8335 and #8336
Type of change :

[x] Bug fix
[x] Improvement
[x] New feature
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation

Checklist:

[x] I have read the CONTRIBUTING document.
[ ] I have commented on my code, particularly in hard-to-understand areas.
[ ] I have added tests that prove my fix is effective or that my feature works.
[x] All new and existing tests passed.

Reviewers
Ingestion: @open-metadata/ingestion

added all the changes in https://github.com/open-metadata/OpenMetadata/pull/8338


wip: feat: add initial deno package

i think we want/need to add this to the workspaces in the root package.json too


Allow displaying help images in editor

Include image urls in editor seed via new AssetUrlsHelper

Provide Backbone view to render image

REDMINE-15846

Coverage increased (+0.0006%) to 99.195% when pulling 1619cbf82df3067f3a9f105b5058fbd00c61d795 on tf:editor-help-images into 7d7f973058364ecfcceb95a137d25346d490415f on codevise:master.


Automated cherry pick of #2975: lblistener: 改正监听update

Cherry pick of #2975 on release/2.12.
#2975: lblistener: 改正监听update

Codecov Report

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

@@              Coverage Diff               @@
##             release/2.12   #2976   +/-   ##
==============================================
  Coverage                ?   7.06%
==============================================
  Files                   ?     477
  Lines                   ?   76394
  Branches                ?       0
==============================================
  Hits                    ?    5396
  Misses                  ?   70546
  Partials                ?     452

Flag
Coverage Δ

#aFlag
7.06% <0%> (?)

Impacted Files
Coverage Δ

pkg/compute/models/loadbalancerlisteners.go
1.68% <0%> (ø)

pkg/compute/models/loadbalancerlistenerrules.go
2.03% <0%> (ø)

Continue to review full report at Codecov.

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

/lgtm

/approve


Transaction builder implementation

Closes #17

@luishurtado before testing the code, it'd be great if you can check this implementation out and provide some feedback. Thank you!

Let's discuss!

@luishurtado
All the changes make sense to me, thanks for the feedback!!. A new structure was added also a "raw implementation"  for the TxBuild.
Question. Should we keep the TxBuild module as an abstraction or does the raw implementation works for this specific case?

Let's discuss!

@luishurtado All the changes make sense to me, thanks for the feedback!!. A new structure was added also a "raw implementation" for the TxBuild.
Question. Should we keep the TxBuild as an abstraction or does the raw implementation work for this specific case?

About this, TxBuild is an abstraction layer.

@luishurtado this is ready for a new review. Changes were implemented and tests added!! Thank you


CharacterData does not get reset correctly

When changing the input, the characterData (including data for the actor and items) should get reset to default, but currently some data is retained (e.g. vision etc.).

Fixed in 3.0.1 but not at all elegant.


Custom dataset classes not detected

First of all, I would like to express my appreciation for the work you spent on this project - a fantastic toolkit that helps a lot the productivity for image segmentation tasks.
I am writing this issue not to raise a problem regarding the project, but more about a question I am dealing with.
I want to train one of the models implemented using a custom dataset with five classes (including background); hence I created my own data loader using ade20k as a template.
I did not do too many things, but a few adjustments on the config's hyperparameters and initiated a training - but 2 of the classes are not detected at all. I did check both training/testing dataset for an eventual unbalanced dataset but no problem there.
Do you have any idea about what could be the cause of the problem or where I should start at for a more in-depth investigation?
I tried to follow all indication provided in your readme, such as setting:
   ignore_index = -1

or
self.num_classes = 5

or adjust self.MEAN, self.STD accordingly with the dataset images.
In the end, no matter the number of epochs passed, the last 2 classes keep indicating a 0 mIoU.
        ## Info for epoch 5 ##
         val_loss       : 0.34638
         Pixel_Accuracy : 0.84
         Mean_IoU       : 0.375
         Class_IoU      : {0: 0.83, 1: 0.458, 2: 0.586, 3: 0.0, 4: 0.0}

Still, adding an "extra" class hence 6 classes, I get the following output:
     ## Info for epoch 15 ##
         val_loss       : 0.30863
         Pixel_Accuracy : 0.88
         Mean_IoU       : 0.34599998593330383
         Class_IoU      : {0: 0.879, 1: 0.514, 2: 0.671, 3: 0.0, 4: 0.0, 5: 0.009}

Somehow 3 and 4 indexes are not actually my classes, and 5 seems to be one of them .. and probably the last one finds itself somewhere between 5 and 149.
Am I missing something?
Thank you in advance.

Interestingly, while performing a
label = np.asarray(Image.open(label_path), dtype=np.int32) - 1
np.unique(label)

I get as output:
array([-1,  0,  1,  4,  5], dtype=int32)

which means that my indexes are ordered labels.
It seems to be a rookie mistake, but should I perform any greyscaling of the pictures before?

Jinx - involuntary I did exactly how you recommended.  Thank you very much!
Overall, do you consider that it is better to have a background as a label instead of ignoring it?

If the background is correctly annotated, yes, better to use it I guess.


Rendering bug with class attribute and create_memo derived signal from resource (SSR)

Describe the bug
When using create_memo as a derived signal for a Resource from create_resource (not an issue with create_local_resource) there is a rendering/hydration problem when used in a class attribute.
Leptos Dependencies
leptos = { version = "0.5" }
leptos_axum = { version = "0.5", optional = true }
leptos_meta = { version = "0.5" }
leptos_router = { version = "0.5" }

Expected behavior

"red" class attribute should not be present
button is enabled

Actual behaviour

"red" is present
button is enabled

Screenshots

Example
use leptos::*;

#[component]
pub fn App() -> impl IntoView {
    let (count, set_count) = create_signal(1);

    // Get max count
    let resource = create_resource(|| {}, move |_| async move { Ok::<i32, ()>(2) });
    // Max count or 0
    let max_count = move || {
        resource
            .and_then(|count| *count)
            .and_then(|res| res.ok())
            .unwrap_or(0)
    };

    let disabled = Signal::derive(move || count.get() >= max_count());

    let max_count_memo = create_memo(move |_| {
        let max = max_count();
        // Logs 0 on the server and 2 on the client
        logging::log!("max_count {}", max);
        max
    });

    let disabled_with_memo = Signal::derive(move || count.get() >= max_count_memo.get());

    view! {
        <button on:click=move |_| { set_count.set(2); }>next</button>

        <Transition fallback=||()>
            <Button disabled=disabled />
        </Transition>

        <Transition fallback=||()>
            <Button disabled=disabled_with_memo />
        </Transition>
    }
}

#[component]
fn Button(disabled: Signal<bool>) -> impl IntoView {
    let style = "";
    view! {

        <button
            class=("red", disabled)
            disabled=disabled
            >
            "should be red and disabled only after clicking next"
        </button>
    }
}

An update:
it works fine with:
class=move || (if disabled.get()  { "red" } else { "" }).to_string()

instead of the tuple syntax class=("red", disabled) or class=("red", move || disabled.get())

@gbj similar issue with create_blocking_resource, SsrMode::PartiallyBlocked , {signal or memo using resource (I tried both)}
I wonder if there should be a Resource::derive

I assume you also get the warning that I get:
At src/app.rs:16:14, you are reading a resource in `hydrate` mode outside a <Suspense/> or <Transition/>. This can cause hydration mismatch errors and loses out on a significant performance optimization. To fix this issue, you can either:
1. Wrap the place where you read the resource in a <Suspense/> or <Transition/> component, or
2. Switch to using create_local_resource(), which will wait to load the resource until the app is hydrated on the client side. (This will have worse performance in most cases.)

This bug doesn't really have anything do with rendering, but with the fact that the memo starts looking for context (and therefore for Suspense) in the location it was created in, not read in; the Transition that reads from the memo is acting as if there's never actually a registered resource read, and it's using your provided default value of 0, which is why the class is red.
I'll dig in a little as this is not my intended behavior for the relationship between where a memo is read and its access to context.

Checked and this example seems to work as intended on current main branch (0.7), although some things have been renamed/etc.
#[component]
pub fn App() -> impl IntoView {
    let (count, set_count) = signal(1);

    // Get max count
    let resource =
        Resource::new(|| {}, move |_| async move { Ok::<i32, ()>(2) });
    // Max count or 0
    let max_count =
        move || resource.get().and_then(|count| count.ok()).unwrap_or(0);

    let disabled = Signal::derive(move || count.get() >= max_count());

    let max_count_memo = Memo::new(move |_| {
        let max = max_count();
        // Logs 0 on the server and 2 on the client
        logging::log!("max_count {}", max);
        max
    });

    let disabled_with_memo =
        Signal::derive(move || count.get() >= max_count_memo.get());

    view! {
        <button on:click=move |_| {
            set_count.set(2);
        }>next</button>

        <Transition fallback=|| ()>
            <Button disabled=disabled/>
        </Transition>

        <Transition fallback=|| ()>
            <Button disabled=disabled_with_memo/>
        </Transition>
    }
}

#[component]
fn Button(disabled: Signal<bool>) -> impl IntoView {
    let style = "";
    view! {

        <button class=("red", disabled) disabled=disabled>
            "should be red and disabled only after clicking next"
        </button>
    }
}


[TMP] - Debug Implicit Operator

Placeholder PR for debugging the implicit test that is failing with out-of-memory on GPU

Tried splitting the parametrizeed test out into separate tests for each combination to check if it was something to do with pytest parametrize not cleaning up after each test run. It turns out to still fail like this, so it seems there is some memory being managed in implicit possibly that is not being cleaned up automatically.


Layer editable by anyone

We should create a new type of layer, editable by anyone who has an valid account.
This layer would be called 'public', and the current 'public' type layer should be called 'visible'.

Closed due to inactivity.


Request Enhancement

Hai all, i request add some features like

Asynchronous Validators
Focus and Scrolling
customize error messages
Form submit validator fields Errors return Like Map
onSubmitted focus fields

Same. Would be great if they would add these specially Asynchronous Validators and focus fields.

@nousath Can create separated issue for this features? Thanks


Docs: add note about the grays being "cool"

Closes https://github.com/twbs/bootstrap/issues/27332

@mdo? doesn't seem like a huge compromise to add this wording...but up to you...

Ping @mdo

as i managed to mess up this PR by accidentally deleting the branch...successor: https://github.com/twbs/bootstrap/pull/27824


[Task] Improve readme

@Nikhil-Nandagopal Can you provide a description on what to do ?

@Wittty-Panda this issue has already been assigned. please have a look at our hacktoberfest issues to begin contributing :)

@Nikhil-Nandagopal okay sir.


SAM-3153 Make Hot Spot images not responsive

This makes the Hot Spot questions ready for SAK-31978 and  SAM-3155 to be merged without problems in the Hot Spot questions

Did a different PR modify the same CSS?

But in different places. The main PR makes the images responsive, and this one makes the Hot Spot ones non responsive.


Fixed computation of timestamp

This was an error that impacted most calculations. The database is being rebuilt.
The root issue was that the timestamp of a trace was being computed since midnight, when it should be computed from a specific timestamp available in the trace's json.
Added regression test.
Also improved the ergonomics of the library.

Codecov Report
Attention: Patch coverage is 79.28571% with 29 lines in your changes are missing coverage. Please review.

Project coverage is 72.97%. Comparing base (9aea0bf) to head (40b05bc).

Files
Patch %
Lines

src/fs.rs
77.08%
11 Missing :warning:

src/fs_s3.rs
33.33%
10 Missing :warning:

src/icao_to_trace.rs
94.02%
4 Missing :warning:

src/trace_month.rs
60.00%
4 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #55      +/-   ##
==========================================
+ Coverage   71.73%   72.97%   +1.24%
==========================================
  Files          16       16
  Lines        1033     1051      +18
==========================================
+ Hits          741      767      +26
+ Misses        292      284       -8

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


Invariant Violation with react-native 0.70

Hi! 👋
Firstly, thanks for your work on this project! 🙂
Today I used patch-package to patch<EMAIL_ADDRESS>for the project I'm working on.

Here is the diff that solved my problem:
diff --git a/node_modules/react-native-keyboard-spacer/KeyboardSpacer.js b/node_modules/react-native-keyboard-spacer/KeyboardSpacer.js
index 14560d8..806f044 100644
--- a/node_modules/react-native-keyboard-spacer/KeyboardSpacer.js
+++ b/node_modules/react-native-keyboard-spacer/KeyboardSpacer.js
@@ -8,11 +8,12 @@ import {
   LayoutAnimation,
   View,
   Dimensions,
-  ViewPropTypes,
   Platform,
   StyleSheet
 } from 'react-native';

+import {ViewPropTypes} from 'deprecated-react-native-prop-types';
+
 const styles = StyleSheet.create({
   container: {
     left: 0,

This issue body was partially generated by patch-package.

We are also facing this issue. A fix would be really great :)


[ Fabric ] Add peer couchdb volume in case of DR

Signed-off-by: abevers<EMAIL_ADDRESS>Changelog

Update peer CouchDB to use a volume instead of in-memory for its files

To be reviewed by
@jagpreetsinghsasan @lakshyakumar
Linked issue
#1132 - see issue for more info on approach for this story, major reworks were deemed out of scope and bug(s) were created

Currently working on reworking the PVC to be used properly. Will update PR soon


Add TextDocument.md5

This allows users to use -P:semanticdb:text:off while ensuring that
SemanticDB files are in sync with file contents on disk.
Fixes #1593.

This opens the question whether we should disable TextDocument.text by default. The MD5 fingerprint is sufficient for Scalafix to replace .text.

I updated the PR to let metac use text:off by default and updated the spec to document text, uri and md5 separately.

The guide.md is not incorrect since the examples for metac Test.scala assume text:on. Should we?

update the metap output for the case when text:off, or
make text:on by default for the metac cli, or
use -P:semanticdb:text:on in guide.md

Let's go for the first option, i.e. update the metap output for the case when text:off.

I propose we handle the guide update in a separate ticket to get some executable code out of the door asap.


Update stable version in repo from 0.0.0 to 0.0.3

Think about adding version 0.0.3 to stable repository.
Version: stable=0.0.0 (0 days old) => latest=0.0.3 (526 days old)
Installs: stable=0 (0%), latest=4 (100%), total=4
Click to edit

Pull request done


[Feature]: Comply to Semantic Versioning

Is there an existing issue already for this feature request/idea?

[X] I have searched for an existing issue, and could not find anything. I believe this is a new feature request to be evaluated.

What problem is this feature going to solve? Why should it be added?
Currently container updates are often tagged as minor (something.x.something, where X is the increase), even though there are changes that technically break compatibility with the helm chart.
This is also way more compatible with automatic processes  for downstreams and gitops users.
Describe the solution you'd like
It would be nice if the containers/software starts to comply to semver, to correctly tag breaking changes as a major version increase. This correctly gets detected by nearly all versioning software as (potentially) breaking.
Describe alternatives you've considered
Continuing the status quo
Additional context
No response
Backport?
N/A
Code of Conduct

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

Hi @Ornias1993
This is related to the operand images right? Please open this issue on the related container with some clear examples of which image are you talking about, for now, for us is hard to understand which image you mean since the operator image is semver compliant
Closing for now

@sxd Honestly, when you ask almost half a year later, even I don't know what my original issue was about...


On-Chain NFT implementation

This is an on-chain nft implementation based on the nft implementation in gear-contract-libraries.

Stale PR? Need to fix tests and force reviewers to review 😄
@Akado2009

Stale PR? Need to fix tests and force reviewers to review 😄 @Akado2009

OnChain is ready and has been reviewed I believe by @LouiseMedova and @fluiderson. The tests that are crashing are the nft-marketplace tests :(
As far as I could see the problem is the same in all the tests -- the user performing actions with the auction does not have enough balance. Might try to fix it, but it will take some time.


avn compatibility with current Node.js versions

avn is listed in the Compatibility Testing table as being compatible with Node.js 14, however there are multiple reports listed in avn issues of problems with Node.js > 10.  The avn site has not been updated for over three years.
Should avn still be listed?
Node.js 14 enters end-of-life status on April 30, 2023, so the compatibility table using Node.js 14.15.0 as reference version becomes of limited use if it does not cover currently supported Node.js versions.

@shadowspawn

Actually I only used 14.5.0 as an example three part numeric version rather than a reference version.

I had a feeling that might be the case!

I know a few of the node version managers are not currently being developed and multiple are fairly inactive. I could do a revisit and downgrade the ones which are not working with modern Node.js versions.

If you are able to revisit that would make the list much more useful. From my perspective, I would expect that at least Node.js LTS would be supported, better still though would be all of the versions currently listed in the Node.js Release schedule table, also with Node.js 20 just around the corner.
BTW: Thanks for your related comment in the Cypress issues list 🙂 !

A small update. I started looking into avn. The issue I reproduced is that avn setup fails on on node >10. Once installed (using an old node version) avn does allow changing to new node versions when changing directories.

All of the node version managers other than avn have had commits within the last year.

I am playing about with what to do with avn. I am reluctant to simply delete it, but that is a fallback! How about a notes column in the compatibility table?
https://github.com/shadowspawn/node-version-usage/tree/feature/deprecate-avn#compatibility-testing

Hmm, I wasn't entirely happy with the numbers at the top, and your confusion was understandable.
I moved the example versions out of the title row, dropped to 10 so does not look like LTS related, and put in code block so looks a bit more example-ish. Is it more self-evident now, or do you feel still in need of an explanation?

Utility
simple
leading v
partial
Unix EOL
No EOL
Win EOL
Notes

10.1.2
v10.1.2
10.2

asdf
:white_check_mark:
:white_check_mark:
:x:
:white_check_mark:
:white_check_mark:
:x:

I think you need to say somewhere that these are example Node.js version formats only. Maybe write "e.g." in front of each version, add something into the Notes column or write a text comment immediately after the table ends?
Your suggestion so far is in any case an improvement on the previous presentation! 👍🏻

Maybe write "e.g." in front of each version, add something into the Notes column

For amusement, I already had tried both of those but didn't like the appearance or resulting increased table width.

or write a text comment immediately after the table ends?

I'll stop deluding myself that does not need it, and do that. 😆

avn repo now archived:

This repository has been archived by the owner on Dec 7, 2023. It is now read-only.


[Bug] Result preview pane does not refresh when it does not have focus

Insomnia Version:

Operating System: Windows 10

Details

Click Send on a url, see the response
Click into Header to see the headers which were sent to the client
Change the url to one which will produce a different response
Press Send
See the different headers (good!)
See that the header pane still has focus (great!)
Click on Preview
See that it still has the previous response, and that it hasn't updated
Click Send again while the preview pane has focus
See that it now updates the response

The Preview pane needs to update even when Header, Cookie, Timeline or any of the other tabs are selected.
But please do not fix this by forcing the preview pane to receive focus every time you click send! I want to able to review the headers generated by different requests without having to click on the header tab after each send.
Thank you.

Woah, I can't believe no one has noticed this before. I think this actually has to do with Codemirror not updating when it's not visible. I hope this is an easy one to fix...

Hi, I would like to work on this. :smile:


Two layout error in admin panel.[With solution]

Gitea version (or commit ref): 1.12.0-dev
Git version: 2.11
Operating system: Debian
Database (use [x]):

[ ] PostgreSQL
[ ] MySQL
[ ] MSSQL
[x] SQLite

Can you reproduce the bug at https://try.gitea.io:

[ ] Yes (provide example URL)
[x] No  I have no permission🤣This question was discovered in admin panel
[ ] Not relevant

Log gist:
This issues is a front end issues, don't need logs.

Description

When you adding authentication source in the Authentication Source Page. layout errors when selecting the options SSPI, SSPI options will not disappear after you select another option. And the option "Skip TLS Verify" has the same issue. You'd better try it yourself in this page.
When you adding an OAuth2 authentication source and enable the "Use Custom URLs Instead of Default URLs" the content in the input box will not change with the option.

Screenshots
Issue1

Issue2

Issues causes and solution
Issue1
https://github.com/go-gitea/gitea/blob/062f35109df236a06a00d403d006b90760f9cfac/web_src/js/index.js#L1941
In the above array, the last two items are missing commas. You should add them.
Issue2
https://github.com/go-gitea/gitea/blob/062f35109df236a06a00d403d006b90760f9cfac/web_src/js/index.js#L1910-L1922
If the input box is not null there is no action will do. i had try to clear the input box if the input box is not null, it works and the issus was solved.
This is the screenshot:

I hope some one can fix these two issues. I worry if i pull requests, i can't be approved😭.
By the way I really love this project. hope you can develop better in future❤.

Thanks for your pull request❤, i would like to close this issue.


Schema to data classes

I think it would be nice to have a way of generating the data classes from existing avsc schema files. I'm assuming that's possible.

+1 on the above.
There are valid reasons why you would want to do a "schema first" approach and generate classes from schemas instead of a "code first" approach and generate schemas from classes.
For example: You're consuming avro schemas another team/service published.

Definitely a nice thing to have but not something I'll support in this library. Similarly to my avro4s project there's a scala case class generator maintained seperately. I think @thake is working on something.

@domgomIW @grantatspothero I developed a maven plugin supporting the "schema first" approach using avro4k. Would be glad to get your feedback :)
You can find it at https://github.com/thake/avro4k-compiler


Fix Xcode 13 build error

Short description 📝
This fixes the build on Xcode 13 RC.
The similar issue for PathKit.
xcodeproj/Sources/XcodeProj/Extensions/Path+Extras.swift:26:18: error: value of optional type 'UnsafeMutablePointer<CChar>?' (aka 'Optional<UnsafeMutablePointer<Int8>>') must be unwrapped to a value of type 'UnsafeMutablePointer<CChar>' (aka 'UnsafeMutablePointer<Int8>')
            free(cPattern)
                 ^
xcodeproj/Sources/XcodeProj/Extensions/Path+Extras.swift:26:18: note: coalesce using '??' to provide a default when the optional value contains 'nil'
            free(cPattern)
                 ^
                          ?? <#default value#>
xcodeproj/Sources/XcodeProj/Extensions/Path+Extras.swift:26:18: note: force-unwrap using '!' to abort execution if the optional value contains 'nil'
            free(cPattern)
                 ^
                         !
xcodeproj/Sources/XcodeProj/Objects/Project/PBXObjects.swift:141:13: warning: using '_' to ignore the result of a Void-returning function is redundant
            _ = self.add(object: $0)

Solution 📦
strdup returns an optional. I just add unwrapping for this.
In case of nil, I will return an empty array which looks like a valid solution.
Note: To solve all issue with Xcode you will need also apply fix for PathKit (link above)

Thanks a lot for fixing this @CognitiveDisson 🙏🏼

@all-contributors add @CognitiveDisson for fix

@all-contributors add @CognitiveDisson for bug

@CognitiveDisson I had to revert the PR because it causes the builds to fail.

@pepibumur What was the problem? Can I help solve this?

@CognitiveDisson  PathKit Not updated yet

Updated PathKit was just released with fix. Can we release update too?

probably better like PathKit
guard let cPattern = strdup((self + pattern).string) else {
    fatalError("strdup returned null: Likely out of memory")
}

@pepibumur  @CognitiveDisson


Fix filter component pod in profiler

What this PR does / why we need it:
ClusterProfiler FG enables to run the profiler inside the component pod (virt-api, virt-controller, virt-operator, virt-handler). Currently, to filter these pod, we retrieve the full list of the pods and then search if the name contains virt-api,virt-operator,virt-handler,virt-controller. This can result in failure if other external components match these name(e.g. kubevirt-apiproxy from HCO). In this patch there are two improvements:

use label selector to filter pods that have kubevirt.io label.
use a regex to filter the components.

Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
Release note:

NONE

/cc

@enp0s3 Can you have a look? Thanks

/approve
@fossedihelm Thank you!


Fix contract for FileSourceBlobStore

Greetings, please look at my PR for #2420

Submitter checklist

[x] Recommended: Join WireMock Slack to get any help in #help-contributing or a project-specific channel like #wiremock-java
[x] Recommended: If you participate in Hacktoberfest 2023, make sure you're signed up there and in the WireMock form
[x] The PR request is well described and justified, including the body and the references
[x] The PR title represents the desired changelog entry
[x] The repository's code style is followed (see the contributing guide)
[x] Test coverage that demonstrates that the change works as expected
[x] For new features, there's necessary documentation in this pull request or in a subsequent PR to wiremock.org

Looks okay though I would think about proper diagnostics logs, because now all exceptions would be silently suppressed. Would be also nice to add a human readable PR title for the changelog

Can you tell me how nice it is to make diagnostic logs, maybe there is an example somewhere?

WireMock uses com.github.tomakehurst.wiremock.common.LocalNotifier.notifier for logging purposes, e.g.
notifier().info("Started recording with record spec:\n" + Json.write(spec));

I normally use stock logging frameworks, so not sure how much of debug logging we can make with it

@oleg-nenashev

@oleg-nenashev thanks for the hint, I added it in the next commit+ fixed code style

Thanks!


Infinite loop when trying Pi Pico example host_hid and USB dongle from Logitech K400r

Set up
[Mandatory] Provide details of your setup help us to reproduce the issue as quick as possible

PC OS   : "Raspbian GNU/Linux 10 (buster)"@ Pi Zero W connected with SWD, compilation and gdb on OSX.
Board   : Pi Pico
Firmware: https://github.com/raspberrypi/pico-examples/tree/master/usb/host/host_hid

Describe the bug
When running unmodified example it gets stuck - led stops blinking. There is a USB dongle from Logitech K400r connected via usb-otg adapter.
After some debugging I see that execution loops in lines https://github.com/raspberrypi/tinyusb/blob/pico/src/host/usbh.c#L911-L917 :
  while( p_desc < _usbh_ctrl_buf + desc_cfg->wTotalLength )
  {
    // skip until we see interface descriptor
    if ( TUSB_DESC_INTERFACE != tu_desc_type(p_desc) )
    {
      p_desc = tu_desc_next(p_desc); // skip the descriptor, increase by the descriptor's length
    }else {...}
  }

with p_desc having constant value.

Log:
HID init
HUB init
USBH DEVICE ATTACH
Get 8 byte of Device Descriptor
Control Setup: 80 06 00 01 00 00 08 00
on EP 00 with 8 bytes
on EP 80 with 8 bytes
Control data:
  0000:  12 01 00 02 00 00 00 08                          |........|
on EP 00 with 0 bytes
Port reset
Set Address
Control Setup: 00 05 01 00 00 00 00 00
on EP 00 with 8 bytes
on EP 80 with 0 bytes
Control Setup: 80 06 00 01 00 00 12 00
on EP 00 with 8 bytes
on EP 80 with 8 bytes
Control data:
  0000:  12 01 00 02 00 00 00 08 00 00 00 00 00 00 00 00  |................|
  0010:  00 00                                            |..|
on EP 00 with 0 bytes
Get 9 bytes of Configuration Descriptor
Control Setup: 80 06 00 02 00 00 09 00
on EP 00 with 8 bytes
on EP 80 with 8 bytes
Control data:
  0000:  09 02 54 00 03 01 04 A0 00                       |..T......|
on EP 00 with 0 bytes
Control Setup: 80 06 00 02 00 00 54 00
on EP 00 with 8 bytes
on EP 80 with 8 bytes
Control data:
  0000:  09 02 54 00 03 01 04 A0 00 00 00 00 00 00 00 00  |..T.............|
  0010:  00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00  |................|
  0020:  00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00  |................|
  0030:  00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00  |................|
  0040:  00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00  |................|
  0050:  00 00 00 00                                      |....|
on EP 00 with 0 bytes

The TinyUSB port has now been merged upstream to the main repo, see https://github.com/hathach/tinyusb/pull/598 and other followup PRs. So going forward you need to report problems with the port there, https://github.com/hathach/tinyusb.


change Release Support Policy to local URL in include

The Release Support Policy was showing as an external link in the include on release pages. It is now a local URL.

This change is

Online preview: http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/4c6e871fa478b96f1a6575d4e5310429dc062748/

@taroface Oh, good to know! NP.
On Mon, Feb 8, 2021 at 4:33 PM Ryan Kuo<EMAIL_ADDRESS>wrote:

@taroface commented on this pull request.
In _includes/unsupported-version-soon.md
https://github.com/cockroachdb/docs/pull/9729#discussion_r572388863:

@@ -1,3 +1,3 @@
{{site.data.alerts.callout_danger}}
-Cockroach Labs will stop providing Assistance Support for this version on May 12, 2021. Prior to that date, upgrade to a more recent version to continue receiving support. For more details, see the Release Support Policy.
+Cockroach Labs will stop providing Assistance Support for this version on May 12, 2021. Prior to that date, upgrade to a more recent version to continue receiving support. For more details, see the Release Support Policy.

TFTR! I tried Markdown, which our callouts normally support. But it didn't
render correctly on this one. I think it's because it is being displayed
programmatically for the affected versions.
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/cockroachdb/docs/pull/9729#discussion_r572388863, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ASEMKYW5XMK3SULUIFZK3KTS6BKDJANCNFSM4XJMEVNQ
.

Unfortunately it looks like this change is breaking the link on many other pages. I had thought that it was limited to release notes, but it's not. So YZ's intuition to add the full URL was correct. I'm going to close this out.


Rule idea: calculations-braces-around

Inspired by Sass Guilelines
Top-level numeric calculations should always be wrapped in parentheses. Not only does this requirement dramatically improve readability, it also prevents some edge cases by forcing Sass to evaluate the contents of the parentheses.
// Ok
.foo {
  width: (100% / 3);
}

// not ok
.foo {
  width: 100% / 3;
}

Sounds good, the only thing that I'm worried about is that this kind of rule could have a lot of edge cases when linting.
I did a bit similar stuff to Stylefmt to format code with parens, so I know that it is not easy to get right:
e.g. https://github.com/morishitter/stylefmt/pull/70

Well, yeah...
I just decided to drop all the more-or-less decsent ideas here instead of piling then in a local .txt document :)

calculation-no-missing-surrounding-parentheses?

@jeddy3 here options: "always" and "never" be good here too. Some like add parentheses, some not like.

Sure. I wasn't really following my own advice when I came up with that name suggestion... whoops.

I hate to admit it, but more and more I doubt about this rule. Particularly, after working on operators-no-unspaced and figuring out all edge cases of what SCSS considers an operation.
For exapmle,
a {
  width: (10px +1); // compiles into an operation: `width: 11px;`
  width: 10px +1; // compiles into a list `width: 10px +1`
}

Right now both Sass compilers and our parsing helpers consider the secont as a list indeed. But how do we know if it is intended to be a list? Of cource in the example above the secont decl is obviousely malformed, yet... To be honest, I don't fancy the idea of analysing the semantics of the SCSS code to quess if what is not an operation at the moment is really intended to be one.
Any thoughts?

I hate to admit it, but more and more I doubt this rule will be possible to implement.

I understand what you mean.

Any thoughts?

I feel the same way as David about how it might not be worth the effort in trying to work around the complexities of this type of syntax. My suggestion would be focus on the rules that check SCSS-like language extensions that more closely conform to standard CSS syntax. That way this plugin could be part of a shift towards using SCSS-like processors based on only standard CSS syntax. The alternatively is to put a lot of energy into solving a problem that can be more easily solved another way i.e. only using PostCSS plugins that conform to standard CSS syntax e.g. in this case wrapping those math operations in a calc() function that is reduced down at compile time.

I hate to admit it, but more and more I doubt this rule will be possible to implement. Particularly, after working on operators-no-unspaced and figuring out all those edge cases of what SCSS considers an operation.

That's what I was afraid of. Should we drop this rule @dryoma ?

Let's remove it from the milestones list for now. I'll try to give it a thorough thought later again (or maybe someone else comes up with some good suggestion).


Enable nodeFit filtering before eviction

Node Fit filtering is available in upstream descheduler.
It prevents from evicting pods if prerequisites are not satisfied.

nodeFit feature should also evolved to prevent eviction with LowNodeUtilization strategie to check weither or not the underutilized nodes match the pods selector before evicting them.
an issue is already opened here : https://github.com/kubernetes-sigs/descheduler/issues/886


Support not saving the thumbnail to disk?

@martinmaillard wrote the following in a comment on the roadmap gist

A note about the API: I don't think that get_thumbnail should save the thumbnail on disk, or at least there should be a way to tell it not to.

@martinmaillard Thanks for the input. What did you have in mind? What about putting the raw data from the image engine in thumbnail.data when a flag is sent to get_thumbnail.


$routeProvider deep-copies its route objects before using them, which is probably not necessary

Do you want to request a feature or report a bug?
Bug, I guess ?
What is the current behavior?
$routeProvider deep-copies its route object in the beginning of the when('...', route) call, which is a heavy way of solving #8181 and #9731.
What is the expected behavior?
Handle routes without needing a deep-copy.
What is the motivation / use case for changing the behavior?
I'm trying to plumb the Closure goog.html types in Angular as an alternative to the $sce types. These rely on a private marker that is passed to each instance by builders to ensure other classes can't fake them, and the deep copy breaks that link when passing an instance of that type in templateUrl.
Which versions of Angular, and which browser / OS are affected by this issue? Did this work in previous versions of Angular? Please also test with the latest stable and snapshot (https://code.angularjs.org/snapshot/) versions.
Recent ones ? That was initially fixed in https://github.com/angular/angular.js/commit/b4770582f84f26c8ff7f2320a36a6b0ceff6e6cc .
Other information (e.g. stacktraces, related issues, suggestions how to fix)
I think this issue can be worked around by replacing templateUrl : safeTypeInstance with templateUrl : function(){return safeTypeInstance;}, but that's ugly. Martin Probst asked us to write an issue, saying "deep copying the entire object seems overkill if all you wanted was to (shallow) copy the fields in the angular.extend call.".

I guess a repro would be
var obj = {};
$routeProvider.when('foo/bar', {
  someProp: obj
});
// route change somehow, then:
expect($route.current.someProp_.toBe(obj);

Thanks for the report. The issue seems pretty minor, but if there's an easy solution that fixes it, we can merge it in. Do you want to open a PR for this?

Cool, thanks :) I was a little out of my depth on this one.


remove deleted threads/questions

Issue #, if available:
closes #475
Description of changes:
Today when a thread is deleted the question remains in the database and is not able to be "solved".
This PR fixes the issue by deleting questions from the database when their associated thread is deleted.
client.on(Events.ThreadDelete, async (thread) => {
  const log = (message: string) =>
    console.log(`[client:events.ThreadDelete] ${message}`)
  log('started')
  const timeMessage = '[client:events:ThreadDelete] finished'
  console.time(timeMessage)
  log('deleting question...')
  try {
    await prisma.question.delete({
      where: {
        threadId: thread.id,
      },
      include: {
        participation: true,
        answer: true,
      },
    })
  } catch (error) {
    console.error('Unable to delete question', error)
  }
  console.timeEnd(timeMessage)
})

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

blocked by #479

Closing in favor of https://github.com/aws-amplify/discord-bot/pull/570


Web-browsable IHM dictionary?

It might be easier for newcomers to the dictionary if they could browse it with a web browser in the same way as existing extension dictionaries, e.g. something like http://mmcif.wwpdb.org/dictionaries/mmcif_em.dic/Index/. Is there any way to feed the existing draft dictionary into whatever tool generates those views? Ideally that would be automated and update weekly (or whenever the repository is updated) and the data dumped somewhere accessible, such as pdb-dev.

The I/H methods extension dictionary is now available on the wwPDB mmCIF website. It can also be downloaded from the same site.


IOSP-5535: New prefix RNFetchBlob-custom added

According to doc, user can use RNFetchBlob-file:// prefix to order native to directly upload a file in the filesystem to an upload API.
This change added one more prefix, RNFetchBlob-custom:...://, it has some more features.  The prefix can be specified as below:
RNFetchBlob-custom:<classname>://<params_for_class>

An example is like below:
<EMAIL_ADDRESS>
With the above example, the library will now try to instantiate an instance of classname SMCryptoFile.  This class is expected to implement the RNFetchBlobCustomFile protocol, and the entire string is passed into instance of this class for processing.  The instance will interpret params_for_class. In the above example, the parameters are the decryption key and the file path

There is no need to merge this PR, the consumer will simply update the commit hash in package.json


Revert or rebase queued logins

This causes issue #107 and issue #154
@WizardJesse1 and I reverted commit b3bc5607d1b27dcee7d7e8884122c9aba781c656 on my local copy of master, unedited, both of these issues were not present.
@lare96 @Major- @garyttierney

Okay, let's revert and we'll redo this at another point in time


Add test for running cassandra backup and restore in soak

@mrbrowning In the soak cluster, there are multiple instances of cassandra running. In fact, sdk-cassandra runs on non-default ports. You'll have to update the jobs/*.json cqlsh queries to use the non-default port. (https://github.com/mesosphere/soak-cluster-configurations/blob/master/common/packages/sdk-cassandra.json#L13)


Create directory gui (cpp)

This package will contain every entity related to the application displays.

Done.


Prevent/handle NoNodeAvailableExceptions

As explained in an older forum post, it seems the Transport Client often encounters NoNodeAvailableException when used with Found/Elastic Cloud.
It would be nice if this example did not suffer the issues, either through one of the recommended changes as seen in the linked forum post, or through use of the new HostFailureListener.

Closing this issue and moving the discussion to issue #9.


Remove deprecated :tokenizer option from guide

Hello, Rails Guide maintainers.
Would you review this change?
Overview
The part is already removed from the original guide.
ref: https://github.com/rails/rails/commit/7b679affe0e78107888b48e42c65060df6821fe2#diff-89f8485afc5f198c1b7aa9f6eb023e5d

Thanks for your PR and reference to the original guide. 😸
I've checked and am merging this PR. 👍 ✨

@yasulab
Thanks for your review and merge!


Make graph layout zoom limits constant

Fixes #2642.

You can zoom a tiny bit too far out perhaps.
But otherwise LGTM!

You can zoom a tiny bit too far out perhaps.

Yeah, I increased minimal nodes size from 1px to 3px, should be better now :)
p.s. I didn't wanna increase it even more because of the very wide layout we sometimes get for Processes topology.


Sprite Transform Abnormally

Describe the bug
When creating a block larger than sprite size and use transformation to align center, it has one more square at the top right corner
Reproduction Steps
(Objects)
Wall #
brown
000000000
000000000
000000000
000000000
000000000
000000000
000000000
000000000
000000000
translate:down:2 translate:left:2
(Level)
...
.#.
...
What did you expect would happen?
A brown square (without the top right block)
OS & Browser version, desktop/mobile
Chrome Desktop
Screenshots, example projects, other relevant issues, etc.

Thanks for raising this issue. It would help if you could provide the whole program as your repro case.
It looks to me that your spritesize is 5 or close to it, and that rather than one extra square you are missing quite a number of squares.
So I'm guessing you haven't used any collision layer groups. Check out how to do this here:
https://puzzlescriptnext.polyomino.com/Documentation/collisionlayers.html
What then happens is the background of the next cell to the right and below overwrite part of your sprite. I really need to add a note to the docs about that, but meanwhile just add -- after the background and see how you go.

Thanks! I'm keeping this here to remind me to update the documentation.


How to set the axis upper limit

I am new to c3 and finding it very helpful. Thanks!
I have a bar chart rotated to make a horizontal bar chart. Possible data values are between 0 and 10. I wish to set the bottom axis to display ticks at 0, 1, 2, .. 10
My chart displays ticks from 0 to one more than the max data value.
eg
if the max data is 8, I get ticks from 0 .. 8 plus an unlabelled one at the 9th position (the last one)
if the max data is 10, I get ticks from 0 .. 11 (11 is the last displayed)
What I would like is ticks from 0 .. 10 with 10 being the last tick on the axis.
I have searched the docs and tried lots of options but cant find the config for this.
Thanks!
Murray

C3 version 0.4.18:
D3 version 3:

Hi, you can set max property on axis.  Reference http://c3js.org/reference.html#axis-y-max

Thanks Anish. I did try that.
var chart = c3.generate({
    bindto: '#chart1',
    size : {
        height : 150,
        width  : 300
    },
    data: {
      types: {
        eval_1: 'bar'
      },
      columns: [
         ['eval_1', 3, 8, 6]
      ]
    },

    axis: {
        rotated : true,

        y: {
            max : 10
            ,tick : {
                values : [0,1,2,3,4,5,6,7,8,9,10]
            }
        },

        x : {
            type : 'category',
            categories : ["You", "Person 1", "Person 2"]
        },
    }
});

And without the axis.y.tick object (to remove the axis.y.tick.values array):

Got it! Need to set max to 9 and must have the values array specified or the "10" label is missing from the final tick.
y: {
    max : 9
    ,tick : {
        values : [0,1,2,3,4,5,6,7,8,9,10]
    }
},

Ok. Now I see what that comment about padding meant. I was assuming it was talking about css type padding. This works as per previous image.
y: {
    max : 10,
    padding : 0
},

Thanks!


Feat: react-i18next

add translation for this project.  Need to add a Language dropdown menu as well

I'll do this. Can you assign me? Thanks.

I will assign you. Don't forget to star this repo

I created a Header section and Dropdown for language UI.
This is the PR: #20
Can you check and merge it? After that, I'll implement the react-i18next implemantation.
Thanks.

Nice. I checked. Could you please update the rest part I will merge with the i18next PR

I updated. Thanks.

I updated. Thanks.

If you need help to create the translations (json) files, we can call the same people from the issue #15

Nice @esau-morais Could you please review it.

Thank you @erdemuslu for your PR and thank you @esau-morais  for your review. I merged the PR


Add doc strings to method annotation classes

Method annotation classes in uplink/decorators.py are missing class doc strings. To improve code documentation, we need to add doc strings to the following classes, adhering the Google Style Guide for consistency with the rest of the codebase:

[ ] uplink.decorators.headers
[ ] uplink.decorators.form_url_encode
[ ] uplink.decorators.multipart
[ ] uplink.decorators.json
[ ] uplink.decorators.timeout
[ ] uplink.decorators.args

Hello @prkumar  great project you have here. I could give a hand with this issue if you'd like 👍

Thanks, @FabioRosado! I'll assign this issue to you!

@FabioRosado, feel free to open a PR that references this issue! Thanks for contributing!

I've been playing around with uplink while reading the documentation but I'm unsure as to the     can_be_static = True that can be found on uplink.decorators. How should I describe this in the docstring?

can_be_static = True indicates that the annotation can be a decorator without arguments, like so:
@form_url_encoded  # Notice we don't instantiate `form_url_encoded` directly
@post("/user")
def set_user(self, user): pass

This is required because of an implementation detail. Some decorators we provide can be called with arguments (e.g., headers) and others without (e.g., form_url_encoded). However, they all share some similar decorating logic, which we put in uplink.decorators.MethodAnnotation.__call__. Hence, without call_be_static, decorators that don't accept arguments, like form_url_encoded, would have to be instantiated first so that form_url_encoded.__call__ is the one wrapping the consumer method:
@form_url_encoded()
@post("/user")
def set_user(self, user): pass

Awesome! Thank you for the great explanation 👍


GetIgnoreTemplat doesn't work

Hi
It seems there is a problem with the function GetGitIgnoreTemplate()
I tried it on many repository but it return nothing all the time
I also create a repository with a C gitignore template here : https://github.com/joan-saum/testGitIgnore
but still the function return me nothing.
Is it the good way to know if a repository contain a .gitignore file or should i use something else ?

Hi @joan-saum,
When in doubt, it is a good idea to check the source code:
https://github.com/google/go-github/blob/master/github/github-accessors.go#L7895-L7901
// GetGitignoreTemplate returns the GitignoreTemplate field if it's non-nil, zero value otherwise.
func (r *Repository) GetGitignoreTemplate() string {
	if r == nil || r.GitignoreTemplate == nil {
		return ""
	}
	return *r.GitignoreTemplate
}

What you are seeing here is an accessor that makes it easy to access a field of a returned Repository struct, and this (at least by itself) is probably not what you want.
If you actually want the contents of the .gitignore files in many different repos, then you will want to actually access the contents of the repos themselves.
I don't have time at the moment to put together an example program for you, but it can be done with the GitHub API.
If you are still stuck and would like an example program, please let me know and I will try to whip one up later for you.
Good luck.

Hi @gmlewis and thank you for your answer
Actually, i don't really care about the contents of the .gitignore files. I just want to know if there is a .gitignore file in a specific repository.
I thought GetGitignoreTemplate could help me to do it simply but i think this is not for what i want.
I will use the GitHub API, list the content of the repo and check if there is a .gitignore
Thanks for your time!


[ICON]-Request For Whatsapp

Icon Request

Icon name:Whatsapp

Upvote & Fund

We're using Polar.sh so you can upvote and help fund this issue.
We receive the funding once the issue is completed & confirmed by you.
Thank you in advance for helping prioritize & fund our backlog.

Duplicate of #369


After logging in, it is up to 404 Page Not Found

I downloaded it to my server that uses AppServ.After logging in, it is an error to 404 Page Not Found. Please provide instructions. This is address on display "http://localhost/schoolerp/home/admin_login"

This is address on display http://localhost/schoolerp/home/admin_login

Please Change the directory name in the .htaccess file in root directory according to your directory name.


Broken Popover when disableAnimation = true

Closes #797
📝 Description
Unfortunately the popover without animation has problems with re-rendering after opening state change.
If we disable animation by setting arbitrary values in CSSTransition we enforce a re-render and react-aria is able to correctly define position state because the DOM element 'exists'.
⛳ Current behavior (updates)
When setting disableAnimation = true, popover / react-aria wont be able to define "left" / "right" positions because the DOM element is not rendered in time?! This means the opened content of the popover exists outside of the browser-view.
🚀 New behavior
Enforces re-rendering of Popover.Content while using arbitrary values with CSSTransition if disableAnimation = true.
Popover.Content will be visible with disableAnimation = true.
💣 Is this a breaking change (Yes/No):
No
📝 Additional Information
This solution to the problem is as hacky as hacktober 2022 :)
Maybe it will help to find a proper solution or alteast provide a hackfix for the time being.

Hey @0x2aff I really appreciate you sending this PR, but unfortunately I'll close it due to we fix this internally https://github.com/nextui-org/nextui/pull/861, sorry 🙏🏻


WithBalancerName is deprecated

in grpc

@hagaidayan could you share a place in docs with me? I'm a little bit surprised because WithBalancerName is a 'new' way to pick up right balancer  https://github.com/grpc/grpc-go/blob/36f3126920fe326b7874e730e3cc26546186d8f0/examples/features/load_balancing/README.md

	grpcConn, err := grpc.Dial(statsAddr, 		grpc.WithInsecure(), 		grpc.WithDefaultServiceConfig({"loadBalancingConfig": [{"round_robin":{}}]}), 	)
this one might me a fix


Add serialization traits

This commit adds experimental version of new serialization traits. Blanket implementation are provided to support older traits.
For now there are no comments / other implementations with real type checking as the purpose of this PR is to unblock the work on macros
and other parts of refactor.
Pre-review checklist

[x] I have split my patch into logically separate commits.
[ ] All commit messages clearly explain what they change and why.
[ ] I added relevant tests for new features and bug fixes.
[ ] All commits compile, pass static checks and pass test.
[x] PR description sums up the changes and reasons why they should be introduced.
[ ] I have provided docstrings for the public items that I want to introduce.
[ ] I have adjusted the documentation in ./docs/source/.
[ ] I added appropriate Fixes: annotations to PR description.

Adressed review comments


Publish 8.0-20221220005555+0000

Publish 8.0-20221220005555+0000

@bot-gradle test and merge

OK, I've already triggered a build for you.


Decentralized Caching via IPFS or Torrent

In the standard GNU Make, the build system's efficiency lies within the nice DAG that ensures that a file is not re-created if it already exists.
A modern take on this may be that a git-commit's hash determines the files (and therefore the file hashes) that are built from the source code.  Therefore, it is obvious that for a given repo there is a hashmap of a gitcommit to a list of data-hashes.
It would be a wonderful feature to use this hashmap to ensure that very long running commands not be re-run during a build process.
In some sense, GNU's make does a first pass attempt at this problem, by simply checking for the local file.  However, tools such as ipfs or torrent today may eliminate some of these long running tasks even further, by looking up the hashes (guaranteed to be the correct output from this source-code build) online and downloading them directly rather than re-running long processes.
An example use case may be an ETL processes of generating a database, or generating exchange correlation functionals (each of which may take a few days on a supercomputer, but in some cases only be a few MBs or GBs).
I have been looking for this functionality for quite some time, as it should be standard in modern build-tools such as make.
Snakemake appears to be one of the few that is actively maintained and used compared to Make, and therefore I am interested in how to implement this functionality here.
Describe the solution you'd like
A very simple syntax addition (perhaps a prepended single character) to denote when a filepath should be checked on IPFS in addition to locally.
The idempotent logic should be something like:

Is the filepath found locally?
Yes) Goto (2) & return the file
No) Goto (2)
Is the ipfs_hash(filepath) on IPFS?
Yes) Goto (4) & return the file
No) Build the file and Goto (3)
Upload the file to IPFS
Download the file from IPFS to the filepath location

Describe alternatives you've considered
I have thought of doing this with Makefile, using something similar to this:
#!/bin/bash

file_path=$1
hash_file="$1.ipfs-hash"

# Check if the file exists
if [ -f "$file_path" ]; then
    # If file exists, check if it has an updated IPFS hash
    if [ -f "$hash_file" ]; then
        # File exists and has a known hash, use it as-is
        ipfs_hash=$(cat "$hash_file")
        echo "Using existing file with IPFS hash: $ipfs_hash"
    else
        # File exists but no known hash, upload to IPFS and store the hash
        new_hash=$(ipfs add -Q "$file_path")
        echo "$new_hash" > "$hash_file"
        echo "File uploaded to IPFS with new hash: $new_hash"
    fi
fi

And the the Makefile:
# Function to handle IPFS or local file
define ipfs_or_local
$(if $(findstring ipfs(,$1),\
$(eval file_path := $(shell echo '$1' | sed -e 's/ipfs(//g' -e 's/)//g'))\
$(shell ./ipfs_handle.sh $(file_path))\
$(file_path),\
$1)
endef

# Example target that generates a file
your-target: $(call ipfs_or_local,ipfs(path/to/generated/file))
    # Commands to generate the file
    <build-commands-for-your-file>
    # After build, handle IPFS upload and hash storage
    $(shell ./ipfs_handle.sh path/to/generated/file)

But it's not the nicest solution to be depending on across millions of repos, and would be better to be integrated into the build tool itself.

The cached function in snakemake may be helpful. However, it is mentioned but vaguely in the document.

@Hocnonsense Yes, this cached function does seem useful, though after funding the storage capabilities, it seems that the best route is to create a storage plugin that works by calling and handling the ipfs protocol:
rule example:
    input:
        storage("ipfs://{{cid}}/{{path}}")
    output:
        "example.txt"
    shell:
        "..."

This appears to be achievable to create this for downloading, although the automatic publishing and offering of the data to the community may be more challenging, as it is slightly different than the storage systems provided at the moment.  The ideal case for ipfs or torrent would be to automatically start or continue running a daemon which will add a node to the pool of servers containing the file for download.
The benefits of this are numerous and obvious:

Data becomes more available with popularity
- Likely surpassing the download speeds of centralized services
Popular or useful data doesn't randomly disappear
- Scientific data goes missing constantly due to researchers moving institutions (any university site download), or not continuing to pay huge fees to companies, or randomly deleting to open storage for personal files (most Google drive / Dropbox / AWS data disappears quickly in academia)
3) Free

IPFS doesn't require paying any centralized company to host the data, but also doesn't restrict it

I don't have any experience with writing these snakemake plugins, but I would imagine getting the download functionality on ipfs or torrent would be fairly simple, while the real solution of having each client also have a server daemon with absolute minimal resources used might be slightly more tricky.


Rename item variable to "my variable"

Fixes #1539

Could we not have a default variable?

Not sure what would happen to the OO apis @riknoll ?

I actually have no idea. Blockly might just add the variable if it doesn't already exist.

Okay will try.

From: Richard Knoll<EMAIL_ADDRESS>Sent: Monday, March 6, 2017 5:30:45 PM
To: Microsoft/pxt
Cc: Sam El-Husseini; Author
Subject: Re: [Microsoft/pxt] Rename item variable to "my variable" (#1547)
I actually have no idea. Blockly might just add the variable if it doesn't already exist.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoft%2Fpxt%2Fpull%2F1547%23issuecomment-284591931&data=02|01|Sam.El-Husseini%40microsoft.com|ed468047a5b948bf9ec608d464f996f7|72f988bf86f141af91ab2d7cd011db47|1|0|636244470569371840&sdata=anunCCI%2Bo9Or8c54zEDALte%2F5NQhVOKXUhTJV7DRWyg%3D&reserved=0, or mute the threadhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAP6rzJF197pPVB57494JS_VlgVi26RmKks5rjLNFgaJpZM4MUOuW&data=02|01|Sam.El-Husseini%40microsoft.com|ed468047a5b948bf9ec608d464f996f7|72f988bf86f141af91ab2d7cd011db47|1|0|636244470569381844&sdata=EsxMQRZr33xlaMyx6PmGsTwApv3ANI7vGxqluIRu2Js%3D&reserved=0.

Hey there--I manage variables for Blockly.  Feel free to ping me with questions.

@abchatra  We should either take this or close it.

I don't know if the 'myvariable' is a good name, 'item' is definitely not. May be a topic for design discussion.

Renaming it would be a UI breaking change.
Get Outlook for iOShttps://aka.ms/o0ukef

From: Abhijith Chatra<EMAIL_ADDRESS>Sent: Monday, April 17, 2017 7:24:38 AM
To: Microsoft/pxt
Cc: Peli de Halleux; Review requested
Subject: Re: [Microsoft/pxt] Rename item variable to "my variable" (#1547)
I don't know if the 'myvariable' is a good name, 'item' is definitely not. May be a topic for design discussion.
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoft%2Fpxt%2Fpull%2F1547%23issuecomment-294495067&data=02|01|jhalleux%40microsoft.com|d2f94027d2f84e27f73808d4859d7bd2|72f988bf86f141af91ab2d7cd011db47|1|0|636280358812913341&sdata=4QRLBbcTcdELjPi5NoWGIvFathtsEl0WN%2Flg9TTCwoc%3D&reserved=0, or mute the threadhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAD-4KREKgbmmnt8t9dAjo0fVosdWDltmks5rw3YmgaJpZM4MUOuW&data=02|01|jhalleux%40microsoft.com|d2f94027d2f84e27f73808d4859d7bd2|72f988bf86f141af91ab2d7cd011db47|1|0|636280358812913341&sdata=Yc5jWgrPDq%2B8n6eOi54auLJqWK%2FjUpvCGmNtr3P0juU%3D&reserved=0.

Yes, only for microbit

Maybe it should be called "that" or "stuff"

We need a default variable.  My vote is for "myvariable", or "myvar" if that's too long.  It's the clearest indication of what it is.  Imagine a teacher using MakeCode to teach students about variables in computer science.  "that" or "stuff" would not be a good name in this scenario.

Scratch uses "thing" to refer to items in their lists.

I like "thing"
Get Outlook for iOShttps://aka.ms/o0ukef

From: Richard Knoll<EMAIL_ADDRESS>Sent: Friday, April 28, 2017 9:49:21 AM
To: Microsoft/pxt
Cc: Peli de Halleux; Review requested
Subject: Re: [Microsoft/pxt] Rename item variable to "my variable" (#1547)
Scratch uses "thing" to refer to items in their lists.
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoft%2Fpxt%2Fpull%2F1547%23issuecomment-298049248&data=02|01|jhalleux%40microsoft.com|2aa9d2ed2d4540f221ad08d48e568652|72f988bf86f141af91ab2d7cd011db47|1|0|636289949653045489&sdata=RcU%2BfV3cUWnbIitTsvO2l46ihkiCsoB6jsp2AufulM0%3D&reserved=0, or mute the threadhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAD-4KTjTyc9r2Zv1idGQIo4MxaLZ--dvks5r0hiRgaJpZM4MUOuW&data=02|01|jhalleux%40microsoft.com|2aa9d2ed2d4540f221ad08d48e568652|72f988bf86f141af91ab2d7cd011db47|1|0|636289949653045489&sdata=I%2F2ZV2qV7KoY0I0vTYkWjgyUvmDbcgsmjg37KWYBZu0%3D&reserved=0.

@Jaqster @pelikhan @abchatra are we retiring this?

I am indifferent. You folks decide.

I'd like to leave this open - we need to be consistent with our default naming.  I'd like to suggest: "myvariable", "myindex", "myarray", "myfunction".  Makes it much clearer/standard than random "item", "index", "list", "dosomething".  And will work across targets.

How about no default? Variable list should be empty to start from.
Get Outlook for iOShttps://aka.ms/o0ukef

From: Jacqueline Russell<EMAIL_ADDRESS>Sent: Wednesday, June 21, 2017 8:25:50 PM
To: Microsoft/pxt
Cc: Peli de Halleux; Mention
Subject: Re: [Microsoft/pxt] Rename item variable to "my variable" (#1547)
I'd like to leave this open - we need to be consistent with our default naming. I'd like to suggest: "myvariable", "myindex", "myarray", "myfunction". Makes it much clearer/standard than random "item", "index", "list", "dosomething". And will work across targets.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoft%2Fpxt%2Fpull%2F1547%23issuecomment-310265113&data=02|01|jhalleux%40microsoft.com|260e2465a0104a152b0208d4b91e62d9|72f988bf86f141af91ab2d7cd011db47|1|0|636336987547191551&sdata=1B394QSnpdNywnswcapQyvkfzeMMmZCsmLup1pdexBc%3D&reserved=0, or mute the threadhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAD-4KTSo38QEO4qqVIiqcMxzRXXmCFueks5sGd6-gaJpZM4MUOuW&data=02|01|jhalleux%40microsoft.com|260e2465a0104a152b0208d4b91e62d9|72f988bf86f141af91ab2d7cd011db47|1|0|636336987547191551&sdata=VK8LF7Zw4ChVhbbVBVb9%2B8jpDx%2BIsc9VH%2BVEOa1xgUU%3D&reserved=0.

Reopen when we are ready to tackle this.


Change from flake8-putty to flake8-per-file-ignores.

This allows us to support newer Python 3 syntax including f-strings and
type annotations.
Report after changing extensions:
samuelwilliams:~/git/digitalmarketplace-admin-frontend [master]$ make test-flake8
[ -z $VIRTUAL_ENV ] && [ ! -d venv ] && virtualenv -p python3 venv || true
/Users/samuelwilliams/git/digitalmarketplace-admin-frontend/venv/bin/flake8 .
./app/main/views/__init__.py:0:1: X100 Superfluous per-file-ignores for C901
./app/main/views/admin_manager.py:0:1: X100 Superfluous per-file-ignores for C901
./app/main/views/agreements.py:0:1: X100 Superfluous per-file-ignores for C901
./app/main/views/buyers.py:0:1: X100 Superfluous per-file-ignores for C901
./app/main/views/buyers.py:19:5: E722 do not use bare except'
./app/main/views/communications.py:0:1: X100 Superfluous per-file-ignores for C901
./app/main/views/service_updates.py:0:1: X100 Superfluous per-file-ignores for C901
./app/main/views/services.py:0:1: X100 Superfluous per-file-ignores for C901
./app/main/views/stats.py:0:1: X100 Superfluous per-file-ignores for C901
./app/main/views/suppliers.py:0:1: X100 Superfluous per-file-ignores for C901
./app/main/views/users.py:0:1: X100 Superfluous per-file-ignores for C901
make: *** [test-flake8] Error 1```

@samuelhwilliams rebased, resolved conflicts and ran locally just to test my theory and it worked so pushed up. 👍


Array rotation Reversal Algorithmm

Please star this repository, in the meantime I am reviewing the code

@amritansh22 done, now accept 🙏


doc: fix some_query syntax

@mnencia
this it the PR following the Slack Thread
so that the "some_query" query can be copy/pasted into the example-monitoring configMap
br
Daniel

/ok-to-merge


Don't complain and fail about unknown TLSv1.3 PSK identities in s_server

An unknown PSK identity could be because its actually a session resumption
attempt. Sessions resumptions and external PSKs are indistinguishable so
the callbacks need to fail gracefully if they don't recognise the identity.
Fixes #7433

Checklist

[ ] documentation is added or updated
[ ] tests are added or updated

Pushed. Thanks.


rimraf 6.0.1 or electron-store 10.0.0 can't be build correct with bytecodePlugin

Describe the bug
pnpm create @quick-start/electron testrimraf --template vue
✔ Add TypeScript? …  Yes
✔ Add Electron updater plugin? …  Yes
✔ Enable Electron download mirror proxy? …  Yes
add these lines in main/index.ts bottom
import { rimrafSync } from 'rimraf';
import { mkdirSync } from 'fs';
mkdirSync('aaa', {recursive: true});
rimrafSync('aaa');

rimraf 6.0.1  latest
plugins: [externalizeDepsPlugin(),bytecodePlugin()]
it will show when running the exe Uncaught Exception: TypeError: Class extends value undefined is not a constructor or null at Object.<anonymous> (/Users/xxx/code/projects/app/testrimraf/dist/mac-arm64/testrimraf.app/Contents/Resources/app.asar/node_modules/path-scurry/dist/commonjs/index.js:124:40)
and if using
plugins: [externalizeDepsPlugin({exclude:['rimraf']}),bytecodePlugin()]
it will show when pnpm run build [vite:bytecode] unknown file: Unable to transform arrow inside class property
and it's working for run dev
pnpm create @quick-start/electron testelectronstore --template vue
✔ Add TypeScript? …  Yes
✔ Add Electron updater plugin? …  Yes
✔ Enable Electron download mirror proxy? …  Yes
add these lines in main/index.ts bottom
import Store from 'electron-store';
const store = new Store();

electron-store 10.0.0 latest
plugins: [externalizeDepsPlugin(),bytecodePlugin()]
it will show when running the exe or run dev Error [ERR_REQUIRE_ESM]: require() of ES Module<EMAIL_ADDRESS>from /Users/xxx/code/projects/app/testelectronstore/out/main/index.js not supported.
and my project can't using type:module for other codes, here just for the demo case.
but if using
plugins: [externalizeDepsPlugin({exclude:['electron-store']}),bytecodePlugin()]
it will show when pnpm run build [vite:bytecode] unknown file: Unable to transform arrow inside class property
Electron-Vite Version
2.3.0
Electron Version
31.0.2
Vite Version
5.3.1
Validations

[X] Follow the Code of Conduct.
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.

Customizing Protection
Or
https://github.com/alex8088/electron-conf (Recommand)

Customizing Protection Or https://github.com/alex8088/electron-conf (Recommand)

In the future, this 'esm' kind or 'arrow inside class property' packages may become more and more, will it support more convenient methods?
and for rimraf 6.0.1 plugins: [externalizeDepsPlugin()] want to ask a question
do you know why running the exe and the error is like this?
it will show when running the exe Uncaught Exception: TypeError: Class extends value undefined is not a constructor or null at Object.
Is it also caused by esm?, even the error he reported is not esm

if i using manualChunks for process bytecodePlugin
and if my child process using mime 4.0.4
I need to write like this
  if(id.includes('mime')){
            return 'xxx';
  }

otherwise it will exit the child process, i don't know why, just add this comment as a mark
{
  plugins: [externalizeDepsPlugin({exclude:['rimraf','mime']}),bytecodePlugin({chunkAlias:['index']})],
  build: {
    rollupOptions: {
      input: {
        index: resolve(__dirname, 'src/main/index.ts'),
        child: resolve(__dirname, 'src/main/child.ts')
      },
      output: {
        manualChunks(id){
          if(id.includes('mime')){
            return 'xxx';
          }
          if(id.includes('node_modules')){
            return 'vendor'
          }
        }
      }
    }
  }
}

it seems bytecodePlugin({transformArrowFunctions:false}) can avoid this issue.
but don't know why need this for running after build if using this in child
          if(id.includes('mime')){
            return 'xxx';
          }

anyway this maybe not related this issue.


fix: refine add = prefix logic

Description
Refine add = prefix logic

Drop adding = prefix if property value starts with =, like =abc or  = abc or  =abc, then ui will be

property = `abc`

Add = prefix if property value doesn't start with =, like abc

property = abc

Task Item
closes #2222
Screenshots

@alanlong9278 we decided we wanted to keep the expression = sign but replace the assignment operator in the node with :, so:
somval: =somefunc
Can you update this?

@alanlong9278 we decided we wanted to keep the expression = sign but replace the assignment operator in the node with :, so:
somval: =somefunc
Can you update this?

Sure.


Unable to load Home Assistant interface via port 8123 through Nebula

I've got the nebula add-on installed and running successfully (As of the fixes in PR #1 ), and I can confirm that:

My Home Assistant server is connected to my overlay network
I can ping my Home Assistant server via its nebula IP
My port 8123 firewall rule is present (level=info msg="Firewall rule added" firewallRule="map[caName: caSha: direction:incoming endPort:8123 groups:[axel] host: ip: proto:6 startPort:8123]")

However, I can't seem to actually load the web interface via port 8123. Should I expect that to be working, or is there something else that I need to configure for that?

Yeah, I know there's a few steps needed. There's a particular magic combination of iptables rules I need to figure out as a post_up equivalent from wireguard, but after fiddling with it awhile last weekend I got fed up, but I'll re-visit it with fresh eyes soon.
Essentially I need to figure out iptables magic is necessary to forward traffic from nebula -> the local interface inside the docker container that surfaces home assistant.
So far I've worked out that in order to advertise the route to home assistant, first the nebula-cert command that you use to sign the cert being used by home assistant needs a -subnets <IP_ADDRESS>/23 (approximately), then you need to, on the devices that want to route to home assistant, add the home assistant docker-internal ip (for me it's <IP_ADDRESS>) as an unsafe_route in the config.
That's where I got tired and stopped, but pretty sure the iptables config is what I need to work out next.

Heh, if you can't tell it's very much still in development. I was also working on the bash-scripts to be used to do all the nebula-cert management. I'll throw that into master as well in the semi-developed state that it's in so far.

@V13Axel I came back to this today with a clear head and got it sorted out and working forwarding not just Home Assistant, but also the entire home network. (The only behavior for now, but later this will be configurable).
If you want to try again these were the manual changes needed (after my code change):

Your advertise_addrs clause in your nebula config for HASS matches any external port-forwarding you're doing
Make sure in your nebula client configs you include an unsafe_routes clause matching the IP range you'd like to access over the Nebula tunnel (in my case, on my phone I added an unsafe_route for 192.168.X.0/24 to access my whole network)
You'll need to sign the HASS addon certificate to include the above unsafe_route subnet. If you're using my script to generate them, you can just add the following to the end of your host.txt file on the line generating your HASS certs: -subnets 192.168.X.0/24

Next I'll get on to handling all this manually so it will spit out server and client configs and certs/keys automatically for you, but for now it's stably granting me access to the HASS and home network IPs. Next will be also handling DNS seamlessly as well.

Oh, neat! I'll definitely give this a look at some point shortly then, thanks for the update.
Only reason I didn't look into this too hard is that I had a short-term need at the time and ended up putting together a temporary solution of setting up a reverse-proxy on one of my other homelab servers that already had Nebula up and running and then accessing it that way. Not perfect, but it got the job done in a pinch; We had needed to remotely monitor some things for my wife's reptiles while we were out of town a while back.
However, that need will definitely continue to recur, which means it's worth it for me to look into this!


Auto Gift Exchange offests incorrect?

Are all of the offsets for the Auto Lottery/Auto Gift Exchange script off? When I start the script, it start clicking in the upper left corner, near Nero's head. I tried messing with the offsets in the consts area, and I managed to get it to start clicking on the correct button for spinning the box, but it still doesn't do things like auto reset the boxes.

If you haven't updated in a while then they would indeed be off. I have already fixed this issue about a week and a half ago or more.


Rewrite LocalFolder.purgeToVisibleLimit

This PR makes the purgeToVisibleLimit method in LocalFolder more efficient by fetching only the required messages and message fields from the database.

The MessageRemovalListener is used to pass deleted messages to MessagingListener.synchronizeMailboxRemovedMessage(). The implementation of this interface in the K9 class uses this Message instance to send broadcast intents. See https://github.com/k9mail/k-9/blob/b9c0c92e7435a83a74238a8dd5740eca6364d9ba/k9mail/src/main/java/com/fsck/k9/K9.java#L615 and https://github.com/k9mail/k-9/blob/b9c0c92e7435a83a74238a8dd5740eca6364d9ba/k9mail/src/main/java/com/fsck/k9/K9.java#L572-L591
At least the message properties used there need to be loaded from the database to preserve the current behavior.

@cketti Done

Drat, I hadn't considered this listener when I thought we could get away without loading the message. I really don't like partially loading a message in this way, but also I don't think we should do something like "load all messages to delete into memory".
The number of messages we query here will usually be small, but if it's big we're better off taking a long time than running out of memory.
I pushed a suggestion, please take a look.

Valodim's change looks good to me.
On another note: This branch needs to be rebased after merging #2693

done

added a test

I merged the code without the test. I'd rather have such functionality tested with an integration test, in particular against a real sqlite. That's far less brittle and ensures the used SQL statements are correct.
For now I think we're better off without test than with a test that mocks everything.


DropDown ignores new options on componentWillReceiveProps

Steps
Use DropDown as a live search component with server-side searching/filtering, where the options passed to the DropDown are updated whenever the user changes the search query in the input box (like in the Remote example given on http://react.semantic-ui.com/modules/dropdown, but update the options in the handleSearchChange method). This means that the DropDown will receive new options on componentWillReceiveProps().
Pass the options as a list of plain objects, e.g., options = [{ value: 1224, text: "item 1" }, { value: 12344, text: "item 2" }].
The error will not show up if you use sub components, i.e., use DropDown.Menu and pass options as children of type DropDown.Item (but doing so will make you lose auto state management).
Expected Result
The options should be updated and show up in the list immediately.
Actual Result
The options do not or only party show up. You can make them show up by performing certain actions in the browser (like pressing CTRL+SHIFT+i in Chrome to open the developer console).
Version
0.64.4
Testcase
Currently working on a minimal example, will update issue with a link when ready...

Currently working on a minimal example, will update issue with a link when ready...
❤

@dfsp-spirit Any updates?

It worked in the minimal example I tried to made, so it seems it is related to other stuff. We are still using the workaround I mentioned above.

Thanks for the update.  Let us know if you are able to reproduce the issue in an isolated environment and we'll help out with a fix.


HADOOP-16706. ITestClientUrlScheme fails for accounts which don't support HTTP

Adds a new service code to recognise accounts without HTTP support; catches
that and considers that a successful validation of the ability of the
client to switch to http when the test parameters expect that.
Tested: Azure cardiff, with a storage account which doesn't support HTTP

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
30m 34s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

+1 :green_heart:
test4tests
0m  0s
The patch appears to include 1 new or modified test files.

_ trunk Compile Tests _

+1 :green_heart:
mvninstall
22m 36s
trunk passed

+1 :green_heart:
compile
0m 27s
trunk passed

+1 :green_heart:
checkstyle
0m 22s
trunk passed

+1 :green_heart:
mvnsite
0m 32s
trunk passed

+1 :green_heart:
shadedclient
17m 39s
branch has no errors when building and testing our client artifacts.

+1 :green_heart:
javadoc
0m 29s
trunk passed

+0 :ok:
spotbugs
1m 11s
Used deprecated FindBugs config; considering switching to SpotBugs.

+1 :green_heart:
findbugs
1m  8s
trunk passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
0m 33s
the patch passed

+1 :green_heart:
compile
0m 26s
the patch passed

+1 :green_heart:
javac
0m 26s
the patch passed

+1 :green_heart:
checkstyle
0m 17s
the patch passed

+1 :green_heart:
mvnsite
0m 30s
the patch passed

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
shadedclient
17m 26s
patch has no errors when building and testing our client artifacts.

+1 :green_heart:
javadoc
0m 24s
the patch passed

+1 :green_heart:
findbugs
1m  8s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
1m 29s
hadoop-azure in the patch passed.

+1 :green_heart:
asflicense
0m 33s
The patch does not generate ASF License warnings.

98m 18s

Subsystem
Report/Notes

Docker
Client=19.03.5 Server=19.03.5 base: https://builds.apache.org/job/hadoop-multibranch/job/PR-1844/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/1844

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient findbugs checkstyle

uname
Linux d9fb28d9d39b 4.15.0-74-generic #84-Ubuntu SMP Thu Dec 19 08:06:28 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
personality/hadoop.sh

git revision
trunk / f09710b

Default Java
1.8.0_242

Test Results
https://builds.apache.org/job/hadoop-multibranch/job/PR-1844/1/testReport/

Max. process+thread count
308 (vs. ulimit of 5500)

modules
C: hadoop-tools/hadoop-azure U: hadoop-tools/hadoop-azure

Console output
https://builds.apache.org/job/hadoop-multibranch/job/PR-1844/1/console

versions
git=2.7.4 maven=3.3.9 findbugs=3.1.0-RC1

Powered by
Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.

@bgaborg could you look at this?Thanks

thx, will rebase and merge ASAP


Pipeline editor: allow for changing of the notebook [name/location]

As a user I would like to change the notebook name or location of a node. Currently the node properties display the name/location but do not allow for editing.  Therefore if a user wants to change the name/location of a notebook the following steps need to be completed::

create a new node for the "renamed" notebook
select a runtime container
copy and paste "file dependencies"
copy and paste environment variables
copy and paste "output files"
remove old node
link notebook nodes

With this proposed enhancement a user would only have to change the name/location of the notebook.

Bringing this under the umbrella issue #822 to handle updating a node's file location

A quick note on implementation: We definitely do not want to make this update via a text fields, but some form of UI. (as per previous discussions around this topic)


Importing problems

Hi,
thanks for this library!
I'm having some problems importing only one picker
import ColorPicker from 'vue-color/src/components/Compact';

fails in my project (with webpack, and vue-loader installed) with this error:
This dependency was not found:

* !!vue-style-loader!css-loader?{"minimize":false,"sourceMap":false}!../../../vue-loader/lib/style-compiler/index?{"id":"data-v-ade7e162","scoped":false,"hasInlineConfig":false}!stylus-loader?{"sourceMap":false}!../../../vue-loader/lib/selector?type=styles&index=0!./Compact.vue in ./~/vue-color/src/components/Compact.vue

To install it, you can run: npm install --save !!vue-style-loader!css-loader?{"minimize":false,"sourceMap":false}!../../../vue-loader/lib/style-compiler/index?{"id":"data-v-ade7e162","scoped":false,"hasInlineConfig":false}!stylus-loader?{"sourceMap":false}!../../../vue-loader/lib/selector?type=styles&index=0!./Compact.vue

Importing with
import { Compact as ColorPicker } from 'vue-color';

works but it loads the entire library and not just the Compact picker
Thanks

I'm having the same problem... @paglias have you solved this?

Are you sure that it's loading the entire library using
import { Compact as ColorPicker } from 'vue-color'; ?
We are exporting each component individually, importing like that should import only the component, not the whole library.

@2one2 I think I ended up importing the whole project because importing a single file doesn't work and import { Compact as ColorPicker } from 'vue-color'; imports the entire project anyway because it's compiled as an es5 module and not exported using es6

@paglias thanks
@cassioscabral yes importing with import { Compact as ColorPicker } from 'vue-color'; imports entire library... with webpack

@paglias Can I ask did you install stylus-loader in your project? Need more information to debug.

@linx4200 no, I don't use stylus-loader

@paglias Thanks for the reply. We have figured out what causes this problem. Would fix it asap.

Fixed.
Please upgrade to the latest version.
Now you can import component individually. For example import compact from 'vue-color/src/components/Compact.vue';.

Similar issue here but with babel-preset-es2015.
For the record, install it in your project (npm install babel-preset-es2015) and change vue-loader options in Webpack:
{
  test: /\.vue$/,
  loader: 'vue-loader',
  options: {
    // your options here,
    presets : [
      require.resolve('babel-preset-es2015')
    ]
  }
}

In addition, since there is a mixin/color.js file (non vue file) which also needs transpilation:
{
  test: /\.js$/,
  loader: 'babel-loader',
  options: {
    presets : [
      require.resolve('babel-preset-es2015')
    ]
  }
}

@linx4200 Perhaps this could be fixed in a new version? Providing an ESM ready folder would be ideal but might be a bit hard. I think using babel-preset-env, which is more common than babel-preset-es2015, is easier. Also, color.js would need to be full ES5. I think the only required change in that regard is syntax like this:
{
  data() {
  }
}

Should be:
{
  data: function() {
  }
}

Same for computed props, watch and methods.


[BUG] Learning rate is not passed to network scripts

From benchmark.py and configs/*.config, we know dlbench provide capability of changing learning rate.
However, only Caffe, Torch, MXNet accepts learning rate argument while CNTK, TensorFlow ignores them.
# tools/cntk/cntkbm.py has no lr argument defined.
# tools/tensorflow/tensorflow.py has no lr argument defined.

Furthermore, the learning rate is not the same when running benchmark. For example, TensorFlow uses constant value while MXNet's learning rate will change during training.
# From tools/mxnet/common/fit.py
steps = [epoch_size * (x-begin_epoch) for x in step_epochs if x-begin_epoch > 0] # Default value of step_epochs is '200,250' from tools/mxnet/train_cifa10.py
return (lr, mx.lr_scheduler.MultiFactorScheduler(step=steps, factor=args.lr_factor))
......
optimizer_params = {'learning_rate': lr,
            'momentum' : args.mom,
            'wd' : args.wd,
            'lr_scheduler': lr_scheduler} # This scheduler will change learning rate during training

Please let all tools support learning rate parameter or just delete learning rate from config.

The schedule of learning rate of MXNet is not used. Please check the code: https://github.com/hclhkbu/dlbench/blob/master/tools/mxnet/common/fit.py#L8. The parameter of lr_factor is set to be None. For CNTK and TF, we set them to be fixed.

@shyhuai But you set the default value of lr_factor to 0.1 at https://github.com/hclhkbu/dlbench/blob/master/tools/mxnet/common/fit.py#L63.
train.add_argument('--lr-factor', type=float, default=0.1, help='the ratio to reduce lr on each step')

So, no matter wheter we explicitly set lr_factor explicitlly in command-line arguments, argparse.ArgumentParser will always set the lr_factor. Check the log:
INFO:root:start with arguments Namespace(batch_size=1024, data_dir='/home/shaocs/dlbench/dataset/mxnet/mnist', disp_batches=100, gpus='0', kv_store='device', load_epoch=None, lr=0.05, lr_factor=0.1, lr_step_epochs='10', model_prefix=None, mom=0.9, monitor=0, network='mlp', num_classes=10, num_epochs=2, num_examples=60000, num_layers=None, num_nodes=1, optimizer='sgd', test_io=0, top_k=0, wd=1e-05)
......
INFO:root:Update[586]: Change learning rate to 5.00000e-03

@shishaochen Thanks for you feedback. Since we set lr_factor=1 in the script of: mxnetbm.py, the learning rate will not be changed during training. If you use the script of mxnetbm.py, it could be no problem. Here is the log for your reference: http://dlbench.comp.hkbu.edu.hk/logs/?f=mxnet-fc-fcn5-gpu0-K80-b512-Tue_Mar__7_10:52:06_2017-gpu20.log. In case of misunderstanding, I have revised the code to set the default value to None. Thank again for your report.

@shyhuai Sorry. I cannot find "factor" set in https://github.com/hclhkbu/dlbench/blob/master/tools/mxnet/mxnetbm.py. Maybe you set it locally but the change is not committed yet.