Review children of GO:0004312 fatty acid synthase activity

Seems the children of GO:0004312 fatty acid synthase activity are a bit of a mess, as indicated by the mismatched EC xrefs (bold).
 [PMID:8088535, RHEA:41908]
xref: RHEA:41908
is_a: GO:0004312 ! fatty acid synthase activity
is_a: GO:0019171 ! (3R)-hydroxyacyl-[acyl-carrier-protein] dehydratase activity

[x] remove is_a: GO:0004312 ! fatty acid synthase activity

id: GO:0004316
name: 3-oxoacyl-[acyl-carrier-protein] reductase (NADPH) activity
namespace: molecular_function
def: "Catalysis of the reaction: (3R)-3-hydroxyacyl-[acyl-carrier protein] + NADP+ = 3-oxoacyl-[acyl-carrier protein] + NADPH + H+." [RHEA:17397]
xref: EC:<IP_ADDRESS> {source="skos:exactMatch"}xref: MetaCyc:3-OXOACYL-ACP-REDUCT-RXN
xref: MetaCyc:RXN-10659 {source="skos:narrowMatch"}
xref: MetaCyc:RXN-11476 {source="skos:narrowMatch"}
xref: MetaCyc:RXN-11480 {source="skos:narrowMatch"}
xref: RHEA:17397 {source="skos:exactMatch"}
is_a: GO:0004312 ! fatty acid synthase activity
is_a: GO:0016616 ! oxidoreductase activity, acting on the CH-OH group of donors, NAD or NADP as acceptor

[x] remove is_a: GO:0004312 ! fatty acid synthase activity

id: GO:0016297
name: acyl-[acyl-carrier-protein] hydrolase activity
namespace: molecular_function
alt_id: GO:0010281
def: "Catalysis of the reaction: acyl-[acyl-carrier protein] + H2O = [acyl-carrier protein] + a fatty acid." [EC:<IP_ADDRESS>, MetaCyc:RXN-7902]
xref: EC:<IP_ADDRESS>
xref: MetaCyc:RXN-7902
xref: RHEA:30131
is_a: GO:0004312 ! fatty acid synthase activity
is_a: GO:0016790 ! thiolester hydrolase activity

[x] remove is_a: GO:0004312 ! fatty acid synthase activity

id: GO:0004318
name: enoyl-[acyl-carrier-protein] reductase (NADH) activity
namespace: molecular_function
def: "Catalysis of the reaction: acyl-[acyl-carrier protein] + NAD+ = trans-2,3-dehydroacyl-[acyl-carrier protein] + NADH + H+." [EC:<IP_ADDRESS>]
xref: EC:<IP_ADDRESS>
xref: MetaCyc:ENOYL-ACP-REDUCT-NADH-RXN
is_a: GO:0004312 ! fatty acid synthase activity
is_a: GO:0016628 ! oxidoreductase activity, acting on the CH-CH group of donors, NAD or NADP as acceptor

[x] remove is_a: GO:0004312 ! fatty acid synthase activity

id: GO:0004319
name: enoyl-[acyl-carrier-protein] reductase (NADPH, B-specific) activity
namespace: molecular_function
def: "Catalysis of the reaction: acyl-[acyl-carrier protein] + NADP+ = trans-2,3-dehydroacyl-[acyl-carrier protein] + NADPH + H+." [EC:<IP_ADDRESS>]
xref: EC:<IP_ADDRESS>
xref: MetaCyc:ENOYL-ACP-REDUCT-NADPH-RXN
is_a: GO:0004312 ! fatty acid synthase activity
is_a: GO:0016628 ! oxidoreductase activity, acting on the CH-CH group of donors, NAD or NADP as acceptor

[x] remove is_a: GO:0004312 ! fatty acid synthase activity

id: GO:0009922
name: fatty acid elongase activity
namespace: molecular_function
def: "Catalysis of the reaction: fatty acid (C-16 or longer) + 2-C = fatty acid (C-16 or longer + 2-C)." [GOC:tb]
xref: MetaCyc:ICOSANOYL-COA-SYNTHASE-RXN
is_a: GO:0004312 ! fatty acid synthase activity
This def does not seem to be a subclass of "fatty acid synthase activity", which is defined as "Catalysis of the reaction: acetyl-CoA + n malonyl-CoA + 2n NADPH + 2n H+ = long-chain fatty acid + n+1 CoA + n CO2 + 2n NADP+."

[ ] change parent to "acyltransferase activity, transferring groups other than amino-acyl groups (EC:2.3.1.-)"??

id: GO:0004315
name: 3-oxoacyl-[acyl-carrier-protein] synthase activity
namespace: molecular_function
alt_id: GO:0033817
def: "Catalysis of the reaction: acyl-[acyl-carrier protein] + malonyl-[acyl-carrier protein] = 3-oxoacyl-[acyl-carrier protein] + CO2 + [acyl-carrier protein]." [RHEA:22836]
xref: EC:<IP_ADDRESS> {source="skos:narrowMatch"}
xref: EC:<IP_ADDRESS> {source="skos:exactMatch"}
xref: MetaCyc:<IP_ADDRESS>-RXN
xref: MetaCyc:3-OXOACYL-ACP-SYNTH-RXN
xref: RHEA:22836
is_a: GO:0004312 ! fatty acid synthase activity
This def does not seem to be a subclass of "fatty acid synthase activity", which is defined as "Catalysis of the reaction: acetyl-CoA + n malonyl-CoA + 2n NADPH + 2n H+ = long-chain fatty acid + n+1 CoA + n CO2 + 2n NADP+."

[x] change parent to "acyltransferase activity, transferring groups other than amino-acyl groups (EC:2.3.1.-)"??

id: GO:0004313
name: [acyl-carrier-protein] S-acetyltransferase activity
namespace: molecular_function
def: "Catalysis of the reaction: acetyl-CoA + [acyl-carrier protein] = CoA + acetyl-[acyl-carrier protein]." [RHEA:41788]
xref: EC:<IP_ADDRESS>
xref: MetaCyc:ACP-S-ACETYLTRANSFER-RXN
xref: RHEA:41788
is_a: GO:0004312 ! fatty acid synthase activity
is_a: GO:0016418 ! S-acetyltransferase activity
This def does not seem to be a subclass of "fatty acid synthase activity", which is defined as "Catalysis of the reaction: acetyl-CoA + n malonyl-CoA + 2n NADPH + 2n H+ = long-chain fatty acid + n+1 CoA + n CO2 + 2n NADP+."

[x] remove is_a: GO:0004312 ! fatty acid synthase activity

id: GO:0004314
name: [acyl-carrier-protein] S-malonyltransferase activity
namespace: molecular_function
def: "Catalysis of the reaction: malonyl-CoA + [acyl-carrier protein] = CoA + malonyl-[acyl-carrier protein]." [EC:<IP_ADDRESS>]
xref: EC:<IP_ADDRESS>
xref: MetaCyc:MALONYL-COA-ACP-TRANSACYL-RXN
is_a: GO:0004312 ! fatty acid synthase activity
is_a: GO:0016419 ! S-malonyltransferase activity
This def does not seem to be a subclass of "fatty acid synthase activity", which is defined as "Catalysis of the reaction: acetyl-CoA + n malonyl-CoA + 2n NADPH + 2n H+ = long-chain fatty acid + n+1 CoA + n CO2 + 2n NADP+."

[x] remove is_a: GO:0004312 ! fatty acid synthase activity

@pgaudet Thanks for the explanation and fixing all these.
I see there is one you didn't change - what were your thoughts on GO:0009922?:
id: GO:0009922
name: fatty acid elongase activity
namespace: molecular_function
def: "Catalysis of the reaction: fatty acid (C-16 or longer) + 2-C = fatty acid (C-16 or longer + 2-C)." [GOC:tb]
xref: MetaCyc:ICOSANOYL-COA-SYNTHASE-RXN
is_a: GO:0004312 ! fatty acid synthase activity
This def does not seem to be a subclass of "fatty acid synthase activity", which is defined as "Catalysis of the reaction: acetyl-CoA + n malonyl-CoA + 2n NADPH + 2n H+ = long-chain fatty acid + n+1 CoA + n CO2 + 2n NADP+."

[ ] change parent to "acyltransferase activity, transferring groups other than amino-acyl groups (EC:2.3.1.-)"??


Zulip 3.0: logrotate configuration not installed

After doing a production Zulip 3.0 install by the manual at readthedocs, no logrotate configuration was added for Zulip. As a fix I copied the file from deployments/current/puppet/zulip/files/logrotate to /etc/logrotate.d/zulip. From reading other issues, the file should have been installed by puppet.

Yes, it certainly should have been.  What is in your /etc/zulip/zulip.conf?  It's possible if you disabled the relevant puppet rule this might not happen.  And does /home/zulip/deployments/current/scripts/zulip-puppet-apply attempt to create that?

puppet/zulip/manifests/app_frontend.pp installs it.

Yes, it certainly should have been. What is in your /etc/zulip/zulip.conf? It's possible if you disabled the relevant puppet rule this might not happen. And does /home/zulip/deployments/current/scripts/zulip-puppet-apply attempt to create that?

This is my /etc/zulip/zulip.conf. Installation was on a recent Debian/buster.
[machine]
puppet_classes = zulip::voyager
deploy_type = production

[postgresql]
version = 12

[certbot]
auto_renew = yes

[rabbitmq]
nodename = zulip@localhost

If you temporarily move /etc/logrotate.d/zulip out of the say, does running /home/zulip/deployments/current/scripts/zulip-puppet-apply as root re-create it?

It did. Here is the output.
root@talk:~# mv /etc/logrotate.d/zulip /root/zulip.cron
root@talk:~# /home/zulip/deployments/current/scripts/zulip-puppet-apply
Notice: Compiled catalog for talk.lfda.de in environment production in 3.31 seconds
Notice: /Stage[main]/Zulip::Apt_repository/Exec[setup_apt_repo]/returns: current_value 'notrun', should be ['0'] (noop)
Notice: Class[Zulip::Apt_repository]: Would have triggered 'refresh' from 1 event
Notice: /Stage[main]/Zulip::App_frontend/File[/etc/logrotate.d/zulip]/ensure: current_value 'absent', should be 'file' (noop)
Notice: Class[Zulip::App_frontend]: Would have triggered 'refresh' from 1 event
Notice: /Stage[main]/Zulip::Rabbit/File[/etc/rabbitmq]/owner: current_value 'rabbitmq', should be 'root' (noop)
Notice: /Stage[main]/Zulip::Rabbit/File[/etc/rabbitmq]/group: current_value 'rabbitmq', should be 'root' (noop)
Notice: Class[Zulip::Rabbit]: Would have triggered 'refresh' from 2 events
Notice: Stage[main]: Would have triggered 'refresh' from 3 events
Notice: Applied catalog in 1.60 seconds
Apply changes? [y/N] y
Notice: Compiled catalog for talk.lfda.de in environment production in 2.94 seconds
Notice: /Stage[main]/Zulip::Apt_repository/Exec[setup_apt_repo]/returns: executed successfully
Notice: /Stage[main]/Zulip::App_frontend/File[/etc/logrotate.d/zulip]/ensure: defined content as '{md5}4e87145fd2f6c141ce2103171ff66d06'
Notice: /Stage[main]/Zulip::Rabbit/File[/etc/rabbitmq]/owner: owner changed 'rabbitmq' to 'root'
Notice: /Stage[main]/Zulip::Rabbit/File[/etc/rabbitmq]/group: group changed 'rabbitmq' to 'root'
Notice: Applied catalog in 14.01 seconds

root@talk:~# ls -l /etc/logrotate.d
total 60
-rw-r--r-- 1 root root 120 Apr 19  2019 alternatives
-rw-r--r-- 1 root root 173 May 28  2019 apt
-rw-r--r-- 1 root root 130 Aug 29  2018 btmp
-rw-r--r-- 1 root root  82 May 26  2018 certbot
-rw-r--r-- 1 root root 112 Apr 19  2019 dpkg
-rw-r--r-- 1 root root 146 May 13 18:01 exim4-base
-rw-r--r-- 1 root root 126 May 13 18:01 exim4-paniclog
-rw-r--r-- 1 root root 329 Aug 13  2019 nginx
-rw-r--r-- 1 root root 173 Jun 10  2017 postgresql-common.disabled
-rw-r--r-- 1 root root 260 Oct 30  2018 rabbitmq-server
-rw-r--r-- 1 root root 124 Jul 19 14:29 redis-server
-rw-r--r-- 1 root root 501 Feb 26  2019 rsyslog
-rw-r--r-- 1 root root 145 Feb 19  2018 wtmp
-rw-r--r-- 1 root root 314 Apr 28  2018 zabbix-agent
-rw-r--r-- 1 root root 427 Aug 11 13:28 zulip

For versions: I have yet to upgrade to 3.1. Still on 3.0.

If puppet puts it in place now, then I'm mystified.
One very faint possibility is that there was an ordering issue in puppet, and /etc/logrotate.d didn't exist when it tried to put /etc/logrotate.d/zulip into place -- but that would have caused an error and installation would have failed noisily, which I presume you would have mentioned.
I can't replicate this with a stock 3.0 install, though.  Short of any other debugging information that clarifies what happened, we may need to resolve this.

One very faint possibility is that there was an ordering issue in puppet, and /etc/logrotate.d didn't exist when it tried to put /etc/logrotate.d/zulip into place -- but that would have caused an error and installation would have failed noisily, which I presume you would have mentioned.

I did not experience any errors during installation.

Hm, OK.  I'm out of ideas for causes, then.

The other possibility could be that you're running some other configuration manager tool that deleted our file; @itsafire is that possible?

I think it's pretty likely based on this investigation that there isn't a bug in Zulip's puppet code; @itsafire please report back with whether you have any other configuration management tooling that could have caused this.

The other possibility could be that you're running some other configuration manager tool that deleted our file; @itsafire is that possible?

I installed on recent vanilla Debian 10. Install tasks: Base system + ssh server. Debian is not provide configuration management in this configuration.


[BUG] - cardano-cli genesis create-staked - wrong supply calculation

Internal/External
Internal
Area
Other   Any other topic (Delegation, Ranking, ...).
Summary
The cardano-cli genesis create-staked command calculates wrong values for amounts provided by --supply and --supply-delegated. Moreover, the supply distribution is not even across stake holders.
On master, command
cardano-cli genesis create-staked --genesis-dir ./create_staked --testnet-magic 42 --gen-pools 3 --gen-utxo-keys 1 --supply<PHONE_NUMBER>0000000 --gen-stake-delegs 3 --supply-delegated<PHONE_NUMBER>000000

produces
  "initialFunds": {
    "604d0e0e76cab5a2a555fc75155fd1da16f419eb0244c6db25431b540b":<PHONE_NUMBER>0000000,
    "00b746543e04dfdf4896c98bb15043d8a49773f07aba9a12fa4bfd3ff8f4abc3fa15cdd7985dab83d3bb2d7e563e784f153df11926141eb0ca":<PHONE_NUMBER>000000,
    "003fda4ada3ac089f4718ef17e53b053b9d52f624fa99a97af2ec042d503f8f239cabc48211bbbc223b37182cd9e48b680f42645fbfd96d3c1":<PHONE_NUMBER>000000,
    "00b971958504ace5f5935a09f53d2c217be64e90b86e29d5c21fb05e347542b8dc32681e8f699807cd089ebe81c17aeaf9851b101a652458ce":<PHONE_NUMBER>000000
  },

Notice that the supply is not evenly distributed. The non-delegated supply is 1.8 x supply, the sum of delegated supply is 1.6 x supply-delegated.
With cardano-cli 1.35.3, the same cardano-cli genesis create-staked command as above produces
  "initialFunds": {
    "000069e8386434384760d27bc56b4ea46153e64569dc69f70d193bb7f9990d846a566a8b0537d96b24a66a4986fd816f120d33ad7357ce8c4b":<PHONE_NUMBER>000000,
    "00069e090076834b90a6cfba0354d1c08ea98c5c5b0a779f14b72f25e11cc4b644bccff4bce828d519ff6da58b428c3ce0e44b7873e9affe5e":<PHONE_NUMBER>000000,
    "00b6b146e51260d83d3d90fdeecd03a5b6238fc292cfa3969b356ab33934392533f957313474d21d6ac780004b07a829838aa4cb6e02f28dad":<PHONE_NUMBER>000000,
    "608f8eb872777c385181e558fc274daea850adb5b0328d07a56159828d":<PHONE_NUMBER>0000000
  }

In this case, the supply is evenly distributed across stake holders and each sum (delegated and non-delegated stake) doesn't exceed the provided supply.
Expected behavior
The supply is evenly distributed and doesn't exceed the provided amount.
System info:

OS Name: Fedora 36
Node version: 8832f86728ef6a425452b44f2f269acde149448c
CLI version: 8832f86728ef6a425452b44f2f269acde149448c

@mkoura has this been resolved on master?

@Jimbo4350 no, still present on master.

This is what happens when I try this on master:
$ cardano-cli version
cardano-cli 1.36.0 - darwin-aarch64 - ghc-8.10
git rev d9ff0ef9e53fad2761077b90aeeb0f337b863689
$ cardano-cli genesis create-staked --genesis-dir ./create_staked --testnet-magic 42 --gen-pools 3 --gen-utxo-keys 1 --supply<PHONE_NUMBER>0000000 --gen-stake-delegs 3 --supply-delegated<PHONE_NUMBER>000000
generated genesis with: 3 genesis keys, 1 non-delegating UTxO keys, 3 stake pools, 3 delegating UTxO keys, 3 delegation map entries,
$ cat create_staked/genesis.json | jq .initialFunds
{
  "6070b0c5de1fdc3a9bf9b48eb112964fa7efe7cb8e7a8223a62b77c053":<PHONE_NUMBER>0000000,
  "009e87ae3ca7e162bc82027bc4ab5b0e37768eefc22fb18e10bd8b6d5a6c2fb48537d888d977ee76fca25ac88c94d689461257731e73afaa59":<PHONE_NUMBER>000000,
  "00f379be67d3c2ee7a993b3a45f56fd2097873830387eabae4ff0c18f0b7e53bf2d9570c78ecbaf3a58926e57c93a7e2b8412ea20245ca8278":<PHONE_NUMBER>000000,
  "00c5d9feff590ce9457f026a9eeabaf994309b0e1c8e0ca2c41158a06394b951edda2b604511f246ee8dcc185dc0bce2f1facdba23c45a48e4":<PHONE_NUMBER>000000
}

I'm unable to reproduce the uneven distribution.

@newhoggy I can confirm your results, looks like this was recently fixed by https://github.com/input-output-hk/cardano-node/pull/4765.


The translations on docs.docker-cn.com are not completed

Problem description
I'm very excited that Docker has officially supported a Chinese version of the docker documents, https://docs.docker-cn.com, it's huge help for Chinese developers and ops. Thank you.
However, I found many documents are not fully translated, such as:

https://docs.docker-cn.com/engine/userguide/

I understand that there might be some resource limitations here. What can I do to help the translation?

Is anyone here can give some response? Did I raise the issue in the wrong repo?

ping @johndmulhausen

We have identified several typos in the docker-cn website, such as:
https://docs.docker-cn.com/get-started/part2/#dockerfile
ADD ./app

should be
ADD . /app

And https://docs.docker-cn.com/get-started/part3/#docker-composeyml
version:"3"
services:
  web:
    # 将 username/repo:tag 替换为您的名称和镜像详细信息
    image: username/repository:tag
    deploy:
      replicas:5
      resources:
        limits:
          cpus:"0.1"
          memory:50M
      restart_policy:
        condition: on-failure
    ports:
      - "80:80"
    networks:
      - webnet
networks:
  webnet:

It missed all the spaces between keys and values.
These have caused many troubles for people learning Docker already. If there is a way we can contribute the translation, then it would be great to minimize the trouble, and it would be also very helpful for Chinese docker users to contribute back to the community.
Thanks.

There are two things that I have proposed doing to help:

I have signed off a Statement of Work clearing our translation vendor to translate the entire site. We have a quote from them to do this that seemed reasonable to me.

We are in talks with another vendor to help us open-source the Chinese docs so that they can benefit from corrections from the community. We would host the repo and they would maintain it in the proposal as outlined.

The proposal could be rejected as it does cost the company money to do these things but I believe in them and I know they would help improve things immensely. Will let this space know when there are answers.

The latest docs version in chinese is v17.06, and the english version is v18.03 now, will you update synchronously？

Closing this ticket due to its age, and the impending refactor. If you think this is in error, feel free to reopen. Thanks!


New version: JinweiZhiguang.MasterGo version 1.8.2

Created by 🥟 Dumplings in workflow run #6430.

Log
Updated: 1.8.1 → 1.8.2
Submitting WinGet manifests
Creating manifests
Uploading manifests and making commits
Creating a pull request

Microsoft Reviewers: Open in CodeFlow

/azp run

/azp run

https://github.com/microsoft/winget-pkgs/issues/142854


BE: Implement JWT / Token authentication

This includes supporting users within the platform. In the end it will also introduce a collection for the users!


Restores showing original API errors. Closes #2138

Restores showing original API errors. Closes #2138

Merged manually


API

there are some API for gatus?

Well, there's one, but it's pretty much tailored to be used by Gatus' dashboard.
Here's an example: https://status.twinnation.org/api/v1/statuses

The uptime during the last 1h, 24h and 7d will be exposed through the API soon.
See #62
Closing this because the question has been answered.

thank you, that is good for now! :)


[Feature Request] Launching the Unreal Editor through a Distrobox container

Is your feature request related to a problem? Please describe.
I'm now using NixOS on my main system, and the problem currently is that I cannot run Unreal's editor directly on NixOS, due to how it doesn't have a shared library directory. I can launch the Unreal Editor through something like a Fedora distrobox (Where I have my development toolchain and IDEs like Jetbrains Rider installed on) however.
Describe the solution you'd like
It would be nice to have an option to launch the Unreal Editor inside of a Distrobox container, if one is found on the system.
Describe alternatives you've considered
I've considered enabling stub-ld, but the issue with this solution is that it doesn't seamlessly integrate into my solution for getting Jetbrains working with my installed toolchains and SDKs (For example, MinGW and the Vulkan SDK).
https://nix.dev/permalink/stub-ld

I was thinking about implementing like steam has with the steamcmd customization where you can add prefix/suffix to the engine launch. Would that work? I have no idea how NixOS and Distrobox for that matter work.

I was thinking about implementing like steam has with the steamcmd customization where you can add prefix/suffix to the engine launch. Would that work? I have no idea how NixOS and Distrobox for that matter work.

A prefix should work.
For example, if I wanted a shortcut to run something in a Distrobox, you could do this:
distrobox-enter  -n distrobox-container-name-here  -- executable-target-here


Fix berkshelf test with version bump

Description
Fixing Berkshelf test from the test Linux box
Related Issue

Types of changes

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

Checklist:

[ ] I have read the CONTRIBUTING document.
[ ] I have run the pre-merge tests locally and they pass.
[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.
[x] All commits have been signed-off for the Developer Certificate of Origin.

Changing to a chef-17 based PR from @tpowell-progress account instead


copy image to another region support is needed

Feature Request
I have an image in south central us and i want to copy the image in eastus.
I am not getting the proper method in the sdk.

Hi @himanshurraj thanks for this issue.
I am not quite sure whether there is direct support for this in REST API of azure. If there is not such a way, there will not be such a method in the go SDK. Sorry about this.
But you can try some work around of this. For instance, take a snapshot of that image, and recreate a new image in the new location: https://docs.microsoft.com/en-us/azure/virtual-machines/windows/snapshot-copy-managed-disk


Support missing for libsodium's crypto_hash_sha256 C function (through crate libsodium-sys-stable)

I got this error while testing a project in miri
error: unsupported operation: can't call foreign function `crypto_hash_sha256` on OS `linux`

Related docs: https://doc.libsodium.org/advanced/sha-2_hash_function
The project depends on https://crates.io/crates/libsodium-sys-stable btw

Thanks for the report! However, I'm afraid I don't think we will support cryptographic APIs like that, sorry. Miri can't implement every C library on the planet, and cryptographic APIs require enough care that I don't think we can provide them with reasonable effort.


SVG images?

Hi,
Does the plugin work with SVG images?
Thanks

It inherits from Xamarin.Forms Image so whatever it supports currently


bug(PAPIv2): command keys can reconverge after diverging, breaking the run log

Overview
If a PAPIv2 analysis differs from the actual run, the actual run can (and usually will) produce command keys that exist in the diverged portion of the analysis, producing an incorrect run log in the app.
Steps to reproduce
See #9503 for repro protocol where this bug combined with another bug caused by an incorrect protocol analysis.

Run diverges from analysis
Run produces new command keys that still exist in other locations in the incorrect analysis
Run log gets real messed up

Current behavior
In PAPIv2 protocols, command keys may "reconverge" in a non-deterministic protocol due to the naivety of the current implementation.
Implementation notes
As much as a hate to type this, this can be solved by implementing a rudimentary blockchain for the command keys. (Please send all complaints and comments to @SyntaxColoring)

Tracked elsewhere, cleaning this off my issue list


burpsuite: 2020.1 -> 2021.2.1

Motivation for this change
Bump version to current stable version 2021.2.1
Version currently in nixpkgs does not install, as the jar is no longer available for download from the portswigger site.
Things done
Bump version to 2021.2.1 and update the sha256 hash

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

[*] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[*] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[*] 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
[*] Fits CONTRIBUTING.md.

Version currently in nixpkgs does not install, as the jar is no longer available for download from the portswigger site.

Can you please upload the new version to web.archive.org?

Uploaded to web.archive.org, updated my PR, and tested it out!

Building hangs forever at:
burpsuite.jar> trying https://web.archive.org/web/https://portswigger.net/Burp/Releases/Download?productId=100&version=2021.2.1&type=Jar
burpsuite.jar>   % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
burpsuite.jar>                                  Dload  Upload   Total   Spent    Left  Speed
burpsuite.jar>   0     0    0     0    0     0      0      0 --:--:--  0:00:01 --:--:--     0
error: interrupted by the user

I tested again on another workstation and indeed I can reproduce the hang. It works fine when I pull it directly from portswigger.net so maybe something like this would be more appropriate anyway:
src = fetchurl {
  name = "burpsuite.jar";
  urls = [
    "https://portswigger.net/Burp/Releases/Download?productId=100&version=${version}&type=Jar"
    "https://web.archive.org/web/https://portswigger.net/Burp/Releases/Download?productId=100&version=${version}&type=Jar"
  ];
  sha256 = "0x6x8jq33144wi5x3cf1887hhbmcvgx29n82l114n5f1kirkavxn";
};

Other updates are merged.


Performance improvement with cell reuse

Regarding https://github.com/SoySauceLab/CollectionKit/issues/97
@Banck gives us a brilliant idea of not removing the cells from CollectionView.
This brings major improvement in main thread cpu time.
With 10s scrolling session, reuseManager's lifeSpan set to 5.0s. and scrolling from top to the end every 1 second. We get the following result on an iPhone X.
Notices that CollectionView.loadCells() cpu time changed from 558ms to 288ms.
Before

After

UICollectionView
And just for comparison with UICollectionView.
UICollectionView has about the same scroll performance as CollectionKit after this patch.

Absolutely amazing improvement! 👏🏻

Can’t wait for this to be accepted! :)

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@cc63503). Click here to learn what that means.
The diff coverage is 75%.

@@           Coverage Diff            @@
##             master    #100   +/-   ##
========================================
  Coverage          ?   56.1%
========================================
  Files             ?      35
  Lines             ?     991
  Branches          ?       0
========================================
  Hits              ?     556
  Misses            ?     435
  Partials          ?       0

Impacted Files
Coverage Δ

Sources/Extensions/UIView+CollectionKit.swift
58.33% <75%> (ø)

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 cc63503...33ad680. Read the comment docs.


removes npm dependency by using mermaid ink service | removes referen…

Remove the dependency on npm
This issue is document as issue 167
Changes

removes the dependency on npm by using the mermaid ink service
removes references to the npm version mermaid and the related issues that is causes

Description of changes
This changes the way that mermaid.py works. Instead of doing checks to see if the npm command exists for mermaid, it makes a call to the mermaid ink service. This service takes a base64 encoded string as the mermaid code and can return either an svg or png depending on the path.
Testing
A simple test to see if this new approach works is to run the mermaid.py file and see if the mermaid charts are created in the tmp folder in the project root.
Warning
This does remove the pdf file output type.
I don't see this as an issue but someone else might.

This is a very good submission that can lower the threshold for users to use
But when faced with third-party network requirements, I have several questions

This requires a default network dependency, but this assumption may not be true in the future, such as using Llama's intranet deployment, which means it is more reasonable to make it an option?
What is the payment policy for third-party services? Is it free forever? It promises service stability
If the third-party service is down, what is our strategy?

had better submit as a optional solution, not directly replace the original one.
feel free to have a look mine solution https://github.com/geekan/MetaGPT/pull/289

Good idea! It may be helpful when the mermaid install fails.
Can you resolve the conflicts? And provide test codes for this, please.
In addition, it will be convenient if it is configurable for installation and usage

This PR has been closed because it took too long, feel free to reopen it.


Ambiguity in example used for inheritance principle

From Common principles:

Example 2: Multiple runs and recs with same acquisition (acq) parameters
sub-01/
    anat/
    func/
        sub-01_task-xyz_acq-test1_run-1_bold.nii.gz
        sub-01_task-xyz_acq-test1_run-2_bold.nii.gz
        sub-01_task-xyz_acq-test1_rec-recon1_bold.nii.gz
        sub-01_task-xyz_acq-test1_rec-recon2_bold.nii.gz
        sub-01_task-xyz_acq-test1_bold.json

For the above example, all NIfTI files are acquired with same scanning parameters (acq-test1). Hence a JSON file describing the acq parameters will apply to different runs and rec files. Also if the JSON file (task-xyz_acq-test1_bold.json) is defined at dataset top level directory, it will be applicable to all task runs with test1 acquisition parameter.

The NIfTI files here indicate that there are:

Two different "runs", i.e. the protocol acquisition was executed twice;
Two different "recs", i.e. reconstruction algorithms.

However what is not clear from this example is: what data were provided as input to the two distinct reconstructions? Given they don't include "_run-#", should it be assumed that both reconstructions used as input the concatenation of all data across the two runs?
Moreover, the specific details of "_run-" and "_rec-`" aren't provided until a later section (Modality specific - MRI).
I would propose that an alternative, more simple example may be preferable here: one that does not introduce such potential ambiguity or dependence on other sections.

I agree that this example is more complicated than necessary. +1 from my side to work on a simpler one.

As to me inheritance principle lacks necessary formal algorithmic definition and thus is ambiguous for any (like here) but trivial scenario, thus examples indeed need to be dumbed down more.  More of formalized whining in https://github.com/bids-standard/bids-specification/issues/102 .

@Lestropie do you want to open a PR to improve the example?


yaml_generator.py writes files into site_packages directory

When using an installation of mirage, yaml_generator.py writes files into site_packages/*.egg/config/ directory. I don't think that is the desired default location.
I suspect the reason is in line 879 ff of yaml_generator.py:
scripts_path = os.path.dirname(os.path.realpath(__file__))

This issue was reported by Matteo Correnti.


Inconsistent naming "terminusdb-client" (js) vs. "terminusdb-client-python" (py)

at least include a reference to javascript if "terminusdb-client" is not some meta-client or the core for all others clients, e.g. name it: terminusdb-client-js or terminusdb-client-javascript, the latter would match the schema used by "terminusdb-client-python"

This is a good point - the js client has a privileged name as it was the first client and is used in console, but now that we have multiple clients, it should be renamed to terminusdb-client-js to make it match with the other clients


revup explodes on PRs with team reviewer

Describe the bug
revup trips over PRs that have non-User (ie: team) reviewers.
reviewRequests.requestedReviewer can be a user or team.  See https://docs.github.com/en/graphql/reference/unions#requestedreviewer
Expected behavior
Support team reviewers, or at least safely ignore non-User reviewers.
This is sufficient to fix it for me:
            for revs in this_node["reviewRequests"]["nodes"]:
+               if revs["requestedReviewer"]:
                    reviewers.add(revs["requestedReviewer"]["login"])
                    reviewer_ids.add(revs["requestedReviewer"]["id"])

To Reproduce
revup upload to create PR. Add a team reviewer to the PR via github UI.  Attempt to run revup upload again.
Logs
Terminal output from the command. If possible, rerun the command with revup -v to get more verbose logs.
It's a private repo, so I can't point you at the PR, nor include the full logs.  But this PR has a regular reviewer and a team reviewer - and I think the interesting bits of the logs are:
% revup -v upload
[...]
      "reviewRequests": {
       "nodes": [
        {
         "requestedReviewer": {
          "login": "joe_user",
          "id": "M...Q=="
         }
        },
        {
         "requestedReviewer": null
        }
       ]
      },
[...]
Traceback (most recent call last):
  File "/Users/gus/bin/revup", line 8, in <module>
    sys.exit(_main())
  File "/Users/gus/revup/lib/python3.10/site-packages/revup/__main__.py", line 17, in _main
    sys.exit(asyncio.run(main()))
  File<EMAIL_ADDRESS>line 44, in run
    return loop.run_until_complete(main)
  File<EMAIL_ADDRESS>line 646, in run_until_complete
    return future.result()
  File "/Users/gus/revup/lib/python3.10/site-packages/revup/revup.py", line 357, in main
    return await upload.main(
  File "/Users/gus/revup/lib/python3.10/site-packages/revup/upload.py", line 46, in main
    await topics.query_github()
  File "/Users/gus/revup/lib/python3.10/site-packages/revup/topic_stack.py", line 961, in query_github
    ) = await github_utils.query_everything(
  File "/Users/gus/revup/lib/python3.10/site-packages/revup/github_utils.py", line 301, in query_everything
    reviewers.add(revs["requestedReviewer"]["login"])
TypeError: 'NoneType' object is not subscriptable

actually supporting Teams is opened as a new feature https://github.com/Skydio/revup/issues/76


The object playground should take the local cache into account

The reimplementation of the Inspector did not cover the caching of the scripts. We should recover that ability.

The cached Playground content is back:


App crashes when segue presentation is Over Full Scren

Currently using these transitions, but I have ran into one significant issue. When segueing to a storyboard with it's presentation as type Over Full Screen, the app crashes with the error:
fatal error: unexpectedly found nil while unwrapping an Optional value
Currently segueing using storyboards and have a prepareForSegue function in my view controller.
override func prepareForSegue(segue: UIStoryboardSegue, sender: AnyObject?) {
        if (segue.identifier == "settings") {
            let duration = 0.6
            WYTransitionMgr.configureTransition(duration, toViewController: segue.destinationViewController, handGestureEnable: true, transitionType: .Up)
        }
    }

Any help would be appreciated, thanks!

Gotcha. It seems that when the presentation set to Over Full Screen, the function animateTransition(transitionContext: UIViewControllerContextTransitioning) no longer can capture the origin view context. I'll fix that, no worry.

Awesome, really appreciate the quick reply! Enjoy your weekend.

No problem. Just pushed a newer version of pod. Please pod update to get latest version.
Thanks for supporting. Have a good weekend too. I'll add another critical update to this library later sometimes (won't affect your current development).


Syntax highlighting

Migrating progress from #97 over. That branch was a reused branch, and the new branch includes recent changes of line numbers on main and keeps the changes only related to syntax highlighting.
Still couldn't get it out :/

We'll crack that nut... eventually.


v2.0.0-beta.1

Preparing release

@bsutton
I have published a beta version:
shelf_letsencrypt: ^2.0.0-beta.1
https://pub.dev/packages/shelf_letsencrypt/versions/2.0.0-beta.1
If you can test it, I would greatly appreciate it.
Best regards.


Stream read errors result in uncaught exceptions

Currently, message errors from StreamMessageReader's onData method end up as uncaught exceptions, which can be very difficult to handle when used in server contexts.
https://github.com/Microsoft/vscode-languageserver-node/blob/master/jsonrpc/src/messageReader.ts#L220
Would it be reasonable to convert such errors into error events (via this.fireError) so that the stream owner can perform shutdown / restart steps as necessary? I can submit a PR to do so. cc @ljw1004

Yes, but we should clearly defined that the readere then moves into an error state and doesn't process any new onData events since it can not meaningful recover from a missing content length.

Any workaround? It is really really bad if your server crashes just because some LSP stream got broken.

The got improved already an errors are now reported on the connection. What esle do you expect the reader to do if it receives a broken message

When an error occurs, the entire Node.js process is crashed for me. Do I have to register an error handler to prevent this from happening, or am I doing something else wrong that is causing me to mistakenly blame this package?

Not sure I understand that correctly. The server node process crashes ?

Yes. The Node.js process that is running vscode-languageserver-node crashes.

What seems strange to me is that the node process crashes. It shouldn't :-)
But I am open for a PR that converts the exception into a fireError.

OK. Now I understand. Under crash I was assuming a real process crash.
For the servers I maintain I have a process.on('uncaughtException', (error: any) => { ... }) handler since exception can occur in other parts of your server as well.

This got addressed in the meantime. The exception are now reported via the onError event.


Ordering of installation.

So, it appears that I must install CRD definitions before their use.  This places an ordering constraint on helm installations.  However, I cannot control the order of helm installations in the Helm operator.
For example, I am trying to use the Bitnami Sealed Secrets Controller.  I need to install it before I provide a SealedSecret CRD.  To do so, I cannot simply create a HelmRelease for the Controller (w/ CRDs).  Instead, I must install the Controller outside of the Operator.
Is this correct?  Is there a way to impose an ordering?

The helm-op will retry the installation every 3 minutes. If you have a release with the Sealed Secrets Controller and another one that contains SealedSecrets, the 2nd one will fail at first but at the next run, the CRD is installed and it will succeed.

Thank you.

I had the same question. Should this be in the documentation proper? Also, is that 3 minutes configurable? In particular, can it be force retried from fluxctl (say a quick fix was known and performed to release a blocking item)?


img-vuer lack of initialization function

When i enter the first page, the numbers of big picture is correct. However when i enter the second page, the numbers is wrong. It's seems to add imgs after the first time. How can i fix that?


1024*27000 picture can't be displayed

Maybe 27000 is too long, is there any way to fix it?

I cannot help with this kind of problem unless you include details - error messages, what gets displayed, the image you're using, the phone you have a problem with, whether other phones are affected etc.

Here is an example picture: http://imanga.co/img/d39dd2e38939b39e/12/1.jpg,
several users which are using my app reported this problem to me, and I have tested on my phone, it can't be displayed, but the smaller pictures(<20000) can be displayed. Maybe it's a tile size issue.

Now I got it, it will be displayed after about 5s because the pictures are too long. Sorry about it!


Title: HelperBird chrome extension enhancement [UI]

Which pre-existing extension will you be working on?
A Full set of tools for Chrome Extension to be implemented. This includes a lot of features such as Dyslexia Mode, Full Immersive Reader, Text to Speech, Voice Typing, Word Prediction, Screenshot, Color Blindness, Reading List, Bookmarks
Describe your enhancement/improvement
Need a new UI with more clear icon design and better UI interaction
Add any other context or screenshots about the enhancement here.
No response

I want to work under this issue under JWOC


mraa_gpio_lookup can't  check all physical pin names

https://github.com/intel-iot-devkit/mraa/blob/d24457f9a15f5ad4470d55bc69fbece22ce97bfc/src/mraa.c#L920
When plat->gpio_count is less than plat->phy_pin_count, the function can't check pins between [ plat->gpio_count, plat->phy_pin_count );
I think it should be changed like this:
@@ -880,7 +880,8 @@ mraa_gpio_lookup(const char* pin_name)
         return -1;
     }

-    for (i = 0; i < plat->gpio_count; i++) {
+    for (i = 0; i < plat->phy_pin_count; i++) {

@turmary mraa_gpio_lookup is supposed to check only GPIO pins not the physical pins exposed on the board. So, limiting the count to plat->gpio_count is fine here.

@Mani-Sadhasivam
OK, If it check GPIO only,  in the loop, the pin[i] type should be checked.
Make it has GPIO function or else don't incremment index variable i.

Aug 14 12:25:26 raspberrypi libmraa[20886]: platform_detected = 9
Aug 14 12:25:26 raspberrypi libmraa[20886]: libmraa initialised for platform 'Raspberry Pi 3 Model B' of type 5
Aug 14 12:25:39 raspberrypi libmraa[20886]: [0] pin_name = 'GPIO12' plat_pins_name = 'INVALID'
Aug 14 12:25:40 raspberrypi libmraa[20886]: [1] pin_name = 'GPIO12' plat_pins_name = '3V3'
Aug 14 12:25:40 raspberrypi libmraa[20886]: [2] pin_name = 'GPIO12' plat_pins_name = '5V'
Aug 14 12:25:40 raspberrypi libmraa[20886]: [3] pin_name = 'GPIO12' plat_pins_name = 'SDA0'
Aug 14 12:25:40 raspberrypi libmraa[20886]: [4] pin_name = 'GPIO12' plat_pins_name = '5V'
Aug 14 12:25:40 raspberrypi libmraa[20886]: [5] pin_name = 'GPIO12' plat_pins_name = 'SCL0'
Aug 14 12:25:40 raspberrypi libmraa[20886]: [6] pin_name = 'GPIO12' plat_pins_name = 'GND'
Aug 14 12:25:40 raspberrypi libmraa[20886]: [7] pin_name = 'GPIO12' plat_pins_name = 'GPIO4'
Aug 14 12:25:40 raspberrypi libmraa[20886]: [8] pin_name = 'GPIO12' plat_pins_name = 'UART_TX'
Aug 14 12:25:40 raspberrypi libmraa[20886]: [9] pin_name = 'GPIO12' plat_pins_name = 'GND'
Aug 14 12:25:40 raspberrypi libmraa[20886]: [10] pin_name = 'GPIO12' plat_pins_name = 'UART_RX'
Aug 14 12:25:40 raspberrypi libmraa[20886]: [11] pin_name = 'GPIO12' plat_pins_name = 'GPIO17'
Aug 14 12:25:40 raspberrypi libmraa[20886]: [12] pin_name = 'GPIO12' plat_pins_name = 'GPIO18'
Aug 14 12:25:40 raspberrypi libmraa[20886]: [13] pin_name = 'GPIO12' plat_pins_name = 'GPIO27'
Aug 14 12:25:40 raspberrypi libmraa[20886]: [14] pin_name = 'GPIO12' plat_pins_name = 'GND'
Aug 14 12:25:40 raspberrypi libmraa[20886]: [15] pin_name = 'GPIO12' plat_pins_name = 'GPIO22'
Aug 14 12:25:40 raspberrypi libmraa[20886]: [16] pin_name = 'GPIO12' plat_pins_name = 'GPIO23'
Aug 14 12:25:40 raspberrypi libmraa[20886]: [17] pin_name = 'GPIO12' plat_pins_name = '3V3'
Aug 14 12:25:40 raspberrypi libmraa[20886]: [18] pin_name = 'GPIO12' plat_pins_name = 'GPIO24'
Aug 14 12:25:40 raspberrypi libmraa[20886]: [19] pin_name = 'GPIO12' plat_pins_name = 'SPI_MOSI'
Aug 14 12:25:40 raspberrypi libmraa[20886]: [20] pin_name = 'GPIO12' plat_pins_name = 'GND'
Aug 14 12:25:40 raspberrypi libmraa[20886]: [21] pin_name = 'GPIO12' plat_pins_name = 'SPI_MISO'
Aug 14 12:25:40 raspberrypi libmraa[20886]: [22] pin_name = 'GPIO12' plat_pins_name = 'GPIO25'
Aug 14 12:25:40 raspberrypi libmraa[20886]: [23] pin_name = 'GPIO12' plat_pins_name = 'SPI_CLK'
Aug 14 12:25:40 raspberrypi libmraa[20886]: [24] pin_name = 'GPIO12' plat_pins_name = 'SPI_CS0'
Aug 14 12:25:40 raspberrypi libmraa[20886]: [25] pin_name = 'GPIO12' plat_pins_name = 'GND'
Aug 14 12:46:09 raspberrypi libmraa[19018]: libmraa version v1.9.0-1-gce748f6 initialised by user 'pi' with EUID 1000

@turmary Agree with you! There has to be a check for GPIO capability inside mraa_gpio_lookup. This will avoid unnecessary comparisons.
Will submit a PR for that! Thanks for bringing this up.

@turmary Sorry, I was wrong. The count should be extended to plat->phy_pin_count since we have to check all the pins. Will change it. Thanks once again!

Hi @turmary ,
I have submitted the fix #918. Can you please let me know if it addresses your issue or not?

It's OK I think.thanks.


HW for lessons 3.3 and 3.4

Submit a link to your own issues here:
PLEASE  TRY TO SUBMIT BY THE END OF DEC 1ST
(It would be helpful if you could add your name as well)
All questions:
Part 1:
1:
Evaluate the variable "result".
Num1 = 50
Num2 = Num1 % 9 + 15
Num3 = Num2 / Num1 + ( Num2 * 2 )
Num4 = Num3 + Num1 / 5 - 10
Result = Num4 - Num2
2:
Num1 = 10
Num2 = Num1 % 3 * 4
Num1 = Num2
Num3 = Num1 * 3
Result = Num3 % 2
Part 2:
1:
Num1 = 50
Num2 = Num1 % 9 + 15
Num3 = Num2 / Num1 + ( Num2 * 2 )
Num4 = Num3 + Num1 / 5 - 10
Result = Num4 - Num2
2:
Num1 = 10
Num2 = Num1 % 3 * 4
Num1 = Num2
Num3 = Num1 * 3
Result = Num3 % 2
3:
valueA = 4
valueB = 90
valueC = 17
valueB = valueC - valueA
valueA = valueA * 10
if valueB > 10:
print(valueC)
4:
type = "curly"
color = "brown"
length = "short"
type = "straight"
hair = type + color + length
print(hair)

link to hw

Hacks

Emma Shen
Period 4 Yeung
Unit 3.3-3.4 Ticket

Notes and Hacks

Link to HW, It includes both assignments, notes, and organization. You're looking for the HW link, should be after the quiz.

link to the Homework and Hacks

Soham Kamat
My notes and homework

https://t-dev-ccm.github.io/Fastpage-setup/2022/11/29/Homework2.html

Max Wu
Homework
Notes

Akshat Parikh
Period 3 Mort
Notes/HW

Gene Chang
Period 3
https://genechang1.github.io/fastPage1/jupyter/lesson2

Kalani Cabral-Omana
HW

Toby Leeder
Issue

Derek Sol
Period 3
My Notes and Homework Completed

Samit Poojary
Hacks

Evan Aparri
Period 3
Hacks and Notes

James Armstrong
Period 3
notes is first screenshot
others are homework

Jeffrey Lee
Mortensen Period 3
Hacks/HW

Notes
Hacks/HW

Link to hacks

Group has not posted. Following issue is just to prove work was submitted on time. Notes Hacks/HW

Hi, I couldn't contact you through slack since I couldn't seem to find you, so I'm hoping you'll see this. I can't seem to open your fastpages.. could you check for any jekyll or commit errors?

Alan Liu-Sui Period 3 Mr. Mortensen
Link to Notes and Hacks

Shreya Sapkal -- Period 3
Link to 3.3-3.4 Notes and Hacks

Link to Issue

Dash Penning


Offline Plotting

Circumvent making calls to FastF1.get_session (and maybe others) so the visualizations can be made without internet access.

Mostly solved with caching. This is no longer a priority.


Missing Inputs

Hi, we tested the latest Version. Our Attribute File for Inputs look like this
node.default['telegraf']['inputs'] =  { 'cpu' => {'percpu' => true,'totalcpu' => true,'drop' => ['cpu_time']},  'disk' => {},  'io' => {},  'mem' => {},  'net' => {},  'swap' => {} }
Only CPU Input is creted in the default_inputs.conf
[inputs.cpu]
drop = ["cpu_time"]
percpu = true
totalcpu = true
when my attribute file is defined like that
node.default['telegraf']['inputs'] =  { 'cpu' => {'percpu' => true,'totalcpu' => true,'drop' => ['cpu_time']}, 'disk' => {  'mount_points' => ['/']}  }
Then the default_inputs.conf looks like this
[inputs.cpu]
drop = ["cpu_time"]
percpu = true
totalcpu = true
[inputs.disk]
mount_points = ["/"]
So all { } entries will not be created . Please help.
Cheers Sven

I saw the same. It looks like the toml rubygem doesn't output the empty hash values. I am getting around this by setting the value to a comment, i.e.
node.default['telegraf']['inputs'] = {'mem' => {'#null' => ''}}

This should produce something like the following
[inputs.mem]
#null = ""

Thanks for providing the workaround. Hope for a better solution in the next release.

Hi @skares - thanks for the report, I believe this will be fixed in the next release (due shortly).  Please reopen if you continue to have problems.  Thanks!

@slillibri thanks for the workaround!


New stewardship and direction

Howdy peeps,
Prince from Netlify! We're going to be working on this project to bringing the latest changes of Hugo so you all have the best experience. For now, my main goal is to break it down into a few phases:

Updating dependencies so deploying is smooth
Migrating into new pipelines so we follow best practices
Optimizing the layouts for accessibility

Is there anything that y'all have for suggestions for the project? Is there prior art you'd like involved? We're happy to listen to folks out here using Hugo.

So I think #674 will be the manifestation of this work! Thank you so much y'all for your patience!
This solves at least some big dependency changes. I would love any help y'all could provide with the ways we could upgrade the pipeline process as well.


GET & POST functions were missing req, res, next

Previously, this code wouldn't run if you try to use it in a project. I found that req.isAuthenticated() returned false each time the function ensureAuthenticated() was called.
I realized this was happening because the app.get('/auth/openid/return', &   app.post('/auth/openid/return', were missing function(req, res, next) from their signatures.
Now, when the user calls
function ensureAuthenticated (req, res, next) {
    if (req.isAuthenticated()) {
      return next();
    }
    res.redirect('/login');
}

isAuthenticated() will return true if user is actually logged in.

@DaveVoyles : Thanks for your contribution! The author, @CelesteDG, has been notified to review your proposed change.

This file is no longer present in the repository.


Fact Unit Test

Additional unit test to bring full code coverage to Fact.

Looks pretty 👍 to me, let's see what @khouzam and @RebeccaAnne think :)

Nice :-) :white_check_mark:

In reply to: 415591421 [](ancestors = 415591421)


fix(button): Avoid breakage by angular change

Type is becoming Type<?> in https://github.com/angular/angular/pull/10616

I signed it!


UITableViewCell dynamic height with Autolayout not being applied

I already had a UITableviewCell configured with autolayout that i was using it in a different tableview and the constraints were being applied perfectly.  I want to use SLKTextViewController but when i make a nib containing that cell and load it into the tableview the height of each cell is fixed and my constraints aren't being applied at all.  What am i missing here...??

Hei, thanks for this awesome library. I just implemented and it works perfectly (at the moment :))
And I'm using UITableViewAutomaticDimension and it's also working perfectly.
All I did was doing this
- (void)viewDidLoad {
    [super viewDidLoad];
    self.tableView.rowHeight = UITableViewAutomaticDimension;
}

- (CGFloat) tableView:(UITableView *)tableView estimatedHeightForRowAtIndexPath:(NSIndexPath *)indexPath
{
    return UITableViewAutomaticDimension;
}

That did it!  Thanks!


[Bug]:

Describe the bug
Traceback (most recent call last):
File "/root/agentscope/agentscope-main/examples/conversation_with_RAG_agents/rag_example.py", line 150， in  main( )
File "/root/agentscope/agentscope-main/examples/conversation_with_RAG_agents/rag_example.py",line 86, in main knowledge bank.equip(agent，agent.knowledge id list)
File "/root/agentscope/agentscope-main/src/agentscope/rag/knowledge_bank.py", line 188, in equip knowledge = self.get_knowledge(
File "/root/agentscope/agentscope-main/src/agentscope/rag/knowledge_bank.py", line 156, in get_knowledge
raise ValueError(
ValueError: agentscope_tutorial_rag does not exist in the knowledge bank.
Expected behavior
When I get the above error when I clone the entire project to test the RAG, would anyone be willing to help me analyze the following?
Environment (please complete the following information):

AgentScope Version:v0.1.0
Python Version: 3.10
OS: ubuntu

Sorry to bother you guys, it's all my fault


Calculate altitude using Leaflet

Hi, I want to calculate altitude of a marker but I didn't find any solution in Leaflet. Do you have idea how to do that? Thank you.

Hi, great to hear that you find Leaflet useful!
However, this issue tracker is used for reporting bugs and discussing new features. For questions on using Leaflet, please use gis.stackexchange.com or stackoverflow.
If you are really sure that this is a bug in leaflet, or a feature request, please change the wording to make it look like a bug report.


Add cluster ID as extra label for newly created PDs.

Use the new extra-labels option introduced in https://github.com/openshift/gcp-pd-csi-driver/pull/11.

/hold
The new CSI driver doesn't seem to be in the CI images yet...

/hold cancel

/test e2e-gcp

/lgtm


Documentation needs correction  - Start-Service sshd

[Enter feedback here]
Powershell command sequence does not work.  Once install is completed the OpenSSH services cannot start (powershell or GUI).
I believe instructions fail to set proper security services and roles for service.
https://github.com/PowerShell/Win32-OpenSSH/issues/907
Overall the document otherwise is well written and looks great.  Just needs some cleanup

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

ID: 18a5a4e5-2604-23c2-1374-68a6974170bc
Version Independent ID: f93809cc-eb5e-6b9b-ed58-c8811a8ee21f
Content: Get started with OpenSSH for Windows
Content Source: WindowsServerDocs/administration/OpenSSH/OpenSSH_Install_FirstUse.md
Product: windows-server
Technology: administration
GitHub Login: @robinharwood
Microsoft Alias: roharwoo

Hey @penguinpages,
Thanks again for contacting Microsoft, we value your feedback.
Going through the steps of installing OpenSSH in our VMs and starting the sshd service via PowerShell and through the Services GUI generated no issues on our end. This was tested in Windows 10, 11, Server 19, and Server 22:

As there are no issues within the steps involved in this document, no updates are to be made at this time.
If you're experiencing issues on your device, please reach out to our community forums as this is outside our scope of work:

Microsoft TechNet
Microsoft Developer Network
Microsoft Support Community

We will now be closing this thread. Thank you and have a most pleasant day.
#please-close


Feat/server start handler

Codecov Report

Merging #419 (d0dda64) into master (e0f4c16) will increase coverage by 0.14%.
The diff coverage is 37.50%.

@@            Coverage Diff             @@
##           master     #419      +/-   ##
==========================================
+ Coverage   59.85%   60.00%   +0.14%
==========================================
  Files          40       40
  Lines        2830     2838       +8
==========================================
+ Hits         1694     1703       +9
+ Misses       1009     1007       -2
- Partials      127      128       +1

Impacted Files
Coverage Δ

core/yerr/errors.go
100.00% <ø> (ø)

core/server.go
65.02% <37.50%> (+0.86%)
:arrow_up:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Keras Models for Homography and Key Points Estimation not using GPU

I am Pytorch User and have not much idea about tensorflow. I am trying to run Keras Models (Homography, Keypoints) on GPU. I have RTX 3090 with CUDA 11.2. I installed Tensorflow-gpu==2.2.0 intsead of simple tensorflow and also tried adding these lines before loading models.
with tf.device('/GPU:0'):
Can you please guide me how can i load these models on GPU instead of CPU.

Hey,
You should try 2 things:

uninstall tensorflow & install tensorflow-gpu
uninstall mxnet & install mxnet-cu101

You already did the first step, and since Keras is now fully encapsulated within Tensorflow, it should use your GPU directly. You can make sure of that with:
import tensorflow as tf
print("Num GPUs Available: ", len(tf.config.list_physical_devices('GPU')))

If it stills do not detect your GPU, try to reinstall a GPU compatible version of mxnet & retry.

Let me know if this helps,
Cheers

@DonsetPG  tf was detecting GPU. The issue was  with CUDATOOLKIT. When i installed torch with bundled  CUDA 11.0, the model automatically started working on GPU.


Outdated

The guide is incredibly outdated, and unfinished.
It is hard to get examples out of it working for newer versions and get started in general.
PLEASE either update and finish it, or add a disclaimer about the state.

@post-rex We have recently updated the entire tutorial on the website, and are going to keep it up to date on a more regular basis. Would you mind to take a look and let us know if there are any issues? Thank you!

I'd consider this issue closed now that the website is more up to date


Add log messages for tenent cleaner job

Type of change

Refactoring

Description

Checklist

[ ] Update/write design documentation in ./documentation/design
[ ] Write tests and make sure they pass
[ ] Update documentation
[ ] Check RBAC rights for Kubernetes / OpenShift roles
[ ] Try your changes from Pod inside your Kubernetes and OpenShift cluster, not just locally
[ ] Reference relevant issue(s) and close them after merging
[ ] Update CHANGELOG.md

@enmasse-ci run tests profile=iot

:heavy_check_mark: Test Summary :heavy_check_mark:
TEST_PROFILE: iot
TEST_CASE:
TOTAL: 40
PASS: 36
FAIL: 0
SKIP: 4
BUILD_NUMBER: 263


Add Python3 metapackage and update library package

Will help with #241

CI test failures due to Python3 version already being installed, okay to overlook

Installing 64-bit python3...
WARNING: Generic MSI Error. This is a local environment error, not an issue with a package or the MSI itself - it could mean a pending reboot is necessary prior to install or something else (like the same version is already installed). Please see MSI log if available. If not, try again adding '--install-arguments="'/lv c:\python3_msi_install.log'"'. Then search the MSI Log for "Return Value 3" and look above that for the error.
ERROR: Running ["C:\ProgramData\chocolatey\lib\python3\tools\python-3.10.0-amd64.exe" /quiet InstallAllUsers=1 PrependPath=1 TargetDir="C:\Python310" ] was not successful. Exit code was '1603'. Exit code indicates the following: Generic MSI Error. This is a local environment error, not an issue with a package or the MSI itself - it could mean a pending reboot is necessary prior to install or something else (like the same version is already installed). Please see MSI log if available. If not, try again adding '--install-arguments="'/lv c:\python3_msi_install.log'"'. Then search the MSI Log for "Return Value 3" and look above that for the error..
The install of python3 was NOT successful.

Got hasty. I added it to the test exclusion list only to realize that since it's a dependency it'll still need to not fail for other packages that depend on it. Will look into and fix. In the mean time I'm reverting libraries.python3.vm to use it's old dependencies.


[BUG] Anomaly detection features count locked at 5

ISSUE
The documentation say that plugins.anomaly_detection.max_anomaly_features can be increased up to 10, (and is limiting but better than nothing)
I increased it to 10 via DevTools:
PUT /_cluster/settings
{
"persistent" : {
"plugins": {
"anomaly_detection": {
"max_anomaly_features": "10"
}
}
}
}
And GET /_cluster/settings/ tell me that is actually persistent even after shutting down the docker compose and bringing it back up.
But at the moment I try to insert the feature in the anomaly detection job in the dashboard, the maximum is locked at 5.
To Reproduce
Steps to reproduce the behavior:

Go to DevTools in dashboard
send and past the previous command
go to anomaly detection
5.try to set more than 5 features.

Host/Environment (please complete the following information):

OS: [Linux Manjaro]
Deployment: Docker compose
Version [latest]

Additional context
Sorry for my broken English, I'm Italian but trying to improve.

@MCdeamon7, this is indeed a bug. We provided support for backend, but the frontend hardcoded the number 5.


Using sphinx apidoc to load django modules

I'm trying to figure out how to get sphinx to load my django modules to generate docs -- it is unable to load modules and I think it is due to django configurations.
I've added the following to sphinx's conf.py:
sys.path.insert(0, os.path.abspath('../..'))
print sys.path
assert(os.environ['DJANGO_SETTINGS_MODULE'])
assert(os.environ['DJANGO_CONFIGURATION'])
from django.conf import settings
import <myapp>.urls

And it fails on the .urls import with:
ImproperlyConfigured: django-configurations settings importer wasn't correctly installed. Please use  one of the starter functions to install it as mentioned in the docs: http://django-configurations.readthedocs.org/

How do I bootstrap the config in a mostly non django script?

I've added an extension point to django-configurations that effectively makes it a Sphinx extension. See https://github.com/jezdez/django-configurations/commit/f2a46fb009f9f50fe192bb243d2b5d63e3816341 for more info.


Clarify explanation for Bash on Window.

Please let me know if I didn't fully understand what the original text was trying to say!

I can definitely see how that sentence might be confusing. Here is what it was supposed to mean:

…, because the read() calls (the ones that look for an escape sequence in editorReadKey()) never time out.

In other words, "calls" is a noun and not a verb. Maybe "function calls" or "invocations" would have been clearer.
Thanks for bringing my attention to this. I've been planning to go through the whole tutorial and fix complicated sentences like this, and improve the wording, etc. I'll make sure to fix this soon!


Renaming (F2 Refactor) within the Vue model throws an error

After the 1.20 update, renaming (F2 Rename Symbol) inside a Vue file causes the following error to occur:

This bug is limited specifically to inside the Vue file script tags. Renaming inside the template view works fine. For some reason it only breaks when renaming inside the script tags. Another odd caveat, is if I rename immediately after loading into VSCode, it works just fine, but if I edit anything inside the template or other Vue files, it breaks randomly.
I noticed this is only an issue with the 1.20+ versions.

thanks for the report! This problem is usually due to overlapping renaming ranges, can you show rename preview from the error location to me?
0.20 refactored a lot of code but did not have too many feature updates, you can temporarily fall back to 0.19 to avoid this problem.

should be fixed in 0.20.2, let me know if still not working :)

Sorry I didn't get back to you sooner on this; the rename preview actually displayed correctly, which was odd since when you executed it, it just failed.
Anyway, I'll test this out immediately and get back to you 😄

It appears to be fixed!! This was bugging me a lot so thanks for the super speedy fix ❤


Do not run  docker container as root #patch

TL;DR
Fixes the Docker container not to run as root.
Type

[x] Bug Fix
[ ] Feature
[ ] Plugin

Are all requirements met?

[ ] Code completed
[ ] Smoke tested
[ ] Unit tests added
[ ] Code documentation added
[x] Any pending items have an associated Issue

Complete description
The change is made in the Dockerfiles, where we create a new group and user to use for running the command.
Tracking Issue
https://github.com/flyteorg/flyte/issues/1606
Follow-up issue

@EngHabu can you take a look at this PR. Also would be good if we can do this across all  flyte docker files.


Base64 Private key

Would it be possible to provide the private key in base64, i think this would be useful when using act to run the actions locally. You can load all secrets from a file  with .env format and its hard to put the private key in there

@hayesOw I had the same issue. Found a hint in act manual about multiline secrets.
https://nektosact.com/beginner/index.html#envsecrets-files-structure
A line break has to be a string \n.
I tested it and it works.
You can convert a key file with this command:
sed ':a;N;$!ba;s/\n/\\n/g' ssh-key-file


set nil as the initial value of the struct reference type fields

for green CI:
https://github.com/vlang/v/pull/19944

@spytheman, Can you review and merge this? Once this is done, CI should be green.  thanks!  :)

It is mainly this PR that needs to be dealt with, I got the wrong branch, and now I'm pushing it again, sorry...


libssl.so.1.1: cannot open shared object file: No such file or directory

:/home/container$ ./omp-server
./omp-server: error while loading shared libraries: libssl.so.1.1: cannot open shared object file: No such file or directory

Libssl1.1 is long EOL and not secure anymore. Tell the devs to update to libssl3 and update your egg.

https://github.com/parkervcp/eggs/pull/2855
Please use this, if their plugin issues after then make a issue on the eggs repo, I have a hacky but working fix.

This is also causing issues for a big amount of CS2 plugins.


Fail test build if cannot find a matching node on Jenkins

Instead of waiting forever, fail test build if it cannot find a matching
node with the specified LABEL on Jenkins
Signed-off-by: lanxia<EMAIL_ADDRESS>

no matching machines: Grinder/10070

Could not find any nodes with 'ci.role.test&&sw.os.linux&&hw.arch.aarch64&&ci.project.openj9' label
[Pipeline] }
[Pipeline] // stage
[Pipeline] End of Pipeline
java.lang.AssertionError: Cannot find any machine that matches with the LABEL.
...
Finished: FAILURE

xlinux: Grinder/10071/


Redesign and Refactor

Hi, we're currently redesigning and refactoring the whole code so if you were looking to contribute any major feature to the repository, I will suggest to wait for a week since the code may change a lot and to avoid lots of merge conflicts later.
Thanks,
Saurabh

Done! #54


Allow subdomain-aware URL reversing to work when serving on non-standard port

This enables {% url ... subdomain='something' %} and reverse() to produce
useful results when used in runserver. Tested with runserver_plus.

Yay! Travis likes it!

Also added support for automatically determining the scheme (http/https) from the request if present in the template.

Currently running into this issue. Would be great to have this merged. Any workarounds for now? (For testing on localhost:8000 get it to return subdomain.localhost:8000/path on reverse)

Very much +1 on this, if this means it solves the problems when using this app during development on localhost!

@hakanw I fixed this by adding the port to the site domain on localhost, and it now works well even with the 8000 port.
That is, in the django shell run:
>>> from django.contrib.sites.models import Site
>>> site = Site.objects.all()[SITE_ID]
>>> site.domain = 'localhost:8000'
>>> site.save()


Fix serialization of Pageable.unpaged

Closes #189
I also rewrote the test assertion using AssertJ capabilities, for more explicit failure messages, I hope it's not a problem

Great work @Glosur ! Congratulations on your PR :)

I've cherry-picked this to 2.1.x branch


Enabled PSSA rule violations to fail build - Fixes #59

Pull Request (PR) description
This PR enables PSSA rule violations to fail build.
This Pull Request (PR) fixes the following issues:

Fixes #59

Task list:

[x] Change details added to Unreleased section of CHANGELOG.md?
[ ] Added/updated documentation, comment-based help and descriptions in .schema.mof files where appropriate?
[ ] Examples appropriately updated?
[ ] New/changed code adheres to Style Guidelines?
[ ] Unit and (optional) Integration tests created/updated where possible?

@johlju - would you mind reviewing this one? Not sure how I left it behind.

This change is

Reviewed 6 of 6 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved, some commit checks failed.

CHANGELOG.md, line 5 at r1 (raw file):

## Unreleased

- Added Hub and Spoke replication group example - fixes [Issue #62](https://github.com/PowerShell/DFSDsc/issues/62).

You got the examples in this PR too. :smile:

Comments from Reviewable

Review status: all files reviewed at latest revision, 1 unresolved discussion.

CHANGELOG.md, line 5 at r1 (raw file):
Previously, johlju (Johan Ljunggren) wrote…
You got the examples in this PR too. :smile:

@johlju - Doh! I created a branch off another branch! But this should be correct now :grin:

Comments from Reviewable

Review status: all files reviewed at latest revision, all discussions resolved, all commit checks successful.

Comments from Reviewable


fix(*): quote -var value to allow for spaces, etc.

Adds logic to surround a given -var value with single quotes, to allow for vars with spaces, etc.

Used the examples/guidelines from Terraform via https://www.terraform.io/docs/configuration/variables.html#variables-on-the-command-line
Noticed that the example bundle was failing as the myvar default value of "porter rocks!" was getting split when running the command:
...
Terraform has been successfully initialized!

You may now begin working with Terraform. Try running "terraform plan" to see
any changes that are required for your infrastructure. All Terraform commands
should now work.

If you ever set or change modules or backend configuration for Terraform,
rerun this command to reinitialize your working directory. If you forget, other
commands will detect it and remind you to do so if necessary.
stat rocks!: no such file or directory
err: error running command terraform apply -auto-approve -input=false -var myvar=porter rocks!: exit status 1
Error: error running command terraform apply -auto-approve -input=false -var myvar=porter rocks!: exit status 1
...

Thanks for the review @jdolitsky !


Dependency package @rollup/plugin-replace has a new version available

We have identified a dependency on version 2.4.2 of @rollup/plugin-replace. A new version (5.0.2) is available for upgrade.
Following are the steps to upgrade package dependency.

Understand the breaking changes between the version being used and the version you want to upgrade to.

Identify all packages that take a dependency on this package.

Go to the root folder for each such package (/sdk/service-name/package-name) and update package.json to have the new version.

Run rush update to ensure the new version is pulled in.

Make relevant changes to absorb the breaking changes.

Repeat steps 3 to 5 for each of the packages that have a dependency on this package.

has been upgraded to v5


Images no longer display when using Elements and a public github repo

Current Behavior
Images hosted on a public github repo no longer display when viewing the project using the stoplight elements widget. They do display when viewing the hosted version.
It appears that the src field is pointing to the wrong URL.
Expected Behavior
Images should display consistently
Steps to Reproduce
View the hosted version of a markdown page and notice that the images correctly display.
View the elements version of the page and you will notice that there are no images displayed.
If you inspect the source of both pages you will notice that the hosted version has

while the elements version has

fetching the image URL https://stoplight.io/api/v1/projects/cHJqOjODI5/branches/YnI6MTUwMDE1/images/%2Fdocs%2FGuides%2FAutomate%example.png
returns a JSON document instead of the expected png image.

Well... It's not pointing to the "wrong" url per se, just a new one that only returns JSON because we weren't thinking clearly. We'll make it return an image!


Fix typo update README.md

Fix typo update README

The proposed change would break the link. Also, the project is called "webhint".


Cloud Native Chapter 5

Summary
Chapter 5 files. TOC and images are in ch1 PR.

There's still some left apparently

looks good now

Ok, thanks @ardalis. @rpetrusha volunteered to review this chapter.

@robvet do you want to complete Cosmos and Azure Storage Services in this PR or as a follow-on?

It’s been done. I’m updating that md right now.
Thanks!
Rob Vettor | Principal Cloud System Architect
| ASD IP Architect
| Developer Advisory Services for Azure
[Description<EMAIL_ADDRESS>214-707-0584
[Description<EMAIL_ADDRESS> robvet<EMAIL_ADDRESS><EMAIL_ADDRESS>www.thinkingincloudnative.comhttp://www.thinkingincloudnative.com/
From: Steve Smith<EMAIL_ADDRESS>Sent: Monday, June 24, 2019 12:30 PM
To: dotnet/docs<EMAIL_ADDRESS>Cc: Robert Vettor<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [dotnet/docs] Cloud Native Chapter 5 (#12932)
@robvethttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Frobvet&data=02|01|robvet%40microsoft.com|5356cfef92c146745f3608d6f8c983f3|72f988bf86f141af91ab2d7cd011db47|1|0|636969941726112793&sdata=ubWf6o6dnAucrLn48BQVebKErc1eAYOvd1ZZQHZzU3g%3D&reserved=0 do you want to complete Cosmos and Azure Storage Services in this PR or as a follow-on?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fdotnet%2Fdocs%2Fpull%2F12932%3Femail_source%3Dnotifications%26email_token%3DABFIQYBESIAHOKQQWYMBUK3P4D77VA5CNFSM4HZ7MCR2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODYNUUOY%23issuecomment-505104955&data=02|01|robvet%40microsoft.com|5356cfef92c146745f3608d6f8c983f3|72f988bf86f141af91ab2d7cd011db47|1|0|636969941726122791&sdata=rMa8YIP5HtPb%2BtFTeiMYBKql7CzE29uJl7KHOJGX90s%3D&reserved=0, or mute the threadhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FABFIQYCTAAJUIRUUQEPK3D3P4D77VANCNFSM4HZ7MCRQ&data=02|01|robvet%40microsoft.com|5356cfef92c146745f3608d6f8c983f3|72f988bf86f141af91ab2d7cd011db47|1|0|636969941726122791&sdata=bUb283wyL5thRUitqqxVbRNWTVX08iVrC5Wcf1PX%2BFo%3D&reserved=0.

Give me an hour to finish updating it and I’ll do a PR.
Thanks!
Rob Vettor | Principal Cloud System Architect
| ASD IP Architect
| Developer Advisory Services for Azure
[Description<EMAIL_ADDRESS>214-707-0584
[Description<EMAIL_ADDRESS> robvet<EMAIL_ADDRESS><EMAIL_ADDRESS>www.thinkingincloudnative.comhttp://www.thinkingincloudnative.com/
From: Robert Vettor
Sent: Monday, June 24, 2019 12:31 PM
To: dotnet/docs<EMAIL_ADDRESS>dotnet/docs<EMAIL_ADDRESS>Cc: Mention<EMAIL_ADDRESS>Subject: RE: [dotnet/docs] Cloud Native Chapter 5 (#12932)
It’s been done. I’m updating that md right now.
Thanks!
Rob Vettor | Principal Cloud System Architect
| ASD IP Architect
| Developer Advisory Services for Azure
[Description<EMAIL_ADDRESS>214-707-0584
[Description<EMAIL_ADDRESS> robvet<EMAIL_ADDRESS><EMAIL_ADDRESS>www.thinkingincloudnative.comhttp://www.thinkingincloudnative.com/
From: Steve Smith <notifications<EMAIL_ADDRESS>Sent: Monday, June 24, 2019 12:30 PM
To: dotnet/docs <docs<EMAIL_ADDRESS>Cc: Robert Vettor <robvet<EMAIL_ADDRESS>Mention <mention<EMAIL_ADDRESS>Subject: Re: [dotnet/docs] Cloud Native Chapter 5 (#12932)
@robvethttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Frobvet&data=02|01|robvet%40microsoft.com|5356cfef92c146745f3608d6f8c983f3|72f988bf86f141af91ab2d7cd011db47|1|0|636969941726112793&sdata=ubWf6o6dnAucrLn48BQVebKErc1eAYOvd1ZZQHZzU3g%3D&reserved=0 do you want to complete Cosmos and Azure Storage Services in this PR or as a follow-on?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fdotnet%2Fdocs%2Fpull%2F12932%3Femail_source%3Dnotifications%26email_token%3DABFIQYBESIAHOKQQWYMBUK3P4D77VA5CNFSM4HZ7MCR2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODYNUUOY%23issuecomment-505104955&data=02|01|robvet%40microsoft.com|5356cfef92c146745f3608d6f8c983f3|72f988bf86f141af91ab2d7cd011db47|1|0|636969941726122791&sdata=rMa8YIP5HtPb%2BtFTeiMYBKql7CzE29uJl7KHOJGX90s%3D&reserved=0, or mute the threadhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FABFIQYCTAAJUIRUUQEPK3D3P4D77VANCNFSM4HZ7MCRQ&data=02|01|robvet%40microsoft.com|5356cfef92c146745f3608d6f8c983f3|72f988bf86f141af91ab2d7cd011db47|1|0|636969941726122791&sdata=bUb283wyL5thRUitqqxVbRNWTVX08iVrC5Wcf1PX%2BFo%3D&reserved=0.

@rpetrusha Are we good to merge now?

I've approved and I'll merge now, @ardalis.


Calculator using Java

Calculator using Java
All the desired operation(addition , subtraction , multiplication and division) are performed.

Merged

This is an automated message from Fork, Commit, Merge [BOT].

Thank you for your contribution! Your pull request has been merged. The files have been reset for the next contributor.
What's next?
If you're looking for more ways to contribute, I invite you to check out my other projects. Just click here to find more. These projects contain real issues that you can help resolve. You can also check out the Influences section in the README to find more projects similar to this one.
Also please leave a star to this project if you feel it helped you, i would really appreciate it.
I look forward to seeing your contributions!


broken link Download the Python Android support package

main page item 3. Download the Python Android support package. This package contains two directories...
The link is broken, there doesn't seem to be any releases from that referred github project available for download,

Thanks for the report. Looks like my documentation got a little bit ahead of my release processes. I've just added a b1 for you to download.

thanks!


[FEATURE] allow selecting Type as strategy for graphql schemas

Thanks for the great improvements to the graphql schemas. I have a little suggestion how to select the operation:
Would it be possible to subclass the dict used for graphql_schema["Operation"] and provide a as_strategy method?
This would allow intuitively selecting an operation and test all fields.
Example:
graphql_schema["Query"].as_strategy()

This shortcut would maybe be nice for openapi too and could allow feature parity with the existing @parameterize decorator.
Further thought: would it also be possible to implement an as_strategy method directly on a schema (to test everything without selecting)?

Great idea! On it

Further thought: would it also be possible to implement an as_strategy method directly on a schema (to test everything without selecting)?

Yep


JSX transformation pragma : Inconsistent documentation

The doc (http://babeljs.io/docs/plugins/transform-react-jsx/) states :
{
    "plugins": [
        ["transform-react-jsx", { "jsxPragma": "dom" }]
    ]
}

But the transformer code does :
var id = state.opts.pragma || "React.createElement";
One of the them should be renamed to pragma or jsxPragma.

Thanks! - renamed to pragma in https://github.com/babel/babel.github.io/commit/e3c936d1b7bd2d489ebc55b042bead097e31dae4


Merge pysb.tools.bngSSA simulator back into pysb.bng

BNGs ssa simulator is currently in pysb.bng.run_ssa. Every time it is called it generates the network, currently only has capabilities to run 1 simulation.
I wrote a bngSSA simulate class located pysb.tools.bngSSA. It is based off Leonards simulate class. We can initialize the simulator, generating the network only once. This allows us to 1.) save time by not regenerating the network when running the model again 2.) allow multiple runs with the same call.
I think the best thing to do is completely separate run_ssa from pysb.bng. Right now pysb.tools.stochkit simulator and pysb.tools.cupSODA solver are in tools. The base ode solver is in pysb.integrate. If we did an overhaul and move all simulate methods to a common area, I think it would be easier to see the options.  I an idea pysb.simulator(model,tspan,method = 'bng_ssa' ['cupSODA','lsoda','stochkit','pla',...], args). Having each method in a separate location seems cumbersome.
Any thoughts?
I wrote this to train models in parallel using ssa. Each process starts a perl call to run_network. I pass the bngl file created by the simulator the pregenerated network. Right now I cannot think of a way to discard this at the end of the simulations. If we recall BNGssa.run(), we don't want to use the cleanup flag because it deletes the network. Right now it just leaves it there.
Does anyone have any suggestions on what to do with the temporary generated network at the end of the script?

This was implemented a while ago (run_ssa is now in pysb.bng). Longer term, it should probably be deprecated in favor of the BngSimulator (or at least make run_ssa a thin wrapper to the Simulator).


[BUG] Incorrect curly brace matching

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
Curly brace is matched with type containing parenthesis:

Expected Behavior
Curly braces should be matched with curly braces
Steps To Reproduce

Put the text cursor next to the curly brace in the following code:

entity UserRoleGroups : cuid {
    name        : String not null @mandatory;
    description : String(500);
}

Versions
Sap CDS Language Support: 1.1.0
Node.js: 20.13.1
OS / Environment
Windows
Repository Containing a Minimal Reproducible Example
No response
Anything else?
No response

Reproduced in WebStorm/macOS with this even simpler example:

Note, I have even separated the tokens String( 500 )

... works in VSCode

Hi @hakimio,
thanks for reporting.
In all likelyhood, this is a bug in IntelliJ's TextMate module. I'll get back to you once the JetBrains issue is clarified.

@tim-sh can you link the JetBrains issue?

sure: https://youtrack.jetbrains.com/issue/IJPL-162118/Bracket-matching


Wallpaper doesn't change / Manjaro Xfce

Thank you for this tool, it works great.
But it doesn't change my wallpaper.

Hum, the workaround for me was to use Styli.sh https://github.com/thevinter/styli.sh with pywal, Styli.sh supports changing wallpaper on Xfce.


"SIGHETU MARMAȚIEI", "Iapa", "Lazu Baciului", "Șugău", "Valea Cufundoasă" and "Valea Hotarului" are listed in the drop-down menu

Description:
"SIGHETU MARMAȚIEI", "Iapa", "Lazu Baciului", "Șugău", "Valea Cufundoasă" and "Valea Hotarului" are listed in the drop-down menu alongside its corresponding county and commune, according to the law
Precondition:
The website is up an running.

Step 1
Type "SIGHETU MARMAȚIEI" in the search bar.

Expected results
The location is listed in the drop-down menu as "Municipiul SIGHETU MARMAȚIEI, MARAMUREȘ" and "Oraș SIGHETU MARMAȚIEI, MARAMUREȘ (SIGHETU MARMAȚIEI)".

Step 2
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 3
Type "Iapa" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Iapa, MARAMUREȘ (SIGHETU MARMAȚIEI)".

Step 4
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 5
Type "Lazu Baciului" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Lazu Baciului, MARAMUREȘ (SIGHETU MARMAȚIEI)".

Step 6
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 7
Type "Șugău" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Șugău, MARAMUREȘ (SIGHETU MARMAȚIEI)".

Step 8
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 9
Type "Valea Cufundoasă" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Valea Cufundoasă, MARAMUREȘ (SIGHETU MARMAȚIEI)".

Step 10
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 11
Type "Valea Hotarului" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Valea Hotarului, MARAMUREȘ (SIGHETU MARMAȚIEI)".


Add localization for table component

Currently, the table component does not support localizations. It is necessary to modify the labels in the js file and it's not practical in case of update.
In general, all labels should be localizable.
ex. :
itemsSteps: "all, 10,25,50,100",         itemsAllTitle: "Show all",         listItemsCountTitle: "Show entries:",         listSearchTitle: "Search:",         listInfoTitle: "Showing $1 to $2 of $3 entries",         paginationPrevTitle: "Prev",         paginationNextTitle: "Next"
tableRowsCountTitle: "Show entries:",         tableSearchTitle: "Search:",         tableInfoTitle: "Showing $1 to $2 of $3 entries",         paginationPrevTitle: "Prev",         paginationNextTitle: "Next",         allRecordsTitle: "All",         inspectorTitle: "Inspector",         tableSkipTitle: "Go to page",

In 4.4.3 to i18n files added section table:
"table": {
    "rowsCount": "Show entries:",
    "search": "Search:",
    "info": "Showing $1 to $2 of $3 entries",
    "prev": "Prev",
    "next": "Next",
    "all": "All",
    "inspector": "Inspector",
    "skip": "Goto page",
    "empty": "Nothing to show"
}

Now translation added for uk-UA, ru-RU. For others used the English version. Wait for PR from the community to change localizations for other locales.

The old attributes work if you define these.


tcp_proxy: save response headers for CONNECT upstreams

We have a network telemetry extension that wants to use some metadata passed as a response header for CONNECT tunneling initiated by tcp_proxy. We want an ability to expose those headers to network extensions beyond just checking for 200. I have a couple of ideas:

copy headers to the downstream L4 filter state; maybe require explicit names and mapping;
copy headers as dynamic metadata to the downstream L4 filter state;
create a mechanism to make upstream L7 filters somehow access downstream L4 stream info;
rewrite the telemetry to be an upstream L7 filter instead for CONNECT.

Thoughts? @alyssawilk

I think optionally (if configured) putting the headers somewhere in L4 (probably in filterState) makes sense.
Upstream filters can be registered for an UpstreamCodecFilter::onUpstreamConnectionEstablished() callback, and I think changing that API to actually share the connection would be simple if that SGTY

I think this is fairly unusual, so the least impact option is I think (1) with explicit header names, so I'll get that going.


[#101512016] Clarification question response emails

See: https://www.pivotaltracker.com/story/show/101512016
This sends an email to the user who submitted a clarification question, to confirm that we have received it.  The email looks like this:

I also tested the do-not-reply email address, which seems to do the job:

:+1:


Use ACL in JedisSentinelPool

and other possible cases.

To track overall changes since 3.2.0, I'd request to take a look at https://github.com/xetorthio/jedis/compare/jedis-3.2.0...sazzad16:user-1


npm by tarball approach

I've updated the post to use a tarball from a the forked repo instead of an npm package

Thanks for contributing this one. Could you issue this pull request against the dev branch ? The master branch is intended for releases only.

My mistake! sorry, I made a pull request for the wrong project! now I understand why the other one didn't make it :)
thank you for your response

No worries :) In the future, however, if you do feel like contributing you are more than welcome to do so - just remember the dev branch detail :)
Andre


Dreams Managers and crew might change order or disappear over time

The issue is revolving around a thing with rails that first and [0] don't necessarily return the same object on association. It hits us in [1] and [2], where due to creations and deletions or people from the system, @camp.people.first != @camp.people[0] and then the person on @camp.people[0] won't show and that person might change over time. I am not sure how rails decides this order, but it is probably has something to do with the ID numbers.
[1]https://github.com/Midburn/Dreams/blob/master/app/views/camps/_form.haml#L357
[2]https://github.com/Midburn/Dreams/blob/master/app/views/camps/_form.haml#L376

Best description of this that I found on google: https://coderwall.com/p/b-kfba/when-limit-1-first-is-orders-of-magnitude-slower-than-limit-1-0-in-rails
first adds an ORDER BY id  to the sql search, which for us on some case will cause the return value to be different from @camp.people[0] and this will hide this entry and display the .first one twice.


Replace "Create GitHub workflow" commands with equivalent Draft command

Now we have added the commands for running Draft, we have two ways of creating a GitHub workflow:

The 'Create GitHub workflow' commands in the Cloud Explorer tree view
The 'Draft: Create a GitHub workflow' command in the Command Palette

To avoid confusion, it would be best to provide a single approach. This PR is to make that change, by adding the relevant Draft commands to the tree view, in place of the existing commands:
Current

New

Once launched from the tree view, any cluster-related fields will be pre-populated:

I'm not sure whether we'd want Draft Deployment to appear there. I added it because it can take the cluster as an input, but it's not really necessary (it's only used for providing a list of namespaces to choose from).

Closing this as we are making these changes in two stages:

adding the Draft menu items and marking the starter workflow ones as deprecated
removing the starter workflow commands


Fix Mouse scroll not working in FireFox

A bug we found in our local fork of Visjs Network, especially when other mouse event listeners are added, FireFox now uses 'wheel' instead of 'DOMMouseScroll' - 'DOMMouseScroll' is deprecated: link

Hi @Tyler-Maclachlan,
thanks for your contribution but I have a question. Won't the handler be fired multiple times? Once for each event supported in given browser?

Hi @Thomaash,
You would think so but no it doesn't. Perhaps due to 'DOMMouseScroll' being deprecated in the browser I'm using.
(It seems 'DOMMouseScroll' fires on mobile sizes, whereas 'wheel' fires on desktop sizes)

Great will do.
I'm using FireFox 69.0.1 (64-bit) on Fedora Linux and I had to add this to get it to work again. Will update for newer FireFox as well.

I've been doing some looking and it seems 'wheel' is meant to be the standard.
https://developer.mozilla.org/en-US/docs/Web/API/Document/wheel_event - their example works on all my browsers. I may have forgotten a part of the code I used originally to fix the issue, will update the PR with the changed comments and if I forgot something I'll add it in.

I'm just gonna change it to @mojoaxel suggestion to just detect which event is supported

Can't we just use the wheel event only? The other events are for IE8, Opera 17, Firefox 16, Chrome 30. I highly doubt any of these work anyway as we don't include polyfills for them.

@Thomaash
We can't only use 'wheel' as just 2 versions of Firefox ago uses 'DOMMouseScroll'

@Tyler-Maclachlan I have no idea where you got this but just to be sure I deleted all the other events from the library and kept only wheel. Then I downgraded to Firefox 52 (the oldest we include polyfills for) and tested it. Zooms without an issue. According to https://caniuse.com/#search=wheel this should go all the way back to Firefox 17. I didn't test that though as we don't include polyfills for that. It probably doesn't work anyway.
Since the only browser that has ever supported DOMMouseScroll seems to be Firefox and it has support for wheel in all of the versions we support I see absolutely no reason to keep DOMMouseScroll.

I have removed the other listeners :+1:

I messed up my fork so I had to re-fork vis, I created a new PR, #208 for the new fork.
I get what you meant now, and I think the code is a lot cleaner now.


there are fewer gems in the new apoteosis (1.20)

why is that can someone explain?

No gems have been removed in 1.20 as compared to 1.19.2's latest release.


fix: support existing deprecated Rebalance.Strategy field usage

see https://github.com/Shopify/sarama/issues/2351

Thanks!


A better literate documentation mechanism

The literate documentation system we have today (where we include a piece of a file in a subdirectory) is not good enough. It requires a lot of overhead, making the additional file for every example you want to include and then referencing it. The overhead makes people skip it for small one-line examples, which makes it miss the point (because the one-line examples inevitably go out of date). This issue describes mechanism which will both lower the bar to entry, and make it impossible to write uncompiled code samples.
I will describe it using an example, which I think will be more clear than a lot of prose:
Current mechanism

test/integ.some-example.lit.ts

import mylib = require('../lib');

class SomeStack extends cdk.Stack {
  constructor(...) {
     super(this, id);

     ///! show
     const myconstruct = new mylib.Construct(this, 'hello');
     // relevant example goes here
    /// !hide
  }
}

const app = new cdk.App();
new SomeStack(app, 'mystack');
app.run();

README.md

Welcome to the readme for my library!

Here is how you use it:

[example of using my library](test/integ.some-example.lit.ts)

As you can see above, ...

Proposed mechanism

test/docfixture.default.tpl

import mylib = require('../lib');

class SomeStack extends cdk.Stack {
  constructor(...) {
     super(this, id);

     /* HERE */
  }
}

const app = new cdk.App();
new SomeStack(app, 'mystack');
app.run();

README.md

Welcome to the readme for my library!

Here is how you use it:

` ` `ts
const myconstruct = new mylib.Construct(this, 'hello');
// relevant example goes here
` ` `

If you want to integrate with another resource:

` ` `ts
/// <prereq>
// This won't be displayed in the README but it will be compiled.
const other = new otherlib.Resource();
/// </prereq>

const myconstruct = new mylib.Construct(this, 'hello');
myconstruct.connectTo(other);
` ` `

Bottom line

We will insert all MarkDown blocks fenced by ```ts into a template and the resulting code must compile.
Additional instructions (using a different template than the default, extra lines of code just to make this example work) can be given via special comment syntax, which also won't end up in the final code. Proposal: ///.
It is not allowed to have code blocks in a README and NOT have a fixture file to compile them against. That is: all code must be compile-checked.
Implemented by jsii.

Extensions:
We should probably extend the same courtesies to code blocks encountered in doc comments.

I am wondering if it would be nicer to simple compile the readme.md file during build (and check in the result), so we can have the readme completely self contained (with fixtures and all).

Will we add the compile step to checkin? I.e., one cannot check in a file with a code example that does not compile?

I am wondering if it would be nicer to simple compile the readme.md file during build (and check in the result), so we can have the readme completely self contained (with fixtures and all).

I am not sure what you mean by "compiling the readme", or what checking in the result means, but I did plan on applying the mechanism describe above during the jsii build.

What I meant is that maybe there's a README.md.src that includes both markdown and also code blocks with directives (//!) to the renderer:
Let's define our first CDK stack.

'''sh
//!exec
//!hide
npm i @aws-cdk/cdk, @aws-cdk/aws-sqs
'''

Create a file `stack.ts`:

'''ts
//!new-file:stack.ts
import { Stack } from '@aws-cdk/cdk';

export class MyStack extends Stack { }
'''

Now, create a file `app.ts`:

'''ts
//!new-file:app.ts
import { App } from '@aws-cdk/cdk';
import { MyStack } from './stack';

const app = new App();
new MyStack(app, 'MyFirstStack');
'''

'''sh
//!exec
//!hide
tsc
'''

Now, let's add a resource to our stack:

'''ts
//!edit-file:stack.ts
//!delete-lines:4-4
export class MyStack extends Stack {
  constructor(..) {
    super();
    new sqs.Queue(this, 'MyQueue');
  }
}
'''

YMMV of course... Just made up those directives...
Then, we have a little renderer that reads this README.md.src, executes all the directives and produces a README.md file as a result without directives or hidden stuff.
The nice thing here is that we can do pretty crazy things and generate portions of the readme, markdown, common headers and footers, etc.
We can decide if we check in the READM.md file. Not required. Eventually the important thing is that the README in npmjs, pypi and our docs website is the rendered version.

But those are details, and it occurs to me that you're trying to solve a slightly different problem than I am.
I'm interested in having code snippets appear in the docs syntax-checked. The snippets just appear by themselves in either the README or the doc blocks of methods and classes. That really doesn't need much more than a project template and some holes. It doesn't need sequencing, it doesn't need file editing, it doesn't need arbitrary package installs.
You seem to be solving the problem of a walkthrough (a la the getting started guide, or the workshop site), where we're taking a user by the hand, telling them "do this now, do that now, run this command, modify this file you created previously to add these lines, etc.", and having that validated to make sure it's still correct.
That problem is more generic, and will need more flexibility. We can probably combine the two in some way.

I don't README.md.src is necessary.
If we did that, we wouldn't have a README visible on GitHub, which is not great, or we would have to commit the generated README.md (also not great).
That decisions doesn't need to impact what gets published to NPM or PyPI, those markdowns could be post-processed, loaded from the JSII Assembly.

While @eladb's solutions strikes me as more flexible, it also feels a little over-complicated to me. I also worry about it being kind of brittle (edit commands for example, are quite brittle).


[Snyk] Security upgrade setuptools from 39.0.1 to 65.5.1

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to fix one or more vulnerable packages in the `pip` dependencies of this project.
Changes included in this PR

Changes to the following files to upgrade the vulnerable dependencies to a fixed version:

requirements.dev.txt

Vulnerabilities that will be fixed
By pinning:

Severity
Issue
Upgrade
Breaking Change
Exploit Maturity

Regular Expression Denial of Service (ReDoS) SNYK-PYTHON-SETUPTOOLS-3180412
setuptools:  39.0.1 -> 65.5.1
No
No Known Exploit

Some vulnerabilities couldn't be fully fixed and so Snyk will still find them when the project is tested again. This may be because the vulnerability existed within more than one direct dependency, but not all of the affected dependencies could be upgraded.
Check the changes in this PR to ensure they won't cause issues with your project.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
🛠 Adjust project settings
📚 Read more about Snyk's upgrade and patch logic

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Regular Expression Denial of Service (ReDoS)

This is a false positive. We are on 68.1, and it wants to upgrade from 39.0 to 65.5 🤔