Resource IDs and ownership of resources

Reservation can (and will) happen on multiple steps. For example to start a container you probably have to do the following:

Create a storage volume
Allocate network resource
Create container, and assign both the volume, and the network resource to that container.

The problem is, on creating a container you need to pass the the information of the volume (may be the volume full path) and the network resource identity as well. But there is no way to grantee that the caller of the container creation is the rightful owner of the associated resources (volume and/or network)
We think that the IDs of the allocated resources should carry some information about the owner of the resource. so in that case, if the id of the volume and the network matches the owner of the container, the container creation should pass.
Proposal
an id of a resource can be a jwt token with payload that contains both the resource information needed, plus the tenant id. The jwt token is signed with the node private key so the jwt is only valid on the node where the resource is allocated.
Example, you first allocate a volume, the volume id can hold the
{
  "path": "/pool/volume",
  "tenant": "id of the owner"
}

then on container creation, the jwt token is passed as the volume id. the node can then verify that the jwt has valid signature, then matches the tenant id to the tenant id of the container.create caller. Once verified, the volume path can be mounted inside the container.
The only drawback for using this technique is the size of the jwt token.

'drawback' ... a mountpoint that big with godknows what characters in there is a big nono

maybe an md5 hash of that token as mountpoint ? (and then even the 1st 8 bytes ?(or the last?))

@delandtj if it's a JWT, the characters are the base64url encoding set (a-zA-Z0-9_-)

@delandtj you didn't fully understand me

the mount point is NOT the jwt. the JWT holds the mountpoint in the data payload this can be something like /mnt/pools/pool1/subvol1 (the path in the payload above) and the tenant id
They u make a signed jwt with this content. this jwt identifies the subvolume. Now when u want to mount this subvolume in a container. u just pass the jwt as the id of the subvol to mount, then the node will verify the jwt, and then uses the pass encoded in the body.

I stand corrected

The problem raised by @muhamadazmy is indeed something we need to solve.
But his proposal cannot be implement as it is. Reason is, his proposal assumes the ID of a reserved volumes is generated by a node and then somehow sent back to the user that requests it.
This is actually not how things works. A reservation schema is something that is fully generated client side. Then it will be written on the bcdb and the node will eventually get the reservation schema and deploy the requests workloads. And the story ends there. As of today there is no way for a node to communicate with anything else then the bcdb itself. So the generation of a JWT used as ID of the workload is not possible.

I think I have a way to solve our problems.
A reservation object already contains the identity of the creator of the reservation.
If we add a signature to reservation object we obtain 2 things

we make sure the reservation has not been modified on the way to the node
we can verify who is the author of the reservation, aka the owner of the capacity the node will provide.

If we use this signature to create the IDs of all the ressources (volumes, container, ...)
Then we can always map a user to a reservation and to a resource.
Checking if a user can use a certain resource becomes as trivial as keeping a map of the resource ID and the user ID.
This mapping doesn't even have to be kept on the node, this is fully stateless since it everying can be rebuilt from the
original reservation object (which is immutable and always available in the bcdb)
In order to avoid doing too many round trips to the bcdb, we can keep a certain amount of resource and user ID mapping in memory
in a LRU cache or something. Then in the case we receive a reservation that reference another reservation that we don't have
in memory we can fetch it from the bcdb. Once we have it we can verify its signature, re-generate the resource id and make sure
the reservation we are processing doesn't try to access some resource from another user.
I've already implemented a POC of all the cryptography required to do this. This all works good.
Some open questions:

what kind of hash to use ? we need something that can be used in a path since the ID of a volume is its mountpoint
do we create the hash from the reservation content or from the signature itself ?

That sound great !

I think sha256 is a quite good for that
Why not both ? A mix of everything, since data are immuable.

I think sha256 is a quite good for that

Good, I think too. That's what I used in my draft PR : https://github.com/threefoldtech/zosv2/pull/143

Why not both ? A mix of everything, since data are immuable.

Well I was thinking since the signature is already a hash of the reservation content already maybe that would just be double work. We could perfectly just hash the signature itself to bring it to a reasonable size.


Module not found: Package path ./generated is not exported from package

contentlayer was working fine until i added a new markdown highlighter and this started happening.

terminal only showed warnings, when it should probably show build failures

note: not a ts issue
i checked the folder, the folder failed to generate, didnt exist


Docs say the sfdx CLI is deprecated, but the release notes don't and commands don't generate warnings

Summary
According to the docs, all sfdx commands are now deprecated as of April 20th, and as it refers to sfdx (deprecated) in the index, they kind of suggest the entire executable is deprecated. The release notes don't seem to match up with this statement, and if I run a deprecated command I don't get any warnings about it. I'd also expect updating the CLI to give me a big warning if all the commands are deprecated, but it just tells me about the latest fixes.
Finally, the VS Code extension is still executing sfdx with deprecated commands, so I'm confused!
Steps To Reproduce:

Execute sfdx alias:list

Expected result
This command is deprecated according to this document. I'd expect a warning about this, according to the CLI Deprecation Policy
Actual result
The command completes without any warnings about deprecation
System Information

zsh

$ sfdx version --verbose --json
{
"cliVersion": "sfdx-cli/7.200.7",
"architecture": "darwin-arm64",
"nodeVersion": "node-v18.15.0",
"osVersion": "Darwin 22.4.0",
"shell": "zsh",
"rootPath": "/Users/kbowden/.local/share/sfdx/client/7.200.7-499d9c2",
"pluginVersions": [
"@oclif/plugin-autocomplete 2.1.9 (core)",
"@oclif/plugin-commands 2.2.14 (core)",
"@oclif/plugin-help 5.2.9 (core)",
"@oclif/plugin-not-found 2.3.23 (core)",
"@oclif/plugin-plugins 3.0.1 (core)",
"@oclif/plugin-search 0.0.17 (core)",
"@oclif/plugin-update 3.1.13 (core)",
"@oclif/plugin-version 1.3.3 (core)",
"@oclif/plugin-warn-if-update-available 2.0.36 (core)",
"@oclif/plugin-which 2.2.19 (core)",
"apex 2.2.15 (core)",
"auth 2.7.14 (core)",
"community 2.2.11 (core)",
"custom-metadata 2.1.19 (core)",
"data 2.3.14 (core)",
"deploy-retrieve 1.8.18 (core)",
"dev 0.7.4 (user)",
"info 2.6.10 (core)",
"limits 2.3.13 (core)",
"org 2.8.1 (core)",
"packaging 1.16.11 (core)",
"schema 2.3.10 (core)",
"settings 1.4.6 (core)",
"signups 1.4.14 (core)",
"source 2.10.7 (core)",
"telemetry 2.2.0 (core)",
"templates 55.4.13 (core)",
"trust 2.4.14 (core)",
"user 2.3.10 (core)",
"bbdoc 4.1.0 (user)",
"bbmutation 1.1.0 (user)",
"sfdx-cli 7.200.7 (core)"
]
}
Additional information
I'm trying to figure out the likelihood of the existing tools/automation built with sfdx breaking, and how long I've got to migrate to sf, but it's really not clear to me what is happening.

Thank you for your question and we apologize for the confusion. We are in the process of moving from 2 CLIs (sfdx and sf) to 1 . We're working hard to make the transition as seamless as possible for our customers, including updating all of our docs to reflect this change. However, we probably jumped ahead of the game when we marked all commands as deprecated in the Salesforce CLI Reference. We fixed that, and you should see the updated docs next week.
Currently some sfdx commands are marked as deprecated and we are encouraging you to try out the newer commands where and when applicable.
Our goal is to guide you through this transition with clear messages (where applicable) within the CLI commands themselves, along with documentation support.
Thank you for following our release notes closely. Keep reading them for more updates.

@keirbowden, as we are going from 2 CLIs (sfdx and sf) to 1, can we get 30 mins of your time for a user interview to better understand your current experiences with the CLI and how we can make this transition more seamless?
Looking forward to hearing from you!
Thanks!

@preddivari sure thing - I'm in the UK so usually late afternoon for me/first thing in the morning for you works well.

we changed the text to explain better that "that set of docs is deprecated."  In some cases the commands are similar/identical in the new docs


Support save actions

Suggestion
JDT.UI has a notion of "save action" - performing code manipulation on save (e.g. "organize imports"). JDT LS also supports this and it's exposed in VScode Java .
JDT LS and JDT UI implementations use different keys for same save actions. As for both the keys are part of user configs it's not feasible to change any of them thus the plan is:

Make JDT LS support both keys for same actions
Feed JDT LS with org.eclipse.jdt.ui.prefs from the project to enable the execution of the save action.

Community

[x] I understand suggesting an enhancement doesn't mandate anyone to implement it. Other contributors may consider this suggestion, or not, at their own convenience. The most efficient way to get it fixed is that I implement it myself and contribute it back as a good quality patch to the project.

@mickaelistria Is it feasible to have different org.eclipse.jdt.ui.prefs per project while using same JDT LS process ?

I'm not sure I get the question ;)

different org.eclipse.jdt.ui.prefs per project

You mean each project would have its own org.eclipse.jdt.ui.prefs? If yes, then yes, it's totally possible.
If you want 1 project to have multiple org.eclipse.jdt.ui.prefs and some consumers automatically picking one or the other under some criterion, it's way less possible. If there is one which is dedicated to JDT-LS, it would be easier to have a org.eclipse.jdt.ls.prefs file that JDT-LS would use (possibly chaining it with the org.eclipse.jdt.ui.prefs one as fail-back).

Fixed upstream with https://github.com/eclipse-jdtls/eclipse.jdt.ls/pull/2870 and verified in latest snapshot. Note that if it takes more that 3 seconds for the LS to compute the save actions, they are discarded.


Made with InfyOm Generator

Add your website to Made with InfyOm Generator List
I just came to the idea that why don't we have a list of websites/projects which were built or already using InfyOm generator.
I came across lots of websites and admin panel which is using this generator. so preparing one list will be great motivation for the team.
Anyone who wants to add his/her website to the list can comment on this issue.
The list will be updated every week for new comments.

Mentioning some contributors:
@dhcmega @NightZpy @phillipmadsen @jschlies @alexanderreiff @claudiobastos @motia @remxcode @Katenkka @xiaohuilam @uniconstructor @JosueMOsunaG12 @rascoop @jeffschuler

@mitulgolakiya https://app.reantamchet.com/ Thanks for the awesome work!


Vague purpose of dev.md

dev.md holds >99% of the official API docs (outside Poggit). I remember one day I spent so long trying to find an official API doc of this plugin. But end up giving up because I omitted dev.md after getting told its purpose.
I originally wanted to complain the purpose, "This document is for developers who want to contribute to this project" is misleading. However, while writing this issue, I realized that making (peripheral) plugins based on the Capital API can be seen as a contribution to the "project". Therefore, I say its purpose is vague instead of misleading.
TL;DR
I do not want to open Poggit to look up the official API doc. But I thought dev.md is for internal dev so I did not read it while looking up on GitHub.

that line is obsolete indeed. it is now for both users and contributors.


Add support for exe game file

I have exe file of the games i want them to be listed using custom shortcuts. Please add support for exe files also

Thank you for your contribution. I will take a look at this feature.
For now, you can create shortcut for .exe inside directory.

Hello.
I am sorry, after much consideration I have decided not to implement this feature. The reason is that the game shortcuts for GameLauncher are stored in the AppData directory, which according to all guidelines should not contain any non-application related data or binaries. This also means that this directory can be easily deleted by uninstalling an application for example, and I want to prevent users from losing some data.
If you really need to keep exe in this directory I would recommend just creating a shortcut next to and exe in this directory. I do not want to add this feature for users to encourage them to keep exe in that directory, it is mainly meant for shortcuts only.


"Build, Test, and Release" CI/CD job failing on "Secrets" step

Steps to reproduce the problem

Go to the project's GitHub Actions and into the Build, Test, and Release job.
See how the jobs have been failing lately.
Go into one of the failing jobs and into the Secrets step.
See the error log.

What is the current behavior?
The "Build, Test, and Release" job is failing. The production build is outdated.
What is the expected behavior?
The job should be executing properly. The "Secrets" step shouldn't have encounter error.
Screenshots (optional but very helpful)

Thanks a lot @Tansito for taking care of this! It's working fine now again. 🙂

And now it failed again 😂 . We'll have to look into this again.


Removes 'Muxiv Music'

Fixes #553

https://github.com/Igglybuff/awesome-piracy/pull/524#issuecomment-848969254


⚠ Itaca Gallery has degraded performance

In 15064f9, Itaca Gallery ($SITE_TCGLR) experienced degraded performance:

HTTP code: 200
Response time: 7210 ms

Resolved: Itaca Gallery performance has improved in 0142401 after 6 minutes.


asFlow() leaks cloned watcher

Current implementation of asFlow is flawed in that it will leave the call active even when the flow's coroutine has been cancelled.
You can observe this behavior on my commit https://github.com/lwasyl/apollo-android/commit/9c6fde79844cf8c31e2f5c2b4e042c7b59ad6d30 in CoroutinesApolloTest#watcherFlowCancellationCancelsWatcher test. The test is failing but I left the logs to make it clear what's happening:
Awaiting first flow
[FoosCall] Got event SCHEDULED
[FoosCall] Got event FETCH_CACHE
[FoosCall] Got event FETCH_NETWORK
[FoosCall] Got response com.apollographql.apollo.api.Response@7e1bcf9c
[FoosCall] Got event COMPLETED
Cancelling first job
Clearing caches
Enqueueing second response
Calling second query
[BarsCall] Got event SCHEDULED
[BarsCall] Got event FETCH_NETWORK
[FoosCall] Got event SCHEDULED
[FoosCall] Got event FETCH_CACHE
[FoosCall] Got event FETCH_NETWORK
[FoosCall] Got failure com.apollographql.apollo.exception.ApolloNetworkException: Failed to execute http call
[BarsCall] Got response com.apollographql.apollo.api.Response@cab19945
[BarsCall] Got event COMPLETED
Received response: com.apollographql.apollo.api.Response@cab19945
Second query value received

As you can see, even after cancelling the first call coroutine, the FoosCall is still active and even tries to reach out to the network (because the cache has been cleared, so behaving as expected as per #2033)
I belive the bug is in the asFlow implementation which creates a clone() and subscribes to it, but it cancels the originally passed watcher, thus leaving the cloned one open and leaking. Simply capturing cloned watcher and cancelling it instead of this@toFlow fixes the issue

Thanks for the detailed issue description. Your solutions sounds reasonable. Can you open a Pull Request with that fix? It would be awesome to include this to upcoming 2.0.0 release.

@tasomaniac Opened https://github.com/apollographql/apollo-android/pull/2175


taze now throws 401 errors when using a custom repository

Describe the bug
We use azure devops as a package repository and mirror which is configured via .npmrc.  From taze 0.14.0 and up it fails to fetch the package versions, throwing a string of 401 errors for each package.  I tried setting --loglevel debug but it doesn't provide any additional output.
Our redacted .npmrc file looks like the below:
registry=https://pkgs.dev.azure.com/#####/_packaging/#####/npm/registry/

always-auth=true

; begin auth token
### auth token here
; end auth token

Command output:
yarn taze
 ERROR

> @apollo/client unknown error
401
> @azure/web-pubsub-client unknown error
401
> @ckeditor/ckeditor5-react unknown error
401
> @emotion/react unknown error
401
...

I'd be happy to try and raise a PR but if I was to, aside from digging into the complexities of npm header configuration, I'd say the fix would be to revert the 0.14.0 performance fix. I think the new way you're fetching versions via raw $fetch instead of npmRegistryFetch.json might not sufficiently support the various ways npm can be configured. And not sure about anyone else but I really didn't see perf as an issue.
Reproduction
configure custom repository, run npx taze
System Info
System:
    OS: macOS 14.5
    CPU: (10) arm64 Apple M1 Max
    Memory: 71.83 MB / 32.00 GB
    Shell: 5.9 - /bin/zsh
  Binaries:
    Node: 18.20.3 - ~/.nvm/versions/node/v18.20.3/bin/node
    Yarn: 3.1.1 - ~/.nvm/versions/node/v18.20.3/bin/yarn
    npm: 10.8.1 - ~/Documents/Repos/client-api-auth/node_modules/.bin/npm
  Browsers:
    Chrome: 126.0.6478.127
    Edge: 126.0.2592.87
    Safari: 17.5

Used Package Manager
yarn
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guide.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A, please open a GitHub Discussion instead.
[X] The provided reproduction is a minimal reproducible of the bug.

Contributions

[?] I am willing to submit a PR to fix this issue
[ ] I am willing to submit a PR with failing tests (actually just go ahead and do it, thanks!)

same question


jobs are completed at the same time from different routes

i run mocha tests withsupertest, passport and passportStub,
i mock 2 users login and try to queue a job for each request on post endpoint.
my console outputs:
job 1 started
job 2 started
job 1 completed
job 2 completed

any ides for what do i miss? thanks

Please provide some code and re-open


Toolbar icons are not populating.

Verify that the layout is being inflated correctly

Just had it commented out for some reason.


rng generators

Created the luxai2021/env/rng folder, which has a function in rng.py called get_n_values(seed, N) which gives you N random numbers that are the same ones as generated by the CLI tool for a given integer seed.

Thanks, merged!


Question of why set last_config to last uncommitted config in log when node becomes leader

Hi @greensky00, I have a question of why set last_config to last uncommitted config in log when node becomes leader.
If broadcast the last uncommitted config in log, whether  the newest last_config will be ignored?
// If there are uncommitted logs, search if conf log exists.
        ptr<cluster_config> last_config = get_config();

        ulong s_idx = sm_commit_index_ + 1;
        ulong e_idx = log_store_->next_slot();
        for (ulong ii = s_idx; ii < e_idx; ++ii) {
            ptr<log_entry> le = log_store_->entry_at(ii);
            if (le->get_val_type() != log_val_type::conf) continue;

            p_in("found uncommitted config at %zu, size %zu",
                 ii, le->get_buf().size());
            last_config = cluster_config::deserialize(le->get_buf());
        }

It is the same as committing normal logs. When a member becomes a leader, it attempts to replicate and commit all uncommitted logs. If there is any uncommitted config log, that should be committed as well.
But, whenever a server becomes a leader, it should generate a new config log with the term it has, and that config should be based on the latest config it has, and that latest config might be an uncommitted one. So above logic is to find the latest config.

Got it, Thank you.


Left sidebar automatically closed when clicking on "Filter stream list" button on responsive condition

On responsive condition, when we click on "Filter stream list" button, the left sidebar automatically closed, I think it should be not automatically closed.
How to reproduce:

Login to Zulip.
Resize the browser window until Zulip transform to responsive.
Click on the Filter stream list button.

And the left sidebar will be automatically closed.

(Sorry for the buggy gif)

Nice find!  Definitely worth fixing; it's probably an interesting CSS bug :)

@timabbott I'll pick this one up.

@timabbott I'd like to work on this :) Can you give me any hint on where in the zulip repository I could start my search for the bug?

Likely in static/js/stream_list.js or the related code in click_handlers.js


[Go][Parquet] Potential inconsistency between TotalBytesWritten tracked by RowGroupWriter and actual bytes written to io.Writer

Describe the bug, including details regarding any error messages, version, and platform.
When using the following props for a ParquetWriter, there is a discrepancy between the sum of RowGroupTotalBytesWritten() for each Write() call and the actual number of bytes seen by the target io.Writer interface.
parquetProps := parquet.NewWriterProperties(
		parquet.WithAllocator(memory.DefaultAllocator),
		parquet.WithCompression(compress.Codecs.Snappy),
		parquet.WithCompressionLevel(flate.DefaultCompression),
		parquet.WithDictionaryDefault(false),
		parquet.WithStats(false),
                parquet.WithMaxRowGroupLength(math.MaxInt64),
	)
arrowProps := pqarrow.NewArrowWriterProperties(pqarrow.WithAllocator(memory.DefaultAllocator))

In this specific case, a 13 MB file had only reported about 10 MB written via RowGroupTotalBytesWritten() calls. Some of the discrepancy can be attributed to metadata that is not included in the row groups, but this likely doesn't explain the entire difference. We should investigate the root cause and either fix it or document the explanation for future users of this API.
Related to arrow-adbc@1456
Component(s)
Go, Parquet

Issue resolved by pull request 43326
https://github.com/apache/arrow/pull/43326


Update observability quickstart sample to show OpenTelemetry exporter usage

Description
Update observability quickstart sample to show OpenTelemetry exporter -> Zipkin usage.
Please note that in this PR the OpenTelemetry Collector Helm chart was used. This is so that we don't have to maintain all the independent components ourselves, but instead rely on the Helm chart parameters. I will follow up with another PR to switch the OpenTelemetry how-to in dapr/docs to use the same for simplicity & consistency.
Issue reference
Please reference the issue this PR will close: #324
Checklist
Please make sure you've completed the relevant tasks for this PR, out of the following list:

[x] The quickstart code compiles correctly
[x] You've tested new builds of the quickstart if you changed quickstart code
[x] You've updated the quickstart's README if necessary

@tcnghia If zipkin exporters are going to be deprecated. Please create corresponding issue in CLI to replace zipkin with opentelemetry configurations. https://github.com/dapr/cli/blob/master/pkg/standalone/standalone.go#L888

@tcnghia If zipkin exporters are going to be deprecated. Please create corresponding issue in CLI to replace zipkin with opentelemetry configurations. https://github.com/dapr/cli/blob/master/pkg/standalone/standalone.go#L888

+1, thanks
I've added you as a reviewer. Can you please help trying out the new instructions?  Thanks!

We should keep the quickstarts simple (they need to be a quick way to show value of the specific area) so I don't think we need to have options (we already have an option for each quickstart to run locally vs k8s - this needs to be added here as mentioned in https://github.com/dapr/quickstarts/issues/288 but outside the scope of this PR).
If using the Open Telemetry collector is where we are headed we should just have users do that and remove the other option.

closing -- we are reconsidering our approach here. will replace with follow ups.


Support device specifier in image captioning example (predict.py)

Fixes #7205
Part of #6661
~Depends on #7204~

Jenkins, test this please.

LGTM!

Jenkins CI test (for commit 2bb7ae64f3646fafb92a88e40280f61d1fc664b6, target branch master) failed with status FAILURE.

Jenkins, test this please.

Jenkins CI test (for commit 2bb7ae64f3646fafb92a88e40280f61d1fc664b6, target branch master) failed with status FAILURE.

Jenkins, test this please.

Jenkins CI test (for commit 2bb7ae64f3646fafb92a88e40280f61d1fc664b6, target branch master) failed with status FAILURE.

Jenkins, test this please.

Jenkins CI test (for commit 2bb7ae64f3646fafb92a88e40280f61d1fc664b6, target branch master) failed with status FAILURE.

LGTM!


tests fail to run after updating to v 2.2.10

Description
I currently run unit tests using MSTest.TestFramework v 2.2.8.
I also used MSTest.TestAdapter which I've upgraded from 2.2.8 to 2.2.10, and
Microsoft.NET.Test.Sdk which i've upgraded from 17.0.0 to 17.2.0.
Everything runs fine. But when i tried upgrading MSTest.TestFramework v 2.2.8 to 2.2.10, the tests would not run.  They are discovered, but don't run.
I work in VS 2022 on a windows 10 machine.
Is this a bug in the package, or is there a command required by the new version?

Hi @krb-vci, could you share some repro or some verbose logs? Sadly, there isn't much we are able to do with the information you are providing.

Hi @Evangelink - Do you mean verbose build logs?  (I am still a newbie - still learning VS - and could use some guidance of what to send you.

A little more info...
After updating the MSTest.TestFramework package to 2.2.10 I tried to debug a test, with a breakpoint at the first line of the assembly init.  The breakpoint was never reached.  The Test output log shows this:
Building Test Projects
========== Starting test run ==========
========== Test run finished: 0 Tests (0 Passed, 0 Failed, 0 Skipped) run in < 1 ms ==========
When i try to Run the test I get the same test output.

As stated already - I am still learning Visual Studio. I did verbose Test output, and am providing 2 files of the logs.  One is before the update, and the other is after.  Again - before the update the tests rtun.  After the update they don't even get to the first line of the Assembly Init.  And again I redacted some of the proprietary references that won't matter to you
TestOutputAfterUpgrade.txt
TestOutputBeforeUpgrade.txt
.

@krb-vci  I will investigate today.

@krb-vci I will need the diagnostic logs (there is nothing relevant in the simple logs you have sent). Please follow this guide https://github.com/microsoft/vstest-docs/blob/main/docs/troubleshooting.md#visual-studio

Have been away from my desk all day.  Will do this tomorrow morning California time.
Can I send the zip file directly, without posting the world?
Thanks,
Ken Blaker
<EMAIL_ADDRESS>or @.***
From: Amaury Levé @.>
Sent: Thursday, July 14, 2022 6:44 AM
To: microsoft/testfx @.>
Cc: Ken Blaker @.>; Mention @.>
Subject: Re: [microsoft/testfx] tests fail to run after updating to v 2.2.10 (Issue #1134)
@krb-vcihttps://github.com/krb-vci I will need the diagnostic logs (there is nothing relevant in the simple logs you have sent). Please follow this guide https://github.com/microsoft/vstest-docs/blob/main/docs/troubleshooting.md#visual-studio
—
Reply to this email directly, view it on GitHubhttps://github.com/microsoft/testfx/issues/1134#issuecomment-1184466961, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AH4Y5A6S3XPTUVDLGBYF5MTVUAKR7ANCNFSM5ZKKUWYQ.
You are receiving this because you were mentioned.Message ID<EMAIL_ADDRESS>

I was too curious to wait until tomorrow.  I’ve attached 2 zip files.  One is three files generated before upgrading to the latest version.  The after zip only has 2 files generated after upgrading the package.
From: Amaury Levé @.>
Sent: Thursday, July 14, 2022 6:44 AM
To: microsoft/testfx @.>
Cc: Ken Blaker @.>; Mention @.>
Subject: Re: [microsoft/testfx] tests fail to run after updating to v 2.2.10 (Issue #1134)
@krb-vcihttps://github.com/krb-vci I will need the diagnostic logs (there is nothing relevant in the simple logs you have sent). Please follow this guide https://github.com/microsoft/vstest-docs/blob/main/docs/troubleshooting.md#visual-studio
—
Reply to this email directly, view it on GitHubhttps://github.com/microsoft/testfx/issues/1134#issuecomment-1184466961, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AH4Y5A6S3XPTUVDLGBYF5MTVUAKR7ANCNFSM5ZKKUWYQ.
You are receiving this because you were mentioned.Message ID<EMAIL_ADDRESS>

@Evangelink -
AfterLogs.zip
Before-logs.zip

I was able to solve this by simply creating a new project and copying in all my classes and tests.
Still have no idea why I had this problem.


Integrate metersphere to KubeSphere DevOps

https://metersphere.io/

I found a video which introduce how to integrate Jenkins with MeterSphere. https://www.bilibili.com/video/BV1yy4y1S75X

I found a video which introduce how to integrate Jenkins with MeterSphere. https://www.bilibili.com/video/BV1yy4y1S75X

/project devops

/project devops

https://kubesphere.com.cn/forum/d/3460-kubesphere-metersphere
https://kubesphere.com.cn/forum/d/3484-kubesphere-devops-metersphere

https://kubesphere.com.cn/forum/d/3460-kubesphere-metersphere
https://kubesphere.com.cn/forum/d/3484-kubesphere-devops-metersphere


Question about Power Spectral Analysis

I want to use the bandpower function of Spectral analysis.
How do I split the rawEDF instance loaded from the mne to see band power every 30 second epoch?
code like
raw[0:3000]
When used for band power like that, an error message "AssertionError: Data must be a numpy array." appears.

Hi @PhD-GOAT,
I would try the following (not tested):
data = raw.get_data(picks="C4-M1", units="uV")  # Pick a single channel, here C4-M1
data = np.squeeze(data)
sf = raw.info["sfreq"]
times, epochs = yasa.sliding_window(data, sf, window=30)
bp = yasa.bandpower(epochs, sf, win_sec=5)

Alternatively you can use the yasa.SleepStaging.get_features() method:
Thanks
Raphael

aha, yasa.SleepStaging.get_features() Does this function output the data you have applied to machine learning as a result value?

Correct @PhD-GOAT , this function outputs the features (30-sec resolution) that are then sent to the machine-learning model for sleep stage classification.


[W6.5][T09-B1] Goh Keng Seng

updated background color, font size and opacity of GUI

Accepting this PR with minimal review.


teams: feature-flagging for hidden chain rotation on a per-team basis

for now, don't hook CLKR into this, we'll be doing that in future PRs (since tests don't currently pass)
if an admin on prod, hit the server and ask if we can use hidden links for this team

merging over flake


Just wondering why it's so tied to Contact data type

Would it be possible to create something that takes Countries, for example? Also, how do you insert a chip at a certain position?
That's it. It would also be nice to have control on the layout of the chip. Oh well, I keep looking. Google should really release an official library for this.

I agree. This library is better than others but I don't want it for contacts

i am not quite clear with the comments of this section, but I think i am having a similar problem. I want to use this library  with chips which are NOT emails . like just some names/number  which will become A chip on spacebar_press/ enterButton_press . is it possible?


ٍError Failed to resolve when Add library

Error:(34, 13) Failed to resolve: com.github.mancj:SlideUp-Android:2.2.2
Show in FileShow in Project Structure dialog
what is this Error?

Add the JitPack repository to your build file. Add it in your root build.gradle at the end of repositories:
allprojects {
  repositories {
    ...
    maven { url 'https://jitpack.io' }
  }
}


upgrade using --id does not filter the right match

Brief description of your issue
I'm trying to upgrade an application with their unique ID (using --id) shown by winget, but getting error of "Multiple installed packages found matching input criteria. Please refine the input."
[D:\Work]> winget upgrade --id "Audacity.Audacity"
Multiple installed packages found matching input criteria. Please refine the input.
Name           Id
--------------------------------
Audacity       Audacity.Audacity
Audacity 2.1.3 Audacity®_is1

I had two version of Audacity in "Program and Features" but according to Winget it is two different IDs
if you remove the old one, issue gone away
Steps to reproduce
try to upgrade Audacity using --id flag:
winget upgrade --id "Audacity.Audacity"
Expected behavior
Upgrade Audacity
Actual behavior
return error:
"Multiple installed packages found matching input criteria. Please refine the input."
Environment
Windows Package Manager v1.1.12701
Copyright (c) Microsoft Corporation. All rights reserved.

Windows: Windows.Desktop v10.0.19043.1348
Package: Microsoft.DesktopAppInstaller v1.16.12701.0

Logs: %LOCALAPPDATA%\Packages\Microsoft.DesktopAppInstaller_8wekyb3d8bbwe\LocalState\DiagOutputDir

Links
---------------------------------------------------------------------------
Privacy Statement   https://aka.ms/winget-privacy
Licence Agreement   https://aka.ms/winget-license
Third Party Notices https://aka.ms/winget-3rdPartyNotice
Homepage            https://aka.ms/winget
Windows Store Terms https://www.microsoft.com/en-us/storedocs/terms-of-sale

I had two version of Audacity in "Program and Features" but according to Winget it is two different IDs
if you remove the old one, issue gone away

Duplicate of #752
We're looking at this specific situation in a couple of other similarly related issues:

#1413
#1192
#1142
#752


⚠ Fortnite has degraded performance

In d812ac3, Fortnite (https://fortnite.com) experienced degraded performance:

HTTP code: 200
Response time: 1697 ms

Resolved: Fortnite performance has improved in a805d6d.


Add ttlcache package

This implements an in-memory cache with a TTL for automatically expiring records

Codecov Report
Attention: 22 lines in your changes are missing coverage. Please review.

Comparison is base (abe711e) 80.10% compared to head (9165c5a) 79.98%.

Files
Patch %
Lines

ttlcache/ttlcache.go
73.80%
18 Missing and 4 partials :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #80      +/-   ##
==========================================
- Coverage   80.10%   79.98%   -0.13%
==========================================
  Files          54       55       +1
  Lines        4248     4332      +84
==========================================
+ Hits         3403     3465      +62
- Misses        702      720      +18
- Partials      143      147       +4

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


feat: add vim-easy-align

A plugin to easily align code.
Would be awesome if you could merge this.
I hope I don't spam you :) and thanks for this very nice repo

junegunn/vim-easy-align is maintained officially in nixpkgs and the version 2019-04-29 in the master branch is up-to-date, so it's duplication.


pip install error

python 3.6.7, install tensorflow, here is log:
Collecting absl-py>=0.1.6 (from tensorflow->-r pip_requirenents.txt (line 21))   Using cached https://files.pythonhosted.org/packages/fa/ef/1fa0376563b1e0495301ada8e881d88b7ebfda433f6b31f44447fe6ef795/absl-py-0.6.0.tar.gz     Complete output from command python setup.py egg_info:     /Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/distutils/dist.py:261: UserWarning: Unknown distribution option: 'long_description_content_type'       warnings.warn(msg)     error in absl-py setup command: 'install_requires' must be a string or list of strings containing valid project/version requirement specifiers; Expected version spec in enum34; python_version<='3.4' at ; python_version<='3.4'

i'm sorry , absl-py == 0.5.0 is right, absl-py == 0.6.0 appear error

What version of pip are you using (pip --version)? I upgraded to 0.6 and uninstalled/reinstalled and both went OK.
Can you provide the exact command line you're running?

python3.5.1, pip18.1
the steps i operated :

make a virtual environment
python3.5 -m venv temp

cd to temp and active environment
cd temp/
source bin/activate

upgrade pip
curl https://bootstrap.pypa.io/get-pip.py -o get-pip.py
python get-pip.py

show the version of pyton and pip
python -V

Python 3.5.1

pip -V

pip 18.1 from /Users/Abelhu/Desktop/temp/lib/python3.5/site-packages/pip (python 3.5)

install absl-py
pip install absl-py

Collecting absl-py
Using cached https://files.pythonhosted.org/packages/fa/ef/1fa0376563b1e0495301ada8e881d88b7ebfda433f6b31f44447fe6ef795/absl-py-0.6.0.tar.gz
Complete output from command python setup.py egg_info:
/Library/Frameworks/Python.framework/Versions/3.5/lib/python3.5/distutils/dist.py:261: UserWarning: Unknown distribution option: ''long_description_content_type''
warnings.warn(msg)
error in absl-py setup command: 'install_requires' must be a string or list of strings containing valid project/version requirement specifiers; Expected version spec in enum34; python_version<='3.4' at ; python_version<='3.4'

Command "python setup.py egg_info" failed with error code 1 in /private/var/folders/86/7ptlxff10831w7wkzf15mypw0000gn/T/pip-install-c038pkm_/absl-py/

Same issue here in RHEL 7.4
{code}
OS RHEL 7.4
pip 18.1 from /usr/lib/python2.7/site-packages/pip (python 2.7)
{code}

After searching around a bit, I found some posts of similar problems and people suggested upgrading setuptools might work around this: pip install setuptools --upgrade
What version of setuptools are you all using?  python -m easy_install --version
I haven't been able to repro this yet

Upgrading setuptools solves the issue. However, why are you writing the python requirement this way:
install_requires=[
        'six',
        "enum34; python_version<='3.4'"
    ],

Instead of:
python_requires='>=3.4',
install_requires=[
        'six',
        "enum34'"
    ],

Is there reason for that syntax you have?

I'm hitting this on some cloud jobs where I can't directly access the setuptools versions they are using, but it's python2.

Absl's supported (and tested) python versions are 2.7 and 3.4+. The intent of that install_requires line is to only install enum34 for Python versions before 3.4 (since enum doesn't exist in 2.7 or earlier python versions).
I think the python_requires arg in setup controsl the required python version for the whole package, not the conditions a dependency should meet to be installed.
@mtdeguzis are you able to recover the setuptools version you upgraded from?
I tried installing back to setuptools v1.0, and pip install absl 0.6 still worked. I found some posts that have some other ways to have python-version-specific dependencies, but without a way to repro the issue, I can't verify whether it'll actually work or not.

@rickeylev yep, I am using a base RHEL 7.4 image via Docker
pip --version
pip 18.1 from /usr/lib/python2.7/site-packages/pip (python 2.7)
python -c 'import setuptools; print setuptools.__version__'
0.9.8

pip install setuptools --upgrade
python -c 'import setuptools; print setuptools.__version__'
40.4.3

That's RHEL for you...  :/
Ok, I see pypa does the same for pipenv: https://github.com/pypa/pipenv/blob/master/setup.py#L24

Thank you! I was able to repro with setuptools 0.9.8
repro steps:
# python --version -> 2.7.15
# pip --version -> 18.1
# easy_install --version -> 0.9.8

pip uninstall -y enum34 absl-py
pip install setuptools==0.9.8
pip install absl-py --no-cache-dir
# results in
$ pip install absl-py --no-cache-dir
Collecting absl-py
  Downloading <snip>absl-py-0.6.0.tar.gz (93kB)
    Complete output from command python setup.py egg_info:
    /home/rlevasseur/.pyenv/versions/2.7.15/lib/python2.7/distutils/dist.py:267: UserWarning: Unknown distribution option: 'long_description_content_type'
      warnings.warn(msg)
    error in absl-py setup command: 'install_requires' must be a string or list of strings containing valid project/version requirement specifiers

    ----------------------------------------
Command "python setup.py egg_info" failed with error code 1 in /tmp/pip-install-KA5Tp3/absl-py/

note: --no-cache-dir is necessary since it appears the cached downloads take precedence over parsing the requires string; without it, a successful install will prevent triggering the bug

So, I think the best fix is to upgrade your setuptools package.
That said, I can understand if that's problematic in some cases. I'm pretty sure I have a fix that I'll commit in the next day or so, see: https://github.com/rickeylev/abseil-py/compare/master...rickeylev:old-setuptools-bug
I've tested it with the following configurations:

RHEL 7.4 (Google Cloud), Python 2.7.5, pip 8.1.2, setuptools 0.9.8
CentOS 7 (Google Cloud), Python 2.7.5, pip 8.1.2, setuptools 0.9.8
Debian/Ubuntu, Python 3.6.6, pip 18.1, setuptools 40.4.3
Debian/Ubuntu, Python 2.7.15, pip 18.1, setuptools 0.9.8
Debian/Ubuntu, Python 2.7.15, pip 18.1, setuptools 40.4.3

For those following along: if you have another OS you're seeing this error with, please tell me the OS and I can see if there's a VM image available for it. Or apply the aforementioned commit and do something like the following to see if it works:
python --version
pip --version
easy_install --version
git clone https://github.com/rickeylev/abseil-py.git
cd abseil-py
python setup.py sdist
pip uninstall enum34 absl-py
pip install --no-cache-dir dist/absl-py-0.6.1.tar.gz -t /tmp/absl-out --upgrade

0.6.1 has been released to pypi with the fixed setup.py

I upgraded pip from ver 9.0.1 to current (20.2.3) and the problem got fixed.
To upgrade:
pip install -U pip


Dropdowns can Render out of the view

I was experiencing an issue where for-downs could render partially out of the view making some of the items unelectable, to replicate make a dropdown nearish the bottom of the screen with quite a few items in and it will be partially cut off.
The issue seems to occur because of useLayoutDropdown.js line 26, if height is not specified in dropdownStyle which is not listed as a required prop it uses devices window height / 4. It's quite simple for me to set the height in my usage but potentially dropdownStyle should be listed as required or for the getDropdownHeight to get a value another way.

Same here, is there another fix to this? I don't want a huge dialog for only 3 options, but then I need a big dialog for long lists. Setting max and min heights do not work, only height.

I have the same issue

Hey Guys, I had the same issues, & by reading @harryfos question & then the explanation, I tried this & it works for me
dropdownStyle={{         height: 300,       }}

Hi, I had dropdownStyle={{ height: 300, }} and now it works, but I had problems when screen orientation change

Running into the same issue, but also as VincentGillot said, having a fixed with is not desirealbe


Update the base rooms

I added some details to the base rooms. Please be patient with me, I have just begun to learn how to use netradiant.
The geometry is almost unchanged. I tried to make the base rooms look less boring by using a few different textures. The elevators have been simplified to work similar to @Masmblr's map fort5-ds. They are usable even by Unvanquished bots.
Screenshots:

Thanks Sweet !


SLP updates

simplify path
add tests
return info about token

:tada: This PR is included in version 1.0.0 :tada:
Your semantic-release bot :package::rocket:


src/molecule/driver/delegated.py: Fix connection plugin name setting

In ansible_connection_options(), when setting connection plugin name, the code does:
            for i in instance_params:
                 if d.get(i[0], i[1]):
                     conn_dict["ansible_" + i[0]] = d.get(i[0])

Looking at instance_params, for the connection, there's a default value neither None nor False, leading to the the being true and setting ansible_connection to d.get(i[0]), which may not be defined. In this case, this will lead to ansible_connection being set to None. This will lead to troubles. For instance, when using 'meta: reset_connection', ansible in _execute_meta() will call plugin_loader.connection_loader.get() with first param being None, leading to :
ERROR! Unexpected Exception, this is probably a bug: 'NoneType' object has no attribute 'startswith'
So, ensure that the default value is set when setting conn_dict.
Fixes: 5b750ff4e829df934d159e1bfbd0cc433139f71c ("Adds Support for Shell Type Instance Param to Delegated Driver")

@apatard Do you think you could show me a minimal example that reproduces the bug? I want to include it in the test suite so we are sure we fix it properly.

took me some time to find a way to reproduce but here it is:
https://gist.github.com/apatard/5ac88bbebc3409c3b896597a4621d1f6
(it's a shell script which will create the reproducer)
Nevertheless, I'm not sure how to convert it into a proper test case


Improve API documentation referenced by chapter 1 of the new Bevy Book

Objective
The upcoming Bevy Book (see bevyengine/bevy-website#176) makes many references to the API documentation of bevy. This PR attempts to improve this documentation in order to help readers who want to delve deeper into technical details.
Solution

Updated App description.
Added a brief description to each method of App

Considerations and questions:

I'm currently thinking to not document the methods of AppBuilder since it will probably be removed for 0.6, as seen here. However, I think that the methods themselves will migrate elsewhere, like App.
DefaultPlugins and MinimalPlugins docs are currently just a list of the plugins they contain. Would it be appropriate to add a brief but general description to each of them?
What about the module level documentation? We can't follow the example of Rust's standard library, since we already have the Bevy Book that gives a general overview of the various topics. Maybe we can add links from API docs to Book to avoid redundancy?
Once the reader lands on the API docs, they have the opportunity to navigate around to learn more. How deep can we go to ensure that the reader won't get discouraged by the lack of documentation? I think going one level deeper is a good compromise between feasibility and user experience.

Collaboration
Any kind of collaboration is welcome, especially corrections, wording, new ideas and guidelines on where the focus should be put in.

The appbuilder methods are where a lot of the interesting content is. I kind of want those documented, but then migrate them over to the new API.

Ok. I may be able to do that in a couple of days.

Status update
I added stub documentation for all the AppBuilder methods.
Design decisions to be taken

All public items of the app_builder module have now documentation. Should we add a #![warn(missing_docs)] at the top of the file to ensure it remains fully covered?
Most methods are just a wrapper around World and Schedule methods. How do we manage to avoid too much doc duplication between bevy_app and bevy_ecs? Since bevy_app depends on bevy_ecs, i think the meat should be there, and bevy_app adds a link to bevy_ecs for more details. I did something like that for the AppBuilder::stage method.

Further tasks

Add an example for each method.
Add a second paragraph where applicable to describe the item more deeply.

@alice-i-cecile I added in the last commit a link stub for the book. I think this may make a good addition for people who want to have a better insight into Bevy's design philosophy, if you find it appropriate.
This would likely go into chapter 2. I thought about expanding the scope of this PR to cover chapters 1 and 2, since they extensively touch topics that span over bevy_ecs.

Could you add a Fixes #2246 to the PR description, so that that Issue gets automatically closed when this is merged.
To me, #2246 seems more like a pure Documentation Problem, unlike #2229 which can also be solved by #2434

Added some examples for AppBuilder methods. I also shown how the builder pattern works on top of the page, in order to make the single examples more concise.

@alice-i-cecile, I think the PR is ready for review. I covered most of the APIs referenced by the Book, to my knowledge.

Link to #1014

@alice-i-cecile, I think the PR is ready for review. I covered most of the APIs referenced by the Book, to my knowledge.

could you rebase on main to solve the conflicts?

@alice-i-cecile, @mockersf Merge conflicts resolved. I also deleted references in documentation to the removed WithBundle query filter.
@Ratysz I also documented part of the schedule API. Probably you can spot potential inaccuracies here better than many others.
Switching from draft to regular PR.

I fixed all rustdoc warnings when generating with command cargo doc -p bevy_ecs --no-deps and cargo doc -p bevy_app --no-deps

@cart, when you get a chance could you do a quick skim and then merge this? I think this is critical to get in for 0.6 and the sooner we merge it the less of a merge-conflict risk it poses (and the better the developer experience on main).

Sure thing! I'll throw it on the review stack :)

@cart Thanks for the review. I think I've applied all the suggestions.

Looks good to me!

bors r+


UpdateCommand  failed on OSGI framework

Search first

[X] I searched and no similar issues were found

Description
Hi,
We are trying to upgrade our liquibase dependency to the latest. (4.24.0)
Our application use OSGI framework.
The UpdateCommand run this code in AbstractUpdateCommandStep
 if(!isDBLocked) {
                LockServiceFactory.getInstance().getLockService(database).waitForLock();
                // waitForLock resets the changelog history service, so we need to rebuild that and generate a final deploymentId.
                ChangeLogHistoryService changeLogHistoryService = Scope.getCurrentScope().getSingleton(ChangeLogHistoryServiceFactory.class).getChangeLogService(database);
                changeLogHistoryService.generateDeploymentId();
            }

The waitForLock() reset the services list and among the rest removing the instances list:
  protected synchronized void removeInstance(T instance) {
        if (this.allInstances == null) {
            return;
        }
        this.allInstances.remove(instance);
    }

Back to AbstractUpdateCommandStep :
Line after the waitForLock() we have the getChangeLogService(database) which reach the getPlugin() method in the AbstractPluginFactory.
Line 62:
 for (T plugin : findAllInstances()) {
            if (getPriority(plugin, args) >= 0) {
                applicable.add(plugin);
            }
        }

protected synchronized Collection<T> findAllInstances() {
        if (this.allInstances == null) {
            this.allInstances = new ArrayList<>();

            ServiceLocator serviceLocator = Scope.getCurrentScope().getServiceLocator();
            this.allInstances.addAll(serviceLocator.findInstances(getPluginClass()));
        }

        return this.allInstances;
    }

The issue is with the findAllInstnces() method: Like you can see since the allInstances is empty and not null (in our scenario) ServiceLocator is not called and allInstances will remain empty and we catch this exception:
liquibase.exception.UnexpectedLiquibaseException: Cannot find ChangeLogHistoryService for postgresql
ServiceLocator is a must for us since we use OSGi and we need to find the class in the osgi bundle.
Thank you,
#Natan
Steps To Reproduce

Expected/Desired Behavior
Services should be loaded
Liquibase Version
4.24.0
Database Vendor & Version
No response
Liquibase Integration
No response
Liquibase Extensions
No response
OS and/or Infrastructure Type/Provider
No response
Additional Context
No response
Are you willing to submit a PR?

[ ] I'm willing to submit a PR (Thank you!)

@filipelautert this is related to https://github.com/liquibase/liquibase/issues/3660
we are now completely blocked on this small issue.
when is the next planned release? it is very urgent.
maybe we can add to the if a check on empty:
if (this.allInstances == null || this.allInstances.isEmpty()) {
            this.allInstances = new ArrayList<>();

            ServiceLocator serviceLocator = Scope.getCurrentScope().getServiceLocator();
            this.allInstances.addAll(serviceLocator.findInstances(getPluginClass()));
}

Thanks @yairogen  @npreisner ! Let's jump to PR https://github.com/liquibase/liquibase/pull/5143 to confirm tht it fixes this issue 👍


Phone Auth screen does not disappear on cancel

Hi there,
I am attempting to migrate from Digits to Firebase and so far it hasn't been easy. We use a custom button to open the phone ui and since we'd like to keep using this flow it seems that I have to jump though a few hoops.
Based on the sample code you have provided I was able to make the Phone auth UI appear after tapping on a button, but when I verify my phone number or tap the cancel button, the modal never disappears and it isn't clear to me how I should proceed..

Objective C or Swift: Swift
iOS version: 10.3.2
Firebase SDK version: 4.x
FirebaseUI version: 4.0.0
CocoaPods Version: 1.1.1

The problem
The Phone auth UI dialog does not get dismissed after verifying my phone number or when tapping cancel.
Steps to reproduce:
Implement a custom button that directly displays the Phone auth ui.
Observed Results:
The Phone auth appears, when tapping cancel it does not dissappear. Also, it gets stuck after validating a phone number; the spinners remains on screen and the window is never dismissed
Expected Results:
I would expect the didSignIn delegate callback to be called and / or the window to be dismissed when I tap cancel
Relevant Code:
This snippet shows the phone auth UI
func confirmPhoneNumber(withViewController viewController: UIViewController,
                      completionHandler: @escaping PhoneConfirmationHandler) {

  guard let authUI = FUIAuth.defaultAuthUI() else {
      completionHandler(.error(GenericError.unkownError))
      return
  }

  authUI.delegate = self
  let provider = FUIPhoneAuth(authUI: authUI)
  provider.signIn(withPresenting: viewController)
}

Delegate implementation
extension AuthenticationManager: FUIAuthDelegate {
    func authUI(_ authUI: FUIAuth, didSignInWith user: FirebaseAuth.User?, error: Error?) {
        guard let user = user else {
            return
        }

        print(user)
    }
}

@donnywals probably you forgot to call:
self.authUI?.providers = [phoneProvider]
Check this comment: https://github.com/firebase/FirebaseUI-iOS/issues/293#issuecomment-306688524
Please let me know if that helped you.
We will update code to verify this case in the next release.

@yramanchuk yes! That seems to have solved the issue :) Thanks.
Also, sorry for posting this duplicate..


How can my colleagues import  libraries into their own vcpkg

I have exported some libraries with extension 7z using export function. How can my colleagues import these libraries into their own vcpkg and view them using vcpkg list

We recommend that you use export to export nuget packages and use nuget to import them in Visual Studio.
To use the exported 7z packages, you may need to manually decompress them and set the corresponding include path and link path.
For better, you can use feature binarycaching to manage these binaries. See documentation.

We hope your question was answered to your satisfaction; if it wasn't, you can reopen with more info.


Add an ordering on proof edges to facilitate checking/viewing

It's easier to visualize and check proofs if there is an ordering on edges, so this PR adds this ordering. It abuses a table as a counter and bumps it after every rule fires.

From our convo: let's sidestep the semi-naive issue by just having a "magic variable" to get the current iteration

Okay this is ready for review. What it now does is

Adds iterations to the proofs just using syntax transformations
Adds a SetNoTrack primitive to set without doing magic set transformation
Makes the tester more robust by testing that we can re-parse desugared egglog

You may notice lambda_with_proofs is suspiciously slow... that's just the proof extraction, it will get fixed soontm


docs: update service instance list intro

A short explanation of the proposed change
As of https://github.com/cloudfoundry/cloud_controller_ng/commit/f444b95067b7937df9642257258b54b67503e6e9, the /v3/service_instances endpoint returns all types (managed and user provided) of service instances. This PR updates the docs accordingly.

An explanation of the use cases your change solves
User reading the docs and getting confused on what's returned by the endpoint

Links to any other associated PRs
N/A

[X] I have reviewed the contributing guide

[X] I have viewed, signed, and submitted the Contributor License Agreement

[X] I have made this pull request to the main branch

[ ] I have run all the unit tests using bundle exec rake

[ ] I have run CF Acceptance Tests

Given it's only a docs change, I haven't run the tests

Looks good, thanks @kirederik !


Variation Image needs to be published when Saved

I have to write an extension to extent Variation class with the following:
 public function onBeforeWrite (){             parent::onBeforeWrite();             $img = $this->owner->Image();             if($img ){                 $img->doPublish();             }         }
I think the above should be in the class by default.

This is covered by the $owns API. There was a bug in previous versions of SilverStripe 4 that required a workaround like you outlined.
With the latest Version of SS4 (4.2.1 as of this writing), the bug should be resolved.


Crashes whenever an application starts

Describe the bug
Whenever an appliaction starts, Hyprland does the animation, and then immediately crashes
Images/videos/anything that would help
https://user-images.githubusercontent.com/76877633/168481832-916bf523-c247-42f6-9fc8-c9b0053a4f17.MOV
Please attach a log (it's in /tmp/hypr/hyprland.log) and, if crashing, a crashdump (coredumpctl, then coredumpctl info )
hyprland.log
coredump.log

Compile in debug and send the coredump from debug, please. I can see you're using a packaged version, too (probably AUR?), have you tried compiling from source?

Compile in debug and send the coredump from debug, please. I can see you're using a packaged version, too (probably AUR?), have you tried compiling from source?

I can reproduce this too, actually. Happens both from source and from AUR (although now the AUR has complete parity with source)

I can't though, someone send me a coredump plox

Actually seemed to be a wlroots issue, forcing an upgrade on wlroots-git fixed it :D

@ERROR-404-NULL-NOT-FOUND update wlr

@vaxerski Actually seems to be something slightly diff, tried to install again and it immediately crashes, even without opening an application.
https://zer0b.in/p96WU6Fg

again, build in debug.

Actually I built from the offline PKGBUILD (not directly from AUR). Debug build works fine. Seems to be an issue only with make release.

well i can disprove that by saying works for me
give the coredump from make release then and NOT the pkgbuild, so the symbols aren't stripped for god's sake

Actually make release on its own is working fine too so something's really weird when making it from the PKGBUILD. Debugging now.

MAGIC.
Try what made Hypr work.  See hypr's pkgbuild, they made it work.

Ok, turns out the old package() function was broken, fixed it and pushed a new ver. Should be fixed.

T1C at 6:46PM: Imagine blaming my amazing AUR pkgbuild smh
T1C at 9:25PM:
:trollface:

:trollface:

@ERROR-404-NULL-NOT-FOUND This issue should now be completely fixed.
@vaxerski I believe this issue can now be closed.

ok


add mTLS configuration support for gxlb (net-lb-app-ext)

The PR introduces the support for mTLS configuration for gxlb (net-lb-app-ext).
mTLS for GXLB became GA recently, however, the Terraform resources still use Beta.
I acknowledge that I have:

[X] Read the contributing guide
[X] Ran terraform fmt on all modified files
[X] Regenerated the relevant README.md files using tools/tfdoc.py
[ ] Made sure all relevant tests pass

Thanks for the PR! I think it could be interesting to add a full mTLS example in the readme.md file.

Yes, thanks for the suggestion, @aurelienlegrand.
I am working on the documentation and adding test cases.

Closing this as there's been no progress in 3 weeks. Feel free to reopen once suggestions have been taken into account.


Notification service starts when phone is rebooted

Notification service starts when phone is rebooted even if I disable the service

v0.10.7 should fix this.
Thanks for reporting.


Error Creating Models From Directory

It seems to be currently impossible to easily upload all models from REC into an ADT resource instance while using the sample at https://github.com/Azure-Samples/digital-twins-samples/tree/master/AdtSampleApp/SampleClientApp.
First of all it seems impossible to upload everything at once due to data limits (which is fine). This is fixable by first uploading the role.json file and then every child directory of /Models/. However, this crashes on the directories /Capability, /Colletion and /Space (not sure why, the CLI says "Validated JSON for all files - now validating DTDL" but then it crashes with "Invalid command. Please type 'help' for more information".
I can then use the CreateModels command to create separate models from these directories, but it crashes when using the LoadModelsFromDirectory command.
Is there any way to overcome this, and if not; will it come in the future?

Upon further investigation; it seems that the error is caused because the models are being uploaded in the wrong order. I have used the CreateModels command to upload all the way to the deepest level in Models\Capability\Actuator\BinaryActuator\DefaultOverrideActuator\DefaultActuator\ResetActuator (which is a pain). On every level I tried to upload the models from directory but that didn't work even at \ResetActuator (which contains 2 .jsons and no other folders).
I then used CreateModels for FaultResetActuator.json and got an error; Then I uploaded ResetActuator.json which worked with no problem, Finally I uploaded FaultResetActuator.json and now it does work. So, I have both models in \ResetActuator uploaded now, but using LoadModelsFromDirectory on that same directory does not work.


Unit test failure in test_hello_pass_plugin.py

I added a print before the location of the failure.
Looks like the hello-funcs pass being registered isn't detected.
$ make test
-----------------------------------------------
Using LLVM_SRC_PATH = $HOME/Documents/llvm_new/llvm
Using LLVM_BUILD_PATH = $HOME/Documents/llvm_new/build
Using LLVM_BIN_PATH = $HOME/Documents/llvm_new/build/Release+Asserts/bin
-----------------------------------------------
python3 test/all_tests.py
.....running  /Users/steven/Documents/llvm_new/build/Release+Asserts/bin/opt ['-load', 'build/hello_pass.so', '--disable-output', '-hello-funcs', 'inputs/fact.ll']
F.....
======================================================================
FAIL: test_hello_on_fact (test_hello_pass_plugin.TestHelloPassPlugin)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/Users/steven/Documents/llvm-clang-samples/test/test_hello_pass_plugin.py", line 12, in test_hello_on_fact
    OUTPUT_HELLO_FUNCS)
  File "/Users/steven/Documents/llvm-clang-samples/test/tools.py", line 71, in assertOptPluginOutput
    self.assertEqual(rc, 0)
AssertionError: 1 != 0

----------------------------------------------------------------------
Ran 11 tests in 0.082s

FAILED (failures=1)
make: *** [test] Error 1

$ /Users/steven/Documents/llvm_new/build/Release+Asserts/bin/opt -load build/hello_pass.so -hello-funcs inputs/fact.ll
opt: Unknown command line argument '-hello-funcs'.  Try: '/Users/steven/Documents/llvm_new/build/Release+Asserts/bin/opt -help'
opt: Did you mean '-mergefunc'?

I'm looking over the docs and clang comes with sample code, which does work:
$ /Users/steven/Documents/llvm_new/build/Release+Asserts/bin/opt -load ~/Documents/llvm_new/build/Release+Asserts/lib/LLVMHello.dylib -hello < inputs/fact.ll > out
Hello: mult
Hello: fact

I'll look into how LLVMHello.dylib is being generated and see if there is a discrepancy.

This seems like a OSX difference here. The Makefile and unit tests are geared towards use on Linux. Please reopen if this isn't the case here.

Okay, I am on board with that and I actually prefer maintaining tools like this on a Linux. However, clang has always been challenging there due to libc++ not existing and needing to be built. See: http://stackoverflow.com/a/25840107
I wonder if you use a less complex approach that also works?

Is it possible to actually use gcc's libstdc++ 4.7 (to build clang) and to use it with clang in c++11 mode?

I reproduced the same exact error on Linux (Centos 7).
GitHub won't allow me to reopen.

Resolved on OS X (11 passing tests!) by applying the changes proposed in #11.


Changed Xianyun's burst stack timing

Updated to reduce at the end of the frame that triggers, instead of the end of the next frame

closing in favor of #2043


adds sales order line item price

As part of #1425

Looks like a great idea :)

not ready due to:

[ ] strange table layout

[ ] pricing table/modal 'calculate price' button not working - looses part information

@SchrodingersGat @eeintech
Any idea where this strange layout can come from? I set the colspan but nothing seems to work.

@matmair Where is that modal view?

@eeintech the template is here

@matmair I imagine you want your elements to align between tables right?
You can edit the  tag with style='width:XX%' to set the column width percentage.
For instance, here is a 3 column table, with no headers:

        {% trans 'Unit Cost' %}
        Min: {% include "price.html" with price=min_unit_buy_price %}
        Max: {% include "price.html" with price=max_unit_buy_price %}

        {% trans 'Total Cost' %}
        Min: {% include "price.html" with price=min_total_buy_price %}
        Max: {% include "price.html" with price=max_total_buy_price %}

            {% trans 'No supplier pricing available' %}

And for the 2 columns header:

        {% trans 'Unit Cost' %}
        Min: {% include "price.html" with price=min_unit_buy_price %}

        {% trans 'Total Cost' %}
        Min: {% include "price.html" with price=min_total_buy_price %}

            {% trans 'No supplier pricing available' %}

Also this might not be the cleanest way to do it in CSS, you may want to create your own class for this table and define the width there. Using the style inside the ` tag overwrites any other CSS styling coming.

@eeintech yes exactly thank you

@SchrodingersGat this closes 2 tasks on #1425, I will tick them after this PR is merged

@matmair looks great, fantastic effort on this :)
I've had a bit of a play around, the only thing that jumps out at me - when you add a part to a sales order which has price breaks associated with it, should it automatically calculate the sale price based on the selected quantity?


(cargo-release) version 0.5.0

Current dependencies on/for this PR:

main

PR #296   👈

PR #297

This comment was auto-generated by Graphite.


Calibrations or Experiments not working with Sampler usage with Qiskit DynamicsBackend

Informations

Qiskit Experiments version: 0.8.0
Python version: 3.10
Operating system: Mac OS Sequoia

What is the current behavior?
Trying to set up calibration experiments or characterization experiments yield an error in the creation of the analysis result, making the whole workflow broken when used with Qiskit Dynamics.
The following is displayed when printing analysis_results():
Options {'execution': {'meas_type': 'kerneled'}} have no effect in local testing mode.
[]
Job data not added for errored job [Job ID: 9ac2115c-19ec-4b76-9b56-e7bd5876a9b6]
Error message: n/a
Steps to reproduce the problem
`
from qiskit.quantum_info import Operator
from qiskit_dynamics import DynamicsBackend, Solver
import numpy as np
from qiskit_experiments.library.calibration import RoughXSXAmplitudeCal
from qiskit_experiments.calibration_management import Calibrations, FixedFrequencyTransmon
Define the backend for the dynamics simulation
nu_z = 10.
nu_x = 1.
nu_d = 9.98  # Almost on resonance with the Hamiltonian's energy levels difference, nu_z
X = Operator.from_label('X')
Y = Operator.from_label('Y')
Z = Operator.from_label('Z')
s_p = 0.5 * (X + 1j * Y)
solver = Solver(
static_hamiltonian=.5 * 2 * np.pi * nu_z * Z,
hamiltonian_operators=[2 * np.pi * nu_x * X],
hamiltonian_channels=["d0"],
channel_carrier_freqs={"d0": nu_x},
dt=0.01,
)
dynamics_backend = DynamicsBackend(solver=solver, subsystem_dims=[2])
Calibrate a X gate
cals = Calibrations(libraries=[FixedFrequencyTransmon(["x", "sx"])])
exp = RoughXSXAmplitudeCal([0], cals, amplitudes=np.linspace(-0.2, 0.2, 21))
exp_data = exp.run(backend=dynamics_backend).block_for_results()
print(exp_data.analysis_results())`
What is the expected behavior?
The analysis result should at the very least not be empty and no error should be generated.
Suggested solutions
Check back how Experiments handles the PUB results from the primitive call and see if something does not break. For now when using backend_run=True, everything works normally.
Also, a change in Qiskit Runtime could be relevant for dealing with options efficiently, as it seems that an execution option for the experiment is dropped down by the call tp the local service

There are a few issues at play here related to getting the measurement level communicated through the stack.

First, for experiments falling back to SamplerV2 from qiskit-ibm-runtime and going through BackendSamplerV2, support for passing options through to backend.run and for processing level 1 data is needed. I added that support to BackendSamplerV2 in https://github.com/Qiskit/qiskit/pull/13357, which will be in Qiskit 1.3 (you can use pip install qiskit==1.3rc1 now).
There is a point in the BackendSamplerV2 data processing here where it checks if the result data memory is truthy. Dynamics returns a numpy array for this which raises an exception on being cast to a boolean. The Qiskit base class specifies list for the type of the memory, so  I think Dynamics should switch to returning a list. We had to add tolist() to data returned by PulseBackend in #1470 as well.
When trying to update Qiskit to 1.3 to take advantage of the updates described in point 1 above, an incompatibility between Dynamics and Qiskit is encountered. I reported that in https://github.com/qiskit-community/qiskit-dynamics/issues/366.
With Qiskit 1.3 and the two fixes to Dynamics, things almost work, but there is still a problem getting options from qiskit_ibm_runtime.SamplerV2 to BackendSamplerV2. I started working on that in https://github.com/Qiskit/qiskit-ibm-runtime/pull/1990, but I still need to finish it. It was written for my original version of https://github.com/Qiskit/qiskit/pull/13357 that passed individual options through and hasn't been updated yet to work with the final version that puts all the backend.run options into a run_options option.

By the way, didn't you make your own SamplerV2 that works with Dynamics? Have you tried passing that to BaseExperiment.run as the sampler argument instead of letting it default to qiskit_ibm_runtime.SamplerV2?

Thanks for this detailed reply @wshanks. I have also tried to pass directly a pre-defined BackendSamplerV2 instance and got the same result, probably because I have not tested with Qiskit 1.3 yet. For your point 2, should I raise this as an issue on Qiskit Dynamics repo? I do not know if @DanPuzzuoli is aware of this issue already.

For point 2, I opened https://github.com/qiskit-community/qiskit-dynamics/pull/367. The CI failed but the failures seemed unrelated to the change. It seems like the Dynamics repo needs some maintenance to run with the latest version of qiskit-ibm-runtime.


About the names

I run the "./darknet detector test cfg/combine9k.data cfg/yolo9000.cfg ../yolo9000-weights/yolo9000.weights data/person.jpg" command.  I can get the detection,but the names is not the same with the README.
for example, i get worker instead of  'goat herder' , horse instead of  shetland pony hunting dog instead of Gorden setter. Is this project updated or something is not right ?

I can't figure out why but names are correct when you compile the CPU version...

Hum yes I found out that the results can be slightly different when running on a CPU and GPU. If you can figure out why, I'd be def interested to know! Maybe a different seed, rounding values...

@hengshanji @moskiteau @philipperemy i met the same issue that when compile with GPU the name it not correct, and when compile with CPU, the result is right.  I wonder How do you fix it?

Hum not sure why. This repo does not contain the code of darknet. It's more a demo to show YOLO with 9000 classes. Refer to: https://pjreddie.com/darknet/yolo/ @xiaoxiongli

@hengshanji @moskiteau @philipperemy @xiaoxiongli i met the same issue, i wonder if any of you had figure it out?

Same Issue.Even I found some images are giving empty result. I have created one issue also regarding  this


Migrate output specification from deprecated rule.outputs parameter

Deprecated. This parameter is deprecated and will be removed soon. Please do not depend on it. It is disabled with --incompatible_no_rule_outputs_param. Use this flag to verify your code is compatible with its imminent removal.

https://docs.bazel.build/versions/master/skylark/lib/globals.html#rule.outputs

Semi-related experimentation https://github.com/bazelbuild/rules_postcss/commit/838cab3515cf4a8cc19a1ad73c3259496a2c02f6 shows how to do the conversion, so it's a matter of taking that and cleaning it up for merge


Use Singleton HttpClient when Unconfigured

Fixes #21134
This PR updates the basic createInstance implementation methods of HttpClientProvider to return a singleton instance instead of creating a new one each time it is called. This could be taken further in the future to use singleton instances for each unique HttpClientOptions used to create an instance.

Good progress. I'm looking forward to getting some perf benchmarks run to understand how this helps (if at all), and also a lot of functional tests run so we can be confident that this works without regressions.

Scenarios where this change bring improvement are where user has to construct clients more often than their application scope. For example when they have multi tenant app where they need to use different credentials OR if their data/workload demands new client per entry.
A good example is the following:

User has CSV with 1M Blob URIs with distinct SAS signatures
they want to process each entry and either retrieve properties or upload/download something.

Today there's mitigation available in a form of user providing http client explicitily and managing it's scope. However, for the purpose of benchmarking we should compare how SDK performs out of the box without such targeted workarounds.

API changes have been detected in com.azure:azure-core-http-jdk-httpclient. You can review API changes here
API changes
-         public JdkHttpClientProvider()
+         public JdkHttpClientProvider()

API changes have been detected in com.azure:azure-core-http-okhttp. You can review API changes here
API changes
-         public OkHttpAsyncClientProvider()
+         public OkHttpAsyncClientProvider()

Closing this PR for merged the inheritance PR:  https://github.com/Azure/azure-sdk-for-java/pull/27982


className removed after styling component

Bug report
Describe the bug
Component with class name, after imported in other component and changed the styling, do not show the classname added before.
To Reproduce
Steps to reproduce the behavior, please provide code snippets or a repository:

Create a simple component and add <FirstComponent className="something" ...etc
Create another component that will use the previous and customise it:

export const StyledFirstComponent = styled(FirstComponent, {
  display: 'flex',
});

Now, in the SecondComponent's tests, if we see in the "StyledFirstComponent", the className added before styling is not there anymore.
Expected behavior
Have the previous className added in the component before extending it in the styling.
Screenshots
If applicable, add screenshots to help explain your problem.
console.log the "FirstComponent", has the icon-Success class:

console.log the "StyledFirstComponent", has not the icon-Success class:

System information

OS: Mac
Browser (if applies):  Chrome
Version of Stitches: [1.2.5]
Version of Node.js: [14.18.1]
Version of NPM: [8.1.1]

Additional context

problem found by<EMAIL_ADDRESS>

Confirmed, and it happens only with server-side rendering

You probably forgot to pass the className prop, can you make a repro on codesandbox.com or repl.it?

@LucasCaixeta I believe that what @caub mentioned is probably the cause of your issue. Could you please create a repro of the issue on codesandbox?

Hi there @hadihallak , thank you for your message.
I moved to another job and I have no access to the code to reproduce the bug.
Should I close it?
Thanks anyways.


Update DOOM 1993.yaml

-Death logic has a 1 in 5 chance to be true
-Difficulty tweaked towards medium, Baby removed
-Goal can be boss levels, unless the slot rolls as episode 1 only
-Levels can be flipped
-Randomized music
-Episodes roll sequentially: Episode 4 can only roll true if 2 and 3 do first, episode 3 can only roll true if 2 does first
-Episode 4 has a low chance of rolling since in my experience most players don't like it
This is my first time using triggers and my first time doing a github PR. Hope I didn't mess anything up

I am Summer Spice on discord

What's the reasoning behind tying the episodes together in the triggers? I think they are fine rolling independently.

It's just how I personally like to play, so I thought maybe others would like it too. To me it feels weird to, say, play episodes 2 and 3 but skip 1, or episodes 1 and 4 but skip over 2 and 3; it's like skipping over a chunk of difficulty


Invalid rpath with macos shared libs

it doesn't find cargs.dylib maybe need to add search path of the @executable_path

Sorry it works well now after cleaning the build folder and rebuild.


[Issue #242] Web: Prunes unnecessary resources and endpoints in the Web Layer and Minor Refactor [PR 2/3]

This PR removes an unnecessary resource class and endpoint SampleResource.java
Removes a test related to this resource and endpoint.
Refactors the SampleResponse class to be used as a generic return object throughout the web layer.

LGTM.


Could you provide the training file?

I encounter a problem when I am trying to implement your work on my own(video shadow segmentation). The Normalized Self-attention block does not work as you do.

UR welcome. Keep in touch with me if you have any progress on it.
I will close this issue temporally.
Best regards,
Ge-Peng.


Fix for NUTCH-2209 : Improved Tokenization for Similarity Scoring plugin (Sujen)

Thanks @sujen1412 will review.

LGTM, minor updates to:

remove author tags
why does lucene have to be added to the main Ivy?
it looks like you did more than simply use Lucene for term vectorizing and for tokenizing. Are you using Lucene to actually store and retrieve the feature vector for similarity?

Thanks @chrismattmann for the feedback.

why does lucene have to be added to the main Ivy?

I tried adding it to the ivy of the plugin but I face a runtime error of NoClassDefFound for the Lucene filters while doing that. And importing the

it looks like you did more than simply use Lucene for term vectorizing and for tokenizing. Are you using Lucene to actually store and retrieve the feature vector for similarity?

I am currently only term vectorizing and tokenizing using Lucene, the storing and retrieval functionality can be used in the future versions of this plugin. I did remove the file from commit, but I guess squashing all created some error. Will update this with the removal of author tags.
Thanks.

looks good to me. @sujen1412 @lewismc

Ack, if all suggestions are taken on board then I am +1 to committing as
this patch is a large improvement over existing code . Sujen and Kim have
been validating this code and it is looking great.
On Friday, February 5, 2016, Chris Mattmann<EMAIL_ADDRESS>wrote:

looks good to me. @sujen1412 https://github.com/sujen1412 @lewismc
https://github.com/lewismc
—
Reply to this email directly or view it on GitHub
https://github.com/apache/nutch/pull/87#issuecomment-180644843.

--
Lewis

@lewismc the license headers and the property in nutch-default.xml have been added, do you have any suggestions for handling the dependency on indexer-elastic for lucene-core differently ?

Nope not at all Sujen, good job. +1

Okay, so i'll go ahead and commit this patch today.
Thank you all for your feedbacks.

@sujen1412 can you commit?

Done. Thanks folks.

I'm going to be very blunt and ask why this was committed in its current state. Now I understand why I had to jump through hoops to upgrade the Elasticsearch dependency for both #118 and NUTCH-2287.
Lucene dependencies should not be in ivy/ivy.xml and I really don't think the scoring-similarity plugin should be relying on the dependencies of the indexer-elastic plugin.
I'd be happy to fix this if someone takes it seriously and merges it soon.

I'm going to be very blunt and ask why this was committed in its current state. Now I understand why I had to jump through hoops to upgrade the Elasticsearch dependency for both #118 and NUTCH-2287.

@naegelejd, the implementation works and was an initial attempt to sync lucene-core (and transitive deps) across the Nutch plugin and core codebase. Lets work on getting #118 in there for the time being. The dependency management issue is nothing new. It is however something which can be improved.

Lucene dependencies should not be in ivy/ivy.xml

I agree

and I really don't think the scoring-similarity plugin should be relying on the dependencies of the indexer-elastic plugin.

Well, this is one way we can sync dependencies. That is the goal however I agree that the direct dependency is not optimal.

I'd be happy to fix this if someone takes it seriously and merges it soon.

Lets revisit your patch for #118 first. We can then come back to this. Thanks for your patience!

OK, gotcha. That makes sense. #118 is the right place to start.


Drug discovery example - DO NOT MERGE

What is the goal of this PR?
This PR adds the drug discovery example to the repo.
What are the changes implemented in this PR?

Adds drug discovery example under drug-discovery directory.
Updates the top-level readme.

Like it a lot, anyway, ex2/3/4 are really close to each other (I understand that you need this to demonstrate the "chain" but maybe we can find a different way?).
Maybe the exercise 4 is also a bit too easy? (and maybe exercise 6 too?)
Except those remarks, still an great usecase!

Following discussion, updated Exercise 04.


Angular 6 CORS issue

After upgrading to Angular 6 I have this issue :
Failed to load https://api.api.ai/v1/query?*** Method undefined is not allowed by Access-Control-Allow-Methods in preflight response.
Indeed the "access-control-request-method" on the OPTION request is undefined :

Have an exactly same problem. In dev build everything works fine. But got the same error when building with --prod flag.

It was working for me before with angular 5.
Recently I migrated to angular 6 and now seeing CORS issue.
Not user if google did change something on the server side or angular6 causing this issue

Did you solve ?

Same problem here

also same here :( ..

same here... any progress on this issue?

same here :(

same here...
any suggestions?

I also need help! :(

you could use proxy to solve this issue:
under root directory of your angular project (in the same level of package.json create new file name it proxy.json add the below content so all your call toward backend will be done through proxy and no CORS issue anymore, by the way using this way for production also is secure than call backend directly :
lets assume you are serving API under /api/v1 on your backend server so :
{     "/api/v1": {         "target": "https://<IP_ADDRESS>:3000",         "secure": "false",         "changeOrigin": true     } }
and in your angular service file you dont need to add full url in order to get or post:
getEmployess() {     return this.http.get('/api/employees')   }

Thks, but it works only for dev not prod environment !

for production you can use nginx to do the proxy function for you, I have already implemented on production and its working fine.

hi but dont work for me un dev, I always have a response OPTIONS, my api is
on Homestead, laravel with passport auth
Nelson Calderón
My Numbers: +506 8789-7230
On Wed, Nov 21, 2018 at 3:11 AM m00zi<EMAIL_ADDRESS>wrote:

for production you can use nginx to do the proxy function for you, I have
already implemented on production and its working fine.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/dialogflow/dialogflow-javascript-client/issues/86#issuecomment-440589958,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AOowzamKlITPWxisKgnuMXqFYX8KCQvWks5uxRipgaJpZM4T_nPZ
.

hi,for me cors issue is solved,but in my case for api i need to call only localhost no need to use port,so in that case for api it is taking localhost:4200 with api path.so that am not getting data from api.please any one have idea.

You can follow this tutorials to solve this issue
http://www.codeyourthought.com/angular/proxy-to-make-http-call-in-angular/

Also facing same issue


Create new IWebSocketFeed when reconnecting

This change is to fix a NullReferenceException which is thrown upon reconnecting.
The reason this exception is thrown is that WebSocket_Closed disposes the WebSocket (webSocketFeed) here. WebSocket.Dispose is setting it's TcpClientSession to null here. As a result, when we call Start and try to Open the web socket again here, we get the exception due to Client being null.
My change modifies our WebSocket class to take a Func<IWebSocketFeed> so that each time we call Start, we can make a new IWebSocketFeed. There is an open issue, kerryjiang/WebSocket4Net#99, about this preventing reuse. Other socket libraries suggest just creating a new client like this PR, which work for me in practice.
It'd be nice if this could be merged, and if a new NuGet package could be published to pick up other changes (like the WebSocketX events that were added).
Thanks!

@zaccharles is correct.  Since the "new WebSocketFeed" was removed from Start and created at the client.  Dispose is not needed on close.    The simplest solution is to remove the "dispose" from close.

I just moved my production code to use this latest...it's a bit broken at the moment, hopefully this weekend it will be back up and able to test.

@BradForsythe If you just remove the Dispose call from Close, you get a different exception:
System.Exception: The socket is connected, you needn't connect again!
   at SuperSocket.ClientEngine.TcpClientSession.Connect(EndPoint remoteEndPoint)
   at WebSocket4Net.WebSocket.Open()
   at GDAXSharp.WebSocket.WebSocketFeed.Open() in G:\<removed>\GDAX.Api.ClientLibrary\GDAXSharp\WebSocket\WebSocketFeed.cs:line 47
   at GDAXSharp.WebSocket.WebSocket.Start(List`1 providedProductTypes, List`1 providedChannelTypes) in G:\<removed>\GDAX.Api.ClientLibrary\GDAXSharp\WebSocket\WebSocket.cs:line 73
   at GDAXSharp.WebSocket.WebSocket.WebSocket_Closed(Object sender, EventArgs e) in G:\<removed>\GDAX.Api.ClientLibrary\GDAXSharp\WebSocket\WebSocket.cs:line 213
   at WebSocket4Net.WebSocket.FireClosed()
   at WebSocket4Net.WebSocket.OnClosed()
   at WebSocket4Net.WebSocket.client_Error(Object sender, ErrorEventArgs e)
   at SuperSocket.ClientEngine.ClientSession.OnError(Exception e)
   at SuperSocket.ClientEngine.AuthenticatedStreamTcpSession.StartRead()
   at SuperSocket.ClientEngine.AuthenticatedStreamTcpSession.OnDataRead(IAsyncResult result)
   at System.Net.LazyAsyncResult.Complete(IntPtr userToken)
   at System.Net.LazyAsyncResult.ProtectedInvokeCallback(Object result, IntPtr userToken)
   at System.Net.Security._SslStream.ProcessFrameBody(Int32 readBytes, Byte[] buffer, Int32 offset, Int32 count, AsyncProtocolRequest asyncRequest)
   at System.Net.Security._SslStream.ReadFrameCallback(AsyncProtocolRequest asyncRequest)
   at System.Net.AsyncProtocolRequest.CompleteRequest(Int32 result)
   at System.Net.FixedSizeReader.CheckCompletionBeforeNextRead(Int32 bytes)
   at System.Net.FixedSizeReader.ReadCallback(IAsyncResult transportResult)
   at System.Net.LazyAsyncResult.Complete(IntPtr userToken)
   at System.Net.ContextAwareResult.CompleteCallback(Object state)
   at System.Threading.ExecutionContext.RunInternal(ExecutionContext executionContext, ContextCallback callback, Object state, Boolean preserveSyncCtx)
   at System.Threading.ExecutionContext.Run(ExecutionContext executionContext, ContextCallback callback, Object state, Boolean preserveSyncCtx)
   at System.Threading.ExecutionContext.Run(ExecutionContext executionContext, ContextCallback callback, Object state)
   at System.Net.ContextAwareResult.Complete(IntPtr userToken)
   at System.Net.LazyAsyncResult.ProtectedInvokeCallback(Object result, IntPtr userToken)
   at System.Net.Sockets.BaseOverlappedAsyncResult.CompletionPortCallback(UInt32 errorCode, UInt32 numBytes, NativeOverlapped* nativeOverlapped)
   at System.Threading._IOCompletionCallback.PerformIOCompletionCallback(UInt32 errorCode, UInt32 numBytes, NativeOverlapped* pOVERLAP)

I haven't looked into WebSocket4Net, but it looks like something isn't being closed properly or recreated when needed. I suppose it would be ideal if the WebSocket could be reused instead of recreated. That doesn't seem to be the case, so for now it's proably best to continue disposing it and making a new one.

thanks @zaccharles!
@BradForsythe let me know what you find before I merge this fix.

@zaccharles  Your method to create each time we open and close is workable.  @dougdellolio I have been able to implement open and closing the socket without recreating.  Kind of using your original approach but without disposing and recreating, and not using the IWebSocket interface.   I am thinking recreating the resources is more time consuming, but maybe not a big deal.    For me ... I have a need to stop and start the socket when I detect errors or my "mini L2 books" need to resync.    This can be done with separate methods to unsubscribe and resubscribe and not have to start and stop sockets.  Even better!
To summarize:
Two options seem to work fine:
A.  @zaccharles approach to pass in the recreate function  or ...
B.   Keep Websocket4Net inside Websocket and allow to open and close without disposing and without recreating.
Either way, new methods would be good to unsubscribe and resubscribe.     Still there are times throughout the day GDAX decided to close your connection, or for whatever reason the subscriptions are not all accepted.   Opening and closing the socket cures this.    In these cases speed is likely not an issue.
Well, time to get something to eat for real now!

going to merge this and release new version. thanks @BradForsythe and @zaccharles!


Cannot remove old rest result

Hi,
I had around 20 days ago a Corona Test and the result was negative and pushed to the App.
When I now start the App the GUI is stuck on the "Datenabruf / "Das Ergebnis wird aktualisiert" and nothing happens anymore (only the circle going). I also cannot add a new Corona Test or remove the old test result.
I guess, that the lab (Berlin) has removed the test result from the server. Now either the server produces a error code, which the app doesn't recognize. Or the App cannot deal with the circumstances that a once negative corona test doesn't exist anymore.
Device: Essential Phone, Android 10. App 1.5.0

Hi @zeus24
There is a posting https://github.com/corona-warn-app/cwa-app-android/issues/1084#issuecomment-714326433 from @svengabr from the Open Source Team that this issue is going to get fixed.

The latest feedback in Jira is from today morning:

[...] is currently preparing a new state with the information that the test is expired and a delete Button to delete the Test. UI Design will be attached soon

Also there is a workaround from @daimpi in https://github.com/corona-warn-app/cwa-app-android/issues/1084#issuecomment-702193741, that suggests resetting the app, which you could try.
Your issue was reported also in #1435 which was closed as a duplicate of #1084, so if these other issues in your opinion cover your question, you're free to close this issue.

Hey @zeus24 ,
Thanks for your feedback. This problem is already discussed in https://github.com/corona-warn-app/cwa-app-android/issues/1084 as @MikeMcC399 already pointed out. The developers are currently working on a fix on the problem, buut I ave currently no information when the fix is going to be integrated. Therefore, please follow the linked thread for updates. I'll close your issue as a duplicate.
Regards,
CH

Corona-Warn-App Open Source Team

Duplicate of #1084


Create a dark-rating-2.html

One need to make this component using HTML and Tailwind CSS. I would suggest to use Tailwind Playgrounds to make things faster and quicker.
Here is a reference to the component.

make it for dark mode
After building the component please raise a PR with a screenshot of the component and add the component in Rating/src/dark-rating-2.html.
If you need to use any icon please use it from Hero Icons
Good luck.

Don't need this right now.


Support CORS

CORS is tricky. One of the issues is that application/json is not supported.
So it would be nice if Refit:

Either supported CORS automatically (by checking response headers, I guess).
Or that we could easily change the mime-type to whatever we want pr request, or for all requests.

application/json is supported in preflighted requests. Meaning you need to send an OPTIONS request first to figure out if it is safe to send the actual request.
However, simple, not preflighted requests, do not support application/json but require the Content-Type to be either of:

application/x-www-form-urlencoded
multipart/form-data
text/plain

Closing for now as idle. Please re-open if needed.


CI?

once we have testing frameworks in place (https://github.com/Plutonomicon/cardano-browser-tx/issues/13), we'll want some CI checks, namely:

no compiler warnings
the package builds and passes tests

let's get Purty checks here as well.
perhaps nixfmt?

ideally each ci job has a makefile script that runs them inside the nix shell.
we should also be sure to set up cachix.

this can use github actions

It seems like the Hercules instance is working fine for the build. We have #57 as well for formatting so I think we can close this


Test coverage

Given this is external facing and a fresh chunk of code, we need to make sure test coverage on this is high
The words "test-driven development" make me shudder a bit, but I think we should define an API for the objects, write some tests and then build to that. At some point, say a 1.0-RC type milestone we finalise the test schema and then no more changes to those. Then in theory, if code changes alter the input/output values from the library we break.
Thoughts? Unless anyone has any serious objections to this, I think it's pretty important.

I had in mind using the tests here to validate the stability of the API, in addition to Selenium.
I would suggest running the tests here prior to an API release etc.


V2

Hi. Since aqtinstall 2 has not been released yet, the command line options are not stable, adapting to it will increase the difficulty of maintenance.


Can't get a connection when using the WAN verison

When I create a backdoor over LAN, everything works as intended; however, when I use the WAN version, I can't get an incoming connection from outside my network.  I'm using a Kali Linux VM in bridged networking mode, and I have forwarded both ports 1025 and 1026 to the host PC.

It was a problem with my own port forwarding settings!


Dymo Duo - 93493 (300 series)

I have a Dymo Duo (the weirdly named one that is really the 300 series and pre-dates the DUO 450 series) And its a great unit.

Probably the cheapest, tape+label model that prints uing all dymo D1 and Industrial tapes
Prints 6mm > 24mm dymo tape + Address labels on large format labels up to 62mm wide.
Has an automatic cutter
Has the joint top address label speed output
Works with 3rd party labels (Not affected by the issues plaguing the 550 series needing official dymo labels)

I'd love to see support for this little unit and saw it had been mentioned in the post #4. But that the conversation had died.
Is this device currently supported by this project or would you like me to gather logs and data to have it potentially added to the supported list?
I dont suppose the project is available via a docker container and utilizing USB passthru is it for ease of installation and testing?

Oh, that's a really beautiful unit! And thanks for the detailed intro! I only have the little LabelManager PnP, so my testing and development capabilities are extremely limited.
I wonder if it would make sense for you to pair up with @thinkl33t on https://github.com/labelle-org/labelle/pull/81?

I dont suppose the project is available via a docker container and utilizing USB passthru is it for ease of installation and testing?

Indeed, we do! It's right here. It's tailored to be a VS Code development container. But even if you're not running VS Code, you should be able to use something similar to these arguments to get it running.
Welcome, and I hope that gets you started!


Match Component heights in Home page to make it look more professional

@calvinhlee23 Does this still need work or can we archive?