Performance issue in readSam() interacts badly with NFS

In my environment, all files are stored on an NFS volume (AWS EFS, specifically).  This seems to cause a certain amount of performance degradation, because AnchorWave loads genome sequences from disk on demand rather than caching them in memory, and so the code is frequently waiting on a read operation against the network file system.
However, the problem becomes severe when running the proali command, because readSam() reads sequence one base at a time from disk while reading the query SAM file (see https://github.com/baoxingsong/AnchorWave/blob/master/src/service/TransferGffWithNucmerResult.cpp#L124).  In my environment, reading the query SAM file for a typical corn genome takes approximately 40 hours, during which time only 5-10% of a single CPU is utilized, due to excessive I/O wait times.  Most of this is spent in repeated calls to getCharByPos() while processing Match codes in the CIGAR string.
I have found that replacing the repeated calls to getCharByPos() with a single call to getSubsequence2(queryGenome, queryChr, currentqueryPosition, currentqueryPosition + cLen - 1), the I/O bottleneck is eliminated and reading the same SAM files takes only about 25 minutes.  In my testing, this appears to generate the same sequence as getCharByPos() and causes no meaningful increase in memory usage.
If I've understood the issue correctly, I would suggest:

patching the readSam() function to use getSubsequence2() instead of readCharByPos()
updating the docs to warn users to avoid using files on a network file system if possible, especially genome FASTAs

Thank you so much Dr. Davis for this insightful message.
I do not have a lot of experiments with network storage. By using FASTA files locally, reading a corn SAM file only takes a few minutes or less than a minute.
To generate alignment with good accuracy, we tuned the base-paired resolved dynamic programming algorithm using a large windows size, which is 100 Kbp by default. So dynamic programming part is costing a high memory, thus the memory is a bottleneck. To increase the number of threads that could run in parallel, we tried to save memory everywhere. That is why AnchorWave loads genome sequences from disk on demand rather than caching them in memory.
I would like to update the docs to warn users to avoid using files on a network file system. If you or your colleagues do not have enough local storage but have a large memory, the alternative way could copy the FASTA file to the memory /dev/shm.

Dr. Song, thank you so much for the amazingly rapid response!  I see you've already updated the docs.
Your suggestions are good ones.  Alas, the environment that I work in does not have fast local disks, and I do not have sufficient permissions to increase the size of /dev/shm.  I believe that many researchers will find themselves in the same situation, depending on the details of the university cluster they use or the cloud environments they work in.
Given that, I hope you will consider merging my pull request #50.  For me at least, it makes the difference between AnchorWave being usable and unusable.  It should improve performance for everyone.  And because it comes before the memory-intensive alignment portion of the algorithm, it will not increase peak memory utilization at all.  Please do let me know if you see some other drawback to the PR that I didn't notice!


unbounded loop could default to infinity if capture has sensitivity >1 in the body

right now sensitivity loops with variable number of iterations are only allowedif the capture has sensitivity 1 in the body, which is ensured by a corrensponding constraint. when loop is used incorrectly (that is capture sensitivity is >1) the constraint fails and typecheker errors. instead we could in this case just scale all contexts with infinity.

We do NOT want this to happen automatically. We do want a good error message instead.


[Tree] Problem nestedset with doctrine second-level-cache

There is a problem with the nestedset tree working with doctrine second-level-cache.
I create the following nestedset tree via sonata-admin with doctrine second-level-cache enabled:
\root
    \branch1
        \leaf11
        \leaf12
    \branch2
        \leaf21
        \leaf22

Сreation order is: root, branch1, branch2, leaf11, leaf12, leaf21, leaf22.
As a result, I have such data in database:
 id  | root_id | parent_id |  title  | lft | rgt | lvl |  path   |        slug
-----+---------+-----------+---------+-----+-----+-----+---------+---------------------
 296 |     296 |           | root    |   1 |  14 |   0 | root    | root
 298 |     296 |       296 | branch2 |   2 |  11 |   1 | branch2 | root/branch2
 297 |     296 |       296 | branch1 |  12 |  13 |   1 | branch1 | root/branch1
 299 |     296 |       297 | leaf11  |   9 |  10 |   2 | leaf11  | root/branch1/leaf11
 300 |     296 |       297 | leaf12  |   7 |   8 |   2 | leaf12  | root/branch1/leaf12
 301 |     296 |       298 | leaf21  |   5 |   6 |   2 | leaf21  | root/branch2/leaf21
 302 |     296 |       298 | leaf22  |   3 |   4 |   2 | leaf22  | root/branch2/leaf22

It can be seen that branch1 fields lft and rgt are set to 12 and 13 respectively, that is incorrect because branch1 has 2 children, so its lft and rgt should differ by no less than 4.
When doctrine second-level-cache is disabled, all the lfts and rgts are assigned correctly.
It seems that setting Query::HINT_CACHE_EVICT hint to true for all the update queries in https://github.com/Atlantic18/DoctrineExtensions/blob/v2.4.x/lib/Gedmo/Tree/Strategy/ORM/Nested.php fixes the issue.
Is there any other solution that does not require fixing DoctrineExtensions code? Or shall I set new pull-request?
My doctrine config:
doctrine:
    orm:
        auto_generate_proxy_classes: true
        metadata_cache_driver: ~
        query_cache_driver:
            type: pool
            pool: doctrine.local_cache_pool
        result_cache_driver:
            type: pool
            pool: doctrine.local_cache_pool
        second_level_cache:
            enabled: true
            log_enabled: true
            region_cache_driver:
                type: pool
                pool: doctrine.local_cache_pool
            regions:
                query_cache_region:
                    cache_driver:
                        type: pool
                        pool: doctrine.local_cache_pool
                timestamp_cache_region:
                    cache_driver:
                        type: pool
                        pool: doctrine.local_cache_pool
                entity_cache_region:
                    cache_driver:
                        type: pool
                        pool: doctrine.local_cache_pool
                relation_cache_region:
                    cache_driver:
                        type: pool
                        pool: doctrine.local_cache_pool
framework:
    cache:
        pools:
            doctrine.local_cache_pool:
                adapter: cache.system

The doctrine entity uses NONSTRICT_READ_WRITE caching mode to entity_cache_region.

I have same problem.
My tree is broken with second level cache (REDIS)

Looks like updating Extensions is the way to go. From Doctrine's documentation:

DQL UPDATE / DELETE statements are ported directly into a database and bypass the second-level cache, Entities that are already cached will NOT be invalidated. However the cached data could be evicted using the cache API or an special query hint.

Adding the hints makes sense. I'd like to get some testing in place with second level caching, to make sure we're getting the updates we expect.
Thank you both!


Thoughts on a Live Lambda Development enhancement

Any thoughts on adding something like the SST Live Lambda Development mechanism to Barbe-serverless?

Yeah this is definitely something we can, and plan to do. I think something similar can be done with aws_fargate_task as well.
I'm bumping it up on my priority list


Fix repeat sigint

Fix repeat sigint in audit_init

LGTM, seems part of the code is already in #38 , can you please clean it up? @wuchang0201

LGTM


Switch Azure's macOS build to supported OS version

Description
The image macOS-10.14 is not available anymore, but macOS-10.15 is still available. So this PR simply switches to that image.
If this is successful, then at least all the Azure pipeline builds of GIL are passing. :)
References
See https://docs.microsoft.com/en-us/azure/devops/pipelines/agents/hosted?view=azure-devops&tabs=yaml for available images in Azure pipelines.
This builds on the following idea mentioned in #642:

As a general idea, we should probably make sure that the existing CI pipelines pass (except for the outdated stuff that may potentially get removed). Otherwise we may just be migrating existing build failures to another CI configuration with boost-ci.

Tasklist

[ ] Ensure Azure pipeline CI with macOS build passes
[ ] Review and approve

Yay, all jobs on Azure Pipelines are now successful: https://dev.azure.com/boostorg/gil/_build/results?buildId=1790&view=results :-)

Awesome!


fix missing match arm false positive

This fixes #3932 by skipping the missing match arm diagnostic in the case any of the match arms don't type check properly against the match expression.
I think this is the appropriate behavior for this diagnostic, since is_useful relies on all match arms being well formed, and the case of a malformed match arm should probably be handled by a different diagnostic.

Looks like this might be minimal example:
    match loop {} {
        Some(_) => (),
        None => (),
    }

r=me with test added!

Hm, that's not even a type error though :thinking:

So perhaps there are three relevant cases here.

Handle the case where the match expression diverges.

enum Foo {
    A,
    B,
}

fn test() {
    match loop { } {
        Foo::A => (),
        Foo::B => (),
    }
}

Before this PR, this would have fired the missing match arms diagnostic, because the match expression type is ! and none of the match arms fit that type, so they are all filtered out, so that leaves the ! pattern uncovered. After this PR, we would suppress the diagnostic here because the match arm types don't match the type of the match expression. Perhaps a more optimal solution would be to explicitly turn off this diagnostic of the return type of the match expression is !?

Handle the case where the match expression doesn't diverge, but our current type inference infrastructure thinks it does.

enum Foo {
    A,
    B,
}

fn test() {
    match loop {
        break Foo::B
    } {
        Foo::A => (),
        Foo::B => (),
    }
}

If I hover on the loop here, rust-analyzer tells me it is type !, but I believe it should be type Foo. From the perspective of the missing match arm diagnostic, I think this case (given the match expression type is again inferred to be !) is the same as case 1.

Handle the case where the match expression type partially diverges (not sure if this is the correct wording).

fn test() -> u32 {
    match Some(1u32).map(|_| loop {}) {
        Some(val) => val,
        None => 0,
    }
}

rust-analyzer infers the type of val in Some(val) here to be u32. I think this is incorrect, but it means that with this inference the missing match arm diagnostic is not triggered. If we were to infer val as !, I suppose the missing match arm diagnostic should be updated to handle the case where the match expression type partially diverges?
Summary of the change in this PR
This PR takes the broadest approach to "solving" this, operating under the assumption that we definitely don't want any type inference issues causing false positives in this diagnostic. Given that assumption, what this PR does it suppress the missing match arms diagnostic completely if at least one match arm doesn't properly type check against the match expression. I think this solution even makes sense in the case where type inference works perfectly though, because if the users source code has a match arm that is malformed, I think we should specifically tell them that (with a MalformedMatchArm diagnostic or similar) rather than triggering a MissingMatchArms diagnostic.

I guess I'll just r? @flodiebold :)

I've added some additional test cases to demonstrate the behavior after this PR in cases 1-3 above.

bors r+


.py files

I love your work and the ideas associated with it
just curious why your files isn't a .py

I forked this project and tidy them up, https://github.com/Antioch-Tech/NightshadeDetect
If you are still interested @j3g

yeah your own readme.md states:
python nightshade_antidote.py input.jpg
so... the repo needs to have this file:
nightshade_antidote.py

The forked version is better. You can contribute to it as well!
On Mon, Apr 22, 2024, 12:09 PM robertJene @.***> wrote:

yeah your own readme.md states:
python nightshade_antidote.py input.jpg
so... the repo needs to have this file:
nightshade_antidote.py
—
Reply to this email directly, view it on GitHub
https://github.com/RichardAragon/NightshadeAntidote/issues/1#issuecomment-2070698026,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BA44S7JCECJRBJMK23UUIDLY6VN6LAVCNFSM6AAAAAA6R2NOICVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANZQGY4TQMBSGY
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

in what ways is it better? besides the filename?

What interests you about this particular repository, Robert? It is one of my least favorite ones. My brother's name is also Robert. Another one of my least favorite ones.

to prune training data.

Why are so many people interested in that? I think that is a fool's errand.
I think it would be more fruitful to explore different means of model
training beyond data.
On Mon, Apr 22, 2024 at 4:07 PM robertJene @.***> wrote:

to prune training data.
—
Reply to this email directly, view it on GitHub
https://github.com/RichardAragon/NightshadeAntidote/issues/1#issuecomment-2071098464,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BA44S7POJWTLJLFDHZRQ723Y6WJ4HAVCNFSM6AAAAAA6R2NOICVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANZRGA4TQNBWGQ
.
You are receiving this because you commented.Message ID:
@.***>

no, model training requires data.
if someone poisons their data, then it is fruitful to check so I don't use their image.
this is 100% simple.

I forked this project and tidy them up, https://github.com/Antioch-Tech/NightshadeDetect
If you are still interested @j3g
~However, please not there are still other issues after my tidy up anyway, #2 (comment)~

could you update your Repo so I can post issues there?
I just updated my downloaded copy to accept the path to an image file as an argument

It's not 100% simple because you can poison the model if it ingests the
tainted data first. I have thought through the exact use case you are
looking for. This is the best framework to use overall, I think:
https://applicablegroundedinnovations.com/AI-Data-Poison-Taster-A-Framework-for-Mitigating-Data-Poisoning-Attacks-on-Foundation-Models#wbb2
On Mon, Apr 22, 2024 at 4:31 PM robertJene @.***> wrote:

no, model training requires data.
if someone poisons their data, then it is fruitful to check so I don't use
their image.
this is 100% simple.
—
Reply to this email directly, view it on GitHub
https://github.com/RichardAragon/NightshadeAntidote/issues/1#issuecomment-2071119517,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BA44S7OAX3PPAHRYHS6JOPDY6WMVDAVCNFSM6AAAAAA6R2NOICVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANZRGEYTSNJRG4
.
You are receiving this because you commented.Message ID:
@.***>

well the plan is to "taste" the images for poison and prune the dataset ahead of time, to respect the wishes of whoever poisoned the images.

I am not familiar with all this python stuff.
I can install something and get it working if there's clear instructions.
Also, I do some programming as a hobby.
I used chat gpt's help last night to get the script to take a command line argument.
I'm planning on making a wrapper with batch and vbscript so I can batch "taste" a folder of images.
I don't know if it would make more sense to have the python script do all that.
But I spent about 4 hours on it last night and didn't have any luck getting it to output the results.

why don't you put extensions on the script files?
They can't run for people without them.
also
File "D:\AI_Stuff\StableDiffusion\Training\NightshadeAntidote\React_Native.py", line 1
import React, { useState } from 'react';
^
SyntaxError: invalid syntax


Build/Program

lgtm
in future we might want to consider a refactor where presets have arbitrary tags rather than specific object3d properties, especially if we add more categories than object3d/non-object3d.


Unable to parse PDF from directory

Everytime I try to run my code it gives an error that the directory doesnt exist:
parser = LlamaParse(api_key=os.getenv("LLAMA_CLOUD_API_KEY"),result_type="markdown")

file_extractor = {".pdf": parser}
reader = SimpleDirectoryReader('./data', file_extractor=file_extractor,recursive=True).load_data()

The error I receive: Error while parsing the file '<bytes/buffer>': file_input must be either a file path string, file bytes, or buffer object
This is my file structure

Met the same problem as well

I temporarily solved it by making the changes in the base.py file from the 'fix file_input type issue' pull request.

(The change is in line 166)

I temporarily solved it by making the changes in the base.py file from the 'fix file_input type issue' pull request.
(The change is in line 166)

It works!!! Thanks.


Replace dyn object to generic

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

66 of 66 (100.0%)  changed or added relevant lines in 12 files are covered.
22 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.05%) to 98.512%

Files with Coverage Reduction
New Missed Lines
%

core/src/executor/aggregate/mod.rs
7
83.33%

core/src/executor/evaluate/stateless.rs
15
94.09%

Totals

Change from base Build<PHONE_NUMBER>:
-0.05%

Covered Lines:
37947

Relevant Lines:
38520

💛  - Coveralls


Update Fake Data Generator to Allow Injection of Desired Values for a Column

For some clients of fake data, we want to be able to pass in lists of acceptable values for a field, and we need to inject them into the fake data generator.

@GilmoreA6 @lizzieamanning Is this similar to what we discovered in sender interviews where they were saying onboarding would be easier if the sample file showed examples of acceptable values in the required fields?

Closing due to low priority and time frame.


[extensions/common] owner

Problem solved
Short description of the bug fixed or feature added
Changes made

[ ] Public API changes: list the public API changes made if any
[ ] Internal API changes: explain the internal logic changes

How to test

[ ] Automated tests: link to unit test file
[ ] Manual tests: step by step instructions on how to test

Contributor NFT
Paste in your wallet address below and we will airdrop you a special NFT when your pull request is merged.
Address:

PR-Codex overview
This PR focuses on adding a owner function to retrieve the owner of a contract in the thirdweb extension.
Detailed summary

Added owner function in owner.ts to retrieve contract owner.
Added test for owner function in owner.test.ts.
Created a macro version of owner function in owner.macro.ts.
Updated imports and added necessary dependencies.

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

@jnsdls I assume no changesets for this stuff or? 👀

covered by #2399 I believe (owner is available via extensions/common/__generated__/IOwnable/read/owner.js


Exclude workspaces by name from build

Is your feature request related to a problem? Please describe.
Yeah, docusaurus has some packages that need to be workspaces but should not be built with the rest of the packages, but writing an explicit list of targets would be quite long.
Describe the solution you'd like
Something like yarn build --exclude packages/create-docusaurus/classic,packages/create-docusaurus/classic-typescript,packages/create-docusaurus/facebook, or, even better, a config file option.
Describe alternatives you've considered
I haven't really considered any alternatives.
Additional context
https://github.com/facebook/docusaurus/pull/6028

Yeah this is something I've been thinking about too.
Would this be per package? or just overall?

Overall.  We are just using yarn build with no arguments right now, and so having it never attempt to build the template packages would be ideal.

Yeah cool, that should be pretty quick to add.
I can probably get it done shortly.
I've been looking at something similar to allow opting out of building things like golang workspaces. But still wanting them there, so they can be built in CI.

@RDIL sorry this one took a while, but it's in there now if you're still interested :)
--exclude [path|name] supports globs, and also --exclude-current for use in a dev watching command such as yarn build --exclude-current && docusaurus-start --watch


Revert callback-alternative PRs

This reverts the following PRs:
#23361, #23430, #23435, #23562
This new EM-free implementation of callback API enabled a lot of previously excluded tests but some of those new tests are flaking and polluting the dashboards. This is a pure revert of the 4 merge commits associated with those PRs.

Interop failure is infrastructural and this revert solves many bugs.


ImmutableEnumChecker should respect Design Pattern (Singleton using Enum)

Please answer these questions before submitting your issue. Thanks!
What version of Error Prone are you using?
2.3.1 (latest)
Does this issue reproduce with the latest release?
yes
What did you do?
public enum EnumSingleton {
INSTANCE;

private final com.fasterxml.jackson.databind.ObjectMapper mapper;

private EnumSingleton() {
	this.mapper = new com.fasterxml.jackson.databind.ObjectMapper();
}

public com.fasterxml.jackson.databind.ObjectMapper getMapper() {
	return this.mapper;
}

}
What did you expect to see?
No issue, because the enum has only one entry and is following the "Design Pattern – Singleton using Enum" and com.fasterxml.jackson.databind.ObjectMapper is thread-safe.
https://dzone.com/articles/java-singletons-using-enum
http://mydevgeek.com/design-pattern-singleton-using-enum/
Joshua Bloch explained using an Enum in his Effective Java Reloaded talk at Google I/O 2008.
What did you see instead?
[javac] ...EnumSingleton.java:7: warning: [ImmutableEnumChecker] enums should be immutable: 'EnumSingleton' has field 'mapper' of type 'com.fasterxml.jackson.databind.ObjectMapper', the declaration of type 'com.fasterxml.jackson.databind.ObjectMapper' is not annotated with @com.google.errorprone.annotations.Immutable or @javax.annotation.concurrent.Immutable
[javac] 	private final com.fasterxml.jackson.databind.ObjectMapper mapper;
[javac] 	                                                          ^
[javac]     (see http://errorprone.info/bugpattern/ImmutableEnumChecker)

Hi @lgemeinhardt. I think the problem is not that ImmutableEnumChecker disrespects the enum singleton pattern - it seems to me instead that the enum has a field of type ObjectMapper, which error-prone doesn't know is thread-safe.
Now, I don't know much about Jackson, so I don't know if ObjectMapper is immutable or thread-safe. If it is immutable or thread-safe, then the workaround is to annotate the ObjectMapper field with @SuppressWarning("ImmutableEnumChecker"). Otherwise, then ImmutableEnumChe ker had found an actual immutability bug, which means the code doesn't respect the enum singleton pattern anyway. :wink:

Jackson's ObjectMapper decidedly not immutable. W.r.t. thread-safety its class documentation states:
 * Mapper instances are fully thread-safe provided that ALL configuration of the
 * instance occurs before ANY read or write calls. If configuration of a mapper instance
 * is modified after first usage, changes may or may not take effect, and configuration
 * calls themselves may fail.

Under other circumstances @jbduncan's suggestion to annotate the field with @SuppressWarning("ImmutableEnumChecker") would indeed be the way to go, but in the code you posted the ObjectMapper reference may escape through getMapper(). So as-is this class is indeed not immutable. You could fix that by returning this.mapper.copy() instead.

Thanks (@Stephan202 and @jbduncan) for your feedback.
I know that mutable singletons are may not the best choice, but I guess they are valid (if they are thread-safe).
Putting @SuppressWarning("ImmutableEnumChecker") to the code is in my opinion not a good choice, because if somebody adds another enum value/entry, then we should raise the issue "enums should be immutable" (because we then left for sure the singleton design pattern).
On the other side a enum with only one value/entry is not a real enum and should be may not in the focus of the checker?

@Stephan202
@jbduncan
In addition to the previous comment, may I should change the sample a little bit (to avoid sharing the mapper reference) and exchange the getMapper method with
public String writeValueAsString(Object value) { return this.mapper.writeValueAsString(value); }
Then the mapper is still not immutable, but the (thread-safe) singelton (using the enum pattern) makes still sense.

Hi @lgemeinhardt, sorry for taking such a long time to acknowledge your last two comments!
I agree with your last comment regarding hiding ObjectMapper, as by my understanding that would indeed make the enum thread-safe and also (from the perspective of users of the enum) stateless and thus immutable. So please go ahead! :)

@jbduncan Thanks for the feedback, but my goal is not to fix the code - I want to see if we could maybe change the check (to not report a warning/error if the enum has only one item/value). In my opinion the check should work for "normal"/real enums (with more than one item/value) but should enable the enum singleton pattern (for mutable, but theadsafe singletons).
In case of my real code - I changed to another singleton pattern to avoid the SuppressWarnings :-)

Ah, I see! In that case, I don't think I'm qualified to discuss things further as I'm not on the error-prone team myself (or a Googler for that matter).
@cushon, do you have any thoughts thus far?


fetch syntaxError unexpected identifier with babel

I have been using ES6 import syntax for a while in the project, but since I've installed node-fetch I keep getting the error below, than only goes away when I change the import to const fetch = require('node-fetch') instead.
({"Object.<anonymous>":function(module,exports,require,__dirname,__filename,global,jest){import fetch from 'node-fetch'; // const fetch = require('node-fetch')
                                                                                ^^^^^

   SyntaxError: Unexpected identifier

     at ScriptTransformer._transformAndBuildScript (node_modules/@jest/transform/build/ScriptTransformer.js:537:17)
     at ScriptTransformer.transform (node_modules/@jest/transform/build/ScriptTransformer.js:579:25)

I've had babel installed before and it worked fine with the import syntax.
},
  "devDependencies": {
    "@babel/cli": "7.7.5",
    "@babel/core": "7.7.5",
    "@babel/plugin-proposal-object-rest-spread": "7.0.0",
    "@babel/plugin-transform-runtime": "7.7.6",
    "@babel/preset-env": "7.7.6",
    "babel-eslint": "10.0.3",
    "babel-loader": "8.0.6",
    "eslint": "6.7.2",
    "eslint-plugin-import": "2.19.1",
    "eslint-watch": "6.0.1",
    "jest": "^24.9.0",
    "serverless": "1.59.3",
    "serverless-offline": "5.12.1",
    "serverless-webpack": "5.3.1",
    "source-map-support": "0.5.16",
    "webpack": "4.41.2",
    "webpack-node-externals": "1.7.2"
  },
  "babel": {
    "presets": [
      [
        "@babel/preset-env",
        {
          "targets": {
            "node": "10"
          }
        }
      ]
    ]
  },
...

I cannot commit with this error in tests.

Webpack isn't transpiling import syntax to require syntax.

So how to overcome this error? Why can't I just use 'import'?


Add a few unique programs into the C++ folder

@prahuljose please provide details about programs you want to contribute.

@prahuljose please provide details about programs you want to contribute.

I plan to add:

Matrix Multiplication
Swapping 2 numbers w/o an extra variable
Reverse a given number (including leading '0' digit)

@prahuljose Ok, you are good to go.


Update Azure CCM, AzureDisk CSI and AzureFile CSI

What type of PR is this?
/kind feature
What this PR does / why we need it:

Update Azure CCM to v1.24.0, v1.23.11, v1.1.14 (for Kubernetes 1.22), v1.0.18 (for Kubernetes 1.21), v0.7.21 (for Kubernetes 1.20)
Update AzureDisk CSI driver to v1.18.0
Update AzureFile CSI driver to v1.18.0

Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
xref #2011
Does this PR introduce a user-facing change?:
Update Azure CCM to v1.24.0, v1.23.11, v1.1.14 (for Kubernetes 1.22), v1.0.18 (for Kubernetes 1.21), v0.7.21 (for Kubernetes 1.20)
Update AzureDisk CSI driver to v1.18.0
Update AzureFile CSI driver to v1.18.0

/assign @kron4eg

/approve
/lgtm

/retest


[Shadowrun5thA] allow import of critter powers without range and dura…

…tion
Changes / Comments
Roll20 Requests
Comments are very helpful for reviewing the code changes. Please answer the relevant questions below in your comment.

[x] Does the pull request title have the sheet name(s)? Include each sheet name.
[x] Is this a bug fix?
[ ] Does this add functional enhancements (new features or extending existing features) ?
[ ] Does this add or change functional aesthetics (such as layout or color scheme) ?
[ ] If changing or removing attributes, what steps have you taken, if any, to preserve player data ?
[ ] If this is a new sheet, did you follow Building Character Sheets standards ?

If you do not know English. Please leave a comment in your native language.

Character Sheet Info Roll20 Internal Use only.


Declarative custom elements for Cesium

Overview
Use custom element web components to allow cesium to be used declaratively in HTML.
Currently Cesium is an imperative library that requires writing Javascript code with step by step instructions to set up the view.    Many newer UI frameworks (like React, Vue, Svlete) are declarative.  They allow the developer to declare what they want to display.
Example of current imperative Javascript code
const viewer = new Cesium.Viewer("cesiumContainer", {
  animation: true,
  homeButton: true,
  navigationHelpButton: false,
  shadows: true,
  shouldAnimate: true,
});

viewer.scene.primitives.add(new Cesium.Cesium3DTileset({
  url:  "../SampleData/Cesium3DTiles/Tilesets/Tileset/tileset.json"
}));

viewer.entities.add({
    position: position,
    orientation: orientation,
    model: {
      uri: "../SampleData/models/CesiumAir/Cesium_Air.glb",
      minimumPixelSize: 128,
      maximumScale: 20000,
    },
  });

Example of what a declarative custom element interface could look like
<cesium-view shadows animate>
  <cesium-animation-controls></cesium-animation-controls>
  <cesium-navigation-controls hide-help></cesium-navigation-controls>
  <cesium-tileset src= "../SampleData/Cesium3DTiles/Tilesets/Tileset/tileset.json"></cesium-tileset>.
  <cesium-model
    src="../SampleData/models/CesiumAir/Cesium_Air.glb"
    minium-pixel-size="128"
    maximum-scale="20000"
  ></cesium-model>
</cesium-view>

Other custom element functionality
Using custom elements would also allow functionality like:

automatically resizing the canvas to the containing element using ResizeObserver.  This way it just works if the layout changes.
Exposing events as native DOM events eg.  document.querySelector('cesium-view').addEventListener('selected-entity-changed', ...). Makes it easier to learn for developers already familiar with other HTML DOM events.

First steps
I think the first steps would be to create a proof of concept cesium-viewer element that mirrors the exposes Viewer interface today.  It would not support any sub-elements like the example above (there would still need to be some imperative configuration).
This could be used as a building block to start adding more elements like cesium-tileset.
Long term vision
Longer term I would like to see the viewer element be more modular and extensible for widgets.  It provides a generic set of hosting capability for widgets (such as accessing the scene, maybe some basic positioning for widgets). Instead of the current system where the viewer knows about all possible widgets and they all get configured via the viewer object.    That puts cesium created widgets on the same level as user created widgets.  The cesium created widgets can serve as examples for how to create your own custom widgets.

Probably worth taking a look at Resium if you haven't seen it already.

Probably worth taking a look at Resium if you haven't seen it already.

Thanks for pointing that out. Yes this would be similar to Resium, however it would use custom elements instead of React.  The advantage of custom elements is that they are not tied to any particular framework.  So they could be used in React, Angular, Vue, or with plain HTML+JS.

Maybe we look at <model-viewer>, a web component for simple display of glTFs, for inspiration.

Hi @angrycat9000
(1) Let me ask for some clarifications:
"Use custom element web components to allow cesium to be used declaratively in HTML."
I would interpret this to mean 'use Lit elements', since these appear to be the most advanced form of web components. Is this correct, or would you also consider alternative web component implementations?
Will the proposed declarative API be aimed at the current community of Cesium users, or should it also facilitate the access to Cesium to a yet wider public? How should this impact and inform the new API development?
IMO, the users of the new API should rely on the corresponding API documentation (to be developed), rather than having to dig into the existing Cesium documentation. Would you agree?
Your sketch of the declarative API suggest a progressively increasing degree of modularity. Are there plans to modularize Cesium (break up into smaller subsystems)? How would this impact the new API?
May I suggest that you should draft a guidance document that would define objectives aimed at and trace the directions to follow towards the new declarative API?
(2) Apropos the "First steps" and "proof of concept cesium-viewer element that mirrors the exposed Viewer interface":
How would you define the scope of the Viewer interface? I suppose that it goes beyond the formal list of properties and methods of the Cesium.Viewer object. Is this correct?
Lit elements expose a set of attributes whose values can

define the element configuration and appearance when an instance is created
trigger the rendering of the element when these values change

The first group could map directly to the properties of the underlying object (the Viewer), while the second group might map to some method calls on the object or on its child objects. Some attributes could serve both purposes.
I think that you could greatly advance the prototyping work if you could draft a list of  <cesium-viewer> attributes that you want to see implemented in the early development rounds.
(3) My prototypes
I recently started experimenting with Cesium-in-Lit, and I hit and reported a problem (cesium/issues/10907). In her responses, @ggetz pointed me to this thread.
Meanwhile, helped  by my js-mentor @olange I implemented a workaround, and I deployed my  prototype on netlify. I just updated it, based on my thinking since I first saw this discussion.
My questions and suggestions above are a byproduct of my prototyping so far. I would greatly appreciate if you could respond with comments, guidance and possibly with a challenge to implement a more substantial prototype.
@rudifa

Thanks for your feedback @rudifa

I would interpret this to mean 'use Lit elements', since these appear to be the most advanced form of web components. Is this correct, or would you also consider alternative web component implementations?

It could be Lit since there is experience using that framework at Cesium using it.   However, it might also make sense to just write it as a vanilla custom element with no framework if most of API would be focused on changing properties of the Cesium viewer object and not writing HTML.

Will the proposed declarative API be aimed at the current community of Cesium users, or should it also facilitate the access to Cesium to a yet wider public? How should this impact and inform the new API development?

It would probably start with the existing names and structures and see if there are areas that could be simplified for declarative rendering.  That would avoid having two disjointed APIs but also allowing some freedom to make it easier to write declaratively.
If there are areas you think could be improved on the existing API please let us know.

How would you define the scope of the Viewer interface? I suppose that it goes beyond the formal list of properties and methods of the Cesium.Viewer object. Is this correct?

The first pass would probably be focused on the commonly used properties and might leave out some of the less frequently used properties.  I would not expect it to be adding anything that isn't already available.
It would also extend beyond just the Viewer object itself.  A large part of this will be declaring the data that is shown in the viewer.  That brings in things like tilesets and models. It will probably be a whole set of elements needed to represent this complexity, not just a cesium-viewer element.

I recently started experimenting with Cesium-in-Lit, and I hit and reported a problem

Thank you for reporting that issue.  That is great to see others in the community interested in having a declaratively rendered cesium viewer.

Hi  @angrycat9000
Thank you for the comments and clarifications.

it might also make sense to just write it as a vanilla custom element with no framework

I think that LitElement places an useful layer of abstractions over the HTMLElement, hiding the drudgery and boilerplate code needed to use the HTMLElement API directly, see for example the codelab  From Web Component to Lit Element.


Avoid allocating a closure in CopyToAsync

The existing code passes less parameters, but a small closure is allocated on every call, so perhaps it's worth to be explicit.

Good catch!
But that doesn't mean it can't be a local function.

Good catch!
But that doesn't mean it can't be a local function.

It is a local function.

But that doesn't mean it can't be a local function.

It is a local function.

I see it now. But why did you change its scope. It needs only to be scoped to that particular else.

For separation. But I think I'll make it a member function to make things even more explicit.

For separation. But I think I'll make it a member function to make things even more explicit.

When you increase the scope, you're not separating. Making it a full member function is even worst.
That method is used only in that else and it should be scoped to that else only. That makes it "private" to that block. The scope of code artifacts should be the smallest possible to avoid unforeseen uses and side effects from modification.

I see your poin,t but my vote is for a member function. It seems the right compromise here.

I see your point, but my vote is for a member function. It seems the right compromise here.

Why?

https://github.com/microsoft/Microsoft.IO.RecyclableMemoryStream/pull/202#discussion_r710931819

The local function has no compromises. Any compromise would be the wrong compromise.

What's wrong with just passing those two additional parameters into the local function? That's a smaller change, as @paulomorgado, I don't see the downside here.

It's easy to capture smth again, it's what led to the allocation in the first place. Having some separation from the calling code makes it explicit that we don't want to capture anything.

It's easy to capture smth again, it's what led to the allocation in the first place. Having some separation from the calling code makes it explicit that we don't want to capture anything.

But you can fix that by making the local function static. You just need to pass either this or blocks and blockAndOffset as parameters.

Yes, you can make any member function static by explicit;y passing this, but I'd rather not write that code myself :)

Too many parameters and some will end up being passed by stack instead of registers.
I agree that it can happen again.
I wonder if ClrHeapAllocationAnalyzer catches this.

The codegen differences between a static local function and a method are minimal. The static function has a couple of more instruction of stack shuffling, but it's associated state machine is ~36 bytes smaller.
https://sharplab.io/#gist:e0be6566b653dba8f1da54de00d329a4

@grbell-ms Are you recommending one particular way or the other? Is it just kind of arbitrary? It would be nice to come to a good agreement or compromise on this PR.

@benmwatson  I'm coming down in favor of the static local function.

It prevents us from making this mistake in the future
There's no real perf difference
We don't pollute the class with methods with only a single reference

@benmwatson Ben Watson FTE I'm coming down in favor of the static local function.

It prevents us from making this mistake in the future
There's no real perf difference
We don't pollute the class with methods with only a single reference

That's also my reasoning.
It requires C# 8.0 or above, but we are already using C# 9.0.
We need to make sure we are running all tests for code targeting .NET Framework.

@lbargaoanu I see you closed this. Are you going to re-open and do the other way? If not, @paulomorgado or @grbell-ms, would one of you be willing to redo it in the recommended way. I'd love to get this optimization into the next release.

I'd rather not write that code myself. I just don't see the value :)


[Event Request] Table 5741 "Transfer Line" - "Transfer-to Code", CreateDim

Please add the following Events "Transfer-to Code", CreateDim in Table 5741 "Transfer Line"
field(37; "Transfer-to Code"; Code[10])
        {
           <...>
            begin
                //>> Begin
                IsHandled := false;
                OnBeforeTransfertoCodeValidate(Rec, xRec, CurrFieldNo, StatusCheckSuspended, IsHandled);
                if IsHandled then
                    exit;
                //<< End

procedure CreateDim(Type1: Integer; No1: Code[20])
<...>
        "Dimension Set ID" :=
          DimMgt.GetRecDefaultDimID(
            Rec, CurrFieldNo, TableID, No, SourceCodeSetup.Transfer,
            "Shortcut Dimension 1 Code", "Shortcut Dimension 2 Code", TransHeader."Dimension Set ID", DATABASE::Item);
        //>> Begin
        OnAfterCreateDim2(Rec,CurrFieldNo,TableID,No,TransHeader);
        //<< End
    end;

    [IntegrationEvent(false, false)]
    local procedure OnBeforeTransfertoCodeValidate(var TransferLine: Record "Transfer Line"; xTransferLine: Record "Transfer Line"; CurrFieldNo: Integer; StatusCheckSuspended: Boolean; var IsHandled: Boolean)
    begin
    end;

    [IntegrationEvent(false, false)]
    local procedure OnAfterCreateDim2(var TransferLine: Record "Transfer Line"; CurrFieldNo: Integer; TableID: array[10] of Integer; No: array[10] of Code[20]; TransHeader: Record "Transfer Header")
    begin
    end;

Hello @AdrijaSipaviciute
Related to your request for OnAfterCreateDim2 event, existing one:
OnAfterCreateDim(Rec, DefaultDimSource, xRec, CurrFieldNo); should be enough (procedure CreateDim is global so, probably TransferHeader will not be available every time you subscribe on this publisher)

Hello @AleksandricMarko
I agree regarding OnAfterCreateDim2 - we will use OnAfterCreateDim.

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 12007.


Codestral template prevents using it for FIM

What is the issue?
Hi 👋 . The current template of codestral:latest https://ollama.com/library/codestral:latest/blobs/36ee4ce5634b makes it harder to use the model for Fill in the Middle (FIM).
Codestral for FIM requires the following template:
<s>[SUFFIX] {{ suffix }} [PREFIX] {{ prefix }}

While its Instruct mode requires the template to be:
[INST] {{ .Prompt }} [/INST] {{ .Response }}

If a ollama user would like to use codestral for FIM, he/she will have to set raw=True (docs) in order to ignore the default template.
The main issue I'm facing is that I'm using ollama with the litellm openai compatible endpoints, and in this setting, the raw parameter is not supported. Thus I can't ignore the default template of codestral which results in a mix of Instruct/FIM model responses, since the FIM user template will always be wrapped with the Instruct template:
[INST] <s>[SUFFIX] {{ suffix }} [PREFIX] {{ prefix }} [/INST]

Proposal
Is it possible to change the default template to just be {{ .Prompt }} like many other models?
Examples:
bruno@mac ~> ollama show mistral --template
{{ .Prompt }}

bruno@mac ~> ollama show mixtral --template
{{ .Prompt }}

bruno@mac ~> ollama show codegemma:2b --template
{{ .Prompt }}

I'm aware I can modify or create a new Modefile for codestral, but I think it is wrong to have the default template to be the Instruct template for a model that supports both modes.
OS
No response
GPU
No response
CPU
No response
Ollama version
No response

I would like to use FIM with codestral as well. How would one edit the model file to do this?

FIM is implemented in #5207 and will be, I'm hoping, part of the next release. This will allow users to call the API with an additional suffix field in addition to the existing prompt field in /api/generate to pass to the LLM the prefix and suffix. Existing models will require an template update in order to leverage the new feature.

Is it possible to change the default template to just be {{ .Prompt }} like many other models?

Using the API, it's easier to set template="{{ .Prompt }}" for FIM than it is for chat. Since codestral is dual chat and FIM model so the current template is more appropriate than {{ .Prompt }} since many user's interaction with the model isn't through the API but through a chat interface such as the CLI


mqtt integration don't work after upgrading to 0.100.2

Last working Home Assistant release (if known): 0.99.3
**Operating environment (Hass.io/Docker/Windows/etc.):**HASSIO
<!--

After upgrading to 0.100.2 no mqtt connected entity will show up in Hassio; broker connect to HA and ALL devices are connected, with no errors in MQTT and HA logs,  NO entity show up, nor in entity registry nor in mqtt integration; I see all the Tasmota devices are connected in log; revert back to 0.99.3 all works fine...

I've wrote a whole page but did't see anymore...


⚠ Itaca Gallery has degraded performance

In 28fb703, Itaca Gallery ($SITE_TCGLR) experienced degraded performance:

HTTP code: 200
Response time: 11384 ms

Resolved: Itaca Gallery performance has improved in 218e914 after 1 hour, 26 minutes.


Gradle sync fails all the time

I'm running sh ci.sh and the project is built successfully.
However, Android Studio keeps showing me the following error message:

Gradle project sync failed. Basic functionality (e.g. editing, debugging) will not work properly

Because whenever I try to sync Gradle project I'm getting the following error:

Error:(4, 0) Cause: hu/supercluster/paperwork/plugin/PaperworkPlugin : Unsupported major.minor version 52.0

I suppose that this may be caused by the incompatibility of bytecode generated by different java compilers. The JAVA_HOME on my system points to the Java 7 installation, the javac -version output is the

javac 1.7.0_80

...and the java -version gives me

java version "1.7.0_80"
Java(TM) SE Runtime Environment (build 1.7.0_80-b15)
Java HotSpot(TM) 64-Bit Server VM (build 24.80-b11, mixed mode)

Where does this error may come from?

I guess you need JDK 8

Let's ask @zsoltk if it's possible to recompile Paperwork with target and compile bytecode level JDK 6 and upload a new version

I forgot to mention that I was using JDK 8 before: JAVA_HOME was pointing to Java 8 installation and both java -version and javac -version were showing that I use Java 8. I thought that PaperworkPlugin is incompatible with Java 8, switched to Java 7 - the issue persists.

Well, the sh ci.sh runs fine and it invokes more or less the same gradle tasks that I try to run in the Android Studio Gradle tab. I suppose Android Studio somehow uses the wrong JRE but can not figure out which setting I have to change.

Yup, Android Studio 2 uses own bundled JDK, you can find its version in
"About" window. Though, it works fine on mine machines with Mac OS and
Android Studio 2.
On Tue, 16 Feb 2016, 11:12 Artem Gapchenko<EMAIL_ADDRESS>wrote:

Well, the sh ci.sh runs fine and it invokes more or less the same gradle
tasks that I try to run in the Android Studio Gradle tab. I suppose Android
Studio somehow uses the wrong JRE but can not figure out which setting I
have to change.
—
Reply to this email directly or view it on GitHub
https://github.com/artem-zinnatullin/qualitymatters/issues/93#issuecomment-184749510
.

--
@artem_zin

Okay, now build works fine and I cannot break it again.
What I did (possibly one of those actions fixed my build):

changed the javac version to 1.7
changed the java version to 1.7
changed the content of JAVA_HOME so it points to Java 7
removed everything in build subfolders.

Ah, yeah, let's anyway wait for feedback from @zsoltk :)

It has already been recompiled with 1.6, just update paperwork dependency to 1.2.4 :)


Issue with Obtaining Archetype and Reflection Information from std::vector Field in CLASS A

Hello everyone,
compilation backend:  MSVC （Visual Studio 2022 community）
I'm currently encountering a challenge with our project, specifically related to reflection and archetype retrieval from a field within a class. The class in question, class CLASS() A, has a field named b, which is marked using FIELD() and is of type std::vector.
Despite the field being successfully marked, I'm unable to retrieve the archetype of field b through class A's foreachfield method. Furthermore, obtaining any reflection information about this field has also proven to be impossible.
Here is a simplified overview of the issue:
Class Definition:
class A has a field b, defined as std::vector and marked with FIELD().
Problem Encountered:
Unable to access the archetype of field b using foreachfield in archetype A.
No reflection information about field b can be retrieved.
I suspect the issue might be related to how std::vector fields are treated in the context of our reflection system, but I'm unsure about the exact cause or how to resolve it.
Request for Assistance:
I'm looking for guidance or solutions from anyone who has faced similar challenges or has expertise in C++ reflection, especially in handling standard container types like std::vector. Any insights, code snippets, or references to documentation would be greatly appreciated.
Thank you in advance for your help!

thanks, I will try to add the reflection layer


jQuery-viewport-checker not working with latest version of Chrome, Version 63.0.3239.132 (Official Build) (64-bit)

Hi,
I noticed that jQuery-viewport-checker doesn't work with the latest version of Chrome, Version 63.0.3239.132 (Official Build) (64-bit). Kindly help rectify.
Thanks.

I have no issues running the plugin's demo page in Chrome 63. Just ran a few tests in different browser versions and they all seem to work fine.
Please provide me with more information (like console errors) if this problem persists.


Hello sir need help? how to open bootstrap modal when reaching specific pdf page?

Hi
You can perform some actions when reaching a specific page by using the "changed" parameter. For example:
$(function() {
$("#myPDF").pdf( {
source: "demo.pdf",
tabs: [
{title: "Section 1", page: 2, color: "orange"},
{title: "Section 2", page: 3, color: "green"},
{title: "Section 3", page: 5, color: "blue"},
],
changed: function () {
if ( $("#myPDF").pdf("getPageNumber") == 5) {
// do something when page 5 is reached
alert ("You have reached section 3");
}
}
} );
});
Is it what you need?
Cheers
Loic

Thank you so much!! Its works for me.
but i have one another problem using Next and Previous button click bootstrap modal will be appear that's fine.but instead of that we can do same event in the PDF scrolling.
means at the time page loading all pages of pdf is load and scroll will appear in the right side.by scrolling i am going to second page that time bootstrap modal will be appear automatically.
hope you understand the scenario.if this possible please asap me solution for that.
Thanks in advance!!

Hi!
If I understand correctly, you want the user to be able to scroll to the next page, right? eventually showing your modal at a specific page.
Unfortunately scrolling is not possible with TouchPDF. It was designed with touch devices in mind, where swapping makes more sens than scrolling.
Let me know if I didn't get it.
Loic

Thanks for your reply.
On Mon, Sep 14, 2015 at 3:33 PM, loicminghetti<EMAIL_ADDRESS>wrote:

Hi!
If I understand correctly, you want the user to be able to scroll to the
next page, right? eventually showing your modal at a specific page.
Unfortunately scrolling is not possible with TouchPDF. It was designed
with touch devices in mind, where swapping makes more sens than scrolling.
Let me know if I didn't get it.
Loic
—
Reply to this email directly or view it on GitHub
https://github.com/loicminghetti/touchpdf/issues/1#issuecomment-140224625
.


Ability to set clustering to none as first option

I am working now with a 16 Million Rows database, and i lose 50 seconds each time i load the clustering dialog because is trying to compute the binning,fingerpring for 800.000 different objects that my database have. , even if i already did this, and now i want to do binning,n-gram-fingerprint or another method.
17:48:43.159 [ compute-clusters_command] computed clusters [binning,fingerprint] in 47728ms (45453ms)
17:49:33.383 [ compute-clusters_command] computed clusters [binning,ngram-fingerprint] in 95677ms (50224ms)
After the first time, when i do the binning fingerprint, and i want to do another clustering, lets say metaphone3, its a period of around 130 seconds, the different from when is computing the binning,fingerprint (50 seconds ) and metaphone3 ( 180 seconds ) when it`s says :
No clusters were found with the selected method
Try selecting another method above or changing its parameters
Because it had computed for the first option, and the second option is still waiting, and, if the user is not aware, he may think that even the second or third option does not compute any clusters

The misleading message sounds like a bug to me, so I've created a separate issue for it. #976

Closing since issue resolved in #976


Fix haproxy stats

stats scope . ends up blocking all backends
Renaming endpoint from _stats to haproxy_stats so that it doesn't collide with CouchDB's _stats endpoint
Added a commented out stats auth line so that you can easily password protect

Testing recommendations
Visit http://example.com:5984/_stats now and you won't see any stats for any backends. With my changes you will see the backend specified when you visit http://example.com:5984/haproxy_stats

Good call. It's a nit but maybe _haproxy_stats is better so it's guaranteed not to clash with any CouchDB database name?

Changed to _haproxy_stats as per @kocolosk's recommendation


Show active state on tabs

This PR adds active state to routed and unrouted tabs.
Before:

After:

🙌  Good one! 💜  reformatted bits!

✨ Awesome


How to save text without losing formatting? Possibly enhancement.

Hi hope all is well. Great work that i'm benefiting from so thanks very much.
With .SetCellValue(), .SetCellStr(), and .SetCellInt(), they make decisions about the alignment of the result, which is not bad per se.  I know i can use .SetCellStyle() afterwards. But it would be very handy to have a function that respects the existing style.  I wish to be able to set all the styles i need within an Excel file as a kind of report template, outside of Go, then use Go to populate it, and not even need to know what the styles are within my Go code.  For example a column that is already center-aligned, would be great if a function that can keep it center-aligned, or whatever the style might be, including font, color, date formatting, currency etc.
Something like a .SetCellValueWithStyleRetention().  Or is there already a way to achieve this with the existing functions?

Thanks for your issue. I have tested that function SetCellValue(), SetCellStr(), SetCellInt(), and they won't affect the style (font family, color, underline, bold, and align) of cell, but may lose part of date formatting, currency settings of the cell. Please provide file attachments If you can.

Thanks for the quick response.  Attached are Book1.xlsx and Book2.xlsx.
I created Book1.xlsx outside of Go, then made a copy of it, which is Book2.xlsx.
Then i run the Go code (so to speak) on Book2.xlsx.
You are correct when the cells have existing content, but when the cells do not have existing content, that's when the functions appear to make those decisions.
Book1.xlsx
Book2.xlsx
test_excel.go.txt

I'm using Excel 2010 in case that might be relevant.

@gitdlam  I have fix it by adding column style predefined detection set cell functions.

Thanks so much @Luxurioust you're the best!!  :+1:


Add Site Extension to patch

Microsoft.AspNetCore.AzureAppServicesIntegration and Microsoft.AspNetCore.AzureAppServices.HostingStartup are where the Logging reference comes from so we need to patch those too.

LGTM if it works

We're treating this "catch-up" changes as tell-mode (cc @Eilon) so please notify netcoreship about the change and that this is in regard to "re-shipping" the previous patches that were not properly shipped.

@aspnet/build The siteextension build artifact only has microsoft.aspnetcore.azureappservices.hostingstartup as version 2.25-servicing-* and the other bits are 2.2.0. Also, it only contains 2.2.x64 and not 2.1 and 2.2.x86.

@aspnet/build
Could I get assistance for

The siteextension build artifact only has microsoft.aspnetcore.azureappservices.hostingstartup as version 2.2.5-servicing-* and the other bits are 2.2.0.

The other two packages that are bundled as 2.2.0 are microsoft.aspnetcore.azureappservicesintegration and microsoft.extensions.logging.azureappservices. But they should also be 2.2.5-*

The other two packages that are bundled as 2.2.0

Bundled in what? The offline package cache?

Or does the site extension re-distribute these assemblies somehow? If so, it's likely a problem with way the site extensions is getting versions during the build. @pakrym would know that dependency flow works.

@BrennanConroy @natemcmaster @anurse are there still open questions on this PR? Or, are we just waiting for the validation build to complete?

I'm waiting for the build now. I think everything has been figured out.

@BrennanConroy something else went in. You need to resolve the conflicts in this PR.

Yeah, but I'd like the build to go through first so I can verify the artifacts.

Confirmed that servicing versions appear in 2.2 site extension.
Merging in ~1 hr 40 minutes when build passes.

... another conflict...

@aspnet/build Please merge

Done 😺


Notifications are cropping by the first one

Hi! If I set hook to "TR" value, very long notifications are cropping by the length of the first one.

Hey, thanks for the issue!
What value are you using for notification_hook inside your NotificationBlock (doc here?
If you're using notification_hook: Hook(parent_anchor: BL, self_anchor: TL) like in the default config, then this behaviour is expected.  Try using notification_hook: Hook(parent_anchor: BR, self_anchor: TR) instead.
I'm not at my computer for a while so if that doesn't work or if you're already doing that, then I'll look into it when I get back :).

Hey, thanks for the issue!
What value are you using for notification_hook inside your NotificationBlock (doc here)?
If you're using notification_hook: Hook(parent_anchor: BL, self_anchor: TL) like in the default config, then this behaviour is expected. Try using notification_hook: Hook(parent_anchor: BR, self_anchor: TR) instead.
I'm not at my computer for a while so if that doesn't work or if you're already doing that, then I'll look into it when I get back :).

That helped! Thanks!


Modeling Bug: Example Submission cannot be changed after the first assesment

Describe the bug

After having assesed an example submission of a modeling excercise, you cannot change the submission itself anylonger.  You can update the assesment, but you can't change the underlying submission. If you remove all of the assesments (atleast visually), you still cannot change the underlying part.
To Reproduce

Go to 'Course Management'
Click on 'Create Modeling Excercise'
Create an Modeling Excercise
Create an Example submission for the modeling excercise
Assess this submission and save the assesment
Remove the assesed parts and save the assesment again
Try to change the underlying submission
You will get following message displayed, but the assesment does not save:

circular structure to JSON
--> starting at object with constructor 'Object'
|     property 'results' -> object with constructor 'Array'
|     index 0 -> object with constructor 'Object'
--- property 'submission' closes the circle
Expected behavior

You should be able to change the underlying example submission
Screenshots

Environment

 - OS: [Windows]
 - Browser [Chrome]
 - Version [89.0.4389.114]

Additional context

I tried the same procedure with text excercises, but there you can update the underlying submission, even when there is a assesment.
Output in Web Console

  [No Output in the Web Console]

@iozbeyli please have a look into this issue. Thanks


How to link schemas?

I need to generate sample data for a normalized data structure, and I'm not sure how to do it efficiently in Mimesis. Would appreciate pointers in the right direction for me. Thanks.
Let's say I have

an entity called Person, with a primary key of id.
an entity called Email Account, with a primary key of email-address.
an entity called Email, with a primary key of message-id.

Primary key/Foreign key links
A Person can have 0 or more Email Accounts.
An Email Account can have 0 or more Emails.
At the end of the day, I would like to dump all the test data for the 3 entities into 3 DB tables, or 3 CSV files, that can be reconstructed in the future, using the links between the 3 entities.

I'm afraid that this is not the task that Schema solves. You better to use common providers for your case. Problems you described must be solved on the ORM level if I understand you correctly.

@sobolevn @ceccoemi Any additions?

You can use https://github.com/mimesis-lab/mimesis-factory

I'm closing this issue, but if you have more questions feel free to reopen it.


Add new method for getting optional value

Add new method std::optional<T> getOptional<T>(const std::string& key) for getting optional values.
If key does not exist or is null it should return std::nullopt, otherwise value with type T it is castable to that.

I could pick this. Can you assign it to me?

Sure :)


MB-1746 circleci job for loki

Description
The goal of this story is to create a CircleCI job to run the Loki tests against storybook and fail the build if there are any regressions.
Reviewer Notes
This PR is based on #3689 and will require that PR to be merged in before this one can be reviewed.
This PR introduces a couple of new make commands if you could ensure they run locally that would be helpful. Also, there is an update to the relevant doc files so please ensure the docs are clear and note any areas of confusion.
Setup
Following are the new make tasks to try:

make storybook_docker will start storybook inside a docker container. Once it is running you can go to http://localhost:6006 to verify it is up
make storybook_tests will run the loki tests in a container. This is the command the CircleCI job uses

Following are docs updated. If you have grip installed you can preview the changes in your browser via the command below. Then browsing the following urls
grip

http://localhost:6419/docs/frontend.md
http://localhost:6419/docs/how-to/run-loki-tests-against-storybook.md
http://localhost:6419/docs/how-to/run-storybook.md
http://localhost:6419/scripts/README.md

Code Review Verification Steps

[ ] Request review from a member of a different team.
[ ] Have the Jira acceptance criteria been met for this change?

References

MB-1746 for this change
MB-60
PR 3689

I inadvertently marked this for review.  If this causes confusion, sorry gidjin!


Exterior and interior images keep getting broken

Describe the bug
Hi
First: tnx for this GREAT addon!
1 question: most of the time my exterior and interior images get broken:

Sometimes refreshing the browser works, but not always.
If I try to browse to the images URL, I receive a "500: Internal Server Error"
Any hint/idea why this happens please?

Hi,
Same here. Started 2 days ago...

I would like to help you, unfortunately this is an API issue. If the images can't be reached on the Volvo servers, the images seem to be broken. Maybe I will take a look and download the files at the first addon start.
Is this issue already there ?

I would like to help you, unfortunately this is an API issue. If the images can't be reached on the Volvo servers, the images seem to be broken. Maybe I will take a look and download the files at the first addon start.
Is this issue already there ?

Tnx for looking into this.
Both after reloading the addon and even after a full HAOS restart, I can't get it to load the images anymore. It would be nice if I could download them somewhere and just put it offline available.

I will check if the images are available at all...

Please try version 1.9.6. Volvo has severely restricted access to the images. It is very difficult to get the pictures back...

Fixed! I'll try to save a copy offline, tnx!

The addon saves a copy ;) So the images are only downloaded if you update or recreate the addon.


Add references to .NET Standard 2.0

Do this anywhere that Net Standard 1.6 is mentioned.

Done a while back.


Bootstrap triggers virus total on signed executable

I have a signed executable made with nuitka and I uploaded it to virus total, the bootstrap seems to get detected. Can the bootstrap be compiled manually or something
nuitka from pip
version info:
2.0.4
Commercial: None
Python: 3.10.7 (tags/v3.10.7:6cc6b13, Sep  5 2022, 14:08:36) [MSC v.1933 64 bit (AMD64)]
Flavor: CPython Official
Executable: C:\Users\Lenovo\AppData\Local\Programs\Python\Python310\python.exe
OS: Windows
Arch: x86_64
WindowsRelease: 10
Version C compiler: C:\ProgramData\mingw64\mingw64\bin\gcc.exe (gcc 13.2.0).

https://www.virustotal.com/gui/file/6eb1d49c8a9d0196e6f4810a63c09b8ef65e56b83aca7d571eb5566d15850634
I've also contacted the anti-virus provider for false positive

One is pretty good, but as described in the docs, the steps necessary to reduce it further are available for Nuitka commercial subscribers only.


Setup Jenkins jobs

This includes jobs to build the project and run unit tests for the develop branch and pull requests.

The develop and PR builder jobs have been set up.


adding url to source for all web scrapers

this allows a URL to appear in the document store listing when using web scrapers.
added puppeteer and playwright
fixes https://github.com/FlowiseAI/Flowise/issues/2520

fixes https://github.com/FlowiseAI/Flowise/issues/2520

thanks @jaredmcqueen !


flatten out CaseObjectContext#aggregated and #found into the enclosing object, remove redundant #count field

https://github.com/lihaoyi/upickle/issues/258
aggregated is flattened out into individual fields. This removes the need for both the array allocation, as well as boxing primitive types into AnyRefs to place them into the array (the values are still boxed when passed around on the stack, but those boxes are easier to elide than boxes in a long-ish lived array)
found is converted into a simple bitset. Hopefully nobody has case classes with more than 64 fields!
count is redundant, since the found bitset can be used to serve the same purpose
Seems to be a consistent 5-10% speedup on the JVM, 20-30% speedup on JS

RUN JVM: 5000
BEFORE
AFTER

upickleDefault Read
584232
632073

upickleDefault Write
666081
715345

upickleLegacy Read
442174
472077

upickleLegacy Write
490570
515168

upickleBinaryDefault Read
907664
996910

upickleBinaryDefault Write
1056606
1119090

upickleBinaryLegacy Read
607304
647197

upickleBinaryLegacy Write
685262
711517

upickleDefaultCached Read
730160
784096

upickleDefaultCached Write
858471
948827

upickleLegacyCached Read
769819
839172

upickleLegacyCached Write
886125
895340

upickleDefaultBinaryCached Read
1313489
1492671

upickleDefaultBinaryCached Write
1609941
1723321

upickleLegacyBinaryCached Read
1409764
1591342

upickleLegacyBinaryCached Write
1694022
1788002

RUN JS: 5000
BEFORE
AFTER

upickleDefault Read
186702
213663

upickleDefault Write
367346
368026

upickleLegacy Read
133466
202607

upickleLegacy Write
159632
324439

upickleBinaryDefault Read
115097
185866

upickleBinaryDefault Write
102963
130430

upickleBinaryLegacy Read
138546
192640

upickleBinaryLegacy Write
117989
131722

upickleWebDefault Read
165987
209690

upickleWebDefault Write
272457
316333

upickleWebLegacy Read
169189
206341

upickleWebLegacy Write
257847
295743

upickleDefaultCached Read
212467
254910

upickleDefaultCached Write
351903
451659

upickleLegacyCached Read
184531
259669

upickleLegacyCached Write
414168
468164

upickleDefaultBinaryCached Read
187879
222672

upickleDefaultBinaryCached Write
132631
140534

upickleLegacyBinaryCached Read
197706
247491

upickleLegacyBinaryCached Write
136719
151409

upickleWebDefaultCached Read
212396
282512

upickleWebDefaultCached Write
380941
469832

upickleWebLegacyCached Read
230500
272736

upickleWebLegacyCached Write
363212
436217

This change is binary incompatible

Clever. Thanks for the clear write-up!


Simplify CI scripts

Fix #116.

Current coverage is 82.52%

Merging #119 into master will not affect coverage as of e7686e6

@@            master    #119   diff @@
======================================
  Files           19      19
  Stmts          698     698
  Branches        57      57
  Methods          0       0
======================================
  Hit            576     576
  Partial         18      18
  Missed         104     104

Review entire Coverage Diff as of e7686e6

Powered by Codecov. Updated on successful CI builds.


Fix typo in doc-en-US.md

Fixed typo 👇🏼
you can use it to get the html code.

It has been updated, thank you for your contribution.


Hotfix Version information provided at start plus usage screen

Coverage remained the same at 100.0% when pulling 5908b4cb7665ab9ed0b3e1afac8e7c2f789022ea on hotfix/added-missing-version-information into 70281482e3eac553cd911bd4f43381c2da890fd8 on develop.


GPG error on update

Hello, I'm running a variant of Fedora 40 (Nobara). cran2copr has worked amazingly for me so far.
But today, I am getting a GPG error in the system updater while trying to update cran-tm.
Verifying a signature using certificate 3124D2EF76DA4D972F6BE4AC9D60CBB71A3B4456 (iucar_cran (None) <iucar#cran@copr.fedorahosted.org>):
  1. Certificate 9D60CBB71A3B4456 invalid: certificate is not alive
      because: The primary key is not live
      because: Expired on 2024-08-13T00:46:08Z
  2. Key 9D60CBB71A3B4456 invalid: key is not alive
      because: The primary key is not live
      because: Expired on 2024-08-13T00:46:08Z
error: Verifying a signature using certificate 3124D2EF76DA4D972F6BE4AC9D60CBB71A3B4456 (iucar_cran (None) <iucar#cran@copr.fedorahosted.org>):
  1. Certificate 9D60CBB71A3B4456 invalid: certificate is not alive
      because: The primary key is not live
      because: Expired on 2024-08-13T00:46:08Z
  2. Key 9D60CBB71A3B4456 invalid: key is not alive
      because: The primary key is not live
      because: Expired on 2024-08-13T00:46:08Z
Copr repo for cran owned by iucar                                                            57 kB/s | 985  B     00:00
GPG key at https://download.copr.fedorainfracloud.org/results/iucar/cran/pubkey.gpg (0x1A3B4456) is already installed
The GPG keys listed for the "Copr repo for cran owned by iucar" repository are already installed but they are not correct for this package.
Check that the correct key URLs are configured for this repository.. Failing package is: R-CRAN-tm-0.7.14-1.fc40.copr7906035.x86_64
 GPG Keys are configured as: https://download.copr.fedorainfracloud.org/results/iucar/cran/pubkey.gpg
The downloaded packages were saved in cache until the next successful transaction.
You can remove cached packages by executing 'dnf clean packages'.
Error: GPG check FAILED

This is a problem for updating R packages, but it's also stopping me from updating my system as a whole!
In any case, would be happy to troubleshoot, let me know what info you need. Thanks!

Apparently this is an open issue, see https://github.com/fedora-copr/copr/issues/2894
For now, the workaround is to manually drop the old key and import the new one. In our case:
$ # drop the old key
$ sudo rpm -e gpg-pubkey-1a3b4456-5d54ab50
$ # install the prolonged one
$ sudo rpm --import https://download.copr.fedorainfracloud.org/results/iucar/cran/pubkey.gpg

Let's keep this issue open for visibility, because other users will hit the same wall.

This worked! Agreed on keeping it open.


fix: initialize logs right after configurations

As we talked in https://github.com/sxyazi/yazi/issues/703#issuecomment-1954797994 this PR makes it so that the logs are initialized in the beginning of the application.
I made it run right after the yazi_config::init(); because I'm not sure if there's any log that might occur if any error ever happens when initializing yazy_core or the yazi_scheduler.

Thanks!


Unit Testing and Continuous Integration

Now that I have done some tutorials about unit testing and continuous integration, perhaps it would make sense to write some tests to make sure the jekyll build isn't broken and/or any markdown pages contain all the necessarily fields?
I know that @haideralipunjabi created #326 which may be relevant?

This sounds like a really good idea!
Unfortunately, I haven't worked with unit testing and CI yet (and also wasn't able to attend your last stream 😞) but that way I'll learn something as well.

I haven't been in touch with your channel (coding in general) for the past few weeks. I will make sure to see the latest live stream.
#326 shouldn't mess with Jekyll as it only modifies the README.md.

I've collected some ideas for test cases that should help verifying pull requests. We should check for:

general Jekyll build errors
valid URLs (to external sources, other videos, etc.)
correct naming conventions (especially for the filename since it is used to build the URLs)
missing properties (e. g. every contribution has to have an author and a URL; all videos have to have a date, title and number)
use of video_id over a YouTube URL to enable automatic matching at a later point
use of an internal URL (like /CodingChallenges/...) over video_id when the referenced video is already available on the website
duplicated videos (could happen accidentally)

As soon as I notice something else that could be tested automatically, I'll update the list.

@nilsweb do you think you can convert your list to be checkboxes that can be thicked?

Can this be closed with #471 and #501? New Issues can be opened for specific testing bugs and features?

I think I'd like to keep this open until more of the issues in @nilsweb's comment are resolved.

I have zero experience with node, but hopefully I can contribute my ideas.
The valid url issue could be resolved by checking the status code each url returns, directly or using a service (if there is any?).
The naming convention could be achieved using RegEx.

I don't know if checking the status code for each URL is practical since there are a LOT of URLs to test. The tests would take a ridiculous amount of time to complete. Maybe we could use a RegEx here as well just to make sure that it might be a valid URL?
Nils - Contact Using Hop
On June 16, 2018 at 14:48 GMT, Haider Ali Punjabi<EMAIL_ADDRESS>wrote:
I have zero experience with node, but hopefully I can contribute my ideas.
The valid url issue could be resolved by checking the status code each url returns, directly or using a service (if there is any?).
The naming convention could be achieved using RegEx.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

I have a working test version for checking http statuses, but I'm not sure we actually want it to be an 'error' because we'll get people's PRs failing tests out of their own control. Perhaps it can just be printed to the logs so it can be checked manually? Not sure the best idea here, really, but implementing it let me find a bunch of broken links, so that's good.
As for correct naming conventions, can we define what that means, exactly? We already check the leading number so are we wanting to force lower-case kebab-case strings or is there more that should be checked?

can you add a line number to the format checker? If a line is blank the current output is less than helpful?
  ● _CodingChallenges › 130.1-fourier-transform-drawing.md › is formatted correctly

    Extra space at the end of line: '    '

      13 | const checkYAMLFormat = format => {
      14 |   if(format.match(/ \n/)) {
    > 15 |     throw new Error(`Extra space at the end of line: '${format.match(/[^\n]* \n/)[0].slice(0, -1)}'`);
      16 |   }
      17 |   if(format.match(/\n\n\n/)) {
      18 |     throw new Error('Double blank lines in YAML');

      at checkYAMLFormat (files.js:15:11)
      at YAMLFile.checkYAMLFormat [as load] (files.js:66:7)
      at Object.load (yaml.test.js:116:14)

Test Suites: 1 failed, 1 total
Tests:       1 failed, 6 skipped, 2575 passed, 2582 total
Snapshots:   0 total
Time:        8.41s
Ran all test suites.
npm ERR! Test failed.  See above for more details.
Exited with code 1


#9-add-function-to-detect-when-opened-file-is-loaded

What was done?

emitted event for remote and standalone version

This functionality is required for autotests


driftctl: 0.34.1 -> 0.37.0

Semi-automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/driftctl/versions.
meta.description for driftctl is: "Detect, track and alert on infrastructure drift"
meta.homepage for driftctl is: "https://driftctl.com/"
meta.changelog for driftctl is: "https://github.com/snyk/driftctl/releases/tag/v0.35.0"
Updates performed

Golang update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.35.0 with grep in /nix/store/2pkcarpfgx06bzl8b676nscysdg34mjg-driftctl-0.35.0
found 0.35.0 in filename of file in /nix/store/2pkcarpfgx06bzl8b676nscysdg34mjg-driftctl-0.35.0

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

1 package rebuild(s)

1 x86_64-linux rebuild(s)
1 x86_64-darwin rebuild(s)

First fifty rebuilds by attrpath
driftctl

Instructions to test this update (click to expand)

Build yourself:
nix-build -A driftctl https://github.com/r-ryantm/nixpkgs/archive/5bddc620123ef5f8ab425f9a6edc526169d84c17.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/2pkcarpfgx06bzl8b676nscysdg34mjg-driftctl-0.35.0
ls -la /nix/store/2pkcarpfgx06bzl8b676nscysdg34mjg-driftctl-0.35.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

driftctl

Maintainer pings
cc @KAction @06kellyjac @kalbasit for testing.

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/driftctl/versions.
meta.description for driftctl is: "Detect, track and alert on infrastructure drift"
meta.homepage for driftctl is: "https://driftctl.com/"
meta.changelog for driftctl is: "https://github.com/snyk/driftctl/releases/tag/v0.37.0"
Updates performed

Golang update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.37.0 with grep in /nix/store/c59zdl91ypwafpy0m78bkhshk7k2av0x-driftctl-0.37.0
found 0.37.0 in filename of file in /nix/store/c59zdl91ypwafpy0m78bkhshk7k2av0x-driftctl-0.37.0

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
driftctl

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/c59zdl91ypwafpy0m78bkhshk7k2av0x-driftctl-0.37.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A driftctl https://github.com/r-ryantm/nixpkgs/archive/d50bd820c4235b77953e154e335b0e861c10106e.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/c59zdl91ypwafpy0m78bkhshk7k2av0x-driftctl-0.37.0
ls -la /nix/store/c59zdl91ypwafpy0m78bkhshk7k2av0x-driftctl-0.37.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

driftctl

Maintainer pings
cc @KAction @06kellyjac for testing.

LGTM. Looks like it improved since 0.26 and found more unmanaged resources.

Result of nixpkgs-review pr 180485 run on x86_64-linux 1

1 package built:

driftctl


Node: VSS commands support

Describe the feature
VSS command support for Node.js
Full list is listed in https://github.com/valkey-io/valkey-glide/wiki/ValKey-Commands-Implementation-Progress
Commands
see: https://github.com/valkey-io/valkey-glide/wiki/ValKey-Commands-Implementation-Progress#vector-search-module

[ ] FT.CREATE
[ ] FT.SEARCH
[ ] FT.AGGREGATE
[ ] FT.DROPINDEX
[ ] FT.INFO
[ ] FT._LIST
[ ] FT.ALIASADD
[ ] FT.ALIASDEL
[ ] FT.ALIASUPDATE
[ ] FT._ALIASLIST
[ ] FT.PROFILE
[ ] FT.EXPLAIN
[ ] FT.EXPLAINCLI

Java

[ ] FT.CREATE - #2414
[ ] FT.SEARCH
[ ] FT.AGGREGATE
[ ] FT.DROPINDEX
[ ] FT.INFO
[ ] FT._LIST
[ ] FT.ALIASADD
[ ] FT.ALIASDEL
[ ] FT.ALIASUPDATE
[ ] FT._ALIASLIST
[ ] FT.PROFILE
[ ] FT.EXPLAIN
[ ] FT.EXPLAINCLI

Python

[ ] FT.CREATE - #2413
[ ] FT.SEARCH
[ ] FT.AGGREGATE
[ ] FT.DROPINDEX
[ ] FT.INFO
[ ] FT._LIST
[ ] FT.ALIASADD
[ ] FT.ALIASDEL
[ ] FT.ALIASUPDATE
[ ] FT._ALIASLIST
[ ] FT.PROFILE
[ ] FT.EXPLAIN
[ ] FT.EXPLAINCLI

Node

[ ] FT.CREATE
[ ] FT.SEARCH
[ ] FT.AGGREGATE
[ ] FT.DROPINDEX
[ ] FT.INFO
[ ] FT._LIST
[ ] FT.ALIASADD
[ ] FT.ALIASDEL
[ ] FT.ALIASUPDATE
[ ] FT._ALIASLIST
[ ] FT.PROFILE
[ ] FT.EXPLAIN
[ ] FT.EXPLAINCLI

New tickets for Java and Python:

https://github.com/valkey-io/valkey-glide/issues/2428
https://github.com/valkey-io/valkey-glide/issues/2429

FT.CREATE - in progress
other comments are not started

FT.CREATE - in progress
other comments are not started


Tooltip issue in courses

In tooltip district name is in bold letters rest of them in normal

Retested not fixed

District Name keyword is removed and district name will appear in bold similar to other reports in courses(user progress).

Working as expected


Problems loading doujin/thumnails

复现步骤 / Step to reproduce
1.open app
2.only doujin that seem to be loading are on the ranklist
3.Search for a tag any is fine seems to be stuck in the loading animation no timeout error occurs
Sorry if I'm not detailing this correctly it's my first time reporting something like this
预期行为 / Expected behaviour
It all should be loading just fine haven't had this problem until a couple weeks ago
实际行为 / Actual Behaviour
Nothing seems to be loading on the app seems to be stuck loading when searching for any tag or just being on the home screen as well
The only few things loaded are things in the ranklist
I am able to click go to the doujins in the rank list and in history but none of the gallery previews thumnails are loading if I click on the gallery the images load just fine and I'm able to (read /see) the (doujins/image sets) once I click a image number

![Screenshot_20230617_113850]

应用日志 / App logs

备注 / Addition details
I have checked my account and logged into exhentai on the Chrome app to make shure there was nothing wrong with my account it seems to be working fine cleared cookies and everything
Update
I've changed the site settings from ex-hentai to e-hentai It appears I can actually search for tags now doujin are popping up when I search for things but no thumbnails or gallery previews are loading still
I am still able to go click on a image in the Gallery and it loads just fine but no gallery previews seem to load at all

JHenTai 版本号 / JHenTai version
v7.2.0+2129
操作系统版本 / operating system version
Android
设备型号 / Device model
Samsung s22+
自查步骤 / Verify steps

[X] 您已仔细查看并知情 常见问题 中的内容 / You have checked Q&A carefully
[X] 您已搜索过 Issue Tracker，没有找到类似内容 / I have searched on Issue Tracker, No duplicate or related open issue has been found
[X] 您确保这个 Issue 只提及一个问题。如果您有多个问题报告，烦请发起多个 Issue / Ensure there is only one bug report in this issue. Please make mutiply issue for mutiply bugs
[X] 您确保已使用最新版本测试，并且该问题在最新版本中并未解决 / This bug have not solved in latest version #

1、only doujin that seem to be loading are on the ranklist According to your logs, your met Sad Panda which means you are blocked by Ehentai. You can swtich to another network proxy or you need to use Cookie Login.
2、Nothing seems to be loading on the app seems to be stuck. It's a bug after previous version, i have fix it in v7.2.1, now it will show an error message when you meet Sad Panda and you can retry.

Yea I've been switching to diffrent networks useing express vpn no luck there
I have updated to the new version I'm getting that sad panda error you are talking about now
How do I do this cookie login you are talking about
Update: nvm I've logged out completely and relogged back in seem to fix that sad panda error


Dg seo updates

Affected Components

[ ] Content & Marketing
[ ] Pricing
[ ] Test
[ ] Docs
[ X] Learn
[ ] Other

Pre-Requisites

[ ] Code is linted (npm run lint)

Notes for the Reviewer
We updated the structure and content of our Learn section to focus on "Playwright" in this phase.

Removed mentions of Puppetter
Improved the structure of the sidebar nav
Optimized H1s, Titles, and URLs
Built redirects to move traffic over from the old URL

New Dependency Submission

Screenshots

Did not mean to touch the docs on this branch - will revert!

Can you set up redirects/aliases for the links you changed, so the user doesn't hit a 404?

Did you see any specifically besides in the Doc changes? I think all the links in Learn should be redirected.


Update MD file on new branch 2017-12-05_21-32-20

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

split/updateTestRef1.md
:white_check_mark:Succeeded

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.


Fix initial acces point

This wasn't working. It's working as expected now. I made the change of setting a default password for the initial AP. I think it's reasonable just for security reason and because most devices do it on this way.

The other PR should be merge first.. so the changes in here will only appear in the diff.. Sorry about that

ok, then we wait for the other integrations and then we rebase this one before the merge

Resolved by feature/settings-store


[2.8] Update configuration input check in RKE2 ACE validation tests

Issue: https://github.com/rancher/qa-tasks/issues/883

Problem

Current RKE2 ACE tests fail cause of: Should NOT be empty, but was ...
Solution

Updated the the configuration empty check should be updated as a nil check in RKE2 ACE tests.
Engineering Testing
Automated Testing

Test types added/modified:

Validation (Go Framework)

Backport: https://github.com/rancher/rancher/pull/42498


Sensu Client and Check Delete Actions

G'day All,
This "Issue" is to discuss the best design for allowing client and check delete actions to be called.  I'm happy to attempt coding the pull request when I get time but wanted to verify the design first.   All suggestions welcome.... or it might take someone less time to implement than read this ;)
Our Scenario:
We have a wrapper cookbook (mn_monitor) that has "client" and "check" LWRPs that call sensu_client and sensu_check.   This works fine as long as you always call sensu_client create before any sensu_check create or deletes.
In the event you want to remove monitoring, because sensu_client doesn't support delete, mn_monitor_client delete just stops the service and removes package.    However, if you run sensu_check delete in this scenario it then fails the chef run when it tries to notify the sensu_service_trigger as it's not defined in the resources collection
[2015-07-07T15:21:25-07:00] FATAL: Chef::Exceptions::ResourceNotFound: resource file[C:\etc\sensu/conf.d/checks/*****.json] is configured to notify resource ruby_block[sensu_service_trigger] with action create, but ruby_block[sensu_service_trigger] cannot be found in the resource collection.
My temporary hack is to only run deletes if the sensu_client is installed...
def delete_check
sensu_check new_resource.check_name do
action :delete
only_if { run_context.resource_collection.include?("ruby_block[sensu_service_trigger]") }
end
end
So the question is:

Is there a better way than my hack to modify sensu_check delete so that it only tries to restart the service if it exists?
Is there a better way to implement sensu_client delete than a service stop and package delete? (or maybe that's a fine start for now and can be improved later)

I'm running into the same issue.
Plan is:

write a library to curl Sensu api if node['sensu']['client']['delete'] is set to true
have a ternary operator on sensu_client action ( action node['sensu']['client']['delete'] ? :nothing : :create)

Then if you really want, you can add another logic to nuke everything (which is what you want I believe).

Closing as I believe this issue has been addressed circa https://github.com/sensu/sensu-chef/commit/f55b87f0f07bb9e718262d5a1da2c2a5bf6db9d3. Please feel free to open a new issue if you are still experiencing problems.


line Indicator moves away from the perfect CrossLine

Good job on the puzzle-solving user interface.
I noticed there's a glitch when swiping across the line; sometimes, the visual indicator skips the cross line, either going up or down.

@AneesSelinei Could you add any screen recording to demonstrate the issue in more depth?

Uploading Screenrecorder-2024-01-15-09-40-44-463(0).mp4…

@Amonc
This behavior makes it difficult to swipe cross words.
https://github.com/Amonc/crossword/assets/55734339/042f8f8b-ec4e-44e8-9b1f-6e42ac191574

@Amonc
This behavior makes it difficult to swipe cross words.
https://github.com/Amonc/crossword/assets/55734339/0f5ee583-5d32-44ce-9dec-0c9cf00d64a2

@Amonc
This behavior makes it difficult to swipe  the cross words.
https://github.com/Amonc/crossword/assets/55734339/8b7a0f02-0011-402e-8e2a-912ae14f5c5b

@AneesSelinei This is not a bug or glitch.  We intentionally added it to deselect a line while selecting through the wrong letters.

@Amonc  I apologize, but I fail to see this crossed line as a feature because it clearly does not highlight the letters I swiped.

can we disable this type of behavior ? if i drag to a cross line and then moves 1 step it creates an illegal line, also while drawing the line it makes an illegal line (as mentioned in this issue)


Build Failed: use of unstable library feature 'array_from_fn'

Summary of Bug
error[E0658]: use of unstable library feature 'array_from_fn'
   --> /root/.cargo/git/checkouts/algebra-7e23afa68841b66e/d3e888d/serialize/src/impls.rs:415:22
    |
415 |         let result = core::array::from_fn(|_| {
    |                      ^^^^^^^^^^^^^^^^^^^^
    |
    = note: see issue #89379 <https://github.com/rust-lang/rust/issues/89379> for more information

For more information about this error, try `rustc --explain E0658`.
error: could not compile `ark-serialize` due to previous error
warning: build failed, waiting for other jobs to finish...
error: build failed

Version
commit f6df807ea8e3051980bc79e8706b8a9b03f48b1c
Update ark-snark to ark-algebra (#366)
Steps to Reproduce
Run build with cargo build --release
Please suggestion a solution for this issue as I am new to Rust and Arkworks. Thank you!

Hi! The error is probably because we've updated the library to use features from a newer (but still stable) version of Rust. If using rustup, I would run rustup update to get the latest stable version of Rust, and this problem should go away.

My rust should be the latest version, and the problem is still there. Do you have another fix?
info: syncing channel updates for 'stable-x86_64-unknown-linux-gnu'
info: checking for self-updates

  stable-x86_64-unknown-linux-gnu unchanged - rustc 1.64.0 (a55dd71d5 2022-09-19)

info: cleaning up downloads & tmp directories

rustc 1.64.0 (a55dd71d5 2022-09-19). This is the output, and the error showed it is actually using array::from_fn. Is it the same as array_from_fn?

Sorry, I meant the array_from_fn feature was partially stabilized; this stabilization enables the use of array::from_fn on stable.


Update Template Metadata Color Family to Array

Feature Description
This allows us to have family: ["gray", "grey"] and have explore template search results be less confusing.
Alternatives Considered

Additional Context
Will require updating a test and the filter in the dashboard.

So the suggestion is to have an array just to support different types of spelling?

Why not just use „Gray“ everywhere?

this came up in bug bash as a point of confusion if you're searching for "grey" and see one template (because of the tag: "grey") and then you search for "gray" and get a bunch more templates.
So I kinda see it as a way to catch everything that's either got 2 or more spellings (which, I admit, is very much focused on English) but then also colors that are more than 1, say "salmon" - could be pink or orange.
can also wait and see if this even comes up for users in the search.

The P0 item here is to change the 1 spelling of "Grey" in the code base to "Gray" for consistency:
https://github.com/google/web-stories-wp/blob/f505902af9b81b8b84a82ca93c7913d7a2789a82/packages/templates/src/raw/magazine-article/metaData.js#L31
Everything beyond that I'd put on hold for now

Sounds good, PR is here: https://github.com/google/web-stories-wp/pull/9793


Wheel shape width/height adaption

Hey everyone!
First off all, I love the library. Is there a way to change the width/height of the Wheel variant but not use a square?
I am able to change it with the props but the draggable picker still always seems to rotate in a circle?
I'd appreciate any help here!

@alexscro I tested it and it wasn't very good at handling this situation. If you have a solution, please feel free to contribute a PR.

Hey everyone!
First of all, I love the library. Is there a way to change the width/height of the Wheel variant but not use a square? I am able to change it with the props but the draggable picker still always seems to rotate in a circle?
I'd appreciate any help here!

Use scaleY to change dimensions
style={{
transform: "scaleY(0.5)",
}}


Accessing the ref in order to run scroll operations on lists

Hi !
I'm using this library since few months and everything was great so far.
I have tried to add a "scrollTo" usage on the scrollView but I got the "classic" errors related to refs.
 Function components cannot be given refs. Attempts to access this ref will fail. Did you mean to use React.forwardRef()
<StretchyScrollView
      ref={(ref) => this.productsSectionList = ref}
>
  ....

How should I process to do such thing ?
Thanks !

Hey @xavierh81 👋
Thanks for opening this issue and sorry for the late response.
You can now pass ref object in v4.

Hi @hamidhadi. I have tried to use the ref after upgrading the library to version 4.0.
But I still can't access to scroll methods.
I got the "classic error": "this.productsSectionList.scrollTo is not a function"
Do you have any idea on how I can access to scrollTo and scrollToLocation methods of the StretchySectionList?

Actually, I just found the answer here: https://stackoverflow.com/questions/54617702/scrolltolocation-is-undefined-in-sectionlist-ref-of-react-native-neither-in-fla#comment96286933_54711573
As it's an Animated component, we have to use the getNode() function in order to use the SectionList functions.