Update README.md.

Codecov Report

Merging #80 into master will increase coverage by 0.23%.
The diff coverage is 81.81%.

@@            Coverage Diff             @@
##           master      #80      +/-   ##
==========================================
+ Coverage   31.97%   32.21%   +0.23%
==========================================
  Files          44       44
  Lines        1251     1220      -31
  Branches       45       42       -3
==========================================
- Hits          400      393       -7
+ Misses        851      827      -24

Impacted Files
Coverage Δ

.../scala/net/scalax/fsn/slick/atomic/StrSelect.scala
100% <ø> (ø)
:arrow_up:

...la/net/scalax/fsn/slick/helpers/SlickHelpers.scala
33.33% <0%> (-0.88%)
:arrow_down:

...scalax/fsn/mix/operation/PropertiesOperation.scala
29.52% <100%> (ø)
:arrow_up:

.../net/scalax/fsn/json/operation/JsonOperation.scala
25% <100%> (-11.37%)
:arrow_down:

...calax/fsn/slick/operation/StrSelectOperation.scala
75.34% <100%> (+2.36%)
:arrow_up:

.../scala/net/scalax/fsn/mix/helpers/QueryExtra.scala
5.55% <0%> (-6.95%)
:arrow_down:

...ore/src/main/scala/net/scalax/fsn/core/FPile.scala
81.72% <0%> (-0.2%)
:arrow_down:

...t/scalax/fsn/slick/operation/CreateOperation.scala
0% <0%> (ø)
:arrow_up:

...scalax/fsn/slick/operation/RetrieveOperation.scala
0% <0%> (ø)
:arrow_up:

... and 1 more

Continue to review full report at Codecov.

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


do not support setting column comments on a view in Snowflake (DAT-12530)

Impact

[X] Bug fix (non-breaking change which fixes expected existing functionality)
[ ] Enhancement/New feature (adds functionality without impacting existing logic)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Description
Snowflake does not support setting view column comments after the view has been created, so this change prevents that.
Things to be aware of
Things to worry about
Performance impact of querying the Snowflake information schema about whether an object is a view.
Additional Context

@filipelautert Rerequesting your review because I just introduced some API changes to fix a request from product - they would like the database name to be capitalized when printed to the console in the "not supported" message.


Unable to cite gym retro

Issue summary
I use retro for my research and now I would like to cite it. However I can't find a DOI or any trace that this has been made available.
Please fix this.
https://guides.github.com/activities/citable-code/

Thank you for updating the readme, but you have added bibtex for the the paper, related to the repo, not a release of the repo. The former is helpful when searching for your research that led to the code, but I require the latter: a citable release. It's all laid out for you in this article: https://guides.github.com/activities/citable-code/

I see, how is this better than citing the specific pip version or git commit hash of the gym retro version you are using for an experiment?


Source file requires different compiler version

Hi, I am having some issues with the compiler version.
I am inside of a basic forge init repo. I am trying to use solidity version 0.8.22 but I am locked to 0.8.19 no matter what settings I choose.
Here is my settings.json inside the workspace:
{
    "solidity.compileUsingRemoteVersion": "v0.8.22+commit.4fc1097e",
    "solidity.defaultCompiler": "remote"
}

Here is my solidity file:
// SPDX-License-Identifier: MIT

pragma solidity 0.8.22;

And here is my error:
Source file requires different compiler version (current compiler is 0.8.19+commit.7dd6d404.Emscripten.clang) - note that nightly builds are considered to be strictly less than the released version(5333)

I think this could be a bug. Could someone replicate this?

Are you compiling using forge?

@elanzalaco  ^^^

@juanfranblanco seems like a bug, remote compiler is locked to 0.8.19 and cant change remote compiler version. Have tried everything

Seems like this could be a related issue too:
https://github.com/juanfranblanco/vscode-solidity/issues/440

BTW, just downgraded to v0.0.165 and error has disappeared

I have just been testing solidity 24, press F1 and go to User Settings. Then Solidity. And check what version you have in workspace and user. Should be Remote in both and the version you require. It looks looks like your workspace is using the embedded compiler.

@emiliolanzalaco ^^^ Workspace is your project and overrides others

It appears that this issue persists in the latest version: v0.0.170.
My .vscode/settings.json:
{
  "solidity.packageDefaultDependenciesContractsDirectory": "src",
  "solidity.packageDefaultDependenciesDirectory": "lib",
  "solidity.formatter": "forge",
  "[solidity]": {
    "editor.formatOnSave": true,
    "editor.defaultFormatter": "JuanBlanco.solidity"
  },
  "solidity.defaultCompiler": "remote",
  "solidity.compileUsingRemoteVersion": "v0.8.23+commit.f704f362"
}

Error:

As a current workaround, I am using a local compiler:
{
  "solidity.packageDefaultDependenciesContractsDirectory": "src",
  "solidity.packageDefaultDependenciesDirectory": "lib",
  "solidity.formatter": "forge",
  "[solidity]": {
    "editor.formatOnSave": true,
    "editor.defaultFormatter": "JuanBlanco.solidity"
  },
  "solidity.defaultCompiler": "localFile",
  "solidity.compileUsingLocalVersion": "/Users/pyk/github/pyk/zkevm-stb/soljson-v0.8.23+commit.f704f362.js"
}

@juanfranblanco it appears this is a bug as there are a few cases with similar setups. Seems like the compiler is stuck on v0.8.19 irrespective of global or workspace settings.
Would appreciate if you could look into fixing this 🙏🏼

Same issue here, both user and workspace are set to either latest or 24, but it's always stuck on 19.

plus one, the remote compiler is stuck at 0.8.19

This seems to be fixed with the latest update.

The issue was related to the standalone language server pull, that defaults the values to latest so the server won't require to set any values to start with. Whilst this makes sense, the user values were overridden on vscode as they weren't taking preference for a millisecond and not reloaded. *Latest" has been stuck in 19 on the solidity binaries for a few releases now, and there is nothing it can be done about that.


WIP support foreach

my first and only approach which does not work in the wild...

I assume this is superseded by #529


Add winui (w) and maccatalyst (m) to XLogFileLine.PlatformLetter

PlatformText needs to be updated, too.

Done.


md-input - type=number feature requests

feature requests:

Have the ability to disable the number spinner buttons.
I started typing out the suggestions below but began to realize that they probably don't align with the intentions of the spec but I have left them just in case:

Have a default value that is replaced upon first keyboard entry like the placeholder attribute on a basic input type="number" element (see plnkr example).

When the input field is cleared the placeholder does not move back down over the entry area.

What is the expected behavior?

Adding a CSS style to change the appearance of the number input to none should disable stepper buttons on the input element.

A placeholder value should appear (a numeric value) that is replaced when the user enters numeric input into the field.

After clearing the input the placeholder should move back down above the input area.

What is the current behavior?

The stepper buttons always show for input type="number"

When using the attribute value=0 (the only way to get a figure to show upon rendering) the initial value of 0 is appended to, not replaced by user input.

I think this issue may have to do with the value=0 attribute for md-input but the placeholder text does not move back down after clearing the input field and focusing on another field.

What are the steps to reproduce?
See my Plunker demo: http://plnkr.co/edit/Xi8bdAhAIcwhLbGZrtEU?p=preview
What is the use-case or motivation for changing an existing behavior?
The motivation is for UX purposes, especially with having a default value. In the case of a stepper, it doesn't make sense to have it for large numbers (numbers over 100k)
Which versions of Angular, Material, OS, browsers are affected?
Latest versions of Angular, Material, Chrome Canary
Is there anything else we should know?
Thank you for taking the time to look at this and keep up the good work!

@peterwiebe there's a bit too much going on in this issue; it makes it impossible to respond to individual requests / bugs. It sounds like you have a mix of bugs and feature requests here. Can you file them as separate issues?


createTag is in docs but not in code

I was reading the docs and in notice that in the instance API there is a method to createTag but it  is not in the library, any info about it ? Another thing there is way to get the actual dataspace?

Which version you use depends on which version of the document. The master is the latest code, but the new features may not have been released.

thanks

@liihuu  i thinnk, master is not published on npm


Links can sometimes exit the diagram based depending on provided criticality value or other inputs

"Everything in the diagram was looking normal until I linked ECMWF FWI to INFRA AEGIS that it started to look like this.... It has something to do with the criticality rating. When I made it 10, it made those weird curves. When I reset it to 5, it was fine."

Hazel - I tried to recreate this bug but I wasn't able to

This may not be a problem when we move off highcharts. It would be good to save this exact assessment in its broken state as a test case.

Oh you can see this bug here: https://www.usaon.org/apps/benefit-tool/assessment/6
ECMWF = data product
GWIS = data product
INFRA = application
INFRA should show up to the right of both of the others but it's lined up with one of its inputs for some reason

Weird, two minutes later I went back to that link and it's resolved.

Neat. A heisenbug ;)


Added support for a shared SPI bus

While using a display shield from Adafruit, they had the display and the SD card slot on the same SPI bus pins. So I modified this driver to allow a shared spi bus (by passing in a SPI pointer).
I understand this could cause issues when initializing since the SD card specification requires(?) initial clock speeds between 100kHz to 400kHz before compatibility for higher speeds is determined. I added an @note to let the user know this should be taken into consideration and the SD card should probably be initialized first before anything else on the same bus.
Users must also be careful to call SPI::lock() and SPI::unlock() while manually toggling chip select pins to avoid the case where chip select is pulled low by one thread and then another thread initiates a SPI transaction on the same bus before the first thread can lock the SPI bus mutex.
Please write back with any comments or concerns you may have. I have tested this on my end and it appears to work fine.

Hi @AGlass0fMilk, sorry about the unfortunate timing, but we're currently moving the block devices into the mbed-os tree:
https://github.com/ARMmbed/mbed-os/pull/7774#issuecomment-413083326
The prs on this repo will be frozen while this is happening, and then you can either open a pr on mbed-os, or we'll look at moving this pr over when we go through marking the external repositories as obsolete.

Hi @AGlass0fMilk there is also and RFC in PR https://github.com/ARMmbed/mbed-os/pull/7671
that overhaul the SPI hal API & will provide updates of the C++ drivers.
Without breaking changes of the API, this will provide a clear role for SPI instances (1 per slave).
Sharing instances will no longer be required.

Closing this PR, please add the PR to mbed-os repo (https://github.com/ARMmbed/mbed-os/tree/master/components/storage/blockdevice/COMPONENT_SD) if this feature is still required.


update changelog

Added changelog for 11.1.1-hotfix2

I don't think it is hotfix, I would go straight to 11.1.2 since last version number describe patches

Agreed, it will also look nicer when it gets moved to STABLE branch - will update the milestone and changelog now

also bumped assembly version 11.1.2


incorrect intrinsics results with provided pretrained model

Hello, thanks for your work on the code reconstruction. I got some problems when loading the model you provide in the readme page. I output the camera intrinsics and found there is a large distance with GT (taken in kitti raw). My output is as follow. I don't know where the problems lie in.

Hi Ljiaooo,
Do you mean the x and y offsets are large?
I should say the learning of the camera intrinsics is not perfect. It is theoretically able to converge to the true matrix when data and training techniques are proper, and is convenient when you don't know the values.

Thanks for your reply, I found your reproduced IntrinsicsHead() is a little different from the original where the softpuls function is not applied on the offsets by the original. After I deleted the softplus on offsets, the camera matrix got normal.

@Ljiaooo
Thank you very much for the findings. Since the learnings of depth and intrinsics are affecting each other, really appreciate if you may share some experiment results to show the improvement brought about by this revision and send a pull request.

Hello, Thank you  for this repository, What is the image dimensions used for the intrinsic matrix above?


Helper methods with assert/should

I am trying to create a helper method that take you from A to B and B to C, and would like to use them as needed.
For example)
var self = this;
this.horseman = new Horseman();
horseman
    .do(helper.AtoB.bind(self))
    .do(helper.BtoC)
    .then(function() {
        self.horseman.close()
    }).nodeify(done);

....
helper = {
    AtoB: function(done) {
        this.horseman
        .open('http://www.google.com/')
        .url()
        .then(function(data) {
            data.should.equal('http://www.reddit.com/')' // should fail here
        }).nodeify(done);
    },
    BtoC: ...
}

Having horseman passed to helper methods and going from point A to point B works fine; however, I cannot assert within the helper. It simply ignores the should failures and goes on. I am trying to set some helpers that I can reuse and avoid writing same lines to get from one point to another. Any suggestion would be appreciated. Thank you.

.do always resolves for some reason (probably shouldn't). However, it is basically a re-implementation of the bluebird function .fromNode (so it can just be deprecated since I made all the bluebird functions available to horseman).
Anyways, to actually answer you question, you should have your helper methods return Promises and then just give them to .then instead of .do. Changes below:
//var self = this;
//this.horseman = new Horseman();
// ^ Not needed, this inside horseman chain is already bound to the horseman instance
horseman
    .then(helper.AtoB)
    .then(helper.BtoC)
    .then(function() {
        self.horseman.close()
    }).nodeify(done);

....
helper = {
    AtoB: function() {
        return this // Return chain result
            .open('http://www.google.com/')
            .url()
            .then(function(data) {
                data.should.equal('http://www.reddit.com/') // should fail here
            });
    },
    BtoC: ...
}

I am seeing that horseman.then is not a valid function.

I edited my comment. I should maybe add .then as an action.

Is there a way to pass values to helper methods?

Sure,
horseman
    .url()
    .then(helper.AtoB)

will pass the url as the first parameter to helper.AtoB.

How about passing custom variables?

It's all just normal Promises.
horseman
    .url()
    .then(function(url) {
        return helper.AtoB(varA, varB); // call it with whatever
    })

Thank you @awlayton . Saved my day!

Code above doesn't work if the helper.AtoB needs to return horseman chains. this is now undefined.
horseman
.url()
.then(function(url) {
return helper.AtoB('screenshot', '.jpg');
})
helper = {
AtoB: function(varA, varB) {
var varC = varA+varB;
return this.screenshot(varC); // this is now undefined
}
}

horseman
    .url()
    .then(function(url) {
        return helper.AtoB.call(this, 'screenshot', '.jpg');
    })

Please don't open (or reopen) issues just because you don't understand JavaScript, open issues for things that are wrong about horseman.

This helped me understand something completely unrelated... :+1:

Hah, nice @capcrunchIO.


Push constants for materials

This is a subset of the krperf branch with several fixes added to fix stuff that was broken. There is at least one problem remaining: The desktop version of the helmet model has ambient occlusion stored in a separate texture but the support for that is dropped. This breaks the pbr rendering test.
This follows #413 and replaces #411.
Before merging:

[x] Update helmet model to fix ambient occlusion on desktop.
[x] Ensure render tests pass, updating images as needed.

I've updated the textures, but I'm getting some validation warnings:
VUID-vkCmdDrawIndexed-None-02699(ERROR / SPEC): msgNum: -1539028524 - Validation Error: [ VUID-vkCmdDrawIndexed-None-02699 ] Object 0: handle = 0x980b0000000002e, type = VK_OBJECT_TYPE_DESCRIPTOR_SET; | MessageID = 0xa44449d4 | Descriptor set VkDescriptorSet 0x980b0000000002e[] encountered the following validation error at vkCmdDrawIndexed time: Descriptor in binding #3 index 5 is being used in draw but has never been updated via vkUpdateDescriptorSets() or a similar call. The Vulkan spec states: Descriptors in each bound descriptor set, specified via vkCmdBindDescriptorSets, must be valid if they are statically used by the VkPipeline bound to the pipeline bind point used by this command (https://vulkan.lunarg.com/doc/view/<IP_ADDRESS>/windows/1.3-extensions/vkspec.html#VUID-vkCmdDrawIndexed-None-02699)
    Objects: 1
        [0] 0x980b0000000002e, type: 23, name: NULL

I've fixed all the validation warnings except this one:
UNASSIGNED-CoreValidation-Shader-OutputNotConsumed(WARN / PERF): msgNum: 101294395 - Validation Performance Warning: [ UNASSIGNED-CoreValidation-Shader-OutputNotConsumed ] Object 0: handle = 0xea7170000000031, type = VK_OBJECT_TYPE_SHADER_MODULE; | MessageID = 0x609a13b | vertex shader writes to output location 0.3 which is not consumed by fragment shader
    Objects: 1
        [0] 0xea7170000000031, type: 15, name: NULL

As far as I can tell, this is wrong. Either:

0.3 refers to layout location = 3 in which case it's totally wrong, because no vertex shader writes to location = 3
0.3 refers to layout location = 2, in which case it's also wrong because the fragment shader reads from location = 2


Streamline NotIncluded Model Type

It would be nice if subcomponents not included in a LSM configuration (e.g. no snow model in the biosphere + biophysics configuration) did not get added to the state, if even by name only.

this may be useful to us:
https://github.com/CliMA/ClimaAtmos.jl/blob/ec56b8453712bff8e8662030993af3ec89b8b582/src/Models/Nonhydrostatic3DModels/nonhydrostatic_3d_model.jl#L57
(we'd still have NotIncluded, but then wouldnt show up in Y?)


Dealing with unit-less representations

While we should have everything tagged with units when possible, there are likely to be cases in which stripping units is necessary, i.e. casting to a jax.numpy object:
>>> import jax.numpy as jnp
>>> import numpy as onp
>>> import pint
>>> u = pint.UnitRegistry()
>>> vals = onp.random.rand(4) * u.meter
>>> jnp.array(vals)
/Users/mwt/software/miniconda3/envs/openff-system/lib/python3.7/site-packages/jax/numpy/lax_numpy.py:2116: UnitStrippedWarning: The unit of the quantity is stripped when downcasting to ndarray.
  object = np.array(object, dtype=dtype, ndmin=ndmin)
/Users/mwt/software/miniconda3/envs/openff-system/lib/python3.7/site-packages/jax/lib/xla_bridge.py:125: UserWarning: No GPU/TPU found, falling back to CPU.
  warnings.warn('No GPU/TPU found, falling back to CPU.')
DeviceArray([0.776736  , 0.5026873 , 0.08218545, 0.91840476], dtype=float32)
>>> # Same thing happens by default with original NumPy
>>> onp.array(vals)
__main__:1: UnitStrippedWarning: The unit of the quantity is stripped when downcasting to ndarray.
array([0.77673604, 0.50268727, 0.08218545, 0.91840475])

my initial concerns are

Ensuring going from a stripped external representation to a tagged internal representation reliably produces the same units
Possible necessity, and downsides, of a set of "default" units used in stripping units away
User experience when tagged representations aren't necessarily essential (what should be enforced?)

Vectorized representations are all unitless and in PotentialHandler.set_force_field_parameters() assumes that the units are the same as what is stripped out from PotentialHandler.set_force_field_parameters().
AFAIR all other interfaces are unit-bearing where possible.


Restart required to refresh

The app needs to be restarted to detect changes to images/GIFs/videos. However, I believe android supports an implementation in which such changes can be communicated to gallery apps without a restart or manual refresh.

You're right, it's possible to react to changes in the Media Store
without a restart.
It's planned, but some internal changes are required first.
On Mon, Jan 4, 2021 at 1:27 PM Donkey-Doug<EMAIL_ADDRESS>wrote:

The app needs to be restarted to detect changes to images/GIFs/videos. However, I believe android supports an implementation in which such changes can be communicated to gallery apps without a restart or manual refresh.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

You're right, it's possible to react to changes in the Media Store
without a restart.
It's planned, but some internal changes are required first.
On Mon, Jan 4, 2021 at 1:27 PM Donkey-Doug<EMAIL_ADDRESS>wrote:

The app needs to be restarted to detect changes to images/GIFs/videos. However, I believe android supports an implementation in which such changes can be communicated to gallery apps without a restart or manual refresh.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.


[Enhancement] Custom Description field for preset awards

Even when adding, for example, a moderate or severe award after an encounter, you might still want to enter some descriptive text.

Then use the custom description field

OK. That's one extra step -- selecting custom, adding XP value manually, then text description -- but you clearly don't think it's worthwhile, so that's that. No offence meant.

I really like the elegance of this module.


RPGLE and CL highlighting

I am using AvalonEdit in a project and I thought it might be useful to commit some of the new highlighters I created for RPG and CLLE.

Merged in d47b3239cbd3e623fbb4c62fbe7acb162b9fb561.
Thank you for your contribution!

Thanks @siegfriedpammer!


[network] Connectivity Manager updates to support Client Server model

Motivation
Connectivity manager is being added to public interfaces in a maybe_mutual mode.  This allows known peers on a public interface that also allows unknown peers.  This provides the ability to use the discovery functionality without constantly kicking out unknown peers (normal clients).
Additionally, this adds a fix to the way that outbound connection limits are enforced.  The outbound connection limit currently doesn't conflict because we use a second public network interface for outbound connections in VFNs, and the connection limits are per network interface.

/land


24 filter communication

resolves #24

We need to talk about how to reduce some of the columns. In "reduceByIndex" i added a few lines where "missingcolumns" are reduced and added. This results i.e. in ProblemNames being concatenated by default if the index is not unique. I don't think we want that but i am not sure what the best solution is.


Move samples under KEDA 2.0

Move the GCP PubSub sample i.e. Google Cloud PubSub and Kafka sample i.e  Azure Functions Kafka on Openshift 4 under KEDA 2.0 as they have supported KEDA 2.0 version and also modify their links(acc. to keda 1.0) under KEDA 1.0

@tomkerkhove @zroubalik
PTAL!
If it seems good then I will raise a PR for it!
Thanks

If they are upgraded then feel free to send a PR

yes, in their respective repo they have support for keda2.0.
I will soon create a PR for it.

Fixed it here #32


monitor configuration template invalid

In the monitor configuration __agent.conf there is a line:
monitor.binding_port=hostname_or_ip
which is very confusing as it must be "monitor.binding_ip=...".

hi milkotodorov~
Do not be difficult.
When you use this option,
Use when using multiple IP as your target server load balancing.
so never mind.

After the move to a folder to install the ngrinder monitor.
run_monitor.sh(Linux) or run_monitor.bat &(Windows)
check monitor

Pretty simple ^^
Note that the monitor must have specific ports open.
By default 10023.
good luck~

hi songeunwoo,
when I start the run_monitor.sh it is binding to the localhost not the IP assigned to the host. As a result the nGrinder controller cannot connect to the monitor. As soon as I use monitor.binding_ip=x.x.x.x it works. However it is initially described as #monitor.binding_port=hostname_or_ip in the __agent.conf
Here is the whole __agent.conf:
# If you want to monitor bind to the different local ip not automatically selected ip. Specify below field.
#monitor.binding_port=hostname_or_ip
monitor.binding_port=13243

line 2 must be corrected to #monitor.binding_ip=hostname_or_ip isn't it?
Regards,
Milko

yes. Public IP right.
i think, please check the ACL on the port 13243.
( Controller : Any ==> Monitor : 13243 ) <== When using the monitor port to be opened
to summarize,

Sign in to your Controller ngrinder site.
Using the Monitor download button-down menu at the top.
Unzip the downloaded file monitor.tar
run_monitor...

__agent_conf file is nothing to fix.

When I try to run the monitor without changing the monitor.binding_ip=x.x.x.x here is how the monitor starts:
10:32:28.826 [main] INFO  agent config - NGRINDER_AGENT_HOME : /home/ubuntu/.ngrinder-monitor
10:32:29.015 [main] INFO  org.ngrinder.infra.AgentHome - /home/ubuntu/.ngrinder-monitor/native is created.
10:32:29.029 [main] INFO  starter - ***************************************************
10:32:29.029 [main] INFO  starter - * Start nGrinder Monitor...
10:32:29.029 [main] INFO  starter - ***************************************************
10:32:29.171 [main] INFO  o.n.monitor.agent.RegisterMXBean - Added MXBean:org.ngrinder.monitor:name=System.
10:32:29.172 [main] INFO  o.n.monitor.agent.RegisterMXBean - Default MXBean Initialized.
10:32:29.172 [main] INFO  o.n.monitor.agent.MonitorServer - Service URL:service:jmx:rmi://[::]:13243/jndi/rmi://[::]:13243/jmxrmi is initiated.
10:32:29.227 [main] INFO  o.n.m.collector.DataCollectManager - SystemDataCollector started.
10:32:29.228 [main] INFO  o.n.m.collector.DataCollectManager - Collection interval : 1s).

And of course the controller cannot connect to this monitor and they are both running on different hosts.

Monitor Controller and the server must have a port open.
and so it will not have monitor.binding ip settings.
__agent_conf file is nothing to fix.

They do have all ports opened.

Did you try ping test ?
I will find some more issue.

yes - ICMP is also allowed and ping do works between ngrinder controller and the agent and vise versa

Could you run following command so that we can get to know it's connectable?
telnet monitor_ip 13243

And.. did u add the target host(monitoring target) in the test configuration?

Hi,
let me describe the scenario a bit more detailed. So nGrinder controller runs on <IP_ADDRESS> with 1 agent (additionally we have 3 more agents on other 3 VMs).
The monitor is installed on <IP_ADDRESS>. I have ran it without changing anything in its configuration:

__agent.conf:

From the nGridner controller VM I can actually connect to the monitor:

However the NGrinder controller seems that it cannot connect to it:

The VM with IP <IP_ADDRESS> has the monitor installed, but the __agent.conf is changed with monitor.binding_ip=<IP_ADDRESS>
If I make the same changes on <IP_ADDRESS> the nGrinder controller connects to this monitor too!

I hope this helps to make the picture more clear.
Regards,
Milko

OK. It's really weird...
We have such a problem before 3.4 was released.
From 3.4.. the problem was fixed. Could u post the monitor log from the start?

Here is the startup log:
09:26:17.325 [main] INFO  agent config - NGRINDER_AGENT_HOME : /home/vcap/.ngrinder_agent
09:26:17.458 [main] INFO  starter - ***************************************************
09:26:17.458 [main] INFO  starter - * Start nGrinder Monitor...
09:26:17.458 [main] INFO  starter - ***************************************************
09:26:17.536 [main] INFO  o.n.monitor.agent.RegisterMXBean - Added MXBean:org.ngrinder.monitor:name=System.
09:26:17.536 [main] INFO  o.n.monitor.agent.RegisterMXBean - Default MXBean Initialized.
09:26:17.536 [main] INFO  o.n.monitor.agent.MonitorServer - Service URL:service:jmx:rmi://[::]:13243/jndi/rmi://[::]:13243/jmxrmi is initiated.
09:26:17.612 [main] INFO  o.n.m.collector.DataCollectManager - SystemDataCollector started.
09:26:17.612 [main] INFO  o.n.m.collector.DataCollectManager - Collection interval : 1s).
Do you need something more?

Hello, any update? Just need to replace monitor.binding_port with monitor.binding_ip in the default __agent.conf...

This part seems to have been confused.
#monitor.binding_port=hostname_or_ip

If you set it as follows, it works well.
$ vi ~/.ngrinder_agent/agent.conf
monitor.binding_ip=public_ip
$ run_monitor.sh
...
15:26:12.447 [main] INFO  o.n.monitor.agent.MonitorServer - Service URL:service:jmx:rmi://public_ip:13243/jndi/rmi://public_ip:13243/jmxrmi is initiated.
...

Yes, I figured that out already, but you could fix it so it is no longer confusing for the other people that will eventually face the same issue.
Again the default configuration for the agent in __agent.conf should not
** monitor.binding_port=hostname_or_ip**
but ** monitor.binding_ip=public_ip ** instead

I have the same issue . When I started my controller at a tomcat , and opened the 8088 port. I runned my ngrinder-monitor at the target server , i got the following log and no target monitor data display in my controller page
13:47:50.282 [main] WARN  agent config - Please run agent with -o option
13:47:50.454 [main] INFO  starter - ***************************************************
13:47:50.454 [main] INFO  starter - * Start nGrinder Monitor...
13:47:50.454 [main] INFO  starter - ***************************************************
13:47:50.543 [main] INFO  o.n.monitor.agent.RegisterMXBean - Added MXBean:org.ngrinder.monitor:name=System.
13:47:50.544 [main] INFO  o.n.monitor.agent.RegisterMXBean - Default MXBean Initialized.
13:47:50.544 [main] INFO  o.n.monitor.agent.MonitorServer - Service URL:service:jmx:rmi://<IP_ADDRESS>:8088/jndi/rmi://<IP_ADDRESS>:8088/jmxrmi is initiated.
13:47:50.634 [main] INFO  o.n.m.collector.DataCollectManager - SystemDataCollector started.
13:47:50.635 [main] INFO  o.n.m.collector.DataCollectManager - Collection interval : 1s).


Documentation on how to handle conflicts with ember-pouch

Being banging my head against this problem, has it seems there is no good way to do that ( where to put a hook to get the conflict notification, and what to do ).
For the what to do part, it seems fairly simple using pouchdb documentation ( even if they are not using any sample of three way merging ) but I think a complete sample would be a good idea.
I try to override the save method inside my models, and use :
save() {
    _super().then(
        (result) => { return result },
        (err) => { return this.handleConflicts() }
    )
}

But when I arrive there, to get the conflicts for the document I need to re-generate the id format that relation pouch is using, and it seems a bit of a bad idea.
If anyone have some pointer to that, that would be super cool.

Not sure if it would help, but pouch-relational does expose methods for converting to and from its special IDs: db.rel.parseDocID and db.rel.makeDocID.
(FWIW, in my app, data is updated locally and then synced. This means that conflicts only happen during replication, not when saving. I handle replication conflicts using a view like the one in the CouchDB guide to find conflicts after replicating.)

Thank for the tip ! Would still be good to have a way to know if there was any conflict that happened during save ( in case of ). But yeah, a backend process sounds like a better idea anyway.
I still think a bit of documentation on how to handle that ( or at least a link in the documentation ) would be a good idea :)


Handle $tslint5 and gulp errors

Repro:

Clone VS Code
Open in insiders
🚨TSLInt 5 Problem Matcher not found if you didn't install TSLint extension yet
🚨gulp errors:Error: The gulp task detection didn't contribute a task for the following configuration:
{
    "type": "gulp",
    "task": "tslint",
    "label": "Run tslint",
    "problemMatcher": [
        "$tslint5"
    ]
}

This is not good for people who are contributing VS Code for first time, or who just cloned a copy of VS Code without deps.

@octref I'm not seeing the gulp error, where is that one?
For the tslint5, that seems like a helpful error to me. One of the tasks that we expect people to use during development is the Run tslint task so letting people know that something is missing seems reasonable.
Something we could do is move the Run tslint task into @egamma TSLint extension.

IMO I shouldn't be greeted with such an error with I open VS Code source. It would be great if this error can be handled gracefully instead of showing up in the Output channel (this opens up Outuput channel).

We should annotate tasks with errors like this then only send the errors to output when that tasks is run.


gmaps plugins just by weights

How can I ignore the proximity between points and build the heatmap only with weights?

@ifreddyrondon I'm afraid heatmap.js can't help you with that because it heavily relies on overplotting. what you're looking for is a multivariate kernel density estimation


bugfix: restrict upper version of tensordict

Description
This is temporal patch for #229
Motivation and Context
Work around to avoid #229 , but this change restrict upper-version of main dependency.

[x] I have raised an issue to propose this change (required for new features and bug fixes)

Types of changes
What types of changes does your code introduce? Remove all that do not apply:

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds core functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Documentation (update in the documentation)
[ ] Example (update in the folder of examples)

Checklist
Go over all the following points, and put an x in all the boxes that apply.
If you are unsure about any of these, don't hesitate to ask. We are here to help!

[ ] My change requires a change to the documentation.
[ ] I have updated the tests accordingly (required for a bug fix or a new feature).
[ ] I have updated the documentation accordingly.

https://github.com/ai4co/rl4co/issues/229#issuecomment-2428284958


[ Android ] onInstallConversionData cannot be listen

Report
The event is not listening to the onInstallConversionData on Android.
(I checked working onInstallConversionData on 1.4.4)
Plugin Version
1.4.5
On what Platform are you having the issue?
Android
What did you do?
appsFlyer.onInstallConversionData(appsFlyerMetadata => {
  console.log("onInstallConversionData is called");
});

What did you expect to happen?
On "Debug JS Remotely", printing words on Web Console.
What happened instead?
Nothing happens.
Please provide any other relevant information.
I changed code as the following, and onInstallConversionData became working.
android/src/main/java/com/appsflyer/reactnative/RNAppsFlyerModule.java
  private void handleSuccess(String eventType, Map<String, String> data) {
      JSONObject obj = new JSONObject();

      try {
          obj.put("status", afSuccess);
          obj.put("type", eventType);
          obj.put("data", new JSONObject(data));
          if (eventType.equals(afOnInstallConversionDataLoaded)) {
-             sendEvent(reactContext, afOnInstallConversionData, obj.toString());
+             sendEvent(reactContext, afOnInstallConversionDataLoaded, obj.toString());
          } else if (eventType.equals(afOnAppOpenAttribution)) {
              sendEvent(reactContext, afOnAppOpenAttribution, obj.toString());
          }
      } catch (JSONException e) {
          e.printStackTrace();
      }
  }

Maybe it needs to fix sendEvent's argument ( afOnInstallConversionData => afOnInstallConversionDataLoaded ) because it changed the code as the following at 1.4.5.
index.js
- const listener = NativeAppEventEmitter.addListener(
-   "onInstallConversionData",
+ const listener = appsFlyerEventEmitter.addListener(
+   "onInstallConversionDataLoaded",

https://github.com/AppsFlyerSDK/react-native-appsflyer/compare/v1.4.4...v1.4.5#diff-168726dbe96b3ce427e7fedce31bb0bcL161-R257

I confirmed the onInstallConversionData listener is working after applying the above changes.
This bug is also affecting the deferred deep-link feature on the Android since it based on onInstallConversionData event.


Add new open source project

Name
Link

Covid-19 Dashboard
https://github.com/tehkaiyu/covid-19 (https://covid-19.kyh.io/)

Open source project using React + Tailwind

[x] My item is in the right category
[x] My item is logically grouped below similar items
[x] My item's name and description respects the conventions of the list
[x] My item is awesome
[x] I have read and followed the contribution guidelines

Looks really nice, but in order for the category to not be filled with newly-created and not-so-awesome projects, we now require that they're somewhat popular before accepting them.
Sorry, and feel free to pull request again if your project gets some more attention though!


readme atualization

i`ve added a new button on the principal screen

Wich button on what part of the screen?


How to show re-exports as a normal export

Search terms
re-export, module

Question
Hi, I have a function called buildContext that is exported twice in both index.ts (called Search) & recommendation.index.ts (called Recommendation). Since index.ts is above recommendation.index.ts in the entryPoints, typedoc shows the buildContext function in the  Search tab. In the Recommendation tab, that function is in References. How can I duplicate that definition in both tabs ?
Thank you !

Each of our entryPoints is meant to be used separately, never at the same time, in the same file. We have a lot of the same root functions and interfaces in every entry point. What typedoc setup do you recommend for our use case ? Ideally, the doc would get duplicated for each entryPoints based on what is exported in the entry files.
This is our typedoc config :
{
  "navigation": {
    "includeCategories": true,
    "includeGroups": true,
    "includeFolders": true
  },
  "categorizeByGroup": true,
  "groupOrder": [
    "Engine",
    "Controllers",
    "Search controllers",
    "Buildable controllers",
    "Sub-controllers",
    "Core controllers",
    "Actions",
    "*"
  ],
  "entryPoints": [
    "./src/core.index.ts",
    "./src/index.ts",
    "./src/case-assist.index.ts",
    "./src/commerce.index.ts",
    "./src/insight.index.ts",
    "./src/recommendation.index.ts",
    "./src/ssr.index.ts"
  ],
  "gitRevision": "typedoc-POC",
  "plugin": ["./typedoc/dist/index.js"]
}

Thanks a lot.

Hmm, this makes me think that the excludeReferences option maybe ought to be moved to a references enum which supports Exclude, Duplicate and Reference option.
Currently to do this with TypeDoc you'd have to generate JSON outputs for each entry point separately, and then use entryPointStrategy: "merge" to combine them into one HTML site. (This is what TypeDoc's entryPointStrategy: "packages" does under the hood)

Thanks ! This helped a lot. I split my config into all sub packages outputing json files and had a root config combining those json files !


Prepare for big merge

Preparing for the big merge as discussed in #385
I've removed everything except the LICENSE.md so we don't remove original authors coyright.
@lifeart can you solve the conflict by adding yourself to the list when creating your PR

Before I click the button does anyone see any issues with doing this?
@nullvoxpopuli @matchy @locks

@evoactivity yep

I've created an ember-tooling organisation on npm, I've invited @nullvoxpopuli and @lifeart to that organisation.
@lifeart we'll need to update the workflow so it publishes to that npm organisation
If there are others who should have access let me know.

@lifeart is there a reason this https://github.com/lifeart/ember-language-server/blob/component-context-info-origin/.github/workflows/ci.yml#L131 doesn't use the standard secret.GITHUB_TOKEN ?

@evoactivity I think it's legacy from https://github.com/lifeart/ember-language-server/commit/81728b6b77e9dc005e4e5253fb24e6eb1ac2bc22

Before I click the button does anyone see any issues with doing this?
@nullvoxpopuli @matchy @locks

you're the boss!

if only every PR could be like this :sweat_smile:


Code for Creating Training Datasets of Metaworld

Can you provide the scripts to generate videos for training the video diffusion models and the corresponding datasets used for training the checkpoints?

Hello, the dataset is released here
https://huggingface.co/Po-Chen/flowdiffusion/tree/main/datasets


feat: enhance the plugin status command

closes: #3011
While working on this issue, I found that we have an opportunity to reduce the response time  of status API by the way used in this PR https://github.com/cloudnative-pg/cloudnative-pg/blob/af347f76cd234a1f715393c62fb6915904f7d159/pkg/management/postgres/probes.go#L66-L73
Opened a new issue #3121 for tracing it

@YanniHu1996 to fix the linter issues you should run golangci-lint run --fix

/test limit=local

/ok-to-merge

We've problems with the PRs opened from repositories other than cloudnative-pg/cloudnative-pg. The build containers step is failing. See #3154


Update RosettaCode Page

<TimToady> if someone is hankering for some low-hanging fruit, the page at
https://www.rosettacode.org/wiki/Category:Perl_6 could use some updating, since
it still reads as if Perl 6 is not yet released

https://www.rosettacode.org/wiki/Category:Perl_6

Resolved by TimToady: http://irclog.perlgeek.de/perl6/2016-07-11#i_12821228


Create w_amq.h

typedef void (*w_call_amq)(
		std::string pMessage);

we can't use std in c files. change it to const char*


Feature/add auto to y axis new approach

Closes https://github.com/gisce/webclient/issues/495
Replaces https://github.com/gisce/react-ooui/pull/542

:tada: This PR is included in version 2.14.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Create Daily Standup Template

As a DevOps Team Member
I need an organized routine for conducting short standup meetings.
So that we do not waste time on single issues, we are never waiting to be assigned issues, we can identify who has bandwidth to take on more work, and we can assist those who are struggling.
Assumptions:

...Everyone follows the template and sticks to two minute reports.

Acceptance Criteria:
Given ... this standup template for organizing DevOps meetings and a two minute time limit on reporting
When ... implementing this structure in our daily 10 minute standup
Then ... we will stay on track during our team meetings, avoid wasted time caused by tangents, avoid focusing the majority of our time on single issues, hold ourselves in check to ensure we are always working on something, and better plan for longer, detailed meetings.

This will reside in slack. It's pinned.


Trying to migrate to mysql backend

[x] I have checked the superset logs for python stacktraces and included it here as text if any
[x] I have reproduced the issue with at least the latest released version of superset
[ ] I have checked the issue tracker for the same issue and I haven't found one similar

There are similar issues, but none recent or using the same version.
Superset version
Superset 0.22.1
Expected results
Actual results
I can initialize the mysql database, create an admin user, but the web interface does not seem to be using it.
Steps to reproduce
#/home/$USER/.supserset/superset_config.py
SQLALCHEMY_DATABASE_URI = 'mysql://superset:$password@$host/superset'

export PYTHONPATH="/home/$USER/.superset/:$PYTHONPATH"
rm ~/.superset/superset.db
fabmanager create-admin --app superset
superset db upgrade
superset load_examples
superset init

➜  ~ superset runserver
Loaded your LOCAL configuration at [/home/jdr400/.superset/superset_config.pyc]
Starting server with command:
gunicorn -w 2 --timeout 60 -b  <IP_ADDRESS>:8088 --limit-request-line 0 --limit-request-field_size 0 superset:app

[2018-02-23 09:20:57 +0000] [4060] [INFO] Starting gunicorn 19.7.1
[2018-02-23 09:20:57 +0000] [4060] [INFO] Listening at: http://<IP_ADDRESS>:8088 (4060)
[2018-02-23 09:20:57 +0000] [4060] [INFO] Using worker: sync
[2018-02-23 09:20:57 +0000] [4065] [INFO] Booting worker with pid: 4065
[2018-02-23 09:20:57 +0000] [4066] [INFO] Booting worker with pid: 4066
Loaded your LOCAL configuration at [/home/jdr400/.superset/superset_config.pyc]
Loaded your LOCAL configuration at [/home/jdr400/.superset/superset_config.pyc]

If I check the mysql database I see the admin user as expected.  But then when I go to login to the web interface I get either an 'Internal service error' or a long stack trace that indicates superset is still trying to use the superset.db sqlite database.
 File "/usr/local/lib/python2.7/dist-packages/sqlalchemy/engine/default.py", line 470, in do_execute
    cursor.execute(statement, parameters)
OperationalError: (sqlite3.OperationalError) no such table: ab_role [SQL: u'SELECT ab_role.id AS ab_role_id, ab_role.name AS ab_role_name \nFROM ab_role \nWHERE ab_role.name = ?\n LIMIT ? OFFSET ?'] [parameters: ('Public', 1, 0)]

I sort of figured this out. I was able to do a fresh install of superset on a different server, and all works as it should. ;-) There must be a config file somewhere that is pointing to the sqlite db.


Adds documentation for form and plain old ruby objects

I formatted the markdown files for line length and added some documentation:

Shows how to customize action and method on the form
Shows how to use plain old ruby objects as form objects, minimum to implement

Dropped commit for explaining namespaces and collections as requested. Applies line length format to your new documentation changes and rebases them in.

I can't review this commit because the manual line breaks make it impossible to see the changes. Could you please update the PR without manual line breaks? If you're not up for it then you may also close this issue.
Thanks!


method-name startsWith 'set' have higher priority than method-name with non-startsWith 'set',for issue #2230

What this PR does / why we need it?
method-name startsWith 'set' have higher priority than method-name with non-startsWith 'set',for issue #2230
Summary of your change
Please indicate you've done the following:

[x] Made sure tests are passing and test coverage is added if needed.
[x] Made sure commit message follow the rule of Conventional Commits specification.
[x] Considered the docs impact and opened a new docs issue or PR with docs changes if needed.

看下是否合适


Add flag to executable for random seed

A flag should be added to the executable to have it use a randomly generated seed string, rather than having the user supply a seed string.

After some consideration, I decided to change this to be the default behavior of the executable.


Add NTUA

https://www.ntua.edu.tw/

@a26007565 lib/domains/tw/edu/ntua: this request has an incorrect file path format, please ensure that you are familiar with     this repository rules located at the bottom of this page and change     your request accordingly. If you don't know how to change request, close it and open a new one.     If you are not able to make the pull request according the instructions, please send us an email to<EMAIL_ADDRESS>Please mention this request URL in the message.

@a26007565 Please provide us with an URL of a page on the official website where the school offers an IT-related long-term (>1 year) course

@a26007565 I am sorry, this request remains unanswered for a while. I have to close it without merging. Please reopen and comment if you have an update regarding the aforementioned issue. Thank you for your understanding!


Packaging stuff

Hey there, cool template! A couple of points:

uv venv is not needed here.
This is the hardest: publishing packages in the way it's explained here via access tokens is discouraged. You should follow the guide here. I never did this, tho.
The scripts under [project.scripts] here won't work since the project is not configured as a library, but rather as an application (see an explanation here. In general, it mostly comes down to personal preference: you can always design your REST API as a library).

The solution to 3 is relatively easy: you should just add a [build-system] section to pyproject.toml. In general, it's a convention to use - over _ for CLI scripts. This will also make the development section of the readme much simpler: you can remove all of the following lines:

```bash
uv pip install -e .
```

You can now run, for example, a function defined as `scripts` in the [`pyproject.toml`](pyproject.toml)

And just let people clone the repo and run uv sync. (At this point, technically you wouldn't even need the main.py, though you can leave it as an example for people building a streamlit/fastapi app I guess.)

Thank a lot, will have a look at adjust!

Done via #10 , feel free to have a look and then close if you think I fix everything 🤗


Deprecated "traceValIfNot" in nixpkgs-config option.

When running any home-manager command or using the nixos module I get the following message:
trace: Warning: 'traceValIfNot' is deprecated and will be removed in the next release. Please use 'if/then/else' and 'traceValSeq' 1.
traceValIfNot seems to only be called here so I suspect that line is the problem. Maybe this should be fixed?

Thanks for the report! I've replaced the use of traceValIfNot in master and hopefully you won't see the warning after your next update 🙂

I can confirm that I do no longer see the warning. Thank you for the quick fix!


Failed to find file  during the test(TEST_ID_25)

During some of the tests we see the following error :
Error: open ../helm/secrets-provider/ci/take-default-test-values.yaml: no such file or directory
link Logs

Fixed as part of this ticket


pass application_fee to charge

While attempting to just simply do a customer.charge(amount), I'd get an error back from Stripe:

Can only apply an application_fee when the request is made on behalf of another account (using an OAuth key, the Stripe-Account header, or the destination parameter)

Noticed application_fee was being passed in to the charge by default, but the StripeCharge wasn't using the passed in value. It was the same exact value as amount. I changed it to int(application_fee * 100) if application_fee is passed in, but not 100% sure if this is what it wants. This fixed my issue and I can now charge successfully.

Current coverage is 100% (diff: 100%)

Merging #382 into 1.0.0 will not change coverage

@@           1.0.0   #382   diff @@
===================================
  Files         23     23
  Lines       1536   1536
  Methods        0      0
  Messages       0      0
  Branches     183    183
===================================
  Hits        1536   1536
  Misses         0      0
  Partials       0      0

Powered by Codecov. Last update 4fe9b77...78e6137


GUI

A GUI is severely needed. Plus I want to test issues and stuff.

[x] Research
[x] Select platform
[ ] Come up with designs
[ ] Decide on design
[ ] Creating the design on the chosen platform
[ ] Getting the code to work with it
[ ] Debugging
[ ] Cleaning up
[ ] Release

@all-contributors please add Thearchermancoding for code, data, design, ideas, maintenance, and projectManagement


Sorting on overlay array merge

Describe the problem/challenge you have
After patching, of a yaml array, the element are not in alphabetics order. It would be nice to have an option to maintains an order
For now I have a patch that looks like that. The goal is to ensure (create if missing) that a key is inside a yaml array.  But with this code, the yaml generated is not in a good order.
#@ load("@ytt:data", "data")
#@ load("@ytt:overlay", "overlay")
#@overlay/match by=overlay.subset({"kind": "Kustomization"})
---
resources:
  #! we use by=lambda to find equality of values
  #@overlay/merge
  #@overlay/match missing_ok=True,by=lambda indexOrKey, left, right: right in left
  - #@ "{}.{}.yaml".format(data.values.namespace,data.values.project)

Solution
Maybe there would be a way to add sorting ?

@AyWa ty for your question!
Overlays usually merge by appending or replacing in-place.  That's because the user-defined order may be arbitrarily complex. Alphabetical order is only one possible order.
To achieve your desired outcome, my suggestion is to instead replace the parent node via a custom function. That way you can control whichever sorting you require.
I have prepared an example on the playground for you. (Alternatively, go to its gist directly.)
It produces:

kind: Kustomization
resources:
- amazing.coffee.yaml
- boring.stuff.yaml
- fun.mergemania.yaml
- zebra.zulu.yaml

Let me know if that helps you.


Omnisharp not included in Linux build

Our linux build machine does not include the csharp-o/bin folder.

Running scripts/npm.sh install -> csharp-o/bin folder nicely gets created on my linux machine.
Running gulp vscode-linux-x64 also nicely creates the csharp-o/bin folder on my linux machine.

Something is strange on our build machine

Built from source (commit 9c4c52b0d38c23b62708441f564db128f7923183) minutes ago, no Omnisharp.
Since the released binary for linux does not provide Omnisharp and I can't make it work when building on my own, I thinks this issue should be reopened.

Did you try running running scripts/npm.sh install

@isidorn Yes.  In fact, I had to play around the build instructions to make it run.  For instance manually make sure gulp was in my PATH.

@abstrus can you provide the log on the npm install run? Also, try running npm install inside the extensions/csharp-o folder


PyTorch-Tutorial/tutorial-contents/403_RNN_regressor.py contains a weird line of code

I tried some RNN regression learning based on the code in the "PyTorch-Tutorial/tutorial-contents/403_RNN_regressor.py" file, which did not work for me at all.
According to an accepted answer on stack-overflow (https://stackoverflow.com/questions/52857213/recurrent-network-rnn-wont-learn-a-very-simple-function-plots-shown-in-the-q?noredirect=1#comment92916825_52857213), it turns out that the line
h_state = h_state.data
of the tutorial file does not do what it is supposed to be doing. It does not actually break the connection to the previous iteration as specified in the commment. Rather the line
h_state = None
should do the trick (according to the accepted answer).

Indeed, I guess the comment is for the case whereby elements in batches are unrelated, like the MNIST example, however the code is correct here for continuous data.

Is the bug related to pytorch's version? The pytorch '0.4.1' runs fine.

when I use torchvision.datasets.MNIST, why occurs TypeError: init() missing 1 required positional argument: 'total'


Incorrect work pagination  Eloquent

Laravel Version: 5.4
PHP Version: 7.0
Database Driver & Version: MySQL (MariaDB 10.1)

Description:
Incorrect work pagination Eloquent from join
Steps To Reproduce:
Else run code
Eloquent::query()->distinct()
            ->select('table_one.*')
            ->join('table_two', 'table_two.owner_id', '=', 'table_one.id', 'LEFT OUTER')
->paginate(20,['table_one.id'])

then two queries will execute
select count(*) as aggregate from `table_one` LEFT OUTER join `table_two` on `table_two`.`owner_id` = `table_two`.`id`

and
select distinct `table_one`.* from `table_one` LEFT OUTER join `table_two` on `table_two`.`owner_id` = `table_two`.`id`

as the first request is incorrect, it should be of the form
select count(distinct table_one.id) as aggregate from `table_one` LEFT OUTER join `table_two` on `table_two`.`owner_id` = `table_two`.`id`

Why would it be a count?

A count query is required to calculate the total number of rows. The second query fetches the actual  data for the page. The second query looks like it is missing a LIMIT clause in this issue, but I think that is a authoring issue and not related to the reported issue.
The issue seems to be the logic that rewrites the query SELECT DISTINCT * into a SELECT COUNT(*) instead of understanding that it needs to keep the DISTINCT part.
Related: Builder::runPaginationCountQuery

@Dylan-DPC if there are 2 lines in the second table, the first request for count will return 2, but the second query will return 1 model.
compare the two functions and understand that there is a mistake
Query Builder
https://github.com/laravel/framework/blob/5.4/src/Illuminate/Database/Query/Builder.php#L1718
Eloquent Query Builder
https://github.com/laravel/framework/blob/5.4/src/Illuminate/Database/Eloquent/Builder.php#L683

I don't see this as a bug actually. It can be enhanced/improved though. Feel free to send in a PR or make a proposal on laravel/internals.

Well, it returns wrong values. Can we update the docs and state that paginate() does not support joins?

yeah that's better @sisve

it is distinct that breaks this
it looks like the regular query builder paginate method allows for passing fields to count on, but the eloquent builder always does count(*)
it looks like a fairly simple fix to mirror the behaviour of query builder (not completely sure why it isnt just delegating to that anyway tbh), but guessing changing behaviour of eloquent methods is breaking change, even if it is actually fixing a bug
@Dylan-DPC not really sure how this isnt a bug. paginator is getting the wrong value for total count, which then causes it to generate links for more pages than there are results for

Paginator isn't expected to work with joins/distincts, etc. May be it isn't documented enough.

A PR that would fix this was merged into the master branch. This will be released in the next version of Laravel.


Restore pagination information

Notably "breaking" changes here are that the registry may impose a limit on the number of tags it returns.
This was restored from b4e98332eccaf51d1974c9136caf67317bd99104, where it applied to Catalog, which was removed, but given that many registries have `adopted this behavior already, we should document it correctly.
This also restores language that clients SHOULD always use the Link header when present.
This was restored from c90b0f145ac6bc09d2636ee214486ac333edc284.

Does this text now allow a registry to return more than n results? That could be an issue for clients that assume a fixed size array for the results.

Also, guidelines on what the expected behavior is if the tag list changes in between, due to new images being pushed or deleted for that matter?

I don't think any of this is unambiguous and we should just merge it.

@jonjohnsonjr Was this PR accidentally closed, or did you have something else in mind?

I'm interpreting between the lines a little bit (so massive apologies to Jon if I'm inaccurate here), but I think it was probably closed because this isn't actually a change to the spec, but restoring content that was in the spec and got removed during refactoring (so he probably didn't want to spend too many cycles discussing/nit picking the previous behavior/design choices that are already implemented in many registries, and documenting that interop is really why this spec exists :sweat_smile:).

Got it.  Has this revert happened yet? Because the spec in its current form is currently really problematic with regards to pagination: https://github.com/opencontainers/distribution-spec/blob/main/spec.md?plain=1#L532-L533

 is the namespace of the repository, and  is an integer specifying the number of tags requested. The response to such a request MAY return fewer than  results, but only when the total number of tags attached to the repository is less than . Otherwise, the response MUST include  results.

This essentially opens up a DoS vector if the client can expect to put arbitrarily high values in <int> and the registry is expected to respond with it, with no means of pushing back with an error.

Can we re-address this with a fresh PR and then afterwards cut a 1.1.1?


allow beam.clear_env to take a "persist" option which lets you persist beam terms.

make sure that this is also reflected in beam.send

completed in c89093fde9eae4a87ef1ef8f3b5835a4f0372de8


enable blind review

this is a feature request to make it possible to do blind reviews of talks, ie. where the reviewer of submissions doesn’t see who submitted the talk. note I don’t run any conference myself but I think it would be good to make it easy for conferences to be able to do this.
that being said I know some conferences that require speakers to do 2 talks in order to be able to afford paying for travel expenses, maybe this should also be taken into account, ie. making it still possible to see if a prospective speaker submitted two interesting talks while making the final decisions on the schedule. but this is obviously a second step in this feature and maybe its not even relevant enough to even bother.

For last years DPC we forked opencfp to add this kind of functionality. We added a custom review 'module' that allowed reviewers to cycle through talks, while having the option to hide certain information. For example, hiding all speaker details.
Unfortunately due to the architecture of opencfp having changed quite a bit since then it will be hard to merge this feature back in its current state.
However, if you are interested, you are free to look at our implementation, and rebuild it (hopefully a bit cleaner than my version), for the current architecture of opencfp.
If you have any questions about its implementation, feel reach out to me.

@chartjes you closed this because you don’t have intention to add it .. or you are just closing it because its stale and you would still consider a PR?

@lsmith77 Just cleaning up old issues lingering around. Would definitely consider a PR for it.


Migrate to github actions

Workflows have been added, however tests are failing in GitHub actions, this appears to be it unable to fetch from reddit, so might need to mock the fetch early


[feat]: Expanded LiT service logs to surface more from underlying LiT

Prerequisites

[X] I have searched for existing issues that already suggest this feature.

Describe the Feature Request
To surface more of LiTs various logs in those logs displayed is the StartOS UI.
Describe the Use Case
A user had swaps go missing, and in seeking help from LiT he had to search around for swap logs after first setting up SSH and learning how to use scp. If these logs appeared within the service logs, he'd have been able to access them immediately.
Describe Preferred Solution
No response
Describe Alternatives
No response
Anything else?
No response

Can you confirm specifically which log files you would like to see exposed?


dhcp6c - Use Reasons

Use the REASONS given by dhcp6c when it calls its script. This then only calls newipv6 when 'REQUEST' is the reason.
RENEW, REBIND or INFO do not have changes to the leases, therefore no call to update is needed and this prevents reloading which can affect VPNs, Unbound etc.
On RELEASE or EXIT, once again newipv6 is called as the lease addresses should have been removed by dhcp6c.

New Hard Drive,  let's see if it's more reliable now!

Ok, that worked, should have removed the offending articles.   :)

Thanks, as usual, a bit of cleanup follows.

https://github.com/opnsense/core/commit/b5f0902407c52ecead0c8a8d613e7a350b6e665c
I have a small question regarding RELEASE and EXIT flipping: shouldn't we always adhere to RELEASE and only add EXIT to the mix if we have norelease unset?


Add the ability to restart temporarily shutdown instances

Instances shut down after 2 weeks, but there's not currently any way to restart them. This should be figured out.

This is now possible as part of the new Kubernetes-based system :)


Where are CentoOS based images?

Hi everyone
First of all my apologies if I'm not asking in the right place, but I've found something that  concerns me a little bit. I'm using this image available in dockerhub: jenkins/jenkins:2.291-centos, but I can't find here its source project.
Am I doing something wrong? Has something happened with CentOS based versions? Do I have to move to Debian based? I don't feel completely comfortable not knowing what is included in the image I'm using.
Thanks in advance and again my apologies. I will appreciate if you kindly address me to the right place to ask.

Here is the dockerfile for that image: https://github.com/jenkinsci/docker/blob/master/8/centos/centos8/hotspot/Dockerfile

Hmm, I'm sorry but the image you address doesn't contain the Jenkins versión I mentioned (2.291). In other words what I can find in dockerhub  seems to not being referenced here in github (at least as far as I could see)

The Jenkins version is a variable in the Dockerfile. It is overridden during the build process on dockerhub builds.
https://github.com/jenkinsci/docker/blob/master/8/centos/centos8/hotspot/Dockerfile#L53
https://github.com/jenkinsci/docker/blob/master/8/centos/centos8/hotspot/Dockerfile#L57
https://github.com/jenkinsci/docker/blob/master/8/centos/centos8/hotspot/Dockerfile#L60

Nice!
Thanks a lot for your information! You really helped me
Have a nice day. Best regards!


[feature request]modify yurt-manager client's default qps and burst

What would you like to be added:
The default qps and burst of kube-client in yurt-manager component are defined as follwoing:
https://github.com/openyurtio/openyurt/blob/299c5f97dcfbcb32ebacf997e1f4afb12432e759/cmd/yurt-manager/app/options/generic.go#L50-L51
because more and more controllers are added into yurt-manager, and these values is not enough for yurt-manager to access kube-apiserver, so it is a good idea to up these initial values as followin:
 RestConfigQPS:    50,
 RestConfigBurst:  100,

others
/kind feature

/assign @lilongfeng0902


Icalendar support (ICS)

Hi,
I wanted to integrate this into my Android phone calendar and I was missing ICS (did not see it).
It would be nice to have a static ICS link to subscribe to.
Personally, I also maintain https://open-web-calendar.hosted.quelltext.eu
So, I was wondering whether you would like to embed a calendar that shows the events in the time zone of the viewer.
I don't know - these are my thoughts. What do you think?
Basically, I subscribe to this on Github to not forget about it but then, I do not like to read all the pull requests. (my personal problem)

Sounds like a feature I'd like to have too, viewing all events in my Calender via a Subscribed iCal would be great!

Thansk @niccokunzmann & @prathamshroff -- feel free to send a PR!

The framework is react. One might define this next to index.js.
There is already api/events.js and index.js iterates over the events.
The only addition needed is something that creates ICS.

Hey @niccokunzmann! I appreciate your research! I also didn't build the initial site, so your investigations are helpful :)


Allow direct values in obect dictionary on 16bit microcontrollers

Reasoning
Closes #66
In 16bit or 64 bit processor uintptr_t does not match uint32 size. This leads to silent failure in 16-bits. With this change the internal representation is moved to an union type.
Changes

Added test to Object Directory API
Change type of OD data to union type
Add ability to compile on Mac

Observations

All unit-test pass

I'm not completely happy with the change either. It does seem to be more portable, but does potentially break things. Maybe there are other options that would not break compatibility. Open to trying other approaches.

Sorry for the long delay. I'm wondering if this still makes sense or not. I still would like to try in on 16bit INTs but I'm not sure this is the way.

Hi, I recently tried a simple approach - but cannot test the implementation. I'm working with 32bit MCUs only. Can you check, if the implementation in branch develop-16bit-mcu is working on your machine, please?

I'm working with MCU which has 16-bit pointers and this stack works fine as it is. Only exception is that instead of direct access entries I'm using pointers almost exclusively.

This pull request is replaced by PR #111

Grmpfl, the "better approach" is not compiling on some compilers (explicitly the AVR-GCC).
So I try to merge your proposal into the stack. It is currently in the develop branch. If you may have some time - could you have a look on it?
Sorry and "Mea culpa" for using the simple approach - now I have installed AVR-GCC and can at least ensure, that the stack compiles for small microcontrollers.

I tested the development branch and it worked with my old project. Did compile I need to add some of the new handlers, but after that its working on my PlatformIO build.


about dataset ensemble

Hello, I would like to know if the labels for using these datasets are consistent? For example, for the task of segmenting lung lobes, in dataset A, its label ID is 1, while in dataset B, its ID is 3. Does the difference between these two IDs have an impact on the training process? If there is an impact, how do you handle unifying it?

The direct answer is no.
You can directly merge different datasets without changing the label id.

The direct answer is no.
You can directly merge different datasets without changing the label id.

Thank you for your reply. I also want to ask, how is the training process for a dataset with multiple labels? Is each label treated as the foreground and the rest as the background separately, and are slices made for each label separately？

how is the training process for a dataset with multiple labels? Is each label treated as the foreground and the rest as the background separately

Not exactly. MedSAM is box promptable method, the label is associated the given box prompts.

are slices made for each label separately？

No. the data loader will automatically sample labels from the multi-label mask.


[maven] add sh version of mvn to bin

Apache Maven works just find.
Is it possible to add mvn which has no extension along with mvn.cmd?
I tend to use cygwin which is a linux emulation and it's not possible to use mvn command.
alias mvn=mvn.cmd doesn't expand in non-interactive shell.

I prefer using MINGW most of the time, and noticed the same for Yarn, Gradle and others apps, which don't have an EXE-File.
@jinahya The solution described in the "Can I use Scoop in bash?" wiki page works here as well. Just replace scoop.ps1 with mvn.ps1, for example.

FIY, I put alias mvn="powershell -ex unrestricted mvn.ps1" into ~/.bashrs and invoking mvn launches a new command console window. Thanks, anyway.

This issue belongs in the Main bucket. Unfortunately, we cannot move this issue on this end, so it is being closed.
If this is still an issue, please create a new issue in the Main bucket via https://github.com/ScoopInstaller/Main/issues/new?body=Moved+from+https%3A%2F%2Fgithub.com%2Flukesampson%2Fscoop%2Fissues%2F1237


[Ergonomy] Design proposal for sending XDAG

Here is a mock-up of how I imagined a redesign of "Send XDAG" interface.
The goal is to be as "click & play" as possible and easy to understand.
Here is the default screen that uses the idea proposed here to choose a destination address in its contacts, but also in its wallets. And if not, to enter manually the exit address.

A "drop down" menu would appear if you press "My Contacts".

Same operation if the user presses "My XDAG wallets". In this case, the top menu is retracted, and if he chooses a wallet, he can press "Continue".

This is a mock-up, and I would be honored if the developer of XDAG-Pro would like to use it as inspiration for a more intuitive experience of his program.

This feature has been added to the latest version


K8s context switcher ping redirected notify to useNotification.

Notes for Reviewers
This PR fixes #8617
In the code base,
Added Line (to Header.js):

Removed Line(from Header.js):

Video: Notification is working but in the video proper message has not come, As my Kubernetes Config is not valid.
https://github.com/meshery/meshery/assets/107252411/21f82ac7-dd4a-4e26-b086-d5805a617d18
Signed commits

[ ] Yes, I signed my commits.

@Rajdip019 does this follow suit with our latest approach to notifications? // @aabidsofi19

Have we documented that approach?

@Rajdip019 does this follow suit with our latest approach to notifications? // @aabidsofi19

Yes, this does comply with the new useNotification hook.
Here is a documentation
Notification.Center.pdf


Skillmap completion "Edit Your Project with a Full Toolbox" text doesn't fit in button

Describe the bug
When I am going through the Skillmaps, the instructions that show sample code will cut off the right side of the code with no ability to left/right scroll
To Reproduce
Steps to reproduce the behavior:

Go to 'https://arcade.makecode.com/beta--skillmap#docs:skillmap/balloon'
Finish/Step through the first 3 tutorial nodes and get to the "You Did It!" screen
See the "Edit Your Project with a Full Toolbox" text cut off in the instruction panel

Expected behavior
For the text to fit in the button space
Screenshots

Desktop (please complete the following information):

OS: Windows
Browser Edge
Version 103.0.1264.62 (Official build) (64-bit)

Additional context
My screen resolution is 2560x1440 on a 27" monitor

@kiki-lee I believe your changes in https://github.com/microsoft/pxt-arcade/pull/4933 fixed this issue by reducing the text inside the button. Okay for me to close?

Sure.  Also, I believe that Alex wants to take this whole modal in a different direction, so stay tuned!


Add developer getting started guide (#2837)

Pull request opened by the merge tool on behalf of #2837

Merged into branches iso4 in d4a72ae703a00c2d7215aff3f9e90bb551081dbe


The easiest way to start for newbie

Hey!
I wanted to run uchan on Linux, but it fails on build. I replaced docker-compose with docker compose in Makefile, and then run it with command make setup. It shows warning and after some building steps finally crashes:
WARN[0000] The "UCHAN_PORT" variable is not set. Defaulting to a blank string.
[+] Building 0.0s (0/1)
[+] Building 0.0s (0/2)
[+] Building 0.2s (2/3)
 => [internal] load .dockerignore                                                                                0.0s
 => => transferring context: 216B                                                                                0.0s
 => [internal] load build definition from Dockerfile                                                                                0.0s
 => => transferring dockerfile: 788B                                                                                0.0s
[+] Building 0.2s (2/3)
 => [internal] load build definition from Dockerfile                                                                                0.0s
 => => transferring dockerfile: 788B                                                                                0.0s
 => [internal] load .dockerignore                                                                                0.0s
 => => transferring context: 216B                                                                                0.0s
[+] Building 1.1s (3/12)
 => [internal] load .dockerignore                                                                                0.0s
 => => transferring context: 216B                                                                                0.0s
 => [internal] load build definition from Dockerfile                                                                                0.0s
[+] Building 1.0s (3/12)
 => [internal] load build definition from Dockerfile                                                                                0.0s
 => => transferring dockerfile: 788B                                                                                0.0s
 => [internal] load .dockerignore                                                                                0.0s
[+] Building 1.2s (3/12)
 => [internal] load .dockerignore                                                                                0.0s
 => => transferring context: 216B                                                                                0.0s
 => [internal] load build definition from Dockerfile                                                                                0.0s
[+] Building 1.1s (3/12)
 => [internal] load build definition from Dockerfile                                                                                0.0s
 => => transferring dockerfile: 788B                                                                                0.0s
 => [internal] load .dockerignore                                                                                0.0s
[+] Building 1.2s (6/12)
 => [internal] load .dockerignore                                                                                0.0s
 => => transferring context: 216B                                                                                0.0s
 => [internal] load build definition from Dockerfile                                                                                0.0s
 => => transferring dockerfile: 788B                                                                                0.0s
[+] Building 1.2s (7/12)
 => [internal] load .dockerignore                                                                                0.0s
 => => transferring context: 216B                                                                                0.0s
 => [internal] load build definition from Dockerfile                                                                                0.0s
 => => transferring dockerfile: 788B                                                                                0.0s
[+] Building 1.2s (7/12)
 => [internal] load build definition from Dockerfile                                                                                0.0s
 => => transferring dockerfile: 788B                                                                                0.0s
 => [internal] load .dockerignore                                                                                0.0s
 => => transferring context: 216B                                                                                0.0s
 => [internal] load metadata for docker.io/library/alpine:3.6                                                                                0.9s
 => CANCELED [1/8] FROM docker.io/library/alpine:3.6@sha256:66790a2b79e1ea3e1dabac43990c54aca5d1ddf268d9a5a0285e4167c8b24475                   0.2s
 => => resolve docker.io/library/alpine:3.6@sha256:66790a2b79e1ea3e1dabac43990c54aca5d1ddf268d9a5a0285e4167c8b24475                            0.0s
 => => sha256:66790a2b79e1ea3e1dabac43990c54aca5d1ddf268d9a5a0285e4167c8b24475 1.41kB / 1.41kB                                                 0.0s
 => => sha256:36c3a913e62f77a82582eb7ce30d255f805c3d1e11d58e1f805e14d33c2bc5a5 528B / 528B                                                     0.0s
 => => sha256:43773d1dba76c4d537b494a8454558a41729b92aa2ad0feb23521c3e58cd0440 1.51kB / 1.51kB                                                 0.0s
 => => sha256:5a3ea8efae5d0abb93d2a04be0a4870087042b8ecab8001f613cdc2a9440616a 0B / 2.02MB                                                     0.2s
 => [internal] load build context                                                                                0.2s
 => => transferring context: 16.10MB                                                                                0.2s
 => CACHED [2/8] RUN apk update &&    apk add python3-dev             uwsgi uwsgi-python3             build-base bash             libffi-dev   0.0s
 => ERROR [3/8] ADD requirements /opt/app/requirements                                                                                0.0s
------
 > [3/8] ADD requirements /opt/app/requirements:
------
failed to solve: failed to compute cache key: failed to calculate checksum of ref moby::1z31onjgqy8fbxcs0mkshwqzz: "/requirements": not found
make: *** [Makefile:26: start] Error 17

I assume that README.md is not up to date while the last update was 4 years ago. Can you help me with easy start?

I recommend waiting for a bit (famous last words ;))
I'm writing a Helm chart, and the docker-compose stack will be removed (or only for development purposes). With the Helm chart, you'll be able to deploy µchan and it's dependencies on Kubernetes.

@floens ok, I will wait for updates

I published the Helm charts to the repo, and I've started with proper documentation. There's just one page now, but I hope that it will be useful:
https://floens.github.io/uchan/installation.html

@floens thanks, I'll try it in my free time and give you feedback


Relicense to dual MIT/Apache-2.0

This pull request was automatically generated because my scripts detected that
everyone in the relicensing issue agreed to
relicensing.
License headers in files were not updated, if they exist. I've reproduced an
example license header below if you want to copy-paste it in. If you don't
have license headers, don't feel compelled to add
them.
// Copyright 2016 optional Developers
//
// Licensed under the Apache License, Version 2.0, <LICENSE-APACHE or
// http://apache.org/licenses/LICENSE-2.0> or the MIT license <LICENSE-MIT or
// http://opensource.org/licenses/MIT>, at your option. This file may not be
// copied, modified, or distributed except according to those terms.

I recommend publishing a new patch version (the z in x.y.z) so that the
metadata propagates to crates.io.
Please be sure to check the README.md for style. I've try to
review these PRs as they go out, but I might miss something.
Thanks for relicensing!
(For my bookkeeping: this is repository id optional-7dd6b2ce-71e8-4cf9-83dd-76875560742a)

Thank you!


Resolve the column selection conversation

As a fairly new user of Visual Studio Code, and like apparently many other users, I found column selection in vscode to be confusing and unnatural.
As I went to look for help in the issue tracker, I was blown away with how many problems and issues other users have had with this:
#1515 - Column selection
#2789 - Feedback: Missing Plugins
#3130 - Allow customization of mouse shortcuts
#5402 - Improved column selection (Alt+select)
#5851 - Give option to change the key used for multi-selection
#11830 - Column mode selection behavior is inconsistent with user expectations
#21785 - Not able to change keybinding for column select
#27193 - Allow for ctrl+click to create multiple cursors
#27480 - Test editor.multicursorModifier
#27503 - editor.multicursorModifier: cmd should be the default
#27680 - editor.multiCursorModifier ctrl/cmd settings are not portable between OS
#28186 - Column Selection by Middle Mouse Button (Sublime Style)
#36496 - Feature Request: Columns and rows selection like Visual Studio
#41728 - Right clicking column selection cancels selection
#46980 - Column selection in Ubuntu
#64661 - column select mode
I believe there are two small tweaks to resolve the problem in a natural and intuitive way:

Set editor.multiCursorModifier to ctrlCmd
Modify the code in src/vs/editor/browser/view/viewController.ts to:

diff --git src/vs/editor/browser/view/viewController.ts src/vs/editor/browser/view/viewController.ts
index 0279da5670..e5f8873eb5 100644
--- src/vs/editor/browser/view/viewController.ts
+++ src/vs/editor/browser/view/viewController.ts
@@ -193,7 +193,11 @@ export class ViewController {
                                }
                        } else {
                                if (data.inSelectionMode) {
-                                       this.moveToSelect(data.position);
+                                       if (data.altKey) {
+                                               this.columnSelect(data.position, data.mouseColumn);
+                                       } else {
+                                               this.moveToSelect(data.position);
+                                       }
                                } else {
                                        this.moveTo(data.position);
                                }

With these changes:

To use multiCursors, use cmd+click, which is more consistent with other editors anyway, instead of alt+click. This is possible due to editor.multiCursorModifier being set to ctrlCmd.
To use columnSelect, use alt+click to start making a column selection at that position, which fixes the confusing vscode default column selection behavior of extending a column selection from the active cursor position prior to clicking with your mouse. This is possible due to the patch above.
To use columnSelect, you can also use the default behavior of just making a column selection by using the middle mouse button when doing the selection. This is the default behavior.

The net effect is an intuitive key mapping that matches other common editors such as Sublime.

Closed via #66445


MenuButton doesn't apply custom styles

Godot version
3.3.2 and 3.4.beta
System information
Windows 10, GTS 450
Issue description
Tried to edit custom style to MenuButton, but node doesn't apply style on both edtior and play. I though it must be PopupMenu style wich appears when you click MenuButton, but even if overrided style PopupMenu appears with defaults.

Steps to reproduce

open attachment
try to change custom style

Minimal reproduction project
New Game Project.zip

Duplicate of https://github.com/godotengine/godot/issues/30768


All tplink entities fail to load if one is offline

I just recently updated to 0.89 and found my tplink HS110 switches weren't loading.  I had configured them manually with the new method of specifying them under tplink:.  I also disabled the discovery.  I have two of the HS110's currently.  One is not in use at the moment.  The other is plugged in and online.  In the logs, it indicated there was a timeout error when connecting (didn't have details in logs as to which one, but I could guess).  I changed my config to only have the one that was plugged in configured.  It loaded up and all was back to normal.  However, if I specified the other one in the config and restarted, neither would load.
I think it would be much better to still continue to load those that do not fail/timeout on connect.  The current effect is that if one gets unplugged, they will all stop loading.  I haven't tested what happens if both are plugged in and then one is unplugged (when not restarting HA).  I'm not currently at home to test that scenario, but worth doing for sure.
This was not an issue prior to this update (0.88 worked), when the tplink hosts were referenced under the switch: section with platform: tplink .  I can quite reliably create the issue.
Details:
Fails on: HA 0.89.0
Last worked on: HA 0.88.0
Hassio, running on Raspberry Pi
Component: https://www.home-assistant.io/components/tplink/
Config that fails with this issue:
tplink:
  discovery: false
  switch:
    - host: tplink01.example.com
    - host: tplink02.example.com

Config that works:
tplink:
  discovery: false
  switch:
#    - host: tplink01.example.com
    - host: tplink02.example.com

Logs...
Thu Mar 07 2019 17:13:13 GMT-0800 (Pacific Standard Time)
Error setting up entry TP-Link Smart Home for tplink
Traceback (most recent call last):
  File "/usr/local/lib/python3.7/site-packages/pyHS100/smartdevice.py", line 116, in _query_helper
    request=request,
  File "/usr/local/lib/python3.7/site-packages/pyHS100/protocol.py", line 47, in query
    sock = socket.create_connection((host, port), timeout)
  File "/usr/local/lib/python3.7/socket.py", line 727, in create_connection
    raise err
  File "/usr/local/lib/python3.7/socket.py", line 716, in create_connection
    sock.connect(sa)
OSError: [Errno 113] Host is unreachable

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/usr/local/lib/python3.7/site-packages/homeassistant/config_entries.py", line 283, in async_setup
    result = await component.async_setup_entry(hass, self)
  File "/usr/local/lib/python3.7/site-packages/homeassistant/components/tplink/__init__.py", line 120, in async_setup_entry
    await hass.async_add_executor_job(_fill_device_lists)
  File "/usr/local/lib/python3.7/concurrent/futures/thread.py", line 57, in run
    result = self.fn(*self.args, **self.kwargs)
  File "/usr/local/lib/python3.7/site-packages/homeassistant/components/tplink/__init__.py", line 110, in _fill_device_lists
    if dev.is_dimmable:  # Dimmers act as lights
  File "/usr/local/lib/python3.7/site-packages/pyHS100/smartplug.py", line 131, in is_dimmable
    return "brightness" in self.sys_info
  File "/usr/local/lib/python3.7/site-packages/pyHS100/smartdevice.py", line 185, in sys_info
    return defaultdict(lambda: None, self.get_sysinfo())
  File "/usr/local/lib/python3.7/site-packages/pyHS100/smartdevice.py", line 195, in get_sysinfo
    return self._query_helper("system", "get_sysinfo")
  File "/usr/local/lib/python3.7/site-packages/pyHS100/smartdevice.py", line 119, in _query_helper
    raise SmartDeviceException('Communication error') from ex
pyHS100.smartdevice.SmartDeviceException: Communication error

Duplicate of #21725
The fix is already merged (#21726)

Thanks! I did a couple quick searches and hadn’t found it. Sorry for the duplicate.


Flyweights should be able to set sensible default values, rather than require explicit settings

Sensible defaults should be set on FW, rather than requiring explicit setting in all cases.  For example, representation has only one option right now and set throughout the code base
representation((byte) 0)

Fixed in reaktivity/nukleus-maven-plugin#30


different number of loaded photos

Lab/Demo: 05-exercise-explore-image-analysis.md
Task: Image retrieval
Step: 4
Description of issue
The number of photos loaded in the photo-album differs from the number of photos from the screenshot in the exercise. This led me to re-check the steps to make sure that I did everything right and there was no mistake.,
Exercise image with numbers (90 photos) :

Image displayed in the exercise with different number of photos (109 photos)

Suggestion :  inform the student that the number of images might vary

The screenshot has been updated to reflect the expected number of images, and language was added to the upload process to provide the number of images.


Adding intellicard functionality.

About the PR

Intellicards can now be used to remove/insert the current AI from the core.
Why / Balance

Killing AI makes sense... I think
Technical details

The brain slot on intellicards and ai core is unlocked but cannot eject, making intellicards swap the brain between itself and the core.
Media

https://github.com/user-attachments/assets/b26716e3-1b4d-42b5-a62d-f225da3e43fc
Requirements

[X] I have read and am following the Pull Request and Changelog Guidelines.
[X] I have added media to this PR or it does not require an ingame showcase.

TODO

[ ] Make swapping the brain a doAfter.
[X] Fix a bug where actions duplicate on removing and inserting the brain.
[ ] Update laws on inserting into the core, in case the upload console is used while AI brain is taken out. (Maybe)

Breaking changes

Changelog

:cl:

add: Using an intellicard on the AI core now swaps the AI between the core and intellicard. (You can evac with the AI!)

If someone knows how to do anything in the TODO please tell me because I don't 😭

Make swapping the brain a doAfter.

i think ItemSlot has a field for this that rmc added

i think ItemSlot has a field for this that rmc added

From what ive seen ItemSlotsComponent doesnt have a field for doAfter, unless im missing something

God bless fildrance

Just need to make ai cores buildable and anchorable and then AI’s can successfully be revived after being disconnected from the station.

Can we perhaps remove mapped intellicards and instead put one in RD's locker?

Make the intelicard require a lenghty do-after like disarming the nuke does. That way someone can't just suicide rush into AI chamber and intelicard them away, and also provides a MALF AI or just regular AI in general the chance to warn crew or fight back before they are completely neutralized. Perhaps even make it tell the AI "hey someone is uploading you into an Intelicard!"

Thats the plan!
I will be trying to do something similiar when I continue working on intellicards after this, for now this serves as a basic way to be able to kill the AI

Can we perhaps remove mapped intellicards and instead put one in RD's locker?

There is one in RD's locker but I know nothing of mapping to remove the mapped ones


Disputing/Reporting Form

This epic contains small components and integration components to create the style and behavior of both disputing and reporting:

initial reporting (min bond amount and/or pre-emptive stake)
adding pre-emptive stake on current tentative winning outcome
disputing tentative winning outcome

closing this epic, it's done


Localized overview component

Localized the overview page component and its info popover

Codecov Report

Merging #2096 (15cbfa1) into react-intl_changeover (ad80fec) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head 15cbfa1 differs from pull request most recent head 21d7b6b. Consider uploading reports for the commit 21d7b6b to get more accurate results

@@                  Coverage Diff                   @@
##           react-intl_changeover    #2096   +/-   ##
======================================================
  Coverage                  70.40%   70.40%
======================================================
  Files                        410      410
  Lines                       7490     7490
  Branches                    1451     1451
======================================================
  Hits                        5273     5273
  Misses                      2191     2191
  Partials                      26       26

Impacted Files
Coverage Δ

src/locales/messages.ts
100.00% <ø> (ø)

Continue to review full report at Codecov.

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


Kama does not shear animals

Minecraft Version
1.18.2
Fabric/Quilt Version
Fabric 0.14.10
Fabric/Quilt API Version
0.59.0
Tinkers' Construct Version
<IP_ADDRESS>
Describe your issue
The Kama does not allow shearing of sheep.
The description of the tool mentions shearing animals.
right click on sheep shows success, but no shearing happens.
forge tinkers does allow shearing of animals, and the clientside success of right click appears that it should function, so this appears to be a bug.
Crash Report
No response
Other mods
None. Tested issue with Hephaestus and FApi alone
Tried reproducing with just Tinkers?
Yes
Performance Enchancers
None of the above
Searched for known issues?
Checked pinned issues, Searched open issues, Searched closed issues

Thanks for reporting this I'll have a new release out soon!


Remove catching NamingException.

Motivation
In #12590, we wrap NamingException into CompletableFuture in AbstractReplicator#validatePartitionedTopicAsync.
So in BrokerService#createNonPersistentTopic, it will never throw NamingException.
Modifications

Remove catching NamingException.

Documentation

[ x ] no-need-doc

I think we can remove the NamingException thrown in the constructor of the SystemTopic and PersistentTopic.

Yes, right. thanks.


Different track bitrates between browsers

NOTE: For bugs, if you delete this template, we will send it again and ask you to fill it out.
Have you read the FAQ and checked for duplicate open issues?:
Yes
What version of Shaka Player are you using?:
v2.3.2
Can you reproduce the issue with our latest release version?:
Yes
Can you reproduce the issue with the latest code from master?:
Are you using the demo app or your own custom app?: Both
If custom app, can you reproduce the issue using our demo app?: Yes
What browser and OS are you using?:
Mac High Sierra 10.13.3
Chrome 64
Safari 11
Firefox 58
What are the manifest and license server URIs?:
(NOTE: you can send the URIs to<EMAIL_ADDRESS>instead, but please use GitHub and the template for the rest)
(NOTE: a copy of the manifest text or an attached manifest will not be enough to reproduce your issue, and we will ask you to send a URI instead)
Angel One: https://shaka-player-demo.appspot.com/demo/#asset=//storage.googleapis.com/shaka-demo-assets/angel-one/dash.mpd;lang=en-US;build=uncompiled
Azure Trailer: https://shaka-player-demo.appspot.com/demo/#asset=//amssamples.streaming.mediaservices.windows.net/91492735-c523-432b-ba01-faba6c2206a2/AzureMediaServicesPromo.ism/manifest(format=mpd-time-csf);lang=en-US;build=uncompiled
What did you do?
Play the stream
What did you expect to happen?
For the Angel One stream, when selecting the video+audio track combo in the demo page, I would expect the bitrates of each track to be the same or at least similar to each other in Safari, Firefox, Chrome
What actually happened?
The bitrates between Safari and Firefox are similar, but Chrome is pretty different. See attached screenshots
Chrome

Safari

On Chrome, the VP9 streams are selected over the H.264 streams, because the VP9 streams have a lower bitrate.  This is a feature of Shaka Player's multicodec content support.
On Safari, VP9 is not available, so we fall back to H.264.  Compared to VP9, H.264 requires a higher bitrate to achieve the same visual quality.
If you try any non-multicodec content, you will see consistent bitrates across browsers.
Does this help?

@joeyparrish Yes, thanks for the clarification!


How to set  src if the source is a broken link?

How to set  src if the source is a broken link?

Hey @skylee91
Could you please elaborate? What do you want to achieve? Do you want to set an alternative src when the actual src is broken?

Yes, let say the actual src is broken, can we display a Image Not Found picture to user?

You can already do it today in the error callback. Something like this:
var bLazy = new Blazy({
        error: function(ele, msg){
            ele.src = "not-found-image.jpg"
        }
    });

I might add it as an option to define in next release

Thanks. It helps a lot.