Incorrect gradle dependency snippet

Artifacts available on JCenter (Maven not checked) cannot be accessed with dependency snippet provided in readme.md.
Proposed solution:
Change dependency snippet in readme.md file from:
implementation("ru.gildor.coroutines:kotlin-coroutines-okhttp:1.0.0")
to:
implementation("ru.gildor.coroutines:kotlin-coroutines-okhttp:1.0")
or:
implementation 'ru.gildor.coroutines:kotlin-coroutines-okhttp:1.0'

Looks like it was resolved in #6


Error when trying to write boolean type

Hi, there,
We recently had this bug report https://github.com/request/request/issues/1693 in request
It turned out that the user passes a boolean value to one of the form-data's fields (field:false) that results in _http_outgoing error in core.
My question is: Is the best place to fix that issue here, like you do for number types?
// all that streamy business can't handle numbers
if (typeof value == 'number' || typeof value == 'boolean') value = ''+value;

There is more than one way to express boolean value as a string, I mean there are many servers/apps that do it differently. For example just few I bumped into in the recent years:
"true" vs "false"
"1" vs "0"
"1" vs ""
and
"1" vs <don't send that field at all>
So we can't make it as library decision, it's something that developer should decide between their code and server they're talking to.
But I see the problem, I think what we could do here, is it emit meaningful error, instead of letting it slide.

I see, good point. Either error message or stringify should be ok.
And yet, why is the number stringified then? The number is received as a string on the server, which again is not the way you passed it to the module.

Everything is a string on the wire. :) And there are not so many way to represent numbers as a string, I can come up with two on top of my head:
6 -> "6" and 6 -> "VI" :)

Got it. From the server's perspective the stringified true and false words may be too opinionated, like for example if the client is written in Python, these values may as well be True and False.

In PHP land you'd see a lot of if ($_POST['value']) or with latest trend on being strictly equal, something like if ($_POST['value'] === '1'). :)

@alexindigo, i just checked how the browser formdata handles this and it does in fact cast booleans to strings. I think it would be generally convenient for node-form-data to do the same. Consumers of the formdata (another server) are going to have to cast some of the strings sent anyways so I think this makes sense.

It might be bug in the browser implementation. Clearly it's the least expected default, most used would be 1 and 0, but again it doesn't cover all the cases. So I'd go for being more explicit, shorter path to bug discovery. Then let people test it it with if ("true") and fail in prod with if ("false").
Higher order libs could provide default and custom mapping functions, to be more user- / browser- friendly. And low level lib would expect to get proper values.

I'm not sure the "false" thing is an issue. Users are going to have to convert their values to the native types regardless since we can't transfer type information with formdata.
I would argue that it would best to follow the browser spec since it's likely that any server that is going to accept this post data would also want to except multipart forms from the browser and it would be inconvenient to parse the formdata differently based on it's source. Even if this isn't the case it would make sense to have follow a consistent spec rather than have an arbitrary implementation that forces the user to guess (twice).
I also think it's harmful to throw an error in this case, what if a user has no idea and passes in a value with an unknown type? Later in production they pass in a bool and it crashes. Personally I thought it was odd that we don't allow arrays.

In many cases client side passes empty string as false and script just treat it as falsy value.
As for crashing :) You'd want your script to crash other than proceeding in unknown state, like wiping out your database, or charging all your customer's credit cards, or just executing inject piece of code on your server. :)  Crashing in prod is not bad, in many cases it's the best case scenario. :)

I think it's less about crashing in prod and more about it not being the expected result. Also where does JavaScript pass empty strings as false? Surely it's falsy but Its not like its going to get casted to a Boolean before being sent out as form data.

Most webapps still built expecting data coming from html forms (if it comes from js apps, usually it's a json blob, unless files).
And many server side endpoints looks like that:
if ($_POST['optionA'])
{
}

Not only in PHP, but in all flavors of languages.
Examples: https://github.com/search?q=If+(%24_POST\[&type=Code&ref=searchresults
Again, there should be all kinds of libraries doing it in all possible ways :) So people can pick the ones they need for the apps. And to allow that, low level libs should leave this question out and focus on doing stuff like multiparting. :)

All I'm saying is that this is what is outputted when a multi part html form is submitted, I figured we would want to model that.

I see. In the isomorphic- one for sure. And maybe we'd have oldskool-form-data where we'd convert true -> 1 and false -> 0. :)

i was investigating a little bit more and now having second thoughts. That world has changed and accepting false as "false" might be new normal.
I think think this kind of functionality belogs to the higher level libs. And that leveldb approach is the healthiest one.
So we'd let low level lib to deal with multipart and be strict on what it accepts.
And<EMAIL_ADDRESS>be more like it's browser counterpart.
Thst way it will smooth and simple for the app usage and other libraries can assemble required functionality out of lower level modules. (e.g. request doesn't care about submit and all the http dances.
I think we can start creating wiki page for all desired type conversions, so it will be clear for the users what goes where and why.
Thank you everybody for expressing your opinions and being patient with my aeons old petrified brain. :)

@alexindigo So are you thinking this is @2.x fix? Or is this something we should apply to rc4? It's a simple fix and since previously it just errored it should be additive so it might not be bad to include pre 2.0.

I think we should error in 1.x and mimic browser's FormData in 2.0.
Just to clarify, I see 1.0 as stepping stone towards the future mostly focusing on cleaning up tech debt and old deps.
So it will be the same as before but better inside. Like Mountain Lion to OSX. :)
And we'd adding features and split into pieces in 2.0.

@alexindigo what are your thoughts on adding a 2.0 want list? I'd love to see this future as soon as possible. (Casting values to strings by default).

The README still says that boolean is accepted and MDN still says that FormData.set() will convert to string. But this library still explodes, behaving differently than browsers are documented to. Since this is a polyfill to support running browser code outside of the browser, can it be updated to behave like the browser?
Thanks.


explain the purpose of Pipeline Health Trends Up or Down arrows

In Pipeline Health Trends we find Arrows(Up & Down) followed by percentage values. Could you please explain the purpose of Up and Down arrows and it's colors.

The arrows are for showing trend. Up is going high. Down is going low. Some UPs are bad and hence colored red. Some DOWNs are good and hence colored green..
Green = good. Red = bad. Up = trending up in value (% or count etc). Down = trending down.

Thanks for your help


Bail out from launch for unresolvable variables

Typical case:

have a default launch config that uses a ${file} variable for the program attribute
close all editors in VS Code
press F5

Observe: you get the follwoing error:

I will add an option in the configurtionResolverService if something can not be resolved to bail out. Tasks will continue behaving like they do now. @dbaeumer let me know if in debt week you want me to turn this on for tasks as well so we get some user feedback and figure out if this is an improvement

I suggest that the configurationResolverService just throws an error and that the debug launch UI turns that into an appropriate alert, e.g. something helpful for ${file}: "no editor open" and for ${workspaceFolder}: "folder qualifier missing".

We will only flag as errors our pre defined list of variables. We do not touch extension introduced variables.
All error messages will come from the configurationResolverService since those errors are true for whatever client of the service, they are not debug specific. Also I do not want to be in the error massaging business in debug land.

I have added nice error message (can be seen in the attached commit)
Feedback on wording is very welcome.
env variables will not throw an error if they can not be found
@dbaeumer please note that I have changed this behavior for all configurationResolverService clients including Tasks. Let's try it out for a couple of days and if we decide that Tasks want it to not bail out with an error I will add an option. But first wanted to try to be consistent.

Works for launch configs in user settings but nowhere else.
Yes, in my example I was referring to a "default launch config" but the feature should work everywhere.
In addition the error notification should be modal, e.g. like the following:

@weinand I have pushed a change that an unkown variable also throws an error.
For using model dialogs that is a bit larger change and I would ask you to please create a debt item for me.
In some cases we do use modeal dialogs, but in this partiucalr case no

@isidorn I'm not sure whether your fix for unknown variables is really a good idea: above you said

We will only flag as errors our pre defined list of variables. We do not touch extension introduced variables.

We'll have to be careful not to block any launch config that contains a variable that you cannot resolve, but which is privately used by a DA.
My issue from above was actually something else: I got the impression that your bailing out for a unresolvable "${file}" would only work in a default launch config coming from the user settings, but not in a normal launch config from a launch.json.
But I just tried it again and I'm no longer seeing the problem.
Bottom line:

don't flag "unkown variables" too eagerly (or just revert that fix),
fix the missing quotes
try to make the error modal now (but if this is too complex do it in April).

Makes sense.

Reverted
Fixed quotes
Will look into later, if I do not have time will craete an issue for april

Created https://github.com/Microsoft/vscode/issues/46870


Keywords aren't be used on entity tile screen for the search

Need to add keywords to the search, where available.

Created a common script function to filter items based on name and keywords.


made tower_of_hanoi game

PR Description 📜
Fixes # 51 (Tower-Of_Hanoi )
Summary:
This PR resolves issue #51, which involved creating a new game for Tower of Hanoi. The Tower of Hanoi game is now implemented using HTML, CSS, and JavaScript. The game allows players to move disks between towers following the rules of the puzzle. The PR also includes necessary styling and event handling for a smooth gaming experience.
Description:
This PR addresses issue #51, which involved the development of a new game for Tower of Hanoi. The game is now fully functional and interactive. Users can click on the towers to move disks, ensuring that larger disks are not placed on top of smaller ones.
The code follows best practices, adheres to proper indentation, and includes appropriate comments for clarity. The HTML structure is well-organized, the CSS styling enhances the visual appeal, and the JavaScript logic accurately handles the disk movements and win condition.
Overall, this PR successfully addresses issue #51, adding the Tower of Hanoi game as a new feature to the project. It provides an enjoyable gaming experience and serves as an engaging addition to the project's functionality.

Mark the task you have completed ✅

✅ I follow CONTRIBUTING GUIDELINE & CODE OF CONDUCT of this project.
✅ I have performed a self-review of my own code or work.
✅ I have commented my code, particularly in hard-to-understand areas.
✅ My changes generates no new warnings.
✅ I have followed proper naming convention showed in CONTRIBUTING GUIDELINE
✅ I have added screenshot for website preview in assets/images
✅ I have added entries for my game in main README.md
✅ I have added README.md in my folder
❌ I have added working video of the game in README.md (optional)
✅ I have specified the respective issue number for which I have requested the new game.

Add your screenshots(Optional) 📸

Thank you soo much for contributing to our repository 💗

Is there any problem with the game??

@kunjgit, Is there any problem with the game ??

add entries in main README file

Done 🙂, Please review and tell me if there is any other problem.

do that change and create a PR again 😃
everything in one time only !!

Thank you  @aasn0119 , for your valuable time and contribution in our GameZone 💗.
It’s our GameZone, so Let’s build this GameZone altogether !!🤝
Hoping to see you soon with another PR again 😇
Wishing you all the best for your journey into Open Source🚀


[galactic] backport test memory usage improvements

Backport two test CPU/memory improvements to get Galactic debbuild to turn over.

I wonder of precompiled headers could also be used to optimize this process? It's also possible that it would make memory usage worse - I have little experience with PCH.

I think we could try it.  I seem to recall there being cross-platform issues with PCH in the past, so I haven't looked at them recently.


move away from R drive

yep - got a FTP server for kevin for processing, use archivesdirect source storage for transfers
R drive shouldn't be used for anything.


Add build script

This build script allows capng to be linked statically
to libcap-ng and/or select the library's location.
To set the link type the build script will check the
env. variable LIBCAPNG_LINK_STATIC, if its value is
'1' or 'true' the 'static' link type is selected, in
other case the dynlib is selected.
LIBCAPNG_LIB_PATH= will add the  directory
to the compiler's library search path (i.e., -L )
Signed-off-by: German Maglione<EMAIL_ADDRESS>

I think it would be better to use pkg-config, so the user doesn't have to manually specify library paths (they'd just set PKG_CONFIG_ALL_STATIC if they wanted static builds).  This is the approach lots of other Rust crates take.  I've opened #3 to demonstrate this approach.

I think it would be better to use pkg-config, so the user doesn't have to manually specify library paths (they'd just set PKG_CONFIG_ALL_STATIC if they wanted static builds). This is the approach lots of other Rust crates take. I've opened #3 to demonstrate this approach.

Will using pkg-config also allow linking to a libcap-ng compiled from source code (non-system-wide installed)?

Will using pkg-config also allow linking to a libcap-ng compiled from source code (non-system-wide installed)?

I respond to myself, the pkg-config crate is just a wrapper for the pkg-config tool, so no issues here (idk why I thought it was something else)

I think it would be better to use pkg-config, so the user doesn't have to manually specify library paths (they'd just set PKG_CONFIG_ALL_STATIC if they wanted static builds). This is the approach lots of other Rust crates take. I've opened #3 to demonstrate this approach.

Were you able to compile statically in rust:alpine?

Here's a Dockerfile that builds statically using pkg-config:
FROM rust:alpine
RUN apk add autoconf automake gperf libc-dev libtool linux-headers make \
    pkgconf python3 wget
RUN wget https://github.com/seccomp/libseccomp/archive/v2.5.3/libseccomp-2.5.3.tar.gz \
    && tar xf libseccomp-2.5.3.tar.gz \
    && cd libseccomp-2.5.3 \
    && autoreconf -i \
    && ./configure --enable-static --disable-shared --prefix=/opt/static \
    && make \
    && make install
RUN wget https://people.redhat.com/sgrubb/libcap-ng/libcap-ng-0.7.10.tar.gz \
    && tar xf libcap-ng-0.7.10.tar.gz \
    && cd libcap-ng-0.7.10 \
    && ./configure --enable-static --disable-shared --prefix=/opt/static \
    && make \
    && make install
RUN wget https://gitlab.com/ghm-virtio-fs/virtiofsd/-/archive/1b583198d62d2acad810b7e89f373280aadbe743/virtiofsd-1b583198d62d2acad810b7e89f373280aadbe743.tar.bz2 \
    && tar xf virtiofsd-1b583198d62d2acad810b7e89f373280aadbe743.tar.bz2
WORKDIR virtiofsd-1b583198d62d2acad810b7e89f373280aadbe743
RUN printf '[patch.crates-io]\ncapng = { git = "https://github.com/alyssais/capng", branch = "pkg-config" }\n' >> Cargo.toml \
    && PKG_CONFIG_ALL_STATIC=1 PKG_CONFIG_PATH=/opt/static/lib/pkgconfig cargo build --target x86_64-unknown-linux-musl

I actually hadn't tried this before, which is why I took a little while to respond.  I usually build my virtiofsd with Nix, and had tested all the musl/static stuff that way, since it's what I know.

Thank you both, @alyssais and @germag. I've been playing with both #2 and #3. I like the latter more, but I was a bit puzzled that it was able to make link libseccomp-sys without passing LIBSECCOMP_LIB_PATH nor LIBSECCOMP_LINK_TYPE. Turns out that works because both static libraries are placed in the same directory (/opt/static/lib), so the extra args added by pkg-config worked for both libraries.
Knowing this, I think I'm more inclined towards merging #2, for consistency with libseccomp-sys and to avoid adding an additional dependency.

v2:

Rename the env variable from LIBCAPNG_LINK_STATIC to LIBCAPNG_LINK_TYPE to be consistent with libseccomp-sys

LGTM, thanks @germag and @alyssais !


Support Azure Managed Identities for download blob

Similar to #1503 feature request.
When using Azure storage account file share to store source code, we should use blob url with sas token, Currently, kpack doesn't support fetch blob with keychains, it should be supported.
See the reference about Azure Files identity-based authentication options for SMB access

Talked to @zhoufenqin on another forum, the main thing asked here is the ability to use workload identity (i.e. service account identity) to access Iaas specific blob stores. What we came up with is an additional field to the source configuration
source:
  blob:
    url: "blah.blob.azure.com/1234"
    auth: "" | "helper" | "secret"

"" means no auth performed
secret will do the stuff in #1503
helper will delegate to IaaS specific SDKs to get an oauth2 token, which it will use as a bearer token for the GET request.
For now I'll implement a GCP and Azure helper (purely because those are the only 2 environments I have access to test on). But the cred interface should be simple enough for anyone to implement and contribute.

Would azidentity.DefaultAzureCredentials work instead of azidentity.ManagedIdentityCredential? It looks it's the recommended approach by the SDK and will fall back to Managed Identity if none of the other options work.

appr

yes, it works

@chenbh  Do you know which kpack version will release this feature, I didn't see the version tag in PR

@zhoufenqin this feature was released in v0.14.0, but you probably want v0.14.1 which has the fix in #1637


Added Help info for the 'tanner' command

*title says all
Just thought it would be helpful for users to understand the command without having to look at the code

closing request
Apologies, I have the description wrong


changed Parse class to be an interface and added getter/setter for it in...

I wish to support messages that have a string payload, but the current Parser class throws an exception if it is not of type OBJECT.
Is there a real reason for this exception?
I would propose transforming the Parser class into an interface. I created  a DefaultParser class that is used in SocketIOComponent.Awake(), and also added a getter/setter for the parser so that the user can override it with a custom class if his own.
How do you find these changes?
Thanks

I don't like these changes, let me explain why:

They don't provide a feature, nor solve the problem you present.
If anybody downloads the library with these changes merged, there will be no difference with the existing code base. In order to use this solution, new users need to know that there is an extension point here, that solves this particular use case.

It's not documented on the standard socket.io parser library documentation. Maybe I missed it and you can point it out, but the original implementation attemped to be almos 1:1 with the javascript library to smooth the transition. Merging this sounds as it might add an obscure solution that might actually create unexpected bugs.

Creating an interface, implementing it, injecting it on creation sounds like a good Java solution. That's how you would normally implement something with Spring. It's not a good solution in this case. As I mentioned above, it only adds a layer of complexity but doesn't add new fuctionality.

If you need to to send a plain string, the best solution is to wrap it client side in a simple object. In order to do that you only need a simple function that takes the string, and returns it wrapped in an object. Then use that function in the class that you are using to handle your connection.
If you call this object from multiple classes, then you might want to create a helper class with a static function that wrap strings in object.
Thanks a lot for the contribution, please take in mind that the parser is implemented to match the socket.io 1.X specification.

Thank you also for your reply.
The change can solve the problem I am having. I am able to create my own Parser class which I can use instead of the one provided until now.
The only thing I found in the socket.io protocol related to arguments data type is this paragraph below. As I understand it, it says that the arguments can be any type that can be encoded/decoded with JSON.

EVENT
data (Array) a list of arguments, the first of which is the event name. Arguments can contain any type of field that can result of JSON decoding, including objects and arrays of arbitrary size.
id (Number) if the id identifier is present, it indicates that the server wishes to be acknowledged of the reception of this event.

It is taken from this page https://github.com/automattic/socket.io-protocol. If haven't found other protocol description document.
Thanks


Fix Sandcastle jsHint

We were writing out jsHintOptions with the extends attribute, which wasn't working when loaded into the browser.  Now the build process loads the primary jshint file and modifies is to match the gallery in order to write out a complete jsHintOptions object.
This could probably be a little more robust (since not all changes to Apps/Sandcastle/.jshintrc get reflected automatically jsHintOptions, but I doubt that will be an issue given that these files will rarely change.
Fixes #3267

All fixed!  Thanks @mramato! =)


Revise message for ZeroDivisionError when source is not available.

Consider the following:
> python -m friendly_traceback ignore.py

Traceback (most recent call last):
  File "HOME:\github\friendly-traceback\ignore.py", line 2, in <module>
    exec(code)
  File "fake_file", line 1, in <module>

ZeroDivisionError: division by zero

    A `ZeroDivisionError` occurs when you are attempting to divide a value
    by zero either directly or by using some other mathematical operation.

    The following mathematical expression includes a division by zero:

    Execution stopped on line 2 of file HOME:\github\friendly-traceback\ignore.py.

       1: code = compile("1/0", "fake_file", "exec")
    -->2: exec(code)
          ^^^^^^^^^^

            code:  <code object <module>>
            exec:  <builtin function exec>

    Exception raised on line 1 of file fake_file.

There is no mathematical expression that is shown!

New version:
    A `ZeroDivisionError` occurs when you are attempting to divide a value
    by zero either directly or by using some other mathematical operation.

    The following mathematical expression includes a division by zero:

        <'expression not found'>

    Execution stopped on line 2 of file HOME:\github\friendly-traceback\ignore.py.

       1: code = compile("1/0", "fake_file", "exec")
    -->2: exec(code)
          ^^^^^^^^^^

            code:  <code object <module>>
            exec:  <builtin function exec>

    Exception raised on line 1 of file fake_file.

fake_file is not a regular Python file whose contents can be analyzed.

I used quotes in addition to <...> for nicer formatting with friendly (using Rich).


AutoGraph unexpected indent in tf-nightly-gpu-2.1.0.dev20191103

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): Yes
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Linux Ubuntu 16.04
TensorFlow installed from (source or binary): binary
TensorFlow version (use command below): tf-nightly-gpu-2.1.0.dev20191103
Python version: 3.5.2
CUDA/cuDNN version: 10.0/7.1
GPU model and memory: GTX 1080 Ti

Describe the current behavior
After upgrading to tf-nightly-gpu-2.1.0.dev20191103 from tensorflow-gpu-2.0.0, I obtained this error when running my code:
WARNING:tensorflow:AutoGraph could not transform <bound method CRFLayer.mean_field of <models.crf_layer.CRFLayer object at 0x7f6124237b00>> and will run it as-is.
Please report this to the TensorFlow team. When filing the bug, set the verbosity to 10 (on Linux, `export AUTOGRAPH_VERBOSITY=10`) and attach the full output.
Cause: unexpected indent (<unknown>, line 36)

I did export AUTOGRAPH_VERBOSITY=10 but did not observe any other types of output other than the above. It says unexpected indent so I guess something changed in the parsing of Python code when building the graph? The problem does not occur on tensorflow-gpu-2.0.0 (I installed tf-nightly-gpu-2.1.0.dev20191103 because I need to be able to load_weights(pretrained_weights, by_name=True, skip_mismatch=True), which is not available in 2.0.0. There is another bug with this function that I will report in a separate issue.)
Describe the expected behavior
Like in TF 2.0.0: no AutoGraph warning.
Code to reproduce the issue
Unfortunately my code has a lot of dependencies and I was unable to create a minimal reproducible example. But from the warning message I guess it's easy enough to check in the source code.

@netw0rkf10w ,
Will you be able to provide Colab link with code you are trying to execute ?Thanks!

Hi @oanush. Unfortunately this code has a custom TF op in C++ that needs to be compiled. I'm not sure it's possible to do that in Colab. Let me know if sharing a zip file of source code is ok.

@netw0rkf10w Please share the zip file of source code. Thanks!

@netw0rkf10w Can you please share the source code?

@gowthamkpr Sorry for the delay, I have been overwhelmed over the last days. I'm preparing the code.

Closing this issue as it has been inactive for more than 2 weeks. Please add additional comments and we can open this issue again. Thanks!

We have observed similar issues with TF 2.1-rc0. Cannot share source code, but hopefully I could get a minimal reproducible case.
WARNING:tensorflow:AutoGraph could not transform <bound method TransformerDecoder._transform of <bytedseq.decoders.transformer_decoder.TransformerDecoder object at 0x7f727cc9e390>> and will run it as-is.
Please report this to the TensorFlow team. When filing the bug, set the verbosity to 10 (on Linux, `export AUTOGRAPH_VERBOSITY=10`) and attach the full output.
Cause: unexpected indent (<unknown>, line 31)
W1219 06:50:06.045258<PHONE_NUMBER>01632 ag_logging.py:146] AutoGraph could not transform <bound method TransformerDecoder._transform of <bytedseq.decoders.transformer_decoder.TransformerDecoder object at 0x7f727cc9e390>> and will run it as-is.
Please report this to the TensorFlow team. When filing the bug, set the verbosity to 10 (on Linux, `export AUTOGRAPH_VERBOSITY=10`) and attach the full output.
Cause: unexpected indent (<unknown>, line 31)

Gently ping @goldiegadde; anything looks familiar?

Gently ping @mdanatg; does this look familiar to you?

These unexpected indent errors are new to me, but it looks like a regression. Setting AUTOGRAPH_VERBOSITY=10 should have given you a whole lot of extra logging. Since autograph outputs to the abseil logger, try adding this somewhere early in your code:
from absl import logging
logging.set_verbosity(logging.INFO)

In the mean time, could you visually inspect the source code of one of the functions that it complains about? Like e.g. TransformerDecoder._transform - does it contain any thing out of the ordinary, like mixing tabs with spaces, backslash \ continuations or anything like that?
As a side note, to get a minimal repro, could you also try the following command: tf.autograph.to_code(TransformerDecoder._transform). It should allow us to test things outside of your main program, even with unbound variables or missing arguments.

Also having this problem on a fairly innocuous piece of code:

def reverse_edges(edges):
    """
    Reverse edges to point from left_node -> right_node to
    right_node -> left_node

    :param edges: dict(('layer', 'idx')), edges

    :return reversed_edges: dict(('layer', idx')), reversed edges
    """
    llayer = edges['layer'][0, 0]
    rlayer = edges['layer'][0, 1]
    reversed_edges = {
            'layer': tf.constant([[rlayer, llayer]]),
            'idx': reverse_edge_idxs(edges['idx']),
            }
    return reversed_edges

def key_to_layers(key):
    """
    Decodes a key into layer numbers.

    :param key: str, node or edge key

    :return llayer: str, left or sender layer number
    :return rlayer: str, right or receiver layer number
    """
    if '_to_' in key:
        llayer, _, rlayer = key.split('_')
    else:
        llayer = rlayer = key.split('_')[-1]
    return llayer, rlayer
        # Reverse edges do not have to be loaded but can be constructed
          if not load_reverse_edges:
            for k in list(edges.keys()):
                llayer, rlayer = key_to_layers(k)
                if llayer != rlayer:
                    edges[connect_key(rlayer, llayer)] = gbuild.reverse_edges(edges[k])
                    edges.pop(k)

where edges is a dictionary containing tensors as values. Not a complete example, but maybe might help isolating the issue?

@mjlbach I tried your code by passing them directly to autograph, e.g. print(tf.autograph.to_code(build_edges)). I'm not seeing any errors in Python 3.7; I'll try with 3.5 shortly.
If you have the chance, could you try the same test on the function mentioned in the warning message?

Ok, the two errors are:
WARNING:tensorflow:AutoGraph could not transform <bound method ParticleGraphConstructorFromRecords.construct_hierarchy_from_records of <boxphysics.model.fromrecords.BoxGraphConstructorFromRecords object at 0x7fbbf055fd68>> and will run it as-is.
Please report this to the TensorFlow team. When filing the bug, set the verbosity to 10 (on Linux, `export AUTOGRAPH_VERBOSITY=10`) and attach the full output.
Cause: unexpected indent (<unknown>, line 91)
WARNING:tensorflow:AutoGraph could not transform <bound method ParticleGraphConstructorFromRecords.remove_invalid_parts_by_nodes of <boxphysics.model.fromrecords.BoxGraphConstructorFromRecords object at 0x7fbbf055fd68>> and will run it as-is.
Please report this to the TensorFlow team. When filing the bug, set the verbosity to 10 (on Linux, `export AUTOGRAPH_VERBOSITY=10`) and attach the full output.
Cause: expected an indented block (<unknown>, line 27)

If I try to run autograph_to_code on the first one, I get:
*** IndentationError: unexpected indent
Traceback (most recent call last):
  File "/usr/local/lib/python3.6/dist-packages/tensorflow_core/python/autograph/impl/api.py", line 831, in to_code
    experimental_optional_features=experimental_optional_features))
  File "/usr/local/lib/python3.6/dist-packages/tensorflow_core/python/autograph/impl/api.py", line 658, in to_graph
    return conversion.convert(entity, program_ctx)
  File "/usr/local/lib/python3.6/dist-packages/tensorflow_core/python/autograph/impl/conversion.py", line 359, in convert
    free_nonglobal_var_names)
  File "/usr/local/lib/python3.6/dist-packages/tensorflow_core/python/autograph/impl/conversion.py", line 274, in _convert_with_cache
    entity, program_ctx)
  File "/usr/local/lib/python3.6/dist-packages/tensorflow_core/python/autograph/impl/conversion.py", line 508, in convert_entity_to_ast
    nodes, name, entity_info = convert_func_to_ast(o, program_ctx)
  File "/usr/local/lib/python3.6/dist-packages/tensorflow_core/python/autograph/impl/conversion.py", line 668, in convert_func_to_ast
    node, source = parser.parse_entity(f, future_features=future_features)
  File "/usr/local/lib/python3.6/dist-packages/tensorflow_core/python/autograph/pyct/parser.py", line 209, in parse_entity
    return _attempt_to_parse_normal_source(source, future_features)
  File "/usr/local/lib/python3.6/dist-packages/tensorflow_core/python/autograph/pyct/parser.py", line 113, in _attempt_to_parse_normal_source
    return parse(source, preamble_len=len(future_features)), source
  File "/usr/local/lib/python3.6/dist-packages/tensorflow_core/python/autograph/pyct/parser.py", line 226, in parse
    module_node = gast.parse(src)
  File "/usr/local/lib/python3.6/dist-packages/gast/gast.py", line 295, in parse
    return ast_to_gast(_ast.parse(*args, **kwargs))
  File "/usr/lib/python3.6/ast.py", line 35, in parse
    return compile(source, filename, mode, PyCF_ONLY_AST)
  File "<unknown>", line 91
    if not load_reverse_edges:

On the second:
*** IndentationError: expected an indented block
Traceback (most recent call last):
  File "/usr/local/lib/python3.6/dist-packages/tensorflow_core/python/autograph/impl/api.py", line 831, in to_code
    experimental_optional_features=experimental_optional_features))
  File "/usr/local/lib/python3.6/dist-packages/tensorflow_core/python/autograph/impl/api.py", line 658, in to_graph
    return conversion.convert(entity, program_ctx)
  File "/usr/local/lib/python3.6/dist-packages/tensorflow_core/python/autograph/impl/conversion.py", line 359, in convert
    free_nonglobal_var_names)
  File "/usr/local/lib/python3.6/dist-packages/tensorflow_core/python/autograph/impl/conversion.py", line 274, in _convert_with_cache
    entity, program_ctx)
  File "/usr/local/lib/python3.6/dist-packages/tensorflow_core/python/autograph/impl/conversion.py", line 508, in convert_entity_to_ast
    nodes, name, entity_info = convert_func_to_ast(o, program_ctx)
  File "/usr/local/lib/python3.6/dist-packages/tensorflow_core/python/autograph/impl/conversion.py", line 668, in convert_func_to_ast
    node, source = parser.parse_entity(f, future_features=future_features)
  File "/usr/local/lib/python3.6/dist-packages/tensorflow_core/python/autograph/pyct/parser.py", line 209, in parse_entity
    return _attempt_to_parse_normal_source(source, future_features)
  File "/usr/local/lib/python3.6/dist-packages/tensorflow_core/python/autograph/pyct/parser.py", line 113, in _attempt_to_parse_normal_source
    return parse(source, preamble_len=len(future_features)), source
  File "/usr/local/lib/python3.6/dist-packages/tensorflow_core/python/autograph/pyct/parser.py", line 226, in parse
    module_node = gast.parse(src)
  File "/usr/local/lib/python3.6/dist-packages/gast/gast.py", line 295, in parse
    return ast_to_gast(_ast.parse(*args, **kwargs))
  File "/usr/lib/python3.6/ast.py", line 35, in parse
    return compile(source, filename, mode, PyCF_ONLY_AST)
  File "<unknown>", line 27
    llayer, rlayer = key_to_layers(k)

I still have this issue with TF2.1 and you can find my workaround here


[Storage] Make sure API extractor output is clean

Package Name: storage-blob, storage-datalake, storage-file, storage-queue (any of these packages going live for preview)

Describe the bug
Ensure that:

[x] API Extractor is running for each shipping package
[ ] The resulting .d.ts file is being included in the package
[x] The report file is added to the repo
[ ] The report file is clean of warnings

PR is ready, but pushing to next milestone to not disrupt the incoming storage release


Standardize backbone argument in model method

🐛 Bug
Describe the bug
When creating a torchvision model, it fails because backbone argument is passed as function, and model() method expects an nn.Module. The backbone method need to be called before it is passed to the model() method

Another suggestion:
Make all backbones callable, i.e. instead of models.mmdet.retinanet.backbone.resnet18 directly returning a MMDetBackboneConfig it can instead return a function that when called returns a backbone config. Same applies for effdet


Software Dev: Example code on LinkedList in C

Manipulations with LinkedList data structure in C.
@csendranshi @clare0901

There have been similar contributions earlier on linkedlist in c. Please make sure there's no redundant codes.

Please assign this also, yes there are different objectives to it.


add teams color

add the main colors of all teams to the database.

We're doing gradient with another strategy. Thanks for the PR!


Error messages don't appear on iOS?

Unsure whether this is user error / misunderstanding on our part, but how fully does jqBootstrapValidation support mobile and iOS? On our site, viewing in iPhone Safari on recent iOS, when we try to submit an invalid form (a required field is blank), the submission is blocked but no error message appears.
Is this worth us debugging more, or are there known issues with iOS?

Seeing the same issue here... #182

Any body have a solution to this or issue #182 ?

@crabstance None for me so far. Had to move to another JS library in the meantime. Still unsure why it works on the demo site but not on production.

Thanks for the quick response! I will do the same.
On Thursday, December 17, 2015, jeff-mk<EMAIL_ADDRESS>wrote:

@crabstance https://github.com/crabstance None for me so far. Had to
move to another JS library in the meantime. Still unsure why it works on
the demo site but not on production.
—
Reply to this email directly or view it on GitHub
https://github.com/ReactiveRaven/jqBootstrapValidation/issues/167#issuecomment-165590675
.


False positives when searching for caffeinate process

When invoking workflow search command
ps -eo etime,args|grep -w caffeinate|grep -v grep|sed -e 's|^[[:space:]]*||'
on my system, I get following results:
16:39 /bin/bash /Users/schrodinger_cat/Library/Scripts/com.plexapp.plexmediaserver.caffeinate.sh
05:41 /Applications/Alfred 3.app/Contents/Preferences/Alfred Preferences.app/Contents/MacOS/Alfred Preferences /Users/schrodinger_cat/Downloads/caffeinate_control.alfredworkflow
00:07 caffeinate -s -t 300
The problem is, this gives false positive results when searching, therefore workflow is always showing that caffeinate is activated and prevents enabling of caffeinate command.

Good catch, and thanks for the PR. Merged.


feat(SelectPanel): Remove sxProp

Closes https://github.com/github/primer/issues/4264

Changelog

Removed

Remove SxProp from SelectPanel because there are no results on Primer Query: https://primer-query.githubapp.com/?attribute="sx"&name=selectpanel
Rollout strategy

[ ] Patch release
[x] Minor release
[ ] Major release; if selected, include a written rollout or migration plan
[ ] None; if selected, include a brief description as to why

Testing & Reviewing

Merge checklist

[ ] Added/updated tests
[ ] Added/updated documentation
[ ] Added/updated previews (Storybook)
[ ] Changes are SSR compatible
[x] Tested in Chrome
[ ] Tested in Firefox
[ ] Tested in Safari
[ ] Tested in Edge
[ ] (GitHub staff only) Integration tests pass at github/github (Learn more about how to run integration tests)

I think we want to hold off on merging and change the changeset to major per slack convo (hubbers only)


Major update

Fixed issue with Noble integration
Moved to @abandonware/noble
Fixed error with crashing when WinRT fails to return name
Updated outdated libraries
Other fixes

Need to get this merged as its the best version out there so far. Also would be nice to convert the code to work against the latest Windows 10 SDK.

Need to get this merged as its the best version out there so far. Also would be nice to convert the code to work against the latest Windows 10 SDK.

@tony-gutierrez @geovie I've updated packages, and fixed specified issues.
Appologies for the delay.

+1 for getting this code merged.


fix some yellow alert in shenyu-admin module

// Describe your PR here; eg. Fixes #issueNo

Make sure that:

[x] You have read the contribution guidelines.
[x] You submit test cases (unit or integration tests) that back your changes.
[x] Your local test passed mvn clean install -Dmaven.javadoc.skip=true.

Codecov Report

Merging #2355 (9aa1f0e) into master (2f5cc73) will decrease coverage by 0.81%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##             master    #2355      +/-   ##
============================================
- Coverage     58.20%   57.38%   -0.82%
+ Complexity     3955     3919      -36
============================================
  Files           629      629
  Lines         18061    18061
  Branches       1695     1694       -1
============================================
- Hits          10512    10365     -147
- Misses         6716     6870     +154
+ Partials        833      826       -7

Impacted Files
Coverage Δ

...ache/shenyu/admin/exception/ExceptionHandlers.java
5.55% <0.00%> (ø)

...pache/shenyu/admin/listener/etcd/EtcdDataInit.java
91.66% <ø> (ø)

...rvice/converter/SelectorHandleConverterFactor.java
75.00% <ø> (-25.00%)
:arrow_down:

...apache/shenyu/admin/transfer/MetaDataTransfer.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...rg/apache/shenyu/admin/utils/ThreadLocalUtils.java
0.00% <0.00%> (-66.67%)
:arrow_down:

...rvice/converter/DivideSelectorHandleConverter.java
28.57% <0.00%> (-28.58%)
:arrow_down:

...che/shenyu/sync/data/http/HttpSyncDataService.java
63.10% <0.00%> (-26.22%)
:arrow_down:

...apache/shenyu/admin/utils/CommonUpstreamUtils.java
0.00% <0.00%> (-14.29%)
:arrow_down:

...yu/sync/data/http/refresh/AbstractDataRefresh.java
70.00% <0.00%> (-13.34%)
:arrow_down:

... and 9 more

Continue to review full report at Codecov.

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


Options -v -d

These options appear in the a REAME example but doesn't seem to be any more information about them.

Ok, -d is --debug for source-maps, what about -v?

Are you talking about here?  That's just an example of specifying Browserify options.

Then -v is not documented in browserify?

I think that used to be an option in an older version of Browserify

What did it do?

dunno.  it's just an example, so I just picked a few random arguments from the Browserify README

Thanks the -v is the --verbose option for --watch


Fix RayDeviceQuantileDMatrix

RayDeviceQuantileDMatrix became incompatible with recent changes. This PR makes sure it works again.
Please note that we currently have no general support to load from files into GPU memory directly, avoiding CPU memory. Thus the RayDeviceQuantileDMatrix works with conversion (or when cupy arrays are provided directly). In the future we might want to introduce interfaces for loading from files into GPU memory.
Closes #65

Please note that we currently have no general support to load from files into GPU memory directly.

That will require direct access to external memory from GPU.  I think it's best for a 3-party library to carry out this kind of task.  WDYT?

Yes @trivialfis I agree loading from files directly into GPU memory is a task for some other library.
What we could do in this integration though is that currently when we're loading from multiple CSV/Parquet files, we're constructing pandas dataframes in CPU memory and concatenate them. When creating device quantile dmatrices, we convert the (usually full) dataframe to cupy arrays in the iterator and pass them to the device matrix.
Instead we could not create pandas dataframes, but create cupy arrays to begin with instead. We would then not hold a copy of the full data in CPU memory. For construction, data will still flow through CPU memory but only until the cupy array is built.
Does that make sense?

Thanks for the explanation, yeah that make sense.


Fixes #102 : Add a suggestion list

CORS Solved, Suggestion List is added to Fearch.

CORS solved by just removing those comments? How come it is true? @AvinashAgarwal14 Can you elaborate/provide the lines of code that actually solved this because I too have this issue.

No,CORS was solved by adding "content_security_policy":"script-src 'self' https://www.google.com/; object-src 'self'" in manifest,json file.
Actually, those comments were unnecessary that's why I removed them.

@AvinashAgarwal14 Thanks for your help. It would be better if you could give the commit id, file name and line number.

Okay.
Commit ID - 584d9b4116349456ffe695b7b43c7ddc52a471c3
Filename - manifest.json
line number - 3,4,5,412

Hi Avinash, you have done a really great job. Sorry, for addressing it a little bit late. We are planning to release a new version of Fearch by next week. It would be really great to have your PR as a part of that release. I will suggest a few changes it would be awesome if you can update the PR and have it be live for 1200+ users we have right now 😄

Hi, thank you it would be awesome.
What changes am I supposed to do?

These things would suffice -

Can you add autocomplete="off" on the search input field. Right now I am getting my input history being overlayed on the suggestions provided. I hope this should fix this problem.
Please, can you help me understand why did you include jQuery and what problem is it solving? Any of the operation you perform using it can be done by vanilla javascript as such.

$(document).ready(function(){
    $(":not(#query)").click(function(){
        for(var i=0;i<(sug.length);i++)
                {
                    sug[i].classList.add("off");
                }
    });
});

If it is just for this piece of code. I think adding a library is not justified. You can write it in normal javascript. Thankyou

Yes,

autocomplete="off" will do that.
This code is for hiding the suggestion list when the user clicks anywhere out of the input-field box but not when he clicks inside the input box.As selecting all elements other than a specified one easier with jquery :not selector, I used it. Anyways, will look for an alternative solution and inform you.

Thankyou. Let me know if you need help with something.

I have updated the PR with the required changes, please check.

Great work @AvinashAgarwal14
Thanks @nkprince007 @Compro-Prasad  for the help with suggestions.


fix: added meta title property for search engines

Fixes issue #68
'title' meta property added to the root html file

@rrahul32 Thank you so much for your contribution 😊. I'm glad we have you as one of our contributor. I would love it if you can stay and contribute more in future issues.


Backend Mode: Tunnel

The initial implementation of --backend-mode tunnel (#170).

--backend-mode tunnel has become --backend-mode tcpTunnel. The groundwork for a future --backend-mode udpTunnel has also been established in the OpenAPI spec and the database schema.


Cannot see Hero images on my posts after theme update

Expected Behavior
Post images should appear
Current Behavior
In my personal website, after the last theme update, I cannot see images for the posts, unless they are located in /images.
Maybe there is an error in configuration in my part? I just dont know what to do :+
Steps to Reproduce

git clone https://codeberg.org/FlyingFlamingo/pablomarcos-me/
cd pablomarcos-me/
hugo serve -w

Screenshots

Browsers Affected

[x] Chrome
[x] Firefox
[ ] Edge
[ ] Safari 11
[ ] Safari 10
[ ] IE 11

Detailed Description
EDIT: It seems like the issue is related to the language configuration; if I add index.en.md to a post, the image shows up
EDIT 2: The issue seems to have something to do with the default language, because, if I Set  defaultContentLanguage: de; defaultContentLanguageInSubdir: true, it works back. Is having a main blog lang separated from the main page lang not supported anymore?

We haven't changed anything related Language in recent updates. Probably, the behavior has changed in Hugo itself.

Thank you! So, I am guessing, nothing you can do right?

Yes

Closing then :p


[REQUEST] Running with Windows

Could you make it possible in the settings to launch the hydra when Windows starts💛

In the meantime, you can create a shortcut to the software and put it in C:\Users\<your username>\AppData\Roaming\Microsoft\Windows\Start Menu\Programs\Startup. This will start the program when you login ;)

It will be possible in the next update


az-cli stops working if python project called "azure" in cwd

Summary
The az command (with any args or params) fails with an error when there exists a directory in the CWD called "azure" with another file in it called __init__.py.
To Reproduce:

Navigate to any directory on the commandline.
E.g: cd /tmp
Create a folder called azure, and a file under it called __init__.py.
E.g: mkdir azure; touch azure/__init__.py
Attempt to run any az command.
E.g: az version
Observe the command fail, with the following error:
/usr/bin/python3: No module named azure.cli

Expected behavior
The command should work, regardless of folders.
Environment summary
Installed with: apt install azure-cli
azure-cli                         2.0.81
core                              2.0.81
telemetry                          1.0.4
Extensions:
azure-devops                      0.17.0

Using GNU bash, version 5.0.17(1)-release (x86_64-pc-linux-gnu) on Ubuntu 20.04.1 LTS, 5.4.0-58-generic.

This is an issue because many people may have python projects that contain a folder called azure somewhere within them.

hi @fengzhou-msft could you pls have a look? thanks

hi @fengzhou-msft could you pls have a look? thanks

@AddisonG Please follow the instruction to install the latest azure-cli. In the azure-cli package from microsoft repository, we use -I option of python to invoke azure.cli, it will not prepend the current directory to sys.path and it should fix this issue.


Streaming reader taking Huge CPU to read 100 records.

0
I am trying to upload 100k records from xlsx file and using Excel Streaming Reader to iterate excel It was failing in QA server but made changes
StreamingReader.builder()
.sstCacheSize(100)
.open(new FileInputStream(new File(filePath)));
made it work but it is consuming large CPU resources and other users using particular environment complaining of performance hit.
Any suggestions would appreciate

.open(new File(filePath)) is more efficient - with InputStreams, the stream is read into a temporary file and you avoid this with using the File directly.


[Improvement] Supply OpenApi yaml file

Code of Conduct

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

Search before asking

[X] I have searched in the issues and found no similar issues.

What would you like to be improved?
Currently kyuubi supply api for users, so they can integrate their platform with kyuubi. Can kyuubi supply openapi yaml file ?
Link feature: https://github.com/apache/incubator-streampark/issues/2274 .
How should we improve?
Supply OpenApi yaml file.
Are you willing to submit PR?

[ ] Yes. I would be willing to submit a PR with guidance from the Kyuubi community to improve.
[X] No. I cannot submit a PR at this time.

@GOODBOY008, thanks for report the improvement to support OpenAPI yaml. The support makes sense to me because the OpenAPI yaml is beneficial to integration. cc @pan3793


Precompiled Plugin Scripts: Support Constructor Injection for eg ToolingModelBuilderRegistry

Expected Behavior
// myConventionPlugin.gradle.kts
val toolingModelRegistry: ToolingModelBuilderRegistry by injected

Current Behavior (optional)
Not possible, you have to write a normal plugin:
open class MyConventionPlugin @Inject constructor(
    val toolingModelRegistry: ToolingModelBuilderRegistry
): Plugin<Project> {
  override fun apply(project: Project) {
}

Context
I want to migrate my normal plugins to the precompiled plugins because the syntax aligns with build scripts and the type safe accessors makes writing plugins very comfortable.
But I need to expose a custom object to my IntelliJ plugin during sync and I use the ToolingModelBuilder.

Thank you for your interest in Gradle!
This feature request is in the backlog of the relevant team and is prioritized by them


inspect-file children objects' commit id has strange formatting

A commit id usually has the format branch/index (eg master/1), but for files returned in a directory's children array, the commit id seems to have the format repo_name:branch:index (eg page_views:master:2)
Full response example for inspecting a directory called /tmp:
{
    "children": [
        {
            "commit": {
                "id": "page_views:master:2",
                "repo": {
                    "name": "page_views"
                }
            },
            "path": "/tmp/Screenshot from 2017-03-16 11:23:13.png"
        }
    ],
    "commitModified": null,
    "file": {
        "commit": {
            "id": "master/2",
            "repo": {
                "name": "page_views"
            }
        },
        "path": "/tmp"
    },
    "fileType": "FILE_TYPE_DIR",
    "modified": {
        "nanos": 359499818,
        "seconds": "1490201873"
    },
    "path": "/tmp",
    "sizeBytes": null
}

Repro notes on dashboard:
go to http://localhost:4200/app/repos/page_views/commits/page_views-master%2F2/files//tmp
get "resolving with" message
temp1.encodeJSON()

Interestingly, it seems that child commit id also references the commit in which the file was last modified, not the same commit as that of the parent file

stale

This commit syntax died a while ago so this shouldn't apply anymore.


asyncio.CancelledError not caught in asyncSlot error handler

For coroutines decorated with asyncSlot, if the enclosing task is cancelled, the asyncio.CancelledError is left uncaught. This is a problem if another signal triggers the cancelling of that enclosing task, while it is awaiting for something.
I believe the error handler at line 778 of qasync/__init__.py should handle asyncio.CancelledError with something like this:
    def _error_handler(task):
        try:
            task.result()
        except Exception:
            sys.excepthook(*sys.exc_info())
        except asyncio.CancelledError:
            pass

Note that since Python 3.8, asyncio.CancelledError is a derived class of BaseException but not Exception.

Here's a minimal example:
import asyncio
import qasync
from PyQt6.QtWidgets import QApplication, QPushButton

app: QApplication

@qasync.asyncSlot(bool)
async def button_clicked(_checked):
    match app.button.text():
        case "Wait":
            app.button.setText("Cancel wait")

            app.waiting_task = asyncio.current_task()
            await asyncio.sleep(5)

            app.button.setText("Wait")

        case "Cancel wait":
            app.waiting_task.cancel()

            app.button.setText("Wait")

async def main():
    global app
    app = QApplication.instance()
    app_quit_event = asyncio.Event()
    app.aboutToQuit.connect(app_quit_event.set)

    app.button = QPushButton("Wait")
    app.button.clicked.connect(button_clicked)
    app.button.show()

    await app_quit_event.wait()

qasync.run(main())

The exception is uncaught if "Wait" is pressed, and, while "Cancel wait" is shown, the button is pressed again.


Add release v1.3 benchmarks data

Special notes for your reviewer:
The gray dotted line in the graph is July 6, 2023, denoting release of v1.0.0
The blue dotted line in the graph is September 6, 2023, denoting change in k8s provider from v1.25 to v1.27
The red dotted line in the graph is November 6, 2023, denoting release of v1.1.0
The green dotted line in the graph is March 5, 2024, denoting release of v1.2.0
The violet dotted line in the graph is March 5, 2024, denoting change in k8s provider from v1.27 to v1.29
Checklist
This checklist is not enforcing, but it's a reminder of items that could be relevant to every PR.
Approvers are expected to review this list.
Add release v1.3 benchmarks data

/cc @alaypatel07


Load Cell: Basic Connectivity and Usage

[x] Look up part names and how to connect
[x] Look up code tutorial for my part
[x] Write to smartcabinet.py
[x] Commit and push after testing

[img]https://i.imgur.com/qfHZT9E.png[/img]


[hud hide:YES afterDelay:1.0] not working when use popView

I have code
[hud hide:YES afterDelay:3.0];
[self.navigationController popToRootViewControllerAnimated:YES];
[hud hide:YES afterDelay:3.0] doesn't work.
If I comment the second line,everything is ok.

honestly this helped thank you @matej


Notifications from other programs

Hi,
I'm using K9 for mail client , and I when I got on watch notification from K9 usually this looks like this:
https://drive.google.com/file/d/0BySjzBz--msUeGpaUUtuMVdfS3c/view?usp=sharing
Very rarely I see a title of K-9 notification (new e-mail subject & date), in most K-9 notifications show up like on photo...
I'll support you work only if you add for K-9 good notifications :wink:

Notifications implementation is directly using Android NotificationListenerService


expose two missing EmbedFailureCauses tags to python

The recently added double-bond-related tags were missing.

@ptosco sorry to tag you on two PRs in one day, but this is only two lines


Crypto.Random and forking the DRG

Am I right, that forking the DRG is just not possible with every DRG in a pure way (i.e. without mixing entropy or using new entropy)?
If so, can you comment on how you envision it to be done? Maybe provide the building blocks to "purely" (but not referential transparent) mix entropy to get two generators or (better) provide a function to do that.
I might not have thought this out though.

yes, it's not currently possible for the DRG class itself to fork/split.
You could very easily have a main "seed" generator that would seed temporary generators. i.e. something like this:
let (newDRG, seedgenerator2) = withDRG seedGenerator drgNew

If we had a generic 'drgNew' for the DRG class, then you'ld be able to do for every DRG.
Would that'ld be enough to cover your cases ?

Yes, a drgNew (or newDRG) would be enough.
That said, after porting the code that triggered that issue to use the
MonadRandom class, I don't seem to need it anymore.


Game: Fix prompt buttons going offscreen

Description

Fixed an issue where if there are more than 5 button choices in a dialog prompt, the buttons cut into the dialog box and eventually go offscreen.
Type of change

[x] Bug fix (non-breaking change which fixes an issue)

Checklist

[x] I have tested this code

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 34.841%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
4915

Relevant Lines:
13170

💛  - Coveralls

Can I double check if the prompt ordering that was brought up above has been addressed? What's the status / decision?

Yes, the prompt ordering should go from left to right now.

Thank you so much for the feedback, unless there is anything else of note, I'll go ahead and merge it?


Broken version

Tried to install on Ubuntu 14.04, was getting this error:
STDERR: E: Version '2.2.1' for 'td-agent' was not found

Setting these attributes fixed it:
default[:td_agent][:pinning_version] = true
default[:td_agent][:version] = '2.2.1-0'

There might be some problem when you try installing it. Though I couldn't reproduce.
We're testing cookbooks on Ubuntu 14.04 as well and it is just working fine. It must not be an issue of recipe implementation at least.
https://github.com/treasure-data/chef-td-agent/blob/9a7f3f7306841ab317f02923806b8a9be1d4cb84/.kitchen.yml#L24-L29


Add JDBC "driver" option as optional configuration parameter

Once you configured multiple drivers for the same jdbc prefix, you should be able to explicitly select the driver you want to use. This is currently possible in IMPORT FROM JDBC, but not in hadoop etl udfs that also can use IMPORT FROM JDBC behind the scenes.

https://docs.exasol.com/sql/import.htm

Some JDBC drivers are already delivered as default (visible in EXAoperation) and can be used within the connection string (for example, jdbc:mysql, jdbc:postgres). You can additionally configure JDBC drivers in EXAoperation and choose them through the DRIVER option if its prefix is ambiguous.

https://docs.exasol.com/administration/on-premise/manage_software/manage_jdbc.htm

Driver Name: "driver name"

See also SUPPORT-6101.

Decision from review with PM: shelve ticket.


Added basic domain blacklist option, fixes #9

I added a textarea to the extension options which accepts a comma-delimited list of domain names. This works with full domain names, not with protocols or wildcards. When a user presses the shortcut, we get the href attribute, remove the protocol and path using getDomain(href) and check it against every item in the blacklist[] array.
So, we can block maxcdn.bootstrapcdn.com, but can't block *.bootstrapcdn.com or https://maxcdn.bootstrapcdn.com. This could probably use some more thought, and possibly better explanation in the setting screen.

Note that if we wanted to, a tiny bit of tweaking would allow the user to blacklist any urls with certain values they specify. (Instead of only testing the domain portion of the href.) Then the user could blacklist a particular css file, or any in the bootstrap/ folder, or so on.
This is tested in Chrome. I have not tested in Firefox, I am not sure how.

Note that in the Javascript, I changed the shortcutOptions variable name to allSettings, since this seemed more appropriate, given that the settings are not all explicity concerning keyboard shortcuts.

Looks good 👍

Thanks for the contribution @mborn319!
I've added you as a collaborator on the project, as I don't have the time to maintain it.
With great powers, comes great responsibility :)


Some of the candle pattern indicators are not able to find or compute results

2crows | 3blackcrows | 3linestrike | 3starsinsouth | 3whitesoldiers | abandonedbaby | advanceblock | breakaway | concealbabyswall | counterattack | eveningdojistar | eveningstar | gapsidesidewhite | identical3crows | inneck | kicking | kickingbylength | ladderbottom | mathold | morningdojistar | morningstar | onneck | risefall3methods | separatinglines | stalledpattern | tristar | unique3river | upsidegap2crows | xsidegap3methods
The above stated candlestick patterns do not give any output/find patterns. I have also checked with different data and timestamp, but they still provide no output or are not able to find the respective patterns. I can also confirm for sure that some of them which do not find patterns for any data, these are as follows,
2crows | 3blackcrows | 3linestrike | 3whitesoldiers | abandonedbaby | counterattack | eveningdojistar | eveningstar | morningdojistar | morningstar | risefall3methods | separatinglines | stalledpattern | tristar | unique3river |
Please see if this issue occurs at your end and if yes, help me in finding the resolution for the same.

Hello @KrishnaRajule,
Please fill out:
Which version are you running? The lastest version is on Github. Pip is for major releases.
import pandas_ta as ta
print(ta.version)

Do you have TA Lib also installed in your environment?
$ pip list

Have you tried the development branch? Did the upgrade resolve the issue?
$ pip install -U git+https://github.com/twopirllc/pandas-ta.git@development

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Provide sample code.
Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.


Release 2.1.1

Questions
Answers

Description?
Release 2.1.1

Type?
release

BC breaks?
no

Deprecations?
no

Fixed ticket?
N/A

How to test?
N/A

No need QA : only wording & license headers.


Add optional output_root to ngen.config

Add optional output_root: Optional[Path] key. Need to double check all the places where it is accepted. I think just ngen.config.NgenRealization? This was added to NextGen in https://github.com/NOAA-OWP/ngen/pull/531.

Verified that this is only a top level key. See this test and this doc.


TC-SC-4.2 need YAML support to reboot the TH

Summary:
As per testplan TC-SC-4.1 need YAML support for rebooting the TH
Testplan screenshot:

This issue was created by me using a account that i had to delete due to access issues, I will continue to follow these issues with my current id.

Effected tests:

TC-DD-3.8
TC-DD-3.9
SC tests

This can be done as a separate(custom) method on top of Python Yaml runner - Can be done for reboot / list operations

cc @krypton36 @manjunath-grl


Error running the Sample Demo

I just need help to fix this error, I dont know if this is a bug

I think it was because I use .net4.0 to compile the framework.
I change it back to 3.5 and now it work.


Fix avalanche multicall2 address

https://github.com/gnosis/multicall

Source for the multicall contracts: https://github.com/gnosis/multicall (I'm maintaining that repo myself)

Coverage decreased (-0.6%) to 90.385% when pulling b9df57b6d6e09bbf712931364c42f2798e24587d on fix-avalanche-multicall into 92ecc217cfbf063db31cf11305a0799aa8e0a411 on master.


Debug console default suggest widget item icon looks broken

Smoketest-express repo
Start debugging, put breakpoint in index.js:6
localhost:3000 -> hit breakpoints
Debug console -> trigger suggest widget

A lot of icons look broken.
Is this js-debug not providing a proper item type?
Still if this is the default icon it looks like a rendering glitch.
fyi @connor4312 @jrieken

Which icon do you mean? The wrench is for property, the half-shredded paper is for snippets. Maybe the question is why do snippet suggestions appear in the debug console?

Thanks for clarifying so we have two things here as you say:

Snippets should not appear in debug console
Snippet half-shredded paper icon looks like broken rendering to me. I think we need a better icon (though it might be just me) @misolori @daviddossett what do you think?

These icons haven't changed from the originals 😄

Snippets should not appear in debug console

So then that is the issue? Did it ever work that way before?

I believe there is no, at least not a new issues, unless snippet where muted before. I agree that there is value in snippets for the debug console but maybe not the default JS snippets like class, for-each, etc...
@isidorn Can it be that you have been using "editor.suggest.showSnippets": false for the longest time and that it's the first time you see snippets in suggest now?

@roblourens no, this never worked.
@jrieken no, I do not have snippets disabled. I just never saw them taking up most of the items in the suggest widget, thus this is the first time my brain registered this icon fully.
Having said all of that, feel free to close if there are no clear action items here and if I am the only one that find the icon look like a placeholder broken icon. Thanks!


Avoid dumping node error when just fails

In case of just command fails, we should not print
long unnecessary nodejs error but just propagate exit code.
Fixes #6.

Thanks, this is great!
I made one change, it looked like maybe a typo, should have been process.exit(...) instead of just exit(...).


Automatic Right-To-Left text alignment detection in Flutter

In Android TextView somehow detects text direction inside (RTL or LTR) by its language.
In Flutter it depends on current locale.
is there any way to make text aligment automatic per-widget?
i'm filing it as a separate issue because it's different from the one which was described in https://github.com/flutter/flutter/issues/23875
And using the same screenshots:
Flutter

Android

/cc @GaryQian - seems strongly related to work you've been doing.

I wanted the same effect in a textfield where when a user type in a rtl language the direction of the textfield should change, so I made a package that will automatically change the direction based on the text you give it. here is the the package auto_direction

After speaking with Hixie about assuming text directions based on content, we came to the conclusion that it is extremely dangerous to do so for generic fields since basic cases like "1902شارع" will be misinterpreted as LTR in Android's way. It is much more reliable to have users or locales specify the direction.
Auto-detection may be applicable in niche circumstances where the directionality is not fixed or in multilingual situations, but we are not planning on implementing it directly within Flutter. It looks like the plugin @AhmedAlYousef made is a great way to achieve this for the cases where it is necessary (thanks for making it!), though we urge you to fully consider the implications of detecting RTL.

I'm going to close this for now, but please let me know if you have any additional questions or concerns!

What about a widget to change alignment from LTR to RTL based on locale?

I wanted the same effect in a textfield where when a user type in a rtl language the direction of the textfield should change, so I made a package that will automatically change the direction based on the text you give it. here is the the package auto_direction

I've just started using this package and it seems to do the job well. Thanks @AhmedAlYousif


Fix: scroll bar keeps scrolling down exceeding contents length (#920)

Please check my PR, i think that cause of this bug is difference beetween browser height calculations when container size is not an integer value and calculation in code.
Im not quite sure, but according to this warning in MDN:
https://developer.mozilla.org/en-US/docs/Web/API/Element/scrollHeight

This property will round the value to an integer. If you need a fractional value, use Element.getBoundingClientRect().

its possible that browser also makes floor when doing calculations for display, because pixels are integers :)
Thank you very much for your contribution! Please make sure the followings
are checked.

[ ] Read CONTRIBUTING.md
[ ] Run npm test to make sure it formats and build successfully
[ ] Provide the scenario this PR will address(some JSFiddles will be perfect)

perfect-scrollbar JSFiddle

[ ] Refer to concerning issues if exist

It works perfectly, thank you!

When will it be released

When will it be released?

@mattonit  Is there any change to get this already merged pr published in near future?

@Jiang-Xuan @rajibchandrakarmaker it's probably not going to be released. The original maintainer (@mattonit) is (according to the github profile) no longer working for mdbootstrap.
There is however someone who published an updated version with the fix called perfect-scrollbar-latest
You can install that package and alias it as perfect-scrollbar by replacing
"perfect-scrollbar": "~1.5.0",

with
"perfect-scrollbar"<EMAIL_ADDRESS>
Doing so makes this a drop-in-replacement with the fix.


proper id to find text views in ChatHolder

Using wrong id's to finde text views 1 and 2.

I signed the CLA

Thanks for the contribution!


move symlinks to python, in line with other runtime repos

This PR duplicates what was done in the other 3 runtimes, to move the symlinking code to a json config and python script, to make it more easily extendable and manageable. Pytorch runtime reference PR: https://github.com/gradient-ai/Graphcore-Pytorch/pull/52/files
Testing done:
Use diff and tree to compare the sorted tree structure of the tmp directory for a Paperspace notebook started using the main branch vs this feature branch, and verify they are identical. Steps:

Created 2 notebooks on paperspace, one on main branch, one on this feature branch. For each notebook, after setup.sh etc. finished executing, start a terminal, then
apt-get update -y && apt-get -y install tree
cd /tmp
tree -afi > /notebooks/<branchname>.tree
Download the 2 .tree files to local machine, then do diff main.tree feature.tree and verify there's no difference between the two.

Thanks for the detail of how you tested 👌


Error while installing the project

I had try to run this project... But unfortunately some occurs.. What will I do?? C:\Users\user\Downloads\bootstrap-4.0.0>npm install bootstrap
npm ERR! Windows_NT 6.3.9600
npm ERR! argv "C:\Program Files\nodejs\node.exe" "C:\Program Files\nodejs\
node_modules\npm\bin\npm-cli.js" "install" "bootstrap"
npm ERR! node v7.10.1
npm ERR! npm  v4.2.0
npm ERR! code ENOSELF
npm ERR! Refusing to install package with name "bootstrap" under a package
npm ERR! also called "bootstrap". Did you name your project the same
npm ERR! as the dependency you're installing?
npm ERR!
npm ERR! For more information, see:
npm ERR!     <https://docs.npmjs.com/cli/install#limitations-of-npms-install-alg
orithm>
npm ERR!
npm ERR! If you need help, you may report this error at:
npm ERR!     https://github.com/npm/npm/issues
npm ERR! Please include the following file with any support request:
npm ERR!     C:\Users\user\AppData\Roaming\npm-cache_logs\2018-08-07T04_43_16_7
77Z-debug.log

You're doing it wrong. You already downloaded 4.0.0 from what i see. There is no need to install bootstrap via npm

Sorry, but we cannot help with how-to or general troubleshooting issues here; only bug reports and feature requests. You'll want to ask around in the official Slack team or Stack Overflow. See our readme for more details.


Find by tag

#40
Tags are found by find t/ tag1 tag2 tag3

Codecov Report

Merging #43 (1d79efc) into master (092ba64) will decrease coverage by 0.82%.
The diff coverage is 16.66%.

@@             Coverage Diff              @@
##             master      #43      +/-   ##
============================================
- Coverage     72.15%   71.33%   -0.83%
  Complexity      399      399
============================================
  Files            70       71       +1
  Lines          1232     1249      +17
  Branches        125      128       +3
============================================
+ Hits            889      891       +2
- Misses          311      325      +14
- Partials         32       33       +1

Impacted Files
Coverage Δ

...ess/model/person/TagContainsKeywordsPredicate.java
0.00% <0.00%> (ø)

.../seedu/address/logic/parser/FindCommandParser.java
56.25% <22.22%> (-43.75%)
:arrow_down:

...java/seedu/address/logic/commands/FindCommand.java
100.00% <100.00%> (ø)

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 092ba64...1d79efc. Read the comment docs.

find KEYWORD KEYWORD KEYWORD t/TAG TAG TAG now finds all items that matches any of the keywords or tags


'autofill objects' mode ignores 'preserve labels'

Describe the bug
I have 'autofill objects' activated as well as 'preserve labels' from the labeling GUI. If I draw next to an existing label, the label is preserved while drawing but the fill mode sometimes ignores it so that some already labeled pixels get relabeled.
To Reproduce
Steps to reproduce the behavior:

'autofill objects' activated
'preserve labels' activated
draw one labels (convex/concave shapes make the issue more obvious)
draw another label next to it

Expected behavior
I would expect the existing label remains untouched.
napari info
napari: 0.4.15
Platform: macOS-10.13.6-x86_64-i386-64bit
System: MacOS 10.13.6
Python: 3.9.7 (default, Sep 16 2021, 08:50:36) [Clang 10.0.0 ]
Qt: 5.15.2
PyQt5: 5.15.6
NumPy: 1.21.5
SciPy: 1.8.0
Dask: 2022.02.0
VisPy: 0.9.6

OpenGL:
- GL version: 2.1 INTEL-10.36.32
- MAX_TEXTURE_SIZE: 16384

Screens:
- screen 1: resolution 2560x1440, scale 1.0
- screen 2: resolution 1440x900, scale 2.0

Plugins:
- PlatyMatch: 0.0.3
- RedLionfish: 0.6
- animation: 0.0.2
- cellpose-napari: 0.1.4
- clEsperanto: 0.15.4
- console: 0.0.4
- devbio_napari: 0.4.0
- napari-3d-ortho-viewer: 0.0.1
- napari-accelerated-pixel-and-object-classification: 0.6.6
- napari-brightness-contrast: 0.1.6
- napari-clusters-plotter: 0.2.2
- napari-crop: 0.1.5
- napari-curtain: 0.1.1
- napari-folder-browser: 0.1.3
- napari-itk-io: 0.1.0
- napari-ome-zarr: 0.3.2
- napari-plot-profile: 0.2.1
- napari-plugin-search: 0.1.3
- napari-segment-blobs-and-things-with-membranes: 0.2.17
- napari-simpleitk-image-processing: 0.2.1
- napari-tabu: 0.1.5
- napari-time-slicer: 0.4.3
- napari-tools-menu: 0.1.10
- napari-workflow-optimizer: 0.1.3
- napari_skimage_regionprops1: 0.2.9
- napari_skimage_regionprops2: 0.2.9
- scikit-image: 0.4.15
- svg: 0.1.6

problem resolved in 2b36d04 (luckily an easy fix). However, I will create a new release only later. If you want to try the new version, you can directly install it from the repository:
pip install git+https://github.com/bauerdavid/napari-nD-annotator.git@main


Update pages workflow not working

See this job.

Still not working. The error is the fault of EMMOntoPy - importing rdflib's Graph from the wrong module.


Log the Camel Quarkus version on startup

I opened this in draft to get some feedback.
For micro release of Quarkus, we may end with something like this in the console logs:
INFO  [org.apa.cam.qua.cor.CamelBootstrapRecorder] (main) Apache Camel Quarkus 3.12.0 is starting
...
INFO  [io.quarkus] (main) my-cool-app 1.0.0 on JVM (powered by Quarkus 3.12.3)

Do folks think this could be a source of confusion for users? That the CQ version is potentially out of alignment compared to the version of the platform BOM.

@jamesnetherton The PR alignes the Camel Quarkus version to minor quarkus platform version, is that correct?

The PR alignes the Camel Quarkus version to minor quarkus platform version, is that correct?

The CQ version displayed in the logs will just be whatever the platform is aligned with. So it may not match the platform version perfectly.

Well, I doubt that this could confuse tht users, there are camel version and quarkus version, now camel-quarkus version. How about show all of these version one line?
(main) Apache Camel Quarkus 3.12.0 (based on Camel 4.6.0 and Quarkus 3.12.0) is starting

+1 for the proposal from @zhfeng.

Thanks for the feedback all. Seems the general idea is acceptable.
I'll stick with my original proposal of Apache Camel Quarkus 3.12.0 is starting initially. We can always revisit and update it in future.


[Bug]: networkidle0 and networkidle2 take ~2 seconds no matter what is on the page

Minimal, reproducible example
If I create an html document with something as basic as:
<!DOCTYPE html>

  <meta charset="utf-8">
    <title>hello world</title>

hello world

If I navigate to this html, It takes ~2 seconds to trigger the networkidle0 or networkidle2.
Error string
no error
Bug behavior

[ ] Flaky
[x] PDF

Background
I am trying to generate PDF's from simple html and could not figure out why networkidle0 or networkidle2 are taking long.  Waiting for load or domcontentloaded results in 200-300ms processing times (vs 2000ms+ with networkidle).
Expectation
I expected the added latency to be a maximum of 500ms (as documented) since no network calls should be made besides getting the original html.  Are there undocumented external network calls being made?
Reality
There may be network calls happening which should not be happening resulting in higher than expected latencies in the 2000-2500ms range.
Puppeteer configuration file (if used)
No response
Puppeteer version
21.6.0
Node version
20.4.0
Package manager
npm
Package manager version
9.7.2
Operating system
macOS

In the old headless, I get 2 secs, in the new headless ~1sec, and in the headful mode ~500ms for networkidle0
import puppeteer from 'puppeteer';

const browser = await puppeteer.launch({ headless: true });

const page = await browser.newPage();

console.time('goto');
await page.goto('file://path-to-file.html', {
  waitUntil: 'networkidle0'
});
console.timeEnd('goto');

await browser.close();

Looks like the bug in the browser. Will file upstream.

Filed https://crbug.com/1514719


wildcard rules are not reusable

Bug report

[x] I have checked other issues to make sure this is not a duplicate.

Describe the bug
Wildcard rules appear to not be reusable.

For example, I might define a policy like const allowAll = { '*': allow } and try to use it for multiple types. This will not work.
I might also try to define static permissions and then apply them to multiple server instances (in tests). This will not work.

To Reproduce
Steps to reproduce the behavior, please provide code snippets or a repository:

This is my GraphQL Schema.

type Query {
  book: Book!
  author: Author!
}

type Book {
  id: ID!
}

type Author {
  id: ID!
}

This is the invoked query

query {
  book { id }
  author { id }
}

I use these permissions

const allowAll = { '*': allow };
const permissions = shield({
  Query: allowAll,
  Book: allowAll,
  Author: allowAll,
}, { fallbackRule: deny })

This is the error I see

Not authorised!

Expected behavior
Success
Actual behaviour
Authorization failure
Additional context
https://github.com/maticzav/graphql-shield/issues/1341#issuecomment-969462820

I respect your push to get this library sponsored and I hope it's successful!
I'm having a tough time getting started because of this bug, however, plus some other usability concerns that I haven't been able to explore because of the investment I've sunk into debugging the library.

I'm getting exactly the same issue mentioned by @cainlevy, weird thing is that it seems to run the resolver but still throws "Not authorised" error.


[Merge after PR885] Log rule bug

The branch used for this PR is created from the one for PR885. That one must be merged before reviewing this one. This PR contains only one commit.

Closing this one as the starting PR will not be merged. A new PR with the correct set of changes will be opened.

Wrong issue connected.

Just for history references, the issue related will be connected


chore(BIC-1999): couchbase-lite-C 3.1.7

What's new: https://docs.couchbase.com/couchbase-lite/3.1/cbl-whatsnew.html
Mostly the introduction of scopes and collections, many functions targeting the default scope/collection are now deprecated.

I think we need to check it together tomorrow, if you have time.

Tomorrow I don't have much time, CSE all day :-)


(dataset problem) ImageNet has not "meta/train.txt"

Hello author, the imagenet dataset I downloaded does not have a meta/train.txt file, can you provide me with a file?

Hi, thanks for your attention.
You can get the meta files through the link below:
http://dl.caffe.berkeleyvision.org/caffe_ilsvrc12.tar.gz
Also, you can check the detailed instructions of preparing dataset in mmcls docs (thanks to the OpenMMLab team).


Release version 1.0.25 to rubygems

The changelog and tags say that there is a version 1.0.25, but Rubygems doesn't have it yet.

Good point, thanks for highlighting, fixing now.

Done https://rubygems.org/gems/capybara-screenshot/versions/1.0.25, was caused by an OTP issue


Stop building kubefed

Kubefed snaps fail to build, see https://github.com/juju-solutions/bundle-canonical-kubernetes/issues/510

Done!


Identical values for all fields of timers and histograms

Just started using App.Metrics and so far it has been working like a charm in a test project.  However, I have some toruble with reported values in an existing project.
I have integrated a histogram and a timer into an existing project.  Both the timer and the histogram report identical values for all their fields (last, min, max, mean and so on) to InfluxDB.  The value reported seems to be the value that should be reported for the field last.
I have been using App.Metrics 2.1.0 and App.Metrics.Reporting.InfluxDB 2.0.0 in the test project and the existing project.  Configuration of IMetrics should be identical.  Now I am a little out of ideas what the reason for the different behavior coul be.  Any ideas where I should be looking?

I figured it out.  There was a path in the code that caused creation of a new IMetricsRoot.  That raises another question for me:  Is there a possibility to reconfigure an existing IMetricsRoot, eg. to use a different reporter during runtime?  What I have found so far was using an IMetricsBuilder, but that creates a new IMetricsRoot (which I want to avoid to not mess up measurements).


Fixing link to CODE_OF_CONDUCT.md within CONTRIBUTING.md

[X] Have you followed the guidelines in our Contributing document?
[X] Have you checked to ensure there aren't other open Pull Requests for the same change?
[X] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.
[ ] Have you successfully run brew tests with your changes locally?

On https://github.com/Homebrew/brew/blob/master/CONTRIBUTING.md the link to the Code of Conduct page leads to a 404 page.
Updating the link allows a user to navigate to the correct page to view the resource.

@pseudocody Could you squash these two commits together and then force push? Thanks.

@JCount updated

@pseudocody One last request, I promise. 🤞
Could you switch it to using https://github.com/Homebrew/brew/blob/master/CODE_OF_CONDUCT.md#code-of-conduct as the url. The #code-of-conduct href nicely points directly to the beginning of the actual text of the Code of Conduct. 😄

@JCount updated 😃

Thanks! 💯 😄

Thanks for your first contribution to Homebrew!  You rock!


[rate-limit] add condition to the key

Closes "Apply rate limit to just a subset of requests" of #760.

Close once.


ContextMenu ignores window borders

https://github.com/iced-rs/iced_aw/assets/115432842/04a22e5e-9c05-434b-b0d5-5683c9be454d

@sdgoihew  can you please show us a minimal viable product that does this.
@latidoremi look into this please?

can you please show us a minimal viable product that does this.
https://github.com/iced-rs/iced_aw/tree/main/examples/context_menu
has resizable window

https://github.com/iced-rs/iced_aw/assets/115432842/65f9e0d7-72cc-4116-8dbe-8a4df0c634ee

If you are okay with vidrel's behaviour:
https://github.com/iced-rs/iced_aw/assets/115432842/70cf14a1-2d82-4b72-8c55-16768cef1ca9
then here's a patch:
--- a/src/native/overlay/context_menu.rs
+++ b/src/native/overlay/context_menu.rs
@@ -76,6 +76,22 @@ where
         let max_size = limits.max();

         let mut content = self.content.as_widget().layout(renderer, &limits);
+
+        // Try to stay inside borders
+        let mut position = position;
+        // x
+        let right_border_abs_x = position.x + content.size().width;
+        if right_border_abs_x > bounds.width {
+            let diff = right_border_abs_x - bounds.width;
+            position.x = f32::max(0.0, position.x - diff);
+        }
+        // y
+        let bottom_border_abs_y = position.y + content.size().height;
+        if bottom_border_abs_y > bounds.height {
+            let diff = bottom_border_abs_y - bounds.height;
+            position.y = f32::max(0.0, position.y - diff);
+        }
+
         content.move_to(position);

         Node::with_children(max_size, vec![content])

I think the patch itself should also to reposition it if its near the border based on the mouse's location.  so rather being on the right side of the mouse it should be on the left side of the mouse instead. Since most of the time the context menu should close once the mouse is not on or within its borders. For the issue of the context menu not fitting in based on the Height there is not much that can be done for that and most context menus will just clip. but at least for the right or left side they attempt to move to a different spot within the cursor click to avoid clipping.

Here, + menu hiding on window resizing.
--- a/src/native/overlay/context_menu.rs
+++ b/src/native/overlay/context_menu.rs
@@ -12,7 +12,7 @@ use iced_widget::core::{
     mouse::{self, Cursor},
     overlay, renderer, touch,
     widget::tree::Tree,
-    Clipboard, Color, Element, Event, Layout, Point, Rectangle, Shell, Size,
+    Clipboard, Color, Element, Event, Layout, Point, Rectangle, Shell, Size, window,
 };

 /// The overlay of the [`ContextMenu`](crate::native::ContextMenu).
@@ -76,6 +76,16 @@ where
         let max_size = limits.max();

         let mut content = self.content.as_widget().layout(renderer, &limits);
+
+        // Try to stay inside borders
+        let mut position = position;
+        if position.x + content.size().width > bounds.width {
+            position.x = f32::max(0.0, position.x - content.size().width);
+        }
+        if position.y + content.size().height > bounds.height {
+            position.y = f32::max(0.0, position.y - content.size().height);
+        }
+
         content.move_to(position);

         Node::with_children(max_size, vec![content])
@@ -167,6 +177,11 @@ where
                 }
             }

+            Event::Window(window::Event::Resized { .. }) => {
+                self.state.show = false;
+                Status::Captured
+            }
+
             _ => Status::Ignored,
         };

feel free to push that as a PR. that way you can get credit for your work.


Ready for Figure generation

Even more immediate than JBrowse issues 2-8 is the need to get Figures for our paper.  Since the browser context won't be in the paper, we need to focus on just the rendering window.  We can manually input things like start and stop #4 by editing the code.

[x] Adjacent connectors drawn (this will show whether the path is present)
[ ] Vertical Compression: Never show rows that are not present in the current view window
[ ] Vertical Compression: Stack all the participants to the top of each component to show the proportion that participates in a Link (on by default)

I am seeing really weird stuff happening when visualizing the A. thaliana data when checking out https://github.com/graph-genome/Schematize/commit/71217e38fb7a0dcbcd9b3f39c37012a38c7c22e4.

There is a rectangle directly under download. A link arrow is drawn, but no link column, etc.
Maybe https://github.com/graph-genome/Schematize/blob/71217e38fb7a0dcbcd9b3f39c37012a38c7c22e4/src/App.js#L65 is a problem?

When taking a look at ./data/sebastian.Athaliana.all.50000.w100000.schematic.1D.json I get some weird output, too.

Ok, I fixed this via https://github.com/graph-genome/Schematize/commit/d0261fbf3cc94e00d47e2ff0e6874b8e0d914181. I suspect, that we might run into other issues related to .....departures.length - 1 vs. departures.length in our code.

@josiahseaman As you can see here, the width of the adjacent connectors is too wide. Removing  *2 in https://github.com/graph-genome/Schematize/blob/0df3163c76ce7bd612da855cc536313814518124/src/ComponentRect.js#L93 takes care of that.
But then the x-coords of most! components and links does not fit anymore. We need to move a component to the left, if we have drawn any connectors. If not, we should not move them.
See e.g. between the 4th and 5th component in the picture above. The space there is one binsPerPixel smaller compared to when a connector was drawn between two components.

Adjacent connectors have been added as the last LinkColumn in departures.  That's why I added departures.length - 1.  The same could probably be done for arrivals as well.  Since this is to reflect a change in the data, any old data not updated to the current format (generated by component_segmentation after 35807d427a07816e6efdec98bac330701899eaad) will render wrong.
Adjacent connectors should be our column padding size, not binsPerPixel.  Sorry, I didn't know where that came from.  So that we don't run into this in the future, I think I'll introduce a JSON version number which the program checks and raises an error if the data was produced in a different version than what it's expecting.  That'll keep us from getting our data files crossed.
@ekg may want to do the same thing if there's any plans to update the ODGI bin output.

It's not the case that all bins will be connected to adjacent ones. A graph
can have unaligned components, and these won't link to anything else.
On Mon, Feb 3, 2020 at 5:10 PM Josiah Seaman<EMAIL_ADDRESS>wrote:

Adjacent connectors have been added as the last LinkColumn in departures.
That's why I added departures.length - 1. The same could probably be done
for arrivals as well. Since this is to reflect a change in the data, any
old data not updated to the current format (generated by
component_segmentation after 35807d427a07816e6efdec98bac330701899eaad) will
render wrong.
Adjacent connectors should be our column padding size, not binsPerPixel.
Sorry, I didn't know where that came from. So that we don't run into this
in the future, I think I'll introduce a JSON version number which the
program checks and raises an error if the data was produced in a different
version than what it's expecting. That'll keep us from getting our data
files crossed.
@ekg https://github.com/ekg may want to do the same thing if there's
any plans to update the ODGI bin output.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/graph-genome/Schematize/issues/9?email_source=notifications&email_token=AABDQEIMNSD2YEEGNWZIWU3RBA6WTA5CNFSM4KMYJJDKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEKUNGJQ#issuecomment-581489446,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AABDQEODTPIYWOZEUJ6GSGDRBA6WTANCNFSM4KMYJJDA
.

That's a very interesting point!  Wouldn't their co-occurrence in a Path cause them to be linked?  If they weren't connected to anything, wouldn't the sort move them to the end or beginning?

Adjacent connectors have been added as the last LinkColumn in departures. That's why I added departures.length - 1. The same could probably be done for arrivals as well. Since this is to reflect a change in the data, any old data not updated to the current format (generated by component_segmentation after 35807d427a07816e6efdec98bac330701899eaad) will render wrong.
Adjacent connectors should be our column padding size, not binsPerPixel. Sorry, I didn't know where that came from. So that we don't run into this in the future, I think I'll introduce a JSON version number which the program checks and raises an error if the data was produced in a different version than what it's expecting. That'll keep us from getting our data files crossed.
@ekg may want to do the same thing if there's any plans to update the ODGI bin output.

So I raised a false flag by testing with malformed data....... sorry!

Thanks for spotting that.  I'm sorry I didn't communicate the data format change more clearly.  I've learned my Lesson.  I think I'll work on the Histogram feature next, but this would be much more effective with a Toggle Input for the user to turn on and off vertical compression.  @subwaystation would you like to try your hand at getting a toggle connected to store.useVerticalCompression?

All 3 features that I wanted for this issue are now in master.  Mobx state updates could certainly be cleaner, but it meets our MVP needs.  Matrix will be a separate issue.


Add tree view component to component gallery

Summary of the bug
Add tree view component to gallery page with an image

when picking this up, speak to design as to if we have an image to use. if not, could a placeholder one be used?


Train station nbt is too large and Miner nbt is too low

The way the village house pieces knows they can spawn is if their x/z size stays within the boundaries of a village road's x/z bounds. That's why village roads reserve a space to their sides to spawn the houses. You can view village generation here to see what I mean: https://jigsaw.jacobsjo.eu/
Generally, jigsaw pieces can only spawn child pieces if the child piece is entirely within or entirely outside the bounding box of the parent piece. (With villages, they have a hack activated to temporarily stretch the road's height to enclose the the top of the child piece. For our purposes, the x/z bounds and minimum y bounds of roads is what is important when adding houses to villages)

The issue here is the train station is 26 blocks long and no village road is large enough to hold it in vanilla. The solution is to not inject the train station nbt into the house pool. Instead, create a giant road nbt file that reserves enough space for the train station. Have this road target a special template pool file for just the train station alone. Inject that road with a low weight into vanilla's road template pool. Now when your road is picked, it reserves a large enough space and will spawn the trains station without issue.
A similar issue faces the mining camp nbt too. This nbt is too low so it cuts down below the road's bounding box which makes the game not spawn the nbt as far as I know. You can do a similar fix here with a road that does not reserve any bounds to the side. Like what is shown below. Meaning the road can spawn any size nbt no matter how large nor how low it goes. This would spawn the mining camp nbt properly (and can spawn train station too if wanting to reuse this for that)

This will let you actually see your village pieces in villages and no longer need to inject the pieces with a high weight into the house pool anymore.

This issue was incredibly helpful to me and I really appreciate you sharing this information. Should be fixed in the newest release!


Enable package install_options

We need be able to define which repo to enable to install a package.
We expose this option with a new variable for all package resources

@jfroche this new option needs documentation, and tests.

I think enabling repos is the sort of thing people generally expect to do in their profiles.

Hello:
What's the status of this PR? Is it still needed, and can it be squashed?

Closing due to lack of response. Please re-open if needed.


tasks stopped to work, I cannot drag or sort tasks

In the most recent update, The task who once were draggable, now are not.
Windows 10 x64
Atom 1.0.7
imdone-atom 1.1.33
Please help!

I'll have a look today.  Is it on atom restart?  I noticed that myself.

seems after restarting atom it start to work, not sure why. If it freezes again I will let you know

@tbogard This should be fixed in 1.1.34.  Let me know if it happens again.