Update application-proxy-configure-complex-application.md

if you agree with me this needs to be identified and be clear, as the customer will understand the object ID under the enterprise APP blade not the Object ID under the APP registration blade.
old :{<object-id-of--the-complex-app}
suggestion : {<object-id-of--the-complex-app-under-App-Registration}

@mudeeb : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@kenwith
Can you review the proposed changes? IMPORTANT: When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

I emailed with @mudeeb - we will close this one and open in private repo so it can be merged there. Thanks. #please-close


Cleanup Duplicate/Unused Assembly References

Noticed today that the DotNetCasClient project is referencing a lot of .NET assemblies that are never used, more than likely from the bloated templates that Visual Studio provides out-of-the-box. To keep things lean and mean, I'd like to remove the unused assemblies (by no means difficult to add back should they need to be referenced in the future). Also, there are two groupings of references to separate .NET20/35 and .NET40/45, but they reference many of the same assemblies. I plan to merge the shared assemblies together in one item grouping in the .csproj file, leaving only the differing assemblies in the conditional groupings. The end result should be leaner resource usage with no functional changes.
Anyone against these two changes?

I might have to break them back out when I add the .NET Core target.

Ahh, for some reason I was under the impression that .NET Standard/Core was going to be a wholly new project due to the vast changes in the security model.

Included in release 1.2.0


Change cursor style of buttons to pointer

For #9341

Codecov Report

Merging #9989 into master will decrease coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #9989      +/-   ##
==========================================
- Coverage   61.16%   61.15%   -0.02%
==========================================
  Files         564      564
  Lines       30109    30109
  Branches     4555     4555
==========================================
- Hits        18416    18412       -4
- Misses      10663    10666       +3
- Partials     1030     1031       +1

Impacted Files
Coverage Δ

src/client/common/utils/platform.ts
70.58% <0%> (-5.89%)
:arrow_down:

src/client/linters/pydocstyle.ts
86.66% <0%> (-2.23%)
:arrow_down:

src/client/datascience/debugLocationTracker.ts
76.56% <0%> (-1.57%)
:arrow_down:

src/client/common/process/proc.ts
14.49% <0%> (-0.73%)
:arrow_down:

Continue to review full report at Codecov.

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


Trigger shoot care reconcile on ManagedResource state change

How to categorize this PR?

/area control-plane
/kind enhancement
What this PR does / why we need it:
The shoot care controller currently runs in a fixed cycle, which by default is one minute. This leads to some delay between all ManagedResources getting healthy and the shoot status reflecting this. The PR adds a watch on ManagedResources. To map back to the corresponding shoot, the controller also collects a mapping from namespace to shoot object key when first processing a shoot. Entries are removed from the mapping once the controller is no longer responsible for a shoot.
Which issue(s) this PR fixes:
This is part of the hackathon https://github.com/gardener-community/hackathon/tree/main/2024-12_Schelklingen
Special notes for your reviewer:
Release note:

The shoot health state is immediately refreshed once ManagedResources change their state.

/assign
/ok-to-test


Difference between Stable Swarm and AI Horde

AI Horde is a mature REST API which already provides the primary functionality the Stable Swarm aims to achieve, allowing distributed GPUs to be used by the same user. Only it goes much further and allows multiple GPUs to be be used by any number of users.
It seems that you could use the AI Horde to handle the aggregation and then simply build the Swarm UI as a frontend in front of it.
I am just curious why this was developed from scratch instead of using an existing FOSS tool which could speed up the development aims of this tool

This users your own GPUs with unrestricted access to settings as deep as you want to go, in a neat cleanly packaged wrapper.
As opposed to horde which uses public GPUs and naturally needs reasonable access restrictions to prevent abuse.
Additionally, the part of the project that was used for the name and the headline is not the only focus of the project, there's several overlapping goals, that was just the easiest headline.
This originally started development as a hacking ground for internal Stability usage, and so any tool not 100% internal to us and not giving us 100% direct and instantly internal access was insufficient.

Just FYI  the AI Horde can be hosted internally by anyone and used unrestricted in the same way you expect and it's open source allows access to all parts of the internals. I am also currently helping stability.ai (Joe's Team) with this kind of clustered inference. It might be worth collaborating as there's a lot of development you would be able to bypass.


Filtered data save & load problem

Data is automatically filtered by trp and NADH chi sq when opened or imported if those columns are present. The filtering happens without the "Filter Data" checkbox checked. This interferes with the save all/save view: it seems that both save all the data.

Horst brought this problem up along with the need for a "# dropped by filtering." I'll try to put both solutions together


Feature request: limit the total number of cached items

Hi @mbsimonovic
How would you need this feature to behave ?
Would it work like a sort of cache queue: last in, first out ?
Let's say the limit is 10 cached requests. The first 10 requests are cached. When the 11th request is made the first cached request is cleared from the cache. Is that what you had in mind ?
Cheers

@RasCarlito exactly. My worry is if theres no limit then i could get out of memory. Time based exipiration does not prevent this.

@mbsimonovic I've merged this feature onto the master branch. I'll release the new version this week after adding a bit more tests. Thanks again for the request :beers:

@mbsimonovic Actually I just released a new version: v1.6.0


⚠ Bofrost.fr has degraded performance

In 6cd64d0, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1449 ms

Resolved: Bofrost.fr performance has improved in 4f9518c.


Added Execution Hooks around UseCase Interactor

So this PR is really two things (sorry):

renaming all interface{} -> any and ioutil deprecations (lots of files changes)
Usecase interactor hooks (nethttp/handler.go and associated test file)

My original intention was to just do the Usecase hooks, but the linter got the best of me :/
Anyways.... I have a need for running hooks after input is processed/validated but before the usecase is run (and the same for after the usecase is run, but before the output is validated).
It's a slice of function hooks, that are called before and after useCase.Interact() is called at nethttp/handler.go:135.

I'm sorry it took me so long to review this PR (it would be easier without cosmetic changes).
It seems the same functionality can be achieved with already existing usecase middlewares.
Here is an example
	s := web.DefaultService()
	s.Wrap(nethttp.UseCaseMiddlewares(usecase.MiddlewareFunc(func(next usecase.Interactor) usecase.Interactor {
		var (
			hasName usecase.HasName
			name    = "unknown"
		)

		if usecase.As(next, &hasName) {
			name = hasName.Name()
		}

		return usecase.Interact(func(ctx context.Context, input, output interface{}) error {
			err := next.Interact(ctx, input, output)
			if err != nil {
				log.Printf("usecase %s request (%v) failed: %v\n", name, input, err)
			}

			return err
		})
	})))


Changing from a widget type from map to gauge

Create a map widget
Display a location
Change the widget to a gauge
Background becomes bright white (see attached screenshot)
Reload the page and it looks normal again.

Duplicated problem with: https://freeboard.io/board/XLYmYS. Looking into it now


Isolate logging in tests

This PR is an attempt to solve issue #170.
I have removed order-dependent test helpers and used unique names instead. Additionally, this PR introduces a new context fixture in an attempt to better isolate the logging usage in the tests.
An error (previously undetected) in the chdir logging is also corrected.
However, the results of the tests are still non-deterministic: in some occasions caplog.text is unexpectedly blank as indicated by the CI logs. Any tip/hunch/help on this issue is more then welcome, I couldn't completely figure it out yet.
(Who would say, logging would be the most difficult thing to test??)
Regarding performance: in the CI, the speed improvement is not very noticeable (in part because of the setup time and the system tests in bash that do not run in parallel - running everything inside pytest as suggested in #120 could help).

Coverage increased (+0.02%) to 96.549% when pulling c361859f8eb5ef99466ca8ead803fcc6f86d8c13 on abravalheri:issue-170-logging into c8aeb41b751083f016f2882f6194e264a0341fe6 on blue-yonder:master.

Hi Florian, thanks for the review! Please find my answers in line:
This seems to be the final step for complete isolation, right?
Can you yet tell what speedup in the unit-tests we get?

Final step \o/.  In the CI the speed up seems to be negligible (something around 1 minute, but I haven't done proper statistics). When running locally, the speed up is more noticeable when we use more than 10 processes, even in machines with a lower number of cores (I am testing in a VM with 2).
I believe this is because we do a lot of IO. However changing dynamically the number of process in xdist is not easy (I would like to have something like max(10, numcores) or numcores + 5 to fully take advantage of the CI) - right now the configuration is to use the number of cores available.
Pytest allow us to do so, but not from conftest. (I am still investigating if we need an setuptools entrypoint for that or not - if we need I don't think it is worthy the trouble to create a pytest plugin).
And has it any more downsides like unreadable logs?
I could imagine that if several unit tests run in parallel you get an confusing output.

Writing tests become more difficult, but not that much. (Just when you are using shred resources, like files - I think I fixed the logs now!).
It turns out pytest do a good job organizing the test logs... You can check out the CI ones to get an idea.
Can it easily be deactivated if there is any need?
Would be nice to also that that somewhere, how to do it.

It would require a custom command line parameter, something similar to the pytest configuration/plugin I have previously mentioned in my first comment. It can be quite difficult to do.
Last comments:

I might have fixed the non-deterministic behaviour :)
If we decide to not do parallel testing, I still think some changes in this PR are welcome, and having isolation is something good.

Thanks @abravalheri, isolation is definitely something we want and the same goes for deterministic behaviour.
I took a look at the Travis logging and something I noticed is a lot of messages like:
tests/api/test_helpers.py::test_merge_rules_just_in_merged Coverage.py warning: Module pyscaffold was previously imported, but not measured (module-not-measured)
Coverage.py warning: Module pyscaffold was previously imported, but not measured (module-not-measured)
Coverage.py warning: Module pyscaffold was previously imported, but not measured (module-not-measured)

as well as:
Module already imported so cannot be rewritten: coverage

Is this related to xdist?

Hi @FlorianWilhelm,
The warning Module already imported so can not be re-written at the end of the tests, is something that appears to be quite common when using xdist, and is related to the random order pytest imports its plugins (the solution is something still being debated).
My pragmatic fix for the warning was removing the -p coverage from the list of arguments in setup.cfg.
The warning Module pyscaffold was previously imported, but not measured, seems to be related to importing pyscaffold in conftest.py. I reckon pytest loads conftest first because of the plugins and configurations that can be there, but then xdist have to load the entire test environment again for each process, setting up the "coverage probes". At this point, pyscaffold is already cached in memory.

@FlorianWilhelm, since it is not easily to "opt-out" of xdist if we include its configuration on setup.cfg, I have changed to a opt-in approach and added some instructions to the CONTRIBUTING.md.
I hope this is a good compromise.
Soon I will provide you some numbers about the real difference in time of using xdist or not.

@abravalheri, looks cool! Thanks for your work!


LiveComponent and non managed inputs

I am trying to use https://github.com/Choices-js/Choices in a LiveComponent. I have a controller to initialize an instance of Choices.js.
Without LiveComponent it works nicely, but the problem is that Choices.js modifies structure of the input and also adds an additional choice search input, so it is not managed by LiveComponent and I don't see any point to map it in the component.
Question: is it possible somehow to ignore the input changes and let the LiveComponent to rerender after bluring this non mapped input?

It's a bit old but still open so..
If you can't change the hidden input, or if Choice.js force new value without dispatching "change" event, what about a custom stimulus controller that dispatch change event when Choices.js is blured ?
Maybe just have to listen on Choice.js "change" event (https://github.com/Choices-js/Choices#events), then dispatch change on hidden input ?
hiddenInput.dispatchEvent(new window.Event('change', { bubbles: true }));
I made custom datepicker and select2.js work with that.

That's a nice solution... and it makes me think that an even better thing would be something like:
const element = document.getElementById('example');
const example = new Choices(element);

element.addEventListener(
  'addItem',
  function(event) {
        liveController.$render();
  }
);

Basically, call the $render() method on the live controller directly, instead of needing to add a hidden input and then trigger a change event on it. The only question is: what's a good way/pattern to get access to the live controller instance? I'm thinking about a controller like this:
export default class extends Controller {
    connect() {
        this.element.addEventListener('live:connect', (event) => {
            this.liveController = event.detail.controller;
        });

        // or, we would use targets - but you get the idea
        const element = document.getElementById('example');
        const example = new Choices(element);

        element.addEventListener(
          'addItem',
          function(event) {
                this.liveController.$render();
          }
        );
    }
}

This is probably an approach that should be documented. Right now, the live:connect event isn't documented at all.

Thanks for the insights, will check it :)


[Nox.Generators] Dispatch Domain Events before Saving to the db

Override db context savechangesasync to dispatch the event
After the Audit Part, and before the save
Dispatch the events asynchronously
Use MediatR INotification to Dispatch and subscrive them
USe ServiceCollection Extension, AddNoxMediatR method to register all Notification Handlers see for example the
Create a "real" example on ClientAPi Tests how to subscribe the Notification  and do some custom logic

relates to #755 depends on #761

https://regusit.atlassian.net/browse/MP-9237

@rochar we might need to introduce a new flag on the IDomainEvent
    /// <summary>
    /// Gets or sets a value indicating whether this domain event is published.
    /// </summary>
    public bool IsPublished { get; set; }

This would be needed in order to avoid duplicating the sending of the Domain events, for instance, if we have AggregateRoot and its Relationship.
Lets discuss this.

@rochar let's discuss also this thing because we would have to distinguish when the State is Added/Modified/Deleted
private void AddAndDispatch(EntityEntry<IEntityHaveDomainEvents> entry)
{
    switch (entry.State)
    {
        case EntityState.Added:
            entry.Entity.RaiseDefaultCreatedDomainEvents(entry.Entity);
            break;

        case EntityState.Modified:
            entry.Entity.RaiseDefaultUpdatedDomainEvents(entry.Entity);
            break;

        case EntityState.Deleted:
            entry.Entity.RaiseDefaultDeletedDomainEvents(entry.Entity);
            break;
    }

    await DispatchEvents(entry.Entity.Events);
}


Add MessageAwaiter contrib

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

It's great to see your contribution for adding a new plugin!
Could you please file an issue to description this plugin in details about what problem is it trying to solve, and why it is a good solution, so that we can review this new feature , refine the design, and finally decide whether to accept it.
Looking forward to learn from your post, thanks!

Thanks for the reply! the issue is opened, hope we can have more discussion there.

I have switched to use mathcers for message pattern matching, thanks for your detailed directions!

Your updates looks good!
Please resolve the conflicting in the PR before we can merge it, thank you very much!

I've merged with newest master brance and fixed the naming, looks good now

Thank you very much for your contribution!
I have invited you to join the Wechaty Contributors team on GitHub, please accept it by following the below message:

You've invited sine to Wechaty! They'll be receiving an email shortly. They can also visit https://github.com/wechaty to accept the invitation.

And we have a Wechaty Contributors Group on WeChat, please contact @lijiarui to join us.
Cheers!

OK, glad to be one of the contributors, cheers!

Congratulations! 🎉 🎉 🎉
You become a wechaty contributor successfully! 👏 👏 👏
Please:

[ ]  Add my wechat: ruirui_0914 and send me this pr link. I will invite you into Wechaty Contributor Room
[ ] Update your profile here: Contributor List, then other contributors will know you better!
[ ]  Join wechaty org by clicking https://github.com/wechaty


Creating multiple users while staying on the admin/invite page

When an error occurs like in the case of the fist image, there is no errors visible to the user and if we click the invite button multiple times, multiple users will be generated (2d images)

Thanks @ysael - good bug
This makes sense.  A common error when self hosting is that AWS SES email is not configured, and the admin/invite endpoint fails.  Not only should the "Invite" page better handle that error message, the server should not "Internal Server Error".

yes 👍🏼 , in our case it was because SES account is in the sandbox mode and we needed to verify the emails

I believe this is fixed by https://github.com/medplum/medplum/pull/1698 - let us know if you see further issues?


Add Taxonomy Support

This is a Feature Request

What would you like to be added
Adding Taxonomy support to the site so that it align with docsy.
Currently it is disable here
Why is this needed
Check out this issue #41171
Comments

/sig docs
/area web-development

Additionally, we should consider a line between our main content and right-most sidebar to seperate them similar to what docsy site generally have.

/assign

Any update on this ?

@coder12git : Thank you for suggesting this change. However, enabling Taxonomies to align with Docsy will involve way more than just flipping a switch since it does not involve automatic categorisation/tagging.
Reference: https://gohugo.io/content-management/taxonomies/#example-taxonomy-movie-website
Given this, what would be your plan of action if we were to enable this?

@divya-mohan0209 here is my plan -
For Categories , we can consider all these categories for each page of our docs

Installation and Setup:
Getting Started:
Core Concepts:
Workloads:
Networking:
Storage:
Security:
Monitoring and Logging:
Troubleshooting:
API Reference:

For Tags, we can consider all these tags

Platform:
Development:
Tools:
Best Practices:
Use Cases:
Integration:
Release Notes:

@divya-mohan0209 any feedback here?

/priority awaiting-more-evidence
We should assess the benefits of this change before we accept it.


enable updating standalone emoji from recent list

Clicking a recent emoji wasn't updating the selection in standalone mode, this fixes that.
However, I also noticed that clicking a recent emoji doesn't trigger emojibtn.click - not sure the best way to achieve this?

However, I also noticed that clicking a recent emoji doesn't trigger emojibtn.click - not sure the best way to achieve this?

Yes, I know, it need to rewrite the recents functionality (click triiggering)

ok cool, let me know if I can be of any help :)

Looking forward to seeing this fix! (emojibtn.click)


Feature Request - Include Corona Keys fix xeBuild patch

On Corona RGH images, certain keys related functions don't work properly (notably XeKeysSetKey for title keys, meaning games that use it for decryption don't work, e.g. Rock Band games suffer from the "no preview audio"/"endless loading" bug). I made a patch for this, and it would be useful to have an XeBuild option in the GUI, or to enable by default when building Corona freeboot 17559 images.
The patch in hex format:
00003B8C 00000001 389F0010
0000611C 00000001 387C0010

This bug is caused by, somewhere and somehow, the 1BL key copy at 0x16510 (17559) being blanked out on Corona. I've mitigated this issue by patching HvxKeysSetKey (for title keys) and HvxKeysExecute to instead direct those reads to the first 1BL key copy at 0x163F0 (17559). I haven't been able to extensively test this on actual Corona hardware, so that is definitely something that needs doing before implementing. (I have since had a friend test on Corona and can confirm the Rock Band issue is fixed.) There could also be other places the bad copy is used, but it looks like they're in firmware update routines so aren't needed.
I'd have submitted a pull request, but I can't seem to get J-Runner to want to build, and I'm not sure whether it would be best to have this as an option that has to be enabled, or to enable it by default when building Corona images.

Thx. Will address this in December when I'm home.
Kind Regards,
Josh

Done.


Implement reasoning for OpenAI functions

When I use LLM without functions, I am able to force using COT before actually predicting json. But I want to both use advantages of COT and structured output of Openai functions

Also it would be cool, I LLM with openai tools would be able to produce BaseModel object (the same as we can to with PydanticOutputParser and usual LLM)

Ideally, with support of BaseModelV2

https://platform.openai.com/docs/api-reference/chat/create
It may be implemented as nested funciton call (wrap function schema into layer of thinking function) or as sequential functions call (at first call thinking function and only then real one)


When will open source code

Hi, I would like to know when the source code will open.

Hi @AnnieYu0401,
Thanks for your comments. We currently plan to open source our codes after the review process.

Today is 20221123 🤣

Yet our paper is still not accepted by any conference.:)
We are working on it to make it more solid.


drivers: crypto: Organizational changes in this driver

Proper namespaces (crypto and cipher used properly)
Rename files

The general idea is make it clear what is general to a crypto driver accelerator and what is specific to ciphers. Doing it because crytpo acceleratos can have more functionalities than cipher, for example hashing.

I am closing this one in favor of https://github.com/zephyrproject-rtos/zephyr/pull/42914


"Popești" is listed in the drop-down menu

Description:
"Popești" is listed in the drop-down menu alongside its corresponding county and commune, according to the law
Precondition:
The website is up an running.

Step 1
Type "Popești" in the search bar.

Expected results
The location is listed in the drop-down menu as "Popești, Argeș (Cocu)".


fn bug fixed geo column on different chart layer

Description
geo 컬럼이 일반 차트 선반에 올릴수 없도록 제약을 추가해야합니다.
How Has This Been Tested?
geo 컬럼을 일반 차트에 올릴려고 하면 alert 발생
Types of changes

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

Checklist:

[ ] My code follows the code style of this project. it will be added soon
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have read the CONTRIBUTING document. it will be added soon
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

run build
deploy to 4

@choonglee170924 데이터 패널에서 click 이벤트에는 처리를 안하신듯합니다.


[FIX] Admin Page Type not working when adding Intern

The type field is not updated when clicking on the "Add" Button.

Hey, Is this issue closed now?
I missed this error earlier actually.

Hey, Is this issue closed now? I missed this error earlier actually. Like just inform me about what's the issue or requirement here and I will try to work on it

It's open for now. You can fix it.


Improve psudo openpose.

I use motion data visualizer node to generate openpose pictures but it could not use with openpose controlnet perfectly. I tried dwpose convertion but dwpose cannot recognize some poses. I request to improve psudo openpose to generate better openpose picture. Thank you.

I'm having the same issue. The DWPose Estimator seems to randomly skips frames for no apparent reason. Does anyone have any tips on this?

I'm having the same issue. The DWPose Estimator seems to randomly skips frames for no apparent reason. Does anyone have any tips on this?

With the latest updates, there's now options to change the character shape and render the view as normal map, which I find works really well with dwpose.

You can use "Render OpenPose from SMPL Multiple" (only appliable to 4DHuman tho)


[prepared stmt]The prepared statement switches to a different db for execution, and the db change operation is invalid

Steps to reproduce the behavior (Required)
Create two dbs, db1 and db2, and create a table named t under each db.
Create a prepared statement insert data under db1. After execution, the data is written to db1.t.：
mysql> create database db1;
Query OK, 0 rows affected (0.00 sec)

mysql> create database db2;
Query OK, 0 rows affected (0.00 sec)

mysql> create table db1.t(a int,b string);
Query OK, 0 rows affected (0.01 sec)

mysql> create table db2.t(a int,b string);
Query OK, 0 rows affected (0.01 sec)

mysql> use db1;
Reading table information for completion of table and column names
You can turn off this feature to get a quicker startup with -A

Database changed
mysql> prepare stmt_insert from insert into t values(?,?);
Query OK, 0 rows affected (0.00 sec)

mysql> set @v_a=1, @v_b='China';
Query OK, 0 rows affected (0.00 sec)

mysql> use db1;
Database changed
mysql> execute stmt_insert using @v_a,@v_b;
Query OK, 1 row affected (0.20 sec)
{'label':'insert_113b04fc-7863-11ee-aaf5-00163e22c708', 'status':'VISIBLE', 'txnId':'185'}

mysql> select * from db1.t;
+------+-------+
| a    | b     |
+------+-------+
|    1 | China |
+------+-------+
1 row in set (0.01 sec)

After change the db to db2, execute the previous prepared stmt again, and the data is still inserted into db1.t.：
mysql> use db2;
Reading table information for completion of table and column names
You can turn off this feature to get a quicker startup with -A

Database changed
mysql> execute stmt_insert using @v_a,@v_b;
Query OK, 1 row affected (0.05 sec)
{'label':'insert_190fb82a-7863-11ee-aaf5-00163e22c708', 'status':'VISIBLE', 'txnId':'187'}

mysql> select * from db2.t;
Empty set (0.01 sec)

mysql> select * from db1.t;
+------+-------+
| a    | b     |
+------+-------+
|    1 | China |
|    1 | China |
+------+-------+
2 rows in set (0.01 sec)

Prepared stmt is now session-isolated, so I understand that it can be executed under each db in the session, but if the db name is not specified, the object that should be operated on is the current db.
Expected behavior (Required)
Real behavior (Required)
StarRocks version (Required)

You can get the StarRocks version by executing SQL select current_version()
branch-3.2-da7ced0

this is behavior is expected

tried on mysql, save behavior
mysql> create database db1;
Query OK, 1 row affected (0.02 sec)

mysql> create database db2;
Query OK, 1 row affected (0.01 sec)

mysql>  create table db1.t(a int,b string);
ERROR 1064 (42000): You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'string)' at line 1
mysql> use db1;
Database changed
mysql>  create table t(a int,b string);
ERROR 1064 (42000): You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'string)' at line 1
mysql>  create table db1.t(a int,b int);
Query OK, 0 rows affected (0.03 sec)

mysql>  create table db2.t(a int,b int);
Query OK, 0 rows affected (0.04 sec)

mysql> use db1;
Database changed
mysql> prepare stmt_insert from 'insert into t values (?,?)';
Query OK, 0 rows affected (0.02 sec)
Statement prepared

mysql> set @v_a=1, @v_b=1;
Query OK, 0 rows affected (0.00 sec)

mysql> use db2;
Database changed
mysql> execute stmt_insert using @v_a,@v_b;
Query OK, 1 row affected (0.01 sec)

mysql> select * from db1.t;
+------+------+
| a    | b    |
+------+------+
|    1 |    1 |
+------+------+
1 row in set (0.00 sec)

mysql> select * from db2.t;
Empty set (0.00 sec)


Missing v0.4.0 tag

There is a v0.4.0 of this cookbook available on Chef SuperMarket:
https://supermarket.chef.io/cookbooks/chocolatey
However, there is no v0.4.0 listed for this repo.

same for v0.4.1 tag. I have some automatic tools relying on tag presence, could you push them ?
thanks

My apologies. I neglected to push those upstream. Just pushed them now.


Temperature Set Error

Moving set point causes the following error:
2021-06-07 17:04:53 ERROR (MainThread) [homeassistant.components.websocket_api.http.connection] [140289994317200] setCoolSPF() missing 2 required positional arguments: 'scheduleId' and 'tempF'
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/components/websocket_api/commands.py", line 185, in handle_call_service
    await hass.services.async_call(
  File "/usr/src/homeassistant/homeassistant/core.py", line 1491, in async_call
    task.result()
  File "/usr/src/homeassistant/homeassistant/core.py", line 1526, in _execute_service
    await handler.job.target(service_call)
  File "/usr/src/homeassistant/homeassistant/helpers/entity_component.py", line 213, in handle_service
    await self.hass.helpers.service.entity_service_call(
  File "/usr/src/homeassistant/homeassistant/helpers/service.py", line 658, in entity_service_call
    future.result()  # pop exception if have
  File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 760, in async_request_call
    await coro
  File "/usr/src/homeassistant/homeassistant/helpers/service.py", line 695, in _handle_entity_call
    await result
  File "/usr/src/homeassistant/homeassistant/components/climate/__init__.py", line 548, in async_service_temperature_set
    await entity.async_set_temperature(**kwargs)
  File "/config/custom_components/lennoxs30/climate.py", line 289, in async_set_temperature
    await self._system.setCoolSPF(sp)
TypeError: setCoolSPF() missing 2 required positional arguments: 'scheduleId' and 'tempF'

This is repeatable by moving temp set point up or down several degrees.

Looks like I have the wrong version of climate.py in GitHub.  My bad, I should have caught that.  I’ll let you know when  get it sorted out.

If it matters, I’m using the custom simple thermostat card instead of the standard version.

Thanks, that helps.  I looked at it closer, not a version issue.  I missed a branch in the code..  The regular climate card always sends both the high and low set points together.   The simple sends only one.  Looks like a simple change will get to it tomorrow.

This should be fixed.  Please re-grab this branch and let me know how it works.
https://github.com/PeteRager/lennoxs30/tree/2021.06.11

Following two core restarts to get all zones reporting, moving the set point several degrees no longer generates an error in the HA log. I tested on both thermostat cards.

Having intermittent issues with reporting still. Can require several core restarts to have both the single zone and multi-zone data displayed.

I think that 2 restarts issue is a manifestation of not having the "South" S30 show up sometimes.  It's  a race condition on when the messages arrive vs creation of the climate entities.  So you could get just North, just South or both (rarely)
This should no longer happen with the new update for Issue #10

Do you think this item is now closed?

Yes.


RNG and XSD Samples Not Building

The DTD sample files will build using the project file, but the RelaxNG and XML Schema sample files do not currently produce any output.

The XML Schema examples need to have the xsi:noNamespaceSchemaLocation attribute with the catalog URI of the appropriate schema file. The xml-model processing instruction cannot be used.

There is still some issue with the bentley-faq.xsd or faqMod.xsd which causes some elements to not have a default @class assigned. This causes a DITA-OT error unless the classes are manually added to the topic.


implement "onError" event listener

as well as onDedupe, onHit, onMiss, we can have "onError" event
consider to implement also "stale on error" and relative options when an error occur, for example the first implementation can be serve the latest response for an amount of time

Would it make sense for Cache to extend EventEmitter? Then we could emit events and devs could register listeners instead of passing all the functions to the constructor options.

That's a possibility. However this would not have to emit an error event (otherwise it would crash the process.

do we need to have multiple listener for an event?

Not really and it's unnecessary overhead.

so I would avoid the EventEmitter

ah ok makes sense


Test profile seems to download wrong FASTQ files

Checking the results of my test profile run against the published results, it seems that I have a different set of fastqs. My files are labeled with the convention WT_R* and RAP_*, and alignment is consistently very poor (6-8%). I would guess this is probably a genome mismatch. I previously used revision 1.4.2 fairly extensively, so it's possible these are cached somewhere, but I have removed everything from the ~/.nextflow/assets directory. Is there a way to purge these files? I just want to be sure all is running correctly.
Alignment results:

-[nf-core/rnaseq] 5/5 samples passed STAR 5% mapped threshold:
7.07%: RAP1_IAA_30M_R1
6.76%: WT_R2
6.62%: RAP1_UNINDUCED_R1
7.87%: WT_R1
6.38%: RAP1_UNINDUCED_R2

Hi @RoganGrant ! Happy new year! Hope you and loved ones are well.
This is all expected because I updated the test dataset after 1.4.3. However, as you have observed the alignment rates are quite poor which can lead to confusion. See  https://github.com/nf-core/rnaseq/issues/544
Also, the published results on the website are from the full test dataset and not from the minimal test dataset which is why the names are different as you can see in this samplesheet https://github.com/nf-core/test-datasets/blob/rnaseq/samplesheet/samplesheet_full.csv

Hi @RoganGrant ! Happy new year! Hope you and loved ones are well.
This is all expected because I updated the test dataset after 1.4.3. However, as you have observed the alignment rates are quite poor which can lead to confusion. See  https://github.com/nf-core/rnaseq/issues/544
Also, the published results on the website are from the full test dataset and not from the minimal test dataset which is why the names are different as you can see in this samplesheet https://github.com/nf-core/test-datasets/blob/rnaseq/samplesheet/samplesheet_full.csv

Just wanted to make sure. Thank you! And happy new year to you as well!

Just wanted to make sure. Thank you! And happy new year to you as well!


Not rendering charts

Hi,
Excellent work here. However, event though 'python-highcharts' installed OK, my charts are not rendering.
There are some errors in the console output:
[IPKernelApp] ERROR | No such comm target registered: ipython.widget
[W 16:13:46.389 NotebookApp] 404 GET /api/kernels/66a4092a-4ea4-4b57-99da-60982c7d56bf/channels?session_id=15B39D0998384375969F9ED94E6EA5D2 (<IP_ADDRESS>): Kernel does not exist: 66a4092a-4ea4-4b57-99da-60982c7d56bf
[W 16:13:46.398 NotebookApp] 404 GET /api/kernels/66a4092a-4ea4-4b57-99da-60982c7d56bf/channels?session_id=15B39D0998384375969F9ED94E6EA5D2 (<IP_ADDRESS>) 10.93ms referer=None

Any ideas where I should look ?
Thanks.

Hi,
It seems to me that it is due to setting in the environment: https://github.com/jupyter/notebook/issues/397 Could you check it first and see if it fixes that problem? Please feel free to let us know if not.
Hank


Fix strings being converted to lowercase

When testing different possible value types, the final String_ case branch didn't use the original string that was passed in. So instead, it was passing along a lowercase version!
This pull request closes issue #1.

Thank you!


Support TLS 1.2 for scoop search

scoop-search fails with the error below when trying to access GitHub API endpoint.
invoke-webrequest : The request was aborted: Could not create SSL/TLS secure channel.
At C:\Users\sekost\scoop\apps\scoop\current\libexec\scoop-search.ps1:53 char:15
+     $result = invoke-webrequest $url -UseBasicParsing | select -exp c ...

This fix tells .NET runtime to use TLS 1.2 security protocol for HTTPS.
Fixes the problem on Windows 10 1709, PowerShell 5.1
Not sure if it introduces any compatibility issues on older versions of Windows / PowerShell.
Possibly this fix needs to be placed in another file.
Fixes #2063

Probably related to https://github.com/lukesampson/scoop/issues/2040, we need to work on a global fix for that.

I think the correct fix is to set:
[Net.ServicePointManager]::SecurityProtocol = [Net.ServicePointManager]::SecurityProtocol -bor [Net.SecurityProtocolType]::Tls12

since this avoids mysteriously breaking your own access to sites that do not yet support TLS 1.2.
@r15ch13 could we just put this in core.ps1? There's not a lot of places where you're doing net.webclient or invoke-webrequest, and all the ones I can find have an import of core.ps1.

superseded by https://github.com/lukesampson/scoop/pull/2074


Performance comparison

The simplicity and usage of this plugin are very effective and I have looked into both the packages, the division and styled_widget. But I am not sure which has more performance as there are method calling for each and every style. Calling methods for each single style may affect performance as compared to plain dart code. Kindly guide me in the right direction. Which way will be more performant/fast executing, the devision way, styled_widget way or plain dart code as provided by the framework itself for a large scale mobile app. If there would be an option to apply multiple styles in single method call then it would be great.

This has been a concern for me as well. Would be interesting to to a proper performance test to see the actual difference.
Plan Dart/Flutter will always be faster, given you write your code reasonable efficient. This will be the same as C++ will be more performant than JavaScript because it's more low level, again given you write the code efficient.
I think styled_widget is more performant than division since the widgets are more splited but I'm not quite sure.
If you define a widget as final (if you know it won't change), the methods will only be called once which will only be a potential performance issue on first render.
I have build a couple bigger projects myself with styled_widget and I haven't noticed a performance issue so far

There is already an option to apply multiple styled using for example decoration and constraints

Awesome project. Is this library production ready?

It depends on what you mean on production ready. Feature wise the package is good but it is still subject to change.
I am working on version 0.2.0 which will have a couple of breaking changes which I am making to improve performance.

@ReinBentdal that's ok. We can freeze the library on this version. What I'm asking is that if I could use it in production since this package helps write maintainable UI code.

I see, i would say yes. I would recomend using the newest version 0.2.0, since it has some performance improvements.
I would not recomend using the Text and icon extensions heavily since each method has to initiate a new Text widget. For example i would not write this:
Text('some text')
  .bold()
  .fontSize(24)
  .padding(all: 24)
  .backgroundColor(Colors.red)

And instead to something like this:
final TextStyle textStyle = TextStyle(
  fontSize: 24,
  fontWeight: FontWeight.bold,
);

Text('some text', style: textStyle)
  .padding(all: 24)
  .backgroundColor(Colors.red)

And if you are applying more than one styles originating from the DecoratedBox widget:
Text('some text') // do`nt
  .backgroundColor(Colors.red)
  .borderRadius(all: 24)

Text('some text') // do
  .decorated(
    color: Colors.red,
    borderRadius: BorderRadius.circlular(24)
  )

What i am saying here can be summed up as: The package is not a replacement of the current way of building widgets with flutter, but a complimentary tool. When used right i think its a very useful.
I (or someone else) should probably spend some time improving the documentation to address these things


Lazy load 3rd party script, not in global scope

angular-cli: local (v1.0.0-beta.11-webpack.2, branch: master)
node: 6.2.1
os: darwin x64
I need to add a 3rd party script which doesn't have typings..
The Doc says to include it in angular-cli.json in "scripts" section. But I assume that it will add the external lib in global scope.  Any suggestions?

I don't think it's even adding the scripts to global scope. I tried including material lite js and css through the scripts section and styles section. Nothing was added to the final bundle.

I tried to add bootstrap using the docs and its not working either.

it looks like the feature is not supported in beta.11-webpack.2

I can confirm the "styles" are getting loaded and not the "scripts" with the cli master version.

The Problem is when the 3rd party script having runtime dependencies.. it is not resolved at runtime.
I just need to do like something below with the CLI..where I can do this with WebPack Config.
module: {     loaders: [       {         test: require.resolve('tinymce/tinymce'),         loaders: [           'imports?this=>window',           'exports?window.tinymce'         ]       },       {         test: /tinymce\/(themes|plugins)\//,         loaders: [           'imports?this=>window'         ]       }         ]   }

I tried .. adding in angular-cli.json
"scripts": [         "../node_modules/tinymce/tinymce.min.js"       ],
But I end up getting the dependent script failed to load error at runtime..
Failed to load resource: the server responded with a status of 404 (Not Found) VM16610:4 Failed to load: http://localhost:8000/themes/modern/theme.js http://localhost:8000/plugins/code/plugin.js Failed to load resource: the server responded with a status of 404 (Not Found) VM16610:4 Failed to load: http://localhost:8000/plugins/code/plugin.js

@StevePavlin +1

@MasterRyd3l @StevePavlin the current readme refers to functionality added recently (global styles and scripts) that currently isn't yet released. We're a bit late for a release but will do it soon.
@a5hik on the latest master the global scripts must all be imported exactly as if they were script tags. I'm not familiar with how tinymce works, but if you get me a repo that I can debug on I will try to figure out what the issue is.

@filipesilva If I want to add bootstrap, I need to wait to the next release? Or there are another way at this moment to add it?

@filipesilva after doing the import of dependent plugins in the component the error is resolved.
import 'tinymce/themes/modern/theme'; import 'tinymce/plugins/code/plugin';

@a5hik happy to hear you got working!
@aralroca on the latest release you can use bootstrap using the global scripts.


Provide a Version Only Exporting Format Helpers

Importing and mutating decimal.js seems like it could cause conflicts with other modules. It seems like this is necessary for some reason according to this comment, but why is unclear to me.
https://github.com/pjbatista/decimal.js-i18n/blob/c9c4b7bb96c97a9f22866eb7ceb319ac4fc1e18c/src/index.ts#L47
Could we instead have a version of this library which does no automatic patching but instead exports the Decimal.Format object?
Happy to make a PR once we've decided! Thanks so much for making this library!!

Sorry about the delay! I was working exactly on that before my burnout haha
If that PR is still possible I'm very much looking forward to it. Otherwise, I'll start working on that again.


resuming from an epoch-end checkpoint does not continue training (fault-tolerant training)

🐛 Bug
With PL_FAULT_TOLERANT_TRAINING=1 we cannot currently resume from an epoch-end checkpoint (no failure) when validation is turned on.
To Reproduce
from pytorch_lightning.callbacks import ModelCheckpoint
import os

from pytorch_lightning import Trainer
import torch
from pytorch_lightning import LightningModule
from torch.utils.data import Dataset, DataLoader

class RandomDataset(Dataset):
    def __init__(self, size, length):
        self.len = length
        self.data = torch.randn(length, size)

    def __getitem__(self, index):
        return self.data[index]

    def __len__(self):
        return self.len

class BoringModel(LightningModule):
    def __init__(self):
        super().__init__()
        self.layer = torch.nn.Linear(32, 2)

    def training_step(self, batch, batch_idx):
        print(self.trainer.current_epoch, "I'm in the training step")
        return self.layer(batch).sum()

    def training_epoch_end(self, outputs) -> None:
        pass

    def validation_step(self, batch, batch_idx):
        print(self.trainer.current_epoch, "I'm in the validation step")

    def configure_optimizers(self):
        return torch.optim.SGD(self.layer.parameters(), lr=0.1)

def run(checkpoint_path=None, max_epochs=2):
    train_dataloader = DataLoader(RandomDataset(32, 64), batch_size=2)
    val_dataloader = DataLoader(RandomDataset(32, 64), batch_size=2)

    model = BoringModel()
    trainer = Trainer(
        max_epochs=max_epochs,
        callbacks=ModelCheckpoint(save_top_k=-1, filename="{epoch}"),
        progress_bar_refresh_rate=20,
        num_sanity_val_steps=0,
        limit_val_batches=1,
        limit_train_batches=1,
        enable_model_summary=False,
        enable_progress_bar=False,
    )
    trainer.fit(model, train_dataloaders=train_dataloader, val_dataloaders=val_dataloader, ckpt_path=checkpoint_path)

if __name__ == "__main__":
    os.environ["PL_FAULT_TOLERANT_TRAINING"] = "1"  # With "1" - Fail. With "0" - Passes.
    run(max_epochs=2)

    max_version = -1
    for dir in os.listdir("lightning_logs/"):
        if "version_" in dir:
            max_version = max(max_version, int(dir[len("version_") :]))
    assert max_version > -1
    checkpoint_path = f"lightning_logs/version_{max_version}/checkpoints/epoch=0.ckpt"

    run(checkpoint_path, max_epochs=3)

On the second run, when resuming from the epoch-end checkpoint, training gets completely skipped.
Note: everything works fine if we set limit_val_batches=0 which turns off validation.
Expected behavior
Continues to train the extra epoch.
Environment
PyTorch Lightning master
torch 1.9
Reported by @eladsegal on Slack

With "1" - Fail. With "0" - Passes.

What is the failure exactly? I don't see any assertion failing

The failure is not expressed via an assertion, but you can see it in the output prints:
https://colab.research.google.com/drive/1F9wFN2pZy55pBW_PQLLbWtKh11dLOsuq#scrollTo=4Dk6Ykv8lI7X
Failure is when training/validation steps not running at all when resuming.

This is fixed in master 🎉


Config creation/disposal changes

Changes to creation/disposal of the config. Previously with scene switching (making new instances of OnnxInference), the code was trying to configure a disposed instance of SessionOptions (since the instance was stored in a static field, the reference itself would stay after the object is freed), causing crashes.
Removed the static references to config, this also allows every instance having its own config if needed in the future, and set the method names to make it clear that after these changes the SessionConfigurator is now generating new pre-configured config objects rather than retrieving a shared config.
A possible alternative would be to have one config instance that stays until Godot is closed, and then only dispose it at that time.

LGTM!


Workaround for the inaccurate math functions of Intel GPUs

Use the fp64 version of log() function for Intel GPUs. I added a #ifdef to the custom shaders of all existing layers. Please review, test and verify that it fixes the offset issue.
@ibgreen @Pessimistress @abmai @twojtasz

The headless test failed...because glGetDebugInfo is returning nil. Is this an error or it's actually a limitation of our tests? @ibgreen


Add support for ResponseHeaderModifier for HTTPRouteRule objects

Proposed changes
Problem: Users want to add, set, and remove response headers.
Solution: Use add_header NGINX directive to support ResponseHeaderModifier.

If the action is set, we simply configure the add_header directive with the given value in the HTTPRoute spec.
If the action is remove, we configure the add_header directive with the value set to an empty string.
If the action is add, we create a mapping for the $http_header_name variable appending a comma at the end of any client provided headers (if present) and prepend this to the given value in the HTTPRoute spec.

Testing: Describe any testing that you did.
Please focus on (optional): If you any specific areas where you would like reviewers to focus their attention or provide
specific feedback, add them here.
Closes #1397
Checklist
Before creating a PR, run through this checklist and mark each as complete.

[x] I have read the CONTRIBUTING doc
[x] I have added tests that prove my fix is effective or that my feature works
[x] I have checked that all unit tests pass after adding my changes
[x] I have updated necessary documentation
[x] I have rebased my branch onto main
[x] I will ensure my PR is targeting the main branch and pulling from my branch from my own fork

Release notes
If this PR introduces a change that affects users and needs to be mentioned in the release notes,
please add a brief note that summarizes the change.

Kevin's PR - https://github.com/nginxinc/nginx-gateway-fabric/pull/1494

Shouldn't we be adding the responseHeaderModifier validation here ? @kate-osborn

Shouldn't we be adding the responseHeaderModifier validation here ? @kate-osborn

@salonichf5 it looks like the intent was to make the HTTPRequestHeaderValidator more generic so it could validate both request and response headers, and then add the response header-specific validation separately in the validateFilterResponseHeaderModifier function in httproute.go.
I like your suggestion of adding a new struct to the HTTPValidator that validates response headers. I would move the functionality of validateFilterResponseHeaderModifier to that new validator.

pre-commit.ci autofix

pre-commit.ci autofix"

pre-commit.ci autofix

Btw when we squash and merge, we can add @kevin85421 as a co-author to the commit so proper credit is given :)

@sjberman Thank you! You guys are so nice.


fix: check if systemd-timesyncd is dead, only stop+disable if not

What type of PR is this?
Originally, we always stopped and disabled systemd-timesyncd, however, in the latest builds, we are getting the following errors:
timeout 120 systemctl stop systemd-timesyncd
==> azure-arm: Failed to stop systemd-timesyncd.service: Unit systemd-timesyncd.service not loaded

This PR first checks if that service is dead, if yes, it skips the stop and disable.

What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #
Requirements:

[ ] uses conventional commit messages

[ ] includes documentation
[ ] adds unit tests
[ ] tested upgrade from previous version

Special notes for your reviewer:
Release note:
none

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 78.984%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
2240

Relevant Lines:
2836

💛  - Coveralls


Import/Export T265 localization map API

New APIs:

Import and export T265 tracking device localization map
(rs2_export_localization_map, rs2_export_localization_map)
Set/Get T265 static node positional marker
(rs2_set_static_node, rs2_get_static_node)
Provision for T265  wheel odometer inputs
(rs2_load_wheel_odometry_config, rs2_send_wheel_odometry)

The localization maps can be used to establish initial location features, as well as sharing the tracking features between multiple devices.
The functionality is available in idle state only. Try it with Realsense-viewer

Propagate the APIs to C# and  Python wrappers
Extend unit-tests:

import/export localization map; set/get static node
Sensor callbacks API.

Related to #3129

Hi.  I’m excited to try the localization map export/import function and could use some help understanding how it works using the RealSense Viewer.  It doesn’t seem to be relocalizing but maybe I’m doing something wrong.  Here’s my test:

Run RealSense Viewer and then plug in the T265
Put the T265 at a known position and turn the Tracking Module on.  I see the streams including the pose stream.
Move the T265 around to get a good look at the area
Turn the tracking module off
Select "Export Localization map" and save the file.  I get a binary file of a couple hundred KB.
Reset the T265 by unplugging it and plugging it back in
Select "Import Localization map" and select the file previously saved.  The log says "Importing localization map from  ... completed"
Put the T265 at a different place than the original known position and turn the Tracking Module on.
Move the T265 around to get a look at the area

At this point I expect the origin to eventually snap back to where I started the T265 the first time but it never does. It stays where the T265 was started the second time.  I’ve tried this with the 2.19.1 release (FW version <IP_ADDRESS>29) and a more recent development branch (86af003f) Am I doing something wrong or do you have any ideas about how I can see what’s going on?   Thanks!

Hi @bill-wright,
Glad to hear you're excited about T265; we're excited to hear how people make use of it.  You are using the latest firmware, 2.19.1.  Once relocalized, the loaded map snaps to you (i.e. your current map created since startup) instead you snapping to loaded map, so your coordinates will not visibly jump when this happens.
The current way you interact with the map is via static nodes.   You can think of them as fixed positions in the world (map) whose position you care about and whose position is defined relative to the local scene.  Once created, you can ask about their positions and the T265 will provide their location relative to current tracking origin.   Currently, the way to tell that we have relocalized to a loaded map is the static nodes defined in that map will begin to return valid poses.
You might create a static node in the center of the play area in a VR context and then define play boundary relative to it.   For an AR experience, you might define a static node at the location of a set of virtual objects.
We are working on adding visualization of maps and static nodes in the realsense-viewer.

Thanks @radfordi,
I think I get it.  I'm doing persistent AR so I want to know the difference between the coordinate system this time and coordinate system when the map was created.  I could place a static node at the origin when making the map and then its pose in the new coordinate system would tell me the transform between the new coordinates and the map coordinates.  I'll give that a go.  That should work.

Hi again @radfordi,
That worked!  I modified one of the examples to save a map with a static node in it (I'm using the origin (0,0,0) as the static node). This seems to work fine and I can see that the map file has the text of the static node name toward the end of it so I believe my static node is getting into the map file.
Then on a subsequent run, I can load that map and get the position of the static node relative to the new coordinate system. That seems to work and the location of the static node relative to the new coordinate system makes sense.
My issue now is what can I do to help the relocalization process?  I've only been able to get it to relocalize when I let the camera dwell for several seconds at the starting place I used when I made the map so it gets the same look from the same place it originally started. When I make the map, I try to move the camera all over the room so it gets to see the room from all angles but that doesn't seem to make much difference. Is there something I can do when making the map to help it relocalize better? Is there any way for me to tell how complete the map is?  Any heuristics would be helpful.  Thanks again!

Hi @radfordi I'm working with @bill-wright on the T265 relocalization too and I'm seeing that it can take roughly 2 minutes for the static nodes to start returning valid poses.  We'd like to relocalize to a loaded map within a few seconds.  Any advice on how to reduce the relocalization time would be much appreciated.  Thanks!

@bill-wright, @JBBee, I'm glad to hear the basics work for you so that now we can talk about performance!
The time to first relocalization is one of our key metrics and we are actively working to improve it.  Our goal is the same as yours, a time measured in seconds not minutes.  We have found a few related bugs since 2.19.2 whose fixes may help in your scenario.  We are working on the next release and would love to hear about your feedback when it's availble.  We plan to continue to make progress in this area.

@bill-wright, @JBBee, the latest version of librealsense, v2.21.0, has quite a few fixes with respect to relocalization.  We'd love to hear if the performance improvement is noticeable in your application.

Hi @radfordi, thanks for checking in with us!  @bill-wright and I are using the new version in our application and we are seeing a definite improvement.  But it seems we have to focus on an area in our room in order to be able to re-localize using that area.  For example, we'll start the pose stream and walk around a bit and then stop and focus on a certain feature in the room for about 10-15 seconds, then continue.  After exporting and importing this map in a new session it's able to re-localize when looking at that feature that was focused on, so it doesn't necessarily need to be the starting location anymore, which is great!
Ideally though we'd be able to continuously move at a slow walking pace around the room once or twice and then be able to re-localize after importing by just repeating a similar walk around.
If there's any additional tips on how to get to this continuous movement that would be much appreciated.
Thanks.

@JBBee, @bill-wright, thanks for reporting on your progress.  We have a known issue where we aren't relocalizing while moving.   We are working on a fix and hope to have something for the next release.

@bill-wright can you please share the example for relocalization as a starter ?

Hi @zainmehdi
The relocalization performance has been getting better in successive releases.  @JBBee and I appreciate the efforts of the RealSense team to make this better.
I can share some snippets.  We're working in Unity so YMMV. I also edited this a little for clarity so please forgive any little errors that I introduced.  Hope they help.
internal PoseSensor GetActivePoseSensor() {
   PoseSensor sensor = null;
   sensor = ActiveProfile?.Device.QuerySensors<PoseSensor>().First();
   return sensor;
}

internal bool SetOrigin() {
   var pos = new Intel.RealSense.Math.Vector();
   pos.x = 0;
   pos.y = 0;
   pos.z = 0;
   var rot = new Intel.RealSense.Math.Quaternion();
   rot.x = 0;
   rot.y = 0;
   rot.z = 0;
   rot.w = 1;

   ret = false;
   try {
       using (PoseSensor poseSensor = GetActivePoseSensor()) {
           Dbg("Setting origin with handle: " + poseSensor.Handle);
           ret = poseSensor.SetStaticNode("origin", pos, rot);
       }
   }
   catch (Exception ex) {
       Debug.LogException(ex);
   }
   return ret;
}

internal bool GetOrigin(out Matrix4x4 trs) {

  Intel.RealSense.Math.Vector pos;
  Intel.RealSense.Math.Quaternion rot;
   try {
       lock (this) {
           using (PoseSensor poseSensor = GetActivePoseSensor()) {
               if (poseSensor != null && poseSensor.GetStaticNode("origin", out pos, out rot)) {
                   // Convert T265 coordinate system to Unity's
                   // see https://realsense.intel.com/how-to-getting-imu-data-from-d435i-and-t265/

                   Vector3 unityPos = new Vector3(pos.x, pos.y, -pos.z);

                   Quaternion q = new Quaternion(rot.x, rot.y, rot.z, rot.w);
                   Vector3 e = q.eulerAngles;
                   Quaternion unityRot = Quaternion.Euler(-e.x, -e.y, e.z);
                   trs = Matrix4x4.TRS(unityPos, unityRot, Vector3.one);
                   return true;
               }
           }
       }
   }
   catch(Exception ex) {
       Debug.LogException(ex);
   }

   trs = Matrix4x4.identity;
   return false;
}

https://neilyoung.serveblog.net


Update PyPI release

Hello! Looks like the pre-release version is about a year old, and the version in the repo is still 0.1.0b0 despite substantial changes since then. Wondering if there's a particular reason the PyPI version isn't being kept synchronized, or if I should just pin a commit in the git repository rather than by package version in downstream packages?

Thanks for the report @sneakers-the-rat.
We just went through a development sprint.  We will be updating the version, tagging, and releasing these changes to pypi by the end of May.  For the interim, please install from source:
pip install git+https://github.com/datajoint/element-event.git

Install a specific commit:
pip install git+https://github.com/datajoint/element-event.git@<Enter commit hash>

Hi @sneakers-the-rat, we just published a new release (0.1.2) of this package.  Please find it on PyPI.  Thanks.

y'all rock, thanks


Runtime reference post-release edits

Description
Corrected a number of issues with the runtime reference information:

Removed an unused annotation in table
Corrected broken links to internal documentation
Restructured bulleted list to keep numbering continuous

How Has This Been Tested?
Built a local Gatsby instance and previewed the changes
Merge criteria:

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

LGTM


Allow for nulls in the exists method

If the allow_null argument is true we now allow any result
to return as true. This nullifies the bool() check
So seems bad, but in some cases when an attribute exists
When looking up 'get_object_attribute_names' we want
the value of that attribute even if it is null

@mkanoor would you mind looking at this.


Public accounts do not receive memos

The help for wallet_account_register claims that when account_type is public_account, the account will not be able to receive memos.
I can't think of any good reason why public accounts can't receive memos.  It seems like it would be trivial to simply add a random nonce to the memo and encrypt it with the recipient's public key.

Related: #1241


Only hide the interface after everything has been set up for execution

Fixes #2268 while keeping the "clear on close" behavior of aeaa261352e3caa42838e3a1f4caa2a9ef90953c

Seems to work. Wish I could figure out why saveMnemonic gets hit twice when you type ↩. It claims to be coming from insertNewline: both times, but if I move the breakpoint there, it only gets hit once.
Another issue for another day.


Update scale-pool.sh

@CarlRabeler,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

#sign-off


Async Field validation on Field attribute

If we want to extract behavior to a component is necessary to have an asyncValidation at level field.
Something like
I need to extract this behavior to the field and not in the form, because i want to use this async  validation in  many forms and i don't think that repeat it in every form is the best way to do that.
If we encapsulate the behaviour in the Field, we only have to do it once.

You're not the first one to suggest this - see #2763.


Fit function to work with adaptive step size

Hi,
Is there a way to make the generic fit function work but have different steps in each epoch?
I'm using tf.dataset.sample_from_datasets and I don't get the same amount of steps each epoch, I don't want to use repeat and I do want to run over all my positive samples without setting the steps_per_epoch.

Hi @Danojungo ,

Could you use a custom training loop instead of the generic fit function. This approach provides more flexibility, allowing you to adjust the number of steps dynamically for each epoch.
The generic fit function in tensorflow does not support varying the number of steps per epoch directly.

Thank you!


Video, images, news sitemaps

Consider adding built in support for other types of sitemaps: videos, images and news.

+1 this feature would be really useful


feat: add overrideable FavoriteToggle to core-components

Hey, I just made a Pull Request!
Background
We need to allow customisation of the favorite icon color, while also standardising the pattern for marking something as "favorite" (currently used on entitiy rows, cards as well as scaffolder templates).
Closes https://github.com/backstage/backstage/issues/26385
Changes

Add generic controlled component FavoriteToggle
FavoriteToggle defaults to previous hard-coded yellow #f3ba37
Expose icon separately as FavoriteToggleIcon for use with MaterialTable
Replace all occurrences of favorite icon and tooltip with FavoriteToggle

Screenshots

:heavy_check_mark: Checklist

[x] A changeset describing the change and affected packages. (more info)
[ ] Added or updated documentation
[x] Tests for new functionality and regression tests for bug fixes
[x] Screenshots attached (for UI changes)
[x] All your commits have a Signed-off-by line in the message. (more info)

@benjdlambert strange, yarn changset did not include core-components for some reason. Added it now :+1:

@benjdlambert thank you!


Conflict with DFred’s Convenient Effects

With DFred’s Convenient Effects enabled, automatic cover doesn’t work because of existing conditions in Dfred’s.
Suggestion: Check for Dfred’s enablement and use those conditions instead of inbuilt ones if on.
Thanks!
Chris

Just saw the update! Huzzah!

Yep! Let me know if you find any weirdness with it; I am sure it could use additional testing.


time block color coded

-grey for past
-yellow for present
-green for future

red is for present color!

this is done when we added the different classes in if statements during the check time function.


Dashboard - Bookmark Template Functionality

Feature Description
As an editor I want to be able to bookmark templates
Feature Brief (Being Written)

Do not alter or remove anything below. The following sections will be managed by moderators only.
Acceptance Criteria

Write the JS function in in order to bookmark a template
This is executed on the Template Grid View #774 and the Template Detail View #775
Setup the data structure to be stored somewhere in WP (contact @swissspidy or @spacedmonkey for assistance)

QA Instructions

@carlos-kelly to confirm the 1.2 assignment for prioritization


Clone error when grouping actions

In my Zustand store I have all the actions grouped together like this (I got the idea from this blog post: https://tkdodo.eu/blog/working-with-zustand#separate-actions-from-state)
const useBearStore = create((set) => ({
  bears: 0,
  fish: 0,
  actions: {
    increasePopulation: (by) => set((state) => ({ bears: state.bears + by })),
    eatFish: () => set((state) => ({ fish: state.fish - 1 })),
    removeAllBears: () => set({ bears: 0 }),
  },
}))

But this approach results in a clone error when trying to share the store between tabs. I'm guessing that's because the actions object contains functions, that can't be send as mentioned on the docs.
So to make it work I have to flatten my store so the actions are on the same level as the state, like this:
const useBearStore = create((set) => ({
  bears: 0,
  fish: 0,
  increasePopulation: (by) => set((state) => ({ bears: state.bears + by })),
  eatFish: () => set((state) => ({ fish: state.fish - 1 })),
  removeAllBears: () => set({ bears: 0 }),
}))

Is there any way to keep my original structure that doesn't error?

Thanks for your reply! Don't worry, I'll just flatten the structure, it's not a big deal.


List of version features

Hi all,
Is there a place where I can see the new features of the versions that are releases?
Many thanks!

@acamenhas https://github.com/GeekyAnts/NativeBase/releases


RUSTSEC-2020-0056: stdweb is unmaintained

stdweb is unmaintained

Details

Status
unmaintained

Package
stdweb

Version
0.4.20

URL
https://github.com/koute/stdweb/issues/403

Date
2020-05-04

The author of the stdweb crate is unresponsive.
Maintained alternatives:

wasm-bindgen
js-sys
web-sys

See advisory page for additional details.

fixed


Cherry Pick Merge of @S0xbad1dea PR and my FQDN fix

Merge of https://github.com/ukncsc/lme/pull/4 with cherry picked changes
This PR mostly takes from @super0xbad1dea with some extra additions and removals.
The hash selection has not been changed on the command line, This is because this setting is overwritten by the config xml.
FQDN has been fixed by stripping whitespace rather than by removing.
Option to change install path was not added. We want to keep consistency on LME installs to allow upgrades to be 'easy'

Looks good Duncan. Thanks @super0xbad1dea again for taking the time to submit a PR.

This also fixes #13


Database Grooming

Overview
We need to groom the database and remove the "junk" data needs to be deleted. CNK spoke with Brian
who said front end can automate tests
We need to add labels as well for sorting purposes. Add labels "New Issue Approval" and "Project Management."
Resources/Instructions
See #877

@ladissi can this issue be closed?


OpenXml Does not fully support .Net 4.5 (TargetFramework)

Description: OpenXml Does not fully support .Net 4.5 (TargetFramework)
I will have to inform you that the version of DocumentFormat.OpenXml 2.8.1 does not fully support .Net 4.5.2
VisualStudio:

.NET Target: 4.5
DocumentFormat.OpenXml Version: 2.8.1

Repro

Create a .Net 4.5 Target Console Project.

Test the following code:
using (SpreadsheetDocument spreadSheetDocument = SpreadsheetDocument.Open(excelStream, false)

Error: Method not found: 'System.String System.String.Format(System.IFormatProvider, System.String, System.Object, System.Object, System.Object)'.

My conclusion:
The Method only exists in mscorlib that comes with .Net 4.6..
Please test, and note me if my assumptions are correct. As pr. now i do not have a workaround for this error since i am not allowed to upgrade the .Net version on the environment...
Please send me a mail if you have fixed this and/or if you know of any workaround.

I set up a machine with only .NET 4.5.2 and could not repro this issue. Can you supply a full repro project? Preferably a GitHub link to a project, but a zip would work too. Thanks!

For clarification, you are targeting .NET 4.5, but running on a machine with .NET 4.5.2, right? Can you provide the following information:

What version of Windows? (ie what does WinVer say?)
The version of .NET as provided by the documentation here: https://docs.microsoft.com/en-us/dotnet/framework/migration-guide/how-to-determine-which-versions-are-installed

I received more details in an email (@kjhano if that wasn't you, then let me know, but the issue was the same). This was occurring because the assemblies for .NET 4.6 were being taken from the nupkg rather than using NuGet to manage the package deployment.


ArrayIndexOutOfBoundsException from Android Runtime when scan is initiated

Running example app on Android 6.0, I get this exception when scan is initiated.
D/BluetoothDevice(11526): mAddress: 25:0A:C4:0C:FF:2C
D/BluetoothDevice(11526): getName: name = MyBLE
D/BluetoothDevice(11526): getType: type = 2
D/AndroidRuntime(11526): Shutting down VM
E/AndroidRuntime(11526): FATAL EXCEPTION: main
E/AndroidRuntime(11526): Process: com.pauldemarco.flutterblueexample, PID: 11526
E/AndroidRuntime(11526): java.lang.ArrayIndexOutOfBoundsException: Not enough data.
E/AndroidRuntime(11526):        at com.pauldemarco.flutterblue.AdvertisementParser.parse(AdvertisementParser.java:60)
E/AndroidRuntime(11526):        at com.pauldemarco.flutterblue.ProtoMaker.from(ProtoMaker.java:30)
E/AndroidRuntime(11526):        at com.pauldemarco.flutterblue.FlutterBluePlugin$2.onScanResult(FlutterBluePlugin.java:625)
E/AndroidRuntime(11526):        at android.bluetooth.le.BluetoothLeScanner$BleScanCallbackWrapper$1.run(BluetoothLeScanner.java:362)
E/AndroidRuntime(11526):        at android.os.Handler.handleCallback(Handler.java:815)
E/AndroidRuntime(11526):        at android.os.Handler.dispatchMessage(Handler.java:104)
E/AndroidRuntime(11526):        at android.os.Looper.loop(Looper.java:207)
E/AndroidRuntime(11526):        at android.app.ActivityThread.main(ActivityThread.java:5728)
E/AndroidRuntime(11526):        at java.lang.reflect.Method.invoke(Native Method)
E/AndroidRuntime(11526):        at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:789)
E/AndroidRuntime(11526):        at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:679)

I got the same issue, with a specific hardware device in development. There are two times a long local name (Type 9) in it and no Type 8 part (for the short local name). The implementation of the AdvertismentParser breaks out of the loop as soon as it founds a second long local name, without moving the position in the bytestream to parse further.
I tried a change in code adding one line, to get it run through:
case 0x09: { // Long local name.
                    if (seenLongLocalName) {
                        // Prefer the long name over the short.
                        data.position(data.position() + length); // <-- new line
                        break;
                    }

Although I think it would be fine, to not throw the exception, but to skip the device entirely. The hardware is not setup correctly, so I guess it would be fine for the use case of this lib,

Hi, I wanted to bump this back up to see if anyone was able to figure out another solution. I am trying to connect to my ESP32 dev board, and that is exactly the device that crashes the app when a scan is initiated--scanning is perfectly fine if . Running the scan by itself is fine and it produces a _ControllerStream object but trying to access any part of that object or running any method throws the ArrayIndexOutOfBoundsException. I was trying to figure out if there was a way to change the BLE settings on my board but that did not work either.
Is there a way to simply connect to board manually if I know the MAC address? I wasn't able to get it working and I read somewhere that for iOS you have to scan then connect for BLE.

@gautam-chebrolu

Did you debug, where the ArrayIndexOutOfBounds happens? Is it only when running the Android App or on iPhones/iPads too? When only on Android then see my answer from April 13th.
We also used an ESP32 and got this Exception. Our firmware developer fixed the problem by setting the long name and the short name of the device explicitly. By default the ESP32 seem to be having two long name entries, but no short name entry. This revelead the bug in the parser of the lib.

@MSchrenker

Thank you for your solution! I had it running with that but I haven't been able to check how it runs on iOS yet and I wanted something that would run cross-platform. I was assuming that this would fix only Android issues.
I will look into editing espressif's code as well. I don't have too much experience in it, but will report back with how it works.

Hey im having the same issue! have you found a solution?

@lemonbuzz  We modified the Arduino sample from here:
https://github.com/nkolban/esp32-snippets/blob/master/cpp_utils/tests/BLETests/Arduino/BLE_notify/BLE_notify.ino
and replaced:

// Start advertising
 pServer->getAdvertising()->start();

with something like this:
// Start advertising
 //pServer->getAdvertising()->start();
 BLEAdvertising* pAdvertising = pServer->getAdvertising();
 BLEAdvertisementData data;
 data.setFlags(ESP_BLE_ADV_FLAG_GEN_DISC | ESP_BLE_ADV_FLAG_BREDR_NOT_SPT);
 data.setName("2nd Server");
 data.setShortName("4th Server");
 pAdvertising->setAdvertisementData(data);
 pAdvertising->start();

HTH

@MSchrenker Thank you very much! Your solution of Apr 13 works perfect.


build: use tsup dts option

Seeing as our tests do not need composite builds we can disable that in src and then enable dts in tsup to make the build process quicker and easier

Codecov Report

Merging #152 (f909c85) into main (ebc46de) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #152   +/-   ##
=======================================
  Coverage   99.31%   99.31%
=======================================
  Files          56       56
  Lines        3230     3230
  Branches      801      801
=======================================
  Hits         3208     3208
  Misses         13       13
  Partials        9        9

Continue to review full report at Codecov.

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


feat: support Smithy 2.0

Issue #, if available: #105
Description of changes: Implements Smithy 2.0 support. Concretely, this updates Smithy dependencies to the latest version (1.28.1 as of this writing) which can read Smithy IDL 2.0 models.
Most of smithy-dafny's code generation logic is unchanged because the underlying Smithy semantic model has not undergone any breaking changes. The exception is that supporting Smithy 2.0's new enum shape (as opposed to @enum-attached string shapes) requires changes to shape discovery and generated symbol names. Thankfully the Smithy tooling attaches SyntheticEnumTrait (extends EnumTrait) to each Smithy 2.0 enum shape, so that the codegen logic can inspect enum shapes just like the old @enum string shapes.
Notes for reviewers:

The SimpleEnumV2 test model is mostly copied from the SimpleEnum test model, with names changed to include "V2". I'd recommend only skimming over these files.
The most important changes are those made in the codegen logic. In particular, I'm less familiar with the Java codegen structure, so I'd appreciate extra attention to it (cc @texastony).

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@ShubhamChaturvedi7 This may unblock your use of https://github.com/aws/aws-cryptographic-material-providers-library/compare/main...tony/feat-update-ddb-model


remove unnecessary std::string in source

Signed-off-by: Tarun Sharma<EMAIL_ADDRESS>https://github.com/envoyproxy/envoy/issues/11318

Commit Message:
Additional Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional API Considerations:]

/retest

@yanavlasov  can you take a another look i have updated this PR and will also add few more if this works.
Also the test which are failing in azure CI is passed locally, do we have to run it again ?

/retest

/wait

/retest


Split cases

In some other languages and editors what we call "generate union pattern match case" is called case splitting. Can we please use that description for this action? Either "case split" or "split cases".
https://github.com/ionide/ionide-vscode-fsharp/blob/63c91aeb0f7b7acf81431a13107530f483b772c5/src/Components/UnionCaseGenerator.fs#L32

Thanks for reviewing this @Krzysztof-Cieslak. It is the first issue I've raised for this project.
So that I'm getting the terminology right with respect to F#, from the section of the language manual on pattern matching, this editor action is offering to write out identifier patterns for each case value of the discriminated union being matched.
A convenience of this description to split out of each element of the set of possible cases is that it covers the single case discriminated union equally well with no need to awkwardly write split case(s).


Fix Rubygems outdated certificate

The bundled Rubygems certificate AddTrustExternalCARoot.pem is outdated; replaced it with the current GlobalSignRootCA_R3.pem, from the Rubygems project.
See:

https://github.com/rubygems/rubygems/issues/4099
https://github.com/rubygems/rubygems/pull/4100
https://github.com/rubygems/rubygems/blob/master/lib/rubygems/ssl_certs/rubygems.org/GlobalSignRootCA_R3.pem

Closes #141.

Rebased - my fork master branch was old.

https://github.com/rubygems/rubygems/blob/master/lib/rubygems/ssl_certs/rubygems.org/GlobalSignRootCA_R3.pem

Where am I supposed to replace.

This issue is affecting me too. Looks like since the checks have all failed that this merge has been put on hold.

This issue is affecting me too. Looks like since the checks have all failed that this merge has been put on hold.

There are multiple issues in the current master, that I've fixed in several PRs, however, the repository is effectively abandondoned, and the maintainer is ignoring people's contributions. I'm planning to create a fixed fork, but I don't have currently the resources.

@saveriomiroddi Thanks for the update.  I have been trying to get the michaeltelford/broken_link_finder) gem into a binary for a box that I do not have root on, but I think that am running into multiple issues.

@saveriomiroddi Thanks for the update. I have been trying to get the michaeltelford/broken_link_finder) gem into a binary for a box that I do not have root on, but I think that am running into multiple issues.

In order to get a working version, fork my repository, and merge all the branches corresponding to the open PRs.
You should get a working version. I can't guarantee 100% it works because a month passed since when I opened the PRs. But a month ago the project was building! :smile:

Seems this is not the only project to make Ruby programs easy to install that is facing the issue of maintainability. In searching for other alternatives I found Traveling Ruby and then a blog post by the maintainer about its future. After reading the blog post I am first surprised that two programmer manage to mostly single handed to get two projects to make Ruby programs easier to install up and running. But I am not surprised that they are having a hard time maintaining them and keeping up with the steady march in software technology. Looks like a team of maintainers and sponsorship will be needed to keep one afloat.

Seems this is not the only project to make Ruby programs easy to install that is facing the issue of maintainability. In searching for other alternatives I found Traveling Ruby and then a blog post by the maintainer about its future. After reading the blog post I am first surprised that two programmer manage to mostly single handed to get two projects to make Ruby programs easier to install up and running. But I am not surprised that they are having a hard time maintaining them and keeping up with the steady march in software technology. Looks like a team of maintainers and sponsorship will be needed to keep one afloat.

It's important to split the theoretical aspect from the practical one.
It's absolutely true that open source maintainers owe nothing to the users, so if they struggle to maintain, they are entirely entitled to deal with the project cold feet.
On the other hand though, it's disrespectful towards the users not to make it clear that a project is abandoned, for two reasons:

users waste their time thinking that a project works, while it's broken;
even worse, developers may spend their time fixing the issues, and have their contributions ignored.

Another additional problem is that the reference, broken, project will divert attention from the fixed forks.
It takes a couple of clicks to make a project read-only, so there's no excuse for not doing it.
I'm a maintainer myself, not only a user, so I'm familiar with maintenance conduct.

@saveriomiroddi I agree whole heartily with you. I too am a maintainer of projects and have had to abandon them. I always try to find someone to maintain them if there seems to be a number of users but I alway archive the project if it is done.  I have run into this before. Now I try to check what the last commit to a project is before looking to use it. Also a look at the issue queue give a good feel for a projects health.


Using deprecated parameters already removed from superclass in Sampler causes erros

Describe the bug
It is not possible to instantiate a sampler from the runtime library
File ~/qiskit_git/lib/python3.10/site-packages/qiskit_ibm_runtime/sampler.py:125, in Sampler.__init__(self, circuits, parameters, service, session, options, skip_transpilation)
     88 """Initializes the Sampler primitive.
     89
     90 Args:
   (...)
    118         Ignored if ``skip_transpilation`` is also specified in ``options``.
    119 """
    120 # `self._options` in this class is a Dict.
    121 # The base class, however, uses a `_run_options` which is an instance of
    122 # qiskit.providers.Options. We largely ignore this _run_options because we use
    123 # a nested dictionary to categorize options.
--> 125 super().__init__(
    126     circuits=circuits,
    127     parameters=parameters,
    128 )
    130 if skip_transpilation:
    131     deprecate_arguments(
    132         "skip_transpilation",
    133         "0.7",
    134         "Instead, use the skip_transpilation keyword argument in transpilation_settings.",
    135     )

TypeError: BaseSampler.__init__() got an unexpected keyword argument 'circuits'

Steps to reproduce
Instantiate a Sampler
Expected behavior
The inheritance works as expected
Suggested solutions
Remove deprecated parameters from class and from reference to superclass
Additional Information
Tested agains the main branch od qiskit-terra
This commit https://github.com/mtreinish/qiskit-core/commit/75673c7867222a9d8048d5264053162c5cfbe802 3 days ago removes the "circuits" parameter from BaseSampler, and it is not already adjusted in the ibm_runtime library.

qiskit-ibm-runtime version:0.9.4
Python version:3.10
Operating system:MAC

This will be fixed in the newest release, 0.9.4
https://github.com/Qiskit/qiskit-ibm-runtime/pull/797


Refactor booking to support all types

Closes #112
TODO

[x] airlines logo needs to be fixed
[x] render header properly
~- [ ] render past booking if future not available~LiveURL: https://kiwicomsmart-faq-dzlicfzjnd.now.sh

All should be fixed or resolved, cc @git-toni

Great job, thx!


Print/OLog from replay

A button in Replay to print a scan would be very helpful, also, a button to send it to olog together with some comments.
requested by @skalbfleisch

@tacaswell asked "What do you mean by 'print a scan'?"
I am also not sure what you mean by "print a scan"

Moved to NSLS-II/replay#17


two handlebars security audits

Link to bug demonstration repository

Expected Behavior
no security audits
Observed Behavior
violations due to use of handlebars version 4.5.2 (fixed in 4.5.3)
https://npmjs.com/advisories/1324
https://npmjs.com/advisories/1325
Troubleshooting steps

[x] still occurring when I put cache: false in my nyc config

Environment Information

  System:
    OS: macOS Mojave 10.14.6
    CPU: (16) x64 Intel(R) Core(TM) i9-9880H CPU @ 2.30GHz
    Memory: 664.65 MB / 16.00 GB
  Binaries:
    Node: 12.13.0 - /usr/local/bin/node
    Yarn: 1.5.1 - /usr/local/bin/yarn
    npm: 6.12.0 - /usr/local/bin/npm
  npmPackages:
    nyc: 14.1.1 => 14.1.1
    typescript: 3.7.2 => 3.7.2

This is not a bug in nyc, we use semver ranges to install so the fixed versions are already allowed by nyc.  The problem is with your yarn.lock or package-lock.json blocking the update.  See https://github.com/istanbuljs/istanbuljs/issues/476 for details and how resolve the lockfile error.

thanks, yes, we have done this. i was reporting it here so that the fix could roll through our upstream deps. thanks for the response.

nyc 15 actually removes handlebars entirely so it will never result in another audit report via nyc.  See #1104 if you want to try that version which will become latest stable very soon (be sure to check changelogs posted as comments for breaking changes).
As far as updating the dependency chain of 14 we have to prioritize the work we do.  Older versions of nyc bundled dependencies which meant that npm audit vulnerabilities were in-fact our fault.  We no longer bundle so this means that nyc itself is no longer the cause of most audit reports.  If the vulnerability fix were an out of range update we would likely perform a backport to update our dependency but for in-range updates I simply don't have time (and it's fixable on the user side).


Capture stdout and stderr of go list separately

Because go list outputs the log for go modules to stderr.
[goxz] [!!ERROR!!] can't build artifact for non main package: "go: finding github.com/davecgh/go-spew v1.1.1\ngo: finding github.com/iancoleman/orderedmap v0.0.0-20190318233801-ac98
e3ecb4b0\ngo: finding github.com/stretchr/testify v1.3.0\ngo: finding github.com/davecgh/go-spew v1.1.0\ngo: finding github.com/stretchr/objx v0.1.0\ngo: finding github.com/pmezard/
go-difflib v1.0.0\ngo: downloading github.com/iancoleman/orderedmap v0.0.0-20190318233801-ac98e3ecb4b0\ngo: extracting github.com/iancoleman/orderedmap v0.0.0-20190318233801-ac98e3e
cb4b0\nmain"

Thanks. released as v0.3.3.


Add examples to README

@jcnelson Could you add some usage examples to the readme?

Added.  Will add more as we need them.

I updated the README to direct people to our public api in blockstack.js.
Usage docs for this library are in docs/


Fast Mersennes Prime Field

[ ] Implement the 32 bit Mersennes Prime Field, F = 2^31 - 1, with an efficient backend
[ ] Implement the degree 4 field extension
[ ] Implement mixed radix FFT/NTT for Mersennes Prime Field* (A compatible API with current FFT/NTT should be discussed)

It's recommended for new collaborators to only do the first task, since the code has no examples of field extensions for FRI
The last task needs discussion on how would be the best approach to implement it, API wise.
For references, see:
Reed-solomon over circle fields
Plonky 3 implementation

Will take this


Migrating from Swagger 2.0 to OpenAPI 3.0

I wanted to migrate the Swagger 2.0 specification of the project to OpenAPI 3.0 to achieve better compatibility with OpenAPI code generators and to keep the project up-to-date with the latest standards.
Additionally, for personal needs, I have already recreated and corrected the specification in OpenAPI 3.0 to accurately reflect the response payloads and match the actual behavior of the API requests.
So if it's ok, I can open a pull request 🙂

Hey @Paul-vrn . Thanks for the initiative and the collaborative spirit.
I am sorry to redirect you to #442
There is little use for yet another abandoned pull-request unless the project is rebooted, and that seems unfeasible. For all it's worth, just keep those improvements in your fork and announce that in #375 / #399

@Paul-vrn Hi, could you please share Open Api 3.0 specification?


An option to configure RSpec/NamedSubject to make it allow expect { subject }

It would be very nice if there is an option to configure RSpec/NamedSubject cop so that it allows expect { subject }.
RSpec/NamedSubject works great when we test against an object returned by a method; however, I think it is a little bit difficult to handle when simultaneously testing side effects in the same describe group or context. (I found a discussion on the same topic, but seems there is no definite solution yet, except disabling the cop.)
For example, the cop warns against this code because it uses subject for expectations.
RSpec.describe Car do
  describe '.build' do
    subject { Car.build({ color: color }) }

    context 'with an available color' do
      let(:color) { 'red' }

      it { expect(subject.color).to eq 'red' }
      it { expect { subject }.to change(BuildLog, :count).by(1) }
    end

    context 'with an unavailable color' do
      let(:color) { 'lightblue' }

      it { expect { subject }.to raise_error(CarError).with_message('Unavailable color.') }
    end
  end
end

Following the cop's suggestion, the code will probably be like the below;
RSpec.describe Car do
  describe '.build' do
    subject(:car) { Car.build({ color: color }) }

    context 'with an available color' do
      let(:color) { 'red' }

      it { expect(car.color).to eq 'red' }

      # "It except car to change BuildLog count" ? :(
      it { expect { car }.to change(BuildLog, :count).by(1) }
    end

    context 'with an unavailable color' do
      let(:color) { 'lightblue' }

      # "It expect `car` to raise error" ? :(
      it { expect { car }.to raise_error(CarError).with_message('Unavailable color.') }
    end
  end
end

Otherwise;
RSpec.describe Car do
  describe '.build' do
    subject(:build_car) { Car.build({ color: color }) }

    context 'with an available color' do
      let(:color) { 'red' }

      # "It expect build_car's color to be red" ? :(
      it { expect(build_car.color).to eq 'red' }
      it { expect { build_car }.to change(BuildLog, :count).by(1) }
    end

    context 'with an unavailable color' do
      let(:color) { 'lightblue' }

      it { expect { build_car }.to raise_error(CarError).with_message('Unavailable color.') }
    end
  end
end

The code above smells, so I think we need to have something like an option to configure the cop to make it more flexible. If this idea sounds reasonable to others, I would love to make a PR. Any opinions?

How do you feel about:
RSpec.describe Car do
  describe '.build' do
    def build_car!
      Car.build(color: color)
    end

    subject(:car_color) { build_car!(color: color).color }

    context 'with an available color' do
      it { expect(car_color).to eq 'red' }
      it { expect { build_car! }.to change(BuildLog, :count).by(1) }
    end

    context 'with an unavailable color' do
      let(:color) { 'lightblue' }

      it { expect { build_car! }.to raise_error(CarError).with_message('Unavailable color.') }
    end
  end
end

From RSpec docs:

we recommend that you reserve it (subject helper) for support of custom matchers and/or extension libraries that hide its use from examples.
We recommend using the named helper method over subject in examples.

You might not have the best example, but the previously mentioned discussion seems to come to a conclusion that named subjects facilitate reading specs:

extra clarity that a named subject could give in this situation defeats the mental energy required to think about a good name for it

Thanks for the suggestion and some useful links. I was mistaken about the use of subject :)
Just for additional reference, I found an old discussion on a similar topic, and they concluded the same.
https://github.com/betterspecs/betterspecs/issues/7#issuecomment-150391098


Make custom component directories consistent

Potentially move custom form fields and table columns into Filament/ in the app instead of top-level Forms/ and Tables/ directories.

Donate 💰 to fund this issue

You can donate funding to this issue. We receive the money once the issue is completed & confirmed by you.
100% of the funding will be distributed between the Filament core team to run all aspects of the project.
Thank you in advance for helping us make maintenance sustainable!

@danharrin could you reference the commit that solves this issue? 🙂

No sorry, it was done very early in v4 development and I don't really want to go back through the history, you can check the blame if you want

Aight, found it: https://github.com/filamentphp/filament/commit/19c142c232824a96442bf6c842cd8a7323ebb696.
Thanks!


[Declaration macros] Initial version

Whoops, didn't mean to open this just yet. It hasn't been pitched