Limit issue body to 500 characters

Relevant Issues
Closes #486
Description
This PR puts a limit on how long the response sent by the .hacktoberissues is going to be by limiting the issue body up to it's first 500 characters. This avoids http 400 errors (see #486 ) when dealing with issues with very long body.
Did you:

[x] Join the Python Discord Community?
[x] If dependencies have been added or updated, run pipenv lock?
[x] Lint your code (pipenv run lint)?
[x] Set the PR to allow edits from contributors?

@kwzrd 500 is just a short number which I think is decent amount characters for embed descriptions. I agree this is opinionated, (large embeds just don't look good to me/I feel it's bad UX). So being pointed out, I'll limit it back to 2048 characters if that's agreeable.

It's not necessarily a bad choice, if you think that 500 looks and works better. I was just wondering about why it was chosen. Maybe you could show a screenshot of 500 vs 2048?

@kwzrd Here are the screenshots you requested-
https://ibb.co/M21rqX2
https://ibb.co/gDM642P
The one with 2048 characters totally covers my laptop screen.

@quanta-kt Thanks for the examples, I agree that 500 is a more sane value, especially since you can just click the link to read more.

@Den4200 @Akarys42 Are you sure that textwrap.shorten is the better choice here? The problem with it is that it removes newlines. Consider this issue, which gets formatted like so:

If we just slice the string, we get to maintain the newlines, e.g. this issue:

Ahh right, I forgot that textwrap.shorten removes newlines. Good catch,
sorry @quanta-kt.
On Tue, Oct 6, 2020 at 3:08 PM kwzrd<EMAIL_ADDRESS>wrote:

@quanta-kt https://github.com/quanta-kt Thanks for the examples, I
agree that 500 is a more sane value, especially since you can just click
the link to read more.
@Den4200 https://github.com/Den4200 @Akarys42
https://github.com/Akarys42 Are you sure that textwrap.shorten is the
better choice here? The problem with it is that it removes newlines.
Consider this issue
https://github.com/vJechsmayr/PythonAlgorithms/issues/382, which gets
formatted like so:
[image: image]
https://user-images.githubusercontent.com/44734341/95248481-a3c81180-0817-11eb-93af-db65a0e07172.png
If we just slice the string, we get to maintain the newlines, e.g. this
issue https://github.com/BitByte-TPC/beginners-byte/issues/3:
[image: image]
https://user-images.githubusercontent.com/44734341/95248661-e853ad00-0817-11eb-8d51-31a5426ba2dc.png
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/python-discord/seasonalbot/pull/488#issuecomment-704493688,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AMRF7T4OXNYECADS6BOKT63SJNTLLANCNFSM4SGFPGEQ
.

TextWrapper apparently has a replace_whitespace keyword-arg but shorten simply ignores it, weird.

Yes I am familiar with rebasing but still a little hesitant to do it, can you check this dummy PR here and confirm if that's what I'm supposed to be doing?
Commands used-
git rebase -i issue-486~5 issue-486 # And picking only the first commit
git push origin +issue-486 # A forced push

Also, shouldn't it get squashed into a single commit automatically if one chooses to "Squash and merge" on GitHub?

Your dummy PR looks good, but you would want to keep the 2nd commit as well (reducing the dots from 4 to 3).
I think there is an easier way, we actually don't need to rebase because we only need to drop the last 3 commits:
git reset --hard HEAD~3

You can use git log to verify that only commits 5f02c61 and 8645da6 remain. Then git status should say that your local branch is 3 commits behind the remote. Finally git push --force will remove the commits from the remote as well.

Also, shouldn't it get squashed into a single commit automatically if one chooses to "Squash and merge" on GitHub?

Yeah, we had a chat internally and we wanted to give you a chance to do it as it's both a good learning opportunity and also makes sure that the actual commits that get merged are authored by you, rather than the person who squashed them.

Lovely, thank you!


Add Kaiser-Ruby to implementation list

This is an implementation of a Rockstar to Ruby transpiler, which is obviously named Kaiser-Ruby, because what else could I name a thing like that.
It's currently a work in progress (at the moment of making this PR), however I'm likely to finish it before Dylan finishes moving and it all gets merged, so I think that's acceptable ;) There aren't many things left for me to implement anyway.
Also what is worth mentioning, due to Ruby being really Unicode friendly, it handles Metal Umlauts #83 both in values and in variable names, because why not.

Admittedly I'm not personally a huge fan of Ruby, but I have to admire the name here. Took me seeing "kaizer-ruby" to realise what you were referencing :+1:

As a cultural idiot, could someone explain?

This is a really awesome addition to the soirée of implementations!

@cwfitzgerald Kaiser Chiefs are a rock band from Leeds, UK, most famous for their song Ruby

Exactly that. It's pretty common for Ruby developers to give fancy names to their gems (we have Rails, but also Paperclip, Sinatra, Jenkins and so on, for a few examples), so I've figured it can't hurt to do the same :)

In a similar vein, we have maiden named after iron maiden, because it's written in rust, and thrash, as in thrash metal, because it's written in Go and designed to be fast

God I am bad at naming things....

I'm not sure if editing the original PR makes a notification (I suspect not), so this comment is just a notice that the implementation is finally working and can run FizzBuzz.
$ kaiser-ruby execute ./examples/fizzbuzz.rock
1
2
Fizz
4
Buzz
Fizz
7
8
Fizz
Buzz
11
Fizz
13
14
FizzBuzz
16
17
Fizz
19
Buzz
Fizz
22
23
Fizz
Buzz
26
Fizz
28
29
FizzBuzz

$


document read and write path

Would be great to have doc folder so that as a developer i can see how te system works architecturally and its features architecturally.
Tasks:
In a doc folder, explain the Read and Write flow and interception points such as Security, Tracing and Logging).
It should use a common naming convention. A glossary might be needed.
It should delineate if else logic in how each part of the flow works that is pertinent to the outcome.
It can have links to code if its pertinent later.
It should be written for architectural analysis and not as a how to use perspective.
DOD ( Definition of done):

Ability to understand the conceptual architecture
Be enough to conceptualise a refactoring of the architecture in whatever way seems fit going forward.

Currently I will create a new /docs folder containing:

res-protocol.md
Overview of the protocol
Description of the server stack and how they connect (client, resgate, NATS, microservices)
Description of core concepts (resources, subscription, query resources, etc.). This may also serve the purpose of glossary
Request and event flows (including path for read and write) as described in the issue.
res-service-protocol.md
Specification of the protocol used between (micro)service & resgate
Mostly renaming of the existing resprotocol.md file
res-client-protocol.md
Specification on the protocol used between client & resgate

To describe the flows in the res-protocol.md, sequence diagrams might be used for visual aid. For the conditional logic, one can use "alternatives": https://stackoverflow.com/questions/8114770/how-to-show-if-condition-on-a-sequence-diagram

Solved in https://github.com/jirenius/resgate/pull/28
Gateway flow sequence diagrams not linked to from README, but available under /docs folder.


small code change causes 70% build time regression (20s->34s) [strictGenericChecks]

TypeScript Version:  2.6.0-dev.20170921
Code
Sorry it's a bit bigger codebase, but it would be very hard I guess to reproduce a performance issue with little code.
If you pull https://github.com/emmanueltouzery/prelude.ts and run npm install and then tsc.

On the v0.2.1 tag the build time is ~20 seconds for me.
On the v0.2.2 tag the build time is ~34 seconds for me.

The difference between both tags is very small!
https://github.com/emmanueltouzery/prelude.ts/compare/v0.2.1...v0.2.2 6 files changed, 8 insertions(+), 5 deletions(-) (mostly I export an interface which was not marked as exported)
And also if I build with strictGenericChecks: false, the build time collapses from 34 seconds to 5 seconds even for v0.2.2.
Expected behavior:
A bit slower build with strictGenericChecks enabled, same build time for 0.2.1 and 0.2.2.
Actual behavior:
Almost 7 times slower build time with strictGenericChecks, 70% build time increase between 0.2.1 and 0.2.2.

I think the performance is even worse in the equals_experiments branch of the same repo (currently commit c09a286e90d06db0beb7ce4922ed30d479a55ae2). it's slightly worse in runtime than master on my laptop, but on my older desktop I killed it after 3 minutes.

sorry, correcting my previous comment, I tested the wrong code on my laptop: the equals_experiment branch is building for minutes at a time on both my laptop and home computer. Killed it after 3 minutes 20 on my fast laptop.

After digging 0.2.1 and 0.2.2 I find that the culprit is a change from Seq to Vector, not the difference in exports. It's almost certain that the change causes us to compare some Seqand Vector structurally somewhere now, where before Seqand Seq could be compared just by comparing T and U. I'll investigate further to track down the exact cause.

The problem is indeed Vector<T> ==> Seq<U> assignments; when I remove the <Seq<number>> cast on line 63 of tests/Option.ts, compile time goes back down to the same as 0.2.1. And if I add more casts by cloning that test with different types, each one adds about 13 seconds to the compile time.
I'll look to see if we can improve the time it takes to relate Vector to Seq.

I can also improve compile time down to 4.2 seconds (on my machine) by removing assignments to Seq. See https://github.com/sandersn/prelude.ts/commit/cf627ceb5585bf4ca4e872353a532a09db034e9c in the ts-perf-boost branch for what I did.

Thanks for the tip, I understand how you sped it up, but in the end Seq is useful to me, to make sure I have about the same methods in all the 'sequence-like' classes (although it's not really useful to users of the library as type signatures are less precise on Seq, so I don't export it from the library in the index.ts).
So I would keep for instance Vector implementing Seq, otherwise I might as well remove Seq completely. The build time is not a blocker for me (it's OK since 2.5.3 now), but I thought you might want to know about the regression and have a good test to reproduce this slowness which could crop up in other applications elsewhere (which might be closed source) I guess.

I'm checking right now to see why our caching fails on Seq relatability, so I hope I will find a real solution instead of an unsafe workaround.

FYI this is still an issue with typescript 2.9.1. My prelude.ts library has less than 7000 LOC of typescript, tess than 6000 without comments, and needs 45 seconds to build :( [on a slow computer admittedly but still].
I think it's because I use lots of intersection types (eg T & WithEquality).


TwitchIO 2.x feature implementation: fetch_user doesn't currently work - would also like fetch_user to get a user that can be whispered

User is independant of IRC (I.E. it's API only), so this isnt possible


Mac M1: Franz high resources and battery usage

Hi,
I have noticed that Franz is responsible for a significant amount of battery consumption on my Macbook M1 Pro machine. I checked the Activity Monitor and noticed that the Kind of application is Intel. Would it be possible to develop a native Apple ARM version of the App for MacOS to improve battery usage?
Best regards,

+1

If I understand correctly, Beta 5.9.0 adds silicon support. Can't download it somehow yet. :(


Fix annoying visual bug with settings

I noticed the same bug than @choroba : https://github.com/kyzn/PRC/issues/31
I have no idea what I'm doing (what is detach() ?) but it seems to fix the bug :grinning:
Edit : Tested locally

Hi! Thanks a lot for your time 🙂 This looks good to me, but let me merge & get back to you next week.

It is just a small update :grinning: thank you for this website, it is very cool !

I left a comment in #31 and tried to tag you there as well. I couldn't confirm the bug.. Can you possibly share your browser information, and step by step repro instructions?

So I've tried to use GitHub UI to leave some feedback, but I don't think it came out as I expected. In any case, this fix looks good to me! I just wanted to point to two things:

There are 6 forms loaded in /settings. 2 of them already had reload, and your patch adds reload to other 2. I'd like all of them do have reload so this issue doesn't come back.
In those 4 forms that don't reload, we were passing success message to stash. With the reload added, this will cause message to not show up. We should switch those to use session instead.

I truly appreciate you spending your time on fixing this issue! 🙂

I updated the PR with a new commit taking account of your feedback.
I tested locally and checked that alerts are well printed.
Do I missed something ?

I tested locally and checked that alerts are well printed.

Huh, that's weird. When I was locally testing, alerts were not showing up. I wonder if it was a browser thing (I was on chrome).
With your latest commit it's working for me too. I think this is great now! 🙂 I'll be merging in a moment. Thanks so much for your time & help!

Great thank you 😃

Thank you! I've also added your GitHub name to README.md/contributors. If you want me to list a different name, or take it off the list feel free to let me know :)

Thank you 😃
It is not that important but I prefer Thibault Duponchelle instead :)

Done! Thanks again.


DDEV restart raise an error with Symfony migrations.

Is there an existing issue for this?

[X] I have searched the existing issues

Run a Diagnostic and Paste Link Here
flovntp@VNTP-Book-3 sfcon-bigfoot1-workshop % ddev debug test
Running bash [-c /var/folders/8l/2wnkl8mj56j7gn39n0ts32080000gn/T/test_ddev.sh]
OS Information: Darwin VNTP-Book-3.local 21.6.0 Darwin Kernel Version 21.6.0: Wed Aug 10 14:28:25 PDT 2022; root:xnu-8020.141.5~2/RELEASE_ARM64_T8110 arm64
ProductName:	macOS
ProductVersion:	12.5.1
BuildVersion:	21G83
User information: uid=501(flovntp) gid=20(staff) groups=20(staff),12(everyone),61(localaccounts),79(_appserverusr),80(admin),81(_appserveradm),98(_lpadmin),33(_appstore),100(_lpoperator),204(_developer),250(_analyticsusers),395(com.apple.access_ftp),398(com.apple.access_screensharing),399(com.apple.access_ssh),400(com.apple.access_remote_ae),701(com.apple.sharepoint.group.1)
DDEV version:  ITEM             VALUE
 DDEV version     v1.21.3
 architecture     arm64
 db               drud/ddev-dbserver-mariadb-10.4:v1.21.3
 dba              phpmyadmin:5
 ddev-ssh-agent   drud/ddev-ssh-agent:v1.21.3
 docker           20.10.17
 docker-compose   v2.10.2
 docker-platform  docker-desktop
 mutagen          0.16.0
 os               darwin
 router           drud/ddev-router:v1.21.3
 web              drud/ddev-webserver:v1.21.3
PROXY settings: HTTP_PROXY='' HTTPS_PROXY='' http_proxy='' NO_PROXY=''
======= DDEV global info =========
Global configuration:
instrumentation-opt-in=true
omit-containers=[]
mutagen-enabled=false
nfs-mount-enabled=false
router-bind-all-interfaces=false
internet-detection-timeout-ms=3000
disable-http2=false
use-letsencrypt=false
letsencrypt-email=
table-style=default
simple-formatting=false
auto-restart-containers=false
use-hardened-images=false
fail-on-hook-fail=false
required-docker-compose-version=
use-docker-compose-from-path=false
no-bind-mounts=false
project-tld=

======= DOCKER info =========
docker location: lrwxr-xr-x  1 root  wheel  54 13 fév  2021 /usr/local/bin/docker -> /Applications/Docker.app/Contents/Resources/bin/docker
Docker Desktop Version: Docker Desktop for Mac 4.11.1 build 84025
docker version:
Client:
 Cloud integration: v1.0.28
 Version:           20.10.17
 API version:       1.41
 Go version:        go1.17.11
 Git commit:        100c701
 Built:             Mon Jun  6 23:04:45 2022
 OS/Arch:           darwin/arm64
 Context:           default
 Experimental:      true

Server: Docker Desktop 4.11.1 (84025)
 Engine:
  Version:          20.10.17
  API version:      1.41 (minimum version 1.12)
  Go version:       go1.17.11
  Git commit:       a89b842
  Built:            Mon Jun  6 23:01:01 2022
  OS/Arch:          linux/arm64
  Experimental:     false
 containerd:
  Version:          1.6.6
  GitCommit:        10c12954828e7c7c9b6e0ea9b0c02b01407d3ae1
 runc:
  Version:          1.1.2
  GitCommit:        v1.1.2-0-ga916309
 docker-init:
  Version:          0.19.0
  GitCommit:        de40ad0
DOCKER_DEFAULT_PLATFORM=notset
======= Mutagen Info =========
======= Docker Info =========
Docker platform: docker-desktop
Using docker context: default (unix:///var/run/docker.sock)
docker-compose: v2.10.2
Using DOCKER_HOST=unix:///var/run/docker.sock
Docker version: 20.10.17
Able to run simple container that mounts a volume.
Able to use internet inside container.
Docker disk space:
Filesystem                Size      Used Available Use% Mounted on
overlay                 132.9G     38.8G     87.3G  31% /

unable to create project at project root '/Users/flovntp/Sites/Customers/PlatformSH/sfcon-bigfoot1-staging-faker': project root /Users/flovntp/Sites/Customers/PlatformSH/sfcon-bigfoot1-staging-faker does not exist
unable to create project at project root '/Users/flovntp/Sites/Customers/PlatformSH/sfcon-bigfoot3-workshop': project root /Users/flovntp/Sites/Customers/PlatformSH/sfcon-bigfoot3-workshop does not exist
unable to create project at project root '/Users/flovntp/Sites/Customers/PlatformSH/sfcon-bigfoot4-workshop': project root /Users/flovntp/Sites/Customers/PlatformSH/sfcon-bigfoot4-workshop does not exist
unable to create project at project root '/Users/flovntp/Sites/Customers/PlatformSH/test-add-postgresql-to-ddev': project root /Users/flovntp/Sites/Customers/PlatformSH/test-add-postgresql-to-ddev does not exist
Container ddev-sfcon-bigfoot1-workshop-dba  Removed
Container ddev-sfcon-bigfoot1-workshop-db  Removed
Container ddev-sfcon-bigfoot1-workshop-web  Removed
Network ddev-sfcon-bigfoot1-workshop_default  Removed
Project sfcon-bigfoot1-workshop has been stopped.
The ddev-ssh-agent container has been removed. When you start it again you will have to use 'ddev auth ssh' to provide key authentication again.
Existing docker containers:
CONTAINER ID   IMAGE                                       COMMAND                  CREATED       STATUS                     PORTS     NAMES
4eb1c8feb993   drud/ddev-webserver:v1.21.3                 "sh -c '( test -f /v…"   4 hours ago   Created                              admiring_perlman
90e96451b2e4   drud/ddev-webserver:v1.21.3                 "sh -c '( test -f /v…"   4 hours ago   Created                              nifty_moser
26eec3cdcf28   drud/ddev-webserver:v1.21.3                 "sh -c '( test -f /v…"   4 hours ago   Created                              youthful_goldberg
caffd447b385   schickling/mailcatcher                      "sh -c 'mailcatcher …"   4 weeks ago   Exited (0) 3 weeks ago               bigfoot_mailer_1
ee531a9dbdd5   postgres:14-alpine                          "docker-entrypoint.s…"   4 weeks ago   Exited (0) 3 weeks ago               bigfoot_database_1
424587753f09   blackfire/blackfire:2                       "/usr/local/bin/entr…"   4 weeks ago   Exited (1) 4 weeks ago               bigfoot_blackfire_1
f871bdb474dc   docker.registry.platform.sh/postgresql-13   "/lando-entrypoint.s…"   6 weeks ago   Exited (137) 5 weeks ago             symfonytemplate61_database_1
6e0e7c8ee13d   docker.registry.platform.sh/php-8.1         "/lando-entrypoint.s…"   6 weeks ago   Exited (137) 5 weeks ago             symfonytemplate61_app_1
b1b8015bedc9   traefik:2.2.0                               "/lando-entrypoint.s…"   6 weeks ago   Exited (137) 5 weeks ago             landoproxyhyperion5000gandalfedition_proxy_1
Creating a new ddev project config in the current directory (/Users/flovntp/Sites/Customers/PlatformSH/tryddevproject-8926)
Once completed, your configuration will be written to /Users/flovntp/Sites/Customers/PlatformSH/tryddevproject-8926/.ddev/config.yaml

Configuring unrecognized codebase as project type 'php' at /Users/flovntp/Sites/Customers/PlatformSH/tryddevproject-8926/web
Configuration complete. You may now run 'ddev start'.
Network ddev_default created
Starting tryddevproject-8926...
Container ddev-ssh-agent  Started
ssh-agent container is running: If you want to add authentication to the ssh-agent container, run 'ddev auth ssh' to enable your keys.
Pushed mkcert rootca certs to ddev-global-cache/mkcert
Network ddev-tryddevproject-8926_default  Created
Container ddev-tryddevproject-8926-db  Started
Container ddev-tryddevproject-8926-dba  Started
Container ddev-tryddevproject-8926-web  Started
Container ddev-router  Started
Successfully started tryddevproject-8926
Project can be reached at https://tryddevproject-8926.ddev.site https://<IP_ADDRESS>:64205
======== Curl of site from inside container:
HTTP/1.1 200 OK
Server: nginx
Date: Mon, 28 Nov 2022 13:59:33 GMT
Content-Type: text/html; charset=UTF-8
Connection: keep-alive
Vary: Accept-Encoding

======== Curl of site from outside:
HTTP/1.1 200 OK
Server: nginx/1.20.1
Date: Mon, 28 Nov 2022 13:59:33 GMT
Content-Type: text/html; charset=UTF-8
Connection: keep-alive
Vary: Accept-Encoding

======== Project ownership on host:
drwxr-xr-x  4 flovntp  staff  128 28 nov 14:59 ../tryddevproject-8926
======== Project ownership in container:
drwxr-xr-x 4 flovntp dialout 128 Nov 28 13:59 /var/www/html
======== In-container filesystem:
Filesystem     Type          1K-blocks      Used Available Use% Mounted on
grpcfuse       fuse.grpcfuse 971350180 444624972 526725208  46% /var/www/html
Success accessing database... db via TCP/IP
ddev is working. You will want to delete this project with 'ddev delete -Oy tryddevproject-8926'
Thanks for running the diagnostic. It was successful.
Please provide the output of this script in a new gist at gist.github.com
Running ddev launch in 5 seconds
If you're brave and you have jq you can delete all tryddevproject instances with this one-liner:
    ddev delete -Oy $(ddev list -j |jq -r .raw[].name | grep tryddevproject)
In the future ddev debug test will also provide this option.

Please delete this project after debugging with 'ddev delete -Oy tryddevproject-8926'
flovntp@VNTP-Book-3 sfcon-bigfoot1-workshop %

Expected Behavior
After using this line of code :
symfony get llynyw5ygskhg
ddev config
ddev get platformsh/ddev-platformsh
ddev pull platform
ddev restart

it should not result in a HTTP 500 and display my web pages.
Actual Behavior
When restarting DDEV, it executes the PSH build with this :
  - exec: |
      set -x -e
      curl -s https://get.symfony.com/cloud/configurator | bash
      symfony-build

But there is an error that i can't explain, when trying to execute migration :
Executing script assets:install public [OK]

+ [[ -d var/cache/prod ]]
+ mkdir -p tmp/cache
+ mv var/cache/prod tmp/cache/
mv: cannot move 'var/cache/prod' to 'tmp/cache/prod': File exists
Task failed: Exec command 'set -x -e
curl -s https://get.symfony.com/cloud/configurator | bash
symfony-build
' in container/service 'web': exit status 1
A task failure does not mean that ddev failed, but your hook configuration has a command that failed.
+ symfony-deploy
+ symfony-start
+ rm -rf var/cache/prod 'app/cache/*'
+ [[ -d tmp/cache/prod ]]
+ [[ -z '' ]]
+ composer show symfony/flex
+ SYMFONY_APP_CACHE_DIR=var/cache
+ mkdir -p var/cache
+ cp -Rp tmp/cache/prod var/cache
+ [[ 0 != \0 ]]
+ pkill -USR2 -f php-fpm
+ symfony-database-migrate
+ composer show doctrine/doctrine-migrations-bundle
+ symfony console doctrine:migrations:migrate -n --allow-no-migration
13:02:21 CRITICAL  [console] Error thrown while running command "doctrine:migrations:migrate -n --allow-no-migration". Message: "Class "Symfony\Bridge\Doctrine\SchemaListener\PdoCacheAdapterDoctrineSchemaSubscriber" not found" ["exception" => Error { …},"command" => "doctrine:migrations:migrate -n --allow-no-migration","message" => "Class "Symfony\Bridge\Doctrine\SchemaListener\PdoCacheAdapterDoctrineSchemaSubscriber" not found"]

In getDoctrine_Orm_Listeners_PdoCacheAdapterDoctrineSchemaSubscriberService.php line 20:

  Attempted to load class "PdoCacheAdapterDoctrineSchemaSubscriber" from namespace "Symfony\Bridge\Doctrine\SchemaListener".
  Did you forget a "use" statement for another namespace?

If i execute again a composer install, then executing this command success:
flovntp@VNTP-Book-3 sfcon-bigfoot1-workshop % composer install
Installing dependencies from lock file (including require-dev)
Verifying lock file contents can be installed on current platform.
Nothing to install, update or remove
Package twig/extensions is abandoned, you should avoid using it. No replacement was suggested.
Generating optimized autoload files
composer/package-versions-deprecated: Generating version class...
composer/package-versions-deprecated: ...done generating version class
74 packages you are using are looking for funding.
Use the `composer fund` command to find out more!

Run composer recipes at any time to see the status of your Symfony recipes.

Executing script cache:clear [OK]
Executing script assets:install public [OK]

flovntp@VNTP-Book-3 sfcon-bigfoot1-workshop % ddev php bin/console doctrine:migrations:migrate -n --allow-no-migration

 [OK] Already at the latest version ("DoctrineMigrations\Version20191010184120")

so, i guess there is something that went wrong during the symfony-build stuff, maybe due to the var/cache/prod that could not be removed?
Steps To Reproduce
symfony get llynyw5ygskhg
ddev config
ddev get platformsh/ddev-platformsh
ddev pull platform
ddev restart

Anything else?
No response

I would think you would have to do a ddev composer install

Class "Symfony\Bridge\Doctrine\SchemaListener\PdoCacheAdapterDoctrineSchemaSubscriber

certainly seems to say it doesn't get done.
 So the key is to debug symfony-build. You can do that manually by running it manually.
I think symfony-build is a PHP script right, so you can debug it?

Unfortunately, I think you may have intended this issue for platformsh/ddev-platformsh, but I can't transfer it there. I'll close it here, but it seems like you understand it very well now and could just create a simpler issue about ddev-platformsh/symfony-build failing?

To prevent this kind of fail-too-late scenario you can use ddev config --fail-on-hook-fail as mentioned in

https://github.com/platformsh/ddev-platformsh/issues/65

No response, so closing. I'll be happy to work on this with you, just pick it up here when you get back. This doesn't really belong in this issue queue, but really happy to work with you on it.


How to get executable path with lime?

Hello. How I can get current executable path in Haxe when programm was launched by lime/openfl?
for example:
#if neko
    trace('neko >> ${neko.vm.Module.local().name}');
#elseif cpp
    trace('cpp  >> ${Sys.executablePath()}');
#end

will return:

result for neko target (lime test linux -64 -neko): neko >>
result for cpp target (lime test linux -64 -cpp): cpp >> /usr/local/bin/lime

(this example works fine when I launch compiled files "bin/Main" or "neko bin/Main.n" directly, but I need to get this path when programm was launched by lime/openfl too)
(theme on stackoverflow http://stackoverflow.com/questions/39253573/how-to-get-executable-path)
Thanks.

Sys.executablePath() was buggy (but you can use it on neko too I think),
if you have haxe 3.3+ can you try Sys.programPath()?

No, I haven't haxe 3.3 yet
When I install it, I'll try to call this function
Thank you for your answer!


Continous timeout error between CL<>EL communication

Describe the bug
Connection times out on CL when trying to reach EL.
Steps to reproduce

Run Reth
Run Lighthouse

Node logs
RETH logs:
2023-07-13T10:37:49.281359Z  WARN connection{remote_addr=<IP_ADDRESS>:52728 conn_id=199}: jsonrpsee_server::server: HTTP serve connection failed hyper::Error(IncompleteMessage)

Lighthouse:
Jul 13 10:42:12.776 WARN Execution engine call failed            error: HttpClient(url: http://localhost:8551/, kind: timeout, detail: operation timed out), service: exec
Jul 13 10:42:12.776 WARN Error whilst processing payload status  error: Api { error: HttpClient(url: http://localhost:8551/, kind: timeout, detail: operation timed out) }, service: exec

Platform(s)
No response
What version/commit are you on?
commit 99240906a844ae7a8b6b6032fc3b0f64dce22f25 (HEAD -> main, origin/main, origin/HEAD)
What database version are you on?
Current database version: 1
Local database version: 1
If you've built Reth from source, provide the full command you used
cargo install --locked --path bin/reth --bin reth
Code of Conduct

[X] I agree to follow the Code of Conduct

can you please also attach the reth logs that happen at the same time?

@Rjected @shekhirin
this looks weird why is this now happening while the pipeline is active

I wonder if this change somehow caused it https://github.com/paradigmxyz/reth/blob/dbafe23cce45917a43f6640d71a0216fe3268428/crates/consensus/beacon/src/engine/mod.rs#L1573-L1574

yes found it
the poll loop is now flawed
working on it


Add missing mapbox GL removal for cleaning up

Fix #46

@alex3165 Should be fine. Can be included in the next release, I guess


Developer

🤔 This is a

[ ] Feature
[ ] Bug
[ ] Documentation Update
[X] Translation

✍ Description

I wrote and translated the README file to German.
I hope that helps to make learning more accessible to the German community.

Added to documentation

[ ] readme file
[ ] No documentation needed
[X] Just translated it

🐱💻 Use a meme/gif to describe how you feel for contributing (optional)

Hey, thanks for translating it to german.
Instead of using the develop branch, create feature/German-translation. 👍🏽


Leaky Linkification

On iOS 7.1, linkified URLs causes other messages to be linkified when they should not be.
This is almost trivially reproducible in the demo included with JSQMessagesViewController:

Run the demo on an iOS 7.1 device (or the simulator)
Under “Demo Options / Settings”, enable “Load extra messages”
Select one of the presentation options
Send a message beginning with a URL (e.g. “http://vice.com”)
Scroll up and find that "Now with media messages!” is now linked to vice.com...

Perhaps a UITextView bug?  Perhaps a duplicate of #482?

Thanks @bteitelb !
Closing as duplicate of #613


Configure a web-server for serving maps files

Installing nginx and create proper directory structure for serving mwm files.

Done


Can I deactivate used env if I cd to folder

Could current activated env be deactivated if I cd to folder which is not subfolder of current directory?
Also if I using such .env:
source oenv/bin/activate
it works when I enter to this folder, but then when I go to subfolder I'm getting:
bash: oenv/bin/activate: No such file or directory
I can fix it using absolute path, but it will be activated twice, isn't it.

+1

:+1: for resetting environment variables.
@kennethreitz Limiting the scope for environment variables only (not any bash script) sounds good, but dramatically changes the functionality of autoenv, right?

Correct.

How about calling a 'tear down' function if it is defined in .env? For example, to deactivate a conda environment...
on_leave() {
    echo "Deactivating environment"
    source deactivate
}

i mkdir "dac" a directory  and touch .env with the line in the "dac"
deactivate
cd ..

now i can cd dac and deactivate and back

Closing as this has been implemented in v0.3.0 with the .env.leave file. It's not perfect (autoenv is meant to be as simple as possible), but it's what you would use to do this.


Viva products are missing from the data locations

The data locations for Viva Goals, Viva Learning, Viva Topics and Viva Engage missing from the Viva section and if the tables for each country
[Enter feedback here]

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

ID: 61c6fa7b-7e9e-21a5-40da-adbd35040ec1
Version Independent ID: 530df3b2-2048-8351-2f09-59e604aedf5b
Content: Microsoft 365 data locations - Microsoft 365 Enterprise
Content Source: microsoft-365/enterprise/o365-data-locations.md
Service: o365-solutions
GitHub Login: @kelleyvice-msft
Microsoft Alias: kvice

@kelleyvice-msft please help us in resolving this issue. Thanks


Support HEAD method

This is one way to solve HEAD requests. I don't like fiddling with err.mesage, but I don't like idea to pass req into send method.
Closes #184

@tim @leo another way is to add req to res object and move checks into send method.

I'm -1 on this. I think send would be too magical

@rauchg yeah, I don't like this too, but is there a better way to support HEAD methods?

https://github.com/zeit/micro/pull/189#issuecomment-287581566
@rauchg please describe how this can be solved.

@leo it is possible to wrap send and sendError in user land, guess something like module micro-methods will do.

Is this really not coming back? When would you ever want HEAD to include the body when using send()? I don't get how that's too magical...


Add token properties extension

Simple on-chain tokenId-based properties extension with support for all data types indexed by numbers.

Have you considered having an array of properties and then only assigning the IDs of these resources? So instead of:
mapping(uint256 => mapping(uint256 => string)) private _stringProperties;

This would be split into two parts:
mapping(uint256 => string) private stringProperties;
mapping(uint256 => mapping(uint256 => stringProperties)) private _stringProperties;

I agree it would be beneficial to save on storage costs when applying a token property on multiple tokens this way. Only issue is that this syntax is not valid
mapping(uint256 => string) private stringProperties;
mapping(uint256 => mapping(uint256 => stringProperties)) private _stringProperties;
It would go something like this and it looks a bit ugly:
mapping(uint256 => mapping(string => mapping(uint256 => string))) private _stringProperties;

I've changed uint256 indexes to use a string key-value store and will add tests next.

Have you considered having an array of properties and then only assigning the IDs of these resources? So instead of:
mapping(uint256 => mapping(uint256 => string)) private _stringProperties;

This would be split into two parts:
mapping(uint256 => string) private stringProperties;
mapping(uint256 => mapping(uint256 => stringProperties)) private _stringProperties;

I agree it would be beneficial to save on storage costs when applying a token property on multiple tokens this way. Only issue is that this syntax is not valid
mapping(uint256 => string) private stringProperties;
mapping(uint256 => mapping(uint256 => stringProperties)) private _stringProperties;

It would go something like this (with string keys) and it looks a bit ugly:  mapping(uint256 => mapping(string => mapping(uint256 => string))) private _stringProperties;

How about a single mapping-value struct?
So something akin to:

    struct StringProperties {
        mapping(uint256 => StringProperties) stringProperties;
    }

    mapping(uint256 => mapping(string => StringProperties)) private _stringProperties;

I can't decide whether this is an improvement or not. It adds a bit more complexity, but also adds the gas saving feature we are discussing 🤔

Also, the docs were removed in one of the latest pushes and need to be regenerated and readded.


feat: support dynamic width and dynamic precision specs

Fixes #9

Codecov Report
Patch coverage: 95.12% and project coverage change: -0.04% :warning:

Comparison is base (75f8f04) 98.75% compared to head (5914457) 98.72%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #29      +/-   ##
==========================================
- Coverage   98.75%   98.72%   -0.04%
==========================================
  Files          43       44       +1
  Lines        4514     4555      +41
==========================================
+ Hits         4458     4497      +39
- Misses         56       58       +2

Files Changed
Coverage Δ

include/emio/formatter.hpp
95.23% <88.88%> (-4.77%)
:arrow_down:

test/unit_test/test_dynamic_format_specs.cpp
100.00% <100.00%> (ø)

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


[BUG] files input with directories not working as expected

Is there an existing issue for this?

[X] I have searched the existing issues

Does this issue exist in the latest version?

[X] I'm using the latest release

Describe the bug?
Up to v35 it was possible to specify directories to check for changed files without adding a glob pattern at the end. Now this seems to be required, but I am not sure if this is desired behaviour or accidental bug since README says it should be possible without glob pattern:
files: docs/*.{js,html}  # Alternatively using: `docs/**` or `docs`

To Reproduce
Using the following steps:
      - name: Get changed files
        id: changes
        uses: tj-actions/changed-files@v36
        with:
          files: |
            .github/workflows

      - name: Set files changed flag
        if: steps.changes.outputs.any_changed == 'true'
        id: changeset
        run: echo "run_job=true" >> $GITHUB_OUTPUT

Make any changes in a workflow file. Second step gets skipped. Adding the glob pattern at the end (.github/workflows/**) makes it work again. Alternatively, change version to v35 and it also works again.
What OS are you seeing the problem on?
ubuntu-latest or ubuntu-20.04
Expected behavior?
The changed files in a directory should be recognized again without explicitly adding the glob pattern at the end. Alternatively, the README could be clearer to bring out that this is required now.
Relevant log output
Run tj-actions/changed-files@v36
  with:
    files: .github/workflows

    separator:
    include_all_old_new_renamed_files: false
    old_new_separator: ,
    old_new_files_separator:
    files_from_source_file_separator:

    files_separator:

    files_ignore_separator:

    files_ignore_from_source_file_separator:

    path: .
    quotepath: true
    diff_relative: true
    dir_names: false
    dir_names_exclude_root: false
    json: false
    escape_json: true
    fetch_depth: 50
    since_last_remote_commit: false
    write_output_files: false
    output_dir: .github/outputs

...

##[debug]Node Action run completed with exit code 0
##[debug]Set output added_files =
##[debug]Set output copied_files =
##[debug]Set output modified_files =
##[debug]Set output renamed_files =
##[debug]Set output type_changed_files =
##[debug]Set output unmerged_files =
##[debug]Set output unknown_files =
##[debug]Set output all_changed_and_modified_files =
##[debug]Set output all_changed_files =
##[debug]Set output any_changed = false
##[debug]Set output only_changed = false
##[debug]Set output other_changed_files = .github/workflows/reusable.frontend.check-changes.yml
##[debug]Set output all_modified_files =
##[debug]Set output any_modified = false
##[debug]Set output only_modified = false
##[debug]Set output other_modified_files = .github/workflows/reusable.frontend.check-changes.yml
##[debug]Set output deleted_files =
##[debug]Set output any_deleted = false
##[debug]Set output only_deleted = false
##[debug]Set output other_deleted_files =
##[debug]Finishing: Get changed files for frontend

Anything else?
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

@florianbuth-sympower This should be resolved now.

Indeed, now it is working again. Thank you so much for your quick action!


Wrong port for connecting to bot.

3978 not 3999

Thanks @dmvtech
@EricDahlvang has incorporated this in a set of other changes just going in. Closing this one here.


Android: Fix lineCap of Polyline

Does any other open PR do the same thing?

No
What issue is this PR fixing?
https://github.com/react-community/react-native-maps/issues/2374
How did you test this PR?

Verified on Android, HTC M8

Hi, thanks for the PR ❤
Does it work correctly on ios? (both ios and google maps)

Hi @rborn, on iOS, the MapKit version works correctly.
for GoogleMaps on iOS, the native SDK doesn't support lineCap unfortunately (as of today).
https://developers.google.com/maps/documentation/ios-sdk/shapes

@rborn I send you the screenshots of iOS and Android

@kruyvanna that's great, one more thing: could you please amend the  docs that it's not supported yet? https://github.com/react-community/react-native-maps/blob/master/docs/polyline.md

@rborn done. Not sure about my English wording.

@rborn I corrected the docs changing from lineJoin to lineCap

All good, thanks a lot ❤
@alvelig LGTM 🐽

@rborn Btw, I added support for lineDashPattern on Android.
The appearance of the dash is dependent on lineCap property.
So, should I wait for this PR to be merged first then make a new PR or should I make a new PR right a way.
https://github.com/kruyvanna/react-native-maps/commit/f0ac2e5996e6b1f2e7f1ac1c1a11e2a669ab45f9

@kruyvanna new PR please

@rborn ok


How to setup routing with angular-cli: 1.0.0-beta.22-1 and webpack

Hi
I am using angular-cli: 1.0.0-beta.22-1 AND NG2 v 2.3 on mac OS Sierra.
Most of the times when i update angular-cli my routing files start giving errors (even though they were working just fine previously). I just wanted to know what is the correct way to do routing?
Here is my file which i have been using and tweaking slightly with every major release but now i am pretty much stuck:
import { NgModule, ModuleWithProviders }	from '@angular/core';
import { Routes, RouterModule } from '@angular/router';

import { PageNotFoundComponent } from './shared/page-not-found/page-not-found.component';

export function loadCatalog() {
	return new Promise(resolve => {
		(require as any).ensure([], () => {
				resolve(require('./catalog/catalog.module').CatalogModule);
		})
	});
}

export function loadLogin() {
	return new Promise(resolve => {
		(require as any).ensure([], () => {
				resolve(require('./login/login.module').LoginModule);
		})
	});
}

export const routes: Routes = [
	{
		path: 'catalog',
		loadChildren: loadCatalog,
	},
	{
			path: 'login',
			loadChildren: loadLogin,
	},
	{ path: '', redirectTo: '/catalog', pathMatch: 'full' },
	{ path: '**', component: PageNotFoundComponent },
];

@NgModule({
  imports: [ RouterModule.forRoot(routes) ],
  exports: [ RouterModule ]
})
export class AppRoutingModule {}

This is the error i get when i use ng build
 10% building modules 2/2 modules 0 activeentry.split is not a function
TypeError: entry.split is not a function
    at Function.ModuleRoute.fromString (/Users/hassan/Code/project/node_modules/@ngtools/webpack/src/plugin.js:25:27)
    at /Users/hassan/Code/project/node_modules/@ngtools/webpack/src/plugin.js:274:43
    at Array.map (native)
    at AotPlugin._processNgModule (/Users/hassan/Code/project/node_modules/@ngtools/webpack/src/plugin.js:273:14)
    at /Users/hassan/Code/project/node_modules/@ngtools/webpack/src/plugin.js:242:39
    at process._tickCallback (internal/process/next_tick.js:103:7)

If i comment these three blocks:
{
	path: 'catalog',
	loadChildren: loadCatalog,
},
{
	path: 'login',
	loadChildren: loadLogin,
},
{ path: '', redirectTo: '/catalog', pathMatch: 'full' },

i can get the application to compile without issue.
Can someone please tell me what i am doing wrong?
P.S: It was a working application before angular-cli update to beta 22-1 and NG2 2.3
I know i am not alone when i say this that updating an app is getting increasingly frustrating when one has to debug so many issues after each update :'(

loadChildren takes a string of the form path/to/file#ModuleClass. Looks like in your case that would be './catalog/catalog.module#CatalogModule'.


Deleting cart item view should not raise an error

When cart item does not exists, view should return status code 404.

Closed. Please check version 0.9.


Accept Service records via CSW-T with custom metadata

Accept Service records via CSW-T with custom metadata, create the Service and propagate the “” metadata back to the layers.

Fixed.


Keybindings for project tasks

Requested feature
As a user, I would like to set my own keybindings to run my project tasks through a shortcut. Currently, I need to set them in a global keybindings.json file. However, if I want to use the same shortcut for several tasks, which are located in different projects, I cannot do that, because shortcuts must be defined globally, so a keybinding can be only used by only one task.
A good solution to solve this problem would be to define keybindings per project. By doing that, I could use the same shortcut for running all the project servers. For example, if we have a web application with a front-end and a back-end server in separated projects, ctrl+shift+1 shortcut could be set in both servers to run their applications.
Besides, this was a functionality requested in a previous issue, so I think it would be worth to add it to improve the IDE.
Possible solutions

As proposed in the mentioned issue above, we could set a local keybinding file inside the project worskpace, which would be ubicated in .vscode/keybindings.json and overwrite the configuration defined in the global keybindings.json file. By doing that, we could set any action to run in the project.

Add a new option for task content in the tasks.json file. It would be called keybinding and have the shortcut as a value. With this option, we could establish the same shortcut for tasks located in different projects. An example task would be this one:

{
     "label": "Example task",
     "type": "shell",
     "command": "echo \"Example task\"",
     "options": {
          "cwd": "${workspaceFolder}"
     },
     "keybinding": "ctrl+e"
},

Personally, I would prefer the second option as this would allow to associate more easily the shortcut with the created task. However, if we would want to extend this to any action, I would choose the first option.
I think that this would be a functionality that would be worth adding it. For that, I would be open to contribute in the development for this desired functionality.

All keybindings should live in the same place. Duplicate of https://github.com/microsoft/vscode/issues/101182.


[Bug]: Feedback 405 error

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
405 error on feedback page
https://github.com/user-attachments/assets/da223485-6e46-40f6-b717-af98e2f54269
Add Screenshots
No response
What browsers are you seeing the problem on?
Chrome
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC contributor
[X] I want to work on this issue
[X] I'm willing to provide further clarification or assistance if needed.

@4F24L
How can you self assign this issue?

@4F24L
How can you self assign this issue?

She asked me to fix that issue. And she wasn't available when the issue was being raised.


github/provider: introduce owner

Continuing the work done by @elislusarczyk in #96 to support individual github accounts for repos
Changes here address last review comments in #96 :

transition off of GITHUB_ORGANIZATION to GITHUB_OWNER (across acctests as well)
check whether the owner is org and if not, return a user-friendly error (#checkOrganization inutil.go)
deprecate organization in provider.go
skip acctests where organization is needed
add IsOrganization to Owner struct

Output of acceptance tests with owner set as Github User:
--- PASS: TestEtagTransport (0.00s)
--- PASS: TestRateLimitTransport_abuseLimit_get (0.00s)
--- PASS: TestRateLimitTransport_abuseLimit_post (0.00s)
--- PASS: TestRateLimitTransport_abuseLimit_post_error (0.00s)
--- PASS: TestAccValidateTeamIDFunc (0.00s)
--- PASS: TestAccGithubUtilRole_validation (0.00s)
--- PASS: TestAccGithubUtilTwoPartID (0.00s)
--- PASS: TestAccGithubReleaseDataSource_invalidRetrieveMethodReturnsError (0.03s)
--- PASS: TestAccGithubReleaseDataSource_fetchByTagNoTagReturnsError (1.57s)
--- PASS: TestAccGithubReleaseDataSource_fetchByIdWithNoIdReturnsError (1.62s)
--- PASS: TestAccGithubUserDataSource_noMatchReturnsError (1.72s)
--- PASS: TestAccGithubReleaseDataSource_fetchByLatestNoReleaseReturnsError (2.01s)
--- PASS: TestAccGithubMembershipDataSource_noMatchReturnsError (2.06s)
--- PASS: TestAccGithubIpRangesDataSource_existing (8.22s)
--- PASS: TestAccGithubUserGpgKey_basic (11.04s)
--- PASS: TestAccGithubUserSshKey_basic (11.17s)
--- PASS: TestAccGithubUserDataSource_existing (12.14s)
--- PASS: TestAccProvider_individual (12.56s)

* remaining tests are skipped

Gave this one a skim and am 👍 on the approach.  It is a large diff so I plan on reading it over the next few days.  Excited to see this 🚢 !

Looks like owner is not needed at all - only token matters https://developer.github.com/v4/guides/forming-calls/#authenticating-with-graphql
For example GitLab doesn't use any kind of username for API access https://github.com/terraform-providers/terraform-provider-gitlab/blob/74c5685c69a9926c39c6beccebc89e29217494b0/gitlab/config.go#L14

@abitrolly this is good news, but may cause us to take a step back and re-evaluate how best to accomplish things.  We are currently trying to leverage the v3 API and is unclear to me whether relying on the token is available outside of v4.  My aim is to read more but welcome a more thorough explanation of what could be changed.

@anGie44 - I've gotten this branch into a strange state by fixing conflicts using the web UI 😕.
Also, I am now realizing that local test runs with the configured user account are incompatible:
--- SKIP: TestAccGithubRepositoryWebhook_secret (0.50s)
    resource_github_repository_webhook_test.go:73: Skipping because GITHUB_OWNER "github-terraform-test-user" is a user, not an organization.
PASS
ok  	github.com/terraform-providers/terraform-provider-github/github	0.867s

Should a non-organization user be able to run acceptance tests? My original assumption was that owner was an abstraction of organization and personal-account.  Will give this another read through to better understand things, but please clarify if I've misunderstood.

Yeah ideally we'd want an owner set with an individual user to be able to run all tests but that would require that we remove the checkOrganization method calls in a lot of the resources/data_sources and also adjust acctests where certain values are only available for organizations vs. users (e.g. branch protection, repo collaborator permissions) ..i remember going about this but held back as that would be a lot to review in one go. instead I was thinking of partitioning out changes like in #465 by resource and/or datasource.

Is there a new Terraform provider version with these changes yet? I'm running into the same issue:
module.example_repo.github_repository.repo: Creating...

Error: This resource requires GitHub organization to be set on the provider.

hi @emmasax4 👋 ! Not yet, but will be part of the next provider milestone which is the release of v2.9.0 😃

@anGie44 Awesome! Do we have any idea yet of when that next release will happen?

hi @anGie44 .. this causes 'token required' errors in our CI now... so seems to be a breaking change. Is this expected to be breaking as it was released as a minor version change rather than a major one? the token is provided via an environment variable in our case and we are still using GITHUB_ORGANIZATION env to specify the organization. So not expecting this to break in a minor release.

we were facing some issues.. will dig deeper before re posting my deleted post... sorry for the noise

@mariux no worries! I believe you should still be able to source your token from an environment variable and initialize the provider as usual however maybe we could reconsider moving this back to Optional @jcudit ? at one point the token attribute was required and then changed to Optional to support the anonymous OAuth, though now removed in this version.

I was going to ask if the next version should be 3.0.0 because of potential compatibility break, but got lazy. :D

😄 was considering a major version bump as well.  Hoping the community is forthcoming with bug reports so we can mop them up during v2.9.1.

Seems to be only terraform validateto be affected.. will get back, once confirmed.

I can now confirm that with provider 2.8.0 terraform validate runs fine without a token and with 2.9.0 terraform validate needs a token to run.
specifying any token (token does not need to be valid) via GITHUB_TOKEN environment variable fixes it.
I am not sure if this is worth to be called a full-blown regression as we can easily fix this in our CI by providing the required env variable. (we run automated tests for our open source github modules)
Should I open a bug report anyway?

@mariux what could you do with anonymous GitHub provider before?

Should I open a bug report anyway?

@mariux yes, please.  Capturing this is valuable.

I just started receiving this Warning which seems spurious because I'm not using or declaring Organization - I already use Owner. Is this Warning always emitted intentionally?
Warning: "organization": [DEPRECATED] Use owner field (or GITHUB_OWNER ENV variable)
My codebuild github config looks like this
action {
  category         = "Source"
  ~ configuration    = {
    "Branch"               = "***my branch***"
    + "OAuthToken"           = "*******"
    "Owner"                = "***my github company org***"
    "PollForSourceChanges" = "false"
    "Repo"                 = "********removed********"
  }

Initializing provider plugins...
- Checking for available provider plugins...
- Downloading plugin for provider "github" (hashicorp/github) 2.9.0...
- Downloading plugin for provider "aws" (hashicorp/aws) 2.68.0...

@mariux what could you do with anonymous GitHub provider before?

We provide a module and run automated tests for static analysis and unit testing. The static analysis runs tools like tflint and terraform validate. terraform validate started failing with 2.9.0 if no token is provided. As this is a module we do not configure the provider at all but just have a version constraint defined:
terraform {
  required_version = "~> 0.12.9"

  required_providers {
    github = "~> 2.4"
  }
}


Following the guide doesn't produce a build-able project

Working through the quick start copying and pasting snippets (in order) wont produce a build-able project.
As another user has pointed out already, the New Method isn't guiding the user to success.
If the user blindly copies and pastes then they'll also end up with duplicate declarations for:
const string RECOGNITION_MODEL2 = RecognitionModel.Recognition02;
const string RECOGNITION_MODEL1 = RecognitionModel.Recognition01;

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

ID: a1ed5f85-d750-c7a9-c976-6bce2e439346
Version Independent ID: d0176b47-447e-c04f-f4fe-8ca6e1b5eee8
Content: Quickstart: Face client library for .NET - Azure Cognitive Services
Content Source: articles/cognitive-services/Face/QuickStarts/csharp-sdk.md
Service: cognitive-services
Sub-service: face-api
GitHub Login: @PatrickFarley
Microsoft Alias: pafarley

@MikeCodesDotNET we will review your feedback and get back to you shortly. Thanks.

We did have a few conditional instructions in this guide ("If you haven't done X already, do so here"). It's becoming clear that this isn't preferred. I've combined two sections so that the method can be fully defined from beginning to end. And I've moved the common code to the beginning of the guide, so users will have it regardless of what scenario they want to code. https://github.com/MicrosoftDocs/azure-docs-pr/pull/111574

PR merged, public withing 24 hrs
#please-close


[bug]: Unhandled Runtime Error TypeError: Cannot destructure property 'getFieldState' of '(0 , {imported module [project]/nodemodules/.pnpm/react-hook-form<EMAIL_ADDRESS>[app-client] (ecmascript)}.useFormContext)(...)' as it is null.

Describe the bug

React does not recognize the handleSubmit prop on a DOM element. If you intentionally want it to appear in the DOM as a custom attribute, spell it as lowercase handlesubmit instead. If you accidentally passed it from a parent component, remove it from the DOM element.
React does not recognize the setValue prop on a DOM element. If you intentionally want it to appear in the DOM as a custom attribute, spell it as lowercase setvalue instead. If you accidentally passed it from a parent component, remove it from the DOM element.
React does not recognize the getValues prop on a DOM element. If you intentionally want it to appear in the DOM as a custom attribute, spell it as lowercase getvalues instead. If you accidentally passed it from a parent component, remove it from the DOM element.
React does not recognize the resetField prop on a DOM element. If you intentionally want it to appear in the DOM as a custom attribute, spell it as lowercase resetfield instead. If you accidentally passed it from a parent component, remove it from the DOM element.
React does not recognize the clearErrors prop on a DOM element. If you intentionally want it to appear in the DOM as a custom attribute, spell it as lowercase clearerrors instead. If you accidentally passed it from a parent component, remove it from the DOM element.
React does not recognize the setError prop on a DOM element. If you intentionally want it to appear in the DOM as a custom attribute, spell it as lowercase seterror instead. If you accidentally passed it from a parent component, remove it from the DOM element.
React does not recognize the setFocus prop on a DOM element. If you intentionally want it to appear in the DOM as a custom attribute, spell it as lowercase setfocus instead. If you accidentally passed it from a parent component, remove it from the DOM element.
React does not recognize the getFieldState prop on a DOM element. If you intentionally want it to appear in the DOM as a custom attribute, spell it as lowercase getfieldstate instead. If you accidentally passed it from a parent component, remove it from the DOM element.
React does not recognize the formState prop on a DOM element. If you intentionally want it to appear in the DOM as a custom attribute, spell it as lowercase formstate instead. If you accidentally passed it from a parent component, remove it from the DOM element.
Invalid values for props trigger, register, watch, reset, unregister on  tag. Either remove them from the element, or pass a string or number value to keep them in the DOM. For details, see https://react.dev/link/attribute-behavior
⨯ src/components/ui/form.tsx (47:11) @ useFormField
⨯ TypeError: Cannot destructure property 'getFieldState' of '(0 , TURBOPACK__imported__module$5b$project$5d2f$node_modules$2f2e$pnpm$2f$react$2d$hook$2d$form$40$7$2e$53$2e$2_react$40$18$2e$3$2e$1$2f$node_modules$2f$react$2d$hook$2d$form$2f$dist$2f$index$2e$esm$2e$mjs__$5b$app$2d$ssr$5d$$28$ecmascript$29$.useFormContext)(...)' as it is null.
at useFormField (./src/components/ui/form.tsx:47:11)
at FormLabel (./src/components/ui/form.tsx:93:33)
digest: "2630363313"
45 |   const fieldContext = React.useContext(FormFieldContext);
46 |   const itemContext = React.useContext(FormItemContext);

47 |   const { getFieldState, formState } = useFormContext();
|           ^
48 |
49 |   const fieldState = getFieldState(fieldContext.name, formState);
50 |

Affected component/components
Form
How to reproduce

install nextjs latest project and shadcn
add form
run the development server

Codesandbox/StackBlitz link
No response
Logs
No response
System Info
## Hardware Information:
- **Memory:**                                      8.0 GiB
- **Processor:**                                   Intel® Core™ i5-10210U × 8
- **Graphics:**                                    Intel® UHD Graphics (CML GT2)
- **Graphics 1:**                                  NVIDIA GeForce MX250
- **Disk Capacity:**                               512.1 GB

## Software Information:
- **Firmware Version:**                            1.24.0
- **OS Name:**                                     Ubuntu 24.04.1 LTS
- **OS Build:**                                    (null)
- **OS Type:**                                     64-bit
- **GNOME Version:**                               46
- **Windowing System:**                            X11
- **Kernel Version:**                              Linux 6.8.0-49-generic

Google Chrome

Before submitting

[X] I've made research efforts and searched the documentation
[X] I've searched for existing issues

I have a similar problem


SharePoint and OneDrive Functional Testing

💡 Summary
As part of Iceberg release, to ensure that the tool's update is accurate, detailed functional testing has been planned. Each policy will be functionally tested to check that all pass/fail scenarios have been correctly processed by the tool using existing automated functional test plans.
This issue functionally tests each policy of SharePoint/OneDrive product of ScubaGear tool. The main objective is to test SharePoint and OneDrive policies and ensure that ScubaGear tool is providing the correct result for each possible configuration option.
Motivation and context
This functional testing ensures that ScubaGear results of pass/fail are accurate based on the latest version of the code.
Implementation notes
For each SharePoint and/or OneDrive policy do the following:

[ ] Configure the automated testing and run against each tenant type for which a variant testplan is present.  Run the automated test plans (e.g. regular, g5, etc.) and note the results.
[ ] For each test failure, investigate and note the cause of the failure and log any new issues needed to resolve

For details on how to use and execute the functional tests, please see the Testing documentation in Testing/Readme.md or reach out to the dev team via other channels.
Acceptance criteria

[ ] All existing test plan variants for the product have been successfully run
[ ] Test plan results have been reviewed and any issues resolved or reported as new issues (Link created issues below)

G3 PnP Test
Ran with service principal and got three failed test cases which are expected for policy 1.4 due to a recently discovered settings that is being deprecated by Microsoft. See #1267 for details.

G5 Pnp Test
Same results as G3 Pnp test.

E5 Pnp test
Same results as G3 Pnp test. No new issues.

GCC High Pnp test
Same results as G3 Pnp test. No new issues.

E5 SPO test
Same results as G3 Pnp test. No new issues.

G5 SPO test
Same results as G3 Pnp test. No new issues.

I think this should be closed.


Customizable npc loot

self explanatory. option to add random amounts of said item would be good to have as well.

Test in 1.4.3 please
CRI's are generated when loading a save or whatever you're wanting to configure, gets spawned by the game

Test in 1.4.3 please CRI's are generated when loading a save or whatever you're wanting to configure gets spawned by the game

config file is blank

Did you enable the Custom Random Inventories option in the cfg?

yes

Try making a new save, and if loading it doesnt generate all the cri's try making it to day 2 on that same save

Try making a new save, and if loading it doesnt generate all the cri's try making it to day 2 on that same save

yes, that did the trick. i also figured out that the the shell item from bugs is actually called chitin. Other than that the log keeps spewing errors saying the following:
[Error  :DarkwoodCustomizer] [USERCUSTOMRECIPES] Item weapon_submachine_full does not exist, stopping as to not break the plugin!
this same error popped up when i had this item registered in the customitems file as well

That happens in this line of code here in case when the item being loaded, wasn't loaded by the game and my plugin also can't load it, which means it's a NRE (doesn't exist), try finding something similar to that in the itemlog

That happens in this line of code here in case when the item being loaded, wasn't loaded by the game and my plugin also can't load it, which means it's an NRE (NullReferenceException, aka doesn't exist), try finding something similar to that in the itemlog, maybe it's called something else

i literally just copy pasted that item's name from the item log. you must've coded it wrong

Can you try weapon_submachine_01_full instead
And if that doesnt work try witthout the _full

Can you try weapon_submachine_01_full instead And if that doesnt work try without the _full

yes, that did it, now it works thanks. Now i have a question on how the chance values work. if the chance is set to 1.0 does that mean its a 100% chance of spawning? or is it 0.100 instead?

1 is 100%

1 is 100%

1 as in 1.0? or just 1

1 is 100%

1 as in 1.0? or just 1

Yes

1 is 100%

1 as in 1.0? or just 1

Yes

so if i want the chance to be 50% how do i do it? i keep seeing the other item's chances being 0.598 or 0.678 for example,

I'd say 0.5

I'd say 0.5
and how do i make the chance between say like 20 or 40? do i just do 0.24 and 0.33?

It's a float and is probably used like 100% chance being 1.0 and 0.01 being 1% chance
I haven't looked into the code for it to see how it functions but that's how i'd do it

It's a float and is probably used like 100% chance being 1.0 and 0.01 being 1% chance I haven't looked into the code for it to see how it functions but that's how i'd do it

i see, thanks.

Test in 1.4.3 please CRI's are generated when loading a save or whatever you're wanting to configure gets spawned by the game

its partially working, so far ive only seen it work with traders nothing else

Can't figure this out for the life of me, this will take a while

Added in 1.4.6, check out the wiki if you need help with it


MAINT: Remove unused event_count from tracker.

event_count is not referenced anywhere, so remove extra bit of state
tracking.

Checks out to me.

:+1:


Add to|from convention for the direction.

Add to|from convention for the direction.

Hello @haileyajohnson, PR made.
Two things I am not sure.
In the Vectorize.java I trow a IllegalArgumentException("The convention must be either 'to' or 'from'.");. Should I use this error or just trow  a NullPointerException?
Math.atan2 is undefined when u and v are zero, so I added an if in VectorDirection.java to check for this to avoid "surprises" with random garbage numbers being returned. Some tests fail without this if when using from convention.
Thank you.

I forgot to include this, but I did a comparison of results using "to" and "from" convention, and everything seems OK.
to

from

Thanks for the contributions! This looks great!
I think either Exception is fine, I keep meaning to open a PR on netcdf-java to catch any exception thrown by an Enhancement implementation to handle it...


[WIP] Hello Teleop

Description
Please include a summary of the change and link to an issue this PR fixes. Please also include relevant context. List any downstream components that might be affected by this change.
Fixes # (issue)
Type of change
Please check the options that are relevant.

[ ] Bug fix (non-breaking change that fixes an issue)
[ ] Proposes a change (non-breaking change that isn't necessarily a bug)
[ ] Refactor
[ ] New feature (non-breaking change that adds a new functionality)
[ ] Breaking change (fix or feature that would break some existing functionality downstream)
[ ] This is a unit test
[ ] Documentation only change
[ ] Datasets Release
[ ] Models Release

Type of requested review

[ ] I want a thorough review of the implementation.
[ ] I want a high level review.
[ ] I want a deep design review.

Before and After
If this PR introduces a change or fixes a bug, please add before and after screenshots (if this is causes a UX change) or before and after examples(this could be plain text, videos etc ) to demonstrate the change.
If this PR introduces a new feature, please also add examples or screenshots demonstrating the feature.
Testing
Please describe the tests that you ran to verify your changes and how we can reproduce.
Checklist:

[ ] I have performed manual end-to-end testing of the feature in my environment.
[ ] I have added Docstrings and comments to the code.
[ ] I have made changes to existing documentation where needed.
[ ] I have added tests that show that the PR is functional.
[ ] New and existing unit tests pass locally with my changes.
[ ] I have added relevant collaborators to review the PR before merge.
[ ] [Polymetis only] I ran on hardware (1) all scripts in tests/scripts, (2) asv benchmarks.

Codecov Report

Merging #1366 (8b458d8) into main (5e4d64d) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head 8b458d8 differs from pull request most recent head 43dd2fa. Consider uploading reports for the commit 43dd2fa to get more accurate results

@@           Coverage Diff           @@
##             main    #1366   +/-   ##
=======================================
  Coverage   34.13%   34.13%
=======================================
  Files         358      358
  Lines       35541    35541
=======================================
  Hits        12131    12131
  Misses      23410    23410

Impacted Files
Coverage Δ

agents/_agent.py
40.92% <0.00%> (-10.82%)
:arrow_down:

agents/droidlet_agent.py
51.73% <0.00%> (+10.81%)
:arrow_up:

Changes to be merged in #1371 instead.


handle mut parameters per issue #400

Thanks a lot for your contribution but:

You can't modify auto files manually
Some of those functions takes a &mut because they expect you to pass a pointer in C of for example the String to write something on and so on.  Just dropping &mut won't fix most of those. You need to check the C docs and see how that function is expected to behave. Those wrongly generated methods with &mut are often caused by wrong annotations in the C code and we should either contribute fixes or at least open an issue upstream about them.

What I would suggest is trying to handle only 1 method for now and see how the process looks like, don't hesitate to ping me here or on Matrix and I can help guiding you through it if needed

Looks good to me! I will squash the commits into one. If possible, please split things into multiple commits.


hashTracking not working

Hi, I've tried using data-remodal-options="hashTracking: false, REMODAL_GLOBALS and { hashTracking: false }. Nothing seems to be stopping the modal to open when there's a hash.
Can you please confirm?
My current code:
   $('[data-open-form]').click(function (e) {
                e.preventDefault();

                var $this = $(this);

                $modalForm.remodal({ hashTracking: false }).open();
                (...)
    }

With this, when I go to http://MYURL/#modal-hash, the modal opens automatically.

If you initialize it with javascript, do you remove remodal class from the node? https://github.com/VodkaBears/Remodal#initialization-with-javascript.
$modalForm.remodal({ hashTracking: false }).open();

Globals work. Do you have the latest version? There is already v1.0.0.
window.REMODAL_GLOBALS = {
  DEFAULTS: {
    hashTracking: false
  }
};

Sorry, forgot to mention.
I'm actually on Remodal - v1.0.1. Should I use v1.0.0 instead?

Globals should be add before you include the library.
https://github.com/VodkaBears/Remodal#globals

True. That did it.
Thanks!
Just a heads up: it's not working when using data-remodal-options="hashTracking: falseor { hashTracking: false }.

Just a heads up: it's not working when using data-remodal-options="hashTracking: false"or { hashTracking: false }.

Let's investigate. For me it works. Are there any errors in the console?

$modalForm.remodal({ hashTracking: false })
Are you sure that you initialize it only in this place? It takes arguments only for the first call, then it returns the instance.

This is what I'm doing:
            $modalForm.remodal({ hashTracking: false });

            $('[data-open-form]').click(function (e) {
                e.preventDefault();

                var $this = $(this);

                $modalForm.remodal().open();

                (...)

I'm confused.
90% that you initialize it without options somewhere. Please, check it carefully.
Is node without the .remodal class right now? If you set the class for the node, it will be auto-initialized.

Oh man, I'm embarrassed :(
Just found another $modalForm.remodal() where there shouldn't be one. Didn't realize it was there at first... totally my fault.
Sorry for calling this a bug.

Hah, no problems, closed.


No user creation

Updated:
Used the instructions and inserted the following in register_screen in order to patch the app so that user creation actually lead to a user record in the database. Alas it throws the dreaded "Error: Could not find the correct provider ..."  - So what is the deal with register_screen?
FirestoreDatabase firestoreDatabase =         Provider.of<FirestoreDatabase>(context);

Patched:
                            if (userModel == null) {                               _scaffoldKey.currentState.showSnackBar(SnackBar(                                 content: Text(AppLocalizations.of(context)                                     .translate("loginTxtErrorSignIn")),                               ));                             } else {                               // Patched to allow user creation in DB                               // Terrible hack - but life is too short for providers and other gluecode                               final FirestoreDatabase firestoreDatabase =                                   FirestoreDatabase(uid: userModel.uid);                               firestoreDatabase.setUser(userModel);                             }                           }
And in firetore_database.dart:
  //Method to create/update userModel   Future<void> setUser(UserModel user) async => await _firestoreService.setData(         path: FirestorePath.user(uid),         data: user.toMap(),       );
And in user_model.dart:
`class UserModel {
String uid;
String email;
String displayName;
String phoneNumber;
String photoUrl;
UserModel(
{this.uid,
this.email,
this.displayName,
this.phoneNumber,
this.photoUrl});
Map<String, dynamic> toMap() {
return {'uid': uid, 'email': email};
}
}`
Apart from this - great template !!!

Hi @psimondk ,
Thanks for the info. Been cought up with project work. Yes, we need to capture the user into Firebase. Thanks for the sharing of tips - yup, adding the UserModel and the services will resolve the issue. 👍


Clarification

Hi ... I wanted to discuss a few things with you. I think you need to mail me at<EMAIL_ADDRESS>since I don't know your mail id.

Hi, could you send me a message on Slack?


Add to HLint README

Happy to add this to the HLint readme once it's up on Hackage. Just let me know.

Ah, good timing! I just published splint-<IP_ADDRESS> to Hackage.

Added in https://github.com/ndmitchell/hlint/commit/ef5cd6e69a9d4563c903746aed8be69a80a4ca63

Awesome, thank you!


Use vue instead of plain template to render suscribe div of issue

This PR did two things.

Introducing initComponent which can make the component initlize simple than before
Introducing issue/Suscribe.vue to instead of old template method

This can be compared with #28908 about a different framework to do the same thing.

Isn't for example the loading indicator missing?
Overall, I can mostly agree with @wxiaoguang - we should not mix go templates and vue heavily, it creates confusion, makes refactoring harder, adds additional code which can be avoided usind htmx and so on.
I could imagine full Vue rewrites being beneficial for some components that are designed as dynamic in the first place, but for such small elements htmx looks like a better solution.

Isn't for example the loading indicator missing?
Overall, I can mostly agree with @wxiaoguang - we should not mix go templates and vue heavily, it creates confusion, makes refactoring harder, adds additional code which can be avoided usind htmx and so on.
I could imagine full Vue rewrites being beneficial for some components that are designed as dynamic in the first place, but for such small elements htmx looks like a better solution.

The loading added, but the real problem is the div flickering @wxiaoguang mentioned. An obvious layout reaggrange can be found in the area of the subscribe button. The external div of the button needs a fixed height if we want to avoid that.

Close as htmx version merged.


libsForQt5.bonsai: init at 2.2.0

Description of changes
add missing maui application bonsai, mobile git repository manager
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

I'm very confused by the versions in https://download.kde.org/stable/maui/bonsai/
Can someone join https://t.me/mauiproject and ask upstream to fix their versioning? They should upload their files in the same way as Plasma (Mobile).

Opened an upstream issue about this https://invent.kde.org/maui/bonsai/-/issues/2

Created an other bug report upstream https://bugs.kde.org/show_bug.cgi?id=463064


Merge pull request #1 from google/master

Correctly increment # of pages per TCP conn.

I'm not seeing any actual change here... Could it be that this was a dup of https://github.com/google/gopacket/commit/2b92a1effab6f201b6035a66eec5717dd832cc59 ?

Yes, a dup of 2b92a1e


[styles] Add missing classes prop to resulting hook of makeStyles

We can extend classes by passing external className to hook:
const useStyles = makeStyles({
  root: {
    width: ({ size }: SomeProps) => size,
  },
});

function component(props: FullProps) {
  const css = useStyles({ classes: { root: props.className }, size: props.size });
}

But without these changes we can't because classes prop is missing in hook.

@oliviertassinari, @eps1lon have you a time to review this?

Can we keep the overload-approach? Otherwise this breaks some usage where the type arguments were set explicitly.

Unfortunately, we have the same args length and types so TS takes the first declared.
But fortunately TS will notify about wrong type arguments. :)
In old version inference of component props doesn't work too.

Can we keep the overload-approach? Otherwise this breaks some usage where the type arguments were set explicitly.

I have added overload declaration. Could you check it?

Closing for #22342


keyboard event has expired after click other html element

In the official Demo anchors.html, if you click the autoLayout button and delete the selected node, you will find that the keyboard event has expired. I found out that mxKeyHandler.prototype.isGraphEvent will judge the source of evt, and find that the source of evt is still the autolayout button, and it has not been switched to the node. I also encountered this problem on my project. In one sentence, as long as I clicked on the html element other than mxgraph containtor, subsequent keyboard events would not be captured.

There is no autoLayout button in https://jgraph.github.io/mxgraph/javascript/examples/anchors.html

Anchors example for mxGraph

That is not the anchors example, https://github.com/jgraph/mxgraph/blob/master/javascript/examples/anchors.html is

like code at "paper-play commented Jul 24, 2019", when I click autoLayout button then key event will expired. If I refresh the browser and delete the node first , it's OK.

@davidjgraph yes, there is no the example. I modify the example to describle my problem. You can refer to my code above.

I'm sure it's a issue, Can you position it?

@paper-play the correct solution is #277


Rename dashboards

The dashboards so fare are more 'traffic' than 'E&M'.
Let's rename them to create some space for us to create dashboards that more explicitly track metrics for E&M such as relevance, repeat engagement etc.

Done


Examples now support iOS 11 and iPhone X landscape mode

Update all examples so they support iOS 11 and iPhoneX landscape mode. They use the new UIView.safeAreaInsets property.
Add method margin(_ directionalInsets: NSDirectionalEdgeInsets)

Codecov Report

Merging #85 into master will decrease coverage by 3.07%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master      #85      +/-   ##
==========================================
- Coverage   99.35%   96.28%   -3.08%
==========================================
  Files          17       29      +12
  Lines        3906     5281    +1375
==========================================
+ Hits         3881     5085    +1204
- Misses         25      196     +171

Impacted Files
Coverage Δ

Sources/PinLayoutImpl.swift
85.63% <ø> (ø)

Sources/PinLayout.swift
100% <ø> (ø)

Tests/MarginsSpec.swift
100% <100%> (ø)
:arrow_up:

Sources/TypesImpl.swift
100% <0%> (ø)

Sources/PinLayout+Filters.swift
0% <0%> (ø)

Sources/PinLayoutGlobals.swift
100% <0%> (ø)

Sources/PinLayoutImpl+Relative.swift
88.05% <0%> (ø)

Sources/PinLayoutImpl+Warning.swift
87.87% <0%> (ø)

Sources/Impl/UnitTests.swift
100% <0%> (ø)

Sources/Percent.swift
100% <0%> (ø)

... and 5 more

Continue to review full report at Codecov.

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


[Release] beta.1

Major aesthetic improvements, mainly responsive design (945daba27c66a1d176d62ef0c555d74c8f1751a2) and support for iOS/Apple devices running WebKit/Safari browser engines (44d409b7c5f862507ad1e58dfea2faf60733c91c)
Several important fixes, including consistent word generation (67954c4fc6f9d353eb61b2b5f2875999ed8412d9) and greater safety period enforcement (ec3d70aa81bf47450e2e7f6ae96684e490621210)
Big updates: dark mode (e1f7fa4aa1192ff70300219179907a9ef85dd213) and combined spellcheck and frequency dictionaries (5111b348a2474ade41a2f01a9d725b880ca68bcc)

@checkly good, waiting on review from @Quantalabs before the official merge

Change of plans, this is being backlogged as @Quantalabs won't be able to review until late next week. Meanwhile, this PR will merge into main since there seem to be no errors, and @Quantalabs will review from 9fc59afbd37d2eb1509f949df6ab83d59efb67b4 (will be tagged as beta.1) in a new discussion.


Sign out from Identity Server 4 using Client (.net framework)

Hi,
I have used Identity Server 4,I have client(.net framework),As we can not use
HttpContext.Authentication.SignOutAsync("Cookies");
HttpContext.Authentication.SignOutAsync("oidc");
Please help me

Not enough info.

Hi,
If we create client using Asp.Net Core then we can sign out using below code
HttpContext.Authentication.SignOutAsync("Cookies");
HttpContext.Authentication.SignOutAsync("oidc");
but i have created Client using .Net Framework so we can not use packages of Asp.Net Core,i need to sign out.
Please help me

You need to do a redirect to the end session endpoint manually.
Check the spec:
https://openid.net/specs/openid-connect-session-1_0.html#RPLogout

Moar docs: https://identityserver4.readthedocs.io/en/release/endpoints/endsession.html


Update Nginx Ingress Documentation

Problem
The Nginx Ingress documentation is incorrect. The example DNS is for Google Cloud. Google Cloud resources and Digital Ocean resources in Terraform are different and require different tf files. It would be nice if the example under Digital Ocean was for Digital Ocean and not for another service with incompatible resources.
I would be willing to fix it, but I haven't figured that out :joy: I am trying to figure out TF and thought I'd do so by using this to project to move my personal stuff to k8s. So far I have this:
resource "digitalocean_record" "example" {

  # DNS record
  name    = "example"
  domain  = "www.example.com"
  type    = "CNAME"
  ttl     = 300
  value   = ???
}```

I am not sure what to put for values. I know the example docs says to use the workers_dns variables and iterate through them, but I can't seem to get that to work.

The docs are correct. Modules have an output that you may use in DNS records. Those DNS records may be hosted by any provider you like, it makes no difference that your cluster resides on DigitalOcean.
Examples show a Google Cloud app DNS record resolving to a cluster on each platform. Picking a favored DNS provider for apps (independent of the platform of the cluster the app happens to reside on today) is the common case.
If you want DNS records hosted by DigitalOcean, just make a DNS record hosted by DigitalOcean via https://www.terraform.io/docs/providers/do/r/record.html. Set the value the same as you see in the example "${module.nemo.workers_dns}." (with or without the "." depends on the platform).


Document /register data and events

For the mobile app, the most important part of the API that isn't documented is the data from the events system.
What does exist is some documentation of the high-level design, and documentation of how to invoke the two central endpoints /register and /events. What's still needed is documentation of:

the data returned by /register. This is a large data structure with lots of different fields and subfields.
all the types of events, and the data they contain.

Among events, the most important single event type is message. There are a couple of examples of that one in the GET .../events doc. Those examples already show that message.display_recipient in such an event is kind of complicated, with a couple of different forms it can take. There are some other fields whose interpretation isn't obvious, too.
Then there are a bunch of other types of events too, of course, many of which are also important. I think the priority goes mostly in order of how frequent they are, because that's the major driver of how likely it'll matter if a client handles them wrong or even just hasn't implemented handling them. So the top of the priority list looks something like:
message
presence
typing
update_message_flags
update_message
delete_message
user_status
realm_user

One piece of this work is happening in #13499, which aims to make available a full set of programmatically generated examples.

@zulipbot claim

@gnprice should I add the descriptions in the /events docs?

This may be a challenging issue for newbies.

There may be some interesting possibilities in using the new mypy feature to type-check events, as it's now possible to discriminate a union of TypedDict's by a Literal, if I understand correctly. I've been thinking of using that in ZT, and we could then port that over to at least the API, or generate it from the server, even?

@zulipbot claim

@hashirsarwar FYI I am doing the openapi documentation of these endpoints for my GSoC project.

Sure @orientor!

https://github.com/zulip/zulip/pull/15899 and it's successors I expect to address the /events part of this, along with improvements to formatting (E.g. https://github.com/zulip/zulip/issues/15967).
Based on the progress so far, I feel good about @orientor being on track to finish this issue during the summer, which would be a huge milestone.

The events part of this was completed via the portion of #16000 that was merged today.  I expect the /register part to be finished in a few days as well.


Jun 28 update breaks pynYNAB

I have the following issues with pynYNAB:
The simple example from the README gives me:
Traceback (most recent call last):   File "/Users/jo/dev/m2ynab/m2ynab/ynab.py", line 8, in <module>     client = nYnabClient(nynabconnection=connection, budgetname='X')   File "/Users/jo/.virtualenvs/m2ynab/lib/python2.7/site-packages/pynYNAB/Client.py", line 20, in __new__     return factory.create_client(passed)   File "/Users/jo/.virtualenvs/m2ynab/lib/python2.7/site-packages/pynYNAB/ClientFactory.py", line 72, in create_client     client_id = connection.user_id AttributeError: 'nYnabConnection' object has no attribute 'user_id'
After changing line 72 to client_id = connection.id, I get
pynYNAB.connection.NYnabConnectionError: Unknown API Error "invalid_session_token" was returned from the API when sending request ({'request_data': '{"user_id": "X", "changed_entities": {}, "device_knowledge_of_server": 0, "ending_device_knowledge": 0, "starting_device_knowledge": 0}', u'operation_name': 'syncCatalogData'})

Which example are you following ? There might be a problem in the example. Live tests on CI are working (https://travis-ci.org/rienafairefr/pynYNAB/).
OK, can reproduce with the first example in /wiki/Examples-of-usage
Coming back with an updated wiki in a sec :-)

OK it was just a missing init_session() , you create the connection, then init it to get the session token set up, then you can use it in a Client 👍
I've updated the wiki /wiki/Examples-of-usage


Fixing Labels issue in INJI App

Details:
Due to a code merging issue, it was noticed that labels that were working properly were broken. This was noticed during the feature demo of the 30th Jan build.
This was fixed by addressing the code merge issue.
Test scope: All the labels in the app should work as before.

@Sujithbn this is a task, qa team will not be verifying/closing the tasks.


Makefile Requires a Git Repo

The githash command in the make file requires the directory that Dscanner is being built in to be a git repo. This means that the source downloads on the releases page cannot be built.

Oh. Right. Github generates those, doesn't it....

You have DUB receipt now, assuming not be an issue anymore i close.


Strange Glitch where Spears and Pups fall off the back but teleport with you into new rooms.

I'm unsure what causes this, especially since I got no logs for it, but if this sounds familiar, perhaps it can be recreated? Was in past Garbage Wastes with Long Legs grabbing at my pups while messing around with invuln, mouse grab, and arti. Something seemed to happen to cause the spears my second player was carrying to start falling off and through the floor but teleporting back onto his back for a millisecond before falling again every time they entered the room. They also could not grab their pup and the pup was stuck in a similar state, having no collision with either of us but teleporting with player 2 whenever they entered a new room and staying stuck in the pipe unless moved by a outside force seemingly unable to move on its own.  I hope the description is enough to explain.

I can't check it out for the next two weeks. I can only give some context for now:

There is a general thing that connects things. Like when you throw a spear at a creature it is connected (stuck).
Backspears are connected like that as well. Besides that the spear itself has a special state (no collision) that needs to go along with it. If a spear simply falls through the floor means that it is in this state but not connected in the general sense.
Since it teleports with you to new rooms the player and spear are still connected. This makes it more confusing.
If it would happen with only slugpups then it would be unrelated to this mod. These are separate things. If it is related then it means that the interaction (like, for example, you can't pick up pups when you carry spears on your back) is involved in the issue.

My best guess is that you had somehow a slugpup and spear on your back at the same time. Can you confirm whether or not that was the case?
You might want to check the section BUG REPORTS. There are additional logs that can in some cases have the necessary information to identify issues. And having steps to reproduce issues make things a lot easier.
Best regard.

I played around with brother long leg and slugpups + backspears but could not reproduce the issue. Letting the slugpup be grabbed before or after carrying it on the back didn't let it fall through the floor.
Let me know if you find consistent steps to reproduce the issue. They are required anyways in order to test if the issue has been fixed. (And they can be used to confirm if this issue is caused by this mod or not.)
I managed to have backspears and a slugpup on the back at the same time (fixed in v2.1.8). But that didn't cause issues for me either.


add libs click and huggingface_hub

./scripts/download_weights.py breaks without these two

i also fixed the error that happens when you run the demo from any directory that is not the demo folder itself


Separate table creations in init.db  \ problem 6; ohtu

reserve spot for problem6; ohtu

SQL queries are separated as a dictionary. This could also be a config file.
(final suggestion; problem 6 - OhTu).
Have a good day!


Color bar for event replies

A new concise display element that shows the state of user replies on an event would be extremly helpful. One idea for such an element is a colored bar somewhere in the list element with 5 colors, one for each voice plus a color for non-replies. Maybe, when the event is expanded (as suggested in #12), the bar could have more colors for sub-voices and even striped parts for maybe-replies.

improvements made in pull request #53, criticality and severity should be decreased now


Installation Opencv path

Installing package into ‘/home/skailasam/R/x86_64-pc-linux-gnu-library/4.4’
(as ‘lib’ is unspecified)

installing source package ‘VoltRon’ ...
** using staged installation
This is pkg-config 1.8.1
Found pkg-config cflags and libs!
Using PKG_CFLAGS=-I/usr/include/opencv4  -D_DATA_PREFIX=/usr
Using PKG_LIBS=-lopencv_stitching -lopencv_alphamat -lopencv_aruco -lopencv_barcode -lopencv_bgsegm -lopencv_bioinspired -lopencv_ccalib -lopencv_cvv -lopencv_dnn_objdetect -lopencv_dnn_superres -lopencv_dpm -lopencv_face -lopencv_freetype -lopencv_fuzzy -lopencv_hdf -lopencv_hfs -lopencv_img_hash -lopencv_intensity_transform -lopencv_line_descriptor -lopencv_mcc -lopencv_quality -lopencv_rapid -lopencv_reg -lopencv_rgbd -lopencv_saliency -lopencv_shape -lopencv_stereo -lopencv_structured_light -lopencv_phase_unwrapping -lopencv_superres -lopencv_optflow -lopencv_surface_matching -lopencv_tracking -lopencv_highgui -lopencv_datasets -lopencv_text -lopencv_plot -lopencv_ml -lopencv_videostab -lopencv_videoio -lopencv_viz -lopencv_wechat_qrcode -lopencv_ximgproc -lopencv_video -lopencv_xobjdetect -lopencv_objdetect -lopencv_calib3d -lopencv_imgcodecs -lopencv_features2d -lopencv_dnn -lopencv_flann -lopencv_xphoto -lopencv_photo -lopencv_imgproc -lopencv_core
Found C++17 compiler: g++
Module <opencv2/xfeatures2d.hpp> has not been found on your system
Make sure to have an opencv4 library installed with the module available
ERROR: configuration failed for package ‘VoltRon’

removing ‘/home/skailasam/R/x86_64-pc-linux-gnu-library/4.4/VoltRon’
Warning message:
In i.p(...) :
installation of package ‘/tmp/RtmpnHVykc/file1c55b352e3af/VoltRon_0.2.0.tar.gz’ had non-zero exit status.
In Ubuntu 24.04 LTS. opencv gets install /usr/local. How to configure installation such that it recognizes the path for opencv

Added actions for ubuntu-24.04 just to be sure, here https://github.com/Artur-man/VoltRon/commit/9f4a1d740c37dbf485dd937b5fea95a8f7af7875

R CMD check on ubuntu-24.04 was implemented in https://github.com/Artur-man/VoltRon/commit/9f4a1d740c37dbf485dd937b5fea95a8f7af7875 and was successful. See here:
https://github.com/Artur-man/VoltRon/actions/runs/12496923097/job/34869111195
OpenCV was built as was shown at https://github.com/BIMSBbioinfo/VoltRon/blob/main/inst/extdata/install_ubuntu.md

Thanks 🙏 works fine in LTS 24 ..


Ke/npm workspaces

use npm workspaces so we can specifically install dependencies only for lf-documentation or only for lf-ui-components.
notes: we can use npm install -w @laserfice/lf-ui-documentation (the name of the package.json) to install only the dependency of lf-documentation, however, this also creates a npm package in node_modules (and the files inside are exactly the same as the repository, so they are not built)
pro: in the future, we can run commands just in the context of the workspace. you can read more about workspace here.
also, installing lf-documentation's dependency will not uninstall lf-ui-components' dependencies. So in the workflow we don't have to worry about orders of executions.

close this PR because we decided to not use workspaces.


Tracking "Explore-Components" deployment work

https://carnegielearning.atlassian.net/browse/CICD-95

@amurphy-cl  Update: I converted the explore-components codebase to an nx-managed workspace and integrated a parent-app. This required a fair number of code updates just to get the linting to pass.
We may still need to do more to pass sonarqube gates in the future, as we have no unit test coverage and have a few code-quality things that it wants addressed, but those shouldn't be blockers do at least get this deployed to dev region.
This currently building appropriately now, we may see it deployed soon hopefully. Then I suppose we'll need to verify that a variant will work with these IFrames in it. I'm not sure what work that will require!

Great! Maybe we can loop @kawcl to help with the variant embedding.

@amurphy-cl FYI This has been deployed to qa-cli.com as a test but Shay is on vacation this week, some changes still need to be merged so we'll have to pick back up later trying to actually see these explore-components in Mathia.

Explore components are now in https://apps.qa-cli.net/explore-components/#/circle_area
Not in staging or prod yet
@amurphy-cl to reach out ljoy for stat work

@amurphy-cl It is in staging now, I was able to run the job Shay set up to deploy to qa / staging / and prod on Jenkins via these instructions https://github.dev.carnegielearning.com/CarnegieLearning/explore-components/blob/master/.jenkins/README.md. So those deployments / builds are largely up to us.


CRM批量上传功能有问题

Raul，你好
CRM批量上传功能有点问题，我在家里的网络可以上传，但是另外3个同事在自己家里的网络就无法批量上传表格，大家都使用chrome浏览器，请帮忙看下这个问题，应该如何解决，谢谢~
https://user-images.githubusercontent.com/74414333/171174583-b7897ec4-4b89-4899-8dd2-869ecadb998f.mp4

The CDN that some of the assets, like Vue and xlxs.js, was using stopped working in the mainland. The functionality broke first for new users, that didn't have the old assets cached, while old users could still use it until the cached versions expired.