Identity Governance - Access Reviews

Running into an issue attempting to interact with /identityGovernance/accessReviews/definitions
type AccessReviewClient struct {
	BaseClient msgraph.Client
}

func NewAccessReviewClient(tenantId string) *AccessReviewClient {
	return &AccessReviewClient{
		BaseClient: msgraph.NewClient(msgraph.VersionBeta, tenantId),
	}
}

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

	clientSecret := os.Getenv("CLIENT_SECRET")
	authConfig := &auth.Config{
		Environment:            environments.USGovernmentL4,
		TenantID:               "<redacted>",
		ClientID:               "<redacted>",
		ClientSecret:           clientSecret,
		EnableClientSecretAuth: true,
	}

	authorizer, err := authConfig.NewAuthorizer(ctx, auth.MsGraph)
	if err != nil {
		log.Fatal(err)
	}

	accessReviewClient := NewAccessReviewClient(authConfig.TenantID)
	accessReviewClient.BaseClient.Authorizer = authorizer
	accessReviewClient.BaseClient.Endpoint = environments.MsGraphUSGovL4Endpoint

	resp, _, _, err := accessReviewClient.BaseClient.Get(ctx, msgraph.GetHttpRequestInput{
		ValidStatusCodes: []int{http.StatusOK},
		DisablePaging:    false,
		Uri: msgraph.Uri{
			Entity: "/identityGovernance/accessReviews/definitions",
			HasTenantId: true,
		},
	})
}

The response I get back is
"could not read response body: http: read on closed response body"
Did some digging with the help of @MarkDordoy
And found what we believe is the source of the issue
When performing this call via https://developer.microsoft.com/
Below is an example response (some details redacted)
{
<EMAIL_ADDRESS>"https://graph.microsoft.us/beta/$metadata#identityGovernance/accessReviews/definitions",
<EMAIL_ADDRESS>4,
    "value": [
        {
            "id": "<redacted>",
            "displayName": "<redacted>",
            "createdDateTime": "<redacted>",
            "lastModifiedDateTime": "<redacted>",
            "status": "InProgress",
            "descriptionForAdmins": "",
            "descriptionForReviewers": "",
            "customDataProvider": null,
            "customData": null,
            "createdBy": {
                "id": "<redacted>",
                "displayName": "<redacted>",
                "type": null,
                "userPrincipalName": "<redacted>"
            },
            "scope": {
<EMAIL_ADDRESS>"#microsoft.graph.accessReviewQueryScope",
                "query": "<redacted>",
                "queryType": "MicrosoftGraph",
                "queryRoot": null
            },
            "instanceEnumerationScope": {
<EMAIL_ADDRESS>"#microsoft.graph.accessReviewQueryScope",
                "query": "<redacted>",
                "queryType": "MicrosoftGraph",
                "queryRoot": null
            },
            "reviewers": [
                {
                    "query": "<redacted>",
                    "queryType": "MicrosoftGraph",
                    "queryRoot": null
                },
                {
                    "query": "<redacted>",
                    "queryType": "MicrosoftGraph",
                    "queryRoot": null
                }
            ],
            "backupReviewers": [],
            "fallbackReviewers": [],
            "settings": {
                "mailNotificationsEnabled": true,
                "reminderNotificationsEnabled": true,
                "justificationRequiredOnApproval": true,
                "defaultDecisionEnabled": false,
                "defaultDecision": "None",
                "instanceDurationInDays": 30,
                "autoApplyDecisionsEnabled": false,
                "recommendationsEnabled": true,
                "recommendationLookBackDuration": "P30D",
                "recurrence": {
                    "pattern": null,
                    "range": {
                        "type": "numbered",
                        "numberOfOccurrences": 0,
                        "recurrenceTimeZone": null,
                        "startDate": "<redacted>",
                        "endDate": "<redacted>"
                    }
                },
                "applyActions": [
                    {
<EMAIL_ADDRESS>"#microsoft.graph.removeAccessApplyAction"
                    }
                ]
            },
            "additionalNotificationRecipients": [],
<EMAIL_ADDRESS>"https://graph.microsoft.us/beta/$metadata#identityGovernance/accessReviews/definitions('<redacted>')/instances",
            "instances": []
        }
    ]
}

Specifically the return value of<EMAIL_ADDRESS>4, is important to note
https://github.com/manicminer/hamilton/blob/main/odata/odata.go#L89
type OData struct {
	Context      *string<EMAIL_ADDRESS>	MetadataEtag *string<EMAIL_ADDRESS>	Type         *Type<EMAIL_ADDRESS>	Count        *string<EMAIL_ADDRESS>	NextLink     *string<EMAIL_ADDRESS>	Delta        *string<EMAIL_ADDRESS>	DeltaLink    *string<EMAIL_ADDRESS>	Id           *Id<EMAIL_ADDRESS>	Etag         *string<EMAIL_ADDRESS>
	Error *Error `json:"-"`

	Value interface{} `json:"value"`
}

when we changed the Count type from *string to *int the call was successful
Is it possible msgraph sometimes returns  *string for Count and other times *int ?

Hi @amarbut24, at first I thought I remembered seeing it returned as a string, however I've tested all the currently supported endpoints and none of them have returned @odata.count, either with or without the ConsistencyLevel header in the request.
It seems reasonable to change this to a *int, the *string type may just be a holdover from earlier testing where I hadn't considered the likelihood of an unmarshaling error.

Hey @manicminer
Thanks for the quick response.
I went ahead and opened this PR - https://github.com/manicminer/hamilton/pull/105

Just for clarity - I take it your implementing this endpoint?

Hey,
Yeah I'm attempting to implement both Access Review Definitions and Access Review Decisions.
Was hoping to open up a PR against this repo once I'm done.
Thanks,
Anthony

From: kaovd @.>
Sent: Saturday, September 18, 2021 4:47 PM
To: manicminer/hamilton @.>
Cc: amarbut24 @.>; Mention @.>
Subject: Re: [manicminer/hamilton] Identity Governance - Access Reviews (#104)
Just for clarity - I take it your implementing this endpoint?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/manicminer/hamilton/issues/104#issuecomment-922389732, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ALZSJ4TAFFX45KJHT63CZZ3UCUQK7ANCNFSM5EDR6R7Q.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.


Chore: use the test ID as the filename when writing to disk

Make PRELUDE_DIR not configurable
Remove print statement
Split curl into 2 calls

We will need to test these with the installers, but should be g2g


Improved protocol support, interop with other server software

Allow the client to interoperate with non-Craft server software. After support for Craft is working https://github.com/satoshinm/NetCraft/issues/12 multiplayer support for web version, expand to other servers. Make a short list for review.

https://github.com/satoshinm/WebSandboxMC


Add contributing guidelines

Add CONTRIBUTING.md with basic contributing guidelines.

@95ych maybe you can take up this?!


Cannot enable Finder integration

Tried checking "Enable Keybase in Finder" in Settings -> Files.
Nothing happens for a while, then "/keybase is unavailable. Please try again." shows at the top of the Keybase app.
Close and re-open the Keybase app then the following error appears:

/keybase contains just KBFS_NOT_RUNNING
my log id: f18787542222b949e86da71c

@reece-bennett I think the problem is that the name of your local account on macOS includes an @.  The finder integration tries to use /Volumes/Keybase <local-user-name> as the mount point for the file system (and /keybase serves as a redirector mountpoint in case there are multiple local users on your system running Keybase).  However, our code isn't expecting an @ since in the local user name -- for security reasons we whitelist the characters that are allowed to be in the mountpoint name, and @ isn't one of them. (See https://github.com/keybase/client/blob/89317cf5ea6f2e1b09e5d515149bc77325e5a3ec/osx/KBKit/KBKit/Component/KBComponent.m#L53 if you want the nitty gritty details.)  cc @cjb.
Perhaps we should allow that character, but it would take a while to get a release out with that change.  But it's pretty easy for you to change the mountpoint yourself.  For example, you can do this from a terminal:
keybase config set mountdir /Volumes/Keybase\ \(me\)

And then restart the Keybase app, and see if that helps.  It doesn't really matter what the mount point is, as long as it's not /keybase or /Volumes/Keybase.  (We discourage you from putting it in your home directory, as that will slow down or confuse programs that back up or crawl your home directory.)

@strib Thanks, that seems to have worked!


nuclio update

hi, just curiosity.
i have seen that niclio is in the 1.9 version, and cvat is still useing (recommending) 1.5

can i change and update the nuclio version ? ( on the docker compose) and it will work?
its there any advantage in updating the nuclio version

thanks

@malfonso0 , it was updated: https://cvat-ai.github.io/cvat/docs/administration/advanced/installation_automatic_annotation/
From the documentation:
You have to install nuctl command line tool to build and deploy serverless functions. Download version 1.8.14.


Setup monitoring for the Marketplace

We have no monitoring setup to know if the Marketplace is alive or what its usage is. For example, to help with branding *.paratext.bible was redirected to *.paratextstudio.org. This broke the marketplace, and we didn't notice for 2+ weeks. I know it's not used right at this moment, but it's going to become very important to know quickly if it goes down for whatever reason. Also, we'll want to know roughly how often the API is getting hit.

LT Ops has Grafana integrated into Rancher, so there is a way to see charts of resource usage. We still need to setup alerting, though.


Remove/unused vif drivers

See #25

go build

go build

PR #25 is merged now, this was just a test.


Please check before merging

With these commits letsencrypt_s3front is working ok, but I'm getting an error:
(from /var/log/letsencrypt/letsencrypt.log)
2015-12-23 10:38:26,954:DEBUG:botocore.hooks:Event needs-retry.iam.UpdateServerCertificate: calling handler <botocore.retryhandler.RetryHandler object at 0xb4df5d30>
2015-12-23 10:38:26,958:DEBUG:botocore.retryhandler:No retry needed.
2015-12-23 10:38:26,961:DEBUG:botocore.hooks:Event after-call.iam.UpdateServerCertificate: calling handler <function json_decode_policies at 0xb51669f0>
2015-12-23 10:38:27,031:DEBUG:letsencrypt.cli:Exiting abnormally:
Traceback (most recent call last):
  File "/root/.local/share/letsencrypt/bin/letsencrypt", line 9, in <module>
    load_entry_point('letsencrypt==0.2.0.dev0', 'console_scripts', 'letsencrypt')()
  File "/root/.local/share/letsencrypt/lib/python2.7/site-packages/letsencrypt-0.2.0.dev0-py2.7.egg/letsencrypt/cli.py", line 1398, in main
    return args.func(args, config, plugins)
  File "/root/.local/share/letsencrypt/lib/python2.7/site-packages/letsencrypt-0.2.0.dev0-py2.7.egg/letsencrypt/cli.py", line 565, in run
    le_client.enhance_config(domains, config)
  File "/root/.local/share/letsencrypt/lib/python2.7/site-packages/letsencrypt-0.2.0.dev0-py2.7.egg/letsencrypt/client.py", line 411, in enhance_config
    redirect = config.redirect if "redirect" in supported else False
TypeError: argument of type 'NoneType' is not iterable

It doesn't breaks anything (the cert is installed correctly on Amazon), and I'm not sure if the error comes from the plugin or the letsencrypt version I'm using (0.2.0.dev0), but the error could confuse the end user.
Also, I'm including 2 domains on the same call (letsencrypt -d mydomain.com,www.mydomain.com ...) and using the option --reinstall instead renewing (I already depleted my 5 cert renews per 7 days to check further)
Could you test with a stable letsencrypt version and using only 1 domain?

/cc @moebiuz

Go ahead :)

They were cherry picked, I am going to leave the other changes out. Thanks @moebiuz!


TPH Where filter not working for TypeIs lambda

EF7 7.0.0-rc1 does not filter when using IsOfType:
var yy = db.Items.Where(i => i is ItemA).ToList();
translates to
SELECT [i].[ItemId], [i].[ItemType], [i].[PropA], [i].[PropB], [i].[PropC]
FROM [Item] AS [i]
WHERE [i].[ItemType] IN ('C', 'B', 'A')

while
var aItems = db.Items.OfType<ItemA>().ToList();
translates to
SELECT [i].[ItemId], [i].[ItemType], [i].[PropA], [i].[PropB], [i].[PropC]
FROM [Item] AS [i]
WHERE [i].[ItemType] = 'A'

The first statement should only allow 'A' as discrimintor value in the SQL statement. Here filtering occurs client-side. The result set is fine but pulls too much data from the SQL server.

Note for triage we were going to put this on the backlog, but I just tried it on the latest nightly and it actually throws now (rather than client eval). I confirmed it did work with client eval on RC1. I think it is still probably ok to go on backlog (since it's not a pattern we intended to support), but we should discuss again.

Unhandled Exception: System.NotImplementedException: The method or operation is not implemented.
at Microsoft.Data.Entity.Query.Internal.ExpressionEqualityComparer.GetHashCode(Expression obj)
at Microsoft.Data.Entity.Query.Internal.ExpressionEqualityComparer.GetHashCode(Expression obj)
at Microsoft.Data.Entity.Query.Internal.ExpressionEqualityComparer.GetHashCode(Expression obj)
at Microsoft.Data.Entity.Query.Internal.ExpressionEqualityComparer.GetHashCode[T](IList1 expressions)    at Microsoft.Data.Entity.Query.Internal.ExpressionEqualityComparer.GetHashCode(Expression obj)    at Microsoft.Data.Entity.Query.CompiledQueryCacheKeyGenerator.CompiledQueryCacheKey.GetHashCode()    at Microsoft.Data.Entity.Query.Internal.RelationalCompiledQueryCacheKeyGenerator.RelationalCompiledQueryCacheKey.GetHashCode()    at Microsoft.Data.Entity.Query.Internal.SqlServerCompiledQueryCacheKeyGenerator.SqlServerCompiledQueryCacheKey.GetHashCode()    at System.Collections.Generic.ObjectEqualityComparer1.GetHashCode(T obj)
at System.Collections.Generic.Dictionary2.Insert(TKey key, TValue value, Boolean add)    at Microsoft.Extensions.Caching.Memory.MemoryCache.Set(Object key, Object value, MemoryCacheEntryOptions cacheEntryOptions)    at Microsoft.Extensions.Caching.Memory.CacheExtensions.Set[TItem](IMemoryCache cache, Object key, TItem value)    at Microsoft.Data.Entity.Query.Internal.CompiledQueryCache.GetOrAddQuery[TResult](Object cacheKey, Func1 compiler)
at Microsoft.Data.Entity.Query.Internal.QueryCompiler.CompileQuery[TResult](Expression query)
at Microsoft.Data.Entity.Query.Internal.QueryCompiler.Execute[TResult](Expression query)
at Microsoft.Data.Entity.Query.Internal.EntityQueryProvider.Execute[TResult](Expression expression)
at Remotion.Linq.QueryableBase1.GetEnumerator()    at System.Collections.Generic.List1..ctor(IEnumerable1 collection)    at System.Linq.Enumerable.ToList[TSource](IEnumerable1 source)

Rowan,
I use the construction .Where(x => x is DerivedY) in list screens. We need a way to filter on type without casting to that type.
For example we want all items that are of type TypeA or of type TypeB we could write:
.Where(x => x is TypeA || x is TypeB)
This can be used in EF6 but not in EF7 at this moment.
Van: Rowan Miller<EMAIL_ADDRESS>Verzonden: woensdag 23 december 2015 0:16
Aan: aspnet/EntityFramework<EMAIL_ADDRESS>CC: Willem-Derk Nijdam - GlasOperator<EMAIL_ADDRESS>Onderwerp: Re: [EntityFramework] TPH Where filter not working for TypeIs lambda (#4070)
Note for triage we were going to put this on the backlog, but I just tried it on the latest nightly and it actually throws now (rather than client eval). I confirmed it did work with client eval on RC1. I think it is still probably ok to go on backlog (since it's not a pattern we intended to support), but we should discuss again.
Unhandled Exception: System.NotImplementedException: The method or operation is not implemented.
at Microsoft.Data.Entity.Query.Internal.ExpressionEqualityComparer.GetHashCode(Expression obj)
at Microsoft.Data.Entity.Query.Internal.ExpressionEqualityComparer.GetHashCode(Expression obj)
at Microsoft.Data.Entity.Query.Internal.ExpressionEqualityComparer.GetHashCode(Expression obj)
at Microsoft.Data.Entity.Query.Internal.ExpressionEqualityComparer.GetHashCodeT<IList%601%20expressions>
at Microsoft.Data.Entity.Query.Internal.ExpressionEqualityComparer.GetHashCode(Expression obj)
at Microsoft.Data.Entity.Query.CompiledQueryCacheKeyGenerator.CompiledQueryCacheKey.GetHashCode()
at Microsoft.Data.Entity.Query.Internal.RelationalCompiledQueryCacheKeyGenerator.RelationalCompiledQueryCacheKey.GetHashCode()
at Microsoft.Data.Entity.Query.Internal.SqlServerCompiledQueryCacheKeyGenerator.SqlServerCompiledQueryCacheKey.GetHashCode()
at System.Collections.Generic.ObjectEqualityComparer1.GetHashCode(T obj)
at System.Collections.Generic.Dictionary2.Insert(TKey key, TValue value, Boolean add)
at Microsoft.Extensions.Caching.Memory.MemoryCache.Set(Object key, Object value, MemoryCacheEntryOptions cacheEntryOptions)
at Microsoft.Extensions.Caching.Memory.CacheExtensions.SetTItem<IMemoryCache%20cache,%20Object%20key,%20TItem%20value>
at Microsoft.Data.Entity.Query.Internal.CompiledQueryCache.GetOrAddQueryTResult<Object%20cacheKey,%20Func%601%20compiler>
at Microsoft.Data.Entity.Query.Internal.QueryCompiler.CompileQueryTResult<Expression%20query>
at Microsoft.Data.Entity.Query.Internal.QueryCompiler.ExecuteTResult<Expression%20query>
at Microsoft.Data.Entity.Query.Internal.EntityQueryProvider.ExecuteTResult<Expression%20expression>
at Remotion.Linq.QueryableBase1.GetEnumerator()
at System.Collections.Generic.List1..ctor(IEnumerable1 collection)
at System.Linq.Enumerable.ToList[TSource](IEnumerable1 source)
—
Reply to this email directly or view it on GitHubhttps://github.com/aspnet/EntityFramework/issues/4070#issuecomment-166759860.

@rowanmiller on the System.NotImplementedException above, actually failing all the time in RC2 release when expression contains any InvocationExpression, someone forget peace of code like this:
case ExpressionType.Invoke:
{
    var invocationExpression = (InvocationExpression)obj;
    for (var i = 0; i< invocationExpression.Arguments.Count; i++)
    {
        hashCode += (hashCode * 397) ^ GetHashCode(invocationExpression.Arguments[i]);
    }
    hashCode += (hashCode * 397) ^ GetHashCode(invocationExpression.Expression);
    break;
}

I'll create new issue, seems it's still not fixed in latest builds.

dupe of #5007


/find  command to let you know if someone is online, and which room they're in

I would like to undertake this task :)

Sure! Should be a pretty simple server one based on commands stuff added yesterday.  You'll need to search soundbounceServer.users for the name, then loop through the rooms + listeners to find them. Good luck!


docs(readme): add syntax highlights

This PR makes the provided code snippets in README.md file easier to read.

Cool, that does look nicer. Thanks for the contribution 👍


Table needs to be centered, but sized based on internals

Maybe remove the containing div and apply its styling to the table?

A related bug is that max-width is basically ignored, but I don't care right now?


Kodadot not reflecting ownership changes when using statemine

What happened?
Mr. @daiagi helped me troubleshooting this specific bug:
Transaction: https://assethub-kusama.subscan.io/extrinsic/5060519-2
NFT: https://canary.kodadot.xyz/stmn/gallery/u-2024-11160
User profile: https://canary.kodadot.xyz/stmn/u/FSMdmCLv5gFZ87YerB3M8ZSQ58BGHdSnreC4J6znWpqPQK9?tab=collected
Chain transaction successful, but no NFT appears on user side and neither changes its ownership in the gallery.
@vikiival
Expected Behavior
should be owned by buyer
What browsers are you seeing the problem on?
Firefox
At which address did you encounter bug?
canary.kodadot.xyz

Is something wrong on indexer maybe?
a few buy transactions on statemine were executed well on the chain but no change of state is shown in Kodadot
this one for example:
https://assethub-kusama.subscan.io/extrinsic/5060255-2
bought the nfts:
https://kodadot.xyz/stmn/gallery/6-8
https://kodadot.xyz/stmn/gallery/6-15
https://kodadot.xyz/stmn/gallery/6-17
at the moment of writing this comment, 6 hours after the transaction, still, these NFT's don't show change of ownership

gm,
as @daiagi said this could be an issue from the squid side.
I will look into it

Looks like that by unknown reasons indexer crashed. I downgraded prod to v3 and will check what happened to v4

@prury
all your NFT as shown as yours at the moment:
https://canary.kodadot.xyz/stmn/u/FSMdmCLv5gFZ87YerB3M8ZSQ58BGHdSnreC4J6znWpqPQK9?tab=collected&page=1

@prury all your NFT as shown as yours at the moment:
https://canary.kodadot.xyz/stmn/u/FSMdmCLv5gFZ87YerB3M8ZSQ58BGHdSnreC4J6znWpqPQK9?tab=collected&page=1

their STATE show as MINE now, ty both!


Make internal APIs of ExchangeContext private to prevent misuse

Problem
CHIP allows devices to have multiple "conversations" simultaneously. Each separate conversation is represented as an exchange.
After the conversation is ended, the owner of exchange should not free the exchange directly, instead, it should call Close() to give up the ownership of the exchange and allow the ExchangeMgr to handle the remaining CRMP ack.  We have hit two crash issues until now because the users of exchange call Free() instead of Close() at the end of  exchange conversation.
We should make those APIs only used by Exchange Layer internal to prevent such mis-use.

Summary of Changes
Make internal APIs of ExchangeContext private to prevent misuse

@saurabhst @msandstedt @mspang @andy31415 ?


requirements for adding the flipbook to ionic 2

sir,thanks for this opensource helpers,....sir, i added pages,libs,providers and added it in app.module.ts file i getting an error like Cannot read property 'subscribe' of undefined
when doing ionic serve....
i commented out translate,translator(both providers),
i am trying this in browser will that be reason?

@aplimovil ...sorry got instructions in examples/pages/


Stripping private symbols causes BUS ERROR in our tests

I don't have a reproducer yet but stripping private symbols seems to cause a lot of crashes in our tests where we use fishhook to override GCD functions. Commit: cd93b791174e0c263c77bb90fcb7247cd9a9ca89
E.g. Crash Report:
Exception Type:  EXC_BAD_ACCESS (SIGBUS)
Exception Subtype: KERN_PROTECTION_FAILURE at 0x000000114f131f00
Exception Codes: 0x0000000000000002, 0x000000114f131f00
VM Region Info: 0x114f131f00 is in 0x1000000000-0x7000000000;  bytes after start:<PHONE_NUMBER>  bytes before end:<PHONE_NUMBER>35
      REGION TYPE                    START - END         [ VSIZE] PRT/MAX SHRMOD  REGION DETAIL
      commpage (reserved)         fc0000000-1000000000   [  1.0G] ---/--- SM=NUL  ...(unallocated)
--->  GPU Carveout (reserved)<PHONE_NUMBER>-7000000000   [384.0G] ---/--- SM=NUL  ...(unallocated)
      GAP OF 0x5f9000000000 BYTES
      MALLOC_NANO<PHONE_NUMBER>00-600008000000 [128.0M] rw-/rwx SM=PRV
Termination Reason: SIGNAL 10 Bus error: 10
Terminating Process: exc handler [69293]

Triggered by Thread:  0

Thread 0 Crashed::  Dispatch queue: com.apple.main-thread
0   Tests                      0x149ae4558 perform_rebinding_with_section + 340
1   Tests                      0x149ae467c perform_rebinding_with_section + 632
2   Tests                      0x149ae423c rebind_symbols_for_image + 1112
3   Tests                      0x149ae43f8 _rebind_symbols_for_image + 56
4   dyld_sim                   0x102e70cd8 invocation function for block in dyld4::APIs::_dyld_register_func_for_add_image(void (*)(mach_header const*, long)) + 164
5   dyld_sim                   0x102e51da8 dyld4::RuntimeState::withLoadersReadLock(void () block_pointer) + 60
6   dyld_sim                   0x102e70b2c dyld4::APIs::_dyld_register_func_for_add_image(void (*)(mach_header const*, long)) + 120
7   Tests                      0x149ae4334 rebind_symbols + 164 // FROM FISHHOOK

Can you try again with the latest commits? I made a few changes to fix issues in indirect function pointers which fishbook uses to find function addresses.

Running into a segfault now. Unable to create a smaller reproducer for you.
Before your recent changes, it would succeed if I reverted cd93b791174e0c263c77bb90fcb7247cd9a9ca89
-------------------------------------
Translated Report (Full Report Below)
-------------------------------------

Incident Identifier: 2E27AF5E-8DCD-4F06-AC4F-5A5783A6B9A3
CrashReporter Key:   0C4884E0-6580-6980-7EF8-02013F355F7C
Hardware Model:      MacBookPro18,2
Process:             xctest [90488]
Path:                /Applications/Xcode.<IP_ADDRESS>B47b.app/Contents/Developer/Platforms/iPhoneSimulator.platform/Developer/Library/Xcode/Agents/xctest
Identifier:          xctest
Version:             ???
Code Type:           ARM-64 (Native)
Role:                Unspecified
Parent Process:      launchd_sim [43687]
Coalition:           com.apple.CoreSimulator.SimDevice.2CF33C78-2842-4B55-B8C1-51153E646A20 [144493]
Responsible Process: SimulatorTrampoline [86412]

Date/Time:           2023-02-08 17:03:39.3477 -0800
Launch Time:         2023-02-08 17:03:38.8461 -0800
OS Version:          macOS 13.1 (22C65)
Release Type:        User
Report Version:      104

Exception Type:  EXC_BAD_ACCESS (SIGSEGV)
Exception Subtype: KERN_INVALID_ADDRESS at 0x0000000000000020
Exception Codes: 0x0000000000000001, 0x0000000000000020
VM Region Info: 0x20 is not in any region.  Bytes before following region:<PHONE_NUMBER>19648
      REGION TYPE                    START - END         [ VSIZE] PRT/MAX SHRMOD  REGION DETAIL
      UNUSED SPACE AT START
--->
      MALLOC_NANO (reserved)<PHONE_NUMBER>00-600020000000 [128.0M] rw-/rwx SM=NUL  ...(unallocated)
Termination Reason: SIGNAL 11 Segmentation fault: 11
Terminating Process: exc handler [90488]

Triggered by Thread:  0

Thread 0 Crashed::  Dispatch queue: com.apple.main-thread
0   libobjc.A.dylib               	       0x18002d054 objc_retain + 16
1   libobjc.A.dylib               	       0x18004efc8 objc_storeStrong + 44
2   UAuthTests                    	       0x128e4d10c ub_dispatch_barrier_async + 52
3   UAuthTests                    	       0x128e5b768 +[UITextView(GlobalMocks) load] + 204
4   libobjc.A.dylib               	       0x18003bbd4 load_images + 896
5   dyld_sim                      	       0x10094e9a8 dyld4::RuntimeState::notifyObjCInit(dyld4::Loader const*) + 160
6   dyld_sim                      	       0x100954c54 dyld4::Loader::runInitializersBottomUp(dyld4::RuntimeState&, dyld3::Array<dyld4::Loader const*>&) const + 200
7   dyld_sim                      	       0x100954d4c dyld4::Loader::runInitializersBottomUpPlusUpwardLinks(dyld4::RuntimeState&) const + 160
8   dyld_sim                      	       0x100969ec8 dyld4::APIs::dlopen_from(char const*, int, void*) + 856
9   CoreFoundation                	       0x18033c94c _CFBundleDlfcnLoadBundle + 152
10  CoreFoundation                	       0x18030a94c _CFBundleLoadExecutableAndReturnError + 332
11  Foundation                    	       0x180ad8818 -[NSBundle loadAndReturnError:] + 384
12  XCTestCore                    	       0x1027a1894 -[XCTestDriver _loadTestBundleFromURL:error:] + 128
13  XCTestCore                    	       0x1027a0318 -[XCTestDriver _prepareTestConfigurationAndIDESession] + 860
14  XCTestCore                    	       0x10279ff94 -[XCTestDriver run] + 20
15  XCTestCore                    	       0x10277af60 _XCTestMain + 84
16  xctest                        	       0x1007888e4 main + 192
17  dyld_sim                      	       0x100945fa0 start_sim + 20
18  dyld                          	       0x100b1de50 start + 2544

It's not easy to debug this without a reproducer, but let me take a look. It's likely that the indirect symbol table is still wrong.

mold-repro-segfault.tar.gz
Here's a repro. Unfortunately, I cannot narrow down the source code so that I can share it. I hope this is enough for you to figure out the problem

It looks like it segfaults only when it tries to replace dispatch_barrier_async, but I'm having a hard time debugging it further. Do you know how to attach a debugger to the test process running in the iOS simulator, or can you provide me a reproducer that runs on macOS instead of iOS?

Here's an updated tar. It will now wait for the debugger. You can attach to the process by executing the following:
lldb --attach-name xctest

mold-repro-segfault_v2.tar.gz

Thank you for sharing the new reproducer. This was a tough one, but I think I managed to fix it in the above commit.

Thanks, this fixed the problem! If you ever get time, you should have youtube streams to show how you go about debugging such problems 😺

The stream would be a few days long. 😁 But really the most important thing when tackling this kind of issue is to set my mind to do it and convince myself that the problem is fixable. This step is sometimes more difficult than fixing the problem itself.


Kotlin minimum 1.4.30, coroutines 1.4.3

Closes https://github.com/apache/logging-log4j-kotlin/issues/31.

@jvz I'm wondering if we should go directly to 1.6? Being two versions behind stable is not unreasonable. Or 1.5 alternatively.

I'd like to make a release of 1.3.0 without Kotlin 1.4 first, then 1.4.0 with Kotlin 1.4, then we can also update the baseline version requirement for a 1.5.0 release (could even match the Kotlin versions along the way like that). Now that the build has been more automated, cutting releases should be much easier (I've just been a little busy at work this month).

@jvz Ping on releasing 1.3.0? Or should we bother at this point? Kotlin is at 1.9.x now. We could probably skip to 1.7 without causing any undue issues.

Closing in favor of https://github.com/apache/logging-log4j-kotlin/pull/43


filterBy activation

var Kefir = require("kefir");
var a = Kefir.sequentially(1000, [1, 2, 3]);
var b = Kefir.sequentially(1000, [false, false, false]);
a.filterBy(b).log();
console.log(a._active); // true

Seems that a shouldn't be active until b is truthy

Yeah, you're probably right. And same for skipUntilBy.
Also marking this as [breaking] because this change may potentially break peoples code.

Does it means, that in case a.filterBy(b) errors from a will not flow?

Hm, yes. I'm not sure which behavior is better here then.

should be ended if primary was ended also can't be implemented. Only when secondary is truthy.
My case for deactivation - I have external process which is represented as endless stream. It starts external process in stream declaration, and ends it using callback in stream declaration for deactivation.
It's important to have child process only when application needs it, listen to it.
There is another observable which declare whether this stream required now.
So for my case:
a$.flatMapLatest(a => a ? external$ : Kefir.never())

And it works ok for me. Its behavior almost same as external$.filterBy(a$) except things discussed here.
So we can leave filterBy as is for cases when 'errors' and 'end' are important, and it's ok that primary activated.

Sounds good to me. I'll close this then, and we can revisit this later.


Update README about merge algorithms

We currently support two algorithm

Hierarchical: This merge algo tries to maintain, the order of the dependent components to its primary component. For spdx this is done via relationships and for cyclonedx via nested components & dependencies.
Flat: As the name states, are just consolidated lists of components, dependencies, etc.
Assembly: Merge is very similar to Hierarchical, except that it does not create dependency relationships among the merged sboms.

This is three algorithms, not two.
You should add that Flat is not yet implemented.
What is the default algorithm, i.e. when -a, -fand -mare not used?

The default algorithm is Hierarchal Merge for both SPDX and CycloneDX. Yes i plan on implementing both FLAT & Assembly merge for SPDX soon. Hopefully next week.

Added more details here https://github.com/interlynk-io/sbomasm/pull/83


Fix codeowners

codeowners team reference needs an @ mention

I'll integrate these changes in my rewrite PR.

merged in previous PR


[Storage] Swagger stg 96

[x] The purpose of this PR is explained in this or a referenced issue.
[ ] The PR does not update generated files.

These files are managed by the codegen framework at Azure/autorest.go.

[ ] Tests are included and/or updated for code changes.
[ ] Updates to module CHANGELOG.md are included.


feat(primitives) : Chunk Implementation for Parallel Sender Recovery

Added chunk implementation to TransactionSigned::recover_signers to handle parallel sender recovery using rayon. Replaced into_par_iter method handling parallel sender recovery for one signed transaction at a time. Partially resolving Investigate TransactionSigned::recover_signers performance issue.

Thanks for taking this on! How does this impl compare perf wise to the current impl on main?

supportive of @onbjerg comments:

locks are unnecessary here. the better API would be to return the stream wrapper around receiver channels to guarantee the order of recovered signers
we need benchmarks to make sure this is an improvement

supportive of @onbjerg comments:

locks are unnecessary here. the better API would be to return the stream wrapper around receiver channels to guarantee the order of recovered signers
we need benchmarks to make sure this is an improvement

Could you please take a look at the modified stream wrapper implementation? I could not figure out how to benchmark the performance

@onbjerg @rkrasiuk please check the benchmark tests I have added. The chunk implementation outperforms the current into_par_iter method after around 100 transactions.

Fixed the arbitrary constants. Chunk implementation amended to pass previously failing consensus tests. into_par_iter slightly outperforms chunk impl in unoptimized tests but the overhead of work stealing with into_par_iter might increase with the number of transactions.

Can you re-run this with mainnet SenderRecovery stage via reth stage run senders and measure the time before/after this PR?

Can you re-run this with mainnet SenderRecovery stage via reth stage run senders and measure the time before/after this PR?

Does that need sync with transaction history because my system does not have the required specs?

Hi @Arindam2407 wonder if you had a chance to compare performance before/after?

Hi @Arindam2407 wonder if you had a chance to compare performance before/after?

Hi @gakonst, I was not able to run the mainnet SenderRecovery stage because of my device specs. Instead, I ran optimized tests using cargo test --package reth-primitives --release to get more reliable measurement as pointed out by @rakita. I compared the performance before/after (without/with chunks) for recovering 100,000 transactions.
The test:
Additional crates used in mod tests:
use std::{fs::File, io::Write, time::Instant};
use rayon::prelude::*;
    #[test]
    fn benchmark_chunks_parallel_sender_recovery_100000(txes in proptest::collection::vec(proptest::prelude::any::<Transaction>(), *PARALLEL_SENDER_RECOVERY_THRESHOLD * 20000)) {
        let mut rng =rand::thread_rng();
        let secp = Secp256k1::new();
        let txes: Vec<TransactionSigned> = txes.into_iter().map(|mut tx| {
             if let Some(chain_id) = tx.chain_id() {
                // Otherwise we might overflow when calculating `v` on `recalculate_hash`
                tx.set_chain_id(chain_id % (u64::MAX / 2 - 36));
            }

            let key_pair = KeyPair::new(&secp, &mut rng);

            let signature =
                sign_message(B256::from_slice(&key_pair.secret_bytes()[..]), tx.signature_hash()).unwrap();

            TransactionSigned::from_transaction_and_signature(tx, signature)
        }).collect();

        let mut file = File::create("benchmark.txt")?;

        let start_chunk = Instant::now();
        let _par_chunk_senders = TransactionSigned::recover_signers(&txes, txes.len());
        let elapsed_chunk = start_chunk.elapsed();

        let start = Instant::now();
        let _par_senders = txes.into_par_iter().map(|tx| tx.recover_signer()).collect::<Option<Vec<_>>>();
        let elapsed = start.elapsed();

        write!(file, "Time taken for parallel recovery of 100000 addresses: {:.2?} (with chunks) and {:.2?} (without chunks)", elapsed_chunk, elapsed)?;

    }

The results:

Chunk implementation takes 20-30% less time than the current implementation in all runs. It would be great if anyone else could run it and compare the performance. Since they are all optimized test runs, I think the estimates are reliable.


for: add test cases and comments

Partially For kubebb/core#194
Added test cases for Repository.NamespacedName()
Also added comments and corrected typos

Partially fix #194

https://docs.github.com/en/issues/tracking-your-work-with-issues/linking-a-pull-request-to-an-issue#linking-a-pull-request-to-an-issue-using-a-keyword fix keyword will close this issue, but we're not done with it, we don't want to close it, as the kubernetes guys did, we can use the for keyword.

Good point! We can add this to our contribute guidelines https://kubebb.github.io/website/docs/contribute#pr规范

Partially fix #194

https://docs.github.com/en/issues/tracking-your-work-with-issues/linking-a-pull-request-to-an-issue#linking-a-pull-request-to-an-issue-using-a-keyword fix keyword will close this issue, but we're not done with it, we don't want to close it, as the kubernetes guys did, we can use the for keyword.(or any other keyword, we just want link to this issue.)

Good point! We can add this to our contribute guidelines https://kubebb.github.io/website/docs/contribute#pr规范

I think I got something wrong 😭
I will create a new branch and copy the changes there!

All commits are merged into one now!

Very good, now it is in one commit, now we need to modify a little bit, change the commit message of this commit, "All in One!" It doesn't say what it did, you can refer to the unfinished https://gist.github.com/Abirdcfly/b942d4e9d44f3fb71344b92b979c8d40 to write an appropriate commit message.
The reference command may be
$ git commit --amend
$ in vim, change commit message....
$ git push origin documentation --force

Thank you very much! The commit message is now: "chore: add test cases for Repository.NamespacedName".

@Henry-Gao-2004 Thanks for your work! Can you add more test cases for other functions in this repository.go?

Sure no problem at all!


Question about the number of nuclei in the Kumar dataset.

The number of nuclei in the downloaded Kumar dataset is 16966, which is different from the reported 21623 in some papers like Hover-net.
cell_count = 0
for root, _, files in os.walk('raw_data/kumar'):
    for file in files:
        if file.endswith('mat'):
            data = scipy.io.loadmat(os.path.join(root, file))
            cell_count += data['inst_map'].max()
print(cell_count)

please check this, one of my issue. https://github.com/vqdang/hover_net/issues/86
please check if you have used correct dataset.

Thanks. But I'm more confused now. Is the statistical result of 21,623 presented in the Hover-Net paper incorrect? I get the result of 16,966 on the dataset directly downloaded from their provided url.

please check this, one of my issue. vqdang/hover_net#86
please check if you have used correct dataset.

Thanks. But I'm more confused now. Is the statistical result of 21,623 presented in the Hover-Net paper incorrect? I get the result of 16,966 on the dataset directly downloaded from their provided url.

For your reference, the kumar dataset i used has 9914 in train set and 7052 in test set. I don't want to comment. Good luck.

For your reference, the kumar dataset i used has 9914 in train set and 7052 in test set. I don't want to comment. Good luck.

I get it. Thanks very much.


etcd restoration needs work

Had a failure of etcd tonight on my cluster. I smelled it coming because the etcd units had error messages even though etcd was up and running fine, so I had backups. Trying to restore a saved backup has proven to be too much. I manually backed up the data and also ran a snapshot action.
Things I tried:

I nuked all my units and brought one online and tried a restore action. The action completed, but no real information was given in the output. The etcd cluster seemed correct, but it was unhealthy with old members. Unable to reset member list. Suspect it picked up old data dir and ran with it.
I nuked all my units and manually ssh'd around and nuked the snap and any data directories. Brought one unit online and tried snapshot restore action. Nothing happened, cluster was empty. Tried various target arguments such as /home/ubuntu/etcd-snapshots and nothing seemed to happen and the cluster was empty.
Nuked the entire etcd application, all snaps, and all data. Re-deployed it and ran restore action. Empty cluster. Looked at the code in the restore.py file of etcd at this point and got much further. I was able to restore my etcd cluster to a single node using -force-new-cluster and running etcd manually while I updated the advertise address. Added a new unit and the first says unhealthy with 2 known peers and the second unit said errored with 0 known peers. I then added another unit thinking I needed a quorum. This 3rd unit says waiting to retry etcd registration.

That is the point where I ran field agent. I'm at the point where I want to just destroy the entire cluster, release the machines to MaaS and start over. This will be a large undertaking, but it seems easier than trying to get etcd happy.
field agent log

Yep, it sounds similar to my experience: https://github.com/juju-solutions/bundle-canonical-kubernetes/issues/582
This is a serious problem and blocks customers.


catppuccin: add starship theme

Description of changes
This adds the starship theme from https://github.com/catppuccin/starship into the catppuccin package
Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Sure, my bad for not noticing :sweat_smile:

then just squash into a single commit


first() returns and array instead of stdClass object

Hello,
I have setup two fresh Laravel 5.3 with different databases. One with MongoDB(using this package) and another with MySQL.
I have run this simple code on both setup:
$user = DB::table('users')->first();
dd($user);

For above code, Laravel with MySQL returns stdClass object whereas Laravel with MongoDB returns an array  .
Why is that? Is there any way that I can make it same as Laravel MySQL i.e stdClass object?
Due to this different return type issue, I'm not able to use another package with Laravel MongoDB setup.
Any help would be appreciated.
Thanks,
Parth vora

Hey,
Any update on this? By the way my question has nothing to do with any relationship.
Thanks

@jenssegers
Can you please give some reply to this issue?

This also causes issues with Laravel Passport, as certain parts of it use the database connection directly, rather than using Eloquent models (which can easily be copied and swapped out in a service provider): https://github.com/laravel/passport/blob/610b2ea158ae840a656a4e7f1518d3c6b80f0639/src/Bridge/RefreshTokenRepository.php#L80


Window modifier has unexpected background

After updating to 1.1.0, window has white background that hovers whole screen.
In 1.0.3 and earlier, modifier destination was displayed without background.
@State var isPresented = false

var body: some View {
    Button("show") {
        isPresented = true
    }
    .window(
        level: .overlay,
        transition: .move(edge: .top).combined(with: .opacity),
        isPresented: $isPresented
    ) {
        Text("hey")
    }
}

This is fixed by #30

Fixed in 1.1.2


Complete String and Character classes

Check what's missing for the string-related classes String, Character and StringBuilder, and finish them. Some parts, like determining if a char is a Unicode titlecase or letter character, may be a bit tricky to achieve because there is only limited Unicode support in JavaScript.

I added a bunch of methods to String, but most of them are just unimplemented (I just added them to avoid runtime errors that stop the tests due to missing methods).


video: Remove the unused vbuf_curr field from video_framebuff_s

Summary
Impact
code refactor
Testing
Pass CI

https://github.com/apache/incubator-nuttx/actions/runs/3442225986/jobs/5742619033#step:7:329


Setup domain

https://hackernoon.com/how-to-set-up-godaddy-domain-with-github-pages-a9300366c7b

Pointed domain
Have to wait for Github to update security and HTTPS


Feature/tests

This is an implementation of #91 and #92 and partially #93.

The first commit is an error, sorry, the added file has been already removed.

I really like this, seems way clearer to me now.
Also a great step to maybe one day have a documented api that is not the cli, but a python api that can be used by other programs.

Do you need other changes here? Otherwise I'll start working on some next steps.

Sorry, just time constraints on my side. I hope to have a few hours to look at this in detail soon.
I like the general direction and hope to get it integrated sooner than later.

Sorry, I didn't want to hurry you, no problem, I will start from here to
try and split some other parts of the whole program.
Take your time ;)
Leonardo Giordani
@tw_lgiordani http://twitter.com/tw_lgiordani - lgiordani.com
My profile on About.me http://about.me/leonardo.giordani - My GitHub page
https://github.com/lgiordani
On 23 June 2015 at 14:40, Filip Noetzel<EMAIL_ADDRESS>wrote:

Sorry, just time constraints on my side. I hope to have a few hours to
look at this in detail soon.
I like the general direction and hope to get it integrated sooner than
later.
—
Reply to this email directly or view it on GitHub
https://github.com/peritus/bumpversion/pull/95#issuecomment-114480516.

No worries :)

I opened #98 to clean up a bit of mess with Git history, so that new PR do not contain wrong commits.


WIP: Add a concurrent test for issue creation

Related to #7887, #7944

@guillep2k have you try to cherry pick this PR over your PR to validate ?

Have you had any progress with this? Are you still getting random errors from the databases?
I'm thinking that rather than starting with an empty table, this test would be much more efficient if issue is pre-loaded with a large set of rows, like directly inserting 10,000 issues with one task (setting Index manually, no conflicts there) and then launch 1,000 threads to attempt to break the logic.

@guillep2k @sapk
I am still not sure if the code is bug-free. I tested it with the master, and it would hang occasionally.
I tested with @guillep2k 's PR. There seemed to be the internal 500 error from POST as well.
I am also considering if the parallel subtests of the testing package would be a better alternative for such kind of tests. I am not familiar with it. How do you think?

@guillep2k @sapk
I am still not sure if the code is bug-free. I tested it with the master, and it would hang occasionally.
I tested with #7944. There seemed to be the internal 500 error from POST as well.
I am also considering if the parallel subtests of the testing package would be a better alternative for such kind of tests. I am not familiar with it. what do you think?

I think it's important to bypass the HTTP API for this. Otherwise the chain is too long and any one link can fail, not only the potential dup key problem. Also, it's difficult to stress a database unless you're dealing with it in the most direct way possible.
Having that level of concurrency is unrealistic. There will never be 1,000 threads running at the same time. The database connections are pooled and therefore most clients will be waiting in queue, which does not serve the purpose.
In your tests you're hitting:
goroutine 0 [idle]:
runtime.epollwait(0x4, 0x7ffc92970a00, 0xffffffff00000080, 0x0, 0xffffffff005ddaee, 0x10800000000, 0x0, 0x0, 0x0, 0x0, ...)
/usr/local/go/src/runtime/sys_linux_amd64.s:675 +0x20
runtime.netpoll(0xc000060501, 0xc000054a01)
/usr/local/go/src/runtime/netpoll_epoll.go:71 +0x126
runtime.findrunnable(0xc000050000, 0x0)
/usr/local/go/src/runtime/proc.go:2380 +0x51f
runtime.schedule()
/usr/local/go/src/runtime/proc.go:2525 +0x21c
runtime.goexit0(0xc0011aa600)
/usr/local/go/src/runtime/proc.go:2722 +0x1dd
runtime.mcall(0x15ebb90)
/usr/local/go/src/runtime/asm_amd64.s:299 +0x5b

I'm no golang expert, but I do have experience with this kind of scenario. netpoll_epoll.go sounds like a network wait loop, most probably related with your HTTP request than the database.
I'd restructure this test in the following way:

Move the test to models/. It will be tested with all databases anyway.
No HTTP: only database queries through x.Insert() (x is the xorm Engine that will become available inside the models pkg).
Within the main thread, create a large number of issues (1,000? 10,000? it's worth some tests) by manually assigning the Index value. The goal is to have MAX(Index) do some work.
Set up a reference timer (e.g. time.Now())
Set up a var has_errors error with a nil value.
Create only 4 or 8 threads for inserting. Each thread would loop through this:

Insert an issue through models.newIssue(). If errors, set has_errors and return.
If X time has passed counting from the set reference (20 seconds? 60 seconds?), return.
If has_errors is not nil, return.
Repeat.

In the main thread, just wait for the other threads to finish and then check for has_errors. You can assert that it's nil.

With 4 or 8 threads, chances of simultaneous insert are probably the same as with 1,000, but it's less likely to fail for other causes. At least that's what I think.

OK, I couldn't help myself. Please check #7959.

In favor of #7959


Whether you need to prefetch the image on a node in advance

If nodes in the cluster pull the same image at the same time, do you need to preheat a node in advance?

Currently you have to preheat a node in advance if you want to improve deployment situations where multiple nodes pull the same image at once. I have an idea to optimize this by advertising in flight image pulls but I am not even sure if that is possible yet.


Hardhat plugin always rebuilds typechain artifacts

Bug with the @typechain/hardhat package.
When using the typechain plugin for hardhat, typechain artifacts will always be rebuilt, even if the smart contracts haven't changed.
While this isn't breaking, it adds an additional few seconds to every time a hardhat command is run.
Is there any way that the plugin could detect that the typechain artifacts are out of date? Perhaps check the modified time of the artifacts compared to the source solidity files and only regenerate if needed?
I'd be interested in fixing this and submitting it as a PR if it's not already in progress.

Good point 👍
@alcuadrado any suggestions? Perhaps we could use the same machinery as hardhat uses internally to realize which files need recompiling.

The issue isn't related only to hardhat target I think -- other targets could also benefit of only building the needed files.
In my project that uses the ethers target, I have a bash script with some heuristics for that (get contract names from sol files with grep&sed, then check if the expected resulting files exist and are up to date). Maybe such behavior could be integrated into Typechain in some level? Probably hidden behind a flag, to avoid weird edge cases -- for example when someone switches to another git branch where the contract files are different but older, which wouldn't trigger a rebuild.

btw: here are the scripts I mentioned if anyone's interested: https://gist.github.com/quezak/b0cd46ec2d378a34edf0734505a5725e

@alcuadrado any suggestions? Perhaps we could use the same machinery as hardhat uses internally to realize which files need recompiling.

Pretty sure you can. Will think about it and get back to this tomorrow.

Wanted to come back and say thank you so much for adding this feature. Really appreciate it!


Pubsec.yaml issues.

Steps to reproduce
Command lines or steps to take to get out of these loops.
Actual results
I upgraded my packages, and when I saved it showed me the error below:
Because every version of flutter_test from sdk depends on web 0.3.0 and myapp depends on web ^0.4.2, flutter_test from sdk is forbidden.
So, because myapp depends on flutter_test from sdk, version solving failed.
exit code 1.
It repeats the same error across the 6 packages till downgrade them all to the suggested version, but still shows the message below:
myapp] flutter pub get --no-example
Resolving dependencies...
matcher 0.12.16 (0.12.16+1 available)
material_color_utilities 0.5.0 (0.8.0 available)
meta 1.10.0 (1.12.0 available)
path 1.8.3 (1.9.0 available)
test_api 0.6.1 (0.7.0 available)
web 0.3.0 (0.4.2 available)
Got dependencies!
6 packages have newer versions incompatible with dependency constraints.
Try flutter pub outdated for more information.
exit code 0
Trust me I've done everything I could even tried a remote desktop when I called out for help but still all the same. It's frustrating.
Please help the newbie in Flutter.
Logs
Note: web is pinned to version 0.3.0 by flutter_test from the flutter SDK.
See https://dart.dev/go/sdk-version-pinning for details.
Because every version of flutter_test from sdk depends on test_api 0.6.1 and myapp depends on test_api ^0.7.0, flutter_test from sdk is forbidden.
So, because myapp depends on flutter_test from sdk, version solving failed.
Because every version of flutter_test from sdk depends on web 0.3.0 and myapp depends on web ^0.4.2, flutter_test from sdk is forbidden.
So, because myapp depends on flutter_test from sdk, version solving failed.
exit code 1
Flutter Doctor output
flutter doctor
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, 3.16.9, on macOS 14.3 23D56 darwin-x64, locale en-GB)
[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
[!] Xcode - develop for iOS and macOS (Xcode 15.2)
✗ Unable to get list of installed Simulator runtimes.
[✓] Chrome - develop for the web
[✓] Android Studio (version 2023.1)
[✓] VS Code (version 1.86.0)
[✓] Connected device (2 available)
[✓] Network resources
! Doctor found issues in 1 category.

@Jossyboydgenius  use dependency_overrides like
dependency_overrides:
  analyzer: ^6.4.1

It repeats the same error across the 6 packages till downgrade them all to the suggested version, but still shows the message below:
myapp] flutter pub get --no-example
Resolving dependencies...
  matcher 0.12.16 (0.12.16+1 available)
  material_color_utilities 0.5.0 (0.8.0 available)
  meta 1.10.0 (1.12.0 available)
  path 1.8.3 (1.9.0 available)
  test_api 0.6.1 (0.7.0 available)
  web 0.3.0 (0.4.2 available)
Got dependencies!
6 packages have newer versions incompatible with dependency constraints.
Try `flutter pub outdated` for more information.
exit code 0

Exit code 0 is successful; the "6 packages have newer versions incompatible with dependency constraints" message is informational.
What you have described here is correct behavior, leading to a working app with the correct versions of packages based on what is compatible with the version of Flutter you are using. Flutter pins some specific dependencies, so it is expected behavior that you can't always use the newest version of every package with a given version of Flutter.
Closing as this is intended behavior.

@amrgetment In the future, please don't post commands in our issue tracker that are likely to break people's projects in hard-to-understand ways, especially with no context at all.

Alright, like I said I'm new to newbie so I don't know much of this RN.
Thanks anyway.


Build

Hi, how can I make production build?

Hey @vintentou!
It's actually not that difficult, we should really add it to CGS. I haven't had a lot of time to look at it.
Basically we could add a build script that runs babel into a dist directory and a start script that runs index from there. We'd need to add some instructions about setting MONGO_URL and there'd be a few details to make sure we get right.
OTOH; it's not a huge deal to just run the development mode in production (I'm doing so with a couple of apps, although they don't get a lot of traffic so YMMV). If you pass MONGO_URL it won't start up the local mongo, and although babel-node isn't as efficient as node in my experience it's not terrible.


Documentation

Under "Usage".
In the picture, the lines that start with "fetches" and "optionally" are not full sentences, but they end with periods.

This is a great catch. I love this level of detail. If you read the first sentence including the word 'ethslurp' from the line above it is. I fixed the second sentence. By the way--these fixes are still not posted to the website. They are only fixed locally on my computer, so you won't see them until later this week when I publish the changes to the web. I'll let you know and you can run through it again.


hipMalloc() failed in docker container

I am using HIP and openMPI to develop my program in Docker. My Server has 4 Vega 20(MI60, Device Memory is 32GB).  When I using 'mpirun -np 4 ./myApp' to lauch four processes to start my program. I found rank-0 (process 0) failed to executed
hipMalloc(&dvptr, numbytes).
I have already assigned each device to each process to make sure each process has enough device memory to allocate.
And I use GDB to attach rank-0, the value of numbytes is<PHONE_NUMBER>8. (Obviously, My GPU has enough memory to allocate).  But the dvptr is NULL after invoke hipMalloc(). And it returns hipErrorMemoryAllocation.
However, other processes have allocated<PHONE_NUMBER>6 bytes device memory in respective devices successfully.
But the program is aborted cause rank-0's dvptr is NULL.
I also call hipMemGetInfo(&mem_free, &mem_total) to query memory info.  The mem_free=34038743032, mem_total=34342961152. And both variants do not change after invoke hipMalloc() in rank-0.
Therefore, I want to ask is there any caution to invoke hipMalloc() or something special to develop HIP program in Docker? Why dvptr is NULL after invoking hipMalloc()?
The Docker Environment:
Ubuntu 16.04.2 LTS
ROCm 2.3
HIP version: 1.5.19123
OpenMPI 1.10
The Server Environment:
Ubuntu 16.04.6 LTS
Docker version 18.09.5
ROCm 2.3
HIP version: 1.5.19123

Hi @huangjw08, apologies for the lack of response.  Can you please check latest ROCm 6.0.2 (HIP 6.0.32831) to see if your query has been addressed? Thanks.

Closing the ticket as it is stale.  @huangjw08, please re-open if you still see this issue with latest ROCm 6.0.2 (HIP 6.0.32831).


Not listing all Python interpreters on PATH named python.exe

Environment data

VS Code version: 1.27.0-insiders
Extension version (available under the Extensions sidebar): 2018.7.1
OS and version: Windows 10
Python version (& distribution if applicable, e.g. Anaconda): Various
Type of virtual environment used (N/A | venv | virtualenv | conda | ...): N/A
Relevant/affected Python packages and their versions: Python 3.4.4

Actual behavior
Third party programs (Plaxis) has installed Python, but these versions are not shown in the intepreter selection unless they are first in the list from the Windows Path Environmental Variable. If I change the order, once again only the first listed path is shown.
Expected  behavior
All Python interpreters in the Path should be shown.
Steps to reproduce:

Have multiple Python interpreters installed under Program Files.
Reorder PATH to only shown the first version in VS Code.

Order A:

Result A:

Order B:

Result B:

Sorry for the delay.
Python 3.4.4 and Python 3.4.5.
Both directories have "python.exe" and "pythonw.exe" but no other executables in the root.

OK, so the problem is most likely that we don't search every entry for python on PATH, just the first one (probably need to check if this is Windows-only or if we do the same thing on UNIX). And on top of that these interpreters are most likely not registering themselves in the registry, so finding them through PATH is the only way to list them.

@pjkottke we will find all Python installs registered in the registry, but I don't know exact details of what to set to make them appear (there's a PEP specifying it all).

I'm re-opening so we can make sure we are searching all of PATH.

Can you verify this, @ericsnowcurrently ?

I'm re-opening so we can make sure we are searching all of PATH so we can get a proper fix.

This is an enhancement. I can verify this will not work as we do not iterate through all directories in the PATH to look for python interpreters.


JAVA_ENABLE_DEBUG for jdk 11 is not valid.

I modify my dockfile for this  (my port is 8181, not 8080 for my example)
EXPOSE 5005 8181
and I ran this command : (we see that the port 5005 was setted)

C:\workspace\hello>docker run -i --rm -p 8181:8181 -p 5005:5005 -e JAVA_ENABLE_DEBUG="true" hello
exec java -Dquarkus.http.host=<IP_ADDRESS> -Djava.util.logging.manager=org.jboss.logmanager.LogManager -agentlib:jdwp=transport=dt_socket,server=y,suspend=n,address=5005 -XX:+UseParallelGC -XX:GCTimeRatio=4 -XX:AdaptiveSizePolicyWeight=90 -XX:MinHeapFreeRatio=20 -XX:MaxHeapFreeRatio=40 -XX:+ExitOnOutOfMemoryError -cp . -jar /deployments/app.jar
Listening for transport dt_socket at address: 5005
SLF4J: Class path contains multiple SLF4J bindings.
SLF4J: Found binding in [jar:file:/deployments/lib/ch.qos.logback.logback-classic-1.2.3.jar!/org/slf4j/impl/StaticLoggerBinder.class]
SLF4J: Found binding in [jar:file:/deployments/lib/org.jboss.slf4j.slf4j-jboss-logging-1.2.0.Final.jar!/org/slf4j/impl/StaticLoggerBinder.class]
SLF4J: See http://www.slf4j.org/codes.html#multiple_bindings for an explanation.
SLF4J: Actual binding is of type [ch.qos.logback.classic.util.ContextSelectorStaticBinder]
__  ____  __  _____   ___  __ ____  ______
 --/ __ \/ / / / _ | / _ \/ //_/ / / / __/
 -/ /_/ / /_/ / __ |/ , _/ ,< / /_/ /\ \
--\___\_\____/_/ |_/_/|_/_/|_|\____/___/
2020-04-07 13:02:16,542 INFO  [io.quarkus] (main) hello 1.0.0-SNAPSHOT (powered by Quarkus 1.3.1.Final) started in 2.732s. Listening on: http://<IP_ADDRESS>:8181
2020-04-07 13:02:16,544 INFO  [io.quarkus] (main) Profile prod activated.
2020-04-07 13:02:16,545 INFO  [io.quarkus] (main) Installed features: [cdi, hibernate-validator, kubernetes, rest-client, resteasy, resteasy-jackson, resteasy-jsonb, smallrye-context-propagation, smallrye-fault-tolerance, smallrye-health, smallrye-metrics, smallrye-openapi, spring-boot-properties, spring-di, spring-web, swagger-ui]

in docker ps I see this
<IP_ADDRESS>:5005->5005/tcp, <IP_ADDRESS>:8181->8181/tcp
when I try to attach to the port 5005 I obtain this error  (in Intellij)
Error running 'quarkus ': Unable to open debugger port (localhost:5005): java.io.IOException "handshake failed - connection prematurally closed"
in IntelliJ, I have
Attach to remote JVM
socket
host : localhost   port 5005
-agentlib:jdwp=transport=dt_socket,server=y,suspend=n,address=*:5005
It might be a bug in the run script, since for Java 11 it should have added
-agentlib:jdwp=transport=dt_socket,server=y,suspend=n,address=*:5005
instead of
-agentlib:jdwp=transport=dt_socket,server=y,suspend=n,address=5005

the workaround for now, is modify the JAVA_OPTIONS to include the port and not passing the variable
ENV JAVA_OPTIONS="-Dquarkus.http.host=<IP_ADDRESS> -Djava.util.logging.manager=org.jboss.logmanager.LogManager -agentlib:jdwp=transport=dt_socket,server=y,suspend=n,address=*:5005"

I think for Java 11 it doesn't really matter how to specify the address field:
$ java -version
java version "11.0.6" 2020-01-14 LTS
Java(TM) SE Runtime Environment 18.9 (build 11.0.6+8-LTS)
Java HotSpot(TM) 64-Bit Server VM 18.9 (build 11.0.6+8-LTS, mixed mode)

$ java -agentlib:jdwp=transport=dt_socket,server=y,suspend=n,address=*:5005 -jar random-generator-runner.jar
Listening for transport dt_socket at address: 5005
__  ____  __  _____   ___  __ ____  ______
 --/ __ \/ / / / _ | / _ \/ //_/ / / / __/
 -/ /_/ / /_/ / __ |/ , _/ ,< / /_/ /\ \
--\___\_\____/_/ |_/_/|_/_/|_|\____/___/
2020-04-07 17:16:33,254 INFO  [io.quarkus] (main) random-generator 2.0 (powered by Quarkus 1.3.1.Final) started in 0.885s. Listening on: http://<IP_ADDRESS>:8080
2020-04-07 17:16:33,278 INFO  [io.quarkus] (main) Profile prod activated.
2020-04-07 17:16:33,278 INFO  [io.quarkus] (main) Installed features: [cdi, resteasy, resteasy-jsonb, smallrye-health]
^C2020-04-07 17:16:34,352 INFO  [io.quarkus] (main) random-generator stopped in 0.035s

$ java -agentlib:jdwp=transport=dt_socket,server=y,suspend=n,address=5005 -jar random-generator-runner.jar
Listening for transport dt_socket at address: 5005
__  ____  __  _____   ___  __ ____  ______
 --/ __ \/ / / / _ | / _ \/ //_/ / / / __/
 -/ /_/ / /_/ / __ |/ , _/ ,< / /_/ /\ \
--\___\_\____/_/ |_/_/|_/_/|_|\____/___/
2020-04-07 17:16:39,510 INFO  [io.quarkus] (main) random-generator 2.0 (powered by Quarkus 1.3.1.Final) started in 0.759s. Listening on: http://<IP_ADDRESS>:8080
2020-04-07 17:16:39,532 INFO  [io.quarkus] (main) Profile prod activated.
2020-04-07 17:16:39,533 INFO  [io.quarkus] (main) Installed features: [cdi, resteasy, resteasy-jsonb, smallrye-health]
2020-04-07 17:16:40,942 INFO  [io.quarkus] (main) random-generator stopped in 0.033s

However it could be that for Java 11 binding with a --adress=5005 only binds to localhost, which won't work in Docker container.
We should address that in run-java.sh and generate a different options depending on the Java version.
Would you mind to add an issue over there https://github.com/fabric8io-images/run-java-sh ? (that's the startup script). Thanks !

it's not what I did already ?  https://github.com/fabric8io-images/run-java-sh/issues/101   did you give me the right url ?

ah, sorry. Was totally confused (to much on plate these days), thought this issue was opened against https://github.com/fabric8io-images/java . My bad, sorry. Will try to add this asap, but if you would like to give a try and send in pull request, this could speed up things.

here : https://github.com/fabric8io-images/run-java-sh/pull/102

@rhuss does the PR look ok to you ?

Had no time to review it, but should happen towards the end of the week.

I'm back to follow up :)

I think we will be able to close this defect


Start of the implementation of patterns

Thank you for the great work!
I converted the PR into a draft, so you can furthermore expand design patterns if you want.

We recently changed the folder names into the English language, so that they are more general. The naming convention is also snake_case (everthing lowercase with underscores). I am kindly asking you to update the new folder names so that it meets the convention...

I will adapt it and make a new commit


Enable Clang's thready safety analysis

This is enabling -Werror=thread-safety which allows to enforce ABSL_GUARDED_BY among other things.
But first we need to fix a couple of threading problems. Most of them are missing annotations or similar but there are
also some things that might be wrong and need proper fixing.

LGTM, Thanks!


[node-api-headers] Update to 1.0.0

https://github.com/nodejs/node-api-headers/commit/99db75a55c5a90e3d1dce7d0ddfe4c1a0143a0b5

Thanks!


chore: don't parallelize tests by default

Was going to suggest exactly this, but you're way ahead of me. Thanks!


[TeamSpeak3_Transport_Exception] Connection timed out

Hi Taronyuu, Awesome work looks promising!
Let me start of by saying that I am very unfamiliar with Composer...
Here is the issue; I keep getting

[TeamSpeak3_Transport_Exception] Connection timed out

whenever I try to execute a command. (composer install, php artisan migrate...)
Same error on my main computer and on a brand new vm. (Running LAMP on Ubuntu 16.04)
What am I doing wrong?
Best regards,
ChocoboMaster
Update: This is when I'm trying to run it over the network. On the same network, all ports open. I can ping the ts3 server machine and use another ServerQuery library without problems.

Did you add yourself in the whitelist file for your Teamspeak server? query_ip_whitelist.txt

Closing this until a reply or update has been given.

Closing this until a reply or update has been given.


Fail to execute the action 'Clone and Rehydrate' for one archived emulator blob

Storage Explorer Version: 1.22.0-dev
Build Number: 20211009.1
Branch: main
Platform/OS: Windows 10
Architecture: ia32
Emulator Version: Azurite 3.14.0
How Found: Ad-hoc testing
Regression From: Not a regression
Steps to Reproduce

Install and run Azurite 3.14.0 -> Launch Storage Explorer.
Expand 'Local & Attached -> Storage Accounts -> (Emulator - Default Ports)(Key)' -> Blob Containers.
Create a blob container -> Upload one blob -> Update the access tier to 'Archive'.
Right click the blob -> Click 'Clone and Rehydrate'.
Input a valid destination blob name -> Click 'Apply'.
Check whether succeed to execute the action 'Clone and Rehydrate'.

Expected Experience
Succeed to execute the action 'Clone and Rehydrate'.
Actual Experience
Fail to execute the action 'Clone and Rehydrate'.

Additional Context
Error details:
  "name": "TypeError",  "message": "tags.blobTagSet is not iterable".

I'll hide this action in emulator explorers.

The error is about getting tags. Since I have added code to support copying bypassing tags, it is supposed to work in emulator as well. However, I found that it still doesn't https://github.com/Azure/Azurite/issues/1146. I'll disable this action in emulator as planned for now.

Verified on the build 20211102.7. Fixed.(The action is disabled)


Handle special case attributes

fixes #143

[ ] switch to version number for html-parse-string package once that is released, before merging this PR.

The CI isn't able to install from git correctly. This can be tested locally by npm linking html-parse-string into lit-dom-expressions before running npm test.

Any reason to bring regexr in? I know we are talking bytes here but if we didn't need to mess with the regexp much probably isn't worth bringing.

I can remove it
The only reason was regexes being more readable/easy to compose. Initially I thought the problem was there, so it would've been easier to work with, but it ended up being in html-parse-string.
Been imagining a Babel plugin to compile r`` expressions to strings/regexps would be interesting...

PR updated

update html-parse-string, new tests now passing

I'm taking this for a spin in LUME right now

It seems to be that some of the cases that successfully pass through html-parse-string still fail after they pass through lit-dom-expressions.
For example, the following test,
    /** @type {HTMLElement} */
    const el = html`
      <lume-box
        has="shader-material"
        receive-shadow="false"
        size="400 400 400"
        sidedness="double"
        uniforms='{
					"iTime": { "value": 0 },
					"iResolution": { "value": {"x": 0, "y": 0, "z": 0} }
				}'
        vertex-shader="
					varying vec2 vUv;

					void main() {
						vUv = uv;
						gl_Position = projectionMatrix * modelViewMatrix * vec4( position, 1.0 );
					}
				"
        fragment-shader="
					#include <common>

					uniform vec3 iResolution;
					uniform float iTime;

					// The following is the default shader when you start a new shadertoy example.
					// By iq: https://www.shadertoy.com/user/iq
					// license: Creative Commons Attribution-NonCommercial-ShareAlike 3.0 Unported License.

					// BEGIN SHADERTOY CODE {

					void mainImage( out vec4 fragColor, in vec2 fragCoord ) {
						// Normalized pixel coordinates (from 0 to 1)
						vec2 uv = fragCoord/iResolution.xy;

						// Time varying pixel color
						vec3 col = 0.5 + 0.5*cos(iTime+uv.xyx+vec3(0,2,4));

						// Output to screen
						fragColor = vec4(col,1.0);
						//fragColor = vec4(1.0, 0.3, 0.1, 1.0);
					}

					// END SHADERTOY CODE }

					varying vec2 vUv;

					void main() {
						mainImage(gl_FragColor, vUv / 2.0 * gl_FragCoord.xy);
					}
				"
      ></lume-box>
    `;

    expect(el.attributes.length).toBe(7);

fails with 20 instead of 7, but as you can see in the following regexr the attributes are detected fine by the new html-parse-string regex:
https://regexr.com/6pc1b
So, now I need to figure out why lit-dom-expressions is further splitting the attributes.

Found it, and added it to the same PR I have open. Quote characters needed to be escaped into HTML entities: https://github.com/ryansolid/html-parse-string/pull/5/files#diff-d937d19143c8eb7c45d90735ae2a258a0a681496be1259451254beeb64a01fbfR7

Awesome. Thank you.

@ryansolid the version of html-parse-string needs to be updated in this package.json still.

In my other comment I was wondering if you want to just bump html-parse-string to 0.1.0, otherwise we have to also bump it here on every non-breaking change (semver treats all 0.0.x bumps as breaking, but 0.1.x bumps are treated as non-breaking patches).


if 2 hubs are specified, it sees the second one only

https://mastodon.social/users/kevinmarks.atom has
  <link rel="hub" href="https://mastodon.social/api/push"/>
  <link rel="hub" href="https://pubsubhubbub.superfeedr.com"/>

and only the superfeedr one is seen.

The spec says:

If more than one URL is specified, it is expected that the publisher pings each of these URLs, so the subscriber may subscribe to one or more of these.

It was a side effect of this implementation that the second one in the list is used. I changed it to use the first in the list, so at least that should be less unexpected.

It might make sense to test both and report on each - if other subscription
implementations vary which they choose it could make problems with the
other one.
On Thu, 1 Dec 2016, 09:10 Aaron Parecki<EMAIL_ADDRESS>wrote:

The spec says:
If more than one URL is specified, it is expected that the publisher pings
each of these URLs, so the subscriber may subscribe to one or more of these.
It was a side effect of this implementation that the second one in the
list is used. I changed it to use the first in the list, so at least that
should be less unexpected.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/aaronpk/websub.rocks/issues/6#issuecomment-264232500,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAGCwKMjAZizxieSNJJUCBXu_khUXfWYks5rDv9agaJpZM4K_Vad
.

Also, when testing subscribers, see which they choose from multiple choices
and document that. If they are consistent about first/last could change the
spec to match.
On Thu, 1 Dec 2016, 09:17 Kevin Marks<EMAIL_ADDRESS>wrote:

It might make sense to test both and report on each - if other
subscription implementations vary which they choose it could make problems
with the other one.
On Thu, 1 Dec 2016, 09:10 Aaron Parecki<EMAIL_ADDRESS>wrote:
The spec says:
If more than one URL is specified, it is expected that the publisher pings
each of these URLs, so the subscriber may subscribe to one or more of these.
It was a side effect of this implementation that the second one in the
list is used. I changed it to use the first in the list, so at least that
should be less unexpected.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/aaronpk/websub.rocks/issues/6#issuecomment-264232500,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAGCwKMjAZizxieSNJJUCBXu_khUXfWYks5rDv9agaJpZM4K_Vad
.

I'll make it show an option to choose which hub to test when it finds more than one

Good idea: #8


Be more explicit about which crates and features are needed at the Guide chapter of the docs.

Specific Demand
I believe it would make the documentation even better if it was made clear for the reader which crates should be added to the project while following the Guide chapter.
Implement Suggestion
Either add the TOML lines or the cargo add commands needed for the reader to run the code without needing to find out which external crates and features (such as chrono with serde features) to include in the project.

The dependancies are included in the main page of the guide 🙂:
cargo add chrono --features serde
cargo add futures
cargo add reqwest --features json
cargo add serde --features derive
cargo add serde_json
cargo add async_recursion

Dependancies for your renderer of choice are in the getting started section linked near the dependancies.
If you notice any dependancies we missed, we can add those to the guide file in the docsite

I see. I guess I skipped the intro and went straight to the sections. My bad. I believe the list you provided is complete. However, have you considered maybe including the dependencies in the sections as the reader needs them and moving this "all dependencies list" to the "Full Code" section.
To be clear, I think the current structure is great and I'm not very experienced with documentation elaboration and I'm just trying to be useful since I brought it up. I completely understand if you think I'm just being too picky or if what I'm proposing is as good as the current way of listing the dependencies.

I see. I guess I skipped the intro and went straight to the sections. My bad. I believe the list you provided is complete. However, have you considered maybe including the dependencies in the sections as the reader needs them and moving this "all dependencies list" to the "Full Code" section.
To be clear, I think the current structure is great and I'm not very experienced with documentation elaboration and I'm just trying to be useful since I brought it up. I completely understand if you think I'm just being too picky or if what I'm proposing is as good as the current way of listing the dependencies.

That would be great! I generally like all the dependancies at the beginning for single page tutorials so that users don't need to keep switching between the terminal and the rust code (like the liveview guide). But the guide expanded into multiple pages and the dependancies line up nicely into two large chunks:

Chrono + Serde https://dioxuslabs.com/learn/0.4/guide/your_first_component#dynamic-text
Data fetching https://dioxuslabs.com/learn/0.4/guide/data_fetching#defining-the-api

Yeah, that makes sense to me too!

I was not able to find the installation instructions in the 0.5 guide, so I'm a bit lost. Is the issue still ongoing ?

The installation instructions are now in the getting started guide


jellyscrub doesnt work with cloudflared on web

Hello,
Im currently using cloudflared (cloudflare tunnel) as a reverse proxy for my homelab and i having a problem where the scrubing preview on the time line doesnt show on the web client but when using something like jellyfin media player it work so i don't know where the problem come and what can i do about it.
Docker : 23.0.1
Jellyfin : 10.8.9
Jellyfin Media player : 1.9.0
Jellyscrub : <IP_ADDRESS>

Just to confirm, on Jellyfin Media Player you aren't going through cloudflare? If not, is there a reason you aren't accessing the server directly on web but are on JMP?
If JMP is working through cloudflare, then the problem is likely that the client script wasn't successfully injected on web. Could you paste any lines from the log that refer to script injection?

On jellyfin media player im using the url with cloudflared so im using it and it work.
in the logs i found this and I search on google but the other person having the problem when nfo is checked inside the a library but  for me i don't have this so i don't know :
[2023-03-13 19:26:42.373 +01:00] [INF] [1] Nick.Plugin.Jellyscrub.JellyscrubPlugin: Attempting to inject trickplay script code in "/usr/share/jellyfin/web/index.html"
[2023-03-13 19:26:42.374 +01:00] [ERR] [1] Nick.Plugin.Jellyscrub.JellyscrubPlugin: Encountered exception while writing to "/usr/share/jellyfin/web/index.html": "System.IO.IOException: Read-only file system : '/usr/share/jellyfin/web/index.html'
   at Interop.ThrowExceptionForIoErrno(ErrorInfo errorInfo, String path, Boolean isDirectory, Func`2 errorRewriter)
   at Microsoft.Win32.SafeHandles.SafeFileHandle.Open(String path, OpenFlags flags, Int32 mode)
   at Microsoft.Win32.SafeHandles.SafeFileHandle.Open(String fullPath, FileMode mode, FileAccess access, FileShare share, FileOptions options, Int64 preallocationSize)
   at System.IO.Strategies.OSFileStreamStrategy..ctor(String path, FileMode mode, FileAccess access, FileShare share, FileOptions options, Int64 preallocationSize)
   at System.IO.Strategies.FileStreamHelpers.ChooseStrategy(FileStream fileStream, String path, FileMode mode, FileAccess access, FileShare share, Int32 bufferSize, FileOptions options, Int64 preallocationSize)
   at System.IO.StreamWriter.ValidateArgsAndOpenPath(String path, Boolean append, Encoding encoding, Int32 bufferSize)
   at System.IO.File.WriteAllText(String path, String contents)
   at Nick.Plugin.Jellyscrub.JellyscrubPlugin..ctor(IApplicationPaths applicationPaths, IXmlSerializer xmlSerializer, ILogger`1 logger, IServerConfigurationManager configurationManager)"

So because of your setup jellyscrub is unable to automatically inject the web script and you must manually do so into /usr/share/jellyfin/web/index.html as described in the installation instructions on this GitHub.

I'm really sorry I found the problem,
I'm using intro-skipper and I didn't know that with the new version I can delete the modified index.html provided by the plugin and that's why the plugin was not able to inject into and after clearing my cache everything works absolutely fine, sorry for taking your time and have a really good day.


Fix python bindings build on aarch64

Solution to issue cannot be found in the documentation.

[X] I checked the documentation.

Issue
.
Installed packages
.

Environment info
.

See https://github.com/conda-forge/bipedal-locomotion-framework-feedstock/pull/85#issuecomment-2366749638 .


CUDA variant optimised beyond k-caching

Optimisation beyond k-caching:

buffering variables
pipelined global-to-shared memory copies overlapped with compute, TMA loads

NVIDIA blog post: Controlling Data Movement to Boost Performance on the NVIDIA Ampere Architecture

likely still potential for further optimisation ...
Implemented with input from @lukasm91

k-caching vs "opt"-variant performance (compiled with nvhpc 22.11 and executed with 1 262144 128):

double-precision

k-caching: 600 - 700 GF/s
"opt": 700 - 850 GF/s

single-precision

k-caching: 2300 - 3000 GF/s
"opt": 2700 - 5000 GF/s

This is really impressive!

[ ]  I left some comments on the CMake side, which would be good to take care of
[ ]  Tests are currently failing, not sure why
[ ]  Please describe the new variant in the README, also adding some details on what constitutes the optimisation

Tests are failing due to the compute capability?!
/home/runner/work/dwarf-p-cloudsc/dwarf-p-cloudsc/nvhpc-install/Linux_x86_64/21.9/cuda/11.4/include/cuda/std/detail/__atomic:11:4: error: #error "CUDA atomics are only supported for sm_60 and up on *nix and sm_70 and up on Windows."
[2891](https://github.com/ecmwf-ifs/dwarf-p-cloudsc/actions/runs/12279542957/job/34263839987?pr=104#step:11:2892)
   11 | #  error "CUDA atomics are only supported for sm_60 and up on *nix and sm_70 and up on Windows."
[2892](https://github.com/ecmwf-ifs/dwarf-p-cloudsc/actions/runs/12279542957/job/34263839987?pr=104#step:11:2893)
      |    ^~~~~
[2893](https://github.com/ecmwf-ifs/dwarf-p-cloudsc/actions/runs/12279542957/job/34263839987?pr=104#step:11:2894)
make[2]: *** [cloudsc-dwarf/src/cloudsc_cuda/CMakeFiles/dwarf-cloudsc-c-cuda-opt-lib.dir/build.make:95: cloudsc-dwarf/src/cloudsc_cuda/CMakeFiles/dwarf-cloudsc-c-cuda-opt-lib.dir/cloudsc/cloudsc_c_opt.cu.o] Error 1

Therefore, I guess we have to disable the new CUDA optimised variant for testing?!

It shouldn't be run but it should be built.
Try setting the CUDA architecture to 80 in the Github toolchain file: https://github.com/ecmwf-ifs/dwarf-p-cloudsc/blob/main/arch/toolchains/github-ubuntu-nvhpc.cmake

My hat's off 🙇

A peculiar addition to this: @MichaelSt98's CMake changes make sure to now apply --ptxas-options=-O3 -O3, which seems to tip the double-precision performance over the (effective) 1TF/s barrier for throughput, and I'm seeing about ~4.3TF/s in single-precision.


Error matching route rules. TypeError: Cannot read properties of undefined (reading 'entries')

🐛 The bug
[nuxt] Error matching route rules. TypeError: Cannot read properties of undefined (reading 'entries')
NOTE! In production mode using electron builder and installing the electron app it works without errors. The problem only effects dev mode AFAIK.

🛠 To reproduce
https://stackblitz.com/github/nuxt-modules/ionic/tree/main/playground
🌈 Expected behaviour
No errors should occur.
ℹ Additional context
------------------------------
- Operating System: Windows_NT
- Node Version:     v22.1.0
- Nuxt Version:     3.12.3
- CLI Version:      3.12.0
- Nitro Version:    2.8.1
- Package Manager<EMAIL_ADDRESS>- Builder:          -
- User Config:      ssr, router, css, modules, electron, ionic, vite, nitro, devtools
- Runtime Modules<EMAIL_ADDRESS><EMAIL_ADDRESS>- Build Modules:    -
------------------------------

Current workaround is to disable the app manifest it is trying to consume
export default defineNuxtConfig({
  experimental: {
    appManifest: false,
  },
});


Otherwise if we have an IM action in flight at shutdown time we will
effectively leak the object managing it, and after a few
shutdown/restart cycles for the stack not be able to send any more IM
actions.
Problem
If I shut down and restart DeviceController a few times, and the shutdowns happen to come while there is a CommandSender outstanding (in that we sent a command and a response has not come back), then eventually we run out of CommandSenders.
Change overview
Make sure to shut down CommandSenders when we shut down DeviceController.
Testing
Manual testing with a bug added to chip-all-clusters-app that causes it to shut down the DeviceController while CommandSenders are life.  Will try to add some automated tests too.

@andy31415 @Damian-Nordic @LuDuda ?


Update SQLDatabaseChain for intermediate SQLResults

Description:
In the call method within the SQLDatabaseChain class:
Amend the intermediate steps dictionary to include a new key, say sql_result, where the SQL results at different stages would be saved.
During the SQL execution step, save the SQL result into the sql_result key in the intermediate steps dictionary, similar to how sql_cmd is being saved currently.
Issue: #10500
Tag maintainer: zzfab

Gonna close this without a response above, and let me know if you'd like to reopen!


Get-AzTableTable Fails

Describe the bug
Attempt to execute Get-AzTableTable as follows:
$resourceGroup = "WVD-USCENT-RG"
$tableName = "WVD-AI-Storage"
$storageAccount = "cosmos-wvd-sktuoy7"
$table = Get-AzTableTable -resourceGroup $resourceGroup -TableName $tableName -storageAccountName $storageAccount
Error Message
Exception: /Users/gj/.local/share/powershell/Modules/AzTable/2.0.3/AzureRmStorageTableCoreHelper.psm1:133:3
Line |
133 |          throw "Invalid table name: $TableName"
|          ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
| Invalid table name: WVD-AI-Storage
PowerShell Version
Include here the full output of the following command line:
$PSVersionTable

Complete list of Azure related Powershell modules
ModuleType Version    PreRelease Name                                PSEdition ExportedCommands

Script     1.9.0                 Az.Accounts                         Core,Desk {Disable-AzDataCollection, Disable-AzContextAutosave…
Script     4.1.0                 Az.Compute                          Core,Desk {Remove-AzAvailabilitySet, Get-AzAvailabilitySet, Ne…
Script     0.1.6                 Az.CosmosDB                         Core,Desk {Get-AzCosmosDBSqlContainer, Get-AzCosmosDBSqlContai…
Script     0.1.0                 Az.DesktopVirtualization            Core,Desk {Disconnect-AzWvdUserSession, Get-AzWvdApplication, …
Script     1.3.2                 Az.LogicApp                         Core,Desk {Get-AzIntegrationAccountAgreement, Get-AzIntegratio…
Script     2.0.1                 Az.Resources                        Core,Desk {Get-AzProviderOperation, Remove-AzRoleAssignment, G…
Script     2.2.0                 Az.Storage                          Core,Desk {Get-AzStorageAccount, Get-AzStorageAccountKey, New-…
Manifest   2.0.3                 AzTable                             Desk      {Add-AzTableRow, Get-AzTableRow, Get-AzTableRowAll, …
Binary     <IP_ADDRESS>              AzureAD                             Desk
Code Snippet
See Description.
More information

OS: macOS Catalina 10.15.5
Visual Studio Code 1.46.1

Additional context
On  further investigation, it doesn't appear to recognise the new "Azure Cosmos DB account" type...only the old "Storage account" type.
I can quite happily create the Table with cmdlet in Az.CosmosDB.

Can you please try a table name without - or _, like WVDAIStorage

Also, this module doesn't support CosmosDB table yet. Due to lack of resource, we don't have eta yet.

Good afternoon Paul,
The last update lead me to believe you were supporting CosmosDB tables.  Sorry to hear this is not the case.
I'm a little surprised on lack of resource...though one or more were MSFT?
The context in which I need my PowerShell code to run is in Azure Automation Account RunBook.
I had considered whether it would be possible to simply embedded c# in PowerShell to provide access to the .Net API, then though how one would be able to reference the SDK in Azure Automation alluded me.
Any pointers, most welcome.
Kindest Regards, Graham.


Measure times of killed and closed transactions

Instead of just "Completed" or "Aborted" stats, exported the conclusion
to match per-user transaction stats.
BUG=69414609

Change is good, but tests need to be fixed.

Updated tests.

LGTM


why i need to make multiple unnecessary saves to correctly run Makefile ?? (C++)

first of all sorry for my bad english, im Mexican.
I always used IDEs to make my codes so ive never used a Makefile before. i dont know why but to run my program successfully i need to make a save on every file in my project folder (even if i didnt modified anything of the code in the file) and i dont like that.
i want to only save the files that ive modiffied to see all the changes in my program when i running it.
heres my Makefile:
output: list.o main.o menu.o song.o
g++ list.o main.o menu.o song.o -o actividad06
list.o: list.cpp list.h
g++ -c list.cpp
main.o: main.cpp
g++ -c main.cpp
menu.o: menu.cpp menu.h
g++ -c menu.cpp
song.o: song.cpp song.h
g++ -c song.cpp
clean:
rm *.o actividad06
am i doing something wrong or theres a way to prevent this??

Thanks for reaching out!
I would recommend reading more about make files and how C++ programs are compiled and if you are still unsure about what is going on there try to find a place that is meant to help out with C++ questions.
Because we treat our issues list as the Atom team's backlog, we close issues that are questions since they don't represent a task needing to be completed. For most questions about Atom there are a lot of options, check out:

User Documentation — The Atom Flight Manual
Developer Documentation — Atom API Documentation
FAQ — The Atom FAQ on Discuss
Message Board — Discuss, the official Atom and Electron message board
Chat — Join the Atom Slack team

On Discuss and in the Atom Slack team, there are a bunch of helpful community members that should be willing to point you in the right direction.


No serializer registered for ProvidedDependency

No serializer registered for ProvidedDependency
I am getting warnings when running yarn dev with this plugin enabled
<w> [webpack.cache.PackFileCacheStrategy] Skipped not serializable cache item 'Compilation/modules|/Users/tiero/projects/prem/dev-portal/node_modules/webpack/hot/log.js': No serializer registered for ProvidedDependency
<w> while serializing webpack/lib/cache/PackFileCacheStrategy.PackContentItems -> webpack/lib/NormalModule -> Array { 12 items } -> ProvidedDependency

Your Environment

docusaurus

"@docusaurus/core": "2.4.1",
"@docusaurus/preset-classic": "2.4.1"

docusaurus-openapi-docs

"docusaurus-plugin-openapi-docs": "v2.0.0-beta.3",
"docusaurus-theme-openapi-docs": "v2.0.0-beta.3",

You can try with tha main branch
https://github.com/premAI-io/dev-portal

Hi @tiero, typically these show up whenever a given page throws an error that would typically appear in the browser console error logs. That said, we've tried to address such error since your issue was originally opened. Please try a more recent release and feel free to reopen this issue if the problem still exists. Thanks!


Relating an entry from any Section

I've been using Super Table to great affect within various Matrix Blocks, and I've have already used it with related entries where I already know the section, but can't figure out how to get it working when the user can select from entries across multiple sections.
This is basically what I need to update:
{% set link = craft.entries.section(?????).relatedTo(row) %}

Actually, ignore me. I've just realised I can leave the section blank.

Glad you got it sorted! 😃


Package guzzle/guzzle is abandoned ?

➜  me git:(master) ✗ composer update
Loading composer repositories with package information
Updating dependencies (including require-dev)
Package operations: 0 installs, 0 updates, 0 removals
Package guzzle/guzzle is abandoned, you should avoid using it. Use guzzlehttp/guzzle instead.
Writing lock file
Generating autoload files

Also Package omnipay/omnipay is abandoned, you should avoid using it. Use league/omnipay instead.


collect features for a single cow

Data preprocessing on the raw-traffic data table
Managed to locate the most recent traffic event to the milking event and calculate the time difference(T2-T1)
T1: Entry time into the Mjolkfalla
T2: Entry time into the milking robot

To do:

Convert the notebook code to runnable python script
Need to figure out how to fill in the NaN value in DayInMilk
(thoughts: use a recursive function to fill missing values or simply drop the nan values)


Fix reference to inputdataset when calculating total size

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

2 of 2 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 94.299%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
560

Relevant Lines:
585

💛  - Coveralls


functional-tester: put large keys

For testing writes that must span multiple pages. (Cf. #6191)

lgtm.


Pattern: Callout with data pattern with feature flag - limited scope Prod QA testing

User Story

As a [user role below]:
developer using the ibm.com Library Callout with data

I need to:
have an early version of the pattern or component that has been tested for Tier 1 browsers and platforms

so that I can:
use the pattern on a web page or in a cms template to gather design metrics to validate or change the existing design

Additional information

Browser Stack link
Browser Standard
Browser versions to be tested:  Tier 1 browsers will be tested with defects created as Sev 1 or Sev 2.
Platforms to be tested, by priority:  1)  Desktop  2)  Mobile  3)  Tablet
Mobile & Tablet iOS versions: 13.1 and 13.3
Browsers to be tested:  Desktop: Chrome, Firefox, Safari, Edge, Mobile: Chrome and Safari  Tablet: Chrome and Safari
Creating a QA bug
See the Epic for the Design specs information.  Functional specs are not created for patterns behind a feature flag
Dev issue (#2823)
** Once development is finished the updated code is available on the Canary URL for testing.**

Acceptance criteria

[ ] All Tier 1 browser versions are tested
[ ] All Tier 1 operating systems are tested
[ ] All Tier 1 devices are tested
[ ] Defects are recorded and retested when fixed

@Praveen-sr @chetankan Here is the testing issue for the new Callout with data pattern where the limited testing will be performed.  I think I captured all of the changes we talked about in the meeting earlier today.  Let me know if any changes are needed.

@Praveen-sr @chetankan It seems that there has been a change of plans for Drupal.  The testing of these patterns is no longer urgent and we may do Full Prod QA testing.  We will make a decision later in the week.

@Praveen-sr @chetankan This pattern is ready for testing.  This pattern is released behind a feature flag, so limited testing should be performed.  Tier 1 browsers, O/S and platforms.  No accessiblity.  Since this testing is considered a sprint must have the testing must be finished at the beginning of next week.  Thanks.
The canary URL should be used for the testing.

Function testing is complete.


🐛 Do not start supabase autorefresh process

Should fix https://github.com/laminlabs/lamindb-setup/issues/747

Awesome, Sergei! Can you please make a big announcement on lamindb-progress and explain how this resolved the hanging?

Lets wait a bit and become really sure that this solved the problem.