SAP HANA XS Advanced, Creating a Node.js Module

Tutorial URL: https://developers.sap.com/tutorials/xsa-xsjs-xsodata.html
Please specify the step you are referring to:
Step 6: Execute the js module
After running the web module (as a web application) the application just shows "Internal Server Error".
I don't see differences between my code and the tutorial
CF_XSA_app.zip

What is the content of the run console back in the Web IDE?  That should have much more details about the situation.
Also please note that this tutorial is for XSA and Web IDE for SAP HANA.  You are trying to execute it on Cloud Foundry and Web IDE Fullstack.

Hi Thomas,
I am aware that i am using the tutorial although i am not using XSA.
However this tutorial comes closest to the scenario i am trying to study.
The strange thing is that the console doesn't give any details when running
the web module.
If the tutorial is not suited for WebIde FullStack then sorry for
the inconvenience.
regards, Gijs Pruijmboom
Op wo 8 apr. 2020 om 14:46 schreef Thomas Jung<EMAIL_ADDRESS>
What is the content of the run console back in the Web IDE? That should
have much more details about the situation.
Also please note that this tutorial is for XSA and Web IDE for SAP HANA.
You are trying to execute it on Cloud Foundry and Web IDE Fullstack.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/SAPDocuments/Tutorials/issues/4901#issuecomment-610938033,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEFQNFXY4C32L5PDJ5VTOG3RLRW2VANCNFSM4MD3ZXEA
.

The steps of the tutorial are not tested on and most likely will not match since you are not doing it in the environment that the tutorial was designed for.  Also I see in your screenshot that you that might well be using the trial.  The trial doesn't necessarily provide you with enough quota for running multiple services as required for this tutorial.  For the best experience with this tutorial, we would certainly recommend performing it on a HANA on premise/XSA system (like HANA Express).
There doesn't seem to be evidence here that there is a specific issue with the content of this tutorial. For general issues developing; we would request you instead post your question to the SAP community Q&A where it will certainly have more visibility.


Fix 404 error when calling with full URL

Fix a 404 when calling
gittar.fetch('https://github.com/lukeed/mri').then(console.log); where the URL would be constructed incorrectly (https://github.com///github.com/lukeed/mri/archive/master.tar.gz) due to the hostname already existing

npm@6 dropped support for Node 4 so npm install@latest will fail there. Node 4 is now EOL so 🤷♂. Let me know if you'd like me to drop Node 4 from travis/appveyor and add 8/10


unable to index embeddedlist property having linked type embeddedmap or linked class

Hi, i'm wondering if it is possible to achieve any type of indexing on the "rules2" and "rules3" properties in the example below... as you may see i'm getting exceptions for these ones:
CREATE CLASS Organization EXTENDS V
CREATE CLASS Permission EXTENDS V

CREATE PROPERTY Organization.rules2 EMBEDDEDLIST EMBEDDEDMAP
CREATE INDEX idx_org_rules2 ON Organization (rules2) NOTUNIQUE

Response:
com.orientechnologies.orient.core.index.OIndexException: Cannot create the index 'idx_org_rules2'
--> com.orientechnologies.orient.core.index.sbtree.local.OSBTreeException: Error creation of sbtree with nameidx_org_rules2
--> java.lang.NullPointerException
CREATE PROPERTY Organization.rules3 EMBEDDEDLIST Permission
CREATE INDEX idx_org_rules3 ON Organization (rules3) NOTUNIQUE

Response:
com.orientechnologies.orient.core.index.OIndexException: Linked type was not provided. You should provide linked type for embedded collections that are going to be indexed.

Notice that this is marked duplicate but cant seem to find what it is considered dupe of.
Is it still the case that we cant index embedded Lists as of 2.0.6?


Adding support for DigitalOcean as Cloud provider

Is your feature request related to a problem? Please describe.
This tool/linter is a great help for the daily workflow, professionally or even inside projects. Thanks for this!
While many big organizations rely on the supported Cloud Providers like AWS, GCP, or Azure - for side projects, executed in my spare time, I mostly go for DigitalOcean as a cloud provider (main reason: simplicity).
Problem/Request: I would like to integrate tfsec as a security linter for my terraform files that manage infrastructure on DigitalOcean.
Describe the solution you'd like
No particular solution in mind, except of adding support.
Having a look at the existing GCP checks, there are many similarities when it comes to DigitalOcean. E.g.:

digitalocean_firewall vs. GCP003/GCP004

Describe alternatives you've considered
None so far.
Additional context

DigitalOcean Provider @ Terraform Registry

hey @andygrunwald, this is a good idea and we can add some checks. it's worth pointing out I think that you can add missing checks using the custom checks mechanism.
see https://www.tfsec.dev/docs/custom_checks/ for more details

hey @andygrunwald, this is a good idea and we can add some checks. it's worth pointing out I think that you can add missing checks using the custom checks mechanism.
see https://www.tfsec.dev/docs/custom_checks/ for more details

Thanks, @owenrumney. This is indeed a good idea.

Thanks, @owenrumney. This is indeed a good idea.

I'll take a look today at the provider and add some initial checks. Anything you can think of in the meantime would be helpful to speed up what I add

I'll take a look today at the provider and add some initial checks. Anything you can think of in the meantime would be helpful to speed up what I add

Took a lot longer than intended, but this is underway now - https://tfsec.dev/docs/digitalocean/home/

Very cool. Thanks @owenrumney


CONTRIBUTING

Hello Justin!
I just found this package and I'll be giving it a spin.  Rather than just forking into my own repo, I checked to see if you have a CONTRIBUTING.md to guide those of us who might want to help.
I hope you're well, and thank you for this project!

Thank you for reaching our Jeff, any help would be greatly appreciated! A basic contributing.md file has been uploaded.

Awesome, thank you so much Justin!  I’ve got a conference this week, but I’m hoping to integrate it next week.
Thanks again for the project!


reporting needs polishing

@rultor release, tag is 0.8.1

@rultor release, tag is 0.8.1

@yegor256 OK, I will release it now. Please check the progress here

@rultor release, tag is 0.8.1

@yegor256 Oops, I failed. You can see the full log here (spent 8min)
LC_IDENTIFICATION="en_US.UTF-8"
LC_ALL=en_US.UTF-8
++ dirname dataset/temp/env-done.txt
+ mkdir -p dataset/temp
+ date +%s%N
Finished, in 2s

+++ lint +++

------------------------------------
Your code has been rated at 10.00/10

------------------------------------
Your code has been rated at 10.00/10

------------------------------------
Your code has been rated at 10.00/10

------------------------------------
Your code has been rated at 10.00/10

------------------------------------
Your code has been rated at 10.00/10

Inspecting 1 file
.

1 file inspected, no offenses detected
Finished, in 12s

+++ tests +++

filters/test-010-delete-non-java-files.sh:
👍🏻 A binary non-Java file was deleted

filters/test-020-delete-package-info.sh:
👍🏻 A package-info.java file was deleted

filters/test-030-delete-tests.sh:
👍🏻 A Java test file was deleted

filters/test-040-delete-unparseable.sh:
👍🏻 An unparseable Java file was deleted

filters/test-050-delete-long-lines.sh:
No files out of 1 had lines longer than 1,024 characters
❌ Non-zero exit code (TARGET=/cam/test-zone/filters/test-050-delete-long-lines.sh/target)
make: *** [Makefile:75: test] Error 1
container f474e321d87ac4b1d5eb22735ea69bad88bb6d8ffae06da8cbb3696826f47bf1 is dead
Mon Oct 23 08:23:04 AM CEST 2023

@rultor release, tag is 0.8.1

@rultor release, tag is 0.8.1

@yegor256 OK, I will release it now. Please check the progress here

@rultor release, tag is 0.8.1

@yegor256 Done! FYI, the full log is here (took me 11min)


server does not know actual translation(etc) of clients

in this implementation, server is just broadcasting components. it can not send feedback of client's translation, including warning, shutting malclient.

now client's Movement events are including translation and implemented ForceReplicate event


Where to place plugin call?

 The mediaelements.js part is working fine. In particular, I'm trying to get the preview plugin.js add-on to work. I really want the play/pause on hover option. I am sure its a very simple thing, I'm just not understanding where I need to put the call, and then the parameters/features so the hover part will work.
Sorry to bother with such a basic question. Thank you!

No question is stupid ;) As far as I know, you might need to put all the JS files in your theme (I think it is located at wp-includes), and also the JS code to enable the feature you want. Or whenever you edit the footer for your theme should be the proper place to do it. Again, I'm not a WordPress expert. But this is based on what I've seen. Hope it works for you.
Please star us at https://github.com/mediaelement/mediaelement and https://github.com/mediaelement/mediaelement-plugins to continue improving the package and giving it more popularity; also tell your friends/coworkers/etc. to support the project, if you don't mind.It would be greatly appreciated it


Fixed testcases

Fix testcases due to recent Cake3 updates.

Coverage remained the same at 94.44% when pulling 230edc0672ab38aba4fce1972568d5aa79b2428e on fix-testcases into 09e460896a32c1386f1d0889889cbca1522572fc on master.


Ripd1373

New rules for payment paths:

Sanity check on newly created strands
Better loop detection
Better tests (test every combination of path element pairs)
Disallow any root issuer (even for xrp)
Disallow compount element typs in path
Issue was not reset when currency was XRP
Add amendment

The "Fix limiting step re-execute bug" does not need to be reviewed, it's a separate PR.

@scottschurr I added two commits to address your comments. The first contains the misc fixes you suggested. The second replaces uses of variadic templates with std::initializer_list to address your concerns about link size.

Visual Studio can't handle the C++14 constexpr expressions in PayStrand_test.cpp.

Pushed commit to address latest round of feedback

Latest changes look good to me.  Still :+1: assuming conflicts are resolved.

rebased onto 0.60.0 Will squash after ci runs

Codecov Report

Merging #2015 into develop will decrease coverage by 0.05%.
The diff coverage is 75.95%.

@@             Coverage Diff             @@
##           develop    #2015      +/-   ##
===========================================
- Coverage     67.7%   67.64%   -0.06%
===========================================
  Files          680      680
  Lines        49219    49399     +180
===========================================
+ Hits         33323    33416      +93
- Misses       15896    15983      +87

Impacted Files
Coverage Δ

src/ripple/app/main/Amendments.cpp
100% <ø> (ø)
:arrow_up:

src/ripple/app/paths/impl/XRPEndpointStep.cpp
71.87% <100%> (-4.22%)
:arrow_down:

src/ripple/app/paths/impl/Steps.h
82.97% <100%> (+5.2%)
:arrow_up:

src/ripple/protocol/impl/Feature.cpp
100% <100%> (ø)
:arrow_up:

src/ripple/protocol/STPathSet.h
98.36% <100%> (ø)
:arrow_up:

src/ripple/app/paths/impl/DirectStep.cpp
71.18% <100%> (-8.34%)
:arrow_down:

src/ripple/app/paths/impl/BookStep.cpp
79.79% <50%> (-5.97%)
:arrow_down:

src/ripple/app/paths/impl/PaySteps.cpp
71.84% <75%> (ø)
:arrow_up:

... and 10 more

Continue to review full report at Codecov.

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

Squashed and merged to develop as 846723d7719124938570ab3911758321f3025671


Feature request: give some more context info

Now it shows the block content where the tag is presented, add breadcrumb of the orgin of the node (similar to zooming into a node).
Caveat this might become big so maybe this is better:
Node title > ... > last parent node: (and hover over the ... to show the content?)

[b] bladieblad #tag bladiebla

Thanks for the suggestion. Adding breadcrumb is what I was focusing on right now!
I was thinking about treat node as it is(every tag in block will be individually listed), and add breadcrumb to every block entries.
Do you want nodes to be grouped when blocks got a common parent? @tombroens
With your suggestion, a node will be the parent block, right?


[Feature]: UTF8 support

What would you like to see added?
Currently the line will straight up panic if utf8 characters are inputted. Should add utf8 support to CursorBuffer and adjust indexing accordingly to avoid this.

resolved by #139


Image at top of contributing.md is not accurate

Image references npm run tdd which is undefined
Image doesn't reference npm run tester, but documentation does

Yep, this is outdated, too. I'm planning to rewrite this.

Resolved in 13905ed.


fix handling of bareword args matching node flags; closes #3761

maybe not the most elegant solution, but it seems to work.
cc @nknapp @janpot

Coverage increased (+0.01%) to 91.664% when pulling 5e7024ff454b6049f8ce41695600e065e58363f1 on boneskull/issue/3761 into 6535965e8655a66de54fc0ad9465c2eb825f13f8 on master.


⚠ Turnos 24x7 has degraded performance

In 0b622e4, Turnos 24x7 (https://reservas-dev.alternativasinteligentes.com/) experienced degraded performance:

HTTP code: 200
Response time: 1404 ms

Resolved: Turnos 24x7 performance has improved in b436cfa.


fix(Drawer): add 100% height to inner Box

Overview

Adds 100% height to the intermediate Box within Drawer.

I'd missed this in https://github.com/Codecademy/client-modules/pull/2083, it needs to be there so tall content can have a vertical scrollbar.
PR Checklist

[ ] Related to designs:
[x] Related to JIRA ticket: GM-378 WEB-1417
[x] I have run this code to verify it works
[ ] This PR includes unit tests for the code change

📬Published Alpha Packages:@codecademy/gamut-kit@0.5.100-alpha.944cb8.0@codecademy/gamut-labs@24.2.2-alpha.944cb8.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://615b1a4c858de32a773f5a2e--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-kit@0.5.101-alpha.652e65.0@codecademy/gamut-labs@24.3.1-alpha.652e65.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://615b69c1d88f5d00914fba3a--gamut-preview.netlify.appDeploy Logs


Alignment of "Execute R code" keyboard shortcut with R extension

In the Quarto VSCode extension, [Cmd/Ctrl]+Enter executes the R code on the current selected or positioned lines of the document.
In the R VScode extension, the same shortcut executes the current selected text if a selection is made, or otherwise the current statement (traversing both up and down to find the bounds of the current statement).
It'd be great if the Quarto VScode extensions' behaviour was adjusted to align with the R extension, so that there aren't differences depending on whether a user is editing an R script or a Quarto document. (I personally prefer the R extension's behaviour because it means you can do things like execute pipes while leaving the assignment off for exploration, but I accept that it's largely a matter of taste!).

Interesting. On the other side I think we'd want to take inventory of what the Python extension does as well as what RStudio does (which is I believe what our current behavior is based on). We'll dig further into this soon.

That's fair! Another option may be to expose commands for both to the extension API so that users can map the keys to their preference.

Related to #7

I just checked, and for pipe chains and function calls the behaviour of RStudio and the VSCode extension are identical.
CTRL + ENTER runs expression the cursor is on in the console, irrespective of where it is placed in that expression.

Not sure if this change was documented, but as of Quarto 1.1.251 and Quarto VSCode extension 1.54.0, Cmd+Enter now aligns with the R extension in VSCode, running:

selected text (rather than the whole lines) if text is selected, or
the whole statement that the cursor is on if there is no text selected, traversing forward and backward

Thanks very much!


Add index to skin_indices for synthetic skins

Fixes generating skeletons for files without meshes, for example https://sketchfab.com/3d-models/racer-girl-c64ba6f76a804c7f99563bfcef225f06

Merged. I'm rushing this and making the godotengine/godot cicd be the primary.


*: extract types.Cluster from etcdserver.Cluster

The PR extracts types.Cluster from etcdserver.Cluster. types.Cluster
is used for flag parsing and etcdserver config.
There is no need to expose etcdserver.Cluster public, which contains
lots of etcdserver internal details and methods. This is the first step
for it.

The main change is to change Cluster *Cluster to InitialCluster types.NameURLPairs in etcdserver.Config, which decouples how it is set from string and how it is used internally. The benefits of it:

Cluster struct has many additional info, including cluster ID and member ID, which doesn't need to be set and accessed when building Config.
There is no need for Cluster to be printed as name=http://urls anymore. The new Cluster.String is like this: {ClusterID:e9c7614f68f35fb2 Members:[&{ID:a8266ecf031671f3 RaftAttributes:{PeerURLs:[http://localhost:7001]} Attributes:{Name:infra1 ClientURLs:[]}} &{ID:6e3bd23ae5f1eae0 RaftAttributes:{PeerURLs:[http://localhost:7002]} Attributes:{Name:infra2 ClientURLs:[]}} &{ID:924e2e83e93f2560 RaftAttributes:{PeerURLs:[http://localhost:7003]} Attributes:{Name:infra3 ClientURLs:[]}}] RemovedMemberIDs:[]}, which prints out more status.

This also helps make Cluster private because config.Cluster is no longer modified and passed around in etcdserver package.
The code change includes two parts:

use types.NameURLPairs instead of *etcdserver.Cluster when building Config.
create *Cluster variable in NewServer instead of modifying config.Cluster. This also helps make Cluster struct private in the future, which cannot be exposed in Config struct then.

The PR doesn't have behavior change.

all comments addressed.

@yichengq Thanks for refactoring this. This pr looks good to me.

@barakmich Can you take a look?

LGTM if you'd like to merge, one question.


KerbalStuffBot failed travis builds

Recent pull requests made by the bot are failing the travis build. What is it doing wrong?

I'm checking and it seems that most of them are caused by invalid licenses.
Others are more misterious, Jenkins doesn't report any cause for those...

I don't know how adding mods to KerbalStuff works but maybe adding some kind of drop-down menu with common licences (in valid format :wink: ) to choose from would help? Of course this is Kerbal Stuff related request ;)

Yes, the problem is usually with the mods for which the author chose a the "other" option an then typed in a license string unknown to us in the free-form field that appears

Wanted to add to this that there is a checkbox when you add your mod to KerbalStuff that says if you want to add it to CKAN as well.  I UNCHEKCED this and KerbalStuff still submitted a PR here for my mod :(

Yep, but this is now something that has been go to the kerbalstuff repo. I even believe there is an open ticket for that over there.

It's also terrible for mods that need install or depends directives, it seems to me.

I think a lot of Mod Authors don't really understand software licensing. They rarely include the license text in their download, which makes it really hard to know which license they mean when they say "MIT", for instance. Found one that had a license called "CC BY-SA-NC", which I assume meant "CC-BY-NC-SA".

Quick glance over in the KS repo made me find https://github.com/KerbalStuff/KerbalStuff/issues/98 which is a better tracker of this issue so I'll close this.


Add empty file/folder hooks

With these hooks, the end user must define the a pathSrc (path source) and fileName (the name of a file that should be checked) in their respected actions.

Thanks for your pull request.
There are a lot of things I want to get changed before merging it to master it it will likely take a bit communicating and discussing every bit of it.
So what I'll do is I will merge this pull request to a different branch and do the changes on my own :)
So you should be prepared for some things changing when I will release it. Especially the option names and the way to configure the options feel a bit to complicated.

Great that you liked idea. If you need something from me feel free to notify me.


fix #22 ensure phantomsj stderr is processed by line

It adds byline as a dependency, hope it s fine, otherwise i d like to get a suggestion.
It also makes xo happier when validating stream.js, previously there was too many nested callbacks

open to comments if any : )

open to comments if any : )


Use an associated type for the terminal output handle type.

[breaking-change]

r? @alexcrichton
(rust_highfive has picked a reviewer for you, use r? to override)

Thanks!


LaGrange 1.9.1 fails to run on Xubuntu 20.04

With a clean install - recursive clone from github, 1.9.1, release cmake.  Builds fine, but nothing happens upon execution.  --help, --version works as expected.  Providing a url does nothing.

OK, may be my bad.  doing a ps showed that a running instance existed, killing it seems to work.  Rebuilding 1.9.1 (I tried downgrading).  Yes, now it works.
But this points out another issue: starting lagrange when there is already an instance does not bring up the existing instance, leading to confusion (it looks like crash/nothing happened).

I will close this, and open a proper issue for instance handling... Sorry about the chain of issues.


[ja] Fix observability and portability

Describe your changes
Changed the Japanese notation of "Observability" and "Portability" from Kanji to Katakana.
Background
In the Japanese documentation of New Relic and Splunk, as well as in the translation of the book "Observability Engineering," the translation of "observability" is written in katakana "オブザーバビリティ" and "オブザーバビリティ(可観測性)" is used in situations where it is better to have a supplement.
This is expected to improve the searchability of katakana "オブザーバビリティ" as well as kanji "可観測性", and to make it easier to associate the word with English words.
Similarly, I propose that "portability" be changed to "ポータビリティ" and "ポータビリティ(可搬性)".
Related issue number or link (ex: resolves #issue-number)
No issue related to this PR
Checklist before opening this PR (put x in the checkboxes)

[x] This PR does not contain plagiarism

don’t copy other people’s work unless you are quoting and contributing it to them.

[x] I have signed off on all commits

signing off (ex: git commit -s) is to affirm that commits comply DCO. If you are working locally, you could add an alias to your gitconfig by running git config --global alias.ci "commit -s".

@/ reviewer
My apologies for missed operation 🙇

Files changedが90もあるのでおそらくja以外のフォルダも更新している可能性があります(mainをcloneしたのかな)？
dev-jaをcloneして修正をお願いします。

dev-ja から出し直しました。


Use Makefile instead to add consumer to our CI

I'd highly recommend to use Makefile instead so we can even add this to our CI.
To add this to the CI, you just need to create a Makefile that has "test" as the target, and add the directory path to the target in .travis.yaml below:
https://github.com/aws-samples/amazon-kinesis-video-streams-demos/blob/e246e81370751088cb82b32be6e5176bb413d0f7/.travis.yml#L17
It's definitely not the best test, but it can add some sanity check at least.
Originally posted by @lherman-cs in https://github.com/aws-samples/amazon-kinesis-video-streams-demos/pull/64#discussion_r505072363

The makefile works on systems. However, travis has a log limit https://docs.travis-ci.com/user/common-build-problems/#log-length-exceeded, which causes a problem since mvn package generates a lot of logs. This error can be solved using mvn package -q which will set verbosity to error. However, due to no logs for prolonged period, travis build fails with the following error: Build times out because no output was received .
We need to look into the best way to add the consumer-java to .travis.yml.

It's going to be very silly to do this, but maybe we can add something like,
while 1; do echo "Processing" && sleep 1; done &

Yes. I was thinking something along the same lines. But thought of this as a last ditch effort after exhausting all other options :/


fix image pull routine

@steiler check this, I think this method is cleaner, as it doesn't need to list all the images and do hairy tag matching.

Yes, you're right, this mechanism is nicer I had not expected that you'd be able to query for the docker.io/library/alpine image and get the alpine image back.
But it seems to work, at least the CLI works that way ... so yes, its better!


View size not compatible error in rtho.py

Team,
While running the sample script ./bin/rtho.py, using README provided command the following error occurs.
python bin/baselines.py --network vgg --dataset cifar_10 --optimizer sgd --momentum 0.9 --lr-scheduler cyclic
Traceback (most recent call last):   File "bin/rtho.py", line 125, in <module>     train_rtho(args.network, args.dataset, args.num_epoch, args.batch_size, args.optimizer, args.lr, args.momentum,   File "bin/rtho.py", line 92, in train_rtho     hyper_optim.compute_hg(net, first_grad)   File "/home/tyu/sandbox/original_adatune/adatune/adatune/mu_sgd.py", line 68, in compute_hg     hvp_flatten = torch.cat([h.view(-1) for h in hvp])   File "/home/tyu/sandbox/original_adatune/adatune/adatune/mu_sgd.py", line 68, in <listcomp>     hvp_flatten = torch.cat([h.view(-1) for h in hvp]) RuntimeError: view size is not compatible with input tensor's size and stride (at least one dimension spans across two contiguous subspaces). Use .reshape(...) instead.
Same error reported when using adam, in mu_adam.py
Changing line 68 in mu_sgd.py into the following by adding contiguous, fixed the problem and can see the script running.
hvp_flatten = torch.cat([h.contiguous().view(-1) for h in hvp])
It will be good if you can advise whether this will be a correct fix?

I think it's happening due to the PyTorch version issue - feel free to replace view(-1) with  reshape(-1) and let me know if it works.

I'm also getting this error, and I can confirm that it was resolved by changing view(-1) to reshape(-1).
And just so my PyTorch version is recorded here:

>>> torch.__version__
'2.0.0.dev20230115+cu118'


Period After Pragmas

Currently, a period is added after a pragma, which makes the code incorrect. That should not happen

Fixed it! See #34


Netshoot kubectl plugin

Hi, I am a big fan of netshoot and so I've developed a kubectl plugin which can spin up netshoot quickly without having to memorize the correct syntax for kubectl run or kubectl debug:
https://github.com/nilic/kubectl-netshoot
Main goal was to cover all the example use cases from Netshoot with Kubernetes. I have also added a few additional example use cases to the readme (debugging nodes, running command instead of attaching to an interactive shell).
Through flags, plugin allows for selecting which netshoot container image to use, as well as whether to run the container in the host network's namespace.
If this sounds useful, please consider adding a link to my project to netshoot readme. I am also open to any suggestions or ideas how to further improve the plugin. Thanks 🙂

@nicolaka

@nilic this is awesome! I thought of easier ways to integrate netshoot with kubectl and having a plugin definitely helps. Thank you! I will spend some time playing with the plugin and update the README to point to it 👍🏽

Thank you again! I added a reference to your plugin in the readme in #131 !


Use an OrdinalIgnoreCase HashSet instead of ToUpperInvariant on each property name

@stephentoub i will make this change across the entire code base.
Thanks for the help.


parser : add error number for store limit error (#612)

What problem does this PR solve?
cherry-pick #612 to release-3.0
What is changed and how it works?
add error number for store limit error in mysql package
Check List
Tests

Manual test (add detailed scripts or steps below)

Release note

add error number for store limit error (#612)

/run-build-integration

/rebuild

The error reported in CI says
store/tikv/2pc.go:753:69: time.Since(lockWaitStartTime).Milliseconds undefined (type time.Duration has no field or method Milliseconds)

The method Milliseconds of Duration is added go 1.13, it seems we need to set the go version to 1.13 in go.mod.

The error reported in CI says
store/tikv/2pc.go:753:69: time.Since(lockWaitStartTime).Milliseconds undefined (type time.Duration has no field or method Milliseconds)

The method Milliseconds of Duration is added go 1.13, it seems we need to set the go version to 1.13 in go.mod.

Thanks very much, let me try it.


New msgs

I noticed new msgs sometimes do not show up when I log into wifi. The node status shows green and a single new message appears, but my phone app shows more new msgs. Even leaving it open for a while does not display the new messages.

This was fix with the refactor. Feel free to re open it if it happens again


Only count ping replies of type 0 as valid ping replies

myping will count replies of type 11 (Time Exceeded) as well as other rare cases as valid pings.  It is probably safer to just only count replies with the type set to 0 (Echo Reply) as valid pings.

I think the reasoning behind the pull request is sound, however, the commit would introduce hard tabs to line 194 and 195. The rest of the file uses soft tabs. An otherwise identical commit with soft tabs should be accepted.

Good catch on the tabs.  I have fixed the lines to match the rest of the file.

Ref: http://www.faqs.org/rfcs/rfc792.html

The intention of the CheckIcmpReplyHeader() is to count only the Echo Reply type of ICMP message. So it is safer to just compare the return value of CheckIcmpReplyHeader() with 0 at the line 947 to avoid the miscount. Also it would be better for CheckIcmpReplyHeader() to return non-zero value at the end to ensure the consistent logic. Otherwise that return of zero could cause the confusion.

True, when the address is not a IPv4 address the function should return something other than 0.  What do you think would be the best value? -1?

The value of -1 is a good one to use.

Thanks. Good to go.

Reopen the request to merge the changes.


None of the tutorial examples apart from sync_example works

Hi there,
I have read through the tutorial and tried to understand the code.
When calling each example, the only one that actually prints anything is the sync_example.

Hi @molleweide .  I puzzled over that same question for the past few days, and just this noticed this morning that @ms-jpq calls the functions (other than sync_example) with another set of brackets, e.g. async_example()() . This works!  Thanks @ms-jpq for the lovely clean code, and good tutorial.  Next time I will remember to actually read the instructions!
Until next time

Hi @molleweide . I puzzled over that same question for the past few days, and just this noticed this morning that @ms-jpq calls the functions (other than sync_example) with another set of brackets, e.g. async_example()() . This works! Thanks @ms-jpq for the lovely clean code, and good tutorial. Next time I will remember to actually read the instructions!
Until next time

Aaaah, now when you told me this I looked into the plugins/ file and I can see the double ()() too so thanks a lot for telling me about this. However, I still can't see this in the readme instructions :P


Why does each federation have a dataset？

Hi, your work is very inspiring.
I am puzzled that every federation can have a dataset.
Shouldn't it be every federated's client who owns the dataset?

Which algorithm, and which part, you mean? I lost your question.

Federation is the set of different clients, which is a new problem setting in the paper.


Prepare for alpha release

I am thinking to perform an alpha version release of this plugin.
CC @ikedam @oleg-nenashev

Currently the plugin doesn't have any tests (or they are ignored), but I'm planning to allocate half a day (or maybe a day) per week to write tests. Depending on my progress on the other plugin.

Looks good to me.
Let's release the alpha.

IMHO README.md needs an update as well

I've updated the README file with some minimum basic information.
The plan is to add a more complete documentation in the future.


Reconnection logic needs improvement

I daemonize metrics2mqtt via the suggested method (using supervisor). I'm finding that when I restart my MQTT broker, metrics2mqtt errors out quite rapidly – so rapidly, in fact, that at some point, supervisor gives up. Example:
2020-08-12 21:52:20,388 - metrics2mqtt - ERROR - Error while trying to connect to MQTT broker.
2020-08-12 21:52:20,389 - metrics2mqtt - ERROR - [Errno 111] Connection refused
Traceback (most recent call last):
  File "/usr/local/bin/metrics2mqtt", line 8, in <module>
    sys.exit(main())
  File "/usr/local/lib/python3.8/site-packages/metrics2mqtt/base.py", line 201, in main
    stats.connect()
  File "/usr/local/lib/python3.8/site-packages/metrics2mqtt/base.py", line 40, in connect
    self.client.connect(self.broker_host)
  File "/usr/local/lib/python3.8/site-packages/paho/mqtt/client.py", line 937, in connect
    return self.reconnect()
  File "/usr/local/lib/python3.8/site-packages/paho/mqtt/client.py", line 1071, in reconnect
    sock = self._create_socket_connection()
  File "/usr/local/lib/python3.8/site-packages/paho/mqtt/client.py", line 3522, in _create_socket_connection
    return socket.create_connection(addr, source_address=source, timeout=self._keepalive)
  File "/usr/local/lib/python3.8/socket.py", line 808, in create_connection
    raise err
  File "/usr/local/lib/python3.8/socket.py", line 796, in create_connection
    sock.connect(sa)
ConnectionRefusedError: [Errno 111] Connection refused
2020-08-12 21:52:20,432 INFO exited: metrics2mqtt (exit status 1; not expected)
2020-08-12 21:52:23,438 INFO spawned: 'metrics2mqtt' with pid 36
2020-08-12 21:52:23,863 - metrics2mqtt - ERROR - Error while trying to connect to MQTT broker.
2020-08-12 21:52:23,863 - metrics2mqtt - ERROR - [Errno 111] Connection refused
Traceback (most recent call last):
  File "/usr/local/bin/metrics2mqtt", line 8, in <module>
    sys.exit(main())
  File "/usr/local/lib/python3.8/site-packages/metrics2mqtt/base.py", line 201, in main
    stats.connect()
  File "/usr/local/lib/python3.8/site-packages/metrics2mqtt/base.py", line 40, in connect
    self.client.connect(self.broker_host)
  File "/usr/local/lib/python3.8/site-packages/paho/mqtt/client.py", line 937, in connect
    return self.reconnect()
  File "/usr/local/lib/python3.8/site-packages/paho/mqtt/client.py", line 1071, in reconnect
    sock = self._create_socket_connection()
  File "/usr/local/lib/python3.8/site-packages/paho/mqtt/client.py", line 3522, in _create_socket_connection
    return socket.create_connection(addr, source_address=source, timeout=self._keepalive)
  File "/usr/local/lib/python3.8/socket.py", line 808, in create_connection
    raise err
  File "/usr/local/lib/python3.8/socket.py", line 796, in create_connection
    sock.connect(sa)
ConnectionRefusedError: [Errno 111] Connection refused
2020-08-12 21:52:23,921 INFO exited: metrics2mqtt (exit status 1; not expected)
2020-08-12 21:52:24,923 INFO gave up: metrics2mqtt entered FATAL state, too many start retries too quickly

The only way to fix this is to restart supervisor.
After examining the relevant section of the code, I think the problem is that you raise an exception after logging an error message; too many exceptions too quickly will choke supervisor. I don't see that exception being caught anywhere?

FYI, ran into this again today. Any thoughts?

I have the same issue when trying to run it manually.
However it is failing correctly, because my MQTT server is not listening on default port and you can not set the port. Are you sure you can access the MQTT server?

@lipoja Definitely. And even after the MQTT server has been up and accessible for a while, this library never recovers.￼

@bachya Do you have patch for this issue? I wanted to give a try to this - I am currently running everything through MQTT at home.
Next time I have to check also all the forks, I started refactoring the code as well and I saw that you've already done some changes as well.
Are you using it, would you recommend it or should I search for something else?

@lipoja I don't personally have a patch for this; given this issue's stagnancy, I've gone back to using Glances.

@bachya Oh, I have to check Glance. It looks pretty good. Yes, you are right it seems that this projects froze.
Thank you for the suggestion. Have a nice day :]

The main reason I wanted to use this repo was because it seemed lightweight and used MQTT, but it seems now Glances also has an option to use MQTT. If anyone can help with setting that up in HomeAssistant, preferably with auto 'discovery' I'd appreciate the tip!


Use object.assign

@asvny, nice! Could you do this with the VoiceRecognition component, as well?

@grvcoelho I have done for VoiceRecognition   as well ! #15


strip off the '.' & '+' from email when validating so we can handle the gmail and google apps feature

The authentication starts with a query for an Account entity with a specific email address, does this mean we have to query all possible combinations? That is not realistic.


bgpv2/docs: add ebgp multihop documentation

Adding missing ebgp multihop documentation for BGPv2.

@harsimran-pabla Is this change relevant to Cilium v1.16 or older versions? (i.e.: should it be backported?)

/test

@harsimran-pabla Is this change relevant to Cilium v1.16 or older versions? (i.e.: should it be backported?)

Hi @lambdanis , yes it should be backported to 1.16. Will add the label.


Rename src/attention/printFolderStructure.js to processDir.js

Project Description
Hey, Contributor!
Please help us select the best next communication platform for the project!
Vote here: https://github.com/tisztamo/Junior/issues/78
Interpret issues freely. Need help or have ideas? Comment right here.
Note: We eat our own dog food. We use Junior to work on Junior. Please, do not turn off the audit feature!
Keep it friendly, that's our vibe.
Happy contributing!

@tisztamo , I want contribute on this issue, can you please assign to me ?

Hello @VishalD51, if you encounter any bugs, feel free to ask for assistance.


Docker Build Cloud

Adds support for using Docker Build Cloud to build images
Details:

See config.md for needed config variables
Can be used alongside a remote private registry or not
Ideally would use different logins for docker cloud build and registry

Watch out:

.env changes

Testing:

[x] covered by automated tests
Tested locally without k8s using DOCKER_CLOUD_REGISTRY=https://index.docker.io/v1/, a personal access token, and a temporary private Docker Hub repo I created
Tested locally using kind
Tested on staging k8s

$ docker container ls
CONTAINER ID   IMAGE                                                                                COMMAND                  CREATED          STATUS          PORTS     NAMES
1a8b5ba64e31   sjawhar/tmp:v0.1agentimage--headless-human--cd44c4b--advent_of_code--5c49bdf--4148754279--server   "/bin/bash -l -c 'ec…"   11 minutes ago   Up 11 minutes             v0run--1333797380--server

I think this is good to go. We can merge while still using Depot and continuing our tests, though I think the tests on staging were enough.

I think this is good to go, too.


Party Presence is not consistent for the Leader

Description
When you create a party using CreateParty you receive a Party structure. You are listed as the Leader but you don't show up in the Presences field. You (the leader) do get a Join in the ReceivedPartyPresence event.
When you join a party using JoinParty you get a ReceivedParty event. Again, the Leader is specified correctly but again the Leader is not in the Presences field. You (the joiner) do get a Join in the ReceivedPartyPresence event, but the leader is not in that Presences field.
Expected Result
The leader should be in Presences
Context

[X ] Unity

Your Environment

Nakama: 3.3.0

I'm seeing the leader in the presences list after a second user joins, but not the joining user.
        [Fact]
        public async Task LeaderShouldBeInPresences()
        {
            var session1 = await _client.AuthenticateCustomAsync($"{Guid.NewGuid()}");
            var session2 = await _client.AuthenticateCustomAsync($"{Guid.NewGuid()}");

            var socket1 = Nakama.Socket.From(_client);
            var socket2 = Nakama.Socket.From(_client);

            await socket1.ConnectAsync(session1);
            await socket2.ConnectAsync(session2);

            var party = await socket1.CreatePartyAsync(true, 1);

            var joinedTcs = new TaskCompletionSource<IPartyPresenceEvent>();
            socket2.ReceivedPartyPresence += (evt) => {
                joinedTcs.SetResult(evt);
            };

            await socket2.JoinPartyAsync(party.Id);

            var presenceEvt = await joinedTcs.Task;

            Assert.Equal(2, presenceEvt.Joins.Count());
            Assert.True(presenceEvt.Joins.Any(join => join.UserId == session1.UserId));

            // give time for all presence events to fire
            await Task.Delay(1000);

            await socket1.CloseAsync();
            await socket2.CloseAsync();
        }

{"level":"info","ts":"2021-05-21T12:14:18.869Z","caller":"server/session_ws.go:81","msg":"New WebSocket session connected","uid":"88399b40-7cca-497b-943c-3d812c214aca","sid":"116f363b-ba2e-11eb-ae3a-7106fdcb5b46","format":0}

{"level":"info","ts":"2021-05-21T12:14:18.878Z","caller":"server/session_ws.go:81","msg":"New WebSocket session connected","uid":"80f583b0-ec3a-40cf-a189-e2454181cc7b","sid":"11708e57-ba2e-11eb-ae3a-7106fdcb5b46","format":0}

{"level":"debug","ts":"2021-05-21T12:14:18.889Z","caller":"server/pipeline.go:66","msg":"Received *rtapi.Envelope_PartyCreate message","uid":"88399b40-7cca-497b-943c-3d812c214aca","sid":"116f363b-ba2e-11eb-ae3a-7106fdcb5b46","cid":"0","message":{"PartyCreate":{"open":true,"max_size":1}}}

{"level":"debug","ts":"2021-05-21T12:14:18.890Z","caller":"server/session_ws.go:391","msg":"Sending *rtapi.Envelope_Party message","uid":"88399b40-7cca-497b-943c-3d812c214aca","sid":"116f363b-ba2e-11eb-ae3a-7106fdcb5b46","envelope":"cid:\"0\" party:{party_id:\"4f4de0ae-8721-4f57-9696-1de342a2d8de.nakama1\" open:true max_size:1 self:{user_id:\"88399b40-7cca-497b-943c-3d812c214aca\" session_id:\"116f363b-ba2e-11eb-ae3a-7106fdcb5b46\" username:\"fWiXTBSBtK\"} leader:{user_id:\"88399b40-7cca-497b-943c-3d812c214aca\" session_id:\"116f363b-ba2e-11eb-ae3a-7106fdcb5b46\" username:\"fWiXTBSBtK\"} presences:{user_id:\"88399b40-7cca-497b-943c-3d812c214aca\" session_id:\"116f363b-ba2e-11eb-ae3a-7106fdcb5b46\" username:\"fWiXTBSBtK\"}}"}

{"level":"debug","ts":"2021-05-21T12:14:18.890Z","caller":"server/tracker.go:834","msg":"Processing presence event","joins":1,"leaves":0}

{"level":"debug","ts":"2021-05-21T12:14:18.902Z","caller":"server/pipeline.go:66","msg":"Received *rtapi.Envelope_PartyJoin message","uid":"80f583b0-ec3a-40cf-a189-e2454181cc7b","sid":"11708e57-ba2e-11eb-ae3a-7106fdcb5b46","cid":"1","message":{"PartyJoin":{"party_id":"4f4de0ae-8721-4f57-9696-1de342a2d8de.nakama1"}}}

{"level":"debug","ts":"2021-05-21T12:14:18.902Z","caller":"server/session_ws.go:391","msg":"Sending <nil> message","uid":"80f583b0-ec3a-40cf-a189-e2454181cc7b","sid":"11708e57-ba2e-11eb-ae3a-7106fdcb5b46","envelope":"cid:\"1\""}

{"level":"debug","ts":"2021-05-21T12:14:18.902Z","caller":"server/tracker.go:834","msg":"Processing presence event","joins":1,"leaves":0}

{"level":"debug","ts":"2021-05-21T12:14:19.028Z","caller":"server/session_ws.go:200","msg":"Error reading message from client","uid":"88399b40-7cca-497b-943c-3d812c214aca","sid":"116f363b-ba2e-11eb-ae3a-7106fdcb5b46","error":"websocket: close 1006 (abnormal closure): unexpected EOF"}

{"level":"info","ts":"2021-05-21T12:14:19.029Z","caller":"server/session_ws.go:434","msg":"Cleaning up closed client connection","uid":"88399b40-7cca-497b-943c-3d812c214aca","sid":"116f363b-ba2e-11eb-ae3a-7106fdcb5b46"}

{"level":"info","ts":"2021-05-21T12:14:19.029Z","caller":"server/session_ws.go:442","msg":"Cleaned up closed connection matchmaker","uid":"88399b40-7cca-497b-943c-3d812c214aca","sid":"116f363b-ba2e-11eb-ae3a-7106fdcb5b46"}

{"level":"debug","ts":"2021-05-21T12:14:19.028Z","caller":"server/session_ws.go:200","msg":"Error reading message from client","uid":"80f583b0-ec3a-40cf-a189-e2454181cc7b","sid":"11708e57-ba2e-11eb-ae3a-7106fdcb5b46","error":"websocket: close 1006 (abnormal closure): unexpected EOF"}

{"level":"info","ts":"2021-05-21T12:14:19.029Z","caller":"server/session_ws.go:434","msg":"Cleaning up closed client connection","uid":"80f583b0-ec3a-40cf-a189-e2454181cc7b","sid":"11708e57-ba2e-11eb-ae3a-7106fdcb5b46"}

{"level":"info","ts":"2021-05-21T12:14:19.029Z","caller":"server/session_ws.go:442","msg":"Cleaned up closed connection matchmaker","uid":"80f583b0-ec3a-40cf-a189-e2454181cc7b","sid":"11708e57-ba2e-11eb-ae3a-7106fdcb5b46"}

{"level":"info","ts":"2021-05-21T12:14:19.029Z","caller":"server/session_ws.go:446","msg":"Cleaned up closed connection tracker","uid":"80f583b0-ec3a-40cf-a189-e2454181cc7b","sid":"11708e57-ba2e-11eb-ae3a-7106fdcb5b46"}

{"level":"info","ts":"2021-05-21T12:14:19.029Z","caller":"server/session_ws.go:450","msg":"Cleaned up closed connection status registry","uid":"80f583b0-ec3a-40cf-a189-e2454181cc7b","sid":"11708e57-ba2e-11eb-ae3a-7106fdcb5b46"}

{"level":"info","ts":"2021-05-21T12:14:19.029Z","caller":"server/session_ws.go:454","msg":"Cleaned up closed connection session registry","uid":"80f583b0-ec3a-40cf-a189-e2454181cc7b","sid":"11708e57-ba2e-11eb-ae3a-7106fdcb5b46"}

{"level":"info","ts":"2021-05-21T12:14:19.029Z","caller":"server/session_ws.go:471","msg":"Closed client connection","uid":"80f583b0-ec3a-40cf-a189-e2454181cc7b","sid":"11708e57-ba2e-11eb-ae3a-7106fdcb5b46"}

{"level":"info","ts":"2021-05-21T12:14:19.029Z","caller":"server/session_ws.go:446","msg":"Cleaned up closed connection tracker","uid":"88399b40-7cca-497b-943c-3d812c214aca","sid":"116f363b-ba2e-11eb-ae3a-7106fdcb5b46"}

{"level":"info","ts":"2021-05-21T12:14:19.030Z","caller":"server/session_ws.go:450","msg":"Cleaned up closed connection status registry","uid":"88399b40-7cca-497b-943c-3d812c214aca","sid":"116f363b-ba2e-11eb-ae3a-7106fdcb5b46"}

{"level":"info","ts":"2021-05-21T12:14:19.030Z","caller":"server/session_ws.go:454","msg":"Cleaned up closed connection session registry","uid":"88399b40-7cca-497b-943c-3d812c214aca","sid":"116f363b-ba2e-11eb-ae3a-7106fdcb5b46"}

{"level":"debug","ts":"2021-05-21T12:14:19.030Z","caller":"server/tracker.go:834","msg":"Processing presence event","joins":0,"leaves":1}

{"level":"debug","ts":"2021-05-21T12:14:19.030Z","caller":"server/tracker.go:834","msg":"Processing presence event","joins":0,"leaves":1}

{"level":"info","ts":"2021-05-21T12:14:19.031Z","caller":"server/session_ws.go:471","msg":"Closed client connection","uid":"88399b40-7cca-497b-943c-3d812c214aca","sid":"116f363b-ba2e-11eb-ae3a-7106fdcb5b46"}

@Meneleus I've confirmed this is a decoding issue on the dotnet client. Thank you.


fix(xhr-utils): forward approved preflight headers to request

fixes #1799

Admittedly the test could be far more generalized for CORS scenarios, but at least its a start 😄

Thanks so much for this work! I've rebased it on top of the latest changes and will try to get it merged soon; sorry for my absence.
Note that ideally we'd be testing this with web-platform-tests, but our infrastructure right now doesn't allow writing to-upstream WPTs that need server-side support, so this style of test is the best we can do right now. :+1:

:+1:


Reorganize build scripts, part 1

Installers are now reorganized to a package. #5722

pfnCI, test this please.

Jenkins CI test (for commit c047dde72c096a4f23cd9ffe142e3a184dacc4f5, target branch master) succeeded!

/test


[Enhancement]: Improving the UI for footer

Is your enhancement request related to a problem? Please describe.➕
Game enhancement
The footer of GameZone is looking quite simple.
It's not that eye catchy.
Describe the solution you'd like
Solution steps

I would like to improve the footer by adding the hover effects on social media icons in the footer.
I would like to add the waves to make it look more aesthetic.

Colours would be aligned according to the theme of GameZone.

Thank you for raising an issue 💗
You can self assign the issue by commenting /assign in comment 😀
Make sure you follow CODE OF CONDUCT and CONTRIBUTING GUIDELINES 🚀
Don’t Forget to ⭐ our GameZone🎮
Make sure you join our Discord🕹

Hey @dishamodi0910 ! Thank you so much for your raising the issue💗
It’s all yours, you can come anytime again and make some contributions! 🚀
Alone, we can do little, but together we can do so much! 😇


Fixing math example on Linux

The usage of sqrt() requires linking libm on Linux. This adds the appropriate CMake entry to allow for this.
Checklist

[X] I've added at least one test that validates that my change is working, if appropriate
[X] I've followed the code style of the rest of the project
[X] I've read the Contribution Guidelines
[X] I've updated the documentation if necessary

@swift-ci Please test

@swift-ci Please test

@BradLarson Thanks for these fixes! I’ll work on getting a CMake build added as part of the CI.

The one to thank for these is actually @compnerd. He walked me through the fixes as part of related work on a separate PR that I was encountering CMake issues with.
We've been using ArgumentParser in other existing targets there, but I realized none were set up for CMake yet. This was the first.


Spinner dialog not showing  in IOS !

hi i am using inAppBrowser in my app.For loading in InAppBrowser i am using
// Set spinner dialog fixed (Android, iOS)
window.plugins.spinnerDialog.show(null, null, true);
It is working great in android but on ios it is not showing.
Any Idea how to fix this or am i doing something wrong?

Hi @varunnayyar,
It's just a guess, but I think inAppBrowser's view loads over Cordova view. Despite the spinnerDialog is on, inAppBrowser hides it. A solution can be to copy iOS source of spinnerDialog to inAppBrowser and display the spinnerDialog on that view.

ok ,thanks for ur reply, i ll check.

this fork has this fix


Remove account.py

Also removes the PyNaCl dependency, as account.py was the last place using it

Is there a different way to do signmessage for Solana in python?

keypair = Keypair.from_secret_key(secret_key)
signature = keypair.sign(message)
signature_hex = bytes(signature).hex()


duplicateCheck?

I can't seem to make duplicateCheck work in version 3.0.
Moreover the prop is not documented even though there is an event duplicate-file(file) that says:

Fired when duplicateCheck is enabled.

I'm certainly doing something wrong but I can't get it.
Any clue

Hi @FredericLatour
Hmm I'm not sure where that event came from sorry. @AlexanderYW do you have any recollection? I've had a quick look in the dropzone.js source code and can't see any reference to it so I think it hsould probably be removed.

hi @rowanwins
The idea was to prevent adding multiple time the same file ... which make sense if you ask me.
see: https://github.com/rowanwins/vue-dropzone/pull/130

Ahhhh hadn't seen that sorry. It looks like the check is still in our source code, we're just missing the property. Feel free to send through a pull request otherwise I'll try and get it done sometime over the weekend.

@rowanwins
I must admit I'm somewhat of a rookie regarding PR stuff ...
Is the overall process like this ?:

make a fork,
made the necessary modification
push the change to the origin repo

I'm quite new to vue:

do I just need to add duplicateCheck in the props section with a default false value ?

Yep thats pretty much it @FredericLatour , although bonus marks if you update the Props page of the documentation :)
Dont worry you can't break anything and the best way to learn is giving it a go with something nice and easy.
Just sing out if you get stuck

@rowanwins
I forked the repo, made the modif (prop+doc) ...
Now if I would like to make a test before submitting a PR what would be the best approach?
thanks

We currently don't have a test page as such (although it would be handy!)
Probably the simplest thing is to just tweak one of the docs pages quickly to test it and then revert the modification before the pull request.

Was going to submit a PR but then decided to make a test within my app by copying the minified js directly (not the more elegant approach I have to admit :)) and it doesn't work ...
As soon as I add a file it is immediately removed ..;
I'm going to investigate what's going on...

@rowanwins
My assumption is that the file is already added when "addedfile" is triggered (the dropzone documentation is not clear about that).
That would explain why the file is removed immediately even though it's not duplicate.
What do you think?
this.dropzone.on('addedfile', function(file) {
      if (vm.duplicateCheck) {
        if (this.files.length) {
          this.files.forEach(function(dzfile) {
            if (dzfile.name === file.name) {
              this.removeFile(file)
              vm.$emit('duplicate-file', file)
            }
          }, this)
        }
      }
      vm.$emit('vdropzone-file-added', file)
      if (vm.isS3 && vm.wasQueueAutoProcess) {
        vm.getSignedAndUploadToS3(file);
      }
    })

@rowanwins @FredericLatour  In previous version one of the contributor added this feature to component. I've it's local copy which looks like below. We have to add this to current version.

Prop
duplicateCheck:{
    type: Boolean,
    default: false
},

Event addedfile
this.dropzone.on('addedfile', function (file) {
  /**
  * If Duplicate Check enabled remove duplicate file and emit the event.
  */
  if (vm.duplicateCheck) {
    if (this.files.length) {
        var _i, _len;
        for (_i = 0, _len = this.files.length; _i < _len - 1; _i++) {
            if (this.files[_i].name === file.name) {
                this.removeFile(file);
                vm.$emit('duplicate-file', file)
            }
        }
    }
  }
  vm.$emit('vdropzone-file-added', file)
});

@rowanwins & @vrajroham
In the end, I don't think this is necessarily a good idea to add the feature directly. When a file is removed, there will be a round-trip to the server and some unwanted events tiggered (at least as far as I'm concerned).
I ended up by using the accept function option that seems the most appropriate fit in order to discard duplicate files:
  data() {
    return {
      dropzoneOptions: {
        autoProcessQueue: false,
        acceptedFiles: ".img",
        accept: this.checkDupl,

And my checkDupl() function
  methods: {
    checkDupl(file, done) {
      let v:any = this.$refs.myVueDropzone;
      let files = v.getAcceptedFiles()
      let item = files.find( el => el.upload.filename === file.name )
      if (item) {
        done("duplicate")
      } else {
        done()
      }
    },

It's not that having this feature integrated would not be interesting but because dropzone.js doesn't have this feature, it can be tricky to handle it properly in a wrapper/interface.
What do you think?

Hi @FredericLatour
Thanks for investigating, I'll take a look and see if I can work out what's going on.

I have the same problem with the duplicate-file, is not working but not only that, either the accepted files type, even so when try to force by ref.setOption. By the way, I added the settings as a computed props, because of the values that are read dynamically instead of hard coding. Anyway all those features were working good in the previous version but seems broken on 3.0

thanks @FredericLatour i using v:any has 1 error. i change let v= = this.$refs.myVueDropzone;
thanks you !

@thandonguocmo2020 Indeed v:any is a typescript construct. I'm going to add a comment in my post to clarify. Thanks.

Hey Folks,
Duplicate file check feature is back  🎉 .
Please note

The event name is changed from ~duplicate-file~ to vdropzone-duplicate-file.
File is marked as duplicate if, added file matches name, size and lastModifiedDate with existing files and that file is not added to dropzone preview.
To enable this feature you need to set duplicateCheck prop on dropzone to true, by default it is false.
Reference - Props, Demo, Events, V3.4.0

Happy Coding  :tada:


UnknownError: Exception encountered when calling layer 'norm_layer' (type Lambda)

    198 """Check if face2_rgb_image matches the person in the image at face1_path."""
    199 # DeepFace usage
--> 200 verification = DeepFace.verify(img1_path=face1_path, img2_path=face2_rgb_image,
    201                                enforce_detection=False)
    202 return verification['verified']

File ~/deepface/deepface/DeepFace.py:114, in verify(img1_path, img2_path, model_name, detector_backend, distance_metric, enforce_detection, align, normalization)
     59 def verify(
     60     img1_path: Union[str, np.ndarray],
     61     img2_path: Union[str, np.ndarray],
   (...)
     67     normalization: str = "base",
     68 ) -> Dict[str, Any]:
     69     """
     70     This function verifies an image pair is same person or different persons. In the background,
     71     verification function represents facial images as vectors and then calculates the similarity
   (...)
    111
    112     """
--> 114     return verification.verify(
    115         img1_path=img1_path,
    116         img2_path=img2_path,
    117         model_name=model_name,
    118         detector_backend=detector_backend,
    119         distance_metric=distance_metric,
    120         enforce_detection=enforce_detection,
    121         align=align,
    122         normalization=normalization,
    123     )

File ~/deepface/deepface/modules/verification.py:97, in verify(img1_path, img2_path, model_name, detector_backend, distance_metric, enforce_detection, align, normalization)
     95 for img1_content, img1_region, _ in img1_objs:
     96     for img2_content, img2_region, _ in img2_objs:
---> 97         img1_embedding_obj = representation.represent(
     98             img_path=img1_content,
     99             model_name=model_name,
    100             enforce_detection=enforce_detection,
    101             detector_backend="skip",
    102             align=align,
    103             normalization=normalization,
    104         )
    106         img2_embedding_obj = representation.represent(
    107             img_path=img2_content,
    108             model_name=model_name,
   (...)
    112             normalization=normalization,
    113         )
    115         img1_representation = img1_embedding_obj[0]["embedding"]

File ~/deepface/deepface/modules/representation.py:115, in represent(img_path, model_name, enforce_detection, detector_backend, align, normalization)
    110 # represent
    111 # if "keras" in str(type(model)):
    112 if isinstance(model, Model):
    113     # model.predict causes memory issue when it is called in a for loop
    114     # embedding = model.predict(img, verbose=0)[0].tolist()
--> 115     embedding = model(img, training=False).numpy()[0].tolist()
    116     # if you still get verbose logging. try call
    117     # - `tf.keras.utils.disable_interactive_logging()`
    118     # in your main program
    119 else:
    120     # SFace and Dlib are not keras models and no verbose arguments
    121     embedding = model.predict(img)[0].tolist()

File ~/anaconda3/envs/companioncore/lib/python3.9/site-packages/keras/src/utils/traceback_utils.py:70, in filter_traceback.<locals>.error_handler(*args, **kwargs)
     67     filtered_tb = _process_traceback_frames(e.__traceback__)
     68     # To get the full stack trace, call:
     69     # `tf.debugging.disable_traceback_filtering()`
---> 70     raise e.with_traceback(filtered_tb) from None
     71 finally:
     72     del filtered_tb

File ~/deepface/deepface/basemodels/VGGFace.py:114, in loadModel.<locals>.<lambda>(x)
    112 base_model_output = Sequential()
    113 base_model_output = Flatten()(model.layers[-5].output)
--> 114 base_model_output = Lambda(lambda x: K.l2_normalize(x, axis=1), name="norm_layer")(
    115     base_model_output
    116 )
    117 vgg_face_descriptor = Model(inputs=model.input, outputs=base_model_output)
    119 return vgg_face_descriptor

UnknownError: Exception encountered when calling layer 'norm_layer' (type Lambda).

{{function_node __wrapped__Rsqrt_device_/job:localhost/replica:0/task:0/device:GPU:0}} JIT compilation failed. [Op:Rsqrt] name:

Call arguments received by layer 'norm_layer' (type Lambda):
  • inputs=tf.Tensor(shape=(1, 4096), dtype=float32)
  • mask=None
  • training=False

Getting this issue. I referenced other issue tickets on this repo, and tried to implement the following
tf.keras.backend.set_image_data_format('channels_last')

But this did not help at all. Keras, tensorflow, and deepface package details are below:
Name: keras
Version: 2.15.0
Summary: Deep learning for humans.
Home-page: https://keras.io/
Author: Keras team
Author-email<EMAIL_ADDRESS>License: Apache 2.0
Location: /home/blabs/anaconda3/envs/companioncore/lib/python3.9/site-packages
Requires:
Required-by: deepface, mtcnn, tensorflow
---
Name: tensorflow
Version: 2.15.0.post1
Summary: TensorFlow is an open source machine learning framework for everyone.
 I am using 2.9.0 for both, and it is fine.
BTW, I plan to get rid of the normalization layer of VGG-Face model (default model of deepface) in the next release. Meanwhile, you can try to use another model such as Facenet.
verification = DeepFace.verify(img1_path=face1_path, img2_path=face2_rgb_image,
                                model_name="Facenet", enforce_detection=False)

Meanwhil, ChatGPT recommends to disable JIT compilation
import os
os.environ['TF_XLA_FLAGS'] = '--tf_xla_enable_xla_devices'

Additionally, disabling GPU will help you in the worst case
import os
os.environ["CUDA_VISIBLE_DEVICES"] = "-1"


Bouncy castle upgrade

Seems to resolve JDK9 modules related warnings
https://circleci.com/gh/square/okhttp/3973
> Task :mockwebserver:test
WARNING: An illegal reflective access operation has occurred
WARNING: Illegal reflective access by org.bouncycastle.jcajce.provider.drbg.DRBG (file:/root/.gradle/caches/modules-2/files-2.1/org.bouncycastle/bcprov-jdk15on/1.60/bd47ad3bd14b8e82595c7adaa143501e60842a84/bcprov-jdk15on-1.60.jar) to constructor sun.security.provider.Sun()
WARNING: Please consider reporting this to the maintainers of org.bouncycastle.jcajce.provider.drbg.DRBG
WARNING: Use --illegal-access=warn to enable warnings of further illegal reflective access operations
WARNING: All illegal access operations will be denied in a future release

Release notes look quite useful and relevant, n.b. I'm aware our usage is mostly related to certificate utils.  But we install as a provider also.
https://www.bouncycastle.org/releasenotes.html
Builders for X509 certificates and CRLs now support replace and remove extension methods.
BCJSSE: Provider now configurable using security config under Java 11 and later.
BCJSSE: SSLSocket implementations store passed-in 'host' before connecting.
BCJSSE: setEnabledCipherSuites ignores unsupported cipher suites.
CertificateFactory now enforces presence of PEM headers when required.
BCJSSE: Improved ALPN support, including selectors from Java 9.
BCJSSE: Added support for ALPN selectors (including in BC extension API for earlier JDKs).
BCJSSE: Support 'SSL' algorithm for SSLContext (alias for 'TLS').
TLS: Tolerate unrecognized hash algorithms.
TLS: Tolerate unrecognized SNI types.
BCJSSE: Added support for ALPN.
BCJSSE: Various changes for improved compatibility with SunJSSE.
BCJSSE: Provide default extended key/trust managers.
TLS: Added support for TLS 1.2 features from RFC 8446.


[Vscode extension] custom component  shouldn't start with .

just for my understanding as I do not know how components work, why can't you just name them without dot in the config?

只是出于我的理解，因为我不知道组件的工作原理，为什么您不能在配置中不加点命名呢？

I just did the same thing as tailwind
https://tailwindcss.com/docs/plugins#adding-components

Ok I understand!
If you have an Idea how to fix, a PR would be highly welcome.


Entering Players into the Tracker

Before anything else can happen, the admin must enter players who will participate in ping pong matches into the tracker. They'll need to enter name at the very least.

[ ] Create Player Object and determine properties
[ ] Create DB Migration and Update Database


Consider migrating back to uv pip install

Just lost twoish hours of my life because uv pip sync removes any dependency not explicitly listed, including pip itself.

uv pip compile is 💯
uv pip sync is 👎  - I could never get this feature to work with pip-tools either.
👍  for dumping sync.


Update to test disting

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

Comparison is base (632c2a8) 100.00% compared to head (8f151cb) 100.00%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff            @@
##              main        #8   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            2         2
  Lines            4         4
=========================================
  Hits             4         4

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


2e Character sheet alignment with Modiphius version

Addition of Acclaim/Reprimand
Addition of pastime
Addition of career path
Addition of career events

Whitespace made consistent, ids removed from the acclaim and reprimand lines as they aren't called from anywhere, just for users to see on the sheet. Thanks for spotting that!


how to use proper formatting option in the case of skipping unavailable video from playlist

Checklist

[x] I'm asking a question
[x] I've looked through the README and FAQ for similar questions
[x] I've searched the bugtracker for similar questions including closed ones

Question

when a video is unavailable in a playlist the naming format that I use is look like that, the ordering numbers are skipping the number of unavailable video for example in this case it skipped number 3 and other numbers how can i fix that. I use this formatting style
-o "~/Users/apachee/%(playlist)s/%(chapter)s/%(playlist_index)s-%(title)s.%(ext)s"

Fix what? This is correct and expected behavior.

I mean the order of number I want it to be 1,2,3,4,5,6 but it is 1,2,4,6,8,11


Attribute modifiers

The following rules exists for modifers in components:

All unused attributes are applied to the root element
Attributes override existing attributes on the root element
It's possible to 'merge' attributes with the following syntax: class="existing-class-name {{ class }}"

It works but is not very explicit. We should support some modifiers both when defining attributes inside components and when using components.
Inside component definitions:
<div :class="existing-class-name" !id="my-component">

</div>

The : modifier makes the attribute 'mergable'. That means the class attribute will automatically get merged with any classes passed in when using the component.
The ! modifier makes the attribute 'non-overridable'. If passing in an 'id' it will cause a compilation error unless the attribute is explicitly used somewhere else in the component definition.
When using components it's also possible to use the ! modifier. This will make it so the attributes overrides an attribute in the definition markes as mergable

After thinking about this some more I have some more thoughts. It would be good to avoid using : because it clashes with both Alpine and Vue. Additionally I've thought about some more features that would make attributes more useful. I think my new thoughts are something like this:
[name]="value": [] can be used to set an attribute with a dynamic name where name is a variable
[...$record]: ... can be used to spread all items in a record into attributes
<Modal #open="true" />: # can be used to 'bind' data i.e. open will be the boolean true instead of the string "true"
!class="text-red-500": ! can be used to mark an attribute as non-overridable
...class="text-red-500": ... can be used to mark an attribute as 'mergable'
class="text-red-500": By default a normal attribute will be overwritten if an attribute with the same name is passed in


[icon] Add more variants.

launch jenkins icon

:green_circle: balfrin/integration_test/spack_info_icon.log
:green_circle: balfrin/integration_test/spack_spec_icon.log
:green_circle: balfrin/integration_test/spack_spec_icon_serialization=create_claw=std.log
:green_circle<EMAIL_ADDRESS>:green_circle<EMAIL_ADDRESS>:green_circle: balfrin/unit_test/summary.log

:green_circle: daint/integration_test/spack_info_icon.log
:green_circle: daint/integration_test/spack_spec_icon.log
:green_circle: daint/integration_test/spack_spec_icon_serialization=create_claw=std.log
:green_circle: daint/system_test/config/cscs/spack-envs/daint_cpu_cce.log
:green_circle: daint/system_test/config/cscs/spack-envs/daint_cpu_gcc.log
:green_circle: daint/system_test/config/cscs/spack-envs/daint_cpu_nvhpc.log
:green_circle: daint/system_test/config/cscs/spack-envs/daint_gpu_nvhpc.log
:green_circle: daint/unit_test/summary.log


Bug on Android 7 (I think)

Version of Boardwalk:.   1.2
Version of Android: 7.0
Device model:Huawei p9 lite (HUAWEI VNS-L31)
Screenshot: http://prntscr.com/fsct53

I have same problem

Same, but maybe this project is abandoned. I contacted the dev on Twitter but no response.

Yep, project is dead.

Where have you the information about that the project is dead? GoDzM4TT3O


On HackRf One. examples/rx_threaded.rs does not seem to be deactivating/cleaning up the device on close

Hello Folks,
I have been trying seify on HackRf one the past few days. While running some of your examples in the examples directory i noticed that rx_threaded.rs is not deactivating the active stream. The RX LED on the HackRf is still on even after the program is done executing.
Thanks.

Hmm, could you please try the soapysdr bindings (e.g., the soapy-sdr-stream binary https://github.com/kevinmehall/rust-soapysdr/blob/master/src/bin/soapy-sdr-stream.rs) and see if this behaves similar. I guess that might be an issue of the underlying soapy bindings. Maybe they are missing some function calls when the device of the streamer get dropped.

Hi @bastibl
I have verified this by running soapy-sdr-stream and can confirm that this problem doesn't happen. Where as when i run the rx_threaded in seify. I encounter this issue. It is reproducible.
Thanks.

I see, the RX thread was not terminated but just killed when the main thread received ctrl-c. Can you try with the recent commit?


fix: ensure max_fee_per_blob_gas field handles Some(0) gracefully

Motivation

Closes: https://github.com/alloy-rs/alloy/issues/1371
Solution

Related Foundry PR: https://github.com/foundry-rs/foundry/pull/8963 that depends on this implementation
If max_fee_per_blob_gas is Some(0) or None we set it to a valid value next_block_blob_fee().
PR Checklist

[x] Added Tests
[ ] Added Documentation
[ ] Breaking changes

lgtm, this is effectively a safe guard to prevent user error, right?

It is but it mostly enables you do not define the field and rely on the filler. What happens is that the transaction is serialized as 0 as the field is required. This is then interpreted as Some(0) rather than None that would previously then skip the filler.


swiftlint autocorrect

Description
Swiftlint's autocorrect feature mainly reformatting the code
Motivation and Context

How Has This Been Tested?

Checklist:

[ ] I have submitted a CLA form
[ ] If applicable, I have updated the documentation accordingly.
[ ] If applicable, I have added tests to cover my changes.

@shmuelk @ianpartridge @vadimeisenbergibm this is what happens when you run autocorrect with all the auto correctable rules enabled

It all looks sensible, I support this PR. It doesn't catch two spaces in if  let - we should open a bug against SwiftLint for that?

Also, let's look at https://houndci.com/ for automating this stuff?

I disagree. I don't like what it does with if else.
Here is an example:

   }

   else if name.hasPrefix("HTTP_") {

   } else if name.hasPrefix("HTTP_") {

It is placing the else on the same line as the closing brace of the if.

What is wrong with that? Note that in the original code above other else if statements are on the line of the closing brace. These are our Code Conventions, see https://github.com/raywenderlich/swift-style-guide#spacing, https://github.com/apple/swift-package-manager/blob/1bfda9bbdb963e11c8144c89b6b260106d38be6d/Sources/Utility/PkgConfig.swift#L139 .


[WIP] Swap selectize tags input for ember-power-select+ember-drag-drop

refs https://github.com/TryGhost/Ghost/issues/6458

remove unused {{gh-navigation}} component
swap ember-sortable for ember-drag-drop in navigation UI
add new {{gh-token-input}} that wraps ember-power-select and ember-drag-drop to replicate the current selectize based tags input

TODO:

[x] remove unused {{gh-navigation}} component
[x] update Navigation screen to use ember-drag-drop instead of ember-sortable
[x] extract PSM tags input into separate component to reduce PSM complexity
[x] add new {{gh-token-input}} component

[x] implement basic ember-power-select input
[x] allow token creation
[x] handle token removal so new items can be properly destroyed
[x] add drag-n-drop re-ordering of tokens
[x] fix broken re-ordering after adding new tokens
[x] allow per-token customisation

[x] different background color for primary tags
[x] token titles - show primary tag or internal on hover
[ ] minor UX tweaks

[x] matching should be case-insensitive
[x] ensure input is empty after selection
[x] only select create option when no other matches are found
[x] Esc should close dropdown without closing the PSM
[x] if Tab is pressed when an option is highlighted then select it, default to normal Tab behaviour if the search field is empty
[ ] use vertical-collection to avoid perf issues with 1000s of tags
~allow Shift+Tab to tab out of the search input~ (added but then removed as the implementation interfered too much with normal usage)
~allow input to move between tokens using cursor~ can be added later if there's a pressing need

[x] remove selectize
[x] remove jquery-ui
[ ] tests

[ ] check basic tags input functionality in editor acceptance test
[x] gh-psm-tags-input integration tests

[x] fork and submit upstream patch to ember-drag-drop for Node 4.x compatibility

Coverage decreased (-0.05%) to 72.615% when pulling 51cdfa05dfb485d51e10754af0311c59ad6978b4 on kevinansfield:remove-selectize into 8d8bdf727a75f92dbd00ce97334d3a6fd288a67a on TryGhost:master.

Coverage decreased (-1.05%) to 71.764% when pulling 6203073cab2b40cdbc5ab0c1a9e2cb8c6280994f on kevinansfield:remove-selectize into 4eff42bdacaf29b483f6f7d14516c7e9a6e930da on TryGhost:master.

Coverage decreased (-1.1%) to 71.716% when pulling 87112fb8cbc5e0c66342f85942aa296788551d4f on kevinansfield:remove-selectize into 4eff42bdacaf29b483f6f7d14516c7e9a6e930da on TryGhost:master.

Coverage decreased (-1.1%) to 71.716% when pulling eafab70a3688b411798ec544a94b2836bdf8abdd on kevinansfield:remove-selectize into 238983a5dfba25e3ea19704e28c8bf6ae07a20f9 on TryGhost:master.

Coverage increased (+0.1%) to 72.946% when pulling 065a77a93489a5c3bc84fd4cb683207da75b5401 on kevinansfield:remove-selectize into 238983a5dfba25e3ea19704e28c8bf6ae07a20f9 on TryGhost:master.

Coverage increased (+0.1%) to 72.946% when pulling da16642eb52c7e4916a3556c5fd9c5757127f56a on kevinansfield:remove-selectize into 238983a5dfba25e3ea19704e28c8bf6ae07a20f9 on TryGhost:master.

@kirrg001 feel free to squash everything when merging, the top part of the PR before TODO is fine to use as the commit message 😄

Will review tomorrow morning :)

@kevinansfield Feel free to self-merge when Travis does bling bling 😬

Coverage increased (+0.1%) to 72.946% when pulling 2a3b04fa9ccedac0032e09858a2a83f45ce4ef1f on kevinansfield:remove-selectize into 238983a5dfba25e3ea19704e28c8bf6ae07a20f9 on TryGhost:master.


release: render manifests using deployer

Use the deployer binary built as release artifact to generate (and add to released artifact) the manifests.
This enables users interested only in released manifests to download them right away, instead of being force to download the deployer binary first.
Closes: https://github.com/k8stopologyawareschedwg/deployer/issues/81
Signed-off-by: Francesco Romani<EMAIL_ADDRESS>

/approve
/lgtm


error when I try run domainjoin-gui

root@PrintServer:  /opt/likewise/bin/domainjoin-gui
/opt/likewise/bin/domainjoin-gui: error while loading shared libraries: libglade-2.0.so.0: cannot open shared object file: No such file or directory
root@PrintServer:

try installing libglade?
apt-get install libglade2-0

after installing "libglade2-0", now I can open GUI, but after try to join I got this message
A sshd config file was at {nowhere}, and a sshd binary file was found at {nowhere}. Exactly one config file and one binary must exist on the system in a standard location. Uninstall any unused copies of ssh.

Seems you do not have a ssh server on your machine (apt-get install openssh-server). If you want pbis then you need to disable it from trying to configure ssh (option: --disable ssh). This can be done from domainjoin-cli.
example command
/opt/pbis/bin/domainjoin-cli join --disable ssh DOMAIN WINADMIN

thanks for quick help you are awesome


Converging AP and AR on Faster RCNN faster_rcnn_resnet50_fpn

Hello,
I'm trying to implement the torchvision official tutorial for finetuning object detection.
I have 8 classes, and about 4000 images that are labeled.
The metrics at the end of each epoch, seems to be converging at a certain point to values lower that 0.5 (between 0.4 and 0.45 for recall and 0.2 to 0.3 for recall).
On tensorboard my results look like this :

Blue curve is a tryout with a learning rate of 0.01 in the SGD optimizer
Pink curve is with a learning rate of 0.001
It looks like in both cases it stops learning very early.
The weirdest part is that when I'm trying to evaluate on the training dataloader instead of the test, to see if it is overfitting, i'm getting low results as well (0.1 for AP and 0.3 for recall).
Does anyone have an idea why I get such low results ?
My hyperparameters (with learning rate 0.01 or 0.001) are the following :
momentum=0.9, weight_decay=0.0005 in the optimizer, batch_size of 16 on the train, 1 on the test.
The dataloader test size is 50 as in the torchvision tutorial, because I noticed increasing the size of the test dataset was giving worse performance.
My model is the following :
model = torchvision.models.detection.fasterrcnn_resnet50_fpn(pretrained=True)
# get features
in_features = model.roi_heads.box_predictor.cls_score.in_features
num_classes = 9
# last head replaced with the new head that will be trained

model.roi_heads.box_predictor = FastRCNNPredictor(in_features, num_classes)

dataset = Custom("CustomDatasetFolder", get_transform(train=True))
dataset_test = Custom("CustomDatasetFolder", get_transform(train=False))

# split the dataset in train and test set
torch.manual_seed(1)
indices = torch.randperm(len(dataset)).tolist()
dataset = torch.utils.data.Subset(dataset, indices[:-50])
dataset_test = torch.utils.data.Subset(dataset_test, indices[-50:])

# define training and validation data loaders
data_loader = torch.utils.data.DataLoader(
    dataset, batch_size=16, shuffle=True, num_workers=4, collate_fn=utils.collate_fn
)
images, labels = next(iter(data_loader))

data_loader_test = torch.utils.data.DataLoader(
    dataset_test,
    batch_size=1,
    shuffle=False,
    num_workers=4,
    collate_fn=utils.collate_fn,
)

device = torch.device("cuda") if torch.cuda.is_available() else torch.device("cpu")
# change to device
# move model to the right device
model.to(device)

# construct an optimizer

params = [p for p in model.parameters() if p.requires_grad]
optimizer = torch.optim.SGD(params, lr=0.001, momentum=0.9, weight_decay=0.0005)

# and a learning rate scheduler which decreases the learning rate by
# 10x every 3 epochs
lr_scheduler = torch.optim.lr_scheduler.StepLR(optimizer, step_size=3, gamma=0.1)

From your training losses, it seems that you might want to change the step_size from the lr_scheduler to a higher value, as you could potentially get better training values.
Try training for more epochs (like 100) and decrease the learning rate every 30 epochs, and see if it increases the performance.

This issue appears to be stale, so I'm closing it. If you think it should remain open, feel free to reopen it.

This issue appears to be stale, so I'm closing it. If you think it should remain open, feel free to reopen it.


What is unit of batch-size and how should I set its value?

Hi!
Following up on the solutions kindly provided, I'm experimenting with the batch-size method. It is very helpful! Now we are able to compute OT problems at least 10 times larger than before (still trying on larger sizes).

A quick question I got here is–What is the unit of batch-size?

It seems a bit counterintuitive. For example, if I'm computing the optimal transport between two distributions X_1 and X_2, I found that if I fix X_1 and increase the size of X_2 I would need to reduce the size of batch_size.
Right now, it seems I would need to do a manual line search for this parameter to find an "optimal feasible solution" for it :) Is there a more straightforward way to set its value?
Thankss!

A quick question I got here is–What is the unit of batch-size?

The unit is the number of elements of X_1/X_2 to evaluate in chunks (think more apt would be to rename batch_size -> chunk_size). For example, setting batch_size=8192, it would evaluate the LSE (or the kernel) matrix-vector product in a batched way, materializing either (batch_size, X_2.shape[0]) or (X_1.shape[0], batch_size) matrices.
There's no option to specify the batch size as a fraction of the input and we're currently looking into having a more robust implementation of a chunked-vmap (see https://github.com/google/jax/issues/11319 or netket's implementation).

As dataset size grows (either in sample count or feature dimension), the dataset itself can quickly become limited by GPU memory as well. In this case, it may be worth also exploring the use of a jax callback to bring (a chunk of) data to the device.

Better batching has been implemented in #588 , it should fix some of the issues as some operations in PointCloud were not batched and now are.


Controller need to be able to receive setpoints

The old controller need to operate on setpoints provided

Done. See master branch


Add source-kafka plugin

The source-kafka plugin includes:

feature to create a kafka source
feature to delete a kafka source
tests: e2e test and unit tests

/test pull-knative-client-contrib-integration-tests

I will close this PR, refactor it and separate into two PRs for easy review.


ChartData2D Component ImportFromSpreadsheet method — "useHeaders" parameter does not work

Describe the bug
If you set the userHeaders parameter as True (as shown in the attached AIA). Nothing happens. No data at all. Should it be reading the headers and setting them as labels for the axes? Or skip over them?
If you set the parameter as False, and the spreadsheet has headers (as shown in the referenced spreadsheet), then Chart throws up an error, as expected. You are trying to make it read strings as data.

Affects

[ ] Designer
[ ] Blocks editor
[x ] Companion
[ ] Compiled apps
[x] Buildserver
[ ] Debugging
[ ] Other... (please describe)

Expected behavior
Should it automatically label the axes in the Chart? Or should it at least skip over the headers?

Steps to reproduce
Load and run the attached AIA. Press the Get Data button to load the data into your app. Then press Show Data button to load the data into the chart.

Spreadsheet_to_Chart_starter_header_test.zip

@bobparks this is very interesting issue.
I was debugging the code and I found that when the useHeaders is true, The Code is expecting the first value of columns in xColumn and yColumn so in your case the xColumn and yColumn should be "winter" and "ice_days" respectively (if useHeaders is true). But if you set useHeaders to false in that case you have to pass D and E (the actual spreadsheet column) as you have done in your aia.
This makes it very inconsistent for students and new students will find it difficult. So we can do either of following

Update the documentation and specify that if useHeaders is true then xColumn and yColumn should be the frst values and not the column index (A-Z). While this is simple, It will make the overall experience in consistent and is not very intutive.
Second option is to make both the behaviour consistent (I would like to work on this issue if we choose to go with this approach). But the downside is that it will make existing projects in-consistent.

What are your thoughts on this @ewpatton sir?

@ewpatton can you please take a look at above comment. Is this an expected behaviour or is it a bug?
I am referring to this part

The Code is expecting the first value of columns in xColumn and yColumn so in your case the xColumn and yColumn should be "winter" and "ice_days" respectively (if useHeaders is true). But if you set useHeaders to false in that case you have to pass D and E (the actual spreadsheet column) as you have done in your aia.

Example

This is what it should be when useHeaders is false

And this is when useHeaders is true
notice I changed the xColumn and yColumn name to actual columns inputted by user in spreadsheet instead of the default A-Z column names

@devarsh-mavani-19 Yes, that's correct. I felt like I had it working at some point but if you want to poke around and see if you can come up with a fix that would be helpful.

Many thanks, @devarsh-mavani-19. I didn't realize the existing code was looking for "winter" or "ice_days". That's why it wasn't working for me. Great catch.
Yes, I think making it consistent using an alphabetical column name is more intuitive.

Many thanks, @devarsh-mavani-19. I didn't realize the existing code was looking for "winter" or "ice_days". That's why it wasn't working for me. Great catch.
Yes, I think making it consistent using an alphabetical column name is more intuitive. And this change will make the block skip the header row when the useHeaders parameter is set to true?

Yes that's correct. But if we try to make behaviour consistent, It might cause issues in existing user projects.

@bobparks Let's plan to discuss this on Thursday. The semantics you're proposing sound more like they'd be in line with a skipHeaders parameter rather than a useHeaders parameter. My vision with useHeaders was to make it easier for people to reference data by the actual labels used in the data (assuming it's columnar) rather than have to fall back to the spreadsheet column lettering.


[Auto-generated] Update dependencies

Update dependencies
Automatically created PR from ci/dependabot-updates.
For more information see the "Dependabot updates" workflow.
To-do

[ ] Check that the diff is sensible, and that tests and builds pass with the new dependency versions.
[ ] Make sure that the PR is squash merged, with a sensible commit message.

Codecov Report

Merging #174 (a13a1aa) into master (62c4d8d) will decrease coverage by 0.09%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #174      +/-   ##
==========================================
- Coverage   80.70%   80.61%   -0.10%
==========================================
  Files          39       39
  Lines        1099     1099
==========================================
- Hits          887      886       -1
- Misses        212      213       +1

Flag
Coverage Δ

linux
80.61% <ø> (ø)

linux-extra_libs
80.61% <ø> (ø)

windows
?

windows-extra_libs
?

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

oteapi/utils/paths.py
88.88% <0.00%> (-5.56%)
: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 62c4d8d...a13a1aa. Read the comment docs.


:sparkles: Create about page

Description
I added an about-page html under /src/features/pages/about-page.html
fixes issue #76
Type of change

[x] New feature (non-breaking change which adds functionality)

Screenshot (optional)

Checklist

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my code
[x] I have commented my code, particularly in hard-to-understand areas
[x] My changes generate no new warnings

Good work, but closing this pr as issue is done, please pick another issue


Add node-vault as an inbuilt library in script runtime

resolves https://github.com/usebruno/bruno/issues/872
Description
Using require('node-vault'); in a script does not work right now. Adding node-vault to libraries fixes it
Contribution Checklist:

[x] The pull request only addresses one issue or adds one feature.
[x] The pull request does not introduce any breaking changes
[x] I have added screenshots or gifs to help explain the change if applicable.
[x] I have read the contribution guidelines.
[x] Create an issue and link to the pull request.

Note: Keeping the PR small and focused helps make it easier to review and merge. If you have multiple changes you want to make, please consider submitting them as separate pull requests.
Publishing to New Package Managers
Please see here for more information.

Thanks for taking care of this @Oryss !


[SPARKR-183] [WIP] Add option to not reuse connection to backend

This is one of the possible solutions to the slow RPC problem on Linux and so I've marked it as WIP. I'd also like to see it this improves Travis test times

Closing this as #140 is merged


make TsigVerify check time after signature per rfc2845bis

The current implementation of TsigVerify is compliant to RFC2845 in that it first checks 'time signed' and then verifies the signature.  But this logic is known to have a security vulnerability as reported in CVE-2017-3142/3143.
The protocol is being revised as draft-ietf-dnsop-rfc2845bis, reversing the order of these checks.  This PR implements the new behavior in a straightforward way.  While the new protocol spec is still an internet draft, it's already in the RFC editor queue, so I believe it's mature enough.
(Added test covers this specific case with some other error conditions from TsigVerify)

Codecov Report

Merging #1135 into master will increase coverage by 0.10%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master    #1135      +/-   ##
==========================================
+ Coverage   55.32%   55.43%   +0.10%
==========================================
  Files          41       41
  Lines       10109    10110       +1
==========================================
+ Hits         5593     5604      +11
+ Misses       3488     3482       -6
+ Partials     1028     1024       -4

Impacted Files
Coverage Δ

tsig.go
54.67% <100.00%> (+6.78%)
:arrow_up:

msg.go
76.53% <0.00%> (-0.69%)
: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 50b4756...3d75d77. Read the comment docs.

/merge
On Sat, 18 Jul 2020, 06:13 Tom Thorogood<EMAIL_ADDRESS>wrote:

@tmthrgd approved this pull request.
LGTM.
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/miekg/dns/pull/1135#pullrequestreview-451029611, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AACWIWZZNVD4ERRPH73U64LR4EONHANCNFSM4O5COLAA
.


Don't track attribute changes

Hi Alex,
After some thought (and some unexpected behaviour that we found in our team), it seems that checking if the model attribute has changed seems quite unnecessary and moreover, tricky to accomplish.
The issue that we found was that the created event didn't fire because when you create a new resource, $model->wasChanged() and $model->isDirty() will both return false (so $this->wasChanged() will be false)

The package already relies on Eloquent's own model event firing logic by simply using $modelClass::{$observedEvent} so I'd argue that checking if the model was changed at all - is quite opinionated & should be left instead to the developers (users of this package) to handle such logic on their end (in their App\AttributeObservers\ModelObserver classes).
This is a breaking change for the current users of the package so, if PR is accepted, I'd suggest a major version change.

@dm-pf Hi Denis,
So to be clear, this quirk only affects the created event?

I'd argue that the reverse is also true. Firing ghost events, for non-existent changes will definitely lead to unexpected behavior. Effectively, such a change would defeat the very purpose of this package.
Take a look at my suggested changes as a non-breaking change alternative

I agree. Although, it's better to receive a ghost event than no event at all.

In a hobby project, sure. But definitely not in mission critical production apps.
If there are no objections to my suggested changes, I'll merge this PR asap

Okay. Thanks for this discourse and your continued contributions!
Looking forward to the next PR


Help handling callback to send data to websocket

I am starting a project that is currently very close to the advanced_server.cpp application and experimenting by expanding from there.
What I am attempting to do is then a client connects a websocket I want to kick off a process that runs independently and occasionally spits out stdout data.  I want the server to send all of this output to the connected websocket until the pipe is closed. (And in the future also go the other way, pipe text from websocket to application.)
So far I have the application being started and binding to a server function on process output.  This is working fine so far.  I am new to beast an am unsure if the memory I am working in is even valid, does the worker move on and I am just working in old memory at this point?  Also I am unsure out to write data into the websocket, if there are better examples please help point me there.  Do I need to create a new buffer to this... or can I re-use 'buffer_' in the example?
Here is the section of code I am working in (assume the rest is advanced_server.cpp)
`
void on_accept(beast::error_code ec)
{
if(ec)
return fail(ec, "accept");
    if (target_ == "/echo") {
    	do_echo_read();
    }
    else if (target_ == "/print") {

    }
    else if (target_ == "/shared_memory") {
    	// start shared memory process
    	childProcess_ = ChildProcess("../../SharedMemoryCounter/Release/SharedMemoryCounter",
    			boost::bind(&websocket_session::on_sharedmem_read, this, _1)
    	);
    }
}

void on_sharedmem_read(char* data)
{
	std::cout << "got callback data: " << data << std::endl;
	std::cout << "target= " << target_ << std::endl;

	// send 'data' through ws_...
}

The argument to 'ChildProcess'  with the boost::bind callback isboost::function<void(char*)>`
I appreciate any help I can get.  Thanks!

It's late for me now but I can help with this tomorrow.
The general form for sending data in beast/asio is to queue it into an output queue and initiate a write if there is not already a write in progress. Once a write has completed, remove the data from the queue and check whether more data has been queued. If so, initiate another write.
In a server app you will generally be continually initiating and handling reads (in beast, this is actually what keeps the websocket running) and writing when there is data to write.

Thank you for the information!
I figured out how to at least send the data along and that part is working well.
`void on_accept(beast::error_code ec)
{
if(ec)
return fail(ec, "accept");
    if (target_ == "/echo") {
    	do_echo_read();
    }
    else if (target_ == "/print") {

    }
    else if (target_ == "/shared_memory") {
    	// start shared memory process
    	childProcess_ = ChildProcess("../../SharedMemoryCounter/Release/SharedMemoryCounter",
    			boost::bind(&websocket_session::on_sharedmem_read, shared_from_this(), _1)
    	);
    }
}

void on_sharedmem_read(char* data)
{
	beast::flat_buffer b;

	beast::ostream(b) << data;

	ws_.write(b.data());
}

`
(not sure why I can't get the full code to display correctly here)
I used shared_from_this() which I believe holds the websocket structure in memory so that it is still available at the time of the callback?
How can I gracefully close the websocket session?  How do I get a callback/check whena websocket is disconnected (ie: refreshing from the client browser)?

How do I get a callback/check whena websocket is disconnected (ie: refreshing from the client browser)?

You have to make sure you always have a call to async_read pending. The call will error out, either with a timeout if you have configured it correctly, or with a socket closure error.

Just want to update so others can follow what I learned.
Like Vinnie and madmongo said: I need to always have an async call pending.  That means after the connection is accepted I call async_read and the callback (after handling the received data) ends up calling async_read again.
My other problem was that I also wanted to aync_write on the same websocket, but the writes were coming from another thread.  So I used net::post to essentially delegate the write to the correct thread.  I lifted how to do this (as well as using a queue to amke sure we receive all the messages) from the chat_server.cpp demo.
Thank you for the help, I have been able to go a long ways after getting these initial building blocks in place.


[Bug Report] When use  in  the slider is dosen't work!

Element UI version
1.4.2
OS/Browsers version
MaxOs/Chrome
Vue version
2.4.2
Reproduction Link
https://jsfiddle.net/akai007/q1mrohw2/1/
Steps to reproduce
click button and then move the slider.
What is Expected?
Can change the slider
What is actually happening?
Only can change it to Min Or Max value

Fixed in #6593