LocalSend fails to launch with terminal message "xkbcommon: ERROR: failed to add default include path /usr/share/X11/xkb"

This is an upstream snap issue, tracked here: https://github.com/ubuntu/gnome-sdk/issues/232.
It is seemingly fixed, but the update seemingly has not been released yet.
This also seems to be Wayland only according to: https://bugs.launchpad.net/ubuntu/+source/chromium-browser/+bug/2073339
Workarounds
Check for updates in App Center or from the terminal with sudo snap refresh. An update fixing the issue may be out by the time you read this.
You can also try to revert back to an older working version of the gnome 46 runtime by running snap revert gnome-46-2404. This may not work if you never previously installed the gnome 46 runtime.
If that doesn't work, the Xorg session should work.

Hi, the fix, which I confirmed in Chromium, should be soon enough in gnome-46-2404 stable channel, but it is already in candidate channel so an alternative is
snap refresh --candidate gnome-46-2404

Fix was released in stable channel.


fix reconnection error when call client.end()

When the client is waiting for reconnecting, and then call client.end(), it will throw a error.

This is already fixed => closed


Modified the Travis configuration file to enable automated FOSSA scans.

Hello, I'm from FOSSA and I'm working with the Uber OSPO to automate license scanning. In addition to the changes in this PR, we may also need to add an API key as an environmental variable in the build environment.
I noticed that you defined "scripts" inside the build matrix and didn't run any scripts afterward. I added two additional entries to enable FOSSA: a before_scripts entry and a scripts entry. Does that work for your build environment? I'm totally open to suggestions if what I did doesn't make sense.
Thanks!

I noticed that you defined "scripts" inside the build matrix and didn't run any scripts afterward. I added two additional entries to enable FOSSA: a before_scripts entry and a scripts entry. Does that work for your build environment? I'm totally open to suggestions if what I did doesn't make sense.

For the matrix entries that don't have scripts defined, they should be using the default script for Maven Java projects: https://docs.travis-ci.com/user/languages/java/#maven-default-script-command mvn test -B
It looks like Travis didn't build this PR, possibly due to some issue with whether uber/h3-java was actually active on travis-ci.com. It doesn't look like Travis is giving me an option to trigger a build of the PR, could you try retriggering the build?

Hi Isaac, I don't believe I have access to the Travis environment for this project. I can resubmit this PR if that helps. Let me know what you'd like me to do.
One note, the build may not be successful without adding the FOSSA API key to the build environment. You should have received an email from the Uber OSPO with instructions to add the API key. I can provide additional instructions if the original was unclear. Thanks!

Hi Isaac, I don't believe I have access to the Travis environment for this project. I can resubmit this PR if that helps. Let me know what you'd like me to do.

Yes, please add a commit to the PR or resubmit it.

One note, the build may not be successful without adding the FOSSA API key to the build environment. You should have received an email from the Uber OSPO with instructions to add the API key. I can provide additional instructions if the original was unclear. Thanks!

This is added for uber/h3 and uber/h3-java now.

Pull Request Test Coverage Report for Build 163

0 of 0   changed or added relevant lines in 0 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.3%) to 99.461%

Files with Coverage Reduction
New Missed Lines
%

src/main/java/com/uber/h3core/H3Core.java
1
99.28%

Totals

Change from base Build 152:
-0.3%

Covered Lines:
369

Relevant Lines:
371

💛  - Coveralls

Going to continue the integration work in #52.


[SonataFlow] Deploy Workflows locally via operator

Besides deploying on a remote cluster, users could optionally deploy workflows locally via KIND or Minikube. In this scenario, users could connect the Web UI to a local cluster and have the workflows deployed there.

The application should identify if the operator is installed; if not, it should install the operator.

Serverless Web Tools won't be integrated with the SonataFlow at this time, we can rethink this in the future. Closing for now.


Update minimum required version to Xcode 12

Devicelab and CI migration to Xcode 12 is complete as of https://github.com/flutter/flutter/issues/73392.

Fixes https://github.com/flutter/flutter/issues/76900
Blocked by Xcode 12 adoption in plugins CI https://github.com/flutter/plugins/pull/3653
Unblocks re-landing https://github.com/flutter/flutter/pull/73458 for iOS simulator ARM support.
Previous Xcode 11 bump at https://github.com/flutter/flutter/pull/50315.

LGTM
This is quite the chain of PRs you have here :)

It's a long chain.
https://github.com/flutter/plugins/pull/3663
https://github.com/flutter/plugins/pull/3664
https://github.com/CocoaPods/CocoaPods/pull/10477

https://github.com/flutter/plugins/pull/3653 is merged.


Cross-Domain errors

Add a warning when we are in HTTPs domains if signaling or ssh server are in HTTP. Vice versa.

Change url to HTTPs or HTTP when the problem occures.


Support to change the accessibility of a model and change a model to be struct via customized code

Fixes #4259
Fixes #4346

API change check
API changes are not detected in this pull request.


Showcase section

In a similar vein as our #showcase Discord channel, it'd be great to have a showcase section of the website.
It doesn't need to be interactive, we can curate interesting showcase uploads in Discord and share them there.
But it'd be nice if this had a unique design and set of components to show the image, note the author, show the Ghostty config, filter by platform, etc.
It could live initially in the docs section so we don't clutter the top nav.

@DeadEnglish I'm assigning this one to you (just so nobody else accidentally steps on your toes here).  You're good to run with this!

But for this you would also have to create a login page? so you can take user details. Or else anyone can anonymously post. Also a create showcase post page?


more edits

There was an error while validating job ede6a298-fd51-43d6-a8bb-9238e049508a Message:
Result status: Failed
There was a problem accessing Github repository 'https://github.com/Microsoft/cordova-docs.git'.
Details:Microsoft.LightweightJobEngine.Core.Language.ContextCommandBase+ContextCommandException:
Result status: Failed
There was a problem accessing Github repository 'https://github.com/Microsoft/cordova-docs.git'.
---> System.AggregateException: One or more errors occurred. ---> System.ArgumentException: Error clonning repository ---> LibGit2Sharp.NotFoundException: Branch 338 not found. Available branches: origin/master
at Microsoft.DPS.Services.LibGit2SharpSourceControlProvider.GetBranch(Repository repo, String name)
at Microsoft.DPS.Services.LibGit2SharpSourceControlProvider.<>c__DisplayClass30.<>c__DisplayClass34.b__29(KeyValuePair2 branch)    at System.Collections.Generic.List1.ForEach(Action1 action)    at Microsoft.DPS.Services.LibGit2SharpSourceControlProvider.<>c__DisplayClass30.<CloneRepository>b__28()    at Microsoft.Practices.EnterpriseLibrary.TransientFaultHandling.RetryPolicy.<>c__DisplayClass1.<ExecuteAction>b__0()    at Microsoft.Practices.EnterpriseLibrary.TransientFaultHandling.RetryPolicy.ExecuteAction[TResult](Func1 func)
at Microsoft.Practices.EnterpriseLibrary.TransientFaultHandling.RetryPolicy.ExecuteAction(Action action)
at Microsoft.DPS.Services.LibGit2SharpSourceControlProvider.CloneRepository(String url, IDictionary2 branches)    --- End of inner exception stack trace ---    at Microsoft.DPS.Services.LibGit2SharpSourceControlProvider.CloneRepository(String url, IDictionary2 branches)
at Microsoft.DPS.Services.LibGit2SharpSourceControlProvider.CloneRepository()
at Microsoft.DPS.Services.LibGit2SharpSourceControlProvider.GetVersionAsync(String path)
at Microsoft.DPS.Commands.IdentifyDocumentsToPublish.CheckExternalServicesCommand.Do(IDictionary2 context)    --- End of inner exception stack trace ---    --- End of inner exception stack trace ---    at Microsoft.LightweightJobEngine.Core.Language.ContextCommandBase.ThrowIfErrorsLogged()    at Microsoft.DPS.Commands.IdentifyDocumentsToPublish.CheckExternalServicesCommand.Do(IDictionary2 context)
at Microsoft.LightweightJobEngine.Core.Language.JobWorkflow1.RunCommands(Guid jobId, IDictionary2 context, CancellationTokenSource cancelSignal)


React Native CLI Installation Link Unclear

Installing React Native CLI as a prerequisite to contributing is simple, but finding the instructions to do so is not. The link on https://github.com/facebook/react-native/wiki/How-to-Contribute is still several jumps short of providing clear instructions, and searching online leads to other sites with different instructions.
Since the code to install it is so short, may I suggest adding a simple line or two of code onto the How to Contribute page describing how to install it using npm or yarn?
npm install -g react-native-cli
yarn global add react-native-cli
Another possibility is to link directly to the installation instructions at https://github.com/react-native-community/cli/tree/master/packages/global-cli or at https://facebook.github.io/react-native/docs/getting-started.html[https://facebook.github.io/react-native/docs/getting-started.html].

Hey @hawthord thanks again for the feedback!
Have you see our new guide for Submitting Your First PR. This is designed to provide more specific instructions with explanations of each step
I'd support your updates as well if you want to submit a PR and tag me!

Also, please continue to give us this feedback, it is super useful!
Feel free to comment on our umbrella issue for the website/docs rewrite as well https://github.com/facebook/react-native-website/issues/929


fix: infinite loop with jsonschema>3.2.0

Describe the bug
Starting proTES fails with jsonschema versions > 3.2.0. This is due to the tesNextTes schema in pro_tes/api/additional_logs.yaml referencing itself, causing an infinite loop during validation or reference resolution (I'm not sure which).
To Reproduce
Steps to reproduce the behavior (in local python virtual environment):

pip install -r requirements.txt
pip install jsonschema==4.22.0
cd pro_tes
python3 app.py

Result
Server gets stuck here:

Expected behavior
Server should continue with setup process.
Workarounds

Use jsonschema version 3.2.0 (pip install jsonschema==3.2.0) OR
Comment out forwarded_to property in tesNextTes OR
Use oneOf:

    forwarded_to:
       oneOf:
           - $ref: '#/components/schemas/tesNextTes'

Possible solutions

Specify jsonschema version in requirements.txt
Alter additional_logs.yaml to avoid infinite loop

Nice catch, thanks a lot!
So we do need the forwarded_to property in order to keep track of the service that the TES requests ends up on eventually. And we also need to support multiple services in the chain, in case more than one gateway is involved.
But probably we could get away with a non-recursive solution, and luckily we can easily change that, given that it's not part of the TES specs but something we added ourselves. My suggestion would be to make tesNextTes an array of (a new model) tesForward with properties id, url and - to be on the safe side in case we cannot rely on ordering - a forward_time (use the same format as described in this comment. But remove the recursive forwarded_to reference.
This solution should work fine for use cases where forwarding is strictly linear. This assumption may not be warranted for all use cases, but as far as I know, we hadn't yet had a reason to forward an incoming task to multiple services (and anyway proTES doesn't currently support that).


Upgrade react-placeholder to support react 18?

Are there any plans to support react 18?

I'm looking forward to it, too

Any more information about when the support for React 18 comes available?

A pr is open for this issue:
https://github.com/buildo/react-placeholder/pull/133


Revert "Merge pull request #10965 from skateman/actioncable"

This reverts commit 81b8c1528c53712f604eef8577f6d6e450d20882, reversing
changes made to 9c23a2a6109d6bcef07351ed619634fb28cff8e6.
The idea is to revert #10965 until the issues (https://github.com/ManageIQ/manageiq/issues/11098, https://github.com/ManageIQ/manageiq/issues/11095, https://github.com/ManageIQ/manageiq/issues/11092) are resolved..
Cc @martinpovolny, @skateman

Might be solved easier, give me 10min ;)


Call not defined methods in a controller, how?

I need to use the __call in a controller to call not defined methods.
Probably use is_callable in Sanity checks (CodeIgniter.php) resolve but:
https://github.com/bcit-ci/CodeIgniter/blob/develop/system/core/CodeIgniter.php#L423
Even now  you can not use is_callable?
@narfbg 522c73623 your comment refers to php 5.2 issues [?]

You "need" to use __call().
I don't know what else to say about this ... this issue shouldn't have been created in the first place.

Thank you for your reply.
Sorry but my question was only "if" it's possible (now 2016) to fix your workaround (2012) and then to permit  call not defined methods with __call().
thanks


MPS Optimizations

Some basic thought on re-factoring so we can abstract away our CPU / GPU infrastructure?
Build an interface for conforming input pixel buffers to whatever best case infrastructure we have, be it GPU (MPS) or CPU (OpenCV / vImage)

SynopsisFrameConformSession - A long duration session object which replaces our Analyzers VideoTransformScaleLinearizeHelper object - providing some sort of session object so we can create MTL command buffers and pipeline operations, and or alternatively dispatch to operations on the CPU if we need to to prep optimized / cached representations of our input frame.

SynopsisFrameCache - An opaque object that can vend optimized MTL or OpenCV / vImage representations of our input image appropriate for a specific analyzer.

SynopsisFrame  - An opaque object wraps a which can be backed by a MTLBuffer or vImage / OpenCV object that holds a specific frame representation (BGR, BGR32, Gray, Perceptual etc).

Analyzers now consume a SynopsisFrame object and request a particular format?

This also sets a precedent for API parity for Audio if we ever get there


Autogen Studio GroupChatWorkflow not working on pyautogen==0.2.13 but (working on pyautogen=0.2.10)

Describe the bug
broken old feature.
working on pyautogen=0.2.10
Steps to reproduce
Seems fails  on function def _prepare_and_select_agents
        is_last_speaker_in_group = last_speaker in self.agents

        # this condition means last_speaker is a sink in the graph, then no agents are eligible
        if last_speaker not in self.allowed_speaker_transitions_dict and is_last_speaker_in_group:
            raise NoEligibleSpeakerException(
                f"Last speaker {last_speaker.name} is not in the allowed_speaker_transitions_dict."
            )```

### Expected Behavior

using autogenstudio ui
Sample Workflow: Travel Agent Group Chat Workflow
Chatl: Plan two day trip to Delhi

on pyautogen          0.2.13:
<No response>

on pyautogen          0.2.10 -- which is working fine
Based on the suggestions from the local_assistant and language_assistant, here is the final two-day trip plan to Delhi:
Day 1:
1. Start your day by visiting the iconic Red Fort. Explore the historical site and learn about its significance in Indian history.
2. Afterward, head to Jama Masjid, one of the largest mosques in India. Take a moment to admire the stunning architecture and peaceful atmosphere.
3. Next, visit Raj Ghat, the memorial site of Mahatma Gandhi. Pay your respects and learn about the life and teachings of the Father

### Screenshots and logs

_No response_

### Additional Information

_No response_

HI @akhilesh-chander ,
Thanks for the note.
What version of autogenstudio do you have?

It doesn't work on the latest autogenstudio, I also had the issue.

This is what I see when running the travel workflow which is based on group chat

Is this related to #1580 or a different error?
Note current version of workflowmanager

Yup same, note the fix is probably something like this if you want to add the sender to the receiver group:
`        # given the config, return an AutoGen agent object
self.sender = self.load(config.sender, config.session_id)
    # given the config, return an AutoGen agent object
    if config.receiver.type == "groupchat":
        config.receiver.groupchat_config.agents.append(config.sender)
    self.receiver = self.load(config.receiver, config.session_id)

`

What is your version of autogenstudio?
What do you see when you run pip show autogenstudio.
As far as I know, the issue in #1580  (GroupChat not initiating) is fixed in v0.0.44a.
@sidhujag Can you confirm if you are still seeing that issue in  v0.0.44a?

Yup it works

What is your version of autogenstudio? i.e what version do you see when you run pip show autogenstudio. As far as I know, the issue in #1580 (GroupChat not initiating) is fixed in v0.0.44a. @sidhujag Can you confirm if you are still seeing that issue in  v0.0.44a?

Yup it looks better just testing i see some new bugs but not sure if its my fork of changes or official so I will play some more and report.

@victordibia is this related to this issue: https://github.com/microsoft/autogen/issues/1423

groupchat append a new agent, but not rebuild it's allowed_speaker_transitions_dict. so it alwas raise error in groupchat.
https://github.com/microsoft/autogen/blob/09a491892151b2adef9025d741b6e7c26f877fcf/samples/apps/autogen-studio/autogenstudio/workflowmanager.py#L41
https://github.com/microsoft/autogen/blob/09a491892151b2adef9025d741b6e7c26f877fcf/autogen/agentchat/groupchat.py#L346

@hurrytospring , please use the dev branch - https://github.com/microsoft/autogen/tree/autogenstudio

@hurrytospring , please use the dev branch - https://github.com/microsoft/autogen/tree/autogenstudio
https://github.com/microsoft/autogen/blob/autogenstudio/samples/apps/autogen-studio/autogenstudio/workflowmanager.py
Does the issue still apply?

it did work,thanks

is resolved!


No anchors generated for

When building my jekyll site no anchors are being created for h1 headings. I was expecting anchors to be generated in the same way as for h2, h3 etc... Is this expected?
h1

h2

https://github.com/rosieyohannan/circleci-docs/tree/server/tranfer-content

Yes, this is the expected behavior. By default, the doctitle is added by the page layout. So if you find the page layout, you should see where the
 is emitted. (Something like
{{ page.title }}
). You can add the HTML necessary to produce an anchor there.
(If you have the showtitle attribute enabled, disable it. You want the page layout to emit the
 element, not the Asciidoctor processor).

Do you know which template from that repository is being used?

Oh I see! Thanks, yes it's this one I think: https://github.com/rosieyohannan/circleci-docs/blob/server/tranfer-content/jekyll/_layouts/classic-docs.html ... So I guess I need to add the anchor in here: https://github.com/rosieyohannan/circleci-docs/blob/6eb15a5d2d9b9f9646905b18d984eb78f2d85f38/jekyll/_layouts/classic-docs.html#L26 ... I'll give it a try! :-)

Oh no, this line: https://github.com/rosieyohannan/circleci-docs/blob/6eb15a5d2d9b9f9646905b18d984eb78f2d85f38/jekyll/_layouts/classic-docs.html#L28

Sorted, just added:
 to get cross refs to work to page titles in Jekyll and PDF. Thanks for your help again @mojavelinux !

You bet!
If you feel this is resolved, please go ahead and close it.


Zip file bad .blend file

In the zip download, this is what is in the "Blender Bootcamp Course.blend" file that is 1kb
"version https://git-lfs.github.com/spec/v1
oid sha256:4065baea6ad6638e47494cdfc094fa93792df984463b45541bb1166a741c6711
size 659542979"
It supposed to be a blend file that is 644,085 kb
If you clone the repo it works, however your assuming users have git knowledge.

Hi @razor247
Thank you for bringing this to our attention. We have updated the readme file with download instructions.
Happy learning!


Conditional inheritence

Conditional inheritence is not yet supported
As long as the values are constant strings, we should be able to detect it:
{% extends isMobile ? "mobile.html.twig" : "layout.html.twig" %}

Or
{% if isMobile %}
  {% set template = "mobile.html.twig" %]
{% else %}
  {% set template = "layout.html.twig" %}
{% endif %}

{% extends template %}

Or
{% extends "layout" (isMobile ? "_mobile) ~ ".html.twig" %}

the second case is a case of dynamic inheritance (covered in #6), even though a full static analysis could convert this to purely conditional inheritance (but this might be a chicken and egg problem in the current architecture). The third case could be detected as purely conditional inheritance with more complex logic as well (detecting that we can only produce 2 values), but this might not be worth it.
For the case of purely conditional inheritance (i.e. when the name of the various candidates for parent templates are not dynamic but only the condition to choose between them), a solution to support that would be to flatten the template multiple times (once per parent), adding some special analysis for the inheritance condition itself (so that it is also analyzed, enforcing that the expression returns a string).


Can not modify account setting

When I click the login name and click the account setting, I find that the settings can't be modified, it's read only.

Fixed.


Error: ENOENT: no such file or directory, scandir './rules'

Package.json
  "eslintConfig": {
    "extends": "plugin:wordpress/recommended",
    "plugins": [
      "wordpress"
    ]
  },
  "eslintIgnore": [
    "js/**.min.js"
  ],

When I run the eslint js/** on _s I get the following error.
Error: ENOENT: no such file or directory, scandir './rules'
    at Error (native)
    at Object.fs.readdirSync (fs.js:951:18)
    at module.exports (/usr/local/lib/node_modules/eslint-plugin-wordpress/node_modules/requireindex/index.js:18:20)
    at Object.<anonymous> (/usr/local/lib/node_modules/eslint-plugin-wordpress/index.js:20:24)
    at Module._compile (module.js:541:32)
    at Object.Module._extensions..js (module.js:550:10)
    at Module.load (module.js:458:32)
    at tryModuleLoad (module.js:417:12)
    at Function.Module._load (module.js:409:3)
    at Module.require (module.js:468:17)

Similar issue https://github.com/eslint/generator-eslint/pull/20

@ntwb I figured out the issue. The issue is that the version on NPM has not been updated. https://www.npmjs.com/package/eslint-plugin-wordpress
I ended up installing it via npm i -g https://github.com/WordPress-Coding-Standards/eslint-plugin-wordpress

If I understand this correctly eslint-config-wordpress is what I should be using and eslint-plugin-wordpress is where it the rules are managed.
Would that mean the Readme file should be updated to recommend people to use eslint-config-wordpress?

If I understand this correctly eslint-config-wordpress is what I should be using and eslint-plugin-wordpress is where it the rules are managed.

Yes, you should really only use eslint-config-wordpress and as I just noted on the _s's PR I'll be releasing a new version of eslint-config-wordpress in the next couple of days with some pretty big changes, waiting a couple of days before refactoring any code to match these rules would be my recommendation :)

Would that mean the Readme file should be updated to recommend people to use eslint-config-wordpress?

Probably, yes :)

I also just checked that the proposal for _s in https://github.com/Automattic/_s/pull/1159 uses eslint-config-wordpress 👍

@ntwb Thanks, I did start looking at that but I got confused by the two repos as eslint-config-wordpress did not have all of the rules.
Are there different rulesets that we can use or does the WordPress ruleset include everything?
For example space-unary-ops is defined differently in stylistic-issues.js and jscs.js. Which one does get applied?
Or consistent-this in stylistic-issues.js. Why does it not create a warning for self = self.parentElement; in navigation.js#L72

These are the updates that are coming in the next releases...
Up until (and currently including now) eslint-config-wordpress has been a standalone _shared config, with the release of 3.0.0ofeslint-config-wordpressit will extendeslint-plug-wordpress` which will then bring some parity to the both repos.
See also https://github.com/WordPress-Coding-Standards/eslint-config-wordpress/issues/8

Me just experiencing same issue & noticed that it still v0.1.0 on npm?

I ended up installing it via npm i -g https://github.com/WordPress-Coding-Standards/eslint-plugin-wordpress

Is that's the only way to solve this issue? and when the latest version will available on npm?
Thanks

@feryardiant Unless you have got a specific requirement to use this package, you should use https://github.com/WordPress-Coding-Standards/eslint-config-wordpress instead of this package here.

@ntwb

Unless you have got a specific requirement to use this package

What's that mean? Is that mean you wanna say that "We don't have plan to publish latest version on npm for some reason" ??
Actually actually it's my first time trying to use all wordpress coding standards instead of just wpcs package but also eslint-config-wordress and stylelint-config-wordress. But I noticed some inconsistency and odd standards on the eslint-config-wordress package, somehow. For instance:

It's ask me to write
(function( document ) {
	// some codes
}( document ));

instead of
(function( document ) {
	// some codes
})( document );

Note: I'm not going to linting full project just yet.
Anyway, could you please explain further about the "specific requirement" means?

Sure, @feryardiant, eslint-plugin-wordpress is currently undergoing lots of frequent changes and at the moment it also includes some bugs. There are no plans to publish any updates of this package to npmjs.com until some new coding standards changes are implemented upstream from WordPress Core and Gutenberg that fixes the bugs. Then we will look to releasing an update of eslint-plugin-wordpress to npmjs.com.
This is why I recommended using eslint-config-wordpress, seeing as though you have an issue with the wrap-iife rule you can turn that rule off in your local config. Also would you mind creating an issue at https://github.com/WordPress-Coding-Standards/eslint-config-wordpress/issues/new so we can investigate further if we should turn this rule off by default please.

I see, thanks for the explanation @ntwb. Sure thing, will do.


Shader Exception when pressed Start Normal Editor - latest versions of blender and addon, Use Alternative Drawing doesn't fix.

Hi, I'm getting this error message when I press Start Normal Editor

I'm on Windows 10, using Blender 4.0.1, with addon release version 1.2.
Done the full reinstall procedure, and enabled Use Alternative Drawing option in addon (because I saw that suggestion in other tickets).
I've also installed blender onto a second drive from my main (default) one if that's at all relevant.
Thank you.

If you download the master version this should be fixed.

Yep, that did it. Thanks for the quick response.


Fix deletedneed #2577

requires #2586
Changes:

stores Failed uri retrieval within the process-state (either need or connections)
Refactors all Process relevant states and move it from connections/needs to the process-reducer
Adapts the connectionHeader to show if a need was not being able to be loaded from the server
-- adds a button to close the connection directly in the conn-list (if the remoteNeed was not retrievable)
Adapts the postHeader to show if a need was not being able to be loaded from the server
Adapts square-image to indicate if the post given in the uri was unable to be loaded from the server -> this comes in handy if we have a groupchat and a member deletes their connected need

Fixes #2577
I think the scope of this PR is reached with the refactoring and given info for deleted (Remote-)Needs, if there is anything else that needs to be checked for failedToLoad then we might adapt this in the future (especially within post-content, post-content-message, post-content-info or post-content-general, as well as suggested-post-picker/viewer)
Please notice the changes made to the state by looking into the won-owner-webapp/README.md

@quasarchimaere Please resolve conflicts

@fkleedorfer will do first thing tomorrow

currently blocked trying to fix issue #2561 within this PR

closing this until further notice


1.16.4 Problem (Bug)

The newest version of your mod: vanillaexperience-1.16.3-0.2.5 doesn't work in 1.16.4 and makes the launcher crash.
Here's the log: https://pastebin.com/QvXxHFjG

This is not caused by my mod. mixins.mm.json is from modmuss50 mods, probably from OptiFabric.


kafka: record extractor for produce requests in mesh-filter

Commit Message: kafka: record extractor for produce requests in mesh-filter
Additional Description: to get to the Kafka messages (records) we need to do more processing of Kafka payloads (we need to read record_set field from protocol according to format. For now we are going to only support modern (what means Kafka 1.0 from 2017) format with magic = 2 (see line 4).
Risk Level: Low
Testing: Unit tests
Docs Changes: N/A
Release Notes: N/A
Platform Specific Features: N/A

Even if it goes green I still need to pull out some constants.


Service to extend all loans

extend all item (that can be extended) with a single click at all libraries
extend all item (that can be extended) with a single click at specific library

if loan details attribute 'extend_loan_id' is empty, the item can NOT be extended

supported since R1.0.0


Vue.js virtualizer doesn't state update when items count equal to one.

Describe the bug
Hello!
I ran into a problem when using the Virtualizer with TanStack Query on Vue.js, when the number of elements is one, the state of the Virtualizer is not updated.
After looking at the source code of the adapter, I decided that the problem might be an incorrect state update when tracking changes in the Virtualizaer options.
Perhaps the solution to the problem would be to move the call to triggerRef behind the callBack function onChange.
Your minimal, reproducible example
https://codesandbox.io/p/sandbox/tanstack-virtual-vue-k6vou4
Steps to reproduce
Pass in count vue.js virtualizer options Ref value equal to one
Expected behavior
Displaying a list when the number of elements is one
How often does this bug happen?
Every time
Screenshots or Videos
No response
Platform
Windows 10, chrome 109.0.5414.75
tanstack-virtual version
v3.0.0-beta.40
TypeScript version
No response
Additional context
No response
Terms & Code of Conduct

[X] I agree to follow this project's Code of Conduct
[X] I understand that if my bug cannot be reliable reproduced in a debuggable environment, it will probably not be fixed and this issue may even be closed.

I see that @PerpetualWar fixed this in 3.0.0-beta.46. Thanks!


git: option not to update series on tags

Currently, when a new tag is sent on the mailing list, the series is updated in the Git repository.
The push is done with '-o ci.skip', but this seems to be something specific to GitLab. It has no effects with GitHub for example. Not to waste resources on retesting everything and sending notifications once a tag is shared, a new per-project option has been added:
update_series_on_tags

Not to change the current behaviour, series will continue to be updated on new tags by default, except if this option is explicitly disabled.

Thank you for maintaining this project, and for managing patches for MPTCP.
This modification is mainly for the MPTCP project, because it uses GitHub. New tags trigger CI executions, and some machines are CPU time limited. Maybe other projects have similar limitations.
Note: I was not able to properly test the modification, sorry for that. The modification is quite small and limited to one specific area, is it OK like that?

Looks reasonable, just had two small comments.

Thank you for the review!

Apart from that, is there no way on GitHub to push without triggering CI?

Yes, there are, but only by modifying the commit message. Both GitHub Actions and Cirrus CI will skip the build if the HEAD commit contains [skip ci]. But they don't use push options like GitLab:

https://docs.github.com/en/actions/managing-workflow-runs/skipping-workflow-runs
https://cirrus-ci.org/guide/writing-tasks/#conditional-task-execution

Thanks, then I think this solution is okay. It's not ideal to inject [skip ci] in the commit message anyway.
The patch looks good to me now!

Thanks, then I think this solution is okay. It's not ideal to inject [skip ci] in the commit message anyway.

Indeed. We could add it under a new --- line, so git am would drop it. But I guess that will not please people using the commits directly.

The patch looks good to me now!

Thank you!
Please note that I was still not able to test the modification. I will check if we can modify the MPTCP importer (but I don't control it)

For testing, maybe consider adding a case in https://github.com/patchew-project/patchew/blob/master/tests/test_tags.py ?

I looked at adding tests in test_git.py, but I had quite a few errors:
test_apply (tests.test_git.GitTest.test_apply) ... ERROR
test_apply_with_base (tests.test_git.GitTest.test_apply_with_base) ... ERROR
test_apply_with_base_and_brackets (tests.test_git.GitTest.test_apply_with_base_and_brackets) ... ERROR
test_git_push_options (tests.test_git.GitTest.test_git_push_options) ... ok
test_need_apply (tests.test_git.GitTest.test_need_apply) ... ok
test_need_apply_incomplete (tests.test_git.GitTest.test_need_apply_incomplete) ... ok
test_need_apply_multiple (tests.test_git.GitTest.test_need_apply_multiple) ... ok
test_rest_apply_failure (tests.test_git.GitTest.test_rest_apply_failure) ... ok
test_rest_apply_success (tests.test_git.GitTest.test_rest_apply_success) ... FAIL
test_rest_git_base (tests.test_git.GitTest.test_rest_git_base) ... ERROR
test_rest_need_apply (tests.test_git.GitTest.test_rest_need_apply) ... ok
test_rest_unapplied (tests.test_git.GitTest.test_rest_unapplied) ... ok
test_rest_unapplied_target_repo (tests.test_git.GitTest.test_rest_unapplied_target_repo) ... ok
test_result_data_automatic_url (tests.test_git.GitTest.test_result_data_automatic_url) ... ok

I used the same commands as the ones used by GitHub Actions, but with Python 3.11:
python3 -m venv ./venv
. venv/bin/activate
pip install -r requirements.txt
python manage.py test -v 2

I don't know if these tests are supposed to fail.
But when looking closer now, I notice test_git_push_options is passing, and I need something similar. I can check that.

@famz thank you for the suggestion. I just added a new test validating the new feature. I don't know well the environment, I guess it is possible to do better. But I checked with and without the option, and I have the expected behaviour.

For testing, maybe consider adding a case in https://github.com/patchew-project/patchew/blob/master/tests/test_tags.py ?

Thank you! I looked at adding tests in test_git.py, but I had quite a few errors:

I just took a bit of time to understand the issue: it was due to a git config option:
git config --global push.gpgsign true

Without that, tests are OK, I should have checked the tests before, sorry for the noise :)


Implement fabric:open_revs/3

fabric:open_revs/3 is the batched counterpart to fabric:open_revs/4.
Instead of fetching revisions for only one document, like fabric:open_revs/4,
it can fetch batches of document revisions. The main optimization is achieved
by grouping documents by shard ranges and then spawning shard workers only once
for each range (essentially doing what fabric:update_docs/2,3 does).
The idea is to use this implementation for a more efficient version of _bulk_get.
So far this is just the fabric API, not plugged into the HTTP API yet.
The backend API for this is already available as implemented in
https://github.com/apache/couchdb/pull/4185.
While the implementation is intended to be as close as possible to
fabric:open_revs/4 there a key difference -- since we're now dealing with
potentially large batches, we choose not to perform immediate read repair.
Instead, we let the internal replicator handle that. Read repair can spawn an
unbounded number of processes, and now with batches those could be large
fabric:update_docs/2,3 requests. Issuing lots of those, especially when a
cluster has a hard time keeping up, might not be a good idea.
For now we leave the fabric:open_revs/4 implementation intact, as opposed to
re-wiring it to call fabric:open_revs/3 with a one-element list. This helps
have a test reference point (see fabric_open_revs_test for an example), allows
for a configurable fallback for _bulk_get implementation, and lets individual doc
GET fetches work exactly as they have been.
A few details about the implementation itself:

The coordinator state is kept in the #st{} record. Each requested doc ID
and revisions list argument keeps track of its quorum logic and gathers
responses in its individual #req{} record. There is exactly one of those for
each requested {{Id, Revs}, DocOpts} argument.

Since the {{Id, Revs}, DocOpts} arguments are not guaranteed to be unique,
we have to keep track of them so we can return the results in the same order.
To help with that we tag each with a unique reference, and keep track of it in
{atts_since, ...} revisions, it helps to stash away the arguments separately
and just operating with reference tags. Those references are usually called
ArgRef and ArgsRefs in the code.

The #st.workers map keeps track of the spawned workers and which args
they are responsible for. As each worker returns, or is knocked out by an
error, it gets removed from the #st.workers map and at the same time all the
individual requests (#req{} record) are updated. After the update we check if
read quorum has been met and if it's still possible to succeed with the
remaining workers.

There is a 100% test coverage. The tests in the fabric_open_revs_test
module are more end-to-end and they exercise various options. The tests in
fabric_open_revs test how quorum logic and error handling works by emulating
various workers returning and how revisions are merged together into the final
result.

Issue: https://github.com/apache/couchdb/issues/4183

I wonder if there's a minor bug in the coverage plugin because when I look at e.g. .eunit/fabric_open_revs_test.COVER.html I don't actually see any code that's not covered by tests.

I noticed that too. The coverage was reported as 100% at some point. But then I did a minor refactoring and it switched 99% even though it still didn't show any red "missed" lines in the output. I suspect it's rounding error somewhere.


Fix wrong NoWatcher error code and possible session disconnected in watcher remove

Both are discovered in #22. See commit message for detail.

Codecov Report
Attention: Patch coverage is 90.90909% with 1 lines in your changes are missing coverage. Please review.

Project coverage is 83.73%. Comparing base (60fef3f) to head (bcd3a94).

Files
Patch %
Lines

src/session/depot.rs
87.50%
1 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##           master      #24      +/-   ##
==========================================
+ Coverage   83.68%   83.73%   +0.04%
==========================================
  Files          30       30
  Lines        3966     3971       +5
==========================================
+ Hits         3319     3325       +6
+ Misses        647      646       -1

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


args order

Hello,
I am unsure exactly why this was the case on my system (windows), but the lsp did not work out of the box for me. I was getting
Error: Could not find or load main class .application=org.eclipse.jdt.ls.core.id1
Eventually I was able to get it working by trying a tweak to the command that I found from an nvim jdtls user who had the same problem - moving all of the -D arguments to after the java -jar call. My command is now something like:
"command": [
       "${java_executable}",
       "--add-modules=ALL-SYSTEM",
       "--add-opens",
       "java.base/java.util=ALL-UNNAMED",
       "--add-opens",
       "java.base/java.lang=ALL-UNNAMED",
       "-noverify",
       "-XX:+UseParallelGC",
       "-XX:GCTimeRatio=4",
       "-XX:AdaptiveSizePolicyWeight=90",
       "-Dsun.zip.disableMemoryMapping=true",
       "-Xmx1G",
       "-Xms100m",
       "-jar",
       "${serverdir}/plugins/org.eclipse.equinox.launcher_${launcher_version}.jar",
       "-configuration",
       "${serverdir}/config_${jdtls_platform}",
       "-Declipse.application=org.eclipse.jdt.ls.core.id1",
       "-Dosgi.bundles.defaultStartLevel=4",
       "-Declipse.product=org.eclipse.jdt.ls.core.product",
       "-Dfile.encoding=UTF-8",
       "-DwatchParentProcess=${watch_parent_process}",
       "-data",
       "${datadir}"
   ]

Still not sure why my computer can't handle -D args coming first when clearly others can, but I wanted to suggest perhaps rearranging them in the default command in case anyone else could have the same problem solved this way.

There's still this define here:
       "-Dsun.zip.disableMemoryMapping=true",

why does this one not matter?


Passthrough non-transformed strings in compressWhitespace

After
goos: darwin
goarch: arm64
pkg: github.com/corazawaf/coraza/v3/transformations
BenchmarkCompressWhitespace
BenchmarkCompressWhitespace/#00
BenchmarkCompressWhitespace/#00-10         	571122726	         2.125 ns/op
BenchmarkCompressWhitespace/test
BenchmarkCompressWhitespace/test-10        	248496358	         4.828 ns/op
BenchmarkCompressWhitespace/test_case
BenchmarkCompressWhitespace/test_case-10   	29280343	        40.49 ns/op
BenchmarkCompressWhitespace/test____case
BenchmarkCompressWhitespace/test____case-10         	25941014	        46.05 ns/op
BenchmarkCompressWhitespace/_test__c__as___
BenchmarkCompressWhitespace/_test__c__as___-10      	17068617	        70.48 ns/op
PASS

Before
goos: darwin
goarch: arm64
pkg: github.com/corazawaf/coraza/v3/transformations
BenchmarkCompressWhitespace
BenchmarkCompressWhitespace/#00
BenchmarkCompressWhitespace/#00-10         	447815151	         2.590 ns/op
BenchmarkCompressWhitespace/test
BenchmarkCompressWhitespace/test-10        	32739363	        36.20 ns/op
BenchmarkCompressWhitespace/test_case
BenchmarkCompressWhitespace/test_case-10   	16433900	        72.80 ns/op
BenchmarkCompressWhitespace/test____case
BenchmarkCompressWhitespace/test____case-10         	14380442	        83.88 ns/op
BenchmarkCompressWhitespace/_test__c__as___
BenchmarkCompressWhitespace/_test__c__as___-10      	12626217	        96.18 ns/op
PASS

Codecov Report
Base: 78.28% // Head: 78.32% // Increases project coverage by +0.03% :tada:

Coverage data is based on head (5a1b234) compared to base (f507750).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##           v3/dev     #544      +/-   ##
==========================================
+ Coverage   78.28%   78.32%   +0.03%
==========================================
  Files         145      145
  Lines        6884     6896      +12
==========================================
+ Hits         5389     5401      +12
  Misses       1263     1263
  Partials      232      232

Flag
Coverage Δ

default
73.52% <100.00%> (+0.04%)
:arrow_up:

examples
27.13% <0.00%> (-0.05%)
:arrow_down:

ftw
56.37% <100.00%> (+0.07%)
:arrow_up:

tinygo
71.59% <100.00%> (+0.05%)
:arrow_up:

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

Impacted Files
Coverage Δ

internal/strings/strings.go
100.00% <100.00%> (ø)

transformations/compress_whitespace.go
100.00% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Error run JS : �� 㤠���� ���� 㪠����� 䠩�.

I installed the node, but the code does not start, shows this error, �� 㤠���� ���� 㪠����� 䠩�.

Same thing. Help Please.


Create Heroku app and pipeline to aid UI reviews

Email alert frontend needs a Heroku app representing master and a pipeline for reviewing changes to UI and components.

This would need the Rails.env.production? requirement to be removed too:
https://github.com/alphagov/email-alert-frontend/blob/db9996c128539031dd7c1a8f97d98098ba51b389/config/routes.rb#L2-L4


V5 - Cannot convert undefined or null to object (in combineReducers.js)

Hi there! Thanks for the awesome work with redux-form!
I'm having the above error when I upgraded from 4.1.10 to 5.2.5.. Then tried also 5.0.1, same error.
I'm using react 15.0.2, react-redux 4.4.5, redux 3.5.0

This is a usage question, and we encourage you to ask it on Stack Overflow instead. We try to use the issue tracker for bug reports and feature requests, as well as some discussions, but this appears to be a very localized question.
 Could you attach the relevant part of your code?

My root reducer:
import { combineReducers } from 'redux';
import addClientModal from './addClientModal';
import caseManagement from './caseManagement';
import client from './client';
import clients from './clients';
import countries from './countries';
import flash from './flash';
import form from './form';
import helpBox from './helpBox';
import lifeQuoter from './lifeQuoter';
import loading from './loading';
import modal from './modal';
import policies from './policies';
import policy from './policy';
import routes from './routes';
import states from './states';
import user from './user';

export default combineReducers({
  addClientModal
  caseManagement,
  client,
  clients,
  countries,
  flash,
  form,
  helpBox,
  lifeQuoter,
  loading,
  modal,
  policies,
  policy,
  routes,
  states,
  user
});

Here is an usage example for one of the reducers (caseManagement)
import typeToReducer from 'type-to-reducer';
import requirementsUpdateReceived from './requirementsUpdateReceived';
import zendeskUpdateReceived from './zendeskUpdateReceived';
import createExternalComment from './createExternalComment';

const initialState = {
  requirements: [
    { 'category': 'forms', 'baseForms': [], 'supplementalForms': [] }
  ],
  parties: [],
  conversation: [],
  filters: {
    conversation: []
  }
};

export default typeToReducer({
  'FETCH_CLIENT': () => initialState,
  'FETCH_CASE': {
    'FULFILLED': (state, action) => {
      return {
        ...action.payload,
        filters: state.filters
      };
    }
  }
}, initialState);

Here is also the full error trace:
combineReducers.js:101 Uncaught TypeError: Cannot convert undefined or null to
objectnormalizeFields @ normalizeFields.js:83runNormalize @ reducer.js:318(anonymous function) @
reducer.js:325(anonymous function) @ mapValues.js:15mapValues @ mapValues.js:12(anonymous
function) @ reducer.js:313(anonymous function) @ combineReducers.js:49assertReducerSanity @
combineReducers.js:47combineReducers @ combineReducers.js:91(anonymous function) @
index.js:75(anonymous function) @ bundle.js:1780__webpack_require__ @ bundle.js:556fn @
bundle.js:87(anonymous function) @ configureStore.js:8(anonymous function) @
bundle.js:5686__webpack_require__ @ bundle.js:556fn @ bundle.js:87(anonymous function) @
app.jsx:13(anonymous function) @ bundle.js:4462__webpack_require__ @ bundle.js:556fn @
bundle.js:87(anonymous function) @ multi_main:2(anonymous function) @
bundle.js:586__webpack_require__ @ bundle.js:556(anonymous function) @
bundle.js:579(anonymous function) @ bundle.js:582

I understand one (or more) of the reducers are passing undefined, but I haven't been able to find which changes have broken it.

Here is configureStore.js:
import reducer from '../reducers';
import thunk from '../middleware/thunk';
import sessionMiddleware from '../middleware/session';
import errorMiddleware from '../middleware/error';
import reduxPromiseMiddleware from 'redux-promise-middleware';

import {
  applyMiddleware,
  compose,
  createStore
} from 'redux';

export default function configureStore(debug = false) {
  let createStoreWithMiddleware;

  const middleware = applyMiddleware(
    sessionMiddleware,
    thunk,
    errorMiddleware,
    reduxPromiseMiddleware()
  );

  if (debug) {
    createStoreWithMiddleware = compose(
      middleware,
      window.devToolsExtension ? window.devToolsExtension() : f => f
    );
  } else {
    createStoreWithMiddleware = compose(middleware);
  }

  const store = createStoreWithMiddleware(createStore)(reducer);

  if (module.hot) {
    module.hot.accept('../reducers', () => {
      const nextRootReducer = require('../reducers/index');

      store.replaceReducer(nextRootReducer);
    });
  }

  return store;
}

Thanks, any clues would be appreciated! 😄

Well, I'll try to guess!

Where is Redux Form's reducer? I guess it is exported by ./form. Is it true?
The reducer should be imported using:
import {reducer as form} from "redux-form"
//                 ^^^^

It looks like you're using the legacy createStore API. A new one was introduce on<EMAIL_ADDRESS>There are migration details on the release page.

Thank you @ooflorent. It ended up being that one of the reducers returned undefined (that was so before the upgrade but was somehow working). Removing it fixed the error. Thanks also for pointing out the deprecated API. 👍


Improve user messaging, logging and error/exception handling

The implementation of logging, exceptions, and datalad results rendering in this extension is inconsistent. This stems from my unfamiliarity with these concepts as I started developing. The code base could benefit a lot from an experienced datalad developer going through the code to identify places where the current implementation needs changes.
The modules where this would be most applicable are:

catalog.py
meta_item.py
webcatalog.py
utils.py
extractors/datacite_gin.py

I'm noting a few general TODOs and antipatterns that need to be worked on.

f-strings: Logging and result messages should not be be done with f-strings - instead, we can hand down the template and the arguments as two separate arguments. This increases speed as the string formatting is decided upon by the logger/result renderer and only made when necessary. Instead of

msg = f"here is something of importance for {var}"
yield dict(
    ... ,
    message = mgs,
    )

do
yield dict(
    ... ,
    message = (
         "Here is something of importance for %s",
         var
    )
)

action names: Many of the currently existing result records are populated with action names by hand, such as "catalog validate". We should unify action names and make them comply to the design docs (no whitespace, underscores between words) http://docs.datalad.org/en/latest/design/result_records.html


The withSpark helper function always specifies a master URL

The withSpark helper function has a parameter master which defaults to local[*]. When submitting a JAR to run in a spark cluster, it is common to use a spark session which has no master configured. There does not seem to be a way to use the existing withSpark helper function without configuring a master URL.
I am very new to the spark ecosystem, if there is a better way to do what I'm trying to do, please point me in the right direction. My workaround at present is to add my own withSpark function which takes a pre-configured SparkSession:
fun withSpark(session: SparkSession, func: KSparkSession.() -> Unit) {
  session.apply {
    KSparkSession(session).apply {
      func()
    }
  }.also { it.stop() }
}

fun main() {
  withSpark(SparkSession.builder().appName("Kotlin Spark Application").orCreate) {
    // Spark code
  }
}

Hi @wfhartford, thank you for report! Could you please verify that specifying --master yarn in spark-submit won't override master configured in application?

Also, could you please try specifing withSpark(master = "yarn"){ in your applcation? I know, this function is not fully documented in README and I'll fix it

Specifying withSpark(master = "yarn") caused the driver pod to wait for a long time repeating the following message:
20/08/07 15:26:13 INFO Client: Retrying connect to server: <IP_ADDRESS>/<IP_ADDRESS>:8032. Already tried 0 time(s); retry policy is RetryUpToMaximumCountWithFixedSleep(maxRetries=10, sleepTime=1000 MILLISECONDS)

This message was encountered by another user of the spark-on-k8s operator who resolved it by removing the master option: https://github.com/GoogleCloudPlatform/spark-on-k8s-operator/issues/992
I have not been able to try via spark-submit. I'm not sure how to interact with my k8s operator via an external spark-submit command. That is usually left up to the operator to invoke spark-submit when I create a SparkApplication resource via the Kubernetes API.

That's interesting! So is it something you meet only with k8s operator?

I don't think that omitting the master option is specific to the k8s operator, I was under the impression it was quite common. I'm really not familiar enough with spark to speak with any kind of authority on that though. I could probably specify the master as k8s://https://<IP_ADDRESS>:443, but that is very specific to the cluster I'm working on at the moment. It seems to me that if omitting it does the right thing, that's probably best. Behind the scenes, the k8s operator creates a spark.properties file which specifies that as the master URL.

Thank you for the request, I think that we can implement withSpark accepting builder fairly easy.

@wfhartford please, try version 0.3.2

Fixed by https://github.com/JetBrains/kotlin-spark-api/commit/034a0a8d3135a39479b5ae2cc3cb1a02f9097303

Thank you, that's a great addition.

Does it work for you?

Yes, that's a great fix. Thank you.


Respond Error

How To Fix..
Error [ERR_UNHANDLED_ERROR]: Unhandled error. ('Status code: 429')
Please, Help Me.

check the video , make sure the video isn't private , age limit , and blocked

How to fix

The video doesn't violate what you say. I just want to request a song to mp3

You seem to get throttled by the YouTube API / Website: https://developer.mozilla.org/en-US/docs/Web/HTTP/Status/429

@tobilg #SOLVED Thanks Bro.


make sure our SPA packages use service workers correctly

Make sure we use service workers, and we use it correctly
https://serviceworke.rs

The service workers seem to cache everything except the gedis api calls. I am still trying to figure out the problem and doing some research.

Depends on this https://github.com/threefoldtech/jumpscaleX_threebot/issues/168

service workers support in alerta fixed here https://github.com/threefoldtech/jumpscaleX_threebot/commit/523920b643b852df4c2273f548ca7a05534006b2

service workers support in myjobs fixed here https://github.com/threefoldtech/jumpscaleX_threebot/commit/0f4bdedd95e3cd62f1288ce810d191440e4d5477

service workers support in pastebin fixed here https://github.com/threefoldtech/jumpscaleX_threebot/commit/915cc90d2b4f791988122d96201e2daaff380e7d

I removed blog from the service workers because we are facing more problems in its core functionality testing and will keep this issue from being verified

alerta,myjobs and  pastebin are verified


Some configurations (and GUI)

this feature is required to bump up to v1.0.0

malilib might be looks good for this mod  ??

maybe, It would be nice to support modmenu?

closed by #18


CPU Error : There is no runtime available right now

I'm trying to use CPU. but I can't use it.
I've never used CPU this week. How can I handle this issue?

Thanks in advance.

We are adding more capacity... it should be better now.

It works now, thanks

I am getting the same error again nowadays. Will you increase the capacity again?
There is no runtime available right now. Please change the compute type or try again later.

We are addressing this asap. Thank you for your patience.


Fequent errors in CI since 0.27.1

Describe the bug
Before I get started, apologies in advance. This issue has been inconsistent and has not yet been reproduced outside of our CI.
This is the problem in question:
$ yarn vitest run --reporter junit --outputFile "$CI_PROJECT_DIR/vitest-report.xml"
yarn run v1.22.18
$ /usr/src/app/node_modules/.bin/vitest run --reporter junit --outputFile /builds/awareways/aw20/vitest-report.xml
JUNIT report written to /builds/awareways/aw20/vitest-report.xml
FATAL ERROR: v8::ToLocalChecked Empty MaybeLocal.
 1: 0xb09980 node::Abort() [/usr/local/bin/node]
 2: 0xa1c235 node::FatalError(char const*, char const*) [/usr/local/bin/node]
 3: 0xcf758a v8::Utils::ReportApiFailure(char const*, char const*) [/usr/local/bin/node]
 4: 0xb04cf9 node::fs::FSReqPromise<node::AliasedBufferBase<double, v8::Float64Array, void> >::Resolve(v8::Local<v8::Value>) [/usr/local/bin/node]
 5: 0xb14a39 node::fs::AfterInteger(uv_fs_s*) [/usr/local/bin/node]
 6: 0x155ebed  [/usr/local/bin/node]
 7: 0x15633c6  [/usr/local/bin/node]
 8: 0x1575af4  [/usr/local/bin/node]
 9: 0x1563d18 uv_run [/usr/local/bin/node]
10: 0xa43dd5 node::SpinEventLoop(node::Environment*) [/usr/local/bin/node]
11: 0xbd3262 node::worker::Worker::Run() [/usr/local/bin/node]
12: 0xbd39f8  [/usr/local/bin/node]
13: 0x7f4715e76ea7  [/lib/x86_64-linux-gnu/libpthread.so.0]
14: 0x7f4715da6def clone [/lib/x86_64-linux-gnu/libc.so.6]
Aborted (core dumped)
error Command failed with exit code 134.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

The project in question is a Vue 3 SPA, hosted privately on GitLab, uses a "Group runner" from our company and dependencies are managed with a Renovate bot. Because of Renovate, all our dependency versions are pinned in the package.json (no ~ or ^ anywhere). It also makes it very clear and easy to see that as soon as we upgraded from "vitest": "0.27.0" to "vitest": "0.27.1" we started observing this error. The behavior has not changed with the upgrade to "vitest": "0.27.2".
The error is inconsistent. We have only one job in the CI pipeline that uses vitest, and that job fails about 60% of the time. The other 40% of the time, nothing out of the ordinary happens. The Docker image used to run the job is cypress/base:16.14.2-slim

Here is a sample log of a job with no error
$ yarn vitest run --reporter junit --outputFile "$CI_PROJECT_DIR/vitest-report.xml"
yarn run v1.22.18
$ /usr/src/app/node_modules/.bin/vitest run --reporter junit --outputFile /builds/awareways/aw20/vitest-report.xml
JUNIT report written to /builds/awareways/aw20/vitest-report.xml
Done in 16.15s.

I'm motivated to help solve this issue because vitest is by far the best unit testing experience I've had for an SPA. I don't want to move away from vitest or revert to 0.27.0 and never upgrade again because of this error. Please let me know how I can help in getting this issue resolved!
Reproduction
I need help with this. I have only ever seen this in our CI runner, and don't know what parts of the project and setup might or might not affect the reproduction.
I can dump a basic Vue 3 + TS + Vitest reproduction, but if that will create the error is unclear.
System Info
This is from our CI runner:
  System:
    OS: Linux 5.15 Debian GNU/Linux 11 (bullseye) 11 (bullseye)
    CPU: (4) x64 DO-Regular
    Memory: 4.34 GB / 7.76 GB
    Container: Yes
    Shell: 5.1.4 - /bin/bash
  Binaries:
    Node: 16.14.2 - /usr/local/bin/node
    Yarn: 1.22.18 - /usr/local/bin/yarn
    npm: 8.5.0 - /usr/local/bin/npm
  npmPackages:
    @vitejs/plugin-vue: 4.0.0 => 4.0.0
    vite: 4.0.4 => 4.0.4
    vitest: 0.27.2 => 0.27.2

Used Package Manager
yarn
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

Not sure how I didn't think of doing this before, but I have now reproduced it locally by using Node 16.14.2!! It seems like upgrading to e.g. Node 16.18.1 has fixed it locally, going to see if the same is true for our CI :)

Yep, upgrading to Node 16.18.1 did the trick! Sorry for the issue, turns out I could indeed figure this out by myself :)


How to use this library in agular 2/4

Thanks for this lib.
How can I use it in angular 2

There is no support for Angular 2 at the moment. It would be a new library, rewritten almost from scratch, and since I'm not using Angular 2, I won't do it.

@pranavkeke we now have a angular version: https://github.com/angular-slider/ng5-slider/tree/master 🎉


Add an option for applying potion effect on entities inside for the full duration

Entities that are inside a potion cauldron should be applied the contained potion effect for the full duration, using up a cauldron level.

i still can't decide whether to implement this or not. i'll close this issue for now but might reopen this later.

reopening this issue because i decided to implement this

done
pr for 1.20(.1): #10
pr for 1.20.2: #11


Changing signing module to make sure there is replay protection

New Issue Checklist

[x] I made sure the issue observed is not already reported by searching on Github under https://github.com/livepeer/livepeerjs/issues
[x] I have filled out all relevant sections of this issue template

Expected Behavior

There should be replay protection for signed transactions so that they cannot be replayed on different networks once they are submitted on one network.
Current Behavior

Seems like ethjs-signer is used as the package for signing transactions in the SDK, but according to its README it currently DOES NOT have replay protection so it should not be used in a production setting.
Possible Solution

Another candidate package to use for signing transactions is ethereumjs-tx which does have replay protection.

Good catch, @yondonfu. I can work on switching that over very soon. Luckily, the explorer relies on MetaMask to submit transactions, so it's not a vulnerability for anything we have deployed currently 👍


Hide Actions tab in Loot Sheet

Do not display actions tab in the loot sheet:

Should this be handled in this module or on the other one?
https://github.com/jopeek/fvtt-loot-sheet-npc-5e

Ah this is something the other module can (hopefully) opt out of:
If the blockActionsTab option in the Application is true, the actions tab will not auto-inject.
app.options.blockActionsTab = true

Not an issue after I updated both and removed actions injection from NPC sheets


wasm_encoder: how to lift code module function?

I'm playing around with generating byte code with the new Component Model and while I understand that it's the bleeding edge and changing a lot, I just can't get this one piece with the CanonicalFunctionSection to work and was wondering what I'm doing wrong (which is more likely than the code being wrong).
I tried to generate the simplest possible component I could think of (largely borrowed from https://github.com/WebAssembly/component-model/blob/main/design/mvp/Explainer.md):
(component
    (core module (;0;)
        (type (;0;) (func (result i32)))
        (func (;0;) (type 0) (result i32)
        i32.const 6
        i32.const 6
        i32.mul
        )
        (export "main" (func 0))
    )
    (type (;0;) (func (result s32)))
    (core instance $main (;0;) (instantiate 0))
    (func $main (;0;) (type 0) (canon lift (core func $main) string-encoding=utf8))
    (export (;1;) "main" (func $main))
)

Which almost works. It's so close! All that's needed is to change (core func $main) to (core func $main "main"). But I don't understand how to make this work with the current API - this is the relevant piece of code (which generates the line in question):
let mut functions = CanonicalFunctionSection::new();
functions.lift(0, 0, [CanonicalOption::UTF8]);
component.section(&functions);

I don't understand how I can use the API to reference and lift the $main instance's "main" function here.
I'd be grateful for any pointers! I'm sure I'm missing something.

(if it's helpful, I posted the rest of the component model-specific Rust code here)

Oh (core func $main "main") is text-syntax sugar for an (alias outer ...) item. So for example this input:
(component
    (core module $m
      (func (export "main") (result i32)
        i32.const 6
        i32.const 6
        i32.mul)
    )
    (core instance $main (instantiate $m))
    (func $main (result s32) (canon lift (core func $main "main")))
    (export "main" (func $main))
)

which uses the (core func $main "main") sugar will "elaborate to" (which you can see with wasm-tools print):
(component
  (core module $m (;0;)
    (type (;0;) (func (result i32)))
    (func (;0;) (type 0) (result i32)
      i32.const 6
      i32.const 6
      i32.mul
    )
    (export "main" (func 0))
  )
  (core instance $main (;0;) (instantiate $m))
  (type (;0;) (func (result s32)))
  (alias core export $main "main" (core func (;0;)))
  (func $main (;0;) (type 0) (canon lift (core func 0)))
  (export (;1;) "main" (func $main))
)

note the (alias core export $main "main" (core func (;0;))) which popped up. The wasm-encoder crate should then have everything necessary to encode aliases as well.


MessageTrackerTest.testMessageTrackAndApplyMultiThreaded fails abruptly on cloudbees

Either the test is racy or the MessageTracker itself has bug. I have observed this failing once on cloudbees

This might also be due to that fact the operations return before retirement messages are applied. This might result in duplicate message accounting error prone

Fixed by replacing with platform message tracker


[Docs] Document how to become collaborator on the Arrow project

Describe the enhancement requested
With the migration to GitHub we have started using the collaborator role which allows us to give triage role to some users.
I was myself a collaborator before being a committer. This was done in order to be able to label issues, assign them, etcetera.
The role is documented here from the INFRA wiki: https://cwiki.apache.org/confluence/display/INFRA/Git+-+.asf.yaml+features#Git.asf.yamlfeatures-AssigningexternalcollaboratorswiththetriageroleonGitHub
We should document how are we giving triage role so there's a clear way of giving the role to new users.
Component(s)
Documentation

We have been adding collaborators and the process for being added as a collaborator is not clear. The role is useful in order to be able to triage for labeling and assigning issues.
At the moment the process is that someone proposes new collaborators on a PR and this usually get approved and merged by a committer. I would propose adding something like the following, not sure if we should do that here: https://github.com/apache/arrow-site/blob/main/committers.md as this is only for the apache/arrow repo:
### Becoming a collaborator

The collaborator role allows users to be given triage role to be able to help with triaging issues. The role allows users to label and assign issues.
A user can ask to become a collaborator or can be proposed by another community member when they have been collaborating in the project for a period of time. Collaborations can be but are not limited to: creating Pull requests, answering questions, creating issues, reviewing PRs, etcetera.
In order to propose someone as a collaborator you must create a PR adding the user to the collaborators list on `.asf.yaml`. Committers can review the past collaborations for the user and approve.

Collaborators that have been inactive for a period of time can be removed from the list of collaborators.

@kou do you think this (or something like this) should be proposed to be added to the arrow-site repo or is there somewhere else you think this would fit better?

I think that https://github.com/apache/arrow-site/blob/main/committers.md isn't suitable for collabrators because it focuses on "committer".
How about adding this to https://arrow.apache.org/docs/dev/developers/reviewing.html#labelling ? Labelling is one of important permissions for collaborators.


MH-12091, Per-Tenant Capture Agent Users

This patch re-implements the feature of per-tenant capture agent users
which can be used with HTTP Digest authentication.
It provides a configuration file in which users can be defined and be
assigned to organizations. Additional roles can be applied as well.
This patch also allows these users access to the asset manager so that
files ingested by a capture agent can be archived as part of the
launched workflow
The commits include pull request #610 which is required for this to work properly.
Work sponsored by SWITCH

@lkiesow This PR has conflicts!

Fixed merge conflict

This appears to work, however registering a pyCA gives me an offline CA.  I'm guessing, given the pyCA log, that pyCA is settings itself to offline ([pyca.utils:149:register_ca()] pyca set to offline), but it's unclear why it would be doing that.  If this a pyCA bug, or something wonky with our endpoint?

Yep, it is. I can confirm that. It has no real consequences though. pyCA will record anyway and will automatically set itself back to idle later. It's just a timing issue at startup where the status is reported while pyCA is still starting up.
pyCA% ./start.sh
[pyca.config:135:logger_init()] [INFO] Log level set to info
[pyca.config:95:update_configuration()] [INFO] Configuration loaded from ./etc/pyca.conf
[pyca.utils:81:get_service()] [INFO] Endpoint for org.opencastproject.capture.admin: https://develop.opencast.org/capture-admin
[pyca.utils:81:get_service()] [INFO] Endpoint for org.opencastproject.scheduler: https://develop.opencast.org/recordings
[pyca.utils:81:get_service()] [INFO] Endpoint for org.opencastproject.ingest: https://develop.opencast.org/ingest
[pyca.utils:81:get_service()] [INFO] Endpoint for org.opencastproject.capture.admin: https://develop.opencast.org/capture-admin
[pyca.utils:148:register_ca()] [INFO] pyca set to offline
[pyca.utils:81:get_service()] [INFO] Endpoint for org.opencastproject.capture.admin: https://develop.opencast.org/capture-admin
[pyca.schedule:122:control_loop()] [INFO] No scheduled recording
[pyca.utils:148:register_ca()] [INFO] pyca set to idle
[pyca.schedule:122:control_loop()] [INFO] No scheduled recording


Sentry plugin throws type exception on new SentryClient(dsn: dsn);

On a recent master version of Flutter, Sentry is throwing the following exception when calling new SentryClient
E/flutter (30716): [ERROR:topaz/lib/tonic/logging/dart_error.cc(16)] Unhandled exception:
E/flutter (30716): 'package:sentry/sentry.dart': error: line 70 pos 12: file:///Users/DudeGuy/.pub-cache/hosted/pub.dartlang.org/sentry-0.0.6/lib/sentry.dart:70:12: Error: A value of type '() → dart.core::bool' can't be assigned to a variable of type 'dart.core::bool'.
E/flutter (30716): Try changing the type of the left hand side, or casting the right hand side to 'dart.core::bool'.
E/flutter (30716):     assert(() {
E/flutter (30716):            ^
E/flutter (30716):     assert(() {
E/flutter (30716):            ^
E/flutter (30716):
E/flutter (30716): #0      main (file:///Users/DudeGuy/Developer/Testing/sentry0d0d6test/lib/main.dart:7:7)
E/flutter (30716): #1      _startIsolate.<anonymous closure> (dart:isolate/runtime/libisolate_patch.dart:279:19)
E/flutter (30716): #2      _RawReceivePortImpl._handleMessage (dart:isolate/runtime/libisolate_patch.dart:165:12)

Steps to Reproduce

flutter create a new project
add sentry: "^0.0.6" to the pubspec.yaml
call new SentryClient(dsn: YOUR_DSN);

Logs
See above
Flutter Doctor
[✓] Flutter (Channel master, v0.2.4-pre.66, on Mac OS X 10.12.6 16G1212, locale en-US)
    • Flutter version 0.2.4-pre.66 at /Users/KG/Developer/Flutter/flutter
    • Framework revision d9e3715620 (4 hours ago), 2018-03-20 08:14:08 -0700
    • Engine revision c3ab0c9143
    • Dart version 2.0.0-dev.39.0.flutter-f1ebe2bd5c

[✓] Android toolchain - develop for Android devices (Android SDK 27.0.3)
    • Android SDK at /Users/KG/Library/Android/sdk
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-27, build-tools 27.0.3
    • ANDROID_HOME = /Users/KG/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-915-b08)
    • All Android licenses accepted.

[✓] iOS toolchain - develop for iOS devices (Xcode 9.2)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 9.2, Build version 9C40b
    • ios-deploy 1.9.2
    • CocoaPods version 1.3.1

[✓] Android Studio (version 3.0)
    • Android Studio at /Applications/Android Studio 3.0RC2.app/Contents
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-915-b08)

[✓] Android Studio (version 3.0)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-915-b08)

[✓] IntelliJ IDEA Community Edition (version 2017.2.7)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    • Flutter plugin version 21.2.2
    • Dart plugin version 172.4343.25

[✓] VS Code (version 1.21.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Dart Code extension version 2.10.0

Seems like the new head on the repo needs to be pushed to pub

Ah, Yegor is already on it.

@xster / @yjbanov, you poibably know but I think flutter_image needs a similar treatment: https://github.com/flutter/flutter_image

Ya, I think a bunch of them do. We'll try to go through all of them

To be fair, @pulyaevskiy is on it. I'm just reviewing PRs 😎

v2.0.0 of package:sentry has been published to pub. It should fix this issue. https://pub.dartlang.org/packages/sentry


Make usage of the new stop semantics to properly shutdown the plugin

Implements stop to return from run according to https://github.com/elastic/logstash/issues/3210
Depends on https://github.com/elastic/logstash-devutils/pull/32 and https://github.com/elastic/logstash/pull/3812
Fixes #10

LGTM

Merged https://github.com/logstash-plugins/logstash-input-relp/commit/3e8352b6a85ecd097fca6c3f63c0178926fe9a8e


flow $Values utility type does not match against literal

originally brought up by @brentertz in baseui slack
Current Behavior
const FOO = {
  bar: 'bar',
  bas: 'bas'
};

type Foo = $Values<typeof FOO>

// Works
const v1: Foo = 'bar';

// Errors
const v2: Foo = 1;

// Does not error, but should
const v3: Foo = 'quux';

Context
Flow's $Values utility type by default does not compare against the string literal, but instead compares to simple string type. The referenced object should be frozen to operate as expected.
const FOO = Object.freeze({
  bar: 'bar',
  bas: 'bas'
});

type Foo = $Values<typeof FOO>

// Works
const v1: Foo = 'bar';

// Errors
const v2: Foo = 'quux';

5 or so instances of this in the code base

Thanks for opening this Chase! Adding it to our backlog


⚠ Bofrost.at has degraded performance

In 78874d5, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1864 ms

Resolved: Bofrost.at performance has improved in cdde3c7.


SBCL warnings seem to break the REPL connection

Environment

macOS 13.3.1 (22E261)
SBCL 2.3.3
vim 9.0

Issue
Triggering a warning during evaluation seems to break the connection to the REPL. Simple test case to reproduce is to send the following to the REPL. I'm using interaction mode to send these:
(defun foo ())
123
(defun foo ())
123

The Vlime Connection buffer then shows:
SWANK version 2.28, pid 40766, remote <IP_ADDRESS>:7002
====================================================
--
FOO
--
123
--
WARNING: redefining COMMON-LISP-USER::FOO in DEFUN
--
--

So instead of seeing 123 again after the warning, every subsequent evaluation just produces a --.

getting this on arch too

Hmmm, I can't reproduce. Which vlime (version, git commit?) and which swank (in the slime QL) are you using?

vlime version (installed and managed with vim-plug):
chris@chris-air vlime % pwd
/Users/chris/.vim/plugged/vlime
chris@chris-air vlime % git log
commit 3205f02306314ab8cfc9034cf72097891c923e9d (grafted, HEAD -> master, origin/master, origin/HEAD)
Author: Philipp Marek<EMAIL_ADDRESS>Date:   Tue Nov 9 05:56:38 2021 +0000

    Merge pull request #68 from ab-10/patch-1

    Update SBCL version

SWANK version shows as 2.28 in the vlime REPL buffer.

Getting this as well.
However, I don't even need an error to be thrown at me in order for this to issue to crop up.
It just decides to randomly not show output other than "--" after a certain point.
I am running Neovim on Arch, using Packer as my package manager, all latest version.

Well, these are quite old versions.
Would you please update vlime and swank and try again?
The matching swank is https://github.com/vlime/slime.
Thanks!

:PlugUpdate tells me this is the latest version of vlime - and GitHub seems to agree? Double-checked and I'm definitely on master, which seems to be correct.
For SWANK, I think vlime is using the quicklisp one, right? I seem to be using the latest of that:
* (ql:update-dist "quicklisp")
You already have the latest version of "quicklisp": 2023-02-15.
NIL
* (ql:quickload 'swank)
To load "swank":
  Load 1 ASDF system:
    swank
; Loading "swank"

(SWANK)

Would you please try a new version? The direct-json branch from https://github.com/vlime/vlime, and master from https://github.com/vlime/slime?
Thanks.

Checked out that branch, but now I'm getting:
#<THREAD "main thread" RUNNING {70087A06B3}>:
  READ error during LOAD:

    Package SWANK does not exist.

      Line: 44, Column: 36, File-Position: 1236

      Stream: #<SB-INT:FORM-TRACKING-STREAM for "file /Users/chris/.vim/plugged/vlime/lisp/src/vlime.lisp" {7006F155A3}>

It's definitely installed with quicklisp though, so I'm not sure why it's not resolving.
For the slime repo, where am I checking that out to for SBCL/vlime to find it?
Worth adding, I tried doing a manual load through SBCL by first (ql:quickload 'swank) and then (load "~/.vim/plugged/vlime/lisp/start-vlime.lisp") -- which gets further, but then hits this instead:
* (ql:quickload 'swank)
To load "swank":
  Load 1 ASDF system:
    swank
; Loading "swank"
.
(SWANK)
* (load "~/.vim/plugged/vlime/lisp/start-vlime.lisp")
While evaluating the form starting at line 33, column 0
  of #P"~/.vim/plugged/vlime/lisp/start-vlime.lisp":

debugger invoked on a SB-INT:SIMPLE-PROGRAM-ERROR @7005A74B10 in thread
#<THREAD "main thread" RUNNING {70087906B3}>:
  invalid number of arguments: 1

Type HELP for debugger help, or (SB-EXT:EXIT) to exit from SBCL.

restarts (invokable by number or by possibly-abbreviated name):
  0: [REPLACE-FUNCTION     ] Call a different function with the same arguments
  1: [CALL-FORM            ] Call a different form
  2: [CHOOSE-DIFFERENT-PORT] Choose a different port
  3: [RETRY                ] Retry EVAL of current toplevel form.
  4: [CONTINUE             ] Ignore error and continue loading file "/Users/chris/.vim/plugged/vlime/lisp/start-vlime.lisp".
  5: [ABORT                ] Abort loading file "/Users/chris/.vim/plugged/vlime/lisp/start-vlime.lisp".
  6:                         Exit debugger, returning to top level.

((LAMBDA (VLIME::ADDR VLIME::PORT) :IN VLIME:MAIN) 7002) [external]
   source: (LAMBDA (ADDR PORT) (FORMAT T "Server created: (~a ~a)~%" ADDR PORT))

Also a problem in neovim

I was able to load up vlime manually in SBCL, but it makes no difference for the output buffer bug.

The giveaway here is that the warning buffer doesn't show up like it is supposed to.
I managed to downgrade my swank and get it to work so this is a regression

This is a sufficient test case:
(defun foo ())
; FOO
(defun foo ())
; FOO (or nothing -- failed)

first bad commit is 78ad57b7455be3f34a38da456183ddf8d604bdf8

Git apply my patch y'all
From 9f4ed641c3117efe71f9015afa33a26331bc457a Mon Sep 17 00:00:00 2001
From: Spenser Truex<EMAIL_ADDRESS>Date: Fri, 26 May 2023 18:52:35 -0300
Subject: [PATCH] Revert commit that broke vlime's output buffer

While this might be more responsive, it broke vlime. I left the buffer
size increase in though.

Test case was:
start vlime
load (defun foo ()) ; FOO
load (defun foo ()) again
did vlime output?: ; FOO
---
 contrib/slime-repl.el   | 4 +---
 contrib/swank-repl.lisp | 6 +-----
 swank.lisp              | 3 +--
 swank/gray.lisp         | 4 +++-
 4 files changed, 6 insertions(+), 11 deletions(-)

diff --git a/contrib/slime-repl.el b/contrib/slime-repl.el
index d0d48831..03c726eb 100644
--- a/contrib/slime-repl.el
+++ b/contrib/slime-repl.el
@@ -1728,10 +1728,8 @@ expansion will be added to the REPL's history.)"

 (defun slime-repl-event-hook-function (event)
   (slime-dcase event
-    ((:write-string output &optional target thread)
+    ((:write-string output &optional target)
      (slime-write-string output target)
-     (when thread
-       (slime-send `(:write-done ,thread)))
      t)
     ((:read-string thread tag)
      (cl-assert thread)
diff --git a/contrib/swank-repl.lisp b/contrib/swank-repl.lisp
index f6db28b9..259c9ea3 100644
--- a/contrib/swank-repl.lisp
+++ b/contrib/swank-repl.lisp
@@ -122,11 +122,7 @@ DEDICATED-OUTPUT INPUT OUTPUT IO REPL-RESULTS"
   "Create function to send user output to Emacs."
   (lambda (string)
     (with-connection (connection)
-      (send-to-emacs `(:write-string ,string nil ,(current-thread-id)))
-      ;; Wait for Emacs to finish writing, otherwise on continuous
-      ;; output its input buffer will fill up and nothing else will be
-      ;; processed, most importantly an interrupt-thread request.
-      (wait-for-event `(:write-done)))))
+      (send-to-emacs `(:write-string ,string)))))

 (defun open-dedicated-output-stream (connection coding-system)
   "Open a dedicated output connection to the Emacs on SOCKET-IO.
diff --git a/swank.lisp b/swank.lisp
index 07af0954..162ffb7e 100644
--- a/swank.lisp
+++ b/swank.lisp
@@ -1010,8 +1010,7 @@ The processing is done in the extent of the toplevel restart."
           &rest _)
          (declare (ignore _))
          (encode-message event (current-socket-io)))
-        (((:emacs-pong :emacs-return :emacs-return-string :ed-rpc-forbidden
-           :write-done)
+        (((:emacs-pong :emacs-return :emacs-return-string :ed-rpc-forbidden)
           thread-id &rest args)
          (send-event (find-thread thread-id) (cons (car event) args)))
         ((:emacs-channel-send channel-id msg)
diff --git a/swank/gray.lisp b/swank/gray.lisp
index 162e344a..6a3794c4 100644
--- a/swank/gray.lisp
+++ b/swank/gray.lisp
@@ -39,9 +39,11 @@

 (in-package swank/gray)

+(defconstant +string-maximum+ 64000)
+
 (defclass slime-output-stream (fundamental-character-output-stream)
   ((output-fn :initarg :output-fn)
-   (buffer :initform (make-string 64000))
+   (buffer :initform (make-string +string-maximum+))
    (fill-pointer :initform 0)
    (column :initform 0)
    (lock :initform (make-lock :name "buffer write lock"))
--
2.35.1

@equwal I've applied your patch and the vlime repl buffer is working fine now. Thank you!

ac21427d85234b39ae301a0723bf9ab1126157ef might help for the non-JSON version as well - that sends the :WRITE-DONE reply.
@chrisjdavies I've got a symlink to the vlime checkout in my quicklisp/local-projects/ directory, and the vlime/swank checkout is in there too. Shall we find out what goes wrong?

ac21427 might help for the non-JSON version as well - that sends the :WRITE-DONE reply.
@chrisjdavies I've got a symlink to the vlime checkout in my quicklisp/local-projects/ directory, and the vlime/swank checkout is in there too. Shall we find out what goes wrong?

That's probably the Right Way To Do It with the current status of the latest developments. I made a very cheap hack above:

I went into quicklisp and deleted slime 2.28
I remade the directory with a git clone
I bisected it so I could make that patch
I checked out 2.28 again
I applied my patch
it werks!

I just wanted to hack some CL code, I'm not any good at vimscript. I hope this explanation cleans things up.

@equwal : Thanks for your patch!
I followed your instructions above but I still can only send one expression to the REPL and get an error then.
vlime-error.txt
Can you tell from the output above what's wrong?

Now it works

That's great!

Why is this already closed? There is a workaround but maintaining my own fork of slime is not a real fix.

As your commit was reported a success, I took the liberty to push it upstream, so you wouldn't have to bother with a pull request.
Was that wrong? Shall I revert it? Sorry, I thought I had mentioned that in an issue?!
Update: ah, yeah, but that was in #85, not this here.

This wasn't my commit. I only tested equwal's commit. With the latest commit of vlime, it works both with v2.28 and master of slime/swank. I can send multiple expressions to the REPL but I also get many warnings.
@equwal : Is your commit still necessary? Can I go back to the quicklisp version of slime/swank?


api: fix compilation after bad rebase

Current coverage is 54.85% (diff: 100%)

No coverage report found for master at 27ae8c7.

Powered by Codecov. Last update 27ae8c7...02a309c

LGTM


USE multi-scale testing strategy

hello
Do you use multi-scale testing strategy in test_RFB.py ?
your paper seems not mention about using multi-scale testing strategy.
thanks!

We don't use any testing strategy like flipping or multi-scale.


How to avoid large batch been killed

I have 5176 images with 5176 prompts
(dynamicrafter) xiangrui@ward-lambda01:~/video_generators/DynamiCrafter$ sh scripts/run_1024.sh 1024
@DynamiCrafter cond-Inference: 2024-09-18-15-48-40
Global seed set to 666
AE working on z of shape (1, 4, 32, 32) = 4096 dimensions.
>>> model checkpoint loaded.
Inference with 16 frames
Killed
(dynamicrafter) xiangrui@ward-lambda01:~/video_generators/DynamiCrafter$

The model worked well with 8 samples but once I replaced the default prompts with my 5000+ prompts. The task is killed.

Hi. It may be due to the memory leakage issue, as we load all input data into memory at the same time. I think a solution is to use a data loader to fetch the input data by batch.


Html component children not parsed correctly

<comp>
inner
</comp>

In this scenario, inner should be rendered as the children of comp, but it is not.

This is fixed by
d82f923


After loading Lavalink.jar, it plays the every first song that we pass in and stops working after that

Summary

After we run the Lavalink.jar file, it establishes connection, after that whenever we play very first song, it plays nicely, but if we disconnect the bot after that and try to play another song, it doesnt work.
Lavalink & System Version

Reproduction
This is the where the jar file gets stuck at (in the terminal)
Loaded playlist Search results for: bekhayali
2020-11-13 12:23:28.786  INFO 34228 --- [   XNIO-1 I/O-4] lavalink.server.io.SocketServer          : {"op": "play", "guildId": "760517589100527656", "track": "QAAAxgIAZEJla2hheWFsaSBGdWxsIFNvbmcgfCBLYWJpciBTaW5naCB8IFNoYWhpZCBLLEtpYXJhIEF8U2FuZGVlcCBSZWRkeSBWYW5nYSB8IFNhY2hldC1QYXJhbXBhcmEgfCBJcnNoYWQACFQtU2VyaWVzAAAAAAAFZtAAC1ZPTEtKSnZmQWJnAAEAK2h0dHBzOi8vd3d3LnlvdXR1YmUuY29tL3dhdGNoP3Y9Vk9MS0pKdmZBYmcAB3lvdXR1YmUAAAAAAAAAAA=="}
Expected Results

The current song will stop when it disconnects.
Actual Results

The previous song never stops. If we disconnect the bot from the vc, and reconnect it, the bots seems to be playing something but nothing is audible
Checklist

[x] Have you checked that there aren't other open Bug Reports for the same problem?
[x] Have you checked that your Lavalink password isn't visible?
[x] Have you given a traceback if possible?

This seems to be a problem with your code handling disconnect events, see the example cog here: https://github.com/Devoxin/Lavalink.py/blob/master/examples/music.py

Oh yes! I figured it out, if we don't clear the queue before disconnecting the bot, it lags out the playlist queue, leading to the issue I mentioned above.


SetLoadFields doesn't work as expected

1. Describe the bug
In CU22, function GetValuationDate, even if I set the SetLoadFields in Event "OnGetValuationDateOnAfterOldValueEntrySetFilters" the system has a strange behavior, and picks up info not requested from TableExtension
2. To Reproduce
I created a TableExtension of table 5802 "Value Entry", adding several fields. Two fields added are "Source Order Type" and "Source Order No." (that have TableRelation to other tables depending on "Source Order Type":
TableExtension 55802 "Value Entry Ext." extends "Value Entry"
{
    fields
    {
        field(50020; "Commercial Cond. Code"; Code[20])
        {
            Caption = 'Commercial Cond. Code';
        }
        field(50021; "Commercial Cond. %"; Decimal)
        {
            Caption = 'Commercial Cond. %';
        }
        field(50200; "Source Order No."; Code[20])
        {
            Caption = 'Order No.';
            Editable = false;
            TableRelation = if ("Source Order Type" = const(Sales)) "Sales Header"."No." where("Document Type" = const(Order), "No." = field("Source Order No."))
            else
            if ("Source Order Type" = const(Purchase)) "Purchase Header"."No." where("Document Type" = const(Order), "No." = field("Source Order No."))
            else
            if ("Source Order Type" = const("Return Sales")) "Sales Header"."No." where("Document Type" = const("Return Order"), "No." = field("Source Order No."))
            else
            if ("Source Order Type" = const("Return Purch.")) "Purchase Header"."No." where("Document Type" = const("Return Order"), "No." = field("Source Order No."))
            else
            if ("Source Order Type" = const(Service)) "Service Header"."No." where("Document Type" = const(Order), "No." = field("Source Order No."));
            ValidateTableRelation = false;
        }
        field(50201; "Source Order Line"; Integer)
        {
            Caption = 'Order Line';
            Editable = false;
        }
        field(50202; "Source Order Type"; Option)
        {
            Caption = 'Order Type';
            Editable = false;
            OptionCaption = ' ,Sales,Return Sales,Purchase,Return Purch.,Service,Transfer';
            OptionMembers = " ",Sales,"Return Sales",Purchase,"Return Purch.",Service,Transfer;
        }
    }
    keys
    {
    }
}

Because of this TableExt, since Value Entry table has more than 40 million records, the function GetValuationDate results really slow, because the join with SetCurrentKey is not optimized. So I decided to insert a SetLoadFields in Event OnGetValuationDateOnAfterOldValueEntrySetFilters, in order to load only the field needed in the process and avoiding join with TableExt:
Subscriber:
    [EventSubscriber(ObjectType::Codeunit, Codeunit::"Item Jnl.-Post Line", 'OnGetValuationDateOnAfterOldValueEntrySetFilters', '', false, false)]
    local procedure OnGetValuationDateOnAfterOldValueEntrySetFiltersCU22(var OldValueEntry: Record "Value Entry"; ValueEntry: Record "Value Entry"; OldItemLedgEntry: Record "Item Ledger Entry")
    begin
        OldValueEntry.SetLoadFields("Valuation Date");
    end;

Removing the TableRelation from field "Source Order No." the function is fast and it's working (doesn't pick TableExt fields, even if it picks table fields not requested, apparently randomly)
3. Expected behavior
I expect that after a SetLoadFields, all the fields not requested won't be picked (and also join with TableExt)
4. Actual behavior
I created a TableExtension of table 5802 "Value Entry", adding several fields. Two fields added are "Source Order Type" and "Source Order No." (that have TableRelation to other tables depending on "Source Order Type", see screenshot below:

Because of this TableExt, since Value Entry table has more than 40 million records, the function GetValuationDate results really slow, because the join with SetCurrentKey is not optimized. So I decided to insert a SetLoadFields in Event OnGetValuationDateOnAfterOldValueEntrySetFilters, in order to load only the field needed in the process and avoiding join with TableExt:
CU22:

Subscriber:

However, the SetLoadFields doesn't work completely, because doesn't load most of the fields of TableExt, but loads the fields "Source Order Type" and "Source Order No.", so the FindLast results very slow (see SQL statement below):
Live Query Statistics:

SQL Statement:

Removing the TableRelation from field "Source Order No." the function is fast and it's working (doesn't pick TableExt fields anymore, even if it picks table fields not requested, apparently randomly), as you can see from SQL Statement below picked from SQL Server Profiler
SQL Statement after modification:

Could you please check what's wrong?
5. Versions:

AL Language: v10.1.707468
Visual Studio Code: Version 1.73.1
Business Central: IT Business Central 21.1 (platform 21.0.48357.48504 + application 21.1.48363.48638 (OnPrem)
List of Visual Studio Code extensions that you have installed:

I believe it is expected that fields used in relations are always loaded, and has been discussed on here. I'm not sure it got documented as I think it was considered an implementation detail.

It doesn't seem so, in Value Entry Table there are many fields that have TableRelation but they're not loaded (Es. "Source No.", "Source Posting Group"), and there are others instead that don't have TableRelation and are loaded (Es. "Source Type", "Entry Type")...
It's a strange behavior from my point of view, it would be nice to know the logic behind in order to avoid unwanted behavior

Hi, this issue is out of scope of for this repository. You should do one of the following:
Open a support request to CSS through the PartnerSource portal
Open a bug in Collaborate
Contact your Service Account Manager (SAM) in your local subsidiary to understand what is included in your contract as of support incident and PAH (Partner Advisory Hours). Your SAM might also direct you "step by step" how to open a support request or how to get credentials, if this is the first time for you or your company
As a reminder, this repository only handles issues to do with the compiler or VSCode development experience for AL.
The reason why this is out of scope is because it relates to the runtime behaviour of BC, so it needs to go to the team who handles the runtime.


[stdib] Audit @_fixed_layout on collection storage classes

Remove __SwiftNativeNSEnumerator from the ABI. (It’s only used in internal classes and in the Objective-C runtime.)
Remove inlinability of init() and deinit for all the __SwiftNativeNSFoo superclasses except Array’s and Data’s.
As an experiment, let’s try removing @_fixed_layout from _SetStorage and _DictionaryStorage. Their current ivars are all defined in their respective superclasses, so (if my mental model of how this works is correct), this may have little effect on benchmarks, but it adds a little bit of flexibility. Maybe.

@swift-ci please test

@swift-ci please benchmark

Build comment file:
Performance: -O

TEST
OLD
NEW
DELTA
RATIO

Improvement

NopDeinit
52623
45432
-13.7%
1.16x

How to read the data
The tables contain differences in performance which are larger than 8% and
differences in code size which are larger than 1%.
If you see any unexpected regressions, you should consider fixing the
regressions before you merge the PR.
Noise: Sometimes the performance results (not code size!) contain false
alarms. Unexpected regressions which are marked with '(?)' are probably noise.
If you see regressions which you cannot explain you can try to run the
benchmarks again. If regressions still show up, please consult with the
performance team (@eeckstein).

Hardware Overview
  Model Name: Mac Pro
  Model Identifier: MacPro6,1
  Processor Name: 12-Core Intel Xeon E5
  Processor Speed: 2.7 GHz
  Number of Processors: 1
  Total Number of Cores: 12
  L2 Cache (per Core): 256 KB
  L3 Cache: 30 MB
  Memory: 64 GB

--------------

Build failed
Swift Test Linux Platform
Git Sha - ca09281b3418b99c2053b078590a9ee7412b229a

Build failed
Swift Test OS X Platform
Git Sha - ca09281b3418b99c2053b078590a9ee7412b229a

Build failed
Swift Test Linux Platform
Git Sha - ca09281b3418b99c2053b078590a9ee7412b229a


Both Client and Server Broken

I can't seem to be able to compile both client and server. I supplied all the libraries and dependancies and such but the code itself has way too many errors :c I just want to be able to host the server locally and try the game on my machine

Unfortunately, this project is not a high enough priority for me atm to find time to fix it publicly. The source will stay as-is for now.

I can't seem to be able to compile both client and server. I supplied all the libraries and dependancies and such but the code itself has way too many errors :c I just want to be able to host the server locally and try the game on my machine

The source has now been fixed


Installing ng2-tour

Hi,
I'm trying to install the package using the standard npm install ng2-tour
I noticed that the package.json file included in the installation has older peer dependencies than those that exist in the root directory package.json
This is what it requires when installing the package:
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
I edited the package.json file in my local node_modules\ng2-tour, and went off from there .

I need to update the documentation for version 1.x.  You'll be installing ngx-tour-md-menu or ngx-tour-ng-bootstrap instead of ng2-tour.  More details to come.

Until version is ready I should use ng2-tour?
Or I should use the other packages already?

Version 1.x should be working, but I haven't updated the documentation to go with it.

I'm having a tough time fixing issue #30 and I don't think the library is very usable until that is fixed.  I'll try to get that fixed as soon as possible.

Finally fixed that issue.  Check out the new documentation.