Modify for P4RT-5.2

Modifying P4RT-5.2 Traceroute Packet-out to select 2 ports out of 4 ports which belongs in the same P4RT Node name.
Traffic is also sent and expected on the same selected ports.

Pull Request Functional Test Report for #2316 / d363f34e28833b69ef49df00771d57fafe372f83
Virtual Devices

Device
Test
Test Documentation
Job
Raw Log

Arista cEOS

P4RT-5.2: Traceroute Packetout

Cisco 8000E

P4RT-5.2: Traceroute Packetout

Cisco XRd

P4RT-5.2: Traceroute Packetout

Juniper cPTX

P4RT-5.2: Traceroute Packetout

Juniper ncPTX

P4RT-5.2: Traceroute Packetout

Nokia SR Linux

P4RT-5.2: Traceroute Packetout

Openconfig Lemming

P4RT-5.2: Traceroute Packetout

Hardware Devices

Device
Test
Test Documentation
Raw Log

Arista 7808

P4RT-5.2: Traceroute Packetout

Cisco 8808

P4RT-5.2: Traceroute Packetout

Juniper PTX10008

P4RT-5.2: Traceroute Packetout

Nokia 7250 IXR-10e

P4RT-5.2: Traceroute Packetout

Help

Can you elaborate as to why ports have to be on the same node?

@LimeHat Please find discussion in comments from this PR  https://github.com/openconfig/featureprofiles/pull/2126/#issuecomment-1737878950

Are you referring to the last comment from the PR?

We have observed "The DUT port1 and port2 needs to be in the same device/node for the packetout traffic with submit_to_ingress set to 0x0 or NIL and egress_port_id set to '2101' for traffic to be forwarded, else traffic will be dropped as port2 belongs to a difference device/node for which the current controller has no client session".

If so, it's still uncelar why that should be the case. Port-2 is a regular ip interface destination, where does the p4rt session dependency comes from (is there a reference to the spec)?

Are you referring to the last comment from the PR?

We have observed "The DUT port1 and port2 needs to be in the same device/node for the packetout traffic with submit_to_ingress set to 0x0 or NIL and egress_port_id set to '2101' for traffic to be forwarded, else traffic will be dropped as port2 belongs to a difference device/node for which the current controller has no client session".

If so, it's still unclear why that should be the case. Port-2 is a regular ip interface destination (with an assigned p4rt id), where does the p4rt session dependency comes from (is there a reference to the spec)?

Hi @LimeHat Consider a scenario where dut:port1 belongs to ASIC1 and dut:port2 belongs to ASIC2.
Now from ASIC1, we can send packetOuts only to ports mapped to ASIC1 for egress_port_id scenario right?

Ok, so it's not about the presence of the session as indicated in the quoted comment above?

Now from ASIC1, we can send packetOuts only to ports mapped to ASIC1 for egress_port_id scenario right?

That's not my interpretation of the spec, but I see how a case can be made for this.
Perhaps it would be good to update the README first to define the expected behavior:

packetOut from a controller should be able to reach any sdn port
packetOut from a controller should be able to reach only ports that belong to the same ASIC
both behaviors are acceptable

Ok, so it's not about the presence of the session as indicated in the quoted comment above? More about the permission management.

Now from ASIC1, we can send packetOuts only to ports mapped to ASIC1 for egress_port_id scenario right?

That's not my interpretation of the spec, but I see how a case can be made for this.
Perhaps it would be good to update the README first to define the expected behavior:

packetOut from a controller should be able to reach any sdn port
packetOut from a controller should be able to reach only ports that belong to the same ASIC (for which the controller is a primary)
both behaviors are acceptable

And after we get this clarification the test can be updated to validate the behavior with both positive and negative (if opt#2 is required) scenarios.

It is related to the session as well. A leader client would be attached to a single device_id right ((device_id, role, election_id) being unique)? I think the question is: should this client be able to send packets out through a port mapped to another device_id.
For submit_to_ingress scenario this is definitely true because you are sending the packet out using an ip address. THe confusion is about the egress_port scenario where we are trying to send a packet out through a specific port (that may not be controlled by the current device_id)

@Ankur19 agree, that's pretty much what i meant / that's another way to phrase it.
If it's challenging to clarify this requirement right away, I'd suggest to modify the PR in a much simpler way that will allow both behaviors for the time being: just swap roles of port 1 and port 2 (right now all submit_to_ingress=1 traffic is expected to arrive on port-1, and submit_to_ingress=0 on port-2, while the client is connected to the port-1 ASIC). E.g. client can be connected to port-2 instead.

It is related to the session as well. A leader client would be attached to a single device_id right ((device_id, role, election_id) being unique)? I think the question is: should this client be able to send packets out through a port mapped to another device_id.
For submit_to_ingress scenario this is definitely true because you are sending the packet out using an ip address. THe confusion is about the egress_port scenario where we are trying to send a packet out through a specific port (that may not be controlled by the current device_id)

Thanks @Ankur19,
Script performs port-selection, and ensures that the selected two ports belongs to the same ASIC, for which P4RT session is established. This helps the packet_out to be the successful when 'submit_to_ingress' is not set, where 'egress_port' will point to the port which belongs in the same ASIC, for which P4 controller has mastership.
On the other hand. when 'submit_to_ingress' is set, we know that packet_out will be purely based on the destination IP address ('egress_port' METADATA  will be silently ignored) .

@ArpitJoshi18 will the above suggestion work for you then?
One-liner with a change at L145 if i'm not mistaken:
https://github.com/openconfig/featureprofiles/blob/1d26646337e0fdecf7d0da8bb43cb4e98d230427/feature/experimental/p4rt/otg_tests/traceroute_packetout_test/traceroute_packetout_test.go#L145
Replace "port1" with "port2" here.

https://github.com/openconfig/featureprofiles/blob/1d26646337e0fdecf7d0da8bb43cb4e98d230427/feature/experimental/p4rt/otg_tests/traceroute_packetout_test/traceroute_packetout_test.go#L145
Replace "port1" with "port2" here.

Yes @LimeHat , it works. I will edit it accordingly.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1536

Relevant Lines:
3154

💛  - Coveralls

thanks for the change @ArpitJoshi18 , lgtm


How to get the file pretrained model-resnet_custom_v3.pt

I have search for a while and do not find where the file is, can you provide to me a way to have this file

我已经搜索了一段时间，但没有找到文件的位置，您能否为我提供一种拥有此文件的方法
https://github.com/AUTOMATIC1111/TorchDeepDanbooru/releases/download/v1/model-resnet_custom_v3.pt

thanks so much, this is so good


fix: disable flaky test_can_change_routing_fees

Re #4300

Was the issue with this addressed in https://github.com/fedimint/fedimint/pull/4278?

This change landed with #4278 , I'm trying https://github.com/fedimint/fedimint/pull/4322 to see if the issue is still there.

Still there. See #4350


[BUG] When selecting text across blocks, all content will be selected by default.

Has this bug been raised before?

[X] I have checked "open" AND "closed" issues and this is not a duplicate

Description
Problem: When selecting text across blocks, all content will be selected by default.

Expectation: Keep the text selected by the mouse.

Screenshots
No response
Do you want to work on this issue?
No

Unfortunately currently it's not possible by design.
Native multi-selection does not work if we use list of contenteditable blocks (without root contenteditable)
But it's planned for next major version


Setup env

It needs to have a bunch of tools configured before we start:

[ ] ~bundler Parcel - let's take a step into cutting edge bundler world~
[x] TypeScript - static types safe us time and neurones
[ ] code linting with ESLint and Prettier - let's make less by hands
[ ] Isolated React component development environment Styleguidist to develop components without a headache
[ ] functional tests with Cypress
[ ] ~unit tests with AVA~
[ ] bundle analyser source-map-explorer to keep the size
[ ] prepush/commit hooks via husky
[ ] layout with React
[ ] state management with Fractal
[ ] css-in-js via emotion
[x] dotfiles

.gitignore
.editorconfig

UPD

Parcel is cool but it becomes complicated to setup isolated react component development environment (storybook and styleguidist works well with webpack - no reason to support both builders)
AVA works very limited with TS

To solve it we'll use create-react-app setup.


Building with plugins requires symlink support

Steps to reproduce
Start a new flutter app
Install flutter_localization -> flutter pub add flutter_localization
Try building on windows
Expected results
The app should build and deploy
Actual results
the build fails with this error
Exception: Building with plugins requires symlink support.

Please enable Developer Mode in your system settings. Run
  start ms-settings:developers
to open settings.

Code sample
flutter run -d windows
run this command on a new project with flutter_localization
Screenshots or Video
No response
Logs
Exception: Building with plugins requires symlink support.
Please enable Developer Mode in your system settings. Run
start ms-settings:developers
to open settings.
Flutter Doctor output
Doctor output
flutter doctor
Doctor summary (to see all details, run flutter doctor -v):
[√] Flutter (Channel stable, 3.16.9, on Microsoft Windows [Version 10.0.22621.3007], locale en-US)
[√] Windows Version (Installed version of Windows is version 10 or higher)
[√] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
[√] Chrome - develop for the web
[√] Visual Studio - develop Windows apps (Visual Studio Professional 2022 17.8.5)
[√] Android Studio (version 2023.1)
[√] VS Code, 64-bit edition (version 1.86.1)
[√] Connected device (4 available)
[√] Network resources

• No issues found!

Also, It is important to add that my dev mode is enabled on my windows machine. I also can install apps from any source and this is including loose files

Thanks for the report @Guidemarcus

run this command on a new project with flutter_localization

Can you confirm if this behavior occurs without adding this dependency ?

@darshankawar Yes I can confirm it. When I just have the new flutter ap without any dependencies, I can build and launch without any problem. The issue starts when I add flutter_localization

Thanks for the update @Guidemarcus
My colleague verified this behavior on Windows 11 machine (same build number as yours) using latest stable and he was able to build properly using the package, so, it is strange that you are seeing the error.
Can you provide verbose log by running flutter run -d windows -v to check if it gives any additional logs that might be useful to properly analyze this issue ?
Also, can you try with any other package to confirm if it occurs only with flutter_localizations ?


Support lower Hadoop versions in client-mr

Currently, uniffle use the default Hadoop version of 2.8.5.
When using the ./build_distribution.sh --spark2-profile 'spark2' --spark3-mvn '-Dspark.version=2.4.3' --spark3-profile 'spark3' --spark3-mvn '-Dspark.version=3.1.1' -Dhadoop.version=2.6.0, it will throw exceptions due to some methods and vars not supported in Hadoop 2.6.0.
Some non-compatible params and methods as follows

CallContext, introduced by >= 2.8.0.
MRJobConfig.DEFAULT_SHUFFLE_MERGE_PERCENT introduced by 2.8.0. ticket link
MRApps.getSystemPropertiesToLog introduced by 2.8.0 ticket link

I think we could use the reflection to be compatible with lower hadoop version.

cc @jerqi @frankliee

And I also found the client-mr is not compatible with Hadoop 3.2.2

What's your company's hadoop version?

2.6.0-cdh5.11.0
Hadoop-3.2.2 packaged by bigtop

I think it's ok for me if we need it in our production environment.

OK. I will go ahead.


[BUG] - Non-dissmissable modal can be closed by pressing ESC key

NextUI Version
2.2.9
Describe the bug
In the docs it says that its possible to make a modal non-dismissible by setting the isDismissable prop to false, but it can still be closed by pressing ESC key. It can be reproduced on the docs.
Your Example Website or App
https://nextui.org/docs/components/modal#non-dissmissable
Steps to Reproduce the Bug or Issue

Go to https://nextui.org/docs/components/modal#non-dissmissable
Press "Open Modal" button
Press ESC
The modal closes

Expected behavior
I expected the modal to be non-dismissible
Screenshots or Videos
https://github.com/nextui-org/nextui/assets/74037904/fc84095b-fa53-4e56-8fcd-17797dd50d90
Operating System Version
Windows 10 Home
Browser
Chrome

@alexbalieiro
It seems like there was indeed an oversight in the documentation as you pointed out. In fact, to prevent the modal from closing with the Esc key, you need to set the isKeyboardDismissDisabled property to true.
I've already submitted a PR for this!

Thank you, I didn't know isKeyboardDismissDisabled property, I tried it in my project and it works.


Refactor Discord bot logic to split response messages in paragraphs

On this PR a new logic was implemented to split OpenAI image review response into paragraphs to avoid Discord API error with characters. ({"message": {"code": 50035, "errors": {"content": {"_errors": [{"code": "BASE_TYPE_MAX_LENGTH", "message": "Must be 2000 or fewer in length."}]}}, "message": "Invalid Form Body"}, "status_code": 400})
Now:

The OpenAI image review response generates review key with --PARAGRAPH-- word, so that afterward,
Discord Utils splits every two paragraphs with --PARAGRAPH-- and sends each split to private message, avoiding the Discord API error

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

16 of 16 (100.0%)  changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.03%) to 99.621%

Totals

Change from base Build<PHONE_NUMBER>6:
0.03%

Covered Lines:
3417

Relevant Lines:
3430

💛  - Coveralls


Shows some warnings on marketplace

marketplace dialog box shows warnings.
How can i solve this issue?
Warning: Invalid argument supplied for foreach() in C:\xampp\htdocs\codiad\components\market\class.market.php on line 75
Warning: file_get_contents(): Unable to find the wrapper "https" - did you forget to enable it when you configured PHP? in C:\xampp\htdocs\codiad\components\market\class.market.php on line 170
Warning: file_get_contents(https://raw.github.com/daeks/Codiad-DiffViewer/master/plugin.json): failed to open stream: Invalid argument in C:\xampp\htdocs\codiad\components\market\class.market.php on line 170
Warning: file_get_contents(): Unable to find the wrapper "https" - did you forget to enable it when you configured PHP? in C:\xampp\htdocs\codiad\components\market\class.market.php on line 170

Tested on Apache2.4 and windows 2008 without any issue. So its XAMPP specific.


Update L.Polyline.addLatLng doc

Thoughts? :)

but can be overriden passing another LatLngs group

I think we need to say this more clearly, e.g. "can be overridden by passing a specific ring as a LatLng array (that you can earlier access with getLatLngs)".
Also forgot to remove spliceLatLngs :)

Also, do we need to adjust getLatLngs description? Is it always clear what gets returned?

I think we need to say this more clearly, e.g. "can be overridden by passing a specific ring as a
LatLng array (that you can earlier access with getLatLngs)".

Thanks! My English is terrible…

Also forgot to remove spliceLatLngs

Meeeeeeh :p

do we need to adjust getLatLngs description? Is it always clear what gets returned?

Made a suggestion :)


Jinja session policy filters don't restrict by IP family

Environment

Python version:  3.8
Peering Manager version:  1.4.2

Steps to Reproduce

Add an IPv4 export policy to a BGP Group
Add an IPv6 export policy to a BGP Group
Add a universal export policy to a BGP Group
Create a IPv4 session in the BGP Group
Render router template with session | merge_export_policies

Expected Behavior
You should have an export policy list with just the IPv4 and universal policy
neighbor <IP_ADDRESS> {
  ...
  export [ ipv4-policy universal-policy ];
  ...
}

Observed Behavior
However,  the IPv6 export policy also ends up in the export policy list
neighbor <IP_ADDRESS> {
  ...
  export [ ipv4-policy ipv6-policy universal-policy ];
  ...
}

Submitted PR #450

@mngan This has already been resolved in b5b097842cb8ccf0281bd03003167c0ea240169e. However, the additions you have made do make it look cleaner.

You guys are too fast for me :)

@gmazoyer Is too fast for everyone :)
For the sake of reducing the amount of duplicate code I am still happy to merge this.. will let Guill give the final approve/deny.

Duplicate of #448


Validation of IngressClass existence can break existing uses of the operator

What steps did you take and what happened:
Knative configures two contour instances - one for external and internal traffic. We use ingressClassName as a way to designate between the two instances.
When using this setup with the operator & contour 1.17 it requires us to create an IngressClass object just to satisfy the existence check added by https://github.com/projectcontour/contour-operator/pull/388
What did you expect to happen:
I can create these two contour instances without the need to create an IngressClass object
Anything else you would like to add:
Knative tracking issue: https://github.com/knative-sandbox/net-contour/issues/549
Environment:

Software version: Contour 1.17
Kubernetes version: (use kubectl version): any
Kubernetes installer & version: any
Cloud provider or hardware configuration: GKE/Kind
OS (e.g. from /etc/os-release): any

Worth noting that Contour standalone (no operator) does not require an IngressClass resource to exist (bc in practice it is not used for anything) when configured with the IngressClass name flag so the Operator is adding an additional restriction Contour does not


Prefix not showing in Chat

I dont know what im doing wrong i added the prefix if you see above please help

You will need to set prefix priority as such: prefix..<prefix_name>
highest the priority prefix of the groups that you are in will show.
how support, please join the discord at https://discord.gg/luckperms

You will need to set prefix priority as such: prefix.[priority].[prefix_name]??
Is there a way I can get in a discord call with you for a short tutorial?

https://github.com/lucko/LuckPerms/wiki/Prefixes,-Suffixes-&-Meta#how-does-prefixsuffix-weight-work
you can join the discord, there will be many ppl that can help you with your issue there.

https://github.com/lucko/LuckPerms/wiki/Prefixes,-Suffixes-&-Meta#how-does-prefixsuffix-weight-work
you can join the discord, there will be many ppl that can help you with your issue there.

ok thanks for the quick response!


[web] Remove the FLUTTER_WEB_USE_EXPERIMENTAL_CANVAS_RICH_TEXT flag

After stable is released, and the new rich text implementation is complete, let's not forget to clean up the experimental flag.

Can't find it in the code any more. Looks like it's gone.


AsyncEnumerator.MoveNextAsync extension method considered harmful?

Problem
We have run at least twice into issues by accidentally using the AsyncEnumerator.MoveNextAsync extension method and I have trouble finding a scenario where it would be applicable.

Which subcomponent library (Ix, Async.Ix)?

Async.Ix

Which library version?

6.0.1

What is the problem?

The extension method is very easy to discover and use, it lives in a namespace that is part of the default global usings nowadays.
It speaks to a correctness argument: "Yes, of course I want cancellation when I call that async method".
During code review everything looks perfectly fine.
It doesn't do what you can reasonably expect. The cancellation token is only synchronously checked when entering the method, but never used along the async chain of the underlying async enumerator.

We have run into issues where the async enumerator was created without cancellation token and the expectation was that calling MoveNextAsync would cancel. However, the resulting code would only cancel when the async enumerator made forward progress and re-entered the next MoveNextAsync call. That didn't happen and we were left with a deadlock or memory leak.

What is the expected outcome?

I would vote for removing / deprecating this extension method, but I also realize that this is a breaking change.
I have put in the global namespace another overload of that extension method with the same signature and ObsoleteAttribute, so it is always preferred by the compiler and disables my own usage.

Why does this method exist in the first place, considering IAsyncEnumerable now exists as a first-class citizen in the framework?


RTSP - Features, Stream is not smooth

Hi thanks again for your work on the app to connect to these cameras.  I noticed that there may be an option to enable the busybox ssh shell on the camera and enable the RTSP server locally from the camera?  is this possible?
Also the RTSP stream from the go application is really laggy with the video and appears to crash often.  Any updates to why this is?
Thanks again.

I haven't experienced a laggy stream or packet loss with my camera and network connection (like your MPlayer console suggests). If there is an RTSP Server on the camera it could be enabled remotely, but as far as I can tell there is none (the manufacturers app preview doesn't use RTSP).
Have you tried the RTP stream too? Can you rule out network connectivity issues?

I also planned to reimplement the RTSP part, this wouldn't change much on how the raw video stream is submitted to the player though.

Yeah for some reason the video lags, while in the XVD app the stream is quite smooth and fluid.

Can you provide me with a packet capture? Just use Wireshark to record
everything that's going on during the laggy video transmission.
mlinton<EMAIL_ADDRESS>schrieb am Di., 14. Apr. 2020, 20:06:

Yeah for some reason the video lags, while in the XVD app the stream is
quite smooth and fluid.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/jonas-koeritz/actioncam/issues/8#issuecomment-613594412,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABRPLCYNSJN75BZXLN7L4RTRMSQZ7ANCNFSM4MDOKYEQ
.

No response, please reopen if still interested.


fix fix core in flush_single_memtable when publish_txn

Proposed changes
ctx might be nullptr, which causes the following core
Fix #20572
Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run buildall


Improve declared inputs and outputs of Sign task

Input and output files are now tracked via the task's Signatures. In
addition, the Signatory and SignatureType are now inputs.
Resolves #7381.

@wolfs PTAL!


SDL issues when deleting and re-creating RenderManagers

If an application deletes one RenderManager and constructs another one (at least in Extended mode), SDL_Init() and SDL_Quit() are called multiple times in the application.  We should only call these once per program, not once per construction/destruction.

Fixed in commit ec925c80465236137dd8aaa0a551b9205867e116 which has been pulled into master.


Show curl code

RESTClient is a nice tool, but sometimes I need to show curl code to others, do we have any support?

+1

+1

seems to be a duplicate of https://github.com/chao/RESTClient/issues/110

+1


[HELP] Kafka Connect Timeout

What did you do

How was the cluster created?

k3d cluster create demo --servers 1 -i rancher/k3s:v1.18.6-k3s1 -p "80:80@loadbalancer" -p "443:443e@loadbalancer"

What did you do afterwards?

install helm
helm install -n kafka stable/kafka
kubectl port-forward  pod/kafka-0 9092:9092
create python3 producer script

import datetime
import json
import time
from kafka import KafkaProducer

producer=KafkaProducer(bootstrap_servers='<IP_ADDRESS>:9092')
for i in range(111):
    future = producer.send('test', json.dumps(
        {"method": "get", "step": i, "type": "test", "testName": "kafka",
         "cid": "{0}".format(datetime.datetime.now().strftime('%Y%m%d%H%M%S')),
         "info": "demo{}".format(1)}).encode())
    record_metadata = future.get(timeout=10)
    print(record_metadata, datetime.datetime.now().strftime('%Y-%m-%d %H:%M:%S'))
    time.sleep(3)

What did you expect to happen

The follow is python3 exception:

Traceback (most recent call last):
  File "producer.py", line 12, in <module>
    record_metadata = future.get(timeout=10)
  File "/usr/local/lib/python3.8/site-packages/kafka/producer/future.py", line 61, in get
    raise Errors.KafkaTimeoutError(
kafka.errors.KafkaTimeoutError: KafkaTimeoutError: Timeout after waiting for 10 secs.

The same method worked fine on Centos's Kubernetes cluster. But k3d is not work. How should I do???

Screenshots or terminal output
If applicable, add screenshots or terminal output (code block) to help explain your problem.
Which OS & Architecture

MacOS

Which version of k3d
➜  merger-server git:(dev) ✗ k3d --version
k3d version v3.0.0
k3s version latest (default)

Which version of docker
➜  merger-server git:(dev) ✗ docker  version
Client: Docker Engine - Community
 Version:           19.03.12

https://note.youdao.com/ynoteshare1/index.html?id=19e05adba70a7f44b02aa5c580e7f9c5&type=note

Hi @herozabbix , thanks for opening this issue!
Unfortunately, I cannot reproduce this, as stable/kafka doesn't exist in the official Helm Chart repository.
Since you're using kubectl port-forward (and it doesn't return an error), there's nothing in k3d blocking you.
Did you verify that the Kafka pod is actually up and running correctly?


Unable to disable fog in Better End & Better Nether. (and possibly other modded dimentions)

Describe the bug
It's not possible to disable fog in Better End and Better Nether. I am using Embeddium along with Rubidium Extra (NOT Extras)
Steps to reproduce

Install Connector, Extras, FFAPI
Install Embeddium & Rubidium Extra
Launch game and try to disable fog. (Changing fog modify type from default to mod compat will do nothing)

Logs
No response
Additional context
Here are the reports I made:
Embeddium: https://github.com/embeddedt/embeddium/issues/102
Rubidium Extra: https://github.com/dima-dencep/rubidium-extra/issues/63
I consider this to be quite a significant issue, because being unable to disable fog makes a lot of mods incompatible. Playing with Distant Horizons for example is basically impossible, as real chunks are fully covered in fog or straight up disappear because of chunk fog culling.

not having this issue but the latest versions past beta 27 dont load better nether and and it's libraries at all in servers

What? Better Nether and Better End work perfectly fine for me with beta 29.

crashes every time i try to load it in a server same with version beta 31

Try singleplayer

Are you sure this is caused by Connector and not Rubidium Extra? The linked issue hints that the fog setting doesn't go well with other mods

I was able to get the fog setting working after changing the client config of bclib. To do this, open <minecraft>/config/bclib/client.json, then set customFogRendering and netherThickFog to false like so:

"rendering": {
   "customFogRendering [default: true]": false,
   "netherThickFog [default: true]": false,
   "FogDensity [default: 1.0]": 1.0
 }

Thank you!  I honestly had no clue what was the problem, as I have checked the config files for Better End and Better Nether multiple times, but missed Bclib somehow... bruh. Anyway, thanks!
There is another potential fog issue with Nostalgic Tweaks and Connector by the way. I don't yet know whether it is caused by Connector itself or any mods that run using Connector, but changing fog type to that of pre 1.6.4 will break the rendering of said fog, making it flicker. Might post a report about this soon.


ImageColors: Illegal character in path

Bug

Environment info

Library
Version

react-native-image-colors
1.5.2

when I try to get the image color for this path : "file:///storage/emulated/0/Images/Duck/Screenshot at Apr 29 08-52-35.png"
Error : ImageColors: Illegal character in path

any solution?

I've been having the same error, any solution until now?

Hey guys, I was searching about how I could fix that and I got a solution to us. 🔥
Firstly I created a function to handle the photo URL and replace all spaces code in a string index 40. So the index 40 is a space code in a string. Look the function below:
`
export const URLEncoder = (path = '') => {
if (isAndroid()) {
return String(path).replace(/\s+/g, '%20');
}
return path;
};
`
So with this function you can import where the ImageColors is beeing used and put as parameter the photo url path, look it below:
ImageColors.getColors(URLEncoder(uri), {     fallback: Theme.colors.primary, }) .then(res => {     ... })
In short, I'm using regex to remove all space to %20 in a string. Hope I've helped.

Hey guys, I was searching about how I could fix that and I got a solution to us. 🔥
Firstly I created a function to handle the photo URL and replace all spaces code in a string index 40. So the index 40 is a space code in a string. Look the function below:
export const URLEncoder = (path = '') => {
  if (isAndroid()) {
    return String(path).replace(/\s+/g, '%20');
  }

  return path;
};

So with this function you can import where the ImageColors is beeing used and put as parameter the photo url path, look it below:
ImageColors.getColors(URLEncoder(uri), { fallback: Theme.colors.primary, }) .then(res => { ... })
In short, I'm using regex to remove all space to %20 in a string. Hope I've helped.

I guess you should be able to use the standard encodeURI(uri) function instead of a custom one.


I2C Functions Problem.

Do 'I2C_Transmit' and 'I2C_Receive' functions work correctly?
When I use them, error 14 is returned.

What's your version and which demo is it?

i use the last version released (V2.112) .
try use DHT12:
char data[5];
I2C_ReadMem(I2C2,0xB8,0x00,1,&data,5,500);

i have the same problem here
i try many things and finally I find out that the read memory i2c read and write memory functions has problem in the addresses part the API doesn't accept any addresses It deal only with one address
the API has this problem I hope the SDK to be update and solve this problem

when I read and write directly from eeprom It work well >> if I write all data then try to read it I get only the last data entered to eeprom that mean that no address update


Add more hints for not using void with mapM/mapM_/forM/forM_

Thanks! For the future, note that there's no requirement to explicitly give the eta reduced version - HLint infers that itself. I've now mentioned that in the readme at https://github.com/ndmitchell/hlint/blob/master/README.md#adding-hints.


Fix small typo in languages_base.json

Related Issue

N/A
Summary of Changes

Just fixing small typos: navigaton -> navigation

Closing this as I included this fix in https://github.com/OpenWebAdvocacy/website/pull/168


Clicking deeplink when Firefly is closed doesn't work properly

Issue description
Clicking a deeplink when Firefly is closed does not work. It will start FF, but after you click a profile and pin only startscreen appears.
Clicking a deeplink when Firefly is open does work: it will bring you to transaction screen which is prefilled with address and optionally amount when given
Wallet version
v0.4
System specification
Windows 10
Is it a repeatable issue?
Yes
Steps To reproduce the issue
Explain how the maintainer can reproduce the issue.

Make a deeplink on a webpage for example: iota://atoi1qqzvmh85v36z054guuscakrxvqhnpu0hwf4989g9hjg529sqyaw9z3fhml0/?amount=1000
Be sure Firefly is not started
Click deeplink
Firefly will start and after clicking profile and pin you will see the startscreen not the (prefilled) transaction screen

Expected behaviour
Firefly should start and after selecting profile and typing pin you should see a prefilled transaction screen.
Actual behaviour
Firefly is started, and after clicking profile and typing pin, you see the startscreen not the (prefilled) transaction screen
Errors
No actual errors

This PR has not been merged yet, will appear in a future build https://github.com/iotaledger/firefly/pull/773

Deep linking has been disabled until a later version.


duplicate symbol _JBZoomMode in:

I manually add the classes to the project and import the header into my view controller and i get this error from Xcode.
duplicate symbol _JBZoomMode in:
ld: 2 duplicate symbols for architecture x86_64
clang: error: linker command failed with exit code 1 (use -v to see invocation)

I'm seeing the same thing.

ya i fixed it by recreating the enum JBZoomMode on .m and everything works fine..

@suu10mac  What do yo mean by recreating the enum in .m?

Same here!

@bharath-kamath found it!
You have to take these from the .h:
@property (nonatomic,assign) enum JBZoomMode zoomMode;

and
NS_ENUM(NSInteger, JBZoomMode) {
    JBZoomModeIn,
    JBZoomModeOut,
    JBZoomModeRandom
};

Then paste them in the .m:
#import "JBKenBurnsView.h"

#define enlargeRatio 1.1
#define imageBufer 3

enum JBSourceMode {
    JBSourceModeImages,
    JBSourceModePaths
};

NS_ENUM(NSInteger, JBZoomMode) {
    JBZoomModeIn,
    JBZoomModeOut,
    JBZoomModeRandom
};

// Private interface
@interface JBKenBurnsView ()

@property (nonatomic, strong) NSMutableArray *imagesArray;
@property (nonatomic, strong) NSTimer *nextImageTimer;

@property (nonatomic, assign) CGFloat showImageDuration;
@property (nonatomic, assign) BOOL shouldLoop;
@property (nonatomic, assign) BOOL isLandscape;
@property (nonatomic, assign) enum JBSourceMode sourceMode;
@property (nonatomic,assign) enum JBZoomMode zoomMode;

@end
...HERE GOES THE REST OF THE FILE...

Hope it helps!

@facundomedica Yep, that did help. Thank you!


failed to produce the pytest-coverage.txt file

I am running the CI inside Docker.
Sharing logs for more details:
**`-- Docs: https://docs.pytest.org/en/stable/how-to/capture-warnings.html
--------------------- generated xml file: /tmp/pytest.xml ----------------------
---------- coverage: platform linux, python 3.8.12-final-0 -----------
================== 7 passed, 27 warnings in 181.54s (0:03:01) ==================
/usr/local/lib/python3.8/site-packages/coverage/inorout.py:519: CoverageWarning: Module src was never imported. (module-not-imported)
self.warn(f"Module {pkg} was never imported.", slug="module-not-imported")
/usr/local/lib/python3.8/site-packages/coverage/control.py:794: CoverageWarning: No data was collected. (no-data-collected)
self._warn("No data was collected.", slug="no-data-collected")
/usr/local/lib/python3.8/site-packages/pytest_cov/plugin.py:294: CovReportWarning: Failed to generate report: No data to report.
self.cov_controller.finish()`
`Run MishaKav/pytest-coverage-comment@main
with:
pytest-coverage-path: /tmp/pytest-coverage.txt
junitxml-path: /tmp/pytest.xml
github-token: ***
title: Coverage Report
badge-title: Coverage
hide-badge: false
hide-report: false
hide-comment: false
report-only-changed-files: false
create-new-comment: false
default-branch: main
File read successfully "/tmp/pytest-coverage.txt"
Error: Error: coverage file "/tmp/pytest-coverage.txt" has bad format or wrong data
File read successfully "/tmp/pytest.xml"
File read successfully "/tmp/pytest.xml"
errors: 0
failures: 0
skipped: 0
tests: 7
time: 181.542
File read successfully "/tmp/pytest.xml"
/tmp/pytest-coverage.txt
Create commit comment`**

from what I see, the problem is not the action itself, you fail to generate the right - coverage.txt
from your logs: Failed to generate report: No data to report.
Try to fix your coverage report.


Implement passing path type and index for change address

This allows for full HDWallet style clients. It guarantees that the change will always go into an address that the user has access to. Not necessarily always where the first UTXO came from.

Codecov Report
Patch coverage: 93.10% and project coverage change: -0.02 :warning:

Comparison is base (f1e9f85) 91.39% compared to head (6406502) 91.37%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #38      +/-   ##
==========================================
- Coverage   91.39%   91.37%   -0.02%
==========================================
  Files          26       26
  Lines        1336     1357      +21
==========================================
+ Hits         1221     1240      +19
- Misses        115      117       +2

Flag
Coverage Δ

unittests
91.37% <93.10%> (-0.02%)
:arrow_down:

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

Impacted Files
Coverage Δ

src/transaction/deserialize.c
93.33% <87.50%> (-2.42%)
:arrow_down:

src/transaction/serialize.c
100.00% <100.00%> (ø)

unit-tests/test_tx_parser.c
100.00% <100.00%> (ø)

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Child Process: stream stderr/out to a log file

Allow users of this module to specify a log file that
should receive stdout / stderr from the webpack process

If it crashes, we'll actually have the output

If the webpack proc crashes, notify the watchers
Respond quickly if trying to ask the child a question
when it's dead.

CC @jarstelfox
Closes #12

CR 🌵

CR 🎏 but deploy_block ✋ on considering a suggestion.

un_deploy_block 👍   Good suggestions!

Note we still suggest checking the log cause the output will be in the log of the parent process.

CR 🌵

Not a clue how to test this. Ideas?
Also, this is not showing up on Pulldasher but a pull is deploy blocked on it.

CR 🌵

I'll post some screenshots of the webpack logs that we get now
Syntax error:

Module not found:

Webpack config error:

Messages when builds fail:

The client now sees this for many of the errors:

QA 👍
I've also added this to pulldasher.


Equality tests on floats: add support for math.isclose() and numpy.isclose()

Feature request

When testing the equality of two floating point numbers, it’s good practices to rely on approximations (e.g., abs(a - b) < epsilon) rather than on exact/bit-to-bit comparison.
In Python we can use math.isclose or numpy.isclose to do such tests. These functions are a bit different but both can be equally useful. Unlike the built-in math.isclose, numpy.isclose is not symmetric in a and b – it assumes b is the reference value – so that isclose(a, b) might be different from isclose(b, a).

Floating point comparison is a common enough operation so that Numba can support it (for both math and numpy libraries).
Moreover, all arguments of each function should be supported (relative tolerance, absolute tolerance, and equal_nan if existing) because each use case can have specific parameters.

I could not find an issue about that when searching issues with keywords "isclose", "float comparison", "float equality".

Thanks for the suggestion. I've added this as a feature request. If you'd like to contribute, the high level Numba extension API @overload decorator should get you on the way to implementing this for Numba. Docs are here. A lot of example uses are in 'numba/np/arraymath.py'. Thanks again.

@alexandrehuat It looks like no one is answering this question 😞 . If you would still like an answer, Rafia at codeFair knows Python and can help! Fill out this form if you're interested: http://tinyurl.com/2uttspwt.

@alexandrehuat It looks like no one is answering this question 😞 . If you would still like an answer, Rafia at codeFair knows Python and can help! Fill out this form if you're interested: http://tinyurl.com/2uttspwt.

This comment appears to be spam since it is out of context and doesn't make sense in the current conversation. My advice is: DO NOT CLICK THE LINK since it may very well redirect you to malware. I have reported the user @codeFairOfficial to the  GitHub abuse program as outlined here: https://docs.github.com/en/github/building-a-strong-community/reporting-abuse-or-spam

This is not spam, but we are so sorry to intrude! We actually are company looking to find new ways to get issues resolved quickly. We currently have a team of programmers finding issues that they could help get fixed. Our appologies for coming off as spam/malware, we will take this into account next time.

@codeFairOfficial thank you for following up on this. We (the maintainers) always do welcome new members to our community, especially when it revolves around offering commercial services related to Numba. To avoid any false-positive classifications of your submissions, I would suggest introducing yourselves, your company and maybe your programmers on our Community forum at: https://numba.discourse.group/   -- this would also allow you to discuss with our community, your approach to finding issues to resolve in order to find consensus on a good process. For example, it will make sense to discuss with users about their needs in the issue itself rather than linking to an external service since "random hyperlinks" are always a cause of suspicion.

@esc thank you for the feedback, we couldn't agree more. We will reach out to your community forum and discuss before posting any further.

@codeFairOfficial thank you!

Hi, I also prefer to see a long URL rather than a short one. At least we
know what domain we are reaching.
Le ven. 5 févr. 2021 à 15:51, esc<EMAIL_ADDRESS>a écrit :

@codeFairOfficial https://github.com/codeFairOfficial thank you!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/numba/numba/issues/5977#issuecomment-774078955, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AFTXC55KEVCHCSRTJA6OHJLS5QAV7ANCNFSM4OWSHSGQ
.

Similar to #6857, I think np.isclose has been implemented:
>>> import numpy as np
>>> import numba as nb
>>> nb.__version__
'0.55.1'
>>> def g(a, b):
...     return np.isclose(a, b)
...
>>> g(np.array([1.1, 2.2, 3.3]), np.array([1.1+1e-12, 2.0, 3.3+1e-12]))
array([ True, False,  True])

Similarly for math.isclose:
>>> import math
>>> def h(a, b):
...     return math.isclose(a, b)
...
>>> h(1.1, 1.1+1e-12)
True
>>> h(1.1, 1.0)
False

So I think this issue can be closed.

Sorry—I have to take that back!
np.isclose and math.isclose are not implemented in Numba.
In my tests, I forgot to add the @nb.njit decorator.

This issue should be reopened!

This issue should be reopened!

Of course, no problem! Errare humanum est! 😘

I'd like to have this feature as well.


[plugin-legacy] hope to support ignoring certain packages

Description
We have a project that uses vite for packaging, there are a lot of pages in this project and they are all individual vue projects, we will iterate through each page (a vue project) and build it using the build api provided by vite. This is actually very fast for a single page, but if we have to wait for all the pages to be packaged it will be very slow. We found that most of the time was spent on legacy, so if we could ignore some of the mature third-party packages, it would reduce the build time a lot!
A big thank you to all those involved
Suggested solution
An option can be added to allow the user to specify which packages are to be ignored, for example
import legacy from '@vitejs/plugin-legacy'

export default {
  plugins: [
    legacy({
      targets: ['defaults', 'not IE 11'],
      ignores: ['Vue', ...more]
    }),
  ],
}

Alternative
No response
Additional context
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.

This is not possible because plugin-legacy runs babel in renderChunk hook. In that hook, all modules are bundled together.
https://github.com/vitejs/vite/blob/0654d1b52448db4d7a9b69aee6aad9e015481452/packages/plugin-legacy/src/index.ts#L452-L475
Sharing a profile would help us find a way to improve perf, it would be great if you can do that.


定位是报错?

Do you want to request a feature or report a bug?
What is the current behavior?
If the current behavior is a bug, please provide the steps to reproduce and if possible a minimal demo of the problem. Your bug will get fixed much faster if we can run. Paste the link to your CodeSandbox (https://codesandbox.io/s/jnnxrwq5y3) example below:
What is the expected behavior?

Cannot read property 'address' of null

显示Cannot read property 'address' of null
直接用示例也是这个错误


Add "random in range" option to alarms cron format

There may exist use cases where scheduling an alarm to fire periodically and randomly in a range of time is useful.  For example, we may want to trigger lengthy backups of several images each day, but have these backups happen at different times.  In a operational environment it may be difficult to keep track of "different times." Rather than hard-code a time, it would be useful to specify fire the alarm "at a random time each day/month etc.".
Jenkins allows a job to be scheduled by a hash of the job's name (which isn't random but handles the above use case).  Explanation below:

The H symbol can be used with a range. For example, H H(0-7) * * * means some time between 12:00 AM (midnight) to 7:59 AM. You can also use step intervals with H, with or without ranges.

The H symbol can be thought of as a random value over a range, but it actually is a hash of the job name, not a random function, so that the value remains stable for any given project.```

Issue moved to openwhisk/openwhisk-package-alarms #38 via ZenHub


Consolidate the PR templates

Unfortunately although you can have multiple issue templates with a nice gui to select them. The PRs don't get that UI element.
Docs are here: https://docs.github.com/en/github/building-a-strong-community/creating-a-pull-request-template-for-your-repository

I'm going to go ahead and merge this to see how it works. I don't think there's any way to tests it, and it's not working at the moment.


tags have no margin

The tags in the markdown files seem to be missing a natural spacing.

Yup, they should have the same spacing between each other as between the tag list and the delimiter border.
I think we could wrap SearchTrigger with a styled-component here, resulting in a StyledSearchTrigger with a margin-left.
Want to lend a hand?
https://github.com/patternplate/patternplate/blob/master/packages/client/src/app/components/pattern/pattern-sheet.js#L45-L47

sure I can look into it.


'CloudflareBypass\CFCurlImpl' not found

I'm trying the example, after having used composer to install. Php newbie, totally.
$ php7.0 curlbypass    # [which is the main README.md example]
PHP Fatal error:  Uncaught Error: Class 'CloudflareBypass\CFCurlImpl' not found in /var/tmp/curlbypass:18
Stack trace:
#0 {main}
thrown in /var/tmp/curlbypass on line 18
PHP 7.0+49 debian  "7.0.33-0+deb9u3"
libcurl3 7.52.1-5+deb9u9
openssl 1.1.0j-1~deb9u1

Fatal error: Uncaught Error: Class 'CloudflareBypass\CFCurlImpl' not found , idm, I get the same error, I installed everything with composer

Try setting the full path to the autoload.php file?

what do you mean? The autoload finds it for me

Also and uuh, since this is the first time ever I've even used composer (whatever it's really for), where do I find the files such as autoload and tell the main program to use it?
(Meanwhile I found cfscrape that does what I want - connect site, wait out the CF timer in im-under-attack-mode, hands back cookie and user-agent string for reuse in cURL, wget or any other non-CF-aware program.)

You must first create a folder on your server, then install the Kyranrana library using composer, then restart apache, then download the extension created by kyranrana from github and take all the files contained in the scr / main folder, and put them in the folder that you had previously created to install composer and then restart apache again

So much effort. All I do is download the zip and upload to a folder. CD to the folder containing the .json file and run "composer install" and its done

You must first create a folder on your server, then install the Kyranrana library using composer, then restart apache, then download the extension created by kyranrana from github and take all the files contained in the scr / main folder, and put them in the folder that you had previously created to install composer and then restart apache again

Ohgod. >.< I don't even have apache installed, just a lighttpd for basic serving needs. This is growing way too big to be worth it for me. :p

Its actually alot simpler than wavdevteam is making out.  Don't be put off contact me and I'll help you

Thanks but nah- cfscraper already does the job for me.

Yeah php isn't really the framework for this job.


Expected identifier but found ":" for host + ng-deep + external styles

🐞 Bug report
Command (mark with an x)

[ ] new
[x] build
[ ] serve
[ ] test
[ ] e2e
[ ] generate
[ ] add
[ ] update
[ ] lint
[ ] extract-i18n
[ ] run
[ ] config
[ ] help
[ ] version
[ ] doc

Is this a regression?
Yes, the previous version in which this bug was not present was: 12.2.x
Description
multiple warnings like this
Warning: ▲ [WARNING] Expected identifier but found ":"

    /Users/drow/projects/tests/ng13-expected-identifier/src/app/app.component.scss:5:2:
      5 │   :host ::ng-deep {
        ╵   ^

for styles like
:host ::ng-deep {
  @import '../../node_modules/katex/dist/katex';
}

katex.css contains @font-face declarations and regular styles
🔬 Minimal Reproduction

git clone https://github.com/artaommahe/ng13-expected-identifier
yarn install
yarn build
see warnings

🔥 Exception or Error
Warning: ▲ [WARNING] Expected identifier but found ":"

    /Users/drow/projects/skyeng/skyeng-frontend/libs/shared/katex/src/component/katex/katex.scss:149:2:
      149 │   :host ::ng-deep {
          ╵   ^

🌍 Your Environment

yarn run v1.22.17
$ ng version

     _                      _                 ____ _     ___
    / \   _ __   __ _ _   _| | __ _ _ __     / ___| |   |_ _|
   / △ \ | '_ \ / _` | | | | |/ _` | '__|   | |   | |    | |
  / ___ \| | | | (_| | |_| | | (_| | |      | |___| |___ | |
 /_/   \_\_| |_|\__, |\__,_|_|\__,_|_|       \____|_____|___|
                |___/

Angular CLI: 13.1.3
Node: 16.13.0
Package Manager: yarn 1.22.17
OS: darwin arm64

Angular: 13.1.2
... animations, common, compiler, compiler-cli, core, forms
... platform-browser, platform-browser-dynamic, router

Package                         Version
---------------------------------------------------------
@angular-devkit/architect       0.1301.3
@angular-devkit/build-angular   13.1.3
@angular-devkit/core            13.1.3
@angular-devkit/schematics      13.1.3
@angular/cli                    13.1.3
@schematics/angular             13.1.3
rxjs                            7.4.0
typescript                      4.5.4

✨  Done in 1.20s.

Anything else relevant?
We use such imports to scope global styles from external lib to current component.

This seems to be an issue with esbuild.
Error
:host ::ng-deep {
  @font-face {
    font-family: "KaTeX_AMS";
    font-weight: normal;
    font-style: normal;
  }
}

Works
:host ::ng-deep {
  div {
    color: red;
  }
}

I looked into this a bit more and from my understanding the failure here is correct.
@font-face rules need to be defined either top-level or in conditional group rules to be valid.  So in your example, by nesting @font-face in the :host selector you are creating CSS which is not valid by standards therefore EsBuild is correctly issuing a warning about this.
See https://developer.mozilla.org/en-US/docs/Web/CSS/@font-face#description for more information.

Closing since nesting @font-face is considered as non-valid CSS which from my understanding browsers don't support. Therefore, a warning in this case is the correct behaviour.


Fix ERB templates for puppet 4

Use @syntax for ERB template variables to fix Puppet 4 compatibility.

FWIW, +1

@danieldreier wanna rebase this?

Looks like this has all been updated already. Closing!

undeleted the branch because we're apparently still using that ref

Sigh.


Integrate Unified Versioning with Existing Build System

This is dependent on the outcome of #8523

Closing, I don't believe this is relevant.


Gaussian elimination

I saw the "Hessian Synchronization" section in the paper, but it is not reflected in the code. Instead, it directly takes Hinv1 = Hinv[i1:i2, i1:i2]. What is the reason behind this?

Hi, this is due to some extra optimizations which we omitted during our main discussion of the algorithm to convey the high level ideas as clearly as possible. First, the inverse Hessian sequence is computed in advance via a Cholesky decomposition, which essentially performs Gaussian elimination. Second, we lazily batch weight updates together to improve GPU utilization; this is why we subselect an inverse Hessian block, process the corresponding columns, and then batch update the remainder of the weight matrix in the end. You can read more about these optimizations in our related GPTQ paper. Overall, these optimizations do not affect the algorithm (they are equivalent), but significantly improve speed and stability.


Publisher: DataSetMessage.SequenceNumber not sequential

DataSetMessage.SequenceNumber should be incremented by exactly one. In my observation, it is far more common for it to be incremented by 2 or even larger amount.

It is not clear what needs to be fixed.

Observed behavior (the two messages are on the same topic, opcua/PC-ZBYNEKZ-4/full/Gate1):
One message published is:
{"MessageId":"b5000411-3847-49d9-83f5-16e36cda3a43","MessageType":"ua-data","PublisherId":"PC-ZBYNEKZ-4","DataSetClassId":"95254774-8145-4df7-92b5-789e5dca9a0d","Messages":{"DataSetWriterId":3,"SequenceNumber":125,"MetaDataVersion":{"MajorVersion":1,"MinorVersion":0},"Timestamp":"2021-06-10T14:30:15.0986718Z","Payload":{"State":{"Type":1,"Body":false},"CycleCount":{"Type":7,"Body":52}}}}
A message that immediately follows is:
{"MessageId":"ddae5c06-aa90-48e4-94e8-109440dbf2ba","MessageType":"ua-data","PublisherId":"PC-ZBYNEKZ-4","DataSetClassId":"95254774-8145-4df7-92b5-789e5dca9a0d","Messages":{"DataSetWriterId":3,"SequenceNumber":130,"MetaDataVersion":{"MajorVersion":1,"MinorVersion":0},"Timestamp":"2021-06-10T14:32:37.319316Z","Payload":{"State":{"Type":1,"Body":false},"CycleCount":{"Type":7,"Body":54}}}}
Expected behavior:
The SequenceNumber in the second message should be 126, not 130.

Pushed fix. It requires a sub module update (bug in UAPubSub library).

This works OK now.


type "EncryptedCredentials" is defined in an assembly that is not referenced

Hello.
I am writing a telegram bot
There is a need for data decryption. I use the libraries "Telegram.Bot.Extensions.Passport" and "Telegram.Bot". I program myself in another language, in which there is no way to do this, but there is the opportunity to use the assembly as a COM-object. I made the COM-object myself, but to build the solution you need to sign it and that all internal assemblies are also signed. I also did this.
But after signing the internal assembly "Telegram.Bot" on the "DecryptCredentials" method, an error appears: type "EncryptedCredentials" is defined in an assembly that is not referenced.
Do not judge strictly since I do not program in C #, but only study
I would be very grateful for any tips.

Проблема решена.
Как оказалось необходимо было установить сборки в GAC .
После этого все заработало.


Compiling with New Architecture fails

Describe the bug
When compiling with the new architecture enabled, this library fails with the following error:
› Compiling @datadog/mobile-react-native Pods/DatadogSDKReactNative » DdSdk.mm

❌  (/Users/rbibk014/sites/ctg-whitelabel-app/node_modules/@datadog/mobile-react-native/ios/Sources/DdSdk.mm:100:5)

   98 |
   99 | + (void)initFromNative {
> 100 |     DdSdkNativeInitialization *nativeInitialization = [[DdSdkNativeInitialization alloc] init];
      |     ^ unknown type name 'DdSdkNativeInitialization'
  101 |     [nativeInitialization initializeFromNative];
  102 | }
  103 |

❌  (/Users/rbibk014/sites/ctg-whitelabel-app/node_modules/@datadog/mobile-react-native/ios/Sources/DdSdk.mm:100:57)

   98 |
   99 | + (void)initFromNative {
> 100 |     DdSdkNativeInitialization *nativeInitialization = [[DdSdkNativeInitialization alloc] init];
      |                                                         ^ use of undeclared identifier 'DdSdkNativeInitialization'
  101 |     [nativeInitialization initializeFromNative];
  102 | }
  103 |

❌  (/Users/rbibk014/sites/ctg-whitelabel-app/node_modules/@datadog/mobile-react-native/ios/Sources/DdSdk.mm:137:31)

  135 |
  136 | - (void)clearAllData:(RCTPromiseResolveBlock)resolve reject:(RCTPromiseRejectBlock)reject {
> 137 |     [self.ddSdkImplementation clearAllDataWithResolve:resolve reject:reject];
      |                               ^ no visible @interface for 'DdSdkImplementation' declares the selector 'clearAllDataWithResolve:reject:'
  138 | }
  139 |
  140 | @end

Reproduction steps
Use expo sdk 51 and setup with new architecture.
Add this library and try to build using expo run:ios
SDK logs
No response
Expected behavior
No response
Affected SDK versions
2.3.3
Latest working SDK version
n/a
Did you confirm if the latest SDK version fixes the bug?
Yes
Integration Methods
Yarn
React Native Version
0.74.1
Package.json Contents
No response
iOS Setup
No response
Android Setup
No response
Device Information
No response
Other relevant information
No response

Hey @bkasten-rbi,
Thank you for reaching out!
The update to Expo 51 is in our roadmap, and we will grant support soon with<EMAIL_ADDRESS>I will keep you posted as soon as we have an open PR.
Can I ask you to move this issue to https://github.com/DataDog/expo-datadog?

Hi @bkasten-rbi, as we are introducing official support for Expo 51, we have reviewed this issue and tried to reproduce it, with no luck.
The only anomaly that comes to mind is that the error might be related to cache or some custom config that is breaking Swift compatibility.
Would you mind creating a backup of your project and then running npx expo prebuild --clean?
If the issue is not resolved it would surely help to provide us with a basic sample app in which the issue can be reproduced and/or add the missing information listed in the issue template.

I will close this issue as there has been no activity for a while now. Feel free to re-open it if the problem persists even after updating the Expo plugin and cleaning your project.


Steal buffer-builder's thing with optimised handling of ASCII key names

(Perhaps when encoding with TH (and so we can determine at compile-time whether a literal is ASCII or not).)

Okay, I tried it and it didn't give any benefit.

For the sake of it, here's some code:
(.=#) :: ToJson a => Addr# -> a -> Series
name .=# value = Value . Encoding $
                 E.textAscii name <> BS.char7 ':' <>
                 fromEncoding (toEncoding value)
{-# INLINE (.=#) #-}

{-# RULES "ASCII key .=" forall a b.
  T.pack (unpackCString# a) .= b =
    a .=# b
  #-}

textAscii :: Addr# -> Builder
textAscii t = B.char8 '"' <> encodeAsciiBuilderEscaped escapeAscii t <> B.char8 '"'
{-# INLINE textAscii #-}

-- | Encode a null-terminated string.
encodeAsciiBuilderEscaped :: BP.BoundedPrim Word8 -> Addr# -> B.Builder
encodeAsciiBuilderEscaped w = \addr -> builder (mkStep addr)
  -- A modified version of primMapListBounded from
  -- Data.ByteString.Builder.Internal (also see GHC.CString.unpackCString#)
  where
    bound = BP.sizeBound w
    mkStep addr = step 0#
      where
        step pos k (BufferRange op0 ope0) = go pos op0
          where
            go i !op
              | isTrue# (ch `eqWord#` 0##) =
                  k (BufferRange op ope0)
              | op `plusPtr` bound <= ope0 =
                  BP.runB w (W8# ch) op >>= go (i +# 1#)
              | otherwise =
                  return (bufferFull bound op (step i k))
              where
                !ch = indexWord8OffAddr# addr i
{-# INLINE encodeAsciiBuilderEscaped #-}


Annotations preserve extensions

Keep file extension on download
Moving the FlyteFile related things out to a separate file.

lgtm 👍


Get-PnPPage returns Object reference not set to an instance of an object.

Reporting an Issue or Missing Feature
Issue
Expected behavior
Should return Page object
Actual behavior
Throws exception
Steps to reproduce behavior
First, I connect to my site like so:
Connect-PnPOnline -Url https://mycompany.sharepoint.com/product/development/repos/ -UseWebLogin

I can run `Get-PnPFolderItem -Identity "Pages" and see:
Name          Type   Items/Size Last Modified
----          ----   ---------- -------------
Forms         Folder          0 1/26/2022 1:50:02 AM
AllRepos.aspx File       902349 1/26/2022 11:20:17 PM
default.aspx  File         4348 1/26/2022 5:34:27 PM
NewRepos.aspx File       481474 1/26/2022 11:24:48 PM

Next, I try to get one of the sites with
Get-PnPPage -Identity "default.aspx"

I get the following exception:
Get-PnPPage : Object reference not set to an instance of an object.
At line:1 char:1
+ Get-PnPPage -Identity "default.aspx"
+ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : WriteError: (:) [Get-PnPPage], NullReferenceException
    + FullyQualifiedErrorId : EXCEPTION,PnP.PowerShell.Commands.Pages.GetPage

The Get-PnPException command returns:
Message          : Object reference not set to an instance of an object.
Stacktrace       :    at PnP.Core.Model.SharePoint.Page.<GetPagesListData>d__99.MoveNext()
                   --- End of stack trace from previous location where exception was thrown ---
                      at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
                      at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
                      at PnP.Core.Model.SharePoint.Page.<LoadPagesAsync>d__98.MoveNext()
                   --- End of stack trace from previous location where exception was thrown ---
                      at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
                      at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
                      at PnP.Core.Model.SharePoint.Web.<GetPagesAsync>d__288.MoveNext()
                   --- End of stack trace from previous location where exception was thrown ---
                      at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
                      at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
                      at PnP.Core.Model.SharePoint.Web.GetPages(String pageName)
                      at PnP.PowerShell.Commands.Base.PipeBinds.PagePipeBind.GetPage()
                      at PnP.PowerShell.Commands.Pages.GetPage.ExecuteCmdlet()
                      at PnP.PowerShell.Commands.PnPSharePointCmdlet.ProcessRecord()
ScriptLineNumber : 1

Note, I can put anything I want for the identity:
Get-PnPPage -Identity "BlahBlah"

and I get the same exception, so I don't think it's even getting to the part of looking up the site.
I think this might be due to the way I connect to the site, but I'm not sure of another way to do it.  I think either Get-PnPPage doesn't support subsites (or nested subsites?) or it doesn't support whatever auth method I'm using.
What is the version of the Cmdlet module you are running?
1.9.0
Which operating system/environment are you running PnP PowerShell on?
Windows

Update on this.  It's something to do with my subsite.  If I connect to one level below, https://mycompany.sharepoint.com/product/development/, I can get pages just fine.

@gautamdsheth I don't understand why this was closed, what's the work around for the issue?  I need to be able to access pages on the /product/development/repos/ site.  If I connect to that site, I get an exception.  I can connect to and access pages on the parent site just fine.  Can you provide me with a way to connect to /development/repos and access pages on /development/repos/ without getting the exception?  Is there anything I need to configure on the site for this to work?  I'm an owner of the site so I don't think it's permissions.  Why does the cmdlet work on one site, but not another?  Sorry if my question doesn't make sense.  Please feel free to ask me more questions instead of closing the bug.  We really need to get this working.

@veronicageek Yea, it must be something to do with the actual site since I can connect and get pages on some subsites, but not others..  The "Object reference not set to an instance of an object." exception is quite vague.  Is there a way to get more details about the problem?

Ok I assumed that the issue is fixed because you mentioned it yourself that you connected to subsite and it started working.
Also Get-PnPPage is used to get modern pages from site pages library.
If you point it towards publishing pages or wiki pages , it will not work as it is not supported

@gautamdsheth Ok that makes sense.  My end goal is to create and publish my own HTML page, but have it have the site master page and menus/navigation.  I can publish a "document", which I've gotten to work, but then I'm responsible for 100% of the HTML.  Basically, we're looking at Sharepoint as a way to host some content we create using a script that generates a bunch of HTML..  We could probably use an App Service or something, but then we'd have to implement authentication and all that boring stuff.
I've posted this question on StackExchange in hopes of getting someone to help me.  Maybe you can point me in the right direction, since it seems that the "PnPPage" Cmdlets might not be what I'm looking for.  Thanks!

Moving this to a discussion


Balance does not display on homescreen when there is no internet connection

Happened to me while working on a plane. Thought it was the bitcoin price api but I stubbed that out and the issue still existed. Maybe we are getting a font from the internet?

Diving into this bug a little bit, I have come across what i believe to be the issue.
The react query object is of type MutinyBalance so even though it stores some data in the cache i believe it is still reaching out to the API to turn it into a number as MutinyBalance is what determines the values for confirmed, lightning, and unconfirmed.

I think if the values could be stored in the cache as in integer or as a string instead of type MutinyBalance then the value could still be displayed. I'm hesitant to just open a PR as I really am not sure how i would go about this.


estimating head pose on several face

thanks so. I have an question, is this model acceptable on several face in a frame to estimate head pose?

Of course, you can try to modify the code yourself, which may be a little complicated

Of course, you can try to modify the code yourself, which may be a little complicated

very good, thanks.


ci: windows: Update vcpkg to 2022.03.10 and use run-vcpkg@v10

Signed-off-by: Otavio Salvador<EMAIL_ADDRESS>

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
5 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-1.3%) to 84.935%

Files with Coverage Reduction
New Missed Lines
%

../../../.cargo/registry/src/github.com-1ecc6299db9ec823/futures-channel-0.3.21/src/mpsc/mod.rs
2
90.41%

../../../.cargo/registry/src/github.com-1ecc6299db9ec823/futures-channel-0.3.21/src/mpsc/sink_impl.rs
3
76.92%

Totals

Change from base Build<PHONE_NUMBER>:
-1.3%

Covered Lines:
327

Relevant Lines:
385

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
332

Relevant Lines:
385

💛  - Coveralls


Track Object and people Detection

It would be great if we had alarm options to warn people and objects on the track. Imagine a scenario where an object or even a person is on the track and the system would inform and record it so that we can later know who and when caused an accident?

Thanks for your input, the API gives you a end point to get the live stream of the objects detected, you could build you custom logic to do some actions depending on this: https://opendatacam.github.io/opendatacam/apidoc/#api-Tracker-Data .


Namespace Issue on Initial Migration

Hey uafrica,
Thanks for this plugin! Looks great so far. I ran into a small namespacing issue while running my initial DB migration. Both<PHONE_NUMBER>1027_initial_migration.php and<PHONE_NUMBER>0800_change_owner_id_field.php threw "Fatal error: Class 'Migrations\AbstractMigration' not found in <path_to_migration_file>.
Being fairly new to Cake 3.0 and having only completed one other project in Cake 2, I simply searched for anything with the name "AbstractMigration" and came up with Phinx\Migration\AbstractMigration. A simple use statement at the top of each migration file seems to have fixed the issue. Of course, it always makes me nervous when modifying vendor libraries directly so I thought I'd let you know.
I'm using Cake 3.0.10. All tables seem to have been created successfully, but I guess I won't know for sure until I get everything up and running!

Hi @csc1cec, the oauth server plugin requires the CakePHP migrations plugin to work. That is the plugin that supplies the correct Migrations\AbstractMigration class. If you installed the plugin using compose (composer require uafrica/oauth-server) then the migrations plugin should have been automatically installed for you.
If you downloaded the files and added the plugin manually, then you'll need to manually install the migrations plugin too.

Hi @dakota, that's odd since I used composer to install.
Like I said, it seems adjusting the namespace solved the problem and all DB tables seem good to go. So no problem on my end. Just giving you guys a heads up there may be a bug lurking somewhere around migrations.

Thanks for the heads up.


Inserted "threshold" log property in Log4j.properties file.

Fixed log4j properties.

Should I change the threshold to WARN in travis settings?

@prasannapramod the first successful Travis build after a long time. To celebrate the occasion, I merged this PR.


No results returned from some React websites

Hi,
I've been using HtmlUnit for the last few weeks and it really is a great tool - very flexible with a nice intuitive API.
In the last few days I've encountered issues trying to scrape the following 2 websites:
https://www.elastic.co/blog/
https://blog.bitrise.io/
Htmlunit: 2.35.0
Both sites seem to be using React which I suspect is related to the problem.
Here is the code to reproduce the issue:
	WebClient client = new WebClient();
	client.setJavaScriptTimeout(30000);
	client.getOptions().setTimeout(60000);
	client.getOptions().setRedirectEnabled(true);
	client.getOptions().setThrowExceptionOnFailingStatusCode(false);
	client.getOptions().setThrowExceptionOnScriptError(false);
	client.getOptions().setCssEnabled(false);
	client.getOptions().setJavaScriptEnabled(true);
	client.getOptions().setPrintContentOnFailingStatusCode(false);
	client.getCookieManager().setCookiesEnabled(false);
	client.getOptions().setUseInsecureSSL(true);
	client.getCurrentWindow().setInnerHeight(60000);
	client.setAjaxController(new NicelyResynchronizingAjaxController());

	// Example 1
	String url = "https://www.elastic.co/blog/";
	String selector = ".main-wrapper .blog-column";

	// Example 2
	//String url = "https://blog.bitrise.io/";
	//String selector = ".articles .article-container";

	HtmlPage page = client.getPage(url);
	client.waitForBackgroundJavaScript(30000);

	DomNodeList<DomNode> results = page.querySelectorAll(selector);
	System.out.println("Found "+results.size()+" results");

Both examples always produce zero results even though if you click "Inspect" on the pages the CSS classes are all present.
I've tried changing the client options based on what I've read from previous issues but none of the changes made any difference.
I'd really appreciate some pointers as I've spent hours trying to crawl these pages but couldn't make it work.
Thanks,
Gerald.

see #755


Pin versions + Pep8

Should resolve 18f/atf-eregs#241

Neat! Now that versions are pinned, we may want to set up gemnasium or requires.io to make sure we don't miss critical updates.


handle refresh (RE: #71)

This could very well be an abomination (sysadmin here), but this seems to work for us.
When using openssl::certificate::x509, if the config changes, the CSR and cert are regenerated.

LGTM at least… @mcanevet what do you think?


Lecture03

Петраки

Coverage increased (+15.8%) to 63.889% when pulling 72cb88067c3009f43457b74b955d8639568c02f4 on javatechno:lecture03 into 16dad782f5d596ae50d0b39665b32df40da60c19 on rybalkinsd:lecture03.


Upgrade registry.scs.community instance of Harbor to the latest (stable) version

Currently, registry.scs.community operates on Harbor v2.9.2.
Upgrade this instance to the latest/stable Harbor 2.10.0 and document it.

After we tried to upgrade to the latest Harbor version using Harbor Helm v1.14.0 we fell into issues.
Hopefully, the rollback was successful.
Later we found that the Harbor Helm chart 1.14.0 is broken: https://github.com/goharbor/harbor-helm/issues/1721
Postpone this upgrade until Harbor Helm chart v1.14.1 with the proper fix is out.


Enhance toHaveProperty() to allow array selection

🚀 Feature Proposal
Enhance the .toHaveProperty() matcher to allow array selection, similar to lodash/get.
Motivation
Will allow very easy assertions on very complex objects which contain nested arrays/objects.
Example
const stateData = {
  attributes: {
    all: [
      {
        name: `Colour`,
        values: [
          { name: `Black`, products: [2, 3] },
          { name: `Orange` },
          { name: `Green` },
          { name: `Purple` }
        ]
      }
    ]
  }
}

expect(stateData).toHaveProperty(`attributes.all[0].values[0].products`, [2, 3])
expect(stateData).toHaveProperty(`attributes.all[0].values[2].name`, `Green`)

Pitch
It's an enhancement to an existing matcher which would allow smaller and cleaner tests when asserting against large and complex objects which contain arrays.
As an enhancement, it wouldn't break any existing functionality.
Would bring the path spec in line with the very popular lodash library.
Would meet many peoples expectations of Jest, as I assumed this is a common task and would already be available.
Notes
If I've missed something and asserting deep properties in objects containing arrays is already easy, then sorry, I'm happy to be pointed in the right direction.

Right now you can do:
expect(stateData).toHaveProperty(
  ['attributes', 'all', 0, 'values', 0, 'products'],
  [2, 3],
);

It's obviously not the same, but possible. lodash/get syntax is nice, but privately I'm very against this library as it looses all type information (at least in Flow codebases) and I prefer to use idx.
I'm open to discussion on whether we want this API or now :)

@thymikee Thank you for that, I didn't know you could access arrays like that, I was expecting to have to use an ugly set of nested expect.objectContaining() and expect.arrayContaining().
Just checked the docs again and there is indeed an example already, which I missed. Sorry, my mistake.
Interestingly, I've been trying to get started with Flow this week, and ran into the issues you reference with lodash right-away.
FWIW, I'm more than happy with the array path method for toHaveProperty().

Closing then (we have too many issues 😫), but happy to accept a PR implementing that, if anybody is interested :)

+1 for this, honestly I don't really care about types in this situation of doing a expect. And the existent dot notation already erases all of that, so adding the array on top won't make any difference.


Separate the method _maybe_format_datakeys_with_stream_name into a method for each of its use cases

Recently the collect method in the run blundler was refactored to reduce mixing of code paths.  Previously _maybe_format_datakeys_with_stream_name would take one of two different data structures layouts (a new and an older) and would format it respectively to it's type. Now these two paths have been separated so that _maybe_format_datakeys_with_stream_name is called with one structure type in declare_stream, and with a different in _describe_collect. Instead of having one method for both it would be worth doing these seperately.

Reviewers: @abbiemery @danielballan


Feature: concave hulls

The GEOS C API main branch has a new feature GEOSConcaveHull which was ported from JTS to evaluate concave hulls described here by Martin Davis.
See also PostGIS docs for ST_ConcaveHull.

There are still some other "new" features from previous GEOS versions as well that are not yet exposed in Shapely. We should probably move over a cleaned-up version of https://github.com/pygeos/pygeos/issues/126 to the Shapely repo.


Answer geographic questions on this data set

Goal:  Answer the following questions:

[x] Is this dataset collected throughout the country?
[ ] Can we access the municipality for each survey?

Relevant resources:
https://www.ine.gob.gt/index.php/encuestas/empleo-e-ingresos

We cannot access the municipality data for this survey.


Remaking readme and adding vietnamese sub for README.md

pull me to contributor, and translator. Tedd#0001


Option to add media to "ignore" list

Feature summary
Sometimes an image shouldn't have any alt text, decorative imagery for example. As the package picks up empty descriptive texts then these will always show in the list.
It would be good to have some sort of ignore list so you can tell it you've checked it, and it's fine.
Additional details
No response

Ohh, I like that!
Now the question is, does it use a property on the media item? or a custom database table to track it?
I spose it would need a view to see "ignored items" as well.


Release BlockReader in CRC check

What changes are proposed in this pull request?
Please outline the changes and how this PR fixes the issue.
Why are the changes needed?
Please clarify why the changes are needed. For instance,

If you propose a new API, clarify the use case for a new API.
If you fix a bug, describe the bug.

Does this PR introduce any user facing changes?
Please list the user-facing changes introduced by your change, including

change in user-facing APIs
addition or removal of property keys
webui

alluxio-bot, merge this please.

alluxio-bot, cherry-pick this to branch-2.10 please.


interactive charts

Is it possible to generate interactive charts in reports, like the ones generated with plotly or chart JS? TIA.

would love to know this as well. Could not get plots.to_plotly() to work


ex_doc delete all files in the doc/ directory

I have just started trying out ex_doc alongside edoc. The first thing I noticed was that the original contents of the doc directory, in my case "overview.edoc" and ".gitignore", was cleared during the run :-(
Of course this can be prevented with the "--output" command line option, but having to type that every time is a bit tedious!
Any chances of supporting the "output" option in the rebar.config file?
I should be able to provide a PR in a week or two, unless someone is already working on this.

It would be appreciated 👍

fixed with PR #36


Update dependencies

I updated the dependencies in line with the main riker repository. You still need to change the riker version in the Cargo.toml once you have published a new version of riker.

I also updated the riker version.


WIP add intersection behavior for RSP

What this PR does / why we need it:
We add intersection behavior because it makes sense to result placement for resource propagation.
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #981
Special notes for your reviewer:

placement computing rule is get from current federated resource calculation. That means the related rule is reused in RSP for placement calculation
I'm trying to add test and add test cases. So, please review when related tasks are done.

/retest

/retest

/test

@irfanurrehman
It is ready to review.

Thanks for doing this @xunpan. I will positively take a look at this tomorrow.

@xunpan Could you address the requested changes by @irfanurrehman ?

Yes. The code was done and I'm testing it locally. The updated diff will be committed by this week.

Yes. The code was done and I'm testing it locally. The updated diff will be committed by this week.

@xunpan I missed checking on this. I see that you did some updates. Were they finalised by you? Should I check them again?

@irfanurrehman I forgot to send you a message after update the PR.
I think I addressed your comments above. One does not changed:
[Q] preferredClusters it seems to be a copy of resultClusters
[A] It is used to transform sets.String to string slice

Thanks for doing this @xunpan.
/lgtm


[BUG] Data is not coming in the pages after running doctor publish command

Describe the bug
Data is not coming up in the published pages after running doctor publish command. No error shown in the command prompt.
To Reproduce
1: Run doctor init in my existing documentation repository.
2: It creates one index.md file inside existing src folder and doctor.json file.
3: Update doctor.json url field with newly created communication site url.
4: Run doctor publish with 1.7.0 and 1.11.0 version.
5: Pages have been published with proper navigation structure. However data is not coming up in index.aspx.
Expected behavior
Data should come in .aspx as given in .md file
Screenshots
CMD:

VS:

Page: It created one page called index.aspx, however given data is not coming up.

Desktop (please complete the following information):

OS: [e.g. iOS] Windows
Version [e.g. 22] 1.11.0 and 1.7.0

Additional context
Its working fine when I'm creating a new project using doctor init and doing same steps.

Can you use --debug flag and verify the output? That way you should see a bit more details about what it does during the publishing process.

Hi @estruyf ,
Thank you ! I believe its not publishing the content when my folder exist after 3rd level i.e. C:\Folder1\Folder2\Repo in my local drive. its working fine if my repo exists till 3rd level folder structure i.e. C:\Folder1\Repo.
Same issue is occurring If I'm creating a new project using doctor init after 3rd level folder in my local drive. Is it supposed to work like that only ?

That should not be a problem, but to be sure, what does your log output say? Please verify by: doctor publish --debug > output.log.

Hi @estruyf .. I've tried it out today and it works. Not sure what was the issue though !
However I've another query related to certificate based authentication. We are planning to create pages on one of the sites where we don't have tenant level access and the team which managed it will definitely not give  Sites.FullControl.All application permission to our app. Password method is also failing and device code will not work as we are publishing pages using build pipeline.
So for certificate authentication , I was trying with Microsoft Graph Site Collection level permission (Sites.Selected) which has been recently introduced . Its authenticating but failing while processing .md files.

Could you please suggest if we have any alternative for it.

I had this issue too.
When copying the folder content to a new directory it worked again.
It looked like the doctor publish command was also trying to do things with stuff from my other test folders when --debug was on...


Fix chain sync from genesis

Increment version <IP_ADDRESS>
New protocol version 70018
Ban old peers after soft fork
Fix for mismatched block data at 264695 that caused sync from genesis to fail.
Add checkpoints to guide nodes.
Minor code tidying..
Update block version to 0x20000100

Soft Fork validated on testnet.


minor fixes

bors r+

It’s probably preferable to accept markdown’s views rather than tampering
with it. If you want  instead there are ways
El El sáb, feb. 10, 2018 a las 8:23 a. m., Vorpal Buildbot <
<EMAIL_ADDRESS>escribió:

Seen on PROD https://pennydreadfulmagic.com/about (merged by @silasary
https://github.com/silasary 19 seconds ago) Please check your changes!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/PennyDreadfulMTG/Penny-Dreadful-Tools/pull/4266#issuecomment-364653214,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAAGBW6iKnNBd-UaEC5Q0tuZUq_0V1coks5tTZhPgaJpZM4SA68y
.


Update Installation.md

Amend misleading language surrounding NONINTERACTIVE=1
As a user who tried to install brew without any user interaction whatsoever, I bumped into the documentation stating it won't ask for a password. The documentation was misleading, one cannot install brew without any password prompts when passing NONINTERACTIVE=1.

[ ] Have you followed the guidelines in our Contributing document?
This is just a docs change, not a package/code contribution.
[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.
No, I don't believe that is applicable to site content changes.
[ ] Have you successfully run brew style with your changes locally?
N/A
[ ] Have you successfully run brew typecheck with your changes locally?
N/A
[ ] Have you successfully run brew tests with your changes locally?
N/A

Hi guys. Apologies for joining late but I think we need to have a closer look at the report.

As a user who tried to install brew without any user interaction whatsoever,

It is possible. If you need to emulate user interaction during Homebrew installation, use something like this (ref: 1):
echo -ne '\n\n\n\n' | bash -c "$(curl -fsSL https://raw.githubusercontent.com/Homebrew/install/master/install.sh)"

I bumped into the documentation stating it won't ask for a password.

Non-interactive mode should not prompt for a password. If your observations are different - let's have a look at them.


Look into FlySystem

It sounds like it's worth investing some time looking into the use of FlySystem to replace the interactions with the file system..
https://github.com/thephpleague/flysystem

Looks like there is a plugin too, https://github.com/WyriHaximus/FlyPie

FYI we've been using FlyPie and Flysystem in production w/ cake 3.x for more than a year without any trouble :) Using adapters for: local, sftp & dropbox :)