help output doesnt seem to be working properly in spyder new version(3.2.3)

Description
What steps will reproduce the problem?

why not in spyder new verion(3.2.3), help is bot working properly? we are getting incomplete info about any objects we uery...whats its remedy..i have python 3.6 installed

What is the expected output? What do you see instead?
Please provide any additional information below
Version and main components

Spyder Version: 3.2.3
Python Version: 3.6.2
Qt Versions: 5.6.2, PyQt5 5.6 on Windows

Dependencies
pyflakes >=0.6.0 :  1.5.0 (OK)
pycodestyle >=2.3:  2.3.1 (OK)
pygments >=2.0   :  2.2.0 (OK)
pandas >=0.13.1  :  0.20.3 (OK)
numpy >=1.7      :  1.13.1 (OK)
sphinx >=0.6.6   :  1.6.3 (OK)
rope >=0.9.4     :  0.10.5 (OK)
jedi >=0.9.0     :  0.10.2 (OK)
nbconvert >=4.0  :  5.3.1 (OK)
sympy >=0.7.3    :  1.1.1 (OK)
cython >=0.21    :  0.26.1 (OK)
qtconsole >=4.2.0:  4.3.1 (OK)
IPython >=4.0    :  6.1.0 (OK)
pylint >=0.25    :  1.7.2 (OK)

Help works differently when call from the Editor and the Console. This is not Spyder's fault, but a limitation of the libraries we use to get help in the Editor.
Closing because there's nothing we can do about it, sorry.

spyder new verion(3.2.3)

That version is over half a year old, FYI. Please update to the latest (3.2.8) via conda update spyder from the Anaconda Prompt.

Carlos,
surely i will update but will that be having any adverse impact to other
Anaconda dependencies? ..I have had terrible experience (while updating
spyder through anaconda ) to the level where i had to uninstall it and then
reinstalled..After having sypder updated then,Jupyter application started
misbehaving pretty badly..
revert awaited,
Jalesh
On Mon, Apr 9, 2018 at 9:04 AM, C.A.M. Gerlach<EMAIL_ADDRESS>wrote:

spyder new verion(3.2.3)
That version is over half a year old, FYI. Please update to the latest
(3.2.8) via conda update spyder from the Anaconda Prompt.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/spyder-ide/spyder/issues/6900#issuecomment-379619845,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AkSdSoKyZ3Y4AkkikRo6JjY3-a_fDP3yks5tmtaugaJpZM4TIFu7
.

--
With Warm Regards,
Jalesh

surely i will update but will that be having any adverse impact to other Anaconda dependencies

It shouldn't, particularly not if you do it in a separate conda environment; that's what those are for. You possibly should just update anaconda as a whole first, since a lot of your other stuff is likely out of date as well.

I have had terrible experience (while updating spyder through anaconda ) to the level where i had to uninstall it and then reinstalled

Most likely because something is messed up with your install (e.g. using pip to install something); conda updates should be close to as smooth as one can get. I've updated Spyder dozens of times on many different machines and environment and never experienced a single issue, though YMMV.

if you do it in a separate conda environment;  ----- does it mean i have
to get some other OS version installed in other PC..and then check its
performance?
On Tue, Apr 10, 2018 at 4:00 AM, C.A.M. Gerlach<EMAIL_ADDRESS>wrote:

surely i will update but will that be having any adverse impact to other
Anaconda dependencies
It shouldn't, particularly not if you do it in a separate conda
environment; that's what those are for. You possibly should just update
anaconda as a whole first, since a lot of your other stuff is likely out
of date as well.
I have had terrible experience (while updating spyder through anaconda )
to the level where i had to uninstall it and then reinstalled
Most likely because something is messed up with your install (e.g. using
pip to install something); conda updates should be close to as smooth as
one can get. I've updated Spyder dozens of times on many different machines
and environment and never experienced a single issue, though YMMV.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/spyder-ide/spyder/issues/6900#issuecomment-379913851,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AkSdShJN81_xpdZKyaatSyzGPCurEmAYks5tm-EUgaJpZM4TIFu7
.

--
With Warm Regards,
Jalesh

Thankx for sharing the link...
could u please also share me free comprehensive video tutorial links for R
language?
would be grateful.
On Fri, Apr 13, 2018 at 7:50 PM, C.A.M. Gerlach<EMAIL_ADDRESS>wrote:

@borntospark0 https://github.com/borntospark0
if you do it in a separate conda environment; ----- does it mean i have to
get some other OS version installed in other PC..and then check its
performance?
Umm, no; the whole point of conda environments is to prevent you having to
do that. I suggest you read the docs
https://conda.io/docs/user-guide/tasks/manage-environments.html to
learn more.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/spyder-ide/spyder/issues/6900#issuecomment-381150456,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AkSdSgAtFH48giQRGrCtPIcVgyrM75itks5toLQ8gaJpZM4TIFu7
.

--
With Warm Regards,
Jalesh

could u please also share me free comprehensive video tutorial links for R language?

Here you go...
¯_(ツ)_/¯

Thankx Team a ton, will go through it..
On Sat, Apr 14, 2018 at 8:54 PM, C.A.M. Gerlach<EMAIL_ADDRESS>wrote:

could u please also share me free comprehensive video tutorial links for R
language?
Here you go...
http://lmgtfy.com/?q=free+comprehensive+video+tutorial+links+for+R+language
¯_(ツ)_/¯
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/spyder-ide/spyder/issues/6900#issuecomment-381336738,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AkSdSkYOGxA9Q5Pt1i2UqRHUk_Fieb_Nks5tohTDgaJpZM4TIFu7
.

--
With Warm Regards,
Jalesh

Since it might not have been clear, the above link was meant to be a gentle reminder akin to the phrase "Let me Google that for you" (LMGTFY), i.e. one could have gotten a quicker answer just Googling the given phrase oneself rather than us doing it for you. This is a particularly acute case since this is the bug tracker for Spyder, a Python IDE, not a help forum for R, which would be a far more appropriate place to ask the question above if Google couldn't find a good result.

i went through the link provided by you through google...but the videos are
of short duration...cant there be a comprehensive  FREE 10+hours of video
tutorials for R..
if answer is YES, request you to provide me that link so that i could take
great help from both it and you :)
On Sun, Apr 15, 2018 at 6:32 PM, Jalesh J<EMAIL_ADDRESS>wrote:

Thankx Team a ton, will go through it..
On Sat, Apr 14, 2018 at 8:54 PM, C.A.M. Gerlach<EMAIL_ADDRESS>wrote:

could u please also share me free comprehensive video tutorial links for
R language?
Here you go...
http://lmgtfy.com/?q=free+comprehensive+video+tutorial+links+for+R+language
¯_(ツ)_/¯
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/spyder-ide/spyder/issues/6900#issuecomment-381336738,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AkSdSkYOGxA9Q5Pt1i2UqRHUk_Fieb_Nks5tohTDgaJpZM4TIFu7
.

--
With Warm Regards,
Jalesh

--
With Warm Regards,
Jalesh

@borntospark0, this is a place to help people about Spyder errors, not to provide help about random stuff.

@borntospark0 I'm sorry, but it is evident the major message of my last two posts has not been received, so I'll abandon overt politeness for the sake of utmost clarity.
This is the bug tracker for the Spyder Python IDE, and is not a help forum, not about a programming language and is not in any way about anything R. Therefore, your question is completely unrelated to both the purpose and the topic of this site, and should be taken elsewhere, as it is wasteful of the time of all involved to continue this further. Thank you.

No Worries...even i would take this opportunity for thanking u in having
extended support wrt Spyder and anaconda..
in hindsight, if u wudnt have clarified as above...i wouldnt have
religiously searched for R course and found the one which i was exactly
looking up..thankx pal for pumping me up!!!
take care...
On Tue, Apr 17, 2018 at 3:46 AM, C.A.M. Gerlach<EMAIL_ADDRESS>wrote:

@borntospark0 https://github.com/borntospark0 I'm sorry, but it is
evident the major message of my last two posts has not been received, so
I'll abandon overt politeness for the sake of utmost clarity.
This is the bug tracker for the Spyder Python IDE, and is not a help
forum, not about a programming language and is not in any way about
anything R. Therefore, your question is completely unrelated to both the
purpose and the topic of this site, and should be taken elsewhere, as it is
wasteful of the time of all involved to continue this further. Thank you.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/spyder-ide/spyder/issues/6900#issuecomment-381767633,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AkSdShuPGltq5BMidaiKA2cCxfDVAC_Mks5tpRgqgaJpZM4TIFu7
.

--
With Warm Regards,
Jalesh

Thanks, and glad you found what you were looking for, Best of luck.

hi team,
Kindly help me out as to why i am unable to plot by using plotly in
anaconda distribution?
below is the code snippet in jupyter notebook
import numpy as np
import pandas as pd
import matplotlib.pyplot as plt
import seaborn as sns
%matplotlib inline
from plotly import version
print(version)
3.1.0
import cufflinks as cf
E:\anaconda\lib\site-packages\plotly\graph_objs_deprecations.py:558:
DeprecationWarning:
plotly.graph_objs.YAxis is deprecated.
Please replace it with one of the following more specific types

plotly.graph_objs.layout.YAxis
plotly.graph_objs.layout.scene.YAxis

E:\anaconda\lib\site-packages\plotly\graph_objs_deprecations.py:531:
DeprecationWarning:
plotly.graph_objs.XAxis is deprecated.
Please replace it with one of the following more specific types

plotly.graph_objs.layout.XAxis
plotly.graph_objs.layout.scene.XAxis

import plotly.offline
from plotly.offline import download_plotlyjs, init_notebook_mode, plot,
iplot
init_notebook_mode(connected=True)
import cufflinks as cf
cf.go_offline()
IOPub data rate exceeded.
The notebook server will temporarily stop sending output
to the client in order to avoid crashing it.
To change this limit, set the config variable
--NotebookApp.iopub_data_rate_limit.
df = pd.DataFrame(np.random.randn(100,4),columns='A B C D'.split())
df.iplot(kind='scatter',x='A',y='B',mode='markers',size=10)
ValueError Traceback (most recent call last)
in ()
----> 1 df.iplot(kind='scatter',x='A',y='B',mode='markers',size=10)
E:\anaconda\lib\site-packages\cufflinks\plotlytools.py in _iplot(self,
kind, data, layout, filename, sharing, title, xTitle, yTitle, zTitle,
theme, colors, colorscale, fill, width, dash, mode, interpolation, symbol,
size, barmode, sortbars, bargap, bargroupgap, bins, histnorm, histfunc,
orientation, boxpoints, annotations, keys, bestfit, bestfit_colors, mean,
mean_colors, categories, x, y, z, text, gridcolor, zerolinecolor, margin,
labels, values, secondary_y, secondary_y_title, subplots, shape, error_x,
error_y, error_type, locations, lon, lat, asFrame, asDates, asFigure,
asImage, dimensions, asPlot, asUrl, online,
*kwargs)761
bargap=bargap,bargroupgap=bargroupgap,annotations=annotations,gridcolor=gridcolor,762
dimensions=dimensions,--> 763
zerolinecolor=zerolinecolor,margin=margin,is3d='3d' in kind,*l_kwargs)
764
765 if not data:
E:\anaconda\lib\site-packages\cufflinks\tools.py in getLayout(kind, theme,
title, xTitle, yTitle, zTitle, barmode, bargap, bargroupgap, margin,
dimensions, width, height, annotations, is3d, **kwargs)
199
200 theme_data = getTheme(theme)
--> 201 layout=go.Layout(theme_data['layout'])
202 layout['xaxis1'].update({'title':xTitle})
203 layout['yaxis1'].update({'title':yTitle})
E:\anaconda\lib\site-packages\plotly\graph_objs_layout.py in init(self,
arg, angularaxis, annotations, autosize, bargap, bargroupgap, barmode,
barnorm, boxgap, boxgroupgap, boxmode, calendar, colorway, datarevision,
direction, dragmode, font, geo, grid, height, hiddenlabels,
hiddenlabelssrc, hidesources, hoverdistance, hoverlabel, hovermode, images,
legend, mapbox, margin, orientation, paper_bgcolor, plot_bgcolor, polar,
radialaxis, scene, selectdirection, separators, shapes, showlegend,
sliders, spikedistance, template, ternary, title, titlefont, updatemenus,
violingap, violingroupgap, violinmode, width, xaxis, yaxis, **kwargs)
3735 self.images = images if images is not None else _v
3736 _v = arg.pop('legend', None)
-> 3737 self.legend = legend if legend is not None else _v
3738 _v = arg.pop('mapbox', None)
3739 self.mapbox = mapbox if mapbox is not None else _v
E:\anaconda\lib\site-packages\plotly\basedatatypes.py in setattr(self,
prop, value)
3590 if match is None:
3591 # Set as ordinary property
-> 3592 super(BaseLayoutHierarchyType, self).setattr(prop, value)
3593 else:
3594 # Set as subplotid property
E:\anaconda\lib\site-packages\plotly\basedatatypes.py in setattr(self,
prop, value)
2691 prop in self.
validators):2692 # Let known properties and private properties through->
2693 super(BasePlotlyType, self)._setattr(prop, value)
2694 else:
2695 # Raise error on unknown public properties
E:\anaconda\lib\site-packages\plotly\graph_objs_layout.py in legend(self,
val)
1224 @legend.setter
1225 def legend(self, val):
-> 1226 self['legend'] = val
1227
1228 # mapbox
E:\anaconda\lib\site-packages\plotly\basedatatypes.py in setitem(self,
prop, value)
3576 if match is None:
3577 # Set as ordinary property
-> 3578 super(BaseLayoutHierarchyType, self).setitem(prop, value)
3579 else:
3580 # Set as subplotid property
E:\anaconda\lib\site-packages\plotly\basedatatypes.py in setitem(self,
prop, value)
2654 # ### Handle compound property ###
2655 if isinstance(validator, CompoundValidator):
-> 2656 self._set_compound_prop(prop, value)
2657
2658 # ### Handle compound array property ###
E:\anaconda\lib\site-packages\plotly\basedatatypes.py in
_set_compound_prop(self, prop, val)
2952 validator = self._validators.get(prop)
2953 # type: BasePlotlyType
-> 2954 val = validator.validate_coerce(val)
2955
2956 # Save deep copies of current and new states
E:\anaconda\lib\site-packages_plotly_utils\basevalidators.py in
validate_coerce(self, v)
1783
1784 elif isinstance(v, dict):
-> 1785 v = self.data_class(**v)
1786
1787 elif isinstance(v, self.data_class):
E:\anaconda\lib\site-packages\plotly\graph_objs\layout_legend.py in
init(self,
arg, bgcolor, bordercolor, borderwidth, font, orientation, tracegroupgap,
traceorder, x, xanchor, y, yanchor, **kwargs)
500 # ----------------------------------
501 _v = arg.pop('bgcolor', None)
--> 502 self.bgcolor = bgcolor if bgcolor is not None else _v
503 _v = arg.pop('bordercolor', None)
504 self.bordercolor = bordercolor if bordercolor is not None else _v
E:\anaconda\lib\site-packages\plotly\basedatatypes.py in setattr(self,
prop, value)
2691 prop in self.
validators):2692 # Let known properties and private properties through->
2693 super(BasePlotlyType, self)._setattr(prop, value)
2694 else:
2695 # Raise error on unknown public properties
E:\anaconda\lib\site-packages\plotly\graph_objs\layout_legend.py in
bgcolor(self, val)
62 @bgcolor.setter
63 def bgcolor(self, val):
---> 64 self['bgcolor'] = val
65
66 # bordercolor
E:\anaconda\lib\site-packages\plotly\basedatatypes.py in setitem(self,
prop, value)
2663 # ### Handle simple property ###
2664 else:
-> 2665 self._set_prop(prop, value)
2666
2667 # Handle non-scalar case
E:\anaconda\lib\site-packages\plotly\basedatatypes.py in _set_prop(self,
prop, val)
2893 # ------------
2894 validator = self._validators.get(prop)
-> 2895 val = validator.validate_coerce(val)
2896
2897 # val is None
E:\anaconda\lib\site-packages_plotly_utils\basevalidators.py in
validate_coerce(self, v, should_raise)
1068 validated_v = self.vc_scalar(v)
1069 if validated_v is None and should_raise:
-> 1070 self.raise_invalid_val(v)
1071
1072 v = validated_v
E:\anaconda\lib\site-packages_plotly_utils\basevalidators.py in
raise_invalid_val(self, v)
214 typ=type_str(v),
215 v=repr(v),
--> 216 valid_clr_desc=self.description()))
217
218 def raise_invalid_elements(self, invalid_els):
ValueError:
Invalid value of type 'builtins.str' received for the 'bgcolor' property of
layout.legend
Received value: 'pearl02'
The 'bgcolor' property is a color and may be specified as:

A hex string (e.g. '#ff0000')
An rgb/rgba string (e.g. 'rgb(255,0,0)')
An hsl/hsla string (e.g. 'hsl(0,100%,50%)')
An hsv/hsva string (e.g. 'hsv(0,100%,100%)')
A named CSS color: aliceblue, antiquewhite, aqua, aquamarine, azure,
beige, bisque, black, blanchedalmond, blue, blueviolet, brown, burlywood,
cadetblue, chartreuse, chocolate, coral, cornflowerblue, cornsilk, crimson,
cyan, darkblue, darkcyan, darkgoldenrod, darkgray, darkgrey, darkgreen,
darkkhaki, darkmagenta, darkolivegreen, darkorange, darkorchid, darkred,
darksalmon, darkseagreen, darkslateblue, darkslategray, darkslategrey,
darkturquoise, darkviolet, deeppink, deepskyblue, dimgray, dimgrey,
dodgerblue, firebrick, floralwhite, forestgreen, fuchsia, gainsboro,
ghostwhite, gold, goldenrod, gray, grey, green, greenyellow, honeydew,
hotpink, indianred, indigo, ivory, khaki, lavender, lavenderblush,
lawngreen, lemonchiffon, lightblue, lightcoral, lightcyan,
lightgoldenrodyellow, lightgray, lightgrey, lightgreen, lightpink,
lightsalmon, lightseagreen, lightskyblue, lightslategray, lightslategrey,
lightsteelblue, lightyellow, lime, limegreen, linen, magenta, maroon,
mediumaquamarine, mediumblue, mediumorchid, mediumpurple, mediumseagreen,
mediumslateblue, mediumspringgreen, mediumturquoise, mediumvioletred,
midnightblue, mintcream, mistyrose, moccasin, navajowhite, navy, oldlace,
olive, olivedrab, orange, orangered, orchid, palegoldenrod, palegreen,
paleturquoise, palevioletred, papayawhip, peachpuff, peru, pink, plum,
powderblue, purple, red, rosybrown, royalblue, saddlebrown, salmon,
sandybrown, seagreen, seashell, sienna, silver, skyblue, slateblue,
slategray, slategrey, snow, springgreen, steelblue, tan, teal, thistle,
tomato, turquoise, violet, wheat, white, whitesmoke, yellow, yellowgreen

@borntospark0

Kindly help me out as to why i am unable to plot by using plotly in anaconda distribution?
below is the code snippet in jupyter notebook

...

plotly
anaconda distribution
jupyter notebook

Sorry, but for the fifth time (with slight alternations for the particular circumstances this time):

This is the bug tracker for the Spyder IDE, and is not a help forum, has nothing to do with plotly,  is not related to Jupyter notebook, and we are not the developers of the Anaconda distribution. Therefore, as your question is completely unrelated to both the purpose and the topic of this site, it should be taken elsewhere, as it is wasteful of the time of all involved to continue this further. Thank you.

Please find somewhere more appropriate to ask these questions, like Stack Overflow, another repo, or a mailing list. Thanks.


lang: init_if_needed should run checks on constraints even if no init is needed

closes #1034

any more comments on this @armaniferrante ?


Release/0.9.0

Coverage remained the same at 96.899% when pulling 38a9962a60a8e84430c7775ca6c6f666fdd9ac81 on release/0.9.0 into a95b3fab35db0b8bfd740f73a2a3074c4231c76d on master.


Multiple updates to docs fixing typos, broken links & clarifications

Category

[x] Content fix
[ ] New article

Related issues:

fixes #1536
fixes #1552
fixes #2068
fixes #3315
fixes #3400

What's in this Pull Request?
Multiple fixes to documentation covering typos, code errors, exceptions, broken links & clarification to steps/descriptions that were ambiguous:

thx @andrewconnell, appreciated.


Reverting Software Oversample Fails

On multiple occasions when testing I've set an oversample rate which the hardware doesn't support, upon reverting the setting all subsequent captures fail anyway.
Issue seems to occur after restarting bloodview, but not after restarting the device so is very likely a firmware issue.

Despite having recently seen this issue, I can't replicate. I'm just going to leave this here until we spot what causes it, or close it if we don't see it again.


Should unquirck Exception: Access to the path 'minion.pem' is denied.

MSI (s) (E4:48) [09:45:08:326]: Unlocking Server
MSI (s) (E4:48) [09:45:08:326]: PROPERTY CHANGE: Deleting UpdateStarted property. Its current value is '1'.
MSI (s) (E4:48) [09:45:08:326]: Doing action: CAC.NukeConf
Action ended 09:45:08: InstallFinalize. Return value 1.
MSI (s) (E4:18) [09:45:08:404]: Invoking remote custom action. DLL: C:\windows\Installer\MSI45FA.tmp, Entrypoint: CS_CUSTOM_ACTION_NUKE_CONF
Action start 09:45:08: CAC.NukeConf.
SFXCA: Extracting custom action to temporary directory: C:\windows\Installer\MSI45FA.tmp-\
SFXCA: Binding to CLR version v4.0.30319
Calling custom action MinionConfigurationExtension!MinionConfigurationExtension.MinionConfiguration.CS_CUSTOM_ACTION_NUKE_CONF
MinionConfiguration.cs:: Begin CS_CUSTOM_ACTION_NUKE_CONF
while deleting c:\salt\conf
Exception: Access to the path 'minion.pem' is denied.
   at System.IO.Directory.DeleteHelper(String fullPath, String userPath, Boolean recursive, Boolean throwOnTopLevelDirectoryNotFound)
   at System.IO.Directory.Delete(String fullPath, String userPath, Boolean recursive, Boolean checkHost)
   at MinionConfigurationExtension.MinionConfiguration.CS_CUSTOM_ACTION_NUKE_CONF(Session session)
CustomAction CAC.NukeConf returned actual error code 1603 (note this may not be 100% accurate if translation happened inside sandbox)
Action ended 09:45:09: CAC.NukeConf. Return value 3.
Action ended 09:45:09: INSTALL. Return value 3.
Property(S): UpgradeCode = {FC6FB3A2-65DE-41A9-AD91-D10A402BD641}
Property(S): BINFOLDER = C:\salt\bin\
Property(S): VARFOLDER = C:\salt\var\
Property(S): INSTALLFOLDER = C:\salt\

C:\git\salt-windows-msi>del C:\salt\conf\pki\minion\minion.pem
C:\salt\conf\pki\minion\minion.pem
Access is denied.

Fix issue after https://github.com/markuskramerIgitt/salt-windows-msi/issues/32

Permissions?

C# replaced by on error resume next vbscript in https://github.com/markuskramerIgitt/salt-windows-msi/commit/c9be74be43fb7a86f0d69637c3734d3b7506a29b
Probably same reason as https://github.com/markuskramerIgitt/salt-windows-msi/issues/40

https://github.com/markuskramerIgitt/salt-windows-msi/commit/c9be74be43fb7a86f0d69637c3734d3b7506a29b


[Feature Request] Add unit tests and coverage report

Is your feature request related to a problem? Please describe.
Add unit tests and coverage reports

Beginnings:
https://codecov.io/gh/firstof9/Home-Assistant-Mail-And-Packages/commits

Currently at 88.06% coverage.
https://codecov.io/gh/firstof9/Home-Assistant-Mail-And-Packages/commits

0.3.0 branch created, currently reporting 96.68% coverage
https://codecov.io/gh/moralmunky/Home-Assistant-Mail-And-Packages/commits

If anyone is familiar with how to mock aiohttp requests I'm all ears


Consider not using temporary GOPATH if env var is set

Using estimate downloads a whole lot of source code and thus tikes a while, every time it's called. However there's a good chance that someone who's packaging Go has an actual Go dev setup, i.e. the GOPATH env var is set. I'd prefer respecting that environment variable. Or at least add a flag that enables respecting the GOPATH env var.

Actually never mind, this is a very "go-module-perspective" - as long as this needs to work with GOPATH-mode it isn't doable: You don't want a script like this to mess with directories on disk that a dev might be actively using (modifying).


Try specify expected JAVA_HOME and bin for mvn-verify-check when github incident [skip ci]

fix #8847
We monitored that setup-java action would not perform correctly if Github is suffering some perf incidents (I guess GitHub was doing some online service migration to mitigate the issue which could mess up w/ the test ENVs)
Let's force the expected versions in commands and stop relying on the actions.

build

LGTM


initialDate is overriden by minDate on DatesRangeInput

After defina a min and max date the initialDate value is overriden and the date picker opens on the mindate value. If I don't set a minDate and maxDate the date picker opens correctly.
My Code
 <DatesRangeInput
               name="datesRange"
               placeholder={t('accounts.movements.date-filter')}
               icon="calendar alternate outline"
               iconPosition="right"
               style={{ color: "#0997A7" }}
               onChange={handleChangeDate}
               closable={true}
               dateFormat={dateFormat}
               maxDate={new Date(new Date().getFullYear(), 11, 30)}
               minDate={new Date(new Date().getFullYear() - 1, 0, 2)}
               value={datesRange}
 />

Expected behavior
The date picker should open on the default of initialDate(today's date)
Dependencies versions
"semantic-ui-calendar-react": "^0.15.3",
"semantic-ui-react": "^0.88.0",

@paulopretoEI - Have you tried just adding initialDate={new Date()} (where new Date() is the correct date format)?

Is there a fix for this issue @arfedulov ?

@paulopretoEI - Have you tried just adding initialDate={new Date()} (where new Date() is the correct date format)?

I have tried this and the calendar still defaults to the minDate value.


Why was Queue Mode removed from the Leshan Implementation?

Hi everybody,
I'm working in a queue mode implementation inside the Leshan Server Core, and I have just seen that there was an initial implementation of it, but it was completely removed in the commit #2826841. I'm wondering what was the reason for this, because I couldn't found any information about it.
Can someone please explain me what was the reason for doing this?
Thank you very much.

We had a first implementation of queue mode in Leshan long time ago.
This implementation used an internal Queue to store pending request, but we found this not so flexible.
In the meantime, the LWM2M specification changed. OMA dropped this queue idea and replaced it by something more like an offline/online mode (without changing the name by the way).
So we removed this queue implementation from Leshan. (More details about that at #191)
A "work in progress PR" was proposed #265 with something simple and more in line with the new specification, but contributor seems to give up :'( ...
Currently, we don't think a queue mode is a priority as you can easily handle it just by using  RegistrationListener and/or ObservationListener.
But if you want to contribute Queue mode based on #265, you're welcome :)

I have tried to find this specific change - where is the no-longer-queuing change? For easiest possible application development on the server side (assuming that the service / application should not know all details of the client / IoT device) it would make sense to have a higher abstraction than knowing the details of on/off state? Or? One possible approach for doing it could of course be to do a very lightweight component that binds in a way so that the application / service decides if it only needs to know on/off state or to use a more fully-fledge queue to collect the requests?! Did anyone ever evaluate the Q-mode mode of operation when it comes to power efficiency, etc?

I have tried to find this specific change - where is the no-longer-queuing change?

You should read the §8.3 Queue mode operation. of the specification  :
"As such, the Queue Mode offers functionality for a LwM2M Client to inform the LwM2M
Server that it may be disconnected for an extended period of time and also when it becomes reachable again."
" Despite the title of the functionality, i.e. Queue Mode, this specification does not mandate an implementation to use queues nor does it specify where such a queue would exist (or any details of such queuing mechanism)."

it would make sense to have a higher abstraction than knowing the details of on/off state?

Maybe, but we should first create this online/offline service and only  if this is really needed create an optional API build on this online/offline service.
But this raises several issues,  should we persist this "queue" ? should we shared it between several instances when you are using leshan in a cluster.
Our production server currently use Leshan in a cluster and we don't need this kind of queue because we already have a mechanism which store "requests", and often request you sent depends on previous response so you will not queue so much request ...

If the specification says only that the queue is not mandatory, you could still have a queue mode implementation with an actual queue that you can activate or not whenever you create your Leshan Server application. You could have an API to register listeners if your application wants to be notified about the state of the client, but you could also decide if you want to have an actual queue to store requests when the client is sleeping. Wouldn't that be a nice feature?
For some applications, like IoT scenarios with sleepy devices, it would be very suitable to have a queue inside the core so that whenever the client informs that it is awake, the Leshan server can send all of them. This will also reduce the latency and therefore the client can go to sleep more quickly and save more energy.
Also it would be nice to have a mechanism that allows the server to know the parameters of the Queue mode operation, i.e. the sleeping time and the time it will stay awake. An option would be to have a LWM2M object in the client with this information, that the server can read. With this the server can easily know when the client is sleeping again after an update message.

If the specification says only that the queue is not mandatory, you could still have a queue mode implementation with an actual queue that you can activate or not whenever you create your Leshan Server application.

I'm not against that idea, but I think this is better to focus on the offline/online mode which is more flexible and then we could consider to implement a kind of queue and see if this make sense to integrate it in Leshan.

Also it would be nice to have a mechanism that allows the server to know the parameters of the Queue mode operation, i.e. the sleeping time and the time it will stay awake. An option would be to have a LWM2M object in the client with this information, that the server can read. With this the server can easily know when the client is sleeping again after an update message.

What could be the need to now the "spleeping time" ? just for information  or this should be used by Leshan server ?
"The time it will stay awake" is defined by the specification :
"The LwM2M Client is recommended to use the CoAP MAX_TRANSMIT_WAIT parameter to set a timer for how long it shall stay awake since last sent message to the LwM2M Server. After  MAX_TRANSMIT_WAIT seconds without any messages from the LwM2M Server, the LwM2M Client enters a sleep mode."
Anyway, I think this new object is out of spec and should not be integrated like this in Leshan, but we can try to make it possible for Leshan users.
But first priority about Queue mode, the offline/online mode.

If Leshan is intended for sleepy devices I do think there is use for some new mechanisms to avoid waiting CoAP MAX_TRANSMIT_WAIT - partly because this is on 90-seconds as default values - which means there is no way you will get multi-year battery life-time with a device that runs LWM2M and make use of reasonably low report rates (e.g. something like report each 1 or 5 minutes). As I see it there are two options on this - (1) improve the update mechanism to include a expected up-time so that the server know how long the device will be awake this specific time or (2) have a LWM2M queue-more object that all nodes that use q-mode are recommended to implement for discovering these values. There is no way to discover this today - and there is no way we can configure all LWM2M devices to use the same MAX_TRANSMIT_WAIT because this is not intended to be a part of the CoAP specification as it is per application. LWM2M is however an application of CoAP and have the possibility to either postpone to applications on top of LWM2M or do something about it ;-)
I have been working with sleepy devices that report on 60-120 seconds basis and with short awake times you can achieve very good battery lifetimes. But going for MAX_TRANSMIT_WAIT as the offline/online mode expectation alone will give you five days in that case...

The LwM2M Client is recommended to use the CoAP MAX_TRANSMIT_WAIT.
It's just a recommendation, the device could leave before. This is not really an issue at server side. Server will try to send requests when device does answer anymore the server consider the device as offline.
But again, I'm affraid we put the cart before the horse. First implement online/offline mode.

Sure - we can put the horse before the cart - but we will not only put the cart there but go for a cart with a motor also ;-)

Hi! I've made a pull request #451 with the online/offline that @sbernard31 requested :)
We can discuss there a bit and, if it gets accepted, we can continue looking into the other things that we've talked about: the presence of actual queues (with possibility of enabling/disabling them) and the knowledge of the client awake time in the server, to reduce it to a much more lower one than the CoAP MAX_TRANSMIT_WAIT.

Thx to the great work of @carlosgp143 which revives the initial work of @balsmn, a first version of QueueMode support is available here #469 and will be soon integrated in master.
Feedbacks are welcomed.


add JsonResponse import

The current anything.py in https://docs.vibora.io/docs/ is not runnable because of an import issue
from vibora import Vibora, JsonResponse

app = Vibora()

@app.route('/')
async def home():
    return JsonResponse({'hello': 'world'})

if __name__ == '__main__':
    app.run(host="<IP_ADDRESS>", port=8000)

so add it from servers should help.

I think import the entire responses module as a shortcut should be better for users (from .responses import *)... What do you think ? (It will be kinda weird to be able to import Response/JsonResponse but not the StreamingResponse)...


Process Explorer v16.22

Updated version of Process Explorer to actual v16.22 (file was modified on the server)

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

sysinternals/downloads/process-explorer.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Uwp targets update to give error text

Allows the project to load for UWP projects even if the SDK isn't installed and gives a build error indicating where to get CPython UWP SDK if not installed

Once we pull out the unnecessary condition, it looks good to go in.


GDB IN-VM: Use DOS line endings sends LF LF instead of CR LF on Linux

Describe the bug
GDB/MI commands sent by Ghidra have two line feed characters (LF LF) instead of CR LF if you set "Use DOS line endings" checked.
To Reproduce
Steps to reproduce the behavior:

Open Debugger
Select "Debug  in GDB locally IN-VM"
Check "Use existing session via new-ui" and "Use DOS line endings (unchecked for UNIX and Cygwin))"
Click "Connect"

Then, I could confirm this issue by following these steps:

Open a terminal emulator, then issue cat -v < /dev/pts/[ghidra's pty num]
On another terminal, issue echo '=thread-group-added,id="i1"' > /dev/pts/[ghidra's pty num]
You can see the input from Ghidra on the terminal opened on Step. 2

-interpreter-exec console "show architecture"

It has two LF characters instead of one (a blank line after the first line).
Expected behavior
GDB/MI commands have CR LF line endings.
In the test case above, we should see the following output (^M is at the end of the first line and no blank line is followed):
-interpreter-exec console "show architecture"^M

I think this happens becase the pty that Ghidra opens has icrnl termios setting by default, which must be off. You can check it by stty -F /dev/pts/[ghidra's pty num] -a
Environment (please complete the following information):

OS: Fedora Linux 39 (Workstation Edition)
Java Version: 17.0
Ghidra Version: 11.0
Ghidra Origin: official GitHub distro

Is this flaw preventing you from using this feature? If I may prod, what is your use case here? Or is this more about the software doing precisely what it says, in principle?
The text on the checkbox indicates you should not enable it on UNIX (Linux is a UNIX, despite the expansion of its acronym) nor on Cygwin. The behavior you demonstrate of the pty is part of Linux, so I do not believe this is truly a flaw in practice.
That said, it seems like a low-risk change, but we won't know until we test. We'd like to know how necessary this actually is in practice before we put in any work.

@nsadeveloper789 Thank you for your reply.
First thing is first, I found this feature is not required for my use case after some investigation. FWIW, let me tell you what I was working on.
I was trying to run GDB on Wine and control it directly through Ghidra. I first tried to use gdbserver as it was described on this page and scripts bundled with Ghidra. Unfortunately, gdbserver did not work well especially with multi-threaded applications. There were problems such as GDB always does break whenever a thread terminates. On the other hand, running gdb directly on Wine worked well even with multi-threaded applications.
I tried new-ui command by passing Ghidra's pty, but GDB stopped responding when I launched the debbuged program (they implemented event loop with select(), it does not work well with Windows and Wine). Next, I tried to connect GDB on Wine to Ghidra with a shell script that mimicked GDB on Linux. To my surprise, they could talk each other even after launching an program. Some tweaks were still needed for features like manual interrupting (I also had to patch GDB to generate info proc mappings but that's another story). Manual interrupts need direct input from a tty (pty), but not from a pipe. Also, Wine pre-processes input when the standard input is a pty, so input filtering was necessary (this was also true for output).
Lastly, I have suceeded in connecting GDB on Wine and Ghidra. However, it still does not meet my requirements. For example, the communication between GDB and Ghidra freezes when GDB receives an interrupt. This is because GDB and Ghidra exchange a lot of data and Wine's pre/post-processing slows down their communication (espacially when there are many threads). In my case, the debugged program has ~80 threads, and their communication stops for around 2-3 mininutes.
That's the whole story. Thank you for taking your time. I'm very surprised the power of Ghidra.
P.S.
I am curious for use cases of this option. Programs on Windows opens stdin in text mode by default, and so does GDB. According to the Microsoft's fopen manual (and some investigation), both a single LF and CR LF sequenses are sent to an apllication as LF in text mode, so I think sending LF terminated lines to GDB has no problem even on Windows. Additionaly, It looks like Ghidra on Windows does not even have "Debug in GDB locally IN-VM" menu item. Is "Use existing session via new-ui" option obsolete?

Interesting. Some of these configurations where I've seen people attempt to debug Windows apps on Linux/Wine have been rather convoluted. This is quite an example.... FWIW, I'm hoping that the upcoming Trace RMI stuff will greatly simplify this. Instead of stdio, it'll use TCP, and for gdb, it's a Python plugin, so this setup should be easier 🤞 . But, that's yet to come.
You might just try using Linux gdb on wine itself. You'll need to unwrap the wine scripts a bit to get at the command line for the actual wine binary, but I've had success doing it for Minesweeper :) . To gdb, wine is just a regular Linux app, so you don't need the wine32 helper scripts, and this simplifies the connection between gdb and Ghidra. However, Linux gdb is unable to parse the PE files that wine maps into memory, at least with the version I tried. It's not terrible unless you need to refer to symbols in the gdb CLI.... You may also need to map the PE-based modules in Ghidra manually.
It's a fair question why that checkbox would be presented at all for the plain gdb case. The checkbox was originally created for the SSH connector, where Ghidra can't know the remote system's line endings a priori. IIRC, for the plain gdb connector, we used to just used CR LF on Windows and LF on Linux. However, the Cygwin and MinGW versions of gdb each behave differently. Thus, we copied the checkbox over to the plain gdb connector and put the onus on the user to set it correctly. A user running Ghidra on Linux should always leave it unchecked. A user running Ghidra on Windows connecting to Cygwin gdb should also leave it unchecked. A user running Ghidra on Windows connecting to MinGW gdb would need to check it. In our experience, just sending LF (as we would on Linux) has not worked with MinGW gdb.

I'm glad to hear that news! That new GDB interface is going to solve our problems. I'm looking forward to that.
Also, thank you for the great tip! I've never thought of attaching wine process to the native GDB. It worked well for me. Thankfully, memory mapping of a wine process is relatively simple, so it was easy to find the target regions and map those to the image. Since my target program did not have that many exported symbols, how to refer to them in GDB was not my problem in the first place . Ghidra, by contrast, found me a lot of functions and classes, though :)
Lastly, thank you for answering my question. I hope the new Trace RMI will solve Cygwin and MinGW related issues as well.
I'm closing this issue now. As I mentioned above, the new Trace RMI will likely solve this issue.


Day0: add DOMAIN_NAME and common/iam sed replacement to writeids.sh

organization_iam_group_secadmin = [
  {
    member       = "group:secadmin@DOMAIN_NAME" # REQUIRED EDIT<EMAIL_ADDRESS>    organization = "REPLACE_ORGANIZATION_ID" #Insert your Ord ID here, format ############

code
michael@cloudshell:~/lzone-oe/lz-oe/deployed/pbmm-on-gcp-onboarding/environments/bootstrap (lzone-oe)$ USER=$(gcloud config list --format json|jq .core.account | sed 's/"//g')
michael@cloudshell:~/lzone-oe/lz-oe/deployed/pbmm-on-gcp-onboarding/environments/bootstrap (lzone-oe)$ echo $USER
<EMAIL_ADDRESS>michael@cloudshell:~/lzone-oe/lz-oe/deployed/pbmm-on-gcp-onboarding/environments/bootstrap (lzone-oe)$ DOMAIN_NAME=$(echo $USER | sed 's/"//g' | cut -f2 -d@)
michael@cloudshell:~/lzone-oe/lz-oe/deployed/pbmm-on-gcp-onboarding/environments/bootstrap (lzone-oe)$ echo $DOMAIN_NAME
obrienlabs.engineering
michael@cloudshell:~/lzone-oe/lz-oe/deployed/pbmm-on-gcp-onboarding/environments/bootstrap (lzone-oe)$ DOMAIN_NAME_SEARCH=DOMAIN_NAME
michael@cloudshell:~/lzone-oe/lz-oe/deployed/pbmm-on-gcp-onboarding/environments/bootstrap (lzone-oe)$ sed -i "s/${DOMAIN_NAME_SEARCH}/${DOMAIN_NAME}/g" ../common/iam-groups.auto.tfvars

member       =<EMAIL_ADDRESS># REQUIRED EDIT<EMAIL_ADDRESS>

20240406: Closing issue during retrofit/rebase of this TEF V1 based/modified repo to TEF V4 standards
This issue may participate in the LZ refactor after rebase
Query on all issues related to the older V1 version via the tag
https://github.com/GoogleCloudPlatform/pbmm-on-gcp-onboarding/labels/2024-pre-tef-v4


Release v0.3.0

Release Checklist

[ ] All OWNERS must LGTM the release proposal
[x] Verify that the changelog in this issue is up-to-date
[ ] For major or minor releases (v$MAJ.$MIN.0), create a new release branch.

[ ] an OWNER creates a vanilla release branch with
git branch release-$MAJ.$MIN main
[ ] An OWNER pushes the new release branch with
git push release-$MAJ.$MIN

[ ] Update things like README, deployment templates, docs, configuration, test/e2e flags.
Submit a PR against the release branch:
[ ] An OWNER prepares a draft release

[ ] Write the change log into the draft release.
[ ] Run
make artifacts IMAGE_REGISTRY=registry.k8s.io/kueue GIT_TAG=$VERSION
to generate the artifacts and upload the files in the artifacts folder
to the draft release.

[ ] An OWNER creates a signed tag running
git tag -s $VERSION
and inserts the changelog into the tag description.
To perform this step, you need a PGP key registered on github.
[ ] An OWNER pushes the tag with
git push $VERSION

Triggers prow to build and publish a staging container image
gcr.io/k8s-staging-kueue/kueue:$VERSION

[ ] Submit a PR against k8s.io,
updating k8s.gcr.io/images/k8s-staging-kueue/images.yaml to
promote the container images
to production:
[ ] Wait for the PR to be merged and verify that the image registry.k8s.io/kueue/kueue:$VERSION is available.
[ ] Publish the draft release prepared at the Github releases page.
[ ] Add a link to the tagged release in this issue:
[ ] Send an announcement email to<EMAIL_ADDRESS>and<EMAIL_ADDRESS>with the subject [ANNOUNCE] kueue $VERSION is released
[ ] Add a link to the release announcement in this issue:
[ ] For a major or minor release, update README.md and docs/setup/install.md
in main branch:
[ ] For a major or minor release, create an unannotated devel tag in the
main branch, on the first commit that gets merged after the release
branch has been created (presumably the README update commit above), and, push the tag:
DEVEL=v0.$(($MAJ+1)).0-devel; git tag $DEVEL main && git push $DEVEL
This ensures that the devel builds on the main branch will have a meaningful version number.
[ ] Close this issue

Changelog
https://github.com/kubernetes-sigs/kueue/blob/main/CHANGELOG/CHANGELOG-0.3.md#v030

only awaiting on #654

LGTM

Now awaiting for #681, but otherwise ready to roll!

Wonderful! A very exciting milestone :)

cc @kerthcet @ArangoGutierrez

Image is building in the bot

https://github.com/kubernetes-sigs/kueue/releases/tag/v0.3.0 :partying_face:
/close


   scottPlot.Plot.Add.SignalXY(new double[0], new double[0]) Crash

scottPlot.Plot.Add.SignalXY(new double[0], new double[0]);
crashes with IndexOutOfRangeException.
Scottplot 5.0.35

Thanks for pointing this out @githubkau! I fixed this issue in a531cb3 and it will be in the next release 👍


pipenv support: create pipfile from settings.ini

I have been experimenting with pipenv and wanted a way to generate the pipfile from the settings.ini.
This is a first attempt at getting that done.

Thanks for the PR. I'd rather not add this -- there's a lot of different dependency systems out there and we're not going to support them all.

sure. i get that. i might just create a helper lib for my own purposes and make it public.

nbdep helper lib created. repo is at https://github.com/restlessronin/nbdep


Beginner question about canvas creation

I'm running back and forth between the basic usage and here in this repo but I don't quite understand how the canvas is created. I also don't see any other library related to the canvas creation. I downloaded the whole source, I can only display the GUI but no canvas at all. :( Also, there isn't any console error.
What I understand so far is that:

GUI - lets me control different variables set for the canvas
main.js - based on the workshop, when I peek into the source, I see that in this file HTML elements are created
FizzyText - this accepts all the property to create the canvas.

But I still don't figure out how I can create my own canvas or FizzyText :(

Hopefully you've been able to make some progress on creating a canvas. If not, here is some general "getting started with HTML5 Canvas" info to check out: https://www.sitepoint.com/html5-canvas-tutorial-introduction/


Add char8_t mode

Overview
This PR adds a new mode to pugixml which uses C++20 char8_t instead of char for the UTF-8 interface. I've dubbed it char8 mode and it can be enabled using the PUGIXML_CHAR8_MODE option/macro.
Motivation
Using pugixml in a mixed ANSI/Unicode Windows codebase is quite errorprone. char is already used for the system codepage and it can easily happen that a UTF-8-as-char string slips through without conversion. I've found char8_t it to be a huge help in migrating stuff to UTF-8.
Implementation
Not much has to be changed. Since pugixml already calls C string functions with UTF-8, we can do the same with char8_t by just casting in a few places.
Stream-based methods received an additional overload, since the char overload may be used to represent arbitrary bytes, and the char8_t overload may be used by string streams.
An additional typedef u8char_t, which represents the type pugixml uses for a UTF-8 code unit, was added for the conversion functions.
Most changes had to be done in the test code. Representing raw bytes as string literals does not work for UTF-8 literals, since hex escape codes are interpreted as a Unicode character (and replaced by multiple bytes). Affected places either received a branch with a u8 literal or use a new RAW() macro which smuggles in UTF-8 code points using chars.
Notes

As implemented, the PUGIXML_CHAR8_MODE modifies the UTF-8 interface, which means PUGIXML_WCHAR_MODE takes precedence if both are defined (no diagnostic).
I haven't updated the CI build descriptions since I don't know the compiler versions. Locally I've tested with Visual Studio 2022 17.3 and clang 15.
Maybe an overload taking char8_t file paths should be added as well?

Just a question on the last note. (The rest sound good to me, thanks)
Are you really using or planning to use char8_t for paths? Especially on Windows where UTF-16 is used? There are no overloads for fopen() or std::fstream::open() AFAIK. Would not std::filesystem::path be the better type?

Is there any chance of this PR ever progressing?

While the changes in the PR itself feel reasonable, I'd like to better understand how widely this will be used. This adds a new compilation mode; for thorough testing it would require increasing number of tested build configurations by 50%. In some sense it's fine, in some other sense it's a notable increase in complexity. It's a C++20 feature so it's not obvious how widely available it would be - I would be more comfortable with some indication from multiple people about this being very valuable in their work.
I think something like this could be merged if there's sufficient demand for a new mode. If there's any way to reduce the differences in tests that would also be nice - this doesn't matter that much for the PR itself, but when adding new tests the wchar_t mode is already a problem because people tend to forget STR macros, so if any of the new macros could be replaced somehow that would be nice. It's not a big deal if they are necessary.

A little related although that one needs a separate fix to also avoid the footgun in regular mode: https://github.com/zeux/pugixml/issues/378

Probably the main thing that would make me comfortable with merging this would be more indication that this is going to be helpful for various users.

For one, it would make pugixml future proof and help adoption of char8_t. Many complaints about char8_t are about how useless it is since nothing supports it; a classic chicken-and-egg problem. User's on Linux have long misused char and treated it as always being UTF-8 but it's not portable (and might not even work on all Linux systems).
On Windows the situation is worse since char is almost never UTF-8 and just compiling with UNICODE does not make all the legacy APIs go away. The options are:

wchar_t-mode: Increased memory usage. Many unnecessary conversions. UTF-16 is not necessarily a good internal encoding even on Windows.
UTF-8 as char: Error-prone and unportable.
Convert to char8_t: Cumbersome since every use of pugixml looks like PugiToU8(node.attribute(U8ToPugi(name)).value()).


Two ViewControllers NowPlaying info not updating...

When you are in StationViewController1 everything plays fine NowPlaying info updates ok, but when enter StationViewController2 NowPlaying updates ok on StationViewController2, when going back to StationViewController1 NowPlaying does NOT update station info playing.
How can i update NowPlaying info for StationViewController1 even if am playing stations from StationViewController2.
Can anyone help?
Thank you...

Hi @dardania84
The RadioPlayer class uses delegates to communicate with the StationViewController and it's one to one relationship, so when you add StationViewController2 it becomes the delegate and StationViewController1 won't be updated, if you'd like to listen to changes in more than one controller, you either have to create a mechanism (a middle class) to handle the delegate and update both controllers, or use notifications instead of delegates in the RadioPlayer class.


#17 - Added reset on transaction sent

Proposed changes

Added reset functionality for KIP7, KIP17, KIP37 form after transaction submission

Types of changes
Please put an x in the boxes related to your change.

[] Bugfix
[x] New feature or Enhancement
[ ] Others

Checklist
Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code.

[x] I have read the CONTRIBUTING GUIDELINES doc
[ ] I have signed the CLA
[ ] Lint and unit tests pass locally with my changes
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have added necessary documentation (if appropriate)
[x] Any dependent changes have been merged and published in downstream modules

Related issues

#17

LGTM


Text disappeared after update to 1.13 [Bug]

VSCode Version: 1.13
OS Version: Windows 10.0.14393

Steps to Reproduce:

Updated to 1.13
Nothing else

Update
Insider version is the same
Update 2
Downgrade to 12.2 fixed the problem
Update 3
Problem is with font-family: system-ui , how to fix it?

@tonoslav Whe you say "Insider version is the same", do you mean that you see the issue in Insiders as well or Insiders works as expected and is same as before ?

@tonoslav this could be due to us changing the default font in 1.13 for the workbench. Can you open the dev tools (from the help menu) and find the monaco-shell CSS class font-family:

Change this to "Arial" or "Times New Roman", does the text show up?
What are your OS font settings?

Could also be some caching problem. Try to delete %AppData%\Code\Cache and %AppData%\Code\CachedData while VSCode isn't running and start it again.
The font should be the same as the one for File, Edit, etc. (which does show up). But maybe it's using a different glyph cache or something?

@ramya-rao-a yeah I meant that it doesnt matter if I use normal VSCode or Insider version
@jhasse Cache clear didnt help
@bpasero I have found out that the problem is with Font-family: system-ui when I delete system-ui from fonts in developer tools, it works, how can I fix this in general?
I have found out I have this problem with system-ui font everywhere in all chromium like browsers I think, firefox for example works ok when I see website with font-family: system-ui

I have found out I have this problem with system-ui font everywhere in all chromium like browsers

That's very good to know! Can you reproduce it here? https://jsfiddle.net/9yeoLewo/
I've created a Chromium bug report, maybe someone from the developers can comment on this problem: https://bugs.chromium.org/p/chromium/issues/detail?id=733219

firefox for example works ok when I see website with font-family: system-ui

Firefox doesn't support system-ui (it's an alias for the default system font).

Given this bug I think we need to revert the change to use system-ui until Chrome fixed this issue. It looks like the fallback is not applied, so there does not seem to be a workaround.

Yes, you should be able to revert 663608d61d0702d20f0c01f0221b3b348f33fca2 and then 45d93e93255f9cdaa37955fdb79b4e460252b45b.

@jhasse @bpasero So what should I do? Should I replace some files by myself or what? Thx

That's a possibility, although VS Code will then complain, that your installation seems to be corrupted.
I would stick with 1.12 for the time being.
Can you reproduce the problem here? https://jsfiddle.net/9yeoLewo/
Also: Do you have any unusable font settings enabled?

@jhasse

yes I can reproduce it there
No I don't have any unusable font settings

Btw does this means that everybody has the same problem, or why I have and some don't?

So far it seems that you're the only one with the problem. I can't reproduce it on Windows 10 and system-ui was enabled for the Insiders version for some time.

So far it seems that you're the only one with the problem.

I think I saw this reported one other time, might have been on Twitter.

Reverted.


Allow limit exceeded errors to be retried

The current implementation of retryCloudKitOperationIfPossible() only performs a retry if a value for retryAfterSeconds is present. However, 'limit exceeded' errors by CloudKit do not include a value for retryAfterSeconds (see Apple documentation). This caused retries of limit exceeded errors not to be performed. Because the entries remain 'cached' in the upload/delete buffer, this also effectively prevents any further uploads/deletions as Cirrus tries to bundle any other updates with the ones still in the buffer.
This fixes that by using a default delay of 0 for limit exceeded errors.
An alternative solution would be to not use retryCloudKitOperationIfPossible() at all, and simply directly perform the chunked requests. It might be a bit cleaner, at the loss of some logging (and there's a possibility that Apple in the future will include a value for retryAfterSeconds'for these errors).

Thanks a lot @jeffreykuiken and apologies for my delay in getting to this!
I think splitting the request is the required way to handle this error, per the documentation for limitExceeded:

An error that occurs when a request’s size exceeds the limit.
If your app receives CKError.Code.limitExceeded, it must split the operation in half and try both requests again.

Because of this it also makes sense that retryAfterSeconds doesn't exist on this response, since retrying won't resolve the error.
In Cirrus limitExceeded is currently handled by splitting the request here: https://github.com/jayhickey/Cirrus/blob/a6bf35a03e30830cafe67c3981c06bf48d145d1b/Sources/Cirrus/SyncEngine%2BRecordModification.swift#L111-L131
However this is only done in response to record modification requests (modifyRecordsCompletionBlock), so maybe this logic should be moved into retryCloudKitOperationIfPossible so it is handled in response to the other types of requests. Have you hit a limitExceeded error in the completion handler response to a different type of CloudKit operation when using Cirrus?

Thanks for looking into this! I do agree that splitting up the request and retrying it that way makes most sense, and indeed that's how Cirrus is already trying to resolve this error. I also have not experienced limitExceeded errors in response to other CloudKit operations myself.
In the current implementation, Cirrus first splits up the request (lines 111-120 in the code you referenced). Then for both halves, Cirrus calls error.retryCloudKitOperationIfPossible with a block that contains a call to modifyRecords
https://github.com/jayhickey/Cirrus/blob/a6bf35a03e30830cafe67c3981c06bf48d145d1b/Sources/Cirrus/SyncEngine%2BRecordModification.swift#L124-L129
However, error.retryCloudKitOperationIfPossible only performs this block when the error contains a retryAfterSeconds attribute (which as you mentioned, these limitExceeded errors do not). If there's no retryAfterSeconds, the method returns early:
https://github.com/jayhickey/Cirrus/blob/a6bf35a03e30830cafe67c3981c06bf48d145d1b/Sources/Cirrus/Error%2BCloudKit.swift#L84-L87
So this means that the blocks attempting to 'resubmit' the two halves of the splitted up request are never performed. That's also what I experienced: as soon as I hit a limitExceeded error, these modifications would never be retried or successfully processed, and they also remained in the buffer forever.
My quick fix was to simply to use a delay of 0 for limitExceeded errors, so that the 'splitted up retry blocks' would still be perfomed. But now that I think about it again, there must indeed be a cleaner way to fix this :)

I see! Makes sense, thanks!


Vagrant, ssh-authorization fails on restart

We set up rancher in Vagrant. It seems to work on the first run, but after the restart ssh-key is no longer accepted and we have to re-create the VM.
On the first run there is a notification about replacing insecure key. I wonder if that is related somehow.

seems to be related to https://github.com/mitchellh/vagrant/issues/5153 and https://github.com/mitchellh/vagrant/issues/5219 (Vagrant 1.7 issue)

adding config.ssh.insert_key = false to Vagrantfile might help. testing it now

It might help the first time with config.ssh.insert_key = false, but after a virtualbox or vagrant upgrade you will get the password prompt again.

Any fixes for this? We are dealing with the same issue. Once I create the vagrant box, if I stop and start it it just loops through "rancher: Warning: Authentication failure. Retrying..."

I'm having the same issue.

We recently cutover to using RancherOS in our Vagrant environment. We are interested to hear how it impacts this issue.

I'm going to close this issue as we haven't heard any issues after we've moved over to RancherOS. If anyone has issues with vagrant file, please open a new one or comment back on this one.


NODE_MODULE_VERSION error

/node_modules/node-expat/build/Release/node_expat.node'
was compiled against a different Node.js version using
NODE_MODULE_VERSION 59. This version of Node.js requires
NODE_MODULE_VERSION 57. Please try re-compiling or re-installing
the module (for instance, using npm rebuild or npm install).
node-expat : 2.3.16
node: 8.10
npm: 5.6
electron: 1.8.4
I also used electron-rebuild to build the package to new target node version
I also used node-gyp to rebuild node-expat. Nothing helped.
Any ideas to fix this issue?

Same Issue here

Same here.  Any solution to this?  It occurred after I upgrade from node 8.12.0 to node 10.13.0.  I am using node-expat via xml2json.  The error I get is when I run my node app:
/root/workspace/myapp/node_modules/bindings/bindings.js:88
		throw e
		^

Error: The module '/root/workspace/myapp/node_modules/node-expat/build/Release/node_expat.node'
was compiled against a different Node.js version using
NODE_MODULE_VERSION 64. This version of Node.js requires
NODE_MODULE_VERSION 57. Please try re-compiling or re-installing
the module (for instance, using `npm rebuild` or `npm install`).
	at Object.Module._extensions..node (module.js:682:18)
	at Module.load (module.js:566:32)
	at tryModuleLoad (module.js:506:12)
	at Function.Module._load (module.js:498:3)
	at Module.require (module.js:597:17)
	at require (internal/module.js:11:18)
	at bindings (/root/workspace/myapp/node_modules/bindings/bindings.js:81:44)
	at Object.<anonymous> (/root/workspace/myapp/node_modules/node-expat/lib/node-expat.js:4:32)
	at Module._compile (module.js:653:30)
	at Object.Module._extensions..js (module.js:664:10)

I have the same problem. I tried building on 8.x, 10.x and 11.x. I am also using node-expat only as dependency of xml2json.
Error: The module 'node_modules/node-expat/build/Release/node_expat.node'
was compiled against a different Node.js version using
NODE_MODULE_VERSION 67. This version of Node.js requires
NODE_MODULE_VERSION 69. Please try re-compiling or re-installing
the module (for instance, using `npm rebuild` or `npm install`).
    at process.module.(anonymous function) [as dlopen] (ELECTRON_ASAR.js:160:31)
    at Object.Module._extensions..node (internal/modules/cjs/loader.js:722:18)
    at Object.module.(anonymous function) [as .node] (ELECTRON_ASAR.js:160:31)
    at Module.load (internal/modules/cjs/loader.js:602:32)
    at tryModuleLoad (internal/modules/cjs/loader.js:541:12)
    at Function.Module._load (internal/modules/cjs/loader.js:533:3)
    at Module.require (internal/modules/cjs/loader.js:640:17)
    at require (internal/modules/cjs/helpers.js:20:18)
    at bindings (node_modules/bindings/bindings.js:112:48)
    at Object.<anonymous> (node_modules/node-expat/lib/node-expat.js:4:32)

Has anyone found a solution to this outside of electron?
I've got the exact same issue as the OP but running on an Ubuntu server.

I have same error with node-expat.
<Error: The module '/home/hefc/GIT/xviz/examples/converters/kitti/node_modules/node-expat/build/Release/node_expat.node'
was compiled against a different Node.js version using
NODE_MODULE_VERSION 46. This version of Node.js requires
NODE_MODULE_VERSION 67. Please try re-compiling or re-installing
the module (for instance, using npm rebuild or npm install).>
Could anyone who fixed this issue share the solution ouside elelctron?

I had the same issue
Downgrading the node version from v12.18 to v10.21 and running npm rebuild helped me.

same issue here


chewy:reset fails on parent indexes

Hi all.
I have this lines in my chewy index file.
define_type Relationship, name: 'follower' do
  root parent: 'user', parent_id: -> { subject_id } do
    field :id, type: 'integer'
  end
end

define_type Relationship, name: 'following' do
  root parent: 'user', parent_id: -> { object_id } do
    field :id, type: 'integer'
  end
end

My model:
class Relationship < ActiveRecord::Base
  belongs_to :object, class_name: 'User'
  belongs_to :subject, class_name: 'User'

  update_index('users#follower') { self }
  update_index('users#following') { self }
end

If I clean up elasticsearch and run rake chewy:reset it works fine(all data passed to elastic).
But if I try recreate indexes (run another rake chewy:reset after the first one) it fails with:
NoMethodError: undefined method `[]' for nil:NilClass
/usr/local/bundle/gems/chewy-0.8.3/lib/chewy/type/import.rb:184:in `block in fetch_indexed_objects'
/usr/local/bundle/gems/chewy-0.8.3/lib/chewy/type/import.rb:183:in `map'
/usr/local/bundle/gems/chewy-0.8.3/lib/chewy/type/import.rb:183:in `fetch_indexed_objects'
/usr/local/bundle/gems/chewy-0.8.3/lib/chewy/type/import.rb:28:in `block (2 levels) in import'
/usr/local/bundle/gems/chewy-0.8.3/lib/chewy/type/adapter/active_record.rb:32:in `import_scope'
/usr/local/bundle/gems/chewy-0.8.3/lib/chewy/type/adapter/orm.rb:84:in `import'
/usr/local/bundle/gems/chewy-0.8.3/lib/chewy/type/import.rb:27:in `block in import'

Any ideas what I'm doing wrong? Thanks.

Well, this looks like a bug, which ES version do you have?

2.2.0

Can you try to downgrade to 1.7.x? If it will work, then it seems like 2.2 support was not completely implemented :( And we have to figure out a patch

Yep, it's works fine with 1.7.5.
Thanks. Waiting for patch...

Well, you probably want to patch it by yourself :) This will significantly speed-up the process. It would be great if you will submit a PR

You actually got this fix in master branch:
parent = hit.has_key?('_parent') ? hit['_parent'] : hit['fields']['_parent']
indexed_objects[hit['_id']] = { parent: parent }

instead of:
indexed_objects[hit['_id']] = { parent: hit['fields']['_parent'] }

When you planing release this feature? =)

Well, I'll do it soon, could you use master for now instead? Master is quite stable

Yep, I'm planning to do it.
Thanks for your gem, it's really cool =)

Thank you and you are welcome


Dynamic change max value

I am trying to change the gauge max value based on a dynamic input.  I am trying to link this input these two parameters. In the index.js file I am trying to alter staticZone value max: 15, this alters where the gauge color bar starts and ends
staticZones: [{strokeStyle: "#b8b8b8", min: 0, max: 15} ],
and in index.html I am trying to change value  :max-value=15 which changes the maximum value the gauge counts down from. I want to make them both equal to the same value and change based on input from Node-Red.
<v-gauge
   :value="countDownTimeData" unit="seconds"
   :options="optionsCountdownTimer"
   :animation-speed=10
   :min-value=0
   :max-value=15
   :width="widthCountdownTime" />
</td></tr>

I can get the gauge to work fine if I hard code in the number values, but if I try to make them variable then it wont work.
I have tried almost every possible permutation I can think of from v:bind to v:if to using mostache braces{{}} to using functions from within computed and methods, but I can't get anything to work properly.
If you have any tips on how I can change these parameters that would be great.

This is a vue


Modify common.ccf file for proper expression coverage

Signed-off-by: Steve Nelson<EMAIL_ADDRESS>

Added "set_expr_scoring -vlog_short_circuit" to address an issue where expression coverage table may have invalid representation when ternary operators are used for default SOP mode. Changed "set_expr_coverable_statements -all" to
"set_expr_coverable_statements -procassign -event_control -misc" to not include "-contassign" as it was indicating the continuous assignments in ternary expressions weren't covered but they are in branch coverage.

@weicaiyang @cindychip FYI


updating README to reflect GitHub docs

updating README to reflect GitHub docs

Pull Request Test Coverage Report for Build 1712

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 97.854%

Totals

Change from base Build 1711:
0.0%

Covered Lines:
1354

Relevant Lines:
1368

💛  - Coveralls


An issue comment repetitively rises up without reason.

Description
When I click the right button of the mouse on the image recently downloaded from the government website, showing the options about what to do with the file, and not long after this, an issue comment appears without asking for a specific action.
Steps To Reproduce

Go to app files
Click on downloads
Click on the image with the right button of the mouse to show options
Wait
An error message appears
The app opens a second time

Steps to fix
If the app is being used the way that it was programmed for, don't send an error message when clicking to show the options of the files, and don't reopen the app.
Files Version
Versão: <IP_ADDRESS> OS Version: 10.0.19045.2364
Windows Version
Windows 10 22H2 19045.2364
Relevant Assets/Logs
debug.log

This issue should be fixed in v2.4.11.


Can u share your trained best model?

I have trained a lot of hours and get loss about 1.5. and can't defeat perfect online AI agent with black or white.

Indeed, you cannot defeat a perfect brute force AI player, just get a draw.

I use http://connect4.gamesolver.org/ to test my best model, and my model can't play the best action as web AI did.


Copied TRS URLs have an additional forward slash

Describe the bug
(Minor bug, doesn't break functionality). The TRS URLs provided on a workflow's information page have an additional forward slash in the URL. This looks to be caused by the concatenation of the primary descriptor's path with the remainder of the URL text, so for example if a descriptor is at /nested/dockstore.wdl, then the TRS URL will end with /descriptor//nested/dockstore.wdl
A real example is: https://dockstore.org/api/ga4gh/trs/v2/tools/%23workflow%2Fgithub.com%2FDataBiosphere%2Ftopmed-workflows%2FUM_variant_caller_wdl/versions/1.32.0/plain-WDL/descriptor//variant-caller/variant-caller-wdl/topmed_freeze3_calling.wdl (note descriptor//variant-caller/variant-caller-wdl/topmed_freeze3_calling.wdl)
To Reproduce
Steps to reproduce the behavior:

Go to a public workflow, such as https://dockstore.org/workflows/github.com/DataBiosphere/topmed-workflows/UM_variant_caller_wdl:1.32.0?tab=info
Click on the TRS hyperlink
See error in the URL

Expected behavior
The URL shouldn't have the extra forward slash.
Additional context
Webservice - 1.11.10
UI - 2.8.4
Compose Setup - 1.11.1
Deploy - 1.11.2
┆Issue is synchronized with this Jira Story
┆friendlyId: DOCK-2045
┆sprint: Backlog
┆taskType: Story

Implementer: see also #3823

So the endpoint in question is /ga4gh/trs/v2/tools/{id}/versions/{version_id}/{type}/descriptor/{relative_path}
which relates to /ga4gh/trs/v2/tools/{id}/versions/{version_id}/{type}/files
which returns
[
{
"path": "string",
"file_type": "TEST_FILE"
}
]
The path value is either a relative path (probably relative to the main descriptor) or an absolute path determined by the a preceding slash
Removing the preceding slash of https://dockstore.org/api/ga4gh/trs/v2/tools/%23workflow%2Fgithub.com%2FDataBiosphere%2Ftopmed-workflows%2FUM_variant_caller_wdl/versions/1.32.0/plain-WDL/descriptor//variant-caller/variant-caller-wdl/topmed_freeze3_calling.wdl
would result in
https://dockstore.org/api/ga4gh/trs/v2/tools/%23workflow%2Fgithub.com%2FDataBiosphere%2Ftopmed-workflows%2FUM_variant_caller_wdl/versions/1.32.0/plain-WDL/descriptor/variant-caller/variant-caller-wdl/topmed_freeze3_calling.wdl
that currently does not work and would require quite a bit of backend rework.

If that's the case, it seems like the second slash should be encoded. But then for consistency you'd might need to encode other slashes later as well, and overall all the encoding would break clients.


[$1000] Profile- Unable to select invalid file extension images

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Login to an account on NewDot
Navigate to Settings > Profile > Upload photo. Choose an invalid file extension (eg. csv)

Expected Result:
Able to select invalid file extension files
Actual Result:
Unable to select the invalid file types, those are showing disabled
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[x] MacOS / Chrome / Safari
[x] MacOS / Desktop

Version Number: <IP_ADDRESS>
Reproducible in staging?: yes
Reproducible in production?: yes
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://user-images.githubusercontent.com/93399543/234680138-f62e532a-891d-4194-bc6a-854b31a29ca2.mp4
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01c58de038f9a8c30a
Upwork Job ID:<PHONE_NUMBER>299625984
Last Price Increase: 2023-04-26

Triggered auto assignment to @tjferriss (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details.

Bug0 Triage Checklist (Main S/O)

[ ] This "bug" occurs on a supported platform (ensure Platforms in OP are ✅)
[ ] This bug is not a duplicate report (check E/App issues and #expensify-bugs)

If it is, comment with a link to the original report, close the issue and add any novel details to the original issue instead

[ ] This bug is reproducible using the reproduction steps in the OP. S/O

If the reproduction steps are clear and you're unable to reproduce the bug, check with the reporter and QA first, then close the issue.
If the reproduction steps aren't clear and you determine the correct steps, please update the OP.

[ ] This issue is filled out as thoroughly and clearly as possible

Pay special attention to the title, results, platforms where the bug occurs, and if the bug happens on staging/production.

[ ] I have reviewed and subscribed to the linked Slack conversation to ensure Slack/Github stay in sync

Proposal
Posting proposal early as per new guidelines
Please re-state the problem that we are trying to solve in this issue.
It will disable invalid file types when user go for choose the profile image.
What is the root cause of that problem?
To select Profile Image or Workspace image it is using AttachmentPicker, we are passing type={CONST.ATTACHMENT_PICKER_TYPE.IMAGE} prop at line 227
https://github.com/Expensify/App/blob/9ec77046be868425e7ff129aaf1cf20cebe64e37/src/components/AvatarWithImagePicker.js#L277
Eventually type prop used here at line 53,
https://github.com/Expensify/App/blob/9ec77046be868425e7ff129aaf1cf20cebe64e37/src/components/AttachmentPicker/index.js#L53
So it will disable other file type during profile image selection. This is the root cause of the problem.
What changes do you think we should make in order to solve the problem?
We have to remove type prop we are passing to  <AttachmentPicker> component at line 227 as shown below:
 // <AttachmentPicker type={CONST.ATTACHMENT_PICKER_TYPE.IMAGE}>   // *** OLD
  <AttachmentPicker>   // *** Updated code

As default value for type prop is this
https://github.com/Expensify/App/blob/9ec77046be868425e7ff129aaf1cf20cebe64e37/src/components/AttachmentPicker/attachmentPickerPropTypes.js#L26
So if we do not pass type prop then user can choose any file. But it will show invalid image type message via this code:
https://github.com/Expensify/App/blob/9ec77046be868425e7ff129aaf1cf20cebe64e37/src/components/AvatarWithImagePicker.js#L175-L182
So now it will not disable other file type and user can choose any file, but it will show error as invalid file type for profile image if user will choose pdf etc, as shown in result video. So this will solve the issue.
What alternative solutions did you explore? (Optional)
None

Result
https://user-images.githubusercontent.com/7823358/234715525-94b2db74-5600-468d-a334-2a298f55c3f2.mov

Job added to Upwork: https://www.upwork.com/jobs/~01c58de038f9a8c30a

Current assignee @tjferriss is eligible for the External assigner, not assigning anyone new.

Triggered auto assignment to Contributor-plus team member for initial proposal review - @thesahindia (External)

Current assignee @marcaaron is eligible for the External assigner, not assigning anyone new.

📣 @PrashantMangukiya You have been assigned to this job by @marcaaron!
Please apply to this job in Upwork and leave a comment on the Github issue letting us know when we can expect a PR to be ready for review 🧑💻
Keep in mind: Code of Conduct | Contributing 📖

Applied to job on Upwork. I will submit pr within 2 hours asap. Thank you.

Hmm actually I am kind of confused. I am testing this locally and we should not be able to pick these invalid extensions right?
Why would you be able to upload a file as a profile picture?

Gonna close this I think there was a mistake sorry @PrashantMangukiya

@marcaaron  Its ok. Thank you.

Navigate to Settings > Profile > Upload photo. Choose an invalid file extension (eg. csv)
Able to select invalid file extension files

This seems wrong. Someone feel free to reopen if you disagree.


Error building Component HuggingFace: max_new_tokens not found in HuggingFaceEndpointsComponent

Bug Description
Error building Component HuggingFace: max_new_tokens not found in HuggingFaceEndpointsComponent
flow Component:
chat input -> HuggingFace -> chat output
Reproduction
my step:

clone hugging face space:https://huggingface.co/spaces/Langflow/Langflow?duplicate=true
flow

chat input
HuggingFace
input API Token(flow input)
chat output
playground(flow run)

Expected behavior
chat at playground
Who can help?
No response
Operating System
hugging face Space cpu basic
Langflow Version
langflowai/langflow:latest
Python Version
None
Screenshot
No response
Flow File
No response

Getting the same error, Did you get any solution for that?

Yes, we found the solution.
There were some conflicting PR that led to the issue. I am in the process of creating a new PR to sort the issue. Thank you for your patience.

hello @edwinjosechittilappilly, Still getting the issue while running it in huggingface space using dockerfile from github page.
"[Clone the space using this link]" using this link.

It's updated in the main branch, but not yet in releases.
Please use the updated component code (huggingface.py) from the main branch.


Integrate alist with Casdoor

https://github.com/alist-org/alist
See: https://github.com/alist-org/alist/issues/1304

@wenxuan70

PRs:

https://github.com/alist-org/alist/pull/1453
https://github.com/alist-org/web-v2/pull/50


Chaining \sqrt can be used to break sites

Consider the following:
\Huge \sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{\sqrt{}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}}

Someone on my /b/ board just discovered this, it makes JS take up 100% CPU. I've had to emergency disable KaTeX.

We are thinking of using your library on our own large site. But are concerned about these security vulnerabilities. What is the status of these?

@jschatz1 no update.  You could count how many times \sqrt occurs in the input string, but this doesn't protect against other as of yet unknown vulnerabilities.  I think rendering to a string in a web worker's probably the way to go when dealing with possibly hostile input.

I tried the same \sqrt chain with MathJax and it renders it in a fraction of the time.

@kevinbarabash do you mean MathJax renders it faster or KaTeX renders it faster? I find that MathJax renders a low-quality version quickly and then still pegs my CPU to 100% for a while while it renders the high quality version.

@xymostech I pasted the code from the original post into the demo on https://www.mathjax.org/ and it shows up quick and it seems to be the final render.  It's using MathJax's HTML renderer.

I tested putting the giant expression in to the "ask a question" box on math.stackexchange.com/questions/ask, and it indeed also pegs my CPU and freezes the page (they use MathJax, for the record). They seem like a pretty high-profile site, so if they don't have protections against it, I wouldn't be tooo worried.
@kevinbarabash Wow, you're right! Much quicker there for me. Maybe it's a newer version of MathJax? It's speeding up! :)

@DouweM Anysite that allows user input that has to run something through JS would eventually get pegged. GH uses a checkbox list renderer that I am sure if you added a couple thousand markdown checkboxes you could get GH to crash. The question is how many sqrt's make it crash. If it's 5 that's a problem. If it's 9000 that's probably fine.

It seems KaTeX rendering speed and performance have been greatly improved, and the nested \sqrts render instantly. If this is still an issue, please reopen the issue. Thank you!


Index.yaml loses previous versions for private repositories

Describe the bug
When publishing a new chart version, the previous chart version is effectively removed from index.yaml on the gh-pages branches.
To Reproduce

Configure yet-another-chart-releaser-action:

- name: Run chart-releaser
  uses: stecky/yet-another-chart-releaser-action@v1
  env:
    YACR_TOKEN: "${{ secrets.GITHUB_TOKEN }}"
  with:
    charts_dir: chart
    packages_with_index: true
    charts_repo_url: https://raw.githubusercontent.com/<org>/<repo>/gh-pages

Create an initial release observe the index.yaml generated in gh-pages contains this version
Create a second release. Observe that while the tgz file of the previous version is still present, the version has been replaced in index.yaml with this second release

Expected behavior
I would expect to see the second version appended:
apiVersion: v1
entries:
  <chart>:
  - apiVersion: v2
    appVersion: 0.1.9
    created: "2021-05-21T15:20:56.683660178Z"
    description: <description>
    digest: cea83d5b4f6cf7a780c8aec185b221ef0aac791daa43b768c801573b97f395f6
    name: <chart>
    type: application
    urls:
    - <chart>-0.1.9.tgz
    version: 0.1.9
  - apiVersion: v2
    appVersion: 0.2.0
    created: "2021-05-21T15:20:56.683660178Z"
    description: <description>
    digest: 141f0d9770883ad2ba4687e775bf74002ce89fdf605569021662d0aefd146f3f
    name: <chart>
    type: application
    urls:
    - <chart>-0.2.0.tgz
    version: 0.2.0
generated: "2021-05-21T15:23:26.683673678Z"

Additional context
I believe this is due to the fact that this call will return 404 due to lack of credentials: https://github.com/stecky/yet-another-chart-releaser/blob/c5b06f45c3c52f4bd6a4b4b562627252c61e4ce2/pkg/releaser/.go#L112. Consequently, this branch of the if statement is executed: https://github.com/stecky/yet-another-chart-releaser/blob/c5b06f45c3c52f4bd6a4b4b562627252c61e4ce2/pkg/releaser/releaser.go#L137 - i.e. a new empty index file is created, and the previous version is effectively lost.
This confirmed by the output of the GitHub action:
Run stecky/yet-another-chart-releaser-action@v1
Looking up latest tag...
Discovering changed charts since '<chart>-0.1.9'...
Installing yacr v0.3.3...
Adding yacr directory to PATH...
Packaging chart 'chart/jaws-journey'...
Successfully packaged chart in /home/runner/work/<chart>/<chart>/chart/jaws-journey and saved it to: .yacr-release-packages/<chart>-0.2.0.tgz
Releasing charts...
Preparing worktree (detached HEAD 4b43f26)
HEAD is now at 4b43f26 Update index.yaml
[detached HEAD dac0609] Publishing chart package for <chart>-0.2.0
 1 file changed, 0 insertions(+), 0 deletions(-)
 create mode 100644 <chart>-0.2.0.tgz
Pushing to branch "gh-pages"
To https://github.com/<org>/<chart>
   4b43f26..dac0609  HEAD -> gh-pages
From https://github.com/<org>/<chart>
   4b43f26..dac0609  gh-pages   -> origin/gh-pages
 * [new tag]         <chart>-0.2.0 -> <chart>-0.2.0
Executing command git [push ***github.com/<org>/<chart> HEAD:refs/heads/gh-pages]Updating charts repo index...
UpdateIndexFile new index at .yacr-index/index.yaml
Found <chart>-0.2.0.tgz
Extracting chart metadata from .yacr-release-packages/<chart>-0.2.0.tgz
Calculating Hash for .yacr-release-packages/<chart>-0.2.0.tgz
Updating index .yacr-index/index.yaml
Preparing worktree (detached HEAD dac0609)
HEAD is now at dac0609 Publishing chart package for <chart>-0.2.0
[detached HEAD de60b35] Update index.yaml
 1 file changed, 6 insertions(+), 6 deletions(-)
Pushing to branch "gh-pages"
To https://github.com/<org>/<chart>
   dac0609..de60b35  HEAD -> gh-pages
Executing command git [push ***github.com/<org>/<chart> HEAD:refs/heads/gh-pages]

Possible solution
Rather than  trying to GET the index.yaml file directly, we could  use git.AddWorktree to checkout a the branch, and use the existing index.yaml file on that branch. However, I'm not sure if there's a use-case that you'd like to support that would prohibit that. If that does seem a reasonable solution, I'd be happy to provide a PR.

Good catch... thanks.
I think a simpler solution would just be to use the github token as the auth to download the index file.
See what you think... https://github.com/stecky/yet-another-chart-releaser/commit/181ff58007ebab0542ddca634f3fdaab306cfe4c

Great, yes that's much simpler. Thanks!


Fix: avoid useless space allocation in SetBit

when exec command setbit aaa 8180 1 setbit aaa 8191 1 in order ,SetBit allocates 2046 bytes, but index 1024-2046 is useless and will casue waste of disk space.

LGTM, but don't read.


examples/basic/main.py cannot find relate imported modules

export PYTHONPATH='./' could resolve this I'd like to find better solution if possible

This could be resolved by executing the script as module


allow expectation values to be evaluated for QubitPauliOperator with …

Allow Backend.get_expectation_value(self, operator: QubitPauliOperator) to work for QubitPauliOperator with either  Dict[QubitPauliString, float] or Dict[QubitPauliString, sympy.Expr].

@ss2165 @cqc-alec pls review as well.
We need this because in our code we have child of QubitPauliOperator that sometimes returns Dict[QubitPauliString, float]  instead of Dict[QubitPauliString, Expr].
This means the get_operator_expectation_value raises an AttributeError, since python floats do not have .evalf()


Prevent Que adapter from collecting metrics of jobs locked for execution

We need to exclude Que jobs locked for execution when looking up latency
metrics, otherwise they will inflate the latency with the time it takes
to execute those jobs, causing Que latency behavior (and thus,
autoscaling) to be different than other adapters.
References:
Issue: https://github.com/adamlogic/rails_autoscale_agent/issues/42
PR: https://github.com/adamlogic/rails_autoscale_agent/pull/43
Que job stats: https://github.com/que-rb/que/blob/620786dc70676ed1d1ad418171e12decfe42802e/lib/que/utils/introspection.rb#L8-L25
We unfortunately can't rely on their job stats introspection because
that doesn't expose the queue, it groups by job class, but the general
idea to exclude locked jobs was borrowed from that query.

@adamlogic thanks for the catch  For some reason I didn't run into it when first setting up the app, and since I had the DB already, running bin/setup wouldn't fail. However, once I dropped it and tried again, I got the same error as you.
I pushed up a PR on top of this one, changing the schema format to SQL, that appears to fix the issue: https://github.com/judoscale/judoscale-ruby/pull/87


Get() fails with empty Session ID

I'm running multiple webdrivers in parallel. At times, webDriver.Get() is sending requests with empty session id:

2018/06/08 15:58:41 -> POST http://localhost:8888/wd/hub/session//url
{"url":"https://www.amazon.com/dp/B00UG9HB1Q/ref=sr_1_1?ie=UTF8\u0026qid=1528453721\u0026sr=8-1\u0026keywords=B00UG9HB1Q"}
2018/06/08 15:58:41 <- 404 Not Found [[application/json;charset=utf-8]]
{
"sessionId": "",
"value": {
"error": "invalid session id",
"message": "No active session with ID ",
"stacktrace": ""
},
"status": 6
}`

Can someone take alook at this for me, or drop a hint as to what is wrong? I can share the code with you if you think this my mistake.

The program runs fine for a while, and then this condition comes. Then the condition stays, even on program restarts.
Here is the part of my code dealing with selenium: http://paste.debian.net/1028460/


Pentest iOS - Unsafe usage of UIWEBVIEW

Unsafe usage of UIWEBVIEW
UIWebView's WebKitAllowUniversalAccessFromFileURLs and WebKitAllowFileAccessFromFileURLs are
enabled by default, thus permitting both the access to the files in the sandbox through js and their silent
upload to the remote end An attacker who successfully exploited the vulnerability could remotely access
all local file system content in the mobile app sandbox, including browser cookies, user profiles and
documents.
There are two kinds of Webview under iOS: UIWebView and WKWebView. UIWebView is not
recommended after iOS 8 but they are still used by Hexa in payload/hexa.app/hexa

This is not an issue anymore. We are not displaying any content in web view. The RN Upgrade ensured that we are compliant.


Unable to ssh to bigip using null resource launch site module

Hi Team,
Using terraform terraform scripts i am able to deploy and configure bigip on public releases but when tried same on latest private builds scripts are failing to connect to ssh (null resource launch site module)
Working build AMI: "F5 BIGIP-<IP_ADDRESS>-0.0.10 PAYG-Adv WAF Plus 200Mbps-211222202459" and "ami-04f64528e4ec2e338"
Facing issue AMI: "F5 BIGIP-tmos-bugs-staging-17.0.0-0.0.8316 PAYG-Adv WAF Plus 200Mbps-220312044158" and "[ami-07ea51ee96ce064ff] and others private builds.
Error: timeout - last error: SSH authentication failed (admin@<IP_ADDRESS>:22): ssh: handshake failed: ssh: unable to authenticate, attempted methods [none keyboard-interactive], no supported methods remain
File links:

User data: https://gitswarm.f5net.com/api-security/apisecurity/-/blob/main/tests/declarative-advanced-waf-policy/terraform/f5_onboard.tmpl
https://gitswarm.f5net.com/api-security/apisecurity/-/blob/main/tests/declarative-advanced-waf-policy/terraform/main.tf

If unable to access let me know i will give access permissions.
Terraform version used: "0.14.4"
f5networks/bigip versions: "1.8.0"
Let me know if any other things needed..

Please reach out if you are facing the issue.

Closing this now. Reopen if you still face the issue. Thanks!

@shshaik FYI..


join_conference_voice_video_#454

Should fix https://github.com/vector-im/vector-android/issues/454
Let users join confs as voice or video

:+1:  LGTM


Create the Ithaca release for raiden-contracts

This can be better kept track in the Ithaca milestone.


Sprite frame textbox arrows and slider dont work

Operating system or device - Godot version:
Godot 2.1.1 - Windows 10
Issue description:
The up/down arrows next to "Frame" on a sprite node don't work. The slider that appears when you select its textbox doesn't work either.
Steps to reproduce:
Create a scene containing only a 2d sprite node

Load in any spritespritesheet on the Sprite Node for its texture (I used a png file)
Set "Hframes" and/or "Vframes" to some value (eg. 2 Hframes, 2 Vframes)

Error 1) In the "frame" textbox type a number and hit enter. The sprite on screen will change, but the textbox still says that you're on frame "0"

Error 2) If you click the up and down arrows as part of the sprite textbox, they don't change the sprite frame

Error 3) If you click the frame textbox, it pops up a slider for changing the frame, but the slider is missing the handle to grab to change frames.

I think I haven't seen this in 2.2/master, can someone confirm for 2.1.2 though?

Just downloaded 2.1.2 - all three bugs confirmed as present in 2.1.2

Will get rid of that slider for 3.0
On Jan 24, 2017 9:10 AM, "balloonpopper"<EMAIL_ADDRESS>wrote:

Just downloaded 2.1.2 - all three bugs confirmed as present in 2.1.2
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/godotengine/godot/issues/7624#issuecomment-274787170,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AF-Z2xX3RAMNrdym2qM0XmBp7r1ys52Fks5rVepOgaJpZM4LsDHb
.

Will we still be able to choose the frame though? Currently the text field doesn't change from "0" when you type a different number in.

Ah, actually, I remember this not working in earlier versions...

Will we still be able to choose the frame though?

@balloonpopper yes, the text field does not update correctly, but the property is updated and you should see the selected frame in the 2D view (assuming of course frame < hframes*vframes).
If you reload the scene in the editor (eg. by switching tabs) you will see the updated value in the inspector and surprisingly enough, both the slider and the arrow will work up to the hframe*vframe values at reload.
So a temporary solution until bug is fixed, is to:

Set hframe and vframe
Reload scene
Now you can use slider, arrows, and manual input.
If you change either hframe or vframe go back to 2

@bojidar-bg I can confirm the bug also affects master and 2.2.

@Faless - Thanks for the information on the workaround - that'll be helpful! The frame question though was in relation to reduz saying the slider will be removed for v3. I was looking to confirm whether only the slider will be removed for v3, or whether the ability to type the frame number in in the interface would be removed as well. Hopefully it's just the slider.

I'm pretty sure he meant the slider when editing int/floats in general. The property to set the frame number will be there.
BTW, I noticed it's enough to just reload the inspector pane (i.e. selecting another node and back) instead of reloading the whole scene

Ah, again the cool effect with the Inspector not reloading.... :laughing:


[Docs]Import Docker-Compose Files into Meshery

Current State
There is currently no documentation on the Layer5 site for importing Docker Compose files into Meshery.
Desired State
A new guide is needed, following a similar format to the existing Starting Helm documentation.
The new documentation on importing Docker Compose files into Meshery will cover both methods: using the Meshery CLI and the Meshery UI. It will follow the structure and style seen in the Importing Designs
This tutorial will also be included in Meshery Docs replacing the Docker-compose link there.

Contributor Guide and Resources

📚 Instructions for contributing to documentation

Layer5 documentation site and source

🎨 Wireframes and designs for Layer5 site in Figma (open invite)
🙋🏾🙋🏼 Questions: Layer5 Discussion Forum and Layer5 Community Slack

Yes, I am here to help! Ask away on Slack!

This issue is related to #352, which covers importing Kubernetes Manifest, Docker-Compose, and Meshery Designs into Meshery.


[Beta] Could not find module 'ember-new-computed'

Currently getting Uncaught Error: Could not find module 'ember-new-computed' imported from 'ember-intl/services/intl' in a dummy-app (https://github.com/al3x-edge/ember-intl-beta1-dummy-app).

Fixed, use beta.4 at the moment