It is suggested to add options。

It is recommended that all endpoints can choose to enable api_token authentication, because when the service is built on an external network and needs to verify the teslamate user and password, api_token using the instruction endpoint will need double verification. When I use the shortcut app of Apple IOS, the verification is a little cumbersome. If api_token authentication is enabled for all endpoints, I don't need to set the password of teslamate user to access.

Bug:
traefik:
Authorization:Basic 6YWFjY3p6e
teslamateapi:
Authorization: Bearer znFsbtg3dAewg342Jak1z1Y
Authorization
The same name conflicts and cannot be verified at the same time.

We could consider adding API_TOKEN_DISABLE with a default of false. true should throw a warning in the logs and skip the Authorization: Bearer  check...

Can I find a solution in this link?
[(https://stackoverflow.com/questions/22229996/basic-http-and-bearer-token-authentication)]
You can pass the token under another name. Because you are handling the authorization from your Application. So you can easily use this flexibility for this special purpose.
curl -i http://dev.myapp.com/api/users
-H "Authorization: Basic Ym9zY236Ym9zY28="
-H "Application-Authorization: mytoken123"
Notice I have changed the header into Application-Authorization. So from your application catch the token under that header and process what you need to do.
Another thing you can do is, to pass the token through the POST parameters and grab the parameter's value from the Server side. For example passing token with curl post parameter:
-d "auth-token=mytoken123"

Hi @lylus,
Hope the PR from @LelandSindt (thanks!) helps you to resolve your issue.
I don't see that changing the name of the Authorization header would be the best way forward, since that can break other users setup. So the implementation of a disable of token check should fit much better. Since you rely on Traefik to do the authorization, this should be just fine.
If you want to test to code, run the docker image tag main to be able to test it. Else I will merge it into latest probably during today evening :)
Regards,
Tobias


Bug in pipenv v2022.4.21 is preventing generation of requirements.txt from Pipfiles

This issue is caused by a bug in a dependency, not by serverless-python-requirements.  However it took quite a bit of digging to track down this issue, so maybe this will help others facing the same issue.
There is a recent bug in pipenv that will cause this plugin to fail if the following conditions are met:

you are using the latest version of pipenv (v2022.4.30)
you are using Pipfiles

An error was introduced into pipenv by this PR https://github.com/pypa/pipenv/pull/5069 which causes
pipenv lock --requirements to write a deprecation warning message to stdout, which then gets written to the requirements.txt file.
The unintentionally injected text looks like this, and it makes the generated requirements.txt file unparseable.
            Warning: The lock flag -r/--requirements will be deprecated in a future version
            of pipenv in favor of the new requirements command. For more info see
            https://pipenv.pypa.io/en/latest/advanced/#generating-a-requirements-txt
            NOTE: the requirements command parses Pipfile.lock directly without performing any
            locking operations. Updating packages should be done by running pipenv lock

As a result serverless deploy terminates like this:
...
Serverless: Generating requirements.txt from Pipfile...
Serverless: [AWS sts 200 0.998s 0 retries] getCallerIdentity({})
Serverless: Parsed requirements.txt from Pipfile in /home/vsts/work/1/s/api/auth/.serverless/requirements.txt...
Serverless: Installing requirements from /home/vsts/.cache/serverless-python-requirements/48be0dccf397293edb052749ee84e17db11e4a514a061b07d1509d06c0c5b39a_x86_64_slspyc/requirements.txt ...
Serverless: Using download cache directory /home/vsts/.cache/serverless-python-requirements/downloadCacheslspyc
Serverless: Running ...

 Error ---------------------------------------------------

  Error: `python3.7 -m pip install -t /home/vsts/.cache/serverless-python-requirements/48be0dccf397293edb052749ee84e17db11e4a514a061b07d1509d06c0c5b39a_x86_64_slspyc -r /home/vsts/.cache/serverless-python-requirements/48be0dccf397293edb052749ee84e17db11e4a514a061b07d1509d06c0c5b39a_x86_64_slspyc/requirements.txt --cache-dir /home/vsts/.cache/serverless-python-requirements/downloadCacheslspyc` Exited with code 1
      at ChildProcess.<anonymous> (/home/vsts/work/1/s/api/auth/node_modules/child-process-ext/spawn.js:38:8)
      at ChildProcess.emit (node:events:526:28)
      at ChildProcess.emit (node:domain:475:12)
      at maybeClose (node:internal/child_process:1092:16)
      at Process.ChildProcess._handle.onexit (node:internal/child_process:302:5)

  Get Support --------------------------------------------
     Docs:          docs.serverless.com
     Bugs:          github.com/serverless/serverless/issues
     Issues:        forum.serverless.com

  Your Environment Information ---------------------------
     Operating System:          linux
     Node Version:              16.14.2
     Framework Version:         2.70.0
     Plugin Version:            5.5.3
     SDK Version:               4.3.0
     Components Version:        3.18.1

make: *** [../../tools/serverless.mak:2: deploy] Error 1

Running the failing command in isolation confirms the issue, pip install is failing because of the spurious lines in the requirements.txt file.
# python3.7 -m pip install -t /root/.cache/serverless-python-requirements/48be0dccf397293edb052749ee84e17db11e4a514a061b07d1509d06c0c5b39a_x86_64_slspyc -r /root/.cache/serverless-python-requirements/48be0dccf397293edb052749ee84e17db11e4a514a061b07d1509d06c0c5b39a_x86_64_slspyc/requirements.txt --cache-dir /root/.cache/serverless-python-requirements/downloadCacheslspyc
ERROR: Invalid requirement: 'NOTE: the requirements command parses Pipfile.lock directly without performing any' (from line 2 of /root/.cache/serverless-python-requirements/48be0dccf397293edb052749ee84e17db11e4a514a061b07d1509d06c0c5b39a_x86_64_slspyc/requirements.txt)

The issue has been addressed in pipenv with this PR https://github.com/pypa/pipenv/pull/5091 so the problem should be resolved when it gets released.

thanks for the great summary, we are using github actions and ran across this same issue. As a temporary workaround, we added a step to generate the requirements.txt file with
      - name: Setup env
        run: |
          echo "\nENV_NAME=staging" >> .env
          pip install pipenv==2022.4.30

      - name: Create Deployment Dependencies File
        run: pipenv requirements > requirements.txt

and then disabled the pipenv integration with
custom:
  pythonRequirements:
    usePipenv: false

Pipenv admin has confirmed the fix is included in pipenv v2022.5.2.

I personally just preinstalled the latest version of pipenv (2022.5.2) before I use the serverless-python-requirements plugin and now it works.

same here

Frist uninstall pipenv:  'pip uninstall pipenv` OR 'brew uninstall pipenv' OR 'sudo uninstall pipenv'
Second install pipenv==2022.5.2:     sudo pip install pipenv==2022.5.2

pipenv: 2022.5.2 works great!! ⚡🚀🤘😃

Thanks for this thread. I am using CircleCI on an older project, python 3.8.6 with default pipenv 2020.11.05. Not an option right now to update the python version, so I had to:

set usePipenv: false in serverless.yml as was suggested earlier in the thread

and change the deploy step in CircleCI config.yml from:

in the build step
- run:
          name: Install Python dependencies
          command: |
            pipenv install --dev --ignore-pipfile

to
 - run:
          name: Install Python dependencies
          command: |
            sudo pip uninstall -y pipenv
            pip install pipenv
            pipenv sync --dev

and the deploy step
- run:
          name: Deploy to staging
          command: |
           # ... aws configure steps
           pipenv run sls deploy --stage staging

to:
      - run:
          name: Deploy to staging preview
          command: |
            sudo pip uninstall -y pipenv
            pip install pipenv
            pipenv requirements > requirements.txt
            . /home/circleci/repo/.venv/bin/activate; echo "COMPLETED" # `pipenv shell` would hang because CI didn't see any return code
            # ... aws configure steps
            sls deploy --stage staging-preview


fixup SFINAE detection of boilerplate API

Fixup for #4 .
The previous version sadly defaulted to the second branch and simply spun until shutdown without calling run(). I only noticed this just now, sorry.
This bit of void_t based partial instantiation implements the proper check.
I learned quite a bit about SFINAE looking for short proper solutions to this.
@mlautman @JafarAbdi

Ping @mlautman @JafarAbdi .
Your upstream branch is functionally broken without easy introspection of the cause at the moment.
This patch fixes behavior and your CI passes.


无法在服务端运行

我自己编译的在服务端启动是会出现以下报错(项目中的gradle runServer)
··· 上午12:48:17: Executing 'runServer'...

Configure project :
Fabric Loom: 0.12.56
Loom is applying dependency data manually to publications instead of using a software component (from(components["java"])). This is deprecated and will be removed in Loom 0.13.

Task :compileJava UP-TO-DATE
Task :processResources UP-TO-DATE
Task :classes UP-TO-DATE
Task :generateDLIConfig
Task :generateLog4jConfig
Task :generateRemapClasspath
Task :configureLaunch

Task :runServer
[00:48:18] [main/INFO] (FabricLoader/GameProvider) Loading Minecraft 1.18.2 with Fabric Loader 0.15.1
[00:48:18] [main/INFO] (FabricLoader) Loading 39 mods:
- carpet 1.4.69
- dimthread 1.2.6-1.18.2-sand-duper-fix.3
- fabric 0.51.1+1.18.2
- fabric-api-base 0.4.4+d7c144a860
- fabric-api-lookup-api-v1 1.6.0+2373a54560
- fabric-biome-api-v1 8.0.4+7b0f312660
- fabric-command-api-v1 1.1.9+d7c144a860
- fabric-commands-v0 0.2.8+b4f4f6cd60
- fabric-containers-v0 0.1.21+d7c144a860
- fabric-content-registries-v0 3.0.3+d82b939260
- fabric-convention-tags-v1 1.0.1+474e4f7960
- fabric-crash-report-info-v1 0.1.11+3ac43d9560
- fabric-data-generation-api-v1 2.0.9+e19de87d60
- fabric-dimensions-v1 2.1.17+2540745460
- fabric-entity-events-v1 1.4.8+d7c144a860
- fabric-events-interaction-v0 0.4.19+d7c144a860
- fabric-events-lifecycle-v0 0.2.14+d7c144a860
- fabric-game-rule-api-v1 1.0.13+d7c144a860
- fabric-gametest-api-v1 1.0.16+91b7aa6660
- fabric-item-api-v1 1.4.0+4457765560
- fabric-item-groups-v0 0.3.11+91896a4960
- fabric-lifecycle-events-v1 2.0.3+2d83e92c60
- fabric-loot-tables-v1 1.0.11+d7c144a860
- fabric-mining-level-api-v1 2.1.0+81ea5b3c60
- fabric-networking-api-v1 1.0.21+d882b91560
- fabric-networking-v0 0.3.8+b4f4f6cd60
- fabric-object-builder-api-v1 2.1.0+fe4ddef060
- fabric-particles-v1 0.2.12+526dc1ac60
- fabric-registry-sync-v0 0.9.8+0d9ab37260
- fabric-rendering-data-attachment-v1 0.3.7+d7c144a860
- fabric-rendering-fluids-v1 2.0.2+54e5b2ec60
- fabric-resource-conditions-api-v1 2.0.3+91b7aa6660
- fabric-resource-loader-v0 0.4.18+2de5574560
- fabric-screen-handler-api-v1 1.2.0+1f6558e860
- fabric-transfer-api-v1 1.6.0+2373a54560
- fabric-transitive-access-wideners-v1 1.0.0+bd8a4a1860
- fabricloader 0.15.1
- java 17
- minecraft 1.18.2
[00:48:18] [main/INFO] (FabricLoader/Mixin) SpongePowered MIXIN Subsystem Version=0.8.5 Source=file:/home/zhenai/.gradle/caches/modules-2/files-2.1/net.fabricmc/sponge-mixin/0.12.5+mixin.0.8.5/8d31fb97c3e0cd7c8dad3441851c523bcfae6d8e/sponge-mixin-0.12.5+mixin.0.8.5.jar Service=Knot/Fabric Env=SERVER
[00:48:18] [main/INFO] (FabricLoader/Mixin) Loaded Fabric development mappings for mixin remapper!
[00:48:18] [main/INFO] (FabricLoader/Mixin) Compatibility level set to JAVA_16
[00:48:18] [main/INFO] (FabricLoader/Mixin) Compatibility level set to JAVA_17
[00:48:23] [main/INFO] (Minecraft) Environment: authHost='https://authserver.mojang.com', accountsHost='https://api.mojang.com', sessionHost='https://sessionserver.mojang.com', servicesHost='https://api.minecraftservices.com', name='PROD'
[00:48:27] [main/WARN] (Minecraft) Ambiguity between arguments [teleport, location] and [teleport, destination] with inputs: [0.1 -0.5 .9, 0 0 0]
[00:48:27] [main/WARN] (Minecraft) Ambiguity between arguments [teleport, location] and [teleport, targets] with inputs: [0.1 -0.5 .9, 0 0 0]
[00:48:27] [main/WARN] (Minecraft) Ambiguity between arguments [teleport, destination] and [teleport, targets] with inputs: [Player, 0123, @e, dd12be42-52a9-4a91-a8a1-11c01849e498]
[00:48:27] [main/WARN] (Minecraft) Ambiguity between arguments [teleport, targets] and [teleport, destination] with inputs: [Player, 0123, dd12be42-52a9-4a91-a8a1-11c01849e498]
[00:48:27] [main/WARN] (Minecraft) Ambiguity between arguments [teleport, targets, location] and [teleport, targets, destination] with inputs: [0.1 -0.5 .9, 0 0 0]
[00:48:28] [Worker-Main-18/INFO] (Minecraft) Loaded 7 recipes
[00:48:28] [Worker-Main-18/INFO] (Minecraft) Loaded 1141 advancements
[00:48:30] [main/INFO] (BiomeModificationImpl) Applied 0 biome modifications to 0 of 61 new biomes in 3.764 ms
[00:48:30] [Server thread/INFO] (Minecraft) Starting minecraft server version 1.18.2
[00:48:30] [Server thread/INFO] (Minecraft) Loading properties
[00:48:30] [Server thread/INFO] (Minecraft) Default game type: SURVIVAL
[00:48:30] [Server thread/INFO] (Minecraft) Generating keypair
[00:48:31] [Server thread/INFO] (Minecraft) Starting Minecraft server on *:25565
[00:48:31] [Server thread/INFO] (Minecraft) Using epoll channel type
[00:48:31] [Server thread/WARN] (Minecraft) **** FAILED TO BIND TO PORT!
[00:48:31] [Server thread/WARN] (Minecraft) The exception was: io.netty.channel.unix.Errors$NativeIoException: bind(..) failed: 地址已在使用
[00:48:31] [Server thread/WARN] (Minecraft) Perhaps a server is already running on that port?
[00:48:31] [Server thread/ERROR] (Minecraft) Exception stopping the server
java.lang.NullPointerException: Cannot invoke "wearblackallday.dimthread.util.ThreadPool.shutdown()" because the return value of "net.minecraft.server.MinecraftServer.getDimThreadPool()" is null
at net.minecraft.server.MinecraftServer.handler$zbl000$dimthread$shutdownThreadpool(MinecraftServer.java:1847) ~[minecraft-project-@-merged-named.jar:?]
at net.minecraft.server.MinecraftServer.shutdown(MinecraftServer.java) ~[minecraft-project-@-merged-named.jar:?]
at net.minecraft.server.dedicated.MinecraftDedicatedServer.shutdown(MinecraftDedicatedServer.java:565) ~[minecraft-project-@-merged-named.jar:?]
at net.minecraft.server.MinecraftServer.runServer(MinecraftServer.java:729) ~[minecraft-project-@-merged-named.jar:?]
at net.minecraft.server.MinecraftServer.method_29739(MinecraftServer.java:273) ~[minecraft-project-@-merged-named.jar:?]
at java.lang.Thread.run(Thread.java:840) [?:?]
Deprecated Loom APIs were used in this build, making it incompatible with future versions of Loom. Use Gradle warning modes to control the verbosity of the warnings.

Deprecated Gradle features were used in this build, making it incompatible with Gradle 8.0.
You can use '--warning-mode all' to show the individual deprecation warnings and determine if they come from your own scripts or plugins.
See https://docs.gradle.org/7.4.2/userguide/command_line_interface.html#sec:command_line_warnings
BUILD SUCCESSFUL in 14s
6 actionable tasks: 4 executed, 2 up-to-date
上午12:48:32: Execution finished 'runServer'.
以上内容只在服务段出现，把它装在客户端时可以正常运行(gradle runCline)

[00:48:31] [Server thread/WARN] (Minecraft) **** FAILED TO BIND TO PORT!

你的端口被另一个程序占用，或许是另外一个Minecraft服务器没有退出
修复了启动失败关闭服务器时的空指针异常


Milestone4 Change to Hooks

Hello,
in this milestone, bellow features have been made.

Refactor Calculator.jsx from a class-based component to a functional component with hooks.
Refactor App.js from a class-based component to a functional component.
No linters issue

@andersonlebon ,
Thank you for your review,
The linters is already configured but Github has any issue, it takes time to run the linters check after pull request.


SHA1 checksum mismatch for aquilon-protocols-1.10

Hi,
yum installation of aquilon using yum -y install aquilon-postgresql fails with:
http://yum.quattor.org/aquilon/aquilon-protocols-1.10-1.noarch.rpm: [Errno -1] Package does not match intended download. Suggestion: run yum --enablerepo=aquilon clean metadata Trying other mirror. Error Downloading Packages:   aquilon-protocols-1.10-1.noarch: failure: aquilon-protocols-1.10-1.noarch.rpm from aquilon: [Errno 256] No more mirrors to try.
I suspect createrepo command needs to be run in the repo because in primary.xml the checksum is
252c56f7707140b93c18c260b35b063e665b2f39
but
sha1sum aquilon-protocols-1.10-1.noarch.rpm
5aa885de61ab12819030d3841addf2fbd57fa6c2  aquilon-protocols-1.10-1.noarch.rpm

Hmm, so why did the checksum change?

I do not know. I have installed the rpm manually meanwhile and then yum proceeded happily.

Ok this is actually a duplicate of an issue I opened in the last workshop https://github.com/quattor/aquilon/issues/16

ok, closing this one then


Remove charset utf8

As discussed... Removing charset utf8 ;)

Should fix https://github.com/mrin9/RapiDoc/issues/248 :smiley:

@ckeboss
thank you... made the PR on my mobilephone and was to lazy to find the right issue :D

Thanks for the PR !!!

@mrin9
Thank you for the fast merge. Will you release a new version please? Would be awesome, as it would make it fully usable for our use case.
Thank you

next version release will be around mid of july.
meanwhile if you depend upon cdn to serve rapidoc, then you can pick it up from dist folder of github


ACVP: add test case for DRBG

[x] tests are added or updated

The Travis failure isn't relevant that I can tell -- evp_test is failing the sskdf which is unrelated.

ping @openssl/committers

Needs rebase

Rebased.

Ping for review ... this isn't essential for beta but should be in before 3.0 release.

Merged to master.
Thanks.


cassandra mountPath wrong

I find the cassandra.yaml use the wrong mountPath, it should be '/bitnami/cassandra' not '/var/lib/cassandra' ,please check it.

Hi I want to write same issue
Please Fix mount path of cassandra data it might cause to data lost of next deployment
for more info https://github.com/bitnami/charts/tree/master/bitnami/cassandra/#persistence


added a member of the abhikrti class without known name

please check this new prosodicPatterns entry too and consider whether we want to add it to EGD as well

@ajaniak : do you understand why the generic label and the pattern are not displayed?

I can't fond any abhikrti in the file. Have you pushed it?

oops! I'd forgotten...

I've checked and found it in good order. Added a comment confirming that it is not in Apte's appendix. I don't think there's a point in adding nameless metres to the EGD appendix. In fact, now that we have the authority file, we should indeed consider discarding that part of the EGD appendix altogether, since it's redundant.


Bugfix pfs coalesce

Fix bugs with the COALESCE operation for static-large objects where it dropped the metadata (headers) associated with the operation on the floor and failed to return the correct modification time.
Lack of headers broke multipart-upload in the S3API because the new object lacked the ACL headers so it could not be accessed via S3API.  It also dropped the S3API and SLO ETag values on the floor, which meant the object did not return the correct ETag later.
First commit: store selected metadata associated with the operation.  Mung the ETag headers os the request values are returned until the object is modified via SMB or NFS, at which point the ProxyFS generated ETag is returned.
Second commit: return the correct modification time for the COALESCE operaiton.

@tipabu can i get you take a look at my last set of changes and see if all of your concerns are resolved?
the two most controversial behaviors are probably this code in POST (post_object()) which allows certain ETAG values to be set if they were not set previously (which, i believe, is the previous behavior):
https://github.com/swiftstack/ProxyFS/blob/9174a329a59d70f76d1df0e486ebc74cc91e53e8/pfs_middleware/pfs_middleware/middleware.py#L1724-L1743
and these comments about the way ETAGs are contstructed which, again, i think matches existing behavior:
https://github.com/swiftstack/ProxyFS/blob/9174a329a59d70f76d1df0e486ebc74cc91e53e8/pfs_middleware/tests/test_pfs_middleware.py#L4569-L4581
but which leads to the need for this code:
https://github.com/swiftstack/ProxyFS/blob/9174a329a59d70f76d1df0e486ebc74cc91e53e8/pfs_middleware/pfs_middleware/middleware.py#L1736-L1739


Add UPDATE … IF EXISTS

Add ifExists clause to UpdateQuery so that this works:
Table.update().where(_.id eqs "good").modify(_.foo setTo "bar").ifExists

Hi @liff many thanks for your contribution. When you make your next one please feel free to add yourself to the contributors list.


Installation instructions for Fedora are outdated

Check header RHEL, Fedora and CentOS based distributions here:
sudo rpm --import https://packages.microsoft.com/keys/microsoft.asc
sudo sh -c 'echo -e "[code]\nname=Visual Studio Code\nbaseurl=https://packages.microsoft.com/yumrepos/vscode\nenabled=1\ngpgcheck=1\ngpgkey=https://packages.microsoft.com/keys/microsoft.asc" > /etc/yum.repos.d/vscode.repo'

Running the code above results in the following error on Fedora 26:
sh: /etc/zypp/repos.d/vscode.repo: No such file or directory
The fix on how to get it to work is on this page:
Create file /etc/yum.repos.d/vscode.repo with the following content:
[code]
name=Visual Studio Code
baseurl=https://packages.microsoft.com/yumrepos/vscode
enabled=1
gpgcheck=1
gpgkey=https://packages.microsoft.com/keys/microsoft.asc

For new users, it might be a good idea to update the website reflecting this.

Confused with the CentOS one... disregard.


add webpack-hud

implemented in https://github.com/FredericHeem/starhackit/commit/b9bcd5bccaf2b9a1f2a23f05373065f20b79ab57
Once all eslint warnings has been fixed, let's enable eslint within webpack to display warnings on page.

done


Roadmap intro should be updated

The first paragraph of the roadmap document says:

Our primary focus is to get the .NET Core 1.0 tooling to RTM quality for the Visual Studio 2017 RTM (please try the 2017 RC update we shipped with improvements in this area).

This is clearly not up to date and I think it should be updated.

Fixed by https://github.com/dotnet/core/pull/581.

Could we maybe also get more exact dates/months for the preview?  Q2 started this month right?


fuzz: fix coverity warnings

cf https://github.com/openssl/openssl/pull/20128#issuecomment-1423466180

This pull request is ready to merge

Merged to master branch. Thank you for your contribution!


fix: Stop showing dashed names when creating a classroom

Checklist:

[X] I have read freeCodeCamp's contribution guidelines.
[X] My pull request has a descriptive title (not a vague title like Update index.md)

Closes #167
Co-authored by: Michelle Tan<EMAIL_ADDRESS>Co-authored by: Fadi Buni<EMAIL_ADDRESS>Changed the title of the classes in the menu to title case format with no dashes.
Before:

After:

Hi there, thanks for taking on this issue!
I wanted to make a quick note that we did not want to get rid of the dashed names outright, we want to also have the title (non-dashed) available to us in order to display them to the user. We use the dashed names in other areas of the app (and eventually will use them in our DB) and switching everything to the title name could end up causing us to refactor other code that does not need refactoring.

@GuillermoFloresV
so do not get rid of the dashed names...
For the sake of UX, you want to modify dashedName (dashed names) to show up non-dashed inside the modal like the following:

In addition, did you wanted the first letter of every word to be uppercase?

@michellejtan
Yes, we do not want to get rid of the dashed names completely. For the sake of UX, we want to have the non-dashed names displayed inside of the modal as described in #167.
You shouldn't need to alter the strings in any way since freeCodeCamp provides the proper names already. See the comment that Utsab left on the issue I've linked.

@GuillermoFloresV
Thank you for the review. We have edited the file so we still conserve the dashedNames. I passed in another variable called displayName that holds the title we wanted. Let us know if this is what was intended.


Keyword order matching

Adds support fot Keyword anotation to override keyword name from method definition.
Adds ordering of methods in Keyword when working with overloaded keywords, so that
best match execution can be achieved and make execution of overloaded keywords deterministic

Currenly only very simple keyword ranking is implemented based on String count in keyword arguments.
It is needed to add more complex ranking instead of current one, but for main cases this should be enought.

Coverage decreased (-0.7%) to 83.972% when pulling 361b0d6ca4ef0e54ca013c314322f42fd92bc1e9 on KW-Ordering into 033e203d4c631dc29e1c3769cb2761ef98060a2d on master.


The range in the form on the website doesn't extend to all jokes by default

Describe the bug
The range selector is restricted to 0-191, however the jokes were recently updated to 207
Steps to reproduce
Press the down arrow on the range selector
Expected behaviour
207 should drop to 206
Instead it jumps to 191
Additional information
-From https://sv443.net/jokeapi/v2/

Hi, thanks for opening an issue!
I can't reproduce this. For me the range selector works as expected, both the minimum and maximum values go from 0 to 207 and clicking the down arrow goes from 207 to 206.
Please make sure you aren't browsing an outdated version from your browser's cache. To clear the cache, try pressing CTRL + F5 or enable the "Disable Cache" checkmark in the dev tools (by pressing F12 and going to the "Network" tab and then reloading the page while the dev tools are active).

Refreshing worked, I’ll be damned.
Thanks, and while I have your attention: I love your API. I made an app for an assignment, but I’m so happy with how it turned out that I’m sending it to everyone I know
From: Sven Fehler<EMAIL_ADDRESS>Sent: Sunday, 14 June 2020 22:25
To: Sv443/JokeAPI<EMAIL_ADDRESS>Cc: Joseph Julian Muscat<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [Sv443/JokeAPI] The range in the form on the website doesn't extend to all jokes by default (#107)
Hi, thanks for opening an issue!
I can't reproduce this. For me the range selector works as expected, both the minimum and maximum values go from 0 to 207 and clicking the down arrow goes from 207 to 206.
Please make sure you aren't browsing an outdated version from your browser's cache. To clear the cache, try pressing CTRL + F5 or enable the "Disable Cache" checkmark in the dev tools (by pressing F12 and going to the "Network" tab and then reloading the page while the dev tools are active).
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/Sv443/JokeAPI/issues/107#issuecomment-643817633, or unsubscribehttps://github.com/notifications/unsubscribe-auth/API4L3VI4BVHLA3XJTCVNP3RWUWZNANCNFSM4N5SI2SQ.

Aww, thank you very much :)


Explicitly cast condition results to bool

This fixes Issue #7.

Fixed in PR https://github.com/caxap/rest_condition/pull/10


ssl client certificate requests reponse status code is 400

Summary.
The server USES nginx to implement HTTPS
Expected Result
ssl client certificate requests reponse status code is 400
What you expected.
Certificate request
Actual Result
the nginx error is  21:unable to verify the first certificate
What happened instead.
Reproduction Steps
import requests
r = requests.get(url,
                 cert='0ea.pem'
                 )
print(r)

System Information
$ python -m requests.help

<paste here>

This command is only available on Requests v2.16.4 and greater. Otherwise,
please provide some basic information about your system (Python version,
operating system, &c).

Can you show us the contents of the .pem file please? Can you also provide the output of python -m requests.help as the template asked for?

From my test, I found that in ubuntu 14.04.3, this won't happened, but in ubuntu 16.04.1, 400 status code is replied when requesting HTTPS.
It's a clue, I haven't dig any deeper yet, but this may give you some hints.


[stable/postgresql] bump major version

What this PR does / why we need it:
Helm performs a lookup for the object based on its group (apps), version (v1), and kind (Deployment). Also known as its GroupVersionKind, or GVK. Changing the GVK is considered a compatibility breaker from Kubernetes' point of view, so you cannot "upgrade" those objects to the new GVK in-place. Earlier versions of Helm 3 did not perform the lookup correctly which has since been fixed to match the spec.
In https://github.com/helm/charts/pull/17281 the apiVersion of the statefulset resources was updated to apps/v1 in tune with the api's deprecated, resulting in compatibility breakage.
This major version bump signifies this change.
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md
[x] Title of the PR starts with chart name (e.g. [stable/mychartname])

LGTM

/lgtm


Use stat source id field for projected and actual points

Fix bug where projected and actual points were switched. #6

Codecov Report

Merging #13 into master will increase coverage by 0.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master      #13      +/-   ##
==========================================
+ Coverage   95.63%   95.64%   +0.01%
==========================================
  Files          12       12
  Lines         412      413       +1
==========================================
+ Hits          394      395       +1
  Misses         18       18

Impacted Files
Coverage Δ

ff_espn_api/box_player.py
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


[Darwin][AttestationVerifier] Expose a mechanism to customise cd signing keys and use it in darwin

Problem
This PR implements some parts of "Certificate Declaration verification need to consume real signing keys (aka #21802)" by exposing a virtual method AttestationTrustStore::GetCertificationDeclarationCert and updating the code so it uses this method if it is implemented or returns the default test certificate otherwise.
In this PR this method is only implemented for the darwin controller, but it could be easily extended to the ArrayAttestationTrustStore and the FileAttestationTrustStore.
Change overview

Add a virtual method AttestationTrustStore::GetCertificationDeclarationCert and makes it return CHIP_ERROR_NOT_IMPLEMENTED for some of the implementations.
Override AttestationTrustStore::GetCertificationDeclarationCert in the darwin implementation

Testing
It has been tested by manually toying with startup params in the darwin-framework-tool command line tool.

As long as the attestation trust store does not return CHIP_ERROR_NOT_IMPLEMENTED, the default key won't be used, right?

Since new public methods are added to common code, a chip-tool implementation must be provided, not just Darwin

I agree a chip-tool implementation would be nice here, but there are all sorts of APIs we have that are exercisable only by other SDK tools, some of them platform-specific (including Android).  This can't be a hard blocker for adding an API, though again I agree a followup adding a way to exercise via chip-tool would be good.

I have updated the patch with a chip-tool implementation if that helps to move forward !


Add unit tests for DefaultRazorIndentationFactsService

https://github.com/aspnet/Razor/pull/1697 adds some unit tests but we need a lot more coverage.

d4994eb4a451203f5cf6c5f7dc1543886f940b96


Remove s3_cli module from test_utils

In rust/src/test_utils.rs we have some shell invocations of the AWS cli which aren't necessary (IMHO) because we can do all of these operations directly from Rusoto without adding the system dependency.

Hey @rtyler, I started taking a look at this issue and I'm wondering if it's going in the right direction with my draft PR https://github.com/delta-io/delta-rs/pull/1570.
For now just creating the bucket with rusoto_s3 using their async API, but it required me to make a lot more functions async as well. Is that an issue? Or is there perhaps a better way of doing it? Curious for your  feedback!


Transpiled React code detects wrong names

        return (0, c.BX)(a.Fragment, {
          children: [
            (0, c.tZ)(t, {
              children: (0, c.tZ)(s.Z, {
                onSubmitError: (e) => {
                  (0, n.Iw)(
                    (0, i._N)("Something went wrong! [error]", { error: e })
                  );
                },
                children: w.map((e) => (0, c.tZ)(o.Z, { field: e }, e.name)),
              }),
            }),
          ],
        });

This is the simplified minified snippet for this original:
https://github.com/getsentry/sentry/blob/6bb5c0b800104c70456dc9fcddc7d94d5132fe90/static/app/components/modals/createReleaseIntegrationModal.tsx#L43-L100
The name is being detected as <object>.childrenchildrenonSubmitError, whereas it should probably just be <object>.onSubmitError.
The parent object is being passed as an argument to a function call at which point we should stop our search upwards the syntax tree.

I'll take this one. Do you think it should be <object>.onSubmitError or maybe rather children.onSubmitError? 🤔


centos 7 microk8s.join as a worker

Traceback (most recent call last):
File "/snap/microk8s/4595/scripts/wrappers/join.py", line 981, in
join(prog_name="microk8s join")
File "/snap/microk8s/4595/usr/lib/python3/dist-packages/click/core.py", line 722, in call
return self.main(*args, **kwargs)
File "/snap/microk8s/4595/usr/lib/python3/dist-packages/click/core.py", line 697, in main
rv = self.invoke(ctx)
File "/snap/microk8s/4595/usr/lib/python3/dist-packages/click/core.py", line 895, in invoke
return ctx.invoke(self.callback, **ctx.params)
File "/snap/microk8s/4595/usr/lib/python3/dist-packages/click/core.py", line 535, in invoke
return callback(*args, **kwargs)
File "/snap/microk8s/4595/scripts/wrappers/join.py", line 974, in join
join_dqlite(connection_parts, verify, worker)
File "/snap/microk8s/4595/scripts/wrappers/join.py", line 766, in join_dqlite
join_dqlite_worker_node(info, master_ip, master_port, token)
File "/snap/microk8s/4595/scripts/wrappers/join.py", line 834, in join_dqlite_worker_node
update_cert_auth_kubeproxy(token, info["ca"], master_ip, master_port, hostname_override)
File "/snap/microk8s/4595/scripts/wrappers/join.py", line 427, in update_cert_auth_kubeproxy
cert = get_client_cert(master_ip, master_port, "kube-proxy", proxy_token, "system:kube-proxy")
File "/snap/microk8s/4595/scripts/wrappers/join.py", line 281, in get_client_cert
subprocess.check_call(cmd_cert.split(), stdout=subprocess.DEVNULL, stderr=subprocess.DEVNULL)
File "/snap/microk8s/4595/usr/lib/python3.6/subprocess.py", line 311, in check_call
raise CalledProcessError(retcode, cmd)
subprocess.CalledProcessError: Command '['/snap/microk8s/4595/usr/bin/openssl', 'req', '-new', '-sha256', '-key', '/var/snap/microk8s/current/certs/kube-proxy.key', '-out', '/var/snap/microk8s/current/certs/kube-proxy.csr', '-subj', '/CN=system:kube-proxy']' returned non-zero exit status 1.

Hi @VenkySVR, can you please give a brief description of what you are trying to do along with minimal steps to reproduce it? Also, can you do a microk8s inspect and provide the tarball so we can look further into this?

I am trying create multi-cluster using microk8s, where my control plane is running in Ubuntu and I want to join slave node which has Centos.
In my slave node(centos):
[centos@slave-centos ~]$ microk8s join <IP_ADDRESS>:25000/ad67de6f54f916202e66237b1ec61f9b/90e27ec97245 --worker
Contacting cluster at <IP_ADDRESS>
Traceback (most recent call last):
File "/snap/microk8s/4595/scripts/wrappers/join.py", line 981, in
join(prog_name="microk8s join")
File "/snap/microk8s/4595/usr/lib/python3/dist-packages/click/core.py", line 722, in call
return self.main(*args, **kwargs)
File "/snap/microk8s/4595/usr/lib/python3/dist-packages/click/core.py", line 697, in main
rv = self.invoke(ctx)
File "/snap/microk8s/4595/usr/lib/python3/dist-packages/click/core.py", line 895, in invoke
return ctx.invoke(self.callback, **ctx.params)
File "/snap/microk8s/4595/usr/lib/python3/dist-packages/click/core.py", line 535, in invoke
return callback(*args, **kwargs)
File "/snap/microk8s/4595/scripts/wrappers/join.py", line 974, in join
join_dqlite(connection_parts, verify, worker)
File "/snap/microk8s/4595/scripts/wrappers/join.py", line 766, in join_dqlite
join_dqlite_worker_node(info, master_ip, master_port, token)
File "/snap/microk8s/4595/scripts/wrappers/join.py", line 834, in join_dqlite_worker_node
update_cert_auth_kubeproxy(token, info["ca"], master_ip, master_port, hostname_override)
File "/snap/microk8s/4595/scripts/wrappers/join.py", line 427, in update_cert_auth_kubeproxy
cert = get_client_cert(master_ip, master_port, "kube-proxy", proxy_token, "system:kube-proxy")
File "/snap/microk8s/4595/scripts/wrappers/join.py", line 281, in get_client_cert
subprocess.check_call(cmd_cert.split(), stdout=subprocess.DEVNULL, stderr=subprocess.DEVNULL)
File "/snap/microk8s/4595/usr/lib/python3.6/subprocess.py", line 311, in check_call
raise CalledProcessError(retcode, cmd)
subprocess.CalledProcessError: Command '['/snap/microk8s/4595/usr/bin/openssl', 'req', '-new', '-sha256', '-key', '/var/snap/microk8s/current/certs/kube-proxy.key', '-out', '/var/snap/microk8s/current/certs/kube-proxy.csr', '-subj', '/CN=system:kube-proxy']' returned non-zero exit status 1.
The microk8s inspect gives this log.
[centos@slave-centos ~]$ microk8s inspect
Inspecting system
Inspecting Certificates
Inspecting services
Service snap.microk8s.daemon-cluster-agent is running
Service snap.microk8s.daemon-containerd is running
Service snap.microk8s.daemon-kubelite is running
Service snap.microk8s.daemon-k8s-dqlite is running
Service snap.microk8s.daemon-apiserver-kicker is running
Copy service arguments to the final report tarball
Inspecting AppArmor configuration
Gathering system information
Copy processes list to the final report tarball
Copy disk usage information to the final report tarball
Copy memory usage information to the final report tarball
Copy server uptime to the final report tarball
Copy openSSL information to the final report tarball
Copy snap list to the final report tarball
Copy VM name (or none) to the final report tarball
Copy current linux distribution to the final report tarball
Copy network configuration to the final report tarball
Inspecting kubernetes cluster
Inspect kubernetes cluster
Inspecting dqlite
Inspect dqlite
Building the report tarball
Report tarball is at /var/snap/microk8s/4595/inspection-report-20230320_144342.tar.gz

Hi @VenkySVR. This sounds similar to an issue with the openssl command that was fixed sometime back https://github.com/canonical/microk8s/issues/3715. The PR with the fix is in https://github.com/canonical/microk8s/pull/3717, and has been backported to the 1.25 and 1.26 branches.
Due to a breaking issue we found and have since fixed, this bugfix has not yet found its way to the stable 1.25 and 1.26 branches. Can you try installing microk8s from the respective edge channel:
# if already installed (use 1.25/edge if on 1.25)
sudo snap refresh microk8s --channel 1.26/edge

# if not installed
sudo snap install microk8s --channel 1.26/edge --classic

And see if the error persists? Thanks a lot!


Additional records to be imported to ILS

We will need to import some 5000 new records to the ILS. Those are non CERN theses that are currently hidden from CDS because non CERN, but we want to keep them in the Library collections, so they will be imported to the ILS.
https://cds.cern.ch/search?ln=en&cc=Hidden+records&p=980%3ATHESIS+and+340%3Apaper&action_search=Search&op1=a&m1=a&p1=&f1=&c=Hidden+records&c=&sf=&so=d&rm=&rg=100&sc=1&of=hb
Since there is no THESIS document type in the ILS, a solution is to import them as BOOK with a specific TAG to be able to distinguish them from the rest of the other collections, unless it is possible to create a document type THESIS.
The idea would be to use the migration tool or the importer to do the importation, the records will be manually modified by the Library before the migration to make sure they are migrated correctly with the migration script.
Some 150 records have files attached, this could be handled manually, as discussed.

Here is the xml file of the CDS records (cleaned ):
hidden-theses2.xml.zip
But we need to import the corresponding items (I don't have these data, if you provide me with the list, I can include them in the XML if it helps).
As mentioned above, either we add a tag THESIS to allow us to distinguish them, or have a document type THESIS.

List of recids recids.xlsx


Provide a mechanism to configure airflow task configurations like pool, queue.

Description
There's a requirement from the user, where they need to put some additional requirements in the job spec. As the current Optimus version uses Airflow as the scheduler, then the focus for this issue is to address it in Airflow specific only. Others will follow if implemented.
Expectation

the user is allowed to put additional configuration specific to the Airflow pool and queue

Chosen Approach
The approach taken is to minimise changes in the job spec, but still accommodate the requirements. As it is specific to Airflow, then the focus is only for this.
The following is the approach taken for this issue, which will be implemented in the job spec. Example of the job spec:
...
behavior:
  ...
task:
  ...
metadata:
  resource:
    ...
  airflow: [struct]
    queue: queueA
    pool: poolA

Using struct for the Airflow is to make it easier to process.
References:

pool
queue


Add function for BYOK

new api: GetParametersForImport and ImportKeyMaterial
update function for user to import their own key into ehsm
only support for AES_GCM and SM4

test: pass unit test

Hi,
I have tried running test_kms_with_cli.py for testing and found that the ehsm_kms_server crashed when running test_import_key function.
The eHSM ffi_ImportKeyMaterial function returns early and then munmap_chunk(): invalid pointer is raised.
Could you please confirm that all unit test (except generateQuote and verifyQuote in Docker environment) is pass? I am not sure if it is a problem related to my dev env.

Hi,
I have tried running test_kms_with_cli.py for testing and found that the ehsm_kms_server crashed when running test_import_key function.
The eHSM ffi_ImportKeyMaterial function returns early and then munmap_chunk(): invalid pointer is raised.
Could you please confirm that all unit test (except generateQuote and verifyQuote in Docker environment) is pass? I am not sure if it is a problem related to my dev env.

Sorry, since the interface is in the process of modification and has not been perfected yet, I have fixed it. You can try again.

Thanks, test passed.

Hi,
I have tried running test_kms_with_cli.py for testing and found that the ehsm_kms_server crashed when running test_import_key function.
The eHSM ffi_ImportKeyMaterial function returns early and then munmap_chunk(): invalid pointer is raised.
Could you please confirm that all unit test (except generateQuote and verifyQuote in Docker environment) is pass? I am not sure if it is a problem related to my dev env.

Sorry, since the interface is in the process of modification and has not been perfected yet, I have fixed it. You can try again.


BUG: split_sequence error, possible HHsearch issue

split_sequence is producing errors for some targets, identifying part of the sequence name as a from-to number, e.g.:
Error in job split_PDB while creating output file /localscratch/microprot/microprot_gneg2-1885_403330/GRAMNEG_T1D_5168/02-split_pdb/GRAMNEG_T1D_5168.^[[0m
^[[31mRuleException:
ValueError in line 148 of /projects/microprot/microprot/snakemake/Snakefile:
invalid literal for int() with base 10: 'EG_T1D_51'
  File "/projects/microprot/microprot/snakemake/Snakefile", line 148, in __rule_split_PDB
  File "/projects/microprot/microprot/scripts/split_search.py", line 561, in mask_sequence
  File "/projects/microprot/microprot/scripts/split_search.py", line 273, in parse_pdb_match
  File "/projects/microprot/microprot/scripts/split_search.py", line 223, in _parse_hit_block
  File "/home/tkosciolek/conda/envs/microprot/lib/python3.5/concurrent/futures/thread.py", line 55, in run^[[0m
^[[31mExiting because a job execution failed. Look above for error message^[[0m
Trying to restart job for rule split_PDB with wildcards {'seq': 'GRAMNEG_T1D_5168'}

Example data and logs are on Barnacle in: /projects/microprot/tmp/split_sequence_errors

I think I fixed my bug, please review #56.
In essence: To determine the start column of the alignment content, I first split the line at \s+, picked the forth element and determined this position by searching for content in the line. This is necessary since split at \s+ might collapse several whitespaces.
The problem arose, when alignment content was e.g. only one column with context "n", but "n" also appeared in the sequence name, i.e. before the content :-/

resolved by PR #56


bug(mat-sidenav): There is no focus trap when Sidenav position is 'end'

Is this a regression?

[X] Yes, this behavior used to work in the previous version

The previous version in which this bug was not present was
11
Description
When you opens the Sidenav in over mode the focus shouldn't leave the Sidenav when it is open. it works in case the Sidenav opens in the left side. but in the right side it is broken.
Reproduction
Steps to reproduce:
stackBlitz

Open the Stackblitz example
choose Mode: Over
click on the toggle button
In the keyboard, click on the tab several times.

Expected Behavior
The focus shouldn't leave the Sidenav.
Actual Behavior
The focus leaves the side pane and there is no way to return to Sidenav with the keyboard.
Environment

Angular: 14.2.10
CDK/Material: 14.2.7
Browser(s): Chrome
Operating System (e.g. Windows, macOS, Ubuntu): Windows

Confirm that I this happens with v15. That's because the stackblitz uses v15


Q&A : How to Run Tesseract ON GPU

Can anyone help, i have NVIDIA Card 2GB delicated with Cuda Installed.
I want to setup Tesseract that it only use GPU memory not CPU.
Currently on training, i am getting these from system viewer:
80% + CPU USAGE

Please respect guidelines for posting issue: use tesseract user forum for asking support.


IRContext: replace uses of FindNamedOrDecoratedIds with use of the decoration manager that will live inside IRContext

I think this is done.


Proposal of a drought duration climate index

xclim version: 0.25.0
Python version: 3.7.6
Operating System: LinuxMint 20.1

Description
A drought duration climate index was developed (currently available in /scen_workflow_afr/indices.py). The algorithm is currently being tested (using climate data from West Africa). This index can be used to identify the impacts of climate change in countries where a monsoon is present.
What I Did
A call to the developed function returns an instance of xarray.DataArray which contains the number of days per year that are comprised in a dry period. 2 methods were implemented to define what a dry period is (based on a daily or total precipitation amount). Parameters allow adjusting detection rules in different countries or regions.
drydurtot ('1d' method): A dry period is a sequence of D_threshold consecutive days during which the daily precipitation amount (P_day) is lower than a threshold value (P_threshold).
drydurtot ('tot' method): A dry period is a sequence of D_threshold consecutive days during which the total amount of precipitation (P_total) is lower than a threshold value (P_threshold).
In both cases, the user can specify the days of year to consider (between day DOY_min and DOY_max).
What I Received
This section is not required.

How does this compare to : dry_spell_total_length ? It looks like this is similar to your tot method. Maybe, the 1d method could be added to the existing function?
https://xclim.readthedocs.io/en/stable/indices.html?highlight=dry_spell#xclim.indices.dry_spell_total_length
It was added quite recently in xclim.
Currently, we don't have many methods with the equivalent of DOY_min and DOY_max. I would try to think of a more general way to add this functionality. It is kinda similar in idea to the problem in #649 where we decided to not implement anything in xclim. Instead, users would mask the data themselves and disable/config missing checks accordingly.

@aulemahal It's indeed very similar to dry_spell_total_length. Adding the 1d method to the existing function would probably be done easily by replacing .sum() < thresh with .max() < thresh.
Regarding DOY_min and DOY_max, I'm not sure how that would work with the mask for the simple reason that I'm not familiar with this concept in xarray.  I need the analysis to be performed on all days of the year, while compiling only the days between DOY_min and DOY_max. For instance, if we have at least D_threshold consecutive days with P_day < P_threshold (occurring from doy 100 to 120), but (DOY_min==105) and (DOY_max==115), then the algorithm needs to compile 11 days (i.e., 115 - 105 + 1) out of the potential 21 days (i.e., 120 - 100 + 1). Do you think it could work with a mask?

The mask idea would simply be to put NaN on days 0 to 104, then the algorithm will be unable to find any dry spells before 105. And given that window <= 11, it would return 11 days as expected. Does that make sense?

@aulemahal My example was not complete. I forgot to assign a value to the parameter D_threshold. The two climate indices (drydurtot vs. masked dry_spell_total_length) would return different values if D_threshold == 14. With the current index (drydurtot), the number of compiled days would be 11 (because 21 > D_threshold), but zero with dry_spell_total_length (because 11 < D_threshold). So, I understand that masking would not work in that case.
I would not recommend adding the climate index drydurtot (or improved name) if there is already a very similar index in xclim (dry_spell_total_length), perhaps only the 1d method. Otherwise, that would be very redundant, and maybe not compatible width dask. In a case like this, a possibility would be to keep the index in the workflow (not in xclim). However, if this type of functionality is often required, I would be in favour of finding a global xclim solution (related to all climate indices).

I see! Indeed my proposed masking is not appropriate.
We could implement it. 2 things

we have a few indices where year-agnostic dates are given as strings, ex: 01-04 (Jan 4th),  I'm guessing this would be ok to use, instead of doys?
Because of the potential to reuse the date_min / date_max functionality, I suggest implementing some sort of mask_between_dates function (perharps in core.calendar?). For our specific case, if bounds are given, this would be called between the thresholding and the aggregation. Thus, spells are computed with the whole year, but only the days between the bounds are counted.

@aulemahal I attached the 30 unit tests that I used to verify the behaviour of an improved dry_spell_total_length climate index. I compared results between the current xclim algorithm (O) and the proposed algorithm (N), whenever a comparison was possible between the 2 algorithms (if the start_date and end_date did not impact results). This required adding the operator "max" in the existing xclim algorithm (locally). Each case was based on a 2-year artificial dataset. See the legend at the bottom of the spreadsheet. I don't always document my unit tests to this extent, but I thought it was the best way to document my observations. Let me know if you would like to talk about it.
dry_spell_total_length.xlsx


For giving intuitive way of managing shortcut conflicts

Changing the way to manage shortcuts in vs code

We do show the warning about conflicts if you use the keyboard shortcuts editor

But we dont show any warnings if you are editing the json file directly.
Are you referring to the keyboard shortcuts editor or the keyboard shortcuts json file?


change it's to its.

fixes issue #48 & #47 (sorry, didn't mean to repeat myself).

@mr-fixit thank you!


Does not sync at all

I tested it with Atoms license file and it does not do anything at all:

Atom 1.0.11
Windows 7

same issue occured on same os/atom version

Package is working for few first lines of .md document, then it isn't working.

It seems there won't be much development of this in the future: https://github.com/mark-hahn/markdown-scroll-sync/issues/33

I've released a new version 2.0.1 that is a total rewrite.  I think it fixes all issues and works much better than before.  Please try it.

Nice it syncs scrolling now.


Improve documentation of gpflow.utilities

(both docstrings and notebooks)

Cover the case of copying parameters of a model to another one
in the documentation for parameters_dict and multiple_assign,
https://github.com/GPflow/GPflow/issues/1119


fix(trace context): only initiate tracer context if exists

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Event when Configuration files are pulled for the first time？

你好，我想通过给ConfigService添加listener的方式，来结合SCG做动态路由功能，以监听路由配置文件（routes.json），但我发现服务刚启动后初次拉取到配置文件时，不会触发receiveConfigInfo方法，而只能是我之后在nacos控制台上手动修改一次内容才能触发回调通知。我想知道有没有方法能够让我在配置文件初次拉取到时就得到通知，从而好在服务启动时就自动注入路由，而无需人工手动修改一次文件才能触发第一次的路由设置。
Hi,
I want to inject dynamic routes with Spring Cloud Gateway by addListener to the ConfigService.
But, I found that the notify method 'receiveConfigInfo' doesn't work after the configuration file(routes.json) was pulled from nacos for the first time.
Then I modify the content of the file(routes.json) on the dashboard of nacos, I got the notify.
How Can I solve this problem? Is there any event or notification to be published when configuration files are pulled for the first time?

the version of spring-cloud-starter-alibaba-nacos-config is 2.1.2.RELEASE

I found a class named ConfigFilterChainManager in NacosConfigService.
I think this class would be helpful to me,  But I can't get it Cause it is private.
Any Idea or Suggestion?

Finally I solved this problem by call method 'getConfig' of ConfigService.
ConfigService configService = nacosConfigManager.getConfigService();
processConfigInfo(configService.getConfig(dynamicRoutesConfig.getDataId(),dynamicRoutesConfig.getGroup(), nacosConfigProperties.getTimeout()));
configService.addListener(dynamicRoutesConfig.getDataId(), dynamicRoutesConfig.getGroup(), new Listener() {
            @Override
            public Executor getExecutor() {
                return null;
            }

            @Override
            public void receiveConfigInfo(String configInfo) {
                processConfigInfo(configInfo);
            }
        });


Prepare release v0.9.0b1

Description

Update license copyright
Bump version

Code of Conduct
By submitting this PR, you agree to follow our Code of Conduct.

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

Do you want to automate setup.version here as well as you did for the Worker:
I just did it.

I noticed that the setup.py causes errors in tests. Because importing dependencies.
I decided to drop it. Not a required change.


ENH: update package version to 7.3.2

Hi! This is the friendly automated conda-forge-webservice.
I've started a version update as instructed in #170.
I'm currently searching for new versions and will update this PR shortly if I find one! Thank you for waiting!
Fixes #170

Hi! This is the friendly conda-forge automerge bot!
I considered the following status checks when analyzing this PR:

linter: passed
azure: passed

Thus the PR was passing and merged! Have a great day!


Cleanup duplicated code in repositories gradle builds

Describe the bug
There is a lot of duplicated code in Knot.x repositories that are part of Knot.x stack and use Gradle 5 with Kotlin to manage projects build.
Expected behavior
Code is packaged and distributed in one place (maybe new repo?) and re-used in projects.

This is fixed with https://github.com/Knotx/knotx-gradle-plugins.


pythonPackages.tensorflow: fix for bazel settings for intel mkl, dnnl

modified:   pkgs/development/python-modules/tensorflow/default.nix

Motivation for this change
Nix Tensorflow does not support optionally using the Intel provided optimizations for Intel MKL, which must be enabled through additional compile flags.
On modern Intel processors, this will generally provide improved performance and is  recommended in Tensorflow's performance guide.
NOTE: This change depends on MKL-DNN, the open PR found here: #68014
I've tested the changes on Linux with the above mentioned PR applied, both with an without Intel MKL enabled to confirm it works in both cases.  I do not have access to test on darwin, but believe that redirects to binary versions.  I would appreciate if someone could test with cuda, as this may need to be disabled when cudaSupport is enabled (I can make the change, but not test that myself).
With MKL enabled tensorflow.python.framework.test_util.IsMklEnabled() returns True
Things done

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

[ ] NixOS
[ ] macOS
[x] 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 nix-review --run "nix-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Notify maintainers
cc @jyp @abbradar

This is blocked on #68014, which has already been reviewed by people with commit access but for some  reason not merged. I don't have time to review that right now.
This PR looks good to me, as long as mkl and dnnl are open source this should be a clear improvement :)

Although it would be nice if you could point out where exactly those guidelines are. You just linked to the benchmarks folder?

@timokau It looks like Google pulled that page since releasing Tensorflow 2.0 and now is redirecting it.  It's possible to use wayback to look at the historical page, but here is Intel's page on the same topic.  And, FWIW, it seems Anaconda now defaults to using MKL.
Having these libraries should generally improve performance for those using CPUs.  While the MKL-DNN would generally qualify under most definitions of free (>= 1.x), the core MKL is shipped as binary so would not be available in Nix by default and has to be enabled per the documentation.  For many people and Hydra, this change will be a no-op.  But for those performance focused, it would be easily available in a similar way to PyTorch and other libraries once they enable MKL in their site configuration.
It looks like you have 1.15 and 2.0 PR updates in progress, so I'm fine with this being rolled into those (I just don't want it to get lost).  Tensorflow 2.0 now enables Cuda by default and I'm unsure of the interaction with MKL and the Cuda piece, so it may be that MKL should only be enabled when Cuda is not enabled.  How you want to handle this depends on how you're planning to approach the "enabled by default" for Tensorflow 2.0 in Nix.

Ah right, I was in a bit of a hurry and didn't realize that users have to opt-in for mkl. Then the only downside I see here is that people opting in for mkl will have to rebuild tensorflow from source, which can take quite some time. Its probably a reasonable assumption that people that care enough for performance to enable mkl are willing to do this, but we should still give people that only want mkl for numpy (which doesn't take quite as long to rebuild) a way out.
So I suggest that instead of directly depending on numpy.blasImplementation, you instead add a new parameter mklSupport that defaults to numpy.blasImplementation = "mkl". That way people can still make an overlay and unconditionally enable or disable it.
Once that is done and the dnnl PR is merged I see no reason to hold this back. Doesn't have to be merged with the 1.15 update. That update could be blocked for as long as upstream needs to adjust tensorflow for bazel 1.0. Its a mystery to me why bazel would make its first major release without making sure that at least its biggest public consumers don't break, but that is a different question.
I think there should be no bad interaction with cuda, although if you have the hardware it would be nice if you could test that. The "cuda enabled by default" question is a difficult one, which I'll postpone to the actual 2.0 update (keeping it disabled for now).

So I suggest that instead of directly depending on numpy.blasImplementation, you instead add a new parameter mklSupport that defaults to numpy.blasImplementation = "mkl". That way people can still make an overlay and unconditionally enable or disable it.

I suppose tensorflow cannot work with MKL if numpy is built without? And why would you build without MKL if numpy is built with already? Note one has to perform a rebuild anyway because the hash of numpy changes when enabling MKL.

I suppose tensorflow cannot work with MKL if numpy is built without?

I'm not sure.

And why would you build without MKL if numpy is built with already? Note one has to perform a rebuild anyway because the hash of numpy changes when enabling MKL.

Right, good point. So in that case this PR should be fine as-is, as soon as the dependency is merged.

I suppose tensorflow cannot work with MKL if numpy is built without?

I'm not sure.

You probably don't want to mix math libraries in your Python stack.  If MKL is used, it should be applied consistently.

And why would you build without MKL if numpy is built with already? Note one has to perform a rebuild anyway because the hash of numpy changes when enabling MKL.

Right, good point. So in that case this PR should be fine as-is, as soon as the dependency is merged.

I attempted to test the enableCuda branch, but it looks like the recent merge of Bazel 1.0 (#69252) now breaks tensorflow since that version of Bazel is known to be incompatible.
You reference this in the WIP PRs, but I think it affects even the master branch.

I attempted to test the enableCuda branch, but it looks like the recent merge of Bazel 1.0 (#69252) now breaks tensorflow since that version of Bazel is known to be incompatible.
You reference this in the WIP PRs, but I think it affects even the master branch.

Right, we'll have to wait for upstream on this. Looks like someone is on it. Of course if you want to take a shot at fixing this yourself, feel free to :)

Note there is a PR for Bazel 1.1.
https://github.com/NixOS/nixpkgs/pull/71612

Bazel has promised to adhere to semver (and leave at least 3 months between breaking changes), so that shouldn't make the situation worse than it is. I would've preferred blocking the 1.0 update on tensorflow compatibility, but now I hope that the upstream support will come soon.

I'm also a bit surprised tensorflow was taken by surprise on that. Good a fix is in sight.
On the other hand, I also want to note bazel was merged to master, not release-19.09, and we call the channel nixos-unstable, so I'm personally comfortable with breaking some things, if the result is something better - we don't really block people relying on stability.

Is this still needed? Note there's a merge conflict.

Yes, this is still desirable and it looks like the prerequisites should now be in place to allow forward progress.  If you have time to review, I'll prioritize resolving the PR conflict over the coming week.

@FRidh Merge conflict resolved. Retested for cpu, cpu-mkl, and cuda-mkl on Linux.
This is ready for review.
In [1]: from tensorflow.python.framework.test_util import IsMklEnabled
In [2]: IsMklEnabled()
Out[2]: True

I will apply this to both versions of tensorflow after the upgrade.

Thanks for the rebase. Did you ever try TF_NEED_MKL to match how cuda does it instead of the flag? I am trying that now.

Thanks for the rebase. Did you ever try TF_NEED_MKL to match how cuda does it instead of the flag? I am trying that now.

No, I followed the instructions from here: https://www.tensorflow.org/install/source
I believe TF_NEED_MKL will download MKL, although I have not tried that approach.

I dunno why ofborg didn't run; this is definitely fine though.

@Ericson2314 IIRC mkl has a unfree license

and recently, ofborg is very slow (10+ hrs) in eval'ing a PR

It's indeed been accumulating jobs https://monitoring.nix.ci/d/000000004/evaluation-jobs-over-last-7-days?orgId=1&from=1585505457363&to=1585678257363


Could not pass CollideEntityEvent$Impl to Plugin

Using spongevanilla-1.11.2-6.0.0-BETA-243 I get this spamming console hard:
GPERROR.txt
Help?

Update your GriefPrevention.


Build failure: gnuplot_qt

Steps To Reproduce
Steps to reproduce the behavior:

build gnuplot_qt

Build log
make[2]: Leaving directory '/build/gnuplot-5.4.6'
make[1]: Leaving directory '/build/gnuplot-5.4.6'
<stdin>: In function 'main':
<stdin>:25:6: error: #error makeCWrapper: Unknown argument --run
<stdin>:26:6: error: #error makeCWrapper: Unknown argument . /nix/store/rwgvww93dwp0dyzmhwjyd9p6nd95wiqh-set-gdfontpath-from-fontconfig.sh
/nix/store/rqhrzja0pngqlgcfd5wd0jy9ircc73jf-make-binary-wrapper-hook/nix-support/setup-hook: line 136: pop_var_context: head of shell_variables not a function context
error: builder for '/nix/store/k1hs6a9l7vcpikrn5cm2x2p7xalwsiq9-gnuplot-5.4.6.drv' failed with exit code 1;

Additional context
Caused by #225881. I'm working on a fix.
Notify maintainers
@lovek323

Will be fixed by #229154.


Create .github repo with README

Checklist

[X] I added a descriptive title
[X] I searched open requests and couldn't find a duplicate

What is the idea?
We'd like to add a org level README.md to link conda-incubator, conda, and conda-archive. Can also link in other communication channels (e.g. discord, slack, twitter, etc.).
Why is this needed?
Improves connections between the different parts of the conda ecosystem and boosts discoverability.
What should happen?
Create .github repo and add README.
Ref:

https://github.blog/changelog/2021-09-14-readmes-for-organization-profiles/
https://docs.github.com/en/organizations/collaborating-with-groups-in-organizations/customizing-your-organizations-profile#adding-a-public-organization-profile-readme

Additional Context
No response

I’ve created the repo and added it to the “conda-core” team’s repos, to kick things off


Bosun/scollector Metrics don't jive with PerfMon

We're looking at using scollector/bosun to monitor a cluster of calc servers. After running a sanity test, we were comparing stats collected via perfmon with stats viewed in bosun. In most cases, after compensating for sampling periods etc., most of the stats aligned acceptably. However, in some cases,  there's just no correlating the data. I am assuming we're just not comparing apples-to-apples, but I could really use a pointer on how to validate these outliers. The simplest example is Processor Queue Length. The following is our rendering of data collected via perfmon and bosun/scollectors on the same host at the same time. Again, I would totally expect some variances on account of collection periods and aggregation, but this one seems inexplicable to me)
PerfMon Collected Data

Bosun Collected Data
(via ./tsdb query 1h-ago avg win.system.cpu_queue)

I am not too familiar with the internals of the Windows, but as I was looking at the data, it almost seemed like the counter might be a reset-on-read, so the perfmon monitor was getting some of the data and scollector was getting the rest. Is that possible ?
Thanks for any insight you can give me.
//Nicholas

This particular metric is a gauge. That means we get a "snapshot" from scollector every 15 seconds. Unless they Perfmon and Scollector were running at the exact same frequency, and are Exactly in sync you will not get the same results.
So I would say this is expected. If this is not a good enough explanation you need to set scollector to a default freq of every second (I'd filter so you are only getting this particular collector), perfmon to every second, and things still look very different we dig deeper. If not I'll just close this.

Also worth mentioning. Whenever it sensible not use a gauge, we go into the raw performance counters to get the data so we don't have that "missing" information. Think that go up and down generally don't work as a counter though, such as disk space or a queue. That is because every increment and decrement would need to be captured, and that isn't sane.

Got it. So I was looking at the code to figure out how to change the frequency. Do I need to rebuild ?  'Cause there's no command line option for frequency......

scollector -h
....
-freq="15": Set the default frequency in seconds for most collectors.
...

Most collectors because some are hardcoded. But those are things we know we don't want to check very often. For example the one that runs yum to look for updates since yum locks, and takes a long time to run. This one probably uses the default freq

Oh wait, sorry the binary on our website might not have that feature yet...
So you may need to pull from github and build it. Is that hard for you? I
can looking into pushing scollector binaries today if it is.
On Wed, Mar 4, 2015 at 2:34 PM, Nicholas Whitehead<EMAIL_ADDRESS>
wrote:

Got it. So I was looking at the code to figure out how to change the
frequency. Do I need to rebuild ? 'Cause there's no command line option for
frequency......
—
Reply to this email directly or view it on GitHub
https://github.com/bosun-monitor/bosun/issues/758#issuecomment-77229135.

I got it. Thanks Kyle !

The win.system.cpu_queue data currently comes from the Win32_PerfRawData_PerfOS_System WMI class, but as Kyle said it is a Gauge that changes very quickly so is difficult to match when you have different sample frequencies.
I tested in powershell using the below command and it seems to follow what permon.exe shows for Processor Queue Length, so the differences you see are most likely do to sampling and/or the fact that Gauge changes VERY quickly.
while($true){gwmi Win32_PerfRawData_PerfOS_System | select -ExpandProperty ProcessorQueueLength;sleep -Milliseconds 500}

Thanks for the info @gbrayut.
I think we're all set, so closing this issue.
Thanks all.


Request for documentation: FasterTransformer vs TensorRT

Sorry if this is covered somewhere; I looked through the docs and issues and couldn't find it.
From what I can tell, FasterTransformer lets you run PT or TF transformer models, but by swapping out and optimizing low-level primitives, allows it to go much faster. NVIDIA also makes TensorRT, which is a different format, but has the same goal of speeding up transformer inference.
So my request is: can you explain in the docs if FT and TRT compete (or am I mistaken and both can be used), and if so, when to pick one or the other? If I have a PyTorch BERT model and I am going to serve it with Triton, when would I choose to use FT vs converting it to TRT?
Thank you!

The goal of FT and TRT are different. TRT provides stable and automatically acceleration, while FT hopes to support latest model and feature in short time. Important features of FT will also be supported in TRT in the future, but it require longer time.
So, no simple solution to reply which one is better. If users want to use some models which are supported well in TRT, then using TRT is better. Bert is an example here.
On the other hand, if users want to use some models which are not supported in TRT or the optimization of TRT is still not good enough, then using FT is better in short time.

This is great information, and I really appreciate the prompt response. I'm happy to open a PR to the README with that information — I think it would help.

One advantage of TensorRT is that it can be used as the Execution Provider for OnnxRuntime. If TRT could support FasterTransformer automatically ONNX would benefit from the optimizations.

Close this bug because it is inactivated. Feel free to re-open this issue if you still have any problem.


Silently Download Previous thread

Whan User login agian SDK start downloading previous thread but not give any events i(ThreadAdded,ThreadDetailsUpdated etc).
Also not give any signal whan all Previous threads is download.

@09009090029 are you using the latest version of the chat?

Closing due to inactivity.


[TRY] Change zero_copy::Parser to take output type as generic type

Hello @zesterer,
This is a tentative at https://github.com/zesterer/chumsky/issues/9#issuecomment-1292788564 , indeed, it wasn't an easy change,
What I did basically (in a few more steps):

Change zero_copy::Parser to take O generic type, and remove its Output associated type. (start review here I think)

Go through all errors one by one and fix all combinators, primitives, etc... until cargo build [...] is happy.
I've left a number of FIXMEs, because I had the error E0207 (a lot):
For a code like this (simplified):
impl<A, OA, ..., O> Parser<.., O, ..> for Map<A, F>
where
    A: Parser<.., OA, ..>,
    F: Fn(OA) -> O,
    ...

I get:
the type parameter `OA` is not constrained by the impl trait, self type, or predicates
For some reason the compiler doesn't see that OA is used in A, which is used for Map<A, F> (Am I missing something? Is it a known limitation/bug(?) of the compiler?).
To fix this I had to add a lot of additional PhantomData to combinator structs (like Map<A, OA, F>) to ensure OA (and others) are mentioned in either the impl trait or self type.
Another thing: I didn't understand how to edit the macro impl_group_for_tuple, so it's not called for now (commented), I've commited what I thought would work, could you help me with that?

Try to run the tests, fix (AFAIK) the callers to use the parsers with the new O generic type.

=> Now I'm blocked at an error I don't understand how to fix...:

error[E0599]: the method `or` exists for struct `zero_copy::combinator::MapSlice<zero_copy::combinator::Repeated<zero_copy::combinator::Filter<zero_copy::primitive::Any<_, _, _>, [closure@src/zero_copy/mod.rs:799:21: 799:31]>, char, _, (), _, _>, fn(&str) -> Token<'_> {Token::<'_>::Ident}, _, _>`, but its trait bounds were not satisfied
   --> src/zero_copy/mod.rs:810:14
    |
810 |             .or(string)
    |              ^^ method cannot be called on `zero_copy::combinator::MapSlice<zero_copy::combinator::Repeated<zero_copy::combinator::Filter<zero_copy::primitive::Any<_, _, _>, [closure@src/zero_copy/mod.rs:799:21: 799:31]>, char, _, (), _, _>, fn(&str) -> Token<'_> {Token::<'_>::Ident}, _, _>` due to unsatisfied trait bounds
    |
   ::: src/zero_copy/combinator.rs:5:1
    |
5   | pub struct MapSlice<A, F, E = (), S = ()> {
    | -----------------------------------------
    | |
    | method `or` not found for this struct
    | doesn't satisfy `_: zero_copy::Parser<'_, _, _, _, _>`
    |
note: trait bound `zero_copy::combinator::MapSlice<zero_copy::combinator::Repeated<zero_copy::combinator::Filter<zero_copy::primitive::Any<_, _, _>, [closure@src/zero_copy/mod.rs:799:21: 799:31]>, char, _, (), _, _>, fn(&str) -> Token<'_> {Token::<'_>::Ident}, _, _>: zero_copy::Parser<'_, _, _, _, _>` was not satisfied
   --> src/zero_copy/blanket.rs:5:8
    |
3   | impl<'a, T, I, O, E, S> Parser<'a, I, O, E, S> for &'a T
    |                         ----------------------     -----
4   | where
5   |     T: Parser<'a, I, O, E, S>,
    |        ^^^^^^^^^^^^^^^^^^^^^^ unsatisfied trait bound introduced here
note: the following trait must be implemented
   --> src/zero_copy/mod.rs:177:1
    |
177 | pub trait Parser<'a, I: Input + ?Sized, O, E: Error<I> = (), S: 'a = ()> {
    | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
    = help: items from traits can only be used if the trait is implemented and in scope
note: `zero_copy::Parser` defines an item `or`, perhaps you need to implement it
   --> src/zero_copy/mod.rs:177:1
    |
177 | pub trait Parser<'a, I: Input + ?Sized, O, E: Error<I> = (), S: 'a = ()> {
    | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

(screenshot for colors)

This errors indicates that zero_copy::Parser is not implemented for zero_copy::combinator::MapSlice, but it is? The error mentions the blanket impl of zero_copy::Parser for all &'a T, but I don't understand why this is needed (there's not blanket in master branch afaik), and what is its relation to the error message..

I hope this work will be useful to you, at least to start rolling the ball for this, learn what not to do, .. :)
I'm happy to continue working on this if you manage to understand this error / unblock the situation, I learned a lot here!
Please tell me if you see any improvement to the structure of the changes, I'll propagate suggestions as needed.

I've just made this implementation of impl_group_for_tuple and I think it should work.

This is a great start!

I get:
the type parameter OA is not constrained by the impl trait, self type, or predicates
For some reason the compiler doesn't see that OA is used in A, which is used for Map<A, F> (Am I missing something?

The reason for this error is that OA is only used in the trait bounds, not in the typing relationship itself. To solve this, you'll likely need to add an OA type parameter to the parser itself (backed by PhantomData).
If you wish, I can try to take this branch over when I next have time (probably in a few weeks).

I've just fixed the MapSlice implementation in https://github.com/herkhinah/chumsky/tree/shekhinah/zero-copy-MapSlice


Store ik-oang-kaar as 1Eå instead of <> for improved accuracy

Would impact character counts and allow the user to select any part of the word

can you explain why there is improved accuracy this way?

ikoangkaar is composed of a number, letter and symbol so this would accurately reflect the 3 characters individually combining to form the word
firstLetter would return ੧ instead of ੧ਓ
render ੧ and ਓ separately, so for a santhiya app, you could add dashes: 1-Eå, A-lK, etc.
user can select ੧ and ਓ separately with their cursor which can be more useful

I don't see the value, but we should add this an option


Add monitoring hooks into KituraNet HTTP

#840 allows server-side monitoring hooks to see requests and responses flowing through a Kitura server. As an extension of that work, SwiftMetrics would like to be able to gather metrics about outbound HTTP requests/responses that are fired from a Swift Application using KituraNet's HTTP/ClientRequest object. I think the easiest thing to do to achieve this is to store some kind of delegate that wraps the callback set in the ClientRequest initialiser e.g.:
if (monitoringDelagate.exists())
self.callback = monitoringDelegate.generateCallback(callback)
else
self.callback = callback.
This would enable the monitoring delegate to intercept the ClientResponse.
FYI, the kind of metrics we're hoping to get (taken from https://github.com/RuntimeTools/appmetrics)
Event: 'http-outbound'
Emitted when the application makes an outbound HTTP request.
data (Object) the data from the HTTP request:
    time (Number) the milliseconds when the request was made. This can be converted to a Date using new Date(data.time).
    method (String) the HTTP method used for the request.
    url (String) the URL on which the request was made.
    contentType (String) the HTTP response content-type.
    statusCode (String) the HTTP response status code.
    duration (Number) the time taken for the HTTP request to be responded to in ms.

Tagging people who might be interested: @tobespc @seabaylea @shmuelk

This sounds quite do able.
However. There is a big push to get rid of KituraNet.ClientRequest and KituraNet.ClientResponse and to move to URLSession and friends. This is will very true in the Swift 3.1 as it seems that they are finally getting the bugs out of URLSession and DispatchSource.

Hmmm. Perhaps it would be better hung from KituraNet.HTTP then?

If you mean KituraNet.HTTP.request(), then that's the same as ClientRequest and ClientResponse.

Closing as no longer in our plans.


enable pyensight api coverage

With the previous change the coverage of the pyensight API got disabled.
The PR fixes it.
PyEnSight, compared to the other projects, builds the 95% of its code in the API repo. This is because PyEnSight replicates already existing Python APIs in EnSight via introspection.
In the previous PRs it was decided to let anything that is automated (in this case, pure Python scripts) in the API repo. However, since the API repo, as mentioned, does not generate only grpc bindings but also actual code, it is important that this is reported in the code coverage

Codecov Report

Merging #262 (e98a4e5) into main (01738a2) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #262   +/-   ##
=======================================
  Coverage   64.21%   64.21%
=======================================
  Files          17       17
  Lines        2540     2540
  Branches      447      402   -45
=======================================
  Hits         1631     1631
- Misses        765      767    +2
+ Partials      144      142    -2

see 2 files with indirect coverage changes

@jorgepiloto how can I change the baseline for CodeCov?
Because it was missing the definitions of the coverage from ansys.api.pyensight, CodeCov still reports that the coverage has not changed, but that's not the case

We can try to remove all data in Codecov and just trigger it again using the CI. This should fix the issue. It will only loose coverage history from this pull-request backwards.

@jorgepiloto that would be great

It's done. Let's give it a second try.

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@01738a2). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #262   +/-   ##
=======================================
  Coverage        ?   64.13%
=======================================
  Files           ?       17
  Lines           ?     2540
  Branches        ?      447
=======================================
  Hits            ?     1629
  Misses          ?      769
  Partials        ?      142

@jorgepiloto still 64%. With the coverage of the API it should bump to 98%. I believe there's something we need to change so that it takes not only the src folder for coverage but also ansys/api/pyensight

On monday I will look at the codecov action parameters and see if we can do something for it

@jorgepiloto @randallfrank
I have looked for documentation on both the codecov action and the codecov website on covering two repositories simultaneously with one upload.
I haven't found a solution.
So, I believe we can start merging this, that at least enables the coverage of the api repository during the pytests (if we are interested one can look at the logs there), while after this has been merged I'll work on increasing the coverage of the pyensight repo itself. It was higher than 80% when I created the first batch of tests, but we have added a few files. It shouldn't take me too much to increase it. But I need this to go in first


session.Engine change to  session.engine, why?

From v0.6.3, session field 'Engine' change to private 'engine'？
In a function, I only has session, and I want to use some engine function, such as Engine.IdOf, now I can not find a function to settle it. （This function we use in many places, we can pass the engine parameter to the function, but it may modify many codes.）

👍


Update to oneDNN v3.0

Description

Motivation and Context

I think the code change is great. Almost everything I did was ask some questions. I think I only suggested a single change.
The other thing that you may want to decided is if some of the line lengths should be broken across a few lines more lines. Some are long enough that they are auto wrapping when doing the side by side comparison. I personally try to make it possible to do side-by-side comparison without random line wraps. I am aware this is a personal choice but I would encourage browsing the changes and deciding if an added line wrap would improve readability in a side-by-side compare.


Add Back EMF and Percent Power Output to Display

Both can be calculated/obtained from our motor controller.

@IshDeshpa What do you hope to gain (from a driver's perspective) by adding these? I would warn against making the driver consume too much information.

The BackEMF would replace the accelerator slider at the bottom we have currently, as is present in most hybrid cars. Power percentage may not be completely necessary, but it could be put in the corner.

You have limited screen real estate. It may in fact be more prudent to use any excess to size more important information larger, as you can't make assumptions about future placements, and it needs to be more versatile.

Power is pretty important information, it'll let the driver know if they're being too heavy on the accelerator.


Chat responses are not read

Testing #188537

Type a request, press enter
Nothing from the response is ever read

I think it's supposed to do this?

Are you in screen reader optimized mode?

also any errors in dev tools console?

here's the problem https://github.com/microsoft/vscode/blob/d0a179cf012f40fb50172a40ac9b7dba4f6431ba/src/vs/workbench/contrib/chat/browser/chatAccessibilityService.ts#L44

cc @rperez030 and @jooyoungseo this is the error you were encountering. should be fixed in insider's tomorrow

regression caused by the fix for https://github.com/microsoft/vscode/issues/188417


Add 'flit develop' command

This adds a flit develop subcommand which acts the same as flit install --symlink with the exception that it also installs any development requirements that are specified in the flit.ini file (under develop-requires).
I'll add tests and update relevant docs if you think this looks ok.

+1

Note from :beers: , check if/how to use test-requirements metadata for that.

I'm happy with the basic idea, but I'm still thinking a bit about what the interface should look like.

Ok, updated as per our conversation in person. Got rid of flit develop and modified flit install so that it also installs any development requirements, if they are given. Additionally, I added -s to the --symlink flag so that you can do flit install -s to get a proper development install rather than having to type the full flit install --symlink.

Thanks, I think this looks good.
I'd quite like to have a way to control which dependencies get installed before we release again. You're welcome to work on this if you'd like to, or I'll hopefully get to it soon.


Use --break-system-packages with pip commands

Testing with https://github.com/gazebosim/gz-math/pull/587.

sorry I didn't notice #69! I could have used that instead, but thanks for merging this


Implement mentor availability endpoint

Purpose
The purpose of this endpoint is for the mentor to update their availability
Goals

Approach

Screenshots

Checklist

[x] This PR doesn't commit any keys, passwords, tokens, usernames, or other secrets.
[ ] I have read and understood the development best practices guidelines ( http://bit.ly/sef-best-practices )
[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation

Related PRs

Test environment

Learning

I havent written the tests yet. Wanted to make sure the endpoint is fine. Will do once approved. Also im not sure about the route @anjula-sack


Table of contents support

Hey!
First of all, thanks for this amazing tool, just what I've been needing for a very long time.
So question, is there any way to add an auto-generated Table of Contents e.g. the [TOC] markup/macro that's isn't really standard markdown but can be found supported in a few implementations.

Hey there @telamon, a table of contents is definitely something I'd like to include! Sadly at the moment we're blocked by a bit of an annoying bug in Phantomjs which means that all our links are being rendered as external links. This issue has been raised as #12.
The fix to this seems to be to use something like Puppeteer but I've not had time to look at this yet as it's a pretty huge job for mdpdf.
I'm going to apply some labels to this and I'll come back to it when I've managed to implement a new bottom level rendering system, at that point we should have greater control over how URL's are transformed! (If this sounds like something someone wants to pickup, then they're more than welcome to give it a crack.)
I'm really glad you're enjoying the tool, if you've got any other feedback then I'd love to hear it.

I see. Well mdpdf is quite useful in its current state as well.


[FR] Grey palette

By using the accent color to style the theme, the hue value for grey colors (red = green = blue) is always 0 (red). This is therefore no possible to create a grey theme.
Would it be possible to add this feature in the style settings? I can think of two ideas, but they are just suggestions:

a toggle that just set any saturation to 0 (except maybe for some text you want to see accented, such as links) --> when enabled, this will make the "Background-to-text contrast" only change the lightness
separate the slider "Background-to-text contrast" into two sliders saturation/lighntess and let the possibility to slide down the saturation to 0

@ElsaTam I was previously considering this!
Perhaps I can make a toggle in the settings for a "Grey mode", where the background and everything that isn't accented is grey?
But then I'd have to consider whether to make the accented stuff their original hue instead of the altered hue... 🤔

@Erallie Basically, that would just be the default color palette from Obsidian that is all grey. Except for accented colors. But yeah, I think that would be it, if you want to make this feature.
As I said, it was more a thought in case this is something that you want because I saw a possible conflict. But that's like giving the possibility to remove half of the signature of your theme.

But that's like giving the possibility to remove half of the signature of your theme.

@ElsaTam It is certainly removing half of the theme signature lol. Changing the accent colors back to default would require a little bit more work than just using the altered accent colors, since I would need to define more variables for it.
Maybe I can allow an option for it, but make the accent colors still have the same saturation and lightness as the new accent colors...

@Erallie Sorry for the delay! Yes, I think if you alow the grey palette, best is to keep how you compute all the colors nonetheless. It's more a fix for when user is targeting a color with saturation 0 that ends up being red. But your theme make more calculations that you should keep, I think. Because they are so cool.


quantum lock lvl 1 grant proposal

The application for lvl 1 grant proposal.
Project Abstract
We are developing an enhancement to the blockchain consensus protocol to enable it to run on a quantum computer. The core development is the Quantum Lock which enables the proof-of-quantum-work. Our Quantum Lock uses quantum computation with marginal energy cost that is only a fraction of its classical counter-part. The most significant benefit is the potential to drastically diminish the energy consumption involved in a classical proof-of-work computation, while the consensus protocol scales with emergence of the quantum computing power. The first application of the Quantum Lock is the QBITCOIN.
For which grant level are you applying?

[x ] Level 1:  Up to $10,000, 2 approvals
[ ] Level 2:  Up to $50,000, 3 approvals
[ ] Level 3:  Unlimited, 5 approvals (for > $100k Web3 Foundation Council approval)

Application Checklist

[x] The application template has been copied, renamed ( project_name.md) and updated.
[x ] A BTC or Ethereum (DAI/USDT) address for the payment of the milestones is provided inside the application.
[x ] I have read and acknowledged the terms and conditions.
[x ] The software delivered for this grant will be released under an open-source license specified in the application.
[x ] The initial PR contains only one commit (squash and force-push if needed).
[ x] The grant will only be announced once the first milestone has been accepted.

How Did You Hear About our grants program?

[ ] Social Media
[ ] Hackathon
[ x] Personal Recommendation
[ ] Substrate Builders Program
[ ] Investor/VC
[ ] Online Search
[ ] Other: _______

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

Sounds like an interesting project and I’m generally happy to give it a try. But I have a few quick questions (and sorry in case I missed something because I haven’t looked into detail into the process):

How computation intensive is calculating the forrelation function on-chain? Doesn’t it make more sense to do this off chain via offchain worker? It might make sense to take a look at Phragmén and how it’s implemented.

This is an idea which we are contemplating. We do internally work with this as a proof of quantum authority (node has ability to perform quantum work). We aim to explore this in later stage, once we have proof of quantum work up and running (that is more vanilla concept). It will be certainly useful when thinking about quantum smart contracts and calculation of useful quantum tasks.

How does it drastically diminish the energy consumption involved in a classical proof-of-work computation? Wouldn’t you still compete like in classic proof-of-work and therefore the more the better?

Quantum computing is per se energy more efficient and energy consumption does not scale in the same way for classical computing. By some estimation, the reduction in energy consumption as result of processing data with quantum computing can be a factor of 10^-2 to 10^-3, which is huge.  It is indeed true, that proof of work will always be on “more the better“ and thus there will be demand for more work; however, we are proposing technology, which will work on different scale compared to current levels. There will be competition in improving the ability to do quantum computing between the nodes but such competition cannot be trivially won by more brute force energy consumption.
Bit of a hope here is that such competition enabled by our protocol can promote genial technological progress in quantum error correction, quantum circuit optimization and distributed quantum computation, which would in turn bring substantial positive impact in various domains. On the contrary, traditional PoW in many sense, unfortunately, has zeroed in on a pattern of destructive competition.

Also you might want to take a look at this: https://docs.substrate.io/how-to-guides/v3/consensus/pow/ and for example https://github.com/kulupu/kulupu, which you can find here: https://github.com/w3f/Grants-Program/blob/master/docs/polkadot_stack.md#heavy_check_mark-consensus

Thanks for links!

Hey @bqpquantum, are you still interested in this? Note that if we don't hear from you within the next two weeks, we will assume you're no longer interested and terminate the grant.


Chart fit and finish - legend drag handle improvement

On the new line chart and scatter chart, there is a drag handle between the legend and the charting area to enable users to dynamically resize. This task is to iterate on the design to improve the UX.
### UX tasks
- [ ] Provide UX guidance on improved legend drag handle

### Engineering tasks
- [ ] Implement improved legend handle design (pre-req on UX)

will reopen once we have UX guidance included within ticket


Documentation for referencing a Shipyard from an external URI

The CLI allows loading a shipyard from an external URI: https://github.com/keptn/keptn/issues/1648
Task:

Update the documentation to show the functionality of this feature

The linked PR was merged and this is very old. Closing.


Don't enumerate classes in META-INF

This helps to avoid exceptions on multi-release jars where
module-info.class file is stored in META-INF/versions/9, since
com.android.dex apparently can't handle these class files

Thanks!