Capacity priority

Hello all, this one isn't clear to me
Capacity priority prioritizes data center capacity for your deployments.
When I stop a Vm and deallocate and then I start Vm same sku type and size what will happen it could be started in different Dc but same zone or the zone could be changed? In case it is different zone stopped Vm is usually connected to one zone when it is configured. What will happen then? I'm looking for scenario where I have test server in pairex dr region and in case of Dr I would like to stop test and start prod with same zone location and server type and size
Thanks

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

ID: 9be8cdc6-1a0e-25d6-8131-a56c5c0029a5
Version Independent ID: d27499e5-c8d4-2de0-273d-09f64aec3121
Content: Manage Azure Reservations
Content Source: articles/billing/billing-manage-reserved-vm-instance.md
Service: billing
GitHub Login: @yashesvi
Microsoft Alias: cwatson

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

Adding @Drewm3

When you reserve VM instances in Azure there are two options to pick from

Size Flexibility
Capacity Priority

Choosing to use Size Flexibility means that the reservation you paid for can apply to multiple sizes of the same VM series. This means that if you have a reservation for say D3 VMs and have size flexibility selected and want to deploy a VM then we go and check to ensure that size is available and if it is not, then we will allocate your VM to a different size in the D family and apply the reserved discount to that size. This means that even if there was no D3 capacity we would still ensure you are able to deploy a VM of the same D series size and your discount will be applied. Size flexibility is good for when you might not require a specific size but know for sure you want to be able to deploy in that region when needed.
Capacity Priority is used for when you want that specific size. So if you paid for a reserved instance of D3 VM and you pick Capacity Priority you are saying I ONLY want that size VM. By choosing to go with Capacity Priority over Size Flexibly we will ensure there is enough of that specific series in the region you reserved for you to deploy that VM in that size.
As mentioned in the doc, when you have a reservation that is limited to single scope (VM Size) you can opt to have the optimization settings set to Capacity rather than flexibility.
Does that help clarify?

Uff it is more and more difficult for me.
When I order reserved capacity with capacity priority. It will be only applied when I order the specific VM?
When I order this I understand that there will be more Vms with this size in this region.  Will this means that when I order for example D4sv3 with capacity priority there will be caunted with one more D4sv3? Will be that connected for my subscription? Or it is only to increase chance that when I order D4sv3 that I will get it?
Thanks

If you pay for a reserved instance and use Capacity Priority we will reserve an instance of the selected size for you in that region. That way if you want to spin up an instance you can be confident that there will be that specific size available for your subscription.
Otherwise, flexibility capacity will ensure you that size or a similar size available but the discount for reserving the capacity will be applied regardless.

The capacity you select should be reserved for you. It is not a specific node per say but we make sure that there is capacity in that size for you to deploy to somewhere in that region.
As per which zone, I am not aware of this being tied to any specific zone in a region. We will simply ensure you have capacity to deploy in one of the selected zones in a region for the VM size you select.
@yashesvi @Drewm3 feel free to add any other details if you feel I missed any information.

Thanks very much. To close this I have last thing on the list. When I create ds2v3 ri for 1 year and ds2v3 for 3years all flex and order server ds4v3 will be this one covered by both first year?

The reserved instance discount will only apply when you select the DS2V3 series. The only time it would apply to a DS4V3 would be if for some reason there was no DS2V3 series available. So simply picking a DS4V3 series will not mean you get the discount because you have a reserved instance for DS2.
When a series is not available and you have a reserved instance running a specific size you will pick that size in the portal and if it is not available on the backend we will automatically find a different size for you and apply the discount. You won't get to pick the different size yourself. It is handled by the backend.

I understand that I don't specify on which Vm the discount will be applied.
It was ment like this.let say In my subscription I don't have anymore Ds2v3 and I have only one Ds4v3.  I assume that system will not find same size therefore it will look for different from the same group. In this case Ds4v3. How the discount wil be applied? For scenario described up?

You won't personally look for any other sizes. You will still select the size you reserved and if it is not available the platform will pick a similar size and apply the discount. At which point you would be charged the discount rate you payed for the original size but applied to the size the platform auto selected for you.

I thought that the flexible allows me change the size. For example when I buy reserved ds4v3.  And deploy ds4v3. After some time I stop ds4v3 and start 2x ds2v3 the system will apply the discount on those two based on factor if there isn't ds4v3

@axatix ah yes sorry I misread that portion. Apologies.
https://docs.microsoft.com/en-us/azure/virtual-machines/windows/reserved-vm-instance-size-flexibility

@axatix thanks for linking the other issue. Lets close this one out and continue on the other one as I will need to check offline with some product group members to confirm. Will reply shortly to that other issue.

@MicahMcKittrick-MSFT Capacity is not reserved. Reserved instances don't provide a capacity assurance or guarantee and doesnt physically reserve the specified capacity in the data center. If a user who has reserved instance with capacity priority experiences a deployment failure due to capacity issues, it causes a high severity issue at our end and we try to provide additional capacity to mitigate by taking steps such as letting users eat into buffers or defragmenting.

#please-close


[AutoDiff] forbid derivative registration using @differentiable

This is the tensorflow branch PR corresponding to https://github.com/apple/swift/pull/30001. Repeating the description:
Forbids the jvp: and vjp: arguments to @differentiable, deletes all the logic that handles them, and updates tests to use @derivative(of:) instead.
I left an error message saying that jvp: and vjp: are deprecated, so that there is a good explanation for anyone who missed the deprecation warning and whose code stops working. I think we should wait until next release to remove this.
I am running all the swift-apis, swift-models, swift-jupyter, swiftai, etc tests on this to catch any leftover jvp: or vjp: that we forgot to remove.

I accidentally deleted tensorflow branch, which closed this PR. That was not intentional, sorry!
It would be nice to protect tensorflow branch against deletion to prevent this from happening again.

Test run caught some 'vjp:' in OpenSpiel! I'll fix that before proceeding with merging this PR.

@swift-ci please test tensorflow

@swift-ci please test tensorflow

@swift-ci please test tensorflow

This seems good to go now, but I'll wait until we have caught up with the master->tensorflow merge so that I don't further pile on conflicts.

@swift-ci please test tensorflow

Merging now that upstream mirror #30001 has been merged.


Stored Procedure with TVP input Returns Error Despite Schema and Type name in the Parameters

Hi,
There seem to be an issue with TVP despite adding the type name and schema names in the parameter list as described in the documentation :https://github.com/mkleehammer/pyodbc/wiki/Working-with-Table-Valued-Parameters-(TVPs)
Here is my code :
def update_permit():
    records_list = ["type_permit", "geoscience"] + df_to_records(select_permits)
    query = f"""
    EXECUTE GEODB.geoscience.usp_update_permit @permit_source_table = ?;
    """
    response = write_query("GEODB", query, (records_list, ))
    return response

And here is a sample of data as it is presented from the "(records_list, )' part of the code:
(['type_permit', 'geoscience', ('378', 'FOUKANDA II'), ('232', 'TCHENDO'), ('112', 'MARINE IV'), ('353', 'PNGF bis'), ('230', 'TCHIBELI-LITANZI'), ('376', 'AWA-PALOUKOU'), ('156', 'LIKOUALA MARINE'), ('379', 'IKALOU / IKALOU SUD'), ('305', 'Marine 28'), ('25', 'YOMBO-MASSEKO-YOUBI'), ('377', 'DJAMBALA II'), ('231', 'TCHIBOUELA'), ('53', 'EMERAUDE'), ('380', 'MWAFI II'), ('306', 'KOMBI-LIKALALA-LIBONDO'), ('320', 'Boatou'), ('386', 'YANGA-SENDJI')],)

Yet, I get this error:
 ('42000', "[42000] [Microsoft][ODBC Driver 17 for SQL Server][SQL Server]Column, parameter, or variable #1: Cannot find data type geoscience.type_permit. (2715) (SQLExecDirectW); [42000] [Microsoft][ODBC Driver 17 for SQL Server][SQL Server]Statement(s) could not be prepared. (8180); [42000] [Microsoft][ODBC Driver 17 for SQL Server][SQL Server]Parameter or variable '@P1' has an invalid data type. (2724)")

Any suggestions please?

Here is the type table definition:
USE GEODB;

GO

DROP TYPE IF EXISTS geoscience.type_permit;

GO

CREATE TYPE geoscience.type_permit AS TABLE(
	id_permit INT PRIMARY KEY,
	permit NVARCHAR(100)
)

And here is the stored procedure:
USE GEODB;

GO

DROP PROCEDURE IF EXISTS geoscience.usp_update_permit;

GO

CREATE OR ALTER PROCEDURE geoscience.usp_update_permit
@permit_source_table geoscience.type_permit READONLY
AS
BEGIN
	SET NOCOUNT ON
	MERGE geoscience.permit AS tgt
	USING @permit_source_table AS src
	ON src.id_permit = tgt.id_permit
	WHEN MATCHED
	THEN
		UPDATE
		SET
			tgt.permit = src.permit
	WHEN NOT MATCHED BY TARGET
	THEN
		INSERT (id_permit, permit)
		VALUES (src.id_permit, src.permit)
	WHEN NOT MATCHED BY SOURCE
	THEN
		DELETE;
END;

Finally, here is my connection and cursor handling script:
from pyodbc import connect, Error
import pandas as pd
import numpy as np

connection_strings = {
	"GEODB" : {
		"SERVER" : <hidden>,
		"DATABASE" : "GEODB",
		"UID" : <hidden>,
		"DSN" : "GEODB",
		"DRIVER" : "ODBC Driver 17 for SQL Server"
    },
}
def get_connection(database):
	driver = connection_strings[database].get("DRIVER", "")
	server = connection_strings[database].get("SERVER", "")
	user = connection_strings[database].get("UID", "")
	pwd = connection_strings[database].get("PWD", "")
	dsn = connection_strings[database].get("DSN", "")

	try:
		connection = connect(f"DRIVER={driver}; SERVER={server}; database = {database}; DSN={dsn}; Trusted_Connection=Yes")
		if pwd:
			connection = connect(f"DRIVER={driver}; SERVER={server}; database = {database}; UID={user}; PWD = {pwd}; DSN={dsn}; Trusted_Connection=Yes")
		return connection
	except Error as err:
		return err

def handle_query_connections(query_executing_function):
	def wrapper(database, query_statement, insert_records = None):
		connection = None #Initialize connection to none to be able to close it in case of failure in the try and except block.
		try:
			connection = get_connection(database)
			cursor = connection.cursor()
			#cursor.fast_executemany = True

			query_result = query_executing_function(database, cursor, query_statement, insert_records)

			cursor.close()
			connection.commit()
			connection.close()
			return query_result
		except Error as e:
			if connection:
				connection.close() #Close connection if not closed in the try block after an error.
			return e
	return wrapper

@handle_query_connections
def write_query(database, cursor, query_statement, insert_records):
	try:
		if insert_records is None:
			cursor.execute(query_statement)
		else:
			cursor.execute(query_statement, insert_records)
		return "Success"
	except Error as err:
		return str(err)

EXECUTE GEODB.geoscience.usp_update_permit

What is the current database when that statement is executed? UDTs cannot be used across databases.

EXECUTE GEODB.geoscience.usp_update_permit

What is the current database when that statement is executed? UDTs cannot be used across databases.

"GEODB". That's also specified in the connection string.

As a proof that the problem is not comming from the database, when I run the following from Python, it works as expected:
def update_permit():
    query = f"""
    USE GEODB;
    DECLARE @table_example AS geoscience.type_permit;
    INSERT INTO @table_example VALUES (10, 'Test3'), (11, 'Test4');
    EXECUTE geoscience.usp_update_permit @table_example;
    """
    response = write_query("GEODB", query)
    return response
    ```

Have you tried without specifying the type of the TVP parameter?

Have you tried without specifying the type of the TVP parameter?

Yes,
I tried all the following:
def update_permit():
    records_list = df_to_records(select_permits)
    query = f"""
    EXECUTE GEODB.geoscience.usp_update_permit @permit_source_table = ?;
    """
    response = write_query("GEODB", query, (records_list, ))
    return response

Returns same error as with the addition of type and schema to the list.
def update_permit():
    records_list = ["type_permit", "geoscience", "GEODB"] + df_to_records(select_permits)
    query = f"""
    EXECUTE GEODB.geoscience.usp_update_permit @permit_source_table = ?;
    """
    response = write_query("GEODB", query, (records_list, ))
    return response

Does not work either.
def update_permit():
    records_list = ["GEODB.geoscience.type_permit"] + df_to_records(select_permits)
    query = f"""
    EXECUTE GEODB.geoscience.usp_update_permit @permit_source_table = ?;
    """
    response = write_query("GEODB", query, (records_list, ))
    return response

Does noot work.
I also tried using only the type or only the schema in the brackets of the records_list but not of these worked.
For now I resolved to manually creating a (temporary) table, populating it and using it as the input for my merge stored_procedure. (Found no other efficient method and I absolutely need to use the merge function of sql for my database updates).
If that could help anyone, below is my code.

Here is my current solution, while waiting for a better solution from pyodbc.
def drop_temporary_permit():
    query = f"""
    USE GEODB;
    BEGIN
    DROP TABLE IF EXISTS geoscience.temp_permit;
    END;
    """
    response = write_query("GEODB", query)
    return response

def create_temporary_permit():
    query = f"""
    USE GEODB;
    BEGIN
    CREATE TABLE geoscience.temp_permit(
	    id_permit INT PRIMARY KEY,
	    permit NVARCHAR(100)
    );
    END;
    """
    response = write_query("GEODB", query)
    return response

def insert_into_temporary_permit():
    records_list = df_to_records(select_permits)
    query = f"""
    USE GEODB;
    BEGIN
    INSERT INTO GEODB.geoscience.temp_permit(id_permit, permit) VALUES (?, ?);
    END;
    """
    response = write_query("GEODB", query, records_list)
    return response

def merge_permit():
    query = f"""
    USE GEODB;
    BEGIN
    DECLARE @source_permit AS geoscience.type_permit;
    INSERT INTO @source_permit SELECT * FROM geoscience.temp_permit;
    EXECUTE geoscience.usp_update_permit @source_permit;
    END
    """
    response = write_query("GEODB", query)
    return response

def update_permit():
    try:
        drop_temporary_permit()
        create_temporary_permit()
        insert_into_temporary_permit()
        merge_permit()
        drop_temporary_permit()
    except Exception as e:
        return str(e)

    return "Success"

That works as expected but requires more code and effort than with pyodbc's TVP.

I am unable to reproduce your issue. This code works for me:
import pyodbc

cnxn = pyodbc.connect(
    "Driver=ODBC Driver 17 for SQL Server;"
    "Server=<IP_ADDRESS>;"
    "UID=scott;PWD=tiger^5HHH;"
    "Database=GEODB;"
)

tvp_rows = [(1, "permit 1"), (2, "permit 2")]

sql = "EXEC geoscience.usp_update_permit ?"
params = ((["type_permit", "geoscience"] + tvp_rows),)
print(params)
# (['type_permit', 'geoscience', (1, 'permit 1'), (2, 'permit 2')],)

crsr = cnxn.cursor()
crsr.execute(sql, params)
cnxn.commit()

I am unable to reproduce your issue. This code works for me:
import pyodbc

cnxn = pyodbc.connect(
    "Driver=ODBC Driver 17 for SQL Server;"
    "Server=<IP_ADDRESS>;"
    "UID=scott;PWD=tiger^5HHH;"
    "Database=GEODB;"
)

tvp_rows = [(1, "permit 1"), (2, "permit 2")]

sql = "EXEC geoscience.usp_update_permit ?"
params = ((["type_permit", "geoscience"] + tvp_rows),)
print(params)
# (['type_permit', 'geoscience', (1, 'permit 1'), (2, 'permit 2')],)

crsr = cnxn.cursor()
crsr.execute(sql, params)
cnxn.commit()

Thank you @gordthompson , turns out the issue was with my connection strings. Here is the corrected strings:
connection_strings = {
	"GEODB" : "DRIVER={ODBC Driver 17 for SQL Server};SERVER=<hidden on purpose>;DATABASE=GEODB;UID=nmbola;Trusted_Connection=Yes",
	"PDS_PRD" : "DRIVER={ODBC Driver 17 for SQL Server};SERVER=<hidden on purpose>;DATABASE=Some_Other_Database;UID=nmbola;Trusted_Connection=Yes",
	"PEDB_HQ" : "DRIVER={ODBC Driver 17 for SQL Server};SERVER=<hidden on purpose>;DATABASE=Some_Other_Database;UID=nmbola;Trusted_Connection=Yes"
    }

def get_connection(database):
	try:
		connection = connect(connection_strings[database])
		return connection
	except Error as err:
		return err


Save db project information when saved in scmp file

This PR fixes #19228.
After the changes that went in for publishing changes to database projects from remote database, a small functionality was missed where schema compare file when opened didn't understand how to read and use database projects information stored in the scmp file. This PR fixes that scenario.
While working on this, this issue was found: https://github.com/microsoft/azuredatastudio/issues/20332 . For better testing: https://github.com/microsoft/azuredatastudio/issues/20334

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 2 (0.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.002%) to 41.654%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

extensions/schema-compare/src/schemaCompareMainWindow.ts
0
2
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
-0.002%

Covered Lines:
27854

Relevant Lines:
62295

💛  - Coveralls


Fix GetSnapshotBlockHeightPeriod

#1282  👈
feature/proof-of-stake

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @lazynina and the rest of your teammates on  Graphite


[DOC] data container conventions, conversions, and data sets documentation

Pull request on data container conventions and related functionality.
This is based on, and branching off the state of #1225 - and requests of @mloning and @TonyBagnall to document this.
The increment in this PR is the notebook examples/00_datatypes.ipynb which is supposed to explain:

the different core in-memory data container datatypes for series and panels used in sktime, with examples to play around with
validity checking, conversion, register, and type functionality
data sets loading functionality

Ready for review

Ready for review now.

I dont think this will be final version of notebook, but it is good for the purposes of this PR.

Well, better this notebook than no notebook whatsoever, no?
This should help our users who were previously struggling with the panel data types because the expected format was nowhere properly recorded...

Well, better this notebook than no notebook whatsoever, no?

Yup, definitely! Not only good for end-users, but developers too.

This should help our users who were previously struggling with the panel data types because the expected format was nowhere properly recorded...

What's even more funny, after I did all due diligence here, there are seven panel data (m)types in use throughout the package...

Not having it as the 00 notebook. The notebooks in which we actually do interesting stuff, like forecasting and classification, should come first, and this notebook should be later.

Hm, how would you organise it then? An "appendix notebook" where numbering starts with A, B, C, etc, and which is linked from the "interesting stuff" notebooks?

Adding contents at the top, for the sections, and subsections, and subsubsections. I presume there is a way to do this automatically.

Sadly, I think, no 😢
There is a semi-automatic way, which is putting it out as an issue waiting for helpful contributors such as @bilal-196 in #1200 to pick it up (thanks again!) 😄 while I work on creating notebook content, sort of a division of labour...
Would be great if this could be somehow automated though. Perhaps we could put that out as an issue?

Reorganising the notebook and/or making the content on types more accessible. Currently, the first section on types is dense and abstract, and I imagine difficult to follow for most users. There are various ideas I have for this (e.g. giving example of each machine type first, before the full specifications), but can be discussed in separate pr.

Looking forward to that 😄 feel free to pick it up once this is in
My main motivation for doing this was sorting this out so I can work on the transformers, I think it's a pre-condition to have the panel types, checks, and conversions sorted to do the refactor cleanly

Hm, how would you organise it then? An "appendix notebook" where numbering starts with A, B, C, etc, and which is linked from the "interesting stuff" notebooks?

I think a short-term option is to renumber it from 00 to 05. Better options depend a lot on what we consider to be the 'ideal end state' of the documentation, which is unknown to me. This stuff on data conversion feels more appropriate for a 'how-to' section of documentation rather than 'tutorial' section.

Better options depend a lot on what we consider to be the 'ideal end state' of the documentation, which is unknown to me. This stuff on data conversion feels more appropriate for a 'how-to' section of documentation rather than 'tutorial' section.

How about this for a practical solution: we have "tutorials" with interesting stuff, numbered 1, 2, etc.
Then, we have "appendix" type tutorials, into more dense/technical parts of the toolbox and its infrastructure, numbered A, B, etc. They would be linked from the tutorials, e.g., "for loading your own data, consult notebook A on data types and data validation".

How about this for a practical solution: we have "tutorials" with interesting stuff, numbered 1, 2, etc.
Then, we have "appendix" type tutorials, into more dense/technical parts of the toolbox and its infrastructure, numbered A, B, etc. They would be linked from the tutorials, e.g., "for loading your own data, consult notebook A on data types and data validation".

Certainly sounds like a sensible option. But I hesitate to say anything definitive because the decision of how to organise the notebook examples folder depends on the whole vision of the documentation for the package (https://github.com/alan-turing-institute/sktime/issues/1059). E.g. maybe the better solution is two have two separate folders for notebooks; one for beginner/intro tutorials and the other for advanced/how-to tutorials. Or something else entirely.

Certainly sounds like a sensible option. But I hesitate to say anything definitive because the decision of how to organise the notebook examples folder depends on the whole vision of the documentation for the package (#1059). E.g. maybe the better solution is to have two separate folders for notebooks; one for beginner/intro tutorials and the other for advanced/how-to tutorials. Or something else entirely.

My advice would be, let's solve one problem at a time - let's solve the small problem and avoid getting stalled by trying to open up a scope which is far beyond this PR.
I don't think a complete vision for documentation is stated currently; it's also easier to develop if we have "all the parts" ready. Next week's doc sprint will be a good time to creat such a vision, but I would not make this Pr dependent on this - I feel there will be multiple iterations of discussions for the duration of a month or so, just an educated guess.

but I would not make this Pr dependent on this

i agree. my original comment did say the notebook was good for this pr and my suggestions were for future iterations of the notbeook.

i agree. my original comment did say the notebook was good for this pr and my suggestions were for future iterations of the notbeook.

Yes, let's revisit the "target state" of documentation during the doc sprint!

so, regarding the mtype function: I removed the old one and replaced it by the new one in the module export.
Since everything was referring to the module export rather than the precise location, nothing broke - so no deprecation is actually needed.

?? why is readtheocs failing now?

is there anything in particular you want me to check (or not check) so that we can get this PR reviewed more quickly?

I would say, focus on the general logic/framework as opposed to the individual checks or converters - many of those were moved into the package rather than implemented from scratch.

@Lovkush-A, are you still reviewing this?

@Lovkush-A, are you still reviewing this?

I intend to finish reviewing this weekend, i.e. later today.


Distance based camera triggering

Adding a third operation mode to the camera_trigger driver, triggering after covering a certain distance.
The function reads out local position and, if valid, checks if the travelled horizontal distance exceeds the given threshold parameter.
The patch has been bench-tested and will be flight-tested in the next days.

Fantastic, thanks! Looking forward to the testing results of this.

Thanks @LorenzMeier for the review. Would logging functionality also be desired at this point (to enable postprocessing of the triggered shots)?

That would be nice

Let us know when this is rebased on master and outdoor tested. Should be easy enough to do a handheld test just on the street.

I will put a patch on master today which enables the VEHICLE_CMD_DO_SET_CAM_TRIGG_DIST command (and also unlocks it in QGC). That should then be used to configure it.

Up and working. Handheld test logs available here: http://logs.uaventure.com/view/MpZykD2yVB9hKdHmpE7BHH
Also checked the signal after rebasing to make sure it stays low on startup.

I noted though that during uploading of new firmware the signal goes to high.

Added the support to enable / disable the triggering in the distance mode via command.
This is not tested yet.

@birchera Could you finish the testing? QGC survey support is close:
https://github.com/mavlink/qgroundcontrol/pull/2892

@LorenzMeier Benchtested mode 4 with enabling and disabling the triggering from the uorb command, and found to work reliably.

Rebased and applied.


Syntax error in program "ZCL_ABAPLINT_BACKEND

Error analysis
The following syntax error has occurred in program
ZCL_ABAPLINT_BACKEND==========CP        :
Type "TY_SERIALIZATION" is unknown.
" "

yea, it broke with one of the recent abapGit changes

@preinboth try again, https://github.com/abaplint/abaplint-sci-client/pull/286 merged

@larshp  thx


make the function controller owns serving an builder

Signed-off-by: wanjunlei<EMAIL_ADDRESS>Currently, the function controller does not watch the builder and serving, so the function controller cannot recreate them when the builder or serving is deleted.
This pr make the function controller owns the builder and serving, thereby solving the following problems:

Recreate the builder when the builder is deleted and the build is not completed. Rebuild can be achieved by deleting the builder when the build fails.
Recreate the serving after serving is modified or deleted.
If the parameters of the function are modified, the rebuild will be triggered automatically, and the serving will be re-run after the build is completed. It is easy to trigger rebuild when the function code is modified by modifying the parameter of the function, such as image version.
If the runtime or parameters of serving are modified, serving will automatically re-run.

Incidentally, a parameter is added to set the log level of the operator

@tpiperatgod pls take review this


extreme FPS loss on new network

when i try to create a new network i get a major drop in frame rates and minecraft becomes completely unresponsive to the point of having to force it to close but i get no crash report from it. im running a custom mod pack only adding one mod at a time playing for a few days to see if there are any issues and this is the last mod i put on and it played fine for about an hour untell i tried to make a new network. once i got it to shut down and restart it was fine with the new network added before the "crash" but i tryed again and again same thing happened. i attached a list of the mods i am running on Minecraft version 1.12.1. i really like this mod and dont want to have to remove it any help would be appreciated thank you

so i have been trying to get this to work and i have went and removed all mods except flux networks and sonarcore. and a fresh new world and i get the same results when i make a second network i cant open any of the capacitors or plugs it just does the animation like im hitting them. though with the fresh world i dont get the flame rate drop. and again no crash

can confirm it. if u create a new Flux network the game will lag out. you have to exit & save and rejoin ur game to be able to paly again.
Versions:
Minecraft 1.12.1 forge-<IP_ADDRESS>5
FluxNetworks-1.12-<IP_ADDRESS>

I confirm that too. It's really annoying.

Fixed in dev


GH files: readme/contributing/licence/etc.

README should contain

How to install

Including manually?

What it looks like

Screenshots

How to use

Click a button
Do not trust the file until you've verified the signature

Licence

I want it to be free, both as in freedom and price. Freedom: people might want to personalise it. But also, freedom...; price: security is important. Putting money into things is annoying and not useful.

Contributing

Search for issues

Code of conduct
Issue template
No pull request template

Good enough


SSH Update not working (for me)

I thought I'd update my YOURLS install and tried to update it using the commands on this page:
https://github.com/YOURLS/YOURLS/wiki/Install-From-Git
I SSH'd into my server and issued this command:
"bash git pull https://github.com/YOURLS/YOURLS.git master"
and without the "bash" too, like this:
"git pull https://github.com/YOURLS/YOURLS.git master"
but received this error message:
"fatal: Not a git repository (or any of the parent directories): .git"
Am I doing something wrong?
Please be nice - it's my first post on GitHub!
Running Yourls ver 1.7.1

If you didn't install from Git, then your install folder isn't a git repository.
I haven't tested this, but you should be able to git init an empty repo in your YOURLs dir, then git remote add origin https://github.com/YOURLS/YOURLS.git && git pull origin master
Again, not tested—so you might have to use git fetch instead of git pull, or git checkout master after pulling, or something else I've not thought of depending on how git handles you turning an existing directory into a repo.

Wow - that's a fullsome reply - thanks!
Firstly, I did a manual install a few years ago, into a subdirectory on my server, so, yes, my "install folder" is not a git repository. I do not know the difference.
I think you are saying that I could create a new directory (you call a repo - right?) inside the same folder where it currently exists, and then somehow (with those commands) convert the existing directory into a "repo" instead - right again?

A git "repo" is all contained in a subfolder called .git. The files you see are the "working copy", which  you can play with and edit and then have git compare the differences to what's stored in the repo and apply the changes in a commit. (That's a very basic explanation, but it hits the basics.)
So if you git init inside the existing YOURLS directory you're running from, Git will create the .git folder and set it up. Then when you add the remote URL and tell it to git pull, it populates your new (empty, local) repo with the data from the remote repo.

OK - thanks again - I am slowly understandning this.
Playing devils' advocate here - is it not easier to do a manual install/update over the top of the existing install? What are the differences of that versus creating a .git subfolder called ".git" and then me updating the .git subfolder and me updating that subfolder with a "git pull"?

It very likely is simplest to just download the current ZIP and extract it over the existing files (ideally after making a backup copy of your existing install folder in case of problems). Unless you want to run pre-release code, bothering with the Git repo is probably not worth it.

Thanks, dgw - I was coming to that conclusion. I think I'll follow your suggestion.

(For the record, there was a couple formatting errors in the doc, like the "bash git", there's no "bash" supposed to be typed in)


vPreProcess camera calibration file

is currently hard-coded but should be exposed as parameters to the module

@lunagava - the code in branch icub23-code is super old and we won't be able to merge any changes in to master. Therefore please remove that local file and only use the up-to-date version in master.
you mentioned one change that was useful would be to be able to  define which calibration file you use to do undistortion. you could add this functionality to the existing code in master and make a pull request if you would find it useful. Let me know if you need help to do so.

#136


Implementation of nullable:: does not work correctly

The Ion Schema Grammar allows for nullable:: to be applied to any type reference.
<TYPE_REFERENCE> ::=           <TYPE_NAME>
                   | nullable::<TYPE_NAME>
                   |           <TYPE_ALIAS>
                   | nullable::<TYPE_ALIAS>
                   |           <TYPE_DEFINITION>
                   | nullable::<TYPE_DEFINITION>
                   |           <IMPORT_TYPE>
                   | nullable::<IMPORT_TYPE>

And the example in the Core Types section of the spec says:

The core types do not include any of Ion's null.* values, but each of the types may have a weakly- or strongly-typed null value if the type name is annotated with nullable. When a strongly-typed null value is encountered, its type must agree with one of the core types of the expected type. For example, if a any_of: nullable::[int, string, struct] is expected, 5, "hi", {}, null, null.null, null.int, null.string, and null.struct are all valid values, but null.decimal is not.

Even ignoring the fact that the example does not match the grammar (nullable:: should be on a type reference, not a list of type references), the ion-schema-kotlin implementation does not do what this says. I also tried several other type definitions that are similar to the example, and the Ion Schema System consistently failed to allow the null values or ran into a StackOverflowError, unless I specifically used an ion type instead of a core type.
Given the test cases mentioned in the quoted paragraph, here are the results for the type in the example, and several similar types.
valid::[ 5, "hi", {}, null, null.null, null.int, null.string, null.struct ]
invalid::[ null.decimal ]

Type
Result

type::{any_of:nullable::[int, string, struct]}
test fails for null, null.null, null.struct, null.int, null.string

type::{type:nullable::{any_of:[int, string, struct]}}
stack overflow error for null.decimal, null.struct, null.int, null.string

type::{any_of:[nullable::int, nullable::string, nullable::struct]}
test fails for null, null.null, null.struct, null.int, null.string

type::{type:nullable::any, any_of:[nullable::int, nullable::string, nullable::struct]}
stack overflow error for null.decimal, null.struct, null.int, null.string

type::{type:nullable::any}
stack overflow error for null.decimal, null.struct, null.int, null.string

type::{type:$any, any_of:[nullable::int, nullable::string, nullable::struct]}
All test cases pass

Also created https://github.com/amzn/ion-schema/issues/38 to address the ambiguity in the spec around nullable::.


DAP requires variable name to be a string

I was getting E5108: Error executing lua .../nvim/site/pack/packer/start/nvim-dap/lua/dap/entity.lua:36: attempt to get length of field 'name' (a  number value) when debug with OSV

That's a suprise lua variables names could also be number! Anyway, great PR merging.

Array indexes

OK, makes sense. I was using dap hover variables which worked just fine, but accessing variables through the REPL fails.


Problems using GCC_INSTRUMENT_PROGRAM_FLOW_ARCS and Firebase/Core

[READ] Step 1: Are you in the right place?
Yes

For issues or feature requests related to the code in this repository
file a Github issue.

[REQUIRED] Step 2: Describe your environment

Xcode version: 10.1
Firebase Component: Core
Component version: 5.0

[REQUIRED] Step 3: Describe the problem
Firebase/Core (since 5.0) blocks the execution of tests with GCC Instruments Program Flow Arcs enabled.
This can be seem by passing the instruction "GCC_INSTRUMENT_PROGRAM_FLOW_ARCS=YES" when building your app for testing, that output the error:
ld: symbol(s) not found for architecture x86_64
clang: error: linker command failed with exit code 1 (use -v to see invocation)

The lack of this flag can have bad impacts on the generation of code coverage data, and limits the checks that can be done on code.
Steps to reproduce:

Create a project (any project) that has Firebase, and tests enabled.
Run the tests from terminal passing GCC_INSTRUMENT_PROGRAM_FLOW_ARCS=YES as parameter.

Relevant Code:
Command:
set -o pipefail && env "NSUnbufferedIO=YES" xcodebuild "-workspace" "PROJECT.xcworkspace" "-scheme" "PROJECT" "build" "test" "-destination" "platform=iOS Simulator,name=iPhone 6s Plus,OS=latest" "GCC_INSTRUMENT_PROGRAM_FLOW_ARCS=YES" "GCC_GENERATE_TEST_COVERAGE_FILES=YES"

Error:
Undefined symbols for architecture x86_64:
  "_llvm_gcov_init", referenced from:
      ___llvm_gcov_init in FirebaseCore(FIRApp.o)
      ___llvm_gcov_init.609 in FirebaseCore(FIRApp.o)
      ___llvm_gcov_init.674 in FirebaseCore(FIRApp.o)
      ___llvm_gcov_init.739 in FirebaseCore(FIRApp.o)
      ___llvm_gcov_init.804 in FirebaseCore(FIRApp.o)
      ___llvm_gcov_init.869 in FirebaseCore(FIRApp.o)
      ___llvm_gcov_init in FirebaseCore(FIRComponentType.o)
      ...
  "_llvm_gcda_end_file", referenced from:
      ___llvm_gcov_writeout in FirebaseCore(FIRApp.o)
      ___llvm_gcov_writeout in FirebaseCore(FIRComponentType.o)
      ___llvm_gcov_writeout in FirebaseCore(FIRConfiguration.o)
      ___llvm_gcov_writeout in FirebaseCore(FIRDependency.o)
      ___llvm_gcov_writeout in FirebaseCore(FIRAnalyticsConfiguration.o)
      ___llvm_gcov_writeout in FirebaseCore(FIRBundleUtil.o)
      ___llvm_gcov_writeout in FirebaseCore(FIROptions.o)
      ...
  "_llvm_gcda_emit_function", referenced from:
      ___llvm_gcov_writeout in FirebaseCore(FIRApp.o)
      ___llvm_gcov_writeout in FirebaseCore(FIRComponentType.o)
      ___llvm_gcov_writeout in FirebaseCore(FIRConfiguration.o)
      ___llvm_gcov_writeout in FirebaseCore(FIRDependency.o)
      ___llvm_gcov_writeout in FirebaseCore(FIRAnalyticsConfiguration.o)
      ___llvm_gcov_writeout in FirebaseCore(FIRBundleUtil.o)
      ___llvm_gcov_writeout in FirebaseCore(FIROptions.o)
      ...
  "_llvm_gcda_emit_arcs", referenced from:
      ___llvm_gcov_writeout in FirebaseCore(FIRApp.o)
      ___llvm_gcov_writeout in FirebaseCore(FIRComponentType.o)
      ___llvm_gcov_writeout in FirebaseCore(FIRConfiguration.o)
      ___llvm_gcov_writeout in FirebaseCore(FIRDependency.o)
      ___llvm_gcov_writeout in FirebaseCore(FIRAnalyticsConfiguration.o)
      ___llvm_gcov_writeout in FirebaseCore(FIRBundleUtil.o)
      ___llvm_gcov_writeout in FirebaseCore(FIROptions.o)
      ...
  "_llvm_gcda_start_file", referenced from:
      ___llvm_gcov_writeout in FirebaseCore(FIRApp.o)
      ___llvm_gcov_writeout in FirebaseCore(FIRComponentType.o)
      ___llvm_gcov_writeout in FirebaseCore(FIRConfiguration.o)
      ___llvm_gcov_writeout in FirebaseCore(FIRDependency.o)
      ___llvm_gcov_writeout in FirebaseCore(FIRAnalyticsConfiguration.o)
      ___llvm_gcov_writeout in FirebaseCore(FIRBundleUtil.o)
      ___llvm_gcov_writeout in FirebaseCore(FIROptions.o)
      ...
  "_llvm_gcda_summary_info", referenced from:
      ___llvm_gcov_writeout in FirebaseCore(FIRApp.o)
      ___llvm_gcov_writeout in FirebaseCore(FIRComponentType.o)
      ___llvm_gcov_writeout in FirebaseCore(FIRConfiguration.o)
      ___llvm_gcov_writeout in FirebaseCore(FIRDependency.o)
      ___llvm_gcov_writeout in FirebaseCore(FIRAnalyticsConfiguration.o)
      ___llvm_gcov_writeout in FirebaseCore(FIRBundleUtil.o)
      ___llvm_gcov_writeout in FirebaseCore(FIROptions.o)
      ...
ld: symbol(s) not found for architecture x86_64
clang: error: linker command failed with exit code 1 (use -v to see invocation)

I'm not able to reproduce in a Firebase quickstart example.
The following command runs successfully:
set -o pipefail && env "NSUnbufferedIO=YES"  xcodebuild -workspace AnalyticsExample.xcworkspace/ -scheme AnalyticsExample build test -destination "platform=iOS Simulator,name=iPhone 6s Plus,OS=latest" "GCC_INSTRUMENT_PROGRAM_FLOW_ARCS=YES" "GCC_GENERATE_TEST_COVERAGE_FILES=YES"
Please provide more details about how to reproduce or send a reproducible example.

Having the same problem here. I also created a minimal project to reproduce this issue:
https://github.com/el-hoshino/FirebaseCocoaPodsTest
My Xcode version is 10.2, and I've tried CocoaPods with version 1.6.1 and 1.7.0.beta.3

@el-hoshino Thanks for the reproducible project!
I'm still not sure what triggers this issue in some projects but not others, but the workaround of adding -fprofile-arcs to Build Settings > Other Linker Flags gets it to build and test successfully.  Thanks to https://stackoverflow.com/a/39579344/556617

@paulb777 Thanks for the information! I'm trying this workaround and will report my result later!
One thing I found is that it seems projects created with Xcode 10.2 will have this problem, but older projects don't 🤔

@paulb777 Confirmed! Passing -fprofile-arcs to OTHER_LDFLAGS works!
And I guess this command should work for you @fwigorcferreira :
set -o pipefail && env "NSUnbufferedIO=YES" xcodebuild "-workspace" "PROJECT.xcworkspace" "-scheme" "PROJECT" "build" "test" "-destination" "platform=iOS Simulator,name=iPhone 6s Plus,OS=latest" "GCC_INSTRUMENT_PROGRAM_FLOW_ARCS=YES" "GCC_GENERATE_TEST_COVERAGE_FILES=YES" "OTHER_LDFLAGS=\$(OTHER_LDFLAGS) -fprofile-arcs"


Use the Dummy audio driver in the project manager

This prevents Godot from appearing in the list of applications outputting sound in the OS while the user is in the project manager.
This also partially addresses #38154.

I'm not sure it's worth hacking it this way. If Godot uses too much CPU due to the coreaudio driver, working it around only in the project manager doesn't help much as 99% of user time is spent in the editor itself.
And if we ever decide to add sound features to the project manager (e.g. for accessibility), it might lead to some wasted hours trying to figure out why it's not outputting any audio.

And if we ever decide to add sound features to the project manager (e.g. for accessibility), it might lead to some wasted hours trying to figure out why it's not outputting any audio.

If we add screen reader support, I think the application emitting sound would be the screen reader, not Godot.

If we add screen reader support, I think the application emitting sound would be the screen reader, not Godot.

That depends on how it's implemented. I've done a bit of work to help out the developer of this screen reader editor plugin, and while I don't think it works in the project manager, using the dummy audio driver would completely prevent any future in-process accessibility tools from working

Rebased and tested again, it works successfully.

That depends on how it's implemented. I've done a bit of work to help out the developer of this screen reader editor plugin, and while I don't think it works in the project manager, using the dummy audio driver would completely prevent any future in-process accessibility tools from working

The project manager doesn't support plugins yet, so I think this would be fine to merge for now.

This could potentially be superceded by a 4.x version of #63458 . The ability to mute until audio output in the PR means that it would also work with screen reading software.

Discussed in the PR review meeting. Reduz suggests that more research is needed to solve https://github.com/godotengine/godot/issues/38154. https://github.com/godotengine/godot/pull/63458 may be an appropriate solution, but even it may be overkill.

Rebased and tested again, it works as expected.
Note that TTS audio output is not emitted by Godot itself (but by the OS instead), so this doesn't affect potential screen-reader support in the future.

Thanks!


opining tsx file "Autostart for angularls failed: matching root directory not detected."

Steps to reproduce

install lvim
rm -rf ~/.config/nvim rm -rf ~/.cache/nvim rm -rf ~/.local/share/nvim git clone https://github.com/lvim-tech/lvim.git ~/.config/nvim
install expo (to generate .tsx)
npm install --global expo-cli
create app with expo
expo init my-app cd my-app
open tsx file
nvim App.tsx
error:

Autostart for angularls failed: matching root directory not detected.
Autostart for angularls failed: matching root directory not detected.
Press ENTER or type command to continue

per https://github.com/neovim/nvim-lspconfig/issues/1359 , I tried sudo npm install -g typescript without any result

Fixed in v1.1.1


QueryBuilder missing AppendLine( FormattableString )

Could be by design and that is fine, but in DapperQueryBuilder, I could do:
qb.AppendLine( fileKey != null
	? (FormattableString)$" AND f.UID = {fileKey}"
	: $" AND fo.Name = {folder:varchar(200)} AND f.Name = {fileName:varchar(200)}"
);

However, I get Argument 1: cannot convert from 'System.FormattableString' to 'InterpolatedSql.IInterpolatedSql' in InterpolatedSql.  I've found a few workarounds so maybe that is intention.  Let me know if method was missed or if I one of the workarounds below is the desired pattern going forward:
Workaround 1 - no Ternary operator
if ( fileKey != null )
{
	qb.AppendLine( $" AND f.UID = {fileKey}" );
}
else
{
	qb.AppendLine( $" AND fo.Name = {folder:varchar(200)} AND f.Name = {fileName:varchar(200)}" )
}

Workaround 2 - Found this method I've never used before
qb.AppendFormattableString( fileKey != null
	? (FormattableString)$" AND f.UID = {fileKey}"
	: $" AND fo.Name = {folder:varchar(200)} AND f.Name = {fileName:varchar(200)}"
);

Workaround 3 - Appending via += seems to compile
qb += fileKey != null
	? (FormattableString)$" AND f.UID = {fileKey}"
	: $" AND fo.Name = {folder:varchar(200)} AND f.Name = {fileName:varchar(200)}";

I'm going to test with number 3, but please advise.  Thanks.

How did it go?
I think both 2 and 3 are good options.
Previously methods were just named "Append/AppendLine" (and would always take a FormattableString) but since I've introduced InterpolatedStringHandlers I had some issues with ambiguous overloads, and therefore in some places I had to use this explicitly-named AppendFormattableString (which does NOT add a leading linebreak, so you might want to start your strings with \n).

@terryaney  Did it work? Can we close this?

Pulled away, but actually was just working on finishing all my integration tests.  You can close this and I'll reopen if the issue presents itself?

@terryaney Did it work? Can we close this?

Yes it worked :)


docs(flask): change redirect URI and update README

Points Flask README to the sgID docs for single source of truth
Changes redirect URI in Flask app to be /api/redirect instead of /api/callback

@kwajiehao sure, done in cd8600b215f2f155be0113102e1d147712b2f515


No getService method in google-cloud-storage

README.MD for google-cloud-storage refers to
Storage storage = StorageOptions.getDefaultInstance().getService();

and a few other times. It appears this is no longer accurate at 0.10.alpha.
I.e. the StorageOptions class does not have a getService method. I'm not sure what it should be replaced with.

getService() is present on ServiceOptions which StorageOptions extends. It was never changed. The code you pasted here should work fine.

OK, I must be doing something wrong. I'll dig into this further.

Multiple markers at this line
	- The method getService() is undefined for the type StorageOptions
	- The type com.google.cloud.HttpServiceOptions cannot be resolved. It is indirectly referenced
	 from required .class files

Digging into my jar I do see the getService method. I don't see com.google.cloud.HttpServiceOptions in google-cloud-core-0.10.0-alpha.jar. What jar should that be in?

I think you are out of sync somehow. If you build both -core and -storage fresh, or make sure that they are both at 0.10.0, it should all work. It looks like you might have the old version of -storage pointing to the new version of -core. (HttpServiceOptions was taken out of the hierarchy and is now a standalone class, HttpTransportOptions)

This appears to be the problem. Thanks. Closing.


Webpack production mode cause ReferenceError

Do you want to request a feature or report a bug?
Bug
What is the current behavior?
mode production, code was formatted, causing some variable be used before be declared(const).
This will causing ReferenceError in runtime.
If the current behavior is a bug, please provide the steps to reproduce.
I simplify code and put it here https://github.com/hjin-me/webpack-uglify-bug
clone it && npm i && npm run build
the result code formatted in /dist/main.js

What is the expected behavior?
should not ReferenceError.
Node: v9.11.1
Webpack: ^4.6.0
webpack-cli: ^2.0.15

update test case make sure code will ReferenceError

this helped me in codemirror I think same will be there - https://github.com/graphql/graphiql/issues/665#issuecomment-381919771


Search which yields 0 results.. throws error

Example:
from gdeltdoc import GdeltDoc, Filters
from gdeltdoc import Filters, near, repeat
f = Filters(
start_date = "2020-05-01",
end_date = "2020-05-02",
num_records = 250,
keyword = "George Floyd",
domain = "theguardian.com",
country = "US"
)
gd = GdeltDoc()
Search for articles matching the filters
articles = gd.article_search(f)
Get a timeline of the number of articles matching the filters
timeline = gd.timeline_search("timelinevol", f)
############################# Error Message ###################################
KeyError                                  Traceback (most recent call last)
 in
16
17 # Search for articles matching the filters
---> 18 articles = gd.article_search(f)
19
20 # Get a timeline of the number of articles matching the filters
c:\python\python37\lib\site-packages\gdeltdoc\api_client.py in article_search(self, filters)
60         articles = self._query("artlist", filters.query_string)
61
---> 62         return pd.DataFrame(articles["articles"])
63
64
KeyError: 'articles'

This got fixed in ea03c6b - thanks for raising the issue and @FelixKleineBoesing for the fix


Something in the snippet swaps ln  for \ln and it isn't in the snippet

Discussed in https://github.com/artisticat1/obsidian-latex-suite/discussions/155

Originally posted by Eneswar May 31, 2023
Hey, something in the snippert turns "ln" into "\ln" and I have searched, manually looked for every single line but I have been unable to find it, can anyone help?

Answered here: #155


fix: add custom unmarshaller for struct with embedded custom interface type

Reason for Change:

Issue Fixed:

Requirements:

[ ] uses conventional commit messages

[ ] includes documentation
[ ] adds unit tests

Notes:

merged with https://github.com/Azure/azure-container-networking/pull/916


feat: adding hermes to e2e tests.

Description

closes: #3315
Commit Message / Changelog Entry
feat: integrating hermes in our e2e tests

see the guidelines for commit messages. (view raw markdown for examples)

Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.

[ ] Targeted PR against correct branch (see CONTRIBUTING.md).
[ ] Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
[ ] Code follows the module structure standards and Go style guide.
[ ] Wrote unit and integration tests.
[ ] Updated relevant documentation (docs/) or specification (x/<module>/spec/).
[ ] Added relevant godoc comments.
[ ] Provide a commit message to be used for the changelog entry in the PR description for review.
[ ] Re-reviewed Files changed in the Github PR explorer.
[ ] Review Codecov Report in the comment section below once CI passes.

Pending upstream merge of the changes in interchaintesting https://github.com/strangelove-ventures/interchaintest/pull/493 and a bump of the cosmos-sdk to <IP_ADDRESS> (since interchaintesting bumped that recently!)

So, the biggest changes here are basically the need to add the following after the hermes relayer starts:
s.Require().NoError(test.WaitForBlocks(ctx, 5, chainA, chainB), "failed to wait for blocks")

it appears as if some time is always required before hermes is able to see that packets have been relayed after starting, one explanation for this is possibly that its startup time is slower than that for rly.
I think it shouldn't be too hard to add the previous snippet in s.StartRelayer and discriminate based on relayer.

Codecov Report

Merging #3408 (33f17ed) into main (8e9d797) will increase coverage by 0.03%.
The diff coverage is n/a.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #3408      +/-   ##
==========================================
+ Coverage   78.76%   78.80%   +0.03%
==========================================
  Files         182      182
  Lines       12688    12689       +1
==========================================
+ Hits         9994     9999       +5
+ Misses       2262     2258       -4
  Partials      432      432

see 5 files with indirect coverage changes

Codecov Report

Merging #3408 (777fced) into main (df2841d) will increase coverage by 0.01%.
The diff coverage is 73.33%.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #3408      +/-   ##
==========================================
+ Coverage   78.78%   78.80%   +0.01%
==========================================
  Files         186      187       +1
  Lines       12769    12847      +78
==========================================
+ Hits        10060    10124      +64
- Misses       2278     2292      +14
  Partials      431      431

Impacted Files
Coverage Δ

modules/core/02-client/genesis.go
20.93% <0.00%> (-1.03%)
:arrow_down:

modules/core/02-client/types/keys.go
75.00% <ø> (ø)

modules/core/03-connection/types/version.go
90.52% <ø> (ø)

modules/core/module.go
47.29% <33.33%> (-1.32%)
:arrow_down:

modules/core/02-client/types/params_legacy.go
50.00% <50.00%> (ø)

modules/core/02-client/keeper/migrations.go
53.33% <66.66%> (+20.00%)
:arrow_up:

modules/core/02-client/types/msgs.go
75.00% <66.66%> (-0.74%)
:arrow_down:

modules/core/02-client/keeper/keeper.go
83.58% <90.47%> (+0.85%)
:arrow_up:

modules/core/02-client/types/codec.go
79.79% <100.00%> (+0.20%)
:arrow_up:

modules/core/02-client/types/params.go
100.00% <100.00%> (+10.00%)
:arrow_up:

... and 2 more

... and 3 files with indirect coverage changes

Hey guys, just want to remind you all that Hermes v1.5 is out. It might be worth updating to the latest version of Hermes in your tests 🙂

thanks for the heads up @seanchen1991 I'll bump it now since we hardcode it atm in our tests until we can update to the latest interchaintest version (which should also be bumped!)


Suggestion: display edit mode

Since there are three edit modes for this extension, it would be nice if there was an indicator of what edit mode you were in, similar to native vim at the bottom of the screen.

I see that there are two modes displayed, command and edit, but these are jupyterlab modes, not jupytelab-vim modes. I was just confused how these correspond to Jupyter command, Vim command, and Vim insert modes. For example, if I'm in edit mode and I type i to insert, it still indicates edit mode, whereas I would expect it to say insert

I tend to use the cursor as an indicator for the current mode, which means you don't need to look away to check the status bar:

No cursor: JupyterLab command mode
Block cursor: JupyterLab edit mode, vim command mode
I-beam cursor: JupyterLab edit mode, vim insert mode


Use Numpy 2.0

NumPy 2.0 was recently released. We need to make PyGSP2 compatible with this new version.
Check the following links for more information:

https://numpy.org/doc/stable/numpy_2_0_migration_guide.html
https://numpy.org/doc/stable/release/2.0.0-notes.html

When creating an environment: astropy requires numpy <2.
ERROR: pip's dependency resolver does not currently take into account all the packages that are installed. This behaviour is the source of the following dependency conflicts.
astropy 6.0.0 requires numpy<2,>=1.22, but you have numpy 2.0.0 which is incompatible.


Windows environment variables don´t work with shortcuts

Description
If you use a shortcut with environment variables (I mean a path with "%username%", "%userprofile%", ...), the shortcut does not work in the files app anymore.
Steps To Reproduce

right-click on a folder and select "create shortcut".
right-click on the shortcut, select the info icon, select the "..." option from top-menu and choose the "shortcut" option.
in the input field of "Destination" replace the part of the path "C:\Users<Username>" with ""C:\Users%username%".
open the shortcut (double click, ...)

Steps to fix
I have no idea.
The path seems to be resolved incorrectly. Maybe write a "translator" for paths that puts them all in the same format.
Files Version
Version: <IP_ADDRESS> OS Version: 10.0.22621.963
Windows Version
Windows 11 22H2 22621.963
Relevant Assets/Logs
2023-01-03 17:38:54.8166|INFO|StartSTATask|An error occurred trying to start process '%userprofile%\Documents<abc>< abc > with working directory 'C:\WINDOWS\System32'. Das System kann die angegebene Datei nicht finden.
System.ComponentModel.Win32Exception (2): An error occurred trying to start process '%userprofile%\Documents<abc>< abc > with working directory 'C:\WINDOWS\System32'. Das System kann die angegebene Datei nicht finden.
at System.Diagnostics.Process.StartWithCreateProcess(ProcessStartInfo startInfo)
at System.Diagnostics.Process.Start(ProcessStartInfo startInfo)
at Files.App.Shell.LaunchHelper.<>c__DisplayClass3_0.<b__0>d.MoveNext()
--- End of stack trace from previous location ---
at Files.App.Shell.Win32API.<>c__DisplayClass3_0`1.<b__0>d.MoveNext()
i have replaced parts of the path with "abc"

Asking @yaira2 for approval.


Growls aren't shown if app is not focused

Describe the bug
In-app growls aren't shown if the app is not the active window. I'm not sure if this is intended but my desire is that they are shown. I use them to display notifications, my app is always on top but not focused. I dont want them to appear on my desktop but within the non focused app. If this is intended, may I ask to provide an additional overload for in-app growl to appear always?
To Reproduce
Steps to reproduce the behavior:

Create async method
Add delay, after show a growl
Launch and set focus to another window

Expected behavior
Growl should appear
Environment (please complete the following information):

.net: 6.0
IDE 2022
Version 3.2.0

Additional context

Here is the demonstration GIF, first info growl shows up because window has focus. On second growl I focus another window, no growl shows up this time.

you may not have read the document carefully, you must put hc: Growl.Token on Panel not Window:
<ScrollViewer VerticalScrollBarVisibility="Hidden" HorizontalAlignment="Right">
    <StackPanel controls:Growl.Token="SuccessMsg" VerticalAlignment="Top" Margin="0,10,10,10"/>
</ScrollViewer>

@Symbai  Does this mean that your problem is solved?

#1497 Growl must be window focused, It's possible to focus manually before show, but it doesn't work if the program is minimized.


TypeError: Cannot read property 'document' of undefined

I installed first - npm install apexcharts --save
then connected to the component - import ApexCharts from 'apexcharts'
I took an example from - https://apexcharts.com/react-chart-demos/radar-charts/basic/
and got an error TypeError: Cannot read property 'document' of undefined
"apexcharts": "^3.10.0"
"react": "^16.8.6"
node v8.9.0
npm 5.5.1

I have the same issue.

I experienced the same issue : it was due to SSR, as apexcharts needs a window object during rendering.
You can use dynamic import to fix that
import loadable from 'loadable-components';
const Chart = loadable(() => import('react-apexcharts'));

I experienced the same issue : it was due to SSR as apexcharts requires a window object during rendering.
To fix it, you can use dynamic import
import loadable from 'loadable-components';
const Chart = loadable(() => import('react-apexcharts'));


[FLINK-25570][streaming] Add topology extension points to Sink V2

What is the purpose of the change
This PR marks the second part of API changes of FLIP-191 https://issues.apache.org/jira/browse/FLINK-25555 introducing the interfaces used to execute arbitrary topologies before and after the sink writer and committer.
Brief change log

Introduce mixin interfaces to construct a topology sink
Add committable message classes to clarify the exchange

Verifying this change
Primarily the PR consists of API changes

Added a test to verify the serialization of the committable messages

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (yes / no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (yes / no)
The serializers: (yes / no / don't know)
The runtime per-record code paths (performance sensitive): (yes / no / don't know)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: (yes / no / don't know)
The S3 file system connector: (yes / no / don't know)

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

@gaoyunhaii sorry for pinging again but it would be great to also hear your opinion on the interfaces :) ff76068

Very thanks @fapaul for drafting the PR! I'll have a look~

@flinkbot run azure

Thanks for your contributions!


[filterset]Can not apply filterset

User can NOT select and apply filterset.
How to reproduce the bug

enable filter set feature flag  "DASHBOARD_NATIVE_FILTERS_SET": True, in config.py
Open any dashboard and expand the filter bar
Create a few(2-3) filters in the ALL FILTERS tab
Go to Filter Sets tab and create a set with those filters
Go back to ALL FILTERS tab and create 2 more filters
Go to FILTER SETS tab to create a new set with these 2 new filters
Stay in FILTER SETS tab, try to click between the 2 sets and apply one of them each time

Expected results
APPLY button should light up when I click on either set
Actual results
APPLY button stay grey
Screenshots
https://user-images.githubusercontent.com/67837651/134580804-ba4d5606-0345-4c00-bceb-5f33c0a2317c.mov
Environment
(please complete the following information):

browser type and version: chrome
superset version: superset version : master
python version: python --version
node.js version: node -v
any feature flags active:  "DASHBOARD_NATIVE_FILTERS_SET": True,

Checklist
Make sure to follow these steps before submitting your issue - thank you!

[ ] I have checked the superset logs for python stacktraces and included it here as text if there are any.
[x ] I have reproduced the issue with at least the latest released version of superset.
[x ] I have checked the issue tracker for the same issue and I haven't found one similar.

Additional context
Add any other context about the problem here.

We have Dashboards with data for a lot of users: We use many filters to segment this data  each user can select and we see the point in using the filter sets so that every user can have shortcuts to their preferred filter configurations.
This simplifies our Dashboard design and management because with a single Dashboard we fit many different specific Dashboards. It also helps transparency as all users have access to all the data.
Right one we are not using it because the users can not create the filters even if they have permissions. Only the Dashboard owner can create and use filter sets. We were looking forward for the full functionality.
The approach of an admin creating common filter sets and the users having also specific ones looked great for us.


Add PyYAML as a dependency

Codecov Report

Merging #15 (2684df4) into main (458ed25) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #15   +/-   ##
=======================================
  Coverage   74.69%   74.69%
=======================================
  Files           5        5
  Lines         664      664
  Branches      136      136
=======================================
  Hits          496      496
  Misses        142      142
  Partials       26       26

Flag
Coverage Δ

python-3.8
74.69% <ø> (ø)

python-3.9
74.69% <ø> (ø)

unittests
74.69% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

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 458ed25...2684df4. Read the comment docs.


Set conditions type to BinaryExpression | BooleanClauseList

I am seeing instances in fidesops where conditions is not a list. I haven't been able to narrow it down to a specific type being sent here so setting it to Any.

good changes @sanders41, I'll let you merge these in the order you see fit


Incorrect Equation

2,400,000,000 / (30000*1) equals to 100,000? Please review that equation part.

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

ID: 6bc978a2-5a8b-c6af-0c56-f762ea9edf16
Version Independent ID: 33693e91-d1a2-10c0-0dd7-43167114868a
Content: Windows Time Service Tools and Settings
Content Source: WindowsServerDocs/networking/windows-time-service/Windows-Time-Service-Tools-and-Settings.md
Service: unspecified
GitHub Login: @billmath
Microsoft Alias: billmath

Resolving in private repo branch GitHubIssues072018

#please-close


expo router layout missing slot

https://github.com/rajatsandeepsen/t3-turbo-biome/blob/eb84e5eff73047a4f01a5319ee588bf95dd3eb07/apps/native/src/app/_layout.tsx#L12
I was refering your code for setting up  the expo router and I noticed a error  here , expo router states that we need to provide a slot in the root layout like children property in react for more info you can refer  link

💀

I made some changes in last 3 days.. hope it works


pythonPackages.tables: 3.4.4 -> 3.5.2

Motivation for this change
Update pytables. Fixes, for instance, the following annoying bug: https://github.com/pandas-dev/pandas/issues/24839
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option 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 nix-review --run "nix-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Assured whether relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

@GrahamcOfBorg build pythonPackages.tables

Included in https://github.com/NixOS/nixpkgs/pull/62245 and landed in staging-next.


Moved all logging into the Python output channel

For #9837

[x] (src/client/logging/transports.ts) add transport
[x] (src/client/logging/logger.ts) use transport
[x] do not use transport when running tests
[ ] drop console transport from logger upon adding output panel to logger (I've removed logging to console when not running tests, is that what this means, @ericsnowcurrently ?)

For the trace decorators, this requires that the logger remain global and that the transport be added during extension init (due to DI).

[x] Pull request represents a single change (i.e. not fixing disparate/unrelated things in a single PR).
[x] Title summarizes what is changing.
[x] Has a news entry file (remember to thank yourself!).
[x] Appropriate comments and documentation strings in the code.
[ ] Has sufficient logging.
[ ] Has telemetry for enhancements.
[ ] Unit tests & system/integration tests are added/updated.
[ ] Test plan is updated as appropriate.
[ ] package-lock.json has been regenerated by running npm install (if dependencies have changed).
[ ] The wiki is updated with any design decisions/details.

@ericsnowcurrently That indeed makes things a lot cleaner, much thanks!

Codecov Report

Merging #11695 into logging-changes-and-drop-old-debugger will decrease coverage by 0.03%.
The diff coverage is 33.33%.

@@                            Coverage Diff                            @@
##           logging-changes-and-drop-old-debugger   #11695      +/-   ##
=========================================================================
- Coverage                                  60.72%   60.69%   -0.04%
=========================================================================
  Files                                        629      629
  Lines                                      33992    34007      +15
  Branches                                    4790     4792       +2
=========================================================================
- Hits                                       20642    20640       -2
- Misses                                     12346    12361      +15
- Partials                                    1004     1006       +2

Impacted Files
Coverage Δ

src/client/logging/transports.ts
65.85% <11.11%> (-15.40%)
:arrow_down:

src/client/logging/_global.ts
81.81% <50.00%> (-3.43%)
:arrow_down:

src/client/logging/index.ts
100.00% <100.00%> (ø)

src/datascience-ui/react-common/arePathsSame.ts
75.00% <0.00%> (-12.50%)
:arrow_down:

src/client/common/utils/platform.ts
64.70% <0.00%> (-11.77%)
:arrow_down:

src/client/linters/pydocstyle.ts
86.66% <0.00%> (-2.23%)
:arrow_down:

src/client/datascience/debugLocationTracker.ts
76.56% <0.00%> (-1.57%)
:arrow_down:

src/client/common/process/proc.ts
14.49% <0.00%> (-0.73%)
: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 5aaca27...be2f55a. Read the comment docs.


Issue with Ember Deploy

I am getting this error when trying to run an ember deploy:
- build failed Error: Build Canceled: Broccoli Builder ran into an error with AssetRewrite plugin. 💥 Cannot read property '2' of null TypeError: Cannot read property '2' of null     at AssetRewrite.rewriteAssetPath (/home/ubuntu/hart/node_modules/broccoli-asset-rewrite/index.js:127:91)     at AssetRewrite.processString (/home/ubuntu/hart/node_modules/broccoli-asset-rewrite/index.js:167:24)     at AssetRewrite.processFile (/home/ubuntu/hart/node_modules/broccoli-filter/index.js:165:31)     at asyncProcessFile (/home/ubuntu/hart/node_modules/broccoli-filter/index.js:122:21)     at tryCatch (/home/ubuntu/hart/node_modules/rsvp/dist/rsvp.js:525:12)     at invokeCallback (/home/ubuntu/hart/node_modules/rsvp/dist/rsvp.js:538:13)     at /home/ubuntu/hart/node_modules/rsvp/dist/rsvp.js:606:14     at flush (/home/ubuntu/hart/node_modules/rsvp/dist/rsvp.js:2415:5)     at _combinedTickCallback (internal/process/next_tick.js:67:7)     at process._tickCallback (internal/process/next_tick.js:98:9) Pipeline aborted

Issue was with using ember browserify.


For gvrvideo, you can't stop audio after pressing home key.

You can hear audio still playing after stop the app. It means the Media Player is still running even though we quit the app. Only killing the process can stop everything( and it definitely should be.)..

Fixed.


BUG:IndexError: list index out of range

I am running MOOSE in a patient folder with two subfolders for CT and PET under DCIOM format. However, I am getting this error  message:
moose_ct_atlas = ie.segment_ct(ct_file[0], out_dir)
File "/export/moose/moose-0.1.0/src/inferenceEngine.py", line 78, in segment_ct
out_label = fop.get_files(out_dir, pathlib.Path(nifti_img).stem + '*')[0]
IndexError: list index out of range
Any suggestion, please?
Thanks,

Hello, does your PET image have the brain in the field-of-view? If no, please delete the PET dicom folder and everything should run fine.

Yes, it contains the brain in the FOV.

Ok, interesting! Would you be kind enough to share the folder structure (along with the generated files), I can have have a look. If the files are anonymous you can also send them to my mail ID<EMAIL_ADDRESS>would make the debugging process much easier :)

Thank you so much! I just sent you the screenshots via email. The DICOM files are not anonymous. If it doesn't help, let me know, and I will find anonymous DICOM files.
Again, thank you so much.

I have anonymized the patient data. I can send it to you. I am not familiar with github, can I send it through BOX?'
Thanks,
Mahdjoub

Sure share it via box. I saw your screenshots. It seems like the models are not being run on your dataset. Can be a path issue. But let me confirm once I have anonymous datasets from you :)!
Cheers!

I have shared the DICOM data with you.
Let me know if you need anything.
Again, your help is very appreciated.
Thank you!

Excellent! I got your mail. I am traveling now. I will work on it first thing tomorrow :)!

Thank so much!

Hi @Ompsda,
I ran your dataset, the results are sent to you via mail.
I am writing the response here, so that we avoid duplicates in the future.
Couple of things that you should check.

When I first ran it, MOOSE recognised the AC map as ‘MR’. Because I assume you pushed the CT maps to MR-AC headers. Since MOOSE organises things based on the DICOM tags, I changed the modality tag of the DICOM header to CT.
Then I ran the stuff, MOOSE worked. However, when I checked your screenshot of the MOOSE print logs, It seems that the inference of the CT abdominal organs was not done. Would you be kind enough to check the environment variables by using the following commands in your command line terminal
'echo $RESULTS_FOLDER’
This should display the path where the nnUNet based models are stored. If you don’t see anything, you should source your bashrc file. If it still doesn’t work, I would ask you to check your .bashrc file for the nnUNet based environment variables entry. You need to ideally have three entries in the file.
Please let me know how it goes.
Will be glad to help.

@Ompsda : any luck with getting moose to run? Also if it didn't work, please use the latest version of moose!

Hi,
I think the issue is that the software was not developed to work on CHPC. Our CHPC admin stated he cannot install it following your instructions on Github. Also, the best workstation we have has 64 G RAM, which is lower than what MOOSE requires.
Thanks,

Hi there,
I am sorry that the admin was not positive to the installation of moose.
If it helps, I am available to discuss with your IT admin to see if moose can be installed in your cluster. Ideally the docker image should work without any issues.
I am happy to help if your admin is interested in coordinating this effort.
Cheers,
Lalith
Sent from my iPhone

On 02.11.2022, at 18:47, Ompsda @.***> wrote:
CHPC

Hi Latih,
Thank you for being responsive. I wanted to give MOOSE a try, however, it is not a priority for us now. I will let you know in case we want to use it in the future.
Thank you,


[Feature][ST-Engine] Storage Running Job and it's status to hazelcast IMap

Search before asking

[X] I had searched in the feature and found no similar feature requirement.

Description
A sub task of https://github.com/apache/incubator-seatunnel/issues/2501
Update runningJob, Job Status(JobStatus, PipelineState, ExecutionState, stateTimestamps) to hazelcast IMap
RunningJobInfo
The runningJobInfo will contain initializationTimestamp  and jobImmutableInformationData. initializationTimestamp is the job init timestamp.
The IMap name is runningJobInfo, the key is jobId and the value is RunningJobInfo
IMap<Long, RunningJobInfo> runningJobInfoIMap = nodeEngine.getHazelcastInstance().getMap("runningJobInfo");

PhysicalPlan.jobStatus
Now jobStatus type is AtomicReference<JobStatus>, I will change it to IMap<Long, JobStatus>. The key is JobId and the value is the current JobStatus.
the key of IMap is the job id and the value of the IMap is JobStatus
IMap<Object, Object> runningJobStateIMap = nodeEngine.getHazelcastInstance().getMap("runningJobState");

In the updateJobState(@NonNull JobStatus current, @NonNull JobStatus targetState) we use compareAndSet to update Job state. I will change it to synchronized updateJobState(@NonNull JobStatus current, @NonNull JobStatus targetState) to ensure the state turn right.
PhysicalPlan.stateTimestamps
I will update the stateTimestamps type to IMap too.
The key is job id and value is stateTimestamps.
IMap<Object, Object> runningJobStateIMap = nodeEngine.getHazelcastInstance().getMap("stateTimestamps");

SubPlan.pipelineState
The pipeline state use the IMap runningJobState too. I want to create as few IMap as possible, because each IMap will create a WAL file.
Add a class PipelineId
public class PipelineId {
        private final long jobId,

        private final long pipelineId,
}

The key of IMap is PipelineId and value is PipelineState
IMap<Object, Object> runningJobStateIMap = nodeEngine.getHazelcastInstance().getMap("runningJobState");

SubPlan.stateTimestamps
The key is PipelineId and value is stateTimestamps.
IMap<Object, Object> runningJobStateIMap = nodeEngine.getHazelcastInstance().getMap("stateTimestamps");

PhysicalVertex.executionState
The key is TaskGroupLocation and value is ExecutionState.
IMap<Object, Object> runningJobStateIMap = nodeEngine.getHazelcastInstance().getMap("runningJobState");

PhysicalVertex.stateTimestamps
The key is TaskGroupLocation and value is stateTimestamps.
IMap<Object, Object> runningJobStateIMap = nodeEngine.getHazelcastInstance().getMap("stateTimestamps");

I will modify the JobMaster.initFuture method to check the current state of PhysicalVertex/SubPlan/PhysicalPlan while init the future. If the state is a End state, the future will be complete.
Usage Scenario
No response
Related issues
No response
Are you willing to submit a PR?

[X] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

@hailin0  @Hisoka-X  @CalvinKirs  @ashulin  @ic4y  @laglangyue  @TyrantLucifer  @2013650523
Do you have any suggestions?

The pipeline state use the IMap runningJobState too. I want to create as few IMap as possible, because each IMap will create a WAL file.
The WAL file count is not determined by this. So you can rest assured on this one.

The pipeline state use the IMap runningJobState too. I want to create as few IMap as possible, because each IMap will create a WAL file.

The WAL file count is not determined by this. So you can rest assured on this one.

I share his opinion too, you can boldly use.

The pipeline state use the IMap runningJobState too. I want to create as few IMap as possible, because each IMap will create a WAL file.

The WAL file count is not determined by this. So you can rest assured on this one.

Ok, thank you

Now the  type of JobStatus, PipelineState, ExecutionState  are AtomicReference, we use compareAndSet to ensure the atomicity of state updates. To use IMap to storage state and ensure the stateTimestamps  and states update together, I will use an Object Lock in method updateTaskState and turnToEndState.

The runningJobInfo will contain initializationTimestamp and jobImmutableInformationData. initializationTimestamp is the > job init timestamp.
The IMap name is runningJobInfo, the key is jobId and the value is RunningJobInfo

How about using  Tuple2<Data, Long> instead of RunningJobInfo? I think and a class RunningJobInfo is no need.

Closed by https://github.com/apache/incubator-seatunnel/pull/2699


Add physical optimization example

Hi all,
Thank you for the continued work on JAX. I was able to use JAX to optimize a basketball backboard. Here is some docs on my blog: link. AFAIK this is only possible thanks to JAX.
Best,
Ricky

@googlebot I signed it!

@gnecula @jakevdp this example looks kind of cool, based on the author's post  https://rickyhan.com/jekyll/update/2020/06/24/physical-optimization-with-JAX.html WDYT? There's a CLA issue.

"The idea is very simple: shoot lots of balls, simulate elastic collision, calculate distance from hoop and backpropagate the loss into the board shape"

@0b01 Thank you for the kind words about JAX, and for sharing the example. The example is cool indeed, but we would prefer it to be hosted elsewhere. Each example that we add to the JAX repo increases the maintenance cost, and we need to be careful to ensure that we incur cost only where there is a unique advantage to maintaining the code in our repo. Perhaps we should have been more careful in the past with some of the examples that are already in the repo.
I did a quick search for physical optimization jax and this example pops up. If we want even more visibility, it may be useful to carry on a discussion.


Page is unreadable

https://docs.microsoft.com/en-us/dotnet/api/microsoft.entityframeworkcore.ef.compileasyncquery?view=efcore-6.0 is unreadable

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

ID: c0a68bee-6e78-b2b3-bacc-b3ef6873252b
Version Independent ID: cd5b3292-61d7-c3f8-09aa-ce974a0b9f32
Content: EF.CompileAsyncQuery Method (Microsoft.EntityFrameworkCore)
Content Source: dotnet/xml/Microsoft.EntityFrameworkCore/EF.xml
Product: ef-core-api
GitHub Login: @dotnet-bot
Microsoft Alias: avickers

Note from triage: consider removing this page from the published docs. There is already an internal issue with the docs team about its rendering.


Access external address in alerts

This PR adds the external URL as a template variable for alerts. Because the flag config is separate from the yaml config, I chose to not add it to all the reloaders and instead used a variable set above for the external URL.
https://github.com/prometheus/prometheus/blob/51e4e2a5264172412506e5cb2e6c26507bd16c56/cmd/prometheus/main.go#L531
https://github.com/prometheus/prometheus/blob/51e4e2a5264172412506e5cb2e6c26507bd16c56/cmd/prometheus/main.go#L594-L599
Fixes #8695

Thanks, I just have left one comment.
cc @dgl, if you have the time to look at the changes in promql testing code

Thanks!


envoy: Lock stream server while adding remove completion.

Internal slice manipulations must be protected, and one of the calls
to addCompletions() missed this.
Signed-off-by: Jarno Rajahalme<EMAIL_ADDRESS>

test-me-please

test-me-please


JObject ignores JsonSerializerSettings during serialization

Using settings.NullValueHandling = NullValueHandling.Ignore, returning the following object from an asp.net core controller:
return new
{
    Foo = (string)null,
    Bar = "bar",

    Baz = JObject.FromObject(new
    {
        Foo = (string)null,
        Bar = "bar"
    })
};

The json generated is:
{
   "bar":"bar",
   "baz":{
      "Foo":null,
      "Bar":"bar"
   }
}

See that the null value is included in the JObject, but not in the object directly. Also, the property name doesn't have the first letter lowercased per asp.net core defaults. The serializer seems to be ignoring the settings for JObjects completely.
I'm using 9.0.1

No settings are being passed to FromObject

Hi.  I'm not sure I'm following.  In order to get the output serialized correctly,  I'm supposed to pass the settings to the parser? Doesn't make much sense, since we don't get to control how jobjects are created.

You do get control.  Ask on stackoverflow

I looked around on SO before posting this bug report. The only answer there creates a helper that manually removes null tokens. Doesn't look like what you're referring to.
I still feel there is something wrong here, so I'll insist one last time.
Consider the following code:

// startup.cs has the following
services.AddMvc().AddJsonOptions(o =>
{
    o.SerializerSettings.NullValueHandling = NullValueHandling.Ignore;
});

// actual sample
public class SampleController : Controller
{
    JsonSerializerSettings _settings = new JsonSerializerSettings
    {
        NullValueHandling = NullValueHandling.Ignore
    };

    [HttpPost]
    [Route("object")]
    public object PostObject([FromBody] SomeObject data)
    {
        return JsonConvert.SerializeObject(data, _settings);
    }

    [HttpPost]
    [Route("jobject")]
    public object PostJObject([FromBody] JObject data)
    {
        return JsonConvert.SerializeObject(data, _settings);
    }

    [HttpGet]
    [Route("jobject-parse")]
    public object GetJObjectParse()
    {
        var json = @"{ ""Foo"": ""Foo"", ""Bar"": null }";
        var data = JObject.Parse(json, new JsonLoadSettings { /* nothing here to control formatting */ });
        return JsonConvert.SerializeObject(data, _settings);
    }

    public class SomeObject
    {
        public string Foo { get; set; }
        public string Bar { get; set; }
    }
}

I'm posting { "Foo": "Foo", "Bar": null },

/object returns {"Foo":"Foo"}
/jobject returns {"Foo":"Foo","Bar":null}
/jobject-parse returns {"Foo":"Foo","Bar":null}

This definitely is weird. And the main reason it's weird is that I'm manually calling SerializeObject passing the output settings.
It seems JObject ignores the settings during the parsing, and also ignores it during the output formatting.
Even if there is a bug at the parsing side, I'd argue that it makes no sense to control the output formatting of an object during the parsing phase. But from what I can see, ASP.NET always uses the defined settings when creating the JsonSerializer to parse the body. Also, there is no overload of JObject.Parse that accepts a JsonSerializerSettings object.
Something is strange here. If as the creator of this this really makes sense for you, I ask you politely please just give a brief explanation on if the JObject is not supposed to be used this way, or there is another way to control the output of JObject using the JsonSerializerSettings we have.

A JObject isn't serialized. It is written as is.
Yes you can control the output of a JObject. Like I said, ask on stackoverflow. Someone will help you.

Hi. Apparently no one has any idea how to achieve this. Would you care to answer there?
http://stackoverflow.com/questions/40244395/how-to-serialize-a-jobject-the-same-way-as-an-object-with-json-net
Thanks.

@nvivo, did you find out how to do this? I've run into the same issue and I can't work it out either.

@nizmow I ended up manually doing the things I needed by cloning the JObject, walking the json tree and applying the changes manually (like changing things to camelCase, removing nulls, etc) before calling ToString().

@JamesNK Would it be possible to add an option to JObject.ToString() or just some other method on JObject that removes null values? https://stackoverflow.com/questions/33027409/json-net-serialize-jobject-while-ignoring-null-properties includes example code for this. However, it seems that removing null properties is something common enough that there should be some kind of helper method included with JObject by default. It seems like an oversight to me.


add bitdeli badge

What about this one?

Since the last post on their blog (http://blog.bitdeli.com/) is one year old and says their badge isn't really going to work any more, I'm not sure this is a good idea.

Yeah, that post looks discouraging.

Ok :)


Node returned from replace leave callback isn't replacing the existing node?

I've got a simple leave function (for use with estraverse.replace(leaveFn)) like this:
function leaveFn(node, parent) {
    if (
        node.type == 'ExpressionStatement'
        && parent.type == 'Program'
        && node.expression.type == 'CallExpression'
        && node.expression.callee.name == 'module'
    ) {
        node.expression.callee.name == 'define'
        return node
    }
}

I'm modifying a subnode (node.expression.callee), then returning node, but when I check output, I still see "module" instead of "define".
Am I doing it right?
This is my first time doing an AST transform. Basically, I'm trying to convert some code like
module({
  //
}, function() {
  //
});

to
define({
  //
}, function() {
  //
});

among other transformations that I'll try after I get past this first step.

Doh! It was a typo, I had == instead of =.
        node.expression.callee.name == 'define'


Set url trailing slash

Summary
url-parse latest version is returning pathname: '/' which is failing OAUTH/SAML logins.
This PR removes trailing slash while setting the url.
Ticket Link
https://mattermost.atlassian.net/browse/MM-34855
Device Information
This PR was tested on: iPhone 12 Simulator, iOS 14.4

Building app in separate branch.

Successfully building:
https://circleci.com/gh/mattermost/mattermost-mobile/15129
https://circleci.com/gh/mattermost/mattermost-mobile/15128

Artifact links:
https://15129-70265724-gh.circle-artifacts.com/0/tmp/artifacts/Mattermost_Beta.ipa
https://15128-70265724-gh.circle-artifacts.com/0/tmp/artifacts/Mattermost_Beta.apk

Cherry pick is scheduled.


Unable to install driver using device manager

Tried as #49 said but in the very end of wizard it said Not installed, having 1 Error or sth else.
Meanwhile, seems the installer version was installed nothing.

WELL DONE, ISSUE TEMPLATE YOU MADE.

I take it you solved the problem, or you asked/will ask in one of the suggested places?

unsolved, annoyed, tiredMaybe I’ll come back later and try to fill your issue template. 发自我的 iPhone在 2024年11月10日，01:02，Cherry @.***> 写道：
I take it you solved the problem, or you asked/will ask in one of the suggested places?
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you modified the open/close state.Message ID: @.***>

Feel free to open a discussion thread on this repo (discussion section) or post in my discord server. Discord will be the fastest way for you to get help.
I also edited my last comment with the instructions in case you weren't able to see that yet


Options in demo for CSS perspective-origin have no effect

Request type

[ ] Bug report

Details
On the MDN CSS perspective-origin page, selecting the different options in the demo has no effect.
See video:
https://youtu.be/fSfOspWmXP8
That video was done with Firefox Developer Edition 84.0b3. The behaviour (or lack of it!) is the same in Chrome 87.0.4280.67.

@adamalton you are correct - looks like it is missing the perspective property. I'll go fix that in the example though it may take a little while to be released onto the site.

Great, thanks @rachelandrew.

@chrisdavidmills this now shows up on the page so can probably be closed.

Great stuff, thanks for the note.


[ci] prebuild sccache.

Motivation
Since xbuild/xtest will automatically install sccache (currently configured to be from a forked git repo to support anonymous reads from a public s3 bucket (v4 s3 api) see:   https://github.com/mozilla/sccache/pull/869 and https://github.com/libra/libra/commit/ac9775c67a709ad6080f587ea8037d260b66ba4d#diff-35f96293e9215fbdc2ca9bc277d6b326144f84b71138810a6afcb79c01425c8bR23-R27
Now that we're using sccache from a forked git repo, prebuild sccache and use docker hub circleci-2 tag of our build image.
Have you read the [Contributing Guidelines on pull requests]
Yes
Test Plan
Tested and built here:   https://app.circleci.com/pipelines/github/libra/libra/34073/workflows/507d860b-1d0e-45d2-a719-5acc35e4152e
Related PRs
None

@metajack update the comment to provide more context, and yes until sccache support s3 buckets with the v4 api (merges my pr, built on top of another pr) we'll need a forked sccache.

/land


--id description is ambiguous for Managed HSMs

Describe the bug
As of version 2.13.0, the CLI says of az keyvault security-domain download --id:
Id of the HSM.

That's not very helpful, given ID is pretty overloaded. I tried the resource ID to no avail (e.g. /subscriptions/{guid}...). Could you clarify what this is supposed to be?
Command Name
az keyvault security-domain download
Errors:
security_domain_download() got an unexpected keyword argument 'vault_base_url'

To Reproduce:
Steps to reproduce the behavior. Note that argument values have been redacted, as they may contain sensitive information.

az keyvault security-domain download --help

Expected Behavior
The help clearly defines what the --id value should be.
Environment Summary
Linux-4.19.128-microsoft-standard-x86_64-with-debian-buster-sid
Python 3.6.10
Installer: DEB

azure-cli 2.13.0

Additional Context

hi @bim-msft , could you pls refine the message for all HSM related commands, thanks

@heaths thanks for the feedback. Actually --id here means the base uri of the HSM ("https://xxx.managedhsm.azure.net"), not the resource id. I will clarify this in help messages.

That's not consistent with --hsm-name that's used everywhere else I've seen. I would suggest changing that. Everywhere else, --id typically refers to either the full URI (e.g. az keyvault secret delete --id refers to the full secret URI) or the full resource URI (e.g. az resource show --ids /subscriptions/{sub}/...). Using --id here is just confusing and inconsistent.


[release-1.6] Move Muticluster/VM cert setup tool to install/ dir.

This is an automated cherry-pick of #23895

@myidpt Will this require a docs change in 1.6.1?

@jacob-delgado do not merge this for 1.6.0 please. This will have ripple effects through all of the documentation. Yes this will require documentation changes for 1.6.1.

@jacob-delgado do not merge this for 1.6.0 please. This will have ripple effects through all of the documentation. Yes this will require documentation changes for 1.6.1.

Approving now for 1.6.1.


app.use() requires middleware functions

Maybe you should consider fixating some of the dependency versions. It seems there are some API changes in express and related packages. Getting this:
TypeError: app.use() requires middleware functions
  at EventEmitter.use (/Users/kewisch/github/foaas/node_modules/express/lib/application.js:209:11)
  at EventEmitter.wrappedAppUse [as use] (/Users/kewisch/github/foaas/node_modules/newrelic/lib/instrumentation/express.js:348:11)
  at Object.module.exports.register (/Users/kewisch/github/foaas/lib/filters/i18n.coffee:17:9)
  at FOAAS.module.exports.FOAAS.loadFilters (/Users/kewisch/github/foaas/lib/foaas.coffee:90:14)
  at FOAAS.loadFilters (/Users/kewisch/github/foaas/lib/foaas.coffee:1:1)
  at new FOAAS (/Users/kewisch/github/foaas/lib/foaas.coffee:42:6)
  at Object.<anonymous> (/Users/kewisch/github/foaas/lib/server.coffee:6:13)
  at Object.<anonymous> (/Users/kewisch/github/foaas/lib/server.coffee:1:1)
  at Module._compile (module.js:456:26)

ran into the same error :(

Have pushed a fix, but very simple edit to pin.
https://github.com/tomdionysus/foaas/pull/117/files

All done.


v1.10.2-<IP_ADDRESS> hanging on startup after version check fail

Hi any thoughts on this?  Installed the update.  First run works!  But subsequent runs, for some reason, hang on this error:
DynamicSurroundings-1.10.2-<IP_ADDRESS>.jar

[09:19:18] [Forge Version Check/INFO] [ForgeVersionCheck]: [dsurround] Starting version check at https://raw.githubusercontent.com/OreCruncher/DynamicSurroundings/master/version.json
[09:19:18] [Forge Version Check/INFO] [ForgeVersionCheck]: [dsurround] Found status: UP_TO_DATE Target: null

It's not an error.  The status is found up to date and as such there isn't an update target.  If the mod version were behind the Target would indicate what the new version is.  BTW, this is the Forge version check routines that is doing this.

@gmalone Any more info on the hang?  I notice that with MultiMC it has a pretty long delay during/after initializing the sound engine.  That's not Dynamic Surroundings. :)

Thanks for following up with me on that. It turned out that it was not
Dynamic surroundings causing the problem. It was what was loading right
after it. So all is well my friend. Thanks. I love the plug-in!
On Jan 9, 2017 20:42, "OreCruncher"<EMAIL_ADDRESS>wrote:

@gmalone https://github.com/gmalone Any more info on the hang? I notice
that with MultiMC it has a pretty long delay during/after initializing the
sound engine. That's not Dynamic Surroundings. :)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/OreCruncher/DynamicSurroundings/issues/23#issuecomment-271478752,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADk2p_uENgrzeLru9qIpy9xoEEcpobfIks5rQv44gaJpZM4Lc3pR
.

Glad you like it.  Going to close this issue then.  If you find anything else be sure to let me know.


Move to 8.0.0 and add security page sql

Questions
Answers

Description?
The next version is 8.0.0 and not <IP_ADDRESS> and add security pages requirements

Type?
improvement

BC breaks?
no

Deprecations?
no

How to test?
Please indicate how to best verify that this PR is correct.

Possible impacts?
Please indicate what parts of the software we need to check to make sure everything is alright.

@PierreRambaud Hi, could you fill the table please ?


Mod loading error

I have a mod loading error with the 1.19 version of the mod.
I attach the crash log.
crash-2022-08-30_16.47.52-fml.txt

the 1.19 version has been coded under 41.0.3 iirc, 41.1 brought breaking changes.
I advice to update to 1.19.2 like any mod should, given the vast amounts of breaking changes between 1.19 and 1.19.2
the reason mine, or a bunch of mods, haven't updated in a while is because of the breaking changes. theres been like 5 between 1.19 and .2 that i know of.
.2 should be more stable and less prone to future breaking changes

closing issue as resolved
using an earlier version of forge, or a later version of the mod fixes this issue