Deprecate generateOEMolFromTopologyResidue

May of the ProposalEngine subclasses use openmoltools.forcefield_generators.generateOEMolFromTopologyResidue , which is turning out to be highly problematic. We should see if we can avoid this and always go from the stored SMILES to OEMols, rather than ever having to go backward and try to perceive an OEMol from a simtk.openmm.app.Topology.
Alternatively, we could switch to the newopenforcefield.Topology object once it allows us to store residue information as well, or we can simply convert fully to the OpenEye approach which allows residues to be defined as well. See this code, which allows us to convert a polymer OEMol to OpenMM Topology object.

This will be solved by implementing #521

Tackling this now in #626


solidstart with auth -> failure:  Missing "./data" export in "solid-start" package

<project-folder> $ npm init solid@next

create-solid version 0.1.0-alpha.75

Welcome to the SolidStart setup wizard!

There are definitely bugs and some feature might not work yet.
If you encounter an issue, have a look at https://github.com/solidjs/solid-start/issues and open a new one, if it is not already tracked.

✔ Which template do you want to use? › with-auth
✔ Server Side Rendering? … no
✔ Use TypeScript? … no

found matching commit hash: cee57b7708d6c90b2e8336d4116c891b21f95046

...

<project-folder> $ npm install
...
<project-folder> $ npm run dev

22:52:00 [vite] Internal server error: Missing "./data" export in "solid-start" package
  Plugin: vite:import-analysis
  File: <project-path>/src/routes/login.jsx

Heyy this was an issue in the examples are they are served from main branch here, but we changed some APIs without doing a release yet that caused this mismatch. We are going to do a release soon and that should fix this!

I'm also having issues running the ssr version of the with-auth template.
Not sure if this is related but at least the final error is different.
✔ Which template do you want to use? › with-auth
✔ Server Side Rendering? … yes
✔ Use TypeScript? … yes
found matching commit hash: 25406e4ff746066b11123fb511b0d41ba2b0e12b

Error Log
GET /
Missing "./data" export in "solid-start" package
Missing "./data" export in "solid-start" package (x2)
11:16:36 AM [vite] Error when evaluating SSR module /node_modules/cookie/index.js?v=2a8788d2:
ReferenceError: exports is not defined
    at /node_modules/cookie/index.js?v=2a8788d2:15:1
    at instantiateModule (node_modules/vite/dist/node/chunks/dep-c9998dc6.js:50311:15)
11:16:36 AM [vite] Error when evaluating SSR module /node_modules/solid-start/session/cookies.ts:
ReferenceError: exports is not defined
    at /node_modules/cookie/index.js?v=2a8788d2:13:1
    at instantiateModule (node_modules/vite/dist/node/chunks/dep-c9998dc6.js:50311:15)
11:16:36 AM [vite] Error when evaluating SSR module /node_modules/solid-start/session/cookieStorage.ts:
ReferenceError: exports is not defined
    at /node_modules/cookie/index.js?v=2a8788d2:11:1
    at instantiateModule (node_modules/vite/dist/node/chunks/dep-c9998dc6.js:50311:15)
11:16:36 AM [vite] Error when evaluating SSR module /node_modules/solid-start/session/index.ts?v=2a8788d2:
ReferenceError: exports is not defined
    at /node_modules/cookie/index.js?v=2a8788d2:9:0
    at instantiateModule (node_modules/vite/dist/node/chunks/dep-c9998dc6.js:50311:15)
11:16:36 AM [vite] Error when evaluating SSR module /src/db/session.tsx:
ReferenceError: exports is not defined
    at /node_modules/cookie/index.js?v=2a8788d2:7:0
    at instantiateModule (node_modules/vite/dist/node/chunks/dep-c9998dc6.js:50311:15)
11:16:36 AM [vite] Error when evaluating SSR module /src/routes/index.tsx?data:
ReferenceError: exports is not defined
    at /node_modules/cookie/index.js?v=2a8788d2:5:0
    at instantiateModule (node_modules/vite/dist/node/chunks/dep-c9998dc6.js:50311:15)
11:16:36 AM [vite] Error when evaluating SSR module /node_modules/solid-start/root/Routes.tsx:
ReferenceError: exports is not defined
    at /node_modules/cookie/index.js?v=2a8788d2:3:0
    at instantiateModule (node_modules/vite/dist/node/chunks/dep-c9998dc6.js:50311:15)
11:16:36 AM [vite] Error when evaluating SSR module /node_modules/solid-start/root/index.tsx:
ReferenceError: exports is not defined
    at /node_modules/cookie/index.js?v=2a8788d2:1:0
    at instantiateModule (node_modules/vite/dist/node/chunks/dep-c9998dc6.js:50311:15)
11:16:36 AM [vite] Error when evaluating SSR module /node_modules/solid-start/entry-server/StartServer.tsx:
Error: `line` must be greater than 0 (lines start at line 1)
    at originalPositionFor$1 (node_modules/vite/dist/node/chunks/dep-c9998dc6.js:16780:19)
    at node_modules/vite/dist/node/chunks/dep-c9998dc6.js:50162:25
    at String.replace (<anonymous>)
    at node_modules/vite/dist/node/chunks/dep-c9998dc6.js:50152:21
    at Array.map (<anonymous>)
    at ssrRewriteStacktrace (node_modules/vite/dist/node/chunks/dep-c9998dc6.js:50151:10)
    at instantiateModule (node_modules/vite/dist/node/chunks/dep-c9998dc6.js:50316:32)
11:16:36 AM [vite] Error when evaluating SSR module /node_modules/solid-start/entry-server/index.ts?v=2a8788d2:
Error: `line` must be greater than 0 (lines start at line 1)
    at originalPositionFor$1 (node_modules/vite/dist/node/chunks/dep-c9998dc6.js:16780:19)
    at node_modules/vite/dist/node/chunks/dep-c9998dc6.js:50162:25
    at String.replace (<anonymous>)
    at node_modules/vite/dist/node/chunks/dep-c9998dc6.js:50152:21
    at Array.map (<anonymous>)
    at ssrRewriteStacktrace (node_modules/vite/dist/node/chunks/dep-c9998dc6.js:50151:10)
    at instantiateModule (node_modules/vite/dist/node/chunks/dep-c9998dc6.js:50316:32)
11:16:36 AM [vite] Error when evaluating SSR module src/entry-server:
Error: `line` must be greater than 0 (lines start at line 1)
    at originalPositionFor$1 (node_modules/vite/dist/node/chunks/dep-c9998dc6.js:16780:19)
    at node_modules/vite/dist/node/chunks/dep-c9998dc6.js:50162:25
    at String.replace (<anonymous>)
    at node_modules/vite/dist/node/chunks/dep-c9998dc6.js:50152:21
    at Array.map (<anonymous>)
    at ssrRewriteStacktrace (node_modules/vite/dist/node/chunks/dep-c9998dc6.js:50151:10)
    at instantiateModule (node_modules/vite/dist/node/chunks/dep-c9998dc6.js:50316:32)
ERROR Error: `line` must be greater than 0 (lines start at line 1)
    at originalPositionFor$1 (node_modules/vite/dist/node/chunks/dep-c9998dc6.js:16780:19)
    at node_modules/vite/dist/node/chunks/dep-c9998dc6.js:50162:25
    at String.replace (<anonymous>)
    at node_modules/vite/dist/node/chunks/dep-c9998dc6.js:50152:21
    at Array.map (<anonymous>)
    at ssrRewriteStacktrace (node_modules/vite/dist/node/chunks/dep-c9998dc6.js:50151:10)
    at instantiateModule (node_modules/vite/dist/node/chunks/dep-c9998dc6.js:50316:32)

FIY I just created my own createCookie implementation using @tinyhttp/cookie and it also works fine.
(If you don't want to maintain your own cookie implementation)

We might use this, don't want to maintain our own implementation, but esm was becoming a problem.
Also with installing latest solid-start version, this ./data export issue should go away..


Drops certain torrents

I've been using the newest release for about a month now, but it seems to drop certain torrents when I install them. It seems it's more the magnets than full torrent downloads, but I'm not 100% sure. My Transmission environment was a bit buggy, but using Transdrone, I could see these same torrents, despite missing from my queue in Gearshift. I think it was to do with the folders I had the torrents downloading to, also.

Honestly, I can't do anything with such info. What do you mean by "a bit buggy"? Maybe you should fix that first and the problem might go away :) If it doesn't, enable debug logging, send in the resulting logcat, and maybe we might find the problem


ListView within a Pivot not re-rendering?

Hey All,
First I'd like to say thanks for creating this. While I'm not particularly excited with the "magic" I don't yet understand on the layers of abstraction added, I am very interested in how much easier it is to reason about building WinJS apps using React...
I am struggling to debug the following scenario.
I've created a SearchHub component (think of this component as very similar to the example search movies http://winjs.github.io/react-winjs/examples/movies/index.html component).
The biggest difference is I'm trying to place it inside of a Pivot control.
When the app first loads up and I search, it fills out the ListView great. I click on the item to navigate to the details view. When I come back the ListView is not re-rendering and is blank...
My first issue was due to React state problem, but I've worked out the state correctly. If I use the SearchHub component by itself it correctly re-renders the ListView on back navigation. However once placed inside a PivotItem back navigation does not appear to correctly render the ListView.
Correctly renders search/listView on back navigation from details page
return (
                // <Pivot style={pivotStyle}>
                //     <Pivot.Item key="search" header="Search">
                        <SearchHub search={this.state.search} />
                //     </Pivot.Item>
                //     <Pivot.Item key="featured" header="Featured">
                //         <FeaturedHub />
                //     </Pivot.Item>
                // </Pivot>
    }

Does NOT render search/listView on back navigation from details page
return (
                <Pivot style={pivotStyle}>
                    <Pivot.Item key="search" header="Search">
                        <SearchHub search={this.state.search} />
                    </Pivot.Item>
                    <Pivot.Item key="featured" header="Featured">
                        <FeaturedHub />
                    </Pivot.Item>
                </Pivot>
    }

Any thoughts or tips on how to debug this?

NOTE: the input search box IS rendering correctly - so it's not the whole view inside the pivot having issues, just the ListView...

Hmm, I was going to try to repro in a Plunk - but struggling to get it to render the original data now :(

I've tried triggering a forceLayout, but no luck there
                <ListView
                    ref={function(listView) {
                        if (listView && listView.winControl) {
                            listView.winControl.forceLayout();
                            console.log("forceLayout");
                        } else {
                            console.log("no forceLayout");
                        }
                    }}

@staxmanade This is due to a bug in the Pivot introduced in WinJS 4.1. In general, ListViews fail to render inside of a Pivot. I filed an issue for this: winjs/winjs#1497.
In case you are curious, here are additional details. The ListView happens to render inside of the Pivot sometimes in react-winjs due to a race condition. react-winjs calls React.render to render the Pivot Item. If React.render runs synchronously, then the ListView fails to render. If React.render runs asynchronously, then the ListView may render. See winjs/winjs#1497 for more details about the cause.
You should be able to work around this by calling forceLayout() on the ListView -- you just have to make sure this is called after the Pivot Item is visible. A good time to call forceLayout() on the ListView would be in the Pivot's itemanimationend event.

Fantastic. I'll go back to my project and give this a try! Thanks for digging in.

Thank you @rigdern. I implemented the following work-around based on your suggestion and it's working great.
In the React component where render returns a Pivot I have this:
    hackFixForListView() {
        // due to a bug in WinJS
        // https://github.com/winjs/react-winjs/issues/27
        // we need to forceLayout here for listViews
        console.warn('hackFixForListView called: https://github.com/winjs/react-winjs/issues/27');
        [].slice.call(document.querySelectorAll('.win-listview'))
            .forEach(function(item){
                item.winControl.forceLayout();
            });
    }

On the Pivot control I use this:
                <Pivot onItemAnimationEnd={this.hackFixForListView}>
                    <Pivot.Item key="search" header="Search">
                        <SearchHub search={this.state.search} />
                    </Pivot.Item>
                </Pivot>

This is working great for now. I'll keep an eye on the open WinJS issue and remove the work-around when it's updated.


issue_comment.created not working

Bug Report
Current Behavior

 app.on('issue_comment.created', async (context) => {
    app.log("HELLO PR")
    app.log(context)
    context.log("HELLO")
    const issueComment = context.issue({ body: 'Thanks for opening this PR!' })
    await context.github.issues.createComment(issueComment)
  })

I have this code.
I receive this webhook event
18:33:51.402Z DEBUG probot: Webhook received
  event: {
    "event": "issue_comment.created",
    "id": "ac5781e0-6788-11e9-9f68-1190ac421f8c",
    "installation": 862497,
    "repository": "PeopleNet/user-profile-tid"
  }

But it never triggers the code.
Other events work fine and trigger code.
Expected behavior/code
issue_comment.created should trigger code looking for issue_comment.created
Environment

Probot version(s): [e.g. v^7.2.0]
Node/npm version: [e.g. Node 8.3/npm 6.4.1]

@ivelichkovich What did you do to make it work? Why is this issue closed?


Update Condition command

This PR fixes #37 implementing Condition behavior and stuff into editor.

Requires #58

Requires to be remade due breaking changes


Any issues with IASP?

I am able to set up and see my source file and library. However, when I click on some members I don't see these sources.
nothing happens.
I am able to open source on QSYSINC/H.
I am not sure if IASP is an issue? I see all the source code files expanded, it's just that it does open when double clicked.

Yep. That'd be an ASP issue 👎
I know the fix. Just got to do it :)

My plan is to make the ASP a configurable item in the settings.

@AjaykulkarniIBMi Grab 0.0.8 and see the source ASP config in the settings.


feat: decode parquet files

Decode parquet files for visualization purposes.

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

Project coverage is 81.70%. Comparing base (724bdd0) to head (f945e9e).

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #21   +/-   ##
=======================================
  Coverage   81.70%   81.70%
=======================================
  Files          15       15
  Lines        1536     1536
  Branches     1536     1536
=======================================
  Hits         1255     1255
  Misses        210      210
  Partials       71       71

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


feat: invalidate SSM cache upon AMI invalidation

Fixes #N/A
Description
When an EKS-optimized AMI is deprecated, and a user is using a family@latest alias, Karpenter will continue to launch with that AMI until the 24 hour SSM cache entry has expired. This cache ensures Karpenter doesn't cause a thundering herd by upgrading all users within a region simeltaneously upon a SSM parameter rollout. However, Karpenter should respond faster in the event of an EKS-optimized AMI deprecation.
This PR introduces a mechanism to invalidate the SSM cache upon detection of a deprecated EKS-optimized AMI. This cache invalidation is still staggered over 30 minutes to reduce the risk of a thundering herd, but this is still up to 48x the reaction time Karpenter previously had. Additionally, thanks to the 24 hour cache, only a subset of users using family@latest will have been upgraded to the deprecated AMI in the first place, further reducing the chances of a thundering herd upon rollback.
Some assumptions made:

The latest SSM parameter for EKS optimized AMIs should be rolled back before AMI deprecation. If this is not the case, Karpenter will continuously invalidate the SSM cache until the parameter points to a non-deprecated AMI.
Karpenter will only need to rollback a subset of users if the issue is detected and the AMI is deprecated within 24 hours of the AMI's release. Otherwise, we expect all users with a family@latest alias (and non-blocking drift budgets / PDBs) to have been upgraded to the now-deprecated AMI.

How was this change tested?
make test (additional tests in-progress)
Does this change impact docs?

[ ] Yes, PR includes docs updates
[ ] Yes, issue opened: #
[x] No

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>3
Details

80 of 148 (54.05%)  changed or added relevant lines in 14 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.7%) to 82.162%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pkg/operator/operator.go
0
1
0.0%

pkg/controllers/controllers.go
0
2
0.0%

pkg/providers/amifamily/al2.go
20
23
86.96%

pkg/providers/amifamily/al2023.go
19
22
86.36%

pkg/providers/amifamily/bottlerocket.go
10
13
76.92%

pkg/providers/amifamily/custom.go
0
3
0.0%

pkg/controllers/nodeclass/status/ami.go
1
6
16.67%

pkg/apis/v1/zz_generated.deepcopy.go
0
10
0.0%

pkg/providers/ssm/provider.go
6
44
13.64%

Totals

Change from base Build<PHONE_NUMBER>4:
-0.7%

Covered Lines:
5670

Relevant Lines:
6901

💛  - Coveralls


Binary encoding of larger data types in

It would be nice to support arrays of larger types in arbitrary block.
Currently, there is only one function for parameter SCPI_ParamArbitraryBlock() and one for result SCPI_ResultArbitraryBlock()
It is sometimes needed to transfer larger data types e.g. array of 32bit integers, but endiannes is up to the user of the library.
This problem was partially discussed in #51
Possible solutions

introduce helper function to translate array of native values to "normal" or "swapped" e.g. on x86, swap bytes while transfering array of int16_t in SCPI's "normal" byte order or do nothing special on AVR32.
introduce new SCPI_ParamArbitraryBlockXYZ functions to do this on the fly but this will probably need additional data buffer

Specification
In the SCPI specification, there are two main chapters about binary encoding of these types
SCPI-99:Volume 1 - 9.1 [:DATA] <type>[,<length>]
SCPI-99:Volume 3 - 6.4.2 FORMat
Among others, there are two types of encoding "normal" and "swapped". It seems that "normal" is big-endian representation and "swapped" is little-endian.

type name
description

ASCii
Comma-separated  data

IFP32
IEEE floating point, 32 bits.

IFP64
IEEE floating point, 64 bits.

INT8
IEEE 8-bit signed integer (default).

INT16
IEEE 16-bit signed integer.

INT32
IEEE 32-bit signed integer.

INT64
IEEE 64-bit signed integer.

SFP32
IEEE swapped floating point, 32 bits.

SFP64
IEEE swapped floating point, 64 bits.

SINT16
Swapped IEEE 16-bit signed integer.

SINT32
Swapped IEEE 32-bit signed integer.

SINT64
Swapped IEEE 64-bit signed integer.

SUINT16
Swapped IEEE 16-bit unsigned integer.

SUINT32
Swapped IEEE 32-bit unsigned integer.

SUINT64
Swapped IEEE 64-bit unsigned integer.

UINT8
IEEE 8-bit unsigned integer.

UINT16
IEEE 16-bit unsigned integerUINT16.

UINT32
IEEE 32-bit unsigned integerUINT32.

UINT64
IEEE 64-bit unsigned integerUINT64

Just some thoughts, instead of appending the type to the function name, the type could be provided as an argument. If the type endianness is the same as the platform, then only the length is checked to be type size aligned, otherwise swapping is also provided. There are libraries available for swapping, but it is actually trivial the only advantage from a library would be it SIMD instructions could be used.

It is also possible to have one function and parameter as type as you propose.
It can look like your implementation of processing of array/buffer of numbers. But this will be better to have separate functions.
This is straightforward
SCPI_ResultBufferUInt8(..., values, countof(values), NORMAL)
SCPI_ResultBufferInt32(..., values, countof(values), SWAPPED)
SCPI_ResultBufferFloat(..., values, countof(values), ASCII)
SCPI_ResultBufferDouble(..., values, countof(values), NORMAL)
This will consume all following parameters or only one arbitrary block with swapped UInt16 values.
SCPI_ParamBufferUInt16(..., values, countof(values), ASCII_OR_SWAPPED, &result_count, ...)
This will consume only one arbitrary block of NORMAL (big-endian) 64bit integers, no ASCII encoding allowed.
SCPI_ParamBufferInt64(..., values, countof(values), NORMAL, &result_count, ...)
For me, it sounds better SCPI_ResultArrayFloat then SCPI_ResultBufferFloat, for that purpose.
We can also officially divide SCPI_ResultArbitraryBlock into two functions - SCPI_ResultArbitraryBlockHeader and SCPI_ResultArbitraryBlockData. You can later call SCPI_ResultArbitraryBlockData more then once.

I have introduced first support in 3b99af699baba3ddbec2fbe74af6392678fbe31d for returning arrays. It is still not well tested, it is not optimal but it is first step.
I would like to introduce also reading array parameters and also unit tests for all new functions.

Result functions are now ready. They are also covered by tests.
Missing part is Parameter functions.

Array parameters are prepared but currently support only text.


Enhancement: Publish to Terraform Registry

It would be great if this provider were available on registry.terraform.io. This is probably blocked by #62 (since 0.11 doesn't support the registry anyways).
More info at:
https://www.terraform.io/docs/registry/providers/publishing.html
https://www.terraform.io/guides/terraform-provider-development-program.html

Hi Adam,
We are upgrading our sdk to support the plugin SDK and  validating with terraform 0.12.29. Publishing to registry will be done after that.

What is the state of the terraform .12 branch?   Should I be using that or the master which is .11?

What is the state of the terraform .12 branch? Should I be using that or the master which is .11?

We have added a branch named Terraform-0.12 . If you want to run your configuration files using Terraform 0.12 binary please checkout that branch and run your *.tf files. Do follow the README specific to Terraform-0.12 branch. You can also check the wiki section .Do let us know if it is working for you

Closing this  since provider is now available at registry.
https://registry.terraform.io/providers/HewlettPackard/oneview/latest?pollNotifications=true


Apply button. Submission Object

For the MVP the "Apply" button should:

If "isConnected=false" should say "Connect Wallet to Apply"
If "isConnected=true" should:
Open a form
Form should have
Some description like: "You are applying to the {issue_name} bug bounty, etc...

{
    "title":"string", // description: Tittle of your sollution
    "description": "string", // Describe your solution
    "github_link": "string", // Put link of your solution's repo
},

Button "Apply now". Dummy button. If pressed triggers a verification function.

verify that there are no empty fields
if ok. console.log("Applied") and close form

The form has been added and is working perfectly fine with the API

Hasura API is linked with it


Link to source on creator view

Description

It might be nice to link the source on the creator view, given that we have that information and are displaying it anyway.
Example:

Visit https://staging.openverse.org/preferences and enable additional_search_views
Visit https://staging.openverse.org/image/source/smithsonian_air_and_space_museum/creator/Unknown/
The source name should be clicked on too

Alternatives

Not sure if this is necessary or ideal, but just something I thought while navigating through the pages.
Additional context

Related to #410

This is a more interesting case, actually. I'll assume the creator is not really named "Unknow," but we don't know who the creator is. How is this case being handled? Do we show all the unknown creator's works under that source?
CC @obulat @sarayourfriend

@krysal I've opened https://github.com/WordPress/openverse/discussions/3508 to avoid pulling the discussion here off of the issue Madison originally opened, which might need its own conversation here.

@fcoveram, could you update the creator view design with the source link, if you agree with this feature?

Here is an idea already applied in the mockups

I had to hack the frame element in Figma as wrapping a link component in two lines is not possible, but the visual outcome is the main point in the design.


Telemetry chart x-axis values overlapping

Type of issue

[x] Bug
[ ] New feature
[ ] Improvement

Description

will be fixed by changing telemetry to show 15 minutes of data.

close with #570


Check connection is ready in all connection required handlers

This is for https://github.com/hyperledger/aries-cloudagent-python/security/advisories/GHSA-cxwq-rwgr-qp5m.
I'm pretty sure this fixes the main issues reported with basic_messages and handshake_reuse protocols. I also applied the connection ready check to all other protocols that were missing and that I think need an established connection.
I didn't apply it to any problem report protocols because I don't think a connection is required in the cases. I'm not entirely sure if the problem report protocols could still be exploited in this way. Maybe an nefarious actor could send problem reports before the connection was established and cause issues?
The details of all these protocols isn't a strong point for me. I did my best to remediate the problem but I good review by someone more knowledgeable would be appreciated.

The code coverage wants me to tests all the exception scenarios.


pkg/endpoint: remove ProxyWaitGroup field from Endpoint

This field is only populated, and propagated in endpoint.regenerateBPF. Thus,
there is no point in exposing this field; it should only be accessible at the
time of regeneration of an endpoint. Update all users of the endpoint's
ProxyWaitGroup field to take in a *completion.WaitGroup parameter instead of
accessing the field from within the endpoint.
Signed-off by: Ian Vernon<EMAIL_ADDRESS>

This change is

test-me-please

test-me-please


[Doc] add \N descriptions to load docs

Signed-off-by: amber-create<EMAIL_ADDRESS>What type of PR is this：

[ ] BugFix
[ ] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Which issues of this PR fixes ：

Fixes #
Problem Summary(Required) ：

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr will affect users' behaviors
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[ ] I have checked the version labels which the pr will be auto backported to target branch

[ ] 2.5
[ ] 2.4
[ ] 2.3
[ ] 2.2

@Mergifyio backport branch-2.5

@Mergifyio backport branch-2.4

@Mergifyio backport branch-2.3


Help and support pages — GitBook Search Integration

When a user performs a search on the help and support pages we should search and link to content on GitBook if there's a match.

GitBook search API https://developer.gitbook.com/rest-v1/search
Design files https://www.figma.com/file/ZQBMWhnGGtRWeIZknFW1eP/[OC-Design]-Production-Ready-✅?node-id=14465%3A179840

On docs.opencollective.com (what we want to replicate), the search is directly powered by Algolia, not GitBook API.

About the GitBook API itself:

The GitBook API is currently in beta. This means you should not rely on it's availability, and that it may change without prior warning. If you wish to be one of our testers, please contact our team

I would really rather we move off gitbook before investing in further integration with the docs platform.


task manager is lying

It shows this:

Note the highlighted default-typescript project which was running last -- and which is still running despite what task-manager says. This is also why this (default-typescript) app runs at port 9002 - I did not realize that the previously running browser-sync instance are still alive

Specific command cancel task does the right thing, so this issue is a pilot error that will likely be repeated by others:

what is your proposal? the (1) behind Aurelia-CLI in the taskbar indicates that something is running

Note the highlighted default-typescript project which was running last -- and which is still running despite what task-manager says

Perhaps we should consolidate the taskmanager and the taskrunner screens. Right now the taskmanager installs NPM modules and starts "au run --watch", after which you need to look in the taskrunner (link) for the output and to be able to cancel it.
I think you are looking for an overview of running gulp or au run tasks. The reason we have a taskmanager is that it allows you to see what monterey is working on, regardless of what project you have selected. So it would make sense to show the running au run tasks in the taskmanager, but this may add to the confusion.
I think we can improve the taskmanager UI so it's able to start gulp watch  or au run tasks as well. This was the main reason for creating the taskrunner

@adriatic thoughts?

I want to think about it some more - realizing that we need to ensure uniform behavior across building of all different application types (so far I was only focused on testing CLI apps). For example, I just tried skeleton-esnext-aspnetcore, which I believe is being fetched from the Github aurelia skeletons collection.
In this situation NPM and JSPM tiles are activated as a consequence of selecting the  checkboxes in the last wizard panel (see issue 182) - and even Gulp can be activated there. So, we need some global view for all projects and all stages of the project from which every of these actions could be handled.

in https://github.com/monterey-framework/monterey/issues/166 a better UI will be implemented


Mod doesnt load on curseforge

Whenever you add the mod to curseforge,  (yes, i got the right one for fabric 1.19.2) It would crash and say something like this:
The game crashed whilst initializing game
Error: java.lang.RuntimeException: Could not execute entrypoint stage 'main' due to errors, provided by 'msd'!
i have no real clue why it does this on curseforge, But if this has to do something with curseforge and not the mod, then idk how to install it.

It looks like it's related to the MTR Station Decoration mod and not NTE.


[Docs] Supported Platforms table missing bottom row

Description
The Supported Platforms table in the Meshery repo's README.md is missing a cell.
Expected Behavior
The table should have a single row, spanning two columns at the bottom of the table.
Screenshots

I'll like to fix this issue

Thank you, @Veri5ied


ReadNovelFull source novels are empty

Novels i downloaded from ReadNovelFull source are empty such as the legendary mechanic, mech touch, 48 hours a day and more.

The source works fine.
Delete the downloaded chapters and check if the chapters have text or not.

Just wanted to say that readnovelfull is completely empty


No esdf point cloud with cow_and_lady_dataset

Good day,
I did try the example data file cow_and_lady_dataset (data.bag)
I can see the slice of the esdf and esdf_map_out has messages on the ROS Topic.
On ROS Topic esdf_pointcloud is never a message.
How can I fix this?
here is my launchfile:
cow_and_lady_dataset.txt

Try adding the following parameters:
<param name="publish_pointclouds" value="true" />
<param name="publish_slices" value="true" />

Maybe we should consider changing the defaults... ;)
On Tue, Aug 14, 2018 at 3:00 PM schnph<EMAIL_ADDRESS>wrote:

Good day,
I did try the example data file cow_and_lady_dataset (data.bag)
I can see the slice of the esdf and esdf_map_out has messages on the ROS
Topic.
On ROS Topic esdf_pointcloud is never a message.
How can I fix this?
here is my launchfile:
cow_and_lady_dataset.txt
https://github.com/ethz-asl/voxblox/files/2286945/cow_and_lady_dataset.txt
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/ethz-asl/voxblox/issues/191, or mute the thread
https://github.com/notifications/unsubscribe-auth/AFWzCNcFktJCbMfIqtM4z5HZqoQ7RUyxks5uQsnigaJpZM4V8Y5O
.

You can either call the publish_pointclouds service, or set the parameter publish_pointclouds to true. This was fairly poorly documented and I have now updated the readme

No problem, the publish_pointcloud flag was missing from the readme


How to change the model to GAN and dataset to cifer10 or own Custom dataset

How We can change the model and datset

Can you please describe the question properly? I didnt get you

@Manoj-2702
I recently checked out your project where you used the Flower framework with a CNN model on the MNIST dataset—really impressive work!
I'm interested in modifying this project by using a different dataset, like CIFAR-10, and implementing a different deep learning model, such as a GAN. Could you provide some guidance on how I might go about making these changes? If possible, could you share your WhatsApp number for more direct communication?
Thanks a lot for your help!

Dear Sir,
I recently explored your GitHub repository where you used the Flower
framework with a CNN model on the MNIST dataset. Your work is quite
impressive, and it has motivated me to work on a similar project.
I'm planning to use a different dataset, such as CIFAR-10, and I want to
implement a Generative Adversarial Network (GAN) instead of the CNN model.
If you have any resources or guidance on how I can achieve this, I would
greatly appreciate your help. Additionally, if you're open to more direct
communication, could you please share your WhatsApp number?
Thank you for your time and assistance.
Best regards,
Ibad
On Mon, Aug 26, 2024 at 9:18 PM Manoj Kumar H S @.***>
wrote:

@ibad321 https://github.com/ibad321
Can you please describe the question properly? I didnt get you
—
Reply to this email directly, view it on GitHub
https://github.com/Manoj-2702/PoseSync-Flower-Federated-Learning-for-Yoga-Poses/issues/1#issuecomment-2310582179,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASI2TRWQEHJ36V3XTJ7YQGDZTNIMBAVCNFSM6AAAAABNEE3WRCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGMJQGU4DEMJXHE
.
You are receiving this because you were mentioned.Message ID:
<Manoj-2702/PoseSync-Flower-Federated-Learning-for-Yoga-Poses/issues/1/2310582179
@github.com>

@ibad321 Can you please describe the question properly? I didnt get you

@Manoj-2702
I run the project Actually the model is overfit its give 100 training Acuuarcy on client 2 and around 60 to 70 Accuarcy on validation and testing Accuracy also around 60 to 70 and one thing Another you created A stremlit App which does not predict actual pose you write only  Placeholder Class Name 7 and Same yoga name.

@ibad321 Yes, the model is overfitted due to scarcity of the data. We have trained it on very less amount of Yoga Poses.

@ibad321 In your Clients, you can define the model you want to train it on. So, basically that should contain the architecture of the model you are training on. Make sure that the architecture remains the same in all the clients.
In the streamlit app, I have used a placeholder as of now, but as you can see there is a commented line for predicting the poses as well. Hence please go through the code once again.

@ibad321 Yes, the model is overfitted due to scarcity of the data. We have trained it on very less amount of Yoga Poses.

@Manoj-2702
Thanks For Responce
Actually i am Requested to You i am looking GAN  Mode implementation  on any Dataset  Through Flower There is no Github Repository With Due Respect i Request to you  Add new Branch To implement GAN model On any dataset like cifer10 Etc
Thanks

@ibad321 Yeah sure, we can do that. I will create a branch for that. You can start contributing.

@ibad321 Yeah sure, we can do that. I will create a branch for that. You can start contributing.

@Manoj-2702
Thanks For Responce
I am new in federated learning and I have Run Cnn on Cifer10 dataset  but i am fail to train GAN model i will proved Code So i am  requesting  to you please if you kindly do it  Will help and I will appreciate it.
Thanks

@ibad321 You can provide me the code. I will try my best to integrate it. But due to my college timelines, it may take some time.

@ibad321 You can provide me the code. I will try my best to integrate it. But due to my college timelines, it may take some time.

@Manoj-2702
Thanks for responce
I think the code is fine may be flower version issue  you look its one i think you can do Easily.
client.py
import flwr as fl
import tensorflow as tf
import argparse
from keras.datasets.mnist import load_data
from numpy import expand_dims
from layers import create_model, generator_optimizer, discriminator_optimizer, generator_loss, discriminator_loss, seed, generate_and_save_images

BATCH_SIZE = 256
noise_dim = 100

def main() -> None:
    # Parse command line argument `partition`
    parser = argparse.ArgumentParser(description="Flower")
    parser.add_argument("--partition", type=int, choices=range(0, 2), required=True)
    args = parser.parse_args()

    x_train, x_test = load_partition(args.partition)
    x_train = expand_dims(x_train, axis=-1)
    x_train = x_train.astype('float32')
    x_train = x_train / 255.0

    x_test = expand_dims(x_test, axis=-1)
    x_test = x_test.astype('float32')
    x_test = x_test / 255.0

    model = create_model()

    client = GanClient(model, x_train, x_test)
    fl.client.start_numpy_client("[::]:8080", client=client)

def load_partition(idx: int):
    (x_train, y_train), (x_test, y_test) = load_data()
    x_train = expand_dims(x_train, axis=-1)
    x_train = x_train.astype('float32')
    x_train = x_train / 255.0

    x_test = expand_dims(x_test, axis=-1)
    x_test = x_test.astype('float32')
    x_test = x_test / 255.0
    return x_train[idx * 30000 : (idx + 1) * 30000], x_test[idx * 5000 : (idx + 1) * 5000]

class GanClient(fl.client.NumPyClient):
    def __init__(self, model, x_train, x_test):
        self.model = model
        self.x_train = x_train
        self.x_test = x_test
        x_train_ds = tf.data.Dataset.from_tensor_slices((x_train))
        self.x_train_ds = x_train_ds.batch(BATCH_SIZE)  # batch_size can be 1
        x_test_ds = tf.data.Dataset.from_tensor_slices((x_test))
        self.x_test_ds = x_test_ds.batch(BATCH_SIZE)  # batch_size can be 1

    def get_parameters(self):
        return self.model.get_weights()

    def fit(self, parameters, config):
        self.model.set_weights(parameters)
        generator = self.model.layers[0]
        discriminator = self.model.layers[1]
        for i, images in enumerate(self.x_train_ds):
            noise = tf.random.normal([BATCH_SIZE, noise_dim])
            with tf.GradientTape() as gen_tape, tf.GradientTape() as disc_tape:
                generated_images = generator(noise, training=True)

                real_output = discriminator(images, training=True)
                fake_output = discriminator(generated_images, training=True)

                gen_loss = generator_loss(fake_output)
                disc_loss = discriminator_loss(real_output, fake_output)

            gradients_of_generator = gen_tape.gradient(gen_loss, generator.trainable_variables)
            gradients_of_discriminator = disc_tape.gradient(disc_loss, discriminator.trainable_variables)

            generator_optimizer.apply_gradients(zip(gradients_of_generator, generator.trainable_variables))
            discriminator_optimizer.apply_gradients(zip(gradients_of_discriminator, discriminator.trainable_variables))
            print('%d d=%.3f, g=%.3f' % (i + 1, disc_loss, gen_loss))
        return self.model.get_weights(), len(self.x_train), {}

    def evaluate(self, parameters, config):
        generator = self.model.layers[0]
        discriminator = self.model.layers[1]
        noise = tf.random.normal([BATCH_SIZE, noise_dim])
        generated_images = generator(noise, training=True)
        real_output = discriminator(self.x_test, training=True)
        fake_output = discriminator(generated_images, training=True)
        loss = discriminator_loss(real_output, fake_output)
        generate_and_save_images(self.model.layers[0], seed)
        np = float(loss.numpy())
        return np, len(self.x_test), {}

if __name__ == "__main__":
    main()

server.py
import flwr as fl
import numpy as np
from typing import Optional

class SaveModelStrategy(fl.server.strategy.FedAvg):
    def aggregate_fit(
        self,
        rnd: int,
        results: list[tuple[fl.server.client_proxy.ClientProxy, fl.common.FitRes]],
        failures: list[BaseException],
    ) -> Optional[fl.common.Weights]:
        aggregated_weights = super().aggregate_fit(rnd, results, failures)
        if aggregated_weights is not None:
            # Save aggregated_weights
            print(f"Saving round {rnd} aggregated_weights...")
            np.savez(f"round-{rnd}-weights.npz", *aggregated_weights)
        return aggregated_weights

# Create strategy and run server
strategy = SaveModelStrategy(
    fraction_fit=0.1,  # Sample 10% of available clients for the next round
    min_fit_clients=2,  # Minimum number of clients to be sampled for the next round
    min_eval_clients=2,
    min_available_clients=2,
)
fl.server.start_server(config={"num_rounds": 20}, strategy=strategy)

layer.py (Architecture of GAN)
import tensorflow as tf
from tensorflow.keras.models import Sequential
from tensorflow.keras import layers
import matplotlib.pyplot as plt

def define_discriminator():
    model = tf.keras.Sequential()
    model.add(layers.Conv2D(64, (5, 5), strides=(2, 2), padding='same',
                                     input_shape=[28, 28, 1]))
    model.add(layers.LeakyReLU())
    model.add(layers.Dropout(0.3))

    model.add(layers.Conv2D(128, (5, 5), strides=(2, 2), padding='same'))
    model.add(layers.LeakyReLU())
    model.add(layers.Dropout(0.3))

    model.add(layers.Flatten())
    model.add(layers.Dense(1))

    return model

def define_generator():
    model = tf.keras.Sequential()
    model.add(layers.Dense(7*7*256, use_bias=False, input_shape=(100,)))

    model.add(layers.BatchNormalization())
    model.add(layers.LeakyReLU())

    model.add(layers.Reshape((7, 7, 256)))
    assert model.output_shape == (None, 7, 7, 256)  # Note: None is the batch size

    model.add(layers.Conv2DTranspose(128, (5, 5), strides=(1, 1), padding='same', use_bias=False))
    assert model.output_shape == (None, 7, 7, 128)
    model.add(layers.BatchNormalization())
    model.add(layers.LeakyReLU())

    model.add(layers.Conv2DTranspose(64, (5, 5), strides=(2, 2), padding='same', use_bias=False))
    assert model.output_shape == (None, 14, 14, 64)
    model.add(layers.BatchNormalization())
    model.add(layers.LeakyReLU())

    model.add(layers.Conv2DTranspose(1, (5, 5), strides=(2, 2), padding='same', use_bias=False, activation='tanh'))
    assert model.output_shape == (None, 28, 28, 1)

    return model

def create_model():
    model = Sequential()
    model.add(define_generator())
    model.add(define_discriminator())
    return model

def discriminator_loss(real_output, fake_output):
    real_loss = cross_entropy(tf.ones_like(real_output), real_output)
    fake_loss = cross_entropy(tf.zeros_like(fake_output), fake_output)
    total_loss = real_loss + fake_loss
    return total_loss

def generator_loss(fake_output):
    return cross_entropy(tf.ones_like(fake_output), fake_output)

def generate_and_save_images(model, test_input):
    predictions = model(test_input, training=False)

    fig = plt.figure(figsize=(4, 4))

    for i in range(predictions.shape[0]):
        plt.subplot(4, 4, i+1)
        plt.imshow(predictions[i, :, :, 0] * 127.5 + 127.5, cmap='gray')
        plt.axis('off')

    plt.savefig('image.png')
    plt.show()

cross_entropy = tf.keras.losses.BinaryCrossentropy(from_logits=True)
generator_optimizer = tf.keras.optimizers.Adam(1e-4)
discriminator_optimizer = tf.keras.optimizers.Adam(1e-4)
seed = tf.random.normal([16, 100])

@ibad321 You can provide me the code. I will try my best to integrate it. But due to my college timelines, it may take some time.
@Manoj-2702
Thanks you So much For Accepting My Request i think You can debug This code And add new branch of GAN mean GAN model Implementation.


hi

hi,
You may submit changes in your own repo.
xh54056 @.***>于2024年9月30日 周一09:01写道：

hi
—
Reply to this email directly, view it on GitHub
https://github.com/senli1073/senli1073.github.io/pull/3#issuecomment-2381802458,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A2XV32BC5UOXWDHTAPUQ4WDZZCPG3AVCNFSM6AAAAABPCG4EH6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGOBRHAYDENBVHA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


Improve executable path detection

Improve unix python3 detection. Right now, it's assumed that there is a python3 binary in the bin directory. This isn't necessarily the case, such as when pyenv is misconfigured.

Hi @rpkilby, I am windering would that interpreter detection not better be a job for tox directly or another plugin? Maybe you mean something different, but I would not like us to have to do duplicate work.

ah o.k. that is specific to using venv then. Thanks for the detailed explanation.

Another possible case is when a linux distro provides multiple python 3 packages. eg, CentOS epel-release provides both python34 and python36 packages. The former is considered the "current" version of python, and provides both the python3 and python3.4 binaries, while the latter only provides python3.6.
Versions should be detected with decreasing specificity (3.4.1 => 3.4 => 3).


Qq

[Enter feedback here]

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

ID: 1a8e7568-6b0d-1b26-38f9-58f0367537cf
Version Independent ID: 3d45e246-9d55-a607-1b3e-7f60efeaf2b6
Content: Set up sign-up and sign-in with a QQ account using Azure Active Directory B2C
Content Source: articles/active-directory-b2c/identity-provider-qq.md
Service: active-directory
Sub-service: b2c
GitHub Login: @garrodonnell
Microsoft Alias: godonnell

@RaBenz168
Could you please elaborate the query for better understanding. So, we can assist you further.

@RaBenz168
It would be great if you could add some information. This would help us redirect the issue to the appropriate team. Thanks!

@RaBenz168
We are going to close this thread if there are any further questions regarding the documentation, please tag me in your reply and we will be happy to continue the conversation.


refactor healthcheck

under-pressure is currently set up only for the healthcheck while it should be set up for all routes. Morever, it should use https://github.com/fastify/under-pressure#status-route and https://github.com/fastify/under-pressure#custom-health-checks instead of building its own route.

@mcollina when replacing the custom /healthcheck url with the under-pressure one there is a catch to consider.
Right now Titus /healthcheck returns an object of this form
{
  version,
  serverTimestamp,
  status,
  memoryUsage,
  db: 'ok'/'fail'
}

With under-pressure we can do the same checks Titus does, but we cannot change the response to match the above object (it will always be { status: 'ok' })
Should we go ahead anyway?

Let's make that configurable in under-pressure

@mcollina I would change the onStatus function to allow healthCheck to return

false/true
object

In case an object is returned, under-pressure will merge it with { status: 'ok', ...<result healthCheck call> }.
Would that be fine? should I create an issue in https://github.com/fastify/under-pressure ?

yeah let's extend under-pressure to support returning a custom response if @mcollina agrees

Go ahead and send a PR and refer to this issue. This is open source anyway.

@mcollina @simoneb PR in under-pressure is here https://github.com/fastify/under-pressure/pull/84


genhtml third party extension to write markdown in python

genhtml allows to write python code that will be executed at compile time, and its standard output will be copied in place, replacing the python code itself by either markdown or html.
I'm using it for auto-generating images and charts in my blog, or compile-time templating.
You can think of it as a reversed jupyter notebook.
You may want to include it in the third-party list of extensions.

You are free to edit the wiki and add 3rd party extensions. The wiki is maintained by the community.


Easily list the names of the env var required to set a field

Is there an easy way to get/print the name of the env var that is used to set the value of a field?
Problem:
You have a bunch of configs (potentially nested) and you quickly want to know what the name of the env var for a given field is.
Current workaround:
Right now, I read the code, the nesting of the models, and the SettingsConfigDict to infer the name of the field. Then I try names until I find the correct one.
Potential solution:
Add a new function like print_env_var_names() that works something like this:
class MyModel(

class MyModel(BaseSettings):
    model_config = SettingsConfigDict(env_prefix='FOO__')

    bar: int=5

print_env_var_names(MyModel)
# MyModel.bar -> FOO__BAR

Thanks @sotte for this issue. It is not possible to list env var names right now. I think it would be good addition and will help a lot

Thanks @hramezani. #287 is indeed very similar (though I would not have guessed that from just the headline). I'm closing this as duplicate.


deno: 1.1.0 -> 1.1.1

Motivation for this change
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[X] NixOS (x86)
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Thanks for the suggestion @danieldk
At the cost of 14 extra characters + a newline I personally prefer the appearance and clarity of the original.
Also it makes it very easy to add stuff if in the future all architectures buildInputs, or if there's another architecture that needs a set of specific buildInputs

At the cost of 14 extra characters + a newline I personally prefer the appearance and clarity of the original.

I kindly disagree. An additional glitch is that stdenv.lib is introduced to the lexical scope of []. But at any rate, this is only a small nitpick :wink:.

Thanks for merging.

An additional glitch is that stdenv.lib is introduced to the lexical scope of [].

Interesting. I guess I'll change it going forward :sweat_smile:
I'm pretty new to nix so I'd love to know the potential consequences of doing this. :slightly_smiling_face:
I'm guessing adding with here buildInputs = with stdenv; lib.optionals isDarwin [ Security CoreServices ]; run into the same issue?


Set inputs of startPoint and endPoint in the shipBoard

https://github.com/sopra-fs23-group-11/sopra-fs23-group-11-client/commit/7cc6e640ee973e093fa07f80ac4e99b2fe38ebee


Eliminate Foreign Data (or get close to it)

react-native supports a subset of CSS styles created using the (effectful?) StyleSheet.create function. It'd be awesome to give this a full Union type (as DOM.css should have as well..).

this is trivial to add, but I'd like your opinion - many of the types on the fields of the CSS types are Typescript Union types of the form number | string - for such types I've been simply shortening them to string. Thoughts?

That makes sense, since the string version can do everything the number one can.

I changed the name of this issue to because it's essentially the same as "make better types for events" - I want to fully eliminate all of the foreign data imports and expand the types as much as possible. I have just ~25 types remaining, and I think it's possible.

@spicydonuts I wonder if you've encountered this - ImageLoadEventData has a field uri that may be undefined. https://github.com/DefinitelyTyped/DefinitelyTyped/blob/master/types/react-native/index.d.ts#L3679
I've used purescript-nullable, but this is undefined? Any suggestions for how to handle it?

Looks like there is purescript-undefinable

That might work. I usually end up defining an FFI import when I need undefined

(there's also a decent chance null will work, as they're usually treated the same in TS)

update on this  - I've just about got this all worked out in the events branch (bad name of the brand). Hopefully it'll be in a place to merge to master today and then make a release

I need some feedback. I'm about ready to merge this, but I'm unsatisfied with the way I've got events working. Take a look at the controlled-input example. In order for that example to work, the persist function/method must be called on the event or accessing the fields will result in an NPE. I could do something like make the NativeSyntheticEvent type opaque and make the record available via some function like persist :: NativeSyntheticEvent -> Effect RecordType. I haven't used React Basic or react enough to really know what the best way to go next is.  I'll do that if I don't get any feedback, but I think there is probably a better way.
Thanks!
cc @spicydonuts  @jvliwanag @joelmccracken @jkachmar

My proposal is to define NativeSyntheticEvent as a foreign data type:
foreign import data NativeSyntheticEvent :: Type -> Type

From what I understand, the closest definition, say for timeStamp to the behavior of react is:
timeStamp :: forall a. NativeSyntheticEvent a -> Effect (Maybe Number)

So it's an Effect since the timeStamp js field can change value. In particular, if accessed within the handler, it is Just, and Nothing when accessed asynchronously.
But having Maybe is a bit tedious, and we know that if it were accessed within the handler (say logShow evt.timeStamp in the handler), it really should have a value.
So instead of the previous definition of timeStamp, react-basic instead use:
timeStamp :: forall a. EventFn (NativeSyntheticEvent a) Number

and then we have handler:
type EventHandler n = EffectFn1 (NativeSyntheticEvent n) Unit

handler :: forall n a. EventFn (NativeSyntheticEvent n) a -> (a -> Effect Unit) -> EventHandler n

So the idea is you build an EventFn to access the fields you want on the event, and convert it to some a. At this stage, you can expect that all fields of the event are available.
There are a few caveats though:
The EventFn, handler, merge, etc functions in react-basic are hardcoded against SyntheticEvent which simply is kind Type. The difference with react-native right now, is we know the type of nativeEvent, so it's Type -> Type. So we might have to duplicate variants of these for react-native-basic.
See draft here: https://github.com/jvliwanag/purescript-react-basic-native/commit/93ea11fb07c94afa73ac949df53c5c60af6622a6

@jvliwanag this necessitated some pretty nasty type annotations  when using capture/merge. I haven't really got time to work on this today. If you have some ideas and want to make a PR, that'd be great!

We use the linear EventFn type for this in the DOM implementation: https://github.com/lumihq/purescript-react-basic/blob/master/src/React/Basic/DOM/Events.purs
The machinery lives outisde the DOM modules though, so you can use it here too: https://github.com/lumihq/purescript-react-basic/blob/master/src/React/Basic/Events.purs#L48
It'd be nice to keep that pattern. The linear type makes property access safe (a synthetic event can't leak out into a callback's scope -- values have to be read synchronously).

Doh, sorry, @jvliwanag already explained that. I'm not sure a new type is needed for SyntheticEvent, just a React.Basic.Native.Events module similar to the DOM one which defines the valid accessors.

The event types in React Native are a bit more exotic/rich than they are in React, so I'd like to keep the NativeSyntheticEvent as @jvliwanag suggests. The only thing that kinda sucks is the need for the type annotations when using merge, but on reflection, I don't think it's all that bad. I'm not sure how often one needs merge, so I think I'll just fill out the rest of the EventFns we need and then merge this


Heads Up: Pending release notification

I performed a quick GitHub repo search to see who was using eslint-plugin-wordpress via the GitHub repo using master or a commit hash rather than a packages release on npmjs.com:
• imath/gutenprez - @imath
• manishsongirkar/rt-gutenberg-blocks - @manishsongirkar
• rtCamp/gutenberg-supplements - @sagarkbhatt
• rtCamp/gutenberg-plugin-boilerplate - @sagarkbhatt
• skyshab/taproot-hero-image-block/ - @skyshab
• usine/wp-voxusini - @medfreeman
Consider this a "heads up" notification that there will be a new 1.0.0 release in the coming days that will break backward compatability, once the 1.0.0 release is shipped I suggest that you change your repos to use that rather than the repo URL.

Closing this as there is nothing else actionable required from eslint-plugin-wordpress here


Update Pages “contact”

Automatically generated by Netlify CMS

👷 Deploy Preview for safehavenaccounting processing.

Name
Link

🔨 Latest commit
4b78d7424f8c838b95325e64f59f89600dd390aa

🔍 Latest deploy log
https://app.netlify.com/sites/safehavenaccounting/deploys/64933368ac73c40008d338b8


Frontend: Vanilla JS

Description 📜
Fixes #3636

Type of change 📝

[x] Doc#umentation (Content Creation in the form of codes or tutorials)

Domain of Contribution 📊

[x] Frontend Dev #HTML/CSS/JS

Checklist ✅

[x] I follow Contributing Guidelines & Code of conduct of this project.
[x] I have performed a self-review of my own code or work.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generates no new warnings.
[x] I'm GWOC'21 contributor

@ghulamyazdani please review


[Suggestion] Proper strafe prediction method

Let x, y, and z be the 3-Dimensional coordinates of the target.
Let vₓ, vᵧ and vz be the x, y, and z velocities of the target.
Let γ, ϕa,and ϕ2 be the initial roll, pitch and yaw values respectively.
Let γ˙,ϕ˙a, and ϕ˙2 be the angular roll velocity, angular pitch velocity and angular yaw velocity respectively.
Assuming that roll does not have an effect on the trajectory of the target, let γ = O and γ˙ = O, where O is a zero element.
If the player data at time instant ti and ti+1 are known, the positions of the target is expanded using a second⁃order Taylor series at ti as
$X_{i+1}=X_i+V_iΔt+\frac{1}{2}A_iΔt^2$
where Xi and Xi+1 are the position coordinates of the target at ti and ti+1 ， respectively; Vi is the velocities of the target at ti ； $Δt=t_{i+1}−t_i$ the difference between ti and ti+1 ； Ai the accelerations of the target at ti .
Further, it can be deduced that $A_i=\frac{2}{(Δt^2)}(x_{i+1}, x_i, V_iΔt)$. $A_i$ can also be expressed as $(\frac{d_{v_x}}{dt},\frac{d_{v_y}}{dt},\frac{d_{v_z}}{dt})$i. $(\frac{d_{v_x}}{dt},\frac{d_{v_y}}{dt},\frac{d_{v_z}}{dt})$ can be considered as a nonlinear function with respect to $(γ˙,ϕ_a,ϕ_2)$ at $t_i$, namely, $F_i(γ˙,ϕ_a,ϕ_2)= A_i - \frac{2}{Δt^2}(X_{i+1}-X_i-V_iΔt)$.


Build dashboard for Prometheus cost

As we are moving forward with Mimir we want to be able to compare cost between the upcoming Mimir and our current Prometheus setup.
Let's build a dashboard to expose the current Prometheus cost with the following informations :

CPU/Memory usage
Storage usage
Network transfer usage
Number of timeseries

This dashboard will ideally be similar to what was done for Loki https://github.com/giantswarm/roadmap/issues/2182

Dashboard deployed


Update Cargo.toml add repository

to allow crates.io, rust-digger, and others to link to it

Thanks for accepting the PR. I am building the Rust Digger to find such "low hanging fruits" to fix. There is a page listing all of your repos. I hope it will be useful to you too.

@szabgab Excellent


Can we pass data volume to ext_authz

Title: Is it possible to send bytes sent/received through ext_authz
Description:
I want my external authorization to receive information about bytes sent/received to make a decision. Is there some way to do this in envoy.filters.http.ext_authz and envoy.filters.network.ext_authz? Or any workaround that may help? I see that we may be able to work around this by adding additional headers with the needed info using the http lua filter, but don't see anything similar in network filter.

Authorization decision have to be made on the earliest bytes in the request headers or the request body. See https://github.com/envoyproxy/envoy/blob/3717be0b78a9b9601eddfc912db42cb30ead9094/api/envoy/extensions/filters/http/ext_authz/v3/ext_authz.proto#L74 for how to include request body. If you want to inspect further, consider using ext_proc.


tsp.interpret returns nested lists

The return type of tsp.interpret seems quite strange: https://github.com/Qiskit/qiskit-optimization/blob/3276eaa9917e03923a76390050ed5cea2ecae6e6/qiskit_optimization/applications/tsp.py#L65
Sometimes when running tsp.interpret on a valid result, one can get complete nonsense like [4, [2, 3], [], 1]. In a project of mine, I solved this with np.hstack (see my code here) but it's strange to me that this is a problem at all.
I would expect a simple one-dimensional array of indices, and the tutorial does too! See https://github.com/Qiskit/qiskit-optimization/blob/3276eaa9917e03923a76390050ed5cea2ecae6e6/docs/tutorials/06_examples_max_cut_and_tsp.ipynb?short_path=a29aa13#L852.
After a little bit of poking around in the code, it seems to me like the iteration in the tsp.interpret function is handled wrong.
I would not mind fixing this if I get confirmation that this is an actual issue and not me misinterpreting something :)

This issue runs deeper than I thought. Looking through old commits, it seems like there was a note about how the output can contain nested lists if the result is infeasible, but this was removed when the whole applications module was refactored in #31  https://github.com/Qiskit/qiskit-optimization/blob/c14e2603b29ded8847d57288831c5ee0563dd5c4/qiskit_optimization/applications/ising/tsp.py#L257-L261
I'm still getting weird results even when the result is considered feasible; consider this run I produced:
feasible: True
solution: [3, [], 1, 2]

Parameters I used to generate this error
This is just a sketch of the code.
tsp = Tsp.create_random_instance(nodes, seed=333337)
algorithm_globals.random_seed = 123
seed = 10598

qp = tsp.to_quadratic_program()
qp2qubo = QuadraticProgramToQubo()
qubo = qp2qubo.convert(qp)
qubitOp, offset = qubo.to_ising()

quantum_instance = QuantumInstance(Aer.get_backend("qasm_simulator"), seed_simulator=seed, seed_transpiler=seed)
spsa = SPSA(maxiter=300)
ry = TwoLocal(qubitOp.num_qubits, "ry", "cz", reps=10, entanglement="linear")
vqe = VQE(ry, optimizer=spsa, quantum_instance=quantum_instance)

result = vqe.compute_minimum_eigenvalue(qubitOp)

x = tsp.sample_most_likely(result.eigenstate)
print("feasible:", qubo.is_feasible(x))
z = tsp.interpret(x)
print("solution:", z)

# x is [0 0 0 0 0 0 1 0 0 0 0 1 1 0 0 0]

I'm still looking for the source of the discrepancy (interestingly, using NumPyMinimumEigensolver does not have this issue at all).

Could you take a look at it, @a-matsuo?

Sure, I look into it.

The behavior is expected, isn't it? Since qubo does not have constraints, it will return feasible for any solution. If you use qp.is_feasible(x) instead of qubo.is_feasible(x), it will return False.

Why is qp.is_feasible(x) false here, given that this is a valid, solvable TSP problem? Is this indicative of a problem somewhere in the TSP -> QP chain of conversions?

I mean x=[0 0 0 0 0 0 1 0 0 0 0 1 1 0 0 0] as you showed in the above is an invalid solution, isn't it? It never visits city 0. is_feasible(x) checks whether a given x violates constraints or not.

VQE does not guarantee the optimality and the solution does not always satisfy all constraints. So, although qubo.is_feasible(x) == True of the solution x=[0 0 0 0 0 0 1 0 0 0 0 1 1 0 0 0] (QUBO has no constraint), qp.is_feasible(x) == False. If qp.is_feasible(x) == False, interpret may generate a nonsense list due to the infeasibility.

Makes sense, thank you for your help.


Add webkit media control pseudo elements

I've added a collection of webkit pseudo elements to the whitelist. These selectors are used for styling parts of the shadow DOM for media controls.
Bumped into a couple of these today and had to surround the affected code with disable/enable comments for now.
Let me know if there's any issues, and thanks for the awesome lib! 👍

Can you please link to some documentation or code somewhere that shows these?

@lencioni Certainly!
Here's a bunch: https://chromium.googlesource.com/chromium/blink/+/master/Source/core/css/mediaControls.css
The -webkit-media-controls-start-playback-button one is unique to iOS
https://github.com/WebKit/webkit/blob/c595dc9b3993d095e25311b0ec1797bd665447e8/Source/WebCore/Modules/mediacontrols/mediaControlsiOS.css

Thanks for the links!


[CLOSED] Add missing which argument to get_issues_by_id call

 Issue by gordonwoodhull
Sunday Nov 03, 2013 at 23:12 GMT
Originally opened as https://github.com/IQAndreas/github-issues-import/pull/18

thanks for the useful tool!
--issues was failing due to a missing argument to the get_issues_by_id function
seems to work with this change

gordonwoodhull included the following code: https://github.com/IQAndreas/github-issues-import/pull/18/commits

 Comment by IQAndreas
Sunday Nov 10, 2013 at 20:19 GMT

Great catch, thanks!


chore: set unchangeable tag in rendered

Description
Set changable tags to one CURRENT-VERSION so it doesn't pops up on every PR after makr ender process.
Fixes # (issue)
Type of change
Please delete options that are not relevant.

[x] Refactor/improvement

How Has This Been Tested?
N/A
Checklist
N/A

Codecov Report

Merging #768 (c18219d) into develop (f3b9967) will not change coverage.
The diff coverage is n/a.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@           Coverage Diff            @@
##           develop     #768   +/-   ##
========================================
  Coverage    87.43%   87.43%
========================================
  Files           27       27
  Lines         1815     1815
========================================
  Hits          1587     1587
  Misses         228      228

:tada: This PR is included in version 1.9.2-beta.2 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


feature request: indicate a request should not generate a completion event

When adding a set of linked request, the application is mostly interested when the last request completes. Completions of the IOSQE_IO_LINK requests are ignored.
Perhaps a flag can be added so these requests won't create completion entries, and they don't count for io_uring_enter min_complete?

This is already in motion, see:
<EMAIL_ADDRESS>and also:
https://github.com/axboe/liburing/issues/53

This is already in motion, see:

Great!
In my use-case, I'm doing poll, linked tosend/rev.
The poll should be cancellable even when IOSQE_IGNORE_CQE is set.
It would be nice to ensure the linked requests can use the same user_data. This can then be used to cancel the linked sequence, and is used for the completion event.
It avoids having to generate unique user_data for each request of a linked sequence.

#53

Maybe it makes sense to have a sentinel value for user_data that is detected when IOSQE_IGNORE_CQE is set? This value indicates the request isn't cancellable and avoids collisions with requests which are cancellable/generate completion events (and therefore need a user_data).
@axboe can you take a look at the questions I've added as issues?

This value indicates the request isn't cancellable and avoids collisions with requests which are cancellable/generate completion events (and therefore need a user_data).

A use-case, next to LIBURING_UDATA_TIMEOUT, is with IORING_OP_ASYNC_CANCEL.


update proxy of release-6.5 to raftstore-proxy-6.5 by proxy pr 289

What problem does this PR solve?
Issue Number: ref #4982
Problem Summary:
update proxy to raftstore-proxy-6.5
Proxy PR:
https://github.com/pingcap/tidb-engine-ext/pull/289
Including:\nSubmodule contrib/tiflash-proxy aff4270ced..3fa3f4867d:

Update raftstore-proxy-6.5 for openssl bugfix (#289)

What is changed and how it works?
Check List
Tests

[ ] Unit test
[ ] Integration test
[ ] Manual test (add detailed scripts or steps below)
[ ] No code

Side effects

[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[ ] Breaking backward compatibility

Documentation

[ ] Affects user behaviors
[ ] Contains syntax changes
[ ] Contains variable changes
[ ] Contains experimental features
[ ] Changes MySQL compatibility

Release note

None

/build

/run-all-tests

/run-integration-test

/merge

/run-integration-test


Minor: Replace magic string in JsonConverter by constants

Committer Checklist (excluded from commit message)

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

this is fixed by #7354


Fix BaseName initialization

Initializing the BaseName with a ServiceDirectory that contains
a "/", because it is a multiple level path, causes the BaseName
initialization code to fail because it doesn't support the
validation pattern.
In all the known cases we already pass the ResourceGroupName
explicitly so don't need to set the BaseName so we can
skip the initialization in those cases.

The following pipelines have been queued for testing:java - templatejava - template - testsjs - templatenet - templatenet - template - testspython - templatepython - template - tests You can sign off on the approval gate to test the release stage of each pipeline. See eng/common workflow

Testing on the python smoke-test pipeline https://dev.azure.com/azure-sdk/internal/_build/results?buildId=835055&view=results.

/check-enforce override


Add support for freq=None in missing_* functions

Pull Request Checklist:

[x] This PR addresses an already opened issue (for bug fixes / features)

This PR fixes #432

[x] Tests for the changes have been added (for bug fixes / features)
[ ] Documentation has been added / updated (for bug fixes / features)
[ ] HISTORY.rst has been updated (with summary of main changes)
[ ] bumpversion (minor / major / patch) has been called on this branch
[ ] Tags have been pushed (git push --tags)

What kind of change does this PR introduce?
missing functions can now handle freq=None, which is useful when computing climatologies.

Does this PR introduce a breaking change?
No

Other information:

Pull Request Test Coverage Report for Build 1803

24 of 24 (100.0%)  changed or added relevant lines in 3 files are covered.
9 unchanged lines in 3 files lost coverage.
Overall coverage increased (+0.03%) to 76.365%

Files with Coverage Reduction
New Missed Lines
%

xclim/sdba/utils.py
1
0%

xclim/sdba/base.py
2
0%

xclim/core/checks.py
6
69.87%

Totals

Change from base Build 1798:
0.03%

Covered Lines:
2391

Relevant Lines:
3131

💛  - Coveralls


Tooltip conflict with other scripts

Url: http://studentedimedicinadimerda.it/
Username: Rahul
Password: anspresstesting
Here you can see the error, just hover over an avatar or something else.
http://studentedimedicinadimerda.it/domanda/ciao-come-se-fa-a-fa-na-cosa/
I think that there is a conflict with my theme, when a user hover the tooltip used in AnsPress it appears suddenly another scrollbar at the right corner of the screen, and when the user move the mouse from the hovered link the bar disappear leaving a black bar.
The wordpress theme used is: Unicon (themeforest)

Hi, there is a overflow-x issue in tooltipster v3 that has been fixed in the upcoming v4, it may well be that. Thank you for the report though.

Any ETA?

I'll try to release it before the end of the summer, if I can. But it's already available on the v4 branch of the project, it's basically just lacking the support of image maps and documentation updates.


fix: add array values to stack to fix decirc hang

Add items in array to stack prior to calling decirc to fix problems with deep nesting, reported in https://github.com/pinojs/pino/issues/990

Closing as not a valid fix


Connectors stay static with collapsible blocks.

i have multiple collapsible blocks but when i collapse them, the connectors dont move.

Hello @iamnikhilrohan
Update the node connections with the method. When it collapses or uncollapses.
Use method:
updateConnectionNodes(id)

Hello @iamnikhilrohan
Update the node connections with the method. When it collapses or uncollapses.
Use method:
updateConnectionNodes(id)

i tried doing that but still it does not work.

Any error in the console?
Sorry, look at the documentation again
It is called like this:
editor.updateConnectionNodes('node-5');

use
editor.updateConnectionNodes('node-'+selectedNode);

Any error in the console?
Sorry, look at the documentation again
It is called like this:
editor.updateConnectionNodes('node-5');

use
editor.updateConnectionNodes('node-'+selectedNode);

i tried that too and it didn't work.
is there any way i can move the bubble position to the top of the part where by block header is placed instead of the middle of the block?

You can modify them with css


Remove unwanted filters from node table

Description
Remove unwanted filters

Min SSD
Min HDD
Min Ram

Changes

Related Issues

#2132

Checklist

[ ] Tests included
[x] Build pass
[x] Documentation
[x] Code format and docstrings
[x] Screenshots/Video attached (needed for UI changes)

As we gonna merge the dedicated machines page into the node finder page, we will need all the filters


Bug in drawn line

hello
I'm getting this as the jason description of a line drawn on the canvas:
I want to calculate the gradient of the line so was checking x1,y1 x2, y2 and they always come back as negatives of each other.
[{'type': 'line', 'version': '3.6.3', 'originX': 'center', 'originY': 'center', 'left': 130.5, 'top': 213.5, 'width': 89, 'height': 15, 'fill': '#65EA47', 'stroke': '#65EA47', 'strokeWidth': 3, 'strokeDashArray': None, 'strokeLineCap': 'butt', 'strokeDashOffset': 0, 'strokeLineJoin': 'miter', 'strokeMiterLimit': 4, 'scaleX': 1, 'scaleY': 1, 'angle': 0, 'flipX': False, 'flipY': False, 'opacity': 1, 'shadow': None, 'visible': True, 'clipTo': None, 'backgroundColor': '', 'fillRule': 'nonzero', 'paintFirst': 'fill', 'globalCompositeOperation': 'source-over', 'transformMatrix': None, 'skewX': 0, 'skewY': 0, 'x1': -44.5, 'x2': 44.5, 'y1': -7.5, 'y2': 7.5}], 'background': '#eee'

Hey @Aya-S and welcome!
I've been digging the Fabric.js docs, and it seems that by definition:

x1 = -width/2
y1 = -height/2
x2 = width/2
y2 = height/2

This would be because the line center is considered the origin of the line so (x1;y1) is where you first created the line and (x2;y2) where you released it, and those coordinates are relative to the line center.
Using 2 * x2 and 2 * y2 values yourself give the correct gradients. Careful as JS Canvas consider x grows by going from left to write, and y grows by going from top to bottom.
Beware as if you transform your lines after drawing (like rescaling or rotating), the x2 and y2 won't change, you'd have to apply the transformations yourself by applying the scaleX, scaleY and angle properties.
Hope this helps, don't hesitate if you have more questions,
Fanilo


About baselines PFENet and PFENet\dagger

Hi, thanks for your great work!
For your baselines, PFENet and PFENet\dagger,  could you please share some details about them? Because I could reproduce the results of PFENet, but when I replace the dataset as that of BAM (i.e., remove training samples with novel classes, while all other things remain the same), my obtained results are not as good as what you provide, e.g., 66.35% mIoU of PFENet\dagger (Res-50) provided in Table 1, while <64% in my experiments.

In Table. 10, PFENet-Res50-w/0 has mIoU of 63.34%, but the original one is 60.8%.

@Sam1224 Thanks for your interest.  We reproduced PFENet in the BAM framework. The BAM framework has improved the original pfenet framework. For details, refer to BAM (https://github.com/chunbolang/BAM).

@Sam1224 Thanks for your interest. We reproduced PFENet in the BAM framework. The BAM framework has improved the original pfenet framework. For details, refer to BAM (https://github.com/chunbolang/BAM).

Thanks for your reply. Is it correct that I just need to (1) drop the base learner and merge part of BAM? (2) replace the backbone as that of trained base learners of BAM?

@Sam1224 Thanks for your interest. We reproduced PFENet in the BAM framework. The BAM framework has improved the original pfenet framework. For details, refer to BAM (https://github.com/chunbolang/BAM).

Thanks for your reply. Is it correct that I just need to (1) drop the base learner and merge part of BAM? (2) replace the backbone as that of trained base learners of BAM?

Yes, and you should be noted that the loss of pfenet is different from that of BAM.

@Sam1224 Thanks for your interest. We reproduced PFENet in the BAM framework. The BAM framework has improved the original pfenet framework. For details, refer to BAM (https://github.com/chunbolang/BAM).

Thanks for your reply. Is it correct that I just need to (1) drop the base learner and merge part of BAM? (2) replace the backbone as that of trained base learners of BAM?

Yes, and you should be noted that the loss of pfenet is different from that of BAM.

Thanks a lot! I will close this issue now.

@syp2ysy Hi, thanks for your great work! I found a typo in table 10 of the paper. The mIoU for PFENet-ResNet50-w/0 is 63.34%, not 66.34%.


Question: Possible to include HTML in markdown?

Hi
sorry if this is in the docs already and i was just too stupid to find it:
from my former SSG, i am used to be able to include the occasional  into the markdown which then will not be transformed by the SSG.
One usecase would be to include the usual github/lab badges that everybody loves so much :laughing:
Is there a was to do this/something similar in ZNAI ?

Hi,
sorry was traveling and forgot to reply.
Not possible to include HTML by design. Does :include-image: url works for you? If not, that is a regression and I will be happy to fix it.
Also had plans to have native support for GitHub metrics integration, but didn't have enough use cases for myself.


Team Events Does Not Support type group_external_sharing_setting_override_changed

Describe the bug
The team event group_external_sharing_setting_override_changed is returned as event type "Other", with empty details.
To Reproduce
Modify external sharing override setting for a group, query event using api library.
Expected Behavior
An event with event type GroupExternalSharingSettingOverride and full model support with details tag support for  group_external_sharing_setting_override_changed_details with PreviousValue and NewValue.
Actual Behavior
Event details are not returned, only an event of type "Other", with empty OtherDetails info.
Versions

What version of the SDK are you using? Dropbox.Api v7.0.0
What version of the language are you using? 12.0
What platform are you using? (if applicable) .net 8.0

Additional context
Recommend updating API schema source and re-generating code.

Thanks for the report! I'll ask the team to release an update to support this event type.


view all notes about a patient

As a private nurse I want to view all notes about a patient

Closed via #249.


auth:renew should respect set alternative AccountManager url

Currently, auth:renew calls obtainToken with no AccountManager override. So when one was initially set, renew will try to re-authenticate with the standard AccountManager.

@wundrian is this still relevant?

I'll need to retest, but the need hasn't gone away.

@wundrian Do you make use of the env var SFCC_LOGIN_URL? This is respected by obtainToken. I did a quick test, see below:
$ export SFCC_LOGIN_URL=<your-alt-am-host>
$ sfcc-ci client:auth '<client-id>'' '<client-secret>' --renew
Authentication succeeded Auto-renewal enabled.
$ sfcc-ci client:auth:renew
Authentication renewal succeeded

Feedback?

Good point. Our invocations go via the -a <am-host>, but I think we can work with environment variables.

I think respecting the alternative Account Manager host passed to sfcc-ci client:auth via -a flag, is not trivial. The setting is only visible to the current Node process running the command, this includes playing with process.env.SFCC_LOGIN_URL and setting the host. One option would be to persist the setting into the CLIs local configuration. That however, has the downside that it would be visible across shells and sessions.
I'd personally favor to deprecate the flag -a for command sfcc-ci client:auth and rely on env var SFCC_LOGIN_URL.

I've created #129 for the removal of flag -a,--authserver and closing this one.


fix(explore comma): make that the comma can be added by removing it from token separators…

SUMMARY
Cannot add comma to Number Formatting
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
Before:
Comma doesn't show up, and the cursor jumps to the beginning of the line. Typing a period then erases everything.
It is because that the comma is used as the token separator in select component.
After:
I fixed this issue by removing the comma from the token separators.

ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Required feature flags:
[x] Changes UI
[ ] Includes DB Migration (follow approval process in SIP-59)

[ ] Migration is atomic, supports rollback & is backwards-compatible
[ ] Confirm DB migration upgrade and downgrade tested
[ ] Runtime estimates and downtime expectations provided

[ ] Introduces new feature or API
[ ] Removes existing feature or API

Hi @geido .
Thank you for your review.
As a second way for solving the 'Cannot add comma to Number Formatting' in explore, I can ignore the token separators in case of single selector. And so I mean that the token separators can only work in case of multiple or tags mode.
In other case, thank you if you let me know the solution for this.

If you do enable the AntD prop, it might be good to add a control to the interactive Storybook.

@michael-s-molina @evans .
Resolved the review.

@michael-s-molina
Resolved your comment.

@prosdev0107 just give this a quick rebase, and we should be good to go.


backtrace: no backtrace on macos

For investigating #1827.
Repro with bazel test --test_output=streamed //test/common/buffer/...

once you do the submodule update you should re-test this use case, should see the stack trace now


Should Phony provide Liberator-like features out of the box?

Liberator is pretty great when you need it. However, hindsight from writing Phony's mocks has made me realise it probably doesn't work for passed-by-reference arguments, variadics, and other things that Phony excels at dealing with.
Phake also supports bypassing access modifiers out of the box, so there's precedent for such a feature in a mocking framework.
I also received a question about how to mock protected methods via Twitter, because currently the docs don't mention access modifiers at all (see #186).
@jmalloc, @darianbr, @koden-km, @parkertr Thoughts?

After discussion with @jmalloc, we both agreed it's probably a better idea to just improve Liberator, adding support for the features it's missing right now.


Slash command support

pretty simple, the bot requires slash command support as the new discord API requires it.

Not necessarily. It only requires it for verified bots in over 100 servers.
Because we are not ever intending to expand this bot beyond this server, it is not as big of a deal to switch to slash commands. (Also, personally I just don't like slash commands?)

Not necessarily. It only requires it for verified bots in over 100 servers.
Because we are not ever intending to expand this bot beyond this server, it is not as big of a deal to switch to slash commands. (Also, personally I just don't like slash commands?)

true that discord doesn't require the commands (i had read it wrong), but we can put support for both slash and normal commands simultaneously, as slash commands have a number of benefits such as better error messages and autofill (plus i think they are cool and easier to implement).


Book evolution in main Bevy repository

RENDERED
proposal to help start and improve on the book

This doesn't acknowledge or address the recent plan as laid out here: https://github.com/bevyengine/bevy-website/issues/623#issuecomment-1518887997. I personally think that is still the best path forward.
I'm also not particularly excited about moving book work into the bevy repo:

This increases the friction to work on book in a WYSIWYG way. With the plan above, you can clone the bevy-website repo, run zola serve, and get a hot-reloaded rendered version of the book content. Moving the book content to the bevy repo would mean cloning two repos. If we're embedding bevy content in the bevy-website zola site, that would mean cloning the bevy repo inside bevy_website, which is not how people currently work with these repos.
Adding another largely orthogonal chunk of dev work to the main Bevy repo gives me pause. Sifting through activity in the Bevy repo is already a challenge
It clutters the bevy repo commit history. We'll have real "bevy lib changes" intermingled with "tweak language of Intro Book section" and "fix spelling mistake in render app book section".

I don't think the merged approach is a meaningful improvement to the "sync problem". As long as we have bevy-website/main track either bevy/main or bevy/latest by convention, this feels just as good. And I think we generally want people submitting pull requests to a book branch that tracks bevy/latest (like the plan in https://github.com/bevyengine/bevy-website/issues/623#issuecomment-1518887997), not bevy/main (as this RFC suggests). Imo we should generally be landing and publishing book changes as soon as they are ready. Not holding them back until the next release.

This doesn't acknowledge or address the recent plan as laid out here: bevyengine/bevy-website#623 (comment). I personally think that is still the best path forward.

I think the fact that it hasn't moved in almost 2 months shows it may not be the best path forward...

This increases the friction to work on book in a WYSIWYG way. With the plan linked above, you can clone the bevy-website repo, run zola serve, and get a hot-reloaded rendered version of the book content. Moving the book content to the bevy repo would mean cloning two repos. If we're embedding bevy content in the bevy-website zola site, that would mean cloning the bevy repo inside bevy_website, which is not how people currently work with these repos.

It could be used as a git submodule, and keep working the same way. There are technical solutions to that issue

Adding another largely orthogonal chunk of dev work to the main Bevy repo gives me pause. Sifting through activity in the Bevy repo is already a challenge

With git it's easy enough to filter commits based on what they modified. Here are all the commits on the main repo that modified something in the crates folder: https://github.com/bevyengine/bevy/commits/main/crates. Having the book in another folder would make it easy to ignore if you want.

It clutters the bevy repo commit history. We'll have real "bevy lib changes" intermingled with "tweak language of Intro Book section" and "fix spelling mistake in render app book section".

For me that's actually a feature? No more people opening an issue on the wrong repo because bevy-website is much less visible. It will also increase visibility of book changes which is good.

I don't think the merged approach is a meaningful improvement to the "sync problem". As long as we have bevy-website/main track either bevy/main or bevy/latest by convention, this feels just as good. And I think we generally want people submitting pull requests to a book branch that tracks bevy/latest (like the plan in bevyengine/bevy-website#623 (comment)), not bevy/main (as this RFC suggests).

I disagree, as I think that means the book will always be late. Maybe it doesn't matter that much until Bevy reach 1.0.

Imo we should generally be landing and publishing book changes as soon as they are ready. Not holding them back until the next release.

That seems true while we have a lot to catch up, but not once it's more complete. Which admittedly we still have a way to go to reach.

I've written up a fresh discussion about this in https://github.com/bevyengine/bevy/discussions/16430


Ubuntu - missing icon when using deb package but not when building from source

Problem:
The launcher icon on Ubuntu shows a question mark for a prescribed (as prescribed on https://code.visualstudio.com/docs/setup/linux) install of vscode in an ubuntu:16.04 docker container. Installing instead from source fixes the issue with the question mark icon but there is no clear way to run vscode without having the explorer window open up automatically. It would be nice if setting VSCODE_CWD="" would cause the explorer window not to open up for the source built vscode.

VSCode Version: 1.11.1
OS Version: Ubuntu 16.04

Steps to Reproduce:
Create a docker image of vscode based on the ubuntu:16.04 image using the install instructions from https://code.visualstudio.com/docs/setup/linux. Add X11 support to the image and run a container for this image. Result is that visual studio code can be made to work properly except that the launcher icon shows a question mark.
On the other hand, if you go the long way and create a docker image that installs vscode from source and build it (for me using node 7.8.0 and electron 1.6.3) the launcher icon shows appropriately through X11. A folder is always opened up using this route though and there wasn't any simple configuration that I could find to override this behavior. The most logical way to me seemed to be setting VSCODE_CWD="" but that didn't work.

I've never seen this on Ubuntu 16.04, 16.10 or 17.04. Maybe it's something to do with the docker image?
Not quite sure I understand what you mean with VSCODE_CWD? VS Code'ss cwd should be the cwd of the process that launched it by default.

I've never seen this on Ubuntu 16.04, 16.10 or 17.04. Maybe it's something to do with the docker image?

I am thinking this statement means that if vscode is installed directly on an Ubuntu 16.04, 16.10 or 17.04 machine the vscode icon shows up in the Unity Launcher instead of the gray question mark (this askubuntu question has a good visual image of the issue). If instead you mean that there is a working vscode Docker image that uses X11 (or something similar) which does not show the gray question mark in the Unity launcher please provide the link as that would resolve this whole thing for me.
It doesn't seem likely that the Docker image is to blame. I have run other images using Docker with X11 such as Firefox and Android Studio which have had no issue with their icons. This leads me to believe that my machine and its setup aren't likely to blame. Also when I follow the same X11 setup in a Docker file with vscode built from source and run using electron . the issue goes away. So this leads me to believe that Docker and its X11 interactions aren't likely the cause. After all of this it seems to me that there must be something to do with how the executable versions of vscode are made since all other variables being seemingly the same changing between a downloaded executable version vs a built from source version will either fix or create the issue.

Not quite sure I understand what you mean with VSCODE_CWD? VS Code'ss cwd should be the cwd of the process that launched it by default.

VSCODE_CWD is an environment variable that can be used to set the directory that vscode will start with when it is opened. See main.js for details.
When running from a pre-built version of vscode that is provided by Microsoft the first argument to the executable specifies the working directory that vscode will open to. When building from source and running directly with electron (electron . in the main directory of the source code) the working directory that vscode will open to is the current directory that electron is running in. In this case you will get vscode opening to the directory where the vscode source code is located.
The only way that I found not to have this default behavior of electron . opening to the directory where the vscode source code is located was to set VSCODE_CWD to some other directory. Whatever directory VSCODE_CWD is set to will change the opening directory when running with electron .. The only issue is that I don't want to have an initial opening directory when running with electron . and running export VSCODE_CWD="" doesn't accomplish this as I would have expected.
Is there some other alternative to VSCODE_CWD when running with electron . or is it possible to have `export VSCODE_CWD=""' make it so that no initial directory is selected when vscode starts up?
Thank you for all of your time and help :)

I think I have the VSCODE_CWD issue worked out. Apparently there is another environment variable VSCODE_DEV which you have to set to a string that will evaluate to true in an if statement (I did export VSCODE_DEV=1). There is code that will check VSCODE_DEV in parseMainProcessArgv and if it evaluates to true in the if statement will strip the first non-optional argument to electron which stops a folder from being immediately opened since it removes the . from electron ..
This resolves my issue as I am able to get everything working with vscode in Docker on X11 by building from source. It would certainly be much nicer if the official builds worked and there wasn't a need to rebuild vscode just to get this to work, but I do have a working version now so I would be fine stopping things here.


[stable/kiam] Port stable/kiam xlock race bugfix to helm chart

Port https://github.com/uswitch/kiam/pull/180 to stable/kiam
Fixes #9261

@max-lobur could you please add signoff to your commits to make the DCO pass?

@vsliouniaev thanks for taking a look, done.

@linki @scottrigby any of you guys have a minute to look please?

bump

/assign @paulczar

bump

@ewbankkit can you take a look pls?

bump

rebased

😍

/retest

Hi! We've now released v3.0 so I'd suggest updating the container image to support that. There's a few breaking changes when upgrading from 2.x that are highlighted in https://github.com/uswitch/kiam/blob/master/docs/UPGRADING.md but we'd definitely suggest adopting v3.0 for new releases.

I know, I already passed v2->v3 transition with rc1 image.
@pingles you're saying it's an rc1 issue and should not happen in v3 stable?

oops, wrong thread

@pingles I believe container image wont solve wrong mount behavior here.
I will make a separate PR to bump image to stable tho

@max-lobur , @pingles https://github.com/helm/charts/issues/9790 has been opened to make sure the Helm chart has full support for v3.0.

bump

rebased

We are running into this same issue this PR addresses.
@max-lobur can you rebase 1 more time?
@paulczar can you give a lgtm again?

rebased

rebased, bump

getting a 1 line merged has never been harder :)

@max-lobur We should move this to helm-hub. It  would allow the chart to be self-hosted by kiam

kiam maintainers are adding their own helm chart: https://github.com/uswitch/kiam/pull/232

oh a rebellion, cool :D
I'll make sure my change is there then

@max-lobur rebellion is going slow so reopening this PR makes sense 😄

@danmx done

@paulczar could you take a look at it again?

rebased

rebased

@paulczar / @unguiculus / @lachie83 could one of you give the final approval /lgtm for this PR? Its been open since November and is a very minor change

rebase

@maorfr could you help us push this through?

/lgtm
I would consider helm hub, as previously mentioned here.

/retest

/lgtm

Thanks @maorfr !

🎉


Create Kubernetes metadata files

As a developer
I need create Kubernetes metadata files
So that Kubernetes will know how to deploy the application
Details and Assumptions

Need to create the necessary Kubernetes metadata files for deployment: deployment.yaml, service.yaml, secrets.yaml, postgres.yaml, pod.yaml (slide 57)

Acceptance Criteria
Given an application
When ready for deployment
Then Kubernetes deploys application given provided metadata files.

pending secrets.yaml


Modify downloaded theme

When I export my theme is there a way to copy and paste product codes, css files, basically the whole content, when I export I just get the six folders. Templates, assets, snippets... Where can I just and past my modifications to, like the index file, product files, and such

Also how do I get it all back together so all files are in the correct folder


how to bind events in a new button?

I want to add a button in web/viewer.html and js files, I study the source code but doesn't success, how did pdfjs bind events worked ?

I tried follow download button's logic, but failed.

Unfortunately you've not shown what you actually tried, and thus more specifically what didn't work.
Please see https://github.com/mozilla/pdf.js/blob/master/.github/CONTRIBUTING.md (emphasis mine):

If you are developing a custom solution, first check the examples at https://github.com/mozilla/pdf.js#learning and search existing issues. If this does not help, please prepare a short well-documented example that demonstrates the problem and make it accessible online on your website, JS Bin, GitHub, etc. before opening a new issue or contacting us in the Matrix room -- keep in mind that just code snippets won't help us troubleshoot the problem.

Please keep in mind that this is first and foremost an open-source bug tracker, rather than a support forum. Hence all issues are required to contain enough details to be easily actionable in order to be valid.

thanks for your advice, its every funny, the button has disabled attributes that I don't realized. hahaha, my mistake.