Blogging link in work.md (/work page) is broken.

Hi Isabel, I noticed that one of your links on your /work page points to possibly an old folder name and is therefore broken.  I will submit a PR to fix this.

Thank you @SuzeShardlow 🙌


Do not export phy::wait on non-Unix platforms

Necessary to use smoltcp on seL4 (and perhaps other platforms).
By @aisamanra

Thanks!


Background Music and Click Sound

Hi firstly I'm grateful to you secondly I want to add Background music and Click sound but I could not this.Could you help for solutions?
Thx

I need help for the background music too.

Hello,
i would like to work on this issue. Is is possible?
Thanks.

I need help for the background music too!


Accelerate TRK and TCK loading

List of fixed issues (if they exist):

This doubles the speed for loading TRK and TCK format tractography data
@jens-ox the new javascript code runs, but the TypeScript generates errors as the functions return vertices are of type Float32Array rather than number[] and offsets of of type Uint32Array rather than number[]. My own sense is that all tractography data types (TRACT, TT, TRX, TRK, TCK, and VTK) should return these typed arrays. Do you have a strong opinion on this?

@neurolabusc using typed arrays everywhere makes sense to me. Do you want me to change the code accordingly?

@jens-ox I would be grateful if you could change these to be always typed arrays.

#847


Adding a property to disable the zoom

It would be nice to add a bool property to easily disable the zoom

did you solve your problem?
how can i disable zoom?

I have not given it enough time to try due to lack of it is importance to me right now.
Once I find a solution, I will update here Insh'Allah.


output in console

The popup is super annoying plus nobody can really read what's going on.
It might make more sense to print into the console.
Sorry but this currently leads to me removing this package again.
I hope you can find a solution that makes everybody happy

Moving away from Github. Feel free to reopen if interested.


Kanban board setup

Hi Reviewer 👋
This issue is created to provide task assignments and team information for Rails group project.
We have a team of 3 working on this project:
Team member #1: Tajemouti
Team member #2: Ngala
Team member #3: Newton

Hi Team,
You can go ahead with your project
Congratulations! 🎉

Highlights
✅ All tasks are created correctly and assigned fairly
✅ Tasks are descriptive 👍
Optional suggestions
N/A
Every comment with the [OPTIONAL] prefix won't stop the approval of this PR. However, I strongly recommend you to take them into account as they can make your code better. Some of them were simply missed by the previous reviewer and addressing them will really improve your application.
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.

As described in the Code reviews limits policy  you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.


Family column is showing instance type instead of family

The family column is showing the instance type instead of family to which they belong.

There isn't a column named "family" currently. Which are you referring to?

@powdahound , I'm referring to the family key of the JSON available in  http://www.ec2instances.info/instances.json..

Thanks. I forgot that was there. It looks like it now contains the family part of the ID where it used to contain the family's "pretty name" like "General Purpose" or "Compute Optimized". Is that the issue you're running in to?
If you are looking to use the pretty names, mind sharing your use case? We haven't made a big effort to keep the list of pretty names up to date and I see that many are missing.
$ curl -sL http://www.ec2instances.info/instances.json | jq '.[] | {type: .instance_type, family: .family}'
{
  "type": "c5d.xlarge",
  "family": "c5d"
}
{
  "type": "m5a.2xlarge",
  "family": "m5a"
}
{
  "type": "c5.9xlarge",
  "family": "c5"
}
... etc ...

Yes @powdahound , we are using pretty names and segregating the instance types based on them. Could you pls help us here..

I'm sorry, that regression is my fault.
curl -sL https://github.com/powdahound/ec2instances.info/raw/9692781cf5fc9410fbb0d9532afcb97fb6f52483/www/instances.json | jq '.[] | {type: .instance_type, family: .family}'

{
  "type": "m1.small",
  "family": "General purpose"
}
{
  "type": "m1.medium",
  "family": "General purpose"
}
{
  "type": "m1.large",
  "family": "General purpose"
}
{
  "type": "m1.xlarge",
  "family": "General purpose"
}

@jprakash402 could you check the PR I made and see if it's what you were looking for?

Just merged #402, @powdahound please trigger a rebuild when you have some time.

@blckct @cristim thanks for the speedy PR & merge! 🎉
@jprakash402 hope that works out for you

Thank you so much guys, for the quick response...
you made my day..!!


Why include parameters in Dash docset index?

Is this needed? Do you usually search using a parameter name? Just including the methods should be fine as far as I can tell. I don't use node though so I have no idea.
You could include them in the in-page anchors only and not inside the main search index.

I had included parameters so that you can search for sub-parameters like this:

Is there a better way to make them searchable?

Nope, that's the only way to make them searchable. My concerns were whether or not they should be searchable in the first place. None of the existing docsets allow searching of parameters.
Is there any reason why this would be useful for node packages?

This is true. Perhaps it's not as necessary as I had originally thought, as they are still visible through the Table of Contents.


React36: Manage...: Question: What's happening - part 1

I have two questions - this is the first. My code is below. When the code first runs, it outputs the following:
[Log] will receive   ---> (This is from componentWillReceiveProps)
[Log] {message: "first"}
[Log] {message: "TestMessage"}
[Log] Component is about to update...
[Log] component did update
My question: Where did "TestMessage" come from?
My code:
componentWillReceiveProps(nextProps){
  console.log("will receive");
  console.log( this.props);
  console.log( nextProps);
}
  componentDidUpdate(){
    console.log("component did update");
  }

I also added a log to
changeMessage = () => {
    **console.log("change message");**
		this.setState({
			message: 'second'
		});
	}

My next question will be about the order of events, but maybe your response will answer it.

Oh, so that message is actually coming in from the tests. We're setting the state on the component in a test and verifying the h1 tag displays the current text from the component state. We might want to change that given that you may be looking at the console messages and get confused by that. What do you think?
And does that answer your other question?

It’s definitely confusing, especially as you recommend that we look at the console output.
It would be OK if you mentioned in the notes that it was going to happen.
My second question is quite a bit more general. I think I should open an issue as it may be of general interest.
If you don’t mind a little digression - I like to understand in some detail what is going on under the covers, and with JSX there are a lot of covers. I wrote a document introducing React without using JSX. I experimented a lot and decided to write it up. It just touches the surface - not nearly as complete or useful as these challenges - more of a compulsive approach. But there are things in there that I think might be useful at least for your notes. Would you mind taking a brief look at it? I think there may be some mistakes as well. You certainly don’t have to look.  Anyway, it’s here.
https://github.com/sabrawer/hello-world/tree/master https://github.com/sabrawer/hello-world/tree/master
Thanks
Steve
On Jan 27, 2017, at 2:32 PM, Sean Smith<EMAIL_ADDRESS>wrote:
Oh, so that message is actually coming in from the tests. We're setting the state on the component in a test and verifying the h1 tag displays the current text from the component state. We might want to change that given that you may be looking at the console messages and get confused by that. What do you think?
And does that answer your other question?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/bonham000/fcc-react-tests-module/issues/167#issuecomment-275753366, or mute the thread https://github.com/notifications/unsubscribe-auth/AWkV2wJWKyUb3GWvvnZkX2vad-Gm2qHNks5rWkZOgaJpZM4LwHgV.

Ok! Firstly, about the challenge I've just removed the test that was causing this additional console.log because it's not essential to this challenge. Now you will just see the relevant messages in the console, I think that sould fix it.

Probably a good idea - makes everything simpler.
On Jan 27, 2017, at 3:00 PM, Sean Smith<EMAIL_ADDRESS>wrote:
Ok! Firstly, about the challenge I've just removed the test that was causing this additional console.log because it's not essential to this challenge. Now you will just see the relevant messages in the console, I think that sould fix it.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/bonham000/fcc-react-tests-module/issues/167#issuecomment-275760142, or mute the thread https://github.com/notifications/unsubscribe-auth/AWkV23GEwYfJEUbX4Rl0Dsr8d2JjPF6uks5rWkzygaJpZM4LwHgV.

Secondly, yeah your document looks good! I think a lot of people do get stuck on JSX when they are learning React and some pick it up more quickly. Because it's used so ubiquitously, we did choose to just introduce it and use all of the examples using JSX in these challenges, but I think having a more in-depth understanding of how React actually creates HTML elements from JSX is useful to have.


Allow passing in additional jwt headers

What kind of change does this PR introduce? (check at least one)

[ ] Bugfix
[x] Feature
[ ] Code style update
[ ] Refactor
[ ] Other, please describe:

Does this PR introduce a breaking change? (check one)

[ ] Yes
[x] No

[x] You consent that the copyright of your pull request source code belongs to Authlib's author.

Referencing https://github.com/lepture/authlib/issues/391
I'd like to pass in addtional JWT headers into the client assertion, such as the kid so the recipient of the token can find the corresponding key at the registered JWKS endpoint.
Describe the solution you'd like
When setting up an PrivateKeyJWT() instance to register as client auth method, I'd like to pass in additional JWT headers such as the kid.

This PR is based on https://github.com/lepture/authlib/pull/390.
Please let me know what I can do to make the PR qualify for merge. Thanks!

You may already be doing this, but if you are allowing users to specify this algorithm, make sure you are very careful about discouraging (maybe even explicitly disallowing) the use of the none algorithm, which has been a source of many vulnerabilities in JWT-related libraries.
I notice some places where None is set as a default value and special checks like if not None are involved. We need to make it hard for users or future developers of this library to accidentally shoot themselves in the foot with insecure tokens. :slightly_smiling_face:

You may already be doing this, but if you are allowing users to specify this algorithm, make sure you are very careful about discouraging (maybe even explicitly disallowing) the use of the none algorithm, which has been a source of many vulnerabilities in JWT-related libraries.
I notice some places where None is set as a default value and special checks like if not None are involved. We need to make it hard for users or future developers of this library to accidentally shoot themselves in the foot with insecure tokens. 🙂

Thanks for the heads-up. I wonder: the PR is for code that creates the token, but, the vulnerabilities you refer to seems to be about the verification of tokens, right? I do think indeed we should help developers to the right thing.

You're correct about create/verify, but this code that creates the token should make it difficult (or IMHO impossible) to create an unsecured token.

You're correct about create/verify, but this code that creates the token should make it difficult (or IMHO impossible) to create an unsecured token.

I agree and thanks for confirming my understanding of your remark. I hope the reviewer can assess if my approach at least makes it not too easy to forget passing in an signing alg.

And thanks @lepture for accepting the PR!!


question: should we show "locked until ..." in transaction list if still locked

Currently (Coin) Output(s) shown in the Transaction List that are still locked, are not shown yet. Should we show it? If so, could you do it.
Not sure if we should do this only for incoming outputs or also outgoing, but for sure I think the incoming ones should have it.

Any input on this one @DylanVerstraete???

We now show locked input/outputs in the transaction list.
closed in https://github.com/threefoldfoundation/tfchain-wallet-electron/commit/a11e71f71ff75faeba276490df2747cf2417d600


[3708] Add Dqt::Params::TrnRequest params presenter

Context
Add a Dqt::Params::TrnRequest presenter
Changes proposed in this pull request
Add a presenter similar to the ones we already have.
This presenter receives a trainee record and outputs json in the format defined in the DQT api.
Guidance to review
There are a couple of areas that might need a bit of attention:

Training Route mappings (programmeType in DQT), and
Degree grade mappings (qualification->class in DQT).
Please see notes added in the PR against those two areas.

Important business

~Does this PR introduce any PII fields that need to be overwritten or deleted in db/scripts/sanitise.sql?~
~Does this PR change the database schema? If so, have you updated the config/analytics.yml file and considered whether you need to send 'import_entity' events?~

 [L] Dqt::Params::TrnRequest


Update bartender to 3.0.8

After making all changes to the cask:

[x] brew cask audit --download {{cask_file}} is error-free.
[x] brew cask style --fix {{cask_file}} left no offenses.
[x] The commit message includes the cask’s name and version.

Additionally, if updating a cask:

[ ] sha256 changed but version stayed the same (what is this?).
I’m providing public confirmation below.

Closing in favor of #38952, the issue wasn't open when I started editing it.


Revert dotnet-install.sh script

Type of issue
Missing information
Description
I wanted a way to quickly and easily get .NET 8 SDK running on a fresh Ubuntu Server for some AOT testing. I saw that you provide an installation script so I tried it out. As normal user, and then as root with sudo. None give me a dotnet command in the console, and I'm not looking for a manual installation here. I don't know what this script did exactly. I want to revert it again. Sadly this information is missing here.
And actually I was looking for that script that properly sets up the package source so I will also get updates later and have the dotnet command available right away. I'll continue looking for that thing. Hope it's still somewhere around.
Page URL
https://learn.microsoft.com/en-us/dotnet/core/tools/dotnet-install-script
Content source URL
https://github.com/dotnet/docs/blob/main/docs/core/tools/dotnet-install-script.md
Document Version Independent Id
5bc7caa5-f1c2-d7d5-fe02-80c65cca9672
Article author
@tdykstra
Metadata

ID: a2e484a9-5e0c-eb3e-1413-7b507c4fa07d
Service: dotnet-fundamentals

The Purpose section of the documentation article explains that this script is intended for CI scenarios:

The intended use of the scripts is for Continuous Integration (CI) scenarios, where:

The SDK needs to be installed without user interaction and without admin rights.

The SDK installation doesn't need to persist across multiple CI runs.

For information about how to install .NET on Ubuntu, see https://learn.microsoft.com/dotnet/core/install/linux-ubuntu


Background media playback on mobile browsers and Safari

Mobile browsers and current versions of Safari do not allow autoplaying video elements unless they are muted. Using WebAudio API, audio files can be autoplayed after a first user interaction has unlocked the audio context.
Previous behavior on mobile browsers:

We mute page background video loops to allow autoplay.
Atmo is disabled since the current implementation uses one audio elements per audio file which would have to be unlocked by user interaction individually to be played.
Autoplay of media on video and audio pages is disabled. User can start playback via play button.

Since Safari adopted autoplay restrictions on desktop, background video was basically broken. #949 does most of the work required to improve this situation:

Replaces previous audio library with Howler, which supports playing all audios via a single audio context that can be unlocked by a single user interaction.
Plays sound of muted background videos via this audio context once it is unlocked.
No longer needs to disable atmo on mobile devices.
Displays a button to unmute background media which provides the required user interaction.

Remaining open questions:

Should there be logic to only display the unmute button if the entry actually contains either background videos or pages with atmo audio?
Detecting pages like this might be tricky since (a) background videos might not have an audio track to begin with and (b) figuring out if a page has an atmo audio file requires actually looking up the file since audio file ids in the page configuration are allowed to point to files which have been removed from the entry.

Should clicking the "Let's go" button inside the multimedia alert already unmute background media?
There still might be value in letting the user view the entry with background media disabled. On the other hand, having to click an unmute button after just having been alerted that there will be sound might be confusing.

Remaining tasks:

[ ] Use Howler to play atmo and background video sound (#949)
[ ] Update help texts to remove hint about disabled atmo on mobile

@tilsammans We are finally getting around to tackling this issue. Happy to hear your thoughts.

It look's like the problem also occurs on Desktop Browsers:

Tested with Chromium 73.0.3683.75.

@danielmorlock Far as I know, the console warning does not indicate a problem, but rather happens when we try to resume the the audio context since there is now way of knowing whether we come from a user gesture. The Hound approach described above and implemented in #949 still has synchronicity issues when playing uncached videos and is therefore blocked. The current state of autoplay support is outlined in this doc file.


Placeholder does not conform to NSTextAlignment

I am using a SLKTextView and I have its text to be centered align. When actual text is entered, it works as expected as centers the input text, but the placeholder text itself still sticks to left aligned when no text is entered.
Any help would be appreciated!

Awesome, thanks a bunch for the quick fix! :+1:

where to change a placeholder alignment in textview ? For me it does not work (I subclass it)


CommonJS support?

Thanks for the great package! It looks like it'll make strongly typing our mocks so much easier.
However I'm trying to use it with msw on a nodejs server app. The app is compiled as cjs and therefore I'm having issues using openapi-msw since it is an esm-only package.
Would it be possible to also provide a cjs build and require exports in the package.json?

Thanks @christoph-fricke , I agree it would be ideal if we could use ESM for everything. Unfortunately I support a lot of older websites that still use CJS server builds and it will take the teams time to migrate them to build as ESM. Having CJS support will allow us to implement strongly-typed mocks in the meantime so it's really appreciated that you are open to supporting. We also can't move to Node.js v22 until it goes LTS in October.

@jrestall-finact Great news. Support for CommonJS has been released with v0.6.0. 🎉

Thanks so much @christoph-fricke, it's working great! 🎉


Disable WebUI in production environment

@aronchick has requested we disable the WebUI in the prod env

Done in https://github.com/bacalhau-project/bacalhau/pull/4133


Functional tests - Unable to resolve service for type 'Microsoft.Extensions.Logging.ILogger' while attempting to activate 'Application.Infrastructure.RequestLogger`

Hi,
thank you for your great talks and the project!
I've been trying to use this but one thing doesn't work in functional test. When I run a functional test in my project (which is based on this one as you can imagine) I get this:

Unable to resolve service for type 'Microsoft.Extensions.Logging.ILogger' while attempting to activate 'Prj.Application.Infrastructure.RequestLogger`1[Prj.PageBlocks.Commands.CreatePageBlockCommand]'.

The only dependency I have on the ILogger is in RequestPerformanceBehaviour.
I'm missing on how to make the ILogger available in tests.

ok, I fixed this in my project. And I have no #$%#$% idea how :(( I'd really like to know.
Yeah, I know this is not an issue with this repo but still...

Hi Alex - I have no idea either. I did recently rebuild my machine, and was able to clone and run tests from scratch - no problems. Maybe a glitch in the matrix? :P


The expression to get an indexed field is only valid for List types (common_sub_expression_eliminate)

Describe the bug
I am trying to perform a trivial join in the CLI using latest from master and I get the following error:
Plan("The expression to get an indexed field is only valid for `List` types")

To Reproduce
$ cat /tmp/a.csv
foo
1

$ cat /tmp/b.csv
b
bar

$ ./target/debug/datafusion-cli

DataFusion CLI v10.0.0
❯ create external table a stored as csv with header row location '/tmp/a.csv';
0 rows in set. Query took 0.016 seconds.
❯ create external table b stored as csv with header row location '/tmp/b.csv';
0 rows in set. Query took 0.009 seconds.
❯ select * from a;
+-----+
| foo |
+-----+
| 1   |
+-----+
1 row in set. Query took 0.011 seconds.
❯ select * from b;
+-----+
| b   |
+-----+
| bar |
+-----+
1 row in set. Query took 0.010 seconds.
❯ select * from a join b on a.foo = b.bar;
Plan("The expression to get an indexed field is only valid for `List` types")

Expected behavior
Query should work
Additional context
None

The query in the repro is invalid so it looks like we need to improve the error reporting here to an "invalid field" error.
If I specify the schema then the query works.
DataFusion CLI v10.0.0
❯ create external table a (foo varchar) stored as csv with header row location '/tmp/a.csv';
0 rows in set. Query took 0.002 seconds.
❯ create external table b (bar varchar) stored as csv with header row location '/tmp/b.csv';
0 rows in set. Query took 0.001 seconds.
❯ select a.foo, b.bar from a join b on a.foo = b.bar;
0 rows in set. Query took 0.038 seconds.

The issue seems to be related to having relations containing columns with the same name as the relation.
$ cat /tmp/a.csv
foo
1

$ cat /tmp/b.csv
bar
1

DataFusion CLI v10.0.0
❯ create external table a stored as csv with header row location '/tmp/a.csv';
0 rows in set. Query took 0.017 seconds.
❯ create external table b stored as csv with header row location '/tmp/b.csv';
0 rows in set. Query took 0.003 seconds.
❯ create external table bar stored as csv with header row location '/tmp/bar.csv';
IoError(Os { code: 2, kind: NotFound, message: "No such file or directory" })
❯ create external table bar stored as csv with header row location '/tmp/b.csv';
0 rows in set. Query took 0.003 seconds.
❯ select a.foo, b.bar from a join b on a.foo = b.bar;
+-----+-----+
| foo | bar |
+-----+-----+
| 1   | 1   |
+-----+-----+
1 row in set. Query took 0.038 seconds.
❯ select a.foo, bar.bar from a join b on a.foo = bar.bar;
Plan("The expression to get an indexed field is only valid for `List` types")
❯

The first query works but the second query fails. The queries are identical except for the name used to register the table. In the second case the table name is the same as the column name in the table.

select b.b from b doesn't work originally, solved by your pull request as well. I've sent a pull request to yours for adding this simple test case


Resource env var, with transport interface

Resolves https://github.com/ignitionrobotics/ign-gazebo/issues/123.
~Requires https://github.com/ignitionrobotics/ign-common/pull/70~
~Builds on top of #181~

This allows models and meshes to be found according to the the IGN_GAZEBO_RESOURCE_PATH environment variable.

It's the Ignition Gazebo equivalent of both GAZEBO_MODEL_PATH and GAZEBO_RESOURCE_PATH.
That variable is already being used to find worlds, now it's extended to <include>s and mesh <uri>s.
The name "resources" is better fitting than "model", since it can load different types of resources.
We've already been supporting the SDF_PATH and IGN_FILE_PATH variables with the same behaviour, so this implementation is backwards compatible and keeps those variables in sync with IGN_GAZEBO_RESOURCE_PATH.
I added a transport interface so these paths can be queried and updated.

The first consumer will be the new GUI resources plugin, see #31
This keeps the ground truth in the server. Gazebo-classic users often would have issues because gzserver and gzclient would be in different shells and have access to different environment variables. See this outstanding issue: https://github.com/osrf/gazebo/issues/2403
I kept the services in the global namespace because worlds can be found through the paths, so the paths are at a level above worlds.

Try it
Get paths:
ign service -s /gazebo/get_resource_paths --reqtype ignition.msgs.Empty --reptype ignition.msgs.StringMsg_V --timeout 5000 --req "unused: false"
Add path:
ign service -s /gazebo/add_resource_paths --reqtype ignition.msgs.StringMsg_V --reptype ignition.msgs.Empty --timeout 5000 --req "data: '/tmp/lala'"

TODOs
I'll leave this in draft state until:

[x] Updates to this tutorial are added to this PR: https://github.com/ignitionrobotics/ign-gazebo/blob/master/tutorials/resources.md
[x] This design document is updated to reflect this PR: https://github.com/ignitionrobotics/design/pull/1/files I opened a new design PR instead: https://github.com/ignitionrobotics/design/pull/4

@osrf-jenkins run tests please

@JShep1, I think this is ready to be used by #173 . Let me know if you run into any problems.

I just ran this and am running into an issue with ign gazebo only showing a black screen with the message
[GUI] [Dbg] [Gui.cc:144] GUI requesting list of world names. The server may be busy downloading resources. Please be patient.

being printed every few seconds.  I replicated this issue on my laptop as well, are you running into this? @chapulina

@JShep1 , a086574 has a temporary fix to that issue. For some reason the global node works when doing ign gazebo -s and ign gazebo -g separately, but not for ign gazebo :thinking: I'll look better into it tomorrow. For now, be warned that if you add any paths through the add service, the GUI will not know about them automatically, which may cause some models to fail.

Ok, this should be good to be reviewed again. @JShep1 , let me know if you need help using it on #173 .

The 3 test failures are not there on ign-gazebo2, I'll take a look: https://build.osrfoundation.org/job/ignition_gazebo-ci-pr_any-ubuntu_auto-amd64/3765/testReport/

The 3 test failures are not there on ign-gazebo2

Not true, they're there on ign-gazebo2 now.


Print reproducible command line on crash

As @adamnovak brought up, using toil-vg in vg's CI makes it difficult for vg developers to quickly reproduce and debug crashes.  The main problems are:

failed command line is buried in the cryptic toil logs, often in Python list format, appearing many times.
error message usually even better hidden than the command
intermediate files used in the command often lost

There should be some easy ways to at least improve most of these issues.  Ideally this would mostly happen within call() in vg_common.py, but may need some help from vg's Jenkins code and/or Toil

I think we might actually be OK with regard to reproducing problems. If we
keep the Jenkins tests granular enough, we can now run them one at a time
through jenkins.sh, by passing something like -t jenkins/vgci.py::VGCITest::test_sim_brca2_snp1kg.
Sometimes we might want to go and run the individual vg commands, though,
if we manage to trigger a crash bug in vg or something.
On Mon, Jul 10, 2017 at 12:53 PM, Glenn Hickey<EMAIL_ADDRESS>wrote:

As @adamnovak https://github.com/adamnovak brought up, using toil-vg in
vg's CI makes it difficult for vg developers to quickly reproduce and debug
crashes. The main problems are:

failed command line is buried in the cryptic toil logs, often in
Python list format, appearing many times.
error message usually even better hidden than the command
intermediate files used in the command often lost

There should be some easy ways to at least improve most of these issues.
Ideally this would mostly happen within call() in vg_common.py, but may
need some help from vg's Jenkins code and/or Toil
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/BD2KGenomics/toil-vg/issues/231, or mute the thread
https://github.com/notifications/unsubscribe-auth/AE0_X3_fOdD6i7nAiSZGESIJ2A2o6d6Yks5sMoEogaJpZM4OTVHv
.

Yeah it's the crash case I'm thinking about.  For instance, Jenkins caught a segfault in Jordan's snarl patch that Travis didn't.  But he was unable to reproduce in gdb without a fair bit of help.


Repainting the user interface after certain changes take place

Seems that PySide2 needs to have some extra "repaint()" or "update()" or refresh() or something called after certain UI elements are changed. This can lead to a lot of user confusion during operation because the user will think they did something but get no feedback from the UI and then try the same thing again.

I have been able to reproduce on MacOS, Windows 10 and Linux.

Is this a case of data in one of the models changing without a dataChanged signal being emitted? What is the step by step scenario to reproduce this? What UI elements are not being updated correctly?

In the HyperThoughtDialogImp.py file comment out lines 58,59 and 60. Then using the "Key" button to paste what ever is on your clipboard. If nothing appears, then just try to resize the dialog and then the pasted content appears.

I was able to reproduce on Windows 10 and MacOS and I have seen it happen on Raheem's screen. I'm just not sure what the underlying issue is.

https://bugreports.qt.io/browse/QTBUG-68740 looks like was not really fixed.
Using the "Fusion" style or applying our own style helps fix the issue.

#12 is related to this bug. By implementing a style this issue will be fixed.

Fixed if using the "Dark Visual Style" on macOS


Should an AdminSet manager automatically get the approving roles for the workflows?

This would save an extra setup step.

Yes. See #692


log_validation function error.

I found that log_validation function in train.py does not contain image_encoder, image_processor modules. So, how it works to validate joint image and text prompts as inputs. @yunkchen @bubbliiiing

I will find a time to make the necessary modifications

Thank you very much.


Factory Reset Geolocation is not working

Hi,
I using the SampleApp to test on the android. I faced an issue, where i have factory reset the android s5, open the sampleApp app, click locate and start the tracking, the tracking is not working(android setting is all in default mode, without google account).
Then i added google account to the android, the app is working. May i know why? Do users requires to setup google account on the phone then this background geolocation will only works?

Did you used the ANDROID_DISTANCE_FILTER provider or ANDROID_FUSED_LOCATION?
Did you also install any software upgrades, like Android System WebView or Google Play Services?

@freezyfridge Any updates?

No response for 16 days. Closing, reopen if necessary.


⚠ Spryr has degraded performance

In 5a6f616, Spryr (https://spryr.com) experienced degraded performance:

HTTP code: 200
Response time: 5262 ms

Resolved: Spryr performance has improved in 3c1d61c after 22 minutes.


Add rule for multiplication node

Let us consider the scenario when we need to multiply univariate gaussian with multivariate point-mass. One example of this can be illustrated by adding a 'control' variable u to the first component of a state vector x=[x1, x2]^T, i.e.

ForneyLab implements the outgoing (green) message i.e.
function ruleSPMultiplicationOutNGP(msg_out::Nothing, msg_in1::Message{F, Univariate},
msg_a::Message{PointMass, Multivariate}) where F<:Gaussian

However, the backward (red) message is missing.
This PR extends multiplication node by adding the backward message towards univariate gaussian
function ruleSPMultiplicationIn1GNP(msg_out::Message{F, Multivariate}, msg_in1::Nothing,
msg_a::Message{PointMass, Multivariate}) where F<:Gaussian

Thanks for the reply @ThijsvdLaar
Maybe I didn't get something, but I think we aren't on the same page.
In the specification that I showed we have:
    | a ~ Multivariate{Pointmass}
    v  out ~ Multivariate
-->[x]-->
in1 ~ Univariate

ForneyLab has the rule for computing message out which is a pointwise multiplication of multivariate pointmass a by a scalar random variable in1:
function ruleSPMultiplicationOutNGP(msg_out::Nothing, msg_in1::Message{F, Univariate},
msg_a::Message{PointMass, Multivariate}) where F<:Gaussian

The rule I've added allows computing msg_in1 (backward message) when msg_out is available:
    | a ~ Multivariate{Pointmass}
    v  out ~ Multivariate
<--[x]<--
in1 ~ Univariate

After discussion with @albertpod we decided to view the Multivariate*Univariate updates as an explicit special case of Ax=y that is implemented for convenience.


Feat: Make radio group focus request optional

Hi, thanks for the great library!
Could be possible to turn off requestFocus with a parameter? The reason is to avoid scrolling to selected radio on change as this might be a desired behaviour. Maybe something like that?
  onChanged: (val) {
   if(requestFocusOnChange) {
       state.requestFocus();
    }
    state.didChange(val);
  },

Thanks again!

Not only for radio group but also to chips form. This mandatory request focus embarrass with forms which are able to scroll. I've implemented an optional attribute, so we can decide whether an option should request focus.

This should also be implemented for Dropdowns, as I am having a similar issue. Dropdown, when changed, requests focus and moves to the top of the screen and moving the scroll view. This is annoying for users, and not good practice if you aren't completing a form in the same order (top form component to bottom form component) each time.

On free time I will check it and maybe implement on https://github.com/danvick/flutter_form_builder/pull/881 if on moment it is not merged yet

With @hazzo we found a temporary solution for this problem (that also happens with FormBuilderDateTimePicker).
We use FocusScope.of(context).requestFocus(FocusNode()) after focus action

Check if https://github.com/danvick/flutter_form_builder/pull/881 solvs

Can you please confirm if flutter_form_builder: ^7.1.0 solves it?

Closing this issue because it seems to be resolved. In case not, feel free to open again/create a new one.
Thanks for contributing.


Fix volatile pufferfish spell id

This spell was recently hotfixed by Blizzard and now it finally does damage but under a different spell ID.

Changes are also in #28.


Free hosting idea

From the API spec, it appears it only supports GET all records, or one record by type. Therefore, this dataset could be traversed and all possible queries could be written as JSON files. Then, using Github pages or Cloudflare pages, you could serve all these JSON files for free on a GH/CF domain, or on a custom domain you provide.
Also, if the clients of this data are only web browsers, then you could use this strategy https://phiresky.github.io/blog/2021/hosting-sqlite-databases-on-github-pages/ to async query a static hosted (free) SQLite DB

Yes, there's no need for the SQLite database really, as we are effectively serving static data... if you are feeling like it @jpillora , you could make an MR that deploys to github pages using some action e.g. this one and gets rid of the database.

No time unfortunately, just thought I'd throw the idea out there :+1:


Re-work dataset and snapshot parsing

I originally set out to make the process of getting everything out of ZFS multi-threaded, but, as it turns out, that's pretty unnecessary. It also adds complexity, especially in debugging and trying to interpret logs, since output order to the log is non-linear.
However, even if the thread worker pool is restricted to a single thread, issuing separate commands to get pools, then datasets in those pools, and then snapshots, with 5 test pools each having 200 volumes/filesystems up to 4 levels deep in the tree, plus snapshots (for a total of over 6,000 ZFS objects, resulting in over 2 million total lines parsed), SnapsInAZfs still parsed all that data, while also performing trace logging to console and disk, in just over 2 seconds, on the system I tested it on, which is a 6 year old server running RHEL 8, and no particularly fancy hardware, while it was handling its other duties. And a significant portion of that time was just waiting for the zfs get process to collect its data and dump it to stdout.
So, to simplify things, I'm going to replace the code that spawns all the worker threads with one of these two options:

One single loop that just gets all objects from ZFS in one big zfs get operation
Two operations - one to get datasets and one to get snapshots

Each has pros and cons, but they're likely VERY negligible in the vast majority of cases, considering how quickly parsing happens.
I'll note which solution I end up choosing in this issue when I make that decision.

I went with the one-shot approach.
This is now implemented.


Vstanciu

Too many problems with code style. Please run eslint and fix all problems.

fixed

@rapi could you please review?


Fix sorted blocks in restoreHeadState()

It's not a fix for #1045 because the printed values are the same, however it's worth to fix.

curious how can we do it? I thought leveldb only sort based on keys but in this case its blockRoot ...

curious how can we do it? I thought leveldb only sort based on keys but in this case its blockRoot ...

Oh yeah true, they aren't sorted then probably


Remove recent resignations

Doctrine: https://groups.google.com/forum/#!topic/php-fig/3PGhfQGoCj8
Guzzle: https://groups.google.com/forum/#!topic/php-fig/mnqZ5ztS-pM
Laravel: https://groups.google.com/forum/#!topic/php-fig/HWX2IeiRx9Y
Propel: https://groups.google.com/forum/#!topic/php-fig/HgjJJXhDYsc
sabre/dav: https://groups.google.com/forum/#!topic/php-fig/FV4aL-noVZk

:cry:

Thanks @dragoonis :bow:

Nae danger :)

My soul has been released!


curl: (6) Could not resolve host: launchermeta.mojang.com

Hello, when creating your Docker Container in Container Station on my QNAP NAS and I choose Bridge mode with custom local IP, I always get this error:
curl: (6) Could not resolve host: launchermeta.mojang.com
This doesn't happen when I choose NAT mode in network settings. I read through many issues in this repo, but none of them helped me. I even tried SKIP_NSSWITCH_CONF which doesn't help either. Thanks in advance.

I'm not aware of anything I can do to address this. I would prefer to close this, but welcome any others to find and add to this issue as more is known.

I ran into the same curl error but was running in an instance of microK8s on Ubuntu 20.04 on an Intel NUC.
After reading through #756 I spun up a container from the adoptopenjdk/openjdk11:alpine-jre image & found that I couldn't resolve any DNS names.
After a bunch of googling I found this Stack Overflow post which summarizes what I found.
TL;DR It's caused by Alpine & the musl library not playing nice with Kubernetes ndots (see resolv.conf) strategy. Probably also has something to do with my network's specific DNS environment.
Switching to the multiarch tag which is based on Debian instead of Alpine fixed my issues.

@pahansen95 Glad to hear the Debian based image resolved the issue for you. As a counter-datapoint I run primarily on microk8s on Ubuntu 20.04 and the alpine based image works fine for me; however, I'm using standard DNS provided by my ASUS router.

That's good to know @itzg. Do you have the coreDNS Addon enabled for microK8s?
I'm think it's specifically my microk8s environment. I don't have any issues if I spin up a adoptopenjdk/openjdk11:alpine-jre container using docker on the host.
@M4TY Does switching to the multiarch image resolve your issue as well?

Do you have the coreDNS Addon enabled for microK8s?

I double checked and yes, I do.

Please excuse my late post, but I am having the exact same issue on K8s (Debian host) with both the Deb and Alpine-based images. The image was spun up in Rancher.

I had related troubles, and worked around my bad default k3s pod config like this (<IP_ADDRESS> is my non-cluster dns):
apiVersion: apps/v1
kind: Deployment
spec:
  replicas: 1
  template:
    spec:
      dnsPolicy: None
      dnsConfig: {nameservers: [<IP_ADDRESS>]}
      containers:
        - name: minecraft
          image: itzg/minecraft-server:2021.17.0-java16-openj9
etc

Same issue @MegaShinySnivy I tried both multiarch and latest.  Using Rancher k3s

@drewp 's solution worked for me, but this option is not in the helm chart, so not ideal


Travis: Use latest stable - jruby-<IP_ADDRESS>

This PR updates JRuby to latest stable.
Thanks for keeping JRuby in your test suite!

Thank you for updating it!


Unhelpful error message when git branch does not exist.

I had problems with failed function app deployments which had a sourcecontrols resource.
The branch name was of the form 'feature/do-something-new' which I eventually realised needed to be escaped (i.e. "branch": "[replace(variables('branch'), '/', '/')]",). I was also able to replicate the deployment error by using a branch name which didn't exist (masterxxx).
My issue is specifically with the error message appearing in the activity log in the portal which was the uninformative:
        "statusCode": "Conflict",
        "serviceRequestId": "716e6819-cdfe-4fc5-b34e-d1528f1f4b60",
        "statusMessage": "{\"status\":\"Failed\",\"error\":{\"code\":\"ResourceDeploymentFailure\",\"message\":\"The resource operation completed with terminal provisioning state 'Failed'.\"}}"

Interestingly when opening the function app in the portal after the failed deployment with the unescaped 'feature/do-something-new' branch name the source code and functions were visible but not when opening the app following deployment using the unknown branch masterxxx.
The working template can be seen here

I'm not convinced it is a escaping character issue now as I have failing deployments even with the code change described above.
When looking at the continuous integration details for the function app I discovered some further logging for the failed github deployment:
Command: "D:\home\site\deployments\tools\deploy.cmd"
Handling node.js deployment.
KuduSync.NET from: 'D:\home\site\repository' to: 'D:\home\site\wwwroot'
Copying file: '.eslintignore'
Copying file: '.eslintrc.json'
Copying file: '.gitignore'
Copying file: '.istanbul.yml'
Copying file: '.tern-project'
Copying file: 'host.json'
Copying file: 'package.json'
Copying file: 'README.md'
Copying file: 'A_SplitFile\function.json'
Copying file: 'A_SplitFile\index.js'
Copying file: 'A__A_ProcessFirstSplitFile\function.json'
Copying file: 'A__A_ProcessFirstSplitFile\index.js'
Copying file: 'A__A_ProcessSecondSplitFile\function.json'
Copying file: 'A__A_ProcessSecondSplitFile\index.js'
Copying file: 'B_GenerateUrl\function.json'
Copying file: 'B_GenerateUrl\index.js'
Copying file: 'C_Download\function.json'
Copying file: 'C_Download\index.js'
Copying file: 'D_TransformAndStore\function.json'
Copying file: 'D_TransformAndStore\index.js'
Copying file: 'D_TransformAndStore\transform.js'
Copying file: 'scripts\build.sh'
Copying file: 'scripts\edispensary-50.csv'
Copying file: 'scripts\template.json'
Copying file: 'test\unit\transform.js'
Looking for app.js/server.js under site root.
Node.js versions available on the platform are: 0.6.20, 0.8.2, 0.8.19, 0.8.26, 0.8.27, 0.8.28, 0.10.5, 0.10.18, 0.10.21, 0.10.24, 0.10.26, 0.10.28, 0.10.29, 0.10.31, 0.10.32, 0.10.40, 0.12.0, 0.12.2, 0.12.3, 0.12.6, 4.0.0, 4.1.0, 4.1.2, 4.2.1, 4.2.2, 4.2.3, 4.2.4, 4.3.0, 4.3.2, 4.4.0, 4.4.1, 4.4.6, 4.4.7, 4.5.0, 4.6.0, 4.6.1, 5.0.0, 5.1.1, 5.3.0, 5.4.0, 5.5.0, 5.6.0, 5.7.0, 5.7.1, 5.8.0, 5.9.1, 6.0.0, 6.1.0, 6.2.2, 6.3.0, 6.5.0, 6.6.0, 6.7.0, 6.9.0, 6.9.1, 6.9.2, 7.0.0, 7.1.0, 7.2.0, 7.3.0.
Selected node.js version 7.3.0. Use package.json file to choose a different version.
Selected npm version 3.10.10
Updating iisnode.yml at D:\home\site\wwwroot\iisnode.yml
Missing server.js/app.js files, web.config is not generated
Thread was being aborted.
Thread was being aborted.

Interestingly if I then hit the Redeploy button in the portal it is successful
Using cached version of deployment script (command: 'azure -y --no-dot-deployment -r "D:\home\site\repository" -o "D:\home\site\deployments\tools" --node --sitePath "D:\home\site\repository"').

Command: "D:\home\site\deployments\tools\deploy.cmd"
Handling node.js deployment.
KuduSync.NET from: 'D:\home\site\repository' to: 'D:\home\site\wwwroot'
Looking for app.js/server.js under site root.
Missing server.js/app.js files, web.config is not generated
Node.js versions available on the platform are: 0.6.20, 0.8.2, 0.8.19, 0.8.26, 0.8.27, 0.8.28, 0.10.5, 0.10.18, 0.10.21, 0.10.24, 0.10.26, 0.10.28, 0.10.29, 0.10.31, 0.10.32, 0.10.40, 0.12.0, 0.12.2, 0.12.3, 0.12.6, 4.0.0, 4.1.0, 4.1.2, 4.2.1, 4.2.2, 4.2.3, 4.2.4, 4.3.0, 4.3.2, 4.4.0, 4.4.1, 4.4.6, 4.4.7, 4.5.0, 4.6.0, 4.6.1, 5.0.0, 5.1.1, 5.3.0, 5.4.0, 5.5.0, 5.6.0, 5.7.0, 5.7.1, 5.8.0, 5.9.1, 6.0.0, 6.1.0, 6.2.2, 6.3.0, 6.5.0, 6.6.0, 6.7.0, 6.9.0, 6.9.1, 6.9.2, 7.0.0, 7.1.0, 7.2.0, 7.3.0.
Selected node.js version 7.3.0. Use package.json file to choose a different version.
Selected npm version 3.10.10
Updating iisnode.yml at D:\home\site\wwwroot\iisnode.yml
Finished successfully.

I'm pretty sure that this error was happening because of a missing dependsOn for sourcecontrols - it seems to have needed both Sites and Sites/config declared as dependencies. Closing this in view of my findings.
Error message could have been more helpful though :)


Is it possible to use this library from Objective-C?

Hello, is it possible to use this library from Objective-C?

Afraid not. It's heavy reliance on Swift's enum functions make it impossible to back port.


Latest 0.2 release on crates.io does not compile with nightly anymore

A newer version of the proc-macro2 crate is needed, and https://github.com/rust-lang-nursery/futures-rs/pull/1003 and possibly https://github.com/rust-lang-nursery/futures-rs/pull/1025

@sdroege what nightly are you using? I tried this locally and saw the following:
evan.simmons at esimm-mb15tbr in ~/rust/futures-rs (tags/0.2.1)
$ rustup show
Default host: x86_64-apple-darwin

installed toolchains
--------------------

stable-x86_64-apple-darwin
nightly-x86_64-apple-darwin (default)

active toolchain
----------------

nightly-x86_64-apple-darwin (default)
rustc 1.28.0-nightly (524ad9b9e 2018-05-29)

evan.simmons at esimm-mb15tbr in ~/rust/futures-rs (tags/0.2.1)
$ cargo build
    Updating registry `https://github.com/rust-lang/crates.io-index`
 Downloading syn v0.13.11
   Compiling futures-macro-await v0.2.1 (file:///Users/evan.simmons/rust/futures-rs/futures-macro-await)
   Compiling futures-core v0.2.1 (file:///Users/evan.simmons/rust/futures-rs/futures-core)
   Compiling proc-macro2 v0.3.8
   Compiling futures-channel v0.2.1 (file:///Users/evan.simmons/rust/futures-rs/futures-channel)
   Compiling futures-io v0.2.1 (file:///Users/evan.simmons/rust/futures-rs/futures-io)
   Compiling futures-sink v0.2.1 (file:///Users/evan.simmons/rust/futures-rs/futures-sink)
   Compiling futures-util v0.2.1 (file:///Users/evan.simmons/rust/futures-rs/futures-util)
   Compiling quote v0.5.2
   Compiling syn v0.13.11
   Compiling futures-executor v0.2.1 (file:///Users/evan.simmons/rust/futures-rs/futures-executor)
   Compiling futures-stable v0.2.1 (file:///Users/evan.simmons/rust/futures-rs/futures-stable)
   Compiling futures-async-runtime v0.2.1 (file:///Users/evan.simmons/rust/futures-rs/futures-async-runtime)
   Compiling futures v0.2.1 (file:///Users/evan.simmons/rust/futures-rs/futures)
   Compiling futures-macro-async v0.2.1 (file:///Users/evan.simmons/rust/futures-rs/futures-macro-async)
    Finished dev [unoptimized + debuginfo] target(s) in 16.15s

With latest nightly as of today. You have to enable the nightly feature though.
Inside the futures crate (not the workspace):
$ cargo +nightly build --features nightly
   Compiling futures-core v0.2.1 (file:///home/slomo/Projects/rust/futures-rs/futures-core)
   Compiling proc-macro2 v0.3.8
error[E0433]: failed to resolve. Could not find `Pin` in `mem`
   --> futures-core/src/future/mod.rs:150:35
    |
150 |             unsafe { ::core::mem::Pin::get_mut(&mut self.as_pin()).poll(cx) }
    |                                   ^^^ Could not find `Pin` in `mem`

error[E0433]: failed to resolve. Could not find `Pin` in `mem`
  --> futures-core/src/stream/mod.rs:92:35
   |
92 |             unsafe { ::core::mem::Pin::get_mut(&mut self.as_pin()).poll_next(cx) }
   |                                   ^^^ Could not find `Pin` in `mem`

error[E0599]: no method named `as_pin` found for type `&mut std::boxed::PinBox<F>` in the current scope
   --> futures-core/src/future/mod.rs:150:58
    |
150 |             unsafe { ::core::mem::Pin::get_mut(&mut self.as_pin()).poll(cx) }
    |                                                          ^^^^^^

error[E0599]: no method named `as_pin` found for type `&mut std::boxed::PinBox<S>` in the current scope
  --> futures-core/src/stream/mod.rs:92:58
   |
92 |             unsafe { ::core::mem::Pin::get_mut(&mut self.as_pin()).poll_next(cx) }
   |                                                          ^^^^^^

error: aborting due to 4 previous errors

Some errors occurred: E0433, E0599.
For more information about an error, try `rustc --explain E0433`.
error: Could not compile `futures-core`.
warning: build failed, waiting for other jobs to finish...
error[E0433]: failed to resolve. Could not find `Op` in `proc_macro`
   --> /home/slomo/.cargo/registry/src/github.com-1ecc6299db9ec823/proc-macro2-0.3.8/src/unstable.rs:131:42
    |
131 |                 let mut op = proc_macro::Op::new(tt.op(), spacing);
    |                                          ^^ Could not find `Op` in `proc_macro`

error[E0433]: failed to resolve. Could not find `Term` in `proc_macro`
   --> /home/slomo/.cargo/registry/src/github.com-1ecc6299db9ec823/proc-macro2-0.3.8/src/unstable.rs:446:59
    |
446 |             Span::Nightly(s) => Term::Nightly(proc_macro::Term::new(string, s)),
    |                                                           ^^^^ Could not find `Term` in `proc_macro`

error[E0412]: cannot find type `Term` in module `proc_macro`
   --> /home/slomo/.cargo/registry/src/github.com-1ecc6299db9ec823/proc-macro2-0.3.8/src/unstable.rs:439:25
    |
439 |     Nightly(proc_macro::Term),
    |                         ^^^^ not found in `proc_macro`
help: possible candidates are found in other modules, you can import them into scope
    |
3   | use Term;
    |
3   | use imp::Term;
    |
3   | use stable::Term;
    |

error[E0412]: cannot find type `Term` in module `proc_macro`
   --> /home/slomo/.cargo/registry/src/github.com-1ecc6299db9ec823/proc-macro2-0.3.8/src/unstable.rs:473:44
    |
473 |     fn unwrap_nightly(self) -> proc_macro::Term {
    |                                            ^^^^ not found in `proc_macro`
help: possible candidates are found in other modules, you can import them into scope
    |
3   | use Term;
    |
3   | use imp::Term;
    |
3   | use stable::Term;
    |

error[E0599]: no function or associated item named `empty` found for type `proc_macro::TokenStream` in the current scope
  --> /home/slomo/.cargo/registry/src/github.com-1ecc6299db9ec823/proc-macro2-0.3.8/src/unstable.rs:45:34
   |
45 |             TokenStream::Nightly(proc_macro::TokenStream::empty())
   |                                  ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ function or associated item not found in `proc_macro::TokenStream`

error[E0599]: no variant named `Op` found for type `proc_macro::TokenTree` in the current scope
   --> /home/slomo/.cargo/registry/src/github.com-1ecc6299db9ec823/proc-macro2-0.3.8/src/unstable.rs:228:13
    |
228 |             proc_macro::TokenTree::Op(tt) => {
    |             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ variant not found in `proc_macro::TokenTree`

error[E0599]: no variant named `Term` found for type `proc_macro::TokenTree` in the current scope
   --> /home/slomo/.cargo/registry/src/github.com-1ecc6299db9ec823/proc-macro2-0.3.8/src/unstable.rs:237:13
    |
237 |             proc_macro::TokenTree::Term(s) => ::Term::_new(Term::Nightly(s)).into(),
    |             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ variant not found in `proc_macro::TokenTree`

error: aborting due to 7 previous errors

Some errors occurred: E0412, E0433, E0599.
For more information about an error, try `rustc --explain E0412`.
error: Could not compile `proc-macro2`.

stale


support for llama-3-70B

Hi, such an amazing work!
I am following closely on your progress and already using the new bot function feature which works great!
I find the LLM a bit insufficient for more complex reasoning though, do you plan to add support to llama-3-70B as well?. I tried running it (on the AGX) but it gave me garbled output.

@guyo-shifters I noticed this as well, I think I probably need to update MLC.  I also got AWQ working again which should be able to run it.  For the time being you could still use llama-2-70b until this is worked out.

thanks, I'll eagerly wait for the update :)
It seems there is a big difference between llama-2 and llama-3
also there might be a problem of additive lag with the 70-B models and the riva server, I think maybe it is waiting for all of the text to be transcribed for TTS even if the prompt was skipped resulting in long silent periods. I might be wrong though.
keep up the good work!

I was able to run llama-3 70B with text-generation-webui, from an AWQ version, specifically https://huggingface.co/PrunaAI/Meta-Llama-3-70b-instruct-AWQ-smashed.

Thanks! I don't have experience running AWQ yet but I'll give it a go

is it something like this:
jetson-containers run --env HUGGINGFACE_TOKEN=XXX   $(autotag nano_llm)   python3 -m nano_llm.agents.web_chat --api=awq     --model PrunaAI/Meta-Llama-3-70b-instruct-AWQ-smashed     --asr=riva --tts=piper
seems to fail on "cannot import name 'AWQModel' from 'nano_llm.models'"


Unsupported Swift Version

After pod install, when open on Xcode 13.2.1, show the following alert.

The target “AKImageCropperView” contains source code developed with Swift 3.x. This version of Xcode does not support building or migrating Swift 3.x targets.
Use Xcode 10.1 to migrate the code to Swift 4.

Need to update pod spec.

deployment_target: 8.0 -> 9.0
https://github.com/artemkrachulov/AKImageCropperView/blob/3436f26c9da545aaabd650616bf757c31fac2f42/AKImageCropperView.podspec#L16

swift_version: 3.0 -> 5.0
https://github.com/artemkrachulov/AKImageCropperView/blob/3436f26c9da545aaabd650616bf757c31fac2f42/AKImageCropperView.podspec#L24

refs #38 #40


Update strings.js

Seems like your string is not equal with https://raw.githubusercontent.com/adobe/brackets/master/src/nls/root/strings.js. (Mine is).
Major changes, especially "berkas" which was changed into "file".
Reasons:

Most people don't know what "berkas" means in the digital sense (although I use "berkas" in academic stuff)
Google Chrome uses "file" too

For maximum consistensy, please re-review again

indentasi -> inden
Any references?

I remember seeing it on Google Docs
On Thu, Mar 19, 2015, 7:39 PM Resi Respati<EMAIL_ADDRESS>wrote:

indentasi -> inden
Any references?
—
Reply to this email directly or view it on GitHub
https://github.com/resir014/brackets/pull/2#issuecomment-83539230.

Alright, everything looks good. Merged.
If there are any other things that needs changing I can just commit them directly.


Pkg: Update column gutter to 32px

This is a global setting.
http://localhost:3030/layout

@LeoT7508 Is this correct?

@xak
Yes 32px for gutters is correct.


Add monochrome icon

Currently, the Zulip app on Android has no monochrome icon, making it look very out of place on modern homescreens.

Thanks for the feedback!
It looks like the feature you're using here is "themed icons". In the system settings on my Pixel 8 running Android 14, the switch to enable that feature is labeled as "Beta". Turning it on and browsing around a bit, it looks like some apps have provided these monochrome launcher icons but most apps haven't yet.
You're certainly not alone in wishing that more apps would — here's a couple of different threads of people complaining that so many apps haven't:
https://www.reddit.com/r/GooglePixel/comments/17ceeiz/themed_icons_why_is_it_so_difficult/
https://www.reddit.com/r/android_beta/comments/16chny5/wasnt_a14_supposed_to_fix_themed_icons/
Anyway, for our reference when we go to implement this feature, here's the developer docs for it (which also covers the related #4211):
https://developer.android.com/develop/ui/views/launch/icon_design_adaptive
Because this codebase is in maintenance mode, this is a feature we'd only add in the upcoming Flutter-based app. But I definitely expect we'll implement it there at some point after that migration is complete.


Improve scale fitting logic and performance

core.layouts calls scale.update multiple times. This is unnecessary. We do not need to calculate the ticks multiple times for example. We should instead call fit rather than update
fit, autoSkip, and calculateTickRotation do not work well together

There's some trade-off between auto-skipping and label rotation.  It's unclear how these relate. Clarification proposed in https://github.com/chartjs/Chart.js/pull/6079
_autoSkip should probably happen in update rather than draw. Right now fit assumes all ticks will be drawn, which isn't true when autoSkip: true

I agree to:

fit instead of updated when determining layout
_autoSkip (and rotate) in update.

I don't think that auto skipping and auto rotation should be mutually exclusive. because after rotating, there might still be the need to skip.

Closed by https://github.com/chartjs/Chart.js/pull/6304


Add support for newer C++ language standards.

This adds support for two newer C++ standards.

What do I need to do here w.r.t. package versioning?

You'll need availability annotations. I'm assuming we target 5.3 for this?
@available(_PackageDescription, introduced: 5.3)
case cxx17 = "c++17"

@swift-ci please smoke test

Looks good, if this is also going to be merged into the 5.3 branch.  If it's just for master, the availability should be vNext.

Trunk is fine, will change to vNext.

@swift-ci please smoke test


feat(connections): Move the globe outside

As discussed in #213 this removes the globe from the
connections window and replaces it with a link.
In addition the peers list is transformed into a table.
Closes #213

Currently missing is the globe which is being developed here: https://github.com/ipfs/connections-globe
There are no locations listed as ipfs-geoip is simply to slow and unstable to use for this many peers.

Preview

@Dignifiedquire

oh i thought we were going to keep a visual too -- what about the 2d map you had in another branch? i think we should have that. the list on its own is not very useful. (and yes link to the globe outside)
i like the peers table, much nicer. the ID should be the full thing, even if it is hidden by the other column. should be selectable and copy-pastable (the whole thing) (also use the markdown ticks / html code? like this)
why is geoip so slow? it can fetch in the background, no? it can start without locations but over time add them? (would be nice to have flags like bittorrent clients)

@jbenet tldr one thing at a time

oh i thought we were going to keep a visual too -- what about the 2d map you had in another branch? i think we should have that. the list on its own is not very useful. (and yes link to the globe outside)

This is not implemented yet, and yes it will come in the future but for now we just want peoples battery life to be a bit happier.

i like the peers table, much nicer. the ID should be the full thing, even if it is hidden by the other column. should be selectable and copy-pastable (the whole thing) (also use the markdown ticks / html code?

Okay will do

why is geoip so slow? it can fetch in the background, no? it can start without locations but over time add them?

It's not only slow but also unreliable and not available on the 0.4 network. I agree we want flags and nice locations but I would like to drop it from this list for now and figure out how to make location detection better and more reliable and when that's done adding it back.

It's not only slow but also unreliable and not available on the 0.4 network. I agree we want flags and nice locations

We should add it to the 0.4 network, it has to be added there.

but I would like to drop it from this list for now and figure out how to make location detection better and more reliable and when that's done adding it back.

historically, our community tends to do this and then take months adding things back. using geoip is very useful in demos. Showing where things are is a big deal. And in my experience always reliable -- it is indeed slow the first times, but once you have the core of the tree cached it's fine. Do not remove this feature. (i think i use the webui locations tab for demos more than any other user in the network right now)
If you want to not load it, then make it a toggle checkbox that people can turn off. it will be redone shortly to add the 2d map anyway.

Also, the "people's batteries" thing is really a fault of the content not being removed + garbage collected properly. it should be just fine for most mobile phones to load this tab and move out in 10s and not have any noticeable effect on battery

Will be resolved in #229


New concepts for trader and markets and preliminary definitions

Description of the issue
Here are some new concepts for trader and markets and preliminary definitions.
Ideas of solution
Markup: A markup is added to the total cost incurred by the producer of a good or service (here electricity producers) in order to cover the costs and/or create a profit. Serves to distinguish power plants in our models.
Conventional trader: Traders are individuals who engage in buying and selling of goods, commodities etc. for themselves or an institution. Conventional traders are restricted to conventional plants.
Demand trader: a fictitious synthesis for all demand in a particular country during a time period.
Direct marketer: is the party between power plant owners or portfolio holders and the energy exchange. They facilitate the buying and selling of electricity.
Message: Discrete unit of communication.
Signal: anything that indicates that a communication should be started
Channel: construct to bundle similar messages
Product: thing made available for consumer use; it is anything that can be offered to a market
Portfolio: a portfolio is a collection of power plants held by a portfolio holder, company, institution or individual.
Merit order: The merit order is a way of ranking available sources of energy, especially electrical generation, based on ascending order of price (which may reflect the order of their short-run marginal costs of production) together with amount of energy that will be generated.
Workflow checklist

[x] I discussed the issue with someone else than me before working on a solution
[x] I already read the latest version of the workflow for this repository
[ ] I added this issue to the Project 'Issues'. If suitable, I add it to further Projects.
[x] The goal of this ontology is clear to me

I am aware that

[x] every entry in the ontology should have a definition
[x] classes should arise from concepts rather than from words

From oeo-dev-meeting 8.2:
Definition for trader (taken from fibo, adapted): An agent that engages in the transfer of financial assets in any financial market on behalf of a client or the financial services provider
parent class: oeo:policy
fibo-equivalent: fibo-fbc-pas-fpas:Trader
in fibo, market is a synonym for exchange, see #341


Make the module compatible with terraform v0.7.0

PR no longer required. Branch kept open for reference.


npm Publishing Permission

@adambom would be great if you could grant me permission to publish to npm. I don't have anything to publish now but for the future this will be very useful.

Hi @amilajack
I recommend we get the lib ready to next release (as soon as tests = OK ) and then contact adambom directly so he can see that we are getting some results and feel good about adding more people as npm contributers


View arguments

View arguments don't work unless there is a slash after them, even when they are at the end of the route. For example:
app = Flask(__name__)

@app.route('/view/<article_id>')
def view_article(article_id):
    return article_id

fails to work, going to localhost:5000/view/1 gives a 404 error. On the other hand:
app = Flask(__name__)

@app.route('/view/<article_id>/')
def view_article(article_id):
    return article_id

works perfectly well, notice that the only difference is the / at the end of the app.route.
Environment:

Python version: 3.9.6
Flask version: 2.0.1

I can't reproduce this issue. Both routes work as expected.


API and site terms

Hey folks.
We've been encountering issues with a lot of bots using this to scrape Unsplash and use the photos, which unfortunately is against the terms and the license of Unsplash.
We make all of the images available via the API so that developers can build with the library in a way that's consistent with our contributor's wishes. I'd ask that out of respect for their contributions that you remove this repo from Github. While we love seeing people build things with Unsplash, we have a better and more scalable way for people to build with Unsplash via the API that aligns with the motivations of our community.
Thanks for understanding @crisbeto

I'm sorry @lukechesser, this was a side project that I did a long time ago, I wasn't expecting it to still be working or people to be using it. I'll make the repo private and unpublish the package from npm.

Thanks @crisbeto, really appreciate that 👍


feat(runtime-vapor): component attrs

closes: #48
Summary
I have implemented attrs.
This branch is derived from the branch for implementing emits.
Please merge the emits PR first.

main

https://github.com/vuejs/core-vapor/pull/40

https://github.com/vuejs/core-vapor/pull/42

https://github.com/vuejs/core-vapor/pull/47 <- current

https://github.com/vuejs/core-vapor/pull/54

Points I'm unsure about

inheritAttrs
Should be inheritAttrs option is implemented?
If it seems necessary, I'll address it in a separate PR. (I've added base issue Todo)

How to confirm the operation
Since the compiler implementation is not yet available, I have written the expected JavaScript code as the compilation result in playground/src/attrs.js.

code
import { watch } from 'vue'
import {
  children,
  on,
  ref,
  template,
  effect,
  setText,
  render as renderComponent // TODO:
} from '@vue/vapor'

export default {
  props: undefined,

  setup(_, {}) {
    const count = ref(1)
    const handleClick = () => {
      count.value++
    }

    const __returned__ = { count, handleClick }

    Object.defineProperty(__returned__, '__isScriptSetup', {
      enumerable: false,
      value: true
    })

    return __returned__
  },

  render(_ctx) {
    const t0 = template('<button></button>')
    const n0 = t0()
    const {
      0: [n1]
    } = children(n0)
    on(n1, 'click', _ctx.handleClick)
    effect(() => {
      setText(n1, void 0, _ctx.count)
    })

    renderComponent(
      child,
      {
        get count() {
          return _ctx.count
        }
      },
      n0
    )

    return n0
  }
}

const child = {
  setup(_, { attrs }) {
    console.log('attrs: ', attrs)
    watch(
      () => attrs.count,
      v => console.log('attrs.count changed', v)
    )

    const __returned__ = {}

    Object.defineProperty(__returned__, '__isScriptSetup', {
      enumerable: false,
      value: true
    })

    return __returned__
  },

  render(_ctx) {
    const t0 = template('

')
    const n0 = t0()
    const {
      0: [n1]
    } = children(n0)
    effect(() => {
      setText(n1, void 0, _ctx.$attrs.count + ' * 2 = ' + _ctx.$attrs.count * 2)
    })
    return n0
  }
}

https://github.com/vuejs/core-vapor/assets/71201308/bd093d35-0dfa-40e7-a72a-af0a3b5c27d1

⚠  Mark as draft until Emits PR is merged.

Oh, I was under the impression that the render function would receive attrs as ctx, so I was trying to implement publicInstance, but it seems that this implementation was removed by this commit. What approach should I take for the implementation...? 🤔
https://github.com/vuejs/core-vapor/commit/efc32b716fece04e3c5e7035f2bffc0327d4b347


Selected highlight isn't clear in iTerm

Searching while using neovim in iTerm is significantly less convenient while using nova.  Here is how it looks using Nova:

While using another theme, it looks like this:

Cycling through my searches (n) is difficult.  I have to count to know which I have selected.
Highlighting appears to work properly in Hyper (though, it's still far more subtle than most color schemes), so I'm assuming this is an issue with iTerm.

This is how it looks in Hyper:

It looks like I might have something wrong with my colors (this isn't colored in iTerm).

Thanks for the feedback! I agree; I think search results should use novaColors.uiGroups.userCurrentState (cyan) as the foreground (text) and use a novaColors.uiGroups.gray3 (light gray) for the background. Feel free to submit a pull request using the "CONTRIBUTING" guidelines; otherwise, I'll get to this :)


[MNT] tensorflow-addons dependency in networks /_encoder.py

Describe the issue
Linked to #1144 and #1421 : tensorflow-addons dependency need to be removed.
This has probably just been forgotten from the large work in #1421.
        "python_dependencies": ["tensorflow", "tensorflow-addons"],

code ref
Suggest a potential alternative/fix
Remove the tensorflow-addons dependency.
Additional context
Searching for tensorflow-addons and tensorflow_addons with the search engine, I also find something in utils/networks, but I don't know what to do with it.
https://github.com/aeon-toolkit/aeon/blob/6bd3324624dda8835fa17579e6575c470d287c5a/aeon/utils/networks/__init__.py

@aeon-actions-bot assign @Cyril-Meyer


create a separate model for category

Figma for reference: https://www.figma.com/file/tfm2m2vGTdaZT8u5iQZS3B/Youtube-Playlists?node-id=631%3A0
In the design, we need to render sidebar/tabs with categories.
These categories should come from BE. Create a model for category and link it with playlist and video models.

@thesanjeevsharma Can a playlist have more than one category, in the old setup I just completed this field has a list of categories.

Well, a playlist can have more than one category for example - Python and Django.
But, let's hear @angle943 on this.

Yes, a playlist can have more than one category


support no-std

I'd actually like to use this lib in a no-std crate I'm developing, so here are some minimalistic changes to support no-std environments. The only bad part is that now, if the default std feature is explicitly disabled, we can panic during another panic because core has no way to check this like std does.
One detail that is not addressed by this PR: some no-std users might want to not use alloc, which is needed for alloc::borrow::Cow. So in the future we might want to add &'static str methods and add a different feature flag to enable alloc and the Cow<'static, str> versions. But I don't need that and it's an interface change, so I'll leave that to someone else to decide if needed.

:thinking: my gut feeling is that this crate is really time, more of a pattern than a crate even, so, if whatever is here doesn't not work for your use-case exactly, it might be better to just copy the code? I imagine no_std use-cases might want to customize panic behavior or  what not

To be honest I actually hand-rolled my own version for my own use case before I knew this crate existed, so that tracks. But I figured I'd contribute a change back since this crate almost worked in no-std anyway. But feel free to close if not needed


Export to android - godot 3.3 rc7

Godot version : 3.3 rc7

OS/device including version: Pop_Os 20.10

Issue description: Can't export to android, i'm using the same paths in editor/settings/export/android than godot 3.2.3 stable, in the export android tab shows : " invalid android sdk path", "unable to find android sdk" and related errors.
I can export normaly with the same paths with godot 3.2.3, i already download export templates for godot 3.3 rc7 from internet and from godot editor, reassign paths and redownload android components with same results

Steps to reproduce: export to android with godot 3.3 rc7

You need to configure the path to your Android SDK as mentioned in the error message.
See the updated documentation for 3.3: https://docs.godotengine.org/en/3.3/getting_started/workflow/export/exporting_for_android.html


delete mathlib Vector/Matrix

Ran full calibration and compared parameters before and after. No significant differences. Commander didn't require any additional stack either.

@PX4/testflights on a couple random boards (fmu-v2, fmu-v4, etc) could you please do some basic bench testing with this branch? Save your parameters from before (master baseline) and after a full calibration on this branch and we'll compare them.

Pixracer (V4)
fresh flash with master d40d165
reset all parameters to default and ran full setup
master.txt
Flashed this PR
reset all parameters to default and ran full setup
Mag calibration just hangs and does not complete; after waiting a few minutes, qgc flagges the fc as disconnected

These are the parameters with setup until getting stuck at mag calibration
pr_9181.txt

Definitely need more stack for the mag cal routine.

I'm looking forward to seeing this merged.

We can keep the mat_inverse for now and still get rid of the rest.

@dagar @RomanBapst We need to make sure to bring the Snapdragon fixes onto master and test this rebased onto them.

Rebased on master (with snapdragon fixes). Tested snapdragon basic usage and full recalibration of a pixracer.


Missed JSON changes for 1050 rebase

This commit adds the missed JSON changes when we
did a 1050 rebase.
Signed-off-by: Pavithra Barithaya<EMAIL_ADDRESS>

This PR fixes the vmi ip configuration issue and the guard feature too.


Filtered Environment Variables in API

Currently, environment variables are not filtered when getting a container through the API. A filter list would be handy, to help prevent exposure of secrets set as env vars.

While we don't have a better secrets management system to offer as an alternative yet, I would highly suggest not using environment variables like this; they aren't secret.
All the environment variables on a container are available to any other container that links to it or anyone with access to the host's docker daemon as standard Docker behavior.  We actually used to specifically not copy linked env vars in Rancher because (it's a terrible idea and) we have some additional control over the linking process vs standard Docker, but too many people's applications depend on this behavior.
So hiding them from the API would just be obfuscation.

@vincent99 - that makes sense. Right now, the using an external MySQL requires the use of environment variables, leading to me opening this issue. Is this something that will be remedied with secrets management and, if so, is there a working issue/PR for this?

It's on our roadmap, there's an issue you can track (https://github.com/rancher/rancher/issues/1269) but no concrete work yet.  There are a few possible ways go to about it, involving such things as rancher-metadata service that will be available soon, a special volume or volume-driver that provides the secrets as files (but can't be --volumes-fromed), and asymmetric encryption.
Providing a bridge from the secrets management to environment variables seems like a reasonable request.  If that didn't exist, it would still be trivial to run export MYSQL_PASSWORD=$(whatever_you_do_to_get_secrets "the_name_of_the_mysql_password_secret") before invoking the main $your_application process.

Ok. Great. I'll track and hopefully be able to contribute on that side! Closing this issue.


Initialize without loading?

Is it possible to initialize the plugin (so one can save) but without automatically loading data at this time?
By looking at jquery.phoenix.coffee it seems not possible but I might miss some option.

No, it is not possible by now. By you can make a PR.


Add missing qr codes and links for legacy apps

When moving from 8 to 9 somehow the QR codes and links weren't included.

I will merge this, then look into what caused this to be omitted during the migration. Thank you very much @stmarkidis


Update links to OWASP cheat sheet

OWASP has moved their cheatsheet to Github pages, so I've updated the qhelp links accordingly.

I'll merge this once the tests have finished.

I retriggered the tests since the status seemed to have been lost and the PR couldn't be merged.

It turns out that it was probably worth running the tests as the Java test failed with:
[ERROR] Input file ql/java/ql/src/semmle/code/java/security/XmlParsers.qll is not correctly formatted
Could you run the autoformatter on that file to fix whatever's upsetting the PR check?

This PR changes only qhelp files in the Python directories, so the test failure seems to be unrelated. Merging.


Disabled Prop not working in Web when used with react-native-web

Can you check if you can reproduce with the last version of @sharcoux/slider (npm i @sharcoux/slider)

Hello @hpurohit97!
Could you provide this issue with some reproduction example and the environment's information?

Closing due to no feedback from an author.


Markdown preview doesn't support img tag starting with /

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

Environment
Version: 1.70.2 (system setup)
Commit: e4503b30fc78200f846c62cf8091b76ff5547662
Date: 2022-08-16T05:35:13.448Z
Electron: 18.3.5
Chromium: 100.0.4896.160
Node.js: 16.13.2
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.22000
Steps to Reproduce

Write the following markdown.

[![](/yt.jpg)](https://github.com)

Place any image to workspace root as yt.jpg.
Open preview.

Expected behavior
Both should show preview image in the workspace root.
Actual behavior

Addition context
The following is developer tool screenshot of the preview page. I think this is a bug.

We closed this issue because we don't plan to address it in the foreseeable future. If you disagree and feel that this issue is crucial: we are happy to listen and to reconsider.
If you wonder what we are up to, please see our roadmap and issue reporting guidelines.
Thanks for your understanding, and happy coding!

Could you explain the reason why this is out of scope?


Add more tests

Description
Provide a brief description of the PR's purpose here.
Todos
Notable points that this PR has either accomplished or will accomplish.

[ ] TODO 1

Questions

[ ] Question1

Status

[ ] Ready to go

Codecov Report

Merging #6 (109fde9) into main (2d4ea35) will decrease coverage by 0.03%.
The diff coverage is n/a.


RenderComponent throws unexpected error for component rendered using RenderTreeBuilder

Describe the bug
When rendering a component via bUnit I get an error but when rendering in Blazor (both Server and WASM) the same component works fine.
Example:
A stripped down (only dependency is MudBlazor) version of the component is defined below:
using System;
using System.Collections.Generic;
using System.Diagnostics.CodeAnalysis;
using System.Threading.Tasks;
using Microsoft.AspNetCore.Components;
using Microsoft.AspNetCore.Components.Rendering;
using Microsoft.AspNetCore.Components.Web;
using MudBlazor;

namespace MyProject.ComponentRenderer;

/// <summary>
/// A component that renders another component dynamically according to its
/// <see cref="Type" /> parameter.
/// </summary>
public class DynamicComponentWithErrorBoundary : IComponent
{
    private RenderHandle _renderHandle;
    private readonly RenderFragment _cachedRenderFragment;

    /// <summary>
    /// Constructs an instance of <see cref="DynamicComponentWithErrorBoundary"/>.
    /// </summary>
    public DynamicComponentWithErrorBoundary()
    {
        _cachedRenderFragment = Render;
    }

    /// <summary>
    /// Gets or sets the type of the component to be rendered. The supplied type must
    /// implement <see cref="IComponent"/>.
    /// </summary>
    [Parameter]
    [DynamicallyAccessedMembers(DynamicallyAccessedMemberTypes.All)]
    [EditorRequired]
    public Type Type { get; set; } = default!;

    /// <summary>
    /// Gets or sets a dictionary of parameters to be passed to the component.
    /// </summary>
    // Note that this deliberately does *not* use CaptureUnmatchedValues. Reasons:
    // [1] The primary scenario for DynamicComponent is where the call site doesn't
    //     know which child component it's rendering, so it typically won't know what
    //     set of parameters to pass either, hence the developer most likely wants to
    //     pass a dictionary rather than having a fixed set of parameter names in markup.
    // [2] If we did have CaptureUnmatchedValues here, then it would become a breaking
    //     change to ever add more parameters to DynamicComponent itself in the future,
    //     because they would shadow any coincidentally same-named ones on the target
    //     component. This could lead to application bugs.
    [Parameter]
    public IDictionary<string, object>? Parameters { get; set; }

    /// <summary>
    /// Gets rendered component instance.
    /// </summary>
    public object? Instance { get; private set; }

    /// <inheritdoc />
    public void Attach(RenderHandle renderHandle)
    {
        _renderHandle = renderHandle;
    }

    /// <inheritdoc />
    public Task SetParametersAsync(ParameterView parameters)
    {
        // This manual parameter assignment logic will be marginally faster than calling
        // ComponentProperties.SetProperties.
        foreach (var entry in parameters)
        {
            if (entry.Name.Equals(nameof(Type), StringComparison.OrdinalIgnoreCase))
            {
                Type = (Type)entry.Value;
            }
            else if (entry.Name.Equals(nameof(Parameters), StringComparison.OrdinalIgnoreCase))
            {
                Parameters = (IDictionary<string, object>)entry.Value;
            }
            else
            {
                throw new InvalidOperationException(
                    $"{nameof(DynamicComponentWithErrorBoundary)} does not accept a parameter with the name '{entry.Name}'. To pass parameters to the dynamically-rendered component, use the '{nameof(Parameters)}' parameter.");
            }
        }

        if (Type is null)
        {
            throw new InvalidOperationException($"{nameof(DynamicComponentWithErrorBoundary)} requires a non-null value for the parameter {nameof(Type)}.");
        }

        _renderHandle.Render(_cachedRenderFragment);
        return Task.CompletedTask;
    }

    void Render(RenderTreeBuilder builder)
    {
        builder.OpenComponent<ErrorBoundary>(0);
        builder.AddAttribute(1, nameof(ErrorBoundary.ChildContent), (RenderFragment)(childContentBuilder =>
        {
            childContentBuilder.OpenComponent(2, Type);
            if (Parameters is not null)
                foreach (var entry in Parameters)
                    builder.AddAttribute(3, entry.Key, entry.Value);
            childContentBuilder.AddComponentReferenceCapture(4, component =>
            {
                Instance = component;
            });

            childContentBuilder.CloseComponent();
        }
            ));
        builder.AddAttribute(5, nameof(ErrorBoundary.ErrorContent), (RenderFragment<Exception>)(context =>
                errorContentBuilder =>
                {
                    errorContentBuilder.OpenComponent<MudStack>(6);
                    errorContentBuilder.AddAttribute(7, nameof(MudStack.Class), "border-2 mud-border-error");
                    errorContentBuilder.AddAttribute(8, nameof(MudStack.AlignItems), AlignItems.Center);
                    errorContentBuilder.AddAttribute(9, nameof(MudStack.Justify), Justify.Center);
                    errorContentBuilder.AddAttribute(10, nameof(MudStack.Style), "min-height: 50px");
                    errorContentBuilder.AddAttribute(11, nameof(MudStack.ChildContent),
                        (RenderFragment)(alertBuilder =>
                        {
                            alertBuilder.OpenComponent<MudAlert>(12);
                            alertBuilder.AddAttribute(13, nameof(MudAlert.Severity), Severity.Error);
                            alertBuilder.AddAttribute(14, nameof(MudAlert.ChildContent),
                                (RenderFragment)(alertContent =>
                                {
                                    alertContent.AddContent(15,
                                            "An internal UI error occurred. Please contact support.");
                                }
                                ));
                            alertBuilder.CloseComponent();
                        }
                        ));
                    errorContentBuilder.CloseComponent();
                }
            ));
        builder.CloseComponent();
    }
}

With this test:
    [Fact]
    public void ComponentRenderer_WrapsRenderedComponentInErrorBoundary2()
    {
        const string text = "this is a test";
       // this lines throws the error (if the Parameters parameter is added.
        var cut = Context.RenderComponent<DynamicComponentWithErrorBoundary>(parameters =>
        {
            parameters.Add(r => r.Type, typeof(MudTextField<string>));
            // commenting out the following line (and the line that checks the Text propery value below allows the test to pass
            parameters.Add(r => r.Parameters,
                new Dictionary<string, object> {{nameof(MudTextField<string>.Text), text}});
        });
        var errorBoundary = cut.FindComponent<ErrorBoundary>();
        var textContent = errorBoundary.FindComponent<MudTextField<string>>();
        textContent.Instance.Text.Should().Be(text);
    }

Results in this output:
System.InvalidOperationException
Attributes should only be encountered within RenderElement
   at Bunit.Htmlizer.RenderCore(HtmlRenderingContext context, ReadOnlySpan`1 frames, Int32 position) in /_/src/bunit.web/Rendering/Internal/Htmlizer.cs:line 96
   at Bunit.Htmlizer.RenderFrames(HtmlRenderingContext context, ReadOnlySpan`1 frames, Int32 position, Int32 maxElements) in /_/src/bunit.web/Rendering/Internal/Htmlizer.cs:line 73
   at Bunit.Htmlizer.RenderChildComponent(HtmlRenderingContext context, ReadOnlySpan`1 frames, Int32 position) in /_/src/bunit.web/Rendering/Internal/Htmlizer.cs:line 122
   at Bunit.Htmlizer.RenderCore(HtmlRenderingContext context, ReadOnlySpan`1 frames, Int32 position) in /_/src/bunit.web/Rendering/Internal/Htmlizer.cs:line 104
   at Bunit.Htmlizer.RenderFrames(HtmlRenderingContext context, ReadOnlySpan`1 frames, Int32 position, Int32 maxElements) in /_/src/bunit.web/Rendering/Internal/Htmlizer.cs:line 73
   at Bunit.Htmlizer.RenderChildComponent(HtmlRenderingContext context, ReadOnlySpan`1 frames, Int32 position) in /_/src/bunit.web/Rendering/Internal/Htmlizer.cs:line 122
   at Bunit.Htmlizer.RenderCore(HtmlRenderingContext context, ReadOnlySpan`1 frames, Int32 position) in /_/src/bunit.web/Rendering/Internal/Htmlizer.cs:line 104
   at Bunit.Htmlizer.RenderFrames(HtmlRenderingContext context, ReadOnlySpan`1 frames, Int32 position, Int32 maxElements) in /_/src/bunit.web/Rendering/Internal/Htmlizer.cs:line 73
   at Bunit.Htmlizer.GetHtml(Int32 componentId, RenderTreeFrameDictionary framesCollection) in /_/src/bunit.web/Rendering/Internal/Htmlizer.cs:line 61
   at Bunit.Rendering.RenderedFragment.UpdateMarkup(RenderTreeFrameDictionary framesCollection) in /_/src/bunit.web/Rendering/RenderedFragment.cs:line 164
   at Bunit.Rendering.RenderedFragment.Bunit.IRenderedFragmentBase.OnRender(RenderEvent renderEvent) in /_/src/bunit.web/Rendering/RenderedFragment.cs:line 142
   at Bunit.Rendering.TestRenderer.UpdateDisplayAsync(RenderBatch& renderBatch) in /_/src/bunit.core/Rendering/TestRenderer.cs:line 177
   at Microsoft.AspNetCore.Components.RenderTree.Renderer.ProcessRenderQueue()
--- End of stack trace from previous location ---
   at Bunit.Rendering.TestRenderer.AssertNoUnhandledExceptions() in /_/src/bunit.core/Rendering/TestRenderer.cs:line 380
   at Bunit.Rendering.TestRenderer.Render[TResult](RenderFragment renderFragment, Func`2 activator) in /_/src/bunit.core/Rendering/TestRenderer.cs:line 239
   at Bunit.Rendering.TestRenderer.RenderFragment(RenderFragment renderFragment) in /_/src/bunit.core/Rendering/TestRenderer.cs:line 48
   at Bunit.Extensions.TestContextBaseRenderExtensions.RenderInsideRenderTree(TestContextBase testContext, RenderFragment renderFragment) in /_/src/bunit.core/Extensions/TestContextBaseRenderExtensions.cs:line 43
   at Bunit.Extensions.TestContextBaseRenderExtensions.RenderInsideRenderTree[TComponent](TestContextBase testContext, RenderFragment renderFragment) in /_/src/bunit.core/Extensions/TestContextBaseRenderExtensions.cs:line 23
   at Bunit.TestContext.Render[TComponent](RenderFragment renderFragment) in /_/src/bunit.web/TestContext.cs:line 66
   at Bunit.TestContext.RenderComponent[TComponent](Action`1 parameterBuilder) in /_/src/bunit.web/TestContext.cs:line 52
   at MyProject.ComponentRendererTests.ComponentRenderer_WrapsRenderedComponentInErrorBoundary()

Expected behavior:
The test should pass without error.
Version info:

bUnit version: 1.9.8
.NET Runtime and Blazor version: 6.0.301
OS type and version: Windows 10

Additional context:

Thanks @jimitndiaye. We'll investigate further and get back to you.

@egil Thank you.
I'd like to add that doing the exact same component described above but in razor works. The following razor syntax is functionally equivalent to to what's going on in the component above:
<ErrorBoundary>
    <ChildContent>
        <DynamicComponent @ref="_dynamicComponent" Type="ComponentType" Parameters="ComponentParameters"/>
    </ChildContent>
    <ErrorContent>
        <MudStack Class="border-2 mud-border-error" AlignItems="AlignItems.Center" Justify="Justify.Center" Style="min-height: 50px">
            <MudAlert<EMAIL_ADDRESS>internal UI error occurred. Please contact support.</MudAlert>
        </MudStack>
    </ErrorContent>
</ErrorBoundary>

Wait, the razor version works?
That's surprising. Have you looked at the code the Blazor/Razor compiler generates from the razor version to make sure they are 100% the same?

@linkdotnet your interpretation of that line of code isn't quite accurate. The code sets the ChildContent property to a RenderFragment, not Type. Like I said, the code actually works at runtime and renders the component indicated (in the test it is a MudTextField)

@egil The razor version I posted above uses the built-in DynamicComponent and outputs the following render code:
        #pragma warning disable 1998
        protected override void BuildRenderTree(Microsoft.AspNetCore.Components.Rendering.RenderTreeBuilder __builder)
        {
            __builder.OpenComponent<Microsoft.AspNetCore.Components.Web.ErrorBoundary>(0);
            __builder.AddAttribute(1, "ChildContent", (Microsoft.AspNetCore.Components.RenderFragment)((__builder2) => {
// this part is where it differs from the original component in the issue. instead of using DynamicComponent I just directly render the component (using the same code that DynamicComponent does)
                __builder2.OpenComponent<Microsoft.AspNetCore.Components.DynamicComponent>(2);
                __builder2.AddAttribute(3, "Type", global::Microsoft.AspNetCore.Components.CompilerServices.RuntimeHelpers.TypeCheck<System.Type>(

                                                         ComponentType

                ));
                __builder2.AddAttribute(4, "Parameters", global::Microsoft.AspNetCore.Components.CompilerServices.RuntimeHelpers.TypeCheck<System.Collections.Generic.IDictionary<System.String, System.Object>>(

                                                                                ComponentParameters

                ));
                __builder2.AddComponentReferenceCapture(5, (__value) => {

                                _dynamicComponent = (Microsoft.AspNetCore.Components.DynamicComponent)__value;

                }
                );
                __builder2.CloseComponent();
            }
            ));
            __builder.AddAttribute(6, "ErrorContent", (Microsoft.AspNetCore.Components.RenderFragment<System.Exception>)((context) => (__builder2) => {
                __builder2.OpenComponent<MudBlazor.MudStack>(7);
                __builder2.AddAttribute(8, "Class", "border-2 mud-border-error");
                __builder2.AddAttribute(9, "AlignItems", global::Microsoft.AspNetCore.Components.CompilerServices.RuntimeHelpers.TypeCheck<MudBlazor.AlignItems?>(
#nullable restore
                                                                AlignItems.Center

                ));
                __builder2.AddAttribute(10, "Justify", global::Microsoft.AspNetCore.Components.CompilerServices.RuntimeHelpers.TypeCheck<MudBlazor.Justify?>(

                                                                                Justify.Center

                ));
                __builder2.AddAttribute(11, "Style", "min-height: 50px");
                __builder2.AddAttribute(12, "ChildContent", (Microsoft.AspNetCore.Components.RenderFragment)((__builder3) => {
                    __builder3.OpenComponent<MudBlazor.MudAlert>(13);
                    __builder3.AddAttribute(14, "Severity", global::Microsoft.AspNetCore.Components.CompilerServices.RuntimeHelpers.TypeCheck<MudBlazor.Severity>(

                                 Severity.Error

                    ));
                    __builder3.AddAttribute(15, "ChildContent", (Microsoft.AspNetCore.Components.RenderFragment)((__builder4) => {
                        __builder4.AddContent(16, "An internal UI error occurred. Please contact support.");
                    }
                    ));
                    __builder3.CloseComponent();
                }
                ));
                __builder2.CloseComponent();
            }
            ));
            __builder.CloseComponent();
        }
        #pragma warning restore 1998

@egil no the razor version works in both. It's the code version that works in Blazor but not in bUnit.

Like I mentioned on Gitter, much of the code in Htmlizer is lifted from Blazors prerenderer logic, but we might need to upgrade to a newer version.

Yea I just found it interesting that functionally identical code behaves differently in bUnit. If you run either version (code or razor) in Blazor you'd see the the output is identical.

Yea I just found it interesting that functionally identical code behaves differently in bUnit. If you run either version (code or razor) in Blazor you'd see the the output is identical.

Me too. Just sharing that detail from our Gitter conversation with Steven.
But we also need to make sure the two are actually functionally the same, or rather, they do not seem to be, since they behave differently, which by definition means they are not. But it's interesting none the less.

Btw. Here is the original code for DynamicComponent. https://source.dot.net/#Microsoft.AspNetCore.Components/DynamicComponent.cs

@egil yea I borrow quite a lot of code from that in the above component.

The code inside this RenderFragment delegate is pretty much a lift from the Render function in DynamicComponent:
        builder.AddAttribute(1, nameof(ErrorBoundary.ChildContent), (RenderFragment)(childContentBuilder =>
        {
            childContentBuilder.OpenComponent(2, Type);
            if (Parameters is not null)
                foreach (var entry in Parameters)
                    builder.AddAttribute(3, entry.Key, entry.Value);
            childContentBuilder.AddComponentReferenceCapture(4, component =>
            {
                Instance = component;
            });

            childContentBuilder.CloseComponent();
        }
            ));

And that part is the only difference between the code version and the razor version - the code version just in-line's the Render function from DynamicComponent while the razor version just calls DynamicComponent to do it instead.

You get the error in that section where it's checking if Parameters is null. If not null you get the error. If null or empty you don't get the error.

For future reference the current version of the HtmlRenderer in the aspnetcore repo: https://github.com/dotnet/aspnetcore/blob/main/src/Mvc/Mvc.ViewFeatures/src/RazorComponents/HtmlRenderer.cs
there are some smaller changes since the original version. On first sight nothing exciting. Will have a look tomorrow. We also made changes on our own, even though I don’t think they changed anything in the behaviour.

I found the issue. Very easy to miss:
void Render(RenderTreeBuilder builder)
    {
        builder.OpenComponent<ErrorBoundary>(0);
        builder.AddAttribute(1, nameof(ErrorBoundary.ChildContent), (RenderFragment)(childContentBuilder =>
        {
            childContentBuilder.OpenComponent(2, Type);
            if (Parameters is not null)
                foreach (var entry in Parameters)

                    // This should be childContentBuilder and not builder
                    builder.AddAttribute(3, entry.Key, entry.Value);

The reason is simple: You use the wrong object / builder ;)
Using builder instead of childContentBuilder inside your lambda will for sure result in invalid HTML ;)

I would close the issue for now. If you have further questions let us know.

@linkdotnet good catch! Can't believe I missed that.