all chunks are corrupt from any export, the worlds are unstable

So, when importing the world with two separate multiworld management plugins on a spigot server and attempting to import the world in worldpainter I am getting issues with corrupt region data, this happens on all export types, box exports and citywide exports,
you can reproduce this error by going into a exported world and seeing your latest log output thousands of messages a second about chunk location errors, im assuming this is because the export command requires a existing world to overwrite the chunks which is causing the world to assume the chunks are in the wrong location causing the error
[00:14:14] [Server thread/ERROR]: Chunk file at [460, 760] is in the wrong location; relocating. (Expected [460, 760], got [12, 24])
[00:14:14] [Server thread/ERROR]: Chunk file at [450, 761] is in the wrong location; relocating. (Expected [450, 761], got [2, 25])
[00:14:14] [Server thread/ERROR]: Chunk file at [465, 760] is in the wrong location; relocating. (Expected [465, 760], got [17, 24])
[00:14:14] [Server thread/ERROR]: Chunk file at [467, 760] is in the wrong location; relocating. (Expected [467, 760], got [19, 24])
[00:14:14] [Server thread/ERROR]: Chunk file at [451, 761] is in the wrong location; relocating. (Expected [451, 761], got [3, 25])
[00:14:14] [Server thread/ERROR]: Chunk file at [468, 760] is in the wrong location; relocating. (Expected [468, 760], got [20, 24])
[00:14:14] [Server thread/ERROR]: Chunk file at [452, 761] is in the wrong location; relocating. (Expected [452, 761], got [4, 25])
[00:14:14] [Server thread/ERROR]: Chunk file at [463, 760] is in the wrong location; relocating. (Expected [463, 760], got [15, 24])
[00:14:14] [Server thread/ERROR]: Chunk file at [453, 761] is in the wrong location; relocating. (Expected [453, 761], got [5, 25])
[00:14:14] [Server thread/ERROR]: Chunk file at [455, 761] is in the wrong location; relocating. (Expected [455, 761], got [7, 25])
[00:14:14] [Server thread/ERROR]: Chunk file at [454, 761] is in the wrong location; relocating. (Expected [454, 761], got [6, 25])
[00:14:14] [Server thread/ERROR]: Chunk file at [456, 761] is in the wrong location; relocating. (Expected [456, 761], got [8, 25])
[00:14:14] [Server thread/ERROR]: Chunk file at [457, 761] is in the wrong location; relocating. (Expected [457, 761], got [9, 25])
[00:14:14] [Server thread/ERROR]: Chunk file at [459, 761] is in the wrong location; relocating. (Expected [459, 761], got [11, 25])
[00:14:14] [Server thread/ERROR]: Chunk file at [458, 761] is in the wrong location; relocating. (Expected [458, 761], got [10, 25])
[00:14:14] [Server thread/ERROR]: Chunk file at [460, 761] is in the wrong location; relocating. (Expected [460, 761], got [12, 25])
[00:14:14] [Server thread/ERROR]: Chunk file at [463, 761] is in the wrong location; relocating. (Expected [463, 761], got [15, 25])
[00:14:14] [Server thread/ERROR]: Chunk file at [461, 761] is in the wrong location; relocating. (Expected [461, 761], got [13, 25])
[00:14:14] [Server thread/ERROR]: Chunk file at [462, 761] is in the wrong location; relocating. (Expected [462, 761], got [14, 25])
[00:14:14] [Server thread/ERROR]: Chunk file at [464, 761] is in the wrong location; relocating. (Expected [464, 761], got [16, 25])
[00:14:14] [Server thread/ERROR]: Chunk file at [467, 761] is in the wrong location; relocating. (Expected [467, 761], got [19, 25])
[00:14:14] [Server thread/ERROR]: Chunk file at [468, 761] is in the wrong location; relocating. (Expected [468, 761], got [20, 25])```

I know about this bug and I also tried to run the server with the same error, I tried to run it through worldpainter but nothing worked, I think that this is a problem with anvil parser or with the code associated with it

Having the same issue on singleplayer, but the city renders fine anyway.
here's the log
latest.log

Singleplayer seems to be working fine every time, but I already noticed Minecraft detecting some problems.

Not sure if it might help, but we can look into updating the anvil package to this continued development fork: https://github.com/Intergalactyc/anvil-new

Singleplayer seems to be working fine every time, but I already noticed Minecraft detecting some problems.
Not sure if it might help, but we can look into updating the anvil package to this continued development fork: https://github.com/Intergalactyc/anvil-new

i'll try replacing the package and see if it changes anything.

Singleplayer seems to be working fine every time, but I already noticed Minecraft detecting some problems.
Not sure if it might help, but we can look into updating the anvil package to this continued development fork: https://github.com/Intergalactyc/anvil-new

i'll try replacing the package and see if it changes anything.

i changed the package and not much has changed. since this one doesnt have the Block.from_id function, i had to adjust the block definitions
import anvil

air = anvil.Block("minecraft", "air")
birch_leaves = anvil.Block("minecraft", "birch_leaves")
birch_log = anvil.Block("minecraft", "birch_log")
black_concrete = anvil.Block("minecraft", "black_concrete")
blue_flower = anvil.Block("minecraft", "blue_orchid")
brick = anvil.Block("minecraft", "bricks")
carrots = anvil.Block("minecraft", "carrots", {"age": 7})
cauldron = anvil.Block("minecraft", "cauldron")
cobblestone = anvil.Block("minecraft", "cobblestone")
cobblestone_wall = anvil.Block("minecraft", "cobblestone_wall")
dark_oak_door_lower = anvil.Block("minecraft", "dark_oak_door", {"half": "lower"})
dark_oak_door_upper = anvil.Block("minecraft", "dark_oak_door", {"half": "upper"})
dirt = anvil.Block("minecraft", "dirt")
farmland = anvil.Block("minecraft", "farmland")
glass = anvil.Block("minecraft", "glass_pane")
glowstone = anvil.Block("minecraft", "glowstone")
grass = anvil.Block("minecraft", "grass")
grass_block = anvil.Block("minecraft", "grass_block")
gravel = anvil.Block("minecraft", "gravel")
gray_concrete = anvil.Block("minecraft", "gray_concrete")
green_stained_hardened_clay = anvil.Block("minecraft", "green_terracotta")
hay_bale = anvil.Block("minecraft", "hay_block")
iron_block = anvil.Block("minecraft", "iron_block")
light_gray_concrete = anvil.Block("minecraft", "light_gray_concrete")
oak_fence = anvil.Block("minecraft", "oak_fence")
oak_leaves = anvil.Block("minecraft", "oak_leaves")
oak_log = anvil.Block("minecraft", "oak_log")
oak_planks = anvil.Block("minecraft", "oak_planks")
podzol = anvil.Block("minecraft", "podzol")
potatoes = anvil.Block("minecraft", "potatoes", {"age": 7})
rail = anvil.Block("minecraft", "rail")
red_flower = anvil.Block("minecraft", "poppy")
sand = anvil.Block("minecraft", "sand")
scaffolding = anvil.Block("minecraft", "scaffolding")
sponge = anvil.Block("minecraft", "sponge")
spruce_log = anvil.Block("minecraft", "spruce_log")
stone = anvil.Block("minecraft", "stone")
stone_block_slab = anvil.Block("minecraft", "stone_slab")
stone_brick_slab = anvil.Block("minecraft", "stone_brick_slab")
water = anvil.Block("minecraft", "water")
wheat = anvil.Block("minecraft", "wheat", {"age": 7})
white_concrete = anvil.Block("minecraft", "white_concrete")
white_flower = anvil.Block("minecraft", "azure_bluet")
white_stained_glass = anvil.Block("minecraft", "white_stained_glass")
yellow_flower = anvil.Block("minecraft", "dandelion")

# Variations for building corners
building_corner_variations = [
    anvil.Block("minecraft", "stone_bricks"),
    anvil.Block("minecraft", "cobblestone"),
    anvil.Block("minecraft", "bricks"),
    anvil.Block("minecraft", "mossy_cobblestone"),
    anvil.Block("minecraft", "sandstone"),
    anvil.Block("minecraft", "red_nether_bricks"),
    anvil.Block("minecraft", "blackstone"),
    anvil.Block("minecraft", "smooth_quartz"),
    anvil.Block("minecraft", "chiseled_stone_bricks"),
    anvil.Block("minecraft", "polished_basalt"),
    anvil.Block("minecraft", "cut_sandstone"),
    anvil.Block("minecraft", "polished_blackstone_bricks"),
]

# Variations for building walls
building_wall_variations = [
    anvil.Block("minecraft", "white_terracotta"),
    anvil.Block("minecraft", "gray_terracotta"),
    anvil.Block("minecraft", "bricks"),
    anvil.Block("minecraft", "smooth_sandstone"),
    anvil.Block("minecraft", "red_terracotta"),
    anvil.Block("minecraft", "polished_diorite"),
    anvil.Block("minecraft", "smooth_stone"),
    anvil.Block("minecraft", "polished_andesite"),
    anvil.Block("minecraft", "warped_planks"),
    anvil.Block("minecraft", "end_stone_bricks"),
    anvil.Block("minecraft", "smooth_red_sandstone"),
    anvil.Block("minecraft", "nether_bricks"),
]

# Variations for building floors
building_floor_variations = [
    anvil.Block("minecraft", "oak_planks"),
    anvil.Block("minecraft", "spruce_planks"),
    anvil.Block("minecraft", "dark_oak_planks"),
    anvil.Block("minecraft", "stone_bricks"),
    anvil.Block("minecraft", "polished_granite"),
    anvil.Block("minecraft", "polished_diorite"),
    anvil.Block("minecraft", "acacia_planks"),
    anvil.Block("minecraft", "jungle_planks"),
    anvil.Block("minecraft", "warped_planks"),
    anvil.Block("minecraft", "purpur_block"),
    anvil.Block("minecraft", "smooth_red_sandstone"),
    anvil.Block("minecraft", "polished_blackstone"),
]```

Thanks for checking, if it has a major speed improvement for setting, reading blocks or saving the regions we should consider updating to this package. What's your opinion on that?

Thanks for checking, if it has a major speed improvement for setting, reading blocks or saving the regions we should consider updating to this package. What's your opinion on that?
I didn't specifically notice a performance difference, but that doesnt mean there isnt one. I haven't benchmarked it. I think we should we upgrade regardless, to futureproof

Please check if this issue is still occuring in the new release (Arnis v2.0.0)! https://github.com/louis-e/arnis/releases/tag/v2.0.0
Additionally, one more idea: Try joining the world to load the chunks and save it by leaving again. I think Minecraft has some sort of chunk fixing techniques implemented.

This issue is still happening for me in v2.0.0 Log below:
2024-09-21-5.log

Same here whet testing on Paper server:
...
minecraft-arnis-1  | [11:22:04 ERROR]: Ignoring absolute chunk [9, 63] in regionfile as it is not contained in the bounds of the regionfile '/data/./world/region/r.0.0.mca'. It should be in regionfile (0,1)
...

...
minecraft-arnis-1  | [11:22:04 WARN]: Data for local chunk (0,0) could not be recovered in regionfile /data/./world/region/r.0.0.mca, it will be regenerated
...

it seems like chunk{x,z} and region{x,z} values are correctly passed to fastnbt

Should be fixed by https://github.com/louis-e/arnis/pull/74 :)


Allow whitelist of keys to be excluded from object-literal-key-quotes

Feature Request
Working with Jackson-serialized classes. I have a "@type" field that I need to quote, but I don't want to quote everything else.
TypeScript code being linted
return {
    "@type": "MyFancyClass",
    someField: "foo",
};

Actual behavior
object-literal-key-quotes reports a failure on someField not being quoted.
Expected behavior
I would like to configure an exception whitelist:
"object-literal-key-quotes": [true, {
    "exclude": ["@type"]
}],

object-literal-key-quotes now options such as "consistent-as-needed" to automatically pick this need up. Closing this for housekeeping purposes. 🎉


Merge main into release 2.6

Summary and Scope
We don't use the main branch for anything on this repo. Especially now that SAT is fully included in CSM 1.6, we don't need to keep the main branch around. This will merge the changes from main into release/2.6, and then we'll make that the default branch for this repo.
Issues and Related PRs

Related to DST-13333

Testing
N/A. No testing is needed here. This is not a functional change. It is just preparing to prune an unused branch.
Risks and Mitigations
No known risks. The main branch isn't used for any builds anymore, and
Pull Request Checklist

[x] Version number(s) incremented, if applicable
[x] Copyrights updated
[x] License file intact
[x] Target branch correct
[x] CHANGELOG.md updated
[x] Testing is appropriate and complete, if applicable
[x] HPC Product Announcement prepared, if applicable

One doubt, Is version updates could be done here?

I'm not sure I understand the question, but I'll try to provide some background.
We used to use the main branch to occasionally create unstable builds of the sat-product-stream. Those builds would include unstable artifacts from the main branches of the other repos. E.g., we could include a build of the cray-sat container image from the main branch of the sat repo. We could do this to test the install of the product prior to creating a stable release. In practice, we didn't use this too much, and eventually we stopped updating the versions in the main branch of sat-product-stream, and we only updated the release/ branches with the stable artifacts.
I think moving forward, this will be even less important because the last release of SAT as its own product is SAT 2.6, so this repository is only going to be used to provide updates to that older version.
If, for some reason, we do need to revive this repository, we could always create a new main branch from the HEAD of release/2.6. But that seems pretty unlikely.
Does that address your question?

I talked with Annapoorna, and she clarified that she thought we needed a .version update. I agreed and in fact meant to do so, but forgot to push that change. Now this will create a 2.6.20 release, but this is functionally the same as 2.6.19, so we won't update the recipe to include that new version.


Bump to version 1.2

Description
Bump OpenSearch and OpenSearch Dashboards plugins to 1.2
Signed-off-by: Kawika Avilla<EMAIL_ADDRESS>Issues Resolved
https://github.com/opensearch-project/dashboards-reports/issues/167
Check List

[ ] New functionality includes testing.

[ ] All tests pass, including unit test, integration test and doctest

[ ] New functionality has been documented.

[ ] New functionality has javadoc added
[ ] New functionality has user manual doc added

[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

CI failed, i'll try to push to your branch to fix it


Collection summary not synced with collection view.

Even after a page refreshes and waiting a couple of hours, "Number of katas" (and "Updated at") doesn't sync between collection summary and actual collection view.
Both screenshots were taken at the same time.
Collection summary:

Collection view:

Problem occurs even if I view them logged out and/or on my phone (mentioning this in case this means something).

I've noticed this with Collections in general ever since they were released on Codewars - it usually takes at least a few days before all pages related to said collection(s) are fully synced.  For example, when I initially add a Description for a collection and click "Save", the collections page still says that a Description is yet to be added.  Subsequent refreshes and logging in/out of Codewars are useless in speeding up the sync process.  Only after a few days when I check the Collection again do I see the Description as expected.  The same goes for any other action on a Collection - reordering, deleting, etc. - there just appears to be a standard lag for at least a few days which is sometimes a bit annoying as I cannot immediately inspect the changes that I made to a Collection.

This should be resolved now.


Make use of cjdns for Prometheus/API/Gateway authentication

gateway at http://[fc12::3456]:8080
api at http://[fc12::3456]:5001
prometheus at http://[fc12::3456]:80 / http://metrics.ipfs.io

All of them require authentication with a whitelisted
cjdns IPv6 address (see secrets.yml).
The other three commits are semi-random fixes.

I think this LGTM.

prometheus at http://[fc12::3456]:80 does it have to be 80 ? maybe we could keep 80 allocated (for nginx)? (would be nicer to reuse the same ports)

That's simply a vhost, which only gets hit if the request's Host header is metrics.ipfs.io -- anything else will hit the gateway

@lgierth ah ok sounds good :


ApolloApi function doesn't have default role after upgrade from v0.35.1 to v0.36.0

Below is the relevant part of my GraphQlApiStack. After upgrading to version 0.36.0 graphqlApiPostFn?.role is falsy because graphqlApi.getFunction("POST /") returns undefined.
const graphqlApi = new ApolloApi(this, "Api", {
  server: {
    handler: "apps/api/src/lambda.handler",
    bundle: { /** */ },
    environment: { /** */ },
  },
})

const graphqlApiPostFn = graphqlApi.getFunction("POST /")

if (graphqlApiPostFn?.role) {
  props.dbInstance.secret.grantRead(graphqlApiPostFn.role)
  someSecret.grantRead(graphqlApiPostFn?.role)
} else {
  throw new Error(`For some reason "POST /" handler doesn't have a role`)
}

I logged graphqlApiPostFn.routes and it shows [ 'GET /' ].
Is this expected with ApolloApi having only one function for both GET and POST?

Well, after being confused for a bit I simply changed from "POST /" to "GET /" since now ApolloApi uses the same function for both routes:
- const graphqlApiPostFn = graphqlApi.getFunction("POST /")
+ const graphqlApiPostFn = graphqlApi.getFunction("GET /")

Maybe an abstraction over Api.getFunction() would make accessing the function more straight forward. Something like a public ApolloApi.function property:
- const graphqlApiPostFn = graphqlApi.getFunction("POST /")
+ const graphqlApiPostFn = graphqlApi.function

@iquabius Good point. Let me put something in.

Hey @iquabius, added serverFunction property for sst.ApolloApi in v0.38.0. Here's a link to the doc - https://docs.serverless-stack.com/constructs/ApolloApi#serverfunction


404 error when trying to open anything except the /public folder

Hello everyone,
Recently I setup Bookstack and followed all the instructions on the bookstack website (https://www.bookstackapp.com/docs/admin/installation/)
I was able to login when I opened the index.php in the public folder but I can‘t view my profile or or do anything else because I always get the message that the requested url was not found.
Which makes sense because I don‘t see a lot folders in the /public folder.
Did I forget something?
I can‘t seem to figure out where my error is.

Hey @DJMcMuffin0
Sorry to hear that you are having trouble.
Did you by chance miss this step?

Set the web root on your server to point to the BookStack public folder. This is done with the root setting on Nginx or the DocumentRoot setting on Apache.

I'd also like you to check / answer the following,

mod_rewrite is enabled and working.
Is BookStack at the root folder, have you set APP_URL option in the .env file properly?

Can you also give us some more details regarding your server and setup?

Hey @Abijeet,
sorry for replying so late. Thank you for your help.
I made sure that mod_rewrite is enabled. Is there a way to check if it is actually working?
The DocumentRoot wasn‘t set to /var/www/bookstack/
I also tried setting the APP_URL in the .env file because I have a couple other things running on that server as well.
Right now the DocumentRoot is back to normal. Just /var/www/
The server is not reachable from the internet. (Only internally)
I set the APP_URL to http://<IP_ADDRESS>/bookstack/

Since the document root should point to the public directory, your APP_URL needs to include the public folder (http://<IP_ADDRESS>/bookstack/public/) and you need to add a RewriteBase to the .htaccess-file in the public directory. Im not sure but I think something like the following should help
RewriteEngine On # should already be there
RewriteBase /bookstack/public/
#rest of existing htaccess file

Please keep in mind the you should not use BookStack like this in a production environment because all files in the folders are exposed to the web. Everyone can download your http://<IP_ADDRESS>/bookstack/.env file and see your database credentials.

I did everything you said. I edited the .env file and corrected the APP_URL. I also edited the .htaccess-file in the public folder.
I still get redirected to http://<IP_ADDRESS>/bookstack/public/login when I try to open http://<IP_ADDRESS>/bookstack/public/ and then I receive a 404 error.

Can you please provide us with the content of the webserver configuation for /var/www (should be somewhere like /etc/apache2/sites-enabled/000-default)
For mod_rewrite to work there needs to be atleast AllowOverride FileInfo instead of AllowOverride None in there.

Thank you guys for helping me out 👍
Here is the 000-default.conf
<VirtualHost *:80>
        ServerAdmin webmaster@localhost
        DocumentRoot /var/www/

        ErrorLog ${APACHE_LOG_DIR}/error.log
        CustomLog ${APACHE_LOG_DIR}/access.log combined

        RedirectMatch ^/$ /bookstack/public/
</VirtualHost>

I figured it out with the help of @lommes
I had to edit the apache2.conf file and add following lines
<Directory /var/www/bookstack/public/>
Options Indexes FollowSymLinks
AllowOverride All FileInfo
Require all granted
</Directory>

Glad you got it working!
But again the hint that you should not use this configuration in production! BookStack is designed to use an own domain with the DocumentRoot pointing to the public directory, so config files are not accessible in the browser.


added support for actions

I haven't had the chance to write unittests for this since I couldn't get the existing pyramid_orb tests running locally. But, I wanted to share this PR early to get feedback on the overall approach.

@erichulser When you have a minute, let me know if you have any further feedback. My most recent changes help to get the tests running in Travis. Thanks!

Sorry, forgot about this PR.  all good!


[bug]: replace or clarify hybrid in prompt documentation

https://github.com/invoke-ai/InvokeAI/blob/76b0bdb6f9aa1cb0b12c2f5ea741997b7abe8c63/docs/features/OTHER.md#L68-L74
seen also here:
https://github.com/invoke-ai/InvokeAI/blob/76b0bdb6f9aa1cb0b12c2f5ea741997b7abe8c63/docs/features/PROMPTS.md#L292-L297
Problems:

People often read this as hybrid is a keyword or relevant to the syntax, which, as far as I know, is not the case.
This is either duplicated content, or two features using the same syntax?
The description in PROMPTS.md is more accurate, describing the result as "blending concepts." The OTHER.md language is "tell the sampler to invest 25% of its effort," and I wouldn't mind if that were just another metaphor, but "sampler" is a technical term and implies things we are not doing in prompt blending.

The detailed prompt blending example does eventually clarify that "hybrid" is a word here, a part of the prompt introducing a third concept that is neither "blue sphere" or "red cube", but only at the very end. The example should either be clear from the start that it is blending three concepts (and the weight of the third is 1.0 by default), or stick to just two.
https://github.com/invoke-ai/InvokeAI/blob/31146eb7975d3755d23b9819ff11b1a8275c25fe/docs/features/PROMPTS.md#L363-L364
Additional context
When searching the code for hybrid https://github.com/invoke-ai/InvokeAI/search?l=Python&q=hybrid
a number of things come up.
Some of them are related to the inpainting model?
There is this, which does pass more stuff to the sampler and was perhaps the original subject of the "tell the sampler" documentation: https://github.com/invoke-ai/InvokeAI/blob/76b0bdb6f9aa1cb0b12c2f5ea741997b7abe8c63/ldm/models/diffusion/shared_invokeai_diffusion.py#L184
but I don't know if that's still live code. It hasn't come up in the diffusers integration.

Good timing. I’m reviewing all the documentation and will fix this ambiguity.

the referenced passages are still in the documentation, stalebot

If I am understanding how this works (the syntax is very hard to parse), it functions as 'prompt editing' (I don't like that term but it's standard), as in A1111 syntax [item 1:item 2:% change] to transition a concept?
It would be nice if we could have a better syntax for this operation. Perhaps (item 1, item 2).switch(% change) (since .blend is already used for full prompt blending)? Or the current .blend() could be adjusted to handle single terms instead of needing full prompt strings. Alternation (switch back and forth every iteration rather than after X% of iterations) would be the only case not covered by that syntax.


Update sbt-guardrail to 0.64.3

Updates com.twilio:sbt-guardrail from 0.64.0 to 0.64.3.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.twilio", artifactId = "sbt-guardrail" } ]

labels: sbt-plugin-update, semver-patch

Superseded by #42.


Agressive optimizations

Removed useless text, added new terminating commands, aggressively optimized declarations

This PR should be closed, i found some problems with bindings

Most of the changes are personal preferences. I recommend that you keep your own fork so you can apply any change you like.


[Config] Add export components and types

I didn't create a unique file index.ts in form/ because when I wanted to import it in general index.ts, I can't import all the types seperatly from the components. And I can't export types like type.
I prefered to import and export them clearly to be sure the use is correct.

This pull request has been deployed to Vercel.

✅ Preview:
https://matou-technologies-purrfect-ui-storybook-24.vercel.app

🔍 Inspect:
https://vercel.com/matou-technologies/purrfect-ui/E2Ztgo1QuYv26A3SH9VpxHzkp2LM


Game Changes Platformer 4x

Change #1:
Drawing the art for the background (visual change):
I used Ibis Paint X to draw this image and I also used a reference.

Switching this out for the old background images:

Change #2
Adding different sound effects:

implenting it in index.md to switch the sound effects.

This was my plan how to originally make my changes - Lara Banjac
Change 1: Changing the sound effects
Why I want to change it: I didn't really like the sound effects in the game so I decided that I could change it. To do that I am going to find audios I want to use in the game. After that I will make sure they are in mp3 format, since it seems like it's a supported format by VSCode, based off of the given sound effects.

Step 1: Go to VSCode and switch the sounds with my new sounds in index.md, since that is where they are located/used.
Step 2: Commit and check if it works.

Change 2: Graphics and Background:
Why I want to change it: I didn't really like the ones used and on top of that, they didn't really match with each other. This why, I will try find a background that I like and follows a certain colour palette, so the that the overall result is more pleasant to look at.

Step 1: Since I recently started doing digital Art, I will draw the background for the level. I believe this is a good idea because, first of all I can create something the exact way I wanted it and second of all I am guaranteed to not get copyrighted since I will be the owner of my art.
Step 2: Find out where the graphics for the game are and replace the old images with my new ones.

Here's a picture of the current game for reference

###Recap - Why I changed this

I changed the sounds and visuals, because it helps us see different versions and options. In addition to that, I liked the idea of drawing something that fits the theme more, unlike the current game that has sprites, images and visuals with no particular colour scheme or general idea.

Things I am going to do better next time

Plan better with my team and work on collaboration: If you want me to be honest, I'd say our collaboration could have been better. Whether it's because some people didn't participate or didn't listen to each other.

Final game:


merge package template from arlac77/npm-package-template

package.json

chore(devDependencies): update<EMAIL_ADDRESS>from template

Codecov Report

Merging #32 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##           master    #32   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           2      2
  Lines          29     29
=====================================
  Hits           29     29

Continue to review full report at Codecov.

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

Coverage remained the same at 97.143% when pulling 819d5802905ad2b8254b5312df544c48b466fc9b on template-sync-1 into 695d5f6726a3354d5f2289cad5d59960fc09683d on master.


[FLINK-24538][runtime][tests] Fix race condition when offering information to leader event queue in TestingRetrievalBase

What is the purpose of the change
There currently exists a race condition during ZooKeeperLeaderElectionTest.testLeaderShouldBeCorrectedWhenOverwritten, which is caused by TestingRetrievalBase.offerToLeaderQueue assigning a value to the base's leader, which happens because of multiple threads offering to the leader queue and this assignment is not synced.
This assignment is not necessary as waitForNewLeader and waitForEmptyLeaderInformation already assign the leader by polling the blocking queue, which offerToLeaderQueue adds to.
This NPE occurs consistently every 20-30 retries of the test. With this fix, I am unable to reproduce the NPE across 1000+ retries.
Brief change log

Fixed race condition caused by unsynchronized leadership assignment in TestingRetrievalBase

Verifying this change
This change added tests and can be verified as follows:
(example:)

Running the  ZooKeeperLeaderElectionTest.testLeaderShouldBeCorrectedWhenOverwritten test until failure.

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

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

Documentation

Does this pull request introduce a new feature? (no)

@XComp I did check that one - I don't believe that a check is strictly necessary for testKubernetesLeaderRetrievalOnModifiedWithEmpty, but I will add a waitForEmptyLeaderInformation call just for consistency's sake.

Test failure is independent of the changes applied in this PR: FLINK-25442


Fix memory leaks related to the glTF loader

There are probably some more to fix, but this should fix most of the cases.

I searched a bit and it doesn't sound like there is a way to check for memory leaks. Any suggestions on how to automate testing this would be great.


Added FAQ about public cloud and national cloud

@vitomaz-msft : Thanks for your contribution! The author, @allenwux, has been notified to review your proposed change.

Thank you @vitomaz-msft :)
#sign-off

@JennieHubbard: I'm sorry - only the author of this article, @allenwux, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.


The closing single quote needs to be indented...

if it's on its own line.
Current output:
% cd libyaml
% echo "+STR
+DOC
+SEQ
=VAL '\\\n
-SEQ
-DOC
-STR" | ./tests/run-emitter-test-suite
- '

'

With this patch:
% echo "+STR
+DOC
+SEQ
=VAL '\\\n
-SEQ
-DOC
-STR" | ./tests/run-emitter-test-suite
- '

  '

Test suite suceeds.
The same rule would apply to double quoted strings, but I haven't been
able to produce a case where the closing double quote was on its own line.

Applied. Pushed. Thanks!


Feature/tutorials page

Description
https://jira.mesosphere.com/browse/DCOS_SITE-160
Urgency

[ ] Blocker
[ ] High
[ ] Medium

Requirements

Test all commands and procedures.
Build content locally and test for formatting/links.
Add redirects to dcos-website/redirect-files.
Change all affected versions (e.g. 1.7, 1.8, and 1.9).
See the contribution guidelines.

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Hi @judithpatudith,
I've setup the tutorials page based on the new demos page. Currently there are no images added, you can add them if you want the same way as on the demos page. (in tutorials.json).
Regards,
Rene


Update Czech translation

Changes:

Add Czech translation of the missing strings

This has conflicts, please resolve it and then we can merge it! Thanks for the contributions!


Update more doc about failsafe ports including 6443

Description

Related issues/PRs

Todos

[ ] Tests
[ ] Documentation
[ ] Release note

Release Note

None required

Deploy preview for calico ready!
Built with commit 1c86505ca60342c114cb938f1441d76d0aa5639e
https://deploy-preview-3586--calico.netlify.app


Terminal output paused when RDP session disconnected

Environment
Windows build number: [run `[Environment]::OSVersion` for powershell, or `ver` for cmd]
Windows Terminal version (if applicable): Microsoft Windows NT 10.0.18362.0

Any other software?  msbuild

Steps to reproduce

Log into a machine over RDP
Open a terminal cmd session
Start a build using msbuild.exe
Close RDP session

Expected behavior
It's expected that the build will continue in the background even after the RDP session is disconnected.

Actual behavior
Terminal output is blocked, causing the build to be blocked as well. This means that the build will not proceed until reconnecting the RDP session or logging into the host machine.

If only Command Prompt is used, does it behave same?

No, this behavior is only in the new terminal. It seems to have some weird interaction with RDP.

Just to confirm: Terminal continues running, it just doesn't complete building until you reconnect?

Yes the terminal stays open, but it seems like stdout is blocked during this time.

Hey this issue is like, 3 years old. You still seeing this/? There's a pile of other renderer deadlocks we're tracking these days, but this doesn't sound like one of them.


Adding GitHub identifier prefix

Description
Including GitHub off-chain prefix identifier to allow mapping of usernames of this repository hosting with addresses in SocialConnect.
Other changes
None
Tested
None
Related issues
None

closing. added changes to https://github.com/celo-org/social-connect/pull/104


Changed Font

Changed Font to the new discord text font
Fixed button padding to look like discord buttons

Unfortunately I can't accept this, because GG sans is not available for open source usage:

Q: Can I use the custom font for my own projects?
A: gg sans is not currently an open-source font, so you are not able to use it at this time.


Doesn't seem to support SB 0.9.4

Note gets created, including title, metadata, etc. But the article content isn't copied over.
Example article: https://selfh.st/newsletter/2024-11-08/
Tried with both Firefox and Brave
SB: 0.9.4
Clipper: 1.2.4

You are correct that it does not copy the entire page over, but if you highlight some text on the page with the mouse and then clip, it will add the highlighted text to the new note.

Thanks for answering @dklawren. This is correct.

I think it might be good to document this.
The README says:

To use this add-on, simply click the extension icon while you are browsing the page you want to capture.

So I expected it to behave like other clippers (Obsidian's new clipper extension for example), where the extension attempts to automatically pick up the article content.

Thanks for the feedback Mufeed. I've updated the readme to be more clear.

Thanks! Loving this btw :)


Feature Request: Add Emmet support

Is your feature request related to a problem? Please describe.
Writing HTML is slow, and would be much faster if I could use emmet expansions.
Describe the solution you'd like
Support for Emmet expansions (examples here https://code.visualstudio.com/docs/editor/emmet).
As a basic example, I could then type
.my-element#id

After pressing tab, it would be expanded to

It seems like we don't need to do anything. The following .vscode/config.json works (This is what the other extension does:)
{
      "emmet.triggerExpansionOnTab": true,
      "emmet.includeLanguages": {
        "liquid": "html"
      }
}

There's just one thing I don't like though: this adds the emmet completion options everywhere. Even when you're typing Liquid code. They also appear to have a very high priority in the completion window.

Maybe we simply tell people through this issue that you can turn that on yourself? There would be 0 code changes other than switching this to on by default with the extension.

🤦 Wish I'd realised it was that simple.
Also I'd never noticed the completions being included within Liquid contexts before. Can't be unseen and I totally agree that I don't like that. I imagine letting people know they can enable it this way if they want to, with this caveat is fine for now, but think it would be better enabled by default in the future if the issue you raised can be resolved.


remove TokenListEditor from public API; get rid of hybrid.util package

I'd also like to remove BaseHybridSynchronizer from public API via replacing it with HybridSynchronizer interface, but not now.
There are corresponding pull requests to ot and datapad:
https://github.com/JetBrains/jetpad-ot/pull/50
https://github.com/JetBrains/datapad/pull/6

The indentation below is not according to our code convention:
 public HybridSynchronizer(Mapper<?, ?> contextMapper, Property<SourceT> source, Cell target,                             HybridEditorSpec<SourceT> spec) {

Looks fine. You can merge it in all repos. I don't have the write access to the ot repo.


Question about facil.listen() argument

Hi there,
I'm writing a simple websocket server based on a sample from the site.
I needed to change some rows of code to compile, 'cause they have a different prototipes.
In particular
facil_listen(
	  .port = "7681",
		  .on_open = create_protocol,
Now create_protocol() is
static inline void create_protocol(intptr_t uuid, void * arg) {
returns nothing and then I'm wondering how to return the pointer to protocol_s structure.
In this way the connection is not possible.
Thanks!

Hi @SteMMo33 ,
I'm sorry for the late response, I'm on vacation and don't have access to my computer on a regular basis.
I think I see the problem, but I'm not not sure... I don't know what example you are pointing to, but it could be that the example uses the 0.5.x version API instead of the 0.6.x version API.
In the new API please use the facil_attach function to connect the uuid to the protocol_s object.
Please note that the facil_attach function should be called within the .on_open callback.
If you have any more questions, please let me know. I'll answer as soon as I can.
Kindly,
Bo.

Hi Bo,
thanks a lot for your reply.
The example is the basic echo server.
I added the facil_attach function and now the app is alive.
I'd like to handle some websocket messages sent by a browser. If I open the connection from a browser the following is the output to the console (I've added some debug fprintfs):
mora@ubuntu-virtualbox:~/Desktop/Socket/facil.io$ LD_LIBRARY_PATH=. ./ioServer
* Listening on port 7681
onProtStart
Server is running 1 worker X 10 threads, press ^C to stop
* Detected capacity: 4096 open file limit
* Root pid: 4736
New connection 0x7f0238000b20 for socket UUID 0x801
protOnReady 0x7f0238000b20 0x801
protOnReady 0x7f0238000b20 0x801
data 0x7f0238000b20 0x801:
<Echo: GET / HTTP/1.1
Host: <IP_ADDRESS>:7681
User-Agent: Mozilla/5.0 (X11; Ubuntu; Linux x86_64; rv:61.0) Gecko/20100101 Firefox/61.0
Accept: text/html,application/xhtml+xml,application/xml;q=0.9,*/*;q=0.8
Accept-Language: en-GB,en;q=0.5
Accept-Encoding: gzip, deflate
Sec-WebSocket-Version: 13
Origin: null
Sec-WebSocket-Protocol: lws-minimal
Sec-WebSocket-Extensions: permessage-deflate
Sec-WebSocket-Key: BOw0PlOXkfSD4ZSiDzBxqA==
Connection: keep-alive, Upgrade
Pragma: no-cache
Cache-Control: no-cache
Upgrade: websocket

>
protOnReady 0x7f0238000b20 0x801
protOnReady 0x7f0238000b20 0x801
Freed Echo protocol at 0x7f0238000b20

In my understanding the library shoud handle the initial autentication for the websocket, is it?
I'd like to handle only the hi-level messages between server and browser.
Is it possible with your library??
what is the purpose of the onReady callback ?
thanks again,
regards :)

@SteMMo33 Hello,
Yes, the example code you chose is from the core module (included through facil.h). The example uses plain TCP/IP sockets (without the extra WebSocket protocol).
The WebSocket protocol is part of the HTTP and WebSockets module, which you can use by including the http.h header.
A WebSockets example should available on the Website.
Like many of the examples, the API in the example is dependent on the release version.
I believe the API on the website reflects the latest 0.6.x release, but it might be updated soon to use the master branch (the upcoming 0.7.0 release API) which includes some changes to the pub/sub logic and API.
As for the on_ready callback:
The on_ready callback is "called when the socket is ready to be written to". Which basically means that once the user buffer is empty and the kernel's socket buffer has some empty space (all the previous calls to sock_write are done writing to the socket's outgoing buffer), the callback is called.
The callback behaves the same with WebSocket connections, which use websocket_write to "pack" and send the data using the WebSocket protocol.
You don't have to use it.
Bo.


Metronome Battle: Ban Pokestar Spirit

Why Pokestar Spirit specifically?

It's the one mon that is turning the meta really stale due to its typing and BST. Fairy type attacks barely get called by metronome so


Saving markdown after mutating task list

Markwon version: v4.6.2

How can we save the markdown file after mutating task list using this sample code

Hello @damionx7 ,
markdown is parsed one-way, so after it is parsed the input is not longer referenced. So there is no way to rebuild raw markdown from the data displayed in a TextView. You could maintain own datastore for tasks and identify them by raw text. But this would mean that markdown is dynamically generated from this datastore, which might not be optimal

@noties Hi !Do you mean,I can not parse EditTest.getText()to Markdown format string?The text style in EditText is formated by some Spanned.


When user_id is not found, raise an specific error message

I believe this is fixed in 0.2.4: https://github.com/SeasonedSoftware/croods-rails/commit/693d3af01802e42158542765dd8bf92947676a41


[improve][ml] Filter out deleted entries before read entries from ledger.

PIP:  I'm not sure if the PR needs a PIP
Motivation
In https://github.com/apache/pulsar/pull/19035 we introduced skipCondition before read entries from ledger.
It filters out delay-messages for the purpose of optimizing performance: to avoid reading unnecessary entries  to save memory usage, network usage and reduce latency of reading entries.
As the same purpose, this PR wants to filter-out deleted entries before reading entries from ledger to optimize performance.
This PR won't change the semantics of readEntries , it  just made some enhancements.
It may brings huge performance optimization in the individualDelete mode, especially when bookie client metadata loss, bookie auto-recover disabled and some bookie nodes down.
Modifications

Verifying this change

[ ] Make sure that the change passes the CI checks.

(Please pick either of the following options)
This change is a trivial rework / code cleanup without any test coverage.
(or)
This change is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:
(example:)

Added integration tests for end-to-end deployment with large payloads (10MB)
Extended integration test for recovery after broker failure

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

If the box was checked, please highlight the changes

[ ] Dependencies (add or upgrade a dependency)
[ ] The public API
[ ] The schema
[ ] The default values of configurations
[ ] The threading model
[ ] The binary protocol
[ ] The REST endpoints
[ ] The admin CLI options
[ ] The metrics
[ ] Anything that affects deployment

Documentation

[ ] doc
[ ] doc-required
[x] doc-not-needed
[ ] doc-complete

Matching PR in forked repository
PR in forked repository: https://github.com/dao-jun/pulsar/pull/4

/pulsarbot run-failure-checks

It may brings huge performance optimization in the individualDelete mode, especially when bookie client metadata loss, bookie auto-recover disabled and some bookie nodes down.

Could you please explain more about this line? I don't understand why it pertains to bookie client metadata loss, the disabled auto-recovery and bookies unavailability.

It may brings huge performance optimization in the individualDelete mode, especially when bookie client metadata loss, bookie auto-recover disabled and some bookie nodes down.

Could you please explain more about this line? I don't understand why it pertains to bookie client metadata loss, the disabled auto-recovery and bookies unavailability.

Because from my understanding, BK pursues the reading semantics of AT_LEAST ONCE, bk-client's metadata and bookie's auto-recovery are to be able to read only once as much as possible. When client's metadata lost or bookie auto-recovery disabled, bookie client may take multiple readings to read the data. In this situation, if we filter-out deleted messages before read, it should be able to significantly reduce latency.

@dao-jun
Do we still need to make changes to ManagedLedgerImpl.java? As I understand, if we applied the combined skip condition, we will leverage the existing skip condition support in ManagedLedgerImpl, right? If the skip condition support in ManagedLedgerImpl need to improve or fix. Is it possible to move to another PR to keep this PR well-scoped.

@dao-jun
Do we still need to make changes to ManagedLedgerImpl.java? As I understand, if we applied the combined skip condition, we will leverage the existing skip condition support in ManagedLedgerImpl, right? If the skip condition support in ManagedLedgerImpl need to improve or fix. Is it possible to move to another PR to keep this PR well-scoped.

Yes, because of message skipCondition, one single segment can be spit into segments, such as:
entries to be filter-out: [3, 5, 7]
entries to read: [1, 10]
then, it will be split into: [[1,2],[4], [6], [8,10]], right?
In the previous implementation, after read [1,2] finished, then begin to read [4], after read [4] finished, then start to read [6]...
I believe it will lead to latency increasing, memory(allocated for entries) will also be retained for a longer period of time, it will affect the throughput of the system.
So I tried to read these segments parallel, and it would make changes to ManagedLedgerImpl.

Yes, because of message skipCondition, one single segment can be spit into segments, such as:
entries to be filter-out: [3, 5, 7]
entries to read: [1, 10]
then, it will be split into: [[1,2],[4], [6], [8,10]], right?
In the previous implementation, after read [1,2] finished, then begin to read [4], after read [4] finished, then start to read [6]...
I believe it will lead to latency increasing, memory(allocated for entries) will also be retained for a longer period of time, it will affect the throughput of the system.
So I tried to read these segments parallel, and it would make changes to ManagedLedgerImpl.

@dao-jun Is it better to split them into 2 PRs? The current one is focused on filter-out deleted entries before reading entries and the another one for making the reads parallelly. I haven't checked the details yet, but we should be carefully to make this improvement to avoid any regression to the ordering dispatch.

@dao-jun Is it better to split them into 2 PRs? The current one is focused on filter-out deleted entries before reading entries and the another one for making the reads parallelly. I haven't checked the details yet, but we should be carefully to make this improvement to avoid any regression to the ordering dispatch.

@codelipenghui It makes sense, I'll do it later.

@codelipenghui @coderzc PTAL

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (22838ea) 36.46% compared to head (276358b) 73.58%.

Additional details and impacted files

@@              Coverage Diff              @@
##             master   #21739       +/-   ##
=============================================
+ Coverage     36.46%   73.58%   +37.11%
- Complexity    12390    32381    +19991
=============================================
  Files          1725     1861      +136
  Lines        131701   138596     +6895
  Branches      14401    15187      +786
=============================================
+ Hits          48027   101985    +53958
+ Misses        77254    28700    -48554
- Partials       6420     7911     +1491

Flag
Coverage Δ

inttests
24.14% <0.00%> (+0.02%)
:arrow_up:

systests
23.71% <0.00%> (+0.04%)
:arrow_up:

unittests
72.86% <100.00%> (+40.81%)
:arrow_up:

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

Files
Coverage Δ

...che/bookkeeper/mledger/impl/ManagedCursorImpl.java
79.22% <100.00%> (+30.88%)
:arrow_up:

...che/bookkeeper/mledger/impl/ManagedLedgerImpl.java
80.53% <ø> (+32.69%)
:arrow_up:

...he/bookkeeper/mledger/impl/ReadOnlyCursorImpl.java
95.00% <100.00%> (+95.00%)
:arrow_up:

... and 1435 files with indirect coverage changes

approve

I have added the cherry-pick labels since this improvement fixed the issue which can impact the bookies if the cursor rewind to the earlier position (the mark delete position can't move forward due to individual acks).


fix: flush byte is not covered in decode

I'm currently working on extended query protocol, but every time I get a Flush message, decode returns an error.
It looks like someone forgot to cover MESSAGE_TYPE_BYTE_FLUSH in PgWireFrontendMessage::decode, and this patch fixes that.

Nice! Always let me know when you have any issue with extended query implementation.


Extracting single value yields error

I'm attempting to extract a single value out of json in decode function. Doing so in two ways yields an error
let foo: String = json <| "foo" # Binary operator '<|' cannot be applied to operands of type 'JSON' and 'String'

let bar: String = json["bar"] # 'JSON' does not have a member named 'subscript'

I'm using Argo 1.0.3 with Swift 1.2 / Xcode 6.3.2.

The <| operator returns Decoded<T>. If you'd like to pull out the String and don't care about the error state you could call .value like so: let foo: String? = (json <| "foo").value

Thanks, it'd be good to add this information to README.

@eimantas glad that worked for you. We're trying to get better documentation done all the time, but it's slow going.


Cluster doesn't respect minimum distance

Describe the bug
Features are closer together than specified minimum distance.
Version 6.5.0
To Reproduce
Open Cluster example from documentation, potentially zoom in/out or slightly adjust distance to see overlap:
https://openlayers.org/en/v6.5.0/examples/cluster.html
Features have a radius of 10 pixels. Distance is at 40 pixels. Still some features overlap:

Expected behavior
My expectation would be that the centers of the clusters are at minimum distance pixels apart, as stated in the documentation:
https://openlayers.org/en/v6.5.0/apidoc/module-ol_source_Cluster-Cluster.html

Thanks for confirming the bug. Is it planned to repair the clustering algorithm?
Can you expose the "center interpolation" tunable?
If not, the documentation is broken and needs adjustment, clarifying that the "minimum distance" is no minimum distance.
If I understand the current implementation description correctly, I can never guarantee that cluster icons do not overlap, regardless how high the distance setting is set.

A minimum distance is indeed not guaranteed. Depending on feature distribution clusters may more or less overlap.
I am proposing a solution with #12080.


remove include_deleted from protocol

I removed it from the client in https://github.com/simon-weber/gmusicapi/issues/513, but I didn't have the chance to clean it up all the way.

Hi. Do you perhaps know if it's anyhow possible to get songs that disappeared from playlists (due to label/GPM licensing changes)? By getting, I would just like to get its names and position within the playlist.
Thanks!

Hm, I'm not sure. I don't think I've ever seen an example of that. I suspect they would show up in plentries, but it doesn't look like that call is actually in gmusicapi (it has plentryfeed and I can't remember how they differ). It's in Autoplaylists, though, if that's helpful.

Tnx; I'll see what I can do with that...
On Wed, 5 Feb 2020 at 17:35, Simon Weber<EMAIL_ADDRESS>wrote:

Hm, I'm not sure. I don't think I've ever seen an example of that. I
suspect they would show up in plentries, but it doesn't look like that
call is actually in gmusicapi (it has plentryfeed and I can't remember
how they differ). It's in Autoplaylists
https://github.com/simon-weber/Autoplaylists-for-Google-Music/blob/25efa46bbb6815eaf3cba69122c308f5f05257bd/src/js/googlemusic_oauth.js#L186,
though, if that's helpful.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/simon-weber/gmusicapi/issues/518?email_source=notifications&email_token=AGH2RWI45EX6FICTK762NMLRBLTDFA5CNFSM4CZHD6QKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEK4CUBQ#issuecomment-582494726,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AGH2RWP4FTUQXL6BE3BKLIDRBLTDFANCNFSM4CZHD6QA
.


Change color of dialog fade out

How do I change the color of the overlay outside the dialog but inside the application window? Using dark theme this getting a white fade out color which I would like to change to a gray color.

@Symbai You can change this with this property on MetroWindow
https://github.com/MahApps/MahApps.Metro/blob/cb9264828def32d2c55ec32dc0426c8a72aa793d/src/MahApps.Metro/Controls/MetroWindow.cs#L722-L729


docs: link repair, misc cleanup

What I did

fixed a couple broken relative paths

today, the projects guide link returns a 404 when visited via the README or published docs

misc. cleanup for more consistent markdown use

How I did it
within a time-box, i couldn't figure out how to make GitHub relative paths satisfy both links served via the README on GitHub and via the published gh-pages docs. Opted for the hard-coded docs link, as is used elsewhere in the README.
How to verify it

verify the links in the README of my forked repo
build the docs locally or via CI and test the relevant links in the Quick Start guide

Checklist

[x] All changes are completed
[x] New test cases have been added
[x] Documentation has been updated

Thanks @marcgarreau !


Is it possible to perform inference using your paper on GrayScale Stereo pairs

I have a stereo camera that produce only grayscale stereo pairs.
I'm wondering if we can use your amazing algorithm to do inference and evaluate the results.
The images are taken using head mounted camera and from a person face

[NOTICE] Zhang Jiawei(Gary) deleted my workspace without notice, which containes data and network models of a paper under review.


Incomplete import on multi-accounts set-up

Hello,
I have a multi-account set-up on my Google My Business and I need to extract insights and reviews to analyse them on BigQuery.
I have mounted the alligator2 repo on a VM with Compute Engine.
Nevertheless my last import, which was completed without errors gave out uncompleted results.
As a patch, we tried to download the data on an account by account basis, which lead to incomplete results as well.
What is the most surprising is that on previous import, the account by account way was functionnal.
Thank you for that repository which is of great help.

How were the results incomplete? Were you missing accounts or reviews?

Hello i had a lot of errors with GMB Api who seems to be not very happy when yours reviews are too many (above 5000) and started to send 500 error... to avoid that, i modified reviews() fonction in api.py by catching the error and go to the next location.
its a quick fix (a ticket has been sublitted to GMB api) but at least, i can manage to get all y 550 locations over 4 directions...
`
def reviews(self, location_id):
page_token = None
while True:
  try : response_json = self.gmb_service.accounts().locations().reviews().list(
      parent=location_id,
      pageToken=page_token).execute(num_retries=MAX_RETRIES)
  except HttpError :
    logging.error("============= "+location_id+" has a too big list ==========")
    break #go to next location
  try : data = response_json.get("reviews") or []
  except HttpError :
    logging.error("============= "+location_id+" has too much reviews ==========")
    break #go to next location

  logging.debug(json.dumps(data, indent=2))
  self.to_bigquery(table_name="reviews", data=data)
  page_token = response_json.get("nextPageToken")
  if not page_token:
    break

`
I still don't know how to propose my api.py version in github (yes, i know) but if the devellopers want to put it in ther code, no soucy 👍

Hello,
The problem was due to the expiration of the session of the Cloud shell, set to 20 minutes.
For the same reason when running from Compute Engine, the import wasn't complete.
But when run with a closer attention, the import was successful.
Thanks a lot.


Move period inside quotation

Why:
Move period inside quotation to align with style guide
Closes: #33344
What's being changed (if available, include any code snippets, screenshots, or gifs):

Move the period on line 111 to the inside of the ending quotation

Check off the following:

[x] I have reviewed my changes in staging, available via the View deployment link in this PR's timeline (this link will be available after opening the PR).
[x] For content changes, I have completed the self-review checklist.

@SeanAverS Thanks so much for opening a PR with this fix! I'll update the branch and get this merged once tests are passing 🍏


Help Wanted: Not able to upgrade to 1.6.0

Describe the bug
When tring to upgrade from 1.5.4 to 1.6.0
the compute node keep crash due to probe failed, but no other error logs in compute node and meta node

Error message/log
No response
To Reproduce
No response
Expected behavior
No response
How did you deploy RisingWave?
helm upgrade
The version of RisingWave
1.6.0
Additional context
No response

and k8s version:

It's a cluster issue not related to risingwave, close it

Still have this issue, the compute pods keep crash with exist code 1 after startup, no other message collected

I have try to enlarge the probe timeout and delay time, but not help

the compute pods keep crash with exist code 1 after startup

what's the log of RW
If RW crashes, there must be some error or panic logs.
what's the status of pods, what does it say in the REASON field?

I just rerun the upgrade test for v1.5.4 and v1.6.0, and it works as expected.
Normally, if the upgrade fails, there should be some logs about certain inconsistencies, e.g.:

key values in the old meta cannot be recognized by the new meta
key values in the state of old operators cannot be recognized by the new operators

They must trigger some panic and will be shown in the log.

I just rerun the upgrade test for v1.5.4 and v1.6.0, and it works as expected.
Normally, if the upgrade fails, there should be some logs about certain inconsistencies, e.g.:

key values in the old meta cannot be recognized by the new meta
key values in the state of old operators cannot be recognized by the new operators

They must trigger some panic and will be shown in the log.

Thank you for the test,
I didn't collect any panic log from compute node, maybe it's another new instance up, or somthing
Based on your test, later today(once our tester not using current env) I will start another upgrade to see if there any usefull log

Hi @sheltonsuen
could you check if "connector node" is still set to replica "1" in the yaml file?
If it is, please delete it or set the replica to 0.
The connector node is embedded into compute node, and no longer need to be set explicitly.

Hi @sheltonsuen
could you check if "connector node" is still set to replica "1" in the yaml file?
If it is, please delete it or set the replica to 0.
The connector node is embedded into compute node, and no longer need to be set explicitly.

It's working after setting replica to 0, thanks

FYI, connector node should be disabled via
connectorComponent:
  enabled: false


bug: com.intellij.diagnostic.PluginException: ActionUpdateThread.OLD_EDT is deprecated and going to be removed soon. 'co (...)

Plugin version: 5.3.27
IDE version: WS-241.9959.35
Stacktrace:
com.intellij.diagnostic.PluginException: `ActionUpdateThread.OLD_EDT` is deprecated and going to be removed soon. 'com.sourcegraph.cody.statusbar.CodyEnableLanguageForAutocompleteAction' must override `getActionUpdateThread` and chose EDT or BGT. See ActionUpdateThread javadoc. [Plugin: com.sourcegraph.jetbrains]
	at com.intellij.diagnostic.PluginProblemReporterImpl.createPluginExceptionByClass(PluginProblemReporterImpl.java:23)
	at com.intellij.diagnostic.PluginException.createByClass(PluginException.java:90)
	at com.intellij.diagnostic.PluginException.reportDeprecatedUsage(PluginException.java:125)
	at com.intellij.openapi.actionSystem.ActionUpdateThreadAware.getActionUpdateThread(ActionUpdateThreadAware.java:21)
	at com.intellij.openapi.actionSystem.AnAction.getActionUpdateThread(AnAction.java:198)

I'm closing this one as we are already working on a fax for this bug as part of the https://github.com/sourcegraph/jetbrains/issues/610
It will be fixed in a release which will be available in a couple days from now.


improve error messages and docs for backup crypto key

Currently the error messages and docs are not clear enough about the backup crypto key. The requirements are that it has a fixed size of 32 bytes, and cannot be all zeroes. Yet none of this is communicated explicitly to the user, and can only be found by digging through the code.

Error messages should clearly say the reason why the crypto key is invalid, rather than just saying "crypto key is invalid";
Docs about the import/export zeroctl commands should mention these requirements;

Fixed by 3b1425b785ed564758417be5fbe0907c04b71afb.


Increase time for fetching orders

I have fetched the orders and order details with products as well. But when i set it to -24 hours my script have script timeout issue .  When i set -2 hours then it works fine. I have around 2500 orders a day from amazon. Can you help me how can i remove the script timeout issue.

I have one more question : Is our script fetches Replacement Order. Please let me know. I do not want Replacement Order to be fetched.

The script timeout problem is likely part of your PHP settings. The library uses cURL with CURLOPT_TIMEOUT set to 0, so the library will wait as long as your PHP setup allows it to.

Where i can set CURLOPT_TIMEOUT to 0?

As for your other question, I do not know for sure. I just checked the API documentation, and it seems they have added new data fields named IsReplacementOrder and ReplacedOrderId, but the library does not yet have support for these fields. If you need this information, you can use getLastResponse to view the raw XML.

I think script does not support PromotionDiscount field also. Please confirm.

The library already has CURLOPT_TIMEOUT set to 0, it is your other PHP settings that need to be changed.

Where i can find the CURLOPT_TIMEOUT 0. I need to check.

The library does support PromotionDiscount. You can access it by using getPromotionDiscount in the AmazonOrderItemList class.

Where i can check the CURL client is enabled. I have not found this in library.

https://github.com/CPIGroup/phpAmazonMWS/blob/stable/includes/classes/AmazonCore.php#L763
This is where cURL is being used. This is as far as I can help you regarding the timeout problem.

Ok. What you suggest about PHP setting?

Sorry, I can't help you with your personal PHP settings.

Hello @Peardian , I have one question. Is our API fetches multiple products with Single order Id. Please let me know.

Are you referring to the AmazonOrderItemList  class's fetchItems method? If so, yes, it fetches all of the order's items at once, as indicated by the documentation. I suggest reading through the methods and documentation for classes you are interested in, as this may answer other questions you might have about using this library.

So it will automatically fetched multiple products with single order ID?

As I said, yes. I highly suggest reading the documentation in each class, as it describes what the class can do and may answer other questions you have.


[BUG] Icon Screens, Screen Time... not honored?

Bug report
Calling various icon screens from Home Assistant, it would appear that the screen times are not being honored.  I'd like to keep my icon screens short but I can see in the code that the default is 5 seconds and shortening the time in Home Assistant doesn't change that...
      - service: esphome.pixelclock_yellow_icon_screen
        data:
          default_font: true
          icon_name: >-
            weather_{{ states('sensor.openweathermap_condition')|replace("-",
            "_") }}
          text: "{{ states('sensor.openweathermap_temperature')|round(1) }}°"
          lifetime: 5
          screen_time: 3
          r: 192
          g: 192
          b: 192

I also tried this in my YAML and it doesn't lengthen the default clock screen:
esphome:
  comment: "EHMTXv2 from LuBeDa"
  name: $devicename
  on_boot:
    then:
      - ds1307.read_time:
      - delay: 30s
      - lambda: |-
          id(rgb8x32)->set_clock_time(30);

So far, I fixed this by simply doing this in my Home Assistant (deleting the default clock screen and inserting my own):
  - service: esphome.pixelclock_yellow_del_screen
    data:
      icon_name: "*"
      mode: 2
  - service: esphome.pixelclock_yellow_clock_screen
    data:
      default_font: false
      lifetime: 5
      screen_time: 20
      r: 64
      g: 128
      b: 255

Sidenote that I miss one of the old features of clock intervals from v1... that the default clock screen would display a minimum of every x seconds (that was clock intervals)... this made it easier for me to force every 2nd screen to be the clock (ie. Clock, Weather, Clock, Inside Temperature, Clock, Humidity, etc.)  You know what I mean?  It easy enough just to change the clock interval to 5 seconds, info screens 5 seconds, forcing my preferred behavior... I can ignore losing that if I can speed through all my info screens...

I will investigate the bug and think about a solution for the lost feature!

OK, I found the error.
What would be the best way, thinking of two possible text length?
For short text, that fits without scrolling:

use the screen time?
For long text, that has to be scrolled:
use screen time, regardless if the text is scrolled at least once?
use the maximum of screen_time or the time it takes to scroll the text scroll_count times?
use the time it takes to scroll the text scroll_count times?

The bold lines are my favorite options.

Please check again, the clock_intervall will follow next weekend

Hi, you can try the develop branch for testing clock_interval. By default, it is disabled!

Switched to the develop branch, added this to my YAML:
  on_boot:
    then:
      - ds1307.read_time:
      - delay: 30s
      - lambda: |-
          id(rgb8x32)->set_clock_time(8);
          id(rgb8x32)->set_clock_interval(2);

Then set my icon screens to 3 seconds... It seems to behave a little different than the old interval.  Icon screen will finish out whatever its time was set to before returning to the clock.
So, ie I set the interval to 5 and Icon screens set to 4 each, then 2 icon screens will go by before returning to the clock.
In other words, it works perfectly!
BTW, I'm guessing that this clock is actually the default clock and adding any clock screens may interfere with this expected behavior.  I haven't had time to test that out.  I just removed the clock screens from my Home Assistant automation before playing around.
Awesome job!


Change refernece format in DB to allow for not only dois

update docs with new input mask
modify database to the new format
Update website generation to use the new reference format

Closes #67


Hotswap broken?

I have subfolders in some of my apps. Hotswap will reload for index.js but not for my other js files in the subfolders. How can I get hotswap to listen to all files in my app's hierarchy?

From the hotswap readme...
And then add this line to modules you want to be reloaded (it tells node-hotswap to watch this file):
module.change_code = 1;

I get the message hotswap reloaded but I don't see those changes until I kill the server and start it again. :/

I can second that, lets reopen this issue.

same here
i also tried adding a "console.log()"  to the index.js file and it dos seem to load the module and print the log message but does not change the in-memory app

Thanks, @aaronrogers. I have the same issue - glad you recorded it!

There is another module called hotmop. While I haven't tried it myself, it's worth a try in place of node-hotswap.

seems like it doesn't usually work here, either

Not working for me either.

Not working for me either.
OS: macOS 10.12.5
Node: v6.10.3
NPM: 3.10.10

Looking for some PRs!

It's not totally ideal in terms of efficiency, but for most use-cases node-dev is sufficient during dev:
npm install -g node-dev
node-dev server.js

On change, the server reboots and the app-server restarts.


remaining WWT issues in solara update

[ ] remove chrome (set so it doesn't appear by default then disappear)
[ ] use SDSS as background for stages 1 & 3
[ ] SDSS galaxy table in Stage 1 only seems to load the first time a user accesses the solara app. If a user didn't select all 5 of their galaxies on that first load, there is no way for them to add to their dataset.

These should be fixed on https://github.com/nmearl/hubbleds/pull/1 in conjunction with https://github.com/nmearl/ipywwt/pull/4 on @nmearl's branch

In my testing this all worked well, I didn't see any issues. The only concern that I have is whether or not it's possible for the message from the mounted hook to be emitted before the Python handler is set up. My hunch is that this won't happen, but I couldn't convince myself that it's impossible.
I might try and set up a ping-pong message based version of this, mostly since the fewer deviations we have from the upstream research app the easier the easier this will be to integrate later on. But I think this is fine for now.


Rebrand fyde-cli to access-cli

TODO:

[ ] Move repo name to access-cli
[ ] Test deb upgrade
[ ] Fix pipeline
[ ] Test everything

Heads up for this related repo https://github.com/fyde/homebrew-tap


Groupby benchmark fails after PR #2317 merge

System information

OS Platform and Distribution (e.g., Linux Ubuntu 16.04):

Ubuntu 20.04.1 LTS

Modin version (modin.__version__):

<IP_ADDRESS>+16.g3e32d02

Python version:

Python 3.8.6

Code we can use to reproduce:

python -u -m pytest --log-cli-level=info --benchmark-sort=name --benchmark-verbose --benchmark-json=results.json -r A -s --tb=native 'ci/benchmarks/test_benchmarks.py::test_groupby_sum[agg(sum)-no_indexing-10000x10000-int-modin]'
Benchmark fails with exception:
Traceback (most recent call last):
  File "/localdisk/gashiman/modin/ci/benchmarks/test_benchmarks.py", line 206, in test_groupby_sum
    result = benchmark.pedantic(
  File "/nfs/site/home/gashiman/.local/lib/python3.8/site-packages/pytest_benchmark/fixture.py", line 139, in pedantic
    return self._raw_pedantic(target, args=args, kwargs=kwargs, setup=setup, rounds=rounds,
  File "/nfs/site/home/gashiman/.local/lib/python3.8/site-packages/pytest_benchmark/fixture.py", line 213, in _raw_pedantic
    runner(loops_range)
  File "/nfs/site/home/gashiman/.local/lib/python3.8/site-packages/pytest_benchmark/fixture.py", line 87, in runner
    sys.settrace(None)
  File "/localdisk/gashiman/modin/ci/benchmarks/test_benchmarks.py", line 170, in benchmark_groupby_agg_sum_function
    result = gb.agg(sum)
  File "/localdisk/gashiman/modin/modin/pandas/groupby.py", line 399, in aggregate
    result = self._apply_agg_function(
  File "/localdisk/gashiman/modin/modin/pandas/groupby.py", line 887, in _apply_agg_function
    new_manager = groupby_qc.groupby_agg(
  File "/localdisk/gashiman/modin/modin/backends/pandas/query_compiler.py", line 2633, in groupby_agg
    new_modin_frame = self._modin_frame._apply_full_axis(
  File "/localdisk/gashiman/modin/modin/engines/base/frame/data.py", line 1301, in _apply_full_axis
    return self.broadcast_apply_full_axis(
  File "/localdisk/gashiman/modin/modin/engines/base/frame/data.py", line 1673, in broadcast_apply_full_axis
    new_axes = [
  File "/localdisk/gashiman/modin/modin/engines/base/frame/data.py", line 1674, in <listcomp>
    self._compute_axis_labels(i, new_partitions)
  File "/localdisk/gashiman/modin/modin/engines/base/frame/data.py", line 289, in _compute_axis_labels
    return self._frame_mgr_cls.get_indices(
  File "/localdisk/gashiman/modin/modin/engines/ray/pandas_on_ray/frame/partition_manager.py", line 99, in get_indices
    new_idx = ray.get(new_idx)
  File "/localdisk/gashiman/miniconda3/lib/python3.8/site-packages/ray/worker.py", line 1452, in get
    raise value.as_instanceof_cause()
ray.exceptions.RayTaskError(TypeError): ray::deploy_ray_func() (pid=4067967, ip=<IP_ADDRESS>)
  File "python/ray/_raylet.pyx", line 446, in ray._raylet.execute_task
  File "python/ray/_raylet.pyx", line 468, in ray._raylet.execute_task
ray.exceptions.RayTaskError: ray::deploy_ray_func() (pid=4067969, ip=<IP_ADDRESS>)
  File "python/ray/_raylet.pyx", line 482, in ray._raylet.execute_task
  File "/localdisk/gashiman/modin/modin/engines/ray/pandas_on_ray/frame/axis_partition.py", line 105, in deploy_ray_func
    result = func(*args)
  File "/localdisk/gashiman/modin/modin/engines/base/frame/axis_partition.py", line 224, in deploy_axis_func
    result = func(dataframe, **kwargs)
  File "/localdisk/gashiman/modin/modin/engines/base/frame/data.py", line 1036, in _map_reduce_func
    series_result = func(df, *args, **kwargs)
  File "/localdisk/gashiman/modin/modin/backends/pandas/query_compiler.py", line 2634, in <lambda>
    axis, lambda df: groupby_agg_builder(df)
  File "/localdisk/gashiman/modin/modin/backends/pandas/query_compiler.py", line 2627, in groupby_agg_builder
    return compute_groupby(df)
  File "/localdisk/gashiman/modin/modin/backends/pandas/query_compiler.py", line 2618, in compute_groupby
    result = agg_func(grouped_df, **agg_kwargs)
  File "/localdisk/gashiman/modin/modin/utils.py", line 123, in wrapper
    result = func(*args, **kwargs)
TypeError: unsupported operand type(s) for +: 'int' and 'tuple'

There is a problem with fixing this bug. When as_index=False Pandas moves by column to the first place in result DataFrame. This rearrangement is done for all .agg calls of DataFrameGroupBy object. But this rearrangement doesn't happen for .apply and .dtypes calls of DataFrameGroupBy object.
Currently we implement all three mentioned and some other calls using the same function _apply_agg_function which has no way to distinguish between .agg and .apply inside of it. I implemented necessary rearrangement in my last commit https://github.com/gshimansky/modin/commit/70ff9e2733a8769278a759d3050decb1ee7459bd but then it breaks .apply and .dtypes tests.
So the question are

what is the correct way to do this rearrangement where should it be done? Inside of Pandas backend only in groupby_agg or should it be done on API level in _apply_agg_function?
How should we distinguish when rearrangement happens and when it doesn't happen? Should we add another flag that .apply and .dtypes. pass to _apply_agg_function?

Here is a test script that you can run on Pandas and Modin (my branch with fixed exception) and Modin before PR #2317 was merged. Maybe also @dchigarev with multi-column aggregation which does significant changes to groupby but still has this bug.
import modin.pandas as pd
#import pandas as pd

df = pd.DataFrame({
    "col1": [ 1,  2,  1,  2,  1,  3,  3,  2,  3],
    "col2": [11, 12, 13, 14, 15, 16, 17, 18, 19],
    "col3": [ 4,  4,  4,  4,  5,  5,  5,  5,  5],
    "col4": [ 1,  1,  1,  1,  1,  1,  1,  1,  1]
})
print(df)

gb0 = df.groupby('col3', as_index=False)
df0 = gb0.agg(sum)
print("agg func:\n", df0)

gb1 = df.groupby('col3', as_index=False)
df1 = gb1.agg('sum')
print("agg string:\n", df1)

gb2 = df.groupby('col3', as_index=False)
df2 = gb2.apply(sum)
print("apply:\n", df2)

@gshimansky
I'm not sure that I understood the problem, as far as I know, pandas do the following logic in groupby:
if not `apply` then:
    drop 'by' columns
else:
    do not drop 'by' columns

result = do_groupby()

if `as_index` is False:
    insert 'by' columns at the beginning of the `result` in case, these columns not in the resulted frame [1]

if any by is categorical: [2]
    drop all 'by' columns from the resulted frame;
    insert all 'by' columns at the beginning of `result`

[1] [2]
We have tests for all that case that passes in test_simple_row_groupby (apply + as_index=False + categorical 'by')
About reproducer:
Only the first case seems to be failing, it happens because we're wrapping built-in function into wrap_udf_function inside groupby_agg which we shouldn't do. The simple fix is to translate built-in function into its name at the API level of aggregate.
if callable(func) and isinstance(func, types.BuiltinFunctionType):
    func = func.__name__

These two lines added at the beginning of modin.pandas.DataFrameGroupBy.aggregate fixes that issue, and makes output of all three reproducers similar to pandas

Additional extension to the above test program is by adding user lambdas so that not only builtin functions are tested:
gb3 = df.groupby('col3', as_index=False)
df3 = gb3.agg(lambda df: df.sum())
print("agg lambda func:\n", df3)

gb4 = df.groupby('col3', as_index=False)
df4 = gb4.apply(lambda df: df.sum())
print("apply lambda:\n", df4)


Refresh tokens not being used

Scenario is I'm using Electron + custom storage (so not in browser, using the async storage PR we did) and I've turned refresh tokens on (AllowOfflineAccess set to true etc and sending offline_access scope) confirmed by looking in electron-store file that a refresh_token is being stored.
When I re-open the app with an expired access_token (say after 30 minutes, they expire after 10 mintes) it doesn't seem to renew using the refresh token and just takes me to the login page.
We're using https://github.com/authts/react-oidc-context (modified for the async PR)

ah bingo, realised I needed to use signinSilent ( https://github.com/authts/oidc-client-ts/blob/61bb01d/src/UserManager.ts#L220 ) as:
https://github.com/authts/oidc-client-ts/blob/61bb01d/src/UserManager.ts#L228
Uses the refresh token.


Errors inside generators are ignored

Bug Description
I would expect an error when iterating over a generator to be returned as an Err, it seems not.
This seems to contradict
https://github.com/PyO3/pyo3/blob/6503128442b8f3e767c663a6a8d96376d7fb603d/src/types/iterator.rs#L50-L55
Steps to Reproduce
I would expect the following to panic, but it runs fine.
use pyo3::prelude::*;
use pyo3::types::{PyDict};

fn main() {
    println!("benchmarks");
    let error_generator = r#"
def gen():
    yield 1
    yield 2
    raise RuntimeError('error')
"#;

    Python::with_gil(|py| {
            let context = PyDict::new(py);
            py.run(error_generator, None, Some(context)).unwrap();

        let generator = py.eval("gen()", None, Some(context)).unwrap();

        for (actual, expected) in generator.iter().unwrap().zip(&[1, 2]) {
            let actual = actual.unwrap().extract::<usize>().unwrap();
            assert_eq!(actual, *expected)
        }
    });
}

Backtrace
No response
Your operating system and version
Ubunut
Your Python version (python --version)
3.10.4
Your Rust version (rustc --version)
rustc --version
Your PyO3 version
0.16.5
How did you install python? Did you use a virtualenv?
pyenv
Additional Info
No response

Ignore me, it works fine if I use PyIterator correctly.


de-version navbars

Description
Basically:
mv _data/master/navbars _data/

Related issues/PRs

Todos

[ ] Tests
[ ] Documentation
[ ] Release note

Release Note

None required

Deploy preview for calico ready!
Built with commit ae796438e92e871d02d037616a7e6d360fdd8d4c
https://deploy-preview-3122--calico.netlify.com


IG Rest API gives error.security.client-token-invalid after 13-14 hours of continuous usage

I observed that the IG Rest API gives the error.security.client-token-invalid error after 13-14 hours of continuous usage in the demo account.
I would appreciate guidance on possible ways to overcome this error.
Thank you.

https://trading-ig.readthedocs.io/en/latest/faq.html#what-if-i-have-a-problem

I don't experience this issue.  Can you please detail exactly how you see it?

@galburn
I put it on a continuous run using the demo account and after 13-14 hours, I got this error,
I tested it twice and got this error everytime.

I don't see this issue at all.  My code runs all week without a problem.  Can you put up the code?  Particularly which version of the login calls you are using.  Ideally, a minimal code version that shows the issue.

No updates or responses, closing


Enchancement of counter example

Is your feature request related to a problem? Please describe.
The current Counter example is very basic, so the idea to build a better application which showcases the project properly.
Describe the solution you'd like
The approach is to add the following details in the examples:

Using styling to make sure example looks great.
Using comments to describe the proper usage.
Adding relevant links to the project, so the user can straight dive in the docs.

Describe alternatives you've considered
None
Additional context
None

Reference to #11

@lakshyabatman you may proceed.

@lakshyabatman reminder I have changed the file structure a bit before you start working pls pull the new changes.

#14 solved this.


chore(tutorial): Adding resilience evaluation with LitmusChaos, Prometheus, and Keptn tutorial

Signed-off-by: Shubham Chaudhary<EMAIL_ADDRESS>Litmus & Keptn Upgrade PR should be merged before this PR
Issue Ref - https://github.com/keptn/keptn.github.io/issues/1452

When this is completed, I recommend that we include a link to it in https://keptn.sh/docs/0.19.x/define/ .  If you scroll to the bottom of that page, you will see that I have started to include links to other content that is of interest.  I'll write up a paragraph or two in the page that contains the link to the tutorial -- just to get terms like "LitmusChaos" and "resiliency" accessible to the search engine.

Thanks, @StackScribe to review it closely. I have made all the suggested changes
I have raised another PR in litmus-service repository to make this tutorial work for the latest versions of litmus & keptn. We can review and take this dependent PR first


Allow to definitly say "not duplicate" to kresus

Hi,
I have some operations which are alike (same date +/- 24h, same amount) but not duplicates.
I would love to be able to tell kresus "these two operations are not duplicate of each other"

In the duplicate section add a "Not duplicate" button.
Server side:  add a field in the model "isNotADuplicateOf" [string]
On click on the "Not duplicate button" add ID of operation to each others "isNotADuplicateOf"
In duplicate detection, add a check over isNotADuplicateOf field

A simple string isn't enough: a single transaction can be a duplicate of a few others, so we'd need an array at this point. I was actually planning to add a new doctype "KresusDuplicatePair", with 3 fields: 2 operations ids, and a status (among "new", "seen", "merged"). "new" would be for new duplicates computed on the server; "seen" for pairs that are marked as "not duplicates" in the frontend; "merged" for pairs that have been merged. Thinking about it, we need slightly more than 3 fields, as the merged operation should also be saved, in order to be able to undo the merge.

[string] is an array ;), But new docType could also work.


Make module capable of handling multiple clusters

This PR includes a few related changes:

Replace kubeconfig output with kubeconfig_env (current behaviour) and kubeconfig_file (basically the same as previous but without the 'export KUBECONFIG=' stuff.)

Added variable for config_output_path (in-line with other provider modules) and defaulted that to generated/ (in-line with your current implementation.)

Replaced all hard-coded references to kubeconfig with the new dynamic filenaming strategy.

Adds output value of the API endpoint (incl. HTTPS)

These changes combined should allow k8s clusters to be easily provisioned and then instantly used within a single TF deployment.

This is for Oracle Cloud. Can you please amend your description where you mentioned AWS?

Sure thing, done!

@hyder Can we get this merged with that change in place?

The helm dependency issue is already fixed in master: https://github.com/oracle-terraform-modules/terraform-oci-oke/blob/master/modules/oke/helm.tf.
I will look into the multi-cluster soon.

Hi @cawilliamson, could I please ask you to take a look at our CONTRIBUTING.md file in the root.  We need contributors to follow a few basic steps. Thanks a lot, if you have questions please feel free to contact me mark.x.nelson (at) oracle.com

I'm going to close this PR off. 4 months and zero movement is not exactly inspiring.


Protection for account resurrection attacks on GitLab OIDC Trusted Publishing

(For context, a GitLab namespace is like a GitHub organization/user. E.g: samplenamespace is the namespace of the https://gitlab.com/samplenamespace/repository project.)
As specified in the GitLab OIDC publisher documentation, currently PyPI does not fetch the GitLab namespace ID when setting up a Trusted Publisher.
This means the claim verification is done against the namespace's (string) name. For example, setting up a trusted publisher for https://gitlab.com/samplenamespace/sampleproject//release.yml will check that the OIDC claims contain the namespace samplenamespace.
In contrast, the GitHub Trusted Publishing implementation fetches the org/user ID during creation of the Trusted Publisher, and checks the claim against that ID. This protects against account resurrection attacks, since a namespace can be deleted and a new one with the same name created, but IDs are unique.
Currently, implementing this the same way we do it on GitHub TPs is not possible due to a limitation of GitLab's API. It's not possible to fetch the ID of a public personal namespace (think username in https://gitlab.com/username/repository). See this comment for an explanation and more details of the limitation.
A possible solution for this could be implementing a TOFU (Trust On First Use) scheme, where the ID of the namespace is read from the claims of the OIDC token the first time the Trusted Publisher is used, and stored as part of the TP's configuration and used to validate future claims.
cc @di @woodruffw

Thanks @facutuesca!
(I asked @facutuesca to file this as a separate issue so that we don't lose track of it. As previously discussed, it isn't a priority for the current implementation given limitations on GitLab's side + complexity tradeoffs on PyPI's side.)

There is an endpoint that could potentially be used to periodically check the existence of a (private) namespace and invalidate it as soon as it's gone - https://docs.gitlab.com/ee/api/namespaces.html#get-existence-of-a-namespace as a mitigation, if it scaled well.
But like the other endpoint discussed in the previous issue, this actually requires authenticated access, even on gitlab.com, and both always return 401 otherwise.

A possible solution for this could be implementing a TOFU (Trust On First Use) scheme, where the ID of the namespace is read from the claims of the OIDC token the first time the Trusted Publisher is used, and stored as part of the TP's configuration and used to validate future claims.

Because of the restrictions above, I think that might be a better solution even if an API becomes available, especially with https://github.com/pypi/warehouse/issues/15838 in mind. GitLab instances can have their discovery endpoints publicly exposed, their APIs behind a proxy, and runners with internet access to publish to pypi.org.


Refs #32539 -- Fixed facet filter tests on Oracle.

Follow up to 868e2fcddae6720d5713924a785339d1665f1bb9.

buildbot, test on oracle.

@sarahboyce Thanks for checking :+1:


Command not found

I have been having issue after typing in the username I get the response "./build.sh: line 61: dotnet: command not found", I downloaded dotnet and tried to find an solution for this but none worked.

Could you try opening a fresh terminal window and executing: dotnet --version.
If that prints a version (e.g. 7.0.100), try running the script again. If it does not print the version, the dotnet install did not succeed.

I used the offical Dotnet installer and it seems that it didn't install properly. I tried the method with Homebrew and it no longer showed that message after entering the username.


Implements AssetsUnderManagementCapacityManager

Description
Calculate an estimate of the capacity of the strategy backtest/live trade. This will provide funds with an understanding of the maximum dollar allocation they can make to the strategy to maximize its return potential without moving the market.

Implements AssetsUnderManagementCapacityManager
Adds an instance of AssetsUnderManagementCapacityManager in the result handlers to include AUM Capacity to algorithm statistics
Adds "AUM Capacity" entry to ExpectedStatistics of regression algorithms with a considerable amount of orders.

Performance test over 10 executions of RegressionAlgorithm (5433 trades):
master :: Mean: 7.71 Std: 0.16
new :: Mean: 7.853 Std: 0.22
Speed decrease of 1.85%.
Related Issue
Closes #3539
This pull request was created on top of #3571
Motivation and Context
Include statistics that can be used to improve algorithms.
Requires Documentation Change
No.
How Has This Been Tested?
New unit tests and regression algorithms.
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Non-functional change (xml comments/documentation/etc)

Checklist:

[x] My code follows the code style of this project.
[x] I have read the CONTRIBUTING document.
[x] I have added tests to cover my changes.
[x] All new and existing tests passed.
[x] My branch follows the naming convention bug-<issue#>-<description> or feature-<issue#>-<description>

Closing as inaccurate and we should implement a pluggable solution ICapacityEstimateModel and study existing papers about the solutions they've created.


Seeking input on clustering multiple samples

Hi,
I have a ~150 samples of protein fasta files, which I'd like to generate a non-redundant set of proteins from. I have tried concatenating the files to run the following:
mmseqs easy-linclust --cov-mode 0 -c 0.8 --min-seq-id 0.3 all_nomis_proteins.faa mmseqs2_output tmp

However, the concatenated file has 400 Mio genes, making it computationally infeasible. What may be the best approach here?

cluster smaller sets of samples and then re-cluster?
for point-1 above, can one provide multiple inputs or should concatenate the smaller sets?
for re-clustering after an initial round, is it possible to output the non-redundant fasta file?

Thank you for your input!
-Susheel

400 Million is not a big problem for linclust, it should run on any reasonably sized server in a day or so. We have clustered billions of sequences with linclust previously.
However it won’t reach 30% sequence Identity, for that you will need the normal clustering workflow. That will run linclust first and then use the MMseqs2 search algorithms to cluster further. That might run for a few days/weeks.

@milot-mirdita I'm running into segmentation fault issues when running with 2 Nodes carrying 128 cpus each - total memory == 448G.
Any recommendations on how kind of reasonable-sized server you are referring to? Any by 'normal clustering' you mean the mmseqs cluster?
I'm trying to reproduce the cascaded-clustering approach described here: https://elifesciences.org/articles/67667#bib118. Which might that be?
Thank you!

Can you try to run it on a single node (without MPI, etc). Issues in MPI support might have gone unnoticed since we switched to 128 core machines.
Yes, i mean  mmseqs (easy-)cluster with normal clustering. That one should also successfully finish eventually on a single of these compute nodes.
Can you please post the full log output? Maybe something else went wrong.

Okay, will give this a go and report back. Thank you!!

@milot-mirdita Below is the log file from one of the runs. Looks like it's running out of memory, before the job dies.

chunk00_clustering_stdout.log
And here is the job efficiency report from SLURM
Job ID: 359779
Cluster: aion
User/Group: sbusi/clusterusers
State: TIMEOUT (exit code 0)
Nodes: 1
Cores per node: 128
CPU Utilized: 10:13:07
CPU Efficiency: 1.00% of 42-16:44:48 core-walltime
Job Wall-clock time: 08:00:21
Memory Utilized: 206.26 GB
Memory Efficiency: 92.08% of 224.00 GB

Do you think merely providing more cores will do the trick or is there something else that I'm missing?
Thank you!

Update: Managed to successfully run the clustering using a full 3 TB node with 112 threads. The SLURM efficiency output is below:
Job ID: 2976046
Cluster: iris
User/Group: sbusi/clusterusers
State: COMPLETED (exit code 0)
Nodes: 1
Cores per node: 112
CPU Utilized: 73-15:37:58
CPU Efficiency: 21.99% of 334-22:48:00 core-walltime
Job Wall-clock time: 2-23:46:30
Memory Utilized: 197.78 GB
Memory Efficiency: 6.70% of 2.88 TB


KAFKA-17854: add tests for ReadOnlyWindowStoreStub#fetch(null, null ...)

KAFKA-17854
https://github.com/apache/kafka/pull/17540 fixed the incorrect check, but our test cases did not properly detect the incorrect check before it was merged.
PR #17540 corrected the check, but the test cases did not correctly capture the incorrect check before the PR was merged. This is because the expected result asserted in shouldFetchKeyRangeAcrossStoresWithNullKeyFrom() coincidentally matched the result obtained at line 272.
To demonstrate what my change covers, I reverted the changes from #17540 and applied my fix. The test fails as expected, as shown below:
$ ./gradlew streams:test --tests CompositeReadOnlyWindowStoreTest
----snip----
org.apache.kafka.streams.state.internals.CompositeReadOnlyWindowStoreTest.shouldFetchKeyRangeAcrossStoresWithNullKeyFrom() failed, log available in /home/yungh/kafka/kafka_fork/kafka/streams/build/reports/testOutput/org.apache.kafka.streams.state.internals.CompositeReadOnlyWindowStoreTest.shouldFetchKeyRangeAcrossStoresWithNullKeyFrom().test.stdout

Gradle Test Run :streams:test > Gradle Test Executor 57 > CompositeReadOnlyWindowStoreTest > shouldFetchKeyRangeAcrossStoresWithNullKeyFrom() FAILED
    java.lang.AssertionError:
    Expected: <[KeyValue([a@0/30000], a), KeyValue([b@10/30010], b)]>
         but: was <[KeyValue([a@0/30000], a), KeyValue([b@10/30010], b), KeyValue([c@10/30010], c)]>
        at org.hamcrest.MatcherAssert.assertThat(MatcherAssert.java:20)
        at org.hamcrest.MatcherAssert.assertThat(MatcherAssert.java:6)
        at org.apache.kafka.streams.state.internals.CompositeReadOnlyWindowStoreTest.shouldFetchKeyRangeAcrossStoresWithNullKeyFrom(CompositeReadOnlyWindowStoreTest.java:380)

----snip----

org.apache.kafka.streams.state.internals.CompositeReadOnlyWindowStoreTest.shouldBackwardFetchKeyRangeAcrossStoresWithNullKeyFrom() failed, log available in /home/yungh/kafka/kafka_fork/kafka/streams/build/reports/testOutput/org.apache.kafka.streams.state.internals.CompositeReadOnlyWindowStoreTest.shouldBackwardFetchKeyRangeAcrossStoresWithNullKeyFrom().test.stdout

Gradle Test Run :streams:test > Gradle Test Executor 57 > CompositeReadOnlyWindowStoreTest > shouldBackwardFetchKeyRangeAcrossStoresWithNullKeyFrom() FAILED
    java.lang.AssertionError:
    Expected: <[KeyValue([a@0/30000], a), KeyValue([b@10/30010], b)]>
         but: was <[KeyValue([a@0/30000], a), KeyValue([c@10/30010], c), KeyValue([b@10/30010], b)]>
        at org.hamcrest.MatcherAssert.assertThat(MatcherAssert.java:20)
        at org.hamcrest.MatcherAssert.assertThat(MatcherAssert.java:6)
        at org.apache.kafka.streams.state.internals.CompositeReadOnlyWindowStoreTest.shouldBackwardFetchKeyRangeAcrossStoresWithNullKeyFrom(CompositeReadOnlyWindowStoreTest.java:423)

----snip----

32 tests completed, 2 failed

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':streams:test'.
> There were failing tests. See the report at: file:///home/yungh/kafka/kafka_fork/kafka/streams/build/reports/tests/test/index.html

BUILD FAILED in 6s
97 actionable tasks: 3 executed, 94 up-to-date

The incorrect check can be detected after applying my change. Additionally, I think this change covers more possible cases.
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

@chia7712
If this change makes sense to you, should we update the Jira and Git commit headers and descriptions?

If this change makes sense to you, should we update the Jira and Git commit headers and descriptions to something like Improve tests for ReadOnlyWindowStoreStub#fetch and #backwardFetch?

yes, you are right. I will update them after merging this PR

Thanks for review :)


Details could be checkboxes

Ask Ana

New issue about this https://github.com/mleitao27/gcs-urban-green-spaces/issues/15


Add Feature AWS Network Firewall

LGTM
Need to add separate macro template to support multi subnets from users' inputs. if don't use macro , template will only support one subnet.
Two solutions for multi subnets support: custom resource or macro.  macro one was chosen.