Readme-Update

List of things, which has to be changed in the README file:

Wine is needed for Win Icon
dev:watch is now dev:liveserver

electron


Add lights dimmer support for _TZE200_w4cryh2i

Closes https://github.com/zigpy/zha-device-handlers/issues/2125

Codecov Report
Base: 83.60% // Head: 83.60% // No change to project coverage :thumbsup:

Coverage data is based on head (1d3bb74) compared to base (a16c153).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##              dev    #2126   +/-   ##
=======================================
  Coverage   83.60%   83.60%
=======================================
  Files         254      254
  Lines        8156     8156
=======================================
  Hits         6819     6819
  Misses       1337     1337

Impacted Files
Coverage Δ

zhaquirks/tuya/ts0601_dimmer.py
100.00% <ø> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

Nice! Did you test that everything works as expected?

Unfortunately, I did not test it. I am not sure how can I quickly alter code in my HA setup for a test. I assume it would work, because it is another variant of the device like _TZE200_9i9dt8is.
Let me know how I could run a test on my HA OS.

Search for "ZHA custom quirks" and/or check the wiki section on the zigpy repo.
You shouldn't need to alter any more code hopefully. Just copy the altered quirks file to your custom quirks folder (set in your configuration.yaml) and restart HA. Then see if the quirk applies.

Thanks! Tested and it works 👍

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
6819

Relevant Lines:
8156

💛  - Coveralls


Revert InMemoryBinaryCriteoIterDataPipe from IterableDataset to IterDataPipe

Summary: After D33955933 is landed, the OOM issue should have been fixed for IterDataPipe. So reverting the original Diff D33965445 (https://github.com/pytorch/torchrec/commit/bc5d3105ecc95326f7d283b29851647fbf60f1e5) to provide the feasiblity to use DataLoader2 to load this Dataset.
Differential Revision: D41781152

D33955933 translates to https://github.com/pytorch/pytorch/pull/72209 :)


Problem training SBERT and SBERT DA to reproduce results

I'm having problems to run the training scripts related to SBERT and SBERT DA.
My setup is:
LSB Version:    :core-4.1-amd64:core-4.1-noarch
Distributor ID: CentOS
Description:    CentOS Linux release 7.9.2009 (Core)
Release:        7.9.2009
Steps to reproduce:
1.- After cloning and installing successfully run:
python3 scripts/ml/train_nli.py --model-name digitalepidemiologylab/covid-twitter-bert --batch_size=10 --epochs=10 --lr=5e-5 --accumulation_steps 32 --train data/multinli_1.0/multinli_1.0_train.jsonl --dev data/multinli_1.0/multinli_1.0_dev_matched.jsonl --ckpt my_model
The error message is:
Traceback (most recent call last):
File "scripts/ml/train_nli.py", line 17, in
from backend.ml.sentence_bert import SentenceBertClassifier
File "/nobackup/ptdhv/covid19-backend/backend/ml/sentence_bert.py", line 34, in
class SentenceBertBase(Detector, torch.nn.Module):
File "/nobackup/ptdhv/covid19-backend/backend/ml/sentence_bert.py", line 50, in SentenceBertBase
loss_kwargs: Dict[str, Any] = None) -> torch.FloatTensor:
File "/home/home01/ptdhv/.local/lib/python3.7/site-packages/overrides/overrides.py", line 88, in overrides
return _overrides(method, check_signature, check_at_runtime)
File "/home/home01/ptdhv/.local/lib/python3.7/site-packages/overrides/overrides.py", line 114, in _overrides
_validate_method(method, super_class, check_signature)
File "/home/home01/ptdhv/.local/lib/python3.7/site-packages/overrides/overrides.py", line 135, in _validate_method
ensure_signature_is_compatible(super_method, method, is_static)
File "/home/home01/ptdhv/.local/lib/python3.7/site-packages/overrides/signature.py", line 93, in ensure_signature_is_compatible
ensure_return_type_compatibility(super_type_hints, sub_type_hints, method_name)
File "/home/home01/ptdhv/.local/lib/python3.7/site-packages/overrides/signature.py", line 288, in ensure_return_type_compatibility
f"{method_name}: return type {sub_return} is not a {super_return}."
TypeError: SentenceBertBase.forward: return type <class 'torch.FloatTensor'> is not a <class 'NoneType'>.

Whoops! Looks like the latest version of overrides is actually doing something at runtime.


Fix term functions to have optional arguments so they work again

In current develop the keybinding for opening multi-term in both current window (SPC o t) and in a popup (SPC o T) are broken and throwing the error message:
Wrong type argument: commandp, +term/open

The patch is pretty simple, so I hope it can get in and save other people this trouble! Part of it is also that the keybindings are in default but by default term is not enabled in the init.example.el.

The root of the problem is that:
a) :emacs term is not enabled by default,
b) And many keybindings in :config default are not conditionally bound. Meaning there are many keybinds for modules that may or may not be enabled.
I am working on a solution for B. As for A, I'd like to keep term disabled by default, because there is no consensus on which of term or eshell is a more reasonable default. Some users use one or the other, both or neither, and Doom tries to adhere to an "best default or opt-in" philosophy. If no best default exists, then it should be disabled.
I may add conditionals to :config default in the meantime or just wait until I roll out my WIP solution for B, but in either case, I must refuse this PR.
Thanks for bringing it to my attention, however!


Installation through Carthage fails with Xcode 10

I am using Xcode 10.0(was reproducible with betas) and Carthage version 0.30.1
This is the error message I get from Carthage:
*** Building scheme "AFNetworking iOS" in AFNetworking.xcworkspace
*** Building scheme "AFNetworking watchOS" in AFNetworking.xcworkspace
Build Failed
Task failed with exit code 70:
/usr/bin/xcrun xcodebuild -workspace /Users/username/Documents/work/application/Carthage/Checkouts/AFNetworking/AFNetworking.xcworkspace -scheme AFNetworking\ watchOS -configuration Release -derivedDataPath /Users/user/Library/Caches/org.carthage.CarthageKit/DerivedData/10.0_10A255/AFNetworking/3.2.1 -sdk watchsimulator -destination platform=watchOS\ Simulator,id=B35EB6D9-8741-4DE3-9C02-D7D3D60AE244 -destination-timeout 3 ONLY_ACTIVE_ARCH=NO CODE_SIGNING_REQUIRED=NO CODE_SIGN_IDENTITY= CARTHAGE=YES build (launched in /Users/user/Documents/work/application/Carthage/Checkouts/AFNetworking)
This usually indicates that project itself failed to compile. Please check the xcodebuild log for more details: /var/folders/l6/4zrqf_m91mzcx175p7t8klq40000gn/T/carthage-xcodebuild.5z9bHS.log
Thanks!

I do face the same error message.

@ivan-konov Can you copy that log file here?

It turned out to be a bug in Carthage when it builds watchOS schemes with Xcode 10. They should be releasing the fix soon. If somebody really needs it now they can just install Carthage using the master branch.

Great, thanks for following up!


Ollie: Watch mydresshut.com

Ollie requests the watch of the watch_keyword mydresshut\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

mydresshut\.com has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by Jeff Schaller in Charcoal HQ


[$500] Web – Chat - Unable to open chat with CTRL+Enter

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: v1.3.85-0
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause_Internal Team
Slack conversation: @
Action Performed:

Go to https://staging.new.expensify.com/
Log in with any Gmail account.
Click on the green + icon on the home screen
Click on Start chat
Use the arrow keys to highlight a chat.
Press CMD+Enter

Expected Result:
The selected chat is open.
Actual Result:
Nothing happenes.
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos

Android: Native

Android: mWeb Chrome

iOS: Native

iOS: mWeb Safari

MacOS: Chrome / Safari
https://github.com/Expensify/App/assets/115492554/7c713063-b812-4ad1-8d43-83d1ee4e46d4

MacOS: Desktop

View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~0133f9833c7b1a9ad2
Upwork Job ID:<PHONE_NUMBER>537739264
Last Price Increase: 2023-10-17

Now we open the chat by pressing [Enter]. Isn't this an expected behavior?
Ctrl + Enter is used to confirm when group chat is created now

Sounds good!


Add enum_chron

Work towards #933

Coverage increased (+0.07%) to 78.712% when pulling 94c71d12101cd94191b2bfb04219a7af37ee1733 on more_scsb into ebde1f1b79063ea49ad64ba0202c8b448042aa58 on alma.

Coverage increased (+0.07%) to 78.712% when pulling 94c71d12101cd94191b2bfb04219a7af37ee1733 on more_scsb into ebde1f1b79063ea49ad64ba0202c8b448042aa58 on alma.


HTTPS + Query causes connection errors

http://de.serlo.org/auth/login?redir=http://de.serlo.org/subscribe/22964/0
redirects to
https://de.serlo.org:80/?redir=http://de.serlo.org/subscribe/22964/0
which obviously doesn't work, because https doesn't listen on port 80

Should be fixed now


v3.12.0

[v3.12.0] - ESignature API v2-<IP_ADDRESS> - 09/02/2021
Changed

Added support for version v2-<IP_ADDRESS> of the DocuSign ESignature API.
Updated the SDK release version.

AppVeyor build and test pass.


[BUG][PUBLISHER]Publisher key shouldn't be unique

such that, different users will be able to create publishers with same key

models.Publisher fix UI_index_publisher_key


fix: Fixed incorrect page rendering in horizontal paginated mode on ios.

I opened an issue a couple of days ago about this problem and I think this is a fix for it.
Details can be found here: https://github.com/ttu-ttu/ebook-reader/issues/170
I added the attribute column-width to .book-content-container and I set it to auto when the columns are more than one and to the width of the page if there is only one column. This fixes a problem that only happens on mobile ios when column-count is set to value 1. This bug makes the page load vertically instead of horizontally causing the reader to be unusable, since vertical scrolling is disabled.
I found the solution here: https://stackoverflow.com/questions/71319077/columns-or-column-count-1-not-working-safari .
I tested it and it works both on mobile and on chrome browser on pc.

Ok I made the changes thank you!


Visualizations: move Y-axis labels in RxC plot

Move Y-axis labels to the left axis rather than in the legend, and center based on the boxes in the first row. May need additional UI work if there are many values for the Y-axis and the first row (X-axis value) doesn't have some of those values, since there will be no box for it.
Users should not be able to remove one of the values for the axis.

Hi Sheena! Do you envision the y-axis labels being to the right of the numbers, to the right of the variable name, or to the left of the variable name?

There certainly will be an issue with boxes in the first column that are too small for either the label to fit or the color to be visible. I don't think we can lose the square color indicators present in the current legend. These are going to be challenges from an implementation perspective as well.

Yeah there are definitely challenges here and @d-callan can weigh in as well. I think we want something like this where we loose the y-axis scale and have the labels there instead. I could see there being an issue with long value labels pushing the variable label too far to the left unless we truncate the labels. In the case that there are boxes in the first column that are too small for the label or the color isn't visible, would it be possible to draw a short line from the label to where that box would be, so the labels could still be in the same order as the values, but offset if needed? And if you moused over the label, could it highlight all associated boxes for the user?

@SheenaTomko Spent some more time thinking about this---it would certainly be a problem. Consider this mosaic with hover labels:

The left-hand y-axis labels you describe are essentially these hover labels but always visible. In the image the tiny boxes at the bottom correspond to a lot of vertical space taken up by labels. You can image this getting even worse, such that the labels take up more vertical space than the whole plot itself. That's fine with hover labels because they're usually hidden, but these axis labels would always be visible. I don't think that's feasible.
I think this is no longer just an implementation problem. It's a UI problem that needs to be sorted out before talking about implementation issues.

So I don't mind if we punt on this for now I suppose, though I think we should drop the numeric scale either way and probably leave the ticket open. And if I'm honest, I don't really see how any of these issues are specific to the y-axis. I think we'll have these problems on the x-axis too potentially, so I didnt think we were introducing new challenges. Though I get it increases the magnitude or urgency of the problem...

Assigning to @SheenaTomko (and others by extension) for UI discussion

@danicahelb Any comments on the above? If not, I'll assume these minor differences are fine

Y-axis now looks good now, closing ticket.


Storage Timeouts

So, this is an issue thats difficult to reproduce. it seems to occur in the server after its been running for some time, usually several hours ( or days).
We use mongo as the storage connector and Redis for cache and messages.
Its hard to explain what exactly is happening, its like the server gets into a broken state. The logs will show all storage requests failing with timeouts. Hundreds to thousands of them. Sometimes there is indications of connection drops, but not always.
Usually the server no longer handles records correctly until I restart it.
I had been restarting the instances daily overnight to avoid this issue, but today it happened after running for about 11-12 hours.
The logs don't provide enough information to know if this is an issue with the connection to mongo, or some bug happening internally. There is no memory leak as far as I can tell, or increase in traffic.
Its possible the record request refactor might resolve this. I will look at updating things on staging before production. But has any seen an issue like this?

I dug up some more info.
It seems as if something is causing a bunch of extra cache and storage requests. Looking at metrics on the redis at about the time this failure happened, there was a huge increase in get commands and cache misses.
All these requests likely cascaded into the storage layer and caused the failure.
I am going to look more into the particular records that were showing errors and if they should have been in deepstream at the time of the error. These records may have possibly been deleted already which would explain the cache stampede. If that is the case, something is causing record names to be cached within deepstream?

The storage timeouts do seem to be based on records that were deleted several hours prior. There is definitely some odd behavior that is causing deleted records to be required.

Interesting! I was looking for a reason to improve record deletion, hoping this is it. We'll take a look. Thanks for the insight, helps a lot!

We are using this in conjunction with the javascript client. The behavior does not seem consistent with issue though. The javascript client only ever subscribes to a subset of the data for a given a customer. The cache stampede seemed to involve records across all customers.

@datasage we'd quite like to look into this one. Is there any steps you could provide to reproduce?
We're thinking of running a script for ~24 hours that imitates your scenario, ie. clients connecting, subscribing to records, deleting records etc.
Is there anything specific or of note that you're doing that we could use to help write this?

It doesn't always happen within 24 hours, sometimes takes several days. But that might provide info as to where record names might be getting cached. I'm not using 2.3.0 yet, currently just 2.2.2. The issue has happened on some earlier versions too which is why i restart nodes each day.
In production running two nodes with a load balancer balancing connections between both instances. Redis is used for messaging and cache, Mongo Storage.
Data is organized by customer, using customerName/ in all record keys. Permission is used to limit what customers can read by the record prefix. It matches a key in the user data.
Most data updates are done by the server, often using upsert. Records are used for a limited amount of time, then the record is removed. We usually have around 400-500 clients during the peak period during the day.
Most records are referenced with list records. These list records always exist and are never deleted. The javascript client does subscribe to most active records for a given client. Once a record is removed, its removed from the list first, then the actual record is removed.
We also use events but most of those channels only have 1-2 subscribers. Server mostly publishes events.

Thinking about this a bit more, the client could actually be involved. We do run a status board component which customers use to monitor agents and calls. They are usually on this page for up to 24 hours.
If something caused a number of connections to drop at once, and the clients reconnected to the other node, its possible this could have caused the stampede.
I will look at some logs to see if I can correlate other events to this.

It does appear that for some reason I am keeping records bound on the client side, I need to do more digging into this to figure out if its an issue with the deepstream client or with my code.
This could definitely explain the stampede of requests if the client gets disconnected. Over several hours, some clients may end up binding thousands of records. If the records are getting a released correctly, there should only be 100-200 at most for a given client.

Yup, thats a relief / makes sense! deepstream doesn't hold any references to records other than when a transition is in progress, meaning this would have been a substantial issue somewhere!
Thanks for keeping us updated!

Do messages get removed from the client if a delete event is passed? I'm not seeing a path through which that should get handled. It seems like the records stay in the client indefinitely.

Doing some tracing, i am seeing some records go through _removeRecord, but not all of them. All records should have delete triggered on it so I am not sure yet why some records are being removed and others are not.

I have tested in a very limited volume case, and everything works correctly. So it seems this issue has some sort of volume component to it.
I did some heap checking on the client and I did see my subscribed records grow to over 1000 after a couple of hours.

Do records get removed from the client if a delete event is passed? I'm not seeing a path through which that should get handled. It seems like the records stay in the client indefinitely.

Should be yes, we will need to replicate this, probably by doing bulk deletes and making sure things don't go wierd in edge cases

There might be some edge case going on, with enough volume. My very limited narrow test case seemed fine, but looking at a client with around 125 record subscriptions, the number just grows over time.
We are mostly relying on delete events to remove the records. But im checking to see if there are scenarios where something else is happening.

Are delete events property broadcast through all nodes?

I think that might be the bug, it appears that a delete event is not being passed into the message connector like updates are.
This is being called on update, but not on delete.
if (originalSender !== C.SOURCE_MESSAGE_CONNECTOR) {
this._options.messageConnector.publish(C.TOPIC.RECORD, message)
}

I think your right. We will be looking to fix this ASAP.
Thanks alot for pointing this out.

It looks like the issue is fixed, closing this.


Apple Notes: Support inline internal links

Closes #133
Apple Notes has two kinds of internal links, one of which is stored as an attachment, and which has a little note icon. Its text always matches the name of the note.
This PR adds support for the other, which is stored using the same mechanism as external links. In the screenshot below, the first link is the new type and the second one is the attachment type.

Thanks! Looks good!


Now just displays the work 'Notification' below the message (Windows 10 only)

A recent change in Windows 10 explained here makes some internal text visible in the popup. In short :

if the program specifies a custom icon for its notification, then we show that custom icon, but we show it a bit larger, so that it looks different from any icon that already exists in the notification area. And since there is nothing to tie the notification visually to the icon in the notification area that generated it, we label the notification with the program so the user knows where it came from.

The field that is displayed is the "File Descrition" field, visible in the details tab of the executable properties. This version will only display "Notification".

Thanks! 🎉


Cannot import files using tsconfig.json paths

I am getting an error for component testing with nextjs:
[0-0]  Error:  Test failed due to the following error: [plugin:vite:import-analysis] Failed to resolve import "@/app/page" from "__tests__\app\page.test.tsx". Does the file exist?
C:/wob/wdio-nextjs/__tests__/app/page.test.tsx:16:33
14 |  const {
15 |    default: Home
16 |  } = await import("@/app/page");
   |                                  ^
17 |
18 |  describe("React Component Tests", () => {

I copied the project from component-testing-examples/nextjs and tried to make it work with tsconfig paths, by adding the following:
"paths": {
  "@/*": ["./*"]
}

to the tsconfig.json.
Additionally I moved around a couple of files to resemble a real life project of mine.
Demo repository: https://github.com/Winify/wdio-nextjs-tsconfig-paths
Steps to reproduce:

git clone https://github.com/Winify/wdio-nextjs-tsconfig-paths
npm ci
npm run wdio

If I change the import from import Home from '@/app/page' to import Home from '../../app/page' then the test passes.
Any advice?

Sorry I have jus tread the Discord channel and noticed the thread from april that I can do this with:
https://webdriver.io/docs/runner#runner-options


Can't install BeatifulSoup

beautifulsoup4  is the version that is compatible with python 3 . it could be installed but not sure if it was successful since it gives errors and says that version for python2 has been installed.
https://www.crummy.com/software/BeautifulSoup/bs4/doc/


Comparison to other open variant solutions

I'm mainly interested on how this compares to haskus Excepts:

https://docs.haskus.org/variant/excepts.html
https://hackage.haskell.org/package/haskus-utils-variant-3.0/docs/Haskus-Utils-Variant-Excepts.html

Just from a quick look, I would guess that using concrete error lists (e.g. '[This, That, Them]) over constraints (e.g. (HasError This e, HasError That e, HasError Them e) => e) means your errors are necessarily ordered? I don't know this library at all, though, so I wouldn't necessarily trust myself.
From a skim read, it would seem closer to the approach I took the first time I tried to build a variant: https://github.com/i-am-tom/learn-me-a-haskell#oneof
This is fine, but it becomes annoying when GHC can't unify Variant '[Int, Bool] with Variant '[Bool, Int], for example, and a constraint-based approach would side-step this problem (because constraints aren't ordered).

This is fine, but it becomes annoying when GHC can't unify Variant '[Int, Bool] with Variant '[Bool, Int], for example, and a constraint-based approach would side-step this problem (because constraints aren't ordered).

It unifies via liftE. I'm using that approach right now in ghcup, but it is not that ergonomic and I need to heavily use type applications in some places and liftE all over the place. And the biggest problem is that you never know whether you have extraneous constraints. With classes, that can get a little better, but the unnecessary constraints solver is only best effort anyway and quickly breaks down with type families.
I've never used plucky. Do you think that is preferable over oops?

I spoke with @parsonsmatt, and he seems to think Plucky will suit your needs! Honestly, it's not a library I've used in anger, but I've heard good things from colleagues, and Matt's work is reliably wonderful. Plucky is certainly more likely to be maintained than this library ever will be; as I say, this will probably disappear (when I have some motivation to write some code again) in favour of a bigger library of strongly-typed utilities.


Lokalise latest may 17

Description:
Latest pull from May 17 after the move to gps/app/locales
Because we have haitian creole still present in Lokalise, it gets pulled in. Let me know if we need to cherrypick and remove it before we create a PR. @tstirrat

Linked issues:

Screenshots:

How to test:

Closing this because the file paths were changed in #836  and pull script fixed in #852


chore(main): release 1.2.0

:robot: I have created a release beep boop
1.2.0 (2024-10-20)
🚀 Features

Add Debug Mode (#160) (2249da5), closes #159

📝 Documentation

Update README with pre-built image and license info (#136) (9ddc6f9), closes #137

🧰 Maintenance

Add Configuration class and refactor repository retrieval (#156) (15425e6), closes #154
Add setup-dependencies directory to Dependabot config (#133) (44ec79d), closes #134
Add test action and output validation (#147) (040eaa8), closes #148
Add version check workflow and version file (#145) (19dd4aa), closes #146
Bump Python Version to Python 3.13 (#140) (3135efc), closes #139
Fix Docker image reference in action.yml (#149) (bbfd6f1), closes #150
Force color output in logs (#164) (2d0d5da), closes #163
Optimise pre-commit hook for Python files (#142) (8e48442), closes #141
Reduce log verbosity (#165) (1c78547), closes #162
Reduce Verbose Logging (#158) (94fea82), closes #157
Refactor GitHub Action structure and usage (#143) (fc7d2d6), closes #144
Simplify Docker build job name and description (#166) (d94299f), closes #167
Update workflow trigger conditions (#168) (f96a17a), closes #169

This PR was generated with Release Please. See documentation.

:robot: Created releases:

v1.2.0
:sunflower:


pip install azureml-interpret-contrib

I believe there's a typo above. 'pip install azureml-contrib-interpret' instead worked for me. It's stated as 'pip install azureml-interpret-contrib' above.

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

ID: c71ab89d-aa07-b190-ac65-7d9eea831ec6
Version Independent ID: 532fc833-12a1-01e3-74b5-f4a4523f4831
Content: Explain machine learning models and predictions - Azure Machine Learning
Content Source: articles/machine-learning/how-to-machine-learning-interpretability-aml.md
Service: machine-learning
Sub-service: core
GitHub Login: @mesameki
Microsoft Alias: mesameki

@ddavis-100 we will review your feedback and get back to you shortly. Thanks.

@ddavis-100 thanks for your feedback. Updates should reflect soon.


Trouble with morphology examples

If you open the example file morphology_linear.hs and replace stages with something like
    stages = map (lowerTransformations . scale 0.8 . pathify . center) $ colorize
      [ latex  "$e^{i\\pi}+1=0$",
        latex "hello W$\\xrightarrow{\\text{orl}}d$"
      ]

it will not compile.  It stays stuck on "Frames generated: 0/480"
Am I doing something wrong?

You're not doing anything wrong. The morphing code is just very immature. I've pushed a patch that reverts to a morphing algorithm that should always work.
The morphing framework will eventually support a slew of different algorithms but right now it's pretty much just a mess.

Ok. Thanks for the response.
I'd been working on making animations with manim and I used to joke "I just wish it were not written in python but something like idris instead". I'd always wanted to learn more idris (or haskell) and that way I could do two things at once. So I was very excited to find reanimate a few days ago. Alas, it seems the code-base is not quite mature enough yet for my needs.
If it's all right with you I'll continue to play with the code and post any issues I encounter. Let me know if it gets irritating, and I'll stop.

Not irritating at all; You're doing me a huge service.
Do you know which features you need from manim? Morphing and fonts are high on my TODO list and should be ready in a couple of weeks. Would that be enough to make reanimate useful for you?

cool.
Yes. Morphing LaTeX is the most important ingredient for me.
Does reanimate have a discord, or a mattermost, or a subreddit, or some place to discuss things that don't quite rise to the level of github issues?

There is a gitter chatroom: https://gitter.im/reanimate/community


Add license

Hey @ameyakarve, would you mind adding a license file to this repo? Maybe MIT? Was looking at using it but wanted to check with you on what was permissible. Thanks!

@comp615 Added. Thanks.


Add pattern for filename

@c2karansingh Please have a look.
https://github.com/lalcebo/json-schema/blob/571eb058fefb4220e0582f173d49fa4fa10b3693/serverless/reference.json#L40856-L40859
It should look something like this:- ${file(.yml)}

@KillDozerX2 See if this scratches the itch

this

That's where you're supposed to add the pattern.

@c2karansingh seems to be working for me. Thanks.

@KillDozerX2 where do you want me to use it?


Add performance metrics for the README.md

Things that can be added to the README.md file :

Add performance metrics for the program such as accuracy when working on a number of images.
Analysis of time and memory space taken while running the model.

getting error
File "id_card_detection_image.py", line 103, in
im = Image.open(image_path)
NameError: name 'image_path' is not defined
Replaced image_path with PATH_TO_IMAGE fixed above error
File "id_card_detection_image.py", line 105, in
im.crop((left, top, right, bottom)).save(output_path, quality=95)
NameError: name 'output_path' is not defined
define output_path in the code fixed above error
but getting new error not sure where to fix
Traceback (most recent call last):
File "/Users/amashi/PycharmProjects/claydesk/venv/lib/python3.7/site-packages/PIL/Image.py", line 2082, in save
format = EXTENSION[ext]
KeyError: ''
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "id_card_detection_image.py", line 106, in
im.crop((left, top, right, bottom)).save(output_path, quality=95)
File "/Users/amashi/PycharmProjects/claydesk/venv/lib/python3.7/site-packages/PIL/Image.py", line 2084, in save
raise ValueError("unknown file extension: {}".format(ext))
ValueError: unknown file extension:

getting error
File "id_card_detection_image.py", line 103, in
im = Image.open(image_path)
NameError: name 'image_path' is not defined
Replaced image_path with PATH_TO_IMAGE fixed above error
File "id_card_detection_image.py", line 105, in
im.crop((left, top, right, bottom)).save(output_path, quality=95)
NameError: name 'output_path' is not defined
define output_path in the code fixed above error
but getting new error not sure where to fix
Traceback (most recent call last):
File "/Users/amashi/PycharmProjects/claydesk/venv/lib/python3.7/site-packages/PIL/Image.py", line 2082, in save
format = EXTENSION[ext]
KeyError: ''
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "id_card_detection_image.py", line 106, in
im.crop((left, top, right, bottom)).save(output_path, quality=95)
File "/Users/amashi/PycharmProjects/claydesk/venv/lib/python3.7/site-packages/PIL/Image.py", line 2084, in save
raise ValueError("unknown file extension: {}".format(ext))
ValueError: unknown file extension:

Fixed the error by passing correct output_path


Using DTO Class in constructor.

Describe the bug
I need help, is it possible to use a DTO class in a entity constructor? I am looking for an ORM that allows it. because I have entities with many parameters and it is cleaner this way
Stack trace
node_1   | [NestWinston] Error  7/2/2021, 2:36:42 PM Method: GET; Path: /123456/concepts; Error: Cannot read property 'storeId' of undefined;  - {"service":"xxxx","stack":["Typ
eError: Cannot read property 'storeId' of undefined\n    at new Concept (/usr/src/app/src/modules/concepts/entities/concept.entity.ts:89:31)\n    at EntityFactory.createEntity (
/usr/src/app/node_modules/@mikro-orm/core/entity/EntityFactory.js:69:20)\n    at EntityFactory.create (/usr/src/app/node_modules/@mikro-orm/core/entity/EntityFactory.js:34:77)\n
    at SqlEntityManager.find (/usr/src/app/node_modules/@mikro-orm/core/EntityManager.js:100:52)\n    at processTicksAndRejections (node:internal/process/task_queues:93:5)\n
at ConceptService.getAll (/usr/src/app/src/modules/concepts/concept.service.ts:13:33)\n    at ConceptController.findAll (/usr/src/app/src/modules/concepts/concept.controller.ts:
33:22)"]}

To Reproduce
Steps to reproduce the behavior:

Create a entity with constructor.

export interface StoreConceptPort {
  storeId: string;
}

import {
  IsNotEmpty,
} from 'class-validator';
import { UseCaseValidatableDto } from '@test/common/dto/use-case-validatable-dto';
import { Exclude, Expose, plainToClass } from 'class-transformer';
import { StoreConceptPort } from<EMAIL_ADDRESS>import { ServiceType } from<EMAIL_ADDRESS>
@Exclude()
export class StoreConceptDto
  extends UseCaseValidatableDto
  implements StoreConceptPort
{
  @Expose()
  @IsNotEmpty()
  storeId: string;

  public static async new(payload: {
    storeId: string;
  }): Promise<StoreConceptDto> {
    const adapter: StoreConceptDto = plainToClass(StoreConceptDto, payload);
    await adapter.validate();

    return adapter;
  }
}

import {
  Embedded,
  Entity,
  EntityRepositoryType,
  Index,
  ManyToOne,
  Property,
  Unique,
} from '@mikro-orm/core';
import { ConceptRepository } from<EMAIL_ADDRESS>import { BaseEntity } from '@test/common/entities/base-entity';
import { StoreConceptPort } from<EMAIL_ADDRESS>
@Entity({ tableName: 'concepts' })
export class Concept extends BaseEntity {
  [EntityRepositoryType]?: ConceptRepository;

  @Index()
  @Property({
    name: 'store_id',
    type: 'varchar',
  })
  storeId: string;

  constructor(conceptDto: StoreConceptPort) {
    super();
    this.storeId = conceptDto.storeId;
  }

}

Find Entity with repository

import { Concept } from './entities/concept.entity';
import { ServiceType } from<EMAIL_ADDRESS>import { EntityRepository, Repository } from '@mikro-orm/core';

@Repository(Concept)
export class ConceptRepository extends EntityRepository<Concept> {
  public async findOneByStore(id, storeId): Promise<Concept> {
    return await this.findOneOrFail({
      storeId: { $eq: storeId },
      id: { $eq: id },
    });
  }
}

And use it.

See error: "TypeError: Cannot read property 'storeId' of undefined..."

Expected behavior
The model is expected to hydrate well
Additional context
Versions

Dependency
Version

node
15.2.0

typescript
4.2.3

mikro-orm
4.5.7

your-driver
?

That stack trace is from different call - you can see it starts with findAll() not findOne(). The ORM will never use entity constructor to create instance, so unless you are using forceEntityConstructor flag, this is not even possible to happen, the instance is created via Object.create() instead. And if you are using that flag, you need to ensure your entity ctors all have optional parameters, as they can never be filled in.


Add proxy auth

Some proxies (in my case luminati.io, but it's a general thing) require authentication and sttp so far models this in a way (on RequestT, setting the Proxy-Authentication header there) which doesn't work for all proxies - forcing the user to manually create the underlying backend before being able to use such a proxy with sttp.
This PR extends SttpBackendOptions to allow proxy username and password to be passed and adapts the akka-http, async-http-client and okhttp backends (these seem to be all which support this) to set themselves up accordingly.
I unfortunately don't know where to get a proper proxy set up with authentication, therefore no dedicated tests…if someone has an idea, please let me know!

Thanks! One thing I'm wondering about, maybe you know, and maybe this could be added to the docs: when using the Proxy-Authorization header is enough, and when to use the options?

Released as 1.5.1


text selection, copy cut paste events

A bunch of stuff...
text can now be selected using shift + arrow keys or ctrl+a, selected text will also be overwritten if new text is pasted or a key is inputted. I've seen you had copy cut paste in your wishlist in readme so i made that too.
I guess this should cover some basic usage, should be good enough for input texts like username/password or some signup info. I'm gonna use this on production so if i notice more stuff will make more PRs.
Also, do you mind if i make some cosmetic changes in code?

Hi!
This is really great! However... I'm not using Pixi that much these days, so I'm not really able to test and merge pull requests that much...
Do you want to be a maintainer of the project? If so I will add you so you can push directly to master, and also give you rights to push to the npm repo.

Yeah that would be nice, we're using this input in our game so chances are I'll keep improving it as we improve our game.
Thanks

Ok you are added as a collaborator...
I thought it was added to the npm repo as well, but after checking I realize I never did that. Feel free to do that as well!


Missing dependencies

Following your tutorial: getting started with Gulp, where you say for the newbies to npm install and bower install but missing the package.json and bower.json?
Can you add those in your repository? It would be a great help!
Thanks.

Oh I see what you're saying. Okay, I'll provide a sample npm and bower file this afternoon.


Make it possible to apply TraceWebServletConfiguration in Servlet applications that don't use Spring MVC

Is your feature request related to a problem? Please describe.
It looks like Spring Sleuth only extracts trace-information from incoming HttpServletRequests requests when Spring MVC is on the classpath. This isn't the case in all Spring Boot web applications of type Servlet. For instance (our case) when using spring-boot-starter-jersey, Spring Web MVC isn't used. But in that case TraceWebServletConfiguration autoconfig won't be enabled, since it has a ConditionalOnClass HandlerInterceptorAdapter from spring mvc.  There is also no easy way to still include configuration from TraceWebServletConfiguration, which basically means that the functionality has to be duplicated in such Spring Boot apps.
(We had to create a configuration that creates traceWebAspect, traceWebFilter, and traceTomcatWebServerFactoryCustomizer beans, and implement a LazyTracingFilter to make web tracing work).
Describe the solution you'd like
Either:

Refactor TraceWebServletConfiguration to move the parts that configure specifically for Spring Web Mvc to another auto configuration, or:
Provide another easy way to apply the tracer beans created by TraceWebServletConfiguration.

Describe alternatives you've considered
As a workaround our code basically duplicates the TraceWebServletConfiguration by creating traceWebAspect, traceWebFilter, and traceTomcatWebServerFactoryCustomizer beans, and copying the implementation of LazyTracingFilter.
Additional context
Nothing to add here

We're not planning on adding any new features to Spring Cloud Sleuth since Sleuth 3.1.x is the last minor version we will support.


#21: Add a configuration option to enable/disable paste listing

I also reworked configuration a bit, for easier extension in the future (one global object instead of N global variables).
Also, I added a parameter to initialize_backend for backend parameters. For now, it's empty.

Hey, thanks.. So, do you think we should pass the parameters to the backend? That's an issue in my opinion because we need to make TorPaste backend-agnostic, and including the configuration parameters in the main file doesn't help. What I was thinking for backends was that torpaste.py will select the appropriate backend, and then the MySQL backend will look for whatever ENV variables it may need, such as TP_BACKEND_MYSQL_HOST. From what I understand, in the current setup, we need to handle everything in torpaste.py and then pass it in the backend_config to every backend. Is that correct?

The way I did things, yes, torpaste.py would handle all configuration. Your idea is actually better. I'll change that;

Ok, I removed the backend-specific configuration from the core. initialize_backend does not take a parameters argument anymore.

Merged, and commit pushed to update README.md.


Add pkgconfig file

Could potentially remove APPLE and/or MSVC guard, not sure...

Turns out it can be leveraged for cross-compilation on all platforms.


Improve warnings when running scripts in the editor

So I noticed that when a script has errors (caught by the analyzer in case of GDScript), if we try to run it in the editor the reported warning is incorrect and misleading:

That's probably because the tool flag on the script is not set if the script fails the analyzer checks. That may change in the future, but after looking at the code, I think the order of checks in the editor itself is wrong. So I've changed it, and also made these warnings into toasts instead of blocking popups. Seems more appropriate this way, and I adjusted the messages that are shown.

First we check if it's a script at all (you can run the command for text files, for example).
Then we try to reload the script and check if that went successfully; we also check if the script is valid.
Then we check if it's a tool script.
Then we check if it inherits EditorScript.

This order ensures that the latter checks are done on a freshly reloaded version of the script, and before that we actually try to parse it and see if it's valid.

While looking up how to do the toasts I noticed one of them was missing the translation macro, so I added that.

Two warnings for the price of one!

Thanks!


Refactor for v2

The main changes are

Allow postgres to not use the cli-generated schema types, but instead function similarly to mysql and sqlite
Simplify the existing test suite but using the above feature
Add comprehensive new test suite which checks that we play well with diesel-cli and the auto-generated schema.rs
Bump to 2021 edition and MSRV to 1.57 (only necessary for the test suite, should work on lower versions otherwise)

TODO before merge:

Update documentation

Closes #81

Hi,
I tried this branch with the last release of diesel and it all works fine, I do not  use any  any schema generation so I did not test that part, I see this is not released yet, would be possible to get another beta release with this branch, is there any way I can help to push this forward ?
Bye

Could QueryId trait be implemented as part of DbEnum?
        use diesel::query_builder::QueryId;

        impl QueryId for #diesel_mapping
        {
            type QueryId = #diesel_mapping;

            const HAS_STATIC_QUERY_ID: bool = true;
        }

This way diesel-derive-enum could depend on released version diesel-cli (no need for custom_type_derives feature). Though I'm not sure this wouldn't cause som compatibility in future (if diesel maintainers decide to implement QueryId automatically for all postgres generated types).


Fix possible deadlock when creating scripts during a background garbage collection

Fixes #87405.
I'd really appreciate someone else looking over this, but since holding on to the mutex seems like an oversight (compare with virtually the same code in the constructur) and this change is very localized, this might be a suitable quick fix for a problem that's rare but deadlocks the game and is hard to diagnose if it materializes.

Thanks!

Cherry-picked for 4.2.2.


RFC0434: add jwe_suite

Signed-off-by: George Aristy<EMAIL_ADDRESS>

@kdenhartog

We're not currently using the OOB protocol for initial setup and only are using DIDCommV2. How would we be expected to convey this?

I understand we are punting the switch from DIDComm V1 to V2.
cc @swcurran to see if Stephen has any thoughts on this.

@kdenhartog -- how are you doing the initial setup?

@kdenhartog -- how are you doing the initial setup?

Given right now we're only doing issuance/verification of VCs and DID Auth through OIDC provider we are able to use the 0-RTT functionality to build a DIDComm connection while issuing a VC / Verifying a VC during an OIDC interaction

@kdenhartog

@kdenhartog -- how are you doing the initial setup?

Given right now we're only doing issuance/verification of VCs and DID Auth through OIDC provider we are able to use the 0-RTT functionality to build a DIDComm connection while issuing a VC / Verifying a VC during an OIDC interaction

Let me re-phrase the question: how are the two parties negotiating (JWE?) encryption parameters?
Should we assume you're using published OIDC provider metadata?

@kdenhartog

@kdenhartog -- how are you doing the initial setup?

Given right now we're only doing issuance/verification of VCs and DID Auth through OIDC provider we are able to use the 0-RTT functionality to build a DIDComm connection while issuing a VC / Verifying a VC during an OIDC interaction

Let me re-phrase the question: how are the two parties negotiating (JWE?) encryption parameters?
Should we assume you're using published OIDC provider metadata?

We support all types defined in DIDCommV2 whether optional or required so don't need to negotiate. This is a luxury afforded to us because we're only inter-oping with ourselves at the moment.
In talking with my team we see 3 potential options that we're weighing at the moment.

Include the discovery information in the DID Document.

this works well for did peer method, but presents challenges for basically every other method type so it probably won't be a good option.

Include the discovery information in a public metadata point similar to OIDC

This offers a layer of indirection that adds some form of privacy, but because the metadata point still would need to be published in some way (requires effectively the same amount of disclosure as the DID Document - e.g. public ledger DID means public metadata endpoint) we're not certain that the privacy would hold. Furthermore, it offers another point that has to be supported for highly available agents / architected around for semi available agents in order to make discovery work. Additionally, we're not confident in the willingness of others to adopt this method as well at this point so likely won't go this path.

Support an optional discovery JWM message similar to OOB

This is the path that makes the most sense to me at this point because it maps well to the same processes of how you'd discover the did document data with a 0-RTT system while also allowing the ability for an explicit discovery of supported protocols and other data related such that it could be passed at any time rather than only at the beginning. This is important if you want to update your support for data without having to restart the connection. Importantly though, it's not the required first message, but can act as a fall back message if a 0-RTT message is not understood. Particularly what I've been thinking about with this is that the majority of enterprise cases don't allow for any inherent negotiation and instead it's more of a "take it or leave it approach" because of the need to operate at scale. However, when you're interacting in a peer-to-peer environment the assumption that a connection will be set and then doing some form of logic beyond a connection later on seems far more logical and so if you wanted to you could send the discovery message as the first message, but it shouldn't be required in our view. This is the thing that makes a fair amount of sense to us at the moment, but we're open to considering options 1 or 2 / other options if there's strong reasons for doing so.

In summary, right now we don't need to do anything like this because all of our flows operate in a 0-RTT flow and we're supporting all forms of DIDCommV2 encryption. We also recognize the need for this in a subset of the use case we're considering in the future, but have deferred on this to this point because we see it as something that we can upgrade to overtime. Additionally, because we've opted to stick with DIDCommV2 only, we've not spent much time considering the usage of the OOB protocol to date, but want. to make sure that what we consider will align with everyone when Aries begins to converge on DIDCommV2.

@kdenhartog

Option 1 does not sound appealing to me because it would complicate even further the encoding of ledger-less DIDs. We are already struggling to come up with ways to "just" encode more than one key in a way that makes sense.
Option 2 puts an onerous burden on agents that are not always online - just like you said

As for Option 3 - agree it should not be required. In fact, this proposal pretty much enables this use case while still not invalidating the use of the Features Discovery protocol after connection establishment.
Perhaps we can delay discussion about the exact details of how the JWE parameters will be conveyed and instead come to agree on where in the protocols this should happen? My proposal is:

allow the sender of an OOB message to declare their supported JWE parameters somehow (precedent: already disclosing supported handshake protocols)
also allow an agent to discover the JWE parameters via the discovery protocol

I think the first point is more pressing - we're already discussing adoption of RFC0334 which would smooth the transition to DIDComm V2.
@swcurran @TelegramSam @dhh1128 thoughts?

@kdenhartog

Option 1 does not sound appealing to me because it would complicate even further the encoding of ledger-less DIDs. We are already struggling to come up with ways to "just" encode more than one key in a way that makes sense.
Option 2 puts an onerous burden on agents that are not always online - just like you said

As for Option 3 - agree it should not be required. In fact, this proposal pretty much enables this use case while still not invalidating the use of the Features Discovery protocol after connection establishment.
Perhaps we can delay discussion about the exact details of how the JWE parameters will be conveyed and instead come to agree on where in the protocols this should happen? My proposal is:

allow the sender of an OOB message to declare their supported JWE parameters somehow (precedent: already disclosing supported handshake protocols)
also allow an agent to discover the JWE parameters via the discovery protocol

I think the first point is more pressing - we're already discussing adoption of RFC0334 which would smooth the transition to DIDComm V2.
@swcurran @TelegramSam @dhh1128 thoughts?


Implemented new design for login, signup and all password change/request pages

This PR implements the designs of the login, signup and all password reset/change/reqeust pages.
Some screenshots:

Empty fields.

Focus in the Email field.

@krisutofu Actually, what you described was supposed to be configured in ESlint already but I must have missconfigured something. I will raise a PR to fix this linting issue soon.


only 64bit

Launch LND: 32bit not working

Yikes! What are your specs? OS version? Not sure that I'll be able to get this to build with 32 bit

old PC (Win7)

Sorry, I don't think I'll be able to fix this


Custom name for --resources-qube in qvm-create-windows-qube

It would be comfortable for additional automation tools if qvm-create-windows-qube  allows --resources-qube.
Right now I create disposable vm as windows-mgmt. It have random name. But then I can't use  qvm-create-windows-qube as it hardcoded to windows-mgmt

Feature has been added in: https://github.com/ElliotKillick/qvm-create-windows-qube/commit/dbbdcffe778cc53f6602a4b90c0a4862685da6d7
I decided to add it as an environment variable you can set instead because it's a pretty advanced use case. For example: RESOURCES_VM=disp1234 qvm-create-windows-qube -i ... -a ... test-win10 (or just export beforehand)

Would it be also possible to add RESOURCES_QUBE to installation script?
Because in the qubes-task it uses install.sh first to create the mgmt qube

resources_qube="${RESOURCES_QUBE:-windows-mgmt}"


Edit Admin rolebinding to add jenkins serviceaccount

[x] Have you created an issue and used it in the commit message?
https://issues.jboss.org/browse/OBST-219
[x] Have you checked to ensure there aren't other open Pull Requests for the same issue?
[x] Have you built the project locally prior to submission with mvn clean install?
Tested locally using minishift and this project "https://github.com/cmoulliard/spring-boot-configmap-booster"

Will that work in OSO free too?

If the user has the appropriate rights (= basic_user), that should be the case. To be tested

This error is reported during launchpad-mission creation of the project -> https://gist.github.com/cmoulliard/ca3961800d66cfa5e58e510c01094c47#file-gistfile1-txt-L23

Ticket updated : https://issues.jboss.org/browse/OBST-219?focusedCommentId=13396171&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-13396171

@gastaldi, @ALRubinger launchpad-mission control running locally using WildFlySwarm has been able:

To create a git project of the booster SpringBoot ConfigMap,
Create an OpenShift project (= zz1)
Create from the jenkins-ephemral template the jenkins (master/slave) pod
Launch the pipeline as defined within the project
To finally build & run the SpringBoot configMap on https://api.free-int.openshift.com

https://www.dropbox.com/s/kuoedd1s05rqdaj/Screenshot 2017-04-21 08.51.57.png?dl=0
http http://spring-boot-configmap-zz1.d800.free-int.openshiftapps.com/api/greeting
HTTP/1.1 200
Cache-control: private
{
"content": "Hello, World from Kubernetes ConfigMap!",
"id": 7
}
Remark : No permissions issues have been reported

Feedback about the PR ?

Merged based on your feedback

Do you know when this change will be part of the next release ?

When we release the next version, but you can test with master:
https://raw.githubusercontent.com/openshiftio/launchpad-templates/master/openshift/launchpad-template.yaml
Em 23 de abr de 2017 07:58, "Charles Moulliard"<EMAIL_ADDRESS>escreveu:

Do you know when this change will be part of the next release ?
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/openshiftio/launchpad-missioncontrol/pull/238#issuecomment-296435130,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AADTdV-MG6xIVyt5SBIZTl9ZuweGvn8gks5ryy7rgaJpZM4NC-bK
.


Version updates for strongSwan

strongSwan installation procedures for Ubuntu have changed since this article was written. strongswan-ikev2 and strongswan-plugin-eap-tls no longer exist as standalone packages, and have been rolled into strongswan package. libstrongswan-standard-plugins is autojmatically installed with strongswan. Also switch from old school apt-get to newer and more user friendly apt.

@doherty100 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Thank you for letting us know. I'll take this as a doc bug.  #please-close


Reduce spammy logging

@samoht observed a recent test run generated 227MiB of log files, mostly "Errors" explaining that the remote server refused a connection (as it is totally at liberty to do)
This PR:

increases minimum debug level to Info (was debug)
downgrades some "errors" to "debug"

I'd like to test this a bit manually with the stress test @samoht mentioned to check if the spam has been reduced enough.

I ran the spammy test and it was much reduced: down to ~300 lines.


[wip]feat: deno lint [skip ci]

[ ] Implement napi_create_external in napi-rs
[ ] Figure out whether the benchmark is fairly enough

First benchmark:
@node-rs/deno-lint x 287 ops/sec ±2.45% (80 runs sampled)
eslint x 56.22 ops/sec ±10.73% (60 runs sampled)
Lint benchmark bench suite: Fastest is @node-rs/deno-lint
[1]    75582 segmentation fault  node packages/deno-lint/benchmark/lint.js

Got segmentation fault because we need napi_create_external instead of napi_wrap

Segment fault cased by double free input source code Buffer
@node-rs/deno-lint x 282 ops/sec ±2.39% (79 runs sampled)
eslint x 56.84 ops/sec ±8.65% (65 runs sampled)
Lint benchmark bench suite: Fastest is @node-rs/deno-lint

After upgrade to<EMAIL_ADDRESS>@node-rs/deno-lint x 2,172 ops/sec ±0.91% (91 runs sampled)
eslint x 56.42 ops/sec ±8.18% (59 runs sampled)
Lint benchmark bench suite: Fastest is @node-rs/deno-lint


Observing history

Implemented the first version of the Observing History.

This PR implements a data table to list the exposures processed by QLF
@crisingulani nice job with the jquery data table plugin. I've just removed the QA and Targets tabs because they are not working yet.
Changes pending in the design: the QA tab will be implemented in the QA app and we are not going to list the Targets (at least for now).


kebab-case missing

I might have missed this, but is the kebab-case convention missing - e.g., helloThere => hello-there?

Woops - just found dasherize.


Add support for k8s secrets

As discussed https://github.com/istio/istio/pull/9508#issuecomment-435593826 it would be good to support k8s secrets, to enable (for example) username and password to be passed to Elasticsearch and Cassandra.

Due to recent PR https://github.com/jaegertracing/jaeger-operator/pull/82, we now have support for volumes/mounts which can be used to provide access to the secrets. This should be adequate for situations where the secrets are accessed via files (e.g. certificates).
However we still have the case where the secrets are being passed as parameters (e.g. ES/Cassandra passwords).
An example of how a container can be defined to pass such details is here.
This shows the values being passed as environment variables, which is an option with the Jaeger executables - instead of as parameters. We just need to provide a similar definition, that could sit alongside the options nodes, which instead captures the environment variable name (possibly lower case form automatically converted to appropriate uppercase representation), the secret name and key.

A possible configuration:
  storage:
    type: elasticsearch
    options:
      es:
        server-urls: http://elasticsearch:9200
        username: elastic
    secureOptions:
      - name: es.password
        secret: my-secret
        key: es-password

thoughts @jpkrohling ?

True - although not sure how else to pass the password to the Jaeger executables.

It would certainly require a change to the underlying Docker image. We would mount the secret on the container and a wrapper script would read the secret, passing the options to the binary.

support Secrets in our JaegerSpec

I think we could add it for the JaegerStorageSpec first only, to test waters. We can increase the scope later, if necessary.

add handlers in Deployments, Statefulsets to propagate these secrets into environment variables.

That's my understanding, yes.
I know that @objectiser already confirmed, but one thing is not clear to me:
      - name: es.password
        secret: my-secret
        key: es-password

If the key is the one used to specify the env var name, what's the purpose of the name property? Can't we have only a secret and a list of keys to use? Like:
      - secret: my-secret
        keys:
        - es-password
        - ...

Name is intended to be the env variable and key is key for the secret. So could be different values.

@annanay25 Any idea when you may be able to provide a PR for this? It may be required soon as part of the work to include the operator in the Istio helm chart. Thanks.

Hi @objectiser, I'll start working on this right away :)

@annanay25 Great, much appreciated!


Preprocessing

In the opening comments to this notebook, there is a reference to "our next two weeks". Depending on how the workshop is taught, this probably needs to be updated to the next two days or lessons.
Reading in .csv
How many entries are in "raw'? -> "tweets"
This notebook brings in numerous packages and libraries without introducing or explaining how to use them, such as codecs, BeautifulSoup, glob, etc. This could be confusing for learners who have not used these libraries before.
Add some open cells after the challenges for learners to insert their code.
Reading in multiple files -
There is missing text after "How..."
There are so many sections in this notebook. Numbering the sections might help learners to keep their place and follow along.

Need to explain/contextualize use of multiple libraries - or drop?

Need to explain/contextualize use of multiple libraries - or drop?

Anything helpful in Notebooks 4-4, 5-1, and 5-2 here? https://github.com/dlab-berkeley/DIGHUM101-2020

Anything helpful in Notebooks 4-4, 5-1, and 5-2 here? https://github.com/dlab-berkeley/DIGHUM101-2020

Hi @brooksjessup -- Can you commit and push these changes? Please close this comment when you are done. Let me know if you have any questions. Thanks!


Update bUnit docs

Fixes #30165
Following changes were made:

Showcasing .razor file based tests
Showcasing that TestContext should not be created in each test
Moved wording from "selecting" a button to "clicking"

CC @egil

Internal previews

📄 File
🔗 Preview link

aspnetcore/blazor/test.md
Test Razor components in ASP.NET Core Blazor

Even though we recommend just inheriting from TestContext to make tests easier to write, I don't think we should show that here since it only works with xUnit unless you take special steps. Users are likely to drop by bUnit.dev anyway to learn more.
We could add it as a tip with a deep link to the bUnit page where we discuss it.

The section afterward references bUnit - so I guess that's covered.
I'll revert that part - even though I am inclined to say that for an example it doesn't have to be 100% applicable for every user out of the box.

I'll review and update now.

Moved wording from "selecting" a button to "clicking"

I'll revert that on my edit pass. That's a MS Style Manual requirement because the docs are written for devices with and without keyboards. They felt that "select" was better than "click" for devices generally.

cc: @Rick-Anderson @tdykstra @wadepickett ... FYI: There's a bad link popping up on builds ...

aspnetcore/migration/proper-to-2x/mvc2.md
'https://vswebessentials.com/features/browserlink' points to a page  that doesn't exist. Check the path or URL and update the link.

When I hit that address, Firefox isn't to keen on my going there 😨 ...

Looks good to me.

When I hit that address, Firefox isn't too keen on my going there 😨 ...

Me neither :D :


Compile with LinterCop after release of AL 13 fails

Due to AL Language extension 13, we can no longer debug our AL extension against BC versions older than 20. So get around this issue, we manually have to Uninstall the latest version of the VSCode extension and install an older version. After changing the older version, LinterCop correctly shows warnings and errors in the problem tab.
But if I try to compile the app, then the we get lots of warnings. like:
warning AD0001: Analyzer 'BusinessCentral.LinterCop.Design.Rule0001FlowFieldsShouldNotBeEditable' threw an exception of type 'System.InvalidOperationException' with message 'System.InvalidOperationException: Nullable object must have a value.
at BusinessCentral.LinterCop.Design.Rule0001FlowFieldsShouldNotBeEditable.AnalyzeFlowFieldEditable(SymbolAnalysisContext ctx)
at Microsoft.Dynamics.Nav.CodeAnalysis.Diagnostics.AnalyzerExecutor.<>c__DisplayClass49_2.b__0() in D:\a_work\1\s\source\Prod\Microsoft.Dynamics.Nav.CodeAnalysis\DiagnosticAnalyzer\AnalyzerExecutor.cs:line 658
at Microsoft.Dynamics.Nav.CodeAnalysis.Diagnostics.AnalyzerExecutor.ExecuteAndCatchIfThrows_NoLock(DiagnosticAnalyzer analyzer, Action analyze, Nullable`1 info) in D:\a_work\1\s\source\Prod\Microsoft.Dynamics.Nav.CodeAnalysis\DiagnosticAnalyzer\AnalyzerExecutor.cs:line 1102'
But also lots of false errors like:
c:\xxx\Source\Features\EnvironmentSetup\EnvironmentFeature.Table.al(1,13): error LC0004: Property "LookupPageID" and "DrilldownPageID" must be filled in table "Environment Feature" because it is used in page "Environment Features" (list)
(The page have the properties)
The only option we have found is to disable LinterCop, when compiling.
Compilation works fine with LinterCop in our pipelines - where we use the latest version of both the compiler and LinterCop.

Same issue here. Using AL Extension v12.6.936426 and LinterCop <IP_ADDRESS>.

warning AD0001: Analyzer 'BusinessCentral.LinterCop.Design.Rule0025InternalProcedureModifier' threw an exception of type 'System.InvalidCastException' with message 'System.InvalidCastException: Unable to cast object of type 'Microsoft.Dynamics.Nav.CodeAnalysis.Syntax.EventDeclarationSyntax' to type 'Microsoft.Dynamics.Nav.CodeAnalysis.Syntax.MethodDeclarationSyntax'.
at BusinessCentral.LinterCop.Design.Rule0025InternalProcedureModifier.AnalyzeInternalProcedures(SyntaxNodeAnalysisContext ctx)
at Microsoft.Dynamics.Nav.CodeAnalysis.Diagnostics.AnalyzerExecutor.<>c__DisplayClass52_1.b__1() in D:\a_work\1\s\source\Prod\Microsoft.Dynamics.Nav.CodeAnalysis\DiagnosticAnalyzer\AnalyzerExecutor.cs:line 745
at Microsoft.Dynamics.Nav.CodeAnalysis.Diagnostics.AnalyzerExecutor.ExecuteAndCatchIfThrows_NoLock(DiagnosticAnalyzer analyzer, Action analyze, Nullable1 info) in D:\a\_work\1\s\source\Prod\Microsoft.Dynamics.Nav.CodeAnalysis\DiagnosticAnalyzer\AnalyzerExecutor.cs:line 1079' warning AD0001: Analyzer 'BusinessCentral.LinterCop.Design.Rule0025InternalProcedureModifier' threw an exception of type 'System.InvalidCastException' with message 'System.InvalidCastException: Unable to cast object of type 'Microsoft.Dynamics.Nav.CodeAnalysis.Syntax.EventDeclarationSyntax' to type 'Microsoft.Dynamics.Nav.CodeAnalysis.Syntax.MethodDeclarationSyntax'.    at BusinessCentral.LinterCop.Design.Rule0025InternalProcedureModifier.AnalyzeInternalProcedures(SyntaxNodeAnalysisContext ctx)    at Microsoft.Dynamics.Nav.CodeAnalysis.Diagnostics.AnalyzerExecutor.<>c__DisplayClass52_1.<ExecuteSyntaxNodeAction>b__1() in D:\a\_work\1\s\source\Prod\Microsoft.Dynamics.Nav.CodeAnalysis\DiagnosticAnalyzer\AnalyzerExecutor.cs:line 745    at Microsoft.Dynamics.Nav.CodeAnalysis.Diagnostics.AnalyzerExecutor.ExecuteAndCatchIfThrows_NoLock(DiagnosticAnalyzer analyzer, Action analyze, Nullable1 info) in D:\a_work\1\s\source\Prod\Microsoft.Dynamics.Nav.CodeAnalysis\DiagnosticAnalyzer\AnalyzerExecutor.cs:line 1079'
warning AD0001: Analyzer 'BusinessCentral.LinterCop.Design.Rule0024SemicolonAfterMethodOrTriggerDeclaration' threw an exception of type 'System.NullReferenceException' with message 'System.NullReferenceException: Object reference not set to an instance of an object.
at BusinessCentral.LinterCop.Design.Rule0024SemicolonAfterMethodOrTriggerDeclaration.AnalyzeSemicolonAfterMethodOrTriggerDeclaration(SyntaxNodeAnalysisContext ctx)
at Microsoft.Dynamics.Nav.CodeAnalysis.Diagnostics.AnalyzerExecutor.<>c__DisplayClass52_1.b__1() in D:\a_work\1\s\source\Prod\Microsoft.Dynamics.Nav.CodeAnalysis\DiagnosticAnalyzer\AnalyzerExecutor.cs:line 745
at Microsoft.Dynamics.Nav.CodeAnalysis.Diagnostics.AnalyzerExecutor.ExecuteAndCatchIfThrows_NoLock(DiagnosticAnalyzer analyzer, Action analyze, Nullable1 info) in D:\a\_work\1\s\source\Prod\Microsoft.Dynamics.Nav.CodeAnalysis\DiagnosticAnalyzer\AnalyzerExecutor.cs:line 1079' warning AD0001: Analyzer 'BusinessCentral.LinterCop.Design.Rule0024SemicolonAfterMethodOrTriggerDeclaration' threw an exception of type 'System.NullReferenceException' with message 'System.NullReferenceException: Object reference not set to an instance of an object.    at BusinessCentral.LinterCop.Design.Rule0024SemicolonAfterMethodOrTriggerDeclaration.AnalyzeSemicolonAfterMethodOrTriggerDeclaration(SyntaxNodeAnalysisContext ctx)    at Microsoft.Dynamics.Nav.CodeAnalysis.Diagnostics.AnalyzerExecutor.<>c__DisplayClass52_1.<ExecuteSyntaxNodeAction>b__1() in D:\a\_work\1\s\source\Prod\Microsoft.Dynamics.Nav.CodeAnalysis\DiagnosticAnalyzer\AnalyzerExecutor.cs:line 745    at Microsoft.Dynamics.Nav.CodeAnalysis.Diagnostics.AnalyzerExecutor.ExecuteAndCatchIfThrows_NoLock(DiagnosticAnalyzer analyzer, Action analyze, Nullable1 info) in D:\a_work\1\s\source\Prod\Microsoft.Dynamics.Nav.CodeAnalysis\DiagnosticAnalyzer\AnalyzerExecutor.cs:line 1079'
warning AD0001: Analyzer 'BusinessCentral.LinterCop.Design.Rule0052InternalProceduresNotReferencedAnalyzer' threw an exception of type 'System.MissingMethodException' with message 'System.MissingMethodException: Method not found: 'Microsoft.Dynamics.Nav.CodeAnalysis.Packaging.NavAppManifest Microsoft.Dynamics.Nav.Analyzers.Common.ManifestHelper.GetManifest(Microsoft.Dynamics.Nav.CodeAnalysis.Compilation)'.
at BusinessCentral.LinterCop.Design.Rule0052InternalProceduresNotReferencedAnalyzer.MethodSymbolAnalyzer..ctor(CompilationAnalysisContext compilationAnalysisContext)
at BusinessCentral.LinterCop.Design.Rule0052InternalProceduresNotReferencedAnalyzer.CheckApplicationObjects(CompilationAnalysisContext compilationAnalysisContext)
at Microsoft.Dynamics.Nav.CodeAnalysis.Diagnostics.AnalyzerExecutor.<>c__DisplayClass48_1.b__0() in D:\a_work\1\s\source\Prod\Microsoft.Dynamics.Nav.CodeAnalysis\DiagnosticAnalyzer\AnalyzerExecutor.cs:line 618
at Microsoft.Dynamics.Nav.CodeAnalysis.Diagnostics.AnalyzerExecutor.ExecuteAndCatchIfThrows_NoLock(DiagnosticAnalyzer analyzer, Action analyze, Nullable1 info) in D:\a\_work\1\s\source\Prod\Microsoft.Dynamics.Nav.CodeAnalysis\DiagnosticAnalyzer\AnalyzerExecutor.cs:line 1079' warning AD0001: Analyzer 'BusinessCentral.LinterCop.Design.Rule0025InternalProcedureModifier' threw an exception of type 'System.InvalidCastException' with message 'System.InvalidCastException: Unable to cast object of type 'Microsoft.Dynamics.Nav.CodeAnalysis.Syntax.EventDeclarationSyntax' to type 'Microsoft.Dynamics.Nav.CodeAnalysis.Syntax.MethodDeclarationSyntax'.    at BusinessCentral.LinterCop.Design.Rule0025InternalProcedureModifier.AnalyzeInternalProcedures(SyntaxNodeAnalysisContext ctx)    at Microsoft.Dynamics.Nav.CodeAnalysis.Diagnostics.AnalyzerExecutor.<>c__DisplayClass52_1.<ExecuteSyntaxNodeAction>b__1() in D:\a\_work\1\s\source\Prod\Microsoft.Dynamics.Nav.CodeAnalysis\DiagnosticAnalyzer\AnalyzerExecutor.cs:line 745    at Microsoft.Dynamics.Nav.CodeAnalysis.Diagnostics.AnalyzerExecutor.ExecuteAndCatchIfThrows_NoLock(DiagnosticAnalyzer analyzer, Action analyze, Nullable1 info) in D:\a_work\1\s\source\Prod\Microsoft.Dynamics.Nav.CodeAnalysis\DiagnosticAnalyzer\AnalyzerExecutor.cs:line 1079'
warning AD0001: Analyzer 'BusinessCentral.LinterCop.Design.Rule0024SemicolonAfterMethodOrTriggerDeclaration' threw an exception of type 'System.NullReferenceException' with message 'System.NullReferenceException: Object reference not set to an instance of an object.
at BusinessCentral.LinterCop.Design.Rule0024SemicolonAfterMethodOrTriggerDeclaration.AnalyzeSemicolonAfterMethodOrTriggerDeclaration(SyntaxNodeAnalysisContext ctx)
at Microsoft.Dynamics.Nav.CodeAnalysis.Diagnostics.AnalyzerExecutor.<>c__DisplayClass52_1.b__1() in D:\a_work\1\s\source\Prod\Microsoft.Dynamics.Nav.CodeAnalysis\DiagnosticAnalyzer\AnalyzerExecutor.cs:line 745
at Microsoft.Dynamics.Nav.CodeAnalysis.Diagnostics.AnalyzerExecutor.ExecuteAndCatchIfThrows_NoLock(DiagnosticAnalyzer analyzer, Action analyze, Nullable1 info) in D:\a\_work\1\s\source\Prod\Microsoft.Dynamics.Nav.CodeAnalysis\DiagnosticAnalyzer\AnalyzerExecutor.cs:line 1079' warning AD0001: Analyzer 'BusinessCentral.LinterCop.Design.Rule0033AppManifestRuntimeBehind' threw an exception of type 'System.MissingMethodException' with message 'System.MissingMethodException: Method not found: 'Microsoft.Dynamics.Nav.CodeAnalysis.Packaging.NavAppManifest Microsoft.Dynamics.Nav.Analyzers.Common.ManifestHelper.GetManifest(Microsoft.Dynamics.Nav.CodeAnalysis.Compilation)'.    at BusinessCentral.LinterCop.Design.Rule0033AppManifestRuntimeBehind.CheckAppManifestRuntime(CompilationAnalysisContext ctx)    at Microsoft.Dynamics.Nav.CodeAnalysis.Diagnostics.AnalyzerExecutor.<>c__DisplayClass48_1.<ExecuteCompilationActionsCore>b__0() in D:\a\_work\1\s\source\Prod\Microsoft.Dynamics.Nav.CodeAnalysis\DiagnosticAnalyzer\AnalyzerExecutor.cs:line 618    at Microsoft.Dynamics.Nav.CodeAnalysis.Diagnostics.AnalyzerExecutor.ExecuteAndCatchIfThrows_NoLock(DiagnosticAnalyzer analyzer, Action analyze, Nullable1 info) in D:\a_work\1\s\source\Prod\Microsoft.Dynamics.Nav.CodeAnalysis\DiagnosticAnalyzer\AnalyzerExecutor.cs:line 1079'
Unhandled exception. System.InvalidCastException: Unable to cast object of type 'Microsoft.Dynamics.Nav.CodeAnalysis.Syntax.TriggerDeclarationSyntax' to type 'Microsoft.Dynamics.Nav.CodeAnalysis.Syntax.GlobalVarSectionSyntax'.
at BusinessCentral.LinterCop.Design.Rule0044AnalyzeTransferFields.FindGlobalVariables(SyntaxNode node)
at BusinessCentral.LinterCop.Design.Rule0044AnalyzeTransferFields.AnalyzeTransferFields(OperationAnalysisContext ctx)
at System.Threading.Tasks.Task.<>c.b__128_1(Object state)
at System.Threading.QueueUserWorkItemCallback.<>c.<.cctor>b__6_0(QueueUserWorkItemCallback quwi)
at System.Threading.ExecutionContext.RunForThreadPoolUnsafe[TState](ExecutionContext executionContext, Action`1 callback, TState& state)
at System.Threading.QueueUserWorkItemCallback.Execute()
at System.Threading.ThreadPoolWorkQueue.Dispatch()
at System.Threading.PortableThreadPool.WorkerThread.WorkerThreadStart()
at System.Threading.Thread.StartCallback()
[Info  - 12:36:59 PM] Connection to server got closed. Server will restart.

Currently the releases of the LinterCop are only build against the latest version of the AL Language.
I'm looking into to build against multiple AL Languages, where we maybe can resolve this.

It's going to be challenging and some rules will are not going to work on previous versions of the AL Languages.

Working together with @StefanMaron to support multiple version of the AL Language.

I've did an overhaul of the build pipeline, to build against multiple version of the AL Language. The next step is to update the VS Code extension to retrieve the matching artifact based on the installed version of the AL Language in VS Code.
Currently we're down to version 12.0 of the AL Language, it this sufficiënt of do we need to go further back? (11, 10, 9, ..)

That's great! For us, v12(.7.964847) is enough. We use it to be able to debug against BC14.

Sounds great. V12 is also good enough for us.
But how do we download these? Cannot see these version from the Uninstall->Install Another Version in VSCode.
And how about the Powershell script where we now download via https://api.github.com/repos/StefanMaron/BusinessCentral.LinterCop/releases?

Good news!

The combination of the new v0.30.18 version of the LinterCop, combined with the new 0.1.7 version of the VS Code now supports multiple version of the AL Language.
This works automatically on the background, where the VS Code extension grabs the version of the installed AL Language and then retrieves the a matching LinterCop .dll file.

Looks like LinterCop works again in VSCode. Thanks!
But how do we download the correct version for our Pipelines?

@epernst : In our pipelines we always retrieve the latest AL Language from the marketplace and use that to compile the App.
You can find documentation on this here: https://github.com/StefanMaron/BusinessCentral.LinterCop/blob/master/.assets/DevOps.md
Is this a possible solution?

That's basically the same script we run today. For our pipelines we download the latest AL Language VSIX and then use the script to download LinterCop.
But we keep getting these errors for our v14 builds (works fine with later version i.e. 23.4):
warning AL1003: An instance of analyzer BusinessCentral.LinterCop.Design.Rule0001FlowFieldsShouldNotBeEditable cannot be created from C:\bcartifacts.cache\onprem\14.44.49619.0\vsix\extension\bin\Analyzers\BusinessCentral.LinterCop.dll

I'm now down to that it works with your download scripts and my compile scripts, when building a local container.
But it fails when I do the same using ALOps!

I'm unsure what's wrong in your ADO setup. We're also using ALOps, and works without any issues for us.
I'm getting the idea it's maybe a ADO setup/configration issue (and not an LinterCop issue).
Let's continue the conversation on ALOps to figure out what's wrong.

I just created a simpler repo pipeline (without any external scripts or artifacts), but I could not recreate the errors.
Then I ran some of our other failing pipelines again, and it seems that the issue somehow was fixed Friday! We didn't run anything yesterday, as this was a holiday here in Denmark.
I'm not sure if ALops or LinterCop was updated, or what else happened. But not getting any compile errors or warnings, except the ones we expect.


Editor - Show blank spaces suggestions

From<EMAIL_ADDRESS>on 2015-02-14T12:29:43Z
Spyder Version:  2.4.0dev (4931: r1ff46f376d07 )
Python Version:  2.7.9
Qt Version    :  4.8.5, PySide 1.2.1 on Linux
pyflakes >=0.5.0:  0.5.0 (OK)
pep8 >=0.6      :  1.4.6 (OK)
IPython >=1.0   :  2.3.1 (OK)
zmq >=2.1.11    :  14.4.1 (OK)
pygments >=1.6  :  2.0.1 (OK)
pandas >=0.13.1 :  0.15.2 (OK)
sphinx >=0.6.6  :  1.2.3 (OK)
psutil >=0.3    :  2.1.1 (OK)
jedi >=0.8.1    :  None (NOK)
rope >=0.9.2    :  0.10.2 (OK)
matplotlib >=1.0:  1.4.2 (OK)
sympy >=0.7.0   :  0.7.6 (OK)
pylint >=0.25   :  None (NOK)
What steps will reproduce the problem?

Activating the 'Show blank spaces' in the Source menu

What is the expected output? What do you see instead?
The function works but I have some suggestions/comments...
Please provide any additional information below
.
I would suggest:
1.) The dots are the same color as main text, they should be lighter, to provide guidance without being too intrusive
2.) It would be nice to sync the indentation characters selected in the Editor Advanced preferences to display the same, so if 4 spaces is selected, every 4 spaces, the dot should become a bar | or should be a bit different to show this clearly
3.) I think having dots for the indentation before any content in the line is more than enough. The idea of having this is to guide with indentation, but right now every single space is "highlighted", which can be distracting in the middle of definition of variables and statements of code
Original issue: http://code.google.com/p/spyderlib/issues/detail?id=2175

From ccordoba12 on 2015-02-15T15:18:53Z
Good suggestions. About them:

No idea how that could be done. I just activated a Qt option to show blank spaces in QPlainTextEdit widgets. So I don't know how dots color is selected.

Vertical bars can be activated too, but that's not related to blank spaces, that's a different issue. So please open a new one for that.

That's a Qt limitation. I know, a bit ugly, but I did a bit of research and it seems there is no way to change it. Could be wrong though :-)

Status: HelpNeeded
Labels: Cat-Editor

Well a direct way of doing it is not available on the Qt side, but there is a way but seems to convoluted just for this.... but it is something like this:
An editor would be in reality two, the one that is displayed and the one that holds the information, the one that is displayed will have the characters replaced but whatever our needs and the one that is on the "background"  will hold the original text, but they should remain in sync ... but seems like a lot for such a simple thing, unless there is something extra we could gain from this approach....

@goanpeca Yes, there is: Vim support :-) We need to Editors for Vim support: one in command mode and in edit mode.
But it's true, it's too much work...

Hmmm, just out of curiosity I will give it a quick go (just to see if it can work) but I will not go in depth in keeping the rest functional.... completio etc... just to see if it can work or not..

FYI, I think it's possible to enable vim shortcuts without  having two editors, see https://github.com/Nodd/spyderplugins.vim

A brighter color or different glyph for the spaces would indeed be nice. Apparently more Spyder users are interested: https://bugreports.qt.io/browse/QTBUG-46072
There are several different syntax coloring groups already. Would it be possible to add a 'space' syntax group and assign a specific color to that?
With respect to the last suggestion, I like to see all my spaces, especially the trailing ones.

The problem is we don't have access to the dots that represent blank spaces to give them a different color. They are not real dots, they are just added by Qt and have the same color than regular text.

But we should be able to change the color of the regular text for the spaces (or tabs), right ? (Even if we can't change the dots themselves).

Yes it is possible:

I've taken some syntax highlighter example code and modified it such that it shows and highlights spaces by adding a rule for a single space.
In MainWindow.setupEditor():
        opts = self.editor.document().defaultTextOption()
        opts.setFlags(opts.flags() | QtGui.QTextOption.ShowTabsAndSpaces)
        self.editor.document().setDefaultTextOption(opts)

In Highlighter.__init__():
        spaceFormat = QtGui.QTextCharFormat()
        spaceFormat.setForeground(QtCore.Qt.red)
        self.highlightingRules.append((QtCore.QRegExp(" "), spaceFormat))

Perhaps I'll make fork of Spyder and make a pull request. This should be enough information for someone else to pick it up in case I won't get around to it.

A first attempt that works well enough for me: https://github.com/hugobuddel/spyder/commit/8704814eb4055c7c7fa268ac52e2235642ffb3e1
It simply assigns the comment color to the new space pattern. The default comment color is a bit lighter than main text, which was my immediate goal. The spaces within colored texts, e.g. within a string, retain the surrounding color. The space pattern has to be at the start of the join at the end of make_python_patterns() because otherwise some keywords will match the space before them.
It might be too much work for me to generalize this, given my very limited experience with Spyder (and Qt). Therefore I didn't create a pull request just now. Does this warrant a user-defined color? Does this solution work with other schemes? Are there problematic edge-cases? Etc.
Perhaps a more beautiful solution would be to just make every space a bit brighter than the surrounding text, or a bit darker for dark schemes.

@hugobuddel, it's really great that you've found a way to do this!! Kudos to you! :-)
But please, please make a pull request so we can better discuss not only your code but the colors assigned to spaces.

Done @ccordoba12 !
Pull request made, with different code. I'll add some screenshots to the pull request.


ERROR Request timedout after 30 seconds

Hi,
I have issue when parsing a file with 7 thousand + records. That xlsx just have 2 columns.
I have tried to increase the time limit by adding "ini_set('max_execution_time', 300);" or "set_time_limit(300)" on top of my script but it's not working. Next, I even try to set the php.ini but still the same error comes out. So is there any ways I can increase the exception time when parsing a xlsx file?
Thanks in advance.
MaxC

7k records - 1-3 sec, pls optimize other code, database inserts/updates, for instance

I have the same issue using SimpleXLSX for large amount of data too like 10k+ records... so there is no way to change the maximum time out limit except optimizing other code?


Typo "@microsoft/～-sharepoint"→"@microsoft/～-sharepoint"

Bold with escape characters
https://docs.microsoft.com/en-us/sharepoint/dev/spfx/release-1.5
Category

[ ] Content fix
[ ] New article
[x] Example checked item (delete this line)

DELETE THIS LINE BEFORE SUBMITTING | For the above list, an empty checkbox is [ ] as in [SPACE]. A checked checkbox is [x] with no space between the brackets. Use the PREVIEW tab at the top right to preview the rendering before submitting your issue.

Related issues

fixes #issuenumber
partially #issuenumber
mentioned in #issuenumber

DELETE THIS LINE BEFORE SUBMITTING | If this fixes (aka: closes) or references an issue, please reference it here. This helps maintaining the issue list as it will (1) link the PR to the issue & (2) automatically close the issue when this PR is merged in.

What's in this Pull Request?

DELETE THIS LINE BEFORE SUBMITTING | Please describe the changes in this PR. Sample description or details around bugs which are being fixed.

Submission guidelines

!!IMPORTANT!! - All submissions must complete the baseline sections included in this template. Ignoring or deleting this template may result in closing the issue with the label type:incomplete-submission.
Follow our guidance on How To Create Good Pull Requests.
Target the master branch of this repo. Released documents are in live branch.

DELETE THIS SECTION BEFORE SUBMITTING

Thank you for your submission... in the future, please use the PR template provided to explain what the changes are that you are submitting and why so we don't have to look at the code to figure it out.


Rendering problem

During the rotation, there's a rendering problem as show in the video and picture.
https://user-images.githubusercontent.com/47852787/169686307-a4bece59-d991-424a-8b9f-f1e8e595a0e9.mp4

Do you have the code? Did you install Manim Rubikscube from PyPI? What version of Manim?
I have seen various rendering bugs in the past, but typically it's a problem of the way Manim's animations work, not a fault of the plugin. I haven't been keeping up with the stuff too much as of late, but I'm assuming that is still the case.

It's the same code as shown in the https://manim-rubikscube.readthedocs.io/en/stable/ AllTogetherExample. It seems your example also has this bug.

Thanks for your response. Have you tried that? It seems did not work for me.


Ability to stop collection of Device OS Info

Before opening a feature request against this repo, consider whether the feature is generic enough for the SDK, or it should be a specific plugin/extension to the SDK.
Is your feature request related to a problem?
There is no way to disable collecting device OS version using the SDK. As of now, you can't pass null, empty string or any arbitrary string to semanticContext->SetOsVersion() to turn off or redact the OS version field.
Describe the solution you'd like.
Add ability to disable collecting OS version. Ideally, make semanticContext->SetOsVersion() accept null, empty string, "redacted" or take an option to turn it off.
Describe alternatives you've considered.
Additional context.

@lulululululu - you may use custom decorator to clear it, OR is there a reason why you cannot clear that data (drop it) server-side?

As discussed, please use custom decorator to clear the field.

Closing, please use as suggested by Max above.


ERROR unknown command "vender" in mac os x

hi, i had success install gb and build some project , but some error like there when i use vendor command.
 >gb vender fetch github.com/constabulary/gb
ERROR unknown command "vender"
Usage:
  gb build [flags] [package] - build a package
  gb plugin [flags] [package] - run a plugin
  gb test [flags] [package] - test a package

Flags:
  -R="/Users/qinshen/git/go-project/dashboard-backend-go": set the project root
  -goroot="/usr/local/Cellar/go/1.4.2/libexec": override GOROOT
  -q=false: suppress log messages below ERROR level
  -v=false: enable log levels below INFO level

would someone help for this ??
thanks a lot.

I think you have an old version of gb, please follow the upgrade instructions on the website. getgb.io/doc/install

On 20 Jun 2015, at 15:54, TsingsonQin<EMAIL_ADDRESS>wrote:
hi, i had success install gb and build some project , but some error like there when i use vendor command.

gb vender fetch github.com/constabulary/gb
ERROR unknown command "vender"
Usage:
gb build [flags] [package] - build a package
gb plugin [flags] [package] - run a plugin
gb test [flags] [package] - test a package

Flags:
-R="/Users/qinshen/git/go-project/dashboard-backend-go": set the project root
-goroot="/usr/local/Cellar/go/1.4.2/libexec": override GOROOT
-q=false: suppress log messages below ERROR level
-v=false: enable log levels below INFO level
would someone help for this ??
thanks a lot.
—
Reply to this email directly or view it on GitHub.

Try vendor instead vender

@andevery , thanks.

 >gb vendor fetch github.com/constabulary/gb
ERROR unknown command "vendor"
Usage:
  gb build [flags] [package] - build a package
  gb plugin [flags] [package] - run a plugin
  gb test [flags] [package] - test a package

Flags:
  -R="/Users/qinshen/git/go-project/dashboard-backend-go": set the project root
  -goroot="/usr/local/Cellar/go/1.4.2/libexec": override GOROOT
  -q=false: suppress log messages below ERROR level
  -v=false: enable log levels below INFO level

@davecheney,  thanks, i will try to upgrade to new one.

gb plugins start with gb-, check the directory that gb is installed in,
probably $GOPATH/bin, there should be a gb-vendor command there as well. If
not,
http://getgb.io/docs/install/
On Sat, Jun 20, 2015 at 5:37 PM, TsingsonQin<EMAIL_ADDRESS>wrote:

@andevery https://github.com/andevery , thanks.

gb vendor fetch github.com/constabulary/gb
ERROR unknown command "vendor"
Usage:
gb build [flags] [package] - build a package
gb plugin [flags] [package] - run a plugin
gb test [flags] [package] - test a package

Flags:
-R="/Users/qinshen/git/go-project/dashboard-backend-go": set the project root
-goroot="/usr/local/Cellar/go/1.4.2/libexec": override GOROOT
-q=false: suppress log messages below ERROR level
-v=false: enable log levels below INFO level
@davecheney https://github.com/davecheney, thanks, i will try to
upgrade to new one.
—
Reply to this email directly or view it on GitHub
https://github.com/constabulary/gb/issues/232#issuecomment-113721263.

@davecheney  thanks.
in my mac os X 10.10 and 10.11 ( beta )
build the gb-vendor via gb and copy or set the gb-vendor to $PATH, then it work!
 >cd ../example-gsftp/
 >gb vendor list
github.com/kr/fs	https://github.com/kr/fs	master	2788f0dbd16903de03cb8186e5c7d97b69ad387b
golang.org/x/crypto/ssh	https://go.googlesource.com/crypto/ssh	master	c10c31b5e94b6f7a0283272dc2bb27163dcea24b
github.com/pkg/sftp	https://github.com/pkg/sftp	master	f234c3c6540c0358b1802f7fd90c0879af9232eb
 >

thanks for help.

I have the same issue, but during build commit on Gitlab CI:
$ go get -u github.com/constabulary/gb/...
$ ls -la $GOPATH/bin
total 20088
drwxr-xr-x 2 root root     4096 Jan 29 10:54 .
drwxrwxrwx 5 root root     4096 Jan 29 10:54 ..
-rwxr-xr-x 1 root root 10191936 Jan 29 10:54 gb
-rwxr-xr-x 1 root root 10366816 Jan 29 10:54 gb-vendor
$ $GOPATH/bin/gb vendor restore
FATAL: unknown command "vendor"
gb, a project based build tool for the Go programming language.

Usage:

        gb command [arguments]

The commands are:

        build       build a package
        doc         show documentation for a package or symbol
        env         print project environment variables
        generate    generate Go files by processing source
        info        info returns information about this project
        list        list the packages named by the importpaths
        test        test packages

Use "gb help [command]" for more information about a command.

Additional help topics:

        plugin      plugin information
        project     gb project layout

Use "gb help [topic]" for more information about that topic.

What is wong with this case?

What does
which go-vendor

Report ?

On 29 Jan 2016, at 22:20, Alex Zemlyakov<EMAIL_ADDRESS>wrote:
I have the same issue, but during build commit on Gitlab CI:
$ go get -u github.com/constabulary/gb/...
$ ls -la $GOPATH/bin
total 20088
drwxr-xr-x 2 root root     4096 Jan 29 10:54 .
drwxrwxrwx 5 root root     4096 Jan 29 10:54 ..
-rwxr-xr-x 1 root root 10191936 Jan 29 10:54 gb
-rwxr-xr-x 1 root root 10366816 Jan 29 10:54 gb-vendor
$ $GOPATH/bin/gb vendor restore
FATAL: unknown command "vendor"
gb, a project based build tool for the Go programming language.
Usage:
    gb command [arguments]

The commands are:
    build       build a package
    doc         show documentation for a package or symbol
    env         print project environment variables
    generate    generate Go files by processing source
    info        info returns information about this project
    list        list the packages named by the importpaths
    test        test packages

Use "gb help [command]" for more information about a command.
Additional help topics:
    plugin      plugin information
    project     gb project layout

Use "gb help [topic]" for more information about that topic.
What is wong with this case?
—
Reply to this email directly or view it on GitHub.

Yeah, it doesn't know where to find gb-vendor. Fixed. Thank you!

It's possible that whereever $GOPATH/bin was, it wasn't in your $PATH, and
the gb that was in your path was stale.
On Fri, Jan 29, 2016 at 11:31 PM, Alex Zemlyakov<EMAIL_ADDRESS>wrote:

Yeah, it doesn't know where to find gb-vendor. Fixed. Thank you!
—
Reply to this email directly or view it on GitHub
https://github.com/constabulary/gb/issues/232#issuecomment-176730240.


No sound in Windows 7

OS:
Windows 7 Enterprise
Issue Descriptions:
On occasion, GPMDP runs, but no sound is output. The Windows Sound mixer (SndVol.exe) shows a blank space.
Steps to Reproduce:
After listening to music all day, I pause the music (but don't shut down GPMDP) and put the computer into Hibernate mode at night. The next morning, I click Play, and the track starts playing, but there is no sound. If I open the Sound mixer (SndVol.exe), there is one (sometimes 2!) blank spaces where GPMDP should be. However, unlike #974, the volume and mute bars don't work at all (i.e. they're not there, rather than just hidden by an oversized icon).

The above image shows what I mean.
I can reliably recreate it by following the steps above. Completely stopping/restarting GPMDP solves the problem.

This seems like an upstream chromium bug.
Unable to reproduce on latest master, I'm assuming an update in Electron fixed it 👍


from hardware view, no data points

Hardware page:
There is an issue of no data data points. http://<IP_ADDRESS>:9286/cluster-view/hardware/detail.html?instance=<IP_ADDRESS>:9100
seems remove url instance's var port will make it work.
https://github.com/Microsoft/pai/blob/e4244ee08d29f33d914a2648049247171efbaf7d/webportal/src/app/cluster-view/hardware/hardware-detail.component.ejs
Call Grafana from url
Not work:
http://<IP_ADDRESS>:3000/dashboard/db/pai_nodeview?var-node=<IP_ADDRESS>:9100
Work:
http://<IP_ADDRESS>:3000/dashboard/db/pai_nodeview?var-node=<IP_ADDRESS>

Fixing in https://github.com/Microsoft/pai/pull/543


Cannot find symbol

Hi,
Trying to execute your code, I cannot find classes AfterRecording and ApplerWrapper in ApplerPrameters.java and Extensions.java. They seem to be missing in the project, is that right?
Thanks!

Did you download SRAExtender?


[WFLY-7692] part 2, add same default value and validator to MAX_AJP_PACKET_SIZE in ModClusterDefinition.

https://issues.jboss.org/browse/WFLY-7692
Add default value and validator to MAX_AJP_PACKET_SIZE in ModClusterDefinition as previous fix in https://github.com/wildfly/wildfly/pull/9693 missed in that resource definition.

retest this please


Fix issue when tapping next to a Thread #192

@tomsmith8 I reviewed and tested this. It's approved. I will merge it into a branch on our repo so then I can include it in the next release

Awesome, thank you!


chore(package): update documentation to version 12.2.0

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Use that fancy discord bot list upvote endpoint

To add a useless feature, but i want to so kek

uwu


Different results across different browsers

Hi @tutaf
Thanks for the feedback!
The accuracy can be low because of:

Custom browser settings or flags - The demo was designed for the default setup, but that doesn’t mean your custom setup is not vulnerable.
Poorly performant hardware (including virtual machines) - Some timings are just hardcoded and were tested on the MacBook hardware.
Fullscreen mode - The demo will work faster and more accurate if the browser is not in a fullscreen mode
Slow internet connection
Gestures during the process

I also made a special branch for Chromium (Chrome, Brave, Edge, etc.) that works much slower, but should be more accurate.
https://609d9f4d79c4f6000700782c--boring-visvesvaraya-dbefd4.netlify.app/
It still may not work for your browser with a custom configuration. Like downloading PDFs instead of opening it.


Consider setting CHIP_CONFIG_RMP_RETRANS_TABLE_SIZE to the min of pool size and exchange context count

Problem
Right now if we have a limited number or possible exchange contexts but a large packetbuffer pool, we can end up with a large CRMP retransmit table.
Proposed Solution
Take both values into account when defining the retransmit table size.
@yufengwangca Do you have time for this?

Yes, as I said in the email, current setting is based on the assumption exchange size is no less than buffer pool size.  In Weave, we set the maximum exchange pool size based on the available buffer pool size, so we can guarantee  that.
But take the min of two is better.


fix kubernetes-dashboard: https://github.com/kubernetes-sigs/kubespra…

…y/issues/5347#issuecomment-580099787
Signed-off-by: siaimes<EMAIL_ADDRESS>

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1441

Relevant Lines:
3477

💛  - Coveralls