Reintroduce functions that were removed with 5.10 (accidentially)

Pull request type
Please check the type of change your PR introduces:

[x] Other (please describe):

Revert function renaming introduce by 17f576b
What is the current behavior?

Issue Number: #671
What is the new behavior?
Bring back C-Api  function that were renamed in 5.10, but mark as deprecated
Does this introduce a breaking change?

[ ] Yes
[x] No

clang-tidy complained about ECALC_API, ECAL_HANDLE etc. not being known. Guess what, it is right 😄
Currently this only compiles because ecalc.h includes the headers
We should include add #include <ecal/ecal_defs.h> to all of those files and also propagate the ´ECAL_HANDLE´ properly, whcih is currently hardcoded in ecalc.h:
https://github.com/continental/ecal/blob/cd65b54e5022f7681232840a083fb9c2b2c3a2b7/ecal/core/include/ecal/ecalc.h#L40
This can however be done in a separate PR, if we don't want those changes on the 5.10 support branch.

Ah ok, yeah I'd prefer to solve this in a separate PR, where I would also remove all the ECALC_API definitions from the function definitions.
And this one can be easily cherry picked to the 5.10 support branch (just without deprecating the functions).

Also I agree that something is not quite right with the includes. My build problems went away after a clean build. Which is never a good sign.


Move IFileSystem/FileSystem

Move IFileSystem/FileSystem -> Microsoft.VisualStudio.IO namespace  …

Utilities has become a grab bag of everything - it's not a great namespace name, and something we avoided using on the BCL.

Updated doc comments for FileSystem/IFileSystem

Rename "FileSystem" -> "Win32FileSystem"  …

This better describes it's implementation.

tag @dotnet/project-system


Set all MapBlocks to mt.Ignore on server change

If implemented there should be an option to disable this since storing the positions could be memory intensive and this isn't an essential feature.

I ran into several server-side segmentation faults while implementing this. This is likely because the client tells the server when it unloads a block and these packets get forwarded to the new server. So far all attempts to fix this have failed.
This fragility is not worth the purely cosmetic purpose so this feature will not be implemented. I will investigate the segmentation faults later and report them upstream if appropriate.


Doesn't work on Windows if launched from a shell with lowercase drive letter

Hey guys,
congrats for the work on storybook, it's blowing my mind and transforming the way I make apps for the better 😄 !
If I run npm run storybook from a shell starting with a lowercase drive letter : c:\Users\dagobert\dev\awesomeApp>, it warns me of conflicts between modules and then crashes with an error 500.
WARNING in C:/Users/dagobert/dev/awesomeApp/~/babel-runtime/~/core-js/library/modules/_to-index.js
There is another module with an equal name when case is ignored.
This can lead to unexpected behavior when compiling on a filesystem with other case-semantic.
Rename module if multiple modules are expected or use equal casing if one module is expected.
[...hundreds of these]

Error: File C:\Users\dagobert\dev\awesomeApp\node_modules\react-native\packager\react-packager\src\Resolver\polyfills\prelude_dev.js not found in any of the roots
    at Fastfs._getAndAssertRoot (C:/Users/dagobert/dev/awesomeApp/node_modules/react-native/packager/react-packager/src/node-haste/fastfs.js:163:21)
    at Fastfs._getFile (C:/Users/dagobert/dev/awesomeApp/node_modules/react-native/packager/react-packager/src/node-haste/fastfs.js:173:25)
    at Fastfs.readFile (C:/Users/dagobert/dev/awesomeApp/node_modules/react-native/packager/react-packager/src/node-haste/fastfs.js:93:23)
    at Polyfill._read (C:/Users/dagobert/dev/awesomeApp/node_modules/react-native/packager/react-packager/src/node-haste/Module.js:120:40)
    at _cache.get (C:/Users/dagobert/dev/awesomeApp/node_modules/react-native/packager/react-packager/src/node-haste/Module.js:139:16)
    at Cache.get (C:/Users/dagobert/dev/awesomeApp/node_modules/react-native/packager/react-packager/src/node-haste/Cache/index.js:82:35)
    at Polyfill.read (C:/Users/dagobert/dev/awesomeApp/node_modules/react-native/packager/react-packager/src/node-haste/Module.js:134:24)
    at Bundler._toModuleTransport (C:/Users/dagobert/dev/awesomeApp/node_modules/react-native/packager/react-packager/src/Bundler/index.js:577:14)
    at module (C:/Users/dagobert/dev/awesomeApp/node_modules/react-native/packager/react-packager/src/Bundler/index.js:412:14)
    at Array.map (native)

It took me two hours to realize that when I'm launching the command from this location C:\Users\dagobert\dev\awesomeApp> instead, it works as intended.
The conflict comes from the uppercase / lowercase drive letter on windows, as both work for navigating in the terminal.
I did some digging and found these issues in other npm modules
https://github.com/webpack/webpack/issues/2694
And this one that might be the solution (26 Sept by Allov)
https://github.com/webpack/webpack/issues/2815#issuecomment-249612774
It seems like it can be fixed by forcing the script to cd to the same directory(but starting with an uppercase letter) before running any other command.
If you let me know where the script happens I can try to fix it.

Please open an issue over at the new repo if you're still experiencing this issue, thanks!


feat(matches): Implement Issue Matching ON creation of new component instances

Description
This PR requests handles the business logic whenever a new ComponentInstance is created.
For this particular event type IssueMatches will be created depending on the componentInstanceID, issueId and issueVariants.
What type of PR is this? (check all applicable)

[x] 🍕 Feature
[ ] 🐛 Bug Fix
[ ] 📝 Documentation Update
[ ] 🎨 Style
[ ] 🧑💻 Code Refactor
[ ] 🔥 Performance Improvements
[x] ✅ Test
[ ] 🤖 Build
[ ] 🔁 CI
[ ] 📦 Chore (Release)
[ ] ⏩ Revert

Related Tickets & Documents

Related Issue #168

Added tests?

[x] 👍 yes
[ ] 🙅 no, because they aren't needed
[ ] 🙋 no, because I need help
[ ] Separate ticket for tests # (issue/pr)

Added tests in

issue_match_handler_test.go

especially the ones in the Ginkgo context "OnComponentInstanceCreate"

Added to documentation?

[ ] 📜 README.md
[ ] 🤝 Documentation pages updated
[x] 🙅 no documentation needed
[ ] (if applicable) generated OpenAPI docs for CRD changes

Checklist

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes

Split up into:
PoC scope + non-PoC scope.
PoC scope - run issue match algorithm when comp instances are created

Creation of new comp instances for issue matching
Creating default repository for new service

The actual work package is:
on the creation of a component instance, we perform the issue matching:

It fetches the services ( through relation to it) and issue repositories (through component version) related to the component instance
identifies the issue repositories with the highest priority
fetches the issue variants for these repositories and the given issues
creates an issueMatch for each of this issues(of the variants) for this component instance

@dorneanu
The intermediate implementation draft can be found here:
https://github.com/cloudoperators/heureka/blob/basic_matching/internal/app/issue_match/issue_match_handler_events.go#L93-L286

better descriptions and maybe also acceptance criteria

I agree, but that is also an additional effort to take, but maybe it's worth it.

I definitely need to improve the unit tests here. So this PR is not ready yet.

What happens if a service owner changes the priority of an issue repository?

This event should trigger a new issue matching?
What happens with the existing issue matching that considered the old issue repository?

What happens if a service owner changes the priority of an issue repository?
This event should trigger a new issue matching?
What happens with the existing issue matching that considered the old issue repository?

We agreed for the PoC to handle only this one "issue matching" type and not the subsequent updates etc.
On such a change as you described there is no new issue match but rather a change on the existing issue match happening no?


Question: mounting host filesystem?

Maybe an obvious questions, but I don't have a lot of experience using Docker for these types of applications. I assume it must be possible to mount the host filesystem inside the Docker containers (e.g. mounting a media library for Plex to consume), but I'm not exactly sure how to do this given the lack of documentation :grin:
Thanks for taking the time to create these Dockerfiles by the way! :cupid:

Mounting is pretty straight forward if you're on Linux. just run  docker run -v /your/media:/data -v /your/config:/config <name of the image>. If you're on windows or osx it can get a little bit more tricky, but it's pretty well covered in the docs

Seems way simpler than I thought. Thanks for your help!


[Bug]: Coolify installation not completed properly  |  Error: Command failed: docker inspect coolify-proxy --format '{{json .Config.Cmd}}' Error: No such object: coolify-proxy

Is there an existing issue for this?

[X] I have searched the existing issues

Description
After installing coolify on a new Ubuntu 22.04 based VPS using the wget -q https://get.coollabs.io/coolify/install.sh -O install.sh; sudo bash ./install.sh command, (coolify v2.9.11), accessing the http://my-vps-ip-address:3000 hangs (i.e. in the browser --> This site can’t be reached x.x.x.x took too long to respond) and does not show up the coolify dashboard.

Running sudo docker ps -a shows the following:

CONTAINER ID   IMAGE                       COMMAND                  CREATED       STATUS                   PORTS                                       NAMES
516b9f77653e   coollabsio/coolify:2.9.11   "docker-entrypoint.s…"   2 hours ago   Up 2 hours               <IP_ADDRESS>:3000->3000/tcp, :::3000->3000/tcp   coolify
c37ffd77063c   redis:6.2-alpine            "docker-entrypoint.s…"   2 hours ago   Up 2 hours               6379/tcp                                    coolify-redis
6cb752e8fda2   coollabsio/coolify:2.9.11   "docker-entrypoint.s…"   2 hours ago   Exited (0) 2 hours ago                                               pensive_keldysh

Running sudo docker logs coolify shows the following

><EMAIL_ADDRESS>start /app
> npx prisma migrate deploy && npx prisma generate && npx prisma db seed && node build/index.js

Prisma schema loaded from prisma/schema.prisma
Datasource "db": SQLite database "prod.db" at "file:../db/prod.db"

SQLite database prod.db created at file:../db/prod.db

33 migrations found in prisma/migrations

Applying migration `20220131142425_init`
Applying migration `20220210104005_redis_aol`
Applying migration `20220212142309_unique_secret_by_application`
Applying migration `20220217211304_dualcerts`
Applying migration `20220219231255_prmr_secrets`
Applying migration `20220220141136_public_portrange`
Applying migration `20220301101928_proxyhash`
Applying migration `20220304141408_service_secrets`
Applying migration `20220311213422_autodeploy`
Applying migration `20220320141424_phpmodules`
Applying migration `20220322135800_persistent_storage`
Applying migration `20220327180323_ghost`
Applying migration `20220402135305_python`
Applying migration `20220402210645_meilisearch`
Applying migration `20220405151428_wordpress_sftp`
Applying migration `20220407220809_unique_storage_fix`
Applying migration `20220408070805_added_expose_port`
Applying migration `20220418214843_persistent_storage_services`
Applying migration `20220419203408_multiply_dockerfile_locations`
Applying migration `20220420202031_deno_configurations`
Applying migration `20220420210057_branch_for_builds`
Applying migration `20220425071132_umami`
Applying migration `20220425075326_auto_update_coolify`
Applying migration `20220426125053_select_base_image`
Applying migration `20220427133656_hasura`
Applying migration `20220429202516_fider`
Applying migration `20220429214112_fider_correction`
Applying migration `20220430111953_ssl_dns_check_settings`
Applying migration `20220430124553_expose_port_for_services`
Applying migration `20220509130501_custom_plausible_script`
Applying migration `20220510081125_custom_wordpress_db`
Applying migration `20220517081328_traefik`
Applying migration `20220519095648_minio_apifqdn`

The following migrations have been applied:

migrations/
  └─<PHONE_NUMBER>2425_init/
    └─ migration.sql
  └─<PHONE_NUMBER>4005_redis_aol/
    └─ migration.sql
  └─<PHONE_NUMBER>2309_unique_secret_by_application/
    └─ migration.sql
  └─<PHONE_NUMBER>1304_dualcerts/
    └─ migration.sql
  └─<PHONE_NUMBER>1255_prmr_secrets/
    └─ migration.sql
  └─<PHONE_NUMBER>1136_public_portrange/
    └─ migration.sql
  └─<PHONE_NUMBER>1928_proxyhash/
    └─ migration.sql
  └─<PHONE_NUMBER>1408_service_secrets/
    └─ migration.sql
  └─<PHONE_NUMBER>3422_autodeploy/
    └─ migration.sql
  └─<PHONE_NUMBER>1424_phpmodules/
    └─ migration.sql
  └─<PHONE_NUMBER>5800_persistent_storage/
    └─ migration.sql
  └─<PHONE_NUMBER>0323_ghost/
    └─ migration.sql
  └─<PHONE_NUMBER>5305_python/
    └─ migration.sql
  └─<PHONE_NUMBER>0645_meilisearch/
    └─ migration.sql
  └─<PHONE_NUMBER>1428_wordpress_sftp/
    └─ migration.sql
  └─<PHONE_NUMBER>0809_unique_storage_fix/
    └─ migration.sql
  └─<PHONE_NUMBER>0805_added_expose_port/
    └─ migration.sql
  └─<PHONE_NUMBER>4843_persistent_storage_services/
    └─ migration.sql
  └─<PHONE_NUMBER>3408_multiply_dockerfile_locations/
    └─ migration.sql
  └─<PHONE_NUMBER>2031_deno_configurations/
    └─ migration.sql
  └─<PHONE_NUMBER>0057_branch_for_builds/
    └─ migration.sql
  └─<PHONE_NUMBER>1132_umami/
    └─ migration.sql
  └─<PHONE_NUMBER>5326_auto_update_coolify/
    └─ migration.sql
  └─<PHONE_NUMBER>5053_select_base_image/
    └─ migration.sql
  └─<PHONE_NUMBER>3656_hasura/
    └─ migration.sql
  └─<PHONE_NUMBER>2516_fider/
    └─ migration.sql
  └─<PHONE_NUMBER>4112_fider_correction/
    └─ migration.sql
  └─<PHONE_NUMBER>1953_ssl_dns_check_settings/
    └─ migration.sql
  └─<PHONE_NUMBER>4553_expose_port_for_services/
    └─ migration.sql
  └─<PHONE_NUMBER>0501_custom_plausible_script/
    └─ migration.sql
  └─<PHONE_NUMBER>1125_custom_wordpress_db/
    └─ migration.sql
  └─<PHONE_NUMBER>1328_traefik/
    └─ migration.sql
  └─<PHONE_NUMBER>5648_minio_apifqdn/
    └─ migration.sql

All migrations have been successfully applied.
npm notice
npm notice New minor version of npm available! 8.5.0 -> 8.13.1
npm notice Changelog: <https://github.com/npm/cli/releases/tag/v8.13.1>
npm notice Run `npm install -g<EMAIL_ADDRESS>to update!
npm notice
Prisma schema loaded from prisma/schema.prisma

✔ Generated Prisma Client (3.11.1 | binary) to ./node_modules/.pnpm/@prisma+client@3.11.1_prisma@3.11.1/node_modules/@prisma/client in 2.77s
You can now start using Prisma Client in your code. Reference: https://pris.ly/d/client

import { PrismaClient } from '@prisma/client'
const prisma = new PrismaClient()

┌─────────────────────────────────────────────────────────┐
│  Update available 3.11.1 -> 3.15.2                      │
│  Run the following to update                            │
│    npm i --save-dev prisma@latest                       │
│    npm i @prisma/client@latest                          │
└─────────────────────────────────────────────────────────┘
Running seed command `node prisma/seed.cjs` ...
Error: Command failed: docker inspect coolify-proxy --format '{{json .Config.Cmd}}'
Error: No such object: coolify-proxy

    at ChildProcess.exithandler (node:child_process:399:12)
    at ChildProcess.emit (node:events:526:28)
    at maybeClose (node:internal/child_process:1092:16)
    at Process.ChildProcess._handle.onexit (node:internal/child_process:302:5) {
  killed: false,
  code: 1,
  signal: null,
  cmd: "docker inspect coolify-proxy --format '{{json .Config.Cmd}}'",
  stdout: '\n',
  stderr: 'Error: No such object: coolify-proxy\n'
}

🌱  The seed command has been executed.
Listening on <IP_ADDRESS>:3000

Any thoughts on why this happens?
Steps To Reproduce
On a fresh Ubuntu minimal 22.04 Oracle Cloud VPS

mkdir coolify && cd coolify/
wget -q https://get.coollabs.io/coolify/install.sh -O install.sh; sudo bash ./install.sh

Version
2.9.11

Thanks for the details. Let me reproduce the same steps on Digital Ocean droplet.

This happens when the proxy hasn't started yet and docker couldn't find it.
I was able to run this locally on a Ubuntu server without issues, but starting the project locally on a WSL2 VM I encountered this problem, but it did not affect the deployment of the project.

Except for the error in the console, does Coolify work properly? I tried to reproduce the issue but could not.

@andrasbacsai
No. After the install script finishes, accessing the URL http://my-vps-ip-address:3000 hangs, (i.e. in the browser --> This site can’t be reached. x.x.x.x took too long to respond) and does not show the Coolify dashboard.
Some more information about this which might be important to debug the issue:

It is using the Free Tier from Oracle Cloud.  (Feel free to spin one up to test this out at https://www.oracle.com/cloud/sign-in.html)
The VPS specs are 2 AMD based cores and 1GB ram.  (Is 1GB of ram a hard blocker?)
The Image that is used is Ubuntu 22.04 Minimal.  (Maybe there is a package missing that is essential for the installation?)

Is 1GB of ram a hard blocker?

I know I had issues with docker in the past trying to create / pull images or build images with 1 GB of ram. I at minimum use 2 gigs now (note this was with a different deployment system, not coolify)
Maybe try on a VM with 2 gigs minimum?
If that fails, then maybe 1 gig with ubuntu 20? I've been running 2 gigs with ubuntu 20.x so far without issues.

Silly question. Is the port allowed/opened in Oracle’s firewall?

@andrasbacsai
Thanks for mentioning this, indeed the main reason was that the created network security group was not automatically attached to the VPS. Sorry for the inconvenience, I am closing the ticket :)

I'm glad my tip was right! 😅

Would have never guessed it myself 😀

i have the same issues with aws ec2 fresh instance, checked whether the ports are available but ufw is disabled.when i comment sudo docker logs coolify showing me same error

@dennisaga
AWS uses a similar approach to Oracle, where there is firewall configuration not only on the VPS (ec2) level, but also on the subnet/network level. I believe that you would have to change the Security Group used by your ec2 instance.
Hope this helps.


Plot history optimization

When VS receives a new plot, it queries r-host to get the status of the plot history and update the VS command UI.
We could have the message that sends the plot to VS include the status of the plot history inside the message, and avoid the additional round-trip.

I had to do this as part of the move of plots to R components .


Fix #14 HA config cluster_addr and api_addr

This PR implements the same changes as this PR in terraform-aws-vault which addresses hashicorp/vault#4383. This also fixes #14.
I have tested this change and it fixes the issue.
A future PR should also merge in this PR from terraform-aws-vault which allows for a custom API address in order to support HA client redirection with Vault behind a load balancer.
(Sorry for all of the whitespace changes my editor made. If thats an issue I can fix them. We might want to add an editorconfig.org file to this project so that contributors can follow the same style guidelines.)

Thank you for this PR! I ran into this issue myself. I just manually validated correctness so merging now.


Account for new Sanoid dependency in docs.

Sanoid now requires an additional perl package that wasn't installed by default on Ubuntu 18.04 server.
-Dale

Thanks for the doc update :+1:


Fix image for wip/at-home/Whats-new-wip-at-home

Some images in Whats-new-wip-at-home.md is broken due to file name with space and some indent. Also occurs in https://docs.microsoft.com/en-us/windows-insider/at-home/whats-new-wip-at-home too.
Fixing these:

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


0.2.0

@aidenprice Thank you for your pull request. Could you please send PR to this repository? (Because this examples repository is synced with the examples directory of pipe repository.)
https://github.com/pipe-cd/pipe/tree/master/examples

@aidenprice Thank you for your pull request. Could you please send PR to this repository? (Because this examples repository is synced with the examples directory of pipe repository.)
https://github.com/pipe-cd/pipe/tree/master/examples

Sorry, I made this PR in error while following the QuickStart tutorial. Clicked ahead too quickly.
If I could delete the PR entirely I would.
Sorry again and thanks.

Sorry, I made this PR in error while following the QuickStart tutorial. Clicked ahead too quickly.
If I could delete the PR entirely I would.
Sorry again and thanks.

No problem. Thank you for your interest and trying PipeCD. 👍

No problem. Thank you for your interest and trying PipeCD. 👍

You're welcome, and I'm actually really interested in PipeCD. Hoping to use it for Kubernetes clusters on site and one day on the edge.

You're welcome, and I'm actually really interested in PipeCD. Hoping to use it for Kubernetes clusters on site and one day on the edge.

Thank you for your kind words. ❤
Please raise issues in the pipe repository if you want any help from PipeCD team.
Thanks.

Thank you for your kind words. ❤
Please raise issues in the pipe repository if you want any help from PipeCD team.
Thanks.


Rename build script to prepare

Enables Umzug to be installed from a Git repository. From the npm documentation:

If the package being installed contains a prepare script, its dependencies and devDependencies will be installed, and the prepare script will be run, before the package is packaged and installed.

Hello! I see you are a first-time contributor, thank you for taking the time to help Sequelize! :)


xosview2: init at 2.2.2

Motivation for this change
xosview is a handy utility that I wanted to have a nix package for.
http://xosview.sourceforge.net/
This program does not yet support all of the possible libraries it can be compiled to use. From the README:

The configure script will attempt to look for the xpm library to load
background images with.  It will also look for the Xft library for freetype2
font support.  If it finds the SM/ICE libraries it will configure support
for working as a session client.

Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option build-use-sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[x] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

@vyp Is there a way to manually trigger the build? Seems like it's not starting up again.

@SeanZicari I'm not sure if there's a way actually. :/

What is the next step then? How would I go about having the package validated?

Well in these cases you could probably just redo a commit and force push it again, so it would probably trigger a new ci build. Or in this particular case, you could also squash all those extra 'fixup' commits into one, which would probably trigger it too (and cause it'd probably be nice to squash them anyway). But of course doing these just to trigger ci builds is not ideal so I wouldn't consider them actual solutions to you question.
I should make it clear that I don't have commit access to the repo, so I can't merge this, I just wanted to help with reviewing simple issues to the PR. Also sometimes the CI builds fail due to unrelated reasons, but the packages still get accepted so I don't think it's a hard requirement to have the checks pass, but of course it is helpful to see it. FWIW to anyone reading, the two linux builds seems to pass and it looks like the mac one just seems to have been hung indefinitely somehow, and in addition travis has decided to not start any new builds, because the latest one got 'autocancelled'.

Thanks! You’ve been very helpful.

@GrahamcOfBorg What does that error mean? What do I need to resolve?

hmm not sure, I'll check in to it. In the mean time, note you should add it to all-packages.nix, making it possible to use :)

@grahamc Ah, thank you. I did have it in all-packages.nix at one point, but that must have somehow gotten lost when I was squashing commits. I'll get that done.

@grahamc I've added the package to all-packages.nix. Just waiting for the build process to complete successfully.

@SeanZicari thank you for your contribution and your patience!

Thanks, @peterhoeg, but sadly I've had some trouble getting all the changes in. That last .net removal didn't end up making it in due to some bad merges. I apologize - can I fix and push the corrections?

I was also missing myself as a package maintainer - that got left off while I was squashing some previous commits.

Open a new PR and ping me in it.

Will do - working on it now.

The maintainer part is OK.


Snakemake does not recognize input/output elements as separate

Dear all,
I used Snakemake before, and now I wanted to return. To my surprise I am unable to run the simplest toy example of functionality (basically copied-pasted from tutorial). I provide 2 files as an input as such:
rule NAME:
	input: "data_q/deblur_FeatureTable[Frequency].qza", "data_q/deblur_singletons_FeatureTable[Frequency].qza"
	output: "data_q/filtered_deblur_FeatureTable[Frequency].qza", "data_q/filtered_deblur_singletons_FeatureTable[Frequency].qza"
	shell: "qiime feature-table filter-features --i-table {input} --p-min-frequency 10 --p-min-samples 11 --o-filtered-table {output}"

run command:
snakemake -j 3 --printshellcmds
... and snakemake in turn just shoves both inputs one after the other into single command as such:
Error in rule NAME:
    jobid: 6
    output: data_q/filtered_deblur_FeatureTable[Frequency].qza, data_q/filtered_deblur_singletons_FeatureTable[Frequency].qza
    shell:
        qiime feature-table filter-features --i-table data_q/deblur_FeatureTable[Frequency].qza data_q/deblur_singletons_FeatureTable[Frequency].qza --p-min-frequency 10 --p-min-samples 11 --o-filtered-table data_q/filtered_deblur_FeatureTable[Frequency].qza data_q/filtered_deblur_singletons_FeatureTable[Frequency].qza
        (one of the commands exited with non-zero exit code; note that snakemake uses bash strict mode!)

I am dumbfounded. Would You be so kind and help?

Hi @AdrianS85,
Can you explain a bit more about what you expected Snakemake to do in this situation? For example, what shell command did you expect it to construct?

Dear Arya,
Sorry for being unclear.
I was certain that my rule was suppose to produce 2 calls (one for each of two values in the input/output):
qiime feature-table filter-features --i-table data_q/deblur_FeatureTable[Frequency].qza --o-filtered-table data_q/filtered_deblur_FeatureTable[Frequency].qza

qiime feature-table filter-features --i-table data_q/deblur_singletons_FeatureTable[Frequency].qza --o-filtered-table data_q/filtered_deblur_singletons_FeatureTable[Frequency].qza

instead of single call with both my input/output values within it, like so:
qiime feature-table filter-features --i-table data_q/deblur_FeatureTable[Frequency].qza data_q/deblur_singletons_FeatureTable[Frequency].qza --o-filtered-table data_q/filtered_deblur_FeatureTable[Frequency].qza data_q/filtered_deblur_singletons_FeatureTable[Frequency].qza
BTW: Now that I look at my post, Im not sure if this a right place to ask such questions. If not - please feel free to delete my post - I will re-post it to stack exchange. Just accept my apologies - I never before engaged in conversations on github and am unsure of social convensions.

In general, "input" and "output" fields will resolve into a list or dictionary and you can access elements from them in a similar way to how you would access any list or dictionary element. To get the behavior you are after, you would write the following rule:
rule NAME:
	input:
            "data_q/deblur_FeatureTable[Frequency].qza",
            "data_q/deblur_singletons_FeatureTable[Frequency].qza"
	output:
            "data_q/filtered_deblur_FeatureTable[Frequency].qza",
            "data_q/filtered_deblur_singletons_FeatureTable[Frequency].qza"
	shell:
            """
            qiime feature-table filter-features --i-table {input[0]} --p-min-frequency 10 --p-min-samples 11 --o-filtered-table {output[0]}
             qiime feature-table filter-features --i-table {input[1]} --p-min-frequency 10 --p-min-samples 11 --o-filtered-table {output[1]}
            """

I'm not sure what you are trying to do with the weird "[Frequency]" thing. As far as I can tell, those brackets don't mean anything and you are just going to get a weird filename.
It may also be worth noting that your architecture here is going to make parrelel exeuction of the two commands impossible. You should probably be using wildcards to run both of your "qiime" jobs which would look something like....
rule NAME:
	input:
            "data_q/{basename}_FeatureTable[Frequency].qza",
	output:
            "data_q/filtered_{basename}_FeatureTable[Frequency].qza",
	shell:
            """
            qiime feature-table filter-features --i-table {input} --p-min-frequency 10 --p-min-samples 11 --o-filtered-table {output}
            """

...For more infromation on wildcards and rule writing, check out the docs https://snakemake.readthedocs.io/en/stable/snakefiles/rules.html


GET /products to include productTag ?

ProductTags are not associated with Products in the master data for Products. Should it be included? 1 or multiple?

add a list of productTags to the Product.  (to be used in Masterdata, not necessary in offers)


[WinUI]C++ create and reference a WinUI 3 Component library project failed with MIDL5157 MIDL1001 MIDL2011errors and MIDL2214 warning

Describe the bug
C++ Create and reference a WinUI 3 Component library failed with the below errors, please download the repro project and build this project.
MyCppDesktopProject (2).zip

Severity
Code
Description
Project

Error
MIDL5157
[msg]An invalid   winmd file was specified for import [context]:   D:\Users\v-liliyatang\source\repos\MyCppDesktop\x64\Debug\ThermometerWRC\ThermometerWRC.winmd
MyCppDesktop

Error
MIDL1001
[msg]cannot open   input file [context]Error opening or processing WinMD   d:\users\v-liliyatang\source\repos\mycppdesktop\x64\debug\thermometerwrc\thermometerwrc.winmd.   HRESULT 0x80070002.
MyCppDesktop

Warning
MIDL2214
[msg]semantic check   incomplete due to previous errors
MyCppDesktop

Error
MIDL2011
[msg]unresolved type   declaration [context]: Microsoft.UI.Xaml.Controls.UserControl [ RuntimeClass   'ThermometerWRC.UserControl'  ]
ThermometerWRC

Steps to reproduce the bug
Step1:Create a functional Windows Runtime component (WinUI 3 Component Library)
1.Create a Windows Runtime Component (C++/WinRT) project, and name it ThermometerWRC (for "thermometer Windows Runtime component"). Make sure that Place solution and project in the same directory is unchecked.
2.Don't build the project yet, and make sure the version of the Microsoft.Windows.CppWinRT package is updating to a more recent version(>2.0.22*).
3.In Solution Explorer, rename "Class.idl" file to "Thermometer.idl".
4.Replace the contents of "Thermometer.idl" with the listing below:
// Thermometer.idl
namespace ThermometerWRC
{
    runtimeclass Thermometer
    {
        Thermometer();
        void AdjustTemperature(Single deltaFahrenheit);
    };
}

5.Save the file. Build the project.
6.Right-click the project node and click "Open Folder in File Explorer".
7.Copy the stub files "Thermometer.h" and "Thermometer.cpp" from the folder
\ThermometerWRC\ThermometerWRC\Generated Files\sources\ and paste into the folder that contains your project files, which is \ThermometerWRC\ThermometerWRC.(Replace the existing files)
8.Open "Thermometer.h" in VS, replace the content with below code:
// Thermometer.h
#pragma once
#include "Thermometer.g.h"
namespace winrt::ThermometerWRC::implementation
{
    struct Thermometer : ThermometerT<Thermometer>
    {
        Thermometer() = default;

        void AdjustTemperature(float deltaFahrenheit);
    private:
        float m_temperatureFahrenheit{ 0.f };
    };
}
namespace winrt::ThermometerWRC::factory_implementation
{
    struct Thermometer :ThermometerT<Thermometer,implementation::Thermometer>
    {
    };
}

9.Open "Thermometer.cpp" in VS, replace the content with below code:
// Thermometer.cpp
#include "pch.h"
#include "Thermometer.h"
#include "Thermometer.g.cpp"

namespace winrt::ThermometerWRC::implementation
{
   void Thermometer::AdjustTemperature(float deltaFahrenheit)
   {
       m_temperatureFahrenheit += deltaFahrenheit;
   }
}

Save and build the project.

Step2: Create new C++ Desktop Project
Create a C++ Blank App, Packaged (WinUI3 in Desktop) and name the project MyCppDesktopProject.
Step3: Create a functional Windows Runtime component (WinUI 3 Component Library)

Open the MyCppDesktopProject solution file in visual studio
Right click on the solution node in the solution explorer and select Add->Existing Project
Select ThermometerWRC.vcxproj, Note: make sure the version of the Microsoft.Windows.CppWinRT package is updating to a more recent version(>2.0.22*).
Right click on the MyCppDesktopProject project node and select Add->Reference
Click the checkbox for ThermometerWRC and click OK. Your MyCppDesktopProject.vcxproj should have a reference to the ThermometerWRC.vcxproj file.
Right click on the ThermometerWRC project node and select Add-> New Item
Select User Control (WinUI 3) and press OK.
Build the solution. It won't successfully build but that's fine.
Replace the existing my_ButtonClick method in UserControl.xaml.cpp with the following

void UserControl::myButton_Click(IInspectable const&, RoutedEventArgs const&)
   {
       myButton().Content(box_value(L"Clicked"));
       ThermometerWRC::Thermometer t;
       t.AdjustTemperature(20);
   }

In the pch.h file under the MyCppDesktopProject Node, add the following line:
#include <winrt/ThermometerWRC.h>
Replace the contents of MainWindow.xaml with the following

<Window
    x:Class="MyCppDesktopProject.MainWindow"
    xmlns="http://schemas.microsoft.com/winfx/2006/xaml/presentation"
    xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml"
    xmlns:local="using:MyCppDesktopProject"
    xmlns:d="http://schemas.microsoft.com/expression/blend/2008"
    xmlns:mc="http://schemas.openxmlformats.org/markup-compatibility/2006"
    mc:Ignorable="d"
    xmlns:control="using:ThermometerWRC">
    <Grid>
        <control:UserControl/>
    </Grid>
</Window>

Delete the myButton_click function from MainWindow.xaml.cpp
Build and Run the Solution (F5)

Expected behavior
Project can run successfully and no error message in Error List
Screenshots

NuGet package version
WinUI 3 - Windows App SDK 1.5.5: 1.5.240627000
Windows version
Windows 11 (22H2): Build 22621, Windows 10 (21H2): Build 19044
Additional context
It didn't repro on rel. d17.12-35209.166 (WindowsAppSDK version: 1.5.240311000 for C++)

Do not even try to rename an idl file after it is created. All the internal file references will go wrong. Rename the idl when you create it, not after.


chore: the new black is the new black

It seems that black has finally turned stable, which also brings the usual assortment of formatting changes.

They seem reasonable enough!


update to 1.19.4; fixed abstract class; added version as var

Will resolve #23

can confirm gradle build creates a usable jar for 1.19.4.
nit: the correct URL for fabric versions is https://fabricmc.net/develop/

tested it too. Plugin works on 1.19.4 when built from this PR


SSL Certification error

Description
I've built a bot in python 3.7 using a virtual engine, when I come to run the code I get this error:
/Users/sophie/Dropbox/Programming/gallagherbot/lib/python3.7/site-packages/urllib3/connectionpool.py:857: InsecureRequestWarning: Unverified HTTPS request is being made. Adding certificate verification is strongly advised. See: https://urllib3.readthedocs.io/en/latest/advanced-usage.html#ssl-warnings
  InsecureRequestWarning)
Traceback (most recent call last):
  File "/Users/sophie/Dropbox/Programming/gallagherbot/lib/python3.7/site-packages/slackclient/server.py", line 179, in connect_slack_websocket
    http_proxy_auth=proxy_auth)
  File "/Users/sophie/Dropbox/Programming/gallagherbot/lib/python3.7/site-packages/websocket/_core.py", line 494, in create_connection
    websock.connect(url, **options)
  File "/Users/sophie/Dropbox/Programming/gallagherbot/lib/python3.7/site-packages/websocket/_core.py", line 217, in connect
    options.pop('socket', None))
  File "/Users/sophie/Dropbox/Programming/gallagherbot/lib/python3.7/site-packages/websocket/_http.py", line 126, in connect
    sock = _ssl_socket(sock, options.sslopt, hostname)
  File "/Users/sophie/Dropbox/Programming/gallagherbot/lib/python3.7/site-packages/websocket/_http.py", line 253, in _ssl_socket
    sock = _wrap_sni_socket(sock, sslopt, hostname, check_hostname)
  File "/Users/sophie/Dropbox/Programming/gallagherbot/lib/python3.7/site-packages/websocket/_http.py", line 232, in _wrap_sni_socket
    server_hostname=hostname,
  File "/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/ssl.py", line 412, in wrap_socket
    session=session
  File "/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/ssl.py", line 850, in _create
    self.do_handshake()
  File "/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/ssl.py", line 1108, in do_handshake
    self._sslobj.do_handshake()
ssl.SSLCertVerificationError: [SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: unable to get local issuer certificate (_ssl.c:1045)

During handling of the above exception, another exception occurred:
Traceback (most recent call last):
  File "/Users/sophie/Dropbox/Programming/gallagherbot/lib/python3.7/site-packages/slackclient/client.py", line 52, in rtm_connect
    self.server.rtm_connect(use_rtm_start=with_team_state, **kwargs)
  File "/Users/sophie/Dropbox/Programming/gallagherbot/lib/python3.7/site-packages/slackclient/server.py", line 147, in rtm_connect
    self.connect_slack_websocket(self.ws_url)
  File "/Users/sophie/Dropbox/Programming/gallagherbot/lib/python3.7/site-packages/slackclient/server.py", line 186, in connect_slack_websocket
    raise SlackConnectionError(message=str(e))
slackclient.server.SlackConnectionError: [SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: unable to get local issuer certificate (_ssl.c:1045)

Connection failed. Exception traceback printed above.
This seems similar to this: https://github.com/slackapi/python-slackclient/issues/325 but I tried the User's suggestion and it hasn't changed anything.
I have the bot's 'Bot User OAuth Access Token' stored as a variable which I refer to in the code
Any suggestions?
Describe your issue here.
What type of issue is this? (place an x in one of the [ ])

[x ] bug
[ ] enhancement (feature request)
[ ] question
[ ] documentation related
[ ] testing related
[ ] discussion

Requirements (place an x in each of the [ ])

[x ] I've read and understood the Contributing guidelines and have done my best effort to follow them.
[x ] I've read and agree to the Code of Conduct.
[x ] I've searched for any related issues and avoided creating a duplicate issue.

Bug Report
Filling out the following details about bugs will help us solve your issue sooner.
Reproducible in:
slackclient version:
python version: 3.7
OS version(s):
Steps to reproduce:

Expected result:
What you expected to happen
Actual result:
What actually happened
Attachments:
Logs, screenshots, screencast, sample project, funny gif, etc.

I have run into the same exact problem today. I fixed it by downgrading the websocket-client library to 0.47.0

Are you both using a proxy configuration? If so, it may be that your proxy has a certificate which doesn't have a root in the CA bundle shipping with this package.

0.48.0 of websocket-client appears to have removed the CA bundle actually which I think the problem is. I am on a Mac, I have not had the chance to test this on another OS to see if the load_default_certs for Mac is the problem.

@KoalaTea wow, you're right, and also that's no great.
it looks like https://github.com/websocket-client/websocket-client/issues/413 is the issue where this decision was made. i hope the maintainers of that package will consider reversing this decision. for example, the requests package depends on certifi to provide a CA bundle, which seems like a favorable approach to solve the staleness problem. the current approach is untested for py2 and depends on some very specific system configuration to be done correctly (which seems to be misconfigured at least often enough for a couple users of this package).

Thanks @KoalaTea and @aoberoi for figuring this out, I'm new to programming - so this is awesome!

Also ran into this issue, not sure if work is being done to resolve this or should I just implement a hotfix to my own proj by downgrading the websocket version.

For those of you affected, I think it would be helpful for you to express support of proposed change linked above. The easiest way would be a thumbsup reaction.

so I am still running into the same issue here, I am new to python and have tried to downgrade the websocket-client library to 0.47.0, but it still returns the same error!
any tips on how to finish get around this?

I named possible solutions in https://github.com/websocket-client/websocket-client/issues/451#issuecomment-417918570 if you care to try.

Hey,
I just ran into a similiar problem in another project and doing as you linked fixed the issue.
"./Applications/Python\ x.x/Install\ Certificates.command" caused it to work. on versions above 0.47.0

There are a couple of things that can be done: 1. Downgrading the websocket-client library to 0.47.0 2. Or, download the certificate (wget https://www.tbs-certificats.com/issuerdata/DigiCertGlobalRootCA.crt), then set the environment variable export WEBSOCKET_CLIENT_CA_BUNDLE=DigiCertGlobalRootCA.crt

This is sloved my problem..
cd /Applications/Python\ 3.7/
./Install\ Certificates.command

This is sloved my problem..
cd /Applications/Python\ 3.7/
./Install\ Certificates.command

Worked for me too

@hammao @rpujakesuma ,
Thanks for your information.
I encounter same issue, and I tried this option: download the certificate (wget https://www.tbs-certificats.com/issuerdata/DigiCertGlobalRootCA.crt), then set the environment variable export WEBSOCKET_CLIENT_CA_BUNDLE=DigiCertGlobalRootCA.crt, it doesn't work for me.
I would like to know how to execute the command, thank you!

This is sloved my problem..
cd /Applications/Python\ 3.7/
./Install\ Certificates.command

Worked for me too

There are a couple of things that can be done: 1. Downgrading the websocket-client library to 0.47.0 2. Or, download the certificate (wget https://www.tbs-certificats.com/issuerdata/DigiCertGlobalRootCA.crt), then set the environment variable export WEBSOCKET_CLIENT_CA_BUNDLE=DigiCertGlobalRootCA.crt

I've also found success setting the Requests or OpenSSL certificates:
    # https://docs.python-requests.org/en/msater/user/advanced/#ssl-cert-verification
    os.environ['REQUESTS_CA_BUNDLE'] = 'YourRootCertificate.crt'  # Requests
    os.environ['SSL_CERT_FILE'] = 'YourRootCertificate.crt'  # OpenSSL
    os.environ['SSL_CERT_DIR'] = '/path/to/your/certificates/'

Due to the fact that no changes should be made in this library I'm closing this issue. The workarounds posted above should be explored if you need to remain on v1 of this project. (i.e. You need to use Python 2.7) I'd encourage all others to take advantage of all the new features in v2 by following the migration guide. SSL configuration is better supported in v2.

Build is still broken here with the same error...took me some time to realize it is slack related issue and not a general OS issue...

@AlmogCohen can you provide any more details to your conclusion? I've not been able to discover any issues where the root cause is related to Slack.

Since this package was the official one supported by slack for a long time I do have some expectations for it to work including some accountability to the dependencies it is shipped with. Maybe it is too much to expect that. If it says any websocket-client version is OK, but the client is breaking... It is odd to me.
Also, I did see the v2 SDK and it looks amazing...BUT it is not trivial to shift existing code bases to use it as if it is a drop-in replacement to issues with the v1 SDK. v2 is awesome, but quite different (for good, yet different)

@AlmogCohen I understand now. Thank you for the feedback. I apologize for the frustration that was caused. I've recently taken on this project and will be leading the development efforts moving forward.
Next week I'll be switching part of my focus on bug fixing v1 where it makes sense. Around this same time I'll consider locking down the websocket-client version to the previous 0.47.0 version.
If you can work with Python 3.6+ and would like to use the new v2 SDK. Please open a new issue to let me know if there's anything I can do to ease the migration efforts.

Thank you man for the understanding and support! Good luck with project
support :)
On Wed, May 8, 2019 at 3:19 PM Rodney Urquhart<EMAIL_ADDRESS>wrote:

@AlmogCohen https://github.com/AlmogCohen I understand now. Thank you
for the feedback. I apologize for the frustration that was caused. I've
recently taken on this project and will be leading the development efforts
moving forward.
Next week I'll be switching part of my focus on bug fixing v1 where it
makes sense. Around this same time I'll consider locking down the
websocket-client version to the previous 0.47.0 version.
If you can work with Python 3.6+ and would like to use the new v2 SDK.
Please open a new issue to let me know if there's anything I can do to ease
the migration efforts.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/slackapi/python-slackclient/issues/334#issuecomment-490671348,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA5VOZTMWMMVC2TK4WAP2MLPUNGVZANCNFSM4FIZ2KAQ
.

--
Almog Cohen
https://htmlsig.com/signatures/01ADE8F6/edit
[image: Linkedin] http://linkedin.com/in/almogcohen
US:<PHONE_NUMBER>

This is sloved my problem..
cd /Applications/Python\ 3.7/
./Install\ Certificates.command

Worked for me too

mee to

that works for me, i've seen it, the aiohttp  expects a ssl_context, buuut..
the web_client really needs to avoid this kind of exception, because the ssl parameter is optional in constructor.
import ssl
import slack
ssl_context = ssl.create_default_context()
ssl_context.check_hostname = False
ssl_context.verify_mode = ssl.CERT_NONE
client = slack.WebClient(token='yourtoken',
                         ssl=ssl_context)

response = client.chat_postMessage(
    channel='#yourchannel',
    text="message, etc..")

How to downgrade to websocket_client 0.47.0 ? i mean what should I write in Mac terminal to get this done ?

Depends on your package management tool - is it pipenv? Is it pip?
Depending on your tool /environment you can set the version of the package
On Sat, Aug 24, 2019, 15:11 Omer-Khabeer<EMAIL_ADDRESS>wrote:

How to downgrade to websocket_client 0.47.0 ? i mean what should I write
in Mac terminal to get this done ?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/slackapi/python-slackclient/issues/334?email_source=notifications&email_token=AA5VOZT5M5LBGWFH5SPD2JLQGE6QLA5CNFSM4FIZ2KA2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD5CAXZY#issuecomment-524553191,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA5VOZSNQ2W5HRYE5HA5STTQGE6QLANCNFSM4FIZ2KAQ
.

it is pip

that works for me, i've seen it, the aiohttp expects a ssl_context, buuut..
the web_client really needs to avoid this kind of exception, because the ssl parameter is optional in constructor.
import ssl
import slack
ssl_context = ssl.create_default_context()
ssl_context.check_hostname = False
ssl_context.verify_mode = ssl.CERT_NONE
client = slack.WebClient(token='yourtoken',
                         ssl=ssl_context)

response = client.chat_postMessage(
    channel='#yourchannel',
    text="message, etc..")

This is what fixed it for me.

How to downgrade to websocket_client 0.47.0 ? i mean what should I write in Mac terminal to get this done ?

pip3 install websocket-client==0.47.0

that works for me, i've seen it, the aiohttp expects a ssl_context, buuut..
the web_client really needs to avoid this kind of exception, because the ssl parameter is optional in constructor.
import ssl
import slack
ssl_context = ssl.create_default_context()
ssl_context.check_hostname = False
ssl_context.verify_mode = ssl.CERT_NONE
client = slack.WebClient(token='yourtoken',
                         ssl=ssl_context)

response = client.chat_postMessage(
    channel='#yourchannel',
    text="message, etc..")

This worked for me unlike the other proposed solutions here.

The issue is still in place
Python 3.6
slack-sdk==3.8.0

from slack_sdk.web import WebClient
cl = WebClient(token=<token>)
cl.chat_postMessage(channel=<channel>, text="Hello world!")

Traceback (most recent call last):   File "/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/urllib/request.py", line 1318, in do_open     encode_chunked=req.has_header('Transfer-encoding'))   File "/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/http/client.py", line 1239, in request     self._send_request(method, url, body, headers, encode_chunked)   File "/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/http/client.py", line 1285, in _send_request     self.endheaders(body, encode_chunked=encode_chunked)   File "/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/http/client.py", line 1234, in endheaders     self._send_output(message_body, encode_chunked=encode_chunked)   File "/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/http/client.py", line 1026, in _send_output     self.send(msg)   File "/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/http/client.py", line 964, in send     self.connect()   File "/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/http/client.py", line 1400, in connect     server_hostname=server_hostname)   File "/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/ssl.py", line 407, in wrap_socket     _context=self, _session=session)   File "/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/ssl.py", line 817, in __init__     self.do_handshake()   File "/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/ssl.py", line 1077, in do_handshake     self._sslobj.do_handshake()   File "/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/ssl.py", line 689, in do_handshake     self._sslobj.do_handshake() ssl.SSLError: [SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed (_ssl.c:852)


Not compatible with richtext

Version: 2.1.1
Description bug:

When the contentType have CKedtior richtext. The draft and live view button are not display.
I debugged and found out that the error was due to this line of code encodeURIComponent(decodeURIComponent(val)) in the parse_url function.
Note: Using default richtext strapi is also got this error

Please help me to take a look this issue. Thank you!

@trungnouden1108 I'm still unable to reproduce this. What does the config for preview button found in config/plugins.js look like? It seems whatever URL is being passed to decodeURIComponent is an invalid URL format.

@trungnouden1108 You may also want to try installing the latest version 2.2.0 of this plugin which just released yesterday. Not sure if it relates to your problem but try using Strapi v4.14.x with v2.2.0 of this plugin and let me know how it goes for you 👍🏻

@trungnouden1108 I'm still unable to reproduce this. What does the config for preview button found in config/plugins.js look like? It seems whatever URL is being passed to decodeURIComponent is an invalid URL format.

I have debugged and see that your code will decode and then encode the value of the entire field. And when any field has this '%' character, the code will error
You can create content-type contains rich-text field, and paste the content below
- 200 grams of broccoli can provide up to 75% of the body's vitamin A needs. Glutathione transferase in improved cotton has detoxifying and fat-reducing effects

@trungnouden1108 I'm still unable to reproduce this. What does the config for preview button found in config/plugins.js look like? It seems whatever URL is being passed to decodeURIComponent is an invalid URL format.

Here is my config/plugins.js

Hi again @trungnouden1108 Thank you for that perfect example text with the % character 🙌🏻
The plugin is now upgraded to version 2.2.1 which should resolve your issue. Let me know how it goes for you! 👍🏻


Uncaught Error: Cannot read property 'object' of undefinedTypeError: Cannot read property 'objec...

[Enter steps to reproduce:]

Add new Project Folder
Select Folder hit "Open Folder"

Atom: 1.16.0 x64
Electron: 1.3.13
OS: FLOMPUTER
Thrown From: Atom Core
Stack Trace
Uncaught Error: Cannot read property 'object' of undefined
TypeError: Cannot read property 'object' of undefined
at ObjectsRegistry.get (g:\Flo\AppData\Local\atom\app-1.16.0\resources\electron.asar\browser\objects-registry.js:41:28)
at EventEmitter. (g:\Flo\AppData\Local\atom\app-1.16.0\resources\electron.asar\browser\rpc-server.js:306:31)
at emitMany (events.js:127:13)
at EventEmitter.emit (events.js:201:7)
at WebContents. (g:\Flo\AppData\Local\atom\app-1.16.0\resources\electron.asar\browser\api\web-contents.js:222:13)
at emitTwo (events.js:106:13)
at WebContents.emit (events.js:191:7)
At C:\Users\Flo\AppData\Local\atom\app-1.16.0\resources\electron.asar\renderer\api\remote.js:210

Error: Cannot read property 'object' of undefined
TypeError: Cannot read property 'object' of undefined
    at ObjectsRegistry.get (g:/Flo/AppData/Local/atom/app-1.16.0/resources/electron.asar/browser/objects-registry.js:41:28)
    at EventEmitter.<anonymous> (g:/Flo/AppData/Local/atom/app-1.16.0/resources/electron.asar/browser/rpc-server.js:306:31)
    at emitMany (events.js:127:13)
    at EventEmitter.emit (events.js:201:7)
    at WebContents.<anonymous> (g:/Flo/AppData/Local/atom/app-1.16.0/resources/electron.asar/browser/api/web-contents.js:222:13)
    at emitTwo (events.js:106:13)
    at WebContents.emit (events.js:191:7)
    at metaToValue (~/AppData/Local/atom/app-1.16.0/resources/electron.asar/renderer/api/remote.js:210:13)
    at undefined.remoteMemberFunction (~/AppData/Local/atom/app-1.16.0/resources/electron.asar/renderer/api/remote.js:111:18)
    at ReopenProjectMenuManager.updateWindowsJumpList (/app.asar/src/reopen-project-menu-manager.js:111:16)
    at ReopenProjectMenuManager.update (/app.asar/src/reopen-project-menu-manager.js:79:12)
    at Function.module.exports.Emitter.simpleDispatch (/app.asar/node_modules/event-kit/lib/emitter.js:25:14)
    at Emitter.module.exports.Emitter.emit (/app.asar/node_modules/event-kit/lib/emitter.js:141:28)
    at HistoryManager.didChangeProjects (/app.asar/src/history-manager.js:77:20)
    at HistoryManager.addProject (/app.asar/src/history-manager.js:95:12)
    at /app.asar/src/history-manager.js:36:20
    at Function.module.exports.Emitter.simpleDispatch (/app.asar/node_modules/event-kit/lib/emitter.js:25:14)
    at Emitter.module.exports.Emitter.emit (/app.asar/node_modules/event-kit/lib/emitter.js:141:28)
    at Project.module.exports.Project.addPath (/app.asar/src/project.js:288:29)
    at /app.asar/src/atom-environment.js:1028:40
    at /app.asar/src/application-delegate.js:31:16)
    at emitTwo (events.js:106:13)
    at EventEmitter.emit (events.js:191:7)

Commands
     -0:28.9.0 application:add-project-folder (atom-workspace.workspace.scrollbars-visible-always.theme-atom-light-syntax.theme-atom-light-ui)

Non-Core Packages
isotope-ui 2.5.1

Thanks for taking the time to contribute!
We noticed that this is a duplicate of #11692. You may want to subscribe there for updates.
Because we treat our issues list as the Atom team's backlog, we close duplicates to focus our work and not have to touch the same chunk of code for the same reason multiple times. This is also why we may mark something as duplicate that isn't an exact duplicate but is closely related.
For information on how to use GitHub's search feature to find out if something is a duplicate before filing, see the How Can I Contribute? section of the Atom CONTRIBUTING guide.


runtime: high startup address space usage (RLIMIT_AS) on Linux AMD64

What version of Go are you using (go version)?

$ go version
go version go1.14 linux/amd64

Does this issue reproduce with the latest release?
Yes
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GO111MODULE=""
GOARCH="amd64"
GOBIN=""
GOCACHE="/home/vorvvbgc/.cache/go-build"
GOENV="/home/vorvvbgc/.config/go/env"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOINSECURE=""
GONOPROXY=""
GONOSUMDB=""
GOOS="linux"
GOPATH="/home/vorvvbgc/go"
GOPRIVATE=""
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/home/vorvvbgc/go1.14/go"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/home/vorvvbgc/go1.14/go/pkg/tool/linux_amd64"
GCCGO="gccgo"
AR="ar"
CC="gcc"
CXX="g++"
CGO_ENABLED="1"
GOMOD=""
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build969828813=/tmp/go-build -gno-record-gcc-switches"

What did you do?
I am trying to get an FastCGI server running behind an Apache2 webserver on a shared hosting system using the net/http/fcgi library. The webserver is limiting my software to 512MB memory.
This is the code: https://play.golang.org/p/Z-Gc6icOpw5
What did you expect to see?
I expect to see "This was generated by Go running as a FastCGI app" on the website generated by the FastCGI server.
What did you see instead?
I have modified the sysReserve() function in the runtime to include println() to print out the error code from mmap() and the requested memory size. This is a diff of src/runtime/mem_linux.go and my version:
157a158,159
>       println(err)
>       println(n)

I kept the output in the following output in the hopes that it might be useful.
The application crashes with this trace:
0
131072
0
1048576
0
8388608
0
67108864
12
536870912
fatal error: failed to reserve page summary memory

runtime stack:
runtime.throw(0x6f3456, 0x25)
        /home/vorvvbgc/go1.14/go/src/runtime/panic.go:1112 +0x72 fp=0x7ffc17e5b170 sp=0x7ffc17e5b140 pc=0x433a12
runtime.(*pageAlloc).sysInit(0x939428)
        /home/vorvvbgc/go1.14/go/src/runtime/mpagealloc_64bit.go:80 +0x13f fp=0x7ffc17e5b1e8 sp=0x7ffc17e5b170 pc=0x42ac1f
runtime.(*pageAlloc).init(0x939428, 0x939420, 0x94db38)
        /home/vorvvbgc/go1.14/go/src/runtime/mpagealloc.go:297 +0x75 fp=0x7ffc17e5b210 sp=0x7ffc17e5b1e8 pc=0x4288b5
runtime.(*mheap).init(0x939420)
        /home/vorvvbgc/go1.14/go/src/runtime/mheap.go:694 +0x274 fp=0x7ffc17e5b238 sp=0x7ffc17e5b210 pc=0x425ad4
runtime.mallocinit()
        /home/vorvvbgc/go1.14/go/src/runtime/malloc.go:470 +0xff fp=0x7ffc17e5b268 sp=0x7ffc17e5b238 pc=0x40c41f
runtime.schedinit()
        /home/vorvvbgc/go1.14/go/src/runtime/proc.go:545 +0x60 fp=0x7ffc17e5b2c0 sp=0x7ffc17e5b268 pc=0x437100
runtime.rt0_go(0x7ffc17e5b2f8, 0x1, 0x7ffc17e5b2f8, 0x0, 0x7fd2029790ca, 0x1, 0x7ffc17e5cbb6, 0x0, 0x7ffc17e5cbc4, 0x7ffc17e5cbe6, ...)
        /home/vorvvbgc/go1.14/go/src/runtime/asm_amd64.s:214 +0x125 fp=0x7ffc17e5b2c8 sp=0x7ffc17e5b2c0 pc=0x460655

The application works fine with golang 1.13.9.
I have no idea how to debug this further.

@aclements

@pkramme Hi, would you mind to post your Apache setup for this too? I did not reproduce this on a fresh Apache. Maybe it is due to my configuration.

@pkramme is this shared hosting environment cPanel by any chance?
We also started getting reports of this same panic with our Go application, which exposes itself as a .live.cgi FastCGI server integrating with cPanel's LiveAPI, as soon as we upgraded to 1.14.
Going to downgrade to 1.13.9 for now.

@aleksator No, it is not, it is a custom build setup. @ivzhh I'm not able to share the config, as it is proprietary.
Theoretically, if we execute any code with 512MB memory limitation, the problem should become visible. I will try to produce something not based on fcgi as a reproducer, so that no apache2 setup is necessary.

Tagging a proper person here: @alexzorin

I think what @pkramme is correct about the 512MB memory limit - specifically RLIMIT_AS.
"Back in the day" (EL5-ish era), shared web hosting admins did not have access to the not-yet-invented RSS cgroups controller, and so controlling VSZ limits was the only choice available to them. In the long term, this has resulted in a lot of misguided admins keeping these VSZ limits around for no good reason.
Anyway, the Apache-based reproducer is straightforward. (For some reason, a simple Go hello world  wrapped in a bash ulimit -v didn't repro for me, not sure why).

Compile a very simple net/http/cgi binary using Go 1.14.1 and stick it in Apache httpd 2.4's cgi-bin/:

package main

import (
	"net/http/cgi"
)

func main() {
	if err := cgi.Serve(nil); err != nil {
		panic(err)
	}
}

go build -o /var/www/html/cgi-bin/reproducer.cgi reproducer.go

Configure Apache with a 512MB RLimitMEM and restart Apache:

RLimitMEM 536870912

apachectl -k restart

Access http://localhost/cgi-bin/reproducer.go. It will produce an HTTP 500, and in Apache's error_log, you will see the panic stack from the original report.

I would prefer not to ask our customers to remove the rlimit (or else we'll be stuck shipping with Go 1.13 for all eternity).
Would it be practical for the Go runtime to try work within whatever it sees by getrlimit?

Adding that I'm also seeing this issue in a different memory-limited environment with a 512mb limit (a Grid Engine setup). Raising the memory limit to 950mb fixes the issue, but it's unclear to me why it should ever be an issue anyway - the program does not use that much memory during running.

apologies for the "me too" post, but this has also prevented to migrate a little Go-based "script" of one of my colleagues at CERN from Go-1.13.x to the latest Go-1.14.x.

Well, after reinvestigating this issue I stumbled over the proposal for the new page allocator which was introduced in go1.14: https://github.com/golang/proposal/blob/master/design/35112-scaling-the-page-allocator.md

There are only two known adverse effects of this large mapping on Linux:

ulimit -v, which restricts even PROT_NONE mappings.
Programs like top, when they report virtual memory footprint, include PROT_NONE mappings.

In the grand scheme of things, these are relatively minor consequences. The former is not used often, and in cases where it is, it's used as an inaccurate proxy for limiting a process's physical memory use. The latter is mostly cosmetic, though perhaps some monitoring system uses it as a proxy for memory use, and will likely result in some harmless questions.

So, this explains it. @aclements Is there a workaround for cases like this?

cc @mknyszek

As @pkramme points out, we were aware of this issue when the changes to the page allocator were proposed. As @alexzorin points out, ulimit -v is an out-dated mechanism for limiting memory use.

I would prefer not to ask our customers to remove the rlimit (or else we'll be stuck shipping with Go 1.13 for all eternity).
Would it be practical for the Go runtime to try work within whatever it sees by getrlimit?

The short answer is no. The virtual memory mappings made to support structures in the page allocator significantly simplified the improvements made in the 1.14 release. Earlier on in the release cycle the amount of memory mapped was much larger which caused problems on certain platforms where the default ulimit -v value for default users was fairly low, so out-of-the-box Go programs would not work on an out-of-the-box system without having additional privileges (see #35568). This is generally not true on Linux where ulimit -v is unlimited by default (at least for the versions I'm aware of). We took steps to reduce the size of these mappings at the cost of additional complexity and a small performance regression. We experimented a little with additional mitigations but concluded they weren't practical.
@sbinet @mashedkeyboard @alexzorin @pkramme:
In order to understand your situations better, could you elaborate on the reasons why your and/or your customers cannot set RLIMIT_AS/ulimit -v to unlimited, or an otherwise sufficiently high number for your Go programs?
As a side note, (and to be totally clear, I'm not recommending this as an official workaround) compiling your code with GOARCH=386 should allow your code to run on amd64 platforms with a low ulimit -v, since the memory mapping we make is proportional to the size of the address space and the address space is much smaller on 386. I recognize that this has its issues, and is not generally a feasible alternative. The most notable issues that come to mind are that your code might run slower (due to 32-bit registers and a lack of certain intrinsics) or  some libraries you code depends on might not support 32-bit platforms (I'm not sure how common it is for libraries to support amd64 but not 386, but it is possible).

could you elaborate on the reasons why your and/or your customers cannot set RLIMIT_AS/ulimit -v to unlimited, or an otherwise sufficiently high number for your Go programs?

This is our plan. It's going to be a challenge for XX,000 hosts between X,000 customers, so we are first planning to add telemetry to our 1.13 builds to see how many systems run the CGI under restricted virtual memory.

Sure!
The Go software I am writing is supposed to run on a shared hosting server.
The technical foundation is a LAMP (linux apache2 mysql php/python/...)
stack. Inside a shared hosting LAMP stack, the Apache2 webserver is spawning
CGI/FastCGI software inside a restricted environment, which is heavily
controlled in access and resources by the provider, in order to prevent one
user taking up all the resources. On my shared hosting account one FastCGI
process is limited to 512MB "memory".
The important part is that in managed hosting, you simply cannot make that
change, because you do not control the environment. The only possibility for
me is to upgrade to another, more expensive hosting plan, so that I can use
1GB memory or more so that this allocation works.

We don't have much lever on how to configure the CGI environment.
and CERN-IT is a bit conservative w/ changing the configuration of services they provide for their physicists (who are sometimes a bit "cavalier" with how they setup their things.)
nonetheless, I've sent a ticket on raising the RLIMIT_AS.
I've also passed on to my colleague the 32b workaround.
we'll see.
(anyways, it's not a high profile CGI service, we won't miss supersymetry nor mini-blackholes, or loose the beam if we're stuck w/ Go-1.13.x b/c of that. :P)

@mknyszek Is there any progress on this from your end?

we are first planning to add telemetry to our 1.13 builds to see how many systems run the CGI under restricted virtual memory.

To put a conclusion on this from my end, we gathered some RLIMIT_AS stats and the number of affected users is around 0.5%. The majority have a limit of 4096MB set on Apache, which is the vendor default on this platform.
As long as Go continues to work within that limit, we're happy to live with it and will ask those other users to adapt. Thanks.

any progress on it?

Can't believe this remains unresolved. Are we supposed to downgrade golang to 1.13?

any progress on it?

I don't know that anybody knows of a feasible way to fix this.  The Go memory system expects that address is space is available.  Address space costs nothing.  It makes sense to constrain program's use of actual memory.  It does not make sense to constrain program's use of address space.

I don't know that anybody knows of a feasible way to fix this. The Go memory system expects that address is space is available. Address space costs nothing. It makes sense to constrain program's use of actual memory. It does not make sense to constrain program's use of address space.

I am not really an expert in memory use / allocation. I am not even sure what you mean by address space. If it is only testing and not using. ;) The fact is I wasted quite a bit of time before noticing it was related to this imposed memory limit somewhere of the parent application
This makes me also wonder about container environments applying resource limits, are these affected as well by this?

I am not really an expert in memory use / allocation. I am not even sure what you mean by address space. If it is only testing and not using. ;)

See https://go.dev/doc/gc-guide#A_note_about_virtual_memory.

This makes me also wonder about container environments applying resource limits, are these affected as well by this?

They are not. Container limits pertain to actual physical memory usage (RSS in top) not virtual memory footprint (VSS in top).


Cant Debug Typescript

Hello,
you cant use debugger with typescript. I add the sourceMapRoot and generatedSourceRoot to the launch.json, but every time you connect to the debugger i got this message:
Failed to find source files. Check that launch.json 'localRoot' contains scripts or if using source maps that 'sourceMapRoot' is correct.

#28 should resolve this I believe.

(sorry for the delayed response).  There have been a few updates since this time, and the readme has been updated as well. If this still reproduces with updated launch.json configs, please re-open!


Fixed the bug CTS-1787 Deprecate webapi-devicecapabilities-w3c-tests.

BUG=https://crosswalk-project.org/jira/browse/CTS-1787

LGTM


Existing drug terminologies and normalization services

This thread tracks known and public drug terminologies and normalization services. Initial list here: https://docs.google.com/spreadsheets/d/1bhA4RVt7HT2oIMVWeaXcnxFNcUQ4rEl9mB5cBSK8Kzw/edit#gid=0

Practicing writing a comment - next I'll figure out how to push the new rows I added (regimens)

Anatomical Therapeutic Chemical Drug classification system.

Anatomical Therapeutic Chemical Drug classification system.

Yes - we have that one in the list of databases (ATC). Row 2 of the Google Doc. Please let us know if you see other missing terminologies!

I found two additional terminologies while poking around today - DTP/NCI, and EPA DSSTox. I've added to my local database but might be better if I add the document to the Git repository? I also have a separate spreadsheet of web resources, which I distinguish from terminologies in that they don't maintain their own (externally available) code set to represent the concepts contained within.

I'm calling this one "stale"


Cookies not limited to single port

I'm running kanboard in docker containers, so I have multiple instances on a single IP.
For example:
http://<IP_ADDRESS>:20080/kanboard/?controller=board
http://<IP_ADDRESS>:30080/kanboard/?controller=board
http://<IP_ADDRESS>:40080/kanboard/?controller=board
All three point to separate kanboard container instances.
All three instances try to share the same cookie in the browser.
This causes issues when trying to access the instances simultaneously in the browser.
Not really an important issue, just thought it was interesting.

Apparently, cookies doesn't provide isolation by port. See http://stackoverflow.com/questions/1612177/are-http-cookies-port-specific
In your case, you can setup a reverse-proxy configured with sticky session.


Upgrade community-edition's framework (CLI) version to v0.10.0

This request was sourced in some capability work individuals want to bring into TCE.

At this point, we have upgraded main to v0.10.0:
https://github.com/vmware-tanzu/community-edition/blob/9e16cb3323b3591b0f1ceba021eab075f175b77f/hack/FRAMEWORK_BUILD_VERSION#L1
This means it'll be part of the v0.10.0 release, which we're still hoping to get our in January. If anything changes, it'll be reflected here.


"Blocked post" message should indicate the blocker

Is your feature request related to a problem? Please describe.
When the "Blocked post" message appears, I can't tell if the blocked user was blocked by me, one of the block-lists I subscribe to, or someone in the thread.
Describe the solution you'd like
The message for a blocked post should say "Blocked by you", "Blocked by (user)", "The author of this post has blocked you", or whatever detailed message is appropriate.

Additional context

I see the potential for abuse here. Block lists can be a dangerous revenge tool. If someone doesn't like you, they add you to some block lists and all of a sudden you're blocked by hundreds of people.

This is a problem for quote posts too. A friend was blocked by someone they quoted (not detached, but blocked) but anyone looking at the thread just sees "Blocked." instead of the quote, leading to confusion by any third party reading the thread (since it looks like they were blocked by the quoted poster instead)

It should at least mention the block list when it's blocked by one; I rather doubt that has much revenge potential...


Widgets: Attempt to re-register for a scalar token if ours is invalid

@manuroe commented on Tue Mar 12 2019
Riot mobile apps should validate scalar_token before using it as riot-web does with https://github.com/matrix-org/matrix-react-sdk/pull/1668.

Done, by intercepting 403 error from the WebView


Create a web interface that works with trezor bitcoin

[ ] Sign nlocktime tx with trezor
[ ] Download backup as a .md or .txt
[ ] If recipient is not set, generate a new key, set it as the recipient and add the private key to the backup file
[ ] Download calendar reminder
[ ] Include a revoke transaction that spends the funds back to the signer's address

This would be completely non custodial and could later include alerts that watch the user's wallet for transactions that invalidate the backup.

Another option could be forking the Jade firmware to create backups each time a transaction is done via the wallet.


PowerShell 7.1.0 error using this command

Hello, i've encountered with a problem using Powershell 7.1.0
When using Get-WinUserLanguageList command you get an error :
Get-WinUserLanguageList: Cannot marshal 'parameter #2': Invalid managed/unmanaged type combination.

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

ID: 9a3d9b94-9f70-6c11-552d-a6f00f20fec8
Version Independent ID: 0603585f-7db5-8ccf-99ca-beef7cf671b4
Content: Get-WinUserLanguageList (International)
Content Source: docset/windows/international/Get-WinUserLanguageList.md
Product: w10
Technology: ****
GitHub Login: @Kateyanne
Microsoft Alias: v-kaunu

The feedback section you have submitted this feedback from is intended for product documentation issue.
Please consider checking with PowerShell community for your problem so that it gets resolved.
PowerShell Community Links
For bug reports specific to PowerShell, please visit, PowerShell at GitHub.
Thanks.


Clicking on videos doesn't start them

The poster image for videos shows a big play button, but clicking on it doesn't start the video. You have to click on the Play button in the video controls instead.
Example: https://developers.google.com/web/tools/chrome-devtools/debug/breakpoints/step-code?hl=en

Thanks for the feedback, we've got an update coming next week that should fix this.


Support for ' ` ^ ~ = \u . " \r \H \v text-mode accents

I took a stab at implementing the text-mode accents that were easy to implement using current fonts, as outlined by @gagern in #638.  Here's a sample screenshot:

\begin{array}{l}
\text{e \'e \`e \^e \~e \u{e} \.e \"e \r{e} \H{e} \v{e}}\\
\text{l \'l \`l \^l \~l \u{l} \.l \"l \r{l} \H{l} \v{l}}\\
\text{X \'X \`X \^X \~X \u{X} \.X \"X \r{X} \H{X} \v{X}}
\end{array}

This seems to work, but feels a bit hacky because the accents are combining characters so don't behave well in isolation, so there's a magic number in a CSS style (similar but different to the \vec accent).  It also misses key accents like \c because they're not in the font.  Makes me wonder whether this is worthwhile, or we should correct the font somehow to make non-combining characters for these accents like we have for math mode.  On the other hand, this support is better than nothing for now.

Does this work properly on Safari? Issue #735 is a combining character problem.

@ronkok Probably not -- should have the same issue.  Another reason to "wonder whether this is worthwhile, or we should correct the font somehow to make non-combining characters for these accents like we have for math mode. On the other hand, this support is better than nothing for now."

We should look at porting this fix https://github.com/mathjax/MathJax/pull/1775/files.  I feel like that's a separate issue/PR.

Yesterday, I  tried inserting a non-breaking space:
if (group.value.label === "\\vec") {
    accent.value = "\u00A0" + accent.value;
}

... and then adjusting the advance value in CSS class accent-vec by 0.25em (because nbsp is 0.25 em wide). I found that the arrow came in too low. I don't think I'll get farther with it. I've got other issues at the moment.

Cool, I hadn't seen that.  Now that I understand the accent generation code, I pushed a fix which ought to work for Safari.  Can someone test?  In my experiments, 0.262em seems a closer approximation to the width of a space -- how did you compute 0.25em?

I got the 0.25 from http://glyphrstudio.com/online/. I don't take it as conclusive.

Safari on the left, Chrome on the right.

@edemaine Instead of combining characters, could you use modifier characters?

Hmm, not sure why that's not working.  Perhaps the combination of left and margin-left?  I don't think we have a choice of the type of character, if we fix the font as-is.

The Chrome changes look good for me.  I started playing around with the styles on Safari and it looks like margin-left should be -spaceWidth + xMin for each of the accents.  The xMin for the accents is a negative number b/c it's to the left of the origin.
The width of the space character is 0.250em according to opentype.js (I uploaded KaTeX_Main-Regular.ttf).  For each of the accents the average of xMin and yMin is -0.250em which ends of centering it perfectly over the center of the space.  Since Chrome renders the accent character correctly over the space, all we have to do is to correct for the space alone which is why the margin-left: -0.250em does the trick.
Chrome however treats the accent as a character with a real width from xMin to the origin.  In the case of the umlaut which has an xMin of -0.405em, the resulting correction is margin-left: -0.655em.

Notice that this doesn't quite fix the single dot accent to the left of the umlaut.  This is because its xMin is a different value.
This means that the solution is different for Safari and Chrome.  The MathJax hack special cases Safari as well.  The unfortunate result of this is that server side rendering will only work if you return different markup based on the browser's user agent.  For client side rendering special casing shouldn't pose a problem.
One last note: fontMetricsData.js doesn't actually contain the values we need for the accent characters to fix Safari so we're going to need to extract those from the font.

@edemaine let's not worry about fixing Safari accent positioning in this diff.

@kevinbarabash I'm a little confused about whether I should include the space hack in this patch at all.  My impression was that it wasn't necessary at all for Chrome, but perhaps your investigation has found that it makes it easier to get the horizontal offset more consistent?  Still grokking.

@edemaine I didn't try removing the hack.  If all the new accents work on Chrome without the hack, let's take it and get this merged and we can deal with Safari separately.

opentype.js was insightful!  I realized that most of these accent characters have noncombining versions in the current font.  I switched to those, so they should work in Safari.  The only missing one is \H.  I still use combining characters for that one.
I found that the width of a space (0.250em) looked best for \H.  I would have thought something like: offset by -xMin to start at position 0; then shift back by half of the width (xMax - xMin) to center the accent.  This worked well for e.g. \" (not that I need that anymore), but I think only because it ended up resulting in an almost identical offset of 0.2505em...  @kevinbarabash If you understand this better, let me know if you think this offset is slightly off...
Other changes:

Removed the Safari hack (no more space before the accent)
Added allowedInMathMode to forbid these commands in math mode (as they don't work there -- and nor do they in LaTeX).  This will interact with #766.
Added a screenshot test

To fix \H for Safari we could look at updating the font to create a non-combining version of the glyph, but that's another PR.  Also, if we end up not using the combining versions of these glyph we might look at removing them completely from the font to save bytes.

@kevinbarabash Thanks for the review!  I agree that we don't need combining characters in the font, and it'd be nice to add \H, \", and \vec accents as non-combining characters.


Typo: ID should be activedescendant, not activedescendent

This line: https://github.com/ianmcburnie/jquery-active-descendant/blob/master/jquery.activedescendant.js#L28

Fixed in v0.3.0.


Add cli option to specify src directory

In my project python packages live under the src directory. I want to be able to run the strawberry serve command from the root directory. The uvicorn cli has a --app-dir option to support this use case. The present change adds the same mechanism to the strawberry CLI in the form of a --src-dir option.
I still need to add a test case to tests/cli/test_server.py, and to describe the feature in docs/features/server.md, but I'd like  to get preliminary approval for the approach first, if that's OK.
Checklist

[x] My code follows the code style of this project.
[x] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[x] I have tested the changes and verified that they work and don't break anything (as well as I can manage).

Hi, thanks for contributing to Strawberry 🍓!
We noticed that this PR is missing a RELEASE.md file. We use that to automatically do releases here on GitHub and, most importantly, to PyPI!
So as soon as this PR is merged, a release will be made 🚀.
Here's an example of RELEASE.md:
Release type: patch

Description of the changes, ideally with some examples, if adding a new feature.

Release type can be one of patch, minor or major. We use semver,so make sure to pick the appropriate type. If in doubt feel free to ask :)

@nat-n looks good to me, should we follow uvicorn and name the parameter --app-dir as well?

@patrick91 Thanks.
I was thinking the semantics are a slightly different. Uvicorn targets an ASGI app, whereas strawberry server targets a schema. Though consistency is also valuable; I don't feel strongly about it.
I'll prepare the rest of the PR.

Updated the RELEASE.md with an example.

@patrick91 Thanks! I like the way you've set things up here :)


Repo sync for protected CLA branch

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

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


OTA Client increasing size issue (TZ-545)

Question
On increasing size of OTA_UPGRADE_MAX_DATA_SIZE,  ota functionality not works.
Default value of OTA_UPGRADE_MAX_DATA_SIZE = 64, and if change it to greater tha 64 like 128 or 512, ota functionality not works.
Additional context.
No response

@mundevx ,
The maximum data size for OTA is currently limited to below 64 in the current version. We are working on providing an improved solution to enhance the OTA speed.

@mundevx ,
The maximum size of an OTA upgrade is now supported up to 255 (limited by the ZCL specification) in esp-zigbee-sdk v1.5.0. Please use the OTA example for testing.
If the issue has been resolved, please consider closing it.

Thank you very much @xieqinan


chore(release): merge develop into main branch

Making a new release of PSA.

:tada: This PR is included in version 5.2.6 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


best Realsense model for toothpick

Hi everyone,
I would like to purchase an Intel realsense for a project.
Before buying it, I would like to know which model of Realsense is the best to sees a toothpick with stereo/depth module and with what quality?
Because a toothpick is pretty thin.
Could someone post some pictures of how the different realsense sense the toothpick?
Thanks
This is a picture I've found online:
https://m.imgur.com/a/04pmrWp
This could be an example scenario of my project

Hi @vipelz   You are correct that the small thickness of a toothpick makes it a challenging object to take a depth image of.  I set up a scene with a plain backdrop an a toothpick mounted vertically standing like in your image, and ran tests with all of the RealSense depth camera models.  I was able to produce a clear and close-up image of the toothpick only with the RealSense D435i camera, and with a specific configuration that mapped RGB color to depth in a 3D point cloud image.

The configuration that this image was achieved with in the RealSense Viewer program is:

RealSense D435i (it has a minimum depth sensing distance of 0.1 meters, enabling the camera to be positioned close to objects).
3D point cloud mode, with RGB color mapped to depth points
A depth unit scale set to 0.0001 instead of the default 0.001 scale.

Hi @vipelz   You are correct that the small thickness of a toothpick makes it a challenging object to take a depth image of.  I set up a scene with a plain backdrop an a toothpick mounted vertically standing like in your image, and ran tests with all of the RealSense depth camera models.  I was able to produce a clear and close-up image of the toothpick only with the RealSense D435i camera, and with a specific configuration that mapped RGB color to depth in a 3D point cloud image.

The configuration that this image was achieved with in the RealSense Viewer program is:

RealSense D435i (it has a minimum depth sensing distance of 0.1 meters, enabling the camera to be positioned close to objects).
3D point cloud mode, with RGB color mapped to depth points
A depth unit scale set to 0.0001 instead of the default 0.001 scale.

I just ordered a D435i. Could you please upload here the current stereo module setting to file for the realsense viewer?
Thank you
I just ordered a D435i

Wow you are so kind.
I just ordered a D435i. Could you please upload here the current stereo module setting to file for the realsense viewer?
Thank you
I just ordered a D435i

You are very welcome.  :)
The Custom json preset that I used to create these images is built into the Viewer's list of pre-made configurations and is its default preset when the Viewer is launched, so there is no need to upload a json.
The Depth Unit Scale option is under the Controls sub-section of the Stereo Module options.  You can input 0.0001 manually by left-clicking on the pencil icon beside the option.
You can enter 3D point cloud mode by left-clicking on the 3D option in the top corner of the Viewer.  You can map RGB to the depth points by enabling Depth and RGB, and then left-clicking on the Texture option and selecting Color from its menu if it does not automatically switch to the RGB-aligned image.

You are very welcome.  :)
The Custom json preset that I used to create these images is built into the Viewer's list of pre-made configurations and is its default preset when the Viewer is launched, so there is no need to upload a json.
The Depth Unit Scale option is under the Controls sub-section of the Stereo Module options.  You can input 0.0001 manually by left-clicking on the pencil icon beside the option.
You can enter 3D point cloud mode by left-clicking on the 3D option in the top corner of the Viewer.  You can map RGB to the depth points by enabling Depth and RGB, and then left-clicking on the Texture option and selecting Color from its menu if it does not automatically switch to the RGB-aligned image.

Hi @vipelz  Do you require further assistance with this case, please?   Thanks!

Hi @vipelz  Do you require further assistance with this case, please?   Thanks!

Perfect thank you very much for sharing, I will make the suggested changes and evaluate the RMS error.
I am using the D435i model.
I shared in the link below the example file that I am testing.
https://1drv.ms/u/s!At7Yi4YwjQcBnxmZ78dZ5mNcAkdX?e=fW06gD

Case closed due to no further comments received.


Pypi page looks strange

The pypi page of your project looks strange: https://pypi.python.org/pypi/django-health-check
AFAIK pypi does not support mark-down. You need to use reStructuredText.

Looks good now. Thank you


Elm Reactor should pass flags: null

Quick Summary: To develop apps with JS-Interop it would be great if reactor would set init to null, so that on the Elm side you can declare main like main: Program (Maybe <some-type>) Model Msg.
Currently it doesn't set any value which means that undefined is returned, and this error will be shown
(1) Problem with the given value:

    undefined

    Expecting null

SSCCE
Try this in playground --
import Html exposing (text)
import Browser

main: Program (Maybe Int) Model Msg
main =
    Browser.document
      { init = \a -> ({}, Cmd.none)
      , view = \m -> { title = "t", body = [text "hi"]}
      , update = \_ _ -> ({}, Cmd.none)
      , subscriptions = \_ -> Sub.none
      }

type alias Model = {}
type Msg = None

On a side note, I sort of expected to be able to do this:
type alias Undefined = ()

type Startup = Undefined | Maybe Int

main : Program Startup Model Msg

But that results in:
BAD FLAGS - Your `main` program wants a `Startup` value from JavaScript.

21| main =
    #^^^^#
I cannot handle that. The types that CAN be in flags include:

    Ints, Floats, Bools, Strings, Maybes, Lists, Arrays, tuples, records, and
    JSON values.

Since JSON values can flow through, you can use JSON encoders and decoders to
allow other types through as well. More advanced users often just do everything
with encoders and decoders for more control and better errors.

Which is reasonable, but seems a bit inconsistent when ()  compiles with Program () Model Msg.

type alias Undefined = ()
type Startup = Undefined | Maybe Int

type alias Undefined = () has no effect on the Startup declaration. Startup happens to have a type constructor called Undefined (which takes no parameters). The type alias Undefined is not referenced anywhere and can be removed without changing anything.
Similarly, Maybe is declared as a type constructor for Startup in this example, and it takes an Int parameter - it happens to be called Maybe but it does not use Elm's built in Maybe type.
type in Elm works a lot differently than in TypeScript - basically it's for creating "tagged union" types (unlike with TS, you don't need to have an object with a special "determinant" field). Each option for the type (the first word) is your own custom type constructor function, while following words are types. For example:
type = Fruit String | Vegetable String | Other

means you can declare variables like this
apple = Fruit "Apple"
broccoli = Vegetable "Broccoli"
wheat = Other

If it was possible to have a fruit with no name, you could do:
type = Fruit (Maybe String) | Vegetable String | Other

apple = Fruit (Just "Apple")
secretFruit = Fruit Nothing
broccoli = Vegetable "Broccoli"
wheat = Other

This is how you'd use an Elm Maybe in a custom type.
For flags, you need to do something a bit more involved in order to accept data with different shapes... I'm not super knowledgeable with the different techniques for decoding flags so I'll stop here and just drop the Elm guide's link on them for convenience: https://guide.elm-lang.org/interop/flags.html
Hope this helps!

@adamdicarlo Thanks for replying. That makes sense. Let me rephrase my example so I can get some clarity.
What is this type known as, () ?
And if I rewrite my example, then would this expression make sense,
type Startup = NoFlags () | Edit (Maybe Int)

So what I'm looking for is the ability to do this,
type Startup = NoFlags () | IntFlag (Maybe Int)

main: Program Startup Model Msg
...

The reason is because this is valid,
main: Program () Model Msg

and this is valid,
main: Program (Maybe Int) Model Msg

But the reason the second example wouldn't work in the playground is because the playground doesn't pass in null as a flag parameter, and by extension the reactor does not pass null as well. But if you were to write the html on your own and pass null it would work. But if you don't pass anything (undefined), then () is expected.
So I guess the limitation here is that main (elm compiler) doesn't accept union-types as a flag parameter?


nerdctl run -t is not greppable (terminal output is not connected to stdout)

Docker (working as expected):
$ docker run -t --rm alpine sh -c 'echo foo; echo bar' | grep foo
foo

nerdctl 1.1:
$ nerdctl run -t --rm alpine sh -c 'echo foo; echo bar' | grep foo
foo
bar

Looks like the terminal output (-t) is not connected to the stdout.

closed by https://github.com/containerd/nerdctl/pull/2167


app("blade-extensions.helpers")->get('minifier')->open('html') is null

I got Error Message below.
ErrorException (E_ERROR)
Call to a member function open() on null (View: /home/vagrant/myapp/resources/views/mypath/myview.blade.php)

This error does not occurs at first access . But next time this error occurs.
Is this because of this code on view cache file?
<?php echo app("blade-extensions.helpers")->get('minifier')->open('html'); ?>
I confirmed like this.
<?php dd( app("blade-extensions.helpers"))?>
HelperRepository {#416 ▼
  #helpers: []
}

I use this
"name": "radic/blade-extensions",
            "version": "7.0.2",
            "source": {
                "type": "git",
                "url": "https://github.com/RobinRadic/blade-extensions.git",
                "reference": "2ddc36e99e9e462e67ba90a8d889ced651426d88"
            },

I resolved this error that I write my application class extend Radic\BladeExtensions\BladeExtensionsServiceProvider
    protected function registerHelperRepository()
    {
        $this->app->singleton('blade-extensions.helpers', function ($app) {
            $helpers = new HelperRepository();
            $helpers->put('minifier', $app->build(Helpers\Minifier\MinifierHelper::class));
            return $helpers;
        });
    }

Is this correct way?
regards..

Hello,
This should be fixed in 7.1.
For 7.0 your fix is correct.


[WIP] feat: Add class and prompts for issue label generator

Add Issue Label Generator
Overview
This pull request introduces a new feature to automatically generate issue labels based on the issue title and description. It leverages a large language model (LLM) to analyze the issue content and recommend appropriate labels from a predefined list.
Changes

Key Changes:

Added issue_label.py file containing the IssueLabelGenerator class.
Added issue_label_prompts.py file containing prompts for the LLM.

New Features:

Implemented IssueLabelGenerator class to generate issue labels using an LLM.
Integrated LLM usage tracking and cost calculation.

Refactoring:

None.

✨ Generated with love by Kaizen ❤

Original Description

Is this task ready for review or are we waiting on anything specific?

The code analysis generation is done. I'm still working on adding it to the github app.

That's great! I'll close this issue now.

Maybe we should merge this and create a separate task for github app?

That sounds good. I'll put this PR up for review.

@sauravpanda Do I need to make anymore changes to this PR? I'm still working on the GitHub app integration for issue analysis and will make a PR soon.


[FIX] Close gaps in routing support of ComponentAnalyzer

accept routes with multiple targets
merge target-local configuration with main router configuration

Pull Request Test Coverage Report for Build 229

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

Totals

Change from base Build 224:
0.0%

Covered Lines:
633

Relevant Lines:
870

💛  - Coveralls


Can't open PC or NPC sheets after update today

Forge Hosted
PF2 v5.14.1
Foundry 11.315

Uninstall and reinstall the PF2e system

Fixed, ty


Update documention: Pharo12 class deprecatedAlias support

.. it just works.
I will add a test, in the long term we could remove the soil specific class rename feature

But is the deprecated stuff stored along the code?

yes, it is normally in a class side initialize method now. Later I want to have this as meta-data in the class definition