Is WritePackedUInt worth it?

Benchmark script:
using UnityEngine;
using Mirror;

public class Benchmark : MonoBehaviour
{
    void WriteRegular(int count)
    {
        // write(int)
        NetworkWriter writer = new NetworkWriter();
        var watch = System.Diagnostics.Stopwatch.StartNew();
        for (int i = 0; i < count; ++i)
            writer.Write(i);
        watch.Stop();
        Debug.Log("WriteRegular: time=" + watch.ElapsedMilliseconds + " ms" + " size=" + (writer.Position / 1024) + "kB");
    }

    void WriteFixedRegular(int count)
    {
        // write(int)
        NetworkWriter writer = new NetworkWriter();
        var watch = System.Diagnostics.Stopwatch.StartNew();
        for (int i = 0; i < count; ++i)
            writer.Write(1337);
        watch.Stop();
        Debug.Log("WriteFixedRegular: time=" + watch.ElapsedMilliseconds + " ms" + " size=" + (writer.Position / 1024) + "kB");
    }

    void WriteReadRegular(int count)
    {
        // write(int)
        NetworkWriter writer = new NetworkWriter();
        var watch = System.Diagnostics.Stopwatch.StartNew();
        for (int i = 0; i < count; ++i)
            writer.Write(i);
        NetworkReader reader = new NetworkReader(writer.ToArray());
        for (int i = 0; i < count; ++i)
            reader.ReadInt32();
        watch.Stop();
        Debug.Log("WriteReadRegular: time=" + watch.ElapsedMilliseconds + " ms" + " size=" + (writer.Position / 1024) + "kB");
    }

    void WritePacked(int count)
    {
        // writepackedint
        NetworkWriter writer = new NetworkWriter();
        var watch = System.Diagnostics.Stopwatch.StartNew();
        for (int i = 0; i < count; ++i)
            writer.WritePackedUInt32((uint)i);
        watch.Stop();
        Debug.Log("WritePacked: time=" + watch.ElapsedMilliseconds + " ms" + " size=" + (writer.Position / 1024) + "kB");
    }

    void WriteFixedPacked(int count)
    {
        // writepackedint
        NetworkWriter writer = new NetworkWriter();
        var watch = System.Diagnostics.Stopwatch.StartNew();
        for (int i = 0; i < count; ++i)
            writer.WritePackedUInt32(1337);
        watch.Stop();
        Debug.Log("WriteFixedPacked: time=" + watch.ElapsedMilliseconds + " ms" + " size=" + (writer.Position / 1024) + "kB");
    }

    void WriteReadPacked(int count)
    {
        // writepackedint
        NetworkWriter writer = new NetworkWriter();
        var watch = System.Diagnostics.Stopwatch.StartNew();
        for (int i = 0; i < count; ++i)
            writer.WritePackedUInt32((uint)i);
        NetworkReader reader = new NetworkReader(writer.ToArray());
        for (int i = 0; i < count; ++i)
            reader.ReadPackedUInt32();
        watch.Stop();
        Debug.Log("WriteReadPacked: time=" + watch.ElapsedMilliseconds + " ms" + " size=" + (writer.Position / 1024) + "kB");
    }

    void Awake()
    {
        // 1 million writes
        int n = 1000000;

        WriteRegular(n);
        WriteFixedRegular(n);
        WriteReadRegular(n);

        WritePacked(n);
        WriteFixedPacked(n);
        WriteReadPacked(n);
    }
}

Results:

Packed version is 2-3x slower than regular version.
Packed version uses half bandwidth for values like '1337'. No real difference for values from 0..1million.
We should consider if this is worth it at all.

we are CPU limited, not bandwidth limited
uses a lot of bandwidth for negative values
uses extra code

I would like to see how much bandwidth it saves on ummorpg.
I suggest capturing a few messages with wireshark before and after to see how much bandwidth they are saving.

A common technique to store negative numbers in a varint is to zigzag them. this allows small numbers to also take up a low amount in the varint.
Also your varint implementation could be more optimised.
And your test here is very limited, a very common use case is that users are sending ints for count and other values whilst their values could easily fit in a single byte.

there is something else that comes to mind: C# really likes int, and doesn't like byte/short/ushort/etc. too much. converting all the Mirror code to use optimal types would be very painful.
being able to use int + packed is useful.
will benchmark this in ummorpg in a few days.

closing this for now, because weaver makes heavy use of varint


What is the debug mode in safe_eval.py and model.py's class GPT?

A EVAL_DEBUG = False is set in safe_eval.py, and the variable is passed into class GPT in model.py.
The question is why do you set such a variable? What is its functionality?

Why do you catch openai.BadRequestError in model.py and check the repetitive pattern in error? It make the code hard to understand?

These are designed to resolve the special issue we encountered when evaluating.

EVAL_DEBUG is to help not using retry mechnisim, that is primarily designed for API calling stability. As its name, help debug if any issue with the API calling.
openai.BadRequestError is a rare case, which we did not know the root cause for the corresponding case, and to a fair setup, we set it as a fail attack attempt.

All these designs are primarily designed for running stable inference pipelines, and this can be replaced with any other inference library with similar functionality.


The theme will receive any updates?

It's break in new version of bootstrap.

Hi, there is a 2.2 version.
But it isn't available on npm.


[2023b] Update OWNERS file

This is a cherry pick of the following PRs: #546, #561 and #611.

[x] The commits are squashed in a cohesive manner and have meaningful messages.
[ ] Testing instructions have been added in the PR body (for PRs involving changes that are not immediately obvious).
[ ] The developer has manually tested the changes and verified that the changes work

/lgtm


Travis: jruby-<IP_ADDRESS>

This PR updates the CI matrix to use latest JRuby.
http://jruby.org/2017/09/06/jruby-9-1-13-0.html

Ah, the whole build fails on [!] There was an error parsing Gemfile: RubyDep Error: Unrecognized JRuby version: "<IP_ADDRESS>". Bundler cannot continue.

See #35


Add Hover Effect for the footer section

🎮 feature Request
feature logic and basic description- i want to add hover effect for the footer section to look website more attractive.
Point down the features
feature points
Select program in which you are contributing
GSSoC24
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINE of this project.

/assign

assign me this task under gssoc24 #106

Please Assign the issue to me i will work on issue @Shubhamsingh2110

go ahead @KARTHIK174 in case of any issues feel free to reach out on discord

This issue is still open if no one is working on it then pls assign me this under GSSOC'24.

@Mukesh751 i have completed this issue.check it once

Okay 👍

please assign this task to me


no-cache flag, gopath tmp variable...

...minor format cleanup
I have replaced GOPATH rewinding by traversing to temp original GOPATH variable, added new flag which removes cache after vendoring (can be useful for containers) and made a few code formatting and cleanup (pointed by IDE).

@bbrodriges What is your use case for trash that really needs the changes you're proposing?
I needed a tool to be run at development time (not even at build time) with 2 cases in mind: to add libraries to ./vendor and to minimize unused vendored code. Auto-removing cache is not important for trash to have, and having stuff in the cache makes the vendoring phase much faster for later uses (because we often work with a bunch of the same libs from project to project), and you can always come up with your own cache dir (that will be created and populated by trash) and remove it afterwards.

@bbrodriges Looks like I'm going to close this PR, but I really want to know how you're using trash and your needs as a user, so welcome to comment.

Hi! I was thinking to use trash in Docker image building process. I don't really want to push vendor folder to my git repo. So "no-cache" flag is just a handy shortcut for rming trash folder later, since I don't need it after my app build anymore.
As of GOPATH^  I can clearly see that trash changes it on line 120 to trashDir (which equals c.String("cache")) and not changing it afterwards. Because of it I had error Error removing unused dirs as GOPATH was wrong after traversing.
I now can clearly see that it was bad idea to store original GOPATH under temporary env var, it is more convenient to store it under global package variable. I must have been very tired and forget to remove unused trashDir param.


It is not possible to achieve a printer effect in a Stream mode similar to ChatGPT

MarkdownView(text: markdownText)

When using the Stream mode of the OpenAI API, the markdownText in the above code will change in real-time based on the content returned by the API. However, the updates on the UI cannot keep up with the addition of content and will have a delay of 1-3 seconds for updating large blocks of text, thus unable to achieve the typewriter effect.

This delay is called debounce. It can hugely improve performance when live previewing a large markdown file. Currently, it will have a 0.3s debounce.
Maybe introducing a modifier to disable debounce works for your use case. But the performance when displaying a huge markdown may decrease significantly.(But for ChatGPT‘s sample code, maybe enough)

Yes, after adjusting the debounce to 0.01 seconds, I achieved the printer effect. However, I have not yet assessed the increase in performance consumption caused by this.

Hello. I added a new MarkdownRenderingMode for you to choose.
Try this:

Update your package to rendering-mode branch.
No STEP 2. :)

Now, default rendering mode is immediate.

A big thank you

Try it out and give me some feedbacks. Thanks😉

Already updated and currently no significant performance issues have been found, thanks a lot

Hope you enjoy this package😊

1.4.0 released. I will delete that fix branch.


[ENH] Add Sequence Editor List Add Sorting much like in GSE Keybind Area

🟢 How does GSE currently work
Puts All GSE Sequences under Class
🟢 Describe the solution you'd like
[ENH] Main Sequence Editor List Add Specs for sorting much like in GSE Keybind Area,
Current Class:

Spec 1 Sequences:

Spec 2 Sequences:

Spec 3 Sequences:

Common Class Sequences:

Global Common Sequences:

🟢 Describe alternatives you've considered
It works Great Now! - I think personally it would bring a extra level of organization for us Creators
🟢 Additional context
Loving the NEW GSE Tim!

Way back in GSE 1 these were all sorted like that but i just left it at class as I couldn't figure out how to do drop down filters in the menu stuff.  I will figure this out.


marvin: 21.3.0 -> 21.9.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/marvin/versions.
meta.description for marvin is: "A chemical modelling, analysis and structure drawing program"
meta.homepage for marvin is: "https://chemaxon.com/products/marvin"
meta.changelog for marvin is: ""
Updates performed

Version update

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

Warning: no invocation of /nix/store/g2p2x5hvlbv4glcvhjcf5m3qr96yxy24-marvin-21.9.0/bin/LicenseManager had a zero exit code or showed the expected version

/nix/store/g2p2x5hvlbv4glcvhjcf5m3qr96yxy24-marvin-21.9.0/bin/MarvinSketch passed the binary check.

/nix/store/g2p2x5hvlbv4glcvhjcf5m3qr96yxy24-marvin-21.9.0/bin/MarvinView passed the binary check.

/nix/store/g2p2x5hvlbv4glcvhjcf5m3qr96yxy24-marvin-21.9.0/bin/cxcalc passed the binary check.

/nix/store/g2p2x5hvlbv4glcvhjcf5m3qr96yxy24-marvin-21.9.0/bin/cxtrain passed the binary check.

/nix/store/g2p2x5hvlbv4glcvhjcf5m3qr96yxy24-marvin-21.9.0/bin/evaluate passed the binary check.

Warning: no invocation of /nix/store/g2p2x5hvlbv4glcvhjcf5m3qr96yxy24-marvin-21.9.0/bin/molconvert had a zero exit code or showed the expected version

/nix/store/g2p2x5hvlbv4glcvhjcf5m3qr96yxy24-marvin-21.9.0/bin/mview passed the binary check.

/nix/store/g2p2x5hvlbv4glcvhjcf5m3qr96yxy24-marvin-21.9.0/bin/msketch passed the binary check.

5 of 9 passed binary check by having a zero exit code.

5 of 9 passed binary check by having the new version present in output.

found 21.9.0 with grep in /nix/store/g2p2x5hvlbv4glcvhjcf5m3qr96yxy24-marvin-21.9.0

found 21.9.0 in filename of file in /nix/store/g2p2x5hvlbv4glcvhjcf5m3qr96yxy24-marvin-21.9.0

directory tree listing: https://gist.github.com/a40742ea22db20af4d6a43358eafccd9

du listing: https://gist.github.com/d33ef31489529ef7600f9437a0dffd86

Rebuild report (if merged into master) (click to expand)

3 total rebuild path(s)

1 package rebuild(s)

1 x86_64-linux rebuild(s)
1 i686-linux rebuild(s)
0 x86_64-darwin rebuild(s)
1 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
marvin

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/g2p2x5hvlbv4glcvhjcf5m3qr96yxy24-marvin-21.9.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A marvin https://github.com/r-ryantm/nixpkgs/archive/cb70ce62ab26553c9174244bdc37a847cfa28151.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/g2p2x5hvlbv4glcvhjcf5m3qr96yxy24-marvin-21.9.0
ls -la /nix/store/g2p2x5hvlbv4glcvhjcf5m3qr96yxy24-marvin-21.9.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

marvin

Maintainer pings
cc @fusion809 for testing.

Result of nixpkgs-review pr 124094 at cb70ce62 run on x86_64-linux 1

1 package built successfully:

marvin

1 suggestion:

warning: missing-phase-hooks
installPhase should probably contain runHook preInstall and runHook postInstall.
Near pkgs/applications/science/chemistry/marvin/default.nix:21:3:
   |
21 |   installPhase = ''
   |   ^


Periodically check for new config file

We should periodically have the miners check for an updated blessed block from the block-tracker and if theres a config newer than currently synced block they pull the new one and restart miner
Otherwise the block tracker only really works if you power cycle or its a new device just being turned on
Relates to https://github.com/NebraLtd/hm-block-tracker/issues/17

Will be solved using https://github.com/NebraLtd/hm-watchdog


extend CBC status_map_mip

First of all, I really like working with cvxpy, it makes working with different solvers so much easier!
The issue I have is related to cbc and to this issue: https://github.com/cvxgrp/cvxpy/issues/585 of setting MIP stopping criteria
If the solver quits on maximumSeconds, or any other stopping criteria CyCBC accepts, the cbcModel returns a status not specified in the STATUS_MAP_MIP dictionary resulting in a key not found error.
    prob.solve(solver= 'CBC', verbose=True, allowablePercentageGap =10)
  File "C:\Python36\lib\site-packages\cvxpy\problems\problem.py", line 289, in solve
    return solve_func(self, *args, **kwargs)
  File "C:\Python36\lib\site-packages\cvxpy\problems\problem.py", line 571, in _solve
    self, data, warm_start, verbose, kwargs)
  File "C:\Python36\lib\site-packages\cvxpy\reductions\solvers\solving_chain.py", line 194, in solve_via_data
    solver_opts, problem._solver_cache)
  File "C:\Python36\lib\site-packages\cvxpy\reductions\solvers\conic_solvers\cbc_conif.py", line 177, in solve_via_data
    solution["status"] = self.STATUS_MAP_MIP[status]
KeyError: 'stopped on gap'

stopping on maximumSeconds returns: 'stopped on time '
stopping on allowableGap returns: 'stopped on gap'
stopping on allowablePercentageGap returns:  'stopped on gap'
stopping on maximumNodes returns: 'stopped in nodes'
There is one problem, however,  stopping on time does not guarantee a feasible solution but the status message does not hold this info it simply returns stopped on time.
I could extend the STATUS_MAP_MIP dictionary in cbc_conif.py and make a pull request if you think this would be a useful feature.

There should never be a key not found error. You'll need to map all the statuses into the existing status codes. You can always use solver error if it's not clear what status is appropriate. But to address your question, yes, a contribution would be welcome!

Resolved in #802 by @BastiaanvG


Add option to compile packages with -fno-omit-frame-pointer

Frame pointers are critical for production profiling, which is crucial for running production services.
I propose we have an option in Mariner that supports compiling all packages with this option and providing a flavor of this build to customers.
It would be great if Mariner became the first Linux distribution that had a flavor for frame pointers disabled.

We have discussed the option of enabling frame pointers, however this comes with a performance penalty, and is understood to be an all-or-nothing situation.  For now frame pointers will not be supported.


Make 201s more RFC 7231 compliant

The “Creating Resources” section says that a Location header SHOULD be sent on a 201 response.
But RFC 7231 § 6.3.2 says:

The primary resource created by the request is identified by either a Location header field in the response or, if no Location field is received, by the effective request URI.

A (hypothetical) implementation of this RFC 7231 text would infer the wrong URI from JSON API’s 201 response without Location.
To avoid confusing implementors, I suggest elevating the requirement for Location to a MUST, if that’s OK by your compatibility policy. If not, then maybe just include a note along the lines of:

If you choose not to send Location with a 201 response, you will violate RFC 7231 § 6.3.2.

JSON API resources are not HTTP resources. The RFC requirement is based on resources being uniquely identifiable by their URIs, which is not the case for JSON API resources.
Consider bulk operations (that are planned to become a core part of the spec AFAIK). The server might not even support canonical multi-resource URIs, at least not for arbitrarily large collections. In that case there simply is no single URI that would be guaranteed to return those newly created resources.
Of course there are solutions to this, such as assigning an identifier to the operation as a whole, which could then be used as part of a request URI to retrieve the same set of resources. But specifying a MUST-level requirement for this sort of thing seems a bit too much, don't you think?

@bintoro

JSON API resources are not HTTP resources.

But the HTTP status code 201 is about HTTP resources, not some other arbitrary notion of “resource.”

In that case there simply is no single URI that would be guaranteed to return those newly created resources.

Then the server should not respond with 201.

Of course, this is a fairly minor issue.

@vfaronov That's an entirely valid point. My view is that REST is actually very impractical in reality and that it's a fool's errand to try to shoehorn complex API design into strict HTTP semantics. How the whole world got into this bandwagon is beyond me.
The options are:

Adhere to HTTP/REST principles and suffer the limitations with no apparent offsetting gain.
Consider HTTP as a transport protocol and ignore the inconvenient semantics.
Roll your own protocol.

I think the middle option offers a happy balance.

As you suspected, a new MUST is probably a no-go for compatibility reasons.
However, I would be ok changing the sentence that reads:

The response SHOULD include a Location header identifying the location of the newly created resource.

to instead say:

The response SHOULD include a Location header identifying the location of the newly created resource, in order to comply with RFC 7231.

That change seems less obtrusive to me than a standalone note, which would, imo, be too much text to add to the spec for this issue.

@bintoro “REST” and “HTTP semantics” are to a large degree orthogonal. JSON API is clearly designed to be RESTful, but it doesn’t have to use HTTP status codes to communicate operation results—it can use its media type instead. The spec already defines error objects that can refine the meaning of the 4xx/5xx status code they were served with. Similarly, in cases where 201 is not applicable but 200 is insufficient, JSON API could define some extra object fields to be served with 200.
But I do not intend to propose major changes to the spec.

@ethanresnick I agree. Should I submit a PR?

“REST” and “HTTP semantics” are to a large degree orthogonal.

I'm really curious what you mean by this, given that HTTP explicitly references REST as a design principle.
Obviously HTTP doesn't have to be used RESTfully, but that alone hardly makes them orthogonal.

JSON API is clearly designed to be RESTful, but it doesn’t have to use HTTP status codes to communicate operation results—it can use its media type instead.

What would this accomplish? If you want to interact with an HTTP client, there's got to be some status code. I mean, how is inventing your own status code & message body scheme a lesser offense than omitting a Location header from an otherwise standard 201 response?

@bintoro

Obviously HTTP doesn't have to be used RESTfully

That’s what I meant by “orthogonal,” perhaps not the best wording.

I mean, how is inventing your own status code & message body scheme a lesser offense than omitting a Location header from an otherwise standard 201 response?

I’m not sure what you mean by “inventing your own status code.” I suggested 200 (OK) for cases where a more specific status code does not apply. 200 is a standard code that can be used in most successful responses, except for a handful of cases where HTTP explicitly requires a different code (such as in RFC 7231 § 4.3.4).
I’m also not sure what you mean by “message body scheme.” Did you mean “media type”? A cornerstone of HTTP is that it is media type agnostic. As an implementor, you choose a media type that serves your purposes (or define your own), and “drop it into” the protocol. The JSON API spec defines the application/vnd.api+json media type. This definition can easily incorporate new fields to communicate more information between the server and the client.
If you wish to continue this discussion, let’s bring it to another venue such as email, as it seems off-topic to this issue.

@vfaronov I may have miscommunicated my original response. Just because JSON API resources and HTTP resources are not 1:1 the same thing, it does not follow that there are no HTTP resources created with JSON API POST requests.
As long as the records you've created become accessible at some URI, you will have created HTTP resources as well, and 201 is the correct response. Where the HTTP spec falls short is its assumption that there is always a well-defined "primary resource", suitable for presentation as Location. But difficulty in producing such a URI does not magically make the status code itself inapplicable.

@bintoro I see your point.
I prefer to use HTTP status codes as a channel of standardized machine-readable information, which means that if I cannot use a status code compliantly, I’d rather not use it at all. RFC 7231 § 4.3.3 does not require a 201 response when a resource is created (that’s only a “SHOULD”), so 200 is compliant in this case.
But I realize that this is not the only possible approach. I also don’t like that part of RFC 7231 I quoted in the original comment—it wasn’t present in RFC 2616 and it contradicts real-world usage. So I do not insist on any particular solution.

@vfaronov Yes, a pr would be great!


Compilation error: xargs: shfmt: No such file or directory

Anything else is fine except:
~/go/src/github.com/travis-ci/worker ~/go/src/github.com/travis-ci/worker
+ gometalinter --disable-all -E goimports -E gofmt -E goconst -E deadcode -E golint -E vet --deadline=1m --vendor --tests --errors ./...
[proxychains] DLL init: proxychains-ng 4.12-git-14-g06c20ed
+ git grep -l '^#!/usr/bin/env bash'
+ xargs shfmt -i 2 -w
[proxychains] DLL init: proxychains-ng 4.12-git-14-g06c20ed
[proxychains] DLL init: proxychains-ng 4.12-git-14-g06c20ed
xargs: shfmt: No such file or directory
Makefile:49: recipe for target 'lintall' failed
make: *** [lintall] Error 127

proxychains is what I'm using for network proxy.

go get -u mvdan.cc/sh/cmd/shfmt solves this and another error:
ok      github.com/travis-ci/worker/ratelimit   1.020s
?       github.com/travis-ci/worker/ssh [no test files]
[proxychains] DLL init: proxychains-ng 4.12-git-14-g06c20ed
=== RUN   TestNothing
--- PASS: TestNothing (0.00s)
PASS
ok      github.com/travis-ci/worker/cmd/travis-worker   1.124s
Makefile:59: recipe for target 'test-no-cover' failed
make: *** [test-no-cover] Error 1


How to list all the topics when Publish message to emqtt before subscribe?

When we subscribe the topic, the topic will be listed in the dashboard.
How about publish? If we only doing publish, how to list all the topics?
Thanks,
Tristan

Any plan to have this function?

@magixyu Publish will not create topic/route in the broker. Close the issue.


Allow text to be copied from terminal UI

Summary
Text can be copied from logs and every other page from Devtron UI but in terminal nothing works, not even ctrl + c or right click and copy
Motivation
This would be great as sometimes we need to copy some directory names or file names as it as to be used in extra containers or anywhere else we need
Proposal
Would be good if it is same as logs, whole selected text is automatically copied to clipboard

hi Guys,
can you please add this feature for linux/windows systems as well in coming releases, That will help us lot. Thanks in advance.


Search  functionality on left menu

Search on left side bar is not working.It would be great if there's a functionality of live search.Thanking You

Hello,
Since this functionality hasn't been implemented yet, I strongly encourage you to add the feature yourself because it is simple and I currently don't have much time. Here is a tutorial on how to do it: http://www.jqueryscript.net/layout/Filterable-Tree-View-Plugin-with-jQuery-Tree-Filter.html
Thanks and sorry for the inconvenience!


ImportError: No module named coros

Steps to reproduce

pip install crackmapexec
crackmapexec

Command string used
CME verbose output (using the --verbose flag)
OS
Windows
Target OS
Detailed issue explanation
Just installed it with pip, tried from repo as well, on a windows 10 x64 machine.
When having crackmapexec in a cmd just throws:
C:\Users\USER\Desktop\CrackMapExec-master\CrackMapExec-master>crackmapexec
Traceback (most recent call last):
File "C:\Python27\Scripts\crackmapexec-script.py", line 11, in
load_entry_point('crackmapexec==3.1.5.dev0', 'console_scripts', 'crackmapexec')()
File "C:\Python27\lib\site-packages\pkg_resources_init_.py", line 561, in load_entry_point
return get_distribution(dist).load_entry_point(group, name)
File "C:\Python27\lib\site-packages\pkg_resources_init_.py", line 2631, in load_entry_point
return ep.load()
File "C:\Python27\lib\site-packages\pkg_resources_init_.py", line 2291, in load
return self.resolve()
File "C:\Python27\lib\site-packages\pkg_resources_init_.py", line 2297, in resolve
module = import(self.module_name, fromlist=['name'], level=0)
File "C:\Python27\lib\site-packages\crackmapexec-3.1.5.dev0-py2.7.egg\cme\crackmapexec.py", line 10, in
from cme.connection import Connection
File "C:\Python27\lib\site-packages\gevent\builtins.py", line 93, in import
result = _import(*args, **kwargs)
File "C:\Python27\lib\site-packages\crackmapexec-3.1.5.dev0-py2.7.egg\cme\connection.py", line 7, in
from gevent.coros import BoundedSemaphore
File "C:\Python27\lib\site-packages\gevent\builtins.py", line 93, in import
result = _import(*args, **kwargs)
ImportError: No module named coros
Trying with the latest pull from repo.
Thank you in advance.

It appears to be an issue with an update to gevent, I have gevent version 1.2.0 installed it the gevent.coros module is missing.

@Amurmurmur this pr fixes the issue, if youd like to try it out.
https://github.com/byt3bl33d3r/CrackMapExec/pull/145 you can pull it as a patch to test if you'd like

Fixed in https://github.com/byt3bl33d3r/CrackMapExec/commit/ab4d0a440602bdab7727ca336778c5b7311892e5, thanks @archey  and @Amurmurmur !


Privacy module

This introduces Privacy Module, which is based on Adversarial Robustness Toolbox. Currently includes

rule-based member inference privacy attack
model-based member inference privacy attack

Next Lens steps:

Create a demo
Extend to include more attack approaches from the toolbox

The new tests are now added. The updates look great.


socket.io config and lib will break npm start

In the case of responding "n" to socket.io , app.js will still have code loading socket.io config and module and it won't work obviously since they are missing.

Hello @jeveloper. Thank you for your issue report.
Can you explain the issue in more detail?
What do you mean with responding "n"?

Sorry @jeveloper, I understand now. You are right, optional socket.io support is not completed yet. It does not need much to be done but I am preparing a switch to the new Angular router, so I removed optional socket.io support in version 0.1.2. This should close this Issue, but I am not satisfied with the solution. Optional socket.io support is definitely on my list.
Closed with 1ad8342220ba6eadd47bc207755fc3ee2aa3e31d


[Concurrency] Add a .isolation member on dynamically isolated function values.

This change implements the .isolation member on function values with dynamically isolated function type. The approach is very straightforward:

In the constraint system, member lookup recognizes the name isolation on function types with erased isolation and records a special ExtractFunctionIsolation overload choice.
When the ExtractFunctionIsolation overload choice is selected, CSApply injects ExtractFunctionIsolationExpr around the function expression.
SILGen then emits a call to Builtin.extractFunctionIsolation.
That's it

Once isolated captures are implemented, we can teach the actor isolation checker to recognize when a context is isolated to an actor value that came from ExtractFunctionIsolationExpr and understand that calls to the base function value do not cross an isolation boundary.

@swift-ci please smoke test


JPEG XL Support: Does not support container format.

This viewer does not open JXL files that have a container, instead returning the error VipsForeignLoad: source is not a known format.
It does work for images with the container removed, or which where created without the container (which is the default for encoding jxl images from pixels.)
Here is a working example: (Remove .txt extension, its only there to allow GitHub upload)
no-container.jxl.txt
Here is a non-working example:
container.jxl.txt

Oh hey, thanks @Fraetor. I'll have a look.

OK, fixed in libvips master, thank you for pointing out this dumb thing.


DSL Viewer ignored literals; this fixes that and tidies the code a bit

note that DSL Editor still doesn't support literal

@tbouron thanks for reviewing and flagging these two but for the reasons above I think they are best as is

significant update @tbouron -- discovered some things broke on nested evaluation, and made some minor tidies.  see last commit for more details.


Superfrog occasional freeze in one spot

Hi,
I have raised this via the bug report form on the WHDLoad install page for Superfrog, as it is an issue with the WHDLoad package (i think). I can replicate on MiSTer, WinUAE, etc.
Load stage 2.1 - code 392822
Hold down jump and aim for the first candle above where you spawn. Sometimes it will freeze quickly, other times it will take minutes to replicate so it's a bit random. You can come at it in different directions, or jump straight up, i've had it freeze in that spot in a variety of situations.
It is low impact as it doesn't happen often, but it did catch me out twice before I decided to try and replicate it and report.
Requires a full reboot once it freezes. Temporarily uploaded a very amateur video via WinUAE here:
https://www.youtube.com/watch?v=6INkB0H63hY
It does the same on MiSTer.
I understand that the disk version is used in MegaAGS. There are two separate entries for Superfrog - the main game and the intro. I think this must be because the disk version had the intro on a separate disc, but this wasn't required to be loaded if you just wanted to play the game. Hence why they are listed separately.
I am not sure if the same issue above occurs in the CD32 version or not via WHDLoad. I have not tested this. But the CD32 version is apparently supported by the WHDLoad installer. Maybe it is worth considering using the CD32 version of Superfrog as it has a skippable intro and there would only be a need for one entry in the games list (not Superfrog and Superfrog - Intro). Not sure if you have any thoughts on that. There are no enhancements on the CD32 release though. It might not be worth messing about with tbh.
Again, just logging this here in case you wanted to add it to the issues DB. It is such an unusual bug to find though, and won't happen regularly, so it might not be worth trying to come up with a few words of text to describe the issue.
I did log it via the WHDLoad site though - maybe someone will look into it.
Cheers

Sounds like the CD32 version is the better pick in this case. Thanks!

No worries! No idea if it will have that same random glitch on the CD32 version using WHDLoad, but I suppose the benefit of using that version either way in MegaAGS is only having one entry in the games list, instead of two. If you do change it I can always test it again and see what happens.
I should add that I couldn't replicate this on a regular 'cracked' floppy release.
Cheers

I assume that the same issue will be present on the CD32 install too, but let’s find out.
Going from two Superfrogs to one is one step closer to the ideal amount of Superfrog, so it’s a win either way. 😈

Ha!! I quite agree!
My girlfriend loves it however - she isn't really a gamer but did have an Amiga growing up and she remembers both Superfrog and 'Gloom' vividly (no idea why), so as soon as she spotted Superfrog I couldn't get her off it for a while!

Changed default version in fd924a1b1894bcf4eb54a2c3bb476ee608182d21.

Thanks!
Went back and forth on the issue I posted on the Mantis bugtracker. In the end I was able to replicate it on the CD32 iso as well as floppies. And the debug info for WinUAE apparently indicates it's a game bug.
Still, probably worth making CD32 the default Superfrog release as you can have a single entry for it then.
Thanks!


Number of categories

Hello, @stevewongv
Correct me, if I'm wrong. We have only 2 categories, that model could recognize: object and object`s shadow. Model doesn't recognize object by specific categories like persons, cars, dogs, etc. And in case we don't have any shadow on image, but object (person, for example) present, we don't get any predictions at all.

You are right. We only recognize the object instances whose shadows are in the image. And we only have two classes: object and shadow.


added full solution

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

Closed and Reopened this PR to attempt to resolve a specific Travis build failure.

Congrats @magic-hat on your first pull request (PR)! 🎉
Thank you for your contribution to the page! 👍
We're happy to accept these changes, and look forward to future contributions. 📝


Fix model loading from stream

Fix bug introduced in 17913713b554d820a8ce94226d854b4a5f1d8bbc (allow loading from byte array)
When loading model from stream, only last buffer read from the input stream is used to construct the model.
This may work for models smaller than 1 MiB (if you are lucky enough to read the whole model at once), but will always fail if the model is larger.

Thanks for the PR!  Is there a way we can add a test for this?

Is there a way we can add a test for this?

Perhaps add a test that creates 1.5MB large model definition and tries to load it? I can try creating such test case, perhaps by crafting some minimal model and (as it is a json) put some generated whitespace inside to make it 1.5 MB large (and it will fail to parse if only first/last 1 MB is seen), then send it via a stream.

@mpetricek-corp Better to add a unit test for your fix. Thx very much

Can we generate the model with random data.  To get a large model, easiest way is to use classification with random forest.

Merged since it's difficult to create unittest and is confirmed to work.


.* in .cfignore is treated as ignore everything.

Command
cf push
CLI Version
6.19.0+b29b4e0-2016-06-08
Error
We were stepping up from 6.12 to 6.19 and encountered this regression.
We have in our .cfignore  the line
.*
This worked in 6.12, ignoring all files starting with period, but 6.19 is treating this as ignore ALL files. And we get the message no app files found.
This is a regression and breaks most of our built applications.
Stack Trace
CC API Endpoint Version
CF Trace
Platform Details
Linux and Windows at least.
Shell
bash or windows command prompt

Hi @rlk833
.* looks like the right pattern to use for ignoring files starting with a period. We'll take a look.
Regards,
Dies Koper
CF CLI PM

Looks like this broke between versions 6.13.0 and 6.14.0

This has been fixed in cf CLI 6.20.0.
https://github.com/cloudfoundry/cli#downloads
Thanks for bringing it to our attention!
Regards,
Dies Koper
CF CLI PM


how do i use Drag and drop feature in react-big-schedule?

Checklist

[X] I've looked through the README
[X] I've verified that  I am running react-big-schedule latest version.
[X] I've tested the issue with the latest version of react-big-schedule
[ ] I've searched the existing issues and discussions for a similar question
[ ] I've provided relevant code snippets or error messages
[ ] I've included steps to reproduce the issue
[ ] I've checked the browser console for any errors

Please make sure the question is worded well enough to be understood

errorMsg:
(0 , react_dnd__WEBPACK_IMPORTED_MODULE_0__.DndContext) is not a function

@moreswapnil95  can you please send a video with console while dragging

@moreswapnil95 can you please send a video with console while dragging

ok

@moreswapnil95 as you created new issue #137 so I am closing this ticket


Need stricter control of module versions to use

The latest versions of Tedious (v1.12.0 & v1.12.1) introduced breaking changes for my environment.
Because the Tedious module is defined as tedious": "^1.11.2 this library forces the upgrade to v1.12.1. If I was using Tedious directly I could pick which version to use and be fine. But using this library I am forced to upgrade and can no longer release code. So Tedious' breaking change, became a breaking change for this library which is impossible to rollback from.
^ is too loose for something as important as the database driver.
I'd suggest having the ability to select the version of Tedious to use (latest being default, but ability to choose build version).
At the very least change:
tedious": "^1.11.2 --> tedious": "~1.11.2
Then when you feel v1.12.* is stable you can release a new version of this library which has:
tedious": "~1.12.0
This would allow people to choose which underlying library version to use.

It may be answered here as that is your question exactly in a module-generically speaking
http://stackoverflow.com/questions/11233133/when-installing-a-package-with-npm-can-you-tell-it-to-use-a-different-version-o

I aggree so I have just released a new version with updated package.json. This should force npm to install Tedious 0.11.4.

@devotis Good suggestion for a workaround. Wasn't aware that was possible.
@patriksimek :thumbsup:

@patriksimek - verified that 0.12.2 fixes the NTLM issue that was causing the issue (it 0.12.* had broken our NTLM auth as well). I'll send in a PR for the version change.


chore: bump Rust version

Context

Related Tasks: Update Rust to 1.80.0
Why? We need a newer Rust version for #593
Description

Bumps the Rust version to 1.80.0. Fixes clippy lint issues.
Manually testing the PR

Existing tests & lints pass:
nix flake check -j auto

@zcabter The patch coverage is low since this PR updates lots of existing code that isn't tested by our CI at the moment 😔


[Bug] YARM makes it extremely hard to rename map tags

I have narrowed it down to YARM causing this on 2.0.24 w/ SpaceAge.
I add a tag to the map, close the map, and when you open the map up, and click on the tag to rename it, approx. every 2 seconds the tag reverts to the text it was originally.  You are able to change the text if you are fast enough and try enough times, it's just very hard.
I believe this has something to do with the YARM re-drawing it's information in the top right somehow resetting the tag, but I can't figure out how.

This might actually count as a Factorio bug -- the issue is not with the YARM HUD but with its map markers; as a workaround, you may turn off "Enable map markers" in the YARM mod settings, but note this disables them for all players (it's a global setting because the markers themselves are per-force).
When YARM updates its own map markers (which it does every time it recounts the sites), it apparently resets the marker rename window even if you're editing a completely unrelated marker (even if it's on a completely different surface!).

I have created a report: https://forums.factorio.com/viewtopic.php?f=7&t=124220

Devs say it is fixed for next release!
Thanks much for confirming that it's a Factorio bug and getting it reported.


Remove std.experimental.typecons

See<EMAIL_ADDRESS>

Is there a reason Final was removed without copying to std.typecons? At least one project I use depends on this: https://github.com/burner/graphqld/blob/3ac6ed64b289ee6996dbd831db4269fcfc69a7d9/source/graphql/validation/querybased.d#L44

Hm... nevermind. The import is there, but it's not actually used. haha.


fix: Remove logging in templating

:tada: This PR is included in version 1.0.2 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


issue on restoring

Hi
Currently using your script which i need to migrate from a docker container to another host but cant seem to get it working
Migrating openproject
This is what i did,
on my production server ran
docker-volume-snapshot create /projects open.tar
then copied the tar file to my docker host server and ran
docker-volume-snapshot restore open.tar /projects
and copied the files to the same location
then i ran the docker compose file of openproject which is the same docker compose from the production so the volume location are the same
but when it shows login shows as if its the first time
Thank  you

Hi @killmasta93, In create command you have to specify docker volume name. Not the path to any directory. Have you mounted any docker volume to your container?

Thanks for the reply, im a bit lost
This is my docker compose
the information is saved in /projects
version: "3.7"

networks:
  frontend:
  backend:

volumes:
  pgdata:
  opdata:

x-op-restart-policy: &restart_policy
  restart: unless-stopped
x-op-image: &image
  image: openproject/community:${TAG:-12}
x-op-app: &app
  <<: [*image, *restart_policy]
  environment:
    RAILS_CACHE_STORE: "memcache"
    OPENPROJECT_CACHE__MEMCACHE__SERVER: "cache:11211"
    OPENPROJECT_RAILS__RELATIVE__URL__ROOT: "${OPENPROJECT_RAILS__RELATIVE__URL__ROOT:-}"
    DATABASE_URL: "${DATABASE_URL:-postgres://postgres:p4ssw0rd@db/openproject?pool=20&encoding=unicode&reconnect=true}"
    RAILS_MIN_THREADS: 4
    RAILS_MAX_THREADS: 16
    # set to true to enable the email receiving feature. See ./docker/cron for more options
    IMAP_ENABLED: "${IMAP_ENABLED:-false}"
  volumes:
    - /projects/${OPDATA:-opdata}:/var/openproject/assets"

services:
  db:
    image: postgres:13
    <<: *restart_policy
    stop_grace_period: "3s"
    volumes:
      - /projects/${PGDATA:-pgdata}:/var/lib/postgresql/data"
    environment:
      POSTGRES_PASSWORD: ${POSTGRES_PASSWORD:-p4ssw0rd}
      POSTGRES_DB: openproject
    networks:
      - backend

  cache:
    image: memcached
    <<: *restart_policy
    networks:
      - backend

  proxy:
    <<: [*image, *restart_policy]
    command: "./docker/prod/proxy"
    ports:
      - "${PORT:-8085}:80"
    environment:
      APP_HOST: web
      OPENPROJECT_RAILS__RELATIVE__URL__ROOT: "${OPENPROJECT_RAILS__RELATIVE__URL__ROOT:-}"
    depends_on:
      - web
    networks:
      - frontend

  web:
    <<: *app
    command: "./docker/prod/web"
    networks:
      - frontend
      - backend
    depends_on:
      - db
      - cache
      - seeder
    labels:
      - autoheal=true
    healthcheck:
      test: ["CMD", "curl", "-f", "http://localhost:8080${OPENPROJECT_RAILS__RELATIVE__URL__ROOT:-}/health_checks/default"]
      interval: 10s
      timeout: 3s
      retries: 3
      start_period: 30s

  autoheal:
    image: willfarrell/autoheal:1.2.0
    volumes:
      - "/var/run/docker.sock:/var/run/docker.sock"
    environment:
      AUTOHEAL_CONTAINER_LABEL: autoheal

  worker:
    <<: *app
    command: "./docker/prod/worker"
    networks:
      - backend
    depends_on:
      - db
      - cache
      - seeder

  cron:
    <<: *app
    command: "./docker/prod/cron"
    networks:
      - backend
    depends_on:
      - db
      - cache
      - seeder

  seeder:
    <<: *app
    command: "./docker/prod/seeder"
    restart: on-failure
    networks:
      - backend

but the docker mount point
root@dockers:/projects/openproject/compose# docker volume inspect compose_pgdata
[
    {
        "CreatedAt": "2022-07-04T11:01:31-05:00",
        "Driver": "local",
        "Labels": {
            "com.docker.compose.project": "compose",
            "com.docker.compose.version": "1.29.2",
            "com.docker.compose.volume": "pgdata"
        },
        "Mountpoint": "/var/lib/docker/volumes/compose_pgdata/_data",
        "Name": "compose_pgdata",
        "Options": null,
        "Scope": "local"
    }

root@dockers:/projects/openproject/compose# docker volume inspect compose_opdata
[
    {
        "CreatedAt": "2022-07-04T11:01:31-05:00",
        "Driver": "local",
        "Labels": {
            "com.docker.compose.project": "compose",
            "com.docker.compose.version": "1.29.2",
            "com.docker.compose.volume": "opdata"
        },
        "Mountpoint": "/var/lib/docker/volumes/compose_opdata/_data",
        "Name": "compose_opdata",
        "Options": null,
        "Scope": "local"
    }
]

Thank you

/projects is a directory in the host machine not a docket volume. This tool is only to create a snapshot of docker volume. You can directly tar your /projects directory and transfer it to another machine.

oh gotcha thank you


Maybe add the "first form only" argument in the script?

When I work on annotating the congates, I realize that there are some duplicated words for the same concepts and languages. For example  the concept "wake up".
Should we add the "first form only" argument in the script to prevent the duplicated entries?

I do not think they are duplicated, there are rather synonymous forms.

I agree that they are synonymous forms, but I think the synonymous forms cause the trouble when it comes to cognate annotation.

Should we leave these in the data, or remove them?

Yes, this is an issue for the evaluation, not for the CLDF dataset: when writing "remove", please write "!remove!" instead.

I see. Then I close this issue for now.


⚠ SceneTime has degraded performance

In 418d2de, SceneTime ($ST) experienced degraded performance:

HTTP code: 200
Response time: 10268 ms

Resolved: SceneTime performance has improved in f8e06a5 after 6 minutes.


Error: Unable to build with error "Undefined symbols for architecture x86_64"

react-native-cli: 2.0.1
react-native: 0.39.2
react: 15.4.1
Firebase sdk: 3.11.0
Xcode: 8.2 (8C38)
building iOS version stoped with errors:
Undefined symbols for architecture x86_64:   "_deflate", referenced from:       +[NSData(GTMZlibAdditionsPrivate) gtm_dataByCompressingBytes:length:compressionLevel:mode:error:] in GoogleToolboxForMac(GTMNSData+zlib_133a1619f6f498b1872062af2031468a.o)   "_deflateEnd", referenced from:       +[NSData(GTMZlibAdditionsPrivate) gtm_dataByCompressingBytes:length:compressionLevel:mode:error:] in GoogleToolboxForMac(GTMNSData+zlib_133a1619f6f498b1872062af2031468a.o)   "_deflateInit2_", referenced from:       +[NSData(GTMZlibAdditionsPrivate) gtm_dataByCompressingBytes:length:compressionLevel:mode:error:] in GoogleToolboxForMac(GTMNSData+zlib_133a1619f6f498b1872062af2031468a.o)   "_inflate", referenced from:       +[NSData(GTMZlibAdditionsPrivate) gtm_dataByInflatingBytes:length:isRawData:error:] in GoogleToolboxForMac(GTMNSData+zlib_133a1619f6f498b1872062af2031468a.o)   "_inflateEnd", referenced from:       +[NSData(GTMZlibAdditionsPrivate) gtm_dataByInflatingBytes:length:isRawData:error:] in GoogleToolboxForMac(GTMNSData+zlib_133a1619f6f498b1872062af2031468a.o)   "_inflateInit2_", referenced from:       +[NSData(GTMZlibAdditionsPrivate) gtm_dataByInflatingBytes:length:isRawData:error:] in GoogleToolboxForMac(GTMNSData+zlib_133a1619f6f498b1872062af2031468a.o) ld: symbol(s) not found for architecture x86_64 clang: error: linker command failed with exit code 1 (use -v to see invocation)

See my PR https://github.com/evollu/react-native-fcm/pull/239 . Rebuild your schemes.

Remove/add schema didn't work for me but adding libz.tbd what i found in your linked issue help

Can this be closed? Maybe we should include to add libz.tbd in the fix documentation PR

seems to be a firebase document issue? I don't remember where I got it but libz.tbd is in my solution too


Import error: trace.so

ImportError                               Traceback (most recent call last)
<ipython-input-1-005053b25f37> in <module>()
----> 1 import iris

/Users/Yefee/miniconda2/lib/python2.7/site-packages/iris/__init__.py in <module>()
    108
    109 import iris.config
--> 110 import iris.cube
    111 import iris._constraints
    112 import iris.fileformats

/Users/Yefee/miniconda2/lib/python2.7/site-packages/iris/cube.py in <module>()
     37 import numpy.ma as ma
     38
---> 39 import iris.analysis
     40 from iris.analysis.cartography import wrap_lons
     41 import iris.analysis.maths

/Users/Yefee/miniconda2/lib/python2.7/site-packages/iris/analysis/__init__.py in <module>()
     59 import scipy.stats.mstats
     60
---> 61 from iris.analysis._area_weighted import AreaWeightedRegridder
     62 from iris.analysis._interpolation import (EXTRAPOLATION_MODES,
     63                                           RectilinearInterpolator)

/Users/Yefee/miniconda2/lib/python2.7/site-packages/iris/analysis/_area_weighted.py in <module>()
     21 import numpy as np
     22
---> 23 from iris.analysis._interpolation import get_xy_dim_coords, snapshot_grid
     24 import iris
     25 import iris.experimental.regrid as eregrid

/Users/Yefee/miniconda2/lib/python2.7/site-packages/iris/analysis/_interpolation.py in <module>()
     29
     30 from iris.analysis._scipy_interpolate import _RegularGridInterpolator
---> 31 from iris.analysis.cartography import wrap_lons as wrap_circular_points
     32 from iris.coords import DimCoord, AuxCoord
     33 import iris.util

/Users/Yefee/miniconda2/lib/python2.7/site-packages/iris/analysis/cartography.py in <module>()
     31 import numpy.ma as ma
     32
---> 33 import cartopy.img_transform
     34 import cartopy.crs as ccrs
     35 import iris.analysis

/Users/Yefee/miniconda2/lib/python2.7/site-packages/cartopy/__init__.py in <module>()
    108 # Commonly used sub-modules. Imported here to provide end-user
    109 # convenience.
--> 110 import cartopy.crs
    111 import cartopy.feature

/Users/Yefee/miniconda2/lib/python2.7/site-packages/cartopy/crs.py in <module>()
     36
     37 from cartopy._crs import CRS, Geocentric, Geodetic, Globe, PROJ4_RELEASE
---> 38 import cartopy.trace
     39
     40

ImportError: dlopen(/Users/Yefee/miniconda2/lib/python2.7/site-packages/cartopy/trace.so, 2): Library not loaded: @rpath/./libgeos_c.1.dylib
  Referenced from: /Users/Yefee/miniconda2/lib/python2.7/site-packages/cartopy/trace.so
  Reason: Incompatible library version: trace.so requires version 10.0.0 or later, but libgeos_c.dylib provides version 9.0.0

A similar issue is here: https://github.com/SciTools/iris/issues/1543
I am a  freshman to python. Recently transferring from MATLAB. Sorry I have no idea to solve it.

This is to do with the version of libgeos you have installed. Note that trace.so is part of cartopy, this closed ticket may be helpful to you: https://github.com/SciTools/cartopy/issues/660.
How did you install iris? If it was via conda I think the solution might be to obtain iris (and newer version of geos, and maybe shapely...) from the scitools conda channel:
conda install -c scitools iris geos shapely

I agree with @ajdawson's solution here. Alternatively, the conda-forge channel could be used to similar effect:
conda install -c conda-forge iris geos shapely

Either way, this is certainly a cartopy issue rather than and Iris one, and I'd be more than happy to help (on SciTools/cartopy#660 or in a new issue on cartopy) if there is anything more I can do.
Thanks @Yefee - and welcome to Python! :+1:


How can I determine if the enter key was pressed?

The onTextChange function provides a delta argument that gives you the last keystroke.
If I hit the enter key, the delta returns "↵", but I can't query for that in my JS code.
Is there another way to query for certain key strokes?

You can bind onKeyDown or onKeyUp on the element containing the editor and you'll receive an event as usual. You can wrap the editor into another component, or build a custom component using the mixin.
I want to allow a few more events on the editor, such as keypress events, but I want to hear what they would be used for before. Do you have a specific use-case?

The methods onKeyDown/onKeyUp do not listen for the "enter" key button press (but they do for other keys).  I just tested this by binding them to the element containing the editor.
However, I did find an alternate solution.  Via the onChange method on the ReactQuill component, I can query for the "enter" key button using this statement which yields a value of 10:
delta.ops[delta.ops.length -1].insert.charCodeAt(0);

My use case is very specific.  I'm allowing users to use the text editor while watching videos.  When pressing letter keys the video automatically pauses.  When pressing the "enter" key, the video resumes playing.

It seems there is an issue where Quill prevents keydown events for enter, backspace and delete from bubbling up. But you can still receive them when using onKeyUp.
I've just pushed some "official" hooks for keypress events so that you won't have to bind them on a container. See also [the note about onKeyDown] in the README.

I just downloaded version 0.2.2, and onKeyPress does not work for me. Neither does onKeyDown.

There is a note in the README that explains that why you have to bind to onKeyUp to receive events from keys like enter, backspace or delete. Please try this and let me know.


Enable normalize button in normalization tab if ...

sample and ob have been loaded
but if no obs have been loaded
if there is at least one background region selected

Update Instructions text according to condition:

if no ob, tells the user to select at least one background region
if ob, background to help the normalization

and

sample ROI just here to help figure out where is the sample.


update feathers-shallow-populate

See https://github.com/Mattchewone/feathers-shallow-populate/pull/22 from @areiterer
As of ^1.4.0 I come across the same problem as @areiterer figured out. Populating an empty object instead of not setting the item at all, leads to many items in my FeathersVuex which are empty temp records.
Version ^1.2.0 which I used lastly did not behave that way. I'm afraid it's a problem I introduced with my changes. Sorry about that!
Please @marshallswain, it would be really nice if you could make this a feathers-shallow-populate release and update the dependency in feathers-graph-populate to that version.
As always, I highly appreciate your work! Thanks in advance!

Thanks a lot!


A question about Exclusive Loss

Sorry for disturbing you. When I read the code about the Exclusive Loss, I found that Exclusive Loss in the “forward“ function uses targets labels. However, in the paper, the Exclusive Loss doesn't need labels to training.

@xiekun2019
Hi,  the target is exactly the index i in the paper. It is used to figure out the i-th vector as indicated in the numerator.
We do not use any ID labels here.
Since we implement the exclusive loss using the cross entropy function, it would be better if you can have a look at the cross entropy function.

That's it, I get it, thanks!


netsurf.buildsystem: 1.8 -> 1.9

Semi-automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/netsurf-buildsystem/versions.
meta.description for netsurf.buildsystem is: "Build system for netsurf browser"
meta.homepage for netsurf.buildsystem is: "http://www.netsurf-browser.org/"
meta.changelog for netsurf.buildsystem is: ""
Updates performed

Version update

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

directory tree listing: https://gist.github.com/8007041bc72873fc4a300604d290c76e

du listing: https://gist.github.com/a56e3ef9f615829bc9dc4bc2f98a9663

Rebuild report (if merged into master) (click to expand)

94 total rebuild path(s)

36 package rebuild(s)

36 x86_64-linux rebuild(s)
25 i686-linux rebuild(s)
8 x86_64-darwin rebuild(s)
25 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
gegl_0_4
gimp
gimp-with-plugins
gimpPlugins.exposureBlend
gimpPlugins.focusblur
gimpPlugins.fourier
gimpPlugins.gap
gimpPlugins.gimplensfun
gimpPlugins.gmic
gimpPlugins.lightning
gimpPlugins.lqrPlugin
gimpPlugins.resynthesizer
gimpPlugins.resynthesizer2
gimpPlugins.texturize
gimpPlugins.ufraw
gimpPlugins.waveletSharpen
gnome-photos
gnome3.gegl_0_4
gnome3.gnome-photos
netsurf.browser
netsurf.buildsystem
netsurf.libcss
netsurf.libdom
netsurf.libhubbub
netsurf.libnsbmp
netsurf.libnsfb
netsurf.libnsgif
netsurf.libnslog
netsurf.libnspsl
netsurf.libnsutils
netsurf.libparserutils
netsurf.libsvgtiny
netsurf.libutf8proc
netsurf.libwapcaplet
netsurf.nsgenbind
ufraw

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/1bvfi5vggfbnl3j1n5bd1p11gdakswj8-netsurf-buildsystem-1.9 \
  --option binary-caches 'https://cache.nixos.org/ https://r-ryantm.cachix.org/' \
  --option trusted-public-keys '
  r-ryantm.cachix.org-1:gkUbLkouDAyvBdpBX0JOdIiD2/DP1ldF3Z3Y6Gqcc4c=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(r-ryantm's Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A netsurf.buildsystem https://github.com/r-ryantm/nixpkgs/archive/0bc5e9d4eb0d35fb474bb0f6e2dceebcb8f7adff.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/1bvfi5vggfbnl3j1n5bd1p11gdakswj8-netsurf-buildsystem-1.9
ls -la /nix/store/1bvfi5vggfbnl3j1n5bd1p11gdakswj8-netsurf-buildsystem-1.9/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review 1

3 packages marked as broken and skipped:
- gimpPlugins.exposureBlend
- gimpPlugins.focusblur
- gimpPlugins.texturize

30 packages built:
- gegl_0_4
- gimp
- gimp-with-plugins
- gimpPlugins.fourier
- gimpPlugins.gap
- gimpPlugins.gimplensfun
- gimpPlugins.gmic
- gimpPlugins.lightning
- gimpPlugins.lqrPlugin
- gimpPlugins.resynthesizer
- gimpPlugins.ufraw
- gimpPlugins.waveletSharpen
- gnome-photos
- netsurf.browser
- netsurf.buildsystem
- netsurf.libcss
- netsurf.libdom
- netsurf.libhubbub
- netsurf.libnsbmp
- netsurf.libnsfb
- netsurf.libnsgif
- netsurf.libnslog
- netsurf.libnspsl
- netsurf.libnsutils
- netsurf.libparserutils
- netsurf.libsvgtiny
- netsurf.libutf8proc
- netsurf.libwapcaplet
- netsurf.nsgenbind
- ufraw

Maintainer pings
cc @vrthra for testing.

already on master

already on master


Copy address to try a more internationalized interaction

Updates:

Update the copy address feature to incorporate a more internationalized interaction. For example, some leading web3 companies include openzeppelin.com 、consensys.io、etherscan.io and others.
Provide feedback in the connectwallet example after providing the link, as the Metamask pop-up won't appear when connecting again.

I'm not sure how to fix this CI issue I'm facing. I can assure you that I have already passed the local test coverage and test CI. If someone could give me some hints, I would greatly appreciate it.

You need to add test case to cover this line.

 You need to add test case to cover this line.

But why are all my tests passing when I run it locally?

 You need to add test case to cover this line.

But why are all my tests passing when I run it locally?

The passing of the unit test can only indicate that the test results of the current existing test cases are not problematic, but this report shows that the test case coverage has decreased, that is, after you add new code, no new test cases are added to test your code. You can run the coverage test locally by running pnpm test:coverage. And view the ./coverage/index.html test rate coverage report in your browser

 You need to add test case to cover this line.

But why are all my tests passing when I run it locally?

The passing of the unit test can only indicate that the test results of the current existing test cases are not problematic, but this report shows that the test case coverage has decreased, that is, after you add new code, no new test cases are added to test your code. You can run the coverage test locally by running pnpm test:coverage. And view the ./coverage/index.html test rate coverage report in your browser

Thank you very much for your response. However, what confuses me is that the test coverage is also perfect when I run it locally.

 You need to add test case to cover this line.

But why are all my tests passing when I run it locally?

The passing of the unit test can only indicate that the test results of the current existing test cases are not problematic, but this report shows that the test case coverage has decreased, that is, after you add new code, no new test cases are added to test your code. You can run the coverage test locally by running pnpm test:coverage. And view the ./coverage/index.html test rate coverage report in your browser

Thank you very much for your response. However, what confuses me is that the test coverage is also perfect when I run it locally.

You can open ./coverage/index.html in your browser and check this file change, you will get it.

And you can see the report online: https://app.codecov.io/gh/ant-design/ant-design-web3/pull/448?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=checks&utm_campaign=pr+comments&utm_term=ant-design

And you can see the report online: https://app.codecov.io/gh/ant-design/ant-design-web3/pull/448?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=checks&utm_campaign=pr+comments&utm_term=ant-design

I peeped that, anyway thanks


Adding a suitable Logo

Is your feature request related to a problem? Please describe.
Adding a suitable logo for AgriLearn throughout the website would help in its overall enhancement.

@Suchitra-Sahoo  kindly assign me this issue under gssoc'24

@Suchitra-Sahoo I would like to really add a nice and meaningful logo to this website which would add to the overall effectiveness and user-friendly nature of this website.Kindly assign me this task.

@Nayanika1402  please create your own issue and work upon it. @Suchitra-Sahoo  kindly assign this issue to me since I created The issue

@Suchitra-Sahoo kindly assign me this issue

@Suchitra-Sahoo i would like to make changes in   color , font , font weight and other necessary by adding a suitable logo


chore: add cloudantUrl

Adding functionality for Kube and Knative toolchain deployment

@ChuckCox Have you tested this locally?

Tested locally:
npm start

><EMAIL_ADDRESS>start /Users/ChuckCox/GitHub/node-red-app
> node --max-old-space-size=160 index.js --settings ./bluemix-settings.js -v

7 Aug 10:56:06 - Starting Node-RED on IBM Cloud bootstrap
7 Aug 10:56:06 - Loading bluemix-settings.js
[2020-08-07T10:56:06.607] [INFO] ibm-cloud-env - Initializing with /Users/ChuckCox/GitHub/node-red-app/config/mappings.json
[2020-08-07T10:56:06.608] [WARN] ibm-cloud-env - File does not exist /Users/ChuckCox/GitHub/node-red-app/config/mappings.json
7 Aug 10:56:06 - Failed to find Cloudant url
7 Aug 10:56:06 - No Cloudant service found
7 Aug 10:56:06 - Falling back to localfilesystem storage. Changes will *not* be saved across application restarts.
7 Aug 10:56:06 - Loading application settings
7 Aug 10:56:06 - Starting first-use setup
7 Aug 10:56:06 - Waiting for first-use setup to complete
7 Aug 10:56:46 - Received first-use setup configuration
7 Aug 10:56:47 - Stopping first-use setup application
7 Aug 10:56:47 - Enabling adminAuth security - set NODE_RED_USERNAME and NODE_RED_PASSWORD to change credentials
7 Aug 10:56:47 - Disabled anonymous read-only access - set NODE_RED_GUEST_ACCESS to 'true' to enable
7 Aug 10:56:47 - [info]

Welcome to Node-RED
===================

7 Aug 10:56:47 - [info] Node-RED version: v1.1.3-git
7 Aug 10:56:47 - [info] Node.js  version: v11.14.0
7 Aug 10:56:47 - [info] Darwin 19.6.0 x64 LE
7 Aug 10:56:47 - [info] Loading palette nodes
7 Aug 10:56:49 - [info] Settings file  : /Users/ChuckCox/GitHub/node-red-app/bluemix-settings.js
7 Aug 10:56:49 - [info] HTTP Static    : /Users/ChuckCox/GitHub/node-red-app/public
7 Aug 10:56:49 - [info] Context store  : 'default' [module=memory]
7 Aug 10:56:49 - [info] User directory : /Users/ChuckCox/GitHub/node-red-app/.node-red
7 Aug 10:56:49 - [warn] undefined
7 Aug 10:56:49 - [warn] Projects disabled : set editorTheme.projects.enabled=true to enable
7 Aug 10:56:49 - [info] Flows file     : /Users/ChuckCox/GitHub/node-red-app/.node-red/flows.json
7 Aug 10:56:49 - [info] Creating new flow file
7 Aug 10:56:49 - [warn]

---------------------------------------------------------------------
Your flow credentials file is encrypted using a system-generated key.

If the system-generated key is lost for any reason, your credentials
file will not be recoverable, you will have to delete it and re-enter
your credentials.

You should set your own key using the 'credentialSecret' option in
your settings file. Node-RED will then re-encrypt your credentials
file using your chosen key the next time you deploy a change.
---------------------------------------------------------------------

7 Aug 10:56:49 - [info] Server now running at http://<IP_ADDRESS>:1880/red/
7 Aug 10:56:49 - [info] Starting flows
7 Aug 10:56:49 - [info] Started flows
^C7 Aug 10:57:06 - [info] Stopping flows
7 Aug 10:57:06 - [info] Stopped flows

@ChuckCox Have you tested the docker files locally? Are they copied from somewhere? The images don't match between the two.

I did my testing of the CF deployment, using the "node-red-app-code-monitor" toolchain, and switching the Input value to my ChuckCox-cloudantUrl branch: https://cloud.ibm.com/devops/toolchains/5aafbebd-34a6-4e2d-adb9-4c03d229a463?env_id=ibm:yp:us-south
Here is my toolchain that I did all of my testing of the Helm and Knative deployments in: https://cloud.ibm.com/devops/toolchains/9519c38d-152f-44aa-b170-7c38cadedb70?env_id=ibm:yp:us-south
The "Skit-Pipeline-Helm" and "Skit-Pipeline-Knative" pipelines show failing in the Verification stage only because I did not supply a Slack webhook environment variable for the "slack_message.sh" script (https://github.com/IBM/devex-skit-assets/blob/development/scripts/verification_experience_test.sh#L36), so it exits incorrectly. If you look at the logs for the Verification stage, you will see the message "Skit Experience Test Passed". The Slack webhook will be provided in the newly created "node-red-app-code-monitor" toolchain.


fix a bug in consistent-spelling hook

fix #26
I was able to test this locally with the same example I used to report the issue.

Correct ! It made me realize I forgot to setup the end-of-file-fixer hook from pre-commit's first party collection ;)


Reduce nesting level in output ELM for binary operators such as union and intersect

When translating binary operators that can be chained such as union, intersect, and except, the translator will produce a right-deep tree. For long chains, this can result in a deeply nested ELM output.
Within ELM itself, this could potentially be addressed by defining these operators as nary instead of binary operators. That should be submitted as a comment against the CQL specification.
Within the translator, without changing ELM, the output should be changed to produce a bushy tree, rather than a right-deep tree as it currently does.

See also http://www.hl7.org/dstucomments/showdetail_comment.cfm?commentid=1500

Included in the 1.3 CQL Ballot release.


Mj-social in mj-container problem

Hello,
I found something and got no clue how to fix it.
Something happens and the page doesn't render anything custom if the mj-social tag is nested in the mj-container, if the mj-container tag, is removed, it works as it should.
https://mjml.io/try-it-live/SyDt3Hy2m
Any clue on how to approach this problem?
Thank you in advance!

mj-container is deprecated in MJML 4, just remove it. If you leave it, your template is considered to be MJML 3 so the syntax you are using for mj-social is invalid (as it changed between MJML v3 and MJML v4).

Thank you for the clarification!


chore: add e2e tests for codegen add, both with and without apiId

Description of changes
Adding a regression test for basic happy-case functionality from https://github.com/aws-amplify/amplify-category-api/issues/38 this issue. Verified that using CLI v8.1.0 will fail the second test here.
Issue #, if available
https://github.com/aws-amplify/amplify-category-api/issues/38
Description of how you validated changes
Ran test, and verified test failed for 8.1.0
Checklist

[x] PR description included
[x] yarn test passes
[x] Tests are changed or added

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Codecov Report

Merging #10566 (184dcd3) into master (b4ca7fd) will increase coverage by 0.00%.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master   #10566   +/-   ##
=======================================
  Coverage   47.30%   47.30%
=======================================
  Files         665      665
  Lines       32880    32880
  Branches     6632     6632
=======================================
+ Hits        15554    15555    +1
+ Misses      15657    15656    -1
  Partials     1669     1669

Impacted Files
Coverage Δ

...li/src/domain/amplify-usageData/getUsageDataUrl.ts
100.00% <0.00%> (+12.50%)
:arrow_up:

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more


[question] Is duckcli a wrap of a specific version of duckdb or does it use my duckdb install underneath?

I am using duckdb v0.10.0
I noticed that when I try to do a simples select with duckcli in a duckdb db, it just freezes.
It works with vanilla duckdb.

I should mark this repo archived. I haven't had time to work on it anymore.


Can't sign in to app

Expected Behavior
Sign in to the app
Actual Behavior
Can't sign in. We have a local standalone Youtrack server. When I try to login into the app to use I've got an error like this. FQDN is correct and so is token. I don't understand what's the problem.

Steps to Reproduce the Problem

login to the app

Specifications

T-REC Version: 1.2.0
Platform: windows

Hi @volcano1111 thanks for your issue, I will check that

same here, have you been able to have a look?


Fixd undefined issue in build

Missing "blog" and "comment" in themeConfig.

It has fixed in version 1.2.0


Global memory statistician for small memory fragments

RisingWave will occupy some small memory during its working process, for example:

actor channel will buffer 2048 rows(the upper limit may be size based 1MB late)
stateful executors' mem tables will occupy memory until it's size exceed 4MB

Although these memory footprints are small, they vary dynamically with workload. If there are many materialize views/Actors in the current system, the total amount of these small memories will also increase accordingly, thereby increasing the risk of OOM(We have witnessed such issues during longevity test).
The current strategy involves global memory management, where we use jemalloc to monitor the memory usage of Compute Node. Once a certain threshold is reached, we start evicting the LRU cache. However, we currently do not track these small memories because we consider them to be small and quickly released. Since customers create a lot of MVs in the cluster, we should count these small memories and make some mitigation strategies after they add up.
possible methods:

introduce some kind of memory tracking to monitor these fragmented memories.
more reasonable memory allocation according to the memory usage(eg. more reserved memory or allocate a fixed size to it in streaming memory)
introduce some kind of control strategy when these memories are too large.

RisingWave will occupy some small memory during its working process, for example:

actor channel will buffer 2048 rows(the upper limit may be size based 1MB late)
stateful executors' mem tables will occupy memory until it's size exceed 4MB

Two more examples worth mentioning:

source reader may buffer source messages. For example, by per librdkafka consumer instance can buffer 64MB message by default
sink may buffer stream chunks (without spill) per epoch in order to do compaction if sink has different PK than upstream

Any specific tasks?


added Java11 base images for docker builds

fixes #1647

@dev-gaur : Could you please resolve conflicts?

@dev-gaur : Could you please resolve conflicts?

working on it


nginx crash observed on nginx 1.4.5 and ModSecurity 2.7.7

CentOS release 6.5 (Final) 2.6.32-431.el6.x86_64 #1 SMP
I have ModSecurity turned on with "SecRuleEngine DetectionOnly" set in modsecurity.conf
I also have enabled Caching in nginx.conf
When I am sending a request with a sql injection pattern in the URL, I am seeing that nginx crashes with Segmentation Violation with the following backtrace:
(gdb) bt
#0  ngx_http_write_filter (r=0x1f467c0, in=0x1f4a778) at src/http/ngx_http_write_filter_module.c:121
#1  0x0000000000464b8d in ngx_http_chunked_body_filter (r=0x1f467c0, in=)
at src/http/modules/ngx_http_chunked_filter_module.c:111

#2  0x0000000000476067 in ngx_http_spdy_body_filter (r=0x1f467c0, in=0x1f4a778) at src/http/ngx_http_spdy_filter_module.c:618
#3  0x000000000046a4a5 in ngx_http_gzip_body_filter (r=0x1f467c0, in=0x1f4a778) at src/http/modules/ngx_http_gzip_filter_module.c:325
#4  0x000000000046b3ea in ngx_http_postpone_filter (r=0x1f467c0, in=0x1f4a778) at src/http/ngx_http_postpone_filter_module.c:83
#5  0x000000000046bb49 in ngx_http_ssi_body_filter (r=0x1f467c0, in=)
at src/http/modules/ngx_http_ssi_filter_module.c:396

#6  0x000000000046ffbf in ngx_http_charset_body_filter (r=0x1f467c0, in=)
at src/http/modules/ngx_http_charset_filter_module.c:553

#7  0x00000000004914cb in ngx_http_modsecurity_body_filter (r=, in=)
at .../modsecurity-apache_2.7.7/nginx/modsecurity/ngx_http_modsecurity.c:1254

#8  0x000000000041de58 in ngx_output_chain (ctx=0x1f4a5c0, in=0x7fffa65158e0) at src/core/ngx_output_chain.c:66
#9  0x000000000045327f in ngx_http_copy_filter (r=0x1f467c0, in=0x7fffa65158e0) at src/http/ngx_http_copy_filter_module.c:143
#10 0x0000000000464f1c in ngx_http_range_body_filter (r=0x1f467c0, in=0x7fffa65158e0)
at src/http/modules/ngx_http_range_filter_module.c:587

#11 0x00000000004479e2 in ngx_http_output_filter (r=0x1f467c0, in=0x7fffa65158e0) at src/http/ngx_http_core_module.c:1956
#12 0x0000000000468fe4 in ngx_http_cache_send (r=0x1f467c0) at src/http/ngx_http_file_cache.c:1022
#13 0x000000000045de17 in ngx_http_upstream_cache_send (r=0x1f467c0, u=0x1f49978) at src/http/ngx_http_upstream.c:868
#14 0x00000000004607ae in ngx_http_upstream_cache (r=0x1f467c0) at src/http/ngx_http_upstream.c:774
#15 ngx_http_upstream_init_request (r=0x1f467c0) at src/http/ngx_http_upstream.c:493
#16 0x0000000000460ecf in ngx_http_upstream_init (r=0x1f467c0) at src/http/ngx_http_upstream.c:466
#17 0x00000000004567de in ngx_http_read_client_request_body (r=0x1f467c0, post_handler=0x460dba <ngx_http_upstream_init>)
at src/http/ngx_http_request_body.c:84

#18 0x0000000000482bf0 in ngx_http_proxy_handler (r=0x1f467c0) at src/http/modules/ngx_http_proxy_module.c:725
#19 0x0000000000448e5c in ngx_http_core_content_phase (r=0x1f467c0, ph=0x27daf98) at src/http/ngx_http_core_module.c:1408
#20 0x0000000000443723 in ngx_http_core_run_phases (r=0x1f467c0) at src/http/ngx_http_core_module.c:888
#21 0x0000000000443833 in ngx_http_handler (r=) at src/http/ngx_http_core_module.c:871
#22 0x000000000044e841 in ngx_http_process_request (r=0x1f467c0) at src/http/ngx_http_request.c:1828
After digging around the code, I found a potential issue in the function move_brigade_to_chain() in the following piece of code:
if (APR_BUCKET_IS_EOS(e)) {
if (cl == NULL) {
cl = ngx_alloc_chain_link(pool);
if (cl == NULL) {
break;
}
            cl->buf = ngx_calloc_buf(pool);
            if (cl->buf == NULL) {
                break;
            }

            cl->buf->last_buf = 1;
            *ll = cl;
        } else {
            cl->buf->last_buf = 1;
        }
        apr_brigade_cleanup(bb);
        return NGX_OK;
    }

Adding 'cl->next = NULL;' between the lines 'cl->buf->last_buf = 1;' and '*ll = cl;' fixed the issue for me.
Is this a valid fix?

Marking as won't fix in 2.x as it is no longer a concern in libModSecurity:
https://github.com/SpiderLabs/ModSecurity/tree/v3/master


Raise an error when attempting to set an expectation on nil

Thanks so much for the contribution!
To make reviewing this PR a bit easier, please fill out answers to the following questions.
What was the end-user problem that led to this PR?
The problem was we could accidentally set an expectation on nil, leading to an RSpec warning
What was your diagnosis of the problem?
My diagnosis was we should just error when we do so

@bundlerbot r+


LoadBalancerFrontendIPConfigurationCountLimitReached but total exposed microservices are 4

Is this a request for help?:
Yes
Is this an ISSUE or FEATURE REQUEST? (choose one):
ISSUE

Orchestrator and version (e.g. Kubernetes, DC/OS, Swarm)
kubernetes : 1.5.3
What happened:
I am receiving Exception "LoadBalancerFrontendIPConfigurationCountLimitReached" when exposing a kubernetes service using LoadBalancer. I deployed multiple pods and exposed them using load balancer, but then I deleted the services and currently only 4 kubernetes services are exposed using LoadBalancer. But when I try to expose a 5th service, it is saying that the limit has been reach. I know the default limit is 10 IPs but I deleted the previously exposed services. My question is that when we delete a kubernetes service, does it removes the IP address from the load balancer at the same time or it is deleted after some delay? If it deletes the IP at the same time then why in my case it failed to do so?
What you expected to happen:
When we delete a kubernetes service, it should remove the IP address from the load balancer at the same time.
How to reproduce it (as minimally and precisely as possible):
Anything else we need to know:
I tried to delete the unused IP addresses using azure portal but it return an error:
Fail to delete public IP adresss. [name] can not be deleted since it is still allocated to resource [resource name].

It does remove the public IP (it's async process), but I do observe leaking in a few cases. Especially when you re-create the service using same name.
To mitigate the issue, you will need find the load balancer, and remove the load balancing rules of the leaked public IP and then remove the leaked public IP from frontend IP pool. Only after these 2 step you can then delete the public IP.

+1

@mboersma We still face this issue with recent versions of aks-engine. Should we reopen this issue in the new repository?

@palmerabollo yes, please. It would help to have a fresh issue with current steps to reproduce to focus on.


Add talosctl

[ ] Have you signed the Contributor License Agreement?
[x ] Have you checked that there aren't other open pull requests for the same manifest update/change?

[ x] Does your manifest conform to the 1.2 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

@microsoft-github-policy-service agree

Can someone please help me understand failure reason? The response code is Ok, but response is empty.