getAllApplications

@mbeier-tgm please implement /getAllApplications[?user=]
This should be structured like the following:
Input:

optional: user id

Output:

Array of all application elements
example:
[
{
id: 1234,
title: "Skiwoche",
leader: "Stefan Zakall",
edate: "2018-01-12",
status: "Angenommen",  (Could be "Angenommen", "Abgelehnt", "In Bearbeitung")
active: false
},
{
...
}
]

If user id is specified only return all applications from the specific user

should this endpoint also return applications in which the specified user is only a participant and not a leader?

should this endpoint also return applications in which the specified user is only a participant and not a leader?

Yes, all Applications in which the user is leader or participant should be returned

implemented a method in the database connector to further implement this issue as an endpoint in c60f0a6

Final implementation in a102f35
This returns all applications if a security token is provided and the user requesting has the needed permissions (AV, Administration or PEK). Also a filter (short name as username) of a teacher might be provided. This will then return all applications of the provided teacher. When a filter is applied a teacher might lookup their own applications permission wise


Update to the latest version of ADIOS2

Whether or not we do this hinges on what the outcome of #19796 is.


feat: add incremental append and merge for iceberg tables

Now Iceberg tables support incremental models with append (for both v2 and v3) and merge (v3 only).
I added the specifications in the README.md.
I am also agree with a comment made in an other PR about splitting the incremental sql file according to dbt-core standard. Should we do this in this PR or in an other one ?

@Jrmyy nice work, please rebase and fix the conflict, then we are ready to ship.


Screens for CatalogueItems (CRUD)

[x] Read
[x] Add to Cart (POST to ~/CartItems)
[x] Create (postponed)
[x] Update (postponed)
[ ] Delete (postponed)

Creating and saving catalog items creates often error messages but without valuable information
-> better validation and information of user


CB-9049 Add list filtering with authorization

Everyone starts reviewing can get a good understanding of the implementation by reading the authorization-common module's README.md file's list filtering section.

More testing and real ums test configuration is needed. I'll add that later.

Filtered items:

environments,
datalakes,
datahubs,
freeipas,
credentials,
image catalogs,
recipes.


implement CAST(col as type)

Closes #398

@jussisaurio Please drop testing/testing.db-wal from the PR.


Divide(split) video to parts

how i can divide video to parts, like that each parts should be 15 minute?
this code divide but  not all part for 15 minute. My video duration  is 40 minute,
and code divide to parts: 15, 18,8  minutes
FFmpegBuilder builder = new FFmpegBuilder()
     .setInput(file.getAbsolutePath())
     .overrideOutputFiles(true)
     .addOutput(file.getName() + "%03d.mp4")
     .setFormat("mp4")
     .setAudioChannels(1)
     .setAudioCodec("aac")
     .setAudioSampleRate(48_000)
     .setAudioBitRate(32768)
     .setVideoFrameRate(24, 1)
     .addExtraArgs("-map", "0")
     .addExtraArgs("-f", "segment")
     .addExtraArgs("-segment_time", "15")
     .addExtraArgs("-reset_timestamps", "1")
     .setDuration(15, TimeUnit.MINUTES)
     .setStrict(FFmpegBuilder.Strict.EXPERIMENTAL)
     .done();

PS: Sorry for my English

Do you have an example of how to do this with the plain ffmpeg command line? If so I can tell you how to use the java API to do it.

hello, @bramp this question means something like this I guess (for simple usage)
ffmpeg -i test.mp4 -f segment -segment_time 3 -c copy out%03d.mp4


user with EDIT access can undo PUBLIC access but cannot re-institute it.

@dtarb @pkdash @hyi
This quandary arose when testing new code.

User Alice creates a resource and makes it public, and gives edit access to User Bob.
User Bob deletes a file from the resource..
This invalidates public access because the file was required. The resource is set to private.
Bob adds back another version of the file, making the resource possible to make public.
Bob cannot set the resource to public, because Bob is not an owner.

Worse, this is the way we expect Bob to work. So edit users can't safely modify public resources.

I think that we have to accept this as a limitation.  We could implement warnings, but I propose we do not at this stage change the functionality.
Here is what I suggest

Implement a warning that if the last file is being deleted that the resource will revert to private and an owner will then need to set it to public.  Indicate in the warning that if the file is to be replaced, first adding the replacement file, then deleting the old file will retain the public setting

Rationale

If Bob inadvertently ends up making the resource public, it is not that much trouble for them to contact the owner to have them reset it to public.  Owners and editors are supposed to be people that trust each other so working together.
The work around above is functional
We do want a resource that is required to have content, without any content to be private.

@dtarb The way we have done things, it is impractical to implement a pre-check. We won't know that the resource reverts to private until we delete the file and then run the validator. We don't have the concept of a pre-action validation.
There are other possible solutions, including recording the "intent to make public" (which is different from the actual public flag). If one sets 'make_public_if_possible', then when Bob uploads a file that passes, the system takes over and acts on prior instructions of an owner to make something public.

Indicate in the warning that if the file is to be replaced, first adding the replacement file, then deleting the old file will retain the public setting

In some of the specific resource types, it won't be possible to add a replacement file before deleting the existing file. For example, if a time series resource already contains a SQLite file, it won't be possible to add another SQLite file.

The problem is that -- regardless of what we do to pre-validate -- we have to do something different for every resource type.  Lots of work. Hence "impractical".

I am trying to get an understanding of the requirements for a file pre delete check function:
If we had a pre-delete-file-check() which says deleting this file will set the resource to private what we (system) suppose to do with that information?

We already have a message "Confirm files deletion".  Lets just change this to stronger wording.
"This file will be permanently deleted.  Consider saving a copy if it is important to you.  If this is the last file in the resource and it is public, the sharing status will revert to private.  If you are not the owner of this resource, then an owner will need to reset this to public after a new file has been added.  If you want to replace this file, add the new file first then delete the old one, so that sharing status does not change."
The same problem occurs with deleting the last keyword.  We could add additional explanation to that message.
"Note: Deleting all keywords will set the resource sharing status to private.  If you are not the owner of this resource, then an owner will need to reset this to public after a new keyword has been added.  To avoid changing the sharing status add new keywords before deleting existing ones."
Regarding the concerns @pkdash raised about some resources only having one file, deleting the one and only file is essentially deleting the resource.  A new file added to the same container is essentially a new resource, so I am OK with it becoming private until the owner makes it public.  I think this is really a social issue for editors and owners and I do not see this as a major problem.  I see it as a relatively low priority minor inconvenience, and I am not sure pre and post change checks and balances are going to make it easier for a user, even if we do take the time, which appears considerable, to work it out.

Assigned to @pkdash to change messages per above.  No change to functionality or policy at this time.

@dtarb @pkdash The "last file" condition is not the only condition under which a resource will become private. A resource with a set of shapefiles will not pass tests for public if some of the files are missing.
This becomes more serious when people are "replacing" files. At present, this is done by deleting a file and uploading another. Because of the way validations are written, this turns off public if the resource without the file does not pass inspection!

@alvacouch Agreed. File delete and file upload are two separate actions and not part of single transaction. The issue is that under the current implementation the system sets the public to private on file delete but it lacks the capability to set private to public when the resource has the required content files. However, If we keep provenance of resource flag (may be just a resource level bool type attribute private_set_by_system) , then the system should be able to set private to public when the file gets added if that's what we want.

While I have been reluctant to add additional complexity, I think some of the concerns raised here could be solved with a "public pending" sharing attribute.  This would be set in cases where public is intended, but where content (data files or metadata) does not pass validation rules.  Public pending resources would behave exactly as private, but instead of being labeled private would be labeled public pending on the landing page, and as soon as the content passes validation rules, they become public.  If content, through editing, results in failing validation, they become public pending.  Public pending would not be an attribute that an owner sets (remember, it is only owners that can set these sharing attributes).  Rather an owner would set public, and then the state public or public pending would be the result of rule validation.  Public pending would make the intent to be public transparent and automatic for editors, rather than reverting to private.

@dtarb I like this "public pending" approach which makes a lot of sense to me, and does not appear to me to involve a lot of work and complexity to implement. What do you think @pkdash @alvacouch ?

Reviewing this as part of the old issue clean up, I would like to implement the warning message approach.  @pkdash can we work together on this once your workload on model aggregations subsides.

Message to add to confirm files deletion box if the file is the last one being deleted and is going to revert sharing status to private.  “Deleting this file will set the resource sharing status to private.”

adding @dtarb as this pertains to Access and Control.


[TypeScript] Add support to skip import type

In TypeScript 3.8 the new feature of import type has been introduced but Madge does not handle it. As we use the syntax import type { Foo } from 'bar' we can skip the import on import type keyword.
But note that this won't solve the problems of regular export of types and imported as modules.
I can write a PR for this if needed.

See https://github.com/pahen/madge#how-to-ignore-import-in-type-annotations-in-typescript


fix dependencies for command_execution

Signed-off-by: Victor Drobny<EMAIL_ADDRESS>

Description of the Change
Fix dependencies for command_execution target

The purpose of this pr is to fix command_execution target, not chain_validator


Error 404 and 403 on big update

Gitea version (or commit ref): Gitea version 1.4.2 built with: bindata, sqlite (from binaries)
Operating system: debian 9.4
Git version: git version 2.11.0
Database:

[ ] PostgreSQL
[x] MySQL
[ ] MSSQL
[ ] SQLite

Can you reproduce the bug at https://try.gitea.io:

[ ] Yes (provide example URL)
[ ] No
[x] Not relevant

Log gist:

2 kinds of error:
[...routers/repo/view.go:295 Home()] [E] Home: Cannot find any unit on this repository which you are allowed to access
[...ules/context/repo.go:691 func1()] [E] CheckUnit: You are not allowed to visit this repository unit: 1

Description
I migrated from Gogs a time ago with Gitea version 1.1.2 built with: bindata, sqlite and switched back to gogs afterwards for a moment.
Now, I want to fully migrate to Gitea, so, I reactivated this gitea which works well.
Then, I tried to upgrade to Gitea version 1.4.2 built with: bindata, sqlite by replacing the old binaries with the new one. Everything seems to work well for new repo but all the old ones gives 404 or 403.
My guess is this comes from the database ; as if some migrations were not done. Do I need to upgrade from version to version (like 1.1.2 -> 1.1.3 -> 1.1.4 -> 1.2.0 -> etc)?
Screenshots

Yes, when upgrading from gogs upgrade path should be gogs -> 1.0.2 -> 1.1.4 -> 1.2.3 -> 1.3.4 -> 1.4.2
This error can be related to: https://github.com/go-gitea/gitea/issues/1794#issuecomment-348717495

Alright, first of all, it must be 1.3.3 instead of 1.3.4 (which doesn't exist).
But anyway, from 1.1.4 (which works) to 1.2.3, I got the same error. My test repository goes 404 :(

OK, running https://github.com/go-gitea/gitea/issues/1794#issuecomment-347831784 made it works on 1.2.3. Continuing the migration process…

Alright, other migrations went well afterwards. I have to repo with error 500 which is not associated with this ticket.
My question here is still relevant. It would have save me a lot of time…

No there is not but it would be great if you could update gogs -> gitea migration manual in docs and submit PR, any help with docs are really appreciated.


Add option to change percent bar for current player

Add option to change percent bar for current player so that you can more easily see yourself in the party list.

This is now live.


Add Requested By to Imaging

Fixes #362.
Changes proposed in this pull request:

Adds Requested By to Imaging Requests and Completed lists
Refactors Imaging to use components in blocks
Adds Add Note link to the Requests list items that don't have notes
Adds No Notes and No Results to Completed list items that don't have either Notes or Results

Add Notes link:

No Notes and No Results

cc @HospitalRun/core-maintainers

@cacqw7 thanks for the PR!
A couple of things:

Can you change your code to use localization for the text displayed?  More details about localization using ember-i18n can be found here: https://github.com/jamesarosen/ember-i18n/wiki/Doc:-Translating-Text
Can you provide screenshots of the change you made.

@jkleinsc done and done!

@cacqw7 thanks for the update!
@jglovier Any objections/comments on the UI changes for this PR?

Right now there is a lot of wrapping of very short fields. It'd be better if we could shorten up the Notes field (maybe max-width: 300px?) and give space for the other fields to not wrap so easily.

Sorry, entered that last comment before finished...

I'm not a fan of the "No results" and "No notes" placeholder text because it's redundant and unnecessary, and adds visual noise. If there are no notes to display, I think the absence of them is clue enough to that.
I like surfacing the Add notes link, but I'm curious to know more of your intentions there. Does it only appear on hover, or visible by default on every item without notes? I'd be opposed to adding it on every single item w/o notes for the same visual noise concerns as above, even if it is more useful

Unfortunately some of this just comes back to incomplete work on how actions are surfaced on items in the first place. I don't want to block too much here on account of that, but those are important concerns to resolve before we go too far with implementing more pieces that will be addressed by that.
Thanks for jumping in here @cacqw7! :sparkles:

@jglovier
I'll go ahead and remove No Results, No Notes and the Add Notes. Those were all outside the scope of #362 and I agree that there are more important things to resolve first.
I'll work on the margins this weekend and push up changes/screenshots 👍 😄

@cacqw7 awesome. :metal: As we get closer to resolving the surfacing actions in index views piece, I'll rope you back in and we can revisit these pieces under the new paradigm.

@jglovier I added the min-width you suggested.

I added the min-width you suggested.

@cacqw7 that looks much better. I'd love to add a min-width for the patient name, too. But that's probably more of a global concern that we can address in another PR.

@jkleinsc I removed those classes 👍

Looks good @cacqw7.  I am going to merge it in.


[Question/ENH] Is stratefied/custom batching available?

Per title, maybe I missed it somewhere but I cannot seem to find this.
One scenario is dealing with highly class imbalanced dataset, so there's a big chance that many batches only contain one class, which disables metrics like AUC that often require two classes in preds and labels in one batch.
Any thoughts?

If it's only for metric, you can change the update function such that it stores result on update call and only start computing when more than one classes are available.

@piiswrong Got it. I'll try it. Thanks!

That solution is ok but it's better to have a way to do stratified batching, or at least make this part customizable. Any thoughts? Would this be easy given current structure?

Hmm, I think making some further twists in update function would be enough to achieve what I want. Closing this now. @piiswrong Thanks again.


Wayland Zoom in/out types +/- when vscode is in the background and some other app with wayland natural zoom capabilities in the background

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

VS Code Version: 1.69.2
OS Version: Pop!_OS 22.04

Note: The team can actually ignore this bug since it's not a big deal and prolly don't even affect 5% of the vscode users out there since the issue is reproducible (partially) only on laptops running linux on Wayland display protocol. Still reporting this as my responsibility.
This issue can only be reproduced in wayland sessions. Also that it doesn't always occur
Steps to Reproduce:

Open VSCode and some file
Open Chrome with ozone platform as Wayland and try pinch natural zoom in out or using laptop touchpad
Instead of the expected behaviour of web page zoom, the WIndow manager switches to vscode and types + for pinch in gesture and - for pinch out (or the vice versa) untill I lift my fingers. Lifting the fingers and retrying finger gestures of vscode actually changes the text size/ scaling in vscode which is expected. But if i switch back to chrome and retry those same gestures, the bug is most of the time reproducible.

Browser gestures work fine after closing vscode from the background.
This behaviour doesnt also occur if browser is open in a different workspace and vscode in another

Does the issue repro with other Electron apps like Slack or Teams in the background ? Seems like a generic issue to the Electron runtime.

Yep I've seen it occur with other electron apps too

Unable to repro with the steps provided on Ubuntu 22.04 using latest stable 1.74, are you also running VSCode with --ozone-platform=wayland --enable-features=WaylandWindowDecorations ?

No I'm not using those options


Enable Docker Support checkbox for Blazor Server apps and gRPC apps.

Summary of the changes

Enable "Enable Docker Support" checkbox for Blazor Server apps.
Enable "Enable Docker Support" checkbox for gRPC apps.

Addresses #938514 And #938373.

I've updated your original comment with links to the AZDO issues you seem to have been trying to mention.
Could you give us some more context here? What does it mean to "Support Docker" in this case? I would have guessed that that means there's a docker file which the template can use, and we don't have such a file. If what it means is that VS can create a docker-file for you, what would even by the case under which a template DOESN'T support docker?
@danroth27 for the experience and background here.

Hi @ryanbrandenburg ,
The changes are related to the following dialogue box -

The Blazor apps and gRPC apps support Docker.
Blazor Server apps and gRPC apps have been tested with Docker and they work as expected.

@pratiksanglikar what I'm asking is what the difference between a template which supports docker and a template without is. If @danroth27 says it's fine I'm sure it is, but I'd like to understand how and why.

@ryanbrandenburg Sorry my mistake, the image link in my earlier comment doesn't work, but here it is - here so when a template supports docker, the checkbox in the image is enabled or if the template doesn't support docker, then the checkbox is disabled.
That checkbox gives ability to enable docker support while creating the project instead of adding the docker support through context menu after the project is created.
Does that answer your question?

I think I'm being a bit unclear, what I'm trying to ask is how I, as a template author, know if my template "supports docker". That is, if I'm creating a new template and I get to that property in the .json and I ask myself "Should this property be true or false?" how do I answer that question? Surely if the property exists then there must be templates for which the answer is false, and I'm trying to learn how to tell the difference.

Okay, I got your question now.
So, any .NET Core web project (which supports http/https) should work with docker as long as there are no new dependencies.
There is no fixed  formula as such which tells certainly if a project works with Docker.
Any project can be tested if it works with Docker by -

Create the project
From the right click context menu of the project, select Add Docker Support.
Test F5 in Debug and Release mode.

If it works, it is safe to add that flag in in the template.

I'm closing this PR to open a new one due to this merge commit -
https://github.com/aspnet/AspNetCore/commit/f6942804faab30faf7ae2b8165ee7138f1a685ae.

#13016 is the new PR.

Thanks, that's good to know for the future.


Layer selection (and multi-selection) in the Layer Panel

Click or Shift-click layers in the layer panel (and change their color so they appear selected) and notify the Editor backend.
Complexity: 1
Involves: Frontend (JS), Editor (Rust)

Assigned to @akshaykalbhor.

Sorry for the delay. Started working on this!

Glad to hear it! It looks like you had to comment in this issue before I was able to actually assign the GitHub issue to you (that's why I resorted to commenting about that).

Weird

Layers normally have a #555555 background color. Selected layers can be given a #777777 background color by adding a selected class to the selected elements (through Vue's :class attribute) and adding a .selected {...} CSS rule.

Good point! Noted

Status?


[WebExt] proxy/webRequest, Fx 71 tasks

Trello

Acceptance criteria

[x] Remove "ip" from proxy.json

[x] Remove proxy.register/unregister

Changes made for Remove "ip" from proxy.json to proxy.RequestDetails and pinged Shane Caraveo for review. Also queried whether urlClassification, which is included in the proxy.json definition, needs to be added to proxy.RequestDetails?

@chrisdavidmills could I get some advice for Remove proxy.register/unregister: we have notes in proxy and each of the API pages about the removal of proxy.register/unregister and the compatibility data is updated. Given that these APIs will still be "available" at least until the ESR version goes beyond 71, should we be doing any more now?

@rebloor yeah, this looks sufficiently covered. Does this need any rel notes? If not, then I think we can consider this done.

@chrisdavidmills thanks, it probably does need a release note to be clear about the removal of proxy.register() and proxy.unregister(), and I have added one.


Access denied 403 when attempting to install this package.

What happened?
Pulumi plugin install is not working, because the URL https://get.pulumi.com/releases/plugins/pulumi-resource-aws-quickstart-vpc-v0.0.2-darwin-arm64.tar.gz is not accessible. (403 access denied).
Steps to reproduce
Create a typescript pulumi project and install the package:
$ npm install @pulumi/aws-quickstart-vpc

Run pulumi up and follow the instructions it gives about missing plugin aws-quickstart-vpc:

$ pulumi plugin install resource aws-quickstart-vpc v0.0.2

Expected Behavior
Installs the plugin so that the error no longer appears when running pulumi up.
Actual Behavior

Output of pulumi about
CLI
Version      3.37.2
Go Version   go1.17.12
Go Compiler  gc
Plugins
NAME    VERSION
nodejs  unknown
Host
OS       darwin
Version  12.5
Arch     arm64
This project is written in nodejs: executable='/Users/gsuess/.nvm/versions/node/v18.7.0/bin/node' version='v18.7.0'
Current Stack: Qualifyze/gsuess-dev
Found no resources associated with Qualifyze/gsuess-dev
Found no pending operations associated with Qualifyze/gsuess-dev
Backend
Name           pulumi.com
URL            https://app.pulumi.com/gsuess
User           gsuess
Organizations  gsuess, Qualifyze
Dependencies:
NAME                                    VERSION
@pulumi/aws-quickstart-aurora-postgres  0.0.3
@typescript-eslint/eslint-plugin        4.33.0
eslint-plugin-prettier                  3.4.1
husky                                   7.0.4
@changesets/changelog-github            0.4.6
@pulumi/aws-quickstart-vpc              0.0.2
jest                                    26.6.3
shx                                     0.3.4
ts-jest                                 26.5.6
ts-node                                 10.9.1
@changesets/cli                         2.24.3
@typescript-eslint/parser               4.33.0
eslint                                  7.32.0
lint-staged                             11.2.6
eslint-config-prettier                  8.5.0
is-ci                                   3.0.1
prettier                                2.7.1
typescript                              4.7.4
Pulumi locates its logs in /var/folders/_j/n3zrsq6n373c_18tsgvfs2yc0000gn/T/ by default
Additional context
No response
Contributing
Vote on this issue by adding a 👍 reaction.
To contribute a fix for this issue, leave a comment (and link to your pull request, if you've opened one already).

I am guessing that the problem is related to the fact that it's an arm64 architecture for which you are lacking builds.

Also getting this 😞 .  Thanks for raising @gsuess.
I'm using an Macbook Pro M1. Until this gets resolved I worked around this. However, not ideal and produces a warning.
warning: resource plugin aws-quickstart-vpc is expected to have version >=0.0.2, but has ; the wrong version may be on your path, or this may be a bug in the plugin

Use make to build for your platform
Init go mod
$ make

Build binary for your platform
$ make build

Move the binary to your plugins directory
$ mv ./pulumi-aws-quickstart-vpc/bin/pulumi-resource-aws-quickstart-vpc ~/.pulumi/plugins/resource-aws-quickstart-vpc-v0.0.2/

Hey folks, I've triggered a fresh build of this provider and it should produce a macOS binary for Apple Silicon.
As you have noticed, this repository is not under active maintenance, so I can't assert the fitness or quality of the release, but I will monitor our build and ensure that we're able to publish a plugin for your platform.

This should be fixed in v0.0.5.

Much appreciated @AaronFriel


Placeholder title when creating a PR does not behave like GitHub website

Issue Type: Bug
Repro steps:

Create a branch
Commit once
Push branch
Create a new pull request using the VS Code extension

Expected:

As there is only one commit, the first line of the commit becomes the placeholder of the PR title and the rest of lines are pasted in the description
This is what happens if you create the PR in the GH website

Actual:

The placeholder of the PR title is the branch name

Extension version: 2021.9.1985
VS Code version: Code 1.60.0 (Universal) (e7d7e9a9348e6a8cc8c03f877d39cb72e5dfb1ff, 2021-09-01T10:54:53.442Z)
OS version: Darwin x64 20.6.0
Restricted Mode: No

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-9750H CPU @ 2.60GHz (12 x 2600)

GPU Status
2d_canvas: enabledgpu_compositing: enabledmetal: disabled_offmultiple_raster_threads: enabled_onoop_rasterization: enabledopengl: enabled_onrasterization: enabledskia_renderer: disabled_off_okvideo_decode: enabledwebgl: enabledwebgl2: enabled

Load (avg)
3, 3, 3

Memory (System)
16.00GB (0.09GB free)

Process Argv
--crash-reporter-id 28bd1807-0bda-4011-b469-c3c4de71e6be

Screen Reader
no

VM
0%

A/B Experiments
vsliv368cf:30146710
vsreu685:30147344
python383:30185418
pythonvspyt678:30270856
pythonvspyt602:30300191
vspor879:30202332
vspor708:30202333
vspor363:30204092
pythonvspyt639:30300192
pythontb:30283811
pythonvspyt551:30345470
pythonptprofiler:30281270
vshan820:30294714
vstes263:30335439
vscorecescf:30358481
pythondataviewer:30285071
pythonvsuse255:30340121
vscod805:30301674
pythonvspyt200:30340761
vscextlang:30333561
binariesv615:30325510
pythonvssor306:30344512
bridge0708:30335490
pygetstartedt2:30360495
bridge0723:30353136
pythonf5test824cf:30361778
vsdyn478:30356813
pythonvspyt187:30362782
pydsgst2:30361792
vscus224cf:30358040
vssid140:30362080

I've been seeing this recently and I suspect it's due to #2846 (since that was the only change in the right timeframe).
@joshuaobrien any thoughts on what might have happened?

Thanks for the tag. I'll take a look @mtaran

I'm not seeing this now. @mcaballeropinto does your repo have a PR template?

Nop, it does not. This is issue is actually intermittent, I sometimes get the right title but others I get the branch name. I can't figure out how to repro it consistently.

Closing for now. If I see it again I'll re-open.

I'm seeing this now as well. (Spent awhile looking for the setting to fix before realizing the settings were removed)
I'm not sure which info is needed though. I first reproduced it with an unpublished branch but can also reproduce it with a random branch like: https://github.com/melink14/rikaikun/tree/melink14/xo
(Creating pull request in Github UI behaves as expected with title == commit)

Reopening for further investigation.

As a follow up, I realized that melink14/xo branch has two commits so it's not the best example. Though the behavior still differs from the Github default of using the first commit.
I went ahead an published the branch I was just using without opening a PR for easier testing:
https://github.com/melink14/rikaikun/tree/melink14/pure-esbuild
In this one as well, Create Pull Request in vscode gives Melink14/pure-esbuild while github gives title from commit.

+1!
I also still hit this pretty regularly. I only noticed that it's not especially consistent, but happens less on already-uploaded branches.
Would love to see this fixed, since it's a key part of one of the most common and high-impact uses of this extension.

One way I've been able to reproduce:
when creating a PR that targets a non-default branch, after switching to it the title and description should be updated, iff the new branch has a single commit only compared to the new "into" branch.
This is something that happens quite often when developing against an "unstable" branch, while the "stable" branch is set as the default.
Github automatically detects when switching into branches and fills in the title / description.

Verfied but opened https://github.com/microsoft/vscode-pull-request-github/issues/3357 as a result of testing this

Commenting here to preserve past context but let me know if opening a new issue would be preferred.
I'm still seeing this for about 50% of new PRs I create but I'm not sure exactly what could be causing it.
It happened when creating this PR most recently: https://github.com/melink14/rikaikun/pull/980
This branch was ontop of main with only one commit made via the vs code source control UI.
The output doesn't appear to have anything logged when creating a pull request; is there someway to turn on verbose logging and get output containing the reasons why it chose fallback PR title?

@melink14 I just pushed some additional logging around how we get the title. It will be available in the preview version of the extension tomorrow. If you want to help track this down, you can install the preview version tomorrow and use the setting "githubPullRequests.logLevel": "debug" to enable the debug logging. Then, next time you see the issue can you share the output from "GitHub Pull Request"? The important lines should have "CreatePullRequestViewProvider" in them. Thank you!

Thanks alex, I switched to preview version and will set log level in preparation for helping debug!


Lesson path screen

The story:
As a user that ** wants to keep track of my progress through the course**, I want to visually see how many lessons I have finished and how many I have left

What we should design:
We should build a lesson path with detailed hover states for each lesson so that  users can quickly see where they are and what they are learning.
To be successful we need to:
Check these off as they are completed:

[x] Lesson path
[x] Hover state with lesson details (concepts)
[x] Distinctive shape between lessons and assignments

Done


Allow SetBalance to handle error when trying to kill acount with reference counter.

The set_balance extrinsic could lead to an unexpected state change when trying to kill an account that cannot be killed.
This resolves that issue, and adds a test to verify the behavior is working as expected.

If we can do it in tests, we probably should. If there are exceptions to be had, we should have some syntax to enable the exception, but most often than not, we follow this rule.

bot merge

Cumulus CI seems to be broken. This should be fine to merge though.

bot merge


carousel slider need

Feature Description
We need very light weight customizable carousel slider (example: multi card slider ,chips slider just like YouTube etc).
Use Case
we can use it for banner image for product card slider, product category slider(chips)

In the past we've closed this request because Material spec doesn't define a carousel.  It would be nice to provide one but it's tricky for us to come up with our own spec without designer influence. It's a bit ironic because our material.angular.io homepage has a homemade one.
https://github.com/angular/components/issues/1081
https://github.com/angular/components/issues/4793
https://github.com/angular/components/issues/5671
https://github.com/angular/components/issues/5693
https://github.com/angular/components/issues/6626
https://github.com/angular/components/issues/14661
https://github.com/angular/components/issues/17437
https://github.com/angular/components/issues/23424
We'll leave this up for the voting system to collect thumbs up

We need exactly similar behavior of Android View-Pager https://developer.android.com/training/animation/screen-slide-2

Related:

https://material.io/blog/material-3-carousel-research-design - the evolution of the Material 3 carousel (Oct 5, 2023)
https://m3.material.io/components/carousel/overview Material Design 3 Carousel Component

This feature request is now candidate for our backlog! In the next phase, the community has 60 days to upvote. If the request receives more than 20 upvotes, we'll move it to our consideration list.
You can find more details about the feature request process in our documentation.

when carousel feature will be added?

carousel slider need


Added lexer for parsing GSQL files

Created lexer for TigerGraph GSQL query language and .gsql files.

Looks like the test files are not matching? Could you please regenerate the test output files?

Thanks. Is this correct that you want things like QUERY identified as Builtin.Name and not Keyword?


[CI/CD] Backport release workflow to 1.1.latest

resolves # N/A
Description
This PR is just a backport of the following commits:

#266
#292

Checklist

[x] I have read the contributing guide and understand what's expected of me
[x] I have signed the CLA
[ ] I have run this code in development and it appears to resolve the stated issue
[x] This PR includes tests, or tests are not required/relevant for this PR
[x] I have opened an issue to add/update docs, or docs changes are not required/relevant for this PR
[ ] I have run changie new to create a changelog entry

@nathaniel-may @emmyoop Is there any reason we wouldn't merge this?

@mikealfare Yes. It doesn't work.  It's blocked by https://github.com/dbt-labs/dbt-redshift/issues/311.  I'll close this as won't merge.
Because this is only for 1.1.latest and we're unlikely to need to release it again before end of life, it was decided to not fix this for the automatic release.  Release can be done manually in a pinch.


Add convenience application factory for self-hosting

This is a convenient application factory that calls the IAuthorityHistoryPairFactory followed by the CartesiDAppFactory, to simplify the deployment of application that will be validated by an authority.
It's being used by the sunodo deployment system at https://sunodo.io/deploy
With the move from Sunodo CLI to Cartesi CLI we will probably have to move this contract away from the sunodo repo.

There is a way to keep this contract under the sunodo context, if we assume that it's only for the sunodo webapp, as a convenience.
And make the Cartesi CLI not use it, and send two transactions on application startup.

The  IAuthorityHistoryPairFactory is not needed in newer versions. The consensus now inherits the history functionalities from AbstractConsensus  and there is no need for the History contract anymore.

Ok, but this is for V1

I've rebased release/1.4.x on top of release/1.3.x because of the 1.3.1 release.

I've made some small adjustments, and added tests.
We can squash everything once it's merge time.

@tuler Do you approve these changes? I can't tag you as reviewer since you are the one who created this PR.


Expected only three 'NPDE' markers

Getting this output:
[mauch@M80 NVIDIA-vBIOS-VFIO-Patcher]$ python nvidia_vbios_vfio_patcher.py -i mx150.rom -o mx150patched.rom
Opening the ROM file...
Scanning for ROM offsets...
Offsets found!
Running sanity checks...
Traceback (most recent call last):
File "nvidia_vbios_vfio_patcher.py", line 194, in <module>
main()
File "nvidia_vbios_vfio_patcher.py", line 172, in main
rom.run_sanity_tests(args.ignore_sanity_check)
File "nvidia_vbios_vfio_patcher.py", line 103, in run_sanity_tests
"footer, found %d" % npde_count)
main.CheckException: Expected only three 'NPDE' markers between header and footer, found 2

NVIDIA includes MX150 as a part of the Pascal series, but the vBIOS format it uses seems to differ from the rest of the series. If I had to guess, it would have something to do with the card not being a PCI-e card.
This script works by using the differences between full (what you'd get by dumping the BIOS with GPU-Z or a similar tool under Windows) and partial copies (what you'd get by dumping the ROM under Linux) of vBIOSes. Here's a forum post on Lime Technology forums I posted about this.
If I had a full and a partial copy of the same vBIOS, I might be able to do the same for this GPU, though at that point you wouldn't need this script in the first place. I also released this script since I was able to verify that it works with my own GTX 1070, so I wouldn't be able to make the same claim of compatibility with the MX150.

I have the same issue with the "GTX 1060 Mobile with Max-Q Design".
Is there anything you or I can do?
Here's what I did:
## Download the vBIOS of the GTX 1060 Mobile with Max-Q Design
$ wget https://www.techpowerup.com/vgabios/205874/205874.rom
## Download the script
$ wget https://raw.githubusercontent.com/Matoking/NVIDIA-vBIOS-VFIO-Patcher/master/nvidia_vbios_vfio_patcher.py
## Make the script executable
$ chmod +x ./nvidia_vbios_vfio_patcher.py
## Run the script
$ ./nvidia_vbios_vfio_patcher.py -i 205874.rom -o 205874_patched.rom
Opening the ROM file...
Scanning for ROM offsets...
Offsets found!
Running sanity checks...
Traceback (most recent call last):
  File "./nvidia_vbios_vfio_patcher.py", line 194, in <module>
    main()
  File "./nvidia_vbios_vfio_patcher.py", line 172, in main
    rom.run_sanity_tests(args.ignore_sanity_check)
  File "./nvidia_vbios_vfio_patcher.py", line 103, in run_sanity_tests
    "footer, found %d" % npde_count)
__main__.CheckException: Expected only three 'NPDE' markers between header and footer, found 2

New Laptop, this time with a normal GTX 1060 Mobile (without Max-Q design): Same problem.

Same problem with GTX 1070 Max-Q

Same here with Quadro P600 (Pascal based).

I have a 1050 Ti Mobile and the same issue. Is there anything we can do about this?

Same issue with 1050 TI Mobile.
I extracted Nvidia BIOS from UEFI BIOS for my laptop, here it is
FX705GE-AS.307_NVidia_BIOS.bin.zip
.


Update rstudio repository keys

Repo keys changed in https://cloud.r-project.org/bin/linux/debian/

/gcbrun


Testing shap: Pure noise rank higher than the only meaningful feature

I would like to share a simple test that I ran for testing shap, which gives counterintuitive results.
I generated data with 3 features. Two of them are pure noise and has no connection to the target. The third feature is a binary feature with 10% 1s and 90% 0s. This is the only feature that the target has a dependency on. However, shap ranks it after the pure noise features.
* Link to colab notebook
I am curious to understand what might be going on here and if a similar issue would impact importance scores with real data.

The xgboost model might actually rely on the noise feature, in case of overfitting. If overfitting happened, it would not be surprising when those features are ranked highly. SHAP is for explaining how the model makes predictions, and a bad model will show results that deviate from the true data generating process.
You could try to optimize xgboost number of iterations and see if it the ranking changes.

Thanks for looking into this. After properly regularizing the XGBoost model, the ranking changed in the expected direction.


provide version information somewhere

It would be nice, especially at current early stages of development, to display the deployed version (tag "releases" and use git describe) somewhere visible but not too prominent (bottom of the page?) on https://gui.dandiarchive.org

Do you mean the version of the gui client or of the dandi plugin?

Both?

Probably doable at the next front end dev sprint.


some BUG with a gif to show

VSCode Version: 1.17.2
OS Version: win10

Steps to Reproduce:

Reproduces without extensions: Yes/No

Fixed in latest


hi! /glowing text

I like your idea and I was wondering if you could do something similar with glowing texts?

@fioxu0 What are the steps to recreate this?

I mean fix glowing effect on signs
it works similarly to bold text and also needs your help ^^


Added TypeScript declarations for the Alexa address objects

Added TypeScript declarations for responses returned from the device-api

Looks like builds for node 1 to 4 are failing, but I don't see how my commit would have caused this :)

The failure actually relates to npm v2, which tries to satisfies the peerDependency "typescript": ">=2.1.0 || >=2.1.0-dev || >=2.2.0-dev || >=2.3.0-dev || >=2.4.0-dev || >=2.5.0-dev || >=2.6.0-dev || >=2.7.0-dev || >=2.8.0-dev" from tslint, but dtslint is using "typescript": "next" which resolves to 2.9.0-dev (and npm versions below 3 fail if there's a peerDependency conflict).
I've opened palantir/tslint#3792 which should resolve this errors, but I think it's safe to ignore them for now as they relate to the linter, and not to the actual performance of the library.
@dblock what do you think?

Well I don't want to check in something that turns the build to broken. Can we resolve this by locking down things to an older version?

@kobim Got a clean build now, thanks for fixing the dtslint issue

We can merge this right? @lazerwalker ?


Review KCS Draft SOPs for ModelMesh Serving

We are in the process of migrating the RHODS Standard Operating Procedures (SOPs) from our internal GitLab repository to the KCS solutions platform. This migration aims to consolidate all SOPs into one accessible location, promoting collaboration across teams including engineering, SRE, and CEE.
This consolidation ensures that anyone working with a customer, and potentially the customers themselves once solutions mature, will have access to this centralized knowledge repository.
As part of this effort, draft KCS SOPs have been created for the existing SOPs. I kindly request the expertise of an engineer to review the draft KCS SOPs and provide feedback on the following two items:

Accuracy of Commands: Some SOPs in GitLab lacked precise steps, which have been added to the KCS drafts based on my testing. Please verify the accuracy of commands in the KCS drafts and let me know if any of the steps should be changed/improved.

Context, Symptoms, and Impact of Alerts Firing: Provide additional context, symptoms, and impact insights for alerts. If possible, identify dependencies on other components and suggest pre-escalation checks for SRE and CEE teams. Are there any additional checks that SRE or CEE could perform before requesting engineering support, which are not currently outlined in the SOPs?

The KCS drafts for ModelMesh Serving can be reviewed here:
7025310
7025312
Feel free to reach out to me with any questions.

I think the KCSs you created are the good starting point to gather logs for modelmesh/odh-model-controller and also logs for other components. I think it is a good idea to make it as landing trouble shooting KCS for RHODS. Then add sub link like modelmesh or kserve KCSs .. odh-model-controller log are always needed for both cases

@maryfrances01 are there any outstanding drafts we need to look into?

@heyselbi No, the draft has been updated and I will publish it once the other component drafts are complete. Thanks - I'll close this!


[Event Request] - ItemJnlPostLine.Codeunit.al - OnBeforeIsInterimRevaluation

We like to request the following Event:
Object: ItemJnlPostLine.Codeunit.al
Where is the event placed:
local procedure IsInterimRevaluation(): Boolean
var
    IsInterimRevaluationResult: Boolean;
    IsHandled: Boolean;
begin
    OnBeforeIsInterimRevaluation(ItemJnlLine, IsInterimRevaluationResult, IsHandled);
    if IsHandled then
        exit(IsInterimRevaluationResult);

    with ItemJnlLine do
        exit(("Value Entry Type" = "Value Entry Type"::Revaluation) and (Quantity <> 0));
end;

Event Code:
[IntegrationEvent(false, false)]
local procedure OnBeforeIsInterimRevaluation(var ItemJnlLine: Record "Item Journal Line"; var Result: Boolean; var IsHandled: Boolean)
begin
end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


Optimization needed (probably)

Alright so, signatureTimestamp (sts) extractor from my PR https://github.com/kkdai/youtube/pull/199 is very basic, and it needs further optimizations.
Main considerations:

It looks like non-encrypted songs (those without cipher but with plain url in formats) works perfectly fine if we didnt send signatureTimestamp.
Because of ^1, maybe parsing player config for each particular song isn't the most optimal solution? Though, we can understand this only after making youtubei/player request (and getting 403 after decoding cipher, lol).
Currently lib loads player config two times:
3.1. once to get sts, and its being proceed for every song,
3.2. and second time to get decipherOpts when song contains cipher.
As it seems, sts changes not very often, so it may be reasonable to store it in cache same as decipherOpts.

Small digression about current DecipherOperationsCache:
It stores operations with videoID as key, but operations are player specific, not video. So, operations from "player/4fbb4d5b/.../base.js" will always be the same. And it seems like all videos using player with same version.
Also, why exactly default cache expiration = 5 minutes?
Maybe we could store values (both sts and decipherOpts for long time? And for validation we may send small HEAD request after decoding url and check for response code: If its 403 - clear cache and start again

This already happens, if format.URL != "", correct?
That's true. To get the player ID, we need to fetch https://youtube.com/embed/... for each video, correct?
In this case the player config should be cached too.
Agree

Do you suggest to use the cache for videos and players? We also need to remove entries from the cache after some time. Maybe we can use a LRU cache instead of 5 minutes? I don't know a reason why it is excatly 5 minutes.

https://github.com/kkdai/youtube/pull/201

Hey, I have been using ANDROID requests for about a week now with no problem [1], just wondering if you all are not using that for a particular reason.
If not, then its a good option, as you dont have to do any decrypting.

https://github.com/89z/mech/blob/master/youtube/android.go

DecipherOperationsCache  is to solve #140 ,  no special reason with 5 min cache time , only to  make it simple at that moment, if necessary just change it.


checkpoint files

Thank you for your great work，could you share the checkpoint files？

@WangShuaixian are you looking for the final checkpoint files or the final mesh files?

Thank you for your reply and support ！  If possible, I want both. Thank you very much！

@WangShuaixian are you looking for the final checkpoint files or the final mesh files?

Thank you for your reply and support！ If possible, I want both. Thank you very much！

Thanks for the great work，I have slove it


Add check for fx compatibility on segmentation and video models

Similar to #3662, we are able to check if segmentation models are fx compatible.

@NicolasHug Thanks for flagging.
Unfortunately the FX compatibility requires traceability which is not supported by all models (for example object detection). Any other model that is traceable (for example all fully-convolutional models) can be tested against FX compatibility, so I updated the PR to include the ones on test_video_model() method.


Firefox: Performance Issue

Hi,
I prepared code in Plunker in which I would like to display 3 opinions
http://plnkr.co/edit/hiZ7GW?p=preview
As you can see on Firefox "Opinions" header is animated first and after few seconds opinions are being displayed. In Chrome/Edge header and opinions are being animated at the same time.
Second issue I noticed is that when I want to display opinions that have only uuid and put in opinion tag if.bind="opinion.uuid" it doesn't work on Firefox and nothing is being displayed.
In Chrome/Edge everything works fine.

@zewa666 Can you take a look into the animation issue.
@jdanyow If you get a chance, can you look into the "if.bind" issue?

@lares83 are you still hitting an issue with if.bind?  I've updated your plunker to the latest aurelia bits, it's working in firefox now:
http://plnkr.co/edit/uRFQwi?p=preview

@jdanyow Hi, yes I still have it.  When you put

opinions are not being displayed in firefox but on chrome they are.

@lares83 - thanks for the additional info.  We're tracking the firefox issue with if and repeat on the same element in https://github.com/aurelia/templating-resources/issues/111.


Feature/sm expansion

Added the 'prepare move through'-state. It reads the odometry and depending on where the AUV enters the circle of acceptance of the goal position, the AUV positions itself to safely move through the gate.

I added some package info to README.md. I can be more specific about the implementation of the state machine if you want.

I added some package info to README.md. I can be more specific about the implementation of the state machine if you want.


OLS-252: Ensure that OLS supports cluster-wide HTTP/HTTPS proxy

Description
Make the operator pass the http[s]_proxy, etc environment variables that the OLM sets up for it to the operands: the OLS and the console server.
Type of change

[ ] Refactor
[x] New feature
[ ] Bug fix
[ ] CVE fix
[ ] Optimization
[ ] Documentation Update
[ ] Configuration Update
[ ] Bump-up dependent library

Related Tickets & Documents

Related Issue #
Closes # https://issues.redhat.com/browse/OLS-252

Checklist before requesting a review

[x] I have performed a self-review of my code.
[ ] PR has passed all pre-merge test jobs.
[ ] If it is a core feature, I have added thorough tests.

Testing

Please provide detailed steps to perform tests related to this code change.
How were the fix/results from this change verified? Please provide relevant screenshots or results.

/lgtm

/lgtm


Add testcase for tenant user login with scope assignment

This PR adds the testcase for the tenant user login issue when the user is assigned a custom role with only scope assignments.
Related issue: https://github.com/wso2/api-manager/issues/1169

Closing the PR since this scenario is tested in other test cases.


Private Channel Error

Hi, i'm trying to subscribe to a private channel, but it gives me some error. I know what the error means I just needed to know how to send the right data on the request:
The error says:
"Auth value for subscription to private-message-answered-5 is invalid: should be of format 'key:signature'"
I need to call my auth endpoint that is /broadcast/auth, and I also need to pass a token on header, when calling the pusher.subscribe method
Does anybody knows how can I achieve that ?

I just got it working by making a normal post request to my api channel auth and then pass the auth string to pusher.subscribe(channel_name=channel_name, auth=auth)


🚀 : Discord App Prototype Client

Is your feature request related to a problem? Please describe.
Discord users currently have limited options for integrated language learning experiences within the platform. This represents a missed opportunity to cater to a growing global audience eager to connect across linguistic boundaries.
Describe the solution you'd like
Integrate a language learning web app as an Activity within Discord, leveraging the Activity Game Canvas to allow users to participate in social, interactive language learning experiences together. This aligns with Discord's recent push towards fostering richer activities within the platform (https://discord.com/developers/docs/activities/overview).
Describe alternatives you've considered

Standalone language learning bots within Discord: These lack the richness and interactivity that a fully-fledged language learning web app can provide.
Third-party language learning applications: These require users to leave the Discord client, disrupting the flow of communication and community engagement.

Additional context
The Discord Activity Game Canvas would allow for the creation of innovative language learning experiences, such as:

Collaborative translation tasks
Interactive language quizzes and games
Flashcard-based activities where users can test each other's knowledge
Role-playing scenarios where users practice conversation in a safe and engaging environment

By integrating a language learning web app as an Activity, Discord can foster a more engaging and social language learning environment, potentially attracting new users and strengthening existing communities. This proposal also aligns with Discord's interest in supporting innovative app ideas (https://discord.com/build/app-pitches-2024).
Contributor: @dcsan
If you want speed up process of this issue, please send 🚀 as a reaction.

labelled not-in-roadmap-yet
because its not easily convertible as DG app for now. Maybe later on we can add discord app as a new client but not for now. thank you for sharing this issue with us.


Question about Neo4j browser

Hi sorry for bothering you guys, how can i access neo4j http://localhost:7000/browser/, on my vps like for example http://ipaddress:7000/browser ? now my neo4j start on port 7474.
Thank you

If you've installed neo4j from the neo4j gem, you can use rake neo4j:config[development,7000].  If you installed it otherwise you can find the port configuration in the neo4j-server.properties file

Hi @cheerfulstoic thanks for the reply, i tried to rake neo4j:config[development,7000] and tried to access my vps http://ipaddress:70000/browser but got a response unable to connect. so i tried to change neo4j-server.properties
and uncomment this section with my ip address
org.neo4j.server.webserver.address=xxx.xxx.xxx.xxx

Sorry, I'm not sure I understand what your VPS is what exactly you're trying to do.  Maybe you should try over on Stackoverflow with the tag neo4j as this isn't really related to the gem (as far as I can tell).


#read returns all rows instead of just the remaining ones

Here's the documentation of the #read method:

Slurps the remaining rows and returns an Array of Arrays.

Let's check this with the following sample script:
# frozen_string_literal: true

require 'csv'

data = StringIO.new <<~DATA
  foo,1
  bar,2
  baz,3
DATA

csv = CSV.new(data)
puts "Shift 1: #{csv.shift.inspect}"
puts "Shift 2: #{csv.shift.inspect}"
puts "Rest   : #{csv.readlines.inspect}"

$ ruby -v
ruby 2.5.3p105 (2018-10-18 revision 65156) [x86_64-darwin18]

$ ruby sample-csv.rb
Shift 1: ["foo", "1"]
Shift 2: ["bar", "2"]
Rest   : [["baz", "3"]]

Perfect - using ruby-2.5.3 this is all fine.
Running this same example with ruby-2.6.2 it shows #read returns all rows of the file instead of just the remaining ones:
$ ruby -v
ruby 2.6.2p47 (2019-03-13 revision 67232) [x86_64-darwin18]

$ ruby sample-csv.rb
Shift 1: ["foo", "1"]
Shift 2: ["bar", "2"]
Rest   : [["foo", "1"], ["bar", "2"], ["baz", "3"]]

It has been fixed at the latest release.
You can use the latest release with gem install csv.
I'll upgrade bundled csv in Ruby 2.6.3.


docs($router): Note that router === this.$router

@chrisvfritz Can I have your review on this, please?

The computed code block might not even be necessary though that is another great insight for getting params! Super helpful knowing we can refer to this.$router (a lot of the answers on the internets do things with this.$route as a legacy from vue 1.0)

@connor11528 That's a bit terrifying... It's a very basic feature. This means it's probably not clear in the docs 😞

Thanks a lot for the nice review @chrisvfritz 🙂


Remove hack, canot use statelessComponent

Right now on the definition of
class MemberPage extends React.Component<Props, {}> {
We are using React.Component, we should be using React.StatelessComponent, if we make this change we get an error, we had to rollback to React.Component, and add an ugly hack when gluing the reducer:

// TODO: Hack to bypass the issue when declaring StateLessComponent
// Pending research here
var nonTypedMemberPage : any = MemberPage;

const ContainerMemberPage = connect(
                                   mapStateToProps
                                  ,mapDispatchToProps
                                )(nonTypedMemberPage)

We need to unsderstand what's going on here and how this could be fixed, found in stackoverflow that we should upgrade to typescript 1.8 in order to fix that done but no luck, maybe is just the tsd that is not well defined.

This issue was resolved in #8 Redux support. Before that is not needed.


Updated siesta URL to the github one

Updated siesta URL to the github one, so https://awesome-repos.ecp.plus/ios.html and other similar sites can infer the star count and index it properly.
In case you want to detect this issue in other entries in the readme a regex that helps to find them is:
^(?!.*//github\.com).*https.*$

          1 Warning

:warning:
Found 6 link issues, a project collaborator will take care of these, thanks :)

Link issues by awesome_bot

Line
Status
Link

26
500
https://twitter.com/awesome_ios

1610
301
https://github.com/facebook/facebook-objc-sdk redirects tohttps://github.com/facebook/facebook-ios-sdk

1890
301
https://github.com/xcpretty/trainer redirects tohttps://github.com/fastlane-community/trainer

3213
301
https://github.com/shinobicontrols/iOS8-day-by-day redirects tohttps://github.com/ScottLogic/iOS8-day-by-day

3218
301
https://github.com/shinobicontrols/iOS9-day-by-day redirects tohttps://github.com/ScottLogic/iOS9-day-by-day

3220
Error
https://ioscookies.com/  Failed to open TCP connection to ioscookies.com:443 (getaddrinfo: Name or service not known)

  Generated by :no_entry_sign: Danger

@acecilia Thanks.


[Impeller] PipelineDescriptors must have pixel format specifiers.

This is necessary for the Vulkan backend. The backend could create pipelines with a reasonable default and then create PSO variants on demand. However, there is no need to do this as it may cause jank even if a pipeline cache is wired up an a parent pipeline present. Moreover, Impeller already knows the formats upfront.

Right now, these are being hardcoded. And we are getting away with it. But we will need to support this correctly.

The color attachment descriptor already has a pixel format. I was confused about the depth and stencil pipeline attachment descriptors not having them. But, this information is present in the pipeline descriptor itself. So we have everything we need already. Closing.


Looking for a css parser and selector.

Hi,
For my project I need a css parser and selector. I personally am really bad at writing parsers and don't want to spend the time learning how to do it properly just yet. I found your myCSS parser and the css parser in this project. Now I am wondering, which one is best for me to use? I ofc hope to get the parser that is most css3 compliant.(But doesn't neceserily have to be)
Which one do you reccommend me to use?
Thanks!
Edit:
I only need the css parser and selector, so having as little dependencies on other stuff I don't need would be nice.

Hi @timleijt
I would not use MyCSS (only if in Modest).
In the lexbor project, the CSS parser has been completely rewritten and its API has changed radically. But, I'm still testing it, the code has not been published. The selectors are next.
Everything to increase speed and save memory.
You can wait, though it's not clear how much, I have a very large workload now at my main job.

Ok thank you.

Hi @timleijt
I would not use MyCSS (only if in Modest).
In the lexbor project, the CSS parser has been completely rewritten and its API has changed radically. But, I'm still testing it, the code has not been published. The selectors are next.
Everything to increase speed and save memory.
You can wait, though it's not clear how much, I have a very large workload now at my main job.

I hope you're find to finish it. We love to hear from you soon.
Very nice lib by the way.
All the best.

@Dan-Do
I finished refactoring CSS parser and ended the selectors. Testing before commit.
Thanks for using the library.

@Dan-Do
I finished refactoring CSS parser and ended the selectors. Testing before commit.
Thanks for using the library.

Hi there! Im just curious about how the testing is going? 😁 looking forward!

@BarryThrill
Hi!
A lot of work lately. Everything was stretched out. But I am already continuing to test selectors.

Hey again! :D Just wanted to see how is the testing going? I have seen that there has been quite few testing commits so im glad to hear how its going! Excited to see how good lexbor will be!

Hi @BarryThrill
In progress. Testing revealed a fundamental bug. I'm changing my approach.
I think that in 1 week everything will be ready.

Hello
@lexborisov what is the status ?
Vincent

It seems many members are waiting for this great feature :)

Hi @BarryThrill
In progress. Testing revealed a fundamental bug. I'm changing my approach.
I think that in 1 week everything will be ready.

Hello! Hope you are alright and having it good and safe!
Im excited to hear if there has been any progress as I and I believe other people are also looking forward for it :D Is there any hints on a beta test/release? :D Or any updates at all

@BarryThrill
Hello! I have had a coronavirus and now I am catching up on lost time at my main job. (nginx, njs)
The selectors are complete and tested. But the code requires optimizations for which I still cannot allocate time.
Please wait.

@BarryThrill
Hello! I have had a coronavirus and now I am catching up on lost time at my main job. (nginx, njs)
The selectors are complete and tested. But the code requires optimizations for which I still cannot allocate time.
Please wait.

Hope you had it okey when having the covid. Myself I almost ended up in the hospital. Hope you doing great now and we all looking forward 😁

@lexborisov will this include xpath selectors as well? Ref: https://github.com/lexbor/lexbor/issues/23#issuecomment-779336818

@jayflo
No, Selectors Level 4 only.

@BarryThrill @vtorri  @mario-132 @jayflo @Dan-Do
The first implementation of selectors is commented.
I would be glad to see performance tests, comparisons with Modest.

So it is in a somewhat usable state? I would like to give it a try if so

@mario-132
Usable condition.
There are tested, testing with a fuzzers, and so on.
Any feedback on usage will be helpful.

Very nice, I'll give it a try tomorrow!

benchmark update here: https://github.com/kostya/myhtml#benchmark

@kostya did you try to add some optimization flags, like -march and -mtune ?

@vtorri

@kostya did you try to add some optimization flags, like -march and -mtune ?

In real life, these optimizations (-march=native) are not used.

anyway, i would like to see the differences :)

added benchmark on apple m1: https://github.com/kostya/lexbor#benchmark

Selectors and CSS Styles was added in the project.


17191 fix is relative function

Q
A

Is bugfix?
✔

New feature?
❌

Breaks BC?
❌

Fixed issues
#17191

I fixed the BaseUrl::isRelative method. I wrote some tests to cover all the  cases that came up to my mind.
Thanks

@bizley just noted this today. This is the note from parse_url:

Caution This function may not give correct results for relative or invalid URLs, and the results may not even match common behavior of HTTP clients. If URLs from untrusted input need to be parsed, extra validation is required, e.g. by using filter_var() with the FILTER_VALIDATE_URL filter.

https://www.php.net/parse_url
This means that we're using parse_url incorrectly since we're expecting to use it on relative URLS.

Ugh, right. Looks like reverse is in order then. On it.

Hm, after some more research I think we can keep it - we rely here only on the scheme part (whether it's present or not) - the warning is for checking other url parts, there are known bugs with it, but I could not find something about scheme itself being not recognized properly.

BTW: benchmark that I mentioned in https://github.com/yiisoft/yii2/pull/20089#issuecomment-1871927855: https://3v4l.org/fNYfS
parse_url() is 10x slower slower than strpos(), but it is still pretty fast.

I checked https://3v4l.org/ioiFVo - maybe we should go for triple strpos?

I would go with regexp. It is comparable to triple strpos in terms of performance for relative URLs (and faster for absolute URLs) and supports all protocols at the same time: https://3v4l.org/Vb0rJ

Hm, after some more research I think we can keep it - we rely here only on the scheme part (whether it's present or not) - the warning is for checking other url parts, there are known bugs with it, but I could not find something about scheme itself being not recognized properly.

That's good news, so this is no longer a discussion about functional regression.
If it's just about performance it can lower the urgency a bit....

I checked https://3v4l.org/ioiFVo - maybe we should go for triple strpos?

This is not a fair test; you're testing cases that are very favorable; either the pos is 0, or in case of // it's 4. The 3rd case is bad because it doesn't have contain it.
I've added str_starts_with which is even faster but not available on lower versions.
Btw: what do we do with mixed case URLs?
I'm working on a more detailed performance test; give me a few more mins!

Not sure what you mean by the fair test, I was testing whether the url starts with http:// or https:// or // hence the 0 (in real code this would require inversion of course). But I would go with the regex Rob proposed (@^((\w+):)?//@) since it's fast (and on top of everything it will take care of mixed case protocols :)

With not fair meant you're only testing cases where the URL is not relative.
https://3v4l.org/Dpob1
This is a cleaned up version if your original tests @bizley.

It has more examples, buth relative and non relative ones.
It has 6 implementations

Output php 8.3.1:
preg_match: 0.168
preg_match (case insensitive): 0.173
strpos: 0.226
str_starts_with: 0.243
substr_compare (case insensitive): 0.204
substr_compare (case sensitive): 0.221
parse_url: 0.376

Output 7.4
preg_match: 0.181
preg_match (case insensitive): 0.182
strpos: 0.190
str_starts_with: 0.189
substr_compare (case insensitive): 0.133
substr_compare (case sensitive): 0.144
parse_url: 0.231

This is from doing 200.000 iterations for each of checking 6 examples. (Don't compare the times with your tests since I've added more boiler plate and am verifying the results).
Interestingly, as we're always told: don't optimize too early. As we see here at some point preg_match becomes faster, likely because the regex engine actually compiles the regular expression. This is very expensive but amortized over large number of iterations it becomes cheaper.
With 300.000 reps on 8.3.1 preg_match wins vs all other strategies.
preg_match: 0.184
preg_match (case insensitive): 0.172
strpos: 0.244
str_starts_with: 0.257
substr_compare (case insensitive): 0.219
substr_compare (case sensitive): 0.223
parse_url: 0.322

With 30.000 reps however (a lot less consistent) we see cases where it's twice as slow.
preg_match: 0.034
preg_match (case insensitive): 0.017
strpos: 0.025
str_starts_with: 0.027
substr_compare (case insensitive): 0.021
substr_compare (case sensitive): 0.023
parse_url: 0.033

Given these results I agree that we should just do preg match. It is the most readable code and works more than fast enough.
TLDR: I agree with using preg_match

Great work, guys! Let's go with preg_match then. @SamMousa you are already working on the PR, right?

Correct! ;-)  Just looking at the RFC for the scheme part:
 scheme        = ALPHA *( ALPHA / DIGIT / "+" / "-" / "." )

https://www.rfc-editor.org/rfc/rfc3986
How'd you guess?


v0.2

Fix CI
Adding support to pydantic 1.6.1

Codecov Report

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

@@            Coverage Diff             @@
##           master      #19      +/-   ##
==========================================
- Coverage   92.62%   92.59%   -0.04%
==========================================
  Files           7        7
  Lines         217      216       -1
==========================================
- Hits          201      200       -1
  Misses         16       16

Impacted Files
Coverage Δ

pydantic2graphene/converter.py
90.74% <0.00%> (-0.09%)
:arrow_down:

Continue to review full report at Codecov.

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


Adds #reject_attribute to the DSL ValueArray

Expands the functionality of #match_attribute to accept a block to match against and adds an inverse #reject_attribute that deals with some cases brought on by Tennessee's MODS.

An example of the TN mapping:
 relation record.field('mods:relatedItem')
  .reject { |rights| rights.try(:type) == 'isReferencedBy' }
  .reject { |rights| rights.try(:type) == 'references' }
  .field('mods:titleInfo')
  .fields('mods:url', 'mods:title')

The above can now be done like:
 relation record.field('mods:relatedItem')
  .reject_attribute(:type, 'isReferencedBy')
  .reject_attribute(:type, 'references')
  .field('mods:titleInfo')
  .fields('mods:url', 'mods:title')

or better:
 relation record.field('mods:relatedItem')
  .reject_attribute(:type) { |attr| attr == 'isReferencedBy' || attr == 'isReferencedBy' }
  .field('mods:titleInfo')
  .fields('mods:url', 'mods:title')

:+1:

Yes, this is more convenint.  Could I just check my understanding, as follows?
It appears that it's designed for XML with attributes, such as:
<some-parent-element>
    <mods:relatedItem type='isReferencedBy>something to reject</mods:relatedItem>
   <mods:relatedItem>
        <mods:titleInfo>
            <mods:url>http://example.tld/</mods:url>
           <mods:title>The title</mods:title>
        </mods:titleInfo>
    </mods:relatedItem>
</some-parent-element>

Should I understand reject_attribute and match_attribute to be XML-only methods?

Should I understand reject_attribute and match_attribute to be XML-only methods?

I don't think so, no. These methods should work with any format that has nodes with attributes. The Krikri::Parser, carries the relevant abstraction.

If I have JSON like this ...
{
  'relatedItem': {'type': 'isReferencedBy', '#text': 'something to reject'},
  'relatedItem': {'#text': 'good thing'}
}

... could I then write the following?
relation record.field('relatedItem')
  .reject_attribute(:type, 'isReferencedBy')
  .field('#text')

If that would work, then can you confirm that match_attribute or reject_attribute would also work if it were handed an object that's not a hash?  (If record.field('relatedItem') returned a string?)
It also may be out of scope, but I'm wondering what I'd do about an object that needs to be evaluated deeper than one level down, such as if the 'relatedItem' in the example above were {'typeInfo': {'type': 'something'}} and we wanted the whole 'relatedItem' where typeInfo.type is 'something'.  Is this taken care of somewhere?  (Would I just use a reject call in that case?)

If I have JSON like this ...   ... could I then write the following?

No. Sorry, I think this discussion has been confusing.  The Krikri::Parser deals with nodes ("fields"), which may have attributes. The JSON parser considers relatedItem, type, #text, etc... to be nodes and does not support attributes.
Your example code will throw NotImplementedError* if run against a record parsed with the JSON parser---it will try to access attribute type and hit the unimplemented #attributes.
You could do something like:
relation record.field('relatedItem')
  .reject { |n| n.child?(:type) }
  .field('#text')

If that would work, then can you confirm that match_attribute or reject_attribute would also work if it were handed an object that's not a hash? (If record.field('relatedItem') returned a string?)

All operations run though record are eventually called on, and return a Krikri::Parser::ValueArray. That array may be empty from the start, or continue to have values to the end. This means that there's safe method-chaining as deep as you like. Eventually the Declaration (property or child) calls Krikri::Parser::ValueArray#values to get the, e.g. the String.

It also may be out of scope, but I'm wondering what I'd do about an object that needs to be evaluated deeper than one level down, such as if the 'relatedItem' in the example above were {'typeInfo': {'type': 'something'}} and we wanted the whole 'relatedItem' where typeInfo.type is 'something'. Is this taken care of somewhere? (Would I just use a reject call in that case?)

It should be possible to do this, but it might not be as pretty as we'd like at this point.  I think something like record record.field('relatedItem').select { |item| item[:infoType].field(:type).include? 'something'} would work. This seems like a gap to me; I think a design goal should be for the user not to need access to anything lower on the parser tree than the current node when they use #select and #reject. Anything like this example really needs a DSL syntax.
Does that help? :D
*this is maybe not the best response, maybe parsers should be expected to explicitly have 0 attributes if they are unsupported? cc: @anarchivist

This seems like a gap to me; I think a design goal should be for the user not to need access to anything lower on the parser tree than the current node when they use #select and #reject. Anything like this example really needs a DSL syntax.

To clarify this record.field('relatedItem').select { |item| item.value == 'blah' } seems like a good pattern. So does record.field('relatedItem').map { |item| "somestuff ... #{item.value} "}. But record.field('relatedItem').map { |item| item[:someField].field(:blah).map { |blah| blah.value }` should (and, in this case does) have a nicer syntax.

And here's a real life example of something I would like to abstract away: https://github.com/dpla/heidrun-mappings/blob/feature/TennMapping/heidrun/TN_mods.rb#L51
I think this line should be something more like:
select_field('mods:role') { |role| role.include?('contributor') }

Discussion:

@markbreedlove:

Should I understand reject_attribute and match_attribute to be XML-only methods?
@no-reply
I don't think so, no. These methods should work with any format that has nodes with attributes. The Krikri::Parser, carries the relevant abstraction.
@markbreedlove:
If I have JSON like this ... ... could I then write the following?
@no-reply:
No. Sorry, I think this discussion has been confusing. The Krikri::Parser deals with nodes ("fields"), which may have attributes. ...
Your example code will throw NotImplementedError* if run against a record parsed with the JSON parser---it will try to access attribute type and hit the unimplemented #attributes. ...
*this is maybe not the best response, maybe parsers should be expected to explicitly have 0 attributes if they are unsupported? The more I think about this the more I'm convinced that it is very bad :tm: that attributes break method chaining for JSON records. cc: @anarchivist

Yeah, this was a decision in some ways that was made for the sake of expediency that I punted on while working on the Krikri::JSONParser. The NotImplementedError is defined [here](https://github.com/dpla/KriKri/blob/develop/lib/krikri/parsers/json_parser.rb#L29-L31) ... and it's defined that way because, as the message says, 'Attributes are not supported for JSON'`.
Put somewhat differently, this is made most obvious by the distinction between JSON (its objects have keys, which at the time I thought were roughly like elements but get abstracted to "fields") and XML (which enforces a distinction between elements and attributes). I think part of the bias that's showing here is that the Krikri::Parser class we started with was the XML parser.

I think part of the bias that's showing here is that the Krikri::Parser class we started with was the XML parser.

This is absolutely the source of a bias in the terminology for parsing. Suggestions for a better/more general terminology are welcome.

Thanks, @no-reply and @anarchivist.  That helps!
I don't have any objection to this being merged, if the topics being covered in this thread can be taken into consideration as the mapping DSL is developed further.  This changeset makes a commonly-occurring expression easier to write.

is this also ready to go? @markbreedlove: :merge: :grey_question:


Backport several changes to the 7.x branch

#26684
#26357
#26853
#26933
#24284
#26853

Oops failed to change the target branch.


json config: specify Deadline as string with time.Duration units

Was: {"Deadline":<PHONE_NUMBER>0}
Now: {"Deadline": "60s"}

This will break existing configs that use<PHONE_NUMBER>0, but given how annoying that is I suspect nobody will complain.

Thanks.


How to add class for table, cell and header?

Hi, this plugin is very awesome.
Is it possible to add class for table, or cell , header?

Other markdown-it plugin appends attributes to HTML elements. I'd recommend markdown-it-decorate for its great simplicity in implementation and flexibility in specifying any element.
For example:
var md = require('markdown-it')({html: true})  // for markdown-it-decorate
            .use(require('markdown-it-multimd-table'))
            .use(require('markdown-it-decorate'));

const exampleTable =
"| Hex  |\n" +
"|-----------|\n" +
"| `#A54242` | \n" +
"| `#8C9440` |\n" +
"| `#5F819D` |\n" +
"\n" +
"\n";

console.log(md.render(exampleTable));

generates HTML below

Hex

#A54242

#8C9440

#5F819D

Thanks. It works


Switching to a dialog and selecting a button for a customized form

I was trying to extend the tests for a customized form. In my case, before saving a form, a dialog might pop-up in certain conditions. Does the API provide any option to switch to the dialog and accept/ reject the dialog?
Please see the attached screenshots.

Since it looks like the dialog is not within an iFrame the existing SwitchToDialogFrame will not work.  I would add a explicit wait to look for the dialog.    Here is a example you can try. You may have to adjust the logic a little but it will wait for 2 seconds and if the dialog becomes visible then it will click the "Ok" button.
xrmBrowser.Driver.WaitUntilVisible(By.XPath("id("alertJs-dialog")"),
new TimeSpan(0, 0, 2),
x => { x.FindElement(By.XPath("id("alertJs-tdDialogFooter")/button[1]")).Click(); });

Let me know if that worked for you and I will close the issue.
Thanks

Hi, thank you for the reply. I followed your suggestion and used WaitUntilVisible. However, it seems the control does not go to the dialog box. The system interacts with UI elements in the background, and closes the entity form after the wait times and the dialog remains hanging. Please see the attachment.

ScriptExecutionRecording.zip

Hi, I am closing this issue.
I have been able to resolve this by switching to DefaultContent() before selecting the button in the custom dialog.
Thanks for your assistance.


Update ohw hub for this year's event

[x] Github teams based auth
team that should have access:

[x] @oceanhackweek/participants_2022
[x] @oceanhackweek/ohw22-organizers
[x] check if sub-teams get access too: so far it looks like people were able to login. Not 100% it was because they were authorized from last year's event, but will keep an eye on it.

[x] have different Python/R profiles
[x] figure out bumping the image tag given different profiles -> we should probably use https://github.com/sgibson91/bump-jhub-image-action
[x] check if there can be different defaultUrls for different profiles
[x] make sure the user image uses at least jupyterhub 2.3.1 otherwise default_url won't work
[x] why is this error still showing up in the hub logs: WARNING: hub.config.JupyterHub.authenticator_class was set to github and c.Authenticator.allowed_users was set, custom 2i2c jupyterhub config is now resetting allowed_users to an empty set.

[ ] remove existing auth0 oauth2 app for ohw hub

Reference: https://github.com/2i2c-org/infrastructure/issues/1515

This PR is missing some things, like the different images for the profiles and a better thinking about profile resources.
But since the images are not available, I set them to be the 2i2c default user image and I was thinking to merge this if it seems appropriate so that we make sure the authentication and profile works. WDYT?

But since the images are not available, I set them to be the 2i2c default user image and I was thinking to merge this if it seems appropriate so that we make sure the authentication and profile works.

I endorse this!


Start queue processor before failover callback registration

What changed?
Start queue processor before failover callback registration

Why?
Failover callback will try to create a failover queue processor for domains that need to be failover. On host start, the process can happen before tx or timer queue is started.
In new queue processor logic, queue readlevel(which will be used as max level for failover queue) can't be retrieved before queue processor is started, and failover callback registration will be blocked.
When tx or timer queue is not started, we don't need failover processor at all because we don't need to scan any tasks. Also we can start the queue processor before failover callback is registered to avoid the deadlock.

How did you test it?
Tested in staging2

Potential risks
Tasks get dropped when domain tries to failover while there are shard movements.

Coverage increased (+0.06%) to 66.898% when pulling 0fbee9176831dc58f1bfc9e3decae93568e0f0df on yycptt:fix-failover-on-start into 7acfe1323aaf91af71daf13cc1bec2e9f42ab9ce on uber:master.


Deprecated outSequence added when creating a proxy service through the VSCode plugin

Description:
When using the VSCode plugin to create a proxy service, the initial code of the proxy service will contain the deprecated outSequence

Suggested Labels:

Suggested Assignees:

Affected Product Version:
OS, DB, other environment details and versions:
Steps to reproduce:
Related Issues:

The latest release v1.1.1 has resolved this issue. Please reopen if the problem persists.


Add tests for optional guests & fixed bugs

It also looks like in your function, you're both computing both optionalAvailableTimes and availableTimes regardless of whether you'll need the latter. It might make sense to only compute it if optionalAvailableTimes is empty.

Makes sense, just made the change


Getting exception Operation could destabilize the runtime.

Hi,
I'm getting way beyond strange error when try to convert to BSON object from MS CRM SDK. Here is my test method (it's in file ComplexTypeTest.cs, line numbers in the file are the same as line numbers in snippet):
using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using System.Threading.Tasks;
using Microsoft.VisualStudio.TestTools.UnitTesting;
using Microsoft.Xrm.Sdk;

namespace LiteDB.Tests.Document
{
    [TestClass]
    public class ComplexTypeTest
    {
        [TestMethod]
        public void ComplexType_Test()
        {
            var mapper = new BsonMapper();
            var o = new Entity("account");

            var doc = mapper.ToDocument(o);
        }
    }
}

I would expect it not serialize fully, or correctly, but Operation could destabilize the runtime???
To reproduce error, please add Microsoft.CrmSdk.CoreAssemblies/6.1.2 nuget package.
As far I can see, it creates mapper, which contains property Item. While this property is invisible normally (it is protected I guess), or hidden in any other way, it shows this exception.
Here is the stack trace:
   at lambda_method(Closure , Object )
   at LiteDB.BsonMapper.SerializeObject(Type type, Object obj, Int32 depth) in C:\Workspace\shytikov\LiteDB\LiteDB\Mapper\BsonMapper.Serialize.cs:line 165
   at LiteDB.BsonMapper.Serialize(Type type, Object obj, Int32 depth) in C:\Workspace\shytikov\LiteDB\LiteDB\Mapper\BsonMapper.Serialize.cs:line 119
   at LiteDB.BsonMapper.ToDocument(Type type, Object entity) in C:\Workspace\shytikov\LiteDB\LiteDB\Mapper\BsonMapper.Serialize.cs:line 21
   at LiteDB.BsonMapper.ToDocument[T](T entity) in C:\Workspace\shytikov\LiteDB\LiteDB\Mapper\BsonMapper.Serialize.cs:line 29
   at LiteDB.Tests.Document.ComplexTypeTest.ComplexType_Test() in C:\Workspace\shytikov\LiteDB\LiteDB.Tests\Mapper\ComplexTypeTest.cs:line 20

Would appreciate any clue how to workaround this issue.

Hi! With the objective of organizing our issues, we are closing old unsolved issues. Please check the latest version of LiteDB and open a new issue if your problem/question/suggestion still applies. Thanks!


Proof Aggregation and ZKP Composing

An idea as described here: https://youtu.be/c0cKR78TIBg?t=1771 using "ZP Snark" as a technique to ZKP composing.
This will enable Semaphore composable with other ZKP primitives like private assets, token attestation, etc. so Semaphore can be used without explicitly adding the membership.


nimble/ll: Fix os_cputime comparisons in ll_adv

Need to use macros, otherwise it won't work as expected...

Excellent, very nice!
Don't have the resources right now but will re-run my experiment described in #1105 to verify this later this week. But please feel free to merge this beforehand :-)


Unable to read environment variable and also unable to read variables provided by -var

I'm using alicloud provider. ACCESS_KEY and SECRET_KEY was configured in environment variables but the task cannot read it and raised error Message: The input parameter "AccessKeyId" that is mandatory for processing this request is not supplied.
So to make it works I had to workaround it by specifying access_key and secret_key as parameters of -var, and meanwhile I defined these two variables in my variables.tf, but it came to error again as following.
2020-07-17T15:31:23.7460723Z [command]/usr/local/bin/terraform version
2020-07-17T15:31:23.7460980Z Terraform v0.12.28
2020-07-17T15:31:23.7779064Z + provider.alicloud v1.90.1
2020-07-17T15:31:23.7833614Z
2020-07-17T15:31:23.7966921Z [command]/usr/local/bin/terraform plan -var 'access_key=***' -var 'secret_key=***' -input=false
2020-07-17T15:31:23.9001693Z
2020-07-17T15:31:23.9003352Z Error: Value for undeclared variable
2020-07-17T15:31:23.9004237Z
2020-07-17T15:31:23.9005330Z A variable named "'secret_key" was assigned on the command line, but the root
2020-07-17T15:31:23.9006470Z module does not declare a variable of that name. To use this value, add a
2020-07-17T15:31:23.9007086Z "variable" block to the configuration.
2020-07-17T15:31:23.9008141Z
2020-07-17T15:31:23.9008659Z
2020-07-17T15:31:23.9009336Z Error: Value for undeclared variable
2020-07-17T15:31:23.9009669Z
2020-07-17T15:31:23.9010400Z A variable named "'access_key" was assigned on the command line, but the root
2020-07-17T15:31:23.9010982Z module does not declare a variable of that name. To use this value, add a
2020-07-17T15:31:23.9011445Z "variable" block to the configuration.
2020-07-17T15:31:23.9011946Z
2020-07-17T15:31:23.9157042Z
2020-07-17T15:31:23.9157947Z Error: No value for required variable
2020-07-17T15:31:23.9158319Z
2020-07-17T15:31:23.9158833Z   on variables.tf line 24:
2020-07-17T15:31:23.9159395Z   24: variable "access_key" {
2020-07-17T15:31:23.9159851Z
2020-07-17T15:31:23.9160322Z The root module input variable "access_key" is not set, and has no default
2020-07-17T15:31:23.9160968Z value. Use a -var or -var-file command line argument to provide a value for
2020-07-17T15:31:23.9161325Z this variable.
2020-07-17T15:31:23.9161702Z
2020-07-17T15:31:23.9169963Z
2020-07-17T15:31:23.9170678Z Error: No value for required variable
2020-07-17T15:31:23.9171010Z
2020-07-17T15:31:23.9171427Z   on variables.tf line 28:
2020-07-17T15:31:23.9171927Z   28: variable "secret_key" {
2020-07-17T15:31:23.9172523Z
2020-07-17T15:31:23.9173015Z The root module input variable "secret_key" is not set, and has no default
2020-07-17T15:31:23.9174293Z value. Use a -var or -var-file command line argument to provide a value for
2020-07-17T15:31:23.9174588Z this variable.
2020-07-17T15:31:23.9174866Z
2020-07-17T15:31:23.9194628Z
2020-07-17T15:31:23.9320758Z ##[error]Terraform command 'plan' failed with exit code '1'.:  Value for undeclared variable |  Value for undeclared variable |  No value for required variable |  No value for required variable
2020-07-17T15:31:24.5497762Z ##[section]Finishing: Terraform plan for VPC

BTW, I could successfully execute terraform plan -var 'access_key=xxx' -var 'secret_key=xxx' locally by using the same configuration.

Should your envvars have been set as ALICLOUD_ACCESS_KEY and ALICLOUD_SECRET_KEY, you have missed out ALICLOUD_
For the second error, the error states
" 'secret_key"
And
" 'access_key"
Note the odd single quote '
Could that be your issue.
Scott

Thank you Scott for your reply.
Regarding the first point, I did export the envars as ALICLOUD_ACCESS_KEY and ALICLOUD_SECRET_KEY, I only just wrote them in short in the description. If you could have a try on your side then you would  reproduce the issue.
Regarding the second points, I think I clearly mentioned I was using command terraform plan -var 'access_key=xxx' -var 'secret_key=xxx' which you could find the var name is access_key rather than 'access_key, and also you could find the executed command in the log output, which is [command]/usr/local/bin/terraform plan -var 'access_key=***' -var 'secret_key=***' -input=false.

Did you use TF_VAR_ALICLOUD_ACCESS_KEY. And same for secret key. Terraform needs envvars setup with TF_VAR_ in front of the actual variable name. But still put just the actual variable name (without TF_VAR_) when declaring the variable in .tf file.
Also, If you use secret/hidden variables I have found these are not picked up. If so then try in a secret.tfvars file in the pipeline and read into the init stage (no need for TF_VAR_ then) or as clear text variables.
My point re the extra single quote, is that it appears that somehow terraform (azure devops, or something) is sticking in an extra single quote. Or maybe the error message itself is just putting in the extra single quote ?  Either way, that looks weird to me.

Closing as this issue appears to have gone stale. Can reopen or revisit priority if additional information is provided or there is more interest from the community. Thanks @scott-doyland-burrows for helping again on this one!


Update LNDg to latest version 1.5.0

PR to update the LNDg install script bonus.lndg.sh to version 1.5.0. Tests good for me.

Second commit fixes an issue a user had who had added lndg with the PR prior to it being incorporated. When the user went to update raspiblitz to v1.9, lndg installed without an issue but couldn't read the database at /mnt/hdd/app-data. After troubleshooting, the problem was that the owner of /mnt/hdd/app-data/lndg was mempool for some reason. Adds a couple of commands to set the user and permissions of old databases in case they are messed up prior to installing lndg when doing a new install but using an existing database.

Last commit fixes indents


scriptEl.parentNode appears to be null when executing Phantom JS and IE tests involving sails.io

I'm using Karma / Jasmine to run my front end tests that involve sails.io as part of my dependencies and I seem to be getting false positive failed tests, or even non-executed tests in IE / Phantom JS, because of scriptEl.parentNode being null instead of an object in dist/sails.io.js line 207 column 9 when running the tests (if I open the app in IE myself it doesn't produce this error).
I came up with a simple solution that seems to fix the problem and decided to share so it might be looked at and possibly added. Replacing the said line (207) with this one fixes the issue and doesn't interfere with the sails.io operations in my normal application as far as I can tell:
if (scriptEl && scriptEl.parentNode) scriptEl.parentNode.removeChild(scriptEl);

+1

Fixed by #93


Change table name via ModelBuilder and ForSqlServer does not work for dnx451

Hi!
I am trying to change the default name in a table mapping to a new name via ModelBuilder because i have pluralized names in database but i cant get this to work for dnx451, it only works for dnxcore50
            {
                e.ForSqlServer().Table("MyEntitys");
                e.Key(m => m.UserId);
            });```

The extension method ForSqlServer() does not exist in dnx451 is the message I get and i cant find where to add this manually in the EntityType or where this name really is located.

Any ideas?

And i have the same problem with .Relational() extension method.

Can you share your project.json file?

Sure :)
{
"version": "1.0.0-*",
"description": "",
"authors": [ "" ],
"tags": [ "" ],
"projectUrl": "",
"licenseUrl": "",
"dependencies": {
    "Rinsen.Common": "1.0.0-*", // My own code
    "EntityFramework.Core": "7.0.0-beta4",
    "Rinsen.DatabaseInstaller": "1.0.0-*" // My own code
},

"frameworks": {
    "dnx451": { },
    "dnxcore50": {
        "dependencies": {
            "System.Collections": "4.0.10-beta-22816",
            "System.Linq": "4.0.0-beta-22816",
            "System.Threading": "4.0.10-beta-22816",
            "Microsoft.CSharp": "4.0.0-beta-22816",
            "EntityFramework.SqlServer": "7.0.0-beta4",
            "EntityFramework.Commands": "7.0.0-beta4"
        }
    }
}

}

Add EntityFramework.SqlServer and EntityFramework.Commands to the general dependencies property, not the dnxcore50-specific one.

Thanks!
That solved the problem, what difference does it make and when should I put dependencies under general dependencies and when to put them under version specific dependencies?

Put it in a framework-specific dependency list if you only want it for that framework. This is primarily used for dependencies that aren't available on some frameworks. In general, put it under the general one. :wink:

That´s simple enough :)
I probably will find out when i need it!
Thanks again! :)


Add hack for XXX protobuf fields

This is to stop fields starting with XXX coming from auto generated protobuf from ending up in the DB as extra unwanted columns.

@icio There are no tests I can see, but I might be missing something.

My date with gorm wil have to wait for another day. Closing this in favour of removing the offending protobuf.


juttle-test-utils.js - ensure program is always deactivated after test run

cc: @dmajda @demmer

Fixed tests that relied on delayed deactivation but did not set it up and also the 'deactivate' tests themselves - they do not need to call deactivate explicitly.

+1


WAMP Session IDs start at 1 per the RFC/spec text

Change documentation and implementation to use 1 as the
starting point, and a unit-test for wraparound. See also #541

Current coverage is 60.28%

Merging #542 into master will increase coverage by +0.05% as of 83f8692

@@            master    #542   diff @@
======================================
  Files           57      57
  Stmts         9032    9039     +7
  Branches      1460    1460
  Methods          0       0
======================================
+ Hit           5440    5449     +9
+ Partial        477     476     -1
+ Missed        3115    3114     -1

Review entire Coverage Diff as of 83f8692

Powered by Codecov. Updated on successful CI builds.

@hawkowl or @oberstet?


Replace deprecated registry.all() by registry.values()

The tests didn't pass with python 3.7. I think it was an issue in gym with the newest version of importlib-metadata
https://stackoverflow.com/questions/73929564/entrypoints-object-has-no-attribute-get-digital-ocean


MINOR: Add virtual env to Kafka system test README.md

@ewencp @granders Minor change to system test README.md

LGTM. Test failure is obviously unrelated.