Kuma not able to get transformed request from Azure API Management

⚠ Please verify that this question has NOT been raised before.

[X] I checked and didn't find similar issue

🛡 Security Policy

[X] I agree to have read this project Security Policy

📝 Describe your problem
Hi, I am trying to develop custom API for Ansible REST API and Azure API Management. I kinda figured everything out, but when it came to the final step - which was the whole point of doing this.
But after I implemented some outbound transformations in Azure APIM, Kuma started to throw 500 codes for the API. When I remove the set-variable, I am able to get 200 from the API, but without the important json query that tells the policy whether to set 200 or 400 response.
But my main point is, it's only Kuma that throws the 500 error. Postman is pretty OK with the output and works fine, Azure test module in API also prints everything fine, including trace.
Here is my policy code for the API:

<policies>
    <!-- Throttle, authorize, validate, cache, or transform the requests -->
    <inbound>
        <base />
    </inbound>
    <backend>
        <base />
    </backend>
    <!-- Customize the responses -->
    <outbound>
        <base />
        <set-header name="Content-Type" exists-action="override">
            <value>application/json</value>
        </set-header>
        <set-variable name="statusValue" value="@(context.Response.Body.As<JObject>()["summary_fields"]["last_job"]["status"].ToString().Trim().ToLower())" />
        <choose>
            <when condition="@(context.Variables.GetValueOrDefault<string>("statusValue") == "successful")">
                <return-response>
                    <set-status code="200" reason="OK" />
                    <set-body template="liquid">
{
"status": "Workflow run went OK"
}
                    </set-body>
                </return-response>
            </when>
            <when condition="@(context.Variables.GetValueOrDefault<string>("statusValue") == "failed")">
                <return-response>
                    <set-status code="400" reason="Workflow Failed" />
                    <set-body template="liquid">
{
"status": "Workflow run failed"
}
                    </set-body>
                </return-response>
            </when>
            <!--<otherwise>
                <return-response>
                    <set-status code="500" reason="Request Failed - Bad API" />
                </return-response>
            </otherwise>-->
        </choose>
    </outbound>
    <!-- Handle exceptions and customize error responses  -->
    <on-error>
        <base />
    </on-error>
</policies>

And basically when I remove these policies, Kuma is able to retrieve 200, but its full json with a lot a data, which is useless.
I also tried JSON Query with Kuma, but that doesn't work either.
My point is, that when I send request with Kuma I get 500 and error message from Azure AppInsights below.
When I sent the same request from Azure or Postman, I get 200 and desired response.
So is it possible, that Kuma somehow cannot get transformed json? Or where can be the problem, obviously the policy code is pretty basic hence I totally don't know this XML language, but if it was fundamentally wrong, wouldn't also podman throw 500?
Thank you for every advice or notice.
📝 Error Message(s) or Log
Expression evaluation failed. The message body is not a valid JSON. Unexpected character encountered while parsing value: <. Path '', line 0, position 0.
   at Newtonsoft.Json.JsonTextReader.ParseValue()
   at Microsoft.WindowsAzure.ApiManagement.Proxy.Runtime.Configuration.Models.SyncOnlyDepthTrackingJsonTextReader.Read() in D:\a\1\s\Proxy\Gateway.Pipeline.IO\SyncOnlyDepthTrackingJsonTextReader.cs:line 62
   at Newtonsoft.Json.Linq.JObject.Load(JsonReader reader, JsonLoadSettings settings)
   at Gateway.Pipeline.IO.JsonConverter.Convert[T](Stream stream, Encoding encoding, ILog log, Object conversionSettings) in D:\a\1\s\Proxy\Gateway.Pipeline.IO\MessageConverters\JsonConverter.cs:line 57
   at Microsoft.WindowsAzure.ApiManagement.Proxy.Gateway.MessageBody.As[T](Boolean preserveContent) in D:\a\1\s\Proxy\Gateway.Pipeline.IO\MessageBody.cs:line 83

🐻 Uptime-Kuma Version
1.23.13
💻 Operating System and Arch
Azure B1 ASP
🌐 Browser
Safari 17.5 (196<IP_ADDRESS>.6)
🖥 Deployment Environment

Runtime:  Docker
Database:
Filesystem used to store the database on: Linux ASP
number of monitors: 3

I have never heard of that, nor worked with it.
The error message states that the first char is <...
=> have you checked that comments like <!--<otherwise> are supported?

Hi, not sure if I understand correctly, but yes, <!-- comment is native to this horrible Azure XML language.. I can remove those, but wouldn't this also be shown with Postman for example? I still quite don't understand where is the difference between Postman and Kuma processing response body.

True, those would be the same, but maybe that case is triggered due to different request headers like Accepts (See #4491)
Have you specified all relevant headers?
Maybe azure does some magic to requests? (You would need to ask in their forum or support. We spcecify this)
I am not quite sure if the outgoing body from azure or the incoming body to azure is causing the error you are seeing. Please recheck if the Content-Type and body content is set in postman and kuma identically.

You can likely also debug the policy via https://learn.microsoft.com/de-de/azure/api-management/trace-policy

Ok thank you for your advices.
A revisioned the code and made some changes and came to even more weird outcome. The body was probably indeed in invalid JSON, because now I changed the policy to take whole body in json and extract the object value I needed. In trace I see that all these operations went well, taking full body, then transforming to JSON and then extracting the particular value and successfully did the condition.
BUT
In podman I got 200 again, no problem, I also added Content-type: application/json to response header.
Kuma again throws 500, but now I don't see any error regarding the JSON body or anything else in the Azure log. Now it just shows that backend answers 200 and back from GW to Kuma it throws 500. Without any obvious reason as before.
So now I am kinda blind with what is the reason of 500 response from Kuma. Is there any place I can see it directly in Kuma? In log I only see that it returned 500 but no error message.
Thank you.

And the Accepts header?

Ok I got you now, I have set same Accept header as Kuma for Podman and it is indeed the problem. I now get the same response, so I changed the policy to send response in text/html and I get 200 on Kuma too. The problem now is, that with text I am not able to perform the condition that takes the value from the json response and decide whether to send 200 or 400.  Even when the status is failed now, I get 200 with full html page response :/

Ok so I got it fixed!  :D
All I had to add was Content-Type: application/json to Kuma headers. That's it.
Now I am able to get transformed json responses.
Somehow it doesn't work when you specify Accept headers in request like: / or just text/html or even combined application/json,text/html. That didn't work even with Postman.
But I sorted it out and now it works just as it should with Kuma.
Still thank you very much with every advice and super fast responses. I really appreciate it.


RowClick not working on DataGrid as documented, error: cannot convert from 'method group' to 'EventCallback'

Bug type
Component
Component name
MudDataGrid
What happened?
Implementing a RowClick handler, like specified in the documentation, results in the error: cannot convert from 'method group' to 'EventCallback'
Below is a simple repro using a copy/paste of the Grouping example from the docs, with added RowClick handler:
https://try.mudblazor.com/snippet/maQdYIYLfZVMEZKM
Expected behavior
Adding RowClick handler to MudDataGrid should work
Reproduction link
https://try.mudblazor.com/snippet/maQdYIYLfZVMEZKM
Reproduction steps

Click TryMudBlazor link
Click Run
See failure

Relevant log output
No response
Version (bug)
6.2.0
Version (working)
unknown
What browsers are you seeing the problem on?
Microsoft Edge
On what operating system are you experiencing the issue?
Windows
Pull Request

[ ] I would like to do a Pull Request

Code of Conduct

[X] I agree to follow this project's Code of Conduct

What error?
A error cause a crash.
What?
A MudDataGrid is an handler.

What error?
A error cause a crash.
What?
A MudDataGrid is an handler.

If this is intended to be a response to my issue, I don't know what to make of it.

Hello there! The issue is that the RowClick event uses generics and you need to specify the type. To fix this, add T="Element" to your MudDataGrid component like this: <MudDataGrid T="Element" />. This is how razor works and it's by design.

Thanks @ScarletKuro, that helped me out.

Thanks @ScarletKuro, that helped me out.

You have helped the Sausage King of Chicago, good on you!
Abe-Froman, that username makes me laugh every time you post.

What can be done to solve this problem?

What can be done to solve this problem?☹

Have you read the comments?

What can be done to solve this problem?☹

 private void OnRowClick(DataGridRowClickEventArgs args)


Integrate with Azure.Identity

SqlClient already allows an access token to be set, either in the connection string or programmatically on SqlConnection. However, the actual work of retrieving (and refreshing, and caching) that token is left to the user. Azure.Identity handles the retrieving of tokens, and SqlClient could integrate with it
This was originally requested by @sopelt in https://github.com/dotnet/SqlClient/pull/730#issuecomment-700566412. @cheenamalhotra, below that you made the point that Azure.Identity doesn't support .NET Framework like SqlClient. However, wouldn't it be possible to do this integration for .NET Standard 2.0, allowing .NET Core users to take advantage of this?
Note that if this is done, for good perf SqlClient would be responsible for caching the access token for a delay configured by the user (see https://github.com/dotnet/efcore/issues/13261#issuecomment-714923330).
/cc @ajcvickers

Do any other ADO providers provide this level of integration with federated identity?

Hi Shay,
Ack: We'll update here after a more concrete design decision is made.

@Wraith2 I'm not sure... But I'm also not aware of a credential rotation scheme such as the one SQL Azure does here - especially where old connections which validated successfully are broken after a while (see #767).

@roji is that really a big issue if you just you connection resiliency (which you always should with Azure SQL DB anyway) ?

Looking into it AAD looks like it uses the same model as openid and if that's the case the management of the access token it's revocation, refresh and lifetime should be managed above the driver layer.

@ErikEJ this isn't a transient error, since the user must start using a new access token (ideally before any exception occurs). In other words, it's not enough to just retry the operation - you must set SqlConnection.AccessToken to a new value (which must be fetched and cached appropriately too). It's true that the user could handle this by using the same token until the exception occurs, and use that as a trigger to rotate the token - though that's weird; the user has all the information to know beforehand that the previous trigger is about to get invalidated and could avoid the exceptions.
Another possible point is that I'm not sure access token authentication is only for Azure SQL - maybe it can be used with regular SQL Server too (on-premises or cloud-hosted)? If so, then you're much less expected to use connection resiliency in general.
@Wraith2

if that's the case the management of the access token it's revocation, refresh and lifetime should be managed above the driver layer.

Why do you say that, is there some rule somewhere saying where this should managed? Also, OpenID is a general auth framework that's relevant to a wide multitude of services, whereas here we're talking about something that's specific to SQL Azure. Any reason to require all SQL Azure to go through the same extra hoops rather than just doing it for them in the driver for that specific service?

Another possible point is that I'm not sure access token authentication is only for Azure SQL - maybe it can be used with regular SQL Server too (on-premises or cloud-hosted)? If so, then you're much less expected to use connection resiliency in general.

@roji I've definitely heard talk of federated authentication potentially coming to on-premise servers. So I've been trying to keep that in mind when directing feature work in the drivers.

@roji @Wraith2
I'll try to simply thoughts as per current behavior.
If customer provides access token via "Access Token" connection property, then it's validity is in the hands of user (who is also the token provider). So whenever customer opens a new connection, it is required to provide a "valid" access token. Driver does not intercept this access token and passes the same directly to server, letting customer applications take full control of it.

It's true that the user could handle this by using the same token until the exception occurs, and use that as a trigger to rotate the token - though that's weird; the user has all the information to know beforehand that the previous trigger is about to get invalidated and could avoid the exceptions.

Usually customers do this by capturing "Expiry Time" and comparing it before passing the token to client applications.
The token is refreshed/renewed with a buffer of ~5 mins to token expiry, depending on client's preferences.
On the other hand, if user only provides credentials/ auth mode etc. and it is the driver that requests token from MSAL/Azure.Identity (third-party libs), that is where SqlClient works along with libraries to "maintain" this access token that's owned by the driver. This token is cached/refreshed/renewed as needed. Most of the caching logic is done in third-party libs, but SqlClient also has a tiny cache when it comes to acquiring tokens for new connections in the same pool.

Sound great @cheenamalhotra. One small note on what you said:

Most of the caching logic is done in third-party libs, but SqlClient also has a tiny cache when it comes to acquiring tokens for new connections in the same pool.

For Azure.Identity specifically, it seems that the responsibility of caching the token is with the user of the library - it's a perf anti-pattern to just call GetTokenAsync every time you want to open a connection, and expect Azure.Identity to cache internally (see https://github.com/dotnet/efcore/issues/13261#issuecomment-714923330). If SqlClient integrates with Azure.Identity, then SqlClient becomes the "user", so if I understand everything correctly, it would have to cache the token internally.

@roji
As of now SqlClient relies on MSAL (Microsoft.Identity.Client) to cache access tokens, the current implementation of ActiveDirectoryAuthenticationProvider in the driver does not cache access tokens. As I said, we only have a small cache when it comes to pooled connections, but that's outside the ActiveDirectoryAuthenticationProvider.
For perf reasons, I agree it may be possible to cache tokens in the driver, but honestly none of the SQL Drivers do that actually. Also as Azure SDK and MSAL libraries have advanced their caches, there're less network calls made anyways, so I'm also not sure how much benefit we will get from implementing our own cache.

@cheenamalhotra I made that comment on the necessity of caching the results of GetTokenAsync from Azure.Identity because I just had that discussion with @schaabs (quoted in https://github.com/dotnet/efcore/issues/13261#issuecomment-714923330), that's their recommendation. But we may be discussing different things.

@roji I'd also recommend also syncing with Azure SDK teams to understand how Azure Identity is intended to be used by their client applications and whether or not caching is required. e.g. This blog post is very helpful for Azure Client apps: Best practices for using Azure SDK with ASP.NET Core that recommends using "DefaultAzureCredential" which also internally tries to fetch token from SharedTokenCacheCredential that contains token cache. There are also customization options you can use to take benefit from this cache, i.e. DefaultAzureCredentialOptions
Now these are definitely password-less solutions and new recommendations and do not offer everything/every mode of token authentication. But maybe you don't even need that in EF Core and could only use this to provide token implicitly to SqlClient. If at all you need a cache, you may think about that but I don't feel that's highly essential given there is that capability from Azure.Identity already.
If users want to go for password/service principal modes, they could directly specify that in SqlClient connection string and why needing that support from EF Core. Also there's a possibility that these modes will come to SqlClient too in future, so doing a lot of work in EF Core doesn't seem very valuable.

Additional notes:
I think you're discussing correct topics there, I agree ManagedIdentityCredential doesn't make use of cache so that's something to think about. I also know Azure's App Authentication library caches tokens from Managed Identity for 24 hours (fixed lifetime) and uses the same for new requests. You may wanna check with them to understand it's pros and cons too.

I think we're saying the same thing. SqlClient doesn't seem to have any issue at the moment with regards to SqlConnection.AccessToken - it's a relatively low-level API that makes it the user's concern to fetch, cache and refresh the access token. So you don't have much to worry about.
If, however, you do decide to expose higher-level integration with Azure.Identity, in order to remove that burden from the user and make it more straightforward to use access tokens, then at least with Azure.Identity it would be necessary to implement some form of simple caching (according to the info I got). Ideally, the user just configures a TokenCredential and a refresh interval, once at startup, and SqlClient takes care of all the rest (hopefully I'm getting things right).
I don't know about other authentication methods and libraries - some may be indeed doing their own caching fully internally.

As a consumer, what I'd like to be able to do is use the unified Azure.Identity library
just like I can with the Key Vault V4 SDK, and other recently 'Azure-SDK standardized' libraries,
and not to have to think about caching or anything whether using managed identity creds, localdev, etc:
https://www.rahulpnath.com/blog/defaultazurecredential-from-azure-sdk/

Hi @cheenamalhotra, hi @roji,
I hope you'll continue your discussion and find a way to combine Azure.Identity and SqlAuthenticationProvider in some way. I appreciate all the work in these areas and think there are lots of customers on Azure that would benefit from a first class solution. Right now (with connection string support for AAD integrated/interactive/MSI) many scenarios are solvable quite well but some more complex ones (e.g. home tenant not the same as resource tenant - does that work? integrated/cached authentication with non-federated/pure AAD accounts) would really benefit from the added flexibility.
I know going towards a solution as proposed in https://github.com/dotnet/SqlClient/pull/730#issuecomment-705307179 should not be too bad but if every other project needs to implement that ...

Hi @cheenamalhotra, hi @roji,
I hope you'll continue your discussion and find a way to combine Azure.Identity and SqlAuthenticationProvider in some way. I appreciate all the work in these areas and think there are lots of customers on Azure that would benefit from a first class solution. Right now (with connection string support for AAD integrated/interactive/MSI) many scenarios are solvable quite well but some more complex ones (e.g. home tenant not the same as resource tenant - does that work? integrated/cached authentication with non-federated/pure AAD accounts) would really benefit from the added flexibility.
I know going towards a solution as proposed in https://github.com/dotnet/SqlClient/pull/730#issuecomment-705307179 should not be too bad but if every other project needs to implement that ...


[Question] error while compiling png files to avif

I get this error when I want to compile png files to avif.
  public/dist/assets/logo-1wwMKIqh.png
   └─ public/dist/assets/logo-1wwMKIqh.png.avif                    ERROR  Error processing file:
/home/developer/projects/VT-68944/node_modules/@vheemstra/avifenc-bin/vendor/avifenc: /lib/x86_64-linux-gnu/libc.so.6: version `GLIBC_2.32' not found (required by /home/developer/projects/VT-68944/node_modules/@vheemstra/avifenc-bin/vendor/avifenc)
/home/developer/projects/VT-68944/node_modules/@vheemstra/avifenc-bin/vendor/avifenc: /lib/x86_64-linux-gnu/libc.so.6: version `GLIBC_2.34' not found (required by /home/developer/projects/VT-68944/node_modules/@vheemstra/avifenc-bin/vendor/avifenc)

Do I need to install something to use this module?

It looks like, my Linux is just too old.


Change name from factory to workspace

Codecov Report

Merging #40 (dff56a9) into main (eaf5503) will increase coverage by 0.00%.
The diff coverage is 97.42%.

@@           Coverage Diff           @@
##             main      #40   +/-   ##
=======================================
  Coverage   98.37%   98.38%
=======================================
  Files          38       39    +1
  Lines        5166     5186   +20
=======================================
+ Hits         5082     5102   +20
  Misses         84       84

Flag
Coverage Δ

unittests
98.38% <97.42%> (+<0.01%)
:arrow_up:

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

Impacted Files
Coverage Δ

pDESy/model/base_team.py
97.40% <ø> (ø)

pDESy/model/base_worker.py
100.00% <ø> (ø)

pDESy/model/task.py
100.00% <ø> (ø)

tests/model/test_base_workflow.py
100.00% <ø> (ø)

pDESy/model/base_project.py
94.63% <89.47%> (+0.14%)
:arrow_up:

pDESy/model/base_organization.py
96.72% <96.07%> (ø)

pDESy/model/base_component.py
95.27% <100.00%> (ø)

pDESy/model/base_facility.py
100.00% <100.00%> (ø)

pDESy/model/base_priority_rule.py
100.00% <100.00%> (ø)

pDESy/model/base_product.py
99.46% <100.00%> (ø)

... and 13 more

Continue to review full report at Codecov.

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

Oh. I mistake the branch for this merging.


Notes come back from the dead.

Describe your problem here.
I don't if this is a bug or I am being dumb but for some reason when I zoom in or out the notes that I removed come back after I zoom in or out. So is this a bug or a feature?
Are you modding a build from source or with Lua?
Lua
What is your build target?
Windows x64
Did you edit anything in this build? If so, mention or summarize your changes.
No response

I was not modding or anything I was just messing with the chart editor then this random thing happened

https://github.com/JordanSantiagoYT/FNF-PsychEngine-NoBotplayLag/assets/162759283/02d82e0e-f0ac-4ab2-8516-5c44de72b462

it’s a bug I’ll try to fix

yes, I am on the latest version. It first happened on verison 1.20.0.


Hello, I'd like to contribute

Hello I'm Pach,
First of all, thank you so much for your support to the firebase community. I've recently used your packages flutter_firebase_chat_core and flutter_chat_types. The problem I had was, I already had a users collection with my own configured params on what's on the firestore document. I had to fork those 2 repositories and changed the params to match the ones on my firebase users document. I'd like to get your opinion if you're open to the idea of me contributing to find a solution for this issue. No strings attached, just a way of saying Thank you.

Hi @pacholoamit
Thanks for the feedback :) That's actually something I realized after a release "what if somebody already has a users table", and indeed somebody has :) Yes, we are open to any contributions, we can start with flutter_chat_types and discuss what's the possible solution we might have here. Let me know your thoughts, how can we combine our User type with any different User type? For sure we will need to have avatarUrl (we are thinking of changing this to imageUrl) firstName, lastName and our id (which is actually a UID from Firebase Auth) because pretty much everything depends on those values, including Chat UI and id as UID is very important to have correct Security Rules. Maybe we can just rename the users table and I recently have added a metadata map to the user which can contain additional user fields?

Hmmm. Yeah. I agree and most of the changes I made pertained to variable names. Instead of avatarURL, I changed the variable to photoURL. Hmmm. I'll see what I can do about this. Usually on a production firestore collection, apps already have their own Users collection and documents containing the values your library requires albeit named differently.

 IDs are what's I'm worried about, because as you said, production apps usually have their own IDs and those might not be UIDs which will totally break security rules, like completely :) Do you think we can have 2 different tables, one for customers users and one for chat users? And then you can save IDs as pointers to each table so you can find a chat or customers's user as needed?

It is possible but it would not be efficient though, to have 2 collections to store similar user data. I was going to do that initially though but it would be confusing for me and I'd have to Refactor a lot of the app that's already been built that uses user data. I don't believe users will have at any point have the same ID, at least the doc ID would be unique. When I checked chat_core you're already referencing the doc.id, yes? In my case, I just use the value of the uid given from firebase after the user is authenticated (Google or Facebook, etc.)
No doubt the ids for users must be unique.

It is possible but it would not be efficient though, to have 2 collections to store similar user data

The only thing we require is id. You don't need to store similar user data, you store everything in your users table and you add a chatUserId field there and whenever you use chat you select a user from you table (looking for that chatUserId match) to extract firstName, lastName and image. In this scenario you don't need to refactor anything, at least I think so.

I don't believe users will have at any point have the same ID, at least the doc ID would be unique.

That's correct. My point is that we use UIDs (I understood that you are doing the same) as doc IDs for users, but there might be someone with random doc IDs (which are not UIDs) which will break the security rules, that's why I don't think that there is a way to combine 2 different users tables.

Closing due to inactivity. I have created another issue #23 to allow an option to rename users and rooms collections, I think it will resolve the problem.


Excessive webhooks payloads

I want to merge this change because it handles lazy payload generation on webhooks (only if there is an event handler)

Impact

[ ] New migrations
[ ] New/Updated API fields or mutations
[ ] Deprecated API fields or mutations
[ ] Removed API types, fields, or mutations
[ ] Documentation needs to be updated

Pull Request Checklist

[ ] Privileged queries and mutations are guarded by proper permission checks
[ ] Database queries are optimized and the number of queries is constant
[ ] Database migration files are up to date
[ ] The changes are tested
[ ] GraphQL schema and type definitions are up to date
[ ] Changes are mentioned in the changelog

Task linked: SALEOR-5047 Saleor produces webhook payload no matter if any event exists


Mock contract transactions with single parameter will not accept an object for the params entry

When mocking a transaction for a contract interaction, if the transaction has a single parameter then for some reason you can no longer specify the params entry as an object. You can see this in effect here (this test passes): https://github.com/larvalabs/cryptopunksmarket/blob/cypresstests/cypress/integration/owner-actions.spec.js#L122 Whereas multiple params can be specified as an object as you would expect: https://github.com/larvalabs/cryptopunksmarket/blob/cypresstests/cypress/integration/owner-actions.spec.js#L100
Any idea why this might be the case? I was expecting to be able to specify single params as usual, for example:
params: {
    punkIndex: 0
}

Correct. Just fixed in v11.2.1
New test case:
https://github.com/DePayFi/web3-mock/commit/879127623e0c1a849fc132568bbdf81ce754dcb7#diff-fe6fe320fa60ac9c617d97b7b8057683af963e998a16d2ab5b894dac9619207eR85-R87

Confirmed fixed in 11.2.1 - thanks!


Broken links:

Broken links:

[ ] https://cloud.google.com/vision/docs/face-tutorial > Begore you begin > step five > node.js tab > package.json.
[ ] https://cloud.google.com/vision/docs/face-tutorial > Create the service object > node.js tam > line four.

Hey @Optimus-73, thanks for reporting! We actually don't control that page from here, so the best way to get the feedback to the right team is with the "SEND FEEDBACK" link at the bottom of the page.


commits webpage shows 500 error

We can see the directories ,files, and the contents of file already, but it shows 500 error if we click the commits or history button. It looks like there are something wrong if we would like to see the logs.

Just an idea...
Do you experience this problem on every commit? I've experienced problems when commit messages contain non-standard characters, e.g. Unicode emojis. Are your commit messages often or mostly in a non-western language?

Yes, every commit. We use english to commit logs. No emojis.

I had the same issue. Inspecting apache's error log showed a missing SimpleXmlIterator php class.
Installing the php7.0-xml package and restarting the webserver fixed the issue for me.

I had the same issue. Inspecting apache's error log showed a missing SimpleXmlIterator php class.
Installing the php7.0-xml package and restarting the webserver fixed the issue for me.

Thank you, @daumiller !!!!

This was the missing piece in my install as well!
Using version 1.0.2.
Can this please be mentioned in the troubleshooting steps? Together with how to detect this.
Thanks!


can i use this process on raspberry ? can i train my own model using raspberry pi 3 ?

Sure, you can refer to this link.

it will take at least a month, good_luck


Show comment button only to authorized users

Fixes https://github.com/NASA-IMPACT/nasa-apt/issues/542

Adds a permissions check to the hasCommentButton flag
Pass the ATBD object for the permissions check

How to test
Use the public ATDB http://localhost:9000/documents/test-atbd-1/v1.1
Switch to different users and check of the comment button shows when you hover over a headline

Owner Olivia should see the button
Andre Author should see the button
Allison Auth should not see the button

Log out completely, you should have access to the ATDB but not see the button

Confirmed.


Unknown measures

I'm using Ruuvitag-discovery with Home Assistant (MQTT)
From Ruuvitag-discovery UI if i choose all measures (21 measures) for my Ruuvitag i can see all measures in my HA but
Of all 21 measures only battery, humidity, pressure, temperature have values.. all the rest of measures are unknown ?

Apparently for the unknown parameters i had to restart ruuvitag-discovery to see actual values in HA
Weirdly enough this was not needed for battery, humidity, pressure and temperature


5.1.2 (3016) - No images showing in feed items

Desktop, 5.1.2 (3016), no feeds are displaying any images.

Do you have any feeds you can give us as an example?

https://www.jwz.org/blog/feed/

and all other feeds that have images

This is only happening since i just upgraded from previous public version

Are you using a syncing service like Feedly or Feedbin?

No, this is straight from the site's RSS feed

So you are using the On My Mac account, correct?

Correct

I'm unable to recreate your problem.  That's not good for me, because I have no way to debug it.  Unfortunately we might be have to try some last resort measures to get you back working again.
First, let's delete NetNewsWire and download a new copy to install.
https://github.com/Ranchero-Software/NetNewsWire/releases/download/mac-5.1.2/NetNewsWire5.1.2.zip
Once that is done, please restart your computer.  This works best if you have a goat or chicken to sacrifice while doing the restart.
Let me know how it goes and if that doesn't resolve your issue, we'll try to come up with another plan.

A reboot did it!
No goats, chickens, or even donkeys were harmed.


add ocaml support

this adds support for the ocaml lsp and formatter
everything seems to be working, the only issue is the lsp gives me a warning about not being able to find the formatter on format:

but it still formats so ¯\_(ツ)_/¯
I think it could be fixed by adding ocamlformat to nvf's environment but I don't know how to do that sowwy uwu

I think it could be fixed by adding ocamlformat to nvf's environment but I don't know how to do that sowwy uwu

nvf exposes a vim.extraPackages option to insert packages into the runtime PATH. You can consider adding ocamlformat there.

that worked thanks


Fix error when pasting more rows than current number of rows in ResultSetViewer

When I use Advanced Paste in a table to add new rows, I have to ensure that I leave enough room for the rows I am creating, otherwise it shows me an error dialog:

It was not the case in 7.3.2 version of dbeaver. The error seams to come from this commt: https://github.com/dbeaver/dbeaver/commit/f740fb41e970a382eddefd3c49465cb8144f1e25
I found a way to fix it by adding an out of bounds check in LightGrid.posToCell (make sure that target position is less than grid length). Not sure if it's the most relevant fix but it seams to make sense since there is already a check in the other way (make sure that position is greater than 0).

Is there an easy work around for this issue, until a fix is released?

Hello @bnickelsen,
I don't think so.
Also I'm having hard time reproducing this issue, can you share exact steps so I can reproduce it? Because currently I'm just hoping for the best.

Starting with a table open in the Data view.

Use the add line icon at the bottom to add one line.

I then have a spread sheet with all the data that I want to put into this table. So high ight the spread sheet and copy. Then in the data view Select the first cell of the line you added and paste. that is when I get the error. The issue seems to be that with this last update when you past more lines then you have added you get this error. For some reason it is not adding the new lines. the work around for me was to add enough line to fit eth data I was pasting 1027 row later. no error.
So again a work around is to add enough lines to fit the data you are pasting.

Thank you for response, I will investigate later.


feat: add latest version and outdated flag to plugin list command

What this PR does / why we need it:
Added LATEST column to plugin list output and --outdated flag to show only plugins with available updates, helping users identify plugins that need updating.
example:

If applicable:

[x] this PR contains user facing changes (the docs needed label should be applied if so)
[ ] this PR contains unit tests
[x] this PR has been tested for backwards compatibility

@gjenkins8 hi! What do you think about the feature? It was inspired by Homebrew's outdated command, which makes it easy to see which plugins have updates available.


The result of the RAG example based on LlamaPaser and MongoDB is “Empty Response”

Describe the bug
I built the LlamaIndex, LlamaPaser, MongoDB, and Atlas CLI environments locally according to the Readme file, and tried to run the "demo_mongodb.ipynb" example in the Github project. Unfortunately, "Empty Response" was output in the "Test Query" link (Figure 1). I think my operation is correct because I can see the saved data in the corresponding deployment of MongoDB Compass (Figure 2), and when I run the MongoDB example the correct results are returned (Figure 3).
Screenshots

Figure 1

Figure 2

Figure 3
Files
attention.pdf
Client:
Please remove untested options:

Python Library

Additional context
Ubuntu: 22.04
MongoDB: 8.0.0
Using Mongosh: 2.3.2

to fix your trouble check this solution click
maybe this will solve your problem.


Probes not shown in Navigator

Steps to reproduce

open subject01_metabolics.osim from Simulation-Based Design to Reduce Metabolic Cost

Expected result
Probes -> metabolics shown in Navigator.
Actual result
Probes does not appear.
Environment and GUI version
OpenSim-2416f17a-2018-02-25.zip ("OpenSim 4.0 User Experience and Feedback Session" version)

Addressed by PR #573 and PR #565

Fixed on windows OpenSim-e7831221-2018-03-06


Added usage via Maven

Updated README with a gradle config to be included in dependent projects. Addresses #134.

Can one of the admins verify this patch?

ok to test


Test group in unit tests should allow a name property

Proposal
A unit test file could have multiple test groups defined for different edge cases, all testing the same alert.
The current output only informs the user of the alertname, exp and got for failing tests, with no indication of which of the actual test groups failed.
The proposal is to add an optional name property to a test group that would be output if there are failing tests.

Thank you for this! I will dig into this and look at the PR soon.

Fixed in #8440. Thanks!


Brush doesn't update when cloning things by dragging off a locked paddle

Should be redone at same time as #126

Closed because this feature was removed


ShareClient::GetStatistics fail with invalid map<K, T> key

< HTTP/1.1 200 OK
< Transfer-Encoding: chunked
< Content-Type: application/xml
< Server: Windows-Azure-File/1.0 Microsoft-HTTPAPI/2.0
< x-ms-request-id: c4c25aeb-701a-0052-0966-46250e000000
< x-ms-version: 2021-12-02
< Date: Wed, 22 Feb 2023 02:38:35 GMT
<
<?xml version="1.0" encoding="utf-8"?><ShareStats><ShareUsageBytes>0</ShareUsageBytes></ShareStats>* Connection #0 to host premiumfileut.file.core.windows.net left intact

https://github.com/Azure/azure-sdk-for-cpp/blob/main/sdk/storage/azure-storage-files-shares/src/rest_client.cpp#L1655-L1657

Should we make them nullable or remove them? Both of these are breaking changes. But I think the first is less affected.

Should we make them nullable or remove them? Both of these are breaking changes. But I think the first is less affected.

Let's make them optional when parsing response, but keep the API interface unchanged. It's non-breaking this way.
You may refer to this link https://github.com/Azure/azure-sdk-for-cpp/blob/main/sdk/storage/azure-storage-blobs/swagger/README.md?plain=1#L541~L544 for how to do that.

Yeah, I found the way to do this without breaking change yet. So I deleted the message.

https://github.com/Azure/azure-sdk-for-cpp/blob/main/sdk/storage/azure-storage-files-shares/src/rest_client.cpp#L1655-L1657
REST API doesn't return ETag or Last-Modified at all.

@Jinming-Hu is that something we should fix in the code gen logic (or update the swagger)? Why does it generate code which expected ETag when the swagger doesn't?

@ahsonkhan These headers are present on both public REST API reference and swagger. I tend to think it's a server-side bug.
https://github.com/Azure/azure-rest-api-specs/blob/main/specification/storage/data-plane/Microsoft.FileStorage/preview/2022-11-02/file.json#L1777~L1785
https://learn.microsoft.com/en-us/rest/api/storageservices/get-share-stats#response-headers


iOS - Add debug mode + emit events without data

Hi @farfromrefug!
Here is a PR fixing two bugs.
About the issue with events emission, I noticed it in my app and decided to handle it the same way it is handled on Android.
I also wanted to fix the Webpack issue reported in #9, but I don't know what is the best way. Personnaly, I would completely get rid of the normalizeUrl part and use urls as they are. What do you think?

@sebj54 that seem fine to me. and I agree let s drop normalizeUrl. can you do this in this PR?

Yes, sure! I will do this during the day

I've tried to build to check if it works on a real project, but I'm actually stuck with an error thrown by Lerna:
yarn build
lerna notice cli v8.0.0

 >  Lerna (powered by Nx)   Running target build for 2 projects
    ✖  @nativescript-community/ui-webview:build
       ><EMAIL_ADDRESS>build.www
       > cd ../../www-src && rollup -c && cd .. && ts-node make-bridge-loader

       bridge.android.ts → ../build/bridge.android.js...
       created ../build/bridge.android.js in 1.2s

       bridge.ios.ts → ../build/bridge.ios.js...
       created ../build/bridge.ios.js in 957ms

       metadata-view-port.ts → ../build/metadata-view-port.js...
       created ../build/metadata-view-port.js in 688ms

       ><EMAIL_ADDRESS>tsc
       > cpy '**/*.d.ts' '../../packages/webview' --parents --cwd=../../src/webview && tsc -skipLibCheck -d

       ../../src/webview/index.android.ts:871:18 - error TS2339: Property 'setAppCachePath' does not exist on type 'WebSettings'.

       871         settings.setAppCachePath(pth);
                            ~~~~~~~~~~~~~~~

       ../../src/webview/index.android.ts:872:18 - error TS2339: Property 'setAppCacheEnabled' does not exist on type 'WebSettings'.

       872         settings.setAppCacheEnabled(!!pth);
                            ~~~~~~~~~~~~~~~~~~

       Found 2 errors in the same file, starting at: ../../src/webview/index.android.ts:871

       npm ERR! Lifecycle script `tsc` failed with error:
       npm ERR! Error: command failed
       npm ERR!   in workspace<EMAIL_ADDRESS>       npm ERR!   at location: /Users/sebjean/projects/ui-webview/packages/webview

 ————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————

 >  Lerna (powered by Nx)   Ran target build for 2 projects (14s)

    ✔    0/1 succeeded [0 read from cache]

    ✖    1/1 targets failed, including the following:
         - @nativescript-community/ui-webview:build

It seems to be a false positive but I can't figure out how to fix this.
Also, I had to tweak some yarn and npm related files, I will submit them with the PR, so be sure to check if my changes are relevant when it will be ready to merge.

@sebj54 does vscode report the same errors? Please dont submit npm/yarn changes in this PR. either create a new one or create an issue to explain the issue you have with it
So the PR is ready from your side?

PR is ready but I couldn't test the build. Vscode does not report the same error, I don't know what's wrong on Lerna's side. Maybe it is because android.webkit.WebSettings is a class and a namespace at the same time.
Ok I won't submit any other change then!

@sebj54 ok the issue is not in lerna but in tsc from the command line. Lerna is only there to run the tsc command.

Ok then, let me know when it's published, I will test it right away

@sebj54 it is released. BTW the lint error was 100% normal and i am glad we got it. That api was removed from android

@farfromrefug Everything works fine!

BTW the lint error was 100% normal and i am glad we got it. That api was removed from android

Oh I see! typings were outdated then? Glad you found the issue, it would have taken me so long to find out


make thanos field option for backcompatibility.

ref: open-cluster-management/backlog#12807
Signed-off-by: morvencao<EMAIL_ADDRESS>

/lgtm


Cleanup: remove or narrow common.unwrap usage

Most of it should be in translate_ids, normalize_ids, etc. What do we still need it for? Wrapping urls in fediverse objects?

Started on this in the tighten/narrow unwrap stash.

Updated the tighten/narrow unwrap stash, getting closer!

Oops, I forgot that unwrap now handles protocol subdomains and /convert/ paths as well, so it's generic/cross-protocol now after all. Never mind!
(I kept this in the DO NOT MERGE: tighten unwrap (#967) stash, but obviously I don't plan to, um, merge it. 😁)


Implement platform channel response handling in GLFW embedding

https://github.com/flutter/engine/pull/9655 added the necessary APIs to add the missing handling of responses when calling from native to Dart.

This was fixed in https://github.com/flutter/engine/pull/9948


Wincrate CI PR

This PR is to check the CI for a merge of https://github.com/rust-pcap/pcap/pull/238 with the newest main after another merge and with clippy fixed

Passes


[INTEL oneDNN][Bug Fix]Use correct data type for dim size

This is a follow-up PR of https://github.com/tensorflow/tensorflow/pull/60568
This PR makes sure that all occurrences of dim size usage is tied with int64_t data type (versus int)
within all oneDNN kernel op implementation.

All 3 CI failures are unrelated.

Py+CPP Ubuntu CPU: Tests timed out.
Py+CPP Ubuntu GPU: Tests timed out.
AMD ROCm: Error unrelated to this PR.

Analyzing: target //tensorflow/tools/pip_package:build_pip_package (648 packages loaded, 30206 targets configured)

ERROR: /workspace/tensorflow/compiler/xla/service/gpu/BUILD:853:11: in deps attribute of cc_library rule //tensorflow/compiler/xla/service/gpu:gpu_executable: Label '//tensorflow/tsl/platform:random' is duplicated

ERROR: /workspace/tensorflow/compiler/xla/service/gpu/BUILD:853:11: Analysis of target '//tensorflow/compiler/xla/service/gpu:gpu_executable' failed


Added questions to html-quiz

Summary of changes
Added questions to html quiz
Checklist

[x] I have read and followed the contribution guidelines.
[x] I have read through the Code of Conduct and agree to abide by the rules.
[x] This PR is for one of the available issues and is not a PR for an issue already assigned to someone else.
[x] My PR title has a short descriptive name so the maintainers can get an idea of what the PR is about.
[x] I have provided a summary of my changes.

closes #XXXXX

Thank you for your contribution!
Please review the requested changes. Once those changes have been implemented, then we can merge your PR into main. 👍

Committed all changes.
Thank you Ma'am


Allow episode ID based access control via roles

If activated, users with a role like ROLE_EPISODE_<ID>_<ACTION> will have access to the episode with the given identifier, without this having to be specifically specified in the ACL attached to the episode.
For example, ROLE_EPISODE_872dc4ec-ca8a-4e12-8dac-ce99784d6d29_READ will allow the user to get read access to episode 872dc4ec-ca8a-4e12-8dac-ce99784d6d29.
Your pull request should…

[ ] have a concise title
[ ] close an accompanying issue if one exists
[ ] be against the correct branch
[ ] include migration scripts and documentation, if appropriate
[ ] pass automated tests
[ ] have a clean commit history
[ ] have proper commit messages (title and body) for all commits

(To-do) What is missing?
ACL file permission checks already work, and checks via Solr work as well. But Elasticsearch also holds a representation of the ACL and needs to honor the new ID roles.
In the best case, it does that without having to modify the stored ACLs in Elasticsearch. Unfortunately, that seems to be hard.
An alternative would be to write modified ACLs to the search index. This would already work when doing a full re-indexing of asset manager data, but that's probably an unreasonable amount of work. Acceptable would be, however, if we could trigger a re-index/update of only the ACLs from the asset manager: Essentially, run through all events in the asset manager, take the ACL from the latest snapshot and write that (including the ID roles) to the correct field in the search index.

Attempted to implement the to-do lkiesow mentioned above, following his suggested alternative (because I agree with his assertion that the implementation for the "best case" would be very hard).
Also I don't understand why the docs are not compiling.

Aaand I just noticed that new events are not getting the custom roles >.< Will look into that.

Looks like this is resolved, too? @Arnei ?

Uuh, remind we what we were talking about here? ^^'

Looks like this is resolved, too? @Arnei ?

Uuh, remind we what we were talking about here? ^^'

Never mind, I'll check for the next review pass. :D

Uff, thanks for catching those Greg. Should all be fixed now.

To do: Re-target to 16

Now points to 16. Means we can drop changes to Solr, since Solr is gone. Instead, this now updates the Search Index in Elasticsearch in a similar fashion to the event index. Not yet sure how to make an ACL-update-only version for fast reindexing of the Search index, since the current code is very "all or nothing" with its indexing approach.

Can now view publications, but not view assets in event details or start workflows. Both seem to be related to an issue with ASelectQuery generating different queries for episode id roles users, but I am having trouble figuring out the details.

So ASelectQuery generates an authorization predicate prior to generating the query, based on whether the user is admin, their organization and roles. Unfortunately, it does so before any episode id comes into play (and for some queries an episode id never comes into play), so we can't just circumvent it. To circumvent it, we would have to rewrite the query logic somewhat, but that would be a major pain as ASelectQuery is used in quite a few different places and based off of entwine code. Not sure how to proceed here.

My episode id role user was able to view and download Assets, as well as start a workflow.

Oh, and in regards to testing: I tested this by creating an unprivileged user and giving it the new roles for an event by either configuring those roles in custom.properties first so I can select them in the UI or by creating a group via the API so I can set arbitrary roles (probably also possible for the user directly).

Re: no errors from the index endpoint when the wrong data type for re-indexing is entered, see https://github.com/KatrinIhler/opencast/commit/ac5f74a2f9882c03b664bb6b897615f19461a215. This compiles fine, but i didn't test it because this took way longer than I anticipated...)  Could you take a look at this and pull it in if it works?
Basically what this does is check the supported services and data types before entering the asynchronous part of the code (and some very minor cleanup that I couldn't resist).

Just for clarification: feel free to ignore my previous comments, it might be harder than I think and I really don't know a lot about this part of Opencast. I just want this feature in OC :P

I think the ACL that we get from the index should not contain the new roles. If that leads to breakage in e.g. the Workflow Service, we should extend those permission checks in the same way we did for e.g. the AssetManager and similarly to how we treat ROLE_ADMIN.

Latest commit should achieve that. It still puts the roles in the index, but not in the ACL in the index, but in other ACL-related fields that are used for permission checks (I think).

Shouldn't it be possible to only change one or very few central methods that would adjust the query sent to ES?

That would be very nice, but I could not come up with a solution on how to achieve that.

This works as expected, so I'm going to merge it.  Found https://github.com/opencast/opencast-admin-interface/issues/903 in the process of testing this (again), but it does not block the backend feature's use.


Fixing openID Apps that return Dynamic Mode

Certain OAuth apps can return DYNAMIC as a issuer mode, this wasn't supported previously by the IssuerModel as outlined in #286 and #287 and https://github.com/okta/okta-management-openapi-spec/issues/121

Hi, any updates?

I really need this fix, because I can't use the list_all_apps() method anymore.

any update?

@csanders-git could you please involve someone who's capable of reviewing and merging this pull request? That would be great. Thanks in advance

hey, please give this fix a priority

@serhiibuniak-okta could you please review this PR?

@csanders-git could you please assign a second reviewer to this Pull Request?

This can be closed as @drewcarmichael-okta silently duplicated and made redundant this PR without assigning credit.
https://github.com/okta/okta-sdk-python/commit/15b2a9986c5746bbacf387a75bf6a9b9010171c9


BUG: When turning off right click, this causes a bug in combat mode

When turning off "right click" under "base clicks," right-clicking while Combat Mode is active turns off Combat Mode and must be manually turned back on using the assigned keybind. Right-clicking again does not do anything. Left-clicking while Combat Mode is active makes the player move forward (presumably because holding both right & left-click makes the player move as well)

if you disable the base clicks overriding AND have Free Look locked in:
when right-clicking, it unlocks the cursor. Subsequent right-clicks do nothing
when left-clicking the player moves forward

More of a feature request than a bug. One that I'm not too keen on adding. The mod is made with the assumption that you'll override the base keybinds at the very least. I don't wanna deal with the ramifications of using Free Look and at the same time trying to preserve the base left and right click functionalities.

Solved with #28


fix for the monaco editor

Please make sure to label your PR with "bug", "new feature" or "breaking change" label(s).
To prevent this PR from going to the changelog marked it with the "skip changelog" label.

Snapshot stored with reference name:
refs/pull/13991/merge
Test environment:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/13991/merge/index.html
To test a playground add it to the URL, for example:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/13991/merge/index.html#WGZLGJ#4600
Links to test babylon tools with this snapshot:
https://playground.babylonjs.com/?snapshot=refs/pull/13991/merge
https://sandbox.babylonjs.com/?snapshot=refs/pull/13991/merge
https://gui.babylonjs.com/?snapshot=refs/pull/13991/merge
https://nme.babylonjs.com/?snapshot=refs/pull/13991/merge
To test the snapshot in the playground with a playground ID add it after the snapshot query string:
https://playground.babylonjs.com/?snapshot=refs/pull/13991/merge#BCU1XR#0


Add documentation to Dynamic Persistent Volumes about DefaultStorageClass admission control

From https://github.com/kubernetes/kubernetes/issues/54369

This is a...

[X] Feature Request

Problem:
I went to the documentation page, and read about dynamic provisioning. A default class was set, but the DefaultStoreClass admission plugin wasn't enabled, and I spent several hours searching as to why my pvc's weren't auto provisioning.
Proposed Solution:
I would have liked to have seen more documentation mentioning that the Admission Plugin is required for dynamic provisioning, with a link here.
Page to Update:
https://kubernetes.io/docs/concepts/storage/persistent-volumes/#dynamic

/assign


Intent Added in Location and Speaker's List #154

Intent added #154

Done,@creativepsyco ,  fixed #162 along with adding missing intent #154

Ok give me until 2moro to review this. Thanks for the patience.

:smiley:

There was some merge conflict...I have resolved them also

@creativepsyco  done with the changes.

travis is failing due to codestyles...it also includes code that are not changed in this commit.

The codestyle issues will only be posted if you have made them in the current commit, i wouldn't worry about those.

@creativepsyco ,I have manually fixed all that i have made in current commit...but still its failing. :(

okay let me take a look into it, i think some gem is missing.

solved the merge conflict.

Build failing @opticod. Check the travis logs.

@mananwason @creativepsyco . Build passed

Am I missing anything ? @creativepsyco

there are some conflicts lemme resolve and merge.

Ok,thanks.

merge via commit 9252760


Fix #336 (accept text/html inspection results)

Also added heuristics to guess a selection to inspect, when no text has been selected.

Added two TODO items:

handle inspection errors
improve selection heuristics using kernel.complete

13213ad Works for me 👍
I'm not sure about the heuristics. I thought they are handled by the kernel. Jupyter messaging specs says about the cursor_position:

The cursor position within 'code' (in unicode characters) where inspection is requested

What improvement would the heuristics on our side bring for the inspect result? I get the same introspection results with and without e86d241 on python.

IJulia and IJavascript fail when cursor_pos is 0.
It is also up to the kernel how to handle cases like Arr|ay. Does it inspect Arr or Array?

IJulia and IJavascript fail when cursor_pos is 0.
It is also up to the kernel how to handle cases like Arr|ay. Does it inspect Arr or Array?

Oh I see. Then it makes sense!


Upgrade get-port dependency

Thanks, but this is not needed. Semver handles this for us. Read more here: https://nodesource.com/blog/semver-tilde-and-caret

Thanks for the clarification!


Publish a new beta version of react-router-config

Hello,
The current version on NPM of react-router-config is beta1 from march 11th. That version does not match the current doc and is fundamentally broken/not suitable for use because it does not support strict /  exact for routes, which was added in #4826 .
I think it would really be worth publishing a beta2 version of that package right now to fix that.

We'll publish something soon, I'm sure.


Tag 1.0.0 release

[ ] Make sure there's nothing to add.
[ ] Tag it!

Strict types could be added.


feat: added autocomplete, configured oclif settings

Summary generated by Reviewpad on 04 Jul 23 10:50 UTC
This pull request includes changes to the file package.json. The changes involve adding a dependency, modifying the "oclif" section, and updating the "plugins" array. Please review these changes and ensure their correctness.

Adds autocompletion support to the standalone CLI using @oclif/autocomplete.
The instructions currently have a display bug in the topic separator command. This is caused by @oclif/core installing in the project root, instead of the CLI package.
Checklist
All items have to be completed before a PR is merged

[x] Adhere to Contribution Guidelines
[x] Updates to Decision Records considered?
[x] Updates to documentation at DX@Scale Guide considered?
[x] Tested changes?
[x] Unit Tests new and existing passing locally?

still pulling for this


[stacked_services] [snackbar] Unhandled Exception: Null check operator used on a null value

Steps to reproduce.
Call show custom snackbar.
_snackbarService.showCustomSnackBar(
variant: SnackbarType.error,
message: exception.message,
duration: const Duration(seconds: 2),
);
The exception occurs on the following line in the snackbar_controller class.
 Line 91: _overlayState = Overlay.of(Get.overlayContext!);

Unhandled Exception: Null check operator used on a null value
#0      SnackbarController._configureOverlay (package:get/get_navigation/src/snackbar/snackbar_controller.dart:91:50)
#1      SnackbarController._show (package:get/get_navigation/src/snackbar/snackbar_controller.dart:331:5)
#2      GetQueue._check (package:get/get_utils/src/queue/get_queue.dart:42:47)
#3      GetQueue.add (package:get/get_utils/src/queue/get_queue.dart:29:5)
#4      _SnackBarQueue._addJob (package:get/get_navigation/src/snackbar/snackbar_controller.dart:357:31)
#5      SnackbarController.show (package:get/get_navigation/src/snackbar/snackbar_controller.dart:63:27)
#6      ExtensionSnackbar.showSnackbar (package:get/get_navigation/src/extension_navigation.dart:366:16)
#7      GetSnackBar.show (package:get/get_navigation/src/snackbar/snackbar.dart:288:16)
#8      SnackbarService.showCustomSnackBar.<anonymous closure> (package:stacked_services/src/snackbar/snackbar_service.dart:216:37)

I'm confirming if this happens in the stacked example now.

Used the same code as above with a single update of the variant. I have to change the variant to blueAndYellow because I don't have an error variant registered. This code:
   _snackbarService.showCustomSnackBar(
                variant: SnackbarType.blueAndYellow,
                message: 'exception.message',
                duration: const Duration(seconds: 2),
              );

Produces the image below

Can you post the code that registers your error variant @mugikhan

service.registerCustomSnackbarConfig(
    variant: SnackbarType.error,
    config: SnackbarConfig(
      snackPosition: SnackPosition.TOP,
      backgroundColor: ColorPalette.error,
      textColor: ColorPalette.onError,
      borderRadius: 1,
      dismissDirection: DismissDirection.up,
    ),
  );
  Shows it using this

  _snackbarService.showCustomSnackBar(
variant: SnackbarType.error,
message: exception.message,
duration: const Duration(seconds: 2),
);

This is my main app class. I am using auto_route directly as I need route guards and stacked doesn't cater for it. If I need stacked navigation key I will register my own service to show snackbars I guess. @FilledStacks
class MyApp extends StatelessWidget {
  MyApp({Key? key}) : super(key: key);

  final _appRouter = GetIt.I<AppRouter>();

  @override
  Widget build(BuildContext context) {
    return ThemeBuilder(
      defaultThemeMode: ThemeMode.light,
      statusBarColorBuilder: (theme) => Colors.transparent,
      lightTheme: lightTheme,
      darkTheme: darkTheme,
      builder: (context, regularTheme, darkTheme, themeMode) {
        return MaterialApp.router(
          title: 'Transylvania',
          routerDelegate: _appRouter.delegate(),
          routeInformationParser: _appRouter.defaultRouteParser(),
          theme: regularTheme,
          darkTheme: darkTheme?.copyWith(
            pageTransitionsTheme: const PageTransitionsTheme(
              builders: {
                TargetPlatform.iOS: NoShadowCupertinoPageTransitionsBuilder(),
                TargetPlatform.android: FadeUpwardsPageTransitionsBuilder(),
              },
            ),
          ),
          themeMode: themeMode,
          localizationsDelegates: const [
            GlobalMaterialLocalizations.delegate,
            GlobalWidgetsLocalizations.delegate,
            GlobalCupertinoLocalizations.delegate,
          ],
          supportedLocales: const [Locale('en', '')],
        );
      },
    );
  }
}

@mugikhan when you supply your router delegate I think you have access to the nav key in there. I know there are some stacked users that use auto route with the latest version of stacked.
Closing this issue as it's not a bug

This is all I have access to. How can I use this to add the navigator key?

Mmmmmm, maybe not in the auto router delegate. I don't know how to access it there.

@FilledStacks
Just an update tried to use GetX directly with the GetMaterialApp and showing a snackbar using Get.snackbar(...)

This also errored out with the same error so I guess I'll wait until that package is updated.

Thanks for the update @mugikhan

Yes I can. I will do so a little later tonight. I’ve been on leave and didn’t have access to my laptop. I think the problem is that I’m not using the stacked navigator key and generate route. I need route guards so I am using auto router directly.
Regards,
Mughees Khan

From: Dane Mackier @.>
Sent: Wednesday, September 7, 2022 5:13:47 PM
To: FilledStacks/stacked @.>
Cc: Mughees Khan @.>; Mention @.>
Subject: Re: [FilledStacks/stacked] [stacked_services] [snackbar] Unhandled Exception: Null check operator used on a null value (Issue #730)
Can you post the code that registers your error variant @mugikhanhttps://github.com/mugikhan
—
Reply to this email directly, view it on GitHubhttps://github.com/FilledStacks/stacked/issues/730#issuecomment-1239523764, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AE6VZM7OLIQZODYX3K7AYDDV5CWKXANCNFSM6AAAAAAQD5KEEM.
You are receiving this because you were mentioned.Message ID: @.***>


New PR to test SignCLA at 2019-07-08 17:26:45.822013134 -0700 PDT m=+4.535223254

Body of PR

corp-pass-staging: test succeeded


Add markdown support using jupytext

If we have a filetype that can be converted to a notebook using jupytext, by default allow that.
Also have a switch to disallow particular filetypes or disable jupyext parsing.

Fixed by https://github.com/innovationOUtside/nb_quality_profile/commit/56ca1619a15729a0bb6e908d78725d33ba579994


Feature: Disable Automatic Logout

Feature Description
When I work on security assessments and pause my virtual machine to call it a day, the next day I come back I see my BHCE tabs for one second after which the application notices the expired session and logs me out.
That way the last query is gone, which sometimes is an issue for me.
Suggestion
There could be an option to disable automatic logout entirely or make it take some more time like a week of inactivity. Alternatively a redirect to the last query after logging in again would be also a valid solution.

https://github.com/SpecterOps/BloodHound/issues/781 should include the query within your browser history. I'm going to close this ticket as we do not intend to remove the session expiration functionality at this time and the other feature request will handle the alternative option you requested.

Fine for me 👍


e2e: check goroutine leaks

@heyitsanthony

Obviously much needed. I think the bulk of this should go in pkg/testutil so it can be shared with the integration tests.

Agree. This was just to raise the issue. We can figure out why e2e leaks goroutines later.
Thanks!


Disable SortField but not effect

i use this to disable SortField:
JSON.DEFAULT_GENERATE_FEATURE = JSON.DEFAULT_GENERATE_FEATURE & ~SerializerFeature.SortField.getMask();
but it seems not be effect, does this flag used when serializing ?

Can you show more codes ?


Error saving session: securecookie: the value is too long

When SingleToken = false, every unique page that generates a token can grow to a point where this error message is displayed:
token.go:56: Error saving session: securecookie: the value is too long
Tokens need to be removed on next page load if singletoken = false.

Fixed in: https://github.com/josephspurrier/csrfbanana/commit/94b0691ae8f68e1b23b877a9b96c00420383cd73


supporting OSX backend

A few projects got the right to experiment with new osx boxes on Travis.
Right now this requires working around: https://github.com/ocaml/opam/issues/1853
(brew install opam --HEAD seems to do it for now).
See these scripts using the OSX backend:

https://github.com/hammerlab/ketrew/blob/use_trakeva/tools/travis_ci_test.sh
https://github.com/samoht/assemblage/blob/master/.travis-ci.sh

That's a very good idea, and patches are welcome :-)


WIP: Workaround change in tifffile's memmap feature

In tifffile version 0.13.0+, it appears that the memmap keyword to asarray has been dropped. Instead one must use out and set it to "memmap". To handle this change while remaining compatible with the old versions of tifffile, we try the new syntax and fallback to old syntax if a TypeError is raised (happens when the keyword doesn't exist).

Replaced with PR ( https://github.com/nanshe-org/nanshe/pull/471 ).


Build failed with error '_swift_getTypeByMangledNameInContextInMetadataState' when adding Notification Content Extension to iOS project.

Description
Build failed with an error when adding Notification Content Extension to iOS project.
Undefined symbols for architecture arm64:
  "_swift_getTypeByMangledNameInContextInMetadataState", referenced from:
      ___swift_instantiateConcreteTypeFromMangledNameAbstract in NotificationViewController.o

React Native version:

react-native info result
System:
    OS: macOS 10.16
    CPU: (8) x64 Intel(R) Core(TM) i5-8259U CPU @ 2.30GHz
    Memory: 107.45 MB / 16.00 GB
    Shell: 2.7.1 - /usr/local/bin/fish
  Binaries:
    Node: 10.16.0 - ~/.nvm/versions/node/v10.16.0/bin/node
    Yarn: 1.22.4 - ~/.yarn/bin/yarn
    npm: 6.10.3 - ~/.nvm/versions/node/v10.16.0/bin/npm
    Watchman: 4.9.0 - /usr/local/bin/watchman
  SDKs:
    iOS SDK:
      Platforms: iOS 13.5, DriverKit 19.0, macOS 10.15, tvOS 13.4, watchOS 6.2
    Android SDK:
      API Levels: 23, 28, 29
      Build Tools: 28.0.3, 29.0.2
      System Images: android-25 | Google APIs ARM EABI v7a, android-29 | Google APIs Intel x86 Atom
  IDEs:
    Android Studio: 4.0 AI-193.69<IP_ADDRESS>14223
    Xcode: 11.5/11E608c - /usr/bin/xcodebuild
  npmPackages:
    @react-native-community/cli: 4.9.0 => 4.9.0
    react: ^16.13.1 => 16.13.1
    react-native: 0.62.2 => 0.62.2
  npmGlobalPackages:
    create-react-native-module: 0.16.0

Steps To Reproduce

in Xcode, select File -> New -> Target and choose Notification Content Extension.
Build project and error occurs.

Expected Results
Add Notification Content Extension to React Native project and build it successfully.

Same here, but when trying to implement Apollo for Graphql as a swift package

Undefined symbols for architecture x86_64:
  "_swift_getTypeByMangledNameInContextInMetadataState", referenced from:
      ___swift_instantiateConcreteTypeFromMangledNameAbstract in ShareViewController.o
ld: symbol(s) not found for architecture x86_64
clang: error: linker command failed with exit code 1 (use -v to see invocation)

how to resolve?

I use the following method to solve the problem
LIBRARY_SEARCH_PATHS = (
   "\"$(TOOLCHAIN_DIR)/usr/lib/swift/$(PLATFORM_NAME)\"",
-  "\"$(TOOLCHAIN_DIR)/usr/lib/swift-5.0/$(PLATFORM_NAME)\"",
+  "\"$(TOOLCHAIN_DIR)/usr/lib/swift-5.2/$(PLATFORM_NAME)\"",
   "\"$(inherited)\"",
 );

@hengkx worked for me :)

I also had this issue and asked about it on the Swift forums. The cause is trying to link against an older Swift runtime while targeting a newer version of the OS.
You should be able to just remove those LIBRARY_SEARCH_PATH entries entirely (or just leave $(inherited)) and that will fix the problem. This is a bug in the ReactNative template for iOS.

@hengkx, it works for me! Thanks a lot!

Sorry for commenting on a closed issue, but I'm still experiencing this problem and nothing seems to help. I've tried changing my IPHONEOS_DEPLOYMENT_TARGET to 9.0, 10.0, 11.0 and 13.5, nothing works. I've tried @hengkx 's fix, didn't work. I've tried leaving only inherited in there, and removing it entirely, still didn't work. Can anyone help me out here?

Same issue here, adding a WidgetExtension to a blank react-native app causes this issue. None of the above fixes seem to work

I also had this issue and asked about it on the Swift forums. The cause is trying to link against an older Swift runtime while targeting a newer version of the OS.
You should be able to just remove those LIBRARY_SEARCH_PATH entries entirely (or just leave $(inherited)) and that will fix the problem. This is a bug in the ReactNative template for iOS.
As mentioned in the above Swift forums thread:

You shouldn't need to add the toolchain library paths to LIBRARY_SEARCH_PATHS yourself, because Xcode will automatically include those libraries if your minimum deployment target is old enough to need them.

👆 This is the real fix

@mcoevert true. I just remove those LIBRARY_SEARCH_PATH entries completely, it'll fix the problem

@mcoevert true. I just remove those LIBRARY_SEARCH_PATH entries completely, it'll fix the problem

I also had this issue and asked about it on the Swift forums. The cause is trying to link against an older Swift runtime while targeting a newer version of the OS.
You should be able to just remove those LIBRARY_SEARCH_PATH entries entirely (or just leave $(inherited)) and that will fix the problem. This is a bug in the ReactNative template for iOS.
As mentioned in the above Swift forums thread:

You shouldn't need to add the toolchain library paths to LIBRARY_SEARCH_PATHS yourself, because Xcode will automatically include those libraries if your minimum deployment target is old enough to need them.

👆 This is the real fix

But I got this cocoaPods error
    - Use the `$(inherited)` flag, or
    - Remove the build settings from the target.
    - Use the `$(inherited)` flag, or
    - Remove the build settings from the target.

I also had this issue and asked about it on the Swift forums. The cause is trying to link against an older Swift runtime while targeting a newer version of the OS.
You should be able to just remove those LIBRARY_SEARCH_PATH entries entirely (or just leave $(inherited)) and that will fix the problem. This is a bug in the ReactNative template for iOS.
As mentioned in the above Swift forums thread:

You shouldn't need to add the toolchain library paths to LIBRARY_SEARCH_PATHS yourself, because Xcode will automatically include those libraries if your minimum deployment target is old enough to need them.

👆 This is the real fix

But I got this cocoaPods error
    - Use the `$(inherited)` flag, or
    - Remove the build settings from the target.
    - Use the `$(inherited)` flag, or
    - Remove the build settings from the target.

I use the following method to solve the problem
LIBRARY_SEARCH_PATHS = (
   "\"$(TOOLCHAIN_DIR)/usr/lib/swift/$(PLATFORM_NAME)\"",
-  "\"$(TOOLCHAIN_DIR)/usr/lib/swift-5.0/$(PLATFORM_NAME)\"",
+  "\"$(TOOLCHAIN_DIR)/usr/lib/swift-5.2/$(PLATFORM_NAME)\"",
   "\"$(inherited)\"",
 );

Thank You! This solved it!  Holy S@#%!

I use the following method to solve the problem
LIBRARY_SEARCH_PATHS = (
   "\"$(TOOLCHAIN_DIR)/usr/lib/swift/$(PLATFORM_NAME)\"",
-  "\"$(TOOLCHAIN_DIR)/usr/lib/swift-5.0/$(PLATFORM_NAME)\"",
+  "\"$(TOOLCHAIN_DIR)/usr/lib/swift-5.2/$(PLATFORM_NAME)\"",
   "\"$(inherited)\"",
 );

Works well! Thank you 🙏


pass in token as a parameter to MarkdownExporter

Currently, there is no way to pass the Notion_Token as a parameter to the MarkdownExporter. This means that the only way for it to work in a python script is to set the environment variable to the correct notion_token.
Can we add notion_token as parameter to markdownexporter and check if it's passed use whatever was passed otherwise fetch from os.environ ?
This makes it hard to work with multiple notion keys in the same code.

Sounds Good. At that time I had no idea how to pass the token, but now I think this can be implemented by adding the token argument from cleo Command(block_command). Thank you for bring useful idea


Directly activate keyboard in searchfield when opening app list

Would make handling more easy.

I definitely see how this would eliminate an extra tap for users that primarily use the search box.  However, anyone who does not use the search box too much probably does not want the keyboard popping up every time they open the app drawer.  Maybe we could add an option to toggle the behavior....
Honestly, if we implement the change from https://github.com/jkuester/unlauncher/issues/43 to move the search box to the bottom, then it seems like the extra tap in the search box is probably not too troublesome since the user's hands would not have to move/stretch from the "typing position".

I find very suitable adding an option to toggle the keyboard automatically after sliding.


collector name search and JSON implementation (people linkage)

Hej-hej,
I started commenting in https://github.com/DiSSCo/SDR/issues/27 but here is a more suitable place to ask.
I’m involved in 4.6 People Linkage for collectors etc.. So far I managed to write a python wrapper to add a collector by a command like so (following the SDR glalaxy workflow wrapper):
python 'main.py' -i ./test-data/open-ds-input.json -o ./test-data/open-ds-output.json --collector 'Linné'

As input/output I expect:

input: a name tagged or specified or identified as collector
output: add a list to JSON openDS

So far I contemplated and I came up with this:
"original_collector_search": {
    "results": [
        {
            "item": {
                "type": "uri",
                "value": "http://www.wikidata.org/entity/Q1043"
            },
            "personLabel": {
                "xml:lang": "en",
                "type": "literal",
                "value": "Carl Linnaeus"
            },
            "personAltLabels": {
                "type": "literal",
                "value": "Carl Linn\u00e6us; Carl Nilsson Linn\u00e6us; Carl von Linnaeus; Carl von Linne; Carl von Linn\u00e9; Caroli Linn\u00e6i; Carolo Linnaeo; Carolo Linn\u00e6o; Carolus Linnaeus; Carolus Linn\u00e6us; Carolus a Linne; Carolus a Linn\u00e9; Karl von Linn\u00e9; L.; Linn.; Linnaeus; Linne; Linn\u00e6us; Linn\u00e9"
            },
            "genderLabel": {
                "xml:lang": "en",
                "type": "literal",
                "value": "male"
            },
            "dateOfBirth": {
                "datatype": "http://www.w3.org/2001/XMLSchema#dateTime",
                "type": "literal",
                "value": "1707-05-23T00:00:00Z"
            },
            "dateOfDeath": {
                "datatype": "http://www.w3.org/2001/XMLSchema#dateTime",
                "type": "literal",
                "value": "1778-01-10T00:00:00Z"
            },
            "image": {
                "type": "uri",
                "value": "http://commons.wikimedia.org/wiki/Special:FilePath/Carl%20von%20Linn%C3%A9.jpg"
            },
            "numApiOrdinal": {
                "datatype": "http://www.w3.org/2001/XMLSchema#int",
                "type": "literal",
                "value": "0"
            },
            "VIAF_ID_URI": {
                "type": "uri",
                "value": "http://viaf.org/viaf/34594730"
            },
            "ISNI_ID": {
                "type": "literal",
                "value": "0000 0001 2127 4957"
            },
            "WIKIDATA_ID_URI": {
                "type": "uri",
                "value": "http://www.wikidata.org/entity/Q1043"
            },
            "occupations": {
                "type": "literal",
                "value": "professor; ornithologist; bryologist; pteridologist; autobiographer; botanist; mycologist; entomologist; zoologist; geologist; biologist; naturalist; physician"
            },
            "occupationsWithLangPrefix": {
                "type": "literal",
                "value": "en:professor; en:ornithologist; en:bryologist; en:pteridologist; en:autobiographer; en:botanist; en:mycologist; en:entomologist; en:zoologist; en:geologist; en:biologist; en:naturalist; en:physician"
            }
        },
        {...},
        {...}
    ],
    "summary": "3 results searching collector \u201cLinne\u201d: Carl Linnaeus; Carl Linnaeus the Younger; Johan Stensson Rothman."
}

Ideas behind this:

summary is always present
results may be empty if nothing was found
add as many meaningful properties as possible to let a human decide what collector matches best (numApiOrdinal may help: it’s the order of findings from the WikiBase Api (?the lower the value, the higher is the match?))
the query works at first against WikiBase to get the people properties; the found wiki data ID could serve as a linkage to query further open linked data from other interfaces (e.g. BotanyPilot at BGBM or its extended version later on, because now its limited to herbarium sheets only)

Questions to clarify:

How should the openDS list structure for collectors be constructed?
What types of persons are to be tagged from OCR?

original collector, secondary collector, collector manager, … ?

specimen collector (OBI_0001895): «A person who collects the specimen»—presumably the primary collector who took the being by hand at first
specimen collector role (OBI_0001769): «An Investigation agent role borne by a person or organization which is realized in a specimen collection process.»—presumably the persons involved in the collection process later on
recorded by (Darwin Core term) as generic term definition: «A list (concatenated and separated) of names of people, groups, or organizations responsible for recording the original Occurrence. The primary collector or observer, especially one who applies a personal identifier (recordNumber), should be listed first.»
…

determiner, …
descriptor, …
other types ? Or is there already an ontology for person types to draw property names  from, and to use it here?

Thank you for any clarifications
Andreas

Hej-hej,
I’m contemplating further if there are multiple collectors—one after another—to be searched and annotated through and it’s implementations in a JSON structure that may be meaningful. The dwc:recordedBy is thematising it by pointing out in providing the primary collector first and then the others:

recorded by (Darwin Core term) definition: «A list (concatenated and separated) of names of people, groups, or organizations responsible for recording the original Occurrence. The primary collector or observer, especially one who applies a personal identifier (recordNumber), should be listed first.»

So the later in time a collector worked on a specimen, the further after in the collector name list he should be written, and I would separate them by semicolon, because there could be reversed names with comma, like “Linné, Carl” or so. Hence my proposal would be to think in list values (always), e.g.:

Collector Search
In Command Line
JSON part

one: “Linné”
--collector-list "Linné"
{"result_list": [0:[{1st}, {2nd}, {3rd}, …, {max.10}]]}

multiple: “Linné; Linnaeus the Younger”
--collector-list "Linné; Linnaeus the Younger"
{"result_list": [0:[{1st}, {2nd}, {3rd}, …, {max.10}], [1:[{1st}, {2nd}, {3rd}, …, {max.10}]]]}

… where

{1st}, {2nd}, {3rd}, …, {max.10} means

the 1st possibility of a match,
the 2nd possibility,
the 3rd and up to maximal the 10th possibility of a collector search match, and

the zero-based index in {"result_list": [0:[…], 1:[…]]} indicates

0 is meant to be the primary collector
1 is meant to be the secondary collector etc.

Does it make sense? Any objections or improvements?
Good bye, Andreas

@infinite-dao Pleased to see that you're working on this. I suggest you visit with Sabine von Mering, Museum für Naturkunde if you haven't yet a chance to do so. That may be the quickest way to see & learn of others' work on this front. It touches on Attribution and so here's one place to see the other individuals involved https://www.tdwg.org/community/attribution/people/.
It's not clear to me if what you're looking for is:

clarity on how to structure search results,
how to structure objects about people that may aid in linking activities,
how to structure the link between person and object (what was their role & the activity they executed)

To be sure, there is some overlap between all these three needs but they are sufficiently different that a single structure (or sets of actionable tickets to do work) may not serve all three equally. However, if I understand what are your motivations, it seems to me that it's 2 above that is your primary driver. The first one gets you into issues of aliases, search patterns, etc. and the last one gets you into annotation space. Those are important, but less so than 2 for you at the moment. Correct?
So...what's the schema?
The Attribution group of folk have been working on an extension to DwC called "Agent Actions". It's long overdue & I'm the one who needs to give myself a kick. See https://github.com/tdwg/attribution/tree/master/people/dwc and the issues. Of course the schema does depend on what it is you want to do with it (i.e. 3 in your list) but if the business of making linkages to Person and specimen record are done elsewhere and the the openDS is merely the presentation of that effort, does your openDS schema about people need to store anything more than: name, alternateName, verbatimName, and an array of identifiers? And, leave it to whatever is behind those identifiers to store the (potentially contentious) demographical information.

Thank you David (@dshorthouse) for directing me to useful standardised terms.

It's not clear to me if what you're looking for is:

clarity on how to structure search results,
how to structure objects about people that may aid in linking activities,
how to structure a link between person and object (what was their contextual role & the activity they performed) AND who is it that made the link

The 4.6. People Linkage fixes on “Output: URIs with reference to entity. Multiple entities will be supported, but no distinction made between roles.”
So I want to provide a JSON scheme which makes sense by human reading and provides help for whatever decision later is made in the process (which is not my concern here right now), I’m concerned solely with providing automated (good) search results, so it regards mainly: “1. clarity on how to structure search results”
I will implement some remapping of my first proposal and I will post then again about a JSON search result structure.

Thanks for clarifying the intent @infinite-dao. That helps focus on the deliverables. Should it be of use, here is the Elasticsearch "schema" I use for people in Bionomia, https://bionomia.net: https://github.com/bionomia/bionomia/blob/master/lib/elastic_user.rb#L13. And then the query used to display search results uses cross fields in a multi match in Elasticsearch: https://github.com/bionomia/bionomia/blob/master/helpers/query_helper.rb#L8 that weights parts of names and aliases to improve scores. This may help you work backwards to ascertain what are the necessary key/value pairs in the model to support such basic functionality like search or autocomplete to accommodate the vagaries of how human names may be expressed when family names and given names may be swapped (eg John Peter vs Peter John).

Well @dshorthouse, now I become aware, that I do not have to invent the wheel twice ;-) perhaps. With your work on gathering person data of biological fields at https://bionomia.net I could just merge the results into the openDS herein and be done with it more or less. I’m reading into https://bionomia.net/developers and I …

see the API user.json as JSON

provided by e.g. https://api.bionomia.net/user.json?limit=5&q=Carl+Linné which gets probably the most suitable data packages that we need, I’m inclined to use this, and I …

see the the API with more deep analysis as JSON-LD

provided by e.g. https://api.bionomia.net/users/search?limit=5&q=Carl+Linné which I feel is perhaps to much data overload for the openDS itself but may serve for deep analysis OR …
I could just extract parts of it, e.g. "dataFeedElement"

As there could be more than one name to be searched for, the outer openDS could keep the same {"result_list": [0:[…], 1:[…]]} but inside there would be just (modified) binomia api results and that’s it. And as long as the searched persons are within the “world of natural history specimens” binomia seems a suitable gate to search through than to query wikidata manually.
I wonder, what IDs you plan to provide or if you want to keep it simple and provide WikiData-ID and ORCID only?

I wonder, what IDs you plan to provide or if you want to keep it simple and provide WikiData-ID and ORCID only?

It'll be purposefully simple because wikidata itself is a much better place to gather other identifiers and reconcile them against people entries. Keeping pace with wikidata, ORCID, and GBIF is hard enough.
I should say that there are consequences to the high-level decision in Bionomia that living people come from ORCID, deceased come from Wikidata. Living collectors/determiners/taxonomists who do not have an ORCID (but may have a wikidata entry) will not be in Bionomia & will not be in your search results. And, as this is a crowd-sourcing project of sorts, it is as complete or incomplete as we make it.

Closing this old ticket. SDR has been finished and we have made some progress in the DiSSCo sandbox with ORCID integration. We can make use of bionomia API if needed. The annotation framework will provide options for the community to add wikidata items to the collector names.


Outdated links

For more information, see Backup and restore considerations for virtualized domain controllers and Planning for Active Directory forest recovery.
The two links in this sentence go to documentation marked as no longer being updated (for WS 2003 and WS 2008). Is this still accurate?
The Backup configuration page states "Restoring a domain controller (DC) VM that is part of a multi-DC configuration is supported only through PowerShell." and links here, but the "only through PowerShell" part is not described here.
https://docs.microsoft.com/en-us/azure/backup/backup-azure-arm-vms-prepare
I don't see a PowerShell procedure for restoration at either of the obsolete links.
https://docs.microsoft.com/en-us/previous-versions/windows/it-pro/windows-server-2008-R2-and-2008/dd363553(v=ws.10)#backup-and-restore-considerations-for-virtualized-domain-controllers
https://docs.microsoft.com/en-us/previous-versions/windows/it-pro/windows-server-2003/cc786327(v=ws.10)
Please update this and the page that links here to make the recommended procedure regarding backup of DCs clear.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 66081bde-1e22-dd9c-8357-a0485b23b0a7
Version Independent ID: eb2a52a5-d290-84fb-c3fe-2a5d8512669a
Content: Azure Backup: Restore virtual machines by using the Azure portal
Content Source: articles/backup/backup-azure-arm-restore-vms.md
Service: backup
GitHub Login: @markgalioto
Microsoft Alias: markgal

@dcbrown16 Thanks for the feedback! We are currently investigating and will update you shortly.

The up to date Forest Recovery link is here:
https://docs.microsoft.com/en-us/windows-server/identity/ad-ds/manage/ad-forest-recovery-procedures
I haven't been able to find an equivalent to the "Backup and restore considerations for virtualized domain controllers".

There are some other relevant links here:
https://docs.microsoft.com/en-us/windows-server/identity/ad-ds/get-started/virtual-dc/active-directory-domain-services-virtualization
@markgalioto is this item under consideration? We receive frequent inquiries through Pro Direct about backup of Active Directory servers on Azure VMs and there is a paucity of clear guidance from the Azure side.

@Adam-Smith-MSFT, please assign this issue to GeethaLakshmiG

Thanks! These links are now fixed. #please-close

Oh, thanks @rayne-wiselman! I heard those pages were supposed to be updated eventually but I guess it hasn't happened yet. I'll check in again in case we find the links break when the update does happen.


Remove leshan-all module  ?

Currently when we release, we deliver a leshan-all module.
But we have serious doubt about its usefulness ?
We are considering to remove it from repository.
Is it actually used by someone ? Are you thinking this is a useful module ?

As nobody sounds to be really interested by this module, I prepared a PR #706  which remove it from.

leshan-all is now removed from master


Lib.rs Example and Async Block Callout (#1203)

Not ready for merge.
This example is meant to address #1203 providing clarification of how context is created and passed through macros and the async keyword.
The code in examples folder is currently meant to be temporary (quick compilation of single example), and to be removed when example is finalized.
Re-open from https://github.com/rust-lang-nursery/futures-rs/pull/1220

@MajorBreakfast Another pass at the docs, cleaned up code per your suggestions.

Not ready for merge.

@dbcfd are there more changes you want to make to this? Are you interested in getting it updated and landed?

@cramertj Was waiting on feedback, but if it seems useful, can rebase it and prepare it for merge.

It seems useful to me! Thanks

@cramertj Updated based on changes to the crate (e.g. spawn! removal). The examples are now full examples and referenced in the workspace.

Looks like there are some test failures

@cramertj Noticed in fixing those that minimum version is still 1.36. Is that going to move to 1.39, or should I flag this behind async-await feature?

I don't have much of a preference either way-- despite what CI says, the README only promises compatibility with 1.39 and later, so I have no objections to changing CI.

I don't have much of a preference either way-- despite what CI says, the README only promises compatibility with 1.39 and later, so I have no objections to changing CI.

FYI: filed #1966 to fix ci issue.

@cramertj Can you review this again, I added the examples to the Cargo.toml, and fixed things up against latest master.

Looks reasonable to me! thanks!


TICKET #504 NEEDS TO BE FULLY ADDRESSED - PLEASE RECONSIDER YOUR DECISION

Thanks for this @tschubotz
However, I think it worth noting that 25% reward is one important thing, making the precious $SAFE token going to right hands is another and that's the essence of this exercise.
My added comments were as a result of the timing constraint. I spent more hours typing the notes to ensure that addresses identified as sybils are convincing enough and that's the reason I was late but the fact is that all the works were done at the same time.
Please reopen this ticket to ensure:

My 4 days efforts without good rest doesn't go in vain
Precious $SAFE doesn't go into wrong hand
The aim of this exercise is fully achieved

I happened to be the last person to submit report because I took my time digging in order not to be a copy cat.
The section you want ignore contains more sybil addresses than the ones you've attended to.
Attending to the other sections shouldn't be seen as unfair to anyone if their report doesn't contain my findings and I was only hours late not days because I don't want to be a copy cat.
:exclamation: I have filtered addresses found by others before I do and I see I have a valid 596 addresses that are either Sybil or Relayer's
Approving only 73 out of 596 is a wasted effort for me
PLEASE RECONSIDER YOUR JUDGEMENT
PLEASE RECONSIDER YOUR DECISION AND ATTEND TO OTHER ADDRESSES in TICKET #504. 🙏
Cc: @lukasschor

Hey @CaptainTee. I'd support @tschubotz reasoning here.
While I very much appreciate the effort you put in, we want to stick to the rules and not reconsider the deadline. We have also disregarded several other reports submitted after the deadline.
Additionally, I sampled some Safes from #504 and they seem to show legit usage. Especially some Safes showing usage as far back as 2019/2020, where the Safe Token was not even announced. Therefore it is unlikely that they are systematic airdrop farmers.

While I appreciate your quick response @lukasschor
I still think many addresses in the Sections ignore belongs to farmers, you're chanced to lay your hands on the SAFE addresses that were created many years ago.
(

See attached screenshot)
I truly wish you Reopen it and keep my morale alive.
Rendering 4 days with sleepless nights works useless will definitely be painful

Why would you think a Safe created in 2019 can be in any way related to airdrop farming? At that point we didn't even have the Uniswap airdrop, so airdrop farming was not even a thing back then. I do understand that this is not the result you hoped for, but hope for your understanding that we:

do want to stick to our rules
do not want to risk having legitimate users excluded from the airdrop

I agreed that airdrop farming wasn't a topic for discussion in 2019 but it is also important that we don't let a single entity which could have possibly forget about SAFE controls larger percentage of the token.
Those SAFEs weren't used since the creation date.
Making someone who hasn't used SAFE products in the last 3 years have the possession of large token which automatically give larger voting right determining the direction of the SAFE in the future is not too satisfactory.
However, I have filtered those addresses reported and now, I'm only left with 523 from 2000 addresses reported. This should take you lesser time to work on
I will appreciate if you work on it for the next few minutes. This remaining 523 list is summarised in the below Excel sheet
https://docs.google.com/spreadsheets/d/15aSkXazOxbOK-_JojsqUfuvYcOfPR5wF/edit?usp=drivesdk&ouid=104966231854140804909&rtpof=true&sd=true

Cc: @lukasschor

Making someone who hasn't used SAFE products in the last 3 years have the possession of large token which automatically give larger voting right determining the direction of the SAFE in the future is not too satisfactory.
I don't consider this as valid reasoning. The airdrop criteria did not include any "last active" metric. To the contrary, early users actually contributed much more to the success of the project than recent users, as they took more risks.

Work on those 523 addresses and keep the 25% reward into the Treasury.
It's painful to see my findings not being checked at all

Look, with all due respect. I have checked about 20 Safes and all of them are legitmate users. With Mesa, a lot of users were onboarded to Mesa and conducted a similar sequence of transactions. Therefore they look like airdrop farmers, but they are not.

Thanks for this, I appreciate :pray:

@CaptainTee enemy of progress with African mentality and selfish interest.
Go and huge transformer