error message related to Moon

Hello, I encountered the error messages with the following comment:
python -u predict.py --img aaa.jpg
/home/user/mxnet/lib/python3.6/distutils/init.py:4: DeprecationWarning: the imp module is deprecated in favour of importlib; see the module's documentation for alternative uses
import imp
INFO:root:Namespace(gpus=None, img='aaa.jpg', model_load_epoch=82, model_load_prefix='../model/lightened_moon/lightened_moon_fuse', opencv='../model/opencv/cascade.xml', pad=None, size=128)
Traceback (most recent call last):
File "predict.py", line 83, in
main()
File "predict.py", line 14, in main
_, arg_params, aux_params = mx.model.load_checkpoint(args.model_load_prefix, args.model_load_epoch)
File "/usr/local/lib/python3.6/dist-packages/mxnet/model.py", line 419, in load_checkpoint
symbol = sym.load('%s-symbol.json' % prefix)
File "/usr/local/lib/python3.6/dist-packages/mxnet/symbol/symbol.py", line 2535, in load
check_call(_LIB.MXSymbolCreateFromFile(c_str(fname), ctypes.byref(handle)))
File "/usr/local/lib/python3.6/dist-packages/mxnet/base.py", line 149, in check_call
raise MXNetError(py_str(_LIB.MXGetLastError()))
mxnet.base.MXNetError: Failed loading Op Moon of type MoonOutput: [18:33:52] src/core/op.cc:55: Check failed: op != nullptr Operator MoonOutput is not registered
Stack trace returned 10 entries:
[bt] (0) /usr/local/lib/python3.6/dist-packages/mxnet/libmxnet.so(+0x1c05f2) [0x7f0b787145f2]
[bt] (1) /usr/local/lib/python3.6/dist-packages/mxnet/libmxnet.so(+0x4842a55) [0x7f0b7cd96a55]
[bt] (2) /usr/local/lib/python3.6/dist-packages/mxnet/libmxnet.so(+0x4874311) [0x7f0b7cdc8311]
[bt] (3) /usr/local/lib/python3.6/dist-packages/mxnet/libmxnet.so(+0x4879420) [0x7f0b7cdcd420]
[bt] (4) /usr/local/lib/python3.6/dist-packages/mxnet/libmxnet.so(+0x4872ff6) [0x7f0b7cdc6ff6]
[bt] (5) /usr/local/lib/python3.6/dist-packages/mxnet/libmxnet.so(+0x295ebdf) [0x7f0b7aeb2bdf]
[bt] (6) /usr/local/lib/python3.6/dist-packages/mxnet/libmxnet.so(+0x4845cdd) [0x7f0b7cd99cdd]
[bt] (7) /usr/local/lib/python3.6/dist-packages/mxnet/libmxnet.so(+0x295fc71) [0x7f0b7aeb3c71]
[bt] (8) /usr/local/lib/python3.6/dist-packages/mxnet/libmxnet.so(+0x295ebdf) [0x7f0b7aeb2bdf]
[bt] (9) /usr/local/lib/python3.6/dist-packages/mxnet/libmxnet.so(+0x4845cdd) [0x7f0b7cd99cdd]
Environment: python 3.6.5 + mxnet version 1.2.0
This error seems to be related to issue #50 as well. Please help!

You should copy the files in src/operator into MXNet and recompile MXNet again.

@iamweiweishi I'm having the same issue as @pinyuchen. You mention, "copy the files in src/operator into MXNet...", when you say "...into MXNet...", do you mean copy the files in mxnet-face/src/operator into incubator-mxnet/src/operator?

I clone the ' https://github.com/dmlc/mxnet'. I did not try other versions of MXNet. But they might be the same.

@pinyuchen were you able to solve this issue? I was able to solve the same issue with @iamweiweishi comment, "You should copy the files in src/operator into MXNet and recompile MXNet again." I can provide details of what i did exactly to solve this issue, if you are still having this issue.

Yes it worked! Thanks for the fix.

@hozachief Please Can you guide for copying and recompiling the mxnet so i can also solve the issue of moon operator.

I'm not 100% sure since this was awhile back, but if i remember correctly this is what you need to do. You need to copy one of these or all three files (i think all three), "moon_output.cc", "moon_output.cu", "moon_output-inl.h" into ~/incubator-mxnet/src/operator
Also i'll attach my Makefile's for reference, hopefully they're helpful to you.
MakefileLocal.txt

I have copied all three files to incubator/src/operator, and then running pip install -e but getting ctype error @hozachief

Hmm i don't think i used "pip install". If i remember, i installed incubator-mxnet and i used one or both of my Makefile's to install mxnet-face. I think i copied those three files that i mentioned and i might have ran the Makefile for incubator-mxnet again. It wasn't a clear cut process, it was very messy and time consuming so i don't remember every step exactly.
For sure you probably need to get incubator-mxnet successfully installed before messing with mxnet-face. I believe you run the Makefile in the incubator-mxnet directory, yes.

Okay Thanks @hozachief  I will try to first install the incubator-mxnet and then modify it and rebuild it.

Hey @xyzdcgan I found my documentation for how i successfully installed all this stuff.
So to successfully compile incubator-mxnet:
// Command from MKLDNN_README.md
$ make -j $(nproc) USE_OPENCV=1 USE_MKLDNN=1 USE_BLAS=openblas USE_INTEL_PATH=/opt/intel
Apparently this is something i did when i was working on this. Try it!

Okay, i solved my issue by various steps, thanks @hozachief

when I install mxnet by pip install mxnet then where should I paste the files of src/operator.

Thanks @hozachief your answer solve my problem as well!

when I install mxnet by pip install mxnet then where should I paste the files of src/operator.

@AkshatRepo Instead of using pip, You should follow steps below and compile mxnet from source:

git clone https://github.com/apache/incubator-mxnet;
moon_output.cc, moon_output.cu, and moon_output-inl.h from mxnet-face/src/operator to incubator-mxnet/src/operator;
Follow instructions here to build mxnet from source and bind it with Python.

And then the problem will be solve :D
(at least it works for me).

Thanks @hozachief @Eroica-cpp your answer did work for me

Thanks @hozachief @Eroica-cpp your answers did work for me

@Eroica-cpp  Hey 3rd point you mentioned is not available now, Can you help me i'm having save issue


Fixed an issue with prefixer going into an infinite recursion

It's something that has been reported here: https://github.com/emotion-js/emotion/issues/2117
Gonna think about a potential fix for this later.

Pull Request Test Coverage Report for Build fcf38c7634108eb7293a123e92c057b09a22c3cf-PR-244

4 of 4 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build 9799734dcf2b04c1216818f8b74cba92b6de4f76:
0.0%

Covered Lines:
238

Relevant Lines:
238

💛  - Coveralls

LGTM

Given that the final change is really proposed by you - I'm merging this right away.

Published v4.0.4 with these last 2 PR's.
Thanks


add socket support (alfred)

Hello,
recently the alfred daemon has been patched to support the configuration for the unix socket path: Link
After this the alfred client alfred-json has also been patched to support this: Link
This allows communities which have currently multiple batman_adv mesh clouds on the same gateway to run one alfred daemon instance for each of the cloud. One implication is that all alfred clients have to patched in order to connect to the correct socket.
So please add support in ffmap-backend for this. Finally, bat2nodes.py should be called with a new command-line-option like -u | --unix-path (alfred daemon style) or -s | --socket (alfred client alfred-json style) in order to return the wanted unix socket to alfred.py and batman.py to make in the mkmap.sh magically stuff like:
./bat2nodes.py -A -s /var/run/alfred0.sock -a somealiases.json -m bat0 -d $DEST
./bat2nodes.py -A -s /var/run/alfred1.sock -a somealiases.json -m bat1 -d $DEST
./bat2nodes.py -A -s /var/run/alfred2.sock -a somealiases.json -m bat2 -d $DEST

And some time later someone should make some brainwork to consider the implications supporting multiple batman_adv mesh clouds in one single map, like e.g. duplicate gateways, vpn links?, etc.

This is now implemented.

Thanks for the implementation. But as I mentioned above the alfred socket has also be passed to batadv-vis "-u".


[security] audit repository tooling

The Security SIG is looking to ensure that security tooling is setup consistently across the organization. As a result, we're asking maintainers to ensure the following tools are enabled in each repository:

[ ] CodeQL enabled via GitHub Actions
[ ] Static code analysis tool (the collector uses govulncheck [https://pkg.go.dev/golang.org/x/vuln] on every build)
[ ]  Repository security settings

[x] Security Policy ✅
[x] Security advisories ✅
[ ] Private vulnerability reporting ✅
[ ] Dependabot alerts ✅
[ ] Code scanning alerts ✅

Parent issue: https://github.com/open-telemetry/sig-security/issues/12

Hello @dmitryax @TylerHelmuth
Please confirm if the items on the check list above are enabled/configured for the repo as I do not have access to see some of them.
Thank you.
cc @codeboten

@EjiroLaurelD @codeboten we have no code in this repo.  Do the first 2 tasks apply to this repository?

Private vulnerability reporting is enabled.
Dependabot alerts are disabled and I do not have permission to update them. I believe we'd need the TC to make changes.
Code scanning alerts are disabled.  As mentioned before, this repo does not have code to scan.

@EjiroLaurelD @codeboten we have no code in this repo. Do the first 2 tasks apply to this repository?

The repo has Helm charts in it. As a suggestion, we can use Aqua Trivy as a static code analysis tool for the repo.

I am not familiar with that tool.  Is it testing that the helm chart is installable? If so, we are using chart-testing for that.
Is it testing whether or not the thing the helm chart installs has security issues? If it is, then would that produce duplicate security warnings since the applications we deploy are OpenTelemetry applications that would trigger security alerts in their own repos?

I am not very familiar with the tool myself but I have seen it used on a few Iac projects online.
This is what I found out about Aqua Trivy upon investigating its usecase for Helm Charts.
I found that Trivy can be used to test whether a Helm chart is installable, but it is not primarily a testing tool. It is a security scanning tool that is used mainly in development to scan the Helm Charts for security vulnerabilities, misconfigurations and secrets without executing the chart.
Yes it may produce duplicate security warnings for the OpenTelemetry applications, but this can be reduced if Trivy is configured to ignore certain types of warnings. For example, it can be configured to ignore warnings for known vulnerabilities that have already been patched in the OpenTelemetry applications.
Another way to reduce the number of duplicate security warnings is to integrate Trivy with the CI/CD pipeline. This will allow to run Trivy scans on Helm charts as part of the build process. The CI/CD pipeline can then be configured to only fail the build if Trivy finds new security vulnerabilities that have not already been patched in the OpenTelemetry applications.
I hope this helps


Rego schema support

OPA version 0.41.0 supports schema enforcement - https://www.openpolicyagent.org/docs/v0.41.0/schemas.
Please add schema for the object that passes to the Rego files and define the schema of the returned value.

@naortalmor1 - could you please create new discussion under Ideas tab


[Docs] Include Layer5 Release Documents and Meshery Release Lead Role in Build and Release Strategy

Notes for Reviewers
This PR fixes #12021
Signed commits

[x] Yes, I signed my commits.

Description
This PR addresses issue #12021 by adding detailed information about Layer5 release documents and Meshery Release Lead responsibilities to the Meshery Build and Release Strategy document. The changes include:

A new section detailing the types of Layer5 release documents used in Meshery's release process.
Comprehensive description of Meshery Release Lead responsibilities throughout the release cycle.
Information on Release Manager qualifications and selection process.

These additions will standardize the release process, clarify the role of the Release Lead, and provide clear guidelines for creating and managing release documentation.

🚀 Preview for commit d2269ccc8209421df980975f60bfa8ff70c5d19d at: https://66ecec502bb456ce4bcd2172--meshery-docs-preview.netlify.app

🚀 Preview for commit 839f948447ebbdb157732deda8b453d5a335a760 at: https://66efb1307b850640e9f1b7b4--meshery-docs-preview.netlify.app

@Tharanishwaran Thank you for your contribution!
Let's discuss this during the website call on Monday at 5:30 PM IST (7:00 AM CT).
Please add it as an agenda item to the meeting minutes.

@vishalvivekm Thank you for your message. I am so sorry, but I had some kind of problem and could not join that meeting, but now everything is okay. Please let me know whether there is something important that I should know.

Hi @vishalvivekm, could you please review this PR and let me know if any changes are needed for merging? Thanks!

Hi @Ashparshp and  @Aviral0702 , could you please review this PR and let me know if any changes are needed for merging?


Fix links in readme

Fixes some stale links.

@ketkarameya @lazaroclapp please take a look, small changes.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Offset larger then 60 minutes?

It would be good if you could specify a larger offset than 60 minutes, e.g. 300 minutes

it is possible, but the offset does start to create complications around the on/off times if too large.  Out of interest is there a particular use case you are trying to handle that needs such a large amount o random variation?

I need it e.g. to lower the shutters 140 minutes after solar noon.

Ive upped this to 180 minutes in 1.2.1.  Let me know if that works for you.

Yes, thats is ok for me.
Thank you


Special case HN Comments to PDF

I tried to generate pdf from HN comments.
ref: https://news.ycombinator.com/item?id=18267468
Looks like only question is converted to pdf. Not the comment thread. Not sure it is bug or the desired behavior.  But for HN reader, it's really wealth of information and definitely love to see it converted to pdf.
Your thoughts?

use a completely different Reader View algorithm

I wonder how Pocket does it.

Another idea is to hack into Readability's regular expressions:
https://github.com/mozilla/readability/blob/3be1aaa01c078c25b67ed8dfd1c9aa8f9963490b/Readability.js#L113-L127
So that we can provide custom CSS classes / IDs for things we want to include (positive) or exclude (negative). I ran a quick test on a HN thread and including comment in the positive regexp worked to include it, but the rendering is all messed up due to the <table> in <table> layout HN uses. But it seems like a promising approach.

May be we can somehow extend this capability to command line argument like --css. So we can pass --positiveClass or --positiveId or something like that. Because each article may be slightly different needs.

I've did just that in this PR where you can pass --positive and --negative CSS selectors, but apparently it's not enough to nudge Readability in the right direction for hacker news threads, or other articles for that matter.
On the same PR I am experimenting with a homegrown variety of Reader View functionality that is showing promising results.

Closing this for now. It seems unlikely that we will be able to process correctly markup like the one featured on Hacker News threads with Readability / percollate.


Automatic exhumation

Keter should be exhumated automatically to keter.codingteam.org.ru or something like that.

Alright, we'll serve the application from our main web host (codingteam.org.ru), and will build and push it via GitHub actions. Here's the plan:

Add a special user to ctor who'll only be owner of one directory (say, /opt/codingteam/keter)
Create an SSH key for that user
Allow this user to log-in remotely and perform the only task: rsync to that directory, as described here,
Set up nginx to serve the data from that directory via keter subdomain
Set up a GitHub Actions job that will build Keter and deploy it via SSH+rsync (that will require us to upload the SSH key to GitHub Actions, but that's okay: even if it is compromised, we'll only lose some static web content)

@Minoru, does the above sounds right?

And terminal requirement for myself is that the whole setup should be automated. I don't want to twiddle with file ownership, users or SSH keys manually.

Yep, that sounds right!

Add a special user to ctor who'll only be owner of one directory (say, /opt/codingteam/keter)

Note that you can achieve that by making /opt/codingteam/keter the home directory of that user. I don't think it adds any security, but it makes it slightly easier to figure out the relationship between directories and users: even if you bork the permissions and ownership, you can still grep /etc/passwd and figure stuff out.

This task is currently blocked on:

[ ] [REDACTED]2

[ ] [REDACTED]6

[ ] [REDACTED]8

[ ] [REDACTED]11

[ ] [REDACTED]6

[ ] [REDACTED]8

[ ] [REDACTED]8

[ ] [REDACTED]4

[ ] [REDACTED]6

[ ] [REDACTED]8

[ ] [REDACTED]11

[ ] [REDACTED]6

[ ] [REDACTED]8

[ ] [REDACTED]14

[ ] [REDACTED]6

[ ] [REDACTED]8


Fix failing tests in windows (reloaded)

Using a patched version of cabal-helper (until the next release)

It includes the remove of the private lib

Azure windows agent have changed their drives space now C: has lot of more space than D:, so i've switched again cache to C:
Some of unit failing tests had been fixed by #1654

mmm, i've found submodules more suited to mid or long term forks due to the additional effort to setup and maintain them, i hope this is a  provisional workaround


Update

Remove deprecated dependencies and replace with newer alternatives. Change rust edition and syntax to 2021 edition.

@acidnik I want to try to add parallelization to ppcp (I'm on a slow NFS, and copying a lot of files is VERY slow). I wanted to see if I can help with this PR.
I added a commit to it, and opened another PR so it will be easier to view the changes: https://github.com/acidnik/ppcp/pull/8
Regarding the removal of pb_done: now there's no need, and you can't, call .join() of indicatif. So I think there's no reason to have the extra mutex at all. In my PR I just removed it.
WDYT?

@noamraph I still don't see how parallelization would speed up your network, but if you implement it and prove that it is faster (by showing the before/after numbers) I will merge the PR


"Standard Enum" over operators

It might make sense to have a single enum over a very large set of common operators, and mask this set according to what a user passes.
This would allow pretty much the entire library (!!) to be compiled into a static binary, so long as the user doesn't define any custom operators (which is true for most users anyways). The Node type has op::Int, so there aren't really any space requirements on the numbers of operators passed.
Given the user's choice of operators, the common operator set would simply be masked with a boolean vector. For the mutation operations, only those operators which are unmasked could be sampled.
I think this would actually be pretty simple to implement. The tricky part would be verifying that the precompilation catches everything.

This is basically done now, since now the operator enums don't specialize to the operators. Changed with https://github.com/SymbolicML/DynamicExpressions.jl/commit/a758af9a51a39d2e5c4510b509dce20bf16d042b


⚠ ClamAV has degraded performance

In fa1c4c2, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 78 ms

Resolved: ClamAV performance has improved in fb52c94 after 541 days, 11 hours, 56 minutes.


Update svelte please.

The svelte version here is 1.6, please update to 1.9

As with svelte-loader, the semver range means that installing svelte-cli should install the latest svelte.

As of version 2.0.0, svelte is in peerDependencies rather than dependencies, which should prevent any version problems in future


Can't connect to Vagrant box

Hey, I'm unable to run vagrant ssh to connect to the box after running vagrant up.
I enabled the GUI and saw that ubuntu is prompting me to login, is this normal behavior? How would I be able to get around this?
Thanks for your help :)

@rafkhan, vagrant up followed by vagrant ssh should work without a hitch. A few questions:

What does vagrant status return?
What version of vagrant are you running, vagrant --version?

That info should help us get the issue resolved.

Closing due to inactivity. @rafkhan please reopen if I can do anything to help.


added a break before footer

just felt there could be a little space between the text and footer

Thanks!


Have to reboot the wallet to have it scan the address

It happens very often that I can't test the electrum server for some reason there's seems to be a connection issue that only restarting the app fixes.

Which electrum server? Roughly how many used addresses in the wallet?
Electrs has issues with large (many used addresses) wallets, as it effectively becomes unresponsive at 100% CPU. That said, the upcoming Sparrow release has many improvements specifically aimed at making usage on Electrs more robust.

Using electrs with not a lot of txs... brand new wallet for testing.
Would be great to have a "Refresh" button to force check of the blockchain/mempool when stuck.

It's now possible to trigger a full refresh of the wallet by going to Settings, clicking the Advanced... button at the bottom, and changing the birth date (any change will do).
Technically it shouldn't be necessary to do this, and I'm hoping that electrs will soon perform better which should obviate this need in future.

It's now possible to trigger a full refresh of the wallet by going to Settings, clicking the Advanced... button at the bottom, and changing the birth date (any change will do).
Technically it shouldn't be necessary to do this, and I'm hoping that electrs will soon perform better which should obviate this need in future.


Strange characters produced for keys configured in default key_bindings

I'm using Ubuntu 19.04, Wayland (Sway 1.2), and alacritty 0.3.3 installed as deb from the ppa.
Some default  configured key_bindings produce strange characters pasted, i.e. when I press Super+F5 these char sequence gets pasted in alacritty: ;3~. Same for Super+F6 up to Super+F12. I also do get other weird sequences with other key_bindings.
I guess something doesn't properly work with escaping ?

What's exactly weird for you here?
The default bindings for Super + F1-F12 are CSI 1;3P CSI 1;3Q CSI 1;3R CSI 1;3S CSI 15;3~ CSI 17;3~ CSI 18;3~ CSI 19;3~ CSI 20;3~ CSI 21;3~ CSI 23;3~ CSI 24;3~ respectively. The sequence that was pasted to your shell could be a bit different though (in terms of chars thing). This thing was pasted to fish shell on Super + F1 press: [1;3P.
You can try running cat without args and try pressing your bindings.
FWIW I'm running the same compositor as you are.

Yeah, the escapes for Super + F1-12 are going to change in #2785, however in general this is mostly intentional. We send these escapes to tell terminals about special characters and they're just not handled in some applications which leads to these escapes showing up.

@kchibisov This is what I get with cat and Super+F1 up to Super+F7:
❯ cat ^[[1;3P^[[1;3Q^[[1;3R^[[1;3S^[[15;3~^[[17;3~^[[18;3~
I'm just wondering about the purpose of such keybindings, but if they get removed anyway I don't need to understand...

Hugh, I do think this is a bug though.
Even when trying to setup a custom keybinding like this:
- { key: F8,            mods: Super,    command: {program: "/home/varac/bin/dark.sh"} }
I get a 3~ pasted in my terminal each time I press Super+F8 (dark.sh does gets executed fine).

@varac That has been fixed on the latest master.

@chrisduerr Unfortunatly not. I just compiled latest master and both issues (https://github.com/jwilm/alacritty/issues/2797#issuecomment-531156374 and https://github.com/jwilm/alacritty/issues/2797#issuecomment-531158372) still remain for me.

The first one is not a bug, this is intended.
Could you post your configuration? The second one might just be a configuration issue.

Ok, here are the steps to reproduce with lastest master:
wget https://raw.githubusercontent.com/jwilm/alacritty/master/alacritty.yml -O ~/.config/alacritty/alacritty.yml
echo '  - { key: F8,       mods: Super,   command: {program: "/usr/bin/gedit"} }' >> ~/.config/alacritty/alacritty.yml

And then press Super+F8. gedit starts, and ;3~ gets pasted on the command line.

You should remove the old binding before adding the new one.

@chrisduerr Ah, my fault - but is this intended behaviour that muliple definitions of the same key binding append each other ? I though they rather replace the preceding.
Could you please document those default keybings which are not self-explanatory ? Thanks !

The default keybindings are listed in the default configuration. What they do is presented there exactly. If you want to understand what these mean, you'll have to check out the terminal specifications listed in our wiki, but generally they just tell the shell which key has been pressed.
And I'm not sure what you'd expect when multiple keybindings are defined, we have to be able to make it possible to define multiple bindings for the same key. If you don't want any of the default bindings, there's documentation on how to get rid of them in the default config.
I don't think there's any need for action here, everything is working exactly as I would expect it to.

Actually I think I've missed an update to our default alacritty.yml when I made a batch of changes to the comments a while back.
https://github.com/jwilm/alacritty/blob/1067fa609b6a5a0017814cb96d8b21b39b2f83f2/alacritty.yml#L537-L539
The first sentence of this paragraph seems wrong now.

@nixpulvis What about that is incorrect? It seems right to me.


[terra-toggle-section-header] Add isOpen prop

Feature Request
Description
Add an isOpen prop to toggle-section-header so that it can be programmatically opened and closed.
Additional Context / Screenshots

@ Mentions
@mjhenkes recommended opening an issue.

Currently this control can be open and closed only during the click action. there is a prop isInitiallyOpen.  to make this open during initial rendering.  not sure whether this issue logged was taken into account during the initial design of this component tor not.  adding @neilpfeiffer  from UX to share his thoughts on this ?


Added outputs for generated iam_role resource.

This PR exposes the arn/name of the iam_role which is created by this module.

Please accept my apologies for the delay in review on this.
I like the idea of adding the outputs, but this PR looks like it effectively reverts #26  Can I suggest rather then removing the logic for using a custom IAM policy you add a conditional to the output so that it only fires if the module creates the policy? Basically, I'm concerned  with potentially loosing a feature of the module. It's also possible that I just misread the diff too...


Add a startup check to fail startup when using invalid configuration …

https://app.circleci.com/pipelines/github/jacek-lewandowski/cassandra/1213/workflows/2f85e188-f1e5-4db6-b13b-fee016d208c4
https://app.circleci.com/pipelines/github/jacek-lewandowski/cassandra/1213/workflows/e196c050-478c-42f9-a2bc-64e62e5844de

https://app.circleci.com/pipelines/github/jacek-lewandowski/cassandra/1233/workflows/bb617340-f1da-4550-9c87-5541469972c4
https://app.circleci.com/pipelines/github/jacek-lewandowski/cassandra/1233/workflows/5ae4f362-a864-4cc5-82ab-6148479979fc
Tickets for all flaky failures were created

https://github.com/apache/cassandra/commit/f0ea12c6d7683697d9a5ca0c99c2b7dc3bc11230


Any cause to hope for a 2019 version?

VS 2019 is out and is unfortunately not compatible with the 2017 version.
No pressure! I'm just crossing my fingers that you're planning to do another update.

@aheward NoMorePanicSave2017 is compatible with VS 2019. See #7
I'll make it more obvious on the VS extensions gallery site https://marketplace.visualstudio.com/items?itemName=PhilippDolder.NoMorePanicSave2017
Please report back if you find any issues

This is the issue I had - Is it simply not possible, or requires a major refactor?

@RobbSadler NoMorePanicSave2017 should work just fine in VS 2019.
Where you able to install it in VS 2019?

It's working fine for me in VS2019.


[Feature] Combine Web Cmdlet Partial Class Files

Reference #5610
This moves the partial class into a single file each. This code is purposely not sorted and the formatting is left alone to make this easier to review. There are cleanup actions planned in the referenced Issue.

@iSazonov let me know if this is easier to work with.

Yes, it is more easy. I spent one hour to review the code. Thanks.

Doc tests failed again on WMF links. I think we could run doc tests with separate test tag. And maybe run some tests on demand by test tag.

I restarted the mac tests, but they failed in webCmdLets.  If they fail again, can you investigate @markekraus

It looks like it has been too long since the last commit to restart the the CI. I will push an empty commit.

@TravisEz13 all passing now :)

😄 Our tests resemble a pinball machine in a casino.


Action Required: Do Not Merge WIP for GitHub Requires Changes to Comply with Chrome Web Store Policy

We wanted to let you know that your item is at risk of being removed from the Chrome Web Store. Please find the details below.
Item name: Do Not Merge WIP for GitHub
Item ID: nimelepbpejjlbmoobocpfnjhihnpked
Violation(s):
Use of Permissions:
Violation reference ID: Purple Potassium
Violation: Requesting but not using the following permission(s):
storage
How to rectify: Remove the above permission(s)
Relevant section of the program policy:
Request access to the narrowest permissions necessary to implement your Product's features or services. Don't attempt to "future proof" your Product by requesting a permission that might benefit services or features that have not yet been implemented. (learn more)
Important information about resubmission:
Please make the necessary changes within 14 days in order to avoid removal.

https://github.com/sanemat/do-not-merge-wip-for-github/blob/d01de90eb906d33b9cc2c83e87e0f3e1a8bc55c5/app/manifest.json#L10-L12

we use it :thinking:
https://github.com/sanemat/do-not-merge-wip-for-github/commit/83e26ca8acc022f07bf0486afce73ba85dc2e438

https://developer.chrome.com/docs/webstore/troubleshooting/#misunderstood-perms-storage

storage
The storage permission exposes the chrome.storage API to the extension.
When is it required?
When using the chrome.storage API.
When is it NOT required?
When using the Web Storage API (document.localStorage, document.sessionStorage) or IndexedDB.

https://developer.chrome.com/docs/extensions/migrating/mv2-sunset/

I submitted for review :pray:

v2.0.3 :tada:
https://chrome.google.com/webstore/detail/do-not-merge-wip-for-gith/nimelepbpejjlbmoobocpfnjhihnpked


Add folders for organizing scripts, container files etc

What would you like to be added:
Althrough the package build is still in a PoC state, we should start to add some folders to this repo and reorganize some of the files so it's easier for external to understand what this package build consists of.
Currently, all the scripts are added to the root directory, making it hard to follow and understand the package build.
Why is this needed:
This change is needed to make it easier for newcomers and external to understand what the package build conists of.

I was thinking of put most of things into a single directory and then we can reduce the layers by a single COPY command for all files into the chroot.
And then I received suggestion from Michael: I was thinking of something like this:
- bin/
- container/*.containerfile
- conf/
- misc/

It seems to me to split to these directories would be much easier to organize for the future. So I will split them from a single folder to the layout like the suggested.


Fixed typo in readme.md

hi uncle Claudio, I fixed a typo for you in readme.md because I have honestly no idea what PHP is, so I assume you meant PhP. Please correct me if you meant something else.
Many hug,
marit

Haha Marit, you're hilarious 🗡


Add documentation for Honeydew failure modes

Some docs about the honeydew failure modes and how to use them.

🙇


[vai 1.4 - oom error ]tf2 fast fintune on custom resnet50

Hi,
I have enabled the tf2 fast finetune on my custom resent50, here is my code.
         model = vitis_quantize.VitisQuantizer(model).quantize_model(calib_dataset=quant_dataset,                                                                   include_cle=True,                                                                    cle_steps=10,                                                                   include_fast_ft=True,                                                                   fast_ft_epochs=10)
I got an oom error as below. I also tried to adjust the fast_ft_epochs to other value but the oom error was still happened.

My VGA VRAM is 8 GB, but it seems it is not enough. Do you any recommend about oom?

try reduce calib_batch_size you can set it in vitis_quantize.VitisQuantizer(model).quantize_model

@hill19850213  Does OOM resolve? this issue is existed for a long time, I will close this issue, if you have any problem you can reopen it ,thank you .


[MINOR][DOCS] Fixes two problems in the SQL programing guide page

What changes were proposed in this pull request?
Removed duplicated lines in sql python example and found a typo.
How was this patch tested?
Searched for other typo's in the page to minimize PR's.

Can one of the admins verify this patch?

Test build #3584 has started for PR 17066 at commit 829bad0.

Test build #3584 has finished for PR 17066 at commit 829bad0.

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

Thanks! Merging to master/2.1


Duplicate static files with mkdocs

Details
On the very first build of a brand new mkdocs project, the docs build runs twice which creates duplicate CSS and JS files in the <head> of the output. When a project is built a second time, this error does not manifest itself.
To reproduce locally, do a manual project import of mkdocs itself and trigger a build.
Expected Result
The HTML build should only run once and there should be no duplicate CSS and JS files referenced in the output.
Actual Result
The output HTML references the same CSS and JS files multiple times.
This first screenshot illustrates that additional CSS and JavaScript files are added twice.

This second screenshot illustrates that the extra_css and extra_javascript have duplicates in the mkdocs.yml file.

I can't replicate this anymore, I think this was something to do with https://github.com/rtfd/readthedocs.org/pull/3946. @davidfischer can you confirm if this issue is solved?

I'm not seeing it! It is fixed.


Unable to neither update settings nor reset the extension

🐛 Describe the bug
Recently the Settings Sync extension reports "Cannot read property 'ignoreUploadFolders' of undefined" after choosing to upload settings.json. Selecting "Reset Extension Settings" reports "Sync : Unable to clear settings. Error logged on console. Please open an issue."
🌴 Visual Studio Code Version : Version 1.34.0-exploration (1.34.0-exploration)
🌴 Code Settings Sync Version : 3.2.9
🌴 Standard or Insiders : Exploration build
🌴 Portable or Installed : Installed
🌴 OSS or Official Build : Official Build (?)
🌴 Operating System : macOS Mojave 10.14.4
🌴 Occurs On: Upload (?) / Configure
🌴 Proxy Enabled: No
🌴 Gist Id: (none)
📰 To Reproduce
Steps to reproduce the behavior:

Choose "Sync : Upload / Upload Settings";
Gets "Cannot read property 'ignoreUploadFolders' of undefined" notification;
Choose "Sync : Reset Extension Settings";
Gets "Sync : Unable to clear settings. Error logged on console. Please open an issue."

💪 Expected behavior
Syncing settings.json (uploading and downloading) normally happens.
📺 Console Error Log
Console output for choosing "Update / Upload"
Console output for choosing "Reset Extension Settings"
📺  Additional context
(None.)

Duplicate of #668


Update MD file on new branch 2017-35-27_09-11-12

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

split/updateTestRef.md
:white_check_mark:Succeeded

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


Question : How to generate assets and html into different folders?

Hi,
I'm trying to document my API with Slate. Once I'm using Spring Boot and I have to separate assets into static folder and api-guide.html into template folder. But I dont have any idea to do that.

You can change js directory locations and such using config.rb: https://middlemanapp.com/advanced/configuration


List which integration tests can easily be converted to unit tests

Currently most of our Python tests are implemented as "integration" tests under tests/integration_tests, and require setting up a database and populating with data in order to run. This is overkill for many of the tests, and makes it hard to develop Superset since a local test run takes minutes.
@ofekisr created a new directory for proper unit tests. In addition to encouraging people to write proper unit tests under the new tests/unit_tests directory we should also convert the old tests. @betodealmeida started this process, but there's still a lot to do. Fortunately, there are a lot of low hanging fruits.
Straightforward

commands_test.py
email_tests.py
form_tests.py
logging_configurator_tests.py
pandas_postprocessing_tests.py
result_set_tests.py
viz_tests.py
databases/schema_tests.py
db_engine_specs/ascend_tests.py
db_engine_specs/clickhouse_tests.py
db_engine_specs/dremio_tests.py
db_engine_specs/drill_tests.py
db_engine_specs/druid_tests.py
db_engine_specs/elasticsearch_tests.py
db_engine_specs/firebird_tests.py
db_engine_specs/firebolt_tests.py
db_engine_specs/gsheets_tests.py
db_engine_specs/hana_tests.py
db_engine_specs/impala_tests.py
db_engine_specs/kylin_tests.py
db_engine_specs/mssql_tests.py
db_engine_specs/mysql_tests.py
db_engine_specs/oracle_tests.py
db_engine_specs/pinot_tests.py
db_engine_specs/presto_tests.py
db_engine_specs/redshift_tests.py
db_engine_specs/snowflake_tests.py
db_engine_specs/sqlite_tests.py
db_engine_specs/trino_tests.py
security/analytics_db_safety_tests.py
utils/core_tests.py
utils/csv_tests.py
utils/date_parser_tests.py
utils/decorators_tests.py
utils/hashing_tests.py
utils/public_interfaces_test.py

Require fixture(s)

cache_tests.py
cli_tests.py
jinja_context_tests.py
db_engine_specs/crate_tests.py
db_engine_specs/hive_tests.py
db_engine_specs/postgres_tests.py
utils/encrypt_tests.py

Thanks for bringing this up @betodealmeida ! I agree this is a really valuable initiative taken on by @ofekisr and will have a great impact on backend test quality going forward.
For the existing db_engine_specs integration tests that require the self.get_dttm() fixture, I've created the dttm fixture in tests/unit_tests/common.py which is used in e.g.test_mssql.py.
Converting these are actually a good exercise and a great way to get into the codebase (see #18251 for an example of a db engine spec test migration), so I warmly encourage new backend developers to take a stab at converting a few of these!

@betodealmeida This one's approaching the 1 year mark since any activity... and it doesn't seem like a bug. I don't quite want to close it as stale, but I'm not sure quite what to do with it. Should we make a project board, and try to get people involved in auditing/tackling the individual items?

Closing this as stale since it's been silent for so long, and we're trying to steer toward a more actionable Issues backlog. If people are still encountering this in current versions (currently 3.x) please re-open this issue, open a new Issue with updated context, or raise a PR to address the problem. Thanks!


Magma's claims

I'll cut it short, magma might regain most of the claims we need in the following days (we need about 550 total to cover all we have built), but i don't want to risk having houses exposed since i have to wait for people to log in. So, can i get a refund of like 400 claims that, as soon as all the active people come back, you can take back? Also i probably won't be able to get my last outpost (the one you get with city at 50 members) so can i get that back too?

Refunded 400 claims to cover back your land, keep me updated in this ticket once you are back to your usual playercount in your town. Thanks!

Ok, the town now has enough claims to survive without the temporary ones. However, in these days i've been reminded about some additional things lost in the corruption, such as the 25 extra claims won in the Town Tournament, the 25 bought claims and 70-75k (don't remember the exact amount, but my members can testify this is not a lie) that were in the town balance. So i ask if, instead of retrieving the 400 claims you gave us, you could only take back 350 (or in alternative 375 and give us 13k for buying back the claims) + the town balance money that was lost. Thanks in advance for your time.

Fixed, so closing.


API documentation: self via Deferred

Please include the following information:
vncdotool version: v1.2.0 / git
We're using vncdotool to automate some GUI click-through testing. We added OCR to it running tesseract as some background process, which uses the Twisted reactor to fork and join these processes. Results are passed via Deferreds, which clashes with vncdotool's use of its Deferres, which must always return self.

On VNCDoToolFactory.clientConnectionMade the deferred is called with the instance of the protocol:

class VNCDoToolFactory(rfb.RFBFactory):
    protocol = VNCDoToolClient  # class
    def clientConnectionMade(self, protocol: VNCDoToolClient) -> None:
        self.deferred.callback(protocol)  # instance

Both vncdotool.command.build_command_list and vncdotool.api.ThreadedVNCClientProxy.getattr use code to get the unbound methods from the class VNCDoToolClient — not the bound method from the instance — which then require the 1st parameter self to be that reference to the instance.

class ThreadedVNCClientProxy:
    def __getattr__(self, attr: str) -> Any:
        method = getattr(self.factory.protocol, attr)  # from class
…
def build_command_list(…):
    client = VNCDoCLIClient  # class
…
            factory.deferred.addCallback(client.keyPress, key)  # from class

While this late binding of the instance allows command to build up the list of methods before the connection is even established, it is annoying in other cases where other information should be passed via the deferred and easily leads to errors, when a method does not return self at the end: then you get the dreaded None type has not attribute XXX error.

This needs to be documented somewhere.
maybe the internal implementation can be changed to use the bound methods.

PS: You also have to be very careful if your implementation raises any exception, which is translated to a Failure instance by Twisted and the deferred switches to errback mode, where only errbacks are executed until one returns a non-Failure instance. If you do not handle that case all following calls of vncdotool.api will be ignored until you reset the deferred to normal callback mode. Basically any raised Exception is fatal for vncdotool.api.

I spent several hours digging in vnc-automate and vncdotool and trying to figure out why it doesn’t work in so obscure way and why are methods returning different types than what I see in the code… and then I found this issue.
Most of this comment has been self-censored.

The design goal of api.py is that is simulates a synchronous API, and shouldn't break users expectations.
This seems like we should change ThreadedVNCClientProxy to use self.protocol and reverse the build_command_list() and connect() calls in build_tool.   We'd get syntax errors after connection which feels a little annoying but fixing that feels like it would be a larger change, that can come later.
If we can do something to reset the api to a working state for recoverable errors, then we should do so.


Adding namespace to class

This adds the namespace to the class for clarity on use for those who are using the package.

Thanks!


Catching exceptions raised in Python Notebooks in Datafactory?

Is there a way to catch exceptions raised in Python Notebooks from output of Notebook Activity? Scenario: ADF pipeline contains a Databricks Notebook activity which is coded in Python. This notebook raises an exception and the ADF activity fails, but there is not error / exception details in the output of Notebook Activity - just the notebook run URL and "failureType": "UserError".

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

ID: a176f27b-f9fc-e0ff-0a48-aa42036051f2
Version Independent ID: d2356b83-3101-4b56-baaf-8bdf31e3bb57
Content: Run a Databricks Notebook with the Databricks Notebook activity in Azure Data Factory
Content Source: articles/data-factory/transform-data-using-databricks-notebook.md
Service: data-factory
GitHub Login: @nabhishek
Microsoft Alias: abnarain

@hiteshtulsani Thanks for the feedback! I have assigned the issue to the content author to evaluate and update as appropriate.

@hiteshtulsani The runPageUrl in the databricks activity output would contain all the exceptions thrown in the notebook. Please navigate to this Url to see the output as well exceptions.
Would this not work?

#please-close

Hi @nabhishek - navigating to runPageUrl and the content therein would help when using/monitoring ADF interactively. How about batch workflows where error logging is done externally? Unfortunately, it doesn't help to just catch the runPageUrl when the databricks log content is only available for 60 days.

@hiteshtulsani how about specifying cluster log path in the linked service. Soon we would be adding parameterization of linked service properties and you could use DateTime partitions to push the logs in DBFS for persisting it.

Still there is no way to capture the notebook logs from ADF pipeline.
Just because of this my client switched back to airflow orchestration.

@nabhishek , could you please reopen this? This piece of code in dbx notebook should definitely  trigger a failure in ADF:
try:
  thisiswrong
except Exception as e:
  raise dbutils.notebook.exit(e)

According to https://docs.databricks.com/user-guide/notebooks/notebook-workflows.html

Calling dbutils.notebook.exit in a job causes the notebook to complete successfully. If you want to cause the job to fail, throw an exception.

We have enabled passing output from notebook into ADF, which can be then consumed within ADF. Please refer to the section in the doc here. @hiteshtulsani

#please-close

@nabhishek - you are ignoring the root couse of this issue - problem is not in passing output from notebook to adf activity on SUCCESS, but on FAILURE
Notebook always returns SUCCESS do adf's activity, even exception is raised in notebook.

Was someone able to resolve this issue in passing FAILURE exception message from Notebook to ADF?

Unfortunately even 6 month later this is still not working according to their own documentation 👎  Notebook always triggers success in ADF and mr. @nabhishek is IGNORING THIS ISSUE!

@igi79 For the quick response. Then we need to add audit within Notebook as an alternate and let Notebook fail.
Do you have any alternate solution to it?

This is quite a shame, how long will it take to implement a failure option when calling exit?

Has anyone already solved this issue when calling exit?

I am also searching for solutions....

Hello, I have also come up on a dead end .  Need to fail on a bad return code from Databricks notebook, however invoked from Airflow. Any help is greatly appreciated

Please reopen tis issue! It is still not working: i try to raise exception
python -c 'raise Exception("ERR")'
dbutils.notebook.exit("ERRR message")
pyspark.sql.functions.raise_error("errMsg11")
and none is signalling to ADF, that NB errored!

Hi! Until there is a new feature, we need a workaround regarding the error message when dbutils.notebook.run() fails, the error is unclear and does not convey the true error. In this case, orchestrated by airflow

Ty

Took me forever to even find mention of this issue and then to find it's 4+ years old with no solution...
+1


deployment location:  SAL functionality

Automatic (best we can find)

By Region:

ask Europe/Asia/MiddleEast
check there is capacity, find a node
tell the node which will be used and where and confirm there was enough capacity
ask for ok, if not ok restart choose location

By Farm:

choose a farm
check there is capacity, find a mode
tell the node which will be used and where
ask for ok, if not ok restart choose location

By Node:

specific node
check if enough capacity
tell something about the node: its life since, capacity is X,Y ..., name...
ask for ok, if not ok restart choose location

branch: development_3botdeployer
commit: e33878915e08cffe07f2a739d25f8b45c08ca040


Badmatch on keyfile when calling apns:connect/2

I get the following error when calling apns:connect(apns_push, ApnsConnection):
{error,{keyfile,{badmatch,[]}}}
where ApnsConnection is a apns_connection-record. The key-file is set to undefined in ApnsConnection.
What could be the reason?

The erlang documentation says that "this option (keyfile) defaults to the same file as given by option certfile" (http://erlang.org/doc/man/ssl.html). My certificate does not contain a key-entry. Can that be the reason?


Update Git Book

The Git Book's posh-git instructions are out-of-date (e.g. no mention of Add-PoshGitToProfile, no mention of PowerShell Core, etc). The screenshot is quite bad, too:

https://github.com/progit/progit2/blob/5f97c64565fa51c87ea06209f9d75342fc294f58/book/A-git-in-other-environments/sections/powershell.asc

I took a crack at this.  See https://github.com/progit/progit2/pull/1207


[pwm/doc] PWM_EN not described in the documentation

in the RTL each channel has its own enable bit - this is not described in the documentation - nor is it shown in the schematic.
or listed in the interface list:

please document how the edge of cio_pwm_en_o relates to the edge of cio_pwm_o
also the interface name in the documentation should match the actual name

I also think there is a mistake in a figure.
I think this pulse should be 3 wide?

I also think there is a mistake in a figure. I think this pulse should be 3 wide?

Hi Rasmus, I think this figure is correct, since this is the wrap up condition at the beginning. The phase delay is 15 and the duty cycle is 3, phase delay + duty cycle is larger than 16, so it will wrap up at the beginning, these two wide pulse is the left over from the previous cycle, the pulse for this cycle starts from clk 15.

regarding the cio_pwm_en_o output.  I think this is a purely formal output, that should always be high.

I also think there is a mistake in a figure. I think this pulse should be 3 wide?

Hi Rasmus, I think this figure is correct, since this is the wrap up condition at the beginning. The phase delay is 15 and the duty cycle is 3, phase delay + duty cycle is larger than 16, so it will wrap up at the beginning, this two wide pulse is the left over from the previous cycle, the pulse for this cycle starts from clk 15.

I agree, the first pulse in the sequence will be shorter because of the wraparound.

Why do we have an output that is always high (should be removed or documented either way)

Because that is what topgen expects for fixed outputs all the comportable IO IPs are like this.

Because that is what topgen expects for fixed outputs all the comportable IO IPs are like this.

Cool - that makes it worse I will raise this issue to a broader audience


[V2V] Filter certain options in the ConversionHost#run_conversion method

At the moment the ConversionHost#run_conversion method potentially exposes some sensitive data on failure. This PR filters anything that ends with "password", "key" or "fingerprint" from the logs and/or UI.


Problem using react-native-maps

Is this a bug report?
Yes.
Have you read the Installation Instructions?
Yes
Environment
"react-native-maps": "github:react-community/react-native-maps"
"react": "16.5.0"
"react-native": "0.57.0"
Steps to Reproduce
Just try to run the application after installing react-native-maps
Expected Behavior
It should be able to use without any problems.
Actual Behavior
I've got this screen:

@Edit
Something I've just remembered. I've seen some answers saying to link using 'react-native link react-native-maps' but when I do it, I got an even worse problem.
this is what happens when I use react-native link react-native-maps
build fail rn rnm.txt
I put it into a txt file because it is too long to paste here.
Reproducible Demo
n/a

duplicate https://github.com/react-community/react-native-maps/issues/2495


x86_64 target machinize start_sp_from_bp_offset

I think I am hitting a bug with the hard coded start_sp_from_bp_offset=8 in target_machinize.
You can have a function that won't generate a prologue/epilogue, doesn't touch stack, but has args passed on the stack. start_sp_from_bp_offset shold be 0 in that case right?
My test function is (e will be passed on stack on WIN64):
int foo(int a, float b, int c, float d, int e) { return a+c+e; }

A portable example function:
int square(int a, int b, int c, int d, int e, int f, int g) {return g;}
ret_type = MIR_T_I32;
func = MIR_new_func (ctx, "test", 1, &ret_type, 7,  MIR_T_I32, "a", MIR_T_I32, "b", MIR_T_I32, "c", MIR_T_I32, "d", MIR_T_I32, "e", MIR_T_I32, "f", MIR_T_I32, "g");
MIR_reg_t g = MIR_reg (ctx, "g", func->u.func);
MIR_reg_t r = MIR_new_func_reg (ctx, func->u.func, MIR_T_I64, "r");
MIR_append_insn (ctx, func, MIR_new_insn (ctx, MIR_MOV, MIR_new_reg_op (ctx, r), MIR_new_reg_op (ctx, g)));
MIR_append_insn (ctx, func, MIR_new_ret_insn (ctx, 1, MIR_new_reg_op (ctx, r)));
Actually the issue is even worse as the ptr used for stack arg access is rbp:
mov    0x10(%rbp),%rax
movslq %eax,%rax
movslq %eax,%rax
retq
A simple fix would be to set bp whenever func has stack args I guess. Although its extra needless instructions.

Thank you for reporting this bug.
Yes, the problem is even worse.  I'll fix it tomorrow.

I've fixed and the fix is in the repository.

Thanks, for completeness I will link the commits here:
Fix: https://github.com/vnmakarov/mir/commit/bfa5da4b9b1c2f2b2053e1ec46def8ceafe1aa1f
Test: https://github.com/vnmakarov/mir/commit/254bfcd8c12fa268249f06e171a74c6b22094106


Update workflow

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 99.52%. Comparing base (b623ce2) to head (4844020).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #218   +/-   ##
=======================================
  Coverage   99.52%   99.52%
=======================================
  Files           7        7
  Lines         211      211
  Branches       43       43
=======================================
  Hits          210      210
  Partials        1        1

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


Weird messages in cmd while running crewai agents in Langflow

Bug Description
Weird messages in cmd while running crewai agents and the sequential task/crew functions in Langflow? It also takes forever to get an output. What is causing this?

Reproduction

Expected behavior
I would expect to get an output to my question what is 2+2 and then a 3 should be added, but instead it takes forever.
Who can help?
No response
Operating System
Windows
Langflow Version
Latest
Python Version
3.12
Screenshot
No response
Flow File
Error.json

Thanks. Should this code go in the Sequential task function or Sequential Crew or in the agents themselves?

@Swephoenix
Could you test it on the new version 1.0.16? Let me know if anything comes up.

Thank you for your contribution! This issue will be closed. If you have any questions or encounter another problem, please open a new issue and we will be ready to assist you.


Add management command to create demo data in bulk

Purpose
We want to be able to generate a demo database for testing purposes. It should allow creating big amounts of objects : millions of users, tens of millions of enrollments,...
Proposal
Add a management command create_demo that creates random objects in database, making use of bulk insertion for models that we expect to create in huge quantities.
I tested this script on my laptop to create a database with 10M users and 200M enrollments.

Translated fields like title or description are not populated 🤔 that is weird.


split packet in packet with maximum 'maxPacketSize' bytes data

I was using lots of metrics and I realized that udp packet were either dropped or truncated on the network, so I splitted the udp packet in multiple packets depending on maximum size per packet.

I was able to reproduce. Thank you for pointing that out.


Remove Serializable from Reference Collection classes

The two commits below are the ones to focus on (not sure why the ones from Oct 19 are showing here).
12df3be is just removing the "implements Serializable" from ReferenceCollection.
eaa4ca1 is more intrusive, as I attempted to remove serialization support classes and methods.
Assuming the above two change sets are valid, there may still be more to remove/refactor from:
org.apache.river.concurrent.AbstractReferrerDecorator
org.apache.river.concurrent.ReferrerDecorator
Also entirely possible I've missed the point, so please steer me back on course as needed.

Thanks Dan ;)


dan6.0: Fun commands ported from 5.1

I don't have everything covered in this, due to URL/api stuff.
New

booty
ding
dongers
fgtlist
lenny
lmgtfy
moretrp
nbc
ping
roll
rpsls
rr
slap
trp
whoopass
pfuff

Changes

Add /.idea to .gitignore
Fix a few typos in README.md

Opening new pull request with latest upstream changes


Group - Offline added avatar is not grayed out

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

Version Number: v1.4.63-0
Reproducible in staging?: Y
Reproducible in production?: N
Email or phone of affected tester (no customers<EMAIL_ADDRESS>Logs: https://stackoverflow.com/c/expensify/questions/4856
Issue reported by: Applause-Internal team
Action Performed:

Crate a group chat
Navigate to the group chat
Apply offline mode
Click on chat header > click on upload avatar icon
Upload avatar

Expected Result:
Avatar should be grayed out
Actual Result:
Avatar is not grayed out
Workaround:
Unknown
Platforms:

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

[x] Android: Native
[x] Android: mWeb Chrome
[ ] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

Screenshots/Videos
https://github.com/Expensify/App/assets/115492554/fc9b6a94-d510-442f-8415-eff2b9060d05

View all open jobs on GitHub

@abekkala I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors.

We think this issue might be related to the #vip-vsb.

This is from https://github.com/Expensify/App/pull/39757 @marcaaron 🙃

Definitely not a Deploy Blocker.

Avatar should be grayed out

@shawnborton What do you think about this?
We could use a greyscale filter on the image. There's a library here that can do it: https://github.com/iyegoroff/react-native-color-matrix-image-filters

@nexarvo I think it would have a default avatar, but not match the user uploaded avatar or default avatars for Group Chats. Or what else would it look like?

Ah, for offline pending patterns, we don't use grayscale filters - we just reduce opacity to something like 50%. Gut check from @trjExpensify & @Expensify/design please!

Yeah, 50% opacity for pending create. The account profile avatar should have a good example to follow.

Ok cool, thanks for the memory refresh @trjExpensify 🙇
@nexarvo There's a small detail though that this already has an open issue here. And @s77rt is assigned. @s77rt have you made progress on that or would you like to work with @nexarvo on this one as the C+?
If so, I will close this and mark that other issue as External and keep you both assigned.

Also quick note: I don't think the button should be 50% opacity, just the image, right? The button isn't disabled, so it should still be full opacity.

@marcaaron Working with @nexarvo sounds great!
@nexarvo Please comment on the other issue so you can be assigned. For the PR let's add the errors logic too, it's pretty straightforward, add errorFields.avatar

I think fading the icon is a bug and we should fix that so that only the picture is faded out and not the pencil icon. Can we fix that everywhere?

I think fading the icon is a bug and we should fix that so that only the picture is faded out and not the pencil icon. Can we fix that everywhere?

Agree. If we can fix that everywhere that'd be awesome.

Closing in favor of https://github.com/Expensify/App/issues/39983


Update pip in Centos 7 CI Dockerfile

Signed-off-by: John Mazanec<EMAIL_ADDRESS>Description
cmake was unable to be installed with pip on centos 7 arm docker image. Following a similar stackoverflow thread, upgrading pip to 21.3.1 fixes this issue.
Confirmed image build is working on both x64 and arm.
Issues Resolved
#824
Check List

[X] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #841 (3c1104a) into main (e35513e) will decrease coverage by 0.22%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #841      +/-   ##
==========================================
- Coverage   91.55%   91.32%   -0.23%
==========================================
  Files          81       81
  Lines        2249     2249
==========================================
- Hits         2059     2054       -5
- Misses        190      195       +5

Impacted Files
Coverage Δ

src/system/temporary_directory.py
82.75% <0.00%> (-13.80%)
:arrow_down:

src/system/os.py
92.30% <0.00%> (-7.70%)
:arrow_down:

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 e35513e...3c1104a. Read the comment docs.


[BUG] Unable to use a TTY error when running environment_check.sh

Describe the bug
Running environment_check as a suggested prereq to install throws an error: "Unable to use a TTY - input is not a terminal or the right kind of file"
To Reproduce
On k8s cluster, run:
curl -sSfL https://raw.githubusercontent.com/longhorn/longhorn/v1.3.0/scripts/environment_check.sh | bash
Expected behavior
I'd start seeing checks being performed
Log or Support bundle
curl -sSfL https://raw.githubusercontent.com/longhorn/longhorn/v1.3.0/scripts/environment_check.sh | bash
[INFO]  Required dependencies are installed.
[INFO]  Waiting for longhorn-environment-check pods to become ready (0/5)...
[INFO]  All longhorn-environment-check pods are ready (5/5).
Unable to use a TTY - input is not a terminal or the right kind of file
Unable to use a TTY - input is not a terminal or the right kind of file
Unable to use a TTY - input is not a terminal or the right kind of file
Unable to use a TTY - input is not a terminal or the right kind of file
Unable to use a TTY - input is not a terminal or the right kind of file
[INFO]  Required packages are installed.
[INFO]  Cleaning up longhorn-environment-check pods...
[INFO]  Cleanup completed.

Environment

Longhorn version: N/A, as this is a prereq
Installation method (e.g. Rancher Catalog App/Helm/Kubectl): N/A
Kubernetes distro (e.g. RKE/K3s/EKS/OpenShift) and version: K3s

Number of management node in the cluster: 1
Number of worker node in the cluster: 4

Node config

OS type and version:  Ubuntu 21.10
CPU per node: 4
Memory per node: 2GB
Disk type(e.g. SSD/NVMe): HDD
Network bandwidth between the nodes: 1GB

Underlying Infrastructure (e.g. on AWS/GCE, EKS/GKE, VMWare/KVM, Baremetal): Raspberry Pi 4
Number of Longhorn volumes in the cluster: 4

Additional context
This is how I fixed it on my system, but I don't know if this is the fix for all cases:

curl -sSfL https://raw.githubusercontent.com/longhorn/longhorn/v1.3.0/scripts/environment_check.sh -O
chmod 733 environment_check.sh
Nano environment_check.sh
Find and replace -it with -i (as suggested here: https://stackoverflow.com/questions/60826194/kubectl-exec-fails-with-the-error-unable-to-use-a-tty-input-is-not-a-terminal)
save and exit
./environment_check.sh

@flkdnt  Thank you for raising the issue. Could you submit the PR fixing the TTY issue?

@derekbit will do

The PR was merged, but still needs QA to walk through the ready-for-testing flow. Assigned to @cchien816 .

Verified in longhorn edc1b83
Result Pass
Can reproduce in v1.3.0
# curl -sSfL https://raw.githubusercontent.com/longhorn/longhorn/v1.3.0/scripts/environment_check.sh | bash
[INFO]  Required dependencies are installed.
[INFO]  Waiting for longhorn-environment-check pods to become ready (0/3)...
[INFO]  All longhorn-environment-check pods are ready (3/3).
Unable to use a TTY - input is not a terminal or the right kind of file
Unable to use a TTY - input is not a terminal or the right kind of file
Unable to use a TTY - input is not a terminal or the right kind of file
[INFO]  Required packages are installed.
[INFO]  Cleaning up longhorn-environment-check pods...
[INFO]  Cleanup completed.

I longhorn master-head the environment check script will get below result which does not contain Unable to use a TTY error massage
curl -sSfL https://raw.githubusercontent.com/longhorn/longhorn/master/scripts/environment_check.sh | bash
[INFO]  Required dependencies are installed.
[INFO]  Waiting for longhorn-environment-check pods to become ready (0/3)...
[INFO]  All longhorn-environment-check pods are ready (3/3).
[INFO]  Required packages are installed.
[INFO]  Cleaning up longhorn-environment-check pods...
[INFO]  Cleanup completed.


Openframe design updates, originally in another pull request.

Copied files from the original pull request into a new one.  Includes a few layout updates since the original pull request.  The openframe design matches the user example in caravel_openframe_project.

@RTimothyEdwards did you mean to update simple_por.mag ?

@jeffdi :  Yes, I did.  I moved the simple_por and the new location flags a DRC error without the change.  Not really something to worry about, though, since I'm going to have to completely redo the simple_por layout to meet stress DRC rules anyway.  But this change keeps it DRC clean for the MR deck for now.


update external resources docs

Update .md for external resources to include new NodeJS/TS based client wiremock-captain

@tomakehurst let me know if this looks good or there might be other changes required


Kyiv National Linguistic University

Added .txt file which contains univ name

@kamiQQ Pull request merged. Thank you!


fix(rpc): apply beacon root contract call in debug_traceTransaction

ref https://github.com/paradigmxyz/reth/issues/11735
We did not apply the beacon root contract call in debug_traceTransaction, but we do in trace_transaction:
https://github.com/paradigmxyz/reth/blob/4c21ebc91c7d5774018d59ae0e9b30929646fa23/crates/rpc/rpc-eth-api/src/helpers/trace.rs#L198-L219
This now does the same thing before replaying transactions

new output of the command:
curl -X POST --data '{"jsonrpc":"2.0","method":"debug_traceTransaction","params":["0x395c82b4fd16307eb98d7d96be21f043d36ff5f1247c43bdbe643dbd296163d4",{"tracer": "callTracer"}],"id":1}' -H 'Content-Type: application/json' http://localhost:8545 | jq .

{
  "jsonrpc": "2.0",
  "id": 1,
  "result": {
    "from": "0xc13ced137e90bc695cb77288962280516a2f9b8b",
    "gas": "0x30d40",
    "gasUsed": "0x906f",
    "to": "0x897a96deb94291b785faff64088f7202a019f38e",
    "input": "0x9103de44000000000000000000000000000000000000000000000000008094a18c072485",
    "calls": [
      {
        "from": "0x897a96deb94291b785faff64088f7202a019f38e",
        "gas": "0x297fb",
        "gasUsed": "0x10e0",
        "to": "0x000f3df6d732807ef1319fb7b8bb8522d0beac02",
        "input": "0x000000000000000000000000000000000000000000000000000000006708306b",
        "output": "0x1c941b19e43f340e1f9d433fdb84f4b6aad556cb8db9f4afb096bcfdc9c8ac6f",
        "value": "0x0",
        "type": "CALL"
      },
      {
        "from": "0x897a96deb94291b785faff64088f7202a019f38e",
        "gas": "0x8fc",
        "gasUsed": "0x0",
        "to": "0xc13ced137e90bc695cb77288962280516a2f9b8b",
        "input": "0x",
        "value": "0x2386f26fc10000",
        "type": "CALL"
      }
    ],
    "value": "0x0",
    "type": "CALL"
  }
}


Feature request: visibility class conjunctions

Hey y'all,
Include (s)css that allows for the conjunction of visibility classes.
That is, let's say the user wants an element to show only when the device is in portrait orientation AND "small or medium".
   @media screen and #{breakpoint(medium down)} and #{breakpoint(portrait)} {
    .show-for-medium-portrait {
      display: block !important;
      }
}

The above works, though, from what I can tell, the code must come after default visibility classes.
Thoughts?

@paxperscientiam closing this feature request out to be revisited in F7. Sounds like it would be a nice addition!


Older Browser fallback broken in 11.1.1

Console throws: SCRIPT438: Object doesn't support property or method 'getFiles'
This is a screenshot of a browserstack session loaded with IE9 on Win7. The library works fine in modern browsers, but the fallback FileAPI path never seems to be set correctly.

FYI I loaded ng-file-upload via browserify if that helps. I see around line 280 of ng-file-upload-all you're trying to make an educated guess as to where the scripts come from. Will post more info based on my setup shortly.

Got it working by adding FileAPI support detection based on the modernizr code and putting the necessary files in the root of my site.
var is_file_api_supported = !!(window.File && window.FileList && window.FileReader);
// shim to help ng-file-upload find its fallback with browserify loaded
if(!is_file_api_supported){
  FileAPI = {
    jsPath: '/',
    staticPath: '/'
  }
}


Fix #1780 to replace Jedis, JedisPool, JedisCluster constructor

@warrenzhu25 Appreciate your effort. But, would you be able to keep the changes only limited to builder pattern and remove those formatting (e.g. import formatting), method renaming, method overloading changes? These can be in separate PRs. Otherwise, review and merge process would be harder. Thanks!

@sazzad16 I got your point. But I would say most of my change are related with build pattern. I wish I could split the change into several commits,  but some parts of it are indeed tightly decoupled. I have to change everything including the test to make compilation pass. Meanwhile, I'm working on the merge with master branch. I understand this contains many files and require many efforts to review. Could you go through it first? Let me know what you think.


add description before selling points

describe what Fiedka is and does, and why it's useful, at a glance
add a little artwork (help, anyone?)

done :)


drivers: ethernet: Fix assert on receiving large PDUs

This patch fixes an assert in net_buf_simple_add() function when the lan_9250 driver would add previously added data to the net buffer on receiving large (>128 bytes) packat. This fix was to use the frags field of the net_buf structure instead.
Tested with a nRF54L15 DK and eth3 click board (with nordic connect sdk 2.8.0 and this patch).

@mariopaja  can you please have a look?

@balaji-srin LGTM
I was constantly working with CONFIG_NET_BUF_DATA_SIZE=2048 and for this reason I did not notice this issue


[CARBONDATA-3542] Support Map data type reading through Hive

Problem:
Map data type is not supported while reading through hive
Solution:
Handle to support MAP data type.

[ ] Any interfaces changed?

[ ] Any backward compatibility impacted?

[ ] Document update required?

[ ] Testing done

[ ] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.

Build Failed  with Spark 2.3.2, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/616/

Build Failed with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.2/615/

Build Failed  with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.1/611/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.1/641/

Build Failed with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.2/646/

Build Success with Spark 2.3.2, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/646/

Retest this please

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.1/642/

Build Success with Spark 2.3.2, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/647/

Build Success with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.2/647/

@dhatchayani : Can you add a testcase for
a. Reading map datatype from hive ?
b. Map of empty & null values reading from hive ?

@dhatchayani : Can you add a testcase for
a. Reading map datatype from hive ?
b. Map of empty & null values reading from hive ?

done

Build Failed  with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.1/704/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.1/705/

Build Success with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.2/710/

Build Success with Spark 2.3.2, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/711/

LGTM

retest this please

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.1/729/

retest this please

Build Success with Spark 2.3.2, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/741/

Build Success with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.2/741/

Build Failed with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.2/736/

Build Success with Spark 2.3.2, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/737/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.1/733/

retest this please

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.1/744/

Build Success with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.2/750/

Build Success with Spark 2.3.2, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/752/


Robustness improvements when loading OpenSSL certificates

The PR adds a slight improvement with respect to calling free() in case of error. There are two cases where the current implementation does not call free() on the buffer in case of error. These cases are addressed here.

[ ] Did you create an Apache Jira ticket?  (not required for trivial changes)
[ ] If a ticket exists: Does your pull request title follow the pattern "THRIFT-NNNN: describe my issue"?
[x] Did you squash your changes to a single commit?  (not required, but preferred)
[x] Did you do your best to avoid breaking changes?  If one was needed, did you label the Jira ticket with "Breaking-Change"?
[ ] If your change does not involve any code, include [skip ci] anywhere in the commit message to free up build resources.

Build errors are all unrelated. Merging.


Offline generation

Is there any .jar browser (for an old phone using java (MIDP 2.1) that I could install to generate offline passphrase?

I don't know of any browser specifically but it should work in most browsers. The main limitation is likely to be whether crypto.getRandomValues is present in an old browser, but you can always supply your own entropy if it's not available. Other than that the tool has polyfills for NFKD etc so I should expect it to work on almost all browsers.
If you do test / find one I'd be interested to hear your results.

Yes, I tried internat browser on an old samsung phone - it opened the website but cannot generate seed. There's also opera mini browser (still developed?) but on their page they write it's unable to run offline files (wonder why in settings there's tab for local files).
Maybe other users tried any jar file?
Still if we talk about offline generation - do you think that it would be safer to use dedicated laptop or old phone? My samsung do not have wifi, I turn on airplane mode - any chance to have some leak when there is no sim inserted as well?

Not really an issue with the tool, more with how it's used, and has no feature that can be implemented to resolve, so I'm closing the issue.


Hey guys check out this cool change

By the way, if we add this we have to do blah blahy.

The best thing since your last PR.

Very cool.
~ Angie, from my cell.
On Fri, Feb 5, 2021, 2:45 PM Carter Medlin<EMAIL_ADDRESS>wrote:

The best thing since your last PR.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/cagov/covid19.ca.gov-site-eng-playbook/pull/2#issuecomment-774328864,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACI5BXA557LIM2UAU2MU753S5RYKBANCNFSM4XFRTDZA
.


Fix Sequential RS linting

Stack from ghstack:

#915
#879
-> #982

@nivekt has imported this pull request.  If you are a Meta employee, you can view this diff on Phabricator.


[Go SDK]: Implement mongodbio.Read with an SDF

Implements mongodbio.Read with a splittable DoFn. Fixes #24822.

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
[ ] 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.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

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

R: @lostluck

Very big PR! I didn't have a chance to look at it earlier in the week, and it's larger than I expected, so it may take me a bit to get through it. I will be getting this first pass done in the next few hours however.

As per usual, spectacularly well done. LGTM. I couldn't find anything to suggest.
You've been doing amazingly as a contributor for the Go SDK, with consistently high quality PRs, and being largely independant. I would love to nominate you for a Google Open Source Peer Bonus for your efforts.
If that's something you're interested in, could you email me at<EMAIL_ADDRESS>with an email address I can provide to the nomination form, with "Peer Bonus" in the subject?
I have all the other information I would need to nominate you (name, github ID, why I'd nominate you).
Thanks again!

Wow thank you, that is so kind of you! I appreciate it and will send you an email. And thanks for the review and your comments!
Apologies for the PR being so big, hard to do one part without the other. This was an interesting task to work on, I might have a look at the open issue for allowing the SDF functions to have a context parameter and an error return type next.

Email received!
No worries about PR length. Your code is well factored and very clear, so it wasn't hard to follow for me.


Update Dockerfile after dependency installation change

Recently we moved from declaring an entire transitive closure in requirements.txt (and generating that with Bazel) to declaring just the top-level packages along with version information in requirements.in. This change brought with it a different set of commands for setup which breaks the existing Dockerfile. This patch fixes the Dockerfile to match the new setup.

Thanks for fixing this!


TEIIDTOOLS-751:

Correcting view editor namevalidation endpoint annotations
View deletion always returning 404, because of the duplicate deletes, which is corrected
Updating MySQL version as current version does not work with Spring Boot 2 or Java11
when undeploying the VDB, do not use "-vdb.xml" name, use vdb name
Lots of checkstyle issues, not sure why did not pop up before

LGTM


[Cherry-pick] Add zipkin deployment option to helm (#9910)

#9910: From master to release-1.1
cc: @adriancole

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: douglas-reid
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: costinm
If they are not already assigned, you can assign the PR to them by writing /assign @costinm in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

install/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@hklai for approval.


python3.4 make error

ubuntu@i-9gzoo80x:~/Multicorn-master$ make
Python version is 3.4
gcc -Wall -Wmissing-prototypes -Wpointer-arith -Wdeclaration-after-statement -Wendif-labels -Wmissing-format-attribute -Wformat-security -fno-strict-aliasing -fwrapv -g -O2 -DMAP_HUGETLB=0x40000 -fPIC -I. -I./ -I/opt/local/edb/languagepack-10/Python-3.4/include/python3.4m -I. -I./ -I/opt/pgsql/include/postgresql/server -I/opt/pgsql/include/postgresql/internal -I/opt/local/Current/include -D_GNU_SOURCE  -I/opt/local/Current/include/libxml2 -I/opt/local/Current/include  -c -o src/errors.o src/errors.c
In file included from src/errors.c:15:0:
src/multicorn.h:1:20: fatal error: Python.h: No such file or directory
compilation terminated.
: recipe for target 'src/errors.o' failed
make: *** [src/errors.o] Error 1
But I have already installed the python3-dev, and even the python3.4-dev. This error existed however. My PG version is 10.9. Could you help me?

You have to provide more detailed information about your environment. But given you mentioned python3-dev I infer you are using Debian-based system. You may want to install libpython3-dev too and try again.

I used Ubuntu16.04 system. And I found the 'Python.h'  under the dir '/usr/include/python3.4m'. But why did it can't be included in 'multicorn.h' or other files?

Use PYTHON_OVERRIDE=python3 make. It did work for me.

Thanks, it worked!


Tg/clear gem sources option

Description
Added a config option called clear_gem_sources and defaulted gem_package.clear_gem_sources to it. This supports airgapped environments that want to exclude implicit remote gem sources.

I believe this address https://github.com/chef/chef/issues/6729

looks like you have a chefstyle issue

A workaround for folks on older Chef that need this feature:
class Chef::Resource::GemPackage
  def clear_gem_sources
    true
  end
end

Drop that or something like it in a libraries/ file.


Duplication Glitch

Version: 1.8.8 Flux (Fork of Paper)
Plugin: 1.2.9
I believe how I did it is that I was in /ah and used the number keys to take an item directly out of the GUI into one of my inventory slots:

Can you replicate it again? It might be a visual glitch and the item just shows in your inventory but isn't really there. One way to check this is to do it then log out and back in and see if it's still there. Also, Creative mode seems to cause duplication bugs like this so make sure you're not in creative mode.

If your inventory name (for thaïs plugin) contains Colors code, remove  spigot doesn’t support colors with inventory.getName or Title :)


Avoid multi-line update() app crash by checking first if the focused TextField is a descendant of the ScrollView

If you have multiple KASVs and multiple text input fields across multiple containers, and wish to use the KASV update() solution to resolve your multi-line scrollView issue - you may come across this particular error screen.

We reference and summarize @alvaromb’s thorough explanation below, regarding checking React node descendants - specifically his discussion around checking if a focused TextInput is a descendent of a specific ScrollView.
For example, say there is a base screen that has ScrollView and multiple input fields. The user then taps on a button and navigates to another screen which also has ScrollView and input fields. Since the base screen did not unmount, both ScrollViews continue to exist and both will receive the keyboard events. This will cause your app to crash when the user taps on an input field on that second screen.
By checking first in the update() function that the focused TextInput field is a descendant of a specific ScrollView, we can avoid the ScrollView (that does not have the focused input as a descendent) attempting to scroll to the focused input and then crashing.

Closing this pull request for now. The descendant check is too slow to be called on componentDidUpdate.

@shigggster what is the solution to avoid this message?


Gradle build is not working

Tried to build the application as described in the README, got the following error (used Cloud9 as environment):
docker run --rm -v /home/ec2-user/environment/eb-java-scorekeep:/home/gradle/project -w /home/gradle/project gradle:4.3 gradle build
Starting a Gradle Daemon (subsequent builds will be faster)
FAILURE: Build failed with an exception.

What went wrong:
Could not create service of type ScriptPluginFactory using BuildScopeServices.createScriptPluginFactory().

Could not create service of type CrossBuildFileHashCache using BuildSessionScopeServices.createCrossBuildFileHashCache().

Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output.

Get more help at https://help.gradle.org

BUILD FAILED in 3s
make: *** [build] Error 1

Same problem here, it looks like this was build with an ancient gradle version (2.3)....

As @arbjerg mentioned, this is due to building with an older gradle version. There are probably several ways to resolve this, but I updated build.gradle by removing the following the wrapper task:
@@ -43,6 +43,3 @@ dependencyManagement {
         mavenBom("com.amazonaws:aws-java-sdk-bom:1.11.67")
     }
 }
-task wrapper(type: Wrapper) {
-    gradleVersion = '2.3'
-}

and I modified gradle-wrapper.properties by changing the distributionUrl to a recent gradle version.

Worked for me following changes in build.gradle
springBoot {
mainClass = "scorekeep.Application"
}
// task wrapper(type: Wrapper) {
//     gradleVersion = '2.3'
// }

Thanks everyone for your feedback. Elastic Beanstalk's Java 8 platform still uses Gradle 2, so leaving the build as is for now. Added a new branch corretto to test on the new Corretto platforms that are in beta. When they are final, will migrate master.


Classification Check

Classification Check
TL;DR
1st party software must have a business impact classification. See our list of classification to help you determine the classification of your application.
Why is this being asked?
It's critical that we understand the business impact of the software we write. The classification of your service determines the level of support you must provide as owners and helps us set standards.
What will happen if it doesn't get done within the expected timeframe?
This is mandatory. If this is not done you won't be able to have any runtimes.
This doesn't apply to my service. What do I do?
First, leave a comment explaining why it doesn't apply. Then, leave another comment as /not_applicable, and close the issue.
If you change your mind, make sure to comment any reason and reopen the issue. Once you've fixed it, services-db will close the issue for you next day.
I have questions/concerns about this
Please contact the Production Excellence team using Slack at #production-excellence-team.
Your service: ess/production

This compile down to a binary that's then packaged alongside an actual "runtime"... So not a service per se effectively.

/not_applicable


Is it possible to distribute EngineIO on Nuget.org ?

I mean that you as FactoryIO company could distribute the EngineIO dll on Nuget in order that we could use it easilly...


Chat formatting problem

Hello,
I am having a problem with chat formatting. To manage the chat, I use the plugin TownyChat, which is essential to me. I added in the default format {permprefix} so, to display the prefixes.
I have also configured the meta formatting, to display two different prefixes (one representing the player rank and another the Staff rank).
Only one of them is necessarily duplicated, even though I have removed the duplication option.
I leave you below pictures for you to understand, I hope you will help me, thank you very much!
TownyChat config
Meta formatting
Chat exemple
Thanks !

Removing the duplicates: first-only does exactly the opposite of what you want. Re-add it to only have each prefix once without any duplicates.
Also, next time add the images directly into your issue by just drag and drop them here to upload or with ![](url-to-image)

Hello,
Thank you for your reply.
But with or without this option, the result is the same, except that it is the prefix "[Citoyen]" which is duplicated ...

Hello,
I'm not one to ask for support as soon as I can not do something. I think I did everything I could on my side, so I turn to you ...

Hello,
The problem was actually TownyChat. I would have taken many hours to understand it, but I was not using the good placeholder. Indeed, I had to use {permuserprefix} and not {permprefix} ... Too bad for me.


feat: fastify cors options can be set in config file

We should add some documentation here within the README + check if <IP_ADDRESS>, ::1 has to be added to the default origins.


Was vswhere.exe supposed to find a "Build tools" install?

On a build agent I installed the "Build tools for Visual Studio 2017" ( taken from https://www.visualstudio.com/downloads/ , under "Other Tools and Frameworks", "Build Tools for Visual Studio 2017" ).
Running vswhere doesn't find that install. Shouldn't it?
PS S:\temp> .\vswhere -latest -requires Microsoft.Component.MSBuild
Visual Studio Locator version 1.0.62 [query version <IP_ADDRESS>636]
Copyright (C) Microsoft Corporation. All rights reserved.

PS S:\temp>

It does find the install for a "legacy" version, if that's relevant:
PS S:\temp> .\vswhere -legacy
Visual Studio Locator version 1.0.62 [query version <IP_ADDRESS>636]
Copyright (C) Microsoft Corporation. All rights reserved.

instanceId: VisualStudio.10.0
installationPath: C:\Program Files (x86)\Microsoft Visual Studio 10.0\
installationVersion: 10.0
PS S:\temp>

As of version 1.0.40 you can add -products * to include Buildtools is the search
See last paragraph of https://github.com/Microsoft/vswhere/wiki/Find-MSBuild

As stated in the usage output and documentation herein, vswhere searches, by default, for Community, Professional, and Enterprise products. Build Tools and other products installed by the new Installer can by found either by passing in a comma-separated list of product IDs to -products or just specifying -products * as @refack mentioned.

So sorry, I must have missed that. I wasn't considering BuildTools to be a
product, but makes sense.
Thanks for the responses.
On Tue, May 2, 2017 at 5:52 PM, Heath Stewart<EMAIL_ADDRESS>wrote:

As stated in the usage output and documentation herein, vswhere
searches, by default, for Community, Professional, and Enterprise products.
Build Tools and other products installed by the new Installer can by found
either by passing in a comma-separated list of product IDs to -products
or just specifying -products * as @refack https://github.com/refack
mentioned.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Microsoft/vswhere/issues/61#issuecomment-298693998,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAFEWsrY_Uqi8-53vWSAQCGf2M2yOMQ0ks5r119OgaJpZM4NOFZr
.

Just to be clear, the reason why I'm not seeing any output on our build server is because the build tools were not installed with "the new installer"?
If so, what is "the new installer"?

I think for 2015 and earlier you need to use "-legacy"
On Wed, May 31, 2017 at 8:33 AM, Uli Weltersbach<EMAIL_ADDRESS>wrote:

Just to be clear, the reason why I'm not seeing any output on our build
server is because the build tools were not installed with "the new
installer"?
If so, what is "the new installer"?
[image: vswhere exe and msbuild tools]
https://cloud.githubusercontent.com/assets/1416006/26620711/2f3a20cc-45e4-11e7-957a-70bd4921d8e8.png
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Microsoft/vswhere/issues/61#issuecomment-305109525,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAFEWllziaKOjLa0DYrWPuRucENLVUcRks5r_RfIgaJpZM4NOFZr
.

Thanks a lot, that did the trick :)

Because that is not a VS path. MSBuild was located under Visual Studio starting with 2017. It was in the same place under %ProgramFiles(x86)%\MSBuild before that. There are other ways to find MSBuild prior to 2017 that are outside the scope and purpose of vswhere.


VS2015 didn't find boost/uuid/uuid.hpp

Hi,
i had updated vcpkg yesterday and since then VS2015 didn't find boost/uuid/uuid.hpp.
Physicaly exists uuid.hpp under $vcpkginstall\installed\x86-windows\include\boost\uuid.
My Steps are:

git pull in the $vcpkginstall (on my Computer $vcpkginstall=d:\tools\vcpkg)
run scripts\bootstrap.ps1 in powershell
vcpkg integrate remove
vcpkg integrate install

and i had the same error if i use the integrate Project
PS D:\Tools\vcpkg> .\vcpkg.exe list
boost:x86-windows           1.62-2           Peer-reviewed portable C++ source libraries
cpprestsdk:x86-windows      2.9.0-1          C++11 JSON, REST, and OAuth library The C++ RES...
openssl:x86-windows         1.0.2j           OpenSSL is an open source project that provides a...
zlib:x86-windows            1.2.8            A compression library
zlib:x86-windows-static     1.2.8            A compression library
Can anybody help me?

Hi Pantau,
Could you:

Right click project -> properties
Navigate to C/C++ -> General -> Additional Include Directories
Open the edit dialog
Expand the "Macros>>" pane
Search for "vcpkg"

What are the values for VcpkgEnabled, VcpkgRoot, VcpkgConfiguration, and VcpkgTriplet? Additionally, is your project inheriting $(VcpkgRoot)include?

VcPkgConfiguration=Debug
VcpkgEnabled=true
VCPkgLocalAppDataDisabled=true
VcpkgRoot=d:\Tools\vcpkg\installed\x86-windows
VCLibPackagePath=C:\Users\MyUser\AppData\vcpkg\vcpkg.user

Ok, those values look correct for the per-project integration. Could you check that your "Additional Include Directories" is properly inheriting $(VcpkgRoot)include?

yes it is including

Von: Robert Schumacher<EMAIL_ADDRESS>Gesendet: Donnerstag, 17. November 2016 05:47:23
An: Microsoft/vcpkg
Cc: Brian Dahl; Author
Betreff: Re: [Microsoft/vcpkg] VS2015 didn't find boost/uuid/uuid.hpp (#277)
Ok, those values look correct for the per-project integration. Could you check that your "Additional Include Directories" is properly inheriting $(VcpkgRoot)include?

You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/Microsoft/vcpkg/issues/277#issuecomment-261154526, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AQHcHheb28hzvK3mwuuw9vlaPxtH9VeKks5q-9xbgaJpZM4Kvg4s.
Mit freundlichen Gr?ssen eMail: Informdata Dahl c/o Brian Dahl Gmeindr?tistr. 1C 8630 R?ti ZH, Schweiz

Just to be 100% clear, does the VcpkgRoot above have a trailing backslash or not? If it does not, that looks like the issue and we can diagnose why that's happening.
Otherwise, could you build your project from the command line using the Developer prompt and look for any reference to vcpkg on the compilation lines?

$(VcpkgRoot)include was interpolated as D:\Tools\vcpkg\installed\x86-windows\include
But i had see, in my solution i use it in two Projects. In one their find the correct Path but in the other their failed to find it.

Hmm, that's very strange. In the second project, it is not added to the include path?

Did you add the NuGet package to both projects?
Could you try again using the user-wide integration, with no NuGet packages, from the command line?

If you'd like to send us a repro solution, I'd be happy to take a look at it. You can get our email using vcpkg contact.

Ok i had found it.... it was another dependency Project that hadn't include $(VcpkgRoot)include;
And this had said it didnt found it.
I had made a user wide Integration but if i dont have $(VcpkgRoot)include; in the include path in the user wide Integration than the Compiler didnt find it. it this behavior correct?

We try to inject $(VcpkgRoot)include in automatically for you, however this will be disabled if you remove "inherit from project defaults" in a project (which removes the default %(AdditionalIncludeDirectories)). I suspect you were getting the correct behavior, however we should find a better way to diagnose this in the future.

Ok it is finding all now... thx


How to run in local computer.

I have installed the packages using npm install build and I tried using npm run dev to  watch for the files change. Could you please help me how to access the page with localhost(means what is the address)?.. Please let me know if I'm doing wrong. Thanks in advance for your help.

you need to run npm start.

Thanks for your help. I did that but I'm getting below error,
npm ERR! Windows_NT 6.1.7601
npm ERR! argv "C:\Program Files\nodejs\node.exe" "C:\Program Files\nodejs\
node_modules\npm\bin\npm-cli.js" "start"
npm ERR! node v7.5.0
npm ERR! npm  v4.1.2
npm ERR! missing script: start
npm ERR!
npm ERR! If you need help, you may report this error at:
npm ERR!     https://github.com/npm/npm/issues
In package.json, I don't see a Start command in scripts sections. Am I missing anything?

sorry, i changed this with another module. unfortunatelly this module does not come with a demo included, but I will prepare one in a near future.

Thanks. I will try to implement by myself.