Move blockfinding algorithm to this library

Blockfinding = https://dabreegster.github.io/talks/aiuk_ltn/slides.html#/blockfinding, tracing around street-edges
This would have use in the urban morphology research community, and a simple API for them to grab polygons would be great. Just a reminder to myself to eventually do this.

I've been experimenting in https://github.com/a-b-street/osm2streets/tree/planar_graph_literally. It's a long way from working, and I think the resulting code belongs in a new repo, unrelated to osm2streets. Just want a place to vent frustration and share some progress. :)
https://user-images.githubusercontent.com/1664407/212067620-a0a282b9-9810-461d-8fd6-449ddbda60db.mp4
How it works now:

take all of the intersection polygons, thickened road polygons, and the boundary polygon
turn those into each line segment
find every pair of line segments that intersect, and split them (in a horrible brute force way)
now we have a graph of nodes and edges. sort the edges around a node and track edges that have a left/right side and a forwards/backwards orientation
it's fairly straightforward to then start tracing from every edge+side+direction and make a loop

There's a bunch of hacky UI code to do four-coloring, detect overlapping faces, nodes too close to each other, etc.
Current big problems:

most maps wind up with one gigantic face covering everything. it's kind of valid/correct, maybe just tracing the boundary from the wrong side. I'm filtering it out just by checking for huge area
most maps are missing some faces. we wind up with a bad Ring that doubles back on itself somehow
borough by st george has a face that covers too much. there are very close nodes in there probably to blame.

I got very stuck for a while until I started aggressively reducing the resolution of points, 0.1m right now. Even now it doesn't work in some areas. The more funky geometry coming from osm2streets, the more points we wind up with very close together, and the harder it is to correctly trace faces in the planar graph

Alright, it turns out I've maybe been reinventing a wheel totally unnecessarily! MapShaper does practically all of the work:

Bring in road+intersection geometry and the boundary
Merge the two layers if needed (merge-layers target=Geometry,Boundary)
Turn line intersections into new polygons, handling all sorts of bridge/tunnel problems: mosaic
Four-coloring to visualize: classify non-adjacent save-as=fill colors=category10

Things to think about next:

[ ] How can we use this from Rust? (Depending on the application of course. If I were to build a new version of the LTN tool that's web-native, for example, depending on the JS libraries would be fine.)
[ ] Preserve properties per polygon or edge (so we know what a road, intersection, block spanning a bunch of roads, etc is)
[ ] Figure out how to remove edges / dissolve adjacent polygons
[ ] (More on the LTN side) Figure out what it means for a neighbourhood to not be bounded by roads necessarily. Still find the border intersections.

Preserving properties is taking some getting used to, but we can at least distinguish blocks from roads/intersections:
mapshaper lyon_geom.geojson -mosaic calc='overlap=count()' -colorizer name=showOrig colors='red,green,grey' breaks=0,1 -style fill='showOrig(overlap)' stroke=black -o format=geojson topo.json


Refactor word cloud code to operate through back end

--Replaced word-object constructor with AJAX call (pair-code w/ Geoff)
--Got AJAX POST req. set up to send new word text, topicID, userHash and vote state to back end (pair-code w/ Geoff)

[ ] Run jQCloud on empty array to set up real estate in view for word cloud

[ ] Set up new click handler for wd. cloud items as AJAX POST to back end (TopicController.initCloudItemClickHandler) and move to appropriate file (topic-controller.js)

[ ] Set up callback to GET updated wd. cloud from back end

[ ] Run jQCloud on back end data to populate or refresh cloud graphic


Update encoding/decoding logic to use *64 rather than *size

This ensures we have consistent behaviour across 32bit and 64bit environments.

Actually, I'm not sure whether we should actually use *32 so that things like Vec can be ensured to work on 32 bit targets.

Im pretty sure - everywhere we use usize its as an index to a vector and thus usize makes sense.   I would expect a 32bit machine running automerge to break on more than MAX_INT ops or changes etc (but also run out of memory before then).


Merge pull request #1 from covid19india/master

.
Description of PR
Type of PR

[ ] Bugfix
[ ] New feature

Relevant Issues
Fixes #...
Checklist

[ ] Compiles and passes lint tests
[ ] Properly formatted
[ ] Tested on desktop
[ ] Tested on phone

Screenshots

There is nothing here. Seems got created by mistake


Implement comand builder & text objects

The command builder is a mechanism for composing commands which can be interpreted by Iota. These commands use structures called text objects which can refer to any object inside a buffer (char, line, word etc).
Thanks to @crespyl for most of the design and implementation on this.
Fixes #89

Woo!  It's happening (.gif)!
There's a lot of changes in here, and plenty of holes and new breakage I'm sure, so getting some fresh eyes on this stuff before we merge would be great.
We're also pretty close to being able to close #8.  With this stuff, pretty much all the groundwork is in place to wrap that one up.  We just need an Insert mode and some touch ups to round out the supported TextObject kinds/anchors (and maybe some adjustments to the default keybinds).

Any objections / comments before I merge this?

aaaaaand merged! Thanks again @crespyl


SEC's show up when you search C's. SR's show up when you search R's.

When you search rarity on using the buttons on the side bar.
SEC's show up when you search C's. SR's show up when you search R's.

On it

Fixed with the next release.


History view

With the new SLEEP version, dat can download, save the changes, and revert to a previous point in history. It can also see a diff of which files changed. It could also see the content of the files that changed, but let's just have this iteration focus on looking at the history of changes and restoring to a previous point in time.
Here are some examples:
Restore to earlier version

While a file is open

Table view

GitHub history

GitHub Desktop App

cool, thanks for sharing!

Example from Dropbox:

DAT creates a lot of versions in a short time. It would be good to allow a DAT client to have some sort of "bundled versions" that act a little bit like Keyframes in Movies. Is there a system for this in place?

Beaker-browser implements versions like this:
https://github.com/beakerbrowser/beaker/blob/37981f57d5c6f30aab0f94b72213d9505c272d78/app/builtin-pages/views/library-view.js#L685-L803

For me the name version is a bit misleading, or it's not what I would expect as a user.
The current version history, that dat log returns is in my opinion limited useful: it can grow very long easily but does not provide much information beyond individual file changes. I would more expect something like git log.

https://github.com/datprotocol/DEPs/pull/5#issuecomment-439130827

Version: Internally every dat data-structure is composed of append-only logs (hypercores). Any time an entry is appended to the log, a new version is created. The version is identified according to the semantics of the data-structure. In the case of single-writer hyperdrive, it's currently being identified by the metadata log's latest message number.

Maybe this feature's main-mean is "history".
But, martin's told "version" have mean is "history of file operation".
I think so too.
Which is user friendly ?
I think ... "Time based versioning".
Time is chunk. And, exist params (versions) has a datetime.
So, can show squashed version with timebase.
Users is can configuration the time-span. (it be like so Financial chart)
( maybe, it is can only by frontend-task. )
Sorry for read the my ugly-english.

Thank you for take a knowledge.
My suggestion was required a timestamp of Delete and rename 😭
hmmmmmm.....

Note: I've proposed self-reported timestamps (git style) to be included in hyperdb/hyperdrive:

https://github.com/datprotocol/DEPs/pull/11
https://github.com/bnewbold/dat-deps/blob/dep-hyperdb-timestamps/proposals/0000-hyperdb-timestamps.md


Host mode does not work on Linux.

Host mode does not work in the Linux version of Stardew Valley
[ERROR] FATAL UNHANDLED EXCEPTION: System.MissingMethodException: Method 'TcpListener.Create' not found.
  at System.Threading.ThreadHelper.ThreadStart_Context (System.Object state) <0x40eb72f0 + 0x00083> in <filename unknown>:0
  at System.Threading.ExecutionContext.RunInternal (System.Threading.ExecutionContext executionContext, System.Threading.ContextCallback callback, System.Object state, Boolean preserveSyncCtx) <0x40eb6e10 + 0x001f5> in <filename unknown>:0
  at System.Threading.ExecutionContext.Run (System.Threading.ExecutionContext executionContext, System.Threading.ContextCallback callback, System.Object state, Boolean preserveSyncCtx) <0x40eb6c20 + 0x00023> in <filename unknown>:0
  at System.Threading.ExecutionContext.Run (System.Threading.ExecutionContext executionContext, System.Threading.ContextCallback callback, System.Object state) <0x40eb6b60 + 0x0005b> in <filename unknown>:0
  at System.Threading.ThreadHelper.ThreadStart () <0x40eb6a20 + 0x0002f> in <filename unknown>:0
AL lib: (EE) alc_cleanup: 1 device not closed

This appears to be similar to the following bug: https://bugzilla.xamarin.com/show_bug.cgi?id=38167

Huh. That's weird.
What version of Mono are you using?

The system version I have installed is
Mono JIT compiler version 4.6.1 (Stable <IP_ADDRESS>/abb06f1 Mon Oct  3 07:51:42 UTC 2016)
Copyright (C) 2002-2014 Novell, Inc, Xamarin Inc and Contributors. www.mono-project.com
	TLS:           __thread
	SIGSEGV:       altstack
	Notifications: epoll
	Architecture:  amd64
	Disabled:      none
	Misc:          softdebug
	LLVM:          supported, not enabled.
	GC:            sgen

However, Steam/Stardew may have its own version.
Kernel Version:
Linux Manjaro 4.9.0-1-MANJARO #1 SMP PREEMPT Sun Nov 27 23:57:02 UTC 2016 x86_64 GNU/Linux

Worked for me on Arch Linux when I changed "TcpListener.Create(port)" to "new TcpListener(IPAddress.Any, port)" in Multiplayer.cs.

Yes, but that isn't a good solution. See #44

I believe platform issues are fixed in the current master, although I'd appreciate it being tested.

I'm still seeing the same with 2.12 and the builtin Mono of the GOG version (which should be 4.6).


arm/modifyreg: Use common lock to protect the modify address to ensure consistency

Summary
arm/modifyreg: Use common lock to protect the modify address to ensure consistency
Signed-off-by: chao an<EMAIL_ADDRESS>Impact
N/A
Testing
ci-check

[Experimental Bot, please feedback here]
No. This PR does not adequately meet the NuttX requirements.  While it provides a summary of what was changed, it lacks crucial details.
Here's why:

Insufficient Summary:  "Use common lock to protect the modify address to ensure consistency" states the how, but not why. What problem does this solve? Was there a race condition? Data corruption?  Which functional part of the code is impacted?  Are there any related NuttX issues?
Missing Impact Assessment: "N/A" is unacceptable.  Even seemingly minor changes can have impacts.  At a minimum, the author needs to justify why there's no impact. For example, the change likely impacts the arm architecture and potentially specific boards. This needs to be explicitly stated, even if the impact is deemed minimal.  Security implications of using a lock should also be considered (e.g., potential for deadlocks).
Inadequate Testing: "ci-check" is insufficient. While CI is important, it doesn't replace providing specific test results.  The PR needs to demonstrate how the change was tested locally.  What were the test cases?  What were the observed outputs before and after the change?  What host and target platforms were used for testing?

The author needs to significantly revise the PR description to meet the NuttX requirements.  Simply put, more context and detail is needed to properly evaluate the change.


Added ability to run sql queries on sqlite files

Added ability to run execution and selection sql queries during sqlite db exploration.

There's a lot of changes here. so when I have time, I'm going to review this on my machine and clean up the commit history a bit. I do want to merge this eventually though!

Great

Hmmm, it appears you didn't check the option to give maintainers access to your fork when you created it.

Okay, I got everything squashed into a single commit on master and properly rebased.
After looking over your code, it doesn't match our coding conventions. Can you go through each of the files you've added or changed and make sure you're following the style set out in this codebase? Some examples:

Do not indent @property declarations (I took care of this already)
Never put spaces between arguments and their labels

❌[self foo:  bar];
✅[self foo:bar];

Pointer markers go with the variable name, not the type

❌UILabel* textLabel;
✅UILabel *textLabel;

There is no need to manually @synthesize properties unless you override the getter and the setter (so, use self.isSelectionType instead of isSelectionType)
Prefer properties over ivars in almost all cases

Ok - I'll fix code style

I left some comments, there's a lot you missed 😅 I didn't mark them all, just a bunch I found.
Also, before you go any further, make sure your HEAD is at origin/master to avoid the merge commit. You don't need to merge, just fast forward. Lemme know if you need help with avoiding a merge commit.

Also, before you go any further, make sure your HEAD is at origin/master to avoid the merge commit. You don't need to merge, just fast forward. Lemme know if you need help with avoiding a merge commit.

Fixed comments you made - fixed, were could found.
Thanks, that you are interested in my pull request!

@revolter Can you tell me what this macro and code segment is for? 😅I'm trying to decipher some of this old code, I've never used the sqlite APIs
https://github.com/Flipboard/FLEX/blame/0de3a9d65c54533848b6721d0d66bc3fb0d740fa/Classes/GlobalStateExplorers/DatabaseBrowser/FLEXSQLiteDatabaseManager.m#L38

That macro is set by the SQLCipher pod and the code is used to decrypt a crypted database after opening it, or else any query would fail.

Thanks! I'm trying to add comments to these files

@LesykMelnychuk Can I ask why you added executeNonSelectQuery: and executeSelectionQuery:error: instead of making use of executeQuery:?
I've read the code more thoroughly now and there seems to be a lot of duplication now. At the very least, would you be able to make those two methods call into executeQuery:?

@NSExceptional Hi, cause executeSelectionQuery:error returns result of selection

Your executeSelectionQuery:error: is almost identical to the original executeQuery: unless I'm overlooking something. You return an error in an argument but that's the only difference

Your executeSelectionQuery:error: is almost identical to the original executeQuery: unless I'm overlooking something. You return an error in an argument but that's the only difference

Understud, - fixed.

Hello @NSExceptional , can you please merge this pull ?

Hey @LesykMelnychuk,
This PR isn't in a state where I'm ready to merge it. I have a fork of your branch I started working on a while back but I got sidetracked and it fell off my todo list. I will try to get it done and merged this week. (At the moment I don't even remember what I was working on specifically, probably adding comments or something.)

Shouldn't I fix the conflicts in #305, modify things if needed, merge it, and then rebase this, instead of having https://github.com/Flipboard/FLEX/pull/294/commits/f7659a8910dd1fc685f7ea180bdf9ea22719e905 here?

@revolter @LesykMelnychuk I'm currently in the process of rebasing and resolving conflicts between this branch and the 4.0 branch, should have it done tomorrow sometime

Accidentally named the branch 4.0-flex when I meant to just name it 4.0, so I renamed it just now, FYI

What I meant was that this PR included #305, so you merged 2 things at the same time, while #305 looks now like it wasn't merged.

Are you saying @LesykMelnychuk somehow included #305 in this original PR, and that I unknowingly discarded it when I made my changes and merged this just now?

Oooh, it was probably incorrectly rebased. Not it's ok, sorry I didn't check again, today. So it's all good 👍

I'm so confused :P What's wrong?

I am what's wrong 😆 Sorry for the confusion, and completely disregard this conversation. As I said, everything is ok 😃


[PE-D][Tester A] User Guide: Inaccurate format for exclude

Describe the bug
The format given for exclude is exclude ci/INDEX, [MORE_INDICES]. This slightly inaccurate since it's missing the ellipsis "..." following the [MORE_INDICES] to indicate that we can provide more than 1 extra indices.
Expected behaviour
The user guide should be updated to have a more accurate format.
Screenshots

Labels: type.DocumentationBug severity.Low
original: AgentHagu/ped#16

easy fix also


Istio analyze: Referenced selector not found: "app=istio-ingressgateway"

Bug Description
Bug description
Having a gateway living in the gateways namespace and istio-ingressgateway living in istio-system namespace.
The ingressgateway has the label app=istio-ingressgateway as follows
k get pod -n istio-system -l "app=istio-ingressgateway"
NAME                                            READY   STATUS    RESTARTS   AGE
istio-private-ingressgateway-8657c4fcdb-6h4qb   1/1     Running   0          20d
istio-private-ingressgateway-8657c4fcdb-bwkvp   1/1     Running   0          20d

The traffic works fine but when running istioctl analyze -n gateways the following error would show up:
Error [IST0101] (Gateway gateways/private-gateway) Referenced selector not found: "app=istio-ingressgateway"

What I have tried
I have verified that the PILOT_SCOPE_GATEWAY_TO_NAMESPACE flag in istiod configuration is set to false.
I have also tried putting my gateway inside the istio-system namespace, but it is still complaining about the same error.
Version
➜ istioctl version
client version: 1.15.2
istiod version: 1.16.0
istiod version: 1.16.0
pilot version: 1.15.2
data plane version: 1.16.0 (7 proxies)

Additional Information
No response
Affected product area

[ ] Ambient
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Extensions and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
[ ] Upgrade
[ ] Multi Cluster
[ ] Virtual Machine
[ ] Control Plane Revisions

cc @hanxiaop

Can you provide your Gateway and ingressgateway configurations?

Also, try using the latest istioctl to see if you are still receiving unexpected messages.

Thanks for the swift reply. A colleague is using the latest istioctl 1.17.2 and he is seeing those errors as well.
gateway.yaml
apiVersion: networking.istio.io/v1alpha3
kind: Gateway
metadata:
  name: gateway
  namespace: gateways
spec:
  selector:
    app: istio-ingressgateway
  servers:
    - hosts:
        - xxx
      port:
        number: 80
        name: http
        protocol: HTTP
      tls:
        httpsRedirect: true
    - hosts:
        - xxx
      port:
        number: 443
        name: https
        protocol: HTTPS
      tls:
        mode: SIMPLE
        minProtocolVersion: TLSV1_2
        credentialName: xxx

istioOperator.yaml
apiVersion: install.istio.io/v1alpha1
kind: IstioOperator
spec:
  profile: default
  components:
    ingressGateways:
      ...
      - enabled: true
        name: istio-private-ingressgateway
        k8s:
          hpaSpec:
            maxReplicas: 25
            minReplicas: 2
          strategy:
            rollingUpdate:
              maxSurge: 100%
              maxUnavailable: 0
          podDisruptionBudget:
            maxUnavailable: 20%
          overlays:
            ...
          service:
            type: NodePort
        label:
          app: istio-ingressgateway

Are you using revisions? If so, try to analyze with --revision

Tried on another cluster that uses istio 1-17 with the revision flag and got the same error
➜ ./istioctl analyze -n gateways --revision 1-17
Error [IST0101] (Gateway gateways/xxx-gw) Referenced selector not found: "istio=ingressgateway"

➜ ./istioctl version
client version: 1.17.2
istiod version: 1.15.0
istiod version: 1.15.0
istiod version: 1.17.0
istiod version: 1.17.0
data plane version: 1.17.0 (2139 proxies)

Do you have both revisioned and non-revisioned control planes? I cannot reproduce the issue if only one control plane exists, as both analyze and analyze --revision work fine.

Thank you very much. I have cleaned up my cluster, having only one version 1-17 of istio running and used the revision flag and the error resolved.


compiling semver-checks fails

Steps to reproduce the bug with the above code
cargo install cargo-semver-checks
Actual Behaviour
   --> /home/USER/.cargo/registry/src/github.com-1ecc6299db9ec823/cargo-semver-checks-0.15.2/src/baseline.rs:387:31
    |
387 |             if err.class() == git2::ErrorClass::Index && err.code() == git2::ErrorCode::Locked {
    |                               ^^^^^^^^^^^^^^^^^^^^^^^ expected enum `git2::ErrorClass`, found enum `ErrorClass`
    |
    = note: enum `ErrorClass` and enum `git2::ErrorClass` have similar names, but are actually distinct types
note: enum `ErrorClass` is defined in crate `git2`
   --> /home/USER/.cargo/registry/src/github.com-1ecc6299db9ec823/git2-0.15.0/src/lib.rs:225:1
    |
225 | pub enum ErrorClass {
    | ^^^^^^^^^^^^^^^^^^^
note: enum `git2::ErrorClass` is defined in crate `git2`
   --> /home/USER/.cargo/registry/src/github.com-1ecc6299db9ec823/git2-0.16.1/src/lib.rs:225:1
    |
225 | pub enum ErrorClass {
    | ^^^^^^^^^^^^^^^^^^^
    = note: perhaps two different versions of crate `git2` are being used?

error[E0308]: mismatched types
   --> /home/USER/.cargo/registry/src/github.com-1ecc6299db9ec823/cargo-semver-checks-0.15.2/src/baseline.rs:387:72
    |
387 |             if err.class() == git2::ErrorClass::Index && err.code() == git2::ErrorCode::Locked {
    |                                                                        ^^^^^^^^^^^^^^^^^^^^^^^ expected enum `git2::ErrorCode`, found a different enum `git2::ErrorCode`
    |
    = note: enum `git2::ErrorCode` and enum `git2::ErrorCode` have similar names, but are actually distinct types
note: enum `git2::ErrorCode` is defined in crate `git2`
   --> /home/USER/.cargo/registry/src/github.com-1ecc6299db9ec823/git2-0.15.0/src/lib.rs:165:1
    |
165 | pub enum ErrorCode {
    | ^^^^^^^^^^^^^^^^^^
note: enum `git2::ErrorCode` is defined in crate `git2`
   --> /home/USER/.cargo/registry/src/github.com-1ecc6299db9ec823/git2-0.16.1/src/lib.rs:165:1
    |
165 | pub enum ErrorCode {
    | ^^^^^^^^^^^^^^^^^^
    = note: perhaps two different versions of crate `git2` are being used?

For more information about this error, try `rustc --explain E0308`.
error: could not compile `cargo-semver-checks` due to 2 previous errors
error: failed to compile `cargo-semver-checks v0.15.2`, intermediate artifacts can be found at `/tmp/cargo-installBVlg0U`

Expected Behaviour
this to run without issue
Generated System Information
n/a
Build Configuration
n/a
Additional Context
No response

I've managed to reproduce the issue, I'm looking into it.

For now I'll say that the command cargo install cargo-semver-checks --locked  (from https://github.com/obi1kenobi/cargo-semver-checks#quick-start ) successfully compiles the tool @demoray.

@obi1kenobi I don't really understand why the tool fails to compile. When installing a non-locked version, for some reasons cargo installs two versions of git2 -- v0.16.1 and v0.15.0. But the Cargo.toml of the tool uses explicitly v0.15.0.

I've tried to raise the git2 version from v0.15.0 to v0.16.1 and after a local cargo clean I have the same compilation error, but the v0.15.0 shouldn't be used anymore...

Wow. Thanks for the report @demoray, sorry you had to deal with it. We're looking into it.
Most bin crate maintainers strongly recommend --locked when cargo install-ing anything, because many of us have found that despite our best efforts, stuff like this tends to happen from time to time. Occasionally even because of semver breaks in a dependency! 😅

By the way, it's quite ironic that a tool to detect semver breaks fails to compile because of semver breaks.

Just yet another piece of evidence for the fact that getting semver right is hard!
Also, I don't currently believe this specific instance is a semver violation. We seem to somehow be pulling in and using two different major versions of the same crate, and rustc correctly argues that their types are not equivalent.

While this crate uses git2 v0.15.0, the latest version of crates-index uses git2 v0.16.0.
If you run cargo update && cargo build, you will experience this issue.
Updating this crate to use git2 v0.16.0 along with updating the dependencies using cargo update resolves this issue.

Thanks! If you have a PR ready, I'd be happy to merge it and make a new release.
I'll also look into why the versions diverged like this so we can prevent something similar from happening again in the future.

Unfortunately, this was a semver issue after all. While we can update our requirement to git2 = "0.16", that won't prevent a similar problem in the future.
Fortunately, the upstream fix should be reasonably easy: https://github.com/frewsxcv/rust-crates-index/issues/91


[Dribbblish] What's the easiest way to switch positions of control buttons and title+cover section?

I would like to switch positions of control buttons section and cover+title section.
What would be the easiest way to achieve that in all views (normal, fullscreen etc.)?

You should swap order values here:
https://github.com/morpheusthewhite/spicetify-themes/blob/bed10d36d3fb21717616481e2467070970910fe4/Dribbblish/user.css#L1085-L1102

THX, it worked!


HBASE-28728 No data available when scan cross-region and setReversed(true) in Spark on HBase sc.newAPIHadoopRDD

HBASE-28728 No data available when scan cross-region and setReversed(true) in Spark on HBase sc.newAPIHadoopRDD

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 42s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  0s

codespell was not available.

+0 :ok:
detsecrets
0m  0s

detect-secrets was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

+1 :green_heart:
hbaseanti
0m  0s

Patch does not have any anti-patterns.

_ master Compile Tests _

+1 :green_heart:
mvninstall
3m  0s

master passed

+1 :green_heart:
compile
0m 40s

master passed

+1 :green_heart:
checkstyle
0m 15s

master passed

+1 :green_heart:
spotbugs
0m 36s

master passed

+1 :green_heart:
spotless
0m 45s

branch has no errors when running spotless:check.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
2m 55s

the patch passed

+1 :green_heart:
compile
0m 39s

the patch passed

+1 :green_heart:
javac
0m 39s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

+1 :green_heart:
checkstyle
0m 13s

the patch passed

+1 :green_heart:
spotbugs
0m 41s

the patch passed

+1 :green_heart:
hadoopcheck
10m 42s

Patch does not cause any errors with Hadoop 3.3.6 3.4.0.

+1 :green_heart:
spotless
0m 44s

patch has no errors when running spotless:check.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
4m 26s

the patch passed

+1 :green_heart:
compile
0m 30s

the patch passed

+1 :green_heart:
javac
0m 30s

the patch passed

+1 :green_heart:
javadoc
0m 23s

the patch passed

+1 :green_heart:
shadedjars
7m 40s

patch has no errors when building our shaded downstream artifacts.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
3m  4s

the patch passed

+1 :green_heart:
compile
0m 20s

the patch passed

+1 :green_heart:
javac
0m 20s

the patch passed

+1 :green_heart:
javadoc
0m 14s

the patch passed

+1 :green_heart:
shadedjars
5m 46s

patch has no errors when building our shaded downstream artifacts.

_ Other Tests _

-1 :x:
unit
19m  4s
/patch-unit-hbase-mapreduce.txt
hbase-mapreduce in the patch failed.

39m 47s

Subsystem
Report/Notes

Docker
ClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6079/2/artifact/yetus-jdk17-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/6079

Optional Tests
javac javadoc unit compile shadedjars

uname
Linux 34a310af4643 5.4.0-1103-aws #111~18.04.1-Ubuntu SMP Tue May 23 20:04:10 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / a3d59b8ea98f2e3a847339fb24f73e530e219d69

Default Java
Eclipse Adoptium-17.0.11+9

Test Results
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6079/2/testReport/

Max. process+thread count
3217 (vs. ulimit of 30000)

modules
C: hbase-mapreduce U: hbase-mapreduce

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6079/2/console

versions
git=2.34.1 maven=3.9.8

Powered by
Apache Yetus 0.15.0 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 43s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  0s

codespell was not available.

+0 :ok:
detsecrets
0m  0s

detect-secrets was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

+1 :green_heart:
hbaseanti
0m  0s

Patch does not have any anti-patterns.

_ master Compile Tests _

+1 :green_heart:
mvninstall
2m 55s

master passed

+1 :green_heart:
compile
0m 38s

master passed

+1 :green_heart:
checkstyle
0m 13s

master passed

+1 :green_heart:
spotbugs
0m 33s

master passed

+1 :green_heart:
spotless
0m 44s

branch has no errors when running spotless:check.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
2m 51s

the patch passed

+1 :green_heart:
compile
0m 36s

the patch passed

+1 :green_heart:
javac
0m 36s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

+1 :green_heart:
checkstyle
0m 12s

the patch passed

+1 :green_heart:
spotbugs
0m 40s

the patch passed

+1 :green_heart:
hadoopcheck
10m 26s

Patch does not cause any errors with Hadoop 3.3.6 3.4.0.

+1 :green_heart:
spotless
0m 43s

patch has no errors when running spotless:check.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
3m  1s

the patch passed

+1 :green_heart:
compile
0m 20s

the patch passed

+1 :green_heart:
javac
0m 20s

the patch passed

+1 :green_heart:
javadoc
0m 13s

the patch passed

+1 :green_heart:
shadedjars
5m 40s

patch has no errors when building our shaded downstream artifacts.

_ Other Tests _

+1 :green_heart:
unit
17m 56s

hbase-mapreduce in the patch passed.

38m  6s

Subsystem
Report/Notes

Docker
ClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6079/3/artifact/yetus-jdk17-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/6079

Optional Tests
javac javadoc unit compile shadedjars

uname
Linux 1f89d46f52c0 5.4.0-1103-aws #111~18.04.1-Ubuntu SMP Tue May 23 20:04:10 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / cb2e0c04526dcdd54fa78bcc6abf44a0410d95b4

Default Java
Eclipse Adoptium-17.0.11+9

Test Results
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6079/3/testReport/

Max. process+thread count
3245 (vs. ulimit of 30000)

modules
C: hbase-mapreduce U: hbase-mapreduce

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6079/3/console

versions
git=2.34.1 maven=3.9.8

Powered by
Apache Yetus 0.15.0 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 40s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  0s

codespell was not available.

+0 :ok:
detsecrets
0m  0s

detect-secrets was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

+1 :green_heart:
hbaseanti
0m  0s

Patch does not have any anti-patterns.

_ master Compile Tests _

+1 :green_heart:
mvninstall
3m 13s

master passed

+1 :green_heart:
compile
0m 37s

master passed

+1 :green_heart:
checkstyle
0m 13s

master passed

+1 :green_heart:
spotbugs
0m 34s

master passed

+1 :green_heart:
spotless
0m 43s

branch has no errors when running spotless:check.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
2m 50s

the patch passed

+1 :green_heart:
compile
0m 36s

the patch passed

+1 :green_heart:
javac
0m 36s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

+1 :green_heart:
checkstyle
0m 12s

the patch passed

+1 :green_heart:
spotbugs
0m 39s

the patch passed

+1 :green_heart:
hadoopcheck
10m 10s

Patch does not cause any errors with Hadoop 3.3.6 3.4.0.

+1 :green_heart:
spotless
0m 43s

patch has no errors when running spotless:check.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
3m  3s

the patch passed

+1 :green_heart:
compile
0m 20s

the patch passed

+1 :green_heart:
javac
0m 20s

the patch passed

+1 :green_heart:
javadoc
0m 14s

the patch passed

+1 :green_heart:
shadedjars
5m 43s

patch has no errors when building our shaded downstream artifacts.

_ Other Tests _

+1 :green_heart:
unit
18m 19s

hbase-mapreduce in the patch passed.

39m 11s

Subsystem
Report/Notes

Docker
ClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6079/4/artifact/yetus-jdk17-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/6079

Optional Tests
javac javadoc unit compile shadedjars

uname
Linux 58f780d42d0c 5.4.0-1103-aws #111~18.04.1-Ubuntu SMP Tue May 23 20:04:10 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / 684b7230e729dc0e9794e0a1360b54521359c8f9

Default Java
Eclipse Adoptium-17.0.11+9

Test Results
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6079/4/testReport/

Max. process+thread count
3218 (vs. ulimit of 30000)

modules
C: hbase-mapreduce U: hbase-mapreduce

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6079/4/console

versions
git=2.34.1 maven=3.9.8

Powered by
Apache Yetus 0.15.0 https://yetus.apache.org

This message was automatically generated.


Add link to "linkage" documentation in Creating Resources

Resolves gh-1162

/cc @lanej

@jmeas - thanks for following up here! Sorry to take so long to merge.

No worries @dgeb . Thanks! ✌


Add QE test mappings for ISC components

Update QE test mappings for ISC components Compaliance Operator, File Integrity Operator, Security Profiles Operator, and oc-compliance.
One junit example:
https://mastern-jenkins-csb-openshift-qe.apps.ocp-c1.prod.psi.redhat.com/job/ocp-common/job/ginkgo-test/266121/artifact/private/junit_e2e_2024-09-01-23-48-48.xml

/ok-to-test

@stbenjam Could you please help to review? Thanks.

@jhou1  @stbenjam  @xueqzhan Could you please help to review? Thanks.

It doesn't look like this changed any mappings.  We're not importing anything from Jenkins AFAIK.

/test mapping

/test mapping

/lgtm

/approve


[4.2.0][Dependency Upgrade]libthrift version upgrade

Description
This ticket is created to track progress on libthrift version upgrade from 0.92.0 in SI 4.2.0.
Affected Component
SI
Version
4.2.0
Related Issues
No response
Suggested Labels
No response

Maybe fixed with https://github.com/wso2/carbon-analytics-common/pull/819. We need to get this verified from someone with thrift expertise.


LinuxCommand: fix typo of 01ls

Thank you for pointing out these typos.
And the have been corrected in recent commits while I'm normalize the format of all the exsiting docs. You can check if there are other typos.
Closes #4


[DO NOT REVIEW] [WIP] ODSC-64654 register model artifact reference

This Pull Request is for adding new API for Model by Reference, i.e. register_model_artifact_reference in DataScienceModel in ADS.
The new API will enable users to simply give us the object storage locations of their existing artifacts as reference for using against a newly created DataScienceModel, instead of having to upload or export artifacts which is a costly operation.
API Spec :- https://confluence.oci.oraclecorp.com/pages/viewpage.action?spaceKey=ODSC&title=API+changes+for+Model+store+for+Aqua
It has been tested in my local environment - Attached Screenshots. One screenshot is for successful scenario, and another is for one of the failure scenarios to demonstrate that in case of failure, we are fetching the work request logs and showing the same in ADS logs.

@tkg2261 please add PR description.

@tkg2261 please add PR description.


Example usage of F#+ Cont

Ported examples to F#+

What do you think @gusty ? I think the f#+ variant looks similar to the extcore one, but nicer due to the Cont.run method in order to switch back to non cont context.

I'm glad that you like the Cont.run, the difference is of course, because in order to be able to generalize, we're using here the Haskell approach, which means wrapping and unwrapping, so we 'mark' the types instead of the operations.
I also find it nice, but it's true that when stacking many monads it becomes a bit cumbersome, although it avoids a looooot of boilerplate code.


remove iot poc from cn470

Stakeholders: This probably needs to specifically call out makers / distributors of CN470 Hotspots.
Deployment Impact: This should probably mention the makers/distributors again here but the governance process should provide ample opportunity to provide feedback and vote.


Focus window on scrolling

Describe your PR, what does it fix/add?
This is the new PR for #8279. The other PR introduced a regression on merge which this implementation addresses as well.
In short: You are now able to focus a window on scroll (like with mouse-movement and button-clicks). This particularly happens, after you switched your workspace and returned back.
What does it better? It now calls the simulateMouseMovement only once when the window differ and also when mouse_follow event is set to 1. This should avoid weard touchpad behaviour happening in firefox when you constantly call simulateMouseMovement.
Is it ready for merging, or does it need work?

Tested it locally with mouse and touchpad on my Surface Pro 8
Scrolling in Firefox works also on touchpad (was the regression)

No Problem. I currently have the opportunity to contribute to open source projects so if there are some "manageable" issues or task, I would be down to look at them

there are 1.1k issues open, pick one you like :P


V4 How to keep location.state when refresh?

Version
4.0.0
Test Case
https://codepen.io/ReactJSTraining/pen/jyYjeW
Steps to reproduce
Expected Behavior
Actual Behavior

You can't. It's intentionally not stored in the URL and does not survive a refresh. If you need something to survive a refresh, use the search property and a query string.

I'm not sure what is going on with your code, but location.state persists across refreshes. All that history does is to grab the state property from window.history.state (history stores state and key properties for locations) .


Add gap between columns in two-cols-* layouts

This way the background of code boxes don't butt against each other.

I am not very sure. I feel it should leave to the users to decide.

That arguments kinda works in both ways. The user can customize the default by adding or removing any gap. I however think having a gap is the better default over not having a gap.

You can hack one in easier (margin on both columns), but adding a gap using the proper css (where it's one space properly aligned) takes the same amount of work of a custom layout as far as I see.

I second this. It would be great to have an appropriate gap by default.

Hello, I solved it by creating a custom layout two-cols-with-gap.vue in the layouts folder:

<template>

        <slot name="left" />

        <slot name="right" />

</template>

The new layout can then be applied in a slide like so:
---
layout: two-cols-with-gap
hideInToc: true
---

Cheers


ImThemes won't open sometimes

Sometimes when I go to use ImThemes, it just won't open. I've tried running it normally, as admin, and even running it in cmd to see if it outputted any errors, but nothing. I've tried building it from source, but nimble keeps complaining about "invalid indentation".
Heres an example with Process Hacker open:

(running as admin stopped the recording, but it didn't open even running as admin)
Any fixes/solutions?

I have this issue too idk what to do abt it

C:\Users\User\AppData\Local
and delete "ImThemes"

@yhqe and @paysonism, the issue is probably related to the configuration/cache files that are stored in the directory mentioned by Fiendssov C:\Users\User\AppData\Local\ImThemes. To prevent this issue in future versions it would be really helpful if you could send the files of that folder to me.
Thank you and sorry for answering this late.


logger: add critical log level support

When changing the loglevel in the tcmu.conf for the dynamic logger,
we need to verify it is really working. Currently this hit log is
one INFO level message, which won't work in the ERR and WARN loglevel
cases.
Signed-off-by: Xiubo Li<EMAIL_ADDRESS>

@mikechristie @pkalever
Please review, thanks.

@pkalever Yeah, update and exposed it.
Thanks.


Question - PHPUnit Test $response->redirect()?

Hi,
So I have integrated Omnipay Paypal and it redirects successfully but I wanted to write some PHPUnit code to check that a redirect happens but I seem to be having problems working out a way to test it.
My code that I'm trying to test is as follows:
$paypal = new PayPal;
$response = $paypal->purchase(request()->all(), Cart::total());

if ($response->isRedirect()) {
    $response->redirect();
}

So I'm trying to check for a redirect in PHPUnit but it doesn't seem to be doing one?
$response = $this->followingRedirects()->post(route('cart.pay', $data));
$response->assertRedirect();

And the redirect text just gets dumped in to the unit test as follows:
<!DOCTYPE html>

    <meta charset="UTF-8" />
    <meta http-equiv="refresh" content="0;url=https://www.sandbox.paypal.com/cgi-bin/webscr?cmd=_express-checkout&useraction=commit&token=EC-3B700573HD007635J" />

    <title>Redirecting to https://www.sandbox.paypal.com/cgi-bin/webscr?cmd=_express-checkout&useraction=commit&token=EC-3B700573HD007635J</title>

    Redirecting to https://www.sandbox.paypal.com/cgi-bin/webscr?cmd=_express-checkout&useraction=commit&token=EC-3B700573HD007635J.

Is there a way for me to test that my code is actually redirecting users?
The code in the file AbstractResponse.php for Omnipay calls the following:
public function redirect()
{
    $this->getRedirectResponse()->send();
}

If I do dd($this->getRedirectResponse()->send()); then I can see that there is a redirect there and can see the #targetURL etc but if I do dd($response->redirect()) in my code, it returns null. Tres confused!

Is there any issue in me using return redirect($response->getRedirectURL()); instead of $response->redirect(); so that I can PHPUnit test my code?


[fact] Convert content resolver to kotlin

#155

Codecov Report

Merging #165 (25ca4cb) into master (a7393e4) will decrease coverage by 0.06%.
The diff coverage is 94.87%.

@@             Coverage Diff              @@
##             master     #165      +/-   ##
============================================
- Coverage     81.74%   81.67%   -0.07%
+ Complexity      428      413      -15
============================================
  Files            43       43
  Lines          1764     1763       -1
  Branches        275      276       +1
============================================
- Hits           1442     1440       -2
- Misses          195      196       +1
  Partials        127      127

Impacted Files
Coverage Δ
Complexity Δ

.../main/kotlin/se/kth/spork/spoon/contentresolver.kt
93.93% <93.93%> (ø)
0.00 <0.00> (?)

src/main/kotlin/se/kth/spork/base3dm/ChangeSet.kt
87.03% <100.00%> (ø)
23.00 <1.00> (ø)

...rc/main/kotlin/se/kth/spork/spoon/Spoon3dmMerge.kt
98.66% <100.00%> (+0.65%)
19.00 <1.00> (ø)

.../kotlin/se/kth/spork/spoon/wrappers/NodeFactory.kt
80.50% <0.00%> (-1.70%)
22.00% <0.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 a7393e4...25ca4cb. Read the comment docs.


Add last seed to layer name

Is your feature request related to a problem? Please describe.
I don't see a way to get the last used random seed. Having the seed allows refining a generated image further.
Describe the solution you'd like
The easiest way would be to add the seed to the name of the new layer. Even better would be the seed and the prompt.
Describe alternatives you've considered
A 'reuse seed' button like the one in the webui.

Somehow I never thought of this.


How to rotate rectangles?

I am new to sdl_gpu and I know I can draw rectangles using GPU_RectangleFilled2
How would I go about rotating this? Thank you in advanced

You can use GPU_Rotate to rotate the matrix, then all proceeding rendering would be rotated. You can rotate the matrix back when you're done

Thank you very much!

So GPU_Rotate works, but is there any way to control the speed of the rotation?

If you want to rotate over time to sort of "spin", you'd just have to rotate using higher values to rotate "faster". This also depends on how quickly you're calling the function.

OK so I've been trying to use rotation for some time now and i can't really get it to work that well. Could you please send some example code? Thank you very much in advanced


Can't print plain text file from Linux command-line

I am running Ubuntu 20.04. cups has been installed from aptitude an it is up-to-date.
When trying to print from a command line a plain text file, the cups texttopdf filter crashes.
I am using a network printer, Brother MFC-L2710DN. The print attempt was: lpr -P Brother_MFC_L2710DN_series testprintfile.
I write here as suggested in this reply from the cups Mailing List.
Here are:

the file I would like to print, testprintfile.txt;
the latest file from /var/spool/cups which starts with 'd', d00022-001.txt;
the ppd file of my print queue, Brother_MFC_L2710DN_series.ppd.txt.

Note that all the above files originally didn't have the .txt extension: I added it just to let Github upload them (files with no extension are not supported).

I would also like to highlight (from another reply to the same thread in the Mailing List) that:

the PDF generating filters should revert to embedding full fonts if subsetting is not supported

In fact, it seems that

texttopdf filter forces subsetting when embedding fonts

but the

font embed part of the sources states that subsetting with OTF fonts is not yet supported.

Also see Message 11 at the link given in https://github.com/OpenPrinting/cups-filters/issues/248 and https://github.com/apple/cups/issues/5787. The later link gives a workaround.

I checked them out, thank you. Trying the first workaround (the quicker one, moving the font away from /usr/share/fonts/truetype), I successfully printed a text file from terminal.
However, this is just a temporary solution. I wish the code can be modified to fix this in a permanent way.

@smilingthax, it seems that certain fonts make texttopdf crash, simply by their presence, even if they are not actually used by the filter. See the issues/threads linked by @debiantriage. There is an emoji font triggering an assert() in fontembed. assert() should never be used in production code, it is only for development and debugging. Also a font not needed for the filtering process should not stop the filter from working. Could you have a look.

Fixed by pull request #276.

@jarviswyatt, could you test this fix and report here? Thanks in advance.


Add Japan 4swordshax support

My minitwlpayload didn't turn out as small as what fincs reported (4564 vs. 5524 for me), but it is still an improvement in size and did actually work with recent-libnds homebrew.

Make sure you are using the latest devkitARM (r47 at the time of writing).

Yeah, devkitpro/arm was freshly installed on my linux machine just to get this compiled.

devkitPro is not a program. It is the name of the organization that makes and distributes devkitARM and the tools

Sure, right here.
payload.zip

Ok, I see what happened. The '4564' figure is the size of the bootloader (minitwlpayload/data/ndsloader.bin), not the entire payload (minitwlpayload/payload.dat).


Switch to react-admin as template for frontend

https://github.com/marmelab/react-admin

Maybe not


pytest.warns requires at least pytest 2.8

Please merge this to your branch so it can be included in https://github.com/pallets/click/pull/838.  Thanks!
https://github.com/pytest-dev/pytest/blob/2.8.0/doc/en/recwarn.rst#asserting-warnings-with-the-warns-function

Thanks @carlwgeorge .


passthrough props?

This component does a great job wrapping the react-native-video component, and adding visually appealing UI components. Any thoughts on enabling an ability to pass through props to the react-native-video component, so we can leverage some of the other props that are available on the video player?

Yep, i'll add support for destructuring operator support. :)

@ppatel890 2 years later but released a new 2.0.0 version that could help to solve this issue, also with full TS support.
Video is now external


[FEATURE] Generate resources.json files for libs, components, themes

Introduces new task "generateResourcesJson" which creates the file resources.json.
It contains the list of resources of the build with meta information.

Coverage decreased (-1.4%) to 89.226% when pulling b7cb9702b0227a06cb4a6ac398ca72744d291796 on create-resources-json into b0ff7b8c0a46f39397126b368f7490c6cb342ce5 on master.

Note: failing test requires #391.

I think I won't be able to review the functional correctness of this PR in coming week. I simply don't know enough about what this task does and should do.
I'm surprised by the amount of new test fixtures. But for the same reason as above I can't tell whether they are needed or not.
But I tried something that might be interesting: I removed the new task execution from ApplicationBuilder.js, LibraryBuilder.js and ThemeLibraryBuilder.js. With that change, the code coverage for ResourceCollector.js, ResourceInfo.js and resourceListCreator.js went from almost 100% to below 70% each. This gives me the impression that some of the new functions are only tested through integration tests and do not have dedicated unit tests:

For new modules we typically try to have at least one unit test per function. First, to efficiently test a variety of inputs. And secondly, to have failing tests which already contain the name of faulty function instead of a more generic integration test which first needs to be debugged on its own to identify the issue.

Building the openui5-sample-app with this change currently fails with an error:
⚠  Process Failed With Error

Error Message:
resources.json generation failed with error: There are 1 resources which could not be assigned to components.

Stack Trace:
Error: resources.json generation failed with error: There are 1 resources which could not be assigned to components.
    at module.exports (/Users/d054485/SAPDevelop/dev/meta-ui5-tooling/ui5-builder/lib/processors/resourceListCreator.js:183:9)
    at processTicksAndRejections (internal/process/task_queues.js:93:5)
    at async Object.build (/Users/d054485/SAPDevelop/dev/meta-ui5-tooling/ui5-builder/lib/builder/builder.js:370:4)
    at async Object.handleBuild [as handler] (/Users/d054485/SAPDevelop/dev/meta-ui5-tooling/ui5-cli/lib/cli/commands/build.js:97:2)

The errors logged during the build are:
ERR! lbt:resources:ResourcePool TypeError: Cannot read property 'type' of undefined
ERR! lbt:resources:ResourcePool     at new SapUiDefineCall (/ui5-builder/lib/lbt/calls/SapUiDefine.js:34:16)
ERR! lbt:resources:ResourcePool     at XMLCompositeAnalyzer.analyze (/ui5-builder/lib/lbt/analyzer/XMLCompositeAnalyzer.js:19:23)
ERR! lbt:resources:ResourcePool     at determineDependencyInfo (/ui5-builder/lib/lbt/resources/ResourcePool.js:71:35)
ERR! lbt:resources:ResourcePool     at processTicksAndRejections (internal/process/task_queues.js:93:5)
ERR! lbt:resources:ResourcePool     at async LocatorResourcePool.getModuleInfo (/ui5-builder/lib/lbt/resources/ResourcePool.js:175:11)
ERR! lbt:resources:ResourcePool     at async Promise.all (index 9)
ERR! lbt:resources:ResourcePool     at async module.exports (/ui5-builder/lib/processors/resourceListCreator.js:117:2)
ERR! lbt:resources:ResourcePool     at async Object.build (/ui5-builder/lib/builder/builder.js:370:4)
ERR! lbt:resources:ResourcePool     at async Object.handleBuild [as handler] (/ui5-cli/lib/cli/commands/build.js:97:2)
ERR! lbt:resources:ResourcePool  failed to parse or analyze %s: sap/ui/demo/todo/test/unit/AllTests.js TypeError: Cannot read property 'type' of undefined
ERR! lbt:resources:ResourcePool     at new SapUiDefineCall (/ui5-builder/lib/lbt/calls/SapUiDefine.js:34:16)
ERR! lbt:resources:ResourcePool     at XMLCompositeAnalyzer.analyze (/ui5-builder/lib/lbt/analyzer/XMLCompositeAnalyzer.js:19:23)
ERR! lbt:resources:ResourcePool     at determineDependencyInfo (/ui5-builder/lib/lbt/resources/ResourcePool.js:71:35)
ERR! lbt:resources:ResourcePool     at processTicksAndRejections (internal/process/task_queues.js:93:5)
ERR! lbt:resources:ResourcePool     at async LocatorResourcePool.getModuleInfo (/ui5-builder/lib/lbt/resources/ResourcePool.js:175:11)
ERR! lbt:resources:ResourcePool     at async Promise.all (index 9)
ERR! lbt:resources:ResourcePool     at async module.exports (/ui5-builder/lib/processors/resourceListCreator.js:117:2)
ERR! lbt:resources:ResourcePool     at async Object.build (/ui5-builder/lib/builder/builder.js:370:4)
ERR! lbt:resources:ResourcePool     at async Object.handleBuild [as handler] (/ui5-cli/lib/cli/commands/build.js:97:2)
ERR! lbt:resources:ResourcePool TypeError: Cannot read property 'type' of undefined
ERR! lbt:resources:ResourcePool     at new SapUiDefineCall (/ui5-builder/lib/lbt/calls/SapUiDefine.js:34:16)
ERR! lbt:resources:ResourcePool     at XMLCompositeAnalyzer.analyze (/ui5-builder/lib/lbt/analyzer/XMLCompositeAnalyzer.js:19:23)
ERR! lbt:resources:ResourcePool     at determineDependencyInfo (/ui5-builder/lib/lbt/resources/ResourcePool.js:71:35)
ERR! lbt:resources:ResourcePool     at processTicksAndRejections (internal/process/task_queues.js:93:5)
ERR! lbt:resources:ResourcePool     at async LocatorResourcePool.getModuleInfo (/ui5-builder/lib/lbt/resources/ResourcePool.js:175:11)
ERR! lbt:resources:ResourcePool     at async Promise.all (index 23)
ERR! lbt:resources:ResourcePool     at async module.exports (/ui5-builder/lib/processors/resourceListCreator.js:117:2)
ERR! lbt:resources:ResourcePool     at async Object.build (/ui5-builder/lib/builder/builder.js:370:4)
ERR! lbt:resources:ResourcePool     at async Object.handleBuild [as handler] (/ui5-cli/lib/cli/commands/build.js:97:2)

I also did some benchmarking using:
OpenUI5 sap.m library

ui5-builder ref
Command
Mean [s]
Min [s]
Max [s]

master (e7d4431)
ui5 build
24.354 ± 2.094
21.244
26.661

create-resources-json (18fb726)
ui5 build
27.820 ± 1.174
26.073
30.163

OpenUI5 Sample App (+ 6 dependencies)
(skipping the generateResourcesJson task for applications due to the error above)

ui5-builder ref
Command
Mean [s]
Min [s]
Max [s]

master (e7d4431)
ui5 build --all
65.268 ± 2.128
60.574
68.159

create-resources-json (18fb726)
ui5 build --all
74.221 ± 1.947
70.642
77.307

Benchmark tool: hyperfine with one warmup
Hardware: MacBook Pro 2017 - i7 2,9GHz/16GB

The error seen during the build of the openui5-sample-app should be fixed with https://github.com/SAP/ui5-builder/pull/491.

@codeworrior and I discussed the benchmark results:

The resources.json is not consumed at runtime. Only some known tools make use of the file.

This means many UI5 Tooling consumer do not need it

Both measurements show an increase in runtime of about 14%.
This task might fail for some projects that are currently working fine

For example a library with multiple namespaces would need a way to configure those (currently not possible). Otherwise the build will likely fail

Both, @codeworrior and I therefore think it would be better to skip this task by default, similar to the generateManifestBundle task. For this it needs to be added to the list here: https://github.com/SAP/ui5-builder/blob/master/lib/builder/builder.js#L54
Btw. should we document the structure of the resources.json somewhere? Do we expect other tools to make use of it? I'm not sure.

@RandomByte Reg. your comment:

Btw. should we document the structure of the resources.json somewhere? Do we expect other tools to make use of it? I'm not sure.

To be honest, I didn't remember this, but a search in our Wiki revealed an existing documentation in the SDK.
The documented entries had been the "public" ones at the time of writing. But I guess we could enhance the documentation with a few more properties. A JSON schema also could make sense, but low priority as no one would use it currently (and the file is not intended to be maintained manually).

I cherry-picked the commits from https://github.com/SAP/ui5-builder/pull/491 onto this branch and tried building the openui5-sample-app again using ui5 build --include-task=generateResourcesJson --verbose.
It fails with a different error now:
[...]
verb lbt:analyzer:ComponentAnalyzer adding library dependency  sap/m/library.js false
verb lbt:analyzer:ComponentAnalyzer adding library dependency  sap/f/library.js false
verb lbt:analyzer:ComponentAnalyzer adding library dependency  sap/ui/unified/library.js false
verb lbt:analyzer:ComponentAnalyzer derived model implementation dependency  sap/ui/model/resource/ResourceModel.js
verb lbt:analyzer:ComponentAnalyzer derived model implementation dependency  sap/ui/model/json/JSONModel.js
verb lbt:resources:ResourceCollector   configured external resources filters (resources outside the namespace): (none)
verb lbt:resources:ResourceFilterList filetypes: undefined
verb lbt:resources:ResourceFilterList   sap-ui-core-dbg.js --> include: "sap-ui-core-dbg.js"
verb builder:processors:resourceListCreator     writing 'sap/ui/demo/todo/resources.json'
ERR! builder:processors:resourceListCreator resources.json generation failed because of unassigned resources: sap-ui-version.json
ERR! builder:builder Build failed in 523 ms
info builder:builder Executing cleanup tasks...

⚠  Process Failed With Error

Error Message:
resources.json generation failed with error: There are 1 resources which could not be assigned to components.

Stack Trace:
Error: resources.json generation failed with error: There are 1 resources which could not be assigned to components.
    at module.exports (/ui5-builder/lib/processors/resourceListCreator.js:198:9)
    at processTicksAndRejections (internal/process/task_queues.js:93:5)
    at async Object.module.exports [as task] (/ui5-builder/lib/tasks/generateResourcesJson.js:88:24)
    at async Object.build (/ui5-builder/lib/builder/builder.js:382:4)
    at async Object.handleBuild [as handler] (/ui5-cli/lib/cli/commands/build.js:97:2)

This seems to be related to sap-ui-version.json.
I also don't really like that the ultimate error message is the same as before, even though it's seemingly caused by something else(?)
This will be the error message people will open issues with. Currently it doesn't seem to be very helpful to us.


[FEATURE] add exclusion to application cache strategy

feature "exclude" to application cache strategy
minor improvements
bugfix when cache.put fails
bugfix when cache.get fails on safari (cache being undefined)

I've been adding some bugfixes to "my" forked main, which apparently automatically ended up in this PR.
let me know if these need to be separated in several PRs
would be nice if someone could have a look at this


Update quickstart-powershell.md

Modified typo

@tsubasaxZZZ : Thanks for your contribution to the Azure documentation! The author, @maertendmsft, has been notified to review your proposed change.

@tsubasaxZZZ thanks for the PR
#sign-off


dependent fields do not work when the form is an inline admin form

I found a problem with dependent_fields when using django-select2 as an inline admin form. The result is that the select options are never updated to filter based on the dependent field.

I have created a PR to fix this PR-379

is this fixed? i'm having difficulty populating dependant fields in django-admin. To be honest I can get the select to work at all in admin... thanks

@gwsampso I believe this should be fixed. Besides, I would recommend to use autocomplete fields in the Django admin.


segmentation fault when import pytorch_lightning

I am trying the very minimum code which imports torch and pytorch_lightning, with the following code:
import pytorch_lightning as pl

The import of pytorch_lightning fails immediately with the error: 29144 segmentation fault (core dumped)  python
I am using pytorch-dev 1.7.0 as it is required for cuda11 with new GPUs, and I installed pytorch-lightning from source.

OS: Linux
Packaging: pip
Version 1.0.2

Hi @zye1996 thank you for reporting.
While we have nightly support for PyTorch 1.7, we have only tested in cpu environment. Also we currently only support CUDA 10.1 and 10.2 just like pytorch did. So this issue may come from CUDA 11 + PyTorch 1.7-dev as the binaries are not built for CUDA 11 yet.
https://discuss.pytorch.org/t/pytorch-with-cuda-11-compatibility/89254/4
If you don't have a specific requirment to use cuda 11, it'd probably better to use pytorch 1.6/1.7-dev and cuda 10.1 or 10.2.

I ran into this same issue, and fixed it by updating torch under the hood:
conda install pytorch torchvision torchaudio -c pytorch


SSR/History.js - trying to modify constant variable in a loop

https://github.com/skatejs/skatejs/blob/eae5f683d31ad9ef0f2e928edf491352dc12697d/packages/ssr/register/History.js#L31

Good catch! Maybe that part is untested?

Yeah, probably. The SSR stuff is still pretty alpha hence still being in dev releases. Would love a PR fixing it, if possible.

I fixed this in the dev branch.


Win11pro 22631.4460 1000.22700.1047.0 Not Supported

termsrv.zip

Issue title does not match the version on screenshot => invalid.
10.0.22621.4391 is tracked in https://github.com/stascorp/rdpwrap/issues/3316 => duplicate.


Discovery crashes when on different port

Crash when running discover on port 3307
When discovery on port 3307, orchestrator crashes with memory nil pointer

panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x0 pc=0x962c87]

goroutine 1 [running]:
github.com/github/orchestrator/go/app.Cli(0x7ffd025eb8e3, 0x8, 0x0, 0x7ffd025eb8ef, 0x1a, 0x0, 0x0, 0x0, 0x0, 0x0, ...)
        /home/shlomi-noach/dev/go/src/github.com/github/orchestrator/go/app/cli.go:1262 +0xe787
github.com/github/orchestrator/go/app.CliWrapper(0x7ffd025eb8e3, 0x8, 0x0, 0x7ffd025eb8ef, 0x1a, 0x0, 0x0, 0x0, 0x0, 0x0, ...)
        /home/shlomi-noach/dev/go/src/github.com/github/orchestrator/go/app/cli.go:149 +0x23e
main.main()
        /home/shlomi-noach/dev/go/src/github.com/github/orchestrator/go/cmd/orchestrator/main.go:162 +0xe4f

Version
[root@dba /var/log]# orchestrator --version
3.0.2
992507c56e99fedb4660968c8a293e9a6b290df0

that's unexpected, of course. Line at fault: https://github.com/github/orchestrator/blob/v3.0.2/go/app/cli.go#L1262
Will look into it.

I'm just wondering, since this issue is titled "...when on different port" -- does it not crash for you when you're discovering on port 3306?
I'm unable to reproduce this. It seems like this would happen when ReadTopologyInstance would return nil for instance and nil for error, which doesn't make sense.
Is there an actual server at this host:3307?

@shlomi-noach - yes, only when using a port other than the default and yes, there is a server on the host at port 3307

https://github.com/github/orchestrator/pull/325 will mitigate by producing an error if instance is not discoverable -- but I still can't figure out why this happened to you.
FWIW, I'm running and testing on non-3306 ever since orchestrator was created, so using non-standard ports is nothing new. I'm unable to reproduce locally.
@djuntgen any hint on your side as for what other difference there is would be helpful. Different kind of proxy on 3307? Different firewall rules? Privileges?

@shlomi-noach - Yes - I found the issue:
Add the following to your .json config, the error will occur. the host.
{
   "HostnameResolveMethod": "default"
   "MySQLHostnameResolveMethod": "@@report_host"
}

By removing both lines the discovery works.

"MySQLHostnameResolveMethod": "@@report_host"

Do you have report_host configured on your box in the first place?
I prefer using
{
   "HostnameResolveMethod": "default"
   "MySQLHostnameResolveMethod": "@@hostname"
}

Apparently, I do not for the server running on port 3307. The reason for using @@report_host is that I am monitoring mysql servers that are within a third party data Center NAT'ed over VPN. The hostname of the machines is meaningless, VMs with random hostnames. Also, the local @@hostname is not resolvable from the Orchestrator service running on my server, DNS is not configured to reach over the VPN.
Is there a way to use @@report_host for an aliased cluster only? Also - a better fix would be to do a waterfall approach, if @@report_host fails, fall down to @@localhost.
Suggestion:
   "MySQLHostnameResolveMethod": ["@@report_host", "@@hostname"]

Is there a way to use @@report_host for an aliased cluster only?

There is no such way at this time.
A waterfall approach is possible. For backwards compatibility it would have to be a different variable. This would lead to some confusion, though, I suspect.
But, if hostnames are meaningless, I'd suggest:
{
   "HostnameResolveMethod": "none",
   "MySQLHostnameResolveMethod": ""
}```

Premature "Enter". The above would instruct orchestrator to not resolve at all.
You would need to use IPs all over the place. That is, you'll need to configure all your replicas like so: change master to master_host='<IP_ADDRESS>'.


[Samples][Java] Fix disparities of 05.multi-turn-prompt sample

Related to microsoft/botbuilder-java#1165
Proposed Changes
We compared the documentation/code migration/behavior of the 05.multi-turn-prompt sample between Java and C# and we found disparities and fixes that this PR includes.

Update README
Update UserProfileDialog adding missing try/catch which is present in C#
Add missing comments
Clean-up autogenerated files/folder related to Eclipse (e.g. target/.classpath)
Add package-info.java file

Testing
Multi Turn Prompt sample working as expected

Hi @tracyboehrer & @LeeParrishMSFT, we created this PR fixing some disparities that we found in the 05.multi-turn-prompt sample.


Added features list to README.md.

If you think I missed anything on the list, please let me know or if you think a better wording might help.

Made suggested changes.


Allow folding range response from clangd

clangd supports folding range in stable/20221013. Update the test to allow an actual response.

@swift-ci please test


Encoder AudioObjectType configurable

Hi @haileys! Thanks for making this crate.
We have a need to configure the Audio Object Type to a different value. Hopefully this change is ok to merge.
Thanks,
Martin

Thank you!

Released as 0.7.0


[Event Request] Codeunit 5601 "FA Insert G/L Account" OnBeforeInsertBufferEntryMaintAccNo

Hello
My customer needs to change Account No. on FAGLPostBuffer before it is inserted.
So I need a new Event right before InsertBufferEntry is used.
Example in function InsertMaintenanceAccNo
FAGLPostBuf."FA Entry Type" := FAGLPostBuf."FA Entry Type"::Maintenance;
GLEntryNo := "G/L Entry No.";

LOCAL [IntegrationEvent] OnBeforeInsertBufferEntryMaintAccNo(VAR FAGLPostBuf : Record "FA G/L Posting Buffer";MaintenanceLedgEntry : Record "Maintenance Ledger Entry")

Hi @rungis75 .
It's not clear if you need event in particular place before InsertBuffer or before all calls of InsertBuffer function. If before all then there is event OnInsertBufferEntryOnBeforeBufferInsert(TempFAGLPostBuf, FAGLPostBuf) in the InsertBufferEntry() function. Does it fit to you? Or please be precise where you it.

Hi Roman
Sorry, this event should only be in function InsertMaintenanceAccNo
Because I need to have MaintenanceLedgEntry in my Event and use it in my subscriber
I can not use OnInsertBufferEntryOnBeforeBufferInsert(TempFAGLPostBuf,FAGLPostBuf)  because when I use it from InsertMaintenanceAccNo I can not get FAGLPostBuf because it is inserted after that.
Hope this will explain what I am talking about
Brg
Runar
Rúnar Birgir Gíslason
Hugbúnaðarsérfræðingur / Software Specialist
BC ráðgjöf – LV1 / Viðskiptalausnir
https://www.advania.is/disclaimer Skilmálar/Disclaimer
From: RomanZanizdra<EMAIL_ADDRESS>Sent: föstudagur, 4. september 2020 09:55
To: microsoft/ALAppExtensions<EMAIL_ADDRESS>Cc: rungis75<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [microsoft/ALAppExtensions] [Event Request] Codeunit 5601 "FA Insert G/L Account" OnBeforeInsertBufferEntryMaintAccNo (#8413)
Hi @rungis75 https://github.com/rungis75  .
It's not clear if you need event in particular place before InsertBuffer or before all calls of InsertBuffer function. If before all then there is event OnInsertBufferEntryOnBeforeBufferInsert(TempFAGLPostBuf, FAGLPostBuf) in the InsertBufferEntry() function. Does it fit to you? Or please be precise where you it.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/microsoft/ALAppExtensions/issues/8413#issuecomment-687048316 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AQ2OF2B6N5OLBBE3P6Q4HF3SEC2PLANCNFSM4QRSBFNQ .

Hi again @rungis75 .
Please check at least the latest 16 version. I see the required event is already there. And if this is what you need than let's close current issue.
procedure InsertMaintenanceAccNo(var MaintenanceLedgEntry: Record "Maintenance Ledger Entry")
begin
    with MaintenanceLedgEntry do begin
        Clear(FAGLPostBuf);
        FAGLPostBuf."Account No." := FAGetGLAcc.GetMaintenanceAccNo(MaintenanceLedgEntry);
        FAGLPostBuf.Amount := Amount;
        FAGLPostBuf.Correction := Correction;
        FAGLPostBuf."Global Dimension 1 Code" := "Global Dimension 1 Code";
        FAGLPostBuf."Global Dimension 2 Code" := "Global Dimension 2 Code";
        FAGLPostBuf."Dimension Set ID" := "Dimension Set ID";
        FAGLPostBuf."FA Entry No." := "Entry No.";
        FAGLPostBuf."FA Entry Type" := FAGLPostBuf."FA Entry Type"::Maintenance;
        GLEntryNo := "G/L Entry No.";
        OnInsertMaintenanceAccNoOnBeforeInsertBufferEntry(FAGLPostBuf, MaintenanceLedgEntry);
        InsertBufferEntry;
        "G/L Entry No." := TempFAGLPostBuf."Entry No.";
    end;
end;


TagBot trigger issue

This issue is used to trigger TagBot; feel free to unsubscribe.
If you haven't already, you should update your TagBot.yml to include issue comment triggers.
Please see this post on Discourse for instructions and more details.
If you'd like for me to do this for you, comment TagBot fix on this issue.
I'll open a PR within a few hours, please be patient!

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/113156

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/115382

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/116013


python310Packages.social-auth-core: 4.3.0 -> 4.4.1

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python310Packages.social-auth-core is: Module for social authentication/registration mechanisms
meta.homepage for python310Packages.social-auth-core is: https://github.com/python-social-auth/social-core
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 4.4.0 with grep in /nix/store/f3qww02vdb8hf8200a6sxz1ssim7z41v-python3.10-social-auth-core-4.4.0
found 4.4.0 in filename of file in /nix/store/f3qww02vdb8hf8200a6sxz1ssim7z41v-python3.10-social-auth-core-4.4.0

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

5 total rebuild path(s)

5 package rebuild(s)

First fifty rebuilds by attrpath
netbox
python310Packages.social-auth-app-django
python310Packages.social-auth-core
python311Packages.social-auth-app-django
python311Packages.social-auth-core

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/f3qww02vdb8hf8200a6sxz1ssim7z41v-python3.10-social-auth-core-4.4.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

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

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/f3qww02vdb8hf8200a6sxz1ssim7z41v-python3.10-social-auth-core-4.4.0
ls -la /nix/store/f3qww02vdb8hf8200a6sxz1ssim7z41v-python3.10-social-auth-core-4.4.0/bin

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

3 packages failed to build:

netbox
python310Packages.social-auth-app-django
python311Packages.social-auth-app-django

2 packages built:

python310Packages.social-auth-core
python311Packages.social-auth-core

Maintainer pings
cc @n0emis for testing.

@ofborg build netbox

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python310Packages.social-auth-core is: Module for social authentication/registration mechanisms
meta.homepage for python310Packages.social-auth-core is: https://github.com/python-social-auth/social-core
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 4.4.1 with grep in /nix/store/gvhyi4caw1czc3b4q9vllghjz2sjds2f-python3.10-social-auth-core-4.4.1
found 4.4.1 in filename of file in /nix/store/gvhyi4caw1czc3b4q9vllghjz2sjds2f-python3.10-social-auth-core-4.4.1

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

5 total rebuild path(s)

5 package rebuild(s)

First fifty rebuilds by attrpath
netbox
python310Packages.social-auth-app-django
python310Packages.social-auth-core
python311Packages.social-auth-app-django
python311Packages.social-auth-core

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/gvhyi4caw1czc3b4q9vllghjz2sjds2f-python3.10-social-auth-core-4.4.1 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

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

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/gvhyi4caw1czc3b4q9vllghjz2sjds2f-python3.10-social-auth-core-4.4.1
ls -la /nix/store/gvhyi4caw1czc3b4q9vllghjz2sjds2f-python3.10-social-auth-core-4.4.1/bin

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

5 packages built:

netbox
python310Packages.social-auth-app-django
python310Packages.social-auth-core
python311Packages.social-auth-app-django
python311Packages.social-auth-core

Maintainer pings
cc @n0emis for testing.


use_count_query changes rule behavior

Here is my rule:
es_host: ********
es_port: 9200
name: Warnning_Elasticsearch
type: frequency
index: logstash-*
num_events: 5
timeframe:
hours: 1
use_count_query: true
filter:

term:
level: "warn"

alert:

"email"

email:

"*****"

And in the config file:
run_every:
minutes: 1
buffer_time:
days: 1
es_host: *******
es_port: 9200
highlight=send_get_body_as#transport
writeback_index: prueba
realert:
minutes: 0
alert_time_limit:
days: 1
smtp_host: **********
smtp_port: ***
from_addr: **********
I hope that the alert start one day ago, run every minutes and use intervals of one hour, to compare the numbers of docs in the query with the num_events, but the alert look for docs in 1 minute intervals, every 1 minute. This behavior is only viewed when use_count_query is set.

Whether you use use_count_query or not, all the events are added to a 1 hour sliding time window. You will still only get an alert if num_events documents occur within an hour of each other. Is that not the expected behavior?
You expect a query to be made that covers an entire hour every minute? If the window's overlap, how would we know exactly how many documents there are?


Add Support for Nippro brand meters

Please add support for Nippro devices like
TrueMetrix
TrueMetrix Air
A majority of patients we see use these devices and it would be a great help if they were supported.

I believe there is some work in progress for this in this branch: https://github.com/tidepool-org/chrome-uploader/tree/truemetrix
being tracked in this Trello card: https://trello.com/c/OFkyZUe6/335-uploader-trividia-true-metrix

Support for True Metrix, True Metrix Air and True Metrix Go added in v2.6.0.


Add b version of home page for testing

Description
This PR adds a B version of home page that includes a code block in the hero.
Checklist:

[ ] I have reviewed the style guide.
[ ] If blogging, I have reviewed the blogging guide.
[ ] I have manually confirmed that all new links work.
[ ] I added aliases (i.e., redirects) for all filename changes.
[ ] If making css changes, I rebuilt the bundle.

how do i view the B version in preview?

@gthuang Just updated the preview link


Faruk/add config files

[Err] [InsertModelWidget.cc:403] Missing model.config for model "/home/frk/catkin_ws/src/auv-software/auv_sim/auv_sim_description/auv_common_sim_descriptions/urdf/actuators"
[Err] [InsertModelWidget.cc:403] Missing model.config for model "/home/frk/catkin_ws/src/auv-software/auv_sim/auv_sim_description/auv_common_sim_descriptions/urdf/sensors"
used to get this eror  messages  after launching the file start_gazebo.launch.
things ı have tried:
Created a folder named config in the urdf directory and added the necessary files.
Created a folder named configuration in the urdf directory and added the necessary files.
Added the model.config files in every config directory related to auv_sim_description.
Only adding the file this way solved the issue:
added file to: /auv-software/auv_sim/auv_sim_description/auv_common_sim_descriptions/urdf/actuators/model.config
added file to: /auv-software/auv_sim/auv_sim_description/auv_common_sim_descriptions/urdf/sensors/model.config

Hmm, I think this PR brings a bit more than "add config files" :)
You can change the base branch to add auv sim, since your branch is initiated off from that branch it should be bringing those changes back to that branch, not main.

The issue has been resolved. These kinds of mix-ups can happen, especially with initial PRs. It's always a good reminder to double-check before submitting a pull request.

thanks for the heads up. ı will double check everything from now. Looking forward to get feedback or suggestions for any corrections about the addition of config files.


Can't find the specified path after running fast_start.bat

Running fast_start.bat manually immediately closes the cmd saying something about a fatal error.
After running it through the cmd with cd, it says it can't find the specified path

its in german, but it basically says "The system can't find the specified path"
*I am running cmd as admin

Nevermind, really dumb error on my side, i forgot to update to net7


Retarget test client

Tested the other 2 integrate test configurations manually and they work! Going to start working on cleaning things up.


[1.22] p. 49, point 4.

It says

The fifth line, tasks: , tells Ansible that what follows is a list of tasks to run as
part of this playbook.

Should probably say

The fifth line, tasks: , tells Ansible that what follows is a list of tasks to run as
part of this play.

Love the book. Keep up the good job!

Right you are! This fix will be in the next version of the book.

(the next version will be 1.23, out soon!)


Links to other docs pages are broken in github pages due to additional /

🐛 Bug Report
Prerequisites

[X] I'm using the latest version of Docusaurus.
[X] I have tried the npm run clear or yarn clear command.
[X] I have tried rm -rf node_modules yarn.lock package-lock.json and re-installing packages.
[X] I have tried creating a repro with https://new.docusaurus.io
[X] I have read the console error message carefully (if applicable)

Description
Deep links don't work correctly when deployed to github pages. Example:
this link points to .../create-a-page, but is redirected to .../create-a-page/. Displaying the page works, but clicking a link to another page (here Congratulations), loads .../create-a-page/congratulations instead of the correct .../congratulations
Have you read the Contributing Guidelines on issues?
Yes
Steps to reproduce
Use https://new.docusaurus.io to create a CodeSandbox reproducible demo of the bug.
(Write your steps here:)

Clone Demo
Add a link to a doc page (2) to a docs page (1)
Deploy to Github Pages
Access page (1) via direct link
Click on the added link
Observe Page Not Found

Expected behavior
Links to other pages behave correctly when the url has an additional slash appended.
Actual behavior
Links to other pages don't work when the url has an additional slash appended.
Your environment

Public source code: https://github.com/F43nd1r/docusaurus-link-issue-reproducer
Public site URL: https://f43nd1r.github.io/docusaurus-link-issue-reproducer/
Docusaurus version used: 2.0.0-beta.1
Environment name and version (e.g. Chrome 78.0.3904.108, Node.js 10.17.0): Github Pages
Operating system and version (e.g. Ubuntu 20.04.2 LTS):

Reproducible demo
https://github.com/F43nd1r/docusaurus-link-issue-reproducer
Go to https://f43nd1r.github.io/docusaurus-link-issue-reproducer/, click the link and then click on the congratulations link in the page.

That's funny because this issue is solved by the brand new beta.1 trailingSlash option.
https://github.com/facebook/docusaurus/issues/3372
https://github.com/facebook/docusaurus/pull/4908
For GH pages we now recommend using trailingSlash: false as it will emit /x.html instead of /x/index.html and prevent GH pages to add that trailing slash, breaking relative links resolution.
See also https://github.com/slorber/trailing-slash-guide to understand how GH pages behave.
https://docusaurus.io/docs/deployment#trailing-slashes
As this is a new option, please let me know if false is a good value for GH pages. If it is a good setting fr GH pages, we may add a warning on docusaurus deploy so that people always use trailingSlash: false with GH pages. Your feedback is important so let me know.

As this is a new option, please let me know if false is a good value for GH pages.

Yes.

If it is a good setting fr GH pages, we may add a warning on docusaurus deploy so that people always use trailingSlash: false with GH pages.

That would be nice, although I'm deploying with a dedicated github action which doesn't use deploy (https://github.com/ACRA/acra/blob/master/.github/workflows/deploy-docs.yml#L25)

Thanks for the feedback.
For GH action deploy then we can't do anything apart from documenting it.

For GH action deploy then we can't do anything apart from documenting it.

Yeah, makes sense. Was just mentioning to say it won't affect me personally.
Thank you for the tip and replies :+1:


Allows for options to be redefined after init

By creating an instance identifier each time the wavify function is called and storing that instance identifier outside the function we can test if the instance identifier in the function still matches the instance identifier stored outside the function. If it doesn't, we've received a new set of options and should not call any more frame updates for the now outdated options.
This is absolutely a quick and dirty solution to the problem, but it's reasonably efficient and gets the job the done.
The one caveat is it relies on the target element having a unique id attribute set. If it does not the animations will not work as expected if more than one wave exists on a page.
This pull request is branched off of pull request #3, so that pull request should be reviewed accepted before this one.

Thanks for your PR. I dont know if changing options after init is a good idea, i see the possible usages for this but I would rather like pass a more complex boot options with different values and timing in order to allow values update.
I'm open to discuss it with you guys

Hello, if you can give me feedbacks about the new version of the plugin, we will see if this PR is still needed. Thanks


Repo sync for protected CLA branch

The pull request is created from master637532409710694840 to master to fix git push error for protected CLA branch

@v-alje : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#sign-off


UI/UX Design: The name for a specific field be equidistant from its intended field and another field

Issue: The name for a specific field be equidistant from its intended field and another field
Solution requested: The name for a specific field should be closer to its intended field than another field
Feature Request
On the UI/UX design, the name for a specific field is currently equidistant from its intended field and another field.
Solution
The name for a specific field should be closer to its intended field than another field.
Additional context

Thanks
Would you like to work on the issue?
@jingtang10 @f-odhiambo @sebamutuku @MargaretNjenga

this should be also achievable using the customization options from https://google.github.io/android-fhir/use/SDCL/Customize-how-a-Questionnaire-is-displayed/#localize-questionnaires

I tried to reduce the space between the view and question text without requiring changes in the datacapture module. You need to override the attribute questionnaireQuestionHeaderStyle by providing a custom style where you modify the padding and margin values.
<item name="questionnaireQuestionHeaderStyle">
            @style/Catalog.QuestionHeaderStyle
</item>

<style name="Catalog.QuestionHeaderStyle" parent="">
        <item name="android:paddingBottom">1dp</item>
        <item name="android:layout_marginTop">16dp</item>
    </style>

Closing as per @santosh-pingle 's solution above.


Need attachment-upload feature

Is your feature request related to a problem? Please describe.
Sometimes we have to upload large-text-file(such as large-HTTP-response/large-logging) to questions. If we put these text within qustion description, it makes we hard to read the core question text.
Describe the solution you'd like
Just like add attachment to email.
Describe alternatives you've considered
I think add-attachment is the most popular manner, or we have to make another CDN like to store these files.

@fenbox Hi~
I have some ideas regarding this issue. Perhaps we could develop a package to manage files? This package would support local storage or storage on S3, allow for duplicate file processing, and could serve as a foundational package for development related to #300.

@sosyz We can use the Kano Model to analyze the "files management package". I think it's not a Must-be or One-dimensional requirements. So this feature is not suitable as a core feature.
But there are some users who need this feature, so it's suitable as a plugin.

@sosyz We can use the Kano Model to analyze the "files management package". I think it's not a Must-be or One-dimensional requirements. So this feature is not suitable as a core feature.
But there are some users who need this feature, so it's suitable as a plugin.

@fenbox Got it, I agree with your perspective. Implementing it as a plugin would indeed be a better approach.


Docker version 18.03.0-ce-win59: unable to finish Pulling bind (cytopia/bind:0.11) as unauthorized: incorrect username or password

I have done the check list as being inform,
I have check with another machine with older version of docker (Docker version 17.12.1-ce, build 7390fc6), This issue does not occur.
However, at my machine This issue occur:

Perhaps, It would be great if there is any advice to further debug the root cause?
My Machine information:
Docker Version:
Version 18.03.0-ce-win59 (16762)
Channel: stable
D70d80d

[Docker 18.03.0-ce]
(https://github.com/docker/docker-ce/releases/tag/v18.03.0-ce)
Docker Machine 0.14.0
Docker compose 1.20.1
Notary 0.6.0

@alzen8work are you by any chance logged into a different or private docker repository?
Can you check if any of those work separately:
docker pull cytopia/bind:0.11

docker pull devilbox/php-fpm:7.2-work

wow, that's seems to be the root cause, i've sign out from docker hub and it work marvelously...
wierd thing though it has no issue if i din sign out from docker version 17.12.1-ce...

Lucky shot, but good to hear


Error: Cannot find module 'eslint-config-airbnb/base' from '/usr/lib/node_modules'

I am using vim's syntactic plugin, the plugin was not working with airbnb's .eslintrc, to investigate the issue I tried linting my file from command line by issuing the command $ eslint app.js, it gave the error mentioned in the subject. I am pasting the stack trace here

Referenced from: /home/saqib/blog/.eslintrc
at Function.module.exports [as sync] (/usr/lib/node_modules/eslint/node_modules/resolve/lib/sync.js:33:11)
at resolve (/usr/lib/node_modules/eslint/lib/config/config-file.js:417:38)
at load (/usr/lib/node_modules/eslint/lib/config/config-file.js:436:24)
at /usr/lib/node_modules/eslint/lib/config/config-file.js:355:36
at Array.reduceRight (native)
at applyExtends (/usr/lib/node_modules/eslint/lib/config/config-file.js:338:28)
at Object.load (/usr/lib/node_modules/eslint/lib/config/config-file.js:464:22)
at loadConfig (/usr/lib/node_modules/eslint/lib/config.js:67:33)
at getLocalConfig (/usr/lib/node_modules/eslint/lib/config.js:129:23)
at Config.getConfig (/usr/lib/node_modules/eslint/lib/config.js:222:22)

I have a coworker who has had to do some gymnastics to get syntactic working as well.
I suspect one or both of eslint and the plugin are not in the same package area.
npm -g list | grep eslint vs cd /home/saqib/blog && npm list | grep eslint?

I am encountering this problem.How did you solve it?@segmentationfaulter

@xl-y still could not solve it.

I think you probably need to globally install any ESLint plugins.
Try running npm install -g eslint-config-airbnb in a terminal.


Feat loading script

This is only when script is not loaded yet. At the same time it shows that the launcher app server is on (in case of script errors)

oups wrong base branch


Testing realtime

How do I test the code in realtime video?

Hi @Zumbalamambo , you can apply the algorithm per frame. Th CNN architecture for this code is 2D.


test: dialog should not leak events on touch devices when overlay is clicked

add regression test for https://github.com/melt-ui/melt-ui/pull/1118

let me know what you think about whether the test script should run both vitest and playwright. currently, the DX is really nice with vitest and watch mode and only rerunning tests that depend on the new files changed. not sure if we should include playwright in the default test scripts or just have it run on pull requests in github actions.


self reactions

Seems there are no self-reactions.. is there a use case for this..? Think we talked about it but just wanted to record a discussion here for future reference.

yep. it's being worked on as part of #88
it's needed for a lot of teams involving sucrose or kazuha