Preston writes cycles in PHP

If anyone has anything adequate about php, then leave it here.
Especially interesting is the solution that makes tdjson.so work under apache.
Later here will be PHP
Probably many people want to know who is Preston?
Please connect tdjson to apache and I'll tell you.

There is already some kind of meat https://github.com/tdlib/td/issues/23

Hey. You already wrote about #23. And there I already wrote about the library. For the year we have grown a bit. Together with @maxvgi we corrected a lot of things, added a stubs for IDE, and there are also some things for a Symfony. Can this be considered adequate? =) I know still a lot of work. So if somobody is interesting - PR is welcome! =)

Nice work, @yaroslavche! I'll expand section about phptdlib in example/README.md.
It seems that there is plenty of room for improvement still. For example, it's odd, that you need to manually call json_encode for each query, or example asks to comment and uncomment code back and forth.

@levlam thanks for those kind words =)
Yeah, I agree. Good points.

json_encode because BaseJsonClient::query (and other similiar) expects the first argument to be a string with valid json. But nothing prevents from overriding the method query that will accept the array, or @type, and the rest of the parameters and translate it into a valid json (as I did in bundle for symfony). Honestly, I think this is not task for extension, but maybe I'm wrong and use array is better idea =)
Well, about why an example asks to play with the comments)) I agree, this is not very clear and informative. But the fact is that during the setAuthenticationPhoneNumber request, we still do not have the code and we cannot call checkAuthenticationCode. Exactly as we don't need to set the number again when check received code.

But thanks for the comments, I will definitely consider this moments =)

In fact, to log in you need to handle updateAuthorizationState as described in Getting Started. By handling this update application always knows, whether it needs to call setAuthenticationPhoneNumber  or checkAuthenticationCode. There are a lot of tricky cases, so you can never be sure that after restart you don't need to enter phone number again or that you need to enter a code, and not a password.

Yes, I know about this. You can check TDLibService::checkReceivedResponses. There in all received responses i check updateAuthorizationState and set authorizationState property. After this, I just check state in controller and show forms if it needed:
    public function getAuthorizationState(): ?Response
    {
        $authorizationState = $this->tdlibService->getAuthorizationState();
        if ($authorizationState === TDLibService::AUTHORIZATION_STATE_READY) {
            // ok
        }
        switch ($authorizationState) {
            case TDLibService::AUTHORIZATION_STATE_WAIT_PHONE_NUMBER:
                return $this->showPhoneNumberForm();
            case TDLibService::AUTHORIZATION_STATE_WAIT_CODE:
                return $this->showAuthenticationCodeForm();
            default:
                return null;
        }
    }

This is not good at all. Just for an example =)

Good example. It remains to implement interface. (The Interface Segregation Principle)

Hi. I had some free time and an idea how to work with tdlib json client via WebSocket in PHP. Sounds weird? xD I know, but.. Maybe someone will find this useful. The idea is: you can run ws server with json client inside as symfony console application and just interact with it. And yes, as always, there is still a lot of work =)
You can see example in readme here. And there is a screenshot:

But it is not necessary to add this package, as it is just experimental right now. It will be enough to require symfony/console and add classes for the server and the command (see command in package).

You can also interact directly from Vue without escaping to "server proxies".


Add offset counter and resetter

Add a destination task context
Some connectors (notably snowflake) expects offsets to be serial (as they should be?), use an atomic counter
In some cases, offset consumption may be reset to the same offset.. add this to the context

close in favour of #349


Revert metadata.json license back to MIT

Relicensing can only be done with the permission of all the copyright
holders.

Instead of https://github.com/voxpupuli/puppet-iis/pull/132


Module _multiarray_umath

Hi,
I follow the tuto and I have  :

I use Windows
Python 3.9.10 (I tried with version above also)
NumPy 1.25.1 (I wrote import numpy ; print(numpy.__version__)
I installed with pip, it was not working and conda, it was not working also
I tried de  pip uninstall numpy; pip install numpy

And when I try to click on go-web.bat I have this error. Could you help me ?
F:\Download\RVC-beta\RVC-beta0717>runtime\python.exe infer-web.py --pycmd runtime\python.exe --port 7897
Traceback (most recent call last):
  File "F:\Download\RVC-beta\RVC-beta0717\runtime\lib\site-packages\numpy\core\__init__.py", line 23, in <module>
    from . import multiarray
  File "F:\Download\RVC-beta\RVC-beta0717\runtime\lib\site-packages\numpy\core\multiarray.py", line 10, in <module>
    from . import overrides
  File "F:\Download\RVC-beta\RVC-beta0717\runtime\lib\site-packages\numpy\core\overrides.py", line 6, in <module>
    from numpy.core._multiarray_umath import (
ImportError: DLL load failed while importing _multiarray_umath: Le module spécifié est introuvable.

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "F:\Download\RVC-beta\RVC-beta0717\infer-web.py", line 10, in <module>
    import numpy as np
  File "F:\Download\RVC-beta\RVC-beta0717\runtime\lib\site-packages\numpy\__init__.py", line 140, in <module>
    from . import core
  File "F:\Download\RVC-beta\RVC-beta0717\runtime\lib\site-packages\numpy\core\__init__.py", line 49, in <module>
    raise ImportError(msg)
ImportError:
IMPORTANT: PLEASE READ THIS FOR ADVICE ON HOW TO SOLVE THIS ISSUE!

Importing the numpy C-extensions failed. This error can happen for
many reasons, often due to issues with your setup or how NumPy was
installed.

You should uninstall your python environment in your system.


[docs] Add example layouts

Closes #10986

Great work! Finally I can copy & paste nice working design examples! :)
I found one minor issue within the Dashboard example: https://material-ui.com/page-layout-examples/dashboard/
When you make your browser windows small, the Toolbar and Drawer use the dense appearance. But the padding of the Menu Button within the Toolbar won't adapt.
I'm no CSS expert, but I guess that it should become margin-left: 4px when dense layout is active.

Another nice feature request would be to exchange the Drawer variant on smaller screens. Something like:
<Hidden mdUp={true}>
    <Drawer variant="temporary">
        ...
    </Drawer>
</Hidden>
<Hidden smDown={true}>
    <Drawer variant="permanent">
        ...
    </Drawer>
</Hidden>


Where did you get the icons?

Hello! I see you have the GitHub, YouTube, and Twitter icons. I really want a Discord icon in the same style but I can't find any that work online. Where did you guys get the YouTube, GitHub, and Twitter icons from?

I got the icons from https://react-icons.github.io/react-icons


Added HSB color adjustment to the atmosphere and a sandcastle demo

Resolves #3439.
I added hue/saturation/brightness color adjustment to the atmosphere and a sandcastle demo for this.
Sean and I thought color post processing would be the best approach for adjusting the atmosphere color, and it seems like a lot of photo manipulation tools already have HSB color adjustment.

Also updated CHANGES.md.

I made a small change to prevent weird things from happening when you have additional brightness but also have sun position based atmosphere color and are looking at the night sky.

That's all my comments right now. The Sandcastle demo is a really nice touch, it's looking good.

Ah one more thing, make sure to add a test for this. Something simple like rendering the atmosphere normally, then rendering after changing the shifts, and checking that the resulting colors are different.

@likangning93 can you make a nice animated gif of this so we can tweet it?  Use http://www.cockos.com/licecap/ and just paste it into a comment in this PR.

I'm having some trouble writing the test case that @lilleyse suggested. Here's what I have right now:
it('draws sky with color correction active', function() {
    var s = new SkyAtmosphere();
    var previousSkyAtmosphere = scene.skyAtmosphere;
    scene.skyAtmosphere = s;
    scene.isReadyForAtmosphere = true;

    scene.camera.setView({
        destination : Cartesian3.fromDegrees(-75.5847, 40.0397, 1000.0),
        orientation: {
            heading : -Math.PI_OVER_TWO,
            pitch : 0.2,
            roll : 0.0
        }
    });

    var command = s.update(scene.frameState);
    expect(command).toBeDefined();

    var noCorrectedPixels = scene.renderForSpecs();
    expect(noCorrectedPixels).toEqual([0, 0, 0, 255]);

    s.colorCorrect = true;
    expect(scene.renderForSpecs()).toEqual(noCorrectedPixels);
    s.hueShift = 1.0;
    expect(scene.renderForSpecs()).toEqual(noCorrectedPixels);
    s.hueShift = 0.5;
    expect(scene.renderForSpecs()).not.toEqual(noCorrectedPixels);

    scene.skyAtmosphere = previousSkyAtmosphere;
    scene.isReadyForAtmosphere = false;
});

I'm pretty sure the command for the skyAtmosphere is getting to the scene (when I console.log it from scene it's defined) but the last expect here keeps failing. Can I have renderForSpecs() render larger than just one pixel?

Is this ok? Licecap's output seems to produce a lot of color banding.

@likangning93 perhaps use a lower resolution but more bits or fps?  I think it needs to be under 4MB for twitter.
If it turns out to be too much work, just create a 2-4 screenshots and we'll attach those to a tweet.

How about this one? Giphy has a video to gif convertor that seems to do a better job at preserving color. It's about 1.7 MB.

I think I streamlined the test some, but I'm still not sure if the SkyAtmosphere is drawing, or if that drawing would be visible in any configuration with how I'm currently testing. A lot of the other tests for SkyAtmosphere note that there's problems testing the pixel values from drawing with the SkyAtmosphere command.
    var s = new SkyAtmosphere(Ellipsoid.WGS84);
    var context = scene.context;
    var command = s.update(scene.frameState);
    expect(command).toBeDefined();
    command.execute(context);

    var noCorrectedPixels = context.readPixels();

    s.colorCorrect = true;
    command = s.update(scene.frameState);
    command.execute(context);
    expect(context.readPixels()).toEqual(noCorrectedPixels);
    s.hueShift = 1.0;
    command = s.update(scene.frameState);
    command.execute(context);
    expect(context.readPixels()).toEqual(noCorrectedPixels);
    s.brightnessShift = 1.0;
    command = s.update(scene.frameState);
    command.execute(context);
    expect(context.readPixels()).not.toEqual(noCorrectedPixels);

I'm basing this comment off of your previous code snippet:
So I think the main problem with the previous code was that you were setting scene.isReadyForAtmosphere = true; instead of scene._environmentState.isReadyForAtmosphere = true;. Also while it didn't affect the test, you use Math.PI_OVER_TWO which will be undefined because it's using JavaScript's Math library. When including Cesium/Core/Math, the "name" of the module is typically CesiumMath to get around this.
This code snippet worked for me:
scene.skyAtmosphere = new SkyAtmosphere();
scene._environmentState.isReadyForAtmosphere = true;

scene.camera.setView({
    destination : Cartesian3.fromDegrees(-75.5847, 40.0397, 1000.0),
    orientation: {
        heading : -CesiumMath.PI_OVER_TWO,
        pitch : 0.2,
        roll : 0.0
    }
});

var color = scene.renderForSpecs();
expect(color).not.toEqual([0, 0, 0, 255]);

scene.skyAtmosphere.hueShift = 0.5;
var hueColor = scene.renderForSpecs();
expect(hueColor).not.toEqual([0, 0, 0, 255]);
expect(hueColor).not.toEqual(color);

Actually I'm wrong that Math.PI_OVER_TWO will be undefined, though it is still good practice to use CesiumMath in case you also want to use the JS math library.

Awesome! Thanks Sean! Updated.

@likangning93 that animated gif is perfect, thanks!  I scheduled a tweet for next week when folks are back from vacation!

Can we aim to merge this today so it can make the 1.23 release tomorrow?

Updated, fixed a merge conflict in CHANGES.md.

Updated

Looking good!  Just those comments from me.

Updated

Looks good, thanks!
@lilleyse can do the honor of merging when ready!

Looks good to me too.


fix(docs): fix command to retrieve Falco pid for hot reloading

What type of PR is this?

Uncomment one (or more) /kind <> lines:

/kind bug

/kind cleanup

/kind design

/kind user-interface

/kind content

/kind event

Any specific area of the project related to this PR?

Uncomment one (or more) /area <> lines:

/area blog

/area documentation

/area community

What this PR does / why we need it:
This PR fixes the command used in the documentation to retrieve the Falco pid for hot reloading.
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: ekoops
Once this PR has been reviewed and has the lgtm label, please assign vjjmiras for approval. For more information see the Kubernetes Code Review Process.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

content/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

LGTM label has been added.  Git tree hash: b669e9a75f4454ecafbd44fa90df9a3f73d86d39

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: ekoops, Issif
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

content/OWNERS [Issif]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


How to get IP Address of the request?

Hi, Thank you for your awesome project.
Is there any way we to access the ip address of the client?

See https://github.com/oven-sh/bun/issues/3540


fix: Location component was showing wrong column with inlay hints.

The location component had been previously changed to use
vim.api.virtcol (the screen position of the cursor) instead of
vim.api.col (the byte position of the cursor), in order to prevent
showing the wrong column when multibyte characters are present.
Unfortunately, the new inlay hints in neovim 0.10 make heavy use of
virtual text, and therefore the column shown in the location component
is often incorrect.
This change fixes it to use vim.api.charcol, which correctly handles
variable-width characters without including virtual text.

Thanks for the fix


Example to run a case study of generative model training with Docking Score in Reward function.

Good afternoon! we were able to experiment with training the model for different targets from our dataset.
In addition, we are interested in training a generative model with Docking Score. As we understand, this can be done by specifying a function that takes into account Reward with Docking score, calculated using Auto Dock Vina. However, we could not get the LLL model to generate a function for calculating the reward using Auto Dock Vina.
We decided that it was necessary to pass an example of such a function into the system prompt. And then we added to the prompt the sample code for calculating the Docking Score from your code located at the link. We also specified the path to the receptor file and added it to the container at the corresponding path. However, unfortunately, we were not able to achieve success.
Then we tried to add a new configuration file from your repository to the ChatChemts-chatbot container. And also to specify in the system propmpt the sample code for the Vina_reward class. I have created a pdbqt file and specified the centers and boxing needed for Auto Dock Vina. And still we have no success, LLM responds that the error is in the configuration file, but since we don't have an instruction, we can't figure out exactly where we are wrong.
Could you please suggest a working example of a case study for training a generative model considering Docking Score as a reward function target?

In the Developer Document, I have added a section on debugging Docker Containers when encountering bugs or unexpected behaviors. You can use docker logs command to inspect the raw logs from a Docker Container.
Based on your work, it seems necessary to install meeko and vina via the pip command in the api_chemtsv2/Dockerfile. As mentioned in the reward/README of ChemTSv2, these two packages are required when using Vina_reward.py. Please note that ChemTSv2 follows a policy of not including reward-specific dependencies as part of its core requirements. Please note that If you are using AutoDock Vina, we recommend using Vina_binary_reward.py, as it provides a more stable performance.
As described in the preprint, ChatChemTS currently does not support docking simulation software. We are actively working on implementing this functionality, so please wait a little longer for the official release.


webpage title needs correction

The title for the webpage should be "Connect to Synapse SQL with Power BI Professional". Currently it is "Connect to Power BI ...", which is incorrect.
Thanks.

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

ID: b8260fd6-20a4-70cc-e0fe-19492a2a2b44
Version Independent ID: da27cb9e-5cd3-8e74-a80e-4cfb2ddac648
Content: Connect to Power BI Professional - Azure Synapse Analytics
Content Source: articles/synapse-analytics/sql/get-started-power-bi-professional.md
Service: synapse-analytics
Sub-service: sql
GitHub Login: @azaricstefan
Microsoft Alias: v-stazar

Hi @abdale thanks for noticing that.
Change will be visible in couple of hours.


PoW 7 update does not work with pool

No matter what configuration I try, I get no hashes forwarded to pool.
Counter is counting on Xcode debug area but none of the hashes appear on pool's site.
Neither directly using pool's IP or indirectly using xmr-node-proxy on my server.
Config I've tested (with my own username/password):
{
"pools": [
{
"hostname": "pool.supportxmr.com",
"port": 7777,
"ssl": false,
"allowSelfSignedSSL": false,
"share": 80,
"username": "46XWBqE1iwsVxSDP1qDrxhE1XvsZV6eALG5LwnoMdjbT4GPdy2bZTb99kagzxp2MMjUamTYZ4WgvZdFadvMimTjvR6Gv8hL",
"password"<EMAIL_ADDRESS>"keepAlive": true,
"coin": "xmr",
"default": false
}
],
"listeningPorts": [
{
"port": 8080,
"ssl": false,
"diff": 5000,
"coin": "xmr"
},
{
"port": 8443,
"ssl": true,
"diff": 5000,
"coin": "xmr"
},
{
"port": 3333,
"ssl": false,
"diff": 10000,
"coin": "xmr"
}
],
"bindAddress": "<IP_ADDRESS>",
"developerShare": 1,
"daemonAddress": "<IP_ADDRESS>:18081",
"coinSettings": {
"xmr":{
"minDiff": 100,
"maxDiff": 300000,
"shareTargetTime": 15
},
"aeon":{
"minDiff": 100,
"maxDiff": 300000,
"shareTargetTime": 15
}
}
}

I have updated an code with some fixes recently, have you updated to use latest version?

Yes

You should change the deployment target to 11.3 in the project settings. NOT other versions to get hash rate. Just try.


Duplicate reactions and reaction redactions

When more than one url is  posted in the same message, the plugin tries to process them both. This is intended, but it causes an exception because the reaction is applied more than once. Also, reactions need to be removed when the video is done being processed.

Perhaps an instance specific database could store filenames or URIs and check if they still exist during startup.


Add an inverse to object_set_instance

Godot version
4.0.dev (7f8ecff) (Beta 6)
System information
Windows 11
Issue description
Own C# Bindings for GDExtension
If I construct a C#-Object <<ClassName>> which extends a Godot-Object I use object_set_instance to combine both. I return the Godot-Object back to the Engine so everything works.
static unsafe IntPtr CreateObject(IntPtr userdata) {
	var instance = new <<ClassName>>();
	return instance._internal_pointer; //pointer to a godot-object
}

If I get a <<ClassName>> from the Engine (assign it in the Editor, ...), I get the pointer to the Godot-Object, but I need the C#-Object.
I see no obvious way to do that. Currently I use pointer arithmetic, which is dependent on the internal implementation of Object
public static <<ClassName>> Construct(IntPtr ptr) {
	ptr = *(IntPtr*)(void*)(ptr + 16); //change from godot-object-pointer to c#-object-pointer
	return (<<ClassName>>)ptr;
}

Did I miss something or would it be possibly to expose a object_get_instance to the GDExtension interface?
Steps to reproduce
N/A
Minimal reproduction project
N/A

I think you want to use object_set_instance_binding and object_get_instance_binding for this (rather than object_set_instance).
All extension objects that are bound to Godot objects should use object_set_instance_binding (even ones that aren't custom classes from your extension), and object_set_instance should additionally only be used on custom classes from your extension.
Given that this issue is rather old, I'm going to close it, but if there are any additional questions we can re-open it.


Check consumed parser tokens for validity

Closes https://github.com/eclipse-langium/langium/issues/1262
This is effectively a second (but better) attempt for https://github.com/eclipse-langium/langium/pull/1135. This change validates any parser token before putting it in the CST and AST. Invalid tokens are simply ignored after consuming them from the token stack.
This prevents the issue of invalid CST ranges (and therefore issues in LSP and value assignments) completely.

cc @Yokozuna59 This should fix the EOF issue you were experiencing.


Support JSON that isn't a dictionary

First, thanks for forking j2cli.  I filed an issue with the upstream repo but that is dead now so I thought I'd file it here :)
A lot of the times that I could use j2cli the input JSON isn't a dictionary but is a plain array, for example the output from GitLab's pipeline API is an array of dictionaries.
Currently, this causes j2cli to error out.  Maybe j2cli could detect that the top-level object is a list not a dict and pass it as a named variable (data?) in the environment?

Hi Ross. This sounds like a case I hadn't accounted for. Could you provide some extra input so I can have this fixed?
One of the features I have added is dictionary squashing. I.e. you can provide multiple json (dict-like) files, which will be squashed into a single dictionary to be used during rendering. E.g. if you supply two files containing {"a": 1, "b": 2} and {"a": 10, "c": 3}, the template will be rendered with {"a": 10, "b": 2, "c": 3}.
This works fine as long as all json input is dict-like. But what would be the expected behaviour if json input was:

only lists (this should be close to your case)
mixed lists and dicts (this may as well result in an error, but there may be use cases where this could make sense)

Of course, a PRs would also be welcome if you have already worked something out.

So the GitLab JSON I'm using looks like this:
[
   {
      "allow_failure" : false,
      "artifacts" : [
         {
            "file_format" : null,
            "file_type" : "trace",
            "filename" : "job.log",
            "size" : 591952
         }
      ],
    ...

My straw-man proposal would be to probe the parsed JSON and if the top-level object is a list then transform that to a dict with a data key pointing at the list.

I believe I have already added support for this case, but I haven't updated the docs:
% j2 --help
usage: j2 [-h] [-V] [-v] [-f {ini,json,yaml,env}] [--filters python-file [python-file ...]]
          [--tests python-file [python-file ...]] [--customize python-file] [--no-compact]
          [-U {strict,normal,debug}] [-I] [-o outfile]
          template data [data ...]

Command-line interface to Jinja2 for templating in shell scripts.

positional arguments:
  template              Template file to process.
  data                  Input data specification. Multiple sources in different formats can be
                        specified. The different sources will be squashed into a singled dict.
                        The format is <source>:<context_dest>:<format>. Parts of the
                        specification that are not needed can be ommitted. See examples at the
                        end of the help.

I.e.:
% j2 foo.tpl list.json:data

Would this work for you?

Oh, that looks promising.  I'll try it shortly.

So my own jinja CLI does this:
args.output.write(template.render({"data": json.load(args.json)}))

But using :data doesn't work with j2:
$ j2 gitlab-pipeline-timeline-template.html  pipeline-4.json:data
[ ... ]
  File "/home/ross/Mess/gitlab-pipeline-timeline-template.html", line 22, in <module>
    {% for job in data|rejectattr("started_at", "none") %}
jinja2.exceptions.UndefinedError: 'str object' has no attribute 'stage'

I'll be able to dig further later.


Add play_pause action support

Add play_pause action support in order to support mini media player default action for media control (play_pause)

This PR does contain a lot of changes that shouldn't be part of this change and should be reverted

I did a fork for myself. I have no interest in modifying anything for a pull. My coding abilities are far from good.


Problems encountered when running the sample project（运行示例项目碰到的问题）

Q1：../lib/mp/chart/chart.dart:51:12: Error: The method 'then' isn't defined for the class 'FutureOr<dynamic>'. Try correcting the name to the name of an existing method, or defining a method named 'then'.           .then((value) {
A1:change ‘../lib/mp/chart/chart.dart:51’
before：
ImageGallerySaver.saveImage(Uint8List.fromList(imgFile.readAsBytesSync()))           .then((value) {         imgFile.delete();       });
after：
var result = ImageGallerySaver.saveImage(Uint8List.fromList(imgFile.readAsBytesSync()));       if(result is Future<dynamic>){         result.then((value) => imgFile.delete());       }
Q2:`../lib/mp/core/utils/painter_utils.dart:15:34: Error: The getter 'text' isn't defined for the class 'InlineSpan'.

'InlineSpan' is from 'package:flutter/src/painting/inline_span.dart' ('/E:/flutter/packages/flutter/lib/src/painting/inline_span.dart').
Try correcting the name to the name of an existing getter, or defining a getter or field named 'text'.
var preText = painter.text.text;A2:change '../lib/mp/core/utils/painter_utils.dart:15' before:var preText = painter.text.text;after:var preText = (painter.text as TextSpan).text;`

If you have other questions, you can try to leave me a message

Q3:If you are running the flutter app, the [mark] of [chart] can be displayed normally, but the [mark] cannot be displayed normally after inheriting to Android, you may need to make the following changes:
Source file: line 197 of core/utils/screen_utils.dart, the variable [_screenDensity] is changed to [_screenWidth]
(如果你在运行flutter app时，[chart]的[mark]能正常显示，但是继承到Android后却无法正常显示[mark],可能要做如下修改：
源文件：core/utils/screen_utils.dart的第197行，的变量[_screenDensity]改为[_screenWidth])


Bump psr/log dependency

I've got a similar problem to #59.
Composer fails to install a big project due to some dependency conflict in this package.
Composer only shows a cryptic message exactly like in the issue instead of providing precise information, however, experimentally I was able to trace it to psr/log version. php-fig/log v1.1.0 has been released recently and this breaks all of our builds.
There are no BCs between 1.0 and 1.1 so this should be totally safe to bump.

@free6k thank you

Hello there, I'm the guy who tagged psr/log 1.1.0. Since that package is following Semantic Versioning, for you it's safe to require it as ^1.0, and allow both 1.0.x and 1.1.x. See also Composer docs.

Well the strangest thing is that it somehow breaks dependency resolution:

I can't imagine any other package in the project explicitly requires psr/log 1.1+ :)
There's no explicit error that would indicate problem with this package
Why composer won't just install psr/log 1.0?

Maybe I will investigate this sometime, this is not the first time I see strange composer failures like this.

https://github.com/psliwa/image-optimizer/pull/65 merged.


Usage with async/await

hi
I'm trying to use this plugin for eslint my test files but getting error:
Parsing error: Unexpected token t
My test is using async:
test('resolve issues', async t => {
 ...
}

My eslint config is:
{
  "extends": "plugin:ava/recommended",
  "plugins": [
    "ava"
  ]
}

I suppose it will overwirte parser options so that it will use babel-eslint-parser for tests.
Could you suggest what should I do to run eslint over my tests?
thx!

The default ESLint parser unfortunately doesn't support async functions, so you'll have to use babel-eslint. We should document that in the readme.

thanks for explanation @sindresorhus !
btw, could you point how to extract babel rules used by ava to use the same for parsing for eslint?

@vitalets this is what I use in my eslint

thank you @novemberborn , it works!

@vitalets cool!

Having just come across this issue, ESLint 3.6 now supports async functions
Add "ecmaVersion": 8 to your ESLint config parserOptions:
"parserOptions": {
    "ecmaVersion": 8 // or 2017
},


fix getBoundingClientRect error in ssr applications

Added condition to skip the dom api call getBoundingClientRect() on server. This change fixes #63

There's some conflict between this pr and a recently merged one. If you update it I'd be happy to test and merge this one too.

Closing this due to lack of progress. Feel free to open a new one if interest resurfaces.

You have not been able to fix a conflict in 8 lines of code?
Please go a head merging this fix or provide a solid SSR fix solution.

I am not following the repo currently. that's why i couldn't reply to the comments. @DDurakISH thanks for bringing it to my attention by tagging me i will look into the conflicts and create a new pr.

@DDurakISH  @wittlock no new pr is required as the current code in master branch will not result in this issue.

@urohith17 thank you very much for your investigation and response!

Note: the issue still exists in the current latest version 2.1.0. So the next release should fix it.

I'll try and send out a release this weekend from master. It'll probably be
a major version release as it's a big refactoring and could break. I'll
post here when it's out.
On Fri, Oct 27, 2023, 10:12 urohith173 @.***> wrote:

Note: the issue still exists in the current latest version 2.1.0. So the
next release should fix it.
—
Reply to this email directly, view it on GitHub
https://github.com/wittlock/ngx-image-zoom/pull/76#issuecomment-1782489022,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAKGGNQKYEZ75ACYPOLSA6TYBNUFBAVCNFSM4UGN7CWKU5DIOJSWCZC7NNSXTN2JONZXKZKDN5WW2ZLOOQ5TCNZYGI2DQOJQGIZA
.
You are receiving this because you were mentioned.Message ID:
@.***>

Alright. Weekend got away from me, but now there's a 3.0.0 up at npm. I hope this works as expected and sorts your ssr related issues. Let me know if there's anything wonky with it.

Working fine 👍.

I think you can close the issue  #63 .


Manual creation of email service domain on communication services

Had to manually provision the email services for sending on top of Azure communications services. It's currently preview so doesn't seem to be available from the azurerm provider

Note that these must have the same data region situation AND be manually connected as well

At present this is still an issue, but there's no evidence to suggest it'll change. In some ways it makes sense but this no longer belongs under here, but rather in icenet-etl, so moving the issue if I can


[BUG] V11 Error #24

Fixed the complaint by adding the ownership attribute to module.json

I have to find time to fix this module... in the meantime I thank you tonight I will implement quesot PR


[Update]: Update config.json to turn test_runner on

Note: this should only be merged once the test runner is working and deployed (which will only be after https://github.com/exercism/cobol-test-runner/pull/2#issuecomment-1195433922 is approved and merged).

@axtens @KTSnowy
This change needs to wait until a test runner is deployed.


PAYSTACK

After paying the modal then waits to show that another modal is loading then it shows connection timed out

After paying I doubt if there is a loading modal from the example but if you can provide more details (sample code, screenshots) bout this issue, it'll be helpful.

Hi yes theirs no modal. That was a error on my part
However once the modal shows up on my debug console it says did not find
Frame lockedcanvas
On Monday, August 19, 2024, Bin Emmanuel @.***> wrote:

After paying I doubt if there is a loading modal from the example but if
you can provide more details (sample code, screenshots) bout this issue,
it'll be helpful.
—
Reply to this email directly, view it on GitHub
https://github.com/binemmanuel/flutter_paystack_max/issues/6#issuecomment-2296535149,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A4M7FNENWSVQAGE7OQDUIULZSHUWFAVCNFSM6AAAAABMVHWGJ6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEOJWGUZTKMJUHE
.
You are receiving this because you authored the thread.Message ID:
@.***>

Also when i cancel payment it still pays
And how do i handle that
this is what i have so far
if (response.status) {
await firestore.createOrder(order);
cartProvider.clearCart();
ScaffoldMessenger.of(context).showSnackBar(const SnackBar(
backgroundColor: Colors.green,
content: Text('Payment successful!'),
));
} else {
ScaffoldMessenger.of(context).showSnackBar(SnackBar(
backgroundColor: Colors.red,
content: Text('Payment failed: ${response.message}'),
));
}

If you log/print the response you'll see that it returns
{
  "status": true,
  "message": "Verification successful",
  "data": {
    "id":<PHONE_NUMBER>,
    "domain": "test",
    "status": "abandoned",
    "reference": "ps_1724349127944175"
  }
}

or
{
  "status": true,
  "message": "Verification successful",
  "data": {
    "id":<PHONE_NUMBER>,
    "domain": "test",
    "status": "success",
    "reference": "ps_1724349091658435"
  }
}

The point is, response.status always returns true and you should keep an eye on response.data.status instead. So, if you update to check for the status like below then you'll be good.
if (response.data.status == PaystackTransactionStatus.success) {
  // Payment was successful
}

PaystackTransactionStatus is an enum that has the following values

PaystackTransactionStatus.reversed
PaystackTransactionStatus.failed
PaystackTransactionStatus.abandoned and
PaystackTransactionStatus.success

thank you


Update README.md (fix broken link - the blog moved)

Updated broken link to the new blog site: https://www.pokt.network/blog/pokt-network-rolling-into-the-modular-future-of-the-protocol-a-technical-deep-dive
Summary
Fixed broken link
Human Summary
AI Summary

Summary generated by Reviewpad on 28 Nov 23 15:21 UTC
This pull request updates the README.md file. It fixes a broken link to the new blog site and replaces it with the correct link: https://www.pokt.network/blog/pokt-network-rolling-into-the-modular-future-of-the-protocol-a-technical-deep-dive.

Sanity Checklist

[ ] I have tested my changes using the available tooling
[x] I have performed a self-review of my own code
[ ] I have commented my code, updated documentation and left TODOs throughout the codebase

Duplicate and/or related to #222

Closing as a duplicate of #222


ekf2: fixed bug in baro ground effect triggering logic

If we have or had valid terrain data then don't use the oversimplified ground effect flag from the land detector.
@dpettenuzzo Could you please review?

@RomanBapst I switched the condition as we discussed.

@RomanBapst I also rebase to make CI work, see #11931
Can you comment about our field test of this PR yesterday?

@MaEtUgR @cmic0 I will check the logs today in the morning.

This was tested on an F450 with a lidar lite module used for terrain estimation.
I verified in the logs that the ground effect flag in the ekf was only true when the vehicle was closer than 0.5 m to the ground.


Creating Loading Spinners and Progress bars for your site.

Proposed title of the article
Loading Spinners and progress bars for your site.
Introduction paragraph (2-3 paragraphs):
We need, at some point, to track the progress of underlying processes of our applications or display to the user that a process is going on. To do that, we need to implement Loading Spinners and Progress bars. In this article, we will look at creating these and apply them in various scenarios such as file reading, reading and writing records to the database, and during loop counters using JavaScript and CSS.
Key takeaways:

Designing the spinners and bars using CSS.
Displaying the elements for underlying processes.
Controlling the display using JavaScript and a backend language(PHP).

References:
Please list links to any published content/research that you intend to use to support/guide this article.
N/A

@munubi254
Good afternoon and thank you for submitting your topic suggestion.
Your topic form has been entered into our queue and should be reviewed (for approval) as soon as a content moderator is finished reviewing the ones in the queue before it.

I found this PR similar to https://github.com/section-engineering-education/engineering-education/issues/3186. Please check it out.

Please close it for me then
On Mon, Aug 16, 2021, 12:59 WanjaMIKE @.***> wrote:

I found this PR similar to
https://github.com/section-engineering-education/engineering-education/issues/3186.
Please check it out.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/section-engineering-education/engineering-education/issues/3353#issuecomment-899382303,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ATOF26F7I2FVKDX6CAMHFCLT5DOOZANCNFSM5CGGNFHQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email
.


Row Select and Expand Row position issue

I used position left both for rowSelect and rowExpand.
Is there any way to render rowSelect before rowExpand in columns.
rowExpand render first automatically

@kaleemullah-innovo sorry we don't support this feature and it's minor feature for me, any PR is weclome. thanks!


Problem with download videos from mirror.co.uk/video

hey guys first of all thank you for this amazing app it really useful, but unfortunately i cant download videos from http://www.mirror.co.uk/video/ i wish if you can fix this problem also make new option that can give me the download link directly for example site.com/video.mp4.
thank you

Which video are you trying to download?

Post an example URL.

this one http://www.mirror.co.uk/news/weird-news/forget-pizza-rat-watch-milkshake-6498785

Seems to be using brightcove.

Yeah, and it's using a custom script to load the Brightcove video.


Fixes #46 issue

Fixes #46 issue.
Wrong word in index.html

Thaaanks! :sparkles:
Can you fix it in the README file as well?

Done .... :heavy_check_mark:
I didn't see the "reposotry" in README.md before. :last_quarter_moon_with_face: I'm sorry.

@tkovs No problem. Thanks again! :four_leaf_clover:


Span width value more than available

For now if I put span 4 and on item span.width = 6, it crash, would be nice to have a dedicated error for this case. Also a mode to allow higher values to narrow down to max span automatically and be safe.

Your idea and PR were good, but I implemented it on a different method so it would crash earlier. Thanks 👍 .


1.9: build caching broken from symlink??

This is an odd one but I tried to extract out the offending files out of a real build.  In the end second build in the below script will end up using the cached image for CP . ./ when it shouldn't
#!/bin/bash

cat > Dockerfile << "EOF"
FROM busybox
COPY . ./
EOF

mkdir -p Godeps/_workspace/src/github.com/fsouza/go-dockerclient/testing/data
ln -s doesnotexist Godeps/_workspace/src/github.com/fsouza/go-dockerclient/testing/data/symlink

docker build -f Dockerfile .
echo ${RANDOM} > "$(date)"
docker build  -f Dockerfile .

This will output
Sending build context to Docker daemon  12.8 kB
Step 1 : FROM busybox
 ---> 8c2e06607696
Step 2 : COPY . ./
 ---> Using cache
 ---> 09b76d77bc36
Successfully built 09b76d77bc36
Sending build context to Docker daemon 13.82 kB
Step 1 : FROM busybox
 ---> 8c2e06607696
Step 2 : COPY . ./
 ---> Using cache
 ---> 09b76d77bc36
Successfully built 09b76d77bc36

Notice the offending lines
Step 2 : COPY . ./
 ---> Using cache
 ---> 09b76d77bc36

Version stuff
docker version; docker info
Client:
 Version:      1.9.0-rc2
 API version:  1.21
 Go version:   go1.4.3
 Git commit:   60d36f7
 Built:        Fri Oct 23 00:20:59 UTC 2015
 OS/Arch:      linux/amd64

Server:
 Version:      1.9.0-rc2
 API version:  1.21
 Go version:   go1.4.3
 Git commit:   60d36f7
 Built:        Fri Oct 23 00:20:59 UTC 2015
 OS/Arch:      linux/amd64
Containers: 17
Images: 1249
Server Version: 1.9.0-rc2
Storage Driver: overlay
 Backing Filesystem: extfs
Execution Driver: native-0.2
Logging Driver: json-file
Kernel Version: 4.0.1-040001-generic
Operating System: Ubuntu 14.10
CPUs: 4
Total Memory: 15.4 GiB
Name: inotmac
ID: FPSO:Z7SO:BZJ3:IXH4:DNKV:UPZQ:KLP2:IRXW:5CBY:PBDM:HAKO:UOC7
Debug mode (server): true
 File Descriptors: 12
 Goroutines: 20
 System Time: 2015-10-22T19:45:15.96379998-07:00
 EventsListeners: 0
 Init SHA1:
 Init Path: /home/darren/Downloads/docker-1.9.0-rc2
 Docker Root Dir: /var/lib/docker
Username: ibuildthecloud
Registry: https://index.docker.io/v1/

Hi!
Please read this important information about creating issues.
If you are reporting a new issue, make sure that we do not have any duplicates already open. You can ensure this by searching the issue list for this repository. If there is a duplicate, please close your issue and add a comment to the existing issue instead.
If you suspect your issue is a bug, please edit your issue description to include the BUG REPORT INFORMATION shown below. If you fail to provide this information within 7 days, we cannot debug your issue and will close it. We will, however, reopen it if you later provide the information.
This is an automated, informational response.
Thank you.
For more information about reporting issues, see https://github.com/docker/docker/blob/master/CONTRIBUTING.md#reporting-other-issues

BUG REPORT INFORMATION
Use the commands below to provide key information from your environment:
docker version:
docker info:
uname -a:
Provide additional environment details (AWS, VirtualBox, physical, etc.):
List the steps to reproduce the issue:
1.
2.
3.
Describe the results you received:
Describe the results you expected:
Provide additional info you think is important:
----------END REPORT ---------
#ENEEDMOREINFO

Somebody let me know if they can reproduce.

@ibuildthecloud hum, using docker-1.9.0-rc2 I cannot reproduce on my laptop.
Sending build context to Docker daemon 6.656 kB
Step 1 : FROM busybox
 ---> d7057cb02084
Step 2 : COPY . ./
 ---> Using cache
 ---> 9f24ac82cfc0
Successfully built 9f24ac82cfc0
Sending build context to Docker daemon  7.68 kB
Step 1 : FROM busybox
 ---> d7057cb02084
Step 2 : COPY . ./
 ---> 2f80eb99c501
Removing intermediate container fac333456dd4
Successfully built 2f80eb99c501

@vdemeester I've yet to find anyone who can reproduce and I haven't reproduced except on my laptop....  But it consistently fails on my laptop.

@ibuildthecloud hum.. more info on your laptop (storage driver & co) ?

I can't repo this

Okay, I'm closing. I wiped out my /var/lib/docker and now I can't reproduce anymore.  So.... fixed that :)

I'm reopening because I consistently get this issue.  I will work on a new test case.

All on overlay? or also on other drivers?

@thaJeztah preparing a new test case, more involved but should be more reproducible...

thanks @ibuildthecloud we've moved this to the 1.9.1 milestone for now, but thanks so much for this :+1:

@thaJeztah I found the root cause and shared it on #docker-maintainer (sorry, on plane, and have spotty internet, i'll try to update issue later)

@ibuildthecloud yup, saw it on IRC, thx!

If this could be fixed in 1.9.0 it would be greatly appreciated :)

The whole team is currently looking at the issue to see if there's a way to still get it fixed before the 1.9 release, but (as you probably understand) no promises :heart:

I'm in SF, I'll personally show up to the office for moral support if needed :smile:  (I think I might have made this release hell...)

@ibuildthecloud unfortunately, it didn't make it in the release, but I'm sure they welcome you for moral support (and proper coffee :smile:) :heart:

we <3 moral support

Just got bit by this on the 1.9.0 stable release. It was not easy to track down either.
Can we put a big notice out that 1.9.0 has a bug that breaks many builds that use symlinks?
Maybe in the release notes?
Thanks!

@tiborvass perhaps on the releases page on GitHub? ^^

I just lost several hours editing code I wasn't running to this exact situation. The failure mode is pretty gnarly. I also have a vendor'd copy of go-dockerclient. It's amusing what broken symlinks can do to you. I would very much appreciate a release of this ASAP so we can use docker for our builds again!

@tiborvass Is this showcased on the releases page on GitHub? I still don't see it?

Containers: 1
 Running: 0
 Paused: 0
 Stopped: 1
Images: 35
Server Version: 1.12.0-rc3
Storage Driver: aufs
 Root Dir: /var/lib/docker/aufs
 Backing Filesystem: extfs
 Dirs: 23
 Dirperm1 Supported: true
Logging Driver: json-file
Cgroup Driver: cgroupfs
Plugins:
 Volume: local
 Network: overlay null bridge host
Swarm: inactive
Runtimes: runc
Default Runtime: runc
Security Options: seccomp
Kernel Version: 4.4.14-moby
Operating System: Alpine Linux v3.4
OSType: linux
Architecture: x86_64
CPUs: 4
Total Memory: 1.954 GiB
Name: moby
ID: ZUAS:CKCG:YAG7:XWIN:TYFK:INOY:TPZE:REI6:DEO7:AQCX:QD4T:UTAU
Docker Root Dir: /var/lib/docker
Debug Mode (client): false
Debug Mode (server): true
 File Descriptors: 16
 Goroutines: 30
 System Time: 2016-07-06T15:57:55.119832527Z
 EventsListeners: 1
No Proxy: *.local, 169.254/16
Username: carbonr
Registry: https://index.docker.io/v1/
Experimental: true
Insecure Registries:
 <IP_ADDRESS>/8

Happening again on this version

@reejosamuel do you have a way to reproduce?

Was this ever added to the release notes?
Thanks,
Trevor
On Wed, Jul 6, 2016 at 12:49 PM, Sebastiaan van Stijn <
<EMAIL_ADDRESS>wrote:

@reejosamuel https://github.com/reejosamuel do you have a way to
reproduce?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/docker/docker/issues/17290#issuecomment-230833588,
or mute the thread
https://github.com/notifications/unsubscribe/ABQGIuYkI25DQMRwJUoKuPc5wtrXUm3-ks5qS9yKgaJpZM4GUPoe
.

It's in the release notes for 1.9.1 https://github.com/docker/docker/releases/tag/v1.9.1


Remove old class path entries. Allow a single test to be executed.

I've backed out a bunch of the testing stuff and the auto-run of CI. We can remove the CI all together really, but I left it in case others thought it was useful.

Thanks @jamezp


Claritiy on administrator requirements / steps for new VMs

Does a support ticket need to be in-place for each new VM?

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

ID: 65e01a44-4fdd-f1fd-f6f6-fc4c333a949a
Version Independent ID: 67a4dc71-9b2f-0587-963d-0475696499e7
Content: Guidance for mitigating speculative execution in Azure
Content Source: articles/virtual-machines/windows/mitigate-se.md
Service: virtual-machines-windows
GitHub Login: @cynthn
Microsoft Alias: cynthn

Thanks for the question! We are investigating and will update you shortly.

@CHDAFNI-MSFT not sure I fully understand your ask. Can you elaborate?

@MicahMcKittrick-MSFT
The article does not state if "Step 1: Contact Azure Support to expose updated firmware (microcode) into your Virtual Machines." is a 1-time action needed for the Subscription, or tenant or VM. If it is not 1-time, how often does it need to be performed i.e. every new VM? every new subscription?
Does this make sense?

@CHDAFNI-MSFT It would just need to be done once. You can provide the engineer a list of your subscription and they can submit the request to apply this to your subscriptions.
It states
Step 1: Contact Azure Support to expose updated firmware (microcode) into your Virtual Machines.
So multiple machines. There is nothing stopping you from requesting multiple subscriptions as well. However in the support request you can only specify a single subscription. But you can just include the other subs in the details.

@MicahMcKittrick-MSFT Can the verbiage be updated though to explicitly call out that that the code is exposed at the subscription level then? This would clear up any confusion if the code exposure is only for existing VMs

@CHDAFNI-MSFT The code would be exposed at the VM level. I am just saying that support will have access to see a complete list of all VMs in your subscription. So step 1 can be applied across the board if requested. After that though the user would have to go into each VM and complete the instructions.

@CHDAFNI-MSFT thanks for chatting with me offline.
The question/ conerns in this doc we are looking to address are:

Does the user need to follow steps 1-4 for each VM they create? For example, if a user follows these steps in their environments and then at a later date adds another VM, does the process of engaging support need to be done each time?  Assuming the user wants enable additional security for every VM they create from here on out.

We noticed the link we provide to engage support automatically fills the fields to make it easier for the user.

Contact Azure Support

We discovered that this link will not populate correctly if the user is using a non-commercial business cloud. Such as Azure Gov. Hence we should likely note in this doc that the link only works for commercial clouds. And either provide a second link that works for Gov or add some info that directs the user what select.
@CHDAFNI-MSFT feel free to add anything I missed.
@cynthn @dlepow do either of you know the answer to point 1 above? Or someone we can reach out to in order to clarify?

@CHDAFNI-MSFT closing this and will follow up with you offline shortly.


⚠ Bofrost.fr has degraded performance

In 1215b6f, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 2222 ms

Resolved: Bofrost.fr performance has improved in 25317fd after 53 minutes.


add Hemant Mishra to contributors list

Hi  @hemant5454, I'm quite elated about your pull request. I wanna evolve this project to addresses various problems faced by first-time contributors. I'd love to learn about your journey in open source community, the problems, pain points you had etc.
Could you explain how you felt when you went through the tutorial, made a pull request and learned that I merged it?
We’ve recently added social share  to our web app. Could you please go to https://roshanjossey.github.io/first-contributions/#social-share and share your first contribution to open source?
Also, check out projects with easy issues while you’re there.

thanks a lot, sir, this is my first and most unexpected experiences regarding GitHub contribution.I  will thankful to you for your great help.

Please don't call me sir. You can call me Roshan.
I’d urge you to continue contributing to other open source projects.
We also have a slack team to help/mentor people trying to contribute to open source projects.
You can join the team by going to https://goo.gl/PM1Jdh


"NgAdminProvider" instead of "nga" ?

Hi,
I'm pretty new in nodejs in general, and ng-admin in particular, so I apologize in advance in case of mistake.
README.md (line 65): does "NgAdminProvider.configure(app);" should be "nga.configure(app);" instead ?
Thanks.

You're right, it should either be nga or NgAdminConfigurationProvider, that's the same. Fixing it.


Microintegrator Dashboard not accessible after fresh installation

Description:
Microintegrator Dashboard not accessible after fresh installation

Hey @invent360, could you please let us know your OS details?

@Vathsan thanks for picking this up.
I am using MacOS Catalina, 10.15.6

I also found out that the Dashboard binary is not found in the https://bintray.com/wso2/binaryGA/ hence the any attempt to build the image fails.

@invent360, you may try starting the dashboard as a service [1] and see if that solves your initial issue.
Also, if you want to use the docker, you may find the images for MI, Monitoring Dashboard is available in [2], [3]
[1] https://ei.docs.wso2.com/en/latest/micro-integrator/setup/installation/install_in_vm_installer/#running-the-mi-dashboard
[2] https://hub.docker.com/r/wso2/wso2mi
[3] https://hub.docker.com/r/wso2/wso2mi-dashboard

@invent360: hope the issue is sorted now. To add to what Vathsan mentioned, you should be able to get the dashboard by started by following the instructions in the installation guide[1].
Closing this issue now. Let us know if you run into any issues.
[1] https://ei.docs.wso2.com/en/latest/micro-integrator/setup/installation/install_in_vm_installer/


Add a Playwright fixture that allows retrieving POST request parameters

Problem

Currently, we use the following code to extract the Analytics payloads from POST requests in Playwright tests:
 page.on("request", (req) => {
    if (req.method() === "POST") {
      const requestData = req.postDataJSON()
      if (requestData?.n == "SELECT_EXTERNAL_SOURCE") {
        eventData = JSON.parse(requestData?.p)
      }
    }
  })

This code is repeated in every test that tests analytics.
Description

Abstract this out into a utility or custom Playwright fixture that provides this more generally (like a list of all events that get sent that can be asserted against).
Alternatives

Additional context

Based on quote from @sarayourfriend 's comment

Added in #1158


feat(testing): Merge special panics like UPDATE=1

Description:
NormalizedOutput::compare_to_file now simply stores the information required for panicking, and the test fails if there's any snapshot mismatch.
Related issue (if exists):

Closing as I don't have time


elide lifetimes

clippy suggestion to remove explicit lifetimes, i.e.
error: the following explicit lifetimes could be elided: 'a
  --> src/param.rs:51:6
   |
51 | impl<'a> AsHciBytes for RemainingBytes<'a> {
   |      ^^                                ^^
   |
   = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_lifetimes
help: elide the lifetimes
   |
51 - impl<'a> AsHciBytes for RemainingBytes<'a> {
51 + impl AsHciBytes for RemainingBytes<'_> {
   |

error: the following explicit lifetimes could be elided: 'a
   --> src/transport.rs:144:6
    |
144 | impl<'a, T: HostToControllerPacket> WriteHci for WithIndicator<'a, T> {
    |      ^^                                                        ^^
    |
    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_lifetimes
help: elide the lifetimes
    |
144 - impl<'a, T: HostToControllerPacket> WriteHci for WithIndicator<'a, T> {
144 + impl<T: HostToControllerPacket> WriteHci for WithIndicator<'_, T> {
    |

Yeah, that's a new clippy lint. We should fix it to avoid noise.

@jamessizeland I could have a look at this at the end of the week?


Put the initializers sequentially so they're run in the right order.

This was introduced in #7432.
Rails goes through the initializers top to bottom and adds
an "after" if it doesn't exist. See here.
When the initializers are in this order top to bottom:

prepare_productization (after append_asset_paths)
load_vmdb_settings     (before load_config_initializers)

Rails ends up altering load_vmdb_settings (adding an after):

prepare_productization (after append_asset_paths)
load_vmdb_settings     (before load_config_initializers, after prepare_productization)

When evaluating the initializers in this way, load_vmdb_settings needs to run
before load_config_initializers and after prepare_productization.
Therefore prepare_productization is forced to run early, BEFORE append_asset_paths,
thereby ignoring our after => append_asset_paths.
This causes productization to break because assets paths prepended by prepare_productization
end up coming after the normal asset paths.
Making prepare_productization follow append_asset_paths fixes this.
On Master, public/assets contains copies from app/assets instead of productization:
RAILS_ENV=production be rake assets:clobber assets:precompile
...

$ ls -l public/assets/bg-login*.png app/assets/images/bg-login.png productization/assets/images/bg-login.png
39807 Mar 18 13:54 app/assets/images/bg-login.png
 5040 Sep  8  2015 productization/assets/images/bg-login.png
39807 Mar 18 13:54 public/assets/bg-login-034ab86f5006293d2c1f776dbb532e16fe16dcc446256b387750348b7dbd0b20.png
34693 Mar 18 13:54 public/assets/bg-login-2-2cfdb45832ff4770b7aa71a49fe8e488bf5acee84b334683d19b15008c805cae.png

After this patch, public/assets contains copies from productization/assets:
RAILS_ENV=production be rake assets:clobber assets:precompile
...

ls -l public/assets/bg-login*.png app/assets/images/bg-login.png productization/assets/images/bg-login.png
39807 Mar 18 13:54 app/assets/images/bg-login.png
 5040 Sep  8  2015 productization/assets/images/bg-login.png
 3428 Sep  8  2015 public/assets/bg-login-2-1991f4c5dfc4e55b0e293a7e29860caea53e2dcbed53d999fe5a1a9f605b29db.png
 5040 Sep  8  2015 public/assets/bg-login-6d6e23c3b3abe1321ed612a89b6b737e4d34cf239358f845d2cbb4e59fc7f06b.png

@simaishi @matthewd @Fryguy Please review.

Note, this seems like a bug in rails that I'll try to fix but it's probably a good idea to put these initializers in order anyway.

Great find LJ.

Thanks @kbrock.  :heart:


feat(accessibilité): utilisation dsfr-toaster-nuxt-module #264

…er les alertes

:tada: This PR is included in version 1.2.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Any way to specify the overshoot from app level ?

what does overshoot do ?

This project is now being maintained by the team at @heroku.
To get the project healthy and maintainable again, we're doing a bit of spring cleaning of older issues and pull requests. This issue hasn't been touched in a while so we're closing it.
Please feel free to re-open if you'd like to resume the conversation.


App doesn’t save “Enlarge links”

As title.

Same as #175


[BUG] Failed at mobile searches?

Before submitting a bug report...

[X] This bug wasn't already reported.
(I have checked every bug report on GitHub)

Title

[X] The title is no longer "[BUG] Title" and I edited it with the right error name.

Branch
master
Commit
142?
Describe the bug
seems to always fail after bing edge searches and exits.
Copy and paste your error
2024-07-09 16:03:38,644 [WARNING] [BING] Possible blockage. Refreshing the page.
2024-07-09 16:03:40,448 [INFO] [BING] Finished Desktop Edge Bing searches !
DevTools listening on ws://<IP_ADDRESS>:61392/devtools/browser/b97b0602-0159-4b56-8688-2f44f6316022
Traceback (most recent call last):
File "C:\Users\steph\GitHub\MS-Rewards-Farmer\main.py", line 43, in main
earned_points = executeBot(currentAccount, args)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "C:\Users\steph\GitHub\MS-Rewards-Farmer\main.py", line 245, in executeBot
with Browser(mobile=True, account=currentAccount, args=args) as mobileBrowser:
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
Screenshots
2024-07-09 16:03:38,644 [WARNING] [BING] Possible blockage. Refreshing the page.
2024-07-09 16:03:40,448 [INFO] [BING] Finished Desktop Edge Bing searches !
DevTools listening on ws://<IP_ADDRESS>:61392/devtools/browser/b97b0602-0159-4b56-8688-2f44f6316022
Traceback (most recent call last):
File "C:\Users\steph\GitHub\MS-Rewards-Farmer\main.py", line 43, in main
earned_points = executeBot(currentAccount, args)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "C:\Users\steph\GitHub\MS-Rewards-Farmer\main.py", line 245, in executeBot
with Browser(mobile=True, account=currentAccount, args=args) as mobileBrowser:
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
Value of dashboard variable
na

Give develop a shot, see if that fixes your issue

Check to see if the new release resolves your issue


Fix Azure Content Safety sample issue

fixes #28441
@joheredi, @mayurid @achandmsft and @lemonicetea for notification.

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

@joheredi, @mayurid @achandmsft and @lemonicetea Could you help review this PR? Thanks a lot.

@jeremymeng updated


RHOAIENG-3128: Make GitOps update pipeline more robust and flexible

JIRA: https://issues.redhat.com/browse/RHOAIENG-3128
Description

The intention of this change is to make the GitOps update pipeline both more robust and more flexible. Before this change, this pipeline was very tied to the way we do things in the example GitOps repositories that we've got in this repo, and users didn't have the ability to deviate from that very much.
This proposed change allows them to specify a script to be run, with the intention that they'll specify one or more yq commands, to update whichever files in their GitOps repository need to be updated with the new image details. IMAGE_NAME and IMAGE_DIGEST for the new image will be available as environment variables to the script at runtime. See the examples for example commands/scripts.
This PR also fixes a minor bug where the git-clone task would just checkout the default branch (i.e. main) despite what the user had specified in the PipelineRun using the gitRepoBaseBranch parameter. While this was previously causing the PR to be correctly created against the expected target branch, the change wasn't being made on top of that branch. This would cause issues if the file(s) that the GitOps Update pipeline was targeting didn't exist on the default branch (would cause pipeline error), or were different enough from the version of the file(s) on the target branch (would cause an unmergeable PR).
Breaking change:
The imageReferenceFilePath parameter has been removed, and the yq-script has been added. In a previous iteration of this PR, the yq-script had a default value, but this has been removed now, since it referred to the imageReferenceFilePath param. Instead, each of the example PipelineRuns contain a yq-script value equivalent to the previous behaviour.
How Has This Been Tested?

~This has just been tested with the existing examples in this repo so far. I will update here when I've tried it with different scenarios, per the acceptance criteria in the Jira issue.~
This has been tested with the pre-existing examples in the repo here (see example PR for bike-rental-app, and example PR for tensorflow-housing-app), and also with the new JSON example GitOps repo file that this PR adds (see example PR).
Merge criteria:

[x] The commits are squashed in a cohesive manner and have meaningful messages.
[x] Testing instructions have been added in the PR body (for PRs involving changes that are not immediately obvious).
[x] The developer has manually tested the changes and verified that the changes work

/auto-cc

/remove-approve
(new process)

Looks good with the new changes too!


National Codes not correct

National Codes are not correct, e.g.
REVISION PROCEDURE TYPE:

ANAESTHETIC_TYPE_(JOINT_REPLACEMENT) has 01, 02, 03, 04, 10, 11, 5, 9
DD ANAESTHETIC TYPE FOR JOINT REPLACEMENT has 5, 9, 10, 11
Comment still valid, codes still incorrect

ASA PHYSICAL STATUS CLASSIFICATION SYSTEM CODE

ASA_PHYSICAL_STATUS_CLASSIFICATION_SYSTEM_CODE_(NATIONAL_JOINT_REGISTRY) has 1, 2, 3, 4, 5, 8
ASA PHYSICAL STATUS CLASSIFICATION SYSTEM CODE FOR NATIONAL JOINT REGISTRY has 1, 2, 3, 4, 8
Codes still not consistent

ASSOCIATED PROCEDURE TYPE

ASSOCIATED_PROCEDURE_TYPE_(ANKLE_REPLACEMENT) has 01, 02, 03, 04, 05, 06, 07, 08, 09, 1, 10, 11, 12, 2, 3, 4, 5, 6, 7, 8, 9
DD ASSOCIATED PROCEDURE TYPE FOR ANKLE REPLACEMENT has 1, 2, 3, 4, 5, 6, 7, 8, 9, 12, 10, 11,
Comment still valid, codes still incorrect

Other examples include BIRTH_LENGTH

This is now correct but how would the user know what the format/length is?

Steve has processed it for you - see example below (**):

file:/Users/steveharris/bitbucket/nhsdd-preprocessing/April_2020_release/transformedoutput/Data_Dictionary/Data_Field_Notes/A/As/ASA_PHYSICAL_STATUS_CLASSIFICATION_SYSTEM_CODE_(NATIONAL_JOINT_REGISTRY).txaClass20
http://www.datadictionary.nhs.uk/data_dictionary/data_field_notes/a/as/asa_physical_status_classification_system_code_(national_joint_registry)_de.asp
fjgl1hl4snp9n-fjh87
Version 3.0<;>3.0.1
ASA_PHYSICAL_STATUS_CLASSIFICATION_SYSTEM_CODE_(NATIONAL_JOINT_REGISTRY)
ASA PHYSICAL STATUS CLASSIFICATION SYSTEM CODES (NATIONAL JOINT REGISTRY)

KernelAssociation20

DDAttribute
ASA_PHYSICAL_STATUS_CLASSIFICATION_SYSTEM_CODE_FOR_NATIONAL_JOINT_REGISTRY

an1

These items are now correct.


[mask] Expected events

To be able to have a nice integration of the mask cool in pyFAI calibration i need to have few signals.

sigMaskSaved(filename)
sigMaskLoaded(filename)
sigChanged

Right now i only use change event from a protected API: masktoolswidget._mask.sigChanged.

Also load and save feature are not an easy task while the application would like to custom the dialog. Then as an integrator, it is difficult to know what to do with it.


[Python] Remove deprecated use_async

Describe the enhancement requested
Has been deprecated since Arrow 7.0.0, time to make things nice and clean: https://issues.apache.org/jira/browse/ARROW-13554
Component(s)
Python

Duplicate of #30774


Change classes to use the new-style

See https://docs.python.org/2/glossary.html#term-new-style-class to learn about new-style classes.
There are two things to make sure to change:

[x] class must subclass object
[x] use super(...) to call a parent's method

Nothing left to be done


Formatting issue secondary_info last_changed

Describe the bug
Thanks for this really great card. I love it!!!
Ran into one of the small issues:
The stock example for secondary info works fine:
secondary_info: '{last_changed}'

Result:

But when I add some additional info, it fails:
secondary_info: 'since {last_changed}'

Result:

Version

V3.1.5

This is expected. To convert value to relative time you need to pass it through the reltime() function

The only reason why '{last_changed}' works is that this was working like this before and when I have added the support for "having a relative-time values between the string" I didn't want to have a "breaking change" hence bumping major card version.
But I agree that now it can be confusing why the val alone works as expected but not between strings.
Please reopen if it doesn't work with the above


{$request.status} should be ${request.status}

I have signed the CLA.

Thanks for the contribution!


[clusteragent/clusterchecks] Add new rebalance based on the workers utilization

What does this PR do?
Adds a new rebalance algorithm for the cluster checks. It takes into account the workers utilization of each runner pod instead of the busyness function that we're currently using.
This is enabled with a config option cluster_checks.rebalance_with_utilization. It should still be considered subject to change until we run more tests. It's disabled by default.
When all the workers of a runner are busy running checks (utilization = 1), they are not able to accept any new requests to run other checks. If other checks need to be run, the agent will be marked as unhealthy and the runner pod will be restarted shortly after. What we're trying to achieve is to avoid that situation by balancing the checks in a way that all the runners are at similar utilization level and none of them approaches a utilization of 1, if there's enough capacity in other runners.
I have included more details about how this works and its limitations in the documentation of the rebalanceUsingUtilization function.
Describe how to test/QA your changes
Try on a Kubernetes cluster with the new option enabled cluster_checks.rebalance_with_utilization. It also needs cluster_checks.advanced_dispatching_enabled set to true.
This needs to be tested in clusters with many cluster checks deployed. The idea would be to check that the utilization among the workers is well balanced (there's a new metric in 7.48: collector.worker_utilization). Also, there should be no or at least less pod restarts than before cause by a worker_utilization of 1. We should also check that this doesn't try to move too many checks when the distribution is already good.
Reviewer's Checklist

[ ] If known, an appropriate milestone has been selected; otherwise the Triage milestone is set.
[ ] Use the major_change label if your change either has a major impact on the code base, is impacting multiple teams or is changing important well-established internals of the Agent. This label will be use during QA to make sure each team pay extra attention to the changed behavior. For any customer facing change use a releasenote.
[ ] A release note has been added or the changelog/no-changelog label has been applied.
[ ] Changed code has automated tests for its functionality.
[ ] Adequate QA/testing plan information is provided if the qa/skip-qa label is not applied.
[ ] At least one team/.. label has been applied, indicating the team(s) that should QA this change.
[ ] If applicable, docs team has been notified or an issue has been opened on the documentation repo.
[ ] If applicable, the need-change/operator and need-change/helm labels have been applied.
[ ] If applicable, the k8s/<min-version> label, indicating the lowest Kubernetes version compatible with this feature.
[ ] If applicable, the config template has been updated.

Rebased on top of main to fix CI issues.


Base protected property replaces child protected property

Look's like a bug

Hi
Could you give an example of the bug in action? I only merge changes that have a unit test that illustrates the change.

Hi, James!
As you asked bug in action:
public class BaseClass
{
    protected virtual string MyProperty { get; set; }
}

public class ChildClass : BaseClass
{
    [JsonProperty]
    protected override string MyProperty { get; set; }

    public string GetPropertyValue()
    {
        return MyProperty;
    }
}

public class CustomContractResolver : DefaultContractResolver
{
    protected override IList<JsonProperty> CreateProperties(Type type, MemberSerialization memberSerialization)
    {
        var jsonProperties = base.CreateProperties(type, memberSerialization);

        Assert.AreEqual(1, jsonProperties.Count, "MyProperty should be founded");

        return jsonProperties;
    }
}

[TestClass]
public class JsonTest
{
    [TestMethod]
    public void Test1()
    {
        var propertyValue = "test";
        var testJson = @"{ 'MyProperty' : '" + propertyValue + "' }";

        var testObject = JsonConvert.DeserializeObject<ChildClass>(testJson, new JsonSerializerSettings
        {
            ContractResolver = new CustomContractResolver()
        });

        Assert.AreEqual(propertyValue, testObject.GetPropertyValue(), "MyProperty should be populated");
    }
}

Hi
I've fixed this but your PR created some failing tests and I used a different approach.
https://github.com/JamesNK/Newtonsoft.Json/commit/5269ce81c0ea4fbfa141cc886fc120e23b21625e


Fixes for num_results=1

Fixes C++ implementation for issue #260 .

Change MdsPlayer::PickMove and MctsNode::MostVisitedPath to break visit count ties using action score.
Always expand the root node if it has no visit counts: when the root has 0 visits, the first readout will expand the children but not actually visit any of them, so child_N(i) will be 0 for all i. By expanding the root if N == 0, SelectLeaf will actually expand the first child. Note that during self play, this initial expansion of the root only happens once.
Also fixes MctsNode::PickMove to squash probabilities before performing soft pick.

/retest

/retest


APEXMALHAR-1852 Attempt to fix the recovery of partial file test

Unrelated test failure
Failed tests:
KafkaInputOperatorTest.testInputOperatorWithFailure:291->testInputOperator:373 TIMEOUT. testName: testtopic0; Collected data: [c1_1, c1_3, c1_5, c1_7, c1_9, c1_11, c1_13, c1_15, c1_17, c1_19, END_TUPLE, c2_2, c2_4, c2_6, c2_8, c2_10, c2_12, c2_14, c2_16, c2_18, c2_20]

Discovered the problem. Some files have 5 blocks and some have 6. This test relies on a file having 5 blocks but any other file can be picked.

Tests pass! Will wait for your take on the two comments.


Some stuff for Windows

Hi!
I've started making plugins for the Corona SDK and decided it might be fun to have a go at doing luaproc. I'm on Windows, so I've approached it from that angle.
I ran into a couple issues. As far as I can tell, this must be the pthreads library in use: pthreads-win32. Unfortunately, the pthread_t type seems to be a struct, throwing off the lua_pushlightuserdata() calls.
_MSC_VER is probably not the appropriate define; it's tied more to that library, but I didn't find any key constants in my admittedly brief search. The Lockless library, on the other hand, seems light userdata-friendly.
That's in lpsched.c; in luaproc.c, Visual Studio doesn't seem to like the static luaproc mainlp; line unless I move the struct beforehand, so I moved the two non-proc structs up with that in mind. Maybe there's a way to allow this, but I don't know it.
After doing the above, plus a couple Corona-specific export details, the test script ran fine.
Anyhow, I just wanted to bring these details to your attention; do with them as you wish.

Hi!
I tried  to compile your version of luaproc with mingw32 via
gcc -c -D_MSC_VER -I%include% *.c
gcc -shared -o luaproc.dll *.o -lpthread -L%lib% -llua
It created a luaproc.dll, and i could run the test file.
However, as soon as i put os.exit() at the end of the file, i get

The application requested the runtime to terminate it in an unusual way .

Any ideas?

Well, since it is running actual threads, maybe it's complaining because they're not done? Do you have any better luck doing a luaproc.wait() before your exit call?

Does not help. It even crashes when i just execute
local luaproc = require "luaproc"
os.exit()

Hmm, weird.
What are you using to run the Lua files?
The DLL registers a __gc that will do an implicit wait on lua_close(), but if that's done through a C++ destructor or __attribute__((destructor)) or something like it, maybe it would blow up. Kind of a wild guess, though. :smile:
Does any of this suggest anything? Similar problem?

I had similar problems in the past:
https://github.com/DhavalKapil/elasticsearch-lua/pull/7
The culprit was lunitx: https://github.com/dcurrie/lunit/blob/master/lua/lunitx/atexit.lua
When running with Lua 5.1 or LuaJIT, the DLL got unloaded before the code in the __gc was executed, resulting in a crash. Maybe this is a similar issue?

@ggcrunchy i compiled Lua  with mingw/msys using the provided makefile.
after than i compiled luaproc, and executed above examples with lua.exe.
so there is no c++ involved here, and i also intepreter and luaproc lib are linked against the same
lua51.dll

it also crashes when i use the lua interpreter interactively

Ok, here is some fix:
The bug seems to be related to the "libgcc_s_dw2-1.dll crashes on exit" bug, which is floating around some while.
It seems that the DWARF version of mingw (and also mingw32 as whole) is broken. I installed the recent SEH version  of the mingw fork mingw-w64. In addition I used the version with posix threads.
Recompiled Lua and luaproc, now it works like this: If I put
os.exit()
at the end of the test file, nothing is shown, and the program just exits, no crash. In the previous case hello world from luaproc
was shown, but after that, it crashed.
When I put os.execute "sleep 1"; os.exit()
at the end, the message is shown and the program quits normally.
In the previous case, adding os.execute "sleep 1" does not help, it will still crash.
When I omit os.execute "sleep 1"; os.exit(), above message is shown, and the program exits also without crash (this is the same in the previous case).
So in the end I have the following results:
Compiled with dwarf mingw32:

test file shoes message, does not crash
test file with exit shows message, crashes
testfile with sleep and exit shows message, crashes

Compiled with seh posix threads mingw-w64:

test file shows message, does not crash
test file with exit does not show message, does not crash
testfile with sleep and exit shows message, does not crash

@blasti Good to hear you've solved it. The newer issue (not specific to luaproc) might simply be a matter of buffering; if so, you could try something like io.output():setvbuf("no")  at the beginning of your program.

@ggcrunchy using io.output():setvbuf("no") seems not to help. but anyway, it is a minor issue, it does not happen if you use luaproc.wait() before os.exit()
`


Create and use global font objects in the UI

Need to use the font and size selected in the options for other UI panels when we make them

No longer needed.


scheduler: top utility

I'm working on https://github.com/getkyo/kyo/issues/347 and need more visibility on the execution of the scheduler to debug. This PR introduces a top utility that can be used in two ways:

Console: It's possible to print the information to the console periodically by setting -Dscheduler.enableTopConsoleMs=<<interval>>. Disabled by default.
JMX: The functionality is exposed as an mbean and can be accessed the main class kyo.scheduler.util.Top. Enabled by default since there shouldn't be a significant performance penalty.

The utility is lightweight and designed to be used in production as well. Demo:

I've addressed the feedback and made a few rendering improvements:


azure/api: remove TestRateLimit

TestRateLimit started failing in some environments [1] after bumping the
Azure SDK in commit fe70a299d356 ("go.mod, vendor: update cloud SDK
modules"). For some reason, the test passes in Travis CI.
[1] https://cilium.slack.com/archives/C7PE7V806/p1642082363019600
It looks like the failure is due to the Azure SDK trying to reach a
hard-coded MSI endpoint. This was worked around previously by setting
the MSI_ENDPOINT env (see #15493), but this doesn't seem to do the trick
anymore.
The main purpose of TestRateLimit is testing the rate-limiting, which is
implemented in a cloud-agnostic way in pkg/api/helpers. That package
already unit-tests that functionality in its own TestRateLimit which
looks almost exactly like TestRateLimit in pkg/azure/api. The only
difference is the creation of the Azure API client, which - as stated
above - seems to be the cause for repeated issues when bumping the Azure
SDK.
Thus, remove the test from pkg/azure/api.

Travis CI passed and review are in, marking as ready to merge.


Change link on the homepage to point to the ng-conf playlist

keynote1 + playlist: https://www.youtube.com/watch?v=QHulaj5ZxbI&index=1&list=PLOETEcp3DkCoNnlhE-7fovYvqwVPrRiY7
or playlist alone: https://www.youtube.com/playlist?list=PLOETEcp3DkCoNnlhE-7fovYvqwVPrRiY7

Can you please also update the link at https://angular.io/docs/js/latest/resources.html and move it before the ng-europe link?

Sure, David updated this earlier but I rolled back because it blew away some previous changes. Can fix tonight

On Mar 11, 2015, at 4:56 PM, Igor Minar<EMAIL_ADDRESS>wrote:
Can you please also update the link at https://angular.io/docs/js/latest/resources.html and move it before the ng-europe link?
—
Reply to this email directly or view it on GitHub.

fixed


Add datetime type & visualization

datetime is a common type in python that we should probably have a good visualization for

I would like to contribute. Can I take it @augray ?

Hi, @tscurtu I have created a PR https://github.com/sematic-ai/sematic/pull/220. Not sure if this is something you had in mind, but let me know what you think 😄  Thanks!


Problem running maven sample plugin

Hello guys,
I'm trying to execute the maven sample plugin (https://github.com/google/error-prone/tree/master/examples/plugin/maven), but it's not working. I just got the sources, then compiled the sample_plugin, and then I executed com.google.errorprone.sample.Hello without any error being displayed.
Is there any special configuration?

Here's how I tested that example:
$ cd examples/plugin/maven
$ mvn package
...
Hello.java:24: error: [MyCustomCheck] String formatting inside print method
    System.err.print(String.format("Hello: %s\n", Arrays.toString(args)));
                    ^
    (see example.com/bugpattern/MyCustomCheck)
  Did you mean 'System.err.printf("Hello: %s\n", Arrays.toString(args));'?
1 error

What maven commands are you running?

Ok Cushon, got it working. :)
The problem was that I've created a concrete BugChecker (DefaultBugChecker extends BugChecker) and I added @AutoService(DefaultBugChecker.class)
But then I've changed to @AutoService(BugChecker.class) and got it working. Thanks!

Great! If you seen any places where things were unclear or poorly documented, feel free to suggest improvements or send a pull request.


Add HomeManager module

In this PR I

split the packages into their own files
created a Home Manager module
added udev rules to the dotool package
added nixpkgs-fmt as formatter and
added aarch64-linux and i686-linux for vosk-bin
(And #4 is included as well, to avoid mistakes during rebasing. So #4 should either be merged first or closed.)

If you prefer, I'll split it into smaller PRs.

I think I'd like to add a bit more information to the README

I made some small changes and submitted them here: https://github.com/Lykos153/numen-nix/pull/1 I can also submit them to this project, but do not think it is maintained anymore. This comment is mainly for visibility, in case someone needs it.


Update flow to v0.54.0

These changes solve the breaking changes of flow v0.53.0, which are
layed out here.
Many of the changes were trivial, however there is one noticeable exception:
The props of the Element HOC has been changed to become all optional (I'm assuming this was the original intention), which in turn forced me to check if the props existing before using them in _setupEventListeners().
Please double check that the changes in src/components/Element.js are correct before considering this PR, I wasn't quite sure about those changes.

Thanks for the PR, @puradox! The change that necessitates checks for all of the event handlers confuses me a bit, because if we typed it as we did before w/ the right defaultProps, they should never be undefined or null, right?

@michelle Oh I must have missed defaultProps in the first run-through! You are correct, they should never be missing so there's no point of having them as optional. I'll make the appropriate changes.

Thanks! \o/

The problem seems more complicated than initially anticipated. Simply removing the optional flags on the Props of Element makes Flow freak out.
It probably comes from the fact that Element really isn't a HOC, but it wants to act like it.

Concretely, a higher-order component is a function that takes a component and returns a new component.

Maybe we should consider changing Element to something like this:
function makeElement(type: string, options: Options = {}): React.StatelessComponent<Props> {
  return (props) => <Element {...props} type={type} options={options} />
}

Sure, that seems reasonable to me. Though, Element components also need access to context.

I've not looked too deeply into it, but could this be related? https://github.com/facebook/flow/issues/4644#issue-251181403

Thanks @michelle for the link, that issue seems to be the problem we're running into here. I'm going to try the changes I mentioned above and see if that solves anything.

Sorry about the messy merge. It was not very fun.
Also, I'm getting eslint errors after the merge:
Cannot read config file: /home/sam/js/react-stripe-elements/node_modules/eslint-config-stripe/index.js
Error: Cannot find module 'eslint-config-airbnb-base'
Referenced from: /home/sam/js/react-stripe-elements/.eslintrc.yml
Error: Cannot read config file: /home/sam/js/react-stripe-elements/node_modules/eslint-config-stripe/index.js
Error: Cannot find module 'eslint-config-airbnb-base'
Referenced from: /home/sam/js/react-stripe-elements/.eslintrc.yml

When I add the additional devDependencies (which I didn't commit), it works fine. However, shouldn't eslint-config-stripe install those automatically?
Upon yarn install I get these warning:
warning<EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>warning<EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>warning<EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>warning<EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>warning<EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>warning<EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>warning<EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>warning<EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>

Closing since #82 merged the necessary changes

Closing since #82 merged the necessary changes


Android native activity sample?

Thank you very much for this library, I was able to get it working in my Win32 OpenGL app without any issues. However, now I was trying to do the same for Android and I am stuck. I am very new to Android programming, so most likely it's my fault that I cannot get it to work.
I am using the OpenGL C++ Android template generated by Visual Studio. It uses NativeActivity (C++) instead of Java. The Android integration in this library did not support loading the HttpClientRequest java classes in such case, so I've added:
jclass FindClassFromNativeActivity(HCInitArgs* args, JNIEnv* jniEnv, const char* className)
{
jobject nativeActivity = args->applicationContext;
jclass acl = jniEnv->GetObjectClass(nativeActivity);
jmethodID getClassLoader = jniEnv->GetMethodID(acl, "getClassLoader", "()Ljava/lang/ClassLoader;");
jobject cls = jniEnv->CallObjectMethod(nativeActivity, getClassLoader);
jclass classLoader = jniEnv->FindClass("java/lang/ClassLoader");
jmethodID findClass = jniEnv->GetMethodID(classLoader, "loadClass", "(Ljava/lang/String;)Ljava/lang/Class;");
jstring strClassName = jniEnv->NewStringUTF(className);
jclass flurryClass = (jclass)(jniEnv->CallObjectMethod(cls, findClass, strClassName));
jniEnv->DeleteLocalRef(strClassName);
return flurryClass;
}
And I replaced the code to load these classes:
jclass localHttpRequest = FindClassFromNativeActivity(args, jniEnv, "com.xbox.HttpClientRequest");
if (localHttpRequest == nullptr)
{
HC_TRACE_ERROR(HTTPCLIENT, "Could not find HttpClientRequest class");
return E_FAIL;
}
This works and gets a reference to the HttpClientRequest instance. As for adapting the Win32 samples, I did:
std::vector<std::vectorstd::string> ExtractAllHeaders(In HCCallHandle call)
{
uint32_t numHeaders = 0;
HCHttpCallResponseGetNumHeaders(call, &numHeaders);
std::vector< std::vector<std::string> > headers;
for (uint32_t i = 0; i < numHeaders; i++)
{
	const char* str;
	const char* str2;
	std::string headerName;
	std::string headerValue;
	HCHttpCallResponseGetHeaderAtIndex(call, i, &str, &str2);
	if (str != nullptr) headerName = str;
	if (str2 != nullptr) headerValue = str2;
	std::vector<std::string> header;
	header.push_back(headerName);
	header.push_back(headerValue);

	headers.push_back(header);
}

return headers;

}
neosmart::neosmart_event_t g_stopRequestedHandle;
neosmart::neosmart_event_t g_workReadyHandle;
neosmart::neosmart_event_t g_completionReadyHandle;
neosmart::neosmart_event_t g_exampleTaskDone;
int g_targetNumThreads = 2;
pthread_t g_hActiveThreads[10] = {  };
int g_defaultIdealProcessor = 0;
int g_numActiveThreads = 0;
void* background_thread_proc(void* lpParam)
{
neosmart::neosmart_event_t hEvents[3] =
{
g_workReadyHandle,
g_completionReadyHandle,
g_stopRequestedHandle
};
JNIEnv* jni;
JavaVM* vm = (JavaVM*)lpParam;
vm->AttachCurrentThread(&jni, NULL);

XTaskQueueHandle queue;
XTaskQueueDuplicateHandle(g_queue, &queue);

bool stop = false;
while (!stop)
{
	int index = 0;
	int dwResult = WaitForMultipleEvents(hEvents, 3, false, -1, index);
	switch (index)
	{
	case 0: // work ready
		if (XTaskQueueDispatch(queue, XTaskQueuePort::Work, 0))
		{
			// If we executed work, set our event again to check next time.
			neosmart::SetEvent(g_workReadyHandle);
		}
		break;

	case 1: // completed
		// Typically completions should be dispatched on the game thread, but
		// for this simple XAML app we're doing it here
		if (XTaskQueueDispatch(queue, XTaskQueuePort::Completion, 0))
		{
			// If we executed a completion set our event again to check next time
			neosmart::SetEvent(g_completionReadyHandle);
		}
		break;

	default:
		stop = true;
		break;
	}
}

XTaskQueueCloseHandle(queue);
return NULL;

}
void CALLBACK HandleAsyncQueueCallback(
In void* context,
In XTaskQueueHandle queue,
In XTaskQueuePort type
)
{
//UNREFERENCED_PARAMETER(context);
//UNREFERENCED_PARAMETER(queue);
switch (type)
{
case XTaskQueuePort::Work:
	neosmart::SetEvent(g_workReadyHandle);
	break;

case XTaskQueuePort::Completion:
	neosmart::SetEvent(g_completionReadyHandle);
	break;
}

}
void StartBackgroundThread(JavaVM* vm)
{
g_stopRequestedHandle = neosmart::CreateEvent(true, false);
g_workReadyHandle = neosmart::CreateEvent(false, false);
g_completionReadyHandle = neosmart::CreateEvent(false, false);
g_exampleTaskDone = neosmart::CreateEvent(false, false);
for (uint32_t i = 0; i < g_targetNumThreads; i++)
{
	pthread_create(&g_hActiveThreads[i], NULL, background_thread_proc, (void*)vm);
}

g_numActiveThreads = g_targetNumThreads;

}
void ShutdownActiveThreads()
{
neosmart::SetEvent(g_stopRequestedHandle);
for (int i = 0; i < g_numActiveThreads; i++) {
	pthread_join(g_hActiveThreads[i], NULL);
}

neosmart::ResetEvent(g_stopRequestedHandle);

}
struct SampleHttpCallAsyncContext
{
HCCallHandle call;
bool isJson;
std::string filePath;
};
void DoHttpCall(std::string url, std::string requestBody, bool isJson, std::string filePath)
{
std::string method = "GET";
bool retryAllowed = true;
std::vector<std::vectorstd::string> headers;
std::vector< std::string > header;
header.clear();
header.push_back("TestHeader");
header.push_back("1.0");
headers.push_back(header);

HCCallHandle call = nullptr;
HCHttpCallCreate(&call);
HCHttpCallRequestSetUrl(call, method.c_str(), url.c_str());
HCHttpCallRequestSetRequestBodyString(call, requestBody.c_str());
HCHttpCallRequestSetRetryAllowed(call, retryAllowed);
for (auto& header : headers)
{
	std::string headerName = header[0];
	std::string headerValue = header[1];
	HCHttpCallRequestSetHeader(call, headerName.c_str(), headerValue.c_str(), true);
}

printf("Calling %s %s\r\n", method.c_str(), url.c_str());

SampleHttpCallAsyncContext* hcContext = new SampleHttpCallAsyncContext{ call, isJson, filePath };
XAsyncBlock* asyncBlock = new XAsyncBlock;
ZeroMemory(asyncBlock, sizeof(XAsyncBlock));
asyncBlock->context = hcContext;
asyncBlock->queue = g_queue;
asyncBlock->callback = [](XAsyncBlock* asyncBlock)
{
	const char* str;
	HRESULT networkErrorCode = S_OK;
	uint32_t platErrCode = 0;
	uint32_t statusCode = 0;
	std::string responseString;
	std::string errMessage;

	SampleHttpCallAsyncContext* hcContext = static_cast<SampleHttpCallAsyncContext*>(asyncBlock->context);
	HCCallHandle call = hcContext->call;
	bool isJson = hcContext->isJson;
	std::string filePath = hcContext->filePath;

	HRESULT hr = XAsyncGetStatus(asyncBlock, false);
	if (FAILED(hr))
	{
		// This should be a rare error case when the async task fails
		printf("Couldn't get HTTP call object 0x%0.8x\r\n", hr);
		HCHttpCallCloseHandle(call);
		return;
	}

	HCHttpCallResponseGetNetworkErrorCode(call, &networkErrorCode, &platErrCode);
	HCHttpCallResponseGetStatusCode(call, &statusCode);
	HCHttpCallResponseGetResponseString(call, &str);
	if (str != nullptr) responseString = str;
	std::vector<std::vector<std::string>> headers = ExtractAllHeaders(call);

	if (!isJson)
	{
		size_t bufferSize = 0;
		HCHttpCallResponseGetResponseBodyBytesSize(call, &bufferSize);
		uint8_t* buffer = new uint8_t[bufferSize];
		size_t bufferUsed = 0;
		HCHttpCallResponseGetResponseBodyBytes(call, bufferSize, buffer, &bufferUsed);
		/*HANDLE hFile = CreateFileA(filePath.c_str(), GENERIC_WRITE, 0, NULL, CREATE_NEW, FILE_ATTRIBUTE_NORMAL, NULL);
		DWORD bufferWritten = 0;
		WriteFile(hFile, buffer, (DWORD)bufferUsed, &bufferWritten, NULL);
		CloseHandle(hFile);*/
		delete[] buffer;
	}

	HCHttpCallCloseHandle(call);

	printf("HTTP call done\r\n");
	printf("Network error code: 0x%0.8x\r\n", networkErrorCode);
	printf("HTTP status code: %d\r\n", statusCode);

	int i = 0;
	for (auto& header : headers)
	{
		printf("Header[%d] '%s'='%s'\r\n", i, header[0].c_str(), header[1].c_str());
		i++;
	}

	if (isJson && responseString.length() > 0)
	{
		// Returned string starts with a BOM strip it out.
		uint8_t BOM[] = { 0xef, 0xbb, 0xbf, 0x0 };
		if (responseString.find(reinterpret_cast<char*>(BOM)) == 0)
		{
			responseString = responseString.substr(3);
		}
		web::json::value json = web::json::value::parse(utility::conversions::to_string_t(responseString));;
	}

	if (responseString.length() > 200)
	{
		std::string subResponseString = responseString.substr(0, 200);
		printf("Response string:\r\n%s...\r\n", subResponseString.c_str());
	}
	else
	{
		printf("Response string:\r\n%s\r\n", responseString.c_str());
	}

	neosmart::SetEvent(g_exampleTaskDone);
	delete asyncBlock;
};

HCHttpCallPerformAsync(call, asyncBlock);

neosmart::WaitForEvent(g_exampleTaskDone, -1ul);

}
And my android_main:
void android_main(struct android_app* state) {
struct engine engine;
JNIEnv* jni;
JavaVM* vm = state->activity->vm;
vm->AttachCurrentThread(&jni, NULL);

HCInitArgs args;
args.javaVM = vm;
args.applicationContext = state->activity->clazz;

HCInitialize(&args);

XTaskQueueCreate(XTaskQueueDispatchMode::Manual, XTaskQueueDispatchMode::Manual, &g_queue);
XTaskQueueRegisterMonitor(g_queue, nullptr, HandleAsyncQueueCallback, &g_callbackToken);
HCTraceSetTraceToDebugger(true);
StartBackgroundThread(vm);

std::string url1 = "https://.../tiger.svg";
DoHttpCall(url1, "", false, "tiger.svg");`

This creates the threads as in the Win32 sample, but to be able to call into Java code I had to call JavaVM->AttachCurrentThread in my background threads.
The issue with this approach seems to be that the HCInitialize is called from one thread, but the Http requests are processed from other threads, internally sharing the HttpClientRequest java object instance from the HCInitialize call cross-threads - and the JNI apparently does not support it, so I get SIGKill and my app is terminated once I attempt to access HttpClientRequest from another thread.
Putting HCInitialize on the same thread that calls into HttpClientRequest solves the issue, but then I get a deadlock right after executing the HTTP call...
I am thinking that perhaps my approach is completely wrong and this is not how the library is supposed to be used on Android. Could you please provide some working example of an Android NativeActivity implementation using this library?

Hi! Thanks for the question and background info. Apologies for the slow response.
Unfortunately, I don't know of any examples around using libHttpClient in this way. However, you shouldn't have to call into the Java layer yourself - the API for libHttpClient is intentionally a C layer (which will handle calling into Java itself).
I wanted to reply to let you know that in addition, we've deprecated the Visual Studio-based Android build for libHttpClient - more details at issue/announcement #573. Going forward, if you'd like to keep this project as a Visual Studio-based project, we'll recommend that you build libHttpClient externally (via the new build system) and reference the resulting static C++ libraries/Java archive from your VS build, rather than having the existing .vcxprojs referenced in your .sln. That shouldn't, however, affect how you write your code - the interface you'll want to use will still be libHttpClient's C API.


Fill up required POM fields.

Issue #, if available:
Description of changes: Missing a few fields that maven central will need
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Codecov Report

Merging #31 (217a649) into main (22fc85b) will increase coverage by 50.00%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##             main      #31       +/-   ##
===========================================
+ Coverage        0   50.00%   +50.00%
- Complexity      0        3        +3
===========================================
  Files           0        2        +2
  Lines           0       10       +10
  Branches        0        1        +1
===========================================
+ Hits            0        5        +5
- Misses          0        3        +3
- Partials        0        2        +2

Impacted Files
Coverage Δ
Complexity Δ

...y/javaagent/providers/AwsOpenTelemetryFactory.java
62.50% <0.00%> (ø)
3.00% <0.00%> (?%)

...emetry/javaagent/providers/AwsAgentProperties.java
0.00% <0.00%> (ø)
0.00% <0.00%> (?%)

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 22fc85b...217a649. Read the comment docs.


Added CADisplayLink setting for iOS

Added setting for CADisplayLink on iOS, so you no longer need to recompile to change it.

looks good
On 22 October 2015 at 21:35, romulox-x<EMAIL_ADDRESS>wrote:

Added setting for CADisplayLink on iOS, so you no longer need to recompile
to change it.
You can view, comment on, or merge this pull request online at:
https://github.com/okamstudio/godot/pull/2663
Commit Summary

Added setting for CADisplayLink on iOS, so you no longer need to
recompile to change it

File Changes

M platform/iphone/app_delegate.mm
https://github.com/okamstudio/godot/pull/2663/files#diff-0 (2)
M platform/iphone/gl_view.h
https://github.com/okamstudio/godot/pull/2663/files#diff-1 (9)
M platform/iphone/gl_view.mm
https://github.com/okamstudio/godot/pull/2663/files#diff-2 (75)
M platform/iphone/globals/global_defaults.cpp
https://github.com/okamstudio/godot/pull/2663/files#diff-3 (1)

Patch Links:

https://github.com/okamstudio/godot/pull/2663.patch
https://github.com/okamstudio/godot/pull/2663.diff

—
Reply to this email directly or view it on GitHub
https://github.com/okamstudio/godot/pull/2663.