[BUG] file is not the path of pyx file, it points to compiled shared library instead.

Describe the bug
The __file__ value in pyx (cython) file is the path to shared library, it is not point to pyx file.
Code to reproduce the behaviour:
print(__file__)

Expected behaviour
It should point to .pyx file or .py file.
OS
No response
Python version
No response
Cython version
No response
Additional context
One possible solution is to manually modify the value of file in Importer and Loader before execute the module.

@scoder , What is your opinion, *.so file is always located in a temporary directory, it is useless to user.
And __file__ value are supposed to point at source, instead of intermediate file.

What Python and Cython versions have you tried? Does this happen with Cython 3.0?

What is your opinion, *.so file is always located in a
temporary directory, it is useless to user.

Erm, no? It is the file that gets executed. There is actually no guarantee that the original source files that it was created from are available at runtime. They are not needed and must be packaged and installed explicitly by the developer in order to be copied to the runtime environment.

And __file__ value are supposed to point at source, instead of
intermediate file.

I second that. This was corrected in recent Python versions and Cython should behave the same, if possible, because Cython sources are really Python sources.
Note, however, that "file" in non-Cython extension modules does not point to the original .c or .cpp file. Those are even more unlikely to be available at runtime.

What is your opinion, *.so file is always located in a temporary directory, it is useless to user.
Erm, no? It is the file that gets executed. There is actually no guarantee that the original source files that it was created from are available at runtime. They are not needed and must be packaged and installed explicitly by the developer in order to be copied to the runtime environment.
And __file__ value are supposed to point at source, instead of intermediate file.
I second that. This was corrected in recent Python versions and Cython should behave the same, if possible, because Cython sources are really Python sources. Note, however, that "file" in non-Cython extension modules does not point to the original .c or .cpp file. Those are even more unlikely to be available at runtime.

Source file here means the file user going to be edited. I believe the one of the most important purpose of __file__ for user is to locate resource file in relative manner.

@scoder , I also recommend Cython to preserve pyx file in wheel or other binary package, for the easy of debug in the future. Because text file won't take much disk space. And Cython is extension of Python, user choose to use Python because it is easy to use. For tools like Jetbrains PyCharm, it can provide more type hint and autocomplete and backtrace information based on pyx file in the package.

Note that packaging is not done by Cython. We cannot take the decision for you which files you include in your wheels.

one of the most important purpose of __file__ for user is to locate
resource file in relative manner.

For that purpose, pointing to the .so file is probably best, because it is guaranteed to a) exist at runtime and b) be in the right place.
However, I still think we should point to the main source file instead, if available at runtime. And encourage users to include the sources in binary distributions. Not everyone will (want to) do that, though.

However, I still think we should point to the main source file instead, if available at runtime. And encourage users to include the sources in binary distributions. Not everyone will (want to) do that, though.

I disagree with this, partly because it seems hard to get right reliably and partly because __file__ is a useful external runtime check that you've actually used a compiled module when a non-compiled source also exists.
The documentation I could find says

__file__ is optional (if set, value must be a string). It indicates the pathname of the file from which the module was loaded (if loaded from a file), or the pathname of the shared library file for extension modules loaded dynamically from a shared library

https://docs.python.org/3/reference/import.html

To me, __file__ pointing at the .so provides useful diagnostic information about what has actually been imported and we shouldn't lose that. If we want to provide information about the original source it shouldn't be at the expense of being accurate about the imported file

or the pathname of the shared library file for extension modules loaded dynamically from a shared library

https://docs.python.org/3/reference/import.html

Hmm, interesting. I didn't remember that this was documented. However, this wasn't written for the specific case of Cython where the sources are essentially Python sources and often available in a predictable place (next to the binary) at runtime. They are needed in order to show source code in tracebacks, for example.
Let's ask the other way - is there an actual problem with "file" pointing to the binary? Something that does not work currently?

Let's ask the other way - is there an actual problem with "file" pointing to the binary? Something that does not work currently?

No, but since file is defined, why don't we make it more consistent with Python ecosystem and satisfy user demand? If technically possible.

To me, __file__ pointing at the .so provides useful diagnostic information about what has actually been imported and we shouldn't lose that. If we want to provide information about the original source it shouldn't be at the expense of being accurate about the imported file

@da-woods
In my point of will, unlink PyO3 or other ffi stuff, Cython is like a blackbox for write speedup static-typed code to accelerate Python. Even if the document is literally said shared library is OK, but I don't think it is the intent outcome, according to the nature and purpose of Cython.

satisfy user demand

For another data point, I prefer that __file__ point to the actual file imported. If that is a compiled C-extension, it should point to the shared object. It does not interest me so much if the compiled c-extension came from cython, PyO3, pybind11, or hand-written C-code, I want to know what is being loaded into the interpreter.

@scoder
As far as I see, Cython is something fill the gap between pure Python code and C/C++ or Rust(PyO3) binding toward Python. And the advantage of Cython over PyPy or GraalPython is the runtime support to CPython.
For C/C++ or Rust, the language is just a method for producing shared-library, the users doesn't care about language. Why it is OK to do like that for C/C++ or Rust? Because, they have their own ecosystem and market, being a language to develop Python extension module is just a be a negligible use case. But for Cython, the situation is different, Cython cannot exist without Python. If user want to do some system native project completely get rid of Python. This user certainly will not choose Cython. It will directly use Rust or C/C++ to finish the project.
For Cython, the focus should be targeting on the language itself and interoperability between Cython and Python runtime. For Cython currently it translate to C code, maybe in future to have a more performance gain, it directly translate to LLVM IR code. Or use lower overhead technique other than dynamic loading *.so files to keep executing faster. But All those doesn't change the Cython language and how end-user use it. The only difference which user can notice is execution speed.
If Cython want to get interest from new user and new project, beside legacy usage. The points I described above is notable. Otherwise, when Mojo (an AI-inspired general purpose programming language) (https://github.com/modularml/mojo) become open source to public. Cython will likely be deprecated.

I prefer that __file__ point to the actual file imported

Note that this is not even true for Python. The variable now always points to the .py source file, regardless of whether the .py, .pyc or .pyo file was loaded. The reasoning is that the .py file is the most relevant of the three files for users, the only one that users can actually look into and read. I could argue that the same applies to Cython generated modules, in a way.


iOS Authentication Issues

Hello Elringus.
I am getting this error when running on iOS.
UnityGoogleDrive: Failed to execute authorization procedure. Check application settings and credentials. UnityGoogleDrive.AuthController:HandleAccessTokenProviderDone(IAccessTokenProvider) UnityEngine.AsyncOperation:InvokeCompletionEvent()
Settings are correct as it is working on Android.
Any ideas?

Thanks!
The browser does not present the loopback response HTML, but the app finishes the request.

I've published a new release with the fixes for both Android and iOS authentication issues. Please refer to the project's readme to find out how to configure the plugin to work on these platforms.

Great, Thanks!
The only issue I am facing is that my bundle id for my app contains some Capital letters. As this app is already published there is no way to correct this. Is there a workaround so it could still work?

Great. I'll try that. Thanks again

Can you please verify that it's working when you set original bundle id to lowercase (just to be sure that the issue is not with something else).

it is, as soon as bundle Id is lowercase it works.

Ok, thanks for checking. I've created a new issue (#27) and will try to remove that requirement for the app id to be lowercased.

I think I've actually found a temporary workaround for this. I've changed the requested redirect URI so that it will always use lower-cased app id and now the trick with adding lower-cased app id to the Supported URL Schemes list should work (it did in my test project, at least).

Great, that works!!
Good job.


Calling ".create_from_surface()" in a WorkerThreadPool task freezes the game

Godot version
4.0. beta 6 official [7f8ecff]
System information
Windows 10, Vulkan API 1.2.0, NVIDIA GeForce RTX 3070
Issue description
Calling the <MeshDataTool>.create_from_surface() function in a worker pool task causes the game to hang for what seems to be indefinitely. The same behavior happens with a single task and with group tasks.
This issue has the same behavior as #69076.
Calling the .create_from_surface() function as deferred no longer makes it freeze the game but makes the MeshDataTool instance unusable in the task.
Steps to reproduce
Create a new project with a Node3D node and attach one of the two following scripts to it:
Example script with one task:
extends Node3D

func do_stuff() -> void:
	var st := SurfaceTool.new()
	var mdt := MeshDataTool.new()
	var mesh := MeshInstance3D.new()

	var mesh_array: ArrayMesh

	st.begin(Mesh.PRIMITIVE_TRIANGLES)

	st.add_vertex(Vector3(0, 0, 0))
	st.add_vertex(Vector3(1, 0, 1))
	st.add_vertex(Vector3(0, 0, 1))

	st.add_index(0)
	st.add_index(1)
	st.add_index(2)

	mesh_array = st.commit()
	mesh.mesh = mesh_array

	# The following function call freezes the game:
	mdt.create_from_surface(mesh.mesh, 0)

	self.add_child.call_deferred(mesh)

func _ready():
	var task = WorkerThreadPool.add_task(Callable(self, "do_stuff"))
	WorkerThreadPool.wait_for_task_completion(task)

Example script with a group task:
extends Node3D

@onready var tasks: Array = []

func do_stuff(element: int) -> void:
	var st := SurfaceTool.new()
	var mdt := MeshDataTool.new()
	var mesh := MeshInstance3D.new()

	var mesh_array: ArrayMesh

	st.begin(Mesh.PRIMITIVE_TRIANGLES)

	st.add_vertex(Vector3(0, 0, 0))
	st.add_vertex(Vector3(1, 0, 1))
	st.add_vertex(Vector3(0, 0, 1))

	st.add_index(0)
	st.add_index(1)
	st.add_index(2)

	mesh_array = st.commit()
	mesh.mesh = mesh_array

	# The following function call freezes the game:
	mdt.create_from_surface(mesh.mesh, 0)

	self.add_child.call_deferred(mesh)

func _ready():
	for x in range(0, 5):
		tasks.append(x)

	var task = WorkerThreadPool.add_group_task(Callable(self, "do_stuff"), tasks.size())
	WorkerThreadPool.wait_for_group_task_completion(task)

Run the game and it will hang/freeze/become unresponsive.
Minimal reproduction project
N/A

bump: Still having issue in 4.4 dev5/6


Mac OS - Resolution problem / locateonscreen/get position

Hello,
I have a problem with locate onscreen function. I try to let my python script find a specific picture in the screen and then clicking on it. The script finds the specific picture, but it doesn't click right on it.
For example:
The searched picture is at coords 40/255(that says the pyautogui.getposition). When I now try to use locate onscreen and click, it says me the picture is at 80/510. So it always double the coordinates.
I work on a Mac, the screen resolution is 2560x1600. But pyautogui.size says 1280x800, so again the half.
What can I do to fix this issue??
Edit: It seems like my resolution is 2560x1600. But due to retina its just 1280x800. I changed it manually to 2560x1600 and everything works good then. Is there any fix that python use the resolution which is currently used?

You are on a Retina laptop at 2x resolution and have to account for that.  It’s been discussed.

The wrapper python-imagesearch solves this problem.
Check out this lines.
https://github.com/drov0/python-imagesearch/blob/88bf9eecb772ff5e657d4780d65cdff87cc5527d/imagesearch.py#L113-L127
CLOSE


add the os version and excluded processes

add the os version (windows server 2016 later).
add excluded processes (cluster.exe/rhs.exe).
I believe that need to exclude processes because experienced resolving the problem by excluding processes.

@no2601 : Thanks for your contribution! The author(s) have been notified to review your proposed change.


transformer ts/color/hexrgba does not apply to borders

https://github.com/tokens-studio/sd-transforms/blob/ec8d0f32be7158f730e6acdc692c5a0cadaab25e/src/registerTransforms.ts#L47
This transformer only works for color tokens so:
{ ..., value: "rgba(#FFFFFF, 50%)" } => rgba(255, 255, 255, 0.5)
{ ..., value: {..., color: "rgba(#FFFFFF, 50%)" } } => rgba(#FFFFFF, 0.5)
In my project, I changed the transformer to the following as we need to perform this transformer on border tokens:
StyleDictionary.registerTransform({
  name: "ts/color/hexrgba",
  type: "value",
  transitive: true,
  matcher: /** @param {DesignToken} token */ (token) =>
    (typeof token.value === "string" && token.value.startsWith("rgba(#")) ||
    (typeof token.value === "object" &&
      typeof token.value.color === "string" &&
      token.value.color.startsWith("rgba(#")),
  transformer: /** @param {DesignToken} token */ (token) => {
    if (typeof token.value === "string") {
      return transformHEXRGBa(token.value);
    }
    return { ...token.value, color: transformHEXRGBa(token.value.color) };
  },
});

It would be nice to support composed tokens as border

I think this also applies to shadow object values. I agree it would be good for this transformer to match any color token, whether the token value is a color itself or whether the color is nested inside an object.


Add 'dist' directory for webui

consul_0.6.0_web_ui.zip is not wrapped by dist directory.
$ wget https://releases.hashicorp.com/consul/0.6.0/consul_0.6.0_web_ui.zip -O webui.zip
$ unzip webui.zip
Archive:  webui.zip
  inflating: index.html
   creating: static/
  inflating: static/application.min.js
  inflating: static/base.css
  inflating: static/base.css.map
  inflating: static/bootstrap.min.css
  inflating: static/consul-logo.png
  inflating: static/favicon.png
  inflating: static/loading-cylon-purple.svg

so I got error
unzip: can't change directory to 'dist': No such file or directory

This PR is adding mkdir dist before unzip.

Should be fixed in #117.


Blur video background

Currently the background is replaced by a green screen in case no background image is given.
I thought it would be nice to blur the background of the video stream instead.
Do you have any special use case for the green screen mode?
In this case I can add an extra cli argument for the blur.

The green screen mode is essentially useful in combination with other software like OBS Studio for further processing as you can use OBS to filter that solid green background and replace it by more complex compositions that deepbacksub itself does not implement.

Ok, I see.
I changed the PR to add an extra blur option -u.
-b is used for the background image.
Maybe its better to rename this to -i and use -b for the blur?

What about -p bgblur:<strength> (-p for "post-processing"), so we could have others.
Also an idea I had was to blur the mask a bit when copying the image over, so the edges of the selected image part aren't that sharp. The radius for this could similarly be set with something like -p edgeblur:<strength>.

Moving this to experimental for now, so we can experiment a bit more with this feature.

The post-processing option sounds nice. I also thought about an option to dilate the mask, to optimize the clipping result.
I changed the PR to add the  -p bgblur:<strength> option.
Unfortunately there is already the -p "Show progress bar" option. I renamed this to -s for now.

I noticed, the experimental branch is behind master. Do you plan to rebase?

Will have to take a look, but IIRC there's nothing special preventing to rebase experimental ATM. Will do in a moment.

@peckto, @BenBE polite nudge for this PR, anything we want done before merging to experimental ?

@phlash LGTM.


Image snapshot returned HTTP client error

I did open an Issue in the repository of the plugin I'm trying to configure, however, I wonder if the problem I'm experiencing is rather platform agnostic and homebridge related instead.
I'm running iOS 12.1 on an iPhone X (not sure if this matters).
Expected Behavior
I have a camera accessory added to homebridge and streaming video works just fine. But homebridge never triggers my handleSnapshotRequest method. I can see the error mentioned in homebridge-camera-ffmpeg/issues/223 in my console App. My hunch is that this might have to do with the EventedHTTPServer or HAPServer failing early and not forwarding the snapshot request to the plugin to begin with.
Is there any better way to debug homebridge besides launching it with Debug options enabled?
DEBUG=* homebridge -D -P

Versions

Homebridge: 0.4.45
Node:       v10.11.0
OS:         macOS Mohave

Console.app Log
Image snapshot returned HTTP client error: Error Domain=HMErrorDomain Code=58 "Failed with OSStatus code 200405 (Method Not Allowed)" UserInfo={NSLocalizedDescription=Failed with OSStatus code 200405 (Method Not Allowed), NSUnderlyingError=0x100bbaff0 {Error Domain=NSOSStatusErrorDomain Code=200405 "Method Not Allowed" UserInfo={NSLocalizedDescription=Method Not Allowed}}}

[<redacted>] Accessory sent Error Domain=HMErrorDomain Code=58 "Failed with OSStatus code 200405 (Method Not Allowed)" UserInfo={NSLocalizedDescription=Failed with OSStatus code 200405 (Method Not Allowed), NSUnderlyingError=0x100bbaff0 {Error Domain=NSOSStatusErrorDomain Code=200405 "Method Not Allowed" UserInfo={NSLocalizedDescription=Method Not Allowed}}} for snapshot request of resolution HMDVideoResolutionType480x270

Please disregard, it was not a homebridge issue. I merely had a mistake in my camera control...

@aphex3k Any insight into what your issue was? Not sure issue is the same but I cant get handleSnapshotRequest to trigger in my plugin either...


Length of a number and null are allowed

length(null) -- null
length(1234) -- null
type_of(length(<any>)) -- always a "number" or throw an error if not iterable/countable

I've decided to leave the number type for length() because of floating issues but you are right I should have made error for these. But what do you exactly mean by the last one?

As a user, I would expect built-in functions such as length, min, max, etc. to always produce a number, a NaN or throw an error but never a null.


cache llvm library to save time for CI build

Hi @jlebar, can we have a try in CI for this PR?

Hi @scxiao I think there are a few places we should improve. I'll post my comments here a bit later.

Fixed by #3562


fix the bcc installation

bcc-tools will be installed using apk, which will allow automatic updation of the
package
fixes #491
Signed-off-by: Ankur Kothiwal<EMAIL_ADDRESS>

Codecov Report

Merging #510 (0ac647c) into main (09ff1bb) will decrease coverage by 0.06%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #510      +/-   ##
==========================================
- Coverage   37.37%   37.30%   -0.07%
==========================================
  Files          22       22
  Lines        9141     9141
==========================================
- Hits         3416     3410       -6
- Misses       5330     5334       +4
- Partials      395      397       +2

Impacted Files
Coverage Δ

KubeArmor/monitor/syscallParser.go
79.32% <0.00%> (-0.65%)
:arrow_down:

Continue to review full report at Codecov.

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

Hey @Ankurk99, would you be able to take up updating the tests as part of the same issue?

Okay, I will mark this PR as draft till then.

@Ankurk99 so far, we used bcc and bcc-dev. what is the difference between the previous packages and bcc-tools?


Nested Associations

Hi,
I was wondering if there was any way to "restangularize" nested associations without having to perform two API calls?  i.e. Say I had a /ships endpoint and everywhere I use a ship I always need to know about it's crew.  My API actually returns the crew information as part of the response.  e.g. /ships/1 returns:
    {
        id:1,
        name: "Queen Anne's Revenge",
        crew: [{
            id:1,
            name: "Blackbeard"
        }]
    }

What I'd like to be able to do is somehow let restangular know that my response contains a crew collection and that it should also be restangularized to give me all the restangular method decoration goodness on the crew members.
This would also be useful on 'has one" associations too.  ie. (for want of a better example) If requesting by crew member first: /crew/1 returning:
    {
        id:1,
        name: "Blackbeard",
        ship: {
            id:1,
            name: "Queen Anne's Revenge"
        }
    }

It'd be good if we were able to tell it to restangularize Blackbeard's ship.
Great library BTW - Thanks,
Dan

Still not getting the complete picture how you would cleanly implement the case above if "crew" was a collection.
I understand that Restangular.restangularizeCollection(parent, elem, route) is responsible to restangularize the "crew" collection, but from where would i call it?
I tend to hide my Restangular object behind a service and configure them with Restangular.withConfig(). Where would I place the call to restangularizeCollection then?

Hi mgonto, same problem like airist. my restangular object is on a service, and the configuration on a config file for all the app.Where would I place the call to restangularizeElement ?

What i get from my api is a thing like this:
{
  "total": 2,
  "per_page": 10,
  "current_page": 1,
  "last_page": 1,
  "next_page_url": null,
  "prev_page_url": null,
  "from": 1,
  "to": 2,
  "data": [
    {
      "id": 1,
      "start_date": "2006-08-20 05:11:32",
      "end_date": "2009-10-18 23:03:06",
      "created_at": "2016-06-23 08:41:51",
      "updated_at": "2016-06-23 08:41:51",
      "unity": {
        "id": 224,
        "code": 20060810,
      },
      "user": [
        {
          "id": 1,
          "username": "Kiara Runolfsdottir",
          "created_at": "2016-06-23 15:56:56",
          "updated_at": "2016-06-23 15:56:56"
        },
        {
          "id": 2,
          "username": "Zoe Dach",
          "created_at": "2016-06-23 15:56:56",
          "updated_at": "2016-06-23 15:56:56"
        },
        {
          "id": 3,
          "username": "Mrs. Zelma Johnston III",
          "created_at": "2016-06-23 15:56:56",
          "updated_at": "2016-06-23 15:56:56"
        },
      ]
    }
  ]
}

Tnks !

Any help,  please?
The best try that I can get , is having all the test objects on the edit route. using the getList() like:
 this.show = function(id) {
        // GET /api/scale/:id
        var data = _scaleService.getList('scaleuser');
        return data;
}

But with this I can't pass the id, to choose the right data object to be edited. The output of this is $object : Array[2] and I need only the selected object (id).

Ok, after a one day fight, finally I find a solution for my edit route list problem:
Something simple using a javascript promise:
this.show = function(id) {
// GET /api/scale/:id
var data =  _scaleService.getList().then(function (users) {
var data = users[id-1];
scalesCache.put('show' + id, data);
return data;
})
};
@mgonto  I suggest a better documentation about nested objects. Tnks for this amazing plugin!


Contract verifier doesnt wait for DeferredResult

Maybe im doing something wrong.. but seems that using DeferredResult somehow breaks waiting for result in contract verifier (or how to call it:) ). It gets empty result as soon as I return DeferredResult from controller, I believe it should wait for actual response underneath?
contract:
`org.springframework.cloud.contract.spec.Contract.make {
		request {
			method 'GET'
			url '/get'
		}
		response {
            status 200
			body 'Passed'
		}

}`
controller:
`@GetMapping
public DeferredResult get(){
    DeferredResult<String> deferredResult = new DeferredResult<>();
    CompletableFuture.supplyAsync(()-> {
                deferredResult.setResult("Passed");
                return deferredResult;
            }
    );
    return deferredResult;
}

`
running tests:
org.junit.ComparisonFailure: expected:<"[Passed]"> but was:<"[]"> 	at sun.reflect.NativeConstructorAccessorImpl.newInstance0(Native Method) 	at sun.reflect.NativeConstructorAccessorImpl.newInstance(NativeConstructorAccessorImpl.java:62) 	at sun.reflect.DelegatingConstructorAccessorImpl.newInstance(DelegatingConstructorAccessorImpl.java:45) 	at org.springframework.cloud.contract.verifier.tests.TipTest.validate_getContract(TipTest.java:29)
curl:
`curl localhost:8095/get
Passed`
do I have to configure something to use contracts to test my API?

It's not written in the docs but in the response add the async() block
org.springframework.cloud.contract.spec.Contract.make {

        request {
            method 'GET'
            url '/get'
        }
        response {
            status 200
            body 'Passed'
            async()
        }
}

I'll have to update the docs :/

Marcin... I havent even managed to format my question properly.. thanks for fast reply, works! :)

Sweet - I'm not closing this issue - it will be a reminder for me to update the docs. Very good catch!

waaaat - I'm checking it out

maybe it wasnt your change, other docs also have this issue:
https://github.com/spring-cloud/spring-cloud-contract/blob/master/docs/src/main/asciidoc/verifier/stubrunner.adoc

I broke with another commit

OK I fixed it

Should I wait for some CI to build it? Wanted to read this doc :)

It's right above this https://cloud.spring.io/spring-cloud-contract/spring-cloud-contract.html#_messaging_top_level_elements nothing special ;)


Cargo warning

Warning when build project
cargo c
warning: unused manifest key: target.aarch64-unknown-linux-gnu.linker
   Compiling libc v0.2.119

We can probably remove gcc linker from aarch64 target in Cargo.toml.

Ah yes, I forgot to remove that after trying to make aarch64 build in workflows. I'll fix this up.

Fixed in e3c533ffdc3db76d58e8fad305b7a55f3f4bc3ce


Problem with double quotes - Command Line 3.1

Hi,
we have a problem when we try to migrate the following sql :

-- procedure get_stats_bonus_cumul

DROP procedure IF EXISTS get_stats_bonus_cumul;
DELIMITER //
CREATE PROCEDURE get_stats_bonus_cumul (isdn BIGINT, bid VARCHAR(100), dt0 DATE, dt1 DATE)
BEGIN
DECLARE sql_cmd      VARCHAR(10000);
SET sql_cmd = CONCAT(
'SELECT
s1.msisdn AS msisdn,
b.bonus_name AS bonus_name,
"',dt0,'" AS from_date,
"',dt1,'" AS to_date,
s1.qty_earned - COALESCE(s0.qty_earned,0) AS qty_earned,
s1.qty_used - COALESCE(s0.qty_used,0) AS qty_used,
s1.qty_expired - COALESCE(s0.qty_expired,0) AS qty_expired,
b.recommended_price,
b.optin,
b.unit
FROM stats_bonus s1
LEFT JOIN stats_bonus s0 ON (s0.msisdn = s1.msisdn AND s0.agg_date="',dt0,'")
LEFT JOIN bonuses b ON (s1.bonus_id = b.bonus_id)
WHERE s1.agg_date="',dt1,'"');
IF NOT (isdn = 0) THEN
SET sql_cmd = CONCAT(sql_cmd, ' AND s1.msisdn=',isdn);
END IF;
IF(bid <> '') THEN
SET sql_cmd = CONCAT(sql_cmd, ' AND b.bonus_id IN (',bid,')');
END IF;
SET @sqlstatement = sql_cmd;
PREPARE sqlquery FROM @sqlstatement;
EXECUTE sqlquery;
DEALLOCATE PREPARE sqlquery;
END
//
DELIMITER ;
The problem is related to the double quote followed to the single quote .
The issue resolved on https://github.com/flyway/flyway/issues/611 consider only the single quote followed to the double quote.
Please, can you fix this bug, we have a lot of this files to migrate .
Thanks.

Can you distill it down to the simplest possible statement that fails? (I am assuming you are using MySQL...)
Cheers
Axel

Hi Axel,
yes, can you try this statement , the same error :
CREATE PROCEDURE get_stats_bonus_cumul (isdn BIGINT, bid VARCHAR(100), dt0 DATE, dt1 DATE)
DELIMITER
//
BEGIN
DECLARE sql_cmd      VARCHAR(10000);
SET sql_cmd = CONCAT(
'SELECT
"',dt0,'" AS from_date,
FROM stats_bonus s1
WHERE s1.agg_date="',dt1,'"');
SET @sqlstatement = sql_cmd;
PREPARE sqlquery FROM @sqlstatement;
EXECUTE sqlquery;
DEALLOCATE PREPARE sqlquery;
END
//
DELIMITER ;
I confirm you that we use Mysql.
Thanks,
Marco.

Thanks Axel for a quick resolution !


Added blockchain books

Added two books about BlockChain
What does this PR do?
Add Resources
For resources
BlockChain books.
Description
Regarding issue #3223 . I have added two great blockchain guides.
Why is this valuable (or not)
This resource did not exist before.
How do we know it's really free?
One is an official pdf, the other one is the official guide.
For book lists, is it a book?
N.A.
Checklist:

[x] Not a duplicate
[x] Included author(s) if appropriate
[x] Lists are in alphabetical order
[x] Needed indications added (PDF, access notes, under construction)

I shall close this PR as mentioned above. Thanks for all the suggestions everybody has given.


no text if bidcount < 1

The purpose of this PR is to propose a solution to issue #599 .

Cool, CI isn't working for outside contributors, probably needs a few snapshot tests updated, I'll merge later manually. Thanks!

I'm going to close this in favour of #607, which fixes the snapshot tests. Thanks again!


Basic pipeline transaction support

Add support for transaction=True in pipelines. This is pretty much the implementation of "Suggestion three" from the pipelines doc. Right now there's no support for WATCH, and no support for the currently blocked commands, although the latter could be added easily.
The execute_transaction method is based off the equivalent from redis-py, and the tests are based on the existing pipeline tests with the keys changed to work.
This is based off a fork we've been using in production since late last year.

Coverage decreased (-3.4%) to 82.207% when pulling ca83606fd935332e0ee88694f11872eb859e55cc on mc3ander:pipeline-transaction into 8fe8b718a3f4b50ae0f4144205dc6d9f504e7ffc on Grokzen:unstable.


Implement foreman test environment

I added an INSTALL.md with some instructions. This doesn't pass yet. So far this works:
puppet agent -t accounts,prometheus,chrony,yumrepo,auditd,tftp,convenience,debugutils,rsyslog,discovery,puppetserver,host,irqbalance,ssh,lldpd,sysstat

Somehow something breaks the dynflow database if I apply the whole catalog and then foreman doesn't start anymore. I will investigate next week.

@jhoblitt this works for me now:
[root@lsst ~]# puppet agent -t
Info: Using environment 'bastelfreak'
Info: Retrieving pluginfacts
Info: Retrieving plugin
Info: Loading facts
Warning: Facter: invalid or malformed lldp data: MalformedDataError: no lldp interface data found
Warning: The current total number of facts: 2490 exceeds the number of facts limit: 2048
Notice: Requesting catalog from $host:8140 ($ip)
Notice: Catalog compiled by $host
Info: Caching catalog for $host
Info: Applying configuration version '1707679430'
Notice: System Reboot Required Because:
  audit => modified
Notice: Applied catalog in 24.96 seconds
[root@lsst ~]#

There were a few minor hiera changes and I had to update the theforeman/foreman module. I only tested the latest version 24.1.0. The majority of the changes are related to the apache in front of foreman, not foreman itself.
I also think that the foreman module has some ordering issues and cannot apply the foreman_config_entry during the first puppet run and a bunch of foreman related packages are installed afterwards. I didn't dig too deep into this. They were all applied without issues during the second run and the third one is idempotent and has no changes.
To make the testing a bit easier, I made a few of your profiles optional/opt-out and provided those changes as dedicated PRs:

#1048
#1046

For what it's worth, an upgrade to 3.9.1 works as well:
$ git diff bastelfreak -- hieradata/site/test.yaml
diff --git a/hieradata/site/test.yaml b/hieradata/site/test.yaml
index 9c8d5879..fe389676 100644
--- a/hieradata/site/test.yaml
+++ b/hieradata/site/test.yaml
@@ -28,8 +28,8 @@ profile::core::yum::versionlock:
     release: "1.el8"
     before: "Package[puppetdb-termini]"

-foreman::repo::repo: "3.8"
-foreman::version: "3.8.0"
+foreman::repo::repo: "3.9"
+foreman::version: "3.9.1"
 puppetdb::globals::version: '7.16.0'

 profile::core::common::manage_sssd: false


⚠ Bofrost.es has degraded performance

In 88bb6b9, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1216 ms

Resolved: Bofrost.es performance has improved in b9c5cb3 after 6 minutes.


support pause/unpause/start/stop/restart VM

https://github.com/rancher/harvester/issues/66

updated


Remote server configuration

Hello,
I have a machine at home with Ubuntu 18.04 which i would like to use as a remote server for processing images/videos which are send by the app using Alamofire.  I am new to Alamofire and setting up the backend, so i am wondering how to receive the image/video -> feed it into my processing pipeline -> send image/video back to smartphone. Is there any tutorial how to set up this Api? Any step-by-step guide? Thanks!

Sorry, we use our GitHub project for bug reports and feature requests. In the future, you should open questions like this on Stack Overflow.
Cheers. 🍻

From our Contribution Guidelines
Asking Questions
We don't use GitHub as a support forum. For any usage questions that are not specific to the project itself, please ask on Stack Overflow instead. By doing so, you'll be more likely to quickly solve your problem, and you'll allow anyone else with the same question to find the answer. This also allows maintainers to focus on improving the project for others.


Typo at the end of the homepage

The last sentence of the homepage of Fortran-lang.org is ended by two dots:

Or just write Fortran software for your research, business, or schoolwork. You can learn how to get started here..

Although I don't know why because there is only one dot in the index.html file:
You can learn how to get started here.

I think you are looking at the wrong source. The main page is generated from https://github.com/fortran-lang/webpage/blob/main/source/index.md.

Thanks Sebastian.
I have just made a PR for this .md file.


Adding browsing ability to work with  more up-to-date libs/frameworks

YOU MAY DELETE THE ENTIRE TEMPLATE BELOW.
Issue Template
when asking for a code- since using GPT3.5/4- the newer versions of next(13 example) are not reconginzed
Expected Behavior
To be familiar with the latest tools/libs to used
Please describe the behavior you are expecting.
Current Behavior
using only tools since 2021...
What is the current behavior?
Failure Information (for bugs)
Please help provide information about the failure if this is a bug. If it is not a bug, please remove the rest of this template.
Steps to Reproduce
Please provide detailed steps for reproducing the issue.

just ask for a project and the tools will be not up to date..

Failure Logs
Please include any relevant log snippets or files here.

Isn't this a problem with all GPT models?

@AntonOsika it's not about the GPT, it's about using external tooling along with it... Why did you closed the issue? This is very crucial part for the ultimate GPT-engineer.


New alternative import and start process for lite strategies.

What

Introduce new configuration file (yml) format: it must have the following 2 configs
strategy: strategy name (string)
markets: connectors and markets (dictionary)
Example:
strategy: new_strategy
markets:
binance: [BTC-USDT, ETH-USDT],
kucoin: [BTC-USDT]
Upon import, the bot checks if config_map and template are missing, if so the bot applies generic config map to the configuration. The config data type can be derived from yml data type. This requires a new load_yml_into_cm function. The prompt for this generic config should read "Enter value for [config_name] >>> "
config command still works on this generic config map.
The bot applies generic start to the strategy, it initiates markets and instantiates a new strategy object with parameters from the config file.
Since there are no config_map nor template, create command cannot be used to create a new config file.

Definition of Done

New simplified strategy can be imported and started
Existing strategies work without any changes on the import/config/start process

┆Issue is synchronized with this Clickup task by Unito

No longer needed


Needed frontend changes to make it work with Affiliations updates in backend

The backend made some updates to how Affiliations work in this ticket: https://github.com/CDLUC3/dmsp_backend_prototype/issues/67
In order for Affiliations to work on the frontend, we need to make some changes to the code once #67 is merged.
Blocked: on the merge of this PR: https://github.com/CDLUC3/dmsp_backend_prototype/issues/67

I assigned this to Epic Account Management because I believe the signup pages will be using Affiliations search.


Improve building so that a godot plugin gets created

Hey 😊,
it depends on what you exactly want to build. If you follow the instructions, you should get a plugin in the build/py4godot directory.
If you want to create a full plugin for windows and linux, you have to install docker and then run the script create_plugin.py. this should create a plugin like the release at the Godot Asset Library.
I hope, this helps


ADD PHI NETWORK RPC

ADDED PHI NETWORK TO RPC ALREADY ADDED TO ANDROID VERISION
If bug:
Fixes #
If not a bug:
Closes #
1-liner summary:
Details if any (this should probably be in the commit message too):
Screenshots of Before PR and After PR if it's applicable (sometimes this make it much faster/easier to review):
Anything particular to test:

@Phinetwork there are a few more changes needed:

Would you help to fix the indentations and the errors (some missing strings and images)?
There are other files that need to be modified (you can find them by building and looking at the compilation errors)

Alternatively, if you like, grant me write access to your repo https://github.com/Phinetwork/alpha-wallet-ios and I can chip in directly.

ok great thank you for the information and your help we have added you to the repo appreciate your assistance.

ok great thank you for the information and your help we have added you to the repo appreciate your assistance. can help how ever you would like just let me know what lines to fix but gave access to make easy so you can go ahead and fix thank you.

Thanks. Got the invite. I'll look into it and reply here if there's anything else we'll need.

Great thank you.

Have updated the PR. @oa-s would you help do a check and merge?
@Phinetwork would you help give us a mention in https://docs.phi.network/phi-wiki/use-phi-smart-chain/compatible-wallets/create-smart-chain-wallet/additional-wallets-setup?

For Sure We Will Be Highlighting Alpha Wallet To Users As The Go To Wallet For PHI Network. Is it possible you Can Also Add Pricing Api For Prices Show UP That pull from https://info.phiswap.com ? can send you API for price to add

For Sure You can check it has been updated. Highlighted Alpha Wallet To Users As The Best Alternative Go To Wallet For PHI Network on the page requested. Is it possible you Can Also Add Pricing Api For Prices Show UP That pull from https://info.phiswap.com ? can send you API for price to add.
Also was the logo added or do you need a logo url?

I have added the logo into the PR.
For pricing API, would you help to create a new issue (or PR if you like) and we can look at it separately? This PR as it is can be merged by itself.

Ok Great Yes will create a new pull request for api thank you

Thanks for the merge Two questions
what file is the price api for change?
and when can we expect the update to reflect on the app?

Thanks for the merge Two questions
what file is the price api for change?

It depends on how the API is like. Can you help create an issue with how the API works?

and when can we expect the update to reflect on the app? @hboon

We just did a release yesterday, next one might be 2-3 weeks time.

Thanks for the merge Two questions
what file is the price api for change?

It depends on how the API is like. Can you help create an issue with how the API works?

and when can we expect the update to reflect on the app? @hboon

We just did a release yesterday, next one might be 2-3 weeks time.

Ok Great Thanks For The Information & Updated Have Created A New Issue. Thanks Again.


[#7] older numpy

Closes #7

Coverage decreased (-1.03%) to 88.077% when pulling 661e3eb3cf0256ed9232af635be4eb82aeaa7799 on feature/7-numpy-interface into ac8b92a33bde14fdc0fb4e6ae5615a790dd05888 on master.

Coverage increased (+1.4%) to 90.53% when pulling 9f96b351a51b0880ad6db0f1d76164067dec7af3 on feature/7-numpy-interface into ac8b92a33bde14fdc0fb4e6ae5615a790dd05888 on master.


Icon request: Pcloud

Icon request for "Pcloud"
https://my.pcloud.com/
Thank you.

URL of the site or app (required):
Logo resources for the icon (optional, but recommended):

Already exists: https://github.com/aegis-icons/aegis-icons/blob/master/SVG/pCloud.svg

My fault! Excuse me :( Have a good luck!

No problem 😅


Event Request - Event available/updated in BC OnPrem CU3, but not updated on BC Cloud

Hi Team,
Given below events and parameters are present on BC On-Premise CU3, but are missing in BC Cloud.
This is stopping us from creating package and deploying on BC Cloud.

[EventSubscriber(ObjectType::Codeunit, 5760, 'OnBeforePostWhseJnlLine', '', false, false)]
Issue:
procedure OnBeforePostWhseJnlLine_WhsePostRecCU(var PostedWhseReceiptHeader: Record "Posted Whse. Receipt Header"; var PostedWhseReceiptLine: Record "Posted Whse. Receipt Line"; var WhseReceiptLine: Record "Warehouse Receipt Line"; var TempTrackingSpecification: Record "Tracking Specification" temporary; var IsHandled: Boolean)

var TempTrackingSpecification --> Parameter Missing

[EventSubscriber(ObjectType::Codeunit, 7307, 'OnAfterRegisteredWhseActivLineInsert', '', false, false)]
Issue:
OnAfterRegisteredWhseActivLineInsert --> Event Missing

[EventSubscriber(ObjectType::Codeunit, 7313, 'OnBeforeCreateNewWhseActivity', '', false, false)]
Issue:
OnBeforeCreateNewWhseActivity --> Event Missing

Please update/add these events in BC Cloud as well.
Thanks!

We use same code base for OnPrem or Cloud, so you simply need t wait until next update will be deployed.

Got it.. Thanks!


winget downloading wrong version .NET SDK and get stuck/crashes after couple of minutes

Winget gets stuck at downloading dotnet-sdk-6.0.202-win-x64.exe while I have <IP_ADDRESS>07 installed.
If I try the download link separately it seems to be down.
Also the winget upgrade --all does not continue with the next 5 updates of other  software as it just "crashes"/ closes the Terminal screen after 3 of 4 minutes.
Below is what it is trying to do;
(2/7) Found Microsoft .NET SDK [Microsoft.dotnet] Version <IP_ADDRESS>07 This application is licensed to you by its owner. Microsoft is not responsible for, nor does it grant any licenses to, third-party packages. Downloading https://dotnetcli.azureedge.net/dotnet/Sdk/6.0.202/dotnet-sdk-6.0.202-win-x64.exe

Duplicate of #2105

Duplicate of #2105
The bug with a single upgrade failing during upgrade --all should be fixed now. It will be available in the next release.


VTK file format 5.0

I created a vtk file using paraview 5.8 and it wirtes the file using format 5.0. This file contains keywords like offset and connectivity which are not recognize by meshio.

Could you attach a (small) example?

Closing for lack of feedback.


Add a Gitter chat badge to README.md

Teradata/covalent now has a Chat Room on Gitter
@kyleledbetter has just created a chat room. You can visit it here: https://gitter.im/Teradata/covalent.
This pull-request adds this badge to your README.md:

If my aim is a little off, please let me know.
Happy chatting.
PS: Click here if you would prefer not to receive automatic pull-requests from Gitter in future.

handling in #12


how to support preview of new files ?

for instance I was navigating a folder with typescript files (*.ts) and yazi 0.1.3 can't preview them ?
I've found no method in the docs (https://github.com/sxyazi/yazi/blob/main/config/docs/yazi.md). By the way I think it would best for the "docs" folder to live at the top-level/root of the repository otherwise it feels like there are no docs

Hey, please take a try for the latest version of Yazi 0.1.4 to see if the issue still exists, and do a file -bL --mime-type your-ts-file.ts and paste the output here.
I've put the docs link in README

$ file -bL --mime-type vite.config.ts
text/plain

Are you using the Nix package to install Yazi? if so you need to add withFile to ensure the file(1) command is available to Yazi.

yes I am but it's the default anyway https://github.com/NixOS/nixpkgs/blob/421fa98aad6a4028fc090e95199bead30bf669e2/pkgs/applications/file-managers/yazi/default.nix#L9 so it shouldn't be the problem. Also I have file in PATH anyway. Let me know how I can help, if there is a way to log more, or just a strace

I added more logs to the file, https://github.com/sxyazi/yazi/commit/66aecd54024231ed42a070ad6952f84f36365304
Please use it and do a cat ~/.local/state/yazi/yazi.log and paste it here.

the file was created but nothing got logged.

Weird, can you record a GIF, let me know what was happening?

I've added some logging info on the file function and the logging library logs to the file as if it was the terminal.
  2023-09-04T13:32:03.224378Z  INFO core::external::file: output of file: "text/plain\napplication/json\ntext/plain\ntext/plain\ntext/plain\ntext/plain\ntext/plain\n"
    at core/src/external/file.rs:23

  2023-09-04T13:32:03.226552Z  INFO core::external::file: output of file: "text/plain\napplication/json\ntext/plain\ntext/plain\ntext/plain\ntext/plain\ntext/plain\n"
    at core/src/external/file.rs:23

I wonder if it's not linked to a timeout issue, this afternoon I could not preview the cpsell.json in the yazi directory and right now I can :s

From the logs, it appears that there are no issues. Is this specific to a certain type of file or is it affecting all files? Can it be reproduced?

It looks random, I'll leave this on hold and investigate when I have time.

Due to the inability to consistently reproduce the issue, I'm temporarily closing this issue.
If the issue persists, please reply to this issue, and I will reopen it.


Rear camera showing solid green color, front camera is fine.

No preview is showing when user record video from rear camera. Front camera preview is working fine. although recorded video are fine in both cases.  How can i remove green color preview issue. Thanks in advance.

This sample has been migrated to a new location (check README for more information).
As recommended by GitHub, we are closing all issues and pull requests now that this older repo will be archived.
If you still see this issue in the updated repo, please reopen the issue/PR there. Thank you!


Modifying chaincode invoke response message

Description
Modifying existing chaincode invoke response message and log message to clarify that the invoke transaction has been submitted, but not completed.
Motivation and Context
This change is made to clarify that the submission of a transaction does not guarantee it's successful execution. The previous response message has always returned "success", which misled the end user and resulted in issue #835.
Fixes #835
How Has This Been Tested?
I verified that the new response and log message were visible whenever a chaincode invoke request took place. All the current UTs pass and no new tests are needed for this change.
Checklist:

[x] I have added a Signed-off-by
[x] Either no new documentation is required by this change, OR I added new documentation
[x] Either no new tests are required by this change, OR I added new tests
[x] I have run goimports, go vet, and golint. I have cleaned up all valid errors and warnings in code I have added or modified. These tools may generate false positives.  The goal is clean, consistent, and readable code.

Modifying chaincode invoke response message to clarify that the
transaction has been successfully submitted, but not completed.
Fixes #835
Signed-off-by: Anna D Derbakova<EMAIL_ADDRESS>

LGTM. Thanks


cannot connect to API?

Hi, i cannot receive values from SolarEdge inverter. I hide API, but in config is correct long API KEY of course.
Thanks for any advice.
Homebride .v1.5.0
Node.js Version | v14.20.0
<EMAIL_ADDRESS>[10/13/2022, 9:53:40 AM] Registering accessory 'homebridge-solaredge-inverter.SolarEdge Inverter'
    at createError (/usr/local/lib/node_modules/homebridge-solaredge-inverter/node_modules/axios/lib/core/createError.js:16:15)
    at settle (/usr/local/lib/node_modules/homebridge-solaredge-inverter/node_modules/axios/lib/core/settle.js:17:12)
    at IncomingMessage.handleStreamEnd (/usr/local/lib/node_modules/homebridge-solaredge-inverter/node_modules/axios/lib/adapters/http.js:293:11)
    url: 'https://monitoringapi.solaredge.com/site/homeserver/overview?api_key=HIDE_API_KEY',
      servername: 'monitoringapi.solaredge.com',
      _host: 'monitoringapi.solaredge.com',
      'Host: monitoringapi.solaredge.com\r\n' +
      responseUrl: 'https://monitoringapi.solaredge.com/site/homeserver/overview?api_key=HIDE_API_KEY',
    host: 'monitoringapi.solaredge.com',
      _currentUrl: 'https://monitoringapi.solaredge.com/site/homeserver/overview?api_key=HIDE_API_KEY',
      url: 'https://monitoringapi.solaredge.com/site/homeserver/overview?api_key=HIDE_API_KEY',
        'Host: monitoringapi.solaredge.com\r\n' +
      host: 'monitoringapi.solaredge.com',

config:
        {
            "name": "SolarEdge Inverter",
            "manufacturer": "SolarEdge",
            "model": "Inverter",
            "serial": "solaredge-inverter-1",
            "site_id": "homeserver",
            "api_key": "HIDE",
            "update_interval": 15,
            "current": true,
            "currentWatts": true,
            "last_day": true,
            "last_month": true,
            "last_year": false,
            "life_time": true,
            "debug": true,
            "accessory": "SolarEdge Inverter"
        }

your site id doesn't look right - it needs to be the site id from SolarEdge. You can get this from the installer or the App as well as the website it's usually a number.

yes you right, now it's working. Thanks


NTR: Coconuts related terms

Hello @Public-Health-Bioinformatics.
I would like to submit a request of new terms. They are related to coconuts.
Requests
label: coconut product
definition: Products that uses coconut as a material.
subClassOf: plant food product

label: cocos fruit product
definition: Products that uses fruits of coconut as material
exact synonyms: cocos fruit product

label: nata de coco
definition: Nata de coco is a chewy, translucent, jelly-like food produced by the fermentation of coconut water.
    dbxref: https://en.wikipedia.org/wiki/Nata_de_coco

Ok, thanks.  I'll do the requests so far today.

Done!


msmtp: compiles with keyring support by default

As it is recommended by msmtp http://msmtp.sourceforge.net/doc/msmtp.html#Authentication.
Motivation for this change
That's the "cleaner" way to retrieve password and recommended way as well.
Not sure it will pass on mac.
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option build-use-sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

How much does this increase the closure size? I wonder because passwordeval works fine (for me, anyway).

I also used passwordeval but it is a more brittle setup IMO, you need an extra setup. For instance in my case I override msmtp to have access to the python keyring utilities (it's not counted in the msmtp closure size but maybe it should).
As for the closure sizes, I ran sthg like  du -h -c $(nix-store -qR /nix/store/cm0vwg6l0mhbimz01jr2d2c5ng68xn1i-msmtp-1.6.6/bin/msmtp) and apparently it goes from  111 to 125M. I am abit surprised by the difference in size. I wonder if I might have done sthg wrong, the 111M closure was in home-manager while the other was via nix-env.

As long as it doesn't pull in all of gnome or something crazy like that it's not so bad ... I'm happy to defer to the maintainer in any case :)

I'm cool with this although I'm also using passwordeval to retrieve passwords from kwallet, so I will not be able to test using libsecret.

I haven't squashed the commits because I believe you can do it from github UI. Otherwise, I will do it.

Thanks @teto !


GPU Used for training?

Assalamualaikum Brother, thank you very much for your wonderful contribution.
I'd like to ask you - what GPU did you use for training this model, and the batch size?

walikum assalam,
i used NVIDIA RTX A6000
Graphics Card (48 gb VRAM). for batch size it depends on model,check my training notebooks that i have shared in this repo already. for training vits i used
batch_size=104,
eval_batch_size=48,
please refer to this notebook -> https://github.com/mobassir94/comprehensive-bangla-tts/blob/main/bn_vits_tts/train_bangla_vits.ipynb  thanks


[zh]Update existing pages for links with '/zh/' prefix, using new prefix '/zh-cn/'

related issue: https://github.com/kubernetes/website/issues/33955, task 6

I think 3,000 occurrence in 500 files is a little bit too much.
Maybe we can split into few directories for better review.


Can't install on CentOS 7

Hey, man!
I'm just trying to install this wonderful server, following this guide and I have troubles at installation step. It says that my CentOS version is too old, but I've updated it just right before running the installation.
Here some copypaste of my terminal
$ sudo ./install.sh
This will install killinuxfloor on this machine. Press y to continue: y
Checking CentOS version... error! Your CentOS is too old.
Update with yum update, then reboot.

$ rpm -qa centos-release
centos-release-7-7.1908.0.el7.centos.x86_64

Could you provide me with some help on this issue?
Thanks a lot!

Hey! What does yum update do?

Hey! What does yum update do?

$ sudo yum update
Loaded plugins: fastestmirror
Loading mirror speeds from cached hostfile
 * base: mirrors.datahouse.ru
 * extras: mirror.tversu.ru
 * updates: mirrors.datahouse.ru
No packages marked for update

Mhm. What does cat /etc/centos-release say?

CentOS Linux release 7.7.1908 (Core)

Ah okay I forgot there's been no release for a while. Please use the git version.
yum install git
git clone https://github.com/noobient/killinuxfloor.git

Then try again.

Ah okay I forgot there's been no release for a while. Please use the git version.
yum install git
git clone https://github.com/noobient/killinuxfloor.git

Then try again.
Thanks, it helped

Awesome, glad to hear that :)


A workaround for a segmentation fault in vcfallelicprimitives

This is a workaround for a seg. fault in vcfallelicprimitives
caused by a VCF file only containing the header. This happened
only when the file was provided as an argument, i.e.
vcfallelicprimitives header.vcf.gz

See the latest vcflib. Making good progress with the new vcfwave tool.


custom-named-bean cannot be resolved to an object. qute(UndefinedObject)

My named CDI beans are not being resolved by the VSCode plugin, but the ones from Quarkus are.
Example:

images will get marked in yellow: "images cannot be resolved to an object. qute(UndefinedObject)"
For the I18N named bean from quarkus, it does not seem to be an issue.

Could you share your project please.

@geertvanleemputten I see a blank image, could you share your project please.


Make cargo install respect lockfiles by default

Rendered
Currently, cargo install does not respect lockfiles by default, which causes breakages when dependencies release breaking but semver-compatible versions. This RFC proposes to change cargo install to respect lockfiles by default.
There's been a lot of discussion on https://github.com/rust-lang/cargo/issues/7169 without much progress, so I thought it would be productive to open an RFC proposing changing the default behavior of cargo install to respect lockfiles by default, to provide a concrete proposal that can be discussed.
Even if this RFC is ultimately rejected, it will provide some forward progress on the issue, since we can then focus on an alternative to changing the default behavior.

There's been a lot of discussion on rust-lang/cargo#7169 without much progress, so I thought it would be productive to open an RFC proposing changing the default behavior of cargo install to respect lockfiles by default, to provide a concrete proposal that can be discussed.

An RFC isn't the best tool to jumpstart progress but instead engaging with the issue is. If there is enough to move forward, i then recommend using Internals wxth a Pre-RFC as a sounding board to help flesh it out.
For myself, i felt this misses a lot of the nuance for why things are the way they are. Without a clear understanding of the problem, you can't talk much on possible solutions. Personally i would recommend closing this while that is iterated on and then going through the process i mentioned. I expect the conversations would then be much more productive, The caveat ill give is the Cargo team just went through a very contentious RFC and i at least am wanting to rebuild my energy before helping mentor the design of this. I can't speak for the others.

An RFC isn't the best tool to jumpstart progress but instead engaging with the issue is. If there is enough to move forward, i then recommend using Internals wxth a Pre-RFC as a sounding board to help flesh it out.

I definitely understand the sentiment. My thought here is that the issue has been known and talked about for a long time, without seeing much progress. Even rejecting this RFC would be beneficial, since I think a lot of people would like the return of the pre-1.37 behavior of respecting lockfiles, and if that's off the table, the conversation can move forward productively to think about alternatives.

For myself, i felt this misses a lot of the nuance for why things are the way they are. Without a clear understanding of the problem, you can't talk much on possible solutions.

I think I touched on this with:

Currently, when running cargo install, the latest semver-compatible versions
of dependencies will be selected. This can be beneficial if these versions have
bugfixes.

Is there something else I'm missing? I checked out the original PR, and only saw picking up bugfixes and new features as the reason to ignore lockfiles.

Personally i would recommend closing this while that is iterated on and then going through the process i mentioned. I expect the conversations would then be much more productive, The caveat ill give is the Cargo team just went through a very contentious RFC and i at least am wanting to rebuild my energy before helping mentor the design of this. I can't speak for the others.

I'm definitely happy to close this, I don't feel too attached to this PR. However, there has been a lot of discussion in https://github.com/rust-lang/cargo/issues/7169, and my personal read of the discussion there is that many users would prefer that cargo install respect lockfiles, and that at least proposing to do that with an RFC is reasonable. I think I understand the tradeoffs involved, and I personally think that they are in favor of respecting lockfiles, and I'm not sure that much more discussion would help.

Even rejecting this RFC would be beneficial, since I think a lot of people would like the return of the pre-1.37 behavior of respecting lockfiles

From what I can tell, the behavior didn't change in 1.37. From the linked issue:

Ah, yea, in that sense there is no change in behavior. I mis-remembered how it used to work. The present stable behavior for install --path is that Cargo.lock is ignored by default, and --locked can be used to honor it. That hasn't changed. (What did change is installing from a registry, which didn't support lock files at all.)

From what I can tell, the behavior didn't change in 1.37. From the linked issue:

I believe the change in behavior was to cargo install <CRATE_NAME>, i.e., installing from crates.io.

since I think a lot of people would like the return of the pre-1.37 behavior of respecting lockfiles

My memory is that this was never the case. cargo install binary-crate-from-crates.io never respected lock files. Could you say where you got the idea that it used to respect lock files from?

I definitely understand the sentiment. My thought here is that the issue has been known and talked about for a long time, without seeing much progress.

While the problem has been talked about for a while, not all of that has been productive and there is more productive conversation to be had.  If nothing else, a summary of the state of the thread would be a big help.

Even rejecting this RFC would be beneficial, since I think a lot of people would like the return of the pre-1.37 behavior of respecting lockfiles, and if that's off the table, the conversation can move forward productively to think about alternatives.

In my mind, this RFC does not produce a good platform for us to outright reject. Instead I would suggest we postpone.

Is there something else I'm missing?

Yes but, as I said, I'm staying out of this.  You will either need to find another mentor or take on yourself a lot more of the burden to move this forward.

However, there has been a lot of discussion in https://github.com/rust-lang/cargo/issues/7169, and my personal read of the discussion there is that many users would prefer that cargo install respect lockfiles, and that at least proposing to do that with an RFC is reasonable

rust-lang/cargo#2644 is our most upvoted and commented on issue and people would have likely said the same thing.  With a gentle nudge, someone dug into it more and found there were fundamental problems with what "many users would prefer".  While there are differences with this situation, I hope this clarifies that that isn't a metric of its own to base designs decisions off of.

A less strict version of this idea is to enable --locked by default when --version is specified.  This is necessary whenever building against an older rustc.  Because many maintainers in the crate ecosystem do not consider bumping MSRV a breaking change requiring an appropriate semver bump, it is nearly impossible for any non-trivial project to correctly build an older version without passing the --locked flag.
This could be fixed in two ways, either by cargo respecting MSRV when resolving dependencies or by defaulting cargo install to use the lockfile for older versions. Both of these things were probably overlooked because they only seem to come up in the situation of attempting to build against an older rustc without a lockfile.


User Survey

We might consider developing a survey to better understand our users.
Larger questions we should consider

What are larger themes of questions that might actually change our behavior?  There are many questions that we might ask in a survey, but focusing on questions that would meaningfully affect the project will probably keep things more concise and make it more likely that users will complete and share the survey.
To what extent do we share the results?  There is a transparency/privacy tradeoff here.  If we don't publish all data, who gets to see this data?
Technologically how do we construct and distribute the survey?  There are a variety of choices out there that offer some pleasant interactions, but also cost some amount of money.

Goals
Here are some goals that I have for such a survey.  I'll also try to include larger goals from comments below (should any arrive).

How aggressive can we drop older versions of things, like Python 2, or older versions of Numpy/Pandas?
In which domains should we focus to broaden the user base?
Can we find advanced users who are willing to share their story more broadly?
What other technologies are people using alongside Dask so that we can ensure that we work well with this technology.  This takes many forms, including cluster resources managers, code editors, other PyData libraries, and so on.
Should we focus effort more on documentation, features, performance, ... ?

I've added some high level goals in the original comment

Some output from the 2018 Python user survey.  This isn't necessarily a good or bad example, but provides food for thought.
https://www.jetbrains.com/research/python-developers-survey-2018/

IIRC, Bokeh ran a user survey. @bryevdv, are there any write-ups on how that went?

No, I never had time to prioritize any writing about it. The results are in the project team drive and a few of us studied them informally.

@bryevdv any wisdom you have from doing this would be welcome (though of course, no pressure).
Also, is the Bokeh user survey still up somewhere?  It would be interesting to see what you all chose to ask about.

messaging you offline

Questions from the Bokeh user survey (thanks @bryevdv !)

How long have you been using Bokeh?
What country do you reside in?
What platforms do you use Bokeh on?
How did you install Bokeh?
How do you use Bokeh?
In what roles do you use Bokeh?
What resources have you used for support in the last six months?
Describe major or primary uses you have for Bokeh (please include links to public work if possible)
If you can share screen shots of apps or plots you have deployed, please upload them (these may be used publicly in documentation, presentations, or grant applications)
Do you use any other visualization tools regularly?
What is your number one suggestion for improvement? (Please be detailed and specific)

In looking around at different survey tooling slowly converged on https://www.sogosurvey.com/ .  I don't have strong confidence in this though.

A strawman
Who are you?

Academia, industry, student
Domain
Optional, include subdomain as short answer like "astronomy" or "quantitative trading"
Role
Experience with PyData in years

How do you use Dask?

Extent (just curious, from time to time, daily, infrastructurally)
Collections we use (array, dataframe, delayed, futures, xarray, ml, other)
Installation/deployment (pip, conda, git, docker, IT provided)
Do you primarily use Dask interactively (like with Jupyter), in batch scripts, or in automated systems?

How could Dask improve?

What resources do you use?

Tutorial
Dask questions on SO
Issue tracker
Documentation

What would help you the most right now?

More documentation
More examples
Bug fixes
Performance improvements
New features

Is Dask stable enough for you?  Yes/No

What else do you use?

Python 2/3
Other libraries (numpy, pandas, sklearn, TF, PyTorch, Spark, Kafka...)
Is it difficult for you to use newer versions of NumPy/Pandas/Python

Scheduling

Do you use the dask.distributed scheduler?  (yes, no, don't know)

Deployed cluster size

laptop, heavy workstation, 2-10, 10-100, 100-1000

Which kind of cluster architecture do you use the most?

HPC (SLURM/PBS/SGE/LSF/...)
Yarn (default for on-premises Hadoop/Spark clusters)
Kubernetes (on-premises)
Kuberentes (Cloud)

Do you use the standard dask package to deploy, or do you roll your own?
If you roll your own, why?

Which of the following distributed features are most important for you

Security/Auth
Ease of Deployment
Managing Deployments
...
I'm fine.  General documentation, features, performance are more important

Bugs and Features
Help direct our time.  Which of these common bugs and features are very
important to you.
Diagnostics, query optimization, task graph overhead, gpus, IT administration,
TOOD
Willing to tell us more? (Optional)

Can we follow up with you for more information?  Y/N

Would you be open to having your work highlighted in a Dask blog or examples page?  Y/N

Name

Institution

Contact address

Links that you think we would enjoy

Can we share these links publicly?  Y/N

Here is a draft of a form using Google Forms
https://docs.google.com/forms/d/e/1FAIpQLSeG12dOna6dIR85Qza6Cr_CZ9rxqH08pilmeQR7ehaJDu25Fw/viewform

I'm guessing "Option 6" is not a real support channel?

Just went through the survey. A few comments.

Took me 5 minutes 45 seconds, while I was writing comments.
The "role"
It may be worth copying some questions directly from https://www.jetbrains.com/research/python-developers-survey-2018/, so that we can benchmark Dask users vs. the general population fo Python devs
Typo in "What Dask resources...". Github "ssues" should be "issues".
In the "What Dask resources..." question, "Examples" is a bit vague. May way to call out dask-examples binder" or examples.dask.org` by name.
Very user focused. Do we want to hear from non-users, people who have heard of dask, but haven't used it yet? (I don't think this is as valuable)
Can you add "Integration with deep learning frameworks" to the "What common feature requests do you care about most?" grid?

@TomAugspurger , is number 2 done -- did you want to say more about "role" ?

Is the Python 2 or 3? question still appropriate if we are already moving forward with https://github.com/dask/dask/pull/4919

@TomAugspurger , is number 2 done -- did you want to say more about "role" ?

Sorry. I think I was going to say "should the role question be a select one, rather than select multiple". But I think select multiple is fine.

Is the Python 2 or 3? question still appropriate if we are already moving forward with #4919

I'm still curious.  I also think that answering it will make people feel satisfied and generally not add to survey fatigue.

Current status: https://docs.google.com/forms/d/e/1FAIpQLSeG12dOna6dIR85Qza6Cr_CZ9rxqH08pilmeQR7ehaJDu25Fw/viewform
I think I'd like to publish this when we release Dask 2.0 and use that release blogpost to drive traffic to the survey.  I think that ideally this happens in a day or two?

I plan to publish this tomorrow if there are no further comments

OK, I've cleared out the current responses.  I'll push this out on twitter soonish.

This is out.  https://twitter.com/dask_dev/status/1143487932491341824

Thanks @mrocklin

If any Dask owners want access to the data please let me know.  I'm going to opt to not share by default due to user e-mails, but I'm happy to share with people if they ask.


Expression data downloaded file have always RPKM units

Fixed. The RPKM hardcoded text was replaced by the project expression units variable.


fix(e2e, playwright):  Flaky test in multifile-editor.spec.ts

Checklist:

[x] I have read and followed the contribution guidelines.
[x] I have read and followed the how to open a pull request guide.
[x] My pull request targets the main branch of freeCodeCamp.
[x] I have tested these changes either locally on my machine, or GitPod.

Closes https://github.com/freeCodeCamp/freeCodeCamp/issues/52783

Tested code changes on local, it's stable

closing in favour of #53031


Update Blog “close-encounters-1”

Automatically generated by Netlify CMS

👷 Deploy Preview for famous-kulfi-ae21db processing.

Name
Link

🔨 Latest commit
f4dd62a44a962734f3790f6e8a2c95a67f03fd45

🔍 Latest deploy log
https://app.netlify.com/sites/famous-kulfi-ae21db/deploys/65841e0bee31f70008d0b690


Loader mistakenly included in client bundle when no component is exported (Vite - Nightly)

What version of Remix are you using?
nightly
Are all your remix dependencies & dev-dependencies using the same version?

[X] Yes

Steps to Reproduce
I am using the unstable vite plugin while testing an integration with effect.
Clone https://github.com/mikearnaldi/vite-remix-effect/tree/chore/bug-bundling
Run pnpm i && pnpm build
The issue is that in the route https://github.com/mikearnaldi/vite-remix-effect/blob/chore/bug-bundling/app/routes/bugged.tsx the loader code is included in the client bundle.
import { redirect } from "@remix-run/node";
import { Effect } from "effect";
import { effectLoader } from "~/services/Runtime";

export const loader = effectLoader(Effect.sync(() => redirect("/")));

When adding a component as a default export the issue disappears. My feeling is that the compiler fails to eliminate action and loader when there is no default export
Expected Behavior
The following is the output with the route changed to:
import { redirect } from "@remix-run/node";
import { Effect } from "effect";
import { effectLoader } from "~/services/Runtime";

export const loader = effectLoader(Effect.sync(() => redirect("/")));

export function Bugged() {
  return

;
}

As you can see the code of bugged is significantly smaller and doesn't include loader code
vite v4.5.0 building for production...

  ⚠  Remix support for Vite is unstable
     and not recommended for production

✓ 50 modules transformed.
public/build/manifest.json                      1.06 kB │ gzip:  0.26 kB
public/build/assets/bugged-a7e55202.js          0.10 kB │ gzip:  0.11 kB
public/build/assets/root-9fa1810b.js            1.40 kB │ gzip:  0.82 kB
public/build/assets/_index-7668716d.js          1.48 kB │ gzip:  0.67 kB
public/build/assets/jsx-runtime-0fbd76cf.js     7.81 kB │ gzip:  3.00 kB
public/build/assets/components-64e00afe.js     78.34 kB │ gzip: 24.97 kB
public/build/assets/entry.client-cb509d92.js  143.08 kB │ gzip: 46.17 kB
✓ built in 1.34s
vite v4.5.0 building SSR bundle for production...

  ⚠  Remix support for Vite is unstable
     and not recommended for production

✓ 16 modules transformed.
build/assets/00001_create_todos-0f9ff181.js   0.35 kB
build/assets/00002_add_todos-e920757f.js      0.36 kB
build/index.js                               10.07 kB
✓ built in 364ms

Actual Behavior
✓ 863 modules transformed.
public/build/manifest.json                            1.81 kB │ gzip:   0.37 kB
public/build/assets/bugged-d7929f89.js                0.09 kB │ gzip:   0.10 kB
public/build/assets/00001_create_todos-d10ac45c.js    0.34 kB │ gzip:   0.27 kB
public/build/assets/00002_add_todos-f25914cc.js       0.35 kB │ gzip:   0.23 kB
public/build/assets/root-400441ed.js                  1.39 kB │ gzip:   0.82 kB
public/build/assets/_index-d096f098.js                1.47 kB │ gzip:   0.67 kB
public/build/assets/components-2967d421.js           44.44 kB │ gzip:  14.16 kB
public/build/assets/router-6a14ae3e.js               47.71 kB │ gzip:  16.00 kB
public/build/assets/entry.client-214ef79c.js        143.08 kB │ gzip:  46.17 kB
public/build/assets/bugged-29e8fb7c.js              467.75 kB │ gzip: 132.26 kB
✓ built in 4.25s
vite v4.5.0 building SSR bundle for production...

  ⚠  Remix support for Vite is unstable
     and not recommended for production

✓ 16 modules transformed.
build/assets/00001_create_todos-0f9ff181.js   0.35 kB
build/assets/00002_add_todos-e920757f.js      0.36 kB
build/index.js                               10.04 kB
✓ built in 366ms

@mikearnaldi thanks for the bug report!
Indeed, looks like we left a comment to "ignore resource routes" when removing server exports for the browser.
But "ignore" in that context should mean "return an empty module" not "return the original module" 🤦
Should be an easy fix!

@mikearnaldi thanks for the bug report!
Indeed, looks like we left a comment to "ignore resource routes" when removing server exports for the browser. But "ignore" in that context should mean "return an empty module" not "return the original module" 🤦
Should be an easy fix!

Let me know if I can help in any way, congrats on shipping vite integration, for Effect's integration it's a game-changer!

Fixed by #7828

Can confirm, the build errors are gone with that fix 👍


Define tmp as dependencies

As cli.js uses tmp, we need to define it in dependencies instead
of devDependencies. Otherwise we get an error Cannot find module 'tmp' when testing the module via the cli.

Thanks :+1:


Caching external API requests

Hi NekR,
For a project I'm using react boilerplate, which uses your plugin to provide offline experience.
After trying to add the API request(s) to "externals" like this:
new OfflinePlugin({
  relativePaths: false,
  publicPath: '/',
  appShell: '/',

  // No need to cache .htaccess. See http://mxs.is/googmp,
  // this is applied before any match in `caches` section
  excludes: ['.htaccess'],

  caches: {
    main: [':rest:'],

    // All chunks marked as `additional`, loaded after main section
    // and do not prevent SW to install. Change to `optional` if
    // do not want them to be preloaded at all (cached only when first loaded)
    additional: ['*.chunk.js'],
  },

  // Removes warning for about `additional` section usage
  safeToUseOptionalCaches: true,
}),

It didn't work out:

I then searched in the issues here and only found this: https://github.com/NekR/offline-plugin/issues/64
So currently I'm not sure, if 1. I do it wrong, or 2. its not implemented.
May you help me out? :)

@fbnklmn did you try the suggested workaround from the issue you linked?

@GGAlanSmithee Not yet, really. I can give it a try though. Just thought there should be something else than a "workaround" after two years :P

@fbnklmn okay, please try it if possible and tell us how it went.
The wording "workaround" is a bit unfortunate by me, since it's not really the case - the feature is simply not implemented (yet). Given how relatively easy it is to accomplish this might a contributing factor as to why that is the case :)

@fbnklmn did you have a chance to try the suggested solution?

@GGAlanSmithee Yes, and it kinda works, although I think a native implementation in the offline plugin would be more convenient. Maybe even with some caching strategies.

@fbnklmn ok great,
to quote @NekR in #64

I am not sure yet. It's kind of complex and has to be done right.

So, yeah, probably there needs to be a native implementation eventually, but probably not right now, unless someone wants to contribute it :)


fatal error C1083: Cannot open include file: 'include/flutter_secure_storage_windows/flutter_secure_storage_windows_plugin.h': No such file or directory error on Windows

Is there an existing issue for this?

[X] I have searched the existing issues
[X] I have read the guide to filing a bug

Steps to reproduce
My app works on Mac and I want to extend to support for Windows. The error below is preventing my whole app from building and running.
I have both tried Visual Studio 2019 and 2022 and installed respective C++ ATL libraries.
I am aware of these issues as well:
https://stackoverflow.com/q/11123088/1968739
https://stackoverflow.com/q/3898287/1968739
Expected results
My app works on Mac and I want to extend to support for Windows.
Actual results
I got this error:
C:\Users\<user>\test_app\windows\flutter\ephemeral\.plugin_symlinks\flutter_secure_storage_windows\windows\flutter_secure_storage_windows_plugin.cpp(1,10): fatal  error C1083: Cannot open include file: 'include/flutter_secure_storage_windows/flutter_secure_storage_windows_plugin.h': No such file or directory [C:\Users\<user>\test_app\build\windows\x64\plugins\flutter_secure_storage_windows\flutter_secure_storage_windows_plugin.vcxproj]

Please see the verbose logs below for more details.
Code sample
My dependencies:
dependencies:
  flutter:
    sdk: flutter
  flutter_localizations:
    sdk: flutter
  ...
  <my-custom-packages-here>
  ...
  flutter_secure_storage: ^9.0.0
  biometric_storage: ^5.0.0
  local_auth: ^2.1.6
  flutter_local_authentication: ^0.0.3
  system_tray: ^2.0.3
  provider: ^6.0.5
  window_manager: ^0.3.5
  get_it: ^7.6.4
  flutter_local_notifications: ^15.1.1
  flutter_platform_alert: ^0.4.0

Screenshots or Video
VS 2019:

VS 2022:

Logs
Verbose Logs where the error occurred:
...
[ +463 ms] Link:
[        ]   C:\Program Files\Microsoft Visual Studio\2022\Community\VC\Tools\MSVC\14.37.32822\bin\HostX64\x64\link.exe /ERRORREPORT:QUEUE /OUT:"C:\Users\<user>\test_app\build\windows\x64\plugins\window_manager\Debug\window_manager_plugin.dll" /INCREMENTAL /ILK:"window_manager_plugin.dir\Debug\window_manager_plugin.ilk" /NOLOGO ..\..\flutter\Debug\flutter_wrapper_plugin.lib "C:\Users\<user>\test_app\windows\flutter\ephemeral\flutter_windows.dll.lib" kernel32.lib user32.lib gdi32.lib winspool.lib shell32.lib ole32.lib oleaut32.lib uuid.lib comdlg32.lib advapi32.lib /MANIFEST /MANIFESTUAC:"level='asInvoker' uiAccess='false'" /manifest:embed /DEBUG /PDB:"C:/Users/<user>/test_app/build/windows/x64/plugins/window_manager/Debug/window_manager_plugin.pdb" /SUBSYSTEM:CONSOLE /TLBID:1 /DYNAMICBASE /NXCOMPAT /IMPLIB:"C:/Users/<user>/test_app/build/windows/x64/plugins/window_manager/Debug/window_manager_plugin.lib" /MACHINE:X64  /machine:x64 /DLL window_manager_plugin.dir\Debug\window_manager.obj
[        ]   window_manager_plugin.dir\Debug\window_manager_plugin.obj
[ +177 ms]      Creating library C:/Users/<user>/test_app/build/windows/x64/plugins/window_manager/Debug/window_manager_plugin.lib and object C:/Users/<user>/test_app/build/windows/x64/plugins/window_manager/Debug/window_manager_plugin.exp
[ +221 ms]   window_manager_plugin.vcxproj -> C:\Users\<user>\test_app\build\windows\x64\plugins\window_manager\Debug\window_manager_plugin.dll
[  +18 ms] FinalizeBuildStatus:
[        ]   Deleting file "window_manager_plugin.dir\Debug\window_m.0D08092A.tlog\unsuccessfulbuild".
[   +1 ms]   Touching "window_manager_plugin.dir\Debug\window_m.0D08092A.tlog\window_manager_plugin.lastbuildstate".
[   +1 ms] Done Building Project "C:\Users\<user>\test_app\build\windows\x64\plugins\window_manager\window_manager_plugin.vcxproj" (default targets).
[   +1 ms] Done Building Project "C:\Users\<user>\test_app\build\windows\x64\runner\test_app.vcxproj" (default targets) -- FAILED.
[        ] Done Building Project "C:\Users\<user>\test_app\build\windows\x64\ALL_BUILD.vcxproj" (default targets) -- FAILED.
[        ] Done Building Project "C:\Users\<user>\test_app\build\windows\x64\INSTALL.vcxproj" (default targets) -- FAILED.
[  +42 ms] Build FAILED.
[   +7 ms] "C:\Users\<user>\test_app\build\windows\x64\INSTALL.vcxproj" (default target) (1) ->
[        ] "C:\Users\<user>\test_app\build\windows\x64\ALL_BUILD.vcxproj" (default target) (3) ->
[        ] "C:\Users\<user>\test_app\build\windows\x64\runner\test_app.vcxproj" (default target) (4) ->
[        ] "C:\Users\<user>\test_app\build\windows\x64\plugins\flutter_secure_storage_windows\flutter_secure_storage_windows_plugin.vcxproj" (default target) (8) ->
[        ] (ClCompile target) ->
[        ]   C:\Users\<user>\test_app\windows\flutter\ephemeral\.plugin_symlinks\flutter_secure_storage_windows\windows\flutter_secure_storage_windows_plugin.cpp(1,10): fatal  error C1083: Cannot open include file: 'include/flutter_secure_storage_windows/flutter_secure_storage_windows_plugin.h': No such file or directory [C:\Users\<user>\test_app\build\windows\x64\plugins\flutter_secure_storage_windows\flutter_secure_storage_windows_plugin.vcxproj]
[        ]     0 Warning(s)
[        ]     1 Error(s)
[        ] Time Elapsed 00:01:39.81
[  +54 ms] Exception: Build process failed.
[   +6 ms]
           #0      throwToolExit (package:flutter_tools/src/base/common.dart:10:3)
           #1      RunCommand.runCommand (package:flutter_tools/src/commands/run.dart:667:9)
           <asynchronous suspension>
           #2      FlutterCommand.run.<anonymous closure> (package:flutter_tools/src/runner/flutter_command.dart:1350:27)
           <asynchronous suspension>
           #3      AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:150:19)
           <asynchronous suspension>
           #4      CommandRunner.runCommand (package:args/command_runner.dart:212:13)
           <asynchronous suspension>
           #5      FlutterCommandRunner.runCommand.<anonymous closure> (package:flutter_tools/src/runner/flutter_command_runner.dart:348:9)
           <asynchronous suspension>
           #6      AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:150:19)
           <asynchronous suspension>
           #7      FlutterCommandRunner.runCommand (package:flutter_tools/src/runner/flutter_command_runner.dart:294:5)
           <asynchronous suspension>
           #8      run.<anonymous closure>.<anonymous closure> (package:flutter_tools/runner.dart:112:9)
           <asynchronous suspension>
           #9      AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:150:19)
           <asynchronous suspension>
           #10     main (package:flutter_tools/executable.dart:90:3)
           <asynchronous suspension>
2

[  +35 ms] "flutter run" took 133,843ms.
[  +58 ms] ensureAnalyticsSent: 51ms
[        ] Running 1 shutdown hook
[  +13 ms] Shutdown hooks complete
[        ] exiting with code 1

Exited (1).

Flutter Doctor output
Doctor output
[√] Flutter (Channel master, 3.16.0-6.0.pre.29, on Microsoft Windows [Version 10.0.22000.2295], locale en-US)
    • Flutter version 3.16.0-6.0.pre.29 on channel master at C:\tools\flutter_windows_3.10.6-stable\flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 5207a309e8 (34 hours ago), 2023-10-07 02:51:32 -0400
    • Engine revision 8711c1fd21
    • Dart version 3.3.0 (build 3.3.0-0.0.dev)
    • DevTools version 2.28.1

[√] Windows Version (Installed version of Windows is version 10 or higher)

[√] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
    • Android SDK at C:\Users\<user>\AppData\Local\Android\sdk
    • Platform android-34, build-tools 34.0.0
    • Java binary at: C:\Program Files\Android\Android Studio\jbr\bin\java
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-b2043.56-10027231)
    • All Android licenses accepted.

[√] Chrome - develop for the web
    • Chrome at C:\Users\<user>\AppData\Local\Google\Chrome\Application\chrome.exe

[√] Visual Studio - develop Windows apps (Visual Studio Community 2022 17.7.4)
    • Visual Studio at C:\Program Files\Microsoft Visual Studio\2022\Community
    • Visual Studio Community 2022 version 17.7.34031.279
    • Windows 10 SDK version 10.0.22621.0

[√] Android Studio (version 2022.3)
    • Android Studio at C:\Program Files\Android\Android Studio
    • Flutter plugin can be installed from:
       https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
       https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-b2043.56-10027231)

[√] VS Code (version 1.83.0)
    • VS Code at C:\Users\<user>\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension version 3.74.0

[√] Connected device (3 available)
    • Windows (desktop) • windows • windows-x64    • Microsoft Windows [Version 10.0.22000.2295]
    • Chrome (web)      • chrome  • web-javascript • Google Chrome 117.0.5938.134
    • Edge (web)        • edge    • web-javascript • Microsoft Edge 117.0.2045.60

[√] Network resources
    • All expected network resources are available.

• No issues found!

I have both tried Visual Studio 2019 and 2022 and installed respective C++ ATL libraries.
I am aware of these issues as well: https://stackoverflow.com/q/11123088/1968739 https://stackoverflow.com/q/3898287/1968739

I'm not sure why you believe this is related to ATL; the error isn't about ATL headers.
The error here is coming from the build of a third-party plugin, so this needs to be reported to the plugin authors. Closing as this is not a Flutter bug.


[firestore-counter] Use pubsub trigger for controller function

This PR makes the following changes:

swap out the https controller function for a pubsub topic on publish function (as discussed previously)
updates the POSTINSTALL doc to use a pubsub schedule gcloud command. [docs]
fixed some small typos in the code

I also noticed there's several unused imports & exports in the extension, but I've not touched these - I can update this PR to remove them if you'd accept it as part of this PR;

https://github.com/firebase/extensions/blob/39a845f8a0eb6fc84a4342d8b4a8cf941e113a33/firestore-counter/functions/src/common.ts#L85 (delay fn is not used anywhere)
https://github.com/firebase/extensions/blob/39a845f8a0eb6fc84a4342d8b4a8cf941e113a33/firestore-counter/functions/src/aggregator.ts#L18 (FieldPath is unused)
https://github.com/firebase/extensions/blob/39a845f8a0eb6fc84a4342d8b4a8cf941e113a33/firestore-counter/functions/src/common.ts#L18-L20 (all 3 import statements unused)

Thanks

Yes, please remove the unused imports. Thanks!

@Salakar you can now update to EXT_INSTANCE_ID. Let me know if it doesn't work,

@Salakar you can now update to EXT_INSTANCE_ID. Let me know if it doesn't work,

Cool, thanks - I've pushed up the change, will test on Monday (as I can't deploy extensions from local source on the latest tools EAP on my home Mac - I'll follow up on the issue separately but have been using another Mac which has the old EAP installed which I don't have access to right now)

Have tested and confirmed all working. I made some small changes to the gcloud commands in the POSTINSTALL file to fix some small issues with them.
This is ready for final review & merge now, thanks
cc @laurenzlong

We are going to delay the merge (and launch) of this PR for a few weeks. We need to create better UX for breaking changes first. At the moment, users won't be sufficiently alerted of breaking changes when they update.

@Salakar I realized that cloud functions does not support update of trigger type. So this won't work. Can you rename the function another name? (That way, updating the extension will actually delete the old HTTPS function and create a new PubSub function)

@googlebot I consent.

@laurenzlong controller has now been renamed to aggregator

@Salakar We actually realized that there's a way to do this update without it being a breaking change. (Context: HTTP functions that get updated after Jan 15 will still remain public, only newly created HTTP functions will be private)
Can you:

Keep the new pubsub function, but add back an http function called controller. The http function becomes a trampoline that gets a request and simply dumps a message into the pubsub topic. The pubsub function will actually have the business logic
the http function's name will be unchanged (controller) so that it would constitute an in-place update of the function, and it will remain public for existing users
the new pubsub topic can perhaps be named controller-core? @rachelsaunders thoughts?
the postinstall should still instruct users to set up a Cloud Scheduler that publishes to the pubsub topic. But existing users that have schedules that call the HTTP function will still have everything working as before. In the release notes, we put up a "deprecation notice" saying that they should change their schedule to call the pubsub topic instead. (You don't have to worry about the release notes in this PR)

If you can make the change early next week that would be ideal, so we can test it out. Thanks!

Hi Lauren,
As per your request, we have:

Reinstated the controller as a http function that triggers an event on the relevant subscriber (onPublish).
Renamed the subscriber to coreController.
Updated the relevant metadata in extension.yaml.

Notes

We have updated the roles to include pubsub.publisher, and we aren’t sure if this constitutes a breaking change.
I have signed the CLA but I hadn't updated my username on my local machine when I committed the code which is why googlebot doesn't recognise one of the 'co-authors'.

I've updated the naming in the PREINSTALL.md & POSTINSTALL.md to match the exported function naming.
Please be aware that I've named the pubsub function controllerCore as per your previous comment (pending agreement from @rachelsaunders), and reinstated controller as the legacy http function. therefore, the docs make reference to controllerCore.
In light of testing this extension numerous times (installing and uninstalling). It came to a point where I was unable to publish further changes due to this error:
Error: Error occurred while deploying resources: RESOURCE_ERROR: '/deployments/firebase-ext-firestore-counter/resources/worker' Deployment Field Violation Error: gcp-types/cloudfunctions-v1:projects.locations.functions: INVALID_ARGUMENT: Field: event_trigger (Expected value test_shard/workers/{workerId} to match regular expression [^/]+/[^/]+(/[^/]+/[^/]+)*)
I asked @Ehesp to publish from his machine in order to test in my place. He also encountered another error which stopped him from publishing any further:
Error: Error occurred while deploying resources: RESOURCE_ERROR: '/deployments/firebase-ext-firestore-counter/resources/controllerCore' {“ResourceType”:“gcp-types/cloudfunctions-v1:projects.locations.functions”,“ResourceErrorCode”:“400",“ResourceErrorMessage”:“Build failed: Build has timed out”}
We haven't been able to test that this still works since changing to coreController.
I haven't encountered errors publishing any of the other extensions, only firestore-counter.

@russellwheatley Thanks for discovering these really important issues! The first error is due to mis-formed document path (I've pushed up a commit that adds regex validation). I managed to repro the second error when I installed this extension on a project that has the Cloud Function and PubSub APIs turned off (i.e. never used Cloud Functions or Extensions before). I think it's a subtle race condition with when APIs are enabled that caused the build to fail. We'll work on fixing the race condition on the backend, but in the meanwhile I've moved the initialization code for the PubSub instance into the function body itself. So it no longer gets run during the build step when the function is first deployed. @Ehesp can you try again to see if the issue has been resolved?
@rachelsaunders I think this PR is now ready for a last review of copy, and any modifications you wish to make to change Scheduled Function to Cloud Scheduler job like we discussed. I think it's probably easier for you to directly make a commit.

@laurenzlong We have now tested. It is ready to merge once naming & comments have been agreed to, and updated.


Set EnableDiagnostics to false by default for loadtests

@cpanato Not sure what's wrong with the build here, but it looks unrelated.

@crspeller maybe it is related with go 1.11
I cant build this branch in my local machine either

fixed, please take a look @crspeller
I found this answer which also fixed here https://github.com/golang/dep/issues/1439#issuecomment-412402161


File Explorer Directory Tree shuffles position after deleting a file, and it can cause major disorientation

VSCode Version: 1.18.1
OS Version: Win 10 Pro

Steps to Reproduce:

Open VS Code

Open a project that has lots of folders and files

Expand a bunch of directories in the File Explorer (left-hand-side directory tree)

Create a new file inside any folder, arbitrary name

Ensure there is a scrollbar

Open a few files from other areas of the directory tree (not close to the file you just created)

Now, open the file you just created a few seconds ago

Right click the file in the File Explorer

Delete the file

Notice how the directory tree shifts violently

This is the issue I do not like

Reproduces without extensions: Yes
I see what VS Code is doing. When I delete a file while it is currently open, the editor shifts to the previously open file, or whatever file it goes to, but this is very jarring. It happens to me almost every time I delete a file. My current focus is the file I deleted and its position in File Explorer.
It is very disorienting and sometimes I do not recall exactly where the file was that I just deleted, mostly because it was a garbage file somewhere that I was browsing.
I think there is a better solution that needs to be implemented here. I leave that open to interpretation. I do not like how it is currently, and I know others experience this also. Maybe if there was a transition animation so it slowly transitioned away, so I could at least see which direction the directory was.
Right now, I can't even tell if I need to scroll up or down after VS Code shuffles my position. That is the basis of why it's so confusing. I just want to go back to where I just deleted a file.
Here is an animated GIF of me reproducing it:
https://gifyu.com/image/pESP

@amackintosh you can disable the explorer from auto revealing via "explorer.autoReveal": false


Update README.md

I have corrected the English, added proper capitalization and added one line command for installing modules.

Thanku


Test on Linux

Test everything on Linux.

[ ] detect SDKs
[x] add an opam SDK
[x] add an opam-like SDK
[ ] behavior with sources
[x] dune template
[ ] Makefile template

And add tests as we have for WSL, Cygwin, ...

Sources were not added for local SDK


Globally manage DAPR options in kubernetes

/area runtime
Wondering if a way exists to globally set an option like dapr.io/http-max-request-size or dapr.io/http-read-buffer-size. I have several services in kubernetes cluster and instead of putting everywhere these settings I would like to set and manage those centrally (maybe overridden by the deployment itself). Is it possible?
Thanks

Thanks, I think the global configuration makes sense.
Add a configMap maybe named dapr-config-sidecar contains configuration items for the sidecar, injector gets this config map content at startup and uses configs defined as the default value. example:
apiVersion: v1
data:
  http-max-request-size: "4"
  http-read-buffer-size: "4"
  # ... more configs
kind: ConfigMap

Configs from annotions > configs defined in config map > default value.
WDYT? @yaron2

reactivating issue because I think it is important to have such a configuration. @yaron2 what do you think about this?