Query Builder Not Escaping Values For Update/Insert When Disabling _protect_identifiers

public function set($key, $value = '', $escape = NULL)
	{
		$key = $this->_object_to_array($key);

		if ( ! is_array($key))
		{
			$key = array($key => $value);
		}

		is_bool($escape) OR $escape = $this->_protect_identifiers;

		foreach ($key as $k => $v)
		{
			$this->qb_set[$this->protect_identifiers($k, FALSE, $escape)] = ($escape)
				? $this->escape($v) : $v;
		}

		return $this;
	}

@narfbg This is the same type of problem you fixed for me yesterday.  It looks like the escape of the value is going off the protect_identifiers
I believe the line in the foreach should be
			$this->qb_set[$this->protect_identifiers($k, FALSE, $escape)] = $this->escape($v);

Fixed in 2 more instances, shouldn't be any more of these left ... thanks for digging into it.
Still, I should repeat what I said on the forums - you should NOT ever rely on the possibility to disable _protect_identifiers. Also, it's no longer called "Active Record", but "Query Builder" instead.

Referencing 43afc71b777b00cfc2638add6fa3c47d333c5e04

Thank you!  Warning is understood.  My bad on using the old "Active Record".  I fixed the title of the issue.

Damn auto-closing ... I specifically reopened this, so I can comment on it as an open issue.
Anyway, the changes related to this have been reverted. I don't want to be spreading bad advice via our issue tracker, but I'll tell you this - there's another hacky way to avoid field name escaping. It's up to you to find out what it is if you really need it that bad.

subselects are the real issue as to why they need to be shut off in my case.  It is surrounding them with `` and I use them quite heavily.    It does not make a lot of sense to me.  I would disabled it just from a performance standpoint as it seems pretty unnecessary in most cases.

Well, if you ask me - the moment you start passing raw SQL to the query builder, you're just wasting your time with using QB in the first place and you gain absolutely nothing from it. Use raw queries, there's nothing wrong with them.

So if I understand correctly, this does fix the issue I reported in #3592 ??

It is not a waste of time at all!  It stops from having to concatenate your query and the code is less verbose. I also do not have to check if I have already included my "AND" or start off my where clause with a 1=1 to get around that.  It just feels much cleaner.  I don't have to worry about escaping my values because it is already being done.  The whole point of it is to assist with dynamic queries, but right now you are saying that if your dynamic query contains a subselect you are out of luck and go use raw sql.  Raw queries are fine if you have a fixed amount of criteria and selected columns.
What is the point of trying to escape everything?  People should be trying to stay away from reserved words in their tables and column names anyway.  Why not just have the option to turn it on when needed, it would be one less thing the query builder would have to run through?

@aanbar Yes, although I still think your use case is invalid, given that you can do the same via where() or a raw query (you are in fact building it yourself anyway).
@zepernick
Start with a WHERE condition of '1' and you can always append ' AND something' to it, while parametirized queries solve the escaping problem. The only real benefit in using a query builder is portability (that is, you can transfer from e.g. mssql to mysql without rewriting your queries).

Your argument about what's the actual point of trying to escape everything is more logical, but it's people reporting bugs that it didn't escape something who are demanding it. I for one don't see a substantial benefit in either choice - it already did escaping when I joined the project.
You're free to make a feature request for a configuration option on identifier escaping of course, but do it properly in a separate issue.

@narfbg really, calm down man, lol.  Yes, you did not say that.  That is what I was deriving from our conversation. Sorry for getting your knickers in a twist.

I use dirty words regularly, that has nothing to do with my mood. :)

@narfbg

I'm not saying you're out of luck, I'm saying it makes no sense to shove raw SQL into a query builder, because the whole point is that the query builder generates SQL for you.

If you build your initial SQL with query builder, then use something like get_compiled_select() to get the raw SQL to use in a sub-query, it is almost impossible to pass it back into query builder without breaking something, especially if you pass the sub-query into from(), which has no option for disabling escaping the query. Obviously, double-escaping is part of the problem if I do things this way, but my point is simply that sub-queries don't have to start as raw SQL.
I'd really be happy if I could just call a method to disable escaping until _reset_select()/_reset_write() is called. I generally only have a problem with it for 1-2% of the queries I might run, and most of those are on pages which require a larger number of queries than usual, so the choice of disabling it through the back door through unsupported configuration or optionally on some methods (but not having the option to disable it on other methods) just doesn't work.
I ended up hacking together a version of query builder's where() functionality in my model to return the text for my where clause, to make it easier to build a SQL string to pass to query(), when I only had to build the string and use query() because I couldn't stop the from() method from breaking my sub-query.

@mwhitneysdsu
public function from($from) -> public function from($from, $escape = TRUE)
Is this a good deal for you?

@ivantcholakov I think that would address the issue of that particular method not having the option, which may or may not fix my specific case, but I think it's indicative of a design problem that nearly every method in Query Builder needs an extra parameter to disable escaping queries.

@mwhitneysdsu It doesn't matter how it started, you're passing raw SQL to QB.

I was just pointing out that it's possible to pass raw SQL to QB without actually hand-coding the raw SQL...

because the whole point is that the query builder generates SQL for you.

but I'll stop, because I've dragged it far enough off the original point of this issue. Thanks for working on this.


Create Namespace node that references all NamespaceDeclarations which add declarations to its Scope

Currently, we add a Namespace Declaration whenever we see one, however, this means that in java we create a NamespaceDeclaration for every file in a package, and the declarations in the package don't know each other, neither do the NamespaceDeclarations. In C++ a Namespace can be declared multiple times and here we have the same problem, both NamespaceDeclarations add Declarations to the same Namespace, but there is no way of querying the graph for this relation. The query could be written as all Declarations in a NamespaceDeclaration of a specific Name, but no concrete Entity of a Namespace is present.

The added problem is, that current each NamespaceDeclaration has its own NameSpace, making resolution of functions that are in different files but the same namespace impossible.
Thus, in #378 I have created a workaround that very naivly looks up exist namespaces in the current scope and uses their NameScope, only creating a new name scope if necessary.


CI Update

Build.Reason:Manual by Alma Jenks
Build.Url:https://apidrop.visualstudio.com/Content CI/_build/results?buildId=358258&view=results
source_repo.branch:master
source_repo.csvPath:bundlepackages/aspnetcore
source_repo.url:https://apidrop.visualstudio.com/binaries/_git/mrefconfig

@mkArtakMSFT @captainsafia @adityamandaleeka who can review the ASP.NET Core 8 API's?

What's the fix for Cross reference not found: 'Duende.IdentityServer.Models.ApiResource'

It looks like it is an external API. It can be added to the missing_apis.yml file.

The files tab on this PR completely hangs the tab in my browser, so I cannot easily look at it.

That's a know issue.
We'll call out specific questions and I can review the API's once published.
@v-alje  can you answer

Where is <IP_ADDRESS> coming from?

@Rick-Anderson I'm not certain where it is coming from. The version number is coming from the assembly number of one of the libraries. The API was removed after ASP 5.0, and that configuration wasn't updated. For the current packages I only see version number mismatches for two packages:
The Microsoft.AspNetCore.RateLimiting NuGet package last published version 7.0.0-rc.2.22476.2. I believe this was added to .NET proper. Should the package be removed from the 7.0 and 8.0 versions?
For version 7.0 Microsoft.AspNetCore.Grpc.Swagger published version 0.3.5 for 7.0. For 8.0 it matches the rest of the packages. I have updated the configuration so it will pick up this version for the 7.0 moniker and kicked off the CI job again.
Were there any new packages introduced that we need to add to the configuration?


Failing build with html5ever

Hi,
I'm having issues building with nix for crates with html5ever as a dependency.
https://github.com/nuxeh/crate2nix-test is an example repo with a default.nix generated by crate2nix 0.3.0-beta.0
The build seems to fail due to the missing extern crate, new_debug_unreachable:
error[E0463]: can't find crate for `debug_unreachable`
  --> src/lib.rs:10:1
   |
10 | extern crate debug_unreachable;
   | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ can't find crate

error: aborting due to previous error

For more information about this error, try `rustc --explain E0463`.

I've done some debugging but haven't been able to track down what the issue is. The build order differs, and besides that, the only difference i can see is that the rustc invocations differ in the output directory, compared with building with cargo:
Running rustc --crate-name futf src/lib.rs --crate-type lib -C opt-level=3 -C codegen-units=4 --edition 2015 -C metadata=7a698c1757 -C extra-filename=-7a698c1757 --out-dir target/lib --emit=dep-info,link -L dependency=target/deps --extern mac=/nix/store/bjcw280v92drg93d89zpkh997xljfl89-rust_mac-0.1.1/lib/libmac-c77a325080.rlib --extern new_debug_unreachable=/nix/store/qz5ffk70ibllz27awjnpp9v7rbcrxnq5-rust_new_debug_unreachable-1.0.3/lib/libnew_debug_unreachable-6437ae92c0.rlib --cap-lints allow --color always

--out-dir target/lib
Running `rustc --crate-name futf /home/ed/.cargo/registry/src/github.com-1ecc6299db9ec823/futf-0.1.4/src/lib.rs --color never --crate-type lib --emit=dep-info,link -C debuginfo=2 -C metadata=b65310681f505b82 -C extra-filename=-b65310681f505b82 --out-dir /home/ed/git/url-bot-rs/target/debug/deps -L dependency=/home/ed/git/url-bot-rs/target/debug/deps --extern mac=/home/ed/git/url-bot-rs/target/debug/deps/libmac-705013cbc305aca9.rlib --extern debug_unreachable=/home/ed/git/url-bot-rs/target/debug/deps/libdebug_unreachable-6884cd8b03bd695e.rlib --cap-lints allow`

--out-dir /home/ed/git/url-bot-rs/target/debug/deps
Thanks.

Thanks for reporting this including a repo for reproducing the error and your detailed report!
I see a further difference:
--extern new_debug_unreachable=/nix/store/qz5ffk70ibllz27awjnpp9v7rbcrxnq5-rust_new_debug_unreachable-1.0.3/lib/libnew_debug_unreachable-6437ae92c0.rlib
vs
--extern debug_unreachable=/home/ed/git/url-bot-rs/target/debug/deps/libdebug_unreachable-6884cd8b03bd695e.rlib
Which explains why the library is available as debug_unreachable in the second case...
I'll investigate further...

I suspect this is down to the fact that the crate is called new_debug_unreachable but the library debug_unreachable: https://github.com/mbrubeck/rust-debug-unreachable/blob/fork/Cargo.toml#L12-L14
[lib]
name = "debug_unreachable"
path = "src/lib.rs"

That is exposed in the cargo metadata output:
{
        "id": "futf 0.1.4 (registry+https://github.com/rust-lang/crates.io-index)",
        "dependencies": [
          "mac 0.1.1 (registry+https://github.com/rust-lang/crates.io-index)",
          "new_debug_unreachable 1.0.3 (registry+https://github.com/rust-lang/crates.io-index)"
        ],
        "deps": [
          {
            "name": "mac",
            "pkg": "mac 0.1.1 (registry+https://github.com/rust-lang/crates.io-index)"
          },
          {
            "name": "debug_unreachable",
            "pkg": "new_debug_unreachable 1.0.3 (registry+https://github.com/rust-lang/crates.io-index)"
          }
        ],
        "features": []
      },

  {
      "name": "new_debug_unreachable",
      "version": "1.0.3",
      "id": "new_debug_unreachable 1.0.3 (registry+https://github.com/rust-lang/crates.io-index)",
      "license": "MIT",
      "license_file": null,
      "description": "panic in debug, intrinsics::unreachable() in release (fork of debug_unreachable)",
      "source": "registry+https://github.com/rust-lang/crates.io-index",
      "dependencies": [],
      "targets": [
        {
          "kind": [
            "lib"
          ],
          "crate_types": [
            "lib"
          ],
          "name": "debug_unreachable",
          "src_path": "/home/peter/.cargo/registry/src/github.com-1ecc6299db9ec823/new_debug_unreachable-1.0.3/src/lib.rs",
          "edition": "2015"
        },
        {
          "kind": [
            "example"
          ],
          "crate_types": [
            "bin"
          ],
          "name": "simple",
          "src_path": "/home/peter/.cargo/registry/src/github.com-1ecc6299db9ec823/new_debug_unreachable-1.0.3/examples/simple.rs",
          "edition": "2015"
        },
        {
          "kind": [
            "test"
          ],
          "crate_types": [
            "bin"
          ],
          "name": "check",
          "src_path": "/home/peter/.cargo/registry/src/github.com-1ecc6299db9ec823/new_debug_unreachable-1.0.3/tests/check.rs",
          "edition": "2015"
        }
      ],
// ...

Now I need to investigate on how to make this work with buildRustCrate. Renaming the crate alone does not work but we can additionally provide an explicit source:
diff --git a/default.nix b/default.nix
index de58004..50fb7ea 100644
--- a/default.nix
+++ b/default.nix
@@ -235,10 +235,14 @@ rec {
             };
         "new_debug_unreachable 1.0.3 (registry+https://github.com/rust-lang/crates.io-index)"
             = buildRustCrate {
-                crateName = "new_debug_unreachable";
+                crateName = "debug_unreachable";
                 version = "1.0.3";
                 edition = "2015";
                 sha256 = "1lxbd0n9gwvzg41bxiij0c28g1sp1jhp4h1rh10qisc8viyhsdj0";
+                src = builtins.fetchGit {
+                    url = "https://github.com/mbrubeck/rust-debug-unreachable";
+                    rev = "e374f6990882871d62ddcd1d9964b26ac50c5c7e";
+                };
                 libPath = "src/lib.rs";
                 authors = [
                     "Matt Brubeck<EMAIL_ADDRESS>
(Obviously, the source should be fetched from crates.io instead, this is just a proof of concept.)
With this fix, I run into a similar problem with the utf_8 crate, though, which exports the utf8 library.
I'll try to fix this today -- if that doesn't work out, you'll probably need to wait until at least next weekend :(

Does this solve your problem?
Please continue to send me such awesome bug reports, @nuxeh!

@kolloch it does appear to have solved the problem for the test repo, but for my own project, https://github.com/nuxeh/url-bot-rs, there's still the following error, this time for cssparser:
Building src/lib.rs (cssparser)
Running rustc --crate-name cssparser src/lib.rs --crate-type lib -C opt-level=3 -C codegen-units=4 --edition 2015 -C metadata=15b8375349 -C extra-filename=-15b8375349 --out-dir target/lib --emit=dep-info,link -L dependency=target/deps --extern cssparser_macros=/nix/store/8yl9hhivlcpr1q485d9h9kywwlxiia91-rust_cssparser-macros-0.3.5/lib/libcssparser_macros-57e42b786a.so --extern dtoa_short=/nix/store/4v891ffdrzxa2yv547mn3dcdc7iqvak0-rust_dtoa-short-0.3.2/lib/libdtoa_short-77d68b4a44.rlib --extern itoa=/nix/store/adhj88qqyin8zf9bd10xqv1dsizv3bhr-rust_itoa-0.4.3/lib/libitoa-a6f76c7f27.rlib --extern matches=/nix/store/j3laann65lzck6lyg9bpjk7gjw9ihk2k-rust_matches-0.1.8/lib/libmatches-47ddbc585f.rlib --extern phf=/nix/store/qcvaa9ya6vgcrpmmyiq042a8jpb4niid-rust_phf-0.7.24/lib/libphf-95b8a51ebb.rlib --extern procedural_masquerade=/nix/store/rwzpk2qq99bkvr83hckj8akf6xq84dfy-rust_procedural-masquerade-0.1.6/lib/libprocedural_masquerade-e6f18a276f.rlib --extern smallvec=/nix/store/nh1i2602n3nphcrxmznrjbnvmpn747iq-rust_smallvec-0.6.9/lib/libsmallvec-99bacd0cdd.rlib --cap-lints allow -L /build/cssparser-0.24.1.tar.gz/target/build/cssparser.out --cfg rustc_has_pr45225 --color always
error: extern location for cssparser_macros does not exist: /nix/store/8yl9hhivlcpr1q485d9h9kywwlxiia91-rust_cssparser-macros-0.3.5/lib/libcssparser_macros-57e42b786a.so

error[E0463]: can't find crate for `cssparser_macros`
  --> src/lib.rs:73:14
   |
73 | #[macro_use] extern crate cssparser_macros;
   |              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ can't find crate

error: aborting due to 2 previous errors

For more information about this error, try `rustc --explain E0463`.
builder for '/nix/store/dj2mwhaqg6yma100xscp0rx0s43h7zi8-rust_cssparser-0.24.1.drv' failed with exit code 1
cannot build derivation '/nix/store/378wm523mf1mscvbqc7vjhvy8n3hmdj8-rust_url-bot-rs-0.2.0.drv': 1 dependencies couldn't be built
error: build of '/nix/store/378wm523mf1mscvbqc7vjhvy8n3hmdj8-rust_url-bot-rs-0.2.0.drv' failed

So as far as i can tell, not out of the woods yet.
Thanks for looking at this, i'll certainly report anything else i find :)

Hmm, cssparser-macros seems to be a sub-crate within cssparser, I wonder if that has anything to do with this.
I can't imagine that that's uncommon, but it looks slightly suspicious for a normal Cargo project to me, just lib.rs and Cargo.toml.
https://github.com/servo/rust-cssparser/tree/master/macros

I created #5 for this.


assign latitude/longitude for Jin et al., 2018 data

There are data from the following sections:

Gukongkou section, Hubei Province
Hengdu section, Zhejiang Province
Jieling section, Hubei Province
The latitude/longitude values are not provided in the study (http://dx.doi.org/10.1130/G45302.1).

Liang et al., 2014 provides latitude/longitude for the Jieling section, Hubei Province:

The Jieling section (31°07′2.2″N, 111°24′13.7″E) is about 21 km west of Yuan'an town and 62 km north of Yichang city, western Hubei, central China (Fig. 1).
Latitude: 31.1173°N
Longitude: 111.4038°E

Hengdu: 28.748251, 118.544273
https://maps.app.goo.gl/gyESAYYxTXrWV4Wz8
Gudongkou instead of Gukongkou: 31.364509, 110.765805
See http://dx.doi.org/10.1144/jgs2018-208


Methods template

Installed product versions

Visual Studio: 2017 Professional
This extension: 1.9.10

Description
Hello in Test Framework (NUnit) and Moq Framework (Moq) I have $TestMethods$ variable in template. Could You please advise how to change how generated functions are built? Can't find anything on how to do that.
For example I mean that function is generated like:
`[Test]
public void GetNewCardData_StateUnderTest_ExpectedBehavior()
{
// Arrange
var unitUnderTest = CreateService();
        // Act
        var result = unitUnderTest.GetNewCardData();

        // Assert
        Assert.Fail();
    }`

but I want it to look like ie.:
`[Test]
public void GetNewCardData_StateUnderTest_ExpectedBehavior()
{
var result = _unitUnderTest.GetNewCardData();
        // Assert
        Assert.Fail();
    }`

Kind regards.

Hi,
You should be able to achieve that:
Go to Options > Unit Test Boilerplate Generator > Templates
Select and NUnit and Moq
Set 'Tested object creation format' to empty
Set 'Tested object reference format' to _unitUnderTest

Thanks,
for the info, it worked great.
Kind regards.


Redis server-assisted client side caching

Feature Request: Redis server-assisted client side caching
Envoy is an incredible tool that we've found critical for proxying redis connections, with the release of Redis 6, there seems to be an opportunity to take Envoy's redis capabilities to the next level by acting as an even faster local cache to Redis through the support of server-assisted client caching. Adding this capability to Envoy will make it simple to increase cache locality for many applications with a small, hot set of data, making it straightforward to build efficient sharded applications by handling asynchronous cache invalidation.
There would need to be substantial thought in terms of the memory profile, since it will shift Envoy from being a lighterweight proxy to also a relatively "thick" proxy-cache. I could imagine users tending to run a dedicated Envoy with a larger memory allocation separate from the lighterweight proxy variant for the purposes of isolation.
Regardless -- I really appreciate the work that Lyft, Matt Klein and all other contributors have put into this project, thank you all for the hard work 🙌

This sounds interesting. Definitely needs a design proposal. cc @envoyproxy/redis-dev

Hello! I have been working on it for sereveral days.
I tried to use lua script in envoy   to request  a outside redis controller whitch is implemented by go.
If envoy can enbed a internal redis controller and pirovide a lua srcipt handle to manipulate it,manty thing would be much easier.

Hello! I have been working on it for sereveral days.
I tried to use lua script in envoy   to request  a outside redis controller whitch is implemented by go.
If envoy can enbed a internal redis controller and pirovide a lua srcipt handle to manipulate it,manty thing would be much easier.

Hey @mattklein123 Wanted to check in and understand what the process usually looks like for new Envoy feature development. Has there ever been a sponsorship model used successfully? We'd be interested in contributing, but do not necessarily have Envoy expertise, so if there are Envoy developers who are lacking funding and interested, there may be an opportunity to collaborate.
Thanks again,
Cory

Has there ever been a sponsorship model used successfully? We'd be interested in contributing, but do not necessarily have Envoy expertise, so if there are Envoy developers who are lacking funding and interested, there may be an opportunity to collaborate.

We don't have anything formal here, but I can probably find someone for you to contract with. Can you reach out to me over email (available on my website) to discuss details?

I just wanted to check in and see if anyone is actively working on this or has thought about the general design. @ronakts and I would love to collaborate if there is ongoing work, otherwise, we're going to ramp up on this area of the code base and propose a design and work on an implementation.


Get a grey screen while the full pdf is downloaded before display

I am testing ViewerJS as a way to display large pdf files with navigation in webpages. The files are large, soem are multiple MB in size. eg. http://sanskritdocuments.org/ViewerJS/#..doc_z_otherlang_hindi/manas5_i.pdf
The page displays with a gray background - no progress bar or any indication that file is loading -

Is there any option that can be set for displaying first page after it has been downloaded, rather than waiting for whole file to download.
edit: Even after waiting for a long while, the pdf is not displaying.

Yes, not displaying, because there is a typo in the url: "/" missing after ".." :)
This works for me http://sanskritdocuments.org/ViewerJS/#../doc_z_otherlang_hindi/manas5_i.pdf
But still a loading indicator is missing, right. Tracked as #96

As a temporary solution for the loading, nit fancy, but to just let the user that the file is loading I did this:
on index.html file
Find:
Replace with: Loading...

on PDFViewerPlugin.js file
Find: completeLoading();
Replace with: completeLoading(); document.getElementById("canvasLoading").style.display = 'none';
Hope this helps!
Thanks

I have did a workaround as following, as I couldn't find the method completeLoading();
On index.html

Find:
Replace with: Loading...

On PDFViewerPlugin.js

Find: for (i = 0; i < pdfDocument.numPages; i += 1) {
pdfDocument.getPage(i + 1).then(createPage);
}
             initialized = true;
         });

Replace:  for (i = 0; i < pdfDocument.numPages; i += 1) {
pdfDocument.getPage(i + 1).then(createPage);
}
             document.getElementById("canvasLoading").style.display = 'none';
             initialized = true;
         });


MacOS plugin broken after upgrade to Flutter 2.8

Running pod install...
Building macOS application...
--- xcodebuild: WARNING: Using the first of multiple matching destinations:
{ platform:macOS, arch:x86_64, id:D3B1C5E9-B0C8-5575-95B2-94FFFBE90C9D }
{ platform:macOS, name:Any Mac }
Undefined symbols for architecture x86_64:
  "___gxx_personality_v0", referenced from:
      getAppWindow() in bitsdojo_window_macos(bitsdojo_window.o)
      ____Z10moveWindowP8NSWindow_block_invoke in bitsdojo_window_macos(bitsdojo_window.o)
      setSize(NSWindow*, int, int) in bitsdojo_window_macos(bitsdojo_window.o)
      getScreenInfoForWindow(NSWindow*, _BDWScreenInfo*) in bitsdojo_window_macos(bitsdojo_window.o)
      setPositionForWindow(NSWindow*, _BDWOffset*) in bitsdojo_window_macos(bitsdojo_window.o)
      ____Z20setPositionForWindowP8NSWindowP10_BDWOffset_block_invoke in bitsdojo_window_macos(bitsdojo_window.o)
      setRectForWindow(NSWindow*, _BDWRect*) in bitsdojo_window_macos(bitsdojo_window.o)
      ...
ld: symbol(s) not found for architecture x86_64
clang: error: linker command failed with exit code 1 (use -v to see invocation)
note: Using new build system
note: Planning
note: Build preparation complete
note: Building targets in parallel
** BUILD FAILED **

Exception: Build process failed

Adding libc++ to Link Libraries in Build Phases fixes the error.

Yes it does.  Thank you.

great！

Adding libc++ to Link Libraries in Build Phases fixes the error.

Thanks, works for me.
For other macOS newbies:

Thank you.

Adding libc++ to Link Libraries in Build Phases fixes the error.

It worked fine. Thank you

Wonderful! Thanks!

The solution above did not work for me:
Apple M1 Pro
OS version: MacOS Monterey 12.1

👍

Thanks ! ❤ ❤ ❤ ❤ ❤ ❤ ❤ ❤ ❤ ❤ ❤ ❤ ❤ ❤ ❤

libc++

how to do this in vscode? Thanks

@ingangi you'll have to do it in xcode

Thank you! It works! My first time to use xcode ^^

I did not need that on the M1 and a fresh Flutter 3.0.5 project. Just for reference.


[dv/lc_ctrl] initial dv testbench for LC

This PR:

Add a dv testbench for LC using uvmdvgen.
Add pins to trigger LC init.
Add LC testplan in build_doc.py

Signed-off-by: Cindy Chen<EMAIL_ADDRESS>

The latest push fixed CI failure.


ModuleNotFoundError: No module named 'multi_person_tracker'

Hi! Thanks for this great work!
I cannot find MPT in your code except being imported in demo.py. Could you please help me figure out the solution?
Regards
Rahul Bhalley

use this one: https://github.com/mkocabas/multi-person-tracker


What's the difference?

Hello. I was just wondering what the difference is between your dumper and Dumper-7. I see Dumper 7 doesn't require you to find offsets while this one does.

Hey there, you would see the difference if you actually used both projects.
Dumper-7 is a internal dumping tool, you need to inject it into the target process. It also just dumps the SDK, it does not have

a editor
live editor
overriding certain members in structs/classes
overriding certain structs/classes
dumps host (dumpspace) support
project saving
and more.
Dumper-7 is only a dumper, but a pretty good dumper that's actively in development. You can't compare these two projects as the focus of my project is not only generating a SDK like Dumper-7.
I doesn't need offsets because it tries to find these offsets. I didn't implement that because it's harder to do so externally and i don't see a issue with browsing the game dump for 2 minutes for offsets.


XML Marshalling is locale dependent

I tried running the test on my machine with a locale that has ',' as the decimal separator instead of '.' and the S100ExchangeSetUtilsTest.testMarchallS125 test fails because we get
Expected:
<posList>-10.0000000 -10.0000000</posList>
<posList>-11.0000000 10.0000000</posList>

Actual:
<posList>-10,0000000 -10,0000000</posList>
<posList>-11,0000000 10,0000000</posList>

Can be reproduced by anyone by temporarily changing this code.
@Test
void testMarchallS125() throws JAXBException {
    Locale.setDefault(Locale.FRANCE);  //Warning sets it globally, but good enough to reproduce

I think we only want to generate xml that always uses . for decimal separator?

Good question... I don't think the positions number format in S-100 are based on the Locale.
I can raise a question though... for now let's assume it will always be a ".".
In the meatime, I will see if we can set the Marshaller Locale manually to not be system dependent.

That is correct, I was aware of the 7 decimal places. We can check S-100 for the decimal point a well.


PaDEL 3D descriptor

[ ] 3D autocorrelation

[x] Charged partial surface area

[x] Gravitational index

[x] Length over breadth

pending
Rotating calipers

[x] Moment of inertia

[x] Petitjean shape index

[ ] RDF

[ ] WHIM

splitted


Capacitor 3 support

I made this plugin compatible with the new Capacitor version("@capacitor/core": "3.0.0-beta.3")

When will the support for Capacitor 3 be published?


SUBMARINE-868. Add save model to pysubmarine ModelsClient

What is this PR for?
Add save model function to submarine-sdk model management.
What type of PR is it?
Feature
Todos
What is the Jira issue?
https://issues.apache.org/jira/browse/SUBMARINE-868
How should this be tested?
Screenshots (if appropriate)
Questions:

Do the license files need updating? No
Are there breaking changes for older versions? No
Does this need new documentation? Yes

@ByronHsu Help me review the code. Thanks.

LGTM


HDFS-17105. mistakenly purge editLogs even after it is empty in NNStorageRetentionManager

Description of PR
https://issues.apache.org/jira/browse/HDFS-17105
This PR adds a non-negative check for the parameter dfs.namenode.num.extra.edits.retained.
How was this patch tested
(1) Set dfs.namenode.max.extra.edits.segments.retained to -1
(2) Run test: org.apache.hadoop.hdfs.server.namenode.TestNNStorageRetentionManager#testNoLogs
An IllegalArgumentException is thrown rather than the IndexOutOfBoundsException.
For code changes:

[x] Does the title or this PR starts with the corresponding JIRA issue id (e.g. 'HADOOP-17799. Your PR title ...')?
[ ] Object storage: have the integration tests been executed and the endpoint declared according to the connector-specific documentation?
[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[ ] If applicable, have you updated the LICENSE, LICENSE-binary, NOTICE-binary files?

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 49s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  0s

codespell was not available.

+0 :ok:
detsecrets
0m  0s

detect-secrets was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

-1 :x:
test4tests
0m  0s

The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ trunk Compile Tests _

+1 :green_heart:
mvninstall
47m 31s

trunk passed

+1 :green_heart:
compile
1m 22s

trunk passed with JDK Ubuntu-11.0.20+8-post-Ubuntu-1ubuntu120.04

+1 :green_heart:
compile
1m 14s

trunk passed with JDK Private Build-1.8.0_382-8u382-ga-1~20.04.1-b05

+1 :green_heart:
checkstyle
1m 10s

trunk passed

+1 :green_heart:
mvnsite
1m 23s

trunk passed

+1 :green_heart:
javadoc
1m  9s

trunk passed with JDK Ubuntu-11.0.20+8-post-Ubuntu-1ubuntu120.04

+1 :green_heart:
javadoc
1m 36s

trunk passed with JDK Private Build-1.8.0_382-8u382-ga-1~20.04.1-b05

+1 :green_heart:
spotbugs
3m 19s

trunk passed

+1 :green_heart:
shadedclient
40m 23s

branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
1m 13s

the patch passed

+1 :green_heart:
compile
1m 14s

the patch passed with JDK Ubuntu-11.0.20+8-post-Ubuntu-1ubuntu120.04

+1 :green_heart:
javac
1m 14s

the patch passed

+1 :green_heart:
compile
1m  8s

the patch passed with JDK Private Build-1.8.0_382-8u382-ga-1~20.04.1-b05

+1 :green_heart:
javac
1m  8s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

+1 :green_heart:
checkstyle
1m  0s

the patch passed

+1 :green_heart:
mvnsite
1m 13s

the patch passed

+1 :green_heart:
javadoc
0m 55s

the patch passed with JDK Ubuntu-11.0.20+8-post-Ubuntu-1ubuntu120.04

+1 :green_heart:
javadoc
1m 29s

the patch passed with JDK Private Build-1.8.0_382-8u382-ga-1~20.04.1-b05

+1 :green_heart:
spotbugs
3m 19s

the patch passed

+1 :green_heart:
shadedclient
40m 59s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

-1 :x:
unit
237m 54s
/patch-unit-hadoop-hdfs-project_hadoop-hdfs.txt
hadoop-hdfs in the patch passed.

+1 :green_heart:
asflicense
0m 46s

The patch does not generate ASF License warnings.

391m 46s

Reason
Tests

Failed junit tests
hadoop.hdfs.TestRollingUpgrade

hadoop.hdfs.server.datanode.TestDirectoryScanner

Subsystem
Report/Notes

Docker
ClientAPI=1.43 ServerAPI=1.43 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-6036/1/artifact/out/Dockerfile

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

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient spotbugs checkstyle codespell detsecrets

uname
Linux f80630ff9395 4.15.0-212-generic #223-Ubuntu SMP Tue May 23 13:09:22 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / 7f40e2614a67c4374a8aa0c76cce29da19dcb50e

Default Java
Private Build-1.8.0_382-8u382-ga-1~20.04.1-b05

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.20+8-post-Ubuntu-1ubuntu120.04 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_382-8u382-ga-1~20.04.1-b05

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-6036/1/testReport/

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

modules
C: hadoop-hdfs-project/hadoop-hdfs U: hadoop-hdfs-project/hadoop-hdfs

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-6036/1/console

versions
git=2.25.1 maven=3.6.3 spotbugs=4.2.2

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

This message was automatically generated.

LGTM. +1 from my side.

Committed to trunk. Thanks @teamconfx for your contributions.


Not available for VS 2022

VS 2022 officially launched today. Will this be getting an update? I'd really appreciate it!

@sharwell @jnm2 any chance of an updated version for 2022?

I'd like that. I think it would be a similar level of learning experience for any of the three of us here, possibly Sam as well (haven't asked him).

What is sad is there are no code changes required.   Nor where there for VS 2019 by the looks of it.  Manually download it from: https://marketplace.visualstudio.com/items?itemName=SamHarwell.JustMyCodeToggle you should get a vsix file.  Rename this to .zip.  Extract the "extension.vsixmanifest" file.  Edit it change the two references to 17.0 to 22.0.   Add the file back to the zip.  Do the same for catalog.json (only one reference to 17 in it). Rename your zip back to vsix.  Now you are good for the next 5 versions of VS.  If you want to do it properly just change the 17 to 18 but hey where is the fun in that.

With source link becoming more popular (or just me using it more) it would be REALLY great to see this updated for 2022.

note the PR does work for VS2022 been using it happily for a year now.


[BEAM-1542] A transform for reading the Spanner schema

A part of https://github.com/apache/beam/pull/3729.
Ideally, spanner schema should be an introspection  part of the Cloud Spanner library.

R: @jkff

Added tests

Do you think it makes sense to expose this transform as part of SpannerIO, e.g. SpannerIO.readSchema()? It seems quite useful to me.

Yes, but it would be better spanner schema to be part of Spanner Client library first. To do that we need more introspection features.

Please remove the MutationGroupEncoder and MutationSizeEstimator commits from this PR.

Oops, removed those commits

ptal


Can the resume function only target a node,not target a workflow

Summary
Current implementation of resume targets workflow. When I have two nodes both suspend template, I just  want only one node resume to go on, other node is still suspend. Like this:
Use Cases

I have two paths both suspend, when I click resume, both of them are resume. I just want D or E to resume instead of workflow.

This would be a nice enhancement to the UI.

@anankun would you like to submit a PR to fix this?

@anankun would you like to submit a PR to fix this?

Sorry，I'm beginner in go , not good at go and ts. Maybe after a while


I Can not select a compendium for importing items orr Journals...

Hello
I can not specify a compendium to import items or journals... the box is empty.
I tried to create a Folder Realm Works, and empty compendium too but it still does'nt work ...
I use Savage World as World Rules.
Any idea?

I use foundry V12.331

This is caused by a regression in the support provided by version 12 of core Foundry for being able to populate the drop-down menu dynamically.
I am exploring alternative solutions that might work.

The dropdown menus should be available in version 2.12.1


Fix logging

Fix the logger that previously changes the global setting.
Each class now gets its own logger. Should now behave like this:
In [10]: p = Poscar(file_path="tmp/POSCAR")

In [11]: p._logger.debug("Foo")
2022-02-09 10:26:02,840 - parsevasp.poscar.Poscar - DEBUG - Foo

@zhubonan Thanks a lot for fixing this.


menu in bounds issue

basically if you click close to the outside of the checkbox it will enable it, shouldn't it be only when clicked inside of it?

correction the check boxes will turn on if you click on the text that's beside them wouldn't that be a bug and if it's meant to be like that how would you fix it?

I don't know what way was intended but the "fix" for this is very simple just change 	if (this->mouseCursor->IsInBounds(this->vecPosition, (this->vecPosition + this->vecSize)))  to 	if (this->mouseCursor->IsInBounds(this->vecPosition, (this->vecPosition + this->vecSelectableSize)))

It was intended, checkbox selectable is so small that its much easier to use it that way.
I'll leave it for discussion maybe?

Played one game with only the checkboxes and it seems alright (after i got used it it).

It's Wando's decision, but the expected behaviour from a user would be that clicking the text activates the checkbox too. This is how it works on the web, and it can feel fiddly without it.
my 2c

For ease of use the old way works better yeah. I don't mind either way
though its a simple change maybe just leave a comment for new users and
fhey can decide?
On 25 Feb. 2018 2:16 pm, "Luke"<EMAIL_ADDRESS>wrote:

It's Wando's decision, but the expected behaviour from a user would be
that clicking the text activates the checkbox too. This is how it works on
the web, and it can feel fiddly without it.
my 2c
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/Wando1423/Antario/issues/33#issuecomment-368280742,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ALl6DlIRtIJqnheWQc_5dztvSFbNsoeMks5tYNeJgaJpZM4SQPGr
.


Reproduce reported results

Hi @hugochan,
Could you please public the actual 20 news data that you used to train model and get the results reported in paper?
I downloaded data from the link http://qwone.com/~jason/20Newsgroups/ (filename is 20news-bydate.tar.gz) and used your code but cannot reproduce your reported results. (I also used your processed data but didn't work as well)
Thanks,

Hi @hugochan,
Could you please public the actual 20 news data that you used to train model and get the results reported in paper?
I downloaded data from the link http://qwone.com/~jason/20Newsgroups/ (filename is 20news-bydate.tar.gz) and used your code but cannot reproduce your reported results. (I also used your processed data but didn't work as well)
Thanks,

Hi @trannhuthuat96 , that is exactly the data we used in our experiments. You should be able to reproduce the results following the instructions in README. Can you report your results here? Also I am not sure if the issue is related to different TF versions, so please check #24 as well.

Sorry @hugochan for late response.
I checked issued #23 and #24 as well and change TF version to 1.2.1 but still didn't reproduce your results.
For document classification on 20 news data, I got 62% accuracy with 128 topics with KATE. Moreover, using your processed data mentioned in #23, the results is 73% instead of (74.4%) as reported.
Do you have any ideas/comments to reproduce resullt?

Sorry @hugochan for late response.
I checked issued #23 and #24 as well and changed TF version to 1.2.1 but still didn't reproduce your results.
For document classification on 20 news data, I got 62% accuracy with 128 topics with KATE. Moreover, using your processed data mentioned in #23, the results is 73% instead of (74.4%) as reported.
Do you have any ideas/comments to reproduce result?

Sorry @hugochan for late response.
I checked issued #23 and #24 as well and changed TF version to 1.2.1 but still didn't reproduce your results.
For document classification on 20 news data, I got 62% accuracy with 128 topics with KATE. Moreover, using your processed data mentioned in #23, the results is 73% instead of (74.4%) as reported.
Do you have any ideas/comments to reproduce result?

@trannhuthuat96 I am not sure what will be other reasons besides package (e.g., TF, Keras) versions. BTW, did you use cpu or gpu version of TF? I used cpu version when doing the experiments. Not sure if this is the cause.
You might also try running the experiments multiple times with different random seeds, and report the avg/std scores.

Thanks @hugochan for your suggestions.
I used both GPU and CPU for training but cannot reproduce your reported results.
May be it is needed further exploration to reproduce the results.


feat(yumafuu/s1m): scaffold yumafuu/s1m

#27524 yumafuu/s1m: TUI Application for AWS Systems Manager Parameter Store (ssm)
Check List

[x] Read CONTRIBUTING.md

[x] Read OSS Contribution Guide

[x] All commits are signed

This repository enables Require signed commits, so all commits must be signed

[x] Avoid force pushcreate-pr-new-pkg
[x] Do only one thing in one Pull Request

Description
Add https://github.com/yumafuu/s1m to standard registry
$ which s1m
/home/yuma/.local/share/aquaproj-aqua/bin/s1m

$ s1m -v
v1.0.1

Thank you for your contribution!
If cmdx new failed, the following comment may be helpful. https://github.com/aquaproj/aqua-registry/issues/26113#issuecomment-2302902723

Thanks!
I'll do this next time :)

v4.230.0 is out 🎉
https://github.com/aquaproj/aqua-registry/releases/tag/v4.230.0


requestKey for notifications is fileprivate

Alamofire version:
5.0.0-beta.6
I'm trying to catch a notifications Request.didResumeTaskNotification, Request.didSuspendTaskNotification and so on, which send the Request with them. But the key for this payload is
fileprivate static let requestKey = "org.alamofire.notification.key.request"
I can't understand why it's private. And why it's a string extension (not Notification.Key.Task like before.

You don't need access to the key any more, there is now an extension on Notification which will give you the associated Request: notification.request.
Feel free to let us know if you have any other questions.


Make cmdline work like stock neovim

I like your plugin.  Thanks for sharing it.
The standard configs you post make the wildmenu popup automatically in the command line.  I would prefer that it would only show the popup menu after I press the tab key.
Is there anyway to specify this behavior in the cmp.setup.cmdline(":", {  }) lua code?

:h cmp-config.completion.autocomplete


Infinite loop with lured_pokemon_found and catch_above_iv

If I have any capture filter set (i.e. catch_above_iv), and it will keep trying to catch the lured pokemon that doesn't match the filter without moving away from said forts. Might need some way to signify that the pokemon have already been blacklisted, stop trying to catch them, and to move on to the next fort.
I was able to work around it by placing the MoveToFort and FollowSpiral tasks before the CatchLuredPokemon task.
2016-08-15 11:40:41,720 [CatchLuredPokemon] [INFO] [lured_pokemon_found] Lured pokemon at fort Joe's Crab Shack (6aefdc1391664e7f825315c5ddbc08f3.16)
2016-08-15 11:40:43,454 [CatchLuredPokemon] [INFO] [lured_pokemon_found] Lured pokemon at fort Castagnola's (680dc1c30b8746e7820ed1f655cbadb7.16)
2016-08-15 11:40:43,454 [CatchLuredPokemon] [INFO] [lured_pokemon_found] Lured pokemon at fort Joe's Crab Shack (6aefdc1391664e7f825315c5ddbc08f3.16)
2016-08-15 11:40:45,238 [CatchLuredPokemon] [INFO] [lured_pokemon_found] Lured pokemon at fort Castagnola's (680dc1c30b8746e7820ed1f655cbadb7.16)
2016-08-15 11:40:45,238 [CatchLuredPokemon] [INFO] [lured_pokemon_found] Lured pokemon at fort Joe's Crab Shack (6aefdc1391664e7f825315c5ddbc08f3.16)
2016-08-15 11:40:49,135 [CatchLuredPokemon] [INFO] [lured_pokemon_found] Lured pokemon at fort Castagnola's (680dc1c30b8746e7820ed1f655cbadb7.16)
2016-08-15 11:40:49,135 [CatchLuredPokemon] [INFO] [lured_pokemon_found] Lured pokemon at fort Joe's Crab Shack (6aefdc1391664e7f825315c5ddbc08f3.16)
2016-08-15 11:40:50,671 [CatchLuredPokemon] [INFO] [lured_pokemon_found] Lured pokemon at fort Castagnola's (680dc1c30b8746e7820ed1f655cbadb7.16)
2016-08-15 11:40:50,671 [CatchLuredPokemon] [INFO] [lured_pokemon_found] Lured pokemon at fort Joe's Crab Shack (6aefdc1391664e7f825315c5ddbc08f3.16)
2016-08-15 11:40:52,133 [CatchLuredPokemon] [INFO] [lured_pokemon_found] Lured pokemon at fort Castagnola's (680dc1c30b8746e7820ed1f655cbadb7.16)
2016-08-15 11:40:52,133 [CatchLuredPokemon] [INFO] [lured_pokemon_found] Lured pokemon at fort Joe's Crab Shack (6aefdc1391664e7f825315c5ddbc08f3.16)
2016-08-15 11:40:54,252 [CatchLuredPokemon] [INFO] [lured_pokemon_found] Lured pokemon at fort Castagnola's (680dc1c30b8746e7820ed1f655cbadb7.16)
2016-08-15 11:40:54,252 [CatchLuredPokemon] [INFO] [lured_pokemon_found] Lured pokemon at fort Joe's Crab Shack (6aefdc1391664e7f825315c5ddbc08f3.16)
2016-08-15 11:40:55,938 [CatchLuredPokemon] [INFO] [lured_pokemon_found] Lured pokemon at fort Castagnola's (680dc1c30b8746e7820ed1f655cbadb7.16)
2016-08-15 11:40:55,938 [CatchLuredPokemon] [INFO] [lured_pokemon_found] Lured pokemon at fort Joe's Crab Shack (6aefdc1391664e7f825315c5ddbc08f3.16)
2016-08-15 11:40:57,409 [CatchLuredPokemon] [INFO] [lured_pokemon_found] Lured pokemon at fort Castagnola's (680dc1c30b8746e7820ed1f655cbadb7.16)
2016-08-15 11:40:57,409 [CatchLuredPokemon] [INFO] [lured_pokemon_found] Lured pokemon at fort Joe's Crab Shack (6aefdc1391664e7f825315c5ddbc08f3.16)
2016-08-15 11:41:00,796 [CatchLuredPokemon] [INFO] [lured_pokemon_found] Lured pokemon at fort Castagnola's (680dc1c30b8746e7820ed1f655cbadb7.16)
2016-08-15 11:41:00,796 [CatchLuredPokemon] [INFO] [lured_pokemon_found] Lured pokemon at fort Joe's Crab Shack (6aefdc1391664e7f825315c5ddbc08f3.16)
2016-08-15 11:41:02,411 [CatchLuredPokemon] [INFO] [lured_pokemon_found] Lured pokemon at fort Castagnola's (680dc1c30b8746e7820ed1f655cbadb7.16)
2016-08-15 11:41:02,411 [CatchLuredPokemon] [INFO] [lured_pokemon_found] Lured pokemon at fort Joe's Crab Shack (6aefdc1391664e7f825315c5ddbc08f3.16)
2016-08-15 11:41:04,131 [CatchLuredPokemon] [INFO] [lured_pokemon_found] Lured pokemon at fort Castagnola's (680dc1c30b8746e7820ed1f655cbadb7.16)
2016-08-15 11:41:04,132 [CatchLuredPokemon] [INFO] [lured_pokemon_found] Lured pokemon at fort Joe's Crab Shack (6aefdc1391664e7f825315c5ddbc08f3.16)
2016-08-15 11:41:06,049 [CatchLuredPokemon] [INFO] [lured_pokemon_found] Lured pokemon at fort Castagnola's (680dc1c30b8746e7820ed1f655cbadb7.16)
2016-08-15 11:41:06,050 [CatchLuredPokemon] [INFO] [lured_pokemon_found] Lured pokemon at fort Joe's Crab Shack (6aefdc1391664e7f825315c5ddbc08f3.16)
2016-08-15 11:41:08,046 [CatchLuredPokemon] [INFO] [lured_pokemon_found] Lured pokemon at fort Castagnola's (680dc1c30b8746e7820ed1f655cbadb7.16)
2016-08-15 11:41:08,046 [CatchLuredPokemon] [INFO] [lured_pokemon_found] Lured pokemon at fort Joe's Crab Shack (6aefdc1391664e7f825315c5ddbc08f3.16)
2016-08-15 11:41:09,678 [CatchLuredPokemon] [INFO] [lured_pokemon_found] Lured pokemon at fort Castagnola's (680dc1c30b8746e7820ed1f655cbadb7.16)
2016-08-15 11:41:09,678 [CatchLuredPokemon] [INFO] [lured_pokemon_found] Lured pokemon at fort Joe's Crab Shack (6aefdc1391664e7f825315c5ddbc08f3.16)
2016-08-15 11:41:11,435 [CatchLuredPokemon] [INFO] [lured_pokemon_found] Lured pokemon at fort Castagnola's (680dc1c30b8746e7820ed1f655cbadb7.16)
2016-08-15 11:41:11,435 [CatchLuredPokemon] [INFO] [lured_pokemon_found] Lured pokemon at fort Joe's Crab Shack (6aefdc1391664e7f825315c5ddbc08f3.16)
2016-08-15 11:41:13,873 [CatchLuredPokemon] [INFO] [lured_pokemon_found] Lured pokemon at fort Castagnola's (680dc1c30b8746e7820ed1f655cbadb7.16)
2016-08-15 11:41:13,873 [CatchLuredPokemon] [INFO] [lured_pokemon_found] Lured pokemon at fort Joe's Crab Shack (6aefdc1391664e7f825315c5ddbc08f3.16)
2016-08-15 11:41:15,547 [CatchLuredPokemon] [INFO] [lured_pokemon_found] Lured pokemon at fort Castagnola's (680dc1c30b8746e7820ed1f655cbadb7.16)
2016-08-15 11:41:15,547 [CatchLuredPokemon] [INFO] [lured_pokemon_found] Lured pokemon at fort Joe's Crab Shack (6aefdc1391664e7f825315c5ddbc08f3.16)
2016-08-15 11:41:16,991 [CatchLuredPokemon] [INFO] [lured_pokemon_found] Lured pokemon at fort Castagnola's (680dc1c30b8746e7820ed1f655cbadb7.16)
2016-08-15 11:41:16,991 [CatchLuredPokemon] [INFO] [lured_pokemon_found] Lured pokemon at fort Joe's Crab Shack (6aefdc1391664e7f825315c5ddbc08f3.16)
2016-08-15 11:41:18,651 [CatchLuredPokemon] [INFO] [lured_pokemon_found] Lured pokemon at fort Castagnola's (680dc1c30b8746e7820ed1f655cbadb7.16)
2016-08-15 11:41:18,651 [CatchLuredPokemon] [INFO] [lured_pokemon_found] Lured pokemon at fort Joe's Crab Shack (6aefdc1391664e7f825315c5ddbc08f3.16)
2016-08-15 11:41:20,374 [CatchLuredPokemon] [INFO] [lured_pokemon_found] Lured pokemon at fort Castagnola's (680dc1c30b8746e7820ed1f655cbadb7.16)
2016-08-15 11:41:20,374 [CatchLuredPokemon] [INFO] [lured_pokemon_found] Lured pokemon at fort Joe's Crab Shack (6aefdc1391664e7f825315c5ddbc08f3.16)
2016-08-15 11:41:23,195 [CatchLuredPokemon] [INFO] [lured_pokemon_found] Lured pokemon at fort Castagnola's (680dc1c30b8746e7820ed1f655cbadb7.16)
2016-08-15 11:41:23,195 [CatchLuredPokemon] [INFO] [lured_pokemon_found] Lured pokemon at fort Joe's Crab Shack (6aefdc1391664e7f825315c5ddbc08f3.16)
2016-08-15 11:41:24,822 [CatchLuredPokemon] [INFO] [lured_pokemon_found] Lured pokemon at fort Castagnola's (680dc1c30b8746e7820ed1f655cbadb7.16)
2016-08-15 11:41:24,822 [CatchLuredPokemon] [INFO] [lured_pokemon_found] Lured pokemon at fort Joe's Crab Shack (6aefdc1391664e7f825315c5ddbc08f3.16)
2016-08-15 11:41:26,715 [CatchLuredPokemon] [INFO] [lured_pokemon_found] Lured pokemon at fort Castagnola's (680dc1c30b8746e7820ed1f655cbadb7.16)
2016-08-15 11:41:26,715 [CatchLuredPokemon] [INFO] [lured_pokemon_found] Lured pokemon at fort Joe's Crab Shack (6aefdc1391664e7f825315c5ddbc08f3.16)

This might be actually in interaction with "lure_attraction": true and the filter. I disabled lure_attraction and everything seems to be working now.

Setting lure_attration to false only reduce the bug occurence.
The bug is the interaction beetween pokemon_catch_worker and catch_*_pokemon. Because catcher will try to catch the nearest no wanted pokemon meanwhile all tick.
Quick (and ugly) fix in catch_* : call catch_pokemon in the loop.

i've also seen this happen using followpath or movetomappokemon as well.
it always looked like a multiple lured in range bug to me but interesting that it will work with no filter.

In move_to_map_pokemon.py, there is a caught table so you don't teleport to the same pokemon more than once. Maybe set something similar up in pokemon_catch_worker.py

so what is the fix for this problem?

Who knows with the limited information provided we are unable to assist, there is a template for a reason :(

ran into this problem..it was alternating between two lured_pokemon_found

ran into this problem, too... I use followpath


README.md: replace TiKV roadmap url

The previous file is inaccessible. so I changed the url.

/cc @skyzh

@skyzh agree. At least we should remove misleading information.

@hongrubb would you please help remove that section in this PR? By the way, it seems that you are using different email for git command line and GitHub, which makes your avatar missing from that commit.

@hongrubb would you please help remove that section in this PR? By the way, it seems that you are using different email for git command line and GitHub, which makes your avatar missing from that commit.

sure, I will try to fix this.

@hongrubb would you please help remove that section in this PR? By the way, it seems that you are using different email for git command line and GitHub, which makes your avatar missing from that commit.

Very thanks for pointing out the problem with the avatar.
But unfortunately, I don't know why, I couldn't update the commit to this PR...
I can get the commit log in my own repo, but they are not connected.
As you can see, on this page, the link in the title hongrubb:master is grey.

I have no idea either. You may open a new PR, attach a link to this PR, and cc me and tisonkun for review.


Refactor

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 100.00%. Comparing base (842c1f5) to head (376c9e7).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@             Coverage Diff              @@
##             main       #31       +/-   ##
============================================
+ Coverage   25.00%   100.00%   +75.00%
============================================
  Files           1        11       +10
  Lines           8        28       +20
  Branches        1         2        +1
============================================
+ Hits            2        28       +26
+ Misses          6         0        -6

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


Make batteries (applied energistics energy cells, rename TBD) non-voidable

For example by making them explode when voided, etc..
This forces players to find a creative way to store depleted batteries long term, similar to IRL spent nuclear fuel

@Yarden-zamir do you know what he means by 'voided'?
like if thrown into lava?

 yarden.zamir on Discord says

the thought there was that you cannot easily get rid of them so like in real life you'll need to figure out a way to dispose them by piling them in an electronic dump mound or something automatically. since then the battery changed to milk cell and the idea was kind of scrapped so this can be closed


test(math.vector): add tests for GetAngleBetweenVectorsOnPlane function

I generated 1000 variants of usage the GetAngleBetweenVectorsOnPlane function base on these values:
const possibleValues = [
        // negative values
        Number.MIN_SAFE_INTEGER,
        Number.MIN_SAFE_INTEGER / 2,
        -Math.PI,
        -0.0000000001,

        0,

        // positive values
        0.0000000001,
        Math.PI,
        Number.MAX_SAFE_INTEGER / 2,
        Number.MAX_SAFE_INTEGER,
    ];

So I got time comutation statistics that added to jsdoc.
What do you think about unit tests like this? Or probably we should set just several examples without edge values (like max_safe_integer)?

Thanks for that!
I wonder - how were these numbers generated? And why do we need 1000 tests and not a few fixed (and tested) ones, including edge cases?

I wonder - how were these numbers generated? And why do we need 1000 tests and not a few fixed (and tested) ones, including edge cases?

Yes, I want to discourse! =)
As edge cases I take Number.MIN_INTEGER_VALUE and max. Also I take zero and value close to zero 0.00000001 (negative and positive).
Also I choose some normal value between zero and max value — Math.PI (negative and positive).
In fact, I also added half value of Number.MIN and MAX_INTEGER_VALUE as normal value, but currenly I think it was redundant because Math.PI do same work.
So at least we have possible values:
const possibleValues = [
  Number.MIN_SAFE_INTEGER,
  -Math.PI,
  -0.0000000001,
  0,
  0.0000000001,
  Math.PI,
  Number.MAX_SAFE_INTEGER,
]

The function take 3 input Vector3 (3 input by 3 value), so to calculate all possible test cases we should Math.pow(possibleValues.length, 3 * 3); it equal to 40 353 607. Of course it is a lot. So I generate by random take value and got 1000 test cases.
Is all of them required? It is hard question. But I got some interesting insight.
After create these tests I start to modify the function and what I found - if we remove v0.normalize() and v1.normalize() then just a several tests of 1000 will failed, other of them works fine.
It's mean for, these test keep guarantee to the function works well in most common tasks.

How were they generated? using the same function?

No, just simple for loop. I dont save it but it was looking like:
for (let i = 0; i < 1000; i++) {
  testCases.push({
    v0: { x: getRandomPossibleValue(), /* y: ... */ },
    // same for
    // v1:
    // normal
  });
}

after run it I fill the result and save it as fixed test cases as array in this PR.

How were they generated? using the same function?

No, just simple for loop. I dont save it but it was looking like:
for (let i = 0; i < 1000; i++) {
  testCases.push({
    v0: { x: getRandomPossibleValue(), /* y: ... */ },
    // same for
    // v1:
    // normal
  });
}

after run it I fill the result and save it as fixed test cases as array in this PR.

I mean the results :-) You are testing the function against the results it has generated, right? So this test is only testing if the function has changed? Or have you generated the results using some other method?

I mean the results :-) You are testing the function against the results it has generated, right? So this test is only testing if the function has changed? Or have you generated the results using some other method?

Right, I got current results by current state of the function with trust that it work correct.

So this test is only testing if the function has changed?

Yep.
If function work incorrect then tests wrong too.
What do you think, which is the best way to write unit tests in such case?

I mean the results :-) You are testing the function against the results it has generated, right? So this test is only testing if the function has changed? Or have you generated the results using some other method?

Right, I got current results by current state of the function with trust that it work correct.

So this test is only testing if the function has changed?

Yep. If function work incorrect then tests wrong too.
What do you think, which is the best way to write unit tests in such case?

Nope, I just wanted to be sure what is being tested :-)

I just thought that this way to testing is good for long living high-order functions and I want to discourse it with you, what is the best way? Leave public functions without any tests is not good definitely, but how to test them right?

And what do you think? Will some similar PRs be helpful if I make some of them? For me, it is an interesting way to get to know framework.

Of course. I would rather not have 1000 test cases (10-20 would be great :-)), but we are always happy for these kinds of PRs!

I would rather not have 1000 test cases (10-20 would be great :-))

I understand you, but this situation looks like a potential problem when we talking about limited test cases:

After create these tests I start to modify the function and what I found - if we remove v0.normalize() and v1.normalize() then just a several tests of 1000 will failed, other of them works fine.


Fix up ADAS CI

This PR attempts to fix the ADAS CI. The changes are:

Update to GMAOpyobs v<IP_ADDRESS> which allows for not building f2py code in CI (see https://github.com/GEOS-ESM/GMAOpyobs/compare/v1.0.6...v<IP_ADDRESS>)
Backs off the circleci-tools orb to v2.10 which is before Intel 2021.13 came in

It worked! Yay!

@mathomp4 Why does this take so long to run the CI tests?

Huh. I did not notice that. Something must be wonky. Let me take a look...

I just tried a push of a quick fix. I don't think it's perfect, but should make the build faster I hope.

I just tried a push of a quick fix. I don't think it's perfect, but should make the build faster I hope.

I just tried a push of a quick fix. I don't think it's perfect, but should make the build faster I hope.

ok - thank you.

I just tried a push of a quick fix. I don't think it's perfect, but should make the build faster I hope.

ok - thank you.

The "real" issue seems to be the compiler in CI land is dying trying to compile a file...but then it works the next time around. Obviously the file compiles on discover, etc., so my only thought is maybe I just need less make parallelism for that one file (i.e., compiler running out of memory?). Fingers crossed!


create java docker image

Put dockerfile in java folder

Example dockerfile in /Vantage6Wrapper

We cannot create a separate java image, this is one of the deliberate restrictions on v6


⚠ File access has degraded performance

In ccabd76, File access (https://fyle.uk/file/fish) experienced degraded performance:

HTTP code: 200
Response time: 6317 ms

Resolved: File access performance has improved in 03d0b6c after 176 days, 23 hours, 41 minutes.


Skills Generator

A plug-in that gives you tasks to do. An excellent plug-in when you have nothing to do.
Overlay

Panel

Your branch here isn't rebased and has conflicts

I don't have the option to fetch my BoredGenerator branch. Can this have something to do while its in a pull request? Or is there another easy way to rebase by branch?
Sorry if these are noobie questions. Not that familiar with github.

https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/addressing-merge-conflicts/about-merge-conflicts

Thanks I read that but my conflicted file is the new commit hash and the new repository url. So I don't understand what's wrong.

First add the runelite repo as a remote to your local repo. If it says it already exists during add that's fine, fetch to update your references:
git remote add upstream https://github.com/runelite/plugin-hub.git
git fetch upstream

Now either do the correct, or quick-and-dirty fix:
"Correct" fix by going through the merge process:
git rebase -i upstream
# walk through the prompts until it tells you there are conflicts
# resolve the plugins/boredgenerator file to the intended version
git rebase --continue
git push --force # need --force since this is overwriting remote history

Quick-and-dirty fix via discarding your changes, rebasing, and redoing your changes:
git reset --hard upstream/master
# edit plugins/boredgenerator to intended values
git add plugins/boredgenerator
git commit -m "your commit msg here"
git push --force # need --force since this is overwriting remote history

Sorry i think i am here at the wrong place because i just want to update my plug-in.


Issue 520 spmecc

Description

Adds model to compute effective resistance of current collector
Updates how BCs are handled in finite elements
Add new solver for algebraic only systems (basically the root finding algo used to get consistent ICs in DAE solver
Adds comparison of 2plus1D and 1D models
Adds 2+1D DFN model

Fixes #520
Type of change
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Key checklist:

[x] No style issues: $ flake8
[x] All tests pass: $ python run-tests.py --unit
[x] The documentation builds: $ cd docs and then $ make clean; make html

You can run all three at once, using $ python run-tests.py --quick.
Further checks:

[x] Code is commented, particularly in hard-to-understand areas
[x] Tests added that prove fix is effective or that feature works
[x] Any dependent changes have been merged and published in downstream modules

changing html_static_path = ["_static"] to html_static_path = [] in conf.py seems to fix this (see here) @martinjrobins I don't really understand what this does or if changing it will cause another problem? also, I'm not sure why it broke in the first place?


Handle empty parameter groups

Filter out empty parameter groups in both engine and zero3.
Fix #2820

This PR is the proper fix and is linked to #4270. But both fixes will only work with HF transformers==4.31.0. New HF versions are still broken with zero-stage 3. Actively being debugged by @tjruwase and @awan-10 .


 Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

⚠ Artifact update problem
Renovate failed to update an artifact related to this branch. You probably do not want to merge this PR as-is.
♻ Renovate will retry this branch, including artifacts, only when one of the following happens:

any of the package files in this branch needs updating, or
the branch becomes conflicted, or
you click the rebase/retry checkbox if found above, or
you rename this PR's title to start with "rebase!" to trigger it manually

The artifact failure details are included below:
File name: go.sum
Command failed: go get -d -t ./...
warning: ignoring symlink /tmp/renovate/repos/github/freiheit-com/kuberpult/services/frontend-service/api
go: downloading k8s.io/client-go v11.0.0+incompatible
go: downloading k8s.io/klog v1.0.0
go: downloading github.com/googleapis/gnostic v0.5.5
go: downloading github.com/googleapis/gnostic v0.7.0
github.com/freiheit-com/kuberpult/services/cd-service/pkg/argocd/reposerver imports
	github.com/argoproj/gitops-engine/pkg/utils/kube imports
	k8s.io/client-go/discovery imports
	github.com/googleapis/gnostic/OpenAPIv2: cannot find module providing package github.com/googleapis/gnostic/OpenAPIv2
github.com/freiheit-com/kuberpult/services/cd-service/pkg/argocd/reposerver imports
	github.com/argoproj/gitops-engine/pkg/utils/kube imports
	k8s.io/kubectl/pkg/cmd/create imports
	k8s.io/client-go/kubernetes/typed/policy/v1: cannot find module providing package k8s.io/client-go/kubernetes/typed/policy/v1
github.com/freiheit-com/kuberpult/services/cd-service/pkg/argocd/reposerver imports
	github.com/argoproj/argo-cd/v2/util/argo imports
	k8s.io/client-go/kubernetes imports
	k8s.io/client-go/kubernetes/typed/auditregistration/v1alpha1 imports
	k8s.io/api/auditregistration/v1alpha1: cannot find module providing package k8s.io/api/auditregistration/v1alpha1
github.com/freiheit-com/kuberpult/services/cd-service/pkg/argocd/reposerver imports
	github.com/argoproj/argo-cd/v2/util/argo imports
	k8s.io/client-go/kubernetes imports
	k8s.io/client-go/kubernetes/typed/batch/v2alpha1 imports
	k8s.io/api/batch/v2alpha1: cannot find module providing package k8s.io/api/batch/v2alpha1
github.com/freiheit-com/kuberpult/services/cd-service/pkg/argocd/reposerver imports
	github.com/argoproj/argo-cd/v2/util/argo imports
	k8s.io/client-go/kubernetes imports
	k8s.io/client-go/kubernetes/typed/settings/v1alpha1 imports
	k8s.io/api/settings/v1alpha1: cannot find module providing package k8s.io/api/settings/v1alpha1


METRON-2275 Solr Indexing Topology Fails to Start on Secure Cluster with HDP 3.1

The Solr indexing topology will fail to start on a secure, kerberized cluster with HDP 3.1.  The following error will occur when starting the topology.  The topology will not start successfully.
+-         Apache Storm        -+
+-  data FLow User eXperience  -+
Version: 1.0.1
Parsing file: /usr/hcp/<IP_ADDRESS>-94/metron/flux/indexing/random_access/remote.yaml
797  [main] INFO  o.a.s.f.p.FluxParser - loading YAML from input stream...
804  [main] INFO  o.a.s.f.p.FluxParser - Performing property substitution.
808  [main] INFO  o.a.s.f.p.FluxParser - Not performing environment variable substitution.
1118 [main] INFO  o.a.c.f.i.CuratorFrameworkImpl - Starting
Exception in thread "main" java.lang.reflect.InvocationTargetException
	at sun.reflect.NativeConstructorAccessorImpl.newInstance0(Native Method)
	at sun.reflect.NativeConstructorAccessorImpl.newInstance(NativeConstructorAccessorImpl.java:62)
	at sun.reflect.DelegatingConstructorAccessorImpl.newInstance(DelegatingConstructorAccessorImpl.java:45)
	at java.lang.reflect.Constructor.newInstance(Constructor.java:423)
	at org.apache.storm.flux.FluxBuilder.buildObject(FluxBuilder.java:294)
	at org.apache.storm.flux.FluxBuilder.buildComponents(FluxBuilder.java:342)
	at org.apache.storm.flux.FluxBuilder.buildTopology(FluxBuilder.java:75)
	at org.apache.storm.flux.Flux.runCli(Flux.java:153)
	at org.apache.storm.flux.Flux.main(Flux.java:98)
Caused by: java.lang.NoSuchMethodError: com.google.common.util.concurrent.MoreExecutors.sameThreadExecutor()Lcom/google/common/util/concurrent/ListeningExecutorService;
	at org.apache.curator.framework.listen.ListenerContainer.addListener(ListenerContainer.java:40)
	at org.apache.curator.framework.imps.CuratorFrameworkImpl.start(CuratorFrameworkImpl.java:246)
	at org.apache.metron.common.utils.KafkaUtils.getBrokersFromZookeeper(KafkaUtils.java:49)
	at org.apache.metron.storm.kafka.flux.SimpleStormKafkaBuilder.getBootstrapServers(SimpleStormKafkaBuilder.java:246)
	at org.apache.metron.storm.kafka.flux.SimpleStormKafkaBuilder.<init>(SimpleStormKafkaBuilder.java:203)
	at org.apache.metron.storm.kafka.flux.SimpleStormKafkaBuilder.<init>(SimpleStormKafkaBuilder.java:187)
	... 9 more

Root Cause
When kerberized, Storm adds a bunch of jars from $STORM_HOME/contrib/storm-autocreds to the classpath. These jars are not added to the classpath when the cluster is unsecure (no kerberos).
The exception here occurs because...

We build against Guava 17.0 and Curator 2.7.1.
Curator 2.7.1 calls a Guava method MoreExecutors.sameThreadExecutor.
The method MoreExecutors.sameThreadExecutor was removed in Guava 21.0
At runtime Storm is providing Guava 28.0 (which does not include MoreExecutors.sameThreadExecutor) from /usr/hdp/current/storm-client/contrib/storm-autocreds/guava-28.0-jre.jar

The Fix
We need to relocate Guava 17.0 in the Solr uber jar.  This will allow Curator 2.7.1 to use Guava 17.0.  This is the same relocation that occurs for the Elasticsearch indexing topology.
Testing

Spin-up the centos7 development environment.
Kerberize the cluster.
Attempt to start the Indexing topologies.  The topologies should start successfully.

Pull Request Checklist
For all changes:

[x] Is there a JIRA ticket associated with this PR? If not one needs to be created at Metron Jira.
[x] Does your PR title start with METRON-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.
[x] Has your PR been rebased against the latest commit within the target branch (typically master)?

For code changes:

[x] Have you included steps to reproduce the behavior or problem that is being changed or addressed?

[x] Have you included steps or a guide to how the change may be verified and tested manually?

[x] Have you ensured that the full suite of tests and checks have been executed in the root metron folder via:
mvn -q clean integration-test install && dev-utilities/build-utils/verify_licenses.sh

[x] Have you written or updated unit tests and or integration tests to verify your changes?

[x] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?

[x] Have you verified the basic functionality of the build by building and running locally with Vagrant full-dev environment or the equivalent?

Ok, I could not get his one to work.
Spun up a centos7 cluster (1 hour 45 minutes for a deployment :( ) and then kerberised it ok.
The followed the instructions at:
https://metron.apache.org/current-book/metron-platform/metron-solr/index.html
Which should have got solr up and running, unfortunately when restarted Metron in Ambari, I started getting:

Traceback (most recent call last):
  File "/var/lib/ambari-agent/cache/common-services/METRON/0.7.2/package/scripts/enrichment_master.py", line 118, in <module>
    Enrichment().execute()
  File "/usr/lib/ambari-agent/lib/resource_management/libraries/script/script.py", line 352, in execute
    method(env)
  File "/var/lib/ambari-agent/cache/common-services/METRON/0.7.2/package/scripts/enrichment_master.py", line 110, in restart
    from params import params
  File "/var/lib/ambari-agent/cache/common-services/METRON/0.7.2/package/scripts/params/params.py", line 27, in <module>
    from params_linux import *
  File "/var/lib/ambari-agent/cache/common-services/METRON/0.7.2/package/scripts/params/params_linux.py", line 266, in <module>
    hostname_lowercase = config['hostname'].lower()
  File "/usr/lib/ambari-agent/lib/resource_management/libraries/script/config_dictionary.py", line 73, in __getattr__
    raise Fail("Configuration parameter '" + self.name + "' was not found in configurations dictionary!")
resource_management.core.exceptions.Fail: Configuration parameter 'hostname' was not found in configurations dictionary!

I could use the metron solr collection scripts to create the various collections, and I could then log into http://node1:8983/solr/#/bro/query and see that the solr collections were created.
And then I realised I could curl to the SOLR admin interface without any kerberos ticket
klist
klist: No credentials cache found (filename: /tmp/krb5cc_0)
set | grep SECURITY
KAFKA_SECURITY_PROTOCOL=SASL_PLAINTEXT
SECURITY_ENABLED=false./create_collection.sh yaf
  adding: schema.xml (deflated 80%)
  adding: solrconfig.xml (deflated 72%)
{
  "responseHeader":{
    "status":0,
    "QTime":16}}
{
  "responseHeader":{
    "status":0,
    "QTime":2731},
  "success":{
    "<IP_ADDRESS>:7574_solr":{
      "responseHeader":{
        "status":0,
        "QTime":1368},
      "core":"yaf_shard1_replica_n1"}}}

Is SOLR supposed to be kerberised as well?
I dug up some instructions at https://docs.cloudera.com/HDPDocuments/HDP3/HDP-3.1.4/managing-auditing/content/configuring_solrcloud_for_kerberos.html
Is this supposed to be for lucidwork's distrubtion or well this work for the (I assume) apache solr-cloud distro?

+1
Ran this up on a multi node kerberized solr cluster and I could see the random_indexing_topology comes up fine. Thanks for the fix @nickwallen !
These are the step I followed:

Deployed a 10-node cluster w/solr as indexing server
Followed the kerberization steps
At the end of the kerberization steps (during start services), verified that the start_solr_topology.sh script is able to bring up the RA Indexing topology fine.

+1
Ran this up on a multi node kerberized solr cluster and I could see the random_indexing_topology comes up fine. Thanks for the fix @nickwallen !
These are the step I followed:

Deployed a 10-node cluster w/solr as indexing server
Followed the kerberization steps
At the end of the kerberization steps (during start services), verified that the start_solr_topology.sh script is able to bring up the RA Indexing topology fine.

@anandsubbu Did you deviate in any way from the steps that @tigerquoll followed? Would be good to get an understanding of any gaps we potentially have here.

I suspect its probably due to me still being on the learning curve when it comes to Metron and kerberos.  @anandsubbu , where you following https://metron.apache.org/current-book/metron-platform/metron-solr/index.html to turn SOLR on, ir were you using some other means?   Is there an option to install Metron from scratch with SOLR?   Does a multi-node Metron install by default use lucid works SOLR?

@mmiklavc and @tigerquoll -
I typically follow these steps on a multi-node cluster

Spin up bare metal VMs
Install ambari-server
Install metron mpack
Install the HDP-Search 4.0.0 mpack from this link
Deploy the cluster

Once the cluster comes up, I follow the kerberization steps here - https://github.com/apache/metron/blob/master/metron-deployment/Kerberos-ambari-setup.md

This has been merged to the feature branch.


Use Azure AD state to pass the intended URL back to app

The comment explains.

Coverage decreased (-0.4%) to 47.961% when pulling d0f01c7b3c625e25d9ffc1a4cff25e3aec510bcc on azure-ad-pass-intended into 54de51d5a26cb630b931061734146c930715fa5d on develop.


Blog post tags are case sensitive

When using tags in a blog post, the tag has to be all lowercase or it will result in broken links.
To reproduce

Create a new blog post, set tags: ['CamelCase'].
Run the script to create the tag-subpages: ./scripts/generate-tags.
Now the link on the tag will point to /tags/CamelCase.html, but the subpage created is located at /tags/camelcase.html so you will get the 404 page.

It is confusing that the tags have to be all lowercase, I suggest naming the subpages exactly the same as the tag that created them

Maybe it is better to discuss the issue here and not in the PR.
The result of this is that there can be multiple tags like 'tutorials' 'Tutorials' 'TuTorials'. I wouldn't mind to have lower-case only tags - so I think we could adjust the link generation inside the pages to create the correct lowercase links.
The only downside  would be having no tags with proper uppercased letters.
@PanosSakkos any preference?

It's more efficient using a JavaScript validation before return the 404 error code, but only for the tags path.

@joariasl What do you mean by JavaScript validation?
@eiselems I agree, having all tag-pages lowercase makes more sense and is less confusing. For posts, the tags are displayed in all uppercase anyway, so you wouldn't see the difference between different 'versions' of the same tag.
I will amend the pull request.

@siboehm I meant to check the URL path in the request from JavaScript for redirect to lowercase.

Wasn't aware that everything is in uppercase anyway.
I just would suggest to create the link hrefs in lowercase letters and also generate lowercase pages for the tags.


Pre fill values redirect

Describe the problem feature solves
I have an action button that redirects
 handler: async (req, res, context) => {
          const { record, currentAdmin, h } = context
          return {
            record: record?.toJSON(currentAdmin),
            redirectUrl: h.resourceActionUrl({
              resourceId: 'invoice_item',
              actionName: 'new'
            })
          }
        }

I want to be able to pre-populate some of the fields with the redirect.
Describe the solution you'd like
A property that would allow us to map key: value properties
Or the ability to pass the values in the URL with params like ?myPropery=asdf
Describe alternatives you've considered
Wrapping a custom component, but that just seems really messy.
Acceptance criteria
Either the mentioned property or URL based solution

@dziraf any suggestions on how to tackle this if I were to try?

@AshotN
This should now be possible in the latest versions of AdminJS for new action:
https://github.com/SoftwareBrothers/adminjs/blob/da412320e61fe2dddd15ee6ee33adde73dec5565/src/frontend/components/actions/new.tsx#L31
const search = new URLSearchParams();
search.set('name', 'some_name');

h.resourceActionUrl({
  resourceId: 'invoice_item',
  actionName: 'new',
  search: search.toString(),
})


[move-prover] treat store as copy in borrow analysis

Prover panic-ed when trying to verify this simple program:
    fun one_of_two(cond: bool, r1: &mut u64, r2: &mut u64): &mut u64 {
        if (cond) {r1} else {r2}
    }

    fun test1(cond: bool, v1: u64, v2: u64) {
        let r = one_of_two(cond, &mut v1, &mut v2);
        *r = 0;

        spec {
            assert cond ==> v1 == 0;
            assert !cond ==> v2 == 0;
        }
    }

A further analysis found that the borrow summary for function one_of_two is not
properly constructed, with the return result not borrowing from neither r1 or r2
(which it should be a direct borrow of r1 or r2).
After some digging in the bytecode, it seems that we are using r := r1 and r = r2
for the assignment, which is considered as a Store assignment and is treated
the same way as a Move assignment. This seems correct in terms borrow semantics
but there is no way of summarizing that r can be borrowed from either r1 or r2
which is needed in the verification of test1.
A quick solution to this is to model Store as a Copy assignment, which adds
the BorrowEdge::Direct in the borrow graph. The downside, however, is that the
RHS of the assignment now lives longer until it is destroy()-ed, as can be seen in
the changes of the baseline file.
Motivation
To verify the above simple program.
Have you read the Contributing Guidelines on pull requests?
Yes
Test Plan

CI
A new unit test is added
Manually run the prover inconsistency tests against DPN

Still self-debating whether we should call this type of edge 'Direct' or invent a new name, but that can be addressed separately.


multi tenancy proposal

First I want to state Im not expert in security but Ive tested a thought I had about securing the the data deis stores in etcd .
First an key in Etcd that starts with a underscore (_) can not be listed so if the system were to use a large random string that could be set a install or even better updated that started with a underscore would limit any one on the platform from getting to keys that they should not be able to access . Ive tested this and it dose work so for example I replaced the etcd keys that started with deis to
_somereallylongrandomstringwithnumbers/database/adminUser

so as I siad Im no security expert but if we could make this key be updated after install so it can be changed to stop long running brut force guessing of the hash string than I think that can secure access the to hidden keys.

there is no security as of now special not at container level .. a container is not a VM .. its a simple process .. and not even a vm is secure .. you need way more to archive multi tennancy then just a "etcd" scoping and that problem is gone with etcd 2.0
but you realy want to scope the networking ! ( and with the current overlay mechanics  thats not possible) .. + noone realy knows the security implications of an internal vector .. in short dont run anything in a container you did nod audit carefully ..
1 example is just a drain of the bandwidth .. kills etcd / datastore communication and produces an dead cluster .. while the goal is certainly reachable unsure if you are aware of the scope + inovation involved cuz there is near to no tooling for that

while coreos takes care of the os updates who updates sec holes inside a container ? right noone .. please dump the idea of multitennancy with containers unless you know exactly what you are doing .. and thats going to result in a highly time / cash expensive task

https://github.com/coreos/etcd/blob/master/Documentation/rfc/api_security.md

its was just the ectd keys I needed to secure , that's what was on the deis.io site .. I'm not going to use in a public network I just need a way for my clients to be able to run services that we supply that can edit the code but most don't I just wanted to make sure that and passwords in etcd could not be accessed by any one else on the platform. while Im not worried about having one of my client bring down the network as they will kill there own service in the process I do want to make sure any database password or other etcd data is not access able

@apps4u Let's try a discussion on the deis-users mailing list - I'm not sure this approach is the best one, and hopefully more brains can provide input. Thanks for bringing this up!


fix get queries field type

@ysugimoto I have added my observations here https://gist.github.com/tarunvelli/5cb0e12eeb12df29fa0bcb20aab54f4e
In short, the graphql gateway isn't able to read the input value when using FieldType in query argument generation

@ysugimoto any update on this?

@tarunvelli Sorry for being late. I looked at your Gist, thanks.
I understand your point about this PR. I guess it caused by ptypes implementation on here.
I intend to use FieldType and FieldTypeInput for definitely distinguish between type and input definition in GraphQL.
Your change seems to look fine, but I'm still wondering why the query is successful when using FieldTypeInput,  and failure on FieldType, so I'll continue to investigate graphql-go's implementation and ptypes implementation.
If you have any information about it. let me know, thanks!

@tarunvelli I've looked into it deeply and I realized your changes are a good way to fix the problem!
The primal problem is that is not able to pass an object to the Query argument. The reason why is I've planned to extract protobuf's message and list fields of them, and I didn't consider of message type argument.
Hence, it causes not only types like google.protobuf.Int32Value but also any nested message. I reproduced as following:
syntax = "proto3";

import "graphql.proto";
import "google/protobuf/wrappers.proto";

option go_package = ".;echo";

service Echo {
  // gRPC service information
  option (graphql.service) = {
    host: "localhost:50051"
    insecure: true
  };

  rpc Sample(SampleMessage) returns (SampleMessage) {
    option (graphql.schema) = {
      type: QUERY
      name: "sample"
    };
  }
}
message Member {
  int64 id = 2;
}

message SampleMessage {
  Member member = 1;
}

This example is similar to your PR, the difference is that using the user-defined message type as query argument field and this case also responds null for member field.
To solve this problem, we need to modify a bit in protoc-gen-graphql process but I have a plan to do it.
Totally you're right. Nice catch and thank you for making PR and giving me an example!

@tarunvelli I've just published a fixed version as v0.19.0. I'm happy if you could try this version 😄


Nfdeployment CR specifications for AMF, SMF, UPF

This PR specifies the CR definitions for UPF/SMF/AMF deployment according the agreed CRD doc.
https://docs.google.com/document/d/1wtA5BJnn1GGq-yWa7FUAOrCJuyQqifB3f4ky-IePIzo/edit?usp=sharing

Could you please change from file names nf.nephio.org to workload.nephio.org? We had a discussion in sig-auto-workload regarding the group name.

Otherwise looks good. I will rework UPFDeployment once this PR is merged.

/approve


Python Bindings for cpython-36m-x86_64-linux-gnu.so

Would it be possible to provide python bindings with the 36m suffix? The current bindings is pareto.cpython-38-x86_64-linux-gnu.so and the 38 is not supported on my linux machine, while the pyfront.cpython-36m-x86_64-linux-gnu.so was supported.
Would appreciate a 36m version of the bindings.

I'll have a look into that. The bindings depend on pybind11 and I haven't seen anything in the documentation related to that yet.
I guess pybind11 identifies the proper suffixes by inspecting the machine, and that's the proper suffix for the GitHub actions machine that generates the bindings.
If that's the case, it's probably easier for you to compile the library from source on your machine and see if it generates the bindings you need.
In the end, that's what scikit-build does under the hood anyway. A better approach would be to integrate scikit-build into the project, which is actually in the roadmap. But I don't have a lot of experience with scikit-build yet. PRs are welcome though! 😀


Use non-synchronising scan

This switches the inclusive scan in the row partitioner from thrust to cub to avoid the synchronisation of thrust APIs from cuda 10.1 onwards

I need to run some large deterministic tests for cub algorithms.

This one is just integers, but yes we need to watch out for that non deterministic floating point reduction we had with cub.

It's a bigger job to comprehensively go through and change everything. This one is important because it gets called ~15000 times when building 500 trees, the others can wait I think.


Add files via upload

Look like SP copy_monitor_plan_to_workspace.sql here still has the old script (Max D did updated in the database) and with one in parameter. The new SP has two in parameters.

Updated scripts for 5 SP are loaded at the main and deployment. It's in Sprint14, so new folder added at release-v2.0-fix.

@ayang2017 Please work with @maxdiebold-erg to resolve the listed conflicts.

@djw4erg @ayang2017 There is a "..." file that was added in the deployment folder for this ticket.  This file is causing an "invalid path" error when I try to pull these changes to my local.
You will need to create a new PR that removes this file from the deployment directory.


Prep for core model workshop

User story
As a user, I want to find what I need quickly and understand it easily so I can do my job.
Background & context
The core model provides a way to anchor content plans around our goals, user needs, and the user journey, creating guidance for consolidating redundant content and IA development. To have an effective workshop time, you need to make sure the right people are involved and that you the info they need to get started.
Open questions

Attendees who would be best for this work and have the time
Which platform is best to do the work on

Working links
Planning doc

Acceptance criteria

[ ] Update service blueprint with relevant learnings
[ ] Update the questions pool with any questions that came up
[x] Relevant research gathered
[x] Materials created

Deck for the workshop


Quantization - how to do FP8 quantization for the LLama8B and 70B models?

🚀 The feature, motivation and pitch
FP8 is pretty main-stream - just ask Jensen - but I have no idea how to do the FP8 quantization using torch.float8_e4m3fn? Could you please provide the script so I can do the conversion and test/compare the outputs using both 8B and 70B.
I also don't trust the other quantization methods and would prefer to see how it's done using torch.
Alternatives
AWQ and GPTQ - but I would prefer an official example.
Additional context
This is meant to be a tutorial repo!

Hi! Here is some reference codes I can find related to fp8 on llama-stack and the pytorch library for fp8 support. Let me know if you have any questions!


Rename LICENCE to LICENSE

Copyright holder can't be empty.

Thanks for your help with licensing issues.


Drag'n'drop mechanism to order ontologies

Instead of entering a numerical value for the ordering field, a drag'n'drop should be possible on the /admin/ontologies page.

Ontologies can now be ordered with a system of arrows:

It's not ideal, but it will do for now. We can look into the drag'n'drop for a future release.


Is it normal the pdb are not moved to the library folder ?

Hello guys, and thanks for this library.
it works greate except the pdb which stay at the root (yes i want to publish the pdf in release mode to have line number in case of exceptions).
Is it normal ? can it be configured ?
thanks !

pdb should stay with it's dll. can you post a demo repo/bin so i can see what is going on?

yes you can repro with this project : https://github.com/julienGrd/BlazorAppTestNetBeautyWithPdb, i use Visual Studio Community 2022 17.8 Preview 2.0. To reproduce

right click, publish on the server project
go to the publish folder, you will see ClassLibrary1.dll and ClassLibrary2.dll in the libraries subfolder but ClassLibrary1.pdb and ClassLibrary2.pdb are still in the root

fixed in v<IP_ADDRESS>


Include exception data when listener faield to start

Doneness:

[X] Build is okay - I made sure that this change is building successfully.
[X] No Bugs - I made sure that this change is working properly as expected. It doesn't have any bugs that you are aware of.
[ ] Branching - If this is not a hotfix, I am making this request against develop branch

:white_check_mark: Build Titanium-Web-Proxy 3.0.206 completed (commit https://github.com/justcoding121/Titanium-Web-Proxy/commit/baa82682d7 by @honfika)


Config?

Hi, i dindnt know how to change the config for the flag ( Give-effect) im trying to put a flag with Night_vision but i dont know how to set time because its only 7 sec actually ..  Is that only possible?

That is not possible

the problem is when i use night vision, i've a sparkling effect


WIP: Release nginx-slim 0.25

This release adds support for opentracing (zipkin) and HTTP/2 headers compression

This change is

Coverage remained the same at 43.905% when pulling 527f18185db58523edb25ef42b4e3a30217d970e on aledbf:nginx-opentracing into 1a68536e2966e2687e6dbd9a4926f099dcf227d9 on kubernetes:master.

Coverage remained the same at 43.905% when pulling 319a60a3130714f0202c4017112b96a609d519a6 on aledbf:nginx-opentracing into 1a68536e2966e2687e6dbd9a4926f099dcf227d9 on kubernetes:master.

@aledbf did you test this image in a high load environment?

@gianrubio I don't see an increment in the memory (testing with ~13000 rules and moderate load)

Looking forward to test the opentracing feature!!!

@nicksardo please merge and publish the image. Thanks!


Is this dead code?

// Rotate memdb if it's reach the threshold.
if batch.internalLen >= mdbFree {
	db.rotateMem(0, false)
}

The snippet code above is at L248  in db_write.go. I think the condition would never be true because the returned value mdbFree of db.flush at L154 is guaranted to be larger than batch.internalLen.
So is the snippet above a dead code?
Thanks

There are the case were batch.internalLen == mdbFree, the greater than
is just for good measure.
On Tue, Jun 22, 2021, 7:45 PM Alex-duzhichao @.***>
wrote:

// Rotate memdb if it's reach the threshold.
if batch.internalLen >= mdbFree {
db.rotateMem(0, false)
}
The snippet code above is at L248
https://github.com/syndtr/goleveldb/blob/master/leveldb/db_write.go#L248
in db_write.go. I think the condition would never be true because the
returned value mdbFree of db.flush at [L154] (
https://github.com/syndtr/goleveldb/blob/master/leveldb/db_write.go#L154)
is guaranted to be larger than batch.internalLen.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/syndtr/goleveldb/issues/360, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAFRZLOFH5PZFOHRMK4NXTLTUCAV7ANCNFSM47DTUJMA
.

Also mdbFree is not a hard limit, so actually batch.internalLen can be
greater than mdbFree because of batch merging.
On Wed, Jun 23, 2021, 3:34 AM Suryandaru Triandana @.***> wrote:

There are the case were batch.internalLen == mdbFree, the greater than
is just for good measure.
On Tue, Jun 22, 2021, 7:45 PM Alex-duzhichao @.***>
wrote:

// Rotate memdb if it's reach the threshold.
if batch.internalLen >= mdbFree {
db.rotateMem(0, false)
}
The snippet code above is at L248
https://github.com/syndtr/goleveldb/blob/master/leveldb/db_write.go#L248
in db_write.go. I think the condition would never be true because the
returned value mdbFree of db.flush at [L154] (
https://github.com/syndtr/goleveldb/blob/master/leveldb/db_write.go#L154)
is guaranted to be larger than batch.internalLen.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/syndtr/goleveldb/issues/360, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAFRZLOFH5PZFOHRMK4NXTLTUCAV7ANCNFSM47DTUJMA
.

Yes, get it.
Thank you so much.


Basic implementation of column max resize on double click

i. Resize the column to it's max width on double click.

One basic error which I can observe, is that whenever you click on it, it sorts before resizing. Please look into that.

@samrith-s That will be fixed when this is merged. https://github.com/samrith-s/sticky-react-table/pull/19/files#diff-40d712379ef434e2e2e2acfe4703ae89R54


VR: add '-m ' for tcp or udp protocol

Description
This PR fixes missing "-m tcp" or "-m udp" which cause some networking issues.

Types of changes

[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] New feature (non-breaking change which adds functionality)
[x] Bug fix (non-breaking change which fixes an issue)
[ ] Enhancement (improves an existing feature and functionality)
[ ] Cleanup (Code refactoring and cleanup, that may add test cases)

Feature/Enhancement Scale or Bug Severity
Feature/Enhancement Scale

[ ] Major
[ ] Minor

Bug Severity

[ ] BLOCKER
[ ] Critical
[ ] Major
[x] Minor
[ ] Trivial

Screenshots (if appropriate):
How Has This Been Tested?

@blueorangutan package

@weizhouapache a Jenkins job has been kicked to build packages. It will be bundled with  KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

Packaging result: :heavy_check_mark: el7 :heavy_check_mark: el8 :heavy_check_mark: debian :heavy_check_mark: suse15.  It will be bundled with
SystemVM template(s). I'll keep you posted as I make progress.

Packaging result: :heavy_check_mark: el7 :heavy_check_mark: el8 :heavy_check_mark: debian :heavy_check_mark: suse15. SL-JID 3076

@blueorangutan test

@weizhouapache a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

Trillian test result (tid-3815)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 32060 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr6188-t3815-kvm-centos7.zip
Smoke tests completed. 92 look OK, 0 have errors
Only failed tests results shown below:

Test
Result
Time (s)
Test File

Trillian test result (tid-3822)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 378 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr6188-t3822-kvm-centos7.zip
Smoke tests completed. 0 look OK, 1 have errors
Only failed tests results shown below:

Test
Result
Time (s)
Test File