[$1000] Web - Tooltip is not displayed if hovered the users which are down the list

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:
Action Performed:

Go to staging dot on web chrome
Go to search
Hover over the first few profile avatar of the users and notice that tooltipis displayed
Scroll down the list up to bottom and try to hover on the profile avatar (not on the display name/email) and notice that tooltip is not displayed

Expected Result:
Tooltip should be displayed if hovered the users which are down the list
Actual Result:
Tooltip is not displayed if hovered the users which are down the list
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[x] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: <IP_ADDRESS>
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/93399543/f02808d3-8316-410e-bcbc-9452863b05cd
https://github.com/Expensify/App/assets/93399543/9649a7b5-754a-4e37-bc6e-3ee7626060ba
Expensify/Expensify Issue URL:
Issue reported by: @priya-zha
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1686629969783219
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~018ca8b2c30604ca26
Upwork Job ID:<PHONE_NUMBER>159539200
Last Price Increase: 2023-06-20

I can reproduce this!

Proposal
Please re-state the problem that we are trying to solve in this issue.
For some elements in the search (personalDetails) the tooltip is not displayed if hovering the image
What is the root cause of that problem?
The searchPage has a list of OptionRows, each element uses a component called MultipleAvatars to render de image, even tho the prop of shouldOptionShowTooltip this line calculates the tooltip content:
const tooltipTexts = props.shouldShowTooltip ? _.pluck(props.icons, 'name') : [];
For the section of personalDetails  in the searchPage the icons are calculated using the method getIcons in ReportUtils.js
Since there are no reports the method only returns the source of the image.
What changes do you think we should make in order to solve the problem?
Update getIcons and include a new parameter called defaultName
function getIcons(report, personalDetails, defaultIcon = null, isPayer = false, defaultName = '') {
    const result = {
        source: '',
        type: CONST.ICON_TYPE_AVATAR,
        name: '',
    };

    if (_.isEmpty(report)) {
        result.source = defaultIcon || Expensicons.FallbackAvatar;
        result.name = defaultName || '';
        return [result];
    }

So it can be sent in this line
result.icons = ReportUtils.getIcons(report, personalDetails, UserUtils.getAvatar(personalDetail.avatar, personalDetail.login),false,personalDetail.login);

Image of solution working in personalDetails section:

What alternative solutions did you explore? (Optional)
At the beginning I thought this was a backend issue but after seeing Onyx I found that it was about handling the data stored.

Proposal
Please re-state the problem that we are trying to solve in this issue.
Web - Tooltip is not displayed if hovered the users which are down the list
What is the root cause of that problem?
Easy step to reproduce this bug is - Create a workspace invite non-existent user, after that try creating the group ( the new user will appear) and it will not have tooltip.
The root cause of this problem is that some rows doesn't have UserDetailTooltip wrapped around them.
This mainly happens because when we send option prop to OptionRow it has icons field, and in that - name is empty.
While calculating the icons -
https://github.com/Expensify/App/blob/0b7455fa832b53eccb99ca23c70e131b3464fb1d/src/libs/OptionsListUtils.js#L514-L521
We call getIcons() method - And if the user hasn't take a part in any conversation the report is null or probably policyExpenseChart
https://github.com/Expensify/App/blob/0b7455fa832b53eccb99ca23c70e131b3464fb1d/src/libs/ReportUtils.js#L715-L815
We return -
https://github.com/Expensify/App/blob/0b7455fa832b53eccb99ca23c70e131b3464fb1d/src/libs/ReportUtils.js#L716-L720
as it never reaches the -
https://github.com/Expensify/App/blob/0b7455fa832b53eccb99ca23c70e131b3464fb1d/src/libs/ReportUtils.js#L814
That's why name for icon being empty - we don't wrap it with UserDetailTooltip.
What changes do you think we should make in order to solve the problem?
We need to handle the case for getting the icons when the report is empty, using result.participantsList, we have a method getIconsForParticipants() we can pass result.participantList (it will be array of accountID) and personalDetails and get the icons.
This will ensure that all the users have the correct set of icons.
What alternative solutions did you explore? (Optional)

@sobitneupane did you have the chance to review the proposals above? Thanks!

Proposal from @daordonez11 looks good to me.
🎀 👀 🎀 C+ reviewed

Oh my bad. 😂 thanks for the explanation @sobitneupane I thought you were already doing the assignation.

@0xmiroslav  mi proposal is about the first if when no report arrives, which hasn't been touched in the PR.

This is more like a minor improvement  rather than a bugfix:

I mean @grgia since you are already focused on this topic you could just implement the solution proposed it seems it is just a border case for personalDetailsSeciton in searchpage but might fix the issue in other points

I will be OOO until July 5, so assigning another team member. SO for reference.

@0xmiroslav I don't think the PR will affect the behavior being discussed here. So, we should be good to create PR based on https://github.com/Expensify/App/issues/21099#issuecomment-1599809305 proposal

I have reported a bug related to avatars, and I tried understanding the root cause. That is same as of this one.
As mentioned in the contributing guidelines, I should mention the PR that has fixed the bug which I reported. Shall I wait till that gets merged or mention the bug(slack convo link) here?

@luacmartins just needs to assign daordonez11

As mentioned in the contributing guidelines, I should mention the PR that has fixed the bug which I reported. Shall I wait till that gets merged or mention the bug(slack convo link) here?

@BhuvaneshPatil if you think you caught this bug earlier, please tag the PR and bug report here so we can check timestamps. We'll see the PR get merged if it is tagged

@muttmuure The bug is not the same. But when this PR will get merged, the bug will be resolved
This is slack convo link - https://expensify.slack.com/archives/C049HHMV9SM/p1687628052221969

Thanks for the assignation @luacmartins, just applied in Upwork, I expect to have the PR ready by tomorrow June 27th @sobitneupane

PR Ready https://github.com/Expensify/App/pull/21696 @sobitneupane  only waiting for high traffic account to test but still, the PR doesn't change anything in terms of performance

NVM already tested high traffic everything working fine!

Hey @sobitneupane @luacmartins  is the rule of the #urgency bonus specifically 72 hours? This PR was closed on the 3rd business day after assignation so I want to know if the bonus does apply on this issue. And it was due to a variable name change 😅

Noted thanks for the info! @luacmartins

BugZero Checklist: The PR fixing this issue has been merged! The following checklist (instructions) will need to be completed before the issue can be closed:

[ ]  [@sobitneupane] The PR that introduced the bug has been identified. Link to the PR:

Probably this PR or might have caused by some PR involving accountID migration.

[ ]  [@sobitneupane] A discussion in #expensify-bugs has been started about whether any other steps should be taken (e.g. updating the PR review checklist) in order to catch this type of bug sooner. Link to discussion:

https://expensify.slack.com/archives/C049HHMV9SM/p1688977644296659

[ ]  [@sobitneupane] Determine if we should create a regression test for this bug.

Yes.

[ ]  [@sobitneupane] If we decide to create a regression test for the bug, please propose the regression test steps to ensure the same bug will not reach production again.

Regression Test Proposal:

Login with a high traffic account
Go to search
Hover over the profile picture of the user without previous chat history in the search list
Verify that the tooltip is displayed.

Do we agree 👍 or 👎

Payment Requested on Expensify.

@sobitneupane I will review and process the payment by EOD.

@daordonez11 - what's your profile in Upwork? I can't find you there.
@priyeshshah11 - what's your profile in Upwork? I can't find you there.
@sobitneupane - I sent you an offer in Upwork!

I think you might have wrongly tagged me instead of @priya-zha

@isabelastisser I have requested payment and will be paid through Expensify App.

@priya-zha, I sent you an offer in Upwork.
@sobitneupane, I don't see a payment request from you in Upwork, please accept the offer I sent you today, and I will process the payment thorough Upwork.

[@isabelastisser] Link the GH issue for creating/updating the regression test once above steps have been agreed upon:

https://github.com/Expensify/Expensify/issues/299030

@isabelastisser https://www.upwork.com/freelancers/~01e96b8da58232f028 here is my link! thanks

Paid @sobitneupane


put a ArrayList to a property of Vertex

Expected behavior and actual behavior
I want put a ArrayList to the property of nickname,but I couldn't deal it.
OrientVertexType pvertexType = graph.createVertexType("Entity");
pvertexType.createProperty("nickname", OType.LINKLIST,pvertexType);
Vertex vertex = graph.addVertex("class:Entity");
ArrayList testList = new ArrayList();
testList.add("a");
ver.setProperty("nickname", testList);
Exception in thread "main" java.lang.ClassCastException: java.lang.String cannot be cast to com.orientechnologies.orient.core.db.record.OIdentifiable
Steps to reproduce the problem
Important Questions
Runninng Mode

[ ] Embedded, using PLOCAL access mode

OrientDB Version

[ ] v2.2.x - Please specify last number:

Operating System

[ ] Windows

Java Version

[ ] 7

Hi @NLPScott
You should use the following instead:
pvertexType.createProperty("nickname", OType.EMBEDDEDLIST);

Thanks
Luigi

thank you very much @luigidellaquila


[HOLD for payment 2023-08-08] [Performance] Show cached onyx content right away

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

What performance issue do we need to solve?
We are trying to make Onyx more performant by default. Right now, onyx always starts in a loading state, then collects all the
requested keys and sets them to the component state. This is often times inefficient as all the data is already available in a cache which we can read synchronously.
For exact technical details see: https://github.com/Expensify/react-native-onyx/issues/269
What is the impact of this on end-users?

Unnecessary loading states: We often start by default in a loading state, although we could show the content right away. Showing the content directly provides better UX
Performance impacts due to a lot of re-renders: Due to this architecture a component re-renders a lot until it reached its final loaded state. This comes with a performance cost creating janky transitions to screens.

List any benchmarks that show the severity of the issue
See: https://github.com/Expensify/react-native-onyx/issues/269
Proposed solution (if any)
See: https://github.com/Expensify/react-native-onyx/issues/269
Once we have the onyx changes in place we need to update onyx in the app and adjust a few screens to work correctly. Some screens are build with the assumption that we will always start in a loading state.
We will start by adjusting the ReportScreen. The following adjustments need to be made (as discussed here):

Defer the loading of new chat messages to the next batch, so it doesn’t block the initial rendering by changing a state variable on the report.
There was recently a firstRenderRef introduced, which delays the real rendering of the report screen after it has been mounted. We want to get rid of it to show the available content right away.

List any benchmarks after implementing the changes to show impacts of the proposed solution (if any)
See: https://github.com/Expensify/react-native-onyx/issues/269
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android / native
[x] Android / Chrome
[x] iOS / native
[x] iOS / Safari
[x] MacOS / Chrome / Safari
[X] MacOS / Desktop

Version Number: x
Reproducible in staging?: yes
Reproducible in production?: yes
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
Expensify/Expensify Issue URL:
**Issue reported by: @hannojg **
Slack conversation: https://expensify.slack.com/archives/C01GTK53T8Q/p1688580518417739

@hannojg I assume you want this assigned to you since you'll be working on it, right?

Yes, correct, thank you!

The PR is ready for review:

https://github.com/Expensify/react-native-onyx/pull/280

This is on hold as the work and discussions are happening here:
https://github.com/Expensify/react-native-onyx/pull/280#issuecomment-1644245983

This has been deployed to App @hannojg we still need to update the components to work correctly with the change right? Or can we close this?

We need to pay $1000 to @mollfpr for their review and testing

Offer sent @mollfpr https://www.upwork.com/nx/wm/offer/26071692

Paid


chore: fix whitespace issues in the CVSS object

Related Issue:
None.
Description of changes:
The objects/cvss.json object contains description that misses a whitespace but an additional whitespace within the <code> block.
The {{ pattern in the example within the current description string also causes confusion with readers that it is related to templating syntaxes such as handlebars or liquid.

Well spotted Ron, I support this one.

Thank you @taddhar @floydtree @irakledibm and @mikeradka !

Fantastic! I was busy already with calls and will return to this in the coming 2 hours.


Self-closing tag

Hello! Thx for your package! Could you please tell, how I could create  such tag
SOAP-ENV:Header/
I tried { 'SOAP-ENV:Header': [] },but it becomes
SOAP-ENV:Header
</SOAP-ENV:Header>
Not self-closing tag

Hi, i think you mean something like this:
xml({
  'myTag': [],
})

this will build into
<myTag></myTag>

i assume you want it to result in <myTag/>?
i have had the same issue and found that if you set the value of your tag to null. it will close it's own tag
(eg: xml({ myTag: null }))
this is a neat feature, but it doesn't seem work when dealing with _attr. for example:
xml({
  myTag: [
    { _attr: { myAttrib: "myVal" } }
  ]
})

will result in <myTag myAttrib="myVal"></myTag>
i hope this helps. and i hope this feature could be added with maybe a closeEmptyTag option.


uv export --without-hashes

Please could you add a --without-hashes option to uv export similar to pdm export

What's the motivation for excluding hashes?

I'm trying to migrate an existing process. There is a CI pipeline stage which installs dependencies with a requirements.txt which fails with:
ERROR: In --require-hashes mode, all requirements must have their versions pinned with ==. These do not:
    sqlglot[rs] from .....

Previously I was generating the requirements.txt using pdm export --without-hashes and this was working. I'm not 100% sure why it doesn't like the hashes. I'm guessing it could be related to the order in which the index and extra-index-urls are specified, perhaps there is a different index resolution order between the environments where I ran uv lock.
Unfortunately I don't own the code in the ci pipeline stage, otherwise I would just change it to use uv sync

I wonder whether this could be mitigated if uv export were to include the index-url and extra-index-url in the requirements.txt. I know that pip-compile does this. I will test this theory.

Thanks. Is that error coming from a pip install call or a uv call?

pip install

I suppose it's reasonable to allow this given that pip doesn't have a way to turn it off.


Need ability to change parameters/arguments while submitting/re-submitting workflows in Argo UI

Summary
Need ability to change parameters/arguments while resubmitting workflows in Argo UI
Use Case
In case of failures or if we want to run the workflow again with different parameters/arguments, we need ability to do so using Argo UI

Thank you @amoldharamshi I'll speak to my colleagues about your request.

Might be good to fix #3453 at the same time. I'm imagining a new panel that pops up similar to submitting from workflow templates. That would be neat.

I added the ability to set parameters in the workflow submit pane. It's a little trickier to add this in the workflow resubmit pane. I will bring this up with the team tomorrow

I added the ability to set parameters in the workflow submit pane. It's a little trickier to add this in the workflow resubmit pane. I will bring this up with the team tomorrow

Done

Done

Re-opening so we can do this.

Hi,
it would be really useful to allow to the client function resubmit to to this.
Something like
argo resubmit -n argo -p UPDATED_PARAMETER=updated_value EXECUTED_WORKFLOW
The use case is pretty clear, you want to retry a workflow changing some parameters.

This issue is about UI. Please raise new issue.

I'd like to take this one, please. My understanding is that the resubmit UI should have an option in addition to the OK and Cancel buttons to update parameter values before submission. I'm not sure of the best solution for all cases.
For workflow templates and cluster workflow templates, I think it would be good to redirect to the template submission view with the inputs pre-filled on the selected entrypoint, since that would help give a consistent experience submitting workflow templates from the UI.
The difficulty comes for non-template workflows. It seems like we would need a new submit panel that did essentially the same thing as the template submission, but for a workflow that doesn't have a defined template. I don't want to duplicate all the same submit logic in a new place; would it be possible to use the same workflow submission form for both workflow templates and free-form workflows?

Looking through the code a bit more, this actually seems like it's largely already solved in this case:
https://github.com/argoproj/argo-workflows/blob/master/ui/src/app/workflows/components/workflow-creator.tsx
I'd like to propose the following behavior when clicking the Resubmit button:

For a [cluster] workflow template, the SubmitWorkflowPanel side panel is shown, populated with the entrypoint and parameter values that were used in the workflow. Submission will issue a submit request.
For a non-templated workflow, the WorkflowEditor panel is shown, populated with any information available in the executed workflow. Submission will issue a create request.

@toyamagu-2021 Can you comment here so that I can assign this to you?

@terrytangyuan
Thank you for your mention!


release privatedns in compatibale with @azure/ms-rest-nodeauth

issue:  https://github.com/Azure/azure-sdk-for-js/issues/10458

@chenjianfei2017 Please also change the version in XXXClientContext.ts

@chenjianfei2017 Please also change the version in XXXClientContext.ts

Done


How to get the KITTI [0-30)[30-50)[50-inf) type results using OpenPCDet?

How to get the results of distances for the Kitti dataset?
Thank you!

I'm also dying to know how to do it

Hello, have you implemented this function?


Better handle vendored trees

For some reasons there is a need of keeping the dependencies in the tree, vendored.
Istio tooling does not deal well when this is the case. Tools like linters, automatic generation of files, etc. fail when there is a vendor dir and go flags contain things like -mod=vendor.
I'm working on fixing this.

Can this be closed?


Change to vite since 1.7.1 breaks imports in

Appears that since 1.7.1, when this was moved to use vite instead of webpack, projects that import MODAQ can no longer import it. This happens because the behavior for react changed; there's no longer a react.default value defined.

Likely related to changes around esModuleInterop

Fixed by #162


Add count of empty/non empty values in stats

xsv stats gives a lot of useful information, and I use it a lot.
I'm currently missing one thing that may not be too much of a hassle to add.
Would you consider adding the count of empty/non empty values for each field?

Isn't there already a count of null values? I know at least the frequency command should help here.

Unless I missed something, stats just tells whether there are nulls, but doesn't give the count.
xsv frequency -a -l1 does the trick though, since nulls come first

Unless I missed something xsv frequency doesn't do "groupby" operations like mean, std and count.
As an input to a data science workflow, I'm working with EXIF metadata from 70k+ photos. The csv files is about 1.1GB and very sparse. I would like to select only columns that are mostly used.
This is the output of xsv frequency
$  head -5 metadata_frequency.csv
field,value,count
SourceFile,../data/images/6928025/6928025_3fff3925431f80e0dfc83d48e7229a10.jpg,1
SourceFile,../data/images/7031649/7031649_dce502452d13649aa6682060126e2bfb.jpg,1
SourceFile,../data/images/7198981/7198981_6b7e5ef7587dab908ff8864b7c2c4b55.jpg,1
SourceFile,../data/images/7224251/7224251_b68dec5933ec49d3be9213513c7a8e43.jpg,1

This is the output of xsv frequency -a -l1
$  head -5 metadata_count.csv
field,value,count
SourceFile,../data/images/6969761/6969761_9b6c7e93ee82a10a406a34c44d900c60.jpg,1
About,uuid:73abd4a8-2f89-11df-9d3c-a53e4fead10b,1
AccelerationVector,-0.9773887673 -0.006948695972 -0.2358681423,1
Accentuation,0.0,73

It just took the first occurence of "SourceFile" but did not "groupby sum" on it.

@mratsim Could you please create a new issue and describe the problem you're trying to solve? Your problem seems completely distinct from this issue. In particular, I don't understand what you expect to see and why you think "groupby sum" isn't happening.

Sorry, I wasn't clear enough.
What I'm looking for is counting the number of empty, or non-empty values for each columns like OP.
Basically xsv stats is already doing min, max, mean, median to summarize on all the values appearing in a column, I would like to add the count of non-empty values in xsv stats.

Next paragraph is an explanation on groupby sum and may be skipped
Using the database query terms, we would do a count to get the number of non-empty items in a csv column.
However, if we queried the output xsv frequency we would have to do a groupby the "field" column and sum on the matching "count" column to get the count of non-empty items. Those are equivalent.
From your example in the README, it's turning:
$ xsv frequency worldcitiespop.csv --limit 5
field,value,count
Country,cn,238985
Country,ru,215938
Country,id,176546
Country,us,141989
Country,ir,123872
City,san jose,328
City,san antonio,320
City,santa rosa,296
City,santa cruz,282
City,san juan,255
AccentCity,San Antonio,317
AccentCity,Santa Rosa,296
AccentCity,Santa Cruz,281
AccentCity,San Juan,254
AccentCity,San Miguel,254

into
field,groupby(field).sum(count)
Country,897327
City,1481
AccentCity,1402

897327 being 238985+215938+176546+141986+123872
Another way to look at it in term of functional programming/Rust is that we are folding (reducing) by summing the values associated to "Country, City, AccentCity" respectively
Given the format this should probably appear in xsv stats

Ah, gotya, thanks, that's clearer. I can think of a couple of work-arounds, although I suppose it would be nice to provide it by default. For example, you could do xsv search -s Column '.' foo.csv | xsv count to get a count of non-NULL columns (and xsv count foo.csvto give you a count of all rows).

I suppose it would be nice to provide it by default

:+1:
it would be very natural for xsv stats to return null counts for every column

@BurntSushi @seamusabshere I would be happy to implement this and submit a PR that's at least as tidy as what I submitted for xsv partition. :-) Are there any thoughts you have on how this should work?

What would a good output column name be?
Are the any backwards compatibility concerns about adding a new output column?

What would a good output column name be?

I think my only real opinion here is that it shouldn't contain spaces. nullcount or something?

Are the any backwards compatibility concerns about adding a new output column?

If there are, we can just release a new minor version, but I don't think so. In particular, I think CSV data with headers can be expanded with additional columns (and probably even re-orderings of those columns too).

A nullcount for the xsv stats command would be great!

Unfortunately, this is currently about 647 items down my TODO list, at least until the next time I need it for something at work. :-( If somebody else wants to tackle this, please don't wait for me!


Support streaming multipart/formdata with of-watchdog

Expected Behaviour
It should be possible to post a multipart request to the watchdog and work with the parts in such a way that Content-Disposition header information (such as the filename) is available to the function.
If there is only one part, feed it to stdin.
If possible, allow to access multiple parts by their names as stream, too.
Make content-disposition information available to the function similar to the usual request headers.
(The Content-Disposition header is defined as a response header only, but it may occur in multipart/formdata for requests).
Current Behaviour
There is no special support for multipart requests.
Possible Solution

Context
The related issue https://github.com/openfaas/faas/issues/344 asked for multipart support by means of a json object containing all base64 encoded parts, that might not be ideal in terms of memory requirements for big multipart requests.
In https://github.com/openfaas/faas/issues/345 it was concluded that support for multipart should go into of-watchdog

See https://github.com/openfaas-incubator/python-flask-template/pull/17 for a possible solution


[EOEPCA/IAM] Amend Helm chart

Add global HTTP-to-HTTPS redirect in a ApisixGlobalRule
Add apisix.pluginAttrs.redirect.https_port=443 property to default values.yaml

I found an unwanted side-effect with using the redirect http_to_https plugin in ApisixGlobalRule - which interferes with the function of the Cert Manager letsencrypt challenge using http (not https).
Anyway, I think I found the solution is to use a filter on the plugin so that it does not apply for the acme challenge...
apiVersion: apisix.apache.org/v2
kind: ApisixGlobalRule
metadata:
  name: redirect-to-tls
spec:
  plugins:
    - name: redirect
      enable: true
      config:
        http_to_https: true
        _meta:
          filter:
            # Exclude paths used by letsencrypt http challenge
            - [ 'request_uri', '!', '~*', '^/\.well-known/acme-challenge.*' ]


Provide examples/templates for arrangements of components

As a developer I want to speed up my development workflow and don't want to waste time by moving around components.
So why we don't add some example HTML code to speed that up? It's about how components fit together. How does it look good? At which spacing?
If you look at https://daisyui.com/components/footer, you'll see there are multiple layout variants for a footer.
If you talk about Design Systems in Brad Frost speech: Example HTML Code is all about Molecules and Organisms. Molecules for example are something like a text input and search button. We would never add such a trivial component to the elements, but we should add example code of how someone can achieve that with an elements' input and button component.
Organisms are a lot bigger: They may include Molecules but also contain simple/atomic components.
So my proposal here is to introduce new sections to demonstrate how to tie together multiple elements in any web application. Just by copying the HTML code. Let the user play around with the code by using StackBlitz or similar live tooling.

We could distinguish here between Components for the basic structure of WebApps and Components for common use cases.


Modify disable unuse langs

issue
https://github.com/linzhengen/lala/issues/4

Coverage remained the same at 62.95% when pulling fe13562662799a1446410521f30e715bb59b0cdf on feature/4_disable_langs into 93fc065a2a220dab80ea042d8fed9f0c81b2bd8e on master.


IMAPS + Letscrypt

Let's add IMAPS and use LetsEncrypt.
Let's borrow ideas for this from https://github.com/tomav/docker-mailserver

Hey James,
IMAPS should be an easy addition.
I would be inclined to make LetsEncrypt a module and therefore optional,
there is already a self-signed and own certificate option available in the core already -
LetsEncrypt would be another nice addition.
Will try to get some work on these features done soon, thank you for suggestions!

I agree; I really like the way you've architected your setup. I successfully got it up and running last night on a private domain. I think a letsencrypt should be a sepate container/image as well. While we're at it why don't we borrow from other docker images of mine and make the self-signed certs part another separate image too? e.g: Have a look at https://github.com/prologic/autodock-paas/blob/master/docker-compose.yml ; note the volumes_from: sslcerts and the mksslcerts image?

I've implemented IMAPS

Hi James,
I have added the imaps support from your branch.
It does not include the extra certificates you have in your fork, using standard certificates only.
Thank you for your work!

Letsencrypt is an awesome service, there are many images to use it already and it can be integrated with these email images without any changes. I would not want to write and maintain another copy :)

Cool thanks :)
James Mills / prologic
E<EMAIL_ADDRESS>W: prologic.shortcircuit.net.au
On Tue, Apr 26, 2016 at 10:25 AM, Valentin Arkhipov <
<EMAIL_ADDRESS>wrote:

Letsencrypt is an awesome service, there are many images to use it already
and it can be integrated with these email images without any changes. I
would not want to write and maintain another copy :)
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/adaline/dockermail/issues/12#issuecomment-214818889


Example typo fix

I am fairly certain the example meant to adhere C to the P protocol as well as inheriting from NSObject.

would you mind updating the respondToSelector syntax as well? It should be responds(to: "bar")

no problem!

This still generates a warning, but I think we can live with it, as an example.

thanks @voidref !


fix typo on connections page

This corrects "In for" => "In order for" on the connections page of the docs site

@AriLFrankel Please sign the Contributor License Agreement!
Click here to manually synchronize the status of this Pull Request.
See the FAQ for frequently asked questions.

@AriLFrankel Thank you for signing the Contributor License Agreement!

Thank you!


Application/Subscription approval log improvement

Patch_00058 integrated to component-dep from component-dep-support 2.4.2-release.

SonarQube analysis reported 5 issues

 3 critical
 1 major
 1 minor

Watch the comments in this conversation to review them.

SonarQube analysis reported 3 issues

 3 critical

Watch the comments in this conversation to review them.


Add tests and continuous deployments on heroku

What tests can we add to this?

Codecove
Codeclimate
Codacy
Travis
https://houndci.com/configuration (.hound.yml)
http://eslint.org/docs/user-guide/configuring (.eslintrc)

@sudheesh001 Could you please help to get this project on its feet and help to set up testing and automatic heroku deployments?

Please also set up a branch "development" and deploy from development to Heroku. Once a week please merge to Master.

Please give me the link to the Heroku deployment.

@leonmak Is the exact problem that you're facing here with respect to the settings task which is failing ? i.e. the custom_configFile.json isn't getting generated, What you'd need to do here is to set these as ENV variables instead and modify the code to use the value from JSON or the ENV value. https://www.npmjs.com/package/dotenv could be helpful.

@sudheesh001 Please help to do it.

@sudheesh001 I'm not familiar with the steps to deploy to heroku, I can push to it but it just keeps giving error

@mariobehling sorry for the delays, finally managed to set it up at tranquil-river-53177.herokuapp.com

@leonmak Took a minute to load. Will check it out. Thanks!
Could you please change the URL to loklak_wall.herokuapp.com?

@mariobehling ok it is https://loklak-wall.herokuapp.com/ , underscores not allowed

Thanks! Good job!


APT-619 changing radar modes

https://provizio.atlassian.net/browse/APT-619

Code looks good - compiles clean and all tests passing. I don't understand the point of provizio_wait_for_radar_mode_change() though. What's the need for this function/callback if the radar_mode is to be included in the header of point_cloud packets?

It's due to the standard (one of) use logic I anticipate (may use it in APT GUI, but maybe not - we'll see):

My system operates normally with point clouds coming from a radar (or radars)
Due to some event, like ego velocity change or moving from a motorway to a city, I want to change a radar mode and make sure my code now operates with the new mode.
I close an existent connection and stop its receive loop.
I set the new mode
I make sure it's been applied by calling that provizio_wait_for_radar_mode_change and even probably retrying #4 if it didn't happen in time - given UDP may sometimes loose packets.
When I know the radar operates in the new mode, I go back to 1.

I.e. it follows the simplest linear logic, that may often be very useful. In the same time, it doesn't have to be used, and after requesting the new mode you may just keep doing what you were, the radar mode will change at some point (unless it doesn't due to a lost UDP packet, but it should be a very rare thing in local networks).

@iurii-provizio would it not be simpler logic to process UDP packets in a loop and hand off the point clouds to a suitable handler based on the mode, e.g. something like
while(receive_udp_point_cloud_packet_blocking(packet)) {
  if(packet->radar_mode == foo) {
    handle_foo_point_cloud();
  }
  else if(packet->radar_mode == bar) {
    handle_bar_point_cloud();
  }
}

This saves on having to close and re-open the socket, and would allow the system to keep processing "old" radar_mode point clouds until the new radar_mode becomes active.  Alternatively, the logic could check for packet->radar_mode == expected_mode and ignore packets until the expected_mode becomes active (or timeout and retry changing modes).
This seems a more suitable / intuitive design pattern though I'll defer to your judgement on this as I'm unfamiliar with the implementation details of apt_gui.
Btw, do you expect radar_mode to be divided into bit-fields such that it can represent multiple radar configuration aspects or will it be exclusively used for radar range?

@iurii-provizio would it not be simpler logic to process UDP packets in a loop and hand off the point clouds to a suitable handler based on the mode, e.g. something like
while(receive_udp_point_cloud_packet_blocking(packet)) {
  if(packet->radar_mode == foo) {
    handle_foo_point_cloud();
  }
  else if(packet->radar_mode == bar) {
    handle_bar_point_cloud();
  }
}

This saves on having to close and re-open the socket, and would allow the system to keep processing "old" radar_mode point clouds until the new radar_mode becomes active. Alternatively, the logic could check for packet->radar_mode == expected_mode and ignore packets until the expected_mode becomes active (or timeout and retry changing modes).
This seems a more suitable / intuitive design pattern though I'll defer to your judgement on this as I'm unfamiliar with the implementation details of apt_gui.
Btw, do you expect radar_mode to be divided into bit-fields such that it can represent multiple radar configuration aspects or will it be exclusively used for radar range?

I changed now the way we validate mode changes, done safer and more conveniently. Also updated documentation. Please re-review.


Support 3rd Party CA

in ipc.go the newIdentity function does not allow for supplying of a key nor cert path. For third party CA's, the key and cert would need to be able to be specified.

will be coming with <IP_ADDRESS>


Repository gone??

Trying to start this container for unraid but it appears the repo is gone?
https://registry.hub.docker.com/r/wouterdebie/locast2tuner
404 error

Not sure where you got this from.. I never published to dockerhub.

https://github.com/RandomNinjaAtk/unraid-templates/commit/3c55aaac1ebe484f4b17e34efd3bacee263e2e7c Might have something to do with this though, but I don't maintain that.

Thanks... just noticed that.  Changing the repo field to: ghcr.io/wouterdebie/locast2tuner-unraid:latest seemed to do the trick and now it is working.  does using multiple zip codes pull channels from those markets as well?

Thanks... just noticed that. Changing the repo field to: ghcr.io/wouterdebie/locast2tuner-unraid:latest seemed to do the trick and now it is working. does using multiple zip codes pull channels from those markets as well?

yes it does

Thanks... just noticed that. Changing the repo field to: ghcr.io/wouterdebie/locast2tuner-unraid:latest seemed to do the trick and now it is working. does using multiple zip codes pull channels from those markets as well?

yes it does

Thanks!  Should I have changed the repo in your unraid container to make it work?  not sure why it threw the error that it was trying to pull from docker and couldn't find it.
For multi-region if I use multiple zip codes... can i get a channel say CBS in both markets to watch live sports from both?

For multi-region if I use multiple zip codes... can i get a channel say CBS in both markets to watch live sports from both?

yes you can.. somewhat off topic for this thread, so I'll close this issue.. If you want user help, please don't hesitate to join us on Gitter


feat/add task queue

Is your feature request related to a problem? Please describe.
Once "Stop the world" is happened, than quics-client needs to know that which tasks are working . Then wait til end of the transaction of each process and rests should be stopped.
Describe the solution you'd like
make a task queue, so that program recognize which task is working

type JType string

const (
	ConnectServer    JType = "CONNECT_SERVER"
	ConnectRoot      JType = "CONNECT_ROOT"
	DisconnectServer JType = "DISCONNECT_SERVER"
	Rescan           JType = "RESCAN"
	ShowStatus       JType = "SHOW_STATUS"
	ChooseOne        JType = "CHOOSE_ONE"
)

type JStatus string

const (
	Ready   JStatus = "READY"
	Working JStatus = "WORKING"
	Done    JStatus = "DONE"
)

type JPriority string

const (
	High   JPriority = "HIGH"
	Medium JPriority = "MEDIUM"
	Low    JPriority = "LOW"
)

type Job struct {
	Type     JType
	Status   JStatus
	Priority JPriority
	JobID    string
}

/* TODO
0. When start the Program -> Create JobQueue (JobQueue is a list of Job)

When Transcation gonna be started -> Create Job
When Job is created -> Add Job to JobQueue
When Job is added to JobQueue -> JobQueue is sorted by Priority
When Transcation is started -> Change Job Status to Working
When Transaction is done -> Change Job Status to Done
When Job Status is Done -> Remove Job from JobQueue
*/

/* DISCUSS

How to make JobQueue? Queue or Stack
How to sort JobQueue? Priority
Should Job queue is in memory or in disk(badger)?
How to make JobID? UUID
*/

/* DISCUSS
How to make JobQueue? Linked List
type JobQueue struct {
	mut sync.Mutex
	low *Job
	mudium *Job
	high *job
	urgent *Job

}

How to sort JobQueue? Do not sort
Should Job queue is in memory or in disk(badger)? in-memory
How to make JobID? UUID
*/

Delayed


Remove zipkin thrift endpoint from agent

Remove 5775/udp which accepts zipkin.thrift over compact thrift protocol
From https://github.com/jaegertracing/jaeger/pull/1181#discussion_r233632594

so nobody outside of Uber should be using those old clients. If so, we can remove that endpoint and re-keep it in our internal build only.

I would like to validate this internally first (we should have metrics).


Allow method chaining

Allow method chaining, so people can simply do:
$lexer = (new Lexer($quill))
    ->registerListener(new Mention())
    ->registerListener(new TableParser());

Rather than only:
$lexer = new Lexer($quill);
$lexer->registerListener(new Mention());
$lexer->registerListener(new TableParser());

tagged as version 3.3 => https://github.com/nadar/quill-delta-parser/releases/tag/3.3.0


Email content is not in array (to print it)

Feature request or bug
Bug
If a bug, what did you expect to happen?
Get an array with the content of the email somewhere in it, so that I can print the content
$email = $imap->getMessage((int)$_GET["msgno"]);
https://pastebin.com/4L5iawx3
If a bug, what happened?
Nowhere in the array is the content of the email, so I can not print it
If a bug, list steps to reproduce bugs.
If a bug, did you do these steps?
[x] Download and use the latest stable version
[x] See if the issue has already been reported
[-] Debug
If a feature request, what do you want to be added or changed?
If a feature request, is this feature already in a pull request?
If a feature request, do you know anyone who can help?

same here - empty message text
[message] => stdClass Object
(
[text] =>
)


Update ng2-datepicker.ts

Active class being implemented incorrectly. Please review the changes and merge.

I think this should work, thanks!


Teams 2.x Doesn't Logout

How to Reproduce

Log into MS Teams 2.x via Native Windows desktop application
From within the User Profile click the Sign out link
Windows OS will prompt me for system password
Enter system password
Teams 2.x appears to refresh and log me out
Click my account from the list of previously signed in accounts
Teams 2.x loads my account without any password requirements
This ^^ happens for any previously signed in account.   Especially insecure for Kiosk machines with multiple users sharing the same laptop/desktop.

Expected Behavior
I would expect that after I sign out of my account that I would have to re-enter my username/password to sign back in.
System Details
Edition	Windows 11 Enterprise
Version	22H2
Installed on	7/21/2023
OS build	22621.2428
Experience	Windows Feature Experience Pack 1000.22674.1000.0

Teams Desktop Details
You have Microsoft Teams version 23285.3604.2469.4152.
You've got the latest version. It was last updated on 11/10/2023.
The client version is 49/23101227708.

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

ID: 19ddf42e-0a47-7717-52d4-e549155480a2
Version Independent ID: 4bbe9beb-233f-cfd5-097b-f280aab5fde8
Content: Microsoft Teams developer community support and feedback - Teams
Content Source: msteams-platform/feedback.md
Service: msteams
GitHub Login: @surbhigupta
Microsoft Alias: lajanuar

We are mainly responsible for Microsoft Teams app development related issues.
For Microsoft Teams product issues/failures your tenant Admin can reach out to Microsoft 365 Product Support.

Same problem in our org. Actually Kiosk PC are not sure to use with new Teams.


Change mesheyctl to mesheryctl

…l.com>
Notes for Reviewers
Signed commits

[ ] Yes, I signed my commits.

🚀 Preview for commit 9a562ec2dd3bb5e4ed89ac37997a4fb6a00fb0cc at: https://66e0382e7e444a1776e70220--meshery-docs-preview.netlify.app


Autofocus issues on Android

I've been running into repeated issues with the camera not auto focusing on Android. Tap to focus works but otherwise it will stay at the focus level of whatever it was pointing at when the screen was opened. The one exception to this is if you push another page on top of the scanning page and then dismiss it then the camera will auto focus correctly as you move around to look at items closer and farther away.

Maybe, don't use a ScannerPage. Use their MobileBarcodeScanner.
As Igor Kravchenko stated at https://stackoverflow.com/a/66296505/20395535
ZXing.Mobile.MobileBarcodeScanner scanner = new();
ZXing.Result result = await scanner.Scan().ConfigureAwait(false);

if (result != null && !string.IsNullOrWhiteSpace(result.Text))
{
	// result.Text is ready.
}

Kudos to Igor.


Wallet with multiple accounts: sort

When a wallet has multiple accounts it'd be nice if they could be sorted by name and/or manually:

This is not supported, but there is an easy workaround: when you create your wallet, start with creating the 0th account first. Accounts will be sorted in numeric order.


Create Hebrew translation - strings.po

Thank you for this! I'll try and get a translation done for Danish as well as that's my mother tongue. Don't know why I haven't done that before.


Rename IMAGE_BOOTLOADER* variables to MENDER_IMAGE_BOOTLOADER*.

These are mender-specific; rename them so it's more obvious.
Signed-off-by: Drew Moseley<EMAIL_ADDRESS>

This goes with: https://github.com/mendersoftware/meta-mender/pull/517

For stability I won't cherry-pick a variable rename to rocko. Do you think we should merge (and cherry-pick) this PR immediately, even though rocko is still the most recent stable branch of ours? I think I lean towards yes, even though it is slightly premature. It won't be long before it's true and this way we won't forget.

Yeah, the independent versioning of meta-mender and mender-docs does cause some confusion here.  I think it's ok to have this one cherry-picked.

Alright, cherry-picked to 1.4.x and 1.5.x as well.


alpine 3.8

<!--
Feature

traefik 1.7 upgrade alpine images to v3.8

I'm pretty sure I wrote a description....
Humm

there you, description fixed

@ldez want to reconsider?

Fixed by https://github.com/containous/traefik-library-image/pull/35


error compiling cl-protobufs in order to install

I'm having problems installing cl-protobufs. I get errors like
g++ -std=c++14 -I/PATH/TO/MY/LOCAL/lisp/protobuf/include -I.   -c -o generator.o generator.cc
generator.cc: In member function ‘virtual bool google::protobuf::cl_protobufs::LispGenerator::Generate(const google::protobuf::FileDescriptor*, const std::string&, google::protobuf::compiler::OutputDirectory*, std::string*) const’:
generator.cc:62:5: error: ‘GOOGLE_CHECK’ was not declared in this scope
   62 |     GOOGLE_CHECK(annotations.SerializeToZeroCopyStream(meta.get()));
      |     ^~~~~~~~~~~~
generator.cc: In member function ‘virtual bool google::protobuf::cl_protobufs::LispGenerator::GenerateAll(const std::vector<const google::protobuf::FileDescriptor*>&, const std::string&, google::protobuf::compiler::GeneratorContext*, std::string*) const’:
generator.cc:135:5: error: ‘GOOGLE_CHECK’ was not declared in this scope
  135 |     GOOGLE_CHECK(annotations.SerializeToZeroCopyStream(meta.get()));
      |     ^~~~~~~~~~~~
make: *** [<builtin>: generator.o] Error 1

FWIW, I had to use different compilation steps for protobuf than what's included in your README, since protobuf seems to be currently set up for bazel or cmake builds. After trying various approaches, the following  are the steps that wound up successfully compiling protobuf and installing it (and its include/) to a local user directory for me. I don't get an error until I try to compile cl-protobuf:
export PROTOC_ROOT=/PATH/TO/MY/LOCAL/lisp/protobuf
mkdir -p /tmp/lisp/proto
pushd /tmp/lisp/proto

git clone --recursive https://github.com/google/protobuf
pushd protobuf
git submodule update --init --recursive

mkdir -p ${PROTOC_ROOT}
cmake . -DCMAKE_CXX_STANDARD=14 -DCMAKE_INSTALL_PREFIX=${PROTOC_ROOT}
cmake --build . --parallel 10

cmake --install .
path-prepend ${PROTOC_ROOT}/bin
popd # protobuf

git clone git@github.com:qitab/cl-protobufs.git

pushd cl-protobufs/protoc
make  #### ****  <-- error occurs here

popd # cl-protobufs/protoc
popd # /tmp/lisp/proto

@Slids I'm trying to build with Bazel instead of cmake because the protobuf/cmake/README.md only mentions Windows and Linux support, not macOS. If you export cl-protobufs/BUILD I can give it a shot.
Building protoc with bazel worked flawlessly...
git clone --recursive https://github.com/google/protobuf
cd protobuf/
brew install bazel
bazel build :protoc :protobuf
cp bazel-bin/protoc ~/bin/

I suspect I just need the following in my cl-protobufs/WORKSPACE file in order to build cl-protobufs/protoc, once I have the BUILD file.
workspace(name = "the-fantabulous-cl-protobufs-workspace")

local_repository(
    name = "com_google_protobuf",
    path = "../protobuf",  # assume installed in same dir as cl-protobufs
)

@Slids where are you?!!!  I needz my BUILD filez!

Using CMake as done in the yaml CI files will work post:
https://github.com/qitab/cl-protobufs/pull/438

Protocol Buffers and ABSL both had some major changes, and I only (just barely) updated to a newer version of Prtocol Buffer used.  Please check the SBCL-test.yaml file in the CI and the CMake file in protoc.


Update ReadME badges

Fixes #[issue number]
This PR is [not ready] for review.
Risk
This PR makes [no] API changes.
Testing Plan

[x] I have verified that I have not introduced new warnings in this PR (or explain why below)
[x] I have run the unit tests with this PR
[x] I have tested this PR against Core and verified behavior (if applicable, if not applicable, explain why below).
[x] I have tested Android, Java SE, and Java EE

Unit Tests
N/A
Core Tests
N/A
Summary
Updated badges in the ReadME
CLA

[x] I have signed the CLA

Cached tags updated, no need to update them


Species list API: CORS headers for decision tool

Make all public? https://pypi.org/project/django-cors-headers/

For other endpoints too? (consistency)

In fact, if the requests are only GET ones, the security concern is more on the client side than on the server side.
In all cases, if it appears that the API should be restricted in the end, the cors-headers can be configured to only accept requests from client viewing pages from a given URL. So this can be restricted to requests coming from BBPF applications as soon as they will be available online.

I'll enable CORS for all origins and limit it for now to GET requests. I think it's flexible (multiple URLs during development) and more than safe enough!

Done!
@sronveaux: can you check you can now access https://dev-alert.riparias.be/api/species without CORS issues now?

@niconoe: I've tested it just right now and it works perfectly!  Many thanks !


Byron initCursor does not seem to cause DB to rollback

Context

https://github.com/input-output-hk/cardano-wallet/blob/5333afc469294bfa74e07de3317578545661ea05/lib/byron/src/Cardano/Wallet/Byron/Network.hs#L259-L268
initCursor will find the intersection between the node and the wallet tip.
If intersection /= walletTip the wallet DB needs to rollback. I don't think we do that.

Information
-

Version

Platform

Installation

Steps to Reproduce
Imagined scenario:

Sync the chain
Stop the wallet. Wait for the network to fork.
Start the wallet again.

   +- B2a
  /
B1
   \
    +- B2b

Wallet will send [B1, B2a] as known points.
Node will select B1.
Node will send B2b.

Expected behavior
Wallet should rollback to B1, such that it can switch to the new chain and apply B2b.

Actual behavior
(Imagined, not tested ⚠ )
Wallet will stay at B2a. When trying to apply B2b it will crash because of an invariant.

Resolution

QA

So, after investigation this scenario doesn't look plausible. Here's why:

When the restoration loops start, the wallet initialize a cursor using a list of checkpoints.

cps <- liftIO $ atomically $ listCheckpoints (PrimaryKey wid)
[...]
initCursor nl cps

For initializing the cursor, the list of headers is sent to the node via a FindIntersect message and the cursor is initialized at the intersection point:

chainSyncQ `send` CmdFindIntersection points >>= \case
  Right (Just intersection) -> do
      pure $ Cursor intersection chainSyncQ

Then, we know from the protocol specification that the first response to a RequestNext after setting an intersection will necessarily be a RollBackward to that intersection point. Therefore, if the intersection is found anywhere within the list of headers, we'll first start by rolling back to that point.

Each rollback restarts the restoration loops exactly as if the whole process was starting from the beginning, so this bring us back to step 1. Now the question is, how and when do we ever break that loop and continue restoring as normal? This is where the trick is: we do not treat intersection /= walletTip as an exceptional case, but we do treat intersection == walletTip as one! Any request to roll backward to the wallet's tip is indeed ignored:
Right (RollBackward cursor') ->
    -- NOTE
    -- In case the node asks us to rollback to the last checkpoints we
    -- gave, we take no action and simply move on to the next query.
    -- This happens typically with the Haskell nodes which always
    -- initiates the protocol by asking clients to rollback to the last
    -- known intersection.
    case (cursorSlotId nl cursor', cps) of
        (sl, _:_) | sl == slotId (last cps) ->
            step delay0 cursor'
        (sl, _) ->
            pure (FollowRollback sl)

There's one slight limitation / gotcha with this approach. In the step 4, the cps refers to the original list of
checkpoints given to the follow function. So we could imagine a scenario where:
a. The intersection is initially found at the tip.
b. A few blocks are added on top
c. Later, a rollback is requested to where the tip was (but no longer is).
That last rollback requests in c. would just be ignored :/ ... As a consequence, next blocks would fail to apply,
but the worker would recover itself by restarting from the start, looking back for a new intersection. So, not as pretty
as it could be, but also not a significant problem. I do not think changing anything here would be wise provided we intend
to undergo more drastic changes on the network layer approach.


need dev package in terminal post

was following along in https://donniewest.com/build-your-own-minimal-and-scriptable-terminal,
for setting up your machine

sudo apt-get install build-essential libqtermwidget5-0

should be:

sudo apt-get install build-essential libqtermwidget5-0-dev

without the -dev package, you don't get the headers, so pkg-config will fail and the build will fail.
Cool post, thanks!

Not sure how I missed this. Thanks!


Support/parse nested json with serde_json::from_value

I'm playing with tauri-plugin-sql and thinking about moving my app (https://github.com/iohzrd/identia) to using it but I've noticed that when selecting a row that contains columns that contain arrays, objects, whatever, they don't get parsed and are returned as strings.
It would be nice to recursively parse so that no extra parsing is required in JS land.
This is something I was able to accomplish using serde_json::from_value, as can be seen here. I'm happy to implement it myself and submit a pull request, but I figured I'd start a conversation first, so I can get a feel for the best way to proceed. I.e. should we target BLOB, STRING etc...

pull request with simplest possible implementation: https://github.com/tauri-apps/tauri-plugin-sql/pull/92

I think using from_value for STRING and leaving BLOB as an array probably makes the most sense.

hmm, my first PR seemed to work the first time I tested it, but is producing strange behavior upon further inspection. I'll follow up...

After playing with it a bit more, I think the best way to proceed is to use serde_json::from_value for BLOB columns, and leave the rest as is. This way we can retain support for array values, (as it is)[https://github.com/tauri-apps/tauri-plugin-sql/blob/dev/src/plugin.rs#L251], but also support other JSON objects.

tbh personally, i consider this out of scope and would just recommend using TEXT/VARCHAR (seems to be the commonly used default for json in sqlite) + JSON.parse in the frontend

@iohzrd I agree with @FabianLars ... I think the frontend being required to parse a string value is a good solution for now. I'm going to close this for now but if you really want this feature maybe you can setup a zoom call with me and we can discuss why this is worth doing.


enable ppc64le aarch64

Update conda-forge.yml
MNT: Re-rendered with conda-build 3.21.8, conda-smithy 3.19.0, and conda-forge-pinning 2<IP_ADDRESS>.48.06
Update conda-forge.yml
MNT: Re-rendered with conda-build 3.21.8, conda-smithy 3.19.0, and conda-forge-pinning 2<IP_ADDRESS>.48.06
MNT: Re-rendered with conda-build 3.21.9, conda-smithy 3.20.0, and conda-forge-pinning 2<IP_ADDRESS>.26.41
enable aarch64; pin osx compiler; bump build number

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@conda-forge-admin  rerender

@conda-forge-admin rerender

@conda-forge-admin rerender

@njzjz Thanks a lot! I am not expert on conda-forge, so I only review your additional commits. It looks all good and seems the aarch64 forge is generated.
Cheers!


WIP: Pick the requested channel type early

So we don't have to worry about it during later processing steps!

Oh wow this went smoother than I had expected!

Just pick afterwards

CIs complain but diff looks good

Hopefully it works now.
Before ba14f2e:

After ba14f2e:

thx @hoechenberger !


Introduce custom executable id

Before this commit the unique identifier for an executable file or object file was the build id. As the GNU's build ID might not be present, we fall back to the sha256 hash of the .text section. This works fine but takes quite a bit of space as we refer to it in every mapping.
By using a 64 bit id derived from the first 8 bytes of the sha256 hash of the code, we can save a good amount of memory, make comparisons and hashings of these IDs faster, and helps simplify the code. For example, there's no need for a separate ID in the native unwind state. There might be a bit more CPU and IO needed to get the hash, but this might be something we can cache in the future.
This is also useful for future profile formats that require unique 64 bit IDs for object files.
This commit also fixes other minor things, such as using the lower hexadecimal representation.
Test Plan
Lots of manual tests, everything seems good.

Tagging you, @gmarler, because this is changes load-bearing parts of the code, let me know if you spot any issues and I will be happy to take a look 😀 .


Build failure *** No rule to make target `sv/src/comp.c'

The host is CentOS 7.7.1908 (Core) 64bit.
$ sudo yum install -y luajit-devel && CXFLAG=-I/usr/include/luajit-2.0 ./install /home/chaz/.local/local/xmake-git-master

Loaded plugins: fastestmirror
Loading mirror speeds from cached hostfile
 * base: mirrors.coreix.net
 * extras: mirrors.coreix.net
 * updates: mirrors.coreix.net
Package luajit-devel-2.0.4-3.el7.x86_64 already installed and latest version
Nothing to do

installing to /home/chaz/.local/local/xmake-git-master ...
plat: linux
arch: x86_64
compiling xmake-core ...

=============================================================================
compile:
    plat:               linux
    arch:               x86_64
    host:               linux
    demo:               y
    prof:               n
    debug:              n
    small:              n
    ccache:
    distcc:

packages:
    tbox:               y
    luajit:             y
    base:               y

directories:
    install:
    package:

toolchains:
    bin:
    pre:
    sdk:

flags:
    cflag:
    ccflag:
    cxflag:             -I/usr/include/luajit-2.0
    mflag:
    mmflag:
    mxflag:
    ldflag:
    asflag:
    arflag:
    shflag:

clean xmake
clean lcurses
clean sv
clean luajit
clean tbox
clean xmake
clean demo
make xmake
make lcurses
compile.r lcurses.c
link liblcursesr.a
make sv
make[2]: *** No rule to make target `sv/src/comp.c', needed by `sv/src/comp.o'.  Stop.
make[1]: *** [DEP_PROS_sv_all] Error 2
install xmake
make lcurses
make[2]: Nothing to be done for `all'.
make sv
make[2]: *** No rule to make target `sv/src/comp.c', needed by `sv/src/comp.o'.  Stop.
make[1]: *** [DEP_PROS_sv_all] Error 2
make: *** [install] Error 2
cp: cannot stat './core/bin/demo.pkg/bin/linux/x86_64/demo.b': No such file or directory
chmod: cannot access '/home/chaz/.local/local/xmake-git-master/share/xmake/xmake': No such file or directory
install xmake core failed!

Had to run git submodue init and git submodule update.

You can only run
bash <(curl -fsSL https://raw.githubusercontent.com/xmake-io/xmake/master/scripts/get.sh)


fix: return tombstone flag

Signed-off-by: Trent Smith<EMAIL_ADDRESS>Reviewers
Functional:
Readability:

Changes

modify datasets to return a tombstone flag

Definition of Done (from ticket)

The collection owner's private page will displace correctly when a tombstone dataset exists.

QA steps (optional)
Known Issues

This PR is waiting on the front end to support the tombstone field.


404 link for cask Java

brew install Caskroom/cask/java
==> brew cask install Caskroom/cask/java
==> Caveats
This Cask makes minor modifications to the JRE to prevent issues with
packaged applications, as discussed here:
https://bugs.eclipse.org/bugs/show_bug.cgi?id=411361
If your Java application still asks for JRE installation, you might need
to reboot or logout/login.
Installing this Cask means you have AGREED to the Oracle Binary Code
License Agreement for Java SE at
http://www.oracle.com/technetwork/java/javase/terms/license/index.html
==> Downloading http://download.oracle.com/otn-pub/java/jdk/8u74-b02/jdk-8u74-macosx-x64.dmg
######################################################################## 100.0%
curl: (22) The requested URL returned error: 404 Not Found
Error: Download failed on Cask 'java' with message: Download failed: http://download.oracle.com/otn-pub/java/jdk/8u74-b02/jdk-8u74-macosx-x64.dmg

From the template:

- [ ] I understand that if I ignore these instructions, my issue may be closed without review.

curl errors are a part of the first item on the checklist.

Try brew update then brew cask install java


check_powershell and integrate into chocolatey::install

check_powershell runs a powershell command and evaluates the output and return values to determine success or failure. To provide the best flexibility, this resource provides parameters to retry executions.

[x] Add ruby-pwsh to the opv module's dependencies. (.sync.yml; pdk update)

[x] Create new check_powershell resource.

[x] Update check_powershell type to have the following attributes:

command, namevar, String: The powershell command to run.

expected_exitcode, parameter, Array[Number]: an array of acceptable exit codes. Defaults to [0]

output_matcher, parameter, Regexp: a call is considered a success if its output matches this regular expression. Defaults to //

execution_timeout, parameter, Numeric: number of seconds for a single execution to wait for a response to return a success before aborting, defaults to 60

retries, parameter, Integer: number of requests to make before giving up, defaults to 1

backoff, parameter, Numeric: initial number of seconds to wait between requests, defaults to 10

exponential_backoff_base, parameter, Numeric: exponential base for the exponential backoff calculations, defaults to 2

max_backoff, parameter, Numeric: an upper limit to the backoff duration, defaults to 120

timeout, parameter, Numeric: number of seconds allocated overall for the check to return a success before giving up, defaults to 600

[ ] Update the check_http provider to use the insync? feature from https://github.com/puppetlabs/puppet-resource_api/issues/225 to be quiet (in sync) when the check succeeds, and report a regular error otherwise. See https://github.com/puppetlabs/opv/commit/a7f60f2af0d55f70d3c231c4010ef0cb48e8f776 for an example.

[ ] Update the check_powershell provider to use the above attributes to execute the command up to retries number of times with success being defined as having one of the expected_statuses and the output of the command matching output_matcher while taking into account execution_timeout.

[ ] Update the check_powershell provider to wait for backoff ** (exponential_backoff_base * (number_of_attempt - 1) seconds between attempts (up to max_backoff), but aborting completely after timeout seconds if no successful request could be made. Investigate combining this retry implementation with the one in https://github.com/puppetlabs/opv/issues/1.

[ ] Update the check_powershell provider to log detailed messages at the debug level of execution, output and backoff waits.

[ ] Create a PR on the puppetlabs/chocolatey module to integrate the check_powershell into chocolatey::install to verify that choco can be executed. Do not merge that PR as this is still experimental work.

Each change needs to be accompanied with appropriate unit and (changes to) acceptance tests proving the functionality.

Added required dependencies to the sync.yaml file
Created the check_powershell resource type
Created the check_powershell resource provider
Currently testing the provider


FR: support array comprehension.

The below code below code does not get transformed.
[ for (value of ["Harriet", "178"]) value ].join(" was ");

Thank you for reporting this issue and appreciate your patience. We've  notified the core team for an update on this issue. We're looking for a response within the next 30 days or the issue may be closed.

/me digs to find the source of @facebook-github-bot-2

jstransform isn't maintained any more, sorry. Use Babel instead.

@spicyj yup, aware of it, was curious about the bot ;)

We're instituting a nag / autoclose bot for some projects and this one got tagged. You won't find the source for that bot, it's all internal (but if you somehow do manage to find it, please contact us at https://www.facebook.com/whitehat :wink:)

@zpao Roger that :wink:


step 2 fails after clicking Continue button

After I click continue, it seems to do some work but then fails and outputs...

Something went wrong

Not a very helpful message, but better than nothing I suppose.

Checking the Google Cloud Console, I can see the project was created correctly.

However, the bucket was NOT created.

Also - the billing account was assigned correctly.

Actually, it looks like the billing account WAS assigned correctly. So the only problem is with bucket creation.

The problem was we were trying to create the bucket before project initialize. Sometimes project creation takes some time instead of immediately.


pywinrt: provide a mechanism for instantiating Python winrt types from a raw pointer to enable interop with existing Python code using frameworks such as comtypes

There are situations where Python code may already have access to an ABI-level winrt COM object. E.g. a ctypes.c_void_p (void*) pointing to a COM object instantiated through a custom dll, or perhaps a comtypes COM pointer.
It would be useful if there was a way to convert this raw pointer into a true Pywinrt class instance. Similar to copy_from_abi in c++/winrt.
Similarly, going back the other way (copy_to_abi) would also be useful.
A very practical use case for this is when consuming the new Windows.UI.UIAutomation.AutomationElement or Windows.UI.UIAutomation.AutomationTextRange winrt interfaces.
Windows.UI.UIAutomation.AutomationElement is a winrt interface that is supported as an extra interface on the existing IUIAutomationElement COM interface, as part of the Microsoft UI Automation accessibility standard.
Windows.UI.UIAutomation.AutomationElement is not a class that can be activated by itself, nor does it contain all information that IUIautomationElement* interfaces do. Rather it provides a few extra methods (getAppUserModelID, isRemote etc).
There is no way to get an AutomationElement using only other objects and methods in the winrt namespace. You must always start with an IUIAutomationElement which you got from an event or method on IUIAutomation (CUIAutomation coclass, and then convert this IUIAutomationElement  to the AutomationElement winrt interface, which in c++/winrt would be done with copy_to_abi.
a generic feature allowing conversion to a Pywinrt type from a winrt ABI type, would by itself allow for writing Python code to access these new methods provided by AutomationElement (getAppUserModelID) etc. But the power really comes from when we consider access to the new UI automation Remote Operations interfaces. Either the built-in Windows.UI.UIAutomation.Core.CoreAutomationRemoteOperation or the Microsoft.UI.UIAutomation objects, exposed in the microsoft/microsoft-ui-uiautomation project. This infrastructure allows you to batch logic and method calls to UI automation objects, drastically decreasing the amount of cross-process calls.
However, These remote operation interfaces require the caller to "import" existing AutomationElement and AutomationTextRange objects into the batch call so they can be referenced by the logic. Similarly, AutomationElement objects and AutomationTextRange objects may be fetched from the execution result at the end. But, in order to actually do anything useful with these objects, it is necessary to be able to both convert from a raw IUIAutomationElement* to AutomationElement and also from AutomationElement back to IUIAutomationElement*.
One solution that comes to mind for allowing conversion to a Pywinrt type is in pywinrt's pybase.h, in py::convert_to<Windows.Foundation.IInspectable): simply to support a Python object of ctypes.c_void_p, fetch the raw address from the 'value' member, and cast it to a Windows.Foundation.IInspectable.
Windows.UI.UIAutomation.AutomationElement already has a method 'fromElement' which allows passing in a Python object that is convertable to IInspectable, thus, it should then be possible to call fromElement giving it a ctypes.c_void_p instance as its argument. Note that COM interfaces generated by the Python comtypes package all inherit from ctypes.c_void_p, thus it would be possible to pass in a IUIAutomationElement comtypes COM interface pointer.
The one major risk with adding this would be that this would allow the caller to provide an arbitrary opaque pointer, and having pywinrt trust that this is truely an IInspectable. However, this is also true for the C++/winrt mechanism already I think.
To go back the other way, perhaps all winrt_base Python objects could inherit from ctypes.c_void_p, or they could contain a new method: to_abi, or a stand-alown Python function (to_abi) could be exposed on the root pywinrt package.
this request would go some way in allowing us to continue integration of UI automation Remote Operations into the NVDA Screen Reader project. Rather than having to re-write the majority of our code in C++/winrt, we could continue to stay in NVDA's native Python code.

Please see this repo for Python support: https://github.com/pywinrt/pywinrt


Clarify which Python versions are supported

Context
https://github.com/ethereum/staking-deposit-cli/blob/fdab65d33a63632e1935e9a9235119a46e37c221/README.md?plain=1#L199-L205
Problem
The README states Python > 3.8 is supported but that's not the case. See https://github.com/ethereum/staking-deposit-cli/issues/363
Proposal
Until https://github.com/ethereum/staking-deposit-cli/issues/363 is resolved, can the README explicitly state the versions of Python that are supported

Come on over, we have Python 3.11, 3.12 and even 3.13.0-rc.1 in our CI. But minimum 3.9, we dropped 3.8.
Recommend sticking to Holesky with our fork until we have the re-audit done. And, we can use all the help we can get in testing.
https://github.com/eth-educators/ethstaker-deposit-cli


Classify ApplicationCoordinator lib

I think the best place for ApplicationCoordinator lib in in App Routing section.
@lfarah, @dkhamsing, @vsouza What do you think?
close #2022.

          1 Warning

:warning:
Found 6 link issues, a project collaborator will take care of these, thanks :)

Link issues by awesome_bot

Line
Status
Link

789
504
https://github.com/benzguo/MusicKit

877
301
https://github.com/terflogag/FacebookImagePicker redirects tohttps://github.com/floriangbh/FacebookImagePicker

1377
301
https://github.com/SwiftOnEdge/Edge redirects tohttps://github.com/skylab-inc/Edge

2260
502
https://github.com/edopelawi/CascadingTableDelegate

2367
502
https://github.com/FabrizioBrancati/BFKit

2405
301
https://github.com/terflogag/OpenSourceController redirects tohttps://github.com/floriangbh/OpenSourceController

  Generated by :no_entry_sign: Danger

Thank you very much @serhii-londar
Could you make your commits and pull requests with format “Add ProjectName”


add shortcut to "magit-gitflow-popup" with '%' in magit-mode

as title, thks.

I'm not sure if % is the best keybind, but I'll give it a try for a while. Thanks for your help!

The shortcut was referenced from spacemacs,Thanks.


Update specification.md to match implemented types

Some minor (but breaking) changes were made to the actual CRD types compared to what was proposed in the specification.md
A quick skim and I've found the following inconsistencies:

spec.signingKey renamed to spec.issuer.
condition for SeedSecretReady shared across "keypairable" resources.
Operator.spec.identities removed.
Account.spec.limits changed to reflect the same defaults in the nsc CLI.
User.spec.limits may have also changed, need to confirm.

Also need to add Operator.spec.tlsConfig to the above


NTR: regulatory RNA binding

Under the "RNA binding" term, there are currently a number of terms for binding of (eukaryotic) RNA regulators, e.g. piRNA, miRNA and siRNA binding.  No terms currently exist for the prokaryotic small regulatory RNAs.  I propose a new parent term, regulatory RNA binding, as a child of RNA binding.  In addition, I'd like to request creation of a term for "small RNA binding" that would include the bacterial small regulatory RNAs; it could have the "regulatory RNA binding" term as its parent.
Thanks,
Ingrid
Reported by: imk07
Original Ticket: geneontology/ontology-requests/10058

@keseler What do you suggest we do about this ticket; do you want to create 'small regulatory RNA binding'? If you do, please provide a reference and a definition.
Thanks, Pascale

Oh my, that's a long time ago.  Yes, I'd still like a "small regulatory RNA binding" term.
Definition? perhaps: "Interacting selectively and non-covalently with a small regulatory RNA, a short RNA (usually 50-350 nt long) that is either independently transcribed or processed from a longer RNA by an RNAse enzyme."
Publications; hmmm.  A subset of small regulatory RNAs interact with the Hfq protein; a good publication would be PMID 14622403.  Though I think what precipitated my request at the time was a much later publication about RapZ, PMID 23475961.
Thanks,
Ingrid

Looks like piRNAs were forgotten - could you make GO:0034584 piRNA binding a child of GO:0061980 regulatory RNA binding?
Thanks