Migrate to multiple Azure Subscriptions

I have multiple applications on my LUIS account. I'm the owner of some of them and for the others  I'm only a contributor.
Moreover, not all applications are tied to the same customer. I need a way to select to which subscription/resource group/resource an application is migrated.

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

ID: 6e7646c2-31b4-c0a5-41ae-c258d9957cf8
Version Independent ID: df71ee17-a660-a6de-56f5-697276b0dfe1
Content: Migrate to Azure resource for authoring - Azure Cognitive Services
Content Source: articles/cognitive-services/LUIS/luis-migration-authoring.md
Service: cognitive-services
Sub-service: language-understanding
GitHub Login: @diberry
Microsoft Alias: diberry

@sachabruttin Thanks for the question. We are investigating the issue and will update you shortly.

@sachabruttin After the migration process, all your LUIS apps are now assigned to a single LUIS authoring resource.You can create more authoring resources and assign from the Manage -> Azure resources page in the LUIS portal.
Please follow the below document how to assign a resource to an app.
https://docs.microsoft.com/en-us/azure/cognitive-services/luis/luis-how-to-azure-subscription#assign-a-resource-to-an-app

All applications are migrate to the same resource, you can chnage the application assignement under manage/azure resources/authroing

@sachabruttin will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.

@ram-msft so the process would be for each owner to:

Migrate all their owned apps to a single resource in Azure

I guess each owner could create a separate one?
does the azure resource have to be in the same directory as the owner?

Once migrated each app can be switched to a different resource?

Can those resource be in a different directory?
If not, can those resources then be moved into a different subscription, then that subscription be moved into a different directory?

@ram-msft I would appreciate it if someone could look at my comment :)

Yes, each owner can create a different resource with max of 10 per subscription per region
Yes, can be switch to a different resource in the same region

The key But in a different directory?

can you please elaborate on what do you mean by directory ?

An Azure AD Directory or tenant.

This means that the applications before migration were tied to users with different identities under different tenants, correct ?

Yes

For anyone playing along at home, roughly here's what I did that worked:

Move/create an Azure subscription into the test tenant ("switch directory").

I used an MSDN Azure Subscription. In my company I wouldn't normally have access to do this on a 'company owned' subscription.
I invited my main tenant user as a guest into the test tenant to perform the transfer.
I had global administrator in the test tenant. Could probably do it without this but it made it a lot easier. Especially the ability to turn on manage all Azure subscriptions

granted the user(s) in the test tenant that own the LUIS apps access to that newly transferred subscription
performed the migration from the LUIS portal(s)
performed the "switch directoy" operation to transferred the subscription back to my main tenant

One back in the main tenant I could creating new azure resources in different subscriptions, transferring applications, reassign permission and update keys in applications.
What a process! Took several hours of clicking around and relied upon having admin rights to both a azure subscription and the test tenant.


Teams custom bot app is not asking the user concent everytime

I didn't give my consent to my custom app, and when I try to login again it's not asking the consent. What might be the issue?
Sometimes for other users, it is asking sometimes it's not asking.

@samthrusha123 - Thanks for reporting your issue.
We will investigate this issue and get back to you.

When the user installs any teams bot, for SSO authentication it'll ask for user consent right, at that time if the user refuses to give consent it should ask again when user tries to have a conversation with the bot.
Repro Steps:

Create a simple teams bot with authentication, and when we installs it click on cancel when it asks for user consent.
And send 'Hi' again it should ask user consent again but it's not asking.

@samthrusha123 - Thanks for sharing the steps.
We will try it from our end and will let you know the updates.

@samthrusha123 - Sorry for the delay. Engineering team working on this issue. Currently we don’t have ETA or Update on this thread.

@samthrusha123 - Issue is Resolved. Could you please check and confirm once?
Microsoft Teams Version <IP_ADDRESS>78 (64-bit)
https://user-images.githubusercontent.com/93527375/230036081-6d6af43b-d527-40af-99d9-1cb24cbfb438.mp4

Do we need to do any code changes?


Xray Dependency on part database

I ran into an interesting issue where I deleted my fasm2bels and then ran make install_fasm2bels to reinstall it.  At the end of the install steps it runs a flow to regenerate the part database; however, since I had run the flow previously, I got ninja: no work to do.
The XRay tool should probably be updated to add the part database as a dependency so that ninja throws an error if the file does not exist.

@jgoeders To guarantee that you don't get ninja: no work to do., could we just add
mv build build_temp
python scripts/run.py VivadoBitToNetlist designs/basic/and3 --no_tool_checks
rm -r build
mv build_temp build

to the install_fasm2bels.sh to guarantee that the flow runs and the database is generated?


feat: add collapsible fields

Description

Refactor TES component to improve usability and navigation by implementing collapsible fields in the form.
Checklist

[ ] My code follows the contributing guidelines of this project.
[ ] I am aware that all my commits will be squashed into a single commit, using the PR title as the commit message.
[ ] I have performed a self-review of my own code.
[ ] I have commented my code in hard-to-understand areas.
[ ] I have updated the user-facing documentation to describe any new or changed behavior.
[ ] My changes generate no new warnings.
[ ] I have added tests that prove my fix is effective or that my feature works.
[ ] I have not reduced the existing code coverage.

Comments

@uniqueg  Please take a look and comment.

@git-anurag-hub @SalihuDickson please review so I can merge :+1:


ESP32 SEEK command not returning a result code.

After issuing the command $14 and the 4-byte file position, my code is waiting for a return code as specified in the documentation, no return code is ever received. The file position is being changed as expected.

Can you please compare your code with https://github.com/fvdhoef/aquarius-plus/blob/master/SDK/asm/projects/cpm/bios.inc#L705 ?
CP/M is using SEEK extensively and it is working fine there.
The command is $14, then the file descriptor, then the 4 byte file position.
Maybe you forget to send the file descriptor?

I looked at the code again and it is getting a result.


purifycss-extended, a way to make Tailwind lighter?

Hey @adamwathan,
As I said few days ago, I decided to fix some bugs on purifycss/purifycss.
I forked the repo and made some fixes, now it's called purifycss-extended.
Here's the result before and after (from v1.3.2):

Purification level is very outrageous and can be better.
But it means to rewrite the package from scratch or at least a lot of investment because purifycss read each input file and if you have a lot of text, each word will be added to the list of comparison. This list will be compared to CSS rules. It means that if you have Hello World! on your page, it will check for a hello and world class, and ever if you don't have these classes in your HTML/PHP/... files but they are present in your CSS, they'll stay here.
It's a good start and I'll see if I rewrite this code to be more accurate :)
Hope you'll use this package with Tailwind and maybe link it to the documentation!

Hey thanks for linking this @HapLifeMan! I'm going to close this as an actual issue because I don't think we would ever want to bake this directly into Tailwind, but I'm looking forward to testing this out when I have a chance.
Maybe open an issue over at https://github.com/tailwindcss/discuss if you'd like to have a discussion about using this package with anyone else who wants to give it a try 👍


Create class for occupation

Definition proposed by @zhengj2007 in issue #62
occupation: an activity in which one engages An information content entity that is about something people are occupied with and you're not necessarily doing it to exchange for payment. (parent: IAO_0000030 information content entity)

proposed label = occupation information content entity
proposed definition = A data item that is about an activity that realizes a job role and is often performed for payment.
That 'job role' class referred to in the definition is the one we defined in Issue #74 .

Per the August 2021 OMRSE meeting, we will be creating a class to represent 'occupation' or the activities that one carries out as part of their job role in realization of that role. As such, I would like to propose the following class label and definition:
proposed label = occupational activity
proposed definition = A process that is carried out by the bearer of a job role in realization of that job role in the form of labor or services in exchange for payment and as part of some employment process.
A couple of things to note here:

The OMRSE class 'employment process' is defined as "A planned process in which the bearer of an employer role provides wages or salary to the bearer of an employee roll in fulfillment of the arrangements specified in some declaration."

'employment process' is thus different from 'occupational activity' in that it represents the full, often long-term process of employing someone. Unlike 'occupational activity', it realizes some employer role and presumably (no annotations on the current class actually articulate this) begins with the creation of some job role and ends with the termination of that job role(s). It also involves periodically paying or providing benefits to the bearer of the job role.

Possible changes for other classes
Because 'job role' is currently defined as "A role in human social processes that, if realized, is realized when the bearer provides labor or services in exchange for a wage or salary as specified by some deontic declaration," if we go with this definition for 'occupational activity', we may want to simplify the definition of 'job role' to "A role in human social processes that, if realized, is realized by some occupational activity."
I would also like to propose the idea that an employment process does not realize a job role; rather, it realizes some employer role and has part some occupational activities that realize that job role. My rationale for this is that an employment process is not something that exists intermittently, but instead is an ongoing process that begins with the creation of a job role and ends with the termination of that job role. The current axiomatization, therefore, implies that a job role is constantly being realized throughout the duration of their employment, including when they aren't at work. Fixing this would only require a small tweak to the axiom that currently specifies that an employment process realizes a job role.

The key distinction to draw is that you can have one occupation across multiple, distinct employments with different employers.  So Mr. Rodriguez's occupation as an architect is distinct from the five jobs he had over his career with five different architectural firms.  Is his occupation really a distinct entity at all?  When he is "architecting", yes "architecting" is an occupational activity, although I think the restrictions to payment and employment, jobs, etc. might be too severe.  What about freelance artists, journalists, etc.  Shepherds?  But occupation as some kind of role seems like a bad idea.

My only hesitation is with whether it's accurate to say that a person who is currently unemployed (or is not self-employed or doing freelance work) has an occupation.

What do you mean by “having an occupation”? It hinges on that.  What is an
occupation?  What is ones relationship to one’s instance of occupation?
Perhaps what is meant by “occupation” is merely the type of process(es)
that realize your various particular job roles.
Also what if I’m just between jobs?  Technically unemployed but have a
signed offer with my next employer doing the same occupation as my old one,
just a couple weeks in between?
On Tue, Aug 31, 2021 at 4:54 PM Matthew Diller @.***>
wrote:

My only hesitation is with whether it's accurate to say that a person who
is currently unemployed (or is not self-employed or doing freelance work)
has an occupation.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/ufbmi/OMRSE/issues/73#issuecomment-909628782, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAJR55XBYFNRHTNB7JA2AJTT7U6QTANCNFSM4EE7EXPQ
.

The more I think about it, the more I'm leaning toward 'occupation' being defined by the types of processes that realize one's job role and perhaps also by certain broader objectives specifications set for those processes. My reasoning for including the latter is that, practically speaking, every occupation will have processes that realize a job role despite not being part of the "job description" so to speak.
For example, a police officer is expected to patrol whatever neighborhood they are assigned to with the broader goal of this being to maintain the safety of the people there. This occupational activity would therefore obviously realize their job role since it's an expectation laid out by their employment contract. However, let's say that while patrolling the neighborhood they come across a woman who is giving birth on the side of the road. It's unlikely that "assisting women with giving birth" is a process that is explicitly spelled out in their contract as being required for them to perform. But because assisting this woman helps to maintain the safety of her and her child, I think we can say that it also realizes his police officer job role.
The point you bring up about being in between jobs is interesting. I'll have to think about it more.

I think the process view, regardless of how it turns out, is extremely
insightful.  The root meaning of "occupation" is literally about what
occupies you, what occupies your time, your life.  It's about "what you do".
I think the currently unemployed but seeking work situation perhaps could
be handled by (1) "currently has a role realized by activity X" OR (2) "has
been the bearer of a role realized by activity X in the past and intends to
be once again in the future the bearer of a role realized by activity X".
I think your occupation "leaves you" when you stop looking for
opportunities in a given occupation, regardless of reason (changing
occupation, retiring, unemployed and give up looking).
The "has been the bearer in the past" part prevents me from saying my
occupation is "metal lead guitar" when I have absolutely no hope LOL.  It
can't be your occupation until you've done it.
On Tue, Aug 31, 2021 at 6:50 PM Matthew Diller @.***>
wrote:

The more I think about it, the more I'm leaning toward 'occupation' being
defined by the types of processes that realize one's job role and perhaps
also by certain broader objectives specifications set for those processes.
My reasoning for including the latter is that, practically speaking, every
occupation will have processes that realize a job role despite not being
part of the "job description" so to speak.
For example, a police officer is expected to patrol whatever neighborhood
they are assigned to with the broader goal of this being to maintain the
safety of the people there. This occupational activity would therefore
obviously realize their job role since it's an expectation laid out by
their employment contract. However, let's say that while patrolling the
neighborhood they come across a woman who is giving birth on the side of
the road. It's unlikely that "assisting women with giving birth" is a
process that is explicitly spelled out in their contract as being required
for them to perform. But because assisting this woman helps to maintain the
safety of her and her child, I think we can say that it also realizes his
police officer job role.
The point you bring up about being in between jobs is interesting. I'll
have to think about it more.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/ufbmi/OMRSE/issues/73#issuecomment-909705132, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAJR55TE2TJ7NES5ARF37ETT7VMEHANCNFSM4EE7EXPQ
.

Sorry, I'm seeing now that Matt is already onto something similar in his most recent comment.

VEuPathDB database used the term with values:
Agricultural laborer
Salaried service
Daily wage/labor
Cultivator
Service (=salaried)
Trade (=self-employed)
Housewife
Student
Cultivator (independent farmer)
Self-employed/trade
Child, not schooling
Note: It may not be the right term to use to group these values.

First stage of harmonization effort with OccO is complete and thus resolves this issue.


pull request for e2e test at 01/01/2017 17:26:05

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Not compatible with wcout ?

I'd like to use this lib with wcout to dispaly utf16 characters, but when writing
 wcout << dye::green(L"hello");
I get the error:
error: no match for 'operator<<' (operand types are 'std::wostream' {aka 'std::basic_ostream<wchar_t>'} and 'dye::R<const wchar_t*>' {aka 'dye::colorful<const wchar_t*>'})
     wcout << dye::green(L"hello");
     ~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

Any quick solution for this issue?

This library only supports ostream, wostream is not supported currently.


onClick not worked in react app inside webview. it's something to do with #1175

when device is selected to Chrome WebView option, onclick event didn`t worked.
But, when choose Android system webview, it worked.

Im trying to acces to touchend event  when a select a text inside the webview but nothing happens.


More reliable redis connection pool

The current one isn't reliable enough and can suddenly spam with thouands of connections, this shouldn't be possible, and causes more problems down the road.

@jonas747 Close this?

nop


Concurrency setting in pipeline definition

As per this page "schedule trigger" doesn't support concurrency but concurrency level can be set at the pipeline definition level. Is there any difference in the behavior if concurrency levels are set in different properties?
Reference - https://feedback.azure.com/forums/270578-data-factory/suggestions/37560658-schedule-trigger-add-concurrency-flag-to-prevent

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

ID: 3d564675-f719-57fc-a406-6aec51d26b29
Version Independent ID: 84124e8f-efad-274f-c2eb-40a04748296c
Content: Pipeline execution and triggers in Azure Data Factory - Azure Data Factory
Content Source: articles/data-factory/concepts-pipeline-execution-triggers.md
Service: data-factory
GitHub Login: @djpmsft
Microsoft Alias: daperlov

Thank you for your detailed feedback.  We are looking into the issue, and we will respond when we have more information.

@rajatbagchi As mentioned in the uservoice you can set the concurrency at pipeline level.  We do not have any other place to set the concurrency for a scheduled trigger, however, you can set max concurrency level for a tumbling window trigger.

@rajatbagchi We will now proceed to close this thread. If there are further questions regarding this matter, please reopen it and we will gladly continue the discussion.


WIP: net/wol: 2.0 release

please set PLUGIN_REVISION=1 for incoming fixes. no time anymore today, but this can go in as a first step if you want


Adding gcd code and cube skeleton

Make sure these boxes are checked before submitting this pull request~!~ Thank you!!

[] The test of the method you have fixed is passing
[] You have written a new skeleton method for someone else to work on!

Thanks!


Master

merged tinymce4 into master, I do not know the historical background of some changes so I might have been a bit hasty with going with one solution or the other.

Released as django-tinymce4: https://pypi.python.org/pypi/django-tinymce4/


fix absolute path issue

Thank you so much for the library
the problem is when I use example it'll redirect to http://example.com/http://example.com/example my commit fixes this issue. it'll redirect to http://example.com/example

@EslamHiko I think you should not add data-navigo attribute to your link so it to work as you are expecting.
Or I miss something?

@slyapustin Wow man that's very old PR and it's my First one 😄 ... In my case I have a php Laravel web application and most of the links are absolute because they are generated in the back end. I want to use the library on all the links. so what I want is when the link in the href attribute has the base url I want it to trigger the Ajax request for content container I don't want to refresh the page.
I mean it doesn't make sense if I have an href with the "http://base.com/target" and go to "http://base.com/target/http://base.com/target".

@EslamHiko @slyapustin this is now supported by Navigo in its new version 8.0.0. The solution that I introduced is using the URL api of the browser.
@EslamHiko thank you for the PR.
P.S.
Version 8.0.0 is not officially released but can be installed via npm install navigo@beta or yarn add navigo@beta. Migration guide is available here https://github.com/krasimir/navigo/blob/big-rewrite/CHANGELOG.md#migration-guide and the new documentation here https://github.com/krasimir/navigo/blob/big-rewrite/DOCUMENTATION.md.


モデル移行

resolve: #17

build: intervention/image
feat: migrations
feat: model
feat: cappuccino guest_type config
test: term factory
test: ActivityLog factory
test: exhibition factory
test: guest factory
test: reservation factory
fix: apply cappuccino config
test: generate correctly guestId when factory create
test: fix activityLog factory
fix: remove guest_id column
chore: edit order of migration
refactor: separate setting foreign file into each table migration
fix: exhibition model
test: fill exhibition name as same as user name
test: rename attribute name "exh_id" to "exhibition_id"
test: set Term Id unique
test: set relation to guest
feat: resources
feat: user and exhibition relation
fix: use models count function
fix: relation between user and exhibition
style: fix over indent
refactor: rename to ExhibitionResource
fix: dont count already exited guest

exhibition->userのforeignは既にあると思います

Timestampのデフォ値って具体的に何を指してますか useCurrentはCurrentであるべきものが入ってるはずでそうなってないものはnullableになってます


Sync server scalability for live collaboration

I've been running benchmarks for rich-text editors with live collaboration. Using automerge with a modified automerge-repo-sync-server, I noticed performance issues starting at ~4 simultaneous users.
In each benchmark, the users connect to the server using BrowserWebSocketClientAdapter, wait for the server to share the document, and then start typing at 6 chars/sec (plus occasional formatting ops). They also record "remote latency": the time from when one user types some text until it shows up for the other users.
With 4 users, after about 90 sec of (continuous) typing, the server's CPU usage hits 100% and the remote latencies spike to ~20 seconds. With 8 users, the spike occurs within 30 seconds. I believe it depends on (# users) * (size of doc). Full time-series data: allActive-4-automergeRepo.csv, allActive-8-automergeRepo.csv
Variations:

If I use a basic WebSocket echo server instead of automerge-repo-sync-server, it scales to at least 16 simultaneous users.
When I ran similar benchmarks in August (v1.0.2), even 3 users would see a spike in remote latency, but now 3 users seem okay. So there has been recent improvement.

Reproduce
More details about the benchmarks are in Section 7.1 of this preprint. Server code we're using is here, client code is here.
You can easily run the benchmarks locally, although the data above is from running it on AWS (with clients + server in different regions).

Clone https://github.com/composablesys/collabs-rich-text-benchmarks
In server/package.json, update @automerge/... dependencies to the latest versions. (The existing versions are what's used in the preprint.)
Install and build:

npm ci && npm run build

Run a local experiment (7 minutes):

cd client
bash local_exp.sh ../data allActive 4 automergeRepo trial0

"4" is the number of users; replace "automergeRepo" with "automerge" to use the basic WebSocket echo server.
5. Analyze the data:
cd ../analysis
npm start ../results ../data/allActive-004-automergeRepo/

CSVs will be in results/. There are also client CPU profiles but not server CPU profiles (sorry).
Versions:

"@automerge/automerge": "2.1.5"
"@automerge/automerge-repo": "1.0.12"
"@automerge/automerge-repo-network-websocket": "1.0.12"
Node v18.17.1
Ubuntu 22.04.3

Thanks for the report, @mweidner037, this is really helpful. Your results seem vaguely plausible given the performance traces I've seen lately in the browser. (As an aside the Chrome Dev Tools have really excellent performance analysis tools.)
We've been looking at similar problems and have a few patches in the work but getting independent performance testing is always very welcome!

I've been looking into this - There are two issues related to marks causing you to call Automerge.marks() when you shouldn't need to.  I will have a PR soon that fixes this.  As for the automerge-repo falling down - I saw the same behavior but just by updating to a more recent version this seemed to go away.  I was hoping you could confirm
     "server": {
       "dependencies": {
-        "@automerge/automerge": "2.1.2-alpha.0",
-        "@automerge/automerge-repo": "1.0.2",
-        "@automerge/automerge-repo-network-websocket": "1.0.2",
+        "@automerge/automerge": "2.1.5",
+        "@automerge/automerge-repo": "1.0.12",
+        "@automerge/automerge-repo-network-websocket": "1.0.12",
         "@collabs/collabs": "0.13.4",

Also I made an even more lightweight version of the automerge-server.js and haven't seen it go over 1% CPU usage.  Not very useful as it's just an echo server but happy to share if you like.

Also I made an even more lightweight version of the automerge-server.js and haven't seen it go over 1% CPU usage. Not very useful as it's just an echo server but happy to share if you like.

Sure, I can change automerge mode to use that.

As for the automerge-repo falling down - I saw the same behavior but just by updating to a more recent version this seemed to go away.

I'll try this out when I can.

Aside: When you are running locally, you can make it more stressful (as if there were more users) by increasing the rate of edits. Here is the relevant constant: https://github.com/composablesys/collabs-nsdi/blob/master/client/src/scenarios/all_active.ts#L16
Probably 30 edits/second (value 33 ms) with 3-4 users will make things interesting. Remote latency P95 should be the most sensitive indicator.

Made a PR that adds a fast marksAt() method so you dont need to walk the whole text field on every insert
https://github.com/automerge/automerge/pull/785

Something you could add to the benchmark that would be hugely valuable (and I was having trouble doing while reading your benchmarking harness) would be to have a server flamegraph added to the stats gathered maybe via the 0x package.

I updated the code to use Automerge.marksAt and also upgraded to the latest versions (automerge 2.1.6, automerge-repo-* 1.0.15). However, I'm seeing similar performance - e.g. with 8 users on AWS, latency spikes around 30 sec and does not recover.
(The updates are in a dev copy of the repo - I can invite you if you like.)

Something you could add to the benchmark that would be hugely valuable (and I was having trouble doing while reading your harness code) would be to have a server flamegraph added to the stats gathered maybe via the 0x package.

You should be able to change "node" to "0x" on this line. I tried it a few times but it was flaky - it would give me a flamegraph if I killed the process after a few seconds, but not after running a whole experiment (even shortened to 30 sec).


List control planes

Half of https://github.com/istio/istio/issues/23892
The output of this command is, for example,
$ istioctl manfiest list
VERSION REVISION PROFILE   CUSTOMIZATIONS                                                      PODS AGE
1.7.2   <master> <default> components.ingressGateways[0].k8s.service.ports[3].name="tls"       1    19m
                           components.ingressGateways[0].k8s.service.ports[3].port=15443
                           components.ingressGateways[0].k8s.service.ports[3].targetPort=15443
                           hub="docker.io/istio"
                           tag="1.7.2"
latest  18x      <default> hub="gcr.io/istio-testing"                                          2    115s
                           installPackagePath="manifests/"
                           tag="latest"
                           values.global.imagePullPolicy="Always"
                           values.global.istiod.enableAnalysis=true

If the control planes were installed by a version of istioctl that is similar to the one running this test, it looks very nice!
This is a draft because I want your input, @ostromart

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

overall i like it a lot. the output could get big for large customizations.
one nit, i'm not sure about the <> around the label - would it be better to leave those out? what is the reason behind it?

@ostromart Can you comment on the proposed output?  Does this seem useful for an administrator to understand their Istio control plane deployments?

/test all

@esnible: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

gencheck_istio
2521ce31edb7b2619e4dfda13973a2d72d7d9225
link
/test gencheck_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@esnible: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

gencheck_istio
2521ce31edb7b2619e4dfda13973a2d72d7d9225
link
/test gencheck_istio

integ-multicluster-k8s-tests_istio
2521ce31edb7b2619e4dfda13973a2d72d7d9225
link
/test integ-multicluster-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@esnible: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

gencheck_istio
2521ce31edb7b2619e4dfda13973a2d72d7d9225
link
/test gencheck_istio

integ-multicluster-k8s-tests_istio
2521ce31edb7b2619e4dfda13973a2d72d7d9225
link
/test integ-multicluster-k8s-tests_istio

integ-pilot-k8s-tests_istio
2521ce31edb7b2619e4dfda13973a2d72d7d9225
link
/test integ-pilot-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@esnible: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

gencheck_istio
2521ce31edb7b2619e4dfda13973a2d72d7d9225
link
/test gencheck_istio

integ-multicluster-k8s-tests_istio
2521ce31edb7b2619e4dfda13973a2d72d7d9225
link
/test integ-multicluster-k8s-tests_istio

integ-pilot-k8s-tests_istio
2521ce31edb7b2619e4dfda13973a2d72d7d9225
link
/test integ-pilot-k8s-tests_istio

release-notes_istio
df3010aadc9426f82a54e1cbdae635681cd9d31a
link
/test release-notes_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@esnible: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-multicluster-k8s-tests_istio
2521ce31edb7b2619e4dfda13973a2d72d7d9225
link
/test integ-multicluster-k8s-tests_istio

integ-pilot-k8s-tests_istio
2521ce31edb7b2619e4dfda13973a2d72d7d9225
link
/test integ-pilot-k8s-tests_istio

release-notes_istio
df3010aadc9426f82a54e1cbdae635681cd9d31a
link
/test release-notes_istio

gencheck_istio
df3010aadc9426f82a54e1cbdae635681cd9d31a
link
/test gencheck_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@esnible: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-multicluster-k8s-tests_istio
2521ce31edb7b2619e4dfda13973a2d72d7d9225
link
/test integ-multicluster-k8s-tests_istio

integ-pilot-k8s-tests_istio
2521ce31edb7b2619e4dfda13973a2d72d7d9225
link
/test integ-pilot-k8s-tests_istio

release-notes_istio
df3010aadc9426f82a54e1cbdae635681cd9d31a
link
/test release-notes_istio

gencheck_istio
df3010aadc9426f82a54e1cbdae635681cd9d31a
link
/test gencheck_istio

unit-tests_istio
df3010aadc9426f82a54e1cbdae635681cd9d31a
link
/test unit-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@esnible: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
df3010aadc9426f82a54e1cbdae635681cd9d31a
link
/test release-notes_istio

gencheck_istio
df3010aadc9426f82a54e1cbdae635681cd9d31a
link
/test gencheck_istio

unit-tests_istio
df3010aadc9426f82a54e1cbdae635681cd9d31a
link
/test unit-tests_istio

integ-pilot-k8s-tests_istio
df3010aadc9426f82a54e1cbdae635681cd9d31a
link
/test integ-pilot-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@esnible: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

gencheck_istio
df3010aadc9426f82a54e1cbdae635681cd9d31a
link
/test gencheck_istio

unit-tests_istio
df3010aadc9426f82a54e1cbdae635681cd9d31a
link
/test unit-tests_istio

integ-pilot-k8s-tests_istio
df3010aadc9426f82a54e1cbdae635681cd9d31a
link
/test integ-pilot-k8s-tests_istio

release-notes_istio
7d516833b8f9af629b160bf7682a08f8af24cafd
link
/test release-notes_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@esnible: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
7d516833b8f9af629b160bf7682a08f8af24cafd
link
/test release-notes_istio

integ-pilot-k8s-tests_istio
7d516833b8f9af629b160bf7682a08f8af24cafd
link
/test integ-pilot-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@esnible: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
7d516833b8f9af629b160bf7682a08f8af24cafd
link
/test release-notes_istio

integ-pilot-k8s-tests_istio
7d516833b8f9af629b160bf7682a08f8af24cafd
link
/test integ-pilot-k8s-tests_istio

integ-telemetry-k8s-tests_istio
7d516833b8f9af629b160bf7682a08f8af24cafd
link
/test integ-telemetry-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@esnible: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
7d516833b8f9af629b160bf7682a08f8af24cafd
link
/test release-notes_istio

integ-pilot-k8s-tests_istio
7d516833b8f9af629b160bf7682a08f8af24cafd
link
/test integ-pilot-k8s-tests_istio

integ-telemetry-k8s-tests_istio
7d516833b8f9af629b160bf7682a08f8af24cafd
link
/test integ-telemetry-k8s-tests_istio

integ-helm-tests_istio
7d516833b8f9af629b160bf7682a08f8af24cafd
link
/test integ-helm-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@esnible: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-pilot-k8s-tests_istio
7d516833b8f9af629b160bf7682a08f8af24cafd
link
/test integ-pilot-k8s-tests_istio

integ-telemetry-k8s-tests_istio
7d516833b8f9af629b160bf7682a08f8af24cafd
link
/test integ-telemetry-k8s-tests_istio

integ-helm-tests_istio
7d516833b8f9af629b160bf7682a08f8af24cafd
link
/test integ-helm-tests_istio

release-notes_istio
bcef2dcf506cffe816c6e1713c4e50e623897bdf
link
/test release-notes_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

Per offline chat this is good as a start but I'd like to add some info to support upgrades:

list of gateways and revision they are pointing to
per revision, per namespace list of workloads with proxies
I also propose a --verbose flag: setting this would list all workload names, and perhaps customizations, since these two items can lead to very large outputs.
These items can be a followup PR, this looks like a great start as is.

I think output gets ugly when there are lots of customizations deep in YAML. Better way would be to show components that are enabled, gateways installed and give some visual indication that the component is customized along with some common settings (like HUB and TAG). Then the user could drill down per revision to figure out the details. Something like this
$ istioctl experimental istiod list
VERSION REVISION PROFILE   COMPONENTS                                                      PODS AGE
1.7.2   <master> <default> istio-ingressgateway(*)                                          1
                           istio-egressgateway(*)
                           istio-eastwestgateway
                           istiod(*)
                           hub="docker.io/istio"
                           tag="1.7.2"


[Bug] query result's column name becomes value

Search before asking

[X] I had searched in the issues and found no similar issues.

Version
branch: master
latest git version: 715af6eea9f18f0aac690be4dfa5f6330a153ede
What's Wrong?
Column name is missing and turns into value of it:

What You Expected?
display normal column name like MySQL:
mysql> select now();
+-----------------------+
| now()                 |
+-----------------------+
| 2024-07-17 19:52:48   |
+-----------------------+
1 row in set (0.07 sec)

mysql>
mysql>
mysql>
mysql> select current_date();
+--------------+
| current_date() |
+--------------+
| 2024-07-17   |
+--------------+
1 row in set (0.00 sec)

How to Reproduce?
pull latest master code and query like: select now();
Anything Else?
It has been appearing a few days before today issue reported.
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

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

that's because FE constant folding. AFAIK it's not easy or necessary to resolve it.

I think it should be same as MySQL

Yes, MySQL can display the function name in this situation.

which version do u use? cannot reproduce this. order version have this problem and have be fixed

branch: master
latest git version: 715af6e

just like the describe above:
branch: master
latest git version: https://github.com/apache/doris/commit/715af6eea9f18f0aac690be4dfa5f6330a153ede

update:
After git version 912a23887fcae26c34ff6d03ad76bd221e44c5c8 on master, this problem was solved.


error gets thrown immediately

Layer ERROR [10:33:53 AM]: This query may need to quote its value
_fetchData in messages-query is called first with an empty predicate.
Then its called again with the correct conversation id predicate. the first time causes the it to throw an error.
Im calling the component like this following the docs:
var conversationPanel = document.createElement('layer-conversation-panel');
conversationPanel.conversationId = conversation.id;
document.body.appendChild(conversationPanel);

This has been fixed, and is in a PR thats blocked by other projects.  This should not be causing you any issues other than annoying logging which will go away once the blocker lets us merge and ship.


illumos: find OmniOS and OpenIndiana CA bundle locations

Tested on OmniOS (an illumos distribution):
$ head -1 /etc/release
  OmniOS v11 r151038q

$ cargo run --example probe
    Finished dev [unoptimized + debuginfo] target(s) in 0.04s
     Running `target/debug/examples/probe`
cert_dir: Some("/usr/ssl/certs")
cert_file: Some("/etc/ssl/cacert.pem")

And OpenIndiana (another illumos distribution):
$ head -1 /etc/release
             OpenIndiana Hipster 2021.04 (powered by illumos)

$ cargo run --example probe
    Finished dev [unoptimized + debuginfo] target(s) in 0.02s
     Running `target/debug/examples/probe`
cert_dir: Some("/etc/openssl/certs")
cert_file: Some("/etc/certs/ca-certificates.crt")

Assuming this is alright to merge, it would be good to get a 0.1.5 release out as well.  Should I push another commit with a version bump, or?

Thanks very much!


ValueError: API 'QString' has already been set to version 1

Description of your problem
When I launched spyder in terminal with command line: spyder
there war following error
Traceback (most recent call last):
File "/home/xx/anaconda2/bin/spyder", line 3, in
start_app.main()
File "/home/xx/anaconda2/lib/python2.7/site-packages/spyderlib/start_app.py", line 114, in main
from spyderlib import spyder
File "/home/xx/anaconda2/lib/python2.7/site-packages/spyderlib/spyder.py", line 48, in
requirements.check_qt()
File "/home/xx/anaconda2/lib/python2.7/site-packages/spyderlib/requirements.py", line 40, in check_qt
from spyderlib import qt
File "/home/xx/anaconda2/lib/python2.7/site-packages/spyderlib/qt/init.py", line 22, in
sip.setapi('QString', 2)
ValueError: API 'QString' has already been set to version 1
If I lauched from the start menue, no respond (the spyder icon exists)
Please provide any additional information below
I have tried to re-installed spyder. It doesn't work
I have no idea why there exist two qtconsole as follows
qtconsole                 4.3.0
qtconsole                 4.1.1                    py27_0
The anaconda-navigator (1.2.3) doesn't work either.
Traceback (most recent call last):
File "/home/xx/anaconda2/bin/anaconda-navigator", line 6, in
sys.exit(anaconda_navigator.app.main.main())
File "/home/xx/anaconda2/lib/python2.7/site-packages/anaconda_navigator/app/main.py", line 70, in main
window = run_app(splash)
File "/home/xx/anaconda2/lib/python2.7/site-packages/anaconda_navigator/app/main.py", line 38, in run_app
window = MainWindow(splash=splash)
File "/home/xx/anaconda2/lib/python2.7/site-packages/anaconda_navigator/widgets/main_window.py", line 169, in init
self.environments_tab = EnvironmentsTab(parent=self)
File "/home/xx/anaconda2/lib/python2.7/site-packages/anaconda_navigator/widgets/tabs/environments.py", line 120, in init
data_directory=CHANNELS_PATH,
File "/home/liao/anaconda2/lib/python2.7/site-packages/anaconda_navigator/widgets/manager/packages.py", line 266, in init
self.table = TableCondaPackages(self)
File "/home/xx/anaconda2/lib/python2.7/site-packages/anaconda_navigator/widgets/manager/table.py", line 155, in init
self._hheader.setResizeMode(self._hheader.Fixed)
File "/home/xx/anaconda2/lib/python2.7/site-packages/qtpy/_patch/qheaderview.py", line 83, in setResizeMode
raise Exception('setResizeMode is only available in Qt4. Use '
Exception: setResizeMode is only available in Qt4. Use setSectionResizeMode instead.
However, my Qt is 4 not 5.
Versions and main components

Spyder Version: 2.3.8-py27_1  anaconda
Python Version: 2.7
Qt Version: 4.8.7
PyQt Version:     4.11.4                   py27_1
Operating system: openSUSE 13.2 (Harlequin) (x86_64)

It worked in the morning, but not since this afternoon. What I have changed in the anaconda is that I installed package nb_conda.
This problem make me crazy now....
Thanks so much if anyone help

Please update Spyder. You're working with Spyder 2.3, which is not supported anymore.

thanks for your reply. I have tried to upgrade the spyder to 3.1.4 and other dependecies to (automatically)
astroid:           1.4.9-py27_0
chardet:           3.0.4-py27_0
isort:             4.2.15-py27_0
lazy-object-proxy: 1.3.1-py27_0
numpydoc:          0.6.0-py27_0
pylint:            1.6.4-py27_1
qtawesome:         0.4.4-py27_0
spyder:            3.1.4-py27_0
wrapt:             1.10.10-py27_0

qtconsole:         4.1.1-py27_0            --> 4.2.1-py27_0
But the problem is still there. :(

Please run
conda update qtpy

to see if that fixes your problem.

I ran "conda update qtpy" and got
Fetching package metadata .........
Solving package specifications: .
All requested packages already installed.
packages in environment at /home/liao/anaconda2:

qtpy                      1.2.1                    py27_0
but the problem is not fixed :(

I don't know what could be the cause then, sorry.

As I guess, the problem is the API Qt being changed to version 1 by some package. I read the init file in spyder and found that spyder only supports for Qt v2 (right?). Is there anyway to change the Qt to version2? Thanks

@wtliao That's right, spyder needs Qt API v2, so you need to find out which package sets it to v1 before Syder tries to set it to v2.
It may help to update pyqt as well (which is not the same as qtpy!).
If it's possible for you, you can side step the problem by upgrading qt and pyqt to version 5, because that version does not have the old Qt API v1 available anymore.

@jitseniesen Thanks so much. My problem is fixed. I upgrade all qt and pyqt to v5, also in the env.

Great! Thanks for letting us know about it.


Fix user privilege problem in Developping-docker-env

When groups with the same group_name or group_id exist in the docker environment, it causes the user added in a wrong group and  fail to get write permission of /opt directory.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Compile on save vs build

Perhaps my config is just not set up correctly, but I have my tsconfig file set up as follows.
"version": "1.4.1",
    "compilerOptions": {
        "target": "es5",
        "declaration": false,
        "out": "main.js",
        "noImplicitAny": false,
        "removeComments": false,
        "noLib": false,
        "sourceMap": false
    },

    "files": [
        "./main.ts"

    ]

Main.ts has references in it to other classes, e.g "a.ts" with class A.
If I make a change to a.ts, the "save on compile" command triggers, but the resulting js file doesn't seem to have all of the Typescript code such as the __extends function. However, if I run a "build" from the package, then the resulting js is correct.

We don't have good support for out as I don't personally use or recommend using it.
Tracked here : https://github.com/TypeStrong/atom-typescript/issues/206

That's too bad. Just out of curiosity, if you don't use --out, then what's your workflow for using multiple .ts files? If you're using Atom, how do you keep a list of js files current in your HTML markup?

@abogartz

if you don't use --out, then what's your workflow for using multiple .ts files?

Use external modules.

how do you keep a list of js files current in your HTML markup?

Don’t. Use an AMD loader.

@abogartz I've documented my reasons for not doing --out : https://github.com/TypeStrong/atom-typescript/blob/master/docs/out.md
Based on that I'll marking this explicitly as wont-fix, in #206 and also in tsconfig.md.
We can provide an --out story using some external build tools later.


Localstorage _restoreAccounts is causing exponential cache growth

See the following 2 spots:

https://github.com/cosmology-tech/cosmos-kit/blob/main/packages/core/src/manager.ts#L253
https://github.com/cosmology-tech/cosmos-kit/blob/main/packages/core/src/bases/chain-wallet.ts#L134

Basically - only 1 of these places should be getting the whole list OR there needs to be an object instead of array. This logic currently creates more and more records everytime "onMounted" gets called.

Side effect - takes the wallet longer and longer to load

cc @liujun93

Temporary hotfix to dedupe cache:
import type { SimpleAccount } from "@cosmos-kit/core";

const Hotfix = () => {
  useEffect(() => {
    const key = "cosmos-kit@1:core//accounts";
    const accounts = JSON.parse(localStorage.getItem(key) || "[]") as SimpleAccount[];
    if (!accounts.length) return;
    const map = new Map<string, SimpleAccount>();
    for (const account of accounts) {
      map.set(account.address, account);
    }
    localStorage.setItem(key, JSON.stringify(Array.from(map.values())));
  }, []);

  return null;
};

return (
  <ChainProvider>
    {children}
    <Hotfix />
  </ChainProvider>
);

@TrevorJTClarke @grikomsn This problem shall be solved in the latest version. Thanks.
Btw, we still use array instead map in case we allow multiple addresses/accounts for the same chainId in the future. But for now we simply filter out the same chainId before add new account data in.
https://github.com/cosmology-tech/cosmos-kit/blob/e2a6a4c2a374168e3b0d501ed5b5948882ce2168/packages/core/src/bases/chain-wallet.ts#L137-L139

cool beans - thanks @liujun93!!


Add setuptools to host

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

To address lints

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe/meta.yaml) and found it was in an excellent condition.
I do have some suggestions for making it better though...
For recipe/meta.yaml:

ℹ The recipe is not parsable by parser conda-souschef (grayskull). This parser is not currently used by conda-forge, but may be in the future. We are collecting information to see which recipes are compatible with grayskull.

This message was generated by GitHub Actions workflow run https://github.com/conda-forge/conda-forge-webservices/actions/runs/12528347348. Examine the logs at this URL for more detail.


font awesome icons size

font awesomeicon size too big

html code:
  <nb-actions
    size="medium"
    [class.right]="position === 'normal'"
    [class.left]="position === 'inverse'">
    <nb-action *nbIsGranted="['view', 'user']" >
      <nb-user [nbContextMenu]="userMenu" [name]="user?.name" [picture]="user?.picture"></nb-user>
    </nb-action>
    <nb-action class="control-item"
               icon="fas fa-shopping-cart"
               badgeText="99+"
               badgePosition="top right"
               badgeStatus="danger"></nb-action>
    <nb-action class="control-item" disabled icon="nb-notifications"></nb-action>
    <nb-action class="control-item" icon="nb-email"></nb-action>
    <nb-action class="control-item">
      <nb-search type="rotate-layout" (click)="startSearch()"></nb-search>
    </nb-action>
  </nb-actions>

How to keep the size consistent？？？

Hi @yedajiang44!
If there is only one place where you need fa icon, then writing code below in header.scss should be enough.
nb-action .fas {
  font-size: 1.5rem;
}

If you plan to use fa icons across the app, I recommend to change all icons in the header to fa analogue and adjust actions-size-medium, actions-size-small and actions-size-large variables in themes.scss.

This solution doesnt work for me as suggested by @yggg


Set target in .spi.yml, simplify

This fixes the SPM build issues https://github.com/SwiftPackageIndex/SwiftPackageIndex-Server/issues/1633 (once we've updated our builder)
I'll leave this in draft until we've done so.

Thank you, @finestructure!

The builder update is now deployed


Local Installation step for react js.

Deliverables

need Simple and clean local Installation step for Frontend.
Make a change on Readme.md
Make a change in frontend/README.md as well.

@codewithvk  may I work on it?

@codewithvk I'd like to work on this

@Castella1313 Go ahead!

@codewithvk 'm a GSSOC'21 participant. I would like to contribute to this issue. We could use two headers "installation" and "setup environment".


Internal error on a project startup

After empty & fresh project launch, I've got this error:
{"statusCode":500,"code":"ECONNREFUSED","error":"Internal Server Error","message":"request to http://localhost:4000/dist/client/stats.modern.json failed, reason: connect ECONNREFUSED ::1:4000"}

Node.js version: 18.3.0
Tramvai version: 1.110.2
Reproduce step:
1. npx @tramvai/create new-app
2. cd new-app && yarn start

Hello!
Thanks you for the report, catch the same issue on Node.js 18 version.
Will try to release fix soon, but probably only for prerelease version - 2.x.x.

As a quick fix you can modify index.ts file, add this lines:
if (typeof window === 'undefined') {
  const { setDefaultResultOrder } = require('dns');
  setDefaultResultOrder('ipv4first');
}

Fix was released at v2.4.0 prerelease version


print() output not shown in logs for Swift actions

For the following Swift action:
func main(args: [String:Any]) -> [String:Any] {
    print("Hey! Did this get logged?")
    return ["message": "No, it did not."]
}

I would expect to see "Hey! Did this get logged?" somewhere in the activation logs but, alas, I do not.

FYI: this affects both swift2 and swift3 runtimes

Should be fixed after #386 and tested explicitly, for both runtimes.

Yay viva the logs. !


How to change the default values

Hi,
I am interested for the extension. However, the default values usually are not suitable for me. How to change them.
Thanks.

I need this feature too. It would be great if it could remember my last settings.

Me too, please add a configurable setting for users, thank you very much!


Differences in coverage percentage between Python 3.7 and Python 3.8

Describe the bug
The coverage percentage appears to be sensible to the Python version under which coverage run and coverage combine + coverage report are executed. Specifically, the coverage percentage varies depending on the combinations of Python versions those commands are run with (while using the same version of coverage.py).
I have experienced the following patterns:

coverage run
coverage combine + coverage report
coverage percentage

Python 3.7
Python 3.7
100%

Python 3.8
Python 3.8
100%

Python 3.7
Python 3.8
92%

Python 3.8
Python 3.7
100%

To Reproduce

Clone this repository: https://github.com/sanjioh/tox-interpreters.
Run tox -r -e py38-tox314,coverage-report. This should report a 100% coverage.
Run tox -r -e py37-tox314,coverage-report. This should report a 92% coverage.
If basepython for the coverage-report tox env is changed to python3.7, coverage consistently reports 100%, as per the above table.

coverage.py version: 4.5.4
Expected behavior
I would expect to get 100% coverage for the combination of coverage run run with Python 3.7, coverage combine + coverage report run with Python 3.8.
Thanks for your support, please let me know if you need any further information.

This is an unfortunate side effect of changes in details of the trace function between versions of Python.  In 3.7 and earlier, a decorated function definition would invoke the trace function only for the decorator line, not for the def line.   In 3.8 and later, the trace function is called for both the decorator line and the def line.
So your Python 3.7 run collected the data that the decorator lines were run, but not the def line, because that is how 3.7 behaves.  When reporting the coverage on 3.8, coverage.py knows that both the decorator and def lines could have been marked as run, and sees only the decorator line in the collected data, and so marks the def line as not run.
I'm not sure what coverage.py can do to improve this situation.  I guess in theory the "what could have been run" logic could use the Python version noted in the data, not the Python version it's running against, but I haven't thought through how feasible that is.
Can you coordinate to use the same version for both measurement and reporting?

Hi,
thanks for thorough explanation, that makes sense indeed.
My suspects were towards decorated functions, but I couldn't figure out the reasons behind.
Just a little weirdness I've noticed: what's special about the _split() method that makes coverage behave differently (and correctly)? Could this be a starting point for something?

While it would be surely nice for coverage.py to abstract away the details of the underlying tracing logic, I totally understand that's not something trivial to achieve.
For now, I'm going to combine the coverage results across all the Python versions I'm supporting, so the impact of this issue should be minimal.
Thanks again for your help.

I noticed that about _split also.  The default argument for sep causes the trace function to be called for the def line also, which I hadn't realized before.  This gets messy... :(

Just FYI (I ran into the same issue):
I think this problem is bound to become more prevalent now that python 3.8 is starting to become the 'default' python version in places, e.g. CI environments. E.g. I was running coverage over a test matrix of python versions and then combining+reporting with the default (i.e. newest stable) python version, which brings out this decorator problem. Maybe add to FAQ?

@ArturKlauser I'm wondering what kind of thing to put in the FAQ. It could be, "If you measure and report on different versions of Python, you could get confusing results."  Or it could be, "If you measure on 3.7 and report on 3.8, a decorated function will mark the 'def' line as not run."
That is, how specific a FAQ entry are you thinking of?

I was thinking more along the lines of the latter, but a bit more general, like "If you measure on < 3.8 and report on >= 3.8, a decorated function will mark the 'def' line as not run." The < 3.8 is from my experience with reporting on 2.7 and 3.7, but I assume it generally holds for < 3.8 given your description of the cause earlier in this bug report. The >= 3.8 is my assumption that the current 3.8.0 behavior is going to be kept going forward.
I think two good candidate locations for this warning would be the FAQ about "Q: Why do the bodies of functions (or classes) show as executed, but the def lines do not?" or close to it, or the end of the "Things that cause trouble" page.
Thanks for considering it.

I've updated the FAQ in 8ed8cbec7581d942a8aaafd1341fe20fb33aef96

Did the decorator coverage change again in Python 3.11? When measuring branch coverage on both 3.10 and 3.11, everything is fine if I run coverage combine under Python 3.10, but if I combine under 3.11 the branch coverage will be incomplete, saying the def line doesn't jump to the decorator's line.


Add utility method to get enum constant from String value

New utility method StringUtils#fromValue()
Add missing unit tests for StringUtils class

Signed-off-by: Steffan Westcott<EMAIL_ADDRESS>

@clicktravel-steffan Changes look good.  Merge when ready.


Permission denied on Android

I am having a weird problem on Android devices, whenever a user tries to pick an image it says that the permission is denied. Is there any further permission requests necessary for it to work on android?
I've tried checking the documentation but some links don't work. Thanks beforehand and please keep me updated!

After investigating the documentation provided by the FishBun team I detected that Android permissions are indeed required. To avoid this error happening to you add the permission request in the Android Manifest file. Closing this case.


A basic pattern played on an emitter won't have his effect chain applied

I've only tested this on a basic pattern might be affecting other patterns as well.
Using VinylPlay on the same pattern will apply the effects correctly.

This is intended behaviour, but it's bad intended behaviour. I'll need to work around GameMaker's limitations to get this to work.

Effect chain manipulation no longer supported natively.


Feature: Audit API

Audit API is required to monitor pagerduty resources.
Key area of usage is to:

generate compliance reports
Track when users are created/deleted.

Hi @zerolaser i will try and work on this.. in the meantime, pd rest fetch can fetch audit records and give you the results as tables; see the REST API reference for the endpoints, parameters and limitations, but here is an example getting one month (the maximum you are allowed to do in a single query) of user records, and displaying it as a table with some of the more relevant information:
pd rest fetch -e audit/records -P since=$(pd util timestamp 1/1/2022) -P until=$(pd util timestamp 2/1/2022) -P 'root_resource_types[]=users' -t -k execution_time -k 'actors[*].summary' -k action -k root_resource.summary -k 'details.references[*].name'
Can you let me know more specifically what you'd be looking to see so that I can incorporate it into my work as I explore this?

@martindstone Thanks to your pagerduty-cli, Before this I used pdpyras to loop through pagination of results. With your pd-cli I can get all the results at once and then query further based on results. You just made my life simpler.
I currently use your cli for everything except to get this user create date/delete date and for that I'm depending on pdpyras.
These are following that I need to pass actions, since, until and root_resource_types[]
        'cursor': next_cursor,
        'actions[]': ["create", "delete"],
        'since': '2022-08-10',
        'until': '2022-09-10',
        'root_resource_types[]': 'users'
        }

I'm looking for a way from pd-cli to get user create/delete date.
I understand that:

I'll have to loop every month until I find the result.
Pagerduty can only let you go back 1 year only, Beyond that we will have to request them for that data.

hi @zerolaser makes sense... take a look at the following simple Python script and let me know if it could be a good starting point:
#!/usr/bin/env python3

import sys
import json
import subprocess
import datetime
import csv
from dateutil.relativedelta import relativedelta
import argparse

parser = argparse.ArgumentParser('Get users created/deleted')
parser.add_argument('-o', '--output_file', help="File to write CSV results", required=True)
args = parser.parse_args()

def runcmd(cmd):
  r = subprocess.run(cmd, shell=True, capture_output=True)
  return json.loads(r.stdout)

now = datetime.datetime.now()
nowiso = now.isoformat()

start = datetime.datetime(2021, 8, 1, 0, 0, 0)

records = []
while start < now:
  end = start + relativedelta(months=1)
  print(f"Getting audit records from {start.isoformat()} to {end.isoformat()}...", file=sys.stderr)
  r = runcmd(f"pd rest fetch -e audit/records -P since={start.isoformat()} -P until={end.isoformat()}" +
    " -P 'root_resource_types[]=users' -P 'actions[]=create' -P actions[]=delete")
  records.extend(r)
  start = start + relativedelta(months=1)

rows = []
for r in records:
  rows.append({
    'action': r['action'],
    'execution_time': r['execution_time'],
    'name': r['root_resource']['summary'],
    'id': r['root_resource']['id'],
  })

f = open(args.output_file, 'w')
writer = csv.DictWriter(f, rows[0].keys())
writer.writeheader()
writer.writerows(rows)

Thanks...


Parent Child resource type deployment dependency.

Knowing the type of dependency, i.e. a deployment dependency, confirms the dependency was not an resource provider type inheritance dependency.
To Be:
It's important to note that an implicit [deployment] dependency isn't created between a child resource and the parent resource.
was
It's important to note that an implicit dependency isn't created between a child resource and the parent resource.
found at:
https://docs.microsoft.com/en-us/azure/azure-resource-manager/resource-group-define-dependencies#child-resources

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

ID: b9809033-1199-d491-5556-9b72665936e0
Version Independent ID: 17a6b986-dd55-4416-1739-ec765922b3fe
Content: Set deployment order for Azure resources
Content Source: articles/azure-resource-manager/resource-group-define-dependencies.md
Service: azure-resource-manager
GitHub Login: @tfitzmac
Microsoft Alias: tomfitz

@ChrisFg Thank you for your query . We will investigate and update this thread further.

@ChrisFg Thanks for the feedback! We have assigned this issue to the content owner/author to review further and update as appropriate.

@ChrisFg - thanks for the suggestion. I have submitted an update.


Connecting first time with out the app?

I want to avoid the firmware update on my Tapo 110, so I thought I could use this from the beginning. But I can't figure out how to do so,

block the devices from reaching the internet so they dont get a firmware update
check your dhcp (router) for the given ip address to the tapo device in question


https://developer.mozilla.org/en-US/docs/Web/API/Document_Object_Model/Traversing_an_HTML_table_with_JavaScript_and_DOM_Interfaces

Request type

[ ] Please close this issue, I accidentally submitted it without adding any details
[ ] New documentation
[x ] Correction or update

Details

In the "Hiding a column ..." code at the bottom of the page, I think the row and column indexes are reversed, resulting in displaying the first row, not the first column. I flipped "cell is:" + j + i  in the inner for loop as one way to get desired effect. Apologize if I misinterpreted the lesson.

I changed the variable names in the example to row and col to make it clear what is happening. In the original code, j represented the row and i the column.
The example creates a table with two rows and two columns. As it does so, if column is 1, the background is defined as red. Column 2 is defined with display: none, making it invisible.
Run the code and use the page inspector (press Ctrl + Shift + C on Windows or Linux or Cmd + Shift + C on macOS) to look at the contents of the generated table and you will see that there are two rows and two columns.
Does the change make it easier to see what is going on?

The rest of the page could probably use an update as well, so keeping this open for now.


Constant errors in console

The error: "targets is undefined" appears incredibvly often in the sconsole. Also the image for double damadge (the cross) doesn appear the the generic damage buttons (the one with the specific targets appear regularly)

Can you show a screenshot of the issues you are facing?

I get the following errors currently under target-damage.js, I get the first error every time the modules is used (applies damage):
target-damage.js:421 Uncaught TypeError: Cannot read properties of undefined (reading 'options')
[Detected 1 package: pf2e-target-damage]
    at target-damage.js:421:23
    ```

Uncaught TypeError: Cannot read properties of undefined (reading 'length')
[Detected 1 package: pf2e-target-damage]
at target-damage.js:419:12
```
I get the following under foundry.js
Uncaught (in promise) TypeError: Cannot read properties of undefined (reading 'startsWith')
[Detected 1 package: pf2e-target-damage]
    at _parseUuid (foundry.js:4176:13)
    at fromUuidSync (foundry.js:4140:49)
    at get token [as token] (target-damage.js:10:10)
    at target-damage.js:273:29

Here are the errors which happen ecen when it is the only module, also the problem with the X symbol happens when you use the chat portrait module with the target damage module

Make sure you are up to date as this is likely fixed now in 2.1.1.

Ah i see that you just did an update which fixed it, Grat work! ill put the corss thing in the new issue


Modified Add-SPOView to include Paged property

Hi @Laskewitz, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by Microsoft and real humans are currently evaluating your PR.
TTYL, MSBOT;

Thank you Daniel!


Error in executing ansible script

unable to get reason for failing any suggestion to work this playbook. tested on both Macbook and vagrant vm. on both environment getting same error.
ec2_keypair: "pgstaging"
ec2_security_group: "zoo"
ec2_instance_type: "m3.medium"
ec2_image: "ami-561d8241"
ec2_subnet_ids: ['vpc-57b0423','subnet-4feead6']
ec2_region: "us-east-1"
ec2_tag_Name: "ansible-kafka"
ec2_tag_Type: "ansible-kafka"
ec2_tag_Environment: "vpc-stage"
ec2_volume_size: 8
ansible-playbook -vv -i inventory  -e "type=ansible-kafka" provision-ec2.yml
PLAY [localhost] **************************************************************
TASK: [include_vars ec2_vars/{{type}}.yml] ************************************
failed: [<IP_ADDRESS>] => {"failed": true, "file": "/Users/laxmansingh/images-Docker/hadoop-multi-server-ansible/laxman/ec2_vars/ansible-kafka.yml"}
msg: Source file not found.
FATAL: all hosts have already failed -- aborting

@Laxman-SM Did you create the file ansible-kafka.yml inside ec2_vars? The playbook will look for the file in that location by default.

Closing issue. Feel free to reopen if problem still persists.


✨ feat: override == and hashCode (#797)

Checklist

[ ] The title of my PR starts with a Conventional Commit prefix (fix:, feat:, docs: etc).
[ ] I have read the Contributor Guide and followed the process outlined for submitting PRs.
[ ] I have updated/added tests for new/updated/fixed functionality.
[ ] I have updated/added dartdoc comments with ///.

Breaking Change

[ ] Yes, this is a breaking change.
[ ] No, this is not a breaking change.

Related Issues

Codecov Report

Merging #799 (0a186f4) into main (606ea76) will increase coverage by 8.71%.
The diff coverage is n/a.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
@@            Coverage Diff             @@
##             main     #799      +/-   ##
==========================================
+ Coverage   80.36%   89.07%   +8.71%
==========================================
  Files         236        4     -232
  Lines        2923      119    -2804
==========================================
- Hits         2349      106    -2243
+ Misses        574       13     -561

see 232 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


[release/6.x] Make aspnetcore a coherent parent of runtime

Same change in #1660 and #1663 but targeting the release/6.x branch
The aspnetcore and runtime versions are already coherent, so only the attributes need to be applied to the version entries.

/backport to release/6.1


Patch com.snowplowanalytics.snowplow.badrows/size_violation/jsonschema/1-0-0

There are duplicated description fields not reflecting the related field's meaning.

Thanks @oguzhanunlu! Can you point to examples? Otherwise it will be hard to track all of them.

Sure @chuwy , description fields listed below are same and the latter is the duplicate of the former.

#/properties/failure/properties/maximumAllowedSizeBytes/description
#/properties/failure/properties/actualSizeBytes/description


Local compound database search

Basic information

My operating system: Windows 10
My MZmine version: MZmine 3.0.21 Beta

What happened
Local compound database search is one way that we identify metabolites and have done with MZmine 2 for several years.  In the beta version of MZmine 3, performing this search results in no identifications.  I modified the .CSV file according to the updated parameters but still no results are reported out.  Checking m/z and retention time pairs in the .CSV and in the compiled list shows that there is good correspondence.  Wondering if I'm doing something incorrect or if the feature is broken.

hey tjgarret, can you please post a screenshot of the parameters and an example csv file?

Screenshot of settings

For MZMine3 searching.csv
Example file used for searching. Neutral mass checked or unchecked didn't matter

Thanks. I think i found the bug.
If you want you can test this prelimary build here: https://github.com/mzmine/mzmine3/actions/runs/2076649410
Otherwise, the fix will be included in the next release.

Ok, I will give it a try


Adds docker example

Uses vagrant and boot2docker to allow docker-compose to bring up RackHD with ELK.
Dockerfiles:
https://github.com/RackHD/on-core/pull/72
https://github.com/RackHD/on-dhcp-proxy/pull/23
https://github.com/RackHD/on-http/pull/110
https://github.com/RackHD/on-syslog/pull/16
https://github.com/RackHD/on-tftp/pull/28
https://github.com/RackHD/on-taskgraph/pull/53

@mattgam @heckj @benbp @DavidjohnBlodgett

test this please

Updated Dockerfiles to build more effeciently.
Configure isc-dhcp-server from RackHD/docker.
Added Kibana and setup Logstash to listen to RabbitMQ on.logging.
Added missing common files to on-http.

Added convenient script that will checkout my docker branches of RackHD submodules in order to make it easier to run this before it is merged.
Updated README with basic instructions.

Waiting on automated build to test docker hub example...

Added docker images to docker hub here: https://hub.docker.com/u/roland/
Used my account for now. This will allow people to run this docker example without building all the RackHD docker images.

I've added on-statsd which runs Etsy/statsd with a elasticsearch backend.
https://github.com/RackHD/on-statsd/pull/9

test this please

:+1:

:+1:

Please wait to merge this until all the docker files have been merged.

Updated dockerfiles to use Alpine Linux: https://github.com/RackHD/on-core/pull/79
Fixed issues from latest API changes. Now mounting /opt/monorail volume in all RackHD containers.
Added on-statsd to scripts.

New docker images are tiny compared to before. Previously rackhd/on-http was about 1.2GB
REPOSITORY             TAG                 IMAGE ID            CREATED             VIRTUAL SIZE
rackhd/on-taskgraph    latest              cb0050db791c        29 minutes ago      209.8 MB
rackhd/on-tftp         latest              b27a334fb1e2        29 minutes ago      182.1 MB
rackhd/on-syslog       latest              83d55b09d7b8        29 minutes ago      179.6 MB
rackhd/on-statsd       latest              bf0819e29ecc        29 minutes ago      195.3 MB
rackhd/on-http         latest              b01794c9693b        30 minutes ago      515.4 MB
rackhd/on-dhcp-proxy   latest              d9dd3401d685        31 minutes ago      183 MB
rackhd/on-core         latest              adfc59f72b75        31 minutes ago      142 MB
mongo                  latest              b11eedbc330f        8 days ago          317.4 MB
rabbitmq               management          fce3161a1999        8 days ago          305.1 MB
elasticsearch          latest              5f950105f780        10 days ago         345.7 MB
logstash               latest              72e3ea5d6e55        11 days ago         447.5 MB
kibana                 latest              50e2a548d2d9        11 days ago         259.8 MB
mhart/alpine-node      4                   ea3184dbb53a        2 weeks ago         36.41 MB

I also added a nice landing page /opt/monorail/http/index.html

+1

:+1:
@rolandpoulter do you need this to wait on the current outstanding docker PRs or can I merge?

@benbp I think this is safe to merge once I make those changes. Should still work with the old images. The alpine linux docker PRs should be merged more or less at the same time. Or the docker build will fail, which might be okay.

@benbp fixed whitespace, and updated README for running on a Linux host.

@rolandpoulter @benbp is this good to go?


[Question] Speculative Decoding Metrics Variable

❓ General Questions
What is the meaning behind draft_count, accept_count, and spec_draft_length? Thank you in advance!

Hi @bethalianovike, thanks for your question and sorry for the delayed response. Happy to elaborate on the config and metrics here.
spec_draft_length is the number of draft tokens we generate for each request in each round of speculative decoding generation, and it is configurable.
draft_count and accept_count are internal statistics. In our codebase, draft_count[i] means how many times “there are i + 1 tokens generated in the draft,” and accept_count[i] means how many times “there are i + 1 tokens generated in the draft and all these i + 1 tokens are accepted in the spec decode verification phase”. Consequently, draft_count[i] / accept_count[i] means “the history frequency of that generating i + 1 tokens in the draft and all these i + 1 tokens are accepted”.


provide a baked Vagrant box to the same_cfgfile_diff_content VM environment

This solution didn't require VM provisioning.


Specify ns when listing dynakubes during validation (#3469)

Description
cherry pick of #3469
How can this be tested?
same as #3469

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
Attention: Patch coverage is 0% with 1 line in your changes missing coverage. Please review.

Project coverage is 57.28%. Comparing base (e772bf8) to head (283b011).

Files
Patch %
Lines

pkg/api/v1beta2/dynakube/validation/oneagent.go
0.00%
0 Missing and 1 partial :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@             Coverage Diff              @@
##           release-1.3    #3483   +/-   ##
============================================
  Coverage        57.28%   57.28%
============================================
  Files              344      344
  Lines            19794    19794
============================================
  Hits             11339    11339
  Misses            7215     7215
  Partials          1240     1240

Flag
Coverage Δ

unittests
57.28% <0.00%> (ø)

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

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


add coveralls

Changes Unknown when pulling 730ed5a7e7ecb9e9123fcbb2d0180f7ac35f7a51 on ft-unit-testing-116013763 into ** on master**.

Changes Unknown when pulling 7dc580fdcdfcc7a322a3b79960151ddc1f50d11f on ft-unit-testing-116013763 into ** on master**.

Changes Unknown when pulling 6913e13df09d56e25c4111b56d55900d7e8e5a50 on ft-unit-testing-116013763 into ** on master**.


Update the minimum_wp_version to WP 5.8

The minimum version should be three versions behind the latest WP release, so what with 6.1 being out, it should now be 5.8.
Includes adding a little more variation to the example code in the property docblock in the MinimumWPVersionTrait.
Includes updating the list of deprecated functions in the WP.DeprecatedFunctions sniff based on the WP 6.1.1 release. Input for this based on @JDGrimes's WP deprecated code scanner, though that is getting more and more difficult to run as it is no longer maintained and is throwing lots of errors.
Notes:

The sanitize_url() function has been _un_deprecated in WP 5.9. See #2031.
The replacement for the deprecated wp_cache_reset() function has been changed in WP Core.
Ref: https://core.trac.wordpress.org/changeset/52705
The _filter_query_attachment_filenames() function listed the wrong deprecation version in Core. WPCS now contains the correct version.
A patch has been submitted to fix the listing in Core (and has since been committed and backported).
Refs:

https://core.trac.wordpress.org/changeset/54524
https://core.trac.wordpress.org/ticket/56791#comment:10

Other than the above, no verification has been executed on the newly added deprecated function information.

Includes minor tweak to make the generation of the test lines array a little more descriptive.
Previous: #1881
Closes #2031

🆕 WP/DeprecatedFunctions: temporarily exclude one test on PHP 8.2
The logic in the PHP 8.2 Tokenizer needs to be adjusted to allow for PHP 8.2 Disjunctive Normal Types in combination with readonly properties.
While PHP 8.2 still allows for the readonly keyword being used as a function call - see: https://3v4l.org/hL5da -, the PHPCS tokenizer has not been adjusted yet to allow for this in a way which is compatible with DNT.
Also see:

The discussion about this in squizlabs/PHP_CodeSniffer 3667
php/php-src 9512

What is causing the PHP 8.2 test failure?

The call to readonly().

@dingo-d Thanks for catching that - I didn't get notified of the run failure as we still have continue-on-error in place (let's remove that in a next PR).
And yes, @GaryJones is correct, this is about the readonly() test.
I've now added an extra commit to temporarily exclude that test on PHP 8.2+ as this is a issue with the PHPCS tokenizer. Function calls to readonly() are still allowed on PHP 8.2: https://3v4l.org/hL5da (i.e. the readonly keyword exception in PHP which was put in place at my request is still in play in PHP 8.2)

FYI: I've opened issue #2123 as a reminder that we should remove the test exception as soon as possible.


usage instructions don't result in all of the plots shown

Following the usage instructions here: https://pygrb.readthedocs.io/en/latest/user/usage.html
I get output in a products directory, which contains the first plot shown in the usage docs but not the others.
In particular, I don't get this plot:
https://pygrb.readthedocs.io/en/latest/_images/trigger7475chan3.PNG
or the corner plot.
The usage instructions should walk through how those plots are created.

tagging JOSS: https://github.com/openjournals/joss-reviews/issues/2536

The corner plot should have been there? It may have been saved as a pdf.
I updated the corner plot to save as png, I also fixed a bug that was stopping the correlogram and residual probability plot from displaying properly.
I pushed an edit with the changes (to GitHub but not to PyPI)
Let me know if they are still not displaying properly
James

indeed, the corner plot is there as PNG now. And I now see the one above -- I must have missed it somehow previously.
I think it might be helpful to explain what a user should expect to see in the products/ directory.

Awesome :)
There was a bug in that the marker size for the lower plots was too small, I guess it must have been smaller than one pixel for the given plot dimensions. So thank you for bringing that to my attention.
And they were not plotting by default, which I have fixed now.
I updated the docs to more clearly reflect the product files
https://pygrb.readthedocs.io/en/latest/user/usage.html


Add additional normalizing functions

This should include:

Get constellations of a region
Get solar systems in constellation
Get stargates in solar system
Get stations in solar system (although I'm not sure this is made available efficiently yet)
Get planets in solar system
Get moons around a planet

Not going to implement; it's hard to decide when to stop and it depends a lot on how the ESI api evolves.


Use managed pointers for services, modules and the framework

With the newly introduced life-cycle of the framework itself and its associated modules, a Module* could outlive its association with the framework.
A Framework* instance could be shared between different parties.
Service instances currently need to be managed by the registering party. Using shared pointer semantics could ease memory management and keep track of used "handles" from modules within other modules and make ungetService calls superfluous.

The services part has been merged to development from 38-smart-pointers-for-services branch. Jeff is working on merging the part related to Bundle objects.


Update Tools.md

I haven't edited directly but wanted to mention that I struggle with accessing Heroku so it may be prudent, as you mentioned, to choose a different platform in this regard.

I'll make a note.  You shouldn't have to use Heroku - it updates automatically when the repo is updated.  BUT ... the repo it tracks is GitHub.com/CommonAccordCmacc-Org.  NW-NDA is a separate repo, but on my local file system, NW-NDA is also a repo nested in my Cmacc-Org files.  (Ideally, this would be automated via git.include, but Heroku won't expand git.includes when deploying.  So, we make changes in NW-NDA, and I can then include those changes as part of Cmacc-Org, and when I sync to GitHub.com/CommonAccord/Cmacc-Org then it updates at www.commonaccord.org.   We can make a separate repo for a nw-nda.commonaccord.org, though that will add a bit to the management, until someone figures out how to deploy to the web from a git repo with git.include(d) repos.

Have included this notion and a reference to this pull-request in a new version of Tools.md.


Fix bug to make macOS build in observable framework

@geektoni not sure if you need this

Thank you @theartful! I surely needed this! :)
Merging (the python error will be fixed by #4626)!


Module not found: Can't resolve 'material-ui/CircularProgress'

I'm using material-ui v1.0.0-beta.
Your component <Image /> is loading CircularProgress component like this:
import { CircularProgress } from 'material-ui'
the error is here:
var _CircularProgress = require('material-ui/CircularProgress'); in node_modules/material-ui-image/lib/components/Image
However in doc https://material-ui-1dab0.firebaseapp.com/demos/progress/, is like this:
import { CircularProgress } from 'material-ui/Progress';
Anyway, it doesn't work, same error. what am i missing?
Thank

Are you using the @next version of material-ui-image?

Thank you for reporting this! :+1:
The relevant line is https://github.com/TeamWertarbyte/material-ui-image/blob/master/src/components/Image/Image.js#L3, would you mind creating a PR that fixes this issue? :)

Sorry i'm late. I was going to change the code and i saw your email. :+1: . Version 3.0.0-pre.6 works like a charm :)
Thank you!


Feat 452 - Search Sort Order

Adds support & tests for sorting searches (ascending/descending) by document date & document title (or descending score if no other ordering is requested)

sorry about creating a new PR, but the old one was polluted by me trying to add the large test files using LFS before I discovered the ludicrously small free limits. I have instead compressed the large data load file & will add a post-alpha ticket to improve handling of large test datasets/supporting data.


[WIP] New architecture

This PR changes the architecture of the library and decouples logic (parsing, validation, serialization) from POCO classes representing AST nodes such as Link, Collection, Manifest, etc:

it splits parsing into two separate phases: syntax analyses where SyntaxAnalyzer is responsible for parsing raw JSON into AST and SemanticAnalyzer conducts semantic checking of ready AST tree
it adds a separate class for serializing AST into JSON (to be developed yet)

The overall architecture looks great. I'm comfortable merging this in and using it as a basis for further work if that's what you'd like.

@leonardr, would you like to keep the old name python-opds-parser? I was thinking about something more generic like python-rwpm-parser. Do you think it makes sense?

I'd be OK with python-webpub-manifest-parser, since webpub-manifest is the name of the Github project that contains the spec. We'll mention OPDS 2 in the description so this package shows up in searches.

@leonardr, python-webpub-manifest-parser sounds great!

@leonardr, just one more question regarding the naming: would you like to drop python- in the case of the Python package?
So the name of the repository would be python-webpub-manifest-parser and the name of the Python package would be webpub-manifest-parser

Yes, that's exactly what I was thinking. python- helps Github searches but is not necessary for PyPI searches.

@leonardr, I finished, the parser right now should be compliant with RWPM and OPDS 2.0 specs. However, it doesn't allow to extend the specs and add custom sub-collections yet - it won't throw any error but those custom sub-collections won't be available as part of the AST and won't be validated.

You should be able to include feeds if a) the feeds are public in the first place or b) you obscure the metadata -- title, author, identifiers, and links. Show me the feeds privately and I'll advise.

@leonardr, I created a new PR because I messed up this branch and couldn't rebase it anymore.


disable early access feature snapshot storage

via issue https://github.com/hyperledger/besu/issues/4764

Bonsai snapshots certainly have issues. Snap sync itself, the only issue I am aware of is that sometimes it doesn't finish initial sync.
If checkpoint works better for you, I'd say make that change locally and git commit it so it stays.
Ultimately, Besu needs to stabilize its sync.


chore(tests): fix bootstrap network issues when we switched to Mocha

Description
This pull request changes the following:

fix bootstrap network issues when we switched to Mocha

Related Issues

Closes #705

:tada: This PR is included in version 0.31.1 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 0.31.1 :tada:
The release is available on:

npm package (@release-0.31.x dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


add a checkbox to path edit form to allow ignore

default dwell times for associated nodes, and add help to default dwell time

The test failure looks legit

The ignore checkbox does not seem to have any effect...
My nodes have a dwell time of 20 seconds, I put 10, and whether I ignore or not, the stats are the same. If I change the dwell time of the nodes to 10, then the stats do decrease, so it's really the ignore checkbox not working.


Replace tempdir with tempfile

tempdir is deprecated in favor of tempfile. As a bonus vulkano will not depend on winapi crate for linux and macos builds.

Thanks!


fix: fix the problem that the props value appears when the name is 0

fixed https://github.com/alibaba/lowcode-engine/issues/1777

close https://github.com/alibaba/lowcode-engine/issues/1763


Nice to have behavior similar to "patchwork" in R

https://patchwork.data-imaginist.com/

watch patchwork: ggplot2 plot combiner

Related: the plot_grid() function of the cowplot package.
https://www.geeksforgeeks.org/how-to-join-multiple-ggplot2-plots-with-cowplot/?ref=rp


Installation fails in windows with python 3.7.2 in windows env

I am submitting a …

[ x] bug report
[ ] feature request

Description
Installation of this module fails in windows environment with python 3.7.2
When installing the module using "pip install python-mbedtls==1.3.1" in win env, i am getting below error
"WARNING: Retrying (Retry(total=4, connect=None, read=None, redirect=None, status=None)) after connection broken by 'ProxyError('Cannot connect to proxy.', NewConnectionError('<pip._vendor.urllib3.connection.VerifiedHTTPSConnection object at 0x03DCF770>: Failed to establish a new connection: [Errno 11001] getaddrinfo failed'))': /simple/python-mbedtls/
WARNING: Retrying (Retry(total=3, connect=None, read=None, redirect=None, status=None)) "
ERROR: Could not find a version that satisfies the requirement python-mbedtls==1.3.1 (from versions: none)
ERROR: No matching distribution found for python-mbedtls==1.3.1
I think, when i see the installation" section, i came to know that this library supports for linux and macos.
It would be really helpful - if the support is available for win environment,since I could not find a library that supports "PSK" cipher suites PSK-AES128-GCM-SHA256 and PSK-NULL-SHA256  for both TLS and DTLS communication.
I see that this library supports for both TLS and DTLS communication for both the ciphers that is mentioned above.
Current behavior
Getting an error when trying to install python-mbedtls module using pip command.
ERROR: Could not find a version that satisfies the requirement python-mbedtls==1.3.1 (from versions: none)
ERROR: No matching distribution found for python-mbedtls==1.3.1
Expected behavior
Expecting : This library shall be able to install in windows env.
how to build the package or generate the wheel package in case of win env would be really helpful .
Steps to reproduce
1.use "pip install python-mbedtls==1.3.1" in python terminal in windows os.
Minimal demo of the problem
Failed to establish a new connection: [Errno 11001] getaddrinfo failed'))': /simple/python-mbedtls/
WARNING: Retrying (Retry(total=3, connect=None, read=None, redirect=None, status=None)) "
Detailed log is attached for further investigation.
[log_mbedtls.txt]
Other information
Windows 10 64-bit OS
Python 3.7.2
pip 20.1.1
I want to clarify this doubt,however i have only option to bring this notice as an issue.

Hi @thiyaneshece,
Thank you for your feedback. You are correct that Windows is not supported.
However, mbedtls should work under Windows and I do not think that I use any *nix-specific code. With some luck, setting up AppVeyor or another Windows CI service may be sufficient. Would you have any interest in trying it out yourself? Otherwise, I will see what I can do.

Note that I consider this issue a feature request and not a bug.---Not that it makes a huge difference.

Just a short update on this issue. I have created an AppVeyor account and set up the hook to compile and test there. Now, I am also setting up an MS box locally using Vagrant and Ansible to debug a bit faster. Cheers

@Synss - Sorry that i was loaded with other activities, could not respond on time. I would say i'm a beginner in python, also i am not aware of AppVeyor setup.
Thanks for information being updated, It would really help me a lot if i get windows platform support for this library.

Hi! I have not forgotten you but I have had an IRL (small) accident and Windows is out of my confort zone (although I have interest in having this feature). I shall come back to this quickly.

Hi! Issue #36 reported missing files in the archive. That could very well be the cause for the failing installation.

Hi! I still do not have wheel for windows but could you try again with 1.4.1 and report here whether it worked? I have fixed a problem with the archive.

Hi!  You still need to install mbedtls beforehand and set the INCLUDE and LIBPATH variables if mbedtls is not in the expected locations.
I have had success compiling mbedtls from git in the current 2.16 branch with the following call:
MSBuild.exe /NoLogo /MaxCpuCount /p:Configuration=Release /p:Platform=x64 /p:PlatformToolset=v140 /p:WholeProgramOptimization=False .\visualc\VS2010\mbedTLS.sln from the mbedtls root (say, $env:MBEDTLSROOT).
And then, from the python-mbedtls root, you need to set INCLUDE to the include directory from mbedtls (e.g.: $env:MBEDTLSROOT\include) and LIBPATH to something like $env:MBEDTLSROOT\visualc\VS2010\x64\Release and python setup.py install should work.
Note that I run everything under PowerShell and from the "x64 Native Tools Command Prompt for VS 2019". I am not sure whether this is a requirement or not.
This is still a bit involved and I will keep working on making wheels for Windows. That should be easier now.

Hi!  You still need to install mbedtls beforehand and set the INCLUDE and LIBPATH variables if mbedtls is not in the expected locations.
I have had success compiling mbedtls from git in the current 2.16 branch with the following call:
MSBuild.exe /NoLogo /MaxCpuCount /p:Configuration=Release /p:Platform=x64 /p:PlatformToolset=v140 /p:WholeProgramOptimization=False .\visualc\VS2010\mbedTLS.sln from the mbedtls root (say, $env:MBEDTLSROOT).
And then, from the python-mbedtls root, you need to set INCLUDE to the include directory from mbedtls (e.g.: $env:MBEDTLSROOT\include) and LIBPATH to something like $env:MBEDTLSROOT\visualc\VS2010\x64\Release and python setup.py install should work.
Note that I run everything under PowerShell and from the "x64 Native Tools Command Prompt for VS 2019". I am not sure whether this is a requirement or not.
This is still a bit involved and I will keep working on making wheels for Windows. That should be easier now.


Make EVE compatible with 1.2.2

closes #5030

Fails on them depending on eachother, pretty sure this should work and be correct. It's basically a repeat of  #4947 which ran into the exact same issue.

LGTM