[FLINK-30349] Sync missing HBase e2e tests to external repo

hotfix to set correct version number

The e2e tests will need to be re-written to not rely on the legacy E2E infrastructure but use the connector e2e framework instead.

@ferenc-csaky Is this something you could pick up?

The e2e tests will need to be re-written to not rely on the legacy E2E infrastructure but use the connector e2e framework instead.

@ferenc-csaky Is this something you could pick up?

Yep, you can assign the jira to me. Is this blocking the connector release itself or that can happen without finishing this work?

Yep, you can assign the jira to me. Is this blocking the connector release itself or that can happen without finishing this work?

I would consider it a blocker because without the e2e tests we can't fully externalize the code

The e2e tests will need to be re-written to not rely on the legacy E2E infrastructure but use the connector e2e framework instead.

@zentol Do we have any common utils or such that can be used instead of the JUnit4 based ones in the core e2e common module? Or each and every connector should take care about it on their own?

I started to work on this, but I think there are things to discuss:

The HBase (and also Kafka) heavily relies on the custom JUnit4 based resources defined in flink-end-to-end-tests-common (e.g. LocalStandaloneFlinkResource, DownloadCache). On the short term the necessary calsses can be copied over to achieve the e2e test externalization ASAP and at least be able to execute the test, but on the longer term a common module probably would be useful. We can say every connector should take care about these things on their own, but IMO things like running flink, downloading artifacts will be required by multiple connector E2E tests, so eventually they will duplicate it. WDYT?
The HBase e2e test also relies on the flink-sql-client-test JAR, which is currently not deployed to maven. Would that be an option to deploy that as well for future releases so it is not necessary to put thogether the deps by hand?
The one and only HBase E2E test is ignored since 2021 April (according to Jira, that issue should be resolved but we never removed the @Ignore itself), so do we still consider this as a release blocker? It never ran during the whole 1.16 dev. cycle. WDYT?

Maybe it would worth to discuss this in the dev channel in a more general manner, but I felt like posting it here first, cause I phrased everything from the HBase perspective, where these apply.
cc @MartijnVisser @zentol

On the short term the necessary calsses can be copied over to achieve the e2e test externalization ASAP

I'd like to avoid that because realistically a migration just won't happen if we don't force the issue, as it was the case for years.

downloading artifacts will be required by multiple connector E2E tests

By and large we avoid the need for this in newer connector tests by relying on testcontainers.

The HBase e2e test also relies on the flink-sql-client-test JAR, which is currently not deployed to maven. Would that be an option to deploy that as well for future releases so it is not necessary to put thogether the deps by hand?

Does it actually need it though? Why would executing a SQL statement require test utils?

The one and only HBase E2E test is ignored since 2021 April (according to Jira, that issue should be resolved but we never removed the @Ignore itself), so do we still consider this as a release blocker? It never ran during the whole 1.16 dev. cycle. WDYT?

That's a fair point and I'd actually be fine to continue without it (or creating a completely new one).

@ferenc-csaky With @zentol his comments, do you think you could continue working on this?

I had some interrupts, but I'll have time this week, so I plan to have a PR by next week.

Hello! The thing is, my original idea of using the HBaseTestUtility (HBase minicluster) for the E2E tests did not work out, because I could not integrate it with the dockerized Flink env, wiring back the HBase ZK quorum from the host to the docker img. did not work, so I see 2 options at this point:

Create an HBase testcontainers img (there is a Docker img, but no testcontainers intergration)
Write non-dockerized Flink test env. to the flink-connector-test-utils.

IMO 1. makes more sense, so I am currently working on the HBase img. Although, pointing back to my prev. comment, @zentol agreed that missing these E2E tests should not block the connector externalization, cause they were disabled for the whole 1.16 development already. Until the HBase connector itself does not change, it should be okay, which would mean the first external release, I think.

@ferenc-csaky What would be your preference?

@ferenc-csaky What would be your preference?

I already started working on it, creating an on the fly HBase img. with testcontainers and interact with it via the hbase shell, so I reuse the already existing code from the current E2E tests. I had some interrupts, but I plan to open the PR with the containerized solution by the end of this week.

@MartijnVisser @zentol opened #12 with the dockerized E2E tests. The workflow runs currently seems to fail, I'm not sure why.

Subsumed.


Avoid failures when other runners are already registered with same name

Hi, we were running into issues where newly created instances weren't able to register with Github as runners, because by default the hostname is used as --name, and AWS was reusing hostnames from our IP range.
This can be fixed by setting the name explicitly, to something that is unique.
I think --name $(hostname)-$(uuidgen) should work well.
Fixes https://github.com/machulav/ec2-github-runner/issues/128

this is awesome and was also causing us problems.

Awesome fix! I am facing this problem right now on my builds... When this PR will be merged?

We are also running in the same issue, it would be great to see this merged soon


Syncing wallets between Nodes?

Is this possible? It seems that when I create a wallet on Node#1 it never reaches Node#2? The blocks sync fine, but the wallets do not - is this intended?
Cheers my dudes, you've done great work here btw :)

Ok groovy, that makes more sense now :) Thanks!

Sorry to reply after closure, would there be an easy way to switch from Node based private keys to the local device?

You could change the naivecoin to store the private key in a file, instead what it does now.

Yeah I gathered that much... Could you point me in the correct direction?
You basically repeated what I asked.


[KeyVault][Certificates] Align naming to JS/TS guidelines

Certificates needs to be checked to see if it meets the naming guidance from the released guidelines for JS/TS.

It seems fine! Can we close this?

I'll close this. Please feel free to open it again if there's something missing.


Add crypto currency library

I think Bitcoin cash doesn't use the green logo anymore:

what's this one?

I think Bitcoin cash doesn't use the green logo anymore:

what's this one?

I think Bitcoin cash doesn't use the green logo anymore:

https://excalidraw.com/#json=4881929799204864,zU88i9qmD1MMkqifOgLeog (https://excalidraw.com/#json=6402659137355776,Du9RHDB8XXGZqKoPCedSZQ)

Heh, yeah I just grabbed the color off coingecko where it's green for some reason. Updated here: https://github.com/excalidraw/excalidraw-libraries/pull/43/commits/05ebd2323eca0c71853347a83edca269c7b8908d

I think Bitcoin cash doesn't use the green logo anymore:

https://excalidraw.com/#json=4881929799204864,zU88i9qmD1MMkqifOgLeog (https://excalidraw.com/#json=6402659137355776,Du9RHDB8XXGZqKoPCedSZQ)

Heh, yeah I just grabbed the color off coingecko where it's green for some reason. Updated here: https://github.com/excalidraw/excalidraw-libraries/pull/43/commits/05ebd2323eca0c71853347a83edca269c7b8908d

Thanks :)

Thanks :)


Takes more than a week to delete orphaned blobs

Issue
Takes more than a week to delete orphaned blobs
Context
We deploy (and deploy often) a set of large apps via pipelines. This makes the blobstore grow reasonably quickly.
Steps to Reproduce
Create lots of big apps.
Expected result
That a daily (or otherwise) scheduled task would remove orphaned blobs from the blobstore, or that at minimum it be possible for to us to configure (via a BOSH property) a more frequent run of such a task.
Current result
The cleanup schedule task has non-configurable frequency and runs at 1:30am daily.
And - it has been built so that it cleans up a different section of the blobstore depending on the day of the week.
This make it impossible to run a full cleanup (if needed) without waiting up to a week.
https://github.com/cloudfoundry/cloud_controller_ng/blob/1ebab1cbb5270a3d51c0a098a37cd9e8ca0f721d/app/jobs/runtime/orphaned_blobs_cleanup.rb#L126
Possible Fix
I'll see if I can PR something. This current behaviour is unusual and surprising (most particularly the day of week aspect).

btw, the reason that's so surprising, is that one generally separates the schedule for a job, from what the job itself does.
The parent class which actually schedules this job (https://github.com/cloudfoundry/cloud_controller_ng/blob/f330e0aea363d7c1e24f0cd9054966e267b53c31/lib/cloud_controller/clock/scheduler.rb#L16), which is currently on a daily schedule, ought to quite reasonably make decisions like "change this to a weekly job" without needing to inspect every line of code in a job itself.
By making this depend on the day of week, it's unnecessarily tightly coupling itself to whatever needs to schedule it.

Hello!
The job you've found there is designed to clean up blobs that are orphaned, i.e. that exist in the blobstore and that we don't have a record of in our database.  I'm curious which blobstore bucket you're seeing the growth in?
For example, when you push an app, there is a package of app source, and then when you build the app, you get a droplet, and those are stored in separate buckets.  When you delete the app, the associated blobstore resources are also deleted.

The day of the week was the first interesting thing I found here... That said, although I really do think that ought be fixed, I don't think it's the root cause of our issues.
What I found was that on singleton-blobstore that the /var/vcap/store/shared/cc-resources/app_bits_cache appeared to be growing unbounded and was responsible for 84GB of the 95GB used.
All up that directory was using I think 84GB, 57GB of which hadn't been accessed for at least 7 days (find /var/vcap/store/shared/cc-resources/app_bits_cache -type f -atime +7 -print0 | du --files0-from=- -sh --total), 27GB of which had been accessed within 7 days (find /var/vcap/store/shared/cc-resources/app_bits_cache -type f -atime -7 -print0 | du --files0-from=- -sh --total)
I noted that this directory is explicitly excluded by the cleanup job.
For this particular environment, we regularly (multiple times a day) push a set of new Java apps to it, then remove the old ones, and have no other applications present.  As such I was quite happy to simply run find /var/vcap/store/shared/cc-resources/app_bits_cache -type f -atime +7 -delete to remove the older files, and that freed up some immediate space, however it seems like something isn't cleaning up the way it's meant to.

I also meant to say, thanks @selzoc for helping us dig into. Also we are running version v12.45.0 of cf-deployment.

For historical reasons, the blobs in the resources bucket are not automatically cleaned up.  They aren't touched by the OrphanedBlobsCleanup job.  That job in particular is designed to be very conservative in its approach to deleting blobs, which is what you have observed.  However, it's not relevant to your particular issue.
I should have linked this document before: https://docs.cloudfoundry.org/concepts/architecture/cloud-controller.html#manual-clean.  It sounds like you've arrived at basically the same solution.
We don't currently have anything planned to clean the resources bucket.  One tactical thing you could do that we do support is to disable resource matching via a feature flag: http://v3-apidocs.cloudfoundry.org/version/3.83.0/index.html#list-of-feature-flags.  This would make it so nothing gets placed into the resources bucket, but at the cost of having to upload all the files in an app on each cf push.

Thanks @selzoc - so IIUC, the blobstore simply grows unbounded?
Reading the linked doc, is there any good reason to not simply have a daily task that deletes that directory entirely?  If safe, then I'm surprised that someone spent the time to document it, and not the time to write the few lines of code that would do the equivalent as a cron? (perhaps I'm missing something?)
(if not addressable upstream, we'll likely co-locate a daily job that does that on our singleton blobstore, but it sounds like it would be much better handled within cloud_controller_ng so as to be applicable across multiple blobstore providers)

@selzoc - I tried disabling the feature flag as you suggested, however it doesn't seem to stop the files getting stored in the resources bucket. All it seems to do is stop the API reporting that the files already exist, so they are uploaded again, and placed in that same directory.
Tested by disabling the flag, deleting the app_bits_cache, then re-pushing all apps, and noting the app_bits_cache was immediately populated with nearly 2GB of content.

We certainly shouldn't be uploading resources to the cache if that feature flag is not enabled.  I have added a bug to track that here: https://www.pivotaltracker.com/story/show/172707458.  Thanks for the catch!

@sethboyles - unclear how one can draw that conclusion from the commit history and public tracker info alone. I can find references to resource matching back to 2015, but I only see this feature flag to disable from late 2019 - and the only reason given:

As an admin
I want to be able to disable resource matching
So that I can continue to deploy an app when resource matching is causing problems (for instance: a CVE)

I'll admit I don't understand that reason... but it doesn't say anything about blobstore usage being a goal of that "feature flag".
I mean, we'll take this happily enough as a workaround, but it seems fundamentally wrong that an unbounded cache (which is safe to blow away at any time) - is deliberately being written to a persistent disk. That guarantees long-term failure for every installation that uses it, or, for those using a public cloud provider, a guaranteed recurring cost for storage of bytes no longer needed.

Since the issue has disappeared in the "Pivotal Tracker" with no updates, here's the workaounnd we've employed on our cf-deployment  via an operator file:
- type: replace
  path: /instance_groups/name=singleton-blobstore/jobs/-
  value:
    name: pre-start-script
    release: os-conf
    properties:
      script: |
        #!/usr/bin/env bash
        set -euo pipefail

        cat <<EOF > /etc/cron.daily/clear_app_bits_cache
        #!/usr/bin/env bash
        set -euo pipefail
        rm -rf /var/vcap/store/shared/cc-resources/app_bits_cache
        EOF

        chmod a+x /etc/cron.daily/clear_app_bits_cache

        # and run it once now
        /etc/cron.daily/clear_app_bits_cache

With d4d891c, the disabling the resource_matching feature flag will prevent CC from populating the resource cache.

Thanks for the update @Gerg

The fix for this issue is included in capi-release 1.103.0. Thanks for raising this issue!


MFU for other GPUs

extrapolating from spec sheets, yay :)

numbers for the workstation Ampere GPUs currently don't make sense, I get > 100%.

Nice, I manually adjusted for H100 using the datasheet.

Merged to master, made a new file llmc/mfu.h, adjusted the code so that we don't use fancy features so also Windows doesn't break.


Fixes the manage memberships table

Ticket Link
GROUPINGS-1278
done in conjunction with: https://github.com/uhawaii-system-its-ti-iam/uh-groupings-api/pull/388
List of squashed commits

Removes filter for membershipResults
Adds managePersonResults from API
Edits tests for getMembershipResults and makes test for managePersonResults
Adds sanitizer to membershipResults

Test Checklist

[X] Unit Tests Passed:
[X] Jasmine Tests Passed:
[X] General Visual Inspection:

Looks awesome. Can you rename the 'safeInput' variable to 'principalName' or something like that? Also, is it possible to use the safe string for the first argument in makeApiRequest, too?


continuous integration

Set up CI just like in gpkit.
GitHub is currently advertising a lot of CI integrations: https://github.com/integrations/feature/code. @galbramc, any thoughts on whether those are worth looking in to, or is another Jenkins setup our best bet?

your call. should only take a couple of minutes to get things setup with our jenkins, or you can use one that github provides. We have complete control over ours, no idea what they provide.

Another thing to consider is any dependencies you have. I don't know what machines they are providing for the CI testing, or what means you have to install your dependencies.
Let me know what you want me to do. I just need to know how to run your test (which I assume is the same as gpkit), and what dependencies you have for gpfit.

Well, I started looking into TravisCI just so we could see another CI setupo, but quickly learned that "Python builds are not available on the OSX environment."
Therefore, @galbramc if you could help us get set up on jenkins we'd really appreciate it. Dependencies for gpfit are:

numpy
gpkit

I presume you want both pull request and push configurations just like you have with gpkit. I also presume you want me to use the master branch of gpkit as the dependency?

correct, and correct.
probably don't need to test for multiple solvers -- the part of gpkit that gpfit uses doesn't involve solvers.

ok, so do I need to build gpkit, or can I just add the gpkit to the python path?

I'm not sure --
We definitely want to be testing gpfit against a current version of gpkit, such as that installed by pip install gpkit. Would the "add the gpkit to the python path" solution automatically keep gpkit up to date at the resolution of our PyPI releases?

Ok, how about I create a virtualenv were I install gpkit every time we run the tests. That way you always have what you get with pip. Sound good?
Also, could you add from .run_tests import run to init.py in gpfit/tests just for consistency?

sounds great.
Just did so in #32. Do you need that change on master or should I leave that branch open as the first test of jenkins/CI?

Sure leave it open as the first test so I can get the script sorted out

Do you want pylint? If so, you should probably add the pylint.cfg file from gpkit and grab the code from run() in run_tests.py

You also need to give acdl-jenkins the same privlages to gpfit as it has for gpkit before I can test

[x] ok, I gave acdl-jenkins access
[ ] by grab the code in run_tests, I assume you mean the xmloutput stuff. I'll do that now.
[ ] I want pylint and will copy over the config file, but I'm confused / have a question. I don't actually see pylint warnings over at gpkit, even though I know we have lots of them. Can you re-educate us on how to see how we're doing on pylint?

hmmm, still getting this error:
Couldn't create web hook for repository hoburg/gpfit. Does the user (from global configuration) have admin rights to the repository?
Let me check the pylint stuff

nope, has write access, which is the same setting as gpkit. Does it need admin access?

Yeah I think it needs admin access...

Same goes for gpkit, I have seen some complaints about gpkit as well, and this might explain why

Here is what is going on with pylint:
https://acdl.mit.edu/csi/job/gpkit_PullRequest/658/buildnode=reynolds,optimizer=mosek/violations/
You currently have 728 violation, however the jenkins default configuration was to allow 999 violations, so this is why you have not gotten any failed builds from pylint.
I can lower the number, but you probably need to start an initiative to become pylint compliant. Presumably it will be easier to get gpfit compliant as it appears to be smaller at the moment.

We could gradually lower the threshold so you don't have to fix all problems at once

I'm around all day today if you want to get this wrapped up. We should also talk about the pylint configuration

coming over now

add .pylintrc and pylint.sh to the repo and then I think you can close this issue
(don't forget about test/pylint.cfg)


meson-python 0.16.0

meson-python 0.16.0
Destination channel: Defaults
Links

PKG-5347
dev_url:        https://github.com/mesonbuild/meson-python/tree/0.16.0
pypi:           https://pypi.org/project/meson-python/0.16.0
pypi inspector: https://inspector.pypi.io/project/meson-python/0.16.0

Explanation of changes:

bump for 0.16.0

add packaging as a run dependency
remove colorama as a win dependency
change doc_url

tests:

accidentally re-enabled test_wheel.py which works find for most unix platforms
after lots of fiddling it seems the osx builders behave differently during tests

Linter check found the following problems:
ERROR conda.cli.main_run:execute(125): `conda run conda-lint /tmp/abs_a4bg76w2kz/clone` failed. (See above for error)
The following problems have been found:
===== WARNINGS =====

clone/recipe/meta.yaml:28: host_section_needs_exact_pinnings: Linked libraries host should have exact version pinnings.
clone/recipe/meta.yaml:22: host_section_needs_exact_pinnings: Linked libraries host should have exact version pinnings.
===== Final Report: =====
0 Errors and 2 Warnings were found

Linter check found the following problems:
ERROR conda.cli.main_run:execute(125): `conda run conda-lint /tmp/abs_bfesrluauk/clone` failed. (See above for error)
The following problems have been found:
===== WARNINGS =====

clone/recipe/meta.yaml:22: host_section_needs_exact_pinnings: Linked libraries host should have exact version pinnings.
clone/recipe/meta.yaml:28: host_section_needs_exact_pinnings: Linked libraries host should have exact version pinnings.
===== Final Report: =====
0 Errors and 2 Warnings were found


Fix ESLint config + problems in the code

This PR fixes ESLint's configuration, which brought to light issues with the code, which have now been fixed.

(Had to do a double-revert to get the GitHub status checks to run.)


docs: NestJS guide

issue #115

@fahad19  i made few changes can you review it again

@akshayamadhuri: thank you so much for your patience with me! I am only raising questions here so I can learn about NestJS from you as well.

@fahad19 thank you for the reviews i'm too learning on my way

closing since there's no activity for a long time.
please ping if you want this reopened again :)


transfer className property to textarea

or generally transfer all possible props to real textarea component.
import React from "'react";
import Textarea from "'react-textarea-autosize";

React.renderComponent(
  <Textarea onFocus={this.handleFocus} onBlur={this.handleBlur} autofocus></Textarea>,
  document.body
)

I agree. Why closed?

@kun-zhou props are passed through to the underlaying textarea. You can see it here

@Andarist Thank you!


0.6.7 ?

Any new data for the latest OC release  0.6.7 ?

Please update the website to check for 0.6.7. Thank you.

@neemagha, can you not spam us? You statement is exactly the same as the OP. If you feel you need to "+1" on the same request then you can use the "pick your reaction" icon there, which will not trigger an email to all of us like your reply.
Please respect GitHub etiquette like this one. (No one made those rules, but they are just common sense on GitHub.)
Not only you are spamming the maintainer, but also any of the subscribers that want to know the development on this like me.

Is there any update about this?


[Bug]: Handle is called more than once when replaying events that have more than one state

What happened?
When replaying events that have multiple states:
class UserJoinedHousehold extends Event
{
    public function __construct(
        #[StateId(UserState::class)] public int $user_id,
        #[StateId(HouseholdState::class)] public int $household_id,
        public bool $activate_household = true
    ) {
    }
}

When you replay the event, it is replayed for both states causing handle to be called twice.
How to reproduce the bug
Create an event with more than one state.
When replaying the event you will see that the handle method is called once per state.
Package Version
0.1.*
PHP Version
8.3.3
Laravel Version
10.*
Which operating systems does with happen with?
macOS, Windows, Linux
Notes
I assume this is because in the src/Lifecycle/Broker.php we loop through each events states and then call replay.

I looked at this with @skylerkatz a bit today. I think this is just a mistake. Should be a simple fix.


Add libSyntax support for #tfop and @differentiable.

The -verify-syntax-tree flag has been enabled when compiling the stdlib:
https://github.com/apple/swift/commit/9cb383edcee217d2cbd3fdc074db417f89fddec9
-verify-syntax-tree verifies that no unknown nodes exist in the libSyntax
tree. #tfop and @differentiable are used in the TensorFlow module of the
stdlib but did not have libSyntax support, causing verification errors during
compilation.
Todo:

Add libSyntax support for #adjoint and #gradient.

This currently isn't necessary to compile the stdlib because those
expressions aren't used within the stdlib.

Regarding downstreaming changes from master:

The stdlib now compiles successfully. Compilation doesn't fail until lldb, which is not yet in sync (needs to downstream changes also).
Various tests are failing on Linux (it's likely others are failing on Mac):

Failing Tests (10):
    Swift(linux-x86_64) :: IRGen/ordering_x86.sil
    Swift(linux-x86_64) :: Runtime/linux-fatal-backtrace.swift
    Swift(linux-x86_64) :: TensorFlow/crashers.swift
    Swift(linux-x86_64) :: TensorFlow/dataset.swift
    Swift(linux-x86_64) :: TensorFlow/device_placement.swift
    Swift(linux-x86_64) :: TensorFlow/diagnostics.swift
    Swift(linux-x86_64) :: TensorFlow/integration.swift
    Swift(linux-x86_64) :: TensorFlow/no_copy.swift
    Swift(linux-x86_64) :: TensorFlow/sends_recvs.swift
    Swift(linux-x86_64) :: TensorFlow/tensor_autodiff.swift

Closed this PR and created a new one pointing to the tensorflow branch, as requested by @marcrasi.
The new PR is: https://github.com/apple/swift/pull/17428


Refactor Recursion Depth Management with Guard Pattern

Introduced RecursionDepthGuard to manage current_recursion_depth in EntryPointExecutionContext.
Eliminates potential bugs arising from early exit (?) by ensuring each increment has a matching decrement via Drop impl.
Simplifies error-prone manual increment/decrement.
Large change around let result = due to obviating the need for result variable; logic remains unchanged.

This change is

Codecov Report

Merging #1012 (f875f4b) into main (39083f5) will increase coverage by 0.02%.
Report is 12 commits behind head on main.
The diff coverage is 93.93%.

@@            Coverage Diff             @@
##             main    #1012      +/-   ##
==========================================
+ Coverage   66.83%   66.86%   +0.02%
==========================================
  Files          51       51
  Lines        6185     6193       +8
  Branches     6185     6193       +8
==========================================
+ Hits         4134     4141       +7
  Misses       1674     1674
- Partials      377      378       +1

Files
Coverage Δ

crates/blockifier/src/execution/entry_point.rs
94.24% <93.93%> (-0.30%)
:arrow_down:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


%near-rpc agent

"A “sidecar app” that runs a NEAR node alongside an Urbit server and communicates to it through the %lick vane."

@SuperCoolYun Play around with getting a NEAR RPC node running, just to figure out what the process is. I'll get a feel for %lick.


Added connected workspace reference

Recent SR made it clear that the feature disparity for renaming a dedicated SQL pool in a Synapse workspace is not clearly documented.  Renaming cannot occur under a standard workspace (where the workspace is created first and then the dedicated SQL pool is created), but is supported in a connected workspace scenario.

Docs Build status updates of commit 415c774:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/t-sql/statements/alter-database-transact-sql.md
:bulb:Suggestion

Details

docs/t-sql/statements/alter-database-transact-sql.md

Line 1050, Column 173: [Suggestion: docs-link-absolute - See documentation] Absolute link 'https://docs.microsoft.com/azure/synapse-analytics/sql-data-warehouse/workspace-connected-create' will be broken in isolated environments. Replace with a relative link.

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

@WilliamDAssafMSFT - Can you review this PR?
IMPORTANT: When this content is ready to merge, you must add #sign-off in a comment or the approval may get overlooked.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

Docs Build status updates of commit baf8df9:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/t-sql/statements/alter-database-transact-sql.md
:bulb:Suggestion

Details

docs/t-sql/statements/alter-database-transact-sql.md

Line 1048, Column 173: [Suggestion: docs-link-absolute - See documentation] Absolute link 'https://docs.microsoft.com/azure/synapse-analytics/sql-data-warehouse/workspace-connected-create' will be broken in isolated environments. Replace with a relative link.

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 459858b:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/t-sql/statements/alter-database-transact-sql.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit fc6b956:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/t-sql/statements/alter-database-transact-sql.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

#sign-off


feat(deps): bump github.com/NVIDIA/go-nvml

Signed-off-by: Jason Liu<EMAIL_ADDRESS>Ⅰ. Describe what this PR does

Ⅱ. Does this pull request fix one issue?

Ⅲ. Describe how to verify it
Ⅳ. Special notes for reviews
V. Checklist

[ ] I have written necessary docs and comments
[ ] I have added necessary unit tests and integration tests
[ ] All checks passed in make test

/hold

/unhold


to get the replies of a user use docid:<PHONE_NUMBER>391286

I am just quickly going through what I see. Sorry for not doing PRs :D
export async function getReply(userId) {
  const lsd = await getLsd();

  const params = {
    lsd: lsd,
    variables: JSON.stringify({ userID: userId }),
    doc_id:<PHONE_NUMBER>391286,
  };

  const body = new URLSearchParams(params).toString();

  return await fetch("https://www.threads.net/api/graphql", {
    credentials: "omit",
    headers: {
      "Content-Type": "application/x-www-form-urlencoded",
      "X-IG-App-ID": "238260118697367",
      "X-FB-LSD": lsd,
      "Sec-Fetch-Site": "same-origin",
    },
    body: body,
    method: "POST",
  }).then(async (res) => {
    return await res.json();
  });
}

looks good, I'll merge if you make a pr 👍

Works for me. What is not working? Did you pass the userID?

implemented


tests: test the generated .travis.yml for the data model

The generated .travis.yml for a new project needs to be tested and we need to make sure that the tests pass.

So actually travis is not working correctly (it's not testing all the cases), I will check it out.


IMqttServerOptions.ClientMessageQueueInterceptor set?

All the members of IMqttServerOptions have just getters except for ClientMessageQueueInterceptor that also has a setter. This makes pretty annoying to implement the options as an immutable type. Given that the setter is never used, is it possible to remove it?

Hi,
you are right. This is wrong. It was added by accident. I will remove it in the next version.
Best regards
Christian

I released 2.8.5-rc1. Please let me know if version works for you.
Best regards
Christian


PAF approval

Description
The PAF needs approvals from several user roles before it could be used to write a contract. These user roles include: Finance, and VP OR IT Director. This PAF https://test-apply.hypha.app/apply/projects/81/#activity-feed on the test site received approvals from Finance and staff + approver, but has advance to the next stage for contracting.
This PAF is still in the 'awaiting approval' phase

There are "PAF Reviewers Roles" that you configure here: https://test-apply.hypha.app/admin/settings/application_projects/projectsettings/2/
They are different from "user roles" (applicant, staff, reviewers etc.)
The "Spring" projects is not in contracting from what I can see, it is still in "Waiting for approval".
It will not move forward until all "PAF Reviewers Roles" have approved it, just as we decided it should work.

A separate issue is that it might be cumbersome for staff to track what "PAF Reviewers Roles" has approved and who has not. Now they need to look in the "Activity feed" and figure it out. I imagine they will need to "nag" people from time to time?

What roles are included on the platform and workflow?

OTF
User Roles
PAF Reviewer Roles
Invoice

Program Manager
Staff
none
PM

VP Tech
contracting + approver
VP Tech
none

IT Director
contracting + approver
IT Director
none

Finance Director
finance + approver
Finance Director
Finance?

Accountant
finance
none
Finance

General Counsel
contracting
none
none

Attorney
contracting
none
none

CEO
contracting + approver
CEO
none

What roles or approvals are needed during contracting?

Vendor
Program Manager
General Counsel/Contracting User
VP Tech; IT Director; Finance Director

Upload banking details
Fills in PAF
Reviews PAF
Approves PAF (only need two approvals)

What roles or approvals are needed during invoicing?
Vendor | Program Manager| Finance Director/Accountant (Finance user) |
------------ | ------------- | -------------| -------------|
|Upload invoice|Approves or rejects invoice| Approves or rejects invoice|

@fourthletter Note that the "PAF Reviewer Roles" are set up by each organisation however they want. To have none is perfectly ok.
They are not mapped to user groups in any way. It is up to the users to pick the roles that is right for them when they approve PAFs. The honour system, plus that it is logged in the activity feed.


Test PR created from a fork

This PR is just for testing some features when the PR was created from a fork with a topic branch.

fsdf


Uncaught TypeError: Cannot read property 'call' of undefined

Hi ,
I am running my application in two different instances. For load balancing I am mapping a new instance to point the existing two instances. But I am getting the below error.

Thanks in Advance.

Thanks for reporting this issue. However, you didn't provide sufficient information for us to understand and reproduce the problem. Please check out our submission guidelines to understand why we can't act on issues that are lacking important information.
If the problem persists, please file a new issue and ensure you provide all of the required information when filling out the issue template.


allow overloaded query methods

The test failures seem to be a result of https://github.com/hibernate/hibernate-orm/commit/4453681e006671f540bc9a4f6b8a3904f033ef9d, and can't be related to my changes here.


Not working on 192.168.... :8080

Its working on localhost:8080 but when try to access on 192.168.... :8080 it says "you choose not to provide access to camera/microphone. Demo will not work"

Had the same issue. Check your browser's settings.

I think your browser had blocked the camera & microphone because you used the not secure IP. Hope this link can help you https://medium.com/@Carmichaelize/enabling-the-microphone-camera-in-chrome-for-local-unsecure-origins-9c90c3149339


Graph API missing performer information

Category

[ x] Question
[ ] Documentation issue
[ ] Bug
Hi,
We have observed that no information provided about the activity performer in the API response.
For example in case when file is shared and removed by the user other than who created then how could we get this info
In the below response, we even don't know who deleted the file

{<EMAIL_ADDRESS>"#microsoft.graph.driveItem",             "id": "0166T5MOBRHQUSGQXRCRHJZEXTT7XJF5UM",             "cTag": "\"c:{23293C31-F142-4E14-9C92-F39FEE92F68C},4294967295\"",             "size": 0,             "parentReference": {                 "driveId": "b!gl16TrIwKEmJS_q1D0yOw_Wn9CYX0X9LmxyC90FWSC5vV2ltvzsrRIS0McMNLVqM",                 "driveType": "business",                 "id": "0166T5MOF6Y2GOVW7725BZO354PWSELRRZ"             },             "deleted": {                 "state": "deleted"             },             "file": {},             "fileSystemInfo": {}         }     ] }
Is there any ETA about this fix or any alternate way to get the info about the performer?
Thank you.
[ ]: http://aka.ms/onedrive-api-issues
[x]: http://aka.ms/onedrive-api-issues

@shayan-brcm The delta API will combine multiple changes into a singular item.
We would recommend getting the Audit logs to get more granular information about specific changes.


Adds "failOnWarning" tests

Support for "failOnWarning" was added in the 3.6.0 release. This is mostly unit and integration testing for the flag that I still had lying around. Would be a shame to go to waste.

based on the closed PR #60


Update gif showing how to start FreeCodeCamp

To reflect recent UI changes in GitHub (and star count!)
Fixes https://github.com/FreeCodeCamp/FreeCodeCamp/issues/5527

@firstdoit thanks for the PR. Closing it because you've open this PR against our master branch and your branch is named patch-1. Please, review our Guidelines for Contributing very carefully and then open a new PR, thanks. And, please, do not ignore pull request template when you open a PR or an issue.

Sorry, @bugron. Opened up a new one.
BTW, the PR template doesn't actually spells out neither the branching name enforcement, nor that the target branch must be staging. Maybe we should add it there! What do you think?

Actually, there is a mention about the branch name (fix/... or feature/...) but I think we need to mention about staging branch too.

I care to disagree :) The way it's phrased right now makes it sound like a suggestion rather than an enforcement. It would be better to say: "Branches MUST be named with descriptive names that start with either fix/ or feature/ prefixes. Good examples are: ..."

@firstdoit It clearly says:

Name the branch something like fix/xxx or feature/xxx where xxx is a short description of the changes or feature you are attempting to add.

and also says

Submit a pull request from your branch to Free Code Camp's staging branch.

Hi, @AkiraLaine! Yes, the CONTRIBUTING.md indeed covers this.
However, we were discussing the pull request template, which does not:
## FreeCodeCamp Pull Request template
Please, go through steps before you submit a PR.

1. Make sure that your PR is not a duplicate.
2. If not then:

  2.1. Make sure you've done your changes in a separate branch, like `fix/signin-issue` or `feature/issue-templates`.

  2.2. You have a descriptive commit message.

  2.3. You have only one commit (if not, [squash](https://github.com/FreeCodeCamp/FreeCodeCamp/wiki/git-rebase#squashing-multiple-commits-into-one) them into one commit).

  2.4. Make sure `npm test` doesn't throw any error. If it does, fix them first and amend your commit.

3. **After** these steps you're ready to open a pull request. ***Don't*** forget to remove this template first.

  3.1. Give a descriptive title to your PR.

  3.2. Provide a description of your changes.

  3.3. Put `closes #XXXX` in your comment to auto-close the issue that your PR fixes (if such).

I'll propose a PR to improve this template :)

@firstdoit Right. But if you are contributing, you should read the CONTRIBUTING.md.

Yes, indeed. My bad! Please have a look at the PR with a proposal for improving the PR template :)


[analyzer] Handle enabled checker per profile for ClangSA

Currently, the checkers are listed one by one, and the profiles they belong to are listed for each of them. This patch reverses the mapping, and groups the checkers into packages where applicable.

Please add to the commit the information that a checker grouping was done too.
@gamesh411 was the plan to do the conversion without grouping and than a separate PR with the grouping?

There is no consensus on whether this should be 2 commits or one. I will create a two-commit version if that would be more informative.

I have uploaded a 2 commit version. Please make sure to merge them without squashing, so the diff of the grouping can be checked.


Posh.RSS.Feed.get/set_RecentDuration

This should not be on each feed, it should be on the collection of feeds.


Source Map Comments Removed?

Hi,
I'm trying to get sourcemaps available in production for debugging purposes, and while I add
var app = new EmberApp({
  sourcemaps: {
    enabled: true,
    extensions: ['js']
  }
});`

to my Brocfile.js and a local ember build --prod  will have the /# sourceMappingURL=vendor.map comment, it doesn't appear when I deploy to production.
Any thoughts on if ember-cli-rails is suppressing the trailing sourcemap comment at all?

Ugh, seem to be an issue with the rails asset pipeline. If I disable Uglifier it works fine. OK, looks like a rails asset pipeline issue more than an ember-cli-rails issue.


[RFC] Add deprecation error for deprecated fixers

Follow up to https://github.com/FriendsOfPHP/PHP-CS-Fixer/pull/3065. There's only "small" problem:
Remaining deprecation notices (7049)

Fixer "hash_to_slash_comment" is deprecated and will be removed in 3.0. Use "single_line_comment_style" instead: 7049x
(...)

The source for all of them is FixerFactory::registerBuiltInFixers which is used in tests and I haven't figured out how to solve it nicely. I have tried to add parameter $triggerError set by default to true and in tests to false, but that doesn't seem right. Also I have tried to add dummy interface DeprecatedFixerInterface and in 2nd foreach add if for instance of it - better, but still a little bit hacky.
Any suggestions? If not I'd say we could go with the interface solution (with added checks in auto review tests).
Other fixers will be done after the solution is agreed.

please rebase

Rebased, but still no proper solution - I have added QuestionableHelper (it's names says all) to silence deprecation error which is triggered in many, many tests (haven't add it to all as got tired of it).
The problem still stays the same - if we decide to trigger deprecation error in fixer's constructor how to nicely handle it across all tests?

please don't squash :(
no, questionableHelper needs to be reverted (good lack with squashed commits ;p)

I wasn't squashing, but starting all over from master (which I repeated today) as it was different approach and only HashToSlashCommentFixer were staying.

White flag for now.

yeah, this is not easy issue for sure. thanks for the work anyway!


Question about "Finetune" or "Train from Scratch"

I have a question regarding the training strategy for low bitrate models, and I would greatly appreciate your advice.

Is it better to fine-tune a high bitrate model to obtain a low bitrate model, or should I train the low bitrate model from scratch?
If fine-tuning is preferred, how many steps of fine-tuning are recommended to achieve optimal results?

Thank you very much for your help and for the great work you have shared!

The models released in the CompressAI zoo were trained for a high quality model, which was then fine-tuned for lower qualities. (e.g., train Q4, then reuse that to train Q3/Q2/Q1.) Of course, you can also train them separately and obtain similar results, though it takes more training time.
Depends on the model. I would just let the ReduceLROnPlateau scheduler figure it out.


Updated to FontAwesome 4.7

By using a python script , I regenerated the Variables.qml from the font-awesome's official css.

Dear Li3p! Would be nice if you patch README.md, CHANGELOG.md.
There are outdated

Versions Tested
Qt Version   : 5.3
Font Awesome : 4.1.0

Ok , I will patch the document and create a new PR.


Circuit validation tool

Is your feature request related to a use case or problem? Please describe.
When attempting to execute on quantum hardware, circuits may be rejected for a variety of reasons such as the presence of unsupported gates or requiring interactions incompatible with device's connectivity.
Describe the solution you'd like
It is desirable to have a tool that can test or dry-run a circuit to check for all such rejection conditions before sending the circuit for execution on a quantum device.
[optional] Describe alternatives/workarounds you've considered
[optional] Additional context (e.g. screenshots)
What is the urgency from your perspective for this issue? Is it blocking important work?
P2 - we should do it in the next couple of quarters

dupe? https://github.com/quantumlib/Cirq/issues/3960


Add SDK support for FHIRPath Math operations for FHIR Specification R4B

Is your feature request related to a problem? Please describe.
The R4 FHIR spec version supported by the SDK includes an FHIRPath spec version that doesn't have support for Math functions like .exp(), .power(), .abs()... that can be used in FHIRPath expressions for computations such as (%weight/(%height.power(2))).round(1).
Describe the solution you'd like
Support for Math functions should be added by adding implementation of the operations that aren't supported in the FHIRPath Spec.
Describe alternatives you've considered
Some of the alternatives we've considered include:

Upgrading to a much more recent FHIR Spec version that supports FHIRPath Math functions e.g. R5
Delegating of computations that require the math functions to a Service class in the app that can be invoked (probably during post-processing of the QuestionnaireResponse?)

Additional context
See FHIRPath Specification: Publication (Version) History
Would you like to work on the issue?
Yes

This has been addressed in FHIR Engine 0.1.0 beta05 Pre-release
 🙌🏾


Test coverage for databasedump.sh

We need to create a unit test :)

Really…? This is basically just a tiny script you can find in one or the other way in
But yes, if you want, create one. Mock mysqldump (a function in bash/… should do it) and test whatever you want… :smile:
(The functionality/dumping itself of mysqldump likely does not need to be tested.)


[workflow] Update version to 0.9.7

@dengjunplusplus ,could you please take a look:
CMake Error at C:/a/1/s/scripts/buildsystems/vcpkg.cmake:788 (_find_package):
  Could not find a package configuration file provided by "Workflow" with any
  of the following names:

    WorkflowConfig.cmake
    workflow-config.cmake

  Add the installation prefix of "Workflow" to CMAKE_PREFIX_PATH or set
  "Workflow_DIR" to a directory containing one of the above files.  If
  "Workflow" provides a separate development package or SDK, be sure it has
  been installed.
Call Stack (most recent call first):
  src/CMakeLists.txt:35 (find_package)


[Web] Allow sending an image's width and height

Large images currently cause resizing issues in tock-react-kit once they fully load (most apparent being the scrollbar jumping up). Specifying the width and height when sending an image from the server would ensure the corresponding img tags get the right size from the start.

Hello @Fabilin good idea.
As long as it remains optional of course.
How would you implement this:

Specify size in pixels (like <img> attributes), or CSS properties enabling more units like em, %, etc.?
Specify absolute size or max size? What happens if the image is smaller than the configured size?
Specify size by image (programaticaly or in Tock Studio / Stories / Add Media), by bot (Settings / Application Configuration) and/or globally for the whole Tock platform instance (envvar)?

cc @pi-2r @correi-f @elebescond

I had not considered setting the dimensions through CSS, it is true that this would allow for a lot more flexibility, and possibly responsiveness as well.
On the other hand, here are the benefits I can think of from specifying size for the HTML attributes :

Easier validation and typing (kotlin API), only whole numbers would be accepted
No risk of CSS injection
Allows more styling in tock-react-kit (e.g. using clientside options to force all images to be the same width would work as intended and keep the aspect ratio if dimensions are specified through HTML but I believe it would be completely ignored if we specified the size in CSS)

For these reasons I would still recommend setting the HTML width and height, be it only for simplicity's sake.
I believe image size should be (optionally) specified for each image sent by the bot's backend; tock-react-kit already has options to set CSS overrides for all images displayed in the conversation.


text-danger

Challenge http://www.freecodecamp.com/challenges/waypoint-use-spans-for-inline-elements has an issue. Please describe how to reproduce it, and include links to screenshots if possible.

Hello can you please explain what this is?

Here's how you would do this with the "Top 3 things cats hate" element: Top 3 things cats <span class"text-danger">hate
should be class="text-danger"

Ah yes thanks for spotting this! Please be sure to include lots of detail when reporting issues otherwise we cant fix them!


Revert 1 prajwaldp223 patch 1

closes issue #96

@prajwaldp223 Thanks! for your contribution

@prajwaldp223 I have to reverted your PR because it's a duplicate of

#16
Sorry for the miscommunication


use fantomas

Apply fantomas, adjusting default settings for https://github.com/fsprojects/fantomas/issues/1933

Note this found fantomas glitch here: https://github.com/fsprojects/fantomas/issues/1934


[Bug] Telescope mappings not working in frecency

Hi!
In my Telescope config, I set this mapping:
mappings = {
    i = {
        -- other mappings...
        ['<tab>'] = actions.move_selection_next,
        -- other mappings...
    }
}

But in Telescope frecency, the Tab key does not work. Is there something special to configure?

Same as #48.

@delphinus thanks for notifying. Closing this isue then.

Thanks. Now I know many users have trouble with this. I will take more precedence over other problems.

@gbcreation #48 has been fixed in master. Check it!


vips jpegsave logs a warning while sharp handles it as error

Possible bug
Is this a possible bug in a feature of sharp, unrelated to installation?

[ x] Running npm install sharp completes without error.
[x ] Running node -e "require('sharp')" completes without error.

If you cannot confirm both of these, please open an installation issue instead.
Are you using the latest version of sharp?

[ x] I am using the latest version of sharp as reported by npm view sharp dist-tags.latest.

If you cannot confirm this, please upgrade to the latest version and try again before opening an issue.
If you are using another package which depends on a version of sharp that is not the latest, please open an issue against that package instead.
What is the output of running npx envinfo --binaries --system --npmPackages=sharp --npmGlobalPackages=sharp?
System:
OS: macOS 14.4.1
CPU: (12) arm64 Apple M2 Pro
Memory: 80.28 MB / 32.00 GB
Shell: 5.9 - /bin/zsh
Binaries:
Node: 20.11.0 - ~/.nvm/versions/node/v20.11.0/bin/node
Yarn: 1.22.22 - /opt/homebrew/bin/yarn
npm: 10.2.4 - ~/.nvm/versions/node/v20.11.0/bin/npm
pnpm: 8.15.6 - /opt/homebrew/bin/pnpm
npmPackages:
sharp: ^0.33.4 => 0.33.4
Does this problem relate to file caching?
The default behaviour of libvips is to cache input files, which can lead to EBUSY or EPERM errors on Windows.

[ x] Adding sharp.cache(false) does not fix this problem.

Does this problem relate to images appearing to have been rotated by 90 degrees?
Images that contain EXIF Orientation metadata are not auto-oriented. By default, EXIF metadata is removed.

To auto-orient pixel values use the parameter-less rotate() operation.

To retain EXIF Orientation use keepExif().

[x ] Using rotate() or keepExif() does not fix this problem.

What are the steps to reproduce?
I have an obviously somewhat corrupt jpeg, since it throws the error VipsJpeg: Premature end of JPEG file on the .toBuffer() command in my nodejs code.
When I run VIPS_INFO=true vips jpegsave corrupt.jpg test.jpg --Q=80 on command line, I also get the message, but as a warning.
What is the expected behaviour?
I would expect the same log level in sharp as in vips jpegsave command. If it were the same level, the problem would not occur, since we set the option failOn to "error".
Please provide a minimal, standalone code sample, without other dependencies, that demonstrates this problem
————————————————————————————————————————————————
Output console vips jpegsave
————————————————————————————————————————————————
Command
——————
VIPS_INFO=true vips jpegsave corrupt.jpg test.jpg --Q=80
Log
——————
VIPS-INFO: 15:14:01.647: found /opt/homebrew/Cellar/vips/8.15.2_1/lib/vips-modules-8.15
VIPS-INFO: 15:14:01.649: using configure-time prefix
VIPS-INFO: 15:14:01.649: VIPS_PREFIX = /opt/homebrew/Cellar/vips/8.15.2_1
VIPS-INFO: 15:14:01.649: VIPS_LIBDIR = /opt/homebrew/Cellar/vips/8.15.2_1/lib
VIPS-INFO: 15:14:01.649: prefix = /opt/homebrew/Cellar/vips/8.15.2_1
VIPS-INFO: 15:14:01.649: libdir = /opt/homebrew/Cellar/vips/8.15.2_1/lib
VIPS-INFO: 15:14:01.652: searching "/opt/homebrew/Cellar/vips/8.15.2_1/lib/vips-modules-8.15"
VIPS-INFO: 15:14:01.652: loading "/opt/homebrew/Cellar/vips/8.15.2_1/lib/vips-modules-8.15/vips-heif.dylib"
VIPS-INFO: 15:14:01.665: loading "/opt/homebrew/Cellar/vips/8.15.2_1/lib/vips-modules-8.15/vips-magick.dylib"
VIPS-INFO: 15:14:01.670: loading "/opt/homebrew/Cellar/vips/8.15.2_1/lib/vips-modules-8.15/vips-poppler.dylib"
VIPS-INFO: 15:14:01.686: loading "/opt/homebrew/Cellar/vips/8.15.2_1/lib/vips-modules-8.15/vips-openslide.dylib"
VIPS-INFO: 15:14:01.693: loading "/opt/homebrew/Cellar/vips/8.15.2_1/lib/vips-modules-8.15/vips-jxl.dylib"
VIPS-INFO: 15:14:01.698: searching "/opt/homebrew/Cellar/vips/8.15.2_1/lib/vips-plugins-8.15"
VIPS-INFO: 15:14:01.700: im_load_plugins: searching "/opt/homebrew/Cellar/vips/8.15.2_1/lib/vips-8.15"
VIPS-INFO: 15:14:01.701: im_load_plugins: searching "/opt/homebrew/Cellar/vips/8.15.2_1/lib"
VIPS-INFO: 15:14:01.717: threadpool completed with 4 workers
(vips:1172): VIPS-WARNING **: 15:14:01.717: read gave 1 warnings
(vips:1172): VIPS-WARNING **: 15:14:01.717: VipsJpeg: Premature end of JPEG file
————————————————————————————————————————————————
Sharp
————————————————————————————————————————————————
Code
———
const image = sharp(fileBufferToUse, { failOn: 'error' });
image.normalise();
image.jpeg({ quality: 80, mozjpeg: true, optimizeScans: true });
image.rotate();
image.toBuffer((err, buffer, info) => {
console.log('info', info);
console.log('error', err);
return buffer;
});
Error
———
Error: VipsJpeg: Premature end of input file
VipsJpeg: Premature end of input file
at Sharp.toBuffer (/Users/aaa/Dev/Repos/convenience-platform/node/document-storage-service/node_modules/.pnpm/sharp@0.33.2/node_modules/sharp/lib/output.js:161:17)
Please provide sample image(s) that help explain this problem
I cannot provide the image, because the data is to sensitive.

The vips command line tool defaults to a "fail on" setting of "none", which is the least sensitive value. The default for sharp is "warning", which is the most sensitive value.
https://sharp.pixelplumbing.com/api-constructor

...one of (in order of sensitivity, least to most): 'none', 'truncated', 'error', 'warning'. Higher levels imply lower levels. Invalid metadata will always abort.

I hope this information helped. Please feel free to re-open with more details if further assistance is required.


Suggestion: Add default User-Agent/Accept header when send request

It will be great if we have default user-agent header, then we can mark the request source.
And I would expect (with request version 2.85.1):
User-Agent: request/2.85.1
Accept: */*

Are there plans to implement a default User-Agent header?

While it might seem like a good feature, that would break a bunch of other implementations. It’s generally a bad idea to automatically add headers—doing so would hurt a lot more people than it would help.


Parallel priority queue can not be compiled by Visual Studio

Microsoft compilers on Windows has OpenMP suport but do not have GNU Parallel headers that are required by parallel_priority_queue (if STXXL_PARALLEL)
https://github.com/stxxl/stxxl/blob/master/include/stxxl/bits/containers/parallel_priority_queue.h#L29
so the compilation of this header (and tests too) is blocked now.
It seems some workaround or fallback is needed...

Another MSVC problem blocking PPQ is unsigned variables in OpenMP parallel for loops:
https://github.com/stxxl/stxxl/blob/master/include/stxxl/bits/containers/parallel_priority_queue.h#L2399
see
http://stackoverflow.com/questions/2820621/why-arent-unsigned-openmp-index-variables-allowed

And _GLIBCXX_MOVE is also not available.
Maybe it is better to disable parallel_priority_queue and its tests on Windows if no one can fix and check them.

I have made rough fixes in my fork https://github.com/alex85k/stxxl/commits/master , the tests compile and start running.
Most likely they contain some errors. Hope someone interested in Visual Studio compatibility will make appropriate changes.

The PPQ should also work on Visual Studio, unless they "skipped" some part of the OpenMP support. The tests compiled on my machine, but didnt run correctly.

MS are so evil that they decided not to support OpenMP 3.0 at all (2014 info)
http://visualstudio.uservoice.com/forums/121579-visual-studio/suggestions/2276847-support-openmp-3-0-or-3-1
Main problem with OpenMP compatibility is that VS compiler  does not allow unsigned variables in
#pragma omp for
(and 3 of the test cpps contain such loops). Dumb switching to int64 etc. (like I tried in my fork) can cause many warnings and test failures...

I see good progress in https://github.com/thk1/stxxl . That branch compiles on VS2013, VS2013 Novermber CTP and VS2015 RC . Tests on Debug mostly passes (few timeout, failed test_vector_resize needs further checking).


Split the install playbooks for sequential execution

This is needed for required components to wait for previous component to complete, otherwise the rhcos node may go into emergency shell and a reboot is needed to retry the ignition process.
Fixes #170
Signed-off-by: Yussuf Shaikh<EMAIL_ADDRESS>

/cc @Prajyot-Parab @cs-zhang

@yussufsh: PR is not mergeable.

The PR state is: BLOCKED
Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@yussufsh: PR is not mergeable.

The PR state is: BLOCKED
Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@Rajalakshmi-Girish can you please shutdown all the prow instances with your personal key if you are running somewhere..


Is it possible, I could use WPStarter with PHPUnit for WordPress plugin / theme tests?

Note: I am just getting started with PHPUnit integration with my plugin(s). So not well versed.
But as I am already familiar with WPStarter, and I see @gmazzap is involved with: https://github.com/Brain-WP/BrainMonkey so I am assuming you must have utilized this combination?
Any implementation I could see, to get ideas / learn from?

I'm not sure about the qustion.
If you need to do unit tests, then you don't need WordPress, and you can use Brain Monkey indeed.
Here: https://inpsyde.com/en/php-unit-tests-without-wordpress/ there's a (2 parts) blog post about doing that.
If you need integration tests, so you need a full WP installation, you could use WP Starter, but not sure that's the best thing to do.
You can look here: https://github.com/inpsyde/Wonolog/tree/2.x/tests/integration for example of integration tests which are automated via docker-compose.
See:

https://github.com/inpsyde/Wonolog/tree/2.x/.github/workflows/docker
https://github.com/inpsyde/Wonolog/blob/2.x/.github/workflows/integration-tests.yml

There no WP Starter is used because it is easier to make use of ready-made containers


Remove bash command after server

If the server fails to start, then the container should just fail. Right now the container hangs if the minecraft server receives the stop command or fails to start.

@bouk Hi, thank you for the PR, I'll take a look soon


ggcb.txt

Govt Graduate College Burewala

@imrankhanbrw Regrettably, this request has signs of fraud. I have to close it without merging.
It does not mean anything actually bad. It just means that there are factors preventing from considering this domain fully trusted right now. Please apply for free educational license with an official document which proves that you are a student. You can do it using "OFFICIAL DOCUMENT" tab at https://www.jetbrains.com/shop/eform/students.
Thank you for your understanding!


[Event Request] Codeunit 5055 CustVendBank-Update - OnAfterUpdateVendor

Hi, would it be possible to modify the event OnAfterUpdateVendor in codeunit 5055, procedure UpdateVendor, like this :
    procedure UpdateVendor(var Cont: Record Contact; var ContBusRel: Record "Contact Business Relation")
    var
        VATRegistrationLogMgt: Codeunit "VAT Registration Log Mgt.";
        VATRegNo: Text[20];
    begin
        with Vend do begin
            Get(ContBusRel."No.");
            NoSeries := "No. Series";
            PurchaserCode := "Purchaser Code";
            VATRegNo := "VAT Registration No.";
            TransferFields(Cont);
            "No." := ContBusRel."No.";
            "No. Series" := NoSeries;
            "Purchaser Code" := PurchaserCode;
            "Last Modified Date Time" := CurrentDateTime;
            "Last Date Modified" := Today;
            OnAfterUpdateVendor(Vend, Cont,ContBusRel);
            Modify;
            if ("VAT Registration No." <> '') and ("VAT Registration No." <> VATRegNo) then
                VATRegistrationLogMgt.LogVendor(Vend);
        end;
    end;

    [IntegrationEvent(false, false)]
    local procedure OnAfterUpdateVendor(var Vendor: Record Vendor; Contact: Record Contact ; ContBusRel : Record "Contact Business Relation")
    begin
    end;

?
Thanks

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


Best way to measure inference time

Hi Guys,
i am kinda new to mmdetection and i am wondering what is the best way to measure the inference time of a model?.
More specific, what script do i need to use to get the inference time from models like on this page
https://mmdetection.readthedocs.io/en/latest/tutorials/pytorch2onnx.html#results-and-models

For pytorch model, you can use tools/analysis_tools/benchmark.py
For ONNX model, you can try https://github.com/open-mmlab/mmdeploy. MMDeploy provides a script to measure the inference time with multi-backend support.

When i use tools/analysis_tools/benchmark.py i get the follow Error:
i know you can specify the reank with ----local_rank if you start the script.
Can somebody please help me?
anaconda3/envs/open-mmlab/lib/python3.7/os.py", line 682, in getitem
raise KeyError(key) from None
KeyError: 'RANK'
Kind regards

Please follow the document https://mmdetection.readthedocs.io/en/latest/useful_tools.html#benchmark


Help request: Dockerfile for ponzu

I've some experience with Docker, what do you need it for?
Many projects tend to define a dockerfile for development and other one for production, I like this approach.

@Sirikon - I think the ability to just quickly run a golang container that installs the latest version of ponzu would be nice to have.
For people already using Docker, it seems that this is a common way for them to try out a new project.
I am not opposed to a multi-file approach if it is something people are used to, but I'll have to look for one to see how its used. Do you know of an example off hand?

If you have a Dockerfile for development and other one for production, the only difference is that one of them will include the SDKs, runtimes, compilers, or whatever is needed to actually build the project.
The production Dockerfile will just have the minimum dependencies and an already builded project ready to run.
For example, a Golang project development Dockerfile should include the Golang tools. The development one just the compiled project (the executable) and asset files if needed.

@Sirikon - that makes sense!

I'm familiar with Docker but not very familiar with GO. The docker file would probably start from Docker's official GO-Lang image, which is based on Debian.
I believe, the only pre-compiling that could happen in the public docker image would be the CLI, an individual project still needs to run from source with the full build tools, plus things like ssh/ git / curl etc.
The VOLUME feature in Docker, allows a host to persist data between deploys. It maps an outside folder into the container (think symlink).
It would be easier actually to move all persistent resources into a new folder, similar to rails a shared folder. Such as:

project/shared/db/ - where the 2 db files persist
project/shared/uploads/ - the admin uploaded files
is there a log file somewhere? for some reason i don't see it /var/log/ponzu-server.log on my local.

There are actually some un docker-like things about the current ponzu setup. DBs usually come from external containers, and their host/open port is routed into the container.  None the less, this project can be Dockerized.

To create the public image, we do need to figure out somewhere else to put DB files. Otherwise they will be overwritten on each docker deploy. I'm less concerned about the log, its OK if each deploy starts a new log file.
I was wrong in my above comment. For docker to handle the persistence between deploys, they actually need to move completely out of the project directory. Such as /var/ponzu/projectName/

Hi @krismeister -
Thanks for submitting this! I am taking a look through the Dockerfile and will test it out when I have a minute to get my Docker env back in order.
Regarding the file location for data, it is possible for Docker to enable a "host" file to be read/written from inside a container, right?
You're just suggesting that we need to pick a reliable path in the host FS to keep the data and uploads?
These foreign paths will be "resolved" within the container to the local path that Ponzu assumes when starting up though, is that right?
For example, even though the file is at /var/ponzu/$PROJ/system.db, your Ponzu binary will still be able to open ./system.db as if it were in the same directory. If not, we will need to update the CLI to take a different data path.
Although not merged into master yet, there will also be a search directory which will hold the search indexes for the content types. This would need to persist between deploys as well.
Sorry for all the questions, just want to make sure I know how I can help.

When I mentioned choosing a path, i meant a path in the docker container. I have to test if we can symlink these files into place. It might be do-able to have the .dockerignore file not move these files over from a developers local project folder.
To get back to only distributing the CLI executable. If we only distribute the CLI, can a user run all of their project code, including the GO source files, without any other system dependancies?

I see. Well, the CLI is a wrapper around the go tool (compilier, package download & installer, etc) and git - which I believe is a dependency of the go tool. So I think the go tooling would be the only requirement.

Since an individual project needs the full GO build tools, its not helpful to only distribute the Ponzu executable. However an individual development team can after ponzu build copy their executable into a minified Docker Container.

Alright, thanks for the explanation - I definitely need to brush up on my docker skills.
So once I create the Ponzu image and push it to DockerHub, I will update the links where needed in your PR. At that point it should be ready?

Once you get the docker image onto dockerhub, the end result will look like:
https://hub.docker.com/r/krismeister/ponzu/
And it is being used here:
https://github.com/ponzu-cms/examples/blob/master/docker/admin/Dockerfile#L1
I made that container by following their auto build instructions, pointing to my github ponzu dev branch.
Their autobuild instructions do make it easy, but you have to get it merged into a github repo somewhere. so you could merge right now to dev, and do the Dockerhub auto build instructions, Or you can clone my krismeister/ponzu dev branch and make a new /ponzu-cms/ponzu branch called docker do the autobuild from that.
Learning docker is a good idea, as alot of the sexy ways to integrate a restCMS can be aided greatly with other off the shelf docker containers.

I'm currently facing this issue.  My db changes get rewritten with each new docker deploy.  Can all the files be kept in a folder? so instead of ./system.db, we use ./shared/system.db so i can simply mount a persistent colum at ./shared ? or make it possible to pass the shared directory as a cli flag

@tonyalaribe - can you share the pwd of your Ponzu project to the container? It wouldn't be a major change to add a CLI flag for the db location  (maybe a good first PR? 🙂), but personally I have found keeping all the db files within the same directory as the binary to be really useful, especially in deployment.
I do think offering a CLI flag to customize their location would be a good feature, though keep in mind there are 2 db files - system.db and analytics.db which (I think) each would need their own path flag.

These files are symlinked in the ponzu docker example here: https://github.com/ponzu-cms/examples/blob/master/docker/admin/start_admin.sh#L10.
@tonyalaribe  That example repo should solve the persistence issue you face.
When I last touched the docker setup, I felt there was a good amount of files in /shared/ which needed to come directly from the repo and shouldn't be overwritten on a machine by machine basis. This was part of the long discussion above.


Implement roaring_bitmap_internal_validate

When deserializing a bitmap, it is possible that the result might be invalid. This could happen because there was data corruption. The deserialization could still generate a bitmap without failure, but the result could be otherwise unusable.
You can avoid such problems by hashing your saved data (e.g., md5sum). But we could could also directly, at some expense, validate the deserialized data.
The C version of Roaring has an interesting function that can be called after deserializing a bitmap, to make sure it is proper:
https://github.com/RoaringBitmap/CRoaring/blob/a103d3811702b9389c538881c9974e9a7a7552af/src/roaring.c#L435
     roaring_bitmap_t *t = roaring_bitmap_portable_deserialize_safe(serializedbytes, expectedsize);
     if(t == NULL) { return EXIT_FAILURE; }
     const char *reason = NULL;
     if (!roaring_bitmap_internal_validate(t, &reason)) {
         return EXIT_FAILURE;
     }

It is not very difficult to implement and could help users who have production data.

I have written the validation according to C version of Roaring bitmaps. Some checks were removed, e.g. it is impossible to have negative capacity as it is derived from array length, which is always positive.
One TODO left - is it somehow possible to have run container with nbrruns equal to zero?
Do you have some suggestion how to create broken serialized bitmap to make some tests?

Important work!

is it somehow possible to have run container with nbrruns equal to zero

It should not be. We don’t include empty containers.

@xtonik If you are still working on this - You can look at what I did in the recent go release.
https://github.com/RoaringBitmap/roaring/blob/master/roaring_test.go#L2681
You'll need to understand the serialization protocol, but then it's just a matter of corrupting a single byte.

I think that this issue remains to be resolved. We never got a pull request. :-(


introduce debug-container image

Based on https://github.com/app-sre/diag-container which I aim to archive when this gets merged
Changes from diag-container:

added packages: htop, procps, strace
removed packages: pcp
added fancy prompt to easily tell apart from other shells
removed run.sh entrypoint
removed urldecode.sh
removed openshift templates, these are not used anywhere that I could find & diag-container should not be run as a permanent deployment anyway

That's the feedback I was looking for!

Thank you 🙇

TBH, I often use this openshift template. It's very handy to start a diag pod with all postgres and redis env vars set. BTW the diag-container is also referenced in our SOPs

+1 to that


during preprocess  how to save frames without faces？

Hi，this is a great job. I try to use my own dataset to reconstruct the speech.The dataset are videos including medical images of vocal organs without human faces.Can you tell me how to save these frames without faces? Thanks a lot!

Hi, you would need a different preprocessing script, where you specify in each frame which part of the image to save as a "crop". In our evaluation script, for example, we save the face region given by the face detector as the crop for that frame.

Thanks for your reply!Now I can get the medical images list but it seems can't run in training. I am now looking for the cause.
I notice the chem images are about 120180 and mine are 580360.Do I need to adjust the size of my images before training?Besides,my videos are 60 fps not 30 fps (3~5 seconds each),do I need to modify related parameters to match my data?

Thanks for your reply!Now I can get the medical images list but it seems can't run in training. I am now looking for the cause.
I notice the chem images are about 120 x 180 and mine are 580 x 360.Do I need to adjust the size of my images before training?Besides,my videos are 60 fps not 30 fps (3~5 seconds each),do I need to modify related parameters to match my data?


Wrong kernel when running a Jupyter Notebook!

Issue Type: Bug
I have a virtual environment, and my workspace (containing multiple folders) is set to run in that environment (first image).
However, when I open a Jupyter notebook, although it shows that kernel initially (second image), when I run a cell, it switches to another kernel and demands installing ipykernel (third image). If I attempt to change the kernel, my intended kernel is not in the list (last image)! Sometimes, opening a new notebook and running something in it fixes the issue with this specific file, sometimes not.

Extension version: 2021.8.1013163132
VS Code version: Code 1.58.2 (c3f126316369cd610563c75b1b1725e0679adfb3, 2021-07-14T22:10:15.214Z)
OS version: Windows_NT x64 10.0.19042
Restricted Mode: No

This is the output for Jupyter when I open the file:
Info 2021-07-17 08:42:40: Loading web panel for file:///c%3A/Users/user/codebase/W-D/read_results.ipynb
Info 2021-07-17 08:42:40: Loading webview. View is notset
Info 2021-07-17 08:42:40: Attempting to start a server because of preload conditions ...
Info 2021-07-17 08:42:40: Loading web view...
Info 2021-07-17 08:42:40: Webview panel created.
Info 2021-07-17 08:42:40: initialize CommonMessageCoordinator
Info 2021-07-17 08:42:40: IPyWidgetMessageDispatcher.initialize
Info 2021-07-17 08:42:40: Created and initailized CommonMessageCoordinator
Info 2021-07-17 08:42:40: Creating raw notebook for file:///c%3A/Users/user/codebase/W-D/read_results.ipynb
Info 2021-07-17 08:42:40: Getting preferred kernel for file:///c%3A/Users/user/codebase/W-D/read_results.ipynb
Info 2021-07-17 08:42:40: Find preferred kernel for file:///c%3A/Users/user/codebase/W-D/read_results.ipynb with metadata {"language_info":{"codemirror_mode":{"name":"ipython","version":3},"file_extension":".py","mimetype":"text/x-python","name":"python","nbconvert_exporter":"python","pygments_lexer":"ipython3","version":"3.8.10"},"orig_nbformat":2,"kernelspec":{"name":"python3","display_name":"Python 3.8.10 64-bit ('.reil38': venv)"},"metadata":{"interpreter":{"hash":"e04279c8b8433d6bb4ad601b2105c69be664ea37f15dda9270e309e901031365"}},"interpreter":{"hash":"c3416ed0848a595f05e9086dffe759fd59e2a6671fda1539db684c903a99c9d9"}} & preferred interpreter {"sysPrefix":"c:\Users\user\.reil38","envType":"Venv","envName":".reil38","envPath":"c:\Users\user\.reil38","path":"c:\Users\user\.reil38\Scripts\python.exe","architecture":3,"sysVersion":"3.8.10 (tags/v3.8.10:3d8993a, May  3 2021, 11:48:03) [MSC v.1928 64 bit (AMD64)]","version":{"raw":"3.8.10","major":3,"minor":8,"patch":10,"build":[],"prerelease":["final","0"]},"displayName":"Python 3.8.10 64-bit ('.reil38': venv)"}
Info 2021-07-17 08:42:40: findPreferredKernel score for Python3, Python 3.8.10 64-bit (windows store) is 4
Info 2021-07-17 08:42:40: findPreferredKernel score for Python 3.8.10 64-bit (windows store) is 4
Info 2021-07-17 08:42:40: findKernel found Python 3.8.10 64-bit (windows store)
Info 2021-07-17 08:42:40: Find kernel spec, Class name = w, completed in 3ms, has a truthy return value, Arg 1: Uri:c:\Users\user\codebase\W-D\read_results.ipynb, Arg 2: {"language_info":{"codemirror_mode":{"name":"ipython","version":3},"file_extension":".py","mimetype":"text/x-python","name":"python","nbconvert_exporter":"python","pygments_lexer":"ipython3","version":"3.8.10"},"orig_nbformat":2,"kernelspec":{"name":"python3","display_name":"Python 3.8.10 64-bit ('.reil38': venv)"},"metadata":{"interpreter":{"hash":"e04279c8b8433d6bb4ad601b2105c69be664ea37f15dda9270e309e901031365"}},"interpreter":{"hash":"c3416ed0848a595f05e9086dffe759fd59e2a6671fda1539db684c903a99c9d9"}}, Arg 3: undefined, Return Value: {"kind":"startUsingPythonInterpreter","kernelSpec":{"specFile":"C:\Users\user\AppData\Roaming\jupyter\kernels\python3810jvsc74a57bd0c781b2277432b50a1e8eec5cb3e0f73bf5c2f898200d2c4b939b2a58c155c1ef\kernel.json","interpreterPath":"C:\Users\user\AppData\Local\Microsoft\WindowsApps\python3.8.exe","name":"python3810jvsc74a57bd0c781b2277432b50a1e8eec5cb3e0f73bf5c2f898200d2c4b939b2a58c155c1ef","argv":["python","-m","ipykernel_launcher","-f","{connection_file}"],"language":"python","path":"python","display_name":"Python 3.8.10 64-bit (windows store)","metadata":{"interpreter":{"sysPrefix":"C:\Program Files\WindowsApps\PythonSoftwareFoundation.Python.3.8_3.8.2800.0_x64__qbz5n2kfra8p0","envType":"WindowsStore","envName":"","envPath":"","path":"C:\Users\user\AppData\Local\Microsoft\WindowsApps\python3.8.exe","architecture":3,"sysVersion":"3.8.10 (tags/v3.8.10:3d8993a, May  3 2021, 11:48:03) [MSC v.1928 64 bit (AMD64)]","version":{"raw":"3.8.10","major":3,"minor":8,"patch":10,"build":[],"prerelease":["final","0"]},"companyDisplayName":"Microsoft","displayName":"Python 3.8.10 64-bit (windows store)"}},"env":{}},"interpreter":{"sysPrefix":"C:\Program Files\WindowsApps\PythonSoftwareFoundation.Python.3.8_3.8.2800.0_x64__qbz5n2kfra8p0","envType":"WindowsStore","envName":"","envPath":"","path":"C:\Users\user\AppData\Local\Microsoft\WindowsApps\python3.8.exe","architecture":3,"sysVersion":"3.8.10 (tags/v3.8.10:3d8993a, May  3 2021, 11:48:03) [MSC v.1928 64 bit (AMD64)]","version":{"raw":"3.8.10","major":3,"minor":8,"patch":10,"build":[],"prerelease":["final","0"]},"companyDisplayName":"Microsoft","displayName":"Python 3.8.10 64-bit (windows store)"},"id":".jvsc74a57bd0c781b2277432b50a1e8eec5cb3e0f73bf5c2f898200d2c4b939b2a58c155c1ef.C:\Users\user\AppData\Local\Microsoft\WindowsApps\python3.8.exe.C:\Users\user\AppData\Local\Microsoft\WindowsApps\python3.8.exe"}
Info 2021-07-17 08:42:40: Computing working directory file:///c%3A/Users/user/codebase/W-D/read_results.ipynb
Info 2021-07-17 08:42:40: Connecting to raw session for file:///c%3A/Users/user/codebase/W-D/read_results.ipynb with connection {"kind":"startUsingPythonInterpreter","kernelSpec":{"specFile":"C:\Users\user\AppData\Roaming\jupyter\kernels\python3810jvsc74a57bd0c781b2277432b50a1e8eec5cb3e0f73bf5c2f898200d2c4b939b2a58c155c1ef\kernel.json","interpreterPath":"C:\Users\user\AppData\Local\Microsoft\WindowsApps\python3.8.exe","name":"python3810jvsc74a57bd0c781b2277432b50a1e8eec5cb3e0f73bf5c2f898200d2c4b939b2a58c155c1ef","argv":["python","-m","ipykernel_launcher","-f","{connection_file}"],"language":"python","path":"python","display_name":"Python 3.8.10 64-bit (windows store)","metadata":{"interpreter":{"sysPrefix":"C:\Program Files\WindowsApps\PythonSoftwareFoundation.Python.3.8_3.8.2800.0_x64__qbz5n2kfra8p0","envType":"WindowsStore","envName":"","envPath":"","path":"C:\Users\user\AppData\Local\Microsoft\WindowsApps\python3.8.exe","architecture":3,"sysVersion":"3.8.10 (tags/v3.8.10:3d8993a, May  3 2021, 11:48:03) [MSC v.1928 64 bit (AMD64)]","version":{"raw":"3.8.10","major":3,"minor":8,"patch":10,"build":[],"prerelease":["final","0"]},"companyDisplayName":"Microsoft","displayName":"Python 3.8.10 64-bit (windows store)"}},"env":{}},"interpreter":{"sysPrefix":"C:\Program Files\WindowsApps\PythonSoftwareFoundation.Python.3.8_3.8.2800.0_x64__qbz5n2kfra8p0","envType":"WindowsStore","envName":"","envPath":"","path":"C:\Users\user\AppData\Local\Microsoft\WindowsApps\python3.8.exe","architecture":3,"sysVersion":"3.8.10 (tags/v3.8.10:3d8993a, May  3 2021, 11:48:03) [MSC v.1928 64 bit (AMD64)]","version":{"raw":"3.8.10","major":3,"minor":8,"patch":10,"build":[],"prerelease":["final","0"]},"companyDisplayName":"Microsoft","displayName":"Python 3.8.10 64-bit (windows store)"},"id":".jvsc74a57bd0c781b2277432b50a1e8eec5cb3e0f73bf5c2f898200d2c4b939b2a58c155c1ef.C:\Users\user\AppData\Local\Microsoft\WindowsApps\python3.8.exe.C:\Users\user\AppData\Local\Microsoft\WindowsApps\python3.8.exe"}
Info 2021-07-17 08:42:40: Starting raw kernel Python 3.8.10 64-bit (windows store)
Info 2021-07-17 08:42:40: installMissingDependencies C:\Users\user\AppData\Local\Microsoft\WindowsApps\python3.8.exe
Info 2021-07-17 08:42:40: Cached data exists getEnvironmentVariables,
Info 2021-07-17 08:42:40: Process Execution: > ~\AppData\Local\Microsoft\WindowsApps\python3.8.exe -c "import ipykernel"

~\AppData\Local\Microsoft\WindowsApps\python3.8.exe -c "import ipykernel"
Info 2021-07-17 08:42:40: Process Execution: > ~\AppData\Local\Microsoft\WindowsApps\python3.8.exe -c "import ipykernel"
~\AppData\Local\Microsoft\WindowsApps\python3.8.exe -c "import ipykernel"
Error 2021-07-17 08:42:40: Install Missing Dependencies, Class name = w, completed in 428ms, has a falsy return value, Return Value: undefined [a [Error]: IPyKernel not installed into interpreter Python 3.8.10 64-bit (windows store):C:\Users\user\AppData\Local\Microsoft\WindowsApps\python3.8.exe
at w.handleKernelDependencyResponse (c:\Users\user.vscode\extensions\ms-toolsai.jupyter-2021.8.1013163132\out\client\extension.js:90:246461)
at w.installMissingDependencies (c:\Users\user.vscode\extensions\ms-toolsai.jupyter-2021.8.1013163132\out\client\extension.js:90:245644)
at processTicksAndRejections (internal/process/task_queues.js:93:5)] {
category: 'noipykernel',
reason: 1
}
Error 2021-07-17 08:42:40: Raw session failed to start because dependencies not installed
Info 2021-07-17 08:42:40: Shutdown session -- complete
Error 2021-07-17 08:42:40: Failed to determine if a notebook is active for the current editor [a [Error]: IPyKernel not installed into interpreter Python 3.8.10 64-bit (windows store):C:\Users\user\AppData\Local\Microsoft\WindowsApps\python3.8.exe
at w.handleKernelDependencyResponse (c:\Users\user.vscode\extensions\ms-toolsai.jupyter-2021.8.1013163132\out\client\extension.js:90:246461)
at w.installMissingDependencies (c:\Users\user.vscode\extensions\ms-toolsai.jupyter-2021.8.1013163132\out\client\extension.js:90:245644)
at processTicksAndRejections (internal/process/task_queues.js:93:5)] {
category: 'noipykernel',
reason: 1
}
Info 2021-07-17 08:42:41: Web view react rendered
Info 2021-07-17 08:42:41: Request for onigasm file at c:\Users\user.vscode\extensions\ms-toolsai.jupyter-2021.8.1013163132\out\client\node_modules\onigasm\lib\onigasm.wasm
Info 2021-07-17 08:42:41: Request for tmlanguage file.
Info 2021-07-17 08:42:41: Searching for token colors ...
Info 2021-07-17 08:42:41: Attempting search for colors ...
Info 2021-07-17 08:42:41: Searching for token colors ...
Info 2021-07-17 08:42:41: Attempting search for colors ...
Info 2021-07-17 08:42:41: Loading colors from c:\Users\user\AppData\Local\Programs\Microsoft VS Code\resources\app\extensions\theme-defaults\themes\dark_plus.json ...
Info 2021-07-17 08:42:41: Loading colors from c:\Users\user\AppData\Local\Programs\Microsoft VS Code\resources\app\extensions\theme-defaults\themes\dark_plus.json ...
Info 2021-07-17 08:42:41: IPyWidgetMessageDispatcher.initialize
Info 2021-07-17 08:42:41: Searching for token colors ...
Info 2021-07-17 08:42:41: Attempting search for colors ...
Info 2021-07-17 08:42:41: Searching for token colors ...
Info 2021-07-17 08:42:41: Attempting search for colors ...
Info 2021-07-17 08:42:41: Searching for token colors ...
Info 2021-07-17 08:42:41: Attempting search for colors ...
Info 2021-07-17 08:42:41: Searching for token colors ...
Info 2021-07-17 08:42:41: Attempting search for colors ...
Info 2021-07-17 08:42:41: Loading colors from c:\Users\user\AppData\Local\Programs\Microsoft VS Code\resources\app\extensions\theme-defaults\themes\dark_plus.json ...
Info 2021-07-17 08:42:41: Searching for token colors ...
Info 2021-07-17 08:42:41: Attempting search for colors ...
Info 2021-07-17 08:42:41: Loading colors from c:\Users\user\AppData\Local\Programs\Microsoft VS Code\resources\app\extensions\theme-defaults\themes\dark_plus.json ...
Info 2021-07-17 08:42:41: Searching for token colors ...
Info 2021-07-17 08:42:41: Attempting search for colors ...
Info 2021-07-17 08:42:41: Loading colors from c:\Users\user\AppData\Local\Programs\Microsoft VS Code\resources\app\extensions\theme-defaults\themes\dark_plus.json ...
Info 2021-07-17 08:42:41: Loading colors from c:\Users\user\AppData\Local\Programs\Microsoft VS Code\resources\app\extensions\theme-defaults\themes\dark_plus.json ...
Info 2021-07-17 08:42:41: Loading colors from c:\Users\user\AppData\Local\Programs\Microsoft VS Code\resources\app\extensions\theme-defaults\themes\dark_plus.json ...
Info 2021-07-17 08:42:41: Loading colors from c:\Users\user\AppData\Local\Programs\Microsoft VS Code\resources\app\extensions\theme-defaults\themes\dark_plus.json ...
Info 2021-07-17 08:42:41: Attempting search for colors ...
Info 2021-07-17 08:42:41: Attempting search for colors ...
Info 2021-07-17 08:42:41: Loading base colors from c:\Users\user\AppData\Local\Programs\Microsoft VS Code\resources\app\extensions\theme-defaults\themes\dark_plus.json ...
Info 2021-07-17 08:42:41: Loading base colors from c:\Users\user\AppData\Local\Programs\Microsoft VS Code\resources\app\extensions\theme-defaults\themes\dark_plus.json ...
Info 2021-07-17 08:42:41: Attempting search for colors ...
Info 2021-07-17 08:42:41: Attempting search for colors ...
Info 2021-07-17 08:42:41: Attempting search for colors ...
Info 2021-07-17 08:42:41: Attempting search for colors ...
Info 2021-07-17 08:42:41: Attempting search for colors ...
Info 2021-07-17 08:42:41: Attempting search for colors ...
Info 2021-07-17 08:42:41: Loading base colors from c:\Users\user\AppData\Local\Programs\Microsoft VS Code\resources\app\extensions\theme-defaults\themes\dark_plus.json ...
Info 2021-07-17 08:42:41: Loading base colors from c:\Users\user\AppData\Local\Programs\Microsoft VS Code\resources\app\extensions\theme-defaults\themes\dark_plus.json ...
Info 2021-07-17 08:42:41: Loading base colors from c:\Users\user\AppData\Local\Programs\Microsoft VS Code\resources\app\extensions\theme-defaults\themes\dark_plus.json ...
Info 2021-07-17 08:42:41: Loading base colors from c:\Users\user\AppData\Local\Programs\Microsoft VS Code\resources\app\extensions\theme-defaults\themes\dark_plus.json ...
Info 2021-07-17 08:42:41: Loading base colors from c:\Users\user\AppData\Local\Programs\Microsoft VS Code\resources\app\extensions\theme-defaults\themes\dark_plus.json ...
Info 2021-07-17 08:42:41: Loading base colors from c:\Users\user\AppData\Local\Programs\Microsoft VS Code\resources\app\extensions\theme-defaults\themes\dark_plus.json ...
Info 2021-07-17 08:42:41: Using colors to generate CSS ...
Info 2021-07-17 08:42:41: Using colors to generate CSS ...
Info 2021-07-17 08:42:41: Using colors to generate CSS ...
Info 2021-07-17 08:42:41: Using colors to generate CSS ...
Info 2021-07-17 08:42:41: Using colors to generate CSS ...
Info 2021-07-17 08:42:41: Using colors to generate CSS ...
Info 2021-07-17 08:42:41: Using colors to generate CSS ...
Info 2021-07-17 08:42:41: Using colors to generate CSS ...

I compared Jupyter's log when it opens a blank notebook and my own notebook, and removed all metadata that were different. Still, findPreferredKernel for blank notebook can find my environment's Python, and it cannot find it for the saved notebook.
Log for the saved file:
Info 2021-07-17 09:22:17: Find preferred kernel for file:///c%3A/Users/user/codebase/W-D/read_results_2%20-%20Copy.ipynb with metadata {"language_info":{"codemirror_mode":{"name":"ipython","version":3},"file_extension":".py","mimetype":"text/x-python","name":"python","nbconvert_exporter":"python","pygments_lexer":"ipython3","version":3},"orig_nbformat":4} & preferred interpreter {"sysPrefix":"c:\Users\user\.reil38","envType":"Venv","envName":".reil38","envPath":"c:\Users\user\.reil38","path":"c:\Users\user\.reil38\Scripts\python.exe","architecture":3,"sysVersion":"3.8.10 (tags/v3.8.10:3d8993a, May  3 2021, 11:48:03) [MSC v.1928 64 bit (AMD64)]","version":{"raw":"3.8.10","major":3,"minor":8,"patch":10,"build":[],"prerelease":["final","0"]},"displayName":"Python 3.8.10 64-bit ('.reil38': venv)"}
Info 2021-07-17 09:22:17: findPreferredKernel score for Python3, Python 3.8.10 64-bit (windows store) is 4
Info 2021-07-17 09:22:17: findPreferredKernel score for Python 3.8.10 64-bit (windows store) is 4
Info 2021-07-17 09:22:17: findKernel found Python 3.8.10 64-bit (windows store)
Log for the blank notebook:
Info 2021-07-17 09:22:24: Find preferred kernel for untitled:/c%3A/Users/user/codebase/Untitled-7.ipynb with metadata {"language_info":{"codemirror_mode":{"name":"ipython","version":3},"file_extension":".py","mimetype":"text/x-python","name":"python","nbconvert_exporter":"python","pygments_lexer":"ipython3","version":3},"orig_nbformat":4} & preferred interpreter {"sysPrefix":"c:\Users\user\.reil38","envType":"Venv","envName":".reil38","envPath":"c:\Users\user\.reil38","path":"c:\Users\user\.reil38\Scripts\python.exe","architecture":3,"sysVersion":"3.8.10 (tags/v3.8.10:3d8993a, May  3 2021, 11:48:03) [MSC v.1928 64 bit (AMD64)]","version":{"raw":"3.8.10","major":3,"minor":8,"patch":10,"build":[],"prerelease":["final","0"]},"displayName":"Python 3.8.10 64-bit ('.reil38': venv)"}
Info 2021-07-17 09:22:24: findPreferredKernel score for Python3, Python 3.8.10 64-bit ('.reil38': venv) is 14
Info 2021-07-17 09:22:24: findPreferredKernel score for Python 3.8.10 64-bit ('.reil38': venv) is 24
Info 2021-07-17 09:22:24: findPreferredKernel score for Python3, Python 3.8.10 64-bit (windows store) is 4
Info 2021-07-17 09:22:24: findPreferredKernel score for Python 3.8.10 64-bit (windows store) is 4
Info 2021-07-17 09:22:24: findKernel found Python 3.8.10 64-bit ('.reil38': venv)

Thanks for filling this issue. I believe it may be fixed in the insiders version. Please could you install vscode insiders, and let us know if it fixes your issue.

I reinstalled Jupyter extension and it fixed the issue. I can no longer replicate the conditions of the issue to check the insider version.


Only one match

I am trying to use this for keypoint feature matching so I know that each query can only be matched to one entry in the Index, is there any way to enforce this behavior?

Hi @bfialkoff, I think so. That sound like k=1

I don't think so, k=1 means that itll give me only the top nearest vector in the Index.
I'm trying to query with N vectors, such that if one of my vectors has been matched to vector i in the index, no other query vector will be mapped to vector i in the index.

Oh, I see. I do not think there is currently such a functionality without serious C++ code modification as queries are now processed independently from each other.


Distribute over IPFS

It'd be useful to have releases on IPFS, so that we don't need go get for bootstrapping gx.

We'll get this "for free" once/if gx ends up on distributions.

in a similar vein, would be really cool to have a gx package hash for each thing on distributions. I've started working on a gx-bin for using gx to install binaries, would be awesome to tie everything together.


UTF-8 characters are broken with parse method

Because native string methods are used, the following doesn't work:
echo Chalk::parse(
  '{✗} failed step 1',
  [ Color::RED ]
);

Chalk displays the first 8 bits of the surrogate pair in red which is an invalid character. To fix, make mbstring a requirement and use mbstring alternatives to substr, strlen and substr_replace.
Programs that care about pretty colored output are likely to care about fancy-pants UTF-8 characters 😄

First of all I couldn't reproduce the problem it all worked dandy for me, but you have a good point there. To handle UTF-8 characters there's quite a few things to consider and I don't think that depending on mbstring is the right thing for this library at this time. If I ever decide to develop a 2.0 I will definitely take this into consideration.
Thanks a million for the feedback!


Use PathEscape on search query instead of QueryEscape

Fixes #963.

Hmmm... Go 1.7 does not have url.PathEscape apparently.
Shall we stop supporting Go 1.7 ?

@willnorris, @dmitshur, @juliaferraioli - I'm considering removing support for Go 1.7 for this repo due to it missing url.PathEscape...
Are there any objections?

Thank you, @juliaferraioli!
Merging.


added few configurable options

Hi Alex!
I've added a few more configurable options to the component. Please check them out and see if  you think it might be worth adding them to the main  branch.
Also it seems that VsCode auto format messed with the formatting/indenting you had before, if that's a problem please let me know and i will change it back.

exposed a swipeThreshold property which will help decide when the page is switched

exposed a swipeRatio property which will reflect percentage of the viewport the current page has been swiped

exposed a transitionTime property which lets you decide the transition time between pages (if it's not set it will use the calculations you initially added for the decision)

exposed a swipingDirection property. will let other components know which direction the pages are being swiped.

Hi @daniel-vlad ,
Sorry for the slow response! Which version of Polymer were you using to develop these updates? I'm getting a whole heap of errors in my console when testing this locally. Will have a deeper look into it asap but just wondering if there could be a quick fix here.
Cheers for the PR!

Hi @alex-saunders,
No worries :). I didn't mess with the polymer version it's still  "polymer": "Polymer/polymer#^2.0.0-rc.2", however it seems that in my hurry to change stuff i forgot to adapt the demo to the changes so that's probably why you're getting all those errors (sorry about that).
I will make the changes as soon as i get some time and will let you know when it's done.

@daniel-vlad any updates on your PR?

@shawnd  haven't had a chance to look at it been a crazy busy month. I will adapt it to take in the latest pull request from @alejost848 when  i fix the demo

@daniel-vlad @alex-saunders Any updates on this PR? Is there anything I can do to help?

@alex-saunders, @alejost848 : sorry, haven't had a chance to get to it in a while. Mostly just need to fix the demo if I remember right, I will take a look at it tomorrow evening and give you guys a proper update

Slight update: looks like the problem is bigger than just the demo pages, i'm looking into it now

Ok, finally got the time to take a proper look and pushed some more changes. Again, i appologize for the delay