Error on publish

I'm trying to copy my corp's private npmjs repositories to an on-premise hosting (using Nexus OSS 3.3.0), using this I do
npm-copy --from https://registry.npmjs.org --from-token <token-taken-from-dotnpmrc> --to http://nexus.my.corps.lan:8081/repository/npm-my-corp --to-username admin --to-password <my-admin-password @my-corp%2fapp-status @my-corp%2fapp-update

which gives the following errors
info attempt registry request try #1 at 12:56:28 PM
http request GET https://registry.npmjs.org/@my-corp%2fapp-status
http 200 https://registry.npmjs.org/@my-corp%2fapp-status
info attempt registry request try #1 at 12:56:28 PM
http request GET http://nexus.my.corps.lan:8081/repository/npm-my-corp/@my-corp%2fapp-status
http 404 http://nexus.my.corps.lan:8081/repository/npm-my-corp/@my-corp%2fapp-status
info attempt registry request try #1 at 12:56:28 PM
http fetch GET<EMAIL_ADDRESS>http fetch 200<EMAIL_ADDRESS>Error: auth required for publishing
    at RegClient.publish (/usr/lib/node_modules/npm-copy/node_modules/npm-registry-client/lib/publish.js:30:14)
    at RegClient.<anonymous> (/usr/lib/node_modules/npm-copy/node_modules/fibrous/lib/fibrous.js:50:17)
    at RegClient.<anonymous> (/usr/lib/node_modules/npm-copy/node_modules/fibrous/lib/fibrous.js:66:29)
    at Object.publish (/usr/lib/node_modules/npm-copy/node_modules/fibrous/lib/fibrous.js:101:47)
    at /usr/lib/node_modules/npm-copy/lib/cli.js:83:26
    at /usr/lib/node_modules/npm-copy/lib/cli.js:100:7
    at /usr/lib/node_modules/npm-copy/node_modules/fibers/future.js:467:21

For what it's worth, we cloned this project and made some updates: https://www.npmjs.com/package/@appsweet-co/npm-carbon
We will actively maintain the new @appsweet-co/npm-carbon package.


Support gif urls

:wave:
This pull request adds an upload form field below the recently added dropzone.
When the "upload" button is clicked the URL is passed to the existing logic to create and display the chip-ified GIF.

I have also added some regex validation to the URL field to check that the URL ends with a .gif extension.
Hope you enjoy your time off!
Closes #2

Hi @stphnnnn !
Thanks for implementing #2 💯
I love that you added validation as well. Your code changes were also really elegant 👍
I made a small change to the validation to remove the .gif extension requirement because this will differ from host to host and sometimes a gif can be served without the extension on the url. I hope that's ok! 🙇♀


re-based my master branch, please use this request

I've added a figure for a proposed branching model, to keep one stable branch, which always compiles on all systems. please check


DOC-3050 change job ID description for DETACHED backup

Addresses DOC-3050

Fixed wording for on when the job ID is returned in a DETACHED backup
Changes made in v22.2, 22.1, 21.2,. 21.1

Q's:

Should the fact that returning the job ID takes '10's of seconds and in the extreme several minutes' be included?

Staging:

BACKUP - Options

This change is

@ashleykwtam In response to your question — no, I think since there is this possible time span in difference depending on the backup, what you have written covers the issue here. 👍

Changes applied to v20.2
Fixed wording in the DETACHED examples

Staging:

BACKUP - Options
Example

Staging:

Restore


fix bugs in run.py and constants.py

delete documantation of send_message function in src/run.py
delete bot.run()   in src/run.py.
update constants.py to add keys.exit into keyboards.

An important thing is “break up each change into individual commits”.
Here I see three different changes. One, doc fix; Second, adding a functionality and, third, fixing a (maybe redundant) line in code.
Please break them up to three different commits.
Sent from Mailhttps://go.microsoft.com/fwlink/?LinkId=550986 for Windows
From: @.>
Sent: Monday, December 20, 2021 01:37
To: @.>
Cc: @.***>
Subject: [pytopia/project-template-telegram-bot] fix bugs in run.py and constants.py (PR #1)

delete documantation of send_message function in src/run.py
delete bot.run() in src/run.py.
update constants.py to add keys.exit into keyboards.

You can view, comment on, or merge this pull request online at:
https://github.com/pytopia/project-template-telegram-bot/pull/1
Commit Summary

a47ab3ehttps://github.com/pytopia/project-template-telegram-bot/pull/1/commits/a47ab3e92201f0e6e4063388046e663f3b66798e fix bugs in run.py and constants.py

File Changes
(2 fileshttps://github.com/pytopia/project-template-telegram-bot/pull/1/files)

M src/constants.pyhttps://github.com/pytopia/project-template-telegram-bot/pull/1/files#diff-42d0f00c4780305311eae71fecca993a0523587cb4156e039cabd0d1ad8dbbb8 (2)
M src/run.pyhttps://github.com/pytopia/project-template-telegram-bot/pull/1/files#diff-525565c956f875698bc74ac798b474aa561f4570464b770434888cfbf34bde48 (6)

Patch Links:

https://github.com/pytopia/project-template-telegram-bot/pull/1.patch
https://github.com/pytopia/project-template-telegram-bot/pull/1.diff

—
Reply to this email directly, view it on GitHubhttps://github.com/pytopia/project-template-telegram-bot/pull/1, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ABOG2U3JC7UVJIJYQSISSC3URZJQRANCNFSM5KMLEDDQ.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you are subscribed to this thread.Message ID: @.***>

An important thing is “break up each change into individual commits”. Here I see three different changes. One, doc fix; Second, adding a functionality and, third, fixing a (maybe redundant) line in code. Please break them up to three different commits. Sent from Mailhttps://go.microsoft.com/fwlink/?LinkId=550986 for Windows From: @.> Sent: Monday, December 20, 2021 01:37 To: @.> Cc: @.> Subject: [pytopia/project-template-telegram-bot] fix bugs in run.py and constants.py (PR #1) * delete documantation of send_message function in src/run.py * delete bot.run() in src/run.py. * update constants.py to add keys.exit into keyboards. You can view, comment on, or merge this pull request online at: #1 Commit Summary * a47ab3e<a47ab3e> fix bugs in run.py and constants.py File Changes (2 fileshttps://github.com/pytopia/project-template-telegram-bot/pull/1/files) * M src/constants.pyhttps://github.com/pytopia/project-template-telegram-bot/pull/1/files#diff-42d0f00c4780305311eae71fecca993a0523587cb4156e039cabd0d1ad8dbbb8 (2) * M src/run.pyhttps://github.com/pytopia/project-template-telegram-bot/pull/1/files#diff-525565c956f875698bc74ac798b474aa561f4570464b770434888cfbf34bde48 (6) Patch Links: * https://github.com/pytopia/project-template-telegram-bot/pull/1.patch * https://github.com/pytopia/project-template-telegram-bot/pull/1.diff — Reply to this email directly, view it on GitHub<#1>, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ABOG2U3JC7UVJIJYQSISSC3URZJQRANCNFSM5KMLEDDQ. Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub. You are receiving this because you are subscribed to this thread.Message ID: @.>

Agreed Alireza, but this these are very small fixes, I think it's ok to merge them into on PR fix commit.
I think if we change the commit message to fix minor bugs that should be OK.

@arf1372
Thanks for your good advice. I will try to observe that.

An important thing is “break up each change into individual commits”. Here I see three different changes. One, doc fix; Second, adding a functionality and, third, fixing a (maybe redundant) line in code. Please break them up to three different commits. Sent from Mailhttps://go.microsoft.com/fwlink/?LinkId=550986 for Windows From: @.> Sent: Monday, December 20, 2021 01:37 To<EMAIL_ADDRESS>Cc<EMAIL_ADDRESS>Subject: [pytopia/project-template-telegram-bot] fix bugs in run.py and constants.py (PR #1) * delete documantation of send_message function in src/run.py * delete bot.run() in src/run.py. * update constants.py to add keys.exit into keyboards. You can view, comment on, or merge this pull request online at: #1 Commit Summary * a47ab3e<a47ab3e> fix bugs in run.py and constants.py File Changes (2 fileshttps://github.com/pytopia/project-template-telegram-bot/pull/1/files) * M src/constants.pyhttps://github.com/pytopia/project-template-telegram-bot/pull/1/files#diff-42d0f00c4780305311eae71fecca993a0523587cb4156e039cabd0d1ad8dbbb8 (2) * M src/run.pyhttps://github.com/pytopia/project-template-telegram-bot/pull/1/files#diff-525565c956f875698bc74ac798b474aa561f4570464b770434888cfbf34bde48 (6) Patch Links: * https://github.com/pytopia/project-template-telegram-bot/pull/1.patch * https://github.com/pytopia/project-template-telegram-bot/pull/1.diff — Reply to this email directly, view it on GitHub<#1>, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ABOG2U3JC7UVJIJYQSISSC3URZJQRANCNFSM5KMLEDDQ. Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub. You are receiving this because you are subscribed to this thread.Message ID: @.**_>

Agreed Alireza, but this these are very small fixes, I think it's ok to merge them into on PR fix commit.

Uh, I think having separate commit for each change worth it, specially when dealing with "git bisect" and "git blame" stuff.
So, I still insist on breaking commits into fine grained changes per commit.


Add a consistency check on version and core_version

Fixes chance of misconfiguring versions between BUILD and build_handwritten.yaml as introduced in #22991 and fixed in #23576
Although there will eventually be a single source-of-truth for information and build_handwritten.yaml will be deprecated, we need a consistency check in the meanwhile.

Note that this will fail its own sanity check until #23576 is merged.

Thanks for the review! Yes, that would be great to not need this check in the future.


Create a new pull request by comparing changes

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.1%) to 95.8%

Totals

Change from base Build<PHONE_NUMBER>:
0.1%

Covered Lines:
1982

Relevant Lines:
2030

💛  - Coveralls


Incorrect behaviour of CTRL+Backspace hotkey in TextBox

Tested on windows 10 with avalonia 0.5.0 and 0.5.2-build3963-alpha on core project created by Avalonia for Visual Studio extension.

AvaloniaCoreBackspaceProblemApp.zip

#1073


Feature request: Make layer available in GovCloud

Use case
I would like to use Powertools for my project as a Lambda layer. One of the regions I have to support is in GovCloud, which means I cannot use this layer as it currently stands.
Powertools for Python is available in GovCloud (source), so I'd like to request similar functionality here. Thanks for your consideration.
Solution/User Experience
Powertools available as a Lambda layer in GovCloud.
Alternative solutions

Acknowledgment

[x] This feature request meets Powertools for AWS Lambda (TypeScript) Tenets
[ ] Should this be considered in other Powertools for AWS Lambda languages? i.e. Python, Java, and .NET

Future readers
Please react with 👍 and your use case to help us understand customer demand.

HI @rschwartz-qualtrics, thank you for opening this issue.
The request makes sense, allow us to discuss this internally and understand if we can reuse the work we have already done to make it available for the other runtime.


Any suggestions on how to add start button and reset button to the demo?

The thread that polls the cursor position and adds the data to the chart is launched from the MainWindowVM constructor: https://github.com/royben/RealTimeGraphX/blob/master/RealTimeGraphX.WPF.Demo/MainWindowVM.cs#L74. You would have to move the thread management into its own method and then hook that up to a control in the GUI.


adds stream::chain combinator

Adds a Chain combinator and Introduces some changes to Fuse so it's usable in this case, but those need a closer look.

Ref: #129
Stdlib: https://doc.rust-lang.org/std/iter/trait.Iterator.html#method.chain

bors r+


push error

Hello guys,
I manged to set up my Git Thing but on push I receive the below error. On pull everything works great. I tried also with different git repositories but none of them works.
PS: the token has rights to do it.
java.lang.ClassCastException: com.thingworx.types.primitives.StringPrimitive cannot be cast to com.thingworx.types.primitives.BooleanPrimitive
at gb.GitBackupTemplate.Push(GitBackupTemplate.java:221)
at sun.reflect.GeneratedMethodAccessor140.invoke(Unknown Source)
at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
at java.lang.reflect.Method.invoke(Method.java:498)
at com.thingworx.common.processors.ReflectionProcessor.processService(ReflectionProcessor.java:230)
at com.thingworx.handlers.ReflectionServiceHandler.processService(ReflectionServiceHandler.java:50)
at com.thingworx.handlers.ServiceHandlerBase.processServiceWithMetrics(ServiceHandlerBase.java:48)
at com.thingworx.things.Thing.processServiceRequestDirect(Thing.java:8151)
at com.thingworx.things.Thing.processAPIServiceRequest(Thing.java:8088)
at com.thingworx.webservices.BaseService.handleInvoke(BaseService.java:3304)
at com.thingworx.webservices.BaseService.service(BaseService.java:489)
at javax.servlet.http.HttpServlet.service(HttpServlet.java:733)
at org.apache.catalina.core.ApplicationFilterChain.internalDoFilter(ApplicationFilterChain.java:231)
at org.apache.catalina.core.ApplicationFilterChain.doFilter(ApplicationFilterChain.java:166)
at org.apache.tomcat.websocket.server.WsFilter.doFilter(WsFilter.java:53)
at org.apache.catalina.core.ApplicationFilterChain.internalDoFilter(ApplicationFilterChain.java:193)
at org.apache.catalina.core.ApplicationFilterChain.doFilter(ApplicationFilterChain.java:166)
at com.thingworx.security.contenttype.ContentTypeFilter.doFilter(ContentTypeFilter.java:138)
at org.apache.catalina.core.ApplicationFilterChain.internalDoFilter(ApplicationFilterChain.java:193)
at org.apache.catalina.core.ApplicationFilterChain.doFilter(ApplicationFilterChain.java:166)
at com.thingworx.security.contenttype.ContentLengthFilter.doFilter(ContentLengthFilter.java:73)
at org.apache.catalina.core.ApplicationFilterChain.internalDoFilter(ApplicationFilterChain.java:193)
at org.apache.catalina.core.ApplicationFilterChain.doFilter(ApplicationFilterChain.java:166)
at com.thingworx.security.filter.ValidationFilter.doFilter(ValidationFilter.java:22)
at org.apache.catalina.core.ApplicationFilterChain.internalDoFilter(ApplicationFilterChain.java:193)
at org.apache.catalina.core.ApplicationFilterChain.doFilter(ApplicationFilterChain.java:166)
at com.thingworx.security.authentication.AuthenticationFilter.propagateRequest(AuthenticationFilter.java:689)
at com.thingworx.security.authentication.AuthenticationFilter.doFilter(AuthenticationFilter.java:229)
at org.apache.catalina.core.ApplicationFilterChain.internalDoFilter(ApplicationFilterChain.java:193)
at org.apache.catalina.core.ApplicationFilterChain.doFilter(ApplicationFilterChain.java:166)
at org.tuckey.web.filters.urlrewrite.RuleChain.handleRewrite(RuleChain.java:176)
at org.tuckey.web.filters.urlrewrite.RuleChain.doRules(RuleChain.java:145)
at org.tuckey.web.filters.urlrewrite.UrlRewriter.processRequest(UrlRewriter.java:92)
at org.tuckey.web.filters.urlrewrite.UrlRewriteFilter.doFilter(UrlRewriteFilter.java:389)
at org.apache.catalina.core.ApplicationFilterChain.internalDoFilter(ApplicationFilterChain.java:193)
at org.apache.catalina.core.ApplicationFilterChain.doFilter(ApplicationFilterChain.java:166)
at com.thingworx.security.filter.ClickjackFilter.doFilter(ClickjackFilter.java:208)
at org.apache.catalina.core.ApplicationFilterChain.internalDoFilter(ApplicationFilterChain.java:193)
at org.apache.catalina.core.ApplicationFilterChain.doFilter(ApplicationFilterChain.java:166)
at com.thingworx.security.filter.HttpResponseHeadersFilter.doFilter(HttpResponseHeadersFilter.java:172)
at org.apache.catalina.core.ApplicationFilterChain.internalDoFilter(ApplicationFilterChain.java:193)
at org.apache.catalina.core.ApplicationFilterChain.doFilter(ApplicationFilterChain.java:166)
at org.apache.catalina.core.StandardWrapperValve.invoke(StandardWrapperValve.java:202)
at org.apache.catalina.core.StandardContextValve.invoke(StandardContextValve.java:96)
at org.apache.catalina.authenticator.AuthenticatorBase.invoke(AuthenticatorBase.java:541)
at org.apache.catalina.core.StandardHostValve.invoke(StandardHostValve.java:139)
at org.apache.catalina.valves.ErrorReportValve.invoke(ErrorReportValve.java:92)
at org.apache.catalina.valves.AbstractAccessLogValve.invoke(AbstractAccessLogValve.java:690)
at org.apache.catalina.core.StandardEngineValve.invoke(StandardEngineValve.java:74)
at org.apache.catalina.connector.CoyoteAdapter.service(CoyoteAdapter.java:343)
at org.apache.coyote.http11.Http11Processor.service(Http11Processor.java:373)
at org.apache.coyote.AbstractProcessorLight.process(AbstractProcessorLight.java:65)
at org.apache.coyote.AbstractProtocol$ConnectionHandler.process(AbstractProtocol.java:868)
at org.apache.tomcat.util.net.NioEndpoint$SocketProcessor.doRun(NioEndpoint.java:1589)
at org.apache.tomcat.util.net.SocketProcessorBase.run(SocketProcessorBase.java:49)
at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1149)
at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624)
at org.apache.tomcat.util.threads.TaskThread$WrappingRunnable.run(TaskThread.java:61)
at java.lang.Thread.run(Thread.java:748)

Hi, The push operation has been tested and it works. I would like to know more details about your scenario and how you configured your Git Backup Thing:
-Are you trying to push some files to an empty Git repository or to a repository where you already have some files? If you already have some files in the repository, you first need to pull them in thingworx...add your additional/modified files and then push it.
-What repository path are you using in your ThingWorx repository ? Try using a path like "/myproject" instead of the root path of the ThingWorx repository
-Have you tested the Push operation in the Mashup or also by executing the service in the Git  Backup Thing?

Hi, The push operation has been tested and it works. I would like to know more details about your scenario and how you configured your Git Backup Thing:
-Are you trying to push some files to an empty Git repository or to a repository where you already have some files? If you already have some files in the repository, you first need to pull them in thingworx...add your additional/modified files and then push it.
-What repository path are you using in your ThingWorx repository ? Try using a path like "/myproject" instead of the root path of the ThingWorx repository
-Have you tested the Push operation in the Mashup or also by executing the service in the Git Backup Thing?

I already have some files on the git. I did the pull first, and the pull works fine.
I am already using a path like /entities instead of root path
I tested Push operation in the Mashup but also in the Git Backup Thing. - same error

Thanks for sharing the information. One additional questions: are you using the latest Git extension on ThingWorx 9+ ? Please also share the version of the extension that you are using.
The error you provided can appear if the UserExtensions thing shape has not been initialized correctly. It is supposed to be initialized when you first open the Git Main Mashup. If you edit any user inside the Composer, can you let me know if it has the property "UseGitCommitUserValues" in the User Extensions?

Thanks for sharing the information. One additional questions: are you using the latest Git extension on ThingWorx 9+ ? Please also share the version of the extension that you are using.
The error you provided can appear if the UserExtensions thing shape has not been initialized correctly. It is supposed to be initialized when you first open the Git Main Mashup. If you edit any user inside the Composer, can you let me know if it has the property "UseGitCommitUserValues" in the User Extensions?

Yes, I am using the latest Git Extension, version 4.1.0.

I am not that expert in TWX. Maybe you can give me more details how can I do this?

In the ThingWorx Composer, click on Browse, click on Users and then select a User. Go to the UserExtensions tab and verify the list of properties displayed. There should be a property called "UseGitCommitUserValues"
I have attached a screenshot.
Also, I will close this issue since it doesn't represent a bug and the push operation has been verified and works as expected. If you have further questions about how to use the Git extension, you can contact me by e-mail, which is displayed in my profile.


Implement RawValue type

Implements the RawValue type described in #355. On this benchmark, using the RawValue type instead of deserializing/serializing via Value results in a 3-4x speed-up.
test deserialize_and_serialize_raw_value ... bench:       1,759 ns/iter (+/- 354)
test deserialize_and_serialize_value     ... bench:       6,091 ns/iter (+/- 1,776)

I did not adhere fully to the way that the existing arbitrary precision feature is implemented, because I found it easier to use newtypes rather than structs to pass the raw data back and forth. If you want, I'm happy to either change it so that it uses the same approach as the arbitrary precision feature, or to change the arbitrary precision feature to use this (imo) simpler approach.
In the documentation for RawValue I mentioned two important caveats which apply for this implementation:

When serializing, a RawValue will retain its original formatting and will not be minified or pretty-printed.
When deserializing, RawValue can not be used with the #[serde(flatten)] attribute, as it relies on the original input buffer.

What you described in #323 might be a cleaner solution to these problems, but it also sounds more complex and resource hungry (using 4x-8x the amount of memory).
Let me know what you think!

Awesome! Thank you. :+1:

I did not adhere fully to the way that the existing arbitrary precision feature is implemented, because I found it easier to use newtypes rather than structs to pass the raw data back and forth. If you want, I'm happy to either change it so that it uses the same approach as the arbitrary precision feature, or to change the arbitrary precision feature to use this (imo) simpler approach.

The way Number does it is more robust than newtypes when mixing with data formats that are not JSON. See 7e373b6125ee6f19bb9a9538c51c92bf51be1f90.

Ah, I was looking for an explanation like that, might be worth adding a code comment to make this clear to anyone coming across this in the future.
As I said on the other PR (#485), do let me know if I can help move this forward in any way!


fix: Change table name to avoid name conflicts

Pull Request
Description
Fix for changing table name to avoid the issue because of name conflicts.
Type of change
Please delete options that are not relevant.

[X] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[X] Code cleanup/refactoring
[ ] Documentation update
[ ] This change requires a documentation update
[ ] CI system update
[ ] Test Coverage update

Testing
Test Configuration:

Java: v17
LPVS Release: v1.4.1

Checklist:

[X] My code follows the style guidelines of this project
[X] My code meets the required code coverage for lines (90% and above)
[X] My code meets the required code coverage for branches (80% and above)
[X] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[X] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes

In file: doc/quick-start-guide-and-build still "licenses" name presents and should be changed to "licence_list"


use instanceof to determine if arg is an array

as per discussion

why force onto one single line? and use brackets and semicolon in this case
const last = arr
  => arr instanceof Array
   ? arr[arr.length - 1]
   : undefined

:rofl:

@dym-sh Because it's the style we have decided to use in general, so we're sticking to it for consistency reasons.


Is Fonix talk dll yours or an external library?

Title

This project uses code from SharpTalk. All credit goes to the original authors.

Thanks, must've missed that line


Daily pulls

Allow for 1 free pull daily because that's how gacha works.
Requires #9

Code should be done, needs unit testing.


Fix crash in FLEXRuntimeUtility

In AAPLAppDelegate section app crashes while parsing selector URLSession:downloadTask:didFinishDownloadingToURL:
in iOS 10 number of arguments returned by method_getNumberOfArguments() is more than in selectorComponents array which causes crash on line.
NSString *prettyComponent = [NSString stringWithFormat:@"%@:(%@) ", [selectorComponents objectAtIndex:argIndex - kFLEXNumberOfImplicitArgs], readableArgType];
In iOS 9 method_copyArgumentType() returns NULL which causes crash on line
NSString *readableArgType = [self readableTypeForEncoding:@(argType)];

I have changed the places of if statements in order to return right value for single argument methods.
I have also added a comment to explain why it should be done that way.
Adding unit test is a good idea, now I am busy, when I am free I'll add unit tests.

@ryanolsonk  I have added a unit test after my first commit. In the unit test I am checking if previous and new methods are returning the same value.

Thanks for doing this and adding the test! Seems like the type information for that one method on NSURLSessionDownloadDelegate is pretty messed up in iOS 10.


Fix screens.rpy

Currently not a primary issue, but would be great if we could figure out how to fix the grid over full or not full enough issue. Use the comments for suggestions.

Please fix stats, they are core to the enjoyment of the game

Just out of curiosity, how did you fix stats?

I kinda just deleted the whole nodule and reweote it again
Sent from my Samsung SM-N920G using FastHub


Enable compiling LISF on Cray/SLES 15.4/PrgEnv 8.3.3 using Intel 2022.2.1 compilers

Description
These changes address issues encountered when compiling LISF on narwhal after its upgrade to Cray/SLES 15.4/PrgEnv 8.3.3.

This also addresses Issue #911

Code compiles when using user_557ww.cfg.  However, the new Intel C compiler objects to code in RDHM and one of the VIC LSMs (missing function prototypes).  Since these are not used by USAF and will not be used on Narwhal, I will go ahead and approve the pull request.


Mod Jar Uploads Out of Game Version Order

The publish action is called in-game version order. The only cause is concurrency. One easy solution waits for a task to finish before starting another. I am not willing to do this, considering it is a minor issue that would break nothing but the order.


Generalize filterM (etc.?) to Applicative

Today I noticed that PureScript filterM is constrained to Monad, whereas Haskell filterM is constrained to Applicative. After looking at the source code for each and asking on FP Slack, it seems there is no particular reason filterM cannot be more general in PS.
#35 was set to change this and some other things (e.g. replicateM). However, that PR was closed due to staleness. The author @DavidHarrison was going to split it up into individual PRs, but hasn't gotten around to it yet. It seems worthwhile to double-check that PR to see if there is anything else from it you may want to merge.

I would like to do what we did in the arrays library: provide a new filterA, and continue to provide filterM as a alias for filterA, together with a deprecation warnings, until the next major release.

A reason not to do that, at least for lazy lists, is demonstrated by one of the tests:
assert $ filterM (const Nothing) (repeat 0) == Nothing

Since our apply is strict in both of its arguments, a filterA would not be able to terminate on this example, since there's no way to conditionally recurse based on the f Boolean value. If our apply were a little more scalazesque and had type forall a. f (a -> b) -> (Unit -> f a) -> f b, then we could do it, but that would be a pretty big change! (Besides, I'm sure there are reasons PS favors strict classes; it must have come up before with Monoid or HeytingAlgebra or something.)
We could still introduce filterA for strict lists, or even also for lazy lists with a caution that it forces the whole list. But deprecating filterM in favor of filterA doesn't make sense, probably even for strict lists; I don't think we'd want the strict and lazy interfaces to diverge in that way, right?

I think adding a filterA that is strict (and explains @rhendric's comment above) without deprecating filterM is how this issue should proceed.


Add Terminado example

This adds a (not-quite barebones) example of how to use Xterm.js with Terminado.  Note that terminado_attach.js is extremely similar to attach.js, perhaps there's some way we can provide arguments to attach.js so that the file is completely unnecessary.  I'm very much not a Javascript coder, as you can probably tell by my hacked-up index.html that begs, borrows and steals from things like fit.js to hopefully cut down on the amount of code needed to get you up and running on Python.
If you think this can be made simpler, (I definitely think it can) then feel free to just close this PR and remake your own based upon this, I just thought this might be useful for those that want to use a Python backend instead of a node.js one.

In general the content of this PR is quite nice and interesting, but I believe it is out of scope in specific parts.
While a terminado add-on would be useful, I do not think that a terminado demo should be inside this repository.
My proposal

[ ] Make terminado_attach an add-on. It could be called just terminado also, to make things easier.
[ ] Remove the demo completely
[ ] Add a Connecting/Integrating with Terminado guide at http://xtermjs.org/docs (repo: https://github.com/xtermjs/xtermjs.org)

I think this is the best idea, in order to preserve the scope and purpose of this repository, while helping people that want to use xterm.js with Terminado.

:+1: to moving the demo to a guide in http://xtermjs.org/docs/
:-1: to adding this as an addon, that means we need to maintain it until the end of time

:-1: to adding this as an addon, that means we need to maintain it until the end of time

I don't think that is what that means.  It just means that as long as it is useful, your users can actually find it. Feel free to deprecate it in the future if the state of the art moves on.

@scopatz since addons hook into private APIs by design this limits the ability to evolve the code base, since addons are essentially part of the API (or at least will be when they're documented).

I think it is OK to break addons as needed, and then people can submit patches to fix them.  They are explicitly not part of the core. Folks can always roll back to last working version, if needed.  My point is that without an addon, the user is forced to do this work. I am not a hugely experienced web or JS dev, and it was complex enough that after about 8 hours trying to get xtermjs work over the past couple of days, I gave up.  I would really appreciate anything that makes the user experience less painful.

I was looking at the code on my phone and I think I missed some, after another look I know what you mean. It doesn't actually touch any private APIs which is the main thing I was concerned about. So :+1: for the addon.

Great, if you think it's better to have it as a separate addon, I can definitely do that.  In the same spirit as the attach plugin, should I include a demo index.html page?

Do you plan to open a PR at https://github.com/xtermjs/xtermjs.org with a more complete documentation about how to connect to a terminado back-end?

Do you plan to open a PR at https://github.com/xtermjs/xtermjs.org with a more complete documentation about how to connect to a terminado back-end?

Yes

Moving on with some functional tests.

@staticfloat do you have any ready example that I can use to test this out?

Documentation here: https://github.com/xtermjs/xtermjs.org/pull/4

Done, and tested, this time with the console open to make sure there are no errors.

:+1: to having something like this to make things easy for people coming from the Python world. I'd like to add a link from the Terminado docs once this and the associated doc PR are done.

For that matter, if it makes life easier, we could add an option to terminado to send/receive data in the format supported by the existing attach addon. Looking over that code, though, I don't think it can tell the server when the terminal has been resized, so I still like the option to have a slightly richer protocol.

Everything seems to be working great. Thanks for implementing this!
@takluyver let's keep the scope of this PR to where it is right now and add extra functionality down the road in an as-needed base.

Thanks!
PRs welcome to add a link to terminado docs, or I'll get round to it sooner or later.

@staticfloat just letting you know that we mentioned you for this contribution in the 2.1 announcement. Thank you!


How do I use the downloaded model?

Question

...
question:
How do I use the downloaded model?
Code：
source = "https://arxiv.org/pdf/2408.09869"  # document per local path or URL
converter = DocumentConverter()
result = converter.convert(source)
downloaded models

What do you mean exactly? Models are used by the convert pipeline inside Docling.

@Zhengyu-Ju If you want to use/download the models directly, I would refer you to the Huggingface page

@PeterStaar-IBM  I don't think you understand the question, so you just reply seriously.

If you want to use a local path where you have prefetched your models, that's documented here:
https://ds4sd.github.io/docling/usage/#provide-specific-artifacts-path

question:
do we have a function to generate a floder which contains images floder and markdown file
Code:
doc_converter = DocumentConverter(
format_options={
InputFormat.PDF: PdfFormatOption(
pipeline_options=pipeline_options, backend=PyPdfiumDocumentBackend
)
}
)
result = doc_converter.convert(source, max_num_pages=2, max_file_size=20971520)
result.document.export_to_markdown()
note:
i means do we have the function like export_to_markdown()

@Zhengyu-Ju this looks like a new topic, do you mind opening a new feature request issue for this?


Add MDAO index on SOURCE_ID as implementation detail of ConfluenceSyncDAO

This yields substantial improvements in server-side junction DAO performance.

Codecov Report

Merging #129 into master will decrease coverage by 0.09%.
The diff coverage is 25%.

@@            Coverage Diff            @@
##           master     #129     +/-   ##
=========================================
- Coverage   60.57%   60.47%   -0.1%
=========================================
  Files          35       35
  Lines        1788     1794      +6
  Branches      136      137      +1
=========================================
+ Hits         1083     1085      +2
- Misses        705      709      +4

Impacted Files
Coverage Δ

lib/sync_dao.es6.js
11.53% <25%> (+2.84%)
:arrow_up:

Continue to review full report at Codecov.

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

@arobins PTAL at response. I agree is-an-instance is a bit hacky, but I want to clarify the alternative.


Moving FAQ Get Data: Upload tool used with FTP

This file contains steps on loading data using FTP

You have some broken links. For instance ftp-upload/  (More help...) does not exist in the GTN. Maybe the tutorial "Rule Based Uploader" can be helpful.

Hi @annefou I have removed all broken links. Kindly have a look at the updated file and give me your feedback.

Hi @beatrizserrano thanks a lot for the correction I made the steps precise and generic to the servers. Kindly have a look at the updates and give me your feedback.

@jennaj and @beatrizserrano i sujjest since we already mentioned this " Note that the FTP server name is specific to the Galaxy you are working on. This is by default the URL of the server." No need to mention this "For the public Galaxy Main instance at http://usegalaxy.org/ the FTP server name to use is usegalaxy.org.". What are your thoughts on this?

Hi @beatrizserrano please can you review the changes i made for this and give me your feedback. Thanks

This FAQ is also very similar to Upload many files (>10) via FTP.

yes they are similar

Hi @annefou please can you have a look at the changes i did based on @beatrizserrano corrections :). Thanks

Hi @bernandez, as @annefou pointed out, this is covered by another FAQ, so we will close this one. However, you can record your contribution in Outreachy, we will count it as done :)

Thanks @beatrizserrano :)


cache_range test performance improvement

cache_range_requests_cache_complete_responses.test.py had a for loop where it appended a single character to a string 4 million times. This is exceedingly slow in Python as it needed to reallocate, copy, and append with each iteration. This patched version reduces the time to a fraction of a second instead of multiple minutes (4 minutes on my system).

Marking for 9.2.x since this test performance improvement will be nice for that branch too.

[approve ci]

[approve ci autest]

[approve ci autest]

[approve ci autest]

[approve ci autest]

[approve ci autest]

Cherry-picked to v9.2.x


Tags format not supported by InfluxDB

I am using influxdb to consume the metrics but I couldn't find the tags I am sending using statix.
Then I did little digging and found that
InfluxDB understands tags sent in this format

<tag_key>=<tag_value>,<tag_key>=<tag_value>

But when I logged how statix is sending its below format.

"method:GET", "handler:crudcontroller_show", "service:carbon", "env:dev"

Can you please tell me, how can I configure my tags to be visible in influxDB.
How I am sending the metrics from code
MyApp.Statix.increment(status_count, 1, sample_rate: 1.0, tags: [tag1, tag2, tag3, tag4])

Incorrectly filed.

@sahilpaudel-pe thanks for the report, I'll investigate it.
I can also recommend checking https://github.com/lexmag/fluxter for native InfluxDB support.

@lexmag Thanks for getting back.
The thing is all other data is received by influxDB except for tags which is the concern for me right now. I am stuck in tags, if you can figure that out it world be great.
As for rest stuff statix is doing great.


[turborepo] jest cannot resolve the 'ui' directory in the with-create-react-app example

What version of Turborepo are you using?
latest
What package manager are you using / does the bug impact?
Yarn v1
What operating system are you using?
Mac
Describe the Bug
I made a fresh clone of https://github.com/vercel/turbo/tree/main/examples/with-create-react-app, ran yarn install, made a copy change within the web directory (in order for jest to recognize a change within that directory), and the test fails with:
FAIL src/App.test.tsx
web:test:   ● Test suite failed to run
web:test:
web:test:     Cannot find module 'ui' from 'src/App.tsx'
web:test:
web:test:     Require stack:
web:test:       src/App.tsx
web:test:       src/App.test.tsx
web:test:
web:test:       1 | import React from "react";
web:test:     > 2 | import { Link } from "ui";
web:test:         | ^
web:test:       3 | import "./App.css";
web:test:       4 |
web:test:       5 | function App(): JSX.Element {
web:test:
web:test:       at Resolver.resolveModule (../../node_modules/jest-resolve/build/resolver.js:324:11)
web:test:       at Object.<anonymous> (src/App.tsx:2:1)
web:test:       at Object.<anonymous> (src/App.test.tsx:3:1)

Expected Behavior
The test should pass and be able to properly resolve the ui directory.
To Reproduce
Run npx create-turbo@latest -e with-create-react-app, make a change within App.tsx (just a copy change), and see that the test fails.
Reproduction Repo
No response

This sounds like it might be because the types in the packages/ui/package.json are dependent on a dist folder. If you run a turbo build, does the issue still happen?

@kateEcobb This example wasn't getting the love that it needed since create-react-app itself is deprecated. As timing would have it, we deprecated our example over the weekend to free up some focus for the rest of our examples.
@Zertsov is correct in that this was/is a type-level issue. If you'd like to see how to run TypeScript types correctly through your workspace/packages, we've freshly updated all the other examples in our repo with the latest recommended settings from the TypeScript team.


[Feature Request] Dark theme support

django-json-widget version: 2.0.1
Django version: 5.0.2
Python version: 3.12
Operating System: -

Description
The editor is burning my eyes. I would love to see the editor in a dark theme whenever Django admin is on dark mode.

What I Did
I might adjust some CSS to temporarily save my eyes.

More about themes in ace:
https://ace.c9.io/#nav=howto:~:text=demo/autoresize.html-,Setting Themes,-Themes are loaded


fix example in README.md

minor fix

my bad. there was a mismatch between the Github Codebase and what is on npm. Pls disregard


packing & compression

Investigate opportunities and costs of lossy packing and lossless compression of data

XIOS & netCDF support lossless gzip compression.
however, this is only available in more recent netCDF + hdf stacks for parallel write, so some older platforms with old versions of netCDF will not be able to provide gzip compression and parallel writing at the same time
platforms using netcdf 4.8.x and above can support parallel compression

chunking of HDF data
https://portal.hdfgroup.org/documentation/hdf5-docs/advanced_topics/chunking_in_hdf5.html
is important to consider alongside compression, as compression algorithms will run on chunks of data, so how data is segregated within a file has significant influence  on the computational impact and storage reduction of files

there is a demonstration programme using scale_factor & add_offset to pack data using XIOS, to a defined precision
https://github.com/MetOffice/tcd-XIOS-demonstration/tree/main/xios_examples/packing_scale_offset
deciding on the precision control values is a significant effort, requiring computer science and atmospheric science input in order to obtain a computationally and scientifically valid packing
mistakes can lead to numerical error, which are problematic within libraries

https://github.com/MetOffice/tcd-XIOS-demonstration/pull/27
explores an issue with handling missing data, which is unresolved at this time

https://www.researchgate.net/publication/365006139_NetCDF_Compression_Improvements
https://docs.unidata.ucar.edu/netcdf-c/current/md__media_psf_Home_Desktop_netcdf_releases_v4_9_2_release_netcdf_c_docs_quantize.html
https://www.hdfgroup.org/wp-content/uploads/2021/10/HDF5-Users-Workshop-2021-Additional-Compression-Methods-for-NetCDF.pdf
NetCDF Quantization is an interesting approach to packing, with subtle  implementation differences compared to the more traditional type changing
work is required to explore how this facility could interact with XIOS


Enable Python 3.8 builds

Solution to issue cannot be found in the documentation.

[X] I checked the documentation.

Issue
See https://github.com/conda-forge/mujoco-feedstock/pull/56#issuecomment-2230408231
Installed packages
.

Environment info
.

conda-forge itself dropped Python 3.8 (see https://github.com/conda-forge/conda-forge-pinning-feedstock/pull/6344), so we can close this issue.


Will it be blocked by Twitter in the future?

I'm newbee of Twint but I love it so much.
Twint can crawl twitter messages without logging in. It's already pretty powerful. Now twitter's API adds a lot of restrictions. Obviously, twint's features are something twitter is not happy to see.
I'm worried that twint will be blocked by twitter. Technically, is it possible? Will it happen in the future?

Twint does not actually crawl Twitter since the webpage requires JS to be rendered.
Anyway, as you may guess, I don't have the answer to your question. We'll see what will happen


Fix cap java jar loading

cds root is not always correctly set based on the option passed in.
Reload the cds env based on the CAP root.
help with loading cds dependencies from jar files for example.

Thanks @Klaus-Keller @heimwege


Animate all play/pause icons

This PR adds an AnimatedPlayPause widget that encapsulates the animation. It implicitly animates the icon when it is rebuilt with a different playing argument.
Both MaterialControls and CupertinoControls can now simply use this widget without needing an animationConttroller on their own.
Screen recording:
https://user-images.githubusercontent.com/44171190/109505643-17a3d880-7a9d-11eb-9227-2dee7cc0862a.mp4

Love the modularity.
Do you think it's alright that the icon animates now for iOS?
I just noticed, Only on flutter web, the play button seems uncentered somehow for material and cupertino. But this has been the case before this PR, so I'll merge this.

Let me investigate what's going on on the web. Also, looks like I forgot to dispose the animation controller...

Ok that took me a while... The bug on the web is caused by a different visualDensity on that platform (would have also affected desktop platforms). Finally found the solution here: https://github.com/flutter/flutter/issues/52980#issuecomment-709662594. I notices that adding the fix required doing it twice for material and cupertino, so I refactored that part out into a CenterPlayButton widget that can be shared.

Do you think it's alright that the icon animates now for iOS?

From a UI perspective? I don't know. I mean, I don't even own an iPhone so I have no idea what it is supposed to look like on that platform or if it will look out of place.

Oh wow, how annoying. Thanks a lot for getting to the bottom of this.

Sweet i was looking at the code to see why the icon was jacked on flutter web. Thanks for fixing this


Increases power usage across the station

About the PR
Gives engineering a reason to setup a generator.
See file changes but generally increased computer and machine usage across the board. Made the gravity generator particularly hungry too (it has to look after a whole station afterall. When we make compact ones they will need to have a different component for radius based gravity and lower power usage).
Consolidates some of the changes from #5870 to make them less harsh in some cases.
Case study: Saltern
3 minutes: 2x SMES at 30%, all substations now charged
5:30 minutes: both SMES fully drained, very short (sub 5 seconds) brownout.
medbay @ 9:50
engineering @ 10:15 then brownouts till 10:30
bar kitchen @ 11:10
dorms disposals @ 11:45 + brownouts till 11:55
brig and arrivals still between 30 and 40%
cargo 30%
library 70%
bridge 15%
Packed fares better as it has 3x SMES and more substations as well as 3x SMES which aren't connected to grid that can easily be done to buy extra time.
This gives engineering a clear objective at round start to get a minimum amount of power generation going. And although the departments depowering is based on their machines they have primiarly, it make sense for security (+vault to go last).
If you want to delay against this simply map an extra SMES or two to increase the duration till bad stuff happens but currently Saltern spawns with two debug generators anyway which reduce the power draw by 6000 watts, prolonging the time to set up.
Screenshots

Altered packedstation atmos slightly to have it's own substation and also changed the HV wiring so that the engineering substations are now on the supply side of the SMES and not the input side.

Changelog

:cl:

tweak: The power bill has finally arrived. Computers and Machines use considerably more power now. Engineering better be fast at getting some generation going!

Will do some further tests on this with solars and engine setup to ensure balance is okay (as might bring grav gen down some)

Will do some further tests on this with solars and engine setup to ensure balance is okay (as might bring grav gen down some)

did you end up doing this?

Should be good to merge now. Tested on Saltern and Packed with no major issues. Should make brownouts more common and engineering have about 10 minutes before people start shouting.
Decided to also port from Moonys PR so that AME no longer requires power to start so station is recoverable.

Should make brownouts more common and engineering have about 10 minutes before people start shouting.

Is it 10 minutes before blackout or 10 minutes before power starts going down?

Should make brownouts more common and engineering have about 10 minutes before people start shouting.

Is it 10 minutes before blackout or 10 minutes before power starts going down?

For Saltern when I last tested it was 10 minutes until a department actually lost power (med and eng). There isn't really a way to balance power draw vs power production otherwise, the only way to keep it semi realistic and useful would be to add more SMES which would give about 2.5 minutes extra each until substations start to drain. That and better power network designs so subs aren't overloaded.
Saltern only really has 2 SMES so doubling that would make it 10 minutes to a brownout then 20 mins to blackouts. Packed has lots of spare SMES that just need connected.

Can't you just VV the starting charge to be higher for SMES?
10 minutes until brownout seems okay, I would say at least 15-20 before a total blackout.

Can't you just VV the starting charge to be higher for SMES?
10 minutes until brownout seems okay, I would say at least 15-20 before a total blackout.

Just tested, Saltern spawns with the SMES full. With a third SMES mapped in you get about 7-8 minutes until the brownout so with that about 15 minutes until substations start dying. Both stations spawn with an AME so I think this is fair for now - 15 minutes is more than enough time to setup an AME or any other engine and start building more storage. I'll push the 3rd SMES PR for saltern.


[5.2.31] [Illuminate\Contracts\Container\BindingResolutionException] issue

Just ran composer update
result was
Updating dependencies (including require-dev)

Removing laravel/framework (v5.2.30)

Installing laravel/framework (v5.2.31)
Downloading: 100%

Removing intervention/image (2.3.6)

Installing intervention/image (2.3.7)
Downloading: 100%

Now getting

[Illuminate\Contracts\Container\BindingResolutionException]
Unresolvable dependency resolving [Parameter #0 [  $app ]] in class Illuminate\Support\Manager

on even artisan list

Sounds like you're trying to inject a manager class. You won't be able to do that.


fix Fix continuous_agg_count PODArray wrong boundary judgment

Proposed changes
Issue Number: close #xxx
Problem summary
Assuming that the size of RowSourcesBuffer::_buffer is n, when n is fetched, it just crosses the boundary. If the memory allocated by mmap will return 0 at this time, if it is not mmap, it will return a random value. next.agg_flag() in continuous_agg_count is According to whether the return value is 0 or not as a condition.
doris_be_test: /mnt/disk2/liyifan/doris/core/be/src/vec/common/pod_array.h:323: T &doris::vectorized::PODArray<unsigned short, 4096, Allocator<false, false>, 15, 16>::operator[](ssize_t) [T = unsigned short, initial_bytes = 4096, TAllocator = Allocator<false, false>, pad_right_ = 15, pad_left_ = 16]: Assertion `(n >= (static_cast<ssize_t>(pad_left_) ? -1 : 0)) && (n <= static_cast<ssize_t>(this->size()))' failed.

Checklist(Required)

[ ] Does it affect the original behavior
[ ] Has unit tests been added
[ ] Has document been added or modified
[ ] Does it need to update dependencies
[ ] Is this PR support rollback (If NO, please explain WHY)

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

cc @yixiutt

TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 34.91 seconds
stream load tsv:          429 seconds loaded<PHONE_NUMBER>9 Bytes, about 166 MB/s
stream load json:         21 seconds loaded<PHONE_NUMBER> Bytes, about 107 MB/s
stream load orc:          60 seconds loaded<PHONE_NUMBER> Bytes, about 17 MB/s
stream load parquet:          31 seconds loaded 861443392 Bytes, about 26 MB/s
https://doris-community-test-1308700295.cos.ap-hongkong.myqcloud.com/tmp/20230428145024_clickbench_pr_137210.html

LGTM


Issue #112 change scope of Message Object headers to application headers only

Summary of changes:

Changed the headers property from a Schema Object to a Map of Schema or Reference Objects
Changed wording on headers property to constrain it to applications headers i.e. it does not include protocol headers, which will be defined by #42
Updated examples of headers throughout
Changed JSON Schema and used patternProperties to define the map and constraints on types.
Provided an example for testing.

Changes based on feedback submitted


GTD plugin

Learn how to implement dropdown UI that edits field from https://github.com/roma0104/gsd5/tree/master/plugins/gsd5/core/ui
And learn how to use calendar widget to edit "due" field from https://github.com/NicolasPetton/Projectify/tree/master/plugins/nico/projectify/tiddlers

Calendar widget can be used in the calendar import plugin. And should be a separate plugin.


Fix computed single scalar globals

They were being cached, but we were rejecting attempts to use the
cache because of an overly zealous assert. Make the assert more
permissive.
Fixes #6988.

To backport this, I had to pull-in a few snippets from #6613


Codecov settings don't take the main branch into account

What happened: According to Codecov, a master branch exists, but a main branch doesn't.
This could probably be configured via updating the Codecov settings (requires maintainer access).
What you expected to happen: Codecov settings to take the main branch into account.
Minimal Complete Verifiable Example: N/A
Anything else we need to know?: N/A
Environment: N/A

Thanks @hristog -- I've updated the default branch on codecov to be main


Request to support box.com

I would like youtube-dl to be able to download from box.com, please.
Example: https://app.box.com/s/yrvxrv91fenex2acvo0zdpbogjjb9jrf

Bump....
New example:
https://mlssoccer.app.box.com/s/0evd2o3e08l60lr4ygukepvnkord1o1x/file/510727257538?sb=/details

I third this

Fourth'd! Is there a way we can get started on this? Can someone post some guidance?
I tried grabbing the DASH mpd manifest URL (e.g. from the Networks tab in Google Chrome) and trying to download with that and with cookies, but I still get a 401 Unauthorized error like this, and youtube-dl eventually gives up. Has anyone ever seen this before?
...
[dashsegments] Total fragments: 623
[download] Destination: test.mp4
[download] Got server HTTP error: HTTP Error 401: Unauthorized. Retrying fragment 1 (attempt 1 of 10)...
[download] Got server HTTP error: HTTP Error 401: Unauthorized. Retrying fragment 1 (attempt 2 of 10)...
[download] Got server HTTP error: HTTP Error 401: Unauthorized. Retrying fragment 1 (attempt 3 of 10)...
[download] Got server HTTP error: HTTP Error 401: Unauthorized. Retrying fragment 1 (attempt 4 of 10)...
[download] Got server HTTP error: HTTP Error 401: Unauthorized. Retrying fragment 1 (attempt 5 of 10)...
[download] Got server HTTP error: HTTP Error 401: Unauthorized. Retrying fragment 1 (attempt 6 of 10)...
[download] Got server HTTP error: HTTP Error 401: Unauthorized. Retrying fragment 1 (attempt 7 of 10)...
[download] Got server HTTP error: HTTP Error 401: Unauthorized. Retrying fragment 1 (attempt 8 of 10)...
[download] Got server HTTP error: HTTP Error 401: Unauthorized. Retrying fragment 1 (attempt 9 of 10)...
[download] Got server HTTP error: HTTP Error 401: Unauthorized. Retrying fragment 1 (attempt 10 of 10)...
ERROR: giving up after 10 fragment retries

@remitamine  @ThirumalaiK I saw this TODO to add the query to each fragment commented out in https://github.com/ytdl-org/youtube-dl/commit/c84f9475b8df0f892b631966159e1649dafe13f0#diff-d6894be6fa794f1c1ff868400eaac26e2fbf780e7b175ad9815d47445fa8c67cR72
Isn't that what's missing?

I'm hoping if someone could help me see if they are able to download from https://sunsetlearning.app.box.com/s/p7y447it93pra1m88z4ut46wial4tnx2/file/886494523649
All of my attempts have failed. Maybe I'm doing it wrong.

This PR works. Just replace the box.py file in the extractors folder.
https://github.com/ytdl-org/youtube-dl/pull/27874

`#!/bin/bash
number of .m4s files in stream; get from Networks tab of video player after player finishes
N=665
URL with init.m4s for video from Networks tab; example below
URL='https://public.boxcloud.com/api/2.0/internal_files/>FILENAME>/versions/1373998591020/representations/dash/content/video/1080/init.m4s?access_token=&box_client_name=box-content-preview&box_client_version=2.95.0'
'https://dl.boxcloud.com/api/2.0/internal_files//versions//representations/dash/content/video/1080/init.m4s?access_token=&shared_link=&box_client_name=box-content-preview&box_client_version='
URLSPLIT=(${URL//video/1080/init/ })
Create audio and video directories if they don't exist
mkdir -p audio
mkdir -p video
Rimuovi i file esistenti dalla cartella "audio"
rm -f audio/*
Rimuovi i file esistenti dalla cartella "video"
rm -f video/*
download initial video m4s
youtube-dl "${URLSPLIT[0]}video/1080/init${URLSPLIT[1]}" -o "video/vinit.m4s"
download initial audio m4s
youtube-dl "${URLSPLIT[0]}audio/0/init${URLSPLIT[1]}" -o "audio/ainit.m4s"
download all other m4s
for ((MNUM=1;MNUM<=N;MNUM++));
do
youtube-dl "${URLSPLIT[0]}video/1080/$MNUM${URLSPLIT[1]}" -o "video/v$MNUM.m4s"
youtube-dl "${URLSPLIT[0]}audio/0/$MNUM${URLSPLIT[1]}" -o "audio/a$MNUM.m4s"
done
concatenate m4s into one
cat video/vinit.m4s >> vall.m4s
cat audio/ainit.m4s >> aall.m4s
for ((MNUM=1;MNUM<=N;MNUM++)); do cat "v$MNUM.m4s" >> vall.m4s; done
for ((MNUM=1;MNUM<=N;MNUM++)); do cat "a$MNUM.m4s" >> aall.m4s; done
turn m4s into complete.mp4
ffmpeg -i aall.m4s -i vall.m4s -c:v copy -c:a aac complete.mp4
`
All audio and video m4s files are downloaded but then i get and wrong mp4 file impossible to read. This is the console output:
Input #0, mov,mp4,m4a,3gp,3g2,mj2, from 'aall.m4s':   Metadata:     major_brand     : iso5     minor_version   : 512     compatible_brands: iso5iso6mp41     encoder         : Lavf60.3.100   Duration: N/A, bitrate: N/A   Stream #0:0[0x1](und): Audio: aac (mp4a / 0x6134706D), 44100 Hz, stereo, fltp, 128 kb/s (default)     Metadata:       handler_name    : SoundHandler       vendor_id       : [0][0][0][0] [mov,mp4,m4a,3gp,3g2,mj2 @ 0x7fe386c167c0] Could not find codec parameters for stream 0 (Video: h264 (avc1 / 0x31637661), none, 1280x720): unspecified pixel format Consider increasing the value for the 'analyzeduration' (0) and 'probesize' (5000000) options Input #1, mov,mp4,m4a,3gp,3g2,mj2, from 'vall.m4s':   Metadata:     major_brand     : iso5     minor_version   : 512     compatible_brands: iso5iso6mp41     encoder         : Lavf60.3.100   Duration: N/A, bitrate: N/A   Stream #1:0[0x1](und): Video: h264 (avc1 / 0x31637661), none, 1280x720, SAR 1:1 DAR 16:9, 6144 tbr, 12288 tbn (default)     Metadata:       handler_name    : VideoHandler       vendor_id       : [0][0][0][0] File 'complete.mp4' already exists. Overwrite? [y/N] y Stream mapping:   Stream #1:0 -> #0:0 (copy)   Stream #0:0 -> #0:1 (aac (native) -> aac (native)) Press [q] to stop, [?] for help Output #0, mp4, to 'complete.mp4':   Metadata:     major_brand     : iso5     minor_version   : 512     compatible_brands: iso5iso6mp41     encoder         : Lavf60.3.100   Stream #0:0(und): Video: h264 (avc1 / 0x31637661), none, 1280x720 [SAR 1:1 DAR 16:9], q=2-31, 6144 tbr, 12288 tbn (default)     Metadata:       handler_name    : VideoHandler       vendor_id       : [0][0][0][0]   Stream #0:1(und): Audio: aac (LC) (mp4a / 0x6134706D), 44100 Hz, stereo, fltp, 128 kb/s (default)     Metadata:       handler_name    : SoundHandler       vendor_id       : [0][0][0][0]       encoder         : Lavc60.3.100 aac frame=    0 fps=0.0 q=-1.0 size=       0kB time=-577014:32:22.77 bitrate=  -frame=    0 fps=0.0 q=-1.0 Lsize=       0kB time=-577014:32:22.77 bitrate=N/A speed=N/A     video:0kB audio:0kB subtitle:0kB other streams:0kB global headers:0kB muxing overhead: unknown [aac @ 0x7fe386c19340] Qavg: nan

finally this script give me the correct results:
#!/bin/bash

# number of .m4s files in stream; get from Networks tab of video player after player finishes
N=665

# URL with init.m4s for video from Networks tab; example below
URL='https://public.boxcloud.com/api/2.0/internal_files/XXXXXXXX/versions/XXXXXXXXX/representations/dash/content/video/1080/init.m4s?access_token=XXXXXXXXXXXXX&box_client_name=box-content-preview&box_client_version=2.95.0'

URLSPLIT=(${URL//video\/1080\/init/ })

# Create audio and video directories if they don't exist
mkdir -p audio
mkdir -p video

# Rimuovi i file esistenti dalla cartella "audio"
rm -f audio/*

# Rimuovi i file esistenti dalla cartella "video"
rm -f video/*

# download initial video m4s
youtube-dl "${URLSPLIT[0]}video/1080/init${URLSPLIT[1]}" -o "video/vinit.m4s" --user-agent "Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36" --referer "https://app.box.com/"
# download initial audio m4s
youtube-dl "${URLSPLIT[0]}audio/0/init${URLSPLIT[1]}" -o "audio/ainit.m4s"

# download all other m4s
for ((MNUM=1;MNUM<=N;MNUM++));
do
	youtube-dl "${URLSPLIT[0]}video/1080/$MNUM${URLSPLIT[1]}" -o "video/v$MNUM.m4s"
	youtube-dl "${URLSPLIT[0]}audio/0/$MNUM${URLSPLIT[1]}" -o "audio/a$MNUM.m4s"
done

# concatenate m4s into one
cat video/vinit.m4s >> vall.m4s
cat audio/ainit.m4s >> aall.m4s
for ((MNUM=1;MNUM<=N;MNUM++)); do cat "video/v$MNUM.m4s" >> vall.m4s; done
for ((MNUM=1;MNUM<=N;MNUM++)); do cat "audio/a$MNUM.m4s" >> aall.m4s; done

# turn m4s into complete.mp4
ffmpeg -i aall.m4s -i vall.m4s -c:v copy -c:a aac complete.mp4

The solution was already posted in https://github.com/ytdl-org/youtube-dl/issues/5949#issuecomment-1387121658: test the PR.


Update React 0.14.2 + other dependencies

Updates

React 0.14.2
Replace grunt-6to5 with grunt-babel
Replace deprecated grunt-react with grunt-babel
grunt-webpack
Replace deprecated jsx-loader with babel-loader

Updating tests. Will submit another PR in a bit.

Awesome! This looks like great work.
In regard to the error, I will try to take a look tonight and see if we can't get it straightened out :)

@ryanlanciaux Super! thank you.

Hmm. I'm getting some different errors locally (not related to your changes). Trying to figure out what's going on here -- I will keep you posted.

Getting following error:
FAIL  __tests__/gridFilter-test.js (0.578s)
● GridFilter › it calls change filter when clicked
  - TypeError: Can't add property changeFilter, object is not extensible

But I can pass the test on gridFilter-test.js by using
filter.props = {
  changeFilter: mock
};

instead of
filter.props.changeFilter = mock;

But the original props will be lost. Not sure how to properly use jest doing some reading right now.
Let me know if I can help in any way. So close :+1:

Grunt build works as expected.
Main problem here are the tests. Made some more updates + fixed tests (jest).
Using node v4.2.2 and npm v2.14.7.
So far - 15 tests failed, 58 tests passed.
Still getting the following error because of invalid html (according to react v0.14).
E.g.  is missing .
return
    	<thead>
            <tr>
                {node}
            </tr>
    </thead>;

How can we just jest table headers? Any ideas @ryanlanciaux?
Error: Invariant Violation: findComponentRoot(..., .0.0.$one): Unable to find element. This probably means the DOM was unexpectedly mutated (e.g., by the browser), usually due to forgetting a <tbody> when using tables, nesting tags like <form>, <p>, or <a>, or using non-SVG elements in an <svg> parent. Try inspecting the child nodes of the element with React ID ``.

All checks have passed :+1:  @ryanlanciaux

@sajinshrestha @ryanlanciaux I'll test in the next day.

Nice work @sajinshrestha! Thanks a ton! :+1: :star: :star2:

I'll second that, thanks @sajinshrestha :)

@ryanlanciaux @jgwconsulting Glad to help out. There are still two existing issues with the documentation - https://github.com/sajinshrestha/Griddle/issues


keep column selection

When you select only certain column to be displayed, keep the user selection in memory!! It has to be customized depending on user.

Show some example of what you're talking about.

If you select the columns to display, then you click on a record to edit it, when you go back to the change_list it loses the column selection.  Is is a way to keep it? and can it bee USER PERSONALIZED in order that it's kept depending on the user?


Running multiple test configurations gives wrong results

Hello,
I've encountered the following issue on my project and managed to reproduce it on the hello world test CLI sample project in this fork.
To reproduce, I changed the hello world project to define 2 test suites intended to run in two different workspaces.
After doing that, running each test alone from the UI works, i.e. the tests pass. But using the Run All Tests command or UI button makes the tests fail.
Since there are 2 configurations, we find 2 test runs in the UI, both named Unnamed Task. Here are screenshots of the outputs of each run.

After some investigation and debugging, here's what I found.

The extension is not selecting the subset of files belonging to a configuration when running it. We can see that in the command line of each test run where both test suite files were passed using --run even though we are running under one configuration with --label N.

 >  vscode-test
  --label 0
  --reporter<EMAIL_ADDRESS>  --run /home/richa/ancr/src/vscode-extension-samples/helloworld-test-cli-sample/out/test/suite2/extension.test.js
  --run /home/richa/ancr/src/vscode-extension-samples/helloworld-test-cli-sample/out/test/suite1/extension.test.js
...

 >  vscode-test
  --label 1
  --reporter<EMAIL_ADDRESS>  --run /home/richa/ancr/src/vscode-extension-samples/helloworld-test-cli-sample/out/test/suite2/extension.test.js
  --run /home/richa/ancr/src/vscode-extension-samples/helloworld-test-cli-sample/out/test/suite1/extension.test.js

This makes all tests run in both configurations when the intention is that each config should run only the files matching its files patterns.

VS Code is triggering the execution of both configurations in parallel. That's why the second config fails with the following output:

 >  vscode-test --label 1 --reporter<EMAIL_ADDRESS>--run /home/richa/ancr/src/vscode-extension-samples/helloworld-test-cli-sample/out/test/suite2/extension.test.js --run /home/richa/ancr/src/vscode-extension-samples/helloworld-test-cli-sample/out/test/suite1/extension.test.js
Found existing install in /home/richa/ancr/src/vscode-extension-samples/helloworld-test-cli-sample/.vscode-test/vscode-linux-x64-1.87.2. Skipping download
[0318/171559.102180:ERROR:file_io_posix.cc(152)] open /home/richa/ancr/src/vscode-extension-samples/helloworld-test-cli-sample/.vscode-test/user-data/Crashpad/pending/d9d1c5f0-571e-4dfa-ae15-5684858a9060.lock: File exists (17)
[main 2024-03-18T17:15:59.342Z] Running extension tests from the command line is currently only supported if no other instance of Code is running.
[main 2024-03-18T17:15:59.344Z] Error: Running extension tests from the command line is currently only supported if no other instance of Code is running.
    at Ee.e (/home/richa/ancr/src/vscode-extension-samples/helloworld-test-cli-sample/.vscode-test/vscode-linux-x64-1.87.2/resources/app/out/vs/code/electron-main/main.js:110:4461)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
    at async /home/richa/ancr/src/vscode-extension-samples/helloworld-test-cli-sample/.vscode-test/vscode-linux-x64-1.87.2/resources/app/out/vs/code/electron-main/main.js:110:1230
    at async Ee.a (/home/richa/ancr/src/vscode-extension-samples/helloworld-test-cli-sample/.vscode-test/vscode-linux-x64-1.87.2/resources/app/out/vs/code/electron-main/main.js:110:1117)
Exit code:   1
Failed
Error: Test process exited with code 1

IMO (1) should be fixed in the extension. The launch of a config should use --run only for files included in that config. I will submit a tentative fix for review.
As for (2), I figured out that it stems from the fact that both concurrent launches are using the same user-data directory. I was able to work around it by using different user-data dirs as follows:
export default defineConfig([
	{
		files: 'out/test/suite1/**/*.test.js',
		workspaceFolder: 'ws-suite1',
		launchArgs: ['--user-data-dir', 'userdata1'],
	},
	{
		files: 'out/test/suite2/**/*.test.js',
		workspaceFolder: 'ws-suite2',
		launchArgs: ['--user-data-dir', 'userdata2'],
	},
]);

This is likely going to be an issue for any project with multiple test configurations. Should the extension do something internally to use different --user-data-dir automatically in the presence of multiple configurations?
The updated version is also available at the fork.

I can confirm that this now works over my original use case. Thanks!


add check for dead code

🚀 Feature
check if PR contains unused code or if the particular change dead code in not introduced somewhere else in the codebase
Motivation
remove all unused code, keep the codebase healthy
Alternatives
one toll I found - https://github.com/jendrikseipp/vulture
but let's check some alternatives
Additional context

Do we have to write workflow for this? In case if we want to use the mentioned Library Vulture then we will write a workflow CI that will test reach PR?

Do we have to write workflow for this? In case if we want to use the mentioned Library Vulture then we will write a workflow CI that will test reach PR?

@aniketmaurya yes we would add it here https://github.com/PyTorchLightning/pytorch-lightning/blob/master/.github/workflows/code-formatting.yml

@aniketmaurya yes we would add it here https://github.com/PyTorchLightning/pytorch-lightning/blob/master/.github/workflows/code-formatting.yml

@aniketmaurya yes we would add it here https://github.com/PyTorchLightning/pytorch-lightning/blob/master/.github/workflows/code-formatting.yml

Thank you for the clarification @Borda . I've raised a PR with Vulture integrated to workflows. Please let me know if I need to correct something.

@aniketmaurya yes we would add it here https://github.com/PyTorchLightning/pytorch-lightning/blob/master/.github/workflows/code-formatting.yml

Thank you for the clarification @Borda . I've raised a PR with Vulture integrated to workflows. Please let me know if I need to correct something.

cool, to be clear my main concern when I tried the tool yesterday was how to. deal with methods arguments which are tthere because of inheritance but in cold they are not used anymore...

cool, to be clear my main concern when I tried the tool yesterday was how to. deal with methods arguments which are tthere because of inheritance but in cold they are not used anymore...


Changelog for v1.6 ?

Any change warrant an update to the changelog for 1.6?

Huh, I must have forgotten to update it. #230 and #240

Turns out I never saved the editor... Added to the 1.6.1 changelog


Zig => WASM

May overlap / supersede https://github.com/fengb/fundude/issues/8
Zig has some recent progress with WASM tooling. Maybe it's easier to replace the Emscripten shell?
Edit: Zig is still missing libc for wasm. The C compiler in Zig is actually a bit easier to work with, so maybe it's possible to write some Zig shims. Or just port libc to Zig wasm.

Fixed via 5e5dc0a1b5687c993f78fbff5fe1b6fbf5f92f0a


[BUG] the ui is taking more width then the screen

Describe the bug
The current UI is taking more width than the screen making it overflow horizontally
Screenshots

Additional context
please assign this issue to me under GSSOC and also add the necessary labels

@AMISH-KHAN I've assigned you this issue, Best of Luck!

@singodiyashubham87 can you please assign this issue to me.

@singodiyashubham87 I have made the changes can you tell me on which branch should i push ?

@AMISH-KHAN You should create a new branch for your issue and then merge it to the main branch.

@singodiyashubham87 I have made the PR #96. Please review it.

hey @singodiyashubham87 as the been merged can close this issue?


Assembly output include file doesn't honor output directory

https://github.com/mateoconlechuga/convimg/blob/ccc6269c9eb243ee446d3e092da68f057f654e14/src/output-c.c#L405-L407
vs
https://github.com/mateoconlechuga/convimg/blob/ccc6269c9eb243ee446d3e092da68f057f654e14/src/output-asm.c#L195-L197

Um I feel like this could be fixed with a PR faster than opening an issue ;)

Definitely :D

Fixed in e450d04fd725edc14dffa1db785fe571d139f52a


Added support for joins with USING and updated the documentation

Fixes #36
Added support for USING joins with tests. Rewrote the join documentation to omit low level details and to cover both join types.

Coverage decreased (-0.03%) to 98.293% when pulling 276e676f805f7c4164feeafa5aa9c2193e7ba8a3 on using-and-join-strings into fe6bf777316d7044a484d39206c30afa0ea8ec53 on master.

Coverage decreased (-0.03%) to 98.293% when pulling 276e676f805f7c4164feeafa5aa9c2193e7ba8a3 on using-and-join-strings into fe6bf777316d7044a484d39206c30afa0ea8ec53 on master.

@cjstudioz I would prefer to make the changes to the kwargs in a separate PR.  Not sure which format you're referring to, I can't see any that could just use the class dict as params for the format function.

Cool. I only mentioned it as a minor thing as a suggestion. As for
formatting self.dict it was a more general comment since I've seen the
option to do that in many places in the code , maybe not so obvious with
only the changes in this review (sorry)
On 25 Apr 2017 9:47 am, "Timothy Heys"<EMAIL_ADDRESS>wrote:

@cjstudioz https://github.com/cjstudioz I would prefer to make the
changes to the kwargs in a separate PR. Not sure which format you're
referring to, I can't see any that could just use the class dict as params
for the format function.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/kayak/pypika/pull/38#issuecomment-296962538, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ANR4xLdHhdplgr6U05WjSjlQfKwmqlJ5ks5rzbMIgaJpZM4NEmVK
.

No worries, some of the format code is admittedly a bit complex. I'd welcome any suggestions on how to clean it up.


[BUG] Can't filter for CONTAINS a numerical string

The deno-slack versions
    "deno-slack-sdk/"<EMAIL_ADDRESS>    "deno-slack-api/"<EMAIL_ADDRESS>
Deno runtime version
deno 1.46.3 (stable, release, x86_64-pc-windows-msvc)
v8 <IP_ADDRESS>-rusty
typescript 5.5.2

OS info
Microsoft Windows [Version 10.0.19045.5247]

Describe the bug
When creating an Event trigger, if I am filtering for messages that contain a digit string, some part of Slack interprets it as filtering for containing a number and it gives an error.
Steps to reproduce

Suppose I am trying to filter for messages that contain a certain number, say 573. Pretend this is for a leaderboard; I want to see who sends the most messages containing the number 573.
triggers/leaderboard_update.ts:

import { Trigger } from "deno-slack-sdk/types.ts";
import LeaderboardUpdateDefinition from "../functions/leaderboard_update/definition.ts";
import { TriggerContextData, TriggerEventTypes, TriggerTypes } from "deno-slack-api/mod.ts";

const LeaderboardUpdateTrigger: Trigger<
  typeof LeaderboardUpdateDefinition.definition
> = {
  type: TriggerTypes.Event,
  name: "leaderboard update",
  description: "Update the leaderboard for 573 messages",
  workflow: "#/workflows/leaderboard_update_workflow",
  event: {
    event_type: TriggerEventTypes.MessagePosted,
    channel_ids: ["C046CFN4AJ0"],
    filter: {
      version: 1,
      root: {
        operator: "OR",
        inputs: [
          { statement: "{{data.text}} CONTAINS '573'"},
          { statement: "{{data.text}} CONTAINS '5 73'"},
        ]
      }
    }
  },
  inputs: {
    user_id: {
      value: TriggerContextData.Event.MessagePosted.user_id,
    },
    timestamp: {
      value: TriggerContextData.Event.MessagePosted.message_ts,
    },
  }
};

export default LeaderboardUpdateTrigger;

(I have the option with a space there to demonstrate that if it is not a purely digit string, there is no error).
3. Send any message in that channel.
Expected result
The workflow should trigger if the message contains 573, and not trigger otherwise.
Actual result
From slack activity --tail:
2024-12-16 21:22:35 [error] [Wf085AP4AB3P] (Trace=Tr0862EGC5B2) Trigger for workflow 'Get 573 leaderboard' failed: Hermes\Types\IntegerValue is not supported by operator CONTAINS

Note that if I replace the CONTAINS operand '573' with, say, 'hello', there are no errors and it triggers fine.
Requirements
Please read the Contributing guidelines and Code of Conduct before creating this issue or pull request. By submitting, you are agreeing to those rules.

Hey @pigrammer3! 👋 🥧  Thanks for sending this in with setup steps! 👾
I can confirm that numbers aren't being cast to strings in the CONTAINS statement, causing the same error you're finding. This operator is documented as supporting just strings, but it makes sense to support numbers here.
Various other attempts at escaping the number brought no luck for me. Both the SDK and CLI are also passing the provided trigger details to the API as is - shown in --verbose outputs - so I'm marking this as a server-side issue and will share this with the backend team.
Unfortunately I'm unaware of a workaround for checking if numbers are contained for the event trigger, but similar logic can be implemented in a custom function in the meantime. I hope this is helpful for now, but I hope to share updates about some changes to this filtering logic soon 🙏 ✨


subsec_{milli,micro,nano}seconds may have over-inclusive documented ranges

Documentation of these functions promises that they will produce values in the ranges -1_000..1_000, -1_000_000..1_000_000, and -1_000_000_000..1_000_000_000, respectively, but I don't see how they could produce the lowest value in their stated bounds.
In terms of code, I'll use subsec_nanoseconds as an example:
https://github.com/time-rs/time/blob/7a6a800843e999807ddc4a317e95705f4ed82da5/time/src/duration.rs#L912-L923
self.nanoseconds is a Nanoseconds, which is a RangedI32 - in particular, a RangedI32<{ -(Nanosecond::per(Second) as i32 - 1) }, { Nanosecond::per(Second) as i32 - 1 }>. per seems to be defined appropriately, so this gives us RangedI32<-999_999_999, 999_999_999>. Given that a RangedI32<MIN, MAX> promises to contain "[a]n i32 that is known to be in the range MIN..=MAX", it's impossible for -1_000_000_000 to inhabit Nanoseconds, and so to be produced by subsec_nanoseconds.
I believe the same logic can be extended to subsec_microseconds and subsec_milliseconds, since they defer to subsec_nanoseconds.

Yeah, this is a mistake. It should be changed to -999_999_999..=999_999_999, using the same number for both bounds for symmetry. Likewise for other similar values.


The theme doesn't change

In the avalonia web app, the theme does not change to light when it is set in the styles in app.axml
Webtest.zip

Without Material.Avalonia styles everything remains the same.
Seems like this is an css issue, you need to look for Background and remove it from ccs files


Add "until" field to PoetWaitRound, PoetWaitProof, AtxPublished events

Motivation
Many people currently use the gRPC AdminService's EventStream for simple go-spacemesh monitoring. This PR adds an "until" field to the PoetWaitRound, PoetWaitProof and AtxPublished events to make it a bit easier to see until when the node will wait after those events (related recent discussion in Discord).
Example events after this change:
  "poetWaitRound": {
    "current": 12,
    "publish": 13,
    "wait": "29238.819235612s",
    "until": "2024-01-09T07:00:21.151608967Z"
  }

  "poetWaitProof": {
    "publish": 13,
    "target": 14,
    "wait": "987972.244983544s",
    "until": "2024-01-20T20:00:00.000003843Z"
  }

  "atxPublished": {
    "current": 12,
    "target": 13,
    "id": "...",
    "wait": "292017.668117248s",
    "until": "2024-01-12T08:00:00.000001993Z"
  }

Changes
Set the until field in the PoetWaitRound, PoetWaitProof, AtxPublished event.
This change depends on https://github.com/spacemeshos/api/pull/297. Updates to go.mod/go.sum are pending until a new tagged version of api.
Test Plan
Manual testing
TODO

[X] Explain motivation or link existing issue(s)
[X] Test changes and document test plan
[ ] Update documentation as needed
[ ] Update changelog as needed

To use the updated API you have to update the dependency:
go get github.com/spacemeshos/api/release/go

Updated to use the released api 1.26. I incorporated the review comments from fasmat (thanks!).

LGTM. Thanks @xearl4
bors merge

bors merge

bors merge