Re-opened PR to support NodeJS runtime

I closed my old PR (got sidetracked on a ton of different stuff since that PR ;) )
It's similar to the previous one I had (we need to be able to support Node JS environment for coveo headless), except I implement the "Runtime" interface that was suggested by @francoislg .

Codecov Report

Merging #201 into master will decrease coverage by 0.16%.
The diff coverage is 80.59%.

@@            Coverage Diff             @@
##           master     #201      +/-   ##
==========================================
- Coverage   89.00%   88.84%   -0.17%
==========================================
  Files          21       22       +1
  Lines         655      699      +44
  Branches       95       99       +4
==========================================
+ Hits          583      621      +38
- Misses         43       47       +4
- Partials       29       31       +2

Impacted Files
Coverage Δ

src/client/crypto.ts
61.53% <50.00%> (+3.20%)
:arrow_up:

src/donottrack.ts
75.00% <50.00%> (-25.00%)
:arrow_down:

src/client/analyticsBeaconClient.ts
81.81% <75.00%> (-1.52%)
:arrow_down:

src/client/runtimeEnvironment.ts
77.77% <77.77%> (ø)

src/client/analytics.ts
90.27% <80.00%> (+0.06%)
:arrow_up:

src/detector.ts
83.33% <80.00%> (+13.33%)
:arrow_up:

src/storage.ts
82.05% <100.00%> (+7.05%)
:arrow_up:

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 691ab5b...66dda99. Read the comment docs.


Click on a timestamp, get a parse error on query

Go to any zgram.  Click on the timestamp link.  For example, [Sep 16, 2023 12:02 PM] is one such link.
Click "query" in the toolbar at the top, since you are curious what this has linked to.
In the dialog box that comes up, it will announce in red "Parse Error".

Duplicate of https://github.com/kosak/z2kplus/issues/26


Bigquery: Expose CMEK functionality for ML models.

I dont see test cases

@rahulKQL I did according to Table and there is no test case for encryption.

ok then its just builder addition. LGTM.


Add all files from source packages to target APK

Add all files from source packages to target APK
and remove "virtual package" from dependencies as it is empty now.
Resolves #613
Should be tested in the context of mercedes-benz/sechub#1794 // @Jeeppler + @ryanlewis

@ascheman is this pull-request ready to be tested?

@ascheman is this pull-request ready to be tested?

Yes, please test it, @Jeeppler - As soon as you come back with a positive result I will make it official (remove Draft status).

@ascheman
Tested with SecHub 0.37.0 (Spring Boot 2.7.6) on Alpine Image:

Temurin 8: SecHub cannot be build with Java 8
Temurin 11:

temurin-11-jre-11.0.18_p10-r0.apk -> works
temurin-11-jdk-11.0.18_p10-r0.apk -> works

Temurin 17:

temurin-17-jre-17.0.6_p10-r0.apk -> works
temurin-17-jdk-17.0.6_p10-r0.apk -> works

Temurin 19:

temurin-19-jre-19.0.2_p7-r0.apk -> works
temurin-19-jdk-19.0.2_p7-r0.apk -> works

RESULT: All conducted tests were successful. Temurin 11, 17 and 19 work for both JDK and JRE packages.
NOTES: I can see the following warning in the test section:
>>> WARNING: temurin-11*: No arch specific binaries found so arch should probably be set to "noarch"

Tested with SecHub 0.37.0 (Spring Boot 2.7.6) on Alpine Image:

Temurin 8: SecHub cannot be build with Java 8

Temurin 11:

temurin-11-jre-11.0.18_p10-r0.apk -> works
temurin-11-jdk-11.0.18_p10-r0.apk -> works

Temurin 17:

temurin-17-jre-17.0.6_p10-r0.apk -> works
temurin-17-jdk-17.0.6_p10-r0.apk -> works

Temurin 19:

temurin-19-jre-19.0.2_p7-r0.apk -> works
temurin-19-jdk-19.0.2_p7-r0.apk -> works

RESULT: All conducted tests were successful. Temurin 11, 17 and 19 work for both JDK and JRE packages.

Thanks for testing @Jeeppler

NOTES: I can see the following warning while building the temurin packages via gradle:
>>> WARNING: temurin-11*: No arch specific binaries found so arch should probably be set to "noarch"

I could avoid this warning with my latest change (d38b818).
Nevertheless there are more WARNINGs and even ERRORs during the build, e.g., since Alpine JDKs do not contain libawt.so.

If you set the architecture to x86_64 it will probably not work if the architecture is different. It should be possible to compile/build the JRE and JDK packages for other architectures as well.
Is there no variable for the architecture which you can use/set during the build?

If you set the architecture to x86_64 it will probably not work if the architecture is different. It should be possible to compile/build the JRE and JDK packages for other architectures as well.
Is there no variable for the architecture which you can use/set during the build?

To my understanding, there is only x86_64 for Alpine, and it was configured for this arch the hard way anyways (cf. eg., https://github.com/adoptium/installer/blob/8687b392feaeb2cab9b72ab780d6d673ab81609a/linux/jdk/alpine/src/main/packaging/temurin/11/APKBUILD#L12).

Project Management Committee - all Eclipse Foundation top-level projects, including Adoptium has a PMC that is responsible for technical guidance of the project. (Documentation on Dev process Leadership at Eclipse Foundation).


Additional test to check parquet row group skipping for struct type

Additional test to check parquet row group skipping for struct type. We are making use of existing parquet file from examples to test this. There are already tests checking this for simple types and map type. This PR adds one test for struct type.

@yingsu00 I have addressed the comments. Thanks for the review.


[Bug] Set up the map for geo, set the region for nameMap, display an exception when the data is empty.

Version
5.4.3
Link to Minimal Reproduction
https://codesandbox.io/s/namemap-ksjfjl?file=/index.js
Steps to Reproduce

Click on the URL
move the mouse to the highlighted area, and view the log generated by tooltip.formatter.

Current Behavior
Set up the map for geo, set the region for nameMap, display an exception when the data is empty.
When the data is empty, an exception is displayed in the regions within nameMap. The log generated by tooltip.formatter shows that the "name" field is empty. Regions that are not present in nameMap behave normally.
Expected Behavior
The regions within nameMap behave the same as the normal regions.
Environment
- OS:
- Browser:
- Framework:

Any additional comments?
No response

an exception is displayed in the regions within nameMap

Strange demo - there is no data in series and tooltip.formatter is this:
  tooltip: {
    trigger: "item",
    showDelay: 0,
    transitionDuration: 0.2,
    formatter(e) {
      console.log(e);
    }
  },

Shouldn't it be formatter: (e) => { console.log(e); } ?
Either way, I don't see any exceptions - Demo Code.

an exception is displayed in the regions within nameMap

Strange demo - there is no data in series and tooltip.formatter is this:
  tooltip: {
    trigger: "item",
    showDelay: 0,
    transitionDuration: 0.2,
    formatter(e) {
      console.log(e);
    }
  },

Shouldn't it be formatter: (e) => { console.log(e); } ? Either way, I don't see any exceptions - Demo Code.

{
      name: "USA PopEstimates",
      type: "map",
      map: "USA",
      geoIndex: 0, // add the line
      data: [
        // { name: "Alabama11", value: 24822023 }
      ]
 }

You need to add geoIndex: 0 in series[0]

an exception is displayed in the regions within nameMap

Strange demo - there is no data in series and tooltip.formatter is this:
  tooltip: {
    trigger: "item",
    showDelay: 0,
    transitionDuration: 0.2,
    formatter(e) {
      console.log(e);
    }
  },

Shouldn't it be formatter: (e) => { console.log(e); } ? Either way, I don't see any exceptions - Demo Code.

{
      name: "USA PopEstimates",
      type: "map",
      map: "USA",
      geoIndex: 0, // add the line
      data: [
        // { name: "Alabama11", value: 24822023 }
      ]
 }

You need to add geoIndex: 0 in series[0]

@helgasoft   Could you try following the instructions in this message again? I noticed some anomalies after I made the additions.

My recommendation is to stay away from geo and work with series map only. Much simpler.
Demo Code

My recommendation is to stay away from geo and work with series map only. Much simpler. Demo Code

e，If you need to display both a map and scatterplot on the same chart。
Of course, I will consider your suggestion. Thank you.

If you need to display both a map and scatterplot on the same chart

then follow official example


a facet or facetAnchor: "all" option?

In this Plot:

Plot.plot({
  marginLeft: 60,
  marks: [
    Plot.frame({ fill: "silver" }),
    Plot.barX(data, { x: "Cantidad", fx: "type", fy: "p28" })
  ]
})

we would like the silver frame to be present on all facets, including the empty ones (represented with an orange stroke). Should we add a facet: "all" or facetAnchor: "all" option?
(The orange stroke was added with Plot.frame({ stroke: "orange", facetAnchor: "empty" })).
The chart can be fixed currently by duplicating the mark:
[Plot.frame({ fill: "silver" }), Plot.frame({ fill: "silver",facetAnchor: "empty" })],
but it shows that something is missing.
Test & data: https://observablehq.com/d/faf63a6fe3950d75
(Feature request suggested by @rusosnith)
relevant links in the documentation:

https://observablehq.com/plot/features/facets#mark-facet-options
https://observablehq.com/plot/marks/axis

(there is a small error on the axis page, where it says "null - show on every facet" it should say "null - show on every non-empty facet")

Same issue when you specify the facet domain, and want the frame (for example) to appear on all the facets irrespective of there being data in that facet.


Feature request : add support for spark connect without pysark dependencies

it will be really useful to be able to connect to Sail Server without requiring to install pyspark first

i meant, if will be nice to use spark connect python client that does not depends on pyspark, in an ideal scenario, java will not be required at all

Currently, to use the Python Spark Connect client, the pyspark Python package is needed but Java doesn't need to be installed since the jar files in the Python package will not be used. Spark 4.0 will also support a pure Python Spark Connect client via [pyspark-connect](https://pypi.org/project/pyspark-connect/]. This is a lightweight package without jars. Would that be suitable for your needs?


chore: add a GH action to lint PRs to ensure usage of proper titles and commit messages to support semantic release

What this PR does / why we need it:
This should give us back the commit messages check we used to have with the Semantic Pull Request Bot that doesn't work anymore because of https://github.com/zeke/semantic-pull-requests/issues/183

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


issues installing insight

ran npm install on the command line and keep getting the following issue, can anyone help?

I'm also unable to install base58-native... any suggestions? This is on Windows 7 x64.

Outdated


Resource trucking issue

Kill this issue if it working as planned, but...
I accidentally produced too many resources and all the factory storage sites were maxed out. The FOB then maxed out and the truck drove back to the factory and waited until i manually loaded 3 boxes into my vehicle. It then dumped it's boxes into the factory and ended the mission back at the FOB.
This was the only way i could solve the issue logistically, I guess I could have built something for a museum!
Perhaps there could be a back end solution or notification?

You can build more than one supply pad at the FOB

The FOB has 4 large storage.
I'm happy for you to delete this report and call me stupid, just thought I'd report the behaviour in case

No need to delete it. Did you read the wiki entry about the logistics?
It explains the specific behaviour of the logistic groups. Your description seems to be "all fine" to me.
https://github.com/Wyqer/kp_liberation/wiki/EN:Logistic


Moe converters

Ready for review. Instructions on how to use the new converters are in the examples/xglm/README. Implementation of huggingface transformers moe xglm in examples/xglm/transformers_impl (thanks @negar-foroutan!). Let me know what you think :)


feat: non-helm deployments

Hi, we are trying to use helmfile as declarative configuration for our kubernetes clusters. But it can only manage helm deployments, which is not enough to manage all kube related tasks.
Consider the following:

Add support for helm init in helmfile.
Unfortunately helm has no (official) way to make a declarative installation of tiller on kube cluster, which can have a lot of options, not just run command and you're done.
Add support for kubectl apply for remote files.
Some artifacts are to be installed on cluster before tiller, like network plugins. This usually means running kubectl apply with some predefined remote file.
Add support for kubectl delete.
Neither kubectl apply nor helm install has an option to destroy some predefined resources, which might be a part of cluster state. We do delete default and kube-public namespaces for instance.

PS: I know, this might be out of scope of helmfile itself, so I'm interested in your opinion on this.

@astorath Hey!
There seem to be no much momentum in this project to support the use-case yet.
But I'm personally aware of it!
For 1, the easiest way would be to use helm-tiller. So that you don't need to run tiller beforehand and your helmfile command looks like helm tiller run -- helmfile ....
Actually, I've contributed the helm tiller run command myself, for this use-case.
No out-of-box support for helm init would also be good in regard to that helm v3 gets rid of tiller.
For 2, although it doesn't exactly match what you've requested, could try a kind of "helmify" command plus helmfile's "hook" feature? The helmfile + kustomize example in our README would be a good source of inspiration.
I have no idea how No. 3 fits within the scope of helmfile yet. The next best thing helmfile could provide would be a onHelmfileStart hook #331. With that, you would write a custom script that discovers and then removes predefined resources.


Can't drag a file over a Markdown preview in another editor group

VSCode Version: 1.12 Insiders
OS Version: Windows 10

Steps to Reproduce:

Open an Markdown file, then Markdown: Open Preview to the Side (Ctrl+K V)
Now try to drag a file over to the editor group with the Markdown preview.

The drop icon appears but the file is not moved after the drop.

@gregvanl does it work in 1.11?

@bpasero Yes, this is a regression from 1.11.

@gregvanl can you share a video please

I believe this is now working properly. @gregvanl can you please confirm? If it is still not working, please let me know

Still not working reliably on windows:

Still a bug in 2019 if anyone wondered.

/duplicate https://github.com/microsoft/vscode/issues/25854


ISPN-11789 Update Query Guide

https://issues.redhat.com/browse/ISPN-11789

@oraNod This is a good foundation for you to work on, once merged.
@anistor please check if I haven't forgot anything or wrote something wrong.

Updated

Updated

Anything else @oraNod and @anistor ?

Updated again

Updated. Please let me if more changes are needed so that I can squash the commits into 1

Updated. Please let me know if more changes are needed so that I can squash the commits into 1

@gustavonalle LGTM

@oraNod Commits squashed and rebased.

integrated. thanks @gustavonalle !


Validate certificate issued by root/intermediate CA for MSP when revoking

Type of change

Improvement (improvement to code, performance, etc)

Description

Ensure the certificate(s) being revoked were signed by the root and intermediate CA certs associated with the organization's MSP.

Related issues
FAB-17587

/ci-run


cmd: Implement image import and export commands

Implement commands to export / import gadget images to / from a file.
Why
Being able to export and export images from files is useful for the following cases:

It's possible to distribute images by just using files. Airgapped environments. Cases where reaching the internet is not an option
Integrate gadget images directly into the application binary. #2021. (I already have a POC of this working, but it's still missing a lot of things)

Testing
# let's start with an empty image store
$ sudo rm -rf /var/lib/ig/oci-store
$ sudo -E ig image list
INFO[0000] Experimental features enabled
REPOSITORY             TAG                    DIGEST       CREATED

# pull one image (build will work as well)
$ sudo -E ig image pull trace_open
INFO[0000] Experimental features enabled
Pulling trace_open...
Successfully pulled ghcr.io/inspektor-gadget/gadget/trace_open:latest@sha256:1ef0b57afaafe4e7d1befffc902cdda56a3d51936b9301710047b722f8018145

# export image
$ sudo -E ig image export trace_open /tmp/images.tar
INFO[0000] Experimental features enabled
Successfully exported images to /tmp/images.tar

# clean the image store (or copy tar file to another machine...)
$ sudo rm -rf /var/lib/ig/oci-store
$ sudo -E ig image list
INFO[0000] Experimental features enabled
REPOSITORY             TAG                    DIGEST       CREATED

# import the image
$ sudo -E ig image import /tmp/images.tar
INFO[0000] Experimental features enabled
Successfully imported images:
  ghcr.io/inspektor-gadget/gadget/trace_open:latest

#  image is there and can be run
$ sudo -E ig image list
INFO[0000] Experimental features enabled
REPOSITORY             TAG                    DIGEST       CREATED
trace_open             latest                 1ef0b57afaaf 56 minutes ago

$ sudo -E ig run trace_open --pull never
INFO[0000] Experimental features enabled
RUNTIME.C… PID     UID      GID      MNT… … FD     … MODE  COMM  FNAME      TIME

Also one question regarding integration tests. I think we should drop the test for build command and use import/export to simplify (get rid of dependency of ebpf-builder image) them and also to test this new functionality? (can be handled in a seperate PR as well)

I'm wondering where can we import the image from, I mean, we can remove the dependency from the build command, but then we'll need to have an image to import somewhere.

Also one question regarding integration tests. I think we should drop the test for build command and use import/export to simplify (get rid of dependency of ebpf-builder image) them and also to test this new functionality? (can be handled in a seperate PR as well)

I'm wondering where can we import the image from, I mean, we can remove the dependency from the build command, but then we'll need to have an image to import somewhere.

That is good question, I see following options:

Export an image for once and put in testdata since we don't care about content of image
Keep the build command tests and use it to validate the import/export command.

Perhaps 2 is better?

Great addition!
It might make sense to (in a follow-up PR) move the importing functionality to the runtime and thus also add gRPC calls for it. That way you could upload gadgets to ig-k8s as well. That'd still not be a perfect solution, as newly added nodes wouldn't receive the gadget that way, but it might be helpful in airgapped clusters that only need to run gadgets in an ad-hoc way on specific nodes.


Fixed bunch of errors

Quick fix some errors:
How flash appears:

Suggestion page has header 'Suggestions'

No import send_email

LGTM!! @anniemeng


Sometimes when trying to close a contour with the pen tool nothing happens and point_idx becomes contour.len() - 1.

Not sure what's causing this exactly, but it's definitely an issue and something I don't want to forget about.

Just figured this out. The distance from the point used in point picking is less than that used for adding a subdivision to a contour. This is something that just needs a bit of tweaking.

Fixed the part of this bug that caused the point_idx to change relative to the contour. The rest of this issue is more of an enhancement that I'll get to.


Problem with run_output_asserts . Bug?

In line 253 of keras_core.testing.test_case.py
If assertTrue output is a python list then I do not understand why output should have .keys() method,
unless of course output shoud inherit from list but implement a .keys() method.
But I thought outputs of layers can be type pure list.
I am sorry for this stupid question but I can not resolve this assert.
                # In line 253 of test_case.py
                elif isinstance(expected_output_shape, list):
                    self.assertTrue(isinstance(output, list))
                    self.assertEqual(
                        len(output.keys()),
                        len(
                            expected_output_shape.keys(),
                            msg="Unexpected number of outputs",
                        ),
                    )

Yes, that's a bug. I just fixed it. It seems this part of the code is currently unused.


Cannot load entities from world

Hi. I'm trying to load entities from saved world, but when I call ChunkColumn.getEntities(), it returns []. I tried to do it on Minestom, and opened discussion, because this attempt failed. I think, that this is Hephaitos problem, because this error related to chunk loader.

Send me a region file from this world if can.
If you cannot, please try to reproduce with another world and send a region file from that one.
Also, what version of Minecraft was this world created in?

This is the whole world

Could you give me some location where entities are supposed to be?
I don't see any when loading the world in vanilla singleplayer.
I also want to point out that Minestom does NOT save entities. So if you opened your world with Minestom, saved, your entities will no longer be present in the chunk files.

-693 143 693, I've never opened this world via Minestom

Reopen it.  My world: world.zip, coordinates: 0 -61 0.

Entities are no longer inside region/ but inside entities/
https://minecraft.fandom.com/wiki/Entity_format
I have to check whether Hephaistos can read these files properly

So, any changes?

It is possible to load entities with the current version of Hephaistos, it is just a bit involved:
Load your region file from the entities/ folder as usual.
Instead of querying the ChunkColumn, get the NBT data directly via RegionFile#getChunkData(chunkX, chunkZ) ( https://github.com/jglrxavpok/Hephaistos/blob/58458b3ca4bdbb7fbbc477a63091db20a6a2da14/common/src/main/kotlin/org/jglrxavpok/hephaistos/mca/RegionFile.kt#L122 ).
Then you should be able to access the Entities (mind the capital E) NBTCompound array. From there, you can load your entities as you wish.
There will be a better/cleaner/faster way of doing things at some point, but there's quite some changes required.
Here's a small snippet:
String filename = "myworld/entities/r.0.0.mca";
RegionFile f = new RegionFile(new RandomAccessFile(filename, "r"), 0, 0);
NBTList<NBTCompound> list = f.getChunkData(0,0).getList("Entities");
System.out.println(list.toSNBT());

Okay, great.  I get only the 4 armor stands from your example.

Code:
int x = loadedChunk.chunkX / 32;
int z = loadedChunk.chunkZ / 32;
try {
   RegionFile f = new RegionFile(new RandomAccessFile(this.entitiesPath.resolve(String.format("r.%d.%d.mca", x, z)).toString(), "r"), x, z);
   NBTList<NBTCompound> list = Objects.requireNonNull(f.getChunkData(0, 0)).getList("Entities");
   assert list != null;
   System.out.println(list.toSNBT());
} catch (AnvilException | IOException e) {
   e.printStackTrace();
}

Output
world.zip

You are loading from chunk 0;0 for every chunk.
getChunkData takes in the chunkX and chunkZ coordinates

My mistake. New output, world the same. Code:
int x = loadedChunk.chunkX / 32;
int z = loadedChunk.chunkZ / 32;
try {
   RegionFile f = new RegionFile(new RandomAccessFile(this.entitiesPath.resolve(String.format("r.%d.%d.mca", x, z)).toString(), "r"), x, z);
   NBTList<NBTCompound> list = Objects.requireNonNull(f.getChunkData(x, z)).getList("Entities");
   assert list != null;
   System.out.println(list.toSNBT());
} catch (AnvilException | IOException e) {
   e.printStackTrace();
}

Now you're using the region X/Z instead of the chunk X/Z.

wdym?

getChunkData takes the chunk X and Z coordinates, not the region coordinates.
In the code you sent, you are using the region X/Z (chunk coordinates / 32)

Yeah, okay, I got it.
New code:
int chunkX = loadedChunk.chunkX;
int chunkZ = loadedChunk.chunkZ;

int x = chunkX / 32;
int z = chunkZ / 32;

try {
   RegionFile f = new RegionFile(new RandomAccessFile(this.entitiesPath.resolve(String.format("r.%d.%d.mca", x, z)).toString(), "r"), x, z);
   NBTList<NBTCompound> list = Objects.requireNonNull(f.getChunkData(chunkX, chunkZ)).getList("Entities");
   assert list != null;
   System.out.println(list.toSNBT());
} catch (AnvilException | IOException e) {
   e.printStackTrace();
}

Output, world is the same.

Dividing by 32 is not correct for negative values.
You need to use floor(coordinate / 32.0) to have the correct result.
-1 / 32 = 0
floor(-1 /32.0) = - 1.0

I'm sorry. I know I'm really boring you, but I need to fix this problem
int chunkX = loadedChunk.chunkX;
     int chunkZ = loadedChunk.chunkZ;

        int x = (int) Math.floor(chunkX / 32.0);
        int z = (int) Math.floor(chunkZ / 32.0);

        System.out.println("Chunk X: " + chunkX + ", Z: " + chunkZ);
        System.out.println("X: " + x + ", Z: " + z);

        try {
            RegionFile f = new RegionFile(new RandomAccessFile(this.entitiesPath.resolve(String.format("r.%d.%d.mca", x, z)).toString(), "r"), x, z);
            NBTList<NBTCompound> list = Objects.requireNonNull(f.getChunkData(chunkX, chunkZ)).getList("Entities");
            assert list != null;
            System.out.println(list.toSNBT());
        } catch (AnvilException | IOException e) {
            e.printStackTrace();
        }

Output

I also tried this:
int chunkX = loadedChunk.chunkX;
int chunkZ = loadedChunk.chunkZ;

int x = (int) Math.floor(chunkX / 32.0);
int z = (int) Math.floor(chunkZ / 32.0);

System.out.println("Chunk X: " + chunkX + ", Z: " + chunkZ);
System.out.println("X: " + x + ", Z: " + z);

try {
   RegionFile f = new RegionFile(new RandomAccessFile(this.entitiesPath.resolve(String.format("r.%d.%d.mca", x, z)).toString(), "r"), chunkX, chunkZ);
   NBTList<NBTCompound> list = Objects.requireNonNull(f.getChunkData(chunkX, chunkZ)).getList("Entities");
   assert list != null;
   System.out.println(list.toSNBT());
} catch (AnvilException | IOException e) {
   e.printStackTrace();
}

Output

My guess is that you are trying to access files that do not exist.
RandomAccessFile is not really a fan of non-existing files IIRC
Le sam. 12 mars 2022 à 20:33, Aleksey Demchenkov @.***>
a écrit :

I'm sorry. I know I'm really boring you, but I need to fix this problem
int chunkX = loadedChunk.chunkX;
int chunkZ = loadedChunk.chunkZ;
    int x = (int) Math.floor(chunkX / 32.0);
    int z = (int) Math.floor(chunkZ / 32.0);

    System.out.println("Chunk X: " + chunkX + ", Z: " + chunkZ);
    System.out.println("X: " + x + ", Z: " + z);

    try {
        RegionFile f = new RegionFile(new RandomAccessFile(this.entitiesPath.resolve(String.format("r.%d.%d.mca", x, z)).toString(), "r"), x, z);
        NBTList<NBTCompound> list = Objects.requireNonNull(f.getChunkData(chunkX, chunkZ)).getList("Entities");
        assert list != null;
        System.out.println(list.toSNBT());
    } catch (AnvilException | IOException e) {
        e.printStackTrace();
    }

Output https://gist.github.com/Pelfox/0884f4dc930c82c6bd282f6336b22b1d
—
Reply to this email directly, view it on GitHub
https://github.com/jglrxavpok/Hephaistos/issues/18#issuecomment-1065949136,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABJNFMVC6PT3PTMHSBI6ABDU7TWO3ANCNFSM5QGMUMPQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you modified the open/close state.Message
ID: @.***>

So, I simply added check for existing file and now it works. Thank you VERY much!


Flashing on mobile devices

Describe the bug
On a mobile device, beginning to scroll, or changing scroll direction causes some brief flashing. This might be due to mobile browsers which move the browser controls in/out depending on scroll direction (which changes the viewport).
To Reproduce
Test on iPhone or android chrome browser.
Expected behavior
Scrolling should be smooth and seamless.

This is fixed on v4.0.3


Fix TransportCCExtension and AudioLevelExtension to work with new API introduced by #56

By #56, extension ID and length is handled by packet marshaler.
But TransportCCExtension and AudioLevelExtension are still handling ID/length as a first byte of extension data. It makes first byte of the extension duplicated.
I noticed this broke AbsSendTimeExtension and fixed in https://github.com/pion/rtp/pull/62.
I think TransportCCExtension and AudioLevelExtension need same fix.

FYI @tarrencev @adwpc

FYI @tarrencev @adwpc

@at-wat Thanks
fix:
https://github.com/pion/rtp/pull/67

fixed by #67 and #68


Allow relative path to custom reporter

Closes #2434
Previously, mocha --reporter=./path/to/custom-reporter.js would fail.
Also removes code from _mocha that has been unnecessary since 191b88c079d44f527dfeb022fbdb4079440c8428

@boneskull Please review. The nested try-catch approach was pretty gnarly, so I flattened it.

@cletusw any progress on this? I am looking to use this feature. Need help with getting the PR through?

@sul4bh Yeah, I'll be unable to work on this for the foreseeable future. Feel free to take it from here!


Support hiliting the intersection of models and subcategories

#3829

[x] Promote models and subcategories support in HiliteSet to @public
[x] Add new flag to HiliteSet indicating whether models and subcategories should be treated as a union (default) or intersection
[x] Adjust logic in FeatureOverrides to respect the new flag
[x] Add key-in to change the state of the flag
[ ] Improve ergonomics of Uint32Set API
[ ] Provide public frontend API to query subcategories belonging to categories
[ ] NextVersion.md
[ ] Unit tests
[ ] Run ImageTests
[ ] Add new ImageTests views exercising the new feature
[ ] Find out from @grigasp if any adjustments are needed to HiliteSetProvider

Find out from @grigasp if any adjustments are needed to HiliteSetProvider

No changes needed.
I'm a bit worried about the mode being set at application level rather than at instance level. I wonder what we're going to do on the UX side if two independent components will want different modes. At this point this is not an issue since we don't allow selecting Model / Category instances in core components, but it could become a problem in the future.


Building JavaScript bundle often gets stuck at 0%

Description
When the Expo app starts and I click on my project, 1 out of every 3-4 times it gets stuck. This also happens with the same frequency when I have live reloading enabled and save my changes. The text at the bottom of the screen says "Building JavaScript Bundle..." and to the right is written 0.00%.
When this happens, I have to restart the project from Expo XDE and shut down the app on my phone and restart it for it to work again.
Expected Behavior
When I reload the app after changing code, or when I start it up, it should finish building the javascript bundle and start my app.
Observed Behavior
The loading indicator gets stuck at 0%.
If there's an error message, please paste the full terminal output and error message in this code block:
No error message. I can't even shake the phone so I can get the developer tools.
Environment
Please run these commands in the project folder and fill in their results:

npm ls react-native-scripts:  It says empty.

npm ls react-native:

UNMET DEPENDENCY react-native@https://github.com/expo/react-native/archive/sdk-23.0.1.tar.gz

npm ERR! missing: react-native@https://github.com/expo/react-native/archive/sdk-23.0.1.tar.gz, required by<EMAIL_ADDRESS>npm ERR! extraneous: react-native@https://github.com/expo/react-native/archive/sdk-23.0.1.tar.gz FOLDERPATH/app/node_modules/react-native

npm ls expo:

UNMET DEPENDENCY<EMAIL_ADDRESS>
npm ERR! missing<EMAIL_ADDRESS>required by<EMAIL_ADDRESS>npm ERR! extraneous<EMAIL_ADDRESS>FOLDERPATH/app/node_modules/expo

node -v: v8.7.0
npm -v: 5.5.1
yarn --version: 1.3.2
watchman version: 4.9.0

Also specify:

Operating system: MacOS High Sierra 10.13.1
Phone/emulator/simulator & version: iPhone 6s

Here's my package.json if that's of any help.
{
  "name": "MYAPPNAME",
  "version": "0.1.0",
  "private": true,
  "devDependencies": {
    "babel-cli": "^6.26.0",
    "babel-jest": "^21.2.0",
    "babel-plugin-transform-react-jsx-source": "^6.22.0",
    "babel-preset-es2015": "^6.24.1",
    "babel-preset-react-native": "^4.0.0",
    "enzyme": "^3.2.0",
    "jest": "^21.2.1",
    "jest-expo": "^24.0.0",
    "prettier": "^1.7.4",
    "prettier-eslint": "^8.2.1",
    "react-native-scripts": "1.3.1",
    "react-test-renderer": "^16.0.0"
  },
  "main": "./node_modules/react-native-scripts/build/bin/crna-entry.js",
  "scripts": {
    "start": "react-native-scripts start",
    "eject": "react-native-scripts eject",
    "android": "react-native-scripts android",
    "ios": "react-native-scripts ios",
    "test": "jest"
  },
  "jest": {
    "preset": "jest-expo",
    "testPathIgnorePatterns": [
      "/node_modules/"
    ]
  },
  "dependencies": {
    "axios": "^0.16.2",
    "expo": "^24.0.0",
    "firebase": "^4.5.1",
    "lodash": "^4.17.4",
    "moment": "^2.19.1",
    "prop-types": "^15.6.0",
    "react": "16.0.0",
    "react-native": "https://github.com/expo/react-native/archive/sdk-24.0.0.tar.gz",
    "react-native-autocomplete-input": "^3.4.0",
    "react-native-azure-ad": "^0.2.4",
    "react-native-img-cache": "^1.5.3",
    "react-native-router-flux": "^4.0.0-beta.21",
    "react-native-slider": "^0.11.0",
    "react-redux": "^5.0.6",
    "redux": "^3.7.2",
    "redux-thunk": "^2.2.0",
    "watchman": "^1.0.0"
  }
}

Same issue. Reinstall dependencies is not helping.

Just got this issue, I can't build my JS bundle anymore. Deleting node_modules & package-lock.json did not help.

I found a lot of react-native errors are being silenced by Expo's xdl.
See https://github.com/expo/expo/issues/1537

Sooo anyone solved the issue already? I'm having the same issue with expo on windows 10

There are some problems with watchman. But running watchman --foreground in a separate terminal solved the issue with stucked build for me.


[feature] Dynamic GITHUB_TOKEN permissions

Currently reusable workflows require users to grant them all permissions they may ever use even if they aren't actually necessary. For example, a workflow will require contents: write in order to artifacts to a release, but if a user specifies an input that disables that functionality then the permission is not actually used.
We would like to give users the ability to determine what permissions to give the workflow and for the workflow to be able to act accordingly by either skipping the functionality if permissions are not granted or when a user provides an input indicating it should be skipped.

Related to #2044, #1257

This is blocked on the lack of functionality in GitHub Actions.


Implement new designs for settings screen

Feature Description:
Implement new designs for settings screen
**Pre-requisites/ dependencies
-Designs
https://xd.adobe.com/view/f1543c84-c25a-4ad6-81ca-1e0c540e956e-a407/screen/f8070387-e976-4149-87cc-3fa2abfe6e9c/
**Task Break Down

Design settings screens
Implement logic for settings screen

Verified this issue on keeper app v 1.1.11(297)


Update deploy-a-jekyll-site.md for JEKYLL_ENV environment variable

JEKYLL_ENV can be set in two ways: as an environment variable, or as part of the build command. Since the build command is generic to both the Production and Preview environments, an environment variable should be used.
JEKYLL_ENV is important to the build process and must be set to production for certain features to work properly in Jekyll plugins and themes (the default is development, per Jekyll's documentation on environments), so the documentation for Cloudflare Pages should specify how to set it.
I've followed the formatting of the section above it as it is relevant. This should be placed in the documentation after the Build configuration section (as it's part of the same process) and before first deploy section (as it's required to be set before deployments) so that it's in the order of operations of the setup process.

@rita3ko Thanks for the feedback. I've updated the commit with the requested update.

@dotZak can you rebase this PR so we can merge it? thanks!

@dotZak it looks like the rebase here got a little messed up, do you mind opening a new PR and basing it off of our production branch? thanks!


Latest version seems to lint into multiline HTML comments despite html_elements: false

As the title says, we tried rolling out the latest version and noticed that this version is linting into multi-line HTML comments, ignoring our html_elements: false setting, at least for rule MD044.
This doesn't work:

This gets ignored as expected:

It looks like it only works if the line either starts with :

Also have a I problem with this parameter.
Works for single line HTML, but doesn't work for multi-line:

@okalachev, OCR and attempting to reproduce your own example gives me two issues, not one. What am I missing?
https://dlaa.me/markdownlint/#%25m%23 Issue 832

<%2Fdiv>

<%2Fdiv>

@DavidAnson, sorry for using an image instead of text, but I wanted to emphasize the detected error.
Here is what I'm showing: https://dlaa.me/markdownlint/#%25m%23 Issue 832

<%2Fdiv>

<%2Fdiv>

.
Two single-line HTMLs (OK), one multiline (not OK).

Your example shows the same behavior as mine. This difference is probably not important since the fix for both scenarios is probably the same. I'll have a look soon.

(Aha, the difference in samples is whether there are blank lines above/below the middle line.)

Where we're at:
@Ravlen, your issue is fixed by the commit above.
@okalachev, your issue has turned up other issues... :)

@DavidAnson As always, you do wonderful work! I'd love to make the PRs to fix these myself, but I'm only smart enough to use your great tool, not help out with it 😅

@Ravlen :) Thanks! I try to make the rules as simple as possible, but some of them are almost absurdly tricky. Maybe if I were smarter, some of this would be more approachable!

Agreed about wonderful work, thanks to this tool we can keep the source code of our big documentation site clean and consistent.
But due to these MD044 issues with HTML I can't switch do the latest version unfortunately.


[FEATURE] Add copy to clipboard functionality

Description
Finally, I have added the copy to clipboard functionality to the website.
What does your PR belong to?

[x] Website
[ ] Snippets
[ ] General / Things regarding the repository (like CI Integration)

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] Enhancement (non-breaking improvement of a snippet)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have checked that the changes are working properly
[x] I have checked that there isn't any PR doing the same
[x] I have read the CONTRIBUTING document.

Why did you add it to static index and created a bundle.js file?


Simple notes from Migrating to PlusKit

cover_image: https://user-images.githubusercontent.com/6764957/185548551-cdbe7da1-a15f-43dd-9c8c-4aa10a15d1ef.png
"PlusKit" is my affectionate name for the Great SvelteKit Redesign of 2022. I was a little apprehensive at the amount of work and learning curve there would be, but it took about 1 hour all told (for admittedly a very simple sveltekit app like swyxkit).
You can see the final PR here: https://github.com/sw-yx/swyxkit/pull/88/
What follows are just raw notes for "simple" sveltekit users like me. I want to stress that this is an incomplete list - however, I guess that if a "simple" usecase required these refactors, then they are the essential ones that most people will run into.
Run the migrate script
npx svelte-migrate routes

This is pretty straightforward - the migration script moves your code around and sometimes indiscriminately - in some cases you may have to move back some code (by uncommenting it back in) and in others you may have to refactor return signatures and imports. I found all the @migration hints to give enough warning - I solely looked at the @migration hints and that was sufficient.

Follow the instructions, but you'll want to upgrade your sveltekit version as well: npm i<EMAIL_ADDRESS>The 4 kinds of refactors
Static imports
Since we no longer have

This was fairly easy.
Special page values
Now that we don't have load() in a page, we dont have access to url, status, or error from load(). Pull them from the page store instead:
-

-	{#if status === 404}
-
There is no post at the slug {url.pathname}.

-
Try searching for it here!

+	{#if $page.status === 404}
+
There is no post at the slug {$page.url.pathname}.

+
Try searching for it here!

...

You'll want to look up what you can get from the Page store but otherwise it's also pretty straightforward.
Refactor return values
I think this took the most time for me. There a few refactors to do:

Refactoring error values
Refactoring return values and headers for pages
Refactoring return values and headers for API routes

See migration docs.
Refactoring Errors:
// before: page.svelte
-			if (res.status > 400) {
-				return {
-					status: res.status,
-					error: await res.text()
-				};
-			}

// after: +page.js
+ import { error } from '@sveltejs/kit';
// ...
+		if (res.status > 400) {
+			throw error(res.status, await res.text())
+		}

Refactoring return values and headers (using the new setHeaders api):
-	export async function load({ params, fetch }) {
-		    const slug = params.slug;
-		    let res = null;
-		    res = await fetch(`/api/blog/${slug}.json`);
-		    return {
-			    props: {
-				    json: await res.json(),
-				    slug,
-				    REPO_URL
-			    },
-			    cache: {
-				    maxage: 60 // 1 minute
-			    }
-		    };

+ export async function load({ params, fetch, setHeaders }) {
+		const slug = params.slug;
+		let res = null;
+		    res = await fetch(`/api/blog/${slug}.json`);
+		    setHeaders({
+			    'cache-control': 'public, max-age=60'
+		    });
+		    return {
+			      json: await res.json(),
+			      slug,
+			      REPO_URL
+		    }

This refactor looks slightly different for API routes:
-	try {
-		data = await getContent(slug);
-		return {
-			body: JSON.stringify(data),
-			headers: {
-				'Cache-Control': `max-age=0, s-maxage=${60}` // 1 minute.. for now
-			}
-		};
-	} catch (err) {
-		return {
-			status: 404,
-			body: err.message
-		};
-	}

+ import { error } from '@sveltejs/kit';
// ...
+ try {
+		data = await getContent(slug);
+		return new Response(JSON.stringify(data), {
+			headers: {
+				'Cache-Control': `max-age=0, s-maxage=${60}` // 1 minute.. for now
+			}
+		});
+	} catch (err) {
+		error(404, err.message)
+	}

data props
Per docs, we now no longer have individual export let declarations for a +page.svelte - everything comes into export let data and the types are supposed to flow from its corresponding +page.js:
+	/** @type {import('./$types').PageData} */
+	export let data;

	/** @type {import('$lib/types').ContentItem} */
-	export let json; // warning: if you try to destructure content here, make sure to make it reactive, or your page content will not update when your user navigates
+	$: json = data.json;

(in practice I found PageData to still be any here but I am assuming those are growing pains)
Done
There are more smaller changes to handle (I dont use stuff, I don't use session, I dont have a +page.server.js, etc), but the above was all that affected me and so I guess will be the major ones to take care of for most people, hence me writing this up. All told, about 400 lines of code affected for me.

Once again, you can see the final PR here: https://github.com/sw-yx/swyxkit/pull/88/

Thanks you for the notes. Good to know before starting out.
The git annotations  bluish text on violent green background is hard to read. 👴

yes i know - first time i am using the diff annotations thing so i never knew about this clash. made an issue here #93

Nice! I'm not sure if siteConfig is something that's inherent to Swyxkit but since there's a new environment var/secrets handling functionality in SvelteKit maybe this can be deprecated? 🤔

@kevmodrome good point, i should look into that.

@kudadam agree, i just lowered the opacity https://github.com/sw-yx/swyxkit/pull/160


feature: add authentication for api routes

Add an authentication system based on Google Oauth.
To access the dashboard (aka app) to book a shift, register food, etc, a volunteer (or a user) must do login or, in other words, be authenticated. Added authentication and authorization steps in our API to keep this access private for every user with an account that they can use to be authenticated and authorized.
How it works
A user can do login in to our application using Google as a provider. Right now is the only way to do it, so in this case, we keep the same flow that we have nowadays.
When a user accesses, we delegate this process in Google Oauth. If it's ok, we create an internal token (based on jwt) and save it in a cookie.
With a cookie, every time that we make a request to the API, the browser will add the cookie as part of it, so in the API the process to check the authorization is easy. In the frontend, we reduce the code a lot because the way to send the token to the API is automatic by the browser
Considerations
As the project is based on the web platform, using a cookie to keep the session is the easiest and most secure way to keep the information about the authentication. Cookie set in the server, based on HTTPS, secure, signed and based on lax SameSite rules (to be able to send from the app website to the API). In this case, nobody can edit the cookie (as they can do with the localStorage) to edit or try to attack our API.
If a user can't access because of authorization or authentication, we send a 401 error based on authentication or 503 based on service unavailable in case Google fails.
Note: We will need to revisit this decision or provide a new way for the authentication in case we would like mobile devices to have access to the API.

@madboks/devs I just updated and changed the logic to authenticate in the app instead of on the web. I added some routes as an example while we don't work on the app interface.


GetRoleMappings deserialisation exception if response includes array of dn strings

NEST/Elasticsearch.Net version: 7.10.1
Elasticsearch version: 7.10
Description of the problem including expected versus actual behavior:
When a role mapping contains an array of possible distinguished names, it causes a deserialisation error in NEST.
Steps to reproduce:

Add the following role mapping to the Elasticsearch cluster by issuing:

PUT /_security/role_mapping/test%20admin%20role%20mapping
{
    "enabled" : true,
    "roles" : [
      "apm_user"
    ],
    "rules" : {
      "any" : [
        {
          "field" : {
            "dn" : [
			  "CN=Bloggs Joe abcdef01,OU=Users,OU=_Central,OU=S1000,OU=SG001,DC=ad001,DC=example,DC=net",
              "cn=bloggs joe abcdef02,ou=usersfunctional,ou=_central,ou=accadm,OU=SG001,DC=ad001,DC=example,DC=net"
            ]
          }
        }
      ]
    },
    "metadata" : { }
}

Use NEST to get the role mappings

var client = new ElasticClient();
var resp = client.Security.GetRoleMapping();

Expected behavior
The response should be successfully deserialised.
** Actual behavior**
An UnexpectedElasticsearchClientException is throw in the deserialisation code.

Fixed in #5271

Fixed in #5271


Add docs for how to upload static content

resolve #111

Codecov Report

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

@@           Coverage Diff           @@
##           master     #143   +/-   ##
=======================================
  Coverage   62.01%   62.01%
=======================================
  Files          49       49
  Lines        1535     1535
  Branches      168      168
=======================================
  Hits          952      952
  Misses        522      522
  Partials       61       61

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 9ff030c...3241522. Read the comment docs.


Trouble with ThreadPoolExecuter

java.util.concurrent.RejectedExecutionException is thrown from time to time in
CommonSearcherFiltered$FilterAsyncFuture#createFuture
Tests pass, but they most probably shouldn't then.
Seems a bit strange, though it's a ambitious what is done here.
Edit: tests pass even if the Exception is not swallowed as in current version.

Closing.
Exceptions are still thrown in tests, but this is ok and expected within the corresponding test.


Add files via upload

Adding Alrayan Medical Colleges

@tabarani First stage of verification passed, official domain is amc.edu.sa as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision.

@tabarani Thank you for your patience while waiting for a review and processing your request. Pull request merged.


Add missing package property to the pre-commit docs

pre-commit-hooks now require package property which was not present in the docs.

It defaults to null if you use newest pre-commit repo rev.

Can be closed then.


SqlServerBatchRunner handling of null parameters

This pull request contains a small change to address an issue I encountered with the SqlServerBatchRunner when dealing with string parameters in expressions. Where a string parameter in the expression has a null value, the SqlServerBatchRunner currently adds Parameters to the ADO.Net Command objects with null values rather than DbNull.Value which causes an exception to be thrown in some cases.
[I have recreated this pull request as the previous one was accidentally created from the wrong branch]
The type of query expression that triggered this problem is of the type:
db.Users.Where(u => u.EmailAddress.EndsWith(emailDomain) && (string.IsNullOrEmpty(optionalComparisonString) || u.AvatarType == optionalComparisonString))
.Delete();
The query can alternatively be rewritten in pieces to avoid this problem... (ie add the filter expression only where the optional parameter is non-null).. however when there are a number of these kinds of parameters it is more convenient to use the form above.
I have added some tests to cover this.

Already fixed.


Shadow scripts contain the version of Racket

I recently discovered that a shadow script contains a line the specific version of Racket in which it was created.  Note the 8.4 in the line below.
(require quickscript
         (prefix-in shadow: (file "/home/user/.local/share/racket/8.4/pkgs/quickscript-extra/scripts/sections.rkt")))

In this case, it was very surprising to me that deleting the Racket 8.4 files caused me to get errors on startup in Racket 8.15. Fortunately it was not hard to debug, so I suppose one should not spend too long fixing it.
If you don't have a better suggestion, consider including a warning in the documentation for shadow scripts. Generating a warning on first startup of a new version of Racket would be even better.

Someone else than me uses shadow scripts! o_O
The problem is that shadowed scripts may be from package, or not. But this can't be straightforwardly found out just from calling the script unfortunately — I think. Though I do remember considering this at some point :thinking:
Packages are not terribly well handle by Quickscript currently, I'm afraid. There's some work in progress though.
Would adding a line in the docs have helped you in this instance?
I'm not sure how to find out that DrRacket is started after an upgrade of the racket version.

Packages are not terribly well handle by Quickscript currently, I'm afraid. There's some work in progress though.

I looked, but that's all. Thanks for the pointer.

Would adding a line in the docs have helped you in this instance?

No. It only took one more minute to find and fix the problem after reading that page (which otherwise did not help).

I'm not sure how to find out that DrRacket is started after an upgrade of the racket version.

Somewhere the directory for quickscript-extra in (new version of Racket) is added to the Quickscript Directories. That would be one potential place for  "hook". I guess that happens during the package migration process, though.
Maybe Quickscript installation could install a self-removing "first-run" flag (or script? or preference option?) that could be detected when the menu is built after installation.
Thinking about this, Quickscript could check the directories for two different versions of Racket and issue a warning (or error, or offer to fix it automatically). If a hack like me did this check, it would find directories containing the string $RACKET_VERSION and then check to see if there were strings with other numbers in that place. Approached that way, this might be a "good first issue".
Seems like this might be overengineering a small inconvenience.

I'm thinking that reporting a better error, along with giving the user potential ways to fix them might be a better approach in the short term.
This would avoid the potentially nasty and wrong 'automatic fixes' that we may guess could work, while at least being more helpful.
What error message would you have liked to see?

I cannot reproduce the problem. Maybe I didn't read the error message carefully enough the first time. The current error message is fine - perfectly informative.
I was going to say: a good error message would be something like: "script XXX at location FULLPATHNAME failed to load, throwing error: ORIGINAL ERROR", but the error message I get attempting to reproduce has this information.
I think the original error was triggered by a change in the configuration that did not cause a recompilation... but right now everything I try causes a recompilation, which generates a nice error.
OK to close unless you want to test more.

Ok, great to hear. If you happen to find a way to reproduce your original error, let me know! I'm closing the issue for now.


Add machine pkg for machineset support

Add pkg/machine/machineset.go

@kononovn @trewest @cdvultur please review

@kononovn @trewest @cdvultur @achuzhoy please review

@kononovn @trewest @cdvultur @achuzhoy  please review, last comments addressed.  Thanks.

@kononovn @trewest @cdvultur @achuzhoy please review, all comments addressed.  thank you !


#1144 Add SW convenience parameter (max_age)

SWs can now be created using a WITH param:
CREATE CONTINUOUS VIEW cv WITH (max_age = '10 days') AS SELECT count(*) FROM stream
CREATE VIEW v WITH (max_age = '1 day') AS SELECT * from cv;

Looks good! This is probably a more intuitive way of thinking about sliding windows.


Update React Native Example

Closing since it's been a while and will probably have to create a new example from scratch


[question] - How download the free version

Hello,
i m looking how to download the application,
can u send me a link please
thanks

@gamersalpha whats your OS? DM me on @louis030195 on discord/x

https://screenpi.pe/


Remove options method

Hi, im trying to clean the generated client, for example, i want to remove the options method
    Module "api" already has method "optionsApi()"
⚠    This method has been renamed to "optionsApi105()" to solve conflict names.
⚠    Module "api" already has method "optionsApi()"
⚠    This method has been renamed to "optionsApi106()" to solve conflict names.
⚠    Module "api" already has method "optionsApi()"
⚠    This method has been renamed to "optionsApi107()" to solve conflict names.
⚠    Module "api" already has method "optionsApi()"
⚠    This method has been renamed to "optionsApi108()" to solve conflict names.
⚠    Module "api" already has method "optionsApi()"
⚠    This method has been renamed to "optionsApi109()" to solve conflict names.
⚠    Module "api" already has method "optionsApi()"
⚠    This method has been renamed to "optionsApi110()" to solve conflict names.
⚠    Module "api" already has method "optionsApi()"
⚠    This method has been renamed to "optionsApi111()" to solve conflict names.

i tried using the hooks but still didnt find a way to do that
       onInit: (configuration) => {
            const { paths } = configuration.swaggerSchema;
            for (const path in paths) {
                if (Object.hasOwnProperty.call(paths, path)) {
                    const { options, ...rest } = paths[path];
                    if (options) { configuration.swaggerSchema.paths[path] = rest; }
                }
            }

            return configuration;
        }

Did you manage this in the end?


Lack of read-writeonce support?

When trying with updated set of SVD's (taken from nrfx I ended up with following traceback:
Error: In device `nrf52`

Caused by:
    0: In peripheral `APPROTECT`
    1: In register `FORCEPROTECT`
    2: Parsing unknown access at 1010:11
    3: unknown access variant 'read-writeonce' found

And there's also issue with outdated SVD's, at least SVD for nrf52832 seems to originate from between nrfx 1.0.0 and 1.1.0 releases and has some fixes/updates (for example APPROTECT register set)

Ugh, apparently correct capitalization is read-writeOnce.

there's no way to enforce the "write once" part, i'd just treat it as readwrite. PRs welcome.


fix: parquet prefix_ids defaults to wb

Remove unnecessary - suffix from prefix_ids (I should have caught this when reviewing #821).
#821 changed the default prefix_ids to wb-. The parquet output handling code sets the feature location to {prefix_ids}-{location_id}. So, the default would now be wb--{location_id}. We want wb-{location_id}. This fixes that.
related to #821

Thanks, @shorvath-noaa!


Misc fixes 1103

Removed empty tab (Examples) and fixed metadata in the licensing article.

Docs Build status updates of commit e9769dd:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

articles/contributing/style-guide.md
:white_check_mark:Succeeded
View (qsharp-preview)

articles/license.md
:white_check_mark:Succeeded
View (qsharp-preview)

articles/overview/what-is-quantum-computing.md
:white_check_mark:Succeeded
View (qsharp-preview)

articles/welcome.md
:white_check_mark:Succeeded
View (qsharp-preview)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


New icons

Pause, Play, Refresh

Deploy preview ready!
Built with commit ad4109457442449a0f77e9324d8a1605658a5add
https://deploy-preview-305--mcs-lite-introduction.netlify.com

Deploy preview ready!
Built with commit ad4109457442449a0f77e9324d8a1605658a5add
https://deploy-preview-305--mcs-lite-ui.netlify.com

Deploy preview ready!
Built with commit ad4109457442449a0f77e9324d8a1605658a5add
https://deploy-preview-305--mcs-lite-mobile-web.netlify.com


Need bitsandbytes0.35.0

Can you consider supporting bitsandbytes 0.35, please? I really need it and would greatly appreciate it.

Currently building 0.35.0 and will also build 0.35.4 when that finishes.
I am very limited in what I can test with this version due to it not supporting inference on Pascal cards.

0.35.0 is ready, 0.35.4 will be soon:
0.35.0:
python -m pip install bitsandbytes==0.35.0 --prefer-binary --extra-index-url=https://jllllll.github.io/bitsandbytes-windows-webui

0.35.4:
python -m pip install bitsandbytes==0.35.4 --prefer-binary --extra-index-url=https://jllllll.github.io/bitsandbytes-windows-webui

Please let me know if it works. I am unable to test it much due to the aforementioned lack of Pascal support.

0.35.0 and 0.35.4 is ready:
0.35.0:
python -m pip install bitsandbytes==0.35.0 --prefer-binary --extra-index-url=https://jllllll.github.io/bitsandbytes-windows-webui

0.35.4:
python -m pip install bitsandbytes==0.35.4 --prefer-binary --extra-index-url=https://jllllll.github.io/bitsandbytes-windows-webui

Please let me know if it works. I am unable to test it much due to the aforementioned lack of Pascal support.

Thank you for the work you have done. I have tested it, and it is currently no issues.


How about technically moving to tauri

How about technically moving to tauri

Another related issue #25

https://github.com/Shotman/opensumi-tauri
I've integrated the ide lite quite easely in Tauri but I'm not familiar enough with how OpenSumi and React works to change calls from github to the filesystem (tauri JS api have path and fs APIs available) easely so if anyone wants to take a look etc feel free while I still try to figure things out

Maybe we need official support for tauri ? @Shotman


Result variable for DSL

This adds an extra parameter ("variable") to the DSL step. It is used as the name of an environment variable that will receive the comma separated list of the names of the locked resources while the block executes.

+1 for this feature.
Any plans to implement soon?

1
Can anyone provide the hpi for this, we are struggling to build this internally.  please merge

Merged. Delaying the release a bit until I review other PRs that could be released together.

Small question to those of you who use a recompiled version of the plugin including this PR: I have 6 instances of a particular resource type available in Jenkins and a pipeline with a test stage including 10 parallel branches. Each branch requests one instance of the resource before starting.
What I'm observing is that 6 branches start and receive one instance properly through the environment variable. However, for the other 4 branches, instead of blocking and waiting for some resource instances to become available, they start immediately and the environment variable that is supposed to contain the resource instance is not set.
In the Jenkins file, I'm using the following piece of code to acquire the resource:
lock(label: "test-server", quantity:1, resource:null, variable:"SERVER_IP") { ... }
Anyone else is encountering this?

Should I update the lockable resource plugging to get this latest change work in my jenkins ?

I'm finding that the variable is not set if the lock has to queue waiting on a resource to be available.
def steps = [:]

steps['stage1'] = {
    lock(label: 'resourcePool', quantity: 1, variable: "STAGE1_VM"){
        echo "locked resource: ${env.STAGE1_VM}"
        echo sh(returnStdout: true, script: 'env')
        sleep 10
    }
}

steps['stage2'] = {
    lock(label: 'resourcePool', quantity: 1, variable: "STAGE2_VM"){
        echo "locked resource: ${env.STAGE2_VM}"
        echo sh(returnStdout: true, script: 'env')
        sleep 15
    }
}
steps['stage3'] = {
    lock(label: 'resourcePool', quantity: 1, variable: "STAGE3_VM"){
        echo "locked resource: ${env.STAGE3_VM}"
        echo sh(returnStdout: true, script: 'env')
        sleep 5
    }
}
steps['stage4'] = {
    lock(label: 'resourcePool', quantity: 1, variable: "STAGE4_VM"){
        echo "locked resource: ${env.STAGE4_VM}"
        echo sh(returnStdout: true, script: 'env')
        sleep 20
    }
}

node('master'){
    parallel steps
}

Results in stage3 and stage4 printing 'null' for the env.STAGE3_VM and env.STAGE4_VM variables, even though resources are locked correctly.

@boblloyd

Results in stage3 and stage4 printing 'null' for the env.STAGE3_VM and env.STAGE4_VM variables, even though resources are locked correctly.

Interesting, can you file a bug in JIRA please? Just copy & paste your comment in the description. Thanks.
Maybe @sheepdreamofandroids can have a look as the author of the feature? I'll do it otherwise.

I see only one way that could happen: the parameter final List<String> resourcenames of the proceed method is modified before the block starts.
The easiest way to fix that is to move the line final String resources = COMMA_JOINER.join(resourcenames); out of the inner class to right after the enclosing if statement.


[Feature Request] - Lapis lamp

Describe the solution you'd like
I want to see inside the mob farm, but is totally dark, and if I put some light, the curse earth will disappear. In previous versions of Minecraft I could use a Lapis lamp from Random things https://ftb.fandom.com/wiki/Lapis_Lamp. But I don't know of any option in >1.16, and it could be something nice to have.

Added in PR.

I went ahead and compile the latest commit (with @MorningSage PR) and tested the Lapis Lamp.
It doesn't seem to work if Optifine is installed. I don't know if this is something you want to fix or not, but I thought you wanted to know.


Update sbt-tpolecat to 0.1.16

Updates io.github.davidgregory084:sbt-tpolecat from 0.1.15 to 0.1.16.
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 = "io.github.davidgregory084", artifactId = "sbt-tpolecat" } ]

labels: sbt-plugin-update, semver-patch

Codecov Report

Merging #25 (47754a4) into develop (df8955b) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop      #25   +/-   ##
========================================
  Coverage    82.35%   82.35%
========================================
  Files            3        3
  Lines           51       51
========================================
  Hits            42       42
  Misses           9        9

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 df8955b...47754a4. Read the comment docs.


x/pkgsite: cannot update package to v2

What is the URL of the page with the issue?
<EMAIL_ADDRESS>What is your user agent?

Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:82.0) Gecko/20100101 Firefox/82.0
Screenshot

What did you do?

I tried sending a request to update my module to the latest
Repo: https://github.com/Delta456/box-cli-maker
What did you expect to see?
Information got updated
What did you see instead?
404 Page Not Found

Seems to be working now https://pkg.go.dev/github.com/Delta456/box-cli-maker/v2


A banner to notify of possible problems with adblock on analytics page [OSF-6307]

Purpose
fix JIRA problem OSF-6307
Changes
A banner shows up if the user has adblock on the analytics page
Side effects
There shouldn't be any.
Ticket
https://openscience.atlassian.net/browse/OSF-6307

Good start, but needs a couple changes. Pass done. 🐧

Pass done. 🐧


AmethystWindows not working after Windows 22H2 update

Source
AmethystWindows
Summary
Stopped working
Date
27/01/2023 14:09
Status
Report sent
Description
Faulting Application Path:	C:\Program Files\WindowsApps\f04c436a-4cda-455d-b409-4520a608c6d0_<IP_ADDRESS>_neutral__wbnnev551gwxy\AmethystWindows\AmethystWindows.exe
Problem signature
Problem Event Name:	MoAppCrash
Package Full Name:	f04c436a-4cda-455d-b409-4520a608c6d0_<IP_ADDRESS>_neutral__wbnnev551gwxy
Application Name:	praid:App
Application Version:	<IP_ADDRESS>
Application Timestamp:	6355849f
Fault Module Name:	KERNELBASE.dll
Fault Module Version:	10.0.22621.1105
Fault Module Timestamp:	c16aac06
Exception Code:	e0434352
Exception Offset:	00147452
OS Version:	10.0.226<IP_ADDRESS>.768.101
Locale ID:	3072
Additional Information 1:	2beb
Additional Information 2:	2beba6fb4680d73a8c78ca7c24ccdb46
Additional Information 3:	3636
Additional Information 4:	363600d01b0027a84ddf27d112eb0d6d
Extra information about the problem
Bucket ID:	8f0cbb5cb4834283d13da41b9d30293e<PHONE_NUMBER>732376382)

Please move to v2

Thank you for the quick reply! This happened with v <IP_ADDRESS>. I also tried other v2 subversions with the same result.

Same here, I tried installing v<IP_ADDRESS> on 22H2 (22621.1194) and upon launch it tiles windows but none of the keyboard shortcuts work. Manually trying to change them makes no difference.

Same issue as @xmika over her on 22H2 as well.  Crashes every few minutes and only one keyboard shortcut works (mod1 + space).


Question on match_by

According to the documentation, scratchpads can be identified according to pid, class, tittle, when class and not pid is selected as below
match_by="class"
Are there process_tracking  tracked by their pid or their class, or the match_by is only to determine position, decoration e.t.c... and not used in process tracking?

Yes I was wondering that if process_tracking = true and the match_by = class, can we not start terminal apps that share same PID but have  different class set by their scratchpad configs. The solution you proffered in #144 did not work as envisaged, totally disabling process_tracking is the only thing that worked.
I was wondering that can we not force lazy=true when process_tracking = false, I tried to force it on for an app that I want to start at start upbut it did not work.

if you enable process tracking, there will be some conflicts indeed. It assumes the window has been launched by the process specified under "command".
Not sure what you try to achieve, but when process_tracking is False, it's ALWAYS using lazy mode (which is the default anyway).

pardon me, what I want to achieve is set some apps to auto launch at login even  if process_tracking=false, however setting to lazy=false did not achieve the desired result.

yes, because lazy is FORCED when you have no process tracking to avoid some issues. I can tell you how to disable this behavior if you like to experiment.

yes, because lazy is FORCED when you have no process tracking to avoid some issues. I can tell you how to disable this behavior if you like to experiment.

I can give it a try if it causes more issues than the benefits it will provide, then I can continue to use the method I am using which is  exec-once = sleep 5 && pypr show btop && pypr hide btop    in hyprland config, the sleep is to ensure pypr is fully initialized.

You can just remove one line of code in pyprland/plugins/scratchpads/objects.py around line 98:
diff --git a/pyprland/plugins/scratchpads/objects.py b/pyprland/plugins/scratchpads/objects.py
index 9b3fc53..9eff5dd 100644
--- a/pyprland/plugins/scratchpads/objects.py
+++ b/pyprland/plugins/scratchpads/objects.py
@@ -95,7 +95,6 @@ class Scratch(CastBoolMixin):  # {{{
         if not self.have_command:
             opts["match_by"] = "class"
         if not opts.get("process_tracking", True):
-            opts["lazy"] = True
             if "match_by" not in opts:
                 opts["match_by"] = "class"
         if state.hyprland_version < VersionInfo(0, 39, 0):

You'll need to restart pyprland daemon for this change to have any effect.

Any update ?

Yes let us close the issue, I could not find a way of carrying out the modification you suggested and new hyprland versions did not fix the issue.


Customize template

Hey,
can someone help me. How can I modify default template to bootstrap version? Because input's doesn't have a class "form-control".
I have no idea what should I insert/delete/modify in default template :(
Thanks

Did you configure Symfony to use Bootstrap form-templates?
http://symfony.com/blog/new-in-symfony-2-6-bootstrap-form-theme

As @V34s pointed out, you can change the form render as you want. It is a Symfony feature, that we don't override in any way.
Also, if you need to change the main template for the formType, you could probably do it, by extending our template. This is a Symfony feature too: https://symfony.com/doc/current/templating/overriding.html
Closing. If you feel this is an issue still, please ping me to reopen with more information.


(De)serialization of objects with property-name same as object-name stopped working correctly since V3.2

Given the classes
public class Person
{
    public string LastName { get; set; } = string.Empty;
    public string FirstName { get; set; } = string.Empty;
    public string Salutation { get; set; } = string.Empty;
}
public class Item
{
    public string Description { get; set; } = string.Empty;
    public string Person { get; set; } = string.Empty;
    public byte[] Content { get; set; }
}

the code
var item = new Item();
item.Person = "Testperson";
item.Description = "Testing ExtendedXmlSerializer";
item.Content = new byte[] { 12, 13, 14, 15, 16 };

string content = serializer.Serialize(new XmlWriterSettings { Indent = true }, item);

var deserializedItem = serializer.Deserialize<Item>(new XmlReaderSettings(), content);

results in deserializedItem.Person being string.Empty and deserializedItem.Content being null
The resulting xml after serialization looks like
<?xml version="1.0" encoding="utf-8"?>
<Item xmlns="clr-namespace:TestingRetryItem;assembly=TestingRetryItem">
  <Description>Testing ExtendedXmlSerializer</Description>
  <Person xmlns:exs="https://extendedxmlserializer.github.io/v2" exs:member="">Testperson</Person>
  <Content>DA0ODxA=</Content>
</Item>

The problem here seems to be exs:member="" because after manually deleting this from the resulting xml the deserialization works. There is no exception thrown neither at serialization nor deserialization.
The order of the properties is relevant as well, because e.g having the Item class look like
public class Item
{
    public string Description { get; set; } = string.Empty;
    public byte[] Content { get; set; }
    public string Person { get; set; } = string.Empty;
}

the Content property isn't null anymore. It just seems that after accessing a property with a name equal to a class-name the deserialization stops without throwing an exception.

Hi @Heslacher thank you for writing in and reporting this.  Unfortunately I am having a bit of trouble here understanding the exact problem. The following passes for me when I attempt to run it in 3.7.10:
namespace ExtendedXmlSerializer.Tests.ReportedIssues
{
	public sealed class Issue598Tests
	{
		[Fact]
		public void Verify()
		{
			var item = new Item
			{
				Person      = "Testperson",
				Description = "Testing ExtendedXmlSerializer",
				Content     = new byte[] { 12, 13, 14, 15, 16 }
			};

			var container = new ConfigurationContainer().ForTesting();
			container.Cycle(item).Should().BeEquivalentTo(item);

		}
	}

	public class Person
	{
		public string LastName { get; set; } = string.Empty;
		public string FirstName { get; set; } = string.Empty;
		public string Salutation { get; set; } = string.Empty;
	}
	public class Item
	{
		public string Description { get; set; } = string.Empty;
		public string Person { get; set; } = string.Empty;
		public byte[] Content { get; set; }
	}
}

If you are saying that you serialized something in 3.2 but it no longer does so in 3.7.x I am not sure how much of a help I can be toward resolving this.  While I do my best to carry over previous functionality, I am really the only developer on this project so problems may be had. :)
That stated, please let me know what I am missing here and I can continue to see if I can assist you. 👍

Hehe indeed @Heslacher I noticed with your code the configuration of the container is missing so maybe it might have something to do with this? 🤔  With a default container everything appears to work with the code you have provided. 👍

Ok, the problem lies in the creation of the IExtendedXmlSerializer object. Creating it with EnableReferences() will lead to the mentioned problem with deserialization. Omitting this call will lead to a correct result.

Just for completeness this is how I created the IExtendedXmlSerializer object having the mentioned issue:
        IExtendedXmlSerializer serializer = new ConfigurationContainer()
                     .EnableAllConstructors()
                     .EnableReferences()
                     .UseOptimizedNamespaces()
                     .Create();

Ohp... yep, I am able to reproduce this now.  I will poke around to see what's happening here as this seems it should work.  I am very displeased w/ myself for using the exs:member ... there are a couple issues with it and I was seeing if there is some method I added to help with this scenario.  I concede this is not ideal at all, but you may have to consider using a different name for the member name than the type name 😞

I do apologize for the disruption here @Heslacher.  I have been spending the past hour looking into this and it is complex, especially since it's been a few years now since I have been in this code.  The issue is that the XmlReader cursor is on the attribute node and I am not finding an easy way to forward it to the content without breaking existing tests.
However, in your case, since you are using primitive types, we do have an extension called AutoFormatting which moves strings into attributes, and this makes your test pass.
The recommendation here is to use the following configuration:
        IExtendedXmlSerializer serializer = new ConfigurationContainer()
                     .EnableAllConstructors()
                     .EnableReferences()
                     .UseOptimizedNamespaces()
                     .UseAutoFormatting()
                     .Create();

This does change your documents (it should decrease overall size), but you might want to verify things work on your end after the change.  Let me know how you would like to proceed here.  Unfortunately I do not have as much time as I did in the past to look into such issues, so we'll have to do the best with the time we do have. 👍

You don't need to apologize. I have fixed the issue by using EnableReferences() for serialization and omitting it for deserialization. I have tested it and it works.
Thanks again for your help and this wonderful project.

Thank you very much!  I appreciate that.  I definitely feel beholden to this project and all the developers that depend on it, so when things don't work I take that responsibility seriously.
I like your other solution as well.  I didn't even think of that, and hopefully, if someone in the future runs into this problem they have a couple of workarounds they can possibly implement.
Anyways, thank you for your understanding and kind words.  You made my day. 🙏  Closing for now.


Is there a way to filter out some colorschemes?

Info

Operating System: Manjaro Linux, 23.0
Shell: Zsh
Terminal: Kitty
nvim --version: 0.9.0
fzf --version: 0.40.0

Description
First, thank you so much for your work. This plugin is amazing. I use it everyday and I love it :heart:!
I just have a simple question: is there a way to filter out some results when opening the list of colorschemes?
I never use some of the default ones that are ugly (like blue, default or desert) and I would rather filter them out than deleting them manually in my filesystem everytime I update neovim.
Thank you really much for your help.

https://github.com/ibhagwan/fzf-lua/commit/758d4ddb2a9447f21a8dc60799e57a9592edd170
@Einenlum, update to the latest commit and set colorschemes.ignore_patterns  in your setup, can also be called directly:
:= require("fzf-lua").colorschemes({ ignore_patterns = { "^delek$" , "^blue$" } })

Note that the matching is performed using lua's string.match so using the pattern blue will exclude all names that cotnain blue.
Lua patterns are very similar to regex, for more info read this.

Wow, I didn't expect such a quick answer and even an implementation!
It works amazingly. Exactly what I needed.
You rock, @ibhagwan. Thank you so much :heart: !

Yw @Einenlum and ty for the kinds words :)


AttributeError

Dear authors,
I've installed SEDR == 1.0.0
and get an error message below.
AttributeError: module 'SEDR' has no attribute 'graph_construction'
any suggestions?
Thank you in advance!

@skim245  Hi, I have solved this error: `ImportError: cannot import name 'res_search_fixed_clus' from 'SEDR.clustering_func'. This is caused by an imcomplete update. I have revised the code accordingly. Please try again to see if it works for you.
But I have no idea about this one: AttributeError: module 'SEDR' has no attribute 'graph_construction'. Could you provide more message? Such as which command or code generate this error.

@Xuhang01 updates in the code solved all the issues! both of the errors were due to incomplete installation of SEDR.


Is string trim() faulty or am I not using it correctly?

I ran:
fn run(hello) {
    print(hello);
    print(hello.trim());
}
run("    hello   ");

in the rhai playground and saw:
Waiting for Web Worker to finish loading...
Running script at 2022-03-17T08:18:27.440Z

[PRINT]     hello
[PRINT]

Script returned: ""
Finished at 2022-03-17T08:18:27.443Z

which surprised me.
bug?

trim modifies the string inline.
Try:
fn run(hello) {
    print(hello);
    hello.trim();
    print(hello);
}
run("    hello   ");

Turns out the doc-comments for trim has an incorrect example.  It is now fixed.

Thanks very much for the info. I'll close this.


Publish docker images

Refs #96
TODO

[x] Mount cns executable during run instead of copy it during build
[x] Build and push images
[ ] test build images work aka can run job made in scenario page with ok status
[x] try use docker compose up --no-build ~with sparsely checked out repo (aka only files needed for deploy/arq)~ no need for sparse checkout repo is only 7Mb
[ ] make sure cache is used during docker build. For example change in /app should have speedy bartender build
[ ] maybe, cleanup pr-XXX tag when its corresponding PR is closed
[x] ~maybe, add manual workflow submission with custom build args and own tag~ Build locally instead
[ ] after merge check images with latest tag are made
[ ] after release check images with release version tag are made
[x] ~maybe, for playwright job only install playwright deps, not all deps as they are in Docker image to speed up job~ not worth it
[ ] images for Apple M? . It fails on rollup linux-arm64 node missing, not copying package-lock.json in Dockerflile worked, but is not nice.
[ ] only build linux/arm64 on main and release no on a PR. Unless extra build time is neglible

Building arm64 with qemu is much slower than native amd64 build so we only build arm64 on new main or version.


save changes

What changes are you trying to make? (e.g. Adding or removing code, refactoring existing code, adding reports)
What did you learn from the changes you have made?
Was there another approach you were thinking about making? If so, what approach(es) were you thinking of?
Were there any challenges? If so, what issue(s) did you face? How did you overcome it?
How were these changes tested?
A reference to a related issue in your repository (if applicable)
@sidra @amanda
Checklist

[ ] I can confirm that my changes are working as intended

For HW3, For Aggregate task right query is HAVING total spent > 2000, and for date query the right table name is customer purchases, not purchases and group by month and year too along with customer_id.

Thank you for the updates on the previous homework i will try to fix them.

For homework 5 last section i didn't answer in time for deadline so i uploaded what i had before 11:59pm.

Hello Your HW4 and HW5 is same, you did not write the queries for HW4. Please revise it, I marked your HW6 complete.

I mistakenly saved homework 4 as homework 5 on my computer and pushed it. I clicked save  instead of save as a new file.

@SidraBushra correction made. Correct homework_4 pushed


strawberry-graphql 0.250.1

Hi! This is the friendly automated conda-forge-webservice.
I've started a version update as instructed in #247.
I'm currently searching for new versions and will update this PR shortly if I find one! Thank you for waiting!
Fixes #247

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

Hi! This is the friendly conda-forge automerge bot!
I considered the following status checks when analyzing this PR:

linter: passed
azure: passed

Thus the PR was passing and merged! Have a great day!


SOLR-16353: SplitShardCmd.checkDiskSpace can fail for transient core

https://issues.apache.org/jira/browse/SOLR-16353
TODOs:

[x] set default value for solr.shardSplit.checkDiskSpace

Since it's a known issue that this check doesn't make sense for HDFS, we could document that in the ref guide for an HDFS page.

@dsmiley - About toggling this parameter through client requests: It depends on the use cases it will help. Would it be just to reduce the noise of the warnings or for a real utility? If we do not see other use cases to solve, it could be implemented when they are identified so as not to add a non-useful parameter.


Provide hook with access to schema mappings inspection report

We can expose a hook that provides access to the Report and allows applications to take some actions. The default action would be to log it, which is what we do currently.
Requested in https://github.com/spring-projects/spring-graphql/issues/386#issuecomment-1515046472.

Something like this, which can then be applied from a GraphQlSourceBuilderCustomizer bean with Boot:
GraphQlSource.schemaResourceBuilder()
		.schemaResources()
		.schemaMappingInspection(report -> {
			// ...
		})
		.build();

If this is configured, we won't log the report. It'll be all up to the callback.

Yes.  I think that is a good approach with the customizer + default with the option to use our own implementation.

I think we need to significantly refactor the Report contract to provide higher abstractions and prepare for new behavior.

@danielshiplett I've added a Consumer<SchemaReport> hook to GraphQlSource.Builder that also enables the inspection. So you can configure it, and either log and/or take any other action. If not configured, the inspection does not run. Please, have a look to see what's in the report, and give us your feedback.

@rstoyanchev This seems to work as expected.  I have created a demonstration that invokes Spring's Failure Analysis feature to stop the application and provide feedback.  Here is the configuration class that does that.
Here is some example output where a single mapping and a single registration were in error:
2023-04-28T13:50:26.685-04:00  INFO 2216713 --- [  restartedMain] efaultSchemaResourceGraphQlSourceBuilder : Loaded 2 resource(s) in the GraphQL schema.
2023-04-28T13:50:26.828-04:00  WARN 2216713 --- [  restartedMain] ConfigServletWebServerApplicationContext : Exception encountered during context initialization - cancelling refresh attempt: org.springframework.beans.factory.BeanCreationException: Error creating bean with name 'routerFunctionMapping' defined in class path resource [org/springframework/boot/autoconfigure/web/servlet/WebMvcAutoConfiguration$EnableWebMvcConfiguration.class]: Error creating bean with name 'graphQlRouterFunction' defined in class path resource [org/springframework/boot/autoconfigure/graphql/servlet/GraphQlWebMvcAutoConfiguration.class]: Unsatisfied dependency expressed through method 'graphQlRouterFunction' parameter 0: Error creating bean with name 'graphQlHttpHandler' defined in class path resource [org/springframework/boot/autoconfigure/graphql/servlet/GraphQlWebMvcAutoConfiguration.class]: Unsatisfied dependency expressed through method 'graphQlHttpHandler' parameter 0: Error creating bean with name 'webGraphQlHandler' defined in class path resource [org/springframework/boot/autoconfigure/graphql/servlet/GraphQlWebMvcAutoConfiguration.class]: Unsatisfied dependency expressed through method 'webGraphQlHandler' parameter 0: Error creating bean with name 'executionGraphQlService' defined in class path resource [org/springframework/boot/autoconfigure/graphql/GraphQlAutoConfiguration.class]: Unsatisfied dependency expressed through method 'executionGraphQlService' parameter 0: Error creating bean with name 'graphQlSource' defined in class path resource [org/springframework/boot/autoconfigure/graphql/GraphQlAutoConfiguration.class]: Failed to instantiate [org.springframework.graphql.execution.GraphQlSource]: Factory method 'graphQlSource' threw exception with message: See description
2023-04-28T13:50:26.830-04:00  INFO 2216713 --- [  restartedMain] o.apache.catalina.core.StandardService   : Stopping service [Tomcat]
2023-04-28T13:50:26.840-04:00  INFO 2216713 --- [  restartedMain] .s.b.a.l.ConditionEvaluationReportLogger :

Error starting ApplicationContext. To display the condition evaluation report re-run your application with 'debug' enabled.
2023-04-28T13:50:26.853-04:00 ERROR 2216713 --- [  restartedMain] o.s.b.d.LoggingFailureAnalysisReporter   :

***************************
APPLICATION FAILED TO START
***************************

Description:

Invalid schema mappings.  SchemaReport:

GraphQL schema inspection:
        Unmapped fields: {User=[comments]}
        Unmapped registrations: {User.age=UserController#age[1 args]}
        Skipped types: []

Action:

See description for details of invalid schema mappings.

Thanks for confirming and for the example output! I noticed a few bullet items under the Status section in the README. Is there anything there that remains not done?


Emoji with a gender modifier aren't recognised

Emojis that end with a gender modifier aren't recognised. For example, one version of woman police officer is 👮♀ which is encoded as 1F46E 200D 2640 but we don't recognise the female gender modifier 2640

I suppose it's not a modifier as such, but rather the symbol for female. maybe it's just not in the json db?


Petite [Earth] and Petite [Sky]'s stats are different from OathRO's values

@mi output in OathRO for these monsters:
Monster: 'Petite'/'Petite'/'PETIT' (1155)
Lv:44  HP:4899  Base EXP:2102  Job EXP:1811
HIT:123  FLEE:92  |  Against Monster - 100% HIT:112  95% FLEE:198
DEF:25  MDEF:20  STR:1  AGI:48  VIT:62  INT:55  DEX:79  LUK:60
ATK:360~427  Range:1~15~18  Size:Medium  Race: Dragon  Element: Earth (Lv:1)
Drops:
  - Dragon Canine  53.36% - Dragon Tail  3.01% - Rough Oridecon  1.41%
  - White Herb  10.01% - Flail[2]  1.51% - Zargon  15.01%
  - Aloevera  0.16% - Earth Petite Card  0.02%
Monster: 'Petite'/'Petite'/'PETIT_' (1156)
Lv:45  HP:5747  Base EXP:2236  Job EXP:1869
HIT:118  FLEE:90  |  Against Monster - 100% HIT:110  95% FLEE:193
DEF:10  MDEF:40  STR:1  AGI:45  VIT:45  INT:55  DEX:73  LUK:80
ATK:300~355  Range:1~15~18  Size:Medium  Race: Dragon  Element: Wind (Lv:1)
Drops:
  - Dragon Scale  53.36% - Dragon Tail  3.01% - Elunium  0.62%
  - White Herb  10.01% - Zargon  15.01% - Aloevera  0.16%
  - Sky Petite Card  0.02%

I just updated this, also the hp and exp values, thanks for pointing this error


Added checks for configured images and flavors

What does this PR do?
Now that users can specify images and flavors for each role, we need a check that confirms that all configured images and flavors are available. Which is exactly what this PR does.
How should this be manually tested?
Invalid setting:

In all.yml, try setting image/flavor for some roles to something invalid.
Run provisioning and check that it fails with Image <your image name> is not available.

Valid setting:

Run provisioning with valid images and flavors and check that it is successful.

Is there a relevant Issue open for this?
--
Who would you like to review this?
cc: @tomassedovic @bogdando PTAL

End to End CI running here:
https://travis-ci.org/openshift/openshift-ansible-contrib/builds/268310791


Document origin of Wigner-D(-phi, theta, 0)

The function that formulates Wigner-$D$ functions for two-body decay nodes, formulate_wigner_d(), currently returns $D({\color{red}-}\phi, \theta, 0)$. As noted by @mmikhasenko, this is not in Kutschke or any other papers. Usually, it's either:

$D(\phi, \theta, -\phi)$: Jacob & Wick 1959 Eq (24), Kutschke 1996 Eq (10), Richman 1984 Eq (4.17)
$D(\phi, \theta, 0)$: Chung 2014 Eq (4.16)

Yes, the 2 make perfect sense. It is the same D-matrix

The only complexity in the D-matrix is the azimuthal dependence. Hence, the conjugation is equivalent to flipping the sign of the phi

As an example, you need at least 4 body decay with two different decay chains interfering.
phi -> -phi will change the sign of the interference term.
The sign of this interference is unambiguously fixed in case of identical particles in the final state.
4pi0 might be the simplest system. Checking what sign is physical might be non-trivial.

See also the page 199 where they shift the convention to the (phi, theta, 0)

The discussion I refer to in the Martin Spearman is above Eq. (3.48)


BSOD while setting up llvm runtime

As the title says. My laptop suddenly blue screened while setting up one of the dependencies of clang. It does no harm to the machine and the package can continue to install after rebooting.
I managed to reproduce this by fully uninstall the subsystem and install clang again.

Windows build number: 14931.300 (latest slow ring)

@ringosham - Thanks for reporting this. Can you please share the explicit repro steps?

@sunilmut

Completely reinstall linux and removed all user data
sudo apt install clang
That's all I did.

Thanks for the quick response. I was able to sudo apt install clang (also after a complete reinstall) on the 14955 build, which is the latest insider build. If someone else can also confirm the same, then maybe the issue is fixed in the latest build?

Thanks for reporting this.  I took a look at your minidump and verified that the fix for this issue is in build 14935 and later.
Marking as duplicate of #1034.


test api library against non EU server

Main question is whether non EU server uses same public key or has its own to be considered for the login request and encryption.

Both, the EU and COM Anker server public key is the same and login response is provided for both upon an authentication request
However, if country ID assignment is to the wrong server, no sites or devices will be listed for the authenticated account.


Add CtrlCmd-Backspace as a 2ndary delete_selected_layer keybind

References and relevant issues
Partially addresses https://github.com/napari/napari/issues/7448
Description
In https://github.com/napari/napari/issues/7448 it appeared  that the existing delete_selected_layer keybinding didn't work on macOS or was somehow consumed by the OS:
    'napari:delete_selected_layers': [KeyMod.CtrlCmd | KeyCode.Delete],
It turns out the key labeled delete on a MacBook Pro keyboard actually sends the Backspace signal--see also https://github.com/pyapp-kit/app-model/issues/229#issuecomment-2543421435 . Using actual Delete key via external keyboard or using Fn-delete on the laptop triggers the keybinding.
As a result, in this PR I add KeyMod.CtrlCmd | KeyCode.Backspace as a 2ndary keybinding such that the action can be intuitively triggered on macOS.
This is technically not a bug fix, because the original now primary key binding does work properly, so I'm tagging as enhancement

I don't understand the fail.
The test passes locally and the syntax looks correct to me.

@dalthviz @lucyleeow any ideas what could be going wrong here?

App-model keybindings let you set primary, win, Mac,Linux (gnome) keys for a single keybinding. See https://github.com/pyapp-kit/app-model/blob/a0d95b19c0e8d61ccab6bea69d1adace910d9157/src/app_model/types/_keys/_standard_bindings.py#L85
Maybe this could work? (AFK,sorry for briefness)

Hi there, I think the problem comes from a test (napari/_qt/widgets/_tests/test_shortcut_editor_widget.py::test_keybinding_with_modifiers) using as one of the parametrized keybindings to validate things the Ctrl + Backspace keybinding. Since now that is one of the default keybindings when the test runs instead of the keybinding being updated the warning for conflicts shows and causes the test to timeout the CI (since it gets stuck with the warning pop up). So running locally something like on Windows pytest -vv -x napari/_qt/widgets/_tests/test_shortcut_editor_widget.py I see:

Replacing https://github.com/napari/napari/blob/c41596f242d5594349ff3370aa2f46e07c557b49/napari/_qt/widgets/_tests/test_shortcut_editor_widget.py#L153-L157
For somethign like:
        (
            Qt.Key.Key_Backspace,
            Qt.KeyboardModifier.ShiftModifier,
            [KEY_SYMBOLS['Shift'], KEY_SYMBOLS['Backspace']],
        ),

should be enough to make the test pass. Furthermore, maybe the test should try to mock the warning pop up widget and check it is not called and otherwise make the test fail? Maybe in that way will be easier to determine why could be causing troubles in the CI when doing changes to default keybindings? 🤔

Omg! Of course!
Thank you @dalthviz
I think I will fix the test to use Shift and then make an issue with the issue and suggested fix?

Sorry I realise these are the weird shortcuts are for the viewer buttons that are 'app-model' keybindings but the buttons are still actually handled by action-manager.
Not sure what is best approach but do you think doing a modification for mac like what we do for ctrl/meta is more suitable than the alternative keybinding?
https://github.com/napari/napari/blob/fcb31c14249e7ef1df30fbc98bc4d2ec335f6875/napari/settings/_migrations.py#L139-L159

I don't think the fact that on Win/Linux you now have two shortcuts for it is a huge problem? I think they're pretty intuitive and realistically I probably would've gone for the backspace key first anyway 🤷♀

This was my thinking. If you highlight some text, both will "remove" it.


Pre render peaks with PHP / Laravel

Is there a way to create the JSON peak data using php / Laravel?
All the packages I can find generate PNG files and not JSON data.
Thanks

use audiowaveform tool, see https://wavesurfer-js.org/faq/

Yea as @thijstriemstra points out, you can use the audiowaveform tool (it's a cmd tool) and then run that from an artisan command :)

Thanks @Reached and @thijstriemstra - I'll give this a bash but never used a CMD tool before. Lets see how this goes 🤘

@GigaMick Try it out, and let me know if I can help with anything! :)