Add support for entry generator extensions

See https://github.com/utopia-rise/godot-jvm/pull/32

Closing for now as it has no priority and the changes made to support script inheritance and incremental builds make this proposal's implementation incompatible.
Once we're at a more stable stage I reopen the PR with an updated impl.

Closing for now as it has no priority and the changes made to support script inheritance and incremental builds make this proposal's implementation incompatible.
Once we're at a more stable stage I reopen the PR with an updated impl.


Recreating HAProxy doesn't always reconnect security groups correctly for Public IPs

We have had two occurrences of the same issue in different environments (one being my test environment so I can provide detailed logs etc, and try fixes)
This happens when HAProxy is connected to a Public IP ie:
- name: ha_proxy-partition-e98f67a70e18480cae2et :%
  templates:
  - name: haproxy
    release: cf
  - name: metron_agent
    release: cf
  - name: consul_agent
    release: cf
  lifecycle: service
  instances: 1
  resource_pool: ha_proxy-partition-e98f67a70e18480cae20
  networks:
  - name: default
    default:
    - dns
    - gateway
  - name: public
    static_ips: [x.x.x.x] # <--- Replace with your reserved public IP address

If the HAProxy is recreated (single point of failure will create some downtime, that's understood), but when it comes back up no traffic can go through on 443 or 80.
Adding an Azure Network Security  rule for 443 and 80 fixes this, however it should be automatic.

CPI does not create network security group and add rules. You need to create NSG and add rules by yourself. Then you can specify the name of NSG in the manifest. It will be more flexible if you have many security groups and different rules.

@haydonryan https://github.com/cloudfoundry/docs-bosh/blob/master/azure-cpi.html.md.erb#L72 and https://github.com/cloudfoundry-incubator/bosh-azure-cpi-release/releases/tag/v9


Update with Vagrant setup guide

Added the steps I followed to get Vagrant and Node to play nice in Windows, and organized the tutorial to land after the Virtual Box setup instructions.

This is great thanks @nolanruss.


feat: fix tailcall with version

Hi, I'd like to avoid shell scripts since they often create issues down the line, and their versions cannot be updated by renovate. If you guys address the issues with the npm package (https://github.com/graphql-crystal/benchmarks/pull/210#issuecomment-1802579991), we can switch to that.


Prevent error if no collections present

If there are no collections you can get a "Invalid argument supplied for foreach()" without an empty default.

Thanks!


bpf_host: describe the position of {to,from}-{host,netdev} in the data path

Add some comments about when and where exactly the BPF host entrypoints are
attached to.
This would've been hugely helpful in building a mental map of the datapath, and to get a feel of which logic is applied to which packets in which direction. There are likely some things missing, anyone have something to add?

@brb @pchaigno Thanks for the feedback, I believe this is ready for round 2!

I think we can ignore the checkpatch warning.

Comments only, no need to run tests except for checkpatch.


Next release (3.5?)

Is 2.25 the current stable version and will 3.5 be released sometime in the future as a non-SNAPSHOT version?

Yes, 3.5 will be out as zoon as scsslint works with the new sass

Any help on  brigade/scss-lint#877 would be hugely appreciated

I've pushed 3.5-RC1 so an up-to-date sass version is available, but with broken scss-lint reporting.


Add support for --ssl-no-revoke

Issue #47
Add a new ENV var called COCOAPODS_ART_SSL_NO_REOVKE to be able to pass the flag --ssl-no-revoke when using curl with WinSSL

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

recheckcla

recheckcla

@shayfisher -
Looks like this PR is pending for too long! Sorry about that.
Would you mind documenting the new environment varoable in the README as well? We'll make sure to merge this right after you add it.

Just a reminder that there's still a typo in the variable name, someone should make sure that's fixed before the merge!


Slow performance with numpy.record data type (only when passed as parameter)

Reporting a bug

[X] I am using the latest released version of Numba.
[X] I have included below a minimal working reproducer.

The performance with a numpy.record data type when it was passed as a parameter to a jit function is slow compared to passing a single-element numpy.recarray to a jit function and unpacking and using the numpy.record within the function.
Reproducer as gist:
In [1]:

from collections import namedtuple
import numba as nb
import numba.types as types

In [2]:

@nb.jit('int64(int64)')
def ask(a):
    return a + 1

Numpy recordarray (1d):
In [3]:

import numpy as np

state_dtype = np.dtype([
    ('a', np.int64),
    ('b', np.int64),
])

state = np.rec.array([(1, 2)], dtype=state_dtype)

In [4]:

@nb.jit
def test(state):
    state = state[0]
    b = 0
    for i in range(10000000000):
        b += ask(state.a)
    return b

In [5]:

%%timeit -n200 -r10000 test(state)
test(state)

The slowest run took 44.24 times longer than the fastest. This could mean that an intermediate result is being cached.
667 ns ± 531 ns per loop (mean ± std. dev. of 10000 runs, 200 loops each)

Numpy record:
In [6]:

import numpy as np

state_dtype = np.dtype([
    ('a', np.int64),
    ('b', np.int64),
])

state = np.rec.array([(1, 2)], dtype=state_dtype)
state = state[0]

In [7]:

@nb.jit
def test(state):
    b = 0
    for i in range(10000000000):
        b += ask(state.a)
    return b

In [8]:

%%timeit -n200 -r10000 test(state)
test(state)

The slowest run took 12.18 times longer than the fastest. This could mean that an intermediate result is being cached.
1.98 µs ± 1.09 µs per loop (mean ± std. dev. of 10000 runs, 200 loops each)

In [9]:

@nb.jit
def test(state):
    a = state.a
    b = 0
    for i in range(10000000000):
        b += ask(a)
    return b

In [10]:

%%timeit -n200 -r10000 test(state)
test(state)

The slowest run took 11.28 times longer than the fastest. This could mean that an intermediate result is being cached.
1.93 µs ± 916 ns per loop (mean ± std. dev. of 10000 runs, 200 loops each)

I can reproduce this; copy/pasteable reproducer:
from collections import namedtuple
import numba as nb
import numba.types as types

@nb.jit('int64(int64)')
def ask(a):
    return a + 1

# Numpy recordarray (1d):

import numpy as np

state_dtype = np.dtype([
    ('a', np.int64),
    ('b', np.int64),
])

state = np.rec.array([(1, 2)], dtype=state_dtype)

@nb.jit
def test(state):
    state = state[0]
    b = 0
    for i in range(10000000000):
        b += ask(state.a)
    return b

%timeit -n200 -r10000 test(state)
test(state)

# Numpy record:

import numpy as np

state_dtype = np.dtype([
    ('a', np.int64),
    ('b', np.int64),
])

state = np.rec.array([(1, 2)], dtype=state_dtype)
state = state[0]

@nb.jit
def test(state):
    b = 0
    for i in range(10000000000):
        b += ask(state.a)
    return b

%timeit -n200 -r10000 test(state)
test(state)

@nb.jit
def test(state):
    a = state.a
    b = 0
    for i in range(10000000000):
        b += ask(a)
    return b

%timeit -n200 -r10000 test(state)
test(state)

produces for me:
$ ipython repro.ipy
The slowest run took 969.25 times longer than the fastest. This could mean that an intermediate result is being cached.
288 ns ± 2.51 µs per loop (mean ± std. dev. of 10000 runs, 200 loops each)
The slowest run took 269.51 times longer than the fastest. This could mean that an intermediate result is being cached.
809 ns ± 2.07 µs per loop (mean ± std. dev. of 10000 runs, 200 loops each)
The slowest run took 270.75 times longer than the fastest. This could mean that an intermediate result is being cached.
839 ns ± 2.12 µs per loop (mean ± std. dev. of 10000 runs, 200 loops each)

I suspect the boxing / unboxing of a record might be more time-consuming than that of an array.


Explicitly set destroyScheduledDuration to 24h

GCP is changing the default value from 24h to 30 days in a few months. This change makes sure that everything remains at the current value.

100% we want to make this configurable. However, this will at least keep things with the status quo for now. Otherwise it will switch to 30 days and then we'll be paying for destroyed keys even longer.


SVG on your site is not showing

@acburst @Dogfalo There is an issue with your site, the SVG image is not showing and is issuing 2 errors in chrome dev tools. Not sure if you have noticed this at all?

How to reproduce ?


[derive] Disable prelude when testing; fix bugs

This has the effect of ensuring that derive-emitted code will fail to compile if it spuriously relies on certain identifiers being in scope - namely, identifiers which are part of the prelude.
Disabling the prelude surfaced a few bugs which are also fixed in this commit.
Makes progress on #11

Closes https://github.com/google/zerocopy/issues/882


New package: Samsung.EasyFileShare version 1.2.4

[ ] Have you signed the Contributor License Agreement?
[ ] Have you checked that there aren't other open pull requests for the same manifest update/change?
[ ] This PR only modifies one (1) manifest
[ ] Have you validated your manifest locally with winget validate --manifest <path>?
[ ] Have you tested your manifest locally with winget install --manifest <path>?
[ ] Does your manifest conform to the 1.4 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: codeflow:open?pullrequest=https://github.com/microsoft/winget-pkgs/pull/111991&drop=dogfoodAlpha

Manual Validation failed with:

2023-07-21 15:19:02.145 [CLI ] Installer [X64,exe,Unknown,en-US] not applicable: Architecture was excluded by caller : X64
2023-07-21 15:19:02.146 [CLI ] No suitable installer found for manifest Microsoft.DotNet.Framework.DeveloperPack_4 with version 4.8.1
2023-07-21 15:19:02.147 [CLI ] Terminating context: 0x8a150104 at D:\a_work\1\s\external\pkg\src\AppInstallerCLICore\Workflows\DependenciesFlow.cpp:126

It appears that the dependency doesn't have a valid matching installer.

Tag: Dependencies

[Policy] Dependencies

/AzurePipelines run

Automatic Validation ended with:

Unfortunately this is a Windows Feature, and I do not know when (if?) we will support this.

[Policy] Windows Feature

@wingetbot run


Streams 101

Suggest @jakearchibald

Where would this live?  In the Primers section?

Are streams stable enough/ready for prime time use? If not, we should consider this for updates instead.

Prio wise I would say #3469 is probably higher prio for us to knock out this week.

Reading a fetch using streams has been in stable for a year now I think. Creating your own stream in a service worker is much newer, but also in stable.

Before figuring out where this would live, I'd like to get a better idea of what it'll cover. My ideal would be for it to go into Instant and Offline and provide actionable guidance on how to use streams to create (near) instant experiences.


docs: add mkdocs page for documentation

mkdocs page for docs.
I don't know how familiar you are with GitHub Pages, but this can be easily published. See the following:

https://www.mkdocs.org/user-guide/deploying-your-docs/
https://squidfunk.github.io/mkdocs-material/publishing-your-site/

I would also be happy to configure that, but then you'd need to trust me with some higher permissions on the repo.
I suggest that we move all content to the page and keep the readme to a minimum with some pointers to the page.

Thanks, I agree that moving most of the docs into a separate folder would be better.
Considering hosting I'm going to have a look at the links you provided. So far I've been using readthedocs.org for other projects but I don't mind using another approach.

This looks like a big improvement! But this PR doesn't currently touch the top level README at all, so some content is now duplicated (e.g. the table with supported languages). Shouldn't the sections that have been moved to the docs be removed from the README?

@adbar

We could use links to the relevant sections.

Not sure I understood. Was this directed to me? Should I be more specific in pointing out the duplicated text? Sure, I can do that if it helps. I thought it was quite obvious looking at the PR.

Hi guys!
Sorry for the delayer response. You got me on vacation!
I agree with @adbar  that the readme should be reduced to a minimum and just point to the Github page hosting the docs.

I'd be in favor of sphinx because the autodoc function will prove useful to automatically reflect changes made to the functions or classes. I'm open to try mkdocs but I don't know it as well.
As for hosting I've used readthedocs.org because it integrates seamlessly with Github but so do Github pages, it's just not so easy I guess.
Since we need to move on I would be in favor of accepting the PR as is and address the other questions later:

mkdocs or sphinx ?
github.io or readthedocs.org ?

@juanjoDiaz @osma What do you think?

Hi @adbar ,
I think that github.io is preferred as it's becoming the standard and it's simpler to keep everything in the same platform.
Regarding mkdocs vs sphinx, I chose mkdocs because it was easy and I was familiar with it.
I can try to give sphinx a go this week but I can promise that I will be able to complete it.

OK then I think I'll just merge the PR as it is and try to get familiar with the process.


Adjudicator Intake User Story

Currently writing them on the figma file next to the wireframes https://www.figma.com/file/nhOquOVmWD1P3BxEY0KIh5/CCBC-Intake?node-id=0%3A1


Docs: Correct database and user API statements (BTS-429)

Backport of #14617

[ ] http://jenkins01.arangodb.biz:8080/view/Documentation/job/arangodb-ANY-examples/1030/


flow: Add concrete type for EventUpdateMessageAction

This is the event fired when a message on the server have been updated.
There are three cases that could cause that:

message's content have changed
message's topic have changed
both content and topic have changed

Curenly we are handling only the content change and partially the
topic change.
We are also not doing anything with the propagate_mode.
These types are correct but the tests are failing because the reducer
was not aware of all the incoming data formats (it either changed
since we implemented it, or we implemented it incompletely).
Now thanks to Flow we know what remains to be done.

Cool. This is a WIP branch not meant to be merged yet, right? Since it doesn't pass the type-checker as is.
One other change I'd suggest: it seems like the content and topic variants have a lot in common. I'd try to write that part down once, and then write down the content group and the topic group and go and combine them all appropriately below. I think that would help these types to better make it clear what's going on.

I did split the common props in EventUpdateMessageCommon.
@jainkuniya while working on implementing this, make sure to have this branch starts passing the Flow check.


Support SR v4 Power Board

Support SR v4 Power Board.
Prerequisites:

Button Support - #44
Power Output Support - #61
Battery (voltage / status) - #62
Piezo - #60

Since I happen to be looking at this, the mapping of output numbers (0 to 5) used in the wire protocol to outputs labelled on the board is as follows:
0 - H0
1 - H1
2 - L0
3 - L1
4 - L2
5 - L3

For consistency we should use the same numbering for identifying PowerOutput components.


How do you render Dash components passed to other components?

Hello!
I am porting over MUI's DataGrid to Dash and want to pass any kind component (ie. dbc.Button,  dcc.Inpput, etc) to fields in that table and then have access to them in Dash to wire callbacks to them. As of now, I am unpacking the object into its props using a function from the dash-extensions-js package (https://github.com/thedirtyfew/dash-extensions-js/blob/master/index.js), and then using DataGrid's renderCell (https://mui.com/x/react-data-grid/column-definition/#rendering-cells) call in the column to display those props as a React element; this works fine.
The issue comes when wiring a callback to the element in the table: Dash does not know about the element in the table and gives an "ID is not found" error. If I inspect the page the element is there and it has the correct ID.
I was wondering if someone would be able to talk me through how you all unpack components passed to other components to see if there is something I am missing that would help me unlock that wiring component. Happy to share any code if that would help as well.
Thank you!

The my knowledge, this is currently not possible. It would probably require an update to the Dash renderer itself.
If you manage to get it to work, please let know :)


Allow users to create accounts, or at least allow them to request the creation of an account.

As of right now, users have to find someone who can add them to the site in order to try any of the problems. It would be nice if we could instead have some way for users to create their own accounts (maybe just use the one from gauacm.com?). We could force them to register an email, or even just have a request an account tab where they give a name and an email and you can email them a username and password.

This would be a cool feature, I think they should sign up with an email, then have to confirm their email using a randomly generated url.

How do we feel about requiring an auburn.edu email?

I'm fine with that.

I think requiring an .edu address would be sufficient, it would prevent 5 minute mail spam and would allow something like the IBOP to go much smother if we host it on auacm

Or we could run IBOP on a clone and remove that requirement, like we did for Google


Couln't Connect to GDscript language in VS CODE

Godot version
4.0.2
System information
Windows 10,  Intel(R)HD Graphics Family, <IP_ADDRESS>71,  GLES3
Issue description

Steps to reproduce

i downloaded GD-TOOLS
go to project settings
go down to godot
Added  my executable godot path in Editor_PATH
Added my tscn file path(universal) in Scene file config

sadly the error still occurs

Minimal reproduction project
N/A

The

Closing in favor of https://github.com/godotengine/godot-vscode-plugin/issues/473, as this is an issue with the plugin.


⚠ SceneTime has degraded performance

In d6ad2b0, SceneTime ($ST) experienced degraded performance:

HTTP code: 200
Response time: 5771 ms

Resolved: SceneTime performance has improved in bee367b after 8 minutes.


Realms GUI not shifted correctly (button is, rest isn't)

Without ModMenu:
http://imgur.com/uSN1csel.png
With ModMenu:
http://imgur.com/UHb3Tzfl.png
The button itself gets nudged up a bit, but the blue blinking thing and the icon don't.

Info for me on solving: https://discordapp.com/channels/507304429255393322/507982478276034570/577332496295919632

Oops, this was solved in v1.5.4; forgot to close


ticks.display: 'top'

Allow drawing tick labels after datasets.
Fixes: #2808
Fixes: #3724
Fixes: #3738
Fixes: #3722
Demo

Changed the option to 'above' instead of 'top'. Demo changed.
It gets cluttery and ugly fast checking ticks.display === 'above' in core.controller, so did not do that.
Also this leaves the possibility to add gridLines.display: 'above' for example, without any further changes to core.controller`.

Left out  || (above && optionTicks.display !== 'above')  in https://github.com/chartjs/Chart.js/pull/6226/commits/a3d1435a1fcae7985fe3e5784c6bdcbdd74732ab nonintentionally, but I see there are reviews coming so will get to that later.

Closed via #6241


Failed to parse connection id error under CosmosDB/Connections tree item

When at least one connection is broken error message appears. I think will be better to show all others connections that aren't broken and at the bottom of list show one/multiple error nodes/messages.
.
Also, the text of the message error could be better.

Not relevant now.


Plots: Documentation

Description of suggestion or shortcoming:
I'm not sure if this is a bug or feature, but anyway:
The Storybook is doing a mediocre job at giving all of the relevant settings, because many are in ``distributionChartSettings.

Most of the new charts aren't in the Documentation yet.
Maybe:

Improve the documentation a bit. Maybe put links to the Components website there.
Try to find a way to improve the Docs, to make this more apparent.

Done in #1830.


quarto: 1.2.269 -> 1.2.280

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/quarto-dev/quarto-cli/releases.
meta.description for quarto is: "Open-source scientific and technical publishing system built on Pandoc"
meta.homepage for quarto is: "https://quarto.org/"
meta.changelog for quarto is: "https://github.com/quarto-dev/quarto-cli/releases/tag/v1.2.280"
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.2.280 with grep in /nix/store/qzki8k00wr89fkwslgziv8nngcvls8m7-quarto-1.2.280
found 1.2.280 in filename of file in /nix/store/qzki8k00wr89fkwslgziv8nngcvls8m7-quarto-1.2.280

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
quarto

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/qzki8k00wr89fkwslgziv8nngcvls8m7-quarto-1.2.280 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A quarto https://github.com/r-ryantm/nixpkgs/archive/fe0a477fa2a697b9704150cc3c986cd818628df2.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/qzki8k00wr89fkwslgziv8nngcvls8m7-quarto-1.2.280
ls -la /nix/store/qzki8k00wr89fkwslgziv8nngcvls8m7-quarto-1.2.280/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

quarto

Maintainer pings
cc @MrTarantoga for testing.

Result of nixpkgs-review pr 207791 run on x86_64-linux 1

1 package built:

quarto


can't install on windows julia 0.4

I'm trying to install IJulia on windows 7. I made a fresh install of the latest julia 0.4 rc binary. Also a fresh anaconda win64 python3.4 installation and I ran conda install ipython-notebook successfully.
However, adding (and building) IJulia creates following error:
INFO: Downloading https://cache.e.ip.saba.us/http://download.opensuse.org/reposi
tories/windows:/mingw:/win32/openSUSE_13.1/repodata/repomd.xml
INFO: Downloading https://cache.e.ip.saba.us/http://download.opensuse.org/reposi
tories/windows:/mingw:/win64/openSUSE_13.1/repodata/repomd.xml
INFO: Building Nettle
INFO: Updating WinRPM package list
INFO: Downloading https://cache.e.ip.saba.us/http://download.opensuse.org/reposi
tories/windows:/mingw:/win32/openSUSE_13.1/repodata/repomd.xml
INFO: Downloading https://cache.e.ip.saba.us/http://download.opensuse.org/reposi
tories/windows:/mingw:/win64/openSUSE_13.1/repodata/repomd.xml
INFO: Building ZMQ
INFO: Building IJulia
Found jupyter kernelspec version 4.0.0 ... ok.
Writing IJulia kernelspec to C:\Users\Ken\AppData\Local\Temp\jul97CD.tmp\julia-0
.4\kernel.json ...
Copying logo-32x32.png to Julia kernelspec.
Copying logo-64x64.png to Julia kernelspec.
Installing julia kernelspec julia-0.4

julia> Traceback (most recent call last):
  File "C:\CodingProgs\Anaconda\Scripts\jupyter-kernelspec-script.py", line 7, i
n <module>
    main()
  File "C:\CodingProgs\Anaconda\Scripts\jupyter-kernelspec-script.py", line 4, i
n main
    KernelSpecApp.launch_instance()
  File "C:\CodingProgs\Anaconda\lib\site-packages\traitlets\config\application.p
y", line 592, in launch_instance
    app.start()
  File "C:\CodingProgs\Anaconda\lib\site-packages\jupyter_client\kernelspecapp.p
y", line 177, in start
    return self.subapp.start()
  File "C:\CodingProgs\Anaconda\lib\site-packages\jupyter_client\kernelspecapp.p
y", line 113, in start
    replace=self.replace,
  File "C:\CodingProgs\Anaconda\lib\site-packages\jupyter_client\kernelspec.py",
 line 208, in install_kernel_spec
    shutil.copytree(source_dir, destination)
  File "C:\CodingProgs\Anaconda\lib\shutil.py", line 297, in copytree
    names = os.listdir(src)
FileNotFoundError: [WinError 3] The system cannot find the path specified: 'C:\\
Users\\Ken\\AppData\\Local\\Temp\\jul97CD.tmp\\julia-0.4'

FYI:
julia> versioninfo()
Julia Version 0.4.0-rc1
Commit e5c6964 (2015-09-09 16:07 UTC)
Platform Info:
  System: Windows (x86_64-w64-mingw32)
  CPU: Intel(R) Core(TM) i7-3770K CPU @ 3.50GHz
  WORD_SIZE: 64
  BLAS: libopenblas (USE64BITINT DYNAMIC_ARCH NO_AFFINITY Sandybridge)
  LAPACK: libopenblas
  LIBM: libopenlibm
  LLVM: libLLVM-3.3

Let me know what I can do to help and debug.

@minrk, this looks like a problem introduced by #353.  Any ideas?

Arg, I never get things right on Windows on my first go, sorry about that. Yes, it's definitely due to #353. The bug isn't immediately obvious to me, but it may involve escapes and backslashes.

Normally you don't have to escape anything for run, because run bypasses the shell entirely and spawns the process directly with the specified arguments.

julia> Traceback (most recent call last):
  File "C:\Anaconda3\Scripts\jupyter-kernelspec-script.py", line 7, in <module>
    main()
  File "C:\Anaconda3\Scripts\jupyter-kernelspec-script.py", line 4, in main
    KernelSpecApp.launch_instance()
  File "C:\Anaconda3\lib\site-packages\traitlets\config\application.py", line 592, in launch_instance
    app.start()
  File "C:\Anaconda3\lib\site-packages\jupyter_client\kernelspecapp.py", line 177, in start
    return self.subapp.start()
  File "C:\Anaconda3\lib\site-packages\jupyter_client\kernelspecapp.py", line 113, in start
    replace=self.replace,
  File "C:\Anaconda3\lib\site-packages\jupyter_client\kernelspec.py", line 208, in install_kernel_spec
    shutil.copytree(source_dir, destination)
  File "C:\Anaconda3\lib\shutil.py", line 297, in copytree
    names = os.listdir(src)
FileNotFoundError: [WinError 3] The system cannot find the path specified: 'C:\\Users\\mmoh\\AppData\\Local\\Temp\\jul5955.tmp\\julia-0.5'

Also
Julia Version 0.5.0-dev+9
Commit 248eb0b (2015-09-10 04:03 UTC)
Platform Info:
  System: Windows (x86_64-w64-mingw32)
  CPU: Intel(R) Core(TM) i7-4510U CPU @ 2.00GHz
  WORD_SIZE: 64
  BLAS: libopenblas (USE64BITINT DYNAMIC_ARCH NO_AFFINITY Haswell)
  LAPACK: libopenblas
  LIBM: libopenlibm
  LLVM: libLLVM-3.3

Same problem. This happened when updating to newest Julia version. I hope this can be updated soon.

Meanwhile I found out that adding a sleep(1) before removing the temporary directory in the last line of the build.jl will make the build pass.
However, even when the build finishes successful, it will not run jupyter with this kernel on windows. I think it has to do with escape_string for windows backslashes in the path. If a define an extra function
esc_str(s::AbstractString) = @windows ? s : escape_string(s) and substitute the escape_string, it works. I haven't tried on non-windows.
These are hacks, but it now works on my system.
Unrelated to this issue: #353 removed the find_ipython function, but that function is still called in the notebook() default argument on this line.

Thanks, notebook is fixed by b4fd27f49ce2ec5e790fa75b1d34e5ca60a044a5.

I think I'm just going to get rid of the temporary directory thing, and put it in a permanent directory under deps.  I don't think leaving an extra copy of the directory lying around is a problem.

The call to escape_string should be removed entirely because JSON.json already escapes strings as needed.

Should be fixed now; let me know if not.

(Still, it is somewhat bizarre that jupyter kernelspec install is affected by an rm that occurs after the process has returned, since the run function is synchronous.  @minrk, does kernelspec install spawn a detached process or something?)

@stevengj  Thanks! Did a Pkg.checkout("IJulia") and everything is back to working now.

Ah, I think I know what's going on with the return. It's a bizarre Windows behavior, but I think I know how to fix it.

@minrk, I'm glad someone understands these things.  When you fix it, please link the fix from here so that I can be enlightened.

@stevengj it's jupyter/jupyter_core#54. Python provides an exec command on Windows, even though Windows doesn't actually have exec. The behavior is more akin to fork & exec, so the jupyter command was returning immediately, while the exec'd process (jupyter-kernelspec) continued in the background.


feat(sdk): add page to create service account token

Pull Request Test Coverage Report for Build<PHONE_NUMBER>2
Details

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

Totals

Change from base Build<PHONE_NUMBER>2:
0.0%

Covered Lines:
7624

Relevant Lines:
24735

💛  - Coveralls


Error:(52, 57) java: interface expected here

After adding the Ultimate plaftform and Flex I am getting the following error message:
/home/bence/tools/intellij-haxe/src/com/intellij/plugins/haxe/editor/HaxeReferenceCopyPasteProcessor.java
Error:(52, 57) java: interface expected here

The code involved:
public class HaxeReferenceCopyPasteProcessor implements CopyPastePostProcessor  {

When I check the class in question:
public abstract class CopyPastePostProcessor<T extends TextBlockTransferableData> {

it is indeed an abstract class; not an interface.
Note: I am using the UI-139.1117.1 trial downloadable from the IntelliJ website as the platform, but compiling under Community Edition - I don't think this can be a problem though.

You are using the code branch for Idea 13 and compiling with Idea 14.  What you are seeing is the result of an incompatible change between the two versions.  The Idea14 code is available in masterIdea14 branch on the TiVo fork.  We are pursuing options to get a unified source base again.  Until that time, we are merging code into both branches.

I am using the UI-139.1117.1 trial downloadable from the IntelliJ website as the platform, but compiling under Community Edition - I don't think this can be a problem though.

Actually... the plugin uses portions of Idea Ultimate for the navigate->hierarchy and diagram views.  You can probably build using community edition using an Ultimate SDK, but you won't successfully run the plugin under CE. (At least, not all of the functionality will work.)


Submission for Actigraphy-Viz-Challenge by Hannah Rolf

@HannahRolf: Congratulations on this visualization and for participating. Could you please send me an email to<EMAIL_ADDRESS>


Remove vendor prefixes when setting userSelect style

Removes vendor prefixes for userSelect style when disabling or re-enabling text selection. Since this property is now supported by all major browsers it's no longer required to add a vendor prefix.

I feel like the disableTextSelection and enableTextSelection functions could be optimized further. The only browser that does not seem to implement the selectstart event is Safari on iOS (see MDN docs).
Not sure if this info is up-to-date, but I have a gut feeling this could still be optimized. Perhaps by only using user-select, since the property is supported everywhere.
Either way, it would require further investigation I can't get into right now.

@jonkoops I was wary about this change before because MDN states user-select is still prefixed on Safari (both desktop and mobile). Can you please confirm we didn't break it there by removing the prefix?


ShopCommandTest

Test for ShopCommand
Delete a file that leaks from a currently disabled test.
I think there is an error with "reprice d20 @ 1,333 limit 1" that results in a repriced item with no limit.  I will look at that later.

I fixed the parsing error although it is hard to prove without actually running the request.
I feel like ShopCommand needs to be refactored.  I'd have some kind of input validation and abort if there were problems.  I'm not sure whether that is Better or just Different, but for the moment I'm not tackling it.

I would rather we tested based on what the state of our shop was after running certain commands, rather than the logging output which could easily change. But mocking the HTTP traffic would be very complex and this is a good bit of testing to have. Thank you!

Agreed.  But at under 20% coverage every little bit helps.  And I did discover and fix a bug, although in the absence of mocking the request the debugger was required to understand that.  But this does make me want to look at ShopManager for testing :-)


Release library as Nuget Package

While it is great you have an SDK here it would be useful to have it released on Nuget.org as a package (ideally a .Net Standard one) so that updates can easily be consumed. If it is already available on Nuget perhaps you could put it up under releases.

Thanks for reporting this, it is in our backlog and should be online shortly.

Will Mono be supported as well?

Do you plan support .Net standard and uwp target?

Too bad, it's not possible to use de PCL in Xamarin. Do you plan to add it?

Is this yet available on NuGet?

Apparently not yet, not even as prerelease.

Guess we can close this after all :wink:

Yep. I guess I'm glad they never did it, otherwise I would have wasted a lot of time on a project that was destined to die.


Mail Queue Problem Illuminate\Mail with Lumen

I got fatal error: Call to a member function push() when run:
Mail::queue('email.blank', ['html' => $html], function($msg) {
            $msg->to(['email@gmail.com']);
            $msg->from(['email@gmail.com']);
            $msg->subject('Lumen Service Mail');
        });

Its work if change file: Illuminate\Mail\Mailer.php
Add
use \Illuminate\Support\Facades\Queue as Queue;
And change line 205:
from: return $this->queue->push('mailer@handleQueuedMessage', compact('view', 'data', 'callback'), $queue);
to return Queue::push('mailer@handleQueuedMessage', compact('view', 'data', 'callback'), $queue);
this work for me!

That's NOT the correct solution.

All you need to do is make sure you've used the queue before you use mail. That's just due to the way Lumen doesn't even bother to bind stuff unless you try to use it, to save time. If you don't like that behaviour, I'd recommend just using Laravel.
https://github.com/laravel/framework/blob/5.1/src/Illuminate/Mail/MailServiceProvider.php#L77

Thank you for your response.
Im using the queue only to send emails. Do you have any suggestion to start the queue before send email?
-----Mensagem Original-----
De: "Graham Campbell"<EMAIL_ADDRESS>Enviada em: 30/08/2015 04:54
Para: "laravel/framework"<EMAIL_ADDRESS>Cc: "Rodrigo Lacerda"<EMAIL_ADDRESS>Assunto: Re: [framework] Mail Queue Problem Illuminate\Mail with Lumen(#10083)
Closed #10083.
—
Reply to this email directly or view it on GitHub.

I have the same issue and I would also like to know how to start the queue before sending email.
This would also add a valid solution for this issue: https://github.com/laravel/lumen-framework/issues/77

Like graham said this is not the right solution. You can check the implementation of the Illuminate\Mail\Mailer class and you will see it has a setQueue method so by doing:
public function __construct()
{
    $this->mailer = app('mailer');
    $this->mailer->setQueue(app('queue')->connection('redis'));
}

It is done like that. You can even inject the Mailer or the QueueContract with bindings, however you want.

Create a new job - use the follwoing sample code

namespace App\Jobs;
use App\Jobs\Job;

class SendEmail extends Job
{
    /**
     * Create a new job instance.
     *
     * @return void
     */
    public function __construct()
    {
        //
    }

    /**
     * Execute the job.
     *
     * @return void
     */
    public function handle()
    {
        //
    }
}

and in your controller
include the SendMail under the namepsace;

use App\Jobs\SendEmail;

mail sending method

public function mailSend()
{
        $data = array(
            'title' => 'test title',
            'description' => 'test description',
        );
        Queue::push(new SendEmail);

        Mail::queue('email.mytemplate', $data, function ($message) {
<EMAIL_ADDRESS>'Ishan test mail server');
            $message->to('ishanmail@testmail.com')->subject('test subject');
        });
}

@GrahamCampbell if it's an intended behavior what is the right way to test mailables?

Looks like this problem was fixed in Lumen 5.5.

@chimit The queue isn't being bounded in the mail service provider. So the setQueue isn't called properly.
I found a solution for 5.4's.
Look at here: https://github.com/laravel/lumen-framework/issues/590#issuecomment-316363236

FYI @GrahamCampbell this is a bug. IMHO, all Illuminate elements are should be made to work out of the box with Laravel and Lumen even if they are not explicitly included.
The best fix I've found for this (in Lumen 5.5) is to do the following:
In bootstrap/app.php, make sure the MailServiceProvider is listed before your AppServiceProvider, e.g.:
...
$app->register(Illuminate\Mail\MailServiceProvider::class);
$app->register(App\Providers\AppServiceProvider::class);
...

Then along the lines of @soderluk's comment on this thread, in your AppServiceProvider add the following in your register method:
app('mailer')->setQueue(app('queue'));


Cannot use entity variable in az storage entity insert using Azure CLI 2.40.0

Related command
az storage entity insert
Describe the bug
Using a Powershell variable for inserting a Storage Entity results in error using Azure CLI 2.40. Same command was working fine in Azure CLI 2.37.0. Running the same command without using a variables also works fine.
To Reproduce

Open a PowerShell window and set variables $storageAccount, $tableName, $partitionKey, $rowKey and $content to valid values
Run the commands below and notice the error

$entity = @()
$entity += "PartitionKey=$partitionKey"
$entity += "RowKey=$rowKey"
$entity += "Content=$content"
az storage entity insert -t $tableName --account-name $storageAccount --auth-mode login -e $entity
Actual behavior
incorrect usage: entity requires: RowKey PartitionKey
Expected behavior
An entity is inserted to the table storage.
Environment summary
CLI version 2.40.0
Additional context

@evelyn-ys for awareness

Similar with #23797. Please see detailed info in #23797 's comments

@evelyn-ys Thank you, using @entity instead of $entity indeed fixes the issue!


dashboard: enable dashboard by default

This PR enable dashboard deployment by default
Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1726739

@guits @dsavineau
I've added groups[grafana_server] | length < 1 to check if our Ceph cluster has a Grafana server for now. If there is a better/more correct way, let me know.

jenkins test dev-centos-non_container-all_daemons

jenkins test dev-centos-container-all_daemons

jenkins test dev-centos-container-all_daemons

jenkins test dev-centos-container-all_daemons

jenkins test dev-centos-non_container-all_daemons

jenkins test dev-centos-non_container-lvm_osds

jenkins test dev-centos-non_container-lvm_osds

jenkins test dev-centos-container-all_daemons

jenkins test dev-centos-non_container-all_daemons

jenkins test dev-centos-container-lvm_osds

jenkins test dev-centos-non_container-lvm_osds

jenkins test dev-centos-container-all_daemons

jenkins test dev-centos-container-all_daemons

jenkins test dev-centos-non_container-all_daemons

jenkins test dev-centos-container-lvm_osds

jenkins test dev-centos-non_container-lvm_osds

jenkins test pipeline

jenkins test dev-centos-non_container-all_daemons

jenkins test dev-centos-container-all_daemons

jenkins test pipeline

jenkins test pipeline

jenkins test pipeline

jenkins test dev-centos-container-podman

ceph-iscsi repos are broken in shaman. Therefore, this PR won't pass the CI

This has started breaking all ceph-volume tests. What can we do to skip the dashboard for cases like ceph-volume where it doesn't need one?

dashboard_enabled: false in the ansible configuration

use dashboard_enabled: false

Thanks guys!
/cc @jan--f

Thanks a lot, @guits, @dsavineau, @rishabh-d-dave for addressing this so quickly!

Thanks indeed guys, will push a test fix shortly.


Add a module that prevents breaking obsidian

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.

As obsidian is so hard to break, there isn't really a reason to prevent breaking it.


[BEAM-12153] revert "implement GroupByKey with CombinePerKey with Concatenate CombineFn" because of performance issues.

@ibzib sorry for the other PR for the same change. I need to revert my last PR that you reviewed because I discovered performance issues.
I ran a simple group by key on a 174 million lines dataset (GDELT) and the perfs are as follow:
SimpleGroupByKey on the StructuredStreaming runner before the change: 995s, no OOM
SimpleGroupByKey on the RDD runner: 457s
SimpleGroupByKey on the StructuredStreaming runner after the change: killed after some hours.
I did also tests with Beam GroupByKeyLoadTests and I reproduced OOM because spark spill to disk saturated the filesystem (see comments in the ticket).
So I'm reverting the change because perf is very bad and the OOM does not seem to be linked to the materialization
Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

Run Java PreCommit

Run Java PreCommit

@ibzib thanks for the prompt review ! I'm self merging


flutter with vscode shows white screen when pressing F5

flutter doctor -v shows belows:
$ flutter doctor -v
[√] Flutter (Channel stable, v1.2.1, on Microsoft Windows [Version 6.1.7601], locale zh-CN)
    • Flutter version 1.2.1 at E:\Program Files\flutter
    • Framework revision 8661d8aecd (6 weeks ago), 2019-02-14 19:19:53 -0800
    • Engine revision 3757390fa4
    • Dart version 2.1.2 (build 2.1.2-dev.0.0 0a7dcf17eb)

[√] Android toolchain - develop for Android devices (Android SDK version 28.0.3)
    • Android SDK at E:\AndroidSDK
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-28, build-tools 28.0.3
    • ANDROID_HOME = E:\AndroidSDK
    • Java binary at: E:\Program Files\Android\Android Studio\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1248-b01)
    • All Android licenses accepted.

[√] Android Studio (version 3.3)
    • Android Studio at E:\Program Files\Android\Android Studio
    • Flutter plugin version 33.4.1
    • Dart plugin version 182.5215
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1248-b01)

[√] IntelliJ IDEA Community Edition (version 2018.3)
    • IntelliJ at E:\Program Files\JetBrains\IntelliJ IDEA Community Edition 2018.3.3
    • Flutter plugin version 33.4.2
    • Dart plugin version 183.5901

[√] Proxy Configuration
    • HTTP_PROXY is set
    • NO_PROXY is r.tnpm.oa.com,localhost,<IP_ADDRESS>
    • NO_PROXY contains <IP_ADDRESS>
    • NO_PROXY contains localhost

[√] Connected device (1 available)
    • Android SDK built for x86 • emulator-5554 • android-x86 • Android 9 (API 28) (emulator)

• No issues found!

I meet the same problem, I must retry pressing F5. it will solve sometimes.

Can you run the Dart: Capture Logs command and tick the Flutter Run option, then try to launch your app? Leave it on the white screen for a while (30 seconds +) and then click Stop Logging in the notification (which may have collapsed into the bell icon in the status bar) to get the log.


Fix indexMerger to respect the includeAllDimensions flag

Description
includeAllDimensions was added recently in https://github.com/apache/druid/pull/12276. When useFieldDiscovery and includeAllDimensions are both set, IndexMerger should store every column even if it is empty. However, it currently doesn't respect the includeAllDimensions flag but only checks if the given column is explicitly specified in the dimensionsSpec. This PR fixes this bug by checking the flag properly.
This PR also fixes another bug in JsonInputFormat that implicitly filters null fields out even when useFieldDiscovery is set. Instead of filtering them out in an early stage in ingestion, keepNullColumns will be set automatically if useFieldDiscovery is set, so that MapInputRowParser can check the dimensionsSpec to decide whether to keep the null fields or not. This change will not make any user-facing change as 1) keepNullColumns is undocumented and used only in the sampler and 2) even if there is someone using keepNullColumns, the existing behavior should remain the same. The behavior changes only when includeAllDimensions is set as well as useFieldDiscovery.

Key changed/added classes in this PR

IndexMergerV9
JsonInputFormat

This PR has:

[x] been self-reviewed.
[x] added Javadocs for most classes and all non-trivial methods. Linked related entities via Javadoc links.
[x] added unit tests or modified existing tests to cover new code paths, ensuring the threshold for code coverage is met.

@clintropolis @kfaraz thank you for your review!


Update chart versions for 0.4.0-rc.0

Signed-off-by: Dmitri Gekhtman<EMAIL_ADDRESS>

Why are these changes needed?

Update Helm chart versions for KubeRay 0.4.0 release. This is a PR for the release branch.
I think it's fine to use the same version string for all fields.
Dunno how best to set these things in master (set them all to "nightly"?).
Related issue number

Checks

[ ] I've made sure the tests are passing.
Testing Strategy

[ ] Unit tests
[ ] Manual tests
[ ] This PR is not tested :(

FYI @kevin85421 @sihanwang41


[SPARK-8376][Docs]Add common lang3 to the Spark Flume Sink doc

Commons Lang 3 has been added as one of the dependencies of Spark Flume Sink since #5703. This PR updates the doc for it.

Merged build triggered.

Merged build started.

Test build #34936 has started for   PR 6829 at commit f8617f0.

This is built into the assembly though, right?

This is built into the assembly though, right?

No. Spark Flume Sink does not assemble the dependencies. Actually, now we don't have an assembly jar for Flume.

OK but surely it's easier to make an assembly target than tell people they have to piece together the dependencies and keep updating docs about it?

OK but surely it's easier to make an assembly target than tell people they have to piece together the dependencies and keep updating docs about it?

ping @tdas about the assembly idea.

Test build #34936 has finished for   PR 6829 at commit f8617f0.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

I think the assembly is a good idea. Though for that we will have to

publish the assembly JAR instead of package JAR. It will be cumbersome to add additional flume-sink-assembly directory for this. Maybe we can make the existing flume-sink project generate and publish the assembly instead of the package.
Update instructions. I am not sure this will much impact existing deployments, because they are supposed to download and run the version of sink that is necessary for the version of Spark they are running.

@harishreedharan What do you think about this.

Jenkins, retest this please.

(I'm retesting this to see whether our new Jenkins PRB script is properly skipping the tests for doc-only changes)

Merged build triggered.

Merged build started.

Test build #35074 has started for   PR 6829 at commit f8617f0.

Test build #35074 has finished for   PR 6829 at commit f8617f0.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

+1 on creating assembly. I am not entirely sure what it takes to generate the assembly, but if it is possible to add that to the current sink module, that would be great. I doubt this would affect any existing deployments in any way.

@zsxwing Then lets try to build an assembly. But for the benefit of branch-1.4 I am going to merge this PR to master and 1.4 (so that the docs are updated for 1.4.1). But then lets create a separate JIRA and PR for the assembly.

@srowen do you have an example to publish both the single jar and the assembly jar?
Two approaches I'm thinking about:

Upload the assembly jar in the same artifact: flume-sink.
Create a new artifact for the assembly jar just like the kafka-assembly module. And we need to generate a new pom.xml for the assembly jar.

I prefer 1 because I don't know how to implement 2 in maven. What do you think?

Actually, I dont think we can publish two artifacts from same project. Nor
do we need to. No one needs to the single jar for anything, only the
assembly jar. Rather we should publish just publish the assembly. May be
maintain the same name just publish the assembly jar instead.
On Thu, Jun 18, 2015 at 6:59 PM, Shixiong Zhu<EMAIL_ADDRESS>wrote:

@srowen https://github.com/srowen do you have an example to publish
both the single jar and the assembly jar?
Two approaches I'm thinking about:

Upload the assembly jar in the same artifact: flume-sink.
Create a new artifact for the assembly jar just like the kafka-assembly
module. And we need to generate a new pom.xml for the assembly jar.

I prefer 1 because I don't know how to implement 2 in maven. What do you
think?
—
Reply to this email directly or view it on GitHub
https://github.com/apache/spark/pull/6829#issuecomment-113339743.

Publishing the single jar would be helpful if people find some dependency conflicts or want to upgrade the version of a dependency library, and want to resolve it by themselves.
I think people wont't use the assembly jar in a pom.xml. So I think we can publish the assembly jar under the same artifact.

Oh, I didn't realize you wanted to publish the assembly JAR. I don't think it makes sense to publish assemblies as Maven artifacts. Right? anyone that uses it via Maven does not want an assembly and it causes a bunch of problems. So no please don't publish the assembly that way.
You just need a target to build the assembly right? that's just a matter of adding a plugin.
(You can publish multiple artifacts under different classifiers for one group/artifact but this isn't the situation that this would be used.)

The idea for publishing the assembly (which BTW is not that big as it
should contain only the sink code + scala + common lang 3) is same as that
of current publishing of package JAR - so that people can directly download
the JAR and put it in the Flume to run the sink. So the only difference
compared to what is the current behavior is that instead of publishing just
sink code, we will publish sink code + scala + common lang 3, which makes
it more self contained.
On Fri, Jun 19, 2015 at 12:52 AM, Sean Owen<EMAIL_ADDRESS>wrote:

Oh, I didn't realize you wanted to publish the assembly JAR. I don't think
it makes sense to publish assemblies as Maven artifacts. Right? anyone that
uses it via Maven does not want an assembly and it causes a bunch of
problems. So no please don't publish the assembly that way.
You just need a target to build the assembly right? that's just a matter
of adding a plugin.
(You can publish multiple artifacts under different classifiers for one
group/artifact but this isn't the situation that this would be used.)
—
Reply to this email directly or view it on GitHub
https://github.com/apache/spark/pull/6829#issuecomment-113417003.

Yeah I'm not worried about size. Maven isn't really the right place to distribute assemblies as it's not something to depend on. Yes, it just should be downloadable. I get it, that maven artifacts are still a pretty easy way to make it available. In that case maybe a new module makes sense after all: flume-assembly. Or if you're saying nobody would ever use the existing no-assembly artifact anyway, I can see changing it to the assembly. But that is suggesting that the existing module would never be used as a dependency.

So the only reason for an assembly would be to add commons-lang3. I am more in favor of removing that dependency than making the build more complex. Flume already has scala in classpath (since that is pulled in by the Kafka dependency). I am inclined to keep this component as simple as possible and depend only on stuff already pulled in by Flume into its own classpath anyway.

How extensive is our use of commons Lang 3 in flume sink?  If we only use one class or method maybe we can just copy the source into our repository, depending on how complex or large it is.

This is the only usage:
private val seqBase = RandomStringUtils.randomAlphanumeric(8)
I am inclined to just copy the method into the class.

I will open a PR later today for this one.

Yep, lets just remove the dependency on common lang. However, what is the
version of Scala that comes with Kafka? Scala 2.10, or 2.11? This is
something we need to document. Since spark artifacts are now published for
both 2.10 and 2.11, this needs a bit of documentation.
On a related note, should we bump the flume support to latest Flume? It
seems to be stuck in 1.4.0 where as 1.6.0 is out. Does it matter?
On Fri, Jun 19, 2015 at 10:04 AM, Hari Shreedharan<EMAIL_ADDRESS>
wrote:

I will open a PR later today for this one.
—
Reply to this email directly or view it on GitHub
https://github.com/apache/spark/pull/6829#issuecomment-113576860.

Kafka brings in 2.10.

I dont see a dependency on Kafka in Flume 1.4.0
http://mvnrepository.com/artifact/org.apache.flume/flume-ng-sdk/1.4.0
What am i missing?
On Fri, Jun 19, 2015 at 2:21 PM, Hari Shreedharan<EMAIL_ADDRESS>wrote:

Kafka brings in 2.10.
—
Reply to this email directly or view it on GitHub
https://github.com/apache/spark/pull/6829#issuecomment-113648154.

That looks like just the API module. I suspect it comes via the actual implementation such as in http://mvnrepository.com/artifact/org.apache.flume/flume-ng-sources/1.6.0 but I don't know Flume well.

I see. So the Kafka is present only through the flume-kafka-source
http://mvnrepository.com/artifact/org.apache.flume.flume-ng-sources/flume-kafka-source/1.6.0
Furthermore this is not available for Flume 1.4.0 as kafka source was added
only in 1.6.0
So here are two questions

Do installations of Flume always have all the sources loaded? If not,
then its an incorrect assumption that Scala will always be present.
Even if 1 is true, we have to upgrade Flume in Spark Streaming to
version 1.6.0 for this to be feasible. That;s a whole different issue.

I dont know enough about Flume, but I will be very surprised if the kafka
source is always loaded in the classpath in all flume installations.
@harishreedharan please comment.
On Fri, Jun 19, 2015 at 2:50 PM, Sean Owen<EMAIL_ADDRESS>wrote:

That looks like just the API module. I suspect it comes via the actual
implementation such as in
http://mvnrepository.com/artifact/org.apache.flume/flume-ng-sources/1.6.0
but I don't know Flume well.
—
Reply to this email directly or view it on GitHub
https://github.com/apache/spark/pull/6829#issuecomment-113653289.

Yes, all of the libs in the flume-ng/lib directory gets added to the classpath, so scala would get added to the classpath, but get loaded only as required (which is normal JVM protocol). We'd have to bump our dependency to 1.6.0 for scala to be automagically available.
Even if we don't upgrade, we don't need to change the dependency set, as the behavior is the same as before (add scala to flume-ng/lib or plugins dir). Apart from the assembly part, nothing else changes. I am sending a PR soon to get rid of the commons-lang3 dependency anyway

I agree with @srowen that Maven isn't really the right place to distribute assemblies. For the assembly jars, what we need to do is just providing download links for people.
I find now these jars are already assembled in spark-examples-....jar. How bout adding separate assembly jars to the final distribution "tgz" file?

Now that we don't have the commons-lang3 dependency in the flume-sink anymore, the assembly question for this module is moot. But if we want to have a more general discussion, we should perhaps move this discussion to a jira or the dev list?


Maintenance Release: Forces And Motion Basics 2.1.11-rc.1 phet-io branch

@samreid @kathy-phet @DianaTavares @jessegreenberg(responsible dev) FAMB is ready for a maintenance release.
This publication is to support a PhET-iO interaction study, see https://github.com/phetsims/forces-and-motion-basics/issues/259 for details.
In the only code we added was on the wrapper side, with the exception of a new query parameter phet-io.emitStatesInterval.
The testing required for this release is to make sure that the features added are working as expected, as well as testing the full process that will be used in the upcoming study (i.e. recording to metacog).
This is a bit of a weird test, please bare with me, as I try to explain the important features to test.
First
please test the simulation as you would normally for a maintenance release. Make sure that on one or two devices, all the links works as expected, and play around with it for a few seconds.
Link to wrapper index
Notes:

You will need to add the ?launchLocalVersion query parameter  (or &launchLocalVersion if there is already other query parameters) to each link in the index wrapper. I'm sorry.
As always it is good to note that this sim was originally published in Oct 2016. As a result, when finding bugs, see if the bug is present in the previous version and/or in master before creating an issue.

Second
Below is the link to the study link that records to the console, not to metacog:
https://phet-dev.colorado.edu/html/forces-and-motion-basics/2.1.11-rc.1/wrappers/iframe/iframe.html?url=../login/login.html?screens=2.3&wrapper=record&validationRule=validateDigits&encryption=encryptNumberV1&numberOfDigits=7&promptText=This is a test, enter 1000000 as your id&publisher_id=5d4c8ae1&key_name=phet-study_2018_q4&widget_id=forces-and-motion-basics-qatest&phet-io.emitInputEvents=true&phet-io.emitStates=true&phet-io.emitStatesInterval=60&launchLocalVersion&console

[ ] Please test this link fully by entering in the prompted id, "1000000" and testing the redirected wrapper
[ ] Please also test that the phet-io.emitStatesInterval query parameters changes the number of "state" events in the console, where the number is the number of frames between states emitted. (~60 a second)

Third
Next test for metacog. In general this process involves running the link and playing with the sim for about 30 seconds, doing that a few times, then requesting that data from metacog, and then making sure that those sessions work as expected in the playback wrapper. Please follow the below steps as thoroughly as possible, unfortunately it is still quite complicated. This is the first time that QA has tested recording to metacog, and I look forward to feedback about the process. Many of the documentation links below point to https://github.com/phetsims/phet-io/blob/master/doc/metacog.md. This document will likely need updating and tweaking; please make recommendations!!

[ ] Please read this section of the metacog docs about tips while recording to metacog to assure that you understand a few key points about recording with metacog.

[ ] Run https://phet-dev.colorado.edu/html/forces-and-motion-basics/2.1.11-rc.1/wrappers/iframe/iframe.html?url=../login/login.html?screens=2.3&wrapper=record&validationRule=validateDigits&encryption=encryptNumberV1&numberOfDigits=7&promptText=This is a test&publisher_id=5d4c8ae1&key_name=phet-study_2018_q4&widget_id=forces-and-motion-basics-qatest&phet-io.emitInputEvents=true&phet-io.emitStates=true&phet-io.emitStatesInterval=60&launchLocalVersion&metacog and make sure that you are recording to metacog. You can use any 7 digit number (not starting with 0) as your id. Please do this on a variety of devices, with a few sessions per device.  Keep track (ish) of the sessions that you record, i.e. device/login_id/amount of time recording/discrete actions done in the recording.  These notes will help you to make sure that the playback worked as expected later on.

[ ] Once you record, you may have to wait a bit (sorry that I don't have an exact timeframe here) perhaps about 30 minutes just to be safe. This will make sure that these logs have been fully parsed by metacog's processes.

[ ] Next follow this link about retrieving the logs from metacog. Notes:

For step 2, use ?screens=2.3&wrapper=record&validationRule=validateDigits&encryption=encryptNumberV1&numberOfDigits=7&promptText=This%20is%20a%20test&publisher_id=5d4c8ae1&key_name=phet-study_2018_q4&widget_id=forces-and-motion-basics-qatest&phet-io.emitInputEvents=true&phet-io.emitStates=true&phet-io.emitStatesInterval=60&launchLocalVersion&metacog
ignore step 3 and 4
please private message me for the application id in step 5, as we don't have a QA account for metacog (yet).

[ ] Once you have the logs, see this section about how to play back the logs

Note: here is the playback wrapper that will work for this version FAMB: https://phet-dev.colorado.edu/mjkauzmann/FAMB-2.1-phetio-playback/metacog/playback/ . Don't try to use master.

@ariel-phet has given me permission to mark this as top priority. Thank you all for you assistance!

@zepumph The developer's guide link on the wrapper index is broken.

General phet-io rc test done on these devices
-[ ] Mac OS 10.13 all browsers
-[ ] Win 10 all browsers

QA is done, and https://github.com/phetsims/forces-and-motion-basics/issues/267 published to production. Closing. Thank you so much to @KatieWoe and @lmulhall-phet!!!

Good to note that the "generate/launch" features of instance proxies don't work in RC for this version (likely because they were designed with hard coded "spot" specific links), but the production links work as expected.


Create from description command

Thanks for the nice extension! I've added a new command which I missed while using the extension - ability to create code via a prompt and have it inserted at the current cursor location. Please not that my version also added the concept of context and puts surrounding code to the prompt.

Thanks for the quick reply and review. I hope I didn't miss anything in the updates I just did

Thanks @maxim-saplin, merging this. It will go in the next release!


Fix join short circuit when probe is empty

Join short circuit on empty probe does not work correctly. Previously (before join was ported to work processors) join operator was blocking on build side (lookupSourceProviderFuture) before getting first page.
Currently, it is still the case due to dynamic filter support. However, waiting for dynamic filters should be correctly solved by: https://github.com/prestosql/presto/pull/3414. When https://github.com/prestosql/presto/pull/3414 lands, it would be possible to eliminate lookupSourceProviderFuture.isDone() from:
    @Override
    public boolean needsInput()
    {
        return lookupSourceProviderFuture.isDone() && pageBuffer.isEmpty() && !pageBuffer.isFinished();
    }

This would allow to simplify join (make it plain WorkProcessorOperator) and to correctly support short circuiting on empty probe side.

@sopel39 : can this issue be fixed now as you have explained - since #3414 has been merged to master?

I've a draft commits that simplify join. However, I'm not sure if short circuit can be implemented that easily. This is because build side might required to be closed gracefully because of spill support. This requires investigation


copyfiles fails for Windows builds

Describe the bug
During build on Windows, copyfiles fails
As described in #328 , when using Windows the build process is broken for td.site in version 1.x, with copyfiles returning an error and breaking the build process
Expected behaviour
We expect this sequence to work and build the web application : npm install then npm start
Environment

Version: greater than 1.4.0
Platform: Web App and Desktop App
OS: Windows  only
Browser: any of chrome, firefox, safari, etc

To Reproduce

Clone the repo
change to top level of the new directory
run command npm install
run command npm start
build breaks for td.site when invoking copyfiles

Any additional context, screenshots, etc
Console log attached
console.txt

Hello @taoyiyang1 many thanks for the email - I have created a bug report for it.
The reason for copying over the fonts directories is so that they get bundled with the web app, but can also be updated if bootstrap or font-awesome is updated. Neither of these are particularly strong reasons for doing it this way, and there are simpler ways that probably will not break for Windows
I have assigned this to you and me, hope that is OK, and I will upload the console log in a minute or two

I am working on this now @taoyiyang1 , so should have a solution soon  in the form of a pull request


[Feature] Login Using Yahoo Auth Provider

Since Firebase has now added feature to login via Yahoo Auth, it will be a great addition if the same is allowed from Angularfire2.
Tried the official documentation link https://firebase.google.com/docs/auth/web/yahoo-oauth but it does not work with Angularfire2. The gereric class firebase.auth.OAuthProvider is not found.

Can you please follow the issue template? It's extremely important in cases like this. We need to know what versions you are on and a sample of the code you are trying. This way we'll know if we need to upgrade to a new version of if it's a simple upgrade on your end.

Not sure if that's related to this specific issue, but I also have problems with the Auth Provider for Microsoft login (added recently along with Yahoo Auth)

Not sure if that's related to this specific issue, but I also have problems with the Auth Provider for Microsoft login (added recently along with Yahoo Auth)

Firebase 5.9.3
AngularFire 5.1.2

Same issue here even // @ts-ignore does not work on that line, I am not sure may be because the code is in switch case. But this is important and should be fixed asap.

Not sure if that's related to this specific issue, but I also have problems with the Auth Provider for Microsoft login (added recently along with Yahoo Auth)

Firebase 5.9.3
AngularFire 5.1.2

Same issue here even // @ts-ignore does not work on that line, I am not sure may be because the code is in switch case. But this is important and should be fixed asap.

@shakeelosmani I deleted node_modules folder, and then did "npm install" and it fixed this error. (make sure you have latest version of Firebase - 5.9.4). I do agree this is very strange.

@ron4548 Thanks for the tip. I think this issue should now really be closed. Updating Firebase and AngularFire really fixes the issue. On a side note for folks using Yahoo OAuth and if you guys need the email address returned from the OAuth sign in, must create your Yahoo App with profile (read-write-private) permission. Which is kind of weird how Yahoo has it set up. And from Angular you have to add the correct scope which would be like provider.addScope('sdpp-w')

Since Firebase has now added feature to login via Yahoo Auth, it will be a great addition if the same is allowed from Angularfire2.
Tried the official documentation link https://firebase.google.com/docs/auth/web/yahoo-oauth but it does not work with Angularfire2. The gereric class firebase.auth.OAuthProvider is not found.

@nikhilbhalwankar  they way I use it at the moment to make it work is:
import { auth } from firebase/app;
Then when trying to signin with OAuth providers simply call it this way:
new auth.OAuthProvider('yahoo.com')
Hope this helps!

Since Firebase has now added feature to login via Yahoo Auth, it will be a great addition if the same is allowed from Angularfire2.
Tried the official documentation link https://firebase.google.com/docs/auth/web/yahoo-oauth but it does not work with Angularfire2. The gereric class firebase.auth.OAuthProvider is not found.

@nikhilbhalwankar they way I use it at the moment to make it work is:
import { auth } from 'firebase/app';
Then when trying to signin with OAuth providers simply call it this way:
new auth.OAuthProvider('yahoo.com')
Hope this helps!

Thanks. Will try this out and get back.


Update the address for the L1 gateway

based on https://github.com/FuelLabs/fuel-v2-contracts/blob/5f49c4ca866970dd475e04dbe9486fcf260434c3/deployments/deployments.goerli.json#L4

Looks good! Although a user would need to change several other items in that file too for a correct deployment.

Note that I'm updating the contract to work with the latest force as well (required to use the stdlib functions and remove the local implementations), and also bumping the SDK from 0..29 => 0.35.


FancyZone Editor toolbar

Shift in how to handle the toolbar

@niels9001 you think that would be helpful verbiage?

@crutkas @enricogior I think the usability issues in the editor keeps coming back across various issues (especially since it has some non-Windows-standard ways of doing things (e.g. press and hold to select).
Would it make sense to explore a different way of enabling these functions? Here's a toolbar idea what would allow you to switch between different "tools" (e.g. horizontal split tool, vertical split tool, merge tool):

Originally posted by @niels9001 in https://github.com/microsoft/PowerToys/issues/10230#issuecomment-800962201

Would it make sense to explore a different way of enabling these functions? Here's a toolbar idea what would allow you to switch between different "tools"

That's what I call "thinking outside the box" 😜


.NET Core 2.0 and Angular 5

@geoperez @Israel-94 you guys were on the ball today with helping me get our code entries in sync. Muchas gracias.
(ponders) Only holdback now appears to be the matching validator. As you can see from today's commits, I junked the previous AOT/Webpack4 attempt; since Israel's attempt to clean things up made that look rather dumpy. http://www.codiodes.com/match-password-validation/ came up on Google: might be useful.

@unquietwiki do you have your editor to start a code block curly brace on the same line?
if (true) { // do something }
instead of
if (true)  { // do something }
I found some files are the same, only with this format change. We use the curly brace in a new line in all project, like most of the C# open source projects.

Yeah the code format was based on autofornatting from OmniSharp via a VS
Code plugin. And I added the makefile as a quick way to clean up the node
dependencies; it's not used in the actual build process.
On Thu, Apr 26, 2018, 7:22 AM Geovanni Perez<EMAIL_ADDRESS>wrote:

@unquietwiki https://github.com/unquietwiki did you add the makefile?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/unosquare/passcore/pull/139#issuecomment-384659156,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA9fv0d7r_U8dFNsuZR8zwQX513cB6Hfks5tsdgxgaJpZM4TkgDe
.

Can you match the code format like Visual Studio?

I'd have to double check to make sure I don't have some odd OmniSharp.json
sitting somewhere I'm not seeing. AFAIK both use OmniSharp Rosyln for C#
support.
On Thu, Apr 26, 2018, 7:59 AM Geovanni Perez<EMAIL_ADDRESS>wrote:

Can you match the code format like Visual Studio?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/unosquare/passcore/pull/139#issuecomment-384672319,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA9fv7UOOLlqhA3rxx5G6rGakLeZEDboks5tseDRgaJpZM4TkgDe
.

@unquietwiki I updated the passwordMatch validator to use with reactive forms, now the api/password is not returning the labels and the information for the client application

@Israel-94 I'll take a look. Thanks for fixing the validator.
@geoperez I found some resources regarding how to show you all what extensions I've been using; as well as trying to figure out how to enforce your desired style with them.

@Israel-94 I went over the docs best I can, as well as a migration tool I had checked before. It looks like the "this" errors go away if you separate stuff from the old constructor into OnInit & OnChange; however, you & @geoperez are going to know better as to what does what. Please check app.change-password.ts when you guys are able. Thanks!

@Israel-94 @geoperez Sorry for the delay today: had to reload my work machine. I fixed the favicon bundling, and tweaked the checking on the username validation. Testing for an email address defeats the purpose of having "defaultDomain"; there is probably a way to have it to a check for that, if the defaultDomain is null / empty; but is it required for our needs?
I'm actually having trouble verifying that my tweaks actually work: sometimes password resets don't work right here, and it's something that's mystified me and my boss (probably some weird GPO issue). I'll let you know if it end up does working for someone, else please let me know if we're all good on this.  👍

But did you remove the email validator in all cases?

Yeah. I set it to text, not email.
On Mon, Apr 30, 2018, 5:49 AM Geovanni Perez<EMAIL_ADDRESS>wrote:

But did you remove the email validator in all cases?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/unosquare/passcore/pull/139#issuecomment-385388847,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA9fv4UYj-nVO6QjsuOnekf4CI9am-kRks5ttwhegaJpZM4TkgDe
.

@unquietwiki Can you explain a little more why removed the email validator

If using a defaultDomain, then you only need to specify the username for a
reset. Email validator explicitly requires user@domain format. My users
were getting confused about using their company email address, hence the
modifications.
On Mon, Apr 30, 2018, 7:30 AM Israel Ramos<EMAIL_ADDRESS>wrote:

@unquietwiki https://github.com/unquietwiki Can you explain a little
more why removed the email validator
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/unosquare/passcore/pull/139#issuecomment-385415424,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA9fvy7cFjMcK3tV3Dcss7dvJTEny207ks5ttyAEgaJpZM4TkgDe
.

Yeah, but we need to support both cases. Adjusting the input with the server settings will be required.

I figure a custom validator on the backend. Frontend will still be limited
to string or email.
On Apr 30, 2018 7:39 AM, "Geovanni Perez"<EMAIL_ADDRESS>wrote:
Yeah, but we need to support both cases. Adjusting the input with the
server settings will be required.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/unosquare/passcore/pull/139#issuecomment-385418570,
or mute
the thread
https://github.com/notifications/unsubscribe-auth/AA9fv4ZZj3Y-QXej_xB9F7fEy0aErn_nks5ttyIngaJpZM4TkgDe
.

I don't know if there is need to validate an email in backend. @Israel-94 can you check how to handle this?

ASP.NET Core already has the DI integration. It's a cool new feature in WebForms, but we are not using .NET Framework in this branch and neither WebForms.

@geoperez @unquietwiki  I am going to handle the defaultDomain as a custom configuration on the  appsettings , so that the client will have two scenarios one when defaultDomain is configured it and the default

@Israel-94 I was able to successfully compile your latest commit, and have it live on my end. I'll advise if there are any problems; otherwise, from looking at the changes, you appear to have threaded the needle. 🥇

@unquietwiki I ran the application with the new changes and It is working for me, maybe it has something to do with the cache of the old files in the browser, try disable the cache in the browser and reload the page.

@Israel-94 I had tried in a different browser when I ran into the issue, but it's possible something in the cache is conflicting with the newer page. I saw @geoperez just pushed some WinAPI changes: I'll hold off on another test until he's done.

We are done, you can try it?

@Israel-94 can you take a look?

@unquietwiki I made what I think are the last changes, Can you test to know if it works and then do the merge?

I can pull the changes in a bit. Not at the office today, but I can remote
in & get that setup for use. Thanks!
On Tue, May 8, 2018, 10:15 AM Israel Ramos<EMAIL_ADDRESS>wrote:

@unquietwiki https://github.com/unquietwiki I made what I think are the
last changes, Can you test to know if it works and then do the merge?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/unosquare/passcore/pull/139#issuecomment-387475668,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA9fv_KeKokTGsSTD6XHra_heWzTEQAEks5twdKjgaJpZM4TkgDe
.

@Israel-94 @geoperez I have the new build live, and reached out to some folks. The DirectoryServices NuGet package received an RC1 update, so I told it to use that. I'll report back what the results are. Thanks! 👍

@Israel-94 @geoperez Still getting 400 errors on reset attempts. Even made sure I had the latest appsettings.json file change. 🤕
Looking back at all of this, I don't think this has properly worked for the actual reset part since commit ab0fcd9 . Is there something I can do to further get better debugging on this? Anything about your test environment / AD setups that would indicate why it'd work for you; and not for my folks?

@unquietwiki We have the latest changes and everything its fine on our side, Could you check what errors do you have or copy/paste the 400 errors to see whats happening?

@geoperez @Israel-94 I've spun up the latest build, and will report back. Report from one of the other IT guys suggested that he thought folks weren't abiding to our password complexity requirement; I'm not sure from the new logic you guys added as to how that gets expressed.

@Israel-94 I saw you just made some changes, and they look important. Please advise when you want me to rebuild. Thanks!

@geoperez @Israel-94 Yeah, I can see why it's annoying. In looking at the few remaining errors, I found that some of the defaultDomain stuff could use tightening. I'm testing some changes on my end momentarily.

@geoperez @Israel-94 Even with the latest attempt at fixes, and more clear error messages; still getting "invalid username or password" error. :(

Are you testing with default domain? Our side is not using it.
On Wed, May 9, 2018, 5:19 PM Michael Adams<EMAIL_ADDRESS>wrote:

@geoperez https://github.com/geoperez @Israel-94
https://github.com/Israel-94 Even with the latest attempt at fixes, and
more clear error messages; still getting "invalid username or password"
error. :(
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/unosquare/passcore/pull/139#issuecomment-387892645,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABsYsOFsF0SFffEfGmGpxI9TN-gKCZW-ks5tw2tbgaJpZM4TkgDe
.

I am, yes
On Wed, May 9, 2018, 3:43 PM Geovanni Perez<EMAIL_ADDRESS>wrote:

Are you testing with default domain? Our side is not using it.
On Wed, May 9, 2018, 5:19 PM Michael Adams<EMAIL_ADDRESS>wrote:

@geoperez https://github.com/geoperez @Israel-94
https://github.com/Israel-94 Even with the latest attempt at fixes,
and
more clear error messages; still getting "invalid username or password"
error. :(
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/unosquare/passcore/pull/139#issuecomment-387892645,
or mute the thread
<
https://github.com/notifications/unsubscribe-auth/ABsYsOFsF0SFffEfGmGpxI9TN-gKCZW-ks5tw2tbgaJpZM4TkgDe
.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/unosquare/passcore/pull/139#issuecomment-387897174,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA9fv60GbmFdeKHpjyzrVIKywzuNY8yNks5tw3EGgaJpZM4TkgDe
.

@unquietwiki We made some changes and test with default domain, everything works fine, Can you pull the latest and tell us your results? after this, we plan to do the merge.

Let's merge!!!!!!!!!!

@geoperez Should we delete the old branch now?


chore: Add templated Values for the CRDs

Fixes #N/A
Description

Added webhook values to allow users to configure webhooks inside of the conversion block

How was this change tested?

Manually tested

Does this change impact docs?

[ ] Yes, PR includes docs updates
[ ] Yes, issue opened: #
[x] No

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>3
Details

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

Totals

Change from base Build<PHONE_NUMBER>6:
0.0%

Covered Lines:
5972

Relevant Lines:
7534

💛  - Coveralls


HTML to draft js

Once the html has been exported, is there currently the ability to switch back into the ContentState

Oops found my answer https://github.com/sstur/draft-js-import-html


switch between languages for bilingual business

I need to be able to change the language by clicking on a button like many other sites because our business has french employes.
A button by the ? icon on the upper right that redirect to the french page is what I would like to do.
For now, I have hosted 2 different pages and for the french page, I have changed the text language in the appsetting file, but it is not a great method.
I have no idea how to modify source code, but if you have a link that shows how to, I'd like to learn!

Hi @dsavaria12 ,
Passcore doesn't support internationalization and we don't have any plans to add it. However, you can download the free trial of Passcorepro which already supports multiple languages.
Please feel free to give it a try. You can find it at: https://store.unosquare.com/PasscorePro. Let me know if you have any other doubts

Hello @dsavaria12 , did you get a chance to test passcore pro?

Yeah, I tried passcore pro but it is not exacly what we were looking for.
Passcore was easier to work with.
De : Geovanni Perez<EMAIL_ADDRESS>Envoyé : 9 juillet 2019 12:07
À : unosquare/passcore<EMAIL_ADDRESS>Cc : Savaria, Dominic<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Objet : Re: [unosquare/passcore] switch between languages for bilingual business (#284)
This email originated from outside of the organization. Do not click links or open attachments unless you recognize the sender and know the content is safe.
Ce courrier électronique provient de l'extérieur de l'organisation. Ne cliquez pas sur les liens ni n'ouvrez aucune pièce jointe, sauf si vous reconnaissez l'expéditeur et savez que le contenu est sécurisé.
Hello @dsavaria12https://github.com/dsavaria12 , did you get a chance to test passcore pro?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/unosquare/passcore/issues/284?email_source=notifications&email_token=AMQRGSJFPEK5R6SZAK6OBRTP6SZQTA5CNFSM4H5GH6UKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODZQX2EI#issuecomment-509705489, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AMQRGSKE4M64UEZJE4KUR3TP6SZQTANCNFSM4H5GH6UA.

Can I reach you by email to get some insights?

Sure what do you want to know?
De : Geovanni Perez<EMAIL_ADDRESS>Envoyé : 11 juillet 2019 09:51
À : unosquare/passcore<EMAIL_ADDRESS>Cc : Savaria, Dominic<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Objet : Re: [unosquare/passcore] switch between languages for bilingual business (#284)
This email originated from outside of the organization. Do not click links or open attachments unless you recognize the sender and know the content is safe.
Ce courrier électronique provient de l'extérieur de l'organisation. Ne cliquez pas sur les liens ni n'ouvrez aucune pièce jointe, sauf si vous reconnaissez l'expéditeur et savez que le contenu est sécurisé.
Can I reach you by email to get some insights?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/unosquare/passcore/issues/284?email_source=notifications&email_token=AMQRGSMUCFKMSRLGZKJ6MU3P643DTA5CNFSM4H5GH6UKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODZWYL7I#issuecomment-510494205, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AMQRGSMFCGK7PDC4Y2OMVCLP643DTANCNFSM4H5GH6UA.


Tweak: Allows putting the seclite in a security belt

Fixes #10995
Changelog
:cl:

tweak: The security belt can now hold seclites.

FlashlightSeclite already has the tag Flashlight, which it inherits from FlashlightLantern. Adding Flashlight to the whitelist of ClothingBeltSecurity makes more sense to me, unless you really want the security belt to only be able to hold the seclite and no other flashlights.

FlashlightSeclite already has the tag Flashlight, which it inherits from FlashlightLantern. Adding Flashlight to the whitelist of ClothingBeltSecurity makes more sense to me, unless you really want the security belt to only be able to hold the seclite and no other flashlights.

Being able to only hold the seclite is what I initially wanted, being able to hold any flashlight wouldn't make much sense imo (different shapes) and it makes seclites slightly better than normal flashlights for sec officers


lack of yaviks support

Describe the bug
Lack of support for this board
Boardname
yaviks
Logs
debuglog.tar.gz

I just updated the script. Try it again.


Code block renders wrong with DataView

What happened?
Normal Code block renders wrong when DataView is enabled.
Here is the code:
```cpp
#include <iostream>
#include <unsupported/Eigen/CXX11/Tensor>

int main() {
    Eigen::Tensor<float, 3> t(3, 3, 3);
    t.setConstant(2);

    const Eigen::Tensor<int, 2>::Dimensions  d(0, 1);
    std::cout << t.sum(d) << std::endl;

    std::cout << t.sum(Eigen::array<int, 2>({0, 1}));

    return 0;
}
```

Here is what is looks like in Live Preview mode:

The line std::cout << t.sum(Eigen::array<int, 2>({0, 1})); is not shown correctly.
Here is the screenshot of my DataView settings:

How I find it: I turn off the plugins, then close and reopen obsidian to check.
DQL
No response
JS
No response
Dataview Version
0.5.67
Obsidian Version
1.6.7
OS
Windows

@shizidushu Just one last thing - could please you try this with disabling all other plugins except Dataview one time? This will help speed things up a bit. Just trying to isolate the issue here.

@manuraj17 After I disabled all the plugins (refer to the image uploaded before) except Dataview, I close and reopen Obsidian. The problem remains in Live Preview Mode (and no problem in Reading Mode).
The problem may related to the Dataview option Enable inline field highlighting in Live Preview. After I turn off this option, and restart Obsidian, the problem disappears.

@manuraj17 Here is the sample vault:
testvault.zip

@shizidushu So I opened your vault and it doesn't appear to be an issue with Dataview, at least not by itself. I didn't try the various combinations of plugins you have installed but when I disable everything except for Dataview then I am not seeing any such rendering-related issue.

I previously said I couldn't replicate this, but that was wrong. I do see there is some rendering related issue. It's not on every line but it is on some lines such as: std::cout << t.sum(Eigen::array<int, 2>({0, 1}));, specifically with the Eigen::array portion on that line.


Finish implementing BMI abstract functions

Implement remaining BMI abstract functions, can simply be failures for now.
Implement test functions for remaining BMI functions implemented.

Making some progress on this.  Have all abstract methods implemented, and starting testing some of the basics.  Will push some commits likely tomorrow.


Pulse water texture scrolling glitch

The texture scrolling on the water texture on the map Pulse in the big pool of water with broken stairs (viewpos 3700 -3400 -1300) is broken. It jumps discontinuously to a different position about once every 10 seconds.
This doesn't happen on Tremulous (caveat: I tested a different version of the map.) I would expect that it does not change discontinuously in general.

I can reproduce this with the lava in https://github.com/InterstellarOasis/map-tds_src.dpkdir, built from commit 6fb60429093f2a461dfb3c8c09f7a0b77ec01d07, viewpos -645 -208 16 -1 21. It happens about every 50 secons there.
This is the shader:
textures/tds/lava1
{
    qer_editorimage textures/tds/lava1.tga
    qer_trans .5
    q3map_lightimage textures/tds/lava1.tga
    q3map_globaltexture

    surfaceparm noimpact
    surfaceparm lava
    surfaceparm nolightmap
    surfaceparm nobuildsurface

    q3map_surfacelight 500
    q3map_tessSize 64
    cull disable

    deformVertexes wave 200 sin 0 1 .5 .5

    {
	map textures/tds/lava1c.tga
	rgbGen const ( 0.6 0.6 0.6 )
	blendfunc add
	tcMod turb .3 .2 1 .05
	tcMod scroll .01 .02
    }

    {
	map textures/tds/lava1.tga
	rgbGen const ( 0.9 0.9 0.9  )
	blendfunc add
	tcMod turb .2 .1 1 .05
	tcMod scale .5 .5
	tcMod scroll .02 .01
    }
}

This is caused by a combination of different tcMods:

Removing scale and turbulence makes it work
Removing the x - floor( x ) and y - floor( y ) from scroll also makes it work

Removing scale and turbulence makes it work

So it would seem that this affects the majority of such shaders, as using both tcMod scale and tcMod scroll seems to be rather common.