Update image name and remove build

Pull Request Test Coverage Report for Build 64a660e13b7c36391f69963f821c7cbeca77406b-PR-914

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

Totals

Change from base Build 36c605c36fdb45a07e279e51b2da36e0dce0479e:
0.0%

Covered Lines:
1054

Relevant Lines:
1561

💛  - Coveralls


updated domain name and sizing

This PR is:

[ ] Adding a new domain
[ ] Updating existing domain size
[x] Changing domain name
[ ] Removing existing domain from list
[ ] Website code changes (512kb.club site)
[ ] Other not listed

[ ] I used the uncompressed size of the site
[ ] I have included a link to the GTMetrix report
[x] The domain is in the correct alphabetical order
[ ] This site is not a ultra lightweight site
[ ] The following information is filled identical to the data file

I confirm that I have read the FAQ section, particularly the two red items around minimal pages and inappropriate content and I attest that this site is neither of these things.

[x] Check to confirm

- domain:
  url:
  size:
  last_checked:

GTMetrix Report:  https://gtmetrix.com/reports/savajava-one.vercel.app/gcB2Z3uY/

PR Closed. Domain needs changing as app is broken upon testing


Allow custom amplitude instance for use with CommonJS packagers

Currently in our codebase we have to do the following:
window.amplitude = require('amplitude-js')
module.exports = require('redux-beacon/targets/amplitude')

Which we then import and use:
import amplitude from 'amplitude-js'
import { createMiddleware } from 'redux-beacon'
import { Amplitude } from './redux-beacon-amplitude'

…
amplitude.getInstance().init(…)
createMiddleware(eventsMap, Amplitude, { offlineStorage })

It would be nice if redux-beacon-amplitude either required amplitude-js directly (peer dependency?) or could be passed in somehow (e.g. createMiddleware(eventsMap, amplitudeMiddleware(amplitude), { offlineStorage }).

This is a great suggestion @marcello3d!
I think the latter solution (passing it in) would be the simplest most maintainable solution. Something like:
import amplitude from 'amplitude-js';
import { createMiddleware } from 'redux-beacon'
import { Amplitude } from 'redux-beacon/amplitude'

// initialise amplitude
amplitude.getInstance().init(API_KEY)

// create analytics middleware
const options = { instance: amplitude.getInstance('INSTANCE_NAME') };
const middleware = createMiddleware(eventsMap, Amplitude(options), { offlineStorage });

If an amplitude instance is provided the target would use that to track events, otherwise it will look for window.amplitude and use the default instance (i.e. window.amplitude.getInstance()).
Thoughts?
@bertrandk, I'd like to hear your thoughts on this as well.

Passing it in would definitely be the way to go. I can probably make the change sometime this evening if nobody else gets to it first.

This is now the default, documented, way to use the Amplitude target!
https://rangle.gitbook.io/redux-beacon/targets/amplitude


Loop dont' work when not focused

slides stop cycling when i select another program or just clicked outside browser
when i focus my window again it start cycling again
is there a way to change/fix that?

I had same problem.
I customized lightslider.js file and solved problem.
lightslider.js -> set Annotation line 882 : clearInterval(interval);


What suggest to  deploy a 'service pod' on non-docker environment?

We deploy a 'service pod' with a service instance and the sidecar envoy, and other surrounding stuff.
ServicePod => |Service|Envoy |
It's very easy to deploy in k8s or other docker situation, though, not so easy in non-docker machine or vm. We have develop a tool to patch everything required in the service home directory. However,  It's to too weird, I think.
Any good solutions?

@anticpp this is a bit out of scope for what the Envoy project can provide. It really comes down to the specifics of your deployment (e.g., puppet vs. salt, how you do deploys, etc.). cc @louiscryan

Are you using anything for deployment / orchestration? Chef, Puppet etc ?

No, we are not yet using any deployment. It will be in our future plan.

The point is, we have implemented an initial version to our project, we don't want to introduce a 'heavy' deployment in our system at this moment. So, we are looking forward a relative 'light' solution.

Closing this for now as we have no plans to take on this type of ops work in the primary envoy project.


Complete Configu CLI

Closes #503
Tasks:

[x] Test command
[x] Run command
[ ] Backup feature
[ ] Export command
[ ] Find command
[ ] Login command

@RichardAkman

dont add TestCommand to the sdk it is a cli only feature
skip FindCommand for now as Im planing a renovate of it


Update high-availability.md

When administrator add 2nd CP into the cluster with method of stacking masters, the order of next two steps may cause errors:

KUBECONFIG=/etc/kubernetes/admin.conf kubectl exec -n kube-system etcd-${CP0_HOSTNAME} -- etcdctl --ca-file /etc/kubernetes/pki/etcd/ca.crt --cert-file /etc/kubernetes/pki/etcd/peer.crt --key-file /etc/kubernetes/pki/etcd/peer.key --endpoints=https://${CP0_IP}:2379 member add ${CP1_HOSTNAME} https://${CP1_IP}:2380
kubeadm alpha phase etcd local --config kubeadm-config.yaml

Under this order, after the command 'etcdctl member add' executed on etcd pod of CP0,  etcd on CP1 is still not runing, however etcd on CP0 need to communicate with peer etcd on CP1. This would make etcd cluster failed, sequentially the api-server of k8s cannot response. Now creating a second etcd pod is impossible. To avoid this happening, we should reverse the order of 1,2.

^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
Please delete this note before submitting the pull request.
For 1.13 Features: set Milestone to 1.13 and Base Branch to dev-1.13
Help editing and submitting pull requests:
https://kubernetes.io/docs/contribute/start/#improve-existing-content.
Help choosing which branch to use:
https://kubernetes.io/docs/contribute/start#choose-which-git-branch-to-use.
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

/assign
@chuckha @detiber can one of you take a look? I think the logic of this PR might be a little muddled (maybe not quite understanding the kubeadm-config.yaml file?) but I'm not going to call it on my own. Thanks! (If you give it a 👍 I'll poke the contributor about the CLA).

leaving this one for @detiber

/approve

/approve cancel

@Hchnr can you please rebase this PR against master? We don't typically take PRs against older branches, although in this case we can take up separately the question of whether to do so. But we need it against master first. Thanks!

Excuse me, how to rebase this PR against master? Should I close this PR and create a new one?

I rebase it here. #10961 @Bradamant3


[Bug] duplicate messages

when setting chatHistory:{disabled:true} , the first 2 messages are always a duplicate,
i solved it by simply adding the following in ChatBotContainer.tsx line 108  (an empty message if history is disabled )
but i don't think that's a good way to fix it , so if you can find the bug it would be amazing , best of luck!
if (botOptions.chatHistory?.disabled) {
localStorage.removeItem(botOptions.chatHistory?.storageKey as string);
//added to prevent bug
const messageContent = {
content: <></>,
type: "object",
isUser: false,
timestamp: null,
};
setMessages([messageContent]);

Hey @theodor961, thank you for reporting this! I should be able to take a look at this later tonight. In the meantime, if it's alright with you, would you be able to share screenshots showing the duplicate messages along with your bot configurations?

my pleasure,
i used  the default flow and default bot options
below you can find the screenshoted result

@theodor961 Can I check if you're seeing this only in development environment? The issue is not reproducible on the playground. Any chance this is relevant for you?

Hey @theodor961! Just to share some observations:

Issue not observed if you fork and run this repo directly
Issue not observed if you import this library into your own project, then build and serve
Issue observed if you import this library into your own project then run in dev environment

Based on the above, I would assume StrictMode (refer to link in above comment) in react 18 is what is causing the duplicate message seen. There are some suggested workarounds online so I leave that exploration up to you 😝
Thank you for raising this up and I'll update the documentation to include this piece of information in case others come across the same issue in future.
That said, if you feel I am mistaken and you are still facing the issue even without StrictMode, kindly let me know and I'll try my best to look again 😀

okay, perfect!
thanks for checking it out, you have an amazing product.


fix: add custom button label ans add remove button (Issue #783)

Description:

Add custom button label for the upload files from device dialog;
Add possibility to remove custom logo;

Issues:

Issue #783

UI changes

Checklist:

[x] the pull request name complies with Conventional Commits
[x] the pull request name starts with fix(<scope>):, feat(<scope>):, feature(<scope>):, chore(<scope>):, hotfix(<scope>): or e2e(<scope>):. If contains breaking changes then the pull request name must start with fix(<scope>)!:, feat(<scope>)!:, feature(<scope>)!:, chore(<scope>)!:, hotfix(<scope>)!: or e2e(<scope>)!: where <scope> is name of affected project: chat, chat-e2e, overlay, shared, sandbox-overlay, etc.
[x] the pull request name ends with (Issue #<TICKET_ID>) (comma-separated list of issues)
[x] I confirm that do not share any confidential information like API keys or any other secrets and private URLs

/deploy-review


[ExpandCollapse] Clickable type not set to 'button'

Description
Originally reported by @marckuo
When nesting an ExpandCollapse component within a form, clicking on a Panel heading will submit the parent form.
Reproduction Steps

Render a form with a nested ExpandCollapse component that has at least 1 Panel
Click on a Panel heading
The form submits

Reproduction Percentage (if applicable)

10/10 times

Meta

[x] TDS version: v1.0.2
[x] Willing to develop solution: yes, @marckuo volunteers
[x] Has workaround: No
[x] High impact: Yes, this is blocking a certain design pattern

Proposal
Set type='button' to the <button> element rendered by the Clickable component within TDS.
Screenshots
Note: each section of the form is grouped into Panels.

This was resolved in releases:

https://github.com/telusdigital/tds/releases/tag/v0.34.4
https://github.com/telusdigital/tds/releases/tag/v1.0.4


Add file API tests

Adds file server API test for memory backend
Adds file server API test for localfs backend

in addition, these new tests unearthed a couple of issues (resolved in this PR)

cannot write file when storage directory is not created. This is due to a failed attempt to read the current quota (localfs)

fixes #36

localfs always fails to write due to being over quota. The was due to use of request.body.size (should be request.body.length) making the newSize calculation equate to NaN

NOTE: s3 backed tests are not enabled in this 1st iteration

@hardillb
Could you look at the failing tests here please?
getting Error: EACCES: permission denied, mkdir '/var/root'
what test root should we use for GH actions pls?

Committed my suggestion to the path to get the tests to run again

Committed my suggestion to the path to get the tests to run again

oops, we both did it :D
pushed updates to address your concerns.


Updates for fixing head tilt + motion in smaller kitchen

Motivation and Context
Minor changes to configuration to make robot "look better" during some motions. Before, the robot would look away when actually executing placement actions. This looks weird in videos
Tweaking ground plane level (where we count obstacles)
Changelog

How Has This Been Tested

Types of changes

[ ] Docs change / refactoring / dependency upgrade
[x] 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:

[x] I have made changes to existing documentation where needed.
[ ] I have added tests that show that the PR is functional.
[ ] I ran tests/hw_manual_test.py on hardware and observed that robot behavior looks normal.

Weird that it improves place! but great!


training results and training details

Your work is excellent，But I have some questions about the training results and training details.

There are two models in models.py, ModelBinary and BodelContract. Does that mean that the former is a vanilla model? However, GitHub does not give a description of ModelBinary.
Do you have annotations in the code when you train CodeSearchNet alone? If the query is annotations, shouldn't the code be trained with annotations?

Hello,I want to ask why the results of each training and reasoning are inconsistent, but the random seed has been fixed in the code


chore: updated WalletConnect dependencies

Updated WalletConnect dependencies to latest (2.12.2) as the 2.8.0 that was used is almost a year old and missing on tons of updates and improvements
Removed legacy client 1.8.0 as it isn't used anywhere in the wallet

Congratz on the wallet launch!

thanks we will implement this as part of our 1.3 release and you will see the update when we push a new open release


Wrong block comment highlight

/* and */ symbols can be used to comment out an entire section, but only if the symbols appear at the beginning of a line (…)
https://autohotkey.com/docs/Language.htm#comments
It does not correspond with defined syntax:
https://github.com/ahkscript/SublimeAutoHotkey/blob/dc20ea3b615d902fc43045c1073353af5f9e0edf/AutoHotkey.tmLanguage#L34-L41
supposedly yielding:

Correct highlight should be

Ran into this in VSCode port ( stef-levesque/vscode-autohotkey#9 ) and after fixing noticed it is derived from here; it seems most (other derived) syntax definitions for AHK exhibits the same bug.

I recently ran into this as well, just a little differently. Sublime Text has a default key binding ctrl+shift+/ for toggling block comment. Currently with this package, it produces comment like this:
/*#!r::
	Tester.ReloadThisScript()
Return*/

I feel it would be nice if the */ can be automatically put in a new line:
/*#!r::
	Tester.ReloadThisScript()
Return
*/


Nextcloud subscription error

Hey @haydenull , thank you so much for this amazing and useful plugin !
I'm on 1.9.4 and using a subscription link from my personal Nextcloud instance. If I open this link on the web, it prompts for the download of an ics file, just like a Google Calendar link works.
The error on the notification window in Logseq is the following: Parse Calendar Personal data error TypeError: Cannot read properties of undefined (reading 'value')
Inspecting the DEV console, I see these errors:

FYI, I have recurring events in this calendar, if it matters.

My guess is that there was an error reading the event summary or dtstart.
Can you give me a subscription address or ics file where private information is hidden?

Sent you a PM. Thank you for your time !

The URL you gave me seems to display correctly.

Ok very weird, it is also displaying correctly for me with this test
calendar I created. However, my other to Personal and Work calendars are
not giving the same error. I'll send you my work calendar so you can
test please.
Thank you !
On 17/03/2022 11:25, Hayden Chen wrote:

The URL you gave me seems to display correctly.
image
https://user-images.githubusercontent.com/17695989/158798881-1de70c17-ca0a-4383-b92d-456eeec6e2b4.png
—
Reply to this email directly, view it on GitHub
https://github.com/haydenull/logseq-plugin-agenda/issues/20#issuecomment-1070811821,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOMC7F3FQV2CI6ZDJH2S5DLVAMJCZANCNFSM5Q4M6UDQ.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID:
@.***>

1.9.5 version solves this problem.
I have already send the wrong calendar event to your mailbox.


chore: update wrappers

I updated the wrappers because my two documentation PRs failed for ths unrelated reason.

@krzema12 Is it possible to make the job Check if wrappers up to date / check (pull_request) Failing after 1m   not a requirement for merging a PR?
It's a bad developer experience IMHO for someone new to the project to put together a contribution and have the CI fail on you for a reason unrelated to what you are actually doing.
Maybe the PR could add a comment in the PR or something

@jmfayard this problem is tracked as #208. IMO it's not a big deal if the PR doesn't deal with action wrappers. I can bypass this failed check and merge anyway. If, on the other hand, the PR is about changing something in the wrappers, this check is very helpful.
Thanks for this change, I'll review the types once I'm on my laptop.

Will split it in a single commit per action.


Version control for notes

Hi guys,
Fantastic repo. Is there a way to version control the notes (SVN, GIT)?

Hi Noah, no, the notes are stored only in the DB and withohut any versioning.


Fix that the SA1119 code fix can produce invalid syntax.

If the opening parenthesis does not have leading trivia in some cases the code fix would merge the leading node (token) and the syntax node (token) in the parenthesis creating invalid syntax.
Fixes #449

Weird that the test failed. It works on my machine.

Did you catch that the failing test isn't the one you added?

We need a way to check the fix all provider here too. Any ideas?

Superseded by #453


Give a short tutorial on how to install required tools

I suggest to have a short tutorial on how to install programming environments:
Python 2 or 3, Scratch etc.
Another solution is to point the readers to an already existent tutorial :)

Great idea @ner0x652, we won't add a complete tutorial but if we can point them in the right direction that would be great.


Raise runtime error when overwriting an in-memory pointer causes a memory leak.

Must ensure that the pointer isn't alive in a virtual register.

This issue was moved to seec-team/seec#20


Update inaccurate description

Update property descriptions in the following files：
AzureGlobalProperties.java#L207
HttpClientConfigurationProperties.java#L49

API change check for com.azure.spring:spring-cloud-azure-autoconfigure
API changes are not detected in this pull request for com.azure.spring:spring-cloud-azure-autoconfigure

@hui1110, could you help sync this change to this branch https://github.com/Azure/azure-sdk-for-java/tree/release/spring-cloud-azure/4.1.0, to make sure the changes are in the 4.1.0 release?


GitHub is HTTPS by default

This reduces unnecessary redirection from http://... to https://... when someone visits this gem's homepage via https://rubygems.org/gems/ruby-saml or some tools or APIs that use the gem's metadata.

Coverage remained the same at 97.884% when pulling 1e06b047ba173c24511807d7ecd3c0e2f240ef86 on amatsuda:https into 63f43a7f9b830dfd690457a78a78fac48359944f on onelogin:master.


Handle variables can't be initialised

@harshvinay752 @RWitak @sebasj13
I'm having trouble with initialising the handle values. It seems like the values are overwritten when the slider is created. Any help would be appreciated. Thanks!
    min_val = DoubleVar(value = 0.1)
    max_val = DoubleVar(value = 0.9)
    print(f"min_val 1: {min_val.get()}")
    print(f"max_val 1: {max_val.get()}")

    min_val.set(0.2)
    max_val.set(0.8)
    print(f"min_val 2: {min_val.get()}")
    print(f"max_val 2: {max_val.get()}")

    rsl = RangeSliderH(frame, [max_val, min_val], padX=23, digit_precision='.2f')
    print(f"min_val 3: {min_val.get()}")
    print(f"max_val 3: {max_val.get()}")

    rsl.pack()

   #> min_val 1: 0.1
   #> max_val 1: 0.9
   #> min_val 2: 0.2
   #> max_val 2: 0.8
   #> min_val 3: 1.0
   #> max_val 3: 0.0

You might wanna try passing the min before the max. Not at the computer right now, so can't test it.
(I would actually expect you to get an Error "Left Handle value is more than the Right Handle Value, which is not possible." with your input.)

@RWitak Thanks for getting back to me. I've tried switching the order of the values, but unfortunately still the same result.
rsl = RangeSliderH(frame, [min_val, max_val], padX=23, digit_precision='.2f')

You would indeed expect a Left Handle value is more than the Right Handle Value, which is not possible. error, but that is not the case. Neither with [min_val, max_val] nor with [max_val, min_val].

I created a reprex with the example code from the README.md. In both cases there is no value initialisation and no error saying that the second value is larger than the first one - which makes me suspect that somewhere in the creation of a RangeSlider the values are reset. I've browsed through the code, but could not find anything.
from RangeSlider.RangeSlider import RangeSliderH, RangeSliderV
from tkinter.ttk import *
import tkinter as tk

root = tk.Tk()
hVar1 = tk.DoubleVar(value = 0.2)
hVar2 = tk.DoubleVar(value = 0.8)
print(f"hVar1 before RangeSlider: {hVar1.get()}")
print(f"hVar2 before RangeSlider: {hVar2.get()}")
rs1 = RangeSliderH(root, [hVar1, hVar2], padX=15) # first min, then max. No error.
rs2 = RangeSliderH(root, [hVar2, hVar1], padX=15) # first max, then min. No error.
print(f"hVar1 after RangeSlider:  {hVar1.get()}")
print(f"hVar2 after RangeSlider:  {hVar2.get()}")
rs1.pack()
rs2.pack()

root.mainloop()

Output:
hVar1 before RangeSlider: 0.2
hVar2 before RangeSlider: 0.8
hVar1 after RangeSlider:  1.0
hVar2 after RangeSlider:  0.0

Hey @PetervanLunteren, please allow some time to check the code. I will go through and update you with the same. Thank you for pointing the error.

@harshvinay752 Works like a charm! Thanks :) Will this update be published to PyPi too?

@PetervanLunteren thank you for the feedback. I will update the same to PyPi too, was waiting for resolution to the issue you faced.

I have updated the same to PyPi too at RangeSlider:PyPi. It can installed using:
Windows:
pip install RangeSlider --upgrade
Linux:
pip3 install RangeSlider --upgrade


Updated install guide and rename ubuntu.md to debian.md

Install guide updated to allow install to work.
Change from using pip to building within Poetry environment and pulling from github.

It allows the streamdeck to work fully as intended.
Would just require a systemd to start on boot up if desired.
If you try and install through pip system wide it will not allow you on
debian, and prompt you to do it in a virtual environment.
It's a working install guide that allows it to be installed and fully
functional, unlike the current/previous guide.
On Sat, 29 Jul 2023, 10:28 4s3ti, @.***> wrote:

@.**** requested changes on this pull request.
This does not sound accurate,
I don't have a Debian box to test now, but the steps are almost universal
on the installation part.
when you install an aplication you want it to be available system-wide,
therefore the steps you represent don't sound exactly correct.
Poetry is only creating a virtual env and making sure you have the deps
for developing.
doing poetry run, runs the application only there does not install it on
the system.
Poetry should not be needed to install the application or to run it.
Installation from source on the system should be something like:
sudo python -m installer dist/*wheel
the command above should add the streamdeck executable in /usr/bin
or something similar.
—
Reply to this email directly, view it on GitHub
https://github.com/streamdeck-linux-gui/streamdeck-linux-gui/pull/19#pullrequestreview-1553243315,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AXUCZ4RJ7RDIAKUWQOBWIJTXSTJUVANCNFSM6AAAAAA237OTK4
.
You are receiving this because you authored the thread.Message ID:
<streamdeck-linux-gui/streamdeck-linux-gui/pull/19/review/1553243315@
github.com>

In that case use pipx instead of pip.
I am not saying it doesn't work.
what I say is that it does not sound a good way to install applications, just adds more complexity when it should be the other way around.
I am adding the proper steps to install from source, you can piggy back on them later if you want.

actually I believe your steps are really well suited for the contributing guide, there is for sure where they shine because you can test the application and your changes easily without having to build or install!

pip is easy to use. Why should we switch?

There is no reason ,and the suggested solution is not a correct way to install packages.
It should be either via pip, pipx or via distro package manager, or if one wants from source using the appropriate steps.
the steps suggested are better suited for running the application when developing and one wants to test and verify while working on it.
@Narcodepsy  you are welcome to adjust the contributing guide with those steps if you want.

The pip isn't updated yet for the new fork, so pulls the old fork which
also has issues.
Also the pip machine level install on debian gives errors/advisories to do
in a virtual environment.
No reason why when first point is rectified can't look at changing again
but, having a working guide people can use has more value than a one that
worked and might work again but currently doesn't.
On Sat, 29 Jul 2023, 14:22 Tim Bannister, @.***> wrote:

@.**** commented on this pull request.
pip is easy to use. Why should we switch?
—
Reply to this email directly, view it on GitHub
https://github.com/streamdeck-linux-gui/streamdeck-linux-gui/pull/19#pullrequestreview-1553277623,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AXUCZ4QPU4RUJLNCRHIRI3LXSUFCZANCNFSM6AAAAAA237OTK4
.
You are receiving this because you authored the thread.Message ID:
<streamdeck-linux-gui/streamdeck-linux-gui/pull/19/review/1553277623@
github.com>

There is a pip package: https://pypi.org/project/streamdeck-linux-gui/
As I mentioned, the Virtual env regarding pip is solved with pipx not with poetry which is a development and dependency management tool, not a package manager.
A working guide doesn't necessarily mean that is the correct guide.


[hi] Add content\hi\docs\setup\production-environment\turnkey-solutions.md

part of ongoing effort https://github.com/kubernetes/website/issues/29353

/hold
Right now this includes lots of commits that we don't yet want to include in the Hindi branch this way.

I am making a new PR ASAP

New PR https://github.com/kubernetes/website/pull/29966


Middleware triggered multiple times

The Middleware functions seems to be being triggered multiple times and hence causing performance issues
to reproduce, create a middle ware and feed it to the GraphQLServer constructor

in the middle-ware definition, go ahead and console.log debug messages
and you will see it being triggered multiple times per request

same here

@Mercurial i just noticed that the middleware functions are getting triggered not only for the fields but scalar fields as well, that's wierd. I don't know if that's a bug or a feature (oh that sounded like a joke).

same problem here

async function demoMiddleware(resolve, root, args, context, info) {
	console.count('abcd')
	let data = resolve(root, args, context, info)
	return data
}

I am also facing the same issue right now
Weird behaviour is when I call my query from graphql-playground it is calling middleware 11 times and when I call my query from React-Apollo client it gets called 5 times.

In case someone encounters this problem in the future, the reason for this is described in prisma-labs/graphql-middleware#33
The middleware is called for every single field resolver, so to fire it only once you need to explicitly assign it to Query and Mutation resolvers only:
const server = new GraphQLServer({
  // ...
  middlewares: [{
    Query: authMiddleware,
    Mutation: authMiddleware,
  }]
})

@ialexi-bl  Thank you This is working fine.

@ialexi-bl That worked for me, thank you!


README: Add project screenshot

Duplicated of https://github.com/XollabOS/talktometechnology/pull/24


SC Chance figure not updating on rod switch

Describe the bug
Fishing sea creature catch chance no longer updated (in the fishing section) when selecting different rods.
It is updated at the head of the page, however not in that section, these two share the same data stat tag only the former is updated.

A recent commit https://github.com/SkyCryptWebsite/SkyCrypt/commit/8335d0f7dda251e902b9e9c8884060478c91ebf1#diff-cc124a01577ed15df124f49ac790d4d785e8c06f5e1379405d7e65e45f365dcdL637 breaks the fishing SC chance figure. The old anime library used the targets: ... and would allow the updating/animation of all matching elements, the new implementation using flashForUpdate only targets the first instance element.innerHTML (the one at the top of the page).
Profile/Location
https://sky.shiiyu.moe/stats/Pugzy/Pear skills section then fishing.
Expected behavior
The figure should be updated in both locations see https://sky.lea.moe/stats/Pugzy/Pear for example.

I'll fix this


Subscription fetching request fails on production server fails with the same credentials

The following code is running on a Windows 2012 Server.
dotnet core version: 2.1.4
class Program {
        private static string account = "PLAY-ACCOUNT";
        private static string privateKey = "MY-PRIVATE-KEY";
        private static ServiceAccountCredential _credential;
        private static AndroidPublisherService _googleService;
        private static string BundleID = "com.bundle.id";
        private static string token = "token-purchase";
        static void Main(string[] args)
        {
            TestGoogle().Wait();
            Console.ReadLine();
        }

        public static async Task TestGoogle()
        {
            _credential = new ServiceAccountCredential
                (
                    new ServiceAccountCredential.Initializer(account)
                    {
                        Scopes = new[] { AndroidPublisherService.Scope.Androidpublisher },
                    }.FromPrivateKey(privateKey.Replace("\\n", "\n"))
                );

            _googleService = new AndroidPublisherService
                (
                    new BaseClientService.Initializer
                    {
                        HttpClientInitializer = _credential,
                        ApplicationName = "APP-NAME"
                    }
                );
            var request = _googleService.Purchases.Subscriptions.Get(BundleID, "subid", token);
			var purchaseState = await request.ExecuteAsync();
            Console.WriteLine(purchaseState.OrderId);
        }
}

The code is a replication of the actual program. The exception I am getting is
Unhandled Exception: System.AggregateException: One or more errors occurred. (Error:"invalid_grant", Description:"Invalid JWT: Token must be a short-lived token (60 minutes) and in a reasonable timeframe. Check your iat and exp values and use a clock with skew to account for clock differences between systems.", Uri:"") ---> Google.Apis.Auth.OAuth2.Responses.TokenResponseException: Error:"invalid_grant", Description:"Invalid JWT: Token must be a short-lived token (60 minutes) and in a reasonable timeframe. Check your iat and exp values and use a clock with skew to account for clock differences between systems.", Uri:""
   at Google.Apis.Auth.OAuth2.Requests.TokenRequestExtenstions.<ExecuteAsync>d__0.MoveNext()
--- End of stack trace from previous location where exception was thrown ---

But the code is running on the development machine. I am confused as of why this could be an issue in just the production server.

The error message talks about clock skew being a potential problem, and that would definitely fit with it only being a problem on a specific machine. Have you checked the clock on the production server?

@jskeet the clock was set to 5 minutes ahead - Thanks for the heads up. Closing the issue.


Possibility of async methods?

Has there been any investigation into whether we could implement asynchronous methods in this library, similar to bcrypt? I'm hesitant to run this as a password hashing library on a production server, knowing it will block the event loop for 100-300ms every time a user attempts to login.

This would be great but the lib is somewhat big and right now I have no time to do a full rewrite to support async methods. If you want to start down that path it would be awesome.

Got it!


Domain activation is Failed - [BRIEF DESCRIPTION]

App Id: dtbrp5xjorsgi
Region: ap-southeast-2
Step: Domain activation
Status: Failed

Note: Do not include information that is sensitive in nature such as your domain name, company etc.

Issue/question
I am trying to add a domain with AWS Amplify but I am getting this error message:
Error message: One or more of the CNAMEs you provided are already associated with a different resource.
Error message
Error message: One or more of the CNAMEs you provided are already associated with a different resource.
Note: Be sure to check the message for sensitive information.
Additional information
In the past, I have used another AWS account with AWS Amplify to use this domain. At that time, the deployment worked. This time, with my current account, it won't let me. I have since deleted all resources and even deleted the old account. The domain I am trying to add is educats.com.au

Hi @uuykay
These guides are helpful for common custom domain issue. If those guides can not solve your issues, please let us know.
https://docs.aws.amazon.com/amplify/latest/userguide/custom-domains.html#cnamealreadyexistsexception-error
https://aws.amazon.com/premiumsupport/knowledge-center/resolve-cnamealreadyexists-error/

Hello @Athena96 I used these two guides when I have first tried to solve this problem. I don't have premium support, so I cannot message Amazon, but I have created a TXT record that matches my CNAME. Are you able to help me?

Hey... check if you have the app created in another region. I have created the app in region A and another in Region B without me knowing. It passes all SSL etc until the last steps and keep telling me cname has already been used.

Hey @joshqq I only have 1 Amplify App on this account. In the past I used the same domain on a different AWS account using Amplify, but I have deleted that account and all resouces (as far as I know) from that account 2 days ago. I want to be using the current account. Could you please check on it? https://www.educats.com.au/

This is what I see...

I have to go to all the "region" and see if I have used amplify to create an app.
I can't tell you exactly how to resolve it completely but the following:

Ensuring the DNS in Domain Registered and Zone are the same.
Make sure that I have no CloudFront instances that I did not realise I have.
Remove SES / DKIM set up just in case.

I empathise with what you go through... let's hope you find out the root cause...

Hey @joshqq thanks so much for your help. I had to go to work and couldn't reply earlier. You were actually right with your first comment about a possible instance of the app being created previously in a different region. I am not totally sure how this happened - I may have set that one up in the past and forgot about it - but it was in the Ohio region.
I discovered this when I went to my S3 buckets and found one bucket in the Ohio region with the same name as my app. Upon changing regions with Amplify I discovered and deleted that instance of the app and was able to finally link my Sydney region to the domain.
I'm very thankful!


Audit color tokens and remove un-needed ones

We need to do an audit of our color tokens to:

remove un-needed tokens
semantically name other tokens better

We have some ink colours we probably don't need and also some new semantic colours like component-background-hover that don't map to a colour value.
Sometimes in our code we also need to set raw colours like brown.1 so I think it would be best if we can add semantic colour names to describe those better.

For some tasks like re-working the action popout to ignore the theme, working with colours got a bit tricky.
Sample here.
We should have:

only the colours we actually use
semantic colour tokens that can over-ride theme settings e.g. so button-bg doesn't change and we don't need to set it inline as 'darkModeBrown.1' for example

Adding info from Slack to here ...and I think I'll move this issue to the mono repo.
Proposal:

Delete our old ink palette. Then, rename our current brown  palette to ink
Introduce a new color for hover effects, called ink-2-tranparent which maps to component-background-hover. Why that name? Because it resembles the old brown-2 (on a white background) which is used for component background hovers


Render HTML in report

Is it possible to render HTML in reports?
As an example, I currently have a script which collates information for each of my 200 samples based on the results/output of a few different rules and outputs a HTML file which has a table of all of this information. It would be nice if I could have this HTML file rendered in the snakemake report, rather than as an attachment file. I know there is the boilerplate RST files, but they don't allow for performing the kind of logic I need, i.e. reading and performing computation of the contents of multiple files.
If there is a better way of achieving this, I am open to suggestions.

Yes, this is possible. Just mark the HTML file of choice with the report-flag.
Here's the documentation, but yeah... HTML is not clearly mentionedd in there... But for this worked :)
Report-Documentation


Freeze on match customization

Addresses #120 by introducing a new customization that freezes specimens based on a set of criteria.
Usage
The FreezeOnMatchCustomization uses the IRequestSpecification abstraction as a way to match the incoming requests with the frozen specimen. For example:
var fixture = new Fixture();
var freeze = new FreezeOnMatchCustomization(
    typeof(string),
    new PropertySpecification(typeof(string), "Foo"));
fixture.Customize(freeze);

This will assign the same string value to all properties of type string named "Foo".
Multiple matching criteria can also be combined using the OrRequestSpecification. Here's an example:
var fixture = new Fixture();
var freeze = new FreezeOnMatchCustomization(
    typeof(string),
    new OrRequestSpecification(
        new PropertySpecification(typeof(string), "Foo")
        new ParameterSpecification(typeof(string), "bar")));
fixture.Customize(freeze);

When no matching criteria are specified, the FreezeOnMatchCustomization behaves like the current FreezingCustomization, that is it will only match by exact type. More concretely:
var freeze = new FreezeOnMatchCustomization(typeof(string));

is the same as saying:
var freeze = new FreezeOnMatchCustomization(typeof(string), new ExactTypeSpecification(typeof(string));

Follow-ups
The are a few things I deliberately left out until after I gather your feedback:

Since the FreezeOnMatchCustomization effectively supersedes the existing FreezingCustomization, I propose we obsolete it and rename the FreezeOnMatchCustomization to simply be FreezeCustomization for consistency.
I'll add the XML documentation once the feature is finalized.

I look forward to your comments.

All of the existing feedback has been addressed.

It's looking good, but when I attempt to run build-release.sh, it gives me 290 build errors. It looks like XML documentation erros.

It's looking good, but when I attempt to run build-release.sh, it gives me 290 build errors. It looks like XML documentation erros.

Absolutely correct. I had saved that for last but never actually got around to doing it. I'm on it :+1:

I added XML documentation and addressed a few remaining code analysis issues.

Both e6c29a7 and cafaac3 are looking good to me!

Thank you for your contribution! It's now live as AutoFixture 3.23.0.

Nice :+1:


Fix the invalid heed usage

We were incorrectly using LMDB by keeping references into the database while modifying it, it was mostly due to heed not specifying anything about this behavior (blame me). I made sure in this PR that we always keep and owned version of the references we were previously using.
This PR partially fixes #261, we need to bump heed to v0.12 when it is available as it marks the del/put_current, and append methods as unsafe and explains why.
I would like to thank the whole Rust community for their help, especially:

@Pointerbender 🎸 who tracked down the exact point where the memory write was altered,
@Diggsey 💯 who found, in the LMDB documentation, the sentence that we forgot about,
and @RalfJung 🥇 who track and kill undefined behavior in Rust programs in general.

The k variable comes from the grenad iterator, we compare it with the key variable which, this one, comes from the LMDB database. So, no need to change anything here, am I wrong?

I obviously have no right to impose anything here, but I'd just suggest that when you make subtle changes like this, you leave a comment in the source code that at least briefly explains that this was a deliberate choice, and possibly links to an issue with more details. That reduces the risk of someone, in 2 years, going "ah I can eliminate some copies here" and re-introducing the bug. ;)

Thank you @RalfJung,
I am changing the heed functions to be unsafe, make a PR to this repo using the new version, and will add little comments above the functions in this PR. It will be just above an unsafe block, I hope this comment will be visible enough in 2 years 😄

FWIW, you don't need to make the heed functions unsafe - you just need to make sure that all references returned from the database have a lifetime tied to the database, and that all database-modifying functions require a mutable borrow of the database in order to operate.

Thank you @Diggsey, I just opened an issue about your proposal, not sure if it will work but will try it out. I didn't publish the new v0.12 version of heed on crates.io so I can change anything.
Anyway, we will merge this PR for now as it fixes the bug and will use the potentially safe new version of heed if we update it!

@Kerollmops just a heads up that without implementing my proposal, you might have to worry about race conditions, since one thread might be reading a key from the DB while another thread tries to write to the DB. (I'm not sure, maybe everything is already forced to be single-threaded)

@Diggsey, Yeah LMDB already ensures that there can be only one mutable transaction at a time. Thank you anyway!

I think that have to move around a ref mut to the db is too much constraint, since we won't be able to clone it and use references to it anymore. This is redundant to lmdb's locking mechanism.
As soon as we get GATs, which so not be too long now, we should be able to create a Current type which represent a view on the current iterator entry, with a consuming self, and accessors for the value and the data, whose lifetime are bound to the lifetime of the Current object.


Error with fresh install of v3.5.0

Trying to set up a fresh install of koel v3.5.0 on Ubuntu forge server. When I run 'php artisan koel:init' I get the following error:
PHP Fatal error:  Uncaught ReflectionException: Class sentry does not exist
I've tried updating composer but still get the same results. Sentry is listed in the config/app.php file and in the config.json file so I'm really not sure what to do from here.
Thanks in advance!

Can't reproduce this. What happens if you manually composer require sentry/sentry-laravel?

It says it's using version ^0.5.0.
Here is the error and stack trace:

PHP Fatal error:  Uncaught ReflectionException: Class sentry does not exist in /home/forge/jamshark.com/vendor/laravel/framework/src/Illuminate/Container/Container.php:749
Stack trace:
#0 /home/forge/jamshark.com/vendor/laravel/framework/src/Illuminate/Container/Container.php(749): ReflectionClass->__construct('sentry')
#1 /home/forge/jamshark.com/vendor/laravel/framework/src/Illuminate/Container/Container.php(644): Illuminate\Container\Container->build('sentry', Array)
#2 /home/forge/jamshark.com/vendor/laravel/framework/src/Illuminate/Foundation/Application.php(709): Illuminate\Container\Container->make('sentry', Array)
#3 /home/forge/jamshark.com/vendor/laravel/framework/src/Illuminate/Foundation/helpers.php(106): Illuminate\Foundation\Application->make('sentry', Array)
#4 /home/forge/jamshark.com/app/Exceptions/Handler.php(38): app('sentry')
#5 /home/forge/jamshark.com/vendor/laravel/framework/src/Illuminate/Foundation/Console/Kernel.php(320): App\Exceptions\Handler->report(Object(Dotenv\Exception\InvalidFileException))
#6 in /home/forge/jamshark.com/vendor/laravel/framework/src/Illuminate/Container/Container.php on line 749

It then repeats that stack trace and after that gives these errors:

Script php artisan clear-compiled handling the pre-update-cmd event returned with error code 255

Installation failed, reverting ./composer.json to its original content.

Got this too.
I fixed it by
git checkout composer.json
git checkout composer.lock

Then I've edited composer.lock to remove all section concerning pusher (I'd installed it previously, it was a leftover):
git diff master composer.lock
diff --git a/composer.lock b/composer.lock
index 0592e13..42a6b84 100644
--- a/composer.lock
+++ b/composer.lock
@@ -1507,50 +1507,6 @@
             "time": "2017-01-15T17:54:13+00:00"
         },
         {
-            "name": "pusher/pusher-php-server",
-            "version": "2.6.3",
-            "source": {
-                "type": "git",
-                "url": "https://github.com/pusher/pusher-http-php.git",
-                "reference": "7101edc482d1b1de027abadd7a80a0d00222ef08"
-            },
-            "dist": {
-                "type": "zip",
-                "url": "https://api.github.com/repos/pusher/pusher-http-php/zipball/7101edc482d1b1de027abadd7a80a0d00222ef08",
-                "reference": "7101edc482d1b1de027abadd7a80a0d00222ef08",
-                "shasum": ""
-            },
-            "require": {
-                "ext-curl": "*",
-                "php": ">=5.2"
-            },
-            "require-dev": {
-                "phpunit/phpunit": "~4"
-            },
-            "type": "library",
-            "autoload": {
-                "classmap": [
-                    "lib/"
-                ]
-            },
-            "notification-url": "https://packagist.org/downloads/",
-            "license": [
-                "MIT"
-            ],
-            "description": "Library for interacting with the Pusher REST API",
-            "homepage": "https://github.com/pusher/pusher-php-server",
-            "keywords": [
-                "events",
-                "php-pusher-server",
-                "publish",
-                "pusher",
-                "realtime",
-                "rest",
-                "trigger"
-            ],
-            "time": "2016-12-22T10:46:28+00:00"
-        },
-        {
             "name": "ramsey/uuid",
             "version": "3.5.2",
             "source": {

Then did a composer update
Then did a composer install

I just deleted the repository from my server and reinstalled it. No issues the second time.


Need to use returned params when the tab is closed.

I have a login system on a web and im trying to use custom_tabs to login and to get the returned token to use it on the app.
Is there any way to get this to work?

You should instead make use of this package: https://pub.dev/packages/flutter_web_auth

You can redirect to you auth page after successful login and on the page use redirection to custom schema with code and state.
On the redirection page something like this works:
window.onLoaded = function(){
window.location.replace('mycustomschema://auth/mobile?code=123&state=456'
}


OpenDP bug in filler_placement with set_placement_padding

The filler_placement command with set_placement_padding generated overlapped placement result.
There are overlapped cells after OpenDP.
I attached Innovus checkPlace results that contains a list of overlapped cells (bug02_result.rpt).
Could you check the attached testcase?
bug02.tar.gz
I used the following commit in OpenROAD repo:
https://github.com/The-OpenROAD-Project/OpenROAD/commit/4187ce74b7f7dccb126f9870383511eca24132b5
Thanks,

what happens if you remove the second contradictory call to set_placement_padding?
it make absolutely no sense to change the padding mid-flow

I see. It isn't filling the padding sites so you were trying a work around.
I think the real issue is not filling the padding.

Try it now (after commit 3150c4e). Thanks for the test case.

I confirmed that it worked fine. Thank you very much.


PopScope does not prevent bringing app to background by press android back button

Steps to reproduce

Run app on Android device
Press back button

Expected results
Dialog is displayed
Because PopScope with canPop: false should blocks the current route from being popped
Actual results
App goes to background
Code sample
Code sample
import 'package:flutter/material.dart';

void main() => runApp(const App());

class App extends StatelessWidget {
  const App({super.key});

  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      initialRoute: '/',
      onGenerateRoute: (RouteSettings settings) {
        return switch (settings.name) {
          '/' => MaterialPageRoute<void>(
              settings: const RouteSettings(
                name: '/',
              ),
              builder: (BuildContext context) {
                return _HomePage();
              },
            ),
          _ => MaterialPageRoute<void>(
              settings: const RouteSettings(
                name: 'unknown_page',
              ),
              builder: (BuildContext context) {
                return const _UnknownPage();
              },
            ),
        };
      },
      builder: (_, routing) {
        if (routing == null) {
          return const SizedBox.shrink();
        }

        return PopScope(
          child: routing,
          canPop: false,
          onPopInvokedWithResult: (_, __) {
            showDialog(
              context: context,
              builder: (_) => const AlertDialog(
                title: Text('Do you want to close the app?'),
              ),
            );
          },
        );
      },
    );
  }
}

class _HomePage extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(
        title: const Text('Home page'),
      ),
      body: const Center(
        child: Column(
          mainAxisAlignment: MainAxisAlignment.center,
          children: <Widget>[
            Text('A system back gesture here should trigger exit dialog. But it does not.'),
          ],
        ),
      ),
    );
  }
}

class _UnknownPage extends StatelessWidget {
  const _UnknownPage();

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      backgroundColor: Colors.grey.withBlue(180),
      body: const Center(
        child: Column(
          mainAxisAlignment: MainAxisAlignment.center,
          children: <Widget>[
            Text('404'),
          ],
        ),
      ),
    );
  }
}

Screenshots or Video

Screenshots / Video demonstration
[Upload media here]

Logs
Logs
[Paste your logs here]

Flutter Doctor output
Doctor output
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, 3.24.3, on macOS 14.6.1 23G93 darwin-x64, locale
    ru-RU)
[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
[✓] Xcode - develop for iOS and macOS (Xcode 15.4)
[✓] Chrome - develop for the web
[✓] Android Studio (version 2024.1)
[✓] IntelliJ IDEA Community Edition (version 2024.1.2)
[✓] Connected device (4 available)
[✓] Network resources

Hi @MariaMelnik, can you check out https://github.com/flutter/flutter/issues/156967#issuecomment-2416811492 to see if it resolves your issue?

@jellynoone yes, it was the goal to prevent closing app for all routes in one place. I got my mistake but maybe you know any other solution to do it in one place?

@MariaMelnik The best I can think of is using your own Route. Something, like:
class AppPageRoute<T> extends MaterialPageRoute<T> {
  AppPageRoute({required super.builder});

  @override
  Widget buildContent(BuildContext context) {
    return PopScope(
      canPop: false,
      child: super.buildContent(context),
    );
  }
}

However, be warned if you do this on every route, you will be preventing back presses on all of these routes. For instance, if your navigation stack is: /home and /settings. And the user presses the back button on /settings, the dialog will show up. I'm guessing this isn't what you want.
If you only want to prevent app close pops, then you only need to place the PopScope on your root route i.e. home.

@jellynoone yes, my idea was to prevent closing app for every first route despite of which route is first at this moment. But not prevent popping if this route is not first route in the app at this moment.
I will try another ways.
Thank you!
Closing this issues.

Works as expected


Pretrained model using all three corpus

First, I would say thank you for making this awesome repository,
I've been trying to train the model with all the dataset from 3 corpus but it seems will take really long time (need about 30 hours for single epoch. I'm using GTX1070 in SLI).
is it possible to provide a pretrained model with all the three dataset? as the current pretrained model is trained only with VCTK dataset if I'm not wrong
Thank you very much

silly me, i haven't configure the CUDA for the tensorflow in the ubuntu no wonder it runs really slow. after i built again from the source it now detect the CUDA lib and the training is running at decent speed. Training the model using all the corpus now, will report back later

@shiinoandra Any luck training the model? Would you mind uploading it to dropbox or google drive?
Cheers!

Is it possible for you to share the pretrained model which was obtained for all 3 corpuses?
Thanks

I'm 40 epochs in to the training with a 1080ti. Will share the entire training folder output when it's done with 50.

output from the newly trained model (4.8GB!! - uploading soon) using the same examples...
he hoped there would be stoe for dinner turnips and carrits and brose patitoes and fatin lutton pieces to be latled out an thick peppered flower faten sauce
stuffed into you his belly countled him
after early night fall the yellowe lamps whuld light hup pher and there the squalid quarter of the browfls
ilo burti and e good in your mind
nomebut dan freshn knawly is waiting on you could night husband

I got this message while trying to download your pre trained model - "The free hosting period for this file has now expired, only premium users can download it".
Can you please upload it on google drive or somewhere else where it can be accessible?
Thanks

@sc1311 did you get the model ?

@ebadali No

@greigs could you renew your uploaded file or put it on a public server somewhere?  ufile.io is pure dark pattern (clickbait, adds, data hostage taking). That isn't the sort of place you'd like to share your hard-won model data. I'd be happy to host your 5GB model, if you could figure out a way to get it to me (dropbox, S3, google drive?).

@ebadali @hobson @greigs Can you share the trained model, please. Thanks in advance.

@greigs Do you still have the pretrained model?

@ebadali @hobson Here is the file uploaded by @greigs. https://drive.google.com/open?id=1RpAg48ZFIsPWm3dC5GU34pRWEXkOH1S-

Hi @greigs , @MarinescuEvghenii , What is your CTC loss of using all three corpus. I use all three corpus and I got about 103 CTC loss in Epoch 6. After that, the CTC loss became inf. Thanks.


Static width of blames in "File Blame" mode

Hi

GitLens Version: 4.1.2
VSCode Version: 1.13.1
OS Version: Mac OS X

I would expect to have some static editable width value of the file gutter blame area. If I set the format of file gutter annotations to "${id} ${author}" and some authors have longer/shorter names then another, then the lines in the gutter area are indented differently as well as the code in the file.

Thanks!

@obrejla See Advanced Formatting for details on how to control the lengths.
In your example you could use ${id|8} ${author|20}  -- you can replace 20 with however many characters of the author you want displayed.
Hope that helps!

Cool! That did the trick, thanks a lot!


feat: Add support for IAM role boundary policy

Description
This pull request adds permissions boundaries suport for IAM roles

And v2.15.0 has been also just released. :)


Build error on QT NUC

Running docker-compose -f docker-compose-qt-nuc.yml up leads to this error:
Reading package lists...
W: GPG error: http://packages.ros.org/ros/ubuntu xenial InRelease: The following signatures were invalid: KEYEXPIRED<PHONE_NUMBER>
W: The repository 'http://packages.ros.org/ros/ubuntu xenial InRelease' is not signed.
Reading package lists...
Building dependency tree...
Reading state information...
The following NEW packages will be installed:
  ros-kinetic-joint-state-publisher-gui
0 upgraded, 1 newly installed, 0 to remove and 387 not upgraded.
Need to get 11.9 kB of archives.
After this operation, 55.3 kB of additional disk space will be used.
WARNING: The following packages cannot be authenticated!
  ros-kinetic-joint-state-publisher-gui
E: There were unauthenticated packages and -y was used without --allow-unauthenticated
The command '/bin/sh -c APT_INSTALL="apt-get install -y --no-install-recommends" &&     apt-get update &&     DEBIAN_FRONTEND=noninteractive $APT_INSTALL     ros-kinetic-joint-state-publisher-gui     && rm -rf -- /var/lib/apt/lists/*' returned a non-zero code: 100
ERROR: Service 'harmoni_full' failed to build : Build failed

Running docker-compose -f docker-compose-qt-nuc.yml up leads to this error:
Reading package lists...
W: GPG error: http://packages.ros.org/ros/ubuntu xenial InRelease: The following signatures were invalid: KEYEXPIRED<PHONE_NUMBER>
W: The repository 'http://packages.ros.org/ros/ubuntu xenial InRelease' is not signed.
Reading package lists...
Building dependency tree...
Reading state information...
The following NEW packages will be installed:
  ros-kinetic-joint-state-publisher-gui
0 upgraded, 1 newly installed, 0 to remove and 387 not upgraded.
Need to get 11.9 kB of archives.
After this operation, 55.3 kB of additional disk space will be used.
WARNING: The following packages cannot be authenticated!
  ros-kinetic-joint-state-publisher-gui
E: There were unauthenticated packages and -y was used without --allow-unauthenticated
The command '/bin/sh -c APT_INSTALL="apt-get install -y --no-install-recommends" &&     apt-get update &&     DEBIAN_FRONTEND=noninteractive $APT_INSTALL     ros-kinetic-joint-state-publisher-gui     && rm -rf -- /var/lib/apt/lists/*' returned a non-zero code: 100
ERROR: Service 'harmoni_full' failed to build : Build failed

Have you solved it? Or are you stiil facing this issue on the NUC?

I was able to get around it by pulling instead of building but I believe the build issue is still there

This might be related to #84, I have rebuilt and am pushing for the arm builds, will need to try for amd.

Scratch the second part there, the pushes I am making should fix for the nuc.

This should be fixed, please reopen if the issue persists.


Add test for pcap probe

Right now, FlowVLANSegmentation is the only test that makes use of the capture type "pcap". We should have a dedicated test for pcap capture

Just moved the pcap flag at the right place
https://softwarefactory-project.io/r/#/c/7049/


Pastanaga Editor: Hitting return on a tiles field should switch to the next tile

Go to https://plone-react.kitconcept.io
Login
Edit front page title
Hit return
-> nothing happens
Expected behavior:
-> switch to the next tile

Done (at Costa Brava sprint).


Simplify pyproject.toml

There were many napari-related entry points in the pyproject.toml. I think these aren't needed, so I've streamlined them.

I forgot we had that CLI entry point, it's from the dark days before napari had plugins! I've removed it and amended the PR title.

I've re-requested your review, just in case I've done something daft.


Update docker image to include the new ARA 1.0.1 version

Hello,
I'm using your docker image with pleasure because you provide a "pull, run, ready" image which is working fine :)
So, have you notice the recent new ara release ?
I've seen they change the structure of the project, now the "backend" is splitted between an api (2nd tier) and a web client (1st tier). They provide role to install them in a real server.
https://ara.readthedocs.io/en/latest/installation.html
Do you plan to work to integrate this new version ?
I can suppose you will use the same docker image as now with gunicorn ?
Will you provide one or two docker image ?
Thanks in advance for your informations

Hi! Yes I'm aware.of the new release, but I've been busy studying for the
AWS Solutions Architect exam. I will try to make some time to look at it -
I actually haven't even played with the new version yet.
On Sun, Jun 23, 2019 at 15:50 Pierre GINDRAUD<EMAIL_ADDRESS>wrote:

Hello,
I'm using your docker image with pleasure because you provide a "pull,
run, ready" image which is working fine :)
So, have you notice the recent new ara release ?
I've seen they change the structure of the project, now the "backend" is
splitted between an api (2nd tier) and a web client (1st tier). They
provide role to install them in a real server.
https://ara.readthedocs.io/en/latest/installation.html
Do you plan to work to integrate this new version ?
I can suppose you will use the same docker image as now with gunicorn ?
Will you provide one or two docker image ?
Thanks in advance for your informations
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/zswanson/ara-docker/issues/1?email_source=notifications&email_token=AAJWY6TOAGP3SMDXFBLKM7LP37HYZA5CNFSM4H2ZUKW2YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4G3FDOJQ,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAJWY6V5VYPXZ5CAEHDNAKDP37HYZANCNFSM4H2ZUKWQ
.

Ok thanks, for this news. I'm in the same situation with this new release, I've lock my projects to 0.16.5 version because the new structure completly break the old one.
I stay tuned

Hmm yah this will require a different approach because it now has two
separate services. May just have to use the ansible install roles (which
already configure gunicorn!) and then copy the installation to a minimal
image.
On Sun, Jun 23, 2019 at 15:59 Pierre GINDRAUD<EMAIL_ADDRESS>wrote:

Ok thanks, for this news. I'm in the same situation with this new release,
I've lock my projects to 0.16.5 version because the new structure completly
break the old one.
I stay tuned
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/zswanson/ara-docker/issues/1?email_source=notifications&email_token=AAJWY6SK7LSR7MW3VHBRML3P37I2RA5CNFSM4H2ZUKW2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODYLF25Q#issuecomment-504782198,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAJWY6XPHJLSDXHHVPDQP5DP37I2RANCNFSM4H2ZUKWQ
.

Hello, I was lock in my development because of ara v0.16 limitations, so I've passed some hours to work on new docker images for v1.0.
For your information I've setup theses images
https://github.com/Turgon37/docker-ansible-araserver/pull/1
https://github.com/Turgon37/docker-ansible-araweb/pull/1
and they works fine
Have you been able to advance on your side?

I have not, I was in the middle of a job change when you reached out, and
have still been ramping up there. This has been pretty low priority as a
result.
On Thu, Aug 15, 2019, 7:53 PM Pierre GINDRAUD<EMAIL_ADDRESS>wrote:

Hello, I was lock in my development because of ara v0.16 limitations, so
I've passed some hours to work on new docker images for v1.0.
For your information I've setup theses images
Turgon37/docker-ansible-araserver#1
https://github.com/Turgon37/docker-ansible-araserver/pull/1
Turgon37/docker-ansible-araweb#1
https://github.com/Turgon37/docker-ansible-araweb/pull/1
and they works fine
Have you been able to advance on your side?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/zswanson/ara-docker/issues/1?email_source=notifications&email_token=AAJWY6THATLX6XYCZDMLNILQEXUBHA5CNFSM4H2ZUKW2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD4NJQ5Y#issuecomment-521836663,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAJWY6WYYMLERXHL2PCJMMDQEXUBHANCNFSM4H2ZUKWQ
.

Ok thanks for the news, don't be worry I am not angry, I understand your situation very well :), so If you want to use my images feel free to :)

As a result if you want to consider this issue as resolved, you can :)


Update README.md

testing e2e against 3.10

/test e2e

/retest

/retest

Some underlying progress in ci-operator/release must have occurred since mid August ... I'm at least seeing the job get launched now.
Tracking in real time, but so far:

the jenkins centos build failed cause of an env hiccup with rpm/gpg:

Retrieving key from https://copr-be.cloud.fedoraproject.org/results/alsadi/dumb-init/pubkey.gpg

GPG key retrieval failed: [Errno 12] Timeout on https://copr-be.cloud.fedoraproject.org/results/alsadi/dumb-init/pubkey.gpg: (28, 'Connection timed out after 30001 milliseconds')
error: build error: running 'curl https://pkg.jenkins.io/redhat-stable/jenkins.repo -o /etc/yum.repos.d/jenkins.repo &&     rpm --import https://pkg.jenkins.io/redhat-stable/jenkins-ci.org.key &&     yum install -y centos-release-scl-rh &&     curl https://copr.fedorainfracloud.org/coprs/alsadi/dumb-init/repo/epel-7/alsadi-dumb-init-epel-7.repo -o /etc/yum.repos.d/alsadi-dumb-init-epel-7.repo &&     x86_EXTRA_RPMS=$(if [ "$(uname -m)" == "x86_64" ]; then echo -n java-1.8.0-openjdk.i686 java-1.8.0-openjdk-devel.i686 ; fi) &&     INSTALL_PKGS="dejavu-sans-fonts rsync gettext git tar zip unzip openssl bzip2 dumb-init java-1.8.0-openjdk jenkins-2.107.3-1.1" &&     yum -y --setopt=tsflags=nodocs install $INSTALL_PKGS $x86_EXTRA_RPMS &&     rpm -V  dejavu-sans-fonts rsync gettext git tar zip unzip openssl bzip2 dumb-init jenkins-2.107.3-1.1 &&     yum clean all  &&     localedef -f UTF-8 -i en_US en_US.UTF-8' failed with exit code 1

the slave images were built successfully

the test image build failed very early in the init stage ... not sure what is up there yet

@bparees FYI ^^

/retest

The failures are consistent .... on the test image, I see the 3.10 tag in the api.ci openshift namespace ... so not sure why that build fails.
On the centos, perhaps their is a prolonged rpm/gpg hiccup.  But I'm wondering if there are some issues with the base 3.10 images on api.ci we are building off of.
My compare of master.yaml and openshift-3.10.yaml uncovered no deltas, other than the tag used for the openshift base images (i.e. 3.10 vs. 4.0).
The presumit.yaml has some slight differences between master and 3.10.  But I am not familiar enough to discern if then matter.
Going to complete the 3.11 integration @bparees started that he and I chatted about on IRC and see what results are there.
Might need DPTP cycles to sort out (which could be problematic to get short term).


python instagram.py Scottraymond 10-million-passwords-list-top-1000000.txt

python instagram.py Scottraymond 10-million-passwords-list-top-1000000.txt
Originally posted by @scottraymondusa in https://github.com/Pure-L0G1C/Instagram/pull/158#issuecomment-637412920

:)


Plugin settings

I'd like to propose some kind of plugin settings or config block.
Currently, the only way to pass something in to a plugin is as a global (e.g. PhysX). This is less than ideal, and I think a configuration block would help here.
My use case: I've built a new plugin, hecs-plugin-rapier, and I've hard-coded a dependency on hecs-plugin-composable by using ComposableTransform throughout; however, hecs-plugin-rapier could work just as well with the hecs-plugin-core Transform component as the ComposableTransform component. In other words, if there were a config block for plugins, I could allow the user of hecs-plugin-rapier decide which "Transform" component to use, and users of this plugin would not have to buy in to the hecs-plugin-composable way of doing things if they didn't want to.

Does the decorate() function not help here? Your rapier plugin could have something like world.physics.setComposableTransforms(Boolean) or world.physics.configure({ composableTransforms: true })

It might help. How would I intercept the query-builder that converts static queries to instance queries here?
...
import { ComposableTransform } from "hecs-plugin-composable";

export class PhysicsSystem extends System {
  order = Groups.Simulation;

  static queries = {
    default: [RigidBodyRef, ComposableTransform, WorldTransform],
  };
...

Maybe if there were a way to construct this.queries with a parameterized function...? But then I guess I would need to do it inside the System.init() method... maybe I could call this.world.queries.create(...) dynamically there. Hmmm....

See also #23 as an example use case

I see your problem but I'm not 100% sure what the best way to solve it would be.
I prefer that my systems are simple and do one very specific thing really well. I suspect trying to make a system work differently based on params is like an ECS code smell.

I propose we just do this as it's much simpler. Let me know what you think. https://github.com/gohyperr/hecs/pull/26#issuecomment-730060336

I think this is a great way forward. Thanks!


feat(types): add responseTime parameter to customErrorMessage

Description
Add responseTime parameter to customErrorMessage type definition to allow access to the request's response time when creating custom error messages.
Changes

Updated TypeScript type definition for customErrorMessage to include responseTime parameter
Response time is provided in milliseconds, consistent with pino-http's existing timing

Example Usage
const logger = pinoHttp({
  customErrorMessage: (req, res, err, responseTime) => {
    return `${err.message} (took ${responseTime}ms)`;
  }
});

Thanks for opening a PR! Can you please add a unit test? We use tsd for the types


Add Code of Conduct

Can we use the simpler language that we used on the dotnet/wpf repo? When we first added the CoC, it was important to explain it heritage. It isn't anymore. We just need to say that it is the .NET Foundation CoC.
Is the CODE_OF_CONDUCT.md file necessary? Does GitHub respect it?

Yes, GitHub respects CODE_OF_CONDUCT.md file: https://help.github.com/articles/adding-a-code-of-conduct-to-your-project/. We can move it to .github subdir to "hide it" from root dir.
However, GitHub does not (yet) recognize custom text in the file, so technically it does not help with community checklist: https://github.com/dotnet/wpf/community.
@richlander which wording do you want to use? We have 2 in WPF:

https://github.com/dotnet/wpf/blob/master/CODE_OF_CONDUCT.md
https://github.com/dotnet/wpf#code-of-conduct

which wording do you want to use?

The second one. That's the one I linked to initially.
Is there a link that shows what that file does? I cannot see what that is.
I'm wondering if we should go with the text I prefer in the README and then copy the whole contributor covenant text  into the C_O_C.md file? @jongalloway ... do you have thoughts on this?

From looking at other projects, the standard seems to be linking to code of conduct rather than copying the text in (k8s, rails). So LGTM as-is.

That's the one I linked to initially.

Ah, I didn't notice it is specific link - the title hides the subsection link :(

Is there a link that shows what that file does? I cannot see what that is.

The linked docs is all I have: https://help.github.com/articles/adding-a-code-of-conduct-to-your-project/
It is integrated in https://github.com/dotnet/wpf/community checklist.

Okay, after looking at this more, the Contributor Covenant guidance recommends adding the text or markdown version at the root of the project. Also, at a minimum we also need to add a contact method.

I'd agree with @jongalloway. It seems more logical to copy & paste the text into that file. We do the same for licenses and CoC seems to be like that -- the point is, it rarely -- if ever changes.

Just to clarify my understanding: The prefered way is to use "default" GH way and include full text of the CoC in the CODE_OF_CONDUCT.md file in root, is that correct?
If that's the case, I will merge this PR and create a new one via GH "Add CoC".
Please upvote if you agree. Thanks!

@karelz -- yes. However, we need to validate that the default text and the .NET Foundation CoC text match. Otherwise, it would be odd. That's a question for @jongalloway.

This issue has stalled. I will merge the current form of CoC.
If there are changes needed, please submit a separate PR.


Crimereports certificate verify failed

Home Assistant release with the issue:
0.103.6

Last working Home Assistant release (if known):
Unknown
Operating environment (Hass.io/Docker/Windows/etc.):

Hass.io
Integration:

https://github.com/home-assistant/home-assistant/tree/dev/homeassistant/components/crimereports
Description of problem:
ssl.SSLError: ("bad handshake: Error([('SSL routines', 'tls_process_server_certificate', 'certificate verify failed')])",)
Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant):
sensor:
  - platform: crimereports
    name: 'Crime Reports'
    radius: 4828.03

Traceback (if applicable):
2020-01-08 11:50:28 ERROR (MainThread) [homeassistant.components.sensor] crimereports: Error on device update!
Traceback (most recent call last):
  File "/usr/local/lib/python3.7/site-packages/urllib3/contrib/pyopenssl.py", line 485, in wrap_socket
    cnx.do_handshake()
  File "/usr/local/lib/python3.7/site-packages/OpenSSL/SSL.py", line 1934, in do_handshake
    self._raise_ssl_error(self._ssl, result)
  File "/usr/local/lib/python3.7/site-packages/OpenSSL/SSL.py", line 1671, in _raise_ssl_error
    _raise_current_error()
  File "/usr/local/lib/python3.7/site-packages/OpenSSL/_util.py", line 54, in exception_from_error_queue
    raise exception_type(errors)
OpenSSL.SSL.Error: [('SSL routines', 'tls_process_server_certificate', 'certificate verify failed')]

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/local/lib/python3.7/site-packages/urllib3/connectionpool.py", line 672, in urlopen
    chunked=chunked,
  File "/usr/local/lib/python3.7/site-packages/urllib3/connectionpool.py", line 376, in _make_request
    self._validate_conn(conn)
  File "/usr/local/lib/python3.7/site-packages/urllib3/connectionpool.py", line 994, in _validate_conn
    conn.connect()
  File "/usr/local/lib/python3.7/site-packages/urllib3/connection.py", line 394, in connect
    ssl_context=context,
  File "/usr/local/lib/python3.7/site-packages/urllib3/util/ssl_.py", line 370, in ssl_wrap_socket
    return context.wrap_socket(sock, server_hostname=server_hostname)
  File "/usr/local/lib/python3.7/site-packages/urllib3/contrib/pyopenssl.py", line 491, in wrap_socket
    raise ssl.SSLError("bad handshake: %r" % e)
ssl.SSLError: ("bad handshake: Error([('SSL routines', 'tls_process_server_certificate', 'certificate verify failed')])",)

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/local/lib/python3.7/site-packages/requests/adapters.py", line 449, in send
    timeout=timeout
  File "/usr/local/lib/python3.7/site-packages/urllib3/connectionpool.py", line 720, in urlopen
    method, url, error=e, _pool=self, _stacktrace=sys.exc_info()[2]
  File "/usr/local/lib/python3.7/site-packages/urllib3/util/retry.py", line 436, in increment
    raise MaxRetryError(_pool, url, error or ResponseError(cause))
urllib3.exceptions.MaxRetryError: HTTPSConnectionPool(host='www.crimereports.com', port=443): Max retries exceeded with url: /api/crimes/details.json?days=monday%2Ctuesday%2Cwednesday%2Cthursday%2Cfriday%2Csaturday%2Csunday&end_date=2020-01-08&end_time=23&incident_types=Theft%2CArson%2CVehicle+Recovery%2CSexual+Assault%2CRobbery%2CDrugs%2COther+Sexual+Offense%2CBreaking+%26+Entering%2CVehicle+Stop%2CTheft+from+Vehicle%2CTraffic%2CQuality+of+Life%2CPedestrian+Stop%2CLiquor%2CProactive+Policing%2CDisorder%2COther%2CProperty+Crime%2CProperty+Crime+Commercial%2CTheft+of+Vehicle%2CKidnapping%2CAssault+with+Deadly+Weapon%2CFire%2CHomicide%2CSexual+Offense%2CDeath%2CEmergency%2CAssault%2CAlarm%2CMissing+Person%2CWeapons+Offense%2CCommunity+Policing%2CProperty+Crime+Residential%2CFamily+Offense&include_sex_offenders=false&filter_by_viewport=true&lat1=##.###&lng1=-###.###&lat2=##.###&lng2=-###.###&sandbox=false&start_date=2020-01-08&start_time=0 (Caused by SSLError(SSLError("bad handshake: Error([('SSL routines', 'tls_process_server_certificate', 'certificate verify failed')])")))

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 292, in _async_add_entity
    await entity.async_device_update(warning=False)
  File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 461, in async_device_update
    await self.hass.async_add_executor_job(self.update)
  File "/usr/local/lib/python3.7/concurrent/futures/thread.py", line 57, in run
    result = self.fn(*self.args, **self.kwargs)
  File "/usr/src/homeassistant/homeassistant/components/crimereports/sensor.py", line 117, in update
    now().date(), include=self._include, exclude=self._exclude
  File "/usr/local/lib/python3.7/site-packages/crimereports/__init__.py", line 136, in get_incidents
    resp = requests.get(CRIME_URL, params=self._get_params(date, include, exclude))
  File "/usr/local/lib/python3.7/site-packages/requests/api.py", line 75, in get
    return request('get', url, params=params, **kwargs)
  File "/usr/local/lib/python3.7/site-packages/requests/api.py", line 60, in request
    return session.request(method=method, url=url, **kwargs)
  File "/usr/local/lib/python3.7/site-packages/requests/sessions.py", line 533, in request
    resp = self.send(prep, **send_kwargs)
  File "/usr/local/lib/python3.7/site-packages/requests/sessions.py", line 646, in send
    r = adapter.send(request, **kwargs)
  File "/usr/local/lib/python3.7/site-packages/requests/adapters.py", line 514, in send
    raise SSLError(e, request=request)
requests.exceptions.SSLError: HTTPSConnectionPool(host='www.crimereports.com', port=443): Max retries exceeded with url: /api/crimes/details.json?days=monday%2Ctuesday%2Cwednesday%2Cthursday%2Cfriday%2Csaturday%2Csunday&end_date=2020-01-08&end_time=23&incident_types=Theft%2CArson%2CVehicle+Recovery%2CSexual+Assault%2CRobbery%2CDrugs%2COther+Sexual+Offense%2CBreaking+%26+Entering%2CVehicle+Stop%2CTheft+from+Vehicle%2CTraffic%2CQuality+of+Life%2CPedestrian+Stop%2CLiquor%2CProactive+Policing%2CDisorder%2COther%2CProperty+Crime%2CProperty+Crime+Commercial%2CTheft+of+Vehicle%2CKidnapping%2CAssault+with+Deadly+Weapon%2CFire%2CHomicide%2CSexual+Offense%2CDeath%2CEmergency%2CAssault%2CAlarm%2CMissing+Person%2CWeapons+Offense%2CCommunity+Policing%2CProperty+Crime+Residential%2CFamily+Offense&include_sex_offenders=false&filter_by_viewport=true&lat1=##.###&lng1=-###.###&lat2=##.###&lng2=-###.###&sandbox=false&start_date=2020-01-08&start_time=0 (Caused by SSLError(SSLError("bad handshake: Error([('SSL routines', 'tls_process_server_certificate', 'certificate verify failed')])")))

Additional information:
Looks like crimereports.com has moved to cityprotect.com
The certificate on crimereports.com was renewed on or around 12/11/2019, but the certificate chain is incomplete which maybe causing the handshake fail.

Looks like the api has been removed - https://www.govtech.com/biz/Motorola-Parts-with-Socrata-Ends-Access-to-Open-Crime-APIs.html


No swiftSettings: for .plugin targets?

Description
There's apparently no way to specify, e.g. -warnings-as-errors for a .plugin target.  Seems like an obvious oversight.
Expected behavior
No response
Actual behavior
No response
Steps to reproduce
No response
Swift Package Manager version/commit hash
Swift Package Manager - Swift 5.8.0
Swift & OS version (output of swift --version ; uname -a)
swift-driver version: 1.75.2 Apple Swift version 5.8 (swiftlang-<IP_ADDRESS>.2 clang-14<IP_ADDRESS>.100)
Target: arm64-apple-macosx13.0
Darwin DaveA-MBP14.localdomain 22.4.0 Darwin Kernel Version 22.4.0: Mon Mar  6 20:59:28 PST 2023; root:xnu-8796.101.5~3/RELEASE_ARM64_T6000 arm64

cc @neonichu @abertelrud

Yes, that is correct. Not sure if it was an oversight, it matches the proposal.

It can be kludged around by putting all the functionality in a library, so there's no good reason to disallow the settings directly in the plugin.

@abertelrud do you remember the reasoning for not exposing it?


No longer works with iTerm2 (3.0.0)

I've also tried updating the config to use: "iTerm_version": "nightly", and "iTerm_version": "stable", as well as removing it. I haven't been able to get it to work

I have the same problem ATM

Ditto. There is no output at all.

Same for me. Updated to iTerm2 3.0.0 and Shuttle doesn't open terminals anymore. If Iterm is closed, it opens and then quickly closes. If it's already open, it doesn't do anything.

Same problem here, switched back to Terminal.app for now

Me too.

Same here :(

+1

+1

ditto

just changed my JSON, and now it works.
"editor": "default",
"launch_at_login": true,
"terminal": "iTerm",
"iTerm_version": "nightly",
"default_theme": "Man Page",
"open_in": "tab",
"show_ssh_config_hosts": true,
"ssh_config_ignore_hosts": [  ],
"ssh_config_ignore_keywords": [  ],
"hosts":

Confirmed, the @rodaddy changes to the config file work.

Hmmm. The change did not work for me.
"editor": "vim",
"launch_at_login": true,
"terminal": "iTerm",
"iTerm_version": "nightly",
"open_in": "tab",
"default_theme": "Default",
"show_ssh_config_hosts": true,
"ssh_config_ignore_hosts": [  ],
"ssh_config_ignore_keywords": [  ],

My only set of parameters that have anything to do with iTerm are:
"terminal": "iTerm",
"iTerm_version": "nightly",
"open_in": "tab",
"launch_at_login": true,
The differences from before the upgrade are the "iTerm_version" parameter which was changed to "nightly" and the addition (I didn't have it previously) of the "open_in" parameter with the value "tab".
Maybe you could try only with these, without default_theme and editor.
I've tested again and it still works ;) It works with iTerm closed and it also works with iTerm already open. Shuttle version 1.2.5.

I am on 1.2.6. I took out the editor and default_theme and still no luck.

@rodaddy : What version are you using?

I just updated to 1.2.6 from 1.0.0 and with the following config settings, it is working again:
{
    "_comments": [
        "Valid terminals include: 'Terminal.app' or 'iTerm'",
        "In the editor value change 'default' to 'nano', 'vi', or another terminal based editor.",
        "Hosts will also be read from your ~/.ssh/config or /etc/ssh_config file, if available",
        "For more information on how to configure, please see http://fitztrev.github.io/shuttle/"
    ],
    "editor": "default",
    "launch_at_login": false,
    "terminal": "iTerm",
    "iTerm_version": "nightly",
    "default_theme": "Homebrew",
    "open_in": "tab",
    "show_ssh_config_hosts": false,
    "ssh_config_ignore_hosts": [],
    "ssh_config_ignore_keywords": [],
    "hosts": ...
}

New version 3.0.0 of iTerm2
On Jun 2, 2016, at 1:12 PM, kandrei<EMAIL_ADDRESS>wrote:
@rodaddy https://github.com/rodaddy : What version are you using?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/fitztrev/shuttle/issues/174#issuecomment-223375277, or mute the thread https://github.com/notifications/unsubscribe/ABBGmRGGw7fzxQ7gXkdhwj4VGEYNSy78ks5qHx0JgaJpZM4Ir0_v.

@rodaddy - I think he meant what version of Shuttle are you using...

Yes, I was asking about the Shuttle version. But according to @cmgmyr, it should work on the 1.2.6 version also, so maybe it's something special with your setup @skebi69 .

Newest 1.2.6. Didn't work with 1.2.5
Cheers,
Rico
Sorry about the short message:
Sent from my iPhone
On Jun 2, 2016, at 2:15 PM, kandrei <notifications<EMAIL_ADDRESS>wrote:
Yes, I was asking about the Shuttle version. But according to @cmgmyrhttps://github.com/cmgmyr, it should work on the 1.2.6 version also, so maybe it's something special with your setup @skebi69https://github.com/skebi69 .
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/fitztrev/shuttle/issues/174#issuecomment-223393203, or mute the threadhttps://github.com/notifications/unsubscribe/ABBGmZcGp8LYHFsOQq8nlKBzaTk5Ibgmks5qHyvUgaJpZM4Ir0_v.

same for me. changed to 1.2.6 and settings:
	"terminal": "iTerm",
    "editor": "default",
    "launch_at_login": false,
    "iTerm_version": "nightly",
    "default_theme": "Default",
    "open_in": "tab",
    "show_ssh_config_hosts": true,
    "ssh_config_ignore_hosts": [],
    "ssh_config_ignore_keywords": [],

and its working again

thx @rodaddy it works.

Hi All,
I'm updating to iTerm 3 now and will report back.

Hello All,
I have tested the new nightly Build 3.0.20160602-nightly and all is working for me with no changes. I'll be on gitter for about 2 or 3 hours so if you need help ping me there.
Here is the demo config im using:
{
  "_comments": [
    "Valid terminals include: 'Terminal.app' or 'iTerm'",
    "In the editor value change 'default' to 'nano', 'vi', or another terminal based editor.",
    "Hosts will also be read from your ~/.ssh/config or /etc/ssh_config file, if available",
    "For more information on how to configure, please see http://fitztrev.github.io/shuttle/"
  ],
  "editor": "default",
  "launch_at_login": false,
  "terminal": "iTerm",
  "iTerm_version": "nightly",
  "default_theme": "Default",
  "open_in": "tab",
  "show_ssh_config_hosts": false,
  "ssh_config_ignore_hosts": [  ],
  "ssh_config_ignore_keywords": [  ],
  "hosts": [
    {
      "cmd": "ps aux | grep defaults",
      "name": "Grep - Opens in Default-window-theme-title"
    },
    {
      "Spouses Servers": [
        {
          "cmd": "echo '—->WARNING<-- Running a command in this active terminal! Are you sure? y/n'; read sure; if [ $sure == y ]; then echo running command... && tail -f /var/log/current; else echo exiting...; fi",
          "inTerminal": "current",
          "name": "Logs - Opens in the current active terminal window"
        },
        {
          "Jane’s Servers": [
            {
              "cmd": "ssh<EMAIL_ADDRESS>              "inTerminal": "tab",
              "name": "SSH blog - Opens in Tab of active window",
              "theme": "basic",
              "title": "title of tab"
            },
            {
              "cmd": "ssh<EMAIL_ADDRESS>              "inTerminal": "new",
              "name": "SSH Shop - Opens in New Window",
              "theme": "basic",
              "title": "title of new window"
            }
          ]
        }
      ]
    },
    {
      "Vagrant": [
        {
          "cmd": "ssh<EMAIL_ADDRESS>-p 2222 -i ~/.vagrant.d/insecure_private_key",
          "name": "Vagrant - Opens in default-window-theme-title"
        }
      ]
    }
  ]
}

btw.. the gitter link is here  :)

I'm going to close this. It's hard to know for sure because I dont know what version everyone is upgrading from but shuttle has a lot of new directives. If your shuttle.JSON is not correct its not going to work. Above I posted a working config. If you have questions please let me know.

I have the exact same config file as yours (except for the hosts parts) and yet my Shuttle still doesn't work. All it does is focus iTerm but doesn't create a new tab or insert the ssh line. I am using Shuttle 1.2.6.
Is there anything else I can check?


feat: add ack node image list

What type of PR is this?
/kind documentation

What this PR does / why we need it:
Now, we can't query through AlibabCloud API to get the correct image ids, let's maintain this in this repo.
Which issue(s) this PR fixes:

Fixes #none
Special notes for your reviewer:
none
Does this PR introduce a user-facing change?

/lgtm


[Utils] Allow ReorderTransformFunc to be used without param manager

Prior to this commit, the ReorderTransformFunc required several components of the ParamManager to use.  The functionality it provides, reordering dataflow blocks to minimize the liveset, is useful outside of the context of the ParamManager.  This commit makes the following changes, allowing it to be used independently of the ParamManager.

Generate the pidx2binname dictionary outside of ReorderTransformFunc

Allow parameters to be separate func.params, rather than a single bundled tuple parameter.

please rebase, so we will get it in. Would be good to be mindful about the cutoff time, so we stop merging old flow and start removing the old flow as SLM migration rolls out. I am thinking about perhaps in a week. let me know what you think

please rebase, so we will get it.

Thank you for the ping, and rebased!

Would be good to be mindful about the cutoff time, so we stop merging old flow and start removing the old flow as SLM migration rolls out. I am thinking about perhaps in a week. let me know what you think

Good call, and about a week from now sounds good.  I think having this commit will help in the transition, as the re-ordering of computations is now independent of the ParamManager, and can be done as part of IRModule transformations in the new SLM-based flow.  (After export from SLM to IRModule, that is.)


Break dependency cycles.

Merged, thanks @showell!  I think it would be reasonable to rename color_data.js to stream_color_data.js, and stream_color.js to stream_color_ui.js, for clarity.


spark 2 image not downloading

Expected behavior
Actual behavior
i am trying to down load spark2 immage but it is not downloading.
Information about the Issue
i am trying to down load spark2 immage but it is not downloading.
Steps to reproduce the behavior

...
...

The image is 2GB - make sure you are on a stable internet connection.
Try to do a docker pull bigdatauniversity/spark2:latest

no such container404no such id: spark2i receive this error on kinematics.
On Friday, June 24, 2016 3:51 AM, French Ben<EMAIL_ADDRESS>wrote:

The image is 2GB - make sure you are on a stable internet connection.
Try to do a docker pull bigdatauniversity/spark2:latest—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

@aniqurrehman it sounds like the container failed to be created - I would advise you to read the docs on the image as to what needs to be passed to it for the container to be created properly.


Add the new template 'DoubleExcitationUnitary'

Context:
In this PR the quantum circuit to exponentiate the Coupled-Cluster double-excitation operator is proposed to be included in the PennyLane's templates library.
Description of the Change:
The new template is implemented by the function DoubleExcitationUnitary in the file  ./templates/subroutines/double_excitation_unitary.py.
Benefits:
This template is required to build the Unitary Coupled-Cluster Singles and Doubles (UCCSD) ansatz for QChem simulations using VQE.
Possible Drawbacks:
None
Related GitHub Issues:
None

CodeFactor is complaining about the size of the DoubleExcitationUnitary. Perhaps, I will need to split it in smaller functions, e.g., one function for each layer.

too-many-statements (R0915):
Too many statements (%s/%s) Used when a function or method has too many statements. You should then split it in smaller functions / methods.

Sphinx is also failing here:

Traceback (most recent call last):
  File "/entrypoint.py", line 22, in <module>
    action.build_all_docs(github_env, [os.environ.get("INPUT_DOCS-FOLDER")])
  File "/sphinx_action/action.py", line 167, in build_all_docs
    raise RuntimeError("Build failed")
RuntimeError: Build failed

How can I replicate this locally?, so I can check how things render and try to figure out what is going wrong.

CodeFactor is complaining about the size of the DoubleExcitationUnitary. Perhaps, I will need to split it in smaller functions, e.g., one function for each layer and call them from DoubleExcitationUnitary.

Note that this is an optional warning; you can turn it off in if (it is decided that the size of the function is reasonable) with the comment
def func():
    # pylint: disable=too-many-statements
    ...

However, this is normally a sign that a refactor as you suggest would help create a cleaner and smaller function 🙂

Sphinx is also failing here:

This was due to the new pyscf 1.7.2 release. Now that #637 has been merged, the documentation check passes again.

CodeFactor is complaining about the size of the DoubleExcitationUnitary. Perhaps, I will need to split it in smaller functions, e.g., one function for each layer and call them from DoubleExcitationUnitary.

Note that this is an optional warning; you can turn it off in if (it is decided that the size of the function is reasonable) with the comment
def func():
    # pylint: disable=too-many-statements
    ...

However, this is normally a sign that a refactor as you suggest would help create a cleaner and smaller function

Yes, I will think if there is a cleaner way to code it.

Sphinx is also failing here:

This was due to the new pyscf 1.7.2 release. Now that #637 has been merged, the documentation check passes again.

Thanks @josh146. However, I noticed that the merge seems to have broken the test /home/travis/build/XanaduAI/pennylane/tests/test_configuration.py:
 ERROR at setup of TestConfigurationFileInteraction.test_loading_current_directory
Traceback (most recent call last):
  File "/home/travis/build/XanaduAI/pennylane/tests/test_configuration.py", line 81, in default_config_toml
    with open(config_path, "w") as f:
TypeError: invalid file: local('/tmp/pytest-of-travis/pytest-0/test_loading_current_directory0default_config.toml')

What is this test related to?.

Thanks @josh146. However, I noticed that the merge seems to have broken the test /home/travis/build/XanaduAI/pennylane/tests/test_configuration.py:

Interesting, and it's only broken on Python 3.5 🤔 I've just made a commit to see if that fixes it

@agran2018 I've fixed the issue by temporarily skipping the test on Python 3.5. Looks like the tmpdir fixture no longer works on Python 3.5.

@agran2018 I've fixed the issue by temporarily skipping the test on Python 3.5. Looks like the tmpdir fixture no longer works on Python 3.5.

Thank you very much @josh146 !. It has been an "exciting" Friday :-)

Looks good from my end @agran2018, but I didn't check for accuracy in the layers/equations.
I agree that it might make sense to break out the layers as separate utility functions, e.g. _layer1(s, r, q, p).

Thanks @josh146, this is done.

@mariaschuld, I also tried to add this template to the integration test. However, this template needs a minimum of 4 wires and all integration tests are run for wires=2. The same applies for next to come UCCSDansatz. Is there an easy way to create and exception to test these templates?.
Thank you!.

@mariaschuld, I also tried to add this template to the integration test. However, this template needs a minimum of 4 wires and all integration tests are run for wires=2. The same applies for next to come UCCSDansatz. Is there an easy way to create and exception to test these templates?.
Thank you!.

If it's a blocker, it's perfectly acceptable to make manual tests for a new feature

@co9olguy It's a minor change to the tests that will allow for greater flexibility - there was actually no logical reason to restrict templates to 2 wires...

Nice docstring!
I was only wondering if one could summarize the _layer functions to one?
Will try to work on your question with the integration test, haven't foreseen a template that cannot act on 2 wires at all.

Thanks @mariaschuld

I added the template to the integration tests, which required me to make some minor changes to the tests: the FIXTURES can now specify custom wires. Hence the changes to the tests in this PR.

Perfect, thanks!.


[WIP] Add Normalization Support for AutoLinks

Part of #155
Question here: Are we normalizing auto links to the InlineLink representation OR back to their original form.
If we want to go back to the original form, we need to extend the syntax (IMHO)
I am for normalizing to InlineLinks.

Question here: Are we normalizing auto links to the InlineLink representation OR back to their original form.
If we want to go back to the original form, we need to extend the syntax (IMHO)

We should support both (via options). if the link doesn't contain this information, we have to add it so that we can recover from.

I added a InlineLink.IsAutoLink flag to the syntax and added a NormalizeOptions.ExpandAutoLinks option.
The NormalizeAutoLinkRenderer has a customized Accept method checking these two flags and execute if applicable.
Set the default for the option to true (meaning it will expand to the "default" InlineLink representation).
Ready for Review

Thanks, will give a look next week!


Twitter returned : 'has_more_items'

I have using twitterscraper to pull twitter data. Every time it ends with Twitter returned:"has_more_items" even it pulled all the data already. Does anyone else see the same problem?

this is where the result comes from.
has_more_items = json_resp['has_more_items']
if not has_more_items:
logger.info("Twitter returned : 'has_more_items' ")
I think the code above means that it actually got all the data. Am i right?

I met the same output. Although I guess it should be the signal for finishing download the data, the number of output tweets is still smaller than actual number. Does anyone know why and how to refine it?

@zhicheng0501 yes, that log line is misleading. I incorporated a fix  into an existing PR https://github.com/taspinar/twitterscraper/pull/302/commits/8786a5789f8686d4022f14ff44b7ffd86a5abd11