MunkiReport does not run on macOS 12.3

With macOS 12.3 Beta 1, Apple removed Python entirely.
I tried to fix it for myself by replacing the current shebang of all Python files with the one from munki, but it doesn't work. I think all or at least some Python files are written for 2.7 but not with the one munki installs.

This is a known issue being actively worked on. Discussion is ongoing in the #munkireport-dev channel of the MacAdmins Slack.

This has been fixed on Munki 5.7.0 but broke MacOS 12.2.1 and below apparently because Munki now uses the third-party python install.

For 5.7 we decided to use munkireport-python2 and not an autodetect script to keep things simple

As highlighted in the release notes, MunkiReport now requires the installation of MunkiReport Python2 for all clients for 5.7.0 on all versions of macOS: https://github.com/munkireport/munkireport-php/releases/tag/v5.7.0
The wiki has been updated, but please let us know if any additional clarifying details could be added: https://github.com/munkireport/munkireport-php/wiki/Client-setup#python-2
I also wrote a blog post about the change that I hope is helpful: https://www.kevinmcox.com/2022/02/munkireport-and-python-2/


Added workflow for creating documentation pages

Added workflow for generating documentation pages for the TibberSwift project.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 79.84%. Comparing base (fbf7251) to head (325108a).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main       #7   +/-   ##
=======================================
  Coverage   79.84%   79.84%
=======================================
  Files           8        8
  Lines         129      129
=======================================
  Hits          103      103
  Misses         26       26

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


ZMQ support

Current dogecoin sw has no ZMQ support, is it possible to add it?
Or is there any plan to have an updated release of the node that supports it?

This will be coming with the 1.14 release.

Is this still slated for 1.14?

Yup; if you're comfortable building your own versions you could actually try the 1.14-branding branch now, which is essentially beta quality except we're having packaging problems.

@rnicoll What packaging problems are you having?


Broken cookie behaviour with an existing session.

Added pull request: https://github.com/SoftUni/csharp-web/pull/9
As of commit #https://github.com/SoftUni/csharp-web/commit/3fb04b0887ca23e15c97e72a02cbabe204c1459e the server does not correctly check and manage existing sessions and when to set cookies. When a session is present in the repository-like class the SetResponseSession() always attaches a new cookie with the existing GUID as ID, but with a new date (current UTC local time of the machine) to the response headers.
This should not be the expected behaviour as there is no point to always have Set-Cookie headers in the responses when a session is already present. In the current state it always returns the Set-Cookie header with every response and modifies the expiration dates to basically last the current time of when the request is processed to + 3 days.
https://github.com/SoftUni/csharp-web/blob/3fb04b0887ca23e15c97e72a02cbabe204c1459e/2019-May-Season/SoftUni-Information-Services/src/SIS.WebServer/ConnectionHandler.cs#L94-L102
Here it checks for an existing session and if it's not present we add it to the repository, which prevents us from checking in the SetResponseSession() if we need to add a new session and cookie. IMO this is not the correct behaviour.
https://github.com/SoftUni/csharp-web/blob/3fb04b0887ca23e15c97e72a02cbabe204c1459e/2019-May-Season/SoftUni-Information-Services/src/SIS.WebServer/ConnectionHandler.cs#L76-L92

Fixed with #9


Custom partitioner

addresses #619

@hyperlink I also updated the typings for kafka-node to reflect the changes in this PR, could you approve that as well? https://github.com/DefinitelyTyped/DefinitelyTyped/pull/14911
Thanks!

@hyperlink wouldnt it also be confusing to override the provided partitioners?
By providing a CustomPartitionerType the code is more self documenting than using  a partitionerType called "default" that uses a user provided getPartition function

@bkim54 I'm looking at it from the perspective of the module user. So far the user of the module is only exposed to the concepts of partitioner through the partitionerType.  With the code as is to add their own implementation requires them remember to change two things versus a single thing of adding a new argument. To me the later is simpler.

why then even provide the concept of partitionerType?
Wouldnt it then be better to have one "default partitioner" (a cyclic partitioner is probably the best default) and letting the user pass in a getPartition function

@bkim54 are you suggesting partitionerType could not only take 1, 2, 3 but also a function which would override the built in partitioners?

@hyperlink, no im suggesting providing one default partitioner type and giving users the ability to override the partitioner. (Similar to how the java client works).

Ok @bkim54 then could we address my initial concern regarding potential user error by throwing an exception if partitionerType is not 4 and a custom partitioner function was passed in?
Thanks!

@hyperlink thanks for working through this with me! 😃

heh np.


Make docs compile on the mac

So on my mac when I try to build the docs with "mkdocs" I get the following error
ERROR   -  Config value: 'theme_dir'. Error: The path theme doesn't exist

This fixes it for me. Is there a reason that it was the way it was? Does mkdocs work differently on Linux or something? I'm running mkdocs 0.14.0.

I pushed something I shouldn't have, it's fixed now.


NIFI-3677 - ExtractMediaMetadata should close TikaInputStream

Added finally block to close TikaInputStream.

Thank you for submitting a contribution to Apache NiFi.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[X] Is there a JIRA ticket associated with this PR? Is it referenced
in the commit message?

[ X Does your PR title start with NIFI-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.

[X] Has your PR been rebased against the latest commit within the target branch (typically master)?

[X] Is your initial contribution a single, squashed commit?

For code changes:

[X] Have you ensured that the full suite of tests is executed via mvn -Pcontrib-check clean install at the root nifi folder?
[ ] Have you written or updated unit tests to verify your changes?
[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[ ] If applicable, have you updated the LICENSE file, including the main LICENSE file under nifi-assembly?
[ ] If applicable, have you updated the NOTICE file, including the main NOTICE file found under nifi-assembly?
[ ] If adding new Properties, have you added .displayName in addition to .name (programmatic access) for each of the new properties?

For documentation related changes:

[ ] Have you ensured that format looks appropriate for the output in which it is rendered?

Note:
Please ensure that once the PR is submitted, you check travis-ci for build issues and submit an update to your PR as soon as possible.

+1, merging to master, thanks @jskora


feat/vui

add intents to enable/disable ggwave, starts disabled
we dont want devices always processing ggwave tones, this should only happen when enabled by user
TODO: add a timeout, so not keep enabled forever

If the plugin os replaceable with another one that reacts to same.bus messages, then the vui should be a companion skill, if the vui is exclusive to the plugin then it doesn't need to be in it's own repo, it could, but is there ever a reason to do so instead of shipping it with the mandatory code? Avoid the situation of having the skill and no plugin at all

Also note on this case the skill is not exposed as a skill plugin, it's loaded as part of the plugin itself, the plugin bundles it's voice interface. We already do this with OCP and Persona, and same happening in CommonIOT kinda, so it's not a new pattern

Also note on this case the skill is not exposed as a skill plugin, it's loaded as part of the plugin itself, the plugin bundles it's voice interface. We already do this with OCP and Persona, and same happening in CommonIOT kinda, so it's not a new pattern

It's definitely not a new pattern, no, but I wanted to understand what the reasoning was behind choosing one vs. another. It's more clear now. Thanks for clarifying!


Is there a way to disable psych enigne error messages without fixing the error?

What is your question?
Because I'm making a mod and I'm getting error messages that don't affect the game or the gameplay and only the message bothers me.

Normally, if there is an error message from the game, there will be 2 cases
Case 1: your lua script file mod is coded incorrectly
case 2: The game file itself has problems in the asset files

and you can't turn off that notification

chipi chipi chapa chapa!!

chipi chipi chapa chapa!!

It follows me everywhere.

the chipi chipi chapa chapa cat sees you.

luaDebugMode = false, alternatively just fix the errors, shouldn't even be hard at all


Expose theme name in webviews

Fixes #97663
Adds a data attribute data-vsccode-theme to webviews with the current theme name. This is lets extensions write theme specific css for webviews, for example:
body[data-vscode-theme="One Dark Pro"] {
    background: red;
}

There are hundreds of existing themes out there and they can be updated anytime. I don't see how a webview provider could keep track with that.
It were better if views don't define theme specific colors, but that an existing color of a theme is used (or a derivative of it (darker/lighter/mix).
Can you explain the use case?

here is the origin feature request, https://github.com/Binaryify/OneDark-Pro/issues/388
I want custom markdown preview color highlight, so I write some css with the  "markdown.previewStyles" in the package.json, but then I found it also actived in other themes too, this cause some problems, and don't have any way to limit that
Here's a comparison with custom markdown preview

Ok, I see a theme wants to style the markdown preview through markdown.styles.
@mjbvz If it's just for that background color, can't the Markdown define a new color so that all themes can easily theme it? That's more in the spirit of theming.

@aeschli no just for background color, include the code highlight and the elements style

@aeschli Extensions should use the normal theme color variables in almost all cases. We also already expose the theme kind (light, dark, high contrast)
The theme name is helpful if you want to write CSS that is only enabled in one specific theme. For example, the official GitHub theme may contribute CSS rules to the markdown preview that are only enabled when the GitHub theme is active


Made add_label option default true

I think some additional work is needed to make the columns line up nicely when the row label is added. Could you post a screenshot of what your case looks like using this branch?

I think some additional work is needed to make the columns line up nicely when the row label is added. Could you post a screenshot of what your case looks like using this branch?

Correct, it doesn't work yet.
There is a picture here. I'll continue working on this now, I'm happy to rebase this if https://github.com/epics-containers/pvi/pull/42 is merged first.

OK sure I understand.
I think actually Giles' work to update to use pydantic will be merged first because that will cause lots of conflicts that need resolving.


Retain the direction of tcp metric as it is what we need

Think of the request A->B, there are two TCP flows, which are A calls B and B returns to A, associated with it. We tried identifying the TCP flow direction to make it associated with the business request before, such as for the metric of retransmitting packet count, we want to change both the TCP flows of A->B and B->A to A->B so as to know whether the performance of a business request is affected by these two direction flows.
This is designed for simplifying the process of querying. But it turns out that we can not classify these directions accurately, so we decided to retain the real direction of the metric and leave the process of association to querying.

LGTM. By the way, that was strange to use SrcNamespace to show whether the DST is from InternalCluster or ExternalCluster which we did before.

It's a typo and caused a bug. I have fixed it in this PR.


500 error if superuser is logged in and clicks on Account

If you create a user from command line using createsuperuser
login and then click on account then it raises a 500 error
ERROR 2016-11-23 15:17:18,964 django.request Internal Server Error: /account/
Traceback (most recent call last):
  File "/usr/local/lib/python3.4/dist-packages/django/core/handlers/exception.py", line 39, in inner
    response = get_response(request)
  File "/usr/local/lib/python3.4/dist-packages/django/core/handlers/base.py", line 187, in _get_response
    response = self.process_exception_by_middleware(e, request)
  File "/usr/local/lib/python3.4/dist-packages/django/core/handlers/base.py", line 185, in _get_response
    response = wrapped_callback(request, *callback_args, **callback_kwargs)
  File "/usr/local/lib/python3.4/dist-packages/django/views/generic/base.py", line 68, in view
    return self.dispatch(request, *args, **kwargs)
  File "/usr/local/lib/python3.4/dist-packages/django/utils/decorators.py", line 67, in _wrapper
    return bound_func(*args, **kwargs)
  File "/usr/local/lib/python3.4/dist-packages/django/contrib/auth/decorators.py", line 23, in _wrapped_view
    return view_func(request, *args, **kwargs)
  File "/usr/local/lib/python3.4/dist-packages/django/utils/decorators.py", line 63, in bound_func
    return func.__get__(self, type(self))(*args2, **kwargs2)
  File "/usr/local/lib/python3.4/dist-packages/django/views/generic/base.py", line 88, in dispatch
    return handler(request, *args, **kwargs)
  File "/usr/local/lib/python3.4/dist-packages/django/views/generic/edit.py", line 235, in get
    self.object = self.get_object()
  File "./della/user_manager/views.py", line 79, in get_object
    return self.request.user.userprofile
  File "/usr/local/lib/python3.4/dist-packages/django/utils/functional.py", line 235, in inner
    return func(self._wrapped, *args)
  File "/usr/local/lib/python3.4/dist-packages/django/db/models/fields/related_descriptors.py", line 370, in __get__
    self.related.get_accessor_name()
django.db.models.fields.related_descriptors.RelatedObjectDoesNotExist: User has no userprofile.

Same error happens if you click on "Message Santee"

The latest master fixes this. And I have also updated in the docs. So now onwards admins won't have those options in the navbar.
Thanks for reporting the issue! 👍


Can change locale text-to-speech ?

can config to change locale tts if language epub file not english language.

i set locale US to another locale in lib in MediaController.java[cid:d03435bc-60e8-4fb3-934b-9046a5a2440e]

จาก: lampofdrog<EMAIL_ADDRESS>ส่ง: 29 มกราคม 2561 22:31
ถึง: FolioReader/FolioReader-Android
สำเนาถึง: changefouk; Author
ชื่อเรื่อง: Re: [FolioReader/FolioReader-Android] Can change locale text-to-speech ? (#120)
up, i have the same issue, any solution ??
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/FolioReader/FolioReader-Android/issues/120#issuecomment-361282264, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ARyGbHMAyuk7tKsChOG-y5a8DzlBnmzFks5tPeRugaJpZM4P6BwV.


[Feature Request]: use withEventReplay

Relevant Scope
angular application
Description
Once the project updated to v18 (https://github.com/angular-sanctuary/angular-hub/issues/273), add withEventReplay
Describe the solution you'd like
No response
Describe alternatives you've considered
No response
Do you want to contribute with a pull request?
No

Currently failing with the following error:
ERROR RuntimeError: NG04002: Cannot match any routes. URL Segment: 'favicon.ico'
    at Recognizer.noMatchError (/Users/gerome/Documents/GitHub/angular-hub/node_modules/@angular/router/fesm2022/router.mjs:3838:16)
    at eval (/Users/gerome/Documents/GitHub/angular-hub/node_modules/@angular/router/fesm2022/router.mjs:3868:28)
    at /Users/gerome/Documents/GitHub/angular-hub/node_modules/rxjs/dist/cjs/internal/operators/catchError.js:13:51
    at OperatorSubscriber._this._error (/Users/gerome/Documents/GitHub/angular-hub/node_modules/rxjs/dist/cjs/internal/operators/OperatorSubscriber.js:43:21)
    at Subscriber.error (/Users/gerome/Documents/GitHub/angular-hub/node_modules/rxjs/dist/cjs/internal/Subscriber.js:60:18)
    at Subscriber._error (/Users/gerome/Documents/GitHub/angular-hub/node_modules/rxjs/dist/cjs/internal/Subscriber.js:84:30)
    at Subscriber.error (/Users/gerome/Documents/GitHub/angular-hub/node_modules/rxjs/dist/cjs/internal/Subscriber.js:60:18)
    at Subscriber._error (/Users/gerome/Documents/GitHub/angular-hub/node_modules/rxjs/dist/cjs/internal/Subscriber.js:84:30)
    at Subscriber.error (/Users/gerome/Documents/GitHub/angular-hub/node_modules/rxjs/dist/cjs/internal/Subscriber.js:60:18)
    at Subscriber._error (/Users/gerome/Documents/GitHub/angular-hub/node_modules/rxjs/dist/cjs/internal/Subscriber.js:84:30) {
  code: 4002

The fix should be released this wednesday: https://github.com/angular/angular/issues/56122#issuecomment-2134653401


What is fuelphp-storage?

Asking as it seems like this might be a hint that this library is no longer maintained.

None of these are no longer maintained, but we've kept them online as people might be using them.

Gotcha, thanks!


fixing link to user submitted bugs

Jenkins job istio.github.io/presubmit passed

Jenkins job istio.github.io/presubmit passed

Jenkins job istio.github.io/presubmit passed


NEXT-9696 | Listing fiter cut off during overflow hidden

If you have a CMS Section that doesn't have enough space at the bottom - e.g. only the filter element in it, the filters will be cut off when you open them due to overflow: hidden;
I'm not quite sure what the overflow: hidden; is for since it is basically superfluous according to my tests.
Ticket:
https://issues.shopware.com/issues/NEXT-9696
Bug:

Solution:

Hey @ChristopherDosin, thanks for your contribution 💙  Unfortunately we already fixed this with https://github.com/shopware/platform/commit/56569e7b3c5eb1514ca4a2f85ba8d196cb04c0c9


Upload retry support (to mirror download retry behavior)

S3Transfer (and thus boto3) has the num_download_attempts, which handles retries for issues like socket.timeout, socket.error, etc.
It would be greatly helpful if a similar option as added (and exposed up through boto3) for the upload direct.

Marking as a feature request.

Is this implemented?
What is the default no of retry  attempts internally for upload?

This would be particularly helpful for multipart uploads, where one chunk might fail towards the end of a potentially very large upload. With the current behavior, the transfer aborts and the entire file has to be reuploaded. Ideally, the library would retry the failed chunk(s) and proceed with other chunks until it hit a failure threshold for any given chunk.


Use in java source

Hey dude,
very nice project.
Is it possible to use this in java? My project is written in java, not in kotlin.
I dont know how to use
"thread {
val updater = Updater(this, "Link_to_your_repo")
checkForUpdates(updater)
}"
in java

Hey dude,
very nice project. Is it possible to use this in java? My project is written in java, not in kotlin.
I dont know how to use
"thread { val updater = Updater(this, "Link_to_your_repo") checkForUpdates(updater) }"
in java

This article from w3schools should help you on usage of threads.

Thanks for fast answer. Any chance you can "translate" your code snippets to plain java?


Friendly error messages for bad configs

Currently a bad config results in a noisy stack trace.  Validate configs up front with friendlier error messages.
% gateway run
2021-11-01T16:20:08.530-0400	DEBUG	process/tracing.go:72	Anonymized tracing disabled
2021-11-01T16:20:08.530-0400	DEBUG	process/debug.go:37	debug server listening on <IP_ADDRESS>:58073
2021-11-01T16:20:08.530-0400	DEBUG	process/metrics.go:79	Telemetry disabled
2021-11-01T16:20:08.530-0400	INFO	st-gateway/main.go:151	Starting Storj DCS S3 Gateway

2021-11-01T16:20:08.530-0400	INFO	st-gateway/main.go:152	Endpoint: <IP_ADDRESS>:7777

2021-11-01T16:20:08.530-0400	INFO	st-gateway/main.go:153	Access key: insecure-dev-access-key

2021-11-01T16:20:08.530-0400	INFO	st-gateway/main.go:154	Secret key: insecure-dev-secret-key

2021-11-01T16:20:08.530-0400	WARN	st-gateway/main.go:158	Failed to contact Satellite. Perhaps your configuration is invalid?
main.cmdRun
	/Users/bill/storj/st-gateway/main.go:158
storj.io/private/process.cleanup.func1.4
	/Users/bill/go/pkg/mod/storj.io/private@v0.0.0-20210810102517-434aeab3f17d/process/exec_conf.go:363
storj.io/private/process.cleanup.func1
	/Users/bill/go/pkg/mod/storj.io/private@v0.0.0-20210810102517-434aeab3f17d/process/exec_conf.go:381
github.com/spf13/cobra.(*Command).execute
	/Users/bill/go/pkg/mod/github.com/spf13/cobra@v1.1.3/command.go:852
github.com/spf13/cobra.(*Command).ExecuteC
	/Users/bill/go/pkg/mod/github.com/spf13/cobra@v1.1.3/command.go:960
github.com/spf13/cobra.(*Command).Execute
	/Users/bill/go/pkg/mod/github.com/spf13/cobra@v1.1.3/command.go:897
storj.io/private/process.ExecWithCustomConfig
	/Users/bill/go/pkg/mod/storj.io/private@v0.0.0-20210810102517-434aeab3f17d/process/exec_conf.go:88
storj.io/private/process.Exec
	/Users/bill/go/pkg/mod/storj.io/private@v0.0.0-20210810102517-434aeab3f17d/process/exec_conf.go:65
main.main
	/Users/bill/storj/st-gateway/main.go:434
runtime.main
	/usr/local/go/src/runtime/proc.go:255
2021-11-01T16:20:08.530-0400	DEBUG	process/exec_conf.go:394	Unrecoverable error	{"error": "gateway setup error: uplink: invalid access grant format", "errorVerbose": "gateway setup error: uplink: invalid access grant format\n\tstorj.io/uplink.ParseAccess:98\n\tmain.AccessConfig.GetAccess:48\n\tmain.checkCfg:175\n\tmain.cmdRun:156\n\tstorj.io/private/process.cleanup.func1.4:363\n\tstorj.io/private/process.cleanup.func1:381\n\tgithub.com/spf13/cobra.(*Command).execute:852\n\tgithub.com/spf13/cobra.(*Command).ExecuteC:960\n\tgithub.com/spf13/cobra.(*Command).Execute:897\n\tstorj.io/private/process.ExecWithCustomConfig:88\n\tstorj.io/private/process.Exec:65\n\tmain.main:434\n\truntime.main:255"}

On a release binary, this behaves quite differently, which sets the --defaults flag to release and supresses a whole bunch of backtraces and verbose output.
See the difference if it's set correctly:
gateway run --defaults release
2021-11-02T19:02:29.369+1300	INFO	Telemetry enabled	{"instance ID": "54:05:db:37:35:0f"}
2021-11-02T19:02:29.369+1300	INFO	Starting Storj DCS S3 Gateway
2021-11-02T19:02:29.369+1300	INFO	Endpoint: <IP_ADDRESS>:7777
2021-11-02T19:02:29.369+1300	INFO	Access key: insecure-dev-access-key
2021-11-02T19:02:29.369+1300	INFO	Secret key: insecure-dev-secret-key

Error: gateway setup error: uplink: invalid access grant format

Given this, I'm not sure if we should change anything here for dev mode, as it's supposed to be more verbose by design?

Should we limit the stack trace depth?

I think we could do that. I'm having a look at the zap API to see if there's any way to control showing a backtrace on a per-log event basis. I think it's only enable or disable for the entire logger instance from first glance. Perhaps we could create another logger specifically for validation that has the backtrace disabled or limited (?)


[GitHub] badge for (pre-)release date

URI : github/release-date/:user/:repo/all.svg
Actually, there has URI for release version badge like that -
/github/release/Cockatrice/Cockatrice.svg full release
/github/release/Cockatrice/Cockatrice/all.svg included pre-release
So, for consistency, I've done release date like that -
/github/release-date/Cockatrice/Cockatrice.svg full release (Previously Merged #1122)
/github/release-date/Cockatrice/Cockatrice/all.svg included pre-release (This PR)

https://github.com/badges/shields/pull/1122#issuecomment-334113759

Thanks for this!
In terms of the URL, indeed, we've got https://img.shields.io/github/release/qubyte/rubidium/all.svg, and it looks like that is an old badge. Though we also have a number of pre-release version badges, and all of those use /vpre.
Since there are more vpre, how about we try to standardize in that direction? releasepre and release-datepre don't look so great, so I think we should use release-pre/ and release-date-pre/.
Does that make sense?
No need to change release/ in this PR. Just release-date/.../all -> release-date-pre.

I guess that's a bit tricky. Since the /all version will return simply all releases (normal releases AND prereleases, not only prereleases). Prereleases just happen to be the latest releases available, if there are any.
If there are none, then https://img.shields.io/github/release/qubyte/rubidium/all.svg will return the same as https://img.shields.io/github/release/qubyte/rubidium.svg.

Yep! @tooomm  is right.
Actually there is no API for getting pre-release. All means, it is the most latest release (may be Pre or may be NOT).  [Previous release/ works like that. So I've done like that ]
I think, if I do this release-date-pre , then all I've to do - just filter the received data .

Hmm. Is this different from the behavior of vpre?

/release/qubyte/rubidium/all & /release/qubyte/rubidium is giving same result.
As all picks the most lastest version (may or may not PRE).

As case here.

These two badges are both showing

https://img.shields.io/nuget/v/Microsoft.AspNet.Mvc.svg
https://img.shields.io/nuget/vpre/Microsoft.AspNet.Mvc.svg

ok ok! I got that. Okay, I'm just changing the route

I think, if I do this release-date-pre , then all I've to do - just filter the received data.

If that would be possible, that surpasses the way /all works right now for surel
An actual badge that consistently shows only prerelease data all the time would be nice.

An actual badge that consistently shows only prerelease data (date, downloads, ...) all the time would be nice.

I'm not sure I get it. If 2.0.0-rc2 were followed by 2.0.0, why would it be helpful to see 2.0.0-rc2?

Github service tests passing locally when merging master + applying the patch from #1112, except a couple services, unaffected here, which are timing out.
  1) Github Commits since by latest release
	[ GET http://localhost:1111/github/commits-since/microsoft/typescript/latest.json ]:
     Error: Timeout of 2000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves.

  2) Github github issue author
	[ GET http://localhost:1111/github/issues/detail/u/badges/shields/979.json ]:
     Error: Timeout of 2000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves.

Working on fixing the timeout in MarkHerhold/IcedFrisby#71.

I'm not sure I get it. If 2.0.0-rc2 were followed by 2.0.0, why would it be helpful to see 2.0.0-rc2?

You might be right, in case you just published a new version... your pre-release badge would be outdated because there is no newer pre-release than release yet.

Honestly, I think so 😄 👍
Btw, thanks for the merging


Fix page title link

The link of the page title points to localhost:1313 instead of to "https://example-working-group-website.netlify.app/"
I think this is fixed by simply adding href="/".

Good catch! Fixed


GetArnieQuotes Implementation

Implementation of the Arnie Quotes function

I accidentally opened a PR on this repo instead of on my forked repo so I'll leave here instead.

Sorry, was sure I had forked the repo. Obviously not

Apologies. I had forked the repo but accidentally pushed my PR to this repo. I've since closed this and got a PR opened on just my fork


Refactor and rename  graphql-client package to blockchain-providers

Codecov Report

Merging #70 (783ad3b) into master (d05b74d) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##            master       #70   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files          105       104    -1
  Lines        15017     15091   +74
  Branches      1348      1362   +14
=========================================
+ Hits         15017     15091   +74

Files
Coverage Δ

...-providers/src/ergo-graphql/ergoGraphQLProvider.ts
100.00% <100.00%> (ø)

...s/blockchain-providers/src/ergo-graphql/queries.ts
100.00% <100.00%> (ø)

packages/blockchain-providers/src/utils/_tests.ts
100.00% <ø> (ø)

packages/blockchain-providers/src/utils/graphql.ts
100.00% <100.00%> (ø)

packages/blockchain-providers/src/utils/index.ts
100.00% <100.00%> (ø)

packages/common/src/error.ts
100.00% <100.00%> (ø)

packages/common/src/utils/assertions.ts
100.00% <100.00%> (ø)

packages/common/src/utils/utxo.ts
100.00% <100.00%> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Regent: support docstrings

In order to build reusable libraries in Regent, it would be really nice to support docstrings or comments, ideally with some sort of a tool to parse and generate documentation. I don't know that we want to build our own documentation tool, but perhaps we can integrate something that's pre-existing.

Just a suggestion -- Doxygen can be modified to work new languages. There is a pre-existing Lua filter for it https://github.com/alecchen/doxygen-lua so shouldn't be too hard to modify this to work for Regent.

I personally prefer sphinx, it also works with lua as well. I'm not a big fan of doxygen.

I don't really have a strong opinion on which tool to use, but I would like to not fork or modify the tool itself in order to add the integration.
Ideally, we'd add a flag like -fdoc 1 to the Regent compiler itself, so that we can reuse the existing Regent parser and type checker. Then we can split out XML or whatever the tool consumes and run it through the tool to spit out the final documentation.

Another option that I've been playing around with is ldoc. It seems to work with luajit and actually does something right now when I run it on Regent code (with a hack).
I've been looking at the code for ldoc. It seems like it should be possible to modify it to add support for Regent.
Heres an example of the documentation it generates:
https://stevedonovan.github.io/Penlight/api/libraries/pl.data.html

There is a forked version of ldoc here:
https://github.com/syamajala/LDoc
You'll want to do git clone --recursive into the legion/language directory and then do make. Add terra to your path and then you can do ./ldoc.lua. The forked version has been fixed to work with Regent and Terra files, but it doesn't really do anything fancy right now.
It would be nice if for the generated documentation it split the functions up by language and had a section for Tasks, Terra Functions, and Lua Functions. I think for Regent and Terra we should have it automatically extract types for arguments that are tagged with @param. For region arguments we should probably show privileges, but I'm not sure how that well that would work for metaprogrammed tasks that operate on a lot of fields like in S3D. Its probably a good idea to show task annotations as well?
The html pages ldoc generates currently lack a search feature, we could probably borrow that from sphinx? Having a permalink next to each section header would be nice.
Any other ideas?

Ok, @syamajala and I have been pushing on LDoc. There is now a fork here: https://github.com/StanfordLegion/LDoc . This seems to be working reasonably well and has patches to support the things we need for at least basic functionality.
I put up some initial documentation for the standard library here: http://regent-lang.org/doc/
I'm going to consider this done for the moment. If there are further improvements to make we can put those in separate issues.


Improve num_processes question in CLI

Improve the phrasing of asking the number of processes to be used during training
What does this add?
This slightly tweaks the question asking how many devices the user will be using when running accelerate config
Who is it for?
Closes https://github.com/huggingface/accelerate/issues/338
Why is it needed?
Stating the number of processes is very general, and I as well found it a bit confusing until I learned that just means number of GPUs, CPUs, etc that will be the main training device. As a result, PR proposes changing it from:
How many processes in total will you use? [1]

To:
How many devices should be used for (potentially) distributed training? [1:]

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


False negative accessCheck failure on non-chained entity queries

Almost all problems with the accessCheck on entity queries are gone with #476.
Whilst working on https://www.drupal.org/project/drupal/issues/3330981 I've found a few escapees.
This one is about false negative accessCheck failure on entity queries that are not chained.
I've added a test-case in the attached PR, it's probably more easily explained in code (and test failures) than words.

Sounds like the opposite problem https://github.com/mglaman/phpstan-drupal/issues/388 was opened for, where extreme chaining things broke (I think.)

Opened a discussion https://github.com/phpstan/phpstan/discussions/8773
I don't think this is possible because we cannot track side effects on objects.

I am silly; https://phpstan.org/developing-extensions/type-specifying-extensions is the answer after all. I assumed not because I have only seen implementations that read the args.

This was fixed with https://github.com/mglaman/phpstan-drupal/pull/497


Update AutoRest C# version to 3.0.0-beta.20240606.2

Update AutoRest C# version to 3.0.0-beta.20240606.2

We have a later version in https://github.com/Azure/azure-sdk-for-net/pull/44434


Added the hover to open FAQ answer feature

Pull Request
Title
Added the hover to open FAQ answer

Related Issues
Fixes #518

Changes Made
FAQ compoenent

Checklist

[x] I have tested the changes locally
[ ] Documentation has been updated (if necessary)
[x] Changes are backward-compatible

Screenshots (if applicable)
https://github.com/user-attachments/assets/13dfe007-d353-4c18-b304-cf77f2f6c152

Additional Notes

Footer

@VaibhavArora314
I fixed the build error and removed formatting in this one 👍


Upgrade to Listr 2

It looks like the listr project has been abandoned. any chance this could use listr2. It looks like it might be a drop in replacement.
https://github.com/cenk1cenk2/listr2
https://listr2.kilic.dev/

You can make a PR ;)

OK so I tried upgrading, and it was not a 100% drop in replacement. I changed a couple things, but I'm stuck on two errors while running the tests.
First, it seems that the setRenderer function isn't present on whatever the result of this function call is.
src/services/ProjectPackageJson.spec.ts
      const list = projectPackageJson.install({packageManager: PackageManager.NPM});

Second, It's also throwing an uncaught exception error.
Saying  TypeError: Converting circular structure to JSON
The stack trace on this is not really helpful I tried doing some console.log statements around all of the various JSON.stringify calls in the project but I think something in jest is silencing the console.log statements, or it's not happening inside the cli repo.
I'm having issues using my normal debugging tools (console.log) . I'll see if I can spend some more time on this, but I'm not sure I'll be able to.
I'm mostly interested in the Listr2 support for exactly this issue, I'm having trouble developing CLI commands with tsed-cli because listr v1 doesn't really handle printing to the console very well. and Listr v2 seems to have support for using your own logger.

Hello @from-nibly
Can you push the code ? I’ll try to find the issue with the circular json ^^
For the setRenderer, comment the line. We will that later ;)
See you
Romain

done https://github.com/from-nibly/tsed-cli/commit/8c0f0859b4af9c14c1fe2997d9f8641188ba8867

:tada: This issue has been resolved in version 3.9.0 :tada:
The release is available on:

v3.9.0
GitHub release

Your semantic-release bot :package::rocket:


Please could we have a tiny bit of instruction on how to get a locationid

I'm a bit stuck on how to get the locationID feature working as i don't know how to get a valid location ID.
I've found some things in the instagram developer section about location endpoints but I don't really know if that's the right thing.
Not expecting a step by step breakdown at this point but a couple of sentences about where to start would be helpful for a person of my skill level.

Hey @mkstix6. Sorry for lack of support there, that's my fault. Fetching images by location is one of the least-documented parts of the library.
Check #10 for a bit more detail on how to find an ID. If you have any other questions feel free to comment back on that thread.

Thanks Steven. Sorry for the late reply.

Hi Steven,
@stevenschobert I can able to get data from userId, but not with tagged and using locationId also.
This call is going in the network, but not returning proper values.
https://api.instagram.com/v1/locations/1494866470783660/media/recent?access_token=xxx&callback=instafeedCachebae37999dfa1fcc6.parse
it is returning empty data.

@Praburajendran I don't think the Instagram API supports fetching data for locations, see this section of the README.


Frequently ExchangeTablePartition may crash when reach SyncAllSchema

Bug Report
Please answer these questions before submitting your issue. Thanks!
1. Minimal reproduce step (Required)

change maxNumberOfDiffs = 0(To simulate a lot of irrelevant DDL in the middle to reach syncAllSchema)
run this ft, which is modified on tests/fullstack-test2/ddl/alter_exchange_partition.test

# Copyright 2022 PingCAP, Ltd.
#
# Licensed under the Apache License, Version 2.0 (the "License");
# you may not use this file except in compliance with the License.
# You may obtain a copy of the License at
#
#     http://www.apache.org/licenses/LICENSE-2.0
#
# Unless required by applicable law or agreed to in writing, software
# distributed under the License is distributed on an "AS IS" BASIS,
# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
# See the License for the specific language governing permissions and
# limitations under the License.

>> DBGInvoke __enable_schema_sync_service('true')
mysql> drop table if exists test.e;
mysql> drop table if exists test.e2;
mysql> drop table if exists test_new.e2;
mysql> drop database if exists test_new;

mysql> create table test.e(id INT NOT NULL,fname VARCHAR(30),lname VARCHAR(30)) PARTITION BY RANGE (id) ( PARTITION p0 VALUES LESS THAN (50),PARTITION p1 VALUES LESS THAN (100),PARTITION p2 VALUES LESS THAN (150), PARTITION p3 VALUES LESS THAN (MAXVALUE));
mysql> alter table test.e set tiflash replica 1;

mysql> create table test.e2(id int not null, fname varchar(30), lname varchar(30));
mysql> alter table test.e2 set tiflash replica 1;

mysql> create database test_new;
mysql> create table test_new.e2(id int not null, fname varchar(30), lname varchar(30));
mysql> alter table test_new.e2 set tiflash replica 1;

func> wait_table test e
func> wait_table test e2
func> wait_table test_new e2

mysql> insert into test.e values (1, 'a', 'b'),(108, 'a', 'b');
mysql> insert into test.e2 values (2, 'a', 'b');
mysql> insert into test_new.e2 values (3, 'a', 'b');

# disable schema sync service
>> DBGInvoke __enable_schema_sync_service('false')
>> DBGInvoke __refresh_schemas()

# case 2, exchange partition across databases, no error happens
mysql> set @@tidb_enable_exchange_partition=1; alter table test.e exchange partition p0 with table test_new.e2
>> DBGInvoke __refresh_schemas()
mysql> alter table test.e add column c1 int;
>> DBGInvoke __refresh_schemas()
mysql> set session tidb_isolation_read_engines='tiflash'; select * from test.e order by id;
+-----+-------+-------+------+
| id  | fname | lname | c1   |
+-----+-------+-------+------+
|   3 | a     | b     | NULL |
| 108 | a     | b     | NULL |
+-----+-------+-------+------+
mysql> set session tidb_isolation_read_engines='tiflash'; select * from test_new.e2;
+----+-------+-------+
| id | fname | lname |
+----+-------+-------+
|  1 | a     | b     |
+----+-------+-------+
mysql> alter table test.e drop column c1;
>> DBGInvoke __refresh_schemas()

# case 6, exchagne partition across databases, error happens after exchange step 1
mysql> set @@tidb_enable_exchange_partition=1; alter table test.e exchange partition p0 with table test_new.e2
>> DBGInvoke __enable_fail_point(exception_after_step_1_in_exchange_partition)
>> DBGInvoke __refresh_schemas()
>> DBGInvoke __refresh_schemas()
mysql> alter table test.e add column c1 int;
>> DBGInvoke __refresh_schemas()
mysql> set session tidb_isolation_read_engines='tiflash'; select * from test.e order by id;
+-----+-------+-------+------+
| id  | fname | lname | c1   |
+-----+-------+-------+------+
|   1 | a     | b     | NULL |
| 108 | a     | b     | NULL |
+-----+-------+-------+------+
mysql> set session tidb_isolation_read_engines='tiflash'; select * from test_new.e2;
+----+-------+-------+
| id | fname | lname |
+----+-------+-------+
|  3 | a     | b     |
+----+-------+-------+
mysql> alter table test.e drop column c1;
>> DBGInvoke __refresh_schemas()

mysql> drop table if exists test.e;
mysql> drop table if exists test.e2;
mysql> drop table if exists test_new.e2;
mysql> drop database if exists test_new;
>> DBGInvoke __enable_schema_sync_service('true')

2. What did you expect to see? (Required)
run successfully
3. What did you see instead (Required)

4. What is your TiFlash version? (Required)
master

Workaround:
Find out the partition with partition_id equals to the id in logging, and then drop it
> select TABLE_SCHEMA,TABLE_NAME,'' from information_schema.tables where TIDB_TABLE_ID=89 union select TABLE_SCHEMA,TABLE_NAME,PARTITION_NAME from information_schema.partitions where TIDB_PARTITION_ID=89;
+--------------+------------+----+
| TABLE_SCHEMA | TABLE_NAME |    |
+--------------+------------+----+
| test         | e          | p0 |
+--------------+------------+----+
> alter table e drop partition p0;

This issue is caused by:

executing ALTER TABLE ... EXCHANGE PARTITION ... across databases on TiDB
tiflash entre loadAllSchema rather than tryLoadSchemaDiffs
tiflash can not handle ALTER TABLE ... EXCHANGE PARTITION ... across databases correctly

Confirm this issue does not affect v7.5.x/v8.1.x

close as this bug has been fixed in 6.5/7.1


Squashed commit of the following:

commit e3dfe656ac62393ac655de29fde46859dba37e5b
Author: Matthew Templeton<EMAIL_ADDRESS>Date:   Fri Jul 19 13:28:39 2024 -0400
 	modified:   adsingestp/parsers/jats.py
 	modified:   tests/stubdata/output/mdpi_climate-11-00147.json
 	modified:   tests/stubdata/output/mdpi_symmetry-15-00939.json

commit 4914c5a9835fd0ebb7f851b5ce09f2a47351fce7
Author: Matthew Templeton<EMAIL_ADDRESS>Date:   Wed Jul 17 11:51:44 2024 -0400
updates pyproject
 	modified:   pyproject.toml

commit 44cd801902a84fd41579dc5585bd3b24781da2e7
Author: Matthew Templeton<EMAIL_ADDRESS>Date:   Wed Jul 17 10:52:47 2024 -0400
fixes affid issue
 	modified:   adsingestp/parsers/jats.py

commit 146e4d13de2848bcab4e5ecf0346ce16fd98e94e
Author: Matthew Templeton<EMAIL_ADDRESS>Date:   Wed Jul 17 09:28:36 2024 -0400
almost got it right, still needs work
 	modified:   adsingestp/parsers/base.py
 	modified:   adsingestp/parsers/jats.py

commit ebd42ab60cb981916ce7d86034f94763cf537a5d
Author: Matthew Templeton<EMAIL_ADDRESS>Date:   Wed Jul 17 09:00:57 2024 -0400
 	modified:   adsingestp/parsers/base.py

commit c469d621eb3a08a061350b5f745c0dd11b86e62a
Author: Matthew Templeton<EMAIL_ADDRESS>Date:   Wed Jul 17 08:51:15 2024 -0400
aff ids, still in progress
 	modified:   adsingestp/parsers/base.py
 	modified:   adsingestp/parsers/jats.py

commit acc40dc6f455366b22967d35318ca58a5f57706b
Author: Matthew Templeton<EMAIL_ADDRESS>Date:   Wed Jul 10 18:39:44 2024 -0400
updates roman num test with new jats parser
 	modified:   adsingestp/parsers/jats.py
 	modified:   tests/stubdata/output/jats_springer_roman_num_1.json

commit 69f4a78877aba5714c87afaf602e5c5a6a7b242f
Merge: 529248b 7bf0ca9
Author: Matthew Templeton<EMAIL_ADDRESS>Date:   Wed Jul 10 18:28:51 2024 -0400
Merge branch 'main' of github.com:adsabs/ADSIngestParser into update_idm_affilid.20240708

commit 529248b038ac537e5ccb730e438d38a4ead67e2d
Author: Matthew Templeton<EMAIL_ADDRESS>Date:   Wed Jul 10 18:20:24 2024 -0400
mod'ed wiley parser for i_d_m compatibility.
 	modified:   adsingestp/parsers/wiley.py
 	modified:   tests/stubdata/output/wiley_jgra_57392.json
 	modified:   tests/stubdata/output/wiley_swe_21103.json
 	modified:   tests/stubdata/output/wiley_swe_461.json
 	modified:   tests/stubdata/output/wiley_swe_539.json

commit 237ce74933c91832e0cc0a33bcd35a01ef3320a1
Author: Matthew Templeton<EMAIL_ADDRESS>Date:   Wed Jul 10 16:06:45 2024 -0400
improved affstring+affid handling, formatting
 	modified:   adsingestp/parsers/jats.py
 	deleted:    adsingestp/parsers/jtest.py
 	deleted:    adsingestp/parsers/jtestaff.py
 	deleted:    adsingestp/parsers/piffol.py
 	new file:   tests/stubdata/output/els_roman_num_1.json
 	new file:   tests/stubdata/output/els_roman_num_2.json
 	modified:   tests/stubdata/output/jats_a+a_multiparagraph_abstract.json
 	modified:   tests/stubdata/output/jats_a+a_subtitle.json
 	modified:   tests/stubdata/output/jats_aip_amjph_90_286.json
 	modified:   tests/stubdata/output/jats_aj_158_4_139.json
 	modified:   tests/stubdata/output/jats_apj_859_2_101.json
 	modified:   tests/stubdata/output/jats_apj_967_1_35.json
 	modified:   tests/stubdata/output/jats_aps_phrvd_100_052015.json
 	modified:   tests/stubdata/output/jats_aps_phrvx_12_021031.json
 	modified:   tests/stubdata/output/jats_edp_aa_661_70.json
 	modified:   tests/stubdata/output/jats_edp_jnwpu_40_96.json
 	modified:   tests/stubdata/output/jats_iop_aj_162_1.json
 	modified:   tests/stubdata/output/jats_iop_ansnn_12_2_025001.json
 	modified:   tests/stubdata/output/jats_iop_apj_923_1_47.json
 	modified:   tests/stubdata/output/jats_iop_jinst_17_05_P05009.json
 	modified:   tests/stubdata/output/jats_iop_no_orcid_tag.json
 	modified:   tests/stubdata/output/jats_iop_preprint_in_record.json
 	modified:   tests/stubdata/output/jats_iucr_d-60-02355.json
 	modified:   tests/stubdata/output/jats_iucr_d-75-00616.json
 	modified:   tests/stubdata/output/jats_mnras_493_1_141.json
 	modified:   tests/stubdata/output/jats_nature_41467_2023_Article_40261_nlm.json
 	new file:   tests/stubdata/output/jats_nature_natas_tmp.json
 	modified:   tests/stubdata/output/jats_nature_natsd_12_7375.json
 	new file:   tests/stubdata/output/jats_nature_roman_num_1.json
 	modified:   tests/stubdata/output/jats_phrvd_106_023001.json
 	modified:   tests/stubdata/output/jats_pnas_1715554115.json
 	new file:   tests/stubdata/output/jats_sci_376_521.json
 	modified:   tests/stubdata/output/jats_spie_jmnmm_1.JMM.21.4.041407.json
 	modified:   tests/stubdata/output/jats_spie_opten_1.OE.62.4.048103.json
 	modified:   tests/stubdata/output/jats_spie_opten_1.OE.62.4.066101.json
 	modified:   tests/stubdata/output/jats_spie_spie_12.2663029.json
 	modified:   tests/stubdata/output/jats_spie_spie_12.2663066.json
 	modified:   tests/stubdata/output/jats_spie_spie_12.2663263.json
 	modified:   tests/stubdata/output/jats_spie_spie_12.2663387.json
 	modified:   tests/stubdata/output/jats_spie_spie_12.2663472.json
 	modified:   tests/stubdata/output/jats_spie_spie_12.2663687.json
 	modified:   tests/stubdata/output/jats_spie_spie_12.2664418.json
 	modified:   tests/stubdata/output/jats_spie_spie_12.2664959.json
 	modified:   tests/stubdata/output/jats_spie_spie_12.2665099.json
 	modified:   tests/stubdata/output/jats_spie_spie_12.2665113.json
 	modified:   tests/stubdata/output/jats_spie_spie_12.2665157.json
 	modified:   tests/stubdata/output/jats_spie_spie_12.2665696.json
 	modified:   tests/stubdata/output/jats_spie_spie_12.2690579.json
 	modified:   tests/stubdata/output/jats_springerEarly_ExA_s10686-023-09907-7.json
 	modified:   tests/stubdata/output/jats_springer_AcMSn_s10409-023-23061-x.json
 	modified:   tests/stubdata/output/jats_springer_AcMSn_s10409-023-23086-x.json
 	modified:   tests/stubdata/output/jats_springer_AcMSn_s10409-023-23108-x.json
 	modified:   tests/stubdata/output/jats_springer_EPJC_s10052-023-11699-1.json
 	modified:   tests/stubdata/output/jats_springer_EPJC_s10052-023-11733-2.json
 	modified:   tests/stubdata/output/jats_springer_JHEP_JHEP07_2023_200.json
 	modified:   tests/stubdata/output/jats_springer_NatCo_s41467-023-40272-3.json
 	modified:   tests/stubdata/output/jats_springer_Natur_s41598-023-38673-x.json
 	modified:   tests/stubdata/output/jats_springer_SoPh_s11207-023-02231-5_mathtex.json
 	modified:   tests/stubdata/output/jats_springer_ZaMP_s00033-023-02064-z.json
 	modified:   tests/stubdata/output/jats_springer_cldy_84_1543.json
 	modified:   tests/stubdata/output/jats_springer_jhep_2022_05_05.json
 	new file:   tests/stubdata/output/jats_springer_roman_num_1.json
 	modified:   tests/stubdata/output/mdpi_climate-11-00147.json
 	modified:   tests/stubdata/output/mdpi_galaxies-11-00090.json
 	modified:   tests/stubdata/output/mdpi_symmetry-15-00939.json
 	modified:   tests/stubdata/output/mdpi_universe-08-00651.json
 	modified:   tests/stubdata/output/nlm_tf_gapfd_116_38.json
 	new file:   tests/stubdata/output/nlm_tf_roman_num_1.json
 	modified:   tests/test_jats.py

commit 8d7d0954062adb46e24bd449227603b2f7a07893
Author: Matthew Templeton<EMAIL_ADDRESS>Date:   Tue Jul 9 19:00:56 2024 -0400
bugfix -- each affstring was getting all ids
 	modified:   adsingestp/parsers/jtest.py

commit f15b5e7972ccc473ae6d7a9d31db8e624acc832f
Author: Matthew Templeton<EMAIL_ADDRESS>Date:   Tue Jul 9 16:42:13 2024 -0400
Candidate replacement jats parser for external affids
 	modified:   adsingestp/parsers/base.py
 	new file:   adsingestp/parsers/jtest.py
 	new file:   adsingestp/parsers/jtestaff.py
 	new file:   adsingestp/parsers/piffol.py
 	modified:   pyproject.toml

Adds feature requested in #104


Update golang to 1.17.6

Proposed changes
This PR updates golang to 1.17.6.
Types of changes
What types of changes does your code introduce to the project: 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)
[ ] Update only documentation, not any source code.

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
[ ] I have signed the CLA
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have added necessary documentation (if appropriate)
[ ] Any dependent changes have been merged and published in downstream modules

Further comments
N/A

Coverage remained the same at 100.0% when pulling 7ee6919ea1a4c307653131f8cc6adaa4575537a9 on fisuda:hardening/golang_1_17_7 into 36d600508bd2348ecd1db138762bf38fc9a3d517 on lets-fiware:main.


panic: runtime error: slice bounds out of range

<cryptix> oh.. one of my nodes paniced. log: QmbJw2mb4Jh94EUcHhmYvnVh1tjre73gqHDLYtqrdA6yeg  src: http://gotools.org/github.com/jbenet/go-msgio#msgio-go-L189 cc @whyrusleeping @jbenet
<cryptix> not sure how that makes sense.. length > len(msg) should have caught that

links for the layz:

log output
src: http://gotools.org/github.com/jbenet/go-msgio#msgio-go-L189

The only way I can make sense of this, is if msgio returned a negative length like here:
https://play.golang.org/p/q6-mmg5_EI

I guess, there is a possibility where this cast leads to an negative int?
I'd argue to add a sanity check (next > 0) after that cast and maybe move the if length > s.max {		return nil, ErrMsgTooLarge } check into reader.nextMsgLen() as well.
I also dislike the use of the next field in general. I think, It conflates control logic and value usage. I'd make it a bool in the reader struct and use uint as the return param of the nextMsgLen func.

:+1: for uint
:+1: for next > 0 sanity check,  right before http://gotools.org/github.com/jbenet/go-msgio#msgio-go-L203
it's a private field. it's ok (we need to store it between calls, as the reader advances)


Editing article removes title and sort

When we use the built-in editor and save, we lose the comments at the top. We have to manually add them every time. Any idea where this happens so we can fix it and submit PR?

Thanks for submitting @ramezrafla
What form of comments are you using within the markdown? (I've never used that feature).
"<!---" triple-dash?
If you can provide a sample of content before/after, that would help diagnose

Hey @ryanlelek thanks for your attention to this.
We use regular JS comments. I recall this is how we need to add to the sort and description information.

@ramezrafla do using the Markdown comments fix your issue?
If so, please mark as resolved.
Otherwise, please provide more information so we can help diagnose

This never got fixed, see #192

This bug is still alive, issue shouldnt be closed.
Using Editor from browser deletes all meta data info from the header of the md file.


[WIP] Upgrade Ray to version 2.35.0

Description

RHOAIENG-12186
Upgrading Ray version to 2.35.0 to allow for AMD GPU support.
How Has This Been Tested?

Merge criteria:

[ ] The commits are squashed in a cohesive manner and have meaningful messages.
[ ] Testing instructions have been added in the PR body (for PRs involving changes that are not immediately obvious).
[ ] The developer has manually tested the changes and verified that the changes work

@Fiona-Waters Thanks. I've run some tests already to validate the existing images work with the LLM fine-tuning example, which should provide a good level of validation already. I'd suggest we do another cycle of validation when the new images in the modh organisation are available. WDYT?

@Fiona-Waters Thanks. I've run some tests already to validate the existing images work with the LLM fine-tuning example, which should provide a good level of validation already. I'd suggest we do another cycle of validation when the new images in the modh organisation are available. WDYT?

Yes that makes sense to me. Will keep this PR as WIP for now until these images are ready. Thanks


Add match_case to MD043

Fixes https://github.com/DavidAnson/markdownlint/issues/613

@DavidAnson thoughts on when this might be released? Thanks!

how to fix the default MD043 rule?

@BrentHuang Not sure what your comment here is, but there are examples of using MD043 in the repo, such as: https://github.com/DavidAnson/markdownlint/blob/main/test/required-headings-all-present.md?plain=1


Does the library support native SIP?

Does the library support native SIP over UDP or only WebRTC?

Using WebSocket + webrtc is currently the most reasonable combination. If UDP transmits signaling, you have to return the webrtc stack to normal UDP/RTP, because the SDP length of webrtc will exceed 1500, and INVITE signaling cannot be transmitted via UDP.
The technical level of your issue is feasible, which will require the development of a customized version, and the development cost is higher.

Understood, normal SIP/RTP over UDP is not supported now.
Thanks you for answer.


TSan instrumentation of module code

This implements TSan instrumentation in the IREE compiler, and adds a IREE_BYTECODE_MODULE_ENABLE_TSAN flag, which when set causes iree_bytecode_module to enable TSan instrumentation.
When IREE_BUILD_TESTS is on, we enforce early that IREE_ENABLE_TSAN and IREE_BYTECODE_MODULE_ENABLE_TSAN agree, as we would otherwise get test crashes anyway.
IREE_BYTECODE_MODULE_ENABLE_TSAN also requires IREE_BYTECODE_MODULE_FORCE_SYSTEM_DYLIB_LINKER to be set. That too is enforced early.
Putting all that together, with this PR, the incantation to build tests with TSan is:
cmake \
  -DIREE_BUILD_TESTS=ON \
  -DIREE_ENABLE_TSAN=ON \
  -DIREE_BYTECODE_MODULE_ENABLE_TSAN=ON \
  -DIREE_BYTECODE_MODULE_FORCE_SYSTEM_DYLIB_LINKER=ON \
   ...other flags...

cmake --build .
cmake --build . --target iree-test-deps

So clearly this isn't optimal in the sense of minimizing required typing to enable TSan, but at least this is optimizing "options that do one clear thing each" and "reuse existing cmake logic (IREE_BYTECODE_MODULE_FORCE_SYSTEM_DYLIB_LINKER).
This PR is ending up simpler than anticipated because

We used to have trouble with iree_bytecode_module setting the system-linker-path but that was fixed in #8591
We used to have to handle switching to the system-linker as opposed to embedded-linker, but the cmake infra for that (IREE_BYTECODE_MODULE_FORCE_SYSTEM_DYLIB_LINKER) was added in #8659 .
We used to run into lots of trouble with TSan apparently not supporting vectors with more than 16 elements, I first tackled that using LLVM Scalarizer pass but ran into a bug, then I wrote another approach using VectorUnrollPatterns (diff here), but in the end I noticed that none of this seemed to be needed anymore --- I want to believe that that's because something was changed in LLVM but I haven't been able to pinpoint it. Anyway, if we run into issues where some vector load/stores of more than 16 elements are not tsan-instrumented, we can revive that diff.

There's no relation - you pass the flag when you compile a module if you want the module to have the sanitizer enabled. Building the compiler with a sanitizer does not imply the artifacts need to be (or that we want them to be - say if you are cross compiling) and building the runtime with a sanitizer does not imply the artifacts need to be either (or can be - you may be loading a file from someone else, etc). The only requirement is that if you build an artifact with a sanitizer then you must also build the runtime like that. So: enable the sanitizer for the runtime, and pass the sanitizer flag to the compiler when you are building things you want to be sanitized.

Updates:

This is no longer using the clang driver.  All what was needed was passing -shared instead of -static.
In today's meeting we agreed that IREE_ENABLE_TSAN needs to be split into two settings, one enabling TSan in the compiler and the other enabling TSan in the runtime. Once that is done, the iree_bytecode_module logic debated above will just need to follow the runtime setting (which is where it's necessary, due to TSan breaking ABI) and that will thus become decoupled from the compiler. I will get back to this once https://reviews.llvm.org/D121182 is merged, and then, I will need to understand how exactly we implement that compiler/runtime decoupling --- based on source code directories or on individual cmake rules ?

Looks fine to me. I thought you needed vector unrolling. Dont see it here.

See the updated PR description. The problem mysteriously vanished.


bgColor='transparent' doesn't work properly on discord icon

I'm using the latest version of react-social-icons by the time of writing this.
I encountered an issue where when you use bgColor='transparent' on discord, it just makes the background copy the fgColor
Notes:

So far, I've only encountered this issue when using discord icon.

I tried using a hex and rgba with 0 opacity/transparency but its still the same as it copies the fgColor

As I said, when bgColor is set to transparent. It won't really be transparent but instead change to the color that you put in fgColor.

Also noticed this problem today. think this icon may have two backgrounds, one of which is set to white fill which can't be changed.

I noticed it aswell, when I was working on my Portfolio....

hi @john-basilio , this has been addressed in v6 of react-social-icons
CC @rjpeart @Jonak-Adipta-Kalita


Committees are now supported

Fixes committees
Description

Types of changes

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

Checklist:

[x] My code follows the code style of this project found on ESLint.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.

on the removed consol.log I meant to say I removed it from getCommittees (I'm too used to typing events)


Try waiting for ack

Signed-off-by: Miroslav Kovar<EMAIL_ADDRESS>

Codecov Report

Merging #448 (8bfa00c) into main (fb48ce1) will decrease coverage by 21.36%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##             main     #448       +/-   ##
===========================================
- Coverage   47.53%   26.16%   -21.37%
===========================================
  Files         153       19      -134
  Lines       12200      967    -11233
  Branches     2521      189     -2332
===========================================
- Hits         5799      253     -5546
+ Misses       4479      639     -3840
+ Partials     1922       75     -1847

Flag
Coverage Δ

unittests-agency-client
26.16% <ø> (ø)

unittests-aries-vcx
?

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

agency_client/src/utils/comm.rs
0.00% <0.00%> (-25.00%)
:arrow_down:

aries_vcx/src/init.rs
0.00% <0.00%> (-11.67%)
:arrow_down:

agency_client/src/httpclient.rs
0.00% <0.00%> (-9.10%)
:arrow_down:

agency_client/src/agency_client.rs
0.00% <0.00%> (-9.10%)
:arrow_down:

...vcx/src/protocols/issuance/issuer/state_machine.rs

...ocols/issuance/issuer/states/requested_received.rs

...rotocols/issuance/issuer/states/credential_sent.rs

.../src/libindy/credential_def/revocation_registry.rs

aries_vcx/src/messages/error.rs

aries_vcx/src/libindy/schema/mod.rs

... and 129 more

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 fb48ce1...8bfa00c. Read the comment docs.


XGBoost Integration

https://docs.google.com/presentation/d/1jxj9zjeRRu1BJSf8tzaWoQVr5h7HZbOUsSh39Rcwv80/edit#slide=id.g10be2c57ddf\_7\_2
Aha! Link: https://nvaiinfa.aha.io/features/MERLIN-673

Hey! I am not sure if this is the right place to post this, @bschifferer asked me to work on an example for the xgboost integration and I left some thoughts in a PR #522. Not sure on the process side of things, where things should go or if what I share there can be helpful, if I am doing something wrong or these thoughts are not adding value, please let me know 🙂Thank you!

I am not sure if I should have linked my PR here in the first place, but the three of us (@bschifferer, @oliverholworthy, and me) just had a great chat, I also now know what would be the way to capture all this feedback (via feature requests in Merlin Models), so essentially all is well 🙂


[klima data] Left nav loses focus when on a Details page

Page: all detail pages
Issue:

When I open a Details page, the left nav loses focus of the primary page
For example

@biwano are you aware of any two top-level pages linking to the same detail page?
If not then I would restructure the project files and move all detail pages under a single parent page. That would make active path handling a lot easier (change useIsPathnameActive to startsWith instead of ==).
An alternative would be to use a context hook to set the active parent page but it seems hacky and it will be complicated if a detail page can have two parents (What link should be active if we have multiple parents? The one where we came from?).

@biwano are you aware of any two top-level pages linking to the same detail page? If not then I would restructure the project files and move all detail pages under a single parent page. That would make active path handling a lot easier (change useIsPathnameActive to startsWith instead of ==).
An alternative would be to use a context hook to set the active parent page but it seems hacky and it will be complicated if a detail page can have two parents (What link should be active if we have multiple parents? The one where we came from?).

The tokenized credits by bridge is both on the detail page and the off vs on page. I like your idea though. We could duplicate the tokenized credits by bridge detail page to make it work. But I do not know if it is a good idea SEO wise.
Let me tag @Atmosfearful too.


Implements glob search feature

Implements glob search feature as requested in #154

There are three main components to in the implementation of this feature:

NavigationState

there are separate instances for normal and glob modes. This lives inside the AppState and holds the state of there the user has been navigating etc.

Depending in which mode you are you need a different "view" of the tree. This is where new TreeView trait comes in. AppState::process_events has been refactored to create appropriate tree view (normal vs glob) for the mode you are in and us that to handle the events.

Doing a glob search adds a new tree root which gets removed when the search is cancelled.

Glob has its own pane etc. where user input and rendering is handled.
UI components have been refactored to only need basic types and not reference traversal or tree at all.

re: cursor - I think the problem is due to the fact that the cursor is not drawn in the buffer, and then diffed to only draw what updated by tui-react.  Instead the cursor is rendered directly on the screen. I am not sure how to fix it as I am not too familiar with that code.

Me neither, as it should be something related to Ratatui I presume. It would probably also be possible to check there to see how to best do prompting, a topic dua managed to avoid thus far.

fyi, I have fixed the problem with the cursor.

Added long path shortening:

That's great, thanks so much for making all this happen!
Did you consider using the ellipsis character … instead of ...? That way two more character slots might be freed up.

I have just tried …  but don't like how it looks - it it so small it is easy to miss.

I have just tried …  but don't like how it looks - it it so small it is easy to miss.

For a moment I thought that letting it appear on top of the selection is on top would help, similar to what the 'message' line in the mark pane does - it follows the currently selected line to be more obvious.
In this case, it would determine its initial position based on the currently selected line, and then stay there. Even though in theory it can probably also follow the selection if the tab/pane-cycling logic can handle that.

btw. I think search history may be potentially useful future addition to the glob search feature.

It sounds complicated to do that UI wise, so I hope we can postpone this to another PR.
Besides, I'd be starting the review today in the hopes to merge tomorrow.

Review Notes

Evaluated tui-textarea for adoption, and I think it could work if dua would use native crossterm events. Alternatively, one could convert them back from the now mostly unnecessary abstraction layer to native crossterm events. That way, one wouldn't have to deal with the cursor handling as much, which isn't as trivial to get right with unicode in mind.
Unicode handling for the prompt is certainly not correct now, but at least it doesn't crash anymore when pasting/typing 你好 .

This is a work-in-progress - to be continued

@Byron Thanks Sebastian! It wouldn't be possible without the suggestions & feedback of people using it. I have used dua today to do "holiday cleaning" of my hard drive, and the tool you have created is great! I am happy to make my small contributions to help make it even better :)
This new release is going to be a great holiday gift! 🎁


feat(extendedIndexes): SQL Server Index Clustering

Adds information about SQL Server Index clustering.
Also fixes unrelated wrong term "fields" for the "attributes".

I think we should also update the Getting started guide using Prisma and SQL Server as well – for folks using the latest version.


Bug in pcontains

It appears that pcontains is not working properly.
ghci> i1
Interval {ivFrom = LowerBound (Finite (POSIXTime {getPOSIXTime = 120})) True, ivTo = UpperBound (Finite (POSIXTime {getPOSIXTime = 129})) False}
ghci> i2
Interval {ivFrom = LowerBound (Finite (POSIXTime {getPOSIXTime = 120})) True, ivTo = UpperBound (Finite (POSIXTime {getPOSIXTime = 129})) True}
ghci> contains i2 i1
True
ghci> plift $ pcontains # pconstant i2 # pconstant i1
False

As far as I can tell, it works when the upper bound for both are closed, but not when one is open. We should have the $[120, 129) \subseteq [120, 129]$, but we don't.

Note that this is particularly an issue because of this.
Also, we should probably indicate that "gotcha" in the Interval module and haddock comments.

Following that comment, if it isn't supported in Plutus, why support it in Plutarch at all?

@L-as Its support in plutus-ledger-api, so I think it does make sense for compatibility reasons.
But I still think that we should have a separate library that actually does what PLA "means" and not what it "does"

plutus-ledger-api

Fair reason, but arguably even then you should never use it. You could mark it as deprecated but still fix it.


VSCode freezes after most recent update

VSCode Version: 1.24.1
OS Version: Windows 7 Professional Build 7601, SP1 64-bit

Steps to Reproduce:

Launch VSCode
Open any folder
Open the Integrated Terminal
Attempt to type anything in Integrated Terminal
Nothing appears in Terminal at the cursor prompt
Within a second or two, entire computer seems to slow down; then a popup appears noting that VSCode has become unresponsive. Selecting Reopen does not change this issue.
Setup was working correctly recently, no major changes made to computer, no other software displays this issue

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

@ajkandy @wayspurrchen do you get this notification? I can only reproduce a big slowdown (on various operating systems) and then this notification will appear with the workaround to the problem.
One idea here is to be much more aggressive and automatically switch the user over as I've seen many reports of users ignoring the prompt.

I didn't see that notification regarding high GPU usage, but then again I wasn't looking for it. I will double-check when I'm back at the office on Monday.

I had to open Process Explorer to view the high GPU, so you didn't miss it; seems like a similar case to #36913.

@ajkandy you need to have rendered 11+ frames in the terminal before it will show, that's used to get a reasonably reliable number for how long rendering takes.

Thanks @Tyriar. I will see if the behaviour persists. Wondering if this is something due to Win7 and/or it not talking to my graphics card properly... Let me know if you need more info and what I'd need to do to get it to you.

@ajkandy thanks. On the weekend I pushed a change that allows the renderers to be swapped in and out without restarting the terminal, so pressing the notification should immediately fix the problem. The next step on my side is to automatically switch over to the fallback when it's detected as slow, this is deferred to next month though.

Duplicate https://github.com/Microsoft/vscode/issues/56628


New replace_with method does not work as expected

I noticed an issue regarding reference. The following code works:
from selectolax.parser import HTMLParser

html_parser = HTMLParser('
Get

')
img_node = html_parser.css_first('img')

html_parser2 = HTMLParser('
Test
')
img_node.replace_with(html_parser2.body.child)
assert html_parser.body.child.html == '
Get
Test

'

but the following fails:
from selectolax.parser import HTMLParser

html_parser = HTMLParser('
Get

')
img_node = html_parser.css_first('img')

img_node.replace_with(HTMLParser('
Test
').body.child)
assert html_parser.body.child.html == '
Get
Test

'

since this is python, I would expect the two to work the same and are memory safe
Originally posted by @ppwwyyxx in https://github.com/rushter/selectolax/pull/25#issuecomment-668200521

https://github.com/rushter/selectolax/pull/25#issuecomment-668206627

@lexborisov What is the best way to copy a subtree (a node and all its child nodes) from one tree to another? Are there any helpers?

In the current version, there is no way to copy nodes from one tree to another.
Trees have a different memory pool. When cloning nodes, we need to compare node->tree pointer, and if they do not match, then copy using the memory of the tree into which the insertion is made.
I can add this functionality to the myhtml:
myhtml_node_t * myhtml_node_clone(myhtml_tree_t *dest_tree, myhtml_node_t *src);
myhtml_node_t * myhtml_node_clone_deep(myhtml_tree_t *dest_tree, myhtml_node_t *src);

What do you think about it?

What do you think about it?

That would be cool. Is it safe to use two trees and insert a node from another tree?

That would be cool. Is it safe to use two trees and insert a node from another tree?

Yes, it will look something like this:
myhtml_node_t *cloned;

cloned = myhtml_node_clone(dest_node->tree, src);
if (cloned == NULL) {
    /* ... */
}

myhtml_tree_node_insert_after(dest_node, cloned);

I've published a simple fix now. We definitely want to make it more robust in the future by copying nodes.
This example does not work for some reason:
            node = myhtml_tree_node_clone(<myhtml_tree_node_t *> value.node)
            myhtml_node_insert_before(left_node, node)
            myhtml_node_delete(left_node)

I haven't debugged it, but the inserted node gets lost.
Currently, we use two trees, but this is fragile and there are so many ways to break it.

@rushter
I have added two functions, please try it.

@lexborisov Thanks! It works as expected.


[QUERY] Access message details in a workflow?

Question
How can I access the details entered by the user in my workflow?
Context
I have an external application that sends a message to a next-generation Slack App using the chat.PostMessage API. The message contains interactive components (like text fields, radio buttons, etc.), including a button.
When the user enters information and clicks the button, I would like to save the details provided by the end-user into a Datastore.
I set the button with type equal to workflow as detailed here. However, the document says this button will not send 'block_actions' events as it is launching a new workflow.
My question is, how can I access the details entered by the user in my workflow?

Environment

    "deno-slack-sdk/"<EMAIL_ADDRESS>    "deno-slack-api/"<EMAIL_ADDRESS>
deno 1.41.0 (release, aarch64-apple-darwin)
v8 12.1.285.27
typescript 5.3.3

ProductName:		macOS
ProductVersion:		14.3.1
BuildVersion:		23D60
Darwin Kernel Version 23.3.0: Wed Dec 20 21:30:44 PST 2023; root:xnu-10002.81.5~7/RELEASE_ARM64_T6000
Requirements

Hi @mambuganarayana thanks for writing in 💯
If I understand correctly your application has a workflow with a step that posts a message with a button to kicks off another workflow.
An alternative could be to use an interactive message that contains a button scoped to a single step/function. This way you will have access to the entire context of the message.
Let me know if this answered your question

Thank you, William. Let me explain my use case.
I have two applications.

Internal application. It is not a Slack application. Let's call it 'InternalApp.'
Custom Slack application containing workflows, functions, and interactions with data stores. Let's call it 'SlackApp.' It is created using Slack CLI like this. It has a bot token.

The InternalApp sends an interactive message containing components (like text fields, radio buttons, etc.), including a button. To send the message, it uses a bot token of the 'SlackApp' and calls the HTTPS endpoint chat.postMessage.
The message appears in the 'Messages' tab of the SlackApp for the end user.
The user provides information (enters data into the text field, selects radio button options), and clicks on Submit button.
At this point, I want to trigger a workflow function associated with the SlackApp and pass all the information the user provided. Is this possible? If so, how?
The function then uses the information and inserts it into a data store.

Since the bot token provided by your workflow Slack app is not permanent / static, your non-Slack app cannot utilize the token for sending a message.

The InternalApp sends an interactive message containing components (like text fields, radio buttons, etc.), including a button. To send the message, it uses a bot token of the 'SlackApp' and calls the HTTPS endpoint chat.postMessage.

If your "internal" app needs the ability to send a message at a certain timing, here are two possible options:

Create a new app at https://api.slack.com/apps just to allow your "internal" app to send a message to Slack; The message has a button with a link trigger URL to start a workflow
Create a new app at https://api.slack.com/apps and do everything with the app (= no workflow)

With the option 1., the workflow triggered by the button click first opens a modal dialog to collect user inputs and relay the data to the next step. Here is an example code: https://github.com/slack-samples/deno-code-snippets/tree/main/Block_Kit_Modals
If you go with option 2., you can do everything within a single Bolt app: https://slack.dev/bolt-js

Thank you very much.

If your "internal" app needs the ability to send a message at a certain timing,

Yes, the flow starts from our internal app.
I will try out option#1.
Regarding option#2, I am assuming we need to expose a request URL that is accessible publicly OR make use of WebSockets. Is this correct?
For security reasons, we are not allowed to expose a public URL. I have tried WebSockets for development using Bolt SDKs. It works very well. Does Slack recommend it for production as well?

Yeah, you can go with either Request URL (publicly accessible URL) or Socket Mode (WebSocket protocol).

Does Slack recommend it for production as well?

Yes, we do. You can use Socket Mode for production use cases too. When your app needs more scalibility, you can run up to 10 instances for an app.
It seems that everything is clear now. Would you mind closing this issue?

Yes, Thanks again.!


[BUG] Symbols from coc.nvim are not parsed properly

I'm using the latest symbols-outline.nvim with coc.nvim and get the following error messages when calling :SymbolsOutline:
...ols-outline.nvim/lua/symbols-outline/utils/lsp_utils.lua:45: attempt to index field 'location' (a nil value)
stack traceback:
^I...ols-outline.nvim/lua/symbols-outline/utils/lsp_utils.lua:45: in function 'get_selection_range'
^I...gged\symbols-outline.nvim/lua/symbols-outline/parser.lua:30: in function 'parse'
^I...vim\plugged\symbols-outline.nvim/lua/symbols-outline.lua:296: in function 'on_symbols'
^I...mbols-outline.nvim/lua/symbols-outline/providers/coc.lua:90: in function <...mbols-outline.nvim/lua/symbols-outline
/providers/coc.lua:89>

After some debugging, I found that it may be related to the returns of request_symbols function in coc.lua (link). The returned table cannot be resolved by the sort_symbols function in lsp_utils.lua.
Since I don't have nvim-lsp configured, I have no idea how to fix the request_symbols in coc.lua without a reference. I will be happy to create a PR if the return contents of other symbol provider are provided. Thanks.

@simrat39 can you please acknowledge this issue ? Symbols-Outline.nvim seems to be completely unusable when using CoC.nvim as the LSP client.

Facing the same problem here. Any suggestions for a quick fix?
Thanks in advance for your assistance!


[SQA]-THREAD TC-DL-2.9 Unable to execute the credential commands.

Problem:
In TC-DL-2.9 The Credential command(set,get,clear) are unable to execute in the Chip-tool as we are unable to provide Struct values for Credentials (Credential Type and Credential Index)
Excepted outcome :  Success response
Actual Outcome : Unable to execute the Command
Step to reproduce: ./chip-tool doorlock set-credential 0 1 2 xxxx 2 0 0 1 1
Platform :

Chip-Tool - RPI-4 , 8 GB RAM
DUT:- Platform : THREAD N/W - nrfconnet sdk board

Commit used: 11587f5bf0bb9e621edd4c867c0bc942d804d1ce
Reference: Test-Plan
Logs:

./chip-tool doorlock set-credential OperationType Credential CredentialData UserIndex UserStatus UserType node-id/group-id endpoint-id-ignored-for-group-commands [--paa-trust-store-path] [--commissioner-name] [--commissioner-nodeid] [--trace_file] [--trace_log] [--ble-adapter] [--timedInteractionTimeoutMs] [--suppressResponse] [--repeat-count] [--repeat-delay-ms] [--timeout]
[1652348232.057935][30233:30233] CHIP:TOO: Run command failure: ../../commands/common/Commands.cpp:137: Error 0x0000002F

The format of the command is incorrect. The Credential parameter needs to be JSON-encoded. For example:
chip-tool doorlock set-credential 0 '{"credentialType":1, "credentialIndex":1}' '5555' 1 1 0 555 1 --timedInteractionTimeoutMs 1000
See https://github.com/CHIP-Specifications/chip-test-plans/issues/1298

@Survensa Can we close this ticket ?

Cert Blocker Review: Assigning to myself and @franck-apple to find an assignee. Tagging @bzbarsky-apple as well

@Survensa Please attach (not paste) complete logs from the client (including, critically, the exact command being executed) and the server.
My guess is that this is using a credential type not supported by the server app, but the logs would tell for sure.

@bzbarsky-apple Here I have attached the log of server and client for this command .
setcerdential-cntl.txt
setcerdential-dut.txt

@jrhees-cae this test is run on nordic nrfdk board

Yep, that matches my guess from https://github.com/project-chip/connectedhomeip/issues/18418#issuecomment-1137600252
@cjandhyala @Survensa does the feature map for this device claim PIN support?

@bzbarsky-apple In lock app, Feature map value: 275
Binary value : 100010011 ( Which supports PIN,RID,SCH,NOT )

FeatureMap value for door-lock example is likely common to all platforms implemented for the example. https://github.com/project-chip/connectedhomeip/blob/master/examples/lock-app/lock-common/lock-app.zap
Therefore, if a certain platform example doesn't support the common features, it would need to override the featuremap value.

Pinging @kkasperczyk-no and @Damian-Nordic

Yes, sounds like some apps need to set the value at startup, not just use the default from the ZAP file...

@Survensa @cjandhyala can we close that one? The only issue I see are differences between feature maps vs the real features that are implemented, but it affects probably more examples. If we want to fix this, it could be addressed in a more generic issue that this one.

per @kkasperczyk-no nordic platform chose not to implement a USR feature, hence this is not a cert blocker

@cjandhyala Note that the USR feature is required if any of PIN, RID, FPG, or FACE are supported.

@bzbarsky-apple right, however nRF platform in particular doesn't implement any of those.


All Puppet AIX Automation for VIOS Update

To deliver what has been done for Puppet AIX Automation VIOS Update
Refer to README.md which is delivered in source tree.
Refer to documentation "Automate VIOS updates in NIM environment with ‘Puppet AIX and VIOS Automation’ module" v0.8 dated Oct 19th 2018 (or higher)
And refer to documentation "Automate AIX updates and patch management in NIM environment with ‘Puppet AIX and VIOS Automation’ module" v0.8 dated Oct 19th 2018 (or higher)

Want to redo the Pull Request to include documentation


Store a list of current repos which use the JSON Schema topic and when they were created #4

Resolves #4
This PR is for 'Store a list of current repos which use the JSON Schema topic and when they were created
#4'
Requirement
Here's the work that needs to be done on this:

 No longer check the way back machine for earliest use of the json-schema github topic and no longer record the data related to the way back machine
 Get the date of the first commit for a repo and store that in the CSV as a timestamp
 Confirm the initial data script can be run fully and include the created CSV as a commit
 Catch any potential error and log into a file, without causing the script to crash, and allowing it to continue if appropriate.
Stretch objectives:

![2024-03-27_03-36-48](https://github.com/json-schema-org/ecosystem/assets/47830372/35f647fe-ca4c-46d7-8c09-f36c2ed5ce87)
Write tests which [mock the Github API calls](https://mswjs.io/) and [mock the file writing](https://medium.com/nerd-for-tech/testing-in-node-js-easy-way-to-mock-filesystem-883b9f822ea4), and check it has the correct content for the API data mocked.

What I have done
This commit includes several enhancements to the initial data script:

The script now retrieves the date of the first commit for each repository and stores it in the CSV file as a timestamp.
The script has been confirmed to run fully, and the resulting CSV file is included in this commit.
Error handling has been added to the script. Any errors that occur are logged to a file, and the script is able to continue running where appropriate.

This is my second qualification task work!
second qualification task.pdf

This looks much improved. I'm scheduling some time to test it locally before it is merged. Thanks for your work on this.

[previous git commit history]

[after fixed]

[mock jest]

Hi Ben,
Thank you for your insightful feedback! I appreciate the guidance and the opportunity to learn from real-world problems provided by this project.
I've completed a git rebase and resolved the divergent commits as you've noticed. I've also re-requested a review of the PR to ensure everything aligns with the project's current needs.
Regarding the API key update, your clarification was very helpful. I understand now the importance of designing tests that don't make real API calls and will take this as a key learning point.
Thank you for your support and for providing a nurturing environment for newcomers like me. I look forward to continuing my contribution to the project and will keep in touch via Slack as needed.
Best regards,
David

Hey David, thanks for waiting on this. I spent a good part of the day yesterday trying to modify one of the tests to make it work correctly. I've changed approaches and should be updating this PR today.

Thank you, Ben. I deeply appreciated your support:)

Thanks for your work on this! Much appreciated. I made a few improvements to the testing.


Update and greatly improve the schedule code.

Codecov Report

Merging #22 into master will increase coverage by 10.83%.
The diff coverage is 81.31%.

@@            Coverage Diff             @@
##           master     #22       +/-   ##
==========================================
+ Coverage   61.66%   72.5%   +10.83%
==========================================
  Files           4       5        +1
  Lines         240     280       +40
  Branches       43      52        +9
==========================================
+ Hits          148     203       +55
+ Misses         61      48       -13
+ Partials       31      29        -2

Impacted Files
Coverage Δ

src/schedule.c
85.15% <78.68%> (+17.35%)
:arrow_up:

src/schedule_print.c
86.66% <86.66%> (ø)

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 1dcd4db...56bb098. Read the comment docs.


Update the wxwidgets key for Ubuntu Noble.

The package name has changed to libwxgtk3.2-dev in Ubuntu Noble, as well as in newer versions of Debian.
Please update the following dependency in the rosdep database.
Package name:
libwxgtk3.2-dev
Package Upstream Source:
https://www.wxwidgets.org/
Purpose of using this:
The wxwidgets key is needed by mrpt2: https://github.com/MRPT/mrpt/blob/6f1ac5b7e91e1bc49f78363c6dda933f9f6d885a/package.xml#L58
Links to Distribution Packages

Debian: https://packages.debian.org/

https://packages.debian.org/search?keywords=libwxgtk3&searchon=names&suite=all&section=all

Ubuntu: https://packages.ubuntu.com/

https://packages.ubuntu.com/search?keywords=libwxgtk3&searchon=names&suite=all&section=all

This one supersedes https://github.com/ros/rosdistro/pull/40227 , so I'm closing that one. Thanks.

This one supersedes #40227 , so I'm closing that one. Thanks.

Very sorry, I missed that you already had this open.  Oh well, it should be fixed either way :).

No problem! Fixing it, it will end up being cool... there is a cascade of packages failing because of that right now.

I'm going to go ahead with this one with just one review, as since I am also a rosdistro reviewer I count as a self-review.

Thanks!

@clalancette The build farm seems not to catch up with the new keys for u24.04, the latest build still fails trying to install the Debian package libwxgtk3.0-gtk3-dev.
Are keys latched at bloom-release time and I should make a new release?

Are keys latched at bloom-release time and I should make a new release?

Yes, exactly.  You'll have to do a new bloom-release.


Replace GenEvent with gen_event

Minimal Fix for Issue #59
The Erlang error_logger interface's add_report_handler function requires a gen_event interface. Elixir has deprecated GenEvent, and for use cases as these recommends using the gen_event behaviour. This change does that.
Confirmed that tests passed.

It's maybe worth noting that the previous init/2 function was actually never getting invoked. The default implementation of init/1 as defined by GenEvent was getting called (and returning {:ok, []}).

Thanks for handling this @shantiii! Changes look good to me. I think we should cut a new release soon, I'm planning to update and merge #49 this week so probably once that's done.

Awesome, thanks for the speedy turnaround on this @grantovich !


Enable IP level encryption

To enable V2Ray for iOS and Android, we need IP level encryption, as Socks protocol is not supported natively in these platforms.

iOS可以参考surge

I don't think this is reasonable. On mobile platforms, V2Ray will still support TCP and UDP only. Packets with other types will pass through as is.


Napari

Motivation for this change
Adds napari, a scientific image viewer.
Please note, it's my first PR to nixpkgs, I may not be aware of some rules or conventions
Things done

Dependencies also packaged in separate commits but in the same PR - should they be merged separately?

One of the dependencies - cachey - is part of dask suite; should I add @FRidh to maintainers, as in other dask packages?
qt-related packages (like superqt, magicgui) segfault during tests, although the final app works

I also don't know if some special steps need to be taken in regards to nixpkgs-update integration, cc @ryantm
Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.05 Release Notes (or backporting 21.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/672

@SomeoneSerge I quickly looked over your changes and I think automated updates from r-ryantm/nixpkgs-update should work with them.

@SomeoneSerge I quickly looked over your changes and I think automated updates from r-ryantm/nixpkgs-update should work with them.


JPG to JPEG Converter App

Requirements for the app:-

The app should convert jpg to jpeg format
User should be able to save the image

Requirements for the README.md file:-

Screenshots
Concepts used
Documentation and resources used
Future Aspects of the app

Note:- Add the app to the Android Projects section

I would like to work on this 👍

@FaMoSi Sure, go for it.

@FaMoSi are you done with this?


error log don't write to custom log file

error log don't be written to custom log file
the custom log:

the develop log:

can written the error info to custom log file?

It should be able to be written into a custom log file, yes. You can give the middleware a logger than is configured to write to a new/different file (meaning: it doesn't have to be the Rails.logger). That should do what you're looking for ...
Can you show your Faraday connection build to see how you're initializing the middleware?

Oh, I just realized that in the log you listed, you were receiving an SSL failure. That's totally different.
So, it looks like the custom log was working as expected. It contains the request INFO and DEBUG details. However, the server you were connecting to had a bad SSL certificate. Faraday (more specifically, actually, the Adapter Faraday is using) throws errors for failed SSL. Those errors are (correctly) being caught by your Rails application.
You see no response in the custom log, because no connection was successfully made and no response came back. The net-layer stopped the request at the point of the handshake failure.

I've created and pushed v2.1.0. This should at least log out the error class name, message, and first backtrace line to your custom log file.


title The style directive

page https://learn.svelte.dev/tutorial/styles
Tutorial says do this
style="transform: {flipped ? 'rotateY(0)' : ''};
When answer is actually this
style:transform={flipped ? 'rotateY(0)' : ''}
Hope that helps someone

The directions say to do the first thing (along with the rest of the style attributes), and then they talk about the style: directive. The "solve" button shows the result of following all of the directions.


null-aware operation Warnings

I just upgraded to latest version and I am suddenly getting the warnings below
../../../../.pub-cache/hosted/pub.dartlang.org/giphy_get-3.0.4/lib/src/views/tab/giphy_tab_detail.dart:238:22: Warning: Operand of null-aware operation '!' has type 'List' which excludes null.

'List' is from 'dart:core'.
'GiphyGif' is from 'package:giphy_get/src/client/models/gif.dart' ('../../../../.pub-cache/hosted/pub.dartlang.org/giphy_get-3.0.4/lib/src/client/models/gif.dart').
if (_collection!.data!.isNotEmpty && mounted) {
^
../../../../.pub-cache/hosted/pub.dartlang.org/giphy_get-3.0.4/lib/src/views/tab/giphy_tab_detail.dart:240:35: Warning: Operand of null-aware operation '!' has type 'List' which excludes null.
'List' is from 'dart:core'.
'GiphyGif' is from 'package:giphy_get/src/client/models/gif.dart' ('../../../../.pub-cache/hosted/pub.dartlang.org/giphy_get-3.0.4/lib/src/client/models/gif.dart').
_list.addAll(_collection!.data!);

I got the same issue.

Same issue here...

WIP

Can you post your flutter doctor please @rlee1990 @mashegoindustries

Raygun found:
Message: Null check operator used on a null value
_GiphyTabDetailState._loadMore in package:giphy_get/src/views/tab/giphy_tab_detail.dart:213
_GiphyTabDetailState._scrollListener in package:giphy_get/src/views/tab/giphy_tab_detail.dart:251

@bazookon was this fixed with the newest update?

sorry no.
_AssertionError._doThrowNew in dart:core-patch/errors_patch.dart:51
_AssertionError._throwNew in dart:core-patch/errors_patch.dart:40
ScrollController.position in package:flutter/src/widgets/scroll_controller.dart:108
_DraggableScrollableSheetScrollController.position in package:flutter/src/widgets/draggable_scrollable_sheet.dart:725
_GiphyTabDetailState._scrollListener in package:giphy_get/src/views/tab/giphy_tab_detail.dart:685
ChangeNotifier.notifyListeners in package:flutter/src/foundation/change_notifier.dart:308
ChangeNotifier.notifyListeners in package:flutter/src/foundation/change_notifier.dart:308
════════ Exception caught by foundation library ════════════════════════════════
ScrollController attached to multiple scroll views.
'package:flutter/src/widgets/scroll_controller.dart':
package:flutter/…/widgets/scroll_controller.dart:1
Failed assertion: line 108 pos 12: '_positions.length == 1'
════════════════════════════════════════════════════════════════════════════════

_GiphyTabDetailState._scrollListener in package:giphy_get/src/views/tab/giphy_tab_detail.dart:250
correcting with 'positions.last' ..... if ((widget.scrollController.positions.last.extentAfter

@pburel thank


Add Replicate demo

Hi @yl4579 , I accidentally discard the changes and closed the previous pull request. The issue is now fixed and I updated the Replicate demo as well!

@alaradirik awesome work with the Replicate demo.  Any idea as to how to add training to the Replicate imlementation?

@platform-kit sorry for the late reply! I'd be happy to add a train endpoint to the Replicate demo

@alaradirik Amazing, thank you so much, and happy new year!

HI @alaradirik. Thank you for your demo. I have a discussion/question that I was hoping I could get your help with #183.
I've been using replicate to train my model on audio with 50 diffusion steps and it still sounds like the robotic default voice. Is my reference_file being recognized? Any help would be appreciated.
output = replicate.run(
model_id,
input={
"beta": 0.3,
"reference_file": "training_audio.WAV", # also tried: "training_audio.mp4" "training_audio.mp3"
"seed": 2,
"text": text_to_synthesize,
"alpha": 0.3,
"diffusion_steps": 50,
"embedding_scale": 5,
}
)
print(output)

Hi @alaradirik ! I know you're busy with Replicate stuff -- have you had any time to dig into adding the training script?
I'm super excited to work with this lib on Replicate. If a donation would help you find some time for this, let me know how much you need. Thanks!