Add can view full contributor detail flag/group

Overview
This permission acts as an override to allow members of the "can submit private facilities" group to still see contributor details.
It replaces checking if the user has contributed any public data, allowing us to have more control over the visibility of contributors and to avoid a situation where a contributor submits thousands of private facilities and a single public one to get access to the full contributor details.
We also updated the history endpoint to follow the design of the recent changes to the facility details serializer where we conditionally show the contributor type instead of the full name unless the user making the request has the appropriate permissions.
Connects #874
Demo

Notes
Also fixes a bug where items submitted individually would appear in the history as being from "list [Unknown]." We now exclude the list description if there is no list associated with the Source.
Also includes a new user_groups management command. The Django Admin does not have a default UI for managing group memberships. This command allows us to use ecsmanage to update group memberships in ECS deployments where do not have access to an interactive shell_plus session.
Testing Instructions

Run ./scripts/resetdb
Log in a c2@example.com:password
Browse http://localhost:6543/?q=regina and select the facility in Shenzhen. Copy the OAR ID.
Browse http://localhost:8081/api/docs/#!/facilities/facilities_get_facility_history. Submit the test form with the OAR ID. Verify that a 403 error is returned.
Run management command to allow history access

./scripts/manage user_groups -e<EMAIL_ADDRESS>-a add -g can_get_facility_history

Submit the API test form again and verify that history is returned and that the "detail" values contain full contributor details.
Run management command to allow private submission access

./scripts/manage user_groups -e<EMAIL_ADDRESS>-a add -g can_submit_private_facility

Submit the API test form again and verify that history is returned but that the contributor details are now anonymous.
Browse http://localhost:6543/facilities/{the-oar-id} and verify that the contributor details are anonymous.
Run management command to allow full contributor details

./scripts/manage user_groups -e<EMAIL_ADDRESS>-a add -g can_view_full_contrib_detail

Submit the API test form again and verify that history is returned and that the "detail" values once again contain full contributor details.
Browse http://localhost:6543/facilities/{the-oar-id} and verify that the full contributor details are displayed.

Checklist

[x] fixup! commits have been squashed
[x] CI passes after rebase
[x] CHANGELOG.md updated with summary of features or fixes, following Keep a Changelog guidelines

Thanks for the review.


Support for docker-compose

Hello @ekalinin. Support for docker-compose. Merge it, please.

Great patch! Thanks!


Add TLS/Auth to all UIs

Covers Prometheus, Alertmanager, and Grafana
TLS Added to all connections

No shortcuts, full cert verification enabled

Uses oauth-proxy for all authentication

Users log in with OCP credentials

Must have access to get namespaces

Prom -> Alertmanager uses token auth delegation
Grafana -> Prom uses basic-auth

Secure session cookie handling
Mandatory security, will require doc updates

Our docs only thinly document any UI access anyways

NOTE: Routes are now mandatory due to their use in oauth

At least they are secure

Known Issues:

oauth-proxy image spec needs attention
some inline k8s objects should probably be templates

I just finished testing this like so:

Delete / recreate service-telemetry project
Deploy STF as normal
Build and push local STO image
Edit CSV to use local image

I found an error I made with quotes during cleanup, and a stuck secret object from iterative testing, but when I corrected those everything converged to running pods and an idle STO.
I verified the following:

Login as kubeadmin via UI on all three services
"Save and test" on the datasource in grafana worked
Alarms propagated to Alertmanager

This looks great! I may have missed it, but I don't see the oauth sidecar being added to smart-gateway. Is that something you plan to do?

Yes, I have a POC working for prom -> SG, but most of the work will be in the smart-gateway-operator, not here in the STO.
This PR covers https://bugzilla.redhat.com/show_bug.cgi?id=1949651 which mentions only the "dashboards". You can track STF-607 on our internal JIRA for progress on the smart-gateway piece.

This looks great! I may have missed it, but I don't see the oauth sidecar being added to smart-gateway. Is that something you plan to do?

Yes, I have a POC working for prom -> SG, but most of the work will be in the smart-gateway-operator, not here in the STO.
This PR covers https://bugzilla.redhat.com/show_bug.cgi?id=1949651 which mentions only the "dashboards". You can track STF-607 on our internal JIRA for progress on the smart-gateway piece.

Ah right that makes sense. Thanks!

Walked through this again, and it looks pretty good to me.

Thanks, @leifmadsen. With the docs in place I'm pretty much ready to merge this, but thought that maybe I should put the final container image handling pieces in place first so that it has a hope of doing the right thing downstream. If you'd prefer to merge this now and handle the downstream stuff separately (arguably a good idea?), I'm good with that too - feel free to hit the merge buttons at your discretion, I'm happy with this content.

Walked through this again, and it looks pretty good to me.

Thanks, @leifmadsen. With the docs in place I'm pretty much ready to merge this, but thought that maybe I should put the final container image handling pieces in place first so that it has a hope of doing the right thing downstream. If you'd prefer to merge this now and handle the downstream stuff separately (arguably a good idea?), I'm good with that too - feel free to hit the merge buttons at your discretion, I'm happy with this content.

Awesome. I think doing the downstream work separately makes this easier to manage, and gives us an opportunity to test and develop against these changes. I think merging it sooner than later is the better approach here.
With that being said, I merged a few things last week, and it might be good to merge the current master branch into this one and just do one last CI test run (which is back online now), and if that passes, then I am happy to push the merge button along with the documentation PR.

Awesome. I think doing the downstream work separately makes this easier to manage, and gives us an opportunity to test and develop against these changes. I think merging it sooner than later is the better approach here.

:+1:

With that being said, I merged a few things last week, and it might be good to merge the current master branch into this one and just do one last CI test run

That's what the "continuous-integration/jenkins/pr-merge" does, right? (Or rather the reverse, it would checkout master, merge this, and test) - or am I mistaken/misled by the name?

recheck

recheck

recheck

recheck

recheck

recheck

Okay, we got though enough hurdles that now I can see that the smoketests are indeed broken since they use curl to the non-https interfaces which have been removed. I'll fix them up.

recheck

recheck


Adding new article for Paginated View for Paginated Reports

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


defsystem compilation error in slime

For a freshly created project
(cl-project:make-project #p"~/projects/common-lisp/hello-world2/")

I try to compile asd file with C-c C-k and I have the error:
../hello-world2/hello-world2.asd:16:53:
  read-error:
    READ error during COMPILE-FILE:

      The function COMMON-LISP-USER::SUBPATHNAME is undefined.

      (in form starting at line: 5, column: 0, position: 54)

Now if I remove :long-description and try to recompile:
Execution of a form compiled with errors.
Form:
  ((:MODULE "src" :COMPONENTS ((:FILE "hello-world2"))))
Compile-time error:
  illegal function call

And if I add asdf: to the defsystem I can actually compile asd file:
(asdf:defsystem "hello-world2"
  :version "0.1.0"
  :author ""
  :license ""
  :depends-on ()
  :components ((:module "src"
                :components
                ((:file "hello-world2"))))
  :description ""
  :in-order-to ((test-op (test-op "hello-world2-test"))))

Do I miss some crucial configuration for my sbcl?

I have the same problem.

ASD files are not intended to be compiled or to be loaded in that way. Use asdf:load-system.

@fukamachi according to https://lispcookbook.github.io/cl-cookbook/systems.html it should be possible. This also led me to think that C-c C-k should be possible.
Regarding your statement that .asd files are not meant to be compiled or loaded via C-c C-k is there some official source of information on this?


[<EMAIL_ADDRESS>B2C & knownAuthorities not working

Library

[ ]<EMAIL_ADDRESS>or<EMAIL_ADDRESS>[x]<EMAIL_ADDRESS>[ ]<EMAIL_ADDRESS>[ ]<EMAIL_ADDRESS>[ ]<EMAIL_ADDRESS>
Important: Please fill in your exact version number above
<EMAIL_ADDRESS>Framework
<EMAIL_ADDRESS>(irrelevant)
Description
After creating a new PublicClientApplication with a custom b2c authority and this authority in knownAuthorities, the client fails during logins, trying to validate the authority (calling https://login.microsoftonline.com/common/discovery/instance?api-version=1.0&authorization_endpoint=[authority] even tho the autority is in knownAuthorities in the conf).
Configuration
const msalConfig: Configuration = {
  auth: {
    clientId: '[redacted]',
    authority: 'https://[tenant name].b2clogin.com/[tenant name].onmicrosoft.com/[policy]',
    knownAuthorities: ['[tenant name].b2clogin.com'],
  },
  cache: { cacheLocation: 'localStorage', storeAuthStateInCookie: false },
};

Reproduction steps
First, the call stack where the problem occurs:

detectAuthorityFromUrl (d:/Dev/msal-common/dist/index.es.js#1835)
createInstance (d:/Dev/msal-common/dist/index.es.js#1854)
BaseClient (d:/Dev/msal-common/dist/index.es.js#1887)
SPAClient (d:/Dev/msal-common/dist/index.es.js#2947)
constructor (d:/Dev/ts/src/app/PublicClientApplication.ts#78)

Explanation:

PublicClientApplication creates a SPAClient

SPAClient invokes its base ctor (ignore the breakpoint)

The base ctor creates an Authority

While being created, the authority tries to detect its type

This detection looks for b2c known hosts

Doesnt find any and resolves to AuthorityType.Aad.

The problem is that B2CAuthority.B2CTrustedHostList is empty at this point.
B2cAuthority.setKnownAuthorities(_this.config.authOptions.knownAuthorities); in SPAClient is only called after the authority has already been created (look at step 2).
The Authority will always be of kind Aad (except for the Adfs special case). This authority kind checks the validity of the authority agains microsoft discovery service, which fails.
Expected behavior
knownAuthorities taken into account on PublicClientApplication creation.
Browsers
Not browser specific, but Firefox 76.0.1 (64 bits) used in my tests

From the codebase, looks like it's fixed but not released.
Is there any plan of releasing a new version soon ?

@known-as-bmf I will push out a new release as soon as we have merged in #1675.

@known-as-bmf<EMAIL_ADDRESS>has been released and should fix this issue. Please be aware there is currently an issue with the b2c service which is being tracked in #1795 . Closing this as the original issue has been resolved.

I have mentioned this issue due to receiving this error with the latest version 2.0.0 of msal 2 (msal-browser). Please see my initial comment in:
https://github.com/MicrosoftDocs/azure-docs/issues/59656

@vaughanh1 Please open a new issue if you are not able to get this to work. As mentioned above this was fixed in version 2.0.0-beta.3

Not to worry. Thanks to @tnorling, he pointed we out that my configuration of the know authorities field should not have the 'https://' at the start. Thanks again.


Static IP for container with driver overlay

Description

Steps to reproduce the issue:

docker network create --driver overlay --subnet <IP_ADDRESS>/24 --gateway <IP_ADDRESS>  mynetwork
Create test.yml file:

version: '3'
services:
  test:
    image: nginx:stable-alpine
    networks:
     my_network:
       ipv4_address: <IP_ADDRESS>

networks:
 my_network:
   external:
     name: mynetwork

docker stack deploy --compose-file=test.yml ng
docker exec -it 0dbca750f536 hostname -i

Describe the results you received:
docker exec -it 0dbca750f536 hostname -i
<IP_ADDRESS>

Describe the results you expected:
docker exec -it 0dbca750f536 hostname -i
<IP_ADDRESS>

Output of docker version:
docker version
Client:
 Version:      1.13.0
 API version:  1.25
 Go version:   go1.7.3
 Git commit:   49bf474
 Built:        Tue Jan 17 09:55:28 2017
 OS/Arch:      linux/amd64

Server:
 Version:      1.13.0
 API version:  1.25 (minimum version 1.12)
 Go version:   go1.7.3
 Git commit:   49bf474
 Built:        Tue Jan 17 09:55:28 2017
 OS/Arch:      linux/amd64
 Experimental: false

Output of docker info:
Containers: 2
 Running: 1
 Paused: 0
 Stopped: 1
Images: 40
Server Version: 1.13.0
Storage Driver: overlay
 Backing Filesystem: xfs
 Supports d_type: false
Logging Driver: json-file
Cgroup Driver: cgroupfs
Plugins:
 Volume: local local-persist
 Network: bridge host macvlan null overlay
Swarm: active
 NodeID: *******************
 Is Manager: true
 ClusterID: *******************
 Managers: 3
 Nodes: 5
 Orchestration:
  Task History Retention Limit: 5
 Raft:
  Snapshot Interval: 10000
  Number of Old Snapshots to Retain: 0
  Heartbeat Tick: 1
  Election Tick: 3
 Dispatcher:
  Heartbeat Period: 5 seconds
 CA Configuration:
  Expiry Duration: 3 months
 Node Address: ********
 Manager Addresses:
******:2377
******:2377
******:2377
Runtimes: runc
Default Runtime: runc
Init Binary: docker-init
containerd version: 03e5862ec0d8d3b3f750e19fca3ee367e13c090e
runc version: 2f7393a47307a16f8cee44a37b262e8b81021e3e
init version: 949e6fa
Security Options:
 seccomp
  Profile: default
Kernel Version: 3.10.0-514.6.1.el7.x86_64
Operating System: CentOS Linux 7 (Core)
OSType: linux
Architecture: x86_64
CPUs: 1
Total Memory: 1.907 GiB
Name: ***.***.pl
ID: ******************
Docker Root Dir: /var/lib/docker
Debug Mode (client): false
Debug Mode (server): false
Username: *****
Registry: https://index.docker.io/v1/
Experimental: false
Insecure Registries:
 <IP_ADDRESS>/8
Live Restore Enabled: false

I'm facing the same issue, any workaround?
network inspect:
[
    {
        "Name": "demo_demo-network",
        "Id": "qkz753dgyi3whzg8v2585xgnw",
        "Created": "2017-03-15T16:34:04.4570162-03:00",
        "Scope": "swarm",
        "Driver": "overlay",
        "EnableIPv6": false,
        "IPAM": {
            "Driver": "default",
            "Options": null,
            "Config": [
                {
                    "Subnet": "<IP_ADDRESS>/16",
                    "Gateway": "<IP_ADDRESS>"
                }
            ]
        },
        "Internal": false,
        "Attachable": false,
        "Containers": {
            "0b4393bfa6f3393b1b9660defd38b42bb1944cf77d53c828c5fb059d40c5180d": {
                "Name": "demo_agilepromoter-web-demo.1.27b74lz56o9u6cvhm65n2nl87",
                "EndpointID": "07fa53340f8934ae3f022388aefbca2afc61fa5bff66d36f5105b2815ec2b90d",
                "MacAddress": "02:42:ac:15:00:06",
                "IPv4Address": "<IP_ADDRESS>/16",
                "IPv6Address": ""
            },
            "1b7c48137f32c170c37462a1efba74219b908584ceeb407febb0d036a6311b44": {
                "Name": "demo_involves-proxy.1.icsct0u57dusjsh54h93bf7h2",
                "EndpointID": "ef722bc9fcda547065102ffbb8b4cf0838e786db226f5170aed7df7eba471b40",
                "MacAddress": "02:42:ac:15:00:05",
                "IPv4Address": "<IP_ADDRESS>/16",
                "IPv6Address": ""
            },
            "3459a415cb3b2c269dda9931cd1f244a476b276a89b46c170c90b8e454713d0c": {
                "Name": "consul_consul.1.okexirwxvnjgg99jtx7mf6t07",
                "EndpointID": "a6347480755a7051e72a360ce392eb9173078c1002cde387f3770a35699fef09",
                "MacAddress": "02:42:ac:15:00:07",
                "IPv4Address": "<IP_ADDRESS>/16",
                "IPv6Address": ""
            }
        },
        "Options": {
            "com.docker.network.driver.overlay.vxlanid_list": "4097"
        },
        "Labels": {
            "com.docker.stack.namespace": "demo"
        },
        "Peers": [
            {
                "Name": "ubuntu-xenial-ab5cbf3f4bc9",
                "IP": "_xXXXXXxxxxxx_"
            }
        ]
    }
]

compose file:
services:
  consul:
    image: consul
    networks:
      demo:
        ipv4_address: <IP_ADDRESS>
    command: agent -server  -bind=<IP_ADDRESS> -client=<IP_ADDRESS>  -bootstrap -ui

networks:
  demo:
    external:
      name: demo_demo-network

docker inspect:
            "Bridge": "",
            "SandboxID": "d026cc5489e61ac81ec367eb19d97cd7a84077614856d32fcf19fd9713f26e39",
            "HairpinMode": false,
            "LinkLocalIPv6Address": "",
            "LinkLocalIPv6PrefixLen": 0,
            "Ports": null,
            "SandboxKey": "/var/run/docker/netns/d026cc5489e6",
            "SecondaryIPAddresses": null,
            "SecondaryIPv6Addresses": null,
            "EndpointID": "",
            "Gateway": "",
            "GlobalIPv6Address": "",
            "GlobalIPv6PrefixLen": 0,
            "IPAddress": "",
            "IPPrefixLen": 0,
            "IPv6Gateway": "",
            "MacAddress": "",
            "Networks": {
                "demo_demo-network": {
                    "IPAMConfig": {
                        "IPv4Address": "<IP_ADDRESS>"
                    },
                    "Links": null,
                    "Aliases": [
                        "352b6e8d1dcf"
                    ],
                    "NetworkID": "qkz753dgyi3whzg8v2585xgnw",
                    "EndpointID": "",
                    "Gateway": "",
                    "IPAddress": "",
                    "IPPrefixLen": 0,
                    "IPv6Gateway": "",
                    "GlobalIPv6Address": "",
                    "GlobalIPv6PrefixLen": 0,
                    "MacAddress": ""
                }

docker info:
 Running: 2
 Paused: 0
 Stopped: 9
Images: 11
Server Version: 17.03.0-ce
Storage Driver: aufs
 Root Dir: /data/docker/aufs
 Backing Filesystem: extfs
 Dirs: 90
 Dirperm1 Supported: true
Logging Driver: json-file
Cgroup Driver: cgroupfs
Plugins:
 Volume: local
 Network: bridge host macvlan null overlay
Swarm: active
 NodeID: 0t5kyn2llpr6wg9gl9ervntsa
 Is Manager: true
 ClusterID: inj7j79lv9qyth7v7opbmwjnc
 Managers: 1
 Nodes: 1
 Orchestration:
  Task History Retention Limit: 5
 Raft:
  Snapshot Interval: 10000
  Number of Old Snapshots to Retain: 0
  Heartbeat Tick: 1
  Election Tick: 3
 Dispatcher:
  Heartbeat Period: 5 seconds
 CA Configuration:
  Expiry Duration: 3 months
 Node Address: <IP_ADDRESS>
 Manager Addresses:
  <IP_ADDRESS>:2377
Runtimes: runc
Default Runtime: runc
Init Binary: docker-init
containerd version: 977c511eda0925a723debdc94d09459af49d082a
runc version: a01dafd48bc1c7cc12bdb01206f9fea7dd6feb70
init version: 949e6fa
Security Options:
 apparmor
 seccomp
  Profile: default
Kernel Version: 4.4.0-66-generic
Operating System: Ubuntu 16.04.2 LTS
OSType: linux
Architecture: x86_64
CPUs: 2
Total Memory: 2.414 GiB
Name: ubuntu-xenial
ID: TJMX:TKKB:Y3CY:NWP2:75KW:UB6U:XKD7:X5LJ:AY4N:BVHK:MFW4:6ENX
Docker Root Dir: /data/docker
Debug Mode (client): false
Debug Mode (server): false
Registry: https://index.docker.io/v1/
WARNING: No swap limit support
Experimental: false
Insecure Registries:
 <IP_ADDRESS>/8
Live Restore Enabled: false

Thanks for reporting; Static IP addresses for services are not yet supported, see #25303 (and #24170, #29816).
I'll close this issue, because we're tracking this feature through those issues, and don't want the discussion to diverge. Implementing this for docker stack deploy depends on the feature being supported for services first. You can subscribe to those issues using the "subscribe" button.
Closing this issue, but feel free to comment after I closed 👍

I searched around and tried different possibilities and I was able to assign static ips to containers. I am pretty new to docker, I don't know if this is the right way though.
I created a swarm. In manager, I created an attachable overlay network with a subnet.
docker network create -d overlay --subnet=<IP_ADDRESS>/22 --attachable overlay-network
My docker compose:
version: "3.2"

services:
  hadoop-master:
    image: devhadoop-master
    container_name: hadoop-master
    .
    .
    .
    networks:
      overlay-network:
        ipv4_address: <IP_ADDRESS>

networks:
  overlay-network:
    external: true

Note the configuration under default networks where external is true. That is what made it work.


Add Generator.Equals

Cool! Thanks!

Suggestion to use the common topic: csharp-sourcegenerator :)


GitHub: to check if a PR is a "fork PR", compare the PR's base repo ID to the PR's head repo ID

Fixes #445

bors try

bors try

bors merge


[octavia-ingress-controller][OCCM] consider support of Gateway API

Is this a BUG REPORT or FEATURE REQUEST?:

Uncomment only one, leave it on its own line:
/kind bug
/kind feature

What happened:
https://gateway-api.sigs.k8s.io/ has been there for a while and it's supposed to be replace ingress object with CRDs defined, we should adopt this feature and consider enhance in various places in order to make CPO able to support it..
What you expected to happen:
How to reproduce it:
Anything else we need to know?:
Environment:

openstack-cloud-controller-manager(or other related binary) version:
OpenStack version:
Others:

/reopen

/reopen
/remove-lifecycle rotten

the Gateway API already enter into Beta ...

Hello there,
Gateway API has been in GA since October 31, 2023.
Is there any news regarding Gateway API support?

I'll be frank here, I don't think CPO currently has resources available to develop a new, complicated feature. We would gladly accept and help with contributions adding it, but I don't think developing Gateway API is on a plate of anyone at the moment.


SW-1054 Support localhost HTTP redirects, round two

The hack in commit 1e987c94 failed to solve the problem in the actual staging
environment; the port number of the original request didn't make it through
the AWS load balancer plus the Nginx proxy. Try a different approach that uses
the redirect URL that's passed in by the client.

Current dependencies on/for this PR:

main

PR #852   👈

This comment was auto-generated by Graphite.

This time I tested using a real AWS load balancer pointing at an Nginx proxy on an EC2 instance, which is how the real staging/prod environments work.


Added support for OpenBSD

Since the dynamic client is compiled by hand and not automatically fetched by EMACS, all that was changed in order to allow cargo to build was env.bash. The client was built successfully, but not installed in the system with cargo install --path ./. It was unclear whether this additional step was needed or not.
After completion of this pull request, I will perform a PR in my forked repo, and then instruct straight to install elisp-tree-sitter from there. This should allow me to then go into the repo and compile the client again by hand for testing.

Needless to say, that although the dynamic client does successfully build in OpenBSD. EMACS will not touch it unless the ELISP source files are modified to allow it to do so in "berkley-unix". So, returning to the editor and modifying those files. This time, I will allow time for my changes to mature like a fine wine before sending a pull request.


APIs double-encode its URL parameters

Today, the implementation double-encodes its URL parameters. This screws up (often) various ids or paths or other such things. To fix this, we need to remove the extra encodeURIComponent in the implementation.

fixed by https://github.com/lucacasonato/deno_googleapis/pull/6


Idea: Add digest statuses to Mastodon Bookmarks

Hello! Thinking aloud momentarily, would love your thoughts and gut check on this.
I've been thinking a bit about how to integrate this kind of digest experience more natively into Mastodon. Generating the stand alone HTML is a great POC, but user experience and availability would be improved if we had a way to get the results of the digest into Mastodon itself.
In an ideal world (imho, and I'm very biased having worked on this feature at Twitter previously) Mastodon's platform would implement something akin to Twitter's Collections API (aka “Custom Timelines”), which are effectively “lists for statuses”: A named data store into which people can curate posts in an arbitrary order. An application like Mastodon Digest would then add its filtered posts and boosts into a “Mastodon Digest” timeline, which the user could browse through any client.
Since Mastodon Collections don't exist today, I wonder if this could be prototyped by overriding the Mastodon Bookmarks feature. With a few additions for polish, I imagine this:

User would create a token with posting and bookmarks write permission, rather than read only.
When a run has matching posts, Mastodon Digest creates a private post “Mastodon Digest for December 19th 2022”, posts it, and adds it to the user's Bookmarks. This would serve as a chapter heading for the posts we've digested.
Each filtered post gets added to the user's Bookmarks.
The user can then view their bookmarks in any Mastodon client, and act on them in their own instance as they see fit.

Obvious caveat: This is a very opinionated use of the Bookmarks feature and I'm sure wouldn't align with how some people already use it. That's fine. It might work for many people and maybe demonstrate the value of arbitrary custom timelines in Mastodon.
You might activate it with a -b option, allowing the existing functionality to be used to preview and refine the filtering before writing anything to production.
Further thoughts

Bookmarks does seem to sort posts in reverse-chronological order of when the post was bookmarked, which is what this feature would need in order to ensure the new digest is at the top regardless of post age.
I'm not sure what happens if you Bookmark a post twice through the API. I'd hope they just get silent ignored, but should verify.
There's probably an interesting exploration to be done regarding the best order to insert digest posts into Bookmarks: If you enter them in reverse-rank and enter the “Mastodon Digest for…” post last, you'd create the optimal reading experience, but maybe fighting too hard against Mastodon's design and be better to add posts chronologically and have the user scroll backwards.
Could also have an “end of digest” post added, so that if a user is using Bookmarks for things besides Digests, they'd be better able to identify them in between the digests.
Running in this kind of native-and-headless mode would also lend itself to running Digest as a scheduled service for multiple-users.

Love your thoughts. Thanks.

Overall, I really like all the ideas here, though I'd probably default to saying they're out of scope for this repository. This might be the beginning of new tools or integrations!
A more specific reaction:

Each filtered post gets added to the user's Bookmarks.

David Colarusso has been doing some interesting work along these lines with their David's Alter (Algo) Ego bot. As I understand it, the bot sources posts in a very similar fashion to Mastodon Digest, but boosts them. All boosts are marked as unlisted, and (I think) David has that bot in a list that they can toggle to so to see the boosts. I believe David mentioned they explored almost exactly the bookmark idea you present, but worried about clobbering the other common uses of bookmarks. (Tagging @colarusso in case they wish to share more about this thought process!)

Closing this. I think it's an interesting discussion and idea, but out of scope for this project.


Is it possible to handle click on the date header in week view?

I can't find in documentation if there is any handler for clicking on a specific day in week view. It would be very user friendly to have a functionality supporting clicking on a date in week view, let's say where it says "Wed 12/20" and then handle this event to change the view into a day view for 12/20. Is it possible? If not can you suggest any workaround?
Thanks!

use the nonAgendaCellHeaderTemplateResolver function. you can customize the header

Thank you


[$1000] Divider line is visible even if there is only one message in thread

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Open any chat
Send message
From context menu select the Reply In Thread

Expected Result:
As there is only one message ( parent message ), there is no need to show divider line
Actual Result:
Divider line is visible for a single message
Workaround:
Can the user still use Expensify without this being fixed? Have you informed them of the workaround?
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[x] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: 1.3.29-0
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/43996225/f960dd2c-b317-4e27-b5ab-13d5c57bee8e

Expensify/Expensify Issue URL:
Issue reported by: @BhuvaneshPatil
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1686329080109869
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01e9ba5c57be811260
Upwork Job ID:<PHONE_NUMBER>853252096
Last Price Increase: 2023-06-20

Proposal
Please re-state the problem that we are trying to solve in this issue.
Divider line is visible even if there is only one message in thread
What is the root cause of that problem?
We hide divider only if it overlaps with new messages marker.
https://github.com/Expensify/App/blob/83ab0160a3b5c385fd26fffe20bdbccabf9249c0/src/pages/home/report/ReportActionsList.js#L119-L120
What changes do you think we should make in order to solve the problem?
Add case if no replies sortedReportActions.length === 1 to shouldHideThreadDividerLine condition.
// shouldDisplayParentAction && sortedReportActions.length > 1 && sortedReportActions[sortedReportActions.length - 2].reportActionID === newMarkerReportActionID;
shouldDisplayParentAction && (sortedReportActions.length === 1 || (sortedReportActions.length > 1 && sortedReportActions[sortedReportActions.length - 2].reportActionID === newMarkerReportActionID));

What alternative solutions did you explore? (Optional)

I reproduced this bug, agree that it's weird and unexpected behavior.

Based on the comment by @benjaminbennet, I'd like to get a second opinion from @Expensify/design to see if this is a bug or not.

Hm I would say this isn't a bug because we designed it to be this way. I don't feel strongly about removing it though. Thoughts @trjExpensify @JmillsExpensify ?

Sounds good to me!


There is a problem with the point cloud generated by the upgrade of OpenSfM.

Hello,
I was using OpenSfM v0.2.0 to generate point cloud data for objects. However, when I upgraded OpenSfM to v0.3.0 and generated point cloud data for the same object, I encountered a problem.
[Problem]
Only a portion of the generated point cloud data will be increased in size.
This problem only occurs when point cloud data is generated with GCP configured.
Does anyone know anything about this issue?
Thank you for your help.

Hi @hutu-nohito ,
Could you sure some dataset so we can investigate the issue.
Thank you,
Yann


remove dependency on NewtonSoft.Json

for .NET 5 and 6
Done #504

Thanks. I’ll review and merge asap.

Did you run into any issues with the .net 3.1 targets that prevented using system.text.json?

@AndrewTriesToCode ;   I ran all the unit tests.   If it's required to add a testing profile for .NET 3.1, let me know.

@AndrewTriesToCode >>  hold off on the approval
I just saw that:
https://docs.microsoft.com/en-us/dotnet/standard/serialization/system-text-json-overview?pivots=dotnet-core-3-1
we can make it work by adding the package explicitely;

@AndrewTriesToCode, here, ever more clean !

@bobbyangers thanks, much appreciated!


i18n(zh-cn): Update community-content.mdx

Description
Update community-content.mdx
Upstream #1914

Lunaria Status Overview
🌕 This pull request will trigger status changes.

Learn more
By default, every PR changing files present in the Lunaria configuration's files property will be considered and trigger status changes accordingly.
You can change this by adding one of the keywords present in the ignoreKeywords property in your Lunaria configuration file in the PR's title (ignoring all files) or by including a tracker directive in the merged commit's description.

Tracked Files

Locale
File
Note

zh-cn
resources/community-content.mdx
Localization changed, will be marked as complete.

Warnings reference

Icon
Description

🔄
The source for this localization has been updated since the creation of this pull request, make sure all changes in the source have been applied.


add support for broadcom-sonic/sonic-vs

I think we should add support for full-fat sonic vm (the non p4 variant) to containerlab and by proxy vrnetlab.
Broadcom have cut a vm here https://github.com/Broadcom/sonic-VirtualSwitch and for some reason the sonic upstream Jenkins pipeline now asks me for GitHub auth to continue to consume it (I might raise this in parallel.
Im unsure which is better to support but it would be nice to use at least one variant in conjunction with both lab projects.

we got a contribution for the upstream sonic vm in #220

looks like bcom removed their image altogether
https://github.com/Broadcom/sonic-VirtualSwitch/commit/04b4e182c6fd1cf55e49c649bc64208bd2f18964


Only Thin Strip of Map Displayed on Phones

5inch screen. Android 5.1.1

Hi
Rather worse than that ( screenshot attached )
Phillip
Sent from Yahoo Mail on Android
On Mon, 16 Jul 2018 at 11:26, Harry<EMAIL_ADDRESS>wrote:
Hi, thanks for the issue.
We have a known bug with styling on small screens at #236, is the screenshot there consistent with the bug you saw?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Nevermind about that screenshot, I think I've been able to replicate the issue.

@falconx Seems like 6294c715c739eaa9e6c11e75d6a953fd992456e0 is the breaking commit; do you know how we could fix this without reintroducing the IE bug?

Hi Harry
Sorry I'm at work during the day so I can only do responses in the evenings.
If you need another screenshot drop me an email.
Yours
Phillip

On Mon, 16/7/18, Harry Dalton<EMAIL_ADDRESS>wrote:
Subject: Re: [neontribe/gbptm] Only Thin Strip of Map Displayed on Android Smartphone (#257)
To: "neontribe/gbptm"<EMAIL_ADDRESS>Cc: "pw3032"<EMAIL_ADDRESS>"Author"<EMAIL_ADDRESS>Date: Monday, 16 July, 2018, 16:33
Nevermind about that screenshot,
I think I've been able to replicate the issue.
—
You are receiving this because you authored the thread.
Reply to this email directly, view
it on GitHub, or mute
the thread.

Additional Reproduction
The same issue appears to be present on the iPhone 5S (see #267).
I can also reproduce this when triggering the small-viewport view on desktop in Firefox.


Option to keep focus on browser address bar

When new tab is opened usually cursor focus is on address bar. I use this extension to redirect to my selfhosted dashboard app. That all works fine. But after redirect I would like to keep focus on address bar. If I want I use dashboard icons to navigate but if not I continue my usual routine to type and search recent pages or search suggestions.

This is not functionality which can be handled via extensions. See https://github.com/jimschubert/NewTab-Redirect#omnibar-support


Added tooljunction.io in ai directory

Use Your directory name not a general SEO title and resend PR


feat: created endpoint for user account deletion

Description
I have fixed the user delete endpoint and wrote comprehensive tests for it to ensure functionality and reliability. The endpoint now allows authenticated users to delete their own accounts.
Related Issue
This pull request addresses issue #123, which detailed the problems with the existing user delete endpoint and requested automated tests for this feature.
Motivation and Context
This change was required to fix the malfunctioning user delete endpoint. It resolves the issue by ensuring users can delete their own accounts securely and includes tests to verify this functionality. This improvement enhances user management and data integrity within the application.
How Has This Been Tested?

The changes were tested using pytest in a controlled environment.
Unit tests were written to cover different scenarios including successful account deletion, unauthorized access, and error handling.
Integration tests were run to ensure the endpoint works correctly with the rest of the application.

Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] My code follows the code style of this project.
[x] My change requires a change to the documentation.
[x] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[x] I have added tests to cover my changes.
[x] All new and existing tests passed.

follow proper PR naming convention. eg, feat: create end …..

Done boss

Fix your merge conflicts


any one please help me how to set 15 sec fast forward and backward in android mode

i am new in flutter plz any one tell me how to set 15 sec fast forward and backward in android mode i need it urgent. 15 sec fast forward  and backward is already available in ios mode but not in android plz hell me how to do it in android mode

This is not implemented for Android. Please don't submit multiple issues

please sir  implement it for android mode i need this functionality in urgent.


Add cluster details in qualification summary output

Fixes #918.  This PR adds columns Vendor, Driver Host, Cluster ID, Cluster Name in the user tools qualification summary. This will assist the customers distinguish different jobs especially when in Databricks where all applications have the same name.
User Tools Output:
File: qualification_summary.csv
,Vendor,Driver Host,Cluster ID,Cluster Name,App Name,App ID,Speedup Based Recommendation,Estimated GPU Speedup,Estimated GPU Duration,App Duration,Unsupported Operators Stage Duration,Unsupported Operators Stage Duration Percent,Speed Up Estimation Model
0,databricks-azure,<IP_ADDRESS>,0220-012345-aaabbcc,job-name-test1,Databricks Shell,app-20240220075000-0000,Recommended,1.89,1206911.55,2282520.00,17298.00,0.76,SPEEDUPS
1,databricks-azure,<IP_ADDRESS>,0220-012345-xxxyyzz,job-name-test2,Databricks Shell,app-20240220074214-0000,Recommended,1.62,418620.75,676256.00,2576.00,0.38,SPEEDUPS
2,databricks-azure,<IP_ADDRESS>,0220-012345-pppqqrr,job-name-test3,Databricks Shell,app-20240220075434-0000,Recommended,1.56,1122375.53,1750824.00,11822.00,0.68,SPEEDUPS
3,databricks-azure,<IP_ADDRESS>,0220-012345-dddeeff,job-name-test4,Databricks Shell,app-20240220083138-0000,Recommended,1.44,209442.94,300577.00,455.00,0.15,SPEEDUPS
4,databricks-azure,<IP_ADDRESS>,0220-012345-ggghhii,job-name-test5,Databricks Shell,app-20240220065555-0000,Not Applicable,1.33,1718690.03,2282016.00,17709.00,0.78,SPEEDUPS

Note:

For TCO, we will still group apps but instead of App Name, we will group by [Vendor,Driver Host,Cluster ID,Cluster Name,App Name]
This change also ensures Top Candidates view shows meaningful results

+----+-------------------------+------------------+-------------------------+
|    | App ID                  | App Name         | Estimated GPU Speedup   |
|----+-------------------------+------------------+-------------------------|
|  0 | app-20240220075000-0000 | Databricks Shell | Small                   |
|  1 | app-20240220074214-0000 | Databricks Shell | Small                   |
|  2 | app-20240220075434-0000 | Databricks Shell | Small                   |
|  3 | app-20240220083138-0000 | Databricks Shell | Small                   |
|  4 | app-20240220065555-0000 | Databricks Shell | Small                   |
+----+-------------------------+------------------+-------------------------+

Report Summary:
------------------  -
Total applications  5
Top candidates      5
------------------  -

How To Test

Use latest dev jar while testing
spark_rapids qualification --eventlogs $EVENTLOGS --tools_jar $SPARK_RAPIDS_TOOLS_DEV_JAR
Tested on Databricks, Dataproc and EMR

Some Design Questions

Should we replace the intermediate cluster_information.json to cluster_information.csv?

Since we are convert the JSON to Dataframe in user tools now

How should empty columns be handled?

For example, in OnPrem and Dataproc only Spark Driver would be present, remaining 3 columns would be empty


Library name change request

Library repository URL
https://github.com/adafruit/Adafruit_Wippersnapper_Arduino

Hi @brentru. I am confirming your request.
Unlike submissions, this type of request is still handled manually. The Library Manager index maintainer occasionally comes through to clear all the outstanding requests and closes the issues when it is done. You will get a notification from GitHub at that time.
After that, you can monitor the indexing of the library via its dedicated logs web page.
Information about that here:
https://github.com/arduino/library-registry/blob/main/FAQ.md#can-i-check-on-library-releases-being-added-to-library-manager

Done in https://github.com/arduino/library-registry/pull/1503#issuecomment-1160190765


blacklist plugin -> gradle resolution-rules plugin

We're pointing out a separate plugin that can solve a specific problem. Closing pull request.

Ah, ok, sorry.
According to http://nebula-plugins.github.io/documentation/plugin_overview.html the resolution-rules plugin is the successor of the Blacklist plugin, the Blacklist plugin is deprecated and the usage of the resolution-rules plugin is recommended instead.
Because of that i thought this is a copy & paste error of not changing it to the new name as I didn't think that you would reference a deprecated plugin to be used from a current plugin.


Changed "in order to" to "to"

Grammatical change
Reason for the change:
Link to original source:

Closes #

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Hello @Airtanker55,Thanks again for the PR! We want to show our appreciation by sending                                              you some swag. Could you please fill out                                              this form so we can send it to you? Thanks!                                             Team SendGrid DX


Require scenepk argument

As reported here.
This changes API, but since it already happened during the port I'm not incrementing to a new minor version.

Thanks @kanbouchou


⚠ Processors has degraded performance

In 8c7edcf, Processors ($STATUS_URL) experienced degraded performance:

HTTP code: 200
Response time: 73 ms

Resolved: Processors performance has improved in eba2e7a.


Incorrect extra number of overlays with multiple dialog

Bug:
What is the expected behavior?
When instantiating multiple dialogs, there should be only one overlay and update it's z-index so that it's over the previous dialog
What is the current behavior?
One overlay is create for each dialog and all of the z-index are always 1 so that it never hides the previous still opened dialog
What are the steps to reproduce?
http://plnkr.co/edit/hQKwfp?p=preview
click open modal 1 on main app + click open modal 2 in opened dialog

you can still interact with dialog 1
you see the overlay darken because there are now 2 overlays

What is the use-case or motivation for changing an existing behavior?

By creating a new overlay for each opened dialog, it darkens then look of the overlay for each on that is opened
Should not be able to interact with dialog 1 until dialog 2 is closed, currently you can still click on "open modal 2" from dialog 1 which keeps creating dialog 2s and overlay keeps getting darker and darker for each overlay instantiated

Which versions of Angular, Material, OS, browsers are affected?
Material: 2.0.0-alpha.11-3
Angular: All
Is there anything else we should know?

I'm not sure I agree that only one dialog should be open at a time. That being said, the backdrop stacking issues should be solved with https://github.com/angular/material2/pull/2276

@crisbeto I'm not advocating for one dialog at a time. Overlay stacking described in #2276 may be close to what I'm saying, though after cloning from master and running "gulp build:components" and rebuilding my app to use that version, I see no difference with the overlays in my dialogs (2 opened sequentially and left opened)

both overlays were still under both dialogs
because there were two overlays, they looked darker

IMO, both of these behavior are not correct. Maybe I'm not building from source correctly to see the change?

Here's what it looks like for me on master @guojenman:

Regarding the overlays looking darker, I think that my example illustrates why it's necessary: when you have multiple dialogs and one doesn't overlap the other completely, you need some kind of indication that the lower one isn't clickable (hence the darkening). It might make sense to have a config option to be able to disable it, though.

@crisbeto your build looks like the right behavior. Thank you.


Adds RunLLM Widget MKDocs configuration

Description
This PR adds configuration for the RunLLM Widget.
In order to test this out in a staging / PR preview environment we will need to add the domain where the site is deployed to our database for reCAPTCHA verification.
If you could please provide the preview URL to us we can handle this on our and and we will comment here once resolved.
Please ping @agiron123 for this.
Without reCAPTCHA verification set up, the Widget will at least show up on the bottom right hand corner of the docs site.

GitHub issue - link to GitHub issue (if there is one)
N/A

Review
Instructions on how to review the PR.
Test your site as usual to ensure that there are no issues after adding the widget.

Closing as this PR has now been opened up here:
https://github.com/quixio/quix-docs/pull/324


Higher Level Exceptions

Adding higher level custom exceptions in src\ansys\fluent\core\solver\error_message.py.

Updating respective tests for added exceptions.

@hpohekar You might have seen my early comment stating that the fact this PR represents a small change is a good thing.
Let's keep these changes focussed in a small area. Let's focus on launch_fluent first. It's a great example of a function that's composed of multiple steps that can go wrong.
Write tests that demonstrate that we can catch distinct custom exceptions for each failure type, including (in the first instance) failed to start fluent and failed to connect to fluent.

https://github.com/ansys/pyfluent/pull/2176


Need an API endpoint for dropping course

User can add course but cannot quit the course...

done. Merged to master


INC-329: Removed (unused) jQuery UI and simplified CSP

jQuery UI was included in all pages (as it was part of the layout
template) but it doesn't seem to be used anywhere.
Even worse, the js/css wasn't served by the application itself which is
a potential security issue.
Remove these means we can remove the CSP rules to allow requests to
code.jquery.com. It also means we can remove the
crossOriginEmbedderPolicy: true option which was added to avoid the
related CSP error.

Tested locally with browser console kept open to see if any errors would pop up. AFAICT after this change things seem to work as before (as expected)


Cta section

I have added the blurry background on the CTA section, added the social links, and aligned the subscription form

I have added the blurry background on the CTA section, added the social links, and aligned the subscription form


Queue depth refactoring along with new metrics

This patch takes a stab at simplifying queueDepth. Specifically, makes the following changes:

Hooks up queueDepth as an event handler within the extentmon iterator
Removes the special handling of DLQ destinations within queueDepth
Removes the special handling of un-assigned dst extents within queueDepth
Removes some of the redundant code in mIterator.go
Emits new metrics indicating count of extents in different states (from extentMon)

Overall, this patch has the effect of removing ~185 lines of code (even after adding new metrics) without modifying behavior. There are still some more things left to simplify in queueDepth in the next pass at this code:

queueDepth <-> store interaction (handleStartFrom / gaft etc)
replicaStatsMRU cache

Testing:
This code passes all the existing unit tests / integration tests for queueDepth.

Coverage increased (+0.5%) to 56.483% when pulling f44a85614a4acda0934f0ab435ef317c279f92f9 on metric into e289430d48fee4ac7f1e94f1c9703d3936c99971 on master.

Coverage increased (+0.6%) to 56.496% when pulling 9625e99acd7368e81ac1427dc0e9071335b923e7 on metric into d9b381a1ba1afe79931176a3bbc35b3938b60057 on master.

Coverage increased (+0.6%) to 56.494% when pulling fbf06ad33ed29ec28973c4e7ddbba2343e232a47 on metric into d9b381a1ba1afe79931176a3bbc35b3938b60057 on master.

Coverage increased (+0.6%) to 56.557% when pulling f3fd9880e2778da9d56ef9e38be442a7da46f132 on metric into d9b381a1ba1afe79931176a3bbc35b3938b60057 on master.

Coverage increased (+0.5%) to 56.713% when pulling 5011ac3cd939375b867e9f3030420b34197794df on metric into a8751aa070825879c9465ea83d1dbb69684b66a4 on master.


feat: add a NewSpan method which can create a span from all existed info

Which problem is this PR solving?

Resolves #397

Short description of the changes

add a New Method: NewSpan

yeah, I understand what are you saying, here I just want to make it clear that, when we are saying Client, there are two meaings:

a client, which is able to create, and upload a standard opentracing trace to agent, as we all know what is trace, what is span, operate name, start/end time, tags and so on, but the data format for all system is different, so a client, transfer this content to format in system is one kind of client
a client, which is easy to use by language using jaeger trace, just like this agent here, it offers StartSpan and so on, all internal info is hidden

In fact, these two client, we all need it in different scenes, for us, a async rpc is used in our system, this async means no response exists for request, all rpc are request, the releationship exists in content of rpc but not the format of rpc, so sometimes we can not get response time directly, so analyze the log, and create legal trace info, and upload it to jaeger agent is a way usging jaeger trace
yes, span pooling, with metrics and other things must be ok, I will try make it better, but I really need this feature, which allow people create a span, all with existed info(including spanId and so on), other wise, We can not use it in our envirment
Thanks!

if you're assembling spans offline from logs, have you considered submitting them directly to collectors in  thrift or protobuf format? That does not require any changes to this  library.

if you're assembling spans offline from logs, have you considered submitting them directly to collectors in thrift or protobuf format? That does not require any changes to this library.

yeah, I can do that
But I must do some work which jaeger agent have done now, such as sampler, buffer, flush time, reconnect and so on, maybe cost too much
And all of this have been done in jaeger agent, so maybe I will still use jaeger-client-go, but I have to fork a private branch
Please consider this pr, as I know, jaeger python client allow new a span all by myself, maybe we can learner from it (if resonable)
thanks!

@xbsura I have a need for this too (loading a span from context).
If this is not possible to support in master, I would be happy to help you maintain a private branch of this.

As a I mentioned in the earlier comment, I am not opposed to a public constructor, but it needs to fit naturally in the implementation, not be a copy-n-paste of the code that already exists in this repo. It's doable with some refactoring, but not how this PR is done.

See https://github.com/jaegertracing/jaeger-client-go/issues/397#issuecomment-517893569 for alternative solution proposal

OK, I will keep following this issue

a better solution was implemented in #411.


rewrite client to use ServiceTemplate based API

❌⚠ This PR is a major breaking change! ⚠❌
This is a rewrite of the Client and Config classes.
I added a modelMapping to the Config class. This way we are able to specify which index should be mapped to what Java Class.
The Client class now has accessors for the specific apis.
client.index() returns the IndexHandler,
client.documents("indexname") returns the DocumentHandler for the specified index.
client.documents("indexname", Movie.class) returns a DocumentHandler for the specified index with Movie as the desired Java Type, even if there is no mapping in the config for it.
client.instance() returns the InstanceHandler
client.keys() returns the KeysHandler
This way we dont have one uber-class that implements everything.
To make this easier, i created a ClientBuilder class. It allows the user to set every part of the ServiceTemplate classes (HttpClient, JsonHandler, RequestFactory, ServiceTempalte). If nothing gets specified, the ClientBuilder autodetects classes in the Classpath and loads the apropriate Implementation.

Hello @niemannd!
We are busy right now but we will come back to you as soon as possible! 🙂
Thanks for your PR!

Hi @niemannd !
I’m Amelie. I will take over from Sam, who changed team. First of all, thanks a lot for this PR! Besides the quality of the code than the rewrite of the Readme and Tests and for your clarity, this is nice from you, and I really appreciate it.
However, we want to keep all the SDK and this one too really simple for the users. So the goal is to keep the README and the usage like they are, even if it’s nice to have abstraction.
A default client still needs to be instantiated easily like this:
Client client = new Client(new Config("http://localhost:7700", "masterKey"));

even if we can let the possibility to customize it if we want.
And an index and document should still be accessible like this:
Index index = client.index("books");
index.addDocuments(documents);

Again thanks a lot for what you propose and your work. I understand your purpose not to have one uber class and create more accessibility. Still, this SDK is small and only contains few methods to bring too much complexity for the primary usage it offers.
I think we need to discuss a way to implement a solution that we could easily maintain, especially with simple usage.

A default client still needs to be instantiated easily like this even if we can let the possibility to customize it if we want

There are a few reasons i chose to not do that.

By circumventing the ClientBuilder you create a second "entrypoint" into the package. Generally i try to avoid that because sooner or later those entrypoints diverge and create additional problems.
Also i think client = ClientBuilder.withConfig(new Config("http://localhost:7700", "masterKey")).build(); shouldn't be much of a problem?
A second entrypoint is not really user friendly. Software evolves. Telling people they have to change the way they create their client to use some advanced features is ....

And an index and document should still be accessible like this:
Index index = client.index("books");
index.addDocuments(documents);

I chose not to do that because this creates a number of problems.

Coming from an object oriented mindset, in this case you are working on an index-Object, not on the index itself. I would not asume that adding documents to the index-Object triggers any network operation.
This would merge the control layer of the code with the data layer. You have to expect that information about indexes are used in other parts of the application or even across applications. Maybe they are serialized/deserialized it. If at some point the underlying client gets dropped, your index will not be able do perform. Thats why, even for the index, i seperated data class from control class.
This way might work for the add-operation. It wouldn't work for get-operatons. you simply would not know how to deserialize the returned documents.

Hi @niemannd,
I hear what you're saying and I completely understand. It's not an implementation problem but that we have to make choices about what we want to offer to developers and what kind of product we want.
And it's part of the Meili philosophy that the SDK should be as developer-friendly as possible so we don't want to change the README and the usage.
If you still want to talk about it you can join me on slack.

Hi @niemannd,

A similar pattern is used in the meilisearch-go sdk. Do you mean the ClientBuilder? Again, a similar pattern is used by meilisearch-go and even Spring Data Elasticsearch

We actually do a refactor of the go SDK for the same reason. It is noted that it’s wasn’t clear for the user to pass the name of the index in the function like:
client.documents("books”); -> Seems to say I get the documents “book” not documents inside the index “book”.
For the ClientBuilder it’s really nice to have the possibility to customize it only if we still can have a default client which can be instantiated like this to:
Client client = new Client(new Config("http://localhost:7700", "masterKey”));
I would totally understand if you didn't want to resume your redesign. But if you don't, it would be very cool if you would take back these few points.


User: update profile

Add method to update profile
Add method to change password

Done, the "user:update" method can be used to change profile information, as well as password, username and email address.


Merge dynamic configuration with cookbook configuration?

Like others here, I'm finding the breadcrumb mechanism a little unintuitive. Additionally, when using sentinel, it means you can't update the redis configuration after the first chef run without messing with the server state. So I'm wondering: is there a reason the provider can't merge its configuration with what the redis server wrote?
Redis delimits the configuration it writes with a comment: # Generated by CONFIG REWRITE, so the provider could easily append that to the configuration generated by the cookbook. I can submit a patch if this sounds good; just wanted to check if this idea has been considered/dismissed already.

I'd like to see a fix for the breadcrumb concept.  The initial PR missed updating the sentinel recipe so now I'm stuck having to manually delete this breadcrumb file in order to complete my chef run.  I tested this out with even setting default['redisio']['default_settings']['breadcrumb'] to false.


Nme webgpu5

Associated with #14929

Please make sure to label your PR with "bug", "new feature" or "breaking change" label(s).
To prevent this PR from going to the changelog marked it with the "skip changelog" label.

Snapshot stored with reference name:
refs/pull/15133/merge
Test environment:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/15133/merge/index.html
To test a playground add it to the URL, for example:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/15133/merge/index.html#WGZLGJ#4600
Links to test babylon tools with this snapshot:
https://playground.babylonjs.com/?snapshot=refs/pull/15133/merge
https://sandbox.babylonjs.com/?snapshot=refs/pull/15133/merge
https://gui.babylonjs.com/?snapshot=refs/pull/15133/merge
https://nme.babylonjs.com/?snapshot=refs/pull/15133/merge
To test the snapshot in the playground with a playground ID add it after the snapshot query string:
https://playground.babylonjs.com/?snapshot=refs/pull/15133/merge#BCU1XR#0

Visualization tests for WebGPU (Experimental)
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/15133/merge/testResults/webgpuplaywright/index.html


Test: WithTimeout executed on start

WithTimeout was not executed on start, so the first run was after
config.Ticker value (default to 5) With this change always will
executed on first run, if fail will executed each config.Ticker.
This change should make test a bit faster.
Signed-off-by: Eloy Coto<EMAIL_ADDRESS>

This change is

test-me-please

test-me-please

test-me-please

test-me-please

test-me-please

test-me-please

test-me-please

test-me-please

test-me-please


Add allowFontScaling option for TabView and `headerTitleAllowFontSc…

Add allowFontScaling option for TabView and headerTitleAllowFontScaling for HeaderTitle (#2377)

This looks awesome! Thanks for submitting :)
Can I only ask you one thing? Could you add this new option to the docs too?

@kelset Sure. I'll update this PR later

@kelset Docs added

THanks @finian - CircleCI is currently having some issues with flowtypes, I'll need to have somebody else help me out to understand why it's failing (doesn't seem related to your code).

@kelset @finian The errors are due to these changes. We can remove the errors by adding type information like https://github.com/react-community/react-navigation/commit/6c383e0e8400b385fde4111f3d59e5c8864b02e2

Thanks @exKAZUu for the insight, @finian ping me once you update the PR and I'll merge it ;)

@kelset PR updated


Source code snippets appear like two concentric boxes (with fix)

Source code snippets appear like concentric boxes, since they're rendered as a highlight div inside another.
This commit fixes it, making snippets appear as a single box.
I'm not creating this fix as a PR, since the project is abandoned, but just leaving here for helping other people with the same problem.

@saveriomiroddi, thanks for sharing the workaround.
For other people having this I want to also highlight that besides applying the commit above you should also change the line
overflow: scroll;

to
overflow: auto

for .highlight in _highlights.scss.

...

By checking the generated HTML code, the problem is caused by CSS on class "highlight" applied twice. To apply CSS only on
, just change the first ".highlight {...}" to "pre.highlight{...}". This would be a better fix for now.


is your code support multiple GPU?

is your code support multiple GPU?

Hi, sorry, I think this codes do not support multiple GPU. Some codes about data loader and training should be modified (I am not very sure since I am not familiar with the implementation of multi GPU in Torch). https://github.com/soumith/imagenet-multiGPU.torch This repo may helps you.


Tests for show_name functionality of the computational graph

This PR adds tests for the show_name option for the computational graph builder, to check if

setting the flag to True will add the variable name to the corresponding label.
setting the flag to False will work vice versa.

LGTM. All Jenkins tests have passed. I'll merge this after releasing v1.23.0 tomorrow.
https://ci-chainer.pfidev.jp/job/chainer-sakura/1308/


Mistaken rename of build.sh

Build scripts (shell and cmd) not removed?
https://github.com/tigerbeetle/tigerbeetle/commit/d2f9bb4ec8a84b188bd6da4d31f2702d111a9807

Build scripts (shell and cmd) not removed?

The script this PR is about is about the docs repo, not this repo. :)


feat: update AbiEncode implementations

Description
This PR updates the AbiEncode impl for Bytes and adds new implementations for String and raw_slice.
Checklist

[x] I have done my best to ensure that my PR adheres to the Fuel Labs Code Review Standards.
[x] I have requested a review from the relevant team or maintainers.

@hal3e Left a small comment, the rest seems perfect.


Make doesn't work on OSX and python 3.7

Issue Type

[x] bug report
[ ] feature request

Current Behavior

Cannot build the source code, although I made sure that:

I have leveldb correctly installed (via homebrew) and there's a link in /usr/local/include
I ran xcode-select --install and when I run gcc -v I correctly see /usr/local/include mentioned there

Expected Behavior

make should work.
Steps to Reproduce

This is perhaps a problem specific to OSX.

git clone git@github.com:omisego/plasma-mvp.git
virtualenv env -p python3
source ./env/bin/activate
make

Full output of error:
Searching for plyvel==1.0.4
Reading https://pypi.org/simple/plyvel/
Downloading https://files.pythonhosted.org/packages/e0/7b/9302e33972c4af861f5a425cb37468b4dbb17d7a4d282a657bfdb240055e/plyvel-1.0.4.tar.gz#sha256=134e78519403a6f71f1e66df945763413c48d0f6dec02e1a61f55adf0f83106d
Best match: plyvel 1.0.4
Processing plyvel-1.0.4.tar.gz
Writing /var/folders/_x/nsr657154_s4z371xj0fvg8m0000gq/T/easy_install-kg9t4l73/plyvel-1.0.4/setup.cfg
Running plyvel-1.0.4/setup.py -q bdist_egg --dist-dir /var/folders/_x/nsr657154_s4z371xj0fvg8m0000gq/T/easy_install-kg9t4l73/plyvel-1.0.4/egg-dist-tmp-_e82k3tc
warning: no files found matching 'plyvel/*.pxi'
plyvel/_plyvel.cpp:547:10: fatal error: 'leveldb/db.h' file not found
#include "leveldb/db.h"
         ^~~~~~~~~~~~~~
1 error generated.
error: Setup script exited with error: command 'clang' failed with exit status 1
make: *** [Makefile:2: init] Error 1

System Specs

operating system: osx high sierra 10.13.5
python version: 3.7
virtualenv: 16.0.0
pyetherem version: 2.3.0
environment (output of running pip freeze):

click==6.7
flake8==3.5.0
plasma==0.0.0
py-solc==3.1.0
pytest==3.7.1
rlp==0.6.0

I managed to get it working:
Without virtualenv
I had to reinstall openssl and follow the instructions from here: https://github.com/ethereum/pyethapp/issues/209. Afterwards, the install ran smoothly.
With virtualenv
I switched to pyenv and pyend-virtualenv and, after fiddling with the tools for a while, I managed to run make successfully inside the environment. Of course, tests don't work but I see that there are already PRs for that.
Did this happen just to me? I might update the README to add some warnings for OSX.
Considering the latest developments of Plasma (Cash, Debit et al), is OmiseGo even working on the mvp at all?

Weird. I haven't seen this issue yet. I guess if it's working now I'll close (?).

Yup! I just wanted to make sure it was something with my machine. Thanks!


[Feature Request] API for loading fabric mods for the other mod locators

Below is targeting 1.20.1 Forge
For now. All the logic is private in https://github.com/Sinytra/Connector/blob/1.20.1/src/main/java/org/sinytra/connector/locator/ConnectorLocator.java
My mod is adding a mod locator that will load mods from the other directory. And provide config GUI for users to disable them by directory or mod ID. connector.additionalModLocations is lacking the folder information.
Need an API like ConnectorLocator#addCandidates(Path... paths) for loading the mods with connector locator.
Since connector's is the last one. It's possible to add the path in any normal mod locator
For now, I have to pass the folders to connector.additionalModLocations and copy(or invoking with reflection?) the ConnectorLocator#filterPaths and transform the jars to get the directory and mod id
Or an API that can get original jar path from the mod info?

connector.additionalModLocations is lacking the folder information.

What information is this exactly?

What information is this exactly?

What's the folder the mod from

As our LTS support target, 1.20.1 now only receives critical fixes related to Connector bugs. Please upgrade to the latest version for mod compatibility support.


Trying to get in touch regarding a security issue

Hey there!
I'd like to report a security issue but cannot find contact instructions on your repository.
If not a hassle, might you kindly add a SECURITY.md file with an email, or another contact method? GitHub recommends this best practice to ensure security issues are responsibly disclosed, and it would serve as a simple instruction for security researchers in the future.
Thank you for your consideration, and I look forward to hearing from you!
(cc @huntr-helper)

Hello!
I apologize for the missing security policy. I will get in touch with the other maintainers to set up a contact point for security-related issues. Truthfully, we already have an issue about this (#2642), so I will close this issue as a duplicate.
In the meantime, you can contact me on my private email (see my profile info).


Cleanup listeners on document before unmount

If a Menu or a Dropdown is removed from the DOM while open React throws Invariant Violation: findDOMNode was called on an unmounted component. when the next click on the document is intercepted because the listeners are not removed.
I don’t think Sliders nor ClockHands throw but there’s still a memory leak as the listeners hold a reference to the component.

Yeah, it makes sense! Please fix linter errors and we are ready to merge. I think this solves #303

There’s no linter errors and npm test runs fine on my machine but fails on Travis with Cannot find module 'postcss-normalize-url'. Could this be a cache issue ?

 Don’t know why but Travis seems happy now !

Maybe it was circumstantial :D
Merged!


Add ability to select number of images shown per page

This PR adds the ability for users to set the number of images shown per page in the images' index view. It's based on #8439.
I've increased the default number of images shown from 20 to 30 to have a wider range of choices when selecting the number of entries to show.
It also includes minor design changes on the image listing grid.

@phildexter and Ben reviewed the design.

[x] Do the tests still pass?
[x] Does the code comply with the style guide?

[x] Run make lint from the Wagtail root.

[x] For Python changes: Have you added tests to cover the new/fixed behaviour?
[ ] For front-end changes: Did you test on all of Wagtail’s supported environments? No

[ ] Please list the exact browser and operating system versions you tested: Firefox, Chrome
[ ] Please list which assistive technologies you tested: None

[ ] For new features: Has the documentation been updated accordingly?

Please describe additional details for testing this change.

Merged in ec64933


Web Accessibility Standards

Hello team, is the chat interface (or ChatJS) compliant with specific web accessibility standards (e.g. WCAG 2.0 AA)? I do see aria tags in the code but wanted to check if this is part of the formal feature set? Thanks.

This is not part of the formal feature set. We try our best to be in compliant but some things can be missed.

Hi @sanjusunny, we have accessibility auditing for our internal chat-interface package and plan to work on those improvements in the future. But we don't have an ETA for making this repo compliant yet.
Can you help me understand your use case and what concerns you have?

Thanks, it was less about a specific use case and more about providing AWS
customers a stronger starting point to help them create a more accessible
experience for their customers.
On Thu, May 11, 2023, 2:44 PM haomingli2020 @.***>
wrote:

Hi @sanjusunny https://github.com/sanjusunny, we have accessibility
auditing for our internal chat-interface package and plan to work on those
improvements in the future. But we don't have an ETA for making this repo
compliant yet.
Can you help me understand your use case and what concerns you have?
—
Reply to this email directly, view it on GitHub
https://github.com/amazon-connect/amazon-connect-chat-interface/issues/41#issuecomment-1544505998,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABREEUOZADR57KJJ6WTCZZLXFUXQ3ANCNFSM5K4N5HFQ
.
You are receiving this because you were mentioned.Message ID:
@.***
com>

Hi @sanjusunny, team will provide the details about accessibility in the README file going forward. These changes will be added in PR https://github.com/amazon-connect/amazon-connect-chat-interface/pull/124.
Since there are no further action items we are taking specific to this issue, I am going to resolve.
Thanks,
Spencer


What did you do with accents?

Hi again, and thanks in advance for your response. We are questioning ourselves how you dealt with accents, which are very important in spanish (e.g. hacia is not the same as hacía). When using your tokenizer with transformers library, it seems that both words (and other example of pairs of this type) have the same id in the embedding and therefore have the same vector representation.

We are accessing your model through AutoTokenizer, AutoModel etc; is there any difference compared to calling it through BertTokenizer, BertModel etc?

Maybe it has to do with the version of transformers; which one are you using?

Hi again,
I tried the following code in version 2.4.1
tokenizer = AutoTokenizer.from_pretrained("dccuchile/bert-base-spanish-wwm-cased") tokenizer.tokenize('Hola, Qué pasó aquí compañero?')
And worked as expected.
pd: the following won't work because using AutoTokenizer and AutoModels depends on the name of the model. The model in this case should have 'bert' in it and its not the case with 'dccuchile/cased'.
tokenizer = AutoTokenizer.from_pretrained("dccuchile/cased")
tokenizer.tokenize('Hola, Qué pasó aquí compañero?')

Hey how can i use this model? Do i have to download the cased file and the work exact same as bert works by loading the file i already downloaded?


Correct instructions for setting up SSO

Updated the instruction as the PEM needs to be uploaded, not pasted in.

@jeevansd
Can you review the proposed changes?
When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@jackihann, thanks for your contribution!
We're creating a separate PR for your changes in our private repo for better content validation.
#please-close


Fix ButtonGroup unpressing

It was possible to unpress a button group that had allow_unpress = false using shortcuts.
Adjust BaseButton::shortcut_input to use the same logic as BaseButton::on_action_event.
resolve #78224

Thanks!


Bump to version 0.20.0

FWIW in other libraries I see maintainers deliberately keep version bumps in their own PRs


Curried constructor

Separate from the regular new/* synatx, a new_partial/0 or similar function. Naming could be better 😉 (Some kind of clear metaphor always preferred)
More context: #19

From #19:

As much as I'd like Elixir to be auto-curried, it does impose some overhead that we may not want by default. (...) I'm not against having a separate curried constructor, however thinking Would be trivial via Quark.

defdatax, described in #2, is using such a function, defpartialx. The latter uses up the entire arity-space, just as defpartial does, but the returned anonymous functions refer to the named function clauses. Used this in defdatax (or more precisely, in Algae.Internal.data_astx/2) to allow overriding the new constructors with currying.
I'll try to do a pull request as soon as possible, with a more coherent explanation.


1.9.3 Feature Update

added a world map heatmap
added shapes files for the world map
cleaned up imports from several files
added a counties csv with country codes, regions, etc.
added a function for coverting country <-> codes
improved corr() so that it supports many more variables
corr() automatically adjusts size based on number of features
line() takes all features in dataframe except 'y' if 'x' is left None
swallow the warning from statsmodels for FutureWarning of pandas datetime
added an exception for MissingImport
updated requirements.txt
reorganized dependcy installs in setup.py
added .pyc files to .gitignore
added possibility to input labels to line() >> this needs to be in all the plots
added outlier filtering to world() >> this needs to be in all the plots
fixed issue #15 regarding default x and y limit settings for plots
small cleanups throughout all the plots
added many example plots

Pull Request Test Coverage Report for Build 57

51 of 131 (38.93%) changed or added relevant lines in 19 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-3.9%) to 60.926%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

astetik/plots/grid.py
2
3
66.67%

astetik/plots/hist.py
0
1
0.0%

astetik/utils/outliers.py
0
1
0.0%

astetik/plots/line.py
2
10
20.0%

astetik/utils/country_code.py
6
18
33.33%

astetik/plots/world.py
14
71
19.72%

Totals

Change from base Build 55:
-3.9%

Covered Lines:
658

Relevant Lines:
1080

💛  - Coveralls

Pull Request Test Coverage Report for Build 57

51 of 131 (38.93%) changed or added relevant lines in 19 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-3.9%) to 60.926%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

astetik/plots/grid.py
2
3
66.67%

astetik/plots/hist.py
0
1
0.0%

astetik/utils/outliers.py
0
1
0.0%

astetik/plots/line.py
2
10
20.0%

astetik/utils/country_code.py
6
18
33.33%

astetik/plots/world.py
14
71
19.72%

Totals

Change from base Build 55:
-3.9%

Covered Lines:
658

Relevant Lines:
1080

💛  - Coveralls


The plugin says "'remoteHub/pullRequest' is not a valid menu identifier."

Extension version: v0.38.1
VSCode Version: 1.65.2
OS: macOS 12.2.1

Steps to Reproduce:

Open VSCode -> Select Extension Panel -> Select this Plugin -> Select RuntimeState (may be this name because I am not using an English version)
See this "'remoteHub/pullRequest' is not a valid menu identifier." (translated)

I've tried to restart VSCode, it keeps appearing. Not a real bug I think, because it doesn't affect anything so far. Seems like a harmless stubborn hint.

@alexr00 this screen shot should do it. (the display is in Simplified Chinese)

Thanks! This is expected as an outcome of https://github.com/microsoft/vscode/issues/140463.
@sandy081, would you consider not showing infos in the extension page like this?

Sounds like this is intended behavior then. It is indeed a harmless stubborn hint. Closing as-designed.


Add support for image, gif, audio and files + add a param to hide all memberships change messages

Fixes #39 and #52
Everything is not perfect (there still some problems with videos from tg to matrix, and gifs transfer from tg to matrix could be improved), but it's working

Seems to work fine, I tested most of the functions

Works fine for me too, can we get this merged?


ImplicintGrantAuth - Error 404

In #283 I had some problems with converting the proper Auth Code by @JohnnyCrazy  from C# into VB.NET. But I think I handled it now. This is my Code now:
Private Shared Sub WebConnect()
   Dim auth As SpotifyAPI.Web.Auth.ImplictGrantAuth = New SpotifyAPI.Web.Auth.ImplictGrantAuth(clientID, "http://localhost:4002", "http://localhost:4002")
   AddHandler auth.AuthReceived, AddressOf AuthOnAuthReceived

   auth.Start()
   auth.OpenBrowser()
   Console.ReadLine()
   auth.Stop(0)
End Sub

Private Shared Sub AuthOnAuthReceived(ByVal sender As Object, ByVal payload As SpotifyAPI.Web.Models.Token)
   Dim api = New SpotifyAPI.Web.SpotifyWebAPI With {
       .TokenType = payload.TokenType,
       .AccessToken = payload.AccessToken
   }
   Dim profile As SpotifyAPI.Web.Models.PrivateProfile = api.GetPrivateProfile
   Console.WriteLine(profile.DisplayName)
End Sub

Unfortunately, I still have problems with this Auth. I added "http://localhost:4002" to the Redirect URIs in the Spotify Developer Center already. After starting WebConnect(), my browser says "Connection Failed". When I change the parameter from auth.Stop(0) to authStop() or authStop(1000), the opened site just tells me "404 - Not Found".
Whats wrong with my code?
EDIT: The AuthOnAuthReceived-Event doesn't get triggered. Why?

It looks like perhaps the callback has changed on the Spotify side?  Looks like the codebase is expecting the /auth endpoint, and what is getting sent from the Spotify side is the root endpoint with the parameters directly (i.e. http://localhost:4002/#?<params>)

Ist the server starting correctly? Any output in the debug console? Can you teach localhost:4002?
The redirect URI is without /auth, /auth is just used to get the parameters to the server

Yes the server appears to start correctly.  Here is the output:
 20:10:43.659 INF >> [WebServer] Web server prefix 'http://localhost:4002/' added.
Hello
 20:10:43.683 INF >> [WebServer] Finished Loading Web Server.
 20:10:43.710 INF >> [WebServer] Started HTTP Listener
 20:10:44.462 DBG >> [WebServer] Start of Request 351fa863 - Source ::1:57942 - GET: / - Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/69.0.3497.100 Safari/537.36
 20:10:44.467 DBG >> [WebServer] Web API Module::WebApiModule.<.ctor>b__2_0
 20:10:44.497 TRC >> [WebServer] Result: False
 20:10:44.514 DBG >> [WebServer] Resource Files Module::ResourceFilesModule.<.ctor>b__2_1
 20:10:44.517 DBG >> Resource System: index.html
 20:10:44.517 TRC >> [WebServer] Result: False
 20:10:44.517 ERR >> No module generated a response. Sending 404 - Not Found
 20:10:44.521 DBG >> [WebServer] End of Request 351fa863

The requested URL that generated this 404 was http://localhost:4002/#access_token=<token>&token_type=Bearer&expires_in=3600&state=771f02b2
Note that the AuthorizationCodeAuth works given similar parameters.

.net version and spotifyAPI nuget version?

.NET Core 2.0 application on Windows.  spotifyAPI version is 3.1.0 (both regular and auth).

@borrrden Thank you borrrden for answering for me.
Does somebody has a temporirily workaround for the auth before the problem is fixed?

@JohnnyCrazy What can I do now?

Was this ever solved?

Well, does it work for you? @OneBobone

I get this error: " Type 'SpotifyAPI.Web.Auth.ImplictGrantAuth' is not defined."
I am really looking for some (any) very simple code to get me started in vb.net

authorization
check a track

Soon I need more advanced authorization to download a playlist, which is ultimately what I need.

Was this ever solved?

I somehow figuered out to authorize to the API. But I didn't worked on this project for months and I am not sure if the solution is still working. As soon I am back in my city I'll send you the code for you trying it out.

Just a friendly reminder that I would appreciate this very much.

@xTheChosen0ne I think I got your code to work.
:-)
Would be very valuable to compare the rest. Thanks!

But getting endless "waiting for host"

Thank you! That worked.
If I may, I would appreciate the discussion in a non closed forum:
https://github.com/JohnnyCrazy/SpotifyAPI-NET/issues/333


Application Insights

It looks like Application Insights may be enable in this extension. Is there a way of disabling it?

If using yaml, set the allowTelemetryCollection input to false. If using classic editor, uncheck Allow Telemetry Collection.


Styling Instructions Table

The instructions table of the possible moves needs to be styled. Anyone with good designing experience can work on this.

@vasu-gondaliya  Can I work on it?

Yes sure @96RadhikaJadhav ! Also an issue has been made for adding 2 instructions. So if possible you can work on that too. Check out issue #7.

Yes sure @96RadhikaJadhav ! Also an issue has been made for adding 2 instructions. So if possible you can work on that too. Check out issue #7.

@vasu-gondaliya Sure, I will try it.

The instructions are already been added. So work accordingly. @96RadhikaJadhav

I want to work on it.. Can I?

I want to work on it, can I?

I want to work on it. Actually I have already added improvisations to the 2 tables.I have attached the screenshot of the current look which I've modified to.

If you permit can I give my contribution too?


SEC-53 don't backto to external urls

Critical security fix, but double-check my logic. See: https://viam.atlassian.net/browse/SEC-53
Would love help testing this.

put up this app PR with temp env so we can test: https://github.com/viamrobotics/app/pull/6610

@ohEmily  @jr22 I tested on the temp env on the app PR and I think it is working. Take a look and feel free to approve/merge this PR if we are good.

tested and this does fix the exact problem described. not sure if it fully closes the security hole (not saying that because i can think of something but because i'm honestly not sure) but feels good enough for now. alternatively, i couldnt get any of the backto logic to work correctly (on this or on prod) so possible we could remove this all together

It's possible the frontend is no longer sending the backto param to /login. I can check that, but it will be a PR in app so not blocking for this PR. Going to merge.


Look for faster alternatives to DataFrame.drop

What?
For e.g. LogTransformer, dropping the original column is one of the slower operations. Would be good to look into faster alternatives, e.g. df.columns.isin(wanted_columns)
Why?
Speed up tubular
How?
Explore alternatives to pd.DataFrame.drop, profile candidates to confirm efficiency benefits

That's a nice idea. The suggested alternative df.columns.isin()  is the Boolean indexing approach and is efficient compared to df.drop() which involves removing unwanted columns iteratively.
Another solution I can think of is a df.filter() used to subset the df according to the conditions in the labels. It's efficient and directly creates a new df with only desired columns without having to rearrange or drop data.

I've created a quick notebook to profile df.drop() as well as df.columns.isin() and df.filter() : https://github.com/lvgig/tubular/blob/196-look-for-faster-alternatives-to-dataframedrop/examples/profile_pd.fillna_alternatives.ipynb
It does seem that df.filter() is a bit quicker than the other two approaches
Let me know what you guys think!

Hi RJ, nice work. I just wanted to let you know that I tried a new method of dropping features using the del statement and its much faster than the other methods. I have added a new block to your notebook, so please take a look at it.
Results from latest run:

https://github.com/lvgig/tubular/blob/196-look-for-faster-alternatives-to-dataframedrop/examples/profile_pd.fillna_alternatives.ipynb

Hi Jagadish, amazing work! As you mentioned, the only concern with del statement is its in-place modification. I guess this could be an issue if we still need the original dataframe after dropping the required columns (however a workarounds for this is to perhaps store the original dataframe in a temporary dataframe before dropping the columns). However, I had a quick look of where .drop() is being used in tubular using the good old search tab. It does look like whenever .drop is used in tubular, the resultant dataframe is directly being returned. So maybe there's no issue with using del for tubular at all!
If we all agree on this approach of using del, let me know and I'll replace all .drop() occurrences in tubular with del, check if all unit tests are passing and create a PR.

Nice work both.
Just thinking about whether del could cause problems, possibly if copy was set to False it would update the original df in a way that we didn't want - might be worth writing a test for this scenario? If it fails we can have a think about what to do next!

Sorry RJ didn't see your comment! I think yeah have a go, just also add some tests around whether it causes issues for copy=False, and any other potential problems you can think of :)

Do note that we're currently looking to remove the copy flag altogether as part of #181 - copy on write will handle how pandas objects are dealt with.
As for updating by reference, users can (and should) create a view:
>>> import pandas as pd

>>> df = pd.DataFrame({"student_id": [1, 2, 3], "grade": ["A", "C", "D"]})
>>> del df['grade']
>>> df
   student_id
0           1
1           2
2           3

>>> df = pd.DataFrame({"student_id": [1, 2, 3], "grade": ["A", "C", "D"]})
>>> df2 = df.iloc[:] # A view of df - no copy is made

>>> del df['grade']
>>> df2
   student_id grade
0           1     A
1           2     C
2           3     D
>>> df
   student_id
0           1
1           2
2           3

Pandas knows that the deletion shouldn't propagate to the view - see docs on newer versions of pandas

Previously, when operating on views, the view and the parent object was modified

Given Adam's comment, maybe ignore my comment around testing for copy=False and just make clear in the ChangeLog entry that this release should also remove copy before publishing!


Where is Manage payment methods?

A canot find Manage payment methods?

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

ID: 4fb8bd80-8dc3-94be-dc1e-7cd6428da63c
Version Independent ID: 523e8db2-6428-6a51-b536-c3d531560aee
Content: Change your credit card for Azure
Content Source: articles/billing/billing-how-to-change-credit-card.md
Service: billing
GitHub Login: @genlin
Microsoft Alias: genli

@jankod, Thanks for bringing this to our attention. Your feedback has been shared with the content owner for further review.

@genlin, can you please review this and provide guidance?

Hi,
thank you for your reply.
I cant see: "Manage payment methods".
I see only this:
--
Janko Diminic
On 16 April 2018 at 11:19, Thomas Lin<EMAIL_ADDRESS>wrote:

@jankod https://github.com/jankod ，Please log into the
https://account.windowsazure.com/Subscriptions by using account
administrator. The "Manage payment methods" is in the lower right corner of
the page. Do you see it? Let me know if there is a problem.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/azure-docs/issues/7177#issuecomment-381534540,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABkdVmogDCUra5NYSowyECdF-R5EF8plks5tpGIOgaJpZM4TUNV2
.

@ jankod, What is the type of your subscription? Is it pay-as-you-go subscription?

No, my subscription was bean free/trial, something like that.
Now I dont have any subscriptions because i was delete all subscription.
Now I see in "Payment methods" tab my VISA card and I canot remove that,
so this is problem for me.
Tnx
--
Janko Diminic
On 17 April 2018 at 11:27, Thomas Lin<EMAIL_ADDRESS>wrote:

@ jankod, What is the type of your subscription? Is it pay-as-you-go
subscription?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/azure-docs/issues/7177#issuecomment-381919118,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABkdVrlClHP08hB2v5-loRbyZ4gMglnrks5tpbWWgaJpZM4TUNV2
.

@jankod For this problem, please contact our support team https://portal.azure.com/?#blade/Microsoft_Azure_Support/HelpAndSupportBlade

We will now proceed to close this thread. If there are further questions regarding this matter, please reopen it and we will gladly continue the discussion.
#Please-close

@jankod
We will now close this issue. If there are further questions regarding this matter, please reopen it and we will gladly continue the discussion.


Characters are visible instead of .png

Characters are visible instead of .png(../media/455b84df-defb-4da6-92f7-0ca21cc8d930.png)
https://docs.microsoft.com/ja-jp/yammer/troubleshoot-problems/limited-functionality-in-explorer-10-document-mode

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

ID: 772c688b-607c-8812-a8d8-13026750156b
Version Independent ID: 0a85eaa3-bd33-2deb-ccb3-cba9934e5aee
Content: Limited functionality in Yammer in Internet Explorer 10 document mode
Content Source: yammer/troubleshoot-problems/limited-functionality-in-explorer-10-document-mode.md
Service: yammer
GitHub Login: @IrenePasternack
Microsoft Alias: v-irpast

@samanthaw

@MollyMalone

@hyoshioka0128 the image is now displaying ok in the Japanese topic.  Thanks again for reporting it.

@AndreaBarr - I believe this issue can be closed. Thank you.


Does php client support add listen event?

Does the client support add listen event like javascript?
var wsUri = "ws://localhost:8080";
websocket = new WebSocket(wsUri);
websocket.onopen...
websocket.onmessage...

If support, then we can use php client as convenient as js websocket client.

Yes, the PHP client support events:
$socket->on("connect", function(\HemiFrame\Lib\WebSocket\Client $client) {
});

$socket->on("receive", function(\HemiFrame\Lib\WebSocket\Client $client, $data) {
});

$socket->on("send", function(\HemiFrame\Lib\WebSocket\Client $client, $data) {
});

$socket->on("ping", function(\HemiFrame\Lib\WebSocket\Client $client, $data) {
});

$socket->on("pong", function(\HemiFrame\Lib\WebSocket\Client $client, $data) {
});

$socket->on("disconnect", function(\HemiFrame\Lib\WebSocket\Client $client, $statusCode, $reason) {
});

$socket->on("error", function($socket, $client, $phpError, $errorMessage, $errorCode) {
});

I saw these code in demo client.php file and tried many times, it doesn't work. I didn't receive any message from server, but the brower client can receive the messages all the time, and I found the server indeed send messages to all clients, when I run the client.php, the server send two messages, one for the brower client, and one for the client.php script, but there is nothing output in the console.
client.php
require_once './vendor/autoload.php';
set_time_limit(0);
$socket = new \HemiFrame\Lib\WebSocket\WebSocket('localhost', '7001');
$socket->on('receive', function (\HemiFrame\Lib\WebSocket\Client $client, $data) {
    echo $data."\n";
});
$client = $socket->connect();
$data = [
    'demo' => 'something'
];
//I don't know how to hold the script,so I use the loop + sleep statement
while (true) {
    $socket->sendData($client, json_encode($data));
    sleep(10);
}

@heminei Can you help me check and test the problem above?


Error: The module was compiled against a different version of node.js

Hello,
I am attempting to use this module in an Electron application but am getting the following error:
Uncaught Error: The module '(removed)\Launcher\node_modules\win-version-info\build\Release\VersionInfo.node'
was compiled against a different Node.js version using
NODE_MODULE_VERSION 64. This version of Node.js requires
NODE_MODULE_VERSION 70. Please try re-compiling or re-installing
the module (for instance, using npm rebuild or npm install).
I have searched for and attempted a few methods to rebuild/recompile the package but am having no luck. I'm not terribly well versed in package management so this is probably a user issue more than project issue.  I would appreciate any help you might have.
Thanks.
Node Version: 10.16.0
Windows 10

See https://github.com/electron/electron-rebuild


Feature/transaction export

This branch includes implementations to support transaction extensions to be able to use the export goal.

Please merge this branch first to the master branch in your repository and then create a pull request from there to this repository. Thanks!


Merge master-qa

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: LucasBrazi06:x: mariansalcedaYou have signed the CLA already but the status is still pending? Let us recheck it.


Added translation to spanish of sections 3.4.3, 3.4.4 and 3.5

The sections were translated with proper links and formatting.

Awesome work @nelruk, thank you for all the contributions! :1st_place_medal:
Is the translation spreadsheet updated?

Yes, it is @medied


DreamFusion Visual Gap and Error on "cannot load weights that are saved in int8"

Hi,
Thanks for the impressive work you have done on your repository. I did encounter an issue when attempting to change the configuration settings to use the IF setting. The error message stated that the weights I was trying to load were saved in int8 format and could not be loaded with load_in_8bit=True when using float32/float16/bfloat16 weights.

ValueError: ('You cannot load weights that are saved in int8 using load_in_8bit=True, make sure you are', ' using load_in_8bit=True on float32/float16/bfloat16 weights.')

I was able to resolve this issue by changing load_in_8bit to False and setting the variant to 'fp16', which should provide me with better guidance (ps: correct me if I am wrong :)); And I then ran the following configuration:
python launch.py --config configs/dreamfusion-if.yaml --train --gpu 0 system.prompt_processor.prompt="a tiger dressed as a doctor" system.background.random_aug=true

However, I noticed a visual discrepancy between my reproduced results and the results on your repository when using the prompt "a_tiger_dressed_as_a_doctor" from the Dreamfusion paper.
Here are my reproduced results:

And here are the results on your repository:

I was wondering if you could kindly let me know if there are any specific settings that I should follow in order to reproduce the following results from your repository. Any insights or suggestions would be greatly appreciated.

No response in 3 weeks... I hope no one minds if I close this issue.


Links navigation#30

Added front and back movement with the controller by traversing the panorama links

With the starting location set to the Fourth Ave Building, we proceed forward to the corner of 4th and Lincoln st, and then  movement begins to get buggy.
Attempting to maintain forward movement, the player bounces backward, then forward, the backwards, etc..
Seems like the links the we are using keep pointing back to each other.
Additionally, each movement, resets the camera position to a default position, when it should maintain the previous camera position.
This is even more important now that we have free look implemented. We'd like to maintain the players head position, as we navigate.