Transparent images not drawn in Safari and iOS

You can see the issue happen in this small clip on Safari:
http://gph.is/2rs5HM3
Working example here:
http://dhhq.duckdns.org/configurator/4
In chrome the same code doesn't produce this effect and images are always shown.
Adding multiple transparent images, each one with a different size and aspect ratio, causes the images to disappear at certain zoom levels and bounds positions.
There seems to be a pattern as images disappear when one of the borders goes offscreen.
In this clip you can see how images disappear even from the sketch canvas:
http://gph.is/2r3oGfE
I can test and debug on Safari and iOS but I'd need directions to the code most probably responsible for the issue.

@sickrandir Thank you for digging into this! I figure the part of the code changed in #1035 is a good starting point:
https://github.com/openseadragon/openseadragon/pull/1035/files
You might try putting some console logs in there to see if you can notice a pattern for when it works vs when it doesn't. Note that in OSD you'll want to use $.console.log since it won't cause problems with the linter.

I'm logging the 'bounds' just before this.context.drawImage but I just can't seem to find a pattern. Non neg values. It would help to break/log on drawImage just for the tiledImage that disappear. Do you know how to do that? Other guesses?

Indeed... perhaps it's analogous to the issue we had blending from sketch to main, but for drawing into sketch originally. That happens here:
https://github.com/openseadragon/openseadragon/blob/master/src/drawer.js#L303
...which calls to here:
https://github.com/openseadragon/openseadragon/blob/master/src/tile.js#L287

Looking at what happens here:
https://github.com/openseadragon/openseadragon/blob/master/src/tile.js#L287 and logging just for a selected tile that disappear.
The method drawCanvas gets executed even when the tile disappear and looking at the values of: position.x, position.y, size.x, size.y I can't seem to find a pattern.
One problem is that I don't fully understand how drawing happens and what can cause an image to not be drawn.

Well, my assumption is that it's something like #1035 where the image would normally be drawn on any other browser, but because of a bug in Safari, it's not being drawn. So it makes sense that drawCanvas is getting called, since OSD thinks it should be drawing that tile, but somehow Safari refuses to draw it.
I'm sorry to hear there's no discernible pattern. I suppose it's possible we're barking up the wrong tree by looking at how it gets drawn to the sketch canvas. One possible thing to try there would be to attach the sketch canvas to the page so you can actually see what's happening in it. If things get drawn to it but don't make it to the final canvas, then we know the problem is in the copy stage not the original draw stage. Either way it narrows it down.
Btw, it might be useful for you to make a simplified repro project... both because it allows you to isolate the issue, and also because then I could take a look directly.

Thanks Ian.
I made the simplest codepen I could. OSD is externally linked to the latest master version in which I just added debug logs. https://codepen.io/sickrandir/pen/qjOode
I logged the values of: rendered.canvas.width, rendered.canvas.height, position.x, position.y, size.x, size.y
in drawCanvas method for the tiles of the disappearing image just for the small amount of canvas drag that makes the image disappear. The log trace stops and the image is not dispayed. The log is here: https://gist.github.com/sickrandir/0cfc017ef97bc2b0128c5b6de0ed533c
This is the log of the inverse drag that make the image reappear:
https://gist.github.com/sickrandir/fbab0995745934dc3706c0ba5879efdc
The only thing I notice is that rendered.canvas.height and position.y keeps oscillating on every log, but that does happen both when the image is displayed and when is hidden.

I tried again the sketchCanvas scenario with this simpler example and I don't see the image disappear. So it seemed that I was mistaken and maybe the drawing on the sketchCanvas with multiple images looked confused and led me to believe the images we actually disappearing. Here the pen with the sketchCanvas: https://codepen.io/sickrandir/pen/vZNVKP

I focused again on the blendSketch method.
I added logging like this:
   if (bounds.x < 0) {             	$.console.log("NEG X: ", bounds.x);                 bounds.width += bounds.x;                 bounds.x = 0;             }             if (bounds.width > this.canvas.width) {             	$.console.log("bounds larger than canvas");                 bounds.width = this.canvas.width;             }             if (bounds.y < 0) {             	$.console.log("NEG Y: ", bounds.y);                 bounds.height += bounds.y;                 bounds.y = 0;             }             if (bounds.height > this.canvas.height) {                 $.console.log("bounds higher than canvas");                 bounds.height = this.canvas.height;             }
and you can see in this small clip that an image can disappear without triggering any of the corner cases addressed in the patch #1035

@sickrandir This is great debugging! I'm sorry I've been so quiet... fighting multiple fires on other projects. I'm still hoping to get to this soon.

@sickrandir Okay, I think I've fixed it... please try #1222.

@iangilman Thank you so much! I can confirm that the bug is fixed both on Safari on Mac and IOS!

@sickrandir Excellent; thank you for verifying!


Added title and made it dynamic with input and responsive #132

Issue #132

Added the title on the Confirm.js page according to input value.
Adjusted the font size for the smaller screens in bestbutton.css file.

Result:

@jumankaushik always remember that one PR must only solve one issue, it is difficult to review a PR doing two things, please keep this PR about adding that text, edit the title and discription accordingly : )

Okay

Done. Kindly check!

Thanks for your contribution @jumankaushik


Add external courses on SolidJS

Add SolidJS course present in Udemy by Filip Jerga

Thank you for your contribution @harishkrishnan24 👍


Merge release to develop for release 4.5.0

PR Details
Description
Merge PR to formalize release of master into develop as part of CARMA release process for 4.5.0.
PR created automatically via CARMA release tool and Github CLI
Motivation and Context
This PR brings the tested and reviewed contents of the master release/candidate into develop for final release preparation
How Has This Been Tested?
This release branch (minus final version number changes) has been tested through the CARMA verification test plan.
Checklist:

[X] I have added any new packages to the sonar-scanner.properties file
[X] My change requires a change to the documentation.
[X] I have updated the documentation accordingly.
[X] I have read the CONTRIBUTING document.
CARMA Contributing Guide
[X] I have added tests to cover my changes.
[X] All new and existing tests passed.

Closing in favor of #76 which includes intermediate branch with fixes to uncouple build and ci scripts from github branch and version


X.509 Name support & invalid output w/ multiple RDNs

Describe the bug:
Ideally, one should be able to precisely control the generated certs, including the X.509 Subject. This is sort of there; in particular these are the relevant docs (for CertificateSpec); it has:
commonName: string
subject:
  organizations  []string | (Optional) Organizations to be used on the Certificate.
  countries  []string | (Optional) Countries to be used on the Certificate.
  organizationalUnits  []string | (Optional) Organizational Units to be used on the Certificate.
  localities  []string | (Optional) Cities to be used on the Certificate.
  provinces  []string | (Optional) State/Provinces to be used on the Certificate.
  streetAddresses  []string | (Optional) Street addresses to be used on the Certificate.
  postalCodes  []string | (Optional) Postal codes to be used on the Certificate.
  serialNumber  string | (Optional)

If we look at this through the lens of an X.509 certificate, this is peculiar. commonName is part of the subject, but isn't listed under subject. Subject itself is a mapping, so control over the actual ordering is lost, but a default would probably suffice for most purposes.
If we actually use this, say,
    organizationalUnits: [Databases, Postgres]

We get back the following certificate:
        Subject: C = US, …, OU = Databases + OU = Postgres, …, CN = postgres

The + here indicates that this is a "multi-valued" RDN (Subject is a sequence of RDNs, here separated by commas); LDAP wiki says,

In practice, Relative Distinguished Name containing multiple name-value pairs (called "Multi-Valued RDNs") are rare, but they can be useful at times when either there is no unique attribute in the entry or you want to ensure that the entry's Distinguished Name contains some useful identifying information.

The RDN sequence is supposed to indicate a location within a hierarchy, and particularly so with OU. A "multi-valued" RDN is a set, that is, A = foo + B = bar is a set (with no particular order) holding two things, A & B; it's used when the same entry in the hierarchy can be identified multiple ways. But we're not naming the same OU two different ways, we're trying to refer to two separate OUs in the hierarchy, that is, an OU named "Posgres" in the OU named "Databases", not a single thing.
This diagram from the X.501 standard is fairly illustrative of the intent:

(The "OU = Sales, L = Ipswitch" level is a multi-valued RDN. What we are attempting to create would be two levels, each with just "OU = …".)
This particular multi-valued RDNs is also not valid:

The set that forms an RDN contains exactly one AttributeTypeAndValue for each attribute which contains distinguished values in the entry; that is, a given attribute type cannot appear twice in the same RDN.

(Here, the attribute type OU is appearing twice in the same RDN.)
Expected behaviour:
The most sensible thing, to me, is to just have the RDN sequence that the Subject is comprised of exposed to the user directly. Ignoring multi-valued RDN, that is a sequence of (type, value) pairs. Providing anything that isn't an RDN sequence leaves a user guessing as to what algorithm gets used to transform what is specified in a Certificate into the actual certificate.
(I understand having a more user-friendly layer, and an more raw layer. But the current interface doesn't really provide either.)
Steps to reproduce the bug:
Use the Secret and Issuer from the CA docs in the sandbox NS.
Call this cert.yaml:
apiVersion: cert-manager.io/v1beta1
kind: Certificate
metadata:
  name: pg.client.postgres
  namespace: sandbox
spec:
  subject:
    countries: [US]
    organizations: [Widget Co]
    organizationalUnits: [Databases, Postgres]
  commonName: postgres
  duration: 720h  # 30 d
  renewBefore: 336h  # 14 d
  secretName: pg.client.postgres
  issuerRef:
    kind: Issuer
    name: ca-issuer

Apply the Secret & Issuer from the docs, then apply this Certificate:
kubectl apply -n sandbox -f cert.yaml

Grab the resulting subject:
» kubectl get secret -n sandbox pg.client.postgres -o 'jsonpath={ .data.tls\.crt }' | base64 -d | openssl x509 -noout -text | grep Subject:
        Subject: C = US, O = Widget Co, OU = Databases + OU = Postgres, CN = postgres

Note that it's a (malformed) multi-valued RDN, instead of two RDNs indicating one OU within another OU; what I was going for was C = US, O = Widget Co, OU = Databases, OU = Postgres, CN = postgres.
Anything else we need to know?
The root of this seems to be that cert-manager, quite reasonably, uses golang's pkix module. But that module states,

ToRDNSequence converts n into a single RDNSequence. The following attributes are encoded as multi-value RDNs: [most of the attributes on CertificateSpec.subject]

By this it seems to mean that it, for all those "helpful" attributes on Name which are lists, it will create a single RDN with all entries from the list represented as a multi-value RDN, which will always be invalid for n > 1. The module also states,

Note that Name is only an approximation of the X.509 structure. If an accurate representation is needed, asn1.Unmarshal the raw subject or issuer as an RDNSequence.

…this is somewhat of an understatement: pkix's Name is not capable of generating arbitrary RDN sequences, it doesn't guarantee that an RDN sequence read into it will round-trip, and it generates malformed RDN sequences given reasonable inputs. It would be great, I think, to take the advice in the docs here, and use RDNSequence directly.
(It would seem to me that putting >1 element into any of pkix.Name's members would result in a malformed RDN, and likely not what a user knowledgeable in X.50x intended. I filed a bug about this with Go, but their position is mostly what I expected: the behavior is regrettable, but unchangeable due to stability guarantees.)
If cert-manager represented an RDNSequence directly in YAML, pretty much 1:1, exactly as it is defined, it would look something like this, I think:
subject:
  - C: US
  - O: FooCorp
  - OU: Databases
  - OU: Postgres

It's still fairly terse, and all valid representations are representable. Even multi-valued RDNs are possible (e.g., the standard's example would be - {OU: Sales, L: Ipswitch})
To implement this, it would require changing the format of CertificateSpec, and taking the route pkix suggests of using RDNSequence directly.
Environment details:

Kubernetes version (e.g. v1.10.2): v1.16.8
Cloud-provider/provisioner (e.g. GKE, kops AWS, etc): N/A (but none, in my setup.)
cert-manager version (e.g. v0.4.0): v0.16.1
Install method (e.g. helm or static manifests): helm

/kind bug

Been looking into this for a bit, I am not too much a fan of changing the subject field as these are not too user friendly IMO.
An alternative could be overrideRDN (name pending) which is a string value that allows full freeform RDN?
Another alternative can be actually fixing the pkix behavior internally but soft forking the library, something we already did with ACME.
What do you think?

Interested in a fix for this issue.
/remove-lifecycle stale

/remove-lifecycle rotten

We are currently being bitten by this bug as well.

An alternative could be overrideRDN (name pending) which is a string value that allows full freeform RDN?
@meyskens sounds some like something that would be useful for us. As we auto-generate the Certificate resources anyway so no harm done.

Is this something that you would be willing to accept as a contribution?

I just took a look at this; thank you for writing such a clear and descriptive issue! The go bug that was linked was particularly descriptive and helpful :raised_hands:  I think I can see the problem here and the motivation for a fix.
I think I see some risk here; the "raw" Subject would be written into the PEM-encoded CSR when a CertificateRequest is created in response to the Certificate but I don't know if all of our Issuers are going to respect that. It'd need some careful testing with e.g. Vault and other issuer types.

overrideRDN which is a string value that allows full freeform RDN?

I'd gladly review a PR which implemented something like this. It seems reasonable. It would probably need to be behind a feature gate to start off with, similar to how https://github.com/cert-manager/cert-manager/pull/4598 was feature gated.
If someone does end up raising a PR relating to this I'd encourage you to mention it in #cert-manager-dev on Kubernetes slack!

Seems like #5002 might address this issue. Will check once it's included in a release.

Now that #5002 has been released in cert-manager 1.9, I think this can be closed! Please feel free to re-open if I was mistaken


Error parsing reference: "golang:alpine as builder" is not a valid repository/tag

$ docker --version
Docker version 17.03.1-ce, build c6d412e
alertmanager-sns-forwarder]$ docker build -t alertmanager-sns-forwarder:0.1 .
Sending build context to Docker daemon 47.62 kB
Step 1/10 : FROM golang:alpine as builder
Error parsing reference: "golang:alpine as builder" is not a valid repository/tag: invalid reference format

Closing due to lack of feedback. If the provided explanation is not enough and you'd like some more help, please reopen.


Mining Blasters are Good

This PR buffs the following mining tools:
Protokinetic Accelerators

Are now fully automatic
Can continuously fire at rifle speeds.

Mining Lasers (All of Them)

Now fire 5 shots at a time
Now have a shot capacity of 20, taking advantage of their base maxbattery charge.

This should fulfill: https://discord.com/channels/1194256269792510022/1312549658656112640/1313905841287725099

No on the protokinetic accelerator changes.

Ist icht vorbei...very well.


AnnData file integration user stories

Stories:
Story 1. As a taxonomy editor, I want my taxonomy to be linked to the cells being annotated in order that my annotations can be synchronised with the a representation of data about these cells, e.g.  in a cell by gene matrix in the form of a file (e.g. AnnData) or a matrix store DB.  I want this in order to be able to update the AnnData file for use in analysis which will inform edits to the Taxonomy. Without a robust system of linking to Cell IDs, we are relying on names and/or cluster IDs to link the two. There is a serious danger that name or ID changes will  break these links.
TDT solutions:

CAS stores cell IDs for clusters.  A link to an H5AD file supports initial population of these IDs  to a taxonomy seeded from a spreadsheet.
TDT supports testing of a linked AnnData file to see if annotations are in sync with a taxonomy and can be safely updatable.
TDT support updating of cell annotations  in an AnnData file from a linked taxonomy.

Story 2. As a taxonomy editor planning to publish an AnnData file to CZ CELLxGENE I want to generate an AnnData file for submission to CellXGene that is synchronised with the latest release of my taxonomy (resolvable via a Persistent URL) this means that:

annotations are synchronised with my taxonomy
appropriate cell ontology annotations are present in the cell_type field mandataed by CZ CELLxGENE
other details of my taxonomy are stored in the the AnnData file header.
(Note - we also need mechanisms to synchronise other metadata with the fields mandatated by CZ CELLxGENE, but is probably outside the scope of TDT)

Story 3: As a taxonomy editor I wish to edit or validate a list of marker genes in TDT, ensuring that they are in-sync with the genes in a reference Cell By Gene Matrix.
TDT support requires:

#38
[ ] Populate gene reference table from Linked CxG file.
[ ] Addition of support for validating gene lists and generating reports - for any column with data-type gene list
[ ] Autosuggest for gene list fields.

Tasks

@AvolaAmg @hkir-dev - please review

Reviewed story one. (more reviews to come)
I re-worded story 1 a little bit. Feel free to incorporate it as much as you think is needed.
User story 1 - Keeping an up to date taxonomy as an editor.
As an editor of a particular taxonomy, I want the information present in the taxonomy to be synchronised to the cells I am annotating. In doing so, the extended informations on the cells will also be synchronised to the taxonomy. In a taxonomy, an example of extended informations of a particular cell is the gene matrix related to that cell, this matrix can be stored as a AnnData file (extension .h5df) or as DB matrix.

what is a DB matrix? this is not part of the user story but I would like to include a small definition of it to be clearer.

The purpose of this integration is to facilitate the updating of the AnnData file, which is essential for conducting analyses that can lead to further refinements in the taxonomy. Currently, Cell IDs are linked to cluster IDs or to cluster names. However, this is a risky approach as changes in cluster names or cluster IDs could disrupt the link between Cell IDs and Clusters. To avoid breaking the links between cell IDs and cluster IDs the Taxonomy Data Tool (TDT) offers several solutions.
As a taxonomy editor, I can avoid the problem of losing the link between Cell IDs and Cluster IDs by using TDT. In fact, TDT stores the information related to each cell ID (including cluster IDs) under the Cell Annotation Schema format, by using TDT I have the informations related to each cell ID stored under the Cell Annotation Schema (CAS). In addition, as an editor I want an updated taxonomy that takes into account the informations present in linked AnnData file for each Cell IDs and I want that the taxonomy synchronises to the AnnData files when changes are applied. For this reason, as an editor I will use TDT which has several in-built features to maintain the link between cell IDs, cell informations present in AnnData files. In fact,
to ensure the link between Cell IDs and informations related to Cell IDs, TDT populates a new taxonomy by extrapolating informations from an .h5df file; TDT test AnnData files linked to Cell IDs to see if annotations are in sync with a taxonomy; and, TDT updates cell annotations in an AnnData file from a linked taxonomy.
These TDT features are designed to streamline the process of maintaining an accurate and up-to-date taxonomy, safeguarding against the potential pitfalls associated with data linkage and synchronisation.


unpfs: fix build

Fix build by updating to the most recent revision.
ZHF: #144627
@NixOS/nixos-release-managers

Motivation for this change
Things done

Built on platform(s)

[X] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[X] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[X] Tested basic functionality of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[X] Fits CONTRIBUTING.md.

Please update the version and follow https://github.com/NixOS/nixpkgs/blob/master/CONTRIBUTING.md

@ofborg build unpfs

Looks like we have a fixed-output hash mismatch problem here?

Does the cargoSha256 include the package version? That's the only change I made. If so I'll update it but I would like to be sure.


What is the Office 365 tenant prefix and can we use an existing tenant to join Office 365 Developer ?

Hello,
This article talks about how we can find the Office 365 Tenant ID
https://docs.microsoft.com/en-us/onedrive/find-your-office-365-tenant-id
To find your Office 365 tenant ID in the Azure AD admin center

Sign in to the Azure Active Directory admin center as a global or user management admin.

Under Manage, select Properties. The tenant ID is shown in the Directory ID box.

I have 2 questions:

Now my question is, what part of this long ID is the prefix?

This part on the page(https://docs.microsoft.com/en-US/sharepoint/dev/spfx/set-up-your-developer-tenant)  is confusing:
"If you already have an Office 365 tenant, see the section Create app catalog site.

If you don't have one, you can get an Office 365 developer subscription when you join the Office 365 Developer Program. ,,"
"Make sure that you are signed out of any existing Office 365 tenants before you sign up for the Office 365 Developer Program."
So, on the one hand it says sign up for Office 365 Developer if you don't have a tenant but then goes on to say sign out of any existing Office 365 tenants. I already have an Office 365 tenant. Can't I sign into Office 365 Developer Program with that tenant?
Your help is appreciated.
Category

[X ] Question

The Office 365 Developer Program gives you a completely different Office 365 Tenant - as a new subscription, so it's not tied your current Office 365 tenant. The documentation is just stating that if you don't have a tenant already, you can use the Developer Program to get one.
However, if you'd like to build and test solutions in your current tenant, you are also able to do that. The option is really yours. Personally, I like have use a developer tenant for developing, that is separate from my production Office 365 environment.


Courthouse and Manage Warrants Features/Bugs

Describe the feature

[x] Could we have the Manage Warrants option enabled via Discord Courthouse role? Right now I only see an option for Expungement Requests and Name Change requests as permissions under it. On the Admin panel when setting up Discord Role permissions.

[x] Also reporting a bug. Manage Warrants permission under User Permissions is not working for the Pending Warrants page. Courthouse members cannot see the Pending Warrants tab at all. They can only see Name Change and Expungement Requests. I have someone who has the permission for their user as well but they cannot see the Pending Warrants page. They can only see the inactive warrant on a citizens profile in Dashboard and thats it.

[ ] Inactive Warrants - can these be hidden by default from everyone? Officers are confusing this and they think its an active warrant. (They are stupid, its FiveM after all) so maybe it should be entirely hidden when its inactive and only show in Dashboard when its active.

Permissions issue has been fixed. You'll have to grant the user ManagePendingWarrants/Manage Pending Warrants permission:

Inactive Warrants - can these be hidden by default from everyone? Officers are confusing this and they think its an active warrant. (They are stupid, its FiveM after all) so maybe it should be entirely hidden when its inactive and only show in Dashboard when its active.

Where are these shown exactly again?

When you search a criminal by name. But I think its fixed now? I don't see Inactive Warrant on their character anymore from LEO Dashboard which is good.

Is it possible to also add Manage Courthose tab here instead of it only being accessible from Admin panel?

Another Question:

Is it possible to also add Manage Courthose tab here instead of it only being accessible from Admin panel?

Sure, will try add this later tonight or tomorrow!

The link has been added! Will close this issue for now since that last item seemed fixed


#516 Added the SRE Roadmap

Issue Title
Please enter the title of the issue related to your pull request.
Enter the issue title here-
Addition of SRE Roadmap

[X] I have provided the issue title.

Info about the Related Issue
What's the goal of the project?
Describe the aim of the project-
Addition of SRE Roadmap

[X] I have described the aim of the project.

Name
Please mention your name.
Enter your name here-
V Pratheek

[X] I have provided my name.

GitHub ID
Please mention your GitHub ID.
Enter your GitHub ID here-
pratheekv39

[X] I have provided my GitHub ID.

Identify Yourself
Mention in which program you are contributing (e.g., WoB, GSSOC, SSOC, Hactoberfest).
Enter your participant role here-
Contributor at GSSOC and Hacktoberfest

[X] I have mentioned my participant role.

Closes
Enter the issue number that will be closed through this PR.
Closes: #issue-number
#516

[X] I have provided the issue number.

Describe the Add-ons or Changes You've Made
Give a clear description of what you have added or modified.
Describe your changes here-
Modified resources.js file to incorporate changes

[X] I have described my changes.

Type of Change
Select the type of change:

[ ] Bug fix (non-breaking change which fixes an issue)
[X] New feature (non-breaking change which adds functionality)
[ ] Code style update (formatting, local variables)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?
Describe how your changes have been tested.
Describe your testing process here.
VS Code local dev environment

[X] I have described my testing process.

Checklist
Please confirm the following:

[X] My code follows the guidelines of this project.
[X] I have performed a self-review of my own code.
[X] I have commented my code, particularly wherever it was hard to understand.
[X] I have made corresponding changes to the documentation.
[X] My changes generate no new warnings.
[X] I have added things that prove my fix is effective or that my feature works.

@rishicds Kindly merge the PR, I have added the roadmap , kindly see the referenced video....Thanks :)
https://github.com/user-attachments/assets/9fa5fd0d-2771-455b-af5e-2cd7c7c9adbb

Pls fix conflicts

Done @pratheekv39


Meetings ?

Hi,
I cant find the meetings link. Do we have any meetings arranged ?
Thanks,
Gareth

Hi Gareth, my intention is to schedule meetings from next week (weekly meetings in principle).
Initially i propose Thursday nex week to begin.
Also i would kindly ask in this forum for locations of people more interested in this WG in order to propose a time that can fit for all of us. So open to hear from everybody on this item.
Best Regards
Pedro

No worries. I think I can accommodate almost any time - whatever works for the rest of the team.
Gareth

Hi Pedro,
Looking forward for the meetings,
GMT+3 here, I suggest using a doodle tool to vote on possible slots.

doodle: https://doodle.com/meeting/participate/id/bk87KRYd

closing this issue as meetings are already scheduled


error  ModuleNotFoundError: No module named 'binance_api'

i did install python package "binance-api" and it doesn't work

Fixed, downloading binance_api folder: https://github.com/EasyAI/binance_api and pasting it in directory: C: \ Users \  \ AppData \ Local \ Programs \ Python \ Python38 \ Lib \ site-packages


feat: add more node provider metadata fields

Description
Fixes #17
Changes made

Extended the ProposalEditableMetadata
Persist the new metadata in the backend
Update review step to show additional meta data fields.

Thanks!


Add "Paths" section (המסלולים שלנו)

Description
make sure to follow the figma design.
Relevant files:

components/Home/Paths.tsx
app/page.tsx

Screenshots
No response
Additional information
If/when creating sub components is necessary, create a new folder called "Paths" and insert the Paths.tsx in there, together with all related sub components.

This looks great, I'd love to be assigned

Good luck Itai :)

#16 solves this issue


Avoid listening on all interfaces by default

This was identified by security review as a vulnerability.
With this change we only listen on the IP/interface that was either specified in configuration (DMLC_PS_ROOT_URI or DMLC_INTERFACE) or if no interface is specified in config, the non-loopback interface automatically picked by ps-lite.

@piiswrong - are you good with this?

I remember ps-lite also supported infiniband. Is this disabling it?


Location of code in [Class Summary]

@Misaki0306 commented on Wed May 06 2020
I could not find where to write the code of [Class Summary].
It is not clear whether the code is written in Program.cs or Transaction.cs.

ドキュメントの詳細
⚠ このセクションを編集しないでください。 docs.microsoft.com で必須です ➟ GitHub の問題のリンク。

ID: 24c8353f-69e7-9068-9b18-df0198a1b122
Version Independent ID: 823031fe-c8a4-69cd-2b1a-9faacd282aab
Content: クラスおよびオブジェクト - C# チュートリアルの概要
Content Source: docs/csharp/tutorials/intro-to-csharp/introduction-to-classes.md
Product: dotnet-csharp
Technology: csharp-get-started
GitHub Login: @BillWagner
Microsoft Alias: wiwagn

@srvbpigh commented on Wed May 06 2020
Hello, @Misaki0306
Thank you for your feedback.
We are actively reviewing your comments and will get back to you soon.
Kind regards,
Microsoft DOCS International Team

thanks for bringing this to our attention.
I looked for which sample you were referring to, and I'm not sure which one you mean. That points to a fix: For each new code sample, we should add clear instructions for which class needs to be modified for the new code.


Debian minbase

Hi! Can i install debian variant Minbase preset?

That's not supported at the moment.


fix: enable/disable the plugin via querystring

Enable and Disabling the plugin via query string not working! (http://example.com/padName?toc=false)
I refactored the getParam function and use URLSearchParams API to obtain search query string params, in order to make it easier to read and maintain.

n1 tnx


Fix flaky cache tests

Join with executor for async operations before teardown of test. Fixes
possible access to freed memory.

@oerling A new test failed
AsyncDataCacheTest.replace
unknown file: Failure
C++ exception with description "No threads available" thrown in the test body.
[  FAILED  ] AsyncDataCacheTest.replace (1 ms)

They are probably due to no join on the executor.  Try with the fix. I had that one flakily.

They are probably due to no join on the executor. Try with the fix. I had that one flakily.

@oerling Orri, the failure is from linux-build CircleCI run on this PR
3: [  FAILED  ] 1 test, listed below:
3: [  FAILED  ] AsyncDataCacheTest.replace
3:
3:  1 FAILED TEST
110/117 Test   #3: velox_cache_test .....................................***Failed   17.10 sec

This was seen in fbcode as well and appears to go away with joining the threads at end of test.

@oerling

This was seen in fbcode as well and appears to go away with joining the threads at end of test.

I'm not sure I understand. The CI build is failing on this very PR. Does it need more changes?


Longer time-series input may performs bad?

Congratudate for your AAAI 2022 work. It's a nice work, but i met some problem when i use your model as a sota in my experiment. It performed very bad when i try to input 16, 24 or 144 steps in electricity and bikeNYC dataset. Even though i try to adjust some parameters like node_embedd dim, Chenb_k (1,2,3), hidden channel (16 may be the best), loss function (L2 and L1), it stilll does not make. I would like to inform you for some good solutions to solve it. I will appreciate it very much if there are some replies.

Congratudate for your AAAI 2022 work. It's a nice work, but i met some problem when i use your model as a sota in my experiment. It performed very bad when i try to input 16, 24 or 144 steps in electricity and bikeNYC dataset. Even though i try to adjust some parameters like node_embedd dim, Chenb_k (1,2,3), hidden channel (16 may be the best), loss function (L2 and L1), it stilll does not make. I would like to inform you for some good solutions to solve it. I will appreciate it very much if there are some replies.

I've encountered the same problem when trying to apply NCDE on longer time series (in fact 1500 steps). Could you please share your solutions with me? Is it about NRDE? Thanks.

Hello @lixus7. We are still exploring longer time-series. It would be beneficial if you could share good information about the dataset you experimented with.
Note: we have a paper related to NRDE under journal review.


🌱 Improve handling of IPClaims with deletion timestamp

What this PR does / why we need it:

If the IPClaim is old (=not matching current Metal3Data), attempt to clean it up
If the IPClaim is in use, accept it even if it has a deletion timestamp. This could happen e.g. if the user tried to delete it but the Metal3Data/Metal3Machine still exists.

/test-ubuntu-integration-main

/test-ubuntu-e2e-main

/retest

/test-all

/test all

/test-centos-e2e-parallel-main
/test-ubuntu-integration-main

/test unit

/test-ubuntu-integration-main
/test-centos-e2e-parallel-main

/test-centos-e2e-parallel-main

/test-centos-e2e-parallel-main
Different flakes...

It has now failed in 3 different places :frowning_face:

When scaling down KCP in e2e pivoting based tests
The flake with empty providerID in e2e remediation
When scaling up the KCP in e2e pivoting based tests

/test-centos-e2e-parallel-main


[stable/openvpn] added ability to configure LoadBalancerSourceRanges

What this PR does / why we need it: Adds the ability to whitelist source IP addresses for connecting to the OpenVPN Loadbalancer. Useful if you want to limit CIDRs that can VPN into the cluster for added security.
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
Special notes for your reviewer: Please let me know if you need anything clarified. Thanks!
ref docs: https://kubernetes.io/docs/tasks/access-application-cluster/configure-cloud-provider-firewall/

/assign @mattfarina

/ok-to-test

/retest

/lgtm


Detect and work around null records

I'm asking the QuestDb devs about this but in the meantime this should work around the issue.

... hang on I'll do one additional check.


mkdir will fail on Windows if the filename contains a colon

When executing mkdir "foo: bar Filesystem.createDirectory will fail:
*** Exception: CreateDirectory "F:\\": does not exist (The system cannot find the path specified.)

Now, this is an issue with Filesystem but since it is deprecated I don't think this will be fixed. I don't know what's the best course of action here.
Related https://github.com/Gabriel439/Haskell-Turtle-Library/issues/254

So like I mentioned in that issue I may take over maintainership of system-filepath soon so I can fix issues like these (or minimally just accept pull requests).  However, I need to sort out some other tasks first before taking this on.  Give me about a month before I can address this

Sure. This is a very minor issue, no need to rush. If I can help somehow let me know.

: is not a valid character to use in Windows folder names:

So then, what would the solution be?

AFAIK a lot of these restrictions are only in the UI, and can be bypassed by using the raw APIs.


Improve PlayerJoinData to support extra data.

History:
https://github.com/SpongePowered/SpongeCommon/issues/1282
https://github.com/SpongePowered/SpongeCommon/pull/1710
PlayerJoinData is inconsistent and impossible to fix correctly without API breaks due to badly named & semantics in API7.
It is planned, that the following data is going to be added to PlayerJoinData.
hasplayedbefore, firstjoined, lastjoined, lastplayed
This will clear up the ambiguity in the current data, as well as giving plugin developers the information they need.

PR merged


Added support for Index Metrics V2

Packages impacted by this PR
@azure/cosmos
Issues associated with this PR
#32064
Describe the problem that is addressed by this PR
The backend currently returns Index Metrics in string format when populateIndexMetrics is sent true in query options. The backend has now introduced a V2 version of index metrics that returns the response in JSON format. This PR implements the changes for SDK to handle the new JSON format seamlessly. This change ensures compatibility with the updated backend while maintaining functionality with the previous version.
What are the possible designs available to address the problem? If there are more than one possible design, why was the one in this PR chosen?
Are there test cases added in this PR? (If not, why?)
Provide a list of related PRs (if any)
Command used to generate this PR:**(Applicable only to SDK release request PRs)
Checklists

[ ] Added impacted package name to the issue description
[ ] Does this PR needs any fixes in the SDK Generator?** (If so, create an Issue in the Autorest/typescript repository and link it here)
[ ] Added a changelog (if necessary)

API change check
API changes are not detected in this pull request.


null as input for primitive types in exports should be allowed

When exporting a method taking as parameter a primitive type, we currently disallow explicitly null as a valid input.
Tests: https://github.com/scala-js/scala-js/blob/8cab6bbbe3c33afca2fd52874428096b07810a76/test-suite/src/test/scala/org/scalajs/testsuite/jsinterop/ExportsTest.scala#L827-L836
Implementation: https://github.com/scala-js/scala-js/blob/8cab6bbbe3c33afca2fd52874428096b07810a76/compiler/src/main/scala/org/scalajs/core/compiler/GenJSExports.scala#L517-L520
This is in contradiction with the general semantics that JavaScript is an erased generic context. Exports should asInstanceOf their parameters, which means unboxing for primitive types, which means null is a valid value and becomes the zero of the type.
The inconsistency appeared to me while working on #1715. Also, it is highlighted by the fact that the code in GenJSExports differs from fromAny only in that it specifically disallows null from primitive types.

I think this actually raises an interesting concern about this specification:
If I write:
trait MyFacade {
  def doSomething(x: Int): Unit = js.unit
}

It actually means that doSomething might receive null and is to interpret it as 0 (and a sufficiently convoluted example + optimizer might also result in that). While this is perfectly valid according to specification, it is definitely not what users (and your average JS library) expect. Maybe we have to reconsider some parts of the spec...

I assume from the name MyFacade that you intended it to extends js.Object?
In that direction, there is no problem, because the argument given to x is a primitive Int and is therefore not null. It just can't be. It is "boxed" by our backend because it is given to a facade, but we know it can't be null anyway. It doesn't matter whether the optimizer comes into play.
What can happen, though, is if you give, say, a js.Array[Int] to JavaScript, and you have filled this array with a convoluted series of generic operations that put zero[Int] everywhere, then JavaScript will receive an array with nulls inside, yes. But there's nothing we can do about that, I fear. This could also happen on Scala/JVM when giving a zero[Int] in a collection to Java code.

I was more thinking of something like this:
def foo[T](f: js.Function1[T, Unit]): Unit = f(null.asInstanceOf[T])

There will be some bridge generation such thatc this actually works. However, the optimizer might be able to inline all this and remove all the unboxing --> Bad

This will not work, indeed, even without optimizer.
Note that the optimizer will never remove an unboxing unless it can prove that its input is already an Int, and hence not null. The optimizer will never introduce such bad behavior. But, the bad behavior could be there before.


debouncing calls source functions twice for every key press

Looks like the debouncing is a little off. I'm noticed multiple calls t a source function on every keypress when debouncing is set to 250 ms.

Here is an example of the issue with a debounce of 1000ms. We can see that two request are triggered. One at the first keystroke, then the second when we stopped typing. We can find an explanation on this issue.


Rename accepted headers for metrics events

After reviewing current events we agreed to rename the headers so they are closer to their corresponding event attributes.
\cc @VictorVelarde

It seems npm is having problems today, so the tests won't pass. Let's wait until it gets stable again :pray:

I'm ok with the new names. Feel free to move on and please ping me when they're activated on prod, so I can adapt & test the client at toolkit

It looks like it is stable now! 🚀


Revert "Merge pull request #54980 from danbst/etc-relative"

This reverts commit 0b91fa43e40c121ff4682256aa46a425c984da6c, reversing
changes made to 183919a0c072061b98ebe9fca2e899ade871ff1c.
Motivation for this change
https://github.com/NixOS/nixpkgs/issues/56278
https://github.com/NixOS/nixpkgs/pull/54980#issuecomment-466224732
https://github.com/NixOS/nixpkgs/pull/54980#issuecomment-466504857
https://discourse.nixos.org/t/nixos-rebuild-switch-upgrade-error-directory-contains-user-files-symlinking-may-fail/2266
The isStatic check now doesn't work, which leads to warnings

@7c6f434c can you backport this to 19.03?

I have no idea what is going on. I would expect you to just directly push a revert directly, given that you didn't even wait for ofBorg checks — and then you could as well push the rever to both branches.
Whatever, reverted in release-19.03 just by git revert — I hope that's the intended action.

@7c6f434c I can't push to master branch (no rights?), so I have to do PRs...
Thanks

:thinking: maybe that's because there is policy about signed commits?..

Nope, that's definitely not policy on signed commits.
I just thought you were not a readonly member (I thought this is still a rare state).
@domenkozar Maybe this could be changed?

@7c6f434c figured this out. I had origin as https, after I changed it to ssh pushes work now. Probably https push won't work because 2FA is forced.

Actually, I push via HTTPS, but using a push token instead of password.


Show top spenders on homepage based on burn rate, not latest expenses

Instead of showing the latest collectives that have spent money which lead to random and poor results:

we should show the collectives that spend the most money per month (aka burn rate)

Also, it's weird to show "Top spenders" and not showing any information about how much they are spending.
So I'd suggest to show the burn rate instead of the number of sponsors in the card (and show the total number of backers which includes individuals and organizations).

I have a PR ready for this but I first need to wait for this one to be merged before I can submit it https://github.com/opencollective/opencollective-api/pull/1371 /cc @HipsterBrown

works for me. Let's get this merged today because homepage announcement is going in the newsletter.

Mmmm... technically burn rate is supposed to be the difference between the money in and out
See: https://en.wikipedia.org/wiki/Burn_rate
The term became popular with startups that would anyway not have any revenue. So burn rate in their case meant the money spent per month. I guess it's not really true for collectives since they are getting some money.
How should we label this then?
Is it acceptable to use this term? Is there another one that would be a better fit? Any ideas?

Alright, we will go with "monthly spending"

I'm still seeing active spenders and a lot of wwcode ones.


Deny Lakeformation Admin Operations

Use Case - Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is.
To prevent users from adding themselves as an administrator with an extract, transform, and load (ETL) script, make sure that all non-administrator users and roles are denied access to these API operations.
AWS Lake Formation is a fully managed service that makes it easier for you to build, secure, and manage data lakes.
AWS Lake Formation Administrators can view all metadata in the AWS Glue Data Catalog. They can also grant and revoke permissions on data resources to principals, including themselves. AWS Lake Formation requires that each principal (user or role) be authorized to perform actions on Lake Formation–managed resources. A principal is granted the necessary authorizations by the data lake administrator or another principal with the permissions to grant Lake Formation permissions.
When you grant a Lake Formation permission to a principal, you can optionally grant the ability to pass that permission to another principal. A principal with IAM administrative permissions—for example, with the AdministratorAccess AWS managed policy—has permissions to grant Lake Formation permissions and create data lake administrators. To deny a user or role access to Lake Formation administrator operations in your account, attach below SCP policy.
{
"Sid": "DenyLakeFormationAdministratorOperations",
"Effect": "Deny",
"Action": [
"lakeformation:GetDataLakeSettings",
"lakeformation:PutDataLakeSettings"
],
"Resource": [
"*"
]
}
Expected Outcome - Describe the solution you'd like
A clear and concise description of what you want to happen.
User is not authorized to perform: lakeformation:PutDataLakeSettings on resource with an explicit deny.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Affected AWS resource
Mention the AWS resources which is affected
AWS Lake Formation
Impact
High: The issue makes a service level improvement which affects all users of AWS
Medium: Single feature which affects a single functionality which is optionally enabled in the AWS service
Low: Niche use case which is particularly affecting the AWS resources if it is configured in a certain way
Supported material
Can be either logs, screenshots or documentation links which provide evidence of need of this issue
I can contribute: Yes/No
If you are able to contribute towards resolving this request.
Additional context
Add any other context or screenshots about the feature request here.
Pull Request number
If a pull request has already been created.

Below is the updated SCP:
{
"Version": "2012-10-17",
"Statement": [
{
"Sid": "DenyLakeFormationAdministratorOperations",
"Effect": "Deny",
"Action": [
"lakeformation:PutDataLakeSettings"
],
"Resource": [
"*"
]
}
]
}


No data displaying

Teslamate is running well on my Synology. Also, MMM-Teslamate is running on my pi. However, non of the data from Teslamate is getting to the MMM-Teslamate app. Does anyone know why this might be?

Please let me know if there are any log files or config files I can share to help in resolution. Thanks.

Not sure what happened. It just started working. Great module!


Improve E2E tests

Description
This PR starts the migration of E2E tests to Ginkgo/gomega framework.

Hi @giorio94 and @aleoli, thank you for your reviews!

Two general comments:
1. The microservices app test appears to me nondeterministic, since no affinities are specified and the actual placement is left to the scheduler. Hence, all pods could be scheduled locally (and you would be testing k8s) and, even in case of errors, they would be considered glitches as non reproducible.

100% agree. Moreover, the check is limited and performed only on the homepage. Since this limitation is not introduced by this PR, for sake of brevity I would let this PR as it is and prepare a follow-up PR with a new YAML and checks that touch also the other microservices.
The idea of this first PR is to have a slightly more consolidated code-base in order to simplify new test additions to the E2E suite.

2. Contexts should be propagated, and not created on-demand, otherwise they become useless.

Ok, no problem with that. Personal question: is it not really clear to me what can contexts bring in for this test scenario.

More comments inline (sorry for their number). Feel free to discard them if they don't make sense to you

Hi @giorio94 and @aleoli, thank you for your reviews!

Two general comments:

1. The microservices app test appears to me nondeterministic, since no affinities are specified and the actual placement is left to the scheduler. Hence, all pods could be scheduled locally (and you would be testing k8s) and, even in case of errors, they would be considered glitches as non reproducible.

100% agree. Moreover, the check is limited and performed only on the homepage. Since this limitation is not introduced by this PR, for sake of brevity I would let this PR as it is and prepare a follow-up PR with a new YAML and checks that touch also the other microservices.
The idea of this first PR is to have a slightly more consolidated code-base in order to simplify new test additions to the E2E suite.

Definitely fine from my point of view. Indeed, this is already a quite big PR.

2. Contexts should be propagated, and not created on-demand, otherwise they become useless.

Ok, no problem with that. Personal question: is it not really clear to me what can contexts bring in for this test scenario.

I think they should allow for a more ordinated termination in case of aborts, to prevent requests hanging (although I am not sure how/whether they are supported by Ginkgo), but I agree they are not that important in case of tests. Still, since some utility functions may be of larger use, I feel it would be better to support them for the sake of generality. Just personal opinion though.

More comments inline (sorry for their number). Feel free to discard them if they don't make sense to you

/rebase

-rebase

/rebase

/rebase

/merge

/merge


Remove txt files

The extension of the files has changed to rst
Clean up PR
cc @sbesson

These files had been renamed in https://github.com/openmicroscopy/ome-documentation/pull/1733 and then automatically re-added in https://github.com/openmicroscopy/ome-documentation/pull/1737. I assume https://github.com/ome/omeroweb-install/pull/23/ contained the corresponding upstream changes to prevent the files to be re-added in the future @jburel

yes see https://github.com/ome/omeroweb-install/pull/23

The merge auto-generated documentation branch with this PR included confirms that the removed text files are no longer automatically re-added.


Header issue while uploading local file to JCDS

While trying to upload a local file to a JCDS, I'm running into this error:
24/08/06 10:21:50 a.m.-ERROR: Failed to copy ZoomMacOutlookPlugin-5.12.0.pkg to JCDS : dataRequestFailed(statusCode: 403, message: Optional("AccessDenied: AWS authentication requires a valid Date or x-amz-date header - max allowed size = "))
Account works to login to the console and it was used with Jamf admin.

If you do the following in a terminal window, you will get some extra debug messages that might help determine what the issue is:
/usr/bin/log stream --predicate "subsystem == 'com.jamf.jamfsync'" --debug
Version 1.3.3 is coming out soon. Let me know if that solves the issue or not.

@rvera53 Check if you've got proxy in the mix, maybe that could be stripping out the header. I was running into similar issue and was able to replicate from other "Managed" devices. But was unable to replicate on an unmanaged device.


Visual design ideas

https://www.youtube.com/watch?v=hoi1BNT-oXs

Disable 3D buildings in the basemap style

The streets-v2 layer is "Building 3D". Should we

Write an async function to fetch the basemap style and modify its JSON before returning it?
Have some kind of callback to disable the layer after the main style is done loading?
Directly keep a copy of the streets-v2 style locally and modify it? (if there are more modifications, might make sense)


make use of explicit index table

This PR rips out the usage of the key primitive, and instead makes use of explicit index tables.
WIP

[ ] create a snapshot-vnext branch of main (once ready)
[ ] merge the requires PR (https://github.com/digital-asset/daml-finance/tree/reintroduce-requires) on top snapshot-vnext
[ ]  rebase this PR with snapshot-vnext, make use of requires, and then point to snapshot-vnext


Update NK2Tray_DisplayFW_V2.ino

I made a little change to the code to make it work (copied someones working code) and I also added some things to add some logos

Closed this because I made a new edit that doesn't unnecessarily change all the formatting


Allow http (not https) link for image and button urls.

Now, for internet, https is by default. However, for intranet, http is still there.
I want to share a http link that is one of intranet contents. But Company Communicator doesn't allow http link.
Is thre any options to send http link on Company Communicator?

Hi @runceel ,
Thanks for raising the query. Currently Company Communicator doesn't support HTTP.

This is a place where check has been added and it is recommendation from Microsoft. If the modification is a must from your end. Here is the link which will direct you to the page where the restriction has been implemented.

Or While sending a message to the intended users. Please caution them during the message drafting by saying that "Please cross check the URL and remove s from https:// to make it "http://" before browsing the link."

@gsv022
Thank you for answering.
I will customize Company Communicator to support http://.
Do you have any plan to support http:// in original Company Communicator?

Not yet. It will be planned for future releases post the discussion with internal team. Please let us know if you have any other queries.

@gsv022 Thank you! :)


header bar align() processed too early

The calculations done within  header bar align() are executed too early.
Even within the callback from requestAnimationFrame, the offsetWidth for the left buttons div might not yet be up to date.
So If you have a left-buttons div that varies in width between the old and the new view the title will be positioned improperly

Greetings @pberkaloff!
Sorry but this issue may no longer be applicable to recent versions of Ionic.
If you are still experiencing this issue, please feel free to reopen and provide the following information so that I may assist you.

Can you provide a detailed description of this issue?
How would a robot go about replicating this issue?
What platform does this happen on? (iOS, Android, all, etc.)
What platform versions does this happen on? (Android 4.2, iOS 7, all, etc.)
Is this an issue with the Ionic Framework, or Ionic CLI?

Additionally, providing a codepen which replicates the issue is extremely helpful.
If you wish to get help using the framework itself, the recommended place is the forum.
Thank you for allowing me to assist you.


Sudoku Grid Cells overlapping, overflowing, and blocked “Go Back” button

When using a browser on PC with a limited length viewport, the sudoku grid cells overlapping, overflowing, and blocked “Go Back” button
Steps to reproduce

Prepare a limited length browser window on PC.
Click “Easy”, “Normal”, or “Hard” buttons.
Click “3”, “5”, or “10” buttons.
Click “Create New Game” button.

Click the Easy, Normal, or Hard button.
Click the "3", "5" or "10" button.
Click the "Create New Game" button.
screenshot


Skip asking for jsdoc in checkObjectLiteral for enums in TS

Pulling this out of #52641 as I think it's a good change to not waste time asking for JSDoc when we're not in JS code. The other half of that PR is a bigger change (even without baseline changes).

@typescript-bot test this
@typescript-bot test top100
@typescript-bot user test this
@typescript-bot user test tsserver
@typescript-bot test tsserver top100
@typescript-bot run dt
@typescript-bot perf test this


Parsing exponent notation in json-parser

I just saw that the to_js_number() does not work when the exponent does not have a sign (like 1.0e8).
A quick look into the JSON schema tells me that it is allowed.

Was this missed or is support for these not required?
Here is a simple fix anyway. I am happy to provide a pull request if this is necessary.
Just confused right now.
https://godbolt.org/z/7bo3d41K6

This is my mistake. First of all a fix to the regex is needed. The way you fixed it in to_js_number is way too complicated.
I would do it like this:
exp_minus = *it == '-';
if (*it == '-' || *it == '+') ++it;

I agree.
Please go ahead!

Fixed


:grey_question: Question about openai free plans and ai-commits :money_with_wings:

:information_source:  Context
Hi, we are trying to use ai-commits with an open-ai free plan, unfortunately here is what we got :

:grey_question: The question
:point_right: Is an openai  paid plan a prerequiste ?

IIRC Open AI doesn't have a free plan.
Perhaps your trial tokens are expired?
https://github.com/Nutlope/aicommits/issues/190#issuecomment-1497429247

:thinking:

IIRC Open AI doesn't have a free plan.

I do use tokens ... but never paid, I got them from here : https://platform.openai.com/account/api-keys

You mean these may not work with ai-commits.. while still allowing me to call api (see my Kaggle 🪄 OpenAI - Gender guess on firstname) ?

If it can help, find the code that actually works below :
!pip install openai

import openai
from kaggle_secrets import UserSecretsClient

user_secrets = UserSecretsClient()
openai.api_key = user_secrets.get_secret("OPENAI_API_KEY")

def guess_genre(fname):
    genre = 'MASCULIN'
    #"""
    response = openai.Completion.create(
      model = "text-davinci-003",
      prompt = "Extract the genre of the given firstname. If a MAN, reply <MASCULIN>, if a woman, reply <FEMININ>, else reply <NOT_GUESSED>. firstname :" +
            fname,
      temperature=0,
      max_tokens=64,
      top_p=1.0,
      frequency_penalty=0.0,
      presence_penalty=0.0
    )
    genre = response["choices"][0]["text"].strip()
    #"""
    # print(response)
    # print("Guessed Genre : <" + genre + ">", sep = '')

    return genre

Maybe it has something to do with the type of free tokens you use.
You can try experimenting to see if gpt-3.5-turbo works here:
https://platform.openai.com/playground?mode=chat
Or maybe you're using gpt-4 but you don't have access yet?

Or maybe you're using gpt-4 but you don't have access yet?

Exact :heavy_heart_exclamation:

You can try experimenting to see if gpt-3.5-turbo works here:
https://platform.openai.com/playground?mode=chat

Yup, I'm indeed using gpt-3.5-turbo : I guess it can be customized in ai-commit ?

:pray: Thanks for the feedback.
:point_right: I'll let you know if we managed to make it work on gpt-3-5 :crossed_fingers:


React to HtmlEncodedString rename to HtmlString.

Updated test output to reflect new HtmlString naming.

aspnet/HtmlAbstractions#25
/cc @rynowak @dougbu @pranavkm

:shipit:

:shipit:


Prefer already-loaded modules?

If we have already loaded a module that meets a semver requirement, should we prefer that to one that has not yet been loaded?  This would come up if we load a bundle after the initial load that has a newer version of a package that was already loaded.

:chicken: :egg:  or :egg: :chicken:

Hahaha, truth.

A subtlety here is that since the loading mechanism caches versions at the file level instead of the package level, you could have package A loading one file from package B version 1.0.0, and then someone loads package B version 1.1.0, and package A would load any new files from package B version 1.1.0. This could really be bad, since there are no guarantees in semver about how file implementations in a single package interact with each other across versions.

  We can fix this in phosphor and jupyterlab by creating sub-packages (copying the model used by angular), but it won't help us with third party things like CodeMirror and MathJax.

I think giving very verbose warnings anytime we have files shadowing other files would go a long way towards understanding how the system is working. We can detect that when they are added to the extension mechanism. We could also cache the semver -> version number mapping per package instead of per file like we do now.

I think giving very verbose warnings anytime we have files shadowing other files would go a long way towards understanding how the system is working.

Hmm, that's probably overly verbose. Perhaps a warning anytime the same semver will return a different version of a package.

I don't fully follow what you mean, we probably need to discuss this live.


Support --require of ESM; closes #4281

Allow files/modules specified in --require to be ESM.
CommonJS loading is still supported and the default.
closes https://github.com/mochajs/mocha/issues/4281
Description of the Change
Support --requireing ESM files in config. Current behavior is to fail on ERR_REQUIRE_ESM.
Re-use existing "require or import" logic for loading test files to load required files.
Alternate Designs
Pipe requiring the ESM file through something like esm or ts-node. Although that seems much more complicated and overkill.
Why should this be in core?
Requiring JS files before the test suite is a core part of Mocha's functionality. This extends that support to ESM-first codebases, which is natively supported via "type":"module", or --exerimental-module.
Benefits
Allows developers more freedom in choosing to write ESM or CommonJS.
Possible Drawbacks
Native ESM in NodeJS is still labeled as experimental (although an --experimental flag is no longer needed). It is possible the usage will change, but that is justification for re-using existing "require or import" functionality.
Applicable issues
This should be an "enhancement" (minor release). --requireing CommonJS is still the default and fully supported, this will just allow additional support for ESM.

No tests included in this first pass.
I don't see any existing tests for this... so not exactly sure where to put them.

Coverage increased (+0.03%) to 93.467% when pulling 5ef04be6976065b723010dfe5f2c4ac17ea2217d on JacobLey:requireESM into 6d60eb01dae2b609bcf7976dc5975670201b7f46 on mochajs:master.

apologies, the organization of test is often not obvious.  probably need to add more info to test/README.md

Because "require" is not typically a word used with ESM ("import" is more generally used), I would add specific messaging in the --require documentation that says that it also accepts ES modules.

LGTM.  Thanks much @JacobLey !


I can't hide PaymentSheet

I want to hide the PaymentSheet or at least clear the information in it. I checked documentation few times but couldn't find any method for that. Do I miss something or there is no such feature?

duplicate of https://github.com/stripe/stripe-react-native/issues/724


Enter password before butler is stopped

Enter password before butler is stopped

The following use case does not work, because the password context is cleared:

Start Butler
Stop Butler by entering the password
Start Butler again
Stop Butler by entering the password - failing because the password context is empty.


Skip missing not working on docker?

I've tried adding SKIP_MISSING as a variable without any luck. Is there something I'm missing? Thanks

SYNCARR_SKIP_MISSING not SKIP_MISSING, got it

Can this please be posted as an option in Docker & Docker compose instruction files. I too missed this information.


Requirements not in setup.py

Currently, the pyserial requirement live in a separate file, when it should be listed in the install_requires field of the setup function. If you try to install this package in a new environment, an error is raised because pyserial cannot be found. Adding it to the install_requires list should solve the problem.

Thank you for pointing this out. This should be fixed now, see
https://github.com/nelsond/thorlabs-mtd415t/blob/715eb2b3203a0e67d0a6b4a9d3f883031ceccaf9/setup.py#L44


remove BOM from eupl-1.1.txt which caused it to not render

BOM = byte order mark

:+1: I'm not sure why tests are failing. Merging in master might resolve it.

Actually, I can reproduce failure locally, which demonstrates I didn't run rake test before pushing.
Here's what I think's happening:

_site/licenses/eupl-1.1/index.html contains a href to http://choosealicense.com/licenses/eupl-1.1/
html-proofer checks all links in _site
http://choosealicense.com/licenses/eupl-1.1/ (or any newly introduced link, not yet live) of course obtains 404

Braindead fix would be to have test tell html-proofer to ignore choosealicense.com links, with obvious downside of not catching unintentionally broken links within the site.
Sophisticated fix I guess would be to compare links in origin/gh-pages and local branch, and tell html-proofer to ignore choosealicense.ccom links introduced locally.
Or even better, looking at https://github.com/gjtorikian/html-proofer href_swap on choosealicense.com links to have those links tested locally rather than against live site. I'll see if I can make that work.
But if my speculation about how things work is obviously off-base, please advise. :smile:

That seems to do it, and if I introduce a http://choosealicense.com/... link that doesn't exist locally, the test fails, as it should.

Tests pass for me locally without the change in 96fc8b1. Theoretically, there shouldn't be any choosealicense.com links locally, since everything uses site.url or site.github.url. Do you know where the link is in the page?

Just kidding. I was using your fix. I think I see what's going on.

<link rel="canonical" href="http://choosealicense.com/licenses/eupl-1.1/" itemprop="url" />

Yep. It's via the SEO tag plugin. Looks like your fix is the simplest. :+1:


Stopped working in Gmail after 2015-05-21: v2.11.8 update

On Firefox 38.0.1 Win8.1 64-bit
I'm note really sure what details are needed so I'll describe the steps I took.
Steps:

Go to gmail.
go to a conversation
click reply
type some reply in markdown
click shortcut (defined in your addon settings) (I've been using the same shortcut before update)

What happens:
All text gets selected
What's expected to happen:
Convert the markdown with rendered HTML.
Let me know if more details are needed.

I noticed that the preview in the Markdown Here settings page also no longer renders the Markdown.

Duplicate of #278, which will be the main issue for this. Fix coming soon...

Is there a workaround for this until fixed?
Regards,
Dheeraj
On 27-May-2015 6:47 am, "Adam Pritchard"<EMAIL_ADDRESS>wrote:

Duplicate of #278 https://github.com/adam-p/markdown-here/issues/278,
which will be the main issue for this. Fix coming soon...
—
Reply to this email directly or view it on GitHub
https://github.com/adam-p/markdown-here/issues/279#issuecomment-105709220
.

Nevermind, got it.
Read all the other issues. Will use the newer albeit unreviewed version
referenced in #278.
On 27-May-2015 9:28 am, "Dheeraj Bhaskar"<EMAIL_ADDRESS>wrote:

Is there a workaround for this until fixed?
Regards,
Dheeraj
On 27-May-2015 6:47 am, "Adam Pritchard"<EMAIL_ADDRESS>wrote:

Duplicate of #278 https://github.com/adam-p/markdown-here/issues/278,
which will be the main issue for this. Fix coming soon...
—
Reply to this email directly or view it on GitHub
https://github.com/adam-p/markdown-here/issues/279#issuecomment-105709220
.

The new version has been approved by Mozilla.

Thanks I've already installed it.
Thanks for the amazing work :)
On Wed, May 27, 2015 at 5:04 PM, Adam Pritchard<EMAIL_ADDRESS>wrote:

The new version has been approved by Mozilla.
—
Reply to this email directly or view it on GitHub
https://github.com/adam-p/markdown-here/issues/279#issuecomment-105878925
.


Convolution is slow

There are probably multiple places where we can improve it.
Biggest bottleneck is the border handling (70% of the time is spent there):
https://github.com/image-js/image-js-typescript/blob/d3f0d998c623259c71d795e0abb9d6925fca89ba/src/filters/convolution.ts#L192-L268
Tested with:
import { Image } from 'image-js';

const img = new Image(2000, 1000);

img.gaussianBlur({ sigma: 10 });

console.time('blur')
img.gaussianBlur({ sigma: 10 });
console.timeEnd('blur')

console.log(img);

First improvement: https://github.com/image-js/image-js-typescript/pull/336

Still seem slow.
At some point, could we consider using WebGPU when supported to handle heavy operations?


Strange shading with WorldEnvironment with sky

Godot version:
3.2 stable
OS/device including version:
Manjaro 18.1.5
Issue description:
When using WorldEnvironment with a sky panorama I am getting strange shading.
In 3.1 it looks normal.
With metallic set to 1 in the material, it looks like it looks normal.

Steps to reproduce:

Add a WorldEnvironement
Set mode to sky
Select a sky panorama
Add a MeshInstance to the scene and add a new Sphere to it
Add a SpatialMaterial to the MeshInstance

Minimal reproduction project:
godot-test.zip

What's the Radiance Size configured in the Sky resource of the WorldEnvironment? Try increasing its value.

@Calinou To me it looks like the problem is the irradiance map. The sun in this PanoramaSky is extremely bright which requires a lot of samples of the irradiance map to compute properly.
Note: the irradiance max size rounds down to the nearest power of two. So it needs to be 256 or higher to see a difference. I would suggest setting max irradiance to 512, and radiance size to 512 as well.

Still valid in 3.2.4 beta4
Mesh is broken in 4.0, so can't test

Would https://github.com/godotengine/godot/pull/58164 be helpful in 3.x to combat fireflies in the (ir)radiance map?


Add individual pages for news articles

This needs front-end and back-end work.
Moving this issue to the news-server.


Typescript definition error

I've been trying to use the definitions in dist but i'm getting the following error while building:

custom_typings\aurelia-configuration.d.ts(211,1): error TS1128: Declaration or statement expected.
custom_typings\aurelia-configuration.d.ts(211,16): error TS1005: ';' expected.

Typescript version<EMAIL_ADDRESS>

Typings should be fixed in latest version 1.0.11


connectcast.tv - 400 Client Error: Bad Request

When trying to use any connectcast.tv stream, Error 400 is displayed. E.g:
livestreamer "http://connectcast.tv/kireita"
[cli][info] Found matching plugin connectcast for URL http://connectcast.tv/kireita
error: Unable to open URL<EMAIL_ADDRESS>(400 Client Error: Bad Request)
Any suggestions what might be missing?

I'm getting the same results from play.afreeca.com

Don't know. Seems they switched to RTMP.
livestreamer "http://connectcast.tv/jane"
[cli][info] Found matching plugin connectcast for URL http://connectcast.tv/jane
error: Unable to open URL<EMAIL_ADDRESS>(400 Client Error: Bad Request)

RTMP Proxy Server v2.4
(c) 2010 Andrej Stepanchuk, Howard Chu; license: GPL

Streaming on rtmp://<IP_ADDRESS>:1935
WARNING: Trying different position for client digest!
Processing connect
app: live
flashVer: WIN 18,0,0,194
swfUrl: http://connectcast.tv/jwplayer/v6.12/jwplayer.flash.swf
tcUrl: rtmp://stream.connectcast.tv/live
pageUrl: http://connectcast.tv/jane
Playpath: mp4:CC-2411FC8A-A3D8-842F-C081-14EE02B707E5-8562
Saving as: mp4_CC-2411FC8A-A3D8-842F-C081-14EE02B707E5-8562
INFO: Metadata:
INFO:   duration              0.00
INFO:   fileSize              0.00
INFO:   width                 1280.00
INFO:   height                720.00
INFO:   videocodecid          avc1
INFO:   videodatarate         700.00
INFO:   framerate             30.00
INFO:   audiocodecid          mp4a
INFO:   audiodatarate         128.00
INFO:   audiosamplerate       44100.00
INFO:   audiosamplesize       16.00
INFO:   audiochannels         2.00
INFO:   stereo                TRUE
INFO:   encoder               obs-output module (libobs version 0.10.1)
WARNING: ignoring too small audio packet: size: 0

C:\Users\Prooxy\Documents\livestreamer>livestreamer https://www.twitch.tv/dreamhackcs best
[cli][info] Found matching plugin twitch for URL https://www.twitch.tv/dreamhackcs
error: Unable to open URL: https://api.twitch.tv/api/channels/dreamhackcs/access_token.json (400 Client Error: Bad Request)
I have the same thing.

Ditto:
$ livestreamer http://www.twitch.tv/lady_fury best
[cli][info] Found matching plugin twitch for URL http://www.twitch.tv/lady_fury
error: Unable to open URL: https://api.twitch.tv/api/channels/lady_fury/access_token.json (400 Client Error: Bad Request for url: https://api.twitch.tv/api/channels/lady_fury/access_token.json?as3=t)

Going to https://api.twitch.tv/api/channels/lady_fury/access_token.json?as3=t I get the following response:

{"error":"Bad Request","status":400,"message":"No client id specified"}


Support all JVM apps

This is a better, and more general detection of a Java app.
The pom.xml only exists for apps using the Java buildpack. But we also want to support:

WAR files
Executable JAR files
Gradle apps
Clojure, Scala, JRuby apps

All of these will have the JDK installed by the heroku/jvm buildpack, which creates the .jdk/bin/java file in the slug.

I've realized that we also need to change bin/compile to accomidate this (so that the agent jar file is copied into the slug). But the .jdk will not exist if this buildpack runs before the JVM buildpack.
One solution is to repeat the logic of the bin/detect scripts in the Java, Scala, Clojure, and Gradle buildpacks. But that will not work for WAR file deployment with heroku deploy:war.
If this buildpack was required to run after all other buildpacks, we could check for the .jdk dir.
Or, we can wait for a solution that doesn't require a buildpack.

Actually, if we want to support the following:

Docker
Heroku Deploy CLI
Platform API /slugs endpoint

Then we need a solution that doesn't use a buildpack. I vote to close this in favor of waiting.

We could support all of the "buildpack" paths by doing all of the work at boot time and have bin/compile just copy in our .profile.d script which does the actual work.
That doesn't solve Docker/CLI Deploy//slugs API and is also meh for other reasons.

Closing this as per @jkutner's ocmment.


Repo feature request: Turn on Branch protection and require reviews for main

Turn on Branch protection for main to prevent invalid submissions like #8

THIS BF LIST SHIT GETS SERIOUS

MAYBE ONE DAY I WILL UNDERSTAND HOW GITHUB WORKS


[F] messed up level when following specific steps

open new instance of Krossr
open Levels list
click 'play' on first level
intentionally lose the game by filling in bad tiles
new level
click 'play' on a larger level
open Levels list
click 'play' on level from step 3
screen is messed up

no longer seeing this one


Only clamp to source extent for safe transforms

This pull request makes the coordinate clamping to the validity extent in ol/proj.createSafeCoordinateTransform more straightforward: it only clamps the source coordinate to the validity extent of the source extent.
Fixes #11238.

I updated this pull request to a much simpler approach that merely ensures sanity of the source coordinate.


Broken link

Hi!
"Plan capacity for embedded analytics with Power BI Premium" link is not working.
Could you fix it?
Thanks!

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

ID: b427b153-9627-17f6-5a01-0b7009394944
Version Independent ID: b5434461-32a3-8c8d-6faa-fe3144718784
Content: Whitepapers for Power BI - Power BI
Content Source: powerbi-docs/guidance/whitepapers.md
Service: powerbi
Sub-service: powerbi-service
GitHub Login: @maggiesMSFT
Microsoft Alias: maggies

Hi, @RoseAmS -- I don't see the link you're referring to on this page, https://docs.microsoft.com/en-us/power-bi/guidance/whitepapers#feedback.
Can you point me to it? Thanks!

Hi @maggiesMSFT !
Seems like that whitepaper was deprecated (I'm seeing page's last edit was yesterday), couldn't find it either.
Thanks anyway!

Thanks, @RoseAmS. What timing! Closing this issue.


Refactor pp card clone to use util instead of calling other command, Closes #5242

Closes #5242

Review ASAP we will 😜.

setting to draft due to the following observations
I will provide more guidance how we should proceed with this PR in near future


How to train a semi-supervised learning

Hi @yueyu1030 ,
Thanks for your great work.
I wonder how to use your code on semi-supervised learning. I want to provide a small amount of gold data and a large amount of unlabelled data and use COSINE for semi-supervised learning.
Which method should I use? clean or selftrain? How should I set up the training data?
Thanks.
Chiyu

Hi Chiyu,
To adapt COSINE for semi-supervised learning, I suggest you first train your model with clean labels (the same as the 'init' step in the paper), then use self-training to bootstrap over all data (labeled + unlabeled).
Best,
Yue

Hi @yueyu1030 ,
Thanks for your reply.
After training on clean data, do I need to apply it to unlabelled data or save predictions to another file?
I saw your unlabelled samples contain labels.
Best,
Chiyu

Hi Chiyu,
Yes, you need to save the pre-trained model training on clean data, which serves as the initialization for self-training.
For unlabeled data, here we keep the labels to make the format of labeled & unlabeled data the same, but we never use them in training.
Best,
Yue

Thanks, @yueyu1030 !
I have another question. Could you tell me hyperparameters used for semi-supervised learning? Leaning rate, batch size, a number of training epochs/steps?
Thanks!

I also wonder about the data loader when the training mode is "clean".
In trainer script, I think we should only use the gold data when the training mode is "clean". But this line combines gold data and unlabelled data. Could you check this?
Thank.
Chiyu

Hi,
For parameter setting, since COSINE is not mainly designed for semi-supervised learning, I haven't tuned the hyperparameters carefully. Overall, I believe the learning rate for fine-tuning can be searched from {1e-5, 2e-5, 5e-5} and {1e-6, 5e-6, 1e-5} for self-training, batch size for labeled data can be {4, 8, 16} and the number of epochs could be around 10 for initialization and {1000, 2000, 3000} steps for self-training.
For training with clean labeled data, we need to use both labeled data and unlabeled data (since this mode is mainly used for estimating the model's performance with full clean data). So we need to combine the gold data and unlabelled data.
I am traveling outside so the response might be a little late. Sorry for that and feel free to ask me other questions wrt. experiments.
Best,
Yue

Hi,
Thanks. Hope you have a nice trip.
I still cannot get the point of why you combine the gold data and unlabelled data when you train on clean mode. I think you train a model with gold data and evaluate on gold data initially, and then do self-training that uses both the gold data and unlabelled data.
Another question: When your training mode is "selftrain", I find you already had two steps (initial training and self-training )in your code. I guess that I don't need to have a separate step to train an initial model with gold data and that I can just give the gold data as training data and train on "selftrain" mode. What do you think?
Best,
Chiyu

Hi,
Thanks. Hope you have a nice trip.
I still cannot get the point of why you combine the gold data and unlabelled data when you train on clean mode. I think you train a model with gold data and evaluate on gold data initially, and then do self-training that uses both the gold data and unlabelled data.
Another question: When your training mode is "selftrain", I find you already had two steps (initial training and self-training )in your code. I guess that I don't need to have a separate step to train an initial model with gold data and that I can just give the gold data as training data and train on "selftrain" mode. What do you think?
Best, Chiyu

I am also using this code for my research. I too agree with your questions. Did you try to find any solutions, or how did you approach ?


chore: EthereumBloomFilter improvement

Added docs, structured code a bit better, some refactoring with at least 10% improvement in the calculation of a filter.
This PR contains refactoring of bloom9 function.
The rest is just renaming a file, reordering of functions, documentation and one performance test.

Thank you, I'll merge it soon, after meeting my huge PR that merges all different transaction types into one.

@JeneaVranceanu Sorry I've broke your PR, could you please update it?

@yaroslavyaroslav Conflicts fixed.


Flow Export Improvements

There are currently three ways to export flows in mitmproxy:

b "save request/response body"
E "export"
P "copy request/response (content/headers) to clipboard"

This is kind of confusing - in particular, there's a lot of overlap between E and P.  I'd propose we do something like this:

Thoughts?

This is certainly better than what we currently have. Apart from the b key duplicating things a little bit, I can't see anything wrong with it.
I'll start working on this as soon as possible! :smile_cat:
P.S: What tool did you use to create that neat chart?

Great, happy to hear that! :smiley:
Chart is made in 5 minutes with Microsoft Visio. :stuck_out_tongue:

Do you think it's possible to account for marked flows? I have the occasional use case where I export multiple raw requests/responses to include them in a report. For example, to document a login or redirect sequence.

I think we want to limit this export feature to single flows for now. The semantics of the whole feature get too complex otherwise I'm afraid.

These hotkeys should be available in both flowlist and flowview right?
In flowview, the key p is already bound to previous flow!

Also, Headers+Content is the same as Raw Request

These hotkeys should be available in both flowlist and flowview right?

Yes! :smiley:

In flowview, the key p is already bound to previous flow!

Hm - unless there are better options, we can probably remove this?

So, having them both doesn't make much sense.

Agreed - let's just keep one of these :+1:

The Up/Down arrow keys seem to free in flowview. I could bind Prev/Next to them?


gopass for my Android

I am currently using Kpcli for ubuntu and Keepass2Android for my Android and they use the same format so I can share them on Dropbox.
It looks like gopass is not an alternative since it's only going to work on my laptop. Are there any plans to also have a mobile (or mobile web) solution?

AFAIK some people use pass-compatible apps on Android. Maybe @metalmatze can help.

Yes. I am using https://play.google.com/store/apps/details?id=com.zeapo.pwdstore for over a year now and its compatible with gopass.
It uses git to synchronize all files.

fantastic! I'll give it a try.

i would like to recommend this to my friends. is there also a version for iphone, Mac, and Windows?

gopass is confirmed to work on Linux and Mac. The maintainers and many contributors use either one of these platforms. Windows is known to work, but it depends a lot on the exact environment (cmd.exe, PowerShell, cygwin shell, bash for windows, ...). There should be a compatible iPhone app, but I'm not sure if it works.

got it. thanks!

Hi, I found this question while looking for some information on how to use gopass and Android and wondered if the mentioned application supports the yubikey to store the gpg keys

For iPhones, I can recommend pass for IOS. It's pass compatible and only supports one store (git repo) but otherwise works fine in my gopass setup.

Hi, I found this question while looking for some information on how to use gopass and Android and wondered if the mentioned application supports the yubikey to store the gpg keys

@zonArt, the Android App works with YubiKeys indeed! I use it every second day on my OnePlus3 with a YubiKey 5C.

in fact you can use gopass as-is directly on your android:

install and init Termux
install gopass with apt install gopass gnupg
[optional, just for fun] install the original pass and otp support with apt install pass pass-otp
[optional, for better usability] install Termux Widgets so that you can add shortcuts directly to your android desktop and get an OTP token with a single tap

I'm using https://github.com/android-password-store/Android-Password-Store on my side and it works well on Android and supports Yubikeys / hardware token through OpenKeyChain and even displays key-values entries properly.

Also using Android-Password-Store BUT the app only supports a single repo + single GPG key and there are no plans to implement multiple repos (my use case). That's why I settled on using the app for my main store and gopass in termux for the additional stores.

Yes. I am using https://play.google.com/store/apps/details?id=com.zeapo.pwdstore for over a year now and its compatible with gopass. It uses git to synchronize all files.

The link is broken :-( Or the app is not anymore listed

@N7K4: the current play store and f-droid links are in the git repo

Sure! My question wasn't precise: Are people from the gopasswd community plan to close the gap on Android.


Update release-process.md

Add step to update download links for MSI installer guide

Guillermo maybe you can help merge this as I don't have permissions.


Update to protractor and Angular versions

A update to run protractor tests with version 3.0.0.
Angular version updated to run the latest stable version 1.4.8.

Fixed in #341.


implement api

Codecov Report

Merging #65 (6d62259) into master (7a6f901) will decrease coverage by 0.84%.
The diff coverage is 0.00%.

:exclamation: Current head 6d62259 differs from pull request most recent head 2ab4046. Consider uploading reports for the commit 2ab4046 to get more accurate results

@@            Coverage Diff             @@
##           master      #65      +/-   ##
==========================================
- Coverage   52.08%   51.24%   -0.85%
==========================================
  Files          68       69       +1
  Lines         912      927      +15
  Branches       26       24       -2
==========================================
  Hits          475      475
- Misses        437      452      +15

Impacted Files
Coverage Δ

...n/scala/org/bitlap/zim/server/api/ZimUserApi.scala
0.00% <0.00%> (ø)

.../bitlap/zim/server/api/endpoint/ApiJsonCodec.scala
0.00% <0.00%> (ø)

...a/org/bitlap/zim/server/api/endpoint/package.scala
0.00% <0.00%> (ø)

...itlap/zim/server/application/impl/ApiService.scala
33.33% <0.00%> (-16.67%)
: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 7a6f901...2ab4046. Read the comment docs.


Writing on Win10 occasionally fails (and loses data)

Operating System and Hardware: Windows 10
NodeJS Version: electron 0.37.2
Serialport version: 2.1.2
Python Version: (unknown)
Expected behavior
Writing to the serialport would occasionally block in a thread, as it does on Linux and OS X. The WriteQueue for that fd will continue to be added to for further writes. (See https://github.com/voodootikigod/node-serialport/blame/master/src/serialport.cpp#L298 , which is code I helped add. 😄 ).
Alternatively, if blocking in a thread isn't acceptable for Windows, the timeout should NOT pop that write from the queue but instead modify it to prevent double-sending of sent portion of the data, and ensure that the send will be attempted again very shortly. (We don't want to cause a performance issue here.)
It's vital that we don't add semantics that pay attention to the flow control of the device. This is regularly abused and causes all sorts of troubles. (Flow control is currently broken in Python for us for this very reason.)
Actual behavior
An error (generated from https://github.com/voodootikigod/node-serialport/blob/master/src/serialport_win.cpp#L374 ) with error code 121 ("Semphore Timout": https://msdn.microsoft.com/en-us/library/windows/desktop/ms681382(v=vs.85).aspx#ERROR_SEM_TIMEOUT ).
Steps and/or code to reproduce the issue
Basically, you want to coerce the device to not read what we write, forcing Windows to block while writing and eventually timing out, losing data in the process.
Option 1 (explicit flow control):

Open connection to a device with flow control enabled (hardware is what we are using)
write() data to the device until the device's buffer fills and it signals that flow control should stop. NOTE: This should NOT be explicitly handled by serialport. The OS handles this properly and that should be left as-is.
write() more data to the serialport. The error will happen and the data written will be lost instead of queued.

Option 2 (USB-based flow control):

Open a connection to a device such as an Arduino Due (native port) that has a USB virtual-serial port sketch running.
Instruct the Due to not read from that serialport, allowing it's buffer to full and it will then stop accepting packets from the USB host.
3.write() data to device until you see the error.

I'm just now starting to work on this again. We have customers waiting on it, so I'll have some more info later today.

@reconbot Some more background on the problem, how we're using it, and what I think/hope will fix this.
Note that this issue doesn't exist on Linux or OS X. In both of those OSs, the write it truly blocking -- which is fine, it's in it's own thread.
Background: I work at @synthetos on the TinyG and G2 Core (the next generation of TinyG) projects. In both cases, we use either a USB->serial port or a true serial port (from a BeagleBone, C.H.I.P, or Pi, for example) to talk to the board.
In G2 Core, we are doing high-speed control of machines like 3D printers. We queue up movements that come in from the serial port, and when our queue or serial port buffers are full, we stop reading from the serial port.
To prevent the data from getting lost to /dev/null during this time, we use hardware flow control (real or simulated with the USB->serial) to tell the host OS (which is the part running node-serialport) to stop writing.
Problem Description: When the host OS is told to stop writing, it continues to buffer data before it starts to block writing to the file descriptor. On OS X and Linux we tell it to use block without a timeout, and that works perfectly. On Windows, we don't configure the timeouts, and then when it does timeout we throw away the "packet" that partially sent and then move to the next packet.
Probable Solution: Simply set the write timeouts to zero. Then the thread would completely block for the write to complete, or the port to close.
There is one other concern. When you compare the linux write call to the windows write call you see that the windows version writes from data->bufferData for data->bufferLength bytes, even though it's in a while loop that adjusts data->offset. The linux version has it right by writing from data->bufferData + data->offset for data->bufferLength - data->offset bytes.
I'll now attempt to work out these fixes and see if I can find a clean test to prove not-working before and working after.

I sent some data to ESP8266's NodeMCU. Same data loss happened. When I set baudrate to 9600 it works perfectly.
@giseburt,  I tried to apply your solution but I haven't succeded yet. The same data loss happened again on Windows 10.
I tried to set "write timeouts" to 0. The same issue happened. I also fixed the data->bufferData + data->offset issue. But i think that this is not the case. I believe that it doesn't iterate. So that one has no effect at all.

I thought about that too. I wasn't sure enough to change it without proof
that it was broken.
Logically, it seems that there should not be timeouts for reads either. I
can't think of what it could harm to disable them as well.
On Wed, Oct 19, 2016 at 8:53 PM Francis Gulotta<EMAIL_ADDRESS>wrote:

Thoughts on the read timeouts? It doesn't seem like we'd want them either
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/EmergingTechnologyAdvisors/node-serialport/issues/781#issuecomment-254988827,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAXj0Sbs1tJUeSW0faXHb60zRIZMPL8Aks5q1sl_gaJpZM4IVps3
.

Sounds good. 👍🏻
On Wed, Oct 19, 2016 at 9:34 PM Francis Gulotta<EMAIL_ADDRESS>wrote:

Lets do it and give it a good manual testing.

Francis Gulotta
<EMAIL_ADDRESS>On Wed, Oct 19, 2016 at 9:56 PM, Rob Giseburt<EMAIL_ADDRESS>wrote:

I thought about that too. I wasn't sure enough to change it without proof
that it was broken.
Logically, it seems that there should not be timeouts for reads either. I
can't think of what it could harm to disable them as well.
On Wed, Oct 19, 2016 at 8:53 PM Francis Gulotta <
<EMAIL_ADDRESS>wrote:

Thoughts on the read timeouts? It doesn't seem like we'd want them
either
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<https://github.com/EmergingTechnologyAdvisors/
node-serialport/issues/781#issuecomment-254988827>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/
AAXj0Sbs1tJUeSW0faXHb60zRIZMPL8Aks5q1sl_gaJpZM4IVps3>
.

—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
<
https://github.com/EmergingTechnologyAdvisors/node-serialport/issues/781#issuecomment-254989231
,
or mute the thread
<
https://github.com/notifications/unsubscribe-auth/AABlbo3VHcnplk4PtBS2-_-qVECiIBn5ks5q1spTgaJpZM4IVps3
.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/EmergingTechnologyAdvisors/node-serialport/issues/781#issuecomment-254993779,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAXj0ad-mMoLZAzL-qkLf1lk9qspXkmOks5q1tNQgaJpZM4IVps3
.

I gave a try to version 5.0.0-beta2.
Result: programm gets no response from device.
Even Device Monitoring Studio shows no signs of response.
I rollwd-back patch #974 and rebuilt binding from source and program started to function again.
I use rs-485, no flow control.

According to https://msdn.microsoft.com/en-us/library/windows/desktop/aa363190(v=vs.85).aspx and other sources referring to that same:
ReadIntervalTimeout
The maximum time allowed to elapse before the arrival of the next byte on the communications line, in milliseconds. If the interval between the arrival of any two bytes exceeds this amount, the ReadFile operation is completed and any buffered data is returned. A value of zero indicates that interval time-outs are not used.
A value of MAXDWORD, combined with zero values for both the ReadTotalTimeoutConstant and ReadTotalTimeoutMultiplier members, specifies that the read operation is to return immediately with the bytes that have already been received, even if no bytes have been received.
So I believe what we want, what other platforms call "non-blocking read," is to set ReadIntervalTimeout to MAXDWORD.
(I emailed this and it's not showing up so hopefully it doesn't duplicate.)

According to
https://msdn.microsoft.com/en-us/library/windows/desktop/aa363190(v=vs.85).aspx
and other sources referring to that same:
ReadIntervalTimeout
The maximum time allowed to elapse before the arrival of the next byte on
the communications line, in milliseconds. If the interval between the
arrival of any two bytes exceeds this amount, the ReadFile operation is
completed and any buffered data is returned. A value of zero indicates that
interval time-outs are not used.
A value of MAXDWORD, combined with zero values for both the
ReadTotalTimeoutConstant and ReadTotalTimeoutMultiplier members, specifies
that the read operation is to return immediately with the bytes that have
already been received, even if no bytes have been received.
So I believe what we want, what other platforms call "non-blocking read,"
is to set ReadIntervalTimeout to MAXDWORD.
-Rob
On Fri, Nov 4, 2016 at 8:12 AM Francis Gulotta<EMAIL_ADDRESS>wrote:

Reopened #781
https://github.com/EmergingTechnologyAdvisors/node-serialport/issues/781
.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/EmergingTechnologyAdvisors/node-serialport/issues/781#event-847859053,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAXj0cNtqmYCM6VkV5ejA5ln7w_eP4q9ks5q6y8hgaJpZM4IVps3
.

How does v4.0.5 work for you?

With<EMAIL_ADDRESS>everything is ok.


HTML markup in text field?

Is it possible to include HTML markup (a link in particular) for a field?

Just redefine itemTemplate where you return markup as string, DomNode or jQueryElement. See field configuration in docs: http://js-grid.com/docs/#fields

Excellent, this works


feat: author component

Signed-off-by: Sepush<EMAIL_ADDRESS>

/cc @gaocegege

Could we move the author section to the last or the right section?

Could we move the author section to the last or the right section?

if you want put to the last just move <Author /> to the last of the paper

SGTM