vCPU thaw triggering VMEXIT_ABORT

In [docker/for-mac#85] the report with diagnostic bundle ID 264231A3-7008-4FAA-9D89-83D089716817/20160817-153827 shows a hyperkit crash on wake: (hostname and username redacted)
Aug 17 14:59:30 Docker[com.docker.driver.amd64-linux][27183] <Notice>: System wants to go to sleep
Aug 17 14:59:30 Docker[com.docker.driver.amd64-linux][27183] <Notice>: Asking com.docker.hyperkit to freeze vcpus
Aug 17 14:59:31 Docker[com.docker.driver.amd64-linux][27183] <Notice>: vcpu 3 waiting for signal to resume
Aug 17 14:59:31 Docker[com.docker.driver.amd64-linux][27183] <Notice>: vcpu 0 waiting for signal to resume
Aug 17 14:59:31 Docker[com.docker.driver.amd64-linux][27183] <Notice>: vcpu freeze complete: allowing sleep
Aug 17 14:59:31 kernel[kern][0] <Notice>: PM response took 105 ms (27183, com.docker.drive)
Aug 17 14:59:31 Docker[com.docker.driver.amd64-linux][27183] <Notice>: vcpu 1 waiting for signal to resume
Aug 17 15:00:10 Docker[com.docker.driver.amd64-linux][27183] <Notice>: System wants to wake up
Aug 17 15:00:10 Docker[com.docker.driver.amd64-linux][27183] <Notice>: Asking com.docker.hyperkit to thaw vcpus
Aug 17 15:00:10 Docker[com.docker.driver.amd64-linux][27183] <Notice>: vcpu 3 received signal, resuming
Aug 17 15:00:10 Docker[com.docker.driver.amd64-linux][27183] <Notice>: vm exit[3]
Aug 17 15:00:10 Docker[com.docker.driver.amd64-linux][27183] <Notice>:       reason          VMX
Aug 17 15:00:10 Docker[com.docker.driver.amd64-linux][27183] <Notice>:       rip             0xffffffff8103b2e4
Aug 17 15:00:10 Docker[com.docker.driver.amd64-linux][27183] <Notice>:       inst_length     3
Aug 17 15:00:10 Docker[com.docker.driver.amd64-linux][27183] <Notice>:       status          0
Aug 17 15:00:10 Docker[com.docker.driver.amd64-linux][27183] <Notice>: virtio-net-vpnkit: initialising, opts="uuid=...
Aug 17 15:00:10 Docker[com.docker.driver.amd64-linux][27183] <Notice>:       exit_reason     2
Aug 17 15:00:10 Docker[com.docker.driver.amd64-linux][27183] <Notice>: Interface will have uuid 7916572e-0a15-49fc-93a3-b8bb31c3ed15
Aug 17 15:00:10 Docker[com.docker.driver.amd64-linux][27183] <Notice>:       qualification   0x0000000000000000
Aug 17 15:00:10 Docker[com.docker.driver.amd64-linux][27183] <Notice>:       inst_type               0
Aug 17 15:00:10 Docker[com.docker.driver.amd64-linux][27183] <Notice>:       inst_error              0
Aug 17 15:00:10 Docker[com.docker.driver.amd64-linux][27183] <Notice>: Connection established with MAC=c0:ff:ee:c0:ff:ee and MTU 1500

It looks like it died in vmexit_vmx with VMEXIT_ABORT

I think docker 4 mac has since disabled the freezing/thawing of vcpus over suspend (which itself was a workaround for an older osx Hypervisor framework bug which appears to have been fixed in OSX) by default. Given that I'm inclined to not pursue this issue further.
We should maybe even consider dropping the code from hyperkit altogether, although I think for now it is still something one can opt into in docker4mac in case you trip over the issue (e.g. you are running an older OSX).


Jupyter autocomplete is super slow

Import pandas
Read in a dataframe
Bring up autocomplete for a dataframe
Takes like 10 seconds
Additionally it shows an invalid type for the icon:

That's a series and it shows the module icon.
It should come from this list here:
https://code.visualstudio.com/docs/editor/intellisense
I've also seen it show snippets.

@DonJayamanne @rchiodo @IanMatthewHuff @DavidKutu
Please find out what VSCode's plans are for supporting incremental intellisense window updates.

Unable to find it, Possible been closed since.
Also, it isn't on their iteration plan for Jan.

We need to do a bit more investigation on this as we are getting reports that intellisense/autocomplete is noticeably slower in the interactive window and notebook editor than it is in a regular python file.

What if we were to turn off the language sever and use Jedi instead?
Is this due to delays in getting a response from Jupyter autocompletion?

All of this slowdown is in Jupyter itself, so changing the extension's language server makes no difference. We should have fixed this in the latest release though. At least we don't wait anymore.

so changing the extension's language server makes no difference.

Yes, agreed. that comment was added before I remembered the jupyter slowness.. should have deleted that. will do so now.

We should have fixed this in the latest release though. At least we don't wait anymore.

Agreed.

So this item has become, fix jupyter's autocomplete in the jupyter code base?

Probably warrants further discussion.


Get the secret from the parameter store.

Also, update the tests to mock the SSM call.
I've also updated some of the dependencies.

Scala steward ignores the AWS dependencies because they release a new version every day and it's annoying. The AWS utils one was still using the dependency from S3 which Scala steward can't read so it will never update it.


How to reproduce the results in the paper

Hi, sorry to bother you.
In the released code of smaple_lora_int_model.py, is seems only one image is sampled as a toy example. How to reproduce the results of 500K samples and save them as npz format, to reproduce results reported in the paper?
If it is convenient to share the related code? Thank you very much !

I also have another question, why the DDIM of ImageNet uses 20 timesetps while the code released seems uses more timestep 100? Does this modification influence the performance?

Hi, you may refer to another issue  to reproduce the results. Also, the steps of diffusion models can be adjusted as you wish.

I will try it, thanks for your reply!

Hi, authors!
I follow the instruction in README, and use the released code with 100 timesteps using DDIM on ImageNet, and use the guided-diffusion codebase for evaluate.
However, I still can not reproduce the results in Table2. Since the 50000 smapling is too long, I use 30000 instead. The results of W4A4 (the default setup in the code) of my reproduce is as follows:

I also use the pre-trained ckpt (20 DDIM steps) you have provided and the results are as follows:

The results reported in the paper is 250.90 ||  6.17 || 7.75.
I suspect the above failure of reproduce might be caused by the sample generation process. I write the sampling code by myself following the smaple_lora_inimodel. But it seems not work well for reproduce.
Could you give me some help about this? Thx!


fix: avoid directives that inherit from components to placate ivy template type checking

As described in the comments on #1982, Ivy does not appreciate it when directives inherit from components. I found three places where this happens:

MdcRippleDirective extends MdcRippleComponent
MdcSelectIcon extends  MdcIcon
MdcTextFieldIcon extends MdcIcon

I apologize in advance if any of these attempts are not done correctly (I'm well out of my comfort zone here), as well as for any needless change caused by code formatting. Perhaps a more opinionated .editorconfig would help.
The last two cases are similar and simpler: as each is apparently designed to augment an <mdc-icon> element, I figured that simply eliminating the inheritance could be performed without negative impact.
Ripple was more challenging. As the component has effectively no template, I decided to try to implement both with a single directive, as is described in the comments inside the commit. Initially, I tried defining a type alias for MdcRippleComponent, in order to be able to fool any existing code (including the tests). This was unsuccessful, as apparently an actual JS object must exist. Therefore, I had to make a couple of small changes to the tests as well, namely replacing instances of MdcRippleComponent with MdcRippleDirective and changing the way that the test grabs the TS controller object from componentInstance to injector.get(). The meat of the tests themselves are unmodified, and hopefully that is acceptable.
I was able to at least build and run an app using Angular 9.0.0-rc0 out of the box using this branch, and hope that it is at least of some use to others, if only as a stepping stone towards being done more skillfully by somebody more intimately familiar with Angular internals than I.

@rapropos Thank you! Gonna merge and publish release candidate npm package under the next tag.


Prefers Site Users Only shouldn't impact curators

Currently unchecking Site Users Only on sites/edit effects the curators that are shown on the people page which shouldn't be the case

more info here https://groups.google.com/forum/#!topic/inaturalist/_UI2ELmcZVg


Error running

C:\Users\fodag\Desktop\DiscordBridge-main>npm start

<EMAIL_ADDRESS>start
node dist/index.js

node:internal/modules/cjs/loader:936
throw err;
^
Error: Cannot find module 'C:\Users\fodag\Desktop\DiscordBridge-main\dist\index.js'
at Function.Module._resolveFilename (node:internal/modules/cjs/loader:933:15)
at Function.Module._load (node:internal/modules/cjs/loader:778:27)
at Function.executeUserEntryPoint [as runMain] (node:internal/modules/run_main:77:12)
at node:internal/main/run_main_module:17:47 {
code: 'MODULE_NOT_FOUND',
requireStack: []
}
C:\Users\fodag\Desktop\DiscordBridge-main>

The plugin is meant to be run by it's self, it is meant to be put inside the plugins folder of BeRP


ref timeout triggers stack leak

Looks like the calls to mainloop create a stackframe that is never cleaned up. Some evidence, on macOS, you can kill the app by pressing ctrl-c twice. After a ref timeout, you need to press it 4 times. After using the app for a while, there are so many stack frames, you can ctrl-c dozens of times and still not exit the app. This may be the reason the app freezes after 5-10 games.

Good catch!

I think the bug is here:
def edit_continuation(self):
    def submit_clicked(game_clock):
        self.mgr.setGameClock(max(game_clock, 0))
        self.ref_timeout_clicked(save_state=False) # <----

    def cancel_clicked(game_clock):
        self.mgr.setGameClock(max(clock_at_pause, 0))
        self.ref_timeout_clicked(save_state=False) # <----

instead of returning back to the existing ref time out picker, it creates another one.


[.NET WPF] Add a checkbox to visualizer to toggle usage of Xceed renderers

Added a checkbox to the visualizer and confirmed that it works as expected.
Fixes #3265

Microsoft Reviewers: Open in CodeFlow

🔥


applying flex to navbar

To ensure that navbar stays intact while scrolling.

#45 issue has been solved.


2.x images

Ticket: #2537
Issue
Timber\Image class generates DivisionByZeroError error when calling aspect method of image was deleted manually from uploads directory. While being deleted attachment object itself and its metadata still present in database and will return something. When calling image.aspect it defines image dimensions and in case if it is not exists returns 0 for both height and width which causes zero divided by zero issue
Solution
Check if image height has 0 value and if it is return 0 as a result. This qay we will prevent site crashing. On init() method we will check if file exists and if not log it with Image path/to/image doesn't exists
Also added protected $thumbnail property to Post object and define it on thumbnail method as instance of Image - currently with thumbnail() method it will initialize Image every time it called
{% if post.thumbnail.aspect > 1.5 %}
    <img src="{{ post.thumbnail.src }}" alt="{{ post.thumbnail.alt }}">
{% endif %}

In this example Image will be initialized 3 times
Impact
Fixes fatal error if image was deleted from uploads directory but not from WordPress media uploader. Also prevents Image object from initializing multiple times
Usage Changes
All lefts the same
Considerations
Partially discussed in #2537
Testing
Steps to reproduce issue: #2537

{{ post.thumbnail.aspect }} will return 0 if image file does not exists (but its object exists in database as an attachment post type)
{{ post.thumbnail }} instantiation will log error Image path/to/image doesn't exists (not causing site crash but gives developer a clue what is may be wrong)
Post thumbnail will be initialized once

Coverage increased (+0.01%) to 92.308% when pulling f97e643e3fdadfa7bc22198b3f640015c7e20c33 on czernika:2.x-images into 68bb27b8110ec08cd237872f29c007e94240dc8c on timber:master.

@jarednovan sorry for misleading. PR was intended for 1.x version as it is causing fatal error. It was my bad I called branch 2.x - to be honest I don't remember why does it happened but I have few pet projects with 2.x version and some others with 1.x. I guess I just lost myself in naming versions... Sorry again
Changes in Image.php file are compatible with version 2.x though. Crucial part is this one - it is the only required part of PR while others are just "recommended"


Add badges to Readme.md

I'll give a new intractive look to Readme.md
Add badges too.
Please assign this to me.

Sure @Thrillseekr , go ahead

Hey @Thrillseekr Are you working on this? if yes whats the progress?

Yes, at the last stage.
Will send you PR by today

I've  created a PR for this issue, please review and merge.
PR link : https://github.com/purnima143/Pluto-UI/pull/55


feat: Text lines within Statement Blocks

Adds ability to insert text lines within a statement block.

Default delimiter is '% ' (note trailing space)
Lines within a statement block starting with '% ' are treated as text
Delimiter is configurable via:

--text-delim | --txt-delim CLI options
BASH_TPL_TEXT_DELIM environment variable
TEXT | TXT property in DELIMS directives

docs: Adds docs for new text line
test: Adds tests for new text line
test: Fleshes out script_block.tpl test
docs: Tries to better-clarify tag defaults
docs: Replace previous usages of "text tags" with "standard tags"
docs: Fixes table format per IntelliJ warnings
refactor: Moves debug info into function
BREAKING CHANGE: Keeps searching for 1st indented line within statement blocks, and uses it to define auto-indent-correction strategy.  Should provide more consistent behavior to otherwise well-indented scripts in the case where the first line of the block happens to not be indented.

@flaix If you're still using Bash-TPL, I'd love to get a review on this PR and maybe your thoughts on this feature in general?

Hi @TekWizely, I am terribly sorry, I did not see this. I noticed there was a PR, but did not see your comment.
I do not have the use case of having to close statement blocks just for one or two lines. So the feature is not something I had been missing. I only use single statement lines between outputs, or statement blocks with only statements, no output.
I just ran the new version against my template file again. The only change is that for me the indentation is now, well, incorrect. This is due to your "fix" for non-indented first lines. Was this an often occurring problem? For me the first line in a statement block is almost never indented. Because, well, it shouldn't be. The statement blocks form the generation script and that starts at indent 0. Any deeper indentation would be explicitly added by me. So with my templates the change has an adverse effect. I am not quote sure I understood the necessity for it from your example.
This is an example block from my template.
%

if [[ -n "${OUPUT_FILE}" ]]; then
    exec > "${OUPUT_FILE}"
fi

%

This used to produce a correctly indented script, as I had intended. Now, all three lines are on the same indentation level: zero.
if [[ -n "${OUPUT_FILE}" ]]; then
exec > "${OUPUT_FILE}"
fi

OK so I realize that you're use-case seems quite valid and that I likely introduced a bug in my last release.
It should be acceptable to have an "indentation of 0" where our base indentation is even with '%'
We should also encourage copious whitespace, ie the leading and trailing blank lines in your example.
I'll have a PR up soon to address this and hopefully you'll be available to give it a check against your current templates.
Thanks !

In case you are curious, this is the template I am using. It has some larger statement blocks in which I put together the general bash script. This could mostly also be one block, but I found no way to add comments to the script other than using % # comment.
After that I just use single statement lines to steer the output. Here you do find statement lines with different indentation.

Just a quick chime in re: comments.
So this input file:
comments.tpl
%# Bash-TPL comment - will not leave template
% # Statement line comment
%
# Block statement comment 1
# Block statement comment 2
%

Should generate:
comments.sh
# Statement line comment
# Block statement comment 1
# Block statement comment 2

Are you seeing something different in your local?

I tried it again and the comments work as you said. I can't remember why I didn't get it to work before. Either something changed meanwhile, or I hit an edge-case somewhere, or I just was too dumb. =)


Sort failures by failure's line and character

PR checklist

[x] Addresses an existing issue: #3335
[x] New feature, bugfix, or enhancement

[ ] Includes tests

[ ] Documentation update

Overview of change:
Is there anything you'd like reviewers to focus on?

CHANGELOG.md entry:
[enhancement] Sort failures by line and character for formatters

Thanks for your interest in palantir/tslint, @eggggger! Before we can accept your pull request, you need to sign our contributor license agreement - just visit https://cla.palantir.com/ and follow the instructions. Once you sign, I'll automatically update this pull request.

@eggggger You're almost done. CI fails due to linting errors:
/home/ubuntu/tslint/src/formatters/stylishFormatter.ts
ERROR: 42:47  semicolon        Missing semicolon

/home/ubuntu/tslint/src/language/rule/rule.ts
ERROR: 261:5  member-ordering  Declaration of public static method not allowed after declaration of public instance method. Instead, this should come after private instance fields.

you can also run linting locally with yarn lint

Thanks @eggggger


Some Foreign Keys are completely ignored casing INSERTs to fail

This is a follow up to #49, although I'm not sure if the issues are technically related, other than having been discovered in the same scenario (trying to manage the ASP.NET Core Identity data model with EasyData).
EasyData seems to ignore some FK columns in some entities. The corresponding column/value is not shown in the data grid, the field is not rendered at all in the CRUD form, and any INSERT will fail because the FK ends up being NULL.
Two examples:

IdentityRoleClaim: EasyData shows Id, ClaimType and ClaimValue. The property/column RoleId is completely ignored;
IdentityUserClaim: Same, we have Id, ClaimType and ClaimValue, and UserId is ignored;

Trying to create a new record on either table will fail with (example for IdentityUserClaim):

Ooops, smth went wrong
Cannot insert the value NULL into column 'UserId', table 'EasyDataDB.dbo.AspNetUserClaims'; column does not allow nulls. INSERT fails. The statement has been terminated.

Steps to reproduce

Create a new project with "Individual accounts" (dotnet new webapp --auth Individual)
Setup EasyData
Browse for example /easydata/IdentityRoleClaim or /easydata/IdentityUserClaim

Expected result

An empty grid (as there are no records yet), including the column RoleId (for Identity Role Claim) and UserId (for Identity User Claim)
Trying to add a new record should render the field RoleId / UserId, accordingly

Actual result

The columns RoleId or UserId are not shown in the query
The fields RoleId or UserId are not rendered in the CRUD form

Hello,
Have you tried to use such option:
options.UseDbContext<AppDbContext>(opts => opts.SkipForeignKeys = false)

Hi, no, I wasn't aware of that option, sorry.
Anyway, I've just tried that. With SkipForeignKeys in false, the FK properties are shown as columns in the query, but they're still ommitted in the CRUD form (so is still not possible to insert a new record, or change the FK through EasyData).

Hello, sorry I was a little busy these days.
I hope this commitment (8c28ef8cc05f3b438e0cdfd164ad507d539e54be) will fix this problem

Hi, no problem!
I've just tried again on an updated dev branch, but unless I'm doing something wrong (for some reason webpack is not called automatically when building, so I'm calling it manually (webpack --mode=development)), the CRUD forms are not fixed. For example, IdentityRoleClaims still fails to render the RoleId field:

To update to the latest changes, you have to run in easydata.js folder
npm run build:prod

And then run watch in the sample with identity (if you use ours (EasyDataAspNetCoreTest02))
Here is a screenshot of Create Identity Role Claim form on my side

P.s. Be sure, that you have also set this option:
options.UseDbContext<AppDbContext>(opts => opts.SkipForeignKeys = false)

Hi, thanks for taking the time to explain that, I appreciate it!
I can confirm that having rebuilt the frontend as per your instructions, the issue appears to be fixed.

Cool!
Thank you for testing.
I hope we will publish the update tomorrow.

Fixed in version 1.3.2. PR #51


Validation support for input

I want to add required attribute to the color input box, I didn't find any option for validation in current docs.

Added in commit 665c9c6a1226e3f94a44f43382dee8d0ffb8c3b0 and released in v2.4.1


chore: exchange cache for a database

requires a no sql database because only json data available to be saved
implement lookup for autocomplete and remove list of hashes

With commit 74ba7b43d68922c5fe5931a06f7be58038b8228f docker has been implement.
Still need to update readme and add query method allowing to lookup which contracts have been analysed.


Dropbox credentials message does not persist long enough

Description
I just tried to import my Dropbox account information on a project. I was met with the message below. However, the message did not persist long enough (at least on Chrome) for me to finish reading it, so I had to go through the steps again and then captured the message quickly.
I was familiar with what the message would say, but it still does not persist long enough.
For longer messages like this one, let's have them persist a little longer on the screen (especially for users unfamiliar with the content). This may apply to other add-ons, as well.

@NatashaRichter I took a look at this, the error is not supposed to disappear at all. I'm having a hard time creating the error itself, can you try this one more time?

Thanks @caneruguz. I am not able to reproduce this right now. Let's close it and if I notice it again, I'll come find you and show you.


Fixed merge order of config files on FreeBSD-based systems

This is an alternative to #13. As noted on that PR, on FreeBSD-based systems - including macOS highSierra - the libc glob function now sorts all the results before returning them, instead of returning the results in brace pattern order (as glibc does).
This PR doesn't suffer from the potential BC-break if people are relying on the order of config results within a brace expression (ie counting on a.global.php being merged before b.global.php).
To do this it passes the $forceFallback param to Zend\Stdlib\Glob::glob() so the brace order is controlled in userland. To ensure this always happens it adds zendframework/zend-stdlib as a dependency.

@weierophinney Is this going to get merged soon? Took me all day to figure out why my app decided to stop working. Managed to trace it down to this issue.

Thanks, @kynx. Merged to master; will release shortly.


Timeout Waiting for Device from RPi but not from another Ubuntu device

Hello,
I'm facing a weird beaviour and I'm pretty sure that I missed something.
Running the command python -m tinytuya scan on my Ubuntu machine is fine (but the machine is not up 24/7)
olivier@olivier-ubuntu:~$ python3 -m tinytuya scan
TinyTuya (Tuya device scanner) [1.12.0]

[Loaded devices.json - 1 devices]

Scanning on UDP ports 6666 and 6667 and 7000 for devices for 18 seconds...

Frigo CR 328 AZD   Product ID = keyjup78v54myhan  [Valid Broadcast]:
    Address = <IP_ADDRESS>   Device ID = ab3961c448d2979e55sean (len:22)  Local Key = b7bd599d6dbbfa2f  Version = 3.3  Type = default, MAC = fc:67:1f:cb:12:58
    Status: {'1': True, '9': 0, '17': 4, '18': 289, '19': 653, '20': 2362, '21': 1, '22': 551, '23': 25500, '24': 14219, '25': 2930, '26': 0, '38': 'on', '39': False, '40': 'relay', '41': False, '42': '', '43': '', '44': ''}
Scan completed in 18.0572 seconds

Scan Complete!  Found 1 devices.
Broadcasted: 1
Versions: 3.3: 1

>> Saving device snapshot data to snapshot.json

Running the same from my RPi (which is up 24/7), it fails
(tinytuya) pi@hassbian:/etc/tinytuya $ python -m tinytuya scan
TinyTuya (Tuya device scanner) [1.12.0]

[Loaded devices.json - 1 devices]

Scanning on UDP ports 6666 and 6667 and 7000 for devices for 18 seconds...

Frigo CR 328 AZD   Product ID = keyjup78v54myhan  [Valid Broadcast]:
    Address = <IP_ADDRESS>   Device ID = ab3961c448d2979e55sean (len:22)  Local Key = b7bd599d6dbbfa2f  Version = 3.3  Type = default, MAC = fc:67:1f:cb:12:58
    Polling <IP_ADDRESS> Failed: No response
Scan completed in 18.0742 seconds

Scan Complete!  Found 1 devices.
Broadcasted: 1
Versions: 3.3: 1

>> Saving device snapshot data to snapshot.json

What I already did:

Look if ports are not in use on the RPi.
Tuya device is reachable

(tinytuya) pi@hassbian:/etc/tinytuya $ ping <IP_ADDRESS>
PING <IP_ADDRESS> (<IP_ADDRESS>) 56(84) bytes of data.
64 bytes from <IP_ADDRESS>: icmp_seq=1 ttl=255 time=80.2 ms
From <IP_ADDRESS> icmp_seq=2 Redirect Host(New nexthop: <IP_ADDRESS>)
64 bytes from <IP_ADDRESS>: icmp_seq=2 ttl=255 time=122 ms
64 bytes from <IP_ADDRESS>: icmp_seq=3 ttl=255 time=144 ms
64 bytes from <IP_ADDRESS>: icmp_seq=4 ttl=255 time=37.9 ms
64 bytes from <IP_ADDRESS>: icmp_seq=6 ttl=255 time=61.1 ms
64 bytes from <IP_ADDRESS>: icmp_seq=7 ttl=255 time=98.8 ms
64 bytes from <IP_ADDRESS>: icmp_seq=8 ttl=255 time=119 ms
64 bytes from <IP_ADDRESS>: icmp_seq=9 ttl=255 time=128 ms
^C
--- <IP_ADDRESS> ping statistics ---
9 packets transmitted, 8 received, +1 errors, 11.1111% packet loss, time 8040ms
rtt min/avg/max/mdev = 37.892/98.930/144.054/34.180 ms

Tuya device is answering on port 6668

(tinytuya) pi@hassbian:/etc/tinytuya $ telnet <IP_ADDRESS> 6668
Trying <IP_ADDRESS>...
Connected to <IP_ADDRESS>.
Escape character is '^]'.

Connection closed by foreign host.

I'm running out of idea :'(
Please help
P.S.: Trying in python directly
(tinytuya) pi@hassbian:/etc/tinytuya $ python
Python 3.11.2 (main, Mar  4 2023, 12:59:45) [GCC 10.2.1 20210110] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> import tinytuya
>>> d = tinytuya.OutletDevice('ab3961c448d2979e55sean','<IP_ADDRESS>','b7bd599d6dbbfa2f')
>>> d.set_version(3.3)
>>> print( d.status() )
{'Error': 'Timeout Waiting for Device', 'Err': '902', 'Payload': 'Check device key or version'}

This feels to me like a network issue, especially with the From <IP_ADDRESS> icmp_seq=2 Redirect Host(New nexthop: <IP_ADDRESS>) and 9 packets transmitted, 8 received, +1 errors, 11.1111% packet loss.  Is the RPi on the same network as the device?  Is there a firewall in there somewhere?

Yes, the 2 devices are in the same network, there is no firewall in between.
The reason why From <IP_ADDRESS> icmp_seq=2 Redirect Host(New nexthop: <IP_ADDRESS>) is because the RPi is hosting Pi-Hole and it was looping to itself, so I added a route especially for the tuya device, maybe it is wrong, and it is the cause of the issue (<IP_ADDRESS> is my gateway, my internet box)
route -n
Kernel IP routing table
Destination     Gateway         Genmask         Flags Metric Ref    Use Iface
<IP_ADDRESS>         <IP_ADDRESS>   <IP_ADDRESS>         UG    600    0        0 wlan0
<IP_ADDRESS>     <IP_ADDRESS>         <IP_ADDRESS>   U     600    0        0 wlan0
<IP_ADDRESS>    <IP_ADDRESS>   <IP_ADDRESS> UGH   0      0        0 wlan0

As a consequence, there is always 1 packet drop, it is the one redirecting to <IP_ADDRESS>.
Without the route, I was not even able to ping the device. All packets were dropped.

If it really is on the same network then it should not be going to/through a gateway.  ip route add <IP_ADDRESS>/32 dev wlan0 would be the way to tell it it's directly attached, however that feels like a band-aid; the better question is: why is it looping?  AFAIK Pi-hole is not a transparent firewall/proxy, so how/why are packets getting looped back to it?

Ok, I removed the route.
It was still working and tinytuya was working too, until it failed again.
But I think I found the reason, the tuya plug is advertising itself with the name of my wlan interface, weirdly enough. My Ubuntu is not using wifi, only a lan interface.
arp | grep fc:67:1f:cb:12:58
wlan0.lan                ether   fc:67:1f:cb:12:58   C                     wlan0

I edited the /etc/hosts file and added
<IP_ADDRESS>     ab3961c448d2979e55sean
And it is working now.

And it failed again.
The plug is back in my arp table as wlan0.lan and ping is not working. Again.
nslookup <IP_ADDRESS>
<IP_ADDRESS>.in-addr.arpa       name = wlan0.lan.

I can ping any other devices
pi@hassbian:/usr/bin $ ping <IP_ADDRESS>
PING <IP_ADDRESS> (<IP_ADDRESS>) 56(84) bytes of data.
64 bytes from <IP_ADDRESS>: icmp_seq=1 ttl=64 time=51.6 ms
64 bytes from <IP_ADDRESS>: icmp_seq=2 ttl=64 time=54.6 ms
64 bytes from <IP_ADDRESS>: icmp_seq=3 ttl=64 time=8.64 ms
64 bytes from <IP_ADDRESS>: icmp_seq=4 ttl=64 time=32.5 ms
^C
--- <IP_ADDRESS> ping statistics ---
4 packets transmitted, 4 received, 0% packet loss, time 3006ms
rtt min/avg/max/mdev = 8.642/36.814/54.578/18.343 ms

But not the tuya plug
pi@hassbian:/usr/bin $ ping <IP_ADDRESS>
PING <IP_ADDRESS> (<IP_ADDRESS>) 56(84) bytes of data.
^C
--- <IP_ADDRESS> ping statistics ---
9 packets transmitted, 0 received, 100% packet loss, time 8339ms

Running a tshark while ping is like that
pi@hassbian:/home/pi $ sudo tshark -i wlan0 -f "host <IP_ADDRESS>" -f "icmp"
Running as user "root" and group "root". This could be dangerous.
Capturing on 'wlan0'
    1 0.000000000 <IP_ADDRESS> → <IP_ADDRESS> ICMP 98 Echo (ping) request  id=0x1a61, seq=6/1536, ttl=64
    2 1.040043499 <IP_ADDRESS> → <IP_ADDRESS> ICMP 98 Echo (ping) request  id=0x1a61, seq=7/1792, ttl=64
    3 2.080053454 <IP_ADDRESS> → <IP_ADDRESS> ICMP 98 Echo (ping) request  id=0x1a61, seq=8/2048, ttl=64
    4 3.120040125 <IP_ADDRESS> → <IP_ADDRESS> ICMP 98 Echo (ping) request  id=0x1a61, seq=9/2304, ttl=64
    5 4.160030649 <IP_ADDRESS> → <IP_ADDRESS> ICMP 98 Echo (ping) request  id=0x1a61, seq=10/2560, ttl=64
    6 5.200063827 <IP_ADDRESS> → <IP_ADDRESS> ICMP 98 Echo (ping) request  id=0x1a61, seq=11/2816, ttl=64
    7 6.240062420 <IP_ADDRESS> → <IP_ADDRESS> ICMP 98 Echo (ping) request  id=0x1a61, seq=12/3072, ttl=64
    8 7.280055125 <IP_ADDRESS> → <IP_ADDRESS> ICMP 98 Echo (ping) request  id=0x1a61, seq=13/3328, ttl=64
    9 8.320066109 <IP_ADDRESS> → <IP_ADDRESS> ICMP 98 Echo (ping) request  id=0x1a61, seq=14/3584, ttl=64
   10 9.360099540 <IP_ADDRESS> → <IP_ADDRESS> ICMP 98 Echo (ping) request  id=0x1a61, seq=15/3840, ttl=64

Request, no reply. There is absolutely no firewall, from my ubuntu, the ping is no issue
olivier@olivier-ubuntu:~$ ping <IP_ADDRESS>
PING <IP_ADDRESS> (<IP_ADDRESS>) 56(84) bytes of data.
64 octets de <IP_ADDRESS> : icmp_seq=1 ttl=255 temps=42.5 ms
64 octets de <IP_ADDRESS> : icmp_seq=2 ttl=255 temps=261 ms
64 octets de <IP_ADDRESS> : icmp_seq=3 ttl=255 temps=98.1 ms
64 octets de <IP_ADDRESS> : icmp_seq=4 ttl=255 temps=99.5 ms
64 octets de <IP_ADDRESS> : icmp_seq=5 ttl=255 temps=25.6 ms
64 octets de <IP_ADDRESS> : icmp_seq=6 ttl=255 temps=33.4 ms
64 octets de <IP_ADDRESS> : icmp_seq=7 ttl=255 temps=75.9 ms
^C
--- statistiques ping <IP_ADDRESS> ---
7 paquets transmis, 7 reçus, 0 % paquets perdus, temps 6011 ms
rtt min/moy/max/mdev = 25,610/90,896/261,282/74,920 ms

Thanks for sharing all these details @OToussaint .  I have several RPi's of different hardware versions and OS, and have never seen this issue.  You could try to disable PiHole and see if that is what is poisoning your arp:
# Disable Pi-hole for 30 seconds
pihole disable 10m

# Or more drastic
sudo systemctl stop pihole-FTL
sudo systemctl disable pihole-FTL

I would also try to hardwire to the network (if not already) to see if that makes a difference.  If not, I would just re-image the Pi (new SD) and see if it is a hardware issue vs config issue.

No, no change, not even with a brand new SD Card.
My work-around, working for 14 hours is awful:

Create the route "via"
ping <IP_ADDRESS> to update any and every arp/dns/whatever
Remove the route
Cron a ping every 30 seconds (this is disgusting but seems to keep everything reachable)

I'd like a real solution but I'm hopeless.

Hi @OToussaint , I can understand the frustration.  It still seems like this is an issue with the RPi configuration if your ubuntu host works.  What OS are you using on your RPi?  For my working units, I'm using Debian version: 11 (bullseye) here: https://www.raspberrypi.com/software/operating-systems/

The same, Debian 11 (bullseye)
uname -a
Linux hassbian 5.15.84-v7+ #1613 SMP Thu Jan 5 11:59:48 GMT 2023 armv7l GNU/Linux

pi@hassbian:~ $ hostnamectl

   Static hostname: hassbian
         Icon name: computer
        Machine ID: 880c5857b62e440f9bf62c65361b7f13
           Boot ID: 8c0e3d95002d4298adb951eea80cad2d
  Operating System: Raspbian GNU/Linux 11 (bullseye)
            Kernel: Linux 5.15.84-v7+
      Architecture: arm

Well, at least, congrats on getting the workaround to work!  Unfortunately, I have no ideas or other suggestions for you.

Issue with the workaround is that if ping is skipped, I might go back to step 1: recreate the route, and everything after.
In theory, I could do a script with
ip route add <IP_ADDRESS> via <IP_ADDRESS> dev wlan0
ping -c 10 <IP_ADDRESS>
ip route del <IP_ADDRESS> via <IP_ADDRESS> dev wlan0
ping -c 10 <IP_ADDRESS>

cron with root every minute but seems like overkill.

Let me close that issue here, I'll try to find help in the appropriate forum.
Thank you for your help.

Good luck, @OToussaint !  It will probably be a simple thing on your network or setup that will be easy to fix once you finally sort it. If you find it, please report back in case someone is following your steps. 🙏

Posted on SuperUser for a few days but everyone is struggling as much as I do ...


how to use in browser

I am writing test cases using Jasmine and need to use ajv through my browser.
I have already attempted to use it with browserify but it doesn't seem to work. Please provide a quick example as to how to use it within browser after bundling the package using browserify.
Here is what I have done so far:
browserify node_modules/ajv/lib/ajv.js -o node_modules/ajv/lib/bundle.js

and added that into my jasmine Spec file using:
  <script src="../../node_modules/ajv/lib/bundle.js"></script>

Then??

We use it with browserify with require('ajv') in dependent module, not as a separate bundle, but it should work the way you do it too. Are you using the latest version? it only works from 0.5.2.

Yes, using the latest versions of browserify and ajv.

ah, yes, it doesn't create any global at the moment. You can:

require it directly from your code as we do
create file bundle.js like this:

window.Ajv = require('ajv');

and then browserify and use this bundle
The last option would create global Ajv variable

It looks like Browserify does not make require function available with the command I was using. Here is what I used instead:
browserify -r ./node_modules/ajv/lib/ajv.js:ajv -o ./node_modules/ajv/lib/bundle.js

Then, I was able to use it normally (after including bundle.js)
    Ajv                  = require('ajv')
  , ajv                  = Ajv()

Cool, thanks. I will add to readme.

Yes, I still load bundle.js in a script tag in HTML file.
  <script src="node_modules/ajv/lib/bundle.js"></script>

For some reason, many of the browserify articles do not mention -r option.

Thanks, added "Using in browser" to readme.

@epoberezkin Can you link this thread in the readme? This help me a lot :D
PD: sorry for my english.

@nsdelgadov it's changed since then - the supplied bundle already creates global (it is "universal"). So if you load bundle from npm package or from cdnjs you can just use global Ajv. Maybe just one line comment can be added to readme.


[Feature-8497][UI] Support dependent task re-run automatically

Purpose of the pull request

Dependent tasks can re-run automatically in the case of complement

Brief change log

Related issues
close #8497

PTAL @SbloodyS @songjianet

Codecov Report

Merging #8739 (3ffa1df) into dev (63e85f3) will decrease coverage by 0.00%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##                dev    #8739      +/-   ##
============================================
- Coverage     45.29%   45.28%   -0.01%
+ Complexity     4075     4074       -1
============================================
  Files           702      702
  Lines         27112    27112
  Branches       2897     2897
============================================
- Hits          12280    12279       -1
- Misses        13678    13679       +1
  Partials       1154     1154

Impacted Files
Coverage Δ

...lphinscheduler/rpc/common/AbstractRpcCallBack.java
0.00% <0.00%> (-100.00%)
:arrow_down:

Continue to review full report at Codecov.

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


Get Started

Set up with git repo
Project brief, analysis and brainstorm
Clarification
Team name

What day do we want to go over this?


Task (or bug?): Investigate Exception Raised during Concurrent processing of datasets

Describe the task
This is related to #1038 and the now-closed PR #1039.
During the experimental runs for parallelizing the ETL tasks, a common exception was raised. After discussion with @lucasmbrown-usds, an initial diagnosis was made that the temporary directory file writes are raising the exception. More specifically, the naming scheme for the temporary file paths seems to be the same across datasets (see below for screenshots of trackback logs). Hence the goal is to ensure that the file paths are unique and to potentially prevent such exceptions during the simultaneous reads and writes across all threads that are spawned from the main ETL process.
Acceptance Criteria
To be discussed after consultation with @esfoobar-usds and @lucasmbrown-usds.
Additional context
Please see here for references to the same temporary path:

This bug comes from expected behavior given that all the ETLs write to (and delete) the same directory


Update erase.md

Remove dangling period

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


[New Resource] DNSClientNRPTRule

Need a resource to add/remove/edit DNS Client NRPT rules, including configuring entries for DA and IPSec.

Are there existing powershell commands for this? If so what are they? Just trying to give a leg up to anyone who wants to come in as a first stab.

@tysonjhayes  - good point!
The cmdlets to use to manipulate this stuff are:
Get-DnsClientNRPTRule
Set-DnsClientNRPTRule
Remove-DnsClientNRPTRule

Here is an example of a rule I have on one of my machines that forces DNS Searches for .plagueho.local to go to the DNS Server <IP_ADDRESS>.

Outside of PowerShell, NRPT rules are usually managed by Group Policy (or Local Policy):

The key tasks for NRPT is for implementing DNSSEC, IPSEC, DA and directing DNS requests to specific servers based on the prefix/suffix etc. So the resource should support at a minimum the following parameters:
GPOName (even though this is not being applied by a GPO) - this is a GUID.
IPsecCARestriction
DirectAccessDnsServers
DirectAccessEnabled
DirectAccessProxyType
DirectAccessProxyName
DirectAccessQueryIPsecEncryption
DirectAccessQueryIPsecRequired
NameServers
DnsSecEnabled
DnsSecQueryIPsecEncryption
DnsSecQueryIPsecRequired
DnsSecValidationRequired
NameEncoding
DisplayName
Comment

I'll start working on this one as I'm going to be needing it at work soon. @tysonjhayes - if you get a mo could you label this issue as in progress? Cheers sir!

@PlagueHO  Are you still working on this?

@tysonjhayes - it's on the backlog... But when I'll actually get to it. I've got a few other resources to get done before I'll get to this (this weekend is assigned to creating the new xCertificateExport resource in xCertificate).


Calling methods from outside app

Hi @vareddy,
I have included sdk with my custom client code and build the static library out of it. Now I want to test with another stand alone test class file just to execute the methods in my static lib. But  its giving me undefined methods error. Below is my make file wherein I am linking the static lib. Could you please let me know what might be the issue? I have a header file along with my stand alone class which has methods defined. libDAIoTSDK is my sdk lib along with client code.
CXX=g++
SYS_HEADER_DIR = /usr/local/include/
APP_DIR = ./
APP_NAME = main_body_pub
APP_C_FILES = $(APP_DIR)$(APP_NAME).cpp
APP_H_FILES = $(APP_DIR)da_iot_device_sdk.h $(APP_DIR)PubSub.hpp  $(SYS_HEADER_DIR)/curl/curl.h
STATIC_LIBS_DIR = ./
STATIC_LIBS = ./libDAIoTSDK.a -lcurl -lmbedtls -lmbedcrypto -lmbedx509
SOURCES = $(APP_C_FILES) $(APP_H_FILES) $(STATIC_LIBS)
$(APP_NAME):$(SOURCES)
$(CXX) -std=c++11 -o $@ $(APP_C_FILES) $(STATIC_LIBS)

methods are from my client code which in turn built along with sdk source files.

We'd need compiler output and also see your build script and some code to be any help here. Please reopen if you have further info.


[Feature] Hide delete button for published dataverse

Purpose
Hides buttons that will invariably return errors if pressed
Changes
More ugly special cases for Dataverse added
Side Effects
None

CR 1 looks good -- double checked delete wasn't an option for published dataverse files locally as well -- :+1: @mfraezz

CR2 done; works and no other comments.
Worth a glance from someone more familiar with treebeard to make sure the assumptions are always valid; there doesn't seem to be a good way to get publication status without resorting to the URL.

Pass finished. Minor change and this is good to merge.


⚠ Bofrost.be has degraded performance

In 4379ef8, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1792 ms

Resolved: Bofrost.be performance has improved in d964271.


screen sharing on client and fixes to distributed streaming. Chat als…

…o added to client

Test server with all changes can be found at https://test-toonin.herokuapp.com/.
Changes include

fixing distributed streaming on client side.
adding chat to client.
adding ability to create and share from web client.


Releases trained models for all reproduced agents

Addresses: #551

Can you fix the Travis CI errors?

/test

/test

/test

To test this locally:
For each of the 9 run reproducibility scripts, run:
python train_<alg>.py --load-pretrained --pretrained-type <final|best> --env <Env>
Note that for some algorithms we don't have both final and best networks stored.
The link to the stored models for the tests can be received from me privately.

/test

/test

/test

/test

/test

/test

/test

/test

/test


pls release

according to @thornleyk in https://github.com/ocaml/opam-repository/pull/5808#issuecomment-259847477 this package needs a release (and previous releases need adjustments) -- I see that @samoht did remove the camlp4 earlier (thus master should work with an up to date sexplib)

Let me have a look at this.

@samoht thx! (I don't have a computer to test this package on...)   I can also see there is some _oasis in this repo..

done in https://github.com/ocaml/opam-repository/pull/7807, thx to @samoht


supportUntrustedWorkspaces override is ignored for source control providers

Issue Type: Bug

Open untrusted workspace
See an error in the source control pane: "None of the registered source control providers work in Restricted Mode"
Observe that vscode.git has been disabled in untrusted workspaces in a recent update
Attempt to re-enable it by adding an override for it to extensions.supportUntrustedWorkspaces in settings.json
Observe that, after a reload, the extension is indeed re-enabled under the built-in extensions, but...
... the source control pane still shows the same error, instead of the actual Git integration

From a glance at the implementation for this error condition, the error seems to be always shown when the editor is in Restricted Mode, regardless of whether any source control providers have actually indicated support for this mode.
VS Code version: Code 1.63.1 (fe719cd3e5825bf14e14182fddeb88ee8daf044f, 2021-12-14T02:02:02.183Z)
OS version: Linux x64 5.14.21
Restricted Mode: Yes

System Info

Item
Value

CPUs
AMD Ryzen 9 3900X 12-Core Processor (24 x 2200)

GPU Status
2d_canvas: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: disabled_offopengl: enabled_onrasterization: disabled_softwareskia_renderer: enabled_onvideo_decode: disabled_softwarevulkan: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
2, 2, 2

Memory (System)
62.76GB (2.78GB free)

Process Argv

Screen Reader
no

VM
0%

DESKTOP_SESSION
plasma5

XDG_CURRENT_DESKTOP
KDE

XDG_SESSION_DESKTOP
plasma5

XDG_SESSION_TYPE
x11

Extensions (14)

Extension
Author (truncated)
Version

Bookmarks
ale
13.2.2

Nix
bbe
1.0.1

language-postcss
cpy
1.1.0

flow-for-vscode
flo
1.9.2

beautify
Hoo
1.5.0

l13-diff
L13
1.0.3

python
ms-
2021.12.1559732655

vscode-pylance
ms-
2021.12.2

jupyter
ms-
2021.11.1001550889

jupyter-keymap
ms-
1.0.0

vscode-quick-open-create
noc
0.6.0

mdx
sil
0.1.0

rewrap
stk
1.15.4

debug
web
0.25.1

(9 theme extensions excluded)

To verify:

Add the following into your settings.json:

    "extensions.supportUntrustedWorkspaces": {
        "vscode.git": {
            "supported": true
        }
    },

Open a workspace containing a git repository and choose to stay in restricted.
Confirm that the SCM viewlet is rendered as expected and no welcome messages are shown


Support partial references

Description of Problem, Request, or Question
Since Conan 1.19 all packages can be created only using name and version e.g. zlib/1.2.11. However, CPT requires an user name and channel so far.
This same feature fixes the case where dependencies are selected to be uploaded, but are partial references:
https://travis-ci.com/mpusz/units/builds/132608827#L1094

closed by #430


Restrict QR Codes to Users Who Can Already See Patient Data

Admin, Basic, and Entry-Only users are being added to SimpleReport. For privacy reasons, Entry-Only users cannot view patient data. A provider user with access to time-of-test QR codes could maliciously view that QR code on their own device, enter the DOB, and access information that they should not be privy to.
For this reason, Entry-Only users must not be able to generate or view QR codes for time-of-test questionnaires.
Acceptance Criteria

[ ] In the UI, Entry-Only users are not able to view or generate QR codes

Given that we're removing the QR code, we can close this ticket.


Simplification should prune topologically separate components

Hi,
I'm running into this problem a lot, here is an example mesh on which it occurs.

  As a result of that, meshopt_simplify / meshopt_simplifyWithAttributes can't remove them (as documented).
This is the result of meshopt_simplify:

The total number of triangles is only slightly reduced.
So then I call meshopt_simplifySloppy, which successfully removes the spotlights.  Unfortunately meshopt_simplifySloppy seems to merge vertices aggressively, resulting in pretty incorrect shading normals on the room itself.  See attached pics.

What I want is some kind of simplification that can remove or merge topologically separate components, but which doesn't merge vertices on edges, and hence maintains reasonable shading normal fidelity.
Do you have any suggestions for that?  Is it possible with meshoptimizer as-is, or would it need to be added?
Thanks,
Nick C.
p.s. please feel free to recategorise as a feature enhancement or discussion item.

Just thinking out loud... what about a pass that computes the topological submeshes, computes bounding boxes of them, and discards any which size below the error threshold?  I think it would help in this case at least.

a pass that computes the topological submeshes, computes bounding boxes of them, and discards any with size below the error threshold

Yeah this happens to be planned as one of the next improvements to simplify.

Oh awesome!   Was thinking of trying it myself.
Presumably there's some code to compute AABBs in there already, i can just adapt it.

I tried an implementation with some of my code, seems to work well:
It's relatively straightforward, breadth first search over adjacent triangles to identify connected components, compute AABB, compare with error, discard small components.  Runs in linear time on the number of triangles.
I had a look at the meshoptimizer code but the data structures look a little tricky so I think I'll leave that to you :)
Can post my code for reference if you want but I imagine you don't need it.
https://github.com/user-attachments/assets/d4c56282-df12-495e-ae51-2d29096e4c5b

If you could share the asset that would be nice; I have a preliminary version of this and would like to make sure it works well on this.

Here you go: https://www.dropbox.com/scl/fi/7886i6nxgv901mdbqik5y/gallery_exported.glb?rlkey=yck4m16jgw65si45l69yrnc0h&st=acuokyz3&dl=0


Don't apply babel-plugin-transform-runtime inside react-on-rails to work with babel 7

Don't apply babel-plugin-transform-runtime inside react-on-rails to work with babel@7
It should be applied by user land.
Why?
The generated code depends on built babel's version.
It means that user couldn' build their code with babel@7 that depends on react-on-rails built by babel@6.
For example, babel-plugin-transform-runtime (babel 6) generates the
following code.
var _assign = require('babel-runtime/core-js/object/assign');

However @babel/plugin-transform-runtime (babel 7) expects the
path that beginning of @babel/runtime/.
The change allows to use react-on-rails on babel@7.

This change is

Coverage remained the same at ?% when pulling 89819abe76946203839bca411a90c19423b50779 on tricknotes:babel-7 into 8920545baa9d9626dc0eb77fa9e776ddd1abed28 on shakacode:master.

Coverage remained the same at ?% when pulling 89819abe76946203839bca411a90c19423b50779 on tricknotes:babel-7 into 8920545baa9d9626dc0eb77fa9e776ddd1abed28 on shakacode:master.

@tricknotes what about babel@6?
@alexfedoseev any idea on this one?


There is no python3-pep8 package on Fedora 32

$ dnf install python3-pep8
Last metadata expiration check: [...]
No match for argument: python3-pep8
Error: Unable to find a match: python3-pep8

$ dnf search pep8
Last metadata expiration check: [...]
=== Name & Summary Matched: pep8 ===
python3-autopep8.noarch : The package autopep8 formats Python code based on the output of the pep8 utility
=== Name Matched: pep8 ===
python3-pep8-naming.noarch : Check PEP-8 naming conventions, a plugin for flake8
==== Summary Matched: pep8 ===
python3-bashate.noarch : A pep8 equivalent for bash scripts
python3-pytest-flake8.noarch : Plugin for pytest to check PEP8 compliance with Flake8

@rleh FYI I opened a similar PR at https://github.com/ros2/ros2_documentation/pull/848 that replaces it with python3-pycodestyle for Foxy and Rolling instructions

@mikaelarguedas See my https://github.com/ros2/ros2_documentation/pull/848#issuecomment-674436714.

I don't know why python3-pep8 was required, but the python3-pycodestyle is not an drop-in replacement, because it does not provide the pep8 command. So anything that depends on pep8 will fail.
As far as I can see there is no dependency on pep8 in the ROS2 installation, I would prefer to remove it to keep the list of dependencies as short as possible (as suggested with 2fbc1c5).

Closing in favor of #848


set Athena connection failed

When trying to set Athena connection according to https://superset.apache.org/docs/databases/athena/
Showing error messages "ERROR: No JVM shared library file (libjvm.so) found. Try setting up the JAVA_HOME environment variable properly."

Would you help to clarify the detailed connection string and where such error message prompt? It seems that some java execution required, which is not required in Athena connection nor Superset container itself.


More explicit syntax for temporary assignments

The current syntax for temporary assignments is borrowed from POSIX shell, and requires there to be no spaces surrounding a =, e.g.
foo=bar put $foo
pwd=/tmp touch x

The syntax is quite subtle, and is a source of syntactical complexity - for example, the syntax for lvalues have to support braced lists, in order for code like this to work (suppose that f outputs two values):
{a,b}=(f) echo $a $b

It may be a good idea to make the syntax for temporary assignments more explicit, for example using a special command with. The examples above can be written as:
with foo = bar { put $foo }
with pwd = /tmp { touch x }
with a b = (f) { echo $a $b }

However, it's not straightforward to generalize this to multiple temporary assignments, which can be done easily using the current syntax, such as a=foo b=bar put $a $b. Nesting with certainly works, but is quite verbose. One possibility is to use lists to surround the individual assignments:
with [a = foo] [b = bar] { put $a $b }

In any case, the with command is going to be more verbose than the current syntax, hence the "maybe" tag.

FWIW, the POSIX semantics for temporary assignments work by instantiating environment variables. Except when it doesn't. That is, x=y ... is (mostly) equivalent to env x=y .... It's an example of a rather silly optimization to eliminate four characters from a statement. The "mostly" is parenthesized because the behavior depends on whether the command is a builtin or external. Even that distinction has subtleties. Consider the following:
bash-5.0$ WTF=wtf eval 'echo $WTF'
wtf
bash-5.0$ WTF=wtf echo $WTF

bash-5.0$ WTF=wtf typeset | grep WTF
bash-5.0$ WTF=wtf typeset -x | grep WTF
bash-5.0$ WTF=wtf eval 'typeset -x' | grep WTF
declare -x WTF="wtf"
bash-5.0$ WTF=wtf env | grep WTF
WTF=wtf
bash-5.0$ type env
env is hashed (/usr/bin/env)

Which is a long winded way of saying that any change to the syntax or behavior of temporary assignments requires careful thought so as not to repeat the mistakes of the POSIX standard.

See also, issue #705. Consider the unexpected behavior of this example:
> x = 1
> put $x
▶ 1
> x=2
> put $x
▶ 2
> x=3 true
▶ 2

The middle "put" should have output 1 rather than 2.  If this issue were implemented that would probably resolve issue #705.

This comment is indirectly related to this issue since the problem I'm describing involves permanent, rather than temporary, assignments. Consider this pipeline:
echo a=b | cut -d = -f 1

In a POSIX shell (e.g., bash) that will output a. In Elvish it outputs nothing because the RHS of the pipeline performs two assignments: cut = -f and -d = 1. That pipeline is a simplified version of a more complex pipeline. I am not arguing that the current Elvish semantics are wrong and should be changed. I'm simply pointing out that the surprising behavior of the above pipeline should be considered in resolving this issue.

Getting back to this. I'm considering stealing a design from Raku's. Introduce a special command (say tmp) that does temporary assignment within the current scope. An example:
var x = foo
{
  tmp x = bar
  # $x is "bar" for the remainder of the scope
  command using $x
}
# $x reverts to "foo"

The original original with proposal had the downside that it was tricky to do multiple temporary assignments. With tmp, just have multiple tmp commands:
var x = foo
var y = lorem
{
  tmp x = (some command)
  tmp y = (some other command)
  command using $x and $y
}

One-liner length comparison:
x=foo some command
{ tmp x = foo; some-command }
with x = foo { some-command }

Still more verbose than the current syntax, but same as the original with proposal.

Also, if there is already a scope, using tmp is actually more concise.
For example, to and run something using each subdirectory as the working directory:
for d [*/] {
  tmp pwd = $d
  # do things in $d
}

While the current syntax will require an additional level of nesting, unless there's only one command in the body:
for d [*/] {
  pwd=$d {
    # do things in $d
  }
}

Another nice thing about tmp is that it has exactly the same syntax with var and set (and the same length; this is why I shortened Raku's temp to tmp).

I like the idea, if only I could suggest let in lieu of tmp. It's javascript's syntax for the same construct, and I prefer it for purely ergonomic reasons. Another option is the even shorter my from perl, again with the same behavior.

This new construct has no corresponding construct in JavaScript or Perl. It does correspond to Bash's local declaration.
Both JavaScript's let and Perl's my are lexical declarations and correspond to Elvish's var.

I don't think referencing Bash's local keyword clarifies the matter and should be avoided. See, for example, https://stackoverflow.com/questions/4419704/differences-between-declare-typeset-and-local-variable-in-bash. The distinction between typeset, declare and local is a muddle in Bash and other POSIX shells. Having said that, a focus on how the proposed tmp keyword differs from var for magic vars like pwd would help. I'm basically a +1 on the proposal to introduce a tmp keyword from a functionality perspective. The challenge will be documenting the behavior (I found the Raku document linked to above borderline incomprehensible).

I've done a little more research, and did originally misunderstand. Would tmp be dynamically scoped, like local in perl?
var x = 1

fn foo []{
  echo $x
}

fn bar []{
  tmp x = 2
  foo
}

foo
bar
foo

Would this print 1, 2, 1?

Would this print 1, 2, 1?

That's how I read it. But I wasn't sure, until I saw the pwd example. And otherwise, it would be just like var. In fact, many of the examples given would produce the same result using var, except for what's under the hood.

@fennewald You're right.
Indeed Perl's local does this too - it is confusingly named and Raku's name temp better reflects what it does.

A +1 for tmp

0.18.0 will deprecate the legacy syntax.
After 0.18.0 is released, the legacy syntax will be removed. Moving this to the 0.19.0 milestone.

I think variables should be limited to their scope, unless specified otherwise, like Python's global or nonlocal.
If we support a simple scope with { ... } which is like ( ... ) except it doesn't return the value, just like Go, then instead of
with [a = foo] [b = bar] { put $a $b }

we write
{ var a = foo; var b = bar; put $a $b }

Also maybe a new command to set multiple variables like
{ vars &a = foo &b = bar; put $a $b }

Doesn't make much different in verbosity though.

@ilius Variables in Elvish are already lexically scoped, so in { var a = foo; var b = bar; put $a $b }, the variables $a and $b are not visible outside the closure. This issue was about giving an already existing variable a temporary value for a certain scope.
Anyway this has been implemented; I'll leave this open until the legacy syntax is fully removed.

I've decided to implement the originally proposed with command as well: tmp is technically fine, but it can sometimes feel counter-intuitive (https://github.com/elves/elvish/issues/1776#issuecomment-2245282976)


[Feature Request] Show/hide own nametag

I have seen a lot of mods doing this, but none of them having the feature to toggle it on/off
Maybe this mod could have such feature?

Four months late but done. Download here


Fix navigation bar tests [ch85362]

Story details: https://app.clubhouse.io/cartoteam/story/85362

Vamos!! 😄


Configurable edgeCaseTargetRowHeights

In my setup, I want to display panoramic photos and unfortunately they are squished if they are too wide. This PR adds the option to override the minimum and maximum row height.
The default behaviour is preserved unless it is changed with a config option.

Coverage remained the same at 100.0% when pulling 329b3774b487a22c05ab90811e3f25817b20d2d9 on GenaBitu:configurable-edge-cases into 9c977ede8516dbdb08ec53bda2d0945019e043fb on flickr:master.

Coverage remained the same at 100.0% when pulling 329b3774b487a22c05ab90811e3f25817b20d2d9 on GenaBitu:configurable-edge-cases into 9c977ede8516dbdb08ec53bda2d0945019e043fb on flickr:master.

Coverage remained the same at 100.0% when pulling 329b3774b487a22c05ab90811e3f25817b20d2d9 on GenaBitu:configurable-edge-cases into 9c977ede8516dbdb08ec53bda2d0945019e043fb on flickr:master.

Closing in favor of #64


ConnectionCursor gets serialized into invalid value

When using the provided helpers for Relay, the ConnectionCursor gets serialized into an invalid value, as ConnectionCursor#toString() is used for serialization. This results that the first cursor is "ConnectionCursor{value='c2ltcGxlLWN1cnNvcjA='}" instead of c2ltcGxlLWN1cnNvcjA=

I'm not sure I understand this ... which serialization do you mean?
Thanks

Sorry, serialization is not the correct word, but coercion.
Eg. when doing the following query:
query Stops {
    stops(first: 1) {
        edges {
            cursor node {
                stop {
                    id
                }
            }
        }
    }
}

I get the following response:
{
    "stops": {
        "edges": [{
            "cursor": "ConnectionCursor{value='c2ltcGxlLWN1cnNvcjA='}",
            "node": {
                "stop": {
                    "id": "U3RvcDphZ2VuY3k6ZW50cmFuY2VfYg=="
                }
            }
        }]
    }
}

Instead of the correct one:
{
    "stops": {
        "edges": [{
            "cursor": "c2ltcGxlLWN1cnNvcjA=",
            "node": {
                "stop": {
                    "id": "U3RvcDphZ2VuY3k6ZW50cmFuY2VfYg=="
                }
            }
        }]
    }
}

This is due to this executing this code instead of this

ok thanks ... that makes it more clear. I will have a look at in detail.
Just a quick thought: I'm not sure the problem is the coercion of  the string. Maybe the graphql schema is not correct: The cursor field should not be of type String, but maybe of type Connection? (I maybe totally wrong ;-) )

According to the specification it could be a custom scalar type, so we could just make ConnectionCursor extend GraphQLScalarType.

Thanks for the link ... these are indeed the details I have missed in the first prototype. It might take me a few days to look at in more detail .. sorry for the delay.

Yes,
the problem is indeed that the type of the cursor field is declared as String. So every Object in coerced into toString().
Probably a custom scalar type might be the best solution, but as a quick fix I changed the toString of ConnectionCursor.
Does this for now solve you problem?
Thanks

Thanks, works perfectly now


Update ccms test run

What
Link to story
Describe what you did and why.
Checklist
Before you ask people to review this PR:

Tests and rubocop should be passing: bundle exec rake
Github should not be reporting conflicts; you should have recently run git rebase master.
There should be no unnecessary whitespace changes. These make diffs harder to read and conflicts more likely.
The PR description should say what you changed and why, with a link to the JIRA story.
You should have looked at the diff against master and ensured that nothing unexpected is included in your changes.
You should have checked that the commit messages say why the change was made.

Codecov Report

Merging #2791 (28d2008) into master (f7d95c4) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head 28d2008 differs from pull request most recent head 882de1e. Consider uploading reports for the commit 882de1e to get more accurate results

@@             Coverage Diff             @@
##            master     #2791     +/-   ##
===========================================
  Coverage   100.00%   100.00%
===========================================
  Files          534       500     -34
  Lines        11168      9846   -1322
===========================================
- Hits         11168      9846   -1322

Impacted Files
Coverage Δ

...ms/parsers/applicant_add_status_response_parser.rb

...ices/ccms/submitters/obtain_document_id_service.rb

app/services/ccms/parsers/base_response_parser.rb

...ms/requestors/non_passported_case_add_requestor.rb

.../ccms/requestors/applicant_add_status_requestor.rb

...rvices/ccms/submitters/upload_documents_service.rb

app/services/ccms/case_add_requestor_factory.rb

app/services/ccms/ccms_error.rb

...es/ccms/parsers/document_upload_response_parser.rb

...vices/ccms/requestors/document_upload_requestor.rb

... and 24 more

Continue to review full report at Codecov.

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


fix filename with $id$ token replacement example

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

@jamesmanning thank you for catching that!


feature/join_circle_bug

When joining secondary circle, users is stuck in re-joining primary circle #295

Note that one of the problem was that i clicked join on a circle Haidhausen from the map view but instead it joined the existing circle again. We need yo vetify that code as well.


Always fails on servo / does too much

Platform: vscode, win10
I'm getting this error on servo:
[ERROR rust_analyzer::reload] failed to switch build data: rust-analyzer failed to run build scripts:
error: failed to run custom build command for `simpleservo_jniapi v0.0.1 (D:\projects\servo\servo\ports\libsimpleservo\jniapi)`

Caused by:
  process didn't exit successfully: `d:\projects\servo\servo\target\debug\build\simpleservo_jniapi-33f3c5db101519e6\build-script-build` (exit code: 101)
  --- stderr
  thread 'main' panicked at 'Please set the ANDROID_NDK environment variable', ports\libsimpleservo\jniapi\build.rs:11:36
  stack backtrace:
     0: std::panicking::begin_panic_handler
               at /rustc/e249ce6b2345587d6e11052779c86adbad626dff\/library\std\src\panicking.rs:495
     1: core::panicking::panic_fmt
               at /rustc/e249ce6b2345587d6e11052779c86adbad626dff\/library\core\src\panicking.rs:106
     2: core::panicking::panic_display<str>
               at /rustc/e249ce6b2345587d6e11052779c86adbad626dff\/library\core\src\panicking.rs:62
     3: core::option::expect_failed
               at /rustc/e249ce6b2345587d6e11052779c86adbad626dff\/library\core\src\option.rs:1638
     4: enum$<core::option::Option<std::ffi::os_str::OsString>, 1,<PHONE_NUMBER>3709551615, Some>::expect<std::ffi::os_str::OsString>
               at /rustc/e249ce6b2345587d6e11052779c86adbad626dff\library\core\src\option.rs:709
     5: build_script_build::main
               at .\build.rs:11
     6: core::ops::function::FnOnce::call_once<void (*)(),tuple$<> >
               at /rustc/e249ce6b2345587d6e11052779c86adbad626dff\library\core\src\ops\function.rs:227
  note: Some details are omitted, run with `RUST_BACKTRACE=full` for a verbose backtrace.
error: build failed

in servo's root directory
tried using "rust-analyzer.cargo.runBuildScripts": false
building servo does work
uncommenting "ports/libsimpleservo/jniapi/" in the root Cargo.toml works, next error is:

failed to run custom build command for `openssl-sys v0.9.67`
[...]
note: vcpkg did not find openssl: Could not find Vcpkg tree: No vcpkg installation found. Set the VCPKG_ROOT environment variable or run 'vcpkg integrate install'

which happens in servo/components/net
Maybe it's more of a question why rust-analyzer seems to look through everything under the root dir?
If I switch to servo/components/allocator, which has it's own Cargo.toml, rust-analyzer will still try to index/run build scripts on the root servo dir.
I don't know how rust-analyzer works internally, but it seems to do a little "too much".
I'm mainly confused because building/running servo works, but this here doesn't

Maybe it's more of a question why rust-analyzer seems to look through everything under the root dir?

It passes --workspace to cargo which builds the entire workspace: https://github.com/rust-analyzer/rust-analyzer/blob/bfb8f73fb3f8a1a7fadfa31bda98a17549a91093/crates/flycheck/src/lib.rs#L214

servo/components/allocator is part of the servo workspace: https://github.com/servo/servo/blob/9693111c9744407730d7fc971a33800a831d575e/Cargo.toml#L1 (it is not explicitly listed, but it is a part because it is an indirect dependency of one of the explicit workspace members and inside a subdir of the workspace root)

I got that too, but let's just say I take a component like allocator and move it to a completely different location.
Now rust-analyzer works (due to the different target-folder and Cargo.lock) - but can that be achieved without moving?
The usecase might be very specific, but these modular sub-components, depending on nothing else, should be buildable without building the entire browser every time.

I don't know how rust-analyzer works internally, but it seems to do a little "too much".

Yeah, that's a surprisingly deep thing. This is needed to make "find usages" and refactorings work. Say, you invoke "rename" on some method in the crate you are working on. The IDE should then update all usages,  which are in reverse dependencies. That is, we need direct dependencies to analyze a particular crate, but we need reverse deps to make refactors useful.
Today, rust-analyzer assumes that "Cargo workspace" is the unit of code you are working on. I can see us providing some extra toggles to tweak this behavior. This is something we need to think about long-term anyway -- if you workspace is google-scale monorepo, you probably don't want a refactor to update the code across monorepo. Though, you might want "find usages" to still work.

@matklad iirc the most confusing part was when I went into one of servos sub-crate folders and ran rust-analyzer, I thought the sub-crate's Cargo.toml would be "used".
This would be a nice default, or maybe another config-option?
Although now I just remembered that the current default is coupled to how Cargo works, right?

Yeah, this all is true. I've opened https://github.com/rust-analyzer/rust-analyzer/issues/11181 for that.


Bump Rack and Rails for security fixes

Vulnerabilities addressed:

CVE-2015-3225 https://groups.google.com/forum/#!topic/rubyonrails-security/gcUbICUmKMc
CVE-2015-3227 https://groups.google.com/forum/#!topic/rubyonrails-security/bahr2JLnxvk

Fixed thanks to a pointer from @heathd

Won't the fix only work for CI as it's pulled in from jenkins.sh?

It works locally if you just run rake default as the env is loaded. This is a temp fix, it'll get fixed properly if/when we bump Rails properly.

Fair enough...


HTMLInfoWindow stays after removing it's marker

Branch: multiple_maps (v2)
If you attempt to remove a Marker, which has currently an open HTMLInfoWindow, the marker will be removed, but the InfoWindow will stay.

Thanks


The net total amount of Profit/Losses over the entire period.

The array has a mix of characters and numbers so need to pick the numbers only then do addition.
xpert has given this message
To pick all the numbers inside the arrays, you can use a combination of map() and filter() methods in JavaScript. Here's an example of how you can do it:
var array = [["x", 1], ["y", 2], ["Z", 3]];
// Use map() to extract the numbers from each subarray
var numbers = array.map(function (subarray) {
// Use filter() to filter out the numbers from each subarray
var subarrayNumbers = subarray.filter(function (element) {
return typeof element === "number";
});
// Return the filtered numbers from each subarray
return subarrayNumbers;
});
// Flatten the array of numbers
var flattenedNumbers = numbers.flat();
console.log(flattenedNumbers);
This code uses the map() method to iterate over each subarray in the main array. Inside the map() callback function, the filter() method is used to filter out the numbers from each subarray. The resulting array of arrays is then flattened using the flat() method to get a single array containing all the numbers. Finally, the flattenedNumbers array is logged to the console.

done closing


Improve error message

I tried to use uWebSocket in alpine linux (docker image) and got error message that 'Compilation of µWebSockets has failed and there is no pre-compiled binary...' and i was able to fix the problem.
https://github.com/uWebSockets/uWebSockets/blob/5f2cf2169ebd5689daf6d337b9b085c6bafae097/nodejs/dist/uws.js#L14-L22
In my case, i was missing 'libc.so.6' file, but instead uwebsockets prints a friendly message, completly ignoring actually exception e. I finally added additional console.log statements to debug the issue.
Exception may happen due to mutiple reasons, so i request you to print e.message also along with friendly error message.

#284


Deprecate extensions

Description

Throw a warning Message to user if any adapter from Extension directory is used for test execution/discovery
We have white-listed MsTestv1, Cpp, Generic, Orderded, CodedUI test adapters

Related issue
RFC https://github.com/Microsoft/vstest-docs/pull/125

@pvlakshm can you please confirm on the warning message


Allocating 8 extra bytes per word?

In glove.c, you increment vector_size at line 67 to account for the bias value.  Additionally, at line 70 you allocate space for (vector_size + 1).  Is this extra 8-byte allocation redundant?
If so, it would increase the amount of memory allocated for word vector, context word vector and gradsq vectors by an unnecessary ~60MB for a corpus of 2M unique words.

It may not be, but we've actually had a lot of issues with segmentation faults in the code base on subtle off-by-one issues like this that we've slowly fixed, so I don't think it's a great idea to change this. It may be implictly expected elsewhere that more space has been malloc'd.


When deletes a dataset, dependant maps preview should have a link to that map

cc @viddo

https://github.com/CartoDB/cartodb/pull/1796

Please @viddo, take care of this issue.

okok

Please, be sure that link is opened in a new tab

I wouldn't force to do that action, user should decide...

Basic users do not know how to open links in new tabs, and that would mean you have to go back to the previous screen, look for the dataset you were looking to delete, and perform the action again (after checking each visualization...)

I'm normally against opening links in new windows, but considering the dialogs are a kind of special case and the browser's back button would not lead to the delete dialog, I agree that opening in a new window is probably a better idea in this case.

You guys decide, I'm not against anything..


Loop when reading hidden prompt from /dev/null

A prompt with {type: "hide"} option loops forever when input is redirected from /dev/null.
To reproduce, create a program like:
const {CliUx} = require('@oclif/core')
CliUx.ux.prompt('Enter input', {type: "hide"})

Then run it taking stdin from /dev/null. Or run npm i and then ./prompt.js < /dev/null in the attached project:
promptloop.zip
Actual behavior: the prompt loops forever.
Expected behavior: the prompt completes as no data is read (as it does without {type: "hide"}).

We're planning on removing the ux module in the next major version and so we don't plan to address any bugs related to it. We suggest switching to a dedicated prompt library like inquirer


⚠ SceneTime has degraded performance

In 4463786, SceneTime ($ST) experienced degraded performance:

HTTP code: 200
Response time: 1491 ms

Resolved: SceneTime performance has improved in 961211d after 8 minutes.


Update the readme to pull a specific version of the UA-Java stack.

Currently the readme just has the user pull the latest snapshot from the UA-Java repo, however, this causes issues with breaking changes upstream. We need to make is so that the pom file is consistent with the readme version.

POM and documentation updated to reflect the 1.3.343 version of the UA stack. Done in commit 249ff4e


ReadOnly mode should provide a simplified & compact rendering of the form.

Is your feature request related to a problem? Please describe.
As discussed in https://github.com/accordproject/concerto-ui/pull/54, when the readOnly prop is true, it would be convenient for the form fields to be more compact.
Describe the solution you'd like
For example, currently,

the buttons and dropdown fields appear as disabled input fields, however it would be more natural for them to be readOnly input fields

Describe alternatives you've considered
Alternatively, all of the fields could be replaced with labels. This is a more fundamental change, and the form would no longer be "a form". This could be an alternative prop, e.g. noFields, textOnly ?

@kanav-raina @jeromesimeon

hello @mttrbrts. Kindly assign this to me.
Thanks!

Odd, I can't seem to add you! GitHub has been having issues recently, I wonder if it's related?


⚠ Bofrost.be has degraded performance

In 7ec9392, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1635 ms

Resolved: Bofrost.be performance has improved in 35e2e57 after 6 minutes.


[DPP-09] Revert to python 2.7

The Boto 2.38 library does not install in NixOs, and version 3 is not ready. Port the app to version 2.7 so that boto can be used.

Created new virtualenv on server named dropp27


Secretless has been through XA with EPV

The demo app is used to validate the experience of adding Secretless with EPV (syncing to DAP) to an existing application using the website and documentation.
Acceptance criteria this card will validate

Secretless is easy to set up, and setup takes less than 10 minutes

This assumes EPV / DAP are already deployed and configured with dual accounts / to use the Kubernetes authenticator

Secretless handles connections transparently, so that requests from the client are appropriately transmitted to the server and messages from the server are propagated back to the client

Requirements
This will involve creating:

The demo app configured to connect via Secretless
Secretless configured to connect to DAP using the Kubernetes authenticator

It is assumed that an administrator has configured EPV to sync database credentials to DAP.
The time to deploy the test app with Secretless will be measured.
The output of this card will be:

[ ] Notes on any issues that arise as a result of these tests
[ ] Info on how long it took to add Secretless to the test app deployment

Since we just ran through XA in #733 and there are minimal differences between EPV and DAP as the source of truth for database credentials (eg you just pull secrets from a different policy branch / add your host as a member of a different group/layer), we will not be doing additional XA as part of this effort
cc @sgnn7


form_for trigger to ff

Changed trigger for form_for, from 'form' to 'ff' because the default sublime 'form' trigger creates a form html tag.

thanks


calibre: fix build

Motivation for this change
Makes calibre compile again, fixing #65313 No real testing done, but it starts again for me, and the viewer works.
Fix maybe technically uncool, splitting qtWrapperArgs, but i don't see how to get PYTHONPATH properly interpolated otherwise.
Things done

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

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

cc @ToxicFrog

Builds and runs for me on NixOS unstable.

Why are we wrapping in installPhase?
I don't think the proper values would even be collected at that phase. It should be postFixup.

It's been wrapped there since initial packaging 10 years ago a4473b820, I'm assuming because https://github.com/kovidgoyal/calibre/tree/master/setup looks a bit extreme.
But :+1: on an overhaul, MAGICK_INC etc look unused as well.

Does it work without https://github.com/NixOS/nixpkgs/issues/65399 fix ?
Here it fails with
$ calibre
qt.qpa.plugin: Could not find the Qt platform plugin "xcb" in ""
This application failed to start because no Qt platform plugin could be initialized. Reinstalling the application may fix this problem.

Aborted

I can fix this today :+1:

Does it work without #65399 fix ?
Here it fails with
$ calibre
qt.qpa.plugin: Could not find the Qt platform plugin "xcb" in ""
This application failed to start because no Qt platform plugin could be initialized. Reinstalling the application may fix this problem.

Aborted

I checked and it appears to be wrapped correctly, though I decided to make it a little cleaner #66483

Please do not use a custom builder function unless really necessary. I know it's documented as the way with Qt5, but we should avoid it when possible.

Please do not use a custom builder function unless really necessary. I know it's documented as the way with Qt5, but we should avoid it when possible.

Did you mean to comment that on my PR? (context confused)

Additionally, I think it would be appropriate for calibre. I believe it is just a qt application?


[FR] Use flag to select identity when using gopass clone

First of all, thank you for the awesome tool that Gopass is!
Summary
I'd like to clone an existing password store (git) using one command without the need for extra input.
Steps To Reproduce
gopass clone<EMAIL_ADDRESS>Configuring git repository ...
Use Identity<EMAIL_ADDRESS>for password store git config? [Y/n/q]: Y
Your password store is ready to use! Have a look around: `gopass list`

Expected behavior
gopass clone --id<EMAIL_ADDRESS><EMAIL_ADDRESS>Configuring git repository ...
Your password store is ready to use! Have a look around: `gopass list`

A flag that would allow to specify either:

The email address associated with the GPG key
The 8 character ID found by typing "gpg --list-keys" and looking at the "pub" line
The GPG key fingerprint found by typing "gpg --fingerprint" and removing all of the spaces

just like gopass init would be awesone.
Note
I've been looking through the documentation and the source code, it seems like it's note implemented but might have missed it.

Just noticed one can simply do the following:
gopass --yes clone<EMAIL_ADDRESS>
To clone the repo in one line. However, this doesn't allow to choose which key to use if said repo has multiple recipients and you have multiple keys associated with the repo.
I don't see any real use case for this so closing this issue but feel free to open it again if needed.

Sorry for this long post, seems like I'm a bit confused about the use of git clone.
I'm trying to create ~/.password-store based on an already existing store on git.
I first tried
gopass --yes clone<EMAIL_ADDRESS>
this returns
Configuring git repository ...
Your password store is ready to use! Have a look around: `gopass list`

But nothing gets created (I guess this is an issue, an error message should probably be returned).
I then noticed the docs says:

Please note that all cloned repositories must already have been initialized with gopass. (See the previous section for more details.)

So I tried the following:
# This commands works fine and create an empty password store at ~/.password-store
gopass --yes init <key-id>

gopass --yes clone<EMAIL_ADDRESS>
But this last commands returns: Error: Can not clone<EMAIL_ADDRESS>to the root store, as this store is already initialized. Please try cloning to a submount: gopass clone<EMAIL_ADDRESS>sub`
I'm confused, the docs says

all cloned repositories must already have been initialized

and this commands says

Can not clone ... to the root store, as this store is already initialized

Am I missing something?
Does the gopass clone only allows to clone to a submount but not the root password store?
Note
I can simply use git directly:
git clone<EMAIL_ADDRESS>/home/$USER/.password-store

which works perfectly.
I'm just wondering whether this is an issue in Gopass, the documentation or whether I missed something.
Thanks a lot for reading (and sorry again for those long posts)

I reopen this issue as there's no answer to the "store is already initialized" error.
Following the documentation I did
gopass init <key-id>
gopass clone<EMAIL_ADDRESS>
and got the same error as OP.

Can you point me at the location in the documentation that does suggest this, please? @yannfill
init followed by clone is wrong. clone should take care of everything already.

In https://github.com/gopasspw/gopass/blob/master/docs/features.md#cloning-an-existing-password-store il says

Please note that all cloned repositories must already have been initialized with gopass. (See the previous section for more details.)

If I just clone the repo and add a secret, then when I gopass sync I get
🚥 Syncing with all remotes ...
[<root>]
   git pull and push ... OK (no changes)
   importing missing keys ... OK
   exporting missing keys ... nothing to do
   done
✅ All done

and the remote repo is not updated.

That piece of documentation is correct but definitely misleading. What I meant was: The remote repo must have been initialized / created before, otherwise clone will fail.
Just cloning the repo and adding a repo should work.
Are you sure the newly created secret does not show up on the remote?
Or are you only worried about it not showing up in the sync output?
I'm asking because gopass started to default to autosync some time ago.

I checked the remote repo and it hasn't been updated.

I just tested using the following:
~ $ GOPASS_HOMEDIR=test1162 gopass init
[...]

~ $ GOPASS_HOMEDIR=test1162 gopass generate test2 15
✅ Password for entry "test2" generated
Not printing secrets by default. Use 'gopass show test2' to display the password.

~ $ GOPASS_HOMEDIR=test1162 gopass git remote add origin git@github.com:AnomalRoil/pwstore-1162.git

~ $ GOPASS_HOMEDIR=test1162 gopass git push
✅ Pushed to git remote

And then I tried cloning it (noticed I changed the HOMEDIR to have a "clean" setup):
~ $ GOPASS_HOMEDIR=test1162-clone gopass clone git@github.com:AnomalRoil/pwstore-1162.git
[...]
Your password store is ready to use! Have a look around: `gopass list`

~ $ GOPASS_HOMEDIR=test1162-clone gopass list
⚠ You haven't checked for updates in a while. Run 'gopass version' or 'gopass update' to check.
gopass
└── test2

~ $ GOPASS_HOMEDIR=test1162-clone gopass generate test-afterclone 15
✅ Password for entry "test-afterclone" generated
Not printing secrets by default. Use 'gopass show test-afterclone' to display the password.
⚠ Failed to pull before git push: exit status 128: From github.com:AnomalRoil/pwstore-1162
 * branch            master     -> FETCH_HEAD
hint: You have divergent branches and need to specify how to reconcile them.
hint: You can do so by running one of the following commands sometime before
hint: your next pull:
hint:
hint:   git config pull.rebase false  # merge (the default strategy)
hint:   git config pull.rebase true   # rebase
hint:   git config pull.ff only       # fast-forward only
hint:
hint: You can replace "git config" with "git config --global" to set a default
hint: preference for all repositories. You can also pass --rebase, --no-rebase,
hint: or --ff-only on the command line to override the configured default per
hint: invocation.
fatal: Need to specify how to reconcile divergent branches.

This might look like it failed, but it's just git complaining a little bit.
I checked the remote and everything actually worked and the new password was uploaded...
We might just want to correct the way we are initializing the cloned repo, to apply the same fix as when we init the git repo in the first place.
But it's kinda strange since it should have ran already:
https://github.com/gopasspw/gopass/blob/bbf17512936f85f9a051e75bc21c5e3f9fdc9a2a/internal/action/clone.go#L101
But still, that seems to be an unrelated git issue, compared to your current issue.

Running gopass fsck correctly fixes the issue of lack of proper git config of the password store after having used gopass clone ...

After deleting ~/.config/gopass/, ~/.local/share/gopass/ and ~/ .cache/gopass/ I tried
gopass init <key-id>

then
gopass git remote add origin<EMAIL_ADDRESS>
and then gopass git pull failed with
exit status 128: From repo
 * branch            master     -> FETCH_HEAD
fatal: refusing to merge unrelated histories

gopass sync failed too:
🚥 Syncing with all remotes ...
[<root>]
   git pull and push ... ⚠ Failed to pull before git push: exit status 128: From repo
 * branch            master     -> FETCH_HEAD
 * [new branch]      master     -> origin/master
fatal: refusing to merge unrelated histories
❌ Failed to push "<root>" to its remote: exit status 1: To repo.git
 ! [rejected]        master -> master (non-fast-forward)
error: failed to push some refs to 'repo.git'
hint: Updates were rejected because the tip of your current branch is behind
hint: its remote counterpart. Integrate the remote changes (e.g.
hint: 'git pull ...') before pushing again.
hint: See the 'Note about fast-forwards' in 'git push --help' for details.
✅ All done

Don't do the gopass init at all.
Delete everything again and try directly with:
gopass clone<EMAIL_ADDRESS>gopass fsck

That fsck step should check that everything is working properly.

Still not working. gopass fsck returned no error. I added a secret and gopass sync:
🚥 Syncing with all remotes ...
[<root>]
   git pull and push ... OK (no changes)
   importing missing keys ... OK
   exporting missing keys ... nothing to do
   done
✅ All done

gopass git push return
✅ Pushed to git remote

But the remote repo is still not updated.

If you go to the password store folder, typically ~/.local/share/gopass/root and you use git there directly, what does the following tell you:
git status
git log
git remote -v

git status:
On branch master
Your branch is up to date with 'origin/master'.

Changes to be committed:
  (use "git restore --staged <file>..." to unstage)
	new file:   secret.gpg

git log shows past commits, no new one.
git remote -v:
origin	my-repo/gopass.git (fetch)
origin	my-repo/gopass.git (push)

What does git branch -a tell you?

Ok, I think something got mixed up there.
@yannfill
First, the documentation can be misleading. If you want to start from zero, then you have to choose between 2 ways.

Remote git is always initialized with gopass => gopass setup
Empty git remote => gopass init && gopass git remote add origin ... && gopass sync

Your other issue is a broken git run. I can reproduce it with incorrect connection of Repo (filled) with the gopass init initialization.
The reason is that "init" creates a new local repo and does the first commits there.
.local/share/gopass/stores/root$ git log --graph
* commit d46a07b2057c0efb52fa41985cef3fcaf3bcb065 (HEAD -> master)
| Author: jens<EMAIL_ADDRESS>| Date:   Wed Dec 22 20:02:13 2021 +0000
|
|     Add current content of password store
|
* commit 1515ba7cba4581511292b2d2e4d25f4acf124e18
  Author: jens<EMAIL_ADDRESS>  Date:   Wed Dec 22 20:02:13 2021 +0000

      Configure git repository for gpg file diff.

And that's why the error message:
fatal: refusing to merge unrelated histories

I did gopass setup, choose the same GPG key used on the remote repo, add the remote repo et get the same error (why would gopass push before pull at init when adding an existing store ?)
So I cloned my repo and rsync the directory with the local store and everything is now working. Issue resolved for me, even if the gopass internal steps are still failing.

The original issue (specifying the identity used for the cloned repo) can be solved by specifying --name and --email to gopass clone. The other issues discussed here I can't reproduce.


Update REX-Ray based on libStorage changes

This patch updates REX-Ray based on the changes to libStorage's build model and package layout.

Codecov Report

Merging #929 into master will decrease coverage by 0.12%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##           master    #929      +/-   ##
=========================================
- Coverage    8.79%   8.67%   -0.13%
=========================================
  Files          25      23       -2
  Lines        2980    3529     +549
=========================================
+ Hits          262     306      +44
- Misses       2682    3189     +507
+ Partials       36      34       -2

Impacted Files
Coverage Δ

util/util.go
25.52% <0%> (-0.7%)
:arrow_down:

cli/cli/cmds_00_other.go
1.69% <0%> (-0.39%)
:arrow_down:

cli/cli/cli_formatter.go
52.94% <0%> (-0.28%)
:arrow_down:

cli/cli/cmds_01_adapter.go
2.7% <0%> (-0.24%)
:arrow_down:

cli/cli/cmds_02_device.go
1.13% <0%> (-0.18%)
:arrow_down:

cli/cli/installer.go
1.14% <0%> (-0.17%)
:arrow_down:

cli/cli/cmds_51_flexrex.go
1.56% <0%> (-0.14%)
:arrow_down:

cli/cli/cmds_04_service.go
1.49% <0%> (-0.13%)
:arrow_down:

cli/cli/cli.go
0.71% <0%> (-0.12%)
:arrow_down:

cli/cli/cmds_05_module.go
0.65% <0%> (-0.12%)
:arrow_down:

... and 12 more

Continue to review full report at Codecov.

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


Legacy context API has been detected within a strict-mode tree. (Collapse)

components: Collapse, Transition

reactstrap version:  `8.6.1'

import method es

react version ^16.11.0

bootstrap version ^4.3.1

What is happening?
things are working but the following console warning is appearing stating that the old contextApi is being used.
What should be happening?
the console should not be generating a warning message;  my team is trained to keep the console clean in order to troubleshoot 'real' errors
Steps to reproduce issue

create a standard react app
create a generic reactstrap menu with collapse
observe console warning message

Error message in console

Warning: Legacy context API has been detected within a strict-mode tree.
The old API will be supported in all 16.x releases, but applications using it should migrate to the new version.
Please update the following components: Transition
Learn more about this warning here: https://fb.me/react-legacy-context
in Transition (created by Collapse)
in Collapse (at header.jsx:30)
in nav (created by Navbar)
in Navbar (at header.jsx:25)
in div (at header.jsx:24)
in Header (at App.js:30)
in Router (at App.js:29)
in div (created by Container)
in Container (at App.js:28)
in div (at App.js:27)
in App (at src/index.js:18)
in Provider (at src/index.js:16)
in StrictMode (at src/index.js:15)

Code
//works but different warning messages
https://stackblitz.com/edit/reactstrap-qtju12?file=Example.js
on my machine (yeah i hate those words as well) it displays the console message when I add Collapse in.

Dupe of #1962


⚠ Bofrost.hr has degraded performance

In d5d73c2, Bofrost.hr (https://www.bofrost.hr) experienced degraded performance:

HTTP code: 200
Response time: 2927 ms

Resolved: Bofrost.hr performance has improved in 51a7a53.


Is this project deprecated?

Is the development of this project stopped?
No updates or fixes.

Well not really, but the core functionality works as intended, so there wasn't the need to do any big feature changes.
After all this is not a commercial product.
If you notice a bug feel free to report it.


Change branch dependencies on TSC, LLBuild, and SwiftDriver from main to release/5.5

This switches the branch dependencies for swift-tools-support-core, swift-llbuild, and swift-driver (as we do for every release branch).  This will be come more important as main starts to diverge from release/5.5.
This depends on https://github.com/apple/swift-driver/pull/626

Will start tests once the SwiftDriver change is merged.  No point until then since they will fail.

@swift-ci please test

Just for the future: it would be better to use cross-repo testing for these kind of changes

Just for the future: it would be better to use cross-repo testing for these kind of changes

Thanks for the reminder.  For reference, I found this:  https://github.com/apple/swift/blob/main/docs/ContinuousIntegration.md#cross-repository-testing