Error in acquiring ApplicationAccessToken

Describe the bug
I need to read a Log Analytics workspace located in a subscription outside of microsoft.com
at java.lang.Thread.run(Thread.java:748)
Caused by: java.util.concurrent.ExecutionException: com.microsoft.aad.adal4j.AuthenticationException: {"error_description":"AADSTS700016: Application with identifier 'xxxxxxx-0d79-41f2-b362-5c6408f7854d' was not found in the directory 'microsoft.com'. This can happen if the application has not been installed by the administrator of the tenant or consented to by any user in the tenant. You may have sent your authentication request to the wrong tenant.\r\nTrace ID: 66dbbf3d-2885-47fa-910a-5e591c916100\r\nCorrelation ID: af9f1363-64e4-4cb2-8d31-fd91fff3ab62\r\nTimestamp: 2021-03-12 19:24:22Z","error":"unauthorized_client","error_uri":"https://login.microsoftonline.com/error?code=700016"}
To Reproduce
Steps to reproduce the behavior:
val conf: Map[String, String] = Map(
KustoSourceOptions.KEY_VAULT_URI -> keyVaultUri,
KustoSourceOptions.KEY_VAULT_APP_ID -> keyVaultAppId,
KustoSourceOptions.KEY_VAULT_APP_KEY -> keyVaultAppKey
)
val query = table
val dfResult = spark.read.kusto(cluster, database, query, conf)
Expected behavior
I supposed I can pass in authorityId but it is only available in SINK:
I need this in READ
.option(KustoSinkOptions.KUSTO_AAD_AUTHORITY_ID, "AAD Authority Id") // "microsoft.com"
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

Databricks
Runtime 7.3

Additional context
Add any other context about the problem here.

Turns out the constant can also be used in read..
https://github.com/Azure/azure-kusto-spark/blob/master/samples/src/main/scala/KustoConnectorDemo.scala#L111


An option to delete document is not available

Hi there,
I don't see an option to delete the documents uploaded on the portal. Would you be able to add an option to delete from Document Viewer? Please advise.
I can see an option to delete from Index Management though, but it is not working.
Thanks,
Arun

Hi! Is it feasible to have a session based chat where the documents should be deleted automatically once the session is over.
Thanks,
Arun

Hi there,
I don't see an option to delete the documents uploaded on the portal. Would you be able to add an option to delete from Document Viewer? Please advise. I can see an option to delete from Index Management though, but it is not working.
Thanks, Arun

You can delete the files directly from Azure Blob Storage. The option in the Index Management deletes the embeddings related to that filename not the file itself.

Hi! Is it feasible to have a session based chat where the documents should be deleted automatically once the session is over.
Thanks, Arun

Yes, you can create such behavior by combining the functionality already available in the code base.

Hi there,
I don't see an option to delete the documents uploaded on the portal. Would you be able to add an option to delete from Document Viewer? Please advise. I can see an option to delete from Index Management though, but it is not working.
Thanks, Arun

You can delete the files directly from Azure Blob Storage. The option in the Index Management deletes the embeddings related to that filename not the file itself.
###############

That's right. However, anyone using the web link can view the documents from Document Viewer. Is it possible to hide/remove the documents from portal itself? (instead of deleting from Blob Storage). It would be super if you can implement that. Pls let me know. Thanks.


Update vsftpd.conf

Changed the default vsftpd log file location to be in the 'logs' docker volume

@simfishing  - This repo seems a bit dead. Want to make it to my clone? I will merge that in.


Temporarily pin nightly version used for builds

In the most recent nightly build of cargo a regression slipped in around
handling of READMEs in published crates [1]. Unfortunately this breaks CI
builds because a 3rd level dependency (pyo3->inventory->ctor) triggers
this regression. The issue has been corrected in cargo and the next
nightly should fix this. However, to unblock things in the meantime this
commit pins the nightly version to a known working version. We can
simply revert this after the fix has been released.
[1] https://github.com/rust-lang/cargo/issues/8351

Coverage remained the same at 82.664% when pulling 1e6d4f99f400801f65ca924c64e4f39d2a071fe1 on mtreinish:pin-rust into 5f53f3d30220b030dfbfb65af892672a54f60bfa on Qiskit:master.


Adding coding examples for Contact and ContactOption - v2 with signed commits

Adding coding examples for Contact and ContactOption - v2 with signed commits
:recycle: Current situation & Problem
When I was creating a contact for our application, I was unable to see coding examples. These added examples hope to clarify how someone might build and implement these features.
All of these changes were copied over from my previous pull request for signed commits.
:gear: Release Notes
N/A
:books: Documentation

Improve documentation by showing an example of how to build new Contact as well as custom ContactOptions embedded within larger body of code

:pencil: Code of Conduct & Contributing Guidelines
By submitting creating this pull request, you agree to follow our Code of Conduct and Contributing Guidelines:

[x] I agree to follow the Code of Conduct and Contributing Guidelines.

@philippzagar: Here is my updated pull request for the other module! I am unable to request a review from you, but if you would be able to review it that would be great! I have copied over all of the changes that we discussed in the previous PR. You hadn't reviewed those, so that PR is still open so you can see how I changed the code according to your responses. Thank you so much!

@philippzagar I wasn't too sure what you meant by combining the examples - if this is not what you wanted please feel free to let me know!

@philippzagar Thank you so much for the detailed example! I went ahead and made those changes :)


Update Compute Diagnostic RP swagger doc API version to 2021-06-01-preview

ARM (Control Plane) API Specification Update Pull Request

[!TIP]
Overwhelmed by all this guidance? See the Getting help section at the bottom of this PR description.

[!NOTE]
As of January 2024 there is no PR assignee. This is expected. See https://aka.ms/azsdk/pr-arm-review.

PR review workflow diagram
Please understand this diagram before proceeding. It explains how to get your PR approved & merged.

Click here to see the details of Step 1, Breaking Changes review
If you are in purview of Step 1 of the diagram, follow the Breaking Changes review process.
IMPORTANT! This applies even if you believe your PR was mislabeled, for any reason, including tool failure.

Click here to see the details of Step 2, ARM review
See https://aka.ms/azsdk/pr-arm-review.

 Click here to see the diagram footnotes
Diagram footnotes
[1] See ARM review queue (for PR merge queues, see [2]).
[2] public repo merge queue, private repo merge queue (for ARM review queue, [1])
The ARM reviewer on-call engineer visits the merge queue twice a day, so the approximate ETA for merges is 12 - 24 hours.

Purpose of this PR
What's the purpose of this PR? Check the specific option that applies. This is mandatory!

[ ] New resource provider.
[ ] New API version for an existing resource provider. (If API spec is not defined in TypeSpec, the PR should have been generated using OpenAPI Hub).
[ ] Update existing version for a new feature. (This is applicable only when you are revising a private preview API version.)
[ ] Update existing version to fix OpenAPI spec quality issues in S360.
[X] Other, please clarify:

This PR is to update existing Compute Diagnostic RP APIs to point to an older API version 2021-06-01-preview. There are no code differences between the two API versions, and all affected APIs function completely the same between the two API versions.
The reason for this "downgrade" is because the new API version ARM manifest is still being rolled out by CRP team, but is taking longer than expected due to unforseen blockers, and this delayed rollout is blocking our targeted release for Az PS and CLI related PRs. https://msazure.visualstudio.com/One/_release?definitionId=18535&view=mine&_a=releases
Once the Compute Manifest has completed the rollout, we will "upgrade" the API version back to the latest API version.

Due diligence checklist
To merge this PR, you must go through the following checklist and confirm you understood
and followed the instructions by checking all the boxes:

[X] I confirm this PR is modifying Azure Resource Manager (ARM) related specifications, and not data plane related specifications.
[X] I have reviewed following Resource Provider guidelines, including
ARM resource provider contract and
REST guidelines (estimated time: 4 hours).
I understand this is required before I can proceed to the diagram Step 2, "ARM API changes review", for this PR.

Additional information

 Viewing API changes
For convenient view of the API changes made by this PR, refer to the URLs provided in the table
in the Generated ApiView comment added to this PR. You can use ApiView to show API versions diff.

Suppressing failures
If one or multiple validation error/warning suppression(s) is detected in your PR, please follow the
suppressions guide to get approval.

Getting help

First, please carefully read through this PR description, from top to bottom. Please fill out the Purpose of this PR and Due diligence checklist.
To understand what you must do next to merge this PR, see the Next Steps to Merge comment. It will appear within few minutes of submitting this PR and will continue to be up-to-date with current PR state.
For guidance on fixing this PR CI check failures, see the hyperlinks provided in given failure
and https://aka.ms/ci-fix.
For help with PR workflow diagram Step 2 (ARM review), see https://aka.ms/azsdk/pr-arm-review.
If the PR CI checks appear to be stuck in queued state, please add a comment with contents /azp run.
This should result in a new comment denoting a PR validation pipeline has started and the checks should be updated after few minutes.
If the help provided by the previous points is not enough, post to https://aka.ms/azsdk/support/specreview-channel and link to this PR.

The PR is failing LintDiff and model validation checks because the API-version does not match the package version, should I request a temporary suppression for this PR?


Issue with azure benchmarking tool on passing input tokens greater than 30k

We are trying to run our benchmarking exercise using the benchmarking tool for gpt4o but getting below error on passing the input context greater than 30k from the command line
Command:
python -m benchmark.bench load --temperature 0.0 --shape-profile custom --deployment 'deployment name' --max-tokens 200 --context-tokens 35000 --api-version 2024-02-01 --rate 10 --duration 600 https://genai-stg-westus3-1.openai.azure.com/
Error:
getting error as "There aren't enough words to choose from. Cannot generate 15991 word(s)

Hey @rajm3180 , this repo is no longer maintained and it is suggested to use the improved fork here: https://aka.ms/ImprovedAOAIBenchmarkingTool.
The fork has already fixed this issue and it is actively maintained, with a large number of fixes and improvements compared to this repo. Make sure to read the README to understand all of the updates that exist.


List OCP clusters in EC2

Signed-off-by: Andrej Podhradsky<EMAIL_ADDRESS>

Thanks for the review! I will apply your suggestions everywhere it is possible.
I need to double check the usage of key regex in TagFilters and let you know.

@apodhrad Ping ! Do you need any help with this PR ?

@apodhrad Ping again!

to me it seems that this PR is outdated in favor of list_resources API

I think this PR will be replaced by another (better) PR. @oharan2 could you provide more details if we can close this PR, please?

I think this PR will be replaced by another (better) PR. @oharan2 could you provide more details if we can close this PR, please?

This PR can be closed, my upcoming PR will be rebased on top of the work done by @apodhrad.

I think this PR will be replaced by another (better) PR. @oharan2 could you provide more details if we can close this PR, please?

This PR can be closed and replaced by me with new changes rebased on top of @apodhrad work.
Note: This PR sets the approach of using the resourcegroupstaggingapi client, which is still backlog on my end. We're working on completing #18 (using AWS Resouce Explorer).

@apodhrad @oharan2 Feel free to close this one and create fresh new PR including changes in this PR(as you wanted to).
OR
you could also push to this PR without closing this one to preserve the review history.


Add Singapore

Thanks for making this contribution. I asked a question onwiki, which would be nice to get an answer on.

Thanks for making this contribution. I asked a question onwiki, which would be nice to get an answer on.

Hi, thanks for asking. We have replied to your answer on wiki. Hopefully this makes things clearer :)


Does Not See MP3 files  Flac ok

I saw this problem mentioned on a reddit  post from 10 months ago but no other mention of it. Love the program great tool but need some mp3 help. Using this on windows 11.

ok figured what was wrong. Extension had to be in lower case .mp3  not .MP3   Not sure why they were saved that way.

I see. Uppercase MP3 should still be a valid mp3 extension though. I will address this in the next update.

update its both flac and mp3 that need to be lowercase

This is fixed in v0.2.1:
https://github.com/tranxuanthang/lrcget/releases


Scaling the --update-freq param with different number of GPUs (Edunov et al. 2018)

Great to see there is now code provided to reproduce the results of Edunov et al. 2018. I was however wondering about this remark in steps 2 and 3:

Note: the above command assumes 8 GPUs. Adjust --update-freq if you have a different number of GPUs.

When using 8 GPUs, is the effective batch size 8 times the 3584 tokens of the --max-tokens parameter? And would the correct course of action be increasing --update-freq if you use fewer than 8 GPUs?
In the paper in section 4.2, the training was described as 30k updates on 16 GPUs, but the command in the example is for a training on 8 GPUs with --update-freq 16. Do those settings in the repo reflect those in the paper?
Thanks in advance!

The large scale results were done on 128 GPUs (Section 5.6). --update-freq=16 accumulates 16 batches on each GPU, thus simulating training on 16*8=128 GPUs.

When using 8 GPUs, is the effective batch size 8 times the 3584 tokens of the --max-tokens parameter? And would the correct course of action be increasing --update-freq if you use fewer than 8 GPUs?

Yes, exactly. If you train on 4 GPUs then use --update-freq=32.


Add setMeta to ResourceAbstract

The setMetaValue() method has a complementary getMetaValue(), but setMeta() did not have a complementary getMeta() method.
This allows a set of meta data to be applied to the resource at once, preventing the need to do something like the following in application code.
foreach ($someMetaData as $metaKey => $metaValue) {
  $theResource->setMetaValue($metaKey, $metaValue);
}

I went with overwriting existing meta instead of merging because that's what setMetaValue() is doing for individual keys.

Tests and scrutinizer are failing. Can you fix travis?

@philsturgeon I'd love to. The suite fails on hhvm with this:

Fatal error: Class already declared: PHPUnit_Framework_Error in phar://phpunit-4.6.4.phar/phpunit/Util/../Framework/Error.php on line 22

Looks like others are having a similar issue.

It’s ok I found a workaround from another league member. Travis deployed a new HHVM which falls over on phpunit, it’s a whole pile of f**ked.
Waiting on travis to confirm my workaround, then I’ll ask you to pull changes.
--
Phil
From: Jason Daly<EMAIL_ADDRESS>Reply: thephpleague/fractal<EMAIL_ADDRESS>Date: April 28, 2015 at 1:27:07 PM
To: thephpleague/fractal<EMAIL_ADDRESS>Cc: Phil Sturgeon<EMAIL_ADDRESS>Subject:  Re: [fractal] Add setMeta to ResourceAbstract (#176)
@philsturgeon I'd love to. The suite fails on hhvm with this:
Fatal error: Class already declared: PHPUnit_Framework_Error in phar://phpunit-4.6.4.phar/phpunit/Util/../Framework/Error.php on line 22
Looks like others are having a similar issue.
—
Reply to this email directly or view it on GitHub.

Ok, pull from master and you should get a green build.

I don't know what the issue is now. The build is passing but Scrutinizer says it was told otherwise.


v0.3.1

Signed-off-by: Brian Smith<EMAIL_ADDRESS>

It would be great if v0.3.1 could be released soon, after PR #79 is merged. This is an attempt to help with that release process as much as I can. This is designed to be merged after PR #79 and after any other changes for the 0.3.1 release.

I merged this manually, but during the release process I found an issue, which resulted in #85.  As a result every crate except prost-build got a 0.3.1 release cut (multi-project workspace releases are a real PITA).  Once that lands I'll cut a 0.3.2.


Increase agent's apiserver ready timeout

Proposed Changes
Since we now start the server's agent sooner and in the background, we
may need to wait longer than 30 seconds for the apiserver to become
ready on downstream projects such as RKE2.
Since this essentially just serves as an analogue for the server's
apiReady channel, there's little danger in setting it to something
relatively high.
Types of Changes
bugfix
Verification

Normal CI checks on K3s.
Should resolve CI failures currently seen when updating K3s in RKE2.

Linked Issues

TBD

User-Facing Change

Further Comments

https://github.com/k3s-io/k3s/blob/master/pkg/daemons/control/server.go#L422 we care about this one too?

No, that one's in a retry loop and prints a status message every 30 seconds if it hasn't succeeded yet, so it's fine.


Pending Repository Updates

This ticket is created to serve as a tracker for pending updates to this repository, which include bug fixes and enhancements in functionality from Comcast.

Fixed by https://github.com/rdkcentral/MACPolicyParse/pull/2


Css class .ball-spin-fade-loader not available in production build

Hi there,
Thank you for this awesome library. I have an issue when I compile the app for production.
.ball-spin-fade-loader class are not present.
Do you know about this?
Many thanks,
Radu

Hi, Radu! Thank you!
Do you import vue-loaders/dist/vue-loaders.css in your project?

Hi Hokid,
Thank you for the very prompt answer.
This is how I use:
import 'vue-loaders/dist/vue-loaders.css';
import VueLoaders from 'vue-loaders';
import VueLoadersBallBeat from 'vue-loaders/dist/loaders/ball-beat';

Vue.use(VueLoaders);

Then in template I use both forms:
<vue-loaders name="ball-spin-fade-loader" :color="color" :scale="scale"></vue-loaders> {{ text }} <vue-loaders-ball-spin-fade-loader :color="color" :scale="scale"/>
When I use
vue-cli-service serve -> works ok
vue-cli-service build -> css classes are missing. I use vue cli 3. I haven't configured anything regarding css loading. I assume this is something that cli does automatically?
Many thanks,
Radu D

Hi Hokid,
Thank you for the very prompt answer.
This is how I use:
import 'vue-loaders/dist/vue-loaders.css';
import VueLoaders from 'vue-loaders';
import VueLoadersBallBeat from 'vue-loaders/dist/loaders/ball-beat';

Vue.use(VueLoaders);

Then in template I use both forms:
<vue-loaders name="ball-spin-fade-loader" :color="color" :scale="scale"></vue-loaders> {{ text }} <vue-loaders-ball-spin-fade-loader :color="color" :scale="scale"/>
When I use
vue-cli-service serve -> works ok
vue-cli-service build -> css classes are missing. I use vue cli 3. I haven't configured anything regarding css loading. I assume this is something that cli does automatically?
Many thanks,
Radu D

Can you please create repo with minimal code example to reproduce the bug? I will clone that repo and make production build to investigate the problem.

@radud what exactly versions of vue-loaders and vue cli you use? I try use vue-loaders 4.1.1 and vue cli 3.12.1 and can`t confirm tthe problem .

I haven't configured anything regarding css loading. I assume this is something that cli does automatically?
Yes

@radud if you still have the problem then reopen issue and provide steps to reproduce or repository with example.


Question on long lived proxy and limits

Hi. First let me start by saying this project is awesome.
I understand the intent of this project is to ease dev / testing / short lived demos. Are there any limitations on using localtunnel for long lived proxying (permanent-ish tunnels)? Also is there a hard limit on how many tunnels can be opened with a single server?
For context, I'm working on a device that I'd like to auto connect to a subdomain once it's online ie. <device-id>.my-domain.com. The main use case is I'd like to enable HTTPS for security. I've spent some time looking at options but haven't found a good solution yet; copying private key certs for 'my-domain' doesn't seem like a good idea since I intend to distribute these devices. Other tunneling solutions do work but are limited on the amount of tunnels, or start charging after 20+ tunnels. Localtunnel makes sense to me for this, but was curious if there are any limitations since my use case is probably not what was intended for it.
Thanks

Hey! There's no limits on long lived proxying except that someone else might take over your subdomain in case the tunnel ever goes down (server/client issues or general internet issues). This is less of a problem if it's a private server like you seem to have.
There's no limit on how many tunnels can be opened (except there's a built in limit of 10 concurrent tunnels per subdomain but it can be changed in your private server fork).
Where did you want https to be used? If it's only on the assigned public subdomain you only need a wildcard SSL cert on the localtunnel-server installation (since device-id.my-domain.com is actually hitting your server all the time and it proxies requests to the device over regular TCP sockets).

Hey @TheBoroer! thanks for your reply. Makes sense, at the moment I have a setup with wildcard SSL certificate and a couple of changes as you pointed out.
What I'm aiming for in the long term is to adapt localtunnel to be used as a public service, with the idea of allowing users to access their private devices out-of-network for home automation. I know that sounds risky to say the least — I've implemented enough security on the private devices themselves to be comfortable with, but securing the proxy would of course be a different animal, that and ensuring other people can't take over a subdomain as you pointed out.
Couple of questions if you don't mind

You mentioned the limit of 10 concurrent tunnels per subdomain; is that the connected_sockets value in the tunnel stats? the one coming from maxSockets?
Why are tunnels using multiple sockets each?
If each tunnel needs a socket, would that limit the server to 64K tunnels? or 6.4K if each use 10
Is it possible to scale the service horizontally by adding nodes? I imagine not out of the box, as the LB would somehow need to know which node owns a tunnel/connection, conflicts and all of that fun.

Thanks!


syntax error on dart sdk version with Flutter 3.0.3

Describe the bug
When upgrading to flutter 3.0.3 I received the following error:
Expected version number after ">=" in ">=2.12-0-0 <3.0.0", got "2.12-0-0 <3.0.0".
Manually changing the version to 2.12.0 resolved this issue. I believe the hyphens or dashes are causing a syntax error.
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
**Logs **
Run flutter analyze and attach any output of that command below.
If there are any analysis errors, try resolving them before filing this issue.
Paste the output of running flutter doctor -v here.
Additional context
Add any other context about the problem here.

Hi @melissa-q-amica 👋
Thanks for opening an issue!
Can you clear your pub cache and try again?
dart pub cache clean

Issue still occurs after
dart pub cache clean

I'm also experiencing this bug with Mocktail and Mockingjay.
Changing sdk to 2.12.0 does not solve it.
Error message:
Invalid version constraint: Expected version number after ">=" in ">=2.12-0-0 <3.0.0", got "2.12-0-0 <3.0.0".

I would recommend removing mocktail from pub-cache and reinstalling. Also please ensure you're using the latest version of mocktail by checking the version in your pubspec.lock.
If you're still having trouble can you please share a link to a minimal reproduction sample? Thanks!

I would recommend removing mocktail from pub-cache and reinstalling. Also please ensure you're using the latest version of mocktail by checking the version in your pubspec.lock.
If you're still having trouble can you please share a link to a minimal reproduction sample? Thanks!

my flutter doctor:
[✓] Flutter (Channel stable, 3.0.5, on macOS 11.4 20F5046g darwin-x64, locale pt-BR)
• Flutter version 3.0.5 at /Users/regisksc/Dev/flutter
• Upstream repository https://github.com/flutter/flutter.git
• Framework revision f1875d570e (4 weeks ago), 2022-07-13 11:24:16 -0700
• Engine revision e85ea0e79c
• Dart version 2.17.6
• DevTools version 2.12.2
[!] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
• Android SDK at /Users/regisksc/Library/Android/sdk
✗ cmdline-tools component is missing
Run path/to/sdkmanager --install "cmdline-tools;latest"
See https://developer.android.com/studio/command-line for more details.
✗ Android license status unknown.
Run flutter doctor --android-licenses to accept the SDK licenses.
See https://flutter.dev/docs/get-started/install/macos#android-setup for more details.
[✓] Xcode - develop for iOS and macOS (Xcode 13.2.1)
• Xcode at /Applications/Xcode.app/Contents/Developer
• CocoaPods version 1.11.2
[✓] Chrome - develop for the web
• Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome
[✓] Android Studio (version 4.2)
• Android Studio at /Applications/Android Studio.app/Contents
• Flutter plugin can be installed from:
🔨 https://plugins.jetbrains.com/plugin/9212-flutter
• Dart plugin can be installed from:
🔨 https://plugins.jetbrains.com/plugin/6351-dart
• Java version OpenJDK Runtime Environment (build 11.0.8+10-b944.6916264)
[✓] VS Code (version 1.70.0)
• VS Code at /Applications/Visual Studio Code.app/Contents
• Flutter extension version 3.46.0
[✓] VS Code
• VS Code at /Users/regisksc/Downloads/Visual Studio Code.app/Contents
• Flutter extension version 3.46.0
[✓] Connected device (3 available)
• iPhone 13 Pro Max (mobile) • 8144C65E-EE21-4B39-A922-A3326C57861D • ios            •
com.apple.CoreSimulator.SimRuntime.iOS-15-2 (simulator)
• macOS (desktop)            • macos                                • darwin-x64     •
macOS 11.4 20F5046g darwin-x64
• Chrome (web)               • chrome                               • web-javascript •
Google Chrome 98.0.4758.109
[✓] HTTP Host Availability
• All required HTTP hosts are available
I'm currently not testing android.

I've run:
dart pub cache clean
pubspec.yaml:
mocktail: ^0.3.0
when running flutter pub get:
Error on line 8, column 8 of /Users/regisksc/Dev/flutter/.pub-cache/hosted/pub.dartlang.org/mocktail-0.1.2/pubspec.yaml: Invalid version constraint: Expected version number after ">=" in ">=2.12-0-0 <3.0.0", got "2.12-0-0 <3.0.0".
╷
8 │   sdk: ">=2.12-0-0 <3.0.0"
│        ^^^^^^^^^^^^^^^^^^^
╵
pub get failed (65;   ╵)
exit code 65


[SPARK-14722][SQL] Rename upstreams() -> inputRDDs() in WholeStageCodegen

What changes were proposed in this pull request?
Per @rxin's suggestions, this patch renames upstreams() to inputRDDs() in WholeStageCodegen for better implied semantics
How was this patch tested?
N/A

LGTM pending Jenkins.

Test build #56168 has started for PR 12486 at commit 60665a8.

Test build #56168 has finished for PR 12486 at commit 60665a8.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/56168/
Test PASSed.

Merged build finished. Test PASSed.

Merging in master.


OCPBUGS-17037: Fix upgrade failure due to immutable label selectors

This fixes upgrade due to immutable selector labels by adding a -v2 suffix to the controller manager deployment.
This is the intended solution for when you need to make changes to an immutable field as per OLM: https://github.com/operator-framework/operator-lifecycle-manager/issues/952#issuecomment-639657949
If we need to make another immutable change in the future then we can just increment v2 to v3 and so on.

/cherry-pick release-4.13

/cherry-pick release-4.12


idl update does not remove old service files

If a service renames a service thrift file, both the old file and the new file are kept around in the idl repository. idl should not just append, but sync the folder to match the folder in the original repository.

This should not be the case. The code definitely blows stuff away when updating a publisDid the service name change at all as well? Can you give me a way to reproduce this so I know exactly what the problem is.

I'll send you the repo information offline.

Can you install 3.1.3 and update the thrift files in your project and see if the publish resolves it?

I installed 3.1.3, and ran idl publish from the root of curly - nothing happened.


Key recognition was corrupted between Atom version 1.35.0 and 1.36.0

Prerequisites

[ ] Put an X between the brackets on this line if you have done all of the following:

Reproduced the problem in Safe Mode: https://flight-manual.atom.io/hacking-atom/sections/debugging/#using-safe-mode
Followed all applicable steps in the debugging guide: https://flight-manual.atom.io/hacking-atom/sections/debugging/
Checked the FAQs on the message board for common solutions: https://discuss.atom.io/c/faq
Checked that your issue isn't already filed: https://github.com/issues?utf8=✓&q=is%3Aissue+user%3Aatom
Checked that there is not already an Atom package that provides the described functionality: https://atom.io/packages

Description

Key recognition is corrupting at least since version 1.36.0 (now 1.40.1).
I uses macOS and JIS keyboard (Japanese keyboard) computer but I changes key setting to US keyboard on OS setting and uses key remapping software: Karabiner-Elements. Some keys that are changed from original JIS setting keys doesn't recognized correctly by recent versions Atom (I can reproduce for 1.36.0-1.40.1) but older versions works fine (I checked for 1.34.0 and 1.35.0).
Even if I set to default profiles in Karabiner-Elements, the problems are still there, so I think this is not related to Karabiner-Elements.
I have not experienced the situations such that my key remapping causes problems in all other applications except recent versions Atom.
In recent versions Atom, inputting words in text file works fine but Key Binding Resolver shows wrong key and key bindings doesn't work correctly.
Examples

I press \ but recognized as ] in Atom Key Binding Resolver but in the text file, \ is inputted.
I press cmd + \ but recognized as cmd + ] in Atom Key Binding Resolver and "Toggle Tree View" doesn't work because recognized as cmd + ].

Steps to Reproduce

On physical JIS keyboarded mac, change keyboard type from SystemPreference > Keyboard to US layout.
Use Atom.
Try the examples.

Reproduces how often:

Every time.
Versions

OS: macOS
Atom versions: 1.35.0 works fine, in 1.36.0-1.40.1, it is corrupted.

Is the related change https://github.com/atom/atom/pull/18864?

thank you

thanks

@Paalon - we wanted to check if this could be related to Electron, can you try Atom Nightly or Atom Beta and let us know if you see the same issue?
Both are currently on Electron 4.2.7 while Atom 1.40.1 is on Electron 3.1.10.

I can also reproduce this behavior on Atom Beta 1.41.0-beta1.

VS Code has similar behavior https://github.com/microsoft/vscode/issues/88218 but the change of the setting "keyboard.dispatch": "code"  to "keyboard.dispatch": "keyCode"` solves this.


[Xcodeproj] Allow adding to LD_RUNPATH_SEARCH_PATHS with a custom xcconfig

I ran into this while trying to allow a project to link against libclang.dylib.
In my --xcconfig-overrides, I included LD_RUNPATH_SEARCH_PATHS = ... $(inherited), but those search paths were being completely overwritten by the target's settings.

@aciidb0mb3r thanks for the review, I've addressed the two comments

@swift-ci please smoke test

Thanks @segiddins!


Replace invalid characters with a placeholder

It seems that when tests print an emoji in the failure reason rspec_junit_formatter fails and prints the xml its already generated to that point which ends up being invalid because it creates an xml element with a missing closing brackets and that cannot be parsed by the tools that attempt to read it.
This fix causes invalid characters to be replaced with ??invalid?? instead of bailing out immediately. rspec_junit_formatter shouldn't output invalid xml regardless of the failures it encounters along the way.
This should fix this issue: https://github.com/sj26/rspec_junit_formatter/issues/92

This would be useful 👍🏼


Feat: Angular 13 Update | Enable Ivy

When angular compiles the library it says:
"Encourage the library authors to publish an Ivy distribution."
Could be possible to update the library in order to have an Ivy distribution?
Thanks a lot!

You can use npm i --save @angular-ex/uploader for Ivy engine. I hope my fork is useful while we wait for author upgrading to v13.

Thanks @panoti , @angular-ex/uploader is the way!
Any plans to update it to NG16?

Thanks @panoti , you are awesome


esp32 c3 coded phy double messages

Hi All!
I am using multi-adv example ( esp-idf ) for esp32c3 . In which i am sending the coded phy messages and receiving from ble5-_security_client example.
Tx Side i am sending the following data using coded phy . " ESP_MULTI_ADV_CODED"
and on Rx side i am receiving
I (8315684) SEC_GATTC_DEMO: extend adv, adv type 0x2 data len rssi -51
I (8315694) SEC_GATTC_DEMO: extend adv, adv type 0xa data len
ESP_MULTI_ADV_CODED rssi -51
My question is that I should receive the single message which is
(8315694) SEC_GATTC_DEMO: extend adv, adv type 0xa data len
ESP_MULTI_ADV_CODED rssi -51
why I am receiving this packet as well.
I (8315684) SEC_GATTC_DEMO: extend adv, adv type 0x2
Thanks in Advance .

Is this issue related to Rust?

Thank you for the response!
The issue is related to the esp32c3, embedded C. Sorry I don't know about Rust.


⚠ Bofrost.be has degraded performance

In 3e7147f, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1752 ms

Resolved: Bofrost.be performance has improved in bfa2b39.


js lint fail

Issue Type: Bug
in this class method definition, the js linter
fails to catch the extra { on fourth line
which is not only imbalanced but also creates an orphan else

start(){
    console.log( "this.preloaded="+this.preloaded );
    if( this.preloaded )
            this.game.scene.start( this );{
    else
            this.latentStart = true;
    }

VS Code version: Code 1.35.1 (c7d83e57cd18f18026a8162d042843bda1bcf21f, 2019-06-12T14:30:02.622Z)
OS version: Windows_NT x64 10.0.17134
Extensions (4)

Extension
Author (truncated)
Version

aws-toolkit-vscode
ama
1.0.0

rainbow-csv
mec
1.2.0

debugger-for-chrome
msj
4.11.7

LiveServer
rit
5.6.1

As I can see - VS Code reports problem with your code.

Similar error will be reported by browsers
Uncaught SyntaxError: Unexpected token 'else' and SyntaxError: expected expression, got keyword 'else'.
You see this error because it is first syntactically invalid piece - { can be placed in the end of the line.
If you remove else than code can be valid, because
{
   // else   - removed
            this.latentStart = true;
    }

is just code block and then missing } will be reported.

Please share a complete code example
Also test this using the latest VS Code version

will do this evening
i am very impressed by your rigor!
On Mon, Aug 19, 2019 at 4:12 PM Matt Bierner<EMAIL_ADDRESS>wrote:

Please share a complete code example
Also test this using the latest VS Code version
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/microsoft/vscode/issues/79436?email_source=notifications&email_token=AACRXX4E7EFWYE25PWDKLR3QFL4XNA5CNFSM4IMZ2EDKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD4UEN6I#issuecomment-522733305,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AACRXXYBOUJMNP7QAOT64R3QFL4XNANCNFSM4IMZ2EDA
.

I have had trouble creating reproduction steps in latest Code
so I suppose we ought to close this ticket
THANKS
Dov Jacobson
Managing Director
GamesThatWork.com
404 806 5428
On Mon, Aug 19, 2019 at 5:32 PM Dov<EMAIL_ADDRESS>wrote:

will do this evening
i am very impressed by your rigor!
On Mon, Aug 19, 2019 at 4:12 PM Matt Bierner<EMAIL_ADDRESS>wrote:

Please share a complete code example
Also test this using the latest VS Code version
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/microsoft/vscode/issues/79436?email_source=notifications&email_token=AACRXX4E7EFWYE25PWDKLR3QFL4XNA5CNFSM4IMZ2EDKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD4UEN6I#issuecomment-522733305,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AACRXXYBOUJMNP7QAOT64R3QFL4XNANCNFSM4IMZ2EDA
.

thx
On Tue, Aug 20, 2019 at 4:44 PM Matt Bierner<EMAIL_ADDRESS>wrote:

Closed #79436 https://github.com/microsoft/vscode/issues/79436.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/microsoft/vscode/issues/79436?email_source=notifications&email_token=AACRXX2VHPJ3AV4PJST2QTDQFRJHNA5CNFSM4IMZ2EDKYY3PNVWWK3TUL52HS4DFWZEXG43VMVCXMZLOORHG65DJMZUWGYLUNFXW5KTDN5WW2ZLOORPWSZGOTE7GC7Q#event-2571002238,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AACRXX7RGBHSPID2MJGG2N3QFRJHNANCNFSM4IMZ2EDA
.


Modal element is not removed if using animateProvider.classNameFilter

If there is a $animateProvider.classNameFilter(/whitelistedelement/) then https://github.com/angular-ui/bootstrap/blob/master/src/modal/modal.js#L231 is never run (looks like the event is never triggered). As a result, the modal window is faded out, but never removed from the dom. It stays in the foreground with 0 opacity so that nothing on the page can be clicked.
Here's a plunk: http://plnkr.co/edit/hPMd9R2jsRzSivVxTUNg?p=preview
As you can see, you can open and close the modal only once.
This problem appeared in 0.13. It was working in 0.12 and 0.11. Not sure why, though.
Temporary fix: use $animateProvider.classNameFilter(/whitelistedelement|modal/).

Thank you for reporting this. I was afraid I missed something. This will be fixed soon

Is there a workaround for this for now? Thx!

+1

With angular 1.4.0 this is not working anymore!
     angular.module('packingApp').config(['$animateProvider', function($animateProvider){
           $animateProvider.classNameFilter(/modal/);
     }]);

@germanger FYI; As of this moment, a stable release of Angular 1.4 hasn't been released yet and thus UI Bootstrap doesn't support use of Angular 1.4 yet.

+1 (with Angular 1.4)

The only workaround for angular 1.4 that I could find, is to set animation: false when creating new dialog.

Is it possible to set that ($modalProvider.options.animation = false;) via config and not in every modal?

@rvanbaalen you're wrong about the official Angular 1.4 release. It was tagged 6 days ago (one day prior to your comment) ~ https://github.com/angular/angular.js/releases/tag/v1.4.0

@germanger yes it is:
$modal.open({
animate: true|false
});

@philBrown ah well.. Cant always be right :wink: thanks for correcting me.

Resolved in #3895
Working Plunker: http://plnkr.co/edit/4gMUSzeLv65atc0flHFx?p=preview

@char0n actually it's animation, not animate:
$modal.open({
   animation: true|false
});

Not very consistent though; the ui.bootstrap.progressbar does use animate.

Yep, thanks for correcting me.
On Tue, Jul 7, 2015 at 1:15 PM, Sjors Provoost<EMAIL_ADDRESS>wrote:

@char0n https://github.com/char0n actually it's animation, not animate:
$modal.open({
animation: true|false
});
Not very consistent though; the ui.bootstrap.progressbar does use animate.
—
Reply to this email directly or view it on GitHub
https://github.com/angular-ui/bootstrap/issues/3647#issuecomment-119170873
.

--
Vladimír Gorej | CodeScale s.r.o.
email<EMAIL_ADDRESS>tel: +420 777 861 279, +421 948 023 011
web: http://www.codescale.net

This looks like it has been fixed according to #3895 - closing as resolved.


American Truck Simulator support

Added American Truck Simulator support (uses the same GSI/Telemetry DLL as Euro Truck Simulator 2).

Thanks for the contribution!


Compiling error on Raspberry Pi 4

Hi, while I am trying to compile the latest mavsdk (0.38.2) on my rPi 4, I had this error when I try to install:
mavsdk/src/core/mavlink_parameters.cpp:279:23: error: too few arguments to function ‘uint16_t mavlink_msg_param_ext_request_read_pack(uint8_t, uint8_t, mavlink_message_t*, uint8_t, uint8_t, const char*, int16_t, uint8_t)’
  279 |                     -1);
      |                       ^

...
mavsdk/src/third_party/mavlink/include/mavlink/v2.0/common/mavlink_msg_param_ext_request_read.h:63:24: note: declared here
   63 | static inline uint16_t mavlink_msg_param_ext_request_read_pack(uint8_t system_id, uint8_t component_id, mavlink_message_t* msg,
      |                        ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

At global scope:
cc1plus: note: unrecognized command-line option ‘-Wno-unknown-warning-option’ may have been intended to silence earlier diagnostics
make[2]: *** [src/core/CMakeFiles/mavsdk.dir/build.make:252: src/core/CMakeFiles/mavsdk.dir/mavlink_parameters.cpp.o] Error 1
make[1]: *** [CMakeFiles/Makefile2:752: src/core/CMakeFiles/mavsdk.dir/all] Error 2
make: *** [Makefile:150: all] Error 2

I thought my gcc version was 8 so I took 4 hours to get the latest gcc version (10.2.0), and this error is still there. Any idea what cause this error?

Sounds like you use mavsdk as a git submodule, and you updated it without updating its submodules (i.e. mavlink). Could it be?

@cjffly try git submodule update --init --recursive in MAVSDK.

Can you show the cmake commands you use to configure and build?
I wonder why it fails at build time, I would have expected it to fail at configure time, here :thinking:

I am using the standard configure and build commands:
cmake -DCMAKE_BUILD_TYPE=Debug -DBUILD_SHARED_LIBS=ON -Bbuild/default -H.
cmake --build build/default

Here is the part from configure output which contains tinyxml2:
-- Preparing external project "tinyxml2" with args:
-- -- -DCMAKE_INSTALL_PREFIX:PATH=......mavsdk/build/default/third_party/install
-- -- -DCMAKE_TOOLCHAIN_FILE:PATH=
-- -- -DCMAKE_BUILD_TYPE=Debug
-- -- -DCMAKE_POSITION_INDEPENDENT_CODE=ON
-- -- -DBUILD_SHARED_LIBS=OFF
-- -- -DBUILD_TESTS=OFF
-- Configuring done
-- Generating done
CMake Warning:
  Manually-specified variables were not used by the project:

    DEPLOYMENT_TARGET
    ENABLE_STRICT_TRY_COMPILE
    PLATFORM

-- Build files have been written to: ......mavsdk/build/default/third_party/tinyxml2
Scanning dependencies of target tinyxml2
[ 12%] Creating directories for 'tinyxml2'
[ 25%] Performing download step (download, verify and extract) for 'tinyxml2'
-- Downloading...
   dst='......mavsdk/build/default/third_party/tinyxml2/tinyxml2/src/7.1.0.tar.gz'
   timeout='none'
-- Using src='https://github.com/leethomason/tinyxml2/archive/7.1.0.tar.gz'
-- [download 100% complete]
-- Downloading... done
-- extracting...
     src='......mavsdk/build/default/third_party/tinyxml2/tinyxml2/src/7.1.0.tar.gz'
     dst='......mavsdk/build/default/third_party/tinyxml2/tinyxml2/src/tinyxml2'
-- extracting... [tar xfz]
-- extracting... [analysis]
-- extracting... [rename]
-- extracting... [clean up]
-- extracting... done
[ 50%] No update step for 'tinyxml2'
[ 50%] No patch step for 'tinyxml2'
[ 62%] Performing configure step for 'tinyxml2'
CMake Deprecation Warning at CMakeLists.txt:11 (cmake_policy):
  The OLD behavior for policy CMP0063 will be removed from a future version
  of CMake.

  The cmake-policies(7) manual explains that the OLD behaviors of all
  policies are deprecated and that a policy should be set to OLD only under
  specific short-term circumstances.  Projects should be ported to the NEW
  behavior and not rely on setting a policy to OLD.

-- The C compiler identification is GNU 10.2.0
-- The CXX compiler identification is GNU 10.2.0
-- Check for working C compiler: /usr/bin/cc
-- Check for working C compiler: /usr/bin/cc - works
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Detecting C compile features
-- Detecting C compile features - done
-- Check for working CXX compiler: /usr/bin/c++
-- Check for working CXX compiler: /usr/bin/c++ - works
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Detecting CXX compile features
-- Detecting CXX compile features - done
-- Configuring done
-- Generating done
-- Build files have been written to: ......mavsdk/build/default/third_party/tinyxml2/tinyxml2/src/tinyxml2-build
[ 75%] Performing build step for 'tinyxml2'
Scanning dependencies of target tinyxml2
[ 50%] Building CXX object CMakeFiles/tinyxml2.dir/tinyxml2.cpp.o
[100%] Linking CXX static library libtinyxml2d.a
[100%] Built target tinyxml2
[ 87%] Performing install step for 'tinyxml2'
[100%] Built target tinyxml2
Install the project...
-- Install configuration: "Debug"
-- Installing: ......mavsdk/build/default/third_party/install/lib/libtinyxml2d.a
-- Installing: ......mavsdk/build/default/third_party/install/include/tinyxml2.h
-- Installing: ......mavsdk/build/default/third_party/install/lib/pkgconfig/tinyxml2.pc
-- Installing: ......mavsdk/build/default/third_party/install/lib/cmake/tinyxml2/tinyxml2Config.cmake
-- Installing: ......mavsdk/build/default/third_party/install/lib/cmake/tinyxml2/tinyxml2ConfigVersion.cmake
-- Installing: ......mavsdk/build/default/third_party/install/lib/cmake/tinyxml2/tinyxml2Targets.cmake
-- Installing: ......mavsdk/build/default/third_party/install/lib/cmake/tinyxml2/tinyxml2Targets-debug.cmake
[100%] Completed 'tinyxml2'
[100%] Built target tinyxml2
-- The C compiler identification is GNU 10.2.0
-- The CXX compiler identification is GNU 10.2.0
-- Check for working C compiler: /usr/bin/cc
-- Check for working C compiler: /usr/bin/cc - works
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Detecting C compile features
-- Detecting C compile features - done
-- Check for working CXX compiler: /usr/bin/c++
-- Check for working CXX compiler: /usr/bin/c++ - works
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Detecting CXX compile features
-- Detecting CXX compile features - done

Can you show the cmake commands you use to configure and build?
I wonder why it fails at build time, I would have expected it to fail at configure time, here 🤔

OK, seems I figure out that I add my include($ENV{VCPKG_ROOT}/scripts/buildsystems/vcpkg.cmake) at the wrong CMakeLists.txt. Previous I thought I should add this into the one under MAVSDK root folder. Now I add the "include vcpkg" to MAVSDK/src/CMakeLists.txt, and build is success.
However, if I don't have vcpkg this will be a problem, so I guess it still needs some kind of fix.

Old, closing.


TSC meeting notes 02-25-2019

Add 02-25-2019 notes as RST
Correct template TSC membership

I think that the PR qualifies for single approver rules.

@hodoulp Does it even need that? I like the fact that we submit the meeting notes as a formal PR to give the other attendees a chance to point out any errors or omissions before it's committed. But -- this is just my opinion -- the secretary of the week can probably let it stand for 2 days, fix any errors pointed out in that time, and then commit it.
I would suggest the review rules be more relaxed for PRs that don't alter code (for example, if they only consist of docs, comments, or administrative stuff). It ought to be sufficient in these cases for the PR to stand for a few days, and if there are no objections voiced, the submitter (if already a person sufficiently senior to have commit privileges) should just do the merge. When risk is sufficiently low and it's a change that can be easily backed out, lack of objection for a few days should be good enough. Just my opinion for you guys to consider: Spend the precious developer attention on reviews where it matters most.

@lgritz It seems that I did not clearly communicate my intent which is to relax the 'merge rule' for pull request on the meeting notes. So, I proposed the approach of a "single reviewer for simple pull requests" because that's the only one approved (by TSC members) for now.
On the other hand, I would agree on having a much more efficient rule (like the one you proposed) for meeting minutes.

@hodoulp I think you were clear. I think we are agreeing.


disable email sending in local dev mode

It needs extra work to configure email sending service. (See https://git.io/JLP6q)
Since developers may not have enough uique emails, printing the email information in console will be much better than sending email to real email address. So developers can use arbitrary emails for testing.

How to implement this feature
The email sending part is here: https://github.com/SaltyNote/saltynote-service/blob/master/src/main/java/com/saltynote/service/event/EmailEventListener.java#L34
Before sending email, we can check whether the user has configure the smtp account.
We can fetch the mail user value at the beginning of EmailEventListener.java.
@Value("${spring.mail.username}")
private String mailUser;

Before sending email, we can have a check whether mailUser value is still the default value.
If it is, then we just log the 3 parameters of the email sending method. Otherwise, sending the mail out.
Reference: spring.mail.username=${MAIL_USERNAME:}

BTW: I use the same strategy to check email conf in the start.sh.
https://github.com/SaltyNote/saltynote-service/blob/master/start.sh#L4


Support Fast Forward playback mode in local video player

Related PR for the online video player https://github.com/killergerbah/asbplayer/pull/323


This mod can without Better Runtime Resource Pack (BRRP) if you have Enhanced Block Entities installed

What the title says. Sort of confusing?

I just checked, this is because Enhanced Block Entities bundles ARRP.
DirTNT works with either ARRP or BRRP, one just has to make sure to not use both at once, so if you're using Enhanced Block Entities, you shouldn't install BRRP.
(ARRP and BRRP not working together is also the reason why I didn't bundle either of them in this mod.)
I hope this answers the question.


Allow selecting calendar/date picker languages without modifying CurrentUICulture

Fixes #288.

The Hebrew on looks cool, but British has gone from:

To:

Which seems a bit funny from a British perspective...we might need to go back and look at the Google examples and the .net culture stuff (I haven't spent any time looking as of typing).

@jdf3 thanks for input, this was picked up by @GabrielGM in https://github.com/ButchersBoy/MaterialDesignInXamlToolkit/pull/388


WIP GEOMESA-933 Better messages/logging/error handling for Convert Framework and Ingest Tools

Stacks on top of #1235

@aheyne the conflicts with master have been fixed in the base branch - you can merge in the changes from there

@aheyne I think another part of this ticket was to show better error message when e.g. parsing an sft string. Right now we get something like:
scala> val sft = org.locationtech.geomesa.utils.geotools.SimpleFeatureTypes.createType("foo", "bar:baz")
java.lang.IllegalArgumentException: org.parboiled.errors.ParsingException: Invalid spec string: Invalid input 'a', expected 'y' or 'o' (line 1, pos 6):
bar:baz
     ^

  at org.locationtech.geomesa.utils.geotools.SimpleFeatureTypes$.createType(SimpleFeatureTypes.scala:95)
  at org.locationtech.geomesa.utils.geotools.SimpleFeatureTypes$.createType(SimpleFeatureTypes.scala:82)
  ... 65 elided
Caused by: org.parboiled.errors.ParsingException: Invalid spec string: Invalid input 'a', expected 'y' or 'o' (line 1, pos 6):
bar:baz
     ^

  at org.locationtech.geomesa.utils.geotools.SimpleFeatureSpecParser$$anonfun$parse$1.apply(SimpleFeatureSpecParser.scala:37)
  at org.locationtech.geomesa.utils.geotools.SimpleFeatureSpecParser$$anonfun$parse$1.apply(SimpleFeatureSpecParser.scala:37)
  at scala.Option.getOrElse(Option.scala:121)
  at org.locationtech.geomesa.utils.geotools.SimpleFeatureSpecParser$.parse(SimpleFeatureSpecParser.scala:36)
  at org.locationtech.geomesa.utils.geotools.SimpleFeatureSpecParser$.parse(SimpleFeatureSpecParser.scala:25)
  ... 67 more

(it expects 'y' or 'o' because it's matching 'bytes' or 'boolean', but no one would know that...)

@elahrvivaz
Could you take a peek at this when you get a chance? It all seems to be working correctly with the exception of the thrown errors. For some reason it's throwing a FileNotFoundException when it should be throwing a ParameterException. As far as I can tell the only time that FileNotFoundException is thrown is in a Try block, so I don't think that's the cause.

@aheyne can you rebase on master? the other PR it's stacked on was merged. I haven't gotten a chance to look at your changes yet, but after you rebase I'll take a peek.

Changes up

looks good - @jahhulbert-ccri can you take a peek?
@aheyne another part of this ticket was to improve the error messages when e.g. parsing a sft string. I'm going to open a new ticket for that so we don't lose track of it.

@aheyne 933 seems like the wrong ticket - I don't see another one assigned to you that matches. Do you know the right number?

opened follow-on ticket here: https://geomesa.atlassian.net/browse/GEOMESA-1571
fixed title to correct ticket number, make sure the commit message it right when merging

@elahrvivaz  taking a look now


Design for discover list

There's no design for the list of discover posts. I think it needs to be different to the connect list so the user knows they are in a different section.

Designs for the discover list:


Update nixpkgs to ad57eef

Corresponds to https://hydra.nixos.org/eval/1806668
Includes gnome 46.2
• Updated input 'nixpkgs':
    'github:nixos/nixpkgs/3eaeaeb6b1e08a016380c279f8846e0bd8808916' (2024-05-21)
  → 'github:nixos/nixpkgs/ad57eef4ef0659193044870c731987a6df5cf56b' (2024-05-29)

I am unable to cross-compile this due to the issues with util-linux. It should be resolved by the next staging merge which is soon I think.
Please also pay better attention to the commit message format in the future and match those within the project.

Thanks, will do.
Can you tell me how to replicate the cross compiling issue for the future? Jdo you just "nix build" on a non aarch64 machine?


Unread cached file not marked as such for later reopenings

If a file in a caching file system (for the HTTP backend, at least) is opened and then closed without reading, fsspec will leave behind in the cache a file of the same size as the actual data but containing only NUL bytes, except for a 1 (0x31) as the last byte.  If the file is then opened again and read from, this cache file will be used as-is, so that the data read from the caching file system will be just NULs and a 1.
MVCE:
import fsspec
from fsspec.implementations.cached import CachingFileSystem

fs = CachingFileSystem(fs=fsspec.filesystem("http"))

with fs.open("https://httpbin.org/encoding/utf8", "r", encoding="utf-8") as f1:
    pass

with fs.open("https://httpbin.org/encoding/utf8", "r", encoding="utf-8") as f2:
    print(f2.read(), end="")

@martindurant

This looks like a different problem - if you run debug, I'd be interested to know the "start" and "end" of the block that is trying to be read

With DEBUG logging enabled, the output from the script (just before the traceback) ends with:
16:24:04 [DEBUG   ] fsspec: MMap cache fetching 974913536-974979072
16:24:04 [DEBUG   ] fsspec: <File-like object HTTPFileSystem, https://cdimage.debian.org/debian-cd/current-live/amd64/iso-hybrid/debian-live-11.1.0-amd64-standard.iso> read: 974979072 - 975044608
16:24:04 [DEBUG   ] fsspec: MMap cache fetching 974979072-975044608
16:24:04 [DEBUG   ] fsspec: <File-like object HTTPFileSystem, https://cdimage.debian.org/debian-cd/current-live/amd64/iso-hybrid/debian-live-11.1.0-amd64-standard.iso> read: 975044608 - 975110144
16:24:04 [DEBUG   ] fsspec: MMap cache fetching 975044608-975110144
16:24:04 [DEBUG   ] fsspec: <File-like object HTTPFileSystem, https://cdimage.debian.org/debian-cd/current-live/amd64/iso-hybrid/debian-live-11.1.0-amd64-standard.iso> read: 975110144 - 975175680
16:24:04 [DEBUG   ] fsspec: MMap cache fetching 975110144-975175680
16:24:04 [DEBUG   ] fsspec: MMap get block #186 (975175680-975175680
16:24:04 [DEBUG   ] fsspec.http: Fetch range for <File-like object HTTPFileSystem, https://cdimage.debian.org/debian-cd/current-live/amd64/iso-hybrid/debian-live-11.1.0-amd64-standard.iso>: 975175680-975175680
16:24:04 [DEBUG   ] fsspec.http: https://cdimage.debian.org/debian-cd/current-live/amd64/iso-hybrid/debian-live-11.1.0-amd64-standard.iso : bytes=975175680-975175679
16:24:04 [DEBUG   ] fsspec: going to save
16:24:04 [DEBUG   ] fsspec: saved

Note the "MMap get block #186 (975175680-975175680", in which the start and end points are equal (and are a multiple of 5 MiB).


Prerequisite to have port 5558 open

To use the Dataverse connector from Power BI Desktop, is also necessary to open the port 5558 besides 443 and enable TDS.
This is not referred on this article, and for companies with restrict security measures this can be an issue.
Can you please update this article with that pre-requisite - Port 5558?
Thanks!
Best regards,
Nuno Costa

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

ID: b9e6e30d-4224-459f-8525-08718d6d77e4
Version Independent ID: fe075fe5-5d2e-7b96-3608-9ad3138380da
Content: Create a Power BI report using the Dataverse connector - Power Apps
Content Source: powerapps-docs/maker/data-platform/data-platform-powerbi-connector.md
Service: powerapps
GitHub Login: @Mattp123
Microsoft Alias: matp

@ndocosta - thank you for submitting your issue. The article has been updated to include the necessary TCP ports. Closing issue.


Support Saudi mobile format reserved for future use

Support mobiles numbers starting with 52 (they're reserved for future use in Saudi Arabia).

Coverage remained the same at 99.171% when pulling 27b661635e180ace4b7dd80b24a15c92f04b48e2 on tam-developers:master into ebd7d866c98cb79902a6a4fde19f9f638ad49258 on floere:master.

@obahareth Released in 2.15.42! 🚀 Thanks so much 😊 Note that normally, a test is required, but here I made an exception since the change is so small.


Cannot leave MAIL_USERNAME and MAIL_PASSWORD unset

Hello,
It is very common that you don't need to authenticate to your mail server when using a smart/relay host setup.
However, when I try to leave them unset I get the following error.
MAIL_USERNAME
none is not an allowed value (type=type_error.none.not_allowed)
MAIL_PASSWORD
none is not an allowed value (type=type_error.none.not_allowed)

You can set them to an empty string and use USE_CREDENTIALS = false


Get on empty Gstore Fails with 500 instead of 404

Get request on a resource in a non-existing repo returns a 500
"org.eclipse.jgit.errors.RepositoryNotFoundException: repository not found: /gstore/git/tester"
Expected behaviour:
Return code is a 404 Not Found

Looool :)

must work now :)

in dev branch


tracking analysis project

add output when analyzing at various steps to print how many files have been done / how many are left

Is this still an on-going effort? @kathleenleeper


Fixed ordering of chained virtual requiredFields

Includes pinning to mongodb 2.2.x driver series

Fixed in https://github.com/simpleviewinc/mongolayer/commit/dd39ee96caf2a6a9195cfd70151d5851ebfb9891 . Closing.


Unable to import components in v2.0.3

Hello
How to use v2? I am trying to update from v1.4.0 to v2.0.3 but I unable to import components.
My code:
import {
  ConnectComponentsProvider,
  ConnectPayments,
  ConnectPayouts
} from '@stripe/react-connect-js';

Errors:
webpack compiled with 2 errors
ERROR in src/<...>/StripeConnect.tsx:6:3
TS2305: Module '"@stripe/react-connect-js"' has no exported member 'ConnectPayments'.
    4 | import {
    5 |   ConnectComponentsProvider,
  > 6 |   ConnectPayments,
      |   ^^^^^^^^^^^^^^^
    7 |   ConnectPayouts
    8 | } from '@stripe/react-connect-js';

ERROR in src/<...>/StripeConnect.tsx:7:3
TS2305: Module '"@stripe/react-connect-js"' has no exported member 'ConnectPayouts'.
     5 |   ConnectComponentsProvider,
     6 |   ConnectPayments,
  >  7 |   ConnectPayouts
       |   ^^^^^^^^^^^^^^
     8 | } from '@stripe/react-connect-js';

Hi!
I had the same issue and I don't know the reason but as far as I see from the repo, the beta components like ConnectPayments and ConnectPayouts are removed from the stable releases.
I checked the docs where it is mentioned to install beta versions like this:
npm install --save @stripe/connect-js@beta

npm install --save @stripe/react-connect-js@beta

Doing so fixed my problem. I hope it will help you too


GHA docs build failing

Traceback (most recent call last):
  File "/entrypoint.py", line 22, in <module>
    action.build_all_docs(github_env, [os.environ.get("INPUT_DOCS-FOLDER")])
  File "/sphinx_action/action.py", line 152, in build_all_docs
    return_code, annotations = build_docs(github_env.build_command, docs_dir)
  File "/sphinx_action/action.py", line 134, in build_docs
    with open(log_file, "r") as f:
FileNotFoundError: [Errno 2] No such file or directory: '/tmp/sphinx-log'

Nothing changed on this end, issue must be with at ammaraskar/sphinx-action@master, sphinx, or perhaps GHA itself.
Same error occurs when building fwapg docs.

found and copied PR updating previous action: https://github.com/adempiere/docs/blob/master/.github/workflows/main.yml


Fix z.void() type

z.void() accepts null or undefined. That’s different from the TypeScript void type, which is only assignable from undefined (unless strictNullChecks is false, in which case every type is assignable from null regardless).
This unsoundness could be observed as follows:
const x = z.string().or(z.void()).parse(null);
if (x !== undefined) {
  // TypeScript infers x is a string, but it’s actually null
  x.trim(); // oops
}

I wonder if the better solution wouldn't be to fix z.void() to in fact reject null. This would be a breaking change however. @colinhacks yay or nay?

I'd prefer to keep the type signature as void (since getting void as the return type of z.function is the only reason this exists). I'm in favor of @lilred's solution. I think Zod's runtime behavior should conform to TypeScript's behavior in strict mode (as it does in many other cases).


⚠ Bofrost.at has degraded performance

In e27f40b, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 2141 ms

Resolved: Bofrost.at performance has improved in 9fe17c4 after 6 minutes.


Try caching ethers provider creation

There seem to be many timeouts caused by the RPCs. It's possible that ethers is not cleaning up the pending HTTP calls, but it might do a better job if we don't recreate the provider object during each iteration.
This seems like an easy thing to test.

As per this I don't think this issue is going to have an effect.

This is definitely a perf issue. See this.
I could see caching the provider to help (depending on how it's written internally), but we've confirmed that it works fine even without implementing this. It seems like a premature optimization to me.


chore: add protocol to url in 'ecs-preview app show'

Prepended the protocol to the url string for a more user-friendly output for 'app show'.
Included changes in go.mod and go.sum because of advice given here: https://github.com/golang/go/wiki/Modules#should-i-commit-my-gosum-file-as-well-as-my-gomod-file.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Could you also update init_test.go, line 96 in the e2e tests? From TrimSuffix to strings.TrimPrefix(app.Routes[0].URL, "http://"),

This looks great!! 🎉 Could you also add output from manually testing the app show command plz? :)

Yes!
Before:
`38f9d33fed75:dinder huanjani$ ecs-preview app show
? Which application of dinder would you like to show? api
About
Project           dinder
Name              api
Type              Load Balanced Web App
Configurations
Environment       Tasks               CPU (vCPU)          Memory (MiB)        Port
test              1                   0.25                512                 5000
Routes
Environment       URL                                                              Path
test              dinde-Publi-EXUI6JTIP4L3-1856349916.us-west-2.elb.amazonaws.com  /api/*
Variables
Name                      Environment         Value
ECS_CLI_APP_NAME          test                api
ECS_CLI_ENVIRONMENT_NAME  test                test
ECS_CLI_LB_DNS            test                dinde-Publi-EXUI6JTIP4L3-1856349916.us-west-2.elb.amazonaws.com
ECS_CLI_PROJECT_NAME      test                dinder
`

Can you also take a look at this block? https://github.com/aws/amazon-ecs-cli-v2/pull/733/files#diff-3c6b0eccdf2aa38416345540f2b37b22R98
You'll have trim the 'http://' as well

which block? wasn't sure if you meant to include some line numbers in that link -- for me it takes me to line 100 of multi_app_project_test.go

This looks great!! 🎉 Could you also add output from manually testing the app show command plz? :)

Yup.
Before:
38f9d33fed75:dinder huanjani$ ecs-preview app show

? Which application of dinder would you like to show? api
About

  Project           dinder
  Name              api
  Type              Load Balanced Web App

Configurations

  Environment       Tasks               CPU (vCPU)          Memory (MiB)        Port
  test              1                   0.25                512                 5000

Routes

  Environment       URL                                                              Path
  test              dinde-Publi-EXUI6JTIP4L3-1856349916.us-west-2.elb.amazonaws.com  /api/*

(Stay tuned for the After...)


PMID:15509866

normal DNA binding at PCB

normal DNA binding at PCB FYPO:0005091

edit file: d8f539f3a092a3067cfa9f98f001e8af27d37905
release: 3ae8ac3b5aa4065e09b06b15182174fcca9754fb


Feature Request: Easier CTAGS execution on specific file extension

Hey there,
So I want to become a product manager, and want to understand codebases better with the help of aider.
For this I would like to do things less manual and more automated.
Here is what I thought about - and asked GPT-4 based on your README:
I want to run aider with the ctags on all .py files in a different repo, how do I do this?

And the answer was:
To run Aider with ctags on all .py files in a different repo, follow these steps:

Open your terminal and navigate to the root directory of the target repository.
cd /path/to/your/repo
Install the universal ctags if you haven’t already:
# For Ubuntu/Debian-based systems
sudo apt-get install universal-ctags

# For macOS (using Homebrew)
brew install --HEAD universal-ctags/universal-ctags/universal-ctags
Generate the tags file for your repo using ctags:
ctags -R *.py
This command will recursively search for all .py files and generate a tags file in the repository root.

Run Aider with the --ctags flag and specify all .py files you want to work on:
aider --ctags file1.py file2.py ...
Replace file1.py, file2.py, etc., with the paths to the source code files you want to work on.

Now, Aider will launch using ctags for better understanding and modification of your codebase while working on specified .py files. If you don’t want to specify each individual .py file, you can use wildcards or other tools like [find](https://man7.org/linux/man-pages/man1/find.1.html) to pass all desired .py files as arguments.

And then I asked further - and received this:
You can accomplish this using the find command along with a wildcard to specify all .py files. Here’s an example of how you can do this:

Open your terminal or command prompt.
Navigate to the directory containing the .py files you want to work with.
Use the following command to pass all .py files as arguments to a script called your_script.py:
python your_script.py $(find . -name "*.py")
In this example, find . -name "*.py" will search for all .py files in the current directory and its subdirectories. The $( ... ) syntax will pass the output of the find command (i.e., the list of found .py files) as arguments

This works out quite nicely. BUT includes all files, also the 'hidden files' within .venv for example.
So to make it work without these folders - GPT-4 answer was the following:
python3 .venv/bin/aider --ctags $(find . -name "*.py" ! -path "./. venv/*") -c .venv/bin/. aider.conf .yml

How about putting this inside the README?

So after executing the command above:
RESULT: I get the following:
Added generate_repo_info.py to the chat
Added main.py to the chat
Using git repo: . git
Use /help to see in-chat commands.
generate_repo_info.py main. py
>

In the repo I am trying to analyse are three .py files.
So this does exclude the "fetch_starred_repos.py" (dont know why)
Any idea why this happens?
Steps I took:

git clone CrawlGitHub
cd into directory
python3 -m venv .venv
python3 source .venv/bin/activate
pip install git+https://github.com/paul-gauthier/aider.git
create new file .aider.conf.yml with openai-api-key:{mykey}
sudo apt-get install universal-ctags
ctags -R *.py
python3 .venv/bin/aider --ctags $(find . -name ".py" ! -path "./. venv/") -c .venv/bin/. aider.conf .yml
see result: "fetch_starred_repos.py" is missing

Thanks for trying aider! I'm not sure I fully understand what you are trying to do, but let me offer some suggestions.
The README for aider explains how to enable ctags. You just install the universal ctags tool and then run aider --ctags. That's all you need to do. It will automatically use ctags to scan every file in your git repository.
I gave this a try against the repo you mentioned:

$ git clone https://github.com/lightningRalf/CrawlGitHub.git
$ cd CrawlGitHub
$ aider  --ctags
Using git repo: .git

eplain the general structure of this project
The project consists of three files:

.gitignore: This file is used to specify files and directories that should be ignored by Git when tracking changes in the repository.

fetch_starred_repos.py: This script fetches the starred repositories of a GitHub user. It contains several variables such as api_url, github_pat, headers, json_data, page, response, starred_repos, and working_directory. The script likely makes API calls to GitHub to retrieve the starred repositories and stores the information in the starred_repos variable.

generate_repo_info.py: This script generates information about a specific repository. It contains a function get_readme_url(github_url, default_branch) and several variables such as api_url, contributors, contributors_data, contributors_response, contributors_url, default_branch, existing_files, existing_numbers, forks, github_pat, headers, latest_release, markdown_file, next_number, readme_url, releases_data, releases_response, releases_url, repo_data, repo_description, repo_name, response, stars, urls, watching, and working_directory. The script likely makes API calls to GitHub to retrieve various information about the repository, such as contributors, forks, stars, and watchers, and generates a markdown file with this information.

main.py: This file is not described in the summaries provided. It is likely the main entry point of the project, which calls the functions from the other two scripts to fetch starred repositories and generate repository information.

The general structure of the project is to fetch information about starred repositories and generate detailed information about each repository, possibly for documentation or reporting purposes.

Well, it seems I could not trust GPT4 to explain it to me well enough.
Thanks for taking the time.
Didnt know that aider --ctags takes the content of the .git files
One question remains:
is there an easy way to exclude folders? For example if I dont want GPT-4 to go through all the documentation? And just ask questions to the code?

One thing that may not be obvious is that GPT only sees a high level "map" of the repo until you explicitly give it permission to load the content of specific files. The map is built using ctags, and just shows GPT the functions and variables which are defined in each file in the git repo.
But aider doesn't sent GPT the full content of any* files, until you explicitly "add them to the chat". You can add files by naming them on the command line or by adding them using /add in the chat.
So unless you explicitly add the doc files to the chat, GPT will only see some high level info about them in the repo map.

Thank you for the clarification!


Powerrename (or any other) option doesn't appear after right click

Microsoft PowerToys version
0.86.0
Installation method
Microsoft Store
Running as admin
Yes
Area(s) with issue?
PowerRename
Steps to reproduce
Rename a few files in bach.
✔ Expected Behavior
To have the options the application showed.
❌ Actual Behavior
As the title said, not Powerrename nor any other option appears after right click. Tried with and without shift.

Tried Powerresize on a PNG, but there was no Powertoys option at all again.

Sorry that Windows is not on english, but it's clearly visible that there is no Powertoys options.
Other Software
No response

Nwm, found the solution with downloading the "Machine wide - x64" version from GitHub instead of Microsoft Store.


Data Explorer Instructions need updating

Hi team,
I am leaving feedback because we need to update the documentation for this page.
I am just going through the lab and some things have changed on Azure portal for Gremlin.
[Enter feedback here]

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

ID: 64cdb516-a4f6-b33f-b993-48197836c6e9
Version Independent ID: ae53aab5-b920-14d7-f54d-cdb3fa256a82
Content: Quickstart: Gremlin API with Python - Azure Cosmos DB
Content Source: articles/cosmos-db/create-graph-python.md
Service: cosmos-db
Sub-service: cosmosdb-graph
GitHub Login: @christopheranderson
Microsoft Alias: chrande

@ciph3rwoman Thank you for the feedback.
Are you referring to the screenshots? Could you please let us know where you are seeing discrepancies?

@ciph3rwoman Thank you for the additional context.
This is being assigned to the content author to evaluate and update the documentation as appropriate.

Thanks for your response. I am still new hence why I am lost in general.  👯♀
Also, I think the docs were written quite awhile ago and Azure has done an update or I am completely blind.
Please note I am also sending this to clients to follow through the demo. So if we can update that! It would be much appreciated 👍🏻

@ciph3rwoman I agree that the data explorer screenshot isn't accurate, yes the portal has changed and I will update it. To your question, the username is in the format mentioned. You need to replace the <YOUR_DATABASE> and <YOUR_COLLECTION_OR_GRAPH> values with your account's database and graph names. For example, if your database name is "graphdb" and graph name is "mygraph" then the user name is:
username="/dbs/<YOUR_DATABASE>/colls/<YOUR_COLLECTION_OR_GRAPH>"
username="/dbs/graphdb/colls/mygraph"

I got it working! Thanks for that.


Add an end time?

The hurricane's long gone, and we're still plotting discharge.
Maybe Oct. 09 11PM

Yes, lets do this.


timerDuration set ignored

Hello,
i'm using this timer, i try to update the duration timer default but any set is ignored.
I have used a function node with following code :
let duration = Number(msg.payload);
msg.payload = {
command: "set",
timerDuration: duration
};
return msg;
where i'm wrong ?

I solve, just send entire paramters, not just one. Note : time unit require "minute" and not "Minutes"


Spanish Doc Update 2.4.9

Please make sure these boxes are checked before submitting your PR, thank you!

[ ] Make sure you follow Element's contributing guide (中文 | English | Español).
[ ] Make sure you are merging your commits to dev branch.
[ ] Add some descriptions and refer relative issues for you PR.

Thank you for solving the problem. I'll be more careful in the next updates.


Update the example with best coordinates callback

Signed-off-by: Omar Shrit<EMAIL_ADDRESS>

Also, should we also apply the same changes to the other examples?

Yes, I will update other examples too.

Thanks @shrit! :+1:


Flesh out plugins ideas

Plugins aren't coming for a little bit, but this issue is just here to remind us to think about them, and as a dumping ground for any ideas we have in the meantime.
Plugin Points

Filter types (already there, sort of)
Resource factory types (filesystem, tarball, zip (see #406), JAR, network, Git repository)
Resource type (plain text, Word document, PDF)
Adding new options (--dirty for Git/Hg)
Search backends (ag, git-grep, code search, mdfind)
Ignore types (this kind of falls under filter types, but I think it bears mentioning.  See #28)
Custom search types (re::engine::RE2 comes to mind, but I was also thinking a special search for Perl using PPI)
Custom resource searchers (more complex than custom search types; think things like searching an XML file for an XPath expression)

We're dumping plugins.


Can you please contact me?

Hi, can you please contact me? I would like to ask you something<EMAIL_ADDRESS>

Please ask me.


Modified project lifecycle documentation for Spring

Modified project lifecycle diagram with dates for phases
Extended Deployment phase from Week 8-9 to Week 8-10 (synced with what we followed in Fall)
Made deployment phase requirements broader (included creating dashboards as a deployment solution)
Removed monitoring plan as most projects dont end up deploying a service endpoint

@trgardos updated the project lifecycle to exclude the specific dates for project weeks. Updated the syllabus with the phase headers. Please have a look


Add sentence expiry and release type screens within sentence information task

Changes in this PR
This adds two new screens to the sentence information task. The screens themselves are reasonably straightforward, though there is a lot of groundwork first to support a page containing multiple date fields
Screenshots of UI changes

Release checklist
Release process documentation.
Pre-merge checklist

[ ] Are any changes required to dependent services for this change to work?
(eg. CAS API)
[ ] Have they been released to production already?

Post-merge checklist

[ ] Manually approve
release to test
[ ] Manually approve
release to preprod
[ ] Manually approve
release to prod

 Add sentence expiry and release type screens within sentence information task

Looks good to me.
We discussed adding the 'For example 01 01 2023' hint text, I think this is probably covered in the trello ticket.


3D Max/Avg Pooling support

Issue Description
We need to bring our 3D / volumetric conv stuff up to speed. :)
(don't think we need any reference/literature here, I think there's only one interpretation)

Ops we merged earlier today, however there'll be some tweaks/fixes probably.


Doc with plans for converting core to C++

From a discussion with @ctiller and @markdroth

****************************************************************

libgrpc.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

****************************************************************

libgrpc++.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

[trickle] No significant performance differences

Merging early to get this publicized, since docs only and sanity passes.

Corrupt JSON data (indicates timeout or crash):
    bm_fullstack_streaming_ping_pong.BM_StreamingPingPongWithCoalescingApi_MinInProcessCHTTP2_NoOpMutator_NoOpMutator__2M_1_0.opt.new: 1

[microbenchmarks] No significant performance differences


chore(rebuild): re-incorporate header links

Src code patched until
https://github.com/ben-eb/remark-autolink-headings/pull/41 is merged, if
ever, we should probably use a fork.

Ohh, seems Ben isn’t very active in remark anymore. Let me see and check in with him if he’s interested in moving it to @remarkjs

@montogeek not seeing any links pop up in the h* elements... do I need to install the package from that fork in order for it to work?

Nvm used @rigor789's fork, @rigor789/remark-autolink-headings, which works. Cleaning up the styling slightly and then pushing/merging and knocking another one down 🎉. Thanks for digging into this @montogeek.

@skipjack Yep, I patched it locally applying same changes as that fork


change version by SemVer 2

[x] package.json
[x] _VERSION

http://semver.org/
@gaishimo

@gavinzhou
I've changed the version to 0.1.0.
https://github.com/orangesys/orangesys.io/commit/46ce0e90589cb0dddfc6732a58b14214258f240d
https://github.com/orangesys/orangesys.io/commit/b47b1db3afd58aa624e9ed55d9df997f86c14063


Force MAKEFLAGS to --jobs=1 to ensure dependencies are executed sequentially and in correct order

If this is not set, make -j2 web or higher job counts will
cause the build to fail as some dependencies are not expressed
directly on the dependent tasks but as a dependency list
on a parent task.
Alternatively one could add the required dependencies for each
task separately, but that would factually sequentiallize the
build, so there's no real difference except this approach
fixes all dependency chains globally.

Codecov Report

Merging #257 (6c2aa0c) into main (db613f8) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #257   +/-   ##
=======================================
  Coverage   65.53%   65.53%
=======================================
  Files          32       32
  Lines        3122     3122
=======================================
  Hits         2046     2046
  Misses        776      776
  Partials      300      300

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update db613f8...6c2aa0c. Read the comment docs.


Use the atom-text-editor tag instead of the editor class.

In keymaps/rails.cson: Use the atom-text-editor tag instead of the editor class.

Closed by #6


Version of Photino.NET for .NET Framework

I am evaluating how to use Photino.NET in a .NET Framework application (in WPF).
Creating a separate window would work well, I already tested using the message loop in WPF (with .NET 5) and it works.
The issue for .NET Framework is that Photino.NET is compiled for .NET Standard 2.1 - if it was .NET Standard 2.0 it would have worked, but there is a dependency on Marshal.GetHINSTANCE that is not available in .NET Standard 2.0.
Do you already evaluated if there could be a workaround?
The alternative seems to create a .NET Framework version of Photino.NET. If I go that route, would it be possible to keep a single version of the source code with conditional directives, or do you think it would be better to create a separate project?
Any idea and comment would be appreciated - thanks!

We're not sure there are many scenarios that make sense for Photino in WPF. That would limit it to the Windows platform, and WPF already has a wrapper for the WebView2 control. Also, at this point the full framework is becoming legacy. Photino would theoretically work in .NET 5 WPF, although again, we can't envision a scenario for it.

The scenario is adding a visualization feature to https://github.com/sql-bi/VertiPaq-Analyzer that can be used by existing applications that consume that library and are written in WPF or Windows Forms (using .NET Framework).
However, I understand the plan and I will evaluate creating a .NET Framework porting just to support the requirement we'll have in our library.
Thanks!


Interaction error occurred using SAM

My actions before raising this issue

[ ] Read/searched the docs
[ ] Searched past issues

Steps to Reproduce (for bugs)

followed this guide to install CVAT
deployed SAM using this command: cd serverless && ./deploy_cpu.sh pytorch/facebookresearch/sam/nuclio/

Expected Behaviour

I expected that use SAM properly.
Current Behaviour

after I go to localhost:8080 and try to annotate my images with Segment Anything after this message Waiting a response from Segment Anything.. I get this error:

Context

I'm trying to annotate my images using SAM. I installed CVAT and deploy SAM using docker and it worked without any error. when I started to annotate I got that error and have no clue what to do.
Your Environment

Ubuntu 20.04 and the latest version of Google Chrome.
Docker version: Docker Desktop 4.19.0

Do you have checked the logs of the SAM serverless function container?
Use the following command to do so: docker logs nuclio-nuclio-pth-facebookresearch-sam-vit-h

@descilla @aligh97 We are also receiving this error on our self hosted GPU Ubuntu segment-anything model.

For some of our tasks the SAM interactor works fine, and some others it doesn't. Do you know why?

@descilla this is the logs of the SAM serverless function container:

@aligh97
Something killed your process with the model. Do you have enough RAM?

@bsekachev
I have 16GB RAM.
When I open docker and use CVAT, it uses 88% of my RAM.

@bsekachev
I have 16GB RAM.
When I open docker and use CVAT, it uses 88% of my RAM.

@aligh97
docker stats --no-stream  ?

for me:
CONTAINER ID   NAME                                           CPU %     MEM USAGE / LIMIT     MEM %     NET I/O          BLOCK I/O   PIDS
605d9b8c99a1   nuclio-nuclio-pth-facebookresearch-sam-vit-h   185.49%   161.7MiB / 7.636GiB   2.07%     266B / 0B         0B / 0B     37

Docker has its own limits for RAM usage. As you can see it allows to use up to 7.6 GB.
I suspect it is not enough when model is runned and docker kills the process.
Try to increase memory limits, modifying function.yaml according to this article:
https://nuclio.io/docs/latest/reference/function-configuration/function-configuration-reference/

When SAM is running it show consumption up to 10 GB on my PC.

oh, I see. then it's most likely the limits for RAM usage.
I will try to increase memory limits.

@bsekachev @aligh97 Do you know which code or script I need to modify and run to increase the memory limits?
Thanks, Sam :)

@bsekachev You need to create a .wslconfig file in your profile directory(open your File Explorer, and type and enter %UserProfile% to go to your profile directory in Windows.) and then add the following commands:
[wsl2]
# Limits VM memory to use no more than 4 GB, this can be set as whole numbers using GB or MB
memory=12GB

You can see this link and set your configurations.

@aligh97 I'm using linux ubuntu

I guess this command line sets the maximum RAM usage:
sudo docker run -it --memory="[memory_limit]" [docker_image]

The value of memory_limit should be a positive integer followed by the suffix b, k, m, or g for example --memory="12g"

how can i fix this problem   I'm trying to annotate my images using yolov8 seg, please help me~


Where are the stubs hosted?

This project seems empty...

The package itself is hosted on PyPI, but seems like they forgot to publish the source here!

I have redone the stubs with better typing and put it on typeshed, you can find it on types-paho-mqtt.


feat: add synchronization statistics

Adds a new structure to the Manager that captures the following
information:

Number of files updated
Number of bytes written
Number of files deleted
Time to complete the sync

This object can be retrieved via the convenience method GetStatistics()
from the Manager.

@at-wat can you take a look please?
Thanks!
/Jordi

/assign @at-wat


feat: added schema.sql to examples

What kind of change does this PR introduce?
SQL schema file added
What is the current behavior?
Very minor feature: I noticed that the v2 user-management react example has a schema.sql file and the others do not.
What is the new behavior?
I've added a schema.sql file to all user-management v2 examples so they are in parity.

We should rather remove the SQL schema.sql from the React version since this is already written in the README.md of each repo.

@silentworks - Cool, I've just deleted all the schema files.


improve "openshift-router-proc" check

Currently the check openshift-router-proc is for masters:
//Proc
"openshift-router-proc": {
  "command": "/usr/local/bin/check-process.rb -p 'openshift-router' -C 1",
  "subscribers": [
    "openshift-master"
  ],
  "handlers": ["mailer", "flowdock"],
  "occurrences": 3,
  "refresh": 86400,
  "interval": 60
},

This may be an issue when:

the number of replicats for routers is less than the number of masters (you will receive false-positive CRITICAL)
infra node for routers are not necessarly masters

This commit start the work to avoid this situation:
https://github.com/redhat-cip/rcip-openshift-scripts/commit/474d00c49f14a3d5a1a02b6a19ec6f293324bedf

ok for me


Android 12 needs exports on the intent-filter

Apps targeting Android 12 and higher are required to specify an explicit value for `android:exported` when the corresponding component has an intent filter defined. See https://developer.android.com/guide/topics/manifest/activity-element#exported for details.
This plugin registers the following receiver:
      <receiver android:name="nl.xservices.plugins.ShareChooserPendingIntent" android:enabled="true">
        <intent-filter>
          <action android:name="android.intent.action.SEND"/>
        </intent-filter>
      </receiver>

Thus needing the android:exported attribute

same issue here I am using 6.0.3 version

Apps targeting Android 12 and higher are required to specify an explicit value for `android:exported` when the corresponding component has an intent filter defined. See https://developer.android.com/guide/topics/manifest/activity-element#exported for details.
This plugin registers the following receiver:
      <receiver android:name="nl.xservices.plugins.ShareChooserPendingIntent" android:enabled="true">
        <intent-filter>
          <action android:name="android.intent.action.SEND"/>
        </intent-filter>
      </receiver>

Thus needing the android:exported attribute

Hi, is this something we need to add to the confi.xml?
I can no longer email (exception for canShareViaEmail). From https://github.com/EddyVerbruggen/SocialSharing-PhoneGap-Plugin/issues/1150, I added the following...

But now the app just crashes on startup

Please resolve Targeting S+ (version 31 and above) requires that one of FLAG_IMMUTABLE or FLAG_MUTABLE
SocialSharing.java
final PendingIntent pendingIntent = PendingIntent.getBroadcast(cordova.getActivity().getApplicationContext(), 0, receiverIntent, PendingIntent.FLAG_UPDATE_CURRENT|PendingIntent.FLAG_IMMUTABLE);

opened a PR
https://github.com/EddyVerbruggen/SocialSharing-PhoneGap-Plugin/pull/1158

Has this issue been solved. I can't upload my app to playstore because of this

To my knowledge, "android:exported = true" is the default if there is an intent-filter.
'''
<receiver android:name="nl.xservices.plugins.ShareChooserPendingIntent" android:exported="true" android:enabled="true">

'''


Benchmarking experiments 2

Let's try again with a new branch.  Again, this doesn't change any code, just a comment.

/benchmark plutus-benchmark:validation

/benchmark plutus-benchmark:validation

Oops: I didn't merge the necessary change.


Seems that tests may be failing in some timezones for month and year

See https://github.com/mattpat/node-schedule/pull/86
I think that having travis test every pull request would probably help so there is a consistent test environment.

@jonhester Can you see if https://github.com/mattpat/node-schedule/pull/89 works for you?

With this changes the tests pass for me with and without your changes

Works for me. I think that it's still better to use the correct values for day and year length.

I'm not sure I agree. Imagine in the 3 months period we include July, August and September. Having July and August 31 days the test may fail. I think better be safe and use 31 days and 366 days for those tests. What do you think?

Easy to test. Just change the date.

On Feb 27, 2015, at 12:59 PM, Santiago Gimeno<EMAIL_ADDRESS>wrote:
I'm not sure I agree. Imagine in the 3 months period we include July, August and September. Having July and August 31 days the test may fail. I think better be safe and use 31 days and 366 days for those tests. What do you think?
—
Reply to this email directly or view it on GitHub https://github.com/mattpat/node-schedule/issues/88#issuecomment-76441115.

#89

#2c3a9f197ada23d145c884984315042d443a6bdc

2c3a9f197ada23d145c884984315042d443a6bdc

After doing some tests, the current approach was still failing depending on the date the tests were running on. I've tried to solve the issues in https://github.com/node-schedule/node-schedule/pull/94/


Indicate trophy case moved to a separate dedicate repository.

I did this for a couple reasons:

It felt a little misplaced to have the trophy case on cargo-fuzz. It's possible to fuzz using libfuzzer without cargo-fuzz, which is what some people have done
I wanted a single unified index for all bugs found fuzzing Rust code, so it's easy to find all the previous fuzzing efforts done

Also, I already did this for afl.rs:
https://github.com/rust-fuzz/afl.rs/commit/7537c88a6b0397e4f4995c547f94e0492b51a06f

bors r+

kinda feel like it's important to show it off on the main repo but w/e


Slow speed on WebKit Browsers

Hi.
When using Firefox the rendering proccess is very fast, but when using wekbit browser ( Chrome or Opera ) the speed is way slower. Why is that and is there something we can do about it?
Thanks.

I'm seeing this right now in a bad way -- we're talking a fifteen to twenty seconds on Firefox versus several minutes on Chrome.
Any chance it's related to canTransfer here?

Hello friends, I spent the weekend digging into the Chrome performance issue so I figured I'd give you all an update. This is a separate issue from the dithering issue.
I discovered the following:

Not related to canTransfer or postMessage() transfer efficiency
Chrome web workers will begin to slow down by about 10X after they have "chugged through" enough code or data. In my tests on my Macbook Pro & Chrome 45, a given worker would pass the threshold after rendering about 5 frames if the frame size was >= 720x1200.
Profiling a Chrome worker showed that NeuQuant.contest was taking the majority of the time during these slow downs.

I tried overly optimizing contest() in a few ways, and also tried porting the entire thing to pure javascript and compiling together with closure ADVANCED OPTIMIZATIONS, but couldn't fix the slow downs.
The hack fix that I was trying to avoid ended up being a simple re-creation of workers after N frames. Here is the code I have in JavaScript. My project is running on the raw javascript so I don't have it in coffee, but someone could easily patch this into gif.worker.coffee lines 103-109 if it's not too much of a hack.
var worker;
console.log("spawning worker " + i);
worker = new Worker(_this.options.workerScript);
worker.framesProcessed = 0;
var onmessage = function(event) {
	_this.activeWorkers.splice(_this.activeWorkers.indexOf(worker), 1);
	worker.framesProcessed++;
	// Re-create worker every 4 frames for big frames in chrome, since each worker starts to go
	// incredibly slow after about 5 frames
	if (browser.name === 'chrome' && event.data.width * event.data.height > 400 * 400 && worker.framesProcessed >= 4) {
		worker.terminate();
		worker = new Worker(_this.options.workerScript);
		worker.framesProcessed = 0;
		worker.onmessage = onmessage;
	}
	_this.freeWorkers.push(worker);
	return _this.frameFinished(event.data);
};
worker.onmessage = onmessage;
return _this.freeWorkers.push(worker);

Since this seems to depend on the amount of data processed, with larger images resulting in slowdowns earlier in the render process, it sounds like this is an issue with leaking memory within the workers and/or Chrome's garbage collector not kicking in properly.
Others have reported similar issues with Web Workers and Chrome, albeit back in 2014: http://stackoverflow.com/questions/24708649/why-does-web-worker-performance-sharply-decline-after-30-seconds


Currently throwing a low bucket error despite full bucket

The dashboard is showing a low bucket error
https://quorum.tedivm.com/console
Error: Extremely low bucket - aborting script run at top level
at main:4:9
at main:77:3
at Object.exports.evalCode (<runtime>:15818:76)
at Object.requireFn (<runtime>:46479:28)
at Object.exports.run (<runtime>:46422:60)

Might be shard related? Because the processes do seem to still be running and the bucket shows full.

And... it just stopped...

This happens when a creep manages to fall into another shard via a portal. Since no CPU is allotted to the other shard the bucket ends up being completely empty. The messages stopped when the creep died- at that point there were no resources on the other shard so no scripts ran.