"Generate Code" not work.

extension version: v3.1.0
operating system: mac os
problem:
I clicked the "Generate Code" command in the right click popup menu, after choosing the target template(e.g. Shell - cURL), nothing happened except a quickly flashed process notify.

Thank you for reporting. Do you get an error message. This would appear in the output in the httpyac - Log tab. Would be great. Otherwise I will try it out myself.

In my case, there is not any error message appear in the httpyacc - Log tab.
I forked your great job and tried it locally with my Macbook Air, finding that there may be something wrong with the method "generateCodeRequest" in (src/provider/harCommandsController.ts):

It never hit my breakpoint on the callback of context.httpClient during "Generate Code".
FYI, here is my testing code:
// gen_code_not_work.http
{{
    exports.date = "2021-09-10"
}}

GET https://httpbin.org/json HTTP/1.1
Date: {{date}}

This is exactly the right method. Probably the error is in httpyac, which does not use the httpClient correctly.
If you want to continue looking there, here's what you'd have to do.

git clone, npm install and npm link in httpyac folder
npm link httpyac in vscode-httpyac folder
modifypackage.json of httpyac (remove "types" and change "main" from dist to src)

{
  "main": "./src/index.ts",
  "types2": "./dist/index.d.ts",

modify tsconfig.json in vscode-httpyac (add httpyac to rootDir)

{
  "compilerOptions": {
     "rootDirs": ["src", "./node_modules/httpyac/src"],
  }
}

npm run watch in vscode-httpyac

After these changes, when vscode-httpyac is started, the httpyac source can also be debugged.
You don't necessarily have to find the bug. I'll be happy to take a look at it though, I'd just check out the support for GRPC before I do that.

The cause was that the httpClient was overwritten in the api. I took the opportunity and rewrote the logic to use logResponse interface. With version 3.2.0 the call should work correctly again


tflite_runtime cant install it?

yea im on pytthon 3.12 and cant install   tflite-runtime
any other way?

It's no longer available for Windows if that's your platform (source issue #144). Just use an ONNX model instead and it should work. Look at the fix in that issue.


[RFC] Make Doctrine 3 usable with a pool of connection

Feature Request

Q
A

New Feature
yes

RFC
yes

BC Break
maybe ?

Summary
Hey,
I know there is actually some heavy work on the v3 of Doctrine, and it would be nice if the next version would support having a connection pool. I have myself played a little with an async driver https://github.com/joelwurtz/doctrine-async (only a poc atm but can certainly be a reality), but this use case need to have multiple connection open (as you can have queries in parallel).
Actually this is done by considering a Connection like a pool of connection and it works greats for a lot of use case.
However there may be some use case where it does not, like getting the last insert id. I think there must be other use case but i don't know the internals well enough to make a list.
I'm wondering if this is something that can be taken into account for the next version ?

This is something to be proposed for doctrine/dbal, I suppose.
Closing here. If you need such an API, please start by suggesting a possible interface first.


Better CFG notation

I think the CFG
string =<EMAIL_ADDRESS>num = \-?[0-9]+(.[0-9]+)?;
ignore = \s|\n|\t|\r;

J = O | A | string | num;

O = "{" FS0 "}";
FS0 = | F FS1;
FS1 = | "," F FS1;
F = string ":" J;

A = "[" EL0 "]";
EL0 = | J EL1;
EL1 = | "," J EL1;

would look better if we copied ideas from Pareas and wrote it as
string =<EMAIL_ADDRESS>num = \-?[0-9]+(.[0-9]+)?;
ignore = \s|\n|\t|\r;

J [object] -> O;
J [array] -> A ;
J [string] -> string;
J [number] -> num;

O -> "{" FS0 "}";
FS0 -> ;
FS0 -> F FS1;
FS1 ->  ;
FS1 -> "," F FS1;
F -> string ":" J;

A -> "[" EL0 "]";
EL0 -> ;
EL0 -> J EL1;
EL1 -> ;
EL1 -> "," J EL1;

The stuff in square braces are optional names that can be assigned to productions, and which would be recognisable in the resulting CST. (Similarly, named terminals should also show up, but we don't need to change the CFG notation to enable that.)

This would make a lot of sense.

Specifically, alpacc should generate a sum type for named terminals, and one for named productions.


Autoroute (legacy) - blocks calls to controllers referenced in routes->add function

PHP Version
8.0
CodeIgniter4 Version
4.2.1
CodeIgniter4 Installation Method
Manual (zip or tar.gz)
Which operating systems have you tested for this bug?
Windows
Which server did you use?
apache
Database
mysql8
What happened?
in version 4.1.5 when you added a route in routes file
for example
$routes->add('testroute/(:any)', 'Testcontroller::view/$1');
then I could make calls to
testroute
and
Testcontroller
directly without issue
but now with 4.2.x
all calls not
Testcontroller
is now in the $protectedControllers variable due to it passing all routes to the first variable in the contructor.

Basically I did not read in the docs or understand from usage in the past that adding a route would block the controller from direct access ,as I need it to have direct access sometimes in this case it is a alternative view to the same controller thanks to the link, but now the alternative view is disabling the main controller
Steps to Reproduce
create any controller
add 2 methods one called
mainview
and
alternativeview
add a route
$routes->add('testroute/(:any)', 'Testcontroller::alternativeview/$1');
you should be able to access the view via
/testroute/
but all calls to
/testcontroller/mainview
is now disabled
Expected Output
controller referenced in views to still work
Anything else?
I will look at alternative code or ways around this, but just wanted to hightlight as a issue of concern

Checked older code
and past forloop only ran for the cli routes not all routes as current one, will adapt current loop on my side for cli only and test

Note comment in now code also indicates it meant to work as old for cli routes only, thus should be save todo it for that only

I can confirm only cli entry is the expected one, not my one added via ->add method

seems it is this line in getRoutes that make the ->add routes part of cli
I will try to register all other methods one by one to get past this issue then

Alternative config works, will go with that. thank you

@devorama You don't upload screenshot of text file. It is difficult to read, can' copy the text.
It is not good practice. You can just paste the text or GitHub provides how to link source code easily.
https://github.com/codeigniter4/CodeIgniter4/blob/b3ebf5bd6bbf5c28618a0d8d62f8994c0d91dda8/system/Router/AutoRouter.php#L130-L142

Thank you for reporting.
It seems this is because of the bug fix.

To prevent unexpected access from the web browser, if a controller is added to a cli route ($routes->cli()), all methods of that controller are no longer accessible via auto-routing.
https://codeigniter4.github.io/CodeIgniter4/changelogs/v4.2.0.html#behavior-changes

Anyway, $routes->add() is very dangerous and you should not use it.
Replacing it with HTTP verb method is good practice.


Generate geometries with steps

Describe the solution you'd like
A sample file where the user uploads an LDraw file and immediately sees the generated instructions. If satisfied, presses "Upload to server" and the three.js geometries are generated for upload to a server where the original LDraw file is no longer available.
Additional context
This suggestion is for allowing users to share building instructions without sharing the original LDraw file.
Other suggestions for making this work:

Compress the data stream
Add watermarks

This was done in the raw_geometry_files branch.


Create GithubActivityGraphGenerator.md

📝 Description
A web app which allows to create github activity graph and download it.
💣 Is this a breaking change (Yes/No):

[ ] Yes
[x] No

🚧 How to migrate?
Don't need.
📝 Additional Information
Hacktoberfest

Only labelled not merged

yep, please be patient and wait for my colleagues to review.
Have a good weekend


normalized openai embeddings embed_query

we weren't normalizing when embedding queries

Just trying to understand - what was broken before? It looks like previously, it only did the chunking and normalization if the length exceeds context. Now, it attempts to do so always. This implies that the normalization is in fact the change introduced. However, aren't OpenAI embeddings already normalized?
What motivated this change? Would those new tests have failed previously?

Hey, I think this commit introduced a breaking change. We are no longer embedding texts but embedding the tiktoken tokens. Which leads to failure when calling OpenAI, can somebody confirm?


a "monster" for dungeon master?

I was killed by a dungeon master. not a "monster"...

Curious bug https://github.com/MinetestForFun/server-minetestforfun/blob/72232087a618cbf947d9535ae7abc90d8926ecb9/mods/death_messages/init.lua#L119
I think it's because of the new attack type of the Dungeon Master, they can now attack directly with they arms players,  so if a players is dead without fireball and with a direct hit of a DM, the message shouldn't be good.
@LeMagnesium it's your baby, can you fix it please ? (if you can teak your code to works well with the Yeti, if im right, he can hit players like DM if there are close to it with his arms)


fix: remove placeholder text on focus

Thank you for your contribution to the KodaDot NFT gallery.
👇 _ Let's make a quick check before the contribution.
PR Type

[x] Bugfix
[ ] Feature
[ ] Refactoring

Context

[x] Closes #4347
[ ] Requires deployment <>

Before submitting pull request, please make sure:

[x] My contribution builds clean without any errors or warnings
[x] I've merged recent default branch -- main and I've no conflicts
[x] I've tried to respect high code quality standards
[x] I've didn't break any original functionality
[x] I've posted a screenshot of demonstrated change in this PR

Optional

[ ] I've tested it at </bsx/collection>
[ ] I've tested PR on mobile
[ ] I've written unit tests 🧪
[ ] I've found edge cases

Had issue bounty label?

[x] Fill up your KSM address: Payout

Community participation

[x] Are you at KodaDot Discord?

Screenshot 📸

[x] My fix has changed something on UI; a screenshot is best to understand changes for others.

Ref

#4324

pay 30 usd

ninja move

but our ninja payout failed, lemme resume that
pay 30

😍 Perfect, I’ve sent the payout
💵 $30 @ 24.65 USD/KSM ~ 1.217 $KSM
🧗 Caiv9TbPz68q5dC8EcHu5xKYPRnremimGzqmEejDFNpWWLG
🔗 0x4638353a1bc72db071ebb278bbd034d64dc6f861a2fc0fd8ec8d2b810536859c
🪅 Let’s grab another issue and get rewarded!
🪄 github.com/kodadot/nft-gallery/issues


feat: extend Message

Just like in the package 'got', the user should be able to do so that the user doesn't have to have many lines of the same code if they send a lot of those same messages. Allows a developer to add them as standards for an app.

Ok. After doing some evaluation, we already have this. Unlike got, this is a class based system. I updated unit testing and the docs to reflect this.

:tada: This issue has been resolved in version 3.0.0-beta.1 :tada:
The release is available on:

npm package (@develop dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Number cast is not working properly.

Code:
const partSchema = new mongoose.Schema({ origin: String, long: mongoose.Schema.Types.Long });
const Part = mongoose.model('Part', partSchema);
const partNumber = new Part({ origin: 'number', long:<PHONE_NUMBER>854775806 });
partNumber.save();
const partString = new Part({ origin: 'string', long: '9223372036854775806' });
partString.save();
console.log(partNumber);
console.log(partString);

Console:
{
id: 5f4d9c84eefc4057c89f8788,
origin: 'number',
long: Long { bsontype: 'Long', low: -1, high:<PHONE_NUMBER> }
}
{
id: 5f4d9c84eefc4057c89f8789,
origin: 'string',
long: Long { bsontype: 'Long', low: -2, high:<PHONE_NUMBER> }
}

MongoDB:
5f4d9c84eefc4057c89f8788 | number |<PHONE_NUMBER>854775807 | 0
5f4d9c84eefc4057c89f8789 | string |<PHONE_NUMBER>854775806 | 0

The issue is the NodeJs core itself that can't handle 17 digits number itself, so it is wrong right before the number is received by this lib.
My mistake, I'm very sorry!


User Experience

Just a collection of small things. The strike-through means it's been done.

Missing something closer to a native OS file explorer.

The "New Directory" button image should have a + instead of a *.

Don't display the "Android SDK" thing when people haven't checked the Android tickbox.

Reorganize the platforms, maybe?

Do not allow Shared in platforms if you don't have Server selected.

Both Desktop platform options should be one after another, rather than separated.

Remove the two Toggle buttons: most probably no one uses them.

Try to figure out how to indicate that there are clickable links that open a browser tab (like those "?").

About the image above: how to reconcile with the Advanced tab which also showcases Java version selection? I think it should be merged.
Prevent selection of the extra Languages if the HTML platform was selected?

The descriptions should probably be left-aligned, and then add a bit more padding in between the columns of the table.

Do not display "KTX" Third-party if "Kotlin" isn't select in Languages.

Increase the alpha of the background of the tooltips.

Remove Polski language translation.

Unclutter this based on previous selections: do not give a choice to select a Java version for the server if the Server platform wasn't selected.

Should we allow selection of the Gradle Wrapper version?

It's weird to have within the same dialog, at the top a gray "Generating", then right below a "Generating new project" in red, and then at the bottom a "Setup complete" in green, all at the same time. This could be reworked a bit.

All in all, the "flow" of the user's interaction could be rethought, and I think making it more of a "step by step" wizard-approach could eliminate a lot of complexity in terms of "if user has selected X, then don't show Y" and such. But that's a lot of work.

14 is already done, 4 and 6 are kinda odd because the platforms are in alphabetical order, 5 is weird because I don't really know what people use Shared for, uh... 16 Gradle wrapper should probably be mandated, it's a hassle otherwise... 17, 13, 11, 9, 7, agreed.  15, 12, 10, 3: there isn't a sequence these happen in; a user could select Kotlin and then select HTML, which would make a partly-non-building project, but they don't actually have to use Kotlin in their app, or could use it only on desktop. This would be fixed in part by 18, but I fairly often jump back to the first tab in practice when I'm setting up a game. 2 I disagree with; that's an accepted icon for making a new folder. 1 will be hard  -- TinyFileDialog may fix it, but I have no experience with it and it probably isn't a scene2d.ui widget. I'm opposed to using JavaFX just for a file dialog.

Can we have kotlin-based project templates? As I understand Dark Matter were making it themselves? And also build scripts look more clean than current one.

Hi, can this be closed after the UI changes or is there something still relevant in it?

I feel all these have been addressed in some fashion. Thanks.
To quote Payne: "But that's a lot of work."
It was. Oh god it was.

Hooray! Closing!


Fix issue with empty line in preformatted text

See https://github.com/tnorbye/kdoc-formatter/issues/23

Thanks!


Accessing child scope?

I am using ApplicationInsights and a job framework (Hangfire) in AspnetCore. Application Insights registers its components as singletons.
My job receives a new scope from DI to run in. I have my components registered in DI as scoped lifetime:
services.AddScoped(typeof(IMyJobContext), typeof(MyJobContext)).
I am creating an object through IServiceProvider.GetService(typeof(IMyJobContext)) in the job's scope.
Appinsights uses a concept of a TelemetryInitializer to process the created telemetry. I am not able to acquire the same IMyJobContext object in the TelemetryInitializer, as it is a singleton and therefore lives in a different (parent) scope.
I am looking for something similar to aspnet's IHttpContextAccessor.HttpContext.RequestServices.GetService(typeof(IMyRequestContext)). Is there anything in DI that would allow access from a singleton's scope to the more specific (child) scope?

A singleton is a singleton no matter which scope you resolve it from. You will always get the same instance :smile:

@khellang, I'm well aware of that... :) I am trying to access a scoped object (IMyJobContext) from the singleton (TelemetryInitializer).

You really don't want to do that. Having a service with a longer lifetime depend on a service with a shorter lifetime is bad news. The only way I can see you pulling that off would be to pass the actual IMyJobContext to a method on TelemetryInitializer and make sure the method returns before the scope is destroyed.

It's actually not that simple. I'll try to explain.
A method (run by Hangfire as a "job") sitting in a class with scoped lifetime calls ILogger.Log. Hangfire manages the scope for the job instance's lifetime. ApplicationInsights has a logprovider (wired to ILogger.Log) that calls TelemetryClient.TrackTrace (TelemetryClient is registered as a singleton) that in turn creates a new TraceTelemetry object and calls a method (InitializeTelemetry) on all registered TelemetryInitializers on the newly created telemetry item. TelemetryInitializers are object instances sitting in the TelemetryClient (so are in the root, singleton scope).
Now, in the TelemetryInitializer.InitializeTelemetry is where I have the first extension point. Here, I would need to read some object (correlationid) that was created in the job's scope. This is where I have found no way to do it, because:

I cannot modify how the job (Hangfire) creates the scope
I cannot modify the signature of ApplicationInsights' methods and constructors
I have found no way to obtain the serviceprovider for the current run instance (scope) of the job

Something very parallel to what I am trying to do happening with Aspnet requests:
A request scope is created for each request. The serviceprovider for the current request is accessible through IHttpContextAccessor.HttpContext.RequestServices. This serviceprovider can be used in a singleton's method to obtain a request scoped object and work with it.
Later I'll put together a small project to demonstrate the issue.

Are you trying to get access to the request scope or just an arbitrary scope?

The scope Hangfire created (so arbitary). The request scope is easily accessible through IHttpContextAccessor.HttpContext.RequestServices.

Quick question; do you have control over the job class? I'm thinking it's time to whip out some code soon, cause explaining what's happening isn't exactly easy. If this job is something that you implement, but Hangfire activates (through the container), can't you just inject IServiceProvider to get the provider of the current scope, i.e. the scope that Hangfire created?


Simplify the configuration procedure

Even with the bot now handling configuration, it is still a bit clunky and error-prone. Need to streamline this process.

Hi - I deployed on Heroku and added to my Discord server, but it still hasn't messaged me to complete the setup and add the Yahoo League ID. How long does it take before it should message me?

Having mods be able to configure the bot would be useful I think for larger guilds. I will open an enhancement for adding that support.


Next review iteration

This PR introduces next review iteration of Acra docs from my side.
Apart from traditional fixes of poor phrases, grammar and typos I also rephrased a bit data re-encryption subsection and moved it to more appropriate place, as we discussed in Slack channel.

@vixentael thanks for your assistance! I applied all your suggestions and returned back "automagically" word


curses.listener: fix, need stream-write1 after changes to stream-bl

a simple fix for curses.listener which wouldn't start at all because it didn't implement stream-write1. It is now needed since<PHONE_NUMBER>75f8777b273118b810db5463d5d47c

Merged, thanks!!


Images are not displayed

With all Clipy versions I have tried, up to 1.2.9 beta 12, images are not displayed in the menu, despite the following settings:

(images are however properly copied to Clippy and pasted from Clippy)

@afeblot  you should also check the Display icons in menu items

Ah, thanks, I get it now.
Do you think you'd be able (and willing) to display the images where you currently just print "(image)", instead of having them at the icon position?
Rationale:

The image is the actual copied content, so it would be consistent (and nicer) to have it located  at the same position as copied texts.
That would allow to have images displayed even if we hide icons (most of the time, icons don't bring so much value, they're almost identical and just bloat the window).


nnndpi resource folder qualifier not supporting arbitrary density values

Per official Android documentation, "drawable-200dpi" is a valid resource folder (https://developer.android.com/guide/topics/resources/providing-resources - see "nnndpi" definition). Gradle refuses to build project with resource folder with this name. If same project is built with Buck, it succeeds and the resource folder works as expected with the custom density on Android 10 device (which confirms that aapt/aosp handles this resource qualifier).
Expected Behavior
APK should be built.
Current Behavior
APK is not built, :packageReleaseResources fails.
22:03:23.522 [ERROR] [org.gradle.internal.buildevents.BuildExceptionReporter] Caused by: <project_dir>/res/drawable-200dpi: Error: Invalid resource directory name
22:03:23.522 [ERROR] [org.gradle.internal.buildevents.BuildExceptionReporter]   at com.android.ide.common.resources.MergingException.throwIfNonEmpty(MergingException.java:152)
22:03:23.522 [ERROR] [org.gradle.internal.buildevents.BuildExceptionReporter]   at com.android.ide.common.resources.DataSet.loadFromFiles(DataSet.java:259)
22:03:23.522 [ERROR] [org.gradle.internal.buildevents.BuildExceptionReporter]   at com.android.ide.common.resources.ResourceSet.loadFromFiles(ResourceSet.java:55)
22:03:23.523 [ERROR] [org.gradle.internal.buildevents.BuildExceptionReporter]   at com.android.build.gradle.tasks.MergeResources.doFullTaskAction(MergeResources.java:242)
Context
We are building for custom hardware, that specifically uses 1.25x density (=200dpi), hence the need for this custom resource folder qualifier. We don't want to add larger assets just to be downsized by the runtime.
Here's code pointer to custom dpi values handled by AAPT: https://android.googlesource.com/platform/frameworks/base/+/master/tools/aapt/AaptConfig.cpp#320
Here's where parsing fails in Gradle, it handles the nnndpi pattern but then tries to match to buckets, does not allow arbitrary value (which is not in line with the SDK documentation): https://android.googlesource.com/platform/tools/base/+/588c433884c745cba0e09eb16a7f269dfee2c1aa/sdk-common/src/main/java/com/android/ide/common/resources/configuration/DensityQualifier.java#78
Steps to Reproduce
Simply add a resource folder called "drawable-200dpi" and place any asset (like a png) inside.
Your Environment
Gradle 5.6.2
build tools/minsdk/targetsdk/app_platform=29

Thank you for your interest in Gradle!
This is an issue with the Android Gradle Plugin which is developed by Google and cannot be addressed in the core of the Gradle Build Tool.
Please open such Android-related issues on the Android Issue Tracker at https://source.android.com/source/report-bugs


Delete App failed with error "The current pipeline status does not allow update"

Summary
create a app and upload past 30 days events, and current pipeline status is Active in control plane and cloudformation, when click delete button it shows the error: "The current pipeline status does not allow update"
Steps to reproduce

create a pipeline
create a app
upload last 30 days events using standalone-data-generator python script
select the app, and click delete button
the error shows

What is the current bug behavior?
Can't delete the app which have already have some events, but the app which have no events, can be deleted expected.
What is the expected correct behavior?
The app can be delete successfully
Relevant logs and/or screenshots

Possible fixes

This is :bug: Bug Report

After about 20 minutes, click delete button, the page will refresh with no error, and the app still in the list, then delete twice the app can be delete. What's the prerequisites for delete a app?

It's already fixed in v1.1.0, which does not allow to delete the CREATING or UPDATING pipeline.


Metadata Registry create new metadata schema doesn't appear without reload

Describe the bug
On Metadata Registry page, when creating a new schema, the new schema is not listed after successful submit.
Name-field keeps old value.
To Reproduce
Create a new schema.
Expected behavior
New schema should list as last entry, both Namespace- and Name-fields should be empty.
Temporary solution
Reload.

@artlowel : Assigning to you (or your team to review).  To me, it sounds like a caching issue.  Placing this as low priority for now, as it works but is annoying

Hi @tdonohue, I want to take this task.


Use all space for ref contrib editor

maybe make the name field a big bigger than the given-names field. That way we make space for long organisation names
Remove 'add contributor' label, just the plus sign is enough I think

As I first shot I reduced a space for a remove button.


No provider for TranslateService!

I'm submitting a ...  (check one with "x")
[x] bug report => check the FAQ and search github for a similar issue or PR before submitting
[ ] support request => check the FAQ and search github for a similar issue before submitting
[ ] feature request

Current behavior
I followed the installation and usage steps for ngx-translate but I get this error :

Error: No provider for TranslateService!

Expected/desired behavior
App should run and translate values.
Reproduction of the problem
If the current behavior is a bug or you can illustrate your feature request better with an example, please provide the steps to reproduce and if possible a minimal demo of the problem via https://plnkr.co or similar. You can use this template as a starting point: http://plnkr.co/edit/tpl:01UjWY3TKfP6pgwXKuEa
What is the expected behavior?
Load the TranslateService and translate values from i18n files.
Here my modules and components code :
AppModule
import { NgModule } from '@angular/core';
import { BrowserModule } from '@angular/platform-browser';
import { HttpModule, Http } from '@angular/http';
import { FormsModule } from '@angular/forms';
import { TranslateModule, TranslateLoader } from '@ngx-translate/core';
import { TranslateHttpLoader } from '@ngx-translate/http-loader';
import { sharedConfig } from './app.module.shared';

// AoT requires an exported function for factories
export function HttpLoaderFactory(http: Http) {
    return new TranslateHttpLoader(http);
}

@NgModule({
    bootstrap: sharedConfig.bootstrap,
    declarations: sharedConfig.declarations,
    imports: [
        BrowserModule,
        FormsModule,
        HttpModule,
        TranslateModule.forRoot({
            loader: {
                provide: TranslateLoader,
                useFactory: HttpLoaderFactory,
                deps: [Http]
            }
        }),
        ...sharedConfig.imports
    ],
    providers: [
        { provide: 'ORIGIN_URL', useValue: location.origin }
    ]
})
export class AppModule {
}

AppModuleShared
import { NgModule } from '@angular/core';
import { RouterModule } from '@angular/router';

import { AppComponent } from './components/app/app.component'
import { NavMenuComponent } from './components/navmenu/navmenu.component';
import { HomeComponent } from './components/home/home.component';
import { FetchDataComponent } from './components/fetchdata/fetchdata.component';
import { CounterComponent } from './components/counter/counter.component';

export const sharedConfig: NgModule = {
    bootstrap: [ AppComponent ],
    declarations: [
        AppComponent,
        NavMenuComponent,
        CounterComponent,
        FetchDataComponent,
        HomeComponent
    ],
    imports: [
        RouterModule.forRoot([
            { path: '', redirectTo: 'home', pathMatch: 'full' },
            { path: 'home', component: HomeComponent },
            { path: 'counter', component: CounterComponent },
            { path: 'fetch-data', component: FetchDataComponent },
            { path: '**', redirectTo: 'home' }
        ])
    ]
};

AppComponent
import { Component, Inject, Injectable } from '@angular/core';
import { Http } from '@angular/http';
import { FormsModule, ReactiveFormsModule } from '@angular/forms';

import { TranslateService } from '@ngx-translate/core';
import { REQUEST } from '../../shared/request';

@Component({
    selector: 'app',
    templateUrl: './app.component.html',
    styleUrls: ['./app.component.css']
})

@Injectable()
export class AppComponent {
    constructor(public translate: TranslateService, @Inject(REQUEST) private request) {
        // this language will be used as a fallback when a translation isn't found in the current language
        translate.setDefaultLang('en');

        // the lang to use, if the lang isn't available, it will use the current loader to get them
        translate.use('en');

        console.log(`What's our REQUEST Object look like?`);
        console.log(`The Request object only really exists on the Server, but on the Browser we can at least see Cookies`);
        console.log(this.request);
    }
}

What is the motivation / use case for changing the behavior?
Please tell us about your environment:
Mac OS
Visual Studio for Mac

ngx-translate version: 6.0.1

Angular version: 4.1.3

Browser: [all | Chrome XX | Firefox XX | IE XX | Safari XX | Mobile Chrome XX | Android X.X Web Browser | iOS XX Safari | iOS XX UIWebView | iOS XX WKWebView ]

I think you forgot to to add TranslateService to your AppModule's providers array.

I started over, it works know !
Thank you @trumbitta

I was getting "Error: No provider for t" during AoT compilation and found out that was happening because I had
constructor(
   translate: TranslateService
  ) {

in my component, but in the same time pipes were working. I replaced direct injection of TranslateService with
constructor(
    protected injector: Injector
  ) {

and
ngOnInit() {
    this.injector.get(TranslateService).get(...

in ngOnInit. Now it's working. I hope that will help to somebody. Still not sure why AoT build not working properly when you injecting directly, I think related to when TranslateService created/provided.

I am getting an error only on running 'ng serve --aot' any update on this issue.

Can you post your app-module.ts ?

请问问题解决了么，怎么解决的


chore: update deps

Part of https://github.com/ooni/probe/issues/2556

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
2407

Relevant Lines:
2841

💛  - Coveralls


Implement a radix-3 FFT

Easy enough implementation of radix-3 FFT. This will improve the performance and, hopefully, make some cases easier to beat FFTW on.

Check out #29


Having trouble mocking websockets.connect in python 35/36

I'm trying to get a websockets/asyncio unittest that was working under python3.4 to work correctly in python3.5.   The websockets author is exposing a class as a coroutine when run in python 35.  I'm trying to determine how to do mocking correctly, and if asynctest can assist in this effort.
I have double-raised the issue under websockets, since I am unsure where the issue would have to be resolved.   https://github.com/aaugustin/websockets/issues/359
I have enclosed a simplified version of the test for discussion purposes.
asynctest == 0.5.0
websockets == 4.0.1
protocol = yield from websockets.connect('ws://server:1234')
RuntimeError: Task <Task pending coro=<mocked_connect() running at/test_websocket_mock.py:17>> got Future <MagicMock id='140610498838480'> attached to a different loop

asynctest_websock_failure.txt

Update, with hints from the websocket developers, and with a patch to asynctest I was able to get my tests passing.   I have raised https://github.com/Martiusweb/asynctest/issues/87 to cover ongoing asynctest-related issues being seen.


EstimateTradeBasedOnPriceImpact Query

What is the purpose of the change

Add a description of the overall background and high level changes that this PR introduces

Added an EstimateTradeBasedOnPriceImpact query to the x/poolmanager/query_proto_wrap.go, the goal of this query is to help external blockchains/smart-contracts to fill in the MsgSwapExactAmountIn details by retrieving input and output values for a trade. Given a total input amount, a maximum price impact, and an external price reference this query should return an input and output trade that respects the price impact of a pool.
This query operates in a binary search way where inputs are recursively tried and reduced each time if the price impact of the output is not met. If no possible trade is found then the outputs would be zero.
Testing and Verifying
This change added tests and can be verified as follows:

*Added unit test that validates the query reacts as needed for the Balancer/StableSwap and Concentrated liquidity pool types under different input conditions.

Documentation and Release Note

[X] Does this pull request introduce a new feature or user-facing behavior changes?
[x] Changelog entry added to Unreleased section of CHANGELOG.md?

Where is the change documented?

[X] Specification (x/{module}/README.md)
[ ] Osmosis documentation site
[X] Code comments?
[ ] N/A

@VitalyVolozhinov Thanks for the PR! Would you be able to rebase your branch to current main so we have a clean git history for this PR?

The rebase is messy causing some breaking code changes for me.

@VitalyVolozhinov No worries, I can review it as of rn!

@mattverse pinging as a reminder, for re-review thank you!

@VitalyVolozhinov Can you take a look at the branching here? Seems like there are conflicts that must be resolved before merging 👀

Nice work! This would definitely be helpful for so much people, thanks for the contribution!
Before we merge, could we do two following things:

Add proto doc + docs in other places about invalid estimation returning 0 instead of erroring
Have manual QA of the query being added? (cli or lcd does not matter)

Once this is done this should be good to go. Again, thanks so much for this 🔥

I think the new changes may have broken the localnet  and I cannot run it but I have previously tested the cli on CL pools doing the following:
Running the code using local-net commands:

make localnet-init
make localnet-start
make localnet-keys
make localnet-cl-create-pool
make localnet-cl-create-positions

We then query the pool with various inputs to see what the responses are using the following command:
osmosisd query poolmanager estimate-trade-amount-in-amount-out-based-on-price-impact<PHONE_NUMBER>0000000uosmo uusdc 1 0.001 2.49

We can see as increase the input of uosmo it gives us appropriate input and output amounts to trade under the given slippage and twap value.

@VitalyVolozhinov Can you run make proto-all + solve linter issues please?

@VitalyVolozhinov Can you run make proto-all + solve linter issues please?

All issues should be solved

Looking forward to seeing this merged 💪

Not sure why the proto file generation failed it had passed on our side: https://github.com/entrypoint-zone/osmosis/actions/runs/6296358703/job/17091183632

@VitalyVolozhinov Did you try generating proto after merging with the latest main? Protobuf gen would often times fail when generated without having the branch merged with latest main, could be the problem 🤔 Try and lmk! If it doesn't seem to work, happy to merge for now and then re-protogen myself

@VitalyVolozhinov Did you try generating proto after merging with the latest main? Protobuf gen would often times fail when generated without having the branch merged with latest main, could be the problem 🤔 Try and lmk! If it doesn't seem to work, happy to merge for now and then re-protogen myself

Yes, and it works on our side: https://github.com/entrypoint-zone/osmosis/actions/runs/6348615589/job/17245570892 not sure what is happening

Hi @mattverse, this looks amazing. When will it be available for CLI?


Allow remove table view cell

Now we can delete cells if we want
You can see a simple example in this project
perhaps, if this you like, I could add other optional callback for move, too
If you have any tip would do another PR

sorry, this is a dirty PR, I close it


Valid MySQL escape characters likely to cause 'IndexError: list index out of range'

Building CTags for D:\broken.sql: Please be patient
error: An unknown error occured.
Check the console for info.
Exception in thread Thread-454:
Traceback (most recent call last):
  File "./threading.py", line 901, in _bootstrap_inner
  File "./threading.py", line 858, in run
  File "ctagsplugin in C:\Users\user\AppData\Roaming\Sublime Text 3\Installed Packages\CTags.sublime-package", line 119, in run
  File "ctagsplugin in C:\Users\user\AppData\Roaming\Sublime Text 3\Installed Packages\CTags.sublime-package", line 907, in build_ctags
  File "ctagsplugin in C:\Users\user\AppData\Roaming\Sublime Text 3\Installed Packages\CTags.sublime-package", line 892, in build_ctags
  File "ctags in C:\Users\user\AppData\Roaming\Sublime Text 3\Installed Packages\CTags.sublime-package", line 331, in build_ctags
  File "ctags in C:\Users\user\AppData\Roaming\Sublime Text 3\Installed Packages\CTags.sublime-package", line 365, in resort_ctags
IndexError: list index out of range

Environment:

Windows
ctags58.exe
Sublime Text 3 - Stable Build 3065

Minimal repro case:
INSERT INTO X (a) VALUES ('\'');
INSERT INTO X (a) VALUES ('DOMAIN');
INSERT INTO X (a) VALUES ('');

DOMAIN is a valid SQL token that happened to be in the original file as a string value.
Other SQL tokens I've tried don't lead to this issue.
Theory: Incorrect handling of backlash escape sequences.

I'll do debugging against head when I have time, unless someone beats me to it. ;)

ctags58.exe produces some trailing garbage ); in the tags file for the repro case above. This looks more like an exuberant-ctags bug.
However, should Sublime CTags try to work around the issue in the meantime?

Related PR: https://github.com/SublimeText/CTags/pull/279


specify php version compat

Is your feature request related to a problem? Please describe.
does not install en php 7.2
Describe the solution you'd like
just have the info on the README ;)
Describe alternatives you've considered
none
Additional context
first try

Thanks for the report!
I'm currently working on the README so I'll add this in the day.

Should now be fixed by https://github.com/Kaudaj/prestashop-maker/commit/b07a2595d40828a8c109ab5fcc65cb500f65a5ca!


Navigating suggestions with <C-n>, <C-p> does not work properly

Make sure you have done the following

[X] I have updated to the latest version of blink.cmp
[X] I have read the README

Bug Description
When navigating between the completion suggestions with <C-n>, <C-p> the panel and therefor the selection breaks.
It seems to open another suggestion panel and then navigate in that? This video probably dose a better job at explaining what I mean:
blink_bug.webm
Relevant configuration
return {
  "saghen/blink.cmp",
  build = "cargo build --release",
  opts = {
    keymap = {
      ["<cr>"] = { "select_and_accept", "fallback" },
    },
    highlight = {
      use_nvim_cmp_as_default = true,
    },
    accept = {
      auto_brackets = { enabled = true },
    },
    trigger = {
      signature_help = { enabled = true },
    },
  },
}

neovim version
NVIM v0.10.2 Build type: RelWithDebInfo LuaJIT 2.1.1725453128 Run "nvim -V1 -v" for more info
blink.cmp version: branch, tag, or commit
main

When not using a preset, you have to define all the keys you'd like to configure. You can copy the preset from the README's default configuration section. I'll make this more clear in the docs

That makes sense, thanks. It would be cool to have some sort of helper function to merge the preset with your custom overwrites something like in nvim-cmp:
mapping = cmp.mapping.preset.insert({
  ["<cr>"] = cmp.mapping.confirm({
    behavior = cmp.ConfirmBehavior.Replace,
    select = true,
  })
}),

I feel like this would be more in line with the "Works out of the box with no additional configuration" goal of the plugins.


feat: Modify provider RPC error log severity from error to warn

The error middleware injected in the provider-side json-rpc-engine was calling log.error() (in practice, log = console) every time it observed a JSON-RPC error response.
We received reports that this error log is caught by the monitoring infra of some dapps. It is ultimately the caller's responsibility to handle errors where they call ethereum.request(), and they can always console.error() at that point themselves. In light of the above, we decrease the severity of our own log statement to warn.

The stretch-goal version of this would be to make logging configurable via some method on the provider object (constructor arguments don't help most of our consumers), but that's definitely a stretch goal.

@rekmarks Going ahead and merging this to get it into next release FYI


Issue/Docs: Dead links and domain hijacking?

The "Compile-time Requirements" section contains a few dead links. The Win-builds link now goes to someones blog "if im wrong my bad!" when I think it should lead to here https://github.com/alex85k/winbuilds/blob/master/README.md
Another issue is the "here" link for MSYS2, the section/header it refers to no longer exists https://github.com/rust-lang/rust#building-on-windows.
The following looks like a good replacement.
https://github.com/rust-lang/rust/blob/master/INSTALL.md#building-on-windows

https://docs.rs/cc/latest/cc/index.html#compile-time-requirements

Thank you, I'll open a PR immediately for this

Thank you, I'll open a PR immediately for this

You removed the new win-builds link from your first commit. Was this on pourpose?


Fix legacy commands

Fix #413.
This fix will correctly map the legacy v1.* commands to the new v2.* commands, with backwards-compatible syntax.
Where necessary, the new commands should be modified to reproduce the legacy behavior when run in legacy mode. Right now, I think the only place we need to do this is aqt tool, where the legacy command would only install a tool if it matches the supplied version.
Right now, all I have are tests; implementation will follow.
Edit: forgot to implement legacy aqt tool behavior
Edit: implemented legacy aqt tool behavior that fails to install the tool when the version numbers don't match. Also re-implemented legacy bug that prevents installation of tools that don't have a valid semantic version. 💩 Hopefully, users who experience this bug will be motivated to use the new aqt install-tool.
We can roll back c5b89f6 if you want to keep the bug fixed, but this will remove the "fail on wrong version" behavior.

Thank you!!!!


Settings API without authentification

Hi,
I found out that the api/settings endpoint isn't protected by authentication, so everyone can access to the slack endpoint.
Is this normal ?
Thanks

Hi @YoannMa ,
thanks for the notice.
Theese settings started out as public, containing stuff like wether user signup should be enabled. The issue you posted was raised with the later integration of slack. And it is valid. I will need to either put slack settings in their own logic or somehow separate public from private settings.
This is a Todo.

@YoannMa , this is fixed starting with Konga 0.11


Implement queries for all storage plugins

[x] Range query
[x] Exact match
[x] Starts-with query

Done!


Delta does not appear to respect environment vars by default

Describe the bug
Delta doesn't appear to actually respond to the env vars I'm passing it
OS: linux
--
2023-04-29 7:34:29 pm | Architecture: amd64
2023-04-29 7:34:29 pm | Hostname: testcustomer-delta-0
2023-04-29 7:34:29 pm | Starting Delta daemon...
2023-04-29 7:34:29 pm | Setting up the whypfs node...
2023-04-29 7:34:29 pm | repo: .whypfs
2023-04-29 7:34:29 pm | walletDir: ./wallet_estuary
2023-04-29 7:34:29 pm | mode: cluster

I'm setting repo and wallet dir, but not passing them as CLI flags, and it doesn't respect them, they should be
  config.env: >
    # Node info

    NODE_NAME=delta-node

    NODE_DESCRIPTION=Experimental Deal Maker

    NODE_TYPE=delta-main

    REPO=/storage/whypfs

    DELTA_WALLET_DIR=/storage/wallet

    # Database configuration

    MODE=cluster

    DB_NAME=deltadb

<EMAIL_ADDRESS>
Delta Info
Latest Delta as of this post, running in k8s

This should be fixed on the latest commit. Please re-open if that's not the case.


add dxPolicy that gives easy properties / policy fallback

goal would be to be able to use, in htl scripts ${dxPolicy.text} where text would first be looked up in component properties, then in bindings content policy

done with #31

https://github.com/apache/sling-org-apache-sling-api/pull/21


new release

please make a new release so that we can use a tagged version
thanks for the container

There already a stable and latest tag.
Is not enough ?

I think the point is people want to control which version they are using. Just using latest or stable you never know for sure whether your container changes on restarting.

Ok i see, i will add version tag

thx this was exactly the problem


evaluation on partial test set

Hello @HYPJUDY,
It seems that, you are not doing evaluation on the THUMOS'14 full test set. As you reported in your paper, THUMOS'14 detection task is evaluated on 213 test videos. But, your test window_info.log  file is missing window info for the following 3 test videos, as your thumos14_test_annotation.csv is missing annotations for those videos. As a result, you are basically evaluating your model on 210 test videos instead of 213.
video_test_0000270
video_test_0001292
video_test_0001496
Can you please comment on why this is the case?
Thanks much.

Hi @rahman-mdatiqur , I am happy to clarify this issue.
TL; DR: The original source of thumos14_test_annotation.csv leaves these three videos out, because it seems that these three videos have incorrect annotations which are longer than their duration.
Details:

thumos14_test_annotation.csv is generated by data/gen_thumos14_anno.py, which is based on SSN. (see Preprocess Data by Yourself)

The ground truth file from SSN is thumos14_tag_proposal_list/thumos14_tag_test_proposal_list.txt, which do not contains these three videos. (see data/gen_thumos14_anno.py)

I also have noticed this issue two years ago (see https://github.com/yjxiong/action-detection/issues/31) and would like to share more detailed information on these three videos:

thumos14_tag_test_normalized_proposal_list.txt has 200 videos while there are 213 videos in TH14_Temporal_Annotations_Test\xgtf_renamed. Two reasonable missing videos are video_test_0000270 (its annotationa are HammerThrow but its ground truth in video is HairCut which doesn't belong to the 20 classes) and video_test_0001292 (it only has ambiguous annotations).
It seems that another missing video video_test_0001496 can be included into test list after modifying the annotations (annotations are CricketShot while ground truth is FrisbeeCatch).

Another related issue FYI: https://github.com/yjxiong/action-detection/issues/13

The original replies to this issue in SSN:

Their annotation seems to be longer than the duration of the videos. It is better to leave them out.

If I remember correctly, these 3 videos have incorrect annotations which are sitting beyond the videos’ time span.

Hello @HYPJUDY,
Thanks again for your quick and wonderful response.
It eventually directs me to raise the following concern.
Since you do not mention in your paper that you are evaluating on 210 videos instead of 213, how does it fare to compare your method in Table. 2 against other SOTA methods that report results on 213 test videos? I mean, does leaving those test videos out from the evaluation give you any advantage over the other SOTA methods in terms of mAP? I know that you are not removing the corresponding annotations from the ground-truth annotations located in https://github.com/HYPJUDY/Decouple-SSAD/tree/master/EvalKit/THUMOS14_evalkit_20150930/annotation. But, I did not check the evaluation script to see if it would be advantageous/disadvantageous to leave some videos out from the evaluation set.
Can you please comment on this?
Thanks in advance.

Hi @rahman-mdatiqur , thanks for your good question.
I only a quick look on the evaluation script since I do not have enough time currently.
https://github.com/HYPJUDY/Decouple-SSAD/blob/e3b65392cc4be7dd3887bd38996792ce6dc698f0/EvalKit/THUMOS14_evalkit_20150930/TH14evalDet.m#L135
It seems that the code only evaluate the common videos of ground truths and detected results. So if the model can produce good (bad) results for these three videos, then the map should be better (worse) by incorporating their results.
If your code is ready, you can quickly validate these by ablation experiments:

Remove more videos from the result file to see the change of performance.
Remove the ground truth annotations from the EvalKit to see the change of perfomance.
Including the evaluation result of these three videos to see the change of performance.

I think if the annotations of some videos are obviously wrong, then we should exclude them. Otherwise the overall result is not correct and the evaluation on these wrong annotated videos is meaningless. I should have clarified the video number (210) in paper. Thanks for you reminding.

Thanks @HYPJUDY for suggesting ways to evaluate the effect of excluding videos from the predictions list. As you said, since doing good(bad) on these videos may improve(downgrade) final<EMAIL_ADDRESS>and  since the SOTA methods report results on all 213 videos without making any modifications to the ground-truth annotations, I believe, new methods should follow the same protocol when comparing with SOTA methods, or mention the video numbers while comparing.
Thanks much for all the thoughts and helpful feedback.


Login

Does the login button not work?

@danicano10 we'll ping you when the cookie-cutter is out!


websocket run but dispatch event have exception

websocket running fine.
on client can connect to server but when server dispatch an event have error
Illuminate\Broadcasting\BroadcastException: Pusher error: . in /media/data/laravel/vendor/laravel/framework/src/Illuminate/Broadcasting/Broadcasters/PusherBroadcaster.php:128
broadcasting.php
            'driver' => 'pusher',
            'key' => env('PUSHER_APP_KEY'),
            'secret' => env('PUSHER_APP_SECRET'),
            'app_id' => env('PUSHER_APP_ID'),
            'options' => [
                'cluster' => env('PUSHER_APP_CLUSTER'),
                'host' => '<IP_ADDRESS>',
                'port' => 6001,
                'scheme' => env('BROADCAST_SCHEME'),
                'curl_options' => [
                    CURLOPT_SSL_VERIFYHOST => 0,
                    CURLOPT_SSL_VERIFYPEER => 0,
                ]
            ],
        ],

boostrap.js
    broadcaster: 'pusher',
    key: process.env.MIX_PUSHER_APP_KEY,
    wsHost: window.location.hostname,
    wsPort: 6001,
    wssPort: 6001,
    disableStats: true,
    forceTLS: true
});```

i have find the problem
on .env i have set this
PUSHER_APP_ID="${APP_NAME}"
PUSHER_APP_KEY="${APP_NAME}_key"
PUSHER_APP_SECRET="${APP_NAME}_secret"

the problem is the APP_NAME on .env
cannot have space on the name
if the name was Laravel Nova would have this issue
if the name was Laravel_Nova dont have issue


Do not enable elemental-system-agent.service

We should prevent having enabled services within the image unless we are fully confident we want them to be always enabled and running. Having services enabled just for the bootstrap phase is risky IMHO, as this is written within the immutable filesystem, we should only call the service when needed without enabling it.
Signed-off-by: David Cassany<EMAIL_ADDRESS>

This is related to rancher/elemental-operator#86

This needs to be followed-up in https://build.opensuse.org/package/view_file/isv:Rancher:Elemental/systemd-presets-branding-Elemental/50-default-Elemental.preset?expand=1

This needs to be followed-up in https://build.opensuse.org/package/view_file/isv:Rancher:Elemental/systemd-presets-branding-Elemental/50-default-Elemental.preset?expand=1

Here it is https://build.opensuse.org/request/show/992532


helm repo url is not valid

Hello,
I'm trying to add kured helm repo but the chart repository is not valid
helm repo add kured  https://weaveworks.github.io/kured
Error: looks like "https://weaveworks.github.io/kured" is not a valid chart repository or cannot be reached: failed to fetch https://weaveworks.github.io/kured/index.yaml : 404 Not Found
I'm i using a wrong repo ?

Yes, we have moved.
Can you try: https://kubereboot.github.io/kured/?

Thanks @dholbach  I'm able to add it , please update the doc accordingly .

Where did you find the doc with the old link?

https://artifacthub.io/packages/helm/kured/kured

Hum... not sure who added us there.
@ckotzbauer Do you know if this is something we can fix?

We have to register the kubereboot organization on artifacthub and take over the kured repo. https://artifacthub.io/docs/topics/repositories/helm-charts/


Keep pymd evaluation happening unless source is newer

Since this part of the docs build takes a while, would be good to skip files that haven't changed.

Been working out great!


Ask for cannolo images only in context menu cannolo

If "Load image" is pressed, use the default image of the network settings.
Make a new context menu action ("Load specific system") for the disk table.

Closing as suggested by @il-palmi


How to pass parameters into template!?

I'm running in circles here...
I want to pass parameters DURING CONTAINER RUNTIME from a hostsList.csv to a nginx.tmpl and generate a "default.conf" like so:
docker-gen nginx.tmpl default.conf
The only way I saw people pass parameters to a template is with "environment variables":
{{ range $myvar, $containers := groupByMulti $ "Env.MYENVVAR" "," }}
	#I wanna print this: {{ $myvar }}
{{ end }}

My goal is to modify the "hostsList.csv" to load hosts on the fly while the container is running and compose my default.conf with new hosts.
But Docker is setup in a way that environment variables are set in stones... I can't find how to modify them once the container run.
Is there another way to pass parameters to the template other than environment variables!?
Something like
docker-gen nginx.tmpl default.conf -params MYVARLIST="111111.com,22222.com,3333.com"
Or at least How can I modify the environment variables at runtime? -_-"
Thank you for your time! :D

I think something like this can work?
template:
{{ .Env.MYVARLIST }}

command:
MYVARLIST="111,222,333" docker-gen my.tmpl


updating joi to 8.x

Using nsp within atom throws a csp error(eval). This was due to module isemail which is used by joi. This was fixed in a later version of isemail which was picked by >7.x of joi

we can't merge this just yet as the nsp client still needs to support node versions 0.10 and 0.12. the newest joi supported in those versions of node is the 6.x that we're on.
there will be a new release later this week that i plan to remove support for 0.10/0.12 in and this will get updated at that time.


Add a user-agent string

What is the desired enhancement?
Every net-friendly tool should use user agents
Add a valid user-agent header that indicates the request is from qiskit_ionq, and a particular version

@Cynocracy / @ColemanCollins Could you assign the issue to me?

@WingCode you got it


17.03.x

- What I did
- How I did it
- How to verify it
- Description for the changelog

- A picture of a cute animal (not mandatory but encouraged)

Please sign your commits following these rules:
https://github.com/docker/docker/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "17.03.x" git@github.com:docker/docker.git somewhere
$ cd somewhere
$ git rebase -i HEAD~842354026600
editor opens
change each 'pick' to 'edit'
save the file and quit
$ git commit --amend -s --no-edit
$ git rebase --continue # and repeat the amend for each commit
$ git push -f

Amending updates the existing PR. You DO NOT need to open a new one.


Improve JarLibrary Class to remove duplicates

Purpose

Describe the problems, issues, or needs driving this feature/fix and include links to related issues.

Fixes #34357 #34412
Approach

Describe how you are implementing the solutions along with the design details.

Samples

Provide high-level details about the samples related to this feature.

Remarks

List any other known issues, related PRs, TODO items, or any other notes related to the PR.

Check List

[ ] Read the Contributing Guide
[ ] Updated Change Log
[ ] Checked Tooling Support (#)
[ ] Added necessary tests

[ ] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[ ] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

Closing since the same changes are merged to the 2201.1.x branch


Add extension which adds bookmarks to Raindrop.io service

This extension allows the user to highlight a URL and add it to the Raindrop.io bookmarking service.
The API documentation for importing a bookmark into Raindrop.io is found here.

Merged (finally)! Thanks for this.


IPFS compatibility, ipfs:// schema support

Hi there!
First off: really cool project. But I'm not entirely sure if I understand it. Some time ago, I was hoping for an embeddable IPFS implementation that would help me deal with 404-resilience for my project (Atomic Data). The existing Rust IPFS implementations unfortunately stagnated, so I was really glad you started working on iroh!
I just ran iroh on my machine cargo run provide ./src, and that seemed to work well and return a hash of all files, but I kind of expected it to return a resolvable ipfs address for this folder. Where is the ipfs://{CID} URL?
Maybe this project does not (yet?) support IPFS urls, but then I don't really get why it mentions IPFS.
Anyways, awesome to see you guys are making lightweight CID based P2P storage possible!

Made a discussion post (#955) about this. Iroh as it was pre-0.3.0 supported grabbing data using IPFS hashes but, AFAIK, that is no longer possible due to breaking changes made since then. I could be wrong?

If it does not support the IPFS schema, that's fine, but I think then a couple of things would need to happen:

Edit project description, make it clear that this is not ipfs but something similar
Add a different scheme and specify it (e.g. iroh://{hash})

Apologies for the delay on this.

I just ran iroh on my machine cargo run provide ./src, and that seemed to work well and return a hash of all files, but I kind of expected it to return a resolvable ipfs address for this folder. Where is the ipfs://{CID} URL?

We aren't doing a good enough job managing expectations here, hopefully this helps:
You can't currently use ipfs:// URI resolution because most of the time ipfs:// URLS are resolved with kubo, which doesn't currently support the data structures Iroh produces. In the coming months Iroh will have a version of a gateway that will let you resolve the hashes iroh produces over HTTP.
Once we stabilize the iroh spec, we'll work on contributing support to kubo. If we can land support there you'll be able to resolve iroh hashes at https://ipfs.io.  as for ipfs:// URIs, The exact definition of what ipfs:// means is still being nailed down, even out side of any iroh interop discussion. Once the dust settles there we can give you a firm answer on weather iroh hashes will work with ipfs:// URIs specifically.
We call Iroh IPFS because we're trying to iterate on an improved design of IPFS, not an IPFS-like-thing. Iroh is fully content-addressed, uses CIDs, and is fully peer-2-peer. We've made numerous different design decisions under the hood to build on lessons we've learned after numerous years of work in the community.  Long term, we want to end in a place that is both performant and compatible, it's just going to take time.
Hope that helps!

If I'm understanding this correctly, you're saying that the ipfs: schema is currently not entirely specified (still being nailed down), but your current implementation of iroh is incompatible with IPFS's current implementation (kubo).

Yep. I think that's totally right. We'd like it to be compatible one day, but I don't want to promise you something we haven't  figured out how to deliver.

I'd still think that it might be better to have an iroh: schema that is fully compatible with your implementation - even if in the long run, it works the same as ipfs:

My biggest concern here is further exacerbating my fumbling on messaging that we see iroh as an IPFS system. Adding an iroh scheme runs the risk of driving kubo & iroh farther apart, where long term we'd very much want both IPFS to converge.
But, I Hear you loud & clear on this, and I think your point has merit. If we look at more targeted use cases similar to github:// URLs that open the github desktop application from the browser, I think we should absolutely reserve the iroh scheme for a similar highly-targeted use case of telling an application "you can use the iroh protocol for this".
But I'm concerned about folks being confused about when to use ipfs and when to use iroh URIs, which is the exact confusion you've run into here, but having a URI would kinda cement a divide that shouldn't need to be there. This is really a thing we should talk about with other IPFS implementations & nail down an answer on. If we can at all get support for iroh hash resolution in the ipfs schema spec, it would be way more useful for the URI-dissemination use cases we see, like the common case of IPFS references from blockchains.
I'll put some time into specifying what an iroh scheme might entail in the coming weeks so we all have something concrete to look at, then take that to the IPFS Implementers working group & see what they think about it's inclusion in the ipfs scheme.
Apologies for this being so convoluted. Paving the future in the open across numerous orgs is messy.

Thanks again for taking the time to write such a thorough response, @b5. I applaud your effort in merging both projects and preventing a fork / split. Fully understand your considerations.

Thanks so much @joepio. I'll keep this issue open until we figure out a path forward on ipfs and iroh schemes.

Closing as stale


ROC-1458: Implement nightly ZAP job

JIRA link (if applicable)
https://tools.hmcts.net/jira/browse/ROC-1458
Change description
Adds an overnight ZAP security scan. Executes using integration tests against a dockerized environment.

I removed active scanning script and pipeline stage as they were redundant. ZAP will perform active scanning when on each page it opens when started up in attack mode, which we do: https://github.com/zaproxy/zap-core-help/wiki/HelpStartConceptsModes


Re-structuring CC folder, shifting CC Response code to Response_Theory,  optimization, cleanup etc..

Description
This PR intends to re-structure CC folder to make it easier to add new reference implementations
in appropriate directories. I am planning to shift the RHF-CC response specific code to the new Response_theory folder. Also, the code needs clean-ups, comments and references to make
it easy to follow.
What are your new additions? Please provide a brief list.

New Features

[X] specific rotation in length, velocity and modified velocity gauges.

Changes

[X] Restructure CC directory
[X] Shift Response code
[X] Optimization and clean-up
[X] References and comments

Status

[x] Click when ready for review-and-merge

@dgasmith this might be ready for a review.


Allow a series to be displayed as a bar graph

Provide an option to draw a series as a bar graph.

Bar graphs work fine, closing this issue.


wpt.runTest () is timing out

Result:
{ [Error: connect ETIMEDOUT] code: 'ETIMEDOUT', errno: 'ETIMEDOUT', syscall: 'co
nnect' }
Running:
var wpt = new WebPageTest('www.webpagetest.org', '[my key here]');
wpt.runTest('www.twitter.com', function callback(err, data) {
console.log(err || data);
});
Not sure what's up, but getting nothing but timeout. Its a windows machine running Node.

@bousquetcm I'm also getting this timeout error from time to time (runTest fails 30% of the time). Can you elaborate about your solution?


node-forge vulnerability

util.setPath doesn't appear to be used by rdf-canonize, which is the dependant

(Only used in a browser, not Node.js)

Alert seemed to reopen so closing as 'Vulnerable code is not actually used' again.

https://github.com/digitalbazaar/rdf-canonize/pull/29 was closed without merging because node-forge got up to date on their master branch.


Multiple MMKV Instances

Thank you for the plugin!
It would be nice if this supported multiple MMKV instances. Right now data only shows up for 1 instance and it seems random which one it picks.
e.g.
const appPrefs = new MMKVLoader()
  .withInstanceID('appPrefs')
  .initialize();
const userData = new MMKVLoader()
  .withInstanceID('userData')
  .withEncryption()
  .initialize();

if (__DEV__) {
  // both should show up in flipper
  mmkvFlipper(prefs);
  mmkvFlipper(encryptedData);
}

Hi @georgeflug, I just published v1.1.0 to support multiple MMKV instances. Please update your code to this and test:
if (__DEV__) {
  mmkvFlipper([prefs, encryptedData]);
  // Put array of instances ^
}

If it's okay, you can close issue.

Thank you very much!


Multiple tasks

👍

I've reviewed the changes in this merge request and I'd like to express some concerns and suggestions regarding the current implementation:
Custom Solution within Parallel Core: The solution appears to be tailored specifically to address a particular problem directly within the Parallel core. While this approach provides a direct fix, it might not be the most sustainable or reusable method. Implementing such custom solutions could lead to challenges in maintaining the code and might limit the flexibility for future enhancements or different use cases.
Modification of the addTask Function: I suggest revising the addTask function to enhance its utility and flexibility. Ideally, the function should allow for modification of the parallel tasks during its invocation (e.g., through a ->addTask() method). This change would provide a more dynamic approach to handling tasks, offering better control and customization based on different requirements or contexts.
Introduction of Task Grouping: Adding the ability to group tasks and include them within the runAfter functionality would be a significant improvement. This feature would enable better organization and sequencing of tasks, making the overall process more efficient and easier to manage. Grouping tasks can also facilitate more complex workflows and dependencies among tasks, which can be particularly beneficial in larger projects or more sophisticated parallel processing scenarios.

let's fork this shit


[patch] Let pint-hinted input values pass through type hinting

By checking the actual datatype associated with their "magnitude".
I'm also boldly going to try setting the lower bound to the last version and hope it works.
Motivated by https://github.com/pyiron/elaston/pull/45

Pull Request Test Coverage Report for Build<PHONE_NUMBER>5
Details

0 of 0   changed or added relevant lines in 0 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage increased (+0.005%) to 91.303%

Files with Coverage Reduction
New Missed Lines
%

type_hinting.py
1
97.73%

Totals

Change from base Build<PHONE_NUMBER>6:
0.005%

Covered Lines:
2950

Relevant Lines:
3231

💛  - Coveralls


Windows 10 support?

Hi,
Is this available on Windows 10 Pro (Consumer and or Business)?
m

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

ID: 06fc7832-1655-f552-de28-b99bf2ab092f
Version Independent ID: 2a99f3c3-3472-1a5a-0a22-b99615108980
Content: Deploy graphics devices using Discrete Device Assignment
Content Source: WindowsServerDocs/virtualization/hyper-v/deploy/Deploying-graphics-devices-using-dda.md
Service: na
Product: windows-server-threshold
GitHub Login: @chrishuybregts
Microsoft Alias: chrihu

Hello,  When I run these commands in Powershell, it says "Dismount-VMHostAssignableDevice" is not recognized,  Is this command supported by server 2012 r2?

@mcheve01
Please take a look at the "Applies To" section of the documentation -- this feature is available for Hyper-V on Windows Server 2016 and Windows Serve 2019.
@maximojoexpress
This feature is currently not available on client operating systems. Can you share more about the scenario that you're trying to light up?

@cwilhit might be interested in learning more about the applications that you're trying to isolate.

Hey @glowtape, can you point me to where you're drawing this info from? Thanks!
"...current transition tech listed is DDA, even for the client version of Windows"

Various places on the web. Since you're with Microsoft, I figure you'd be aware of that? The code is the same for common functionality between client and server. Anything that applies to the server version also applies to the client, say like RDSH.
The quickest direct reference I could dig up on the quick right now is this Twitter statement of a coworker of yours: https://twitter.com/rds4u/status/997418881034719232

Well, anyway, RemoteFX 3D is deprecated. The Hyper-V manager on client disallows adding the virtual hardware to VMs. PowerShell still does it, but that's beside the point, and a guess for how long it'll still do.
DDA also works in client. Theoretically, anyway, because NVidia does their usual and prevents their users from using the passed-through hardware, when their driver detects being run in a VM (same spiel as with Linux KVM, which at least allows hiding the hypervisor).

Thanks for that. DDA has never been a recommendation to our client users; it doesn't work, plain and simple.
Can you tell me more about these graphics intensive apps? Are these games, or other? I periodically see this request come by from users and it's good to get an idea for use cases.

Things like Solidworks or Photoshop. Both are the sort of apps I'd love to run isolated, due to how much things they tend to install that keep running in background.
Solidworks litters the system with old DLLs and Office 2003 components and some background services, Photoshop (or Adobe software in general) installs tons of background and antipiracy services that love to go haywire once a while. I'm sure there's more applications that'd qualify.
They both profit from a GPU. Solidworks for rendering the graphics and Photoshop for both rendering and using GPU compute. The scenario I'd like realized is to boot a VM with a graphics card on a HDMI dongle and then RDP into it. That already sorta worked on Linux with KVM and VFIO, but I'd rather not use Linux.
As far as DDA goes, seems like the relevant people claiming to have it running seemed to implicitly use the server version (without saying so, probably because it wasn't legit). A supposed co-worker of yours posted on Reddit (_hypervguy) that the one of main reasons DDA is disabled on client is that it'd break some consumer scenarios like hibernation and sleep. Considering how much of a niche use case it is, anyone using DDA would probably be fine with that.

If DDA gets popular with all Win10 versions(and also one click to switch gpu in gui), Linux OS gonna be not more than program for devs and Linux allows gpu passthrough for windows.
So Windows are you gonna be nested program in Linux or reverse in the future... =)


[BUG] Installing BD makes discord a browser version

Hello world.
I just today wanted to get back into using Better discord, I left it a while back when Rich presence did not work.
Now trying to install it on my clean installment ( I deleted all Discord registries in windows, I cleaned all appdata on it), I open up Discord, it shows me that I should try downloading Discord, it probably thinks I'm in the browser. I try talking but my mic and Camera are deactivated. What is the deal here?

Have you used the updated installers? https://github.com/BetterDiscord/Installer ?

Settings:

Have you used the updated installers? https://github.com/BetterDiscord/Installer ?

I tried the one provided here:
https://github.com/rauenzi/BetterDiscordApp/releases/tag/v1.0.0
and then:
https://github-releases.githubusercontent.com/155161153/c7d2fb00-d1b5-11ea-99e8-f5ca60f47502?X-Amz-Algorithm=AWS4-HMAC-SHA256&X-Amz-Credential=AKIAIWNJYAX4CSVEH53A%2F20210418%2Fus-east-1%2Fs3%2Faws4_request&X-Amz-Date=20210418T172612Z&X-Amz-Expires=300&X-Amz-Signature=3f683bc55a63a6b75ed3288c9ff0af355ea1efd32f921fc511ddef6e3177a5d2&X-Amz-SignedHeaders=host&actor_id=0&key_id=0&repo_id=155161153&response-content-disposition=attachment%3B filename%3DBandagedBD.exe&response-content-type=application%2Foctet-stream

New installer looks good.
btw Got the "What's new" twice.
cc


initial version of storybook+nuxt demo

Codecov Report

Merging #2773 into dev will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##              dev    #2773   +/-   ##
=======================================
  Coverage   99.91%   99.91%
=======================================
  Files          23       23
  Lines        1227     1227
=======================================
  Hits         1226     1226
  Misses          1        1

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 c415a67...3990498. Read the comment docs.


ARROW-17932: [C++] Implement streaming RecordBatchReader for JSON

See: ARROW-17932.
This adds a json::StreamingReader class modeled after csv::StreamingReader. Some parts of the existing json::TableReader implementation have been refactored to utilize the new facilities.

A couple notes on this:

I decided to hold off on documenting threading/reentrancy characteristics until I could get a second opinion, as this was my first exposure to the project's async facilities. As it's written, the stated limitations of the CSV equivalent should still apply (no async-reentrancy, no multi-threading). However, here, it seems possible to launch the parsing/decoding tasks in parallel once a chunked block has been serially generated. There may be complexities I'm not considering though.
Test coverage handles the common cases, but likely isn't comprehensive enough - although some of it should be handled by the existing TableReader tests since it uses common code.

Some answers to the points above:

It seems like this may be async-reentrant (and theoretically could be, as you point out)

Tests should indeed be complemented with: 1) tests for errors 2) tests with multiple input blocks 3) stress tests, especially async-reentrant calls to ReadNextAsync (MakeReadaheadGenerator may help with that)

By the way, can you also:

rebase/merge from master
add documentation for the new class in https://github.com/apache/arrow/blob/master/docs/source/cpp/json.rst and https://github.com/apache/arrow/blob/master/docs/source/cpp/api/formats.rst#line-separated-json
?

@pitrou  I still have to do the docs, but feel free to bring up any further points on the code in the meantime.

I just opened a small PR that addresses the chunker issue: https://github.com/apache/arrow/pull/14843. If that gets merged, I have a test I can add for the newlines_in_values = true case (which would crash otherwise). Then, this will be ready for re-review.

@pitrou Just pushed a fix.
Also, not sure if my rationale on the std::remove table validation stuff from yesterday is sound - but if not, I can change that too.


[Android] Godot is crashing while creating apk when i put a plugin.

3.2 master
Win10
Godot is crashing while creating apk, not during the build stage but the last stage when it starts show the merging of  the project into apk it crashes at the beginning of the last stage.
I am using admob from Shin-Nil - https://github.com/Shin-NiL/Godot-Android-Admob-Plugin
Although it gives an error for layout in the banner section but I null it down.
It might be occurring due to the merging of plugin into the project.

It seems this issue has been resolved on this github admob plugin:
https://github.com/kloder-games/godot-admob
The plugin that you are using was inspired by this one and an other one.
This github has a new commit 17 days ago:
fixes #72 which caused crash during app startup

Please report module-specific issues to the module's own repo. As seen here this might well be a bug in the module and not in the engine.
The developers of the module can then debug and estimate with you whether that's something they can fix, or if it's an engine bug which should be reported here.

Nevermind, I was misled by https://github.com/godotengine/godot/issues/32553#issuecomment-538648202, but if the crash happens in the editor while exporting, there's indeed something to fix in Godot itself.

Can you start Godot from a terminal and paste the output when it crashes?

Sorry miss understood the problem...

The editor crashes here

ERROR: _update_custom_build_project: Not adding application attributes as the expected tag was not found in '<application': android:icon="@drawable/icon"
   At: platform/android/export/export.cpp:1897
ERROR: get: FATAL: Index p_index=-1 out of size (size()=58)
   At: ./core/cowdata.h:151

You should update your custom build template (delete the "android" folder and re-install the template), this has been fixed already.

@akien-mga It still crashes at the same spot but now it's giving this error
ERROR: get: FATAL: Index p_index=-1 out of size (size()=59)
   At: ./core/cowdata.h:151

Can you share the project, or make a simple new one that triggers the issue?

Well I am using demo project of admob from Shin-Nil but here is the one with plugin( i have deleted build folder and only the plugin is in the Android folder) @akien-mga
demo.zip

Has the bug been confirmed ?

I can reproduce the crash on the master branch on Linux:
Add: src:/home/akien/tmp/godot/bug/demo/android/admob-plugin/src
Section: APPLICATION
Section: DEPENDENCIES
ERROR: get: FATAL: Index p_index=-1 out of size (size()=60)
   At: ./core/cowdata.h:152.
handle_crash: Program crashed with signal 4
Dumping the backtrace.  Maybe a character?
My AndroidManifest.conf has the following code:
This don't works:

This works:

Well I am using demo project of admob from Shin-Nil but here is the one with plugin( i have deleted build folder and only the plugin is in the Android folder) @akien-mga
demo.zip

I can confirm, this compiles for me, but the Godot 3.2 alpha 1 crashes while building the final APK.
Exactly the same for as in the GIFs-- running Windows 10 Pro, hard crash to desktop.

@m4gr3d I'm having a look at it now, just co-assigned you for feedback if you have any :)

@gianllucah That's because the crash happens here, so if you don't have meta-data, it doesn't go beyond the first condition (it crashes on the second, index out of size):
https://github.com/godotengine/godot/blob/781e4f8673857b2b318e919ecb780bfad972a5a6/platform/android/export/export.cpp#L838-L843

@m4gr3d So for some reason attr_value is 0xFFFFFFFF here and we thus crash trying to index the table. This works it around, and it seems to still allow inserting the right values when using Oculus VR xr_mode. That's a hack though :)
diff --git a/platform/android/export/export.cpp b/platform/android/export/export.cpp
index b61575e2aa..0e8e635b6d 100644
--- a/platform/android/export/export.cpp
+++ b/platform/android/export/export.cpp
@@ -828,14 +828,14 @@ class EditorExportPlatformAndroid : public EditorExportPlatform {
                                                        encode_uint32(min_gles3 ? 0x00030000 : 0x00020000, &p_manifest.write[iofs + 16]);
                                                }

-                                               if (tname == "meta-data" && attrname == "name" && string_table[attr_value] == "xr_mode_metadata_name") {
+                                               if (tname == "meta-data" && attrname == "name" && attr_value != 0xFFFFFFFF && string_table[attr_value] == "xr_mode_metadata_name") {
                                                        // Update the meta-data 'android:name' attribute based on the selected XR mode.
                                                        if (xr_mode_index == 1 /* XRMode.OVR */) {
                                                                string_table.write[attr_value] = "com.samsung.android.vr.application.mode";
                                                        }
                                                }

-                                               if (tname == "meta-data" && attrname == "value" && string_table[attr_value] == "xr_mode_metadata_value") {
+                                               if (tname == "meta-data" && attrname == "value" && attr_value != 0xFFFFFFFF && string_table[attr_value] == "xr_mode_metadata_value") {
                                                        // Update the meta-data 'android:value' attribute based on the selected XR mode.
                                                        if (xr_mode_index == 1 /* XRMode.OVR */) {
                                                                string_table.write[attr_value] = "vr_only";

Interestingly it fails only on the second one (value), so it seems like the parser handles the first android:name find but then treats android:value: as a resource (0xFFFFFFFF) instead of a string?
BTW there's related dead code here:
https://github.com/godotengine/godot/blob/781e4f8673857b2b318e919ecb780bfad972a5a6/platform/android/export/export.cpp#L769-L779
Not sure what a correct fix would be here as I'm not so familiar with this black magic @hpvb wrote :D Also CC @BastiaanOlij.

And to be explicit, here's the AndroidManifest.conf from the plugin that triggers this issue:
[application]
        <activity android:name="com.google.android.gms.ads.AdActivity"
                android:configChanges="keyboard|keyboardHidden|orientation|screenLayout|uiMode|screenSize|smallestScreenSize"/>

        <meta-data android:name="com.google.android.gms.version"
                android:value="@integer/google_play_services_version" />

        <meta-data
            android:name="com.google.android.gms.ads.AD_MANAGER_APP"
            android:value="true"/>

It doesn't seem malformed, though even if it were, we should not crash.

I'm pushing my temporary workaround as #32740 as I want to release 3.2 alpha 2 today with this fix (and others). Let's keep this issue open though as we should still figure out the real issue and fix it.

@akien-mga That's actually the correct solution.
The android:name attribute of the meta-data element only accept raw string values, that's why it's not affected by the issue.
The android:value attribute on the other hand can accept both a raw string value or a string resource value which underneath the hood is just an integer value used by the Android build system to point to the actual value.
The bug surfaced by having the string resource @string/facebook_app_id in the android:value attribute. If you were to replace that string resource by the actual string value being pointed at, the issue would vanish.
A slightly cleaner fix would be to remove the check on line 833 for the android:name attribute and use the value variable defined on line 769 since it would only pass our check for the xr_mode_metadata if attr_value != 0xFFFFFFFF.
Though what you have is semantically correct as well so we could keep it as is if desired.

Thanks for the explanations about the Android string resources, that makes sense.

A slightly cleaner fix would be to remove the check on line 833 for the android:name attribute and use the value variable defined on line 769 for the check on line 840 for the android:value attribute since it would only pass our check for the xr_mode_metadata if attr_value != 0xFFFFFFFF.

Sounds good, do you want to PR this as a cleanup of my previous PR, fully fixing this issue?

Sounds good. I'll create a follow-up cleanup PR.

Fixed by PR #32752


Convert GTAG Parameters into Server-Side Parameters too

Studying the PageView request made by GTAG in JavaScript it seems that parameter names are the same as V1 but with limited parameter scope.
https://developers.google.com/analytics/devguides/collection/protocol/v1/parameters
GET Parameters
v: 2
tid: G-XXXXX
gtm: 2oe7d0
_p:<PHONE_NUMBER>
_z: ccd.v9B
cid: 00000.000000
ul: en-us
sr: 1920x1080
sid:<PHONE_NUMBER>
sct: 3
seg: 1
dl: http%3A%2F%2F<IP_ADDRESS>%2F
dr: http%3A%2F%2F<IP_ADDRESS>%2F
dt: Chttr.link
_s: 1

Body Parameters
en=page_view&_ee=1
en=scroll&epn.percent_scrolled=90&_et=8

Found out from quick look-up between parameters and V1 measurement protocol
v:   protocol version : text : 2
tid: tracking id : text : G-xxxx
aip: anonymize ip : boolean
gtm:
_p:
_z:
cid: cliend it : text : cookie._ca cookie._gid random-session-hash
uid: user it : text : unique-user-id
ul:  user language : text : lang-country => en-us
sr:  screen resolution : text : withXheight => 1920x1080
sid:
sct:
seg:
dl:  document location url : text : http://foo.com/home?a=b => http%3A%2F%2Ffoo.com%2Fhome%3Fa%3Db
dr:  document referer : text : http://example.com => http%3A%2F%2Fexample.com
dt:  document title: text : <title>HERE</title>
_s:

Need to figure out what the last ones are somehow

I'm closing this as Won't Fix due to tags that are generated with GTM/GTAG that can not be properly replicated in PHP to my knowledge as is right now.
It might come back up at a later time.


Importing main Unity package and samples package doesn't yield any assets

Importing the 2 packages outlined here to work with the WebRTC tutorial doesn't yield any assets under the Packges dir in the project explorer.

Steps to reproduce:

Download com.microsoft.mixedreality.webrtc and com.microsoft.mixedreality.webrtc.samples from here as outlined in the instructions here.
Import into a new Unity project using UPM.
Navigate to the Package directory in the project explorer.
Observe the empty directory.

Expected outcome:
Packages directory should contains assets I can copy to my working project.

I could be fairly sure this is a Unity issue - have you tried re-opening the editor, or removing and re-adding the packages?
Not sure if this affects asset import too, but make sure you don't have any Script Errors - these commonly prevent new assets from importing properly.

So it seems to be a Unity 2019.4.x issue. 2019.4.x is LTS and I keep it up to date with the latest patches. Currently using 2019.4.21f1 where the packages dir was empty. Tried the import process again in Unity 2020.2.4f1 and everything seems to be here.


Add test sweepers for all terraform acceptance tests

Right now the test suite has partial support for test sweepers. They also are executed by the test suite internally. These sweepers should be exposed to terraform so resources can be cleaned up prior to a test run. This helps in the case that a code change causes the test suite to panic and leaves resources around.
In addition our TestMain function should be able to recover from test panics so that it can still run the teardown code.

This was completed in #187


Please update example on RESTRICTED_USER usage

Hello, in example RESTRICTED_USER/MULTI_USER should be USE [database_name]; instead of USE master;.
When running this query on master,
USE master;
GO
ALTER DATABASE [database_name]
SET RESTRICTED_USER;
GO

error is thrown:
Msg 42008, Level 16, State 3, Line 1
ODBC error: State: 28000: Error: 18456 Message:'[Microsoft][ODBC Driver 17 for SQL Server][SQL Server]Login failed for user '##MS_InstanceCertificate##'.'.
Msg 5069, Level 16, State 3, Line 1
ALTER DATABASE statement failed.

Please update example.
also, in case of running
USE [database_name];
GO
ALTER DATABASE [database_name]
SET RESTRICTED_USER;
GO

it works and DB is in RESTRICTED_USER mode, but when checking database mode with:
SELECT name, user_access_desc FROM sys.databases;
it shows user_access_desc  as MULTI_USER

@osapeliuk -- Oleksandr, thank you for your well-researched feedback.
@WilliamDAssafMSFT -- William, please look into this issue.

Thanks @osapeliuk for contacting Microsoft with feedback about our product documentation. Can you confirm that you are observing this behavior in Azure SQL Database? If so, I believe some doc updates are in order in the Azure SQL Database version, and a review of these examples are underway.


Can't get past step 2

I turned on GitHub pages according to the bot's directions, but the bot isn't recognizing this or the page isn't being activated.

The bot finally started working.


RFP - Doxie for Windows

Checklist

[x] The package I am requesting does not already exist on https://chocolatey.org/packages;
[x] There is no open issue for this package;
[x] The issue title starts 'RFP - ';
[x] The download URL is public and not locked behind a paywall / login;

Package Details
Software project URL : https://www.getdoxie.com/info/apps.html
Direct download URL for the software / installer : https://download.getdoxie.com/DoxieSetup-2.12.2.exe
Software summary / short description:
Companion software intended for use with most Doxie-branded portable scanners. Enables importing of scans via USB connection, SD cards, or Wi-Fi connection, edits for organization (multiple pages, interleaving, stitching, OCR), readability adjustments (contrast, rotations, crops), and exporting via multiple image file formats (JPEG, PNG, PDF).

Status: Review
https://chocolatey.org/packages/doxie

The package as linked above appears to have been published and approved 3 days ago. I think we're clear to close this one out.
Thanks @chtof for picking this up!


Blackboxing?

Hi,
I didn't find info about blackboxing feature. Probably it has been updated in the chrome dev tools.

:+1:

Great, thanks @blittle


Unit for update interval / numeric input field

Because CMI is a bit "sensitive" in regards of the update interval, the new configuration parameter is very useful. Is it possible to add a unit (minutes / seconds / ...) to the UI? I have no idea what value is actually used (10?) - seem to be longer than a minute, but less than 10 minutes - but only guessing.
thanks for your great work

Thanks for the hint, I have overlooked the unit.
The standard query interval is 10 minutes.


Directory API - Users - Delete:  Fails when using a valid user id.

I've tried each of the valid userKey options.  I noticed that on line 3 of my code example, the endpoint.uri has a ? at the end of it...regardless of what kind of userKey is used.  Manually resetting that uri value doesn't help.
I'm using this API https://developers.google.com/admin-sdk/directory/reference/rest/v1/users/delete
Environment details

OS type and version:
Linux
Python version: python --version
3.7.4
pip version: pip --version
21.1
google-api-python-client version: pip show google-api-python-client
Name: google-api-python-client
Version: 2.3.0
Summary: Google API Client Library for Python
Home-page: https://github.com/googleapis/google-api-python-client/
Author: Google LLC
Author-email<EMAIL_ADDRESS>License: Apache 2.0
Location: /home/milesm/woolpert/thryv/thrv-console/.venv/lib/python3.7/site-packages
Requires: google-auth, uritemplate, httplib2, google-auth-httplib2, google-api-core, six
Required-by:

Steps to reproduce
Use the directory_v1 build.
Code example
aValidUserId = 'notarealid'
service = build('admin', 'directory_v1', credentials=validCredentials)
endpoint = service.users().delete(userKey=aValidUserId)
endpoint.execute()

API Error Response
'code':400
'message':'Invalid Input: userKey'
'errors':[{'message': 'Invalid Input: userKey', 'domain': 'global', 'reason': 'invalid'}]

Stack trace
Traceback (most recent call last):
  File "/home/milesm/testapp/.venv/lib/python3.7/site-packages/uvicorn/protocols/http/httptools_impl.py", line 398, in run_asgi
    result = await app(self.scope, self.receive, self.send)
  File "/home/milesm/testapp/.venv/lib/python3.7/site-packages/uvicorn/middleware/proxy_headers.py", line 45, in __call__
    return await self.app(scope, receive, send)
  File "/home/milesm/testapp/.venv/lib/python3.7/site-packages/fastapi/applications.py", line 199, in __call__
    await super().__call__(scope, receive, send)
  File "/home/milesm/testapp/.venv/lib/python3.7/site-packages/starlette/applications.py", line 111, in __call__
    await self.middleware_stack(scope, receive, send)
  File "/home/milesm/testapp/.venv/lib/python3.7/site-packages/starlette/middleware/errors.py", line 181, in __call__
    raise exc from None
  File "/home/milesm/testapp/.venv/lib/python3.7/site-packages/starlette/middleware/errors.py", line 159, in __call__
    await self.app(scope, receive, _send)
  File "/home/milesm/testapp/.venv/lib/python3.7/site-packages/starlette/middleware/cors.py", line 86, in __call__
    await self.simple_response(scope, receive, send, request_headers=headers)
  File "/home/milesm/testapp/.venv/lib/python3.7/site-packages/starlette/middleware/cors.py", line 142, in simple_response
    await self.app(scope, receive, send)
  File "/home/milesm/testapp/.venv/lib/python3.7/site-packages/starlette/exceptions.py", line 82, in __call__
    raise exc from None
  File "/home/milesm/testapp/.venv/lib/python3.7/site-packages/starlette/exceptions.py", line 71, in __call__
    await self.app(scope, receive, sender)
  File "/home/milesm/testapp/.venv/lib/python3.7/site-packages/starlette/routing.py", line 566, in __call__
    await route.handle(scope, receive, send)
  File "/home/milesm/testapp/.venv/lib/python3.7/site-packages/starlette/routing.py", line 227, in handle
    await self.app(scope, receive, send)
  File "/home/milesm/testapp/.venv/lib/python3.7/site-packages/starlette/routing.py", line 41, in app
    response = await func(request)
  File "/home/milesm/testapp/.venv/lib/python3.7/site-packages/fastapi/routing.py", line 202, in app
    dependant=dependant, values=values, is_coroutine=is_coroutine
  File "/home/milesm/testapp/.venv/lib/python3.7/site-packages/fastapi/routing.py", line 148, in run_endpoint_function
    return await dependant.call(**values)
  File "./app/routers/users.py", line 199, in delete
    service.users().delete(userKey=currentUser['id']).execute()
  File "/home/milesm/testapp/.venv/lib/python3.7/site-packages/googleapiclient/_helpers.py", line 134, in positional_wrapper
    return wrapped(*args, **kwargs)
  File "/home/milesm/testapp/.venv/lib/python3.7/site-packages/googleapiclient/http.py", line 935, in execute
    raise HttpError(resp, content, uri=self.uri)
googleapiclient.errors.HttpError: <HttpError 400 when requesting https://admin.googleapis.com/admin/directory/v1/users/103777420240693602120? returned "Invalid Input: userKey". Details: "[{'message': 'Invalid Input: userKey', 'domain': 'global', 'reason': 'invalid'}]">

Thanks, I should of tried that first.  I get the same error there and opened a bug with them.
https://issuetracker.google.com/issues/188695462


Image preview in Terminator

What system are you running Yazi on?
Linux X11
What terminal are you running Yazi in?
Terminator 2.1.2
Did you try the latest main branch to see if the problem got fixed?
Tried, but the problem still
yazi --debug output

Yazi
    yazi 0.2.4 (b10f2de 2024-03-09)

Environment
    OS: linux-x86_64 (unix)
    Debug: false

Emulator
    Emulator.via_env: ("xterm-256color", "")
    Emulator.via_csi: Ok(Unknown([]))
    Emulator.detect: Unknown([])

Adaptor
    Adaptor.matches: X11

tmux
    TMUX: false

Zellij
    ZELLIJ_SESSION_NAME: None

Desktop
    XDG_SESSION_TYPE: Some("x11")
    WAYLAND_DISPLAY: None
    DISPLAY: Some(":0.0")

Ueberzug
    Version: Ok(Output { status: ExitStatus(unix_wait_status(256)), stdout: "", stderr: "Usage:\n    ueberzug layer [options]\n    ueberzug library\n    ueberzug version\n    ueberzug query_windows PIDS ...\n" })

WSL
    /proc/sys/fs/binfmt_misc/WSLInterop: false

Describe the bug
Running ueberzug layer independently shows an image preview, but in yazi it doesn't work whether using tmux or not. I even updated my tmux config, restarted tmux, restarted my laptop. I'm running Debian 12 and using one of the compiled versions of yazi from release
Expected Behavior
Show image preview inside yazi
To Reproduce
Start terminator, open yazi, hover over an image (png, jpg)
Configuration
All default from https://github.com/sxyazi/yazi/releases/tag/v0.2.4
Anything else?

Hi, are you using https://github.com/jstkdng/ueberzugpp?

yeah here's a test with ueberzug layer + {"action":"add","identifier":"preview","max_height":0,"max_width":0,"path":"/home/jin/Pictures/memes/cat-look.png","x":0,"y":0}

What is your ueberzugpp version? I noticed that the yazi --debug you provided above does not include a proper version for ueberzugpp.
yazi --debug is essentially doing ueberzug --version in the background. What is the output when you manually run it?

18.1.9, seems old lol
I'ma grab a newer one and test

It's working now! Just not with optimal placement within the right pane :sweat_smile:

Nice to hear that it works now. You can fine-tune the position of Ueberzug using ueberzug_scale and ueberzug_offset, see https://yazi-rs.github.io/docs/configuration/yazi#ueberzug-specific. I think we can close this issue now.
By the way, you might also want to install nerd-fonts and configure it for your terminal to allow Yazi to display icon fonts properly, or simply disable it ;)