It is too slow for irtr

Hi authors,
Thanks for your great work!
I am trying to reproduce the results but found it is too slow for irtr testing.
It seems that it needs 38 hours for inference on advanced computing architectures (like A100), specifically it is the rank loop that costs so much time.
It is normal?
Thanks!

Thanks! It is normal as for N test instances, ranking requires computing features for NxN image-text pairs. One potential way to increase the inference speed is caching the image and text features at the bottom layers, which I left as a TODO here https://github.com/zdou0830/METER/blob/main/meter/modules/objectives.py#L315.

OK, I see.
Thanks for your reply!

Best Regards,
Shizhe Diao
Ph.D. Student, Department of Computer Science and Engineering
The Hong Kong University of Science and Technology
Clear Water Bay, Kowloon, Hong Kong
Email: @.***

发件人: Zi-Yi Dou @.>
发送时间: 2022年1月26日 9:23
收件人: zdou0830/METER @.>
抄送: DIAO Shizhe @.>; Author @.>
主题: Re: [zdou0830/METER] It is too slow for irtr (Issue #12)
Thanks! It is normal as for N test instances, ranking requires computing features for NxN image-text pairs. One potential way to increase the inference speed is caching the image and text features at the bottom layers, which I left as a TODO here https://github.com/zdou0830/METER/blob/main/meter/modules/objectives.py#L315https://apc01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fzdou0830%2FMETER%2Fblob%2Fmain%2Fmeter%2Fmodules%2Fobjectives.py%23L315&data=04|01|shizhe.diao%40connect.ust.hk|787158867b2648e990fc08d9e06a7f2f|6c1d415239d044ca88d9b8d6ddca0708|1|0|637787570271215648|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=47rcynYvBFImoLG4moNh3vTCN9TKRot0rtMKUYgiCCo%3D&reserved=0.
―
Reply to this email directly, view it on GitHubhttps://apc01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fzdou0830%2FMETER%2Fissues%2F12%23issuecomment-1021766705&data=04|01|shizhe.diao%40connect.ust.hk|787158867b2648e990fc08d9e06a7f2f|6c1d415239d044ca88d9b8d6ddca0708|1|0|637787570271215648|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=lv3ioajPLw5%2FEvxatpK4w9fPQVSUySFTit%2FbklrDDX0%3D&reserved=0, or unsubscribehttps://apc01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAEKH3F2OCH266IFHPZDJ3FDUX5EJ7ANCNFSM5MZYDTYA&data=04|01|shizhe.diao%40connect.ust.hk|787158867b2648e990fc08d9e06a7f2f|6c1d415239d044ca88d9b8d6ddca0708|1|0|637787570271215648|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=ocNGuA%2FTYkjVsRf3iejJcb0%2Fx6ooPQD14CI5oqAh3S0%3D&reserved=0.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID: @.***>


Dashr oil and gas ternary

Issue for app: #[issue number here]
App pull request

[ ] This is a new app
[ ] I am improving an existing app (redesigns/code "makeovers")

About

Playground deployment URL (new version):
Current gallery app URL: (delete this line if inapplicable)
Python app repository link: (delete this line if you are working on a Python app)

Workflow

[ ] I have created a branch in the appropriate monorepo, and the
elements necessary for successful deployment are in place.
[ ] If the app is a redesigned and/or restyled version of an
existing gallery app, I've summarized the changes requested in the
appropriate Streambed issue and confirm that they have been applied.
[ ] If the app is on the Dash Gallery portal, I have added a link to
the GitHub repository for the source code in the portal description.
[ ] If the app is a reimplementation of a Python gallery app for the
DashR gallery, the app in this PR mimics, as closely as possible,
the style and functionality of the existing app.=
[ ] I have removed all Google Analytics code from the app's
assets/ folder.

The pre-review review
I have addressed all of the following questions:

[ ] Does everything in my code serve some purpose? (I have removed
any dead and/or irrelevant code.)
[ ] Does everything in my code have a clear purpose? (My code is
readable and, where it isn't, it has been commented appropriately.)]
[ ] Am I reinventing the wheel? (I have used appropriate packages to
lessen the volume of code that needs to be maintained.)

@rpkyle PR for Ternary app to be merged into master!

@rpkyle i believe everything is corrected. As per the css issue, i did not notice it when i ran it on my system.

@HammadTheOne hmm that is very weird indeed. Ill look into that later today. If all the other options work properly and this one doesn't, then it may not be a problem with the code

@rpkyle @HammadTheOne Fixed the satellite street view. Was a silly filtering where the : was changed to a = in "mapbox://styles/mapbox/satellite-streets-v9". Since R doesn't utilize colons for equate values. Conflicts have been resolved.


Use product name templates for render products on local Houdini render

Changelog Description
This PR extends #447 and make it work with local Houdini render
Testing notes:

Update these settings on ayon server: you can take reference from my screenshots below.

ayon+settings://core/tools/creator/product_name_profiles
ayon+anatomy://{my-project}/templates/publish/1 render template.

Launch Houdini
Publish Render locally. Here's what to expect:

you should find the names of the created runtime instances follow your the template profile
you should find the location of the published render follow your render template (we add support for two more keys renderlayer and aov)

Additional Info
Here are some snippets of my settings.
Render template:

product_name_profiles:

Additional Info 2
I'd like to point our to this todo because I don't know the favored way to implement it.
In essence:
This logic
https://github.com/ynput/ayon-core/blob/62870f239b6d137923adc1fd13d89077063cc830/client/ayon_core/hosts/houdini/plugins/publish/collect_local_render_instances.py#L67-L76
should match this logic
https://github.com/ynput/ayon-core/blob/62870f239b6d137923adc1fd13d89077063cc830/client/ayon_core/pipeline/farm/pyblish_functions.py#L591-L600


bump versions

updated dependencies, fixed errors related to it and added support for python 3.11.
pyaml < 6.0 is broken due to cython 3.0.0.
pyzmq changed namespace for constants
etc.

this PR needs to be merged for the tests to pass.


[GS][W-8486415] Add Resources Videography

Critical Changes
Changes
Add new resource 'Videography' to resources component
Issues Closed
Community Ideas Delivered
New Metadata
Deleted Metadata

W-8486415


webpack5 import error: Should not import the named export 'version'.'replace' (imported as 'version') from default-exporting module (only default export is available soon)

webpack5 import error: Should not import the named export 'version'.'replace' (imported as 'version') from default-exporting module (only default export is available soon)

fix:
import packageInfo from "../package.json";
const version = packageInfo.version;

Hey, thanks for the suggestion, we're trying to fix this for the next release of sandpack.


Update-DbaServiceAccount - Add service account to "Log on as a service"

Summary of new feature

Can you please add the SQL Service Account to Local Security Policy "Log on as a service" when using Update-DbaServiceAccount?

This is added to "Log on as a Service" sec pol when using Install-DBAInstance. The same behavior would be great.
Proposed technical details (if applicable)
My current work around it to use this module: https://gallery.technet.microsoft.com/scriptcenter/Grant-Revoke-Query-user-26e259b0

This would not go under Update-DbaServiceAccount. You would use Set-DbaPrivilege for this functionality.

TBF I seem to recall that changing via WMI took care of rights automatically....

Changing the service account does this yes. You can't update the permissions using WMI though.

Thank you for the updates/comments. @wsmelton Shawn via which method does changing the service account accomplish this?
when using DBATools to set the service account (Update-DbaServiceAccount) it does not set the SeServiceLogonRight privilege, however it sounds like it is not intended to.

We do not have a command that does it directly, but using SQLWMI you can set a service account for the SQL Server database engine. (Basically coded version of changing the service account in SSCM). This operation will set the permissions for the service account.
Having a command that does this would be something like Set-DbaService, which I do not believe has ever been requested. We do have a Get-DbaService.

As Set-DbaPrivilege can now set SeServiceLogonRight I will close this.


Update vuescan to 9.6.07

After making all changes to the cask:

[x] brew cask audit --download {{cask_file}} is error-free.
[x] brew cask style --fix {{cask_file}} left no offenses.
[x] The commit message includes the cask’s name and version.

I downloaded the file from the same link as before.

But the version on the server could have been compromised (it has happened before).

I also still have to old dmg

Not necessary. The VirusTotal verification is enough. Thank you.


GraphQL Shield - 2.0

#6

Hello @maticzav! Is this still valid? I'm still hoping we're going to be going the whitelisting way!


Can this advance webview can show/view local storage pdf,word,ppt etc files within app as of Ios native/default webview does.

Unfortunately, this is not supported by Android’s WebView implementation.
For PDF, you may be able to use pdf.js.


Error when serializing ElasticSearchEmbeddingRetriever from ElasticSearchDocumentStore with authentication

Dear developers,
I'm using an ElasticSearchDocumentStore with authentication like so:
document_store = ElasticsearchDocumentStore(hosts = "https://localhost:9200",
                                            index = INDEX,
                                           basic_auth=("elastic", ELASTIC_PASSWORD),
                                           ssl_assert_fingerprint=CERT_FINGERPRINT,)

and have an ElasticsearchEmbeddingRetriever in my pipeline.
After serializing my Pipeline (relevant part of .yaml file below) and trying to deploy it in hayhooks, I get an Error message about that Python tuple:

yaml.constructor.ConstructorError: could not determine a constructor for the tag 'tag:yaml.org,2002:python/tuple'
in "", line 53, column 23:
basic_auth: !!python/tuple

Is there any way to prevent this (other than using elastic without authentication)?

retriever:
init_parameters:
document_store:
init_parameters:
basic_auth: !!python/tuple
- elastic
- <ELASTIC_PW>
hosts: https://localhost:9200
type: haystack_integrations.document_stores.elasticsearch.document_store.ElasticsearchDocumentStore
filters: {}
num_candidates: null
top_k: 10
type: haystack_integrations.components.retrievers.elasticsearch.embedding_retriever.ElasticsearchEmbeddingRetriever

Thanks for the report. The linked PR should fix this issue once it gets released in a new Haystack build.


killed containers do not disappear from list

dry version 0.3-beta.9, build e0c03ae
Issue: killed containers do not disappear from list immediately, but show as exited when checked with an external tool.
To reproduce:

launch dry
K to kill a container
See text "killed container with id f00..."
container stays in the list of running containers for 30 seconds to 1 minute

I was expecting it to drop from the list like in 0.3-beta.6
Thanks.

Thanks for reporting.
Is the behavior of dry different to the Docker CLI for kill?
I guess that if you do  docker ps -a still shows the container f00..., right? Could you please try to kill the container using  docker kill?
The container list shown by dry should be the one reported by Docker. There was a change on 0.3-beta.6 to refresh the container list upon receiving events from the Docker daemon, which might explain the delay in updating.

Secondary issue:
Because it doesn't refresh, if I hit "kk", I get the following (because the container is either in the process of being deleted or doesn't exist):
Error killing container 1225a521d95f.  I "arrowed" up and down and waited about 45 seconds before I hit [F5] to refresh. Previously, I would kill containers in Docker UCP or a second terminal and dry seemed to update just fine.
From the [F2] screen that lists "non-running/killed" containers, if I do [CTRL-E] it says Removing all stopped containers. Then eventually it says Removed all stopped containers but this screen also acts the same way and doesn't refresh in a timely manner.
dry is talking to a remote Docker Swarm API. 0.3-beta.6 was working with the same swarm  just minutes before I upgraded.
I'm running OS X El Capitan version 10.11
info from dry:
Docker Host:         tcp://myhost:443                           Docker Version:  ucp/0.6.0
Cert Path:           /Users/me/pathto/tls    APIVersion:      1.21
Verify Certificate:  true                                                      OS/Arch/Kernel:  linux/amd64/

Thanks for the extended information :)
My initial guess is that it is related to the use of Docker Swarm, maybe it behaves differently and it is not sending events. But I need to take a look.

Definitely related to Docker Swarm, see: https://github.com/docker/swarm/issues/1203
As a quick fix, dry will now auto-refresh every 15 seconds. Available in v0.3-beta.10.
@funkytaco Could you please check if dry is working as before?

Yes, thanks. All good now.

Hmm, I did a CTRL-E for killed containers, and it somehow killed Docker UCP. I'm lucky this is a dev environment. dry has not done that before.
I'll open another ticket if it seems to be related to dry...

I've must've fat fingered something last time as Docker UCP seems fine this time when I tried to duplicate.


Fix publish.sh to use version from ot-java v0.31.0 branch

It's a wrong branch. we need it in v...

@pavolloffay Sorry, should be fine now.

By looking at it looks fine :+1:


Create curriculoMaykonTavares

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: maykonimpacta sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.


HOOTL Simulation

Pre-Req:
#6
On this ticket:

Build a way to run all ControlTasks in a Simulated Environment
Notably, the WebCam feed must be abstracted and simulated.
Create a flexible system to run "cases" in this Simulated Environment
Run one such HOOTL case in the CI Pipeline. If the MCL cannot complete 10x cycles without failing, the CI pipeline should fail.

Closing as outdated and duplicate, remaining work is captured in the #6 ticket


IsometricTiledMapRenderer amendment to accept a unit scale value for tile height

Currently, tiled maps in LibGdx only covers tiles that have the same width and length. In this case, the isometric renderer applies the same value for unit scale to a tile's height, which will then render the whole map vertically off-center. For applications that rely on screen coordinates matching to world coordinates, that match to an underlying structure such as a graph for pathfinding or a grid, this situation throws things out of whack. It forces users to restrict themselves to tilesets with tiles of the same width and height.
This simple modifications to the isometric renderers provides constructors to pass a unit scale for height, a separate field to reference it, and applies this value in the rendering process.

I'm not sure I understand what the problem is that this is trying to solve. Why are pixels needing to be scaled differently depending on which axis they are on? Can you give a concrete example where pixels should be scaled differently depending on x or y?

Hi, I knew I should have used something different other than X and Y. It should be tile width and height.
To summarize, this is to help make isometric maps draw centered on the screen, so that stuff like camera.unproject will match up to "locations" on the map.
There is only one unitScale field/value on all Tiled map renderers. It's applied in the rendering computation on both tile dimensions, and the tile map itself. This works well for maps using tiles with the same width and height e.g. 32 pixels wide by 32 pixels high.
However, in isometric maps using tile sets that have tiles with a different width from height e.g. 64x32, the map will be drawn vertically off-center, considering the camera position is at default, and the correct unit scale is passed for tile width.
For example, supplying 64 (or 1/64 in world coords) as the unit scale when instantiating an isometric tiled map renderer, will draw the map horizontally centered, and at the bottom of the screen. Since the iso map applies the same unit scale on the map dimensions itself, the rendered result is not quite in the correct scale, when compared to when viewed in Tiled.

I'm going to need you to supply a code example (and possibly some images) showing the actual problem this is solving. I copied your changes and for our test iso map (having a 64x32 grid) and used different unitScale and unitScaleY values and it surely did not have any desired effect.

Okay, may I have the link to which map you used to test so I can write the code? Did this map's tiles have different values for width and height? Grid dimensions are not relevant, if the tile's "sides" are equal.

You should more clearly state the problem conditions (or provide an actual example showing there is a problem, as I've been asking for).
The map I used is here: https://github.com/libgdx/libgdx/blob/master/tests/gdx-tests-android/assets/data/maps/tiled/isometric_grass_and_water.tmx
It does not have different tile image width and height, but I still don't understand how that matters (because you still have not clearly expressed what the actual problem is).

My sincere apologies. English is not my first language. I don't know how to say that a map made up off tiles having different width and height, is not vertically centered, and out of scale, any more clearer.
I used this map as well, and, the tiles that make up that map are each 64x32 (viewed in Tiled). Here's an image of how it projects on my application passing 1/64 for unit scale
(Isometric map centered horizontally, but not vertically.)
Ignore the small white box, it's just the mini-map. I cropped the image, but as can be seen, the map renders at the very bottom of the screen, since the 1/64 unit scale, is also applied to height of each tile, where it's supposed to be 1/32.
I only use these camera settings to render:
float pixelWidth = scene.getPixelWidth(); // tile dimensions queried from Tile map properties
float pixelHeight = scene.getPixelHeight();

OrthographicCamera camera = new OrthographicCamera();
camera.setToOrtho(false, scene.getWidth(), scene.getHeight()); // also pulled from the tile map properties
camera.position.set(scene.getWidth() * 0.5f, scene.getHeight() * 0.5f, 0);
ExtendViewport sceneViewport = new ExtendViewport(pixelWidth, pixelHeight * (Gdx.graphics.getHeight() / Gdx.graphics.getWidth()), camera);
sceneViewport.apply();

With the code amendments, passing 1/64 for unitScale, and 1/32 for unitScaleY should render a centered map, with correct scale. Conversely, just passing 1/64 will result in what is seen in the image.

I passed in 1/32f for the unitScaleY and it was nothing like it should be. The positioning has absolutely nothing to do with the unitScale.
So the actual issue is that you want to offset the map? Your code doesn't do that.

No, not offset. I want the map centered in the middle of screen, on both the x and y axes. With this in mind, shouldn't the unit scales for a 64x32 tile be 1/64 and 1/32, respectively? Isometric maps aside, here's my motivation. Since the renderers for "top-down" tile maps use one unit scale for both width and height, and have tiles with the same width and height, the rendered map is properly centered, a grid drawn starting from position (or "cell) 0, 0, matches perfectly to the tiles.
On the same train of thought, to achieve the same centered look with isometric maps (which is what I want) shouldn't matching values for unit scale be used, when the sides aren't the same? Or is the screenshot I provided the way things are supposed to be (in the bottom middle of the screen)? If it's not too much to ask, may I ask for a screenshot of how the map looked like on your test? (sorry for the bother)
Thank you.

Unit scale only exists to turn the pixel units of tmx maps into camera units. It is not tied to tiles unless you choose to make it so. In the case of a 64 x 32 tile (or grid) size, you may choose to make the width of one tile equal to one camera unit (1/64f) or the height of one tile (1/32f) or perhaps the width of two tiles is 1 unit (1/128f). Maybe you want the height of one tile to be equal to two camera units (1/16f). It is entirely up to you, it is just convenient to make one tile equal to one camera unit.
To vertically center the map, you need to understand where the drawing origin is, and the direction drawing goes from there:
The bottom left of the left most cell is at (0,0). Moving along the width of the map, each cell shifts downward by half the tile height and rightward by half the tile width. Moving along the height of the map, each cell shifts upward by half the tile hight and rightward by half the tile height.

In this screenshot, I've illustrated how this looks - the red box simply starts at 0,0 and extends the size of the camera viewport. The green box shows the map bounds. For further clarification I've annotated the origin as well as the row and column axes.
Knowing this, it should be relatively easy to translate the camera so that the map is centered within the viewport.

Thank you for the detailed explanation, I really appreciate it :)
I apologize that I didn't create this pull request with the adequate code and screenshots to display what I'm seeing. I still felt that there is a need for this code submission, so I started a new project to boil down to what's been bothering me. As a result, I did find something to consider.
When I'm ready to explain further, should I pastebin the test project, or should I just put it here? I'll have it up within the day since I'm not on my home PC.
Thanks for all your help!

I can't imagine when you'd want to scale the y pixels differently than the x pixels (except maybe for a temporary visual effect), but I'm all ears. Go ahead and post a small example here if you have one.
In the meantime I'm going to close this and reopen if necessary.

Only reason why I can think someone would want taller Y scale than X is if they want to make a game with the visual style of Ultima

@nhydock On my part, I just want normal rendering. Thanks.
So I've stripped my rendering code to bare essentials, using only a camera, and no view port. Sorry if something's wrong in there.
Here's my main method:
public class DesktopLauncher {
public static void main (String[] arg) {
	LwjglApplicationConfiguration config = new LwjglApplicationConfiguration();
	config.width = 1920;
	config.height = 1080;
	new LwjglApplication(new Launch(), config);
}

}
...and my ApplicationListener
public class Launch implements ApplicationListener {

private IsometricTiledMapRenderer renderer;
private OrthographicCamera camera;
private SpriteBatch batch;
private float w;
private float h;

@Override
public void create() {
	batch = new SpriteBatch();
	AssetManager assetManager = new AssetManager();
// I had some weird issue w/ the Android project after using the setup app, so I loaded the tmx from the classpath
	assetManager.setLoader(TiledMap.class, ".tmx", new TmxMapLoader(new ClasspathFileHandleResolver()));
	assetManager.load("com/testbed/isometric_grass_and_water.tmx", TiledMap.class);
	assetManager.finishLoading();
	TiledMap tiledMap = assetManager.get("com/testbed/isometric_grass_and_water.tmx", TiledMap.class);

	w = (int) tiledMap.getProperties().get("width");
	h = (int) tiledMap.getProperties().get("height");
	camera = new OrthographicCamera();
	camera.setToOrtho(false, w, h);
	//renderer = new IsometricTiledMapRenderer(tiledMap); // no unit scale
	//renderer = new IsometricTiledMapRenderer(tiledMap, 1/64f); // width unit scale only
	renderer = new IsometricTiledMapRenderer(tiledMap, 1/64f, 1/32f); // my amendment
}

public void resize(int width, int height) {
}

@Override
public void render() {
	Gdx.gl.glClearColor(0, 0, 0, 1);
	Gdx.gl.glClear(GL20.GL_COLOR_BUFFER_BIT | GL20.GL_DEPTH_BUFFER_BIT);

	camera.update();
	batch.setProjectionMatrix(camera.combined);
	batch.begin();
	renderer.setView(camera.combined, 0, 0, w, h);
	renderer.render();
	batch.end();
}

@Override
public void pause() {
}

@Override
public void resume() {
}

@Override
public void dispose() {
}
}

Using this, I tried 3 different scenarios: 1. without passing a unit scale, 2. using the same unit scale for both width and height, and 3. using different values for unit scale. The 1st 2 are the default constructors LibGDX uses, of course.
Here are screen shots for the tests:
Default constructor, w/o unit scale.

Default constructor, w/ unit scale (matching 1/64 for tile width)

Submitted code, w/ unit scales matching width and height (1/64 and 1/32 respectively)

As can be seen, the 1st two images shows the map being drawn weird. The 1st understandably because the camera is in world units, but I believe the second image is not isometric, whereas, the 3rd image is.
Thanks.

That code is very wrong.

Okay, I'm more than willing to admit I'm only going by my limited knowledge. My code could also be cleaner (sorry I just pasted it here when I was able to simulate the problem). Here's the cleaner version:
package com.testbed;

import com.badlogic.gdx.ApplicationListener;
import com.badlogic.gdx.Gdx;
import com.badlogic.gdx.assets.AssetManager;
import com.badlogic.gdx.assets.loaders.resolvers.ClasspathFileHandleResolver;
import com.badlogic.gdx.graphics.GL20;
import com.badlogic.gdx.graphics.OrthographicCamera;
import com.badlogic.gdx.graphics.g2d.SpriteBatch;
import com.badlogic.gdx.maps.tiled.TiledMap;
import com.badlogic.gdx.maps.tiled.TmxMapLoader;
import com.badlogic.gdx.maps.tiled.renderers.IsometricTiledMapRenderer;

public class Launch implements ApplicationListener {

private IsometricTiledMapRenderer renderer;
private OrthographicCamera camera;
private SpriteBatch batch;
//	private ExtendViewport sceneViewport;
private float w;
private float h;

@Override
public void create() {
	batch = new SpriteBatch();
	AssetManager assetManager = new AssetManager();
	assetManager.setLoader(TiledMap.class, ".tmx", new TmxMapLoader(new ClasspathFileHandleResolver()));
	assetManager.load("com/testbed/isometric_grass_and_water.tmx", TiledMap.class);
	assetManager.finishLoading();
	TiledMap tiledMap = assetManager.get("com/testbed/isometric_grass_and_water.tmx", TiledMap.class);

	w = (int) tiledMap.getProperties().get("width");
	h = (int) tiledMap.getProperties().get("height");
	camera = new OrthographicCamera();
	camera.setToOrtho(false, w, h);
	camera.translate(0, -h * 0.5f); // just to see things clearer
	camera.update();

//		renderer = new IsometricTiledMapRenderer(tiledMap);
//		renderer = new IsometricTiledMapRenderer(tiledMap, 1/64f);
	renderer = new IsometricTiledMapRenderer(tiledMap, 1/64f, 1/32f);
}

public void resize(int width, int height) {
}

@Override
public void render() {
	Gdx.gl.glClearColor(0, 0, 0, 1);
	Gdx.gl.glClear(GL20.GL_COLOR_BUFFER_BIT | GL20.GL_DEPTH_BUFFER_BIT);

	renderer.setView(camera.combined, 0, 0, w, h);
	renderer.render();

}

@Override
public void pause() {
}

@Override
public void resume() {
	// TODO Auto-generated method stub

}

@Override
public void dispose() {
	// TODO Auto-generated method stub

}
}

As for the intent of the test, I thought it was as vanilla as could be, in terms of  in order to draw a tile map and have things in world coordinates. If you would be so kind, what is the correct way then to do this?

You are setting your camera viewport without any regard for the aspect ratio of the window, which will cause undesired scaling in one direction. Also, why are you using that version of setView?

As shown on an earlier post, I am using an ExtendViewport set to the aspect ratio in my app, but I assumed (wrongly, it seems) that a vanilla test is what's merited for testing, so I just used a camera.
I have no excuses for using this setView version, other than I was playing around with finding how to center the map, without translating the camera. I should have cleaned that up, sorry. I did go over material online and do know that doing "setView(camera)" suffices for the test code.

Let's move to the forum for additional support on this subject. The comments section here isn't the correct place for support requests.
We won't merge this change as it doesn't actually solve a real problem.

I have no problem with that. Thanks @MobiDevelop for the extra effort :)
PS I've changed the images above to ones that show the map centered. It will be more obvious for people to see what I mean i.e. same unit scale = skewed map, different scale = correct isometric view

Sorry for the double post, I forgot to ask, isn't the fact that the map is skewed (see image above), a real problem? Is the 2nd image above really how an isometric map should look in LibGDX?

It is skewed because you are setting up your viewport wrong. Please use the forum instead of more comments here.

sorry, got caught up in RL. posted to this thread..
Thanks.

@MobiDevelop could you tell why using this version of setView() is bad? According to LibGDX wiki there is no statement about when to use which version.
Sorry for posting here, but you told @adam-law to to forum, but no one ever responded, and I really couldn't find anywhere else info on that topic - so it at least could be a complete information for possible future readers.

Using either version of setView isn't wrong, unless you use them wrong. Pretending that units are different in the x and y direction is what @adam-law was doing wrong.

You know, I'm always the kind of guy who'll be the first to admit a mistake, if there was one. You keep telling me I'm wrong, and that's fine and all, but, with all due respect, what then is the correct way to set the view, given the information I've already provided?
Also, respectfully, I don't get why you have to call my attempts at finding a solution as "pretending". I get that you are a major contributor to LibGDX, and I'm also aware it was you who authored this part of the code. For all intents and purposes, you're obviously way better in this field than I am, but was I ever disrespectful to you to merit your condescension? People have to begin learning from somewhere, you know. I apologize if I'm not as good as you are, and I understand how providing technical knowledge and support to the masses can get repetitive and tiring, but aren't you laying it on a bit thick? In the first place, this wasn't even a question to start with, but a contribution to the code base, regardless of how small and insignificant it may be.
Anyway, this pull request displays the correct view for my use case, regardless of the critiquing it has received, whereas using the available constructors does not. That still remains the issue that needs to be solved. If it's really caused by setting the view wrong, again, then what's the correct way? The test code I supplied seems to be the simplest case for setting the view as it is, but what do I know, right?
I've posted to the forum as you've asked, but like @plunntic mentioned, no one answered, even after all this time, so what was the point? Did I somehow use the wrong format there as well? From experience, it seems it's either a hit or miss getting help there, ever since I started learning LibGDX, and I don't think I'm the only one who feels this way.

I will be posting an correct example for this in the forum post above, please redirect any more questions there.

Hello!
You have a new message, please read http://suntech7.com/traffic.php?w
<EMAIL_ADDRESS>

Hello!
You have a new message, please read http://dotmybusiness.com/catholic.php?cy
<EMAIL_ADDRESS>


Added new marker capabilities.

Now a marker can have a label and an infowindow that pops up whenever the marker is clicked. The infowindow data has to be passed as a dict.

Some of the added things were made by you but maybe aren't in the github, i used the .py that gets downloaded using pip. Also i changed a line on the init to make the coloricon compatible on windows.

Will close this as a duplicate of #119, since that PR allows users to pass in any HTML content for the InfoWindow, instead of a fixed list of key-value pairs.


Istio Config: Handle uniform parsing of ptype.struct

This is required for istio config
golang proto.jsonpb does not yet support Unmarshaling  proto.Any and ptype.Struct
Other languages (python / Java / C++) do not have this problem.
https://github.com/gogo/protobuf/issues/197
https://github.com/golang/protobuf/issues/216
https://github.com/golang/protobuf/pull/208
It can be worked around by using standard go json or yaml for decoders.

This is resolved by pr #189


[GitHub releases] Preinstall Kit for Win11 has wromg name

The preinstall kit for Windows 11 is called Microsoft.WindowsTerminal_Win11_..._Windows10_PreinstallKit.zip.
But it must be Microsoft.WindowsTerminal_Win11_..._Windows11_PreinstallKit.zip.
This is a bit confusing.

Huh. That's an interesting artifact of the build system. I wonder if we could stamp "Windows11" into the win11 preinstall kit pipeline.
I'm sure it's entirely aesthetic, and the actual filename has no bearing on the functionality. But I bet it could be done.


Config: Use shorthand getters and setters

Set default values in empty constructor

Don't you wish we could just use auto-property initializers? :)
Anyways, looks good to me!

Would've been very convenient in this case :)


Implement the PSO-SVM algorithm

https://www.sciencedirect.com/science/article/pii/S0022169422010332
https://link.springer.com/article/10.1007/s00366-021-01299-6

The algorithms listed are actually different and just hyperparameter optimization and feature selection respectively.


PolicySet details test

Signed-off-by: Zack Layne<EMAIL_ADDRESS>Related issue: https://github.com/stolostron/backlog/issues/17916

/cc @RobDolares


Investigate Community Test Suite Issues

Failing Test Runs
As of February 11th, it seems like community test runs have been failing in strange ways

Test 3
##[error]Unhandled exception. System.IO.IOException: No space left on device
System.IO.IOException: No space left on device
   at System.IO.FileStream.WriteNative(ReadOnlySpan`1 source)
   at System.IO.FileStream.FlushWriteBuffer()
   at Microsoft.VisualStudio.Services.Agent.HostContext.Dispose(Boolean disposing)
   at Microsoft.VisualStudio.Services.Agent.HostContext.Dispose()
   at Microsoft.VisualStudio.Services.Agent.Worker.Program.Main(String[] args)
   at System.IO.FileStream.WriteNative(ReadOnlySpan`1 source)
   at System.IO.FileStream.FlushWriteBuffer()
   at System.IO.FileStream.Flush(Boolean flushToDisk)
   at System.IO.StreamWriter.Flush(Boolean flushStream, Boolean flushEncoder)
   at System.Diagnostics.TextWriterTraceListener.Flush()
   at Microsoft.VisualStudio.Services.Agent.HostTraceListener.WriteHeader(String source, TraceEventType eventType, Int32 id)
   at Microsoft.VisualStudio.Services.Agent.HostTraceListener.TraceEvent(TraceEventCache eventCache, String source, TraceEventType eventType, Int32 id, String message)
   at System.Diagnostics.TraceSource.TraceEvent(TraceEventType eventType, Int32 id, String message)
   at Microsoft.VisualStudio.Services.Agent.Worker.Worker.RunAsync(String pipeIn, String pipeOut)
   at Microsoft.VisualStudio.Services.Agent.Worker.Program.MainAsync(IHostContext context, String[] args)
System.IO.IOException: No space left on device
   at System.IO.FileStream.WriteNative(ReadOnlySpan`1 source)
   at System.IO.FileStream.FlushWriteBuffer()
   at System.IO.FileStream.Flush(Boolean flushToDisk)
   at System.IO.StreamWriter.Flush(Boolean flushStream, Boolean flushEncoder)
   at System.Diagnostics.TextWriterTraceListener.Flush()
   at Microsoft.VisualStudio.Services.Agent.HostTraceListener.WriteHeader(String source, TraceEventType eventType, Int32 id)
   at Microsoft.VisualStudio.Services.Agent.HostTraceListener.TraceEvent(TraceEventCache eventCache, String source, TraceEventType eventType, Int32 id, String message)
   at System.Diagnostics.TraceSource.TraceEvent(TraceEventType eventType, Int32 id, String message)
   at Microsoft.VisualStudio.Services.Agent.Tracing.Error(Exception exception)
   at Microsoft.VisualStudio.Services.Agent.Worker.Program.MainAsync(IHostContext context, String[] args)

Test 4
,##[error]We stopped hearing from agent Azure Pipelines 18. Verify the agent machine is running and has a healthy network connection. Anything that terminates an agent process, starves it for CPU, or blocks its network access can cause this error. For more information, see: https://go.microsoft.com/fwlink/?linkid=846610

Fluent UI?
It seems like in recent pull requests (last 30ish days?), we're not seeing Office UI Fabric getting mentioned in our community test suite runs: https://github.com/typescript-bot/TypeScript/pull/
Is the community test suite still testing Fluent UI (formerly Office UI Fabric)? There might be issues because the repo was moved to https://github.com/microsoft/fluentui, but it might be due to the above issue.
We should make sure we haven't lost coverage.

So, an update here, the docker image we built to test office ui fabric is 9.44GB, the image we build ourselves in is 1.5GB. That puts the floor on space required at as 11GB. Azure devops hosted workers only guarantee "at least 10 GB of storage space" in the worker, hence you can see why we'd run into this issue eventually. After realizing that, my first thought was to port the task to actions - Github actions is a bit different - there's no stated limit, but heresay is that actions are run on a disk with ~29GB of free space by default, but may actually be limited to a similar ~14GB cap (it's running on the same underlying node, which only has a 14GB temp disk size). So even if migrating fixed the issue, it'd likely only so do temporarily, as the build/tests are likely to keep growing in size - so instead I started to look into purging the images between builds, but in doing so I had to make the build actually get to the built image stage again (they changed their build process a little bit and had broken our process) - in doing so, I made a change to how we inject ourselves, and, in doing so, on my local machine, halved the image size (which should buy way more than a little time). So... that alone should get it working, and does: #42961


How to integrate with flask or logging?

It no effect when exception raised in flask request because the tracebacks are formatted by logging, the sys.excepthook is not called.
eg:
import tbvaccine
from flask import Flask

tbvaccine.add_hook(isolate=True)
app = Flask(__name__)

@app.route('/')
def index():
    return 1 / 0

if __name__ == '__main__':
    app.run()

the outputs:
 * Running on http://<IP_ADDRESS>:5000/ (Press CTRL+C to quit)
[2017-06-01 21:59:08,086] ERROR in app: Exception on / [GET]
Traceback (most recent call last):
  File "/home/guyskk/.local/share/virtualenvs/next_passport-OfYZSIo0/lib/python3.6/site-packages/flask/app.py", line 1982, in wsgi_app
    response = self.full_dispatch_request()
  File "/home/guyskk/.local/share/virtualenvs/next_passport-OfYZSIo0/lib/python3.6/site-packages/flask/app.py", line 1614, in full_dispatch_request
    rv = self.handle_user_exception(e)
  File "/home/guyskk/.local/share/virtualenvs/next_passport-OfYZSIo0/lib/python3.6/site-packages/flask/app.py", line 1517, in handle_user_exception
    reraise(exc_type, exc_value, tb)
  File "/home/guyskk/.local/share/virtualenvs/next_passport-OfYZSIo0/lib/python3.6/site-packages/flask/_compat.py", line 33, in reraise
    raise value
  File "/home/guyskk/.local/share/virtualenvs/next_passport-OfYZSIo0/lib/python3.6/site-packages/flask/app.py", line 1612, in full_dispatch_request
    rv = self.dispatch_request()
  File "/home/guyskk/.local/share/virtualenvs/next_passport-OfYZSIo0/lib/python3.6/site-packages/flask/app.py", line 1598, in dispatch_request
    return self.view_functions[rule.endpoint](**req.view_args)
  File "app.py", line 10, in index
    return 1 / 0
ZeroDivisionError: division by zero
<IP_ADDRESS> - - [01/Jun/2017 21:59:08] "GET / HTTP/1.1" 500 -

Logging integration:
class TbVaccineFormatter(logging.Formatter):
    def  formatException(self, exc_info):
        return TBVaccine(isolate=True).format_exc()

sh = logging.StreamHandler()
sh.setFormatter(TbVaccineFormatter('[%(levelname)s] %(asctime)s : %(message)s', '%Y-%m-%d %H:%M:%S'))
logger.addHandler(sh)

Thank you, @afeblot. Would you mind issuing a PR to add that to the README? It's very useful.


Implement CEL for DRCluster and remove webhook

closing this PR and opened a new one: https://github.com/RamenDR/ramen/pull/1245


Value not update when the number greater than 1000 only on mac safari

Value not update when the number greater than 1000 only on mac safari

The actual issue was change even didn't fired when value length grater than 3. Please check latest version. I have fix the issue.

@gppack Give me a minimal repo where you are trying to use.


[3.x] Fix incorrect usage for some export variables

Fixes #59157
PROPERTY_USAGE_SCRIPT_VARIABLE was not added when only a type hint exists and the type is non-Object.
export var foo := 42 # this works
export var foo: int  # not added for this
export var foo: int = 42 # not added, as long as the type is from the explicit type hint

An alternative way to fix this is don't bother adding that usage in the parser and always add it here:
https://github.com/godotengine/godot/blob/036f36b5f179b47feac24f065e3033afedd608f9/modules/gdscript/gdscript_compiler.cpp#L1961
That's also how master adds the usage.
I chose the current whitelist-like approach because I'm not sure if using the GDScript 2.0 approach will introduce any regression on 3.x.

Thanks!


Fix fetching certificates

According to Puppet's documentation, an "environment" parameter is
required (and is unused) when the Puppet master runs from Rack or
WEBrick.
https://docs.puppet.com/puppet/5.0/http_api/http_certificate_request.html#find
Add this environment at two places where it is missing.
While here, fix headers sent when fetching the CA.
These problems where discovered at the First User stage of Choria documentation:
% mco choria request_cert
Encountered a critical error: Failed to fetch CA from puppet:8140: 400: Bad Request
% # Some hacking to discover the missing "?environment=production"
% mco choria request_cert
Encountered a critical error: Failed to fetch CA from puppet:8140: 406: Not Acceptable
% # Some hacking to discover the Content-Type / Accept confusion

Thanks. Never tried this on webrick :)
Mind openings issue also and updating the commit here referencing it as per git log?

It's actually Running on Rack on FreeBSD… More PR to come :wink:
Just opened #322 and updated my commit!


add back in rpm -V for java rpms since now on 7.6 rhel

See https://jira.coreos.com/browse/ART-391
/assign @bparees
/assign @sosiouxme

install stutter removed

/lgtm
probably won't get through testing, but i'd at least like to see that the dockerfile can get successfully built.

rpm -V does not like wildcards like rh-maven33* though yum tolerates them:  package rh-maven33* is not installed

the rpm -V on the javas are working again

rpm -V does not like wildcards like rh-maven33* though yum tolerates them: package rh-maven33* is not installed

ok, scrap it as needed.

/lgtm

assuming unit tests passes I say we green button @bparees

yeah that was my plan.

/cherrypick master

/cherrypick openshift-3.10


[FEATURE] Add documentation on how to get to clean state

Description
Currently, if the SOCI snapshotter is killed and restarted, it can end up in one of several broken states (e.g. due to invalid credentials, missing FUSE mounts, missing snapshots, etc). Just restarting the snapshotter usually immediately exits with errors. There should be documentation to explain how to get SOCI back into a good state and understanding what the bad state even is.
Describe the solution you'd like
We should update either the getting started guide or the debugging guide with information about those errors, how to fix them gracefully (with allow_invalid_snapshots + removing the snapshots in containerd), and how to fix them forcefully (blowing away /var/lib/containered and /var/lib/soci-snapshotter-grpc and restarting containerd).
Describe any alternative solutions/features you've considered
No response
Any additional context or information about the feature request
No response

Fixed by #835.


Compatibility with Cluster Autoscaler

Add safe-to-evict annotation to instance-manager-e pods
Set minAvailable to 0 for instance-manager-e pods

Signed-off-by: Andrew Hemming<EMAIL_ADDRESS>

This PR is a more comprehensive alternative to https://github.com/longhorn/longhorn-manager/pull/823, adding full compatibility with CA, without requiring the use of a mutating webhook such as kube-annotate

This PR is a more comprehensive alternative to https://github.com/longhorn/longhorn-manager/pull/823, adding full compatibility with CA, without requiring the use of a mutating webhook such as kube-annotate

Similar to https://github.com/longhorn/longhorn-manager/pull/823 . This PR will break the Longhorn support for Kubernetes drain logic by allowing Kubernetes to drain Longhorn instance manager pod with the engine running on it, which will result in the crash of workload pod that is using the volume.

Sounds like the instance manager pod needs to exit gracefully only after any workloads using the volume(s) have exited. Is a check like that possible?
If no workloads are consuming volumes (which will often be the case since the manager runs on every node and not every node will have workloads consuming LH volumes), then there should be nothing additional to do.

@yasker do you have any recommendation for fixing Issue 2203?
This is a pretty big blocker for those of us wanting to use longhorn with a dynamic autoscaling cluster. We already plan on dedicating data nodes for our longhorn deployment, however that doesn't fix the issue of having these e-managers scheduled onto our scaling worker node group.
The only solution/workaround is to taint our scaling worker node group so that the e-manager's can't run on these nodes. This isn't ideal as now we have to add tolerations to all our workloads we have scaling.
If we can't fix the compatibility with cluster-autoscaler quickly, can we provide a mechanism to enforce node scheduling of the e-manger's using nodeAffinity/nodeSelector instead of a taint on all nodes we don't want them deployed to?

Any update ?


fix: change to official name

change to official name (k8s -> Kubernetes)

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

#label:"aq-pr-triaged"

@08thse, this is a good change for consistency in our docset, thanks for the contribution!
#sign-off


Prod build fails with TypeError: Reduce of empty array with no initial value

I don't have any empty arrays anywhere in my ts modules that I can find.
`> webpack --config webpack.prod.config.js --progress --profile --colors --display-error-details --display-cached
532ms build modules
1ms seal
1ms optimize
75% hashingUnhandled rejection TypeError: Reduce of empty array with no initial value
at Array.reduce (native)
at Compilation. (/Users/JARVIS/src/project7/node_modules/webpack-md5-hash/plugin/webpack_md5_hash.js:27:82)
at Compilation.applyPlugins (/Users/JARVIS/src/project7/node_modules/tapable/lib/Tapable.js:26:37)
at Compilation.createHash (/Users/JARVIS/src/project7/node_modules/webpack/lib/Compilation.js:791:8)
at Compilation. (/Users/JARVIS/src/project7/node_modules/webpack/lib/Compilation.js:554:8)
at Compilation.applyPluginsAsync (/Users/JARVIS/src/project7/node_modules/tapable/lib/Tapable.js:60:69)
at Compilation.seal (/Users/JARVIS/src/project7/node_modules/webpack/lib/Compilation.js:525:7)
at Compiler. (/Users/JARVIS/src/project7/node_modules/webpack/lib/Compiler.js:397:15)
at /Users/JARVIS/src/project7/node_modules/tapable/lib/Tapable.js:103:11
at /Users/JARVIS/src/project7/node_modules/html-webpack-plugin/index.js:50:9
at tryCatcher (/Users/JARVIS/src/project7/node_modules/html-webpack-plugin/node_modules/bluebird/js/release/util.js:16:23)
at Promise._settlePromiseFromHandler (/Users/JARVIS/src/project7/node_modules/html-webpack-plugin/node_modules/bluebird/js/release/promise.js:503:31)
at Promise._settlePromise (/Users/JARVIS/src/project7/node_modules/html-webpack-plugin/node_modules/bluebird/js/release/promise.js:560:18)
at Promise._settlePromise0 (/Users/JARVIS/src/project7/node_modules/html-webpack-plugin/node_modules/bluebird/js/release/promise.js:605:10)
at Promise._settlePromises (/Users/JARVIS/src/project7/node_modules/html-webpack-plugin/node_modules/bluebird/js/release/promise.js:684:18)
at Async._drainQueue (/Users/JARVIS/src/project7/node_modules/html-webpack-plugin/node_modules/bluebird/js/release/async.js:126:16)
at Async._drainQueues (/Users/JARVIS/src/project7/node_modules/html-webpack-plugin/node_modules/bluebird/js/release/async.js:136:10)
at Immediate.Async.drainQueues [as _onImmediate] (/Users/JARVIS/src/project7/node_modules/html-webpack-plugin/node_modules/bluebird/js/release/async.js:16:14)
at processImmediate [as _immediateCallback] (timers.js:383:17)`

@jkyoutsey It seems, this issue is already discussed in https://github.com/AngularClass/angular2-webpack-starter/issues/215#issuecomment-173515930
There was some issue with the webpack-md5-hash when there are some empty files. Refer : https://github.com/erm0l0v/webpack-md5-hash/issues/1
This was already fixed with the latest version of webpack-md5-hash
Can you try updating webpack-md5-hash with the latest version and see if this issue still exists?

Agreed.  I upgraded somewhere along the way and forgot I put this out there.


After you sync a team, confirmation message should remind that the people added still need to accept the invites

...lest it appear broken when you go the website to check if all those people were in fact added :)

Added a message for this,
console.log(`Members who don't belong to the target team have been invited. They should ${chalk.bold('accept')} their invites to have repo access.`);


need a way to minimize customization changes on git pull operation

It will be very helpful if there will be a way to customize test generator sources such a way that pull requests won't effect these customizations. For ex riscv_core_setting.sv file is under the git control and if changed locally may cause pull problems.
Also any local updates to the source cause the same ...

Can you take a look at how it's done for ibex verification?
https://github.com/lowRISC/ibex/blob/master/dv/uvm/Makefile#L78
All the local changes are done in separate files so that pull in upstream changes won't have problem in most cases
https://github.com/lowRISC/ibex/tree/master/dv/uvm/riscv_dv_extension


[Exposing ALTS Context 2/2] Utility Wrapper Class

This PR serves as the same purpose as this reverted one:
https://github.com/grpc/grpc/pull/21196,
but we adopt another approach for providing utilities to get AltsContext.
In that PR, we use upb to serialize the AltsContext in TSI layer, pass it up to the C++ user-facing APIs, and use standard pb to de-serialize it. Here we use upb for serializing and de-serializing, and provide a utility wrapper class for  the class generated from upb.
This change is based on the assumption that upb generated class is not user-friendly enough and we'd better provide more straightforward ways for users to use ALTS context.

@jiangtaoli2016 @yihuazhang Could you please take a look at this PR and see if there is anything that could be improved?


setup automations to publish and refresh the service libraries - CSharp

For the v1 and beta graph service libraries, we're currently relying on the generator pipeline to refresh the metadata (CSDL), kick off the generation process and commit changes to the target repo. The pipeline needs to be updated to also refresh the OpenApi descriptions, and use kiota instead of typewriter for the source generation.
AB#10344

closing as this was done a few weeks ago by @andrueastman


Java+MapDB implementation may be feasible

Contrary to initial impressions, it turns out that with TreeMaps and compression enabled, a MapDB disked-backed DB gives quite reasonable sizes and speeds when storing large amounts of OSM data.
When we get into applying updates and other complicated things, it's not worth the maintenance effort on a C storage backend code if we have a nice Java library that is within say 70% performance-wise.
For comparison:
New York PBF (130MB) loaded into MapDB:

TreeMap is 306MB.
HashMap is much bigger at 626MB.

Netherlands PBF (size 941MB):

Takes 7 minutes to load into TempFileDB and the files are 2.2GB.
Size is 2.3x PBF, time is 7.4 min/GB.
Using MemoryDirectDB, NL PBF takes about 4 minutes to load.
Outputting VEX format takes 3min 22sec.

Extrapolating to planet.pbf at 25.2GB:

should consume 60GB in MapDB
187 minutes loading into MapDB (on HDD)
But we still need the spatial index. That is a navigable map from Slippy map tiles (x,y) to a list of every way in the tile.
There are only 272 Mways in OSM. These will fit neatly in a hashmap as a spatial index.
Index scanning can be done with subMap(Fun.t2(x0, y0), Fun.t2(x1, y1)

On SSD load time of Great Britain (PBF 665MB) is 3 minutes. Writing VEX format takes 1:40.
MapDB file is 1.5G, so again about 2.2x the PBF size, 4.5 minutes per GB of PBF.
Extrapolating to Planet at 25.2GB PBF:

60GB, 113 minutes to load

Twitter exchange in which MapDB author Jan Kotek reports on loading OSM:
https://twitter.com/n_colomer/status/433343722625826817
Recent discussion on loading OSM into MapDB:
https://groups.google.com/forum/#!topic/mapdb/EaU4vV7Gyhk
And a mysterious empty Github repository, intended to use MapDB as an Osmosis data store:
https://github.com/komoot/osmosis-mapdb

Loading the entire planet.pbf on dev.opentripplanner.org to an SSD:

26GB PBF loaded into MapDB in just under 2 hours
Wrote out VEX in 1 hour 17 minutes, size 17GB (but metadata was stripped)

A MapDB based Java implementation now exists within the OpenTripPlanner repo. It is intended to become an external OSM loading library but will remain within OTP until it becomes stable to avoid constantly updating dependencies in both directions.
https://github.com/opentripplanner/OpenTripPlanner/blob/master/src/main/java/org/opentripplanner/osm/VanillaExtract.java

This implementation has been moved to https://github.com/conveyal/osm-lib


Unable to select multiple transaction and use "Edit"

I get the Uh-oh when I click edit at the top menu if more than one transaction is selected.  Using Chrome, never happened before.

I am also having this problem right now.

Same Issue today as well.

same

Not again.  This is getting frustrating.

ditto

Having the same issue today

Can confirm same. Getting in Chrome on both Mac and Win 11. Both OS's and browsers using latest updates.

+1

FYI, same as above.

+1

Experiencing same issue on chrome myself.

+1

Hey @toddaitkins  the fix for this should be in our current release! If you notice the issue still exists and you're definitely running the newest version of the toolkit, please reopen! Thanks!
Please note, all browser can take up to an hour to provide updates to all users. You can force an update if it's available for you by going to chrome://extensions and clicking the "Update" button. Firefox users can go to about:addons, click the setting cog, and click "Check for updates...".


Sending matter sensor data to rainmaker (MEGH-6063)

Answers checklist.

[X] I have read the Rainmaker documentation and the issue is not addressed there.
[X] I have updated my IDF branch (release/vX.Y) to the latest version and checked that the issue is present there. This is not applicable if you are using Rainmaker with Arduino.
[X] I have searched the Rainmaker forum and issue tracker for a similar issue and not found a similar issue.

General issue report
I have flashed the matter controller + rainmaker example with otbr feature to an espressif thread border router and was able to successfully commission a matter door sensor and mattet plug to rainmaker. How can I send the states of those devices to the rainmaker dashboard or aws iot core. Currently one device is not showing any parameters which is the door lock sensor and the matter plug can be turned on and off with rainmaker but those two devices are not shown in rainmaker dashboard.

The ESP RainMaker dashboard is an admin side device management dashboard meant for devices directly connected to the RainMaker backend, which in this case, is the controller. The devices connected to the controller are pure matter devices, not relevant to admin users. They are accessible only to the end user via the phone app. Moreover, some device types are not yet supported, either by the controller, or the phone app. So, even if they may be controllable by API, they may not show correctly in the app.

Is there any possible way to send the data of the sensor to aws iot core via the matter controller

You can check the supported device type by the controller here https://github.com/espressif/esp-rainmaker/blob/f98cf1ec50bff6706c5afe626806fe9d95dbc141/examples/matter/matter_controller_with_touchscreen/main/app_matter_ctrl.cpp#L399
Currently door lock sensor is not supported by the controller and phone app.
Assuming that door lock sensor is supported in the controller. As a proof of concept you can your logic for sending on-off data of light to aws iot core by adding your logic here https://github.com/espressif/esp-rainmaker/blob/f98cf1ec50bff6706c5afe626806fe9d95dbc141/examples/matter/matter_controller_with_touchscreen/main/app_matter_ctrl.cpp#L120 .
We report the updated state of light (A supported device type by controller) .
To check and verify the state. You can run the following commands.
$ cd $RMAKER_PATH/
$ esp-rainmaker-cli login --email <rainmaker-account-email>
$ esp-rainmaker-cli  getparams <rainmaker-node-id-of-controller>

On-Off cluster is 0x06.

Thank you so much. Could you please tell me how to change the startup logo from espressif to a custom logo. I have seen a file in the location matter controller with touchscreen in the file esp_logo.c. But I can see the image is converted into a hex format. How can I convert my custom logo like that. Also what should be the resolution and size of the logo?

@aadhimellow For conversion of image please refer here use the lvgl v8 tool.

After commissioning the matter controller to rainmaker. I can see the current time in the s3box. How can I fetch that time and print it in the log.
After provisioning i have seen the date time details in the monitor under the esp log tag called esp_rmaker_time but couldnt find the function that does it


[Model Update]: return request_v2.0.0

Description
As mentioned in issue: https://github.com/eclipse-tractusx/sldt-semantic-models/issues/313.
-->
Closes #

MS2 Criteria
(to be filled out by PR reviewer)

[x] the model validates with the SAMM SDS SDK in the version specified in the Readme.md of this repository by the time of the MS2 check  (e.g., 'java -jar samm-cli.jar aspect <path-to-aspect-model> validate ). The  SAMM CLI is available here and in GitHub
[x] use Camel-Case (e.g., "MyModelElement" or "TimeDifferenceGmtId", when in doubt follow https://google.github.io/styleguide/javaguide.html#s5.3-camel-case)
[x] the identifiers for all model elements start with a capital letter except for properties
[x] the identifier for properties starts with a small letter
[x] all model elements at least contain the fields "preferred name" and "description" in English language. The description must be comprehensible. It is not required to write full sentences but style should be consistent over the whole model
[x] Property and the referenced Characteristic should not have the same name
[x] the versioning in the URN follows semantic versioning, where minor version bumps are backwards compatible and major version bumps are not backwards compatible.
[x] use abbreviations only when necessary and if these are sufficiently common
[x] avoid redundant prefixes in property names (consider adding properties to an enclosing Entity or even adapt the namespace of the model elements, e.g., instead of having two properties DismantlerId and DismantlerName use an Entity Dismantler with the properties name and id or use a URN like io.catenax.dismantler:0.0.1)
[x] fields preferredName and description are not the same
[x] preferredName should be human readable and follow normal orthography (e.g., no camel case but normal word separation)
[x] name of aspect is singular except if it only has one property which is a Collection, List or Set. In theses cases, the aspect name is plural.
[x] units are referenced from the SAMM unit catalog whenever possible
[x] use constraints to make known constraints from the use case explicit in the aspect model
[x] when relying on external standards, they are referenced through a "see" element
[x] all properties with an simple type have an example value
[x] metadata.json exists with status "release"
[x] generated json schema validates against example json payload
[x] file RELEASE_NOTES.md exists and contains entries for proposed model changes
[x] all contributors to this model are mentioned in copyright header of model file

MS3 Criteria
(to be filled out by semantic modeling team before merge to main-branch)

[ ] All required reviewers have approved this PR (see reviewers section)
[ ] The new aspect (version) will be implemented by at least one data provider
[ ] The new aspect (version) will be consumed by at least one data consumer
[ ] There exists valid test data
[ ] In case of a new (incompatible) major version to an existing version, a migration strategy has been developed
[ ] The model has at least version '1.0.0'
[ ] If a previous model exists, model deprecation has been checked for previous model
[ ] The release date in the Release Note is set to the date of the MS3 approval

generally see requirements from the industry core

All the requested adjustments are now fullfilled.

Please look at the comments of the industry core here:

I think there are still some unresolved topics.
https://miro.com/app/board/uXjVMiC7vAc=/?moveToWidget=3458764571585802082&cot=14

I cannot open the webpage since it requires a password.

@catroest I cannot open the miro webpage since it requires a password.

The PW is networkofnetwork :)

@catroest 1 question regarding the property "productGroup": Is it ok to use this name and link it with ClassificationCharacteristic from shared aspect part_classification? Or I should partClassification property directly?

@catroest 1 question regarding the property "productGroup": Is it ok to use this name and link it with ClassificationCharacteristic from shared aspect part_classification? Or I should partClassification property directly?

Both would be possible. If you link to the Characteristic you could describe you property yourself. Otherwise you take over the description of the partClassification property. As your descriptions are very similiar, I would suggest the "cleaner" version and link to the property, then you do not have to describe the property again.

@yuchengluo1 let me know when the model is ready for MS2 check again.

You may check now, thanks.


'Scan' function type response std::bad_cast issue

Hello, I have no idea how to work with 'scan' functions. For example, when I'm trying to run the following code
auto response = rediscpp::execute(*stream, "scard", "some_set");
std::cout << response.as_integer() << std::endl;
// Output is '4'
response = rediscpp::execute(*stream, "sscan", "some_set", "0");
std::cout << std::boolalpha << response.is_array() << std::endl;
// Output is 'true'
auto vec = response.as_string_array();
// 'bad_cast' error

the last line causes std::bad_cast error, though response.is_array() is true. Furthermore, any available rediscpp cast causes std::bad_cast error. Am i doing something wrong?  Is there any way to avoid this error?

Hi, I'm so sorry; I've just noticed the message. The library is a low-level tool for sending commands and receiving responses. There isn't any type checking for a specific command. It might be a good idea to write a new library with commands, but I stopped at this version, which satisfied my needs some years ago and has proven quite useful for others through usage.
After reading about your issue, I'd suggest using the approach for working with arrays in a similar way to the examples in the library. This is because "SCAN" (and similar commands) return complex arrays: the first part contains additional information, and the second part is an array with the data. In this case, you can extract data using a couple of nested visitors, as described in the "SCAN" command documentation and the library's examples.
SCAN: https://redis.io/commands/scan/
Examples

https://github.com/tdv/redis-cpp/blob/master/examples/pubsub/src/main.cpp#L58
https://github.com/tdv/redis-cpp/blob/master/examples/resp/src/main.cpp#L38


[Compute] BREAKING CHANGE: az disk/snapshot create: Change the default value of --hyper-v-generation from V1 to None

Related command

az disk create
az snapshot create
Description

az snapshot create and az disk create CLI commands to not pass V1 as default value
close https://github.com/Azure/azure-cli/issues/27323
Testing Guide

History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change
[Component Name 2] az command b: Add some customer-facing feature

This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] The PR title and description has followed the guideline in Submitting Pull Requests.

[ ] I adhere to the Command Guidelines.

[ ] I adhere to the Error Handling Guidelines.

Compute

Just wonder if any reason here for not including this BC into our upcoming breaking change
cc: @ReaNAiveD


[release-v1.27] Cherry-pick 2197 and 2158

Description
Cherry-pick the following commits:
https://github.com/tigera/operator/pull/2158
https://github.com/tigera/operator/pull/2197

For PR author

[ ] Tests for change.
[ ] If changing pkg/apis/, run make gen-files
[ ] If changing versions, run make gen-versions

For PR reviewers
A note for code reviewers - all pull requests must have the following:

[ ] Milestone set according to targeted release.
[ ] Appropriate labels:

kind/bug if this is a bugfix.
kind/enhancement if this is a a new feature.
enterprise if this PR applies to Calico Enterprise only.

Close in favor of https://github.com/tigera/operator/pull/2347


Update ungoogled-chromium to 76.0.3809.87

I have compiled ungoogled chromium 76.0.3809.87 with this patchset and it is running well.

I can confirm a successful compilation.


Cube Game

Email :-<EMAIL_ADDRESS>issue #207
Cube Game is a dynamic and interactive web-based game that challenges players to navigate a cube through a series of obstacles and levels. The game is built using a combination of HTML, CSS, and JavaScript to provide a seamless and engaging user experience.
Technical Details:
HTML:
The structure of the game is defined using HTML. This includes the setup of the game canvas, buttons, and other interactive elements.
Semantic HTML tags are used to ensure the code is clean and accessible.
CSS:
CSS is used to style the game elements, providing an appealing visual experience.
Custom styles are applied to the game canvas, buttons, and other UI components to enhance the look and feel.
Responsive design principles are implemented to ensure the game is playable on various devices and screen sizes.
JavaScript:
JavaScript handles the game logic, including player controls, collision detection, and game state management.
Event listeners are set up to capture user inputs and update the game accordingly.
Animations and movement of the cube and obstacles are managed through JavaScript functions to ensure smooth gameplay.
Additional Features:
Power-ups and collectibles are integrated to add excitement and challenge to the game.
A scoring system tracks player performance, encouraging replayability.
Sound effects and background music are added to enhance the overall gaming experience.
Cube Game is designed to be easy to understand yet challenging to master, making it an enjoyable game for players of all skill levels. The use of HTML, CSS, and JavaScript ensures that the game runs efficiently and is easy to maintain and expand upon in future updates.
https://github.com/dhairyagothi/50_days_50_web_project/assets/140265777/2808b7ea-2686-4881-ab38-8782510ae383

@Yaser-123 please resolve the conflicts @Yaser-123

Resolved Conflict


Java: Client classes expose internal classes in public package

Describe the bug
BaseClient and related classes depend on CommandManager, CommManager, etc which are in packages that are not exposed in the module-info. They are protected fields so that they can be unit-tested.
This design makes these internal classes exposed indirectly through the public API.
Expected Behavior
Hidden types such as CommandManager are not visible.
Current Behavior
Hidden types such as CommandManager are visible.
Reproduction Steps
Design issue.
Possible Solution
A fix would be to make RedisClient and RedisClusterClient interfaces (that extend all the same interfaces that BaseClient, RedisClient, and RedisClusterClient currently do) that have static CreateCleint() functions. The CreateClient() functions instantiate RedisClientImpl/RedisClusterClientImpl classes in hidden packages but return them as interface types (similar to ClusterScanCursor and ClusterScanCursorDetail).
Additional Information/Context
No response
Client version used
N/A
Redis Version
N/A
OS
N/A
Language
Java
Language Version
N/A
Cluster information
No response
Logs
No response
Other information
No response

This is very Java specific, better to be handled by Java experts , unassigning


Electric sound

Hi! I've trained waveglow on one female speaker, but results sound very "electric" or "metallic" (I attached audios after vocoder and original audio), and it's not getting better with more iterations.
wavs.zip
Does anybody know how to solve this problem? Maybe I need to do some data preprocessing, I tried facebook denoiser, but it didn't help

@CJai-K I am trying to find the reason of electric sound, it may be whistling frequencies #249


Add x!=0 -> (bool)x pattern [run_process_replay] [no_assert]

Replaces x != 0 pattern with x.cast(bool)

I see (bool)(val0)!=1 in process replay now

I think.
x != 0 != 1 is not(bool(x)),
I could add a bool comparison pattern to cast subsequent CMPNE down


Support dt.floor(), dt.ceil() and dt.round() accessors.

[x] Tests added (for all bug fixes or enhancements)
[x] Tests passed (for all non-documentation changes)
[x] Passes git diff upstream/master **/*py | flake8 --diff (remove if you did not edit any Python files)
[x] Fully documented, including whats-new.rst for all changes and api.rst for new API (remove if this change should not be visible to users, e.g., if it is an internal clean-up, or if this is part of a larger project that will be documented later)

Following on from #1762 , I've tried to implement floor, ceil and round as in pandas.

@shoyer All done.


[Accessibility][Screen Reader] [NVDA] - Screen reader ignores "File", "New Messages" and "Send" buttons on hover

Actual result:
The screen reader ignores "File", "New Messages" and "Send" buttons on hover.
Expected result:
The screen reader will read "File", "New Messages" and "Send" buttons on hover.
Repro Steps:

Turn on NVDA.
Open https://microsoft.github.io/BotFramework-WebChat/01.getting-started/a.full-bundle/ in Chrome.
Hover the cursor on "File", "New Messages" and "Send" buttons.

Dupe of #3048.


Update GHA non-cached third party actions

This change updates several non-cached GHA third party actions to use Roblox-ActionsCache. It's a manually created duplicate of #363 to avoid issues with batch changes triggering the CLA.
Closes #363.

Coverage remained the same at 94.783% when pulling f3609f6bcb6503a2da82f22554067d46f0e76006 on eeci-2596-update-gha-non-cached-actions-manual into bacc7f076e7346d6c5a2090db6180cafe3e87d85 on master.


Go 1.6 vendoring and Makefile

Vendoring
Vendored dependencies were moved from Godeps/_workspace/src to vendor/. Godeps/Godeps.json was updated in the process. Addresses #5.
Makefile
All functionality from bin/ was moved to Makefile, which now includes these targets:

build
build.linux-amd64
clean
default (test)
fmt
get-deps
lint
test
vet

Travis
The Go version was pinned to 1.6, and go_import_path was set to allow contributors with forked repositories to run Travis tests on those repos. Finally, Travis runs the get-deps target instead of using godep. This installs vet, ginkgo, and gomega, but relies on Go 1.6 to use dependencies in the vendor/ dir. The test target is then executed.

Thanks @evandbrown! Added some comments.

Good catch on the linter. I'll add that.
For the build target, what if we only build for the platform and architecture that target is executed from, and add a package target that builds for all possible (platform, arch)?

Sounds good to me.

Thanks @evandbrown!


Updated dependency versions and implementations

Updated byteorder error usage (now returns straight Errors)
Updated Uuid implementation (to_simple_string now .simple().to_string())

All of your beautiful, extensive tests pass and the crate compiles as expected!  Should be no issues here at all; almost all of the updated crates had no breaking changes.

Coverage decreased (-92.1%) to 0.0% when pulling e0ba699c1622246e6d44268a500577179e90772a on Ameobea:master into b6423f9a76dcf3f4f62c6b2de5357ef365956e22 on tantivy-search:master.

Coverage decreased (-92.1%) to 0.0% when pulling e5cfbaa2490ab0d201c823460438a3b45dda7c3e on Ameobea:master into b6423f9a76dcf3f4f62c6b2de5357ef365956e22 on tantivy-search:master.

kudos @Ameobea !


sparsify.package base CLI

This PR represents one of the first in a series of PRs for sparsify.package, alpha release, and represents the user facing CLI for the same.
The scope of this PR includes

[X] Adding a sparsify.package module + importable function + CLI callable

[X] Adding Arguments to sparsify.package.cli based on acceptance criterion

[X] Add relevant tests for CLI

[X] Add click~=8.0.0 as a dependency for sparsify

note - we will need to test a call to package that does not mock the API call, to verify end to end functionality independent of API side tests

Will be added as a part of the following diff: #130


Treewide: change all occurrences of "name" to "pname-version"

Motivation for this change
Things done

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

[ ] 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 nixpkgs-review --run "nixpkgs-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)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

Related issue: https://github.com/NixOS/nixpkgs/issues/103997

Is it expected that none of these derivations will build differently because the interpolated strings are the same as they were before?

Yes. It is just a rewriting of the expressions.

With my changes, Nixpkgs can be eval'd again and the sources that aren't already broken still produce the same output.
Also, nit: Treewide -> treewide

Oh man, this will not be easy.
There are many "legitimate" uses of namein Nixpkgs, in the sense that merely changing a single file from name to pname-version triggers auxiliary functions defined elsewhere - and these functions use name!

I think it's impossible for this to not trigger rebuilds since even in the simplest case, the builder gets after this change the new environment variable pname and version which weren't there before and unfortunately (without structuredAttrs I suppose) it's a change in the inputs.

@doronbehar : then, in a certain sense, the expression itself becomes a "build input" of itself?
And what is structuredAttrs?
Well, in this sense I think we can sprint it to the next viable update.

And what is structuredAttrs?

https://github.com/NixOS/nixpkgs/pull/72074 && https://nixos.mayflower.consulting/blog/2020/01/20/structured-attrs/

@doronbehar : then, in a certain sense, the expression itself becomes a "build input" of itself?

I'm not sure I understand your explanation, but to demonstrate how easily rebuilds can be triggered, consider a change such as:
diff --git i/pkgs/development/compilers/gcc/9/default.nix w/pkgs/development/compilers/gcc/9/default.nix
index 305ed56df78..e39c787284b 100644
--- i/pkgs/development/compilers/gcc/9/default.nix
+++ w/pkgs/development/compilers/gcc/9/default.nix
@@ -83,6 +83,7 @@ in
 stdenv.mkDerivation ({
   pname = "${crossNameAddon}${name}${if stripped then "" else "-debug"}";
   inherit version;
+  dummyVar = "bla";

   builder = ../builder.sh;

That would rebuild the world, and you can see that via running nix show-derivation -f. gcc.cc before and after the change, and inspect the diff in a text editor.
The difference between the derivations can be found in the env attribute of the derivations, where the dummyVar environment variable which you and I know that won't affect the build is the culprint, but Nix doesn't know it.
In general nix show-derivation -f. is the judge, though ofborg has a more sophisticated way of doing that in a batch.
What's so frustrating about it is that our default-builder.sh and most of it's derivatives handle the conditionals which ideally should have been evaluated by Nix. A real world examples of such unnecessary rebuilds triggered is (only the Darwin rebuilds of) e.g: https://github.com/NixOS/nixpkgs/issues/94952#issuecomment-673557750 .
Even changes in the order of inputs in a Nix arrays causes rebuilds, consider this change:
diff --git i/pkgs/development/compilers/gcc/9/default.nix w/pkgs/development/compilers/gcc/9/default.nix
index 305ed56df78..d73089e9c0c 100644
--- i/pkgs/development/compilers/gcc/9/default.nix
+++ w/pkgs/development/compilers/gcc/9/default.nix
@@ -168,7 +168,7 @@ stdenv.mkDerivation ({
     ++ optional targetPlatform.isLinux patchelf;

   buildInputs = [
-    gmp mpfr libmpc libelf
+    mpfr gmp libmpc libelf
     targetPackages.stdenv.cc.bintools # For linking code at run-time
   ] ++ (optional (isl != null) isl)
     ++ (optional (zlib != null) zlib)

It will also trigger a rebuild of the world.
The problem is that mkDerivation doesn't know how to filter out the unnecessary information for our default-builder.sh though I can't think of a better way to design it.

structuredAttrs could perhaps allow us to write a default-builder.sh that will hopefully be resilient to this false interpretations of changed inputs.

@doronbehar order of build inputs matters since it can change the order setup hooks are run and we cannot guarantee arbitrary bash code commutes. Another example where it is a problem are environment variables like PKG_CONFIG_PATH – when multiple different derivations contain the same library, the first one found in the PKG_CONFIG_PATH will be chosen. I routinely encounter this when overriding subset of dependencies with enableDebugging, especially when dependency propagation comes into play.

This is just a parallel work for now, and way far from a clear finish.

Another day, another huma nerror. Now I will try to recover.

GH thinks you force-pushed to a commit already in master and, since that means there are no changes, won't let you open the PR.
Should work again if you push another commit.
Would probably also not be a bad idea to simply push commits and then squash them up once they're ready. Following changes between force-pushes isn't really something you can do very well in git.

Would probably also not be a bad idea to simply push commits and then squash them up once they're ready. Following changes between force-pushes isn't really something you can do very well in git.

Not true as long as you don't do a rebase in the same force push. https://github.com/NixOS/nixpkgs/compare/32986af10085058e08125720467d84f5e1afe4b3..712c2405b1c4ddf461a1eafd640165583d1c0cca

Reopening
https://github.com/NixOS/nixpkgs/pull/104662


Synthesise UX results and publish them

Smera is working on the report right now. We'll also need to include this in the project documentation, which I can help with. :)

The wip report is here: [DRAFT] User Testing Results
Another important component is the Figjam board where I have consolidated the notes from the interviews, and made an affinity diagram to guide the report: User Research Data


A trouble to understand the getitem method in  RoiDataLoader class

I am trying to understand the signature of the "getitem"  method of the "RoiDataLoader" class in the /lib/roi_data/loader.py file. That class is a subclass of the abstract class "dataset" in pytorch. In the definition of "dataset" in pytorch, the "getitem" method supports integer indexing in range from 0 to len(self) exclusive. But for the RoiDataLoader, the parameter for "getitem" method is an index_tuple. Could you explain how it works?
Thanks.

Does anyone have any ideas about how to move forward to understand the "getitem" method?

__getitem__'s input arguments are given from sampler, and the sampler I use is MinibatchSampler.
MinibatchSampler is a generator that yields a tuple of (ratio, index), so dataset's __getitem__ will receive a tuple instead of a single int. That's it.

Thanks Roytseng for your reply. I just figured out. Yes, getitem method allows a tuple as an index(see the link at the end). The definition of the next method of  DataLoaderIter in pytorch is like this:
def next(self):
if self.num_workers == 0:  # same-process loading
indices = next(self.sample_iter)  # may raise StopIteration
batch = self.collate_fn([self.dataset[i] for i in indices])
So the self.sample_iter is the MinibatchSampler in our context.
https://stackoverflow.com/questions/1685389/possible-to-use-more-than-one-argument-on-getitem?utm_medium=organic&utm_source=google_rich_qa&utm_campaign=google_rich_qa
Roytseng, you can close this issue now if you like. Thanks again.


An aborted XHR does not have responseHeaders

After aborting an XHR, there should be an empty responseHeaders object available. This is because after a send, the abort can occur. An onReadyState is fired and a callback that checks if readyState == 4 that proceeds to check responseHeaders.
In a browser, there is an empty responseHeaders after an aborted XHR. In Sinon, it is undefined.
A PR is coming that fixes it (with tests).

The PR has been merged, thanks!

Cheers :beers:


Organizational Diversity - revisited metric

Minor updates to the Organizational Diversity Metrics as part of the revisited metrics process

@klumb - can you do a quick review and merge if the formatting looks good. I made a lot of bullet list changes and want to make sure that I didn't miss anything that will cause you problems on the website.


Release v0.23.10

I'm not going to merge this tonight because it's going to kill the site after I go to asleep.  Anyone else can, but please revert the head commit of gh-pages if you do.

Actually merging it is fine.  Publishing the tag needs supervision.


Django 3 dropped support to django.utils.six

I just try to upgrade my server to Django 3.0 release today for tests and as we can read the module django.utils.six will be dropped with the version 3 of Django.
Here you can find my Traceback:
  File "/home/user/.virtualenvs/dns/lib/python3.7/site-packages/pipeline/finders.py", line 9, in <module>
    from pipeline.conf import settings
  File "/home/user/.virtualenvs/dns/lib/python3.7/site-packages/pipeline/conf.py", line 15, in <module>
    from django.utils.six import string_types
ModuleNotFoundError: No module named 'django.utils.six'

Please evaluate the migration to six as suggested in the #removed-private-python-2-compatibility-apis section of the release note.
Thanks for this amazing library!

As well you can replace string_types in pipeline/conf.py line 101 by (str,)?
In [1]: from django.utils.six import string_types

In [2]: string_types
Out[2]: (str,)

In [3]: string_types == (str,)
Out[3]: True

This is an amazing library. Looking forward to the fix for Django 3.0.0

The concept for metaclass is really different between python 2 and 3 (more than I thought) for PipelineFormMedia...
We have two choices:

Dropping Python 2 support and the Python 3 use modern syntax: class PiepelineFormMedia(object, metaclass=PipelineMediaMetaClass)
Copy the django.utils.six decorator to reuse it.

Merge request #708 fix this

@TheBuky Thanks for getting this merged. Do you have a sense on when this will be available on pypi? We still need to point directly to repo rather then just pip install django-pipeline==1.7.0 --upgrade

@eire1130 The Travis job to automatically publish package and PyPI.org is broken (authentication error). I open an issue to Jazzband for checking the credentials, and fix it.
Meanwhile you can, you can git clone the project, go in the django-pipeline repertory and do python3 setup.py sdist bdist_wheel. That will create new folder dist and you can install the package with pip install -U dist/django-pipeline-1.7.0.tar.gz (or something like that).
Like explained there

@TheBuky thanks. I just need it to live in my requirements.txt for deploys. For now I have it pinned source, which works of course, but I prefer to use package / pypi names rather then pointing to github.
Again, thank you for being on top of this, I really appreciate it!


Fix test conflicts

Closes #6710

Restore config to baseline before testing.
Don't mutate fixture data. 😡

That was fast! :dash:
Also.. this seems pretty healthy:


Revise numpy 1.16 compatibility

Informations
A new version 1.16 of numpy  was released recently, which resulted in a number of linter warnings that might hint towards incompatible changes.
In #1678 the version was pinned to <1.16, to give us time to investigate further and take the appropiate actions - the release notes might give some good hints.

Closed with #1848


Check results reposting for Sun and Abraham implementation

After estimation via Sun and Abraham's method, the coefficients and standard errors are reposted to an xtevent regression, but other elements are not reposted, such as the degrees of freedom. Because of this, some elements of the header in the estimation results after Sun and Abraham estimation are wrong.

@Constantino-Carreto-Romero in https://github.com/JMSLab/xtevent/commit/c9a2fa24cd6c54078616d24c977bc18f37c0e865 I posted a change to post the model degrees of freedom to the baseline regression for you to take a look. From here we can:

Generalize the code to post all of the results from e() to the base regression
Set the interacted regression as the base regression for the Sun and Abraham case, and replace the coefficients for that one.

Either way should work, so try to see which one would be easier @Constantino-Carreto-Romero

@Constantino-Carreto-Romero in c9a2fa2 I posted a change to post the model degrees of freedom to the baseline regression for you to take a look. From here we can:

Generalize the code to post all of the results from e() to the base regression
Set the interacted regression as the base regression for the Sun and Abraham case, and replace the coefficients for that one.

Either way should work, so try to see which one would be easier @Constantino-Carreto-Romero

@jorpppp
in https://github.com/JMSLab/xtevent/commit/ff551c3afa616099cedc4c12f39bd8b7bbc52ae6 I implemented the second alternative. The SA estimation flow is as follows:

Previously, first, it estimated a xtevent base regression, saved it, and then estimated the interacted regression. Now the base regression is not estimated, but only the interacted regression.
Previously, the SA's corrected coefficient vector and covariance matrix were inserted into the base regression. Now, both elements replace the ones from the interacted regression. To be able to do this (and not to have to rely on an initial base regression to be able to post results), I changed the program repostdelta following this post.

After that change, however, there was a bug when posting the new regression table in the GMM-adjustment code if the underlying estimation command is regress. To be able to post the new regression table I changed cmd for _coef_table_header and _coef_table here.
Then, I ran the test file with the code from the main branch and a second time with the code in this issue to verify: i) Estimation output in settings other than SA should be the same; ii) in SA examples, the coefficients and se should not change, but the regression header should, reflecting the computations from the interacted regression. The comparison is in https://github.com/JMSLab/xtevent/commit/0c481da7d8731e1f627424c47e06db92cebc6293.

When inspecting that comparison, the regression output in settings other than SA was shortened in some cases. It is owing to the change of cmd for _coef_table_header and _coef_table.
In the SA examples, the covariate coefficients are not shown in the regression output anymore. Previously, these coefficients came from the base regression since SA corrects only the event-time dummy coefficients. This is similar to EventStudyInteract which reports only the event-time coefficients.

Thanks @Constantino-Carreto-Romero, this looks good. Opening a PR now.

Summary: In this issue we fixed a bug where header results were incorrect for Sun and Abraham estimation.
Thread continues in XX


Nothing plays

Acknowledgement

[X] I have read Getting-Started and FAQ

🐛 Describe the bug
Suddenly, OpenUtau won't play anything. Everything has worked normally in the past. If I press space, or click play, nothing happens. I know it is trying to play, since I can't scroll (like when it is actually playing). I have tried using different usts that have worked before, and creating new ones. But nothing will work.
Explains how to reproduce the bug
I don't know how to replicate this bug. I have tried to get rid of it using different voicebanks, different usts, and I have tried to reinstall the program. None of these changed anything.
OS & Version
macOS 10.14, OpenUtau v<IP_ADDRESS>
Logs
log20231018.txt

Did you select the correct audio output device in Settings?

me too the play button doens't work when i clikc play nothing happens

I will say it agaib: make sure you have the correct audio device selected.

I will say it again: make sure you have the correct audio device selected.

Sorry, I didn't see your response. I had selected the correct audio device, and the audio test was working as normal. The arrow that normally moves when playing didn't move, so I think there was another issue going on. But the problem is somehow fixed now. Not sure how, as I haven't touched the program in a week. Thanks for the advice anyways!

Oh don't worry about it, that second message wasn't really aimed at you. I'm happy the problem got solved though!

I will say it again: make sure you have the correct audio device selected.

Sorry, I didn't see your response. I had selected the correct audio device, and the audio test was working as normal. The arrow that normally moves when playing didn't move, so I think there was another issue going on. But the problem is somehow fixed now. Not sure how, as I haven't touched the program in a week. Thanks for the advice anyways!

Haven't touched thee program in a week and it works? I have the same issue! I'll try that too.


Replace gslcblas with openblas

@ocefpaf, can you review?

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@ocefpaf,  changes here can be incorporated into gsl-feedstock

Do you want to send a PR to do that?

Will do


Added animeted Pre Loader

Title and Issue number

Title : [Feature Request] : Adding animeted Pre Loader.
Issue No. : #198
Code Stack :
Close #198

Video (mandatory)

https://github.com/user-attachments/assets/b0cfd5c5-5307-4414-a51b-366a8cde745c
Checklist:

[x] I have mentioned the issue number in my Pull Request.
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have created a helpful and easy to understand README.md if its a new page/tech stack
[x] I have gone through the  contributing.md file before contributing

Are you contributing under any Open-source programme?

[x] I'm a GSSOC-EXT contributor
[x] I'm a HACKTOBERFEST contributor

Title and Issue number
Title : [Feature Request] : Adding animeted Pre Loader.
Issue No. : #198
Code Stack :
Close #198
Video (mandatory)
Preloader.Fitflex.mp4
Checklist:
* [x]  I have mentioned the issue number in my Pull Request.

* [x]  I have commented my code, particularly in hard-to-understand areas

* [x]  I have created a helpful and easy to understand `README.md` if its a new page/tech stack

* [x]  I have gone through the  `contributing.md` file before contributing

Are you contributing under any Open-source programme?
* [x]  I'm a GSSOC-EXT contributor

* [x]  I'm a HACKTOBERFEST contributor

nice loader, but their is already one which is also good

Hey @Sami3160 But it's look more good and cool

check your code, netlify deployment is failing,


How to change shortcut list in F1 output

Is there any way to change the shortcut list shown in F1's output.
I could add custom shortcut info in the list but can i change the existing ?

Let's track in #102