model prediction fail

I can fit my model normally, but when I try to predict I get this error:
[11:24:44] /net/noble/vol1/home/jmschr/mxnet/dmlc-core/include/dmlc/logging.h:235: [11:24:44] src/operator/./cudnn_batch_norm-inl.h:138: Check failed: (cudnnBatchNormalizationForwardInference(s->dnn_handle_, CUDNN_BATCHNORM_SPATIAL, &a, &b, io_desc_, x.dptr_, io_desc_, y.dptr_, mean_desc_, gamma.dptr_, beta.dptr_, moving_mean.dptr_, moving_inv_var.dptr_, param_.eps)) == (CUDNN_STATUS_SUCCESS)
[11:24:44] /net/noble/vol1/home/jmschr/mxnet/dmlc-core/include/dmlc/logging.h:235: [11:24:44] src/engine/./threaded_engine.h:306: [11:24:44] src/operator/./cudnn_batch_norm-inl.h:138: Check failed: (cudnnBatchNormalizationForwardInference(s->dnn_handle_, CUDNN_BATCHNORM_SPATIAL, &a, &b, io_desc_, x.dptr_, io_desc_, y.dptr_, mean_desc_, gamma.dptr_, beta.dptr_, moving_mean.dptr_, moving_inv_var.dptr_, param_.eps)) == (CUDNN_STATUS_SUCCESS)
An fatal error occurred in asynchronous engine operation. If you do not know what caused this error, you can try set environment variable MXNET_ENGINE_TYPE to NaiveEngine and run with debugger (i.e. gdb). This will force all operations to be synchronous and backtrace will give you the series of calls that lead to this error. Remember to set MXNET_ENGINE_TYPE back to empty after debugging.
terminate called after throwing an instance of 'dmlc::Error'
  what():  [11:24:44] src/engine/./threaded_engine.h:306: [11:24:44] src/operator/./cudnn_batch_norm-inl.h:138: Check failed: (cudnnBatchNormalizationForwardInference(s->dnn_handle_, CUDNN_BATCHNORM_SPATIAL, &a, &b, io_desc_, x.dptr_, io_desc_, y.dptr_, mean_desc_, gamma.dptr_, beta.dptr_, moving_mean.dptr_, moving_inv_var.dptr_, param_.eps)) == (CUDNN_STATUS_SUCCESS)
An fatal error occurred in asynchronous engine operation. If you do not know what caused this error, you can try set environment variable MXNET_ENGINE_TYPE to NaiveEngine and run with debugger (i.e. gdb). This will force all operations to be synchronous and backtrace will give you the series of calls that lead to this error. Remember to set MXNET_ENGINE_TYPE back to empty after debugging.

This is a very new error for me as I've been able to make predictions recently. I'm using cuDNN v5, cuda v5.0. The last time I pulled was a few weeks ago, but I've been able to run it successfully since then. Any idea what the issue could be?

The code looks basically like this:
	X_train = TrainingGenerator(train_sequence, train_dnases, train_contacts, train_regions, window, data_shapes, coord, batch_size)
	X_eval = ValidationGenerator(valid_sequence, valid_dnases, valid_contacts, valid_regions, window, data_shapes, coord, batch_size)

	model = Rambutan( use_coord=coord,
		              ctx=[mx.gpu(0), mx.gpu(1), mx.gpu(2), mx.gpu(3)],
					  epoch_size=5000,
					  num_epoch=1,
					  learning_rate=0.01,
					  wd=0.000,
					  optimizer='adam'
					)

	model.fit( X=X_train,
			   eval_metric=['acc', roc_auc_score],
			   batch_end_callback=mx.callback.Speedometer(batch_size)
			)

	y = model.predict(X_eval)[:,1]
	numpy.save('rambutan-q={}.npy'.format(q), y)

If I put eval_data = X_eval into the fit step, it seems to work. The issue is the predict step.

is cudnn compatible with cuda5.0?

so it fails at this line cudnn_batch_norm-inl.h:138: Check failed: (cudnnBatchNorma
I would try printing out the return value ( currently it only compare it agains success) and look it up in cudnn's manual

Check your eps value if it is within cudnn limit.
I just got an error alike on line 123 when I'm trying resnet in example. I print out the return value of CUDNN_STATUS_BAD_PARAM. So I add it a little bit(1e-9), now it works.

This may also caused by a large batch_size (although the memory usage may be quite low), try a smaller one.


Audio device gets recreated on every USB connection

Even if usbaudio is not running or anything, now an audio device is created on my system everytime I plug in my phone. Audio gets forwarded to here as normal, but I'd expect it to not create the device when not asked, just when using usbaudio. Well, I find some cases where it'd be convenient to just use this, but I also want to know a way to make this not happen.
This happens even after reboots.

This is not expected.
USBaudio just sends a request to change the mode: https://source.android.com/devices/accessories/aoa2
But then, it should be "reset" on unplug.
What is the result of lsusb for your device?

It was just "Android", not my phone model as normal without USB audio.
Well, I had to change it manually from the Developer options, as it wasn't getting reset on unplug/reboot, and I had an issue where it was not making any sound on the speakers even if unplugged, being that the reason.

Normally:
Bus 002 Device 052: ID 1bbb:903a T & A Mobile Phones

With USB audio (it was always getting like that, and I noticed the input in PulseAudio):
Bus 002 Device 053: ID 18d1:2d03 Google Inc.


fix: read-only styles for textArea and richtext

Description
This

Makes sure that textArea is grayed out if it's read-only (currently, it's not)
Makes sure that richText uses --theme-elevation-200 instead of --theme-elevation-150 if it's read-only, to be more consistent with the text and textArea fields. Additionally, I gave it a color: var(--theme-elevation-450); as text and textArea are using that too.

[X] I have read and understand the CONTRIBUTING.md document in this repository

Type of change

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

Checklist:

[ ] I have added tests that prove my fix is effective or that my feature works
[ ] Existing test suite passes locally with my changes
[ ] I have made corresponding changes to the documentation

Hey @AlessioGr - thanks for doing this! Just went ahead and reviewed this - looks good! Going to merge.


Erperall/features/acr terraform module

All Submissions:

[YES] Have you followed the guidelines in our Contributing document?
[YES] Have you added an explanation of what your changes do and why you'd like us to include them?
[YES] I have updated the documentation accordingly.
[NA] I have added tests to cover my changes.
[NA] All new and existing tests passed.
[YES] My code follows the code style of this project.
[YES] I ran lint checks locally prior to submission.
[YES] Have you checked to ensure there aren't other open Pull Requests for the same update/change?

What is the current behavior?

The ability to provision an instance of Azure Container Registry does not exist.
Issue Number: #149
What is the new behavior?

Users can now use Terraform to configure and provision an instance of Azure Container Registry.
Does this introduce a breaking change?

[NO]

I'm going to push my changes so as not to forget them, and will work on rectifying the build error afterwards.

@erikschlegel I'm able to run this locally without encountering the error. Can you send me the error code from the pipeline, or give me access to view it?

@erikschlegel Thank you for the kind words and for approving the PR! I've merged it into master.


Move sample gluster service and endpoint files location.

Signed-off-by: Humble Chirammal<EMAIL_ADDRESS>

This change is

LGTM


Cover set position available?

Is there a reason why support_position returns False, while you do have an implementation for the set_position method? I was hoping to get set position working from home assistant.
https://github.com/Cereal2nd/velbus-aio/blob/7fb693528d429661b804f8ed463da8f66ddfcef1/velbusaio/channels.py#L198
https://github.com/Cereal2nd/velbus-aio/blob/7fb693528d429661b804f8ed463da8f66ddfcef1/velbusaio/channels.py#L219

it depends on the modules, some modules do support it, others don't


design the streaming json

https://github.com/olpa/ailets/blob/master/docs/technical-thoughts.md

Maybe this library: https://github.com/pydantic/jiter, via: https://til.simonwillison.net/llms/streaming-llm-apis


Less boilerplate for operator ==

Our current best-practice for operator == is:
  @override
  bool operator ==(Object other) {
    if (other.runtimeType != runtimeType)
      return false;
    final Foo typedOther = other;
    return typedOther.bar == bar
        && typedOther.baz == baz
        && typedOther.quux == quux;
  }

It's unfortunate that we have to introduce a new variable here. It would be nice if this worked:
  @override
  bool operator ==(Object other) {
    if (other.runtimeType != runtimeType)
      return false;
    assert(other is Foo); // compiler promotes other to Foo
    return other.bar == bar
        && other.baz == baz
        && other.quux == quux;
  }

...or maybe with new syntax:
  @override
  bool operator ==(Object other) {
    if (other.runtimeType != runtimeType)
      return false;
    promote other as Foo; // equivalent of: if (other is! Foo) throw TypeError();
    return other.bar == bar
        && other.baz == baz
        && other.quux == quux;
  }

...or even:
  @override
  bool operator ==(Object other) {
    if (other.runtimeType != runtimeType) // compiler knows this being false means other is Foo
      return false;
    return other.bar == bar
        && other.baz == baz
        && other.quux == quux;
  }

See also:

https://github.com/dart-lang/sdk/issues/35524, which is about the general problem of which this is a specific case
https://github.com/dart-lang/sdk/issues/26703, which is about providing a way to shorthand creating these methods entirely.

I understand that the focus is on equality here, but promotion seems to be an important topic as well.
For that, note that https://github.com/dart-lang/sdk/issues/25565#issuecomment-421914603 lists a number of other issues targeting promotion improvements, and #25565 is in general a good hub for keeping discussions on that topic together.
In the language team we have discussed using an expression statement like other as Foo; to promote other to Foo, and that idea is certainly relevant to the discussion in #25565 (and to proposals like this one).
The last example using other.runtimeType != runtimeType is a smart trick, and it looks really attractive that we might get rid of the type test altogether! ;-)
However, the test other.runtimeType != runtimeType cannot soundly show anything about the type of other, because any class (both that of this and that of other) could override runtimeType to return whatever it wants. We could just ignore soundness and trust the result (and generate dynamic checks to ensure that the heap remains sound at run time), but that would be inconsistent with all recent developments in the definition of Dart. We could make it safe by introducing a Type.runtimeTypeOf(Object) static method, which would be guaranteed to return the actual run-time type of its argument. But even with that, it would still be a somewhat odd and ad-hoc promotion, because subtypes of the run-time type of this would be rejected, even though they would succeed at an as Foo or is Foo test.

We could also just ban overrides of runtimeType. Combine this with proposals elsewhere to add post-fix "fake methods" like .as, and define .runtimeType as one of these fake methods. Or combine it with the proposal to introduce sealed methods, and seal it. Or just special case runtimeType.

Can we update the first post from "Our current best-practice" to "Flutter's current best-practice"? Using runtimeType for equality checks is not a universal best practice.
I do think it would be worthwhile to explore disallowing overrides of runtimeType.

Going further, Flutter SDK's best practice. Most Flutter users do not use this pattern.

Would it not be valid to write the following? Short-circuiting the type-check
@override
  bool operator ==(other) =>
        other is ThisClass
        && other.bar == bar
        && typedOther.baz == baz
        && typedOther.quux == quux;
  }

Using is in operator == is usually wrong as it makes the == operator asymmetric.

Using is in operator == is usually wrong as it makes the == operator asymmetric.
Was my example asymmetric? I think this is the kind of code most people will want to write, now I'm doubting it. :)

consider
class ThatClass extends ThisClass {
  ThatClass(this.bob, Bar bar, Baz baz, Quux quux) : super(bar, baz, quux);
  final Bob bob;
  operator ==(other) =>
    other is ThatClass
    && other.bob == bob
    && super==(other);
}
// ...
print(ThatClass(a, b, c, d) == ThisClass(b, c, d)); // false, good
print(ThisClass(b, c, d) == ThatClass(a, b, c, d)); // true, oops

consider
class ThatClass extends ThisClass {
  ThatClass(this.bob, Bar bar, Baz baz, Quux quux) : super(bar, baz, quux);
  final Bob bob;
  operator ==(other) =>
    other is ThatClass
    && other.bob == bob
    && super==(other);
}
// ...
print(ThatClass(a, b, c, d) == ThisClass(b, c, d)); // false, good
print(ThisClass(b, c, d) == ThatClass(a, b, c, d)); // true, oops


When viewing stops, changing route in route-selection dropdown causes error

 Issue by demory
Tuesday May 01, 2018 at 22:50 GMT
Originally opened as https://github.com/catalogueglobal/datatools-ui/issues/87

Go to feed version https://gtfs-manager-dev.conveyal.com/feed/f317c85a-1258-4c89-981b-47737c29cc3a
Select 'Routes'
Select the first route ("Aub 1..")
Select 'View Stops' for the first pattern ("34 stops from...")
In 'Filtering Stops for Pattern' dropdown, try switching to the 2nd pattern
[ERROR]

 Comment by landonreed
Tuesday May 01, 2018 at 23:34 GMT

This appears to be an issue with the leaflet map where a StopMarker that failed to mount is being updated: https://reactjs.org/docs/error-decoder.html?invariant=136&args[]=StopMarker


Add Support for shared librairies.

Hello. Tested on binary it work perfect, great tools!
But when i test on shared libraries started with another binary, after connecting to the server, the tool won't work.
(no decompilation, breakpoints have offset errors) .
I know original behavior is to start the server on binary and use the connect while debugging the binary with gdb. But in my situation I can't debug the libraries without starting the linked main binary before.
(Maybe adding muliple server syncing ? Decomp2dbg can't export ida decompiled code on gdb because shared librairies is extern but maybe adding another syncing server on shared lib IDA instance + syncing correctly when jumping on shared lib can work)

Hi @0xMirasio thanks for using d2d!
If I am getting this right, the use case you are talking about is one where you want both decompilation output inside the main binary being debugged and inside the shared libraries it uses, right?
As a solution to this scenario, would you be willing to open another instance of IDA, start decomp2dbg on there, an then connect it as well? Currently, we have the ability to load multiple decompiler outputs into one debugger but the problem becomes the main base address which all things are based off.
I think the solution to this problem will be:

Full normal support for multi-decompiler connections
Kill decompiler panes on bad output (so it always looks like 1 in case of libs)
Add optional args for specifying a forced base address for decompiler loads

Hello, thank for your response.
Indeed, i tried to start the server on the shared lib then connect when i jumped to shared code on gdb. But no decompilation unfortunately. Also, offset are broken (that's normal). If i understand correctly, the software try to use ida.decompile() at 0x0x7ffffXXXXxxX adress but won't find it in shared lib .text segment mapped to X -Y . Maybe i can try to add option in GUI to ask for a base adress for this problem ?

@0xMirasio alright, so here is the plan:
For now, so you can use it with only 1 binary at a time, I've started this PR: https://github.com/mahaloz/decomp2dbg/pull/42
You will now be able to configure the base address we load symbols at like so:
decompiler connect ida --base-addr 0x7ffff0000000

@0xMirasio alright, the PR is just about ready to land. Would you be able to checkout into that branch and play with it a little? I tested it with only connecting and IDA instance that had libc decompiled, then I mapped it to the virtual address space and it worked. In this approach we can still only have 1 ida open at a time and connected.
decompiler connect ida --base-addr 0x00007ffff7452000

then break in libc_start_main or something.
Still need to make it not freak out when you are in an address space outside its known addrs.

@0xMirasio actually, its in master now; use the command like the README shows here:
https://github.com/mahaloz/decomp2dbg#advanced-usage
If you find any bugs let me know, and if you are happy with the new feature close the issue :)

Hi again @mahaloz .
Seem there is still problems.
Here are some details :
vmmap-> 0x7ffff7800000     0x7ffff7972000 r-xp   172000 0      /opt/quest/lib64/libvtwrap.so.2.0.0

_parse_special_tag function -> 0x00007ffff78d96f0

d2d server start on libvtwrap on port 3661
gdb args :  decompiler connect ida --host <IP_ADDRESS> --port 3661 --base-addr-start 0x7ffff7800000 --base-addr-end 0x7ffff7972000
Output of gdb when step in:
[+] Connected to decompiler!
pwndbg> si
0x00007ffff78d96f5 in std::_Sp_counted_ptr<vintela::AsdAuthClient*, (__gnu_cxx::_Lock_policy)2>::_M_destroy() () from /opt/quest/lib64/libvtwrap.so.2.0.0
[*] NO DECOMP PRESENT
Traceback (most recent call last):
  File "/opt/tools/pwndbg/pwndbg/commands/__init__.py", line 131, in __call__
    return self.function(*args, **kwargs)
  File "/opt/tools/pwndbg/pwndbg/commands/__init__.py", line 225, in _OnlyWhenRunning
    return function(*a, **kw)
  File "/opt/tools/pwndbg/pwndbg/commands/context.py", line 270, in context
    result[target].extend(func(target=out,
  File "/home/thibault/.local/lib/python3.10/site-packages/decomp2dbg/clients/gdb/pwndbg_client.py", line 63, in context_gdecompiler
    banner = [pwndbg.ui.banner(pane_title, target=target, width=width)] if with_banner else []
  File "/opt/tools/pwndbg/pwndbg/ui.py", line 37, in banner
    title = title.upper()
AttributeError: 'NoneType' object has no attribute 'upper'

Did i miss something ? Will debug in this afternoon, maybe there is still offsets problems.

Adress are correct but it point to the wrong area, weird

@0xMirasio interesting, ok a few things:

Are you sure you picked the correct last base address? As in the the last address of the last mapped region of your target, since vmmap will usually show a few regions of the same binary
Confirm you are using master pwndbg?
When you say the address is correct, do you mean the symbols imported correctly but the decompilation is just not printing correctly?

I've pushed more changes to main to help you with this crash. You should now see a more explicit address printed out when you error at this same place. The address printed out should be an offset... if it's not, that means PIE analysis failed. This should also fix the crash you see on the screen about the banner.

Pretty sure about last base adress/start addr. Will try tomorrow , i'm not sure i have latest pwndbg, will see tomorrow.
3  : Indeed the decompiled output is completely incorrect, it point to code that isn't the libraries.
will try the debug fix .

Really weird, so decompiled adress is okay, it correspond to the adress of the function i want. (_parse_special_tag)
When i try disassemble 0x7ffff78d96f0,+5 or disassemble _parse_special_tag,+5 it point to the good ASM code but name is completely broken. There is no decompiled also.
here are some info :

@0xMirasio I think I just fixed the bug. Try on main now. It was a PIE detection mismatch from your library to your main binary. What I believe was happening was the main binary you were debugging was a no-PIE but ofc the library was PIE,,, so it always though the offests were absolute addrs. Try it out now and lmk :)

Alright, looks like it's time for me to stop blind debugging. @0xMirasio could you:

Send me the main binary you are debugging
The library you are loading in IDA
The gdb commands you are using to get to a point where the decompilation does not work

I'll look at it some point this week.

Hello. Can't send neither librairie neither binary as the software is my companie private data. I will try to debug from my side but unfortunately i cant give you my sample (sorry).

I'm going to try to trigger the bug via other non private sample, and send you if I can retrigger it.

@0xMirasio No problem, but yeah, if you can trigger it another way, that would be great. I would recommend libc if you know a way to trigger the bug.

Closing this issue since remote debugging is now supported (allowing for custom address range assigning); however, you still can't decompile and debug two things simultaneously. This support must be added at some point to accomplish shared lib debugging. Deferring to #43.


Feature Request: Deploy a dashboard automatically with the lights, swtiches and sensors

As a used I'd like to get a dashboard in a draft version where everything is built automatically given the objects: lights, switches and sensors we have discovered on HA.
This means than instead having to edit the config.js and add item by item, it would be a nice idea to have a feature that can build the configuration of the items automatically in 3 columns width. Then add other feature where we can drag and drop the tiles to sort them as we want.
https://github.com/kutlugsahin/smooth-dnd
https://kutlugsahin.github.io/smooth-dnd-demo/
Thanks!

Shouldn't you be able to do exactly that in the config.js file? E.g. by building the config variable programmatically and iterating over the global states variable, instead of adding each item by hand.
Drag and drop sounds a lot more involved...

Check this out: https://github.com/resoai/TileBoard/pull/264


fix: support cd history, use dirs -v

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


AttributeError: type object 'Manager' has no attribute 'from_queryset'

Installed
django-mptt==0.7.4
south==0.7.6
RESOLVED
Another issue now
ImportError: No module named fields

Hey @melbalewis, check out the discussion in #97 and probable fix merged in #98! Feel free to reopen if you still have any problems.


Add GC MS pipeline using xcms

Create GC MS workflow that starts with xcms. Add it as a subdirectory in GCMS directory, alongside with the workflow using aplcms.
It uses the following steps:

msconvert to convert raw to mzML (check Apply peak picking)

peak picking in m/z dimension - actually creates centroids

preprocessing step using xcms

extract information from raw data to obtain a peak table
the result is intensity table, metadata table, and Rdata object

ramclustr

extract mass spectra in msp format

riassigner

assign retention indices based on given reference compound list

matchms

find matches with given reference spectra database

@xtrojak The first draft is there - now we need to test it and add proper annotations for the individual steps.


kola/test/oem: add ignition OEM test

Signed-off-by: Mathieu Tortuyaux<EMAIL_ADDRESS>In this PR, we try to test the usage of OEM - which is too required to run tests for the "new" ignition(v2/v3)
Closes: https://github.com/flatcar-linux/Flatcar/issues/486
Three tests:

try to reuse the partition using a btrfs format
try to reuse the parition using ext4 format
wipe the partition by using wipeFilesystem

How to use
$ ./bin/kola list | grep oem
cl.ignition.oem.regular			[all]								[all]		[cl]		[all]		[all]
cl.ignition.oem.reuse			[all]								[all]		[cl]		[all]		[all]
cl.ignition.oem.wipe			[all]								[all]		[all]		[all]		[all]

Testing done
Only on qemu.

@pothos @krnowak thanks for your reviews - comments have been addressed :)


Links don't have the correct mouse cursor behavior

The screenshot didn't capture it, but when I mouseover the links at the bottom of a page, such as "Source code" or "Send feedback about this page" I would expect the mouse cursor to be the "hand" and visually indicate this is clickable

Current hyperlinks for gallery are built using a Hyperlink component created locally. We should update all of the hyperlinks in the app to use the XAML Hyperlink component via react-native-xaml.


Runelite-mixins build error on JDK 12 Source option 6 no longer supported

Describe the bug
While building the runelite-mixins project it displays the following error:

To Reproduce
Steps to reproduce the behavior:
Building on Intellij Ultimate,
Expected behavior
Built the runelite source files
Environment (please complete the following information):

Maven version : 3.6.1
Java version 12.0.1
Windows 10 x64
RuneLite master branch (20-04-2019)

This is non trivial for us to fix correctly, and probably isn't worth it until Java 14 comes around. For now we support both LTS releases of Java, 8, and 11.

This is non trivial for us to fix correctly, and probably isn't worth it until Java 14 comes around. For now we support both LTS releases of Java, 8, and 11.

Thank you for the quick response, just tried on JDK 11 and worked fine! issue is thereby closed

Well, no, this is still issue so it should still be tracked


TypeError: string is not a function

Challenge Escaping Literal Quotes in Strings has an issue.
User Agent is: Mozilla/5.0 (Windows NT 10.0; WOW64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/51.0.2704.103 Safari/537.36.
Please describe how to reproduce this issue, and include links to screenshots if possible.
My code:

var myStr = "I am a \"double quoted\" string inside \"double quotes\"."

For some reason this game me a string is not a function error as seen in the screenshot,

You are almost there. Please check, is something missing at the end of the line?
Thanks for reporting, if you need any assistance, JavaScript Chat Room.
Closing as not a bug, on website.


Investigate IllegalAccessError running Servlet TCK

The exception that is happening is the following:
ERROR: java.lang.IllegalAccessError: class jdk.incubator.http.internal.common.Utils (in unnamed module @0x2064d52f) cannot access class sun.net.NetProperties (in module java.base) because module java.base does not export sun.net to unnamed module @0x2064d52f

Probably adding --add-exports java.base/sun.net=ALL-UNNAMED will work.
Investigate where to add it, and maybe commuticate the platform TCK of this behavior. (Better yet would be using the standard HttpClient from JDK 11, and not the incubator version of it)

Probably this will be fixed in Servlet TCK 6.0


Missing primary key in some tables

Description:
Mysql InnoDB cluster requires a primary key for each table to work and below tables don't set a primary key.
REG_RESOURCE_COMMENT
REG_RESOURCE_RATING
REG_RESOURCE_TAG
REG_RESOURCE_PROPERTY
UM_SHARED_USER_ROLE
AM_API_SCOPES
Note: Any migration scripts related to this should be fixed as well.
Suggested Labels:

Suggested Assignees:

Affected Product Version:
The issue was identified in APIM 2.1.0, other versions should also be verified.
OS, DB, other environment details and versions:
Steps to reproduce:
Related Issues:
https://github.com/wso2/product-is/issues/3898

Duplicate in https://github.com/wso2/product-apim/issues/9505


add signing functionality

This pr adds signing functionality using filippio's library. Not sure if we want to add other functionality as well until https://go-review.googlesource.com/c/go/+/276272 is merged and released?

Originally the package didn't include signing because the ZIP215 rules don't affect honest signers either way, and signing is already provided by the language package.
Am I understanding the context correctly that

the motivation for adding signing functionality is to pick up the faster curve implementation used by this library but not used by the standard library;
there is some possibility (the PR you linked) that the standard library would pick up that curve implementation?

Originally the package didn't include signing because the ZIP215 rules don't affect honest signers either way, and signing is already provided by the language package.
Am I understanding the context correctly that

the motivation for adding signing functionality is to pick up the faster curve implementation used by this library but not used by the standard library;
there is some possibility (the PR you linked) that the standard library would pick up that curve implementation?

on second thought, this isnt a big need for us. Will close it

on second thought, this isnt a big need for us. Will close it


TF2 fixes.

I'm not sure what your plans are with tf2. Here are some initial fixes to run in graph mode with tf2.

@googlebot I signed it!

Hi JJ, we are recovering from ICML but will take a look shortly. Thanks!


Implement remaining exercises for track

@ErikSchierboom mentioned here that we only have four exercises remaining in the C# track to be implemented:

[x] error-handling
[ ] forth
[x] pov
[ ] zipper

There's light at the end of the tunnel :)

@ErikSchierboom - I'll give error-handling a go

@robkeim Cool!

@robkeim I'm working on the pov exercise.

Sounds good @ErikSchierboom!

@robkeim I'll probabyl work on the zipper exercise tonight.

 That means it's more porting than solving :)

Fair enough :)

Thanks @ErikSchierboom, the C# track now has every available exercise implemented 🎉

Hurray! Thanks for the great help @robkeim!
@kytrinyx We now have all exercises available in C# too! :tada:

@robkeim @ErikSchierboom 🎉 Brilliant! 🎈


fixed PL IBAN/BBAN specification

As mentioned in https://github.com/arhs/iban.js/issues/30

Hi @Necromos, do you mind telling us why you closed the PR? (haven't had a chance to look at it yet)

Hey @LaurentVB, sure. After I patched it I found out 2 issues.

Fixed format for PL BBAN still was not working correctly
My fix bugged previously working version of IBAN validation for PL banks.

Sooo I closed all issues and wanted to take a deeper look into it later and then format proper issue here with solution :)
Sorry for this commotion.

No problem. Don't hesitate to open a new issue when you got the whole thing sorted. Thanks for using and improving the lib.


Can not change time, add users, etc. Grayed out unlock to change

I have install Ubuntu 14.04 LTS using crouton on the 2015 Dell Chromebook 11. All went well except I cant change time settings, add users, etc. The "Unlock to change" is greyed out and when clicked it says "Do have permission see admin" Any advice?

@motorcyclerider are you seeing the greyed-out settings in Chrome OS or in the chroot (i.e., in Ubuntu).

chroot

Pretty strange, @motorcyclerider I'll defer to others. Wish I could offer a helpful suggestion

tl;dr known issue, can't (reasonably) fix.
polkit rears its ugly head again, see previous discussion at #457. We do have polkit running now, but the policy files (/usr/share/polkit-1/actions/com.canonical.controlcenter.user-accounts.policy, /usr/share/polkit-1/actions/com.canonical.controlcenter.datetime.policy) only allows users with an active session to authenticate and change the time/add users. For whatever reason, logind isn't considering the session as "active" even though it should be. If you really want, you can go in and change the no in the <allow_any> tags to auth_admin_keep, but I wouldn't recommend it. System time is best left to Chrome OS to manage. Adding another user to login as isn't well tested or supported either, but you're free to try and see what happens.

Out of interest, I'm just asking the question, before I try it, in case someone knows the answer already and I can save myself a few hours of work.
This isn't an issue that I had before 2 weeks ago and doesn't seem to be an epidemic that's affecting everyone, so while there's no reasonable way to fix it on my existing chroot, and other chroots I've set up seem to inherit the same issue, is it likely to be caused with an issue in my ChromeOS that may be fixed by a complete factory reset? Potentially to an earlier version of ChromeOS?

@fraserYT if it helps, I went just short of a powerwash: popped out of developer mode then back in again, installing crouton from scratch. The issue persists . . . but I did learn that checking "Weekday" and/or "Date and Month" will nuke the time display from the menu bar. If you can get by with just 12- or 24-hour time and, optionally, seconds then apparently that's as good as it will get for the foreseeable future.


Rename operators.

positionoperator->position
momentumoperator->momentum

Coverage remained the same at 98.115% when pulling d840c24d6f1556ebfc7586e050e4eb31cfbe5956 on rename-operators into f0f4a20f308287241dcc1a5c3c8c6ec236699644 on master.


dircycle plugin: clean and fix logic once and for all

STATUS: ready
Description
This pull request fixes the error introduced in #3395, which changes pushd -q -0 to pushd -q +0, which doesn't do anything unless you have the pushd_minus option set. But, if that's the case, then pushd -q +1 doesn't work correctly.
To that end I standardised that option since it seems to change between different zsh setups. You can read more in the PUSHD_MINUS section of the zshoptions manpage.
I also removed the evals and cleaned up the plugin a little bit.
RFC (Request For Comments)
1. Fast switch
I added in commit d63120501ed924b801774359fb3e8be902cbd0e5 the hability to switch between directories and redraw the prompt without using a new line to run the pushd command.
Here are a couple of screencasts showcasing the difference:

Current switch (old version):

Fast switch (new version):

Please comment what you think about this new behavior.
2. Left/right switch direction
Finally the chosen behavior for left/right is (see https://github.com/robbyrussell/oh-my-zsh/pull/3413#issuecomment-67987390):

CTRL+SHIFT+LEFT : moves to last visited directory
CTRL+SHIFT+RIGHT: moves to next visited directory

the previous behavior seemed a little odd an unintuitive to me:

CTRL+SHIFT+LEFT : moves to directory on the left as appears in dirs output
CTRL+SHIFT+RIGHT: moves to directory on the right

Please comment which direction you think is better.

Close #1566
Close #2134
Close #2101
Close #2474

I have added the iTerm2 modification to the pull-request branch. I believe it is ready to merge, with at least 2 testers. Let's see if I can get someone else to chime in though.

Checked it out on Linux, works perfectly, would be great if it will be merged soon.

Thanks for confirming, @MyChaOS87.

Hi folks.
I don't think this iTerm-specific code is correct or necessary.
# add key bindings for iTerm2
if [[ "$TERM_PROGRAM" == "iTerm.app" ]]; then
	bindkey "^[[1;6D" insert-cycledleft
	bindkey "^[[1;6C" insert-cycledright
else
	bindkey "\e[1;6D" insert-cycledleft
	bindkey "\e[1;6C" insert-cycledright
fi

The two branches of that if statement do exactly the same thing. In a bindkey call, \e and ^[ both mean ESC; they're just alternate ways of writing it. You can test it by using both in a row.
$ bindkey "^[[1;6D" insert-cycledleft
$ bindkey "\e[1;6D" backward-word
$ bindkey | grep 6D
"^[[1;6D" backward-word

I think the issue isn't that iTerm uses different escape sequences from other terminals for Ctrl-Shift-arrows, but that by default, iTerm doesn't send anything for Ctrl-Shift-arrows. (Maybe because it's a Mac idiom that Ctrl-Shift-stuff does things at the OS/window management level, not inside a terminal.) The OS X Terminal.app has a similar issue, in that Ctrl-Shift-arrow just sends the same thing as a the normal, unmodified arrow key.
This is a configuration thing that needs to be done at the level of iTerm or Terminal.app if you want to use Ctrl-Shift-arrow inside zsh. I don't think we can detect or change those settings from within zsh (well, without resorting to AppleScript in the OMZ startup sequence, which is just getting ridiculous).
Having the iTerm.app test in the code might be causing confusion or false confidence. Maybe it should be removed, since it's not doing anything?
The escape sequences here match what xterm produces. Other terminal emulators, in particular rxvt/urxvt, emit different sequences for Ctrl-Shift-arrow. Unfortunately, AFAIK terminfo doesn't provide info on Ctrl-Shift-arrow as a capability, so we can't use it for portability. Maybe this is something that could be hacked in to lib/termsupport.zsh as an extension to $terminfo, so at least within OMZ we have access to portable mappings for at least the most popular terminals. (Or maybe there's a third party database for this that I'm not aware of?)
@andrewferrier – are you sure you never did customizations for your iTerm2? Specifically, in the Keys section of your profile – did you maybe "Load Preset..." with something like "xterm defaults"? That will load a bunch of extra key settings that include all the Ctrl-Shift-arrows, with escape sequences that match xterm's. But this isn't the default for iTerm2's initial setting or new profiles it creates. (I don't know why, especially since the default $TERM it reports is xterm-256color.)
This is the default set of Keys for a new profile in iTerm2 2.1.1 for me on OS X 10.9.5. (Curiously, it's not available as a preset either; you can only get it with a new profile.)

You're absolutely right @apjanke, '\e' is the same as '^['. From "Function keys and so on" (zsh docs):

If you move the cursor back over that ^[, you'll find it's a single character --- you can position the cursor over the ^, but not the [. This is zsh's way of inserting a real, live escape character into the line.
...
You can also use the two characters ^[, which is just another way of entering an escape key. However, the kosher thing to do is to turn it into \e.

And if I try it running both commands the keyboard shortcuts still work ok.
I think the problem was that vi-mode was clobbering the key bindings, as he explained in https://github.com/robbyrussell/oh-my-zsh/pull/3413#issuecomment-68036042. I overlooked that fact and kept the unnecessary if as a consequence.
So I'm :+1: for removing that piece of logic and I'll use this opportunity to also a README to the dircycle plugin.

Cool.
You might also be able to fix the load-order dependency wrt vi-mode now. I'm looking in to it for history-substring-search over at #3913. It looks like using bindkey -m to bind keys in specific keymaps (instead of the current keymap), and adding bindings for both emacs and viins maps (and possibly vicmd, too, depending on preference), could take care of that issue.

I put in PR #4623 to de-redundantize these. (Ran in to it today, and submitted the PR before I looked back and saw I'd already commented on this, sorry.)


x = 'foo' x = 'bar' parsing

Hello, in my .ini file I have
ServerPackages=Core
ServerPackages=Engine
....
After parsing it leaves only 1 ServerPackages line, how this can be fixed ?

I think it needs to be:
ServerPackages[]=Core
ServerPackages[]=Engine
This is the way ini's arrays seem to work

Yeah, it would be great but the .ini file is provided by other application, I know it's dumb to assign multiple values to the same "variable", if I do name[] = value1 name[] = value2 app isn't working.

I would suggest some sort of middle parser...
Something that would read the input add the [] and then pass it to the parser.
None the less probably not a problem with this library.

Can you reccommend such middle parser ? Or I should do it by hand like string.replace('ServerPackages','ServerPackages[]') ?

I have same problem ...
Have solution it?

Duplicate of https://github.com/isaacs/ini/issues/52
(Technically https://github.com/isaacs/ini/issues/52 is a duplicate of this, but that's more clearly stated, and probably @robertkowalski will give you what you want :)

see here: https://github.com/npm/ini/issues/52#issuecomment-188531495


All my JsonWritter jobs failed due to WARNING urllib3.connectionpool Connection pool is full, discarding connection. Connection pool size: 10

Hi, I am using JSONWriter to save json files to s3. However, all my jobs failed due to the following error, any thoughts? Thanks.
WARNING urllib3.connectionpool Connection pool is full, discarding connection: Connection pool size: 10
[Rank 9] Watchdog caught collective operation timeout: WorkNCCL(SeqNum=15049, OpType=ALLREDUCE, Timeout(ms)=1800000) ran for 1800272 milliseconds before timing out.
An error occurred while reading from response stream: ("Connection broken: ConnectionResetError(104, 'Connection reset by peer')", ConnectionResetError(104, 'Connection reset by peer'))
Some NCCL operations have failed or timed out. Due to the asynchronous nature of CUDA kernels, subsequent GPU operations might run on corrupted/incomplete data.

Hi @viyjy, Can you try passing max_workers args to JSONWriter to some reasonable value such as 4 or 8 and see if that solves the issue?

Hi @viyjy, Can you try passing max_workers args to JSONWriter to some reasonable value such as 4 or 8 and see if that solves the issue?

I am using max_workers=64. You mean 64 is too large?

@viyjy, I guess so. Can you try a lower value to see if that works?

I have tried max_workers=8 but got the same error

Would you happen to have a reproducible script that you can share? It will help us root-cause the issue faster.

@viyjy Wondering, do have a reproducible script that you can share?

Unfortunately, I am closing this issue because of inactivity. Please feel free to re-open if you are still seeing the problem. Thank You!


unexpected keyword argument 'flushed' in lambda extension

Describe the bug:
Call self.send(data,flushed=true) in https://github.com/elastic/apm-agent-python/blob/main/elasticapm/transport/base.py#L183 uses not declared argument.
To Reproduce

we use elastic-apm-extension-ver-1-0-0-x86_64 as lambda layer
use a newer version of elastic-apm=">6.8.1"
Result : TypeError: send() got an unexpected keyword argument 'flushed'

Environment (please complete the following information)

OS: N/A - use lambda
Python version: any
Framework and version [e.g. Django 2.1]: None
APM Server version: 1.0.0
Agent version: >6.8.1

Some clarification might be needed.
The flushed=True behavior was introduced in 6.9.0, so that might be your issue if you're using 6.8.1. It's also not designed to be called directly, we call it via this line.
Are you using our official lambda integration and extension?

We upgrade quite recently ( last week) - our team which takes care of APM did some upgrades on the server side as well and after that change, we experienced a heavy raise in this issue in our logs. I don't know why but I don't experience any issue since I downgrade elastic-apm to 6.8.1. All my point is that signature is required  "forced_flush" but call "flushed" argument on that specific line -  https://github.com/elastic/apm-agent-python/blob/main/elasticapm/transport/base.py#L183

Huh. You're totally right. I'm actually not sure how my manual testing didn't catch this. I'll have a fix shortly, thanks for the report.


Problem with HDF5 libraries

From<EMAIL_ADDRESS>on 2012-02-01T07:45:14Z
What steps will reproduce the problem?
Each time the HDF5 libraries have been updated spyder becomes unusable with the following message, on the python console (not the internal one). In the past I have seen it again. I am on ArchLinux x86_64
###-Start of paste-#############################################################
Warning! HDF5 library version mismatched error
The HDF5 header files used to compile this application do not match
the version used by the HDF5 library to which this application is linked.
Data corruption or segmentation faults may occur if the application continues.
This can happen when an application was compiled by one version of HDF5 but
linked with a different version of static or shared HDF5 library.
You should recompile the application or check your shared library related
settings such as 'LD_LIBRARY_PATH'.
You can, at your own risk, disable this warning by setting the environment
variable 'HDF5_DISABLE_VERSION_CHECK' to a value of '1'.
Setting it to 2 or higher will suppress the warning messages totally.
Headers are 1.8.7, library is 1.8.8
SUMMARY OF THE HDF5 CONFIGURATION
=================================
General Information:
-------------------
HDF5 Version: 1.8.8
Configured on: Sun Jan 29 11:10:55 UTC 2012
Configured by: nobody@
Configure mode: production
Host system: x86_64-unknown-linux-gnu
Uname information: Linux archlinux 3.2.2-1-ARCH #1 SMP PREEMPT Thu Jan 26 08:40:20 CET 2012 x86_64 Intel(R) Core(TM) i5 CPU M 460 @ 2.53GHz GenuineIntel GNU/Linux
Byte sex: little-endian
Libraries:
Installation point: /usr
Compiling Options:
------------------
Compilation Mode: production
C Compiler: /usr/bin/gcc ( gcc (GCC) 4.6.2 20120120 )
CFLAGS: -march=x86-64 -mtune=generic -O0 -pipe -fstack-protector --param=ssp-buffer-size=4 -D_FORTIFY_SOURCE=2
H5_CFLAGS: -std=c99 -pedantic -Wall -Wextra -Wundef -Wshadow -Wpointer-arith -Wbad-function-cast -Wcast-qual -Wcast-align -Wwrite-strings -Wconversion -Waggregate-return -Wstrict-prototypes -Wmissing-prototypes -Wmissing-declarations -Wredundant-decls -Wnested-externs -Winline -Wno-long-long -Wfloat-equal -Wmissing-format-attribute -Wmissing-noreturn -Wpacked -Wdisabled-optimization -Wformat=2 -Wendif-labels -Wdeclaration-after-statement -Wold-style-definition -Winvalid-pch -Wvariadic-macros -Wnonnull -Winit-self -Wmissing-include-dirs -Wswitch-default -Wswitch-enum -Wunused-macros -Wunsafe-loop-optimizations -Wc++-compat -Wstrict-overflow -Wlogical-op -Wlarger-than=2048 -Wvla -Wsync-nand -Wframe-larger-than=16384 -Wpacked-bitfield-compat -Wstrict-aliasing -Wstrict-overflow=5 -Wjump-misses-init -Wunsuffixed-float-constants -Wdouble-promotion -Wsuggest-attribute=const -Wtrampolines -O3 -fomit-frame-pointer -finline-functions
AM_CFLAGS:
CPPFLAGS:
H5_CPPFLAGS: -D_POSIX_C_SOURCE=199506L   -DNDEBUG -UH5_DEBUG_API
AM_CPPFLAGS: -I/usr/lib//include -D_LARGEFILE_SOURCE -D_LARGEFILE64_SOURCE -D_BSD_SOURCE
Shared C Library: yes
Static C Library: no
Statically Linked Executables: no
LDFLAGS: -Wl,-O1,--sort-common,--as-needed,-z,relro,--hash-style=gnu
H5_LDFLAGS:
AM_LDFLAGS:  -L/usr/lib//lib
Extra libraries:  -lpthread -lz -lrt -lm
Archiver: ar
Ranlib: ranlib
Debugged Packages:
API Tracing: no
Languages:
----------
Fortran: no
                        C++: no

Features:
---------
Parallel HDF5: no
High Level library: yes
Threadsafety: yes
Default API Mapping: v18
With Deprecated Public Symbols: yes
I/O filters (external): deflate(zlib)
I/O filters (internal): shuffle,fletcher32,nbit,scaleoffset
MPE: no
Direct VFD: no
dmalloc: no
Clear file buffers before write: yes
Using memory checker: no
Function Stack Tracing: no
GPFS: no
Strict File Format Checks: no
Optimization Instrumentation: no
Large File Support (LFS): yes
Bye...
###-End of paste-#############################################################
Original issue: http://code.google.com/p/spyderlib/issues/detail?id=924

From<EMAIL_ADDRESS>on 2012-02-06T06:20:13Z
You can either set HDF5_DISABLE_VERSION_CHECK=1 (or 2) as the message states, or you can re-build & re-install the h5py package.
If you are not using the hdf5 functionality, I would suggest the former.


Try to reduce CPU usage on cassandra metadata due to expensive glob

Purpose is to cache also queries that ReadTimeout in order to avoid hammering down cassandra metadata cluster

Coverage decreased (-0.1%) to 81.77% when pulling 577e0298a80b3d2cd03e42391255b48b80c5bf07 on erebe:master into 41d4021b6e7377d91e288e19ac713c202908c6e8 on criteo:master.


Offline streams are not removed from the list, refreshing extension does nothing

Describe the bug
When I open Gumbo, streams that are not currently live or that were recently live are still shown in the list. Refreshing the extension does not do anything. Removing and re-installing the extension fixes the problem temporarily, but it happens again as soon as a new stream is added to the list or when one goes offline.
To Reproduce
Steps to reproduce the behavior:

Go to any webpage.
Click on Gumbo extension icon.
You will see streams that are not live in the list of live streams.

Expected behavior

Click Gumbo icon.
Only streams which are currently live should appear.

Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: Windows 10 Pro 64-bit
Browser: Mozilla Firefox
Version: 122.0.1 (64-bit)

Smartphone (please complete the following information):
N/A
Additional context
Add any other context about the problem here.

i have the same problem, but in my case new streams are not added to the list

This is most likely an issue with the service worker becoming unresponsive. It's unfortunately an old browser issue, still discussed to this day, and there's not much I can do until they find a proper fix.


Add: pre-commit + linting

We should add some code format / linting to the package.

[ ] to begin we might want to use pre-commit setup
[ ] Start with Ruff and the basic pre-commit items
[ ] codespell for docs
[ ] Once linting it setup we'd want to add a CI build to run linters on each pr.

Ci can can be a separate issue if someone wants to focus on getting pre-commit setup first.

i guess the question here is whether we want to use pre-commit or use hatch's tools.


spec/facet/tests: adding example for change in #2756

Problem
The customLineage facet was updated in #2756 but the test case is not up to date.
Closes: #2799
Solution
This example gives all the customLineage for the outputDataSet from this query:
CREATE VIEW CUSTOMER_DISCOUNTS AS
SELECT NAME, AMOUNT_OFF, STARTS_AT, ENDS_AT
FROM DISCOUNTS INNER JOIN CUSTOMERS ON (DISCOUNTS.CUSTOMERS_ID=CUSTOMERS.ID)
Each of the 4 columns (NAME, AMOUNT_OFF, STARTS_AT, ENDS_AT) has 3 inputFields - its own identity from the input, plus 2 indirect inputFields from the join clause, CUSTOMERS_ID and ID.

[ ] Your change modifies the core OpenLineage model
[ ] Your change modifies one or more OpenLineage facets

One-line summary:
Updating the customLineage facet test for the new syntax created in #2756
Checklist

[x] You've signed-off your work
[x] Your pull request title follows our guidelines
[ LULZ] Your changes are accompanied by tests (if relevant)
[X] Your change contains a small diff and is self-contained
[X] You've updated any relevant documentation (if relevant)
[X] Your comment includes a one-liner for the changelog about the specific purpose of the change (not required for changes to tests, docs, or CI config)
[N/A] You've versioned the core OpenLineage model or facets according to SchemaVer (if relevant)
[N/A] You've added a header to source files (if relevant)

SPDX-License-Identifier: Apache-2.0
Copyright 2018-2024 contributors to the OpenLineage project

@tnazarew the unsuccessful test fails because of diffs that don't actually look different (maybe tab instead of whitespace?) - I'm not sure what to do here:
@@ -139,12 +139,12 @@
               }
             ]
           },
-          {
+          {
             "namespace": "SnowflakeOpenLineage",
             "name": "CUSTOMERS",
             "field": "ID",
             "transformations": [
-              {
+              {
                 "type": "INDIRECT",
                 "subtype": "JOIN",
                 "description": "ON (DISCOUNTS.CUSTOMERS_ID=CUSTOMERS.ID)"
@@ -160,7 +160,7 @@
                 "type": "INDIRECT",
                 "subtype": "JOIN",
                 "description": "ON (DISCOUNTS.CUSTOMERS_ID=CUSTOMERS.ID)"
-              }
+              }
             ]
           }
         ],

Exited with code exit status 1

Thanks for the contribution @Sheeri


CLI: Analyzers with DiagnosticSeverity.Info

IAnalyzerExporter have hardcoded DiagnosticInfo.DiagnosticInfoSeverity MinimalSeverity.
Make that configurable.

I have made this update to my dev branch, but I do not want to make Pull request, until @vad710 can merge current pull requests, so I can rebase and this change easier to merge.
Cleaned diagnostics descriptions: https://github.com/Kasperki/UnityEngineAnalyzer/commit/885dd8b8ac7225f0ba36d598cee482f5c9a3edb4
Added command line parser: https://github.com/Kasperki/UnityEngineAnalyzer/commit/4b8f846aed6f04ad25c3eb60c7df9d288bac03b3
And finally Minimali Severity can be given as CLI argument: https://github.com/Kasperki/UnityEngineAnalyzer/commit/032779636ced7b5376bc0069ba5f1bf1a8d020c4

I will try to get all the PRs wrapped up this week

I think it might be better to add the severity level we want in the config file - what are your thoughts on this?

Should be good to submit the new commits


Review: slider.marks prop

The slider API docs says the following about the marks prop, which can be bool or array:

Marks indicate predetermined values to which the user can move the slider. If true the marks will be spaced according the value of the step prop. If an array, it should contain objects with value and an optional label keys.

I have implemented this prop as below, does that make sense?
https://github.com/cmeeren/Feliz.MaterialUI/blob/cbc5dd789a526939fdea917f888f27d4f06d7c1a/src/Feliz.MaterialUI/Props.fs#L3712-L3720
Usage:
slider.marks true
slider.marks(1, 5, 10, 50, 100)
slider.marks(0.5, 1.5)
slider.marks({| value = 37.5; label = Some "body temp" |}, {| value = 50; label = None |})

A less verbose alternative to anonymous records is tuples , either int * string option or float * string option. The final line in the usage example above then becomes

slider.marks((37.5, Some "body temp"), (50, None))

I got compiler errors about incompatible types when trying to pass anonymous records across project boundaries.
I ended up using the tuple alternative.


Fix NPE in cluster client after multiple redirects.

Observed this exception during failover testing:
java.lang.NullPointerException
	at io.aeron.cluster.client.AeronCluster$AsyncConnect.awaitPublicationConnected(AeronCluster.java:1600)
	at io.aeron.cluster.client.AeronCluster$AsyncConnect.poll(AeronCluster.java:1529)
        ....

I believe thing can occur in the following scenario:
Step 1 => Step 2 => Step 3 => Redirect (member X) => Step 1 => Step 2 => Step 3 => Redirect (member Y) => Step 1 (Exception!)
This will occur due to the publication for member Y being closed at the point the client is redirected to member X.

Thanks @JPWatson. Good catch.


Fix *join & *leave

Thanks very much. I'll have to move channels to the top though as I have to run the setup function to assign the new variable.


Improve send flow

@islandbitcoin Updated above screen. Showing destination field if it is lnurl address or username and showing memo filed if it is editable.

@islandbitcoin What should be removed or edited in this screen?

@islandbitcoin What should be removed or edited in this screen?

nothing


Undefined "$promise" variable in Client/WebSocket.php

Error/stack trace happens when specifying a timeout in a $client->open(timeout) call.
Not sure how this variable should be initialised so regrettably can't deliver a patch...

I cant reproduce this. Do you have a code snippet for me?

Apologies for late reply, came down with the flu.
Anyway, I used your own client example in devristo/phpws/examples/echo_client.php and added a timeout argument to the open()-call on line 30 as in:
$client->open(10)->then...
This got me the error/stack trace.
Hope this helps,
Tom.


Typescript: types conflict @types/mongoose

Sounds like an issue with your typescript bindings. Our docs do say that lean() now accepts either a boolean or an object. Please open up an issue here: https://github.com/DefinitelyTyped/DefinitelyTyped

seems like the issue still exists in mongoose v5.4.19

/**
* Sets the lean option.
* Documents returned from queries with the lean option enabled are plain
* javascript objects, not MongooseDocuments. They have no save method,
* getters/setters or other Mongoose magic applied.
* @param bool defaults to true
*/
lean(bool?: boolean): Query & QueryHelpers;

lean doesn't support object.

@flyinghawker false: https://github.com/Automattic/mongoose/blob/5e47d8be8b35599e4997d5f2ef88ef19c7853a32/lib/query.js#L1597 . Either way, this isn't a typescript bindings repo, please open an issue there


how do you determine the container name?

in the powershell example in Using multiple scripts section, how would it know which container in the storage account to look for?

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 40aaccb8-19ba-168f-f5bf-65f1f5331541
Version Independent ID: 6cdfd166-633a-50a3-2b45-cde0e991e540
Content: Azure Custom Script Extension for Windows
Content Source: articles/virtual-machines/extensions/custom-script-windows.md
Service: virtual-machines-windows
GitHub Login: @bobbytreed
Microsoft Alias: robreed

#39077
#39076
#39075

@sasanmcp
Thanks for the question! We are investigating and will update you shortly.

@sasanmcp Container name is part of the URL.
For example, lets take this url https://xxxxxxx.blob.core.windows.net/buildServer1/1_Add_Tools.ps1
In this url buildServer1 is the container name for that blob

@sasanmcp Is there any update

I will close this out for now. If you need additional help please let me know and we can reopen and continue.

Got it. So filesURL is always necessary. Depending on whether you’re hosting it in storage account or not, you can or cannot add storage account name and key.
From: jakaruna-MSFT<EMAIL_ADDRESS>Sent: Thursday, September 19, 2019 5:03 AM
To: MicrosoftDocs/azure-docs<EMAIL_ADDRESS>Cc: Sassan Hajrasooliha<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/azure-docs] how do you determine the container name? (#39075)
@sasanmcphttps://nam10.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fsasanmcp&data=02|01||0bb8a0fc39c845fc750708d73cf94f51|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637044913776691370&sdata=byU%2BfLxjONRVi8mXFjvo8WhlWQ9wWHxt4hCwSm%2FwfZ8%3D&reserved=0 Is there any update
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam10.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fazure-docs%2Fissues%2F39075%3Femail_source%3Dnotifications%26email_token%3DAJXJYFYDXIV6UII6SRWAA2DQKNS7BA5CNFSM4IXV6F5KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD7DHAPY%23issuecomment-533098559&data=02|01||0bb8a0fc39c845fc750708d73cf94f51|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637044913776701374&sdata=MRyf0SnpP1dSavgzkIjO1pedi%2FxzVG5EKaOn2vhyS6M%3D&reserved=0, or mute the threadhttps://nam10.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAJXJYF6HJ667JSBHRD5NF3DQKNS7BANCNFSM4IXV6F5A&data=02|01||0bb8a0fc39c845fc750708d73cf94f51|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637044913776701374&sdata=9WvILR3qYlOU9dzlO53FJi7mXrxy0BBVx7LwL8cQfPI%3D&reserved=0.


Support JWK thumbprints as key ids

This addresses #474.
Allow the default and custom kid generation to be changed to a more "standard" way of generating JWK kids.
Also introduces a configuration concept via::JWK.configuration where the default values has been moved.
Would very much like to deprecate the current way of kid generation, but we would need to notify about the deprecation in some way before that.

The API should not have changed in this case, the intention was that at least. What API do you have on your mind?

@excpt got very curious on what im missing:)

I see that I have chosen my word poorly. 😖
Clarification:
You want to depricate the current used way of kid generation.
This will break existing implementations while continuing using the 2.x versions of this gem.
I recommend bumping the version to 3.0 in that case. So this is will be clear for all the direct and mostly indirect users of this gem.
This allows us to continue the development and it's still possible to continue supporting the 2.x versions.

Thanks for the clarification 👍 . I will in my turn clarify my thoughts:

The changes are not changing the current behaviour at all so it's possible to ship in a 2.x version with optional support for this.
If someone would like to use the thumbprint as the kid the gem can be configured with JWT.configuration.jwk.kid_generator_type = :rfc7638_thumbprint. This will change the behaviour of the kid generation to generate the new kind of kids. The default value for this will be JWT.configuration.jwk.kid_generator_type = :custom
In a future 2.x version we could add a deprecation warning on load that the defaults will change and explain what it means.

@excpt ok to merge? No changes to the current behaviour and deprecation will happen at a later stage with some kind of warnings to the user.


Update model and dataset tools

Added some more dataset tools, replaced the model with an effnet small based one, and added a toggle to use DML.
also, I feel like I'm not using commits correctly...

LSGTM!


Added Signing with Chocolatey Code Signing Certificate

@mwrock @ferventcoder can I get you both to have a look at these changes, and to let me know what you think?

I am still seeing an issue when I try to run ./build.bat quick-deploy regarding the Build-ClickOnce task.  The error message I am seeing is:

error MSB3482: An error occurred while signing: Failed to sign bin\Debug\app.publish\Boxstarter.WebLaunch.exe. SignTool Error: No certificates were found that met all the given criteria. [C:\github_local\forks\chocolatey\boxstarter\Boxstarter.ClickOnce\Boxstarter.WebLaunch.csproj]

This is after updating the Thumbprint in the csproj file locally NOTE That isn't yet included in this PR.

@mwrock @ferventcoder ok, so I have been doing some reading/testing this morning, and I believe that I have now got both the ClickOnce installer, and the ClickOnce manifests being signed using the Chocolatey Authenticode Certificate.  The command .\build.bat quick-deploy now completes successfully.  It was necessary to add the TimeStamp Server URL in order to get the signing of the manifests to complete.
When you have some time, can you please take a look at this, and let me know if you have any concerns, etc.

@mwrock @ferventcoder I have added an AppVeyor build to this repository as well.
For now, this is looking at doing a full build, however, it looks like AppVeyor build image doesn't like doing one of the tests:
https://ci.appveyor.com/project/chocolatey/boxstarter/build/1.0.7#L3157
It isn't clear to me if something else needs to be set up for this to not throw an exception.  We will either need to fix this, or revert to running the quick-deploy task, rather than the full set of tasks.

@mwrock thank you for the review!


[SR-15880] Add ability to swift-format to only re-format certain ranges of a file

Previous ID
SR-15880

Radar
rdar://112190730

Original Reporter
@ahoppen

Type
New Feature

Additional Detail from JIRA

Votes
0

Component/s
swift-format

Labels
New Feature

Assignee
None

Priority
Medium

md5: 55c33a2de9a258402756fbffd509321e

Issue Description:
swift-format currently only has the ability to re-format the entire file. Especially when working in projects whose current indentation cannot be fully described by swift-format configuration files and where re-formatting the entire source code is not an option, it makes swift-format basically unusable.
What I do enjoy is git-clang-format’s ability to only re-format the lines that I touched. It would be nice if swift-format had a similar functionality to only format certain ranges of a file.

Just mentioning that this functionality could be very nice to have 🙂

Noticing Saafo's issue, i just want to bump this up and mention that i think this is a very important feature to make an IDE like vscode useable for writing Swift code.
My attempts to add such a feature to vknabel's formatter extensions did not turn out perfect and it seems to me that it requires built-in support in the formatter tool itself. For now I'm using a local version of vknabel's SwiftFormat (not swift-format) which uses diffing mechanisms to only format a specific range of lines, and although it's not perfect at times, it's still far better than not having it at all, specially combined with on-type formatting.


closes issue #49

Add save and load buttons to the GUI. Keeps the pause state of the current server state.

rustfmt has been added and used


Prevent blow-up with or operator

The library does distribution directly when ORing two CNFs together. This results in exponential blowup in certain cases
from satispy import Variable as v
reduce(lambda x, y: x|y, [v("x" + str(i)) & v("y" + str(i)) for i in range(100)])

Would it be within the scope of this libraries purpose to not do any algebraic simplifications until a solution is requested at which point it does https://en.wikipedia.org/wiki/Tseytin_transformation to produce a CNF?

That's a good idea, and already did some research on the topic, but haven't implemented anything yet. I hope I can invest some time soon.


chore: Redesign the Forest node Grafana dashboard.

Summary of changes
Changes introduced in this pull request:

Redesign the Forest node Grafana dashboard. It's currently being displayed at this public url

Reference issue to close (if applicable)

Closes
Other information and links

Change checklist

[ ] I have performed a self-review of my own code,
[ ] I have made corresponding changes to the documentation,
[ ] I have added tests that prove my fix is effective or that my feature works (if possible),
[ ] I have made sure the CHANGELOG is up-to-date. All user-facing changes should be reflected in this document.

I'd change monotonically increasing metrics to counters, e.g., p2p messages. Otherwise LGTM.

@LesnyRumcajs, i have made some changes please can you Public Url listed in the description.

Looks good, I'd also change the Range Sync Failure Count to counter.

@samuelarogbonlo Also, could you please change the time range to several days? Otherwise it's hard to spot any trends. Or perhaps this could be made variable? I recall it was possible with public DataDog dashboards.

@LesnyRumcajs unfortunately, template variables are not supported for public dashboards on Grafana cloud but i have set the time range to 7 days.

@LesnyRumcajs can you reapprove?


Test classes need to be separated from main classes

I found some examples of Test classes directly along side main classes, an example being nars.lab.narclear.PhysicsModel. These classes need to be pulled out into testing.

Done


poetry + hydra

🚀 Feature

It would be great to have hydra and poetry integrated within Pytorch Lightning.
Motivation

@tchaton sounds good, could you pls add motivation or/and use-case how you would like to use it?

Hey @Borda , I used pytorch-lightning for two on my previous projects and I chosed not to use it for my latest one.
I felt it was missing some flexibility, due to having one logic template for both model and dataset.
We are using hydra for the configuration and poetry for package management.
Here is the project: https://github.com/nicolas-chaulet/deeppointcloud-benchmarks.
I think we have an interesting architecture for managing ML research projects. Please, have a look :)
Best,
Thomas Chaton.

There is already similar suggestion, so please continue the discussion in #807


Simplify the GlobalStateSynchronizer

This PR simplifies the GlobalStateSynchronizer in two main ways:

it now deals with only a single global state root hash at a time
it now uses the PeerList from the BlockSynchronizer (doesn't keep its own peer cache) - although that part is a bit hacky.

Making it draft for now, just so that we can run automated tests and take a closer look at everything before we consider merging this code.
Based on the #3693 branch, so it contains the commits from there, too.

bors try

bors r+

bors r+


PickerIOS这一节太丑了，不知道什么时候加了个css,希望改进一下

Hey @autumnleaves1, thanks for filing an issue. Can you help me understand what the issue is here? I can't tell from the screenshots.

Sorry,I did not describe my point clearly.
I just want to say that the list on left in the first picture should be left and should not be moved. That should be like the second picture。
I think it should be that your adaptation is not doing well.
As a programmer, looking at the uncomfortable

Hi @autumnleaves1, it seems to me that after the website redesign it is not long an issue.
I'm going to close this for now. If you are experiencing similar issues with the new design please create the new issue.
P.S. AFAIK https://reactnative.cn in not a part of the official website and this issue shouldn't be reported in this repository in the first place.


Overly Aggressive polling of robots.txt by Web Extension

I have noticed that my system has started triggering mod_evasive on some Apache Web Servers.  When I looked into the issue it seems my browser is repeatedly trying to load the robots.txt file, eventually triggering the too many requests for the same URI and blocking me.
While the unblock window is fairly small, this seems like this might catch other people as well.
There is no robots.txt file on the server, so it seems like it should probably cache, at least for a little while, that trying to fetch the robots.txt returned a 404.

Wappalyzer doesn't issue http requests on its own. Your issue might come from something else.

@jvoisin Surprise, it's issue.

@jvoisin , are you sure?
I was pretty sure I saw something in 'src/drivers/webextension/js/driver.js' in 'wappalyzer.driver.getRobotsTxt' which was using the fetch method to grab the robots.txt file which is why I opened the issue here.
/**
 * Fetch and cache robots.txt for host
 */
wappalyzer.driver.getRobotsTxt = (host, secure = false) => {
  return new Promise((resolve, reject) => {
    getOption('robotsTxtCache')
      .then(robotsTxtCache => {
        robotsTxtCache = robotsTxtCache || {};

        if ( host in robotsTxtCache ) {
          resolve(robotsTxtCache[host]);
        } else {
          var url = 'http' + ( secure ? 's' : '' ) + '://' + host + '/robots.txt';

          fetch('http' + ( secure ? 's' : '' ) + '://' + host + '/robots.txt')
            .then(response => {
              if ( !response.ok ) {
                if ( response.status === 404 ) {
                  return '';
                } else {
                  throw 'GET ' + response.url + ' was not ok';
                }
              }

              return response.text();
            })
            .then(robotsTxt => {
              robotsTxtCache[host] = wappalyzer.parseRobotsTxt(robotsTxt);

              setOption('robotsTxtCache', robotsTxtCache);

              resolve(robotsTxtCache[host]);

              var hostname = host.replace(/:[0-9]+$/, '')
            })
            .catch(reject);
        }
      });
  });
};

It looks to me like it is only caching if it receives response text, which is why I was suggesting it might be better to cache the 404 somehow.
@AlexByte Thanks for commenting.

Damn, you're right, this is indeed an issue, sorry for the noise :/

@jvoisin It's all good. I should have provided more detail on why I thought this might be the issue in the first place, I was just in too much of a hurry.

It specifically handles 404 by returning an empty string, which then gets cached. When I test this on example.com it only attempts to fetch robots.txt once. Would you be able to share a URL for testing?

@AliasIO
I missed that you were caching the empty string, I am afraid ES6 and Promises are things I am not overly proficient with so I completely overlooked that when skimming the code originally.
Now that I see that the caching is there I am thinking this might be something unrelated to the extension itself, but that because the extension is making the request I am seeing it get caught by mod_evasive.  Since I wasn't expecting the extension to make a web request (I thought it just analyzed the page's DOM for patterns), I probably jumped the gun on opening an issue.
However, just in case you are curious and want to know, the site that it has happened to me the most on is www3.ca.uky.edu.  You won't really see anything on that web address though as we just use it to host registration forms and there are not currently any open.  The main page really just returns a text string of the hostname. So I am not sure it is a great test right this moment.
I can't get you server side logs or look at the server configuration until Monday, but if you let me know the approximate time and what IP address you connect from I can pull something if needed.
Given the above insights, a couple of other things did occur to me:

It does seem to happen the most frequently when resuming Chrome and having multiple tabs open.  There is a chance I had multiple pages from that host open in more than one tab.  is it possible that the Browser Extension is invoked concurrently if multiple tabs are open?  Most pages on the site would have unique URLs, but the robots.txt file is a fixed entity so if extension events get fired for all tabs as soon as Chrome launches, that might explain it.
This one is more for me, but perhaps I should examine the mod_evasive configuration more closely.  There is always a chance the DOSSiteCount/DOSSiteInterval settings are lower than I thought. Since I am on the same network the round trip time is pretty quick, perhaps mod_evasive is actually triggering before the extension attempts to fetch robots.txt, and I what I am seeing is its first request and then any retries it might make since it received a 503 response when it was blocked, which wouldn't be cached.

Thanks for taking the time to look, I will update this on Monday just to let you know what I see.
-jrj

I suspect it's your second point as I'm not seeing this behaviour on that URL. It's not ideal that Wappalyzer attempts request robots.txt once for every detected application in case of an error though, perhaps we should add a cooldown period.
Note that robots.txt will not be fetched if tracking is turned off in the extension settings.

So it turns out there is a form live at https://www3.ca.uky.edu/forestry/forest-management/new.php if you want to see one that is live. For that site Wappalzer is showing me 6 items in use, so from what you said it would try to grab robots.txt file (assuming it wasn't cached) 6 times?  If that is the case and it tried to process those requests in parallel then that might explain it.
The server is configured to block people if the exact same resource is requested 4 times within 1 second.
When you said that the robots.txt file will be hit only when tracking is turned on, did you mean then 'Anonymously send reports on detected applications to wappalyzer.com for research' option?  I don't see any option specifically about "tracking" in the options of the extension:

Additionally, if I click on "Inspect Views" in Chrome and look at the contents of 'Local Storage' I don't see any entry for 'robotsTxtCache' and changing the options on the options above don't seem to change any of the stored values in the Local Storage Cache.  But this could be completely normal since I know that chrome tries to separate the page's local storage from content scripts local storage and I am not sure how to view the data used by the content scripts, I just wanted to provide some additional details.

For that site Wappalzer is showing me 6 items in use, so from what you said it would try to grab robots.txt file (assuming it wasn't cached) 6 times?

Only if fetching robots.txt repeatedly returns an error. The extension still isn't likely to be the cause of the issue.

When you said that the robots.txt file will be hit only when tracking is turned on, did you mean then 'Anonymously send reports on detected applications to wappalyzer.com for research' option?

Yes, that's the one.

Additionally, if I click on "Inspect Views" in Chrome and look at the contents of 'Local Storage' I don't see any entry for 'robotsTxtCache' and changing the options on the options above don't seem to change any of the stored values in the Local Storage Cache.

Right, you probably can't see values stored by extensions. Not sure about this though.

Any update? It broke my company's SSO log page all the time.

This still seems to be somewhat prevalent, so I'll mark as a bug and take a look later.

I'm experiencing this defect as well.


[vcpkg] Enable finer-grained portfile sections

From https://github.com/Microsoft/vcpkg/pull/1205#issuecomment-308566621:

Split the control files into cleaner stages: E.g: file-load; extract-source; patch; configure; build; copy-install. And allow install command to stop at a certain stage.

This is widely done by all other package systems. See Homebrew, Pacman, Apt, and Gentoo for examples.

Allowing the install command to start at a certain stage would be very useful too. Rebuilding a large library every time you tweak a small detail of the install step can make developing a port a bit painful.

No more discussion need here, looks the feature currently supported, this issue can be closed now.


Ruby dependency upgrade 2023 jan

@donrestarone Brakeman indicates that the loofah gem 2.19.1 is vulnerable.

blocked by: https://github.com/restarone/violet_rails/issues/1393

@kripananda-yadav The review-app has been deployed. You can move on with further testing.

@alis-khadka could you please fix merge conflicts here?
CC @kripananda-yadav after that we would need another UAT pass to verify the changes


Optimize outlining

It filters out outlining spans of disabled (in settings) scopes on earlier stage to improve performance. The main change is over here https://github.com/fsprojects/VisualFSharpPowerTools/compare/fsprojects:master...vasily-kirichenko:optimize-outlining?expand=1#diff-d769655d7e66f49c8169b86d6aa975f1R174

Profiling code editing on TypeChecker.fs showed that ITextSnapshot.GetText() took 98% time of UI thread. So I added caching each file text in OpenDocumentTracker.
I have not checked carefully that I removed all GetText() occurrences yet.

Why do you switch the first run to be async on UI thread? It has to be run on UI thread anyway, so why bother?

If it runs on UI thread immediately, OpenDocumentTracker does not receive RegisterView call yet and its TryGetDocumentText returns None. This code
let uiContext = SynchronizationContext.Current
       Async.StartInThreadPoolSafe <| async {
           do! Async.SwitchToContext uiContext
           protectUpdate()
       }

allows UI thread to finish constructing all providers and notifying OpenDocumentTracker about the newly opened Buffer.

@dungpa I've fixed the tests. Please review.

Your code is the same as the old one - it calls first protectUpdate immediately, during DocumentEventListener construction (because Async.StartImmediate (async { doSomething() }) is the same as doSomething since StartImmediate performs everything in async block, up to first let!, do! or return!, in the thread it was called). I.e. the following sequence is executed in UI thread without any breaks, sequentially and blocking:

User open a document
VS instantiates all our taggers, filters etc.
The taggers create (in their constructors) instances of DocumentEventListener
DocumentEventListeners (in their constructors) call protectUpdate
protectUpdates do heavy things like getting IProjectProvider, ITextBuffer.GetText() and even VSLanguageService.GetSymbol()

So until all these stuff finish its work, UI is frozen.
New code
let computation =
    async {
     while true do
         do! Async.AwaitEvent events
         if not skipTimerDelay then
             startNewTimer()
             do! awaitPauseAfterChange()
         protectUpdate() }
// Go ahead and get the first bit of info for the original rendering
let uiContext = SynchronizationContext.Current
Async.StartInThreadPoolSafe <| async {
    do! Async.SwitchToContext uiContext
    protectUpdate()
}
Async.StartInThreadPoolSafe (computation, tokenSource.Token)

works differently:

VS creates taggers, they create DocumentEventListeners
DocumentEventListeners starts asyncs in ThreadPool, which will switch to UI thread and perform the heavy work
DocumentEventListeners starts the old good infinite loop waiting VS events in ThreadPool
UI thread is freed at this point which means that documents are open faster

So the new behavior has very clear benefits and works differently than the dead-locking one from that reverted PR.
Actually code editing experience is very nice now, even on TypeChecker.fs.
I'll use this version tomorrow at work at will see how it works in real world.

I see the point of scheduling the first protectUpdate on thread pools then switch to UI thread and execute on UI thread.
But why don't we do the same for protectUpdate in the while loop? In other words, why aren't do! Async.SwitchToContext uiContext in the loop? What is the reason for this inconsistency?

Well, we don't know what VS API we must call from UI thread only. For some reason we assume that the following code:
maybe {
    let dte = serviceProvider.GetService<EnvDTE.DTE, SDTE>()
    let! doc = dte.GetCurrentDocument(textDocument.FilePath)
    let! project = projectFactory.CreateForDocument buffer doc
    return currentRequest, doc, project
}

, which is copy-pasted in every tagger, must be called from UI thread. Yes, when I called it from ThreadPool threads VS behavied like UI thread was waiting something forever (not working UI, 0% CPU). We should clean up our understanding out this stuff. I remember Valdimir Matveev said something about it.

I removed switching to UI thread for the first protectUpdate call and got the following exception:
[VFPT][Error 05:24:41 ] Exception Message: The calling thread cannot access this object because a different thread owns it.
Stack Trace:    at System.Windows.Threading.Dispatcher.VerifyAccess()
   at System.Windows.DependencyObject.GetValue(DependencyProperty dp)
   at System.Windows.Controls.Panel.get_IsItemsHost()
   at System.Windows.Controls.UIElementCollection.VerifyWriteAccess()
   at System.Windows.Controls.UIElementCollection.Clear()
   at <StartupCode$FSharpVSPowerTools-Logic>.$UnusedDeclarationMargin.clo@28-159.Invoke(Unit _arg2)
   at FSharpVSPowerTools.ProjectSystem.VSUtils.protectOrDefault[a](FSharpFunc`2 f, a defaultVal)

We could add ForegroundThreadGuard.CheckThread https://github.com/fsprojects/VisualFSharpPowerTools/blob/85f4aa175025ec71fd4b44e79ba2b52b910a9176/src/FSharpVSPowerTools.Logic/VSUtils.fs#L383 to the start of every updateAtCaretPosition function (e.g. https://github.com/fsprojects/VisualFSharpPowerTools/blob/85f4aa175025ec71fd4b44e79ba2b52b910a9176/src/FSharpVSPowerTools.Logic/HighlightUsageTagger.fs#L81) to avoid accidental refactoring.
Note that for some features, it's fine with calling from thread pool. Here are the best references I could find https://msdn.microsoft.com/en-us/library/dn448839.aspx or https://github.com/Microsoft/VSProjectSystem/blob/94bd2eac351477d37a4e2a80d32e73a4e1bbd347/doc/overview/3_threading_rules.md. cc @vladima in case he has a clear explanation for this.

Please review the last commit. DocumentEventListener:

cancel previous invocation of callback
starts callback in freshly created async
passes CallInUIThread function into the callback given it an opportunity to call in UI thread whatever it needs.

Great solution. :+1:


feat(ci): automatic make generate

With this PR we let the github actions to run every night a make generate and have it therefore a fresh version of the autogenerated resources. Additionally we have now the Changelog updated in all release branches as well. The new automatic-updates task can be appendend with any further task if required eventually.
Closes #3868

Release Note
feat(ci): automatic make generate

:crossed_fingers: for tonight...


solve deployment bug #433 CloudFront logs does not enable ACL access

Feature or Bugfix

Bugfix

Detail
Solved bug 433, starting from April 2023 S3 default configurations changed, the default for s3 is set to disable ACL. Which is giving an issue for cloudfront logging on s3. The solution was to change the ownership of the object to object writer (enabling ACL for object writer as stated in cloudfront documentation).
Relates
-
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@nikpodsh @dlpzx would you merge this ?

Merged!

@akaitoua thank you so much for fixing this issue!


Add additional columns to the courses table

Context
We need to show more course information on the courses review page. This will be pulled via Find's API in the next PR.
Changes proposed in this pull request
Add columns to the course table for:

Start date
Applications open from
Course length

Link to Trello card
https://trello.com/c/D4sPmcZI/1106-dev-display-additional-course-info-on-summary-page?menu=filter&filter=label:Dev
Things to check

[x] This code doesn't rely on migrations in the same Pull Request
[x] If this code includes a migration adding or changing columns, it also backfills existing records for consistency
[x] API release notes have been updated if necessary
[x] New environment variables have been added to the Azure config

Yea you're absolutely right!


Feat/oauth refactor

Related issue: https://github.com/networknt/light-4j/issues/472
Related RFC: https://github.com/networknt/light-rfcs/blob/jiachen1120-patch-1/light-4j/0020-oauth-enhancement.md

Codecov Report

Merging #473 into master will increase coverage by 0.09%.
The diff coverage is 52.25%.

@@             Coverage Diff              @@
##             master     #473      +/-   ##
============================================
+ Coverage     49.84%   49.94%   +0.09%
- Complexity     1768     1783      +15
============================================
  Files           251      251
  Lines         10827    10923      +96
  Branches       1535     1546      +11
============================================
+ Hits           5397     5455      +58
- Misses         4796     4828      +32
- Partials        634      640       +6

Impacted Files
Coverage Δ
Complexity Δ

.../java/com/networknt/client/oauth/TokenRequest.java
94.59% <ø> (ø)
16 <0> (ø)
:arrow_down:

.../java/com/networknt/client/oauth/TokenManager.java
57.4% <35.48%> (-7.46%)
10 <4> (+3)

.../src/main/java/com/networknt/client/oauth/Jwt.java
60.63% <37.14%> (-9.37%)
18 <1> (+1)

...n/java/com/networknt/client/oauth/OauthHelper.java
45.26% <75.55%> (+4.99%)
33 <9> (+10)
:arrow_up:

...java/com/networknt/client/oauth/TokenResponse.java
83.33% <0%> (+4.16%)
13% <0%> (+1%)
:arrow_up:

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 9016e32...b8c9364. Read the comment docs.


[core] - tracer for client libraries and convenience layer spans should be library-specific with name/versions set to library name/versions

Recommendation from Liudmila that I will not be addressing in #16347
See #16088

Closing in favor of #19299 which by design enforces passing in the correct package name and version


The observation in Combined Arms

Hi,
Thank you for maintaining this fantastic repository. I have a quick question regarding the documentation for Combined Arms. In the documentation(https://magent2.farama.org/environments/combined_arms/#observation-space), Other teams presences/heaths/minimaps feature in the observation space is described as being "in some order".  Could you please clarify what this order is?
Thank you!

Hi @LXXXXR , I think this listed this way because its dependent on the agent type. Each type red_melee, red_Ranged, blue_melee, and blue_ranged are registered within the environment as g0, g1, g2, g3 respectfully. So the observations for: red_ranged (g1) would be:
Red Melee (g0): Presence → Health → Minimap (if enabled)
Blue Melee (g2): Presence → Health → Minimap (if enabled)
Blue Ranged (g3): Presence → Health → Minimap (if enabled)
With presence, health, and Minimap each being a channel and the Minimap being optional, that is why this can be 6/9 channels deep.
I hope this helps!
If you find something to the contrary, please let me know!

Thank you so much for the prompt reply! It really helps!


Variation in Tensorflow execution latency

System information:
Neural Network: 	LeNet 300-100
Target Platform: 	XEON PLATINUM 8000 SERIES - AMAZON AWS EC2 C5 INSTANCE
Supports:  A		VX-512, 72 Cores, 2-4 Threads; Up to 2 Instance Cores and 2 vCPUs
Baseline Framework: 	TensorFlow (AWS Image Version)
AMI ID: 		Deep Learning AMI (Ubuntu) Version 23.1 (ami-07262a45de118922e)
Python version:		p27
Describe the problem:
		To collect baseline results for TensorFlow model of lenet_300_100 ( Find the Accuracy and inference latency). By using MNIST dataset with a batch size of 16 and full(10000) number of batches.

Analysis Tensorflow inference runtime results, There is inconsistent on subsequent sessions. Please review the attached “ref_times” for  inconsistent runtime for various batches.

ref_times.txt
Please review the following logs and advice on this.
Source code / logs:
Time session source code:
with tf.Session(graph=graph, config=config) as sess:
curr_batch = 0
warm_up = 5
for i in range(warm_up):
sess.run(output, feed_dict={input_: zeros})
while True:
batch = input_reader.receive_batch()
if batch is None:
break
start = monotonic.monotonic()
pred = sess.run(output, feed_dict={input_: batch})
sys.stderr.write('Batch Inference Time(s): {}\n'.format(
monotonic.monotonic() - start))
output_writer.output_send(pred)
Please let know need another details? Please review all and advice me on this.

@VinothInspirit ,
Please provide us TF version and also the complete code being used so that we can replicate the issue from our end.Thanks!

Thanks for your response.
I have attached the complete source code for your evaluation, Results of evaluation time text file to understand the inconsistency inference time
And TF version : 1.13.1.
Please review and advice me on this
Results_ref_times.txt

@VinothInspirit,
When i tried executing the last section of the code i faced the following error RuntimeError: Attempted to use a closed Session.Kindly help us to reproduce the issue.Thanks!

#!/usr/bin/env python
# coding: utf-8

import tensorflow as tf
import numpy as np
import sys
from scipy.io import savemat
import matplotlib.pyplot as plt
from tensorflow.examples.tutorials.mnist import input_data

mnist = input_data.read_data_sets("MNIST_data/", one_hot=True)

output_graph =  "./lenet-300.pb"

# Parameters
learning_rate_ini = 0.001
training_epochs = 2
batch_size = 16
display_step = 1
num_batches = 100

# Network Parameters
n_hidden_1 = 300  # 1st layer num features
n_hidden_2 = 100  # 2nd layer num features
n_classes = 10  # MNIST total classes (0-9 digits)

# tf Graph input
tf.reset_default_graph()
with tf.name_scope("input_tensor"):
    x = tf.placeholder("float", [None, 784])
with tf.name_scope("label_tensor"):
    y = tf.placeholder("float", [None, n_classes])

# Create model
def model(_X, _W, _biases):
    layer_1 = tf.nn.relu(tf.add(tf.matmul(_X, _W['fc1']), _biases['fc1']))  # Hidden layer with RELU activation
    layer_2 = tf.nn.relu(tf.add(tf.matmul(layer_1, _W['fc2']), _biases['fc2']))      # Hidden layer with RELU activation
    with tf.name_scope("predictions"):
        pred = tf.matmul(layer_2, _W['out']) + _biases['out']
    return pred

# In[36]:

# Store layers weight & bias
W = {
    'fc1': tf.Variable(tf.random_normal([784, n_hidden_1], stddev=0.01)),
    'fc2': tf.Variable(tf.random_normal([n_hidden_1, n_hidden_2], stddev=0.01)),
    'out': tf.Variable(tf.random_normal([n_hidden_2, n_classes], stddev=0.01))
}

biases = {
    'fc1': tf.Variable(tf.random_normal([n_hidden_1], stddev=0.01)),
    'fc2': tf.Variable(tf.random_normal([n_hidden_2], stddev=0.01)),
    'out': tf.Variable(tf.random_normal([n_classes], stddev=0.01))
}

import time

if(1):
    # Construct model
    pred = model(x, W, biases)
    loss = tf.reduce_mean(tf.nn.softmax_cross_entropy_with_logits(logits=pred, labels=y))  # Softmax loss
    cost = loss
    optimizer = tf.train.AdamOptimizer(learning_rate_ini, beta1=0.9, beta2=0.999,
            epsilon=1e-08, use_locking=False).minimize(cost)
    # Test model
    correct_prediction = tf.equal(tf.argmax(pred, 1), tf.argmax(y, 1))
    # Calculate accuracy
    accuracy = tf.reduce_mean(tf.cast(correct_prediction, "float"))
    # Initializing the variables
    init = tf.initialize_all_variables()
    # Run model
    accuracy_total = None
    config = tf.ConfigProto(device_count={'GPU': 0})
    config.log_device_placement = True
    with tf.Session(config=config) as sess:
        sess.run(init)
        # Training cycle
        for epoch in range(training_epochs):
            avg_loss = 0.
            total_batch = int(mnist.train.num_examples/batch_size)
            # Loop over all batches
            for i in range(total_batch):
                batch_x, batch_y = mnist.train.next_batch(batch_size)
                # Run optimization op (backprop) and cost op (to get loss value)
                accuracy_total, predictions,_, l = sess.run([accuracy,pred, optimizer, loss],
                                                            feed_dict={x: batch_x,
                                                              y: batch_y})
                # Compute average loss
                avg_loss += l / total_batch
            # Display logs per epoch step
            if epoch % display_step == 0:
                print("Epoch:", '%04d' % (epoch+1), "loss=",                     "{:.9f}".format(avg_loss))
        print("Optimization Finished!")
        print("Accuracy : ", accuracy_total)
        for i in range(num_batches):
            batch_x, batch_y = mnist.test.next_batch(batch_size)
            start_time = time.time()
            val_acc = accuracy.eval({x: batch_x , y: batch_y})
            print("Evaluation time batch {} : {}".format(i+1, time.time()-start_time))
            print("Validation Acc : {}".format(val_acc))

I've pasted the indented code. This should work.

@azaks2 do you know off the top of your head what could cause this?

@azaks2 replied offline:

A profile would be nice
We do not generally encourage inference via python, so python gc comes to mind
3 We overlap with the computation of next_batch

Reg:2. We do not generally encourage inference via python, so python gc comes to mind
VinothInspirit: What then would be the tf prescribed way of running(and measuring) inferences?
Reg:3. We overlap with the computation of next_batch
VinothInspirit: We give the inputs here using feed_dict, not by a pipeline, so I'm not sure if sess.run(...) would still overlap with next_batch.

For inference take a look at https://www.tensorflow.org/tfx/guide/serving
One important part is that the process will turn variables into constants.
According to the timeline (comparing slow and fast sess.run) it seems most of the diff comes from the MatMul op. I am not 100% sure but using taskset to run the script (taskset 0x1 python ...) improved the variance for my runs.

slow inference

fast inference

to get timeline
val_acc = sess.run([accuracy],
feed_dict={x: batch_x , y: batch_y},
options=tf.RunOptions(trace_level=tf.RunOptions.SOFTWARE_TRACE),
run_metadata=run_metadata)
...
tl = timeline.Timeline(run_metadata.step_stats)
ctf = tl.generate_chrome_trace_format()
with open(str(i)+'timeline.json', 'w') as f:
f.write(ctf)


commands/set-tree: checks if on windows, if so use \ instead of /

On windows using jpm -l deps I got errors about "Invalid switch /i". After investigation I found that set-tree didn't take windows into account, so I amended that. Now I am able to use jpm -l. :)

May be related a bit to #45.

So one reason I haven't been bullish on this is that modern windows should be able to use "/" in most (all?) scenarios - with the exception of shelling out to certain paths. So please let me know if this is is actually broken or "it just looks more windowsy".

Before #45 was merged I experienced breakage (running jpm -l deps in the cross subdirectory of https://github.com/saikyun/jaylib-wasm-demo (around 83eca893ff73fb9e54a8d9d744392927157a3cc9)).
I just tried with a version of jpm that had #45 merged and I don't see the same problems any more.
May be @saikyun can confirm.

I had the same problem as sogaiu.
For some time I've been seeing errors like "Invalid switch /i" when building, but it hasn't hindered building. However when using jpm -l and (janet-bounded-queue)[https://github.com/saikyun/janet-bounded-queue], the builds would fail (it couldn't find janet-bounded-queue). Only thing I can see is different about janet-bounded-queue is that it's a single file.
After using Janet built with this PR all errors disappeared.

I didn't realize sogaiu mentioned a different PR, gonna have to try that one.

Thanks!
Den lör 9 apr. 2022 19:01Calvin Rose @.***> skrev:

Merged #48 https://github.com/janet-lang/jpm/pull/48 into master.
—
Reply to this email directly, view it on GitHub
https://github.com/janet-lang/jpm/pull/48#event-6401989223, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAS46Z3V2CXXX4FVID6GMBLVEGZU3ANCNFSM5SR4KQDQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

I had the same problem as sogaiu.
For some time I've been seeing errors like "invalid switch /i", but it
hasn't hindered building before I started using janet-bounded-queue and
jpm -l.
Either way, this PR removes the errors and lets me build with -l.


feat: add resource ec2-tgw-connect-peer

This cherry picks the commits from the upstream PR https://github.com/rebuy-de/aws-nuke/pull/1168 and then refactors the resource to the new libnuke/aws-nuke@v3 format. Additionally it adds CreationDate Tags ID and State properties to the resource.
Thank you to the original author @1solomonwakhungu for their contributions.

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