ENH: plot method accessors

Fixes #9124
This PR adds plotting sub-methods like df.plot.scatter() as an alternative to using df.plot(kind='scatter').
I've added meaningful function signatures and documentation for a few of these methods, but I would greatly appreciate help to fill in the rest -- this is a lot of documentation to assemble/reconstruct! The entire point of this PR, of course, is to have better introspection and docstrings.
Todo list:

[x] Basic docstrings/signatures

[x] area
[x] line
[x] bar
[x] barh
[x] box
[x] hexbin
[x] hist
[x] kde/density
[x] pie
[x] scatter

[x] Write tests for the methods
[x] Fix groupby plots (tests currently failing)
[x] Plotting docs
[x] API docs
[x] Release notes

I'll do a PR against your branch for the rest of the docstrings this Saturday unless someone else gets there first.

Thanks for doing this!

Current policy is nice for docstring thus for users, but I feel this makes consistent maintenance little difficult, because maintainer have to care all the separated functions. I think it is better to have a table which organizes what options are available on every plotting function. Maybe on Wiki?
I'm willing to do, but it takes a while as each function have options not listed in the signature, for example,
https://github.com/pydata/pandas/blob/master/pandas/tools/plotting.py#L1428

@sinhrks Agreed, this is going to be tricky. For a first draft (this PR), I think use of **kwargs for some options is unavoidable (every function will have **kwargs in the signature). Eventually, we can try to stop using it but practically that may not be possible.

I hope to get the docstrings done tonight / tomorrow.

line

data
x
y
use_index
ax
subplots
sharex
sharey
layout
grid
title
legend
logx
logy
loglog
xticks
yticks
rot
xlim
ylim
fontsize
secondary_y
mark_right
figsize
style
colormap
position
table
yerr
xerr
stacked
sort_columns
kwds

bar / barh

data
x
y
use_index
ax
subplots
sharex
sharey
layout
grid
title
legend
logx
logy
loglog
xticks
yticks
rot
xlim
ylim
fontsize
secondary_y
mark_right
figsize
style
colormap
position
table
yerr
xerr
stacked
sort_columns
kwds

hist

column
by
~x~
~y~
~use_index~
ax
subplots
sharex
sharey
layout
grid
title
legend
logx
logy
loglog
xticks
yticks
rot
xlim
ylim
fontsize
secondary_y
mark_right
figsize
style
colormap
table
stacked
sort_columns
kwds

box

ax
subplots
sharex
sharey
layout
grid
title
legend
~logy~ : isn't computed correctly.
rot
xlim
ylim
fontsize
figsize
table
kwds

kde / density

ax
subplots
sharex
sharey
layout
grid
title
legend
logx
logy
loglog
xticks
yticks
rot
xlim
ylim
fontsize
secondary_y
mark_right
figsize
style
colormap
colorbar
table
kwds

area

x
y
use_index
ax
subplots
sharex
sharey
layout
grid
title
legend
logx
logy
loglog
xticks
yticks
rot
xlim
ylim
fontsize
secondary_y
mark_right
figsize
style
table
stacked
sort_columns
kwds

pie

y
ax
subplots
layout
grid
title
legend
rot
xlim
ylim
fontsize
figsize
colormap
table
kwds

scatter

x
y
ax
grid
title
~legend~
logx
logy
loglog
xticks
yticks
rot
xlim
ylim
fontsize
secondary_y
mark_right
figsize
style
colormap
colorbar
position
table
yerr
xerr
kwds

hexbin

x
y
ax
grid
title
~legend~
logx
logy
loglog
xticks
yticks
rot
xlim
ylim
fontsize
secondary_y
mark_right
figsize
style
colormap
colorbar
position
table
yerr
xerr
gridsize
C
reduce_C_function
kwds

Unfortunately, the __call__ signature is included in the docstring of the .plot method.
In [9]: df.plot?
Type:        FramePlotMethods
String form: <pandas.tools.plotting.FramePlotMethods object at 0x1087b9860>
File:        /Users/tom/miniconda3/envs/py3/lib/python3.4/site-packages/pandas/pandas/tools/plotting.py
Definition:  df.plot(self, x=None, y=None, kind='line', ax=None, subplots=False, sharex=True, sharey=False, layout=None, figsize=None, use_index=True, title=None, grid=None, legend=True, style=None, logx=False, logy=False, loglog=False, xticks=None, yticks=None, xlim=None, ylim=None, rot=None, fontsize=None, colormap=None, table=False, yerr=None, xerr=None, secondary_y=False, sort_columns=False, **kwds)
Docstring:
DataFrameFrame plotting accessor

Examples
--------
>>> df.plot.line()
>>> df.plot.scatter('x', 'y')
>>> df.plot.hexbin()

The plotting methods can also be accessed by directly calling the
accessor with the ``kind`` argument:
``df.plot(kind='line')`` is equivalent to ``df.plot.line()``
Call def:    df.plot(x=None, y=None, kind='line', ax=None, subplots=False, sharex=True, sharey=False, layout=None, figsize=None, use_index=True, title=None, grid=None, legend=True, style=None, logx=False, logy=False, loglog=False, xticks=None, yticks=None, xlim=None, ylim=None, rot=None, fontsize=None, colormap=None, table=False, yerr=None, xerr=None, secondary_y=False, sort_columns=False, **kwds)

I'm not sure this is avoidable.

I did some work on this a while back. I'll see how it looks and maybe open up a pull against your branch.

@shoyer nice start....for 0.17.0!

@jreback @TomAugspurger Yes, let's get this in for 0.17. Even it's just our best guess for the full doc strings, we can fill in the long tail of options later.

OK, I have been updating this. To keep the complexity under control, so far all I've done is add the kind specific arguments to each plotting method. So at least it's clear which arguments are relevant for each kind, and these can be used positionally.
I would rather get this out in its somewhat incomplete form and add the long tail of plotting methods (as documented by @TomAugspurger above) later. That should be pretty straightforward at this point, if someone can figure out a way to consolidate the docstring and function signature generation without going insane.
Here's what the new API docs look like:

The main remaining blocker is that we get alias of FramePlotMethods as the API documentation for DataFrame.plot, without any docstring or generated page. @jorisvandenbossche do you know how to fix this? I can dig in if necessary.

@jorisvandenbossche any ideas for fixing the doc generation? Even just a pointer to relevant docs would Sphinx would be helpful... my blind experiments did not have much luck.

@shoyer I took a look at this, and I am afraid I don't directly see a way to solve this.
The problem is that DataFrame/Series.plot is no longer a method, but a class.
So also when you do df.plot?, you get the class docstring.
Just this fact is also already a bit annoying, as this will not show the signature as does df.plot? now.
To get the docstring page build for Series.plot, you have to use another template (see how it is done for cat/str/dt):
.. autosummary::
       :toctree: generated/
       :template: autosummary/accessor.rst

       Series.plot

But that does not solve the issue of above of the wrong docstring/wrong signature. The wrong docstring can of course easily be fixed by changing the docstring of the PlotMethods class. But the help of df.plot? will still look like that one of a class

Well, in the worst case scenario we could generate the rst on the API docs for .plot manually? That wouldn't be too bad
On Thu, Jun 25, 2015 at 5:29 AM, Joris Van den Bossche
<EMAIL_ADDRESS>wrote:

@shoyer I took a look at this, and I am afraid I don't directly see a way to solve this.
The problem is that DataFrame/Series.plot is no longer a method, but a class.
So also when you do df.plot?, you get the class docstring.
Just this fact is also already a bit annoying, as this will not show the signature as does df.plot? now.
To get the docstring page build for Series.plot, you have to use another template (see how it is done for cat/str/dt):
.. autosummary::
       :toctree: generated/
       :template: autosummary/accessor.rst
       Series.plot

But that does not solve the issue of above of the wrong docstring/wrong signature. The wrong docstring can of course easily be fixed by changing the docstring of the PlotMethods class. But the help of df.plot? will still look like that one of a class
Reply to this email directly or view it on GitHub:
https://github.com/pydata/pandas/pull/9321#issuecomment-115234427

The generation of the API pages won't be the problem I think. With my suggestion of above (using the correct template), I think this should work.
Where I am more worried about is the df.plot? help in the console. We can make the docstring give the correct information, but it will still say it is a class and show no signature, we seems a potential source of confusion to me ..

@shoyer status?

I think this just needs the fix to generate the right doc pages (and a rebate, of course). I'll need to experiment to figure that out, unless @jorisvandenbossche already know how...
I'm on vacation this week but will try to squeeze in some time :)

status?

Ugh, it is a major struggle to get autodoc to handle Series.plot properly.
At build time (for the docs) Series.plot is a class object, so autodoc insists on using the class template for the entry on api.html. I can use a custom template to make the subpage look reasonable, but it's still not looking right in the API docs table of contents...

This still needs a major rebase, but I have finally succeeded in generating the right API docs!

Here's what the relevant sections look like on the main API page:

Here's the documentation page for DataFrame.plot:

And here's an example of the doc pages for one of the submethods. Note that pandas.DataFrame.plot here is a link to the main DataFrame.plot page:

Nice hacks to get the autosummary working! :-) Glad you found a solution

lgtm.
@sinhrks @TomAugspurger @jorisvandenbossche
needs a rebase / squash
existing docs in visualization.rst to change to the new accessing methods? in a follow-up PR (pls make an issue)

For the existing visualization.rst docs, I think it is OK to leave that for now
(also for later, question is one of the two we want to advertise the most)

Did the big rebase/squash -- let's see if things pass on Travis. Also added two followup issues for documentation. Assuming tests pass, this is ready for a final review.

lgtm.
deprecate .hist? (and point to .plot.hist)
@jorisvandenbossche @sinhrks @TomAugspurger

I don't think a deprecation of hist is directly needed at the moment (they also don't have fully compatible signatures). But could certainly add a pointer in the hist docstring.

:clap:

Added another followup issue for deprecating DataFrame.hist: https://github.com/pydata/pandas/issues/11053
I think this would be a good idea if we can sort out the API differences.

ok, unless anyone has anything else, @shoyer merge on green.

I don't the image in the whatsnew got commited: _static/whatsnew_plot_submethods.png
can you push it up?

Oops. Just pushed to master: https://github.com/pydata/pandas/commit/eca9db97c2c7746ed8a599b81092be9735907a66

awesome thanks!


Fix race conditions in 03-narrow casper tests.

Keypress is an async event so we must wait for the resource to appear
before we can test for it.
Fixes: #1455
@timabbott

Automated message from Dropbox CLA bot
@umairwaheed, it looks like you've already signed the Dropbox CLA. Thanks!

Still happening

I think both #zhome and #zfilt are always visible.

For narrowing, we toggle the focused_table class on zfilt/zhome to control which table is currently visible; so maybe we just need to do things like waitForSelector for e.g. #zfilt.focused_table?

That should work.

@timabbott, I have rearranged the test code. I hope that now the problem will be fixed.

Great, thanks for working on this @umairwaheed!  I've merged this; hopefully we'll get to enjoy a reliably passing Casper build for a while :)


PLASMIDS

Hi RaGOO team, in your document you made mention that Any unlocalized contigs are concatenated and placed in "Chr0_RaGOO". I am working with M. ulcerans and the complete circular sequence and its plasmids are stored in different files.  So if I use lets say the chromosomal reference sequence for the guided assembly, does it mean that contigs which do not match the reference (and possibly are plasmids) will placed in the Chro_RaGOO?  and therefore if I am looking for the plasmids thats the file to look for ?

Hi there,
Firstly, I recommend using RagTag instead of RaGOO if at all possible. It is superior in many ways. The following applies to both RaGOO and RagTag.
I think what you said sounds good. Of course, you always run the risk of the plasmid sequence being placed into chromosomes due to repetitive mappings. If you can identify the plasmid contigs beforehand, you can ask for them to be left unplaced by using the -j parameter.
Thanks

Hi @malonge,
I blasted some of the contigs and they come out as 100% identity with a plasmid. So I guess from your explanation I will have to use the -j parameter. I will update you on the outcome once I implement the pipeline. Thanks

I guess it depends. If a contig maps with 100% identity with equal coverage to two different positions, it is completely ambiguous so it probably shouldn't be placed anywhere. But if it covers one region better than another then that might be sufficient to discriminate between plasmid and chromosome sequence, so you could let ragoo/ragtag take a shot a placing it.

One last thing,
Does Ragtag do stitching (remove Ns). I did not find that in the document.   ( I will also like to generate a circularized result)

Hi there,
I am not sure what you are referring to. Are you asking about filling gaps between adjacent contigs? I can say that both RaGOO and RagTag strictly order and orient input sequences - they don't add (except for "N"s) or subtract or change any sequence.

Yes. I mean filling of gaps

Ok yeah unfortunately these tools don't perform gap filling. Let me know if you have any other questions.


try mounting without fusermount(1) first

This requires root privileges or the CAP_SYS_ADMIN capability, but allows supporting -o suid.
I also addressed a bunch of readability-related nits while touching the code.

hi, @stapelberg  can you help me,with this patch when i mount fuse with root, hit the err
"mount: invalid argument"
seems mount source not work, can you give some advise?
but normal user is ok!

Can you open an issue for that please? In the issue, can you supply an strace log (use strace -fvy -s2048 -o /tmp/strace.log in front of your command), and can you let us know how to reproduce the problem?

@stapelberg

Can you open an issue for that please? In the issue, can you supply an strace log (use strace -fvy -s2048 -o /tmp/strace.log in front of your command), and can you let us know how to reproduce the problem?

https://github.com/jacobsa/fuse/issues/76
thank you very much!

@stapelberg hi， can you tell something about why root not use fusermount？

This is a requirement for supporting the -o suid option. Also, it helps in early-boot environments where fusermount might not be available.

This is a requirement for supporting the -o suid option. Also, it helps in early-boot environments where fusermount might not be available.

got it, thank you


Feature request: Subscription or tokens for proxy clients via real client

Hello,
monitoring hundreds off network devices could be supported by more out of box approach.
Would be interesting if proxy clients could "virtually" subscribe to checks as well.
Now, there seems to be only option do define "source" attribute in the check or resulting event.
This is bit messy when handling hundreds of network devices.
Option could be to subscribe real client for the check - which is working but then when using tokens, those take data from real client and not from proxy client. If tokens could take data from proxy clients, the "command" and "source" attribute could benefit of it. At the end etc. Sensu server could be used for executing the checks itself, however the check would appear as from proxy client.
(for now, "source" field does not even support tokens)
Does this make sense to think about?
Thank you very much

We acknowledge the difficulties in monitoring a large number of network devices with Sensu, beyond leveraging configuration management to create the appropriate round-robin checks to monitor known devices, creating proxy clients from check executions. We are working on this problem, however, treating Sensu proxy clients as anything more than a external source identifier is not possible with how Sensu works and operates. We hope to create a document/issue soon with our proposed solution(s) to this problem.


RuntimeError: copy from Variable to torch.FloatTensor isn't implemented

when running the predict function on video feed I get this error.
pytorch version 0.4
y = net(x)  # forward pass
File "/home/nvidia/Desktop/pytorch/torch/nn/modules/module.py", line 357, in __call__
    result = self.forward(*input, **kwargs)
  File "/home/nvidia/Desktop/ssd.pytorch/ssd.py", line 102, in forward
    self.priors.type(type(x.data))                  # default boxes
  File "/home/nvidia/Desktop/ssd.pytorch/layers/functions/detection.py", line 57, in forward
    boxes[ids[:count]]), 1)
RuntimeError: copy from Variable to torch.FloatTensor isn't implemented

Hi, can you make sure you have the latest version?  If so, I will look into it, but I have not faced this problem.

I got same error.
I didn't get the error with pytorch version 0.3, so version 0.4 would be the cause of the error

+1 - Got the same issue on pytorch 0.4

when my environment with torch 0.4.0, l have the same error!
BUT !  After l uninstall   the PyTorch 0.4.0 and installed the PyTorch 0.3.0 the error no longer appear！

I met the error with pytorch version 0.3.1 and I still don't know why.


Support teams being optional

This pull request is here to ask others to test this branch, and find any bugs or inconsistencies before this I merge this into master. If you know what you are doing please feel free to do everything you can to try and break this, and report back the issues.

What's the purpose of "optional teams" what does that mean?

@Pablete1234 What if the match is FFA? That's what Elly said to me when I asked about it.

i se FFA as 1 team with friendly fire on

@Pablete1234 That wouldn't support per-player scoring

@twizmwazin That makes sense

I found an issue, if you try to /end a regular match (Tested on boom TE) it gives out an error. Log
Also, if you try to cycle to an FFA match, it fails to cycle, the xml is valid. I don't know if it is supposed to be able to cycle already, but I posted it just in case.

@Eudaldca Thanks, fixed


Two Lockfiles and rm command err during build

Here is my fix on this issue Two Lockfiles
and rm command err
Fix the build issue on non-unix systems causing an errorrm: command not found so it is necessary add a script function to scripts/ directory.
Two Lockfiles in the same directory sometimes can cause conflicts between themselves.
So, I consider using the yarn.lock file which i think is better:+1:

Hello, @haneenmahd. I think you should remove yarn.lock instead of package-lock.json as the latter was last used more recently.

Considering that

Closing this as I just removed the yarn.lock file https://github.com/boringdesigners/boring-avatars/commit/e03f1b9ba0ebac725936d34d8b378a1402f8d385.
Thank for the heads up @haneenmahd and @abhijit-hota


Fix RunContainer.andCardinality(RunContainer)

Fixes #153

I have a couple of pull requests (#149) that depend on this for correctness. Should I wait for this to merge and then rebase before submitting?

Coverage increased (+0.009%) to 89.835% when pulling 1e6c66f5b0e3e71cddca814bfdcfb6ad9d4aa295 on andCardinality into bc59514da22ebe3f2709696fa84c274bff98c60c on master.

You can merge my branch into yours for the time being.

LGTM.

I'll issue an immediate release based on this, despite the fact that we have outstanding PRs.


Normalization failed / Invalid start of grapheme sequence Error While training the tesseract model

Normalization failed for string 'ଜୀବନକୁ ନିବିଡ଼ ଭାବେ ଏକନ୍ୱିତ କରିଛନ୍ତି'
Invalid start of grapheme sequence:D=0xb71
Normalization failed for string 'ପରମ୍ପରାକୁ ଅବଲମ୍ୱନ କରିଛନ୍ତି, ସେତିକି ମଧ୍ୟ'
Invalid start of grapheme sequence:M=0xb48
Normalization failed for string 'ଦ୍ୱୈତ ରୂପରେ ଦେଖିଥିଲେ, ଏଠାରେ ପୁରୁଷ'
Invalid start of grapheme sequence:M=0xb47
Normalization failed for string 'ତାଙ୍କ ହୃଦୟ ବିଭୋର ହୋଇଛି ସମ୍ୱେଦନଶୀଳତାରେ;'
Invalid start of grapheme sequence:D=0xb71
I'm getting this error while training the tesseract ocr model for Oriya language  , please help me to resolve this issue .
I'm attaching the ground truth files .
Training on tesseract 4.1.1 :
tesseract 4.1.1
leptonica-1.82.0
ocr_training.zip

Try to shorten those strings in your training data until the error messages disappear, then check what was wrong with them.


P2PK To cashAddress?

is it possible to convert P2PK (Normal Address) to cashAddress?

Forgot to post this here, but there was finally a bridge published for bitcoin-php & BCH  https://github.com/btccom/bitwasp-bitcoin-bch-addon

Yeah, I've seen that. I'm your big follower :)


reinstall: add support for the --git option

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.

[x] Have you successfully run brew typecheck with your changes locally?
Error: Invalid usage: not (yet) working on Apple Silicon or Rosetta 2!
[x] Have you successfully run brew tests with your changes locally?

This Pull request adds the -g or --git option to the brew reinstall command.
I've had several instances where I do this:
brew install flashrom
brew install flashrom --build-from-source
brew test flashrom
brew reinstall flashrom --build-from-source --git --debug
# ugh.
brew uninstall flashrom
brew install flashrom --build-from-source --git --debug

I've found it very useful to when doing a reinstall to be able to pass the git option to avoid the uninstall:
brew install flashrom
brew install flashrom --build-from-source
brew test flashrom
brew reinstall flashrom --build-from-source --git --debug
brew test flashrom

I'm going to fire up a clean install to test this on intel.

brew typecheck passes on intel

This will need rebased to pass CI.

This will need rebased to pass CI.

Rebased.  :)
Curious about removing -g but happy to do whatever is preferable.

Thanks @areese !  You can feel good knowing that you've made the world a tiny bit better for homebrew users around the world! 👍 🎉


Support using downstream TCP destination port for SNI dynamic forward proxy upstream connection

Commit Message: Support using downstream TCP destination port for SNI dynamic forward proxy upstream connection
Additional Description: This change removes the need for users to set the port_value in the SNI forward proxy configuration and instead rely on the downstream TCP connection using the correct destination port.
Risk Level: Medium
Testing: With address based on feedback
Docs Changes: Remove port_value from example configuration since I think this should no longer be needed in most cases
Release Notes: Support using downstream TCP destination port for SNI dynamic forward proxy upstream connection
Platform Specific Features: N/A
Fixes https://github.com/envoyproxy/envoy/issues/16048
This works by appending the downstream TCP connection destination port to the SNI host to use in the DNS cache. This is not necessary for the HTTP dynamic forward proxy since this happens by default if the port isn't 80 (https://www.w3.org/Protocols/rfc2616/rfc2616-sec14.html#sec14.23).

please re-open if you want to continue on this workstream.


Fails when using gradle worker api

Using the Gradle worker api, each worker gets it's own classpath, so trying to run Hephaestus fails with the following error message.
Cannot set the value of task ':some:module:compileReleaseKotlinCheckIncrementalCompilationHephaestus' property 'incrementalSignal' of type com.squareup.hephaestus.plugin.IncrementalSignal using a provider of type com.squareup.hephaestus.plugin.IncrementalSignal.

This is in a project with kapt using the worker API, and Room.
I've attempted to implement this in my branch here, but need to take off for the day.

I wonder if you could provide a sample project? The worker API is enabled for this project, but I don't see the issue. Maybe it's related to Room?

You're right.  While creating a minimally reproducible project, it looks like using gradle kts scripts is what changes things.
Attached is a project that reproduces the issue.  There isn't even any consumption of code, it simply appears that using kts, kapt, and Hephaestus causes the problem.
Hephalumps.zip
To see the issue, run ./gradlew assemble in this project.  To see the fix, alternate the comments in settings.gradle so the groovy based module2 is included instead of the kts based module1 is included.

I can reproduce the issue with your project. But I wasn't able to reproduce it in here. This definitely seems to be a Gradle issue  and is maybe caused by the way plugins are applied.
You do this:
plugins {
    id("com.squareup.hephaestus").version("1.0.3")
}

apply plugin: 'com.android.application'
apply plugin: 'kotlin-android'
apply plugin: 'kotlin-android-extensions'
apply plugin: 'kotlin-kapt'

I changed it to the following in all three modules and cannot reproduce the bug anymore:
apply plugin: 'com.android.application'
apply plugin: 'kotlin-android'
apply plugin: 'kotlin-android-extensions'
apply plugin: 'kotlin-kapt'
apply plugin: 'com.squareup.hephaestus'

I also tested the plugins syntax and the issue is gone:
plugins {
    id("kotlin")
    id("kotlin-kapt")
    id("com.squareup.hephaestus")
}

Only mixing the two seems to cause issues.

Hmm, that doesn't seem right.  The code you're sharing looks like groovy (notice the single quotes).
In the example, the groovy modules aren't a problem, it's the kts build file that introduces the problem. (comment that out in settings.gradle and the problem goes away.)
Hephalumps 2.zip
Here's a new repro project with the updates you propose (changing all modules to use new plugin syntax) and the problem persists as long as you include the kts based module in the build.

Sharing zips isn't fun. This for some reason fixes the issue: https://github.com/vRallev/issue-33/commit/0a0c051b558f9a943a3ab81f96acf26e3c5f8960


After canceling order it is not saving in History

One an order is matched, the resulting trade should go to user's history. Right now, when an order is fully matched the order goes to state "done". But the problem arises when the order is partially matched and the user cancels the order. This results in no history and the user not knowing what happened.

Hi @mkuvshynnikov
I have checked the issue. And found that the functionality is working accordingly i.e if the order is fully fulfilled then the order state is changed to done and when the order is partially fulfilled then it shows open and if we cancel the partial order then the state is changed to cancel. As currently we have three states i.e wait, done and cancel.


Remove multikernel stubs

Since multikernel support is nonexistent as far as I know, we might remove any remains of older multikernel plans.

bors r=stlankes


Add execute flushing

Fix pipeline flush mechanisms by removing inflight instructions inside functional units, recovering the rename map, and adding trace rewind functionality
Add FlushCriteria class for requesting flushes to and from the FlushManager
Update FlushManager to handle and arbitrate between multiple flush sources
Add flush request port from ExecutePipe to FlushManager intended to be used for mispredicted branches
Delay completing instructions in ExecutePipe by a cycle to fix execute flush race conditions
Add branch identification methods for Inst
Add test to send random flushes from the branch unit

@knute-sifive few questions for you

For the flush testing mode we spoke about in the call yesterday. Were you thinking to use a parameter in the ExecutePipe?

I've run into a race condition where a branch mispredicts, completes and retires in the same cycle, before the flush is seen. Meaning any younger instruction that is retired alongside should have actually been flushed
See this log, where PID: 1975 is correctly refetched following the flush but also retired just before it.
{0000004137 00004137 top.cpu.core0.rob info} retireInstructions_: retiring uid: 2551    RETIRED 113fe pid: 1974 'bne	x5,x7 +0xb2'
{0000004137 00004137 top.cpu.core0.rob info} retireInstructions_: retiring uid: 2552    RETIRED 11402 pid: 1975 'bne	x12,x13 +0x3c'
{0000004138 00004138 top.cpu.core0.rename info} getAckFromROB_: Retired instruction: uid: 2551    RETIRED 113fe pid: 1974 'bne	x5,x7 +0xb2'
{0000004138 00004138 top.cpu.core0.rename info} getAckFromROB_: Retired instruction: uid: 2552    RETIRED 11402 pid: 1975 'bne	x12,x13 +0x3c'
{0000004138 00004138 top.cpu.core0.rob info} handleFlush_: flushing uid: 2574 DISPATCHED 1144a pid: 1997 'slli	x14,x12, SHAMT=0x20'

I think we need to fix it so that we don't retire instructions that completed that cycle, and there's a few ways of doing this:
a. Add a sharedData object inside Inst that delays the completed status.
b. Add a completed event inside Inst and have retirement check if the event is scheduled before retiring (event should only be scheduled on the cycle it's completing)
c. Delay setting status to complete inside ExecutePipe and LSU
Any input?

As the race condition was between the flush coming from the execute and the instruction changing to completed status. I decided to separate changing the status to completed and the execution of the instruction.

Thanks for doing this!

For the flush testing mode we spoke about in the call yesterday. Were you thinking to use a parameter in the ExecutePipe?

This is related to the "random" flushing idea (for testing), right?  Here are my thoughts:

Since the flushes are instigated mostly (if not always) from the ROB, add the parameter there.  It's simple integer-based parameter, enable_random_flushes that takes a random key.  If non-zero, it's enabled with that key.
Add a few test with some random keys (maybe using the $RAMDOM variable from bash)

Glad you got the race condition rectified.

I hope I've address your comments above.
I played around with the ROB flushing but I found that doing it in the branch unit found more bugs. This should be a temporary thing to plug the gap until branch prediction is implemented.
For the ROB flushing, I could still put it in. What I did based on your comments was to use a hidden parameter that acted like a UID to trigger a flush from. Obviously this would only cause 1 flush per run, so it didn't have a lot of coverage compared to the branch unit flushing.

Just noticed I forgot to change the FlushManager::FlushCriteria.flush method as requested

Will merge this tomorrow morning CST.

Will merge this tomorrow morning CST.

None of my comments need to be addressed before merging. They can be resolved in another PR.

Daniel, would you like more time to address Kathlene's comments or would you prefer to do that in another PR?

Another PR please


corrected missing make:resource command

On the current version that command is missing

Thanks for this correction, Highly appreciate your contribution.


Update TTree.py

Changed the for-loop for the JaggedArray datatype to a version flattening the array. The for loops runtime scaled quadratically and was not possible to use for arrays with more than 1e5 entries.

That's absolutely a good fix—thank you! I just tested it, and all looks good. I'll release this as a new patch release after fixing some NumPy 1.20 warnings.


Upon start of the debug getting CRRDG1121E tag type 45

Getting error
Illegal base64 character 5f
EQARD1007I The Remote Debug Service daemon started on port 8001.
CRRDG1121E Debug Secure Daemon failed to start. Reason: toDerInputStream rejects tag type 45
Opened case with IBM to verify its not a problem with the system. Re-created all the steps from the walkthrough.
IBM support said type 45 means expired certificates but they were just created .
We are on OS 7.4 and PTF SI83683 and SI81031 are applied.
@

Is this still being worked on?

I still have the problem.

@worksofliam any thoughts on what is failing here

@mkwan01 any ideas here?

This message normally means that the certificate is not valid. The certificate creation is not done in the debugger itself. It is handled by code for i.
@worksofliam Do you know which file in the screen capture is set as the DEBUG_SERVICE_KEYSTORE_FILE environment variable value?

On my system, I can see that if the client certificate (the .crt file) is used on the host, then it would generate the "toDerInputStream rejects tag type 45" message. Please make sure that the .pfx file is used as the host keystore.

Code for IBM i 2.9.0 contains a number of fixes for certificate generation. Please reopen the issue if you can still reproduce the problem in 2.9.0.


Paris P2P #2 - 2019-10-02

Info

Official link: https://p2p.paris/en/event/monthly-2/
Meetup link: https://www.meetup.com/fr-FR/Paris-P2P/events/263191369/

Talks

#24 Workshop: Introduction to Crypto Wallet -- @LesHackeuses
#21 Talk: Web 3.0 with IPFS -- @aeddi
#31 Poster Session: DPOS -- @zooma2k18

Intro slides: https://docs.google.com/presentation/d/1CwY5JAwiX6YhHC_EBGz4pOvhELWqeHBhn5BnvYeRmBw/edit?usp=sharing


Default distribution directory: build instead of dist?

Since the examples use React, and the default distribution directory with React is build and not dist, it might be worth changing the default in awsmobile to build. As it currently is, one first gets an error with
awsmobile publish
and after the error, newcomers have to run
awsmobile configure project
to reconfigure the project. I expect most newcomers to fall into this trap.
This is only a minor pain point and it is easy to correct, but fixing it in awsmobile would improve the first impression and the overall customer experience.

@baharev when running awsmobile init the question/answer can be overridden by typing 'build' when the question is asked.

@mlabieniec Yes, I know that you can change it; I also wrote that in my original post. I am complaining about the default being dist and not build. All other defaults are perfect for React, and the examples also use React, so I see this particular default as an annoying stumbling block.


Upstream osmosis iavl changes

This PR (please edit):

[ ]

Hey @julienrbrt just wondering what is the plan with this?
I think I've upstreamed all changes so far. The last missing PR is: https://github.com/cosmos/iavl/pull/506

Hey @julienrbrt just wondering what is the plan with this?
I think I've upstreamed all changes so far. The last missing PR is: #506

Not sure, I have just been asked to open that PR. @marbar3778 will know more :)


660 Add support for multiple statuses to be selected for filtering subject requests

Purpose
Changes
660 - Add support for multiple statuses to be selected for filtering subject requests (https://github.com/ethyca/fidesops/issues/660)
Checklist

[x] Update CHANGELOG.md file

[x] Merge in main so the most recent CHANGELOG.md file is being appended to
[x] Add description within the Unreleased section in an appropriate category. Add a new category from the list at the top of the file if the needed one isn't already there.
[x] Add a link to this PR at the end of the description with the PR number as the text. example: #1

[x] Applicable documentation updated (guides, quickstart, postman collections, tutorial, fidesdemo, database diagram.
If docs updated (select one):

[ ] documentation complete, or draft/outline provided (tag docs-team to complete/review on this branch)
[ ] documentation issue created (tag docs-team to complete issue separately)

[x] Good unit test/integration test coverage
[ ] This PR contains a DB migration. If checked, the reviewer should confirm with the author that the down_revision correctly references the previous migration before merging
[ ] The Run Unsafe PR Checks label has been applied, and checks have passed, if this PR touches any external services

Ticket
Fixes #
#660

Only the migration-check test is failing which is a known error


Extra title on panels shown before window loads fully

Connect to a remote
Before we finish connecting to the remote, I see this title on panels which lets me collapse them.

adding @sandy081 in case this happened with viewlets before and the fix is known.

It did not happen in viewlets but I suspect in remote it takes long time for extensions be ready. Not sure if we have cache for single view or not for panels which we have for viewlets.

After testing, it looks like this repro's in search viewlet even in much older stable builds but it only happens when there is no cache (a new workspace). It is exaggerated by the remote connection startup time as mentioned by @sandy081

Our behavior is code here: https://github.com/microsoft/vscode/blob/ca05440d7051a764843f2928a118eb6876251863/src/vs/workbench/browser/parts/views/viewPaneContainer.ts#L977-L989
After checking if we have a single view and we should indeed merge if there is no other view, we fallback to not merging if the cache is not yet populated. That means for all single view panels, we will see this flashing on the first time opening a workspace where extensions are not yet ready. Either way we will be wrong sometimes, but I am changing this to the opposite as it is more common I believe.


EKS nodes have delete option enabled

On 2.5-head commit id: 118f999

Deploy an EKS cluster
In the cluster Dashboard --> Nodes page, user is able to select and delete the nodes. (logged in as Admin user/cluster owner)

Expected:
User should not be able to delete the nodes.

Passing over to backend to see if there's a reason the delete option is being exposed for EKS.

This isn't applicable anymore

I'm sorry, I was looking at the wrong UI when I moved this to test. I do indeed see the delete button for EKS clusters under Explore Cluster -> Cluster -> Nodes. This isn't specific to EKS clusters, the button exists for GKE and AKS hosted clusters too.
There's no specific reason in the backend that this needs to be enabled for hosted providers. I think it's just that the UI treats the hosted provider cluster in the same way as it treats other clusters. The button is just doing kubectl delete node <node> on the downstream cluster, so if that's not something we want to expose in the UI then I think the dashboard just needs to add a condition to hide that button for EKS, GKE, and AKS clusters. @gaktive can this be reassigned to UI?

@codyrancher does seem straightforward enough and doable for 2.6.1?


test: fix flaky test case test_setting_concurrent_volume_backup_restore_limit by longer timeout

test: fix flaky test case test_setting_concurrent_volume_backup_restore_limit by longer timeout
For https://github.com/longhorn/longhorn/issues/6260
Signed-off-by: Yang Chiu<EMAIL_ADDRESS>

@yangchiu Could you please help backporting to 1.5.x and 1.4.x if needed?

@mergify backport v1.5.x v1.4.x


Returns success, but file not created. Error code 2.

i get error code 2, file not found.
My code is alright. it did work before, now it doesn't. code never changed on both ends.
winver perhaps?

The issue is not related to kdmapper, your driver returned 0xC0000035


pico flash won't read my nand - Trinity

Hi there,
I installed 2 different pico on my trinity and each time I get the message "Console not found"
Can you please let me know if it is a known issue?

I checked also my solder pins 2 times
Regards,
Jeff

Is the console working? Does it have standby power?

Indeed I needed to change the pico thank you for closing the topic for me


Data Loading Indicator / Site Review (Evan)

AMLNG-783
Review entire site and provide recommendations for implementing data loading indicator to improve the user's experience.
Acceptance Criteria:

discuss w/ PMO team for user input
discuss w/ testing to identify potential known areas
review entire site
create shared document listing recommending areas

Review Submitted


Excalidraw as Figma plugin

I often use Figma for designing but also as a place for scratchpad style thinking and drawing although Figma does lack capability to do free form drawings and lots of excalidraw features.
I would love to build a plugin adds a block or something inside Figma that embeds Excalidraw inside it. To do free form drawing.
Would be cool if one can leverage Figma collaborative style editing to do editing on the Excalidraw embedded block too although that's probably impossible.

Hi. No clue how Figma plugins work, so can't help. A few notes:

There's an open PR to allow embedding of Excalidraw, so that might come in handy: https://github.com/excalidraw/excalidraw/pull/958
Excalidraw currently doesn't allow free form drawings, either (if you meant non-bezier, free draw tool).

Do we really have a use case here.. I think it's either Figma or Excalidraw.. what's the point of doing Excalidraw inside Figma?

I would love this feature too - in fact I came looking for it.
The use case is being able to draw with Excalidraw within Figma without having to leave Figma. The point is to have your quick mock ups and more finished vector art all in one place.
I love using Excalidraw and while it is relatively easy to export, having it right in Figma means you can sketch something out to see some ideas and then move on to more detailed mock ups.
Seems like a very common workflow that would be used by anyone who uses both Excalidraw and Figma.

+1

Hi, I made a Figjam widget for Excalidraw as I found myself copy-pasting screenshots regularly, you can find it here:  https://www.figma.com/community/widget/1047391719101881118
The nice thing about using Figjam instead of Figma is that there's a bidirectional relationship, so you can reopen your initial drawing without switching browser tabs just by clicking on it. In contrast, if you embedded an image in Figma via a Figma plugin, you wouldn't be able to edit your image as an Excalidraw file once it's been inserted.

Thats an amazing plugin @hydrosquall
Although I tried to install it but it's not showing in Figma Jam:

Ah I think it might be classified as a “widget” rather than a plugin  - not
sure if there may be plans to add a widget tab separate from plugins, but
in the meantime here’s the docs page.
https://help.figma.com/hc/en-us/articles/4410047809431-Use-widgets-in-FigJam-files
Let me know if you run into further issues, I can post a public Figjam file
for you to copy paste from if it might help.
On Thu, Dec 2, 2021 at 6:56 PM Nikita Voloboev @.***>
wrote:

Thats an amazing plugin @hydrosquall https://github.com/hydrosquall
Although I tried to install it but it's not showing in Figma Jam:
[image: image]
https://user-images.githubusercontent.com/6391776/144521474-051d0a4c-2a63-4003-9958-612a7bb699d6.png
[image: image]
https://user-images.githubusercontent.com/6391776/144521515-1c80cdc9-d9d6-4c30-a2b1-dd47cc73c3b5.png
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/excalidraw/excalidraw/issues/1035#issuecomment-985096639,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACE2MM2H5KX3ODHVX2YX753UPABSFANCNFSM4LQ7O2UQ
.

Do we really have a use case here.. I think it's either Figma or Excalidraw.. what's the point of doing Excalidraw inside Figma?

My usecase is to do click through prototypes of excalidraw style wireframes.  AFAICT Excalidraw does not allow prototyping.

I would love the SVG import of Excalidraw+ account sketches using the plugin. Excalidraw+ is perfect for gathering and enhancing ideas in the Lo-fi stage of the design process. Figma is perfect for doing high fidelity design based on those sketches.


Add 15 languages from weblate

While browsing weblate, several translations catched my eye that where not included in the nightlies
Went through all of the 46 available ones and found 15 of them to not be announced in .../supported-languages/.
Which is hereby fixed.

Hey, excellent, thanks a lot!


[BUG] The backend for LndRestWallet isn't working properly: 'Unable to connect to http://lnd:8080. All connection attempts failed'

NOTE: I am flagging as BUG for now but since I don't know what the issue is and whether it comes from lnbits or lnd itsefl, this may change.
Describe the bug
Failure to connect to LndRestWallet without much useful information.
To Reproduce
I have a running LND in K8S.
To exclude any LND related issue with the Pod or Service, I tested with a port-forward and after unlocking the wallet, I can send local curl requests to the forwarded Service.
That confirms that:

lnd is working properly
lnd is reachable
auth (macaroon) is working fine

lnbits however refuses to connect and provide very little useful information:
Logs
2023-11-15 13:37:34.07 | INFO | lnbits.app:log_server_info:412 | Starting LNbits
2023-11-15 13:37:34.07 | INFO | lnbits.app:log_server_info:413 | Version: 0.11.1
2023-11-15 13:37:34.07 | INFO | lnbits.app:log_server_info:414 | Baseurl: http://<IP_ADDRESS>:5000/
2023-11-15 13:37:34.07 | INFO | lnbits.app:log_server_info:415 | Host: <IP_ADDRESS>
2023-11-15 13:37:34.07 | INFO | lnbits.app:log_server_info:416 | Port: 5000
2023-11-15 13:37:34.07 | INFO | lnbits.app:log_server_info:417 | Debug: True
2023-11-15 13:37:34.07 | INFO | lnbits.app:log_server_info:418 | Site title: LNbits
2023-11-15 13:37:34.07 | INFO | lnbits.app:log_server_info:419 | Funding source: LndRestWallet
2023-11-15 13:37:34.07 | INFO | lnbits.app:log_server_info:420 | Data folder: ./data
2023-11-15 13:37:34.07 | INFO | lnbits.app:log_server_info:421 | Database: SQLite
2023-11-15 13:37:34.07 | INFO | lnbits.app:log_server_info:422 | Service fee: 0.0
2023-11-15 13:37:34.08 | ERROR | lnbits.app:check_funding_source:142 | The backend for LndRestWallet isn't working properly: 'Unable to connect to http://lnd:8080. All connection attempts failed'
2023-11-15 13:37:34.08 | WARNING | lnbits.app:check_funding_source:156 | Retrying connection to backend in 5 seconds...
2023-11-15 13:37:39.10 | ERROR | lnbits.app:check_funding_source:142 | The backend for LndRestWallet isn't working properly: 'Unable to connect to http://lnd:8080. All connection attempts failed'
2023-11-15 13:37:39.10 | WARNING | lnbits.app:check_funding_source:156 | Retrying connection to backend in 5 seconds...
2023-11-15 13:37:44.10 | ERROR | lnbits.app:check_funding_source:142 | The backend for LndRestWallet isn't working properly: 'Unable to connect to http://lnd:8080. All connection attempts failed'
2023-11-15 13:37:44.10 | WARNING | lnbits.app:check_funding_source:156 | Retrying connection to backend in 5 seconds...
2023-11-15 13:37:49.11 | ERROR | lnbits.app:check_funding_source:142 | The backend for LndRestWallet isn't working properly: 'Unable to connect to http://lnd:8080. All connection attempts failed'
2023-11-15 13:37:49.11 | WARNING | lnbits.app:check_funding_source:156 | Retrying connection to backend in 5 seconds...
2023-11-15 13:37:54.12 | ERROR | lnbits.app:check_funding_source:142 | The backend for LndRestWallet isn't working properly: 'Unable to connect to http://lnd:8080. All connection attempts failed'
2023-11-15 13:37:54.12 | WARNING | lnbits.app:check_funding_source:156 | Retrying connection to backend in 5 seconds...
2023-11-15 13:37:59.13 | ERROR | lnbits.app:check_funding_source:142 | The backend for LndRestWallet isn't working properly: 'Unable to connect to http://lnd:8080. All connection attempts failed'
2023-11-15 13:37:59.13 | WARNING | lnbits.app:check_funding_source:156 | Retrying connection to backend in 5 seconds...
2023-11-15 13:38:04.13 | ERROR | lnbits.app:check_funding_source:142 | The backend for LndRestWallet isn't working properly: 'Unable to connect to http://lnd:8080. All connection attempts failed'
2023-11-15 13:38:04.13 | WARNING | lnbits.app:set_void_wallet_class:169 | Fallback to VoidWallet, because the backend for LndRestWallet isn't working properly
2023-11-15 13:38:04.13 | INFO | lnbits.app:check_funding_source:162 | ✔ Backend VoidWallet connected and with a balance of 0 msat.

Expected behavior
Connection succeeds or provide a useful hint.
Desktop (please complete the following information):

LNbits version: 0.11.1
Database: sqlite

Additional context
Using K8S. Here is the relevant portion of the config:
  LND_REST_ENDPOINT: http://lnd:8080
  LND_REST_CERT: "/lnd/tls.cert"
  LND_REST_MACAROON: "/lnd/data/chain/bitcoin/mainnet/admin.macaroon"

http://lnd:8080 is valid in k8s:

lnd is the name of the service, this one has been tested with the forward and works fine
:8080 is the port where the service (and the pod) expose the Rest API
cert is present: -rw-r--r-- 1 1024 users 895 Nov 15 10:40 /lnd/tls.cert
macaroon is present: -rw-r----- 1 1024 users 293 Nov 14 23:56 /lnd/data/chain/bitcoin/mainnet/admin.macaroon

hey, can you try again on latest 0.12.4
i think you issue was  LndRest uses https so it should have been https://lnd:8080
can you reopen if it still fails?


Post create: Column 'slug' cannot be null

Laravel Version: 5.5.4
Voyager Version: master (7f7c23a)
PHP Version: 7.1.8
Database Driver & Version: mysql

Description:
Multilingual site is enabled. On post create I'm getting error Integrity constraint violation: 1048 Column 'slug' cannot be null.
In VoyagerBreadController store method, request object slug_i18n property is null for each language, I guess here is the problem. Before form submit, slug_i18n field is not updated?
[
  "_token" => "BdoALUBnaVSAdJbENXKoYC4DTxaoKpKwnlPUCQYa"
  "title_i18n" => "{"en":"My new post","et":"","ru":""}"
  "title" => "My new post"
  "body_i18n" => "{"en":"
Post content
"}"
  "body" => "
Post content
"
  "excerpt_i18n" => "{"en":null,"et":null,"ru":null}"
  "excerpt" => null
  "slug_i18n" => "{"en":null,"et":null,"ru":null}"
  "slug" => "my-new-post"
  "status" => "PUBLISHED"
  "category_id" => "1"
  "meta_description_i18n" => "{"en":null,"et":null,"ru":null}"
  "meta_description" => null
  "meta_keywords_i18n" => "{"en":null,"et":null,"ru":null}"
  "meta_keywords" => null
  "seo_title_i18n" => "{"en":null,"et":null,"ru":null}"
  "seo_title" => null
]

Same for post update, if slug is changed, changes are not saved to database.
Steps To Reproduce:
Install voyager
Enable multilingual site
Add languages
Create new post

Labels: locale, possible bug

Yep, I'm seeing this too.

This seems to have been fixed in #1802

Awesome 👍


Is there any hook example?

I searched, but i can't see.
here is my sample table
@Table({
  indexes: [],
  instanceMethods: {},
  classMethods: {},
  timestamps: true
})
export class Role extends Model<Role> {
  @PrimaryKey
  @Column(DataType.UUID)
  id: string;

  @BelongsToMany(() => Group, () => RoleGroup)
  groups: Group[];

  @CreatedAt
  createAt: Date;

  @UpdatedAt
  updateAt: Date;
}

Role.beforeCreate(function(model, options): Promise<any> {
  return new Promise((resolve, reject) => {
      model.myInstanceMethodsSample(model.myFieldSample);
      resolve();
  });
});

My question is... I don't know, model equals above Role class. because as you can see,
I need to use instance method in this beforeCreate hook.
It will be great that i can see any example in hook.
p.s.
In decorator Table, there is classmethods and instancemethods. but it is useless because I just write function in class. am I right?

I've been done changing instance methods to class methods.
and, call like Role.classMethods(model.getDataValue('field')) this.
but, is there any smooth way?
this way feel wierd because this is hook function but I use class methods calling like CLASS.FUNCTION
I think this or model object or something else that i can access class methods or instance methods.

Hey, thanks for contributing.
The classMethods and instanceMethods property comes from native sequelize. You're right, in sequelize-typescript you should use real instance methods and static methods like:
@Table
class User {

  @Column
  name: string;

  @Column
  reversedName: string;

  prepareReversedName() {

    this.reversedName = this.name.split('').reverse().join('');
  }

  static createSpecialUser(values) {
    / * ... */
  }
}

So you want to call an instance method in your hook, am I right? This should work as you already defined in your example.
User.beforeCreate(user => {
   user.prepareReversedName();
});

Please note, that you don't need to use a Promise, if nothing async happens. Otherwise if myInstanceMethodsSample is async, you should consider the asynchronous operation, what you didn't do in your example.
But I'm not really sure, whether I really understand, what your actual problem is. Can you give me an example of what you're trying to achieve?

Hi,
i tried using a hook:
User.beforeCreate((user : User) => {
console.log('BEFORE', user);
})
but im getting the error:
this.options.hooks[hookType] = this.options.hooks[hookType] || [];
                ^

TypeError: Cannot read property 'hooks' of undefined

any hint what im doing wrong?

@zlokomatic Ah, I see. Calling static members of a model is currently only available after these models are initialized by sequelize. So you need to add these models to the sequelize instance before you can call beforeCreate:

@Table
class User extends Model<User> {}

const sequelize =  new Sequelize({
        ...,
        modelPaths: ['path/to/models/dir']
});

// or
sequelize.addModels([User]);

User.beforeCreate(...);
User.findAll(...);

Hmm, this should be added to the documentation, I think. Thanks raising this issue.

Oh, then it just tslint alert..
Yeah I got a same problem with @zlokomatic 's problem.
I was trying hundreds of times, so I did like @RobinBuschmann 's did. :)
I just use this library as a test, before I change my backend as Node.js typescript.
This library is really good for begins with Object Oriented style.
Of course I should do more testing, but this project and https://github.com/evertonrobertoauler/universal-demo this project is really good basic for node.js, angular4, sequelize with typescript.
I close this issue.
p.s. sorry for terrible english


What is the configuration of your computer for experiment?

What is the configuration of your computer for experiment?
Can the code run in GPU[0] only?

1080Ti x4, 12G Memory.
Training on single GPU works well using a smaller batch size.


Creating Solr cores (5.x)

Issue Type

Bug Report / Support Request

Your Environment
The full command "vagrant --version && echo "VirtualBox vboxmanage --version" && ansible --version" is not working from my Windows Powershell, so I gathered the infomration myself.
Vagrant 1.9.2
Virtualbox 5.1.28r117968
ansible <IP_ADDRESS>
  config file = /etc/ansible/ansible.cfg
  configured module search path = Default w/o overrides
  python version = 2.7.12 (default, Nov 19 2016, 06:48:10) [GCC 5.4.0 20160609]

Your OS

Windows (10)

Summary
Hello,
I'm using Drupal VM (4.6.0) for several projects and managed to install Solr 5.5.3 - there were errors during the provisionning and I had to do manual steps myself to make the insallation succeed.
How can I create one core per project?
I tried adding them in config.yml with
solr_cores:

project1
project2
...

without any luck.
Then I tried to create them manually with "/opt/solr/bin/solr create -c new_core" (or similar), I see it in "Cloud" tab from Soilr UI, but not in "Core Admin".
What's the best/standard way to create cores?
Is the core creation stable within Drupal VM (through config.yml)?
Thank you!

@lramarojaona,
It would be useful to know what the errors you came across were during installation, and it would also be useful for us if you could provide your solr config available in config.yml.
Lastly, could you also provide the output of the provision process for the solr role?
Ideally DrupalVM installs the specified cores for v5 and v6, it copies the example files into place and runs the command you've specified. I... have a custom script which replicates this with v6 for use at work. Unfortunately Solr is a temperamental thing to configure.
So, to tell what's happening we'd need to look at the things mentioned above.

Errors during installation
I don't remember exactly the output, but I had to download the Solr archive myself and play with permissions so the script was able to install Solr
config.yml
https://gist.github.com/lramarojaona/1af48116ac326a580ea3f4006eddc0b8
Provision process output
TASK [geerlingguy.solr : Create configured cores.] *****************************
failed: [dev2] (item=project1) => {"changed": true, "cmd": "/opt/solr/bin/solr create -c project1", "delta": "0:00:00.704669", "end": "2017-12-06 10:06:29.642263", "failed": true, "item": "project1", "msg": "non-zero return code", "rc": 1, "start": "2017-12-06 10:06:28.937594", "stderr": "\nERROR: \nCollection 'project1' already exists!\nChecked collection existence using Collections API command:\nhttp://localhost:8983/solr/admin/collections?action=list", "stderr_lines": ["", "ERROR: ", "Collection 'project1' already exists!", "Checked collection existence using Collections API command:", "http://localhost:8983/solr/admin/collections?action=list"], "stdout": "\nConnecting to ZooKeeper at localhost:9983 ...\nRe-using existing configuration directory project1", "stdout_lines": ["", "Connecting to ZooKeeper at localhost:9983 ...", "Re-using existing configuration directory project1"]}
to retry, use: --limit @/vagrant/provisioning/playbook.retry
Now I see that it says "Collection 'project1' already exists!", so I'll try by deleting it.

Ok, my bad, everything's fine!
I did "bin/solr delete -c collection_name" and "bin/solr restart".
Then I ran vargant provision, the core is created!
Many thanks!


[Tooltip] should follow the trigger

Feature request
https://originui.com/sliders
As of now
https://www.originui-ng.com/sliders

@pimenovoleg I can take it, I've got the flow right now :D

@pimenovoleg I can take it, I've got the flow right now :D
it will be nice

@pimenovoleg  look at that https://github.com/radix-ng/primitives/pull/242 - I would need it merged before starting this issue

@pimenovoleg I've raised a PR for the origin-ui repo https://github.com/radix-ng/origin-ui/pull/1


Add pam configs for util-linux's runuser

Merge Checklist

All boxes should be checked before merging the PR (just tick any boxes which don't apply to this PR)

[ ] The toolchain has been rebuilt successfully (or no changes were made to it)
[x] The toolchain/worker package manifests are up-to-date
[x] Any updated packages successfully build (or no packages were changed)
[x] Package tests (%check section) have been verified with RUN_CHECK=y for existing SPEC files, or added to new SPEC files
[x] All package sources are available
[x] cgmanifest files are up-to-date and sorted (./cgmanifest.json, ./toolkit/tools/cgmanifest.json, ./toolkit/scripts/toolchain/cgmanifest.json, .github/workflows/cgmanifest.json)
[x] LICENSE-MAP files are up-to-date (./SPECS/LICENSES-AND-NOTICES/data/licenses.json, ./SPECS/LICENSES-AND-NOTICES/LICENSES-MAP.md, ./SPECS/LICENSES-AND-NOTICES/LICENSE-EXCEPTIONS.PHOTON)
[x] All source files have up-to-date hashes in the *.signatures.json files
[x] sudo make go-tidy-all and sudo make go-test-coverage pass
[x] Documentation has been updated to match any changes to the build system
[ ] Ready to merge

Summary

By default util-linux does not ship with PAM configurations for the runuser tool. Most distros use the default configurations used by Debian.
Change Log

Add default PAM configs for runuser and runuser-l to the util-linux package.

Does this affect the toolchain?

YES
Test Methodology

Doing some debugging with this branch, setting to draft until thats done.


Return compiled CSS as string

Would you be open to extending the plugin to provide the ability to return the compiled css as a string?
import style from './style.scss!';

@Component({
    template: '...',
    style: [style]
})

Cheers!

I've implemented this in a fork, no tests or bundle support:
https://github.com/KarlPurk/plugin-sass/commit/fabd142242caa584fc50e3a4d35083d80ebdee5d

After playing with this idea last week, I realised that this wouldn't be a very good idea.  A better idea is to create a separate plugin for this purpose.

Thank you for the feedback :+1:

I would really like having this implemented in this plugin. Would it be possible for the plugin to detect the import type? So default import vs. module import? If so the functionality should be relatively easy to implement.
Having two separate plugins for importing sass files into my project is no optimal solution and I would like to make use of both ways.

I don't know if this is possible with SystemJS plugins. You can look at the plugin API.
I believe you have to implement fetch and translate but I don't know how to set a result in fetch and give it to translate. The SystemJS internals are not known to me.
Any pull requests are welcome.

This is possible, but the problem with implementing this with a single plugin is what drives the decision to import vs inject?
I originally thought this could be driven at design time by config, but then you are limited to one or the other option.
The best solution would probably be to chain multiple loaders (similar to webpack) but I don't think systemjs currently supports this? - and this would still require two loaders, one for scss and one for injecting into head.
For our use case, my colleague developed plugin-scss which allows scss to be imported as a string.  We can then inject it into the head manually if we desire or work with the compiled scss directly.

Ah I understand :+1:
This is indeed a different approach as this plugin was designed for. Also I don't know if it's a good idea to mix the behaviour of this plugin with the different approach of your plugin so I think it's the best solution to have two different plugins for that. Maybe in the future it is possible to have chained multiple loaders in SystemJS / jspm so we can merge together :sunglasses:

Watching that space :sunglasses:
If nothing else the variation we have is useful for our needs with angular2 and it was highly educating to write when learning JSPM and SystemJS. So thanks for the starting point. :+1:


AssertionError: In PaddlePaddle 2.x, we turn on dynamic graph mode by default, and 'data()' is only supported in static graph mode. So if you want to use this api, please call 'paddle.enable_static()' before this api to enter static graph mode.

Traceback (most recent call last):
File "infer.py", line 182, in
show(audiopath)
File "infer.py", line 176, in show
result = infer()
File "infer.py", line 151, in infer
feeding_dict=data_generator.feeding)
File "/home/port/model_utils/model.py", line 402, in infer_batch_probs
feeder, log_probs, _ = self.create_network(is_infer=True)
File "/home/port/model_utils/model.py", line 120, in create_network
lod_level=0)
File "", line 2, in data
File "/usr/local/lib/python2.7/dist-packages/paddle/fluid/wrapped_decorator.py", line 25, in impl
return wrapped_func(*args, **kwargs)
File "/usr/local/lib/python2.7/dist-packages/paddle/fluid/framework.py", line 231, in impl
), "In PaddlePaddle 2.x, we turn on dynamic graph mode by default, and '%s()' is only supported in static graph mode. So if you want to use this api, please call 'paddle.enable_static()' before this api to enter static graph mode." % func.name
AssertionError: In PaddlePaddle 2.x, we turn on dynamic graph mode by default, and 'data()' is only supported in static graph mode. So if you want to use this api, please call 'paddle.enable_static()' before this api to enter static graph mode.

model.py 新增
import paddle
paddle.enable_static()
就可以执行了。


2.0.4 doesn't conform to Semver

The recent 2.0.4 release doesn't conform to SemVer
https://github.com/weppos/publicsuffix-ruby/commit/0d52a1ba2d3600bb77364ec36d73e842de0dc072#diff-18742d7d3f5c482249adee58c0722c58
Specifically, 2.0.4 requires ruby >= 2.0; 2.0.3 doesn't have a ruby version requirement.

@stevendaniels that's incorrect, Ruby 1.x was dropped on version 2 of the library, in 2014.
See https://github.com/weppos/publicsuffix-ruby/commit/e39a3d7023475da6052b83cd9f676fd6ee5b9032 and https://github.com/weppos/publicsuffix-ruby/commit/0d52a1ba2d3600bb77364ec36d73e842de0dc072#diff-18742d7d3f5c482249adee58c0722c58

actually, it looks like it was v1.5.0 that dropped Ruby 1.x
https://github.com/weppos/publicsuffix-ruby/blob/master/CHANGELOG.md#release-150
https://github.com/weppos/publicsuffix-ruby/commit/e39a3d7023475da6052b83cd9f676fd6ee5b9032
https://github.com/weppos/publicsuffix-ruby/compare/v1.4.6...v1.5.0
(Related to #127)


feature - comment api

Github Issue URL/Tags (required)
What's this PR do? Please give a technical summary (required)

[x] comment api

Any references (stackoverflow …) (optional)

:+1: There are no linter violations.


Dropdown example is non-functional

Issue in /src/pages/uxp/reference-spectrum/User Interface/sp-dropdown.md

`document.querySelector(".yourDropdown").addEventListener("change", evt => {
console.log(`Selected item: ${evt.target.selectedIndex}`);

})`
is given as an examples, but ".yourDropdown" is not defined anywhere in the details above, and no matter how I add IDs, or look for classes, I just can't find out where it should go.  No matter what I do, I'm getting Uncaught TypeError: Cannot read properties of null (reading 'addEventListener') as an error.
As an aside, there's no indication of how a querySelector should work, which may be a valid choice, but as someone new to this means I have a whole lot of extra tabs to open, and an inline link to what a querySelector is and does would be great.  That's not the main issue, though.

(This took me about half an hour to comb through, until I found the correct inheritance and combo of selectors.  That could be cleared up.)

Thanks @DianaProbst this is super helpful. We will (at some point) try to get this resolved and appreciate your debugging. There's a bit of conflation between the template we draw from and our repo which we'll try to sort out before resolving this.


use ubi instead of scratch

I'm not sure if I should modify the downstream directly, but some users have conerns when using the scratch, especially when building the multi-arch image. They have to modify the Dockfile after opm alpha bundle generate xxx. They will get an error if they forget to update the base image. So, I just submitted this PR, any comments are welcome! Thanks!
[yapei@yapei-mac apicast (operators)]$ opm alpha bundle validate --tag quay.io/yapei/apicast-operators-bundle:latest --image-builder podman
Command "validate" is deprecated, This subcommand is deprecated and will be removed in a future release. Migrate to operator-sdk bundle validate
INFO[0000] Create a temp directory at /var/folders/gw/6ckgw32n7rz4yncmkkpsvjl80000gn/T/bundle-2170207577  container-tool=podman
DEBU[0000] Pulling and unpacking container image         container-tool=podman
INFO[0000] running /opt/podman/bin/podman pull quay.io/yapei/apicast-operators-bundle:latest  container-tool=podman
ERRO[0002] Trying to pull quay.io/yapei/apicast-operators-bundle:latest...
Error: choosing an image from manifest list docker://quay.io/yapei/apicast-operators-bundle:latest: no image found in image index for architecture "amd64", variant "", OS "linux"  container-tool=podman
Error: error pulling image: Trying to pull quay.io/yapei/apicast-operators-bundle:latest...
Error: choosing an image from manifest list docker://quay.io/yapei/apicast-operators-bundle:latest: no image found in image index for architecture "amd64", variant "", OS "linux"
...

Hi @jianzhangbjz
You're right that we don't want to modify ./staging contents, since they get sync'd from the CNCF project and could easily get stomped on.
I'm of two minds on the subject of this alpha level feature needing support.
I think it's OK to add a new, optional flag to provide the capability, still defaulting to the existing behavior, like here. And that would be available in downstream opm versions at the latest unreleased OCP version.
But I think that we would not backport this feature functionality without a really compelling argument.

Bundles images explicitly should NOT be multi-arch images. They contain just plaintext files, so they are not platform-specific.
Only images that contain executable binaries need to be multi-arch. And the manifests inside of a bundle images should always reference the multi-arch manifest of the controller and related images.
Is the validate bundle command failing on scratch-based bundle images?


Multi Theft Auto Not working at GameDig.

⚠ Please verify that this bug has NOT been raised before.

[X] I checked and didn't find similar issue

🛡 Security Policy

[X] I agree to have read this project Security Policy

📝 Describe your problem
At first I thought there was a problem with the server I was running,but I tried to add other servers as well. I cannot add any MTA server. I always get the same failed to 1 attempt error.

🐻 Uptime-Kuma Version
1.20.2
💻 Operating System and Arch
Debian GNU/Linux 10
🌐 Browser
Brave v1.48.171
🐋 Docker Version
version 23.0.0, build e92dd87
🟩 NodeJS Version
No response

Just tested it is not working in my side too. I think you should report to gamedig's repo.
https://github.com/gamedig/node-gamedig/issues

Update: The monitor works, the documentation is misleading. The monitor requires the server list port (more precisely 22126) and works with it. When you try to add it, 22003 is entered by default. (I think because many websites/systems use this by default).


fix didn’t read from file while chose record

Thanks for this -- I was using the following in a shell script to get around the URL requirement:
here="file://$(realpath .)/$2"

Essentially turns the filename into an absolute file:// URL.
We could do that in Python too, but it probably makes more sense to open local files with the open syscall than a urllib socket.


ROX-16607: Explicitly specify connection string encoding

Description
We are missing client_encoding in the connection string definition
Checklist

[ ] Investigated and inspected CI test results
[ ] Unit test and regression tests added
[ ] Evaluated and added CHANGELOG entry if required
[ ] Determined and documented upgrade steps
[ ] Documented user facing changes (create PR based on openshift/openshift-docs and merge into rhacs-docs)

If any of these don't apply, please comment below.
Testing Performed
TODO(replace-me)
Use this space to explain how you tested your PR, or, if you didn't test it, why
you did not do so. Valid reasons include, for example, "CI is sufficient",
"No testable changes". Feel free to attach JSON snippets, curl commands,
screenshots.
In addition to reviewing your code, reviewers must also review your testing
instructions and make sure they are sufficient.

/retest

/test gke-postgres-upgrade-tests


Fixing translation for infowindow none template

Mustache templates don't know anything about _t functionality, and also interpolation was not correct. Do you think @alonsogarciapablo we could pass variables to the fields array? Any better idea?
CR: @alonsogarciapablo
cc @matallo
Fixes #9199.

Using .tpl (underscore template) instead of mustache, makes sense, idea by @alonsogarciapablo. What do you think?

👍

Merging, frontend tests ready 💋


feat: Oracle integration: Add support for DB schema and update error infra

Description

Update the Oracle plugin code to use newer error framework infra.
Add support to fetch DB schema and show dynamic query templates.

The following info is shown to the users as part of the DB schema: table names, column names for each table, column types, primary key, foreign key

Update static query templates.
Update datasource form to show mandatory fields with asterik mark
Update datasource validity check to return error on empty password field
Improve data read for the following types: timestamp with local time zone , clob, nclob
Minor refactor into modular re-usable functions

Fixes #20794 #20795
Type of change

New feature (non-breaking change which adds functionality)

How Has This Been Tested?

Manual
JUnit TC to be added via separate PR. Another issue it open to  track it.

Test Plan
Test plan links
Issues raised during DP testing
https://github.com/appsmithorg/appsmith/issues/21487
Checklist:
Dev activity

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] PR is being merged under a feature flag

QA activity:

[ ] Test plan has been approved by relevant developers
[x] Test plan has been peer-reviewed by QA
[ ] Organized project review call with relevant stakeholders after Round 1/2 of QA
[ ] Cypress test cases have been added and approved by either SDET or manual QA
[ ] Added Test Plan Approved label after reviewing all Cypress tests
[ ] Added Test Plan Approved label after developers review JUnit tests

/build-deploy-preview skip-tests=true

/ok-to-test

/build-deploy-preview skip-tests=true

/ok-to-test

/ok-to-test

/ok-to-test

/ok-to-test

/build-deploy-preview skip-tests

/build-deploy-preview skip-tests=true


[email protected] in Tailwind CDN link url

The CDN URL at https://tailwindcss.com/docs/installation/ shows [email protected] instead of what I assume should be the package name and version. The [email protected] link below leads to https://tailwindcss.com/cdn-cgi/l/email-protection

CloudFlare thinks you're a bot.

By enabling Cloudflare Email Address Obfuscation, email addresses on your web page will be obfuscated (hidden) from bots, while keeping them visible to humans. In fact, there are no visible changes to your website for visitors.

From the first link in your "Email Protection" print https://support.cloudflare.com/hc/en-us/articles/200170016-What-is-Email-Address-Obfuscation-


Test https://github.com/marketplace/backhub

Could be a good idea to backup our repos. In theory since git is decentralize-ish (thanks GitHub), we wouldn't need it, but this backup also the PR and issues. Who never knows when the giant could closed our repo, we've seen that recently with youtube-dl (I know, copyright and all, but still).

@curquiza I have no idea if it's needed.

Looks like a security issue, not on devrel side.
If we want to backup our data, this decision should not be implement by the dev rel


Use realmd instead of Samba to for RHEL 7

Use realmd instead of Samba to join the domain and generate a keytab.
https://fedorahosted.org/sssd/wiki/Configuring_sssd_with_ad_server#JoiningtheLinuxclientusingrealmd

You can also use adcli to generate keytab and join the domain on RHEL6 and RHEL7


How to disconnect an active connection?

There is of course the addConnection, but no removeConnection or something similar. So how can one disconnect from an irc server?

Call ircQuit() on the $write parameter sent to the event callback.

Just found PhergieKeepAlive where they indeed do ircQuit. Felt a bit weird to me at first, as it seems a lot more logical to me to call some method on the client to disconnect. But if it works, it's good enough for me :+1:
Thanks for the quick response.


Received an error while rendering: "Rats! WebGL hit a snag."

I was trying to use the badlands generator on Google Chrome Version 57.0.2987.133 (64-bit) and I got an error, twice!

Reloading fixed the problem but I was able to get this error two times in only about 7 consecutive renderings.

Yeah I also got this on my Windows machine, Chrome browser Version 56.0.2924.87 (64-bit)

I got the error a third time on Windows, and this time I was up-to-date.
Version 57.0.2987.133 (64-bit)
Google Chrome is up to date.

This may help: https://www.khronos.org/webgl/wiki/HandlingContextLost


Send node name in syslog

Hi, there
We use our log in node to run frame and a database node outside out cluster to collect performance logs. We`d like to know the compute node name even if the perf checks passed to support more works.
But in my mind,  the only message can be passed is listed in common-logging-handler-properties.
So, is there any way to get that?
Thanks

Hi, there
We use our log in node to run frame and a database node outside out cluster to collect performance logs. We`d like to know the compute node name even if the perf checks passed to support more works.
But in my mind, the only message can be passed is listed in common-logging-handler-properties.
So, is there any way to get that?
Thanks

Hi @difey, currently you can’t select the node list for a test as a formatting field. This is easy to support. I will change that to a feature request.

Hmm, this is a duplicate of #1068, so I will close it. Feel free to comment that issue as well.


[release-4.13] add features introspection for e2e tests

manual backport of #992

/cherry-pick release-4.12


How to make text more readable

Installed your module and I love it.  The white text that is the default for all of the modules is a little hard to read.  What do you suggest to make text more readable?
Thanks...

Add this to your css/custom.css
.module {
  background-color:rgba(0,0,0,0.5)
}

You can add more CSS rules additionally.

Have you solved this problem? I'll close this issue. You can leave any further question.
I wish you have a good luck.


Add missing leapp logger to logger.conf

framework is using the 'leapp' logger which was not being configured in the conf file
it caused certain commands to get stuck on RHEL 7 when leapp package (with the
conf file) was installed, e.g. snactor workflow sanity-check ipu

Can one of the admins verify this patch?


[Wallet CLI] wallet gas  stops working with the new RPC endpoint

fastnft/target/debug on  chris/sdk-rpc [$!]
➜ wallet addresses
Showing 5 results.
C5206DD02C86A510C4848516229B02ADDFACBE55
E9714F35BC09C907A75A8BBCE594A0A0D145B390
16E25C0FE499C87D050E4262E849590A0F71F4C4
83D50FF82D14E34DD8B91D2DA4634AC4E0DD395B
8DC4279BAF7C52FE604A39FE5356CD45B2422196

fastnft/target/debug on  chris/sdk-rpc [$!]
➜ wallet gas
Error: JSON-RPC call failed: RPC Call failed: code: -32601, message: Method not found, data: None

Caused by:
    RPC Call failed: code: -32601, message: Method not found, data: None

fastnft/target/debug on  chris/sdk-rpc [$!]
➜ wallet objects --address C5206DD02C86A510C4848516229B02ADDFACBE55
Showing 5 results.
(8DC6A6F70564E29A01C7293A9C03818FDA2D049F, SequenceNumber(0), o#088f127feb775ebb7987d10d966c91f1b6cc55f83a75fdef483734f066e4f7f8)
(ABD4F956EF290CEB80CC257666E1DE5C1F7A9127, SequenceNumber(0), o#18c25f78ab74052c88339c0b5e5837c8decc4cc6ab9a09d64a74ea4e50a94ebe)
(AE7A39EB7C89BC59FA7587CD30D533CA67FB6182, SequenceNumber(0), o#e42a57d46f35c56ea1a0c3ed8e573288940ac35fdd73cebd293a343e73f3155d)
(C7AD4A1D9F773774D5FC3FBFA07011B9E883F67D, SequenceNumber(0), o#15890fc922e340f90a3459fe5d619a951643176a0a2aa7e44c4ce4047d01cb5f)
(E8DFC7E7F71262B2C6A5BDDB8268AE38A507C53E, SequenceNumber(0), o#e014ee45c8ab1c8f331ffc1bb8f84b111be872a6b4faf8ca914e33bb871ee310)

As shown above, the wallet gas stops working after changing wallet_config to
"gateway": {
    "rpc": "http://<IP_ADDRESS>:5001"
  }

hmm that's weird, I have just tried it on main and it works fine 🫤
I have changed rpc server bin name from rpc_server to rpc-server when addressing PR comments, maybe you are still using the old binary?

Yes, that must be the case. I just tried rebuilding and it works now


Cyoa

Two big things:

Add tests.
There are some organizational changes (putting things in the right places) that will help immensely. I'd like to see the scraping happen asynchronously, which will be much easier if you don't do it in the controller. Let me know if you need more explanation / suggestions on that front.

Thanks!


How to use Custom Editor  API with Virtual Documents

Hi I am looking for dirty indicator for webview, so created an issue in vscode https://github.com/microsoft/vscode/issues/129822
But they pointed me to custom editor api, the current example works with physical documents, for my use case I dont use files.
Can you provide an example of custom editor api for virtual docs similar to webview.
Here is my extension Thunder Client

Like to show dirty indicator like below in my extension webview

What have you tried so far? This is a niche use case so we probably won't add a sample for it here

what should I try, when I can't find an example of how to use your api with virtual documents


Incorrectly warning that a mode is not available

For some reason, way-displays is saying that a mode is not available (even though it should be) and falling back to a preferred mode.
This is currently happening with my DP-2 display (over USB-C). I tried specifying it with MODE: MAX as well as the exact resolution with and without the HZ (and all the different refresh rates)
This is the error

This clearly shows those modes should exist

way-displays.log

I have the same issue with my laptop display, I have a 4k resolution but I cannot set a 1080p

It looks like DP-2 3440x1440@100Hz (99,982mHz) is immediately "skipped".
I [10:59:07] DP-2: No preferred mode, falling back to maximum available
I [10:59:07]
I [10:59:07] DP-2 Changing:
I [10:59:07]   from:
I [10:59:07]     scale:     0.750 (1.000)
I [10:59:07]     position:  1182,0
I [10:59:07]     (no mode)
I [10:59:07]     VRR:       off
I [10:59:07]   to:
I [10:59:07]     mode:      3440x1440@100Hz (99,982mHz)
I [10:59:07]
I [10:59:07] DP-2 Changing:
I [10:59:07]   from:
I [10:59:07]     scale:     0.750 (1.000)
I [10:59:07]     position:  1182,0
I [10:59:07]     (no mode)
I [10:59:07]     VRR:       off
I [10:59:07]   to:
I [10:59:07]     mode:      2560x1440@60Hz (59,951mHz)
I [10:59:07]
I [10:59:07] Changes successful

There may be a way to resolve this "safely", at the cost of speed.
Whilst I'm looking into that change, I'd be most grateful for some tests from you:

plug in DP-2 after way-displays has started
start with only DP-2 plugged in
plug the Samsung display in via HDMI - that's probably not going to work due to bandwidth limitations
start with DP-2 MODE: MAX
as above with the four 3440 x 1440 exactly specified

@github-amine-kherbouche I'd be grateful if you attached a log and your configuration.

@alex-courtis I tried a whole bunch of different permutations (logs attached). Setting the mode  to MAX vs explicit didn't seem to make any difference.
I did notice something interesting. My machine (a Thinkpad X1 Carbon) has 2xUSB-C/TB-3 connectors (doc). They don't appear to be differentiated in any docs I can find. However, when I plug into DP-1 (the left of the 2 ports) and then plug in the HDMI, it switches DP-1 to DP-2 (you can see in some of the logs DP-1 departed....DP-2 arrived). You might be on to something with the limited bandwidth;  I just have no idea how to figure it out.

Attaching the USB-C monitor first

dp1_first.log
dp1_first_max_res.log
dp2_first.log

Attaching HDMI first

hdmi_first_dp1.log
hdmi_first_dp1_restart.log
hdmi_first_dp2.log

This failure on the dell on HDMI-A-1 is odd. It's been successfully used or defaulted earlier. There are also no error messages before its mode-storm in dp1_first.log. Most other cases are reported correctly.
I [15:44:51]     mode:      1920x1200@60Hz (59,950mHz) (preferred)
I [15:44:53]
I [15:44:53] HDMI-A-1: No preferred mode, falling back to maximum available

It's fortunate you're using the thinkpad. I have an X1 yoga that has the same connector configuration. I also have a Samsung C34J79x at work, which I connect to by thunderbolt.
Are you directly connecting to the monitor via displayport, thunderbolt or via a dock?
I think you're onto something with the switching of the DP-*
The monitors appear capable of their maximum res/refresh and can be cooperative, the issue is when multiple are involved. Is that correct?
I think this is a sequencing error, with modes not being recognised as successfully set. There are options.
Which compositor do you use? I'm usually testing on river.

Replicated a situation:

nothing connected
connect thunderbolt dock with HDMI monitor, appears as DP-5
connect HDMI monitor, appears as HDMI-A-1
DP-5 departs, reappearing as DP-6
laptop display freeze and fail: "output commit failed for eDP-1"

river.1.alex.log
way-displays.1.alex.log
Unfortunately that occurs when way-displays is not running:
sway.5.alex.log

The cancelled changes, retries and missing error messages are indicative that there are "transaction" synchronisation issues. More direct detection of failures and perhaps a reset of failed states following output arrive/depart may be useful.

@thefotios did WLR_DRM_NO_MODIFIERS=1 work for you?

@alex-courtis sorry I went dark, I was in the middle of job hunting.
Setting that ENV var seems to have done the trick. I tried a few permutations of (un)plugging the displays and they all work correctly.
For posterity, I set this in ~/.config/sway/environment[^1] and it was sourced correctly. I'm running sway using systemd (from the Fedora Sway spin) but this might work for other WMs as well.
[^1]: I forgot where I found out about that location; not sure if that's a default for xdg-desktop-autostart or systemd in general.

I reckon we're all done here - please do reopen the issue if you have any other questions or issues.


Problem: Postgres regex support of named capture groups (🚀 omni_regex 0.1.0)

It doesn't support these! However, I'd really love to have it supported. For example, it'll be very useful for extracting named captures for routing regular expressions in HTTP handling.
Solution: release omni_regex based on PCRE2
It is based on pgpcre by @petere with numerous improvements.

What's missing?

[ ] Documentation
[ ] Better commentary in the source code

:+1:


Use of SRV records in upstream with active health checks leads to "failure to get a peer from the ring-balancer" on 1.2.1

Summary
When using ring-balancer with a target that resolves to SRV records, if no requests are coming in, kong doesn't do any DNS queries to refresh the target list. If there are no healthy targets when a request comes in, kong just returns "failure to get a peer from the ring-balancer" instead of re-querying the DNS to see if there are new targets.
Steps To Reproduce

a.  Create a Service A with an upstream that contains 1 target that resolve to SRV records.
b.  Enable active health checks on that upstream
Don't send any traffic to that service => The balancer doesn't refresh the target list.
Redeploy Service A => old target die, new target comes in.
The ring-balancer doesn't know about new targets, it keeps health-checking old target and marks them as unhealthy.
Do a request to Service A => You get "failure to get a peer from the ring-balancer" as there is no healthy targets and Kong doesn't requery the DNS to get new SRV records!

It's worth noting that there is a workaround:

One can disable active health-check. In that case, if there are no requests coming in, kong won't do health checks and won't mark targets as unhealthy. When a request finally comes in, it still has "healthy" peers in the load balancer, but it notices the TTL is expired for them and redo a dns query. At which point it gets the new targets and can update itself with it.

Additional Details & Logs

Kong 1.2.1
Kong debug-level logs ($ kong start --vv)

Here are the logs of this scenario, filter on "demo" for clarity we can see that:

from 9:07:23 to 9:07:53, some requests are made and the DNS is queried
from 9:07:53, no requests are made, DNS is not queried, active health checks continue
at 9:10:13, service gets redeployed, old target become unhealthy, kong doesn't know about the new target
from 9:10:13, when requests are made, kong send a 503 with {"message":"failure to get a peer from the ring-balancer"} since there are no healthy targets.


Adding a new property under VNet Properties

MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Changelog
Please ensure to add changelog with this PR by answering the following questions.

What's the purpose of the update?

[ ] new service onboarding
[ ] new API version
[x] update existing version for new feature
[ ] update existing version to fix swagger quality issue in s360
[] Other, please clarify

When you are targeting to deploy new service/feature to public regions? Please provide date, or month to public if date is not available yet.
Targeting NRP 153 release with NRP API Version 2020-11-01.
When you expect to publish swagger? Please provide date, or month to public if date is not available yet.
Targeting NRP 153 release with NRP API Version 2020-11-01.
If it's an update to existing version,  please select SDKs of specific language and CLIs that require refresh after swagger is published.

[x] SDK of .NET (need service team to ensure code readiness)
[ ] SDK of Python
[ ] SDK of Java
[ ] SDK of Js
[ ] SDK of Go
[x] PowerShell
[x] CLI
[ ] Terraform
[ ] No, no need to refresh for updates in this PR

Contribution checklist:

[x] I commit to follow the Breaking Change Policy of “no breaking changes
[x] I have reviewed the documentation for the workflow.
[x] Validation tools were run on swagger spec(s) and errors have all been fixed in this PR. How to fix?

If any further question about AME onboarding or validation tools, please view the FAQ.
ARM API Review Checklist

[ ] Ensure to check this box if one of the following scenarios meet updates in the PR, so that label “WaitForARMFeedback” will be added automatically to involve ARM API Review. Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs, all “removals” and “adding a new property” no more require ARM API review.

Adding new API(s)
Adding a new API version
Adding a new service

[ ] Please ensure you've reviewed following guidelines including ARM resource provider contract and REST guidelines. Estimated time (4 hours). This is required before you can request review from ARM API Review board.

[ ] If you are blocked on ARM review and want to get the PR merged with urgency, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.

Breaking Change Review Checklist
If there are following updates in the PR, ensure to request an approval from API Review Board as defined in the Breaking Change Policy.

[ ] Removing API(s) in stable version
[ ] Removing properties in stable version
[ ] Removing API version(s) in stable version
[ ] Updating API in stable version with Breaking Change Validation errors
[ ] Updating API(s) in preview over 1 year

Action: to initiate an evaluation of the breaking change, create a new intake using the template for breaking changes. Addition details on the process and office hours are on the Breaking change Wiki.
Please follow the link to find more details on PR review process.

Closing this PR for the time being. Will reopen once the RNM changes are in for testing and we can test it out end to end.


Brightness issue

Brightness is not working properly
https://github.com/vivek9patel/vivek9patel.github.io/issues/new

yes, someone already had made an issue with that!
link: #5


Enhancement: Pass model as constructor argument

This PR

[x] Passes the Talk Model to the constructor of the HomePageAction, instead of statically accessing it

This should make the code base more testable since we can easily mock models.
Let me know if we want to do this for the other actions, or if this is a bad idea

@BackEndTea @localheinz What's the status of this? Are we going to put the effort into moving to Doctrine?

@chartjes
I’m quite busy at the moment and can’t spare the time to invest for working on this, unfortunately.

@localheinz @derrabus @BackEndTea Can I close this PR then?

Yeah its okay to close this, i've been working on other projects lately, and don't really have the time to add doctrine here, sorry.


Unable to install on HAOS 12.4

Problem
Install failed on HAOS 12.4. Error message is:
Failed to install add-on
The command '/bin/bash -o pipefail -c apk add --no-cache nftables=1.0.9-r2' returned a non-zero code: 1
Homeassistant version
Core: 2024.7.2
Supervisor: 2024.06.2
Operating System: 12.4
Frontend: 20240710.0

there must be an error on your side. this is the correct version:
https://pkgs.alpinelinux.org/packages?name=nftables&branch=v3.20&repo=&arch=&maintainer=
try again

I have try to test with 3 commands directly on home assistant shell:

apk add --no-cache nftables=1.0.9-r2 - its success
/bin/bash -o pipefail -c apk add --no-cache nftables=1.0.9-r2 - its failure
/bin/bash -o pipefail -c 'apk add --no-cache nftables=1.0.9-r2' - its success

I think, the problem is caused by running commands in the Dockerfile.
#----
#netbird/Dockerfile
#-----

# Set shell
SHELL ["/bin/bash", "-o", "pipefail", "-c"]

# hadolint ignore=DL3003
RUN \
    apk add --no-cache \
    nftables=1.0.9-r2

it has nothing to do with home assistant shell. it's running during docker build INSIDE the container, but i changed a bit the Dockerfile try the latest version.

probably fixed since no response

I think the problem is caused by its conflict with the Old Netbird Client.
I installed the Old Netbird Client add-on first, then I found this add-on. I removed the old add-on and then installed any version of this add-on, but I got the same error message.
I have other Home Assistant hosts. I can install this add-on on those hosts without error.


Bug: Firemission module AI observer

Registering an AI observer as independent returns the following error message.
23:15:46 Error in expression <e _currentShotTargets = [];
while{alive _observer} do
{
private _possibleTargets>
23:15:46   Error position: <_observer} do
{
private _possibleTargets>
23:15:46   Error Undefined variable in expression: _observer
23:15:46 File \Documents\Arma 3 - Other Profiles\goosko\mpmissions\REVIEW\TEST_FW_1.takistan\modules\firemission\functions\FIREMIS\fn_FIREMIS_RegisterForwardObserver.sqf..., line 21
23:15:46 Error in expression <this select 10;
private _obsSide = side _observer;

private _currentShotTargets >
23:15:46   Error position: <_observer;

private _currentShotTargets >
23:15:46   Error Undefined variable in expression: _observer
23:15:46 File \Documents\Arma 3 - Other Profiles\goosko\mpmissions\REVIEW\TEST_FW_1.takistan\modules\firemission\functions\FIREMIS\fn_FIREMIS_RegisterForwardObserver.sqf..., line 18

Put the following into the serversettings.sqf
[ARTY_1,1,0,300,4,3,"Fuck 5-1",true] call FW_fnc_FIREMIS_SetArtilleryData;
[ARTY_2,1,0,300,4,3,"Fuck 5-2",true] call FW_fnc_FIREMIS_SetArtilleryData;
[FO_1,[ARTY_1,ARTY_2],1,300,1000,10,5,10,300,150,0] call FW_fnc_FIREMIS_RegisterForwardObserver;
[FO_1,0,0] call FW_fnc_FIREMIS_SetObserverSkill;

Where the AI observer is named FO_1, and artillery pieces are ARTY_1 and ARTY_2

Discussed this with @goosko.
I am able to make this work myself by putting the function call in the unit's init (in the 3D editor) and not syncing the unit to headless, it should be a server spawned unit. We can look at making it work for headless units in the future but for now we're just replicating functionality of the existing framework.
Example:
[this, [mortar], 1, 50, 1000, 100, 1, 1, 10, 100, 0] call FW_fnc_FIREMIS_RegisterForwardObserver;


Matching Strategy: Outdated equivalence of ModelReference[Submodel] and GlobalReference with same key

Page 36 of the v3.0 Part 1 specification mentions an equivalence relationship:

(Submodel)https://example.com/aas/1/1/1234859590
is identical to
(GlobalReference)https://example.com/aas/1/1/1234859590

At first glance, this is logical in the sense that both References refer to the same (Submodel) object.
However, there is a difference between ModelReference and GlobalReference, as ModelReferences are resolvable in implementations of the AAS (since it references another AAS object), whereas GlobalReference does not necessarily refer to other AAS objects.
Furthermore, in the case of AssetAdministrationShell/submodel, the spec requires a ModelReference[Submodel] anyway, making this equivalence relationship confusing and outdated, as far as I can see.
This issue came up due to an AAS wrongly modelled under this assumption in basyx-python-sdk#231

Is it possible to check only the Identifiers?

You are right. In "in the case of AssetAdministrationShell/submodel, the spec requires a ModelReference[Submodel] " in this case it would be a bug to use an external reference. The documentation needs to be clearer on the use cases these "equivalence"-relationship matches hold. It does not mean you can just substitute them in every context.

@joergneidig: can you please review: https://rwth-iat.github.io/aas-specs-antora/IDTA-01001/snapshot/IDTA-01001_General.html#_matching_algorithm_for_references

@JoergNeidig: can you please review: https://rwth-iat.github.io/aas-specs-antora/IDTA-01001/snapshot/IDTA-01001_General.html#_matching_algorithm_for_references

The new/changed notes add to the clarity of the matching. I agree on making these changes.

Worksteam AAS Specs 2024-06-13
accepted to add these clarifying notes


Make CREDEBL white label (frontend)

User Story
As a CREDEBL user, In the white label dashboard, I can able to customize the application's branding, including the app's name, and logo.
Tasks

[x] Added .env variables
[x] Updated config file with white labeling data
[x] Implement dynamic white labeling data on throughout the application
[x] Handle dynamic .env variables to set globally

This is completed.


The latest version doesn't work for MongoDB v <3.4

Introduced recently collation support (#13) breaks backward compatibility with older MongoDB versions.
The following check is always true and doesn't actually serves its purpose described in the comment.
https://github.com/WebGangster/mongoose-paginate-v2/blob/fdd5aff180986018acd733c811f3fba4e40e08bc/index.js#L67-L70

@ilyahumbar Can you try the updated version 1.0.21

Yes. Works great.
Thank you for your quick response!

@ilyahumbar Please feel to open a new thread if you see any issues.
Cheers & Happy Coding!