Orientdb 2.2 Beta: Cannot update new ORole with inheritedRole writer when writer has record level access

I set record level access to default ORole admin, reader and writer. Only admin rid is in _allow, then I add all other ORole rids in _allowRead.
I created another ORole vertex (say admin2) where inheritedRole is writer (rid #4:2). When I update admin2 vertex it says this error:
com.orientechnologies.orient.core.exception.OSecurityException: Cannot update record #4:2: the resource has restricted access
I'm only updating the admin2 ORole vertex and not the writer ORole. This worked in OrientDB 2.2 Alpha.

hi @cvm1ll3,
Can you post the full stack trace here ?
In some cases is correct that a writer is not allowed to update roles, to avoid role escalation.
Regards

Hi @tglman,
Is this the full stack trace?

{
    "errors": [
        {
            "code": 400,
            "reason": "Bad request",
            "content": "Error on parsing script at position #2: Error on execution of the script\nScript: SETUP_createAlias\n-------^\r\n\tStorage URL=\"plocal:/Users/altermedia/Desktop/CamilleDocs/OrientDB/orientdb/distribution/target/orientdb-community-2.2.0-SNAPSHOT.dir/orientdb-community-2.2.0-SNAPSHOT/databases/StudioSuiteSnapshot001\"\ncom.orientechnologies.orient.core.exception.OSecurityException: Cannot update record #4:2: the resource has restricted access\r\n\tStorage URL=\"plocal:/Users/altermedia/Desktop/CamilleDocs/OrientDB/orientdb/distribution/target/orientdb-community-2.2.0-SNAPSHOT.dir/orientdb-community-2.2.0-SNAPSHOT/databases/StudioSuiteSnapshot001\" in  at line number 2789 at column number 2\ncom.orientechnologies.orient.core.exception.OSecurityException: Cannot update record #4:2: the resource has restricted access\r\n\tStorage URL=\"plocal:/Users/altermedia/Desktop/CamilleDocs/OrientDB/orientdb/distribution/target/orientdb-community-2.2.0-SNAPSHOT.dir/orientdb-community-2.2.0-SNAPSHOT/databases/StudioSuiteSnapshot001\"\nCannot update record #4:2: the resource has restricted access\r\n\tStorage URL=\"plocal:/Users/altermedia/Desktop/CamilleDocs/OrientDB/orientdb/distribution/target/orientdb-community-2.2.0-SNAPSHOT.dir/orientdb-community-2.2.0-SNAPSHOT/databases/StudioSuiteSnapshot001\""
        }
    ]
}

I can update an ORole vertex before using a role with inherited writer role #4:2. The error seems to be that the writer role is also being updated even if its just an inherited role? Thanks.

HI @tglman,
any updates on the error? I just tested it again in the recent sources and rc1 and it still produces the same error. Thanks.

Hi @cvm1ll3,
I haven't been able to write a precise test case to reproduce the problem, that's the main reason why is not fixed, if you can provide a simple test case, will be really easier to fix.
Regards

Hi @cvm1ll3,
I haven't been able to write a precise test case to reproduce the problem, that's the main reason why is not fixed, if you can provide a simple test case, will be really easier to fix.
Regards

Same issue here. I will try to provide a test case.

maybe not related but I get this kind of exceptions when I delete a vertex :
com.orientechnologies.orient.core.metadata.security.ORestrictedAccessHook.onRecordBeforeUpdate(com/orientechnologies/orient/core/metadata/security/ORestrictedAccessHook.java:91)
com.orientechnologies.orient.core.hook.ODocumentHookAbstract.onTrigger(com/orientechnologies/orient/core/hook/ODocumentHookAbstract.java:257)
com.orientechnologies.orient.core.db.document.ODatabaseDocumentTx.callbackHooks(com/orientechnologies/orient/core/db/document/ODatabaseDocumentTx.java:1052)
com.orientechnologies.orient.core.tx.OTransactionOptimistic.addRecord(com/orientechnologies/orient/core/tx/OTransactionOptimistic.java:423)
com.orientechnologies.orient.core.tx.OTransactionOptimistic.saveRecord(com/orientechnologies/orient/core/tx/OTransactionOptimistic.java:370)
com.orientechnologies.orient.core.db.document.ODatabaseDocumentTx.save(com/orientechnologies/orient/core/db/document/ODatabaseDocumentTx.java:2529)
com.orientechnologies.orient.core.db.document.ODatabaseDocumentTx.save(com/orientechnologies/orient/core/db/document/ODatabaseDocumentTx.java:99)
com.orientechnologies.orient.core.record.impl.ODocument.save(com/orientechnologies/orient/core/record/impl/ODocument.java:1794)
com.orientechnologies.orient.core.record.impl.ODocument.save(com/orientechnologies/orient/core/record/impl/ODocument.java:1785)
com.tinkerpop.blueprints.impls.orient.OrientBaseGraph.removeInverseEdge(com/tinkerpop/blueprints/impls/orient/OrientBaseGraph.java:2161)
com.tinkerpop.blueprints.impls.orient.OrientBaseGraph.removeEdges(com/tinkerpop/blueprints/impls/orient/OrientBaseGraph.java:2064)
com.tinkerpop.blueprints.impls.orient.OrientGraph.removeEdgesInternal(com/tinkerpop/blueprints/impls/orient/OrientGraph.java:440)
com.tinkerpop.blueprints.impls.orient.OrientVertex.remove(com/tinkerpop/blueprints/impls/orient/OrientVertex.java:499)

The strange thing is that the vertex is deleted despite the error (which is ok because the user is in _allow).

hi @voondo,
for your case the error is due to a linked record that may not be allowed to update, when you use graph a remove of a vertex means an update of all the linked vertices, it can be possible that one of the lined vertex is not allowed to be modified by the current user.
If you are doing a notTx operation is normal that the removed vertex is actually removed because there is no tx to rollback, a stale reference is left in this case in the record that has caused the exception.
we usually suggest to use tx on most of the cases to avoid stale references.
Bye


newly submitted places dont appear instantly in tabsnav and map

maybe something like revalidate in next can be used here

loading works but form does not redirect to "/" right now


adding the new senator official portraits, resized

[ yes] I'm acknowledging that my submission is free of all known copyright in at least
the United States. (In general, works of the federal government should satisfy
this criteria.)

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

Comparison is base (2f3d436) 57.42% compared to head (02ddf1d) 58.00%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@             Coverage Diff              @@
##           gh-pages     #218      +/-   ##
============================================
+ Coverage     57.42%   58.00%   +0.57%
============================================
  Files             2        2
  Lines           101      100       -1
============================================
  Hits             58       58
+ Misses           43       42       -1

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

Please could you put the originals in original, and then resized ones in both 450x550 and 225x275?


Rules aliasing

Why?
Enable custom rules creation/registering with rules aliasing.
Changes

Enable registering rule alias from JSON's or alias properties

Tests

Aliasing negative and positive tests

Issue

Pull Request Test Coverage Report for Build 67

41 of 48 (85.42%)  changed or added relevant lines in 3 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.2%) to 90.634%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

Livr/Rules/LivrRuleProtocol.swift
21
22
95.45%

Livr/Validator.swift
17
23
73.91%

Files with Coverage Reduction
New Missed Lines
%

Livr/Validator.swift
1
87.68%

Totals

Change from base Build 63:
-0.2%

Covered Lines:
958

Relevant Lines:
1057

💛  - Coveralls


Infer hasChildren from children.length?

Seems redundant.

Hi @corysimmons, thanks for the comment!
Actually they are somewhat different, hasChildren defines whether an item should behave like one that has children, i.e. be expandable and collapsible. So if you want to have an empty folder that still shows the caret icon and can be expanded or collapsed, it would need to have the prop hasChildren set to true, even if the children array is empty. If you want empty folders to behave like normal items, they are in fact the same. On hindsight, I admit that this is pretty bad naming, maybe this should be renamed to hasCaret or something..

Ah, that makes a lot of sense, thank you for the examples in your explanation!


Build infrastructure - CMake and CI

Added a MCParticle class just to test the technicalities - running the class generator, compiling the library, creating the dictionaries, etc. The config is pulled from HSF/tools, HEP-FCC/fcc-edm and AIDASoft/podio, and just a starting point.
It seems I still have trouble getting the CI to mount cvmfs, working on it.

Thanks a lot @vvolkl! Re. the CI, I actually always ended up giving up on the Travis images and rolling a docker container that had everything setup and working inside. It's a lot easier to debug that setup as one then has a lot more control over the machine.

@drbenmorgan - PODIO lives here: https://github.com/AIDASoft/podio. Yep, the CMake is probably a bit long in the tooth now...

Thanks @graeme-a-stewart, I'll take a look (and I should have emphasised - don't take the above as a "review" just that's the easiest way to comment!)

Someone needs to enable the travis service for this Repo

Thanks for the comments! @drbenmorgan those are all great points, we just merged this PR to get the CI to run, but I'll try to clean up the local things, and it's a good opportunity to improve things in PODIO also.


New param hashing spec

When listing all params, lead with _HASH_CHECK value for cache load
When set value on _HASH_CHECK is rx'd, stop any ongoing listing req

This corresponds with mavlink/qgroundcontrol#2586

@NaterGator Good to merge?

Yes

Rebased and applied.


Renamed SupportKit to Smooch

Hey guys, just like to drop this. We rebranded this month :)
Thanks!

:+1: thanks! Sorry for the slow reply. We're moving this list into the wiki here, and I've gone ahead and updated it there.


Length 0 checks should come after checking for mismatched sizes

Summary:
Bug in checking for length 0 vectorized arguments.
Description:
This looks like a bug in the vectorized prob functions (possibly just the cdf / ccdfs). It'll looks something like:
// Size checks
if ( !( stan::length(y) && stan::length(mu) && stan::length(sigma) ) )
  return 0.0;

This will not check that the dimensions are actually ok. It'll return 0 if any are length 0.
Bob writes in this pull request comment:

I'd also rather remove all those redundant spaces.  I also
prefer tighter negations for less nesting and to make it clear
that if any of the lengths is zero, the result should be zero.
So I'd write that as:
if (!length(y) || !length(mu) || !length(sigma)) return 0;
And that test should go after the mismatched size tests, which
would indicate a bug, right?

Additional Information:
Provide any additional information here.
Current Version:
v2.11.0

I also think these are clearer as
if (!stan::length(y) || !stan::length(mu) || !stan::length(sigma))
  return 0;

because the nesting is less deep and it's clearer that if any of them are length zero, we return 0 (don't need that 0.0).
We should also make sure not to allocate a variable if we're going to just return constant 0 because it can cause autodiff overhead.

This is easy, but it'll need to wait until #320 is finished or else we'll end up with lots of merge conflicts.

Closed via #1207


chore: remove default CSI component images

Which issue(s) this PR fixes:

Issue longhorn/longhorn#9580
What this PR does / why we need it:

Remove the default CSI component images from the command flag.
Validate the required command flags are not empty.

Special notes for your reviewer:
None
Additional documentation or context
None

@coderabbitai review


The admin should have the ability to see the statuses and roles of all users in the organization

Acceptance Criteria:

[ ] Create a dashboard where all users with their status and roles are visible.
[ ] The following user statuses should be visible:

Invited
Active
De-activated
Invitation expired

[ ] The following user roles should be visible:

Admin
Member

Can you confirm the difference between an Admin and a Member? Admins can, in addition to uploading expenses, invite Members?

Could you please assign this issue to me?

@kitallis @dvenkatr
This is done now with
https://github.com/nilenso/kulu-backend/pull/15
https://github.com/nilenso/kulu-website/pull/17
Status and Role are displayed, however doing this only for Users in the system i.e. those who have accepted an invite.  Status displayed is either active or de-activated.
Displaying all status now, but we could later have a view which defaults to displaying active users with the option to display de-activated users or all together as needed.
Invited and Invitation expired don't seem to be relevant, since they don't correspond to users in the system and what we want to display here is information pertaining to them.
If we want to be tracking invites, i'll suggest having a separate ui option for that since it's semantically different. On a related note, do we want to track invites since an user with an expired token can ask for another invite instead of the admin having to monitor that.


fix(useResizeColumns):fixed rtl layout by reverseResizer prop

by adding this this optional prop to the useTable hook the resizer in the rtl layouts works just like ltr layouts , also we may have the reverse behaviour of resizer as well .
reverseResizer prop added to useTable hook witch is used in useResizeColumns

Hi @RezaHeydarii, I need this feature in y project, But since PR is not yet merged ow do I used this?

Hello @gauravsbagul , this feature was essential in our project too ... so i user https://www.npmjs.com/package/patch-package to add my changes to the react table and it worked just fine as a temp solution.

Will try that, thanks.

This is a good addition, but really needs to have conflicts resolved and all of the build artifacts removed from the PR. If you would rather open a new PR and start fresh, feel free to close and do that.

Feel free to reopen without all of the untracked change files, etc.


Redirect to DokuWiki/GitHub from the contributing guidelines

Related PRs/Issues
Fixes #373. (See @jcohenadad's https://github.com/ivadomed/ivadomed/issues/373#issuecomment-724201969)
Description

I've added sections to the centralized NeuroPoly Contributing Guidelines (DokuWiki) for Issue labeling and PR labeling.
I've modified contributing.rst to point to relevant contributing pages (GitHub repo, DokuWiki, GitHub wiki) instead of containing its own information.

If this sounds good, then the ivadomed labels may need to be tweaked to match the format described in the DokuWiki. The ivadomed Developer Wiki may also need to be updated to document ivadomed-specific dev processes from contributing.rst.
(Cross-project coordination is tricky, though, so please feel free to share any thoughts you may have. :tada:)

Pull Request Test Coverage Report for Build 359837677

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 90.969%

Totals

Change from base Build 356966781:
0.0%

Covered Lines:
5782

Relevant Lines:
6356

💛  - Coveralls

Thanks for the approval, @jcohenadad. :slightly_smiling_face:
I am also curious about input from @charleygros, @andreanne-lemay, and @lrouhier just to make sure the changes are OK with ivadomed's core team. (I wouldn't want to impose changes that weren't welcome! :sweat_smile:)

Yes, this looks good :) and avoids duplication between ivadomed guidelines and neuropoly guidelines that are essentially the same! Thanks @joshuacwnewton

(sorry for late reply, catching up with emails eheh)
Yeah Nah that's a beauty! Love it, will make the things so much easier! Thanks heaps @joshuacwnewton


feat: Logging server

References:

https://docs.python.org/3/howto/logging-cookbook.html#sending-and-receiving-logging-events-across-a-network
https://docs.python.org/3/howto/logging-cookbook.html#running-a-logging-socket-listener-in-production

TODO:

[ ] Docs
[ ] Make it opt-in?

Example usage and output

Closes https://github.com/meltano/meltano/issues/6641
Closes https://github.com/meltano/meltano/issues/8555

https://msgpack.org/ came up during Office Hours as a potential interchange format instead of Pickle


Fix link to #securityDefinitionsObject

Fix to version 2 documentation link.
Within the section Security Requirement Object there are two links to Security Definitions, the second of which has the wrong URL.
This PR has the fix for that link.

Many thanks for the PR! Just to note that this fix was included (with others) in #991, but it was decided not to merge it at that time, as v2.0 is a released and set-in-stone specification, and so making changes to it now (even for confirmed errata) might be seen as confusing.


[Bug][Android] audio track sometimes dispose or canceled after ~25 minute .

when the player open video , and we setAudioTrack ,
the audio track sometimes stop after (20-35) minute ,
and the video counties play without the audio track we set before .

@Joseph-Nathan
Can you try --ao=audiotrack?
There's a new audio output driver for Android upstream:
https://github.com/mpv-player/mpv/pull/12261

where can i found (ao) attribute  , to used it ?

https://github.com/media-kit/media-kit/issues/14#issuecomment-1373205132


Facades cause Ide-Helper to throw 'Segmentation Fault'

Running ide-helper:generate causes a segmentation fault to be thrown when the faces are enabled from this app? Any idea what might be causing this? I wish I had more to give but there isn't much given to me.
http://i.imgur.com/DRaptgQ.png
EDIT: it looks like it's specifically the AlexaRoute facade that causes issue
EDIT: the issue also occurs with neither facade (only the service provider) when running ide-helper:meta

Thanks for this! I'll take a look at it soon. To be honest I haven't been keeping up with some of the recent changes to the ASK API so I'm going to spend some time soon going through everything again and making sure I'm at least a bit closer in terms of feature parity. And also fixing this :).

Give more context:
Tested it out on laravel 5.2, getting composer to grab source from dev-master. Something affects php-fpm, which runs a segmentation fault and the entire service is rebooted, returning a 502 error.
Downgraded to laravel 5.1, and ran composer without specifying the version from the master - just a regular composer require develpr/alexa-app and everything works.
Will try to debug it, but seems like the problem starts after running vendor:publish task.
RD

Any update on this? I'm also getting segmentation faults after following the installation guide with Laravel 5.2 and alexa-app@dev-master.

hey @austinheap, @robson83 , and @michaeljhopkins - sorry I've been so bad at fixing this issue!
I spent about two hours today trying to get back up to speed with all of this. I know I wrote it all, but still it's a bit tough when you don't look at / use it in a while to let things sink back in!
All of this said, I have not spent much time with Laravel 5.2 yet or the changes that were made. I am able to confirm that in fact there is an error happening, but at this point it seems to be triggered simply by setting the service provider  \Develpr\AlexaApp\Provider\LaravelServiceProvider::class,? Can anybody confirm that? In other words, if you do nothing but install the latest version of AlexaApp and then add the service provider in config/app.php do you see an error?
Thanks all, and sorry again I've been so slow with this!

OK, I'm fairly certain at this point that the issue (at least the issue I'm seeing) is related to the aliases that I'm creating. I think something is happening where Aliases are being registered in the service provider, but for some reason with 5.2 when the classes themselves are being loaded they aren't "matching" with the Aliases. Which is causing some sort of circular recursive loop. Simply commenting out $this->setupClassAliases(); in the AlexaServiceProvider may fix the issue, but then these aliases aren't available. I'll keep looking!

OK, I believe this is fixed. I've tagged 0.2.9 and this should resolve all issues with 5.2.
If not, please let me know and I'll take another look!


All health services are not working

Hi! Are all the health services included in DataType working?ExerciseTypeCapabilities doesn't log all the data types. Am not getting the data for speed/pace/floors etc. Same goes for MeasureTypeCapabilities-it only returns HEART_RATE_BPM.  Do they work or there's something I'm missing out here. Please shed some light here sot hat I can be sure of what to follow next. Thanks in advance

The returned data types depend on the exercise type and the device. Some exercise types will return very few data types. RUNNING/HIKING/WALKING will typically return the most. (Though note the exercise type should reflect the type of exercise for the greatest accuracy, especially with respect to derived values like calories.)
The measure client will also typically returns very few data types. (I think on some devices is is limited to heart rate.)

Thank you so much for the reply. Thought so as well. Since when logging the measure client capabilities I can only see it returns HEART_RATE_BPM. Am using Samsung galaxy4 tho'. But even in exercise types RUNNING/HIKING/WALKING  not able to get speed/pace/incline distance datatypes etc. Is there anyway we can have one example added in the existing repository for speed/pace? Since there is not much am getting out of this. Is this a device issue? Am getting steps count, total calories, distance etc. I was wondering if I could get distance data, isn't the incline distance could be calculated in the same way?

If you add DataType.SPEED to the configs, it should be returned. Apply this (very hacky) diff to ExerciseSample:
https://gist.github.com/ithinkihaveacat/ff926b20462cb8e4b3e59a4e6826224b
On a real device, this should produce output like:
2022-05-27 11:51:44.802 21058-21058/com.example.exercise D/qqqqqq: speed = 1.6660000085830688
2022-05-27 11:51:45.802 21058-21058/com.example.exercise D/qqqqqq: speed = 1.468000054359436
2022-05-27 11:51:46.822 21058-21058/com.example.exercise D/qqqqqq: speed = 1.2740000486373901

(You might need to use a real device. The synthetic mode doesn't seem to support speed.)

If you add DataType.SPEED to the configs, it should be returned. Apply this (very hacky) diff to ExerciseSample:
https://gist.github.com/ithinkihaveacat/ff926b20462cb8e4b3e59a4e6826224b
On a real device, this should produce output like:
2022-05-27 11:51:44.802 21058-21058/com.example.exercise D/qqqqqq: speed = 1.6660000085830688
2022-05-27 11:51:45.802 21058-21058/com.example.exercise D/qqqqqq: speed = 1.468000054359436
2022-05-27 11:51:46.822 21058-21058/com.example.exercise D/qqqqqq: speed = 1.2740000486373901

(You might need to use a real device. The synthetic mode doesn't seem to support speed.)

Thank you so much for the response! Will check and try it in real device.

Did this solve the issue @Srijita-Basak ?

Did this solve the issue @Srijita-Basak ?

Yes! Got data this way in device. Thanks much


Publish to npm?

Sorry, this is an added burden on maintenance that I'm not interested in taking on at the moment. I know that sucks for certain dev processes, but I maintain dozens of OSS projects.
I know npm publish doesn't seem like a lot, but on top of updates, pushing, creating a PR, and merging, it's just one more step to mess up.

I understand and respect that haha i also maintain about a dozen OSS things and its a pain


'while' keyword not colored in codeblocks

The while keyword isn't being colored in the codeblocks where it's used.
See https://docs.microsoft.com/en-us/quantum/user-guide/using-qsharp/control-flow?view=qsharp-preview#while-loop
We've encountered something similar with other keywords in the past, but I forget if this repo is the proper spot for the issue. Anyone know the approach for this (or who to contact)?

@DuncanmaMSFT, saw that you helped us with the same issue a while back (our issue, your solution). I wasn't sure if write permissions may be an issue, so was wondering if you might be able to help us out with another quick PR?
Would you be able to add while to the control list of keywords? I.e. the same list as if, else, repeat, etc.
Thanks!

This is still unsolved


When I first enable any catalog, it doesn't show up

Rancher Versions: master 2/21
Steps to Reproduce:

Disable Rancher Certified and Community library (have a Custom catalog enabled)
Enable either catalogs
Go to Catalog drop-down and select that catalog

Results: Nothing shows up, says there are no compatible templates.  They won't show up until I click on All.  It looks like there is no "pull" when I select specific catalog and only on "All".

Comment: This also happens with custom catalogs, the first time I enter them, they don't show up until I hit All.  I also checked the database and the new catalog folders are not there.

Version - master 2/22
Verified fixed


Not able to run on CPU

Some of the code in the repository assumes access to a gpu device, i.e. model.to(0) - would probably be good for end users if both CPU and GPU were supported

Good point – just pushed some changes that got rid of any explicit to(0) in the code and notebooks.


Challenge 5 wagmi 2 backmerge

note: currently I'm not using burner-connector (but added it to package.json) since in this challenge we need a new burner account in every tab. So I use mostly copypasted code with sessionStorage instead of localStorage. Hopefully we'll add a possibility to use sessionStorage in next versions of burner-connector.
note 2: sorry again for commits list, it looks I need to merge pr without squash

Great findings @Pabl0cks !
I refactored Streamer page and divided it into Guru and Rube components as I wanted for months 😄 . Now it's much easier to understand the logic. I found a bug which caused errors and hopefully 1-3 are fixed. Regarding 4 it didn't work from the start. I think we need to create an issue for it?
Also I changed Readme.md (see last commit). If you think wording is not very good please feel free to update it.
Thanks!

I refactored Streamer page and divided it into Guru and Rube components as I wanted for months 😄 . Now it's much easier to understand the logic. I found a bug which caused errors and hopefully 1-3 are fixed. Regarding 4 it didn't work from the start. I think we need to create an issue for it?

Also I changed Readme.md (see last commit). If you think wording is not very good please feel free to update it.

Awesome job! Now is working like a charm, even the 4th point is working fine to me now 🙌
Just added a small comment to check.

Created an issue for it, will fix it a bit later https://github.com/scaffold-eth/se-2-challenges/issues/180


more robust json generation

concatenating strings into json is... unreliable.
I think the original code was written that way because definedTags and freeformTags must be object, not array, or be rejected as invalid, and by default json_encode makes them array, not object, the original code fixed it by writing the json manually, the new code fix it with 'definedTags' =>(object)array()

the JSON_UNESCAPED_SLASHES | JSON_UNESCAPED_UNICODE just makes the resulting json easier to read/debug by humans, if ever need be.

for example if $displayName is foo"bar then the original code would make the json "displayName": "foo"bar", which is a syntax error, and oci would reject the json as malformed, the new code correctly encodes it into "displayName": "foo\"bar" and the json remains syntactically valid

this patch is now confirmed to work :D i just got my server yesterday


Allow defining Autoscale options in CosmosDbPartitionedStorageOptions

The current class CosmosDbPartitionedStorageOptions used to configure CosmosDbPartitionedStorage doesn't allow to define the container throughput with in auto scale mode. It's possible to consider the change of the Property ContainerThroughput from int to Nullable in order to allow defining it as null. In the private method CosmosDbPartitionedStorage .CreateContainerIfNotExistsAsync this property is used to call the method ContainerBuilder.CreateIfNotExistsAsync that accepts a Nullable and, if I well remember, this will put the container in auto scale mode if is created in a database with auto scale enabled.

@mdrichardson @mrivera-ms could you please triage this issue?

Hi @vifani
Thank you for raising this issue.  We will consider this for a future release.  Until then, it is possible to create an IHostedService implementation to provide custom cosmosdb container creation:
this demonstrates a custom Time to Live setting for the container:
public class CosmosDbStorageInitializerHostedService : IHostedService
{
    readonly CosmosDbPartitionedStorageOptions _storageOptions;
    readonly int _cosmosDbTimeToLive;

    public CosmosDbStorageInitializerHostedService(IConfiguration config)
    {
        _storageOptions = new CosmosDbPartitionedStorageOptions()
        {
            CosmosDbEndpoint = config["CosmosDbEndpoint"],
            AuthKey = config["CosmosDbAuthKey"],
            DatabaseId = config["CosmosDbDatabaseId"],
            ContainerId = config["CosmosDbConversationStateContainerId"]
        };

        _cosmosDbTimeToLive = config.GetValue<int>("CosmosDbTimeToLive");
    }

    public async Task StartAsync(CancellationToken cancellationToken)
    {
        using (var client = new CosmosClient(
            _storageOptions.CosmosDbEndpoint,
            _storageOptions.AuthKey,
            _storageOptions.CosmosClientOptions ?? new CosmosClientOptions()))
        {
            // Create the contaier with the provided TTL
            var containerResponse = await client
                .GetDatabase(_storageOptions.DatabaseId)
                .DefineContainer(_storageOptions.ContainerId, "/id")
                .WithDefaultTimeToLive(_cosmosDbTimeToLive)
                .WithIndexingPolicy().WithAutomaticIndexing(false).Attach()
                .CreateIfNotExistsAsync(_storageOptions.ContainerThroughput)
                .ConfigureAwait(false);
        }
    }

    public Task StopAsync(CancellationToken cancellationToken) => Task.CompletedTask;
}

then in startup:
// Add the Initializer as a HostedService (so it is called during the app service startup)
services.AddHostedService<CosmosDbStorageInitializerHostedService>();

Hi @vifani, let us know if that helps - we'll close this issue in a few days.
Thanks!

Thank you @daveta and @EricDahlvang. Yes, this helps, however is only a workaround. In my opinion, because the CosmosDbPartitionedStorage class creates the container and in the documentation (https://docs.microsoft.com/en-us/azure/bot-service/bot-builder-howto-v4-storage?view=azure-bot-service-4.0&tabs=csharp#using-cosmos-db) is stated "skip manually creating a container since CosmosDbPartitionedStorage will create the container for you". So, if this library creates the container for me, it should support all possible scaling options in my opinion. Obviously creating it with custom code is always an option.

Adding to backlog for further consideration.

This is cut from the same cloth as #4934, but slightly different.


Re-add use-this-casing rarity config option and some fixes

I'm not sure why this was removed, but we've had multiple people mention it in the discord server.
Also updates the default competition config to make sure we don't "break" servers updating from <IP_ADDRESS> (broadcast-range defaulted to 100, when it was global in <IP_ADDRESS>)

Can confirm this solve issues I had!

Sorted ☺


--watch does not work when running with docker-compose

I need to regenerate random data in each GET request. Since the data generation script can only be executed once during the json-server startup, I added a middleware function which opens the static data file, modifies partial data and write back to the same file. When I start up json-server on VM, the watch command seems to work and I get different data every time. However, the watch command does not seem to work when running in Docker container (specifically in docker-compose.yml) and I get the same data even the file is modified. Looks like the watch does not detect the file change and data is not reloaded. Can someone please assist?
Instead of this solution, is there a better solution? For example, is it possible to rerun data generation script after each GET request?
json-server:
image: clue/json-server
command: -w db.json -middlewares ./update.js -host <IP_ADDRESS>
volumes:

./db.json:/data/db.json
./update.js:/data/update.js

I'm not sure if this is related, but it also doesn't work for me in the other direction. It reads from the db.json file, but then never updates it again.

Is there a workaround/alternative to get this working?  Did the same thing (Using it inside a container) but doesn't pick up the file changes

Not that I know of. My "workaround" was to stop using this library.

unfortunately it looks like fs.watch  doesn't play well with docker volumes, it should be replaced with gaze in order to make it work

im also having this issue the db is sitting on a cephfs mount on a kubernetes pod
and updates to db.json is not reloading the server

use nodemon 👍
start.js :
// github.com/remy/nodemon
var nodemon = require('nodemon');
nodemon({
  script: 'index.js',
  ext: 'js json' // watching extensions
});

nodemon.on('start', function () {
  console.log('App has started');
}).on('quit', function () {
  console.log('App has quit');
  process.exit();
}).on('restart', function (files) {
  console.log('App restarted due to: ', files);
});

index.js :
const jsonServer = require('json-server')

const data = require('./api/db.js')
const routes = require('./api/routes.json')

const server = jsonServer.create()       // Express server
const router = jsonServer.router(data) // Express router
const middlewares = jsonServer.defaults()

// https://github.com/typicode/json-server/issues/690#issuecomment-348616467
// json-server options.bodyParser defalut is true
// server.use(jsonServer.bodyParser);

server.use(middlewares)
server.use(jsonServer.rewriter(routes))
server.use(router)

// Avoid CORS issue
// json-server options.noCors defalut is false
// server.use( (req, res, next) => {
//   res.header("Access-Control-Allow-Origin", "*");
//   // res.header("Access-Control-Allow-Headers", "Origin, X-Requested-With, Content-Type, Accept");
//   next();
// });

server.listen(9538, () => {
  console.log('JSON Server is running, see http://localhost:9538')
})

All you need to do is exec the command node start.js to listen to all files

The workaround is to specify directories, not files.
Here is the code that worked.
services:
  json-server:
    image: clue/json-server
    command: --watch /data/db.json --host <IP_ADDRESS>
    ports:
      - "8880:80"
    volumes:
      - ./data:/data

The workaround is to specify directories at volumes: , not files.
Here is the code that worked.
services:
  json-server:
    image: clue/json-server
    command: --watch /data/db.json --host <IP_ADDRESS>
    ports:
      - "8880:80"
    volumes:
      - ./data:/data

That still doesn't work for me. It works for my colleagues on MacOS but not on Windows...


systemd: remove compatibility patch for legacy Plasma

Motivation for this change
This will remove a backward-compatibility patch from systemd/logind that was required to support legacy Plasma. Specifically, the patch revealed a deprecated D-Bus interface that had been hidden, but not removed. Plasma has not used the deprecated interface for some time, so the patch is no longer required.
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[x] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

FYI, you could have changed the target branch in the GitHub ui if you press edit on the title, but new pr also works.


Bugfix/home page typo

 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.:x: MersenneChris sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.


KAFKA-7438: Replace Easymock & Powermock with Mockito in RocksDBMetricsRecorderGa…

Replace Easymock & Powermock with Mockito in RocksDBMetricsRecorderGaugesTest.
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

@cadonna @divijvaidya can you please review this? Thanks!

Weird that we are not tracking this test at https://issues.apache.org/jira/browse/KAFKA-14133 or https://issues.apache.org/jira/browse/KAFKA-14132.

Looking at the state of ticket KAFKA-14133 gives an impression that work on the replacement of Easymock with Mockito is almost over, but based on the latest state of the trunk branch it looks like there are still quite many test classes (not listed in the ticket) that use Easymock.

This is an interesting find. That list was compiled quite sometime ago, but given this information I will go and recheck again - maybe my search was faulty ¯\_(ツ)_/¯

@clolov I've compiled the list of test classes that use Easymock but are not listed in the ticket https://issues.apache.org/jira/browse/KAFKA-14133#:~:text=Hi Christo Lolov %2C ,kafka/pull/14142)


V3 API is currently down

Hi folks, Although, the status check page says its operational, the version 3 API is currently not responding. I have tried on multiple devices to confirm.
GET request on the API is giving a 502 Bad Gateway status.

Hi
I have the same problem and 502 Bad Gateway status for all get requests to V3 API

Looks like it was related to this MongoDB Atlas upgrade to the root cert:
https://docs.atlas.mongodb.com/reference/faq/security/#which-certificate-authority-signs-mongodb-atlas-cluster-tls-certificates-
https://letsencrypt.org/2019/04/15/transitioning-to-isrg-root.html
The upgrade caused some kind of hangup in the container, just had to be restarted.
Should be good now 👍

Yeah, Its good now. Thanks

V3 API is down once again. Please look into it.


Questions concerning back-calculated abundance apportionment

When back-calculating abundance from the apportioned kriged biomass, it is unclear whether the unaged abundance estimates should be redistributed over the age-bins similar to biomass.

From meeting with @rebeccathomas-NOAA and @ElizabethMPhillips on 2024/04/16:
For abundance derived from kriged biomass, in the reporting:

Need global aged and unaged abundance, that is not spatially explicit
Also need spatially explicit aged + unaged but proportioned to aged summed together

Some additional digging around in the matlab code:

These lines get the aged and unaged abundance calculated:
In get_kriged_biomass.m
https://github.com/uw-echospace/EchoPro_matlab/blob/main/general/processing/biomass/get_kriged_biomass.m#L342-L352
No apportioning afterwards to apply the aged proportion to the unaged samples.

The data matrix is then saved to a different variable name: kriged_Num_Len_Age_Matrix_AcoustM/kriged_Num_Len_Age_Matrix_AcoustF
https://github.com/uw-echospace/EchoPro_matlab/blob/main/general/processing/biomass/get_kriged_biomass.m#L392-L396

Then in generate_report.m these variables are used to written to the excel files
https://github.com/uw-echospace/EchoPro_matlab/blob/main/general/report/generate_reports.m#L106-L116

Check https://github.com/OSOceanAcoustics/echopop/issues/219#issuecomment-2053863594 for related discussion, for the google slide with flow diagram.

As an FYI, I was investigating some weird abundance results in the full age estimates (age1+ method), and I am pretty sure that the abundance reports are not correct for the age-2+ total number (when age-1's are being included)
If you look at lines
https://github.com/uw-echospace/EchoPro_matlab/blob/553b46b6a8a4c0dae6831727ef9a59a6a0f97b57/general/report/generate_reports.m#L169C1-L170C138
note that the age-2+ total sums all the unaged fish into the age-2+ total, without the age-'1s being apportioned out of the unaged fish.
However, this doesn't throw into question the main matrix, so it is probably a relatively small detail except that these are the exact numbers stock assessors might use in an age-1+ estimate, so I wanted to flag it in case it shows up in EchoPop.


Redis Scans are not performant

Scan operations in the Redis client are very bad.
I think it is because scans are implemented in a very strange way, essentially a record is kept in Redis that is a set of all the inserted keys and each scan operation has to do a scan of that one record for the proper keys, then retrieve all the records from that result. So, a scan for 1000 items will end up making 1001 calls to Redis, eww.
Redis supports a scan operation: http://redis.io/commands/scan, but the Jedis version the client uses does not. Fortunately, later versions of Jedis do support the scan operation, so it is probably a good idea to change the implementation in the Redis client and bump the Jedis version.

Unless someone who knows Redis better then me has any opinions, it doesn't look like the scan commands in Redis are actually going to help any. The main scan operation is really designed to assist in full iteration scans which is not what we want. The scan cursor is some internal value to Redis and changes frequently so there is no way to start a scan at a specific key using this Redis scan operation.
Might just have to live with the current implementation.


[Fix] Check is auth prompt is undefined

Should fix the error described below which occurs during kirisame init when no ORM (and subsequently no auth) is provided.
Previously code was checking for value of promptResponse.auth !== null, which is true for undefined (Thanks, JavaScript).
This should prevent the init command from crashing when selecting no ORM and no Auth 🤞
Config File
{
  "hasSrc": false,
  "packages": [
    "shadcn-ui"
  ],
  "preferredPackageManager": "pnpm",
  "t3": false,
  "alias": "@",
  "rootPath": "",
  "componentLib": "shadcn-ui"
}%

Describe the bug
kirisame init throwing the following error on a new project;
⠋ file:///Users/jason/Library/pnpm/global/5/.pnpm/kirimase@0.0.44/node_modules/kirimase/dist/commands/add/index.js:122
                        promptResponse.auth[0].toUpperCase() +
                                           ^

TypeError: Cannot read properties of undefined (reading '0')
    at addPackage (file:///Users/jason/Library/pnpm/global/5/.pnpm/kirimase@0.0.44/node_modules/kirimase/dist/commands/add/index.js:122:44)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)

Node.js v20.5.1

To Reproduce
Steps to reproduce the behavior:
➜  ~ cd Code/open-source
Found '/Users/jason/Code/open-source/.nvmrc' with version <20.5.1>
Now using node v20.5.1 (npm v9.8.0)
➜  open-source pnpx create-next-app@latest
Packages: +1
+
Progress: resolved 1, reused 1, downloaded 0, added 1, done
✔ What is your project named? … kirimase-bug-demo
✔ Would you like to use TypeScript? … No / Yes (Yes)
✔ Would you like to use ESLint? … No / Yes (Yes)
✔ Would you like to use Tailwind CSS? … No / Yes (Yes)
✔ Would you like to use `src/` directory? … No / Yes (No)
✔ Would you like to use App Router? (recommended) … No / Yes (Yes)
✔ Would you like to customize the default import alias (@/*)? … No / Yes (No)
Creating a new Next.js app in /Users/jason/Code/open-source/kirimase-bug-demo.

Using pnpm.

Initializing project with template: app-tw

Installing dependencies:
- react
- react-dom
- next

Installing devDependencies:
- typescript
- @types/node
- @types/react
- @types/react-dom
- autoprefixer
- postcss
- tailwindcss
- eslint
- eslint-config-next

Packages: +350
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Progress: resolved 358, reused 350, downloaded 0, added 350, done

dependencies:
+ next 14.0.4
+ react 18.2.0
+ react-dom 18.2.0

devDependencies:
+ @types/node 20.11.5
+ @types/react 18.2.48
+ @types/react-dom 18.2.18
+ autoprefixer 10.4.17
+ eslint 8.56.0
+ eslint-config-next 14.0.4
+ postcss 8.4.33
+ tailwindcss 3.4.1
+ typescript 5.3.3

Done in 3.8s
Initialized a git repository.

Success! Created kirimase-bug-demo at /Users/jason/Code/open-source/kirimase-bug-demo

➜  open-source cd kirimase-bug-demo
➜  kirimase-bug-demo git:(main) kirimase init
✔ Kirimase initialized!                                            3:34:21 PM
ℹ You can now add packages.                                        3:34:21 PM
◐ Checking project for existing packages...                         3:34:21 PM
✔ Successfully searched project and found no additional packages.  3:34:21 PM
? Select a component library to use: Shadcn UI (with next-themes)
? Select an ORM to use: None
? Are you sure you don't want to install an ORM? Note: you will not be able to
install any auth. yes
? Select any miscellaneous packages to add: TRPC, Stripe
⠋ file:///Users/jason/Library/pnpm/global/5/.pnpm/kirimase@0.0.44/node_modules/kirimase/dist/commands/add/index.js:122
                        promptResponse.auth[0].toUpperCase() +
                                           ^

TypeError: Cannot read properties of undefined (reading '0')
    at addPackage (file:///Users/jason/Library/pnpm/global/5/.pnpm/kirimase@0.0.44/node_modules/kirimase/dist/commands/add/index.js:122:44)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)

Node.js v20.5.1

Expected behavior
Expecting the typeError is not thrown
Screenshots
n/a
Desktop (please complete the following information):

MacOS v14.2.1
kirisame v0.0.44
Node v20.5.1

Additional context
n/a

Can confirm, encountered when using on a blank T3 project.

thanks!


Any way to change EGL opengl es version?

Now OPENGL ES version is hardcoded in EGL
https://github.com/AvaloniaUI/Avalonia/blob/fb9fa9403764691f2edeb2aba586eb7c61cb2d11/src/Avalonia.OpenGL/EglDisplay.cs#L75
Is there any way to change this without overriding all classes? I need OPENGL ES 3.2 profile.

#4675 implements this.


[SPARK-33976][SQL][DOCS][3.0] Add a SQL doc page for a TRANSFORM clause

What changes were proposed in this pull request?
Add doc about TRANSFORM and related function.

Why are the changes needed?
Does this PR introduce any user-facing change?
No
How was this patch tested?
Not need

Test build #138026 has started for PR 32375 at commit c145343.

Test build #138026 has finished for PR 32375 at commit c145343.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
    SERDE serde_class [ WITH SERDEPROPERTIES (k1=v1, k2=v2, ... ) ]

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/138026/

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/42545/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/42545/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/42545/

Merged to branch-3.0.


Implement the BIO_ADDR_copy() public API function

We already have BIO_ADDR_dup() but in some contexts that is not sufficent.
We implement BIO_ADDR_copy() and make BIO_ADDR_dup() use it.
Fixes openssl/project#220
This is built on top of #22157 and includes the commits from there. First commit in this PR is "Implement a public BIO_ADDR_copy() function"

I rebased this now that #22157 has gone in. It was a clean rebase so I assume the approval still holds.
Ping for second review.

It was a clean rebase so I assume the approval still holds.

Always!

24 hours has passed since 'approval: done' was set, but as this PR has been updated in that time the label 'approval: ready to merge' is not being automatically set.  Please review the updates and set the label manually.

Merged, thanks.


add: TimSort Algorithm

Describe the algorithm/code
Describe the algorithm/code
TimSort is a hybrid sorting algorithm that combines the strengths of merge sort and insertion sort.
It is designed to efficiently sort a wide range of real-world data types.  It maintains the relative order of equal elements in the sorted output.

Divide into Runs: TimSort starts by dividing the input array into small, already sorted subsequences called "runs."

Merge Runs: It then merges these runs together using a combination of merge sort and insertion sort. This merging process optimizes performance, especially for data with pre-existing order.

Please assign this issue to me as part of Hacktoberfest.
Thanks,
Tanushree
Do you want to work on this feature?

[X] I am willing to implement this feature.

@himanshu-03  kindly assign this issue to me, with hacktober accepted tag

Hi @aggarwal-tanushree ! 👋🏻
You can start working on this issue! Do let me know if you need any help. ✨


dm pull  fails.

Tried dotmesh while pulling, got this error
$dm pull hub moby_counter branch_a
Pulling admin/moby_counter from hub:Mahipal/moby_counter
Calculating...
Awaiting transfer...
error 9.50 KB / 9.50 KB [============================] 100.00% 11.39 MiB/s (1/2)
error: Response event != finished-{push,pull} or peer-up-to-date: <Event maximum-retry-attempts-exceeded: responseEvent: <Event get-failed-pull: err: exit status 1, filesystemId: b9575fe9-d4ab-4025-4ab9-edd0894d4195>>
Response event != finished-{push,pull} or peer-up-to-date: <Event maximum-retry-attempts-exceeded: responseEvent: <Event get-failed-pull: err: exit status 1, filesystemId: b9575fe9-d4ab-4025-4ab9-edd0894d4195>>
Response event != finished-{push,pull} or peer-up-to-date: <Event maximum-retry-attempts-exceeded: responseEvent: <Event get-failed-pull: err: exit status 1, filesystemId: b9575fe9-d4ab-4025-4ab9-edd0894d4195>>
List of from dm is
$ dm list
Current remote: local (use 'dm remote -v' to list and 'dm remote switch' to switch)
DOT           BRANCH  SERVER            CONTAINERS  SIZE       COMMITS  DIRTY

apples        master  a1fbbad082866d3c              19.00 kiB  0        19.00 kiB
moby_counter  master  a1fbbad082866d3c              9.00 kiB

This works fine for me - it might have been a transient problem, and we've just done a lot of work to improve the reliability of push/pull in general that might have fixed this. Please can you try again, and reopen the issue if it's still broken? Thanks!


nix-shell -p: unsafeDiscardStringContext missing error

Describe the bug
Whatever I'm trying to do right now (with nix-shell -p or nix-env -iA), I encounter the following error:
error: attribute 'unsafeDiscardStringContext' missing, at /nix/store/ncj4504zgiabs2dm7ya60k83cpc59vlz-nixpkgs-21.05pre282987.294d1925af6/nixpkgs/lib/strings.nix:429:13
(use '--show-trace' to show detailed location information)

To Reproduce
Honestly, no idea. But I now have it when using nix on my laptop (NixOS) as well as on my other laptop (Ubuntu with nix). So ... I don't think this is isolated.
I was experimenting with a number of things on my NixOS at first, so I thought it was because I was doing something wrong - but the same error without experimenting with only nix installed? Something weird is going on.
Expected behavior
No Error Message when using nix-shell -p or other stuff.
Additional Context
I tried nix-channel --update to no avail.
nix-env -iA nixpkgs.XXX throws the same error.
Installing packages with the path provided from NixDB works (?), if I wouldn't interrupt it after downloading dozens of GB in two hours (and no, it should not have done this for that package).
Metadata
$ nix-shell -p nix-info --run "nix-info -m" --show-trace
error: while evaluating anonymous function at /nix/store/ncj4504zgiabs2dm7ya60k83cpc59vlz-nixpkgs-21.05pre282987.294d1925af6/nixpkgs/pkgs/top-level/impure.nix:15:1, called from (string):1:6:
while evaluating anonymous function at /nix/store/ncj4504zgiabs2dm7ya60k83cpc59vlz-nixpkgs-21.05pre282987.294d1925af6/nixpkgs/pkgs/top-level/default.nix:19:1, called from /nix/store/ncj4504zgiabs2dm7ya60k83cpc59vlz-nixpkgs-21.05pre282987.294d1925af6/nixpkgs/pkgs/top-level/impure.nix:83:1:
while evaluating anonymous function at /nix/store/ncj4504zgiabs2dm7ya60k83cpc59vlz-nixpkgs-21.05pre282987.294d1925af6/nixpkgs/pkgs/stdenv/booter.nix:42:1, called from /nix/store/ncj4504zgiabs2dm7ya60k83cpc59vlz-nixpkgs-21.05pre282987.294d1925af6/nixpkgs/pkgs/top-level/default.nix:122:10:
while evaluating 'dfold' at /nix/store/ncj4504zgiabs2dm7ya60k83cpc59vlz-nixpkgs-21.05pre282987.294d1925af6/nixpkgs/pkgs/stdenv/booter.nix:60:27, called from /nix/store/ncj4504zgiabs2dm7ya60k83cpc59vlz-nixpkgs-21.05pre282987.294d1925af6/nixpkgs/pkgs/stdenv/booter.nix:136:4:
while evaluating 'go' at /nix/store/ncj4504zgiabs2dm7ya60k83cpc59vlz-nixpkgs-21.05pre282987.294d1925af6/nixpkgs/pkgs/stdenv/booter.nix:63:18, called from /nix/store/ncj4504zgiabs2dm7ya60k83cpc59vlz-nixpkgs-21.05pre282987.294d1925af6/nixpkgs/pkgs/stdenv/booter.nix:72:13:
while evaluating 'imap1' at /nix/store/ncj4504zgiabs2dm7ya60k83cpc59vlz-nixpkgs-21.05pre282987.294d1925af6/nixpkgs/lib/lists.nix:116:14, called from /nix/store/ncj4504zgiabs2dm7ya60k83cpc59vlz-nixpkgs-21.05pre282987.294d1925af6/nixpkgs/pkgs/stdenv/booter.nix:78:30:
while evaluating 'reverseList' at /nix/store/ncj4504zgiabs2dm7ya60k83cpc59vlz-nixpkgs-21.05pre282987.294d1925af6/nixpkgs/lib/lists.nix:393:17, called from /nix/store/ncj4504zgiabs2dm7ya60k83cpc59vlz-nixpkgs-21.05pre282987.294d1925af6/nixpkgs/pkgs/stdenv/booter.nix:85:6:
while evaluating anonymous function at /nix/store/ncj4504zgiabs2dm7ya60k83cpc59vlz-nixpkgs-21.05pre282987.294d1925af6/nixpkgs/pkgs/stdenv/default.nix:7:1, called from /nix/store/ncj4504zgiabs2dm7ya60k83cpc59vlz-nixpkgs-21.05pre282987.294d1925af6/nixpkgs/pkgs/top-level/default.nix:118:12:
while evaluating 'elaborate' at /nix/store/ncj4504zgiabs2dm7ya60k83cpc59vlz-nixpkgs-21.05pre282987.294d1925af6/nixpkgs/lib/systems/default.nix:18:15, called from /nix/store/ncj4504zgiabs2dm7ya60k83cpc59vlz-nixpkgs-21.05pre282987.294d1925af6/nixpkgs/pkgs/top-level/default.nix:52:17:
while evaluating 'foldl' at /nix/store/ncj4504zgiabs2dm7ya60k83cpc59vlz-nixpkgs-21.05pre282987.294d1925af6/nixpkgs/lib/lists.nix:80:20, called from /nix/store/ncj4504zgiabs2dm7ya60k83cpc59vlz-nixpkgs-21.05pre282987.294d1925af6/nixpkgs/lib/systems/default.nix:149:13:
while evaluating 'foldl'' at /nix/store/ncj4504zgiabs2dm7ya60k83cpc59vlz-nixpkgs-21.05pre282987.294d1925af6/nixpkgs/lib/lists.nix:82:16, called from /nix/store/ncj4504zgiabs2dm7ya60k83cpc59vlz-nixpkgs-21.05pre282987.294d1925af6/nixpkgs/lib/lists.nix:86:8:
while evaluating the attribute 'parsed.abi.assertions' at /nix/store/ncj4504zgiabs2dm7ya60k83cpc59vlz-nixpkgs-21.05pre282987.294d1925af6/nixpkgs/lib/systems/default.nix:23:7:
while evaluating 'mkSystemFromString' at /nix/store/ncj4504zgiabs2dm7ya60k83cpc59vlz-nixpkgs-21.05pre282987.294d1925af6/nixpkgs/lib/systems/parse.nix:453:24, called from /nix/store/ncj4504zgiabs2dm7ya60k83cpc59vlz-nixpkgs-21.05pre282987.294d1925af6/nixpkgs/lib/systems/default.nix:23:16:
while evaluating 'mkSkeletonFromList' at /nix/store/ncj4504zgiabs2dm7ya60k83cpc59vlz-nixpkgs-21.05pre282987.294d1925af6/nixpkgs/lib/systems/parse.nix:374:24, called from /nix/store/ncj4504zgiabs2dm7ya60k83cpc59vlz-nixpkgs-21.05pre282987.294d1925af6/nixpkgs/lib/systems/parse.nix:453:49:
while evaluating 'splitString' at /nix/store/ncj4504zgiabs2dm7ya60k83cpc59vlz-nixpkgs-21.05pre282987.294d1925af6/nixpkgs/lib/strings.nix:427:23, called from /nix/store/ncj4504zgiabs2dm7ya60k83cpc59vlz-nixpkgs-21.05pre282987.294d1925af6/nixpkgs/lib/systems/parse.nix:453:69:
attribute 'unsafeDiscardStringContext' missing, at /nix/store/ncj4504zgiabs2dm7ya60k83cpc59vlz-nixpkgs-21.05pre282987.294d1925af6/nixpkgs/lib/strings.nix:429:13

Well. I'll happily provide any other information.

This looks like it might be (related to) #118481 ?

Usually this happens when the running nix-daemon and the nix-shell you are running go out of sync in a significant way. This is often triggered by a nix-env -u which hits the mentioned issue. A rollback of your environment should resolve this issue.
What does readlink $(type -p nix-shell) print? Have you tried restarting your nix-daemon?

Thanks for the help! The in #118481 mentioned nix-env --rollback and restarting nix-daemon somehow did the trick, and everything is working properly now.


player not centered, fullscreen top player completely broken

this happened this morning
and scrolling up to use fullscreen top player does the animation but only shows a black screen

waterfox 37.0.1
Developer Version - Build #501

duplicate of
#1695
#1696
#1697


Fix liveview interpreter JS

Previous PR #1063 changed how the javascript interpreter code is handled. Liveview didn't have the necessary changes to accommodate the new JS organization. (Hopefully this is the last thing that #1063 broke 👀)

I've been thinking similarly. It's been pretty rough having all of these issues after the fact. It might even be worth adding a github action to check all JS code for issues such as the one earlier of bool_attrs. Something else I'm not super fond of is the duplicate JS code between sledgehammer and wasm-bindgen.


After voting, vote success modal did not appear

Description
After voting, vote success modal did not appear
Steps To Reproduce

Log in...
Enter a proposal page
vote

Current Behavior

After voting, vote success modal did not appear

Expected Behavior

vote success modal should appear

Environment

Platform: PC
Node: v18.18.0
Browser: Chrome 126.0.6478.56

fixed confirm


get_stream_url: youtube vs googleusercontent links?

Hi,
Sometimes, when i use get_stream_url(song_id), it returns a videoplayback instead of a stream..
I get           http://r4---sn-apn7en7r.c.youtube.com/videoplayback?id=.... like URL
instead of   http://t.doc-0-0-sj.sj.googleusercontent.com/stream?id=.....
Is it something that i need to deal with in my library?
Do you know if there is a policy to get a stream instead of videoplayback?
Thanks in advance,
Franck

hi can you send me please to<EMAIL_ADDRESS>


🐛 Bug: Unnecessary || exit 0 in post-run files cleanup

Bug Report Checklist

[X] I have tried restarting my IDE and the issue persists.
[X] I have pulled the latest main branch of the repository.
[X] I have searched for related issues and found none that matched my issue.

Expected
It makes sense to pnpm build in post-run cleanup. I'd added it in #1152. Despite what I thought while writing that PR, I now think it makes sense to let the user know if that command fails.
Actual
The PR added an exit 0:
https://github.com/JoshuaKGoldberg/create-typescript-app/blob/01704e906969fc3cba8be9618e6e097ec3760cd4/src/shared/createCleanUpFilesCommands.ts#L14
On second thought, let's remove that.
Additional Info
No response

Fixed by #1161.


[PPP] Hwee!

Codecov Report

Merging #381 (6a6748b) into master (4c431df) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head 6a6748b differs from pull request most recent head fd93127. Consider uploading reports for the commit fd93127 to get more accurate results

@@            Coverage Diff            @@
##             master     #381   +/-   ##
=========================================
  Coverage     68.40%   68.40%
  Complexity     1131     1131
=========================================
  Files           135      135
  Lines          3865     3865
  Branches        521      521
=========================================
  Hits           2644     2644
  Misses          996      996
  Partials        225      225

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 4c431df...fd93127. Read the comment docs.


[doc] Remove missed CheckpointedRestoring

What is the purpose of the change
Pull request fixes one missed CheckpointedRestoring interface which was removed in Flink 1.4
Brief change log

*Removed unused CheckpointedRestoring interface in streams docs. This interface was removed in Flink 1.4

Verifying this change
PR is just a documentation change that can be verified in the diff very easily :)
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): no
The public API, i.e., is any changed class annotated with @Public(Evolving): no
The serializers: no
The runtime per-record code paths (performance sensitive): no
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: no
The S3 file system connector: no

Documentation

Does this pull request introduce a new feature? no
If yes, how is the feature documented? docs

merging.


Failed to complete #converge action: [Bad HTTP response returned from server. Body(if present): (400).] on default-windows-2012R2

Opened test-kitchen/test-kitchen#1304

test-kitchen pinned to 1.16 as you suggested :)


Be smarter about working with option

We could use some of Core's facilities to make working with optional types more sane.
Some examples:
Option.map timer.comparison ~f:Archived_run.splits |> Option.bind ~f:(fun comp -> List.nth comp split_num) |> Option.bind ~f:Split.time

Option.merge (List.nth seg_durations i |> Option.join) (List.nth old_durations i |> Option.join) ~f:Time_ns.Span.min )

open Core
open Option.Let_syntax

let some_ten =
    let%map unpacked_option = Some 5 in
    unpacked_option * 2

Or >>| and >>=

#10


Add a way to escape / in regexes

While testing out expression filtering, I noticed that there's currently no way to escape a / in a regex. Ideally we'd support this via a \/ sequence.
cc @Guiguiprim, what's the best way to handle this in your opinion?

I guess in a way similar to how unicode string are parsed.

That is exactly what I was thinking of. I'm going to have a try at it.
Speaking of escaping I realize that the unicode parsing (that I took from nom examples) also support escaping whitespaces, aka:
"test(d\
d)

is equivalent to
test(dd)

I'm not sure if we should keep it or how to document it.
More generally, I wonder if I shouldn't have simplified it a bit more...

Ah yeah I was looking at that. Not really worth keeping I think. We can always add it later if there's demand.

Done in #146, thanks!


Docs updates

Fix links and TOCs in docs
Add new docs page
Add new Hyperledger logo

Codecov Report

Merging #206 (0e7655a) into main (dcff611) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head 0e7655a differs from pull request most recent head c9141da. Consider uploading reports for the commit c9141da to get more accurate results

@@            Coverage Diff            @@
##              main      #206   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files          202       202
  Lines        10994     10994
=========================================
  Hits         10994     10994

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 dcff611...c9141da. Read the comment docs.


Invalid regular expression in latest version of vscode

SyntaxError: Invalid regular expression: /(((?<=((?: +)?)(`{3}\w+\s)))((?:.|\s)*)(?)((?=(\s)((?: +)?)(`{3}))))/g: Invalid group

Version: 1.82.0-insider
Commit: 501baeb5e07b5a404f862d922bacb93771d91331
Date: 2023-08-18T05:34:41.801Z
Electron: 25.5.0
ElectronBuildId: 23084831
Chromium: 114.0.5735.289
Node.js: 18.15.0
V8: <IP_ADDRESS>-electron.0
OS: Darwin arm64 21.6.0

Can you give me your original regex string please?

Can you give me your original regex string please?

this is the regex causing the issue ((?<=( +)?`{3}\\w+\\s)(.|\\s)*?(?=\\s( +)?`{3})) for some reason its giving the error
https://regex101.com/r/SnLLlF/1

You can fix it with this regex ((?<=( +)?{3}\w+\s)(.|\s)*(?=\s( +)?{3})).
But I will correct that soon.

sure np, thanks for ur help & support ❤

The commit 4b10ded933e3369f69ca90d10cc7b4f2518af016 solved the issue.


'Rec' and 'Latest' radios don't update across pages

Have two pages of modpack builds, with the recommended or latest build radio enabled on the second page. Update the radio on any of the first page modpacks, the success message is displayed and the db is in fact updated. However, if you move to the second page, the radio option will not have been cleared yet.
This is a visual glitch only

If you have two tabs showing the same page and only one updateing the radios that is alright, the page  only gets updated upon refresh.


ollama tweaks

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

Should we also take care of this here: https://github.com/huggingface/hub-docs/pull/1447#discussion_r1802880031

ok, merging!


End beautified file with a new line

Do not add space at end of file.
The new line at the end of file will be a LF or CRLF depending on options selected to run the script.

Taking a quick look now... looks great!  I will test and merge tonight.
Actually this is also a great option to add to #11 user configurable whitespace rules; some folks might want the last command to be followed directly by end of file while others might always want a newline.  I will add this an a option to that ticket.

I made a few tweaks to your code.  In Copy-SourceContentToDestinationStream, rather than checking each token in the loop to see if it was the last one or not it now does a single check after the loop is over.  If the last token wasn't a newline then it adds one.
Also, I ended up moving the code for Write-TokenContent_NewLine out to a separate function Write-NewLine which doesn't have any parameters.  Then I made Write-TokenContent_NewLine's Token parameter mandatory again.  Even though the token isn't used in Write-TokenContent_NewLine, I like to keep all the Write-TokenContent_* function signatures the same in case the way they are called ever changes.
Will commit and push shortly.

Cool!
Your version looks much better, I didn't like introducing an if/elseif/else
just for the sake of adding a single \n, your solution is much cleaner.
On Wed, 28 Jun 2017 at 03:14 Dan Ward<EMAIL_ADDRESS>wrote:

Merged #17 https://github.com/DTW-DanWard/PowerShell-Beautifier/pull/17.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/DTW-DanWard/PowerShell-Beautifier/pull/17#event-1141559035,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AD05xxXfT0PgtSDZzrJHXL93ZQxsYmN5ks5sIajpgaJpZM4OGNR5
.


New features: "skip" attribute, support for tuple structs, "transparent" feature on single-length structs.

[ ] Additional tests and docs

Codecov Report

Merging #90 (b18b6fb) into main (dfa59fa) will decrease coverage by 3.77%.
The diff coverage is 70.38%.

@@            Coverage Diff             @@
##             main      #90      +/-   ##
==========================================
- Coverage   98.25%   94.48%   -3.78%
==========================================
  Files           8        8
  Lines        1490     1577      +87
==========================================
+ Hits         1464     1490      +26
- Misses         26       87      +61

Impacted Files
Coverage Δ

arrow2_convert_derive/src/input.rs
80.00% <57.89%> (-6.51%)
:arrow_down:

arrow2_convert_derive/src/derive_struct.rs
89.10% <73.21%> (-10.65%)
:arrow_down:

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

Looks great, thanks @jondo2010!


Fix/transfer ownership of account invitations

Transferring ownership of invitations along with account ownership.
Testing instructions
Invite user to account, change ownership, check if invited_by_id changed to new owner id
Tracking
https://vizzuality.atlassian.net/browse/LET-849

I also tested it locally with Barbara branch, which adds transfer ownership to the frontend, just had to change GET to POST (as I don't like testing with api docs lol).


2D Transfers in Scala Backend

Wen seems to have found an issue with 2D tile loading:

I've been trying to figure out why MatMult_outer doesn't work.  When I replaced the Accel block with:
Accel {
  val tileA = SRAM[T](bm, bp)
  tileA := a(0::bm, 0::bp)
  Pipe(bm by 1, bp by 1){ (ii,jj) =>
    println("A(" + ii + ", " + jj + "): " + tileA(ii,jj))
  }
}

and ran it with the size of matrix A being 4x192, I found that only the first row of tileA has value 1; the last three rows show up as zeros, even though they should also be 1.
I don't know if there're issues with DRAM-to-SRAM transfer.  Is there a way to print numbers directly from DRAM?

Could this be related to the bug we found with the enable signal earlier? The 2D transfer in scala gen worked at one point, and I can't think of any other reason it would be broken now.


Fixed mistakes in README

Description
Please include a summary of the change and which issue is fixed. List any dependencies that are required for this change.
Fixes #388
Replace issue_no with the issue number which is fixed in this PR
Have you read the Contributing Guidelines on Pull Requests?

[x] Yes
[ ] No

Type of change
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update
[x] Documentation Update

Checklist:

[x] My code follows the style guidelines(Clean Code) of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I've edited the SCRIPTS.md and link to my code.
[x] I have created a helpful and easy to understand README.md
[x] My documentation follows Template for README.md
[x] My changes generate no new warnings
[x] I have added tests/screenshots(if any) that prove my fix is effective or that my feature works.

@avinashkranjan Please review this.


#32 Not enough generic parameters were set as 'an…

…y' previously.

Codecov Report

Merging #44 (817282d) into main (ddeb938) will not change coverage.
The diff coverage is n/a.

@@             Coverage Diff             @@
##              main       #44     +/-   ##
===========================================
  Coverage   100.00%   100.00%
===========================================
  Files           12         7      -5
  Lines         3171       719   -2452
  Branches       234       125    -109
===========================================
- Hits          3171       719   -2452

Flag
Coverage Δ

endpoint
100.00% <ø> (ø)

endpoint-prefix
100.00% <ø> (ø)

endpoint-spec
?

server
100.00% <ø> (ø)

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

Impacted Files
Coverage Δ

endpoint-spec/src/common.ts

endpoint-spec/src/stage3.ts

endpoint-spec/src/stage2.ts

endpoint-spec/src/stage1.ts

endpoint-spec/src/stage0.ts


JBPM-4900 Added TaskQueryBuilderTest

+1
shall this go to 6.4.x too? if so PR for 6.4.x please :)

great :) .... here is the PR for 6.4.x https://github.com/droolsjbpm/jbpm/pull/402


Improve onboarding when using --download-selenium

Today @timwingfield and I had a rocky onboarding experience with an existing project using Testium. It worked fine when we ran the test suite with phantomjs, but when running with --browser chrome we got an ENOENT error  stat'ing node_modules/testium/node_modules/bin/chromedriver.
We didn't know how to proceed so we tried these and they didn't help:
$ npm i -g testium
$ testium --download-selenium #<-- the README suggested this might work

Then we got the same error. It was only once @endangeredmassa mentioned we should run locally (e.g. ./node_modules/.bin/testium --download-selenium) that we got to our next error (which was that Java wasn't installed).
Thoughts on improving the onboarding? I feel like it would have been helpful if the README told us to run with the local bin, or if the global actually installed locally. Additionally when the bin is missing (but expected), a clearer error message would have been helpful.

README issues addressed here: https://github.com/groupon-testium/testium/pull/134
What version of testium was this? ENOENT issues should have been fixed in 2.5.0 here: https://github.com/groupon-testium/testium/pull/128

Global support discussed here: https://github.com/groupon-testium/testium/issues/136

When I try this locally without java and without a local chromedriver, I get:
See the full log at: /home/smassa/source/testium/test/integration_log/selenium.log
Error: Unable to find java
  at errnoException (child_process.js:1011:11)
  at Process.ChildProcess._handle.onexit (child_process.js:802:34)

@searls what exactly did you see when trying this locally? How can I reproduce it?

I am running 2.5.0
When node_modules/testium/bin/chromedriver does not exist (e.g. I haven't run --download-selenium yet), I get:
> testium --browser chrome

Preparing chrome...

/Users/justin/code/testdouble/my-project/node_modules/testium/lib/cli/console.js:112
      throw error;
            ^
Error: ENOENT, stat '/Users/justin/code/testdouble/my-project/node_modules/testium/bin/chromedriver'

npm ERR! Darwin 14.1.0
npm ERR! argv "node" "/usr/local/bin/npm" "run" "test-console"
npm ERR! node v0.10.32
npm ERR! npm  v2.1.5
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>test-console: `testium --browser chrome`
npm ERR! Exit status 8
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>test-console script.
npm ERR! This is most likely a problem with the my-project package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     testium --browser chrome
npm ERR! You can get their info via:
npm ERR!     npm owner ls my-project
npm ERR! There is likely additional logging output above.

npm ERR! Please include the following file with any support request:
npm ERR!     /Users/justin/code/testdouble/my-project/npm-debug.log

You should be able to reproduce by just running rm node_modules/testium/bin/chromedriver and then running testium --browser chrome

When I try removing chromedriver and java, I don't see that error.
Are you running that with a local testium? The example makes it appear to be global.

That was executed with npm run test-console which is configured in package.json as:
    "test-console": "./node_modules/.bin/testium --browser chrome"

Ah, I managed to reproduce this. Will fix.

Fix for running without a local chromdriver: https://github.com/groupon-testium/testium/pull/137

Closing this issue in favor of linked issues/PRs.


drivers/flash/flash_stm32l5_u5.c : unable to use full 2MB flash

Describe the bug
With the current driver, it is not possible to access more than 1MB of the 2MB of stm32u58x MCU
To Reproduce
Change storage_partition in dts file (b_u585 board for example) to use offset after 1MB
Build settings sample
west build -b b_u585i_iot02a samples/subsys/settings -p
Expected behavior
It is possible to write at this offset of flash
Additional context
correction seems to be to add a second bank for stm32u58x boards in drivers/flash/flash_stm32l5_u5.c:l384

When extending the storage partition to the bank2 (> 0x810 0000)
	storage_partition: partition@f0000 {
		label = "storage";
		reg = <0x000f0000 DT_SIZE_K(256)>;
	};

I can read/write flash there for example with samples/drivers/flash_shell :
uart:~$ flash_sample write_pattern 0x120000 0x100
Reading back written bytes:
00 01 02 03 | 04 05 06 07 | 08 09 0a 0b | 0c 0d 0e 0f
10 11 12 13 | 14 15 16 17 | 18 19 1a 1b | 1c 1d 1e 1f
20 21 22 23 | 24 25 26 27 | 28 29 2a 2b | 2c 2d 2e 2f
30 31 32 33 | 34 35 36 37 | 38 39 3a 3b | 3c 3d 3e 3f
40 41 42 43 | 44 45 46 47 | 48 49 4a 4b | 4c 4d 4e 4f
50 51 52 53 | 54 55 56 57 | 58 59 5a 5b | 5c 5d 5e 5f
60 61 62 63 | 64 65 66 67 | 68 69 6a 6b | 6c 6d 6e 6f
70 71 72 73 | 74 75 76 77 | 78 79 7a 7b | 7c 7d 7e 7f
80 81 82 83 | 84 85 86 87 | 88 89 8a 8b | 8c 8d 8e 8f
90 91 92 93 | 94 95 96 97 | 98 99 9a 9b | 9c 9d 9e 9f
a0 a1 a2 a3 | a4 a5 a6 a7 | a8 a9 aa ab | ac ad ae af
b0 b1 b2 b3 | b4 b5 b6 b7 | b8 b9 ba bb | bc bd be bf
c0 c1 c2 c3 | c4 c5 c6 c7 | c8 c9 ca cb | cc cd ce cf
d0 d1 d2 d3 | d4 d5 d6 d7 | d8 d9 da db | dc dd de df
e0 e1 e2 e3 | e4 e5 e6 e7 | e8 e9 ea eb | ec ed ee ef
f0 f1 f2 f3 | f4 f5 f6 f7 | f8 f9 fa fb | fc fd fe ff

Can you see the DBANK option bit of your device with the STM32CubeProgrammer

Yes DBANK is activated as we use TF-M:

Could we deactivate this DANK bit in TF-M to access full 2MB of flash ?

@tpennors can you provide the abstract of device tree you're using ?

Here is an extract of our dts :
/*
 * Copyright (c) 2021 STMicroelectronics
 *
 * SPDX-License-Identifier: Apache-2.0
 */

/dts-v1/;
#include "lpp4_dl_proto0-common.dtsi"

/ {
	model = "LPP4 DL Proto-0 board based on stm32u585 MCU";
	compatible = "sofrel,lpp4_dl_proto0";

	sram3: memory@20040000 {
		reg = <0x20040000 DT_SIZE_K(524)>;
	};

	chosen {
		zephyr,console = &uart4;
		zephyr,shell-uart = &uart4;
		zephyr,sram = &sram3;
		zephyr,flash = &flash0;
		zephyr,bt-uart = &usart1;
		zephyr,code-partition = &slot1_partition;
		zephyr,entropy = &rng_psa;
	};

	rng_psa: entropy_psa_crypto {
		compatible = "zephyr,psa-crypto-rng";
		status = "okay";
		label = "psa_entropy";
	};
};

&flash0 {

	partitions {
		compatible = "fixed-partitions";
		#address-cells = <1>;
		#size-cells = <1>;

		/*
		 * Following flash partition is compatible with requirements
		 * given in TFM configuration given for current board.
		 * It might require adjustment depending on evolutions on TFM.
		 */

		boot_partition: partition@0 {
			label = "mcuboot";
			reg = <0x00000000 DT_SIZE_K(208)>;
			read-only;
		};
		/* Secure image primary slot */
		slot0_partition: partition@34000 {
			label = "image-0";
			reg = <0x00034000 DT_SIZE_K(256)>;
		};
		/* Non-secure image primary slot */
		slot1_partition: partition@74000 {
			label = "image-1-nonsecure";
			reg = <0x00074000 DT_SIZE_K(1312)>;
		};
		/* NVS partition storage, for:
			- ID Card
			- littlefs sample
			- anjay_coap_client sample
		*/
		storage_partition: partition@1bc000 {
			label = "storage";
			reg = <0x001bc000 DT_SIZE_K(16)>;
		};
		/* NVS partition for temporary subsys config persistent backend */
		storage_config_nvs: partition@1c0000 {
			label = "storage_config_nvs";
			reg = <0x001c0000 DT_SIZE_K(16)>;
		};
	};
};

With that partition, the page layout detects  256 pages of = 8192   : 2 banks are contiguous

In your example, do you have TF-M activated ?

In your example, do you have TF-M activated ?

No, no TFM ; that's not specified, is it.

Yes we use TF-M in our application.
We saw that regression.sh script put DBANK option byte. Is it absolutely necessary for TF-M applications ? If it is necessary, is it possible to use 2MB of flash when DBANK is activated ?

Unfortunately, I am not able yet to build with TFM the b_u585i_iot02a_ns

I'm able to reproduce the issue. Investigating

We saw that regression.sh script put DBANK option byte. Is it absolutely necessary for TF-M applications ? If it is necessary, is it possible to use 2MB of flash when DBANK is activated ?

I guess this is optional, but has to be configured in TF-M. Please check with ST TF-M support for this.

@tpennors There's an issue in TF-M which prevent to use some NS zones of the flash.
This could be fixed with a fix in TF-M: https://github.com/zephyrproject-rtos/trusted-firmware-m/pull/84
Note that this fix is on TF-M v1.7.0, but should also apply to TF-M v1.6.0.
Then, I strongly encourage you, if not done already to update following TF-M file in order to configure TF-M partition to your use case: https://github.com/zephyrproject-rtos/trusted-firmware-m/blob/master/platform/ext/target/stm/b_u585i_iot02a/include/flash_layout.h
(Note: Don't take into account the comment "Flash layout for b_u585i_iot02a  with BL2 (multiple image boot):", it's not up to date with following xxxx_SIZE definitions)
Basically, flash partition described in Zephyr only applies to non secure.
If you want to make changes to secure, this should be done in this file in TF-M.
But you should ensure both are aligned.
@microbuilder I'm not sure how to consider https://github.com/zephyrproject-rtos/trusted-firmware-m/pull/84 (should it be merged or not in our fork). It's the real fix for an actual bug, but I'm unsure how this will be fixed in upstream.

Hello, I am working on the same team and project as @tpennors
While investigating the issue, these are the test results I obtained using various configurations with the shell command $ flash write_pattern 0x100000 0x10 using the flash_shell sample app on a stm32u585 MCU  :
Tests setup

TF-M enabled, DBANK enabled
TF-M enabled + patch erwango bd3b17b47e , DBANK enabled
TF-M disabled, DBANK enabled
TF-M disabled, DBANK disabled

Results

CPU exception: Division by zero @ drivers/flash/flash_page_layout.c:27
CPU exception: Division by zero @ drivers/flash/flash_page_layout.c:27
CPU exception: Division by zero @ drivers/flash/flash_page_layout.c:27
CPU exception: Division by zero @ drivers/flash/flash_page_layout.c:27

TF-M does not seem to be the issue here, but the DBANK option byte changes the behaviour of the function drivers/flash/flash_stm32l5_u5.c : flash_stm32_page_layout(), still leading to the same CPU exception.
Issues details
The implementation of the function drivers/flash/flash_stm32l5_u5.c : flash_stm32_page_layout() could possibly be incriminated. Especially this part:

Test scenarios 1->3
DBANK is enabled so we execute the first annotated block in line 383, FLASH_PAGE_NB=128 and FLASH_PAGE_SIZE=8192 (checked with debugger), resulting in a single 1MiB bank configuration. Because the FLASH_PAGE_NB is defined as the number of pages for a single bank in modules/hal/stm32/stm32cube/stm32u5xx/soc/stm32u585xx.h:
.
Test scenarios 4
No flash layout is configured at all for the stm32u5 as a consequence of the macro condition directive in line 389.
Possible fix
All the aforementioned test scenarios succeded with this fix.
When DBANK is enabled, add a configuration for the second bank that is contiguous with the first.
When DBANK is disabled, add a configuration for the stm32u5 SOC.

Here is the patch if you want to give it a try: flash_stm32l5_u5.c.patch

@abelradenac In order to ease the tracking of issues and fixes, please open a new bug description for each new point.

@abelradenac Yes, I reproduced the issue initially described by @tpennors. The TF-M changed proposed fixed the issue on my side.
Note that I'm testing on top of main branch (which includes https://github.com/zephyrproject-rtos/zephyr/pull/51525 which fixes the issue you mention), and from what I can see you're using an older version.

We are currently rebased on Zephyr 3.2.0, and when @tpennors initated the thread when we were on the 3.1.0, we failed to mention that.
The initial issue of this thread is then resolved with the PR https://github.com/zephyrproject-rtos/zephyr/pull/51525 you mentioned, this thread can be closed and we'll create a new thread for the case of DBANK deactivated.
Thanks for your time

Note that this fix is on TF-M v1.7.0, but should also apply to TF-M v1.6.0.

@erwango I don't see this fix upstreal in TF-M 1.7.0, which is also what we're currently using the the last commit of TF-M in Zephyr? https://git.trustedfirmware.org/TF-M/trusted-firmware-m.git/tree/platform/ext/target/stm/common/stm32u5xx/secure/target_cfg.c

@microbuilder I'm not sure how to consider zephyrproject-rtos/trusted-firmware-m#84 (should it be merged or not in our fork). It's the real fix for an actual bug, but I'm unsure how this will be fixed in upstream.

As mentionned in the PR, this should first be fixed upstream in TF-M, then we can cherry pick that into the Zephyr fork, unless I'm missing something and this is entirely Zephyr-specific and there is a compelling reason not to fix it upstream first?

As mentionned in the PR, this should first be fixed upstream in TF-M, then we can cherry pick that into the Zephyr fork, unless I'm missing something and this is entirely Zephyr-specific and there is a compelling reason not to fix it upstream first?

Sure, I've pushed this PR to allow testing. I actually expects @jamike to provide a more complete fix. I'll keep you posted.

@microbuilder I've replaced my commit with @jamike's which was merged in between (https://review.trustedfirmware.org/c/TF-M/trusted-firmware-m/+/19226/2/platform/ext/target/stm/common/stm32u5xx/secure/target_cfg.c)

@microbuilder I've replaced my commit with @jamike's which was merged in between (https://review.trustedfirmware.org/c/TF-M/trusted-firmware-m/+/19226/2/platform/ext/target/stm/common/stm32u5xx/secure/target_cfg.c)

Approved the TF-M PR, but we'll need a Zephyr PR to update the west manifest and kick off a CI run. Change looks good, though, thanks.


Status of v5

Hey guys what is the status for v5.
I see there is some work to add tags / labels to metrics for services like prometheus / stackdriver that have this concept. on the developmentv5 branch there seems to be a metricname class which implements this.
https://github.com/dropwizard/metrics/commit/7d8479c86057ac62196f888648ac6ac9fe9d03a0
It looks like its implemented at the registry level but doesnt seem to be added to the metric classes themselves, e.g Meter, counter etc
Is this work still going on?

Is there an update on this? I am very interested in the dimensional features of this release but it's most recent maven push was 5.0.0-rc2 almost a year ago. It looks like work has been done of the development branch too.

Also interested.

@arteam Any updates on the status of v5 or blockers to getting it shipped? In addition to having Timers that track sums, I'm starting to look at splitting hostnames out into proper JMX ObjectName keys or tags for InfluxDB/Prometheus/etc. and the new MetricName feature in v5 would make this really straight-forward.

It was a while ago since 5.0.0-rc2 was released.
@arteam Could you at least create a new 5.0.0-rc3 release of the current 5.0-development branch. It would be nice with the fixes for avoid boxing of longs, as this avoids some garbage created.
Even better would be a plan for the 5.0.0 release... :-)

I would look at micrometer, its similar to dropwizard but has dimensional metrics and much more.  This project seems to show no movement of late.
https://github.com/micrometer-metrics/micrometer

I took a look at Micrometer and it does have some nice features. However, it also has a dependency on DropWizard and several important implementations, such as Graphite and JMX, delegate to DropWizard.
So, it's not a replacement for our use cases :)

in v5, would you guys consider to add a reset() method for every metrics to set the count and other internal state?
for my use case, I want my metrics reset daily. And I have to remove those metrics from registry manually and re-create the instance. Not a big deal, but it is nice if there is a way to reset.
I took a glance at the code. All supporting classes such as MovingAverages interface need to have that reset method.


Minecraft 1.20.1

I tried the 1.20.1 version in Minecraft, but the 1.20.1 version does not appear in the gui. When I tried 1.20, it could not find it. So I installed the 1.20.2 version, tried it and changed it to 1.20.1, but it could not even find my offline map.
It says there are more structures or your data is corrupt and no seed was given.

What version of the mod did you use? I just downloaded the mod version for 1.20.1, created a new world and successfully cracked seed.

What structures did you find or how many different bits did you get? When I was cracking the seed now the mod wrote that there was not enough data, but once I found the witch's house, the seed circle narrowed dramatically and I found what I was looking for.


modify the color of the detection box

What should I do if I want to modify the color of the detection box in the demo.py output image?

The author has disabled the color parameter in the draw_boxes function from their Vizer library if you check here
I'd say create your own function to draw the bounding boxes which should be pretty straightforward or use the _draw_single_box function  from the vizer library to iterate through the boxes and use custom colors

The author has disabled the color parameter in the draw_boxes function from their Vizer library if you check here
I'd say create your own function to draw the bounding boxes which should be pretty straightforward or use the _draw_single_box function from the vizer library to iterate through the boxes and use custom colors

Thank you very much！


implement UiWidgetTree

We need to properly implement UiWidgetTree. This is quite important as ListTable and Tree are required for a reasonable application and an appropriate showcase.
For GWT:
We still have to consider if we want to support Cell instead of Widget and move to CellTree. We found a design how we could render a widget such as UiWidgetLabel as Cell (TextCell) instead of Label. However, this is a tough decision as Cell does not really fit into the concept of the ui-widget framework. How could the widget hold the reference to the Cell and update its style and other things? How would a composite widget work (as CompositeCell)? This Cell stuff seems to open a parallel universe and we currently fear a maintenance hell when going towards this...

So far working including lazy loading on expand, etc.
Various features still missing. Most important is selection mode with multi-selection (via checkboxes). Also expand/collapse should be added to the API.


Improve document about authenticating with GitHub Apps

Hi,
I'm currently trying to have a working example how to properly authenticate as a GitHub app in my organization. In the app settings I can find the application ID. Using:
$ curl -i -H "Authorization: Bearer $JWT_TOKEN" -H "Accept: application/vnd.github.machine-man-preview+json" https://api.github.com/app/installations

I can also get the installation ID.
Here is my code:
package main

import (
    "context"
    "net/http"
                                                                                "github.com/bradleyfalzon/ghinstallation"
    "github.com/google/go-github/github"
    "github.com/sirupsen/logrus"
)

func main() {
    ctx := context.Background()

    // Set here you application ID and installation id
    appId := 111111
    installationId := 22222

    // Wrap the shared transport for use with defined application and installation IDs
    itr, err := ghinstallation.NewKeyFromFile(http.DefaultTransport, appId, installationId, "/tmp/key.pem")
    if err != nil {
         logrus.Error(err)
    }
    access_token, _ := itr.Token()
    logrus.Infof("Installation access token: %s", access_token)

    // Use installation transport with client
    // NewClient returns a new GitHub API client.
    // If a nil httpClient is provided, http.DefaultClient will be used. To use API methods which require authentication,
    // provide an http.Client that will perform the authentication for you.
    client := github.NewClient(&http.Client{Transport: itr})

    // Get org installation
    appService := client.Apps
    appInst, resp, err := appService.FindOrganizationInstallation(ctx, "Scout24")
    if err != nil {
       logrus.Infof("error: %s", err)
    }

    // installations, resp, err := appService.ListInstallations(ctx, nil)
    logrus.Info(resp)
    logrus.Info(appInst)
}

And the output is:
INFO[0000] Installation access token: v1.4933ed8b240cb8be4xxxxxxxxxxxxxxxxxxx
INFO[0001] error: GET https://api.github.com/orgs/Scout24/installation: 401 A JSON web token could not be decoded []
INFO[0001] &{0xc000130090 0 0 0 0 github.Rate{Limit:0, Remaining:0, Reset:github.Timestamp{0001-01-01 00:00:00 +0000 UTC}}}
INFO[0001] <nil>

Obviously I do get the installation access token however when I try to get the organization installation it failed with:
error: GET https://api.github.com/orgs/Scout24/installation: 401 A JSON web token could not be decoded []

I guess theAuthorization: Bearer $JWT_TOKEN header is not properly set.
Any ideas?
Thanks in advance.

@dorneanu @willnorris I was running into this issue as well and found that you can only get certain endpoints when you authenticate as an installation, which is what the github.com/bradleyfalzon/ghinstallation package does. For the other Github App endpoints, we need to authenticate with a JWT.

I was able to solve using following code snippets:
...

// RefreshClient will refresh the github app client based on the github installation ID
func (a *GithubAppAuth) RefreshClient(githubInstID int64) {
	itr, err := ghinstallation.New(http.DefaultTransport, a.githubAppID, githubInstID, a.githubAppKey)
	if err != nil {
		log.ErrorWithFields("Couldn't create new transporter", log.Fields{
			"error":        err,
			"githubInstID": githubInstID,
		})
	}

	// Get new token
	client := github.NewClient(&http.Client{Transport: itr})
	token, err := itr.Token(context.Background())
	if err != nil {
		log.ErrorWithFields("Couldn't refresh token", log.Fields{
			"error":        err,
			"githubAppID":  a.githubAppID,
			"githubInstID": githubInstID,
		})
	}
...

Then you can use the token for cloning repositories (I use go-git):
...

I hope this helps.
// Clone will clone the repository using the repo service
func (i *githubAppInstallation) Clone(org, repoName, dstDir string, depth ...int) (*models.Repo, error) {
	// Authenticating via GitHub App installation token
	// The clone url has this syntax<EMAIL_ADDRESS>name>

	if i.token == "" {
		return nil, errors.New("No auth token was provided")
	}

	// Create URL for clone
	url :=<EMAIL_ADDRESS>i.token, repoName)

	// If depth is not specified do a shallow clone
	// Otherwise do a full clone
	var gitCloneOpts *git.CloneOptions
	if len(depth) == 0 {
		gitCloneOpts = &git.CloneOptions{
			URL: url,
			// Create a shallow clone and therefore clone the repo
			// with the history truncated only to the last commit
			Depth:             10,
			RecurseSubmodules: git.DefaultSubmoduleRecursionDepth,
		}
	} else {
		gitCloneOpts = &git.CloneOptions{
			URL:               url,
			RecurseSubmodules: git.DefaultSubmoduleRecursionDepth,
		}
	}

	repo, err := git.PlainClone(dstDir, false, gitCloneOpts)
	if err != nil {
		return nil, err
	}

....

I ran into the same problem. The thing is that the client requires different authentication methods for different calls

most calls require installation token - full list here
apps related calls require JWT (except get an app call) - full list here

I solved the problem by creating two separate clients - one for Apps calls, and the other for the rest:
const gheBaseUrl = "https://my-ghe-installation-url/api/v3"

tr := http.DefaultTransport
appId := 99
installationId := 99v

jwtTransport, _ := ghinstallation.NewAppsTransportKeyFromFile(tr, appId, "path/to/key.pem")
installationTokenTransport := ghinstallation.NewFromAppsTransport(jwtTransport, installationId)

installationTokenTransport.BaseURL = gheBaseUrl
jwtTransport.BaseURL = gheBaseUrl

ghClient, _ := github.NewEnterpriseClient(gheBaseUrl, fmt.Sprintf("%s/upload", gheBaseUrl), &http.Client{Transport: installationTokenTransport})
appsGhClient, _ := github.NewEnterpriseClient(gheBaseUrl, fmt.Sprintf("%s/upload", gheBaseUrl),&http.Client{Transport: jwtTransport})

I raised an issue in https://github.com/bradleyfalzon/ghinstallation regarding the problem.

Closing inactive issue for lack of activity/interest. Feel free to reopen.


[Docs] Initial documentation for upload event icon

This is the docs part of JOINDIN-530.
(Merge only if #289 is accepted!)

Closing because #289 wasn't accepted