plotly callback and in-place update support

This PR contains initial work on providing plotly callback support in Panel.
To start with, I added callback properties that match what Dash uses (https://dash.plot.ly/interactive-graphing):
- relayout_data
- restyle_data
- click_data
- hover_data
- clickannotation_data
- selected_data
These properties are successfully updated on the Python side when interacting with a displayed figure. For example:
import panel as pn
import plotly.graph_objs as go
pn.extension('plotly')

fig_pane = pn.pane.Plotly(
    go.Figure(
        data=[go.Scatter(y=[1, 3, 2], mode='markers'),
              go.Scatter(y=[3, 2, 4], mode='markers')],
    ),
)
fig_pane

Then perform a box selection and look at the resulting selected_data property:
fig_pane.selected_data

{'points': [{'curveNumber': 0,
   'pointNumber': 1,
   'pointIndex': 1,
   'x': 1,
   'y': 3},
  {'curveNumber': 0, 'pointNumber': 2, 'pointIndex': 2, 'x': 2, 'y': 2},
  {'curveNumber': 1, 'pointNumber': 1, 'pointIndex': 1, 'x': 1, 'y': 2}],
 'range': {'x': [0.8510443353259319, 2.0522968260745795],
  'y': [1.5626990445859872, 3.4990047770700636]}}

One thing I'm not sure of, is how to use these callback properties in a declarative context. For example, I'd like to be able to write a class something like this:
class StockExplorer(param.Parameterized):
    ticker = param.Selector(default='AAPL', objects=tickers)
    window_size = param.Integer(default=6, bounds=(1, 21))

    @param.depends('ticker', 'window_size')
    def plot(self):
        return get_plot(self.ticker, self.window_size)

But have the plot function also depend on the selected_data property.  Is there some way to pre-declare the plot will return a figure so that the selected_data property can be used declaratively?

Excited to start playing with this.

Now this PR supports updating the Pane in response to property updates on the figure itself. e.g.
This is mostly reuse from the FigureWidget class that is included with plotly.py.
import panel as pn
import plotly.graph_objects as go

pn.extension('plotly')

fig = go.Figure()
fig_pane = pn.pane.Plotly(fig)
fig_pane

# _py2js_addTraces
fig.add_scatter(y=[1, 3, 2])
fig.add_bar(y=[1, 0.4, 0.7]);
fig.add_scatter(y=[2, 1.4, 1.7], marker_size=20, mode='markers');

# _py2js_moveTraces
fig.data = [fig_pane.object.data[i] for i in [0, 2, 1]]

# _py2js_deleteTraces
fig.data = [fig_pane.object.data[1]]

# _py2js_restyle
fig.data[0].marker.color = 'green'

# _py2js_update
with fig.batch_update():
    fig.data[0].marker.color = 'green'
    fig.data[0].marker.size = 30

# _py2js_animate
with fig_pane.object.batch_animate():
    fig.data[0].marker.color = 'orange'
    fig.data[0].marker.size = 40

Unlike FigureWidget, this does not sync default properties from the JavaScript side back to the Python side.  It also does not support the on_change, on_selection, etc. figure callbacks.  There's no reason these couldn't be supported, but they would involve a bit more logic to be borrowed from the FigureWidget implementation.

After some experimentation and gaining some more understanding of the bokeh model system, I changed how the property updates are performed.  The prior iteration used the Plotly.js messages generated by plotly.py to update Plotly.js on the JavaScript side.
But this approach doesn't keep the models themselves up to date with the changes in the figure. In FigureWidget this is handled by manually updating the model in response to each message.  This is done on the JavaScript side, as ipywidgets maintains only a single model in JavaScript that may correspond to multiple views.
If I'm understanding bokeh properly, it looks like a separate model is created for each view.  And the model has both a Python object and a JavaScript object.  So manually updating all of these models in response to Plotly.js messages would be a bit challenging.
The new approach updates to full figure object in response to any of these changes.  This passes the synchronization through the already somewhat optimized _update method.  In the future we could work on making this method even more efficient by sending only patches, and applying the patches with Plotly.js methods, when small changes are detected.

@philippjfr, I think this is ready for review when you have time.
I am have one issue that I might need help tracking down.  It seems that occasionally the selected_data property is not successfully synchronized from JavaScript to Python.  Are there any race conditions or throttling situations I should be considering when updating these model properties in JavaScript?

So manually updating all of these models in response to Plotly.js messages would be a bit challenging.

Could you expand on this? I'm not quite following why this is problematic or challenging.

Are there any race conditions or throttling situations I should be considering when updating these model properties in JavaScript?

Not afaik, I'll play around.

Could you expand on this? I'm not quite following why this is problematic or challenging.

Sure. Let's say a user updates a trace with property assignment
fig.data[1].marker.line.width = 5

The Figure object will update it's internal dict representation and call it's _send_restyle_msg method with arguments
style={'marker.line.width': 5}, trace_indexes=[1]

These arguments are exactly what plotly.js needs in order to update the JavaScript figure with
Plotly.restyle(this.el, {'marker.line.width': 5}, [1])

In https://github.com/pyviz/panel/pull/529/commits/259c932846f67a2100d346c8f651ff05edc14178, I transferred these arguments to the JavaScript side with the _py2js_restyle property and then called Plotly.restyle whenever this property changed.  This approach successfully updates the original figure view, but doesn't update the backing model so when the pane is displayed again, the original out-of-date view is shown.
In the FigureWidget case, the FigureWidget class is an ipywidgets subclass and so the source-of-truth dictionary for the figure is the model on the Python side (So there is no need to separately update a model in Python).  To update the figure model in JavaScript, there is a custom do_restyle function that inputs the _send_restyle_msg message data and then performs the update on the JavaScript model. As long as this function is equivalent to what Plotly.js does, then everything stays in sync.
To do something similar with Bokeh models, I think we would need to have a separate Python implementation of restyle that updates each model in-place. And we would need a JavaScript implementation of restyle to update each JavaScript model in-place (this we could reuse from FigureWidget if we can include lodash as a dependency).  If the Python and JavaScript restyle functions are correct then everything would stay in sync.
The restyle function isn't complicated to implement, but it's just one of 7 functions that would need to be implemented, both in Python and JavaScript, to make this work (restyle, relayout, addTraces, moveTraces, removeTraces, update, and animate).  So perhaps I should have said that this would be "tedious" rather than "challenging" 🙂
Though maybe Bokeh has something that would make this simpler than I'm picturing.
Does that make sense?

I think this also needs to happen not only on the constructor but whenever a user updates the object. I think we should probably add a callback on changes to .object which calls _to_figure and caches the created figure. Something like:

I reworked the installation of the message callbacks to happen in an _update_figure method that depends on object.  At this point I'm only keeping track of the most recent figure so that we can skip the callback registration if a Pane's figure is assigned back to itself. And I'm not converting dicts to Figures in here, just checking if object is a Figure and installing the callbacks if it is.
I'm not confident I fully understood your suggestion, so let me know if there are some more things that should change here.

Here is a selection example that sometimes doesn't update for me
import panel as pn
import plotly.graph_objects as go
import plotly.express as px
import numpy as np
import pandas as pd
pn.extension("plotly")

df = px.data.iris()

fig = (go.Figure(layout_showlegend=False)
       .add_scatter(x=df.sepal_width,
                    y=df.petal_length,
                    mode='markers',
                    hoverinfo='x')
       .update_layout(dragmode='select'))
fig_pane = pn.pane.Plotly(fig)

md = pn.pane.Markdown()

def update_selected(event):
    selected_data = fig_pane.selected_data
    if  selected_data:
        point_inds = [point['pointNumber']
                      for point in selected_data['points']
                      if point['curveNumber'] == 0]
    else:
        point_inds = []
    md.object = str(point_inds)

fig_pane.param.watch(update_selected, 'selected_data')

pn.layout.Column(fig_pane, md)

Note: If I disable setting the hover properties from the client, then I don't see the issue at all.  So it looks like it has something to do with other properties changing quickly around the same time as the selection.

Should I merge as a first cut and then we can address issues in another PR?

Will just go ahead with that. We should probably keep track of any problems and long-term plans in a separate issue.

Sounds good, thanks!


Fix Android createLinkProperties

Hello,
I wanted to create a link with alias, unfortunelty this didn't worked. After a bit of research, its look like params : alias, campaign, stage were not properly modified via setAlias, setCampaign, setStage accordinaly in createLinkProperties
Version of Branch sdk : 0.14.12
Version of RN Branch : 1.1.1

@jchesne Thanks for the PR. That looks like some copypasta gone wrong. I'm in the process of setting up native unit tests for exactly this sort of thing. Thanks for catching it before I did.
Branch would like to thank you for helping to improve this product. You can contact<EMAIL_ADDRESS>to receive swag if you so desire.

@jchesne I have added your change to the maintenance-1.1 branch. Would you be able to try that branch and verify? I intend to do a patch 1.1.2 release soon anyway, and this change will be there. (It will also be added to the maintenance-0.9 branch and released as a patch for RN < 0.40.)


[Do not merge] Added info about how to get a copy of a birth or death certificate

⚠ This repo is Continuously Deployed: make sure you follow the guidance ⚠
I've updated the outcomes for the registering a birth and death abroad smart answers to include a sentence about how to get a copy of a birth or death certificate.
I've also updated the negative contractions in the registering a birth outcome.

Please don't merge or deploy, I need to get the previews fact checked by the department.

Please don't merge or deploy, I need to get the previews fact checked by the department.

@hannahwhittaker no worries! Just a tip - if you are still working on a PR and want to make sure it doesn't get merge, you can make it a draft

@hannahwhittaker no worries! Just a tip - if you are still working on a PR and want to make sure it doesn't get merge, you can make it a draft

Amazing, thanks @theseanything! I didn't know I could do that!

Amazing, thanks @theseanything! I didn't know I could do that!

The dept have approved these changes, they can be merged and deployed whenever.

The dept have approved these changes, they can be merged and deployed whenever.


fix(security): secrets-config user connect using TLS

Closes #3694
Signed-off-by: Bryon Nevis<EMAIL_ADDRESS>PR Checklist
Please check if your PR fulfills the following requirements:

[X] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

If your build fails due to your commit message not passing the build checks, please review the guidelines here: https://github.com/edgexfoundry/edgex-go/blob/master/.github/Contributing.md.
What is the current behavior?
Secrets config utility contacts Kong via insecure port.
Interferes with ability to configure Kong remotely.
Issue Number:
#3694
What is the new behavior?
Secrets config utility contacts Kong via secure port.
Does this PR introduce a breaking change?

[ ] Yes
[X] No

New Imports

[ ] Yes
[X] No

Specific Instructions
Are there any specific instructions or things that should be known prior to reviewing?
Other information

Codecov Report

Merging #3698 (0c24509) into main (22189a2) will not change coverage.
The diff coverage is 87.50%.

@@           Coverage Diff           @@
##             main    #3698   +/-   ##
=======================================
  Coverage   47.27%   47.27%
=======================================
  Files         112      112
  Lines        9180     9180
=======================================
  Hits         4340     4340
  Misses       4466     4466
  Partials      374      374

Impacted Files
Coverage Δ

...l/security/config/command/proxy/adduser/command.go
47.61% <75.00%> (ø)

...l/security/config/command/proxy/deluser/command.go
72.97% <100.00%> (ø)

...ernal/security/config/command/proxy/tls/command.go
85.54% <100.00%> (ø)

Continue to review full report at Codecov.

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


[Ionic v4] ion-toolbar with opacity doesn't render the correct color

Ionic Info
@ionic/angular: "4.0.0-beta.2"
Describe the Bug
Setting an --opacity to an ion-toolbar doesn't render the proper cooled. See screenshot white + 0.9 end up being grey
Note: This problem poped with beta.2, was ok in alpha.11 till beta.1
Expected Behavior
A proper rendering of background with opacity on the toolbar
Screenshot

Thanks for the issue! Is the ion-toolbar inside of an ion-footer or another element?
This might be the same issue I am seeing with the conference app when making the toolbar transparent:

cc @manucorporat @camwiegert

@brandyscarney exactly, the ion-toolbar is inside an ion-footer (sorry for having not posted the code in a first place)

@peterpeterparker Thanks! I also noticed this upon upgrading to beta.2. Added it to the project board to look into!

@manucorporat finally it took me hours but I was able to reproduce the problem in a starter app but it's really a tricky one to reproduce
I kind of think that it only happens in "sub-pages" (nav.goForward) and only on iOS devices (in my app it happens too on Android but in this starter app I was only able to reproduce the problem on iOS)
##How to reproduce the problem##
git clone https://github.com/peterpeterparker/ion-toolbar-opacity.git
cd ion-toolbar-opacity
ionic serve
open chrome
open chrome debugger
toggle device toolbar
select iPhone 6/7/8
access http://localhot:8100
click button "Navigate" to access the 2nd page
scroll the all list and observe the color of the footer, from white with opacity it will turns grey

##Code##
component:
<ion-header>
   <ion-toolbar>
      <ion-buttons slot="start">
      <ion-back-button></ion-back-button>
      </ion-buttons>
      <ion-title>
       SECOND PAGE
     </ion-title>
   </ion-toolbar>
</ion-header>

<ion-content fullscreen="true">
  <ion-list>
     <ion-item>
      <ion-label>Whatever item</ion-label>>
    </ion-item>
  </ion-list>

<ion-footer>
 <ion-toolbar>
  <ion-button expand="full" shape="round">
     Whatever action
   </ion-button>
  </ion-toolbar>
</ion-footer>

scss
ion-toolbar {
   --background: white;
   --opacity: 0.9;
}

I confirm that this is solved in v4-beta.5
❤ @manucorporat @brandyscarney

In Beta 7, the opacity is being applied correctly to the toolbar, but the fullscreen is not being applied to the


Store the Yammer authentication token in local storage

Q
A

Bug fix?
yes

New feature?
no

New sample?
no

Related issues?
partially #467 and #790

This change stores the Yammer authentication token in local storage for any subsequent call, so the user doesn't need to sign in again after restarting their session. This is necessary since the Yammer cookie is no longer persistent across sessions.

@sschutten great work !

Excellent add-on @sschutten, thx for this one.

@sschutten Should the token be stored here as well?
https://github.com/sschutten/sp-dev-fx-webparts/blob/dev/samples/react-yammer-api/src/webparts/reactYammerApi/yammer/YammerProvider.ts#L169


onmouseover event over a pushpin not working

Hi Ricky,
I have a similar code and I am not able to check if I have hovered over a particular pushpin. The elementID is always null.
map.AttachEvent("onmouseover", onMouseOverCallback);
…
function onMouseOverCallback(e)
{
// check to see if we have hovered over a pushpin
if(e.elementID != null)
{
// do something
}
}
Best Regards,
Deepu Abraham K

Use the GetId() function. The elementID property was never a documented part of the V6.3 API and you were meant to use the GetId function.


[Snyk] Upgrade tape from 5.6.4 to 5.6.5

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to upgrade tape from 5.6.4 to 5.6.5.
:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 1 version ahead of your current version.
The recommended version was released 23 days ago, on 2023-07-12.

Release notes

Package name: tape

5.6.5 - 2023-07-12v5.6.5

5.6.4 - 2023-07-02v5.6.4

    from tape GitHub release notes

Commit messages

Package name: tape

fea1937 v5.6.5
9bbbcfe [Fix] Results: show a skip string on tests, not just on assertions
109a791 [Deps] update `deep-equal`

Compare

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:
🧐 View latest project report
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

Codecov Report
Patch coverage has no change and project coverage change: -0.34% :warning:

Comparison is base (9034862) 89.89% compared to head (327df11) 89.56%.
Report is 3 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #296      +/-   ##
==========================================
- Coverage   89.89%   89.56%   -0.34%
==========================================
  Files           9        9
  Lines         297      297
==========================================
- Hits          267      266       -1
- Misses         30       31       +1

see 1 file with indirect coverage changes

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


Thanks

We don't have such options right now, but we are going to think about it and get back to you with new feature that is going to support that. Please use a reload of the entire tree until we develop a better way about your case.

Ok, thanks a lot.


Update MSI-enabled resource types

Add API Management to the list of Azure resource types that support managed service identities.

#sign-off


chore: Run tests from migration

Signed-off-by: Lakshmipathi<EMAIL_ADDRESS>

Increased plv8 re-tries to 40 from 20. Because while running inside docker - tests taking more time because of less resources than ami builder.

Even after 40 retries - async within docker fails to find plv8 status. Possible issue with async inside docker https://github.com/ansible/ansible/issues/77046 Will revert retry to 20
@sweatybridge ,  will add a new flag use_async_mode. We can disable this flag for docker builds and let docker use linear build, while enabling this flag for ami build to use async mode. Will send this as separate PR


Fixing notebook flakes

I wanted to test nbsmoke on pyviz, but ran into a few things I evidently hadn't through about before...

examples/tutorial/00_Setup.ipynb has various unused imports - except that the idea is just to check those things import ok before starting the tutorial. In many cases, the version matters, so maybe we could use the names that way? Or otherwise skip from lint checking? Or add a function to pyviz that checks the env is ok?

examples/tutorial/03_Customizing_Visual_Appearance.ipynb contains ls *.png. On windows, I think that notebooks provide ls as an alias of dir. I find these kinds of aliases confusing, and would rather replace with a python function. E.g. if someone does ls examples/*.png it's fine on linux but won't work on windows, because / is a switch for dir, not a path separator.

Notebooks sometimes contain unused imports at the top ready for people to use in exercises. Not sure what to do about that. I guess nobody will want to see something like import scipy.stats as ss # noqa: for you to use later in an exercise, right?

(nbsdev) D:\code\pyviz\pyviz>pytest --nbsmoke-lint --nbsmoke-lint-debug examples\tutorial
=============================================================== test session starts ================================================================
platform win32 -- Python 3.6.8, pytest-4.3.0, py-1.8.0, pluggy-0.9.0 -- d:\venvs\nbsdev\scripts\python.exe
cachedir: .pytest_cache
rootdir: D:\code\pyviz\pyviz, inifile: tox.ini
plugins: nbsmoke-0.2.7.post3+gc839325.dirty
collected 19 items

examples/tutorial/00_Setup.ipynb::D:\code\pyviz\pyviz\examples\tutorial\00_Setup.ipynb FAILED                                                 [  5%]
examples/tutorial/01_Workflow_Introduction.ipynb::D:\code\pyviz\pyviz\examples\tutorial\01_Workflow_Introduction.ipynb PASSED                 [ 10%]
examples/tutorial/02_Annotating_Data.ipynb::D:\code\pyviz\pyviz\examples\tutorial\02_Annotating_Data.ipynb PASSED                             [ 15%]
examples/tutorial/03_Customizing_Visual_Appearance.ipynb::D:\code\pyviz\pyviz\examples\tutorial\03_Customizing_Visual_Appearance.ipynb FAILED [ 21%]
examples/tutorial/04_Working_with_Tabular_Data.ipynb::D:\code\pyviz\pyviz\examples\tutorial\04_Working_with_Tabular_Data.ipynb FAILED         [ 26%]
examples/tutorial/05_Working_with_Gridded_Data.ipynb::D:\code\pyviz\pyviz\examples\tutorial\05_Working_with_Gridded_Data.ipynb PASSED         [ 31%]
examples/tutorial/06_Network_Graphs.ipynb::D:\code\pyviz\pyviz\examples\tutorial\06_Network_Graphs.ipynb PASSED                               [ 36%]
examples/tutorial/07_Geographic_Data.ipynb::D:\code\pyviz\pyviz\examples\tutorial\07_Geographic_Data.ipynb PASSED                             [ 42%]
examples/tutorial/08_Custom_Interactivity.ipynb::D:\code\pyviz\pyviz\examples\tutorial\08_Custom_Interactivity.ipynb PASSED                   [ 47%]
examples/tutorial/09_Operations_and_Pipelines.ipynb::D:\code\pyviz\pyviz\examples\tutorial\09_Operations_and_Pipelines.ipynb FAILED           [ 52%]
examples/tutorial/10_Working_with_Large_Datasets.ipynb::D:\code\pyviz\pyviz\examples\tutorial\10_Working_with_Large_Datasets.ipynb FAILED     [ 57%]
examples/tutorial/11_Streaming_Data.ipynb::D:\code\pyviz\pyviz\examples\tutorial\11_Streaming_Data.ipynb PASSED                               [ 63%]
examples/tutorial/12_Parameters_and_Widgets.ipynb::D:\code\pyviz\pyviz\examples\tutorial\12_Parameters_and_Widgets.ipynb FAILED               [ 68%]
examples/tutorial/13_Deploying_Bokeh_Apps.ipynb::D:\code\pyviz\pyviz\examples\tutorial\13_Deploying_Bokeh_Apps.ipynb FAILED                   [ 73%]
examples/tutorial/A1_Exploration_with_Containers.ipynb::D:\code\pyviz\pyviz\examples\tutorial\A1_Exploration_with_Containers.ipynb FAILED     [ 78%]
examples/tutorial/A2_Dashboard_Workflow.ipynb::D:\code\pyviz\pyviz\examples\tutorial\A2_Dashboard_Workflow.ipynb PASSED                       [ 84%]
examples/tutorial/index.ipynb::D:\code\pyviz\pyviz\examples\tutorial\index.ipynb PASSED                                                       [ 89%]
examples/tutorial/scipy18.ipynb::D:\code\pyviz\pyviz\examples\tutorial\scipy18.ipynb PASSED                                                   [ 94%]
examples/tutorial/strata_ny18.ipynb::D:\code\pyviz\pyviz\examples\tutorial\strata_ny18.ipynb PASSED                                           [100%]

===================================================================== FAILURES =====================================================================
_________________________________________________________________________  _________________________________________________________________________
D:\code\pyviz\pyviz\examples\tutorial\00_Setup.ipynb
line 54 col 0: 'pandas' imported but unused
line 55 col 0: 'datashader' imported but unused
line 56 col 0: 'dask' imported but unused
line 57 col 0: 'geoviews' imported but unused
To see python source that was flake checked: D:\code\pyviz\pyviz\examples\tutorial\00_Setup.nbsmoke-debug.py
_________________________________________________________________________  _________________________________________________________________________
D:\code\pyviz\pyviz\examples\tutorial\03_Customizing_Visual_Appearance.ipynb
Error - see captured stderr, below.
To see python source that was flake checked: D:\code\pyviz\pyviz\examples\tutorial\03_Customizing_Visual_Appearance.nbsmoke-debug.py
--------------------------------------------------------------- Captured stderr call ---------------------------------------------------------------
D:\code\pyviz\pyviz\examples\tutorial\03_Customizing_Visual_Appearance.ipynb:229:5: invalid syntax
ls *.png
    ^
_________________________________________________________________________  _________________________________________________________________________
D:\code\pyviz\pyviz\examples\tutorial\04_Working_with_Tabular_Data.ipynb
line 33 col 0: 'scipy.stats as ss' imported but unused
To see python source that was flake checked: D:\code\pyviz\pyviz\examples\tutorial\04_Working_with_Tabular_Data.nbsmoke-debug.py
_________________________________________________________________________  _________________________________________________________________________
D:\code\pyviz\pyviz\examples\tutorial\09_Operations_and_Pipelines.ipynb
line 37 col 0: 'numpy as np' imported but unused
To see python source that was flake checked: D:\code\pyviz\pyviz\examples\tutorial\09_Operations_and_Pipelines.nbsmoke-debug.py
_________________________________________________________________________  _________________________________________________________________________
D:\code\pyviz\pyviz\examples\tutorial\10_Working_with_Large_Datasets.ipynb
line 113 col 0: 'datashader.colors.inferno' imported but unused
To see python source that was flake checked: D:\code\pyviz\pyviz\examples\tutorial\10_Working_with_Large_Datasets.nbsmoke-debug.py
_________________________________________________________________________  _________________________________________________________________________
D:\code\pyviz\pyviz\examples\tutorial\12_Parameters_and_Widgets.ipynb
line 29 col 0: 'bokeh.document.Document' imported but unused
To see python source that was flake checked: D:\code\pyviz\pyviz\examples\tutorial\12_Parameters_and_Widgets.nbsmoke-debug.py
_________________________________________________________________________  _________________________________________________________________________
D:\code\pyviz\pyviz\examples\tutorial\13_Deploying_Bokeh_Apps.ipynb
line 81 col 0: 'bokeh.document.Document' imported but unused
line 83 col 0: 'holoviews.streams.RangeXY' imported but unused
line 84 col 0: 'cartopy.crs' imported but unused
To see python source that was flake checked: D:\code\pyviz\pyviz\examples\tutorial\13_Deploying_Bokeh_Apps.nbsmoke-debug.py
_________________________________________________________________________  _________________________________________________________________________
D:\code\pyviz\pyviz\examples\tutorial\A1_Exploration_with_Containers.ipynb
line 90 col 4: local variable 'y' is assigned to but never used
To see python source that was flake checked: D:\code\pyviz\pyviz\examples\tutorial\A1_Exploration_with_Containers.nbsmoke-debug.py
======================================================= 8 failed, 11 passed in 23.34 seconds =======================================================

Thanks for doing this!

In many cases, the version matters, so maybe we could use the names that way? Or otherwise skip from lint checking? Or add a function to pyviz that checks the env is ok?

I  vote for skipping lint checking or checking the version in the notebook. I think that the hard part about checking the version will be that we might need to change that section a fair bit. But it'd be nice to have confidence that if that notebook passes, the rest should be good.

examples/tutorial/03_Customizing_Visual_Appearance.ipynb contains ls *.png. On windows, I think that notebooks provide ls as an alias of dir. I find these kinds of aliases confusing, and would rather replace with a python function.

I agree that it should be a python function. Probably isn't just out of convenience and familiarity

Notebooks sometimes contain unused imports at the top ready for people to use in exercises. Not sure what to do about that. I guess nobody will want to see something like import scipy.stats as ss # noqa: for you to use later in an exercise, right?

I think in general as long as we know that people are in a good env, it is probably better to import things when they are needed. So the import could be commented out in the exercise section. Having all the imports at the top with #noqa just makes it feel like it should be a .py file.

I vote for checking the version.  ls *.png is definitely just laziness; feel free to Pythonify that.  Leaving the unneeded imports commented out seems clear enough.

Ok cool cool. I am working on finishing the linting. Will be pushing to this branch shortly.

Closes #173


co19/Language/Classes/Instance_Methods/covariant_A02_t03 fails on dart2wasm

co19/Language/Classes/Instance_Methods/covariant_A02_t03 fails on dart2wasm with the following error
/==========================================================================================================\
| co19/Language/Classes/Instance_Methods/covariant_A02_t03 is new and failed (RuntimeError, expected Pass) |
\==========================================================================================================/

--- Command "dart2wasm" (took 01.000643s):
DART_CONFIGURATION=ReleaseX64 CHROME_PATH=/b/s/w/ir/third_party/browsers/chrome/chrome/google-chrome sdk/bin/dart2wasm_developer -Dtest_runner.configuration=dart2wasm-linux-chrome /b/s/w/ir/tests/co19/src/Language/Classes/Instance_Methods/covariant_A02_t03.dart /b/s/w/ir/out/ReleaseX64/generated_tests/dart2wasm-linux-chrome/tests_co19_src_Language_Classes_Instance_Methods_covariant_A02_t03/covariant_A02_t03.wasm

exit code:
0

--- Command "chrome" (took 157ms):
/b/s/w/ir/tools/sdks/dart-sdk/bin/dart pkg/test_runner/bin/launch_browser.dart chrome http://<IP_ADDRESS>:38553/root_build/generated_tests/dart2wasm-linux-chrome/tests_co19_src_Language_Classes_Instance_Methods_covariant_A02_t03/test.html?crossOriginPort=41805

Runtime error:
RuntimeError: illegal cast
RuntimeError: illegal cast
    at dynamic call entry (http://<IP_ADDRESS>:38553/root_build/generated_tests/dart2wasm-linux-chrome/tests_co19_src_Language_Classes_Instance_Methods_covariant_A02_t03/covariant_A02_t03.wasm:wasm-function[694]:0x24d31)
    at method forwarder for 'call' (http://<IP_ADDRESS>:38553/root_build/generated_tests/dart2wasm-linux-chrome/tests_co19_src_Language_Classes_Instance_Methods_covariant_A02_t03/covariant_A02_t03.wasm:wasm-function[586]:0x223bb)
    at main closure at file:///b/s/w/ir/tests/co19/src/Language/Classes/Instance_Methods/covariant_A02_t03.dart:35:17 (http://<IP_ADDRESS>:38553/root_build/generated_tests/dart2wasm-linux-chrome/tests_co19_src_Language_Classes_Instance_Methods_covariant_A02_t03/covariant_A02_t03.wasm:wasm-function[577]:0x2209f)
    at closure wrapper at file:///b/s/w/ir/tests/co19/src/Language/Classes/Instance_Methods/covariant_A02_t03.dart:35:17 (http://<IP_ADDRESS>:38553/root_build/generated_tests/dart2wasm-linux-chrome/tests_co19_src_Language_Classes_Instance_Methods_covariant_A02_t03/covariant_A02_t03.wasm:wasm-function[585]:0x22339)
    at Expect.throws (http://<IP_ADDRESS>:38553/root_build/generated_tests/dart2wasm-linux-chrome/tests_co19_src_Language_Classes_Instance_Methods_covariant_A02_t03/covariant_A02_t03.wasm:wasm-function[583]:0x222d4)
    at main (http://<IP_ADDRESS>:38553/root_build/generated_tests/dart2wasm-linux-chrome/tests_co19_src_Language_Classes_Instance_Methods_covariant_A02_t03/covariant_A02_t03.wasm:wasm-function[62]:0x1561b)
    at main tear-off (http://<IP_ADDRESS>:38553/root_build/generated_tests/dart2wasm-linux-chrome/tests_co19_src_Language_Classes_Instance_Methods_covariant_A02_t03/covariant_A02_t03.wasm:wasm-function[64]:0x1583f)
    at _invokeMain (http://<IP_ADDRESS>:38553/root_build/generated_tests/dart2wasm-linux-chrome/tests_co19_src_Language_Classes_Instance_Methods_covariant_A02_t03/covariant_A02_t03.wasm:wasm-function[73]:0x15e7e)
    at Module.invoke (http://<IP_ADDRESS>:38553/root_build/generated_tests/dart2wasm-linux-chrome/tests_co19_src_Language_Classes_Instance_Methods_covariant_A02_t03/covariant_A02_t03.mjs:174:28)
    at http://<IP_ADDRESS>:38553/root_build/generated_tests/dart2wasm-linux-chrome/tests_co19_src_Language_Classes_Instance_Methods_covariant_A02_t03/test.html?crossOriginPort=41805:29:23

exit code:
0

--- To debug locally, run:
/b/s/w/ir/tools/sdks/dart-sdk/bin/dart /b/s/w/ir/pkg/test_runner/bin/http_server.dart -p 38553 -c 41805 --network <IP_ADDRESS> --build-directory=/b/s/w/ir/out/ReleaseX64/ --runtime=chrome --packages=/b/s/w/ir/.dart_tool/package_config.json

--- Re-run this test:
python3 tools/test.py -n dart2wasm-linux-chrome co19/Language/Classes/Instance_Methods/covariant_A02_t03

I found what's going wrong, but I'm not 100% sure how to fix it. Minimal repro:
class A {
  void m1(Object a) {
    print("A.m1 called: $a");
  }
}

class B extends A {
  void m1(covariant int a) {
    print("B.m1 called: $a");
  }
}

void main() {
  B b = B();
  dynamic d = b.m1;
  d(Object());
  d(42);
}

Here the class A doesn't have a member m1 in kernel. I guess TFA figures out that it's never instantiated and there's no way to call A.m1 and drops the member.
The problem is, Wasm type of B.m1 (and type of its tear-off) is computed from the selector type of the method m1, and without A.m1 in kernel the "least upper bound" of all m1s is int -> void.
As a result, the Wasm type of the function (computed from the selector) and kernel type of its tear-off (computed from the kernel type of the function, by replacing covariant arguments with Object) do not agree.
(I don't understand why we replace covariant args with Object instead of the superclass's arg type, but even if this is a bug it's a separate one.)
The Wasm type is used as the cast target, and the kernel type is used in the runtime type check before the cast. Because the kernel type is Object the type check succeeds, but cast to int fails.
I see two ways to fix this:

For a covariant arg use the superclass's argument type always (don't use selector type).
Add another type checking step in tear-offs before calling the actual member.

There may be other ways as well.
(2) makes more sense as it doesn't penalize the common case of calling the member directly.

This test is passing on dart2wasm now.
The minimal repro above is behaving the same on wasm as on VM with throwing a type error.
Assuming stale.


Update pvp-performance-tracker to v1.4.4

Minor updates:

Limit amount of previous fights rendered in the Fight History Panel (while keeping their stats saved and included in the Total Stats), to avoid excessive UI Creation/ram usage. This is configurable under the "Max Rendered Fights" config, default to 200.
Added dps/deserved damage support for crystal armor boosts on crystal bow and fbow.

I just noticed that I only applied the first rendering fix while multiple fights are added to the panel, not when only one fight is added. Going to fix quick but it's pretty minor if anyone happened to be about to merge.

Ok it's good to go now, re-tested in LMS with the extra fixes.


[Aura-Safe][Safe Management] New pop-ups when creating/allowing a Safe

Refer to New Mock-ups on Docs:
https://www.notion.so/aura-network/Function-Safe-Management-0f7853ad90fd44f9b91e0ecca2c834df

Chưa đúng yêu cầu
Not meet requirement


No web UI on 8080 locally

I am following the guide for local deployment using docker-compose, and all seems fine.
I can see the admin UI in http://localhost:8000/cfg/admin/safe_apps/client/ but I cannot open the safe UI on http://localhost:8080.
Is there something I am missing?
My .env file is this
REVERSE_PROXY_PORT=8000
REVERSE_PROXY_UI_PORT=8080

CFG_VERSION=latest
CGW_VERSION=latest
TXS_VERSION=latest
UI_VERSION=latest

RPC_NODE_URL=https://testnet.rpc.zora.energy

UPDATE: The UI now loads, it seems that you need to wait for like 10 minutes to be accessible


Unable to login - 2FA

I have 2 factor authentication turned on my Google account and after giving password it asks me to swipe up and tap yes on a notification but I'm not getting any notifications

Not able to login

I'm using Huawei y6p 2020 and everytime I log in it says "There was a problem communicating with google server, try again later" how can I fix that?

Asim hsjhs hdudhduh hdudhduh hdudhduh

Uploading Hato gussa mat dilao mujhe   #trending #explorepage #fypシ   #foryoupage  #lovestatus #followｍe #instradaily.mp4…


Add ECF Go Live Date to the JSON

For a new project we're working on we have a list of older cases that we want to download from PACER. Some of these are in PACER and some are not, depending on the date of the content and when the go live date for the given jurisdiction was.
If we could add the go live date to the JSON and HTML returned by this app, that'd be very useful, since we're trying to scope this project based on the amount of data we'll be able to download.
No idea your interest in adding another field to the scraper, but I might be able to do it if you don't have interest but are willing to deploy an updated version of the code.

Sure I can add that to the HTML and JSON.

Fantastic!

PACER is spectacular in all ways. This is great, @audiodude. I can normalize it from here. Should I file a bug with the people that run PACER to tell them their data is crap? J/k, that'd never work.

Oh, and I'm gonna close this! Thanks for helping out! This is super helpful.


Add Test for GCP Storage Resource Sample

This adds a test for the GCP Storage Resource Sample.
Work in progress: #705.

Thanks guys, I fixed the copy paste error here and in the other storage integration test :)
PTAL


Provide option to clean / clean build

When switching branch, we need to do clean make. support is needed from UI.

Done with b97327e400c42e9358bad3218972d5bad97d15c5


Remove array-find-index-polyfill?

I've upgraded redux-form from v3 to v6, a weird thing happened.

I realized it is redux-form that includes the polyfill.
It is fine in most cases, but it did cause an unexpected bug due to some bad code in our project:
for (var key in arr) { } // wrong
for (var item of arr) { } // right

Just a suggestion, let's remove the (aggressive) polyfill? 😆  ...since it is only used in one place.

I agree that we can remove that polyfill.
...but the code you're having problems with is exactly the same in both cases, just variables renamed :)

Great! @erikras

@gustavohenke ugh.. typo.. I meant for item of arr, updated!

Released as v6.5.0.


Remove DelayedTask

Became unused with c5e6798e4646587a5b33aa1afba581b20a2a1823

@matriv  please review

Current coverage is 81.14% (diff: 100%)

Merging #4535 into master will decrease coverage by 0.02%

@@             master      #4535   diff @@
==========================================
  Files          1293       1292     -1
  Lines         44153      44142    -11
  Methods           0          0
  Messages          0          0
  Branches       7012       7012
==========================================
- Hits          35836      35818    -18
+ Misses         5639       5631     -8
- Partials       2678       2693    +15

Powered by Codecov. Last update 266b14a...7b0ebd2


Code do not inherit environment variables from cmd

I expect that when I start vscode from a cmd that instance of visual studio code should inherit the environment variables from that cmd and no other. However, if I already have a vscode open from another cmd, with other environment variables the new vscode will have the same environment variables set as the one first opened. Even though they are started from cmd:s with different environment variables.
Does this issue occur when all extensions are disabled?: Yes

VS Code Version: 1.64.2
OS Version: windows 11

Steps to Reproduce:

Open new cmd
Run set foo=bar
Run code .
check that foo=bar in integrated terminal
Open fresch cmd
Check that foo is not set
Run code . in new cmd
Then will foo=bar in the new code instance
If I would have closed the first code window the second would not have foo=bar

When you launch the first VS Code instance the "main process" is created with its environment. All successive windows share this environment by design.


seperate out trigger controllers from kubeless code base/repo

Is this a BUG REPORT or FEATURE REQUEST?:
FEATURE REQUEST
Description:
Kubeless starting from 0.5.0 version has very decoupled architecture, in which functions and triggers can be independently deployed and operated. All aspects concerningFunctions are handled by function CRD controller. An event source is modelled as Kubeless Trigger. A Trigger is represented as CRD, and would need a CRD controller to be written. There is no strict requirement for Function controller and Trigger controller to be part of same code base. Integration point between the function and trigger controller is K8S api server, any common code between the controllers is minimum.
Purpose of the issue is to explore how Kubeless can enable writing independent trigger controllers. To start with both Kakfa, Nats trigger controllers need to be moved to any framework we come up with.
This opens up several possibilities.

We might end up having a triggers repo just like https://github.com/kubeless/functions
Also this will ease 3rd party triggers to be developed and released outside of the Kubeless project
one can build custom trigger controller for in-house events
trigger controllers can be polyglot

I created below PR opened against the Kafka trigger repo
https://github.com/kubeless/kafka-trigger/pull/3
Based on the feedback to the Kafka PR, i will raise the rest of the PR to avoid duplication of any potential feedback and effort.
Please take a look and comment.

https://github.com/kubeless/nats-trigger/pull/3
https://github.com/kubeless/kinesis-trigger/pull/2
PR's for Kinesis and Nats


can anyone give a complete tutorial on making the file to app

Hi @ARj-cyber , use android studio, it will build for you.
Thanks !


New package: hrzlgnm.mdns-browser version 0.6.3

Pull request has been created with komac v2.3.0 :rocket:
Microsoft Reviewers: Open in CodeFlow

@microsoft-github-policy-service agree

Automatic Validation ended with:

Installation failed with exit code -1978335226
2024-07-23 11:52:34.597 [CLI ] ShellExecute installer failed: 2

Automated error analysis suggests -1978335226 may mean Running ShellExecute failed
(Automated response - build 886.)

Verify:


Make Customer Endpoint optional in IPSEC tunnel config

Verified tunnel setup without specifying Customer endpoint IP in local setup
Description
In Magic Transit IPSEC Tunnel config API, the customer endpoint IP parameter is optional.
Has your change been tested?
Tested this with a private image. And verified that the tunnel setup is successful without specifying an IP address
Types of changes
What sort of change does your code introduce/modify?

Enhancement : which provides flexibility during tunnel creation.

thanks @jkunder 🥇


provider name needs to be longer to avoid collision

the provider name is appdynamics    so the resources and datasoruce names should start with appdynamics_ rather than appd_
When writing terraform seeing the full name appdynamics is much more descriptive and helpful to future maintainers than the appd_ prefix.

Yep good point, it has only recently been changed to appdynamcis.

Thanks for the issue! :D


about subscription

I subscribed but I did not receive the id and password in my e-mail, can you help me?

sorry, the domain hoster had a DNS failure which caused some emails to bounce, just emailed you from my personal account

Thanks for be interested
On Wed, Feb 8, 2023 at 9:17 PM dgtlmoon @.***> wrote:

sorry, the domain hoster had a DNS failure which caused some emails to
bounce, just emailed you from my personal account
—
Reply to this email directly, view it on GitHub
https://github.com/dgtlmoon/changedetection.io/issues/1401#issuecomment-1423048619,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A5YG6A3LMSSWYRLFN2D6BMLWWPPJ5ANCNFSM6AAAAAAUVQWV2A
.
You are receiving this because you authored the thread.Message ID:
@.***>

[image: image.png] can you help
On Wed, Feb 8, 2023 at 9:18 PM Asemaniko @.***> wrote:

Thanks for be interested
On Wed, Feb 8, 2023 at 9:17 PM dgtlmoon @.***> wrote:

sorry, the domain hoster had a DNS failure which caused some emails to
bounce, just emailed you from my personal account
—
Reply to this email directly, view it on GitHub
https://github.com/dgtlmoon/changedetection.io/issues/1401#issuecomment-1423048619,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A5YG6A3LMSSWYRLFN2D6BMLWWPPJ5ANCNFSM6AAAAAAUVQWV2A
.
You are receiving this because you authored the thread.Message ID:
@.***>

sorry i dont understand.

Got HTML content but no text found (With 200 reply code).  why am i getting
this error
how can i solve this error
I want to see the changes in the part I marked using the visual filter
selector
When I enable this setting, I encounter the error I wrote above.
is there any solution
thank you

please use the contact form at the bottom of https://changedetection.io


[2257] Enable BigQuery request logging

Context
We are sending web request events to BigQuery. This has been tested on QA and staging now and is ready for release.
Changes proposed in this pull request
Set the feature flag to true in staging and production.
Guidance to review
Checklist

[ ] Make sure all information from the Trello card is in here
[ ] Attach to Trello card
[ ] Rebased master
[ ] Cleaned commit history
[ ] Tested by running locally

 Send BigQuery events from TTAPI


fixing unit tests by wrapping writes to core setFailed, debug and warning

unit tests are currently failing to two calls to core debug, core set failed, core warning which print directly to the command line and are captured by the github runner.
By using jest spy on these methods and then checking that they were called using expect we can confirm functionality without triggering a runner failure.
the run method is executed automatically, so core execution logic is moved into bake.ts to allow it to be imported without triggering a run.
We need a unit tests to be passing again in order to be able to cut the next release too.

Any update on this ? as its probably blocking other PR's because those checks are failing.


Describe compiled css

In response to #24262

@XhmikosR: My God, I'm going crazy. So I will not be able to make any other contributions until I decide what I did in the master?

Reject this @XhmikosR. For today I lost my patience. Sorry.

It's pretty simple, no need to lose your cool :P
git fetch upstream
git checkout describe-compiled-css
git rebase -i upstream/v4-dev

and drop the unrelated patches.


Option --output not working with a manually launched context

I am trying to output test results in another directory with the --output option. But it seems to not be working : the default directory (test-results) contains test results at each run, even if I delete the folder.
Using pytest-playwright 0.3.0 .

I was trying it out locally and it works for me. How do you specify the command to pytest? Do you have a repository which you can share?

For example : pytest --output test_dir tests/admin/test_group_management.py
And sorry, I don't have a repo to share..
Pytest is up to date also (7.1.2)

Same here with something like:
$ pytest path/to/some/test_playwright.py --tracing=on --output pytest-tracing

even if adding or changing the tracing with screenshots or video or completely omitting the --output param the results is the same: no folder and no output

I created a couple of fixtures like that:
@pytest.fixture(scope="session")
def mypage(browser, session_cookies):
    context = browser.new_context(storage_state=session_cookies)
    return context.new_page()

@pytest.fixture(scope="session")
def reporting(mygenericpage, backend):
    myfeature = MyFeaturePage(
        page=mypage,
    )
    yield myfeature
    myfeature.close()

the test.py is irrelevant, it just take myfeature and does some action on myfeature.page.something()
looking at the code here I'd say that the params related to the artifacts (video, screenshot, tracing) are set only if the context feature is called, which is not in my case.
does it make sense to you?

I also modified browser context with ```
@pytest.fixture(scope="session")
def browser_context_args(browser_context_args):
return {
**browser_context_args,
"ignore_https_errors": True
}
Removing this fixtures enables --output option, ideed. Thank you !

It's intended that the --output only works with the given context and page fixture. If you manually launch a browser or create a browser context, --output will have no effect.

ok. we could add few documentation lines about it.
in theory it would be possibile to get artifacts even if the called fixture is browser, but it could be tricky because in a browser you can initialize more than one context.
@mxschmitt do you see this as a possibile improvement of the plugin?

Yeah, I think upstream we also support it.
In what artifacts are you exactly interested in? (video, screenshot, trace)?

I'd say everything is feasible 😄
An idea could be iterate over all the contexts defined in browser and print everything.
I could work a bit on this, if it can help

I do not understand why --output option is not compatible with a modified browser_context_args fixture. Is it possible to make them compatible ?

Finally I just used pytest --junit-xml instead of --output, it works like a charm

Yeh this is quite annoying tbh. It would be nice if it was fixed.

Suggested this fix for it https://github.com/microsoft/playwright-pytest/pull/118

Finally I just used pytest --junit-xml instead of --output, it works like a charm

I'm not sure about what you mean but --output provides a folder where the artifacts should be stored, --junit-xml is for the xunit xml file path.

Yes, sorry, --output and --junit-xml are completely different things.
Is there any update on this issue ? Modifying context and outputing artifacts should be possible no ?

I wonder whether there is plan to resolve this. We are trying to move from Selenium to playwright. Not being able to take screenshot as needed is the only step that blocking us from moving(#139).  In our Selenium framework, the screenshot is taken when test fails. The screenshot then being added to Allure report. It is very convenient to look at the screenshot on the Allure report page.  I am currently using pytest-bdd and Allure report too.
This is the reason I need to define context. context = browser.new_context(ignore_https_errors=True)

Hi,
Is there update or plan or work around for this issue?  I have paused the Playwright project for a couple of months because screenshot does not work with custom context.

I stumbled upon this issue but was able to solve it for me by modifying the browser_context_args:  you can configure the creation of context  by modifying the browser_context_args fixture, either per test (as shown in the docs) or by masking the fixture like here https://github.com/microsoft/playwright-pytest/pull/118#issuecomment-1171080654.
screenshots (--screenshot) and videos (--video) are only recorded automatically when using the provided context fixture, see https://github.com/microsoft/playwright-pytest/blob/main/pytest_playwright/pytest_playwright.py#L227.
Documenting this here as there seem to be some open questions about it.

It is worth noting that nothing in the documentation at https://playwright.dev/python/docs/test-runners#cli-arguments even hints that any of the CLI flags are dependent on using the default context fixture, or that the suggestion on the same page for modifying context to session-scope would be incompatible with one or more of those flags.
Worse, the --output flag DELETES the provided directory when (because of custom context) no artifacts are stored there.  That behavior is also undocumented.

This should be fixed in v0.5.0: https://pypi.org/project/pytest-playwright/

The issue is still there

test_results folder is not getting created with context. This issue still exists

Please re-file with more details, thank you. Make sure to use the new_context fixture in order to benefit from the artifact collection.
from playwright.sync_api import BrowserContext
from pytest_playwright.pytest_playwright import CreateContextCallback

def test_foo(new_context: CreateContextCallback) -> None:
    context = new_context()

````

We should probably improve the docs around it, I think the misconception is that you are using `browser.new_context` (assumption).


Not able to run on ubuntu 16.04

Hi Thanks for the awesome project!
I tried your quick start docker run as you mentioned in README.
docker run --name bind -d --restart=always \
  --publish 53:53/tcp --publish 53:53/udp --publish 10000:10000/tcp \
  --volume /srv/docker/bind:/data \
  sameersbn/bind:9.11.3-20180713

However, due to ubuntu systemd-resolved already occupied 53 on both udp and tcp container fail to start.
docker: Error response from daemon: driver failed programming external connectivity on endpoint bind (84714a9b44c7429f35e5a7ce44fcefe2931cfecea213209ad2fca2677f6c85c3): Error starting userland proxy: listen tcp <IP_ADDRESS>:53: bind: address already in use.

netstat -tulpn| grep 53
tcp        0      0 <IP_ADDRESS>:53           <IP_ADDRESS>:*               LISTEN      5261/systemd-resolv
udp    28672      0 <IP_ADDRESS>:5353            <IP_ADDRESS>:*                           820/avahi-daemon: r
udp     2304      0 <IP_ADDRESS>:53           <IP_ADDRESS>:*                           5261/systemd-resolv
udp6       0      0 :::35356                :::*                                820/avahi-daemon: r
udp6   34560      0 :::5353                 :::*                                820/avahi-daemon: r

I then try disable systemd-resolved.
netstat -tulpn| grep 53
udp    28672      0 <IP_ADDRESS>:5353            <IP_ADDRESS>:*                           820/avahi-daemon: r
udp6       0      0 :::35356                :::*                                820/avahi-daemon: r
udp6   34560      0 :::5353                 :::*                                820/avahi-daemon: r

docker container start just fine.
docker ps
CONTAINER ID        IMAGE                            COMMAND                  CREATED             STATUS              PORTS                                                              NAMES
04536fe2c60c        sameersbn/bind:9.11.3-20180713   "/sbin/entrypoint.sh…"   3 seconds ago       Up 3 seconds        <IP_ADDRESS>:53->53/tcp, <IP_ADDRESS>:10000->10000/tcp, <IP_ADDRESS>:53->53/udp   bind

I can see the port being open on host as well
bash-3.2# nmap <IP_ADDRESS>
Starting Nmap 7.70 ( https://nmap.org ) at 2018-08-08 22:12 CDT
Nmap scan report for <IP_ADDRESS>
Host is up (0.035s latency).
Not shown: 994 closed ports
PORT      STATE SERVICE
22/tcp    open  ssh
53/tcp    open  domain
10000/tcp open  snet-sensor-mgmt

However I still got refused.
host www.google.com <IP_ADDRESS>
Using domain server:
Name: <IP_ADDRESS>
Address: <IP_ADDRESS>#53
Aliases:

Host www.google.com not found: 5(REFUSED)

If there any steps I missed on configuring? or can anyone point out a good direction for me to troubleshoot?

can u describe how you have disabled the dnsmasq port?

Same problem here on 17.10. Documented steps here for how far I got and what I tried, in case it helps.
I stopped systemd-resolved and was able to start the container. However webmin wouldn't respond (using ip and port). Also I tried host www.google.com and got the following:
;; reply from unexpected source: 127.0.01#53, expected <IP_ADDRESS>#53
I edited /etc/resolv.conf (outside the container) and changed the nameserver to <IP_ADDRESS> and restarted the container. Then it could successfully resolve google inside the container but not outside. But webmin was still not responding. I edited /etc/webmin/miniserv.conf (inside the container) and changed ssl=1 to ssl=0 and that started working.
I followed the instructions here, and then I could resolve domains outside the container, but only on a different machine.
I have a Windows box at <IP_ADDRESS>, running Ubuntu under VirtualBox at <IP_ADDRESS>. I have multiple docker containers therein that have IP addresses in ranges 172.17.0.* and 172.30.0.*. I pointed my laptop DNS to <IP_ADDRESS> and VirtualBox/Docker works their magic to get port 53 to <IP_ADDRESS> (the bind container) such that host www.google.com works. Similarly it works from <IP_ADDRESS>. But it doesn't work from any of the other containers or the Ubuntu VM - it gets  Host www.google.com not found: 2(SERVERFAIL).
Then I found this to change the DNS Server for the Docker daemon. That didn't work either, but thinking about it, I didn't check it had an impact on the /etc/resolv.conf of the containers, so I should revisit. However, I ran out of time so I took a snapshot and restored.

Even better: disable systemd-resolved
sudo systemctl disable systemd-resolved
Source: http://manpages.ubuntu.com/manpages/xenial/man8/systemd-resolved.service.8.html

Hi guys. First of all I'm a complete ignoramus in networks. But somehow I've managed to get this docker container working. My settings are
Ubuntu Server 18.10
<IP_ADDRESS> - host
<IP_ADDRESS> - bind container

As @michaelblight mentioned you need to get /etc/resolv.conf edited, but you cannot do it directly (at least on 18.10) as changes will be overwritten (there is a disclaimer). So I appended
nameserver <IP_ADDRESS> to /etc/resolvconf/resolv.conf.d/head. I didn't experiment but there are also base and tail files. I believe the results will be similar just order of statements will differ in the resulting file. With head file modified update /etc/resolv.conf with sudo resolvconf -u.
As @BenKoerber suggested I modified docker-compose.yml a bit:
ports:
  - "<IP_ADDRESS>:10000:10000"
  - "<IP_ADDRESS>:53:53/tcp"
  - "<IP_ADDRESS>:53:53/udp"

That allowed me to keep systemd-resolved running, netstat -tulpn produced:
udp        0      0 <IP_ADDRESS>:53        <IP_ADDRESS>:*                           -
udp        0      0 <IP_ADDRESS>:53           <IP_ADDRESS>:*                           -

However I've found people complaining about systemd-resolved so you can disable it as @BenKoerber said. Also see this link (has extra steps for 17.04 Zesty).
And I think this is it. I could omit some details as after that I was struggling to get Nginx working and in the end could get confused by overwhelming info. And describing all the struggle would be off topic.


Feature Request: Java 8 Compatibility

The class file format is greater than what a lot of companies are still using (even in 2023!). When using the released artifact I get:

java.lang.UnsupportedClassVersionError: io/github/secretx33/resourceresolver/PathMatchingResourcePatternResolver has been compiled by a more recent version of the Java Runtime (class file version 55.0), this version of the Java Runtime only recognizes class file versions up to 52.0

Would it be possible to please release this using Java 8?

The library was ported to Java 11 from Spring Framework, which was using Java 14. A port to Java 8 is definitely possible, but I already gave that idea a shot when creating this library, and much code would have to be adapted.
I am unfortunately not in good health, and will not be able to engage on a massive port like this one any time soon.
Though, this project accept PRs, so if anyone is willing to do the port, I can re-release the library compiling to Java 8 too.

I'll pinning and closing this issue, thanks for the interest, and if anyone else is interested in performing this conversion, please open another issue and mention this one.


Unable to use kubectl from Rancher UI, Error from server (InternalError): an error on the server ("") has prevented the request from succeeding

What kind of request is this (question/bug/enhancement/feature request):
question
Steps to reproduce (least amount of steps as possible):
use kubectl from Rancher UI
when excute kubectl get all
Result:
Error from server (InternalError): an error on the server ("") has prevented the request from succeeding
Other details that may be helpful:

open ports:

# ---------------------------------------------------
# rancher
firewall-cmd --zone=public --add-port=22/tcp --permanent
firewall-cmd --zone=public --add-port=80/tcp --permanent
firewall-cmd --zone=public --add-port=443/tcp --permanent
# k3s server
firewall-cmd --zone=public --add-port=6443/tcp --permanent

firewall-cmd --reload

# ---------------------------------------------------
firewall-cmd --zone=public --add-port=22/tcp --permanent
firewall-cmd --zone=public --add-port=80/tcp --permanent
firewall-cmd --zone=public --add-port=443/tcp --permanent
firewall-cmd --zone=public --add-port=2376/tcp --permanent
firewall-cmd --zone=public --add-port=6443/tcp --permanent
firewall-cmd --zone=public --add-port=2379/tcp --permanent
firewall-cmd --zone=public --add-port=2380/tcp --permanent
firewall-cmd --zone=public --add-port=8472/udp --permanent
firewall-cmd --zone=public --add-port=9099/tcp --permanent
firewall-cmd --zone=public --add-port=10250-10252/tcp --permanent
firewall-cmd --zone=public --add-port=10256/tcp --permanent
firewall-cmd --zone=public --add-port=10254/tcp --permanent
firewall-cmd --zone=public --add-port=30000-32767/tcp --permanent
firewall-cmd --zone=public --add-port=30000-32767/udp --permanent
firewall-cmd --reload

when i stop firewalld, use kubectl from Rancher UI is success.
I found logs from Container  k8s_proxy_dashboard-shell-7m6sw_cattle-system_3ebd801d-60dc-497b-b104-7a389abf1b4b_0 ：
2021/01/18 03:12:15 http: proxy error: dial tcp: lookup kubernetes.default on <IP_ADDRESS>:53: read udp <IP_ADDRESS>:39804-><IP_ADDRESS>:53: read: no route to host
2021/01/18 03:12:25 http: proxy error: dial tcp: lookup kubernetes.default on <IP_ADDRESS>:53: read udp <IP_ADDRESS>:53600-><IP_ADDRESS>:53: read: no route to host
2021/01/18 03:12:40 http: proxy error: dial tcp: lookup kubernetes.default on <IP_ADDRESS>:53: read udp <IP_ADDRESS>:58280-><IP_ADDRESS>:53: i/o timeout

Environment information

Rancher version (rancher/rancher/rancher/server image tag or shown bottom left in the UI): 2.5.5
Installation option (single install/HA): HA

Cluster information

Cluster type (Hosted/Infrastructure Provider/Custom/Imported):Custom
Machine type (cloud/VM/metal) and specifications (CPU/memory):VM 4cpu 17GB memory
Kubernetes version (use kubectl version):

Client Version: version.Info{Major:"1", Minor:"19", GitVersion:"v1.19.6", GitCommit:"fbf646b339dc52336b55d8ec85c181981b86331a", GitTreeState:"clean", BuildDate:"2020-12-18T12:09:30Z", GoVersion:"go1.15.5", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"19", GitVersion:"v1.19.6", GitCommit:"fbf646b339dc52336b55d8ec85c181981b86331a", GitTreeState:"clean", BuildDate:"2020-12-18T12:01:36Z", GoVersion:"go1.15.5", Compiler:"gc", Platform:"linux/amd64"}

Docker version (use docker version):

Client: Docker Engine - Community
 Version:           19.03.14
 API version:       1.40
 Go version:        go1.13.15
 Git commit:        5eb3275d40
 Built:             Tue Dec  1 19:20:42 2020
 OS/Arch:           linux/amd64
 Experimental:      false

Server: Docker Engine - Community
 Engine:
  Version:          19.03.14
  API version:      1.40 (minimum version 1.12)
  Go version:       go1.13.15
  Git commit:       5eb3275d40
  Built:            Tue Dec  1 19:19:17 2020
  OS/Arch:          linux/amd64
  Experimental:     false
 containerd:
  Version:          1.4.3
  GitCommit:        269548fa27e0089a8b8278fc4fc781d7f65a939b
 runc:
  Version:          1.0.0-rc92
  GitCommit:        ff819c7e9184c13b7c2607fe6c30ae19403a7aff
 docker-init:
  Version:          0.18.0
  GitCommit:        fec3683

I got the same issue here. I check the logs of dashboard shell.
Starting to serve on <IP_ADDRESS>:8001
2021/01/19 03:31:37 http: proxy error: dial tcp: lookup kubernetes.default on <IP_ADDRESS>:53: read udp <IP_ADDRESS>:58728-><IP_ADDRESS>:53: read: connection refused
2021/01/19 03:31:47 http: proxy error: dial tcp: lookup kubernetes.default on <IP_ADDRESS>:53: read udp <IP_ADDRESS>:48122-><IP_ADDRESS>:53: read: connection refused
2021/01/19 03:32:02 http: proxy error: dial tcp: lookup kubernetes.default on <IP_ADDRESS>:53: read udp <IP_ADDRESS>:50617-><IP_ADDRESS>:53: read: connection refused
2021/01/19 03:32:12 http: proxy error: dial tcp: lookup kubernetes.default on <IP_ADDRESS>:53: read udp <IP_ADDRESS>:57199-><IP_ADDRESS>:53: read: connection refused
2021/01/19 03:32:27 http: proxy error: dial tcp: lookup kubernetes.default on <IP_ADDRESS>:53: read udp <IP_ADDRESS>:51843-><IP_ADDRESS>:53: i/o timeout

My other way is to install kubectl manually on host and config ~/.kube/config from Rancher UI. Then kubectl command works on host.
kubectl cluster-info

Kubernetes master is running at https://<IP_ADDRESS>:444/k8s/clusters/c-8l94s
CoreDNS is running at https://<IP_ADDRESS>:444/k8s/clusters/c-8l94s/api/v1/namespaces/kube-system/services/kube-dns:dns/proxy

To further debug and diagnose cluster problems, use 'kubectl cluster-info dump'.

I got the same issue here. I check the logs of dashboard shell.
Starting to serve on <IP_ADDRESS>:8001
2021/01/19 03:31:37 http: proxy error: dial tcp: lookup kubernetes.default on <IP_ADDRESS>:53: read udp <IP_ADDRESS>:58728-><IP_ADDRESS>:53: read: connection refused
2021/01/19 03:31:47 http: proxy error: dial tcp: lookup kubernetes.default on <IP_ADDRESS>:53: read udp <IP_ADDRESS>:48122-><IP_ADDRESS>:53: read: connection refused
2021/01/19 03:32:02 http: proxy error: dial tcp: lookup kubernetes.default on <IP_ADDRESS>:53: read udp <IP_ADDRESS>:50617-><IP_ADDRESS>:53: read: connection refused
2021/01/19 03:32:12 http: proxy error: dial tcp: lookup kubernetes.default on <IP_ADDRESS>:53: read udp <IP_ADDRESS>:57199-><IP_ADDRESS>:53: read: connection refused
2021/01/19 03:32:27 http: proxy error: dial tcp: lookup kubernetes.default on <IP_ADDRESS>:53: read udp <IP_ADDRESS>:51843-><IP_ADDRESS>:53: i/o timeout

My other way is to install kubectl manually on host and config ~/.kube/config from Rancher UI. Then kubectl command works on host.
kubectl cluster-info

Kubernetes master is running at https://<IP_ADDRESS>:444/k8s/clusters/c-8l94s
CoreDNS is running at https://<IP_ADDRESS>:444/k8s/clusters/c-8l94s/api/v1/namespaces/kube-system/services/kube-dns:dns/proxy

To further debug and diagnose cluster problems, use 'kubectl cluster-info dump'.

Moreover, I found the ~/.kube/config in dashboard shell is mounted on host :
...
        "Mounts": [
            {
                "Type": "bind",
                "Source": "/var/lib/kubelet/pods/9a6dc2d2-eb10-4964-948d-25f8183c9ba3/volume-subpaths/admin-kubeconfig/proxy/0",
                "Destination": "/root/.kube/config",
                "Mode": "ro",
                "RW": false,
                "Propagation": "rprivate"
            }
...

And that config is the default config for k8s:
sudo cat /var/lib/kubelet/pods/9a6dc2d2-eb10-4964-948d-25f8183c9ba3/volume-subpaths/admin-kubeconfig/proxy/0

apiVersion: v1
clusters:
- cluster:
    certificate-authority: /run/secrets/kubernetes.io/serviceaccount/ca.crt
    server: https://kubernetes.default
  name: cluster
contexts:
- context:
    cluster: cluster
    user: user
  name: default
current-context: default
kind: Config
preferences: {}
users:
- name: user
  user:
    as: user-b8chs
    as-groups:
    - system:authenticated
    - system:cattle:authenticated
    tokenFile: /run/secrets/kubernetes.io/serviceaccount/token

So maybe this config should be replaced with the generated config from Rancher UI ?
Or maybe we miss something when we are installing the cluster ?

Moreover, I found the ~/.kube/config in dashboard shell is mounted on host :
...
        "Mounts": [
            {
                "Type": "bind",
                "Source": "/var/lib/kubelet/pods/9a6dc2d2-eb10-4964-948d-25f8183c9ba3/volume-subpaths/admin-kubeconfig/proxy/0",
                "Destination": "/root/.kube/config",
                "Mode": "ro",
                "RW": false,
                "Propagation": "rprivate"
            }
...

And that config is the default config for k8s:
sudo cat /var/lib/kubelet/pods/9a6dc2d2-eb10-4964-948d-25f8183c9ba3/volume-subpaths/admin-kubeconfig/proxy/0

apiVersion: v1
clusters:
- cluster:
    certificate-authority: /run/secrets/kubernetes.io/serviceaccount/ca.crt
    server: https://kubernetes.default
  name: cluster
contexts:
- context:
    cluster: cluster
    user: user
  name: default
current-context: default
kind: Config
preferences: {}
users:
- name: user
  user:
    as: user-b8chs
    as-groups:
    - system:authenticated
    - system:cattle:authenticated
    tokenFile: /run/secrets/kubernetes.io/serviceaccount/token

So maybe this config should be replaced with the generated config from Rancher UI ?


[FEATURE REQUEST] Create Swagger documentation for Comments API

Is your feature request related to a problem? Please describe.
Documentation is required so that consumers of the API have a clear idea of how the API works and what is expected of them when using the API and its endpoints.
Describe the solution you'd like
Create documentation for each comments endpoint, its method, the required body, the response body, and the other possible responses.
Describe alternatives you've considered
None.
Additional context
For more information, see the Contribution documentation.

@simply-alliv We will appreciate if u can help us get this up. Thanks


Add a docker container definition

We can use this docker container definition to create docker containers, and debug the extension in it. Main purpose for now is to test the extension on Linux when using Windows, or develop the extension using Codespaces.

The CMake installed using this docker file is 3.14.5. We can change it to a newer/older version.


Memory hogger on Mac M1

This application very fast starts cosuming a large chunk of RAM, attaching screenshot
System Specification
Macbook Air (M1 2020)
Chip Apple M1
Memory 16 GB
Screenshot

Would you describe what you are doing after starting the application? Anything particular like opening a rust project or something along the line?

Would you describe what you are doing after starting the application? Anything particular like opening a rust project or something along the line?

I was playing around, opened a small rust project and typing some code realized some lag and frame drops. When analysed using Activity Monitor saw it was consuming more than 5gb of Ram.

One thing i noticed is if you installed rust plugin, then rust analyzer takes huge memory https://github.com/rust-analyzer/rust-analyzer/issues/5728 https://github.com/rust-analyzer/rust-analyzer/issues/1252 https://github.com/rust-analyzer/rust-analyzer/issues/9470 https://github.com/rust-analyzer/rust-analyzer/issues/11325 while indexing but after that memory usage get better

I'm also getting very high memory usage on Windows when typing (without rust analyzer). I can consistently reproduce this by opening a file and holding down a key on the keyboard to fill up the editor. Once a line is full, move to the next line and keep entering text. After a few lines, the memory usage will suddenly start to increase significantly and keep increasing linearly as the editor fills. Interestingly, the memory only increases while the text fits on the screen - if it starts scrolling horizontally then it stops increasing, but starts again when I move to the next line.

I get similar behavior on a first generation Mac M1 with 16G ram.  I created a folder with a single file in it, test1.txt. The contents are Testing Lapce.
Lapce takes 1.45G.  I add the line Hello, world! and memory climbs to 2G and slowly creeps up from there.

I am on Apple M1 Pro, MacOS 12.1.
This issue is not happen

Memory now should slowly raise based on the actual usage (typing, etc.)
Tested on M1

I also have about 300-400M memory usage after installing rust code analyzer. However I think this is still much larger than others, for example visual studio code. The bad thing is when you open a second window, memory requirement doubles, which is very strange to me because it is still a rust project, why consuming double memory?
Thanks,
Jianshu

Starting RA server consume tons of memory, they are still there even I force quit VSCode. The memory usage is crazy. I did not have the same issue on another small project, is it because there are so many crates used and they are pre-indexing so they consume so much ram? Is there an option to lazily index the necessary crate only if I need it?


Random Connection Reset by peer when accessing resources on S3

HI David
We're using your library to avoid having to pull in half of the internet (ie AWS SDK v2) into our apps. However, we've been getting random "Connection reset by peer" errors when trying to pull resources from S3. Wondered if you had any ideas ?

How big are these resources? Does sound peculiar.

I'm open to adding some retry logic. The official AWS SDK products have that feature: https://docs.aws.amazon.com/sdkref/latest/guide/feature-retry-behavior.html

Hi Dave,
These are all tiny resources < 1k. They are just config files for the apps
we run.
regards,
Graeme
On Mon, Mar 13, 2023 at 2:02 PM Dave Moten @.***> wrote:

How big are these resources? Does sound peculiar.
—
Reply to this email directly, view it on GitHub
https://github.com/davidmoten/aws-lightweight-client-java/issues/100#issuecomment-1466952505,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAUJNBB2PQG4FUZJ4D5QWHTW36DPHANCNFSM6AAAAAAVZRDOBM
.
You are receiving this because you authored the thread.Message ID:
@.***>

--
Graeme Wallace
CTO
3Victors Inc.
M: +1<PHONE_NUMBER>

I'll have a look at adding retry logic.

What http status code do you get?

and/or do you get a stack trace?

Hi Dave - No status code as the socket is getting closed by the server
before anything comes back.
So its all the way down in HttpClientDefault.request() where it gets
java.io.UncheckedIOException: java.io.IOException: Connection reset by peer
The interesting thing is - today we're not getting any of those errors
at all - its like yesterday there was a bad HTTP endpoint we were
talking to and it was randomly closing the connection.
regards,
Graeme
On Tue, Mar 14, 2023 at 12:39 AM Dave Moten @.***>
wrote:

and/or do you get a stack trace?
—
Reply to this email directly, view it on GitHub
https://github.com/davidmoten/aws-lightweight-client-java/issues/100#issuecomment-1467550352,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAUJNBHBNCIJSH2M3U5SYSTW4AODLANCNFSM6AAAAAAVZRDOBM
.
You are receiving this because you authored the thread.Message ID:
@.***>

--
Graeme Wallace
CTO
3Victors Inc.
M: +1<PHONE_NUMBER>

Righto, thanks. I've written the retry logic (configurable at client level or request level) and is looking good. Multipart upload already had retry logic so I'll look to make that consistent and then get test coverage back up to 100%. There'll be a release in the next day or two. Glad that your problem appears to be transient but we'll make sure you are sorted for it.

Thanks Dave !
On Tue, Mar 14, 2023 at 1:19 PM Dave Moten @.***> wrote:

Righto, thanks. I've written the retry logic (configurable at client level
or request level) and is looking good. Multipart upload already had retry
logic so I'll look to make that consistent and then get test coverage back
up to 100%. There'll be a release in the next day or two. Glad that your
problem appears to be transient but we'll make sure you are sorted for it.
—
Reply to this email directly, view it on GitHub
https://github.com/davidmoten/aws-lightweight-client-java/issues/100#issuecomment-1468780913,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAUJNBDI6I7ILROUSV527CDW4DHGTANCNFSM6AAAAAAVZRDOBM
.
You are receiving this because you authored the thread.Message ID:
@.***>

--
Graeme Wallace
CTO
3Victors Inc.
M: +1<PHONE_NUMBER>

Release 0.1.15 is on Maven Central with retry logic. Please give it a try. Documentation is at https://github.com/davidmoten/aws-lightweight-client-java#retries.


Optimize LeastActiveLoadBalance and add weight test case.

Optimize LeastActiveLoadBalance and add weight test case.
related pr to master:
https://github.com/apache/incubator-dubbo/pull/2172
related issue:
https://github.com/apache/incubator-dubbo/issues/2540

Codecov Report

Merging #2584 into 2.6.x will increase coverage by 0.11%.
The diff coverage is 100%.

@@             Coverage Diff              @@
##              2.6.x    #2584      +/-   ##
============================================
+ Coverage     47.41%   47.52%   +0.11%
- Complexity     4291     4303      +12
============================================
  Files           557      557
  Lines         24433    24433
  Branches       4349     4349
============================================
+ Hits          11584    11613      +29
+ Misses        11054    11023      -31
- Partials       1795     1797       +2

Impacted Files
Coverage Δ
Complexity Δ

...pc/cluster/loadbalance/LeastActiveLoadBalance.java
88.57% <100%> (+17.14%)
9 <0> (+3)
:arrow_up:

...onfig/spring/extension/SpringExtensionFactory.java
84.61% <0%> (+11.53%)
8% <0%> (ø)
:arrow_down:

...rpc/cluster/loadbalance/RoundRobinLoadBalance.java
88.09% <0%> (+30.95%)
9% <0%> (+5%)
:arrow_up:

...bbo/rpc/cluster/loadbalance/RandomLoadBalance.java
88.88% <0%> (+38.88%)
8% <0%> (+4%)
:arrow_up:

Continue to review full report at Codecov.

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


Can Batch Work Run in Reactive Mode

AC:

https://flink.apache.org/2021/05/06/scaling-flink-automatically-with-reactive-mode/
do the things to prove it out

more relevant: https://nightlies.apache.org/flink/flink-docs-master/docs/deployment/elastic_scaling/#adaptive-batch-scheduler

answer is no


c_pydantic

https://github.com/microsoft/DeepSpeed/blob/cc67f22f60f7ec20c61a53b4c3da4e8799d7de5a/requirements/requirements-readthedocs.txt#L1-L2

@Spiesj86, can you please share detail of the issue?


[1.12] Mutes two flaky tests till docker version is upgraded to 17.12

High-level description
Mutes two flaky tests till docker version is upgraded to 17.12
Corresponding DC/OS tickets (obligatory)
These DC/OS JIRA ticket(s) must be updated (ideally closed) in the moment this PR lands:

DCOS-19790 Flaky Test For DCOS Release: test_networking.test_if_overlay_ok
DCOS-41896 Flaky test - test_networking test_ipv6 True False

Checklist for all PRs

[ ] Added a comprehensible changelog entry to CHANGES.md or explain why this is not a user-facing change:
[ ] Included a test which will fail if code is reverted but test is not. If there is no test please explain here:
[ ] Read the DC/OS contributing guidelines
[ ] Followed relevant code rules Rules for Packages and Systemd

Checklist for component/package updates:
If you are changing components or packages in DC/OS (e.g. you are bumping the sha or ref of anything underneath packages), then in addition to the above please also include:

[ ] Change log from the last version integrated (this should be a link to commits for easy verification and review): example
[ ] Test Results: [link to CI job test results for component]
[ ] Code Coverage (if available): [link to code coverage report]

PLEASE FILL IN THE TEMPLATE ABOVE / DO NOT REMOVE ANY SECTIONS ABOVE THIS LINE
Instructions and review process
What is the review process and when will my changes land?
All PRs require 2 approvals using GitHub's pull request reviews.
Reviewers should be:

Developers who understand the code being modified.
Developers responsible for code that interacts with or depends on the code being modified.

It is best to proactively ask for 2 reviews by @mentioning the candidate reviewers in the PR comments area. The responsibility is on the developer submitting the PR to follow-up with reviewers and make sure a PR is reviewed in a timely manner. Once a PR has 2 ship-it's, no red reviews, and all tests are green it will be included in the next train.

@mesosphere-mergebot bump-ee

fast-merging to get a number of CI consolidations in today

@mesosphere-mergebot bump-ee


german translation contains incorrect/misleading information

Hello,
in the german version, in the table , the english text "New label encryption is applied" is currently translated as "Es wird keine neue Verschlüsselung mit Bezeichnung angewendet". The meaning is really the opposite of the english text, please change to something like "Neue Verschlüsselung wird angewendet".
Best regards,
Andreas

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

ID: 7dd3497a-c02a-5fd6-12db-3a093f0fe17c
Version Independent ID: 2ef81556-61b0-47ae-d8fc-3d9aa5240c51
Content: Restrict access to content using sensitivity labels to apply encryption - Microsoft 365 Compliance
Content Source: microsoft-365/compliance/encryption-sensitivity-labels.md
Service: o365-seccomp
GitHub Login: @cabailey
Microsoft Alias: cabailey

@TinaMcN Please help us in resolving this issue. Thanks

@andreaseliasnts thanks for reporting the issue, I'm working to get it fixed in the German article.

@andreaseliasnts the translation has been fixed and changed to "Neue Verschlüsselung der Bezeichnung wird angewendet". https://docs.microsoft.com/de-de/microsoft-365/compliance/encryption-sensitivity-labels?view=o365-worldwide#how-to-configure-a-label-for-encryption   Thanks again for letting us know about this.

@TinaMcN translation is now ok, thanks for the fast fix :) !


cannot open admin console. Error: Client.Timeout exceeded while awaiting headers

I have a kots application installed on a cluster. The admin console pod is running and healthy.
If i run
kubectl kots admin-console --namespace kots-app --port 3000

I get
Error: failed to port forward: failed to query healthz: Get "http://localhost:3000/healthz": context deadline exceeded (Client.Timeout exceeded while awaiting headers)

But if i open a portforward to the kotsadm pod at 3000 and curl http://localhost:3000/healthz, I get the following response
{"version":"v1.92.1","gitSha":"test","status":{"database":{"connected":true},"storage":{"available":true}}}

Don't know how to solve this issue, any help would be greatly appreciated

Hi @kushal-ti I am starting to look into this. In the meantime can you generate a support bundle and share it here please? This will give me more information about the current state of the cluster to continue troubleshooting. Thank you

Hi. I know this is not exactly helpful, but this problem was solved when i switched environments. I was initially trying it on a local mac, which failed as described above, then tried the same thing from an online IDE (gitpod) and it worked. I'm not sure what the bug was, but if i had to guess, there was some kind of misconfiguration with the networking settings on my local machine, not something inherent to the replicated app itself

No problem, thanks for the update. We're going to close this out but please report back if you run into this again.


Fix Mobile Version

Add function to close the Navbar toggle on click.
Fix CSS of about.html page for mobile version
@AM1CODES, it won't break the code at all.. Please look into it.

Can you brief me a little more on it? Because on mobile it seems to work perfectly. So i am not sure what changes did you make.

On mobile version, when we click on any menu button, the the navbar toggle doesn't close automatically.
So, I fixed that to close automatically on click.

Yes understood. The code looks good to me. I will merge it. Thanks for contributing.

@AM1CODES Did you close without merging?

Yes my bad. I am so sorry for the triuble. Could you please resend the PR. I closed it by mistake.

Once again i apologize for all the trouble

Merged.

I was in a hurry so closed it by mistake. Sorry dude.

Once again i apologize for all the trouble

Hehe..
No worries
Thanks buddy.


How to use my model instead of 3dBBox in Objectron?

I would like to substitute the predicted 3d bounding box with a shoe model as you showed in the paper for the try on application.
I understood that I have to modify the GLAnimationOverlayCalculator in some ways but I still don't understand how I could do.
I have been struggling with mediapipe for 2 months and I finally start to understand the basics but most of it is still obscure. I hope someone can help me

Hi @UX3D-mazzini,  This https://ai.googleblog.com/2020/03/real-time-3d-object-detection-on-mobile.html gives you overview of your usecase. Thanks

Thanks for the response, I managed to show the shoes on my feet, but the model is too small, how can I make the model length fit the 3dBbox length in order to make them of the right size?

The model size is read from here


Complied lib files is missing from the TypeScript compilation. Please make sure it is in your tsconfig via the 'files' or 'include' property.

Bug Report or Feature Request (mark with an x)
- [X] bug report -> please search issues before submitting
- [ ] feature request

Versions
Angular CLI: 6.0.8
Node: 8.11.1
OS: win32 x64
Angular: 6.0.4
... animations, common, compiler, compiler-cli, core, forms
... http, language-service, platform-browser
... platform-browser-dynamic, router
Package                            Version
@angular-devkit/architect          0.6.8
@angular-devkit/build-angular      0.6.8
@angular-devkit/build-ng-packagr   0.6.8
@angular-devkit/build-optimizer    0.6.8
@angular-devkit/core               0.6.8
@angular-devkit/schematics         0.6.8
@angular/cdk                       6.2.1
@angular/cli                       6.0.8
@angular/flex-layout               6.0.0-beta.16
@angular/material                  6.2.1
@ngtools/json-schema               1.1.0
@ngtools/webpack                   6.0.8
@schematics/angular                0.6.8
@schematics/update                 0.6.8
ng-packagr                         3.0.1
rxjs                               6.2.0
typescript                         2.7.2
webpack                            4.8.3
Repro steps
https://github.com/gatimus/lazy-lib
ng build shared
ng build lazy-lib
The log given by the failure
ERROR in ./dist/shared/routing/error/shared-routing-error.d.ts
Module build failed: Error: C:\Users\User\source\lazy-lib\dist\shared\routing\error\shared-routing-error.d.ts is missing from the TypeScript compilation. Please make sure it is in your tsconfig via the 'files' or 'include' property.
at AngularCompilerPlugin.getCompiledFile (C:\Users\User\source\lazy-lib\node_modules@ngtools\webpack\src\angular_compiler_plugin.js:709:23)
at plugin.done.then (C:\Users\User\source\lazy-lib\node_modules@ngtools\webpack\src\loader.js:41:31)
at
ERROR in ./dist/shared/routing/not-found/shared-routing-not-found.d.ts
Module build failed: Error: C:\Users\User\source\lazy-lib\dist\shared\routing\not-found\shared-routing-not-found.d.ts is missing from the TypeScript compilation. Please make sure it is in your tsconfig via the 'files' or 'include' property.
at AngularCompilerPlugin.getCompiledFile (C:\Users\User\source\lazy-lib\node_modules@ngtools\webpack\src\angular_compiler_plugin.js:709:23)
at plugin.done.then (C:\Users\User\source\lazy-lib\node_modules@ngtools\webpack\src\loader.js:41:31)
at
Desired functionality
build with out error

+1

I have the same error, but after my test, this error happens when I try to make lazy loading for the library, when the name of the library contains "-" (minus character).
In my example, there is no problem by loading library "dynamic2". The error comes only after I have added another library "dynamic-simple".
https://github.com/eastjie/dynamic-module-imports-loading-with-the-angular-cli
In compiler,  the lazy loaded library has name consisting of "namespace-library name". I don't know if there is something wrong in getCompiledFile() from webpack by parsing path in the case.

Date: 2018-08-03T13:11:58.563Z
Hash: 089fcb074e0789c02b3b
Time: 11867ms
chunk {dynamic-dynamic-module} dynamic-dynamic-module.js, dynamic-dynamic-module.js.map (dynamic-dynamic-module) 5.35 kB  [rendered]
chunk {0} 0.js, 0.js.map () 723 bytes  [rendered]
chunk {main} main.js, main.js.map (main) 25.2 kB [initial] [rendered]
chunk {polyfills} polyfills.js, polyfills.js.map (polyfills) 227 kB [initial] [rendered]
chunk {runtime} runtime.js, runtime.js.map (runtime) 8.1 kB [entry] [rendered]
chunk {sample-dynamic-simple} sample-dynamic-simple.js, sample-dynamic-simple.js.map (sample-dynamic-simple) 1.22 kB  [rendered]
chunk {sample-dynamic2} sample-dynamic2.js, sample-dynamic2.js.map (sample-dynamic2) 4.47 kB  [rendered]
chunk {styles} styles.js, styles.js.map (styles) 15.6 kB [initial] [rendered]
chunk {vendor} vendor.js, vendor.js.map (vendor) 3.5 MB [initial] [rendered]
ERROR in ./projects/dynamic-simple/src/lib/dynamic-simple.module.ts
Module build failed: Error: C:\git_hub_fork\dynamic-module-imports-with-the-angular-cli\projects\dynamic-simple\src\lib\dynamic-simple.module.ts is missing from the TypeScript compilation. Please make sure it is in your tsconfig via the 'files' or 'include' property.
at AngularCompilerPlugin.getCompiledFile (C:\git_hub_fork\dynamic-module-imports-with-the-ngular-cli\node_modules@ngtools\webpack\src\angular_compiler_plugin.js:714:15)
at plugin.done.then (C:\git_hub_fork\dynamic-module-imports-with-the-angular-cli\node_modules@ngtools\webpack\src\loader.js:41:31)
at
at process._tickCallback (internal/process/next_tick.js:188:7)

I have the same problem. Can man add file to some setting in "tsconfig.json" as workround to solve this problem temporarily?
Thanks.

+1

+1

+1

+1

+1

+1

+1

Still broken on v8. Any hack or workaround very much welcome.

After updating to 9 this has gone away. I might have changed something in the configuration files as well but can't really point to anything specific. I'd generally recommend the update to 9, it solved a lot of problems I was having in a workspace with multiple libaries and lazy loading.


WRO-2087: Replaced process.env usage from i18n/Loader and glue

Checklist

[x] I have read and understand the contribution guide
[ ] A CHANGELOG entry is included
[ ] At least one test case is included for this feature or bug fix
[x] Documentation was added or is not needed
[ ] This is an API breaking change

Issue Resolved / Feature Added
Webpack 5 does no longer include a polyfill for this Node.js variable so it's recommended to add the environment variables to DefinePlugin.
Resolution
Replaced process.env access to embedded environment variables
Additional Considerations
This change needs cil, dev-utils prs and change bb file to use new pack option --ilib-additional-path.
Links
WRN-18582
Comments

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.9 out of 10 committers have signed the CLA.:white_check_mark: MikyungKim:white_check_mark: jeonghee27:white_check_mark: 0x64:white_check_mark: alexandrumorariu:white_check_mark: sjro:white_check_mark: stanca-pop-lgp:white_check_mark: juwonjeong:white_check_mark: seunghoh:white_check_mark: daniel-stoian-lgp:x: enact-adminYou have signed the CLA already but the status is still pending? Let us recheck it.


#trivial added user to newdeploy on the cli

          1 Error

:no_entry_sign:
Please include a CHANGELOG entry.

          1 Warning

:warning:
PR is classed as Work in Progress

  Generated by :no_entry_sign: Danger

This has been tested now, should be ready to go.

@lachlanh LG, not sure if there is a way to write a unit test for this, otherwise can be tested on the server side.   Also is this stil WIP, Danger warns of merging because of that flag, assumed not since your last email.

@JC1738 @nyarly can I get this one merged ? danger warning is before I marked it trivial. Not sure how to rerun danger without committing anything ?


Update traffic-analytics.md

Added spaces after certain region names as it was leading to formatting issue with wrapped text in final doc

@Harsha-CS : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#sign-off

@Harsha-CS: I'm sorry - only the author of this article, @damendo, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.

Looks good; Thanks, @Harsha-CS.

@Court72 , Some regions are still merged into a single line e.g. "Brazil Southeast Canada Central"  in final view on public docs.
I tried to adjust the indentation on each line by providing extra spaces at the end of text. This issue still persists and the preview mode does not display the right view and instead displays it similar to edit mode. Therefore I cant make out if the right indentation and new line is considered at the time of edit ? Making changes and waiting for doc to go-live is leading to long round trip time.
Would you be able to guide/help on how to fix this?

Hi @Harsha-CS,
I just checked your updates on the live page. Here's an example:

Let me know if your changes aren't showing up here!
Thanks.

@Court72 , there seems to be a difference in behavior depending on browser window size. FYI I have a 27' monitor.
When window is on full screen mode there is incorrect indentation (Brazil Southeast)

When window occupies half screen, indentation is correct. It also works accurately on laptop (window in full screen mode)

@Harsha-CS
Ah - I see! Screen size does affect how those columns display.
You can submit another pull request to adjust the spacing. In that pull request, leave a comment about how the columns display differently depending on screen size and request that your commits be pulled into the private repo so that they can be tested on staging.


As a server admin, I can limit nodes connecting to me via authentication.

https://github.com/Zhipeng-Chang/CMPUT404-Winter-2019-GroupProject/pull/112


check if date is valid after validation

Thank you for the PR Diego.
Can you write a test case that cover this bug?
Take a look at the test file for the date mask date-spec.js

The base code has diverged.
Please update your patch.

Closing due to inactivity, please reopen if this is still an issue and after updating to the latest code base!


Google Pixel missing

Apologies if I'm going blind but I don't see the Pixel or Pixel XL in your new definitions.
cheers
James

You're not going blind! I mistakenly updated my fork of the webvr-polyfill but did not add those changes here. I have corrected that now. Sorry for the confusion.


Clear the error log notifications on successful transpilation.

Not sure I understand the issue.

The error notifications that I get when have some wrong code don't go away after I fix them. I have to click on the close button to close them. I am hoping if my code transpiles successfully then those error messages can be closed automatically since they are no longer probably relevant.

Thanks for this improvement.


Adding Bangladesh University domain bu.edu.bd

Adding Bangladesh University
Domain is bu.edu.bd
Computer Science Department Official Page for 4 year course curriculum is https://cse.bu.edu.bd/curriculum

@shafqat-a Thank you for your patience while waiting for a review and processing your request. Pull request merged. Please start applying for JetBrains educational subscription at https://www.jetbrains.com/shop/eform/students in an hour, to allow the changes to propagate through JetBrains systems.


Optimize collections get/set

Summary or problem description
Optimize these kind of set in collections:
a = [1, 3, 4, 6]
a[0] = 2

to the following:
a = [2, 3, 4, 6]

For getters, use the value instead of converting using Opcode.PICKITEM, similar to #118

https://app.clickup.com/t/2ewf1ma


Kafka client is not connected to AWS MSK service

Environment

Node version:10.15.0
Kafka-node version:4.1.3
Kafka version:2.1.0

Sample Code to reproduce behavior
const cred = {
    kafkaHost: "xyz1.amazonaws.com:9094,xyz2.amazonaws.com:9094,xyz3.amazonaws.com:9094",
    connectTimeout: 100
};

var kafka = require('kafka-node'),
    Producer = kafka.Producer,
    KeyedMessage = kafka.KeyedMessage,
    client = new kafka.KafkaClient(cred),
    producer = new Producer(client),
    km = new KeyedMessage('key', 'message'),
    payloads = [
        { topic: 'AWSKafkaTutorialTopic', messages: 'hi' }
    ];

producer.on('ready', function () {
    producer.send(payloads, function (err, data) {
        console.log("data >>", data);
    });
});

producer.on('error', function (err) {
    console.log("err >>", err)
});

Output

Can't connect from local development environment

+1

+1

+1


RangeError: Invalid time value

Link to reproduction
No response
Describe the Bug
I found a use case that makes the Payload application crash even when refreshing. It is related with the filters when switching from filter with a text input field to a date input field (ID -> Updated At for instance)
I think this requires some attention cause it's a use case that could potentially happen often.
To Reproduce
The steps on how to reproduce the bug :

Select a collection and go to the list view
Add a filter
Filter through a text field (ID for instance)
Put any operator (is equal, is not equal, etc.)
Enter a a value
Switch the filtered field to any date type field (Updated At for instance)
Watch the application crash. You'll end up with a white screen

The error comes from having the previous entered value not being valid as a date format because the proper field of Updated At is a calendar
To fix this issue, I think a simple solution to fix this is to simply reset the value field to empty string every time a filter property is changed
Payload Version
1.8.2
Adapters and Plugins
No response

Hey @PatrikKozak — please get this able to be reproduced with a debugger, and let me know when you have it pulled up. I can try and help identify.