When using ExpectedConditions after browser.restart(), getting "Driver does not have valid session ID" error

From azachar (copied from #3677)
Hi There,
I am getting a very similar issue still on the version 4.0.13, it happens sometimes...
the test opens the page and waits to have a certain part in the url using the function bellow, the tests are using browser.restart()  in afterEach. Since I am using a random order of running, it happens only sometimes.
  this.waitForUrl = function(url, timeout) {
    timeout = timeout || DEFAULT_WAIT_FOR_TIMEOUT;
    return this.browser.wait(EC.urlContains(url), timeout, 'waiting url to contains ' + url);
  };

I am getting this error:

Failed: This driver instance does not have a valid session ID (did you call WebDriver.quit()?) and may no longer be used.
NoSuchSessionError: This driver instance does not have a valid session ID (did you call WebDriver.quit()?) and may no longer be used.
    at WebDriverError (/home/ubuntu/xxx/node_modules/selenium-webdriver/lib/error.js:27:5)
    at NoSuchSessionError (/home/ubuntu/xxx/node_modules/selenium-webdriver/lib/error.js:170:5)
    at checkHasNotQuit (/home/ubuntu/xxx/node_modules/selenium-webdriver/lib/webdriver.js:395:15)
    at Driver.schedule (/home/ubuntu/xxx/node_modules/selenium-webdriver/lib/webdriver.js:356:5)x
    at Driver.getCurrentUrl (/home/ubuntu/xxx/node_modules/selenium-webdriver/lib/webdriver.js:804:17)
    at /home/ubuntu/xxx/node_modules/protractor/built/expectedConditions.js:306:41

The screenshot from this failure clearly shows that the page is fully loaded and has its content so I guess this might be a bug. I guess that is related to this issue too.
Anyway, thank you for your help.
Best regards,
Andrej
Here is the failing test case:
describe('This driver instance does not have a valid session ID', function() {
  var EC = protractor.ExpectedConditions;

  function waitForUrl(url) {
    browser.wait(EC.urlContains(url), 5000).then(function() {
      console.log(url + ' found');
    })
  }

  it('Browser wait for url fails', function() {
    browser.get("https://angularjs.org/");
    waitForUrl('angular');

    browser.restart();

    browser.get("https://angularjs.org/");
    //fails here // This driver instance does not have a valid session ID (did you call WebDriver.quit()?) and may no longer be used.
    waitForUrl('angular');
  });

  it('Browser ignoreSynchronization fails', function() {
    browser.get("https://angularjs.org/");
    browser.restart();
    browser.get("https://angularjs.org/");

    browser.ignoreSynchronization = true;
    browser.get("https://github.com/");
    waitForUrl('github'); //fails here This driver instance does not have a valid session ID (did you call WebDriver.quit()?) and may no longer be used.

    browser.ignoreSynchronization = false;

    browser.restart();

    browser.get("https://angularjs.org/");
    waitForUrl('angular');
  });

});

here is protractor.conf
{
  rootElement: 'html', //it fails also with [ng-app]
  allScriptsTimeout: 110000,
  baseUrl: 'http://localhost:9000',
  chromeOnly: true,
  directConnect: true,
  exclude: [],
  capabilities: { browserName: 'chrome' },
  framework: 'jasmine',
  jasmineNodeOpts: { defaultTimeoutInterval: 15000, random: false },
  specs: [ 'e2e/**/*.spec.*.js', 'e2e/**/*.spec.js' ]
}

The issue is that you should be saving a reference to ExpectedConditions as in
  var EC = protractor.ExpectedConditions;

If you set EC again after each restart call, But to be sure this is confusing.  I made https://github.com/angular/protractor/issues/3884 to document a possible solution

Agree. EC has a reference back to the browser object. If the browser object goes away, the EC is no longer valid after the session quits.
A work around, for now, would be to get the ExpectedConditions from the browser object:
  function waitForUrl(url) {
   var EC = browser.ExpectedConditions;
    browser.wait(EC.urlContains(url), 5000).then(function() {
      console.log(url + ' found');
    })
  }

Does it make sense to have protractor.ExpectedConditions if people can run into this problem? Should we just make browser.ExpectedConditions the only official way to create ExpectedConditions?

^^^ Maybe we could make this as a breaking change in beta since ExpectedConditions is no longer static. @juliemr thoughts?

Thanks a lot for your investigation, perhaps this will resolve some issues I had to comment out. I am using chat a like app to test, so I do have many browser instances... I will come back to you if this helped.

Is this Fixed?

Problem still occurs. I need to restart browser between 2 specific test cases and I'm not able to due to this issue. After restart page loads but error appears: -

Failed: This driver instance does not have a valid session ID (did you call WebDriver.quit()?) and may no longer be used.

Hi, i have same problem with 'restartBrowserBetweenTests' parameter and 'browser.restart()'. I check selenium logs and i have test cases + 1 selenium session. And of course "This driver instance does not have a valid session ID (did you call WebDriver.quit()?) and may no longer be used."

@wieta after some study I came to the source of issue which I think is common for most of users.
If you are using page object pattern and you initialize and define objects in the "desc" segment, or even in bigger scope these objects will always refer to the first browser instance.
To solve the issue, you must decache them (https://www.npmjs.com/package/decache) and then use "require" again and also agian initialize and define variables.
For my project I created a common utility function which always returns new Page Object. After I restart the browser I use my function to re-load all Page Objects and now it works perfect.
Bare in mind to split data from PO files. If you want to pass data from one spec to another, create a module which does only that task, So you don't have to decache it too.

same issue as mentioned above.
I am facing a similar issue, when using browser.restart. In my case, a subsequent browser.wait call waits almost indefinitely, and times out after an obscene amount of time, although the page is loaded and visible the whole time.


Package/Release on PyPI

It would be cool if users could install this directly using pip from PyPI. I seems like the various modules could be put into a single top level package and then easily reused as a library dependency by other Python libraries/applications.

I've never really investigated making a Py PI package, but I'm open to the idea if someone wants to do the work or show me the way.

I've created an example project and some discussions here: https://github.com/johnthagen/python-blueprint#packaging


Implement extent checks for point-polygon processes

This should ideally test whether that introduces any bugs.

Nice. Got any before/after benchmarks?

Not beyond "keeps running" vs "computation finishes" :D

Finishing is good

@rafaqz would you mind giving this a sense check to see if it makes sense?  I'm not 100% firm on how the maybe_skip interface works.

@skygering maybe better if you review this if you can, I also don't really know the interface


Fuzzing bug test_syntax "Formatting bug; formatting is not stable"

Run on @joshuawarner32 branch for #7353
$ cargo run --bin minimize expr todo
    Finished dev [unoptimized + debuginfo] target(s) in 0.39s
     Running `target/debug/minimize expr todo`
Error found: Initial parse failed: Expr(Pattern(Start(@0), @0), @0)
Proceeding with minimization
Successfully minimized, new length: 42
Successfully minimized, new length: 12
Successfully minimized, new length: 3
No more minimizations found
Final result:
1,

Formatting bug; formatting is not stable.
Original code:
1,A5Q,A
1,A5A,A
,1
,01
,A5

"""

g

d""""INFO: exiting: 19712 time: 146s

Hmm this one seems to have gotten corrupted. Can you upload the original file? e.g fuzzing should have ended with a line that looks like this:
artifact_prefix='/Users/joshw/src/github.com/roc-lang/roc/crates/compiler/test_syntax/fuzz/artifacts/fuzz_expr/'; Test unit written to /Users/joshw/src/github.com/roc-lang/roc/crates/compiler/test_syntax/fuzz/artifacts/fuzz_expr/crash-94d31a614ec259c9ba5d19c793fbdb442b7b912e

If the file at the path after "Test unit written to" still exists on disk, that'd be very useful

Added .txt so GH would accept it
crash-811d6a1d210d35180bb14b1e7fbfeb2591115dec.txt

Thanks!
Looks like this was fixed by one of the fixes I made along the way (not sure which one...)


[Question] Interactive responses

Description
I'm interested in enabling interactions in my app. I played around with sending blocks in the web client and that works well.
I already have a system in place for receiving real-time events with the RTM client.
To my understanding, receiving interactive responses from users are only supported with HTTP requests. Is it not possible to receive these responses in the RTM websocket connection?
What type of issue is this? (place an x in one of the [ ])

[ ] bug
[ ] enhancement (feature request)
[X] question
[ ] documentation related
[ ] testing related
[ ] discussion

Requirements (place an x in each of the [ ])

[X] I've read and understood the Contributing guidelines and have done my best effort to follow them.
[X] I've read and agree to the Code of Conduct.
[X] I've searched for any related issues and avoided creating a duplicate issue.

You would be correct 🙇♀ it's not possible to receive interaction payloads over RTM at this time. 😕 Sorry about that! As you suggested, the message interactivity payloads are currently only sent via HTTP POST.


Make detect print optional

It would be nice to make the printing function of detect optional.  I currently use it to check if the machine the code is running on has GPUs, otherwise it uses a slower CPU kernel, but it prints every time it checks for GPUs.

cuda.detect() is for printing diagnostic.  You can always check with:
>>> cuda.is_available()  # check for cuda driver
True
>>> list(cuda.list_devices())  # check for cuda devices
[<numba.cuda.cudadrv.devices._DeviceContextManager object at 0x10217ba90>]
>>> for d in cuda.list_devices():
...      print(d.compute_capability)
...
(3, 0)

Closing as solution present.


Whether OpenBLAS supports 鲲鹏920

hi， I compilered OpenBlas using soure code in TaiShan 200 service. I get some error as follow:
copy.c:1:0: error: unknown value ‘armv8.2-a’ for -march
//
^
scal.c:1:0: error: unknown value ‘tsv110’ for -mtune
copy.c:1:0: error: unknown value ‘tsv110’ for -mtune
make[1]: *** [saxpy.o] Error 1
make[1]: *** Waiting for unfinished jobs....
make[1]: *** [sswap.o] Error 1
dot.c:1:0: error: unknown value ‘armv8.2-a’ for -march
//
^
dot.c:1:0: error: unknown value ‘tsv110’ for -mtune
make[1]: *** [sscal.o] Error 1
sdsdot.c:1:0: error: unknown value ‘armv8.2-a’ for -march
/*********************************************************************/
^
make[1]: sdsdot.c:1:0: error: unknown value ‘tsv110’ for -mtune
seem not support armv8.2 architecture?

The "unknown value" are error messages from your compiler, apparently it is too old to support these options. (Or if you are cross-compiling, you called the wrong compiler). If you cannot install a newer
gcc (I think you need gcc 9.1 for tsv110 support), you can edit the options in Makefile.arm64

Your compiler does not support optimisations for particular CPU.
You can choose some from kernel/arm64/KERNEL.*, like A72/A73

thanks for your reply ! I get some information from your reply, I will try.

thanks for your reply ! I get some information from your reply, I will try.

Also clang from 2019 can help, just that no fortran compiler there and no lapack will be built, but you can wrap it with less optimized gfortran build too.


Fix small issue when iterating and stdout option is ignore

This PR fixes a small bug when iterating over promise.stdout/promise.stderr and the stdout/stderr option is 'ignore' (or anything else but 'pipe').

Fixed tests.


Add a script for path-finding in the implications graph

This has a large overlap with Graphiti, but I think there's still place for a script like this. It was useful for me when exploring properties of the implications graph, so I polished it a bit to push to the common repo.
Closes #444

Example output:
$ python scripts/explain_implication.py Equation1167 Equation1025
Equation2000  has a model that does not satisfy  Equation1025
    proved for the dual: Equation1659 =/=> Equation2446
    Equation1659_facts  in  ManuallyProved/Equation1659.lean
Equation2000 => Equation1167
    Equation2000_implies_Equation1167  in  Generated/VampireProven/Proofs4.lean

awaiting-review


Handle null pointer in case of unresolved module paths

Dependencies crashes when ModuleSearchStrategy is ApiSetSchema and the module cannot be found. I am not sure if the real problem is that the module path should have actually been found, but at least here is a quick fix. This is similar to DependenciesGui that simply ignores the entries, an alternative would be to return a NOT_FOUND string.

As discussed in this issue https://github.com/lucasg/Dependencies/issues/39, ext-ms dlls are optional api set dlls, so they can be missing on some platforms (that's precisely the point of apiset schema).
I'll merge your fix, but I have to take a mental note to restructurate the findPe algorithm since it becomes pretty hacky


[GEN-10044] chore: updates to allow for async WAL verification

Remaining

[ ] Updated binaries for both admin API and admin-mgr
[ ] Version bump

Changes in this PR are no longer needed. Closing


Meilisearch v1.10.0 language settings

Describe the bug
In v1.10.0 I am able to update the language settings using the main settings route, but I get 404s when I try to update the language settings alone. I'm not sure if I just can't find the correct address or if it is not possible to set language individually like with the other settings?
For example with proximity precision I can get, put, and delete to http://<IP_ADDRESS>:7700/indexes/movies/settings/proximity-precision. I tried this with multiple options for language and get a 404 with all of them: localized-attributes, localization, language, language-settings.
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
To be able to set the language settings individually like other settings.
Screenshots
If applicable, add screenshots to help explain your problem.
Meilisearch version:
v1.10.0-rc.0
Additional context
Additional information that may be relevant to the issue.
[e.g. architecture, device, OS, browser]

Hello @sanders41
Thank you for the report, we indeed already saw it, and we merged a PR as explained here: https://github.com/meilisearch/meilisearch/pull/4842#issuecomment-2258897455
I should have opened an issue for you to avoid to open this one 🙈
Closed by https://github.com/meilisearch/meilisearch/pull/4836. Will be released in rc1 this week 👍


Setup Vercel deployment

Current dependencies on/for this PR:

main

PR #126   👈

This comment was auto-generated by Graphite.


About the minimum  version

Why A / B Testing, and Dynamic Tweaks can use the minimum version is 16,

To guarantee that those features work as expected, we rely on some android functionality that was introduced in API 16 like android:importantForAccessibility :)


⚠ Bofrost.at has degraded performance

In 68e4abf, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 2536 ms

Resolved: Bofrost.at performance has improved in 7569c8d after 6 minutes.


Feature: Prompt for credentials

Instead of failing when no credentials are found, ask them to the user.
This feature is opt-in through the "prompt_credentials" setting, and is only used on 'push', 'pull' and 'fetch'.
Solves #346.
The main drawback is that it add a dependency to "pexpect" for passing the username and password to Git from Sublime.
I used an old version of pexpect where the code was regrouped in only one file.

AFAIK, old versions of Pexpect doesn't support window. Shall we move to newer version instead?

Good point, I only tested on Ubuntu 16.04. But as I said, I'm not sure this dependency is mandatory, Popen should allow to pass Username and Password through stdin by I didn't manage to make it work.

I would not use the handcrafted code with Popen when pexpect, which is more reliable, exists

Have you tested this on Windows?

Yeah, current docs state:

The main features of Pexpect require the pty module in the Python standard library

I really would like to merge this, but I think this is 1) very heavy, and 2) not portable.  As mentioned in another ticket, when I have time I'll be revisiting how GitSavvy interacts with git in the same way that SublimeRepl does it.  That would be cross-platform and hopefully will provide a foundation for interactivity with git when it prompts for input.
Also, this seems to mostly work for pull, push, and fetch, but I'd rather see a solution that works for all commands, since there are other cases when interactivity is desired.  The return-early return-nothing when fatal: could not read Username is detected seems brittle.
For now, I'm going to have to close this without merging.  If you can think of another way to make this happen, I'm definitely open to ideas!

@divmain I agree it's a bit hacky.
I put the limitation on "push", "pull" and "fetch" to avoid catching errors not related to authentication.
I haven't try it on Windows mostly because Git for Windows seems to permanently store credential by default.
Maybe I could take a look at SublimeRepl code to see how they provide this kind of interaction.

If you'd like to take a look (and/or create an issue to track), please feel free.  It'll be a significant change, but definitely worth investigating!


Repackage and release updated gem (Rails >= 3.1 changes not in release gem)

I noticed that the gemspec hasn't been updated in about a year. Recent changes, such as that which only includes the s3_upload.js file if the Rails version is < 3.1, are actually quite useful.
Would love it if the live gem included those commits. Thank you!

Flash just isn't a thing anymore. Calling this closed!


fix(test): Distinguish 'testname' from escaped arguments

When working on clap v4, it appeared that last and trailing_var_arg
are mutually exclusive, so I called that out in the debug asserts in
#4187.  Unfortunately, I didn't document my research on this
as my focus was elsewhere.
When updating cargo to use clap v4 in #11159, I found last and
trailing_var_arg were used together.  I figured this was what I was
trying to catch with above and I went off of my intuitive sense of these
commands and assumed trailing_var_arg was intended, not knowing about
the testname positional that is mostly just a forward to libtest,
there for documentation purposes, except for a small optimization.
So it looks like we just need the last call and not the
trailing_var_arg call.
This restores us to behavior from 531ce1321d282d3a975d0799a322e6bd20f4c3c6 which is what I originally
wrote the tests against.
It looks like #11159 was merged after the last beta branch was made, so we shouldn't
need to cherry-pick this into any other release.
For reviewing this, I made the test updates in the first commit, showing the wrong behavior.  The following commit fixes the behavior and updates the tests to expected behavior.
Fixes #11191

r? @ehuss
(rust-highfive has picked a reviewer for you, use r? to override)

Feel free to r=weihanglo once you've done the confirmation. I may update Cargo in rust-lang/rust after this get merged.

Could you help double confirm that cargo run, cargo rustc, and cargo rustdoc are not affected? From my understanding they should behave as before and just tested by hands them seem fine.

Those commands are unrelated to the problem.  This is a problem specifically with how I handled the change from .last(true).trailing_var_arg(true) which only affected cargo bench and cargo test (I searched  #11159 multiple times to confirm).  Even then, the only way to observe a problem is if people are relying on the specialization of logic that exists only in cargo test..  This precludes observing a change in behavior in cargo bench as it just takes BENCHNAME and prefixes it to args without any logic conditioned on the existence of BENCHNAME; no matter whether -- is used or not, the same behavior will occur.  It took me several tries to even create a good test case for cargo test that clearly showed the problem due to how specific the relevant problem is.

Sorry for my oversight on this behaviour change.

Responsibility first lies with the author, reviewers can't be expected to catch everything. Unless you knew to check this  exact case, I bet we would have missed it anyways.
r=weihanglo

@bors r=weihanglo
(of course I forgot to tag bors)

:pushpin: Commit 3dd8413f770103fda091e88b5e5916313e3a707a has been approved by weihanglo
It is now in the queue for this repository.

:hourglass: Testing commit 3dd8413f770103fda091e88b5e5916313e3a707a with merge 3cdf1ab25dc4fe56f890e8c7330d53a23ad905d3...

I should have thought of that, since I did some fixes around it…
Anyway, thanks for your quick patch!

(hi I'm the one that found the behavior change)
It's definitely good to restore previous behavior here, but it's also I think worth discussing whether cargo test TESTARG and cargo test -- TESTARG should be behaving differently.
To me, it seems quite odd that being explicit about which arguments are passed to the test runner should change how the test runner is run.
Plus, in general, due to how rustdoc (ab)uses the test runner, flags can have distinctly different behavior. (The notable one I'm dealing with is --ignored.)

(hi I'm the one that found the behavior change)
It's definitely good to restore previous behavior here, but it's also I think worth discussing whether cargo test TESTARG and cargo test -- TESTARG should be behaving differently.
To me, it seems quite odd that being explicit about which arguments are passed to the test runner should change how the test runner is run.
Plus, in general, due to how rustdoc (ab)uses the test runner, flags can have distinctly different behavior. (The notable one I'm dealing with is --ignored.)

https://github.com/rust-lang/cargo/issues/6981 is the issue for that.

:sunny: Test successful - checks-actions
Approved by: weihanglo
Pushing 3cdf1ab25dc4fe56f890e8c7330d53a23ad905d3 to master...


Cannot distinguish which resources are in use

This issue is half bug report, half speculation.
Much of Crossplane's design is inspired by that of Kubernetes's PersistentVolume support. Both designs manage concrete 'external' resources - a PersistentVolume or a managed resource such as an RDSInstance. Both designs support provisioning the external resources on demand using parameters set in a 'class' when a 'claim' is created.
One notable difference is that a PersistentVolume must be bound to a PersistentVolumeClaim before it can be consumed by a Pod. This has the nice property that an unbound PersistentVolume can be presumed to be available for binding to a new PersistentVolumeClaim.
Conversely, there's not currently any way for a Crossplane claim (or user) to distinguish whether a resource is in use or available. Resources write out their connection secrets (i.e. URL, username, password, etc) at create time, at which point something may or may not start consuming them. Resources expose a .spec.claimRef, so it's reasonable to assume a resource with a claimRef has been claimed and is in use. Unfortunately there's no way to tell whether a resource has no claimRef set because it's sitting idle ready to be claimed, or because someone created it directly and is actively using it without any intention of claiming it via a resource claim.

This issue is a re-scoping of #517.

I believe it would be a nice property to require, at least by convention, a resource claim to exist in order to consume a resource. Put otherwise, we should let users assume a resource without a .spec.claimRef is available to be used, which would first require claiming it.
This has some interesting implications for connection secrets (#518) in that resource connection secrets could be assumed to be consumed only by resource claims.

Another observation on this topic - it seems appropriate to make resource claims the controller (as in controller reference) of the resources they create (or retroactively claim). If all resources are controlled by a resource claim the claim reference becomes redundant.


kears.layers.concatenate Does Not Work when Saving a Model

Please make sure that this is a bug. As per our GitHub Policy, we only address code/doc bugs, performance issues, feature requests and build/installation issues on GitHub. tag:bug_template
System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): A custom model
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Windows7
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device: No
TensorFlow installed from (source or binary):
TensorFlow version (use command below): 2.0.0
Python version:  3.75
Bazel version (if compiling from source):
GCC/Compiler version (if compiling from source):
CUDA/cuDNN version: 10.0.1
GPU model and memory: Quadro K620M

Describe the current behavior
I built a custom model as follows
	class C3BR(tf.keras.Model):
		''' 3D Convolution + Batch Normalisation + Relu '''
		def __init__(self, filterNum, kSize, strSize, padMode):
			super(C3BR, self).__init__()
			self.conv = layers.Conv3D(filters=filterNum, kernel_size=kSize, strides=strSize, padding=padMode, data_format='channels_first')
			self.BN = layers.BatchNormalization(axis=1)

		def call(self, inputs, ifTrain=True):
			x = self.conv(inputs)
			if ifTrain == True:
				x= self.BN(x)
			return activations.relu(x)

		def build_model(self, input_shape):
			''' A work-around to define dimensions of signals through the NN'''
			self.build(input_shape)
			inputs = tf.keras.Input(shape=input_shape[1:])
			_ = self.call(inputs)

	class SimpleUNet1(tf.keras.Model):
		"""
		Serialise basic units so as to build up a double-layered encoder-decoder U-Net
		Input:
			inDim: (for initialisation) [modaility/channel, tensor dimensions]
			classNum: background included
			name: name for the net
			inputs: 5D tf tensor of [mbSize, modaility/channel, tensor dimensions]. Inputs must be organised into channel first order
			input_shape: a 1X5 tuple [mbSize, modaility/channel, tensor dimensions]
			ifTrain: True for training, and False for validation and testing
		Returns:
			outputs: 5D tf tensor of [mbSize, classNum, tensor dimensions]
		"""
		def __init__(self, inDim, classNum, name='SimpleUNet', **kwarg):
			super(SimpleUNet1, self).__init__(name=name, **kwarg)
			self.inDim = inDim
			self.classNum = classNum
			dimEnSt1End = np.array(inDim)[2:]-2-2
			dimEnSt2Ed = dimEnSt1End/2-2-2
			dimBridgeEnd = (dimEnSt2Ed/2-2-2)*2
			dimDEStd1End = (dimBridgeEnd-2-2)*2
			self.outDim = dimDEStd1End-2-2-2
			temp = ((dimEnSt2Ed - dimBridgeEnd)/2).astype('int32')
			crop3d1 = tuple(np.tile(temp, (2, 1)).T)
			temp = ((dimEnSt1End - dimDEStd1End)/2).astype('int32')
			crop3d2 = tuple(np.tile(temp, (2, 1)).T)

			self.en_st1_cbr1 = C3BR(32, 3, 1, 'valid')
			self.en_st1_cbr2 = C3BR(64, 3, 1, 'valid')
			self.en_st2_mp = layers.MaxPooling3D(pool_size=(2, 2, 2), strides=(2, 2, 2), padding='valid', data_format='channels_first')
			self.en_st2_cbr1 = C3BR(64, 3, 1, 'valid')
			self.en_st2_cbr2 = C3BR(128, 3, 1, 'valid')
			self.bridge_mp = layers.MaxPooling3D(pool_size=(2, 2, 2), strides=(2, 2, 2), padding='valid', data_format='channels_first')
			self.bridge_cbr1 = C3BR(128, 3, 1, 'valid')
			self.bridge_cbr2 = C3BR(256, 3, 1, 'valid')
			self.bridge_tconv1 = layers.Conv3DTranspose(256, 2, strides=2, padding='valid', data_format='channels_first')
			self.de_3dcrop1 = layers.Cropping3D(crop3d1, data_format='channels_first')
			self.de_st1_concat = layers.Concatenate(axis=1)
			self.de_st1_cbr1 = C3BR(256, 3, 1, 'valid')
			self.de_st1_cbr2 = C3BR(128, 3, 1, 'valid')
			self.de_st1_tconv1 = layers.Conv3DTranspose(128, 2, strides=2, padding='valid', data_format='channels_first')
			self.de_3dcrop2 = layers.Cropping3D(crop3d2, data_format='channels_first')
			self.de_st2_concat = layers.Concatenate(axis=1)
			self.de_st2_cbr1 = C3BR(64, 3, 1, 'valid')
			self.de_st2_cbr2 = C3BR(64, 3, 1, 'valid')
			self.final_conv3D = layers.Conv3D(filters=self.classNum, kernel_size=3, strides=1, padding='valid', data_format='channels_first')

<EMAIL_ADDRESS>		def call(self, inputs, ifTrain=True):
			x0 = self.en_st1_cbr1(inputs, ifTrain)
			xEnSt1End = self.en_st1_cbr2(x0, ifTrain)
			x1 = self.en_st2_mp(xEnSt1End)
			x2 = self.en_st2_cbr1(x1, ifTrain)
			xEnSt2Ed = self.en_st2_cbr2(x2, ifTrain)
			x3 = self.bridge_mp(xEnSt2Ed)
			x4 = self.bridge_cbr1(x3, ifTrain)
			x5 = self.bridge_cbr2(x4, ifTrain)
			xBridgeEnd = self.bridge_tconv1(x5)
			xCrop1 = self.de_3dcrop1(xEnSt2Ed)
			print(xBridgeEnd.shape)
			print(xCrop1.shape)
			x6 = self.de_st1_concat([xBridgeEnd, xCrop1])
			print(x6.shape)
			x7 = self.de_st1_cbr1(x6, ifTrain)
			x8 = self.de_st1_cbr2(x7, ifTrain)
			xDeSt1End = self.de_st1_tconv1(x8)
			xCrop2 = self.de_3dcrop2(xEnSt1End)
			x9 = self.de_st2_concat([xDeSt1End, xCrop2])
			x10 = self.de_st2_cbr1(x9, ifTrain)
			x11 = self.de_st2_cbr2(x10, ifTrain)
			x12 = self.final_conv3D(x11)
			outputs = activations.softmax(x12, axis=1)

			return outputs

		def build_model(self, input_shape):
			''' A work-around to define dimensions of signals through the NN'''
			self.build(input_shape)
			inputs = tf.keras.Input(shape=input_shape[1:])

			_ = self.call(inputs)

		def compute_output_shape(self):
			# Override this function if one expects to use the subclassed model in Kera's fit() method; Otherwise, it is optional.
			return tf.TensorShape(np.append(self.classNum, self.outDim))

Please pay attention to the following two definitions. If I use concatenate layer in this way (the C is a capitalised one), when I try and save the model, it works
	self.de_st1_concat = layers.Concatenate(axis=1)
	self.de_st2_concat = layers.Concatenate(axis=1)

For instance,
	modelInDim = (4, 64, 64, 64)
	classNum = 2
	mbSize = 2
	TUNet = SimpleUNet1(modelInDim, classNum)
	TUNet.build_model(input_shape=(mbSize,)+modelInDim)
	x=tf.random.uniform((mbSize, 4, 64, 64, 64))
	y=TUNet(x)
	TUNet.summary()
	TUNet._set_inputs(x)
	TUNet.save(r'...\TTweight', save_format='tf')

But if, as per this page I use layers.concatenate (small 'c') to generate signals x6 and x9, respectively, that is
	x6 = layers.concatenate([xBridgeEnd, xCrop1], axis=1)
	x9 = layers.concatenate([xDeSt1End, xCrop2], axis=1)

Then save the model in the same way above, it raised an error
	  ValueError: A `Concatenate` layer requires inputs with matching shapes except for the concat axis. Got inputs shapes: [(None, 256, None, None, None), (None, 128, 18, 18, 18)]

The detailed log is here
It takes me almost an entire day to figure out the root cause. In conclusion, I think the second one may have to be deprecated; otherwise users like me will be confused and misguided.

@gadagashwini @gowthamkpr
I encountered a further issue when trying to save the model under the mixed precision version. It quite confusing. I followed instructions at here
So I did the following
from tensorflow.keras.mixed_precision import experimental as mixed_precision
curOpt = tf.keras.optimizers.Adam(learning_rate=lr)
curOpt = mixed_precision.LossScaleOptimizer(curOpt, loss_scale='dynamic')
policy = mixed_precision.Policy('mixed_float16')
mixed_precision.set_policy(policy)
# query if it has been well set
print('Compute dtype: %s' % policy.compute_dtype)
print('Variable dtype: %s' % policy.variable_dtype)
from MedNN import SimpleUNet
TUNet = SimpleUNet(modelInDim, classNum)
TUNet.build_model(input_shape=(mbSize,)+modelInDim)

Then if I use a test signal x0 of tf.float32 to do the following:
x0 =tf.random.uniform((mbSize, 4, 64, 64, 64), dtype=tf.float32)
y=TUNet(x0, True)
TUNet.summary()
TUNet._set_inputs(x0, True)
# Method #1
tf.saved_model.save(TUNet, r'...\TUNet_Test')
# Method #2
TUNet.save(r'...\TUNet_Test', save_format='tf')

Whether I ran method #1 or 2, an error was raised. It seems the dtype does not match
ValueError: Python inputs incompatible with input_signature: inputs ((<tf.Tensor 'conv3d/Cast:0' shape=(None, 4, 64, 64, 64) dtype=float16>,)), input_signature ((TensorSpec(shape=(None, 4, None, None, None), dtype=tf.float32, name=None),))

Even if I cast x0 into tf.float16, the same error occurs.
So my questions are:

Is it a bug about tf's functionality of saving custom model or mixed precision?
Can I use mixed precision for custom models safely?
If it is neither, how can I fix the problem?
I am aware that multiple issues are involved here, and I might need to report them in different issues, Yet they are tightly coupled as presented above, hence could they be scrutinised here in this issue altogether? Many thanks, indeed!

At the same time, I would like to say that I surmise that loads of people tend to use custom layers, custom loops, custom models, and custom training, but tf's doc seems not quite to keep apace well. For example, in the mixed precision guide (the link above), it uses the toy example of digit recognition, then states the following:

Why can't the doc use a more explicit or articulating example to state such a delicate and important case?
The underlined sentence is quite confusing. Will the model do it automatically, or shall the user do it, for example in the call() function of a custom model?

Another issue about loading custom models:
After saving it in tf format as I mentioned in the first post, I load it by
newModel= tf.keras.models.load_model(...path)

The path contains the following

Yet tf prompts


I want To Ask Regarding SteamVr_controller Alternate

private SteamVR_Controller.Device steamDevice;
steamDevice = SteamVR_Controller.Input((int)trackedController.GetComponent<SteamVR_TrackedObject>().index);
steamDevice.GetTouchDown (SteamVR_Controller.ButtonMask.Trigger)
what will be the alternate to get i want it to know to set it up for my Multiplayer Controll Syncing I want these Lines to be Done In different Functions but I want to Have Its Alternate with new steamvr plugin.
I am anxiously waiting for reply, i hope you guys will help me out!

Unfortunately there's not a clean transition between the two systems. The new SteamVR Input System uses actions instead of button ids.
You can read the pdf documentation included with the plugin here:
https://github.com/ValveSoftware/steamvr_unity_plugin/blob/master/Assets/SteamVR/SteamVR Unity Plugin - Input System.pdf
Here's a tutorial I wrote:
https://steamcommunity.com/sharedfiles/filedetails/?id=1416820276
Here's a list of some youtube video's I've found by other people:
https://www.youtube.com/watch?v=bn8eMxBcI70 (basic)
https://www.youtube.com/watch?v=lOaR-cUGzkc (haptics)
https://www.youtube.com/watch?v=zs6uNttLh-k (vive tracker)
https://www.youtube.com/watch?v=HnzmnSqE-Bc (simple interaction)


Static resources download and references, with multiple styling fix and documentation updates

[feat] Add Reference Section for each doc

A <detail> section called "资源及外部链接" is added to each existing document, to provide references.

[doc] Polish project README.md

Project README is now up-to-date with n1cogrv/n1cogrv. note:ipkb-bookmark.png,

[fix] Fix _indexing-table.mdx rendering style in mobile devices

In mobile devices or small window size, _indexing-table will become a bit "un-readable". A mandatory width styling is added to the table via a wrapping <div>

[fix] Fix svg icon on homepage display in light/dark mode

Add a css filter attribute to
.HomepageFeatures.featureSvg[data-theme="dark"]

commit ab974c19cbecf8cf3f4f88de55f0f72ab2d262b3
[doc-update] Update administrative procedure law

[fix] fix administrative reconsideration law typo

Corresponding entries in `_indexing-table.mdx` are amended.

commit a311c7732881b4684e2aec43db5e5a35b4f386bf
[doc-update] Update administrative reconsideration law

[reference fix] Update links with static word and pdf files

[fix] homepage svg not displaying correctly in dark/light mode

[Add] Jieba dict specific for law doc

[bug]!STILL EXISTS! search function freezes pages when exceeding limit

Search function will cause the search page to freeze when searching
string is longer than 12-ish CJK characters. A workaround patch is
added. However the bug still exists, it is believed that this bug comes
from upstream dependencies.

commit caca57504bd183fa52503195ab3ae10a5ce8d1bd
[feat] Add Reference Section for each doc

A `<detail>` section called "资源及外部链接" is added to each existing
document, to provide references.

[doc] Polish project README.md

Project README is now up-to-date with n1cogrv/n1cogrv.
note:`ipkb-bookmark.png`,

[fix] Fix _indexing-table.mdx rendering style in mobile devices

In mobile devices or small window size, _indexing-table will become a
bit "un-readable". A mandatory width styling is added to the table via a
wrapping `<div>`

[fix] Fix svg icon on homepage display in light/dark mode

Add a css filter attribute to
`.HomepageFeatures.featureSvg[data-theme="dark"]`


Is there any params for RedisCluster init, to let redis know when did idle client connection should be kicked out

Hi people,
Here is my scenario, when i init about 40 RedisCluster instances(it is neccessary for me) and do a huge concurrent operation, I found a lot  idle client connections on server-side, even i stopped concurrent operation, idle connections won't be released automatically. I tried use max_connections to prevent volume of connections from explosive increasing. But, i think the connections which idle for ages should be kicked out. Is there any possible way to achieve this by setting any params(which i might know yet, or it doesn't realized yet) to RedisCluster
According to redis official document, I noticed that idle client connections won't be closed forever by default. SET TIMEOUT xxx is a good solution on server-side, to let redis know when did it need to kick off the idle client. But I am earger to have a client-side solution.
https://redis.io/docs/reference/clients/

I didn't see any realization in codes that RedisCluster that allows user to handle idle_timeout (like Jedis)
Will it be supported in the future releases? I'd appreciatie if somebody can anwser this
Best,
Yanchi

issue which my be related
https://github.com/redis/redis-py/issues/445

@yanchidezhang No there is no client setting or feature to do this in this library or to my knowledge the versions that redis-py that is supported by us here.
The main problem usually with these things is that you have to implement a threaded connection tracking or reaping feature that runs alongside the normal code flow.  This is not done by default and we use the connection recycle model that redis-py use in order to not break our clients.
You can't set a timeout that will be triggered in a non blocking or unthreaded fashion. It might be possible in the asyncio version in upstream redis-py later versions when this code base was merged into there.
Also comparing to Jedis is difficult as the core language features and how things is implemented is different between languages, things they can do there might not be possible here.
But if you really want to have something you need to sort it out with threads but be careful as you are messing around with code that is not always suitable or 110% stable in a threaded environment and you could face consequences or issues with that solution rather then just either recreating your cluster object every so often to clear things out, or just live with max_connections and let the library do and work the way it was intended to work.
One thing i can say for sure, is that i will not make any effort or put time into investigating any option into this or adding this as a feature, mostly because cluster support is not in redis-py itself and i urge people to upgrade dependencies and move up there instead

@Grokzen Thanks for your quick and comprehensive response! That quite helps.


Move the test data for file_disemvowel to the test directory

Now that I'm moving the test data for summarize_tree into a new test directory, we should probably also consider moving the test data for file_disemvowel there as well to keep things more consistent.

Should there be a testdata directory inside the test directory so that it's easy to tell what things are tests and which things are data that are used to exercise the tests? And, should the testdata directory have multiple directories that help show what they are testing? I don't know at what point it gets more confusing, but I think it's often helpful to be able to glean some information just from the structure.

At the moment there is a sub-directory for the summarize_tree, and we can make one for file_disemvowel if we decide to move that test data there.
I thought about possibly moving the test scripts into that test directory, but I figured that students would find it easier to find find/use the test scripts if they stayed in the same directory as the student code. If we replaced the fairly primitive shell tests with Aruba tests then we would probably want/need a more sophisticated test directory.
I like your idea of changing the name of that directory to testdata. Do you want me to do that and propagate the change to the UMN Github, or just leave it be for this semester?

If you would make that change, I will copy the repository again later
(probably tomorrow sometime in the evening once I've had a chance to go
through more of the code for myself and make sure I'm ready).
On Sat, Feb 14, 2015 at 8:23 PM, Nic McPhee<EMAIL_ADDRESS>wrote:

At the moment there is a sub-directory for the summarize_tree, and we can
make one for file_disemvowel if we decide to move that test data there.
I thought about possibly moving the test scripts into that test
directory, but I figured that students would find it easier to find
find/use the test scripts if they stayed in the same directory as the
student code. If we replaced the fairly primitive shell tests with Aruba
tests then we would probably want/need a more sophisticated test
directory.
I like your idea of changing the name of that directory to testdata. Do
you want me to do that and propagate the change to the UMN Github, or just
leave it be for this semester?
—
Reply to this email directly or view it on GitHub
https://github.com/UMM-CSci-Systems/Traversing-Directories-starter/issues/5#issuecomment-74401076
.

Actually, you won't have to have to re-copy the repo. A nifty big of git goodness that I've used all this lab stuff as an excuse for figuring out is the ability to set up remotes. If you clone a repo you automatically set up the original repo as a remote for your clone, which then lets you pull/push between them. You can set up your own remotes, so I can connect the "main" Github.com repo to both my solution repo and your S15 repo, and then propagate the new commit(s) to the other repos so everyone stays in sync. It's quite slick, actually, once you get your head wrapped around it.

Well, I moved it to the new place as a practice ground for me so I could
really work on the lab and push to my fork, which I wouldn't want to do in
the "real" one. I need the freedom to make mistakes and leave it a mess if
it comes to that. So, I'll be copying it again anyhow.
-KK
On Sat, Feb 14, 2015 at 9:05 PM, Nic McPhee<EMAIL_ADDRESS>wrote:

Actually, you won't have to have to re-copy the repo. A nifty big of git
goodness that I've used all this lab stuff as an excuse for figuring out is
the ability to set up remotes. If you clone a repo you automatically set up
the original repo as a remote for your clone, which then lets you pull/push
between them. You can set up your own remotes, so I can connect the "main"
Github.com repo to both my solution repo and your S15 repo, and then
propagate the new commit(s) to the other repos so everyone stays in sync.
It's quite slick, actually, once you get your head wrapped around it.
—
Reply to this email directly or view it on GitHub
https://github.com/UMM-CSci-Systems/Traversing-Directories-starter/issues/5#issuecomment-74401946
.

Gotcha. Well, I've made the change in the starter, the reference solution, and your practice copy, so I think everyone has the new directory name now.
Now to return to the initial question in this issue, do we want the file_disemvowel test data moved into that directory as well?

I think all the test data being together makes sense.
On Sat, Feb 14, 2015 at 10:10 PM, Nic McPhee<EMAIL_ADDRESS>wrote:

Gotcha. Well, I've made the change in the starter, the reference solution,
and your practice copy, so I think everyone has the new directory name now.
Now to return to the initial question in this issue, do we want the
file_disemvowel test data moved into that directory as well?
—
Reply to this email directly or view it on GitHub
https://github.com/UMM-CSci-Systems/Traversing-Directories-starter/issues/5#issuecomment-74403287
.

Cool. I'm probably not going to do that tonight (sleep beckons), but I could try to do it tomorrow. When do you need that lab ready to go public?

Closed by 197ba5118df7af5a505de11d5388fb9e7d85a86f


Use compact pagination controls for details

We should use PatternFly's compact pagination controls for each details page.
Fixes #1223

Codecov Report

Merging #1224 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1224   +/-   ##
=======================================
  Coverage   75.69%   75.69%
=======================================
  Files         231      231
  Lines        4485     4485
  Branches      872      872
=======================================
  Hits         3395     3395
  Misses        956      956
  Partials      134      134

Continue to review full report at Codecov.

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


Budget notes per block

When adding notes to the TCT, we now have a hard limit on the number of output notes that can be included in a single block: 4^8. Once the TCT is integrated as a consensus-critical part of the chain, this will cause a chain halt if violated.
We should reject transactions in check_tx_stateful if they would cause the block to go "overbudget" on outputs. Additionally, we should restrict the number of funding streams per validator to a hardcoded limit, say, 64, to prevent a DoS attack where someone could upload 10,000,000 funding streams for their validator and cause the chain to reject all other transactions. We should also probably place funding stream outputs first in a block, so that they are always definitely included, then add other transactions as possible.
Once we integrate ABCI++ it will be possible to control the inclusion of transactions in a block more precisely, and we should use this capability to allow transactions to linger to the next block in the mempool. In the current solution above, high-contention blocks may end up in a situation where the block proposer could add too many transactions, then some of them would be rejected for bringing the block overbudget, and they would need to be re-submitted.

We may not need to do this at all, depending on our block sizes.  Each output has a proof (~200bytes minimum), plus a note payload (~200 bytes), plus an encrypted memo (~600 bytes), giving about 1KB of data.  This isn't including any spends or other actions.
If blocks are capped at, say, 1MB, then they won't contain more than ~1000 outputs each, well below the maximum.

Doesn't this mean then that we will still have to budget, but for a different kind of quota? I imagine this is something we can only do effectively with ABCI++?

Yes, we need ABCI++ for this.

This came up in discussions about adding a mint queue, reminder to myself to link this issue when it gets specified.

That issue was #3545 ; we can leave it unplanned for now. Other than the genesis case, the reasoning I posted above still applies.


vue-app:serve does not reload the page

Discussed in https://github.com/paulrberg/create-eth-app/discussions/195

Originally posted by ooper-zz February 17, 2022
The background process does seem to recompile the project, but when I make edits to the page, it does not auto-reload.
I am starting it with yarn vue-app:serve.  I also tried adding --hot, but still no luck. Can you help?

Honestly I am not sure why it doesn't work, I haven't checked the Vue integration in a long, long time.
I'll review this later .. in the meantime any feedback from Vue.js experts would be welcome. Maybe it's the version of the Vue CLI that I'm using in CEA that's the culprit?
@ooper-zz does hot reload work when you upgrade to the latest version of the Vue CLI?

Thank you, Paul for replying. I believe I tried it with the latest Vue CLI, but will check again later. With that said, your React integration is perfect --definitely a good fallback for us!
Best regards, and thank you for the tool!
/Carl
Sent from my iPhone
On Feb 20, 2022, at 1:30 PM, Paul Razvan Berg @.***> wrote:

Honestly I am not sure why it doesn't work, I haven't checked the Vue integration in a long, long time.
I'll review this later .. in the meantime any feedback from Vue.js experts would be welcome. Maybe it's the version of the Vue CLI that I'm using in CEA that's the culprit?
@ooper-zzhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fooper-zz&data=04|01||426518b518b84de77d1008d9f4b83a8b|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637809894363674110|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=9k27bkbhi0WrBy3dd7VkYPumcmMLRwKVUfNcLiHs2iA%3D&reserved=0 does hot reload work when you upgrade to the latest version of the Vue CLI?
—
Reply to this email directly, view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fpaulrberg%2Fcreate-eth-app%2Fissues%2F196%23issuecomment-1046324831&data=04|01||426518b518b84de77d1008d9f4b83a8b|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637809894363674110|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=t4Ga%2BjIACTFOivynTqywWCUfuu6Oma%2B1HV3Q8nMtAok%3D&reserved=0, or unsubscribehttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAAZ473WE3GCQARI6OR3ISRLU4FMPTANCNFSM5OXAGEVA&data=04|01||426518b518b84de77d1008d9f4b83a8b|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637809894363674110|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=ZeHwzxHUARqoZMk1t3j5BoMdamsglDmMFcmzZHTqWM0%3D&reserved=0.
You are receiving this because you were mentioned.Message ID: @.***>


Basic Front-end for user input

Description:

Problem:
Initial setup for the front-end for user interaction with the services.

Solution:
the front-end initially will consist of:

homepage
cardgeneration page
card library

Prerequisites:
Currently built using node might switch to deno in future

Tasks (components for the front-end):

[ ] #66
[ ] #67
[ ] #68
[ ] #70
[ ] #69

Additional context
In future iterations we focus on:

Initial setup process where the models & datasets are downloaded (user should be able to track this progress)
Maybe we should then already test if the user's PC meets all the requirements
Here the user also needs to specify the foreign language it wants to learn for https://github.com/StephanAkkerman/FluentAI/issues/8
The creation of new vocab cards
Fill in: English word
Find nearest word in preferred foreign language (selection of top x)
Generate mnemonic sentence (ability to regenerate / adjust manually)
Generate mnemomic image (ability to regenerate / adjust parameters)
Combine them into Anki flash card and save it to deck
Creation of grammar cards
More?

The process for creating the grammar cards is linked to #7 but not yet detailed

#8 is for the languages the user wants to learn IN, for now only English is supported until #8 is done. The supported languages to learn are described in: https://github.com/StephanAkkerman/FluentAI/blob/main/supported-languages.md


rtp: cache regexes in memory for reuse

Fixes: https://github.com/moov-io/rtp20022/issues/45

Codecov Report
Attention: 2 lines in your changes are missing coverage. Please review.

Comparison is base (957ae88) 84.86% compared to head (94a6826) 85.49%.

:exclamation: Current head 94a6826 differs from pull request most recent head 36c4c28. Consider uploading reports for the commit 36c4c28 to get more accurate results

Files
Patch %
Lines

pkg/rtp/restrictions.go
83.33%
1 Missing and 1 partial :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##           master      #46      +/-   ##
==========================================
+ Coverage   84.86%   85.49%   +0.62%
==========================================
  Files          12       12
  Lines         185      193       +8
==========================================
+ Hits          157      165       +8
  Misses         24       24
  Partials        4        4

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

LGTM
Thx!


Add ability to define the respawn world for groups

MultiVerse-Inventories has a feature that allows you to choose what world players respawn in on death. This should be simple to implement here as well.

EssentialsX uses the normal priority for setting respawn locations to homes/beds. Thus, our priority will be a lower priority to allow this to still happen. If it is demanded, a config option can be added to change the event priority used.


Updating currently running apps

I'd love it if I could install the new version of Terminus from within Terminus, then simply close and reopen Terminus and have it now be the updated version. Additionally, for apps like KeepassXC and Firefox, which I have running most of the time, I'd like it to be able to install while the old version is running, and then the next time I exit and relaunch it will simply be the updated version.
Would this require Scoop to instead create additional shims that perform a headless launch of [app]/[current version]/[executable] rather than a simple symlinked current directory?

Oof, this is more of a feature request for scoop itself, i'll move it, sorry!


Extracting vales from a torch(.t7) object

I was trying to convert a torch object to numpy array, more precisely to .npy. After loading the file like this
t = torchfile.load('weights_file.t7'), I got a torchfile object. In the attached screen shots, you can see how the object looks like. How can I extract the arrays from this now ?

You can see its properties with .keys() or tab completion, like any other
dictionary. A TorchObject is a lua table emulated in Python (so that all
objects, including non-hashable ones, can be keys). If in lua you accessed
the property containing your torch tensor with obj['a']['b'][123], you'd do
the same in python to get the numpy array.

You can use something like np.array(cpt.modules[41].bias)


Update PublishDrop.cmd

This version of the script adds the following changes:
Uses setlocal EnableDelayedExpansion to enable delayed environment variable expansion, which can make variable manipulation more expensive.
Uses ! instead of % to expand variables inside loops, which can also make variable manipulation more expensive.
Uses exit /b to exit the script with a non-zero code, which can make the script fail more expensively.
Adds more output messages to the console, which can be helpful for debugging but can also make the script more expensive.

thanks.


Handle np.inf or -np.inf returned by objective function

This issue is opened because of SimonBlanke/Hyperactive#10.
The user @gr8den reported a problem concerning the case if the objective function returns np.inf or -np.inf.
After I studied those cases I came to the following conclusions:

If the score is always -np.inf then GFO will return the very first parameter set as the best, since there was never any improvement of the score during the optimization run.
GFO must be able to handle np.inf, -np.inf and np.nan
When there is no solution for the selected set of parameters the objective function should return np.nan instead of -np.inf

The first of the conclusions was solved in 79132df3 by setting the current and best position and score to the newest without checking if the score improved. This is only done in the very first iteration.

The requirement "GFO must be able to handle np.inf, -np.inf and np.nan" has been solved in a40d0ae693f7a19bd514a8dc86c524982cb5e7b0, be5baa968075beb98dceb5a3742dbc48825600e4 and 330f075ab21d2a8b850f8c68d06ccdd913a4970a. Tests have been added in 11a2b18a600933aeaf80369a87d98c3303ee5b85.
Adding this functionality was more difficult than expected. I had to add positions/score in the search tracker for non-nan and non-inf scores. Those could then be used by the hill climbing evaluation step and the samples for smbo, which cannot handle nan or inf.
This will not affect the "results"-attribute. Iterations with nan and inf will still occur in it.

I added tests for nan and inf in the warm_start_smbo parameter in cfe3d992b865568bbd674b560198dff595bc33dd
This should now cover all cases. If i find more cases to test or problems in the future i will reopen this issue.


Created card #1519 using tailwind

Pull Request Type
What kind of change does this PR introduce?
[ ] Solved an issue/task
[ ] Bugfix
[Created A card component ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[ ] Other... Please describe:
What is the current behavior?

Issue Number: [#1519 ]
What is the new behavior?
Create a Simple Card Component with Tailwind CSS
Other information

Merged

This is an automated message from Fork, Commit, Merge [BOT].

Thank you for your contribution! Your pull request has been merged. The files have been reset for the next contributor.
What's next?
If you're looking for more ways to contribute, I invite you to check out our forkcommitmerge.io website repository. The repository contain real issues that you can help to resolve or create some of your own if you see any bugs or want to add new features to the site, etc. You can also check out the Influences section in the README to find more projects similar to this one.
Also please leave a star to this project if you feel it helped you, i would really appreciate it.
I look forward to seeing your contributions!


Read up on GAN

Suggestions from Bacci

A gentle introduction of the overall principles (blog post)
Tutorial NIPS'16 (YouTube video)
Tutorial NIPS'16 (paper)

Some resources (assuming C#)

Create Any Image with C# and a GAN

Tools for making neural networks in C#

Using CNTK with C#

Common testing dataset (not for our problem domain, but just for actually making a GAN work)

CIFAR-10


Improve WebUI to show layer2 link between host

a proposal here

http://softwarefactory-project.io/r/#/c/4426/


Fix resh importer, and change error handling method

When I wrote this importer (#69) I failed to notice that one of my last changes broke it. This fixes that.

@Sciencentistguy you might to need to update #71 somehow 🤔

In my opinion its worth merging this now, and then I'll update it to use the generic trait when that is merged.

You can merge this in before, I'll update for my PR later. I still have more testing to do before hand


ImagePicker iOS compile issue

Hello and thank you for the effort put into this library!
Perhaps it's an issue on my end but after integrating calf-imagePicker into my shared module, iOS compilation fails with the following exception:
java.lang.IllegalStateException: FIELD name:com_mohamedrejeb_calf_picker_FilePickerFileType_Image$stable type:kotlin.Int visibility:public [final,static]

On Android everything works well.
And on iOS it compiles if I don't specify FilePickerFileType or FilePickerSelectionMode. So relying on default arguments works for some reason.
Any ideas as to what might be causing this?
Thanks!

Hi,
Thanks for reporting this issue. What's Calf version that you are using? Also what are your Kotlin & Compose versions?

Hello, thanks for coming back to me on this.
My versions are the following:

Calf - 0.5.5
Kotlin - 2.0.0
Compose Multiplatform - 1.6.10

I hope this information helps.
Otherwise I will spend some time and create a minimum reproducible example.

That's an issue with Compose Compiler in Kotlin 2.0.0, updating Kotlin to 2.0.10 or higher will fix it.
You can check the versions table here:
https://mohamedrejeb.github.io/Calf/installation/
https://github.com/MohamedRejeb/Calf/issues/137

Hello again, happy to report that upgrading to Kotlin 2.0.10 fixed the issue.
Thanks so much for your help, I don't even want to know how long it would have taken me to find it on my own.
Feel free to close this issue as resolved.

I faced the same issue with Kotlin 2.0.0, upgraded to 2.0.10 resolve the issue <3


Remove Find API dependence in imageloader

Right now we make a check for duplicate images using find api, this should be possible using the image loader APIs or using bindaas directly. See: https://github.com/camicroscope/Security/blob/release/imageformuploader.php#L164
Right now this dependance makes FindAPI a required part of our distribution. Our goal should be to make it optional for vanilla camicroscope distribution

we can use the API from image loader: https://github.com/camicroscope/ImageLoader#get-subjectidexists

solved w/ #31


w2ui 1.4.3. grid not displaying

I have a previous version of w2ui 1.4.3.
If i inspect the Grid is there but is hidden. I see it generates column headers but without values (captions). I wouldn't be asking but I am using the w2ui 1.4 grid in several projects and it works everywhere but here I can't figure out what is wrong I've been at it a couple of days now.. A have checked the objects(columns and rows) generated and everything but can't figure out the reason for such behaviour.  I would really appreciate if you could take a look, or at least point me in direction in which to look.
Console log is clean and there aren't any js errors. Here you can see the data object I've printed.

Here you can see what is rendered on a web page

And here you can see the html element that is rendered,
w2ui rendered html result.html.txt

The grid expects data in the JSON format from the server as it is described below. The JSON structure will be merged into the grid with jQuery.extend method. It means that you can return any supported grid property from the server and it will be applied.
{
    "status"  : "success",
    "total"   : 36,
    "records" : [
        { "recid": 1, "field-1": "value-1", ... "field-N": "value-N" },
        ...
        { "recid": N, "field-1": "value-1", ... "field-N": "value-N" }
    ]
}

The JSON structure will be merged into the grid with jQuery.extend method. It means that you can return any supported grid property from the server and it will be applied.

On a side note: even though this is still in the docs, this only applies to version 1.4 - w2ui 1.5 will no longer extend grid properties and only the following JSON properties are checked: status, total, records, summary.

Thank you

@dregos can you please create a jsFiddle with the problem.

Hi. Thank you for the insight. how do I add resource link to w2ui in jsFiddle?
However.. I've used JSON stringify. And now I get "ERROR: Cannot add records without recid. ". I then added a column field with a name of "recid" but still the same.

I've upgraded w2ui to w2ui-1.5.rc1.min.
I can't recreate my code in jsFiddle because it needs https source. And when I try to point to git source it says it is in html/text format. (Console log: Refused to execute script from 'https://raw.githubusercontent.com/vitmalina/w2ui/master/dist/w2ui-1.5.rc1.min.js' because its MIME type ('text/plain') is not executable, and strict MIME type checking is enabled.)
This is the fiddle I've prepared, but there are no resources jet. If you can help me with that. It would be great. -> https://jsfiddle.net/dregos/z3u1p6dw/

There you go: https://jsfiddle.net/z3u1p6dw/4/

Thank you mpf82.. I've done the same, but should have apparently put in the right one. Well Now that we have a working fiddle it shows the same error with recid.

Now that we have a working fiddle it shows the same error with recid.

That's because you're converting your prefectly fine records (and columns) into a string:
generatedColumns = JSON.stringify(generatedColumns);
generatedRecords = JSON.stringify(generatedRecords);

You should also give the grid an absolute height in pixels instead of a relative height in percent:
https://jsfiddle.net/z3u1p6dw/6/

Ok I probably missed somewhere something. Because as I understood the problem was exactly that. That I passed an object instead of JSON string.

Why don't the first column items render?

Ok.. It seems the problem is in column property "render".

@dregos Is your problem solved now? Can this issue be closed?

Yes. It seems ok now. I’ve forgotten how I solved it but I think it is ok.


Add a dropdown for toggling which projects get shown

The dropdown should replace the completed checkbox.  The values should be planned, completed, all

Added this to the interactive map page and to the search page.


Feature in README?

What does this mean under features?
* Handle latest submission by sorting via timestamp in file name.
    * all members of a team can submit at any point
    * last submission per team is considered (if there are multiple)
    * this is done by mapping individual submission to a team via `--team-names-file` and timestamp in zip submission file

OK this is in case a team has many zip files, timestamp in filename is used. Simplified explanation in README.


Extend plugin API to observe object lifecycle

Extend the plugin API to allow for observing an object lifecycle as it mutates in a cluster.

I'd like to unpack this a bit. First things first, we will need to figure out how to observe an object's lifecycle. This should be as simple as putting a watch on the object, and detecting changes. 1) How would we represent these changes? 2) Is this the right level for solving the problem?


[Accordion] Double click results in 8 toggles!

How can we reproduce this bug?

Go to either http://foundation.zurb.com/sites/docs/accordion.html or http://codepen.io/avr/pen/EPaXdW
Double click to open one Foundation 6.x Accordion multi-expand

What did you expect to happen?
Should toggle accordion on or off once.
What happened instead?
It slidetoggles the accordion about eight times on and off – quite crazy.

did this get fixed?


Fix handling of module log levels

Motivation
See #2090
One way to fix this would be to instantiate a new logger for each module, but this would lose the hierarchy of loggers and the prefix with node ID. Another possible fix would be to lower the parent logger log level to the level of the lowest child logger. A third possibility is to keep an array/map of loggers, one per required log level.
Changes

store an array of app "root" loggers, one per log level, and reuse them when possible
use these rather than trying to change levels of already-instantiated loggers
introduce a simpler, cleaner way to instantiate a logger (NewWithLevel)
includes some of the same changes as #2087

Notes

With level removed, there's not much need to have our own Log type in the first place. There's still some added niceties around field logging, but I think we could remove a ton of the Log logic and just use the underlying types for most, if not all, of this functionality.

Test Plan
TBD
To do

add a test
check if cmd/node.go functionality to change a module log level still works

Closes #2065
Closes #2090

Looking at this again, it's still unsatisfactory since there's literally no way to change the log level of the parent logger. Need to give this some more thought. This is tricky.

This isn't ready for review yet. Need to tweak this further.

Closing in favor of #2092


Create managed object in IDM for domestic scheduled payment consent

Please add your planning poker estimate with ZenHub @BohoCode


add ID: to the campaign urn

Can you add a property name to the urn to make it explicit?

Fixed @ https://github.com/mobilizingcs/author/commit/e756e0e69a21b0627e889c0f5fdd934f6aae4757


Tool control bar glitch after update to 8.9

This is how it looks like. It's like this for both players and GM. I have tested it with only Twilight UI module and dependencies, and the issue persists.

Has been fixed in the latest update.


Setup Error Reporting

The current version does not give that much information when a request fails. The API does give much more information in the 'message' attribute.
It would be nice if that error message is available in the exception.
Ref: #2

@FirePing32  May I work on this?

@ibrahimsha23 Sure !

@FirePing32 Thanks,
I would like to know, in local setup how will you be running the applications? Am facing some issues in my local , and I need to cross verify with you.
example:
by defining the main method(entry method) in api.py file.
python ./api.py

@FirePing32  Thanks, the above suggestions it helped me out.
I have tried hitting non-existent URLs, using google page speed apis and PyPSI. Both the responses, look the same.  Do I need to add more custom messages on exception? could you please give me a brief on what needs to be done?
Reference:

PyPSI api response:

{'error': {'code': 500, 'message': 'Lighthouse returned error: FAILED_DOCUMENT_REQUEST. Lighthouse was unable to reliably load the page you requested. Make sure you are testing the correct URL and that the server is properly responding to all requests. (Details: net::ERR_CONNECTION_FAILED)', 'errors': [{'message': 'Lighthouse returned error: FAILED_DOCUMENT_REQUEST. Lighthouse was unable to reliably load the page you requested. Make sure you are testing the correct URL and that the server is properly responding to all requests. (Details: net::ERR_CONNECTION_FAILED)', 'domain': 'lighthouse', 'reason': 'lighthouseError'}]}}

google api response.

See #2.

This is still open on gitpay


AttributeError: 'FSMap' object has no attribute 'missing_exceptions'

Apologies for not creating a simpler reproducible example. Unfortunately, I cannot reproduce the error with simple examples, only with a more realistic example involving real data and code. I hope that by sharing it here, the error message will trigger some ideas about how to resolve it.
What happened:
I am trying to generate a large xarray dataset using Dask and write to to GCS using zarr + gcsfs with a distributed dask gateway cluster.
My code looks something like this
import gcsfs
import json

with open('access_token.json') as f:
    token = json.load(f)
    gcs = gcsfs.GCSFileSystem(project='pangeo-181919', token=token)

# now do a bunch of lazy computations to build up an xarray dataset called ds_to_save
# I know these computations are not the problem because my dataset can be computed fine

gcsmap = gcs.get_mapper('/pangeo-cmip6-cape/CanESM5/historical/Convective_'+np.str(year)+'.zarr')
ds_to_save.to_zarr(gcsmap, mode='w')

This produces a long stack trace
---------------------------------------------------------------------------
AttributeError                            Traceback (most recent call last)
<ipython-input-13-35e977e44506> in <module>
    107     #gcsmap = gcsfs.GCSMap('/pangeo-cmip6-cape/CanESM5/historical/Convective_'+np.str(year)+'.zarr', gcs=gcs)
    108     gcsmap = gcs.get_mapper('/pangeo-cmip6-cape/CanESM5/historical/Convective_'+np.str(year)+'.zarr')
--> 109     ds_to_save.to_zarr(gcsmap, mode='w')
    110     end = time.time()
    111     print(end - start)

/srv/conda/envs/notebook/lib/python3.7/site-packages/xarray/core/dataset.py in to_zarr(self, store, chunk_store, mode, synchronizer, group, encoding, compute, consolidated, append_dim)
   1660             compute=compute,
   1661             consolidated=consolidated,
-> 1662             append_dim=append_dim,
   1663         )
   1664

/srv/conda/envs/notebook/lib/python3.7/site-packages/xarray/backends/api.py in to_zarr(dataset, store, chunk_store, mode, synchronizer, group, encoding, compute, consolidated, append_dim)
   1367     # TODO: figure out how to properly handle unlimited_dims
   1368     dump_to_store(dataset, zstore, writer, encoding=encoding)
-> 1369     writes = writer.sync(compute=compute)
   1370
   1371     if compute:

/srv/conda/envs/notebook/lib/python3.7/site-packages/xarray/backends/common.py in sync(self, compute)
    159                 compute=compute,
    160                 flush=True,
--> 161                 regions=self.regions,
    162             )
    163             self.sources = []

/srv/conda/envs/notebook/lib/python3.7/site-packages/dask/array/core.py in store(sources, targets, lock, regions, compute, return_stored, **kwargs)
    979
    980         if compute:
--> 981             result.compute(**kwargs)
    982             return None
    983         else:

/srv/conda/envs/notebook/lib/python3.7/site-packages/dask/base.py in compute(self, **kwargs)
    165         dask.base.compute
    166         """
--> 167         (result,) = compute(self, traverse=False, **kwargs)
    168         return result
    169

/srv/conda/envs/notebook/lib/python3.7/site-packages/dask/base.py in compute(*args, **kwargs)
    450         postcomputes.append(x.__dask_postcompute__())
    451
--> 452     results = schedule(dsk, keys, **kwargs)
    453     return repack([f(r, *a) for r, (f, a) in zip(results, postcomputes)])
    454

/srv/conda/envs/notebook/lib/python3.7/site-packages/distributed/client.py in get(self, dsk, keys, restrictions, loose_restrictions, resources, sync, asynchronous, direct, retries, priority, fifo_timeout, actors, **kwargs)
   2723                     should_rejoin = False
   2724             try:
-> 2725                 results = self.gather(packed, asynchronous=asynchronous, direct=direct)
   2726             finally:
   2727                 for f in futures.values():

/srv/conda/envs/notebook/lib/python3.7/site-packages/distributed/client.py in gather(self, futures, errors, direct, asynchronous)
   1990                 direct=direct,
   1991                 local_worker=local_worker,
-> 1992                 asynchronous=asynchronous,
   1993             )
   1994

/srv/conda/envs/notebook/lib/python3.7/site-packages/distributed/client.py in sync(self, func, asynchronous, callback_timeout, *args, **kwargs)
    831         else:
    832             return sync(
--> 833                 self.loop, func, *args, callback_timeout=callback_timeout, **kwargs
    834             )
    835

/srv/conda/envs/notebook/lib/python3.7/site-packages/distributed/utils.py in sync(loop, func, callback_timeout, *args, **kwargs)
    338     if error[0]:
    339         typ, exc, tb = error[0]
--> 340         raise exc.with_traceback(tb)
    341     else:
    342         return result[0]

/srv/conda/envs/notebook/lib/python3.7/site-packages/distributed/utils.py in f()
    322             if callback_timeout is not None:
    323                 future = asyncio.wait_for(future, callback_timeout)
--> 324             result[0] = yield future
    325         except Exception as exc:
    326             error[0] = sys.exc_info()

/srv/conda/envs/notebook/lib/python3.7/site-packages/tornado/gen.py in run(self)
    733
    734                     try:
--> 735                         value = future.result()
    736                     except Exception:
    737                         exc_info = sys.exc_info()

/srv/conda/envs/notebook/lib/python3.7/site-packages/distributed/client.py in _gather(self, futures, errors, direct, local_worker)
   1849                             exc = CancelledError(key)
   1850                         else:
-> 1851                             raise exception.with_traceback(traceback)
   1852                         raise exc
   1853                     if errors == "skip":

/srv/conda/envs/notebook/lib/python3.7/site-packages/dask/array/core.py in store_chunk()
   3722
   3723 def store_chunk(x, out, index, lock, return_stored):
-> 3724     return load_store_chunk(x, out, index, lock, return_stored, False)
   3725
   3726

/srv/conda/envs/notebook/lib/python3.7/site-packages/dask/array/core.py in load_store_chunk()
   3711     try:
   3712         if x is not None:
-> 3713             out[index] = np.asanyarray(x)
   3714         if return_stored and load_stored:
   3715             result = out[index]

/srv/conda/envs/notebook/lib/python3.7/site-packages/zarr/core.py in __setitem__()
   1113
   1114         fields, selection = pop_fields(selection)
-> 1115         self.set_basic_selection(selection, value, fields=fields)
   1116
   1117     def set_basic_selection(self, selection, value, fields=None):

/srv/conda/envs/notebook/lib/python3.7/site-packages/zarr/core.py in set_basic_selection()
   1208             return self._set_basic_selection_zd(selection, value, fields=fields)
   1209         else:
-> 1210             return self._set_basic_selection_nd(selection, value, fields=fields)
   1211
   1212     def set_orthogonal_selection(self, selection, value, fields=None):

/srv/conda/envs/notebook/lib/python3.7/site-packages/zarr/core.py in _set_basic_selection_nd()
   1499         indexer = BasicIndexer(selection, self)
   1500
-> 1501         self._set_selection(indexer, value, fields=fields)
   1502
   1503     def _set_selection(self, indexer, value, fields=None):

/srv/conda/envs/notebook/lib/python3.7/site-packages/zarr/core.py in _set_selection()
   1548
   1549             # put data
-> 1550             self._chunk_setitem(chunk_coords, chunk_selection, chunk_value, fields=fields)
   1551
   1552     def _chunk_getitem(self, chunk_coords, chunk_selection, out, out_selection,

/srv/conda/envs/notebook/lib/python3.7/site-packages/zarr/core.py in _chunk_setitem()
   1663         with lock:
   1664             self._chunk_setitem_nosync(chunk_coords, chunk_selection, value,
-> 1665                                        fields=fields)
   1666
   1667     def _chunk_setitem_nosync(self, chunk_coords, chunk_selection, value, fields=None):

/srv/conda/envs/notebook/lib/python3.7/site-packages/zarr/core.py in _chunk_setitem_nosync()
   1693
   1694                 # obtain compressed data for chunk
-> 1695                 cdata = self.chunk_store[ckey]
   1696
   1697             except KeyError:

/srv/conda/envs/notebook/lib/python3.7/site-packages/fsspec/mapping.py in __getitem__()
    131         try:
    132             result = self.fs.cat(k)
--> 133         except self.missing_exceptions:
    134             if default is not None:
    135                 return default

AttributeError: 'FSMap' object has no attribute 'missing_exceptions'

By monitoring the dashboard, I can see that this error comes at the very end of a long, successful computation.
I can verify that the gcsmap object DOES have these attributes:
print(gcsmap.missing_exceptions)
# -> (FileNotFoundError, IsADirectoryError, NotADirectoryError)

So this feels like a serialization problem. Also an internal problem with fsspec.
I would appreciate any advice you may have.
Environment:
This is in the latest pangeo-notebook environment.

Fsspec version: 0.7.1
Dask version: 2.30.0
Python version: 3.7.8
Operating System: linux
Install method (conda, pip, source): conda

Also worth noting that this exact workflow used to work fine. So possibly related to some recent changes?

I believe this is fixed in fsspec 0.8.4 https://github.com/intake/filesystem_spec/commit/8fc099dac4f4427f2eb8971e8efc7ede8756a7b2


Interrupting a task with Ctrl+C ?

Is it possible to interrupt a running task with ctrl+c ?

It should be. What happens currently? (and what terminal pgm are you using?)

I tried this on my Ubuntu box:
cash $ cat /dev/zero

Several ^C's couldn't stop it. It froze up my machine, so I couldn't even issue an external kill command, so I had to reboot. ^C does the trick on Bash though.

I tried grunt which started a server.
Several ^Cdid nothing at all while on a normal cmd prompt (on Windows 10) it allows me to stop current running process.

Upvoting this issue.
Over all, good job guys! I'm excited to see where this goes.


Silence form escapes backslashes, leading to double-escaping and thus unintended regexp's

What did you do?
Created a silence with a regexp matcher that needs to match a literal ., which thus needs escaping.
What did you expect to see?
A regexp matcher matching the literal ..
What did you see instead? Under which circumstances?
The regexp got double-escaped, i.e. instead of foo\.bar, it became foo\\.bar, and thus matching a literal \ followed by an arbitrary character.
Looking at the HTTP request sent by the UI, I can see the double-escaped parameter as foo%5C%5C.bar.
This makes it impossible to create silence regexp's matching literal . and similar.
Interestingly, creating a regexp matcher as a filter for the alerts page works just fine. Thus, I assume that's a subtle difference in how the UI is set up for the silence form vs. the matchers on the alerts page.
@stuartnelson3 was a bit puzzled by it. @mxinden and @w0rm , perhaps your Elm foo has something to say here?
Environment

System information:
Linux 4.4.10+soundcloud x86_64

Alertmanager version:
alertmanager, version 0.15.2 (branch: HEAD, revision: d19fae3bae451940b8470abb680cfdd59bfa7cfa)
build user:       root@3101e5b68a55
build date:       20180814-10:53:39
go version:       go1.10.3

I spent more time looking at this and have no idea what's going on. As soon as a user enters \ in the matcher value field, it is being sent to UpdateMatcherValue as \\. As far as I can tell, this appears to be getting added before we have any access to the value.
@w0rm any ideas?

I could not reproduce this locally, only in our production instance. The matcher is displayed with two backslashes for some reason, but when I click it, or edit the silence, it is a single backslash.
API returns a single backslash too, so I think the issue is only when rendering a matcher button when viewing a silence.

This is how it looks locally, using the latest alertmanager from the master branch:


Bug of BGE M3 training

one line cause bug in training bge-m3 with training config:
--knowledge_distillation True \
--unified_training False \

because sparse_vecs = None

Thank you very much for your correction.


Improve GPU BW throughput of transpose by vectorization

This PR improves the GPU BW throughput of transpose op (specifically the ShuffleInTensor3Simple CUDA kernel) by vectorization.
The benchmark code:
https://github.com/kaixih/kernel_tests_public/blob/master/transpose/transpose_tf2_bench.py
Performance comparison on V100 GPU before/after this change:
https://docs.google.com/spreadsheets/d/1N2kxw8DYQP8FGbZcORCmj7WUVY7Dh6xISuQR0DruG5Q/edit?usp=sharing
FYI @nluehr

Updated. PTAL. @chsigg

@chsigg Any more changes that I could do for this PR?


Improve minimizes and mutates

Trie minimize seems to produce solutions of the same value during the minimize mode, and I'm pondering whether that means we don't need to try the trie search for every solution we generate. In particular, if we put all the solutions into the trie already, then all the trie_minimize calls should produce the same result, because they start at the same state. So we should be able to run trie_minimize from one solution instead of all of them.
Additionally, I think we ought to be able to repeatedly call the mutate functions on a solution until we stop improving it, instead of serializing into a list of many disjoint single improvements.

We might also find a way to coalesce the mutate functions together again so that we only generate one full collection history. However, the majority use collection_history_with_range_info which doesn't create state objects.

With some examination in gdb, I see that it's not really too far behind, though it's still working on reducing its first solution of course. In particular it seems to have gotten stuck in recreate_store and in particular it seems like record_one_internal is the main culprit--which makes me think that because this is an improvement to an early state that we are spending lots of time fixing all the downstream states, which can get enormous over time. This explains a lot of stuff I've been concerned about, which means it's time for a new issue.


Sintax error for EscapeString function when alert is created

Provider Version
0.63.0
Terraform Version:
v1.4.6
Describe the bug
if you try to create an alert with some condition that requires single quotes it fails due to syntax error
Expected behavior
The alert should be created with conditions using string values (single quotes)
Code samples and commands
resource "snowflake_alert" "alter_space_usage" {
provider   = snowflake.my_snowflake_provider
database   = "my_db"
schema     = "my_schema"
name       = "my_alert"
warehouse  = "my_wh"
alert_schedule {
interval = 120
}
condition = "select c1, c2 from t1 where c1 = 'value'"
action = "insert into t2 select c1, c2 from t1 where c1 = 'value'"
enabled = true
}
Additional context
The problem is in this file
https://github.com/Snowflake-Labs/terraform-provider-snowflake/blob/99f7621cad1aafa83f96a4ed52d0fd04d6dfb2ae/pkg/snowflake/alert.go#LL130C2-L131C70
--- pkg/snowflake/alert.go
+++ pkg/snowflake/alert.go.new
@@ -127,8 +127,8 @@
 		q.WriteString(fmt.Sprintf(` COMMENT = '%v'`, EscapeString(builder.comment)))
 	}

-	q.WriteString(fmt.Sprintf(` IF (EXISTS ( %v ))`, EscapeString(builder.condition)))
-	q.WriteString(fmt.Sprintf(` THEN %v`, EscapeString(builder.action)))
+	q.WriteString(fmt.Sprintf(` IF (EXISTS ( %v ))`, builder.condition))
+	q.WriteString(fmt.Sprintf(` THEN %v`, builder.action))

 	return q.String()
 }

Commenting for posterity: As an immediate workaround the $$ quote form for string literals can be used in place of ' characters: https://docs.snowflake.com/en/sql-reference/data-types-text#label-dollar-quoted-string-constants
  resource "snowflake_alert" "alter_space_usage" {
    …
-     condition = "select c1, c2 from t1 where c1 = 'value'"
+     condition = "select c1, c2 from t1 where c1 = $$value$$"
-     action = "insert into t2 select c1, c2 from t1 where c1 = 'value'"
+     action = "insert into t2 select c1, c2 from t1 where c1 = $$value$$"
  …
}


Text Field: error state displayed inconsistently when required set to true

What Angular MDC version are you using?
0.42.0
What browser(s) is this bug affecting?
Chrome 71
What OS are you using?
Windows 10
What are the steps to reproduce the bug?
This issue can be reproduced using the first demo in the Text Field documentation. It is important that the required attribute is set to true.

Type the value "test" in the text field
Click on another control so the text field loses focus
Click back on the text field, so that the cursor is at the end of the value
Hit backspace twice
The text field will correctly display the validation error.
Hit backspace once
The text field will no longer display the validation error.

What is the expected behavior?
It is expected that the text field will continue to display the validation error, since the value is below the minimum length of 3.
What is the actual behavior?
The text field no longer displays the validation error.
Any other information you believe would be useful?
I did a little bit of investigation in to the code. It looks like this is caused by the _handleValidationAttributeChange() method when it is called by the setValue() method. The _handleValidationAttributeChange() calls the handleValidationAttributeChange() method on the text field foundation and passes in an array of all attributes that have been set, including required. When MDC Web is told that the required attribute has changed, it removes the mdc-text-field--invalid class from the element by directly manipulating the DOM. It seems to do this whenever the text field value is changed.

@ddaspit Thanks for reporting and the investigative work! 💎  I'll take a look.

PR submitted.

Fixed, merged with #1596


Upload file with multer

When I use with only restify
I use:
server.post('/upload', uploads.single('foto'), async (req, res, next)=>{

      try {
          res.send( saveObj  )
      } catch (error) {
          res.send(error)
      }
      next()
  })

But if I use with restify-router, I get this wrong

Argument of type 'RequestHandler' is not assignable to parameter of type 'RequestHandlerType'.
Type 'RequestHandler' is not assignable to type 'RequestHandler'.
Types of parameters 'req' and 'req' are incompatible.
Type 'Request' is missing the following properties from type 'Request<ParamsDictionary, any, any>': get, acceptsCharsets, acceptsEncodings, acceptsLanguages, and 20 more.ts(2345)

Is possible to do upload with restify-router?

@cbcarlos07 could you test this with the latest version? @otaviotech added some declarations to support this.

@cbcarlos07 could you test this with the latest version? @otaviotech added some declarations to support this.

Of Multer or restify-router?

of restify-router


Resolving issue #37

adding the missing StdErr  class
adding the Stdio class to ease outputting the stream to Standard Output and Standard Error stream

StdErr is now integrated into 3.0 http://climate.thephpleague.com/output/


Add docs for JSON fragment extensions

Turns out the build service for our docs site is down, and we need the JSON fragment extensions documented for the 1.7 release. This is the same file I have set up to go into our docs site once things get up and running again.

@msftbot make sure @PankajBhojwani signs off on this

If the docs repo gets their build pipeline up and running again, we probably don't have to maintain this doc here as well, since this is the same doc I have lined up for our docs site. We can hold on this until we release next week and see if their build service gets fixed.

Looks like our docs have been published on our docs site, so we should keep everything over there. :)
https://docs.microsoft.com/windows/terminal/json-fragment-extensions
Closing this out.


Plugin version 4.3.1 does not work correctly when request type is selected: text

After updating Grafana-Zabbix plugin to 4.3.1, item queries are no longer limited to the selected application.
In a query, selecting a Group, Host then clicking the application field is empty.
In a query, selecting a Group, Host, Application then clicking the item field -  all items associated with the host are presented randomly.

Software versions

Grafana
Zabbix
Grafana-Zabbix Plugin

8.3.3
6.4.0
4.3.1

Same here, any updates on this?


-> 1.7.0-RC9

@confluentinc It looks like @jliunyu just signed our Contributor License Agreement. :+1:
Always at your service,
clabot


Is unlock working here?

Is unlock example working in this repository? I'm working on similar implementation elsewhere using PortalWallet lock but can't make it work.
I see this repo is skipping block_proof in unlock function for witnessargs. Without it I get Error::InvalidArgs from withdrawLock (error code 4).
If I add block_proof ('0x') I get Error::ProofNotFound, error code 11
Here's my GodwokenWithdrawBuilder: https://github.com/jordanmack/ckb-tools/blob/63efb2259a050640d13d204c1a258d06f4724825/src/builders/GodwokenUnlockBuilder.ts

which block_proof in witness args ?

which block_proof in witness args ?

https://github.com/nervosnetwork/godwoken-examples/blob/develop/packages/tools/src/godwoken-cli.ts#L409 - is this needed @classicalliu?

which block_proof in witness args ?

https://github.com/nervosnetwork/godwoken-examples/blob/develop/packages/tools/src/godwoken-cli.ts#L409 - is this needed @classicalliu?

It's outdated.
https://github.com/nervosnetwork/godwoken/blob/v0.7.0-rc1/crates/types/schemas/godwoken.mol#L205-L206
block_proof has been removed.
If you need unlock, refer to this method
https://github.com/nervosnetwork/godwoken-examples/blob/develop/packages/tools/src/account/unlock.ts#L69

@classicalliu
Thank you for clarification. I figured out my problem. I was including StateValidator Cell Dep instead of latest Rollup Cell dependency (that changes once per minute).
Example unlock transaction showing funds nicely returning to Layer 1 account locked by Portal Wallet lock: https://explorer.nervos.org/aggron/transaction/0x13edfac8d9e13533db6b7939c6df54feede4904d4bd46834d77ff7b60687d7e2


Some emails sent from IBM Notes not parsed

Hi,
It could be caused by some not printable characters, but I'm not sure.
Raw contents of the email:
problem-email-original.txt
Thanks

Hmmmm, that's an interesting way of splitting boundary in the Content-Type header's boundary value.  Because it's in double-quotes, I wonder if I may be considering that a "literal" space or how I'm handling that.  Apart from that the message looks normal enough to me...
Can you tell me what you're expecting and how it's failling?  Is it throwing out an error, or just considering the whole document one part, or what's happening with it?  The part of the code that's failing e.g. if you're calling "getTextContent" or something could be helpful too :blush:
Thanks

It doesn't return content. Emails are fetched with empty body. No error thrown.
I've encountered with boudary values containing line breaks before, when utilized zend mail library. I remember there was a problem with parcing too.

Thanks @yurikuzn -- this highlighted an issue with parsing quoted header parts.  My implementation considered the new line to be a valid part of the quoted part and kept it, but should actually be ignored  based on the example here: https://tools.ietf.org/html/rfc822#section-3.1.1 and here https://www.w3.org/Protocols/rfc1341/7_2_Multipart.html in 7.2.1.
I've pushed a fix to 'master'.  Please let me know if you notice any issues... otherwise I'll push a new release version in a couple of days :blush:

Thank you


WIP Runs e2e tests for each commit on the latest release branch

K8S-9688
Description
Runs e2e tests for each commit on the latest release branch. Tests run on latest supported Kubernetes and OpenShift versions.
How can this be tested?
Use forked repo. Change Run in test step of run-in-k8s and run-in-ocp

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 64.88%. Comparing base (5502720) to head (74b36ad).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main    #3691   +/-   ##
=======================================
  Coverage   64.88%   64.88%
=======================================
  Files         386      386
  Lines       21117    21117
=======================================
  Hits        13702    13702
  Misses       6098     6098
  Partials     1317     1317

Flag
Coverage Δ

unittests
64.88% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

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


FPC 1.0 Release finalization

Description
Steps to be done:

[x] create branch release-1.0 (as in fabric, pointing to major.minor , excluding debug level, with future debug releases 1.0.x also pushed onto this branch!)
[x] create commit with

[x] in config.mk change FPC_VERSION from master to v1.0.0 (i.e., fully qualified semantic version including debug level)
[x] in README.md add item to the release section (maybe also add to that section a link to https://github.com/hyperledger-labs/fabric-private-chaincode/releases as a substitute of the missing CHANGELOG ?)
[x] replace all github source references mentioning  master in docu  to equivalent release-1.0 (i.e., referencing branch and minor version, not tag and debug version!); find files via  egrep -r 'https://github.com/hyperledger-labs/fabric-private-chaincode/blob/master' ...

[x] on github create new release with tag v1.0.0 and description pointing out first e2e secure version, reference to RFC, etc etc
[x] announce on rocket chat & Fabric mailing list

Link to feature or bug
Link to dependent issues
Obviously relies on all other issues from FPC 1.0 milestone closed or moved to later milestones

The release was actually renamed to 1.0-RC1, in order to keep the 1.0 version for when FPC joins an official Fabric repo.
Tasks have been completed.


fix: TCP connections prematurely dispose of the underlying Socket without creating a new TcpClient

This is a proposed fix for https://github.com/jacqueskang/IpcServiceFramework/issues/158

Not an expert on how this is supposed to look so open to implementation suggestions, but this resolved the 100% repro of the ObjectDisposedException and seems reliable.

This still seems a bit unreliable, though it is better than before. I'm going to keep looking for a better solution.


doc: translate  /about pages

Add Chinese translation of /about

@myitcv
When you review the translations. Please pay more attention on following that I'm not sure about.

Schema: 规范 . At page start, I give more information about our discuss.
Push, not pull, constraints: 主动而不是被动使用约束. I think of push as initiative and pull as passive.

When you review the translations. Please pay more attention on following that I'm not sure about.

Will do!
Just back from holiday so catching up on things.