Add support of stream migration during slot migration process

The XSETID command was implemented. This command is used only during stream key migration.
It differs from the same Redis command just in one case: in Redis, you can't apply it to a non-existing stream; in Kvrocks you can because streams are allowed to be empty and there should be a way to migrate an empty stream (with no prior XADD command which can create a stream if it doesn't exist).
Related C++ unit tests were added.
Stream key migration was implemented. The key of the  stream  type is migrated via a separate function.
Related Go tests were added.
Refactor and tidy cluster-related code (src/cluster folder).
Closes #1087

Thanks all. Merging...


HAWQ-853. Master standby should avoid incomplete split operation

Please review, thanks!

LGTM

+1

+1 LGTM


Setup build and release actions

Description
Created github actions for standard build and release logic

This is superseded by #30 and should be able to be closed when it's not needed for reference anymore


Add NAACL 2024 / Edit EACL 2024

Added NAACL 2024
Edited EACL 2024

Added h-index for EACL, following google scholar metrics

Thanks for contributing!


Platform and project metadata and tasks house-cleaning

A number of files on the platform were previously managed manually but have not been updated recently. Some of these manually managed files have since been migrated to automated updates using this WARN scraper. Additionally, some stray ETL-related file artifacts have crept into the project.
Time to do some housing-cleaning to bring all our metadata up to date and clean up unneeded files from the platform project.

[ ] Update the WARN metadata spreadsheet
[ ] Update Project Board to align with metadata spreadsheet (create tickets where needed)
[ ] Back up all current files from production project bucket to the data-etl bucket using project ID
[ ] Delete all files that are duplicates or ETL-related file artifacts. Leave the files that were manually updated, but create tickets to automate data acquisition.
[ ] Add a disclaimer to project description saying we're in the process of updating scrapers and data files, so approach the data with caution

I am going to solve one issue here with #327.


feat(homepage): Reverting certain changes

🔧 What does this fix? 🔧
Fixes certain issues from PR #915.
Base background colour revert to slightly lighter colour(base), in accordance with catppuccin userstyle guide.
Base box colour revert to slightly darker colour (mantle), in accordance with catppuccin userstyle guide.
Reverted hover colours to accent colour, else accent colour is completely absent from the styling.
Invite @BPplays (original PR author) for any comments.
🗒 Checklist 🗒

[x] I have read and followed Catppuccin's contributing guidelines.
[x] I have updated the version appropriately in the ==UserStyle== header of the catppuccin.user.css file.

@Gandalf-the-Blue sorry for the late response,
i changed the background color to mantle and the box color to base because i didn't like the boxes being darker,
and i think surface0 for the boxes, and surface1 for the things on top of the boxes like number of movies looks bad. it also seems to have bad contrast on the numbers with no other style changes.
i changed the accent color highlight because it also affects stuff like number of tv shows, and stats like that but it has a weird dark color on those elements


2.8.2104 Occasional Redis Hangs  Out Of Memory allocating

We have a single standalone Redis for Windows node running on a server.
Two other server are communicating with this instance.
We use it with SignalR and for caching.
The dump has a size of about 700MB
From time to time we have hangs for 1-3 minutes. After this it recoverrs by it self.
The error seems to only occur when there is some traffic on our page.
In this time we get the exception you can see below

StackExchange.Redis.RedisConnectionException: SocketFailure on EVAL
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task
task)  at
System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task
task)  at Microsoft.Owin.Cors.CorsMiddleware.d__0.MoveNext()
--- End of stack trace from previous location where exception was thrown ---  at
System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task
task)  at
System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task
task)  at Microsoft.Owin.Mapping.MapMiddleware.d__0.MoveNext()
--- End of stack trace from previous location where exception was thrown ---  at
System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task
task)  at

When I search through the Redis log i can ocasssionally find these errors:

[33144] 18 Feb 18:18:44.843 #
=== REDIS BUG REPORT START: Cut & paste starting from here === [33144] 18 Feb 18:18:44.844 # Out Of Memory allocating 308457 bytes. [33144]
18 Feb 18:18:44.844 # --- ABORT [33144] 18 Feb 18:18:44.844 # ---
STACK TRACE
redis-server.exe!LogStackTrace(c:\release\redis\src\win32_interop\win32_stacktrace.cpp:95)(0x00000016,
0x042E0028, 0x00000000, 0x00000001)
redis-server.exe!AbortHandler(c:\release\redis\src\win32_interop\win32_stacktrace.cpp:206)(0x00000001,
0x89EE7767, 0x40150880, 0xBB7A5ED7)
redis-server.exe!raise(f:\dd\vctools\crt\crtw32\misc\winsig.c:587)(0x00000001,
0x00000000, 0x0004B4E9, 0x042E0028)
redis-server.exe!abort(f:\dd\vctools\crt\crtw32\misc\abort.c:82)(0x00000001,
0x4013F888, 0x0004B4E9, 0x00008000)
redis-server.exe!redisOutOfMemoryHandler(c:\release\redis\src\redis.c:3397)(0x0004B4E9,
0x4007DA07, 0x042E0028, 0x4007A27B)
redis-server.exe!zmalloc(c:\release\redis\src\zmalloc.c:147)(0xBDF01150,
0x4007EB2C, 0xBDF01150, 0x446D6B10)
redis-server.exe!sdsnewlen(c:\release\redis\src\sds.c:59)(0xBDF01150,
0xBDF01150, 0x3E74FD95, 0x00000003)
redis-server.exe!_addReplyStringToList(c:\release\redis\src\networking.c:271)(0xBDF01150,
0xBDF01150, 0x042E0028, 0x400E34FE)
redis-server.exe!addReplyBulkCBuffer(c:\release\redis\src\networking.c:517)(0xFFFFFFFF,
0x042E0028, 0x01B77260, 0x01B77260)
redis-server.exe!luaReplyToRedisReply(c:\release\redis\src\scripting.c:792)(0x00000004,
0xBDF01150, 0x00000002, 0x00000002)
redis-server.exe!luaReplyToRedisReply(c:\release\redis\src\scripting.c:839)(0xFFFFFFFF,
0x00A7F690, 0x67897B20, 0xBDF01150)
redis-server.exe!evalGenericCommand(c:\release\redis\src\scripting.c:1048)(0x71E66870,
0x00000000, 0x00000001, 0x000000B2)
redis-server.exe!call(c:\release\redis\src\redis.c:2016)(0x56C60B04,
0x4008B000, 0x00000000, 0x000000B2)
redis-server.exe!processCommand(c:\release\redis\src\redis.c:2235)(0xBDF01150,
0x000000B2, 0x000023B5, 0x00000001)
redis-server.exe!processInputBuffer(c:\release\redis\src\networking.c:1274)(0xBDF01150,
0x00000000, 0x00000000, 0x00000001)
redis-server.exe!readQueryFromClient(c:\release\redis\src\networking.c:1329)(0xFFE51650,
0x00000001, 0x44726F20, 0x0000012C)
redis-server.exe!aeMain(c:\release\redis\src\ae.c:487)(0x56C5C7F8,
0x00000002, 0x00000000, 0x00000002)
redis-server.exe!redis_main(c:\release\redis\src\redis.c:3524)(0x0024BA50,
0x00000002, 0x56C5C7EB, 0x00000002)
redis-server.exe!main(c:\release\redis\src\win32_interop\win32_qfork.cpp:1363)(0x00000016,
0xFFFFFFFF, 0x00000016, 0x0023F3A0)
redis-server.exe!ServiceWorkerThread(c:\release\redis\src\win32_interop\win32_service.cpp:485)(0x4000B3D0,
0x00000000, 0x00000000, 0x00000000)
KERNEL32.DLL!BaseThreadInitThunk(c:\release\redis\src\win32_interop\win32_service.cpp:485)(0xBB0113B0,
0x00000000, 0x00000000, 0x00000000)
ntdll.dll!RtlUserThreadStart(c:\release\redis\src\win32_interop\win32_service.cpp:485)(0x00000000,
0x00000000, 0x00000000, 0x00000000)
ntdll.dll!RtlUserThreadStart(c:\release\redis\src\win32_interop\win32_service.cpp:485)(0x00000000,
0x00000000, 0x00000000, 0x00000000) [33144] 18 Feb 18:18:44.857 #
=== REDIS BUG REPORT END. Make sure to include from START to END. ===

maxheap is set to 3000mb
the server has a total of 64 GB RAM and about 10GB were free
Looking at the error message _ Out Of Memory _ does this mean redis reached the maxheap in this moment?
There is also one more thing , but i'm not sure wether it is really realted to the problem.
Most of the time the problem increases its  frequency. Then when I reset the iis of one of the server the problem is gone for hours or days completly. I thought about there may be hanging / stacking signalR queues. But I don't have any further signs that this may be the case.
I Also attached the output of the redis info ( but was about an hour after the error)
redisinfo.txt
Is there any thinh i can log when the server is in this state?
Any hints about that?
PS: I also posted on SO (if someone iscollecting points) http://stackoverflow.com/questions/35489732/redis-occassional-hangs

Hi @Gentlehag

Hi @Gentlehag
Redis on Windows 2.8 uses a memory management architecture that is prone to this kind of problems.
The amount of free RAM is influential, the only thing that counts is the fragmentation of the preallocated maxheap (see the wiki for details), one the fragmentation reaches a critical point the memory allocator (dlmalloc) can't do anything about it. That problem has been fixed in 3.0, there isn't anymore a maxheap limitation, the heap is allocated on demand until all system resources are exhausted (if maxmemory is not set). 3.0 also uses a different allocator (jemalloc) that does a better job at preventing the heap fragmentation to grow to fast.
So my suggestion is to consider switching to 3.0.
Thank you.

We currently have this hang each 2-5 hours. sometime redis even don'T recover it (we have cuirrently more data in ram)
When upgraden to redis 3.0 would the dump survive? and If I would have to downgrand can I use the dump which was modified with 3.0 again ?
And some time ago in an other issue (https://github.com/MSOpenTech/redis/issues/379)  you told us that you can provide a version with extended logging, could you create it for me so if I have this problem then you could see what it is?   Or are there some tradeoffs with this logging?
Our cache is very valuable and I wan't to avoid it if not necessary.

@Gentlehag

When upgraden to redis 3.0 would the dump survive? and If I would have to downgrand can I use the dump which was modified with 3.0 again ?

Redis on Windows has the same specification of Redis on Linux in regard of the dump file format and compatibility. Please refer to the official Redis documentation: http://redis.io/documentation

And some time ago in an other issue (#379) you told us that you can provide a version with extended logging, could you create it for me so if I have this problem then you could see what it is? Or are there some tradeoffs with this logging?

We provide private builds for issues occurring with the latest release (currently 2.8.2400 and 3.0.501) if there is a bug report specific to one of those releases.
The extra logging trade-offs are: the log will increase in size and may have a slight impact on performance.

could you provide me this private biuld ? i'Ve the feeling that there is a high chance to get again this bug and in this case it would be very good to not have it twice run in this problem

@Gentlehag
you want a private build for the issue 379?

for the most current 3.0 that i should use to update :-) i'd assume its the 3.0.501 isn't it ?

@Gentlehag
there is a private build with extra logging prepared for a similar issue available here: http://1drv.ms/1oNafsd
I suggest to close this issue and keep track of any problem with the private build in a dedicated issue.
Thank you.

Thank you very much !


fix gotip failed test

Ref https://github.com/schemalex/schemalex/pull/82#issuecomment-521195843

oh failed by https://github.com/schemalex/schemalex/pull/82 reason.
So, I merge this PR to https://github.com/schemalex/schemalex/pull/82


Help screen errors

When running cat --help, I get a Node.js error.
This error appears on all cash programs, and seems to be related to the parser.
Error:
(node:13969) UnhandledPromiseRejectionWarning: undefined
(node:13969) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). To terminate the node process on unhandled promise rejection, use the CLI flag `--unhandled-rejections=strict` (see https://nodejs.org/api/cli.html#cli_unhandled_rejections_mode). (rejection id: 1)
(node:13969) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.

What version of node are you using? Which OS are you on?

I'm on Linux, using Node v13.5.0

I can repro with the same config:
$ nvm use --delete-prefix v13.6.0

$ npm install -g cash-cat

$ which -a cat
[PATH REDACTED]/.nvm/versions/node/v13.6.0/bin/cat
/bin/cat

$ [PATH REDACTED]/.nvm/versions/node/v13.6.0/bin/cat --help
Usage: cat [OPTION]... [FILE]...
Concatenate FILE(s), or standard input, to standard output.

  -A, --show-all           equivalent to -vET
  -b, --number-nonblank    number nonempty output lines, overrides -n
  -e                       equivalent to -vE
  -E, --show-ends          display $ at end of each line
  -n, --number             number all output lines
  -s, --squeeze-blank      suppress repeated empty output lines
  -t                       equivalent to -vT
  -T, --show-tabs          display TAB characters as ^I
  -v, --show-nonprinting   use ^ and M- notation, except for LFD and TAB
      --help               display this help and exit

With no FILE, or when FILE is -, read standard input.

Examples:
  cat f - g  Output f's contents, then standard input, then g's contents.
  cat        Copy standard input to standard output.

Report cat bugs to <https://github.com/dthree/cash>
Cash home page: <http://cash.js.org/>
(node:24024) UnhandledPromiseRejectionWarning: undefined
(node:24024) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). To terminate the node process on unhandled promise rejection, use the CLI flag `--unhandled-rejections=strict` (see https://nodejs.org/api/cli.html#cli_unhandled_rejections_mode). (rejection id: 1)
(node:24024) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.

I can only repro with the standalone executables (ex. cash-cat), not the interpreter (ex. cash).

Yes, it only affects the standalone executables.


Resize mobile UI

Pull Request Type

[x] ✨ feat
[x] 🐛 fix
[ ] ♻ refactor
[x] 💄 style
[ ] 🔨 chore
[x] 📝 docs

Relevant Issues

resolves #xxx
What is in this change?

Additional Information

Developer Validations

[ ] I ran yarn lint from the root of the repo & committed changes
[ ] Relevant documentation has been updated
[ ] I have tested my code functionality
[ ] Docker build succeeds locally

This touches almost every page in the app, has no issue associated with it nor explanation/reasoning for change as well as merge conflicts with almost all files. Closing until all of the above are solved


setup.py install fails with ninja error

I followed the instructions, and made sure that I have all the required libraries,  but setup.py fails due to an error from ninja complaining about missing libbxRelease.a. I am don't have much experience with compiling C++ or using ninja and don't know how to fix this, am I missing a library or something?
Here is the output from setup.py: Console Output.txt
I am running macOS 10.14.6 and python 3.8.1, cmake, make, and ninja are all the most up to date versions on homebrew as of 3/4/2020.

From your console output:
In file included from ../../../../bx/src/allocator.cpp:6:
In file included from ../../../../bx/src/bx_p.h:42:
../../../../bx/include/bx/bx.h:9:10: fatal error: 'alloca.h' file not found
#include <alloca.h> // alloca
         ^~~~~~~~~~

This happens if you don't have installed the MaxOS SDK and/or if you recently updated to Catalina.
Here is a checklist to ensure that your system is up to date:

make sure to have the latest version from master
check if you have the Command Line Tools installed: xcode-select --install
check that this path exists: ls -la /Library/Developer/CommandLineTools/SDKs/MacOSX.sdk

hello sorry to plug here but i'm having the same issue but on windows cmake fails this is the console log
(i'm using Visua Studio 2019 with VSBuildTool 141 (2017) i changed it using this method ) and this is the
CMakeError.log of cmake,
i dont know if its faulty setup or else but it seem that a bx flag during compiling is not set.

i've installed and retried with Visual Studio 2017 but it still give me the same errors

@darkimage Yes there is an issue with windows build right now. I'm currently working on resolving it, thanks for the input!

@fbertola dont worry i thought it was only on my side, if you need more logs or debug infos let me know

@darkimage Windows build should now work as expected.

@fbertola yup it's now working great, thank you!
one thing to mention it's that i had uninstall vs 2017 because the setup.py was still trying to use the v141 compiler tools and that lead to errors during the compilation.
the setup.py still search for 2017 v141 but hookup the v142 just fine (maybe this can cause confusion)

Good point, I'll have a look. Thanks!

I apologize for not getting back to this sooner, I got swept up in the chaos of the last two weeks and completely forgot about this.
It turns out that something was wrong with my installation of the command line tools, I uninstalled them with sudo rm -rf /Library/Developer/CommandLineTools and re-installed them and was able to compile the latest master successfully. However the bindings scripts fail with:
Traceback (most recent call last):   File "make_bindings_bgfx.py", line 9, in <module>     binder_executable = glob.glob( IndexError: list index out of range
I think something might be wrong with the path on line 8 but I don't know what it should be pointing too.

@WanderingMacUser you should install binder in that directory: https://github.com/RosettaCommons/binder.
Basically, clone the repository then execute this script: binder/build-and-run-tests.py. The generated .cpp files needs some touch up, so it's not entirely an automated process, but it helps a lot.

I'm closing this issue for lack of additional inputs.


[Bug]: Moon Cycle length input turns red if value is not an interger

Check for existing bug reports before submitting.

[X] I searched for existing Bug Reports and found no similar reports.

Expected Behavior
Cycle length input should remain the same color whether a whole number or decimal is inputted.
Current behaviour
Cycle length input turns red when a decimal is inputted.
Reproduction

Open Moon menu in calendar editing menu.
Input an integer, followed by a decimal point, and add numbers past the decimal point.

Which Operating Systems are you using?

[ ] Android
[ ] iPhone/iPad
[ ] Linux
[ ] macOS
[X] Windows

Obsidian Version Check
1.3.5
Plugin Version
b12
Confirmation

[X] I have disabled all other plugins and the issue still persists.

Possible solution
No response

@Suleiman212 I can't recreate this issue. Does it happen on the default theme?
I don't do any validation on this input other than setting it to a number type, which allows decimals.

It happens on default theme, but I messed around and tested things out and realized it's cause by a conflict with... AidenLx's Folder Note v0.16.5??  Sorry for the false alarm. No idea why that plugin would cause that behavior.
This is how it looks on default theme:

But it seems like it's just a visual bug, there's no impact on how the moons are actually calculated, and it only occurs with that other random plugin activated, so sorry for the erroneous bug report or wasting your time.

Weird! I might dig through their code to see how it’s setting that. Closing the issue though


Exclude directive is not honored for directories under Windows

Scala-cli 1.2.0
Windows 11
Describe the bug
The "exclude" directive is not honored for directories under Windows.
To Reproduce
create Main.scala:
//> using exclude "input"

@main def go = println("Hello")

create input/Source.scala:
a

When running in Windows 11, produced the error:
[error] .\input/Source.scala:1:1
[error] Illegal start of toplevel definition
[error] a
[error] ^

Expected behaviour
It should not try to compile files underneath the input directory because I have explicitly excluded the whole directory via the exclude directive. Also writing it like that is ignored: //> using exclude "input/*"

I dont think its just windows - I also can't get it to work in https://github.com/bishabosha/ops-mirror


Added raylib-APL to BINDINGS.md

Oh i accidentally removed a newline! give me a minute or two

done, now ready for merge

I realized raylib-apl isn't auto, it's raylib 5.0. will edit that too

Fixed, now ready for merge again.

@Brian-ED thanks for the addition!


[BUG] Issues with netty connection provider resulting  'cosmosTemplate'  threw exception

When trying to upgrade com.microsoft.azure:spring-data-cosmosdb:2.2.4 it is resulting --
Factory method 'cosmosTemplate' threw exception; nested exception is java.lang.NoClassDefFoundError: reactor/netty/resources/ConnectionProvider
which is stopping us to start the application.

IDE : [IntelliJ]

@kushagraThapar I have created a issue here, please look into it

Got solved this issue ---
we added higher version of reactor-netty and core which enabled to find the connection provider.
compile group: 'io.projectreactor.netty', name: 'reactor-netty', version: '0.9.8.RELEASE'
compile group: 'io.projectreactor', name: 'reactor-core', version: '3.3.9.RELEASE'

Hope this works for anyone who comes in future to see similar issue.


✨ Revert updated experiment data export csv format to old version

I'm not sure I'm qualified to review this one, I'm not at all familiar with this area of the code. @bcb37 or @VivekFitkariwala may need to check this one


coerce_numbers_to_str doesn't work with IntEnum

Initial Checks

[X] I confirm that I'm using Pydantic V2

Description
When an IntEnum instance is dumped with the enabled coerce_numbers_to_str option, it produces a stringified enum name (FooEnum.FOO) instead of a stringified integer value ("1").
It appears that pydantic calls str on enum instance instead of calling str on instance’s value.
Example Code
import enum
import pydantic

class FooEnum(enum.IntEnum):
    FOO = 1

class FooModel(pydantic.BaseModel):
    field: FooEnum = pydantic.Field(coerce_numbers_to_str=True)

if __name__ == "__main__":
    model = FooModel(field=FooEnum.FOO)
    # {'field': 'FooEnum.FOO'}
    print(model.model_dump())

Python, Pydantic & OS Version
pydantic version: 2.9.2
pydantic-core version: 2.23.4
pydantic-core build: profile=release pgo=false
install path: ...
python version: 3.9.6 (default, Oct  4 2024, 08:01:31)  [Clang 16.0.0 (clang-16<IP_ADDRESS>)]
platform: macOS-15.1-arm64-arm-64bit
related packages: typing_extensions-4.12.2
commit: unknown

I just checked this code
import enum
import pydantic

class FooEnum(enum.IntEnum):
    FOO = 1

class FooModel(pydantic.BaseModel):
    field: FooEnum

# Without coerce_numbers_to_str I see enum
assert FooModel(field=FooEnum.FOO).model_dump() == {'field': FooEnum.FOO}

class FooModel(pydantic.BaseModel):
    field: FooEnum = pydantic.Field(coerce_numbers_to_str=True)

# with coerce_numbers_to_str I see '1'
assert FooModel(field=FooEnum.FOO).model_dump() == {'field': '1'}

It seems everything works well
or did I do something wrong?
pydantic version: 2.9.2
pydantic-core version: 2.23.4
pydantic-core build: profile=release pgo=false
install path: /Users/aberenda/projects/optifino/optifino/.venv/lib/python3.12/site-packages/pydantic
python version: 3.12.6 (main, Oct  3 2024, 00:52:54) [Clang 16.0.0 (clang-16<IP_ADDRESS>)]
platform: macOS-15.1-arm64-arm-64bit
related packages: typing_extensions-4.12.0 mypy-1.13.0 pydantic-extra-types-2.9.0 fastapi-0.115.4 pydantic-settings-2.4.0
commit: unknown

I see that I used python3.12 not python3.9
I need to check on python3.9

From the IntEnum docs:

Changed in version 3.11: str() is now int.str() to better support the replacement of existing constants use-case. format() was already int.format() for that same reason.

So the problem affects only python<3.11.

The logic for coerce_numbers_to_str during validation is to call str(...) on the validated data, and I don't think we want to introduce extra complexity with version checks and IntEnum in pydantic-core. The enum module has undergone multiple changes across Python versions and we'd like to avoid special casing as much as possible.
Alternatively, you can customize the __str__ method of the enum:
class FooEnum(enum.IntEnum):
    FOO = 1

    # if sys.version_info < (3, 11):
    def __str__(self) -> str:
        return str(self.value)


If you fill in the API key after [ZHIPUAI_API_KEY = ], the node cannot be loaded after the restart

根据AI的反馈才发现，原来密钥要填写在【config.json】文件里。完全没有提示，这对完全没有代码基础的人来说太难了。希望出现同样报错的人可以注意到API密钥要写进【config.json】里
According to the feedback of AI, it was found that the original key should be filled in the [config.json] file. No hint at all, which is too hard for someone with no code base at all. Those who expect the same error will notice that the API key is written into [config.json]

根据AI的反馈才发现，原来密钥要填写在【config.json】文件里。完全没有提示，这对完全没有代码基础的人来说太难了。希望出现同样报错的人可以注意到API密钥要写进【config.json】里 According to the feedback of AI, it was found that the original key should be filled in the [config.json] file. No hint at all, which is too hard for someone with no code base at all. Those who expect the same error will notice that the API key is written into [config.json]

Hi. Yes, it was my mistake, I wrote in the readme that I need to get a key, but I forgot that it needs to be written in the config.json file. 😭 😸 Thanks you!


Bad, sad with multiple packages using local repo

This is with slalpha 5. Building using build.bat. Building in front directory gets a bad, sad (ballerina-internal.log included).
semtype.zip

The problem is intermittent. Sometimes it gets the bad, sad; sometimes it doesn't. This is on Windows.

Just for the record:
Shanker was able to fix it in his machine by adding a Package.md, but issue intermittently happens for James even with that. It could be that we are depending on the directory listing order.

There is a different kind of intermittent error in mac :
Compiling source
	jclark/nballerina.front:0.1.0
Exception in thread "main" java.lang.NullPointerException
	at io.ballerina.cli.launcher.Main.main(Main.java:63)

The NPE in the ImmutableTypeCloner which caused the bad, sad error as in the attached file occurs when an import cannot be resolved (https://github.com/ballerina-platform/ballerina-lang/issues/30561). With beta1 it will result in a "cannot resolve module" error.

There is a different kind of intermittent error in mac :
Compiling source
	jclark/nballerina.front:0.1.0
Exception in thread "main" java.lang.NullPointerException
	at io.ballerina.cli.launcher.Main.main(Main.java:63)

Could be related to https://github.com/ballerina-platform/ballerina-lang/issues/30210. I also observed the same on Ubuntu on Swan Lake Alpha 5.

I also get the error in https://github.com/ballerina-platform/ballerina-lang/issues/30798#issuecomment-848712884 on Windows.

I had the project open in VSCode also. After I closed VSCode, the problem seems less frequent, but still happens.

I checked my stuff in to a branch
https://github.com/jclark/semtype/tree/multipackage

I was able to reproduce the same behavior on Ubuntu with slalpha5.
This issue is most likely due to the same issue mention in https://github.com/ballerina-platform/ballerina-lang/issues/30521. And it is fixed in the latest RC with https://github.com/ballerina-platform/ballerina-lang/pull/30549.
Since the issue is related dependent modules with hierarchical package names I try to rebuild the attached sample with "nballerinatypes" as the package name. After renaming the packages the issue seems to went away.
We will do a proper debug and close this.

RC5 fixes these errors, but I get another multipackage problem #30807

Closing since this is fixed in beta1.


Special characters disallowed

It's possible that it's a mistake on my part, but I've been doing some testing and it appears that special characters that are allowed on League (in my case, the Á in ÁssBlaster) aren't working with the library. Testing the same method with different usernames without them works, which leads me to believe that these characters are screwing it up.

To provide some more information: I'm using the summoner call to get the account id of a player to get a matchlist, with the discord.js library. The command would be %summoner , and it works in the case of a name such as Dropkicked, but doing %summoner ÁssBlaster returns a 404.

Finally, a basic example of what I'm doing:

api.Summoner.by.name(name)
.then(data => {
api.Matchlist.by.id(data.accountId)
.then(matchdata => {
console.log(matchdata.matches[0].lane);
}).catch(error => console.error(error)) // Matchlist by id
}).catch(error => console.error(error)) // Summoner by name

When name is ÁssBlaster, it returns the 404.
When name is Dropkicked, it returns the lane, (usually JUNGLE or BOTTOM).
(Apologies for spacing and such, Github's code option with the backticks was even worse.

you should be doing
api.Matchlist.by.account(data.accountId)
by.id = SummonerId
by.account = AccountId
https://github.com/ChauTNguyen/kindred-api/wiki/SUMMONER-V3
It's rather confusing, but at the time I decided to go with the shorter names because I didn't think Riot would use account ids for some endpoints.
Note that there is a shortcut for what you're doing:
api.Matchlist.by.name('whatever')

will handle the summoner conversion for you.

Okay, thanks! But just for confirmation, this still takes 1 rate count off of summoner, and then 1 off matchlist, correct? It shouldn't matter, considering rates are crazy high, but I was just curious.

@UghThatGuyAgain
Yeah, unfortunately it still does haha. It's just a small way of cleaning up a lot of function chaining.

Alright, thanks. Completely missed that there was a matchlist by name method, I should really read closer. Keep up the good work!


InteractiveWebhookMessageNfmReplyName is not reachable

@lukas-becker0,
I just noticed on your recent changes #47  that InteractiveWebhookMessageNfmReplyName is not currently reachable at https://github.com/MarcosNicolau/whatsapp-business-sdk/blob/1a5b276ebc453dcea270d8906cb67f680d77eaba/src/types/webhooks.ts#L49C1-L50C1
export type InteractiveWebhookMessageNfmReplyName =  "address_message" | "flow" | string;

In this change, response_json exists if name is string, however InteractiveWebhookMessageNfmReplyName is not used anywhere. So there is no way to access for example response_json data.
Can you revisit your changes?
Regards,

It is used in the generics of InteractiveWebhookMessageNfmReplyMessage
but you are right that the type is broken I forgot to put brackets around the conditional type.
Also based on the previous type response_json should always exist.

I improved the type for InteractiveWebhookMessageNfmReplyMessage
The PR draft is here:
https://github.com/MarcosNicolau/whatsapp-business-sdk/pull/50
Does that fix your issue ?


Use TimeSpan for lease duration rather than int

Fixes #7221

Since TimeSpan.MaxValue would overflow an int, we add a swagger transform to make the duration used in the internal generated BlobRestClient be a long. TimeSpan is used within LeaseClient, and is transformed to a long before passing off to BlobRestClient.
The actual service will only allow durations of 15s-60s or -1 for infinite lease duration. As such, added a test to verify that the service returns StorageRequestFailedException along with an InvalidHeaderValue header code for duration longer than 60s. This test also verifies that TimeSpan.MaxValue does not cause overflow exception when duration that is greater than Integer32.MaxValue is used.

Also, do me a favor and edit your PR description to add a little blurb at the top about TimeSpans being the proper .NET type to represent a duration and include a quick before/after code example (just calling Acquire off a LeaseClient would be fine) like the other https://github.com/Azure/azure-sdk-for-net/issues?utf8=✓&q=label%3Abreaking-change+ PRs do.

Heads up @seanmcc-msft @amnguye I'm planning on merging this by end of day. Thanks!


fix typo in index.coffee

Error occurs when using PHABS_DISABLED_FEATURES

Opps! good catch! :+1:


Added {} in templates

What kind of change does this PR introduce?
I refactored some template code to harmonize control structures to follow the use of always having {}

Did you add tests for your changes?
Tried to use the npm run test but didn't work before changes

If relevant, link to documentation update:

Summary
Added {} in templates so Microsoft JS guidelines will work on generated manifest.js for bundles like with vue js will pass the quality test.

Does this PR introduce a breaking change?
No.

Other information

to run the tests locally run
npm link && npm link weback
maybe we should have that on some post local install step @sokra
this appears to be a quite common issue :p

@timse I tried to circumvent that by eslint disables but yes, webpack as devdep makes sense

@timse @sokra The thing is, there are errors, but there are errors in files I did not touch, so I suspect there where there before.

maybe we should have that on some post local install step @sokra
this appears to be a quite common issue :p

good idea.

to run the tests locally run
npm link && npm link weback

maybe we should have that on some post local install step @sokra
this appears to be a quite common issue :p

For this, webpack needs to be a dep or file in the package.json otherwise it still counts as unpublished in one of the files.

Sorry forgot the stats

Sorry, I'm not so much used to forking and submitting pull requests.

Thanks


The window has crashed (reason: 'oom', code: '-536870904')

Does this issue occur when all extensions are disabled?: Yes/No

VS Code Version: 1.74.0
OS Version: Windows 10 19045.2251

Steps to Reproduce:

Run a machine learning program that uses 10-50GB RAM.
Wait for 2-5 hours and it crashes every time.
Attached 2 crash dump files from 2 different machine learning programs.

e80c0aa8-419d-4389-b7d4-30ba64804a5b.zip
a2b4164f-46fb-4567-bcce-403df3a36240.zip

Can you provide a minimal example to repro the issue.

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!

I have the same problem when I run an ng serve


Page was generated in 1916

The classic off by one century error

This seems to have fixed itself.  Yay!


Set up eslint rules in SDK repo

Description
Currently there are no typescript linting rules enforced in the SDK repo. Let's enforce some reasonable lints :)
Reasonable set of typescript eslint rules from Rossi at Hyperlane: https://github.com/hyperlane-xyz/hyperlane-monorepo/blob/main/.eslintrc
Initial reference: https://github.com/mento-protocol/mento-sdk/pull/2#discussion_r1084398819
Acceptance Criteria

[ ] Researched a reasonable set of typescript linting rules from other repos
[ ] Typescript eslint setup in the SDK repo

Hi , Would love to be assigned this issue . I would do due research and perform the setup in the repo

Hi, can I still improve on this issue? I noticed that there are only two rules that has been setup.
eslint-config

please asign this issue to me


Automatically find the name of the exe file

Automatically find the name of the exe file in the windows build folder.

fix in version 0.1.1

fix in version 0.1.1


Add syntax for reducer-less groupby to ZQL

There should be a way to express a "reducer-less" groupby in ZQL, that would compute a group-by table of all the unique values for its keys, without any reducer. (With identical results to count() by <keys> | cut <keys>).
Proposed syntax:
... | group by <keys> | *
Note that this is a grammar-only change; the runtime already supports this, and can already handle a reducer-less groupby AST (as used in #816) :
			&ast.GroupByProc{
				Keys: []string{keyName},
			},

Offline decision: the syntax will be .. | by keys | ..

Offline decision: the syntax will be .. | by keys | ..

The time-binned variant will read a little odd ... | every 1h by keys | ... but maybe that's ok.

I put this in backlog as I've wanted to use it and it should be super easy (famous last words)

This functionality is verified via the user-facing ZQL docs being tracked in #1623.


better recognition of different xml file types

application/xml as mimetype is often not sufficient to determine the type of a file.
Therefor, it would be good to know more based on the xml content to identify i.e. METS, OCR, ...

new indexer action "xml" integrated.
Indexer Extension config needs now "xml" in the actions field and toml file needs new configurations to detect xml filetypes.


Add SWIM-IR

Google released a new crosslingual retrieval dataset:
https://huggingface.co/datasets/nthakur/swim-ir-cross-lingual
We could turn a subset of this into a retrieval and reranking benchmark.
If no one picks this up, I can take at look at this during the weekend.

That'd be great indeed cc @thakur-nandan

Thanks @Muennighoff. The SWIM-IR dataset would be great and contains training splits only as it should be used for training. If that would be desirable we can go ahead and add it into MTEB.
Let me know if you need help @rasdani.
Thanks,
Nandan

Thanks @Muennighoff. The SWIM-IR dataset would be great and contains training splits only as it should be used for training. If that would be desirable we can go ahead and add it into MTEB.
Let me know if you need help @rasdani.
Thanks, Nandan

Oh does it still make sense to use it for evaluation or not at all? Not sure if adding a training dataset makes sense cc @KennethEnevoldsen

I wouldn't add a dataset intended for training unless we expect it to evaluate an aspect which we are currently not evaluating.


[api] use schema $refs instead of imports

Describe the bug
In the future we want to support an ?include= param which is a standardized REST API pattern (similar to graphql relationships).
For example /user may optionally support ?include=listings&include=listings.project to optionally include a Listing and a Project in the response
This is blocked until we start using $ref strings in our api route schemas
Problem

JS module import of the shared schema, like Project, creates circular dependencies.
The stoplight docs don't properly link an endpoint response to the model entity, right now we have duplication of the same model in both sections.

Solution
Type.Ref() to common/global schema definitions

@0xemc I think this would be a good project for @ChazSchmidt let's coordinate on how to get him started

@0xemc I think this would be a good project for @ChazSchmidt let's coordinate on how to get him started

For sure, the refs lift should definitely be done first in an isolated PR/ticket. I had a go at this a while ago here but ran in to circular dependency issues still...
@ChazSchmidt feel free to take a look at that branch and see if you can get it to work (or start fresh if you like). Make sure to regen types in carbonmark and the api when doing this though


Extension 'firebase/storage-resize-images' emulator, not working well with Windows

[REQUIRED] Environment info

firebase-tools: 11.20.0

Platform: Windows 11
[REQUIRED] Test case

Firebase Extentions Emulator: firebase/storage-resize
extensions\storage-resize-images.env
DELETE_ORIGINAL_FILE=false
DO_BACKFILL=false
FUNCTION_MEMORY=512
IMAGE_TYPE=jpeg
IMG_BUCKET=demo
IMG_SIZES=500x500
IS_ANIMATED=true
LOCATION=europe-west1
MAKE_PUBLIC=false
RESIZED_IMAGES_PATH=thumbnails

firebase.json
{
  "functions": [
    {
      "source": "functions",
      "codebase": "default",
      "ignore": [
        "node_modules",
        ".git",
        "firebase-debug.log",
        "firebase-debug.*.log"
      ],
      "predeploy": [
        "npm --prefix \"$RESOURCE_DIR\" run lint",
        "npm --prefix \"$RESOURCE_DIR\" run build"
      ]
    }
  ],
  "database": {
    "rules": "database.rules.json"
  },
  "firestore": {
    "rules": "firestore.rules",
    "indexes": "firestore.indexes.json"
  },
  "storage": {
    "rules": "storage.rules"
  },
  "emulators": {
    "auth": {
      "port": 9099
    },
    "functions": {
      "port": 5001
    },
    "firestore": {
      "port": 8080
    },
    "database": {
      "port": 9000
    },
    "storage": {
      "port": 9199
    },
    "ui": {
      "enabled": true
    },
    "singleProjectMode": true
  },
  "extensions": {
    "storage-resize-images"<EMAIL_ADDRESS>  }
}

[REQUIRED] Steps to reproduce

Upload a JPEG to a {folder} in the Firebase Storage Emulator.
[REQUIRED] Expected behavior

Thumbnail gets generated/uploaded in a thumbnails/ folder next to the user uploaded file in {folder}.
[REQUIRED] Actual behavior

Thumbnail gets generated/uploaded to the root with {folder}\thumbnails\my_image.jpeg as the name``

extension[storage-resize-images]
{
"severity": "INFO",
"message": "Image path 'C:\users\eYAlFqM4ckH6ds3yNmendFtzOuXd\profile-pic' is not supported, these are the supported absolute paths: /users//profile-pic, /users//wishlist"
}
on 0.1.39

I think I've figured out what's going on, I should have a PR in to fix this tomorrow. Here is a branch where i was investigating:
https://github.com/firebase/extensions/pull/1805

reopening as there is still an issue with certain windows paths, see: https://github.com/firebase/extensions/pull/1805#issuecomment-1826361858


Sort all system folder either on top or bottom

Now folders that starts with dot symbol a chaotically placed in top AND in bottom, something one should be picked

will be fixed automatically by https://github.com/ydb-platform/ydb/issues/10150


Fix move to publication

Please review, merge, etc.  This should clear up some of the "conduct_move"  confusion.

Shoot - what happens if you add a commit AFTER requesting a review? ...  bad form!

Also added the chmod 0664 after mapfile write in both "generate_mapfile" and "fix_mapfile_paths"

Nothing happens.  Its only bad form if you add a ton of code after the reviewer is done.


[HOTFIX] [Build] [SPARK-12442] Build testing failed.

./dev/run-tests.py
  File "./dev/run-tests.py", line 532
    if not changed_files or any(f.endswith(".py") for f in changed_files):
     ^

Unable to run the command: ./dev/run-tests.py
Above is the error I got in my local computer.
My another PR hit this too. https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48064/consoleFull
@rxin could you check if my fix works? Thanks!

I pushed directly to master and fixed it. Thanks.

Test build #48065 has started for PR 10397 at commit 4475706.

Thanks!

Test build #48065 has finished for PR 10397 at commit 4475706.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/48065/
Test PASSed.


Nodejs bindings follow config settings

If you configure soletta to not have OIC functionality then the resulting node.js bindings can't have such functionality either.
Addtitionally, this PR fixes #1567 by surrounding the addition of the fortify-related CFLAGS with a check to make sure that -O0 is not already part of the CFLAGS.

What I'm suggesting is to use BUILD_TYPE_DEBUG=y instead of BUILD_TYPE_RELEASE=y (on this case BUILD_TYPE_RELEASE isn't even declared)


Error while building OracleDatabase <IP_ADDRESS>: SEVERE:/etc/oraInst.loc (Permission denied)

I get PermissionDenied error while building OracleDatabase <IP_ADDRESS> in OpenSUSE 42.3:
buildDockerImage.sh -s -v <IP_ADDRESS>

Checking Temp space: must be greater than 500 MB.   Actual 2556786 MB    Passed
Checking swap space: must be greater than 150 MB.   Actual 65362 MB    Passed
Preparing to launch Oracle Universal Installer from /tmp/OraInstall2018-02-28_10-15-14PM. Please wait ...[WARNING] [INS-32055] The Central Inventory is located in the Oracle base.
   ACTION: Oracle recommends placing this Central Inventory in a location outside the Oracle base directory.
SEVERE:/etc/oraInst.loc (Permission denied)
[FATAL]
A log of this session is currently saved as: /tmp/OraInstall2018-02-28_10-15-14PM/installActions2018-02-28_10-15-14PM.log. Oracle recommends that if you want to keep this log, you should move it from the temporary location.
 ---> b01139f5a193
Removing intermediate container 0fd2f8b370f1
Step 10/17 : USER root
 ---> Running in fd119043336a
 ---> b88c62228aed
Removing intermediate container fd119043336a
Step 11/17 : RUN $ORACLE_BASE/oraInventory/orainstRoot.sh &&     $ORACLE_HOME/root.sh &&     rm -rf $INSTALL_DIR
 ---> Running in cdf8f2410d3f
/bin/sh: /opt/oracle/oraInventory/orainstRoot.sh: No such file or directory
Removing intermediate container cdf8f2410d3f
The command '/bin/sh -c $ORACLE_BASE/oraInventory/orainstRoot.sh &&     $ORACLE_HOME/root.sh &&     rm -rf $INSTALL_DIR' returned a non-zero code: 127

ERROR: Oracle Database Docker Image was NOT successfully created.
ERROR: Check the output and correct any reported problems with the docker build operation.

Full build log attached:
output.txt

Hi @dsendkowski, are there any security features enabled that may prevent the container from accessing /etc?

None that I am aware of. Any specific features you have in mind?

SELinux sometimes prevents certain things to be allowed but in your scenario I didn't spot any SELinux related issues.
In my container I see the file to be owned by root, so I'm struggling to understand where the error comes from:
-rw-r--r-- 1 root   root         54 Sep 22 16:51 oraInst.loc
-rw-r--r-- 1 root   root         32 Aug  4  2017 oracle-release
-rw-rw-r-- 1 oracle oinstall    789 Mar 21 02:34 oratab

Any chance you can have a look at /tmp/OraInstall2018-02-28_10-15-14PM/installActions2018-02-28_10-15-14PM.log and see what it complains? It probably will tell you to look into the according trace file.

I set both --rm=false and --force-rm=false so that no intermediate container is removed. Unfortunately, I could not find this log file (/tmp/OraInstall2018-03-21_08-48-59AM/installActions2018-03-21_08-48-59AM.log) in any intermediate container in the specified location for my current session.

Thanks @dsendkowski, could it be that you got a different folder as the one above contains the date and time. Every time a new install is kicked off you will find a new folder under /tmp.

I got the same issue with OpenSuse 42.3

Hi @tonyyao81, can you provide the output from the log file please?

Closing due to lack of response.

I can reproduce it on SLES 12.3 and I cannot provide you a log file.
ls /tmp
tells me that /tmp is complete empty. Any idea how to proceed? Thx.


A wrong in ActivationGrid.ipynb And my solution

When I run this demo, I can't get the predicted picture

, and when I open this demo in jupyter, there is another wrong that shows .
So, to show the picture in Terminal, we can
import matplotlib.pyplot as plt
then:

Note the green line and add those three lines above the green one.
At this time, we obtain the demo:

I wish to pull a request to fix this bug which is not a bug..LOL

Hi @BIRlz, your errors look like the installation process under the "Install/Import heading" was not successful. See if you can ensure these two dependencies install correctly:
pip install --quiet lucid==0.0.5
npm install -g<EMAIL_ADDRESS>
I can't offer more help with setting up your environment, but you can always use the notebooks on Colab. (Where the environment will be the same we use to develop them.)

Thank you sir~


Autocomplete/typeahead recipe using Halogen Select

Recipe Name
HalogenSelect

Recipe Description
A recipe that demonstrates a basic example on using autocomplete or typeahead using Halogen Select.

Doesn't Halogen Select already have examples that show how to do that?


Add clamp transformer

Hey @lukasoppermann, this PR addresses issue: https://github.com/lukasoppermann/style-dictionary-utils/issues/23, that my colleague Jkinley raised the other day. Let me know if any changes are needed. Thanks!

Looks good 🚀


Implement react & svelte suggestions & linker for intertia

Proposed changes
Fix #32
Types of changes
What types of changes does your code introduce?
Put an x in the boxes that apply

[ ] Bugfix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist
Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code.

[x] I have read the CONTRIBUTING doc
[x] Lint and unit tests pass locally with my changes
[x] I have added tests that prove my fix is effective or that my feature works.
[ ] I have added necessary documentation (if appropriate)

Further comments
If this is a relatively large or complex change, kick off the discussion by explaining why you chose the solution you did and what alternatives you considered, etc...

thanks a lot dude! pipeline for windows seems broken but probably not related to this PR


Implement Responsive Navigation for Improved Mobile Experience

The current navigation bar lacks responsiveness, causing issues when viewed on smaller screens such as mobile devices. This issue aims to implement a responsive design for the navigation bar to ensure optimal usability and navigation experience across various devices and screen sizes.
please assign this issue to me?

Already assigned

Assign this issue to me.
As a full Stack Developer and 1 Year experience I can Write optimized code and Make it responsive.

@Nithin9585 What's the updates on this?

ANY UPDATES?

Unassigned! No activity


Incorrect bitwidth for QASM input with pytket 1.30.0

Describe the bug
With upgrade to pytket==0.13.1, all tests involving range predicate fail (see here):
FAILED tests/test_api.py::TestApi::test_pytket_to_phir_no_machine_all[QasmFile.cond_1] - UnboundLocalError: cannot access local variable 'cond' where it is not associated with a value
FAILED tests/test_api.py::TestApi::test_pytket_to_phir_no_machine_all[QasmFile.cond_classical] - UnboundLocalError: cannot access local variable 'cond' where it is not associated with a value
FAILED tests/test_api.py::TestApi::test_pytket_to_phir_h1_1_all[QasmFile.cond_1] - UnboundLocalError: cannot access local variable 'cond' where it is not associated with a value
FAILED tests/test_api.py::TestApi::test_pytket_to_phir_h1_1_all[QasmFile.cond_classical] - UnboundLocalError: cannot access local variable 'cond' where it is not associated with a value

I believe this is because the change in #190 makes the value of UINTMAX depend on 64-bits but the pytket translation of QASM circuits is still using 32-bits for the range predicate translation. See
op = RangePredicate([2,4294967295]), cmd = RangePredicate([2,4294967295]) c[0], c[1], tk_SCRATCH_BIT[1];

    def convert_classicalevalop(op: tk.ClassicalEvalOp, cmd: tk.Command) -> JsonDict | None:  # noqa: PLR0912
[...]
            case tk.RangePredicateOp():  # where the condition is a range
                cond: JsonDict
                match op.lower, op.upper:
                    case l, u if l == u:
                        cond = {
                            "cop": "==",
                            "args": [args[0].reg_name, u],
                        }
                    case l, u if u == UINTMAX:
                        cond = {
                            "cop": ">=",
                            "args": [args[0].reg_name, l],
                        }
                    case 0, u:
                        cond = {
                            "cop": "<=",
                            "args": [args[0].reg_name, u],
                        }
                    case l, u:
>                       raise TypeError(f"{l=} {u=} {UINTMAX=}")
E                       TypeError: l=2 u=4294967295 UINTMAX=18446744073709551615

To Reproduce
Upgrade to pytket==0.13.1
Expected behavior
Tests should pass.
Additional context
Changes in #190

@cqc-alec do you think this is a bug in pytket or do we need to treat QASM inputs specially assuming max 32 bits?
cc: @dlucchetti

@cqc-alec do you think this is a bug in pytket or do we need to treat QASM inputs specially assuming max 32 bits?

There is a maxwidth parameter to circuit_from_qasm, which defaults to 32. Would changing this to 64 in the get_qasm_as_circuit() method (or to 64 if pytket.__dict__.get("bit_width_64", False) else 32 as used in the definition of UINTMAX) fix the issue?

Just tested locally and it looks like this does fix the errors.


Sync with upstream

@benmanns I'm curious how difficult syncing with upstream has been for you recently? Any major pain points? Anything I can do to make it easier?


Add first offer template

This PR adds a first HTML template for an offer.
Checkout the current branch and open the offer.html in your Internet browser. Print out the HTML to see its final rendering layout.
PR Checklist
Please fill in the appropriate checklist below (delete whatever is not relevant). These are the most common things requested on pull requests (PRs).

[ ] This comment contains a description of changes (with reason)
[ ] Referenced issue is linked
[ ] If you've fixed a bug or added code that should be tested, add tests!
[ ] Documentation in docs is updated
[ ] CHANGELOG.rst is updated

Description of changes
Please state what you've changed and how it might affect the user.
Technical details
Please state any technical details such as limitations, reasons for additional dependencies, benchmarks etc. here.
Additional context
Add any other context or screenshots here.

@jenniferboedker thank you very much! I just pushed support for UTF8. Umlauts and special chars should rendered now properly.

LGTM! I checked it on my machine with Safari & Firefox and both browsers were able to get the "Open Sans" Font. 👍

Awesome, thank you for checking!


Automatically add Push Notification capability

Feature Request
Motivation Behind Feature
Currently I have to add the "Push Notifications" capability by hand in XCode.
Feature Description
The capability should be added to the xcode project by default, or create a support command that user can use in the CI workflow.
Alternatives or Workarounds
I'm currently adding the capability by hand.

Nevermind, I've solved it.
Anyone who finds this later: You need to add the -allowProvisioningUpdates build flag.
My build command:
cordova build ios --prod --device --release --buildFlag="-allowProvisioningUpdates" --buildConfig=build.json

In the build.json:
{
    "ios": {
        "release": {
            "codeSignIdentity": "Apple Distribution",
            "developmentTeam": "TEAMD_ID",
            "packageType": "app-store",
            "automaticProvisioning": true,
            "provisioningProfile": "PROFILE_ID",
            "buildFlag": [
                "EMBEDDED_CONTENT_CONTAINS_SWIFT = YES",
                "ALWAYS_EMBED_SWIFT_STANDARD_LIBRARIES=NO",
                "LD_RUNPATH_SEARCH_PATHS = \"@executable_path/Frameworks\"",
                "IPHONEOS_DEPLOYMENT_TARGET=15.0"
            ]
        }
    }
}


has() will propagate an S3 exception rather than returning false

has() should catch the metadata 404 exception if the file doesn't exist and return false, rather than letting it propagate.

getMetadata() may also have this same issue, where what should return false instead throws an S3Exception all the way up the stack...leaky abstraction or something like that. The above issue (which will turn into a PR momentarily) doesn't fix the getMetadata() case, just the has() one, but it's easy enough to move the logic if warranted.

Yeah for me it threw an exception. I was expecting to get false back.

Fixed in 749a4d587dd2d21329324a0bb52c5abb82b2c6d4


Types::Json::Decimal raises NoMethodError on nil

Hello. First of all, thank you for your work. I think I have encountered unexpected behaviour: when Types::Json::Decimal is given explicit nil value, NoMethodError is raised, as opposed to #<Dry::Types::Result::Failure input=nil error="nil must be an instance of BigDecimal"> which seems to be more natural (like when empty string is passed, for example).
Here is how to reproduce:
Types::Json::Decimal.try(nil)
NoMethodError: undefined method `to_d' for nil:NilClass
Did you mean?  to_c
               to_r
               to_h
               to_a
               to_i
               to_s
               to_f
        from /Users/nesaulov/Code/ruby/opensource/dry-types/lib/dry/types/coercions/json.rb:16:in `to_decimal'
        from /Users/nesaulov/Code/ruby/opensource/dry-types/lib/dry/types/constructor.rb:56:in `[]'
        from /Users/nesaulov/Code/ruby/opensource/dry-types/lib/dry/types/constructor.rb:56:in `try'

I have created a PR addressing this issue.

was fixed via #216


Simply clicking once on a block should not place it

It is very confusing when people just click on a block, either in a function or list comprehension, or from the toolbox, and a copy of that block gets placed in the workspace.  We should wait for some minimum drag distance before treating this as a drag.

Interestingly, this was the single biggest cause of confusion in my user testing yesterday.  It was a constant frustration that the user would click on something once, and not even notice that they are now looking at a newly created draggable block, instead of the template that they clicked on.

This fixes the problem for the toolbox.  But the bigger source of confusion is actually the draggable local variable blocks used for functions and list comprehensions.


Why there is not global edit events?

Hi, I have bigger project, where editing layers is not enabled by default. Edit layers can only users with special permissions and is loaded only when user select build mode. Have you any idea, how I can simply add edit event on existing layers? I only need properties of edited layer (polyline). I'm thinking about recreating layers with events when user select build mode, but it seems as dirty solution... Is there any elegant solution or I must "manually" add event to all layers? No other way? :/
Thanks for response.

There is a pm:update event on the layer: L.PM.Edit.Line.js#L133 it fires only if there are changes on the layer after disableing edit mode.
Also you a few other events: Scroll to event table after Edit Mode


Fixed incorrect type formats in SQL database

This checklist is used to make sure that common issues in a pull request are addressed. This will expedite the process of getting your pull request merged and avoid extra work on your part to fix issues discovered during the review process.
PR information

[x] The title of the PR is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For information on cleaning up the commits in your pull request, see this page.
[x] Except for special cases involving multiple contributors, the PR is started from a fork of the main repository, not a branch.
[x] If applicable, the PR references the bug/issue that it fixes.
[x] Swagger files are correctly named (e.g. the api-version in the path should match the api-version in the spec).

Quality of Swagger

[x] I have read the contribution guidelines.
[x] My spec meets the review criteria:

[x] The spec conforms to the Swagger 2.0 specification.
[x] The spec follows the guidelines described in the Swagger checklist document.
[ ] Validation tools were run on swagger spec(s) and have all been fixed in this PR.

Apologies for the Delay @jaredmoo, we lately had a surge of PRs, someone will get on this shortly

Thanks Samer :)

Gotcha

I ran the linter and semantic check tools and this change appears to be clean. I found no errors from the tools. Will sign off, after I do a manual review.

okay my bad here. I forgot to pass in the composite swagger entry point to the tool and the tool happily returned no errors.
I'll go through the list again. 👍

Status as of 4/13: emailed @jaredmoo about the list of linter errors and examples errors that we should consider addressing.

Summary of errors we will track by bugs:

ERROR: Please provide x-ms-examples describing minimum/maximum property set for response/request payloads for operations.
Path: file:///C:/workspace/azure-rest-api-specs/arm-sql/compositeSql.json#/paths
[Jared Moore] Let’s file a separate issue for this

ERROR: The tracked resource, 'Database', must have a list by resource group operation.
Path: file:///C:/workspace/azure-rest-api-specs/arm-sql/compositeSql.json#/definitions/Database
ERROR: The tracked resource, 'ElasticPool', must have a list by resource group operation.
Path: file:///C:/workspace/azure-rest-api-specs/arm-sql/compositeSql.json#/definitions/ElasticPool
ERROR: The tracked resource, 'Database', must have a list by subscriptions operation.
Path: file:///C:/workspace/azure-rest-api-specs/arm-sql/compositeSql.json#/definitions/Database
[Jared Moore] Let’s file a separate issue for this. It’s not implemented in the service right now.

ERROR: The tracked resource, 'ElasticPool', must have a list by subscriptions operation.
Path: file:///C:/workspace/azure-rest-api-specs/arm-sql/compositeSql.json#/definitions/ElasticPool
ERROR: Tracked resource 'Database' must have patch operation that at least supports the update of tags.
Path: file:///C:/workspace/azure-rest-api-specs/arm-sql/compositeSql.json#/definitions/Database
ERROR: Tracked resource 'Server' must have patch operation that at least supports the update of tags.
Path: file:///C:/workspace/azure-rest-api-specs/arm-sql/compositeSql.json#/definitions/Server
ERROR: Tracked resource 'ElasticPool' must have patch operation that at least supports the update of tags.
Path: file:///C:/workspace/azure-rest-api-specs/arm-sql/compositeSql.json#/definitions/ElasticPool
[Jared Moore] Let’s file a separate issue for this. It’s not implemented in the service right now.

ERROR: Model definition 'FirewallRule' must have the properties 'name', 'id' and 'type' in its hierarchy and these properties must be marked as readonly.
Path: file:///C:/workspace/azure-rest-api-specs/arm-sql/compositeSql.json#/definitions/FirewallRule
[Jared Moore] The service really doesn’t send ‘type’ for FirewallRule. This is a bug but not one that I can easily fix. Will file issue for this.

Thanks, let me just double check there are no more changes needed

Reviewed again after the last commit. things are still good 👍

No modification for NodeJS

 https://github.com/Azure/azure-sdk-for-ruby/pull/759

 https://github.com/Azure/azure-sdk-for-python/pull/1094


Transparent background in mouseover text

All mouseover text (the text that pops up when you hover your mouse over a graph bar) has a transparent background. I don't know whether this is intended or not, but it makes the text hard to discern in some circumstances. This issue also occurs when viewing an individual game's activity.

fixed with #166


Crusader builds for other OpenWrt architectures than aarch64?

I think it would be cool to give instructions for running Crusader Server on lots of different OpenWrt devices. (And I would be willing to write the document.) I know the aarch64...linux build works great for the GL.iNet GL-MT6000 router. But there are a zillion combinations that I don't entirely understand.
To get a handle on the problem, I made a table of all OpenWrt devices and their architectures. It's at: https://openwrt.org/playground/richb/alttoh To use it , enter "23.05" in the Supported Current Release column, and sort by the "Package Architecture" column. It shows many variations:

aarch64_cortex-a53
aarch64_generic
arm_arm1176jzf-s_vfp
arm_arm926ej-s
arm_cortex-a15_neon-vfpv4
arm_fa526
arm_xscale
i386_geode
mips64_octeon
mips_24kc
... and on and on...

My questions:

Which of these architectures would support a pre-built version of Crusader?
Is there any way to know ahead of time whether it might work?
Is there any profit to making more native builds or those other architectures?

Many thanks

Good information. Thanks

@Zoxc Follow-up: I think I understand how this works. Is the following correct?
Once it is correct, we can talk about where/when to post it. (I'm not looking to talk Crusader up to a broad audience before you think it's ready...)

Crusader on OpenWrt
The Crusader pre-built binaries work well for certain OpenWrt devices/architectures: aarch64, arm, x86, and i686.
To determine if your router can run a pre-built binary, look it up in the Table of Hardware. Enter its model number and find its Package architecture column. Then retrieve the proper binary for the architecture from latest pre-built binary page using the table below:

Architecture
Latest Pre-built Binary

aarch64
crusader-aarch64-unknown-linux-musl.tar.gz

arm
crusader-arm-unknown-linux-musleabihf.tar.gz

x86
crusader-x86_64-unknown-linux-musl.tar.gz

i686
crusader-aarch64-unknown-linux-musl.tar.gz

other architectures
No pre-built binary

Note: The other architectures (mips, powerpc, risc, etc.) tend to be used in older devices that may not have the CPU power to run Crusader. You could try building your own version from source.

I edited the text above - did I get it right? Thanks

You'd still want the arm_cortex to arm mapping.

You'd still want the arm_cortex to arm mapping.

Here's where I display my ignorance. The ToH linked above lists many, many "arm" architectures. When I search for my own router, I'll see one of the choices below.
How can I tell whether the pre-built arm binary for Crusader will work? Does the crusader-arm-unknown-linux-musleabihf.tar.gz build work on all these? On only the ones labeled "cortex"? Is there other flag/hint? Thanks.

arm_arm1176jzf-s_vfp
arm_arm926ej-s
arm_cortex-a15_neon-vfpv4
arm_cortex-a7
arm_cortex-a7_neon-vfpv4
arm_cortex-a8_vfpv3
arm_cortex-a9
arm_cortex-a9_neon
arm_cortex-a9_vfpv3
arm_fa526
arm_mpcore
arm_xscale

Just the arm_cortex ones and arm_mpcore which is a special Cortex A9 variant for some reason.

More generally crusader-arm-unknown-linux-musleabihf works on ARMv7-A (and higher) processors and crusader-aarch64-unknown-linux-musl works on 64-bit ARMv8-A (and higher).

Thanks. I updated the note above with the info so far.
I also see  that the ToH lists aarch64_cortex-53, aarch64_cortex-72, and aarch64_generic. Which of these would work with a pre-built binary? Anything else I should mention? Thanks again

Which of these would work with a pre-built binary?

All of them should.

Thanks. Any comments on the (new) Installing / Running... section? Thanks again

You'd probably want to run it as a service to deal with crashes or reboots.
https://openwrt.org/docs/techref/initscripts / https://openwrt.org/docs/guide-developer/procd-init-scripts

Good point. I updated the description above to mention procd init scripts.
A different question: how much more work do you expect to do on Crusader before it's "good enough" to release to a broader audience? This relates to testing of new features; documentation of any new features; descriptions of how to use it; etc. I'm happy to help with some of this. Thanks again

I'm moving this conversation to a new issue...


Implement Checksum validation

Hi everyone,
this is my implementation of checksum checking, I tried to make my contributions match the existing style in this codebase. Looking forward to your feedback :)
Closes #58

Converted to draft PR, as I've just noticed that Godot doesn't seem to provide checksums for builds up to 3.2.2-beta1, which will need to be handled by informing the user about the missing checksum and asking how to proceed.
I'll extend the PR for that, and if anybody wants, intermediate feedback is greatly appreciated :)

Thanks for the feedback, I'll address it in the coming days, and seeing the failed CI runs, I'll also test it more thoroughly.

Nice addition! Thanks for working on this @Shaderbug.

Hi everyone,
I need some input from you on how we want to proceed with this PR when there's no published checksum for an archive, or a checksum mismatch.
My first plan would be:

If there's a checksum mismatch, or no published checksum file for a given version can be found, it's an unrecoverable error and we abort installation without option to override

Seems like the safest default, and if for some reason there are use cases that require it, an override can be implemented and shipped quickly in an update.

If there is a published checksum file, but it doesn't contain a checksum for a given downloaded file

Skip verification if a --unsafe-allow-files-without-published-checksum flag was passed

This would allow usage in non-interactive sessions

Otherwise print a big warning for the user and ask interactively if they still want to go ahead with the installation
Generally, this is only the case for old Godot versions, so it's unlikely users will encounter this.

+1 to using a flag for both scenarios over interactive prompts.
I like the shorter flag name. Here are a a few suggestions to consider if you'd prefer the flag to include "checksum" in the name.
--allow-unsafe-checksum
--allow-invalid-checksum
--allow-unverified-checksum

I've updated the description as requested. If further questions arise during the review, I'm happy to amend it :)


Add login method without proxy

Since it's a bit cumbersome for the average person to use the proxy, I've written a method that doesn't require to use the proxy. It only works for people that don't use their social media account.
This is my first time using Node.js, I focused on the essential without regards for best practices. Any advise is welcome.
I intend to write a plugin for Jeedom using your code ... when time allows.

Interesting approach ... I tried something like this at the beginning but in fact failed because that gigya stuff was way too much flucuating and so the proxy approach was more stable, but hey when it works we have two ways.
Could you rewrite it to use got as http request library? I think it is not that good to now use two libs for the same purpose in the code ...

PS: And can you please not mit "var" and "let/const" style? :-)

Sure, I'll replace the var with let.
For the rewrite using http request library, I'll need some guidance (a link to the doc would do). When I started writing, I wondered which library I should use and I saw that the request library was deprecated: https://github.com/request/request/issues/3142. Node-fetch was the first in the alternatives list so I went for that one. Translating to any other should be pretty fast, I just need to be sure which one you would prefer.

@csu333 hehe, yes it's a bit "hidden". The OpenId Lib that I decided to use uses "got" internally and so I also decided to got with "got" for the rest of the calls. But yes I should add that to the official deps too and not rely on the underlaying lib.
So got is "just another request lib" (but I think Ido not use request ... do i? ;-)) ) Got is this package https://www.npmjs.com/package/got

Now that you're telling me, I see I could've found out by myself.
The methods look alike so I should be able to convert my requests easily.

There you go. This is probably not by the book, but it works.

Cool, will check tonight. would you mind adding this also to example/tokensavert.js ? Maybe "when you call it with 2 arguments and first contains an @ then these are email and password and no proxy is started but the login method is used? Would be very convenient for the users.

Sounds like a good idea. I slightly adapted the login method to emit the token_update event as well.

Cool, will review tomorrow ... too tired

Thank you for your guidance :-)

No problem, I hope it is ok for that I force it a bit... if it is "toomuch" just tell and I will fix it after merge :-)

and add a catch clause for each call as you propose.

Think about what makes sense. If in the end the action is invalid independent where the error happens maybe also one "big try/catch" (or then maybe really none?) is also ok ... It is helpful to have individual try/catch if you want to give some guidance where it broke. So here 8was late yesterday)basically it works that way, but then no info is given in which step the error happened ... thats why I thought individual catches with "re-throw" after adjusting the err.message could be an idea ...

if it is "toomuch" just tell and I will fix it after merge :-)

Since developing is not exactly in my line of work, I never had the chance of having a code review. Having yours, especially in a language I'm not used to, is a fantastic learning experience. As long as you have the time and patience, feel free to point out anything that doesn't look good to you.

Think about what makes sense. If in the end the action is invalid independent where the error happens maybe also one "big try/catch" (or then maybe really none?) is also ok ...

I'll do my best to make something sensible and useful.

As long as you have the time and patience, feel free to point out anything that doesn't look good to you.

Ok, I will ... you wanted it that way gg ;-)

There you go. I ended up having a dedicated try/catch for each step but each one will stop the processing as there is no possible recovery.
About best practices, I was wondering if it was a good idea to call the "_retrieveTokens" method of the proxy as it is marked private.

I will do proper review tomorrow.

About best practices, I was wondering if it was a good idea to call the "_retrieveTokens" method of the proxy as it is marked private.

Thats difficult in JavaScript because till now there is no provate or protected or whatever. I would leave it that way for now because it is still meaned to be an "internal method" not wanted to be useed by Library users ...

Would be great success if this turns out to be stable solution.
Also promising given the major steps that have been taken to integrate this in Home Assistant for example (the proxy solution works flawlessly now).
Having a user/pass solution makes it possible to integrate everything using the UI.

@csu333 I prepare some changes and will update Githubv soon and would like to to review my changes before I release, ok?!

Sounds great! I'm eager to learn how the code can be improved.
Le 29 juil. 2021 à 20:08, à 20:08, "Ingo Fischer - notifications(a)github.com" @.> a écrit:
@. I prepare some changes and will update Githubv soon and would

like to to review my changes before I release, ok?!
--
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub:
https://github.com/Apollon77/daikin-controller-cloud/pull/17#issuecomment-889353367

Thanks for checking https://github.com/Apollon77/daikin-controller-cloud/commit/edb31eb4ee8939154ffb0cabfb8b1c9ce63245b7

PS: Mainly codeing style and I thorught it is better to rethrow the real error to see what happened and just adjust the message to be more readable

Please laos have a test shoot on it if possible

OK, I see the differences. I wonder why I used double quotes while I usually use single ones. Now I understand the error handling better.
I just tried it and it works perfectly on my account. Ideally, we should have more testers to make sure this is not tailored to my account only.

Cool, I publish it and then I tell the Homeassistent guys :-)) They will test it for sure


UI - platform web bug fixes

Issue:

bcgov/entity/issues/

Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the namex license (Apache 2.0).

Temporary Url for review: https://strr-platform-dev--pr-197-8p9q6zrj.web.app

Nice!

Temporary Url for review: https://strr-platform-dev--pr-197-8p9q6zrj.web.app


chore(main): release 1.4.0

:robot: I have created a release beep boop
1.4.0 (2023-12-03)
Features

改用使用 babel 删除调用 usbuild 的语句 (257dbb5)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/LinLin00000000/usbuild/releases/tag/v1.4.0 :sunflower:


Found invalid values tensor NaN during training period of DECI model.

I used DECI so train a model,the train dimension is 8500*44. In Epoch 1878, the model pops up this error:

Here is the config of the model:

The problem appears the major time. If you could help me to resolve that issue, that would be great. Thanks!

Can you share the training curves?


Option to mark completed request

Subject of the issue
Currently we can see all the requests. But there is no provision to close the registered requests which are processed/completed.
Your environment
Browser
Steps to reproduce
Search for a registered request. You can see the requests but no way to mark it as complete. This will reduced the count of open requests.

This issue is very important as duplicate request are coming

I can take this up.

Discussed in #20

@midhunkrishna   great

Great. If it is not a big change can you add an optional comment/remarks column. It will be useful to state any reasons or a link to FB post or twitter where this is confirmed.


Support git diff

This breaks away some changes from #77.
Specifically, the changes here define a mechanism to get the list of changed file paths between two commits and the ability to derive this from an RSL entry for some namespace.
Note: I've included some test clean up as I noticed I was defining the same config, clock etc a bunch of times. I can submit that separately if needed. That's all in the last commit.

yep, the next step is to use it meaningfully for file-namespace policies.


Add opening and closing animation to the power menu

It already has them. The system ignores them.

It already has them. The system ignores them.

It only works on the locks green any way to fix this ?

No. Like I said, the system ignores them.

No. Like I said, the system ignores them.

Ok thanks anyway. Very very good app by the way thanks for everything!


Refactor status.condition

refactor condition to only present overall module state

/test pre-main-kyma-operator-lint

/test pre-main-kyma-operator-lint


Indirectly support Jupyter notebooks.

Environment data

VS Code version: 1.20.1
Extension version (available under the Extensions sidebar): 2018.2.1
OS and version: Ubuntu 16.04

Feature Request
While providing direct support for Jupyter notebooks (e.g. kernel mgmt, UI) might not be a good fit for the extension, it shouldn't ignore them either.  It may be worth adding some elements that cooperate with the  separate Jupyter extension, including prompting to install it.

@jflam cc

This is better left to Don's Jupyter extension or some data science-specific extension.


Material-UI 1.2.1 not working with Preact

Hi,
Just create a default template preact-cli project, add material-ui and just use Button at https://material-ui.com/getting-started/usage/
An error is thrown: Material-UI<EMAIL_ADDRESS>or greater is required
Error comes from the file https://github.com/mui-org/material-ui/blob/3353f440053222b4eed36f575593b5bfcfa7a0f3/packages/material-ui/src/ButtonBase/ButtonBase.js and line 48.
How can we make React.createContext?
Can you please help?

Use an older version of material-UI for now. Adding fake API's to preact-compat isn't going to solve this issue for you, we need fundamental changes like fragments before we have proper React 16 compatibility.

We do the following in in order to check the support for getDerivedStateFromProps().
if (process.env.NODE_ENV !== 'production' && !React.createContext) {
  throw new Error('Material-UI<EMAIL_ADDRESS>or greater is required.');
}

We can improve the detection logic if Preact support this hook.

@developit is there an issue tracking the development of React.Fragment?

Yes there is developit/preact#946, developit/preact#1047 and a wip PR for fragements developit/preact#1080

We just shipped getDerivedStateFromProps() in Preact 8.3.0. createContext() likely won't be coming to Preact core, we will need to integrate something like preact-context into compat (see #475).

@developit It's good to know! We the Fragment and Portal API, we should be able to make Material-UI work with Preact.

is there a workaround for this at the moment?

@coder82 We are planning on using the new context API of React. Things won't get any better.

@oliviertassinari  So what do say...will we not able to make preact-compat or inferno-compat work with Material-UI?

@sriharrsha It's going to be a large effort. This will be even more true after the hook migration.

Yeah, Is there any other alternative UI Lib you would recommend for Preact

@sriharrsha I have never used Preact, I can't help with that.

Just an FYI: That we will add hooks in a point release when they are marked as stable. Preact X adds Fragments so there's one less thing to be worried about.

@marvinhagemeister Great for the Fragment support 👍. I guess we only miss createContext() then.

Whats the current status on this? since preact X inplemented most features of hooks, are we able to use material ui with preactX?

@mnues01 you should be able to use it just fine with Preact X. We're not aware of any issues with it and iirc some people in our slack channel are using it.
Just make sure to alias to preact/compat instead of the library in this repo.
Closing, because it's unlikely that we'll backport everything to 8.x.

Also, I would rather start with Material-UI v4 that was 100% rewritten with hooks than v1. v1 might still rely on some legacy lifecycles and private context API, I'm not sure Preact v10 supports it. Starting with v4 is much safer. You can find an example: https://github.com/mui-org/material-ui/tree/master/examples/preact

yes i am using v4 and preact/compat it compiles, but i am having some issue with modal on  component.
any idea?

@mnunes01 Hum, it sounds like an issue with the ref triggers order. We might change ref the logic a bit in https://github.com/mui-org/material-ui/pull/16597. Could you try again when it land? Alternatively, if you want to dive into the problem, feel free to.

@oliviertassinari i guess it is indeed, going a bit deeper on it, it seems some async issue.
if i do pause the debugger just before the if(!containerInfo.restore) the debugger shows containerInfo as undefined, but by the time it throws the error execption containerInfo is and object with the correct values.


General Script API

Features every script should have access to, i.e.:

AddComponent
RemoveComponent
SpawnEntity

etc.
In the Lua case, something like this: https://github.com/PROMETHIA-27/bevy_mod_lua should work

from what i understand , bevy's normal app world (where script hosts exist) is not the same as render app world. does that mean we cannot use lua (or rhai) to do custom rendering in bevy?
I was wondering on how to expose egui paint callbacks to lua and i cannot find a way yet.
not a feature request, just wanted to confirm if its a hard limitation (in that case scripts can still spawn / despawn the Text or Gltf render functionality with commands from bevy after #10 merges ) or whether there is a workaround.

Hmm, I am not familiar with the render world, but from a quick read, it appears the worlds are synchronized each frame via ExtractResource and the like.
Are you using bevy_egui ? Could you point me to the callback functionality you mention in their code/docs?
But yeah #10 will add a whole lot of the bevy interface in, namely everything that implements reflect will be supported, including resources and components.

bevy_egui doesn't implement it yet. although it should be possible as rust callbacks can be static and sent across threads. bevy_egui also clones the egui context  into the render app during extract stage to use it inside the render stage of normal egui stuff (without callbacks). egui context is Arc<Mutex<ContextImpl>>, which might cause performance issues if both threads try to lock / use it at the same time as pipelined rendering happens in parallel to the main bevy app systems.
so, if Script hosts are also inside Arc<Mutex<Lua>>  they can be cloned / shared into render app.

although i don't think if we can send the lua callbacks as they still have 'lua lifetime, i can just be happy with a normal LuaRender event / hook and the script can handle the rest by itself.
we also need to use their unique ID to make sure that the render event is triggered in main app will reach the right script in render sub app as the entity ids might be different between both of them for the same script.
even if bevy_mod_scripting doesn't support the full extent of bevy_render functionality (which is probably huge and idk if its worth the effort), we can just make lua bindings for the wgpu + naga (shaders) and let the scripts do raw calls.

one quirk to note is that:

Every frame, all entities in the render World are erased, but resources are kept. If you need to persist data from frame to frame, store it in resources. Dynamic data that could change every frame should be copied into the render world in the Extract stage, and typically stored using entities and components.

Hmm so if I understand correctly:
In order to make use of the paint callback functionality, you need to be able to send a callback to the render world of this type?
Arc<dyn Fn(&PaintCallbackInfo, &mut (dyn Any + 'static)) + Send + Sync + 'static>

Yeah in that case, as you say, there is no way you'll send any lua-level callbacks from mlua since all functions are of type
Function<'lua>

which is non static.
The only leftover solutions are going to be trying to pass callbacks which invoke the script_host to check for lua callbacks and running them I think. This would probably mean adding some Arc's into the code.
I am happy to accept PR's for this if you come up with a solution, since I am currently focusing on other parts. You're more familiar with both egui and the bevy rendering pipeline to implement this if you want to.

if ScriptContext was Arc<Mutex<Lua>>, and just like lua gets a bevy world pointer, if lua gets a  weak_ptr to its own context Arc<Mutex<Lua>>, then we can impl callbacks like this:
lets say our lua script has a hook function called "on_render".
/// this lua function takes 2 arguments in addition to egui context.
/// Arguments:
/// render_callback_id: String. this will be provided to the "on_render" hook and the script can use this as a key to decide what it needs to render.
/// lua_vm_context: this is the script context's (Arc<Mutex<Lua>>) weak pointer.
fn egui_callback(lua: &Lua, egui_ctx: egui::Context, render_callback_id: String, lua_vm_context:  Weak<Mutex<Lua>>) {
	let lua_vm_context = lua_vm_context.upgrade().unwrap(); // now we have the script context (Arc<Mutex<Lua>>)
	// create the actual callback
	let lua_paint_callback = Arc::new( move |info: &PaintCallbackInfo, renderer: &Renderer| {
		// get lua vm and call the "on_render" function hook with "render_callback_id" as first argument and info / renderer as second and third arguments
		lua_vm_context.lock().globals().get("on_render").call(...)
	}
	// now this callback is static and we don't need to make any changes to bevy_mod_scripting except wrapping scriptcontext in Arc and providing a WeakPtr as a global in lua vm.
}

having a Weak<Mutex<Lua>> is better as we avoid Arc getting stuck in cyclic reference problem.
there's no urgency though. because bevy_egui doesn't support the callbacks functionality anyway.  and lua would need wgpu / naga bindings first before this is actually useful. I am just happy to know that this is possible.

Nice, that looks solid! Wrapping the contexts in an arc is probably the better move since they are inserted removed often!


Incorrect refactor of "Use forM_"

Version: reproducible from master head (ac5a0047)
Minimal example:
import Data.Traversable -- /tmp/min.hs

main = forM [] print >> pure ()

$ stack exec -- hlint /tmp/min.hs
min.hs:3:8: Warning: Use forM_
Found:
  forM [] print
Perhaps:
  forM_ [] print

1 hint

This refactor results in a compilation error because forM_ is not in scope - I would  expect output message to inform me that I haven't brought any forM_ into scope yet.

It used to be that forM and forM_ were in the same module, so we didn't bother. Now we should default to showing Control.Monad.forM_ and dequalify that with the standard things. Fairly low priority though - it would take a fair bit of stuff to get it working, and this area is being actively reworked as we speak.


Implement dependency mapping

This implements dependency mapping for publishing POMs, meaning that for any given declared dependency, we publish a mapped version of that dependency. We resolve all dependencies for a variant using a secondary resolvable configuration that has the same dependencies as the consumable configuration being published.
We can cross-reference the declared dependencies with the first-level edges of a resolution result from the resolution configuration. This lets us know what variant each dependency resolves to. Then, we publish the resolved coordinates.
This specifically does not handle the case where non-consumable configurations are published, as AGP and KGP do. This would require an extra round of attribute matching, which while possible, is not implemented in this PR. The extra round of matching is by nature inefficient since we already perform matching during graph resolution. Furthermore, such an implementation would require non-negligible code duplication.
An internal API is added to expose this new functionality. When adding a variant to a component, an additional resolution configuration can be specified. Once this new functionality is stabilized, we will make the API public.
This new implementation is able to perform "standard" version mapping, as it exists currently. We can also switch a flag in the future to implement the existing version mapping API with the new functionality.
This means that this PR aims to have no behavior changes. Dependency mapping is hidden behind an internal API and is disabled by default. Furthermore, the existing version mapping code path remains intact.

@bot-gradle test this

I've triggered the following builds for you. Click here to see all build failures.

PullRequestFeedback build

@bot-gradle test this

I've triggered the following builds for you. Click here to see all build failures.

PullRequestFeedback build

@bot-gradle test this

I've triggered the following builds for you. Click here to see all build failures.

PullRequestFeedback build

I've triggered the following builds for you. Click here to see all build failures.

BuildDistributions build

@bot-gradle test and merge

I've triggered a build for you. Click here to see all failures if there's any.


Error on plugins page

Clicking on "Plugins" and "Attachments" (to load the/plugins/netbox-attachments/netbox-attachments/ page) fails with an exception:
<class 'django.core.exceptions.FieldError'>

Cannot resolve keyword 'size' into field. Choices are: comments, content_type, content_type_id, created, custom_field_data, file, id, journal_entries, last_updated, name, object_id, parent, tagged_items, tags

Python version: 3.9.2
NetBox version: 3.3.9

Netbox-attachments version: 1.0.6
This is happening on our production instance (docker) as well as our dev server (VM installation).   But, I cannot replicate the problem locally (docker).  In all cases, I've restored the database from the production instance.  I've changed the backend config (from S3 back to default/local) and it still fails.  I've also upgraded Netbox 3.3.8 to 3.3.9.
Can you suggest where I might look to troubleshoot and isolate the cause?

@oasys I'll take a look at it.
I'm using the same version at our production and this problem does not occur.
Does the error show some file location where the problem shows up?

The problem is when you try to order by file size.
The size of the file is not stored in the database, so ordering will fail.
@oasys  Should I add the file size to the database, or forbid the ordering and etc. on it?
When you go to  NETBOX/user/preferences/ and clear out the table preferences, you should see the plugin page without a problem.

Ok, I cannot find how to disable ordering. So I guess I will add file size to the database table.

Clearing the sort preference does work around the issue, thanks.
I saw that #24 attempts to fix this in a more general way, but that fails for me with:
ModuleNotFoundError: No module named 'utilities.json'

Clearing the sort preference does work around the issue, thanks.
I saw that #24 attempts to fix this in a more general way, but that fails for me with:
ModuleNotFoundError: No module named 'utilities.json'

Could you provide some more details? Traceback etc.
I'm using this new version in production, and it work fine.

@oasys  it seems utilities.json is a library created by NetBox.
https://github.com/netbox-community/netbox/blob/develop/netbox/utilities/json.py
So I don't know why your instance cannot be found it as a module.

Oh okay, i think I found the problem.
You are using NetBox 3.3.9
but utilities.json was added at the latest version 3.4.3 I guess https://github.com/netbox-community/netbox/commit/af8bb0c4b9c0b8d6599f110f522bd1e598c40b88
And when I did a migration ( https://github.com/Kani999/netbox-attachments/pull/24/files#diff-a021e1288da10ed634e63b73cfae1a8b5b44908d8b74c404eca857982451326dR19 ) There was another line generated automatically by NetBox, which uses this.


LV: Changed two files

submitted via email by<EMAIL_ADDRESS>

Android 1.0.7.-acc : both certs are valid

iOS 1.0.5 both new codes are valid


is it possible to enable expiration in the headers

Hi,
I just played around with the imageproxy and looked up what kind of headers are send down to the browser. I realized that expiration is missing. Is it somehow possible to enable them?
I'm using -cache s3 storage and there the cache-header is missing for the objects. Is there any way to append them with the expiration and/or cache-control on upload?
Thanks

I was looking into the aws go source (https://github.com/aws/aws-sdk-go/blob/master/service/s3/s3manager/upload.go)
	CacheControl *string `location:"header" locationName:"Cache-Control" type:"string"`

Is it possible to set CacheControl inside of the variables of the proxy somehow?

After a long journey of researching I found out that the images I scaled didn't have a cache-header. It would be nice if a config could be made to set these in case they are missing:
copyHeader(w.Header(), resp.Header, "Cache-Control", "Last-Modified", "Expires", "Etag", "Link")

Currently, imageproxy passes along the same caching headers that the original image has (or at least it should... if it's not, then that's a bug).  I'm not opposed to having a setting that allowed setting a default expiration time, so long as upstream instructions like Cache-Control: no-cache are still respected.  See existing discussion in #28.

@willnorris the idea would be to have the power to overwrite the default behaviour. I use for example an image repository where no cache-control header are present. This means that the use of this imageproxy only can be used if I manipulate the cache-control response before the delivery.
Currently I do this with lambda in AWS but I'd like to see this kind of feature inside of the imageproxy already - otherwise the whole caching mechanism is not working.


Add features to help with RTL languages

Is your feature request related to a problem? Please describe.
I want to use Dialogic with an RTL language (Hebrew) and it is usable, but you have to type all the dialog in reverse in order to get coherent text, which is relatively fine, but some features like text speed just don't work well with it (text appears from last to first characters)
Describe the solution you'd like
Attempt to make certain features like text speed RTL language friendly. Example is making the text appear in reverse order when using text speed.
An option to automatically reverse the text so that it will appear and be typed coherently
Describe alternatives you've considered
Typing the text in an external way, reversing it and pasting into Dialogic.
Using a text speed of 1 which makes text appear instantly

I'll see what things I can add to improve this 👍🏻

After using it for my actual project I also noticed that while you can align the text to the right side you cannot align the speaker's name to the right side

Unfortunately Godot 3.x doesn't support RTL languages, so I can't do much more at the moment. You can set the dialog speed to 0 (so you won't see the animation letter by letter) and then align the name/text to right. I will revisit this when Dialogic comes to Godot 4 and I can implement better support for RTL. Sorry but if you think there is anything else I could implement I can give it a try :)

Okay, that's perfectly understandable. Thanks for all the help so far and hopefully in the future!


tsconfig extend does not deep-clone compilerOptions.types

Bug Report

🔎 Search Terms

extends tsconfig types compilerOptions Cannot find module or its corresponding type declarations svelte
🕗 Version & Regression Information

Affected version is 4.3.2 and earlier

⏯ Playground Link

Repository with relevant code
💻 Code
MCVE: https://github.com/tholewebgods/typescript-extend-not-deep-clone-mcve
🙁 Actual behavior

The compiler does not load modules of a specific type configured via extends config when root config also specifies types.
🙂 Expected behavior
When extending from configs types shall be merged.

Related: #27098

This is the intended behavior; if you didn't want to copy down the parent types, you'd have no way of expressing that, whereas today you can at least write what you mean. It'd be a fairly large breaking change in build system to change this now, especially since there'd be no workaround apart from editing the upstream config file (which you'd have no way of tracking down the child dependencies of).

Sorry for reopening this, but I also stumbled over this behavior. Wouldn't it make sense to point this out in the documentation then? What else is excluded from inheritance then, is there a document list?


I have added support to Windows

Please merge this pull-request. I have added support to windows platform.
To build for windows, just need to do:
npm run build-win
I also attaching a ZIP file, that contain the profram + .exe file. you can directly put in the releases here on GitHub.
Thank you.

By the way, I have testes it, the shourtcuts, and everything works as expected.