Install older version not working on Windows

What version?
0.12.1
Which command?
proto install
What happened?
I'm trying to install this older version (https://github.com/moonrepo/proto/releases/tag/v0.12.1) and it fails because I'm missing the required argument TOOL
Any logs?
error: the following required arguments were not provided:
TOOL
Usage: proto.exe install TOOL [SEMVER] [-- PASSTHROUGH...]
Operating system?
Windows
Architecture?
x64

@devi-000 What commands are you running exactly to hit this error?

I'm just trying to install this version from here: https://github.com/moonrepo/proto/releases/tag/v0.12.1
After downloading and extracting the files, when I try to run the .\proto.exe install in Windows PowerShell as Administrator, I'm getting this error:
error: the following required arguments were not provided:
  <TOOL>

Usage: proto.exe install <TOOL> [SEMVER] [-- <PASSTHROUGH>...]

Can you please tell me the steps I should follow to install this version?
Thanks!

@devi-000 The install command installs tools, it doesn't install proto itself, if that's what you're trying to do.
To install proto, just drag the proto binary to a directory that's on your PATH. If you already had proto installed, it's ~/.proto/bin.


Windows Defender thinks Pickr is a virus

Do you want to request a feature or report a bug?
bug
What is the current behavior?
I can't download it (1.4.0)
If the current behavior is a bug, please provide the steps to reproduce and create a JSFiddle.

What is the expected behavior?
That I can download it
Your environment:
Version (see Pickr.version): 1.4.0
Used bundle (es5 or normal one): n/a
Used theme (default is classic): n/a
Browser-version:  Chrome (latest)
Operating-system:  Windows 10

Virus found: Trojan:Win32/Sprisky.U!cl
Severity: High
File: C:\Users[username]\Downloads\pickr-1.4.0.zip->pickr-1.4.0/dist/pickr.es5.min.js
Why does this come up?

 Thanks for your credit card information, by the way ;)

Just kidding!
Please see this article: https://www.microsoft.com/en-us/wdsi/threats/malware-encyclopedia-description?Name=Trojan:Win32/Sprisky.U!cl
It seems to be part of Windows Defender's new machine algorithms, meaning that we may see a lot more false positives in the future. Unfortunately, there is little at pickr that we can do to fix this bug in Microsoft's software.
I'd recommend that you turn off Windows Defender temporarily or add a rule to ignore the folder that pickr is downloaded to.
Have a great day!


org.apache.commons:commons-lang3 - J2ObjC continuous build

Java utility classes for the classes that are in java.lang's hierarchy.
@britter, @joehni - FYI for you both. This is to support translation of your Java library to Objective-C such that it can be used on iOS. This bug is specifically for testing this as part of a continuous build using Google's J2ObjC conversion tool.
@advayDev1 FYI

@brunobowden Thanks for the heads up. Just to make this clear: Apache Commons Lang is not owned by @joehni or me. It's a product of the Apache Software Foundation. If you want to get in touch with the community about this, you can post to the dev mailing list.

@britter - fair point. I think the Apache Foundation does a lot of great work. I didn't mean to imply this was somehow "your library". Thanks for all your contributions to it though.

@brunobowden just wanted to make that clear. Awesome that you chose some of the commons libraries for this project. Thank you!

@britter - I had two questions if I may. I realize that this doesn't concern commons-lang3 specifically but I'd appreciate any insights from you. I've been searching around online for both questions and haven't found any
Package Naming
Do you know why there are two commons-io packages? That is: commons-io:commons-io (up to version 2.4) and also org.apache.commons:commons-io (no longer updated, limited to version 1.3.2)? It's confusing to say the least.
https://bintray.com/bintray/jcenter/commons-io:commons-io
https://bintray.com/bintray/jcenter/org.apache.commons:commons-io
test-sources.jar
Some of the packages distributed test source, e.g. commons-io-2.4-test-sources.jar. It seems that the Apache packages are fairly consistent about having a test-sources jar but other packages not so much. Do you know how common this is for Apache packages? Are there any evolving standards on this? We need the test sources to build and run all the unit tests, which are an important part of having confidence that the translated code works correctly.

@britter - I had two questions if I may.

Sure! :-)

Do you know why there are two commons-io packages? That is: commons-io:commons-io (up to version 2.4) and also org.apache.commons:commons-io (no longer updated, limited to version 1.3.2)? It's confusing to say the least.

That's probably for historical reasons. Usually most of the older Apache Commons components started with a schema like commons-name:commons-name for version 1.x. After the move to version 2.0 we started to use the org.apache.commons group ID. Further more we add the major version number to the artifact ID so that you can have two versions of the same component pulled in via maven. That's why Apache Commons Lang's coordinates are org.apache.commons:commons-lang3:3.x
I don't know why it's the other way around for commons-io...

Some of the packages distributed test source, e.g. commons-io-2.4-test-sources.jar. It seems that the Apache packages are fairly consistent about having a test-sources jar but other packages not so much.

The test sources jar contains the test source code. We have that configured somewhere in our parent pom.

Do you know how common this is for Apache packages? Are there any evolving standards on this?

I don't know how common that is among all java based Apache projects. At least we do it for all Apache Commons components. There is no standard for this. I think the default for maven is to just deploy the jar and some meta data files.

We need the test sources to build and run all the unit tests, which are an important part of having confidence that the translated code works correctly.

All Apache projects are open source. If a project does not deploy test sources to maven central, you can always check out the latest code and build everything from scratch. A list of all Apache projects can be found here. Most of them are also mirrored to github.
HTH,
Benedikt

The publishing of org.apache.commons:commons-io:1.3.2 was by accident. Look at the pom of that version and you'll detect that even that one refers commons-io as groupId.

Thanks, that's helpful to understand.


[requirements] Updated rest framework version

Coverage remained the same at 100.0% when pulling 68f665d2e22200d04ba94264d8280799bfacf7c7 on rohithasrk:rest-framework-version into f39724f82b52182eb82bbddb59244c7f4b274177 on netjson:master.

PS: thanks :-)

Will take care. :smiley:


Menu should left or right align, not default left

Version
1.2.0
Overview
This menu is quite handy, except for one flaw- the menu always aligns in one direction- towards the left.  For my use case, I need the menu to align to the right.
Similarly to implementing a left click action instead of a right one, there should be an option to select menu orientation.  There was nothing in the documentation on how this may be implemented.
Steps to reproduce

Integrate menu into a project, any type
Click on button
See menu with left alignment.

Expected Behavior

This meets the expected behavior, however the anticipated behavior is to change the orientation.

Actual Behavior

Works as anticipated.

This behaviour is based on how native context-menu works.
I'm ready to accept a PR with the requested feature.


HSET cannot use - syntax error

Hi bro, i cannot use client.HSET because, syntax error:

client.SET work normal
Environment:

**Node.js 14 LTS
Platform:

It works for me in both VSCode and WebStorm. Please try to upgrade your idea, and make sure that it uses the latest typescript version (4.4.3).


Add integer increment operator "++" to Stan language

Summary:
Add the postfix version of the increment operator "++" to Stan language for integer types, which can be in expressions or as a statement, e.g.:
`int i;
...
i++;

The semantics of the postfix increment operator are:

(make a copy of current value), increment value by 1
returns (the copy of) the pre-increment value

The same result can be gotten from:
i += 1;

https://github.com/stan-dev/stan/issues/1787,
which has been implemented here:
https://github.com/stan-dev/stan/tree/feature/1787-compound-assign-ops-scalars
It's not clear we need this feature as well.
Description:
The "++" operator is useful when incrementing (and decrementing) counters.
Both Java and C++ (hence the name) support this operator.
However R and Python don't.    Java and C++ have both prefix and postfix versions of this operator, with different semantics.
Current Version:
v2.15.0

I asked @mitzimorris to file the issue, but wanted to add what we were discussing with @seantalts:

The operators aren't in either R or Python

The postfix form in particular has a confusing semantics (as noted in the issue).

The prefix form will actually change behavior of --x and ++x, which are equal to -(-(x)) and +(+(x)) and both evaluate to x.  Now they are prefix operators.

The operation is restricted to indexed variables, just like assignments, rather than everything else we have which is restricted to assignments.

The main reason we have n = n + 1 and whatnot in the code is that we are either counting or incremeting some index that isn't tracking the loop indexes.

Closing this issue as this should now be discussed in stanc3.
As I am not sure if this feature is still required and the issue is stale (no action in 4 years) I am going to close it and if its still relevant please reopen and we can move to stanc3 (or just open a new issue in stanc3).


How to get metaData info 'Author' name on 'documentload' eventBus

Attach (recommended) or Link to PDF file here: na
Configuration:

Web browser and its version:
Operating system and its version:
PDF.js version:
Is a browser extension:

Steps to reproduce the problem:
1.
2.
What is the expected behavior? (add screenshot) How to get metaData info 'Author' name on 'documentload' eventBus
What went wrong? (add screenshot)
Link to a viewer (if hosted on a site other than mozilla.github.io/pdf.js or as Firefox/Chrome extension):

To access the metadata, the following API method is used: https://github.com/mozilla/pdf.js/blob/3e34eb31d940f5e8add0af5ee9b262c9f3ad1462/src/display/api.js#L596-L604
If you're using the default viewer, then the metadata will be available trough PDFViewerApplication.metadata once the file is loaded.
Or you can can fetch it yourself, using PDFViewerApplication.pdfDocument.getMetaData() in the default viewer (again once the file has loaded).
Note though that this information isn't directly available through the eventBus (duplicate of issue #5765).
Closing as answered, given that lack of details provided.

thanks!!!


Option to use mousewheel as zoom, not scroll on main modeling window

Problem you would like to solve
I prefer to be able to examine a diagram (including zooming in and out) without the use of the keyboard (using ctrl + mouse wheel to zoom). Using the mouse wheel for panning up/down seems unnecessary given the ease of dragging the screen with left-click and hold.
Proposed solution
Either change the function of the mouse wheel to zoom in/out or provide that as an optional setting.
Alternatives considered
None; minor issue so continuing to use ctrl to zoom.
Additional context
No response

Could you elaborate? You can already use CTRL + Wheel to zoom. This is a standard in most applications.

I acknowledge that I can use CTRL+wheel to zoom. It was a request for an option to not use CTRL with the wheel, but instead use the wheel alone to zoom. Not sure how to elaborate or what additional information is desired. It's a preference not a defect. Zooming in/out isn't something I generally do in Chrome, and the other main drawing tool I use (Miro) uses the wheel for zooming. Thanks.

Thanks for clarifying. I updated the description accordingly.
Google maps style WHEEL only zoom is where we're coming from. But we decided to play nicely with (most) others a long time ago. At some point I could imagine to make this a user setting.


Bounty: validate Digital Public Good submissions

Validate Digital Public Good submissions
Prize Bounty
A special blockchain-based badge, the first ever issued by the Digital Public Goods Alliance! 🙌

Challenge Description
Calling all contributors: The Digital Public Goods Alliance is now crowdsourcing reviews of digital public goods! We’re asking you to participate by reviewing open source projects against the Digital Public Goods Standard with the ultimate goal of determining if a project qualifies as a Digital Public Good.
Why participate?
Open source represents an unprecedented opportunity to fundamentally alter power balances in international development. But, we can’t harness that power without the cooperation of many - reviewers, maintainers, creators, policy makers, and so many others (that means you! 👊).
This community sourcing exercise will give you the opportunity to delve into some of the largest up and coming open source projects. You’ll get a chance to understand their licenses and documentation, and how they’re designing for best practices, standards, privacy and more.
By participating, you’ll get a better understanding of open projects that are making a difference in the world, particularly those that are advancing practical solutions to help achieve the Sustainable Development Goals (SDGs). You’ll also join a growing number of innovators working on technology for development (T4D).
There’s more! We want to show our appreciation for your reviews! We’re working with the UNICEF Office of Innovation’s Blockchain Team to recognize those who contribute with a blockchain-based badge, known as Kudos. This unique badge will be displayed on your Gitcoin profile.
Submission Requirements
Here’s how you can participate:

Go to https://validate.digitalpublicgoods.net
Login with your GitHub credentials (or create a profile, if you don't have one already).
Select an open source project that has been nominated as a Digital Public Good.
Follow the step-by-step instructions to review the project and decide whether or not you believe it meets the criteria to become a is a DPG.
The validate webapp will open a Pull Request on your behalf
To receive your blockchain-badge, submit your Pull Request (PR) to the Gitcoin Bounty (login with your GitHub credentials)
Badges will be allocated on a rolling basis - check back on your Gitcoin profile for your badge!

Judging Criteria
This is a collaborative bounty, where we will accept all valid submissions. Any submission needs to be complete in order to be considered as valid. Submissions will be reviewed and accepted on a rolling basis.
What We Learn When You Participate
The Digital Public Goods Alliance (DPGA) strives to embody the principles of openness and transparency in how we operate. We want to engage people who share in our mission of promoting digital public goods for a more equitable world.
The United Nations Secretary-General’s Roadmap for Digital Cooperation states, “Currently, access to digital solutions is often limited through copyright regimes and proprietary systems. Most existing digital public goods are not easily accessible because they are often unevenly distributed in terms of the language, content and infrastructure required to access them.”
To address these issues, we plan to screen and assess digital public goods that have been nominated to our registry, but are not yet screened against the DPG Standard. We are looking for engagement in this screening process.
Our aim is to build a process that is reliable, scalable, transparent and open. Ultimately, we want our vetting process to fit with our open values and to leverage the expertise of not only our staff, but also the community to help assess these projects.
Desired Outcomes
Our hypothesis is that, by crowdsourcing this screening process, we will arrive at some degree of consensus and therefore be able to move a significant number of projects from “nominated” to “Digital Public Good”. We will be testing this by using confidence ratings in our questions and looking at the overall agreement among reviewers.
We also want to hear from you - is there interest in participating in an activity like this in the future? Are there better ways to engage you to support an effort like this? We encourage you to leave feedback in the comment boxes throughout the process to help us learn and grow.

Hi, I'm an Outreachy applicant and I had just made my first pull request reviewing one project.

Thank you @llbrito, your submission has been well received 👍

The underlying problem you have been experience is now solved. Apologies for the inconvenience


Sometimes the keys d, f and b do not respond as expected.

Running Folie v2.10-7-g05bad42 on W10.
I'm not sure this is a W10 related problem but sometimes the d and f keys do not respond and the b key repositions the cursor back to the beginning of the previous word. Only after restart of Folie it's OK again.
Unfortunately I was unable to figure out when this is happening. The problem usually does not occur.

That's an odd one - sounds as if the software thinks that the CTRL key is pressed, in which case it's definitely a bug in https://github.com/chzyer/readline ...

I'm also using W10 now. Latest Folie release and FTDI connections to 2 BluePills with RFM69HW.
After 1 to 10 minutes folie stops accepting some lowercase characters ( d and f at least. Capital characters keep working all the time so there is no real problem. Cursor movement is strange then and  does not work, backspace does.  These issues did not occur when working in an ssh session on a raspberry running folie.

Thanks for the info. We really need to get to the bottom of this.
Could either one of you try the "mini folie" I mentioned #46? - it uses readline in more mainstream mode, and has none of the extra features (in fact, it's all in a single main.go source file). It may be easier to diagnose and chase this bug when we move all the extras out of the way.

I'll find some time next weekend. Do you use a 1.7 version, then i wiil upgrade first.

Thanks. Yes, I'm on 1.7.5 - but I assume that older versions of Go will work as well.

FYI: i'm testing on go1.7.5 windows/amd64

@doewie  Another approach. I set  my commandwindow in UTF-8  "chcp 65001".It seems  to work. Can you try that too?

Tried  the minifolie today  (go run main.go), same errors. Lower case d was the first character to disappear. Characters b, d, f and g ( if i'm correct)are acting like some cursor commands.

Ditto.  I have the same bug in Win7.  Alt-tab seems to cause it sometimes, not sure, it happens when switching to Sublime.
Lowercase commands and ! goes missing as reported.  Backspace also becomes unreliable.

Please retry with the new Folie v2.11. This is a really weird issue. We need to nail and fix it.

OK, just installed v2.11. Will do some tests coming days.

Is a win86 binary available?
c-:
On 3 May 2017 at 15:57, Douwe Brouwer<EMAIL_ADDRESS>wrote:

OK, just installed v2.11. Will do some tests coming days.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/jeelabs/folie/issues/44#issuecomment-298918595, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ADwpXojnfVpjhmS-eLXhnPbXE7NZ-KN7ks5r2IfOgaJpZM4L3OqZ
.

I think so: "folie-windows.gz" on the Releases page?

Cannot find exe inside?  Sorry...
On 3 May 2017 at 20:51, Jean-Claude Wippler<EMAIL_ADDRESS>wrote:

I think so: "folie-windows.gz" on the Releases
https://github.com/jeelabs/folie/releases page?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/jeelabs/folie/issues/44#issuecomment-299002055, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ADwpXgBAXgwlreuLL6uOzLPY-SMp5ZbXks5r2My-gaJpZM4L3OqZ
.

Whoops! My fault, the files are exe's - please either rename the one you have, or download again - I've now fixed the 32- and 64-bit Windows ZIP files.

Dear Jean-Claude,
My antivirus quarantined the file? Any idea why?
c-:
On 4 May 2017 at 22:08, Jean-Claude Wippler<EMAIL_ADDRESS>wrote:

Whoops! My fault, the files are exe's - please either rename the one you
have, or download again - I've now fixed the 32- and 64-bit Windows ZIP
files.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/jeelabs/folie/issues/44#issuecomment-299294977, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ADwpXvS8C6vQx5DStgh9S3TqOaHlF8u2ks5r2jBWgaJpZM4L3OqZ
.

Found Gen:Variant.Zusy.233235
On 6 May 2017 at 08:55, Cornie Malan<EMAIL_ADDRESS>wrote:

Dear Jean-Claude,
My antivirus quarantined the file? Any idea why?
c-:
On 4 May 2017 at 22:08, Jean-Claude Wippler<EMAIL_ADDRESS>wrote:

Whoops! My fault, the files are exe's - please either rename the one you
have, or download again - I've now fixed the 32- and 64-bit Windows ZIP
files.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/jeelabs/folie/issues/44#issuecomment-299294977, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ADwpXvS8C6vQx5DStgh9S3TqOaHlF8u2ks5r2jBWgaJpZM4L3OqZ
.

Possibly a Bitdefender false alarm... reported at Bitdefender.
On 6 May 2017 at 09:39, Cornie Malan<EMAIL_ADDRESS>wrote:

Found Gen:Variant.Zusy.233235
On 6 May 2017 at 08:55, Cornie Malan<EMAIL_ADDRESS>wrote:

Dear Jean-Claude,
My antivirus quarantined the file? Any idea why?
c-:
On 4 May 2017 at 22:08, Jean-Claude Wippler<EMAIL_ADDRESS>wrote:

Whoops! My fault, the files are exe's - please either rename the one you
have, or download again - I've now fixed the 32- and 64-bit Windows ZIP
files.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/jeelabs/folie/issues/44#issuecomment-299294977, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ADwpXvS8C6vQx5DStgh9S3TqOaHlF8u2ks5r2jBWgaJpZM4L3OqZ
.

New funny:  (are you sure this is not a virus?)
!send xman.fs
open xmem.fs: The system cannot find the file specified.
Send failed.
!send xman.fs
open xmem.fs: The system cannot find the file specified.
Send failed.
On 6 May 2017 at 09:44, Cornie Malan<EMAIL_ADDRESS>wrote:

Possibly a Bitdefender false alarm... reported at Bitdefender.
On 6 May 2017 at 09:39, Cornie Malan<EMAIL_ADDRESS>wrote:

Found Gen:Variant.Zusy.233235
On 6 May 2017 at 08:55, Cornie Malan<EMAIL_ADDRESS>wrote:

Dear Jean-Claude,
My antivirus quarantined the file? Any idea why?
c-:
On 4 May 2017 at 22:08, Jean-Claude Wippler<EMAIL_ADDRESS>wrote:

Whoops! My fault, the files are exe's - please either rename the one
you have, or download again - I've now fixed the 32- and 64-bit Windows ZIP
files.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/jeelabs/folie/issues/44#issuecomment-299294977,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADwpXvS8C6vQx5DStgh9S3TqOaHlF8u2ks5r2jBWgaJpZM4L3OqZ
.

The virus alarm may be due to the fact that Go tends to generate 100% self-contained exe's. I've seen it happen before.
That send failing is as puzzling as the asdf keys not being recognised. Something totally weird seems to be going on here!
Maybe @doewie can shed some light on this...

it's tagged as a virus not just by Bitdefender - https://www.virustotal.com/en/file/5872144f1adbf59597b10c1e7b56a3a969017026cb2cb3c1fe123b159f56b971/analysis/1494061227/

Please delete "New funny:" entry, this was operator brain failure!
c-:
On 6 May 2017 at 11:07, Jean-Claude Wippler<EMAIL_ADDRESS>wrote:

it's tagged as a virus not just by Bitdefender -
https://www.virustotal.com/en/file/5872144f1adbf59597b10c1e7b56a3
a969017026cb2cb3c1fe123b159f56b971/analysis/1494061227/
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/jeelabs/folie/issues/44#issuecomment-299626813, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ADwpXqOqD8S2l5vTS1z3l0HFCJdQS49_ks5r3DhBgaJpZM4L3OqZ
.

You should be able to edit this yourself - click the pencil or cross on this page.

Testing on folie 2.11 the problem still exists.
I'm almost sure it happens after ALT-TAB to another window, and then ALT-TAB back to the Folie screen.
Maybe another Windows-user can confirm this.

Also happens after alt-tab here, but not every time.
On 9 May 2017 at 15:58, Douwe Brouwer<EMAIL_ADDRESS>wrote:

Testing on folie 2.11 the problem still exists.
I'm almost sure it happens after ALT-TAB to another window, and then
ALT-TAB back to the Folie screen.
Maybe another Windows-user can confirm this.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/jeelabs/folie/issues/44#issuecomment-300172870, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ADwpXkSI4y_8NTc3ZZvQ2c6s84TgeYZhks5r4HDqgaJpZM4L3OqZ
.

Bitdefender cleared the false alarm - now testing with 2.11 ...
On 9 May 2017 at 19:57, Cornie Malan<EMAIL_ADDRESS>wrote:

Also happens after alt-tab here, but not every time.
On 9 May 2017 at 15:58, Douwe Brouwer<EMAIL_ADDRESS>wrote:

Testing on folie 2.11 the problem still exists.
I'm almost sure it happens after ALT-TAB to another window, and then
ALT-TAB back to the Folie screen.
Maybe another Windows-user can confirm this.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/jeelabs/folie/issues/44#issuecomment-300172870, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ADwpXkSI4y_8NTc3ZZvQ2c6s84TgeYZhks5r4HDqgaJpZM4L3OqZ
.

Now fails reliably on my system.  Do as follows: Run from shortcut:
\folie.exe -r -p COM38 -b 115200
test by typing...
sdfb sdfb not found.
alt-tab to Chrome
alt-tab back
s s not found
Lower case dfb goes missing with:
f-> move to end of line
b-> move to start of line
d-> delete from here
Hope this helps to solve the issue...
On 13 May 2017 at 12:22, Cornie Malan<EMAIL_ADDRESS>wrote:

Bitdefender cleared the false alarm - now testing with 2.11 ...
On 9 May 2017 at 19:57, Cornie Malan<EMAIL_ADDRESS>wrote:

Also happens after alt-tab here, but not every time.
On 9 May 2017 at 15:58, Douwe Brouwer<EMAIL_ADDRESS>wrote:

Testing on folie 2.11 the problem still exists.
I'm almost sure it happens after ALT-TAB to another window, and then
ALT-TAB back to the Folie screen.
Maybe another Windows-user can confirm this.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/jeelabs/folie/issues/44#issuecomment-300172870, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ADwpXkSI4y_8NTc3ZZvQ2c6s84TgeYZhks5r4HDqgaJpZM4L3OqZ
.

Thanks for chasing this down, great progress.
But I'm totally in the dark about what's going on :( - how on earth can some keys get gobbled. We may have to slice up the app and remove more and more to chase this one.

Seems to me that Folie is thinking a modifier key is currently active. Could you try something?
After switching focus back, press each of shift, ctrl, alt and release them again, then try "s" etc.

 After pressing ALT the f,d and b-keys are back again. So indeed it looks as if Folie thinks the ALT is still active after tabbing out and in again.

We could document this as a edit modifier in the help file.  Thus no change
to the code is required.
:-?
On 13 May 2017 at 16:15, Douwe Brouwer<EMAIL_ADDRESS>wrote:

You're right! After pressing ALT the f,d and b-keys are back again. So
indeed it looks as if Folie thinks the ALT is still active after tabbing
out and in again.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/jeelabs/folie/issues/44#issuecomment-301251021, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ADwpXsRGnfNkzvh-PaPFe_UYlIMNtnfDks5r5bsAgaJpZM4L3OqZ
.


add traditional chinese translation

Hi @TkDodo
I wrote the traditional chinese translation of this article. Please check and accept it.
Thank you and have a nice day 😃

awesome 🚀


The "<" in the  logo of Opportunity section  is missing

Describe the bug
The "<" in the  logo of Opportunity section  is missing.
Expected behavior
It should show the logo  in the nav bar of Opportunity section.
Screenshots

Additional context
Add any other context about the problem here.
Are you participating under any open-source program?
Yes . The name of the program is JWOC.

I want work in this issue under JWOC.

@sayan404 Assigning you this issue. Happy Coding !!


Can't save and quit

What is your Quests version / build number (do not say "latest")?
Quests-4.0.6
Describe the problem and how to replicate it.
Hello
Recently I changed the bukkit from magma to catserver, since magma is unavailble in using the skript plugin.
After the change, I gave one quest a very slight modification.
Its reward was origianally a paper with a name and lore, but I wanted it with a different name, and that was all.
Suddenly, the plugin does not allow the save and quit feature anymore.
All I can do is just exit without saving the changes.
I tried to remove the problematic quest from the yml file and remake it, and it was successful.
I can see the quest is working now in the server.
But the plugin itself is still not functioning. Everything else is fine, but only 'save and quit' doesn't work.
Is there anyone who knows why and how to fix it?
I will be tremendously grateful.
Dorori
Any console errors to share?
There is nothing shown in the console

If applicable, can you provide an example quest from quests.yml?
custom13:
    name: 유리병을 만들자
    ask-message: 일감을 받을까요?
    finish-message: 돈과 경험치를 받았습니다.
    npc-giver-id: 2
    requirements:
      quests:
      - 공방에 취직하자
    stages:
      ordered:
        '1':
          items-to-craft:
          - ==: org.bukkit.inventory.ItemStack
            type: GLASS_BOTTLE
            amount: 30
          start-message: '"유리병을 30개 만들어서 가져다주세요."'
          complete-message: (이제 유리병을 배달합시다.)
        '2':
          items-to-deliver:
          - ==: org.bukkit.inventory.ItemStack
            type: GLASS_BOTTLE
            amount: 30
          npc-delivery-ids:
          - 2
          delivery-messages:
          - 유리병을 배달했습니다.
          complete-message: '"유리가 아주 반짝거리네요! 고맙습니다."'
    rewards:
      items:
      - ==: org.bukkit.inventory.ItemStack
        type: PAPER
        meta:
          ==: ItemMeta
          meta-type: UNSPECIFIC
          display-name: 수표 4000원
          lore:
          - 우클릭 시 수표를 환전합니다.

This is the one that caused the issue.

Unsupported server software.


What about logging Javascript errors?

Hi! Thanks for the component.
https://github.com/hellowearemito/yii2-sentry extension have an option to log JS errors to Sentry instance. But it seems like the project has been abandoned :-(
Could you please add this functionality in your code?

Hi, @maximal.
I'll dig into this, but no promises.

It's easy, just follow the https://docs.sentry.io/platforms/javascript/.


A very interesting job. What is the partitioning method for the dataset? What is the ratio of training set to test set? For example, office_home

What is the ratio of training set to test set? For example, office_home

We just follow the splits from the dassl library.
You can see the ratio when the code running.


Background priority sprites are drawn on top of the background when a sprite is in motion

Describe the bug
When player sprites are set to background priority mode they go in front of the background until they stop moving. Only then do they go behind.
To Reproduce
Steps to reproduce the behavior:
-Make a sprite that has background priority set on all of the sprites.
-Walk over background tiles that are supposed to go in front of the actor.
-Colors 2,3,4 will draw under the actor until it comes to a stop.
 I notice it seems to flash back and forth as you move

You are right. I thought that I had triple-checked this but somehow I still missed some frames.


Remove bluebird dependency

This module depends on both native-or-bluebird and bluebird, but only the former is actually required in code.

this was to keep 0.10 support w/o bumping a major version... i don't mind bumping another version and dropping 0.10 support now since it's so old

meh minor's fine.

It doesn't matter for me but I stumbled across this and found it odd :)

Shouldn't it be a patch version though?

dunno. kind of breaking, kind of a feature, kind of a bug fix. when in doubt, bump minor!

Well native-or-bluebird has was already used, the only change is in package.json ^^


add profiling flag with default value of false

This PR is for the same purpose of #1896 . I missed it in the first PR though.

LGTM


App: Unintended walk/run toggle controls issue on macbooks

An issue appeared while doing QA of a specific PR on MacBook m1, whenever I used to record screen by pressing cmd+shift+f5, the avatar would run when W was pressed without pressing or holding shift.
I tested more of it, if CMD+SHIFT+F5/F4 is pressed, the avatars movement toggles from walk to run. It Gets fixed again when shift is pressed.

I think it's a pretty common problem for web games, this can probably also be reproduced with:

Having webA and a second tab open
Hold shift walk
Click on the second tab while still shift walking
Click on webA tab again

This way there is no onButtonLeave event, and it gets stuck in that state.

I think we can solve it by resetting the key state when we blur. I think this is probably a solvable bug.

@ohmed is this UI related?


cant log into procurement

so i downloaded the newest version of procurement.. the old version worked fine but once i downloaded the new one i cant log in anymore. all it says is...
[Error] The remote server returned an error: (500) Internal Server Error.
please help

This is an error on GGG's end, like the error message says "The remote server returned an error".
If you wait a bit and try again you should be able to login.

It's been like this for 3 days now. I've tried reinstalling too
Sent from my iPhone

On Aug 29, 2015, at 3:15 PM, Stickymaddness<EMAIL_ADDRESS>wrote:
This is an error on GGG's end, like the error message says "The remote server returned an error".
If you wait a bit and try again you should be able to login.
—
Reply to this email directly or view it on GitHub.

It's been like that for days now
Sent from my iPhone

On Aug 29, 2015, at 3:15 PM, Stickymaddness<EMAIL_ADDRESS>wrote:
This is an error on GGG's end, like the error message says "The remote server returned an error".
If you wait a bit and try again you should be able to login.
—
Reply to this email directly or view it on GitHub.

Are you using any kind of proxy server?

Nope. I even tried reinstalling the program too and tried downloaded the older version. Still not working.
Sent from my iPhone

On Aug 30, 2015, at 5:31 AM, Stickymaddness<EMAIL_ADDRESS>wrote:
Are you using any kind of proxy server?
—
Reply to this email directly or view it on GitHub.

I somehow lost track of this issue, however it should be fixed with the server upgrades GGG did. Let me know if it is still an issue.

no it did not fix.. i just started pplaying again and it took me forever to find my id again using chrome search. i found it but it told me to copy it and it;ll be on my procurement app automatically but i cant select copy.. and when i try ctrl c and paste it on the app it doesnt work.. it worked the first time and this popped up...
Error downloading account name, details logged to DebugInfo.log. Please open a ticket at https://github.com/Stickymaddness/Procurement/issues and include your DebugInfo.log
idk what that means... please help!

i had the pre release version at first and now i just downloaded the latest version out. i found my id through chrome, F12, cookies, PoE and POESESSID. it wont let me copy it.. so i highlighted my arrow on it and typed it in... now its saying..
[Error] Incorrect session id
i dont get this thing... it was working fine before then out of no where there was a new version and ever since then it doenst work at all...


fix atEnd

Hi, thanks for your PR.
I'm not sure about that, because the getter atEnd will check on the scroll controller offset. And the scroll controller obvisously point at the beggining of the page.
See the test:
test('Is At End', (){
    controller.scrollToStart();
    expect(controller.atEnd, false);
    controller.scrollToEnd();
    expect(controller.atEnd, true);
});

I'm open to heard more about it if you think it's right.

scrollToEnd scrolls to max, but in reality this doesn't happen. Try manually scrolling to the end and comparing max with offset.

I tested with the example app. I just add this to see the value of atStart and atEnd:
Text('At start: ${controller.atStart}'),
Text('At end: ${controller.atEnd}'),

And I got this:

If I apply your modification, atEnd is always at false.
max is returning this: position.maxScrollExtent, so theoretically, checking offset == max should be right.

Oh, you're right. I had an error in the code. I apologize for wasting your time.

Oh, you're right. I had an error in the code. I apologize for wasting your time.

No worries it's fine ;)


Add Clippy calendar

As Clippy is a sub-team of dev-tools, this adds 2 new toml files: dev-tools.toml and clippy.toml. The dev-tools.toml file just includes the clippy.toml file, as the dev-tools team currently doesn't have a recurring meeting.

Question: In my personal calendar I also have reminders for Clippy to do the Rust<->Clippy repo sync, the release, and beta-backports. Can/Shall those reminders also be included in the team calendars?

Question: In my personal calendar I also have reminders for Clippy to do the Rust<->Clippy repo sync, the release, and beta-backports. Can/Shall those reminders also be included in the team calendars?

I think that's up to you and others who will use the Clippy calendar.


RuntimeError: The GPU device does not support Double (Float64) operations!

When I run torch code on Intel A770, I got the RuntimeError as below
RuntimeError: The GPU device does not support Double (Float64) operations!
Win10 Version: 19045.2788
Python Version: 3.9.16
torch_directml version: <IP_ADDRESS>.dev230301

If you have an Intel GPU, I recommend you to use oneAPI, instead of DirectML.

While printing DML tensor to the console, please copy it back to CPU first.
te_dml.to("cpu")

datatype conversion is happening along the printing code path, and we have detected that your device doesn't support float64.


DOC: Update docstring of PeftModel.from_pretrained

Addresses https://github.com/huggingface/peft/issues/430#issuecomment-1666312815
Reword docstring to not be LoRA-specific

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


[Android] Crash on press push notification

After update from 1.3.13 to 1.3.15 appears this error when press on a push:
Looks like you have configured linking in multiple places. This is likely an error since deep links should only be handled in one place to avoid conflicts. Make sure that:
- You don't have multiple NavigationContainers in the app each with 'linking' enabled
- Only a single instance of the root component is rendered
- You have set 'android:launchMode=singleTask' in the '<activity />' section of the 'AndroidManifest.xml' file to avoid launching multiple instances

I use @react-navigation for deeplink.
I only have one NavigationContainer (and this navigation has linking enabled)
I have set singleTask in my activity

So seems that now Iterable, in the last version, its rendering a new instance of the root component
Can be this change in android-sdk?

Thanks for surfacing this with our team, we will take a look


Run test specs in parallel

Ginkgo has nice built-in support for this. It appears the main challenge for the workflow-e2e tests would be to allow multiple BeforeSuite / AfterSuite calls. Currently that would collide on registering and cancelling the default "admin" user expected by the tests.

closed via #135


remove stale comment on ArrayData equality and update unit tests

Which issue does this PR close?
Closes #.
Rationale for this change
remove stale comment and update unit tests, because equality for ArrayData is implemented
What changes are included in this PR?

Are there any user-facing changes?

Codecov Report

Merging #472 (49b03d1) into master (1f1c637) will decrease coverage by 0.01%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #472      +/-   ##
==========================================
- Coverage   82.65%   82.64%   -0.02%
==========================================
  Files         165      165
  Lines       45563    45537      -26
==========================================
- Hits        37662    37635      -27
- Misses       7901     7902       +1

Impacted Files
Coverage Δ

arrow/src/array/array_struct.rs
87.84% <100.00%> (-0.59%)
:arrow_down:

arrow/src/array/builder.rs
85.78% <100.00%> (-0.13%)
:arrow_down:

parquet/src/encodings/encoding.rs
94.85% <0.00%> (-0.20%)
:arrow_down:

Continue to review full report at Codecov.

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


zsh/bash completion

Completion for command line options

There's no command line options yet, but when there are, they should have completions. :smile:

See also https://github.com/rmarquis/pacaur/issues/290

Added basic completion. It would also be nice to use packages.gz for completing packages with aursync.

Oh my!
On Mon, Apr 25, 2016, 1:57 AM Alad Wenter<EMAIL_ADDRESS>wrote:

Added basic completion. It would also be nice to use packages.gz for
completing packages with aursync.
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/AladW/aurutils/issues/7#issuecomment-214194412

aursync now has context-specific options (depending if -c is specified), so the completion should be updated accordingly.


Remove verified links from app

Describe your suggested feature
The app currently has mangadex.org set as a verified link, as such it bypasses any attempt to open a mangadex link using a different tachiyomi app
Other details
I am on OnePlus's version of Android 12 which unfortunately does not give the end user the option to not let an app not open supported links
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open issue.
[X] I have written a short but informative title.
[X] If this is an issue with an extension, I should be opening an issue in the extensions repository.
[X] I have updated the app to version 1.8.1.
[X] I will fill out all of the requested information in this form.

If neko and the extension remove it I will as well, but I don't see it going anywhere. The link handling on Android 12 is just bad so I personally use https://github.com/ivaniskandar/shouko/ which allows something like how Android previously did it.

which unfortunately does not give the end user the option to not let an app not open supported links

Link handling is done at the extension level, not the app. Android 12 lets users disable link handling for the extension "apps" as before.

Android 12 lets users disable link handling for the extension "apps" as before.

By "disable link handling" do you mean the Open supported links option in an apps settings? Because the latest version of OnePlus does not give users that option, either in the extension "apps" settings page or the actual apps settings page

Can't you just tap the checkbox?

Can't you just tap the checkbox?

Nope, that does absolutely nothing.


Pins helm provider version to 2.1.1

Signed-off-by: Sean Sundberg<EMAIL_ADDRESS>

@csantanapr This is how you pin the version for terraform v12
provider "helm" {
  version = "2.1.1"
}


package.json: lock to node version 10.11.x

ja - this actually didn't change package-lock because i cleverly used the version of node already on my machine.  :-)


android 可以提供x86的 so文件吗

ok

It's a nice project. Any plan on releasing the arm64-v8a version of so file?

It's a nice project. Any plan on releasing the arm64-v8a version of so file?
Requesting also be here:
https://github.com/zeusees/HyperLandmark/issues/51


No authToken when refresh

Hi,
I have a middleware subscribe who check if you've got the right to subscribe in function of your authToken.
In localhost there is no problem, it's working.
But when I go to pre-prod, I doesn't have any authToken when I do a getAuthToken() when i'm refreshing the page.
So I can connect without problem, but when I'm refreshing the page, I don't have the AuthToken on the worker
My client application is hosted on a CDN, and my SocketCluster run on an other server. (they don't have the same DNS)
It looks like socketcluster doesn't send the authToken cookie when reconnecting :
Localhost request cookie :
Cookie:n/app/io=w0_5JT1-W0CYeuFPxWkAAAI; n/app/auth=eyJ0eXAiOiJKV1QiLCJhbGciOiJIUzI1NiJ9.eyJpZCI6IjU1Yzg4NzFlMzBkOTNjNWMyNjZmMDljOCIsImFjY2Vzc1Rva2VuIjoidEdBV3NSMVQ0WUo5NFhUeEFZM24ybE5BUWE4OVlaaWwyQmVQa0hSSGFZeGNkZmx5bWVXUHVIejkyeHZMRmg5diIsInJvbGUiOiJhZG1pbiIsImlhdCI6MTQ0MDY3MDU4NSwiZXhwIjoxNDQwNzU2OTg1fQ.kfx_a6RP6m84zeu12ikB04UHqwTypJLAZO-7BMZFk0Y
Pre prod request cookie :
Cookie:n/app/io=w1_TAqybUFFJbyprj_2AAAD
I don't know if it's an error in my code, or in my server configuration...
Any idea ?
PS : I'm on SC1

@KCypher13 Ah yes, that looks like an issue specific to SC1 which uses cookies to store and transmit the JWT token. I guess that's an issue if the script is loaded from a different host from origin?
This is one of the few areas where SC1 diverges from SC2 - This is low priority though, I have a lot do do. Feel free to submit a pull request if you can fix the issue.

Yeah I load the script from a different host.
I'm on it, I try to add the socket domain to the cookie generated with SCSocket.prototype._setCookie.

I've create a PR on socketCluster Client for this issues :
https://github.com/SocketCluster/socketcluster-client/pull/18


Sensitive data fm

updating SD Desktop instruction and adding troubleshooting guides

to me it seems the table is not showing correctly? see bottom of page: https://csc-guide-preview.rahtiapp.fi/origin/sensitive-data-fm/data/sensitive-data/crypt4gh_client/


DeskBands: No transparency

On Windows 8.1 the deskband is not transparent as it should be.

Closing stale issues to focus on ones with end-user activity.


[rancher-dns?] Multiple links to single service with different aliases result in unexpected answers file.

Multiple links to single service with different aliases result in unexpected answers file.
Could be rancher-dns or metadata.
Rancher versions:
rancher/server: 1.5.10
rancher/agent: 1.2.2
Infrastructure Stack versions:
healthcheck: 0.2.3
ipsec: 0.11.2
network-services: (dns:v0.14.2, network-manager:v0.6.6, metadata:v0.9.1)
scheduler: 0.7.5
Docker version: (docker version,docker info preferred)
17.03.1-ce (can provide docker info if needed)
Operating system and kernel: (cat /etc/os-release, uname -r preferred)
Debian GNU/Linux 8 (jessie) - 3.16.0-4-amd64
Type/provider of hosts: (VirtualBox/Bare-metal/AWS/GCE/DO)
AWS
Setup details: (single node rancher vs. HA rancher, internal DB vs. external DB)
HA
Environment Template: (Cattle/Kubernetes/Swarm/Mesos)
Cattle
Steps to Reproduce:
In a service it's possible to link another service more than once with different aliases.
In this example I linked target from source with aliases one and two:

Unfortunately only the last link appears in the answers file of our dns-container, where we expect both links to show up.
  "[ip-of-source-container]": {
    "search": [
      "multiple-links-dns-issue.rancher.internal",
      "linked.multiple-links-dns-issue.rancher.internal",
      "rancher.internal"
    ],
    "recurse": [],
    "authorative": [],
    "a": {
      "two.multiple-links-dns-issue.rancher.internal.": {
        "answer": [
          "[ip-of-target-container]"
        ]
      },
      "two.rancher.internal.": {
        "answer": [
          "[ip-of-target-container]"
        ]
      }
    },
    "cname": {}
  }

Service links is a map.  The result is one key and the last one wins. And they also only support one target.
If you want multiple targets you need to create an alias service and then point at that.

That workaround will probably work for us, thanks!
I still think this should be considered a bug, due to the possibility of adding multiple links in the frontend to 1 target. This should then also be limited.

I would propose to reopen this issue.
Is the limitation to a map impossible to change?
In my scenario, I used the same service (a load balancer) to override the DNS entries for specific URLs (e.g. in test environments).
Both the UI and API indicate that this should be possible, but when actually making the requests, it will only work for one of the domains.
This can lead to very weird behavior.
Having Service Aliases helps, but is just a workaround, not a valid solution IMHO.

+1 on that; in fact this is very misleading right now.
Either restrict it in the UI and the API or (preferably) make multiple links to the same target possible where it is not, as of today.
As routing does not change, it seems only rancher/rancher-dns must be changed to receive the correct answers.json, it that correct? (Just trying to figure out the impact of this issue and what would be needed to fix it)

This has nothing to do with rancher-dns, the answer format is a map of key (fqdn) to array of records there so this is already supported (and how aliases and service records work).  You are taking about the API for defining links, which is a map of name to single target resource.
The API does not in any way indicate that  multiple is possible.  The type is map, and a map by definition has a unique key.  You cannot store {x: 1, x: 2} in JSON, YAML, or any common programming language and get both values back.
Changing the API is a breaking change for an uncommon use-case.  As I said, the supported way of doing this is an alias service.  Which also means you don't have to maintain the list of targets in multiple places if you want to use it in more than one service.
The UI on the other hand is manipulating an array  internally (because you have to be able to have e.g. multiple empty entries during editing) and  could scan through them all and show an error before it constructs the map sent to the API.

Thank you for taking the time to explain that @vincent99, I really appreciate it!
There is one catch, though: I fear there's a misunderstanding of what we mean vs. what you see as a problem.
Please see this screenshot:

This is, what we (@lauvdk, @EyMaddis and myself) are trying to accomplish - a single target (Destination Service) for multiple fqdns (As Name) - as you've said, exactly what should be supported as of today.
And in fact, this is how the API stores this map:

Multiple keys (per fqdn) link to the very same target.
So I think we can agree that the intended behaviour would now be, that both httpbin1 and httpbin2 can be deleted independently from each other - right?

I would actually like to +1 this.
We are in the process of upgrading from Rancher 1.1.4 to 1.6.10 and when previously we could assign multiple names to a single service within one stack we no longer can. So this is already a broken change.
It's not about balancing one name between multiple containers, but about pointing multiple names to one stack.
In v1.1.4 this worked:

In v.1.6.10 only the line beginning with api resolves correctly:


Docs issue

Installing Docs
winget update Bedrock-OSS.regolith gives Unrecognized command : 'update'. The proper command is upgrade.

Fixed with commit: f2cbee773b187d65880caf46764f89b2a6516ba4


DSC extension co-exists with Log Analytics extension

/*The DSC extension for Linux and the Log Analytics virtual machine extension for Linux currently present a conflict and aren't supported in a side-by-side configuration. Don't use the two solutions together on the same VM. */
Is this sentence is valid comment? This page hasn't been updated in 3 years.

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

ID: a2d49683-d807-3ba0-74df-c0542a7d76ce
Version Independent ID: 92834b58-e207-c7fb-9ead-25b4fa6d9214
Content: Azure DSC extension for Linux - Azure Virtual Machines
Content Source: articles/virtual-machines/extensions/dsc-linux.md
Service: virtual-machines-linux
Sub-service: extensions
GitHub Login: @MGoedtel
Microsoft Alias: magoedte

@MikeSpark
Thanks for the feedback! I have assigned the issue to content author to check and update the document as appropriate.We will update you accordingly.

@MGoedtel
Can you please check and add your comments on this doc update request as applicable

@MikeSpark - Per the product group:

We will be plan to support DSC and Azure monitoring side by side soon officially . They will discuss with their PM team and provided the ETA on this.

As of now few customer are using DSC and Azure monitoring side by side internally . Current Linux DSC extension is using DSC Engine - 1.1.1-926 hence both latest azure monitor agent and latest DSC extension <IP_ADDRESS> can work side by side. Customers can use latest DSC extension and latest Azure monitor side by side in their test environment if they don't see any issue then they can proceed with production. If there is any issue let us know.

Thanks. Good to know.


Update precheck info about MySQL 8.0 support

What is changed, added, or deleted? (Required)

Update info about MySQL 8.0 support
Link to the precheck info in migrate-data-using-dm earlier to make
sure the user has information about required privileges when needed.

Which DM version(s) do your changes apply to? (Required)

[x] master (the latest development version, including v3.0 changes)
[x] v2.0 (TiDB DM 2.0 versions)
[ ] v1.0 (TiDB DM 1.0 versions)

What is the related PR or file link(s)?

This PR is translated from:
Other reference link(s):

Do your changes match any of the following descriptions?

[ ] Delete files
[ ] Change aliases
[ ] Need modification after applied to another branch
[ ] Might cause conflicts after applied to another branch

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

/lgtm

/label needs-cherry-pick-2.0
/translation doing
/status PTAL
/cc @lance6716 @CharLotteiu

/merge

/remove-translation doing
/translation done


Ignore p and p-span components in add at macro script

Fixes #267

@superstructor thanks, I have resolved the issues now. Should be good to go.

Thanks @MawiraIke 👍🏻 😄


Feature spring support

Ⅰ. Describe what this PR did
增加spring-schema的支持,减少对应用的侵入
Ⅱ. Does this pull request fix one issue?

Ⅲ. Why don't you add test cases (unit test/integration test)?
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

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

Codecov Report

Merging #573 into feature_spring_support will decrease coverage by 0.02%.
The diff coverage is 32.53%.

@@                     Coverage Diff                      @@
##             feature_spring_support     #573      +/-   ##
============================================================
- Coverage                     29.68%   29.66%   -0.03%
- Complexity                      626      641      +15
============================================================
  Files                           176      180       +4
  Lines                          6962     7073     +111
  Branches                        848      862      +14
============================================================
+ Hits                           2067     2098      +31
- Misses                         4677     4753      +76
- Partials                        218      222       +4

Impacted Files
Coverage Δ
Complexity Δ

...ing/annotation/GlobalTransactionalInterceptor.java
13.95% <0%> (+0.31%)
2 <0> (ø)
:arrow_down:

...a/fescar/spring/schema/FescarNameSpaceHandler.java
0% <0%> (ø)
0 <0> (?)

...scar/spring/schema/FescarBeanDefinitionParser.java
0% <0%> (ø)
0 <0> (?)

.../fescar/spring/schema/TransationConfigManager.java
54.54% <54.54%> (ø)
9 <9> (?)

...aba/fescar/spring/schema/TransationConfigBean.java
70.58% <70.58%> (ø)
8 <8> (?)

...ar/spring/annotation/GlobalTransactionScanner.java
47.05% <71.42%> (-0.62%)
11 <0> (-1)

...m/alibaba/fescar/spring/annotation/MethodDesc.java
0% <0%> (-40%)
0% <0%> (-1%)

... and 1 more

Continue to review full report at Codecov.

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

@cauchy88 thanks for the work.please sign the CLA first so that we can accept your contributions.

根据标签扫描，fescar:txs,自动注入GlobalTransationScanner,fescar:txs里面的属性即为GlobalTransationScanner的构造函数的四个参数；
1、fescar:tx可以支持0个到多个，0个的时候，视为自动扫描GlobalTransational注解；
2、<fescar:tx ref="beanid" method="test">使用的是JAVA的正则匹配，匹配的是beanid里面test的方法；
3、若ref属性没有匹配，那么，可以使用scan-pakeage属性，默认匹配的是包底下的所有类，对应的method的方法；

i left some comments.

Resolve code conflicts

Due to code conflicts and age, if you are still interested in participating in the community, please resubmit this pr after resolving the conflicts.
由于代码冲突以及年代久远，如果你还有兴趣参与到社区中，请解决完冲突后重新请提交这份pr


Can't past images from clipboard (since v0.27.2)

Describe the bug

When I try to paste images into a book I get an error.
This only became an issue today after updating to BookStack v0.27.2

Steps To Reproduce

Copy an image to clipboard.
Paste into a book note where your cursor is.
This happens on the latest Chrome and Firefox browsers.

Expected behavior

Images used to get pasted and uploaded without issue when using steps like CTRL+V.
Images can still be uploaded the long way by saving an image to desktop, and adding it through the WYSIWYG image button.

  Saves time at least.

Hi @davidtessier,
Thanks for reporting and apologies for this issue.
I've just deployed BookStack v0.27.3 which should solve this issue. Please let me know if it solves the issue for you.

I retested the 0.27.2 version and it seems to be working better. Thank you for a super quick resolution!


Fixed ts errors for drawer + form

Fixed ts errors for drawer + form

command

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

Current dependencies on/for this PR:

main

PR #34

PR #35

PR #41

PR #42   👈

This stack of pull requests is managed by Graphite.


[WiP] - Dynamically import of viz plugins, take 1

CATEGORY
Choose one

[ ] Bug Fix
[ ] Enhancement (new features, refinement)
[ ] Refactor
[ ] Add tests
[ ] Build / Development Environment
[ ] Documentation

SUMMARY
Trying to use dynamic loading to pull in plugins. This seems to work fine with using the npm packages as you see in this code. However, if you switch the package name to a relative path pointing at the plugins /src folder, webpack tries to load it but chokes due to not applying the right loaders. Not quite sure what to do with it at this point.
Also note that for this to work, there will be a prerequisite change in superset-ui since dynamic imports return promises. In Preset.ts you need to change:
    this.plugins.forEach(plugin => {
      plugin.register();
    });

to:
    this.plugins.forEach(plugin => {
      if(plugin instanceof Promise) plugin.then(res => res.register())
      else plugin.register();
    });

then:

cd into /superset-ui/ and yarn build
cd into /superset-ui/packages/superset-ui-core and npm link
cd into incubator-superset/superset-frontend and hitnpm link @superset-ui/core
(re)start your npm dev server as normal.

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TEST PLAN

ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Changes UI
[ ] Requires DB Migration.
[ ] Confirm DB Migration upgrade and downgrade tested.
[ ] Introduces new feature or API
[ ] Removes existing feature or API

REVIEWERS
@suddjian @kristw It sure seems like this ALMOST works, but is apparently beyond my webpack knowhow. If you think we can get webpack loaders to somehow process the contents of the /src folders of plugins, this might make things a LOT easier than dealing with npm link.

After seeing the PR, I added some more thoughts regarding dynamic plugins in SIP-38
for question about high-level motivation and if dynamic import is the right path.
For this PR, itself:
1) The plugins are already using dynamic import under the hood. Adding dynamic import on the entire plugin level may not save much bundle size.
Each ChartPlugin has 5 parts

metadata - very small js object
buildQuery - could be big or small, support dynamic import
Chart - could be big or small, support dynamic import
transformProps - could be big or small, support dynamic import
controlPanel - small js object.

For example
export default class LineChartPlugin extends ChartPlugin {
  constructor() {
    super({
      // metadata does not use dynamic import since it is so small.
      metadata: createMetadata(),
      // this plugin uses dynamic import for the Chart part and will only load it when it is used.
      loadChart: () => import('./LineChart'),
      // this plugins does not use dynamic import for the transformProps function because it is not complex and does not require additional dependencies.
      loadTransformProps: () => import('./transformProps'),
      // this plugin uses dynamic import for the buildQuery part and will only load it when it is used.
      loadBuildQuery: () => import('./buildQuery'),
      // this is the plain object we want to move from incubator-superset
      controlPanel: {...},
    });
  }
}

It is also flexible for opt-out. If the part is so small.
export default class BarChartPlugin extends ChartPlugin {
  constructor() {
    super({
      // metadata does not use dynamic import since it is so small.
      metadata: createMetadata(),
      // this plugins does not use dynamic import for the Chart,
      // which could be because it is small and does not require additional dependencies.
      Chart: BarChart,
      // this plugins does not use dynamic import for the transformProps function
      // which could be because it is small and does not require additional dependencies.
      transformProps,
      // this plugins does not use dynamic import for the buildQuery function
      // which could be because it is small and does not require additional dependencies.
      buildQuery,
      // this is the plain object we want to move from incubator-superset
      controlPanel: {...},
    });
  }
}

2) There are reasons why plugin registration must be synchronous.
What happen during the .register() call is each plugin call the 5 singleton to tell that a visualization with this key exists.
from ChartPlugin.ts
  register() {
    const { key = isRequired('config.key') } = this.config;
    getChartMetadataRegistry().registerValue(key, this.metadata);
    getChartComponentRegistry().registerLoader(key, this.loadChart);
    getChartControlPanelRegistry().registerValue(key, this.controlPanel);
    getChartTransformPropsRegistry().registerLoader(key, this.loadTransformProps);
    if (this.loadBuildQuery) {
      getChartBuildQueryRegistry().registerLoader(key, this.loadBuildQuery);
    }

    return this;
  }

All of these registration happens during the setup phase of the app (see all the files under setup directory). All of these operations are executed before any of the application (react, ui) code is called.
Now in the application code, there are a few places that utilizes these registries to figure out what it should do based on the vizType.

SuperChart uses these singleton registries to load the corresponding chart code.
VizTypeControl.jsx get the list of all vis types registered to the singleton to figure out what visualization to show in the dialog box.
The code for deciding whether to call legacy api or /api/v1/query will need to look up metadata from the ChartMetadataRegistry like

Points taken, and thank you for the insight. The approach in https://github.com/apache/incubator-superset/pull/9326 is fantastic for the time being. I'd hoped that this direction could be one small step toward some other means to load plugins via a manifest of sorts (config file or some other state), but that's a mission for another day.


feat: in-memory Alert storage

Uses ServerSentEventStage to implement a simple in-memory store of the current alerts from the V3 API.
Server Sent Events are parsed into Screens.Alerts.Alert structs and stored in a Map which is kept in the process state of the GenStage consumer.

[!NOTE]
This implementation is limited to serial access because the data is kept in process state. This is a known limitation. I'd like to keep this as-is knowing that I will come back through and swap this for a more parallelizable implementation after filtering logic (#2119) is solid.

Closing in favor of #2121 which uses ETS to allow concurrent access


UnboundLocalError: local variable 'status' referenced before assignment

Hi everybody,
I've just installed instamatic via pip install instamatic
I tried to run this short code:
from instamatic import TEMController
ctrl = TEMController.initialize()

but I got this error:
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "C:\Users\AppData\Local\Programs\Python\Python38-32\lib\site-packages\instamatic\TEMController\TEMController.py", line 66, in initialize
    ctrl = _ctrl = TEMController(tem=tem, cam=cam)
  File "C:\Users\AppData\Local\Programs\Python\Python38-32\lib\site-packages\instamatic\TEMController\TEMController.py", line 121, in __init__
    print(self)
  File "C:\Users\AppData\Local\Programs\Python\Python38-32\lib\site-packages\instamatic\TEMController\TEMController.py", line 125, in __repr__
    return (f'Mode: {self.tem.getFunctionMode()}\n'
  File "C:\Users\AppData\Local\Programs\Python\Python38-32\lib\site-packages\instamatic\TEMController\microscope_client.py", line 92, in wrapper
    return self._eval_dct(dct)
  File "C:\Users\AppData\Local\Programs\Python\Python38-32\lib\site-packages\instamatic\TEMController\microscope_client.py", line 106, in _eval_dct
    if status == 200:
UnboundLocalError: local variable 'status' referenced before assignment

Could you help me to solve it, please?
Thank you
Regards

Hi @rds-itga , yup, this seems like a bug. Thanks for reporting. Should not be hard to fix, I will try to have a look at it today or tomorrow.

Hi @rds-itga , this is a bit of an odd error, meaning that it should not happen. That means something else is not working correctly. Can you tell me a little bit more about your environment and did you do anything else before running the command? Did you change the config? Can you print the full output? Thanks!

Hi,
here are all the steps I followed:

pip install instamatic
Collecting instamatic
  Downloading instamatic-1.6.0-py3-none-any.whl (3.1 MB)
     |████████████████████████████████| 3.1 MB 2.2 MB/s
Collecting pyserialem>=0.3.0
  Downloading pyserialem-0.3.1-py3-none-any.whl (36 kB)
Collecting pywinauto>=0.6.8
  Downloading pywinauto-0.6.8-py2.py3-none-any.whl (362 kB)
     |████████████████████████████████| 362 kB 3.3 MB/s
Requirement already satisfied: scipy>=1.3.2 in c:\users\rds\appdata\local\continuum\anaconda3\envs\openclassroom\lib\site-packages (from instamatic) (1.5.0)
Collecting lmfit>=1.0.0
  Downloading lmfit-1.0.1.tar.gz (258 kB)
     |████████████████████████████████| 258 kB 3.3 MB/s
Requirement already satisfied: pandas>=1.0.0 in c:\users\rds\appdata\local\continuum\anaconda3\envs\openclassroom\lib\site-packages (from instamatic) (1.0.5)
Collecting comtypes>=1.1.7
  Downloading comtypes-1.1.7.zip (180 kB)
     |████████████████████████████████| 180 kB 3.2 MB/s
Collecting mrcfile>=1.1.2
  Downloading mrcfile-1.1.2-py2.py3-none-any.whl (41 kB)
     |████████████████████████████████| 41 kB 92 kB/s
Requirement already satisfied: pyyaml>=5.3 in c:\users\rds\appdata\local\continuum\anaconda3\envs\openclassroom\lib\site-packages (from instamatic) (5.3.1)
Collecting virtualbox>=2.0.0
  Downloading virtualbox-2.0.0-py2.py3-none-any.whl (250 kB)
     |████████████████████████████████| 250 kB 3.3 MB/s
Requirement already satisfied: matplotlib>=3.1.2 in c:\users\rds\appdata\local\continuum\anaconda3\envs\openclassroom\lib\site-packages (from instamatic) (3.2.2)
Collecting h5py>=2.10.0
  Downloading h5py-2.10.0-cp37-cp37m-win_amd64.whl (2.5 MB)
     |████████████████████████████████| 2.5 MB 1.3 MB/s
Collecting scikit-image>=0.17.1
  Downloading scikit_image-0.17.2-cp37-cp37m-win_amd64.whl (11.5 MB)
     |████████████████████████████████| 11.5 MB 3.3 MB/s
Requirement already satisfied: pillow>=7.0.0 in c:\users\rds\appdata\local\continuum\anaconda3\envs\openclassroom\lib\site-packages (from instamatic) (7.2.0)
Requirement already satisfied: ipython>=7.11.1 in c:\users\rds\appdata\local\continuum\anaconda3\envs\openclassroom\lib\site-packages (from instamatic) (7.13.0)
Collecting tqdm>=4.41.1
  Downloading tqdm-4.50.2-py2.py3-none-any.whl (70 kB)
     |████████████████████████████████| 70 kB 1.4 MB/s
Requirement already satisfied: numpy>=1.17.3 in c:\users\rds\appdata\local\continuum\anaconda3\envs\openclassroom\lib\site-packages (from instamatic) (1.18.1)
Collecting tifffile>=2<IP_ADDRESS>
  Downloading tifffile-2020.10.1-py3-none-any.whl (152 kB)
     |████████████████████████████████| 152 kB 6.8 MB/s
Requirement already satisfied: six in c:\users\rds\appdata\local\continuum\anaconda3\envs\openclassroom\lib\site-packages (from pywinauto>=0.6.8->instamatic) (1.14.0)
Requirement already satisfied: pywin32 in c:\users\rds\appdata\local\continuum\anaconda3\envs\openclassroom\lib\site-packages (from pywinauto>=0.6.8->instamatic) (227)
Collecting asteval>=0.9.16
  Downloading asteval-0.9.19.tar.gz (207 kB)
     |████████████████████████████████| 207 kB ...
Collecting uncertainties>=3.0.1
  Downloading uncertainties-3.1.4-py2.py3-none-any.whl (246 kB)
     |████████████████████████████████| 246 kB 3.3 MB/s
Requirement already satisfied: pytz>=2017.2 in c:\users\rds\appdata\local\continuum\anaconda3\envs\openclassroom\lib\site-packages (from pandas>=1.0.0->instamatic) (2019.3)
Requirement already satisfied: python-dateutil>=2.6.1 in c:\users\rds\appdata\local\continuum\anaconda3\envs\openclassroom\lib\site-packages (from pandas>=1.0.0->instamatic) (2.8.1)
Requirement already satisfied: pyparsing!=2.0.4,!=2.1.2,!=2.1.6,>=2.0.1 in c:\users\rds\appdata\local\continuum\anaconda3\envs\openclassroom\lib\site-packages (from matplotlib>=3.1.2->instamatic) (2.4.6)
Requirement already satisfied: cycler>=0.10 in c:\users\rds\appdata\local\continuum\anaconda3\envs\openclassroom\lib\site-packages (from matplotlib>=3.1.2->instamatic) (0.10.0)
Requirement already satisfied: kiwisolver>=1.0.1 in c:\users\rds\appdata\local\continuum\anaconda3\envs\openclassroom\lib\site-packages (from matplotlib>=3.1.2->instamatic) (1.2.0)
Collecting imageio>=2.3.0
  Downloading imageio-2.9.0-py3-none-any.whl (3.3 MB)
     |████████████████████████████████| 3.3 MB 6.8 MB/s
Collecting networkx>=2.0
  Downloading networkx-2.5-py3-none-any.whl (1.6 MB)
     |████████████████████████████████| 1.6 MB 6.4 MB/s
Collecting PyWavelets>=1.1.1
  Downloading PyWavelets-1.1.1-cp37-cp37m-win_amd64.whl (4.2 MB)
     |████████████████████████████████| 4.2 MB 3.3 MB/s
Requirement already satisfied: pickleshare in c:\users\rds\appdata\local\continuum\anaconda3\envs\openclassroom\lib\site-packages (from ipython>=7.11.1->instamatic) (0.7.5)
Requirement already satisfied: traitlets>=4.2 in c:\users\rds\appdata\local\continuum\anaconda3\envs\openclassroom\lib\site-packages (from ipython>=7.11.1->instamatic) (4.3.3)
Requirement already satisfied: decorator in c:\users\rds\appdata\local\continuum\anaconda3\envs\openclassroom\lib\site-packages (from ipython>=7.11.1->instamatic) (4.4.2)
Requirement already satisfied: jedi>=0.10 in c:\users\rds\appdata\local\continuum\anaconda3\envs\openclassroom\lib\site-packages (from ipython>=7.11.1->instamatic) (0.15.2)
Requirement already satisfied: prompt-toolkit!=3.0.0,!=3.0.1,<3.1.0,>=2.0.0 in c:\users\rds\appdata\local\continuum\anaconda3\envs\openclassroom\lib\site-packages (from ipython>=7.11.1->instamatic) (3.0.4)
Requirement already satisfied: pygments in c:\users\rds\appdata\local\continuum\anaconda3\envs\openclassroom\lib\site-packages (from ipython>=7.11.1->instamatic) (2.6.1)
Requirement already satisfied: backcall in c:\users\rds\appdata\local\continuum\anaconda3\envs\openclassroom\lib\site-packages (from ipython>=7.11.1->instamatic) (0.1.0)
Requirement already satisfied: colorama; sys_platform == "win32" in c:\users\rds\appdata\local\continuum\anaconda3\envs\openclassroom\lib\site-packages (from ipython>=7.11.1->instamatic) (0.4.3)
Requirement already satisfied: setuptools>=18.5 in c:\users\rds\appdata\local\continuum\anaconda3\envs\openclassroom\lib\site-packages (from ipython>=7.11.1->instamatic) (46.1.3.post20200330)
Requirement already satisfied: future in c:\users\rds\appdata\local\continuum\anaconda3\envs\openclassroom\lib\site-packages (from uncertainties>=3.0.1->lmfit>=1.0.0->instamatic) (0.18.2)
Requirement already satisfied: ipython-genutils in c:\users\rds\appdata\local\continuum\anaconda3\envs\openclassroom\lib\site-packages (from traitlets>=4.2->ipython>=7.11.1->instamatic) (0.2.0)
Requirement already satisfied: parso>=0.5.2 in c:\users\rds\appdata\local\continuum\anaconda3\envs\openclassroom\lib\site-packages (from jedi>=0.10->ipython>=7.11.1->instamatic) (0.5.2)
Requirement already satisfied: wcwidth in c:\users\rds\appdata\local\continuum\anaconda3\envs\openclassroom\lib\site-packages (from prompt-toolkit!=3.0.0,!=3.0.1,<3.1.0,>=2.0.0->ipython>=7.11.1->instamatic) (0.1.9)
Building wheels for collected packages: lmfit, comtypes, asteval
  Building wheel for lmfit (setup.py) ... done
  Created wheel for lmfit: filename=lmfit-1.0.1-py3-none-any.whl size=82005 sha256=767f50cf5b8469f6ea55d7f9da1d6f959583740e31e6d4659d4e9be0716c380a
  Stored in directory: c:\users\rds\appdata\local\pip\cache\wheels\d8\b1\40\a92e2a410f0e836855bb3c077e2ee8e6c93adf1898d69aff8c
  Building wheel for comtypes (setup.py) ... done
  Created wheel for comtypes: filename=comtypes-1.1.7-py3-none-any.whl size=164608 sha256=d0fe8ff326ddd9d60787dfde33cb779bfde7fb4ba708f1ebb7f1b9dcc0c39d45
  Stored in directory: c:\users\rds\appdata\local\pip\cache\wheels\01\6c\bd\be8ac2d1cf71f2231e4d7d692c12235699259c8358ce624092
  Building wheel for asteval (setup.py) ... done
  Created wheel for asteval: filename=asteval-0.9.19-py3-none-any.whl size=17499 sha256=f0bd38635f17db8af8192eff6b9121e154e16732aaec52641d682e73076f4258
  Stored in directory: c:\users\rds\appdata\local\pip\cache\wheels\a6\a4\52\305cdda6cd84c39736fc2cd5ead2985242956dc040a256a74b
Successfully built lmfit comtypes asteval
Installing collected packages: tqdm, imageio, tifffile, networkx, PyWavelets, scikit-image, mrcfile, asteval, uncertainties, lmfit, pyserialem, comtypes, pywinauto, virtualbox, h5py, instamatic
Successfully installed PyWavelets-1.1.1 asteval-0.9.19 comtypes-1.1.7 h5py-2.10.0 imageio-2.9.0 instamatic-1.6.0 lmfit-1.0.1 mrcfile-1.1.2 networkx-2.5 pyserialem-0.3.1 pywinauto-0.6.8 scikit-image-0.17.2 tifffile-2020.10.1 tqdm-4.50.2 uncertainties-3.1.4 virtualbox-2.0.0

(OpenClassroom) C:\Users\rds>python
Python 3.7.7 (default, Mar 23 2020, 23:19:08) [MSC v.1916 64 bit (AMD64)] :: Anaconda, Inc. on win32
Type "help", "copyright", "credits" or "license" for more information.
>>> from instamatic import TEMController
No config directory found, creating new one in C:\Users\rds\AppData\Roaming\instamatic
Configuration directory has been initialized.
Directory: C:\Users\rds\AppData\Roaming\instamatic
Please review and restart the program.

(OpenClassroom) C:\Users\rds>python
Python 3.7.7 (default, Mar 23 2020, 23:19:08) [MSC v.1916 64 bit (AMD64)] :: Anaconda, Inc. on win32
Type "help", "copyright", "credits" or "license" for more information.
>>> from instamatic import TEMController
Config directory: C:\Users\rds\AppData\Roaming\instamatic\config
>>> ctrl = TEMController.initialize()
Microscope: simulate (server)
Connected to TEM server (localhost:8088)
Camera    : simulate (stream)

Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "C:\Users\rds\AppData\Local\Continuum\anaconda3\envs\OpenClassroom\lib\site-packages\instamatic\TEMController\TEMController.py", line 66, in initialize
    ctrl = _ctrl = TEMController(tem=tem, cam=cam)
  File "C:\Users\rds\AppData\Local\Continuum\anaconda3\envs\OpenClassroom\lib\site-packages\instamatic\TEMController\TEMController.py", line 121, in __init__
    print(self)
  File "C:\Users\rds\AppData\Local\Continuum\anaconda3\envs\OpenClassroom\lib\site-packages\instamatic\TEMController\TEMController.py", line 125, in __repr__
    return (f'Mode: {self.tem.getFunctionMode()}\n'
  File "C:\Users\rds\AppData\Local\Continuum\anaconda3\envs\OpenClassroom\lib\site-packages\instamatic\TEMController\microscope_client.py", line 92, in wrapper
    return self._eval_dct(dct)
  File "C:\Users\rds\AppData\Local\Continuum\anaconda3\envs\OpenClassroom\lib\site-packages\instamatic\TEMController\microscope_client.py", line 106, in _eval_dct
    if status == 200:
UnboundLocalError: local variable 'status' referenced before assignment
>>>

Regards

defaults.yaml
gridmontage:
  gridspec:
    direction: 'downup'
    zigzag: true
    flip: true
  overlap: 0.1
montage:
  from_serialem_mrc:
    direction: 'downup'
    zigzag: true
    flip: true
    rot90: 3
    flipud: true
    fliplr: false
predicrystal:
  location: 'C:\predicrystal'
  classifier: holey
  filter_distance: 2.0

settings.yaml
microscope: simulate
camera: simulate
calibration: simulate

# Global toggle to force simulated camera/microscope interface
simulate: False

data_directory: C:/instamatic
#flatfield: C:/instamatic/flatfield.tiff
flatfield:

# Run the TEM connection in a different process (recommended)
use_tem_server: True
tem_server_host: 'localhost'
tem_server_port: 8088
tem_require_admin: False
tem_communication_protocol: 'pickle'  # pickle, json, msgpack, yaml

# Run the Camera connection in a different process
use_cam_server: False
cam_server_host: 'localhost'
cam_server_port: 8087
cam_use_shared_memory: true

# Submit collected data to an indexing server (CRED only)
use_indexing_server_exe: False
indexing_server_exe: 'instamatic.dialsserver.exe'
indexing_server_host: 'localhost'
indexing_server_port: 8089
dials_script: 'E:/cctbx/dials_script.bat'

# JEOL only, automatically set the rotation speed via Goniotool (instamatic.goniotool)
use_goniotool: False
goniotool_server_host: 'localhost'
goniotool_server_port: 8090

# For InsteaDMatic to control the rotation speed on a FEI/TFS system
fei_server_host: '<IP_ADDRESS>'
fei_server_port: 8091

# Automatically submit the data to an indexing server running in a VM (VirtualBox)
use_VM_server_exe: False
VM_server_exe: 'instamatic.VMserver.exe'
VM_server_host: 'localhost'
VM_server_port: 8092
VM_ID: "Ubuntu 14.04.3"
VM_USERNAME: "lab6"
VM_PWD: "testtest"
VM_STARTUP_DELAY: 50
VM_DESKTOP_DELAY: 20
VM_SHARED_FOLDER: F:\SharedWithVM

# Testing variables
cred_relax_beam_before_experiment: false
cred_track_stage_positions: false

# Here the panels for the GUI can be turned on/off/reordered
modules:
  - 'cred'
  - 'cred_tvips'
  - 'cred_fei'
  - 'sed'
  - 'autocred'
  - 'red'
  - 'machine_learning'
  - 'ctrl'
  - 'debug'
  - 'about'
  - 'console'
  - 'io'

Good stuff. I can fix the error quite easily, but it probably will not address the underlying issue.
You can try:
use_tem_server: False

This will avoid the issue altogether by running the interface in the same process (not ideal, but OK for scripting work).
Alternatively, can you try to first run:
instamatic.temserver

And then in a different tab, do the thing from before:
from instamatic import TEMController
ctrl = TEMController.initialize()

Hi, with
use_tem_server: False
it seems to work, here is what the the terminal returned to me:
Python 3.7.7 (default, Mar 23 2020, 23:19:08) [MSC v.1916 64 bit (AMD64)] :: Anaconda, Inc. on win32
Type "help", "copyright", "credits" or "license" for more information.
>>> from instamatic import TEMController
Config directory: C:\Users\rds\AppData\Roaming\instamatic\config
>>>
>>> ctrl = TEMController.initialize()
Microscope: simulate
Camera    : simulate (stream)

Mode: mag1
High tension: 200 kV
Current density: 101870.12 pA/cm2
GunShift(x=3770, y=58589)
GunTilt(x=63225, y=43005)
BeamShift(x=46346, y=40527)
BeamTilt(x=62627, y=64505)
ImageShift1(x=45491, y=29424)
ImageShift2(x=15383, y=27640)
DiffShift(x=44088, y=41496)
Stage(x=32280.0, y=-15346.0, z=148.0, a=6.0, b=-36.0)
Magnification(value=25000, index=10)
DiffFocus(value=n/a)
Brightness(value=12545)
SpotSize(1)
Saved alignments: ('neutral.yaml',)```

Thank you very much for your help and your reactivity. :)

Regards

No worries!

Hi again,
I just changed default tem_server_port 8088 into 8087 (as i don't use a cam_server) and now it is working in tem_server mode.
I don't know why it does not work with port 8088, maybe another application is using it on my machine but how to know it
Finally here is the config file i used and which is working
microscope: simulate
camera: simulate
calibration: simulate

# Global toggle to force simulated camera/microscope interface
simulate: false

data_directory: C:\instamatic
#flatfield: C:/instamatic/flatfield.tiff
flatfield:

# Run the TEM connection in a different process (recommended)
use_tem_server: true
tem_server_host: 'localhost'
tem_server_port: 8087
tem_require_admin: false
tem_communication_protocol: 'pickle'  # pickle, json, msgpack, yaml

# Run the Camera connection in a different process
use_cam_server: false
cam_server_host: 'localhost'
cam_server_port: 8087
cam_use_shared_memory: true

# Submit collected data to an indexing server (CRED only)
use_indexing_server_exe: false
indexing_server_exe: 'instamatic.dialsserver.exe'
indexing_server_host: 'localhost'
indexing_server_port: 8089
dials_script: 'E:/cctbx/dials_script.bat'

# JEOL only, automatically set the rotation speed via Goniotool (instamatic.goniotool)
use_goniotool: false
goniotool_server_host: 'localhost'
goniotool_server_port: 8090

# For InsteaDMatic to control the rotation speed on a FEI/TFS system
fei_server_host: '<IP_ADDRESS>'
fei_server_port: 8091

# Automatically submit the data to an indexing server running in a VM (VirtualBox)
use_VM_server_exe: false
VM_server_exe: 'instamatic.VMserver.exe'
VM_server_host: 'localhost'
VM_server_port: 8092
VM_ID: "Ubuntu 14.04.3"
VM_USERNAME: "lab6"
VM_PWD: "testtest"
VM_STARTUP_DELAY: 50
VM_DESKTOP_DELAY: 20
VM_SHARED_FOLDER: F:\SharedWithVM

# Testing variables
cred_relax_beam_before_experiment: false
cred_track_stage_positions: false

# Here the panels for the GUI can be turned on/off/reordered
modules:
  - 'cred'
  - 'cred_tvips'
  - 'cred_fei'
  - 'sed'
  - 'autocred'
  - 'red'
  - 'machine_learning'
  - 'ctrl'
  - 'debug'
  - 'about'
  - 'console'
  - 'io'

Regards

Hi @rds-itga , nice debugging there! Thanks for letting me know. The choice of port is somewhat abritrary, but it is indeed possible that it was already in use (or maybe some hanging process). instamatic will try to connect to a port if it sees that it is open, but it does not check what it connects too 😅


publish 0.8.1

Codecov Report

Merging #22 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##           master    #22   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           7      7
  Lines          80     80
=====================================
  Hits           80     80

Continue to review full report at Codecov.

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


Update branch to be up to date with CUPiD/main

Hey @nusbaume , Can we merge these so that the adf_gen_timeseries branch is up to date and that we can use the --timeseries flag and update related code that got pulled in recently?
If it's easier, I can also just copy the timeseries.py file over and make a new PR from my own (or a CUPiD-owned) branch, but I figured this may be better for the purpose of tracking the full process.

No worries!  Just in case I have invited both you and @mnlevy1981 to be collaborators on my fork in case there is something on my fork that you'll need to push in the future (although hopefully not if we move to only branches).


Add Variable and Argument placeholders.

:tada: This issue has been resolved in version 5.0.0-rc.2 :tada:
The release is available on:

5.0.0-rc.2
GitHub release

Your semantic-release bot :package::rocket:


Blurry display on mobile phone

The display on the mobile phone is obviously blurrier than other frames, and the moiré pattern is serious. Is there any solution?

The first picture is other frameworks, the second picture is gsplat.js


Upgrade google-api-client dependency to fix Ruby 1.9.x compatibility

Retriable was upgraded today (Jan 30th) and they shipped a version 2.0.0 that
requires Ruby >=2.0.0.
In google-api-client 0.7.1, we were pulling in version >= 1.4 of retriable. In 0.8.2
they have a fix that pulls in ~> 1.4 version of retriable, thus pinning the ceiling
on the upgrade to retriable 2.0.0.
You can see the failure here: https://jenkins.puppetlabs.com/job/platform_puppetdb_intn-sys_pr/506/BEAKER_CONFIG=ec2-west-el7-64mda-el7-64a,BEAKER_OPTIONS=postgres,label=beaker-ec2/console
Signed-off-by: Ken Barber<EMAIL_ADDRESS>

Might possibly be better solved with a pin against retriable, but I thought upgrading google-api-client would be a better longer term solution.

:red_circle: Test failed.
Refer to this link for build results: http://jenkins-beaker.delivery.puppetlabs.net/job/Beaker Combined Smoketest/451/

:red_circle: Test failed.
Refer to this link for build results: http://jenkins-beaker.delivery.puppetlabs.net/job/Beaker Combined Smoketest/450/

retest this please

:red_circle: Test failed.
Refer to this link for build results: http://jenkins-beaker.delivery.puppetlabs.net/job/Beaker Combined Smoketest/452/

error in puppetdb smoketests:
Fetching gem metadata from https://rubygems.org/......
Resolving dependencies...
Bundler could not find compatible versions for gem "activesupport":
  In Gemfile:
    beaker (>= 0) ruby depends on
      activesupport (>= 3.2) ruby
     activerecord (= 2.3.14) ruby depends on
     activesupport (2.3.14)

@anodelman we don't maintain these smoketests, and they differ dramatically from ours, I've told you this before ... if you differ from us you have to maintain these yourselves and sort out the errors. I'm getting tired of managing the differences in your testing, while you just ignore my advice to make sure they match exactly.
But anyway ... just do a --without test when you install the gems, like our tests do. That skips the stuff required by rspec. If you can't take responsibility for tests that differ from ours, because you choose to make them differ, I can't keep helping you with the differences:
https://github.com/puppetlabs/puppetdb/blob/master/ext/jenkins/beaker-tests-source.sh
Sort out the differences, and stop making it my problem please.

@anodelman I am deeply sorry about my last comment. It was uncalled for, and cruel and I hope you can forgive me. @djm68 told me off for it, and he was right. I don't mean to be callous, I just lose my cool sometimes, and I am sincerely sorry for treating a fellow colleague like yourself like this. We're all busy and working our hardest, and no one deserves to be snapped at like that. I won't say I won't do it again, because I know my stupid nature, but I hope you understand now I've thought about it, I realise it was wrong and uncalled for and I deeply regret it and will work hard to not be so harsh going forward. Please, your important to us all, and I'm sorry for treating you less than that.
Anyway ... as far as the problem. I'm not quite sure why this wasn't noticed before, but my best advice would be that you should avoid the 'test' group in our Gemfile. This contains rspec specific stuff, and in this case ActiveRecord which is required by our rspec tests, since we still test against the old ActiveRecord storeconfigs code in Puppet. This is deprecated, in fact its the 'old puppetdb' way, but we have a migration tool we still support that needs it. It will go away in Puppet 4.0.0, since legacy storeconfigs (ie. ActiveRecord) will finally be retired. Kind of a marking point in Puppet history also, this means PuppetDB is the only way left to do it.
We hit this earlier ourselves by the way, you can see our workaround here:
https://github.com/puppetlabs/puppetdb/blob/master/ext/jenkins/beaker-tests-source.sh#L33
By doing this, it avoids the 'test' group, and the problem with ActiveRecord 'disappears'. Which is good enough for our purposes (fwiw, I also do the reverse for running unit tests, that is I skip the acceptance group because beaker pulls in a lot of stuff). This way I can re-use 1 Gemfile for both test types basically, without pulling in croft I don't really need.
So I'm not sure what you're doing on the internal jenkins server (I don't have admin, so can't see the script) but consider switching to calling this script:
https://github.com/puppetlabs/puppetdb/blob/master/ext/jenkins/beaker-tests-source.sh
I recall you doing something special with images and using vsphere versus ec2, I don't know how to handle that in a standard way, but I wouldn't mind 1 place to manage this kind of thing. Although there is a lot to be said about using EC2 for smoke testing also, since that actually reflects our reality today, and I think we could pick up lots of bugs in the EC2 provider this way. Of course @waynr has a lot of ideas about test reproducibility as well, but is less of a fan of the 'checked in shell script', but he's doing the research for us in this arena, so I'd find out how we can make sure these kind of differences don't create differences between our jobs and this smoke test. That is, I trust he'll do something cool & creative that works.
Hope this helps :-).

I'll look over the smoketest based on your comments and see what I can get going.

We've been running with
bundle install --path vendor/bundle --without=test
Since early in January when I last looked these tests over.  I'll dig around some more and see what I can find.

Okay, now I'm confused.  Did you get this working with the --without?  The bundle man page tells me:
{noformat}
While the --without option will skip installing the gems in the specified groups, it will still download those gems and use them to resolve the dependencies of every gem in your Gemfile(5).
{noformat}
I can create a successful install if I comment out the 'test' group, so that is where the dependency conflict is being generated - but I don't see how --without will repair that since bundler still does dependency resolution using gems that are within an excluded group.

@anodelman well then I have little idea, besides jumping onto the box that is showing the problem and poking around. I certainly can't replicate it, our tests run many times a day without issue, so not sure.

Nertz.  I can reproduce on my laptop env, but that doesn't explain why you are getting green runs.  I'll poke around more and compare the two different jobs and see what I can find.

I still haven't found the difference, but the failure is from:
gem 'activerecord', '2.3.14'

In the puppetdb Gemfile.  Do you need to pin activerecord here?  Could you update the pin?  If you either dropped the pin or updated it I think that we could get this worked out.

Yes, we do need the pin for Puppet related testing.

Okay, I know what is throwing us ... its google-api-client 0.8 that requires the latest activesupport:
Bundler could not find compatible versions for gem "activesupport":
  In Gemfile:
    beaker (>= 0) ruby depends on
      google-api-client (~> 0.8) ruby depends on
        activesupport (>= 3.2) ruby

    activerecord (= 2.3.14) ruby depends on
      activesupport (2.3.14)

For some reason that original failure didn't show google-api-client in the list so I thought this was a general problem with your tests, now I understand its just a conflict between the two.

@anodelman so going back to your original idea the pinning will be fine, but only for activerecord 3.2. Anything newer than that, Puppet will fail. Our PR is here: https://github.com/puppetlabs/puppetdb/pull/1250. We'll see if it passes CI.

Thanks for your work here @kbarber!  Nobody likes being stuck in dependency hell.

retest this please

:red_circle: Test failed.
Refer to this link for build results: http://jenkins-beaker.delivery.puppetlabs.net/job/Beaker Combined Smoketest/485/

Looks like the puppetdb fix has been applied to stable, but not master.  Currently, beaker runs against puppetdb master.  I'll see what I can do.

This is pending repair of the puppetdb smoketest.  Working with @waynr to diagnose.

Puppetdb now green.

retest this please

Refer to this link for build results (access rights to CI server needed):
http://jenkins-beaker.delivery.puppetlabs.net//job/Beaker Combined Smoketest/522/
:green_heart: Test passed.

Final nit - please mark your commit with (MAINT) or associate with a bug number.

Moved to https://github.com/puppetlabs/beaker/pull/726 to meet contributor guidelines.


fwhm and line depth changes

Fixed to python 3 syntax and changed out deprecated mlab functions
NOTE: still issue in competently running fwhm

Added the two wrapper packages, antools3, and 2 folders of .fits files to test the wrappers on

antools3.py will be accepted now, but will be simplified to include only the code needed to run the example.


Modified the cross_align attribute of the layout, to fix the first element in the list, taking up entire window height

Fixing Issue #1836

In the eframe::egui::Layout struct, in the functions right_to_left and left_to_right, I have modified the default value of cross_align attribute to Align::LEFT.
To keep the examples intact, where cross_align is not explicitly set using with_cross_align, I have added an explicit call to with_cross_align() method with Align::Center as the parameter value.

This is a scary change, as it is a breaking change.
The current value is intentional. Say you allocate a menu bar (maybe 24 pixels high) and you use right_to_left in it. In this case you likely want the widgets you add to it to be centered. For instance, adding a label and a button of different height it looks best if they are vertically centered. This is also the behavior of ui.horizontal.
Perhaps this can be solved by adding better helpers to the Ui interface instead? There is currently ui.horizontal and ui.horizontal_top - perhaps adding ui.right_to_left and ui.right_to_left_top would scratch your itch without adding a breaking change?


Added to participant

#1 referenced!

Are there any updates or should we close this PR as spam?

spam


Where can I find source of "Gratuitous Animation App" referred in UIExplorer?

--- Please use this template, and delete everything above this line before submitting your issue ---
Description
I built UIExplorer from master "7e2d7f0" and I see a reference to "Gratuitous Animation App" under "Animated" section, but i could not find it in /examples folder.
I would like to have the source or link to github repo so I can test and learn.
Reproduction
Build UIExplorer from master and check the "Animated" section.

Solution
Upload or provide link to correct github repo?
Additional Information

React Native version: react-native-cli: 1.0.0 & react-native@latest
Platform: Android
Operating System: Mac OS

https://github.com/facebook/react-native/tree/master/Examples/UIExplorer/js/AnimatedGratuitousApp

@ericvicenti Why is AnimatedGratuitousApp included in iOS but not Android UIExplorer bundles?


Having an empty line before the first # gives an IndexError

In the function:
def title(md: str) -> str:
    for line in md.splitlines():
        if line[0] == "#":  # IndexError if line is empty
            return line.strip("#").strip()
    raise ValueError("Cannot find any lines that look like markdown headings")

If there is an empty line in md, that line is just an empty string in the list returned by md.splitlines(). Accessing the empty string gives the IndexError.
Checking truthfulness on the line before accessing line[0] should be enough:
        if line and line[0] == "#":  # Check that line is truthy first

May I do a Pull Request if the issue and the resolution proposed is satisfactory?

Good catch. Your fix looks good. PR welcome!


Null PID error might be caused by sleep mode?

Please describe what you were doing when this error happened.
Specifications

OS: win32
IPFS Desktop Version: 0.13.2
Electron Version: 9.3.2
Chrome Version: 83.0.4103.122

Error
TypeError: Cannot read property 'pid' of null
    at Timeout._onTimeout (C:\Program Files\IPFS Desktop\resources\app.asar\node_modules\ipfsd-ctl\src\ipfsd-daemon.js:242:101)
    at listOnTimeout (internal/timers.js:531:17)
    at processTimers (internal/timers.js:475:7)

Thank you, @Madeindreams ! We'll look into this in our next weekly triage.

Note: duplicate here https://github.com/ipfs-shipyard/ipfs-webui/issues/1670

Note: another duplicate here https://github.com/ipfs-shipyard/ipfs-desktop/issues/1706
OS: darwin
IPFS Desktop Version: 0.13.2
Electron Version: 9.3.2
Chrome Version: 83.0.4103.122

Closing due to moving this into larger issue investigating Cannot read property 'pid' of null errors: https://github.com/ipfs-shipyard/ipfs-desktop/issues/1713


improve issue template

Motivation
Improve the ISSUE_TEMPLATE and allow standard between projects.
What
Add two questions in the issue template
Why

Ask for the users to add the Expected Behavior and Actual Behavior will be helpful to have a better understanding of the issues tracked.
Standard of templates across the org

Ignore for now.