MINC / BIDS mapping

Hi everyone,
During some discussions that came up at the CONP (Canadian Open Neuroscience Platform) Hackathon in June, we discovered that there were was a lack of mapping between MINC and BIDS.
My understanding was that the main issue was differences in names for metadata tags? (i.e. something like  "scan date" would have two different phrases that refer to the same thing.) But maybe someone more familiar with the MINC format could offer more insight into what the problem is.
Is this something that's preventing the adoption of BIDS in the MINC community?
@emdupre @jbpoline do you know anything about this?

@Park-Patrick - can you do a quick search on the BIDS mailing list and link here to any discussions about this? I’d be surprised if there hasn’t been anything before....

Your instincts were right on @KirstieJane -- there was a brief discussion here.
I don't use MINC myself, but I'm pinging in @gdevenyi who would be a great resource for these sorts of questions if he's available to help !

I can say that representatives of MINC were included in the original push to create BIDS, and that a draft spec was created during that time with MINC, which can be found here:
https://docs.google.com/document/d/1f5E1Wif9o8M8KAvTZSyAfjIPRXUX0CrH1mKxEjKQSB4/edit
I'm not sure of the current status or whether anyone is working on it. I wasn't involved in the original work.

@samirdas What are your thoughts?


Installation failing

Fails every time I try to run cmake. I downloaded the most recent DynamoRIO build, and verified that the cmake folder does contain DynamoRIOConfig.cmake
Anyone else encountered this issue?
-- Selecting Windows SDK version 10.0.16299.0 to target Windows 6.1.7601.
CMake Warning at CMakeLists.txt:66 (find_package):
  By not providing "FindDynamoRIO.cmake" in CMAKE_MODULE_PATH this project
  has asked CMake to find a package configuration file provided by
  "DynamoRIO", but CMake did not find one.

  Could not find a package configuration file provided by "DynamoRIO" with
  any of the following names:

    DynamoRIOConfig.cmake
    dynamorio-config.cmake

  Add the installation prefix of "DynamoRIO" to CMAKE_PREFIX_PATH or set
  "DynamoRIO_DIR" to a directory containing one of the above files.  If
  "DynamoRIO" provides a separate development package or SDK, be sure it has
  been installed.

CMake Error at CMakeLists.txt:68 (message):
  DynamoRIO package required to build

-- Configuring incomplete, errors occurred!

I had a similar issue, using the full path in the command line parameter -DDynamoRIO_DIR worked. Maybe it will help with your issue too.


update profiler screenshot for 1.5.0 release

This update the profiler screenshots to show HttplugBundle 1.5.0 changes.
Closes #192.

Thank you.


[FEATURE] Add native JSON support

Is your feature request related to a problem?
Ingesting a document today requires supplying it as an IoReader. Starting with an example in https://xeiaso.net/blog/elasticsearch:
data, err := json.Marshal(esEntry)
if err != nil {
	log.Fatalf("failed to marshal entry: %v", err)
}

resp, err := es.Index("site-search-kb", bytes.NewBuffer(data), es.Index.WithDocumentID(entry.ID))

Ugh.
What solution would you like?
Automatically marshal objects to/from JSON. Starting with the canonical example from https://go.dev/blog/json I think I want something that doesn't say "IoStreams" everywhere.
m := Message{"Alice", "Hello",<PHONE_NUMBER>881547000}
es.Index("site-search-kb", m, es.Index.WithDocumentID(1))

Do you have any additional context?
Came from https://xeiaso.net/blog/elasticsearch

@Jakob3xD would love your thoughts!

I went through the code to understand the Issue and found opensearchutil.NewJSONReader. Is this sufficient to resolve the Issue? Is it enough to resolve the Issue or is there some functionality missing?
This utility function seems to be used as follows.
https://github.com/opensearch-project/opensearch-go/blob/c04e36a91a64755d6205a64326a65667d51d285d/opensearchutil/json_reader_integration_test.go#L55-L62

@YuyaAbo Thank you!

Can this be somehow automatic? In this case this would be a feature request.
Do we document this anywhere? Either way we should document this in the user guide, then we can close this. Would you like to help?

@dblock Yes, it would certainly be a useful addition if it could be automated.
I will add an introduction to opensearchutil.NewJSONReader to the user guide. I will send a PR later.
We will leave this issue as is for the purpose of adding functionality.

Should this issue be closed? #341 seems to have committed a merge request to add the functionality unless I am mistaken.

Yes, closing.


[VS Code] Joining collaboration session: Agent terminated with exit code: -4058 and signal null:

%0A%0A%23%23 Error:%0AJoining collaboration session: Agent terminated with exit code: -4058 and signal null: %0A%0A%23%23 Steps to Reproduce:%0A1.%0A2.%0A%0A||Version Data|%0A|-:|:-|%0A|**extensionName**|VSLS|%0A|**extensionVersion**|1.0.337|%0A|**protocolVersion**|2.2|%0A|**applicationName**|VSCode|%0A|**applicationVersion**|1.34.0|%0A|**platformName**|Windows|%0A|**platformVersion**|10.0.17134|

Hi @yjyOfGit, thanks for the feedback! Can you attach your logs for us to investigate? (> Live Share: Export Logs command).
Thanks!

Closing as stale. Please reopen if you have more info for us to investigate.


Misc fixes and improvements

Fixes:

Fixes :rainbowify not working
Fixes :noclip getting stuck on seats
Fixes :noclip falling through terrain
Fixes :noclip falling through parts
Fixes :noclip sometimes getting stuck on running animation
Fixes :material Enum value not recognizing all possible materials
Fixes ScriptBase not having script global
Fixes ScriptBase sometimes failing due to _G API missing
Fixes :rainbowify clogging up network replication and not updating during network lag
Fix effect not respecting topbarsize
Fix effect animations lagging
Fix blur effect blurring too fast
Fix server mutex breaking during deletion
Fix balance song name
Fix oldlogs not respecting NoSave

Added:

Added possibility to run client context scripts (like localscripts but can run at any location inside replicated DataModel that regular scripts can)
Add :clearnote as an alias for :removenote
Add :seize as an alias for :seizure

Change:

Optimize AC texture id pooling
Optimize material number to enum conversion
Optimize effect
Optimize :pixelize
Optimize oldlogs saving by a lot

lint fail :/


Do not skip serializing number of cells for VTU output?

ParaView and meshio complain when NumberOfCells="0" is missing in the output XML. I'm not sure what the reason is for skipping it, but apparently not skipping the serialization is sufficient to fix ParaView. meshio seems to have other troubles. I glanced at the code of meshio and it might just be that their logic is broken for empty meshes. I'm not entirely sure as I'm not so fluent in the style of Python they use.
I wanted to add a test but I couldn't quite figure out the structure of tests, i.e. where to put it, and how best to create a test like this. With some hints I might be able to do that.

Hi @Andlon! thanks the PR! Could you include a new test in the tests subdirectory called "paraview.rs" and add your test there? You can think of each test file being there to address a certain class of features (legacy vs. xml vtk files) or compatibility (pygmesh/meshio). Please indicate the version of ParaView in the comments of the test. Could you also add a subdir in the assets folder called "paraview" and place your test file there so it's easy to find when working with paraview compatibility issues.
Let me know if you have ideas of how to organize the tests. There didn't seem to be a need for this before since there are not a lot of tests.

I think the reason why number of cells is not serialized is to omit it from polygon meshes, which are not supposed to have cells. This is why the test is failing. Perhaps we should decide what to serialize based on the type of mesh.


add currency KRW

🇰🇷

Hi @SongYunSeop, you are probably running the previous release. I included pretty much all currencies in the world recently:

You can see that KRW is now included in the config file:

@geraldoramos Thanks!


[BUG] dataLakeFileSystemClient,listPaths() throws NumberFormatException

Describe the bug
The code below
    DataLakeServiceClient dataLakeServiceClient = new DataLakeServiceClientBuilder()             .endpoint("https://something.dfs.core.windows.net")             .credential(new StorageSharedKeyCredential(                     "accountname",                     "="))             .buildClient();     DataLakeFileSystemClient dataLakeFileSystemClient = dataLakeServiceClient.getFileSystemClient("containename");     List<PathItem> pathItems = dataLakeFileSystemClient.getDirectoryClient("").listPaths().stream()             .collect(Collectors.toList());         Assertions.assertEquals(pathItems.size(), 5);
throws
For input string: "Tue, 02 Jan 2024 14:29:41 GMT" java.lang.NumberFormatException: For input string: "Tue, 02 Jan 2024 14:29:41 GMT" 	at java.base/java.lang.NumberFormatException.forInputString(NumberFormatException.java:67) 	at java.base/java.lang.Long.parseLong(Long.java:711) 	at java.base/java.lang.Long.parseLong(Long.java:836) 	at com.azure.storage.file.datalake.Transforms.toPathItem(Transforms.java:384) 	at java.base/java.util.stream.ReferencePipeline$3$1.accept(ReferencePipeline.java:197) 	at java.base/java.util.ArrayList$ArrayListSpliterator.forEachRemaining(ArrayList.java:1625) 	at java.base/java.util.stream.AbstractPipeline.copyInto(AbstractPipeline.java:509) 	at java.base/java.util.stream.AbstractPipeline.wrapAndCopyInto(AbstractPipeline.java:499) 	at java.base/java.util.stream.ReduceOps$ReduceOp.evaluateSequential(ReduceOps.java:921) 	at java.base/java.util.stream.AbstractPipeline.evaluate(AbstractPipeline.java:234) 	at java.base/java.util.stream.ReferencePipeline.collect(ReferencePipeline.java:682) 	at com.azure.storage.file.datalake.DataLakeDirectoryAsyncClient.lambda$listPathsWithOptionalTimeout$16(DataLakeDirectoryAsyncClient.java:1270) 	at reactor.core.publisher.FluxMapFuseable$MapFuseableSubscriber.onNext(FluxMapFuseable.java:113)
This is on 12.18.1 of the client skd and the code in question is here:
https://github.com/Azure/azure-sdk-for-java/blob/azure-storage-file-datalake_12.18.1/sdk/storage/azure-storage-file-datalake/src/main/java/com/azure/storage/file/datalake/Transforms.java#L384
looking at main, the code appears to be different  and catches this exception
https://github.com/Azure/azure-sdk-for-java/blob/2c940cfb82d1ff2c4691a464e0a6d1dd64b43b2e/sdk/storage/azure-storage-file-datalake/src/main/java/com/azure/storage/file/datalake/Transforms.java#L386
I cant figure out how the releases are managed so not sure when the bug was introduced vs fixed  - can we release another version with the fix that exists in main?
Exception or Stack Trace
 input string: "Tue, 02 Jan 2024 14:29:41 GMT" java.lang.NumberFormatException: For input string: "Tue, 02 Jan 2024 14:29:41 GMT" 	at java.base/java.lang.NumberFormatException.forInputString(NumberFormatException.java:67) 	at java.base/java.lang.Long.parseLong(Long.java:711) 	at java.base/java.lang.Long.parseLong(Long.java:836) 	at com.azure.storage.file.datalake.Transforms.toPathItem(Transforms.java:384) 	at java.base/java.util.stream.ReferencePipeline$3$1.accept(ReferencePipeline.java:197) 	at java.base/java.util.ArrayList$ArrayListSpliterator.forEachRemaining(ArrayList.java:1625) 	at java.base/java.util.stream.AbstractPipeline.copyInto(AbstractPipeline.java:509) 	at java.base/java.util.stream.AbstractPipeline.wrapAndCopyInto(AbstractPipeline.java:499) 	at java.base/java.util.stream.ReduceOps$ReduceOp.evaluateSequential(ReduceOps.java:921) 	at java.base/java.util.stream.AbstractPipeline.evaluate(AbstractPipeline.java:234) 	at java.base/java.util.stream.ReferencePipeline.collect(ReferencePipeline.java:682) 	at com.azure.storage.file.datalake.DataLakeDirectoryAsyncClient.lambda$listPathsWithOptionalTimeout$16(DataLakeDirectoryAsyncClient.java:1270) 	at reactor.core.publisher.FluxMapFuseable$MapFuseableSubscriber.onNext(FluxMapFuseable.java:113) 	at reactor.core.publisher.Operators$MonoSubscriber.complete(Operators.java:1839) 	at reactor.core.publisher.MonoFlatMap$FlatMapInner.onNext(MonoFlatMap.java:249) 	at reactor.core.publisher.FluxSwitchIfEmpty$SwitchIfEmptySubscriber.onNext(FluxSwitchIfEmpty.java:74) 	at reactor.core.publisher.FluxMapFuseable$MapFuseableSubscriber.onNext(FluxMapFuseable.java:129) 	at reactor.core.publisher.FluxHandleFuseable$HandleFuseableSubscriber.onNext(FluxHandleFuseable.java:191) 	at reactor.core.publisher.FluxSubscribeOnCallable$CallableSubscribeOnSubscription.run(FluxSubscribeOnCallable.java:251) 	at reactor.core.scheduler.SchedulerTask.call(SchedulerTask.java:68) 	at reactor.core.scheduler.SchedulerTask.call(SchedulerTask.java:28) 	at java.base/java.util.concurrent.FutureTask.run(FutureTask.java:264) 	at java.base/java.util.concurrent.ScheduledThreadPoolExecutor$ScheduledFutureTask.run(ScheduledThreadPoolExecutor.java:304) 	at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1136) 	at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:635) 	at java.base/java.lang.Thread.run(Thread.java:840) 	Suppressed: java.lang.Exception: #block terminated with an error 		at reactor.core.publisher.BlockingSingleSubscriber.blockingGet(BlockingSingleSubscriber.java:100) 		at reactor.core.publisher.Flux.blockLast(Flux.java:2761)
To Reproduce
See above
Code Snippet
see above
Expected behavior
List file returns a list of files
Screenshots
If applicable, add screenshots to help explain your problem.
Setup (please complete the following information):

OS: [e.g. iOS] mac os
IDE: [e.g. IntelliJ] intellij
Library/Libraries: [e.g. com.azure:azure-core:1.16.0 (groupId:artifactId:version)]
`
implementation platform('com.azure:azure-sdk-bom:1.2.19')
implementation 'com.azure:azure-storage-blob'
implementation 'com.azure:azure-identity'
implementation 'com.azure:azure-storage-file-datalake'

`

Java version: [e.g. 8]
App Server/Environment: [e.g. Tomcat, WildFly, Azure Function, Apache Spark, Databricks, IDE plugin or anything special]
Frameworks: [e.g. Spring Boot, Micronaut, Quarkus, etc]

If you suspect a dependency version mismatch (e.g. you see NoClassDefFoundError, NoSuchMethodError or similar), please check out Troubleshoot dependency version conflict article first. If it doesn't provide solution for the problem, please provide:

verbose dependency tree (mvn dependency:tree -Dverbose)
exception message, full stack trace, and any available logs

Additional context
Add any other context about the problem here.
Information Checklist
Kindly make sure that you have added all the following information above and checkoff the required fields otherwise we will treat the issuer as an incomplete report

[ ] Bug Description Added
[ ] Repro Steps Added
[ ] Setup information Added

Hi @deewar
Thanks for reaching out! There was a fix introduced for the issue you are experiencing and we are scheduled to roll out our patch release fix in February. The latest azure-storage-blob version 12.26.0 that will be released next month will be the version containing the fix.
If this is an urgent need, we can look into releasing patch fix sooner.

@ibrahimrabab  considering this only happens when trying to use the azure-storage-file-datalake sdk against a storage account with hierarchical namespace disabled i think it might be fine to wait for the next release. I  will continue to test with an accout with heirarchical namespace enabled for now.


RQ Scheduler causes workers to sporadically hang after receiving jobs

This occurs on Python 3.11.4 Versions of installed libraries (pared down for relevance):
Package                   Version
------------------------- ------------
arrow                     1.3.0
crontab                   1.0.1
Cython                    3.0.4
Flask                     2.3.3
Flask-RQ2                 18.3
pyarrow                   13.0.0
redis                     5.0.1
Redis-Sentinel-Url        1.0.1
rq                        1.15.1
rq-dashboard              0.6.7
rq-scheduler              0.13.1

# oO0OoO0OoO0Oo Redis is starting oO0OoO0OoO0Oo
# Redis version=5.0.3, bits=64, commit=00000000, modified=0, pid=3999114, just started

About 1 in 10 jobs sent to an RQ worker while RQ scheduler is running will hang indefinitely.  This just began happening last Friday after a pip install to update a venv's deps.
The job's main body is never reached, I have seen jobs whose first call is logging never produce any logs after being anomalously claimed. It does not matter if RQ scheduler was the originator of the job: I have seen both scheduled and manually invoked jobs hang in this manner. The behavior is no longer exhibited when RQ scheduler is closed. The RQ worker still responds to signals and can still have a successful warm shutdown. No logs are produced after the first logger call indicating the worker took the job. Here's an annotated example of one of those logs:
# Previous job successfully finishing
INFO:rq.worker:default: Job OK (92135fb3-a891-4bc5-97be-8f0e4801424c)
INFO:rq.worker:Result is kept for 3600 seconds
# Anomalous job begins
INFO:rq.worker:default: Uploading file (84db2e7a-c99b-463b-8e70-76cce10beb01)
# Hangs here indefinitely, until a signal is sent
INFO:rq.worker:Worker 480298b5bce645e696cd8082c25510bd [PID 3260661]: warm shut down requested

No pertinent logging is reported by RQ scheduler, just the regular Registering birth and Registering death that happens when workers are spun up or killed.
I inspected the worker object as saved in the Redis server, and it has a busy status with the correct job ID listed within. The job objects seem to be mildly corrupted, as the status of the job remains queued and the last_heartbeat corresponding to the second the RQ worker froze up.
I note that there is precedence for RQ scheduler to cause this behavior, even if I was not able to track down the root cause: https://stackoverflow.com/questions/53640418/django-rq-rqworker-freezes-indefinitely / https://github.com/rq/rq-scheduler/issues/197
The config file fed into RQ is minimal and just spins up a copy of the parent Flask app in order to access an app context. Let me know if I need to post any more info or logs here. Thanks!

We're having the same behavior crop up today after removing RQ scheduler from the stack last week. Closing this issue as it likely isn't related to RQ scheduler.


Sounds good :)

Sounds good :)
Originally posted by @ktoso in https://github.com/apple/swift/pull/38815#discussion_r689960502

[ ]


[FR - paid plan] reducing friction in read-only to access drawing

Hi, I use a lot Excallidraw paid plan and embed my drawings in obsidian.md in read-only.
The problem is that people need to click "join" the room to see the drawing, which adds a lot of friction.

Could it be possible to directly access the drawing if it's read-only? Or skip the naming/join room thing, just remove this step?
Thanks & congrats for the amazing product ❤

Hello,
We are currently working on embed/readonly links - should be shipped in few days :) which are focusing on this kind of usecase!

@maielo few days...


Please improve the features of third-party repo

Describe your suggested feature

It's clear that the current third-party repo functionality isn't perfect.I added different repos where different repos have many of the same extensions.I don't know which one I should download, so I want a function that allows us to see the information of the extensions before downloading, such as its repo and release time, etc.
Other details
Btw,thank you guys sincerely for making it to this day and helping so many people around the world who love to read comics.You are the real heroes.
Acknowledgements

[x] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] If this is an issue with an official extension, I should be opening an issue in the extensions repository.
[X] I have updated the app to version 1.10.0.
[X] I will fill out all of the requested information in this form.

Also, hopefully you can add an "enable/disable" button to the repo menu to decide which repos to enable or troubleshoot


Add Font instructions when adding to an existing project

After adding the library to 2 different projects I stumbled upon the problem in both on how I add the font variable to my application for the default tw config to work.
I always had to check out the template in order to copy the next config and the _app.tsx from there.
The following content should be added to the install instructions.
Fonts
To add a font to your project you have to install @nextjs/font and add the following to your next.config.js:
const config = {
  reactStrictMode: true,
  experimental: {
    fontLoaders: [
      {
        loader: "@next/font/google",
        options: { subsets: ["latin"] },
      },
    ],
  },
}

Then declare the CSS variable in your \_app.tsx
import { Inter as FontSans } from "@next/font/google"

const fontSans = FontSans({
  subsets: ["latin"],
  variable: "--font-sans",
  display: "swap",
})

export default function App({ Component, pageProps }: AppProps) {
  return (
    <>
      <style jsx global>{`
	   :root {
		--font-sans: ${fontSans.style.fontFamily};
	  }
	}`}</style>
      <Component {...pageProps} />
    </>
  )
}

After thinking about it a bit more the font is not really related to the component library, but since the tailwind config in the install instructions include the font variable, a decision should be made to either exclude the font addition from the docs entirely or to add the suggested documentation changes.

This turned out to be very helpful to me, as I was just trying to integrate it with an already existing Next project and struggled finding out why fonts were not applied correctly. I'd definitely love to see it mentioned in the docs.

The font loaders option has been removed from NextJS in version 13.3
https://github.com/vercel/next.js/pull/46886

Please add this to docs

Done. https://ui.shadcn.com/docs/installation/next#fonts


How can I export my private key or keystore?

as the title says

Or when I create my address and returns the keyStore and private key

@hsfzmsn Did you get any information? How can I get private key when creating account using web3 PHP?

I'm sorry, I didn't find the information.

@kamran-jabbar I'm sorry, I didn't find the information.

I would like to know how to do this as well.


Add Support for BigInt

On MariaDB/MySQL, sometimes the normal integer field is too small to hold large integers. This may be an issue when using PostgreSQL as well. When using SQLite, BigInt should be the equivalent of INTEGER.

BigIntField is now supported when using SQLite, MariaDB/MySQL, and PostgreSQL.


Create a page listing ways the US is last in the world

parnetal leave
grand juries
Cash bail
...

Universal paid vacation


HANA: Set sessionVariable:APPLICATION in connection string

Pull Request Test Coverage Report for Build 00801c3bc-PR-930

3 of 3 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.01%) to 46.906%

Totals

Change from base Build 62bfffe1d:
0.01%

Covered Lines:
6556

Relevant Lines:
13977

💛  - Coveralls

It looks like the Docker build is failing in the CI. I will investigate and try to push the fix to your branch.


Banana Cake Pop OAuth 2 redirection fails with error HC0013 in Firefox

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the bug
In Firefox, when using OAuth 2 authentication in Banana Cake Pop, when clicking on Fetch Token, a token is retrieved from the authentication provider, and the redirection URL is called, but the browser gets stuck on this page, with the following error:
{
  "errors": [
    {
      "message": "Either the parameter query or the parameter id has to be set.",
      "extensions": {
        "code": "HC0013"
      }
    }
  ]
}

When using Edge (Chromium-based), the same error briefly appears when the redirect URL is called, but the tab quickly closes and the token is provided to Banana Cake Pop. In Firefox, the tab containing the redirect URL (and showing the above error) never closes.
Browser: Firefox 103.0 (current Release channel version)
HotChocolate NuGet package version: 12.12.1
Authentication settings
Type: OAuth 2
Grant type: Implicit
Redirect URL: https://localhost:7001/graphql/oauth2-redirect.html
Response type: Access token
Steps to reproduce

In Firefox, open Banana Cake Pop.
Go to Connection Settings → Authorization.
Enter the authentication settings (as above).
Click on Fetch Token.
Observe that a token is successfully retrieved from the authentication provider (it's visible in the URL), and that the redirect URL is visible in the browser, but that an error message (detailed above) is presented and the token is not returned to Banana Cake Pop.

Relevant log output
{
  "errors": [
    {
      "message": "Either the parameter query or the parameter id has to be set.",
      "extensions": {
        "code": "HC0013"
      }
    }
  ]
}

Additional Context?
No response
Product
Banana Cake Pop
Version
12.12.1

So the oauth2 provider currently only works reliably in the desktop app. The reason for this is that we use a popup window for the authentication. So as soon as you leave the boundary your origin (cross-origin request) the window will detach from the current page, and we cannot longer read the URIs. This is a browser security feature that works different in electron.
We will introduce a way to also solve this with redirects eventually.

@PascalSenn, I am also getting the same error as above, is there any way to fix it?


Add support for FETCH clause, CTEs and set operations

Here a quick overview of the changes I did:

SqmTranslatorFactory now receives SqmStatement during creation instead during translate call
SqlAstTranslatorFactory now receives Statement during creation instead during translate call
Removed SimpleSqmUpdateTranslator, SimpleSqmDeleteTranslator, SqmInsertTranslator and SqmSelectTranslator and use the generic variant SqmTranslator<? extends Statement>
=> This allows to have a single translator class per dialect whereas before, we required 4
Removed SqlAstUpdateTranslator, SqlAstDeleteTranslator, SqlAstInsertTranslator and SqlAstSelectTranslator and use the generic variant SqlAstTranslator<? extends JdbcOperation>
=> This allows to have a single translator class per dialect whereas before, we required 4
Rework CTE support in HQL and SQL AST model to reflect what the SQL standard does
Add support for modifiable CTEs i.e. DML in CTEs as supported by PostgreSQL and DB2 for efficient delete cascading
Rework CTE SqmMultiTableMutationStrategy to use modifiable CTEs as it previously was equivalent to using inline VALUES clause
QueryPart and SqmQueryPart as abstraction over set producing query expressions
Add support for set operations (UNION, INTERSECT, EXCEPT) in HQL and SQL AST model
Move most SQM AST translation logic to BaseSqmToSqlAstConverter for simpler extension for Dialect
Move most SQL AST translation logic to AbstractSqlAstTranslator for simpler extension for Dialect
Avoid rendering subclass or secondary table joins in general and especially for multi-table DML when possible
Avoid rendering null precedence, and thus also avoid emulation, if it matches the null ordering of the database
Create SqlAstTranslator implementations per Dialect and handle a few dialect specifics

This PR is not done yet. I still want to:

add tests for set operations and a API for the JPA Criteria extension
Implement LIMIT/OFFSET/FETCH handling in Dialect specific SqlAstTranslator implementations

Wow, this sounds great!
Can you please give us a few examples of the kinds of queries that are supported here?

For set operations, the prime example would be FROM Person WHERE ...
Deletes with the new CTE strategy implementation would generate queries like this:
with cte_ids as(
  select id from person where ...
), delete_1 as (
  delete from collection_table where person_id in (select id from cte_ids) returning person_id
), delete_2 as (
  delete from person where id in (select id from cte_ids) returning id
)
select count(*) from cte_ids

instead of multiple queries, which reduces the roundtrip time. I have planned to add another strategy called "anonymous block" which can be used for Oracle and other DBMS to combine multiple DML statements into a single statement.
The fetch clause part is essentially based on what PostgreSQL supports: https://www.postgresql.org/docs/current/sql-fetch.html


setup dnsPROVIDER_URL didn't work

when I use the following code (from the example):
String dnsServer = "<IP_ADDRESS>"; List<DnsEntry> entries = new DnsService(dnsServer ).lookup(domain, Type.A);
It didn't work,  the udp package still been sent the the default dns server.
try to use follow code:
`        Properties environment = new Properties();
environment.put(Context.PROVIDER_URL, "dns://"+dnsServer+":53");
environment.put(Context.INITIAL_CONTEXT_FACTORY, "com.sun.jndi.dns.DnsContextFactory");
List entries = new DnsService(environment).lookup(domain, Type.A);
`
It work well, so please help to fix this bug, Thanks.

Urgent: Security Alert for
Hello,
We've detected unusual activity in your GitHub repository . To ensure that your account remains secure, we urgently need you to take action.
What You Need To Do:

Verify Repository Settings

Ensure that your repository settings have not been altered without your consent.

Reset Your Password

Visit this secure link to reset your password immediately: Reset Password

Revoke Unauthorized Access

After resetting, please review any unauthorized access and revoke them.

Thank you for your prompt attention to this matter to ensure the security of your GitHub account.
If you did not initiate these changes, please contact us immediately.
Best Regards,
GitHub Security Team


fix lint for prow

Signed-off-by: Mike Ng<EMAIL_ADDRESS>The prow build is having difficulty with the lint target in the original Makefile lint target. I pulled the pieces that its running into the dedicated Makefile.prow hopefully this will resolve the issue.

/lgtm


Fix sampler failing test

What does this PR do ?
This PR fixes a bug that this PR introduced. I removed a line by mistake. The failing test pass with this PR.

Merging since the docbuilder failing workflow is not related to this PR


Is it possible to tween between icons?

I have a little Icons.favorite and Icons.favorite_border
I’d like to Tween between them when I toggle a “favorite” state. Is this doable at the moment?
Sorry for formatting. On mobile x.x

You could use an AnimatedCrossFade for that.

https://pub.dartlang.org/packages/fluttie
https://pub.dartlang.org/packages/lottie_flutter
might also be useful for this use case.
Did EpicEricEE's suggestion help?

Without additional information, we are unfortunately not sure how to resolve this issue. We are therefore reluctantly going to close this bug for now. Please don't hesitate to comment on the bug if you have any more information for us; we will reopen it right away!
Thanks for your contribution.


Update README.md fixed typo

Changed a typo from "carefully desgined by" to "carefully designed by"

Description

Motivation and Context

How Has This Been Tested?

[ ] Pass the test by running: pytest qlib/tests/test_all_pipeline.py under upper directory of qlib.
[ ] If you are adding a new feature, test on your own test scripts.

Screenshots of Test Results (if appropriate):

Pipeline test:
Your own tests:

Types of changes

[ ] Fix bugs
[ ] Add new feature
[ ] Update documentation

Thanks so much.
Welcome to become one of Qlib's contributors


/userHistory/{user_id} throws an internal server error

Cause: TypeError: get_user_history() missing 1 required positional argument: 'take'
Remedy: Add the missing include_alliance parameter.

Closed with v1.3.0


comby: init at 0.12.1

Motivation for this change
https://github.com/comby-tools/comby/releases/tag/0.12.1
Things done

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

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

This change is

@GrahamcOfBorg build comby

Still interested!

Still interested!

There is upstream interest to get it packed in nixpkgs (likely the author is Nix/Darwin user)
https://github.com/comby-tools/comby/issues/142

I rebased it onto master and fixed a few things:

Rename the patch file to diff
Enable useDune2 = true because that's apparently required now.

And because comby depends on a few custom dependencies we need to either push upstream to upstream their forked dependency changes or patch/package those (see https://github.com/comby-tools/comby/blob/master/comby.opam):

[x] mparser-comby: I temporarily added a new file for this
[ ] patdiff: Requires a different branch that doesn't cleanly merge with upstream, so I can't create a patch file (https://github.com/rvantonder/patdiff.git#0.13.0-patch-compatible-diffs)
[ ] hack_parallel: Not sure if this is the correct version
[ ] bisect_ppx: Not sure if this causes issues
[ ] camlzip: Not sure if this causes issues

1.0.0 is released, hack_parallel can be trivially updated, but comby is tricky

Thanks!
Closing the PR, please see https://github.com/NixOS/nixpkgs/pull/123986, https://github.com/NixOS/nixpkgs/pull/139408


#all, #first, #last, etc are unavailable if 'design' section is omitted in model

Older sample apps fail to work, and readme does not mention it, so it kept me confused for some time.
Model is:
class Case < CouchRest::Model::Base
    use_database 'cases'
    property :_id, String
    property :name, String
    property :person_id, String
    property :info, String
    property :state, String
    property :description, String
    timestamps!

	# design do
 #     	view :by_name
 #  	end

end

2.2.2 :004 >   Case.find('806f380a1be2b7443a64948bf4411ff9')
 => #<Case _id: "806f380a1be2b7443a64948bf4411ff9", name: "blah blah", person_id: "d3209328", info: "info info", state: "ok", description: nil, updated_at: 2015-09-19 12:38:28 UTC, created_at: 2015-09-19 12:38:28 UTC, _rev: "1-44de5d24e84af6257f115479367108bb", type: "Case">
2.2.2 :005 > Case.all
NoMethodError: undefined method `all' for Case:Class
	from (irb):5
	from /home/dy/.rvm/gems/ruby-2.2.2@couch/gems/railties-4.2.4/lib/rails/commands/console.rb:110:in `start'
	from /home/dy/.rvm/gems/ruby-2.2.2@couch/gems/railties-4.2.4/lib/rails/commands/console.rb:9:in `start'
	from /home/dy/.rvm/gems/ruby-2.2.2@couch/gems/railties-4.2.4/lib/rails/commands/commands_tasks.rb:68:in `console'
	from /home/dy/.rvm/gems/ruby-2.2.2@couch/gems/railties-4.2.4/lib/rails/commands/commands_tasks.rb:39:in `run_command!'
	from /home/dy/.rvm/gems/ruby-2.2.2@couch/gems/railties-4.2.4/lib/rails/commands.rb:17:in `<top (required)>'
	from /home/dy/.rvm/gems/ruby-2.2.2@couch/gems/activesupport-4.2.4/lib/active_support/dependencies.rb:274:in `require'
	from /home/dy/.rvm/gems/ruby-2.2.2@couch/gems/activesupport-4.2.4/lib/active_support/dependencies.rb:274:in `block in require'
	from /home/dy/.rvm/gems/ruby-2.2.2@couch/gems/activesupport-4.2.4/lib/active_support/dependencies.rb:240:in `load_dependency'
	from /home/dy/.rvm/gems/ruby-2.2.2@couch/gems/activesupport-4.2.4/lib/active_support/dependencies.rb:274:in `require'
	from /home/dy/tmp/co2/bin/rails:8:in `<top (required)>'
	from /home/dy/.rvm/gems/ruby-2.2.2@couch/gems/activesupport-4.2.4/lib/active_support/dependencies.rb:268:in `load'
	from /home/dy/.rvm/gems/ruby-2.2.2@couch/gems/activesupport-4.2.4/lib/active_support/dependencies.rb:268:in `block in load'
	from /home/dy/.rvm/gems/ruby-2.2.2@couch/gems/activesupport-4.2.4/lib/active_support/dependencies.rb:240:in `load_dependency'
	from /home/dy/.rvm/gems/ruby-2.2.2@couch/gems/activesupport-4.2.4/lib/active_support/dependencies.rb:268:in `load'
	from /home/dy/.rvm/gems/ruby-2.2.2@couch/gems/spring-1.4.0/lib/spring/commands/rails.rb:6:in `call'
	from /home/dy/.rvm/gems/ruby-2.2.2@couch/gems/spring-1.4.0/lib/spring/command_wrapper.rb:38:in `call'
	from /home/dy/.rvm/gems/ruby-2.2.2@couch/gems/spring-1.4.0/lib/spring/application.rb:183:in `block in serve'
	from /home/dy/.rvm/gems/ruby-2.2.2@couch/gems/spring-1.4.0/lib/spring/application.rb:156:in `fork'
	from /home/dy/.rvm/gems/ruby-2.2.2@couch/gems/spring-1.4.0/lib/spring/application.rb:156:in `serve'
	from /home/dy/.rvm/gems/ruby-2.2.2@couch/gems/spring-1.4.0/lib/spring/application.rb:131:in `block in run'
	from /home/dy/.rvm/gems/ruby-2.2.2@couch/gems/spring-1.4.0/lib/spring/application.rb:125:in `loop'
	from /home/dy/.rvm/gems/ruby-2.2.2@couch/gems/spring-1.4.0/lib/spring/application.rb:125:in `run'
	from /home/dy/.rvm/gems/ruby-2.2.2@couch/gems/spring-1.4.0/lib/spring/application/boot.rb:18:in `<top (required)>'
	from /home/dy/.rvm/rubies/ruby-2.2.2/lib/ruby/site_ruby/2.2.0/rubygems/core_ext/kernel_require.rb:54:in `require'
	from /home/dy/.rvm/rubies/ruby-2.2.2/lib/ruby/site_ruby/2.2.0/rubygems/core_ext/kernel_require.rb:54:in `require'
	from -e:1:in `<main>'2.2.2 :006 >

Hi! You can't query documents without a design document in CouchRest Model. You should however by able to call get on the id: Case.get('806f380a1be2b7443a64948bf4411ff9')

exactly, and this is not mentioned in the Readme nor elsewhere, which is quite confusing

I can appreciate that, although this is a limitation of CouchDB more than CouchRest Model. If you'd like to send a pull request with your suggested changes to the docs, more than happy to accept them.


Feature Request: Loadbalancing use-case

Hi all,
First of all, I've been using the SmartEVSE with Serkri for a while now, and I've been extremely happy. Fantastic work!
I'm looking for some help w.r.t. a specific load-balancing use-case, as I've recently decided to upgrade my setup by replacing my old 1-Phase "dumb EVSE" with another SmartEVSE unit.
So, currently i've got two SmartEVSE's, both running Serkri 1.8.0.
Both SmartEVSE's are connected via modbus, to each other, to a Sensorbox, and to two kWh meters (Eastron SDM72D-m).
LoadBalancing is configured, SmartEVSE 1 is Master, SmartEVSE 2 is configured as Node-1 (Slave).
SmartEVSE 1 has a fixed 16A, 3-Phase cable which is controlled by Relay 1, SmartEVSE 2 has a fixed 16A, 1-Phase cable which is controlled by Relay 2 (L1).
While loadbalancing is working correctly by making sure the ChargeCurrent on both units together is not greater than CIRCUIT, I was hoping if would be possible to have the Slave Unit have its own kWh-meter configured. I've noticed that when SmartEVSE2 is configured as Node1 (Slave), it is not actually using the configured kWh-meter (The configured kWh-meter values are also not used to display or even updated in the ev-meter->current fields for the slave-unit).
My setup is as follows.
                   mains
                     |
              [main breaker 3x25A]
                     |
              [kWh meter SENSORBOX]
                     |
  -----------------------------------
  |            |                    |
                          [smartevse breaker 3x16A]
                                    |
                              ----------------
                              |              |
                        [kWh meter 1]  [kWh meter 2]
                              |              |
                        [relay 1]         [relay 2]
                              |              |
                         [car 1]           [car 2]

Configurations are:
// SmartEVSE1
{   "version": "SERKRI-1.8.0",   "mode": "SMART",   "mode_id": 3,   "car_connected": false,   "wifi": {     "status": "WL_CONNECTED",     "ssid": "-",     "rssi": -62,     "bssid": "-"   },   "evse": {     "temp": 25,     "temp_max": 65,     "connected": false,     "access": true,     "mode": 1,     "loadbl": 1,     "pwm": 1024,     "solar_stop_timer": 0,     "state": "Ready to Charge",     "state_id": 0,     "error": "None",     "error_id": 0,     "rfid": "Not Installed"   },   "settings": {     "charge_current": 86,     "override_current": 0,     "current_min": 6,     "current_max": 16,     "current_main": 25,     "current_max_sum_mains": 600,     "solar_max_import": 0,     "solar_start_current": 4,     "solar_stop_time": 10,     "enable_C2": "Not present",     "modem": "Not present",     "mains_meter": "Sensorbox",     "starttime": 0,     "stoptime": 0,     "repeat": 0   },   "mqtt": {     "host": "xxx.xxx.xxx.xxx",     "port": 1883,     "topic_prefix": "SmartEVSE-xxxxx",     "username": "xxxxxx",     "password_set": 0,     "status": "Connected"   },   "home_battery": {     "current": 0,     "last_update": 0   },   "ev_meter": {     "description": "Eastron3P",     "address": 101,     "import_active_power": 0,     "total_kwh": 891.2,     "charged_kwh": 14.8,     "currents": {       "TOTAL": 0,       "L1": 0,       "L2": 0,       "L3": 0     },     "import_active_energy": 891.2,     "export_active_energy": 0   },   "mains_meter": {     "import_active_energy": 0,     "export_active_energy": 0   },   "phase_currents": {     "TOTAL": 159,     "L1": 140,     "L2": 6,     "L3": 13,     "last_data_update":<PHONE_NUMBER>,     "charging_L1": false,     "charging_L2": false,     "charging_L3": false,     "original_data": {       "TOTAL": 159,       "L1": 140,       "L2": 6,       "L3": 13     }   },   "backlight": {     "timer": 0,     "status": "OFF"   } }
// SmartEVSE2
{   "version": "SERKRI-1.8.0",   "mode": "SMART",   "mode_id": 3,   "car_connected": true,   "wifi": {     "status": "WL_CONNECTED",     "ssid": "-",     "rssi": -66,     "bssid": "-"   },   "evse": {     "temp": 28,     "temp_max": 65,     "connected": true,     "access": true,     "mode": 1,     "loadbl": 2,     "pwm": 221,     "solar_stop_timer": 0,     "state": "Charging",     "state_id": 2,     "error": "None",     "error_id": 0,     "rfid": "Not Installed"   },   "settings": {     "charge_current": 130,     "override_current": 0,     "current_min": 6,     "current_max": 16,     "current_main": 25,     "current_max_sum_mains": 600,     "solar_max_import": 0,     "solar_start_current": 4,     "solar_stop_time": 10,     "enable_C2": "Not present",     "modem": "Not present",     "mains_meter": "Sensorbox",     "starttime": 0,     "stoptime": 0,     "repeat": 0   },   "mqtt": {     "host": "xxx.xxx.xxx.xxx",     "port": 1883,     "topic_prefix": "SmartEVSE-xxxxx",     "username": "",     "password_set": 0,     "status": "Connected"   },   "home_battery": {     "current": 0,     "last_update": 0   },   "ev_meter": {     "description": "Eastron3P",     "address": 110,     "import_active_power": 2.7,     "total_kwh": 2.5,     "charged_kwh": 0.9,     "currents": {       "TOTAL": 0,       "L1": 0,       "L2": 0,       "L3": 0     },     "import_active_energy": 2.5,     "export_active_energy": 0   },   "mains_meter": {     "import_active_energy": 0,     "export_active_energy": 0   },   "phase_currents": {     "TOTAL": 159,     "L1": 140,     "L2": 6,     "L3": 13,     "last_data_update":<PHONE_NUMBER>,     "charging_L1": true,     "charging_L2": true,     "charging_L3": true,     "original_data": {       "TOTAL": 159,       "L1": 140,       "L2": 6,       "L3": 13     }   },   "backlight": {     "timer": 120,     "status": "ON"   } }

You'll have to upgrade to v3.5.0 when it comes out; if you like to test you can use current version v3.5.0-RC2.
Serkri was developed without test abilities for Master/Slave configurations, so you are on uncharted territories ...

I just installed v3.5.0-RC1, so far it works exactly as I was hoping. I'll keep testing. Great work!

Cheers. If I run into any instabilities or bugs I'll be sure to let you know.

Great! Please report on the appropriate repo.


Fix OpenShift test for rotation

Desired Outcome
Add missing test config for secrets rotation on OpenShift. OpenShift tests only run on "main", so this was not caught until after merging.
Changelog

[ ] The CHANGELOG has been updated, or
[x] This PR does not include user-facing changes and doesn't require a
CHANGELOG update

Test coverage

[x] This PR includes new unit and integration tests to go with the code
changes, or
[ ] The changes in this PR do not require tests

Documentation

[ ] Docs (e.g. READMEs) were updated in this PR
[ ] A follow-up issue to update official docs has been filed here: insert issue ID
[x] This PR does not require updating any documentation

Behavior

[ ] This PR changes product behavior and has been reviewed by a PO, or
[ ] These changes are part of a larger initiative that will be reviewed later, or
[x] No behavior was changed with this PR

Security

[ ] Security architect has reviewed the changes in this PR,
[ ] These changes are part of a larger initiative with a separate security review, or
[x] There are no security aspects to these changes

I verified the test passes with a Jenkinsfile modified to run it on all branches


cache authentication tokens

Thought after the latest login issue its probably best to cache the authentication tokens by default. Clearing would currently be manually or perhaps later via ignore flag.

Looks solid!


Incorrect sort order for years in sidebar

I'm using tg-archive 0.3.9 with a Telegram group which existed for a few years already.
When creating the static website with tg-archive --build, the generated sidebar is sorting the years in the sidebar incorrectly (see screenshot).

Whew, that's curious.

Have you made any changes to the template?
Is it a public group whose data can  be shared? If yes, could upload the data.sqlite file here? I can try and reproduce and debug the issue.

Have you made any changes to the template?

No, I haven't changed the template.

Is it a public group whose data can be shared? If yes, could upload the data.sqlite file here? I can try and reproduce and debug the issue.

The group is https://t.me/clickhouse_en and I think sharing the data is fine.
🔗 data.sqlite.gz

Found the issue. Please upgrade to v3.1.0

@knadh Thanks for the quick fix!
I can confirm that tg-archive 0.3.10 is indeed fixing the sorting issue:
https://tg-clickhouse-archive.netlify.app/clickhouse_en/


Rage Gain in Cat form

Describe the bug
Rage bar gains while in cat form, not losing rage when going into bear, inflating Feral dps logs
To Reproduce
Steps to reproduce the behavior:

Go to 'Feral druid(cat) sim -> Results -> Timeline'
Click on '....'
Scroll down to 'Start of fight, see rage gain while in cat, inflating dps'
See error

Expected behavior
Rage isn't generated while in catform.
Sim Links and Screenshots
If applicable, add links to your sim settings or screenshots to help explain your problem.
Picture of rage gain

Link to sim
https://wowsims.github.io/wotlk/feral_druid/#eJztU0FIVFEUfff+P+Pzpc0zIl4yI7+/CBHE9//ngwUxf1q5/NCiNoFShlCLYQiahGAKgnSlVIKfyE2LICgpGGpy0NCKWYyIoFEYhERMROQq3Nl7M4VIEEG16yzuve+e+859XO5jByl2gsReCDDEfhiCLOahADAGcBdgGnAJoAZwNEHBwgD6IIS8ZvaQ9rXYFYiH5wYuDuYo5dA+DSzBJ2dQfEzZ8eI8TlWQNfPNNbSN8mNkLfy+4t6lbEMTTbxUQnEzqYItFWymWCtfKaK42mGbqpowxh+o442kDUTJfi2jGE/acUVprQT/9ArFF9VHaS1WtfgjpTKRtI1FRZs8mtF29Im2b19rO1fSTReeoqimGFGJahHtVooSQvOEmYdrAF1jSP5jB4YpBY4CLXgIPUxQ5CBiFhwhxMoEsb6Ok4T0Z7J6G9rYLrr1A8BPCxiF4/IX6Pal50rX8VzPdaUjpe856uxJz5G+43S70lcVUuV+wjoI+mIW+fNZFHfeoxWtYOf8MsqXy1gDcwOaIjjMONtNR/ZZ229icWbSD/cM7fl1tQ7aj1cbfupZ3dPbI8YbIBES+zM210cwlwvaGsPYG+yPJjUq6UONzFL6wJk6VtNuGbO3Lg23hAPnTw1dGMydtVzfOibov16qjcvB35ApVBb+REf2dmUzv1tcH15hYj34fqOWVp/vG5lwpRg=
Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

Nope, this is just a quirk of the implementation currently, yes u gain rage in car, but it's correctly removed down to 0 or 10 with furor when shifting into bear


Situation Awareness: When editing attributes on phone devices, cannot get back to app

This behavior is exclusive to phone devices.
When the attributes of a feature are being added or updated in the full-screen attribute editor popup (only seen on phones), sometimes the window scrolls to the point of the header bar at the top disappearing.  Once this happens, there is no way to close the attribute editor window and return to the application.  Instead, the user must press 'back' or refresh the app to reload and get back to the application.
To repro, you must make the attribute editor popup window scroll down to the point of making the header bar disappear (can be difficult to reproduce).  Here are the repro steps on iPhone:

Open incident area editor popup (however you want – currently existing feature or new feature…doesn’t matter)
Expand the Category drop down and select anything from the dropdown (this is mainly to open the keyboard)
Select the Incident Type dropdown and select anything from the dropdown
Click Done from the keyboard
---Header bar will no longer be available---
Put the cursor back in the Incident Number field
---Header bar comes back---

Please note, the instructions in number 5 to bring the header bar back only work on iPhone.  On Android phone, even tapping to put the cursor back in the Incident Number field does not bring the header back.

Duplicate of issue 68 in devtopia.


injects htmlrewriter when missing

This change was originally discovered & written by @tylerflint in the @point/testing package but is better placed at the root of it's usage, which is here.
Most of the environments we use will provide an htmlrewriter, however, some do not, and in that case we use this addition.

Closing in favour of: https://github.com/qpoint-io/qpoint-htmlrewriter/pull/1


ci(aio): publish hipblas and Intel GPU images

Description
Adds tags for hipblas and Intel GPUs
Closes #1855

Only testable in master, merging


Working with an account on a webpage

Let me explain my request with an example.
The user will log into their Facebook account via WebView in my application.
The application will get images from the user's profile page. The app cant get any images without login. How is this possible? Cookie transfer or something like that?

Yes you can transfer the cookie - only question is if this is somewhat blocked by Facebook (e.g., by fingerprinting the used browser).
I'd recommend looking at the cookie container from AngleSharp.Io. It allows reading cookies from disk (or even writing them). So if you can transfer the cookie file then AngleSharp.Io could deal with it.

Do you know a site with sample codes or tutorials for AngleSharp? except the official docs


Validate nodes returns errors of last node only

Hi, I was using the KGX validator and I think the validate_nodes function returns errors of the last node.
https://github.com/NCATS-Tangerine/kgx/blob/0025e3f90b4ba73d14485fd5c9a3244176bd75cc/kgx/validator.py#L187

@YaphetKG Thanks for reporting this bug.
This should be fixed on master now.


Adaptive Neural Trees (ANT)

https://arxiv.org/pdf/1807.06699.pdf
details later

Hello! Do you know where can i see the code??

Closed in favor of being in #2302. We decided to keep all feature requests in one place.
Welcome to contribute this feature! Please re-open this issue (or post a comment if you are not a topic starter) if you are actively working on implementing this feature.

This was locked accidentally. I just unlocked it. We'd still welcome contributions related to this feature!


Incident reporting to incident management (via PagerDuty?)

eg Notify incident management that “We're having a P1 incident in [slack channel], FYI”

No need/Insufficient value


[stdlib] Inlinability audit: various builtins

Almost all of these were explicitly marked @inline(__always) already.

@swift-ci please test

Build failed
Swift Test OS X Platform
Git Sha - 05ab49240de1ba8f8f4c5185c498ae22e09c5d48

Build failed
Swift Test Linux Platform
Git Sha - 05ab49240de1ba8f8f4c5185c498ae22e09c5d48

@swift-ci please test

Build failed
Swift Test OS X Platform
Git Sha - 05ab49240de1ba8f8f4c5185c498ae22e09c5d48


2023-03-23 meeting minutes

Description
Add 2023-03-23 meeting minutes.
Please have a look @onebeyond/maintainers.
Related Issues
https://github.com/onebeyond/maintainers/issues/1
https://github.com/onebeyond/maintainers/issues/15
https://github.com/onebeyond/maintainers/issues/21
https://github.com/onebeyond/maintainers/issues/28
https://github.com/onebeyond/maintainers/issues/29
https://github.com/onebeyond/maintainers/issues/41
https://github.com/onebeyond/maintainers/issues/42
https://github.com/onebeyond/maintainers/issues/51
https://github.com/onebeyond/maintainers/issues/56
https://github.com/onebeyond/maintainers/issues/58
https://github.com/onebeyond/maintainers/issues/59
https://github.com/onebeyond/maintainers/issues/60
Side effects
Close #57

@inigomarquinez @carpasse  I made some minor changes to improve the web format 416d655

Added youtube link and Iframe
Updated Present list
Improve formatting for web version


Fix result enum registration

Signed-off-by: Bogdan Vaneev<EMAIL_ADDRESS>

Description of the Change

split macro on 2: declare (in hpp) and define (in cpp)
update docs

Benefits

One macro call per enum in hpp, one call per enum in cpp
Possible Drawbacks

None

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@507f54b). Click here to learn what that means.
The diff coverage is 50%.

@@            Coverage Diff            @@
##             master      #50   +/-   ##
=========================================
  Coverage          ?   75.78%
=========================================
  Files             ?       76
  Lines             ?     2870
  Branches          ?        0
=========================================
  Hits              ?     2175
  Misses            ?      695
  Partials          ?        0

Impacted Files
Coverage Δ

core/libp2p/multi/multiaddress.hpp
100% <ø> (ø)

deps/outcome/outcome/outcome-register.hpp
71.42% <0%> (ø)

core/libp2p/multi/multiaddress.cpp
70.09% <0%> (ø)

test/deps/outcome_test.cpp
72.09% <100%> (ø)

Continue to review full report at Codecov.

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


drivers: i3c: common: ENTDAA after request is not possible after RSTDAA

Describe the bug
If Reset command and reset DAA are issued, cannot request dynamic assignment (DAA) again which per spec shall be possible

After I3C bus initialization occurs and DAA takes place, if application sends a reset CCC command and reset DAA CCC, following request dynamic assignment (DAA) is aborted with error indicating the original dynamic address was already assigned.
0:01:17.113,000]  i3c: PID 0x0208006c100b already has dynamic address (0x09) assigned
[00:01:17.113,000]  npcx_i3c: npcx_i3c_do_daa: Assign new DA error
To Reproduce

Simple app performing the following sequence in main:
i3c_ccc_do_rstact_all(i3c_dev, I3C_CCC_RSTACT_RESET_WHOLE_TARGET)
i3c_ccc_do_rstdaa_all(i3c_dev)
LOG_WRN("Perform DAA");
i3c_do_daa(i3c_dev);
west build -c -p always -b npcx4m8f_evb
Expected behavior

After RSTDAA is issue a ENTDAA command shall allow to performed.
Impact

This is against I3C spec expectations. (Section <IP_ADDRESS>.3)
Logs and console output

[00:01:17.112,000]  npcx_i3c: npcx_i3c_do_ccc: Enter npcx_i3c_do_ccc
[00:01:17.112,000]  npcx_i3c: npcx_i3c_do_ccc: CCC[0x2a]
[00:01:17.112,000]  npcx_i3c: npcx_i3c_send_request: npcx_i3c_send_request fc41
[00:01:17.112,000]  npcx_i3c: npcx_i3c_send_request: npcx_i3c_send_request 2
[00:01:17.112,000]  npcx_i3c: npcx_i3c_do_ccc: Enter npcx_i3c_do_ccc
[00:01:17.112,000]  npcx_i3c: npcx_i3c_do_ccc: CCC[0x06]
[00:01:17.112,000]  npcx_i3c: npcx_i3c_send_request: npcx_i3c_send_request fc41
[00:01:17.112,000]  npcx_i3c: npcx_i3c_send_request: npcx_i3c_send_request 2
[00:01:17.112,000]  npcx_i3c: npcx_i3c_do_ccc: Enter npcx_i3c_do_ccc
[00:01:17.112,000]  npcx_i3c: npcx_i3c_do_ccc: CCC[0x01]
[00:01:17.112,000]  npcx_i3c: npcx_i3c_send_request: npcx_i3c_send_request fc41
[00:01:17.113,000]  npcx_i3c: npcx_i3c_send_request: npcx_i3c_send_request 2
[00:01:17.113,000]  i3c_m: Perform DAA
[00:01:17.113,000]  npcx_i3c: npcx_i3c_do_daa: npcx_i3c_do_daa
[00:01:17.113,000]  npcx_i3c: npcx_i3c_do_daa: DAA: ENTDAA
[00:01:17.113,000]  npcx_i3c: npcx_i3c_request_daa: npcx_i3c_request_daa
[00:01:17.113,000]  npcx_i3c: npcx_i3c_send_request: npcx_i3c_send_request 44
[00:01:17.113,000]  npcx_i3c: npcx_i3c_do_daa: DAA: Rcvd PID 0x0208006c100b
[00:01:17.113,000]  i3c: PID 0x0208006c100b already has dynamic address (0x09) assigned
[00:01:17.113,000]  npcx_i3c: npcx_i3c_do_daa: Assign new DA error
[00:01:17.113,000]  npcx_i3c: npcx_i3c_send_request: npcx_i3c_send_request 2
[00:01:17.113,000]  i3c_m: DAA -22
Environment (please complete the following information):

OS: (e.g. Linux, MacOS, Windows)
Toolchain (e.g Zephyr SDK, ...)
Commit SHA or Version used

Additional context

@XenuIsWatching Is it something that can be reproduced on your side? I don't have a I3C setup right now so I cannot even test any changes.

I do not have this platform...
but i think I know what is going on. You need to be manually setting all the i3c devices through the i3c_device_desc dynamic_addr to 0 after you do the RSTDAA.

@XenuIsWatching  I believe this is not platform specific since the check that throws the error is i3c_common code.
Also, is your suggestion that the client's driver code manipulates the i3c controller driver data directly?
Wouldn't be better that this is handled in I3C bus driver?  I created this draft based on 2nd option, this works, but seems this particular scenario (RSTDAA, ENTDAA) needs more changes e.g. with said PR change the ENTDAA succeeds.
https://github.com/zephyrproject-rtos/zephyr/pull/78784
00:00:12.045,000] <dbg> npcx_i3c: npcx_i3c_do_ccc: Enter npcx_i3c_do_ccc
[00:00:12.045,000] <dbg> npcx_i3c: npcx_i3c_do_ccc: CCC[0x2a]
[00:00:12.045,000] <dbg> npcx_i3c: npcx_i3c_send_request: npcx_i3c_send_request fc41
[00:00:12.045,000] <dbg> npcx_i3c: npcx_i3c_send_request: npcx_i3c_send_request 2
[00:00:12.045,000] <wrn> i3c: Reset dynamic address for device i3c-dev2@208006c100b
[00:00:12.045,000] <dbg> npcx_i3c: npcx_i3c_do_ccc: Enter npcx_i3c_do_ccc
[00:00:12.045,000] <dbg> npcx_i3c: npcx_i3c_do_ccc: CCC[0x06]
[00:00:12.045,000] <dbg> npcx_i3c: npcx_i3c_send_request: npcx_i3c_send_request fc41
[00:00:12.045,000] <dbg> npcx_i3c: npcx_i3c_send_request: npcx_i3c_send_request 2
[00:00:12.046,000] <dbg> npcx_i3c: npcx_i3c_do_ccc: Enter npcx_i3c_do_ccc
[00:00:12.046,000] <dbg> npcx_i3c: npcx_i3c_do_ccc: CCC[0x01]
[00:00:12.046,000] <dbg> npcx_i3c: npcx_i3c_send_request: npcx_i3c_send_request fc41
[00:00:12.046,000] <dbg> npcx_i3c: npcx_i3c_send_request: npcx_i3c_send_request 2
[00:00:12.046,000] <wrn> i3c_m: Perform DAA
[00:00:12.046,000] <dbg> npcx_i3c: npcx_i3c_do_daa: DAA: ENTDAA
[00:00:12.046,000] <dbg> npcx_i3c: npcx_i3c_request_daa: npcx_i3c_request_daa
[00:00:12.046,000] <dbg> npcx_i3c: npcx_i3c_send_request: npcx_i3c_send_request 44
[00:00:12.046,000] <dbg> npcx_i3c: npcx_i3c_do_daa: DAA: Rcvd PID 0x0208006c100b
[00:00:12.046,000] <dbg> npcx_i3c: npcx_i3c_request_daa: npcx_i3c_request_daa
[00:00:12.046,000] <dbg> npcx_i3c: npcx_i3c_send_request: npcx_i3c_send_request 44
[00:00:12.046,000] <dbg> npcx_i3c: npcx_i3c_do_daa: PID 0x0208006c100b assigned dynamic address 0x0a
[00:00:12.046,000] <wrn> i3c_m: Find I3C device using PID
[00:00:12.046,000] <wrn> i3c_m: Dynamic addr a

But when using the I3C descriptor it looks like some descriptor data is stale.

[00:00:12.048,000] <inf> i3c_m: Max Read Length: 11450
[00:00:12.048,000] <dbg> npcx_i3c: npcx_i3c_ibi_enable: npcx_i3c_ibi_enable bcr0 dcr0
[00:00:12.048,000] <err> npcx_i3c: npcx_i3c_ibi_enable: device is not ibi capable

Even though I3C device is IBI capable


Can not move items when using web app in Safari

Hi,
I cannot move items around when I use the web app in Safari.
Steps to reproduce:

go to https://app.keeweb.info or https://beta.keeweb.info or on a self hosted instance
open the demo database (or an existing one)
choose an entry from the password list, for example the fourth, "Facebook"
select it, drag and drop it on a new group, for example "Home"

"Home" doesn't turn blue and the item is not moved to the new group.
KeeWeb v1.1.4 (08087c3, 2016-04-21)
Environment: web
User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_10_5) AppleWebKit/601.6.17 (KHTML, like Gecko) Version/9.1.1 Safari/601.6.17

Thanks for your attention
Matteo

Oh I see now. Thanks for adding steps, now it's clear what's not working. I thought you mentioned drag-drop attachments or files. Entries drag-drop is not working, will be fixed soon.

Thanks!
However, it does work in other browsers on the same computer and in the desktop app.

Deployed to beta.keeweb.info, will be available in the next release

Thanks, I can confirm that it works!


jobs from single file

allow setting a folder where to load jobs from. not just from config php. default to snippets.

this could also mean to register the default jobs as snippets to be more consistent.

others could PR their snippets then. related to cli.

from classes is better. will be in v2


Virtual Media native library cannot be loaded

Hello,
while trying to reinstall  a  Dell R610 using a virtual CD ISO, virtual media menu is broken with following error:

Would be very  useful to use this feature . Not necessary to be a bug imho.
P.S. Forgot to mention that was testing on OSX 10.12.6.
Thank you.

Sorry for the long answer. I'll dig into it.

Just compare browser generated .jnlp with the drac-kvm version and complement the difference.
For example for drac v6, add to the drac6.go the next resource section:

 <resources os="Mac OS X" arch="x86_64">
    <nativelib href="https://{{ .Host }}:443/software/avctKVMIOMac64.jar" download="eager"/>
   <nativelib href="https://{{ .Host }}:443/software/avctVMMac64.jar" download="eager"/>
  </resources>


Update how-to-train-distributed-gpu.md

update the documentation since ddp has a bug in azure-ml

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

Hi @sdgilley - Could you review or reassign this pull request? The listed author isn't in the open-source org, so they don't appear as an option to assign the PR or request a review. Thanks!

@dkmiller Thanks for your SME review! We need the content team or a PM to sign off to merge in this repository.
@sdgilley Could you review for the content team? Thanks.

#sign-off

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

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

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

Thanks, all. Merging now.


Make the future::retry functions generic over the sleeper

I realized that the conditional compilation for fn sleep() -> Sleep in async.rs could be represented as a trait, so I did this. I'm not too sure about the names.
This allows for accessing both the async_std and tokio versions of the retry functions if so desired, and also lets you implement your own sleeper if you use some other runtime.

Hello @coolreader18 ! Thanks for your contribution again.
I like that sleeper direction, it's better for testing purposes too.
Let me merge your PR but I won't issue a release right away. This will deserve a 0.3 version and I might bundle in some more changes into a release (mainly more deps updates and simplifications).

Hello @coolreader18 ! Thanks for your contribution again.
I like that sleeper direction, it's better for testing purposes too.
Let me merge your PR but I won't issue a release right away. This will deserve a 0.3 version and I might bundle in some more changes into a release (mainly more deps updates and simplifications).

Alright, sounds good!

Alright, sounds good!


version 3.32 download contains 3.31

the .zip package of 3.32 contains 3.31 instead.

I think the about page is broken only. The changes are included if you have the new option to disable the auto refresh function.

The binary exe in the 3.32 Release was modified on Jan. 14th.


[Event Request] or [External] UpdateTypeText in Sales/Purch. * Subforms

I would like to be able to override default type in OnNewRecord on Sales/Purch. * Subforms.
But I'm not able to call UpdateTypeText.
Is it possible to make varibel TypeAsText  or procedure UpdateTypeText accessible in event OnNewRecord. Or make event in OnNewRecord in pages, something like this (so I can set type before UpdateTypeText is called):

Hi.
I can see that this has been added for sales * subforms, but not for purch * subforms. Can this be added to the purchase module aswell?
Sales Invoice Subform:

Purch. Invoice Subform:


Put back block_area for isochrone & spt

This feature got lost when the block area handling was moved out of createWeighting.

And while we are at it: Currently we do not get an error if block_area is requested (and ignored) when CH is used.


Choice lists

Support for cells containing multiple tags.

See https://support.getgrist.com/newsletters/2021-05/#choice-lists


Combobox menu popover appears empty, list items won't show up

🐛 Bug Report
After using combobox with example usage from spectrum docs the menu is empty. List of item component just don't appear at all. But you can still select them with keyboard. Console don't show any errors

🤔 Expected Behavior
Combobox is showing menu items when clicked on arrow key.
😯 Current Behavior
when clicking on combobox the menu popover appears empty when items are set. Work same for controlled and uncontrolled state.
💻 Code Sample
Codesandbox
🌍 Your Environment

Software
Version(s)

react-spectrum
3.26.0

Browser
Firefox

Operating System
Windows

Looks like this is due to React Strict Mode. You can fix it by removing React.StrictMode in index.tsx: https://codesandbox.io/s/jolly-framework-8lios9?file=/src/index.tsx
Issues with React Strict Mode are actively being worked on. We have an umbrella issue for those here: https://github.com/adobe/react-spectrum/issues/779

Closing in favor of umbrella.


Added bkeymaps page from old wiki

You may find this template handy: https://github.com/kisslinux/wiki/blob/master/help/page-template.txt

I don't think /software/ is the right place for this (maybe /console/?).  It might be better to rename the page to keyboard-layout. So @/console/keyboard-layout? Maybe even tty/keyboard-layout.
What do you think?

console sounds good. We can add the page about console fonts there too.

Are there any new console fonts to add to the list?

bkeymaps.txt exists twice in this PR.


Minimum types proposal

Initial proposal for a minimum list of supported types.

int64
unix epoch (same as int64 with some additional functions)
string
record (contains a data set)
table (a set of records)
module (a set of tables)
application (a set of modules)

What about a enum type? Maybe it is not so 'minimal', but can be useful (like a string, but with some constraints)

I don't think enum is for now, we need to define what is "essential" first, but I think the idea is interesting, open an issue for the enum to discuss how to implement it.
For now it's easy because the types I proposed are either mapable directly to Go or just structures.

I Agree, about open an issue for the enum, I'll wait until there is some implementation so we'll have something to discuss about.


ARC-55 On-Chain Msig Apps

This is an initial draft for the on-chain multisignature application proposal. (If someone can think of a good name for this, please share)
The TL;DR is it's a smart contract design you and your friends can use to help with signing multisig transactions.
The smart contract itself does not perform any multsignature operations, but rather it facilitates the distribution of the unsigned transactions between the multisignature constituents, and allowing them to store their signature on-chain.
This should make using multisig accounts and signing transactions much faster and simpler.

We should also add the arc55_ prefix to each method

@nullun, thanks a lot for this proposal, I'm really looking forward to interoperable multisigs across wallets. Here some feedback:

Wallet providers need a way to discover pending signature requests for a user. For mobile wallets, it'd additionally be useful if a server could easily detect pending signature requests and send out push notifications to its users. In the current design, each msig has its dedicated application, hence a server would have to (a) find out what applications are ARC-55 msig apps and (b) monitor all those apps to see if there are any pending signature requests. There could be a registry where all msig apps opt in to announce themselves, which would address point (a), but not really point (b). @joe-p suggested turning this app into a monolith, which would address both points but require significant changes to the code, I guess.

Currently, only the admin account (i.e., the creator account) can add transactions and ask for them to be signed. I think every signer in a multisig should be treated equally and have the ability to propose and sign transactions.

In the current design, it seems that only one transaction group can be added and signed at the same time. I think it'd be nice if there could be multiple pending transaction groups at the same time. There can be cases where multiple signers create transactions at the same time, or where one signer creates multiple transaction groups at the same time.

Should the app be updateable? Assuming a wallet provider indexes all the ARC-55 msig apps and one deployed app suddenly changes it may break all kinds of code. I don't really see the value in having an updateable contract for the msig application, but I'm happy to be convinced otherwise:)

Why can the threshold in the app be changed? Is an intended use case that the same msig application is reused for different msig configurations?

Adding to the previous point, it seems accounts can also be changed at any time. This means, the msig configuration can change over time. That makes the application very flexible, but more difficult to deal with for wallet providers. If wallets track msig applications on their backend, they'd also have to keep track of changes to the accounts/thresholds in the applications.

What kind of limitations are there on the transactions that can be submitted? I'm wondering if there are any transaction groups that are valid but that could not be submitted through ARC-55. I'm thinking of transaction groups that create an app with lots of extra program pages and hence get fairly big. Currently, hardware ledger devices cannot sign create-app txns that are too big, which is fairly annoying in my opinion. The next most secure way of signing them would be with a multisig accounts, hence my question about what kinds of limitations there are.

I really appreciate that you gave a breakdown of the costs that occur for the various operations (this is super useful for a wallet to show to its users). Could there be further details about whether these costs are recoverable or not and when they're recoverable? What I mean is that certain operations add to the minimum balance (e.g., adding a transaction) and it would be nice to explicitly state when the minimum balance is reduced again (e.g., when a transaction is removed again from the app).

Speaking of minimum balance, who pays for the app's minimum balance requirements for the boxes? Probably the user that posts the transactions?

I know this is a lot and not everything may make it into the final ARC, but this just came to my mind when I went through the ARC. In any case, great work!

Hey @k13n, thank you for this! I appreciate the thorough review and feedback. I'll provide a response to each section below.

Wallet providers need a way to discover pending signature requests for a user. For mobile wallets, it'd additionally be useful if a server could easily detect pending signature requests and send out push notifications to its users. In the current design, each msig has its dedicated application, hence a server would have to (a) find out what applications are ARC-55 msig apps and (b) monitor all those apps to see if there are any pending signature requests. There could be a registry where all msig apps opt in to announce themselves, which would address point (a), but not really point (b). @joe-p suggested turning this app into a monolith, which would address both points but require significant changes to the code, I guess.

@joe-p and I discussed this a little yesterday after the meeting. My main goal with this ARC is to create a common interface that people can use either as it is (using the reference contract), modify the internal logic to meet their own needs, or even implement into unrelated contracts which themselves would now support ARC55 calling conventions. I strongly dislike the idea of us deploying a single version that we deem fit for everyone, just because it would make a centralised notification system easier.
Whilst I do agree that a push notification would be nice, I don't feel it's necessary for the implementation of this ARC. Primarily this ARC attempts to address the struggles with sharing transaction and signature data, and assumes the participants of a multisig are already well enough connected, that sending a text/slack/discord message doesn't need improving.
I imagined this working as followed:

A group of people agree they need a multisig
One of the members deploys the msig app (paying the MBR costs themselves)
The msig appID is then shared with the rest of the party, so they can each add it to their ARC55 supported wallet/app they use
When a transaction is added, either a notification is shown to the user because their wallet/app is listening locally to the state of the app which they've added. Or the person who uploaded the transaction tells the group via any existing communication channels.

Currently, only the admin account (i.e., the creator account) can add transactions and ask for them to be signed. I think every signer in a multisig should be treated equally and have the ability to propose and sign transactions.

I fully agree with this after hearing the feedback. I'll make my reference implementation default to allowing anyone who is part of the multisig able to add transactions. Although touching on a point I made above, if someone decides their requirements only warrant one account to modify it, they should be fully allowed to do so as long as the interface remains the same. If we deployed one single implementation which all wallets look at, we now lose out on reusing the same interfaces for something that should still adhere to the standard and work (albeit gracefully handling it when someone who doesn't have permission to upload a transaction attempts to).

In the current design, it seems that only one transaction group can be added and signed at the same time. I think it'd be nice if there could be multiple pending transaction groups at the same time. There can be cases where multiple signers create transactions at the same time, or where one signer creates multiple transaction groups at the same time.

Yeah, this is definitely something I hadn't thought about, purely because I've never personally experienced or seen a multisig party do it. There isn't actually anything stopping you from uploading 16 individual transactions, or even 2 groups of 8 transactions, etc and then displaying them as separate pending groups/transactions. However the reference definitely has a hardcoded limit of 16 transactions being stored.
I like this idea a lot though. I'll take a look and see how much effort would be involved reworking the box storage. Off the top of my head, I'm thinking we could add an incrementing nonce for every transaction/group that gets uploaded. So the box names could be something like 0txn0, 0txn1, 0txn2 for the first group, then 1txn0, 1txn2, for the next upload. Although this would require signatures to also be placed in boxes, since they could now provide more than 16 signatures at any one time.
This would definitely require some stronger "garbage collection" incentives/wording though, to encourage users to delete historic transactions.

Should the app be updateable? Assuming a wallet provider indexes all the ARC-55 msig apps and one deployed app suddenly changes it may break all kinds of code. I don't really see the value in having an updateable contract for the msig application, but I'm happy to be convinced otherwise:)

I don't think there should be an update feature defined in the ARC. It should either be deploy when you need one, or destroy when it's no longer required. But again, if someone wants to include an update feature for their own version, that's on them. As long as the ARC55 interface remains the same then they can use any ARC55-supported wallet/app to use it.

Why can the threshold in the app be changed? Is an intended use case that the same msig application is reused for different msig configurations?
Adding to the previous point, it seems accounts can also be changed at any time. This means, the msig configuration can change over time. That makes the application very flexible, but more difficult to deal with for wallet providers. If wallets track msig applications on their backend, they'd also have to keep track of changes to the accounts/thresholds in the applications.

Great points! When I was originally testing this, it was easier for me to just update the thresholds and accounts on an existing deployment. But I could very well see these parameters as something you define during the deployment and are then locked. This way you'd have to deploy a new instance if you want different threshold/accounts? I think this makes the ARC a lot simpler to understand and clearer.

What kind of limitations are there on the transactions that can be submitted? I'm wondering if there are any transaction groups that are valid but that could not be submitted through ARC-55. I'm thinking of transaction groups that create an app with lots of extra program pages and hence get fairly big. Currently, hardware ledger devices cannot sign create-app txns that are too big, which is fairly annoying in my opinion. The next most secure way of signing them would be with a multisig accounts, hence my question about what kinds of limitations there are.

Yes, this is absolutely the biggest "issue" in my opinion, but I decided for most multisig use-cases where it's just sending funds this isn't a blocker.
An example where a valid transaction wouldn't be possible in this ARC would be where the group contains transactions that are not from the multisig address, such as an atomic swap between two accounts (the multisig and any other account). The other account in this scenario would be unable to provide their signature to the msig app, unless they were the one who reconstructed and it after everyone else had provided their signatures, and then added their own.
A discussion I had internally lead us down using this same smart contract design, but for more generic situations, where anyone can provide their signature for a transaction. Allowing for true atomic swaps between different individuals, using just the chain as a transport layer. This however falls outside the scope of this ARC.

I really appreciate that you gave a breakdown of the costs that occur for the various operations (this is super useful for a wallet to show to its users). Could there be further details about whether these costs are recoverable or not and when they're recoverable? What I mean is that certain operations add to the minimum balance (e.g., adding a transaction) and it would be nice to explicitly state when the minimum balance is reduced again (e.g., when a transaction is removed again from the app).
Speaking of minimum balance, who pays for the app's minimum balance requirements for the boxes? Probably the user that posts the transactions?

Excellent idea. Whilst I'm changing who can add transactions, I'll make sure it's clear that the uploader of the transactions is responsible for adding the MBR to the application address. Although I don't see an elegant solution to recover this back to the them. Rather it's only recoverable by the creator upon removing all the boxes and destroying the msig app? Unless there's a nice simple alternative?

I know this is a lot and not everything may make it into the final ARC, but this just came to my mind when I went through the ARC. In any case, great work!

Thank you so much again for taking the time to go through it! I'm sorry my response is equally as long. Please let me know if there's anything you strongly disagree with, we're definitely looking at this from different perspectives, so I need to be open to at least think more about them.

@joe-p and I discussed this a little yesterday after the meeting. My main goal with this ARC is to create a common interface that people can use either as it is (using the reference contract), modify the internal logic to meet their own needs, or even implement into unrelated contracts which themselves would now support ARC55 calling conventions. I strongly dislike the idea of us deploying a single version that we deem fit for everyone, just because it would make a centralised notification system easier.
Whilst I do agree that a push notification would be nice, I don't feel it's necessary for the implementation of this ARC. Primarily this ARC attempts to address the struggles with sharing transaction and signature data, and assumes the participants of a multisig are already well enough connected, that sending a text/slack/discord message doesn't need improving.
I imagined this working as followed:

A group of people agree they need a multisig
One of the members deploys the msig app (paying the MBR costs themselves)
The msig appID is then shared with the rest of the party, so they can each add it to their ARC55 supported wallet/app they use
When a transaction is added, either a notification is shown to the user because their wallet/app is listening locally to the state of the app which they've added. Or the person who uploaded the transaction tells the group via any existing communication channels.

Thanks for this context. I see the value in having a common set of calling conventions for msigs with lots of different implementations and a lot of flexibility, but this makes it really difficult to integrate smoothly and provide a good interface for.
It seems there are two issues:

App discovery: how can an integrator detect that an account is opted into a ARC-55 app? When a user imports an account into a wallet, I'd like to be able to tell that a particular app is compatible with ARC-55 and that I should be listening for transactions.
Transaction discovery: how can an integrator detect that there are pending signature requests? With the current approach it's easy to check in the frontend, but hard to check in the backend. From user feedback I can tell you that automatically detecting pending transactions and sending push notifications to users is a highly appreciated feature and I wouldn't want to lose the ability to do that.

It'd be great to hear @yigitguler's opinion on this and how they would potentially integrate ARC-55 in Pera Wallet.

I like this idea a lot though. I'll take a look and see how much effort would be involved reworking the box storage. Off the top of my head, I'm thinking we could add an incrementing nonce for every transaction/group that gets uploaded. So the box names could be something like 0txn0, 0txn1, 0txn2 for the first group, then 1txn0, 1txn2, for the next upload. Although this would require signatures to also be placed in boxes, since they could now provide more than 16 signatures at any one time.
This would definitely require some stronger "garbage collection" incentives/wording though, to encourage users to delete historic transactions.

Great, thanks for looking into this! I kind of liked the idea that the signatures are stored in the signer's local state, just because then the minimum balance requirement is already taken care of when the user opts in. But I think having boxes to support more than 16 transactions is important and makes sense. Perhaps simply whoever creates the transactions should make sure that the msig app has sufficient minimum balance for all signatures. But it'd be good for interoperability that there's a agreed upon way of doing it. If one integrator thinks the transaction creator should pay the min balance and another integrator thinks the signers should pay the min balance, we're going to have problems.

Yes, this is absolutely the biggest "issue" in my opinion, but I decided for most multisig use-cases where it's just sending funds this isn't a blocker.
An example where a valid transaction wouldn't be possible in this ARC would be where the group contains transactions that are not from the multisig address, such as an atomic swap between two accounts (the multisig and any other account). The other account in this scenario would be unable to provide their signature to the msig app, unless they were the one who reconstructed and it after everyone else had provided their signatures, and then added their own.
A discussion I had internally lead us down using this same smart contract design, but for more generic situations, where anyone can provide their signature for a transaction. Allowing for true atomic swaps between different individuals, using just the chain as a transport layer. This however falls outside the scope of this ARC.

I think it's okay that not every conceivable transaction group can be represented in ARC-55, but it'd be great to make these things explicit in the ARC such that it's very clear what is and what isn't possible.

Thanks for this context. I see the value in having a common set of calling conventions for msigs with lots of different implementations and a lot of flexibility, but this makes it really difficult to integrate smoothly and provide a good interface for.
It seems there are two issues:
* **App discovery**: how can an integrator detect that an account is opted into a ARC-55 app? When a user imports an account into a wallet, I'd like to be able to tell that a particular app is compatible with ARC-55 and that I should be listening for transactions.

* **Transaction discovery**: how can an integrator detect that there are pending signature requests? With the current approach it's easy to check in the frontend, but hard to check in the backend. From user feedback I can tell you that automatically detecting pending transactions and sending push notifications to users is a highly appreciated feature and I wouldn't want to lose the ability to do that.

It'd be great to hear @yigitguler's opinion on this and how they would potentially integrate ARC-55 in Pera Wallet.

Yeah, app discovery for specifically supported ARCs is definitely a wider issue than just ARC55. Any smart contract interface that's proposed that doesn't offer a single existing deployed solution will have the same difficulties. I recall a number of discussions in the past about trying to solve it. One that comes to mind is embedded the ARC number in the smart contract, for example appending intcblock 4 55 at the very end would indicate supporting ARC4 and ARC55 To my knowledge there hasn't been anything concrete proposed on this topic. I think I'd still personally prefer to be adding the appID manually myself though, rather than being sent notifications because someone is maliciously adding my account to a msig app that I have nothing to do with.
Transaction discovery is possibly easier to solve, with the inclusion of ARC-28: Algorand Event Log Spec. If each ARC55 implementation requires logging TransactionAdded(bytes) when a new transaction was added, and SignatureAdded(address), along with ThresholdMet(uint64) or something like this, then it just requires a single monitoring service to be keeping track of events in each block. Maybe a specific Conduit plugin could be made to specifically track every single event. I'm using this ARC for another contract I'm working on, and have been implementing a script that monitors the chain for events as they happen. I must point out the current issue related to this though.

Great, thanks for looking into this! I kind of liked the idea that the signatures are stored in the signer's local state, just because then the minimum balance requirement is already taken care of when the user opts in. But I think having boxes to support more than 16 transactions is important and makes sense. Perhaps simply whoever creates the transactions should make sure that the msig app has sufficient minimum balance for all signatures. But it'd be good for interoperability that there's a agreed upon way of doing it. If one integrator thinks the transaction creator should pay the min balance and another integrator thinks the signers should pay the min balance, we're going to have problems.

That's a good idea. I'll see which flow makes most sense and is easiest for integrators to use. Ideally you're not having to calculating the cost for each interaction, so one-and-done feels simpler.

I think it's okay that not every conceivable transaction group can be represented in ARC-55, but it'd be great to make these things explicit in the ARC such that it's very clear what is and what isn't possible.

100% I'll add this to the ARC.


Typo modififedValue in ADHybridHealthService

Should be "modified" in
https://github.com/Azure/azure-rest-api-specs/blob/bd2aca046b487c2ae2f13cc91d26c5f8aae93571/specification/adhybridhealthservice/resource-manager/Microsoft.ADHybridHealthService/stable/2014-01-01/ADHybridHealthService.json#L5381-L5384

@yungezz this is still an issue, please reopen


bug fix #54

I intended to fix #54 on this repo first then merge to origin.  But failed to push to origin repo with VSCode.  I'd like to try again with #55.


Bug: The local filesystem tree is doubling up on entries

Mentioned in a video on YouTube. The content of the local file view is doubling up the entries for some reason:

Weird. My initial suspicion was something relating to the recent changes to DirectoryTree and perhaps the use of a filter; so I removed the filter for a moment, ran up the application, and it fixed the issue. So I put the filer back, ran up the application, and...

So now I can't actually recreate the issue any more!

Recreating this is a bit hit-and-miss. On my own M2 Pro Mac Mini I can generally recreate this if I run the pipx-installed version, but if I run a "local" copy from the repo I can never reproduce it.
I do, however, have a hypothesis as to what might be going on. The recent changes to DirectoryTree in Textual itself, to load directories in the background, does have a possible window of opportunity whereby the root of the tree could end up getting loaded twice, in an additive way. I'm struggling to think of a way of forcing that to happen, but I think there's a chance it could and it would explain what I'm seeing here.

Assuming fix. Unable to reproduce with latest Textual.


It is compatible with vue3?

When I try to install, this appears in the console, despite having vue version 3.2.37
# npm resolution error report

While resolving<EMAIL_ADDRESS>Found<EMAIL_ADDRESS>node_modules/vue
<EMAIL_ADDRESS>from the root project

Could not resolve dependency:
peer<EMAIL_ADDRESS>from<EMAIL_ADDRESS>node_modules/vue-confirm-dialog
  vue-confirm-dialog@"*" from the root project

Fix the upstream dependency conflict, or retry
this command with --force or --legacy-peer-deps
to accept an incorrect (and potentially broken) dependency resolution.

i have fixed it, it was my mistake because there was missing the letters "js" in the name of the package i was trying to install!


feat(control): add optional callbacks

This PR adds optional callbacks to the Control component for when the buttons are clicked.
I also added the callback descriptions to the README.

thanks!


"clinfo" can't open /dev/mem but "sudo clinfo" worked

I installed VC4CL and try to run "clinfo" ,its show like these:
""
[VC4CL] can't open /dev/mem
[VC4CL] This program should be run as root. Try prefixing command with: sudo
terminate called after throwing an instance of 'std::system_error'
what():  Failed to open /dev/mem: Permission denied
Aborted
""
When i try to run "sudo clinfo", its OK.
When i test with PlaidML by "plaidml-setup", the same error:
""
[VC4CL] can't open /dev/mem
[VC4CL] This program should be run as root. Try prefixing command with: sudo
terminate called after throwing an instance of 'std::system_error'
what():  Failed to open /dev/mem: Permission denied
Aborted
""
When i test with PlaidML by "sudo plaidml-setup", its cannot found device:
""
No supported devices found. Run 'clinfo' and file an issue containing the full output.
""
Anyone can help me.
Thanks.

When i use PlaidML by plaidml-setup command its show
 [VC4CL] can't open /dev/mem [VC4CL] This program should be run as root. Try prefixing command with: sudo terminate called after throwing an instance of 'std::system_error' what(): Failed to open /dev/mem: Permission denied Aborted
but when i try "sudo plaidml-setup'  like  your suggest, , its cannot found device:
No supported devices found. Run 'clinfo' and file an issue containing the full output.

I don't know why the device is not found. Did you run both commands with the same environment variables (e.g. LD_LIBRARYPATH)?

So I ran into the same issue. After a quick peek at the plaidml setup code (mostly looking around what is currently line 49 in plaidml_setup.py) it looks like this is the message that happens when at least one device is found, but no supported devices are found. I think that what is missing is a line in their configuration file, i.e. something needs to be added in experimental.json.

I also filed an issue with PlaidML (#214)

@doe300

I don't know why the device is not found. Did you run both commands with the same environment variables (e.g. LD_LIBRARYPATH)?

I set same LD_LIBRARY_PATH,but result still the same.
@credman0

So I ran into the same issue. After a quick peek at the plaidml setup code (mostly looking around what is currently line 49 in plaidml_setup.py) it looks like this is the message that happens when at least one device is found, but no supported devices are found. I think that what is missing is a line in their configuration file, i.e. something needs to be added in experimental.json. But that is as far as I got because even after reading the descriptions of the GPU I just don't know enough to be sure which values are supposed to be the vector size, etc.

Its may miss config of support VC4.

I will try to config experimental.json of PlaidML.
@doe300 , @credman0
Thanks you for yours suuport.


Bookmarks

Hello, just now,  I just found a trick to be able to access all the sites you want, just activate the google service in settings/enable/disable services and voila, you have the google search engine and you can access all the desired websites!! Thank you very much Mr Alex :)
so, since we use several websites, it will be great to add an "add bookmark" feature so that we can save and access our streaming websites quickly and easily, thank you very much !

I can work on adding custom services to the menu without having to edit the config file.

Great ! Thank you so much ! 💯 🥇


Death Summary is covering chat

Mitigated in 9441280c794835c4215e5eaf76bd99fb34b0e4f7 and 1e3bc5cd8fe87bc679b46bf621ceee7831b9ad6a.  However, I am going to leave this open for now and review the changes in the next RC.


Update dependencies & remove unused futures crate

metrics:  https://github.com/metrics-rs/metrics/blob/main/metrics/CHANGELOG.md#0230---2024-05-27
metrics-exporter-prometheus: https://github.com/metrics-rs/metrics/blob/main/metrics-exporter-prometheus/CHANGELOG.md#0151---2024-06-24
Didn't dare to update matchit, there seem to be some breaking changes between 0.7 and 0.8

Thanks, but the upgrade is already in #52, which is very close to being merged. The futures dependency is not removed in that PR, but noted, thanks again.


Possible typo at Export Ordinal Table description

Summary: suspecting typo at description of how to access the export directory export address table.
Details:
At url https://docs.microsoft.com/en-us/windows/desktop/debug/pe-format
Under "Export Ordinal Table"
Quote:
i = Search_ExportNamePointerTable (ExportName);
ordinal = ExportOrdinalTable [i];
SymbolRVA = ExportAddressTable [ordinal - OrdinalBase];
End of Quote.
I use Visual Studio 2017 IDE. I linked the code using visual studio 2017 compiler.
The export dir ordinal base equals 1.
According to the above description, the RVA of the symbol should equal ExportAddressTable [ordinal - 1] but it equals ExportAddressTable [ordinal]

@ginami Can you take a look at this issue?

Hi @lshacham -- thank you for letting us know. We'll work on getting the page reviewed and updated.


Profiles to test the positioning procedure in the Arctic

In our work for deliverable D2.7 we used the terrain-following python code to calculate the under-ice positions of several floats. In Table 4.1 (page 54, copied below) you can find the wmo number of all those floats and the results we got from the interpolation procedure. We could use the floats with bad results and the ones that resulted in errors to see if your improvemed code has better results.

see https://github.com/euroargodev/Coriolis-under-ice-positioning/issues/2