@recipe implementation v1

Main plotting functions are now implemented with recipes:

Drawing points, arrows, segments and trajectories
Ellipsoids
Grids
Domains

Maybe not optimal, don't hesitate to criticize in the review

Codecov Report

Merging #224 (796f2da) into master (8fd9b15) will decrease coverage by 0.53%.
The diff coverage is 0.00%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@            Coverage Diff             @@
##           master     #224      +/-   ##
==========================================
- Coverage   40.80%   40.27%   -0.53%
==========================================
  Files          59       59
  Lines        4544     4588      +44
==========================================
- Hits         1854     1848       -6
- Misses       2690     2740      +50

Impacted Files
Coverage Δ

src/domain/domain_list.jl
50.00% <0.00%> (-2.69%)
:arrow_down:

src/domain/grid.jl
28.68% <0.00%> (-2.42%)
:arrow_down:

src/domain/nested_domain.jl
58.51% <0.00%> (-1.07%)
:arrow_down:

src/symbolic/alternating_simulation.jl
56.86% <0.00%> (ø)

src/symbolic/ellipsoidal_transitions.jl
33.23% <0.00%> (ø)

src/symbolic/lazy_ellipsoidal_symbolic.jl
0.00% <0.00%> (ø)

src/symbolic/lazy_symbolic.jl
60.00% <0.00%> (ø)

src/symbolic/nested_symbolic.jl
0.00% <0.00%> (ø)

src/utils/data_structures/sorted_vector_set.jl
76.74% <ø> (ø)

src/utils/data_structures/tree.jl
50.68% <0.00%> (ø)

... and 3 more

... and 3 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

it's great !


TGDB IDs

👍

LGTM

Current coverage is 100% (diff: 100%)

Merging #15 into master will not change coverage

@@           master   #15   diff @@
===================================
  Files           2     2
  Lines          33    33
  Methods         0     0
  Messages        0     0
  Branches        0     0
===================================
  Hits           33    33
  Misses          0     0
  Partials        0     0

Powered by Codecov. Last update b82acb0...54d2c3c


pygal.maps.world not working with spyder v3.2.4

Description
In Spyder v3.2.4 when attempting to use the pygal.maps.world library, trying to render the svg to a file results in the following traceback error (copied and pasted to bottom of this).  Please note, the following code was tested in Jupyter on same PC.  If it matters, I'm using the Anaconda Navigator.
What steps will reproduce the problem?

To reproduce the problem try the following code:

from pygal.maps.world import World
wm = World()
wm.title = 'North Central and South America'
wm.add('North America', ['ca', 'mx', 'us'])
wm.add('Central America',['bz', 'cr', 'gt', 'hn', 'ni', 'pa', 'sv'])
wm.add('South America',['ar', 'bo', 'br', 'cl', 'co', 'ec', 'gf', 'gy', 'pe', 'py', 'sr', 'uy', 've'])
wm.add('North America', {'ca':34126000, 'us':309349000, 'mx': 113423000})

wm.render_to_file('americas.svg')

What is the expected output? What do you see instead?
The expected output is a svg file that looks like the attached file (I couldn't attach an svg, so this is an equivalent png).

Please provide any additional information below
Traceback (most recent call last):
File "", line 1, in
wm.render_to_file('americas.svg')
File "C:\Users\user\AppData\Roaming\Python\Python36\site-packages\pygal\graph\public.py", line 114, in render_to_file
f.write(self.render(is_unicode=True, **kwargs))
File "C:\Users\user\AppData\Roaming\Python\Python36\site-packages\pygal\graph\public.py", line 52, in render
self.setup(**kwargs)
File "C:\Users\user\AppData\Roaming\Python\Python36\site-packages\pygal\graph\base.py", line 185, in setup
self.state = State(self, **kwargs)
File "C:\Users\user\AppData\Roaming\Python\Python36\site-packages\pygal\state.py", line 35, in init
merge(self.dict, graph.config.class.dict)
File "C:\Users\user\AppData\Roaming\Python\Python36\site-packages\pygal\util.py", line 384, in merge
dict1[key] = mergextend(val, dict1.get(key, ()))
File "C:\Users\user\AppData\Roaming\Python\Python36\site-packages\pygal\util.py", line 370, in mergextend
if list1 is None or _ellipsis not in list1:
TypeError: argument of type 'Key' is not iterable
Version and main components

Spyder Version: 3.2.4
Python Version: 3.6.3
Qt Versions: 5.6.2, PyQt5 5.6 on Windows

Dependencies
pyflakes >=0.6.0 :  1.6.0 (OK)
pycodestyle >=2.3:  2.3.1 (OK)
pygments >=2.0   :  2.2.0 (OK)
pandas >=0.13.1  :  0.20.3 (OK)
numpy >=1.7      :  1.13.3 (OK)
sphinx >=0.6.6   :  1.6.3 (OK)
rope >=0.9.4     :  0.10.5 (OK)
jedi >=0.9.0     :  0.10.2 (OK)
nbconvert >=4.0  :  5.3.1 (OK)
sympy >=0.7.3    :  1.1.1 (OK)
cython >=0.21    :  0.26.1 (OK)
qtconsole >=4.2.0:  4.3.1 (OK)
IPython >=4.0    :  6.1.0 (OK)
pylint >=0.25    :  1.7.4 (OK)

Thanks for reporting. Looking at the error trace, it appears to be entirely an issue in pygal, and as the latter is not on conda that I can tell, it can't be easily reproduced. Does the same code run in qtconsole, IPython and/or the regular python interpreter? If not, it is not an issue with Spyder and thus you should report to whichever package on that list is the last it didn't work in, or pygal itself if it didn't work in any and you can confirm it isn't a user side issue, and close this one. Further, if you installed it with pip, that can also be a major source of problems and we don't really recommend it for non-experts and thus only officially support conda/Anaconda, though you're welcome to get it to work on your own.
If this does end up being isolated only to spyder, please let us know how you installed spyder/its stack and pygal, as well as any other details about the problem and we can try to help, but otherwise we'll have to close this issue after a week (7 days) as from what we have so far, it doesn't appear to be an issue with Spyder's code specifically. Best of luck!

Okay, so I updated to spyder 3.2.5 and it seems to work fine the first time I run it.  Second time I press F5 it doesn't work.  It was a pip install, as the conda is only available on linux.
Regardless, thanks for the heads up on pip installs.


Fix volumeMounts format in helm chart

What changes are proposed in this pull request?
fix volumeMounts format in helm chart
Why are the changes needed?
fix https://github.com/Alluxio/alluxio/issues/14682
Does this PR introduce any user facing changes?
Enable metrics in helm chart can working

alluxio-bot, check this please

alluxio-bot, merge this please


Preserve "better" names in Constant Propagation

Names that do not start with '_' are "better" than those that do
Fixes #378 -- at least the original problem statement, as well as resolves how ConstProp was making good name removal much worse.
There is a reasonable debate to be had about how to pick between names that do not start with _. This does happen a bit in rocket-chip but most of them are replacing something like in_0_d_bits_source with io_out_0_d_bits_source although there are examples of things like nackResponseMessage_size being replaced by probe_bits_size or pmp0_cfg_x, pmp0_cfg_w and pmp0_cfg_r all being replaced by default.
In any case, this improves tons of names, we can fight later about how to differentiate between names that don't start with _.

Thanks. I tested this on my PLIC PR and it is So. Much. Better.


Slack - cannot authenticate / access

Bug description:
Slack in my organization is tied to Okta and Google authentication.
From a clean install, i add Slack service to my workspace.  I enter my credentials, then i get the Okta console and then it is stuck in the Sing in screen.
Other services that uses Okta or Google auth are working (e.g. Gmail, Trello, Evernote, etc)
Steps to reproduce:

Slack tied to Okta and Google authentication.
From a clean install, add Slack service to your workspace.
Enter  credentials
Then it is stuck in the Sing in screen instead of login to the Slack group

Expected behavior
Be able to access Slack group and channels after authentication
Screenshots

Environment:

Operating System: macOS Mojave

Ferdi Version: <IP_ADDRESS>6

Server: Ferdi with account

Debug information:
Attached

devconsole.log
debugingo.log

Hi, I have the same problem, my slack use saml authentication, when I enter the data to login, the callback of slack return the login of slack and don't having access

Hi team, just checking if there is an update.

@raicerk were you able to workaround this issue?

Same problem here. Slack + Google authentication doesn't work with Ferdi
Ferdi Version: <IP_ADDRESS>6
OS : Ubuntu 20.04

@wernerdiers no, I have not been able to solve it, so I analyzed, the problem is in the forwarding of the authentication keys that google has

Same issue on Ubuntu 20.04.1 LTS | Ferdi version 5.5.0 -> after logout from slack can't log in with 'Sign in with Google'

I have the same problem on MacOS 10.14.5

This is still an issue
On Wed, 20 Jan 2021, 10:32 stale[bot]<EMAIL_ADDRESS>wrote:

This issue has been automatically marked as stale because it has not had
recent activity. Please check if this issue is still relevant and please
close it if it's not. This will make sure that our open issues are actually
of use and reduce the list of obsolete issues. Thank you for your
contributions.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/getferdi/ferdi/issues/825#issuecomment-763210901, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ADBES7JSXY6PTRSJRKG3ZXTS2YJCHANCNFSM4OACVR3A
.

This is still an issue
On Wed, 20 Jan 2021, 10:32 stale[bot]<EMAIL_ADDRESS>wrote:

This issue has been automatically marked as stale because it has not had
recent activity. Please check if this issue is still relevant and please
close it if it's not. This will make sure that our open issues are actually
of use and reduce the list of obsolete issues. Thank you for your
contributions.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/getferdi/ferdi/issues/825#issuecomment-763210901, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ADBES7JSXY6PTRSJRKG3ZXTS2YJCHANCNFSM4OACVR3A
.

This is still an issue
El vie, 26 de mar. de 2021 a la(s) 23:35, stale[bot] (
@.***) escribió:

This issue has been automatically marked as stale because it has not had
recent activity. Please check if this issue is still relevant and please
close it if it's not. This will make sure that our open issues are actually
of use and reduce the list of obsolete issues. Thank you for your
contributions.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/getferdi/ferdi/issues/825#issuecomment-808181359, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ADBES7K7A2TU5OWOTFRXURTTFR5QJANCNFSM4OACVR3A
.

Slack in my organization is tied to Okta and Google authentication.
From a clean install, i add Slack service to my workspace. I enter my credentials, then i get the Okta console and then it is stuck in the Sing in screen.

Use Right Click in the bottom msg link (Open in browser) then select "Open link in Ferdi".

I also have this problem, it will go back to the slack login screen after tapping on the user.

This has been fixed in the latest versions of the recipe (and since I see that some of you are also on v5.5.0 of Ferdi - that also needs to be upgraded to the latest). Could I please request you to install the versions from this google drive folder? (This is because the CI process to build and publish the nightlies are currently broken and being worked on.
I have uploaded the versions only for macos, for others, please follow the manual steps for troubleshooting recipes

Using the Arm build 2021-May-15-19-59 - 0e0416ff and seeing this issue.

@Shadester - please see if this fixes the "corrupt file" issue: https://github.com/getferdi/ferdi/issues/875#issuecomment-841679581

Yes, it does.

I can still not login on slack. Might be the Okta login, but it will do the same as was decribed above, back to the login page again.

Something weird is going on. I was able to use Slack with Okta Oauth without any issues.

same here unable to connect using SAML

I'm on the same nightly 66 - and am able to login into a Slack team that is managed by okta. Can you please ensure that you have not manually edited the slack recipe files? and that you have the default settings for user-agent, etc? Are you running via a proxy?

Yeap confirmed...clean install, using SAML with my company google's hosted email....

Is this still an issue? I'm unable to pinpoint what might be the issue - since I have the same setup (slack with okta SSO, etc) on a mac and I dont see any of the issues mentioned. At this point, I'm running v73 of the nightlies.

I tried v73, but I can't add any services. When opening the "available services" screen, I get a "something went wrong" message and I can't add any. So I don't know if it works.

@Shadester - pls try v74 which I just released. This should be fixed now and you should be able to add new services.

I was able to see and add  the service, but still no luck in adding the Slack workspace.

I am also having the same issue here. I enter my credentials, get redirected to authorise Google and then for a fraction of a second I see it prompt me for my MFA Code before it reverts back to the Slack login screen.

I was having the same problem, being redirect to the login screen after putting my credentials, but I fixed it editing the service configuration and adding Mozilla/5.0 (X11; Linux x86_64; rv:90.0) Gecko/20100101 Firefox/90.0 to the User Agent setting:

Thanks for confirming @esaporski - this is a validation that the user-agent override works and that the slack authentication setup is somehow different to some of the users which is causing this weird behavior.

Awesome! It works for me now.

It seems a bit strange to me that this issue has been closed when we've found a potential workaround but no fix has been implemented. Is the expectation that everyone who wants to use Slack in Ferdi should do a Google search until they stumble across this page?

@DarylGraves - we can alway reopen the ticket. But, afaik, I too use Slack via Google/okta - and have not seen the issue that was reported. I had implemented the exposure of the user-agent with the intention of this comment. I feel that this is accomplished by this comment.

Ok no worries, thanks Vijay, and thank you for making this project what it is today!

I would add this to the FAQ or readme, so people would have an easier time to figure out the steps to fix it.

Excellent suggestion @Shadester .
Tagging @kytwb since I dont know how to add this to the "Help Center" on the getferdi website

Confirmed @esaporski works


NIFI-950: Perform component validation asynchronously, other cluster-related performance enhancements

This PR addresses NIFI-950 and a handful of other JIRA's. They were all done in a single PR because the modifications tend to build upon one another, and also because this PR really addresses the issue of making nifi clustering more responsive (in terms of UI) and more stable.
Thank you for submitting a contribution to Apache NiFi.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[ ] Is there a JIRA ticket associated with this PR? Is it referenced
in the commit message?

[ ] Does your PR title start with NIFI-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.

[ ] Has your PR been rebased against the latest commit within the target branch (typically master)?

[ ] Is your initial contribution a single, squashed commit?

For code changes:

[ ] Have you ensured that the full suite of tests is executed via mvn -Pcontrib-check clean install at the root nifi folder?
[ ] Have you written or updated unit tests to verify your changes?
[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[ ] If applicable, have you updated the LICENSE file, including the main LICENSE file under nifi-assembly?
[ ] If applicable, have you updated the NOTICE file, including the main NOTICE file found under nifi-assembly?
[ ] If adding new Properties, have you added .displayName in addition to .name (programmatic access) for each of the new properties?

For documentation related changes:

[ ] Have you ensured that format looks appropriate for the output in which it is rendered?

Note:
Please ensure that once the PR is submitted, you check travis-ci for build issues and submit an update to your PR as soon as possible.

Will review...

@mcgilman I am closing this PR because it needs to be rebased and I ran into an issue with a unit test during the rebase. Will open a new PR once that has been addressed.


Fix: host.docker.internal instead of host.docker.local

Changes
Tutorial.md says to use "host.docker.local" instead of "host.docker.internal" #2970
Submitter Checklist
These are the criteria that every PR should meet, please check them off as you
review them:

[x] Includes tests (if functionality changed/added)
[x] Includes docs (if user facing)
[x] Commit messages follow commit message best practices
[x] Release notes block has been filled in or deleted (only if no user facing changes)

See the contribution guide for more details.
Double check this list of stuff that's easy to miss:

If you are adding a new binary/image to the cmd dir, please update
the release Task to build and release this image.

Reviewer Notes
If API changes are included, additive changes must be approved by at least two OWNERS and backwards incompatible changes must be approved by more than 50% of the OWNERS, and they must first be added in a backwards compatible way.

/assign @dlorenc

Welcome and Thank you @juanri0s for this contribution 🙏
/ok-to-test
/release-note-none

Thanks for the commit!

/lgtm

/kind documentation
The kind check appears stuck, let's poke it and see...

/kind documentation

did I break it lol

@juanri0s nah, something seems fishy, invoking @afrittoli to dig into the why it seems stuck :stuck_out_tongue_closed_eyes:


Fix for Chef client 11, Include Chef::Mixin::ShellOut as needed.

This is to make sure that Chef client 11 does not terminate with
the NoMethodError exception when converging a node.

Ah. I have missed the following commit: a33a58d2a93e48c80ed07afb64aba4b1e25efcac.
Closing this one! Sorry @tas50 :)


filter out older version on docker hub

tool should look at the local tags and try and parse out the version numbers. Then using the oldest (lowest), it should filter results from Docker Hub so that only newer (higher) versions are displayed.
It's quite normal for a project to publish several 'streams' of versions (ie 9.7 and 9.6).  If the local machine only has 9.7, there is no point in listing any 9.6 tagged images, even if they are newer than images on the local machine

Note, if the current version is 2.7, the following versions should still be displayed: 2.7.1, 2.7, 2.  Its common for a project the publish an image under several tags (as in prev example).

completed


Pre-training for image captioning

Hello, and congrats for your brilliant work!
I’d like to ask. For image captioning, you mention in the appendix:

we directly fine-tune Oscar for image captioning on COCO without additional pre-training on Conceptual Captions

Does that mean you only use COCO dataset for pretraining, and not the rest (SBU, Flickr, GQA)? And the cider score of 1.4 is achieved after fine tuning the coco only pretrained model?

Hi, we used Oscar pre-trained model (on 6.5M image-text pair) to fine-tune on COCO for image captioning.
Image captioning uses seq2seq attention mask, which is different from the full attention mask used in Oscar pre-training. If we conduct additional pre-training on CC with seq2seq attention mask, it may further improve the performance. However, we directly fine-tune Oscar without additional pre-train, which shows the generalization ability of Oscar for both understanding tasks and generation tasks.
Hope this answers your question.

I see...so the big improvement in image captioning scores may be due to the additional datasets (other than COCO) used to pre-train the model?

large-scale pre-train data and the pre-train scheme (adding tags, loss functions) are all helpful for the performance improvement.

@xiyinmsu thanks for your reply. I'm just asking to know the performance if trained only with COCO, so it can be compared with image captioning models that don't use VL-pretraining.
Again, beautiful work!


Fix docs folder items shifted by expander arrow

The expander arrows were shifting the folder names and annoying me, so here's a fix. Before and after:

Wasn't sure if this should target master or develop, so I chose develop. I can change it if you want me to.

Hah, this is awesome! Actually, if you want to submit this to master so it goes live right away, that'd be fantastic :)

Made a small adjustment to fix keyboard navigation to bring it back to what it was, and added a visual focus indicator for the arrow.

Thank you! That looks so much more clean :)


Updated the deprecated link in image-proxy.rst

Summary
Updated the deprecated link in image-proxy.rst
Ticket Link

NA

Hello @Rutam21,
Thanks for your pull request! A Core Committer will review your pull request soon. For code contributions, you can learn more about the review process here.


Kernel freeze when running BuildConfig Pods on version 4.7.0-0.okd-2021-08-22-163618

Describe the bug

We have a cluster with user provisioned infrastructure using libvirt KVM virtual machines.
After updating to version 4.7.0-0.okd-2021-08-22-163618, some of the Nodes were becoming NotReady.
When that happens, the node is consuming 100% and doesn't respond to ping or from input in the console.
This is a regression, we didn't have such issue with the previous version 4.7.0-0.okd-2021-08-07-063045.
I noticed that the issue is triggered when running BuildConfig Pods on the node (testcase below).
Just before the node is unresponsive, there are still plenty of resources available (CPU, memory, disk).
According to the console output, it looks like a kernel panic is happening. Unfortunately, the panic message is not printed to the console, the last available line is the cut here message. For instance:
[  729.229244] audit: type=1138 audit(1631135817.973:413): pid=11270 uid=0 auid=4294967295 ses=4294967295 subj=system_u:system_r:spc_t:s0 msg='sw="libblkid-2.23.2-65.el7_9.1.x86_64" sw_type=rpm key_enforce=0 gpg_res=1 root_dir="/" comm="yum" exe="/usr/bin/python2.7" hostname=? addr=? terminal=? res=success'
[  731.433044] audit: type=1138 audit(1631135820.177:414): pid=11067 uid=0 auid=4294967295 ses=4294967295 subj=system_u:system_r:spc_t:s0 msg='sw="libblkid-2.23.2-65.el7_9.1.x86_64" sw_type=rpm key_enforce=0 gpg_res=1 root_dir="/" comm="yum" exe="/usr/bin/python2.7" hostname=? addr=? terminal=? res=success'
[  735.719951] audit: type=1138 audit(1631135824.464:415): pid=11270 uid=0 auid=4294967295 ses=4294967295 subj=system_u:system_r:spc_t:s0 msg='sw="pam-1.1.8-23.el7.x86_64" sw_type=rpm key_enforce=0 gpg_res=1 root_dir="/" comm="yum" exe="/usr/bin/python2.7" hostname=? addr=? terminal=? res=success'
[  738.137203] audit: type=1138 audit(1631135826.881:416): pid=11067 uid=0 auid=4294967295 ses=4294967295 subj=system_u:system_r:spc_t:s0 msg='sw="pam-1.1.8-23.el7.x86_64" sw_type=rpm key_enforce=0 gpg_res=1 root_dir="/" comm="yum" exe="/usr/bin/python2.7" hostname=? addr=? terminal=? res=success'
[  741.316403] ------------[ cut here ]------------

Version

4.7.0-0.okd-2021-08-22-163618, UPI, libvirt.
How reproducible

Happens most of the time when running the testcase below.
oc apply -f - <<"EOF"
apiVersion: build.openshift.io/v1
kind: BuildConfig
metadata:
  name: testcase
spec:
  output:
    to:
      kind: DockerImage
      name: your-registry.example.com/testcase:test
  source:
    type: Dockerfile
    dockerfile: |
      FROM centos:7

      RUN yum update -y && \
        yum -y groups install "GNOME Desktop"
  strategy:
    type: Docker
  resources:
    limits:
      cpu: 500m
      memory: 512Mi
    requests:
      cpu: 100m
      memory: 128Mi
EOF
oc start-build bc/testcase

Log bundle

Must-gather logs: https://1drv.ms/u/s!AloPbwYP-ZZVhYQZSIr6GyZYnxCQow

We consume Fedora kernel, so it needs to be reported there.
Lets leave this bug as a tracker

Hi,
We have excatly same problem here when updating to 4.7.0-0.okd-2021-08-22-163618.
Bare-metal nodes suddenly freeze on starting pods with message like (appear  not every time):
[Mon Sep 13 23:16:13 2021] ------------[ cut here ]------------
[Mon Sep 13 23:16:13 2021] rq->tmp_alone_branch != &rq->leaf_cfs_rq_list
[Mon Sep 13 23:16:13 2021] WARNING: CPU: 112 PID: 0 at kernel/sched/fair.c:401 enqueue_task_fair+0x26f/0x6a0

Issue is fixed with kernel 5.14.14-200.fc34.x86_64, available since OKD 4.8.0-0.okd-2021-11-14-052418.

@spasche @vrutkovs Any chance this going to be backported to OKD 4.7?

Latest OKD 4.7 should include kernel-5.14.14-200.fc34.x86_64


Use abPOA for gap alignments

This replaces the wfa-based gap sequence aligner with abPOA.  The default cactus scoring parameters are hardcoded in. It passes all tests but I haven't yet tested it at scale.    In particular it will only work to about ~100kb  -- should be fine in practice but at the very least needs an explicit check.
@benedictpaten do you mind testing it builds and runs for you?
As an aside, it wouldn't even pass the tests until I sorted the abpoa input by length -- the problem case is below:
printf ">1\nNNNNNNNNNNNNN\n>2\nNNNNNNNNNNNNNNNNNNN\n>3\nNNNNNNNNNNNNNN\n" > x.fa
taffy/submodules/abPOA/bin/abpoa -r 1 -m 0 x.fa
[main] CMD:  taffy/submodules/abPOA/bin/abpoa -r 1 -m 0 x.fa
>1
NNNNNNNNNNNNN-------
>2
NNNNNNNNNNNNN-NNNNNN
>3
NNNNNNNNNNNNNN------
[abpoa_main] Real time: 0.001 sec; CPU: 0.005 sec; Peak RSS: 0.004 GB.

but sorting works
printf ">2\nNNNNNNNNNNNNNNNNNNN\n>3\nNNNNNNNNNNNNNN\n>1\nNNNNNNNNNNNNN\n" > x.sort.fa
taffy/submodules/abPOA/bin/abpoa -r 1 -m 0 x.sort.fa
[main] CMD:  taffy/submodules/abPOA/bin/abpoa -r 1 -m 0 x.sort.fa
>2
NNNNNNNNNNNNNNNNNNN
>3
NNNNNNNNNNNNNN-----
>1
NNNNNNNNNNNNN------
[abpoa_main] Real time: 0.001 sec; CPU: 0.005 sec; Peak RSS: 0.004 GB.

I fixed the makefile, builds fine on my machine. But the Python build is messed up by the inclusion of abPOA. I'm not quite sure the right way to fix it.

Can you try building the Python part on your machine?:
https://github.com/ComparativeGenomicsToolkit/taffy?tab=readme-ov-file#installing-python-library-from-source

actually, maybe if I remove the merge_adjacent_alignments and have that in a separate header file we can keep a separation from the code in the python library...

Okay.. this all works on my mac (i.e. builds/tests okay)


Remove unused types

Type

[x] Bug Fix
[ ] New Feature
[ ] Sample

Related Issues?
No related issues
What is in this Pull Request ?
Remove types that are not referenced anywhere.
CertificateCrypto came up while checking a static analysis report for a security issue, then I ran another tool to find all unused types and it found PemStringType. I couldn't find any VS or Roslyn rule or warning related to this, otherwise it could be automated.

Thanks @jackpoz merged it !


How do I make custom achievements? (No Source Code)

Describe your problem here.
I don't know how to make an achievement :(
Are you modding a build from source or with Lua?
Lua
What is your build target?
Windows x64
Did you edit anything in this build? If so, mention or summarize your changes.
No response

n o

impossible with lua at the moment

Duplicate of #6511


Grenades/other items work as ammo

Description
Need items like grenades for example that can be thrown independently and still function but also work as consumable ammo for guns.
Configuration
Is_ammo?:
  Feature_Option: <true/false>

### Payment (optional)

- [X] I'm willing to pay for this feature

This was completed in a commission. Using the default config
    Ammo:
      Ammo_Types:
        Grenade_Ammo:
          Item_Ammo:
            Bullet_Item:
              Type: RED_CANDLE
              Name: "<gold>Grenade"
            Ammo_Converter_Check:
              Type: true
              Name: true

converts the grande to ammo for the gun you add this to


Route Methods

So I am not sure if I am just misreading the routes and such but given the example below, the first route is always picked up on POST methods.  If I understand this correctly, it shouldn't.  The only way the POST route is picked up is if its before the regular route.  Changing no other code than moving the POST route above the other, it will work as expected.
Flight::route('/someroute', function () {});

Flight::route('POST /someroute', function () {});

The first route being set to all types, it'll be picked for a POST.
If you wish for the second one to be picked, changing the order is the right way.

Related doc: By default, route patterns are matched against all request methods.
Ref: https://github.com/mikecao/flight#method-routing

Related doc: Routes are matched in the order they are defined. The first route to match a request will be invoked.
Ref: https://github.com/mikecao/flight#routing


0.99 modal: complete callback works only when closed via .modal-close

Expected Behavior
When closing a modal, regardless how it is done, the complete callback should be called.

Current Behavior

Right now, complete is only called, when the modal is closed via modal trigger close

There should be a callback, regardless if modal is closed with click on dismissible layer, ESC, etcl.
Possible Solution

While debugging intensively, I could see modals being initialized with proper options on document ready, but when clicking modal triggers, options didn't show up again.

Tried a own solution by "injecting" a third callback "onClose" to a position that should be always called. This is the code you can see on my page (sourcemap included for easier inspection)
Steps to Reproduce (for bugs)

see www.bannholzhof.at
Open Modal (e.g. "Anfahrt") -> URL changes to www.bannholzhof.at/anfahrt
Close Modal via "Schließen" -> URL will return to www.bannholzhof.at
Dismiss Modal otherwise (ESC, Overlay) and URL will stay the same

Context

Added routing to my site and I need the URL to be changed, basing on open/closing modals. Right now, router only gets called when clicking on "Schließen (close)"

        $('.modal').modal({             complete: onModalCloseNavigateToRoot         });
...
function onModalCloseNavigateToRoot() {     router.navigate('/'); }
Your Environment

Version used: 0.99
Browser Name and version:

Google Chrome Version 60.0.3112.101 (Offizieller Build) (64-Bit)
FireFox 54.0.1 (32-bit)
Microsoft Edge 40.15063.0.0

Operating System and version (desktop or mobile): Windows 10 Pro Version 1703
Link to your project (if appropriate): www.bannholzhof.at

This has been fixed in v.100.0


Active session indicator is out of sync

After switching from a remote session to local:

Note that both the Workspaces tool window and the status bar indicator say that the current session is the remote one, but the REPL has already switched to local, and reflects this in its title bar.

Did it finally switched to the local?

When I clicked on the arrow on "Local" to activate it (again), the switch was properly reflected everywhere. From there, it seems to be tracking everything fine.
Before I got into this stage, I started VS with Local, added a new connection (perhaps that is somehow important?), and switched to that remote connection and used it for a bit. Then switched back to local to get to the screenshot.

#2288


Scrolling group customization

We need some how customize scrolling group panel. Possible options:

Locate on top/buttom
Different combintaions of buttons:

[1] [prev] [10] [next] [100]
same with another text
[<<] [<] [current] [>] [>>]
[1] [9] [10] [11] [100]
fewer buttons:
[<] [10] [>]
[<] [>]
[<<] [<] [>] [>>]
all buttons for cases when pages <10
[1] [2] [3] [4] [5] [*6*] [7] [8] [9] [10]
[1] [2] [3] [*4*] [5]
[*1*] [2]
Mixed:
[ << 1 ] [ < 7 ] [ 8 ] [ *9* ] [10  > ]
[ <  1 ] [ *2* ] [ 3 ] [ 4 > ] [10 >> ]
Translation and styling current page should be available
see also: https://habr.com/ru/post/501812/

Very roughly, I imagine it to be something like this.
It allows to create the following pagers mantioned by you:
[1] [prev] [10] [next] [100]
[<<] [<] [current] [>] [>>]
[<] [10] [>]
[<] [>]
[<<] [<] [>] [>>]
It would be nice to use Group with custom pager, but I don't know how to properly pass __value__ (it keeps value of prev_page, current_page etc.) to buttons in a Group. Also, which class is it better to inherit from for the PagerButton class that I have created?
Here is an example.
I would like to hear your proposals and opinion 😀

https://github.com/Tishka17/aiogram_dialog/blob/develop/example/scrolls.py


Use structs to better abstract the server components

With the addition of CLI support (#4) the code started to get a little gnarly, things being passed where they oughtn't be.
Let's refactor the following:

[ ] new streaming processor (port and exec)
[ ] new "command sender" (text, port, exec)
[ ] port -> port and exec (as impls of streaming processor)
[ ] new one-shot processor (text)
[ ] text as impl of one-shot processor

Depends on:

#9

Did this a little differently than imagined ... and better.


Invisible NPCs

After unloading and reloading a chunk the npcs go invisible, their click commands do not work but the nametag is visible.

Same problem, any fix?


Add atomic-product-field-condition component for commerce

Related to #3956
Adds the atomic-product-field-condition component to enable conditional rendering based on commerce-specific fields in Coveo's UI Kit.

Implements the component with properties ifDefined and ifNotDefined to check for the presence or absence of specified fields.
Introduces mustMatch and mustNotMatch properties for conditional rendering based on matching field values, utilizing ProductTemplatesHelpers.
Ensures the component evaluates conditions on component load and removes itself from the DOM if conditions are not met, maintaining clean template structure.
Utilizes @ProductContext to access the current product's properties for condition evaluation.

For more details, open the Copilot Workspace session.

Introduces the atomic-product-field-condition component and implements it in commerce example pages.

Adds the atomic-product-field-condition component definition in components.d.ts, including properties for verifying if specified fields are defined or not defined.
Implements the atomic-product-field-condition component in the atomic-product-field-condition.tsx file, with logic to conditionally render content based on product field definitions.
Updates commerce example pages (homepage.html and search.html) to replace atomic-field-condition with the new atomic-product-field-condition component for conditional rendering based on product fields.

For more details, open the Copilot Workspace session.


Determining platform from a script (improvement 2.0.x)

It took a split of the script depending on whether it is running on any platform. As a result, it is found only "OS.get_name ()". Who also returns a string. Sometimes it is useful. But in most cases it would be necessary to function "OS.get_platform ()" that returns the number. And in the engine would be constant platforms. For example:
const PLATFORM_UNKNOWN = 0
const PLATFORM_ANDROID = 1
const PLATFORM_IOS = 2
const PLATFORM_OSX = 4

That they were flags. In this case, you can conveniently share the script depending on the target platform. For example:
if OS.get_platform () & (OS.PLATFORM_ANDROID | OS.PLATFORM_IOS):

Of course this functionality can be made to function through the statics but I would like that it was implemented in the engine.

Nowhere in godot a platform is harcoded, as a result you can add them and
remove them from platforms/ and they will magically work or not.
As that makes me proud and I want to keep it this way, please use the
string in OS.get_name()
On Wed, Jun 8, 2016 at 7:56 PM, ret80<EMAIL_ADDRESS>wrote:

It took a split of the script depending on whether it is running on any
platform. As a result, it is found only "OS.get_name ()". Who also returns
a string. Sometimes it is useful. But in most cases it would be necessary
to function "OS.get_platform ()" that returns the number. And in the engine
would be constant platforms. For example:
const PLATFORM_UNKNOWN = 0
const PLATFORM_ANDROID = 1
const PLATFORM_IOS = 2
const PLATFORM_OSX = 4
That they were flags. In this case, you can conveniently share the script
depending on the target platform. For example:
if OS.get_platform () & (OS.PLATFORM_ANDROID | OS.PLATFORM_IOS):
Of course this functionality can be made to function through the statics
but I would like that it was implemented in the engine.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/godotengine/godot/issues/5112, or mute the thread
https://github.com/notifications/unsubscribe/AF-Z236p3WK6hskzL7OXd5r4p5ShzDXXks5qJ0ikgaJpZM4IxfNf
.

Yes, I agree that my proposal has an advantage as well as disadvantage as constants. But the chance that someone other than you to add a platform not great. Or it will be added to the cooperation with you. And you may well be able to make one more constant. If all would do themselves and the constant will be able to add themselves. This is not a problem because the platform is rarely added. Think about it.

The cleanest way to solve this is with some configuration flag for whatever
you need. For example you can have this in engine.cfg
[game]
ui="res://ui/hud.scn"
[game.iOS]
ui="res://ui/hud_mobile.scn"
[game.Android]
ui="res://ui/hud_mobile.scn"
Then in your code you use load(Globals.get("game/ui")) to get the scene
with your UI, so you avoid having big branches of code that deal with
different platforms.
On 8 June 2016 at 20:10, ret80<EMAIL_ADDRESS>wrote:

Yes, I agree that my proposal has an advantage as well as disadvantage as
constants. But the chance that someone other than you to add a platform not
great. Or it will be added to the cooperation with you. And you may well be
able to make one more constant. If all would do themselves and the constant
will be able to add themselves. This is not a problem because the platform
is rarely added. Think about it.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/godotengine/godot/issues/5112#issuecomment-224758346,
or mute the thread
https://github.com/notifications/unsubscribe/AGVmPXROv0RM_KYxT4gE026tRpElLnVZks5qJ0vLgaJpZM4IxfNf
.

@punto- Just past the cashier! I discussed the small branch in the script depending on the platform. Do you offer support multiple scenes depending on the platform. Bad practice. I would have been to do so only in the most extreme case, if on the other does not like or what would I have different content on different platforms.

Ok but your branch would depend on a configuration value instead of a
platform. That way you can also test your code on PC by overriding the
config value. So in that case you would test for a flag
"game/use_mobile_platform" or something.
On 9 June 2016 at 04:49, ret80<EMAIL_ADDRESS>wrote:

@punto- https://github.com/punto- Just past the cashier! I discussed
the small branch in the script depending on the platform. Do you offer
support multiple scenes depending on the platform. Bad practice. I would
have been to do so only in the most extreme case, if on the other does not
like or what would I have different content on different platforms.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/godotengine/godot/issues/5112#issuecomment-224823813,
or mute the thread
https://github.com/notifications/unsubscribe/AGVmPRiV4C5nQiYeYjMjAcNm7lCGptXdks5qJ8WXgaJpZM4IxfNf
.

@punto- It makes no sense. Same as: OS.get_name(). Although the latter is even better.

closing, as this won't be implemented


Unable to change testing framework

The first question in the karma init process asks for the testing framework to use, jasmine is pre-selected. I want to use mocha.

When I press backspace, nothing happens.
When I press tab, the cursor is moved 8 spaces to the right.
When I press Ctrl+C, I get a ^C printed in the console.

This is on Windows 8.1 with the Windows Command Prompt.

I have same problem with Windows 10.

Windows cli + node.js keeps on giving..

Well, I'd love to use MINGW, but #1724. Is there anything I can do to help?

@oliversalzburg my solution for fixing this is replacing the the cli handling with either

https://github.com/SBoudrias/Inquirer.js or
https://github.com/dthree/vorpal

those are dedicated libraries that handle all the cross platform issues around cli input and should make it much more stable.

@dignifiedquire I'd love to help with this, but I can only offer to test at this time. Unless this is just a minor change, but it doesn't sound like it.


possible typo in MusyngKite/names.json

@DoomTay in your recent change, is the double underscore intentional in lead_8_bass__lead ?

That is completely intentional. The list for FluidR3 has it, and that's how it's formatted in MIDI.GM's lists


Add minimal template

Inspired by #6270, this adds a cra-template-minimal package which uses the bare minimum file structure necessary to function with React (based on the default cra-template). Optional files like tests and assets have been removed so more experienced users can have full control over what's included, while making less assumptions about tech stack.
Test Plan
npx create-react-app my-app --template file:packages/cra-template-minimal

start displays React App without errors or warnings
build succeeds and server displays React App without errors or warnings
test shows no tests, and errors in CI as per Jest's defaults

Thanks @nickmccurdy.
We have definitely talked about doing this before, and it was one of the motivating factors behind CRA templates.
I'm interested in @ianschmitz and @iansu's thoughts on this.

Yeah I know @iansu and I talked about this as well. Seems reasonable!

For what it's worth, I'm aware that his cra-template-empty template is similar, though I thought it would be a good idea to recreate one based on cra-template and have it maintained in the same repository.

Yeah we had already claimed cra-template-minimal way back before we launched the template feature. So we should be good to go there.

I'd personally like to avoid adding more templates directly to Create React App. We already have issues with maintaining the JavaScript and TypeScript templates. This is exactly why we added external templates.

Personally I’m more interested in having a template named minimal than having it be in this repo. Would you be interested in me moving this into a separate package? Or if you don’t mind giving me the name I’d be happy to publish this myself.

We actually talked about this a few weeks ago and I was just going to publish the "empty" template under that name for both JavaScript and TypeScript. The "empty" name came out of tweet at some point but I think minimal is a better name.

Is there anything I could help out with? Like extracting a repo of this PR, renaming your existing repo, etc

I've published a set of minimal templates: https://github.com/iansu/cra-minimal-templates
Thanks for your PR but I'm going to close this now.


VSTS Platform installer on a machine not connected to the internet

Description
I am trying to install the VTest platform installer to get the vstest demand satisfied on a machine in a deployment group. The machine does not have access to the internet and cant access Nuget.org and fails. Is there a way to download the package locally or the specify out internal Nuget instead?
Steps to reproduce
Use the VsTest Platform installer on a machine not connected to the internet.
Diagnostic logs
2018-06-11T11:28:22.7721611Z Feeds used:
2018-06-11T11:28:22.7722801Z   https://api.nuget.org/v3/index.json
2018-06-11T11:28:22.7723844Z
2018-06-11T11:28:22.7724549Z
2018-06-11T11:28:22.7725017Z
2018-06-11T11:28:22.7725324Z Attempting to gather dependency information for package 'Microsoft.TestPlatform.15.7.2' with respect to project 'D:\vstsagent\A1_work_temp\VsTest', targeting 'Any,Version=v0.0'
2018-06-11T11:28:22.8412685Z Unable to load the service index for source https://api.nuget.org/v3/index.json.
2018-06-11T11:28:22.8414486Z   An error occurred while sending the request.
2018-06-11T11:28:22.8415729Z   The remote name could not be resolved: 'api.nuget.org'
Environment
TFS 2018.2, Windows 2016

@MagnusTim Offline support for VsTest Platform Installer is in our back log and we will be picking it up soon. But it will probably only be available in TFS 2019.
@PBoraMSFT for more details.

@MagnusTim sadly there's no recommended way for acquiring the platform without internet connectivity :(

@MagnusTim - as Shreyas mentioned, support for "not connected" agents will be picked up for implementation soon. Until then, the only way to get VSTest capability for agents that are not connected to the internet is through a full VS install.

Closing this issue. Hope that a clear resolution was provided.

OK, tanks.
/Magnus
Från: Prachi Bora<EMAIL_ADDRESS>Skickat: den 13 juni 2018 12:13
Till: Microsoft/vstest<EMAIL_ADDRESS>Kopia: Magnus<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Ämne: Re: [Microsoft/vstest] VSTS Platform installer on a machine not connected to the internet (#1641)
@MagnusTimhttps://github.com/MagnusTim - as Shreyas mentioned, support for "not connected" agents will be picked up for implementation soon. Until then, the only way to get VSTest capability for agents that are not connected to the internet is through a full VS install.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/Microsoft/vstest/issues/1641#issuecomment-396887315, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AT772uaSj_EVG4FtvaEPLDtMZLn3Z8snks5t8OW8gaJpZM4UijmP.


How to set the torch elastic params (e.g., max_restarts)?

Hi All,
I am using torch==1.9.0 and accelerate=0.9.0
I use the following to start my training script:
accelerate launch --multi_gpu --mixed_precision=fp16 --num_machines 1 --num_processes 8 train.py
As seen, the training runs in a single machine with 8 GPUs.  and the following information is printed.
[INFO] 2022-09-04 18:43:32,155 api: Starting elastic_operator with launch configs:
  entrypoint       : run_clm.py
  min_nodes        : 1
  max_nodes        : 1
  nproc_per_node   : 8
  run_id           : none
  rdzv_backend     : static
  rdzv_endpoint    : <IP_ADDRESS>:29500
  rdzv_configs     : {'rank': 0, 'timeout': 900}
  max_restarts     : 3
  monitor_interval : 5
  log_dir          : None
  metrics_cfg      : {}

We can see that accelerate uses the torch's elastic run in the backend. The thing I want to is to set max_restarts to 0 and  rdzv_configs['timeout'] to 0. How can I do it?
Many thanks!

Hello @jcyk, the above PR should be able to resolve your requirements. Please try it out.


Perplexing COMP_POINT value on bashcompinit tab completion

I use the OMZ/bashcompinit combination in order to write my tab completion scripts.
The tab completion itself works, however, I'm writing more complex scripts, and I find the COMP_POINT value perplexing, since it doesn't match the intuitive value (the position of the cursor in the string, which is what Bash returns).
Let's suppose you have a script named /home/oooh_my_tab/scr, whose autocompletion is defined via complete -C "/path/to/myscript_completion" -o default scr.
When typing:
$ scr <tab>

Bash sends to the completion script the env vars COMP_LINE=scr  and COMP_POINT=4. The number 4 is, intuitively, the position of the cursor (and the size of the COMP_LINE string).
Zsh sends COMP_LINE=/home/oooh_my_tab/scr , which is different but perfectly fine, but sends COMP_POINT=23, which is perplexing, since the position of the cursor is 22.
When typing:
$ scr a<tab>

Bash sends COMP_LINE=scr a and COMP_POINT=5, which, again, has intuitive values.
Zsh sends COMP_LINE=/home/oooh_my_tab/scr a, and the now more perplexing COMP_POINT=25, which adds an unexpected extra unit to what was, in the previous example, an already apparently off-by-one value.
Is this a bug? Am I missing something? With such behavior, COMP_POINT breaks compatibility with Bash tab completion scripts.

I have no idea about this, you might have better support in zsh-users/zsh-completions, or in the mailing list at zsh.org/mla.
I'm closing since this is not OMZ-related per se.

Thanks and sorry. I thought bashcompinit was part of oh my zsh, but it actually is https://github.com/zsh-users/zsh (coincidentally, the last change to the file Completion/bashcompinit was a fix for COMP_POINT.

Good to know!


default background color for html is green from djangosite css

has to be changed to match the current theme

Fixed the issue


CQL being deleted when going through formatter

When running a CQL file through the formatter, CQL code is being deleted when there are errors in the file.
For example, when running this CQL file:
library test version '0.0.000'

using QDM

notvalueset "Female Administrative Sex": '2.16.840.1.113883.3.560.100.2'
valueset "TestValueset": '1.1.111'

context Patient

define "TestDefinition":
	["Encounter, Performed": "TestValueset "]

The output of the formatter is
library test version '0.0.000'

using QDM

valueset "TestValueset": '1.1.111'

context Patient

define "TestDefinition":
	["Encounter, Performed": "TestValueset "]

Another example is, when running this CQL file:
library test version '0.0.000'

using QDM

valueset "TestValueset": '1.1.111'

context Patient

define "TestDefinition":
	['"Encounter, Performed": "TestValueset "]

The output of the formatter is:
library test version '0.0.000'

using QDM

valueset "TestValueset": '1.1.111'

context Patient

define "TestDefinition":

Thoughts on this?

Yes, when there is a syntax error in the input, the parser will not be able to construct a full parse tree, and since the formatter works by visiting the parse tree, any part of the parse tree that couldn't be constructed will be missed in the output.
We discussed this today and will add functionality to the formatter to detect the case that there are syntax errors resulting in partial parse trees and revert to just returning the input source, unformatted.

This fix is included in the 1.2.11-SNAPSHOT release.


Can't use different values for the chip and listitem label

I have a use case where I want to display different values on the chip and the list item.
Would be great if there were and option to create two functions one for chip and one for listItem. Or that you could get another input to the getOptionLabel function that tells me if this is for the chip or not.

Rewrote the renderTags function instead. But would've been optimal to just have any options from the above


Added example-app

Added example app and the ability to add options.
Currently the only option available is python, but it's extensible.
Also if no options or python isn't passed, it uses zx.which() command to find python

Amazing to have the python option! Super nice to have the configurability of options.
My thoughts are

Name the external option python_path, like you put internally, just for sure clarity
I try to keep the example in the separate repo, https://github.com/semicognitive/sveltekit-modal-langchain, any thoughts on pros of in the same repo vs separate?
Make the default path be python3 instead of python - or else this is a breaking change

I agree that the option name should be python_path and happy to move back to python3
I like having an example in the repo for getting started. I left it really bare so it wasn't complex or anything. My preference is to have a really basic version and maybe links to the example in the README

Okay, fine with the example in the repo if that's preffered for some
Push the python3 change and I'm happy to merge 😃

Updated this to use python3

Merged, thank you!


Batching pixel rendering by rows and blocks

Hi All: Thank you so much for making [embedded-graphics] happen! :-)
I spent only 2 weeks studying the code and figuring out how we can make [embedded-graphics] and [st7735-lcd] render faster on PineTime smart watch, here are my findings...
Optimising PineTime’s Display Driver with Rust and Mynewt
Seems if we batch the pixels into rows and blocks, we can get significant improvement in rendering times. Yes it takes a little more RAM, but that can be managed by tuning the Row Iterator and Block Iterator.
So that's my suggestion for improving [embedded-graphics]. Thanks again, you are truly awesome! :-)

Hi! I'm glad you're enjoying this crate! I read through the article and that's a really interesting way of optimising draw calls. Another, simpler approach I've seen is here in the SSD1306 driver where the pixel data is batched into chunks of 16 bytes, but still fills a bounding box as opposed to rows/blocks.
I definitely think there's room for e-g to provide some kind of interface for optimising draw calls, but I'm not sure what that should be right now. I originally provided SizedDrawing, however it was poorly thought out and removed in 0.6.0. I'd like to see what solutions the community comes up with (including the one described in your article) before committing to a first-party API in embedded-graphics itself. Thank you for the suggestion though!

Cool! Will stay tuned for updates as I bundle my code into the PineTime firmware, which will also include the druid UI framework. Thanks! :-)


<EMAIL_ADDRESS>breaks the asset generator

Hi,
After updating @vite-pwa/astro to 0.3.1 from 0.3.0, the asset generator doesn't work properly.
It still generates the artifacts, but specifically, import { pwaAssetsHead } from 'virtual:pwa-assets/head' does not work as expected anymore, as the links array inside of it is empty so it can't be used to render any of the required links or the theme-color meta tag.
Reverting it back to 0.3.0 fixed the problem for now.

what's the error? I cannot run the examples on my local, the pnpm monorepo requires adding the following to the Astro config file:
  vite: {
    server: {
      fs: {
        // Allow serving files from hoisted root node_modules
        allow: ['../..']
      }
    },
  },


Hide Spectator Advancements in Chat

Hello!
While watching Speedrun Showdown I noticed a flaw in the strategy of the game. Specifically, I noticed that players would react and make decisions based off the advancements spectators earned. (I.E.: If a spectator earned the bastion advancement, you may assume that the other team could be close to a bastion and make a decision based off that whether or not it is true)
Personally, I feel like the spectators should be in no way involved with the gameplay therefore their advancement messages should be hidden.
This pull request adds a configuration option that will either enable or disable hiding spectator's advancement messages in the chat.
On first glance this PR might look scuffed as hell but unfortunately Spigot does not provide a Cancellable event for Advancement completion. Fortunately this code works perfectly to hide the messages.

It successfully blocks when you /give spectators items that trigger advancements.
The message won't appear if you enter a structure that gives an advancement.
It also won't send the message if you enter while in the view of the player and they enter an advancement structure (the player's advancement message shows, but not the spectator's).
Entering dimensions hides the spectator's advancement gain.

Tested everything above with multiple accounts on Spigot 1.16.1.
:)

Congrats on the first-ever pull request!
Now we're officially open source 🎉


#670 fees

This PR's checks are dependent upon the changes present in AURC PR #42.

Coverage decreased (-0.03%) to 74.249% when pulling b0d4b6965444c4377e7ce4672cdb49978b2d8817 on #670-fees into d87092870e88ac22f3a62af104159a355fd7d1d6 on master.


Investigate proxy for clients to provision a model

Currently clients need to handle resource creation themselves (e.g. "give me this huggingface resource so I can run my test." This means they have to know the implementation details of the resource they need and we can't easily manage resource allocation, queuing, turning resources off when not needed, etc.
Investigate a proxy that would handle those things for clients, so they can just hit the proxy with a unified interface, and the proxy would handle the details.

Thanks for looking into this! These are the approaches I think have been put forth so far.

Something that is purely a resource management API.

You call it to say, "I need X", it spins one up, and then make sure it gets shut down in a timely fashion.
It collects resource usage data we can pull in into something like Prometheus/Grafana
Maybe it also has primitive quota management

A straightforward HTTP proxy that handles all API requests.

By default all requests get sent through untouched, so minimal code changes.
For specific vendors, we notice activities related to resources, spinning them up and down as needed
Also for specific vendors, we manage resources sharing here. If multiple jobs are running, this shares the limited quota and/or number of GCP GPU machines in a fair fashion
Exports much richer data to Prometheus/Grafana

A full LLM API wrapper

Clients write to this API directly
It abstracts the difference between different model providers, so that the client doesn't care which vendor is running their requested model
Again, stats are collected

For me the main considerations are:

supporting operational needs (e.g., cost control, problem detection, activity monitoring)
making our coding easier
still allowing for others to run the benchmarks, both for general transparency and for model vendors doing practice tests without having to involve us

Glad to chat further, and I look forward to seeing what you come up with.

Thanks for the summary! That covers what I was thinking as well.
A few thoughts:

How often do users want to run a given model on more than one vendor? Intuitively I expect that to be uncommon, and that people are more likely to say "run x on huggingface" (i.e. x and huggingface "go together" even if x is available elsewhere). So exposing a vendor-agnostic abstraction to end users doesn't seem necessary. We could maintain a model-vendor list, so a user can just say "give me a resource to run X" and we provision the right resources and return the connection info, so the end user doesn't need to specify the model AND the runner, but even that seems like a marginal feature to me.

Given the high cost of running models and vendor availability limitations (e.g. our own vllm instance isn't guaranteed to run if the region is out of gpu capacity), it feels important to be able to spin resources up only when needed and down as soon as we're done, so investing on instrumenting usage metrics feels useful.

Letting users provide their own credentials so they can use their own vendor accounts feels worth it. Using MLC credentials for MLC runs is fine, but it won't just be MLC people forever.

I don't know if proxying the API calls to the model transparently brings a lot of value. It exposes us to IP liability as a user's data goes through us, which means we can intercept it, log it, etc. It also incurs runtime costs and complexity to manage its own scaling, fault tolerance, access keys, security, etc. And if some unscrupulous person DDoSes huggingface through the proxy, now we look like the bad guys. OTOH it would allow us to queue up requests and manage backend/vendor resource utilization; but it can't commit to be 100% available when vendors aren't; we can throttle or buffer requests and handle transient issues on the vendor side, but there will be times when something fails hard and we return an error to the client. This means the client needs to be able to handle errors from our proxy. Since they also need to handle errors if they hit the vendor directly, and we can't entirely shield them from handling any errors at all, then I don't think the benefit outweighs the cost/complexity/liability of a full pass-through proxy.

Quota and scheduling management: do the major vendors have APIs we can query to get information about current availability and usage?

Quota and scheduling management (2): in addition to sending data to Grafana, it may be worth exposing usage metrics via an endpoint that a human or a program can query. A user could do their own quota management until we provide that: query the usage endpoint to see what's running, and then decide whether to request a resource now or try again later.

Thanks for the summary! That covers what I was thinking as well.
A few thoughts:

How often do users want to run a given model on more than one vendor?

I think we have four use cases, in rough priority order.

Our team running official benchmarks and related practice/development tests
SUT owners running practice tests against their own SUTs

via modellab and our servers
offline, via their own servers

Workstream participants (and us) doing exploratory work with our tools
Third parties who are trying things out, thinking about registering with us and becoming SUT owners, or want to validate our benchmarks

For the first case, I think switching will be rare. For the second, I think it'll be custom SUTs with the connection details controlled by configuration. For the 3rd, I expect some swapping. E..g, moving around as model availability shifts, or comparing behavior for the same model across multiple servers. And the last doesn't matter, except to the extent we need to build with offline use cases in mind.

Given the high cost of running models and vendor availability limitations (e.g. our own vllm instance isn't guaranteed to run if the region is out of gpu capacity), it feels important to be able to spin resources up only when needed and down as soon as we're done, so investing on instrumenting usage metrics feels useful.

Yes, absolutely. The evaluator set they're talking about could plausibly touch $1000/day, so we're going to have to be careful here. And since we're dealing with multiple services, it will be even easier for something to keep running somewhere.

Letting users provide their own credentials so they can use their own vendor accounts feels worth it. Using MLC credentials for MLC runs is fine, but it won't just be MLC people forever.

Yeah, I think that will be especially important for the vendor case. We're going to want to give them an interface where they can configure SUTs at will, including endpoints and auth keys. And something automated that tracks when those break, so they keep the connections working.

It exposes us to IP liability as a user's data goes through us, which means we can intercept it, log it, etc. It also incurs runtime costs and complexity to manage its own scaling, fault tolerance, access keys, security, etc.

I was thinking of the proxy as only being for internal use, so I don't think the IP/security concerns apply much. To me an advantage of a proxy approach is that we can let SUTs be relatively vendor-specific, working normally for runs we don't control. But when we are doing runs, we specify a proxy to which we can gradually add accounting and quota management.

Quota and scheduling management: do the major vendors have APIs we can query to get information about current availability and usage?

I don't know if there are APIs; I expect we'd instead look at individual responses. Together, like some other services I've seen, gives back quota information in response headers. But we'd also want to look for HTTP responses that indicate being over quota and throttle back connections to the provider.

Quota and scheduling management (2): in addition to sending data to Grafana, it may be worth exposing usage metrics via an endpoint that a human or a program can query. A user could do their own quota management until we provide that: query the usage endpoint to see what's running, and then decide whether to request a resource now or try again later.

Interesting idea! The Together quotas reset very frequently, something like every second. But for longer-lived queries, that could be great. That said, I think just throttling requests could get us most of the benefit without any client work, so I'd want to add that feature later once we see demonstrated need.

Ah, and here's a just-announced look at Together's rate limits: https://docs.together.ai/docs/rate-limits

Our rate limits are currently measured in requests per second (RPS) and tokens per second (TPS) for each model type. If you exceed any of the rate limits you will get a 429 error.


use github actions for pre-commit checks

What does this pull request do?

Related issues
Closes #ISSUE

:grey_exclamation: Build Aborted

There is a new build on-going so the previous on-going builds have been aborted.

the below badges are clickable and redirect to their specific view in the CI or DOCS

Expand to view the summary

Build stats

Start Time: 2022-10-04T11:28:22.633+0000

Duration: 15 min 23 sec

Test stats :test_tube:

Test
Results

Failed
0

Passed
4316

Skipped
3241

Total
7557

Steps errors
Expand to view the steps failures

Restore files previously stashed

Took 0 min 0 sec . View more details here
Description: coverage-python-3.10-sanic-newest

Archive the artifacts

Took 0 min 0 sec . View more details here
Description: [2022-10-04T11:43:43.313Z] Archiving artifacts
org.jenkinsci.plugins.workflow.steps.FlowInterrupted

Error signal

Took 0 min 0 sec . View more details here
Description: Python-python-3.10-starlette-newest tests failed : org.jenkinsci.plugins.workflow.steps.FlowInterruptedException

:robot: GitHub comments
Expand to view the GitHub comments

To re-run your PR in the CI, just comment with:

/test : Re-trigger the build.

/test linters : Run the Python linters only.

/test full : Run the full matrix of tests.

/test benchmark : Run the APM Agent Python benchmarks tests.

run elasticsearch-ci/docs : Re-trigger the docs validation. (use unformatted text in the comment!)

:globe_with_meridians: Coverage report

Name
Metrics % (covered/total)
Diff

Packages
100.0% (67/67)
:green_heart:

Files
100.0% (227/227)
:green_heart:

Classes
100.0% (227/227)
:green_heart:

Lines
91.3% (18070/19792)
:+1:

Conditionals
74.607% (2800/3753)
:+1:


README Typos

Hi, nice plugin idea. I found a couple of little typos and grammer mistakes in your README.md. I can fix them I a PR or you do:

Notes -> notes (line 3)
Movies -> movies (line 3)
disclaimer -> Disclaimer (line 11)
plugin -> plugin, (line 13)
Key -> key (line 13)
here, -> here (line 13)
as is -> as (line 13)
plugins -> plugin (line 13)
movie -> movie, (line 15)
Using with -> Using (line 17)
css -> CSS (line 17)
snippet -> snippet, (line 19)
querry -> query (line 32)

Thanks, glad you like it. I have fixed the typos you mentioned.


JSON structure

Is there a way to hook into the JSON structure when posting to the server (receiving as well)?  IE I use the JSON API standard.

+1

Would you accept a PR on this?

Would you accept a PR on this?

This would be helpful for integration with knock for Rails.

Hi @michaelseanbecker, any PR are welcome. I don't think @jpadilla would mind as long as the JSON API serialization is something that can be opt-in by the users of the addon.
@jpadilla thought?

Isn't this already possible with getAuthenticateData and getResponseData?

@jpadilla can you please explain to me how to modify JSON payload sent to the server without write a custom authenticator in authenticators folder?
For example this is what I have now, with default:
{"identification":"myUsername","password":"myPassword"}
Now I need something like this:
{"auth": {"identification":"myUsername","password":"myPassword"}}
Maybe in controllers/login.js in authenticate() action?
I'm afraid of write buggy code myself in a custom authenticators/auth.js file.
I thought something like this:
authenticate(credentials) {
    const myJSON = JSON.stringify({
      auth: {
        email: credentials.email,
        password: credentials.password
      }
    });

but where and when?


Release

Hi,
Is it possible that you add a version tag, this way I can safely use your fork of the library from packagist?
I think you can tag with version 3.0.0 to match the api version.
Thanks

https://github.com/spotonlive/php-google-my-business/releases/tag/3.0.0

Thanks :+1:


Conformer generation failed for some of the benchmarkset molecules

For some of the ring-cage host guest systems, the conformer generation step (possibly from charge assignment) fails:
from simtk.openmm.app import PDBFile
from openforcefield.topology import Molecule, Topology
from openforcefield.typing.engines.smirnoff import ForceField

sdf_mol = Molecule('/home/yzhao/Code/benchmarksets/input_files/cb7-set1/sdf/guest-17.sdf')
force_field = ForceField('smirnoff99Frosst.offxml')

# Parametrize the ligand molecule by creating a Topology object from it
ligand_system = force_field.create_openmm_system(sdf_mol.to_topology())

Unable to load toolkit <openforcefield.utils.toolkits.RDKitToolkitWrapper object at 0x7f52a02690b8>.
Unable to load toolkit <openforcefield.utils.toolkits.AmberToolsToolkitWrapper object at 0x7f52a02690b8>.
Warning: OEMMFFParams::PrepMol() : unable to type atom 0 N
Warning: OEMMFFParams::PrepMol() : unable to type atom 1 N
Warning: OEMMFFParams::PrepMol() : unable to type atom 12 H
Warning: OEMMFFParams::PrepMol() : unable to type atom 13 H
Warning: OEMMFFParams::PrepMol() : unable to type atom 14 H
Warning: OEMMFFParams::PrepMol() : unable to type atom 15 H
Warning: OEMMFFParams::PrepMol() : unable to type atom 16 H
Warning: OEMMFFParams::PrepMol() : unable to type atom 17 H
Warning: force field setup failed due to missing parameters for molecule MOL
Traceback (most recent call last):
  File "system_setup.py", line 9, in <module>
    ligand_system = force_field.create_openmm_system(sdf_mol.to_topology())
  File "/home/yzhao/Code/timemachine/openforcefield/openforcefield/typing/engines/smirnoff/forcefield.py", line 863, in create_openmm_system
    parameter_handler.create_force(system, topology, **kwargs)
  File "/home/yzhao/Code/timemachine/openforcefield/openforcefield/typing/engines/smirnoff/parameters.py", line 2027, in create_force
    temp_mol.generate_conformers(num_conformers=10, toolkit_registry=toolkit_registry)
  File "/home/yzhao/Code/timemachine/openforcefield/openforcefield/topology/molecule.py", line 1947, in generate_conformers
    clear_existing=clear_existing)
  File "/home/yzhao/Code/timemachine/openforcefield/openforcefield/utils/toolkits.py", line 3027, in call
    return method(*args, **kwargs)
  File "/home/yzhao/Code/timemachine/openforcefield/openforcefield/utils/toolkits.py", line 1130, in generate_conformers
    raise Exception("OpenEye Omega conformer generation failed")
Exception: OpenEye Omega conformer generation failed

Benchmark sets can be found here: https://github.com/MobleyLab/benchmarksets/tree/master/input_files

@SimonBoothroyd is out today, but he had similar problems with conformer generation for ring systems in the property calculator.
Simon eventually resorted to using externally-generated charges for rings using the charge_from_molecules kwarg to create_openmm_system.

This is an OpenEye toolkit MMFF type assignment issue. We should be able to easily construct a test case where you simply try to generate conformers with Omega and report that to their bug tracker.

If I already have a starting geometry, why can't I just use that directly? This is a ring cage so there isn't that much conformational flexibility anyways.

If I already have a starting geometry, why can't I just use that directly to do the AM1 calculations? This is a ring cage so there isn't that much conformational flexibility anyways.

We generate geometries automatically in an effort to make the force field less dependent on the specific choice of geometry you use. If we did not do this, you could get vastly different behavior simply due to which conformer you picked.
In particular, we use the canonical ELF10 AM1-BCC procedure, which is intended to prevent large geometry-dependent charge fluctuations.
You are supposed to be able to specify your own partial charges by providing a charged molecule as an optional argument to ForceField.create_openmm_system(), but I'm not seeing that argument implemented.
@j-wags : Did this user-specified charge assignment option accidentally fall off the roadmap somehow? This is a critical feature.

I'm not seeing that argument implemented

I think it is implemented as Jeff mentioned, but it is absorbed in the kwargs that is forwarded to one of the handlers, and we probably forgot to include it explicitly in the docstring.

I think it is implemented as Jeff mentioned, but it is absorbed in the kwargs that is forwarded to one of the handlers, and we probably forgot to include it explicitly in the docstring.

Great! Can you create an issue to fix the docs?

Can you create an issue to fix the docs?

I've already fixed it in #322 .

OEChem also has difficulties generating conformers for very basic charged molecules, like butylamine (https://github.com/MobleyLab/benchmarksets/blob/master/input_files/cd-set1/sdf/guest-1.sdf)

Looks like using mol2 instead of sdf files works

I'm finally getting back to this. The SDF in question can be loaded and processed, but only if the atomic formal charges are updated. @guoweiqi already figured this out, and opened an Issue + PR at https://github.com/MobleyLab/benchmarksets/issues/68 and https://github.com/MobleyLab/benchmarksets/pull/69.
I think it's reasonable to say that this issue is closed, and the root cause is that the underlying data may be incorrect. Discussion on that should continue on the relevant PRs in benchmarksets


[Fix] Theme selector incorrectly focus on first item after selection

Description
This PR removes the initial focusing state when opening up the theme selector popup.
Issues Resolved
Fixes #1163
Demo
Before
https://github.com/opensearch-project/oui/assets/65143821/c15db0c1-fd19-48d9-a458-cab4642d459d
After
https://github.com/opensearch-project/oui/assets/65143821/6326c9d9-db29-45a0-87ef-fbcc2703195d
Misc
Also fix a typo in guide_version_selector.tsx.
Check List

[ ] New functionality includes testing.
[ ] New functionality has been documented.
[ ] All tests pass

[ ] yarn lint
[ ] yarn test-unit

[ ] Update CHANGELOG.md
[x] Commits are signed per the DCO using --signoff

Hi @kgcreative, could you please assist in determining the best UX for this scenario? Thank you!

This is a challenging one. When opening via mouse, we shouldn't have a focus selector, however, per WCAG accessibility standards, there is an expectation that when you open a control via keyboard interaction, that we move keyboard focus to the first item in the opened control. As such, if we can't differentiate between keyboard or mouse input, the existing behavior is the more correct one.

Thanks for sharing your insights, @kgcreative! Based on what you've described, would you suggest implementing a mechanism where if the element is opened via keyboard (for example, pressing the Tab key), we highlight the selector, but if not, we skip the highlighting?

Thanks for sharing your insights, @kgcreative! Based on what you've described, would you suggest implementing a mechanism where if the element is opened via keyboard (for example, pressing the Tab key), we highlight the selector, but if not, we skip the highlighting?

This may be a larger issue. I would explore using :focus-visible instead of :focus, since that does use browser heuristics to determine whether an element should have a visible focus indicator or not.
cc: @BSFishy + @joshuarrrr

I feel like this is the wrong way to go about this. I think that changing the default value for focus would be breaking for UX. Additionally, OuiContextMenuPanelDescriptor offers initialFocusedItemIndex, which I think would be a better way to implement this. Just set that value to whatever value is currently selected. That way we don't even need to differentiate if the panel was opened by keyboard or mouse

Yeah, this needs to be a component level change, not changing the fous behavior in the implementation

@willie-hung are you interested in taking this forward or would you rather have it closed?

Hi @AMoo-Miki, I think it should be closed for now. However, if you decide to make changes at the component level, I would be happy to join the discussion and assist with the development.

Closing as recommended.


Feature: Add plausibility checks for input YAML

Add the following plausibility checks:

Every top-level element (G,S,C,J,A,Sn) is referenced by at least one other element. (There are no unreferenced elements apart from the top-level goal.)
All references exist.
No duplicate names exist (keys must be unique in YAML, but inContext and supportedBy could have duplicates too).
Do all IDs start with a known prefix. (This check is only necessary for the keys, since all references are checked for existence).

serde_yaml and rust_yaml do not check for duplicates in keys. Add this to known issues.


Feature/join game frontend

Related Issue

#6 Create a basic website template
#26 Create an HTML page to start a new game
#46 Set up server for static resources
#52 Create a method in the contract to get all open games

How Has This Been Tested?

By hand, creating and joining game works. Triggered events are handeled correctly
Types of changes

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

Checklist contributor:

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

Checklist reviewer:

[ ] The code has been peer-reviewed by @YourNameHere

When you play a game against yourself (two browser tabs, two different addresses from your personal pool), the labels "You" and "Opponent" are wrong in one of the windows. But looking at the code I see why that is and have no good idea how to fix it. (It assumes that the player with one of the local addresses is you.) The browser would need to store with which address it joined that game, but that is quite some overhead. Maybe just live with it for now.

Final question: Did we decide against writing tests for the frontend code?

Apart from these 3 comments, all looks good! :+1:

game against yourself: I don't think we need to address this, because

Why should I play against myself and waste ether?
Regarding the off-chain-chess, the game structure will change.

Added "#46 Set up server for static resources" to the related issues

Cool, let's get this merged.


There is an extra scrollbar in the 'Open Table by Name' dialog when typing '' to search emulator tables

Storage Explorer Version: 1.24.1
Build Number: 20220510.2
Branch: hotfix/1.24.1
Platform/OS: Windows 10
Architecture: ia32
Azurite Version: 3.16.0
How Found: From running test cases
Regression From: Not a regression
Steps to Reproduce

Install and run Azurite 3.16.0 -> Launch Storage Explorer.
Expand 'Local & Attached -> Storage Accounts -> (Emulator - Default Ports) (Key)' -> Right click 'Tables' node.
Click 'Open Table by Name...' -> Type '\' in the Table name field -> Click 'Search'.
There is an error message.
Check there is no extra scrollbar.

Expected Experience
There is no extra scrollbar.
Actual Experience
There is a scrollbar.

Additional Context

Related bug: 5600.
There is no error message for emulator blob containers and queues.
There is no error message for normal tables.

Likely another issue in the emulator, but I wouldn't consider this worth fixing, since '' is not a valid character for a table name and would yield zero results regardless.

Verified this issue on the build main/20220613.1. Fixed.


Refactor & clarify API

[ ] Keyword args for detect & classify to simplify their use independently

A bit more background. The code in cellfinder-core was originally part of cellfinder, and wasn't intended as a public API. Now it is used in cellfinder and cellfinder-napari. It can also be used in other workflows, and so the API needs to be exposed in a much more sensible manner.


[CosmosDB] analyticalStorageTtl is an invalid property in service version 2022-05-15

analyticalStorageTtl is defined in swagger, but when sending a request to the service with the property set, an error is thrown:
Azure.RequestFailedException : Message: {"code":"BadRequest","message":"Message: {\"Errors\":[\"The input content is invalid - 'analyticalStorageTtl' is not a valid property in the current payload.\"]}

We ran into as well , was this ever brought up anywhere ?


functionality documentation

Need some core documentation. Maybe a sphinx site? Something generated from docstrings?

We can follow the simple examples set by pyglider and erddapy for making sphinx docs

maybe use this gh action https://github.com/sphinx-notes/pages


Find list of Committee URL's or links to their pages

Try to integrate this with house and senate json short codes(posted in slack channel). Create a data dictionary to map.

I found these on the ProPublica API. I grabbed these, will sort thru the JSON and create a seed file with the data.


Always return data

Is there a way to always return the data array instead of the traditional formatted status and data request like so:
"data": [
    {
      "courseCredits": 4,
      "name": "SEIS45",
      "courseCode": "SESI452",
      "description": "Great class",
      "_id": "4a3nYgGSKTvZtbc5u"
    }
  ]

Unfortunately, the only way to alter the response in the collection routes at the moment is to override the individual collection endpoints, which is a terrible solution. The good news is that this is very high on my priority list, and will be resolved in the first update after the Iron Router dependency is removed. What you need is a hook that runs immediately before the response is returned in collection endpoints, allowing you to modify the default response. I'll also allow the default endpoints to be completely overridden, in case you want to do something completely different but want to keep things DRY. This is related to #42, #12 , and #18. I'll keep you posted on any progress.

Thanks for the quick response

Hi @kahmali, @wbashir
I have more less same question. I want to receive only data without any status and data prefixes:
{
	"_id": "QjCjP2DwqHqZJq54f"
	"name": "item name",
	"createdBy": "BeNTtaupkj5JfpJcH",
	"createdAt": "2016-06-18T10:00:04.113Z",
	"isPrivate": false,
	"isDeleted": false
}

instead of
{
    "status": "success",
    "data": [
        {
            "_id": "QjCjP2DwqHqZJq54f"
            "name": "item name",
            "createdBy": "BeNTtaupkj5JfpJcH",
            "createdAt": "2016-06-18T10:00:04.113Z",
            "isPrivate": false,
            "isDeleted": false
        }
    ]
}

Maybe somebody have solution?


Issue with finding plugin on Homebridge running on Docker (Synology NAS)

Dear all,
I installed the Roomba2 plugin on Homebridge (1.4.1) running via Docker on a Synology NAS. I have trouble finding the folder of the plugin in the first step of the install. It seems that the folder is not in the same place as expected (no such file or directory).
Any ideas as to how to proceed?
BW,
Jeff

Never mind, I got it the work. It seems that if you run Terminal in Homebridge, it is already in the root folder. So a simple: cd node_modules/homebridge-roomba2 did the trick.
Maybe this is useful for others that are running Homebridge on Docker.


ui: Update packages with security issues as well as minor versions

This has updated the needed packages to fix the security issues found by
npm audit fix and next did an update using npn update
Signed-off-by: Otavio Salvador<EMAIL_ADDRESS>

Can you fix this warning? [Vuetify] [UPGRADE] 'v-content' is deprecated, use 'v-main' instead.
@noreng-jg


Owner dropdown should list me first

I.e. put the creator of a PR first:

@bpasero do you want the creator first or you first?

@alexr00 I would mimic what GH does when I open the dropdown, I think it has a way to suggest users that fits best, I would probably put myself and creator of PR as suggestions. Maybe also reviewer.


Calculate G(r) Tab - Save S(Q) to file

We have the Save G(r) button to output the G(r) but lately, we have been adjust S(Q) and creating a new S(Q) workspace. We would like to shift and scale S(Q) and then be able to save it back out.
Probably near the Clear S(Q) button, can we add a Save S(Q) button?

@wdzhou has a pull request with this fix implemented. Just need to fix other issues in request and we can merge.

So, now this is not working. I get the following error:
For *.dat files:
Traceback (most recent call last):
  File "build/scripts-2.7/addie", line 1207, in do_save_sq
    raise RuntimeError('File type {0} cannot be recognized.'.format(file_ext))
RuntimeError: File type  cannot be recognized.

For *.xye files
Traceback (most recent call last):
  File "build/scripts-2.7/addie", line 1207, in do_save_sq
    raise RuntimeError('File type {0} cannot be recognized.'.format(file_ext))
RuntimeError: File type .xye cannot be recognized.
``

I am not sure if this is intended but the Save S(Q) button seems to be disconnected now.

Save S(Q) does not work and has no output.


Maternity paternity calculator content changes

Changes from https://github.com/alphagov/smart-answers/pull/2818

Sorry @cbaines  these commits were rolled into this PR https://github.com/alphagov/smart-answers/pull/3022


Google tag call each time and says multiple initialize when route is change.

This question is available on Nuxt community (#c9479)

I have current ga script in plugins ga.js
export default ({ app }) => {   /*    ** Only run on client-side and only in production mode    */   // if (process.env.NODE_ENV !== "production") return;   /*    ** Include Google Analytics Script    */   (function(i, s, o, g, r, a, m) {     i["GoogleAnalyticsObject"] = r;     (i[r] =       i[r] ||       function() {         (i[r].q = i[r].q || []).push(arguments);       }),       (i[r].l = 1 * new Date());     (a = s.createElement(o)), (m = s.getElementsByTagName(o)[0]);     a.async = 1;     a.src = g;     m.parentNode.insertBefore(a, m);   })(     window,     document,     "script",     "https://www.google-analytics.com/analytics.js",     "ga"   );   /*    ** Set the current page    */   ga("create", "UA-XXXXXX-X", "auto");   /*    ** Every time the route changes (fired on initialization too)    */   app.router.afterEach((to, from) => {     /*      ** We tell Google Analytics to add a pageview      */     ga("set", "page", to.fullPath);     ga("send", "pageview");   }); };
and google tag script in gt.js plugins
export default ({ app }) => {   /*    ** Only run on client-side and only in production mode    */   //if (process.env.NODE_ENV !== "production") return;   /*    ** Include Google Analytics Script    */   (function(w, d, s, l, i) {     w[l] = w[l] || [];     w[l].push({       "gtm.start": new Date().getTime(),       event: "gtm.js"     });     var f = d.getElementsByTagName(s)[0],       j = d.createElement(s),       dl = l != "dataLayer" ? "&l=" + l : "";     j.async = true;     j.src = "https://www.googletagmanager.com/gtm.js?id=" + i + dl;     f.parentNode.insertBefore(j, f);   })(window, document, "script", "dataLayer", "GTM-XXXXX"); };
which works fine in first load both track down by Tag Assistance(by Google). But when i go to another route every things work fine but Tag Assistance(by Google) show multiple google tag. Each time i change route it push one more gtag to Tag Assistance(by Google). But analytics works fine and also track current page. Is this an issue?? And if anyone knows how to implement google analytics enchanted eCommerce to nuxt please help me!!!


Possible error of relativistic gan

in https://github.com/eriklindernoren/PyTorch-GAN/blob/master/implementations/relativistic_gan/relativistic_gan.py
        if opt.rel_avg_gan:
            g_loss = adversarial_loss(fake_pred - real_pred.mean(0, keepdim=True), valid)
        else:
            g_loss = adversarial_loss(fake_pred - real_pred, valid)

        # Loss measures generator's ability to fool the discriminator
        g_loss = adversarial_loss(discriminator(gen_imgs), valid)

        g_loss.backward()
        optimizer_G.step()

Is this expected? Does it look like g_loss is getting overwritten?

Up, error is still there.
I believe you can just discard
g_loss = adversarial_loss(discriminator(gen_imgs), valid)

Why dont author merge this ? I think this is a serious error.


AutoLogin if user is logged in from previous session

Just added a simple override onStart function in LoginRegisterActivity which checks if there is any current user logged in from previous session and navigates to ShopActivity if it is true.
Here's a video recording of app's functioning . . .
https://github.com/Manishshakya6614/Shop_Now/assets/113130559/130e8fb2-36dd-4802-b0ae-a34dc028a795

Hi Yashvant,
Thank you for your contribution! The auto-login functionality is a great addition and enhances the user experience significantly. This I forgot to add. I appreciate you taking the time to improve the project.

Hi Yashvant,
Thank you for your contribution! The auto-login functionality is a great addition and enhances the user experience significantly. I appreciate you taking the time to improve the project.

Thanks @Manishshakya6614 , please consider merging this pull request if it does the job 😀


Di fixes

[DI] When checking class inits for uses of self, ignore strong_retain.
This specific issue came up due to my SILGen changes for copy_value,
destroy_value. Specifically, we used to emit the diagnostic:
super.init isn't called on all paths before returning from initializer.

After my change, we began to emit:
'self' used before super.init call

Since the code was:
init() {}

there is clearly no use, so the new error is incorrect or at minimum
misleading. When I investigated why the change in diagnostic happened, it was
because as a result of my SILGen change, the order of the strong_retain and
return in the use-list swapped in the following code:
%1 = load %0 : $*MultipleInitDerived
strong_retain %1 : $MultipleInitDerived
return %1 : $MultipleInitDerived

Before my change, return was visited first, causing the correct diagnostic to be
emitted. After my change, the strong_retain is visited first causing the second
diagnostic to be emitted. Since we only emit one source error for each
SILLocation, in the 2nd case, the error associated with the `return is not
emitted.
We should not be considering strong_retain to be a use that propagates
liveness. The reason why is that a strong_retain will always be paired with
some local use that consumes it, whether that is an apply, closure, or a store
into memory. Since we will always have a consuming operation for the retain and
the consuming operation actually is able to be something visible at the source
level to the user (versus the retain), it makes more sense to just ignore
strong_retain.
In general though, we should try to be more careful about how we swallow errors
in DI, especially in the light of use-list twiddling. To help with this, I have
added a flag '-definite-init-visit-debuginfo-locs' that causes DI to consider
SIL Location debug info locs when deciding whether or not to swallow
errors. This means one can actually test the error swalling code paths by adding
sil locations to the end of SILInstructions, i.e.:
strong_retain %0 : $Builtin.NativeObject, loc "path":col:line

Any instruction with the same "path", col, line will from DI's perspective be
treated as generating the same "error" set.
rdar://28851920

@swift-ci Please smoke test and merge

@swift-ci Please smoke test and merge

Looks like I was wrong, turns out our diagnostics do not accept DebugLoc. I am not sure why this test worked before. I am going to close this since the work is much smaller and just use the main copy_value, destroy_value PR for this work that way it is atleast commited with a test.


Add AWS SNS Receiver

Looks good otherwise, although I can't speak much for the templates.

Looks good besides last comment. Could you squash your fixup commits?

Any updates on this one ?


Add a force drain option to purge all pod inside a node

When update bosh deployment to rezize disk or update stemcell we need to be able to make a full drain automatically.
we can be inspired by cfcr bosh release : https://github.com/cloudfoundry-incubator/kubo-release/blob/6f9046f29de4b9e3f089502df6887d5d058b4249/jobs/kubelet/templates/bin/drain.erb#L75
and check for attached PV / disk
https://github.com/cloudfoundry-incubator/kubo-release/blob/6f9046f29de4b9e3f089502df6887d5d058b4249/jobs/kubelet/templates/bin/drain.erb#L97

@jhunt is there a good raison to set root for containerd on /var/vcap/store ? and not on /var/vcap/data
https://github.com/jhunt/k8s-boshrelease/blob/4adef9fa622739fb819614650bfeb9a858ca2d8d/jobs/runtime-runc/templates/bin/containerd#L54
It seems to look the persistent disk and forbide the resize of persistent disk

Probably just an oversight.
Out of curiosity, what are you storing on your persistent volume?  Most of my kubelet nodes lack persistent volumes, since they have no data to persist.

I use the persistant volume to store data of PV made by openebs (openebs.io). With this method I'm not dependant of any cloudprovider. It create dynamic LocalPV or Cstor storageclass.
You can also use project like longhorn from rancher they also use a directory to store data. Instead of put it in /var/lib/longhorn you put it on /var/vcap/store/longhorn


Delete graphite backup files older than 2 days

find * -mtime 2 only finds files which are exactly 2 days old, this means if
the process fails the backup for that day will never be deleted. Changing it to
find * -mtime +2 will find all files older than 2 days avoiding this buildup.

Good spot!


main: use in-memory store for integration

Since the integration is now long-running, it no longer needs to persist metrics for delta calculations or other tasks.
This removes the need to mount emptyDir volumes on /tmp for the integration container.
Open questions:

Is InMemoryStore production ready, or meant for testing purposes only?
Will it behave properly in the long term, or will it leak resources?

The change looks good but the open questions scares me, I am not 100% sure that wasn't merely a feature needed for tests

I am not 100% sure that wasn't merely a feature needed for tests

I think that part should be fine, as the on-disk storage actually uses the inMemoryStore. I'll cross that out from the PR.


Messaging Endpoint for Deployment slots

Are there any plans to allow for setting different messaging endpoints for deployment slots. Currently there doesnt seem to be a way to specify a different messaging endpoints.

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

ID: b527247b-38f8-0bc0-6eda-af6382d8f1af
Version Independent ID: e1c371cc-9183-1138-69e3-77a096a9f964
Content: Configure bot settings - Bot Service
Content Source: articles/bot-service-manage-settings.md
Service: bot-service
Product: bot-framework
GitHub Login: @v-royhar
Microsoft Alias: v-royhar

I'll enquire.

Is there any workaround to use slots with bots? So far, the only thing that worked for me was to deploy two different bot Web Apps and two different Bot Services.

I haven't had a chance to try this out, and we'll be moving away from using the .bot file in a few release cycles, but the following might work for you.
Review the information for how-to deploy. Then, update the .bot file to include an additional endpoint for each additional slot, such as:
    {
        "type": "endpoint",
        "appId": "<app-id>",
        "appPassword": "<app-password>",
        "endpoint": "<hosted-endpoint-url>",
        "name": "staging",
        "id": "<unique-id>"
    },

If you're working with a bot already on azure, you could edit the .bot file in place, or download and republish the bot.

I ended up just deploying a bot registeration app. Its essentially the bot connector without the bot. If  you arent using premium channels no cost. Set the messaging endpoint to the slot url.
@JohnathenFingold adding endpoints won't do anything. Only one endpoint is actually listening for messages. Adding extra endpoints is just so you have setting and keys if you want to switch them

Is there any update on this ticket?  I'm exploring whether I can use slots or if I have to create a new bot app with different messaging endpoints.

@ImKator. I don't have any additional information yet. For additional help, you can post questions at Stack Overflow using the botframework tag.

We are planning a devops article that should address this question, but I don't have a schedule yet. Until then, you could ask our community support at Stack Overflow using the botframework tag.

@JonathanFingold  do you have an updated timeline for that Dev Ops article?

Unfortunately no, other than not scheduled for the 4.5 release.

@JonathanFingold  no worries, since bot channel registration connectors are free for non premium chat protocols its not really much of a problem to just create a connector and point it at the slot


Update README for various templates

README needs to be updated for the following templates:

[ ] AWS EBS
[ ] AWS EFS
[ ] Azure Block
[ ] NetApp SAN
[ ] NetApp NAS

Please add to the list if any more templates need to be updated.

NetApp SAN:
Need to update SC related names/keywords in this section: https://github.com/IBM/ibm-satellite-storage/tree/develop/config-templates/netapp/netapp-ontap-san/21.04#verifying-your-netapp-ontap-san-driver-storage-configuration-is-assigned-to-your-clusters
Similarly for NetApp NAS: https://github.com/IBM/ibm-satellite-storage/tree/develop/config-templates/netapp/netapp-ontap-nas/21.04#verifying-your-netapp-ontap-nas-driver-storage-configuration-is-assigned-to-your-clusters
Update for both versions 20.07 & 21.04