CUDA not detected

Hi
I am trying to compile warpctc on Ubuntu 16.04 with CUDA 8.0 and cudnn 5.1
The installer is unable to detect CUDA even though cuda has worked fine on my system with everything from OpenCV, Caffe, TensorFlow etc.
Can you please help me solve this problem. I tried passing the include and library paths as CMAKE commands.
Below is the output from cmake
`
~/Workspace/warp/build/warp-ctc/build$ cmake ..
-- The C compiler identification is GNU 5.4.0
-- The CXX compiler identification is GNU 5.4.0
-- Check for working C compiler: /usr/bin/cc
-- Check for working C compiler: /usr/bin/cc -- works
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Detecting C compile features
-- Detecting C compile features - done
-- Check for working CXX compiler: /usr/bin/c++
-- Check for working CXX compiler: /usr/bin/c++ -- works
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Detecting CXX compile features
-- Detecting CXX compile features - done
-- Could NOT find CUDA (missing:  CUDA_INCLUDE_DIRS CUDA_CUDART_LIBRARY) (found suitable version "8.0", minimum required is "6.5")
-- cuda found FALSE
CMake Warning at CMakeLists.txt:48 (FIND_PACKAGE):
By not providing "FindTorch.cmake" in CMAKE_MODULE_PATH this project has
asked CMake to find a package configuration file provided by "Torch", but
CMake did not find one.
Could not find a package configuration file provided by "Torch" with any of
the following names:
TorchConfig.cmake
torch-config.cmake

Add the installation prefix of "Torch" to CMAKE_PREFIX_PATH or set
"Torch_DIR" to a directory containing one of the above files.  If "Torch"
provides a separate development package or SDK, be sure it has been
installed.
-- Torch found Torch_DIR-NOTFOUND
-- Building shared library with no GPU support
-- Configuring done
-- Generating done
-- Build files have been written to: /home/ferjad/Workspace/warp/build/warp-ctc/build
`
Below is the output from nvcc
~/Workspace/warp/build/warp-ctc/build$ nvcc --version nvcc: NVIDIA (R) Cuda compiler driver Copyright (c) 2005-2016 NVIDIA Corporation Built on Tue_Jan_10_13:22:03_CST_2017 Cuda compilation tools, release 8.0, V8.0.61
`
~/Workspace/warp/build/warp-ctc/build$ cat /usr/local/cuda/include/cudnn.h | grep CUDNN_MAJOR -A 2
#define CUDNN_MAJOR      5
#define CUDNN_MINOR      1
#define CUDNN_PATCHLEVEL 10
#define CUDNN_VERSION    (CUDNN_MAJOR * 1000 + CUDNN_MINOR * 100 + CUDNN_PATCHLEVEL)
#include "driver_types.h"
`

@Govan111 CUDA8 doesnt symlinks the files to /usr/local/cuda/include and /usr/local/cuda/include/include64 from /usr/local/cuda/targets/x86_64-linux
Make these symlinks from terminal and you ll be good to go with warp-ctc and a lot of other open source tools that required manual setup of cuda since the version 8


buffer: release empty slices after commit (#116)

Description: Remove empty slices off the end of buffers after calls to OwnedImpl::commit. The slices reserved when OwnedImpl::reserve is called will sit unused in cases where the 0 bytes are commited, for example, when socket read returns 0 bytes EAGAIN. Trapped slices act like a memory leak until there is a successful read or the socket is closed.
Risk Level: low
Testing: unit
Docs Changes: n/a
Release Notes: n/a
Signed-off-by: Asra Ali<EMAIL_ADDRESS>

/retest


sync: tests fail with mutex profiling

As reported by @hyangah,
$ go test -v  -mutexprofile=prof.mutex -test.mutexprofilefraction=1 -test.bench=Misuse2

... sometimes fails. At least on Linux and with
Can we detect (from the sync tests) that mutex profiling is enabled and skip some tests?

This report is not particularly actionable. Fails how? On Linux and with what?

Please reopen if you have more details. I'm assuming those are lost at this point.


Adding Percentile to the end of STAR Assessments for clarity

Uri had requested this quick UI change to make it clear that STAR assessments are using percentile.
@kevinrobinson @alexsoble Uri is demoing this to assistant principals tomorrow, would be great to ship tonight. ;-)

OK. I'm restarting the test build since the last build appears to have failed non-deterministically.

Merci

@jhilde coming atcha!

@alexsoble Maybe we should take a look at the build, seems like it has been flaky this week.


[SQL Lab] Fix bug when filtering on results that include nulls

CATEGORY
Choose one

[x] Bug Fix
[ ] Enhancement (new features, refinement)
[ ] Refactor
[ ] Add tests
[ ] Build / Development Environment
[ ] Documentation

SUMMARY

Fixes a TypeError: Cannot read property 'toString' of null when you filter results in SQL Lab and the results set includes null values. Also adds a unit test to confirm the fix.
TEST PLAN

Test filtering when the results set includes null values. Also run unit tests.
ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Changes UI
[ ] Requires DB Migration.
[ ] Confirm DB Migration upgrade and downgrade tested.
[ ] Introduces new feature or API
[ ] Removes existing feature or API

REVIEWERS
@graceguo-supercat @betodealmeida @michellethomas @xtinec

Codecov Report

Merging #8231 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #8231      +/-   ##
==========================================
+ Coverage   65.97%   65.98%   +<.01%
==========================================
  Files         479      479
  Lines       23038    23038
  Branches     2552     2552
==========================================
+ Hits        15200    15202       +2
+ Misses       7702     7700       -2
  Partials      136      136

Impacted Files
Coverage Δ

...src/components/FilterableTable/FilterableTable.jsx
79.88% <100%> (+1.14%)
:arrow_up:

Continue to review full report at Codecov.

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


Create a validation tool for threat yaml files

Would check

filename format
filename length
structure of the yaml
OCST id uniqueness (in a global context)

done via bdd


[CLOSED] 8.8?

 Issue by bcpierce00
Friday Aug 17, 2018 at 17:13 GMT
Originally opened as https://github.com/antalsz/hs-to-coq/issues/95

The README file says Coq 8.7, but it looks like other things might be ready for 8.8.  If that's true, please update the README.  (If not, maybe add a note about when it will be?)

 Comment by nomeata
Friday Aug 17, 2018 at 17:25 GMT

We are using 8.8 for a while, the README was not up-to-date – sorry for that.

 Comment by bcpierce00
Friday Aug 17, 2018 at 17:26 GMT

As I thought -- thanks!


Add extra commands

Add commands:

/j alias for /join
/list command https://github.com/erming/shout/issues/635

I kinda disagree with adding one char aliases. Tab completion also works for commands.

would it be possible to add a config option to add aliases to commands?

I agree with @xPaw regarding /j. Typing /j + Tab is not much harder (tab completion is so common we are not being esoteric).
With the package system in the works (or about to be in the works), I don't think adding this option is something we will or should implement. However, we should definitely make sure the API for the package system lets developers make packages to do this sort of things.
For /list it's a different issue and I believe it would be much appreciated. Would like opinion from others.

Closing this as wontfix for triaging purposes. We can re-open if needed.

@astorije did you add a story for /list? I can't see one.

I admit I overlooked the second part of the issue :-)
I just copied https://github.com/erming/shout/issues/635 over to #130 using https://github-issue-mover.appspot.com/. Conclusion: I'm not using this tool again :-)


update: enforce bumping engine version whenever appropriate

What type of PR is this?
/kind cleanup
/kind documentation
Any specific area of the project related to this PR?
/area CI
/area build
What this PR does / why we need it:
This follows-up https://github.com/falcosecurity/libs/pull/1273 and follows the same logic, applying it to the Falco engine version.
Accordingly, the formula for updating the engine checksum has been changed for catching more information. Moreover, two CI checks have been added:

A strict one (that causes a failure) checking both the checksum and the engine version for consistency
A weak one, that attempts catching changes in the Falco engine implementation and sends a warning message

Which issue(s) this PR fixes:
Special notes for your reviewer:
Does this PR introduce a user-facing change?:
NONE

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: jasondellaluce
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [jasondellaluce]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/milestone 0.36.0

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: jasondellaluce, leogr
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [jasondellaluce,leogr]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

LGTM label has been added.  Git tree hash: 5fe1cfee2982adddb7870a0627718f60f6d74a85

CI Build / check-engine-version (pull_request) Failing after 6s

This is of course a false positive. The checksum is changed just because I changed how it's computed.


"Izvoarele" is listed in the drop-down menu

Description:
"Izvoarele" is listed in the drop-down menu alongside its corresponding county and commune, according to the law
Precondition:
The website is up an running.

Step 1
Type "Izvoarele" in the search bar.

Expected results
The location is listed in the drop-down menu as "Izvoarele, Alba (Gârda de Sus)".


Sparse Feature in c++, different shape cause different result

HI :
I was using c++ api of tensorflow-1.14.0 and found a problem.
I had a model in pb format and loaded it use c++. The model had a sparse node, so I construct indices, values shape for it.
Three tensor name for a sparse node:
DeserializeSparse:0  for indices,
DeserializeSparse:1   for values,
DeserializeSparse:2  for shape,
case 1:
shape = [1, 1];  indices = [0, 0]; values = [""]  c++ got a predict result was same as python's result .
case 2:
shape = [1, 1];  indices = [0, 0]; values = ["abc"] c++ got a predict result  was different from python's.
case 3:
shape = [1, 100] indices = [0, 0]; values = ["abc"] c++ got a predict result was different from case 2.

I found python always got the same answer with different shape but with the same indices and values.
I thought, there may be some difference between c++ and python, and may be c++ is wrong.

After many tests, I found that:I must append an empty string into values(the tail of vector), then c++ predict result was same as python's.  Why?

commit:  tensorflow 1.15 also  has this problem.

Can you provide some code that reproduces the buggy behavior? I can't really follow the textual description in the bug.

It has been 14 days with no activity and the awaiting response label was assigned. Is this still an issue?

We are closing this issue for now due to lack of activity. Please comment if this is still an issue for you. Thanks!


name list is returning as "None"

Hello,
I have successfully trained and have a .gzbin file and speakers.txt. When I try to return the name list from recog.identify_speaker() like so:
name = []
vad.record()
name = recog.identify_speaker()
print(name)

name does not contain anything. Is there something I am missing? Thanks

Put recog.debug = True after vad.record()  it will print error log.

JUST FOR INFORMATION

if not self.debug:
               try:
                    names = []
                    names.append(str((name[2].split()[2]).decode('utf-8')))
                    names.append(str((name[3].split()[3]).decode('utf-8')))
                    return names
                except IndexError:
                    logging.error('MARF execution failed.'
                                  ' Please set debug=True to print error'
                                  ' info.', exc_info=True)

            else:
                # sanitize string
                data = (''.join(str(x) for x in name))
                fi = data.encode('ascii', 'ignore').decode('unicode_escape')
                print(re.sub("[b']", '', fi))
In condition you have used 'if not self.debug'. In case you turn on the debug flag you will never get names


"make format" and credits.md - changes affect mkdocs

Great repo, much less overhead than Hypermodem and cookiecutter.
Getting started with the repo and noticed that if a make format is done, it changes the docs/credits.md file contents which mkdocs then fails. In credits.md there is this (python exec="yes" on the open fence):
--8<-- "scripts/gen_credits.py"

which when make format is run with blacken-docs changes the format to:
--8 < --"scripts/gen_credits.py"

with spaces changes. make check-docs will then error with:
 WARNING -  markdown_exec: Execution of python code block exited with errors

  Code block is:

    --8 < --"scripts/gen_credits.py"

  Output is:

    Traceback (most recent call last):
...

I looked into duty and the lazy callable for blacken_docs has an exclude parameter, but it looks like it's never used. I'll continue to pursue  there, but has anyone see this error and was able to correct?

Great repo, much less overhead than Hypermodem and cookiecutter.

Thanks, goes straight to the heart <3
Yeah I'm suffering from the same thing since a few weeks. There's an issue open on blacken-docs to add a way to ignore parts of the markup when formatting, but the maintainer said it's a tough task and they don't have the time to fix it yet.
A quick workaround is to use markdown-exec to defer the snippet inclusion:
print('--8<-- "whatever"')

Not sure that's actually the worakaround, I'm on mobile, let me find the issue later :)

Thanks for the workaround! That should work for me! I was able to correct by adding the following to the duty package:
    for path in paths:
        path = Path(path)  # noqa: PLW2901
        if path.is_file():
            filepaths.add(path.as_posix())
        else:
            for ext in exts:
                filepaths |= {filepath.as_posix() for filepath in path.rglob(f"*.{ext}")}
    # Apply exclude
    excl_filepaths = filepaths.copy()
    for excl in exclude:
        reg_obj = excl
        for file in excl_filepaths:
            if reg_obj.search(file):
                filepaths.remove(file)

With that [unoptimized] approach to remove the matched entries from the filepaths, this change to the project duties.py then works:
    ctx.run(
        blacken_docs.run(
            *PY_SRC_LIST,
            "docs",
            exts=["py", "md"],
            exclude=["credits.md"],
            line_length=120,
        ),
        title="Formatting docs",
        nofail=True,
    )

Reckon something like that might work?

Thanks, we can indeed consider adding an exclusion parameter to the duty callable :slightly_smiling_face:
So here the issue I was speaking of: https://github.com/adamchainz/blacken-docs/issues/193, and the workaround is:
```python exec="1" result="python"
print('--8<-- "docs/examples/model_ext.py"')
```

Implemented and works, thanks!

Spoke too soon. make format succeeds, but make docs host=localhost port=5000 generates the credits with the actual Python script itself and not the rendered credits tables.

Thank you for reporting back, I'll investigate :)

OK, you're right, it didn't work. Here's one that works (even uglier):
```python exec="yes"
print("""```python exec="1"\n--8<-- "scripts/gen_credits.py"\n```""")
```

I opened a PR on blacken-docs and prepared copier-pdm for it, we just have to wait now, closing :slightly_smiling_face:

Thanks for ugly hack, that's working. Will monitor the upstream repo out of interest.


Describe histogram aggregates and percentile configs as comma-separated

Note: Please remember to review the Datadog Contribution Guidelines
if you have not yet done so.
See #364
What does this PR do?
Updates the descriptions of the DD_HISTOGRAM_AGGREGATES and DD_HISTOGRAM_PERCENTILES environment variables to state they are comma-separated.
Motivation
What inspired you to submit this pull request?
I got bitten by this bug when adding the sum aggregate.

thanks @chrisjseaton :)

FYI This is similarly incorrectly described over here:
https://docs.datadoghq.com/agent/docker/?tab=standard#dogstatsd-custom-metrics


Please suppress CC0057 in template method pattern

In the following code, the Method CheckConfiguration serves as an empty template method.
interface IConfigurableJobRunner {
bool CheckConfiguration(Configuration config);
void Run ();
}
public abstract RunnerBase : IConfigurableJobRunner {
public virtual bool CheckConfiguration (Configuration config) { return false; }
public abstract Run ();
}
The RunnerBase class makes the CheckConfiguration an optional method to its sub-classes.
In this case, I think CC0057 should not be triggered, if the following two conditions are met:

The method is virtual;
The method implements a method defined in an interface.

It's make sense. What do you think @code-cracker/owners  ?

I was not able to repro. I do not get CC0057 with this code:
    public class Configuration { }
    interface IConfigurableJobRunner
    {
        bool CheckConfiguration(Configuration config);
        void Run();
    }

    public abstract class RunnerBase : IConfigurableJobRunner
    {
        public virtual bool CheckConfiguration(Configuration config) { return false; }
        public abstract void Run();
    }

I am closing the issue. If you can find a repro we will happily reopen.

The above code triggered CC0057 improperly on the virtual Get method of class A.
abstract class A
{
	internal virtual string Get(string index) { return null; }
}
class C : A
{
	Dictionary<string, string> _values = new Dictionary<string, string> ();
	internal override string Get (string index) {
		return _values[index];
	}
}

Right now only partial, override,  abstract and extern modifiers make the exclusion list. Should we do the same for virtual? I am not really sure, template method is an exception to the more common case.
And we cannot simply search the whole project for other classes overriding the method, that would make the analyzer unusable.
I think that, for this scenarios, the user should suppress the analyzer. What do you think, @code-cracker/collaborators and @code-cracker/core?

Should we do the same for virtual?

I think so. If a method is virtual, the parameter might be used in overriding methods.
FWIW, ReSharper never emits a warning for unused parameters unless the method is private.


java.nio.BufferUnderflowException

When I try to connect to a 0.10.4 multiplayer game with MCPE, flat world, I get an exception:
at java.nio.HeapByteBuffer.get(HeapByteBuffer.java:151)
at java.nio.ByteBuffer.get(ByteBuffer.java:715)
at net.beaconpe.magicclient.network.encap.RakNetInternalPacket.fromCustomPacket(RakNetInternalPacket.java:88)
at net.beaconpe.magicclient.network.encap.RakNetCustomPacket.decode(RakNetCustomPacket.java:43)
at net.beaconpe.magicclient.network.PacketHandler.handleCustomPacket(PacketHandler.java:240)
at net.beaconpe.magicclient.network.PacketHandler.handlePacket(PacketHandler.java:227)
at net.beaconpe.magicclient.network.PacketHandler.run(PacketHandler.java:192)

This is while decoding a packet of type 0x84, length = 317.

This is PocketMine or vanilla?
On Sunday, February 15, 2015, pcbeard<EMAIL_ADDRESS>wrote:

When I try to connect to a 0.10.4 multiplayer game with MCPE, flat world,
I get an exception:
at java.nio.HeapByteBuffer.get(HeapByteBuffer.java:151)
at java.nio.ByteBuffer.get(ByteBuffer.java:715)
at net.beaconpe.magicclient.network.encap.RakNetInternalPacket.fromCustomPacket(RakNetInternalPacket.java:88)
at net.beaconpe.magicclient.network.encap.RakNetCustomPacket.decode(RakNetCustomPacket.java:43)
at net.beaconpe.magicclient.network.PacketHandler.handleCustomPacket(PacketHandler.java:240)
at net.beaconpe.magicclient.network.PacketHandler.handlePacket(PacketHandler.java:227)
at net.beaconpe.magicclient.network.PacketHandler.run(PacketHandler.java:192)
This is while decoding a packet of type 0x84, length = 317.
—
Reply to this email directly or view it on GitHub
https://github.com/jython234/MagicClient/issues/1.

--

Connecting to vanilla MCPE. I think part of the problem is this code:
public static RakNetInternalPacket fromCustomPacket(ByteBuffer bb){
    byte flag = bb.get();
    p.reliability = (byte) (flag >> 5);

If the value of flag is 0x80, this shift results in a negative value for reliability because Java's byte type is signed (ick). If I change it to:
    p.reliability = (byte) ((flag >> 5) & 0x7);

then the values are all correct. However, I still see some problems decoding some encapsulated packets. I've added more logging, and this is what I'm seeing:
public static RakNetInternalPacket fromCustomPacket(ByteBuffer bb){
    RakNetInternalPacket p = new RakNetInternalPacket();
    byte flag = bb.get();
    p.reliability = (byte) ((flag >> 5) & 0x7);
    p.hasSplit = (flag & 0x10) == 0x10;
    System.out.println("fromCustomPacket: flag = " + String.format("0x%02x ", flag) + ", reliability = " + p.reliability + ", hasSplit = " + p.hasSplit);
    int bits = bb.getShort() & 0x0000FFFF;
    if ((bits & 0x7) != 0) System.out.println("non byte-aligned value of bits: " + bits);

RakNetCustomPacket.decode(): seqNum=4, Length is: 302
fromCustomPacket: flag = 0x60 , reliability = 3, hasSplit = false
fromCustomPacket: length = 41, remaining = 288
fromCustomPacket: flag = 0x40 , reliability = 2, hasSplit = false
fromCustomPacket: length = 10, remaining = 241
fromCustomPacket: flag = 0x60 , reliability = 3, hasSplit = false
non byte-aligned value of bits: 41
fromCustomPacket: length = 5, remaining = 221
fromCustomPacket: flag = 0x80 , reliability = 4, hasSplit = false
fromCustomPacket: length = 2048, remaining = 206
Exception in thread "PacketHandler" java.nio.BufferUnderflowException
at java.nio.HeapByteBuffer.get(HeapByteBuffer.java:151)
at java.nio.ByteBuffer.get(ByteBuffer.java:715)
at net.beaconpe.magicclient.network.encap.RakNetInternalPacket.fromCustomPacket(RakNetInternalPacket.java:91)
at net.beaconpe.magicclient.network.encap.RakNetCustomPacket.decode(RakNetCustomPacket.java:43)
at net.beaconpe.magicclient.network.PacketHandler.handleCustomPacket(PacketHandler.java:257)
at net.beaconpe.magicclient.network.PacketHandler.handlePacket(PacketHandler.java:230)
at net.beaconpe.magicclient.network.PacketHandler.run(PacketHandler.java:195)

Moreover, looking at PocketMine's implementation we see:
        $length = (int) ceil(Binary::readShort(substr($binary, 1, 2)) / 8);

So, if the value of bits is not an exact multiple of 8, length will be rounded up to the next byte value.

Ahh I see.
On Sunday, February 15, 2015, pcbeard<EMAIL_ADDRESS>wrote:

Moreover, looking at PocketMine's implementation we see:
    $length = (int) ceil(Binary::readShort(substr($binary, 1, 2)) / 8);

So, if the value of bits is not an exact multiple of 8, length will be
rounded up to the next byte value.
—
Reply to this email directly or view it on GitHub
https://github.com/jython234/MagicClient/issues/1#issuecomment-74443301.

--

Strangely I never had this problem. I guess it's because I always tested
against pocketmine.
On Sunday, February 15, 2015, Alejandro Zeise<EMAIL_ADDRESS>wrote:

Ahh I see.
On Sunday, February 15, 2015, pcbeard<EMAIL_ADDRESS><javascript:_e(%7B%7D,'cvml','notifications@github.com');>> wrote:

Moreover, looking at PocketMine's implementation we see:
    $length = (int) ceil(Binary::readShort(substr($binary, 1, 2)) / 8);

So, if the value of bits is not an exact multiple of 8, length will be
rounded up to the next byte value.
—
Reply to this email directly or view it on GitHub
https://github.com/jython234/MagicClient/issues/1#issuecomment-74443301
.

--

--

I'd really like to get this working with vanilla if possible.

I see, my plans for it included a javascript api with events and all. But I
didn't think many people would use it with vanilla.
On Sunday, February 15, 2015, pcbeard<EMAIL_ADDRESS>wrote:

I'd really like to get this working with vanilla if possible.
—
Reply to this email directly or view it on GitHub
https://github.com/jython234/MagicClient/issues/1#issuecomment-74443676.

--

I'm mainly seeing this underflow when getting chunk data. A RakNetCustomPacket receives 1447 bytes of data to decode. The RakNetInternalPacket reads a short value() (unsigned) of 39656 bits, or 4957 bytes, but the packet only contains 1433 bytes. It's immediately followed by another packet that says it contains 4525 bytes, but only 1433 are present. I'm pretty sure the MTU values start at 1447, so I'm unsure how chunks are sent. I was assuming that's what split packets were for. Confusing!

@pcbeard I think I know what's wrong. Since chunk packets are sent split, my decoding method reads the short value too late or too early and gets the wrong value. Still not sure.

Wait the flag is 0x80? I've never seen that one before...

That may have been caused by mis-reading a split packet as you said.

Strange, even the split packet format has the length come first.
On Monday, February 23, 2015, pcbeard<EMAIL_ADDRESS>wrote:

That may have been caused by mis-reading a split packet as you said.
—
Reply to this email directly or view it on GitHub
https://github.com/jython234/MagicClient/issues/1#issuecomment-75589083.

--


Open source CMS written in Java

I like this project, maybe you could find more inspiration here: https://getmesh.io/
There is also https://dotcms.com/.
Both projects are Java-based and open source, you'll find them on Github.

Thanks for the information @JeroenAdam,
I'm actually doing it for fun and to learn Jhipster (Angular + Spring) while I was doing it. If more people joins me, then we can make a bigger CMS and use it for others to learn how to do it: Spingular Course at Udemy


[5.9] Capture and re-bind route Route:redirect($from, $to) parameters

The current implementation of Route::redirect($from, $to) allows you to specify route parameters for the $from location and redirect to a static $to location.
Route::redirect('/users/{user}/profile', '/members');

$this->get('/users/22/profile')->assertRedirect('/members');

$this->get('/users/99/profile')->assertRedirect('/members');

At times I've personally found it would be handy to reuse the {user} parameter in the redirect $to location. Previously you would have to create a controller, capture the values from the request, and return the redirect. This PR introduces that ability to do it all inline in the routes file:
Route::redirect('/users/{user}/profile', '/members/{user}/profile');

$this->get('/users/22/profile')->assertRedirect('/members/22/profile');

$this->get('/users/99/profile')->assertRedirect('/members/99/profile');

It also allows for optional route parameters...
Route::redirect('/users/{user}/repos/{repo?}', '/members/{user}/projects/{repo?}');

$this->get('/users/22/repos/')->assertRedirect('/members/22/projects');

$this->get('/users/99/repos/laravel')->assertRedirect('/members/99/projects/laravel');

and an example with multiple route parameters...
Route::redirect('/users/{user}/repos/{repo?}/{slug?}', '/members/{user}/projects/{repo?}/{slug?}');

$this->get('/users/22/repos/')->assertRedirect('/members/22/projects');

$this->get('/users/99/repos/123')->assertRedirect('/members/99/projects/123');

$this->get('/users/99/repos/123/laravel')->assertRedirect('/members/99/projects/123/laravel');

Notes
I've given the destination route a name (which you may want to tweak) to make troubleshooting easier as the route name appears in the exception message when you try to use a parameter in $to that you did not specify in $from.
If you are not keen on making the toRoute method public, we can just new up the RouteUrlGenerator in the controller...
- $url = $urlGenerator->toRoute($destination, $parameters, false);
+ $url = (new RouteUrlGenerator($urlGenerator, $request)->to($destination, $parameters);

Travis CI is not happy because it can't resolve the url generator contract from the container 🤔
...am I missing something here? Works on my machine™
Let me know (maybe should just inject the concrete?) and I'll get it sorted.

Naturally I was about to fall asleep and I thought of another, potentially better way, to solve this. We could just register a route on the fly and then we wouldn’t have to expose the toRoute method. Not 100% it would work- but could be a way to go if u don’t dig this approach.

Looks good, thanks!

Awesome. Thanks for reviewing / merging!


Parallel crossvalidation for locked data

Crossvalidation is currently not parallel if the machine is locked

We actually disabled the feature in develop as there are problems with
deadlocks which we need to look into.
You can always activate it back and play with it. Tell us what you do and
if it works and we can put it back in
The locked thing is on my list as well
H
On Wednesday, 26 October 2016, Leon Kuchenbecker<EMAIL_ADDRESS>wrote:

Crossvalidation is currently not parallel if the machine is locked
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/shogun-toolbox/shogun/issues/3479, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAqqv34zcwVpXzRPqBle5QYJA89QUlP7ks5q30xKgaJpZM4KhION
.

--
Sent from my phone

You mean parallel X-val has been disabled alltogether?

For now in develop, yes.
As said you can keep on using it locally if it works
There currently is some problem with memory when parallelising this

#3742 #3732


tweepy.org Nav not linking to newest version

In the navbar of http://www.tweepy.org/ it currently links to http://tweepy.readthedocs.io/en/v3.5.0/ and not http://tweepy.readthedocs.io/en/v3.6.0/.

This belongs as an issue for https://github.com/tweepy/tweepy.github.com and is a duplicate of https://github.com/tweepy/tweepy.github.com/issues/5 and https://github.com/tweepy/tweepy.github.com/pull/7.


Kesser "Glas IR Heater" incorrect min. Temp

Description
The heater is correctly recognized with all entities. However, the recognized value for the minimum temperature of 10°C is incorrect. The heater can be turned down to a temperature of 5°C.
Steps to reproduce
No response
Expected behaviour
No response
Additional context
"pending_state": {},
"connected": true,
"force_dps": [],
"home_assistant": {
"name": "Kesser IR-Heater",
"name_by_user": null,
"disabled": false,
"disabled_by": null,
"entities": [
{
"disabled": false,
"disabled_by": null,
"entity_category": null,
"device_class": null,
"original_device_class": null,
"icon": null,
"unit_of_measurement": null,
"state": {
"entity_id": "climate.kesser_ir_heater",
"state": "heat",
"attributes": {
"hvac_modes": [
"heat",
"off"
],
"min_temp": 10,   should be 5°C
"max_temp": 30,
"target_temp_step": 1.0,
"current_temperature": 16,
"temperature": 23,
"hvac_action": "heating",
"friendly_name": "Kesser IR-Heater",
"supported_features": 385
},
"last_changed": "2024-12-02T23:14:15.459019+00:00",
"last_reported": "2024-12-02T23:14:15.459019+00:00",
"last_updated": "2024-12-02T23:14:15.459019+00:00"
}
}
Confirmation

[X] I am sure this is a bug or improvement, that is well enough described that it can be implemented.

[X] If this is for a specific device, the device config is mentioned above, or diagnostics are included.

[X] Log messages or diagnostics relevant to the issue are included.

[X] This is not requesting addition of a new device.

Hello @make-all,
I just downloaded tuy-local new from Github and reinstalled it. I can't notice any change. "min_temp": 10, is still recognized, but not 5
config_entry-tuya_local-01JE62GRAA8VQEBH8R3QY5JEVY.json
°C.

No change has been made.

Hello @make-all,
i solved the problem for myself. I adjusted the file "kesser_infrared_wall_heater.yaml" in the folder "\config\custom_components\tuya_local\devices".
But i think the solution probably cannot be adopted as the original device is a 400w wall heater and it may not support 5°C?
My heater is a Kesser glass IR heater with 600w.


maintenance: merging latest v1 changes with main

Issue number:
Summary
Merging latest develop v1 changes with main.
Changes

Please provide a summary of what's being changed

User experience

Please share what the user experience looks like before and after this change

Checklist
Please leave checklist items unchecked if they do not apply to your change.

[x] I have performed a self-review of this change
[x] Changes have been tested
[x] Changes are documented
[x] PR title follows conventional commit semantics

Acknowledgment
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.
Disclaimer: We value your time and bandwidth. As such, any pull requests created on non-triaged issues might not be successful.

Build failure due to test failing is a known issue, but not severe enough to block merge


Adding history for builds into json

This should resolve: #6
It's the first pass where i choose json, let me know any comments or adjustments you like to make.

@developit I opted for JSON here initially, we can extend with dotfiles if desired if you can give me some pointers on how you want to have the files formatted.

@developit Odds of getting this merged?  Was looking into doing this exact thing.

Just took a look at this 👍
One issue:
If the dir your storing the file in doesn't exist it will throw
Couldn't store history: Error: ENOENT: no such file or directory, open '/repo-name/reports/build-sizes.json'

Also IMO would be nice to save the the diff info for the file size like the console output shows against the previous entry.  Picturing being able to use this to chart file size changes overtime.
{
filename: x
filesize: x
diff: (+4.26 kB)
}

Current console output
static/nnw6w_3RdMEf0sZwVrMJh/pages/index.js ⏤  1.23 kB (+1.23 kB)
static/nnw6w_3RdMEf0sZwVrMJh/pages/pdf.js ⏤  4.26 kB (+4.26 kB)
static/nnw6w_3RdMEf0sZwVrMJh/pages/_error.js ⏤  2.43 kB (+2.43 kB)

Current JSON
 "1544468967650": {
    "static/AmCaWNJU20AT31h6SVe31/pages/index.js": 1233,
    "static/AmCaWNJU20AT31h6SVe31/pages/pdf.js": 4263,
    "static/AmCaWNJU20AT31h6SVe31/pages/_error.js": 2428
  }

I can make those changes... you need to give me a bit of time though.
Question do you still want to keep the timestamp in the json? or do you just want to do an array of items and add the timestamp as a key in the object something like this:
Adding the diffInBytes and Pretty both to the object.
The missing dir i will fix. Let me know what you think
{    filename: x     filesize: x    diff: (+4.26 kB)     diffBytes: 1233    timestamp: 123456789 }

First - thanks for your work on this :)
Was trying it out and was happy to see there was already a PR to add writing to a file support.
I vote to keep it.
"1544468967650": {
   filename: x
   filesize: x
   diff: +4.26 kB
   diffBytes: 1233
}

Had another thought on this ...
What if this built up a array and passed the results to a callback
Allowing the user to save to a file or db or whatever they want
new SizePlugin({ cb: (jsonObj) =>{
//save to json or whatever you like
}
})
...
let obj = {
              filename: name,
              filesize: size,
              diff: deltaText,
              diffBytes: delta,
              color,
              timestamp: this.buildTimestamp
               };

              jsonOutput.push(obj);
...

this.cb(jsonOutput);

You could create an addon for save to json with the readFile etc...  keeping this repo all that code out of the main repo.

I like the updated format suggestion @timarney. One nit though:  it's super easy to format the output and I'd rather keep that separate from the JSON.  Instead, we should have the JSON include the before & after sizes, both compressed and uncompressed:
{
  // we can hoist the timestamp, it's constant for all files
  timestamp: 123456789,
  files: [
    {
     filename: 'foo/foo.a1e6f.chunk.js',
     previous: 1000,
     size: 1024,
     diff: 24
    },
    // .. etc ..
  ]
}

Seem reasonable @mick-feller + @timarney?
Oh also I love the idea of allowing devs to hook the output saving.  Easiest way to do that is to allow options to specify load() and save():
new SizePlugin({
  json: true,
  async load() {
    return JSON.parse(await someDb.get('stats'));
  },
  async save(stats) {
    await someDb.set('stats', JSON.stringify(stats));
  }
})

Nice ya agreed on the format as you've outlined.
Before & after sizes makes the most sense.

So i did a first pass on this one, not totally done yet don't have more time tonight.
I remapped the json to the desired format, i've also added that if the previous file size comes back as zero or undefined, i'll try to read the json from the location and pick up the previous file size from the json. If that fails set it to current file size.
I still need to do the load and save outside, i'll work on that tomorrow or later this week, i just wanted to give you guys a peek already to see if we are getting close.

We're doing some work with a forked version of this repo https://github.com/timarney/size-plugin/blob/master/src/index.mjs#L177 here https://github.com/cds-snc/bundle-size-tracker
Roughly we're writing a JSON file and than reading / writing the filesizes to a DB and than charting them + connecting them to Github.
In our case from this PR we need the JSON write to file but not the read.  Which make me think as per https://github.com/GoogleChromeLabs/size-plugin/pull/15#issuecomment-446213435 splitting out the save / read functionality is a good solution.
In our case we don't need much info in the JSON file
fileSizes.push({
			  filename: assetNames[i],
			 filesize: size
			});

Needed to pull the filename form assetNames to get the original name of the file.

Hi @mick-feller, what about using something like conf to handle the (default) storage details/logic?

Can anyone merge this feature please?

@developit @mick-feller anyone working on this? I can pick it up as I need similar feature at work


Modify Storage components and Status to fit Trackermon

Modified AB3 classes by duplicating them and renaming the duplicates to fit Trackermon.

Codecov Report

Merging #64 (7bb2707) into master (1155ebf) will decrease coverage by 2.62%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##             master      #64      +/-   ##
============================================
- Coverage     59.94%   57.31%   -2.63%
  Complexity      399      399
============================================
  Files            89       92       +3
  Lines          1483     1551      +68
  Branches        154      164      +10
============================================
  Hits            889      889
- Misses          562      630      +68
  Partials         32       32

Impacted Files
Coverage Δ

src/main/java/seedu/address/model/show/Status.java
0.00% <0.00%> (ø)

...in/java/seedu/address/storage/JsonAdaptedShow.java
0.00% <0.00%> (ø)

...eedu/address/storage/JsonSerializableShowList.java
0.00% <0.00%> (ø)

...ava/seedu/address/storage/JsonShowListStorage.java
0.00% <0.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 1155ebf...7bb2707. Read the comment docs.


433 MHz testing additions

As promised! Updates for 433MHz operation.
Files and readme changes
Added:-

XML and header files for 433 MHz in fixed and variable length operation
Support for a FREQ_433 define to use 433MHz mode
.gitignore added
README updated to reflect additional 433MHz testing, using 433MHz operation, and the SDI/MISO SDO/MOSI reversal issue on the Arduino Pro Mini

Thanks, just a thing before I merge it, about the MISO/MOSI reversal: are you sure you're using the correct pinout? I checked on the internet, and MISO is pin 12, MOSI is pin 11, as it should be. Which is the opposite of yours, and explains your issue with the SPI.
http://www.dominicdube.com/wp-content/uploads/ProMiniPinout.png

In my circuit Zeta SDO goes to Pro Mini MOSI and Zeta SDI goes to Pro Mini MISO. I used the same diagram you shared, and the Zeta PDF download datasheet.

Something's wrong... Is it a genuine pro mini?
Here is my wiring with a pro mini (not genuine but same pinout):

SDO to MISO (12), SDI to MOSI (11)...

Mine is wired up the same as yours (now!), BUT, the Pro Mini pinout I'm using shows the MISO and MOSI the other way around..... Just googled for other pin outs, and looks like it's a typo on my diagram! Very annoying. I'll throw that one away.
So please remove my remarks about the Pro Mini having different wiring sometimes.

Nice, mystery solved :-)
I just merged your commit, thanks. I changed a bit the name of the define, to be consistent with the lib, it's 'ZETARF_FREQUENCY_433MHZ' now ;-)


[RFC][diem-framework] Implemented an api for fetching diem framework in historic commits

…odes from historic git commits.

Motivation
This PR implements a draft API for fetching the Diem Framework bytecodes in old historical commits by using git archive. We expect this api to be used for:

Future backward compatibility testing for Diem Framework across multiple releases
Generate release artifact for writesets upgrade.

This might also impact @vgao1996 's work on separating move stdlib out from the diem-framework. For better testing purpose we might want to have the future move-stdlib to have the same api. Feedbacks would be really appreciated.
Have you read the Contributing Guidelines on pull requests?
Yes
Test Plan
I picked a recent commit on master to make sure the diem framework over there is in sync with the current master. Although this is not a robust test and will need to be modified.

release binaries dependency change summary:
target packages:
  A tar 0.4.33 (direct third-party, crates.io)
    * features: default, xattr
  A filetime 0.2.14 (transitive third-party, crates.io)
    * features: [none]
  A xattr 0.2.2 (transitive third-party, crates.io)
    * features: default, unsupported

Question: how does this compare to the approach of requiring a directory of Move binaries to be checked into the repo as part of the release process? This way we wouldn't need to fetch data from a previous git commit.
- diem-framework-releases
  - 20210101
  - 20210201

Future backward compatibility testing for Diem Framework across multiple releases

Can you elaborate on the kinds of tests we would want to run with this?

Old PR, closing out, feel free to re-open.


Support Two labels for slide toggle

Feature Description
Please support two labels for slide toggle, one on left side and one on right side of toggle button.
An attribute via which it will support two labels or single label on either side at a time.
Use Case
Currently if we click on slide toggle's label which is on right side, the button will work fine, but if we have label on it's left side and use "for" attribute along with "id" attribute to implement the same click as of right side label, then left label click won't toggle the button.

It is actually possible to provide an external label like this:
<mat-slide-toggle #toggle="matSlideToggle"></mat-slide-toggle>

<label [attr.for]="toggle.inputId">I am an external label</label>

We have no plans to add a feature like this directly in the component.


Running kind-setup.sh with no arguments fails to locate image

Describe the bug

When following the instructions given here and running kind-setup.sh with no arguments (on KinD v0.14), the script is unable to locate and therefore load the image named projects.registry.vmware.com/antrea/antrea-ubuntu:latest.
To Reproduce

Have KinD v0.14 installed.
Clone the antrea repo locally.
Run ./ci/kind/kind-setup.sh create antrea per documentation instructions.
See the following output printed (last line)

No kind clusters found.
Creating cluster "antrea" ...
 ✓ Ensuring node image (kindest/node:v1.24.0) 🖼
 ✓ Preparing nodes 📦 📦 📦
 ✓ Writing configuration 📜
 ✓ Starting control-plane 🕹
 ✓ Installing StorageClass 💾
 ✓ Joining worker nodes 🚜
Set kubectl context to "kind-antrea"
You can now use your cluster with:

kubectl cluster-info --context kind-antrea

Have a nice day! 👋
deployment.apps/coredns patched
Configuring networks
Using default kind docker network
load images
docker image projects.registry.vmware.com/antrea/antrea-ubuntu:latest not found

Observe that Antrea agent Pods never become ready

$ k get po -A
NAMESPACE            NAME                                           READY   STATUS                  RESTARTS       AGE
kube-system          antrea-agent-9hjqn                             0/2     Init:CrashLoopBackOff   5 (86s ago)    5m35s
kube-system          antrea-agent-w95zm                             0/2     Init:CrashLoopBackOff   5 (96s ago)    5m35s
kube-system          antrea-agent-wklfx                             0/2     Init:CrashLoopBackOff   5 (101s ago)   5m35s
kube-system          antrea-controller-7d8bbd857-9r6wx              1/1     Running                 0              5m36s
kube-system          coredns-6d4b75cb6d-b2f5j                       0/1     ContainerCreating       0              6m2s
kube-system          coredns-b4b5969d4-fqqdc                        0/1     ContainerCreating       0              5m41s
kube-system          coredns-b4b5969d4-lfnx5                        0/1     ContainerCreating       0              5m41s
kube-system          etcd-antrea-control-plane                      1/1     Running                 0              6m16s
kube-system          kube-apiserver-antrea-control-plane            1/1     Running                 0              6m16s
kube-system          kube-controller-manager-antrea-control-plane   1/1     Running                 0              6m19s
kube-system          kube-proxy-fpsxj                               1/1     Running                 0              6m3s
kube-system          kube-proxy-ttj26                               1/1     Running                 0              5m57s
kube-system          kube-proxy-vznxp                               1/1     Running                 0              5m44s
kube-system          kube-scheduler-antrea-control-plane            1/1     Running                 0              6m16s
local-path-storage   local-path-provisioner-9cd9bd544-7629z         0/1     ContainerCreating       0              6m2s

Expected

A KinD cluster is built with the specified version of Antrea and everything is in a healthy position.
Actual behavior

Antrea does not become healthy and therefore cannot be fully used.
Versions:
Please provide the following information:

Antrea version (Docker image tag). v1.6.1
Kubernetes version (use kubectl version). If your Kubernetes components have different versions, please provide the version for all of them. v1.24.0
Container runtime: which runtime are you using (e.g. containerd, cri-o, docker) and which version are you using? Docker
Linux kernel version on the Kubernetes Nodes (uname -r). <IP_ADDRESS>-microsoft-standard-WSL2
If you chose to compile the Open vSwitch kernel module manually instead of using the kernel module built into the Linux kernel, which version of the OVS kernel module are you using? Include the output of modinfo openvswitch for the Kubernetes Nodes.

Additional context

After reading the documentation further, once I manually pull the docker pull projects.registry.vmware.com/antrea/antrea-ubuntu:latest image and re-run the script, it complete successfully. There either needs to be:

A provision in the script to detect and pull (if missing) the appropriate image prior to attempting the load or;
a note in the documentation stating that is a prerequisite to running the kind-setup.sh script.

@chipzoller thanks for the details. kind-setup.sh is a convenience script typically used for testing a local build so it was made to not pull the image automatically. But as you suggested we could make it more friendly by detecting whether it's available locally before loading it and throwing error.
BTW, if loading local antrea image to kind nodes failed, the nodes would still work as they could pull the image from public repo. I guess anrea-agent went to Init:CrashLoopBackOff status because some required kernel modules in the WSL2 backend were missing as the doc https://github.com/antrea-io/antrea/blob/main/docs/kind.md#antrea-agents-are-not-starting-on-windows-what-could-it-be listed? If you could build a custom Kernel for WSL 2 with the listed modules, it should work.

I can work on adding this functionality to the script.


What exactly does babel-plugin-istanbul support that nyc doesn't?

I just tried using nyc for the first time on a large babel project and it seemed to work fine without the babel-plugin-istanbul. What am I missing out on?
Here's the scripts I'm using:
  "scripts": {
    "test": "nyc npm run test:fast",
    "build": "npm run clean && babel ./src --out-dir ./lib --copy-files",
    "clean": "rimraf ./lib",
    "lint": "eslint src/ test/",
    "pretest": "npm run lint",
    "start": "node ./lib/bin/server.js",
    "start:watch": "babel-watch ./src/bin/server.js",
    "test:fast": "NODE_ENV=test babel-tape-runner test/**/*.test.js",
    "test:custom": "NODE_ENV=test babel-tape-runner test/custom.js",
    "test:watch": "nodemon --exec npm -- run --silent test:fast || true",
    "test:watch:custom": "nodemon --exec npm -- run --silent test:custom || true",
  },

@olalonde if you're running html reports, nyc's source-map used to sometimes have bugs mapping lines to the ES2015 transpiled code. tldr; babel-plugin-istanbul fairly consistently returns more accurate line #s -- having said this, this seems like a bug that should be fixed upstream in the source-map libraries eventually.
The other advantage, I believe, is that babel-plugin-istanbul better fits some developers workflows around webpack, etc.


New modes - more liberty or more pressures?

@OpenScienceMOOC asked me to raise this here as a really cool discussion point. I went through the trouble of registering and posting this here, so this would better be an ace discussion ;):
Good point about researchers being more than their pubs. But are you not replicating the same logics by what you propose? Just with more things & more pressures for researchers? Autonomy & individuality are beneficial aspects, though.

Ha ha, sorry. Just edited the above. Of course I did not mean "ass discussion" but "ace discussion"...

This is a great question to ask, @gorgikrlev, thanks. So, what would be your suggested alternatives to using eg ORCID/OSF/ImpactStory/Publons. Are there ways to maintain autonomy and individuality without using such platforms to document your work?
I agree that some times it can all be too much, platform fatigue and all that, which is why we start with ORCID as this is a fully interoperably 'profile', so should reduce any pressure. In theory.

Well, actually, I guess what we are trying to do is pull people away from those platforms, and use more open, more effective ones! I guess one thing we could do in the future is document all possible alternatives, but the 101 innovations in scholarly communication already largely has this covered.
Science Communication will also play a factor later on in a different module.

Good ambition for sure! Yes, that would solve most of the problems.
I just discovered I had seen the graph on Twitter but the project itself did not strike a note. Interesting in terms of branding. Yes, they seem to have it all. But what in fact would be useful is a life repository with categories, short profiles, maybe ratings and links to the tools. That did not seem to exist (or maybe I've only not looked thoroughly) but would be lots of effort of course.

Yes, living repository is the way forward also in my view.
I have a Skype call tomorrow with Thomas Cruzier - the curator of http://connectedresearchers.com/online-tools-for-researchers/
on how to keep this alive - including user experience reviews and discipline specific reasonable workflows - also considering project specific demands - and suggested workflows with updates on https://101innovations.wordpress.com/workflows/
esp highlighting

https://101innovations.files.wordpress.com/2018/02/workflows-wikimedia1.png
and
https://101innovations.files.wordpress.com/2018/02/workflows-open-science.png and of course
https://osf.io/

Great johav, and thanks for pointing me to all these resources. Hope you'll find a good way to realise the ambition!


[MM-32167] Alphabetical and recency sorted categories will now have the same drop behaviour as DMs

Summary
When dragging and dropping a channel from one category into a category that has a specific sort order set (ie. Alphabetical, Recency), the default drag and drop behaviour is to move the current channels in the category around the new channel when dragging around, giving the user a perception that the new channel would be placed in that exact spot, which wouldn't make sense if the channel is automatically sorted in some way.
This PR takes the same behaviour for dragging and dropping used for the DM category, which is to just outline the category and fade the channel in as it is dropped, and applies that behaviour to other categories that are automatically sorted in the same way.
Ticket Link
https://mattermost.atlassian.net/browse/MM-32167

@amyblais this should be for cloud Jan 26 + v5.32

@amyblais this should be for cloud Jan 26 + v5.32

Thanks @devinbinnie
Seeing one issue with the timing/location the outline is triggered, when dragging into a category that has a specific sort order.  For Example, select specific sort order on 2 custom categories, drag from top category into a lower one over category name. Observe the outline does not trigger until you pass the category name.
This also impacts the ability to drop channels into collapsed category, when dragging channels from top to bottom on the sidebar.
note: Not seeing on Favorites category though.
https://user-images.githubusercontent.com/52937121/105191984-ab30d380-5b05-11eb-8c18-71b94d32b02b.mov

Thanks @devinbinnie
Seeing one issue with the timing/location the outline is triggered, when dragging into a category that has a specific sort order.  For Example, select specific sort order on 2 custom categories, drag from top category into a lower one over category name. Observe the outline does not trigger until you pass the category name.
This also impacts the ability to drop channels into collapsed category, when dragging channels from top to bottom on the sidebar.
note: Not seeing on Favorites category though.
https://user-images.githubusercontent.com/52937121/105191984-ab30d380-5b05-11eb-8c18-71b94d32b02b.mov

Thanks @devinbinnie
Seeing one issue with the timing/location the outline is triggered, when dragging into a category that has a specific sort order. For Example, select specific sort order on 2 custom categories, drag from top category into a lower one over category name. Observe the outline does not trigger until you pass the category name.
This also impacts the ability to drop channels into collapsed category, when dragging channels from top to bottom on the sidebar.
note: Not seeing on Favorites category though.
Screen.Recording.2021-01-20.at.9.49.55.AM.mov

Yeah, this is going to be a problem. The issue is the same one we've encountered with the DM category where dragging into it causes the hitbox not to register immediately. Now unfortunately we can have any arbitrary number of categories like that which is going to mess up dragging even more.
I'll try and see if I can find a solution to this once and for all.

Thanks @devinbinnie
Seeing one issue with the timing/location the outline is triggered, when dragging into a category that has a specific sort order. For Example, select specific sort order on 2 custom categories, drag from top category into a lower one over category name. Observe the outline does not trigger until you pass the category name.
This also impacts the ability to drop channels into collapsed category, when dragging channels from top to bottom on the sidebar.
note: Not seeing on Favorites category though.
Screen.Recording.2021-01-20.at.9.49.55.AM.mov

Yeah, this is going to be a problem. The issue is the same one we've encountered with the DM category where dragging into it causes the hitbox not to register immediately. Now unfortunately we can have any arbitrary number of categories like that which is going to mess up dragging even more.
I'll try and see if I can find a solution to this once and for all.

Thanks @devinbinnie
Seeing one issue with the timing/location the outline is triggered, when dragging into a category that has a specific sort order. For Example, select specific sort order on 2 custom categories, drag from top category into a lower one over category name. Observe the outline does not trigger until you pass the category name.
This also impacts the ability to drop channels into collapsed category, when dragging channels from top to bottom on the sidebar.
note: Not seeing on Favorites category though.
Screen.Recording.2021-01-20.at.9.49.55.AM.mov

Yeah, this is going to be a problem. The issue is the same one we've encountered with the DM category where dragging into it causes the hitbox not to register immediately. Now unfortunately we can have any arbitrary number of categories like that which is going to mess up dragging even more.
I'll try and see if I can find a solution to this once and for all.
Link to original bug for context: https://mattermost.atlassian.net/browse/MM-26399

@jgilliam17 I think I have a fix for it, can you have a look and see if that works better for you?

Thanks @devinbinnie
Seeing one issue with the timing/location the outline is triggered, when dragging into a category that has a specific sort order. For Example, select specific sort order on 2 custom categories, drag from top category into a lower one over category name. Observe the outline does not trigger until you pass the category name.
This also impacts the ability to drop channels into collapsed category, when dragging channels from top to bottom on the sidebar.
note: Not seeing on Favorites category though.
Screen.Recording.2021-01-20.at.9.49.55.AM.mov

Yeah, this is going to be a problem. The issue is the same one we've encountered with the DM category where dragging into it causes the hitbox not to register immediately. Now unfortunately we can have any arbitrary number of categories like that which is going to mess up dragging even more.
I'll try and see if I can find a solution to this once and for all.
Link to original bug for context: https://mattermost.atlassian.net/browse/MM-26399

@jgilliam17 I think I have a fix for it, can you have a look and see if that works better for you?

@devinbinnie
Sticky headers are interfering with the category outline on Chrome, Edge and desktop app (not on Safari or Firefox, no sticky headers)
I probably missed this earlier
https://user-images.githubusercontent.com/52937121/105229970-b5b59200-5b32-11eb-9a4e-8b9e88a14ddc.mov

@devinbinnie
Sticky headers are interfering with the category outline on Chrome, Edge and desktop app (not on Safari or Firefox, no sticky headers)
I probably missed this earlier
https://user-images.githubusercontent.com/52937121/105229970-b5b59200-5b32-11eb-9a4e-8b9e88a14ddc.mov

/update-branch

/update-branch

@jgilliam17 Should be fixed now

@jgilliam17 Should be fixed now

Thanks @devinbinnie
The category name/sticky header issue is fixed, looks good.
One small UX issue
I am not seeing a complete outline on collapsed category, bottom is missing

Thanks @devinbinnie
The category name/sticky header issue is fixed, looks good.
One small UX issue
I am not seeing a complete outline on collapsed category, bottom is missing

@deanwhillier @jgilliam17 Okay so I've fixed the issue by adding a hack, forcing a min-height of 2px on the category content.
In order to make sure the box-shadow shows behind the sidebar header button, we need to force it to have its own box-shadow. This also applies when the header is sticky and needs to continue to show an outline when its parent div is hidden during scrolling. However, we only want it to show the top, left and right sides of the box-shadow on the header, because if there are unreads in the channel, the border will show on the bottom of the header, splitting the header and the content. When the category is collapsed however, the entire parent div is taken up by the header button, blocking off the bottom part of the box-shadow that would normally show. Thus, we need the extra 2px minimum at the bottom of the category content to ensure the bottom of the box-shadow always shows.
If we were able to make the box-shadow show overtop of the button or the header div, that would solve the issue. The reason it does is due to sticky headers requiring an explicit background to be set on the header so that channels do not show underneath it. Removing that would allow the box-shadow to work, but would break the sticky header.
Asking for re-review on CSS in case there are any better solutions, but I think this one impacts UX as minimally as possible.

@deanwhillier @jgilliam17 Okay so I've fixed the issue by adding a hack, forcing a min-height of 2px on the category content.
In order to make sure the box-shadow shows behind the sidebar header button, we need to force it to have its own box-shadow. This also applies when the header is sticky and needs to continue to show an outline when its parent div is hidden during scrolling. However, we only want it to show the top, left and right sides of the box-shadow on the header, because if there are unreads in the channel, the border will show on the bottom of the header, splitting the header and the content. When the category is collapsed however, the entire parent div is taken up by the header button, blocking off the bottom part of the box-shadow that would normally show. Thus, we need the extra 2px minimum at the bottom of the category content to ensure the bottom of the box-shadow always shows.
If we were able to make the box-shadow show overtop of the button or the header div, that would solve the issue. The reason it does is due to sticky headers requiring an explicit background to be set on the header so that channels do not show underneath it. Removing that would allow the box-shadow to work, but would break the sticky header.
Asking for re-review on CSS in case there are any better solutions, but I think this one impacts UX as minimally as possible.

Devin - category drop border looks good; also the issues with collapsed categories border is fixed.

There's a one new issue with moving channels around in the manually sorted category that I haven't seen on master or daily and not consistently reproducing on the PR. Video is from Firefox, but I also saw this on Chrome.
I can file separately for investigation if it is not related to your changes. Let me know.

https://user-images.githubusercontent.com/52937121/105447619-89893680-5c42-11eb-8239-d1e61b338be4.mov

Devin - category drop border looks good; also the issues with collapsed categories border is fixed.

There's a one new issue with moving channels around in the manually sorted category that I haven't seen on master or daily and not consistently reproducing on the PR. Video is from Firefox, but I also saw this on Chrome.
I can file separately for investigation if it is not related to your changes. Let me know.

https://user-images.githubusercontent.com/52937121/105447619-89893680-5c42-11eb-8239-d1e61b338be4.mov

Devin - category drop border looks good; also the issues with collapsed categories border is fixed.

There's a one new issue with moving channels around in the manually sorted category that I haven't seen on master or daily and not consistently reproducing on the PR. Video is from Firefox, but I also saw this on Chrome.
I can file separately for investigation if it is not related to your changes. Let me know.

Screen.Recording.2021-01-21.at.11.12.03.PM.mov

Hmm, looks like there's something odd going on with respect to actually re-ordering the channels. @hmhealey @nevyangelova Any thoughts?

Devin - category drop border looks good; also the issues with collapsed categories border is fixed.

There's a one new issue with moving channels around in the manually sorted category that I haven't seen on master or daily and not consistently reproducing on the PR. Video is from Firefox, but I also saw this on Chrome.
I can file separately for investigation if it is not related to your changes. Let me know.

Screen.Recording.2021-01-21.at.11.12.03.PM.mov

Hmm, looks like there's something odd going on with respect to actually re-ordering the channels. @hmhealey @nevyangelova Any thoughts?

Devin - category drop border looks good; also the issues with collapsed categories border is fixed.

There's a one new issue with moving channels around in the manually sorted category that I haven't seen on master or daily and not consistently reproducing on the PR. Video is from Firefox, but I also saw this on Chrome.
I can file separately for investigation if it is not related to your changes. Let me know.

Screen.Recording.2021-01-21.at.11.12.03.PM.mov

Hmm, looks like there's something odd going on with respect to actually re-ordering the channels. @hmhealey @nevyangelova Any thoughts?

I can't personally reproduce this issue on the test server but I also don't see the manual sorting logic in 'selectors/entities/channel_categories.ts' in MM-redux. What would classify as "Default"?
  if (category.sorting === CategorySorting.Recency) {
            channels = sortChannelsByRecency(state, channels);
        } else if (category.sorting === CategorySorting.Alphabetical || category.sorting === CategorySorting.Default) {
            if (channels.some((channel) => channel.type === General.DM_CHANNEL || channel.type === General.GM_CHANNEL)) {
                channels = sortChannelsByNameWithDMs(state, channels);
            } else {
                channels = sortChannelsByName(state, channels);
            }
        }

Devin - category drop border looks good; also the issues with collapsed categories border is fixed.

There's a one new issue with moving channels around in the manually sorted category that I haven't seen on master or daily and not consistently reproducing on the PR. Video is from Firefox, but I also saw this on Chrome.
I can file separately for investigation if it is not related to your changes. Let me know.

Screen.Recording.2021-01-21.at.11.12.03.PM.mov

Hmm, looks like there's something odd going on with respect to actually re-ordering the channels. @hmhealey @nevyangelova Any thoughts?

I can't personally reproduce this issue on the test server but I also don't see the manual sorting logic in 'selectors/entities/channel_categories.ts' in MM-redux. What would classify as "Default"?
  if (category.sorting === CategorySorting.Recency) {
            channels = sortChannelsByRecency(state, channels);
        } else if (category.sorting === CategorySorting.Alphabetical || category.sorting === CategorySorting.Default) {
            if (channels.some((channel) => channel.type === General.DM_CHANNEL || channel.type === General.GM_CHANNEL)) {
                channels = sortChannelsByNameWithDMs(state, channels);
            } else {
                channels = sortChannelsByName(state, channels);
            }
        }

@jgilliam17 There's two things I could see causing that: either the UI and redux are out of sync for what the sorting order of that category actually is or there's an error somewhere that's causing the server to prevent you from reordering those channels similar to what was happening in the past when the server prevented you from reordering categories with deleted channels in them. Are there any errors in the logs when that happens?
That said, that wouldn't be affected at all by this PR, so could we file a separate ticket for that?
@nevyangelova "Default" doesn't have any option in the UI. It's treated the same as alphabetical, except when the user starts to rearrange the sidebar, it updates the Category.sorting to manual. I had to add that way back so that the sidebar would behave the same as before after migration (ie it would sort alphabetically), but we wanted it to start manually sorting as soon as the user tried reordering their channels.

@jgilliam17 There's two things I could see causing that: either the UI and redux are out of sync for what the sorting order of that category actually is or there's an error somewhere that's causing the server to prevent you from reordering those channels similar to what was happening in the past when the server prevented you from reordering categories with deleted channels in them. Are there any errors in the logs when that happens?
That said, that wouldn't be affected at all by this PR, so could we file a separate ticket for that?
@nevyangelova "Default" doesn't have any option in the UI. It's treated the same as alphabetical, except when the user starts to rearrange the sidebar, it updates the Category.sorting to manual. I had to add that way back so that the sidebar would behave the same as before after migration (ie it would sort alphabetically), but we wanted it to start manually sorting as soon as the user tried reordering their channels.

Cherry pick is scheduled.

Cherry pick is scheduled.

Error trying doing the automated Cherry picking. Please do this manually
+++ Updating remotes...
Fetching upstream
Failed to add the RSA host key for IP address '<IP_ADDRESS>' to the list of known hosts (/app/.ssh/known_hosts).
From github.com:mattermost/mattermost-webapp
   28339d586..682fbb7f5  MM-32156_gm-loading -> upstream/MM-32156_gm-loading
 + 20b217140...aba1dcd06 campaign/applytheme_sidebar-header-bg -> upstream/campaign/applytheme_sidebar-header-bg  (forced update)
   c593f3c96..8358a0b0e  cloud               -> upstream/cloud
   4716206ff..9eba5ecc2  feature/cloud-apps  -> upstream/feature/cloud-apps
   50229ed51..da7bb47ec  master              -> upstream/master
Fetching origin
Failed to add the RSA host key for IP address '<IP_ADDRESS>' to the list of known hosts (/app/.ssh/known_hosts).
+++ Updating remotes done...
!!! 'upstream/release-5.32' not found. The second argument should be something like upstream/release-0.21.
    (In particular, it needs to be a valid, existing remote branch that I can 'git checkout'.)

+++ Returning you to the master branch and cleaning up.

Error trying doing the automated Cherry picking. Please do this manually
+++ Updating remotes...
Fetching upstream
Failed to add the RSA host key for IP address '<IP_ADDRESS>' to the list of known hosts (/app/.ssh/known_hosts).
From github.com:mattermost/mattermost-webapp
   28339d586..682fbb7f5  MM-32156_gm-loading -> upstream/MM-32156_gm-loading
 + 20b217140...aba1dcd06 campaign/applytheme_sidebar-header-bg -> upstream/campaign/applytheme_sidebar-header-bg  (forced update)
   c593f3c96..8358a0b0e  cloud               -> upstream/cloud
   4716206ff..9eba5ecc2  feature/cloud-apps  -> upstream/feature/cloud-apps
   50229ed51..da7bb47ec  master              -> upstream/master
Fetching origin
Failed to add the RSA host key for IP address '<IP_ADDRESS>' to the list of known hosts (/app/.ssh/known_hosts).
+++ Updating remotes done...
!!! 'upstream/release-5.32' not found. The second argument should be something like upstream/release-0.21.
    (In particular, it needs to be a valid, existing remote branch that I can 'git checkout'.)

+++ Returning you to the master branch and cleaning up.

/cherry-pick cloud

/cherry-pick cloud

Cherry pick is scheduled.

Cherry pick is scheduled.

Error trying doing the automated Cherry picking. Please do this manually
+++ Updating remotes...
Fetching upstream
Failed to add the RSA host key for IP address '<IP_ADDRESS>' to the list of known hosts (/app/.ssh/known_hosts).
Fetching origin
Failed to add the RSA host key for IP address '<IP_ADDRESS>' to the list of known hosts (/app/.ssh/known_hosts).
+++ Updating remotes done...
+++ Creating local branch automated-cherry-pick-of-mattermost-webapp-#7348-upstream-cloud-1611581049
Switched to a new branch 'automated-cherry-pick-of-mattermost-webapp-#7348-upstream-cloud-1611581049'
Branch 'automated-cherry-pick-of-mattermost-webapp-#7348-upstream-cloud-1611581049' set up to track remote branch 'cloud' from 'upstream'.

+++ About to attempt cherry pick of PR #7348 with merge commit da7bb47ec46ea98945caf185105c60f7aaf5edaf.

error: could not apply da7bb47ec... [MM-32167] Alphabetical and recency sorted categories will now have the same drop behaviour as DMs (#7348)
hint: after resolving the conflicts, mark the corrected paths
hint: with 'git add <paths>' or 'git rm <paths>'
hint: and commit the result with 'git commit'

+++ Conflicts detected:

UU e2e/cypress/integration/bot_accounts/in_lists_spec.js
Aborting.

+++ Aborting in-progress git cherry-pick.

+++ Returning you to the master branch and cleaning up.

Error trying doing the automated Cherry picking. Please do this manually
+++ Updating remotes...
Fetching upstream
Failed to add the RSA host key for IP address '<IP_ADDRESS>' to the list of known hosts (/app/.ssh/known_hosts).
Fetching origin
Failed to add the RSA host key for IP address '<IP_ADDRESS>' to the list of known hosts (/app/.ssh/known_hosts).
+++ Updating remotes done...
+++ Creating local branch automated-cherry-pick-of-mattermost-webapp-#7348-upstream-cloud-1611581049
Switched to a new branch 'automated-cherry-pick-of-mattermost-webapp-#7348-upstream-cloud-1611581049'
Branch 'automated-cherry-pick-of-mattermost-webapp-#7348-upstream-cloud-1611581049' set up to track remote branch 'cloud' from 'upstream'.

+++ About to attempt cherry pick of PR #7348 with merge commit da7bb47ec46ea98945caf185105c60f7aaf5edaf.

error: could not apply da7bb47ec... [MM-32167] Alphabetical and recency sorted categories will now have the same drop behaviour as DMs (#7348)
hint: after resolving the conflicts, mark the corrected paths
hint: with 'git add <paths>' or 'git rm <paths>'
hint: and commit the result with 'git commit'

+++ Conflicts detected:

UU e2e/cypress/integration/bot_accounts/in_lists_spec.js
Aborting.

+++ Aborting in-progress git cherry-pick.

+++ Returning you to the master branch and cleaning up.


samples/net/http_client: Failed to connect to samples/net/http_server

I was trying to test http_server along with http_client with two different frdm_k64f boards.
The steps I have followed for http_server & http_client is as below:
cd zephyr/samples/net/http_client
mkdir build && cd build && cmake -GNinja -DBOARD=frdm_k64f ../
ninja
cp -vr outdir/frdm_k64f/zephyr.bin /run/media/MBED/
cd zephyr/samples/net/http_server
mkdir build && cd build && cmake -GNinja -DBOARD=frdm_k64f ../
ninja
cp -vr outdir/frdm_k64f/zephyr.bin /run/media/MBED1/
•Here I am trying make one frdm_k64f as http_client & other one as http_server.
•I have done below changes in prj.conf files in http_client sample apps.
CONFIG_NET_APP_MY_IPV6_ADDR="2001:db8::1"
CONFIG_NET_APP_MY_IPV4_ADDR="<IP_ADDRESS>"
CONFIG_NET_APP_PEER_IPV6_ADDR="2001:db8::2"
CONFIG_NET_APP_PEER_IPV4_ADDR="<IP_ADDRESS>"

On http_sever sample I have done any changes in prj.conf file.

•After flashing both codes http_client is not able to connect to http_server on console.
On http_client console :-
***** BOOTING ZEPHYR OS v1.10.99 - BUILD: Feb 28 2018 08:56:05 *****
Zephyr Shell, Zephyr version: 1.10.99
Type 'help' for a list of available commands
shell> [dev/eth_mcux] [INF] eth_mcux_phy_event: Enabledduplex mode.
[http-client] [INF] main: --------Sending 20 sync request--------
[http-client] [INF] do_sync_http_req: [1] Send /index.html
[http/client] [DBG] http_client_send_req: (0x20003838): Cannot connect to server (-60)
[http-client] [ERR] do_sync_http_req: Cannot send GET request (-60)
[http/client] [DBG] http_closed: (0x20003838): [0x20000000] connection closed
[http-client] [INF] main: Done!
I have tested on latest commit today

@tarunkum Can you try with https://github.com/zephyrproject-rtos/zephyr/pull/6167 PR.

@tarunkum ping (FYI @vikramx2112)

@rveerama1
I took the latest branch use the same configuration as I mentioned earlier. I am not able to find the communication between server-client.
Please find the below message log of http_client  on serial window.
***** BOOTING ZEPHYR OS v1.11.99 - BUILD: Mar 28 2018 06:27:55 *****
[dev/eth_mcux] [INF] eth_mcux_phy_event: Enabled 100M full-duplex mode.
~x~] [INF] main: --------Sending 20 sync request--------
[http-client] [INF] do_sync_http_req: [1] Send /index.html
[net/tcp] [DBG] net_tcp_change_staa8): [0x200 CLOSED (0) (2)
[net/tcp] [DBG] net_tcp_trace: (0x200038a8): [0x200029f8] pkt 0x200055dc s0(3 (0/0) flags uaprSf win 28b
[http/client] [DBG] http_client_send_req: (0x200038a8): Cannot connect to server (-60)
[http-client] [ERR] do_sync_http_req: Cannot send GET request (-60)
[http/client] [DBG] http_closed: (0x200038a8): [0x20000000] connection closed
[net/tcp] [DBG] net_tcp_change_state: (0x200038a8): [0x200029f8] state SYN_SENT (2) => CLOSED (0)
[net/tcp] [DBG] net_tcp_release: (0x200038a8): [0x200029f8] Disposed of TCP connection state
[http-client] [INF] main: Done!

@tarunkum http_server and http_client samples are for different purpose. They are implemented in way to communicate each other. http_server sample expects different client and http_client expects different server capabilities.
The reason in your test these samples can not even establish TCP connection is http_server listening on port 80 and client tries to connect on port 8000. You have changed these configuration
CONFIG_NET_APP_MY_IPV6_ADDR="2001:db8::1"
CONFIG_NET_APP_MY_IPV4_ADDR="<IP_ADDRESS>"
CONFIG_NET_APP_PEER_IPV6_ADDR="2001:db8::2"
CONFIG_NET_APP_PEER_IPV4_ADDR="<IP_ADDRESS>"
but forgot to modify port numbers.
diff --git a/samples/net/http_client/src/config.h b/samples/net/http_client/src/config.h
index ae5cff887..d43561fd3 100644
--- a/samples/net/http_client/src/config.h
+++ b/samples/net/http_client/src/config.h
@@ -17,7 +17,7 @@
#if defined(CONFIG_HTTPS)
#define SERVER_PORT  4443
#else
-#define SERVER_PORT  8000
+#define SERVER_PORT  80
#endif
I have tried modifies port number and they are able to created at-least TCP connection. But you do get
[http-client] [ERR] do_sync_http_req: Cannot send GET request (-60).
Reason I mentioned already is both the samples expects different counterparts as client and server.
So this issue is not valid, hence closing.


Adds a small message to the readme.

This Pull Request adds a small explanation about who is working on this repository ✨
@Mike-inc/developers

👍 Added the team members otherwise looks good.


pod with only startupProbe cannot be ready right now when startupProbe succeeded

From kubernetes created by liuhougang: kubernetes/kubernetes#116238
What happened?
pod (only one container) with only startupProbe(without readinessProbe/livenessProbe) cannot be ready right now when startupProbe succeeded like below (it change to ready after 1 min since startupProbe succeeded ):
startupProbe:
          failureThreshold: 30
          httpGet:
            path: /eeadb7c9-3ee0-4136-a1be-051d754614ba/lab
            port: 8888
          initialDelaySeconds: 1
          periodSeconds: 2
          successThreshold: 2

What did you expect to happen?
just like a pod without any Probe which changes to ready right now when container started; I expect that the pod with only startupProbe can change to Ready soon after startupProbe succeeded
How can we reproduce it (as minimally and precisely as possible)?
it can reproduce every time in my k8s（1.19）
Anything else we need to know?
No response
Kubernetes version

$ kubectl version
Client Version: version.Info{Major:"1", Minor:"19", GitVersion:"v1.19.10", GitCommit:"98d5dc5d36d34a7ee13368a7893dcb400ec4e566", GitTreeState:"clean", BuildDate:"2021-04-15T03:28:42Z", GoVersion:"go1.15.10", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"19+", GitVersion:"v1.19.10-r0-CCE22.3.1.B012", GitCommit:"cf078ac3913556bdf46374341f51a4e0fc654113", GitTreeState:"clean", BuildDate:"2022-03-26T10:12:19Z", GoVersion:"go1.15.15", Compiler:"gc", Platform:"linux/amd64"}

Cloud provider

minikube

OS version

# On Linux:
$ cat /etc/os-release
# paste output here
$ uname -a
# paste output here

# On Windows:
C:\> wmic os get Caption, Version, BuildNumber, OSArchitecture
# paste output here

Install tools

Container runtime (CRI) and version (if applicable)

Related plugins (CNI, CSI, ...) and versions (if applicable)

And it is similar to https://github.com/kubernetes/kubernetes/issues/109802#issuecomment-1118675506.

/sig node

And it is similar to #109802 (comment).

thanks, I think it is same issue


chore: add .editorconfig and enable in CI

Closes #11895.

I intentionally let this fail CI once so anyone can see what the failure looks like: https://circleci.com/gh/influxdata/influxdb/26574

Requesting review from those who left +1s on #11895.


Link to release notes from the About page.

We're using the "What's New" feature in the app stores for this; if we want to link to the changelog we can add a lower-level changelog link in that summary.


Add test cases for user_can_delete_user method

This pull request adds three new test cases for the "user_can_delete_user" method. The tests cover the following scenarios:
1. user_to_be_deleted_is_superuser_and_current_user_is_not: Verifies that a superuser cannot be deleted by a user who is not a superuser.
2. user_to_be_deleted_is_superuser_and_current_user_is_too: Verifies that a superuser can delete another superuser.
3. user_to_be_deleted_is_not_superuser_and_current_user_is: Verifies that a superuser can delete a user who is not a superuser.

Thanks for the PR, but I don't think this is adding much value to the test suite.
I'd recommend looking at some of the other test modules to see how Wagtail's tests are usually written - generally we prefer to test end-to-end behaviour rather than individual internal functions. So, for example, if we wanted to test that a non-superuser is not able to delete a superuser, we'd do that by sending a request to the 'delete' view and confirming that it did not succeed. (And, in fact, there is already a test doing this.) Calling the function and passing mock objects doesn't really check that the function is working as intended - it's basically just repeating the logic from inside the function.
Also, the os.environ['DJANGO_SETTINGS_MODULE'] and if __name__ == '__main__' lines are unnecessary - if we want to run a test individually, we can do that by passing the module name to runtests.py. (I don't think the DJANGO_SETTINGS_MODULE line is correct - 'os.environ.setdefault' is not a meaningful value for the settings module).


The option "extra_options" does not exist.

Hi
I'm now upgrading my Symfony 6.3 application to 6.4 and I've also updated symfony/ux-autocomplete to the 2.15.0 version.
Now each time I'm trying to use my autocomplete field I'm getting error:
An error has occurred resolving the options of the form "App\Form\Autocomplete\MemberSingleAutocompleteField": The option "extra_options" does not exist. Defined options are: "action", "allow_extra_fields", ....
I've noticed that this extra_options was introduced in https://github.com/symfony/ux/pull/1322.
Should I stick to 2.14 version in order to use it with 6.4 Symfony?

The extra_options exists in 2.15.0, but it is available for form types extending from:
        return [
            ChoiceType::class,
            TextType::class,
        ];

What does your MemberSingleAutocompleteField descend from ?

I am getting the error, too.
An error has occurred resolving the options of the form "App\Form\KundendatenAutocompleteField": The option "extra_options" does not exist. Defined options are: "action",
The relevant parts of KundendatenAutocompleteField:
use Symfony\Component\Form\Extension\Core\Type\ChoiceType; ... class KundendatenAutocompleteField extends ChoiceType ...
Using Symfony\Component\Form\AbstractType results in the same error.

For autocomplete since 2.13 you need
public function getParent(): string {     return BaseEntityAutocompleteType::class; }
instead of
public function getParent(): string {     return ParentEntityAutocompleteType::class; }
https://symfony.com/bundles/ux-autocomplete/current/index.html#usage-in-a-form-with-ajax
at least that was my issue

@bdujon - brilliant! 😄 It is working now. Thank you!
I wonder that maybe it would be worth to add it as a comment in symfony console make:autocomplete-field?
Autogenerated class uses ParentEntityAutocompleteType::class; which is fine when you are not using Ajax but is not otherwise. Error message was missleading here (at least for me).
For example:
Customize your new field class, then add it to a form:

     $builder
         // ...
         ->add('user', Foo::class)
     ;

Add somethig like: Change getParent() acordingly if using with Ajax.


How to preserve original Cache-Control headers from upstream?

Hey!
I'm using nginx pagespeed only for removing html comments and doing small things which don't require help from beacons.
ngx_pagespeed is hands down the best option for this.
My current flow is:

WordPress sents output from fastcgi (php-fpm).
Pagespeed removes html comments
sr_cache stores response in redis for later requests

I would like to honor the Cache-Control headers from fastcgi and use them in my redis cache. I can't do that because pagespeed forces it's own Cache-Control: max-age=0, no-cache header.
Is there a way to use ngx_pagespeed so that it doesn't alter Cache-Control headers at all?

Sorry! It seems that didn't read the docs enough times. This was the one I was looking for:
pagespeed ModifyCachingHeaders off;


New Spawnable Map: Boarding Shuttle Dart

About the PR
Made a rather small boarding shuttle called Dart, it has a bridge, tiny medical closet, an armory stocked with weapons and hardsuits and a small Engineering with generators, air tanks and such.
Screenshots

Dart Screenshot
Changelog

:cl: Schiene

add: Added a spawnable Boarding Shuttle map called Dart, comes with a small force equipped for Vacuum with heavy weaponry.

No gravity?

No gravity?

It's small enough that Gravity both doesn't fit and isn't needed, every room is so small you can float around without any issues or getting stuck on. Besides it's meant as a boarding shuttle anyways, the Sec force is meant to board the station. (It's Sec force now because it's more a template for an ERT or the nukies or whatever like that.)

Embedded Screenshot

Seems unpowered in-game unless I did something wrong?

Seems unpowered in-game unless I did something wrong?

Unless the wiring broke itself between updates, it should absolutely be working, the generators are wired to the SMES, which are wired ot the Subbie, which itself is wired to the rest of the shuttle.


Amplify.Auth.signInWithWebUI - Remove Confirmation PopUp While Sign in With Google/FaceBook

Describe the bug
Is there any way to get rid of these popup and sent user silently to login page of Google/Facebook ?
To Reproduce
Steps to reproduce the behavior:

Call the method Amplify.Auth.signInWithWebUI(for: .google, presentationAnchor: self.view.window!)
A popup is presented to "App want to use amazoncognito to Sign in" before opening google sign in UI

Expected behavior
Should directly open the google sign in UI
Screenshots

Environment(please complete the following information):

Amplify Framework Version: 1.5.0
Dependency Manager: Cocoapods
Swift Version : 5.0

Hi, @shikhilnuwal
Thanks for reaching out.
So, the signInWithWebUI calls SFAuthenticationSession at some point (see here), the behavior is designed by Apple.
See this StackOverflow link and Apple link
We don't actually have control over the asking of PopUp. Hope this answers your question

Hi, @shikhilnuwal
Thanks for reaching out.
So, the signInWithWebUI calls SFAuthenticationSession at some point (see here), the behavior is designed by Apple.
See this StackOverflow link and Apple link
We don't actually have control over the asking of PopUp. Hope this answers your question

Thanks @ruiguoamz .


fix(type): function type in cjs require

Summary
fix the function type in cjs require
close https://github.com/web-infra-dev/rspack/issues/8095
// in esm
import { rspack } from '@rspack/core';
rspack({ bail: true });

// in cjs
const rspack = require("@rspack/core");
rspack({ bail: true });

Checklist

[ ] Tests updated (or not required).
[ ] Documentation updated (or not required).

This is a solution with a little hack.
Maybe we need to ship both cjs and esm in the future.

Maybe we need to ship both cjs and esm in the future.

Agreed, @rspack/core should be a dual package.


Add symlink, keep README and docs index in sync

Codecov Report

Merging #86 (9598b9d) into main (db08b0c) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #86   +/-   ##
=======================================
  Coverage   98.79%   98.79%
=======================================
  Files          20       20
  Lines         833      833
  Branches       81       81
=======================================
  Hits          823      823
  Misses          5        5
  Partials        5        5

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 db08b0c...9598b9d. Read the comment docs.


Arrow function called within super class constructor

I have a problem I've encountered a few times:
class BaseClass
{
	constructor()
	{
		this.init();
	}

	protected init()
	{
	}
}

class ChildClass extends BaseClass
{
	constructor()
	{
		super();
	}

	protected init()
	{
		var div = document.createElement("div");
		console.log(this.onClick); // undefined
		div.addEventListener("click", this.onClick);
	}

	private onClick = (event: MouseEvent) =>
	{
		// ...
	}
}

var child = new ChildClass();

The problem is that in the init of ChildClass, "onClick" is still undefined because in the compiled JS it is only defined after the super call. This comes up generally when I call functions in the super class that are likely to be overridden in child classes and the child implementations use arrow functions defined in the child class.
I was wondering what was the reason behind this order and wouldn't it be possible to reverse it (ie. define arrow functions first, call super afterwards).
I think it's related to this error message which pops up if you try to call init manually before super():
"A 'super' call must be the first statement in the constructor when a class contains initialized properties or has parameter properties"

Related issues are:
https://typescript.codeplex.com/discussions/444975
http://typescript.codeplex.com/workitem/91
My view of this is that the only reason I'm using arrow functions generally is to make sure the scope (this) is correct, so ideally it would be good to be able to work with them just like if they were "non-arrow", normal prototypical functions.

There isn't any sensible way to redefine the order of initialization of a TypeScript class. See #1617 for an extended explanation of how class emit and other constraints prevent any other initialization order than the one you see today.
More generally, the principle of "don't call virtual methods from constructors" remains good OOP advice.


Cannot run a set of parameters over a single circuit in Sampler

Describe the bug
I have a simple singe-parameter circuit:
th = Parameter('th')
qc = QuantumCircuit(2, 1)

qc.x(1)
qc.h(0)
qc.cp(th, 0, 1)
qc.h(0)
qc.measure(0, 0)

that I would like to bind to 50 parameters

phases = np.linspace(0, 2*np.pi, 50)

and execute.  If I try to do this with the Sampler there is no combination of circuit_indices and parameter_values that works.  For example, following the example here: https://qiskit.org/documentation/partners/qiskit_ibm_runtime/tutorials/how-to-getting-started-with-sampler.html#Multiple-parameterized-circuits-example I would expect that I need circuit_indices = [0]*len(phases) and parameter_values=phases  However this does not work:
sampler = Sampler(circuits=[qc], service=service, options={ "backend": backend })
result = sampler(circuit_indices=[0]*len(phases), parameter_values=phases)

yields:
The number of circuits (50) does not match the number of parameter sets (1)

Ok, then I will just do circuit_indices=[0], but that also fails:

sampler = Sampler(circuits=[qc], service=service, options={ "backend": backend })
result = sampler(circuit_indices=[0], parameter_values=phases)

gives:
The number of values (50) does not match the number of parameters (1) for the 0-th circuit.

I can try the other possible permutations of parameters and they both fail as well

sampler = Sampler(circuits=[qc], service=service, options={ "backend": backend })
result = sampler(circuit_indices=[0], parameter_values=[phases])

gives:
The number of values (50) does not match the number of parameters (1) for the 0-th circuit.

and
sampler = Sampler(circuits=[qc], service=service, options={ "backend": backend })
result = sampler(circuit_indices=[0]*len(phases), parameter_values=[phases])

fails with:
The number of circuits (50) does not match the number of parameter sets (1)

Steps to reproduce
Expected behavior
I should be able to get distributions from a single circuit over multiple parameters.
Suggested solutions
Additional Information

qiskit-ibm-runtime version: 0.4.0
Python version:
Operating system:

I actually figured this out after writing this.  The solution is to put each single parameter in its own list:
phases = np.linspace(0,2*np.pi, 50)
working_phases = [[ph] for ph in phases]

This is not very intuitive nor documented anywhere.

@HuangJunye @tmittal947 @lerongil  (document better on qiskit.org)

I will work on this :)

Sorry I don't have time to work on this issue in the near term so I'll unassign myself. If anyone is interested in fixing this, please go ahead!

I believe this is a bug in Terra, and does not require a documentation change. In qiskit/primitives/base/base_sampler, we have
 def run(
        self,
        circuits: QuantumCircuit | Sequence[QuantumCircuit],
        parameter_values: Sequence[float] | Sequence[Sequence[float]] | None = None,
        **run_options,

so that a sequence of float should be allowed, as defined in the example above.
@jakelishman - can you please have a look?

I'm not on the primitives team, so this would be out of my scope sorry.  @ikkoham is either the right person to ask, or knows the right person to ask.
For what it's worth, the behaviour Paul's seeing matches my understanding of the primitives interface.  Circuits don't take a single float as their "parameters"; it's logically always a sequence of floats.  It just so happens that in Paul's example, that sequence has a length of 1, which leads to confusion.
The solution Paul found in https://github.com/Qiskit/qiskit-ibm-runtime/issues/339#issuecomment-1137386328 looks like the expected behaviour to me - when there was a flat array of 50 elements, it's interpreted (correctly) as one sequence of parameters, so it needs broadcasting to be many sequences of parameters.  The broadcast happens correctly, and then there's a mismatch; each of the 50 circuits wants a sequence of length 1, but a sequence of length 50 was provided.  I think the confusion here is just because each circuit wants a sequence of length 1; I think it'd be clear that the original form can't work if each circuit wants 2 parameters.

Closing, in V2 Sampler you can do

phases = np.linspace(0, 2*np.pi, 50)
sampler = SamplerV2(backend)
job = sampler.run([(isa_circuit, phases)], shots=10)