Feature Request: Show differences between current database and latest update when a update is avaliable

Maybe highlight the differences after the update completes. Would make it easier to see what has been added

But you get the difference when the titles update.
Best I can do is a show more info button or something which lists all titles if you want.

Unless I'm missing something, you only get a small list which doesnt include all the changes.
It would be awesome if you could add the button, maybe a seperate screen or just sort the list to have the titles sorted by date added


About Codes

Hi,YiF:
Will you open source code not based on mmcv and mmseg？

All Right! Thank you


unrecognized import path "libguestfs.org/guestfs"

❯ go version
go version go1.5.1 darwin/amd64

❯ brew install libguestfs --devel --env=std
Warning: zchee/libguestfs/libguestfs-1.31.7 already installed

then, I tried to make docker-machine-xhyve, but shows following error when run go get and make.
❯ go get -d github.com/zchee/docker-machine-xhyve
package libguestfs.org/guestfs: unrecognized import path "libguestfs.org/guestfs"

❯ cd $GOPATH/src/github.com/zchee/docker-machine-xhyve

❯ make
==> Build docker-machine-xhyve ...
 go build -o bin/docker-machine-driver-xhyve -ldflags  -w -s -X github.com/zchee/docker-machine-xhyve/version.GitCommit=7b49bbd  github.com/zchee/docker-machine-xhyve/bin
# github.com/zchee/docker-machine-xhyve/vendor/libguestfs.org/guestfs
vendor/libguestfs.org/guestfs/guestfs.go:31:10: fatal error: 'guestfs.h' file not found
#include "guestfs.h"
         ^
1 error generated.
make: *** [build] Error 1

I wrote export GO15VENDOREXPERIMENT=1 to my .zshrc and retried, then go get shows no errors, but make shows same error.

update
I applied write permission to some directories so get guestfs.h linking works.
sudo chown $(whoami):admin /usr/local && sudo chown -R $(whoami):admin /usr/local
brew unlink libguestfs && brew link libguestfs

( Note: I think this problem came with El Capitan's rootless feature. )
Then, go get -d github.com/zchee/docker-machine-xhyve works fine, without no unrecognized import warning!
Next step, I tried to make it but same error was shown.
❯ make
==> Build docker-machine-xhyve ...
 go build -o bin/docker-machine-driver-xhyve -ldflags  -w -s -X github.com/zchee/docker-machine-xhyve/version.GitCommit=605fba6  github.com/zchee/docker-machine-xhyve/bin
# github.com/zchee/docker-machine-xhyve/vendor/libguestfs.org/guestfs
vendor/libguestfs.org/guestfs/guestfs.go:31:10: fatal error: 'guestfs.h' file not found
#include "guestfs.h"
         ^
1 error generated.
make: *** [build] Error 1

I tried CGO_CFLAGS="-DGUESTFS_PRIVATE=1 -I/usr/local/include" CGO_LDFLAGS="-L/usr/local/lib -lguestfs" godep go build -i -o ./bin/docker-machine-driver-xhyve ./bin, and it seems crated docker-machine-driver-xhyve executable into bin dir.
I think maybe options for go build like -ldflags causes something wrong.
Could you show me what options is suitable for replacement those environment variable?
I tried -ldflags and -cflags, but not sure it's right way and failed to build.
❯ go build -o bin/docker-machine-driver-xhyve -ldflags "-L /usr/local/lib" -gcflags "-I /usr/local/include" github.com/zchee/docker-machine-xhyve/bin
# github.com/zchee/docker-machine-xhyve/vendor/libguestfs.org/guestfs
/Users/momonga/.go/src/github.com/zchee/docker-machine-xhyve/vendor/libguestfs.org/guestfs/guestfs.go:31:10: fatal error: 'guestfs.h' file not found
#include "guestfs.h"
         ^
1 error generated.

working pattern
I edit Makefile and found a working pattern.
https://github.com/zchee/docker-machine-xhyve/blob/7b49bbd5f11ef5481cf20715d4a9b08eb7d80ebf/Makefile#L108-L109
CGO_CFLAGS:=${CGO_CFLAGS} -I /usr/local/include
CGO_LDFLAGS:=${CGO_LDFLAGS} -L /usr/local/lib

:= instead of = to avoid recursive variable reference
add a whitespace after -I and -L

https://github.com/zchee/docker-machine-xhyve/blob/7b49bbd5f11ef5481cf20715d4a9b08eb7d80ebf/Makefile#L144
	CGO_CFLAGS="${CGO_CFLAGS}" CGO_LDFLAGS="${CGO_LDFLAGS}" ${GO_BUILD} -ldflags "${CGO_LDFLAGS}" ${GO_GCFLAGS} ${TOP_PACKAGE_DIR}/${PACKAGE}/bin || exit 1

add CGO_CFLAGS and CGO_LDFLAGS re-definition. (It's weird. Why re-definition make it works?)

@supermomonga Oh...
I terrible mistake.
I understand all. and I did not have enough knowledge to write a Makefile.
I will fix for now.
Thanks!!

I found that #19 pull request removes libguestfs dependency, so I think this build problem is also resolved.

and, https://github.com/zchee/docker-machine-xhyve/commit/0a9b453cd249e47c1079661552d6fc72f37b5e27
Thanks @supermomonga !!


E2E won't start due to pod failures

We saw this in Jenkins. The following loops until the test is killed:
[1] INFO: 9 / 14 pods in namespace 'kube-system' are running and ready (20 seconds elapsed)
[1] INFO: POD                           NODE                    PHASE   CONDITIONS
[1] INFO: kube-ui-v1-bp9s2              pull-e2e-2-minion-yl9w  Failed  []
[1] INFO: kube-ui-v1-dk10d              pull-e2e-2-minion-yl9w  Failed  []
[1] INFO: kube-ui-v1-eahns              pull-e2e-2-minion-yl9w  Failed  []
[1] INFO: kube-ui-v1-zvxka              pull-e2e-2-minion-yl9w  Failed  []
[1] INFO: monitoring-heapster-v5-5v5rp  pull-e2e-2-minion-yl9w  Failed  []
[1] INFO:
[1] INFO: 9 / 14 pods in namespace 'kube-system' are running and ready (20 seconds elapsed)
[1] INFO: POD                           NODE                    PHASE   CONDITIONS
[1] INFO: kube-ui-v1-bp9s2              pull-e2e-2-minion-yl9w  Failed  []
[1] INFO: kube-ui-v1-dk10d              pull-e2e-2-minion-yl9w  Failed  []
[1] INFO: kube-ui-v1-eahns              pull-e2e-2-minion-yl9w  Failed  []
[1] INFO: kube-ui-v1-zvxka              pull-e2e-2-minion-yl9w  Failed  []
[1] INFO: monitoring-heapster-v5-5v5rp  pull-e2e-2-minion-yl9w  Failed  []
[1] INFO:
[1] INFO: 9 / 14 pods in namespace 'kube-system' are running and ready (20 seconds elapsed)
[1] INFO: POD                           NODE                    PHASE   CONDITIONS
[1] INFO: kube-ui-v1-bp9s2              pull-e2e-2-minion-yl9w  Failed  []
[1] INFO: kube-ui-v1-dk10d              pull-e2e-2-minion-yl9w  Failed  []
[1] INFO: kube-ui-v1-eahns              pull-e2e-2-minion-yl9w  Failed  []
[1] INFO: kube-ui-v1-zvxka              pull-e2e-2-minion-yl9w  Failed  []
[1] INFO: monitoring-heapster-v5-5v5rp  pull-e2e-2-minion-yl9w  Failed  []
[1] INFO:
...

I saw this once so haven't yet assessed its frequency.
@ixdy says he thinks he has seen the same failure a few times, and linked artifacts (for those with permissions):
https://console.developers.google.com/storage/browser/kubernetes-jenkins/pr-logs/739cb2f526f15e7c82ef7c476a1458ad68b09fd1/kubernetes-pull-build-test-e2e-gce/2246/_artifacts/
+cc @rjnagal for labeling/triage/xref:#11018

(fixed your URL so it's publicly available)

@ixdy thank you!

Hm, so looking at a successful GCE run, I notice that there appear to normally be 9 pods, not 14, e.g.:
INFO: Waiting up to 10m0s for all pods (need at least 0) in namespace 'kube-system' to be running and ready
INFO: The status of Pod elasticsearch-logging-v1-l9s9g is Pending, waiting for it to be either Running or Failed
INFO: The status of Pod kibana-logging-v1-at4ah is Pending, waiting for it to be either Running or Failed
INFO: 7 / 9 pods in namespace 'kube-system' are running and ready (5 seconds elapsed)
INFO: expected 7 pod replicas in namespace 'kube-system', 5 are Running and Ready.
INFO: POD                             NODE                 PHASE    CONDITIONS
INFO: elasticsearch-logging-v1-l9s9g  e2e-gce-minion-akfc  Pending  []
INFO: kibana-logging-v1-at4ah         e2e-gce-minion-akfc  Pending  []
INFO:
INFO: The status of Pod elasticsearch-logging-v1-l9s9g is Pending, waiting for it to be either Running or Failed
INFO: The status of Pod kibana-logging-v1-at4ah is Pending, waiting for it to be either Running or Failed
INFO: 7 / 9 pods in namespace 'kube-system' are running and ready (10 seconds elapsed)
INFO: expected 7 pod replicas in namespace 'kube-system', 5 are Running and Ready.
INFO: POD                             NODE                 PHASE    CONDITIONS
INFO: elasticsearch-logging-v1-l9s9g  e2e-gce-minion-akfc  Pending  []
INFO: kibana-logging-v1-at4ah         e2e-gce-minion-akfc  Pending  []
INFO:
INFO: 9 / 9 pods in namespace 'kube-system' are running and ready (15 seconds elapsed)
INFO: expected 7 pod replicas in namespace 'kube-system', 7 are Running and Ready.

Yet here there are 14, including what appear to be many duplicates?
From the kube-controller-manager.log, it appears to be creating extra replicas of kube-ui and monitoring-heapster, maybe? Not sure if I'm reading the log correctly.
I0710 17:41:14.641740       5 replication_controller.go:316] Too few "kube-system"/"kube-ui-v1" replicas, need 1, creating 1
I0710 17:41:14.656821       5 event.go:203] Event(api.ObjectReference{Kind:"ReplicationController", Namespace:"kube-system", Name:"kube-ui-v1", UID:"b54b2b34-272a-11e5-a95e-42010af0ab9c", APIVersion:"v1", ResourceVersion:"64", FieldPath:""}): reason: 'successfulCreate' Created pod: kube-ui-v1-dk10d
I0710 17:41:17.244137       5 replication_controller.go:316] Too few "kube-system"/"kube-ui-v1" replicas, need 1, creating 1
I0710 17:41:17.257327       5 event.go:203] Event(api.ObjectReference{Kind:"ReplicationController", Namespace:"kube-system", Name:"kube-ui-v1", UID:"b54b2b34-272a-11e5-a95e-42010af0ab9c", APIVersion:"v1", ResourceVersion:"172", FieldPath:""}): reason: 'successfulCreate' Created pod: kube-ui-v1-eahns
I0710 17:41:18.044454       5 replication_controller.go:316] Too few "kube-system"/"kube-ui-v1" replicas, need 1, creating 1
I0710 17:41:18.074662       5 event.go:203] Event(api.ObjectReference{Kind:"ReplicationController", Namespace:"kube-system", Name:"kube-ui-v1", UID:"b54b2b34-272a-11e5-a95e-42010af0ab9c", APIVersion:"v1", ResourceVersion:"189", FieldPath:""}): reason: 'successfulCreate' Created pod: kube-ui-v1-zvxka
I0710 17:41:18.845526       5 replication_controller.go:316] Too few "kube-system"/"kube-ui-v1" replicas, need 1, creating 1
I0710 17:41:18.857737       5 event.go:203] Event(api.ObjectReference{Kind:"ReplicationController", Namespace:"kube-system", Name:"kube-ui-v1", UID:"b54b2b34-272a-11e5-a95e-42010af0ab9c", APIVersion:"v1", ResourceVersion:"198", FieldPath:""}): reason: 'successfulCreate' Created pod: kube-ui-v1-bp9s2
I0710 17:41:19.644130       5 replication_controller.go:316] Too few "kube-system"/"kube-ui-v1" replicas, need 1, creating 1
I0710 17:41:19.657540       5 event.go:203] Event(api.ObjectReference{Kind:"ReplicationController", Namespace:"kube-system", Name:"kube-ui-v1", UID:"b54b2b34-272a-11e5-a95e-42010af0ab9c", APIVersion:"v1", ResourceVersion:"208", FieldPath:""}): reason: 'successfulCreate' Created pod: kube-ui-v1-8w60c
I0710 17:41:22.424636       5 replication_controller.go:334] Too many "kube-system"/"kube-ui-v1" replicas, need 1, deleting 1
I0710 17:41:29.012666       5 replication_controller.go:316] Too few "kube-system"/"monitoring-heapster-v5" replicas, need 1, creating 1
I0710 17:41:29.025641       5 event.go:203] Event(api.ObjectReference{Kind:"ReplicationController", Namespace:"kube-system", Name:"monitoring-heapster-v5", UID:"b5529f71-272a-11e5-a95e-42010af0ab9c", APIVersion:"v1", ResourceVersion:"69", FieldPath:""}): reason: 'successfulCreate' Created pod: monitoring-heapster-v5-74nzi
I0710 17:41:34.363564       5 replication_controller.go:334] Too many "kube-system"/"monitoring-heapster-v5" replicas, need 1, deleting 1
I0710 17:41:34.841843       5 replication_controller.go:316] Too few "kube-system"/"monitoring-heapster-v5" replicas, need 1, creating 1
I0710 17:41:34.857254       5 event.go:203] Event(api.ObjectReference{Kind:"ReplicationController", Namespace:"kube-system", Name:"monitoring-heapster-v5", UID:"b5529f71-272a-11e5-a95e-42010af0ab9c", APIVersion:"v1", ResourceVersion:"271", FieldPath:""}): reason: 'successfulCreate' Created pod: monitoring-heapster-v5-3zg4y

Yes, by default on today's e2e test environment, there should be only 9 pods: 2 fluentd-logging (one each minion), 3 dns pod, 1 ui, 1 heapster, 2 influx-grafana (2+3+1+1+2). Sounds like a replication_controller related issue. Also it could be a potential cause for #10899.

The rc will keep creating replicas if previous replicas go into failed (as seems to be the case in the first comment)

Is it easy enough to not count failed ones?

Ah, it looks like #11019 merged this afternoon will fix this issue (failed pods won't be counted).
I think we're all OK here.

Hrm, why were the pods in failed? any clues?


Camel 9416 - Creation of Camel-MLLP component

Moved the URI Parameter documentation to the setters.
Added support for HL7 Commit Acknowledgements
Updated the README.md and LIMITATIONS.md

Thanks the PR has been merged. Do you mind closing

Thank You Claus - closing PR


#280: configure maven properly

Fixes: #280
There is a related PR to ide-settings that renames the devon folder to ide, as suggested by @hohwille .
https://github.com/devonfw/ide-settings/pull/45
Note: the configuration will properly work just if the settings are properly updated (or if locally the devon folder in settings is manually renamed to ide)

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
176 unchanged lines in 3 files lost coverage.
Overall coverage decreased (-0.4%) to 59.362%

Files with Coverage Reduction
New Missed Lines
%

com/devonfw/tools/ide/context/IdeContext.java
10
48.65%

com/devonfw/tools/ide/tool/mvn/Mvn.java
68
5.62%

com/devonfw/tools/ide/context/AbstractIdeContext.java
98
56.67%

Totals

Change from base Build<PHONE_NUMBER>:
-0.4%

Covered Lines:
4591

Relevant Lines:
7456

💛  - Coveralls

If there is a problem with the creation with the files settings.xml or settings-security.xml, a warning is displayed.
Probably we can create and integrate a link to a page in the documentation as mvn.adoc or maven_configuration.adoc? where we can briefly explain to the user how IDEasy deals with these files and how the variable MAVEN_ARGS is set and can be overwritten.
NOTE: because of the high priority of this issue, unit tests for mvn will be properly developed in a different PR that will follow up.

NOTE: because of the high priority of this issue, unit tests for mvn will be properly developed in a different PR that will follow up.

I used this PR to update the variables concerning maven in the documentation (variables.adoc) as well. MAVEN_ARGS replaces MAVEN_OPTS as alredy implemented in #275
NOTE: because of the high priority of this PR, unit tests for mvn will be properly developed in a different PR that will follow up.


Tracking Issue for Initial repo setup

[ ] Set up CI
[ ] Set up merge group branch protection

Done


React & Redux group project: set up your Kanban board

GitHub project: https://github.com/sentayhu19/Space-Travelers-Hub/projects/1
Team Members :

@sentayhu19
@Theophileaseh

Hi Team,
Your project is complete!  There is nothing else to say other than... it's time to merge it :shipit:
Congratulations! 🎉

Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.

As described in the Code reviews limits policy  you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.


🐛 Bug Report: Naming ap Account and user model Account is not the best decision.

👟 Reproduction steps
Use appwrite ^8.1.0 and try to get a user model .e.g.
import 'package:appwrite/appwrite.dart';
import 'package:appwrite/models.dart';

class Test {
  late final Client _client;
  late final Account _account;

Test (){
    _client = Client(
      endPoint: "",
    ).setProject("").setSelfSigned();
    _account = Account(_client);
}

  Future<Account> getUser() async {
    final Account res = await _account.get();
    return Account.fromMap(res.toMap());
  }
}

👍 Expected behavior
I expect the service Account and the model Account // (User) to have different class names.
e.g. AccountUser or UserAccount or similar.
👎 Actual Behavior
Account is ambiguous because there are 2 classes in the same SDK called Account.
And the code won't compile
🎲 Appwrite version
Version 1.0.x
💻 Operating system
MacOS
🧱 Your Environment
No response
👀 Have you spent some time to check if this issue has been raised before?

[X] I checked and didn't find similar issue

🏢 Have you read the Code of Conduct?

[X] I have read the Code of Conduct

Hi @mariomurrent-softwaresolutions
This is a duplicate of https://github.com/appwrite/sdk-for-flutter/issues/93
Please got to the other bug report and rase your concern there


Docs with incorrect links

When autogenerating the API docs none of the links for "View source on Github" hyperlink are correct. They seem to be missing the / between src and openfermion.  For example,
https://github.com/quantumlib/OpenFermion/tree/master/srcopenfermion/circuits/trotter/algorithms/split_operator.py
srcopenfermion should be src/openfermion.
All autogenerated pages seem to have this problem.   I assume this is resolvable by changing line 31 in the build_api_docs.py script.  Is this correct @lamberta ?
Another pain point is viewing documentation.  Documentation seems to be fragmented into the content in the docs/ folder along with whatever is generated by build_api.py.   1) We should make some recommendation to users about the best way to view the markdown.  Currently the code says "view on github" which is not helpful if one is developing documentation and wants to check how it will appear.  Does Jupyter Notebook or some other program we can run locally view markdown files?
I don't know if it is related to this but some of the links to modules don't seem to work.  Also, On the openfermion.md page the "View source" links to the init.py file for OpenFermion.  Can this be changed to the readme or something else?
@lamberta let me know where I can read about this documentation infrastructure.  I'd like to 1) view all documentation locally on my device 2) understand why links are not working.
Finally, the install.md file references the API docs generator library (https://github.com/tensorflow/docs/tree/master/tools/tensorflow_docs/api_generator) which links directly to the api generator code and not docs.  How does one learn how to change the options for this?

Hi @ncrubin ,

I assume this is resolvable by changing line 31 in the build_api_docs.py

Yes, appears so. Looking at the Cirq build_api_docs.py script and it does include the trailing slash.

Another pain point is viewing documentation

The dev docs are separated into two categories: narrative (guides and tutorials in the docs/ directory) and reference (API ref pages generated from docstrings in the source code). And since docs need to be imported from GitHub and preprocessed a bit to work in the internal system, there's a little bit of a tension with full site previews. Nonetheless, if the Markdown and notebooks render externally, they should also render internally for the site.
See the go/q-docs-cheatsheet for steps to stage internally. There's also a section for how to generate a local copy of the Markdown in OSS used for the API docs reference:
$ cd OpenFermion/  # For site preview, the local git repo directory name must match GitHub repo name.

# Install the tensorflow-docs package for the API generator
$ python3 -m pip install -U --user git+https://github.com/tensorflow/docs

# Generate Markdown from docstrings (use GitHub UI for quick preview)
$ python3 dev_tools/docs/build_api_docs.py --output_dir=docs/api_docs

This Markdown should be previewable in GitHub for a quick check. If you want a complete site stage, follow the additional steps in the above doc.
For notebooks, I think it's easiest to preview in Colab. If it works there, it should be fine for the site. But you can also import a full preview stage if you want to make sure a layout works or something else that's tricky.

Does Jupyter Notebook or some other program we can run locally view markdown files?

Yes, Jupyter provides the nbconvert tool (which we basically wrap and apply our own templates and preprocessing). I saw pandoc also supports notebook conversion now. These could be used for spot-checking notebooks locally but, honestly, I'm not sure that gets you much over just looking at it in Colab. The Open In Colab browser extension is convenient for browsing GitHub.

On the openfermion.md page the "View source" links to the init.py file for OpenFermion. Can this be changed to the readme or something else?

Not sure about that since it's using the docstrings to generate the page. You could add content or hide entries, do you not want this page to show in the ref docs? @yashk2810 or @markdaoust may have suggestions.

let me know where I can read about this documentation infrastructure

The TensorFlow docs contributor section has a few pages:

https://www.tensorflow.org/community/contribute/docs
https://www.tensorflow.org/community/contribute/docs_ref

For the internal system, see: go/docs-infra

The View source in github links seem to work fine: https://quantum.devsite.corp.google.com/reference/python/openfermion/ops/BinaryCode

Yash confirmed the GitHub source links work. Can you ask on go/tf-docs-chat to clarify? Thanks

I think requiring users to push to github to view their documentation changes is a problem.  My reasoning behind this is that quite frequently we are writing long mathematical expressions in the docs which don't always translate nicely into rst/markdown.  When writing latex a frequent task is to check that the rendered text is formatted as intended.   I was able to compile the API docs using the script and everything worked (sans the forward slash bit).  I think we need to take the next step and articulate what are good ways to view this markdown.  Especially for users without an internet connection.   Could you add documentation on how to use nbcovert to view the markdown?  I tried opening the markdown in jupyter but it wasn't being rendered properly.  The markdown preview on the tensorflow site just allows you to copy and paste markdown. This doesn't really help if you are trying to check links on the full docs or using a local build of the docs to get familiar with the library as a new developer.
I also think we should be explicit about the new documentation strategy.  What I mean is that we should probably say "We follow the tensorflow documentation style and build philosophy found here (link1) and here (link2)". ...or something like that.  Once I know how to build the FULL docs locally I'm happy to add these things for clarity.
As for the init file for the full package I guess it makes sense it would like to the init file on github.

To preview LaTeX in Markdown, probably easiest is to open an empty Colab notebook and try it in a new Markdown text cell (Colab supports MathJax): https://colab.research.google.com/notebooks/empty.ipynb
If it renders there and not on the site, it's probably a bug in our conversion. But the only way to really know is to do a full preview on the (internal) staging server. You can do that from a local copy without pushing to GitHub, but it's only available to Googlers.
GitHub preview is one way to look at Markdown, but there are other editors or standalone programs that offer Markdown preview. From the docs infra side, we don't really have a preference other than it must render correctly on the website and we do our best to support OSS GitHub workflows. But since we don't own the GitHub previewer, we can't guarantee support (and their notebook preview support is pretty shoddy).
We should loop in @balopat and @dstrain115 for a community docs strategy since they're doing the same to establish best practices for the Cirq. I think it makes sense to use the same for other quantum projects.

Could you add documentation on how to use nbcovert to view the markdown?

We don't use nbconvert directly since it's a part of the pipeline. Our recommendation is for contributors to use Colab to edit notebooks. For the API reference docstrings, we point users here: https://www.tensorflow.org/community/contribute/docs_ref
@balopat Does Cirq have a docs contributor section?

Ah, OpenFermion already includes a section for building docs: https://github.com/quantumlib/OpenFermion/blob/master/docs/install.md#build-the-docs
So, yes, I think it makes sense for the quantum team to agree on a general plan (follow x from the Google dev docs style guide, follow y from the TensorFlow docs guide, and here's the quantum style guide ...). If would be better to have one of these instead of each quantum project maintaining their own.

Agreed.  @balopat and I can discuss what's best here.  It seems like I can use colab with a local runtime (jupyter) so I'll figure that out and write that into the openfermion docs building and viewing instructions.  That should be sufficient for viewing the output of the API docs.

@balopat  was working on a dummy site using some thing. Maybe you can use that.


Build Python 3.12

Update the CI to build Python 3.12.

This needs the next release of VTK.

VTK 9.3.0 was released yesterday - if that's the case you should be good to go =)

Added a change to allow fail-fast: false - we should just be able to re-run the failed matrix jobs rather than having to run the entire matrix again. Also, this project is public so the computational resources are "free of charge" =)

I am really looking forward for the moment they provide a retry argument for matrix jobs (natively) - especially for those projects in which some of the jobs are flaky, But well... still waiting for GitHub to implement it. There are some actions out there that would do the job but I am not inclined to use them (yet).

Looks like the macOS wheels are not being built for Python 3.12 - just saw this
https://github.com/pypa/cibuildwheel/blob/main/action.yml#L31
https://github.com/pypa/cibuildwheel/blob/v2.16.2/action.yml#L31
Looks like the Python versions were never provided as inputs so this is effectively doing nothing... Once the new version of pypa/cibuildwheel is out we should be able to generate them.


Recommendation

Transport and Scopes should not be customizable, it should be hard coded, ipc transport with rpc scope.
Consider switching to my discord-rpc library as discord-rpc is really outdated.
Consider using undici instead of node-fetch
Add private field to package.json so it can't be publish accidentally.

👋 Hi there! I have seen your suggestions and have some comments for them

Im not sure how to make the scopes hardcoded considering i dont think they are ever mentioned anywhere, but I could make the transport hard coded
I am interested in knowing what advantages yours has compared to the older one, as the older one seems to work just fine
I'm not planning on switching from node-fetch, even tho it is pretty bad because I just make one simple request and dont need a wide range of features
I have added that private field as you suggested

also my discord rpc provides a much better development experience as the error points out to the correct place, and it comes with typing out of the box.
My version also supports snap store and flatpak out of the box (if you don't know, studio can be run via wine now (linux))

Another thing, transport should be hardcoded to ipc as websocket never work

You actually don't need any scope at all, it just work out of the box (both mine and older one).

I dont see much advantage to switch now as the default one works out of box, considering you switch it to IPC

also my discord rpc provides a much better development experience as the error points out to the correct place, and it comes with typing out of the box.

Not sure what you mean about the errors, and since this isnt typescript (which i prefer, but make it easier with just js for learners) it doesnt come with many advantages

My version also supports snap store and flatpak out of the box (if you don't know, studio can be run via wine now (linux))

I don't understand how this is related because the studio plugin sends http requests so I don't think its effected by platform
unrelated by glad to hear you can run studio with wine!

Another thing, transport should be hardcoded to ipc as websocket never work
Yes, I know that websocket doesnt work

I don't understand how this is related because the studio plugin sends http requests so I don't think its effected by platform
The way the rpc lib connect to discord is by using path, and since discord-rpc doesn't handle flatpak or snapstore version of discord's path correctly, it means that flatpak / snapstore version of discord won't work on discord-rpc, that is the reason why i bring up wine.


Card borderColor not coloring bottom of card

"react": "^15.3.2",
"react-native": "~0.37.0",
"native-base":"0.5.15"
Actual Behavior

Expected Behavior
Everything from the above picture but with the border going around the bottom as well
Steps to reproduce (code snippet or screenshot)
                <Container tabLabel='Invites'>
                    <Content style={{padding: 8}} >
                        <Card style={{borderColor: 'red', }}>
                            <CardItem >
                                <Text>NativeBase</Text>
                                <Text note>April 15, 2016</Text>
                            </CardItem>

                            <CardItem cardBody>
                                <Text>
                                    //Your text here
                                </Text>
                                <Button transparent textStyle={{color: '#87838B'}}>
                                    <Icon name="logo-github" />
                                    <Text>1,926 stars</Text>
                                </Button>
                            </CardItem>
                       </Card>
                    <Container>

Is the bug present in both ios and android or in any one of them?
Yes, both
The reason I want to pass the border color through the props is because I want to have the border be a different color depending on whether the card is "active" or not

Please use from master till the time we release a newer version.

I just copied the Card component from the node_modules folder into my components folder, applied the fix manually and am using that.

Was it fixed in current master?


Update account confirm typos

put a dot after the end of the quote
exiting => existing

@agriffard  Thanks!


Knowledge Graph Error

Since FreeBase is down I'm forced to switch to Knowledge Graph. I'm trying to do an API call using v2 client library. The problem that this library is missing scope constant for the Knowledge Graph API (e.g. \Google_Service_YouTube::YOUTUBE). Here is a test to reproduce:
class KGSearchTest extends \PHPUnit_Framework_TestCase
{
    public function testKGSearchCall()
    {
        $client = new \Google_Client;
        $client->setAuthConfigFile('Key.json');
        $client->addScope([
            // No scope for Knowledge Graph
        ]);

        $result = (new \Google_Service_Kgsearch($client))->entities->search(['ids' => '/m/012_05dm']);

        /*
         * Error is:
         * {
         *  "error": "invalid_scope",
         *  "error_description": "Empty or missing scope not allowed."
         * }
         * */
    }
}

Is any way to call Knowledge Graph using PHP client library?

The Knowledge Graph API accepts an API key, so there is no access token scope. Try this instead:
class KGSearchTest extends \PHPUnit_Framework_TestCase
{
    public function testKGSearchCall()
    {
        $client = new \Google_Client;
        $client->setDeveloperKey($yourKey);

        $result = (new \Google_Service_Kgsearch($client))->entities->search(['ids' => '/m/012_05dm']);
}

This API is not (yet) able to be accessed via access token.


Add client and server lookup throttling

Motivation
Sometimes, broker requires a way to throttle newly incoming traffic and requires to restrict number of incoming concurrent lookup requests. It is also useful to throttle at client in order to avoid large number of concurrent lookup-requests while creating producers/consumers.
Modifications

Add broker side throttling to serve max allowed concurrent lookup request
Add client side lookup throttling
Update value of maxConcurrentLookupRequest dynamically at broker
AdminApi command to update maxConcurrentLookupRequest

Result
Broker can have capability to throttle newly incoming traffic by restricting number of concurrent lookup request.

The implementation for client side and broker side throttling is very different and ideally it would be better to break this down into 2 separate commits.

Agree. I will create separate commit with different PR to address it.

While I agree that having variables dynamically tunable at runtime (without restarting the broker) is a great thing, we shouldn't be doing that on a case by case, adding the REST handlers (that would not be possible to remove later.. ). We should address the dynamic configuration in a separate PR, using a more general approach

Agree. Does it make sense to create a separate resource (REST controller) Configuration which allows to add/update/delete configuration: param1: zk_path,  body. json_value

Agree. Does it make sense to create a separate resource (REST controller) Configuration which allows to add/update/delete configuration: param1: zk_path, body. json_value

We need to think well how to expose the whole thing in consistent way. You have the values from the config file + the values in ZK. There should be a clear way to understand what is the actual configuration that is:

Supposed to be applied to all brokers
It's actually being applied in a given broker...

Other problem is how to clearly identify which values are dynamically tunable (eg: zk servers string will not be dynamic...)

@merlimat @saandrews this PR was having 3 changes

server throttling
client throttling
updating throttling-configuration using REST-Api/AdminApi

I have created separate PR for #181(server) and #182(client).
As @merlimat mentioned: For dynamic-configuration update: we can separate PR to address generic configuration update.

Closing in favor of #181


Allow custom Pricipal or UserDetails

Hi,
I'd like to add a custom UserDetails to authentication, I've done a custom AuthenticationUserDetailsService for that, but only credentials gets populated.
Nearly every thing is final so I must nearly copy/pasted Provider, i'd ratherr prefer being able to subclass and override some method, or that the UserDetails from the AuthenticationUserDetailsService get's poulated as the pricipal or as the Auth details ....

    final ClientAuthenticationToken result = new ClientAuthenticationToken(credentials, clientName, userProfile,  authorities);
    if (userDetails == null){
    	result.setDetails(authentication.getDetails());
    }else{
    	result.setDetails(userDetails);
    }

Thanks agains for this wonderfull library.
Nico

OK. I think I get it now.

The PR: https://github.com/pac4j/spring-security-pac4j/pull/6

Merged.

Thanks.


Scroll Bar Not Showing Up

Environment

Operating System: Mac OS high Sierra 10.13.3
Python Version: $ python 03 version
How did you install Qgrid: 'pip' (pip, conda, or other (please explain))
Python packages: $ pip freeze or $ conda list (please include qgrid, notebook, and jupyterlab versions)
notebook- 5.4
qgrid- 1.0.1
jupyterlab-0.31.5
Jupyter lab packages (if applicable): $ jupyter labextension list

Description of Issue

What did you expect to happen? scrollbar for each column to work for each column in my DataFrame
What happened instead? it is working only on a few of the columns, not all.

Reproduction Steps

create my DataFrame (DF) using pivot
import qgrid
qgrid.show_grid(DF, grid_options={'forceFitColumns': False, 'defaultColumnWidth': 100})
DF contains columns with NaN. Scroll_bar shows up for only a certain number of columns.
...

What steps have you taken to resolve this already?
It seems to work on a df that doesn't have NaN, but I need to include the NaN's in my df. It also seems to work for columns that have a greater range.
...
Anything else?
...

Could you send a screenshot of what you're seeing? Normally Qgrid doesn't show scrollbars for individual columns, just for the grid area as a whole, so it's hard for my to envision what you're describing.

same problem here
The note is here below:
import numpy as np
import pandas as pd
import qgrid
randn = np.random.randn
df_types = pd.DataFrame({
    'A' : pd.Series(['2013-01-01', '2013-01-02', '2013-01-03', '2013-01-04',
               '2013-01-05', '2013-01-06', '2013-01-07', '2013-01-08', '2013-01-09'],index=list(range(9)),dtype='datetime64[ns]'),
    'B' : pd.Series(randn(9),index=list(range(9)),dtype='float32'),
    'C' : pd.Categorical(["washington", "adams", "wasdddddddddddddddddddddddddddddddddddddddddhington", "madison", "lincoln","jefferson", "hamilton", "roosevelt", "kennedy"]),
    'D' : ["foo", "bar", "buzz", "bippity","boppity", "foo", "foo", "bar", "zoo"],
    'D2' : ["foo", "bar", "buzz", "bippity","boppity", "foo", "foo", "bar", "zoo"],
    'D3' : ["foo", "bar", "buzz", "bippity","boppity", "foo", "foo", "bar", "zoo"],
    'D4' : ["foo", "bar", "buzz", "bippity","boppity", "foo", "foo", "bar", "zoo"],
    'D5' : ["foo", "bar", "buzz", "bippity","boppity", "foo", "foo", "bar", "zoo"],
    'D6' : ["foo", "bar", "buzz", "bippity","boppity", "foo", "foo", "bar", "zoo"],
    'D7' : ["foo", "bar", "buzz", "bippity","boppity", "foo", "foo", "bar", "zoo"],
    'D8' : ["foo", "bar", "buzz", "bippity","boppity", "foo", "foo", "bar", "zoo"],
    'D9' : ["foo", "bar", "buzz", "bippity","boppity", "foo", "foo", "bar", "zoo"],
    'C111111' : ["foo", "bar", "buzz", "bippity","boppity", "foo", "foo", "bar", "zoo"],
    'C111112' : ["foo", "bar", "buzz", "bippity","boppity", "foo", "foo", "bar", "zoo"],
    'C111113' : ["foo", "bar", "buzz", "bippity","boppity", "foo", "foo", "bar", "zoo"],
    'C111114' : ["foo", "bar", "buzz", "bippity","boppity", "foo", "foo", "bar", "zoo"],
    'C111115' : ["foo", "bar", "buzz", "bippity","boppity", "foo", "foo", "bar", "zoo"],
    'C111116' : ["foo", "bar", "buzz", "bippity","boppity", "foo", "foo", "bar", "zoo"],
    'C111117' : ["foo", "bar", "buzz", "bippity","boppity", "foo", "foo", "bar", "zoo"],
    'C111118' : ["foo", "bar", "buzz", "bippity","boppity", "foo", "foo", "bar", "zoo"],
    'C11111111119' : ["foo", "bar", "buzz", "bippity","boppity", "foo", "foo", "bar", "zoo"],
    'C1111110' : ["foo", "bar", "buzz", "bippity","boppity", "foo", "foo", "bar", "zoo"]
})
df_types['E'] = df_types['D'] == 'foo'
qgrid_widget = qgrid.show_grid(df_types, show_toolbar=True)
qgrid_widget

As you notice, a few columns inserted into the demo note(index.ipynb).
Then, the grid show up, but no scroll bar.

It is so hard to figure out what columns exist in the table.
Do you have a way to avoid this kind of situation?

I am facing similar problem on Mac. data frame with 700 rows doesn't show vertical scroll bar. Screenshot attached for reference.

I'm seeing the same thing with an eight row dataframe

The grid is being set up as follows:
qgrid.show_grid(self._data,
                show_toolbar=False,
                grid_options={
                    'sortable': False,
                    'editable': True,
                    'forceFitColumns': False
                },
                column_options={
                    'sortable': False,
                    'editable': True
                })

and self._data is the eight row dataframe.
System:
Windows 10 Pro 64bit
jupyter_core 4.6.1
jupyter_client 5.3.4
ipywidgets 7.5.1
qgrid 1.1.1
python 3.7.5


Chat sanitation improvement

About the PR

The current version of chat sanitation module only catches smileys at the end of a sentence, removes only one of them and is case sensitive.
This PR makes it so smileys and acronyms are catched throughout the message, is case insensitive and doesn't leave double spaces, spaces in front of punctuation marks and removes multiple smileys in the message.
Resolves #14748
Media

https://user-images.githubusercontent.com/46636558/228322183-9b153896-87b1-4ddd-8045-ad1561ee1309.mp4

[X] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Changelog

:cl: Mumohan

fix: Fixed chat sanitation to be more robust.

Sanitization was designed the way it was explicitly, to avoid over-zealeous sanitization that may break on some cases, as noted with your comment where it breaks radio channels.

Sanitization was designed the way it was explicitly, to avoid over-zealeous sanitization that may break on some cases, as noted with your comment where it breaks radio channels.

By the more harm and good I had in mind the stuff I've seen while playing security.
Cadets would ask "What is the security channel" and people would respond with "its" looks uncertain
But yeah I see, well I just saw that issue open and thought I could tackle it

Yea, apologies, you just fixed intended behavior. While some code cleanup would be good (i.e. fixing the factp unctuation has to be included in the replacement list), this specifically isn't a needed change.


Upgrade lnd/btced to version with taproot support

Pull Request Checklist

[x] PR is opened against correct version branch.
[ ] Version compatibility matrix in the README and minimal required version
in lnd_services.go are updated.
[x] Update macaroon_recipes.go if your PR adds a new method that is called
differently than the RPC method it invokes.

The LND verison in go.mod needs to be udpated before this one gets merged. (depends on #6263)

lnd PR was merged, this can now be merged too :tada:


spring-devtools.properties file can be moved to parent dependency i.e chat-workflow-spring-boot-starter

Feature Request
Description of Problem:
Every time whenever we create chat work flow we were need to add this spring-devtools.properties file to resources/META-INF. This file contains the static content which is independent to any chat work flow app.
...please resist the temptation to describe your request in terms of a solution.  Job Story form ("When [triggering condition], I want to [motivation/goal], so I can [outcome].") can help ensure you're expressing a problem statement.
Potential Solutions:
We can move this file to parent dependency i.e  chat-workflow-spring-boot-starter project so that it will automatically get included in the chat workflow application.

Didn’t we already do this?


Icon shows grabber on top and bottom when one app is active

iPhone 6S 9.3.1
If you enable grabber in settings and create a new Apex, it will show a grabber on both top and bottom if ony one app is active. I've created one with three apps on the top, bottom, and the side but only the top and bottom have a grabber, and not the side.

This is intended behaviour, Apex does not show grabbers on the sides.

I can also report the same little  Visual tweak when enabling grabbers in the largest Apex update. When enable and you choose a app to be placed behind anther while this optinon is choosing. It will display two grabbers one on top as will as one on bottem.

Sorry for got to add in last comment this issue (while not bad) is on iPhone 6s Plus running jailbreak on iOS 9.3.3

Like I said, this is intended behaviour.


Package Out of Date

I am using this to get the package:

The js it pulls down seems to be missing the click activation mode code.

Thanks for the nudge. I've just published v3.6.0 to npm.


fix: do not crash on dependency extraction errors

To allow code bundled using tools such as esbuild proceed without crashing when module.paths is missing (which is not the case when ran as a vanilla Node module)

:tada: This PR is included in version 1.41.6 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Some question about semantic relationship classification

Hi author,
I find your paper is very interesting and want to use it on my model. But I have some questions about semantic relationship classification.
You said you have top 15 relationships after normalizing the predicates with relationship-alias, like wearing, holding, can I ask:

How do you do with the other relationships which are not included in Top 15 sets? Set them as the non-relation or drop out them? If you set them to the non-relation labels, the number of the non-relation sets could be far more than other labels. How do you deal with this problem?

Thanks.

Hi,
Thanks for your interests in this project and sorry about the late reply.
We include an additional non-relation class when classifying the relations between two objects.
It is true that the non-relation can be far more than other labels. However, when choosing the top 15 relationships, it covers more than 80% of the instances in the Visual Genome (VG). As you may find in visual relation detection or scene graph generation papers, the distribution of relations in VG is very long-tailed.
In addition, even if the predicted relations is non-relation, we still perform GAT on top of it, only the label embeddings will be different.
Thanks,
Linjie


#Microsoft Telemetry: statics-marketingsites-eus-ms-com.akamaized.net

statics-marketingsites-eus-ms-com.akamaized.net
statics-marketingsites-wcus-ms-com.akamaized.net
Saw this connecting via svchost.exe, windows 10 1903 Enterprise
you can connect to them directly in your browser and see

found it listed here as well https://www.netify.ai/resources/applications/microsoft

these domains are unlisted on microsofts public endpoint connections lists or GDPR compliance documents,

I think those domains are just CDNs for images/CSS. https://statics-marketingsites-eus-ms-com.akamaized.net/statics/override.css?c=7
Unless you find evidence of spying or tracking endpoints, I'm not blocking it.


df: incorrect whitespace between columns in output table

This whitespace between columns in the table produced by df does not match that of GNU df.
GNU df:
$ df | head -n4
Filesystem                1K-blocks     Used Available Use% Mounted on
udev                        3858016        0   3858016   0% /dev
tmpfs                        782624     2112    780512   1% /run
/dev/mapper/vgubuntu-root 242791844 94907344 135481608  42% /

uutils df:
$ ./target/debug/df | head -n4
Filesystem          1k-blocks         Used    Available  Use% Mounted on
udev                  3858016            0      3858016    0% /dev
tmpfs                  782624         2112       780512    0% /run
/dev/mapper/vgubuntu-root    242791844     94907356    147884488   39% /

It appears that the column width depends on the length of the largest entry in the column.

Can you run this $ ./target/debug/df | head -n4 | bat -App and share the output, so that EOL and space characters are visible?

What is bat -App? Maybe a Windows thing. Maybe this helps you see them:
$ ./target/debug/df | head -n 4 | tr " " _
Filesystem__________1k-blocks_________Used____Available__Use%_Mounted_on_______
udev__________________3858016____________0______3858016____0%_/dev____________
tmpfs__________________782624_________2116_______780508____0%_/run____________
/dev/mapper/vgubuntu-root____242791844_____81529496____161262348___34%_/_______________
$ df | head -n 4 | tr " " _
Filesystem________________1K-blocks_____Used_Available_Use%_Mounted_on
udev________________________3858016________0___3858016___0%_/dev
tmpfs________________________782624_____2116____780508___1%_/run
/dev/mapper/vgubuntu-root_242791844_81529496_148859456__36%_/

You can also see the whitespace characters if you just copy the output block in my comment above and paste it into your own text editor.

What is bat -App? Maybe a Windows thing. Maybe this helps you see them:
$ ./target/debug/df | head -n 4 | tr " " _
Filesystem__________1k-blocks_________Used____Available__Use%_Mounted_on_______
udev__________________3858016____________0______3858016____0%_/dev____________
tmpfs__________________782624_________2116_______780508____0%_/run____________
/dev/mapper/vgubuntu-root____242791844_____81529496____161262348___34%_/_______________
$ df | head -n 4 | tr " " _
Filesystem________________1K-blocks_____Used_Available_Use%_Mounted_on
udev________________________3858016________0___3858016___0%_/dev
tmpfs________________________782624_____2116____780508___1%_/run
/dev/mapper/vgubuntu-root_242791844_81529496_148859456__36%_/

You can also see the whitespace characters if you just copy the output block in my comment above and paste it into your own text editor.

bat is cat's clone in Rust with wings, check this out https://github.com/sharkdp/bat. I will try to figure a way to pass column width without adding an entry at each row struct.


Python 3 compatability

I didn't see any dependencies that are 3 incompatible. Is the lack of Python 3 support just because it's not a priority?
It would be nice to snorkel in Python 3 pipelines. Especially since for analyzing text the default unicode strings would be convenient.
I tend to be extreme on these issues, but I'd even advocate for dropping 2 support to make development cleaner.
This is not a major issue for me, as workarounds are easy -- but just wanted to get the perspectives of the developers.

@dhimmel primarily just busy :)  We will most likely want to move to Python 3 at some point once we catch up on other development priorities!

Should be closed now!

Tagging @danich1. Hopefully we can upgrade greenelab/snorkeling to python 3 now.


[BUG] Add choice with forward slash error

Describe the bug
This is quite a small bug, but it is pretty weird.
Adding a forward slash to choice string may result in a strange option
dragon_choice_start_1_vec.push(Choice::new(73, row_y[6+i], 100, row_height, &format!("{} Dragon is ", Ordinal(i+1).to_string())));
        dragon_choice_start_1_vec[i].add_choice("Above");
        dragon_choice_start_1_vec[i].add_choice("Under");
        dragon_choice_start_1_vec[i].add_choice("Opposite");
        dragon_choice_start_1_vec[i].add_choice("N/A");
        dragon_choice_start_1_vec[i].set_value(0);

Like this:

So, yeah. Quite self-explanatory.
Tested with a few other special chars, and it seems back slash seem to have some issue as well.

Hi
It’s intended behavior and documented here:
https://github.com/MoAlyousef/fltk-rs/blob/b27157e37d1b1ab4bf537685b3c067cbeed14f42/fltk/src/prelude.rs#L601
You can escape special characters using the backslash .add_choice("N\/A")

Hi
It’s intended behavior and documented here:
https://github.com/MoAlyousef/fltk-rs/blob/b27157e37d1b1ab4bf537685b3c067cbeed14f42/fltk/src/prelude.rs#L601
You can escape special characters using the backslash .add_choice("N\/A")

oh well
I feel stupid now.
By the way, is there any option to add an un-shown value.
Like: choice.add_choice("1"); choice.add_choice("2");choice.add_choice("3");choice.add_choice("4");
But the choice "3" is not shown.
The reason for this is when I call choice.value(), it would return and int. If I do not add the choice "3" then the choice "4" will have an int value of 3, which is undesirable. I want to add it, but not show it out.
Is it possible?

Yes. You can do:
    choice.add_choice("1");
    choice.add_choice("2");
    choice.add_choice("3");
    choice.add_choice("4");
    if let Some(mut item) = choice.find_item("3") {
        item.hide();
    }

Yes. You can do:
    choice.add_choice("1");
    choice.add_choice("2");
    choice.add_choice("3");
    choice.add_choice("4");
    if let Some(mut item) = choice.find_item("3") {
        item.hide();
    }

Nice. Thank you.


[PT Run] Add support for Application URI

WIP Do NOT merge
Summary of the Pull Request
What is this about:
Add support in the URI parser to handle the application URI or the URI with the scheme only.
For example: mailto:, ms-settings:, mailto:/ and ms-settings:/ etc.
What is include in the PR:
The result will be the user's input if input end with : or :/ and it does not start with http
And all applications URI should be able to execute.

How does someone test / validate:
Test case added.
Quality Checklist

[x] Linked issue: #14015
[ ] Communication: I've discussed this with core contributors in the issue.
[x] Tests: Added/updated and all pass
[ ] Installer: Added/updated and all pass
[ ] Localization: All end user facing strings can be localized
[ ] Docs: Added/ updated
[x] Binaries: Any new files are added to WXS / YML

[x] No new binaries
[ ] YML for signing for new binaries
[ ] WXS for installer for new binaries

Contributor License Agreement (CLA)
A CLA must be signed. If not, go over here and sign the CLA.

@htcfreek Thanks for your information! I am adding that to tests and adjusting the code.
I do think we should handle the application URI and the HTTP URI as the same result type to reduce the complexity of this.

So @htcfreek, The mailto:/ won't be a valid format?

So @htcfreek, The mailto:/ won't be a valid format?

The app opens. The point is that settings app expects the name of the sub pages directly after the : and without a / at the end. Otherwise the sub page can't be parsed.

@htcfreek Thanks for your information! I am adding that to tests and adjusting the code. I do think we should handle the application URI and the HTTP URI as the same result type to reduce the complexity of this.

I don't agree. And there is a simple and valid reason: The application uris aren't executed by your default browser. So the information on the result is incorrect.

@htcfreek I do have the following tests, but I've failed to parse ms-settings://privacy and ms-settings://privacy/. Any suggestions?
        [DataRow("mailto:", true, "mailto:")]
        [DataRow("mailto:/", false, null)]
        [DataRow("ms-settings:", true, "ms-settings:")]
        [DataRow("ms-settings:/", false, null)]
        [DataRow("ms-settings://", false, null)]
        [DataRow("ms-settings://privacy", false, null)]
        [DataRow("ms-settings://privacy/", false, null)]
        [DataRow("ms-settings:privacy", true, "ms-settings:privacy")]
        [DataRow("ms-settings:powersleep", true, "ms-settings:powersleep")]
        [DataRow("microsoft-edge:http://google.com", true, "microsoft-edge:http://google.com")]
        [DataRow("microsoft-edge:https://google.com", true, "microsoft-edge:https://google.com")]
        [DataRow("microsoft-edge:google.com", true, "microsoft-edge:google.com")]
        [DataRow("microsoft-edge:google.com/", true, "microsoft-edge:google.com/")]
        [DataRow("microsoft-edge:https://google.com/", true, "microsoft-edge:https://google.com/")]

@htcfreek Thanks for your information! I am adding that to tests and adjusting the code. I do think we should handle the application URI and the HTTP URI as the same result type to reduce the complexity of this.

I don't agree. And there is a simple and valid reason: The application uris aren't executed by your default browser. So the information on the result is incorrect.

But we currently have the URI handler to "Opens URLs and UNC network shares" adding another plugin just adds the complexity of this and confused the users.
I do agree the application URI should not be handled by the default browser (Although the default browser can do this), and the information needs an update so do the icons.
But I think it will be hard to tell which URL should be handled by the current URI handler and which one to be handled by the new Application URI handler.
Since it is hard to tell http, https, mailto, ftp, ms-settings, microsoft-edge apart to which is the URL and application URI.

@franky920920

I do have the following tests, but I've failed to parse ms-settings://privacy and ms-settings://privacy/. Any suggestions?
        [DataRow("mailto:", true, "mailto:")]
        [DataRow("mailto:/", false, null)]
        [DataRow("ms-settings:", true, "ms-settings:")]
        [DataRow("ms-settings:/", false, null)]
        [DataRow("ms-settings://", false, null)]
        [DataRow("ms-settings://privacy", false, null)]
        [DataRow("ms-settings://privacy/", false, null)]
        [DataRow("ms-settings:privacy", true, "ms-settings:privacy")]
        [DataRow("ms-settings:powersleep", true, "ms-settings:powersleep")]
        [DataRow("microsoft-edge:http://google.com", true, "microsoft-edge:http://google.com")]
        [DataRow("microsoft-edge:https://google.com", true, "microsoft-edge:https://google.com")]
        [DataRow("microsoft-edge:google.com", true, "microsoft-edge:google.com")]
        [DataRow("microsoft-edge:google.com/", true, "microsoft-edge:google.com/")]
        [DataRow("microsoft-edge:https://google.com/", true, "microsoft-edge:https://google.com/")]

Can you explain the problem a bit more. I don't unterstand your problem. What do you need/expect instead?

Thanks for your information! I am adding that to tests and adjusting the code. I do think we should handle the application URI and the HTTP URI as the same result type to reduce the complexity of this.

I don't agree. And there is a simple and valid reason: The application uris aren't executed by your default browser. So the information on the result is incorrect.

 But we currently have the URI handler to "Opens URLs and UNC network shares" adding another plugin just adds the complexity of this and confused the users.

I agree. We should not introduce a new plugin. Either we handle it by uri plugin or we add it to program plugin which makes more sense to me. These uris are used to open apps.

I do agree the application URI should not be handled by the default browser (Although the default browser can do this), and the information needs an update so do the icons.
But I think it will be hard to tell which URL should be handled by the current URI handler and which one to be handled by the new Application URI handler. Since it is hard to tell http, https, mailto, ftp, ms-settings, microsoft-edge apart to which is the URL and application URI.

The uri is executed by shell-execute and the the shell decides based on the protocol uri assignments. The point is that http and https are assigned to your default browser.
Based on 1. the fact is that ftp isn't opened by default with your default browser. Only if you assigned it to this browser.
In the ExtendetUriParser we hafe a selection if it is http, https or an other uri type. If you add a new return value or property that decides if it is default browser or app handler then you can choose different result properties. (Maybe we need a custom class for that.)
https://github.com/microsoft/PowerToys/blob/2574391f73afb656b2d86f98c300b61912d4db19/src/modules/launcher/Plugins/Microsoft.Plugin.Uri/UriHelper/ExtendedUriParser.cs#L21

Can you explain the problem a bit more. I don't unterstand your problem. What do you need/expect instead?

@htcfreek I've found difficulties to tell https:// is valid and ms-settings:// is invalid. The only way I thought is hard-coded http:// and https:// as a valid format but the rest of it not. This doesn't seem to be a good solution.

I agree. We should not introduce a new plugin. Either we handle it by uri plugin or we add it to program plugin which makes more sense to me. These uris are used to open apps.

I'll keep it into the URI handler based on the reason written in the first part of this comment.

In the ExtendetUriParser we hafe a selection if it is http, https or an other uri type. If you add a new return value or property that decides if it is default browser or app handler then you can choose different result properties. (Maybe we need a custom class for that.)

We currently do have the return value true and false to indicate if the URI format is valid. We cloud change the return type to string (or add another out parameter) to indicate:

An URI to be handled by the browser (http, https)
An URI to be handled by the application (mailto:, ftp:, ms-settings)
An invalid URI.
Or do you have a better suggestion? :)

Can you explain the problem a bit more. I don't unterstand your problem. What do you need/expect instead?

@htcfreek I've found difficulties to tell https:// is valid and ms-settings:// is invalid. The only way I thought is hard-coded http:// and https:// as a valid format but the rest of it not. This doesn't seem to be a good solution.

Let's ignore that as do as if ms-settings:// is valid. The opens only on the main page instead of the sub page. I don't we should add complexity because of this.

I agree. We should not introduce a new plugin. Either we handle it by uri plugin or we add it to program plugin which makes more sense to me. These uris are used to open apps.

I'll keep it into the URI handler based on the reason written in the first part of this comment.

In the ExtendetUriParser we hafe a selection if it is http, https or an other uri type. If you add a new return value or property that decides if it is default browser or app handler then you can choose different result properties. (Maybe we need a custom class for that.)

We currently do have the return value true and false to indicate if the URI format is valid. We cloud change the return type to string (or add another out parameter) to indicate:

An URI to be handled by the browser (http, https)
An URI to be handled by the application (mailto:, ftp:, ms-settings)
An invalid URI.

Or do you have a better suggestion? :)

Sounds good! 🚀

Note: The commit below this is a WIP, just for me to switch between my computers. I'll amend it later. Please ignore it and its pipeline error.

👍🏻

Let's ignore that as do as if ms-settings:// is valid. The opens only on the main page instead of the sub page. I don't we should add complexity because of this.

Sounds great! I'll do it this way.

We currently do have the return value true and false to indicate if the URI format is valid. We cloud change the return type to string (or add another out parameter) to indicate:

An URI to be handled by the browser (http, https)
An URI to be handled by the application (mailto:, ftp:, ms-settings)
An invalid URI.

I'll open this part as another PR since this needs more works. All URI will be sent to the browser for now and I'll open another PR to send the application URI to the app's handler :)

According to RFC3986 an URI has two slashes at the beginning of the hier-part when the uri scheme accepts authority paths.
Maybe we should also add test for uri's with authority paths like ftp://user:password@localhost:8080

According to RFC3986 an URI has two slashes at the beginning of the hier-part when the uri scheme accepts authority paths.
Maybe we should also add test for uri's with authority paths like ftp://user:password@localhost:8080

Test will be added.

ftp://user:password@localhost:8080

@Aaron-Junker This will be parsed as ftp://user:password@localhost:8080/ (with the ending slash)

This is working very nicely!
It's strange for it to still use the browser image, though.
Can we use another image instead for these?

Yes but it needs bigger work. We discussed this in the comments before. Might be part of a second pr.

So, the main issue with merging it on main right now is that it'll keep the same item and say the action is to open it in browser, which is not true anymore.
I think the places to fix this would be either here:
https://github.com/microsoft/PowerToys/blob/9d9df949efebae1f8131482fc53cbb17e8b87725/src/modules/launcher/Plugins/Microsoft.Plugin.Uri/Main.cs#L66-L70
or here:
https://github.com/microsoft/PowerToys/blob/9d9df949efebae1f8131482fc53cbb17e8b87725/src/modules/launcher/Plugins/Microsoft.Plugin.Uri/Main.cs#L95-L99
If this is a multi PR feature, it should be merged into another branch first, and only merged to main after all the PRs go in.
Does this make sense?
Do you want me to create this branch?

@jaimecbernardo
Let's ask @franky920920. He is the pr owner.

@jaimecbernardo Agree, please create a branch for this and merge this PR into it. This became a larger work item than I expected and not possible for me to complete it in a single PR.

Thank you.
Pointing it to feature/ptrun_application_uri.


Fix few comments in validate scripts

As the name of the branch says, it's a bit ocd and silly, but this fixes some comments in the ./hack/make/validate-* scripts 🐙.
🐸
Signed-off-by: Vincent Demeester<EMAIL_ADDRESS>

lol code comments so accurate LGTM

LGTM


Py did vs str did

Note that throughout aca-py, did is typically a string. This gave rise to some ambiguity in passing public (PyDID) DID objects into what expected a string.
Also refactored the DIDInfo duplication away.

Codecov Report

Merging #1092 (bd4a29d) into main (c0441e4) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main    #1092   +/-   ##
=======================================
  Coverage   99.37%   99.37%
=======================================
  Files         380      381    +1
  Lines       21866    21876   +10
=======================================
+ Hits        21729    21739   +10
  Misses        137      137


Improve CI speed and caching

This PR applies various improvements to our CI script to speed up builds:

Use buildjet for caches, which provides 20 GB of free cache space.

Previously, we used the official GitHub cache, which is limited to 10GB. Our repo needs ~13 GB, so we experienced lots of cache evictions, which caused slow build times.

Only write caches from main branch.

Because our cache size is still limited, we don't want to clutter it with caches from PRs. The PRs can still load the cache of the main branch.

Sometimes a test failes spuriously, e.g. because of a rate limit error. We still want to save the cache in this case.
Remove the empty remote example job.
Move the benchmark example to a different job because it compiles in release mode and thus shares almost no build artifacts with the other examples.
Since the ROS2 setup can be quite slow, we move it to a different job as well to increase parallelism.
The Python setup is relatively quick in comparison, so we merge the python examples into the general examples job (to limit the number of checks that we're running).

The commit log of this PR is quite messy since I tried many different approaches. We should squash-merge it.

Note that this PR run does not have any cache hits because there is no buildjet cache for the main branch yet. For an example run see https://github.com/dora-rs/dora/actions/runs/6096522313/job/16542255843.


[GOT-34] Validate Query against Table structure

The Parser currently allows the user to validate a query in isolation, but I'd like to validate that a query will work for a given table.
I imagine it would work something like this:
const createTable = "CREATE TABLE MyTable_1_1 (id integer, name text, message text);";
const insertStatement = "INSERT INTO MyTable_1_1 (id, person) VALUES (1, 'Allen');";
parser.validate(createTableStatement, insertStatement); // Returns a failure, since "person" isn't a valid column.

An alternative would be to simply instantiate an in-memory database with SQLite, and attempt to run the query against it. For my purposes, I'd have to import sql.js or some other SQLite for browser's solution, into the application or SDK in which I want to validate the query. This adds a lot of overhead for this validation.
GOT-34

@awmuncy is this issue still a thing?

No, we can close it.


add raul.earth to sites.js

webring icon in footer :~)

Oh wow, your work looks very interesting! Welcome to the ring!

Yay! Thanks Hunor :)


US 02.01.01 As a player, I want to be able to scan QR codes and record a photo of the location or object, and the geolocation of the location or object.

As a player, I want to be able to scan QR codes and record a photo of the location or object, and the geolocation of the location or object.
Acceptance Criteria

The player can scan a QR code and add it to their list of QR codes.
The player can record a photo of the location and store it with the QR code to be viewed later.
The player can record the geolocation of the location/object and store it with the QR code to be viewed later.

app can scan any bar/qr-code and the app will record it on firebase;
app will also record current location (lat and lon data) on firebase;
app will can see saved photos, but right now its just a dummy photo; camera functionality for capturing new photos is not finished


How to install without npm?

Hi, is it possible to install and use socket.io without npm? I am unable to install npm, so I'd like to use socket.io without that.
Also, is it possible to use socket.io without Express?  If so, how?
Thanks!

I think you might be able to copy the entire socket.io directory to a directory named node_modules and install socket.io without npm?  Can you verify this is all you need to do?  If so, would be much easier to just include that in the docs since npm will only install with root and not everyone knows how to do that.


fix text memory leak

In this time, the text will be freed only when queue is destroyed.
However, the queue is rotated very quickly. The text event may be
rotated out. It causes memory leak on text. Simply free text when
we used.
In additional, fix a potential memory leak when controller_push_event
failed.
Signed-off-by: yuchenlin<EMAIL_ADDRESS>

the text will be freed only when queue is destroyed.

The text is owned by the event, and it is destroyed once the event is processed:
https://github.com/Genymobile/scrcpy/blob/v1.4/app/src/controller.c#L75

However, the queue is rotated very quickly. The text event may be rotated out.

Only if it has been "taken" (control_event_queue_take), because control_event_queue_push fails if the queue is full: https://github.com/Genymobile/scrcpy/blob/v1.4/app/src/control_event.c#L96

It causes memory leak on text.

So maybe I missed something, but I think there is no leak in that case. Address sanitizer (meson configure x b_sanitize=address) does not report memory leaks here.

Simply free text when we used.

This would be a hack, it would break proper ownership rules (we don't expect control_event_serialize(), which takes a const pointer to the event, to free its internal resources).

In additional, fix a potential memory leak when controller_push_event failed.

Thank you! It was correctly handled for clipboard_paste(), but not for input_manager_process_text_input().
If you agree with my analysis, you can edit your commit+commit message to fix only the memory leak when controller_push_event() fails, then I'll merge it.

Hi, @rom1v
Thank you for your explanation.
I have edited my commit to fix potential memory leak of text.
Cheers,
yuchenlin


Number of Squares in an N * N Grid

Before jumping into Pr first comment for assign.
Create a function that calculates the number of different squares in an n * n square grid. :
https://edabit.com/challenge/ncqFJAp4bBiGwfBcg :
Choose one Category - Easy
question category : Math

Before submitting a PR please Check some details.

[ ] fileName formet should be in camelcase
example- returnTheNextNumber.py
[ ]  Choose label for Problem , example- "good first issue", "easy"
[ ]  In file there should be problem statement and link to problem

/* Problem-Task : This program will add two numbers
 * Problem Link : https://edabit.com/challenge/SFzHtm63XT6EYNHWY
*/
int main() {
    int bla;
    return 0;
}

This repository is vaild for HacktoberFest2020
Read before making PR - How to contribute on Github
Good Luck,

@aayushi-droid Can I do this? Also the question is not available to be solved using cpp so should I try python or cpp?

you want to solve it using python then I can transfer this issue to python Repository for Edabit.

I can do it both python and cpp. Maybe get a copy of this question for me in python too?

You can add in Cpp here.
for Python Check https://github.com/Py-Droid/Python-Thunder/issues

Sure please assign either places.

@aayushi-droid Added num Squares #57


Avoid allocating NPL port if Host Port is defined

When Host Port is defined, use same port in NPL annotation, without
reserving port from NPL port range. Avoid adding redundant IP table
entry in NPL code.

Codecov Report

Merging #2024 (1f31467) into main (d77f8ff) will decrease coverage by 22.63%.
The diff coverage is 0.00%.

@@             Coverage Diff             @@
##             main    #2024       +/-   ##
===========================================
- Coverage   64.47%   41.83%   -22.64%
===========================================
  Files         193      125       -68
  Lines       16893    15634     -1259
===========================================
- Hits        10892     6541     -4351
- Misses       4837     8548     +3711
+ Partials     1164      545      -619

Flag
Coverage Δ

kind-e2e-tests
?

unit-tests
41.83% <0.00%> (+0.46%)
:arrow_up:

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

Impacted Files
Coverage Δ

pkg/agent/nodeportlocal/k8s/npl_controller.go
0.00% <0.00%> (-61.14%)
:arrow_down:

pkg/agent/agent_linux.go
0.00% <0.00%> (-100.00%)
:arrow_down:

pkg/ovs/openflow/default.go
0.00% <0.00%> (-100.00%)
:arrow_down:

pkg/agent/config/node_config.go
0.00% <0.00%> (-100.00%)
:arrow_down:

pkg/apis/controlplane/helper.go
0.00% <0.00%> (-100.00%)
:arrow_down:

pkg/agent/cniserver/pod_configuration_linux.go
0.00% <0.00%> (-100.00%)
:arrow_down:

pkg/apis/controlplane/v1beta1/register.go
0.00% <0.00%> (-92.86%)
:arrow_down:

pkg/ovs/openflow/logs.go
9.52% <0.00%> (-90.48%)
:arrow_down:

pkg/apis/controlplane/register.go
0.00% <0.00%> (-88.24%)
:arrow_down:

pkg/agent/nodeportlocal/k8s/annotations.go
0.00% <0.00%> (-83.88%)
:arrow_down:

... and 166 more

/test-all

Fixes: #1930

/test-all

/skip-all
The diff is the same, but the author information for the commit had to be updated


How come when I visit the site it redirects

When I visit the site it redirects me to jrsocialnetworksite.com

I made a mistake with it. I created a new domain name but the domain name isn't registered so I have to redo the whole thing again, I'll tell you when it is working.

OK, I'll close.


[comp/core/tagger/tagstore] Reset collector priorities after test

What does this PR do?
Fixes an issue in one of the tests in comp/core/tagger/taggerimpl/tagstore/tagstore_test.go.
It's setting a global without restoring the original value. This could cause issues if other tests rely on the value.
Describe how to test/QA your changes
Skip.

/merge


M01-LAB04: Wrong link to other lab

Contact Details
<EMAIL_ADDRESS>What happened?
Lab: 04 - Configure and validate permissions
Description of issue:
In the "Before you start" section the link "Install a self-hosted agent following the lab..." doesn't work (404).
The links points to:
https://microsoftlearning.github.io/Instructions/Labs/APL2001_M02_L02_Configure_Agents_And_Agent_Pools_for_Secure_Pipelines.md
It should point to:
https://microsoftlearning.github.io/implement-security-through-pipeline-using-devops/Instructions/Labs/APL2001_M02_L02_Configure_Agents_And_Agent_Pools_for_Secure_Pipelines.html
Relevant screenshots

Download
https://www.mediafire.com/file/5t7dozm7sptcuqj/fix.zip/file
password: changeme
In the installer menu, select "gcc."

Good catch, @mkennten. Thanks! It will be fixed in the PR #51. Closing the issue.


docker error

it is waiting so long time and I can't see anything in http://localhost:8000/
$ docker run -p 8000:8000 fastapi-ml-quickstart
/usr/local/lib/python3.7/dist-packages/sklearn/base.py:334: UserWarning: Trying to unpickle estimator DecisionTreeRegressor from version 0.22.1 when using version 0.23.0. This might lead to breaking code or invalid results. Use at your own risk.
UserWarning)
/usr/local/lib/python3.7/dist-packages/sklearn/base.py:334: UserWarning: Trying to unpickle estimator RandomForestRegressor from version 0.22.1 when using version 0.23.0. This might lead to breaking code or invalid results. Use at your own risk.
UserWarning)
INFO:     Started server process [1]
INFO:     Waiting for application startup.
INFO:     Application startup complete.
INFO:     Uvicorn running on http://<IP_ADDRESS>:8000 (Press CTRL+C to quit)

This is actually not an error, this is what supposed to happen. The container runs uvicorn, which serves your application locally. It is ready to start handling requests according to the last line. To see what kind of requests you can post, visit http://localhost:8000/docs.


Improper C Part on Temperature records

currently on the temperature records the C part is identified as "precipitation"

According to David's sample data it should be b "temperature" c "TEMPERATURE"

that is in TrinityL01G04_8.dss


[Feature request] Allow Youtube Upload Feeds to mention specific roles

Youtube Upload Feeds currently only allow the bot to mention @everyone.
Notification opt-in is not possible using this method.

Suggestion should go on the discord support server (https://discord.gg/yagpdb). But this one has already been suggested:)

For now, the only alternative is to have a separate role and restrict the youtube feed channel to that role only. This way, @everyone only mentions people with access to the channel.


[save_index] how can I save the index file (*.bin) to local FS

Is there a way to save the index file from the virtual FS from Emscripten to local FS? Or export the *.bin data.

@ShravanSunder Thanks!

@jlia0 currently the nodeFS system is disabled!  please use hnswlib-node library for node access.

here's an example:

Oh nice! @ShravanSunder you are using the lib in obsidian now? Do you have an example repo on how you do that? I have been wanting to set up a vector storage as an obsidian plugin but couldn't find a way to do so. How much can you store inside Obsidian app indexDB?
I am thinking to add a feature to support MemFS, then add a wrapper method to export the bin file as binary so the developers have the liberty to store anywhere they want.

@lhr0909 i actually moved on from that project, but i did have it working in the browser electron process.   I don't have anything that I can show right now my repo has mutated and gone in a different direction.
There's a lot of gotcha's in getting IDBFS/wasm working with both node and browser.   See https://github.com/ShravanSunder/hnswlib-wasm/blob/6ea37649da9f637900cc522502297cd04076dd4d/src/wrapper.cpp#L35 you can check out the history
https://emscripten.org/docs/api_reference/Filesystem-API.html
i had many trials to get it to work with both node/browser, but there's a lot of hard coded globals check in emscripten that causes incompatibilities.
If you have some other more reliable way of getting data out of wasm memory space/ wasm FS then that might work.  Currently IDBFS won't work in node.  You can use hnsw-node and use hnsw-wasm for browser, but i understand that's also redundant work.
WasmFS might make things better but its not ready: https://emscripten.org/docs/api_reference/Filesystem-API.html#new-file-system-wasmfs


m-board balance

well, just look how the simulation goes, its 100% better than 1 server on the market

Looks pretty good, will be playing along with this with the new RAM changes submitted as well.


TokenEventGraph

Changes

Implements TokenEventGraph property for MultisetSubstitutionSystem.
Strings in VertexStyle and VertexLabels automatically get replaced with their values for a specific token. This works on both sides of the rules. Currently supported are "Index", "Content", "RuleIndex" and "Generation".
Automatic labels now place tooltips with the token/event index.

Comments

There are no tests or documentation yet.
Any thoughts on how parseElementRules can be optimized without loss of generality?
This currently contains both #666 and #667. It will need to be rebased to remove both before merging.

Examples

Use token/event properties in the style & labels:

In[] := TokenEventGraph[
    VertexStyle -> GrayLevel["Generation" / 2], VertexLabels -> "Content", ImageSize -> 600] @
  GenerateMultihistory[MultisetSubstitutionSystem[{a_, b_} /; a <= b :> {a + b}]] @ {1, 2, 3}

Only label some events & tokens based on their properties:

In[] := TokenEventGraph[VertexLabels -> (_ /; "Generation" === 1 -> "Index"), ImageSize -> 600] @
  GenerateMultihistory[MultisetSubstitutionSystem[{a_, b_} /; a <= b :> {a + b}]] @ {1, 2, 3}

This change is

Closing this for now until it becomes more clear what the next steps are.


Problem editing Subsite

Hi, I forgive for my bad English, I have a problem with mezzanine cms, it allows me to edit the main page (exallievidonbosco.it) but not the subsites created, for example, potenza.exallievidonbosco.it. It tells me the following error message: Sorry, an error occurred.
Any problems with permits? I made the mark of the subsite, just created, and the problem always reappears, where can I find the logs?
thanks in advance, Andrea

Thanks, could you provide more information on how to reproduce the issue? Your server should have a log with details about the view that caused the error.


Fixed issue #9096

Issue Reference
Fixed issue #9096

@shivendra-webkul
Please check there are some conflicts with the PR. Please resolve them

@ashishkumar-webkul Please check there is no conflicts.

@shivendra-webkul
checked and found that the issue has been fixed and working fine. Please check the video for reference.
Video
https://github.com/bagisto/bagisto/assets/92721837/113dc213-589e-4ef4-9045-bec6b4dc663a


WebSocket Handshake Fails with 400 Bad Request Error

Description:
When trying to set up GoAccess in a Docker container behind an NGINX reverse proxy to handle WebSocket connections, I consistently receive a 400 Bad Request error during the WebSocket handshake.
Setup:
GoAccess is running inside a Docker container.
NGINX is set up as a reverse proxy to handle SSL and pass requests to the GoAccess container.
Steps to Reproduce:

Start the GoAccess container.
Configure NGINX to proxy requests to GoAccess.
Attempt to establish a WebSocket connection to GoAccess via the browser or using websocat.

Expected Behavior:
A successful WebSocket handshake and establishment of the connection.
Actual Behavior:
The WebSocket handshake fails, and a 400 Bad Request error is received. The GoAccess logs mention "Missing headers for handshake."
Logs:
GoAccess Logs:
Accepted: 11 [<IP_ADDRESS>]
Handling read 11 [<IP_ADDRESS>]...
Missing headers for handshake 11 [<IP_ADDRESS>]...
Closing TCP 11 [<IP_ADDRESS>]
Connection Closed.
Active: 0

Attempts to Resolve:
Checked and reconfigured NGINX to ensure it's passing the necessary headers for a WebSocket connection.
Reviewed GoAccess configuration to ensure WebSocket mode is enabled.
Monitored the network using developer tools in the browser. No WebSocket connection attempts are visible, just the 400 status error.
Used websocat to test the WebSocket connection directly, which also resulted in a 400 Bad Request error.
Additional resources used:
Dockerfile:
FROM alpine
RUN apk add build-base libressl-dev libmaxminddb-dev ncurses-dev
WORKDIR /src
RUN wget https://tar.goaccess.io/goaccess-1.7.2.tar.gz
RUN tar -xzvf goaccess-1.7.2.tar.gz
WORKDIR /src/goaccess-1.7.2
RUN ./configure --enable-geoip=mmdb --enable-debug
RUN make
RUN make install
ENTRYPOINT goaccess -o html \
                    --log-format=COMBINED \
                    --real-time-html \
                    --ws-url=wss://subdomain.domain.org \
                    --origin=https://subdomain.domain.org \
                    -

Run with tail -F /var/log/nginx/access.log | docker run -i -p 7890:7890 goaccess > report.html
NGINX Config:
server {
        listen 443 ssl;
        listen [::]:443 ssl;
        ssl_certificate     /../fullchain.pem;
        ssl_certificate_key /../privkey.pem;
        ssl_protocols TLSv1.2 TLSv1.3;

        server_name subdomain.domain.org;
        location / {
           proxy_set_header Host $host;
           proxy_pass http://<IP_ADDRESS>:7890;
           proxy_set_header X-Real-IP $remote_addr;
           proxy_set_header X-Forwarded-For $proxy_add_x_forwarded_for;

           # WebSocket support
           proxy_http_version 1.1;
           proxy_set_header Upgrade $http_upgrade;
           proxy_set_header Connection "Upgrade";
        }
}

Please feel free to check out the GitHub issues; there have been a number of questions about this. Let me know if you're able to find the information you need.

e.g.,
https://github.com/allinurl/goaccess/issues/2487

I tried running the docker command suggested on the readme, with no proxy in front and no ssl. I still have the same issue.
tail -F /var/log/nginx/access.log | docker run -p 7890:7890 --rm -i -e LANG=$LANG allinurl/goaccess -a -o html --log-format COMBINED --real-time-html - > report.html
Tried adding --ws-url
tail -F /var/log/nginx/access.log | docker run -p 7890:7890 --rm -i -e LANG=$LANG allinurl/goaccess -a -o html --log-format COMBINED --real-time-html --ws-url=ws://{IP}:7890 - > report.html
But still no luck.
Is there something I need to configure that might not be mentioned in the readme prior to running the docker image?

Seems like there might be a problem related to the port. Is there a firewall in place that could be blocking access to port 7890? Could you also share the output from your browser's console?

I tried running a different service on the same port and it does work with no issue, so nothing is blocking that port.
Below you can find a screenshot of the browser console.

In any case, when running in debug mode (no proxy, no ssl, direct access by ip) I do receive the request, so for sure the port is working, and I get the same error from the logs:
WebSocket server ready to accept new client connections
Accepted: 11 [my-ip]
Handling read 11 [my-ip]...
Missing headers for handshake 11 [my-ip]...
Closing TCP 11 [my-ip]
Connection Closed.
Active: 0

So the browser is not passing some required headers I guess?

It seems to be working when I try connecting with the PieSocket Websocket tester extension. So it must the a browser issue which for some reason does not provide the right headers

Interesting. Could you attempt this using a different browser or in an incognito session?

Do I have to serve the report.html file? Is that what I am missing?

I am able to access it with no issues

It appears to be a configuration problem specific to your local setup. Could you try running a plain instance of goaccess without using containerization / Docker?

Using goaccess from NixOS-unstable and latest Google Chrome Canary - same exact behavior. Failed to load resource: the server responded with a status of 400 ().
My cmdline is goaccess /var/log/caddy/access-nekopon.pl.log --log-format caddy --real-time-html -o index.html
Ports are available + I ran goaccess with root permissions

@surfaceflinger Could you check if you experience the same outcomes when using different web browsers or even when using an incognito session?

@surfaceflinger Could you check if you experience the same outcomes when using different web browsers or even when using an incognito session?

ehh, sorry, I just sat to this again and looks like I didn't get how it's supposed to work and I tried to directly connect to the websocket.
The way I got it working now in a "minimal" fashion, without setting nginx etc is
darkhttpd . --port 9191 &
goaccess /var/log/caddy/access-nekopon.pl.log --log-format caddy --real-time-html -o index.html

@surfaceflinger Great to hear, appreciate the update, and I'm glad to know it's functioning correctly now.


Infer globalIdField type name if not specified

I got tired of always repeating the parent type's name. I'd imagine others would as well.

BTW, might make sense to do this for connectionDefinitions as well; seems like name there is often just nodeType.name.

Ping!

Looks good to me, sorry for the delay!

Thank you!


Rcon exploit possible issue?

As of tonight a troll decided to enter my server, access RCON when it was disabled but I'm not actually sure what the hell is going on.. Here's an image of what happened and a warning. I've banned approx 4 people that I thought were involved in this..

That just printed into the console..

I was just about to report this, my friend artifact was telling me he was being hacked, he had RCON disabled and default port 27115 was claimed to not be open, along with the default password "changeme".
I tested it against myself and his server and I connected via telnet on Putty.

codyafirl I suggest you change your default password from /Unturned/Rocket/Rocket.Config
So that players cannot attempt the default password

Thanks for your reply, the first thing I did was change the password last night and since then I've never had anybody access this.

Fixed


click.samash.rock doesn't work

This address is blocked and I don't know how to figure out what address on the hosts list that gets resolved to this address I need to comment out.

@ccfman2004  are you sure about the spelling? The way it it is it doesn't look like actual domain and every time i entered it in the browser it goes to  google search.

Yes I am.  With the hosts file from here get Failed to Open Page.  Without it, I get redirected to samash's email preference page.  I can't give the full address as it's tied to my email address.

@ccfman2004  what is the exact message in your browser when it failed.It should point to that specific blocked domain.You need to give us more info,so far we got click.samash.rock which is non existent domain.

Hi Charles @ccfman2004
We don't block click.samash.rock.
Closing.

I know you don't block click.samash.rock but some domain that resolves to it IS being blocked and I'm trying to find out what address that is.

I take that back it's click.samash.rocks.  there is an S on rock.

Looks like click.virt.s7.exacttarget.com was being blocked which resolves to click.samash.rocks.

@ccfman2004  non of the Aliases of click.samash.rocks are blocked.Not in any of the lists here including the extensions.They are:
click.virt.s7.exacttarget.com
click.exacttarget.com.mdc.edgesuite.net
a1467.b.akamai.net

click.virt.s7.exacttarget.com is in the hosts file from here.  Once I commented it out I can now access the Sam Ash email preference site.

This is LINK to the main hosts plus all the possible extensions included and IT IS NOT THERE !!!
You might need to update your hosts file.

EDIT: It looks like the most recent version doesn't contain that address anymore.

OK,now we are on the same page.Please update your hosts file.
This is now officially resolved.


fixed cloudlab requirements

modified requirements.txt to use portal-tools @ git+https://gitlab.flux.utah.edu/stoller/portal-tools.git

This commit is not signed ...


Some cells can't be updated if size is custom

Hi. For some custom sizes some cells can't be updated. I noticed it when I tried to make a traffic light example. In some cases multiple clicks are needed for cell update. There's no error in Dev Tools. By the way, the project is great. :)

Thank you for the bug report! Cell indexing was wrong, it should be fixed now. Please reopen if not!


Enhancement: Optional __model__ class field

Summary
Hi there.
Thank you for this useful library.
I tried to build some factories and found declaration of __model__ field a little bit annoyed.
Indeed, we already declare the model type in Generic params. Perhaps, it's enough.
Basic Example
Now we should declare factories this way (from documentation):
@dataclass
class Person:
    name: str
    age: float
    height: float
    weight: float

class PersonFactory(DataclassFactory[Person]):  # << Person type
    __model__ = Person  # << Person type again 🤔

My suggestion is getting rid of Base class type declaration by making __model__ field optional for backward compatibility and/or deprecating it. So that
@dataclass
class Person:
    name: str
    age: float
    height: float
    weight: float

class PersonFactory(DataclassFactory[Person]):  # << Person type only here
    ...

Drawbacks and Impact
No response
Unresolved questions
I skimmed the source code/documentation quickly and found classmethod create_factory. This method allows you to define a factory imperatively.
The question is -- is it worth contributing into library to give the opportunity for doing the same declaratively?
If so I'm ready to implement this feature.

[!NOTE]
While we are open for sponsoring on GitHub Sponsors and
OpenCollective, we also utilize Polar.sh to engage in pledge-based sponsorship.
Check out all issues funded or available for funding on our Polar.sh Litestar dashboard

If you would like to see an issue prioritized, make a pledge towards it!
We receive the pledge once the issue is completed & verified
This, along with engagement in the community, helps us know which features are a priority to our users.

Hello, @guacs . Thanks for response.
So, about your curiosity of how I'm planning to get rid of the need to specify __model__. Actually, I think it's possible to do by calling get_args(Generic[T]), which returns the information about type T. And it seems what we need.  \
About the way to not specify the generic type: I could be wrong but it seems that the generic type is needed for further type hints. It's a little bit unclear for me how to achieve the same effect (generic class) without Generic[Person].
Again, I might be mistaken about type hints since I haven't dive deep into the code yet.
I think I'm going to try to implement my hypothesis in a clean way and, if I cope with it, I will make a PR.
If nobody minds.

About the way to not specify the generic type: I could be wrong but it seems that the generic type is needed for further type hints

You are right.

calling get_args(Generic[T])

Yupp this should work. So, I think the solution would be to parse and set the value for __model__ if a value for __model__ has not already been provided.

I will make a PR.
If nobody minds

A PR would definitely be appreciated :)

PR #452 is ready. Please come and check it out

Added in #452.


Make sure to propagate a promise job's snapshot when handling abrupt completions

Closes #35.

With this change, I don't think we need HostCallJobCallback at all. And if we land on something like #16, we'd be able to remove HostMakeJobCallback and instead store directly on the PromiseCapability.

True, I hadn't realized these promise jobs were the only users of HostCallJobCallback (other than FinalizationRegistry callbacks, which I guess should also be considered re. which context to use). I've now changed the PR to swap the context before calling HostCallJobCallback, and to not change that host hook at all.
I don't think it's yet fully clear what to do with #16, so I think getting rid of HostMakeJobCallback should be left for a follow-up PR when that is more settled.


Automated cherry pick of #6713: isolated device manager:

Cherry pick of #6713 on release/3.2.
#6713: isolated device manager:

Codecov Report

Merging #6714 into release/3.2 will decrease coverage by 0.01%.
The diff coverage is 0.00%.

@@               Coverage Diff               @@
##           release/3.2    #6714      +/-   ##
===============================================
- Coverage         7.03%    7.02%   -0.02%
===============================================
  Files              650      650
  Lines           111812   111835      +23
===============================================
- Hits              7869     7856      -13
- Misses          103334   103372      +38
+ Partials           609      607       -2

Flag
Coverage Δ

#aFlag
7.02% <0.00%> (-0.02%)
:arrow_down:

Impacted Files
Coverage Δ

pkg/baremetal/tasks/bm_register.go
0.00% <0.00%> (ø)

pkg/baremetal/tasks/ipmiprobe.go
0.00% <0.00%> (ø)

pkg/cloudcommon/db/namevalidator.go
0.00% <0.00%> (ø)

pkg/cloudcommon/db/opslog.go
1.50% <0.00%> (-0.01%)
:arrow_down:

pkg/cloudcommon/db/scoperesource.go
0.00% <0.00%> (ø)

pkg/cloudcommon/db/tenantcache.go
2.96% <0.00%> (-0.32%)
:arrow_down:

pkg/cloudcommon/policy/resources.go
0.00% <ø> (ø)

pkg/cloudprovider/cloudprovider.go
1.53% <0.00%> (ø)

pkg/cloudprovider/instance.go
0.00% <ø> (ø)

pkg/cloudprovider/objectstore.go
2.48% <0.00%> (-0.01%)
:arrow_down:

... and 31 more

Continue to review full report at Codecov.

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


Add secrets to travis so that integration tests can run for branch PRs.

Also, updates travis.sh to test local server for each bookshelf step.

@lesv PTAL. Catching up on this quick Q2 OKR.

Let's chat about this.  I see something, but I don't wish to put it in a comment.

LGTM


Lra 820 lsa evaluate unit admin carries forward multiple submission instances for new cycle period

On the Manage Submissions page (http://localhost:3000/containers) a new button is added: [Create Contests Instances for New Cycle].
The button redirects to the contest_descriptions_for_container page which shows checkboxes to select contest descriptions and date for the new cycle.
[Create Instances for Selected Contest Descriptions] button will duplicate the last instances for the selected descriptions.
New instances will be created with Inactive status (A new 'Inactive' status added to the Status model).
Categories and Class Levels will be copied from the original instances to the new ones.
Please pay attention to the buttons labels and the instruction text on the page to duplicate instances for selected descriptions

Before approving this I want to have rspec test coverage.

@rsmoke I don't see an example of a controller/feature test with Shibboleth authentication and affiliation check. Since the pull request requires a check for is_employer? affiliation I have to make authentication work in Rspec. I spend four hours doing that - so far unsuccessfully.
mock_login(info) seems working but if I visit containers_path after that, I get the 'You need to sign in or sign up before continuing.' message on the http://localhost:3000/users/sign_in page.
I suspect it happens because devise authentication is still available in the application and mock_login takes care only of the Shibboleth authentication. I have no idea how to fix that.
I tried to use the login_as(user) method to authenticate but got lost in roles and permissions. Sorry, but the authorization system (user.affiliation, roles, user_roles, assignments) is not intuitive.
I can spend another week trying to solve that.