Fix issues with enum options filtering logic in enum_ function

The current implementation of the enum_ function in Valibot filters enum options using the following code:
https://github.com/fabian-hiller/valibot/blob/8ab047f44650ab0fbece0ffac4c2972a0a10b836/library/src/schemas/enum/enum.ts#L92-L98
The purpose of this filtering logic is to remove the reverse mapping entries that TypeScript adds when transpiling enums. Specifically, when an enum member is defined as Name = value and value is a number, TypeScript generates a reverse mapping in the form Enum[value] = 'Name'. The key of this reverse mapping is stringified by Number.prototype.toString with radix=10 via ToPropertyKey as defined by the ECMAScript specification.
There are several issues with using +key and isNaN in the filtering logic:

String Conversion: The +key operation follows the StringToNumber abstract operation as defined in the ECMAScript specification. Since enum_ filters out any number, this operation has several implications:

It trims leading and/or trailing whitespaces or line terminators.
If the string is empty after trimming, it returns 0.
Strings such as Infinity or -Infinity are interpreted as their respective number representations.
It interprets prefixes like 0b, 0o, or 0x as binary, octal, or hexadecimal literals.
Decimal variations such as 0.1, .1, and 0. are all valid and yield the respective numeric values.
It also allows leading zeros.
It accepts the + symbol as a valid sign.

NaN Values in Enum: If the enum contains NaN, as in the following example:
enum N { X = NaN }
// equivalent to js: const N = { X: NaN, NaN: 'X' }

The isNaN check will not work as expected since isNaN(key) will return true when key is 'NaN', causing the 'NaN' key to be retained rather than filtered out.

To correctly filter out these reverse mappings, I made the filtering logic more precise. In this PR, rather than isNaN, the parsed value is stringified and is matched against the original key. If it matches, it is filtered out.
Additionally, the filtering logic is not reflected in the EnumSchema type. For example, when using enum_({ 1: "A" }), the options end up being empty, which was not obvious until runtime. Therefore I implemented type-level option filtering.

Thanks for creating this PR! Unfortunately I have almost no time to review and merge it in the next 3 weeks, but I plan to do it before we release our stable v1.

I will improve the filtering of reverse mappings, undo the changes to types and merge this PR if you have no objections.


Research if we can put the rendering visualization bytes logic from clientside to backend

Currently, all the fall back logic if visualization data is not present is in client side (frontend). The code is a bit messy. Check if we can put that to a backend endpoint.

visconfg can have parameters, but does NOT need to have vis data.
for some components (e.g. thumbnail) we don't care where the bytes come from, either visData or raw bytes. so we want an endpoint to get the bytes for a given component - the backend can return from either.


Adds commit status attribute to projects

Adds a new commit_status attribute to projects. This will help us with the Project Monitor tool on the Brigade website.
Closes #298

Merge and run
heroku run python app.py db upgrade --app codeforamerica-api


Add --fusecmd

Description of the Pull Request (PR):
This PR adds the --fusecmd built-in option to the fuse_mount branch.    It includes & supercedes PR #1.  Compared to that PR, this adds a singularity.conf option to disable --fusecmd, adds implying the --pid option (so fuse processes that are started automatically get cleaned up on exit), and adds searching for the fuse command in $PATH.
This fixes or addresses the following GitHub issues:

Before submitting a PR, make sure you have done the following:

Read the Guidelines for Contributing, and this PR conforms to the stated requirements.
Added changes to the CHANGELOG if necessary according to the Contribution Guidelines
Added tests to validate this PR and tested this PR locally with a make testall
Based this PR against the appropriate branch according to the Contribution Guidelines
Added myself as a contributor to the Contributors File

Attn: @singularity-maintainers

This will be replaced with a sylabs/singularity PR, closing.


const3.dd: start work on this one

I moved the stuff about storage classes to the end, they really should be merged with https://dlang.org/spec/declaration.html#typequal_vs_storageclass

@WalterBright your changes overwrote my editorial changes, could you please check what your pushing process is? If you force push (with -f) into your branch, please change that to push without forcing. Then you'll get errors only if you need to pull first.

For example I just created https://github.com/dlang/dlang.org/pull/2156/commits/5d7e5976dcf29ef0040fa3864c1ffbaed7d50569. Can you make sure it doesn't get overwritten. Thanks.

@andralex How do I merge your changes into my git repository?


review: sample_uniform_ubig_below

Closes #2070

#2119  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @Shoeboxam and the rest of your teammates on  Graphite


[util] Add in-file string replacement command.

Originally used in https://github.com/ros-planning/moveit/pull/79#issue-172005695, now I'm making the command callable on my OS-wide.

[ ] Call newly added test py file.

test*.py should NOT be executable in order for it to be run by nose. https://ubuntuusertips.wordpress.com/2014/02/25/python-testing-with-nose-ran-0-tests-in-0-000s/

[ ] Would be nice if this accepts regex for file names.

[ ] Call newly added test py file.
[ ] Would be nice if this accepts regex for file names.

Make them future todo. Merging.


Typescript support

Hi,
I'm trying to use this package in my TS project, and it faild due to missing types.
It will be great if this package will support TS.
Thanks.

I just released version 1.2.15 with Typescript support for modules in the project.  I created a simple Typescript project and was able to import and use it.
If you want types for data parsed from PDF documents then that would have to part of your application.  I could add a module to generate types, but I'd have to think about the use cases.  I have another project where I generate type definitions from a data stream, but it's in my own data definition format.


Got Segmentation fault (core dumped) for run_stdlib_sha256_bench

Dear maintainers,
Thanks for building this great projects! I am interested in the benchmark number of plonk on sha256, but the benchmark script failed.
What's wrong
make run_stdlib_sha256_bench returns error Segmentation fault (core dumped)
make run_stdlib_sha256_bench
...
 Thank you @zac-williamson


Call MPI from jitted code

Is there any possibility to have interoperability between JAX and MPI / mpi4py?
I am porting a physical model to JAX with has lots of interactions like this:
def compute_something(...):
    # instruction block 1
    # ...

    # this communicates boundaries of arr between processes
    # via mpi4py
    arr = boundary_exchange(arr)

    # instruction block 2
    # ...

    return arr

Now, in order to JIT this, I would have to transform all of those into
@jax.jit
def compute_something_block_1(...)
    # ...

@jax.jit
def compute_something_block_2(...)
    # ...

def compute_something(...):
    # instruction block 1
    compute_something_block_1(...)

    # this communicates boundaries of arr between processes
    # via mpi4py
    arr = boundary_exchange(arr)

    # instruction block 2
    compute_something_block_2(...)

    return arr

which doesn't do the code any favors.
So, the question is this:

Is it possible to fall back to Python interactions inside jitted code? (I guess the answer is no)
What would be the most convenient way to make XLA aware of MPI? Is it necessary to wrap the MPI calls as custom call targets?

I think you’ve accurately accessed the current options. I’d love to have the ability to easily custom call into Python, but that doesn’t exist currently.

This is possible now via mpi4jax.

Nice work!
On Fri, Mar 12, 2021 at 3:03 AM Dion Häfner @.***>
wrote:

This is possible now via mpi4jax https://github.com/PhilipVinc/mpi4jax.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/google/jax/issues/2860#issuecomment-797418530, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAJJFVVADCOBKFBZRB4EVJTTDHYJHANCNFSM4MSAQQGA
.


Deduplicate prefix logic

Following up on #70

[x] trivial helpers
[x] attempt (?) to refactor other common logic

This isn't breaking, so it doesn't matter if we merge before or after 2.0.0.

@chipshort Last one on this topic! Thanks a lot.

All good, thanks!


version docs should include versioning

Contributions

[ ] I have read the contribution docs, and understand what's expected of me.

Link to the page on docs.getdbt.com requiring updates
https://docs.getdbt.com/reference/project-configs/version
Doesn't have any mention of model versioning.
What part(s) of the page would you like to see updated?
Maybe it's a disambiguation callout ("you might be looking for versions)?
Additional information
No response

@joellabes great minds think alike!
This looks like a duplicate of https://github.com/dbt-labs/docs.getdbt.com/issues/3290, so I'm going to close this proactively.


Could we implement getSubTimeline to work while providing an element?

Could the getSubTimeline work while providing the component element instead of only by providing the id
this.viewModel.$refs.items.forEach(item => {
    this.transitionInTimeline.add(this.getSubTimeline(item));
});

Think you can match the provided element with the element that is in the child-instance? loop over this.$children -> element === childInstance.element?


Parallel copy using HASH is misleading

Hello,
I just wanted to verify if my understanding is correct when trying to use parallel copy to copy a full table from Teradata. The documentation says the following:
"Partition option: Hash.
During execution, Data Factory automatically detects the PK column, applies a hash against it, and copies data by partitions."
However, I believe that is misleading and leads to errors. I have been trying to use HASH to copy full tables and I get the following error:
"This table is a No Primary Index(NOPI) table"
I then looked at the tables and saw that in Teradata those tables did indeed have a Primary index. I then looked at the tables that did have this issue vs the tables that were successfully copied and I think the reason why is that instead of using Primary Key (as the documentation suggests ADF will do) it instead uses "Partition BY" clause that is written in the Teradata SQL statement.
Below is an example of the SQL code from a tables that were successfully extracted:

Here is an image of the SQL code from a table that got the NOPI error:

As you can see the two tables that were copied in parallel both had a Partition by clause already written in, whereas the table that did not have those got an error out. I have checked with other tables and the same pattern appears.
This is why I wanted to verify if I am correct in my assessment. If I am correct then the documentation would need to be updated because it looks like ADF is not getting the primary key automatically. Instead it is looking for the Partition by key, or at the very least it needs the partition by key to be able to run parallel copy.
As a background what my ADF job does is get a list from a Lookup activity which is then passed into a ForEach activity. Nested in that ForEach activity is a Copy command that takes the table names and then writes the records of those tables into a blob storage for archive.
I had assumed that to speed them up I can just enable the "HASH" partition for all those tables. I understood the documentation saying that it would auto-detect a primary key and then would be able to write all those tables using parallel copy. The end result was that some tables (the one with partition by) were successful, but others were not (getting that NOPI error).
Please let me know if I have misunderstood something or if the documentation needs to be updated.
Thank you

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: a317cac0-e27c-7263-8c85-e44243b21234
Version Independent ID: 8d0f5788-07f2-56e4-79c1-7ee944976f77
Content: Copy data from Teradata Vantage by using Azure Data Factory - Azure Data Factory
Content Source: articles/data-factory/connector-teradata.md
Service: data-factory
GitHub Login: @linda33wj
Microsoft Alias: jingwang

Thank you for your detailed feedback.  We are looking into the issue, and we will respond when we have more information.

@vlado101 : We are engaging the internal team to get more help on this . As per the internal team , can you please see when you try to add manually fill in the primary index column name does that work ?

Hi @himanshusinha-msft , yes when I manually specify a column to be used for a partition then ADF does use parallel copies. The issue comes when I have a large group of tables and am sending them to an ForEach activity and I leave that column blank.
Because the tables are different I am not sure which one to be used for partition and per my reading of the documentation I thought that ADF would automatically detect "PK" which I assumed to be Primary Key. Hence why I left that column blank.
If I use the Teradata connect to even pull just one table and use HASH while leaving the column blank it also returned the "NOPI" error (aka no primary index error).
Let me know if that is enough details.

Hello,
Just wanted to follow up on this issue.
Thank you

@vlado101 : We tried to create a table which is similar with customer, but cannot repro the customer issue.
CREATE SET TABLE EMPLOYEETESTINDEX,FALLBACK (
EmployeeNo INTEGER,
FirstName VARCHAR(30),
LastName VARCHAR(30),
DOB DATE FORMAT 'YYYY-MM-DD',
JoinedDate DATE FORMAT 'YYYY-MM-DD',
DepartmentNo BYTEINT
)
UNIQUE PRIMARY INDEX partitionColumn( EmployeeNo, FirstName, LastName );
Can you please email us<EMAIL_ADDRESS>(with the subject : ATTN HImanshu )  and share more details
1.runid
2.SHIR log
3.exact table DDL, not only part of screenshot?
4.Subscription ID
If the partition column is empty, we use index type to auto detect primary index. We will use Nonpartitioned Primary and Primary index as partition column (index type is P or Q).

@himanshu-mishra I have sent over the email with the DDLs. Working on getting the additional info requested.

@vlado101 : We will go ahead and close this issue now . You are facing this issue as table name is more then 30 character long .


Safety check scheduler MaximumWeight

Adds a config item MinAvailableTaskWeight and checks that in each block at least one task of such weight can be dispatched.
This is important to ensure that runtime upgrades work since #11637 will over-estimate the PoV size quite a lot.
TODO once approved:

[ ] Companions

The CI pipeline was cancelled due to failure one of the required jobs.
Job name: cargo-check-benches
Logs: https://gitlab.parity.io/parity/mirrors/substrate/-/jobs/2175017

This is important to ensure that runtime upgrades work since https://github.com/paritytech/substrate/pull/11637 will over-estimate the PoV size quite a lot.

If it is systematically overestimating weight to that large a degree then it will need fixing. What pallets/calls are the worst offenders here?

The scheduler uses on_initialize which has class Mandatory and can consume as much as it wants.


ESTree compatible AST

Describe the feature
Support ESTree compatible AST.
Some options:

SWC output ESTree compatible AST directly.
New tool like swc-to-estree

Babel plugin or link to the feature description
Nothing.
Additional context
Add any other context about the problem here.
I'm a maintainer of Prettier Code Formatter and I'm developing to use SWC as a parser for Prettier (Disclaimer: It is a 100% personal activity and not affiliated with any organization.) Prettier already has a code base to handle ESTree compatible ASTs well. So, if SWC supports ESTree, it will be easier. Also, ESTree is the de facto standard for JavaScript ASTs, and by supporting it, many other tools may be able to use SWC.
related: https://github.com/swc-project/swc/issues/1392

@kdy1 What do you think about this?

@sosukesuzuki Just a heads up, the parse APIs are being deprecated. But I've asked if there'll be a way to support this scenario in the future as I've seen some perf improvements on my end (replacing acorn with swc+ little AST adapters).

I made a mistake. parse will be preserved to help some tools like ast explorer.

Is it expected that third party libraries will only use the swc parser?

No, but rust <-> js is very slow and it overwhelms time used for actual work.

I see. Out of curiosity, you are doing integration SWC and Next.js at vercel? At there, how to integrate Rust land and JS land?

Yes. We are working on it. We are going to reimplement all transforms in rust, and call transform() from js.

I found https://github.com/vercel/next.js/tree/canary/packages/next/build/swc. It looks good, thanks

@kdy1 What do you think about making swc's AST consistent with ESTree, which is the de facto standard?

What's the advantage if I make it compatible with estree? rust does not have GC, so estree-like ast will degrade performance.

I did not know about the performance drawbacks. The advantage is that it is much easier for existing tools such as ESLint and Prettier to use swc as a parser.

I see. But actually, passing ast to javascript from rust is very slow. Parser itself is fast, but json serialization/deserialization is the problematic part, and I'm not sure about the way to make it fast.
Maybe lazy accessors can be a solution, but I didn't try it.

If you provide the string the user can do things like fast-json. This allows for them to choose which serializer (JSON.parse or otherwise) to use.

@jdalton If it's fast enough, I think we can support babel plugins. So I think it is worth benchmarking.

As a test are raw parse power I tried parsing babylon.max.js which s 10.6MB of JS. It takes acorn 706ms and swc ~2 seconds so I can see what you mean about the Rust->JS transition.

Please see #2175 if you're interested. I've done some experiments into speeding up parse().


RandSCapsule -> Spacedock - closes #3356

Thanks, @nobodyhasthis!

Looks good, merging with my thanks


Added component parser for attraction events

Description
Issue closed #7
This endpoint can be queried via:

get_attraction_events_from_calendar
get_attraction_events

The endpoint returns events for a given attraction.
Change Type

[ ] Bug fix
[x] New feature
[x] Breaking change
[ ] Minor changes in old functionality

[!IMPORTANT]
This will fundamentally change how the API is queried and should be included in the release notes

Checklist

[x] I have added pytest-Test for my changes.
[x] My code changes are commented using docstrings
[x] I reviewed my changes
[x] My changes generate no new warnings
[x] CI/CD test and documentation passes.

Codecov Report
Attention: Patch coverage is 92.33449% with 22 lines in your changes are missing coverage. Please review.
:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.

Files
Coverage Δ

src/pyventim/__init__.py
100.00% <100.00%> (ø)

src/pyventim/exceptions.py
100.00% <100.00%> (ø)

tests/adapters/test_exploration_adapter.py
100.00% <100.00%> (ø)

tests/eventim/test_component.py
100.00% <100.00%> (ø)

tests/eventim/test_exploration.py
100.00% <100.00%> (ø)

tests/utils/test_utils.py
100.00% <100.00%> (ø)

src/pyventim/utils.py
94.73% <93.33%> (-5.27%)
:arrow_down:

src/pyventim/models.py
96.15% <96.15%> (ø)

src/pyventim/adapters.py
83.72% <83.72%> (ø)

src/pyventim/eventim.py
77.35% <77.35%> (ø)


[fix][broker] Fix NPE in MessageDeduplication.

Motivation
When MessageDeduplication#purgeInactiveProducers:
https://github.com/apache/pulsar/blob/d09c6eb26adcaf0aff2c3464e445eb8df5af70a6/pulsar-broker/src/main/java/org/apache/pulsar/broker/service/persistent/MessageDeduplication.java#L455-L477
If MessageDeduplication status is not Enabled, the cursor will be null. and cause to be NPE at line 475.
See cursor initializes:
https://github.com/apache/pulsar/blob/a43981109a9322d94082ae0d87d0de53b8f237e8/pulsar-broker/src/main/java/org/apache/pulsar/broker/service/persistent/MessageDeduplication.java#L272-L292
StackTrace
May 25, 2022 @ 13:39:04.552	2022-05-25T05:39:04,546+0000 [pulsar-inactivity-monitor-22-1] ERROR org.apache.bookkeeper.common.util.SafeRunnable - Unexpected throwable caught
May 25, 2022 @ 13:39:04.552		at org.apache.pulsar.broker.service.BrokerService.lambda$forEachTopic$78(BrokerService.java:1768)
May 25, 2022 @ 13:39:04.552		at org.apache.pulsar.broker.service.persistent.MessageDeduplication.purgeInactiveProducers(MessageDeduplication.java:475)
May 25, 2022 @ 13:39:04.552		at java.util.Optional.ifPresent(Optional.java:183) ~[?:?]
May 25, 2022 @ 13:39:04.552		at org.apache.pulsar.broker.service.BrokerService.checkMessageDeduplicationInfo(BrokerService.java:1735)
May 25, 2022 @ 13:39:04.552		at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:515) [?:?]
May 25, 2022 @ 13:39:04.552		at org.apache.pulsar.common.util.collections.ConcurrentOpenHashMap.forEach(ConcurrentOpenHashMap.java:272)
May 25, 2022 @ 13:39:04.552		at org.apache.bookkeeper.common.util.SafeRunnable.run(SafeRunnable.java:36)
May 25, 2022 @ 13:39:04.552	java.lang.NullPointerException: null
May 25, 2022 @ 13:39:04.552		at org.apache.pulsar.broker.service.BrokerService.forEachTopic(BrokerService.java:1766)
May 25, 2022 @ 13:39:04.552		at org.apache.bookkeeper.mledger.util.SafeRun$1.safeRun(SafeRun.java:32)

Documentation

[x] doc-not-needed
(Please explain why)

I'm not sure if it could solve the NPE. Is there a race condition like the following code snippet I commented?
        if (hasInactive && isEnabled()) { // 1. isEnabled() returns true, managedCursor is not null
            // during 1 and 2, some other methods changed `managedCursor` to null
            takeSnapshot(getManagedCursor().getMarkDeletedPosition()); // 2. managedCursor might be null

I'd prefer the following way.
        final ManagedCursor cursor = managedCursor;
        if (hasInactive && cursor != null) {
            takeSnapshot(cursor.getMarkDeletedPosition(), cursor);
        }

    private void takeSnapshot(Position position) {
        takeSnapshot(position, getManagedCursor());
    }

    private void takeSnapshot(Position position, ManagedCursor cursor) {
        /* ... */
        cursor.asyncMarkDelete(position, snapshot, new MarkDeleteCallback() {

yes, this could avoid the case when there is race condition.

I'm not sure if it could solve the NPE. Is there a race condition like the following code snippet I commented?
        if (hasInactive && isEnabled()) { // 1. isEnabled() returns true, managedCursor is not null
            // during 1 and 2, some other methods changed `managedCursor` to null
            takeSnapshot(getManagedCursor().getMarkDeletedPosition()); // 2. managedCursor might be null

I'd prefer the following way.
        final ManagedCursor cursor = managedCursor;
        if (hasInactive && cursor != null) {
            takeSnapshot(cursor.getMarkDeletedPosition(), cursor);
        }

    private void takeSnapshot(Position position) {
        takeSnapshot(position, getManagedCursor());
    }

    private void takeSnapshot(Position position, ManagedCursor cursor) {
        /* ... */
        cursor.asyncMarkDelete(position, snapshot, new MarkDeleteCallback() {

Method checkStatus and purgeInactiveProducers is synchonized.  Seems no race condition.


[MySQL] Error 1366: Incorrect string value

What happened:
While rsync from local disk to juicefs mount, it suddenly (after several hours) stopped with
Failed to sync with 11 errors: last error was: open /mnt/juicefs/folder/file.xls: input/output error
On the jfs log, i can find these:
juicefs[187516] <ERROR>: error: Error 1366: Incorrect string value: '\xE9sa sa...' for column `jfsdata`.`jfs_edge`.`name` at row 1
goroutine 43510381 [running]:
runtime/debug.Stack()
        /usr/local/go/src/runtime/debug/stack.go:24 +0x65
github.com/juicedata/juicefs/pkg/meta.errno({0x2df8860, 0xc0251d34a0})
        /go/src/github.com/juicedata/juicefs/pkg/meta/utils.go:76 +0xc5
github.com/juicedata/juicefs/pkg/meta.(*dbMeta).doMknod(0xc0000e0c40, {0x7fca7e165300, 0xc00ed28040}, 0x399f6f, {0xc025268160, 0x1f}, 0x1, 0x1b4, 0x0, 0x0, ...)
        /go/src/github.com/juicedata/juicefs/pkg/meta/sql.go:1043 +0x29e
github.com/juicedata/juicefs/pkg/meta.(*baseMeta).Mknod(0xc0000e0c40, {0x7fca7e165300, 0xc00ed28040}, 0x399f6f, {0xc025268160, 0x1f}, 0xc0, 0x7b66, 0x7fca, 0x0, ...)
        /go/src/github.com/juicedata/juicefs/pkg/meta/base.go:594 +0x275
github.com/juicedata/juicefs/pkg/meta.(*baseMeta).Create(0xc0000e0c40, {0x7fca7e165300, 0xc00ed28040}, 0x26b5620, {0xc025268160, 0x2847500}, 0x8040, 0xed2, 0x8241, 0xc025267828, ...)
        /go/src/github.com/juicedata/juicefs/pkg/meta/base.go:601 +0x109
github.com/juicedata/juicefs/pkg/vfs.(*VFS).Create(0xc000140640, {0x2e90348, 0xc00ed28040}, 0x399f6f, {0xc025268160, 0x1f}, 0x81b4, 0x22a4, 0xc0)
        /go/src/github.com/juicedata/juicefs/pkg/vfs/vfs.go:357 +0x256
github.com/juicedata/juicefs/pkg/fuse.(*fileSystem).Create(0xc000153900, 0xc024980101, 0xc022a48a98, {0xc025268160, 0x1f}, 0xc022a48a08)
        /go/src/github.com/juicedata/juicefs/pkg/fuse/fuse.go:221 +0xcd
github.com/hanwen/go-fuse/v2/fuse.doCreate(0xc022a48900, 0xc022a48900)
        /go/pkg/mod/github.com/juicedata/go-fuse/v2@v2.1.1-0.20210926080226-cfe1ec802a7f/fuse/opcode.go:163 +0x68
github.com/hanwen/go-fuse/v2/fuse.(*Server).handleRequest(0xc00179c000, 0xc022a48900)
        /go/pkg/mod/github.com/juicedata/go-fuse/v2@v2.1.1-0.20210926080226-cfe1ec802a7f/fuse/server.go:483 +0x1f3
github.com/hanwen/go-fuse/v2/fuse.(*Server).loop(0xc00179c000, 0x20)
        /go/pkg/mod/github.com/juicedata/go-fuse/v2@v2.1.1-0.20210926080226-cfe1ec802a7f/fuse/server.go:456 +0x110
created by github.com/hanwen/go-fuse/v2/fuse.(*Server).readRequest
        /go/pkg/mod/github.com/juicedata/go-fuse/v2@v2.1.1-0.20210926080226-cfe1ec802a7f/fuse/server.go:323 +0x534
 [utils.go:76]

Nothing was logged at MariaDB side.
Environment:

juicefs version 1.0.0-beta2+2022-03-04T03:00:41Z.9e26080
Ubuntu 20.04
MariaDB 10.4

I'll check it. so there are incompatibilities between a standard FS (zfs in this case) and JuiceFS ? Is this documented ?

@zhijian-pro filename is école-200311.xlsx

We think this is a compatibility issue with mariadb, the same logic can run normally on mysql.
In addition, the encoding of jfs_edge after run juicefs format should be utf8mb4 COLLATE utf8mb4_bin , it is not clear why yours is different. This encoding set can write special characters normally on mysql, but not on mariadb, so we think this is a mariadb compatibility problem.

I can confirm, but server is set to that collation.
MariaDB [jfs_data]> SELECT TABLE_SCHEMA, TABLE_NAME, TABLE_COLLATION
    -> FROM INFORMATION_SCHEMA.TABLES
    -> WHERE TABLE_NAME = 'jfs_edge';
+--------------+------------+--------------------+
| TABLE_SCHEMA | TABLE_NAME | TABLE_COLLATION    |
+--------------+------------+--------------------+
| jfs_data     | jfs_edge   | utf8mb4_unicode_ci |
+--------------+------------+--------------------+
1 row in set (0.014 sec)

Global configuration
[mysql]
default-character-set = utf8mb4

[mysqld]
collation-server = utf8mb4_unicode_ci
init-connect = 'SET NAMES utf8mb4'
character-set-server = utf8mb4

But this can be normally be override.

We did something similar after creating the table, you can see the corresponding logic here. When I tested mariadb, the encoding was utf8mb4_bin as we expected

Yeah but even after manually changing the collation, the same error is produced, and column jfsdata.jfs_edge.name is now utf8mb4_bin...

We think this is a compatibility issue with mariadb, you can insert unicode characters into the jfs_edge table using your familiar programming language. Our test result is that the same is utf8mb4_bin encoding, mysql can be inserted successfully, but mariadb cannot be inserted successfully. So we think it's a compatibility issue with mariadb.
Below is the test using go language.
Before testing, please prepare mariadb and mysql databases, and execute the juicefs format command respectively. Then run the test program and you can see the output is
mysql test insert unicode char success
mariadb test insert unicode char error: Error 1366: Incorrect string value: '\xE9sa' for column `dev`.`jfs_edge`.`name` at row 1

This is the test program
package main

import (
	"database/sql"
	"fmt"
	"math/rand"
	"time"

	_ "github.com/go-sql-driver/mysql"
)

func main() {

	mariadb := "user:password@(ip:port)/database"
	mysql := "user:password@(ip:port)/database"

	_, err := testInsertUnicodeChar(mysql)
	if err != nil {
		fmt.Println("mysql test insert unicode char error:", err)
	} else {
		fmt.Println("mysql test insert unicode char success")
	}

	_, err = testInsertUnicodeChar(mariadb)
	if err != nil {
		fmt.Println("mariadb test insert unicode char error:", err)
	} else {
		fmt.Println("mariadb test insert unicode char success")
	}
}

func testInsertUnicodeChar(jdbcStr string) (int64, error) {
	db, err := sql.Open("mysql", jdbcStr)
	if err != nil {
		return 0, err
	}
	// unicode char
	s := "\xE9sa"

	// random seed
	rand.Seed(time.Now().UnixNano())

	res, err := db.Exec("INSERT INTO jfs_edge VALUES (?,?,?,?)", rand.Intn(1000), s, 1, 1)
	if err != nil {
		return 0, err
	}
	id, err := res.LastInsertId()
	if err != nil {
		return 0, err
	}
	return id, nil
}

@zhijian-pro what if you manually set the collation? Maybe this is required for MariaDB?
https://github.com/go-sql-driver/mysql#collation

Ok I found the root cause. MariaDB sets sql_mode to:
STRICT_TRANS_TABLES,ERROR_FOR_DIVISION_BY_ZERO,NO_AUTO_CREATE_USER,NO_ENGINE_SUBSTITUTION,NO_ENGINE_SUBSTITUTION
Once set to:
ERROR_FOR_DIVISION_BY_ZERO,NO_AUTO_CREATE_USER,NO_ENGINE_SUBSTITUTION,NO_ENGINE_SUBSTITUTION
After that, executing your program, you get
mysql test insert unicode char success
mariadb test insert unicode char success

But i don't know how to solve this JuiceFS side...

Test also passes by replacing, in your program, line
mariadb := "user:password@(ip:port)/database"
by
mariadb := "user:password@(ip:port)/database?sql_mode=NO_ENGINE_SUBSTITUTION"

Also, take this simple test:
CREATE DATABASE test;
CREATE TABLE test (test varchar(255));

with the default sql_mode:
mysql -e "USE test; INSERT INTO test VALUES ('🅰');"
ERROR 1366 (22007) at line 1: Incorrect string value: '\xF0\x9F\x85\xB0' for column `test`.`test`.`col1` at row 1

but this works as expected:
mysql -e "SET NAMES utf8mb4; USE test; INSERT INTO test VALUES ('🅰');"

I found this bit of the code that should alter it.
If you create the table with:
MariaDB [test]> CREATE TABLE test (test varchar(255)) character set=utf8mb4;
Query OK, 0 rows affected (0.002 sec)
``

It works also. Note character set is the range of what it can store. Collation is its sort ordering. Focus on character set.

Per @zhijian-pro 's second comment, what is your `show create table jfs_edge`? You can manually alter it per the `alter table` from the code.

Do filesystems actually enforce a correct encoding of utf8 filenames? If not maybe `varbinary(255)` would be a better database choice.

@solracsf  It can indeed be solved by changing the sql_mode.
You can add the sql_mode parameter after the metaurl, and then format and mount.
example:
juicefs format "user:password@(ip:port)/database?sql_mode='ERROR_FOR_DIVISION_BY_ZERO,NO_AUTO_CREATE_USER,NO_ENGINE_SUBSTITUTION'" test1

juicefs mount "user:password@(ip:port)/database?sql_mode='ERROR_FOR_DIVISION_BY_ZERO,NO_AUTO_CREATE_USER,NO_ENGINE_SUBSTITUTION'"  /test1

@grooverdan  the varbinary(255) is  a good solution, we will upgrade it on later version.

Keep in mind the varbinary is 1 byte per char and the utf8mb4 was obviously 4 bytes per char. Note that the 255 length, for all its common usage, is as far as modern MySQL variants goes a rather arbitrary length (the maximum is 65,532) and the only small overhead is the number of bytes represent the chosen.

@grooverdan

Per @zhijian-pro 's second comment, what is your show create table jfs_edge? You can manually alter it per the alter table from the code.

Here it is:
MariaDB [jfs_data]> show create table jfs_edge;
+----------+---------------------------------------------------------------------------+
| Table    | Create Table                                                                                |
+----------+---------------------------------------------------------------------------+
| jfs_edge | CREATE TABLE `jfs_edge` (
  `parent` bigint(20) NOT NULL,
  `name` varchar(255) COLLATE utf8mb4_bin NOT NULL,
  `inode` bigint(20) NOT NULL,
  `type` int(11) NOT NULL,
  UNIQUE KEY `UQE_jfs_edge_edge` (`parent`,`name`)
) ENGINE=InnoDB DEFAULT CHARSET=utf8mb4 COLLATE=utf8mb4_bin |
+----------+---------------------------------------------------------------------------+
1 row in set (0.001 sec)

Ok I found the root cause.

Removing STRICT_TRANS_TABLES (https://mariadb.com/kb/en/sql-mode/#strict_trans_tables) simply turns off strict mode. It is not the root cause. MySQL does not set it by default.

Ok I found the root cause.

Removing STRICT_TRANS_TABLES (https://mariadb.com/kb/en/sql-mode/#strict_trans_tables) simply turns off strict mode. It is not the root cause. MySQL only sets it as default starting with 5.7. It would be interesting to run the test against MySQL 5.6 and MySQL 5.7. MariaDB sets it as default already on 10.2 and possibly earlier.

I second comments by @grooverdan to be mindful of the length of the field chosen and possible implications (off-by-one, buffer overflows, variable char lengths, special char handling, translations of file names between different file systems, etc.). When it comes to filenames, this is an area test engineers (and likely hackers, regrettably) love for a reason. You will want to test this well.

Closed by https://github.com/juicedata/juicefs/pull/1762

@solracsf  @grooverdan  @mariadb-RoelVandePaar   We have changed it to use varbinary(255), thanks for the feedback! For existing JuiceFS volumes, please use the latest JuiceFS from main branch and change the column manually.


OBJ-viewer, view .obj 3D model in browser

Student, group: Vasilev Aleksei, 3530901/90203
URL: https://github.com/vasilievan/software_engineering
Git approach: git-flow

Sounds interesting.


Add a little paragraph about scopes

This is a proposal to explain a little bit more how to use scopes.
This is how I understood them but I would love to have your opinions on the different ways to use scope.
Should it be a real list across every projects? Project-specific each time? Should it talk about user-centric scopes or technical ones?

I signed it!
(even though, that's repulsing just to propose something on a repo :/)

Thank you for the effort!
I'm not sure if this change would be good to add in this particular markdown file. If there is confusion as to what the scope should entail, we should flesh it out in an issue & determine how best to document it, as opposed to adding uncertainty inside the documentation IMO. This paragraph is a little at odds with the tenor of the document, which is explaining the requirements of the commit message.


Rate limit obsolete

The rate limit for the API was changed to 60/min from 30/min for requests made using OAuth2. The EnforceRateLimit method is still based on the old limit.

yup, yup. If you want to make a pull request for that, it would be appreciated, otherwise I'll get to it.

if not using Oauth2 it's still the old ratelimit, some people aren't using OAuth2 due to bugs & stuff so can this be implemented in some sort of fashion that allows for OAuth2 users to be 2x the requests than non, thanks

I thought they got rid of the ability to not use OAuth2? I know it's deprecated at the very least, but I'm pretty sure it got removed entirely recently

I also thought that was the case since that's what the API rules say.
On Mon, Jul 18, 2016 at 1:34 PM, CrustyJew<EMAIL_ADDRESS>wrote:

I thought they got rid of the ability to not use OAuth2? I know it's
deprecated at the very least, but I'm pretty sure it got removed entirely
recently
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/CrustyJew/RedditSharp/issues/4#issuecomment-233400033,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAa-thcd6RqAsjuM6hWluAZ9RxUCfuErks5qW7kagaJpZM4JOX5n
.

it's deprecated, they can't do "non OAuth2" releases as that would make it impossible to normally use reddit on a browser

The browser doesn't hit the API directly, it does page requests.
I'm fairly certain OAuth2 is the only way to hit the API now

https://github.com/Mnemosyne-20/Mnemosyne-2.0-1 I'm running this program (which I made) without using OAuth2 intentionally as it's annoying to do the OAuth2 method for what I need to do, also refactorings but that's beside the point, in the end the browser is making an API call somewhere for what to update and when in the javascript, so yes, OAuth2 is reccomended, but not the only option

Well crud.. Alright so that needs to get worked in with a flag somehow

I'll poke through the code tomorrow night and work it in.
On Tue, Jul 19, 2016 at 10:15 AM, CrustyJew<EMAIL_ADDRESS>wrote:

Well crud.. Alright so that needs to get worked in with a flag somehow
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/CrustyJew/RedditSharp/issues/4#issuecomment-233646219,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAa-tsVrBjh85KK2Dn2NFASoAVf4qCsRks5qXNvygaJpZM4JOX5n
.

Fixed in #6


Review all toString calls in JdbcRepository

There's  a number of calls in JdbcRepository like:
val ims   = annos.map(_.imagedMomentUuid.toString).distinct

This needs to be changed as almost all fields in an Annotation object are Option so toString calls will result in Option(someValue) instead of someValue

I've added integration tests for Sql Server and Postgres for all JdbcRepository methods. I think we're good now.


Rewrite it in Rust

Lets see if this builds.

Ok as this repo has access rights to upload artifacts to ghcr it seems like it wants to upload them automatically in the action which I don't really want. @patrickskowronekdkfz can you remove this permission from the repo? It should only be able to upload to dockerhub

I agree having a samply/rust-template in the future would be really nice as I copy pasted basically everything accept the source code together from beam and beam-connect.


Hot reloading not working - error parsing the configuration file, unexpected end of JSON input

Been having trouble getting hot reloading to work after switching from Docker to Docker Desktop on Ubuntu.

2023/12/07 16:39:46 KRAKEND INFO: [SERVICE: Gin] Listening on port: 8080
2023/12/07 16:39:46 KRAKEND DEBUG: [SERVICE: Gin][CORS] Enabled CORS for all requests
2023-12-07 16:40:06 [00] Killing service
2023/12/07 16:40:06 Signal intercepted: interrupt
2023/12/07 16:40:06 KRAKEND INFO: [SERVICE: Gin] Router execution ended
2023-12-07 16:40:06 [00] Starting service
2023-12-07 16:40:06 [00] ERROR parsing the configuration file: '/tmp/KrakenD_parsed_config_template_3702000700.json': unexpected end of JSON input, offset: 0, row: 0, col: 0

docker-compose.yml
version: "3"
services:
  krakend_ce:
    image: devopsfaith/krakend:watch
    container_name: api_gateway
    restart: "unless-stopped"
    volumes:
      - ./:/etc/krakend
    ports:
      - "8080:8080"
    command: ["run", "-c", "/etc/krakend/krakend.json"]
    environment:
      - FC_ENABLE=1
      - FC_SETTINGS=/etc/krakend/settings/development
      - API_APP_URL=api_app:8099
    networks:
      - shared_network
networks:
    shared_network:
        external: true

Hi @domantaskudzius
The log you are showing shows that the hot-reload is actually working because it sends the kill signal and restarts again. The problem is with the configuration you use, not with the watch container. It looks like it is receiving an empty file?


App Engine Flex Session Affinity

Hello, I believe the app engine flex uses the cloud load balancers underneath, correct?  When will we be able to configure the load balancer to use session affinity?  You can do it for a normal cloud load balancer in front of compute engine instances, why not app engine?
Using the gcp-datastore-sessions variable does not work in all cases, especially Vaadin apps where session affinity is required.
There are a lot of people wanting this feature!

+1 from me

Note also that if session affinity is available, we can add a more efficient memory cache for sessions in each instance that will reduce the load on the shared session store and improve semantics for co-located requests.  The image can easily be modified to take advantage of affinity.

@Petikoch Thanks for that - I'll test that out (probably next week) and if it works will see what session options should be turned on in the image to best make use of that.

Excellent, thanks a lot @gregw

I've done some testing with the session_affinity and it appears to work fine.  It does not trigger off the JSESSION_ID, rather it set's it's own GCLB cookie.
The normal default in memory session should work OK with this setup, but if the affinity is not entirely sticky (if the GCLB cookie is lost, a node shuts down etc.) then session data will be lost.   So a better setup would be to have an in memory cache fronting the gcloud datastore.  This can be achieved with the following in your app.yaml:
env_variables:
    JETTY_MODULES_ENABLE: session-cache-hash,gcp-datastore,session-store-gcloud

This will avoid hitting the datastore on every request, sessions will only be written when dirty.  I can see in my tests that objects in the sessions are coming back with the same value and same hashcode (so they have not been serialized deserialized)!
I will add this to the doco!

Pull request #260 documents this

Should it enough to set session_affinity to true in app.yaml to enable session affinity or is there a specific Jetty module that also needs to be enabled with JETTY_MODULES_ENABLE? Documentation does not say anything about that and I cannot see any additional cookies (like the GCLB cookie mentioned by @gregw before) set in the response. Or is it just websockets for which App Engine flex supports session affinity?

If you set session_affinity to true, you will get session affinity implemented by the Google load balancer using its own additional cookie.     This will work with no changes to Jetty.
However, one of the main points of session affinity is to allow efficient session management from an in memory session cache, so if you have affinity, you can turn this on with the session-cache-hash module.    It's not necessary, but it's kind of the point!

Thanks for the clarification, @gregw. This was exactly how I thought. However, as I wrote, basic session infinity does not seem to work in my case. I set session_affinity under network: in app.yaml, but do not see any additional (GCLB or other) cookies added to the HTTP responses. This is why I thought that I may be missing one or another Jetty module or the Load Balancer has to be configured in some way (I do not have configured it at all, so as I understand, all App Engine Flex defaults should be functioning).


Assetto Corsa artifacting and GPU hangs

Disabling reflections in Assetto Corsa causes image artifacts and general GPU instability.
Software information
Assetto Corsa
System information

GPU: AMD Radeon RX 560
Driver: mesa-git-c9dbe52f848ae795ae9fdd7cbeca857db27dcc48
Wine version: wine-staging-3.16
DXVK version: dxvk-git-bf5476900ada0714a86737ddc96683721bffc605
LLVM: llvm-git-fbb826fa5c2e828f0e27944ae92c60fe68208609

Apitrace file(s)
https://drive.google.com/open?id=1yvh6uJg-p0IMrEo9bropR-KVCLIc_UEX
Log files

d3d11.log: acs_d3d11.log
dxgi.log: acs_dxgi.log
dmesg log of a GPU hang: dmesg.log

In case of GPU hangs, please check #252.
When replaying the trace I pretty much instantly get a GPU hang, is that what happens in the game as well or is this an additional issue caused by apitrace?

Do you have access to a Windows machine, and if so, can you record an apitrace there? Something's fishy there.
The GPU hang occurs because the game tries to create a texture with a size of 0x0, and DXVK doesn't return the appropriate error code. This is fixed with 837a3f5787c23e4b583ce6cdbca8fe1d8a4b5be6, and replaying the apitrace will now crash because the texture cannot be created.

Update: I got a Windows trace from someone else, and it appears that the game creates a 2048x2048 texture where it creates a 0x0 texture in your trace. No idea why though.
Does the game now work correctly with latest master or are there still issues?

Are reflections disabled in that apitrace?

I asked and got another one with reflections disabled this time. The game does indeed try to create a 0x0 texture and keeps using it, which results in errors and silently dropped calls with native D3D11 but caused DXVK to crash.
Should be fixed in a95f292dcce681269654576d657b5dcb675540d7, please test if it works now.

https://github.com/doitsujin/dxvk/commit/a95f292dcce681269654576d657b5dcb675540d7 fails to build for me: https://pastebin.com/RYVG8Jgp
Am I doing something wrong?

Sorry, my fault. Should be fixed now.

https://github.com/doitsujin/dxvk/commit/35238ec4d11cbf280209fab82ec2f85879e4c128 completely resolves the issue for me: no artifacting, no previous GPU issues.

Thanks for the quick feedback.


Link to the live demos

Hello :)
Would that be possible to add the links to every live demo? I was searching for the moma one and couldn't remember the UR ^^

Yep, I'll take care of it


suggestion: explain what the tool does

Been reading about this tool for a bit now (and in the middle of building it) and I honestly don't know whether this is intended to allow me to connect to a Debian machine via RDP-ish or if it's a tool that allows people to connect to my machine via RDP. In short: should this be installed on the machine I want to connect to or the machine I want to connect from?

closing.


[#130188061] Rename release to rds-broker

What
This updates this repo so that the name of the boshrelease is rds-broker. This is to reflect the fact that this now only packages up the RDS broker, and not a number of AWS service brokers.
Once this is merged, I intend to rename this repository to paas-rds-broker-boshrelease. This is safe to do because Github sets up redirects for everything
How to review
Code review - make sure it all looks sensible.
Who can review
Anyone but @henrytk or myself.

Confirmed with Alex that the remaining uses of aws-broker are innocuous. Good to merge.

I've now renamed the repository.


PHP 7.2?

Hi I'm trying to install sentry/sentry-laravel and recieving this:
composer require sentry/sentry-laravel
Using version ^0.9.0 for sentry/sentry-laravel
./composer.json has been updated
Loading composer repositories with package information
Updating dependencies (including require-dev)
Your requirements could not be resolved to an installable set of packages.
Problem 1
- This package requires php ~7.1.0 but your PHP version (7.2.3) does not satisfy that requirement.
Installation failed, reverting ./composer.json to its original content.
Is it possible to allow for a later version of php?

Using version ^0.9.0 for sentry/sentry-laravel
./composer.json has been updated
Loading composer repositories with package information
Updating dependencies (including require-dev)
Package operations: 10 installs, 0 updates, 0 removals
  - Installing sentry/sentry (1.9.1): Loading from cache
  - Installing symfony/polyfill-mbstring (v1.8.0): Loading from cache
  - Installing symfony/translation (v4.1.1): Downloading (100%)
  - Installing nesbot/carbon (1.31.0): Downloading (100%)
  - Installing psr/simple-cache (1.0.1): Loading from cache
  - Installing psr/container (1.0.0): Downloading (100%)
  - Installing illuminate/contracts (v5.6.26): Loading from cache
  - Installing doctrine/inflector (v1.3.0): Loading from cache
  - Installing illuminate/support (v5.6.26): Downloading (100%)
  - Installing sentry/sentry-laravel (0.9.0): Loading from cache
sentry/sentry suggests installing monolog/monolog (Automatically capture Monolog events as breadcrumbs)
symfony/translation suggests installing symfony/config ()
symfony/translation suggests installing symfony/yaml ()
symfony/translation suggests installing psr/log-implementation (To use logging capability in translator)
illuminate/support suggests installing illuminate/filesystem (Required to use the composer class (5.6.*).)
illuminate/support suggests installing ramsey/uuid (Required to use Str::uuid() (^3.7).)
illuminate/support suggests installing symfony/process (Required to use the composer class (~4.0).)
illuminate/support suggests installing symfony/var-dumper (Required to use the dd function (~4.0).)
Writing lock file
Generating autoload files
sentry-sractch ❯ php -v
PHP 7.2.6 (cli) (built: May 25 2018 06:18:43) ( NTS )```

I don't think Sentry is the package that the error is about.

We require `^5.4||^7.0` so that should include PHP 7.2 (I am using it on multiple 7.2 projects), could you share the full composer.json and/or output of the composer require command?

Sorry, I've only just come back to this, Sentry is working fine now :) Do you know if there is support for Laravel 5.7?

Great to hear back! Will close this issue then :)
And yes 5.7 is supported!


Support drawImage function

Image drawing on canvas and svg

Canvas drawImage now in https://github.com/williamngan/pts/commit/eb035c0f7bc3fffcbbbe61047fa54779d59debb8

This has be implemented as form.image(...)


Fixes issue #499: calling getCharacterBoundsOnScreen on a paragraph w…

…ith selection stops the selection being displayed.
The getCharacterBoundsOnScreen method borrows the selectionShape to do the calculation, but it was only taking a reference to the list beforehand, so the saved "copy" was getting wiped out and thus not restoring the selection properly afterwards.  Simple fix: take copy of the list, not a reference.

Thanks!


Forecast uses Scope object directly

These changes were undone somehow when resolving conflicts in #107 and #106

Closed by #127


// foo throws an exception in the lexer

This should definitely fail gracefully.
Repro:
// foo

refmt: internal error, uncaught exception:
       File "src/reason-parser/reason_lexer.mll", line 700, characters 6-12: Assertion failed
       Raised at file "src/reason-parser/reason_toolchain.ml", line 89, characters 6-15
       Called from file "src/reason-parser/reason_toolchain.ml", line 296, characters 14-57
       Called from file "src/refmt/reason_implementation_printer.ml", line 35, characters 21-32
       Called from file "src/refmt/refmt_impl.ml", line 79, characters 6-51
       Called from file "src/refmt/refmt_impl.ml", line 96, characters 16-35
       Re-raised at file "parsing/location.ml", line 450, characters 12-23
       Re-raised at file "parsing/location.ml", line 450, characters 12-23
       Re-raised at file "parsing/location.ml", line 450, characters 12-23
       Re-raised at file "parsing/location.ml", line 450, characters 12-23
       Re-raised at file "parsing/location.ml", line 450, characters 12-23
       Re-raised at file "parsing/location.ml", line 450, characters 12-23
       Called from file "parsing/location.ml" (inlined), line 454, characters 31-61
       Called from file "src/refmt/refmt_impl.ml", line 101, characters 10-60
       Called from file "src/reason-parser/vendor/cmdliner/cmdliner.ml", line 1303, characters 27-34
       Called from file "src/reason-parser/vendor/cmdliner/cmdliner.ml", line 1350, characters 17-26
       Called from file "src/reason-parser/vendor/cmdliner/cmdliner.ml", line 1375, characters 8-50

Ideally we should say "Sorry, comments starting with // aren't supported yet. Please use /* and */"
If it doesn't take too long, we should just add line comment. Please please please.
@jordwalke

Please add line comments.  Also, why is this closed? You shouldn't crash on this even if it's a syntax error.

@webern Like comments are supported, at least by 3.4.0 and maybe earlier versions.

OK, I did get this same exception from line comments. If this is the right tool then the version is:
bsb -version
4.0.18

Next version of bs-platform will work

Repros with the comments in this file.
Styles.re.zip
yarn build
yarn run v1.13.0
warning package.json: License should be a valid SPDX license expression
$ bsb -make-world
ninja: no work to do.
ninja: no work to do.
[1/2] Building src/Styles.mlast
refmt: internal error, uncaught exception:
       File "reason_lexer.mll", line 725, characters 6-12: Assertion failed

File "/Users/mjb/repos/mjb/ui/js/src/Styles.re", line 1:
Error: Error while running external preprocessor
Command line: /Users/mjb/repos/mjb/ui/js/node_modules/bs-platform/lib/refmt.exe --print binary '/Users/mjb/repos/mjb/ui/js/src/Styles.re' > /var/folders/rm/wb3nkcw91sl3174x90rw5n9r0000gn/T/ocamlpp598a30

ninja: error: rebuilding 'build.ninja': subcommand failed

fwiw I recently ran into an issue with the lexer having this issue when a single line comment is on the last line in the file. As a temporary work around, you could add something after the comments like I did

@Schmavery do you still have the code to reproduce?
@webern what version of bsb are you using?

@IwanKaramazow 4.0.18

@IwanKaramazow
That was on bs-platform 4.0.18 (and I just reproed on bs-platform 5.0.0-dev.5) and get this:
// let _ = 1;

throws
refmt: internal error, uncaught exception:
       File "reason_lexer.mll", line 725, characters 6-12: Assertion failed

If I add a newline (or anything else that's not a single-line comment) after the comment, reformat works again. FWIW that's using refmt with the in-place flag, but my editor integration (vscode reason-language-server) also seems unable to reformat it.

@Schmavery
https://github.com/facebook/reason/issues/2350

Great! (The only reason I mentioned anything in this issue is that I noticed @webern 's example had a single-line comment on the last line, so this probably fixes their problem as well :)


Pekko support

Are there any plans to include support for Pekko Streams in the future?

That would be great to have as it does not make sense to have an open source project on Akka any longer. But the project is in maintenance mode so it will need some push to make that happen.


dep: upgrade tonic + hyper

In https://github.com/neondatabase/neon/pull/9230 we ignored the security advisory because hyper is not used for externally-visible services plus it's a big refactor to upgrade. We should do this at some point.

Reopening as #9234 got reverted in #9268. Now the task is to figure out why there is failures on staging and how to fix them, before reverting the revert.


add datatarget flag for vcfanno

Hi Brad
I feel like there should be a separate flag for the vcfanno and gemini files. Currently, if someone wants to use the vcfanno/vcf2db functionality, you need to add --datatarget gemini which downloads loads of gemini annotations based on GRCh37, which are useless on hg38. Hence the need for a separate download flag/or a check for which builds reference data is needed.
Thanks
M

Matthias -- good catch, thank you. I pushed a fix to the latest development version to only install the GEMINI data for build 37 if hg19/GRCh37 is in the specified genomes so it should avoid the extra downloads if you're only using hg38. Thanks again.


How can we upgrade laravel 5.5

Hey
I like your application, I am also working on extension over this application. Would you please let me know how can we upgrade laravel version to latest 5.5 lts?

Hi @usman101 , I had attempted to upgrade to v5.2 but didn't have time for it. There're some functionally changes in 5.2 onwards that need quite some effort. You could fork this project and do make the changes and submit a pull request. I'll be happy to look through it. :)

@usman101 also, I had begin some work on branch dev to support v5.2. Maybe you wanna look at that branch and see if we can upgrade to 5.5?

Please see issue #175 . Gonna jump to support version 5.5 instead of 5.2.


docs(appsync): add new router feature

Issue #, if available: #776
Description of changes:

Checklist

[ ] Meet tenets criteria
[ ] Update tests
[x] Update docs
[ ] PR title follows conventional commit semantics

Breaking change checklist

RFC issue #:

[ ] Migration process documented
[ ] Implement warnings (if it can live side by side)

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

View rendered docs/core/event_handler/appsync.md

Codecov Report

Merging #821 (594bbb6) into develop (0728aa2) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop     #821   +/-   ##
========================================
  Coverage    99.90%   99.90%
========================================
  Files          118      118
  Lines         5107     5107
  Branches       279      279
========================================
  Hits          5102     5102
  Misses           2        2
  Partials         3        3

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 0728aa2...594bbb6. Read the comment docs.


Dashboard Nav Label Disappearing

when hovering on dash, the text from hover nav gets covered as if its behind an invisible rectangle.
Expected when hovering over DASH:

Incorrect when hovering:

i explain this issue here - https://github.com/spacemeshos/smapp/issues/672

i explain this issue here - https://github.com/spacemeshos/smapp/issues/672


Investigate Active Directory Integration

#Privacy of Information and Protection:
Your opinion is your personal information. Please do not include any information which identifies you or others in your response.
#Description:
Investigate how we integrate with Active Directory  for new hires and employee movements around the organization.
#Outcome:

[ ]
[ ]
[ ]

#To Complete this ticket:

[ ]
[ ]
[ ]

@amoore-github Please report for stand up on your progress with this. Thx.


add tests for CLI tools

https://github.com/yargs/yargs/blob/main/docs/advanced.md#testing-a-command-module

The successor has extensive tests https://github.com/line-o/xst


at42qt1070: Initial implementation

This module implementes support for the Grove QTouch sensor.
Signed-off-by: Jon Trulson<EMAIL_ADDRESS>Signed-off-by: Zion Orent<EMAIL_ADDRESS>

Thank you!  Merged and closing.


Error enabling routeTracing in the angular 2 template

Hi,
often happens that if I add routeTracing
RouterModule.forRoot([
    { path: '', redirectTo: 'home', pathMatch: 'full' },
    { path: 'home', component: HomeComponent },
...
  ], { enableTracing: true }),

I obtain this error message
Exception: Call to Node module failed with error: TypeError: console.group is not a function
at SafeSubscriber._next (D:\Sorgenti\Interno\MyCrm\src\CrmApp\ClientApp\dist\vendor.js:59646:26)

The strange thing is that at first worked: I added this in the appmodule, recompiled and restarted and worked showing the data in the console but nowe, after a restart of vs, I recompile, start the app and obtain this error message

What browser are you using? console.group might not be supported there.

@lucamorelli Apologies, I just realized this is happening on the Node side.
This is because Node (or any server) doesn't have a method group on console.
You can require a package like https://github.com/Rich-Harris/node-console-group and even require it only during Development mode in your boot-server, that should get things working there.

Could you report this to either the Angular or Angular Universal projects? It's an issue in one of those, depending on how you choose to think about it.
Angular's enableTracing relies on the JS runtime having a console.group function, but that's not present in Node.js, hence it failing during server-side rendering. It would be good if either Angular didn't rely on an API that isn't available everywhere, or if Angular Universal included a polyfill for it so it was compatible with Node environments.
In the short term, you might be able to reference something like this to polyfill it yourself: https://github.com/Rich-Harris/node-console-group

Ah, @MarkPieszak beat me to it :)

So close too! :1st_place_medal:
😄

thanks, just saw the reply, installed and added to the boot-server
require('console-group').install();
and now seems to work


does it perform well on Name extraction?

Has anyone trained this model for extracting person name from pdfs? how was the result and how much data is sufficient for better performance?

I did, at first, I was trying to extract the whole name (first & last name), which proved to be quite unsuccessful (It returned a weird string like "a a ee a"). Then changed the "name" field into 2 different fields for first and last name and the results were pretty accurate. I was just trying to get the proof of concept, so I only used 25 PDF files as train data.


Newsletter May 2022

Here's the newsletter draft:

[x] Draft text
[x] GitHub stats data
[x] Flang updates by @AlexisPerry
[ ] LFortran updates by @certik @czgdp1807 or @HaoZeke

Let's plan to merge by Friday, May 6.
Thank you!

Please see https://github.com/milancurcic/fortran-lang.github.io/pull/7 for Flang updates.  Thanks!

@milancurcic would you like me to write a short announcement in May's newsletter for Modern Fortran migrating to fortran-lang?

@gnikit yes, please! I forgot about that. I think best to add it as a bullet point under events. Feel free to commit directly to this PR branch and add your name to the authors list. Thank you!

Done, I was unable to push to your branch so I opened a separate PR. I hope that's okay.

@czgdp1807, @HaoZeke is one of you able to put in LFortran updates?

I can do it tonight (my time) i.e., in 3 hours.

Done. See the LFortran updates in https://github.com/milancurcic/fortran-lang.github.io/pull/9

Thanks all! We have all the updates now. Let's merge later today if there are no objections.


Question? Where should I look in the source code to see all the splats?

I would like to play around with changing colors and removing splats from the Gaussian splat. Where would you recommend looking in the code for that?

SplatMesh.js is the file that contains the core of the rendering code, specifically in the vertex shader: https://github.com/mkkellogg/GaussianSplats3D/blob/77b4af26ef138aeb05323f64b0eb5410d562f3a7/src/SplatMesh.js#L136 as well as the fragment shader: https://github.com/mkkellogg/GaussianSplats3D/blob/77b4af26ef138aeb05323f64b0eb5410d562f3a7/src/SplatMesh.js#L341

I'll close this for now, but let me know if you need any more guidance with the code.

finally got the chance to look into this. Been playing around with the shaders to change the color.
Some things I'm looking for is where is the Root truth for a color? is it in one of the buffers and what is the structure of those?
And do you have any way you reccomend looking points that are around other splats? So if I raycast and get an index how possible would it be to get points around it? Would I need to implement kNN and just run it on all the points or does that already exist?

So for the "real" color of a splat, the source of truth is in the SplatBuffer object for a given splat scene. You can retrieve just color data from a splat buffer by calling the fillSplatColorArray() function.
For updating the color values, do you want to replace the whole texture or just update a sub-section of it? If you want to update/replace the whole texture, you need to do something like:
material.uniforms.centersColorsTexture.value = newtexture;
material.uniforms.centersColorsTexture.value.needsUpdate = true;

If you want to just update a sub-section of the texture, there are a couple ways to do it. You can use three.js' built-in method WebGLRenderer.copyTextureToTexture() (which I haven't personally used). Internally that function just calls the WebGL function texSubImage2D() to update a texture. I update the data textures in SplatMesh by calling texSubImage2D() directly in SplatMesh.updateDataTexture()

how would you recommend updating that data in splatBuffer? Im seeing in the code that there is lots more then color in the data. You get colors specifically by using fillSplatColorArray it seems. Would It be best to modify the splatBuffer, or update the textures that are stored in the mesh?
If i do update this.splatDataTextures.baseData.colors then do the actions to update the material, would that update the splats colors?

@mkkellogg what im looking at right now is fillSplatColorArray. Im trying to figure out how this creates the uint4array but it seems to be dependent on the to and from input info. How would you reccomend getting a uint8array of all the colors in the mesh?
additionally, it looks like just calling refreshGPUDataFromSplatBuffers will update the mesh too. Is this the best way or is there a faster method?

fillSplatColorArray() accepts as its first argument a parameter called outColorArray. That array should be an instance of Uint8Array, and it will get filled with color data from the SplatBuffer using the range values you specify (srcFrom & srcTo). destFrom is the start position in outColorArray to begin populating data.
Once the scene is completely loaded, this.splatDataTextures.baseData.colors will have all the color data extracted & uncompressed from the splat buffer. There is not currently a function that will just take that extracted color data and push it to the texture on the GPU, but you could totally write one :) It would be a lot like updateDataTextures(), except it wouldn't pull any data from the splat buffer. Actually updateDataTextures() should probably be renamed something like updateDataTexturesFromSplatBuffers() and it could call two new functions: one to extract & decompress the splat buffer data and another to push it to the GPU.

I got the function working for changing the color thanks. One thing to note, im not sure the updateDataTextures() works.
when I change the data using:
for all points...
this.material.uniforms.centersColorsTexture.value.source.data.data[colorDestBase] = rgbaArrayToInteger([r,g,b,a], 0);
this.material.uniforms.centersColorsTexture.value.needsUpdate = true;
then call mesh.udpateDataTextures()
the colors dont revert back to their original color. Is that expected behavior? I would assume since I changed the colors away from the splatBuffer it would have gone back to the original color

So you're right about updateDataTextures(), it works only during the progressive loading of a scene (I never intended it to be used after that). In theory, it should be able to be modified to work as I previously claimed it did. In many places you'll see references to this.lastBuildSplatCount and splatCount or splatCount - 1 as parameters to function calls within updateDataTextures() -- Those are just specifying a range of splats to do some update stuff with. When a scene is fully loaded, lastBuildSplatCount should be equal to splatCount so that's why you aren't seeing any effects. If you change those values (as appropriate) to be your desired range, that should make it work.

I should be able to do a pr. just wanna make sure its implemented well. One thing im working on now is implementing that tree search. Im using visitLeaves but that seems to only include a small portion of the nodes. Is there a better function?

this  was done using the SplatMesh.splatTree I assumed it would have been everything?

my bad. looks like the node has node.data.indexes... no need to answer


Ge prices like osbuddy

Feature already implemented / not enough information given.

We already have them, check GE plugin.


[Crash] Provide custom toolbar view.

I need another type of toolbar (for speech recognition, so no textView in it).
I subclassed JSQMessagesToolbarContentView and JSQMessagesInputToolbar. In JSQMessagesInputToolbar I implemented loadToolbarContentView and returns my initialized toolbar. But JSQMessagesViewController try to create JSQMessagesKeyboardController for it, and crashes on textView != nil assertion. How I can disable creating of keyboard controller?

@ypopovych can you please provide some more detail about how you replaced the stock input toolbar with your custom subclass? I can't figure out how to connect subclasses of those with JSQMessagesViewController, as the inputToolbar property, and its contentView are both read only. I've tried fiddling around with overriding the property and only achieved KVO-related crashes for my pains.


Mercurial support?

Q
A

Issue Type
Question

Deployer Version
master

Local Machine OS
N/A

Remote Machine OS
N/A

Is there a way to use it with Mercurial(hg) repo?
repository is mandatory in configuration, and it only accepts git, as I can see, but what to do, if I don't use git?

I have plans to add support for other vcs.

@curious-jim What kind of branching do you use? Common branches or bookmarks?

I use common branches

I think we can come in form of recipe, or plugin.

@elfet Mercurial is DVCS as git and have almost same API: pull, push, clone, merge, branch, hg update equals git checkout. So I guess that it'd better just create VcsInterface and GitVcs, MercurialVcs, etc. Also we can define concrete vcs type by repository env parameter. What do you think about it?

@pluseg nice idea, let me think about it a while. PS. interesting how this done in capistrano and ansible.

+1

any news?

Now you can override deploy:update_code with you own strategy:
task('deploy:update_code', function () {
    run('hg clone {{repository}} {{release_path}}');
});

Tested, working great. Thanks


Fixed #24154 -- Backends can now check support for expressions

This fixes a bug where complex aggregates may not have been checked by the backend. It also extends the utility to all expression types that may need to be checked.
Obviously correct expressions like Col do not opt in to checking - since they will work across all backends.
There are no docs because this is not a public API. 3rd party expressions should not opt into checking, because backends can not know if they are supported or not.
The backend method check_aggregate_support has been renamed to check_expression_support and a warning inserted. Unsure whether or not this should be documented in release notes separately, or just as part of #24106.

@claudep though it's not too different, does this look OK to you?

Looks totally fine for me. I've just complemented a GIS test to ensure your patch is properly functioning for GIS : d69ecf922dd

buildbot, test this please.

I'm happy with the overall design of this.

Once CI verifies the build, this should be ready for merge. The backport to 1.8 will have to remove the deprecation warning though (from backends/base/operations.check_aggregate_support).


SQLx is not taking into account the SET LOCAL ROLE query when verifying the subsequent queries.

Bug Description
When executing a SELECT query in a transaction, after executing SET LOCAL ROLE <role_name>, I am getting a compilation error with a message "error returned from database: permission denied for table user". The user I am using to connect to the postgres database does not have the privileges to execute the query. However, the user I am changing to inside the transaction using SET ROLE does have the privileges to execute the query and this can be confirmed by running the same queries in psql.
Minimal Reproduction
Structure of the database:
CREATE EXTENSION postgis;

CREATE ROLE superman WITH LOGIN SUPERUSER;
SET ROLE superman;

CREATE ROLE migrator WITH LOGIN NOINHERIT CREATEROLE;
GRANT CREATE ON DATABASE db1 TO migrator;
GRANT USAGE, CREATE ON SCHEMA public TO migrator; -- TODO: avoid usage of public schema
SET ROLE migrator;

CREATE SCHEMA IF NOT EXISTS web;
CREATE SCHEMA IF NOT EXISTS auth;

CREATE ROLE authenticator WITH LOGIN NOINHERIT;
GRANT USAGE ON SCHEMA auth TO authenticator;

CREATE ROLE webuser WITH NOINHERIT;
GRANT USAGE ON SCHEMA sthawebnik TO webuser;

CREATE ROLE anon WITH NOINHERIT;
GRANT USAGE ON SCHEMA web TO anon;

GRANT webuser TO authenticator;
GRANT anon TO authenticator;

CREATE TABLE web.user (
  id SERIAL PRIMARY KEY,
  name VARCHAR(255)
);

Steps to verify in psql
SELECT current_user; -- verifies that the current user is authenticator
SELECT * FROM web.table1; -- verifies that authenticator cannot SELECT on web.table1
BEGIN; -- starts a transaction
SET LOCAL ROLE webuser; -- changes the role for the duration of the transaction to webuser
SELECT current_user; -- verifies that the role has been changed to webuser
SELECT * FROM web.table1; -- verifies that as a webuser, one can SELECT on web.table1

Rust code to reproduce the SQLx error
use sqlx::postgres::PgPoolOptions;

#[tokio::main]
async fn main() -> Result<(), sqlx::Error> {
    let database_url = env!("DATABASE_URL");

    let pool : sqlx::Pool<sqlx::Postgres> = PgPoolOptions::new()
        .max_connections(20)
        .connect(database_url)
        .await?;

    let mut tx = pool.begin().await?;

    sqlx::query("SET LOCAL ROLE webuser")
        .execute(&mut *tx)
        .await?;

    let result = sqlx::query!("SELECT * FROM web.table1")
        .fetch_all(&mut *tx)
        .await?;

    tx.commit().await?;

    Ok(())
}

Info

SQLx version: 0.7.4
SQLx features enabled: [ "runtime-tokio", "tls-rustls", "postgres", "macros" ]
Database server and version: [REQUIRED] Postgresql 15.6
Operating system: GNU/Linux
rustc --version: rustc 1.77.0 (aedd173a2 2024-03-17)

No queries are executed during compilation, only prepared. Regardless, the user being used must be defined in the DATABASE_URL during compilation or preparation of the query data (cargo sqlx prepare). As the code is not ran during compilation, changing the user wouldn't do anything to subsequent queries.

@mehcode Does this mean that one cannot use sqlx for Postgrest style transactions with RLS that uses the SET LOCAL ROLE feature as its authentication mechanism? Trying to do this will most definitely result in compilation errors, although the SQL itself is valid for Postgres. Too sad I would have to switch to another library if this is true. Please let me know.

You can always use  non compile time checking query.
That will work just fine. It is not like you have some other option that does compile time checking and supports somehow SET LOCAL ROLE (at least from what I know).


Automatically log uploaded file URLs to output + quick copy functionality

Closes #1027
In this example, we "upload files" to the blob container which I have named "lob" 😆, we see the output log shows the file URLs now.  We get an information message pop-up that allows you to copy the most recent file URL to the clipboard.

If you click the "Copy to Clipboard" button, the following will happen as the next step:

Implementation - I made two main changes/additions:

I split up the copyUrl tree command in BlobContainerTreeItem & FileShareTreeItem into two components => copyUrl & getUrl

I did this because I needed a way to extract the URL without automatically copying to clipboard

I created a new util upload function to handle the core functionality of the issue - outputAndCopyUploadedFileUrls() => main logic found there

Thanks a lot @MicroFish91 👍


[Integrated-Test] Add combination integrated test

Description
To test scenes of mixing plugins, we can add the integrated test to shenyu-integrated-test-combination.
IMO, the following test should be covered. But, if you have more idea, please propose and do it~
Task List
Combination of Response Plugins

[ ] test combination of Modify Response and Cryptor Response plugin.
...

Combination of Request Plugins

[ ] test combination of Param mapping and Rewrite plugin. @lxl910128
[ ] test combination of Param mapping and Redirect plugin. @lxl910128
[ ] test combination of RateLimiter and Rewrite plugin. @lxl910128
[ ] test combination of RateLimiter and Redirect plugin. @lxl910128
[ ] test combination of Request and Redirect plugin. @lxl910128
[ ] test combination of Request and Rewrite plugin. @lxl910128
[ ] test combination of JWT and RateLimiter plugin. @lxl910128
[ ] test combination of JWT and Redirect plugin. @lxl910128
[ ] test combination of JWT and Rewrite plugin. @lxl910128
...

Combination of Request and Response Plugins

[ ] test combination of Cryptor Request and Cryptor Response plugin. @songyuequan
...

Combination of Rpc and Request Plugins

[x] test combination of Dubbo and Param Mapping plugin. @JooKS-me
[x] test combination of Dubbo and Cryptor Request plugin. @JooKS-me
[x] test combination of Dubbo and Ratelimiter plugin. @JooKS-me
...

Combination of Rpc and Response Plugins

[ ] test combination of Dubbo and Modify Response plugin. @JooKS-me
[ ] test combination of Dubbo and Cryptor Response plugin. @JooKS-me
...

PS: Request plugin means the plugin acting on request, and so on.

let me try Combination of Request Plugins

let me try Combination of Request Plugins

Thanks for help!

can I try  test combination of Cryptor Request and Cryptor Response plugin ?

let me try Cryptor Request and Cryptor Response plugin.

let me try Cryptor Request and Cryptor Response plugin.

Welcome~

hi~ , I have finishedCryptor Request and Cryptor Response plugin . please assign some else to me.

hi~ , I have finishedCryptor Request and Cryptor Response plugin . please assign some else to me. @JooKS-me
Thanks, could you take  Combination of Response Plugins and Combination of Rpc and Response Plugins ?

In Combination of Rpc and Response Plugins you can not only test dubbo plugin ,but more rpc plugins, such as grpc, sofa and so on.

hi~ , I have finishedCryptor Request and Cryptor Response plugin . please assign some else to me. @JooKS-me

@songyuequan Thanks, could you take Combination of Response Plugins and Combination of Rpc and Response Plugins ?
In Combination of Rpc and Response Plugins you can not only test dubbo plugin ,but more rpc plugins, such as grpc, sofa and so on.

OK,   I  would  like   to  do  it.  thank you


KrisJordan

you made such an amazing framework, why did you stop.
please start work on it back again. there is lot.
with all the mobile in the scene, this makes a lot of sense to have a framework like this

I'd like to second reejossamuel's sentiment


Find references: show number of children similar to search results

Similar to how search results show a number for their children:

Would be cool to have the same for references view:

Would probably need some API on the custom view layer.
//cc @sandy081

I like but it might look noisy with the those git-decoration on the right

Oh wow, I never realized that search results actually do not have decorations 🙉


Allow specifying the controller name in ControllerAttribute

This is a follow up issue for #3732. See https://github.com/aspnet/Mvc/pull/4316#discussion_r56843279 for more details.
Currently we can mark that a class is a controller using ControllerAttribute as shown below.
[Controller]
public MyControllerClass
{
    ...
}

This bug is to allow the user specifying the name of the controller as part of marking it a controller:
[Controller("ControllerName")]
public MyControllerClass
{
    ...
}

FYI @javiercn @rynowak moving to backlog.

Yes, we can decide to tackle this at a later point in time. It's orthogonal to #4316

Closing because we have no plans to support this.


The color output is not displayed correctly.

It works with:
conpty.Start(`ssh -p 1022<EMAIL_ADDRESS>
But, the output is not displayed correctly:
]0;C:\Windows\System32\OpenSSH\ssh.exe]0;OpenSSH SSH clientLast login:

We should do that outside the pty.
I need the original output, so don't change it.
Here is my solution: https://github.com/trzsz/trzsz-go/blob/main/trzsz/pty_windows.go


Grab obj models

Hello!
I am trying a modified version of the vive example and added a fourth object: a custom obj.
It shows up with a box for a bounding box, and I can hit it with my controller colliders, but I can't grab it, even though it has the same 'cube' class as the cubes do.
Would be happy to post sample code.
Thank you!

The problem is that the custom OBJ has more than one mesh embedded in it, and the sphere-collider component doesn't look at all the nested stuff to compute a bounding sphere.
So, https://github.com/donmccurdy/aframe-extras/issues/156 is the underlying issue — will be able to fix that for sure, but I haven't been able to get to it yet. :)

Ah ok thanks! I guess I was expecting it to collide with the red bounding box and not the custom OBJ geometry, which would be fine for what I am trying to test.  Thanks anyway!
Do you happen to have any suggestions of other a-frame components to look at that allow vive-grabbing of custom objs?
Thanks again!

The versions of grab and aabb-collider components used in the aframe tracked controls example (https://github.com/aframevr/aframe/tree/master/examples/showcase/tracked-controls) should work with custom OBJs. But you'll probably lose physics in that case.
Hopefully/maybe I should be able to look at this more this weekend 🙂

Okay thanks very much! I will check that example out. But yeah, the physics is definitely the interesting aspect for me here! The vive example here in aframe-extras is really great, what is implemented so far!
Just to confirm, it does seem like my custom OBJ here gets a red-mesh bounding box, and I can collide with it, just not grab it (because of the issue you mention), does that sound right to you?

Just to confirm, it does seem like my custom OBJ here gets a red-mesh bounding box, and I can collide with it, just not grab it (because of the issue you mention), does that sound right to you?

Exactly right — grabbing doesn't use physics for collision detection (if it did, your hand would have to go inside the object, which doesn't work if you want to also push the object around), and the non-physics collision detection it uses instead has a bug.

Excellent, thank you! Makes total sense to me now! So interesting! Ok I will sit tight..

I was able to approximate my model with combined a-frame primitives, and the grabbing works there for the most part! :smile:
on the left, obj that is not grab-able --> on the right, a-frame model that is grab-able

Should be fixed, i think!

Oooh yay!  Thank you so much! This is great!


Bug: when i try to test i shows lines to scroll

Issue Type: Bug
When i try to test using the Run In Terminal it works its just that when i click on the rocket icon it has theres weird lines to scroll i have the width and height i used    wn.setup(width=800, height=600)       i used the import turtle to make the program to work

i am on a MacBook Pro M1 Chip

We don't have a rocket icon in the extension or VS Code, so I believe the issue you are having is due to the libraries you are using and not VS Code itself.


Bringing domain-name on par with the rest of the optional configurations

Addition to #698, this brings the domainName() configuration property to feature parity with similar properties, like deploymentName.

perhaps @duchamk should also check this out?

I tested from the bugfix/improve-azure-domain-configuration branch.
For the configuration quarkus.langchain4j.azure-openai.domain-name=azure-api.net does not work, it inserts the domain openai.azure.com. However, for configuration
quarkus.langchain4j.azure-openai.chat-model.domain-name=azure-api.net
quarkus.langchain4j.azure-openai.embedding-model.domain-name=azure-api.net
quarkus.langchain4j.azure-openai.image-model.domain-name=azure-api.net

everything works fine.
I've looked through the code and I don't know why it doesn't work for the parent configuration.

I tested from the bugfix/improve-azure-domain-configuration branch. For the configuration quarkus.langchain4j.azure-openai.domain-name=azure-api.net does not work, it inserts the domain openai.azure.com. However, for configuration
quarkus.langchain4j.azure-openai.chat-model.domain-name=azure-api.net
quarkus.langchain4j.azure-openai.embedding-model.domain-name=azure-api.net
quarkus.langchain4j.azure-openai.image-model.domain-name=azure-api.net

everything works fine. I've looked through the code and I don't know why it doesn't work for the parent configuration.

Can you please attach a reproducer including your entire application.properties?

@duchamk It seems that due to naming conventions, there are some conflicts between the values of the parent configuration, and the named configurations. It's fairly complicated to fix, so after discussing with @geoand we resorted to using "dummy" defaults in the embedded models, and filtering the value selection in the code by comparing them with the default Dummy value.
I made some minor corrections by leaving as much of the code intact at this point, until a better solution comes into play.
@duchamk, can you please check the new configuration and see if this works as expected?

Thanks for taking this up!

@duchamk Can you please attach a reproducer including your entire application.properties?

I tested from the bugfix/improve-azure-domain-configuration branch. For the configuration quarkus.langchain4j.azure-openai.domain-name=azure-api.net does not work, it inserts the domain openai.azure.com. However, for configuration
quarkus.langchain4j.azure-openai.chat-model.domain-name=azure-api.net
quarkus.langchain4j.azure-openai.embedding-model.domain-name=azure-api.net
quarkus.langchain4j.azure-openai.image-model.domain-name=azure-api.net

everything works fine. I've looked through the code and I don't know why it doesn't work for the parent configuration.

@duchamk Can you please attach a reproducer including your entire application.properties?

# server
quarkus.http.http2=true
quarkus.http.access-log.enabled=true
quarkus.http.access-log.pattern=%{REMOTE_HOST} %l %{REMOTE_USER} %{DATE_TIME} "%{REQUEST_LINE}" %{RESPONSE_CODE} %b %{RESPONSE_TIME}
quarkus.http.record-request-start-time=true
# rest client
quarkus.rest-client.http2=true
quarkus.langchain4j.log-requests=true
quarkus.langchain4j.log-responses=true
# azure AI
quarkus.langchain4j.azure-openai.api-key=mykey
# embed 1536
quarkus.langchain4j.azure-openai.resource-name=secret
quarkus.langchain4j.azure-openai.domain-name=azure-api.net
#quarkus.langchain4j.azure-openai.chat-model.domain-name=azure-api.net
#quarkus.langchain4j.azure-openai.embedding-model.domain-name=azure-api.net
#quarkus.langchain4j.azure-openai.image-model.domain-name=azure-api.net
quarkus.langchain4j.azure-openai.embedding-model.deployment-name=usersandbox-open-ai-automated-deployment
quarkus.langchain4j.azure-openai.embedding-model.max-tokens=120000
#opl-usersandbox-open-ai-automated-deployment/embeddings?api-version=2023-05-15
quarkus.langchain4j.azure-openai.embedding-model.api-version=2023-05-15
quarkus.langchain4j.azure-openai.deployment-name=gpt-35-turbo
quarkus.langchain4j.azure-openai.chat-model=gpt-35-turbo
quarkus.langchain4j.azure-openai.api-version=2024-02-15-preview
quarkus.langchain4j.azure-openai.log-requests=true
quarkus.langchain4j.azure-openai.log-responses=true
# 1.0
quarkus.langchain4j.azure-openai.chat-model.temperature=0.2
# 1.0
quarkus.langchain4j.azure-openai.chat-model.top-p=0.1
# qdrant
quarkus.langchain4j.qdrant.host=localhost
quarkus.langchain4j.qdrant.collection.name=demo-test
# easy rag
quarkus.langchain4j.easy-rag.path=src/main/resources/documents
quarkus.langchain4j.easy-rag.max-segment-size=300
quarkus.langchain4j.easy-rag.max-overlap-size=30
quarkus.langchain4j.easy-rag.max-results=5
quarkus.langchain4j.easy-rag.path-matcher=glob:**
# build
quarkus.native.additional-build-args=--initialize-at-run-time=io.grpc.netty.shaded.io.netty.util.internal.logging.Log4JLogger

@duchamk It seems that due to naming conventions, there are some conflicts between the values of the parent configuration, and the named configurations. It's fairly complicated to fix, so after discussing with @geoand we resorted to using "dummy" defaults in the embedded models, and filtering the value selection in the code by comparing them with the default Dummy value.
I made some minor corrections by leaving as much of the code intact at this point, until a better solution comes into play.
@duchamk, can you please check the new configuration and see if this works as expected?

No changes, same as before.

Can you please rebase this onto main in order to pick up a CI fix?

@duchamk can you attach the sample you are using that shows the problematic behavior?
Asking because the latest version of this PR seems to me to work as expected

I will can do it after 14.07, I'm on vacation now.

No problem, in that case we'll merge this for the time being

@csotiriou can you rebase please?

Just made the changes. From what we have been discussing with @geoand , there are some conflicts in the namings of the configuration properties, which - if the next version wants to keep backwards compatibility with properties - is non-trivial and dangerous to try to solve.
From what I can gather from george, there may be a possible fix in the future, but it's somewhat dangerous to try to squeeze it right now into what we have.

I actually don't think there are any conflicts with your latest fix

@duchamk can you attach the sample you are using that shows the problematic behavior? Asking because the latest version of this PR seems to me to work as expected.
Thanks

@geoand
I've included a simple example here https://github.com/duchamk/quarkus-ai-azure

Thanks

Which exactly is the problematic part you are seeing in that sample?

For configuration in application.properties:
# for all models - don't works
#quarkus.langchain4j.azure-openai.domain-name=azure-api.net
# for each one model - works
quarkus.langchain4j.azure-openai.chat-model.domain-name=azure-api.net
quarkus.langchain4j.azure-openai.embedding-model.domain-name=azure-api.net
quarkus.langchain4j.azure-openai.image-model.domain-name=azure-api.net

RAG, embedding and chat work properly, the domain is entered correctly.
However, if we change the configuration to:
# for all models - don't works
quarkus.langchain4j.azure-openai.domain-name=azure-api.net
# for each one model - works
#quarkus.langchain4j.azure-openai.chat-model.domain-name=azure-api.net
#quarkus.langchain4j.azure-openai.embedding-model.domain-name=azure-api.net
#quarkus.langchain4j.azure-openai.image-model.domain-name=azure-api.net

then the domain stops substituting.
This is probably related to the problem that @csotiriou wrote about.

I tried your sample using mvn quarkus:dev and I got:
 io.netty.channel.AbstractChannel$AnnotatedConnectException: Connection refused: localhost/<IP_ADDRESS>:443

Am I missing something?


Add a download() function into the RepoAPI object

RepoAPI should have a download() function that utilizes the common.download.download_files() function to download the whole repo or a directory in a repo in a parallelized fashion.
repo = RepoAPI("user/repo")
repo.download(outdir="/home/user/my-repo")
# OR
repo.download("data/images", "/home/user/images")

Probably should be recursive by default with the ability to turn it off

Hello @kbolashev , please assign this issue to me as I have already worked on this kind of problem in past and has a great experience.

@itssiddhantjain Feel free to make a PR for whatever issue you want to tackle 👍

Implemented in https://github.com/DagsHub/client/pull/389


[Proposal] Normalized Cost Balancer

Adding a new cost balancer which normalizes that the cost function described in CostBalancerStrategy.java
The new normalized balancer ensures that distribution of segments remains fairly good even when we have disks of different capacities across hosts.
The interface BalancerStrategyFactory has been modified and now takes a ListeningExecutorService as an input to create a strategy rather than having a DateTime object.
The DateTime object that was earlier being passed wasn't serving any purpose and has now been removed.
The type of cost balancer can be changed by specifying a new property ("druid.coordinator.balancer.strategy") in the runtime.properties file. The default has been set in such a way that the already existing CostBalancerStrategy will be used.

Note that #3 here is a backwards incompatible interface change to something that can potentially be changed in an extension.  I'm not sure who actually extends the interface, but I suspect it would be either MMX or Imply if anyone.
Sticking with semantic versioning, this change should result in 0.10.x.  Are we ready for the next release to be 0.10?

I'm 👍 on these changes.  But we need to discuss the implications of the interface change and agree on versioning strategy.

I'm cool with 0.10.0
Gives us a chance to introduce a few more backwards incompatible changes that have been stalling (double columns, etc)

cursory comment: can the name be changed to "diskNormalizedCostBalancer"

Re: the 0.10 question, this doc: http://druid.io/docs/latest/development/modules.html suggests that cost balancers are not an expected extension point and so the version bump may not be needed. We could also clarify that doc from "suggests" to "says very clearly" as part of this pr.

My feeling is that it's fine to do this change in 0.9.3 since we did not set an expectation in the docs that cost balancer strategies are a stable API for extensions.

Btw, we have this balancer running on our coordinators in production and it is keeping disk usage on our heterogenous set of historicals in alignment.

👍

@dclim @xvrl can one of you please review this and +1?  It's an opt-in, configurable piece of code that we have been running in production for a few weeks now doing what we expect.

@cheddar I'm still waiting on a response to my comment here https://github.com/druid-io/druid/pull/3632/files#r90071380

@xvrl Added the requested comments here https://github.com/druid-io/druid/pull/3632/files#r90071380

@niketh do you mind filing a github issue for the suggestion to try out so we don't forget?

@xvrl Created the issue here https://github.com/druid-io/druid/issues/3742 , will update it this week.


Add Buttono

Buttono is a Sass mixin based on BEM naming convention. It helps you to create nice buttons in an easy way.

Looks awesome, Hasan! 👍


add blurb to readme about non-support of JSON.stringify options

Since this is clearly a popular ask (given all the closed PRs), added this to README.

Done! Wish there was a markdown style/linter.


Improve Android NDK symbolication documentation - CMake support

The documentation says this:

Option 1: Upload native binaries
Put all .so files from the project's obj/local/$ABI/ directory into a .zip file.

There is no such directory in my build tree. The documentation does not mention that this directory is specific to ndk-build. My app is built with CMake. CMake based Release builds strip the debug symbols from the .so files. I don't know where the unstripped .so files are located in a CMake based build, or even whether they at all exist after the build has finished.
The documentation should indicate that this instruction is for ndk-build based builds, as well as provide equivalent instructions for CMake based builds.

According to https://developer.android.com/build/shrink-code#native-crash-support, the unstripped .so files' location in a CMake based build is dependent on the version of the Android Gradle plugin.
Android Gradle plugin version 4.0 or earlier
app/build/intermediates/cmake/universal/release/obj/
Android Gradle plugin version 4.1 or later
Undocumented, variable path. In my particular case it was app/build/intermediates/cxx/RelWithDebInfo/e2gf2c5g/obj/arm64-v8a/

Even after uploading all of my .so files in a zip, my Android native crashes are not getting symbolicated.


PDF - do not display grace days field in the pasture section

Grace days entry in pasture automates schedule row entry for staff
Grace days can be overwritten by staff in schedule rows
Do not want situation where there are two different grace day values displaying (what do you follow? -- the schedule is the legal one)
Therefore, do not display grace days in pasture section

Duplicate of #554


assert in function driver_exec()

Vireo build: 3881+
After fixing issue #31, the driver_exec() function asserts with the following output:

The problem seems to be an issue with the ISO9660 driver (TYPE_FS | ISO9660 == 0x2000096), which it may not be able to find on systems with more than 1000 MB of memory. However, it may also indicate a different problem
This does not necessarily seem to be an issue that was created by the fix for issue #31. The only thing changed by the fix for #31 was that the shadow map was placed after the paging tables instead of being allocated at the start of kmalloc memory (0x200000, due to being the being the first thing allocated by kmalloc).
It may therefore be a stability issue, perhaps introduced a long time ago.

To keep track of stability issues, #21 was created. It acts as some sort of timeline.

Edit: I have edited this issue since the information given previously was incorrect (driver_exec() does NOT receive memory pointers, it was simply given the ISO9660 driver type number).

The issue was caused by the IDE driver checking if the argument given (a pointer to where the IDE drive types should be copied) was a drive of type PATA, which is not possible. It is an issue that was introduced some time ago, however the kernel somehow did manage to work with that check while having only 4MB of memory.
Because the drive list was never copied, there was no PATAPI device found. This meant that cd_init() (responsible for registering the ISO9660 driver) returned without adding the ISO9660 driver. main() would, however, search for breaker.elf using the ISO driver even though the driver was never registered.


Ferolli/hyundai heatpump

Device type and model (or SN)
Ferroli omnia m 3.2 hi 3.8
The description of new feature
unfortunatelly this does not work anymore with my ferolli omnia 3.2 heatpump, i only get this message "No new available appliances found on the network" :(

Alternatively, have you tried adding the heat pump manually with the IP address?

Try version 0.5.7, a critical bug was fixed.

yea, i updated to 0.5.7 and tried via manual ip, but my heatpump is still not found :(

yea, i updated to 0.5.7 and tried via manual ip, but my heatpump is still not found :(

Use the real IP of the heat pump (check your router), not “auto”.

Also, please provide your logs.
https://github.com/wuwentao/midea_ac_lan?tab=readme-ov-file#debug

no waaayyyy, so i restarted my deco network and reintegrated the pump controller(i think this is the winning point), tried to add it via manual ip and almost works, still have a few "unknown" sensors but this gets solved in time(same issue as the older repo).
ps: auto still does nothing :D
many thanks