Alt-tabbing out of a fullscreen client doesn’t display the next client

It gives the focus to the next client, but the buffer displayed is still the one of the fullscreen client.

Can confirm this bug is still present.


style: ESLint & Prettier

 yarn.lock is added for development purposes. I can remove it from the library with the .npmignore if you prefer so.

@mvayngrib all changes are ready!


Consider enhancing custom db folder path

I would second this, if I understand the issue correctly. I would love a flag to let me set a custom location for the db folder. It is a little inconvenient to have it be right next to the binary. I would prefer to have the binary placed somewhere standard, which is already on my path, and then define a custom location for the db.

yeah


Not all folders are showing up

When I try and use it I only get a certain number of the top folders. Upon adding the name of the maps etc it still doesn't show up. If I fully type out the name, the file does get moved to the correct folder.

It might have been related to the folders not having a file already in them / being freshly created.

Congratulations! You're the first issue!
Unfortunately, it looks like Obsidian's default behaviour is to ignore empty folders, which is why you cannot see folders in the fuzzy search dropdown.
I believe you have a few options:
1. Do not create empty folders.
Instead, use the file-chucker to create them when you need them. With Tab-completion, you don't have to type the whole path of your folder!
For example, if your target folder is: "/Programming/Javascript/Testing" and you already have the folders "/Programming/Javascript" created.
just type "java" into the fuzzy searchbox. Then use the arrow keys to select "/Programming/Javascript" and press the TAB key. The whole path will be filled in the drop down!
Then finish the path by typing in "/Testing", then press ENTER to create and move the file there.
2. Use a plugin that creates a "default" file in a folder
I currently use Folder Note, but you'll have to still manually create the "Folder Note" everytime you make a new folder.
Hope you're enjoying file-chucker!

This is a pretty surprising behaviour. Any chance of a fix ?


Fix Generated Definition File

Replace react-tether custom d.ts with @types/react-tether
Remove useless @types modules
Remove type folder
Add missing type in IReduxAction

Codecov Report

Merging #396 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff            @@
##           master    #396      +/-   ##
=========================================
+ Coverage    99.8%   99.8%   +<.01%
=========================================
  Files         155     155
  Lines        3676    3677       +1
  Branches      878     878
=========================================
+ Hits         3669    3670       +1
  Misses          7       7

Impacted Files
Coverage Δ

src/components/popover/Popover.tsx
100% <100%> (ø)
:arrow_up:

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 49018e6...363253d. Read the comment docs.


openshift-ci: Fix kubeconfig path

The path of the kubeconfig file doesn't match the location in OCP CI clusters.

/lgtm
/approve

/cherry-pick release-4.10


Send public reset when receiving FHOL from client

I keep on thinking that the right thing to do is not to advertise support for Q036.  As long as Chrome supports Q035, the drawback to current approach is that connections chosen for FHL2 experiment will fail (instead of all connections using Q035 and succeeding).  However, as soon as Chrome drops Q035 support, this becomes a "better than nothing" approach.

Hi @Aron-Schats, I agree with you in principle. I'd rather skip version 36 completely instead of implementing this hack.
However, the problem is more severe than you described, because Chrome doesn't do version negotiation (yet?). That means as soon as Chrome decides to speak version 36 by default, quic-go would be unusable.
Furthermore, according to the Google QUIC team, the FHL2 experiment will only be activated in about 1% of all Chromes. Since Chrome is racing a QUIC and a TCP request anyway, the impact on the latency will be quite small for those 1%.

However, the problem is more severe than you described, because Chrome doesn't do version negotiation (yet?).

I don't think this is correct.  Some time ago, I added command-line flags to proto-quic's server and client to specify versions explicitly (specifically to test the fallback mechanism) and the fallback worked as one would expect it to.

I just tried it again. Chrome still keeps ignoring Version Negotiation Packets.

That's odd.  What happens -- does it just fail to connect?  If you run quic_client --v=4 instead of Chrome, it may provide some clues.  Is it possible there is a bug in quic-go?

I just edited the SupportedVersions in protocol/version.go such that version 36 is the only supported version. Still Chrome just keeps on sending packets with version 35.

I think Chrome and the quic_client have always behaved differently when it comes to version negotiation: The quic_client supports it but Chrome doesn't.

Interesting.  Perhaps, then, the better way to go is to send back a version negotiation packet with VERSION=Q035 if CHLO has FHL2=1 instead of public reset.  This way, if or when Chrome supports fallbacks, it can fall back.

I guess that would work, but tbh I'm not sure it's worth the effort. Chrome will always fallback to TCP (without additional latency cost due to the parallel connection) if a connection can't be established using QUIC. Also, it's probably unlikely that we'll see version negotiation in Chrome until after v36 has been retired.
Do you have a specific use-case where this fallback would be an issue?


Implements support for automatically creating a zip archive of the pr…

…otocols on package
Related to https://github.com/cloudstateio/cloudstate/issues/118
This would mean that we could have a checked in zip-archive which lang supports could fetch over HTTP and unzip.

Ping @marcellanz

Ping @marcellanz

@viktorklang tested and works for me.

@marcellanz Cool, thanks for verifying! Should we include this in the developer documentation as well perhaps?

@marcellanz Cool, thanks for verifying! Should we include this in the developer documentation as well perhaps?

@viktorklang sure :-)

@viktorklang examples was left out by intention right? Language support would need it to implement the TCK. "examples" are not part of the spec, but if the consumer if the zip is language support or alternative proxy implementers, the examples are needed by them. WDYT?

@marcellanz Yeah, I kept the shopping-cart protos outside of it. Do we want to have an extra zip-file for the shopping-cart protos?

@viktorklang I don't know, really. And name it tck-protos?

@marcellanz That's a great suggestion, we might have more of those later anyway. I'll amend this PR.

@marcellanz Check the latest commit(s)

@viktorklang
works... should it get cleaned by sbt:cloudstate> clean?

(digged into sbt documentation, let me know if it should get cleaned and I'll find out how, having to learn some sbt I think)

@marcellanz I'll add cleaning

@marcellanz Added the files to clean.

Hi, folks 👋
@marcellanz When this PR will be merged, will you migrate a go-support to using these artifacts?

Hi, folks 👋
@marcellanz When this PR will be merged, will you migrate a go-support to using these artifacts?

@ihostage Its not a high priority yet, but yes I think user support libraries should get the protobuf files through these artefacts. Currently go-supporthas them, together with the *.pb.go files, distributed with a release. Do you have something in mind?

Do you have something in mind?
@marcellanz Nothing concrete 😄
Just I wait for these artifacts and try to find time to play with another language support 😄
But the end of the year is a hard period and find time is very difficult. 😞

Do you have something in mind?

@marcellanz Nothing concrete 😄
Just I wait for these artifacts and try to find time to play with another language support 😄
But the end of the year is a hard period and find time is very difficult. 😞

@ihostage cool! Yeah time... I know, one can squeeze for a few weeks but then one have to pay tribute and compensate. Which language do you have in mind?

@marcellanz Merged! Also, added notes in https://github.com/cloudstateio/cloudstate/blob/master/docs/src/main/paradox/developer/language-support/index.md#creating-language-support-libraries

@viktorklang awesome, thanks!


Build failure: chadwm

Steps To Reproduce
Steps to reproduce the behavior:
{ config, pkgs, ... }:
{
#services.xserver.displayManager.defaultSession = "none+dwm";
services.xserver.windowManager.dwm.enable = true;
nixpkgs.overlays = [
(final: prev: {
dwm = prev.dwm.overrideAttrs (old: { src = /home/erik/.config/arco-chadwm/chadwm ;});
})
];
}
Build log
How to log?
Here is an image

Additional context
Add any other context about the problem here.
Arch Linux has a file called Imlib2.h
Nixos does not have that file
referring to this file will break the build
https://github.com/erikdubois/nixos-configurations/blob/b3cbdfeb169f7e1ccee9f093afff56e7a57c1c70/dotfiles/.config/arco-chadwm/chadwm/drw.c#L7
Notify maintainers
I think the maintainer is empty
https://github.com/NixOS/nixpkgs/blob/nixos-23.05/pkgs/development/libraries/imlib2/default.nix#L76
Metadata

Add a :+1: reaction to issues you find important.

The dwm package does not have Imlib2 as a package dependency, this is not a dwm or Imlib2 package related issue.
Look into overriding to include Imlib2 as a dependency for dwm.

I do believe Imlib2 is installed but we are missing a file imlib2.
If you download the imlib2 package from deepin sourceforge and extract it we see it is part of the scr/lib files

When I try to find this file on NixOS I can not find it.
I thought I should change the path to the folder/file. But if there is no imlib2.h file.

No, that is not how you install libraries, you must override the package itself that needs it.

and the fact that imlib2 is missing a file is not the issue?

Nothing would be missing if the package was correctly overridden, Imlib2.h is present in the .dev output, which is correctly picked when you add imlib2 to the buildInputs of the package.

I think I know what you mean.
Since this is too difficult for me I have requested this package.
https://github.com/NixOS/nixpkgs/issues/282239
I will close this issue for now.


WIP: Taal

This implements a first prototype of Taal: Trixi as a library. There is still a lot of work to do before we reach feature parity with the previous versions, but we can run a simple example such as examples/2d/parameters.toml → examples/2d/parameters.jl. I wanted to push this now so that you can get an impression of the look & feel and we can discuss how to proceed.
TODO

[x] basic infrastructure, semidiscretizations etc.
[x] Euler + gravity
[x] weak form
[x] split form
[x] shock capturing
[x] boundary conditions
[x] non-conservative terms (MHD)
[ ] restarts
[x] SummaryCallback
[x] AMRCallback
[ ] AMRCallback for Euler + gravity
[x] StepsizeCallback
[x] AnalysisCallback
[x] SaveSolutionCallback
[x] AliveCallback
[ ] make it easier to create new callbacks by hiding some DiffEq-specific stuff?
[ ] positivity preservation
[ ] elixir: linear structure
[ ] elixir: Jacobian
[ ] elixir: fluctuation stuff
[ ] porting more setups
[x] more inline documentation and explanations
[x] consistent naming of Vector-like u from DiffEq.jl and "our" solution array u and documentation/explanation of wrap_array
[ ] consistent use of return, update the dev docs accordingly
[ ] fix all remaining # TODO: Taal
[ ] 3D

Connections to issues
Xref #177 #161 #150 #139 #136 #83 #73 #70 #19 #10
Closes #173

By the way, I've marked this as "high-priority", since @gregorgassner categorized Taal in this way in our last meeting.

We now have more analysis stuff, but that requires https://github.com/SciML/OrdinaryDiffEq.jl/pull/1272 for 2N low-storage methods.
Memo: Set compat entry for dependencies on OrdinaryDiffEq accordingly once that PR is merged and a new version is released.

I've now finished a first draft of the most important design decisions and most features are implemented in 2D. Thus, it's a good time to get some feedback!
The current test tolerances make use of one of my PRs into OrdinaryDiffEq.jl.  Nevertheless, the basic setup should also work for you locally with a current release of OrdinaryDiffEq.
@sloede @gregorgassner @andrewwinters5000 Could you please review this PR carefully?

When everything looks okay to you, I would like to merge this into trixi-framework:hr/taal and continue to develop Trixi as a library in my fork, creating PRs for every separate WP into that branch until we're ready to merge into trixi-framework:master.


Go to Definition with more configurations in launch.json

Describe the bug
We are not able show base application objects (.dal) when launch.json contains more than one configuration.
To Reproduce

There is only one configuration in launch.json.
Define variable Item: record Item; and Go to Definition (F12) -> works OK.
Add Configuration in launch.json (clone the same or create different one)
Go to Definition (F12) -> empty definition shown

Expected behavior
Standard object definition downloaded and shown from configuration used in Debug.
Versions:

AL Langauge: 2.0.48254
Business Central: On-premises 13.0.24623

Hi @IvoMol, thanks for opening this issue. I can't reproduce this issue on AL language version 2.0.48254. Can you provide your VS Code version and which OS you are using?

AL Language: 2.0.43900
BC: BE Dynamics NAV 13.0 (24630)
{     "version": "0.2.0",     "configurations": [         {             "type": "al",             "request": "launch",             "name": "BC130",             "server": "http://localhost",             "serverInstance": "BC130",             "port": 7149,             "authentication": "Windows",             "startupObjectType": "Page",             "startupObjectId": 99000,             "breakOnError": true,             //"schemaUpdateMode": "Recreate"         },         {             "type": "al",             "request": "launch",             "name": "BC130BE",             "server": "http://localhost",             "serverInstance": "BC130BE",             "port": 7179,             "authentication": "Windows",             "startupObjectType": "Page",             "startupObjectId": 99000,             "breakOnError": true,             //"schemaUpdateMode": "Recreate"         }     ] }
Works fine

Retested on newer BC on-premises version (CU01):
Microsoft (R) AL Compiler version 2.0.56865
Microsoft_Application_13.0.25924.0
** Tested AL code, Customer List -> Go to Definition: **
pageextension 50100 CustomerListExt extends "Customer List"
{
trigger OnOpenPage();
begin
Message('App published: Hello world');
end;
}
** Original launch.json **
{
"version": "0.2.0",
"configurations": [
{
"type": "al",
"request": "launch",
"name": "Your own server",
"server": "http://localhost",
"serverInstance": "DynamicsNAV130",
"authentication": "Windows",
"startupObjectId": 22,
"startupObjectType": "Page",
"breakOnError": true
}
]
}
** Correct Customer%20List.dal with full code **
OnInit()
SetCustomerNoVisibilityOnFactBoxes;
CaptionTxt := CurrPage.CAPTION;
SetCaption(CaptionTxt);
CurrPage.CAPTION(CaptionTxt);
PowerBIVisible := FALSE;
CurrPage."Power BI Report FactBox".PAGE.InitFactBox(CurrPage.OBJECTID(FALSE),CurrPage.CAPTION,PowerBIVisible);
OnOpenPage()
CRMIntegrationEnabled := CRMIntegrationManagement.IsCRMIntegrationEnabled;
SetWorkflowManagementEnabledState;
SETFILTER("Date Filter",'..%1',WORKDATE);
OnAfterGetCurrRecord()
SetSocialListeningFactboxVisibility;
CRMIsCoupledToRecord :=
CRMCouplingManagement.IsRecordCoupledToCRM(RECORDID) AND CRMIntegrationEnabled;
OpenApprovalEntriesExist := ApprovalsMgmt.HasOpenApprovalEntries(RECORDID);
CanCancelApprovalForRecord := ApprovalsMgmt.CanCancelApprovalForRecord(RECORDID);
WorkflowWebhookManagement.GetCanRequestAndCanCancel(RECORDID,CanRequestApprovalForFlow,CanCancelApprovalForFlow);
etc.
** Doubled launch.json **
{
"version": "0.2.0",
"configurations": [
{
"type": "al",
"request": "launch",
"name": "Your own server",
"server": "http://localhost",
"serverInstance": "DynamicsNAV130",
"authentication": "Windows",
"startupObjectId": 22,
"startupObjectType": "Page",
"breakOnError": true
},
{
"type": "al",
"request": "launch",
"name": "Your own server 2",
"server": "http://localhost",
"serverInstance": "DynamicsNAV130",
"authentication": "Windows",
"startupObjectId": 22,
"startupObjectType": "Page",
"breakOnError": true
}
]
}
** Wrong Customer%20List.dal, only with skeleton and global variables**
page 22 "Customer List"
{
layout
{
area(Content)
{
repeater(Control1)
{
field("No.";"No.")
{
}
field("Registered Name";"Registered Name")
{
}
etc.

Hi,
You are right. Having multiple configuration definitions is not supported with the go to definition feature.  This is by design for now.  The reason being that we have no implementation in our language server code base on how to choose the default configuration. Which is fully supported when debugging.

Even though it is by design, it should be made working. Marking it as a bug.

We have just experienced this bug with our AL environment (official BC15), 😞
additionally the "Run In Web Client" also only works with one configuration in launch.json.
Any chance this issue will be solved in time?

Closing this issue as it has been fixed for AL extensions with version higher than 5.0.235548


Normalizing clusterName to lowercase

This PR lowercases clusterName for albNamePrefix so that the alb names will be requested as lowercase regardless of what clusterName is. Fixes #396.

@seripap can you sign the CLA? LGTM

Signed :+1:


Scala 2.13.0-M4 Support

Initial version to support releasing for scala 2.13.0-M4.

@SethTisue @cheeseng
I added scalatest as done for the community-builds:
https://github.com/scala/community-builds/wiki/scala-2.13.x-migration
Is this correct?

@cheeseng @SethTisue ping again. Is there anything the Scala Center can do to help?

@MasseGuillaume for now, we're minimally okay — the M4 published artifacts seem to work, and scalatest is green in the 2.13 community build and downstream projects there that use ScalaTest seem to be working.
it would be better if ScalaTests's own test suite were passing. the blocker there currently is the compiler crash over at https://github.com/scalatest/scalatest/issues/1367#issuecomment-402752561


Fix linker issues and target support

While I don’t think the 3-6 month update cadence of the windows-sys crate was a substantial issue, dropping that dependency (#374) is certainly your choice. However, by doing so you gave up some functionality and target support which will prevent parking_lot_core from linking in many cases.
The windows and windows-sys crates depend on the windows-targets crate for linker support. The windows-targets crate includes only import libs, supports semantic versioning of libs, and optional support for raw-dylib. It provides explicit import libs for the following targets:

i686_msvc
x86_64_msvc
aarch64_msvc
i686_gnu
x86_64_gnu
x86_64_gnullvm
aarch64_gnullvm

By contrast, the older winapi crate only provided import libs for i686_gnu and x86_64_gnu and lacked support for raw-dylib, link-level semantic versioning, or ARM64. #374 further yanked all of this and means that dependents must now rely on some other implicit dependency to ensure that this crate’s imports will continue to link.
I think restoring windows-sys would be simpler in the long run, but as an alternative you can accept this PR which simply adds a dependency on the windows-targets crate directly to restore the target support described above.
Hope that helps.

Thanks! I think this is good enough for now since this code is unlikely to change for a long time.


Add Data Vault to Ethereum DIDs

At this moment the Data Vault is not appearing when logging in with an Ethereum DID.
Would it be possible to support the main chain and at least one testnet (eg, Kovan)?
Example logging in with Argent using Eth mainnet, no data vault section:

Reopening issue to add Ethereum testnets:

[x] Mainnet
[ ] Kovan
[ ] Ropsten
[ ] Goerli
[ ] Rinkeby

Reopening issue to add Ethereum testnets:

[x] Mainnet
[ ] Kovan
[ ] Ropsten
[ ] Goerli
[ ] Rinkeby


Union Support for MongoDB Embedded Document and Postgres JSON field

Prerequisite
https://github.com/prisma/prisma/issues/8837
https://github.com/prisma/prisma/issues/8272
Problem

MongoDB is known for its schema less feature, thus it result to free form schema. The problem here is that if there is no union support for document or embedded document we still end up putting a lot of null values to different fields like the SQL Table
Postgres will also have this problem if it does not support union for Json field.

Why is this a problem?
Right now, if you define a model in prisma you need to define a single model for example if you are creating a single table inheritance structure like thus making you save unnecessary fields that does not belong to the subtype.
Suggested solution

""" MODELS """
model User {
  id    String @id @default(cuid())
  name  String
  posts Post[]
}

model Post {
  id        String @id @default(cuid())
  authorId  String
  author    User @relation(fields: [authorId], references: [id])
  text      String
  data      TextData | LinkData | PhotoData | PollData | QuestionData
}

""" JSON | EMBEDDED DOCUMENT """
json TextData {
  type          DataType
}

json LinkData {
  ogUrl         String
  ogImageUrl    String
  ogTitle       String
  ogDescription String
  type          DataType
}

json PhotoData {
  url     String
  width   String
  height  String
  type    DataType
}

""" JSON | EMBEDDED DOCUMENT with JSON | EMBEDDED DOCUMENT """
json PollData {
  duration  Int
  options   PollDataOptions[]
}

json QuestionData {
  source    String
  options   QuestionDataOptions[]
}

""" DEEP JSON | EMBEDDED DOCUMENT """
json PollDataOptions {
  id    Int @id @default(autoincrement())
  text  String
}

json QuestionDataOptions {
  id      Int @id @default(autoincrement())
  text    String
  correct Boolean
}

enum DataType {
  text
  link
  photo
  poll
  question
}

This solves the following
IF IMPLEMENTED USING

Single Table Inheritance

Common fields are on the top level, which means they are being implemented in table column level in Relational DB
Uncommon fields are embedded via JSON field which greatly enhanced the Single Table Inheritance by having a free form fields because if it is implemented in table level you will have a bunch of null value for uncommon fields
Single Table Inheritance has an advantage over Class Table Inheritance and Concrete Table Inheritance - If we were to implement a LIKE Feature we don't need polymorphic association. We just need to specify the table Post

Class Table Inheritance - Cons if not implemented in Embedded Document style is that you will end up doing a bunch of Join just to get the subtype relation (https://www.martinfowler.com/eaaCatalog/classTableInheritance.html)

Generated Client
// Text
this.prisma.post.create({
  data: {
    authorId: 1,
    text: 'Hello Prisma',
    data: {
      type: 'text',
    },
  },
});

// Link
this.prisma.post.create({
  data: {
    authorId: 1,
    text: 'Hello Prisma',
    data: {
      type: 'link',
      ogUrl: 'https://www.prisma.io/',
      ogImageUrl: 'https://www.prisma.io/logo',
      ogTitle: 'Next-generation Node.js and TypeScript ORM',
      ogDescription: 'Prisma helps app developers build faster and make fewer errors with an open source ORM for PostgreSQL, MySQL and SQLite.',
    },
  },
});

// Poll
this.prisma.post.create({
  data: {
    authorId: 1,
    text: 'Best Database?',
    data: {
      type: 'poll',
      duration: 3600,
      options: [
        {
          id: 1,
          text: 'Postgres'
        },
        {
          id: 2,
          text: 'MySQL'
        },
        {
          id: 3,
          text: 'MongoDB'
        },
        {
          id: 4,
          text: 'SQLite'
        },
      ],
    },
  },
});

// Question
this.prisma.post.create({
  data: {
    authorId: 1,
    text: 'What is correct PI number?',
    data: {
      type: 'question',
      source: 'https://time.com/4699479/pi-day-history-origins/',
      options: [
        {
          id: 1,
          text: '3.12',
          correct: false,
        },
        {
          id: 2,
          text: '3.13',
          correct: false,
        },
        {
          id: 3,
          text: '3.14',
          correct: true,
        },
        {
          id: 4,
          text: '3.15',
          correct: false,
        },
      ],
    },
  },
});

Related: https://github.com/prisma/prisma/issues/2505


2274 feature concept manuel user registration for admins

🍰 Pullrequest

Issues

fixes #2274

Todo

[X] desgin frontend with mockups
[X] database migration
[X] backend service changes

the flaky test blocks this PR even only document files are changed.
As soon as the PR #2255 is merged the flaky test will be solved.


Blog post for New Account Console

Merged here: https://github.com/keycloak/keycloak-web/pull/144
There was some issues with the adoc file that was causing build warnings.


bug: After #522, (default) Apollo Client subscriptions no longer work with dgs-framework

When following the Apollo Client docs to set up a websocket subscription, DGS can no longer consume the connection_init message from Apollo, as of #522.
Expected behavior
DGS websocket subscriptions work with Apollo.
Actual behavior
Jackson databind throws an InvalidTypeIdException referencing com.netflix.graphql.types.subscription.OperationMessage["payload"]
Steps to reproduce
Run the additional OperationMessageTest case referenced below
-or-
Attempt to use a WebSocketLink with Apollo to an API using DGS
Notes
I write this under the assumption that DGS + Apollo Client is a combination the GraphQL community wants to "just work," as they're the two most popular JVM and React libraries, and file it as a DGS bug rather than an Apollo one as the Apollo behavior has been consistent for quite some time.
Downgrading to any version prior to #522 solves this issue.
The additional test case simulates what Apollo sends on initial connection, which is always a payload of {}.
I spent several hours playing around with Jackson annotation configurations trying to coerce {} to null, but this just isn't possible.
It would seem the only options are writing a fairly complex custom deserializer, or undoing the creation of the shared subscription types module. Neither of these seem perfect...
Test Case
https://github.com/zackary-clark/dgs-framework/commit/c3add1fb167a808c041e114922a525eba45a19cd

Thanks for reporting this @zackary-clark. That's definetely an unintentional side-effect, and we didn't know about this yet!
Apollo client should just work, so this needs to be fixed.

Fixed in 4.8.3 which we just released. Sorry for breaking it!
https://github.com/Netflix/dgs-framework/releases/tag/v4.8.3


[BUG] Premium badge didn't appear despite being a premium user

Has this bug been raised before?

[X] I have checked "open" AND "closed" issues and this is not a duplicate

Where did you find this bug?
Production on biodrop.io
Version of BioDrop (for example "v1.2.3")
2.100.2
Description
Premium badge not displaying in my profile tags despite being a premium user.
Steps to Reproduce
Here are the steps :

Go to Pricing page.
Then Go to Upgrade
Subscribe to the premium service by inserting your email
In your account's menu , It displays "You are now a premium user"
When you navigate to your profile , the premium badge doesn't appear in your profile tags

Screenshots

Do you want to work on this issue?
No
If "yes" to above, please explain how you would technically implement this
No response

@eddiejaoude can you take a look at this.

Sure, thanks @MustaphaArfa00 and @ChinmayMhatre 👍

I have just checked your account @MustaphaArfa00 and it is not a premium account, that is why the badge doesn't show. So I will close this issue, but if you upgrade and the badge doesn't appear, please let us know

ok @eddiejaoude thank you for viewing this issue but why is it displaying "You are now a premium user !" ?


Error messages loading module

Describe The Bug:
First, thank you for your efforts on this plugin!
Vehicles are not appearing on the Home screen.  Restart of Homebridge process shows that there is an unhandled exception somewhere.
To Reproduce:
This is on initial configuration.
Expected behavior:
Expected to see vehicles show on Home screen.
Logs:
(node:2716) UnhandledPromiseRejectionWarning: RequestError: write EPROTO<PHONE_NUMBER>:error:141A318A:SSL routines:tls_process_ske_dhe:dh key too small:../deps/openssl/openssl/ssl/statem/statem_clnt.c:2158:
at ClientRequest.<anonymous> (/usr/lib/node_modules/homebridge-hyundai-bluelink/node_modules/got/source/request-as-event-emitter.js:178:14)
at Object.onceWrapper (events.js:422:26)
at ClientRequest.emit (events.js:327:22)
at ClientRequest.origin.emit (/usr/lib/node_modules/homebridge-hyundai-bluelink/node_modules/@szmarczak/http-timer/source/index.js:37:11)
at TLSSocket.socketErrorListener (_http_client.js:469:9)
at TLSSocket.emit (events.js:315:20)
at emitErrorNT (internal/streams/destroy.js:106:8)
at emitErrorCloseNT (internal/streams/destroy.js:74:3)
at processTicksAndRejections (internal/process/task_queues.js:80:21)

(Use node --trace-warnings ... to show where the warning was created)
(node:2716) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). To terminate the node process on unhandled promise rejection, use the CLI flag --unhandled-rejections=strict (see https://nodejs.org/api/cli.html#cli_unhandled_rejections_mode). (rejection id: 1)
(node:2716) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.
Plugin Config:
{
"bridge": {
"name": "Homebridge 0E45",
"username": "REDACTED",
"port": 51390,
"pin": "REDACTED"
},
"accessories": [],
"platforms": [
{
"name": "Config",
"port": 8581,
"auth": "form",
"theme": "auto",
"tempUnits": "f",
"lang": "auto",
"platform": "config"
},
{
"refreshToken": "REDACTED",
"platform": "Ring"
},
{
"platform": "Garageio",
"name": "Garageio",
"username": "REDACTED",
"password": "REDACTED"
},
{
"credentials": {
"username": "REDACTED",
"password": "REDACTED",
"region": "US",
"pin": "XXXX"
},
"vehicles": [
{
"vin": "REDACTED",
"maxRange": 5
},
{
"vin": "REDACTED",
"maxRange": 5
}
],
"remoteStart": {
"cool": false,
"heat": true,
"defrost": true,
"igniOnDuration": 10
},
"refreshInterval": 20,
"platform": "Hyundai"
}
]
}
Screenshots:

Environment:

Plugin Version: 1.2.4
Homebridge Version:  1.1.7
Node.js Version:  14.15.4
NPM Version:  7.5.2
Operating System:  Raspbian GNU/Linux 10 (buster)

Thanks for the kind words @scottdlowe!
I think this might be the same as https://github.com/athal7/homebridge-hyundai-bluelink/issues/1, I should probably put something about it in the Readme though

Thanks for the kind words @scottdlowe!
I think this might be the same as #1, I should probably put something about it in the Readme though

I made the change suggested in the link - setting SSL SECLEVEL to 1 and it worked.  Not ideal, but I understand that it's on the Hyundai side.  Thank you for the response and, again, your work on this.  It's great!

By the way, if you need anyone to help test in real time, feel free to let me know.  Happy to help where I can.

Thanks for the offer! The Hyundai API has been a bit more finicky than expected (especially over this weekend), so I think adding some good tests are my next best move to keep on top of the increase in complexity.
Collaboration on or review of those certainly would be welcome.

Question - does using the HomeKit plugin do anything to the Hyundai account in general?  We're not able to remote start either of our cars anymore and not sure if Hyundai sees requests via the API and turns it off or if it's something else.

I don’t see a reason why it would affect that, but I was seeing what looked
like rate limiting issues before the most recent release which contributed
to similar behavior to what you are describing.
On Wed, Feb 10, 2021 at 1:29 PM scottdlowe<EMAIL_ADDRESS>wrote:

Question - does using the HomeKit plugin do anything to the Hyundai
account in general? We're not able to remote start either of our cars
anymore and not sure if Hyundai sees requests via the API and turns it off
or if it's something else.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/athal7/homebridge-hyundai-bluelink/issues/4#issuecomment-776918835,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AADSHIEM2N3DKFOR5MVZCTDS6LGCNANCNFSM4XG6BYYA
.

--
Andrew Thal
athal7.com

If there is a limit of 200 calls per day, the 20 second refresh interval would blow through that in 63 minutes. One of the forums says it resets at midnight. I changed my refresh interval to something much higher and will see what tomorrow brings.

Oh! Can you link me to where you found 200 calls per day? Agree something
should be done about refresh if that’s the case. Curious how your
experiment goes!
On Wed, Feb 10, 2021 at 11:30 PM scottdlowe<EMAIL_ADDRESS>wrote:

If there is a limit of 200 calls per day, the 20 second refresh interval
would blow through that in 63 minutes. One of the forums says it resets at
midnight. I changed my refresh interval to something much higher and will
see what tomorrow brings.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/athal7/homebridge-hyundai-bluelink/issues/4#issuecomment-777196279,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AADSHIH4RA22J553EJB5J7DS6NMPPANCNFSM4XG6BYYA
.

--
Andrew Thal
athal7.com

Ok, so it wasn't a documented rate limit on the API, but a comment here: https://forum.abetterrouteplanner.com/topic/1682-access-hyundai-bluelink-data-via-webapi/.

Thanks @scottdlowe, I like the refresh interval strategy in that thread, maybe will try to implement here.
Were you able to start the car first thing in the morning? If not, I'm thinking it may be worth filing an issue with bluelinky since the start interface is being used as documented.


❓ Integrate MLKit to swift in IOS for FrameProcessor Plugin

Question
after finishing the implementation of faceDetector on android, it's ios' turn, after looking between swift and Objective-C I decided to use swift which seemed easier, since I've never worked with either of them.
I have a problem that as silly as it seems took me a few hours and I still can't solve it, I did the implementation of s.dependency GoogleMLKit/FaceDetection in my vision-camera-face-detector.podspec, and I see the mlkit dependencies being installed by cocoapods, but when I go to compile my example project that uses frameprocessor, I always run into this error;

everything seems right to me since the mlkit documentation itself redirects me to this example that does the import in the same way
I've already noticed the other plugins created for vision camera but they all use objective-C and the example you have of swift is a code example only that doesn't show an implementation of dependencies.
to look in my poc project just look at this repo
Cheers.
VisionCamera Version
^2.6.1
Additional information

[ ] I am using Expo
[X] I have read the Troubleshooting Guide
[X] I agree to follow this project's Code of Conduct
[X] I searched for similar questions in the issues page as well as in the discussions page and found none.

 What happens when you do that?

sure, all pods are installed normally including mlkit dependencies

A workaround would be to add pod GoogleMLKit/FaceDetection to your example/ios/Podfile, but that shouldn't be needed... 🤔

that doesn't work too

seems like iam not the only one experiencing this.
https://stackoverflow.com/questions/67363271/missing-required-module-mlkit-error-while-using-a-framework-that-uses-mlkit

solved.
for some reason, google quickstart project of mlkit using swift, and other fonts on internet uses
import MLKit
when the correct way should be
import MLKitFaceDetection import MLKitVision
maybe it's a undocumented recent changes.

Try this https://www.youtube.com/watch?v=mv4c2uTxR4U


Inference transformer issue with sequence length >1024

Hi,
I'm trying to use DeepSpeedTransformerInference, and am getting the following error whenever the sequence length is higher than 1024:
RuntimeError: CUDA error: too many resources requested for launch
For sequence lengths up to 1024 it works great, including with much larger batch sizes.
This is some minimal code which reproduces the error:
import torch
from deepspeed.ops.transformer.inference.transformer_inference import DeepSpeedInferenceConfig, DeepSpeedTransformerInference

hidden = 512
batch = 1
seqlen = 2048

config = DeepSpeedInferenceConfig(
    hidden_size=hidden,
    heads=32,
    fp16=True,
    num_hidden_layers=8)

layers = [DeepSpeedTransformerInference(config).half().cuda().requires_grad_(False) for i in range(8)]

inputs = torch.rand([batch, seqlen, hidden], device="cuda", dtype=torch.half)
input_mask = torch.ones([batch, seqlen], device="cuda", dtype=torch.half)
x = inputs
for i, layer in enumerate(layers):
    print(i)
    x = layer(x, input_mask=input_mask)

Thanks

Hi @opherlieber
Thanks for reporting this.
I will work on this to solve the issue soon. By the way, can you tell me what model architecture is this for? I see you are just creating the layers on your own with some random dimensions, or is it related to some model specification?
Thanks,
Reza

Thanks a lot for the support!
The dimensions here are just for the example, I have various architectures I'd like to try and apply this too.
Can you also maybe give some info about the 'specialized_mode' ? It looks like it fuses/optimizes the ops for the case of decoding with past K/V values, but It seems I'm missing the 'InferenceSpecializedBuilder' to get it enabled. Is there any plan to release this, and do you have any numbers about how much it helps?
Thanks

Hi @opherlieber,
I believe this PR  should fix the issue you were seeing with larger sequence-length.
The specialized kernels are the high-performance version of the inference kernels that fuse the operations at deeper level, such as combining all the attention part in just one kernel as you mentioned (you can find more information in our inference-blog). The benefit of performance improvement varies based on network's architectures/dimensions (50% to 2x speedup over the current version of the inference kernels). We are working on a plan to release the code in near future.
Thanks,
Reza

Works great, thanks!


Freezing Build on MacOS via yarn watch

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: 1.92.2
OS Version: MacOS Sonoma 14.5 (MacBook Pro 16 Apple M1 Pro 16GB RAM)

Steps to Reproduce:

Fork repo & clone
Try to build with yarn watch (looking at this doc) Build section

Here's where it froze and stopped progressing:

Any ideas what might be the issue?

The second/third try is the same, I left it hanging for 2 hours and nothing has advanced, weird

It was my bad, issue resolved.


support inputs other than .wav

Do you think you might automate the conversion to .wav and input into whisper.cpp? Since ffmpeg can also be installed you could take the step off the user to do the conversion separately and then feed your GUI the .wav to reduce errors.

@Baenwort took some time but now whispercppGUI uses ffmpeg to automatically convert the input file to the WAV file whispercpp requires. Thanks for your feedback.


fp = open(temp, 'r') FileNotFoundError: [Errno 2] No such file or directory:

Traceback (most recent call last):
File "YOUR_FILE.py", line 29, in
lines = lsd(gray)
File "C:\YOUR_WORKING_DIRECTORY\pylsd_repo\pylsd\lsd.py", line 24, in lsd
fp = open(temp, 'r')
FileNotFoundError: [Errno 2] No such file or directory: 'C:/YOUR_WORKING_DIRECTORY/536135ntl.txt'

Using Windows, if you installed it using source code, may be this is happening to you. I just faced it and resolved it using a quick (its nasty but works) fix.

open lsd.py and put a print(temp) just when temp has been created, this gives the path where temp file is created
A file would be created every time at this location, but the name if not as per expected, due to some reasons. (EXPECTED: randomno+ntl.txt, ACTUAL: C - in my case)
just replace temp by this new path(location of file+C) at two places - a) where it is read again b) in remove
Done, should work!

@getsanjeev great work!
Sorrowly, it does not work for Linux. Probably, you have any ideas for it too?

Yes it does not work on Linux, I would update for Linux, let me figure out something.

Just replave 15th line and 20th line in pylsd/pylsd/lsd.py as follows:
# temp = os.path.abspath(str(np.random.randint(1, 1000000)) + 'ntl.txt').replace('\\', '/')
temp = os.path.abspath(str(np.random.randint(1, 1000000)) + 'ntl.txt').replace('\\', '/').encode('utf-8')
# lsdlib.lsdGet(src, ctypes.c_int(rows), ctypes.c_int(cols), temp)
lsdlib.lsdGet(src, ctypes.c_int(rows), ctypes.c_int(cols), ctypes.c_char_p(temp))

I upgraded the lsd algorithm version from 1.5 to 1.6, and solved the FileNotFoundError. Now it works on python3.
You can install it with pip install pylsd2


[Migration] URL Redirects

We have to migrate the URL Redirects from Drupal to WP

Unfortunately the URL redirects are not correctly imported from Drupal to WP. We have to develop our own script to do the mgiration.


.append file in README is wrong

Currently if you use the code from the README to append a readable stream, it fails with it being unable to read property name of null.
form.append('my_file', fs.createReadStream('/foo/bar.jpg'), 'bar.jpg');

The options argument is actually mandatory and needs to be either a string or an object with a filename property, or a name or path property for the readable stream. The code is a little confusing, and can be found line 217 of form_data.js
Updating the readme would be nice, thanks.

Apparently, my error was elsewhere (see #416) so I'm not too sure if this is still needed. In which case debugging this took me a while, so now that it works I don't really wish to experiment further with it.


ModelFieldImage shows a black bg when PNG file is chosen

ModelFieldImage shows a black background when selecting a PNG file. In ImageService, the PNG file is passed as a jpeg type in data URL.

Specifically, a PNG with transparency. I think the canvas need a background color filled in with a default color.

Yup, but in ImageService line 54, the data is always returned as a flat jpeg file so there's no way to fill the background color.


Java interop 'new' issue in instarepl

Java's new keyword or its equivalent reader syntax the '.' operator is not working.  Whereas the same is working in the on clojure's repl (lein repl). I'm using clojure 1.6.0 for lein repl
Light Table version
0.7.2
Binary version
0.8.4
Clojure Plugin 0.1.1 (Kodowa)
Below is the screenshot

Confirmed defect on atom-shell branch.
From my experimentation, it appears to be an issue when destructuring in the function arguments, but it works fine in a let destructuring:
(defn bad-destructuring
  [{:keys [date] :or {date (java.util.Date.)}}]
  (println "bad:" date))

(defn good-destructuring
  [input]
  (let [{:keys [date] :or {date (java.util.Date.)}} input]
    (println "good:" date)))

@sashton Thanks for isolating this.
I was also able to reproduce this in atom-shell on Windows 7.

I think I've tracked it down to lein-light-nrepl.
With the following form evaluated in Instarepl:
(defn bad-destructuring   [{:keys [date] :or {date (new java.util.Date)}}]
   (println "bad:" date))

The lighttable.nrepl.sonar/defn++  macro generates the following:
(clojure.core/defn bad-destructuring [{:keys [date], :or {date (new java.util.Date)}}]
  (clojure.core/swap! lighttable.nrepl.sonar/*caps* clojure.core/merge (lighttable.nrepl.sonar/restructure (quote [date date new java.util.Date]) [date date new java.util.Date]))
  nil
  (println bad: date))

I'm guessing the call to (restructure '[date date new java.util.Date] [date date new java.util.Date]) is the source of the problem. It doesn't look like it handles destructuring correctly.
I'm not sure what the fix is yet. I'm still trying to understand what *caps* is and what defn++ is doing.

@sashton Thanks for looking into this.

This issue has moved to https://github.com/LightTable/ClojureInstarepl/issues/13 as the instarepl is now its own plugin

This issue has moved to https://github.com/LightTable/ClojureInstarepl/issues/13 as the instarepl is now its own plugin


Investigate a possibility to run tests for webview-based applications.

Links:

https://testcafe-discuss.devexpress.com/t/testing-ionic-phonegap-apps-with-testcafe/979

Thanks - would love to see this!

Any updates? Have someone try to connect to testcafe remote from a webview?

One more private request - https://www.devexpress.com/Support/Center/Question/Details/T735149/test-cordova-app-with-apk-file-in-testcafe-studio

+1 request
https://www.devexpress.com/Support/Center/Question/Details/T748324/setting-up-test-for-all-our-cordova-apps-in-teststudio

👋 Hey, just wondering, would this cover running tests in Facebook's Webview?

Hi @rob4629
To give you a precise answer regarding the support for Facebook's Webview, we need to research the possibility of running tests for webview-based applications.
As soon as we perform such investigation, we will create a list of supported webview-based application types: mobile, smart tv, facebook webview, etc.

Today I've tried to run testcafe on iOS install app that using cordova.
I've got connect error 500 from server and response : "There are no available _connection to establish".
Testcafe write a message: "ERROR The Safari 13.1.2 / iOS 13.7 browser disconnected. This problem may appear when a browser hangs or is closed, or due to network issues."
In safari works good.
Do you have any updates to work with webview?

Hi @ilchenkoalexey
There is no update yet. As we will get any results, we will update this thread.

We decided to not implement this enhancement and focus on the TestCafe features.


Add vcr for connect test

Coverage remained the same at 13.406% when pulling 1cf458ac58aea45bfe79fb2d715268ad1e7bd8c3 on pragmaticivan:connect_test into f1651dd60e5dc652d12f7cd85aa13476d8b76cdd on robconery:master.


Backup AWS ElectrumX instances

When DogeCoin AWS instance will finish synchronization, we will need to do a backup for the data.
All AWS new ElectrumX instances need to be backed up.

Everything except Doge is done.  Dogecoin is still syncing

Done


Fix sphinx-gallery links

sphinx-gallery has moved from readthedocs to sphinx-gallery.github.io some time ago.

thanks for catching @lesteve !


Azure Tables client library: why is there no example of authenticating using Managed Identity?

I thought a major benefit of the new libraries using Azure.Core was that there would be more standardisation in the ways of accessing Azure resources securely.
Other libraries (e.g. for Key Vault or Azure Storage Blobs or Queues) allow one to use Managed Identity e.g. using DefaultAzureCredential(), but apparently not in the Azure Tables library.  If not, why not and will this be introduced soon?
A major issue for us with Managed Identity has been the lack of a consistent approach across various Azure resources, so it would be good if Azure Tables also supported the same approach as Key Vault, and Azure Storage Blobs/Queues please.

Thank you for your feedback.  Tagging and routing to the team member best able to assist.

@patrickjlee Currently the service doesn't support Azure AD authentication. If you'd like to provide this feedback to the team that would need to implement this, please file an issue here https://feedback.azure.com/forums/263030-azure-cosmos-db .


Correct SP Used To Revert Workspace MP

Correct the SP Used to revert workspace MP.

Specifically, the emissions delete workspace deletes any emissions for locations that belong to the MP, but some of those emissions could belong to another MP for some quarters.
Additionally, the delete of QA_CERT_EVENT_SUPP_DATA is in the QA delete workspace although it is an emissions file with a FK to QA_CERT_EVENT, which is a QA file.

Programming Changes:

delete_monitor_plan_emissions_data_from_workspace: Currently the procedure deletes any emissions for locations associated with the specified MP.  It should only delete them for the RPT_PERIOD_ID of EMISSION_EVALUATION rows with the MP.
delete_monitor_plan_qa_data_from_workspace: Includes the delete for QA_CERT_EVENT_SUPP_DATA, which is an emissions table.  QA_CERT_EVENT_SUPP_DATA has an FK to QA_CERT_EVENT, but that FK has a cascade delete, so if the QA_CERT_EVENT is deleted first, the QA_CERT_EVENT_SUPP_DATA will also be deleted.  So, this delete should move to the delete_monitor_plan_emissions_data_from_workspace procedure.

Need acceptance criteria.

Question regarding camdecmpswks.delete_monitor_plan_data_from_workspace. I only can find SP camdecmpswks.delete_monitor_plan_data_from_workspace(text), not camdecmpswks.delete_monitor_plan_data_from_workspace(text, text)

Operating Supplemental Data Changes
Operating Supplemental Data (OSD) is produced for a specific Emission Report, MP and quarter, but the following are true.

Operating Supplemental Data (OSD) is produced for a specific Emission Reports (EM), and therefore a specific MPs and quarters.
OSD for a submitted EM still exists in the Workspace even when the EM does not.
OSD for unsubmitted EM exists in the Workspace if the EM was evaluated when an open submission window existed.

OSD for these EM may exist in Official if the EM was previously submitted.

Rules for Operating Supplemental Data when reverting an MP.

The deletion of Workspace Supplemental Data should only include data directly related to the MP being reverted.

The MON_LOC_ID is associated with the Workspace MP.
The RPT_PERIOD_ID is associated with an Workspace EMISSION_EVALUATION row that has the MON_PLAN_ID of the MP.
I believe this is handle correctly in the AY versions of the deletes.

The copy of Official Supplemental Data into the Workspace should only insert Supplemental Data directly related to the MP.

The MON_LOC_ID is associated with the Official MP.
The RPT_PERIOD_ID is associated with an Official EMISSION_EVALUATION row that has the MON_PLAN_ID of the MP.
This should wipeout any OSD for MP that were only in the Workspace.
Currently, I believe the copy attempts to copy any OSD associated with Monitor Location in the MP regardless of whether the EM was associated with the MP.

Need to determine how the initial load of the Workspace would be affected by this.

Deletion of Emissions Update
The original description of changes to delete_monitor_plan_emissions_data_from_workspace indicated to use the camdecmpswks schema's EMISSION_EVALUATION table to determine whether an MP is associated with emissions for a reporting period.  It should have indicated to use both the CAMDECMPSWKS and CAMDECMPS tables for that determination.
Note that Operating Supplemental Data Changes comment indicates that the "copy" should use only the CAMDECMPS EMISSION_EVALUATION table, which is correct.  I have updated Move Copy of QA_CERT_EVENT_SUPP_DATA comment to also explicitly indicate the CAMDECMPS table.

With the latest requests, I have worked on the following  5 SPs (One has no change, the rest  have prefix  "AY_" for now for easy review)
copy_monitor_plan_to_workspace (no change)
copy_monitor_plan_emissions_data_to_workspace
copy_monitor_plan_qa_data_to_workspace
delete_monitor_plan_data_from_workspace
delete_monitor_plan_emissions_data_from_workspace
delete_monitor_plan_qa_data_from_workspace
revert_to_official_record


Change to link the main doc to the right Spotfire API

We detected an issue when using wrapper with multiple screens and the new version of Spotfire 12. This resulted in the wrong screen (div element) responding in an open page request. The reason for this is that the _doc element in the spotfire-webplayer class did not bind to the correct SpotfireAPI object.
I have added a test to check the id of the Spotfire API element before the binding, so the correct SpotfireAPI object binds to the _doc element, which makes the correct screen react again.

Thanks Hugo !


Remove unnecessary interface and add remount prop to boxGeometry

Included in https://github.com/toxicFork/react-three-renderer-fiber/pull/66.


Added AnimCurve and AnimData back to API Docs

Fixes #4829
Removed @ignore from AnimCurve and AnimData to fix missing links from AnimTrack
I confirm I have read the contributing guidelines and signed the Contributor License Agreement.

I'm approving as it looks like these classes should have always been public. Thanks for the PR! :)


default to none compression

This happens at the init stage for now so that CLI calls go from "uncompressed" -> None.
In the future we might consider making this a bit less funky: using None meaningfully here is a bit odd compared to it being unspecified elsewhere... but the compression argument in pyarrow doesn't take uncompressed it takes None, so we would need to do a whole getter | setter thing for compression where we store "uncompressed" for all of them but then when we get it for writing we swap to None. Not sure if it's worth all of that, so let's kick that decision down the road.

This causes problems for me because I want to use the compression in the repo file if it is unspecified on the cmdline. But after these changes it is no longer clear whether nothing was specified or if the user gave an uncompressed flag

This causes problems for me because I want to use the compression in the repo file if it is unspecified on the cmdline

I don't think we really want to reference the repo file as the default see more discussion at https://github.com/conbench/datalogistik/pull/95#discussion_r978621118 for some thoughts on that

Ok then let's continue that discussion over there.
But this change I've reverted in #89


Crash when using VK_KHR_MAINTENANCE2_EXTENSION_NAME

My application initializes this extension: VK_KHR_MAINTENANCE2_EXTENSION_NAME, but then Renderdoc crashes my app.

VK_KHR_maintenance2 isn't supported by RenderDoc yet, so I'm not sure what exactly is crashing but this isn't too surprising.
Since it's a device extension you should find that when RenderDoc is attached, VK_KHR_maintenance2 is not enumerated as a supported extension - your code should check that the extension is available before enabling it.

Cool, I'll fix my code to detect availability before initializing this extension.  Thanks!

I've implemented support now, although I didn't really have any test cases so there might be problems.
Let me know if you run into any issues.


Preprocessing of docstrings can be improved

Hello,
first thanks for the challenge, the code and the dataset! Really cool stuff that you're doing and I want to work on this task. :)
I've read the Contribution Guidelines and know that you will not change any of the preprocessing code, but nevertheless I want to discuss the preprocessing of the docstrings here in case someone wants to produce a similar dataset (or maybe v3 ;) ) .
Current preprocessing of docstrings
I read your code and it seems that this is the way you preprocess the docstrings:

You extract the documentation from the method and strip c-style delimiters https://github.com/github/CodeSearchNet/blob/9356b3181eaa9d4a38df4d309018158fd23448cb/function_parser/function_parser/parsers/commentutils.py#L1
Then you extract the relevant part of the docstring (which acts as a summary) using the following heuristics:
2.1 If \n\n is found you take the part before
2.2 otherwise take the part before the first @
2.3. otherwise take the full docstring  https://github.com/github/CodeSearchNet/blob/9356b3181eaa9d4a38df4d309018158fd23448cb/function_parser/function_parser/parsers/commentutils.py#L18-L24
Finally you tokenize the extracted summary using the following regex:
https://github.com/github/CodeSearchNet/blob/9356b3181eaa9d4a38df4d309018158fd23448cb/function_parser/function_parser/parsers/language_parser.py#L5

Problems with this pipeline
This way of preprocessing produces a couple of results that are probably not wanted and could be improved.
Extracting the summary
Compare the first 12 lines of the tokenized docstrings of the java train set to the raw ones
Bind indexed elements to the supplied collection .
Set {
Add {
Set servlet names that the filter will be registered against . This will replace any previously specified servlet names .
Add servlet names for the filter .
Set the URL patterns that the filter will be registered against . This will replace any previously specified URL patterns .
Add URL patterns as defined in the Servlet specification that the filter will be registered against .
Convenience method to {
Configure registration settings . Subclasses can override this method to perform additional configuration if required .
Create a nested {
Create a nested {
Create a nested {

As you can see 6/12 are basically not usable.
**Bind indexed elements to the supplied collection.** @param name the name of the property to bind @param target the target bindable @param elementBinder the binder to use for elements @param aggregateType the aggregate type, may be a collection or an array @param elementType the element type @param result the destination for results
**Set** {@link **ServletRegistrationBean**}**s that the filter will be registered against.** @param servletRegistrationBeans the Servlet registration beans
**Add** {@link **ServletRegistrationBean**}**s for the filter.** @param servletRegistrationBeans the servlet registration beans to add @see #setServletRegistrationBeans
**Set servlet names that the filter will be registered against. This will replace any previously specified servlet names.** @param servletNames the servlet names @see #setServletRegistrationBeans @see #setUrlPatterns
**Add servlet names for the filter.** @param servletNames the servlet names to add
**Set the URL patterns that the filter will be registered against. This will replace any previously specified URL patterns.** @param urlPatterns the URL patterns @see #setServletRegistrationBeans @see #setServletNames
**Add URL patterns, as defined in the Servlet specification, that the filter will be registered against.** @param urlPatterns the URL patterns
**Convenience method to** {@link **#setDispatcherTypes(EnumSet) set dispatcher types**} **using the specified elements.** @param first the first dispatcher type @param rest additional dispatcher types
**Configure registration settings. Subclasses can override this method to perform additional configuration if required.** @param registration the registration
**Create a nested** {@link **DependencyCustomizer**} **that only applies if any of the specified class names are not on the class path.** @param classNames the class names to test @return a nested {@link DependencyCustomizer}
**Create a nested** {@link **DependencyCustomizer**} **that only applies if all of the specified class names are not on the class path.** @param classNames the class names to test @return a nested {@link DependencyCustomizer}
**Create a nested** {@link **DependencyCustomizer**} **that only applies if the specified paths are on the class path.** @param paths the paths to test @return a nested {@link DependencyCustomizer}

However, the relevant information is in the raw docstrings (I added the ** to highlight relevant passages). Simply using the part before the first @ produces pretty bad results (at least in java) as its common practice to highlight code blocks or links with javadoc-tags.
Possible solution: Stripping everything before the first param (or maybe @param) and afterwards removing javadoc-tags (maybe keeping the tokens inside).
Cleaning
The preprocessing does not include any cleaning. This manifests in docstrings that contain html-tags (which are commonly found in javadoc), as well as URLs (which afterwards get pretty verbose tokenized). See these two samples from java
Determine if a uri is in origin - form according to  rfc7230 5 . 3 .
Determine if a uri is in asterisk - form according to  rfc7230 5 . 3 .

Some stats
>>> wc -l java.train.comment
454436 java.train.comment
>>> grep -E '
|
' java.train.comment | wc -l
42750

At least 10% of the tokenized java docstrings still contain html tags.
>>> grep '{ @' java.train.comment | wc -l
44500

Another 10% still contain javadoc.
>>> grep "{ @inheritDoc }" java.train.comment | wc -l
1685

2k consist only of a single javadoc-tag indication that the doc was inherited.
Many of the golang docstrings contain URLs, which are not very useful in the tokenized version the regex produces (see above in java).
>>> wc -l go.train.comment
317822 go.train.comment
>>> grep -E 'http :|https :' go.train.comment | wc -l
19753

~6% contain URLs (starting with http :)
>>> grep -E "autogenerated|auto generated" go.train.comment | wc -l
4850

Around 5k auto generated methods
>>> grep "/ *" go.train.comment | wc -l
33620

10% still contain c-style comment delimiters.
Tokenization
Any specific reason you keep punctuation symbols like ., ,, -, /, :, <, >, *, =, @, (, ) as tokens? Is it to keep code in the docstrings?
Summary
I really think better cleaning and a language-dependent preprocessing would produce higher quality docstrings At least for java a removal of javadoc and html could be beneficial.  As well as using everything before the first param  as  a summary  (maybe in combination with the first paragraph \n\n heuristic).

Thanks for the feature request and suggestions.  This project is currently in maintenance mode, but we are happy to take a pull request.
Thanks

maintenance mode
Yet in maintenance mode? What kind of improvements can we expect?

No improvements.  This project is no longer being maintained

No improvements. This project is no longer being maintained

Ok. Then, what about publishing the relevance judgment used to calculate NDCG?

Hello,
first thanks for the challenge, the code and the dataset! Really cool stuff that you're doing and I want to work on this task. :)
I've read the Contribution Guidelines and know that you will not change any of the preprocessing code, but nevertheless I want to discuss the preprocessing of the docstrings here in case someone wants to produce a similar dataset (or maybe v3 ;) ) .
Current preprocessing of docstrings
I read your code and it seems that this is the way you preprocess the docstrings:

You extract the documentation from the method and strip c-style delimiters https://github.com/github/CodeSearchNet/blob/9356b3181eaa9d4a38df4d309018158fd23448cb/function_parser/function_parser/parsers/commentutils.py#L1
Then you extract the relevant part of the docstring (which acts as a summary) using the following heuristics:
2.1 If \n\n is found you take the part before
2.2 otherwise take the part before the first @
2.3. otherwise take the full docstring  https://github.com/github/CodeSearchNet/blob/9356b3181eaa9d4a38df4d309018158fd23448cb/function_parser/function_parser/parsers/commentutils.py#L18-L24
Finally you tokenize the extracted summary using the following regex:
https://github.com/github/CodeSearchNet/blob/9356b3181eaa9d4a38df4d309018158fd23448cb/function_parser/function_parser/parsers/language_parser.py#L5

Problems with this pipeline
This way of preprocessing produces a couple of results that are probably not wanted and could be improved.
Extracting the summary
Compare the first 12 lines of the tokenized docstrings of the java train set to the raw ones
Bind indexed elements to the supplied collection .
Set {
Add {
Set servlet names that the filter will be registered against . This will replace any previously specified servlet names .
Add servlet names for the filter .
Set the URL patterns that the filter will be registered against . This will replace any previously specified URL patterns .
Add URL patterns as defined in the Servlet specification that the filter will be registered against .
Convenience method to {
Configure registration settings . Subclasses can override this method to perform additional configuration if required .
Create a nested {
Create a nested {
Create a nested {

As you can see 6/12 are basically not usable.
**Bind indexed elements to the supplied collection.** @param name the name of the property to bind @param target the target bindable @param elementBinder the binder to use for elements @param aggregateType the aggregate type, may be a collection or an array @param elementType the element type @param result the destination for results
**Set** {@link **ServletRegistrationBean**}**s that the filter will be registered against.** @param servletRegistrationBeans the Servlet registration beans
**Add** {@link **ServletRegistrationBean**}**s for the filter.** @param servletRegistrationBeans the servlet registration beans to add @see #setServletRegistrationBeans
**Set servlet names that the filter will be registered against. This will replace any previously specified servlet names.** @param servletNames the servlet names @see #setServletRegistrationBeans @see #setUrlPatterns
**Add servlet names for the filter.** @param servletNames the servlet names to add
**Set the URL patterns that the filter will be registered against. This will replace any previously specified URL patterns.** @param urlPatterns the URL patterns @see #setServletRegistrationBeans @see #setServletNames
**Add URL patterns, as defined in the Servlet specification, that the filter will be registered against.** @param urlPatterns the URL patterns
**Convenience method to** {@link **#setDispatcherTypes(EnumSet) set dispatcher types**} **using the specified elements.** @param first the first dispatcher type @param rest additional dispatcher types
**Configure registration settings. Subclasses can override this method to perform additional configuration if required.** @param registration the registration
**Create a nested** {@link **DependencyCustomizer**} **that only applies if any of the specified class names are not on the class path.** @param classNames the class names to test @return a nested {@link DependencyCustomizer}
**Create a nested** {@link **DependencyCustomizer**} **that only applies if all of the specified class names are not on the class path.** @param classNames the class names to test @return a nested {@link DependencyCustomizer}
**Create a nested** {@link **DependencyCustomizer**} **that only applies if the specified paths are on the class path.** @param paths the paths to test @return a nested {@link DependencyCustomizer}

However, the relevant information is in the raw docstrings (I added the ** to highlight relevant passages). Simply using the part before the first @ produces pretty bad results (at least in java) as its common practice to highlight code blocks or links with javadoc-tags.
Possible solution: Stripping everything before the first param (or maybe @param) and afterwards removing javadoc-tags (maybe keeping the tokens inside).
Cleaning
The preprocessing does not include any cleaning. This manifests in docstrings that contain html-tags (which are commonly found in javadoc), as well as URLs (which afterwards get pretty verbose tokenized). See these two samples from java
Determine if a uri is in origin - form according to  rfc7230 5 . 3 .
Determine if a uri is in asterisk - form according to  rfc7230 5 . 3 .

Some stats
>>> wc -l java.train.comment
454436 java.train.comment
>>> grep -E '
|
' java.train.comment | wc -l
42750

At least 10% of the tokenized java docstrings still contain html tags.
>>> grep '{ @' java.train.comment | wc -l
44500

Another 10% still contain javadoc.
>>> grep "{ @inheritDoc }" java.train.comment | wc -l
1685

2k consist only of a single javadoc-tag indication that the doc was inherited.
Many of the golang docstrings contain URLs, which are not very useful in the tokenized version the regex produces (see above in java).
>>> wc -l go.train.comment
317822 go.train.comment
>>> grep -E 'http :|https :' go.train.comment | wc -l
19753

~6% contain URLs (starting with http :)
>>> grep -E "autogenerated|auto generated" go.train.comment | wc -l
4850

Around 5k auto generated methods
>>> grep "/ *" go.train.comment | wc -l
33620

10% still contain c-style comment delimiters.
Tokenization
Any specific reason you keep punctuation symbols like ., ,, -, /, :, <, >, *, =, @, (, ) as tokens? Is it to keep code in the docstrings?
Summary
I really think better cleaning and a language-dependent preprocessing would produce higher quality docstrings At least for java a removal of javadoc and html could be beneficial. As well as using everything before the first param as a summary (maybe in combination with the first paragraph \n\n heuristic).

Have you improved the preprocessing of docstrings？Can you share the url？


Link the libsmartconfig.a failed

............
............
make[1]: Leaving directory '/work/xwifi/esp-idf/examples/07_smartconfig/build/main'
LD smartconfig-demo.elf
/work/xwifi/esp-idf/examples/07_smartconfig/build/main/libmain.a(app_main.o):(.literal.smartconfig_done+0x28): undefined reference to esp_smartconfig_stop' /work/xwifi/esp-idf/examples/07_smartconfig/build/main/libmain.a(app_main.o):(.literal.smartconfig_task+0x8): undefined reference to esp_smartconfig_start'
/work/xwifi/esp-idf/examples/07_smartconfig/build/main/libmain.a(app_main.o):(.literal.smartconfig_init+0x8): undefined reference to esp_smartconfig_set_type' /work/xwifi/esp-idf/examples/07_smartconfig/build/main/libmain.a(app_main.o): In function smartconfig_done':
/work/xwifi/esp-idf/examples/07_smartconfig/main/./app_main.c:65: undefined reference to esp_smartconfig_stop' /work/xwifi/esp-idf/examples/07_smartconfig/build/main/libmain.a(app_main.o): In function smartconfig_task':
/work/xwifi/esp-idf/examples/07_smartconfig/main/./app_main.c:65: undefined reference to esp_smartconfig_start' /work/xwifi/esp-idf/examples/07_smartconfig/build/main/libmain.a(app_main.o): In function smartconfig_init':
/work/xwifi/esp-idf/examples/07_smartconfig/main/./app_main.c:93: undefined reference to `esp_smartconfig_set_type'
collect2: error: ld returned 1 exit status
/work/xwifi/espidf/make/project.mk:214: recipe for target '/work/xwifi/esp-idf/examples/07_smartconfig/build/smartconfig-demo.elf' failed
make: *** [/work/xwifi/esp-idf/examples/07_smartconfig/build/smartconfig-demo.elf] Error 1
.............

I'm not able to reproduce the issue here. Can you please upload output of the following commands to a gist and post link here?
git status
git submodule foreach 'git status'
make clean
make V=1

Thanks.

I have pushed the example to : https://github.com/icamgo/esp-idf/tree/noduino/examples/07_smartconfig/
Using the latest esp-idf, esp32/lib is (should be the latest one):
comcat@jackslab:/work/xwifi/esp-idf/components/esp32/lib$ git status
HEAD detached at b3090d8
nothing to commit, working directory clean

I am able to build smartconfig branch with the latest master without errors.
Please post the output of the commands mentioned above in a gist :)

https://gist.github.com/icamgo/9e297e4b1aad0d838b619e77e75a86c0

Your IDF_PATH points to /work/xwifi/espidf, while you are running everything in /work/xwifi/esp-idf. You may need to export IDF_PATH=/work/xwifi/esp-idf.

This is .... Build system can compute the IDF_PATH automaticlly. Something like the implemetation of the build system in our esp8266/esp8285 sdk enviroment: http://github.com/icamgo/noduino-sdk

This is .... Build system can compute the IDF_PATH automaticlly. Something like the implemetation of the build system in our esp8266/esp8285 sdk enviroment: http://github.com/icamgo/noduino-sdk

This is indeed possible for examples which are in-tree, using relative paths (like you do here). However if the example is copied to a random directory outside of ESP-IDF tree, this relative path will no longer work. The primary use case for ESP-IDF is to be used with applications developed outside of ESP-IDF tree, so for this reason we suggest that developers set IDF_PATH either in the environment, or in application makefile.


Mgirgis Merge master to stable1.5

recheck

recheck

I'm also throwing a do-not-merge flag on this because it is including the entire set of changes targeting STF 1.5.4. If this is just for testing what needs to be pulled in to get stable-1.5 tested that's fine, but please do not merge this into stable-1.5, because it will pull all the future items into the stable branch before we're ready for that.

I'm also throwing a do-not-merge flag on this because it is including the entire set of changes targeting STF 1.5.4. If this is just for testing what needs to be pulled in to get stable-1.5 tested that's fine, but please do not merge this into stable-1.5, because it will pull all the future items into the stable branch before we're ready for that.

This includes changes to zuul configs that will allow us to run against stable-1.5. We're going to revaluate the contents of this as we prepare the release. With this PR, mgirgis has discovered the additions tweaks needed to get the jobs running on stable-1.5.

@elfiesmelfie @mgirgisf is this required any more or can we simply close this out?

@elfiesmelfie @mgirgisf is this required any more or can we simply close this out?

Not required. IIRC, this was a temporary branch for scoping the Zuul updates required to make sure the jobs ran on stable-1.5

I have closed this PR, but will require someone to delete the branch. I don't want to delete it just in case.


Name-style check configurations

As I now want to use the name-style checks of this library (since they give more/better options already), a few questions have came up.
I try to use it through sumneko LuaLS (both VSCode and CLI), but I have now also tried the EmmyLuaCodeStyle VSCode Extension.
Configuration of e.g. whitespaces, so code-style, works fine and the same for both versions.

But is there some configuration missing for name-style checks? I can turn it on for the Emmy extension, but it won't apply my settings for it. I want to use camel-case, it diagnoses for snake-case.
LuaLS doesn't give the warnings for names at all but doesn't seem to have a configuration as well?

What I tried to configure in .editorconfig:
enable_name_style_check = true

local_name_define_style = camel_case
function_name_define_style = camel_case
local_function_name_define_style = camel_case
global_variable_name_define_style = upper_snake_case

I have noticed that the docs for the name-style checks are only in the "old" documentation. But I tried it like it's described there.

as you can see, I didn't give you how to configure namestyle, I refactored all the code after the 1.0.0 update, but I forgot to write the code for namestyle to read from the configuration.
on the other hand, I felt that name-stylecheck should not be configured with editorconfig, and I planned to reprogram a configuration file, but I didn't think of a good name, so I never did it

Okay so as far as I understand the LuaStyle (=code-style) has to parse the .editorconfig itself, while using json should make that a little bit more... compact. And LuaDiagnosticStyle =name-style) is mainly missing this configuration parsing. A json library is also already in the repository.
If that's correct so far, I could write a small prototype using .luarc.json which should be found at project root as well. And we could discuss from there.
As for the LuaLS diagnostic code (which as a first step doesn't necessarily need the configuration) - I would start by taking the https://github.com/LuaLS/lua-language-server/blob/master/script/core/diagnostics/codestyle-check.lua and adapt from there. Or does activating name-style need a different approach then code-style?

luaStyle is simply created from map, not editorconfig file, editorconfig file is read by luaEditorconfig.
so luaDiagnosticStyle is also created by maps or other structures, not .luarc .
the entire CodeService does not rely on the JSON library, which is used for CodeFormatServer.
LuaLs is calling CodeFormatLib for service, if you want to get the configuration from .luarc, you should pass in the Map via CodeFormatLib and parse out LuaDiagnosticStyle

You don't need to do the work anymore, I'm implementing related work, naming style checking will support regular expressions and basic nomenclature.

Oh thank you very much for the work, and thanks for letting me know!
I've figured out the LuaLS diagnostic in the meantime - did or do you implement that as well, or should I just contribute that?

Loosely related issue I've just come across:
The start and finish of diagnosticInfos given by the name_style_analysis do not include the last character of the offending name if it is longer than one character (and it looks like it looking correct with one character names is some special handling as well)
For example:
local NAME = "test"

the namestyle returns start = 6 and end = 9 (resulting in only NAM being marked) , but other diagnostics in LuaLS would give start = 6 and end = 10 to correctly mark the name.
Also, the EmmyLuaCodeStyle VSCode extension does correctly mark the whole name.
The problem in LuaLS can be fixed by just increasing the end-index by 1 in the diagnostic for all cases (one-character names are still marked correctly, as well as all longer names). But before doing that, I'd like to clarify if that discrepancy is intended and should in fact be fixed in the LuaLS diagnostic, or if it is rather a bug and is better changed in the CodeFormatLib?

It is design, my design is [start, end] not [start, end), and my language server can converts. you can add 1 to end-index

I fix the question by myself. I am working for luaLs adaptor. you can do some translation work for me

Yes sure, just let me know where exactly you can use my help.

I have complete all, see https://github.com/LuaLS/lua-language-server/pull/2091

It may be difficult to integrate this part into LuaLs, my own plugin has been released, you can try it

Nice work! I've now tried the EmmyLuaCodeStyle VSCode Extension directly, as well as the implementation of your LuaLS PR in my dev environment.
I really like the way it works. It also works as expected in both plugins if the configuration is done in settings.json, the only thing I'm missing is that for the LuaLS it doesn't apply the configuration when I put it into luarc.json instead (where I actually enable the name-style-check diagnostic in my setup).


(BKR-1162) Remove vsphere hypervisor to beaker-vmware

Matching beaker-vmware PR

Refer to this link for build results (access rights to CI server needed):
http://jenkins-beaker.delivery.puppetlabs.net//job/qe_beaker_btc-intn/4020/

Refer to this link for build results (access rights to CI server needed):
http://jenkins-beaker.delivery.puppetlabs.net//job/qe_beaker_btc-intn/4021/

@puppetlabs-jenkins retest this please

Refer to this link for build results (access rights to CI server needed):
http://jenkins-beaker.delivery.puppetlabs.net//job/qe_beaker_btc-intn/4023/

@puppetlabs-jenkins retest this please

Refer to this link for build results (access rights to CI server needed):
http://jenkins-beaker.delivery.puppetlabs.net//job/qe_beaker_btc-intn/4024/


dask.array.nanmean() generates unstable name (hash)

Code Sample
import dask.array as da
import numpy as np

x = da.ones((5, 5), chunks=(2, 2))
x = da.nanmean(x, axis=0)
print(x.name)

Problem Description
Running the above sample three times outputs:
mean_agg-aggregate-8157871966f12f849b40951c495fef6f
mean_agg-aggregate-c6939e3b5eb3290f17252b895e10c7c3
mean_agg-aggregate-17e2ac663f1cdddf5b6fc02b1ad0b889

which shows the dask name (hash) is unstable.
Expected Output
For processing large datasets, it's convenient to have a stable hash name in order to save intermediate results or compare them among developers.

Workaround: use dask.array.mean() instead of dask.array.nanmean()

Please included how you created your test environment (e.g. conda env export if using conda).

In order to get those results, I created a fresh virtual environment based on virtualenvwrapper and installed dask using pip, as follows:
$ mkvirtualenv dask_test
$ pip install dask[complete]

Here (Python 3.5.2):
click==6.7
cloudpickle==0.5.2
dask==0.16.0
distributed==1.20.2
HeapDict==1.0.0
locket==0.2.0
msgpack-python==0.4.8
numpy==1.13.3
pandas==0.21.0
partd==0.3.8
psutil==5.4.2
python-dateutil==2.6.1
pytz==2017.3
six==1.11.0
sortedcontainers==1.5.9
tblib==1.3.2
toolz==0.8.2
tornado==4.5.2
zict==0.1.3

Thanks for the excellent bug report, this should be fixed in #3024.
import dask
import dask.array as da

x = da.ones((5, 5), chunks=(5, 5))

x2 = da.nanmean(x, axis=0)
print(x2.name)
(dask) jcrist dask $ python test.py
mean_agg-aggregate-307d3de24a463578cdbedbb88a1b7461
(dask) jcrist dask $ python test.py
mean_agg-aggregate-307d3de24a463578cdbedbb88a1b7461


sfMelodyPlugin and sfGuardUserProfile

Look at this http://forum.symfony-project.org/viewtopic.php?f=12&t=33956
I have same trouble. In propel version of sfGuardPlugin, there aren't such fields as first_name or email_adress.
Normally i use sf_guard_user_profile table for such data:
App.yml:

all:
  sf_guard_plugin:
   profile_class:      sfGuardUserProfile
   profile_field_name: sf_guard_user_id

But sfMelodyPlugin not work with the profile, plugin try to use setters of sfGuardUser object only. There is some patch of that problem. Some key in app.yml explains to sfMelody to look for profile setters if it cant't find in sfGuardUser.
For example, my app.yml.

all:
  sf_guard_plugin:
   profile_class:      sfGuardUserProfile
   profile_field_name: sf_guard_user_id
  melody:
    use_profile: true
.....
        first_name:
          call: me
          path: first_name
        last_name:
          call: me
          path: last_name
        nick:
          call: me
          path: username
        ext_id:
          call: me
          path: id
        ext_link:
          call: me
          path: link

After registration i get 2 new tuples in DB:

select * from sf_guard_user;
 id |      username       | algorithm |     created_at      |     last_login      | is_active | is_super_admin |       email
----+---------------------+-----------+---------------------+---------------------+-----------+----------------+-------------------
 19 | Facebook_1464279882 | sha1      | 2012-02-08 13:15:14 | 2012-02-08 13:15:14 | t         | f              |<EMAIL_ADDRESS>(1 row)

select * from sf_guard_user_profile;
 sf_guard_user_id | first_name | last_name |  nick   |   ext_id   |            ext_link
------------------+------------+-----------+---------+------------+---------------------------------
               19 | Maksim     | Ustinov   | madesst |<PHONE_NUMBER> | http://www.facebook.com/madesst
(1 row)

Code is very stupid, but it works for now.
I hope this will be useful.

I've commited improvments in sfMelodyUserFactory, BasesfMelodyActions for working with register_redirect: true.
Do you need any comments about this pull request?


Remove input manager in tooltip plugin and make them back to nodes

For now, the tooltip plugin includes both button manager which controls a list of button, and input manager which shows input for image, link and inline math. However, modify content of link, image, inline math should be part of the node's ability. So I plan to move them back to preset-commonmark and math-plugin. And organize the input style by theme manager.

Done


Markdown body content missing when accessing project pages through direct link

I have noticed that when I try to access a project page through the direct link (e.g. https://mat.qmul.ac.uk/projects/garden-of-forking-paths) it does not display the text defined in the body of the related markdown file. It only appears to use the frontmatter content (title, description, video etc.) which suggests that there might be an issue with the GraphQL query?
I might have a go at this sometime in future, if no one else has a good idea why this is happening?

Good catch @SFRL. Looks like this: https://github.com/gatsbyjs/gatsby/issues/10581. Will try to solve it tomorrow, or otherwise probably next weekend.

Yep, don't wrap an element that generates content from markdown, like the following example, with
 tags:


Add fusionreport data to rnafusion test-datasets

https://github.com/nf-core/rnafusion/issues/556

yes


Channels example throws fatal error

What did you do? If possible, provide a simple script for reproducing the error.

Ran the quick start example on the Coroutine > Channels page

What did you expect to see?

I expected to see it work since it is a straight up copy from the docs

What did you see instead?

[FATAL ERROR]: all coroutines (count: 1) are asleep - deadlock!

What version of OpenSwoole are you using (show your php --ri openswoole)?

openswoole

Open Swoole => enabled
Author => Open Swoole Group<EMAIL_ADDRESS>Version => 4.11.1
Built => May 10 2022 09:02:14
coroutine => enabled with boost asm context
kqueue => enabled
rwlock => enabled
sockets => enabled
openssl => OpenSSL 1.1.1o  3 May 2022
dtls => enabled
http2 => enabled
json => enabled
curl-native => enabled
pcre => enabled
zlib => 1.2.11
brotli => E16777225/D16777225
mysqlnd => enabled
async_redis => enabled

Directive => Local Value => Master Value
swoole.enable_coroutine => On => On
swoole.enable_library => On => On
swoole.enable_preemptive_scheduler => Off => Off
swoole.display_errors => On => On
swoole.use_shortname => On => On
swoole.unixsock_buffer_size => 262144 => 262144

What is your machine environment used (show your uname -a & php -v & gcc -v) ?

uname -a & php -v & gcc -v
[1] 19508
[2] 19509
Darwin Okans-Work-2020 21.4.0 Darwin Kernel Version 21.4.0: Fri Mar 18 00:45:05 PDT 2022; root:xnu-8020.101.4~15/RELEASE_X86_64 x86_64
[1]  - 19508 done       uname -a
Configured with: --prefix=/Library/Developer/CommandLineTools/usr --with-gxx-include-dir=/Library/Developer/CommandLineTools/SDKs/MacOSX.sdk/usr/include/c++/4.2.1
Apple clang version 12.0.0 (clang-1<IP_ADDRESS>)
Target: x86_64-apple-darwin21.4.0
Thread model: posix
InstalledDir: /Library/Developer/CommandLineTools/usr/bin
➜  openswoole PHP 8.1.5 (cli) (built: Apr 16 2022 00:14:18) (NTS)
Copyright (c) The PHP Group
Zend Engine v4.1.5, Copyright (c) Zend Technologies
    with Zend OPcache v8.1.5, Copyright (c), by Zend Technologies

[2]  + 19509 done       php -v

You can also try the following OpenSwoole support channels:

Documentation - Documentation for Open Swoole
Slack - Slack channel of Open Swoole
Discord - Discord server of Open Swoole

@ODelibalta Can you paste the code?


Collision layers and masks don't work same as they do in Godot

Right now, in box2d, you can set a collision mask and category field.
In godot, you can do same, but in godot, if you have the following case:

object
mask
category

object A
1
2

object B
2
2

Then object B will interact with object A, but object A will not interact with object B. That is, object A will always continue it's path, as if no restitution is given to it after the collision.
This is a little bit difficult to do in box2d. Need to investigate more on how to do it.

This isn't really an issue, as it's a very weird interaction. I will write it as a limitation in the readme, and close this issue.


feat: automatically include implementers of interfaces in schema

Description

Types of Changes

[ ] Core
[ ] Bugfix
[ ] New feature
[ ] Enhancement/optimization
[ ] Documentation

Issues Fixed or Closed by This PR

Checklist

[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[ ] I have tested the changes and verified that they work and don't break anything (as well as I can manage).

Summary by Sourcery
Automatically include implementers of interfaces in the schema by updating the schema creation process to account for interface implementers after the schema is created. Add tests to ensure this functionality works as expected.
New Features:

Automatically include implementers of interfaces in the schema without requiring extra types to be explicitly defined.

Tests:

Add tests to verify that implementers of interfaces are automatically included in the schema without extra types.

Hi, thanks for contributing to Strawberry 🍓!
We noticed that this PR is missing a RELEASE.md file. We use that to automatically do releases here on GitHub and, most importantly, to PyPI!
So as soon as this PR is merged, a release will be made 🚀.
Here's an example of RELEASE.md:
Release type: patch

Description of the changes, ideally with some examples, if adding a new feature.

Release type can be one of patch, minor or major. We use semver, so make sure to pick the appropriate type. If in doubt feel free to ask :)
Here's the tweet text:
🆕 Release (next) is out! Thanks to Erik Wrede for the PR 👏

Get it here 👉 https://strawberry.rocks/release/(next)


outFilter option

Added outFilter option to apply ProGuard filters to output jar.

Looks ok to me (didn't test).

So...what's next? Can you merge it and release it?

Sorry, this is a pure background project. @wvengen Do you have time to realesa?

No problem. Thanks!

I'll put releasing it on my todo list. Thanks!


Colors are not shown on Razer devices

Colors that are send from Overwatch to Razer Chroma SDK / Razer Chroma Client are not shown on any Razer devices after Overlights installation. The only place where the colors are actually shown is only Overlights UI. Are there any possibility for the color events to be consumed both by this app and Razer Choma devices at the same time?
Could it be because of the compatibility issues with different DLL and Razer Chroma SDK (Razer Client, Razer Synapse etc) versions? If so, then could you provide the version of the SDK / Synapse / DLL that you used for creating your RzChromaSDK64.dll? Thanks.

This is correct. Since we directly patch the dlls we lose the ability to page out to razer devices. For now, this is not a goal of this project - that said, it's definitely possible to also call out to the original razer dlls to send data to the keyboard as well, and i'd happily take a PR there.


Write network ID in deployment script

This PR modifies the deployment script so that it doesn't rely on a hardcoded network ID. This allows us to use more test networks that don't necessarily have an option for configuring network IDs (like Hardhat).
Test Plan
E2E test no longer uses deterministic network ID, and should continue to pass.

Codecov Report

Merging #1195 (90acc51) into main (f831779) will decrease coverage by 0.00%.
The diff coverage is 48.00%.

@@            Coverage Diff             @@
##             main    #1195      +/-   ##
==========================================
- Coverage   67.26%   67.25%   -0.01%
==========================================
  Files         135      135
  Lines       27189    27209      +20
==========================================
+ Hits        18288    18299      +11
- Misses       8901     8910       +9


Installation Guide

Add installation steps to the documentation.

https://code4recovery.org/docs/tsml-ui