Unable to recreate a container if it was attached to multiple networks

Description
I had a container which was attached to two networks. When I clicked the "Recreate" button (in order to update it to the last image) it stopped in the middle of the recreation, and an error message appeared, complaining about a container being unable to be connected to two network endpoints. When I went back to the container list, the original one was gone.
The error message was:

Container cannot be connected to network endpoints: bridge, <custom_network>

Steps to reproduce the issue:

Create a network (apart from bridge).
Create a container attached to the bridge network. An alpine one will be enough, even creating it without any run command defined.
Once the container has been created, attach it to the second network.
Click the recreate button.

Any other info e.g. Why do you consider this to be a bug? What did you expect to happen instead?
I would have expected the following things:

In case of an error happening in the middle of a recreation, the original container should continue to exist, even if it had been stopped (which it should be in order to bind the ports, if any).
If there are multiple networks and only one can be joined during creation (as portainer offers), the others should be joined after the container is recreated

I did not test it with previous versions
Technical details:

Portainer version: 1.15.4
Target Docker version (the host/cluster you manage): 17.05.0-ce
Platform (windows/linux): Ubuntu 16.04.3 LTS
Command used to start Portainer (docker run -p 9000:9000 portainer/portainer): docker run -p 9000:9000 -v /var/run/docker.sock:/var/run/docker.sock -v {CUSTOM_PATH}:/data portainer/portainer
Target Swarm version (if applicable):
Browser: Google Chrome 65.0.3290.0 (Official Build) canary (64 bits) on macOS High Sierra

I believe that the root cause of this issue is the fact that a container cannot be connected to multiple networks at creation time.
The following flow should be used here:

Remove the previously existing container
Create a new container with the same configuration as well as one of the defined networks
Post-creation, connect the container to the extra networks

In case of an error happening in the middle of a recreation, the original container should continue to exist, even if it had been stopped (which it should be in order to bind the ports, if any).

Due to the fact that container names are unique and we rely on the container name as an identifier to check if we need to replace the container we cannot keep the previous container, it has to be removed.
We might need to review the entire edition flow and try to figure out a way to keep the previously created container in case of error.

Due to the fact that container names are unique and we rely on the container name as an identifier to check if we need to replace the container we cannot keep the previous container, it has to be removed.

What about:

Stop the existing container
Create a new container with the same configuration as well as one of the defined networks with the same name as the previous one with a known difference, such as a "__" prefix, or a "-copy" suffix
Post-creation, connect the container to the extra networks
Delete the previously existing container
Rename the new one to the name it should have

I believe that the container re-creation workflow presented above by @dexafree should be implemented to solve many container re-creation issues.

Why can't it be have like the duplicate/edit button of showing a error but recreating the container just sans all except one network? Why have the extra steps of renaming and removing the old containers?

@poVoq there is no way to re-create a container in the Docker API. It has to be replaced by a new container, hence the requirement of removing the actual container and creating a new one.
The new re-creation flow would prevent the removal of the existing container upon creation of the new one, which would avoid the loss of any container. If the new container is created successfully, we consider the re-creation process to be ok, hence no need to keep the old container and it will be removed.

I just lost 2 containers because of this bug, i hope it can be fixed soon, the same with the ability to connect it to multiple networks because it's like the core functionality that replaces deprecated --link.
Also not to lose container on error would be nice, for example, maybe rename it.
Thanks for good work overall :)


fix(flask): missing fig_json error

related to #428
Problem
When no graph is generated, retrieving a converstation through the history fails as the cache is expecting a fig_json field.
The following error is displayed:

Solution
set fig_json as an optional field instead of a required one for the function VannaFlaskApp.load_question. The conversation is then displayed, followed by an empty graph:

Improvements
It would be good to take into account the possibility of fig being null in the JS displaying the graph so that it shows nothing instead of an epty graph, but as it is in the assets I can't modify it.

Yes, it makes sense for the front-end not to display anything when fig is empty. How should I adjust this code to solve it? It’s a bit troublesome.”

@AI-Mart in the mean time, you can disable all graph generation by overiding the methods responsible for generation:
import pandas as pd
from plotly import Figure

from vanna.remote import VannaDefault

class VannaModel(VannaDefault):

    def generate_plotly_code(
        self, question: str = None, sql: str = None, df_metadata: str = None, **kwargs
    ) -> str:
        return ""

    def get_plotly_figure(
        self, plotly_code: str, df: pd.DataFrame, dark_mode: bool = True
    ) -> Figure:
        return Figure()

    def should_generate_chart(self, df) -> bool:
        return True

“If there is no chat history information, then instead of showing a blank image, nothing should be displayed at all. Currently, I have set the parameter ‘chart=False’; however, it seems that when ‘fig’ is empty,chat history information  it still shows a blank image instead of nothing.”This seems to be a front-end issue, as the backend interface is no longer generating the image.

Hello, similar error is displayed for df.

I assume similar approach can be applied for fixing it as for fig_json.

@Uzda20 it is the same kind of error (cache issue) but definitely not the same source, and not the same way to fix it, because it is not supposed to not be set. If you want to adress it, please create a new issue with details about how to reproduce this error

@Molrn Thanks, I will try to reproduce it and add issue description as advised.


Question: how to reduce spaces between icon and name in lualine component?

Currently there are two spaces between symbol and name which I would like to reduce to one. I couldn't find an option in the config.

actually, i realised I was passing extra space in the config


adding more child docs to book

added the rest of the child docs from Getting Started to the collection of modules listed in the AnVIL_Template book
(also deleted a line from the 02-chapter-of-course.Rmd that was making it show up as two separate chapters, which was weird and confusing)
I put all the billing stuff into one "chapter" for now, but I imaging "Billing" might get broken up if we add more modules (e.g. on creating and managing groups), to keep things easily findable.
As far as organization - It'll be a huge pain to rename or reorganize the child documents themselves once other books start pointing to them, since it'll break the import links.  But, it's no problem at all to reorganize how they're presented in the AnVIL_Template "book".  So I'm thinking we keep the /child directory flat, and use the AnVIL_Template book to keep things organized.

oh, just noticed I need to fix the "spelling errors" (add stuff to dictionary).  I'll take care of that.

Sounds good to me!

So I'm thinking we keep the /child directory flat, and use the AnVIL_Template book to keep things organized.

Do you mean that child/ is where we find everything and there are no subdirectories?

So I'm thinking we keep the /child directory flat, and use the AnVIL_Template book to keep things organized.

Do you mean that child/ is where we find everything and there are no subdirectories?

Yeah, rather than having, say, a "billing" or "workspaces" directory.  Then (a) we can change our minds about how we group content together, and (b) if we really want, we can stick a piece of content in multiple categories (by listing it in multiple chapters).


Fix code coverage generation and upload

Current coverage is 73.25%

Branch #37 has no coverage reports uploaded yet.

No diff could be generated. No reports for master found.

Powered by Codecov. Updated on successful CI builds.

✨


Display Errors inline

Hi, I want to show LaTex that is only delimited by $$, how can I do it? Because it shows this :

Can you provide some additional information? (At least show the LaTeX)

Ich & er/sie/es
and
Wir & Sie/sie
In:
<math:TextView
    Grid.Column="1"
    Margin="5,0"
    FontSize="12"
    LaTeX="{Binding Associa, Mode=TwoWay}"
    TextColor="{DynamicResource TextColor}"
    VerticalOptions="CenterAndExpand" />

You have to add \ before & : \&
See: How to type & in LaTeX?

Thank you so much !


ui-router, ui-view and ui-layout not playing nicely :)

I have a ui-view which can have 2 routes. the first is a page that has ui-layout defined, the second is a straight forward html page
<toolbar>stuff</toolbar>
<ui-view></ui-view>

If I navigate to each page directly, then each page renders well. However, I have the splitter bars overlapping my toolbar. However, if I use the toolbar to navigate to the page in question, I get the following
typeError: Cannot set property 'collapsed' of undefined
at uiLayoutCtrl.ctrl.removeContainer (ui-layout.js:357)
at HTMLDivElement.<anonymous> (ui-layout.js:768)
at HTMLDivElement.jQuery.event.dispatch (jquery.js:4435)
at HTMLDivElement.jQuery.event.add.elemData.handle (jquery.js:4121)
at Object.jQuery.event.trigger (jquery.js:4350)
at jQuery.fn.extend.triggerHandler (jquery.js:4907)
at jQuery.cleanData (angular.js:1773)
at Function.cleanData (jquery-ui.js:341)
at jQuery.fn.extend.remove (jquery.js:5258)
at cleanupLastView (angular-ui-router.js:3931)

and all hell breaks loose :(
any quick fix ?
thanks

Sounds like a big problem.  Can you create a reproduction JSBin or Plunkr?

ok, so after hours of getting frustrated because I couldn't reproduce it on plunkr, I've finally got the problem licked ... but still can't reproduce on plunker.
Turns out I had a  in a wrong place. Removed it, all works fine. I can't understand, but worse, can't reproduce ...

Glad you got your problem fixed.  If you ever do manage to repro, I'll reopen.


[BUG] Random View element with key '$tail_spacer', when I use completeCustomComponent

To Do First

[✓] Did you try latest release?
[✓] Did you look for existing matching issues?

Software Package Version:
Describe the bug
When I open the masonry list (that uses completeCustomComponent) page, it's a bit offset to the left. When I scroll down a bit, it gets fixed. When I opened the debugger, I notice that there's some  appearing, but only when I'm at the top of the page. When I scroll down a bit, it's gone, and when I go back top, it reappears.
To Reproduce

Open masonrylist.
Use completeCustomComponent.

Expected behavior
I surely didn't expect that View element to be there.
Screenshots
Screenshot of debugger
At the top it's like this
Lower like this
Version (please complete the following information):

Device: iPhone 7 and redmi 7 (basically all devices)
react-native-cli: 2.0.1
react-native: 0.59.8

Additional context
Also, forgot to mention, I use expo.
<MasonryList
        spacing={10}
        listContainerStyle={StyleSheet.flatten([t.justifyBetween, t.contentBetween, t.mT2, t.pX6])}
        width={SCREEN_WIDTH}
        sorted
        itemSource
        completeCustomComponent
        images
      />

This is the params I use for masonrylist.

This bug happens when I pass more than 41 elements to images prop.


Performance Issue

there is a performance problem when used as a price filter.
It ends up creating a large dom tree(hundreds of nodes more so when there are more results) and multiple re-renders of the page is ending up hogging memory (like searching a few more times on the page took memory usage of my chrome tab close to 1gb). disabling it steadies memory usage at atound ~200mb most of the time.

Can you share the code you're using that uses rheostat?


Shaqib habib

This is my completed Sprint Challenge! Thank you!

Final Edits Sprint Project
11:53 am
Im super happy I worked more on it during lunch.


Implementation of #26 - Widget that displays all (including inactive) forums

What this PR does
This PR introduces a new fully dynamic widget for displaying every forum as an entry in a table. (fixes #26)
The widget follows the new design philosophy specified on our figma.
Also

I created a reusable component called ControlgroupComponent to act as the container for multiple widgets that are yet to come.
I found multiple issues implemented with #17 and fixed them.

Create a module for displaying forums

Merged #43 and #44 so that we can see correct test results for this PR.


Re-enable ppc64le

Hi! This is the friendly automated conda-forge-webservice.
I've started rerendering the recipe as instructed in #71.
If I find any needed changes to the recipe, I'll push them to this PR shortly. Thank you for waiting!
Here's a checklist to do before merging.

[ ] Bump the build number if needed

Fixes #71

@conda-forge-admin, please rerender


Get the data in a .csv format and add it to the repo

Jenny uploaded the data files and relevant notebooks to the repo. So, I suggest to put this ticket to Done.


Accessing BERT encoding layer output.

How can I access the BERT encoding layer output (not the prediction output)?

You can access them in the model within fast-bert.

Thank you! Which file? Sorry I am a bit new to Pytorch too. So any code helpful code snippet would be appreciated.


Can you add the application to winget

Can you add your application to winget "Microsoft's package manager" ?

@AbdelrahmanBayoumi
https://github.com/microsoft/winget-pkgs/issues/21079


Extension Host terminated unexpectedly

VSCode Version: Code 1.19.3 (7c4205b5c6e52a53b81c69d2b2dc8a627abaa0ba, 2018-01-25T10:36:34.867Z)
OS Version: Windows_NT x64 6.3.9600
Extensions: the listing length exceeds browsers' URL characters limit

Steps to Reproduce:

Once I open vscode, the error pops up and I'm unable to do anything
Also, I have disabled Typescript Hero and nothing has changed

Reproduces without extensions: Yes/No

Please, I'd like a solution to the issue "Extension Host terminated unexpectedly", vscode only works when I'm connected to the internet, can't use it offline. Didn't have this issue with the previous version.

@stizzle123 IMHO, this is a problem with one of the extensions you have installed. Does this issue reproduce when disabling extensions?
To troubleshoot the start-up of the extension host, can you please open F1 > Show logs... > Extension Host and look for the last messages before the extension host crashes. Also, you can run F1 > Developer Tools and check the console.


ci: update setup-python unsupported version

python 38 and 39 are not supported on macos-latest anymore, see more detail at
https://github.com/actions/setup-python/issues/696#issuecomment-1637587760

Brief Summary of The Change

Pull Request checklist
I confirm that the following checklist has been completed.

[x] Add/Change test cases for the changes.
[x] Add/Change the related documentation, should also change docs/source/config.rst when you change file default_config.yaml.
[x] (Optional) Add your change to UPDATING.md when it is an incompatible change.

@SbloodyS fast 🚀


Feature request: API to get the real FW version

Currently, trying to call sceKernelGetSystemSwVersion will return the spoofed version from henkaku. Some homebrew might want to know the real firmware version, but all solutions imply creating a kernel module to get the real version from there.
It would be nice if henkaku could expose another API to get the real FW version. Or maybe we could use the same sceKernelGetSystemSwVersion API but with a special flag, for example setting the most significant bit of size to 1.

_vshSblGetSystemSwVersion?

Great! Thank you!

you can also check the os0: file headers
ex:
os0:psp2bootconfig.skprx
3.01 0x94 = 01 03
3.50 0x94 = 50 03
3.60 0x94 = 60 03
3.61 0x94 = 61 03
3.63 ox94 = 63 03
3.65 0x94 = 65 03
3.67 0x94 = 67 03
3.68 0x94 = 68 03
the same goes for vs0:
vs0:app/XXXXXXXXX/eboot.bin
3.01 0x94 = 01 03
3.50 0x94 = 50 03
3.60 0x94 = 60 03
3.61 0x94 = 61 03
3.63 ox94 = 63 03
3.65 0x94 = 65 03
3.67 0x94 = 67 03
3.68 0x94 = 68 03

I'm re-opening this as I think it could be a good addition for safe homebrew. Currently these methods all require unsafe access.

ye same


Fix gap in UAlert when using description slot

🔗 Linked issue
Fixes #831
❓ Type of change

[ ] 📖 Documentation (updates to the documentation or readme)
[x] 🐞 Bug fix (a non-breaking change that fixes an issue)
[ ] 👌 Enhancement (improving an existing functionality)
[ ] ✨ New feature (a non-breaking change that adds functionality)
[ ] 🧹 Chore (updates to the build process or auxiliary tools and libraries)
[ ] ⚠ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description

Moving the v-if to the parent div removes it when the condition is not met resulting in only one child in the container. Hence, the gap-3 is not applied anymore and the issue is solver
📝 Checklist

[x] I have linked an issue or discussion.
[ ] I have updated the documentation accordingly.

Closing in favor of https://github.com/nuxt/ui/commit/9a1a1b8caf1c040c458230458b9fa9cbfb32a1bb. This wasn't taking the closeButton into account.


Econet component broken in 0.89

Home Assistant release with the issue: 0.89
Last working Home Assistant release (if known): 0.88
Operating environment (Hass.io/Docker/Windows/etc.): Python 3.5.3 on Debian 9.8
Component/platform: https://www.home-assistant.io/components/water_heater.econet/
Description of problem:
Component no longer loads.
Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant):
- platform: econet
  username: !secret econet_email
  password: !secret econet_pass

Traceback (if applicable):
2019-03-07 08:26:39 ERROR (MainThread) [homeassistant.components.water_heater] Error while setting up platform econet
Traceback (most recent call last):
  File "/usr/local/lib/python3.5/dist-packages/homeassistant/helpers/entity_platform.py", line 128, in _async_setup_platform
    SLOW_SETUP_MAX_WAIT, loop=hass.loop)
  File "/usr/lib/python3.5/asyncio/tasks.py", line 400, in wait_for
    return fut.result()
  File "/usr/lib/python3.5/asyncio/futures.py", line 293, in result
    raise self._exception
  File "/usr/lib/python3.5/concurrent/futures/thread.py", line 55, in run
    result = self.fn(*self.args, **self.kwargs)
  File "/usr/local/lib/python3.5/dist-packages/homeassistant/components/water_heater/econet.py", line 66, in setup_platform
    from pyeconet.api import PyEcoNet
  File "/root/.homeassistant/deps/lib/python3.5/site-packages/pyeconet/api.py", line 6, in <module>
    from .equipment.water_heater import EcoNetWaterHeater
  File "/root/.homeassistant/deps/lib/python3.5/site-packages/pyeconet/equipment/water_heater.py", line 133
    error = f"Invalid set point. Must be <= {self.max_set_point} and => {self.min_set_point}"
                                                                                ^
SyntaxError: invalid syntax

Additional information:

CC @w1ll1am23: The pyeconet package is incompatible with python 3.5 - format strings were introduced in py3.6 - HA currently has 3.5 as the lowest supported version.

Oh, sorry about that. I'll get it fixed later tonight.

@w1ll1am23 No problem

This is still broken in 0.89.1:
2019-03-08 09:27:47 ERROR (MainThread) [homeassistant.components.water_heater] Error while setting up platform econet
Traceback (most recent call last):
  File "/usr/local/lib/python3.5/dist-packages/homeassistant/helpers/entity_platform.py", line 128, in _async_setup_platform
    SLOW_SETUP_MAX_WAIT, loop=hass.loop)
  File "/usr/lib/python3.5/asyncio/tasks.py", line 400, in wait_for
    return fut.result()
  File "/usr/lib/python3.5/asyncio/futures.py", line 293, in result
    raise self._exception
  File "/usr/lib/python3.5/concurrent/futures/thread.py", line 55, in run
    result = self.fn(*self.args, **self.kwargs)
  File "/usr/local/lib/python3.5/dist-packages/homeassistant/components/water_heater/econet.py", line 66, in setup_platform
    from pyeconet.api import PyEcoNet
  File "/root/.homeassistant/deps/lib/python3.5/site-packages/pyeconet/api.py", line 6, in <module>
    from .equipment.water_heater import EcoNetWaterHeater
  File "/root/.homeassistant/deps/lib/python3.5/site-packages/pyeconet/equipment/water_heater.py", line 133
    error = f"Invalid set point. Must be <= {self.max_set_point} and => {self.min_set_point}"
                                                                                ^
SyntaxError: invalid syntax

@danodemano looks like your install failed to upgrade the pyeconet version to 0.0.9 can you try another reboot and see if it works? If it still doesn't look in your logs towards the top to see if it calls out trying to install pyeconet.

@w1ll1am23 - just bounced the whole box and it's still not loading.  There is nowhere in the log where I See it trying to install pyeconet:
2019-03-08 09:50:36 WARNING (MainThread) [homeassistant.loader] You are using a custom component for illuminance.sensor which has not been tested by Home Assistant. This component might cause stability problems, be sure to disable it if you do experience issues with Home Assistant.
2019-03-08 09:50:36 ERROR (MainThread) [homeassistant.loader] Unable to find platform alexa. Search path was limited to path of component: homeassistant.components
2019-03-08 09:50:38 ERROR (MainThread) [homeassistant.components.hue] Error connecting to the Hue bridge at <IP_ADDRESS>
2019-03-08 09:50:38 WARNING (MainThread) [homeassistant.config_entries] Config entry for hue not ready yet. Retrying in 5 seconds.
2019-03-08 09:50:39 WARNING (Thread-22) [homeassistant.components.sensor.darksky] Monitored condition temperature_max is deprecated
2019-03-08 09:50:39 WARNING (Thread-22) [homeassistant.components.sensor.darksky] Monitored condition temperature_min is deprecated
2019-03-08 09:50:39 WARNING (Thread-22) [homeassistant.components.sensor.darksky] Monitored condition apparent_temperature_max is deprecated
2019-03-08 09:50:39 WARNING (Thread-22) [homeassistant.components.sensor.darksky] Monitored condition apparent_temperature_min is deprecated
2019-03-08 09:50:39 ERROR (MainThread) [homeassistant.components.water_heater] Error while setting up platform econet
Traceback (most recent call last):
  File "/usr/local/lib/python3.5/dist-packages/homeassistant/helpers/entity_platform.py", line 128, in _async_setup_platform
    SLOW_SETUP_MAX_WAIT, loop=hass.loop)
  File "/usr/lib/python3.5/asyncio/tasks.py", line 400, in wait_for
    return fut.result()
  File "/usr/lib/python3.5/asyncio/futures.py", line 293, in result
    raise self._exception
  File "/usr/lib/python3.5/concurrent/futures/thread.py", line 55, in run
    result = self.fn(*self.args, **self.kwargs)
  File "/usr/local/lib/python3.5/dist-packages/homeassistant/components/water_heater/econet.py", line 66, in setup_platform
    from pyeconet.api import PyEcoNet
  File "/root/.homeassistant/deps/lib/python3.5/site-packages/pyeconet/api.py", line 6, in <module>
    from .equipment.water_heater import EcoNetWaterHeater
  File "/root/.homeassistant/deps/lib/python3.5/site-packages/pyeconet/equipment/water_heater.py", line 133
    error = f"Invalid set point. Must be <= {self.max_set_point} and => {self.min_set_point}"
                                                                                ^
SyntaxError: invalid syntax

@danodemano I am super sorry, just checked the pyeconet code and the commit contains the change log update, the setup.py update, but not the actual fix! I must not have saved the file before running the commit. I will have to push a new version later today. While you are waiting for the next release you could modify  /root/.homeassistant/deps/lib/python3.5/site-packages/pyeconet/equipment/water_heater.py line 133 and remove the error logging.

@w1ll1am23 - no worries, at least I know I'm not going crazy!  That got it though, I commented out lines 132, 133, and 134 then bounced HA and it came back up.  Thanks so much!!

I know this is closed,  but is there a fix for this comming?  im on 89.2  and Im getting the same error.

@neuman1812 - Yeah but it's not made it into a release yet.  #21837 fixes it.  If you look through the above conversation there is a manual workaround you can apply for the time being if you would like.


Polish for release

~Automatically analyze all files or prompt on project config changed~: not going to fix now (not even available in Understand on project open)
[x] Make an account<EMAIL_ADDRESS>with identifier scitools
[x] Remove all right-click menu items that might not work
[x] Add all features to documentation
[x] Fix code action lightbulb on wrong line (actually, the lightbulb is supposed to be above, but the happens with left-clicking anywhere but the left of the token because the client is inconsistent with sending back diagnostics)
[x] Make sure that references and others have commands
[x] Use setContext to make a new context called understandFile to narrow down available commands
[x] Remove duplicate violations (https://github.com/stinb/sti/issues/3146)
[x] Wait for the verified blue checkmark on the marketplace (should be at latest Tuesday, Feb 6)
[x] Automatically analyze all files or prompt on database wrong version
[x] Add another release (https://github.com/stinb/UnderstandForVSCode/releases/tag/v0.9.2-beta)
[x] Have Kate test it
[x] https://github.com/stinb/UnderstandForVSCode/issues/4
[x] https://github.com/stinb/UnderstandForVSCode/issues/7
[x] Make this repo public, fix images in README, remove baseImagesUrl from Makefile and wiki
[ ] Have Heidi assign others to test it

@RobbyCBennett I made the repo public to help with some testing issues.

Okay. Since you did that, I just fixed the README so that the images are visible in the editor. That's only possible when the repo is public. I released it as 0.9.4, which is the latest beta release right now.

Now it's fixed again, for real, in case you downloaded it in the last 6 minutes.

I will release to the VS Code extension marketplace once we push our next release of Understand.

Published


Error in rsync protocol data stream, please review your config file.

Okay,
I've followed the steps, and sometimes this just works like a charm, and other times it gives me headaches galore. (And I've tested atomic-sync and the atom-sync when this troubles start)
ssh works fine, rsync works fine from the cli, but sync log shows:
Ready
Syncing from /Users/hvisage/Work/GlobalRemit/dotransaction-mysql/doTransaction.sql to hvisage@uat-ferari:/home/ferari/dev/dotransaction-mysql ...
Error in rsync protocol data stream, please review your config file.

Syncing from /Users/hvisage/Work/GlobalRemit/dotransaction-mysql/doTransaction.sql to hvisage@uatferari:/home/ferari/dev/dotransaction-mysql ...
Error in rsync protocol data stream, please review your config file

Updating rsync to the latest version did the trick for me. See issue #43

Which version of rsync fails and which version works? Issue 43 doesn't show version numbers.
Any MacOSX specific issues then with RSYNC?
Why does it work in a terminal, but not inside Atom?
I have a bigger suspicion its related to the non-use of ~/.ssh/config which I heavily relies on

@guyvanbael IT's not the *newer* rsync, but the *older* rsync that works O_o
And in this case the Brew installed version 3.1.2 makes atom-sync barf while  the MacOSX 2.6.9 version does work quite fine...
The version that failed:
BlackYos% which rsync
/usr/local/bin/rsync
BlackYos% rsync --version
rsync  version 3.1.2  protocol version 31
Copyright (C) 1996-2015 by Andrew Tridgell, Wayne Davison, and others.
Web site: http://rsync.samba.org/
Capabilities:
    64-bit files, 64-bit inums, 64-bit timestamps, 64-bit long ints,
    socketpairs, hardlinks, symlinks, IPv6, batchfiles, inplace,
    append, ACLs, xattrs, iconv, symtimes, no prealloc, file-flags

rsync comes with ABSOLUTELY NO WARRANTY.  This is free software, and you
are welcome to redistribute it under certain conditions.  See the GNU
General Public Licence for details.
BlackYos% brew remove homebrew/dupes/rsync
Uninstalling /usr/local/Cellar/rsync/3.1.2... (8 files, 749.0K)
BlackYos% which rsync
/usr/bin/rsync
BlackYos% rsync --version
rsync  version 2.6.9  protocol version 29
Copyright (C) 1996-2006 by Andrew Tridgell, Wayne Davison, and others.
<http://rsync.samba.org/>
Capabilities: 64-bit files, socketpairs, hard links, symlinks, batchfiles,
              inplace, IPv6, 64-bit system inums, 64-bit internal inums

rsync comes with ABSOLUTELY NO WARRANTY.  This is free software, and you
are welcome to redistribute it under certain conditions.  See the GNU
General Public Licence for details.
BlackYos%

just upgraded to osx sierra and rsync 312... atom doesn't work at all. Do you have a solution to revert to rsync 2.6.9?

Had the same problem, turns out many versions of rsync don't appear to support the -z option anymore and instead expect -zz. I don't know yet how to enable compression again, but you can work around the problem by changing the flags in your .sync-config.cson from avzpur to avpur.
option:
  flags: 'avpur'


sns.barplot in EDA

i think instead of
ax = sns.barplot(list(tags), list(tag_counts))
it should be
ax = sns.barplot(x=list(tags), y=list(tag_counts))
in code at https://madewithml.com/courses/mlops/exploratory-data-analysis/

What i want to say, is that your code doesn't work for me:
`` gives an error:
TypeError: barplot() takes from 0 to 1 positional arguments but 2 were given
That's probably because x is not the first argument to barplot, it's data - https://seaborn.pydata.org/generated/seaborn.barplot.html

Hi @gexahedron, hmm I'm able to run through all of the code and it produces the plot I'm looking for. Does the plot look different when you run your code compared to what's in the lesson?

Hi @gexahedron, looks like I had a previous version of seaborn that wasn't producing this error but now I see what you see! Thank you for bringing this to my attention and I'll explicitly call out x and y arguments for all plotting functions.


Blank file ecmp, rdp

Hi Luca,
After downloaded and build the file, the ecmp and rdp files are blank. Is it intentionally left blank?

I noticed that the ecmp and rdp files are from different depo. So I guess I should download them in the original file


Location Model

Location:
name :: String
address :: String
city :: String
state :: String
zip :: String(6)
Missing anything?

See: cc76306a4db9cf117aed63a387713955b2ce33e5

See #15.


[APPM-1602] Move 'EnabledAssetTypeList' configuration in app-manager.xml as a tenant configuration

Move the property to appmgt-tenant-conf.xml and rename the property name to 'EnabledAssetTypes'Reference: https://wso2.org/jira/browse/APPM-1602

Not related to IoT server, hence closing the issue.


EC_POINT_point2oct/EC_POINT_oct2point asymmetry (fixed in LibreSSL)

My fuzzer found an issue where for some points, EC_POINT_point2oct() succeeds, but the reverse operation using EC_POINT_oct2point() fails.
Fixed in LibreSSL: https://github.com/openbsd/src/commit/eec146bd45a3fe2bbd65928cb2797588727ad017

@romen Do you have any thoughts on this?

It would be nice to have a reproducer.. The way I see it the patch should be changing only what error code is going on the stack: field_div should be able to recognize a div by zero and return an error, but in that case we skip to add EC_R_INVALID_ENCODING at the top of the error stack.
@guidovranken can you provide a snippet that triggers the issue?
Also your description seems at odds with the linked patch: oct2point still fails for malformed hybrid encodings where X is zero, it just fails earlier.
What is worrying in your description is that it suggests point2oct succeeds but delivers a malformed encoding, but that is not addressed by the linked patch.
I am not against applying the patch they used in LibreSSL which also has the merit of adding useful annotations, and makes things slightly more consistent.
@slontis How does that work for licensing and CLA?

@slontis How does that work for licensing and CLA?
No idea :)

How does that work for licensing and CLA?

All authors must have a CLA on file.

It would be nice to have a reproducer.. The way I see it the patch should be changing only what error code is going on the stack: field_div should be able to recognize a div by zero and return an error, but in that case we skip to add EC_R_INVALID_ENCODING at the top of the error stack.
@guidovranken can you provide a snippet that triggers the issue?
Also your description seems at odds with the linked patch: oct2point still fails for malformed hybrid encodings where X is zero, it just fails earlier.
What is worrying in your description is that it suggests point2oct succeeds but delivers a malformed encoding, but that is not addressed by the linked patch.
I am not against applying the patch they used in LibreSSL which also has the merit of adding useful annotations, and makes things slightly more consistent.
@slontis How does that work for licensing and CLA?

Here's an example snippet
#include <openssl/ec.h>
#include <openssl/obj_mac.h>

static EC_POINT* copy(EC_GROUP* group, EC_POINT* pub, int form)
{
    size_t outSize;
    BN_CTX* ctx = BN_CTX_new();

    outSize = EC_POINT_point2oct(
            group,
            pub,
            form,
            NULL,
            0,
            ctx);

    uint8_t* out = malloc(outSize);
    if ( EC_POINT_point2oct(
                group,
                pub,
                form,
                out,
                outSize,
                ctx) != 0 ) {
        EC_POINT* tmpPoint = EC_POINT_new(group);
        if ( tmpPoint != NULL ) {
            if ( EC_POINT_oct2point(
                        group,
                        tmpPoint,
                        out,
                        outSize,
                        ctx) != 1 ) {
                printf("Conversion back from oct failed\n");
            }
            EC_POINT_free(pub);
            pub = tmpPoint;
        }
    }

    return pub;
}

int main(void)
{
    BIGNUM* x = BN_new();
    BIGNUM* y = BN_new();
    BN_dec2bn(&x, "0");
    BN_dec2bn(&y, "1");
    EC_KEY* key = EC_KEY_new();
    EC_GROUP* group = EC_GROUP_new_by_curve_name(NID_secp256k1);
    EC_KEY_set_group(key, group);
    EC_POINT* pub = EC_POINT_new(group);
    EC_POINT_set_affine_coordinates(group, pub, x, y, NULL);
    pub = copy(group, pub, POINT_CONVERSION_HYBRID);
    return 0;
}

Ping @botovq from LibreSSL

@guidovranken I think your example snippet has a different problem. The point (0, 1) is not on the curve secp256k1, so here EC_POINT_set_affine_coordinates fails and leaves a bogus point in pub. Consequently,  point2oct produces nonsense that oct2point rightfully refuses to decode. A point-on-curve check in point2oct might be worth considering, but that's a different question.
Using NID_sect571k1 instead of NID_secp256k1 would give a reproducer for the problem I fixed.
@romen The corner case that is treated incorrectly is valid hybrid encodings of points with x = 0 on binary curves. This happens for the point (0, 1) on sect571k1, for example. The hybrid encoding of (0, y) must have y_bit unset in its point conversion form. point2oct handles this correctly, but oct2point fails because it tries to check y_bit against the last bit of (y / x) which of course errors for zero x:
https://github.com/openssl/openssl/blob/39da32729401110572da1782c80bef39c6f3f64b/crypto/ec/ec2_oct.c#L339-L342
I have not signed the CLA, but I can do that and send a PR your way.

Thanks @botovq for the comment, I missed the point (pun intended) of the patch!
If you could sign the CLA and throw a patch our way it would be most welcome: actually this is a bug both in master and in 1.1.1 (the master patch won't cherry pick), so if you feel generous you could throw 2 PRs at us for the different branches (or I can take it upon myself to backport from master if you don't have the time for 2 PRs).

@romen I have created a PR against master. It should be trivial to backport to 1.1.1. I'll happily do that once that PR is reviewed and the CLA issue is settled (might take a week or two as I don't have easy access to a printer and scanner).

#15108 is now merged to master
#15112 (the 1.1.1 backport) is waiting for a second review


try to fix freetype include error

should be able to fix https://github.com/Equilibrium-Games/Acid/issues/42

stable now. any more insight @mattparks


Add support for .leet.toml

This PR adds support for the .leet.toml thing.
This is how it works right now:
cargo leet active prints the currently active problem.
cargo leet active [slug] sets the currently active problem, checking that it exists and that it isn't lib.
cargo leet test prepends // to every (pub) mod [slug]; and runs cargo test before reverting lib.rs
I'd also add the copying subcommand, but now that I switched to KDE on Wayland on Arch, btw, at least for now, I can't get it working

Thanks, I'll check it out.

Really good PR. Thank you very much. I think it is mostly fine, there are a few minor nits that I would want but I wouldn't hold it up for those. The only thing I'd want to do before merging is move the copy code into a separate branch until we can get it working instead of merging in code that doesn't work yet if that's ok with you.
Also do you want me to add you to the organization so you can just create branches directly on the repo?

Really good PR

Thx ^^

there are a few minor nits that I would want but I wouldn't hold it up for those

I'd happily fix them

move the copy code into a separate branch

To be clear, I only tried it out in this branch. The copy code isn't actually included in this PR

Also do you want me to add you to the organization so you can just create branches directly on the repo?

That would be great, thanks

I'd happily fix them

Sure it's just two actually now that I look at the list. It's to move ".leet.tom" into a constant so that if we ever need to change it we minimize the number of places that needs to happen and to update the readme but that one can wait as I kinda want to restructure it to be more maintainable.

To be clear, I only tried it out in this branch. The copy code isn't actually included in this PR

That explains why I didn't notice it


Added .editorconfig

EditorConfig helps developers define and maintain consistent coding styles between different editors and IDEs. The EditorConfig project consists of a file format for defining coding styles and a collection of text editor plugins that enable editors to read the file format and adhere to defined styles. EditorConfig files are easily readable and they work nicely with version control systems.

Can you add *.yml with 2 space indent?

Can you also add .scss at 4?

Thanks!


LUCENE-10244: Please consider opening MultiCollector::getCollectors for public use

Signed-off-by: Andriy Redko<EMAIL_ADDRESS>Description
The usage of MultiCollector sometimes is very convenient along with CollectorManager instances (as an alternative to MultiCollectorManager), for example:
class SomeCollectorManager implements CollectorManager<Collector, ?> {
    @Override
    public Collector newCollector() throws IOException {
      return MultiCollector.wrap(new Collector1(), new Collector2(), ...);
    }
    @Override
    public List<Integer> reduce(Collection<Collector> collectors) throws IOException {
      return ...;
    }
}

Unfortunately, the reduce() phase is lacking the access to  MultiCollector::getCollectors method, since it is package protected at the moment. Making MultiCollector::getCollectors  public would make it convenient to use MultiCollector + CollectorManager and implement the reduce phase by accessing individual collectors.
Solution
Make MultiCollector::getCollectors  public
Tests
The test which illustrates the usage of the MultiCollector + CollectorManager has been added.
Checklist
Please review the following and check all that apply:

[X] I have reviewed the guidelines for How to Contribute and my code conforms to the standards described there to the best of my ability.
[X] I have created a Jira issue and added the issue ID to my pull request title.
[X] I have given Lucene maintainers access to contribute to my PR branch. (optional but recommended)
[X] I have developed this patch against the main branch.
[X] I have run ./gradlew check.
[X] I have added tests for my changes.

@jpountz if you don't mind taking a look, really appreciate it, thank you!

This sounds reasonable to me!

I don't have any strong opposition to making this public, but I would be curious to understand why MultiCollectorManager isn't a good fit for your use case. It looks like your desired functionality is exactly what MultiCollectorManager does, so I wonder if there's some deficiency there that should be improved instead of requiring users to implement their own collector manager on top of MultiCollector.

Thanks for looking @gsmiller , we are working on experimental support of the concurrent segment search in the OpenSearch (if you are curious, please take a look at [1]). Since we are dealing with existing code, in certain cases it was much simpler to use the CollectorManager + MultiCollector instead of MultiCollectorManager. I assume it could be the case for other projects (fe [2]). Thank you.
[1] https://github.com/opensearch-project/OpenSearch/pull/1500
[2] https://github.com/Yelp/nrtsearch

It's reasonable to me too. I can imagine how someone would like to implement similar logic to MultiCollectorManager (which internally uses MultiCollector#getCollectors) but return a proper object instead of an array of objects.

I'll merge next week unless someone beats me to it.

+1, I felt the need for this in Solr once. https://github.com/apache/lucene-solr/pull/675#discussion_r288442020

@reta ah yes, thanks for all the detail! I'll add my +1 as well, and while I'm here, I'll go ahead and merge. Thanks for adding tests as well :)


Building on mac osx

I'm trying to follow the readme at https://github.com/xxxajk/esp-open-sdk for setting up the build environment on OSX however I am running into errors. Has anyone set this up to compile on OSX recently that could provide a novice some more up-to-date instructions? I do not use OSX or Linux/Unix very often but would like to test the recent GPIO code I've added.
For instance...
$ brew tap homebrew/dupes
Error: homebrew/dupes was deprecated. This tap is now empty as all its formulae were migrated.
and...
$ brew install binutils coreutils automake wget gawk libtool help2man gperf gnu-sed --with-default-names grep
Error: invalid option: --with-default-names

As a start to try and get around these issues, I have upgraded homebrew, then removed the invalid option from the brew install command based on the following: (it was removed)
https://github.com/Homebrew/homebrew-core/commit/8c4f31e6643469493fc26d275be09cb05b09beac

Created and mounted the case sensitive volume. Cloned the SDK repo. CD to the sdk dir then typed make. Got the following error and fixed it with the following as well based on this: https://github.com/pfalcon/esp-open-sdk/issues/350
The error:
missing required tool: gobjcopy objcopy
Execute this to fix it:
export PATH="/usr/local/opt/binutils/bin:$PATH"

Make made it pretty far but ultimately fails installing ncurses for build...
...
***********************************************************

Now configured for "xtensa-lx106-elf"
sed -r -i.org s%CT_PREFIX_DIR=.*%CT_PREFIX_DIR="/Volumes/case-sensitive/esp-open-sdk/xtensa-lx106-elf"% .config
sed -r -i s%CT_INSTALL_DIR_RO=y%"#"CT_INSTALL_DIR_RO=y% .config
cat ../crosstool-config-overrides >> .config
./ct-ng build
[INFO ]  Performing some trivial sanity checks
[INFO ]  Build started 20190216.151147
[INFO ]  Building environment variables
[INFO ]  =================================================================
[INFO ]  Retrieving needed toolchain components' tarballs
[INFO ]  Retrieving needed toolchain components' tarballs: done in 59.00s (at 01:03)
[INFO ]  =================================================================
[INFO ]  Extracting and patching toolchain components
[INFO ]  Extracting and patching toolchain components: done in 149.00s (at 03:32)
[INFO ]  =================================================================
[INFO ]  Installing ncurses for build
[ERROR]    clang: error: linker command failed with exit code 1 (use -v to see invocation)
[ERROR]    make[4]: *** [clear] Error 1
[ERROR]    clang: error: linker command failed with exit code 1 (use -v to see invocation)
[ERROR]    make[4]: *** [tabs] Error 1
[ERROR]    make[3]: *** [all] Error 2
[ERROR]
[ERROR]  >>
[ERROR]  >>  Build failed in step 'Installing ncurses for build'
[ERROR]  >>        called in step '(top-level)'
[ERROR]  >>
[ERROR]  >>  Error happened in: CT_DoExecLog[scripts/functions@216]
[ERROR]  >>        called from: do_ncurses_backend[scripts/build/companion_libs/220-ncurses.sh@151]
[ERROR]  >>        called from: do_ncurses_for_build[scripts/build/companion_libs/220-ncurses.sh@35]
[ERROR]  >>        called from: do_companion_libs_for_build[scripts/build/companion_libs.sh@29]
[ERROR]  >>        called from: main[scripts/crosstool-NG.sh@646]
[ERROR]  >>
[ERROR]  >>  For more info on this error, look at the file: 'build.log'
[ERROR]  >>  There is a list of known issues, some with workarounds, in:
[ERROR]  >>      'share/doc/crosstool-ng/crosstool-ng-1.22.0-60-g37b07f6f/B - Known issues.txt'
[ERROR]
[ERROR]  (elapsed: 4:22.00)
[04:23] / make[2]: *** [build] Error 1
make[1]: *** [_toolchain] Error 2
make: *** [crosstool-NG/.built] Error 2

Unfortunately it looks as though Crosstool-ng no longer supports OSX:
https://crosstool-ng.github.io/2018/11/26/macos.html

Never could get OSX working however I was able to get up and running quickly with an Ubuntu VirtualBox  image from osboxes.org

brew install ncurses
export CPPFLAGS="-I/usr/local/opt/ncurses/include -I/usr/local/opt/gettext/include"
export LDFLAGS="-L/usr/local/opt/gettext/lib -lintl -L/usr/local/opt/ncurses/lib"

fixes this for me


Allow override Fields from non-abstract Pages

Is your proposal related to a problem?

There are some cases where a field is defined in a non-abstract parent Page, and some properties of that field (help_text, blank, verbose_name) needs to be overridden in the children pages.
Describe the solution you'd like

Overriding field properties should be allowed for non-abstract classes, or there should be an easy an reliable way to modify the properties in the constructor or in the init method of the pages that should apply only for that case without conflicting with other overrides.
Solution 1 allow override fields from non abstract pages:
class ArticlePage(Page):
    image = models.ForeignKey(
        'wagtailimages.Image',
        null=True,
        blank=True,
        on_delete=models.SET_NULL,
       help_text="Image"
    )

# here we want to make the image required
class SpecificArticlePage(ArticlePage):
    image = models.ForeignKey(
        'wagtailimages.Image',
        null=False,
        blank=False,
        on_delete=models.SET_NULL,
        help_text="Image"
    )

# here we want to make the image required and modify the help_text
class OtherSpecificArticlePage(ArticlePage):
    image = models.ForeignKey(
        'wagtailimages.Image',
        null=False,
        blank=False,
        on_delete=models.SET_NULL,
        help_text="16:9 image"
    )

Solution 2 add a way to override the properties in the fields without conflicting with other overrides:
class ArticlePage(Page):
    image = models.ForeignKey(
        'wagtailimages.Image',
        null=True,
        blank=True,
        on_delete=models.SET_NULL,
       help_text="Image"
    )

# here we want to make the image required
class SpecificArticlePage(ArticlePage):

    def __init__():
        self.image.blank = False
        self.image.null = False

# here we want to make the image required and modify the help_text
class OtherSpecificArticlePage(ArticlePage):
        def __init__():
        self.image.blank = False
        self.image.null = False
        self.image.help_text = "16:9 image"

Describe alternatives you've considered

There is a hacky way to achieve this right now: https://stackoverflow.com/a/57498309
But when there are multiple children objects that needs to override multiple properties, the admin interface shows the values randomly
Additional context

(Write your answer here.)

Hi @jkevingutierrez - this isn't generally possible for model field properties due to a limitation of Django model fields; please see Field name “hiding” is not permitted in the Django docs:

In normal Python class inheritance, it is permissible for a child class to override any attribute from the parent class. In Django, this isn’t usually permitted for model fields. If a non-abstract model base class has a field called author, you can’t create another model field or define an attribute called author in any class that inherits from that base class.

As you suggest, one way around this is to use abstract Page classes.

Closing as this is something that would have to be supported at the Django level.


Differentiating between privacy-sensitive and anonymous data

We could consider to process privacy-sensitive data, but to encrypt it before sending it to the server. Keys to the encryption could be provided to researchers that are allowed to access those data (for example, using PEP). For non-sensitive data, we could send the data in plain text as we do now, so that the Kafka streams can aggregate it properly. Using the PEP mechanism, those data could also be encrypted, but the Kafka streams could get a key for only those data.

Read through the PEP paper, this is based on a new encryption algorithm. It would be a complete infrastructure effort to implement this. The issue remains: everything that does not have to be shown in the dashboard or retrieved using the API, could be encrypted. This would involve especially privacy-sensitive stuff. In all cases, we would have to trust the researchers to handle their private keys with care though...

If we encrypt data using a key that is unknown to the Platform, we cannot apply any analysis on this data. What kind of data would you encrypt with this method?

Exactly, that's the point. So for example absolute locations, IP addresses or unprocessed voice data are privacy sensitive. However, we could choose to store them in encrypted way. The platform would not be able to read or process it, but just provide it as-is in the full data extracted from HDFS. For less sensitive data such as battery levels, we'd send unencrypted so our platform can process it. We could decide on a stream-per-stream basis whether we want the data encrypted or unencrypted. Also, we could choose to leave the keys always unencrypted (anonymous patient ID), but just encrypt the values.

Another alternative is to do the data processing on another "trusted" host, where we would provide the decryption key as well. Right now, I don't think we have the budget + motivation to have this additional infrastructure cost though.

The vast majority of collected variables are privacy sensitive (HR, Acc, ecc.. ). We can absolutely design something to provide also this functionality, but we should bring WP8 up in the discussion or wait a clear need/requirement.

As long as the HR and Acc is not coupled to a specific person, I'd consider them anonymised data, which would be fine to process if we don't know the identity. However, something like absolute location can be used to find someones home and then identity. Likewise with voice recognition and IP address.


[SPARK-40377][SQL] Allow customize maxBroadcastTableBytes and maxBroadcastRows

What changes were proposed in this pull request?
Add two parameters to allow customizing maxBroadcastTableBytes and maxBroadcastRows
Why are the changes needed?
Recently, we encountered some driver OOM problems. Some large tables were compressed using Snappy and then broadcast join was performed, but the actual data volume was too large, which resulted in driver OOM.
The values of maxBroadcastTableBytes and maxBroadcastRows are hardcoded, 8GB and 512000000 respectively. Maybe we can allow customization of these values, configure smaller values according to different scenarios, and prohibit broadcast joins for some large tables to avoid driver OOM.
Does this PR introduce any user-facing change?
NO
How was this patch tested?
Unit test

Can one of the admins verify this patch?


I keep getting "Could not verify token" no matter what I do

With no error message as to why it can't verify the token.
My verify function looks like this now:
  def verify_function() do
    %Joken.Token{}
    |> Joken.with_signer(hs256(@jwt_secret))
  end

This is using Plug/Phoenix. I know the secret is correct because if I do @jwt_secret <> "x" it tells me "Invalid signature" when I try running it. I've boiled down my token to the simplest possible thing:
eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.e30.0HgydrRCRtJ7ILCoaOaj3qz971zI2u2eHZ5HraC1JpI (no claims) and it still cannot "verify the token". What am I doing wrong?
These are the relevant functions.
  # anything using the following pipeline requires JWT, as do the API
  # functions.
  pipeline :browser_protected do
    plug :accepts, ["html"]
    plug Joken.Plug, verify: &LIWordsWeb.Router.verify_function/0
  end

  scope "/crosswords", LIWordsWeb do
    pipe_through :browser_protected
    post "/gcg_upload", GcgUploadController, :create
  end

The issue here was that I needed to specify a JSON encoder. This didn't seem to be required from some of the examples and tests.. but I figured out the issue.


Test serialized messages if compatible with Hashicorp

Information
Based on [1], if the messages at zio-memberlist side is compatible with that at Hashicorp env.
Steps

[ ] ~Serialize the message Ping, Ack, and so on using MessagePack at zio-memberlist side~ Test joining hashicorp from zio-memberlist

[x] Build bin from serf
~- [ ] Setup kubernetes~
[x] Test with LocalNode
[ ] Implement tcp transport for communicating with serf
[ ] Check join message sent from zio-memberlist

[ ]  Check if those serialized messages is compatible with that at hashicorp's memberlist

If not compatible, maybe check the reasons (the differences, and so on if possible)

References
[1]. https://github.com/zio/zio-memberlist/issues/12

Zio memberlist default implements tcp only, so  the next step is to implement tcp transport so that the zio side can communicate with serf.
Active Internet connections (only servers)
Proto Recv-Q Send-Q Local Address           Foreign Address         State       PID/Program name
...
tcp        0      0 debian.mine:5558     <IP_ADDRESS>:*                  LISTEN      261015/bin/serf
tcp        0      0 debian.mine:7373     <IP_ADDRESS>:*                  LISTEN      261015/bin/serf
...
udp        0      0 debian.mine:5557     <IP_ADDRESS>:*                              261069/java
udp        0      0 debian.mine:5558     <IP_ADDRESS>:*                              261015/bin/serf

Finding

client/rpc_client.go performs send join request to remote serf agent process

Phrase:

Handshake

header: {Command: handshake, Seq:1}

obj: {Version: 1} (Version is MaxIPCVersion)

Response: {Seq: 1, Error:}

Join

header: {Command:join Seq:2}

obj: {Existing: [<IP_ADDRESS>:5000], Replying:false} (obj = req = joinRequest)

Response: {Seq: 2, Error:}


Closes #330 Changed default choice from No to Yes for wormhole receive

I've simply changed the default for receive to Yes. Unless specifically typing n or N receive defaults to accept receive upon carriage return.

Looks good, thanks! I've tweaked it a bit so that typing "y" or just return accepts the transfer, but typing anything else cancels it, so if someone panics and types "quit" or "help" or something like that, it will cancel too.

Sounds good - thanks for adding the tweak.


First integration tests #LSC-97

It a starting point for integration testing of Neutron network and contracts:
Simple test - basic tests, contract upload, interchain transfer
Interchain txs - interchain txs contract, interchain delegation

Oh well…

 Oh well…

I haven't touched interchaintxs test bc Dima is updating it


().Play(); not work

Description: ().Play(); not work in script.

Build: 0.2.6

Type: A

Severity:

[ ] Critical
[X] Major
[ ] Minor

Priority:

[ X] Urgent
[ ] High
[ ] Normal
[ ] Low

Steps to reproduce: create a game object with particle emitter and call Play() in script.

Frequency: All the time

().Play() works correctly on master


Avoid CoUninitialize if CoInitialize failed

Fixes #105 (in case that issue is valid).
Instead of instantiating the ApartmentThreadedRuntime first and then calling CoInitializeEx, this PR reverses that order and only instantiates the runtime if the CoInitializeEx was successful.
(Also expanded the scope of unsafe to cover the branching logic - while not technically a requirement for the compiler, I figured the guarantee to do or avoid CoUninitialize later should also be part of unsafe.)

Good catch! Thanks for this!


Dockerize Mysql db

Feature Request or Issue?
Feature Request
Overview
Add a docker-compose file to maybe spin up a db with the bot.

Actually a remote db would be better, may make a separate repo for it


Make a website.

You guys should make a website for your Files UWP that explains info and stuff.
If you need a domain just get one at Freenom.
I was thinking of a domain like files-uwp.tk or files-uwp.com.

@pixl-8 The wiki already includes a lot of info, however, creating a dedicated site is something we can look into once the app leaves beta.

Good idea.

@pixl-8 Our website just went live, you can find it over here.


Only run integration tests on master.

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

This pr obviously doesn't change build logic so something seems to have made the build red, guessing unpinned dependencies. @srprash @lupengamzn is it something you could look at?

Can we also change the branch name to "main"? Remove non-inclusive wording if you see it :)

Github has added support for renaming relatively smoothly, including migrating open PRs, it's just the workflows that may need to be updated. It's probably a good time to go ahead and do it for all our repos indeed.

This pr obviously doesn't change build logic so something seems to have made the build red, guessing unpinned dependencies. @srprash @lupengamzn is it something you could look at?

SQLAlchemy tests are failing due to some changes in the latest version of the library. We would need to pin its version for some of the python version tests. @lupengamzn is taking a look.

This pr obviously doesn't change build logic so something seems to have made the build red, guessing unpinned dependencies. @srprash @lupengamzn is it something you could look at?

The issue should be fixed now by pinning the version of SQLAlchemy and Flask-SQLAlchemy for test.

Codecov Report

Merging #277 (db6d0ad) into master (266bc82) will decrease coverage by 1.20%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #277      +/-   ##
==========================================
- Coverage   79.58%   78.38%   -1.21%
==========================================
  Files          83       77       -6
  Lines        3277     2956     -321
==========================================
- Hits         2608     2317     -291
+ Misses        669      639      -30

Impacted Files
Coverage Δ

aws_xray_sdk/ext/aiobotocore/__init__.py
0.00% <0.00%> (-100.00%)
:arrow_down:

aws_xray_sdk/core/__init__.py
75.00% <0.00%> (-25.00%)
:arrow_down:

aws_xray_sdk/core/sampling/local/reservoir.py
88.88% <0.00%> (-11.12%)
:arrow_down:

aws_xray_sdk/core/utils/compat.py
85.71% <0.00%> (-9.53%)
:arrow_down:

aws_xray_sdk/core/patcher.py
83.20% <0.00%> (-4.00%)
:arrow_down:

aws_xray_sdk/core/sampling/connector.py
47.94% <0.00%> (-2.71%)
:arrow_down:

aws_xray_sdk/ext/httplib/patch.py
79.16% <0.00%> (-2.50%)
:arrow_down:

aws_xray_sdk/core/emitters/udp_emitter.py
61.11% <0.00%> (-2.05%)
:arrow_down:

aws_xray_sdk/ext/util.py
89.83% <0.00%> (-1.70%)
:arrow_down:

aws_xray_sdk/ext/sqlalchemy_core/patch.py
70.96% <0.00%> (-1.62%)
:arrow_down:

... and 24 more

Continue to review full report at Codecov.

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


Key Word for Ignoring Spectra in a Profile

It would be helpful to have an attribute of the Profile class that is a list of bad spectra to ignore when plotting absorbances or calculating diffusion profiles. I was thinking either a list of profiles by index number or by micron position.

So: add two keywords to profile.plot_area_profile(ignore_idx=[], ignore_pos_microns=[]) that let you pass in a list of indexes (to ignore_idx) or positions (to ignore_pos_microns) that won't get plotted. Other profile methods that would need these keywords: average_spectra() and fitD(). Sound like what you want?

For the methods (average_spectra(), fitD(), plot_area_profile...) maybe it could just be a keyword such as "ignore_spectra=True". Then the method would look at either the ignore_idx or ignore_pos_microns list. This way it only needs to be entered in one place for each profile.

One thought to consider is if the ignore_spectra option defaults to true. I imagine if a user is entering data into the ignore_spectra lists they would want it to default to true and then could decide to turn it off it they want to consider those spectra.

I'm not convinced you need a third keyword here.
my_list_to_ignore = [0, 3, 10]
profile.plot_area(ignore_idxs=my_list_to_ignore)
I changed my mind and don't want to ignore anything anymore
profile.plot_area()
or
profile.plot_area(ignore_idxs=[])
or
profile.plot_area(ignore_idxs=None)
This way you have a record of which ones to ignore when you want to, and it's about the same amount of typing as switching True/False but with fewer keywords to keep track of. You can still store the ignore list as an attribute:
profile.ignore = [2, 4, 6]
profile.plot_area(ignore_idxs=profile.ignore)
This way I think will give you a bit more flexibility to easily change which points you are ignoring. I am open to being convinced otherwise.

I like your second example. That makes sense to me.
My general goal is to be able to define the spectra I wish to ignore in my profile library so it is always an attribute of the Profile in any script it is imported to. I would also have the flexibility to alter that attribute temporarily in a script if needed.
The True/False would only be useful if you thought it made sense to make True the default. My thinking was that anyone using the ignore setting would want it universally applied for a profile to all methods. That may be wrong, people might want more flexibility to change which spectra are ignored for a particular method.

Hey, I think the best way for me to avoid a few headaches is to first merge in what I've been doing from the while-working-on-olivine-paper branch and then add these keywords, which means that you will need to update your code to work with my latest version. Virtually all of the changes I've made have been either very minor or having to do with handling whole-block data, so the only thing that I think will matter to you is that I've changed profile.profile_name to just profile.name. You can get that latest from the while-working-on-olivine-paper branch now. I'll merge that into the master branch momentarily.

master branch now has profile.name in it, and I'm going to start working on this keyword.

I gotta go, but play with the ignore_idx keyword in profile.plot_area_profile() on the issue28 branch and let me know how it goes.

ok, now ignore_idx is in all 3 methods we mentioned, although I haven't tested fitD or average_spectra. Let me know if that all solves your problem.

Have you had a chance to play with these changes yet? If it's ok, I'll merge the issue28 branch back into the master and close this issue.

plot_area_profile() work really well.
I haven't used fitD too much before but it doesn't seem to have an effect. Ive even gone to the extreme of ignoring almost all the points in a spectra.
average_spectra doesn't appear to work at all even when not ignoring spectra. I get an error for line 240 of the profiles module: "spec_list = self.spectra.copy()"
and the error is: "'list' object has no attribute 'copy'"

I figured out that I made a mistake when running fitD. Now it works fine for me but average_spectra still gives the same error.

What was the mistake with fitD?

It was pretty silly. I used a range function inside of a list and it returned a list within a list.
Easily fixed
Sent from my iPhone

On Jun 30, 2017, at 12:36 PM, Elizabeth Ferriss<EMAIL_ADDRESS>wrote:
What was the mistake with fitD?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Can you show me the code you are writing to try out profile.average_spectra()? I am having trouble duplicating the problem. See the example under "Average all the spectra across the profile"

Here is the output I get for average = prof.average_spectra()

average = prof.average_spectra()

AttributeError                            Traceback (most recent call last)
 in ()
----> 1 average = prof.average_spectra()
/Users/Henry/Library/Enthought/Canopy_64bit/User/lib/python2.7/site-packages/pynams-0.2.0-py2.7.egg/pynams/profiles.pyc in average_spectra(self, ignore_idx)
238         to ignore_idx
239         """
--> 240         spec_list = self.spectra.copy()
241
242         # ignore certain data points
AttributeError: 'list' object has no attribute 'copy'

And Prof is a pynams spectra that plots without problem.
prof
Out[204]: <pynams.profiles.Profile instance at 0x12c937878>

Hi E,
For plot_area_profile() it Looks like the scaling on the Y axes doesn't take into account the ignored spectra list. I like the fact that the X axis doesn't change with the ignored list but it would be helpful if the X axis changed, especially for normalized spectra.

Are you running python 2.7? I think maybe the average_spectra syntax I had been using to copy a list maybe works in python 3 but not 2.7. Anyway, I changed to something that is more readable, works for me, and hopefully works for you.

Oh, I see your point with the scaling because if you want to ignore the crazy high point, it would be nice if the y-axis limits weren't set assuming there was still that crazy high point.

Okey doke. I think that's fixed. Re-reading your last comment here, you say "it would be helpful if the X axis changed" but I think you meant Y axis. I'm going to merge this back issue branch back into the master and close the issue, but do feel free to re-open it if there's more details we didn't cover.


mdl-select not closing when click outside- ipad

mdl-select options drop down closes when clicked outside. But it is not working in ipad

In https://github.com/mseemann/angular2-mdl-ext/blob/master/src/components/popover/popover.ts#L21 we need listen to "click" and "touchend" or "touchstart".


'geographic location' imported as both individual and class

http://purl.obolibrary.org/obo/GAZ_00000448 geographic location
imported as both individual and class

https://github.com/EnvironmentOntology/gaz/issues/34

I added import_GAZ.owl, which contains only GAZ_00000448 and associates it only as a class (not as an individual). eupath_dev.owl will draw from import_GAZ.owl, but (until they fix this issue) GAZ_00000448 will not be in any of the input .txt files we use for importing.


Highlight matching line and column

This PR changes the output so that instead of, e.g.,
106 |     var component = this.refs[key];
107 |     if (component.componentDidAppear) {
108 |       component.componentDidAppear();
109 |     }

it highlights the matching line and column:
106 |     var component = this.refs[key];
107>|     if (component.componentDidAppear) {
                        ^
108 |       component.componentDidAppear();
109 |     }

It uses colors for additional emphasis.
I also added handling for no command line arguments.
If you'd like anything to be changed, or if you'd like this to be broken into smaller PRs, please let me know.

awesome thanks! ill get a new build out tonight


Encode Hackathon - Bug bounty for Harmony Marketplace SDK (Documentation Error)

Description
Related to the Encode Hackathon, this bounty is for promotion of the Harmony Marketplace SDK
Context
Identify documentation errors in the Harmony Marketplace SDK and do a PR to fix the errors. Limited to 10 errors found in the SDK
Acceptance Criteria

[ ] Document the error found in the SDK on this bug bounty
[ ] Link to the PR that is pushed to the Marketplace SDK

Reward
We will reward based on the document errors found and may bundle a few errors together for payment
3 bounties up to $300 USD in ONE per bounty

@jacksteroo please review and confirm

Confirmed

The approve, allowance and transferFrom functions in HRC20 had a wrong description

PR: https://github.com/blockcoders/harmony-marketplace-sdk/pull/54

hi @j05u3 thanks for the PR, let me get the blockcoders folks to confirm and we can start approving it

hey @j05u3 did you find any other documentation improvements?

hey @j05u3 did you find any other documentation improvements?

we couldn't find other improvements, only the ones on the PR


Update Helipad to v0.2.0

Thank you for your contribution to RaspiBlitz. Before submitting this PR, please make sure:

[x] That its based against the dev branch - not the default release branch.

PR merged - will be part of v1.11.3 release

@rootzoll Thanks!


browser challenge screen display issue

i am not able to get complete display of screen on browser both chrome and firefox.
just display challenge detail frame on left and no code window frame is shown
any help how to fix this. thanks.

Can you see what's in the console? You can open the console by Ctrl+Shift+I in Chrome. See if there are any errors and paste it here?

I can not repro this. Do you get any errors in the console when this happens?

Thanks guys.
Got it resolved. My Hotspot shield settings were interfering. I was using challenges earlier on my laptop. just yesterday switched to my Home PC and getting this abnormal behavior.
thanks a lot.


Add pre build environment

Build OCI image when there is a change in the main.
The OCI image will be stored in GitHub Packages.
When the action is executed, the action will use the prebuild container, which makes it a lot faster.
Related issue
Closes #6
Development
If you want to test a new feature in a branch, simple point in the action.yml to the Dockerfile again and you will have a freshly build environment again for each run.
About the container
The prebuild container has some Secure Software Supply Chain meta files attached to it. The container is signed, has a SLSA provenance file so you can prove it was created in this build pipeline, and a Software Bill of Material in SPDX format, so you can evaluate the content of the container before you want to use it.

The checks will initially fail, because the first time, there is no stored package. Rerunning the checks will resolve that.


Backbone 1.2.0

Backbone recently release V1.2.0, any idea when you'll implement this into exoskeleton? Especially the new "update" event on Collections is useful.
http://backbonejs.org/#changelog

Yep, we'll merge over the changes in the next few days. If it takes longer than a week feel free to bug me or Paul.

That's awesome, thanks!

Yup, I'd love to see this too. I'm just about to start rewriting a bunch of backbone stuff and was hoping to make the move to exoskeleton. Thanks.

Great lib, looking forward to 1.2 & +

Hey y'all, I finished porting over the new code from Backbone. Several tests are failing at the moment and I don't have time to look into it. If anyone wants to take a crack they'd have my eternal gratitude: #112


Add iA Writer Mono

Added font mentioned by this comment: https://github.com/chriscoyier/coding-fonts/issues/8#issuecomment-742734924

Nice, thanks!!


Updated logo images

Updated both Logo128.png and Logo512.png

Thanks for contributing! Don't forget to star the repo (and maybe share it for more reach :D ). Happy Coding!
P.S : You are welcome to look for more issues in the app and get yourself assigned for them


Support WITH ORDINALITY clause

https://www.postgresql.org/docs/current/queries-table-expressions.html#QUERIES-TABLEFUNCTIONS

Note: only in FROM clause
xxchan@/tmp:postgres> select abs(1) WITH ORDINALITY;
syntax error at or near "WITH"
LINE 1: select abs(1) WITH ORDINALITY
                      ^
Time: 0.003s
xxchan@/tmp:postgres> select unnest(array[1,2,3]) WITH ORDINALITY;
syntax error at or near "WITH"
LINE 1: select unnest(array[1,2,3]) WITH ORDINALITY
                                    ^
Time: 0.003s
xxchan@/tmp:postgres>
Time: 0.000s
xxchan@/tmp:postgres> select * from unnest(array[1,2,3]) WITH ORDINALITY;
+--------+------------+
| unnest | ordinality |
|--------+------------|
| 1      | 1          |
| 2      | 2          |
| 3      | 3          |
+--------+------------+
SELECT 3
Time: 0.011s
xxchan@/tmp:postgres> select * from abs(1) with ordinality;
+-----+------------+
| abs | ordinality |
|-----+------------|
| 1   | 1          |
+-----+------------+

@TennyZhuang @neverchanje  Can you give the motivating use case of this feature? The simplest form is easy to implement, but IMO not very useful.
Because it can only be used in FROM (but not for ProjectSet), and currently the FROM f() can only be a constant function scan. Correlated subquery for this case is not support yet.
I think the useful use case is like
select * from t , unnest(x) with ordinality;
+-----------+--------+------------+
| x         | unnest | ordinality |
|-----------+--------+------------|
| [1, 2]    | 1      | 1          |
| [1, 2]    | 2      | 2          |
| [3, 4, 5] | 3      | 1          |
| [3, 4, 5] | 4      | 2          |
| [3, 4, 5] | 5      | 3          |
+-----------+--------+------------+

But we will need to support this first.
dev=> create table t(x int[]);
CREATE_TABLE
dev=> select * from t , unnest(x);
ERROR:  QueryError: Item not found: Invalid column: x
dev=>

User’s actual need is
select unnest(arr1), unnest(arr2)

Which is already supported.
Delay with ordinality until another user request.


internal routes for epinio apps (internal domain) work ?

Is your feature request related to a problem? Please describe.
Cloud Foundry apps can communicate with each other securely and directly over internal routes that never leave the platform. I wonder if epinio supports this
Describe the solution you'd like
microservice A has a public route (system domain) and will call an internal microservice B with an internal domain (similar to binding uri of service catalog) Is there such a feature and is it configured ?
Describe alternatives you've considered
This link https://docs.epinio.io/howtos/custom_routes seems to suggest it but the example is ANOTHER public route. We have use cases that the route is internal
Additional context
https://docs.cloudfoundry.org/devguide/deploy-apps/routes-domains.html

Hi @gitricko, yes, this is possible using the internal dns. Basically every deployed application has its own (kubernetes) service, which can be used to reach the application without leaving the cluster.
This is the output of epinio app show sample of a sample application:
-> % epinio app show sample

🚢  Show application details
Namespace: workspace
Application: sample

✔  Details:
|         KEY          |                                VALUE                                 |
|----------------------|----------------------------------------------------------------------|
| Origin               | /home/enrico/Development/SUSE/epinio/epinio/assets/golang-sample-app |
| Created              | 2022-11-04 09:34:30 +0100 CET                                        |
| Status               | 1/1                                                                  |
| Username             | admin                                                                |
| Running StageId      | 31b23c14c7db0546                                                     |
| Last StageId         | 31b23c14c7db0546                                                     |
| Age                  | 16m29s                                                               |
| Active Routes        |                                                                      |
| App Chart            | standard                                                             |
| Desired Instances    |                                                                    1 |
| Bound Configurations |                                                                      |
| Environment          |                                                                      |
| Chart Values         |                                                                      |

✔  Instances:
|                              NAME                               | READY | MEMORY  | MILLICPUS | RESTARTS |  AGE   |
|-----------------------------------------------------------------|-------|---------|-----------|----------|--------|
| rsample-8151325dcdbae9e0ff95f9f9658432dbedfdb209-75f58cfdbfjbvp | true  | 5.4 MiB |         0 |        0 | 16m29s |

The instance name rsample-8151325dcdbae9e0ff95f9f9658432dbedfdb209-75f58cfdbfjbvp is <pod/service>-<instance-id>, and so you can derive the service name leaving out the instanceID.
In this case the endpoint is just http://rsample-8151325dcdbae9e0ff95f9f9658432dbedfdb209.workspace.svc.cluster.local. Since the pod that I've used to curl is within the same namespace you can drop also all the rest, and just use the service name (http://rsample-8151325dcdbae9e0ff95f9f9658432dbedfdb209)

I was thinking that we could show this endpoint in the epinio app show command, something like:
-> % epinio app show sample

🚢  Show application details
Namespace: workspace
Application: sample

✔  Details:
|         KEY          |                                VALUE                                          |
|----------------------|-------------------------------------------------------------------------------|
| Internal Routes      | rsample-8151325dcdbae9e0ff95f9f9658432dbedfdb209.workspace.svc.cluster.local  |

What do you think?

As a bonus point I wanted to tell you that since v1.4.0 you can actually clear out all the external routes of an application. In this case the app is not reachable from the outside, and you can deploy it without any routes.
-> % epinio app update --help
Update the running application's attributes (e.g. instances)

Usage:
  epinio app update NAME [flags]

Flags:
  -z, --clear-routes          clear routes / no routes

-> % epinio app update -z sample

🚢  Update application
Namespace: workspace
Application: sample
Routes: <clearing all>

✔  Successfully updated application

That is an GREAT idea. In the beginning, it was how i tot it would work... looking at the backing service URI, but i could not figure out the naming convention for the app. Having stated explicitly would be helpful to automation and having a way to deploy WITHOUT public route is also needed!
Is it possible to clear the route during the push ? Something like epinio push --no-route

The same option, i.e. -z/--clear-routes works for push as well. Which is why the description of the option contains the text no routes too.

To complete the trifecta, app create takes the option as well.

To complete the trifecta, app create takes the option as well.

Thanks @andreas-kupries, able to show internal routes will be a great help in terms of helping automation


Documentation: Recommended hardware for workloads

Customers have expressed a desire for a minimum acceptable hardware configuration for various workloads.
This task involves:

Starting with 3 OpenFL tutorials (pytorch, tensorflow, jax), Profile how long it takes to run them and make a recommendation for the minimum hardware that should be used (i.e. high core CPU, GPU, memory, etc.). This list should be expanded over time (not as part of this task)
Create a new page in the OpenFL documentation that includes a table of the available tutorials that includes this information

I have read the CLA Document and I hereby sign the CLA

Closing because this issue is stale. We can reopen this as priorities change.


Whitelist "ubuntu-toolchain-r-test" for Bionic

The Ubuntu Toolchain Test PPA which provides current GCCs is blocked on Bionic:
Adding APT Sources

Disallowing sources: ubuntu-toolchain-r-test

To add unlisted APT sources, follow instructions in https://docs.travis-ci.com/user/installing-dependencies#Installing-Packages-with-the-APT-Addon

Full example build log
Build system information
Build language: generic
Build group: stable
Build dist: bionic

Travis.yml example:
language: generic

dist: bionic

matrix:
    include:
    - env: CXX=g++-9 CC=gcc-9
      addons:
        apt:
          packages:
            - g++-9
          sources: &sources
            - ubuntu-toolchain-r-test

script:
  - ${CC} --version
  - ${CXX} --version
  - cmake --version

@offa Did you by any chance find a workaround for this issue?

My current workaround is using sourceline:
addons:
  apt:
    sources:
      - sourceline: 'ppa:ubuntu-toolchain-r/test'

I'll close here, since it's no longer relevant.


Rona354 patch 1

Type
enhancement

Description

Added HTML structure to the test.html file.
Added JavaScript code with bugs and vulnerabilities:

Bug: Using the wrong variable name for multiplication.
Vulnerability: Cross-Site Scripting (XSS) by directly using user input from the URL without sanitization.
Vulnerability: Sensitive data transmission over unencrypted channels by using GET method and not using HTTPS.
Vulnerability: Clickjacking by using untrusted sources in iframes.
Vulnerability: Including outdated JavaScript libraries.

Added a form with a vulnerability for sensitive data transmission over unencrypted channels.
Added an iframe with a vulnerability for clickjacking.
Added a script tag with an outdated JavaScript library.

Changes walkthrough
Relevant files                                                                                Enhancement

test.html

        test.html
**- Added HTML structure

Added JavaScript code with bugs and vulnerabilities
Added form with sensitive data transmission vulnerability
Added iframe with clickjacking vulnerability
Added script tag with outdated JavaScript library**

 +47/-0

/review


Bot starts running slowly after several hundred responses

On my computer the bot delays a few seconds between inputs when I train with with 1000 example lines. It gets worse the more training data there is. Is there any way to speed up the performance, or is this just the nature of the beast?

If you install python-cjson ChatterBot's default database module will detect it and use it instead of python's built in json. The cjson module has much better preformance and supposedly offers up to 250 times the speed when processing data.
Asside from this, as I briefly mentioned in https://github.com/gunthercox/ChatterBot/issues/41#issuecomment-106066687, there is the oppertunity to write storage adapters for other types of databases. ChatterBot's default storage system uses a json file. An adapter to use another type of database could easily be created. Let me know if you end up creating an adapter for another database type, pull requests are always welcome :smile:.

Discovered this project after CleverBot's api (undocumented) went down.
I work daily with Elasticsearch, I think combined w/this (only have just started reviewing the code), it could probably be made to scale very well.  Haven't looked yet how the JSON fields correlate to responses (FYI elasticsearch is JSON)--but I imagine the features of elasticsearch'es SEARCH could be used to greatly improve the speed of responses.

Closing this one off as it is a duplicate of https://github.com/gunthercox/ChatterBot/issues/45

brizzbane - Did u create Elasticsearch  +ChatterBot adapter ?


feat: add capability in batch client to get enrichment activities to use enrichment flags

Hi @JcMinarro can you help to check?

Hi @JcMinarro can you help to check?

Could you check the tests?
It seems they are failing

The test is missing the API key, the API key should be server, or local server env.

Seems missing the environment variable in CI
Run ./gradlew spotlessCheck --no-daemon   ./gradlew spotlessCheck --no-daemon   ./gradlew test --no-daemon   shell: /usr/bin/bash -e {0}   env:     STREAM_KEY:      STREAM_SECRET:      STREAM_APP_ID:

Hi @JcMinarro all the test failed having these errors
java.lang.IllegalArgumentException: API key can't be empty
It is the error from the previous test cases.


fix: MD039/no-space-in-links

Spaces inside link text

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

#sign-off

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


CARLA 0.9.9 cannot launch

I have CARLA 0.9.9 installed on Ubuntu 20.04. I use NVIDIA GPU 1650 with 450.51 drivers. I have CUDA 10.1 installed.
I get the following error when I run ./CarlaUE4.sh in the /opt/carla-simulator/bin directory. How can I fix this? Thank you.
4.24.3-0+++UE4+Release-4.24 518 0
Disabling core dumps.
X Error of failed request:  BadDrawable (invalid Pixmap or Window parameter)
Major opcode of failed request:  149 ()
Minor opcode of failed request:  4
Resource id in failed request:  0x2200041
Serial number of failed request:  386
Current serial number in output stream:  397
terminating with uncaught exception of type std::__1::system_error: mutex lock failed: Invalid argument
Signal 6 caught.
Segmentation fault (core dumped)

@adityapatruni   if still matter, ./CarlaUE4.sh -opengl worked for me

I'm having the same error. But, when I try to execute with the -opengl flag it shows a warning message that says "OpenGL is no longer supported for desktop platforms, Vulkan will be used instead" and the application crashes the same way

I'm having the same error as @lucasmazz

me too!

I had the same problem, -opengl flag did not solve the issue. Hopefully -prefernvidia flag helped.
./CarlaUE4.sh -prefernvidia


cfg meta variables

It's quite often that a cfg directive in an attribute becomes fairy complicated when support for more platforms are added. Even worse, a cfg block is then often duplicated among many locations to configure off different items.
The thread_local/mod.rs file is a good example of the amount of duplication.
It would be nice to have something like the following
#![cfg_shortcut(foo, any(super, long, clause))]

#[cfg(foo)] ...
#[cfg(not(foo))] ...

Here the clause any(super, long, clause) would be shortened to the identifier foo which could then be used elsewhere. In general it would just be nice to make these long cfg directives much shorter and less error prone to copy-paste.

Yeah this would be nice. This and bounds aliases from #804 should give us a more complete aliasing toolkit.

If we expand macros within attributes, then this feature would drop out as a special case.
For example:
macro_rules! foo {
    () => (any(super, long, clause))
}

#[cfg(foo!())] ...
#[cfg(not(foo!()))] ...

Users wouldn't need to learn special syntax either -- just apply their existing macro knowledge.


Remove link to the removed code in Kubernetes repo

This code was removed in https://github.com/kubernetes/kubernetes/pull/44721 and it doesn't seem like the link is somehow useful there anyway.

This change is

Related to https://github.com/kubernetes/kubernetes.github.io/issues/4283
Thanks @ahmetb for pointing this out!

/lgtm


Matrix-vector multiplication prints java version in stdout

The following Code on Scala outputs return java 11 instance.
I'm using openjdk-11 and sbt 1.8.0.
# Main.scala
import breeze.linalg.{DenseMatrix, DenseVector}

object Main  extends App {
    val X = DenseMatrix((1.0, 0.0), (1.0, 1.0), (1.0, 2.0), (1.0, 3.0))
    val Y = DenseVector(0.0, 1.0)

    X * Y
}

# build.sbt
ThisBuild / version := "0.1.0-SNAPSHOT"

ThisBuild / scalaVersion := "2.13.10"

libraryDependencies  ++= Seq(
  "org.scalanlp" %% "breeze" % "2.1.0"
)

What is the reason for this output and how can I avoid this in my projects with Breeze?
Moreover this problem reproduces with other versions of jdk, this code always prints java version from which it was built.

Recieved answer on SO today
https://stackoverflow.com/questions/74539618/matrix-vector-multiplication-prints-java-version-in-stdout

sorry i wasn't able to get to this, but I'm glad you got it figured out!.
@luhenry looks like there are some stray printlns in your netlib...

Indeed there is at https://github.com/luhenry/netlib/blob/b8384e002febbc96bfaef967e345ed6018014dc4/blas/src/main/java/dev/ludovic/netlib/blas/InstanceBuilder.java#L69. Let me fix it and put out a release.

https://github.com/luhenry/netlib/releases/tag/v3.0.3 should be available in your friendly Maven Central repositories in the coming hour.

@luhenry thank you!


fix build issues with macos

This wouldn't build on my mac without these changes.

Codecov Report

Merging #6 into master will not change coverage.
The diff coverage is 100%.

@@           Coverage Diff           @@
##           master       #6   +/-   ##
=======================================
  Coverage   82.72%   82.72%
=======================================
  Files          11       11
  Lines         932      932
=======================================
  Hits          771      771
  Misses        161      161

Impacted Files
Coverage Δ

src/std/io/net/addr.d
79.06% <ø> (ø)
:arrow_up:

src/std/io/net/socket.d
95.48% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

???

@MartinNowak something is wrong with your repository. It keeps saying my fork has a completely different commit history than yours. I had to close and open my repository 2 times just to get this PR to work, and now if I try and compare, it says it can't. I think it has something to do with your CI committing things every few minutes.


scenarionet.sim error in nuplan dataset

My env: python3.9, nuplan-devkit check successful, metadrive <IP_ADDRESS> test successful.
I run python -m scenarionet.sim -d ~/nuplan/scenario_net --render 3D after converting nuplan dataset, but with error:
Known pipe types:
  glxGraphicsPipe
(1 aux display modules not yet loaded.)
Traceback (most recent call last):
  File "/home/lzh/miniconda3/envs/metadrive/lib/python3.9/runpy.py", line 197, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File "/home/lzh/miniconda3/envs/metadrive/lib/python3.9/runpy.py", line 87, in _run_code
    exec(code, run_globals)
  File "/home/lzh/miniconda3/envs/metadrive/lib/python3.9/site-packages/scenarionet/sim.py", line 52, in <module>
    env.reset(seed=index if args.scenario_index is None else args.scenario_index)
  File "/home/lzh/miniconda3/envs/metadrive/lib/python3.9/site-packages/metadrive/envs/base_env.py", line 476, in reset
    self.engine.reset()
  File "/home/lzh/miniconda3/envs/metadrive/lib/python3.9/site-packages/metadrive/engine/base_engine.py", line 294, in reset
    manager.reset()
  File "/home/lzh/miniconda3/envs/metadrive/lib/python3.9/site-packages/metadrive/manager/scenario_map_manager.py", line 40, in reset
    new_map = ScenarioMap(map_index=seed)
  File "/home/lzh/miniconda3/envs/metadrive/lib/python3.9/site-packages/metadrive/component/map/scenario_map.py", line 18, in __init__
    super(ScenarioMap, self).__init__(dict(id=self.map_index), random_seed=random_seed)
  File "/home/lzh/miniconda3/envs/metadrive/lib/python3.9/site-packages/metadrive/component/map/base_map.py", line 57, in __init__
    self._generate()
  File "/home/lzh/miniconda3/envs/metadrive/lib/python3.9/site-packages/metadrive/component/map/scenario_map.py", line 32, in _generate
    block.construct_block(self.engine.worldNP, self.engine.physics_world, attach_to_world=True)
  File "/home/lzh/miniconda3/envs/metadrive/lib/python3.9/site-packages/metadrive/component/block/base_block.py", line 133, in construct_block
    success = self._sample_topology()
  File "/home/lzh/miniconda3/envs/metadrive/lib/python3.9/site-packages/metadrive/component/scenario_block/scenario_block.py", line 40, in _sample_topology
    lane = ScenarioLane(lane_id, self.map_data, self.need_lane_localization)
  File "/home/lzh/miniconda3/envs/metadrive/lib/python3.9/site-packages/metadrive/component/lane/scenario_lane.py", line 43, in __init__
    width=self.get_lane_width(lane_id, map_data),
  File "/home/lzh/miniconda3/envs/metadrive/lib/python3.9/site-packages/metadrive/component/lane/scenario_lane.py", line 106, in get_lane_width
    return max(sum(map_data[lane_id]["width"][0]), self.VIS_LANE_WIDTH)
KeyError: 'width'

Maybe there are still some formatting issues?

supplement: I found this map_data is:
{'type': 'LANE_SURFACE_STREET',
 'polyline': array([[-133.39504793,    4.46195791],
        [-133.39769158,    4.71194356],
        [-133.40181841,    4.96190913],
        [-133.40742827,    5.21184582],
        [-133.41452096,    5.46174482],
        [-133.42309623,    5.71159734],
        [-133.43315379,    5.96139458],
        [-133.44114621,    6.14402199]]),
 'entry_lanes': ['69004', '69985'],
 'exit_lanes': [],
 'left_neighbor': [],
 'right_neighbor': [],
 'polygon': array([[-136.37195491,   -7.96611503],
        [-125.68178997,   -7.9613382 ],
        [-126.86524811,   -6.55469659],
        [-128.22144533,   -4.80540963],
        [-129.02879057,   -3.30085262],
        [-129.61351513,   -1.7177102 ],
        [-129.82762106,   -0.67074242],
        [-130.11904544,    1.13875256],
        [-130.33613772,    3.60028601],
        [-130.44235647,    7.54115289],
        [-136.3086335 ,    7.50745135],
        [-136.30514196,   -7.22451779],
        [-136.30267684,   -7.67788929],
        [-136.37195491,   -7.96611503]])}

It does not have a width key word

And can not find lane for traffic light. occur in metadrive/manager/scenario_light_manager.py, line 33.
WARNING:/home/lzh/miniconda3/envs/metadrive/lib/python3.9/site-packages/metadrive/manager/scenario_light_manager.py:Can not find lane for this traffic light. Skip!
Traceback (most recent call last):
  File "/home/lzh/.local/share/JetBrains/Toolbox/apps/PyCharm-P/ch-0/232.9559.58/plugins/python/helpers/pydev/pydevd.py", line 1500, in _exec
    pydev_imports.execfile(file, globals, locals)  # execute the script
  File "/home/lzh/.local/share/JetBrains/Toolbox/apps/PyCharm-P/ch-0/232.9559.58/plugins/python/helpers/pydev/_pydev_imps/_pydev_execfile.py", line 18, in execfile
    exec(compile(contents+"\n", file, 'exec'), glob, loc)
  File "/home/lzh/PycharmProjects/scenarionet/scenarionet/sim.py", line 53, in <module>
    env.reset(seed=index if args.scenario_index is None else args.scenario_index)
  File "/home/lzh/miniconda3/envs/metadrive/lib/python3.9/site-packages/metadrive/envs/base_env.py", line 476, in reset
    self.engine.reset()
  File "/home/lzh/miniconda3/envs/metadrive/lib/python3.9/site-packages/metadrive/engine/base_engine.py", line 303, in reset
    manager.after_reset()
  File "/home/lzh/miniconda3/envs/metadrive/lib/python3.9/site-packages/metadrive/manager/scenario_light_manager.py", line 38, in after_reset
    raise ValueError(
ValueError: Can not find lane for this traffic light. Set skip_missing_light=True for skipping missing light!

I set it as True and then run successful, but there are no traffic light in the simulation. Maybe there are some mistakes in line 33?
if str(scenario_lane_id) not in self.engine.current_map.road_network.graph

Hi,
For the first issue, please install MetaDrive from the github repo instead of PyPI. We fixed this issue already.
For the second issue, it is raised by the data itself. The traffic light's lane is not in the map feature thus we don't know where to put that light. We are going to relax this restriction. If there are no corresponding lanes, we will not construct the light rather than throw the error.

I think the scenario you are testing has lights at the edge of the map, which are skipped as no lane is found for them.

Hi,
For the first issue, please install MetaDrive from the github repo instead of PyPI. We fixed this issue already.
For the second issue, it is raised by the data itself. The traffic light's lane is not in the map feature thus we don't know where to put that light. We are going to relax this restriction. If there are no corresponding lanes, we will not construct the light rather than throw the error.

Thank you for your advice, the first issue has been solved. Maybe the second problem is that you mentioned, but I can't see the traffic light in the metadrive scene so can't be sure about this.
Moreover, I found the texture of scene (ground) is incomplete. I'm not sure if other datasets have this problem.

Thank you for the feedback. It is highly likely it is a bug. Now we use a new way to build the terrain in the latest main branch, which may help with this issue, I believe

Yes, the simulation picture is much better now. Very quick fix!


Bump gcc and ruby module versions

Latest puppet-gcc release works on systems with latest XCode/CLT— their clang broke gcc compilation.
Latest puppet-ruby has a number of fixes and enhancements.

+1

Closing due to this issue being stale. If you can still reproduce or would like to continue this discussion, please let me know and I'll reopen this.