metrics.circonus: Add support for circonus.submissionurl

There are times when nodes that fabio are deployed on already have a circonus metrics agent added to them. Following the pattern that Consul, Nomad and Vault have taken, we can actually send the application metrics to the local agent by using the SubmissionURL
I am currently using this in our environment with the config as follows:
metrics.target=circonus
metrics.circonus.submissionurl=http://<IP_ADDRESS>:2609/write/fabio

Thanks Paul!


fix: changes in UI of password text input in e-CAS modal

the eye icons are added to see the password in text input box and modified the height of textIput box

Closing as already update in main


What would a hypothetical sodium_compat v2 look like?

As PHP 8.1's release draws nearer, we're looking at the long tail of backwards compatibility (5.2.4+) and wondering if we should ever create a new major version of sodium_compat that uses features from newer versions of PHP. If we ever did that, what would the new minimum version be, and what would motivate such a divergence?
The most attractive candidate for a hypothetical sodium_compat 2.x is to set the minimum to 7.0 and use strict typing everywhere.
Another attractive candidate is 7.4 with ext-FFI, which would allow us to polyfill scrypt/argon2 effectively. (But, alas, that also requires writing C code.)
If we ever did split the codebase, we'd have to commit to maintaining the 1.x branch until WordPress drops completely support for PHP 5.6. (They consider 7.2 the minimum acceptable version today, but I don't foresee any immediate changes that actually break older versions landing in core without a long discussion leading up to it.)
Is it even worth doing?
That depends entirely on whether or not ext-sodium uptake becomes ubiquitous. From the direction large PHP projects (i.e. Symfony) are taking, this seems likely. If there's a significant lapse between Sodium adoption and PHP 7.2+ adoption, that can be interpreted as a signal that we should consider a newer major version in the first place. If there isn't, we're probably better off focusing on more impactful work.

I've opened a draft PR in #154 that answers this question.


Function langugae detection error leads to deploy failure

Describe the bug
Static web app (Vite/VueJs) + Azure functions deployment fails with error (presumably from Oryx)
Error:
The function language detected is unsupported or invalid. The following languages are valid: dotnet, dotnetisolated, node, python. If you are not using Azure Functions, you can skip this check by removing the 'api_location' input from the workflow file.

A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Create a static web app using VueJs
Add a Azure cloud function
Generate the workflow file
Upload to github actions
See error in deploy log

Our GitHub Actions or Azure Pipelines workflow YAML file

name: Azure Static Web Apps CI/CD

on:
  push:
    branches:
      - main
  pull_request:
    types: [opened, synchronize, reopened, closed]
    branches:
      - main

jobs:
  build_and_deploy_job:
    if: github.event_name == 'push' || (github.event_name == 'pull_request' && github.event.action != 'closed')
    runs-on: ubuntu-latest
    name: Build and Deploy Job
    steps:
      - uses: actions/checkout@v3
        with:
          submodules: true
      - name: Build And Deploy
        id: builddeploy
        uses: Azure/static-web-apps-deploy@v1
        with:
          azure_static_web_apps_api_token: ${{ secrets.AZURE_STATIC_WEB_APPS_API_TOKEN_LIVELY_POND_0F8C8B91E }}
          repo_token: ${{ secrets.GITHUB_TOKEN }} # Used for Github integrations (i.e. PR comments)
          action: "upload"
          ###### Repository/Build Configurations - These values can be configured to match your app requirements. ######
          # For more information regarding Static Web App workflow configurations, please visit: https://aka.ms/swaworkflowconfig
          app_location: "/" # App source code path
          api_location: "api" # Api source code path - optional
          output_location: "dist" # Built app content directory - optional
          ###### End of Repository/Build Configurations ######

  close_pull_request_job:
    if: github.event_name == 'pull_request' && github.event.action == 'closed'
    runs-on: ubuntu-latest
    name: Close Pull Request Job
    steps:
      - name: Close Pull Request
        id: closepullrequest
        uses: Azure/static-web-apps-deploy@v1
        with:
          azure_static_web_apps_api_token: ${{ secrets.AZURE_STATIC_WEB_APPS_API_TOKEN_LIVELY_POND_0F8C8B91E }}
          action: "close"

Expected behavior
App should deploy with functions alongside (this previously worked, even this morning).
Note: The repository is private but I'm happy to assist by providing logs and attempting changes.

Full log gist is here:
https://gist.githubusercontent.com/Cypher1/f95862ccb5dce9f5324bf0038b55d938/raw/01a02c67d21ee4ecd5c1b8bbe8f074583ae4a624/Deploy.log

We found a work around / awkward solution.
Setting the apiRuntime in staticwebapp.config.json is necessary, even though Oryx correctly detected the api language, and even built the code (and previous deploys worked...)
Not sure how this is possible but we have a path forward. Best of luck to anyone who finds this, it looks like Azure don't follow this issues page much.


📝 Documentation: Mention what Tidelift does

Bug Report Checklist

[X] I have pulled the latest main branch of the repository.
[X] I have searched for related issues and found none that matched my issue.

Overview
Tidelift is a great thing but not everyone knows what it does. The docs here should mention it's a service that lets companies put money into a fund that gets distributed amongst their open source dependencies. Those dependencies need to show they do some good practices around maintenance & security in order to receive funds. It's a really good idea IMO - win win!
Additional Info
Thanks @matthew-dean for pointing out that this is never mentioned anywhere!
Sibling issue to https://github.com/JoshuaKGoldberg/tidelift-me-up-site/issues/18.

Hey Joshua, just went through your Github and was stunned by the consistency that you have kept in your work.
I am 3rd year student pursuing Bachelor's degree in Information Technology. Currently I am exploring and trying to contribute to tech domain through Open Source Contributions; and starting with something simple would be a great. So can you assign the documentation correction to me? Thank you for considering my request.

❤ thanks so much @NilanchalaPanda! I'm excited you're exploring open source, that's awesome. I actually just wrote a blog post about contributing to my repositories such as this one: https://www.joshuakgoldberg.com/blog/contributing-to-a-create-typescript-app-repository. It might be useful for you.
I don't use any kind of assignment / issue-taking situation for repos (https://github.com/JoshuaKGoldberg/tidelift-me-up/blob/b95f34eab9b176ca3c33b363c2066f5dae6f05da/.github/CONTRIBUTING.md#issue-claiming). So if you want to send a pull request, go ahead!

 Thank you for sharing your blog post about contributing to repositories, I will definitely check it out. I will go ahead and send a pull request if I have any changes or improvements do suggest.
Once again, thank you for your guidance and open approach to contributions!


Show original error and stack trace in logs for handleRetryableExcept…

GH 128 - Could provide errorMessage of AwsServiceException When handleRetryableException #128
Description:
Modify error logs to add throwable stack trace for retry method

Please add Jira number in the PR title.

Please add Jira number in the PR title.

External PRs should have a reference to the GitHub Issue. Please do not include in the PR internal Jira ticket numbers.

PR description should be specific. Also, please do not include links to internal Jira tickets. Jira tickets should have a reference to PR, not PR to Jira issues.
Example:
This PR addresses enhancement - #128 by adding ability to log exception when

Solution:

Added exception as an argument to slf4j logging, i.e. logger.error("description of the error", e) where _e_is an Exception.

Tested performed:

Example of log generated before and after:


Media Query Team: Create Fixed Max-Width Style Component

Title: Implement Fixed Max-Width Style Component

This is needed for uniform content presentation across all pages, especially ultrawide screen resolutions.

Task:
Create a reusable style component to:

[x] Set content max width to 1440px.

[x] Center content at this point and onward so that it does not keep stretching out on ultrawide screen resolutions.

[x] Ensure header and footer backgrounds (bg-color) extend the full width of the page, while their main content (text and such) remains constrained to the max-width limit like other page content.

[x] Fill excess space on screens wider than 1440px with white background (body)

[x] Follow QA dev rules

[x] Test

[ ] Checked by QA

This component will be used for all content that goes on pages

Example image from VG.no on ultrawide resolution:

I checked up on the max-width and it seems that the some of the content inside is larger than max-width 1440, which may also be the reason on why there is horizontal scrollbar. Think this should be looked at a bit more.


ETHMNY  please add logo

0xbF4a2DdaA16148a9D0fA2093FfAC450ADb7cd4aa

Sorry but your PR is not valid, you should only add new contract detials

Closing this because in addition to @MRabenda 's comment above, the base of this PR is wrong. The base should be master. Instead this PR had the feature branch as the base.


Pollution of browser console logs

During development the console logs are spammed with res values due to this console log: https://github.com/forge42dev/Remix-Dev-Tools/blob/66d252e83cd66738b9ee2ab9123fd16911caa42f/src/client/components/jsonRenderer.tsx#L33
I patched it to remove that, but it would be great if this could be configured in the client options.
diff --git a/dist/client.js b/dist/client.js
index 6bdc7f3a7c642679d6a13b0d6b62d49d4a09e8a9..c1108e51664da96bb781084f8fc77160bfb651e5 100644
--- a/dist/client.js
+++ b/dist/client.js
@@ -3446,7 +3446,8 @@ var JsonRenderer = ({ data }) => {
         value.then((res) => {
           if (!ref.current)
             return;
-          console.log(res);
+          // see https://github.com/forge42dev/Remix-Dev-Tools/issues/158
+          // console.log(res);
           setJson((json2) => ({
             ...json2,
             [key]: res

Oh wow this is not supposed to be configured but removed,  I'll release a patch release today!

Thanks @AlemTuzlak


Slim 4 Beta Release

Slim 4.0.0-beta Release
We are excited to announce the Slim 4.0.0 beta release. Please direct all your feedback for this release to the Slim 4 Beta Release Feedback Thread. The new docs are located here.
Stability Status
Very stable. No bugs have been reported since alpha release, we have also increased test coverage to 99% (I know, it's not necessarily a great metric but fantastic achievement nonetheless). Thanks @adriansuter for all the work.

Changelog Since Alpha
Added

2674: Add suggestions for possible PSR-7 implementations to composer.json thanks to @michaelbiberich
2689: Add cacheFile to RouteCollector constructor thanks to @bednic
2683: 4.x - Psr17Factory Provider & Slim-Http Auto-Detection thanks to @l0gicgate
2684: 4.x - Add documentation for Slim-Http automatic decoration to README thanks to @l0gicgate
2688: RouteCollector add optional cacheFile parameter to constructor thanks to @bednic

Changed

2695: 4.x - Add test case for MethodOverrideMiddleware::process() for eof body stream thanks to @adriansuter
2692: 4.x - Add test case for RoutingMiddleware::performRouting() thanks to @adriansuter
2681: Remove empty line thanks to @adriansuter
2680: Remove empty line thanks to @adriansuter
2679: 4.x - Reformat README thanks to @l0gicgate
2675: Move autoload-dev files to tests bootstrap file thanks to @l0gicgate

Fixed

2678: Fix issue in README thanks to @jenssegers

Removed

2694: 4.x - Remove Example Folder thanks to @l0gicgate
2693: 4.x - Remove Unused Dispatcher Parameter thanks to @l0gicgate

Supported PSR-7 Implementations CI Status
Thanks @adriansuter who created a Slim4-CI repository to ensure that the main 4 PSR-7 implementations our AppFactory supports can be integrated correctly.

#
PSR-7 Implementation
Status

1
Slim PSR-7

2
Nyholm

3
Guzzle

4
Zend

Note: Travis-CI is configured to be triggered automatically at least every 24 hours.

Slim 4 DDD Skeleton
@l0gicgate  created an app skeleton with a DDD style directory structure, example files and test coverage. It is a very opinionated and we're not sure yet if it is the right fit for a skeleton but we would like some feedback. You can clone the skeleton and try it out:
git clone https://github.com/l0gicgate/Slim-Skeleton.git
git checkout 4.x

Slim 4 MVC Skeleton
@adriansuter created an MVC skeleton. You can clone the skeleton and try it out:
git clone https://github.com/adriansuter/Slim4-Skeleton.git

Coverage remained the same at 99.745% when pulling 68f1bfc7d84d08f2d62d0909c6141ca11f6b2e00 on l0gicgate:4.x-BetaRelease into 73118afc665d7cd4dc0911d7a959df229bbf0cfb on slimphp:4.x.


Understanding

Hola, perdoname pero mi castillano no es el mejor entonces puedo ententar de hablerte asi, pero creo que voy a hablar ingles para proseguir. Si no me entiendes, puedo ententar mas xD
I'm on the hunt for a solid repository for my Final year project, doing a BSc CS degree, my topic is, Breast cancer/tumour detection at early stages, hence mammograms. I'm intrigued by your work, it's looks very good, I just would like some guidance as it is all in Spanish, so comments help until a certain point.
So far I only managed to find super-advanced material, e.g., from Adam Yaala or Adam from St. Andrews, the latter has an exquisite dissertation where it explains everything nicely. I suppose you have the same, although I could try to follow it, or very basic projects that 100 people have done, i.e., the Wisconsin data set.
Looking forward to hearing from you
Regards
Gabriele

Hey, Gabriele!
Sorry for the late reply. Haven't been here for a long time.
Unfortunately, I'm afraid I can't help you with your request.
I'm only now discovering this DL applied to the medical physics world (I'm pursuing a master's in radiodiagnosis).
The work I have on here I've got from marcelowukong. He works with DL applied to numerous projects and offered to kindly give me a hand with the a model for the dataset I've got online (CBS-DDSM).
I could, however, present your doubts to Marcelo - and maybe transmit his knowledge to you?
Let  me know if you are interested.
Best regards,
Giselle


heatmap does not reflect all contributions

Description
No response
Gitea Version
1.17.2
Can you reproduce the bug on the Gitea demo site?
No
Log Gist
No response
Screenshots
This is what I can see from the dashboard:

But entering at that very same repo, I can see that more contributions have been made recently:

As it can be seen, the user is the same in both screenshots
Git Version
No response
Operating System
Linux
How are you running Gitea?
Self-hosted
Database
MySQL

Hello,
That may be related, but I'm witnessing the same kind of issue when just after logging in, and choosing the correct context, the displayed list is not showing anymore [1] the updated list of commits.
But when entering in each repo, the header line is showing the last commit action.
[1] This has worked for years, and stopped maybe around 1.17.something (let's say maybe 4 months ago)

It seems that your Git Hook doesn't work.
Could you try the FAQ first https://docs.gitea.io/en-us/faq/#push-hook--webhook-arent-running ?

I just noticed that it has not worked for me for a few months. Is there a way to display the missing months?

https://docs.gitea.io/en-us/faq/#push-hook--webhook-arent-running


Allows Grafana dashboards to be embedded

In order to be able to embed grafana iframes, we need to tell grafana to allow that.

Thanks i was struggling with this


This error appears when try to choose Terminal: Select Default Profile

Does this issue occur when all extensions are disabled?: Yes/No

VS Code Version: 1.59.0 (user setup)
OS Version: Windows 10 Home Single Language

Steps to Reproduce:

F1
Terminal: Select Default Profile

Error appears as a pop up message:
Command 'Terminal: Select Default Profile' resulted in an error (Cannot use 'in' operator to search for 'path' in Visual Studio 2019 Dark)

what are your configured terminal profiles? IE terminal.integrated.profiles. ?

The IE terminal.integrated.profiles.  is stated below
Windows:
{
        "terminal.integrated.profiles.windows": {
        "PowerShell -NoProfile": {
          "source": "PowerShell",
          "args": ["-NoProfile"]
        },

any errors in the dev tools console?

Yes, there are 8 errors.

Please provide them

Thanks . Have you tried disabling all of your extensions?

Yes, I disabled all extensions and I have faced the same problem.


chore: upgrade Eslint parser to 6.x

Upgrade eslint parser to 6.x
Upgrade eslint to 8.48.0

Updates #92

Can someone take care of this PR? Currently stencil-eslint prevent us from upgrading to latest version of eslint

Hi this will be very helpfull to upgrading eslint thanks in advance

Hi @raphaelpor,
I am tagging you because I am not sure who maintains this repository, but you seem to have been the last person to merge Pull Request...
I am aware that this is a community plugin and that probably nobody is paid to work on it, and so I am only asking whether you have an idea of when you or others would have the time to look at this PR and make a new release, because as others mentioned above, this is preventing us from upgrading eslint. If it won't be merged for months, then we might choose to drop using the plugin (sadly), but if it were just a question or a few weeks, we'll just keep on patiently waiting.
Thank you for any answer, and thank you for your work on this plugin.

Hi @raphaelpor,
I am tagging you because I am not sure who maintains this repository, but you seem to have been the last person to merge Pull Request...
I am aware that this is a community plugin and that probably nobody is paid to work on it, and so I am only asking whether you have an idea of when you or others would have the time to look at this PR and make a new release, because as others mentioned above, this is preventing us from upgrading eslint. If it won't be merged for months, then we might choose to drop using the plugin (sadly), but if it were just a question or a few weeks, we'll just keep on patiently waiting.
Thank you for any answer, and thank you for your work on this plugin.

@cmotsn Thanks for your comments.
I have been busy with other projects but I will spend some time with this, in the coming days.
I just wanna make sure that this is compatible with Stencil V2, before approving it. :)

Thanks a lot @raphaelpor !
And thanks also to @philippesic for the PR!


code highlighting

Missing the css part for code highlighting.

ah no duh, it's missing from foot https://github.com/rmsrosa/booksjl-franklin-template/blob/2ecab12bfefd000f206fac14911b18555bc365fb/_layout/foot.html#L6-L10

Yeah, I don't like the font either, and the font size is too small compared with the rest of the text. I will look into that after I finish with the integration with Weave.


Send device type in chat.json API

The type of device ie, "iOS" is  required to get device wise usage analytics.
Steps to reproduce it
NA
LogCat for the issue
https://api.susi.ai/susi/chat.json?q=hello&language=en&access_token=PGbzC5zYQBQUuCrYrarqIuRe4lvjfn&latitude=30.1667&longitude=77.3
Screenshots of the issue
NA
Would you like to work on the issue?
Yes

sir i want to code for this issue.

Okay go ahead

@tusharjindal353 is there any progress in it?

sir where i have to send the device type and model??

@AnupKumarPanwar I am taking this issue


Add notifications for play/pause/stop

As all of the Android applications have them

https://blog.xamarin.com/background-audio-streaming-with-xamarin-android/

Yes, this functionality is similar to something I've expected in a first contact with the app.
Going to recently played episode would be great and for me it would be even in the application main menu. I've looked for this in a first contact with the app.


Reimburse expenses - Error continues to appear when changing the rate after it was modified

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Issue found when executing PR https://github.com/Expensify/App/pull/12162
Action Performed:

Navigate to any existing workspace on device A > Reimburse expenses. Go offline.
Switch to device B. Sign into the same account, and navigate to the same workspace > Reimburse expenses.
Edit the Rate input
Observe it shows in the pending state
Switch back to device A. Make sure you're still offline. Edit the same rate field.
Go back online on device A
Observe the unit field fails to update because the workspace was modified while you were offline, displaying the RBR and error message
Attempt to update the rate field again while online.

Expected Result:
You are able to modify the field and no errors show.
Actual Result:
Error continues to appear when changing the rate after it was modified offline
Workaround:
Unknown
Platform:

Where is this issue occurring?

Web
iOS
Desktop App
Mobile Web

Version Number: <IP_ADDRESS>
Reproducible in staging?: Yes
Reproducible in production?: Yes
Email or phone of affected tester (no customers): any
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://user-images.githubusercontent.com/93399543/199835238-84dfcab6-2090-4b5f-8d53-44f2d11e99be.mp4
https://user-images.githubusercontent.com/93399543/199844891-327d1fb6-81c2-4940-b1d4-a8ec9d296a2e.mp4
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal
Slack conversation:
View all open jobs on GitHub

This was my PR so I can investigate. Seems something changed between when I tested it locally and when it was deployed.

Hmm, looking at the issue it's probably best to wait for https://github.com/Expensify/App/pull/12219 to be merged + deployed first so we can be more sure of what the issue is. I think the offline client may seem to have the wrong value after coming back online because the order of writes and reads is incorrect.
This happened for me locally, but after pulling the changes from https://github.com/Expensify/App/pull/12219 that seemed to fix things since it ensures read requests run last.
I did notice a separate issue while having those changes checked out, but I think I know how to rectify it and can open a separate web PR.

Sheesh, fun one! Okay cool, so we should mark this as internal right?

sure, I can QA both PRs since I have the most context about what to look out for

#12219 has merged, so #35433 should come off hold imminently.

Issue not reproducible during KI retests. (First week)

https://github.com/Expensify/App/pull/12219 is on staging. #35433 is reviewed, and should come off hold imminently after the prod deploy.

https://github.com/Expensify/Web-Expensify/pull/35433 was just deployed to prod an hour ago. Testing this out again

Hmm it seems like now I can't even get the error to appear, strangely the API just accepts the new updates from the client that was offline. Perhaps something has changed since https://github.com/Expensify/Web-Expensify/pull/35433 was being deployed, I'll investigate.
https://user-images.githubusercontent.com/31285285/202007556-a0e1739a-b025-4424-85c4-7e67cb764718.mov

Issue not reproducible during KI retests. (Second week)

Seems like a PR definitely went out that broke the error displaying, it was working for me on dev, but then I pulled main on App and it broke.

Issue is reproduced today with build <IP_ADDRESS>
https://user-images.githubusercontent.com/93399543/203875418-df7a6532-000a-4647-9872-9335078dbcc8.mp4

Taking a look into this one today!

@kbecciv it seems to actually be working as expected when I tested it on v1.2.32-1
https://user-images.githubusercontent.com/31285285/204342904-64537b21-1da0-4385-8b3e-9f99706de728.mov
The final values on both clients are correct. Don't mind the replay effect, that's tackled in https://github.com/Expensify/App/issues/12775. Subsequent updates to the values also work.
For these types of issues that require two clients to be used, can you please provide both clients in the screen recording? It's not clear to me what the values were set to on the other client, and therefore it's hard to verify whether the behavior of the offline client is correct. Thanks!


Adopt RDF/Sparql's entity format

Entities are specified by using <entity> or prefix:entity. When a prefix is used, no URI is associated with it, and the prefix itself is used for sorting etc. This should be updated to match Sparql, to allow URIs to be associated with prefixes, and to handle sorting correctly. How this ends up encoded in the eventual KV store key needs some thought as well.

It seems to be standard for prefix:entity to be expanded to a full URI for storage in SPARQL engines. Users can define prefixes however they like in an input .ttl file, in general an engine would have to expand to URIs at insert time.
Upsides to expanding:
a. Common implementation approach / the standard assumes (but does not require) QNames are stored as expanded URIs.
b. Versioning can be handled by just changing a prefix definition in a query or .ttl file. Not something you see people do very often AFAICT.
c. Easier to support N-Quads files and the like where prefixes are already expanded - or .ttl files that don't consistently use prefixes.
Some downsides:

Not being able to get short URIs out easily without post-processing.
Having no enforcement of standard prefixes for a given graph.
Every query and input file needs to specify all prefixes.
More space/time cost for the engine and all downstream tasks.
More complexity for downstream clients - they may need to know prefixes to do any URI matching unless they use an RDF helper library.

In balance I wonder whether it's worth deviating from standard practice here.
One useful feature would be to allow standard prefixes to be defined in the store to simplify use. Side note: Blazegraph supports some [default prefixes],(https://www.mediawiki.org/wiki/Wikidata_Query_Service/User_Manual#Automatic_prefixes) so you don't need to specify them in queries.


Patch size

Hi!
Sorry for disturbing you. I am a bit confused about the patch size setting in your paper.In  L2-Net and HardNet,patch size is 32X32,but your patches  seem to be 12σ × 12σ. I have seen LIFT. But  I still do not fully understand the relationship between patch size in pixel and σ.Could you give me some suggestions？

What is the patch pixel size setting in your project？

Sorry for the late response. Basically, σ denotes the feature scale, which can be obtained from a keypoint detector that constructs image pyramid, such as SIFT. The size of a patch is always 32x32, but the sampling size is 12σ × 12σ. Perhaps the code will better explain its effect.

I also write an example to address this issue. Please refer to this discussion.


Release Google.Cloud.Billing.V1 version 3.4.0

Changes in this release:
Breaking change
The resource annotation for GetProjectBillingInfoRequest.Name has changed from referring to a ProjectBillingInfoName to a ProjectName. This means GetProjectBillingInfoRequest.ProjectBillingInfoName has been removed, and GetProjectBillingInfo.ProjectName has replaced it, and the overloads to the GetProjectName method have changed accordingly.
Although this is clearly a breaking change (as removing a public property always is), we have not taken a major version bump as any customers whose source code is broken by this would already have been broken when making a request. Customers whose code is already working will not be affected.
Bug fixes

Fixed resource_reference for name in GetProjectBillingInfo (commit 58d6168)

Documentation improvements

Update comments (commit 5de2c9b)

Created release for Google.Cloud.Billing.V1-3.4.0


Find-VisualStudio.cs cannot be found in Plug'n'Play environments

In Plug'n'Play environments, e.g. Yarn berry, the Find-VisualStudio.cs [erroneously] cannot be found, because the runtime logic uses __dirname a la path.resolve(__dirname, '..', 'deps', 'Find-VisualStudio.cs'). In PNP environments, the module may not exist in a literal directory. For example, the module may exist inside a .zip file and the patched require/import unzips the module. Unlike PNP, the path to Find-VisualStudio.cs is passed to powershell.exe, which cannot unzip it the way a patched require/import can.
Simple solution
Allow the path to Find-VisualStudio.cs to be injected via an environment variable. This quickly unblocks anyone impacted, by allowing them to generate a reference to the file. For Yarn PNP, a build step to copy the file to a local directory would solve this.
Proper solution?
The best choice may be to copy the file to a temporary path, like writeFileSync(path.join(process.env.TEMP, 'Find-VisualStudio.cs'), ...). I can't propose a perfect implementation of this that would scale as quickly and easily as the simple solution.

This seems more like a deficiency in Yarn. I'd be more open to something like adding such files to a list of files to always unpack in package.json, but I don't know enough about Yarn to know whether they support something like that.

I don't think it's a Yarn deficiency.
It's not a JavaScript standard, however, that the path to a module also exist on disc. For example, require('node:assert') is a valid module path, but you won't find it on disc (require.resolve('node:assert') === 'node:assert'). This issue arises as buildcheck passes the path to a module (generated based on Node's resolution logic) to a non-Node executable, which is then unable to find it on disc.
I think the distinction here would be:

NPM standards place the onus of writing to disc on the package manager.
JavaScript standards place the onus of writing to disc on the package vendor.

I'm just trying to extend support. Within a valid JavaScript ecosystem, it is possible that there is not a file path to the Find-VisualStudio.cs file bundled in this package.

I'm willing to bet there are quite a few packages/modules that rely on bundled non-require()'able files. IMO it would make more sense for Yarn to come up with a sensible, low barrier, standardized solution for such packages to implement instead of expecting each of those packages to bend to Yarn's (or any other NPM alternative's) will.

instead of expecting each of those packages to bend to Yarn's (or any other NPM alternative's) will.

Just to clarify the perspective here, there is nothing about a "package manager's will" about this request. buildcheck is relying on NPM's implementation to reference a file path.
The JavaScript language itself does not specify this same behavior. For other JavaScript ecosystems to adhere to this behavior, it would require those ecosystems to bend to NPM's will.
The pull request #5 is ecosystem-agnostic. It does not require this module to stop using the NPM implementation.


Switch from Travis CI to Github Actions

This PR switches from Travis CI to Github Actions for running CI.
With the use of matrix, I'm also taking this opportunity to start testing on JDK versions 1.8 through 14 (with 15 blocked by an update to JaCoCo).

Nope, nothing super specific! Jitpack handles publishing, so all CI needs to do is run the tests, and call Danger if it can.
For security purposes, Danger will only work right now for PRs originating from inside the project, and not from outside PRs.
That should be possible, but will require some more delicate setup to avoid security risks:
https://securitylab.github.com/research/github-actions-preventing-pwn-requests

Thanks @dperez37 and @shadyoak !


Question:  how do I get relative paths to show instead of full paths

I am currently using "Or, With Gulp" option almost verbatim as per your example.  However if I click on the source button.  I see the full path of the file.  i.e. c:\ .....\folder\file.js.  Is it possible to use relative paths i.e. only \folder\file\js?

As you see in the generated document, it does not use absolute path for href attribute.
https://github.com/allenhwkim/angular-jsdoc/blob/master/default/docs/ngmap.MapController.html#L42
        source
This is the working example, and check the html source code.
https://rawgit.com/allenhwkim/angular-jsdoc/master/default/docs/ngmap.MapController.html

wow... thanks for the quick response.  And I think I did not ask the question correctly.  My problem is not with the href part, but the part that shows up as header when you navigate to the source.  i.e. This header:

On my system this header would show up as-> Source:C: \ ..... folder\map-controller.js.  is it possible to restrict this header only to a relative path and the file name?  I should also perhaps state that all my .js files is situated in [jsfolder / folder / file.js].  on the "gulp settings" the watch then looks like this: -r jsfolder.  All the files do get compiled ok and it works perfectly.  It is just the header that pushes everything out of proportion and the fact that it will be incorrect when the project is run on some other system.
It will be great to show relative paths too, if not then maybe as a start only the name of the file?
Thank you Allen.

Sorry... I clicked the wrong button... it's not closed -> not sure how to remove that now :(

Interesting, is it a Windows thing? Could you try to use relative path for your command, or add you gulp task here, so that I can see it. I do not have windows, so I may need other's help.

Well I do run it from windows... but not sure if that is the problem?  I thought there was only an option to set which path variable to use.  Here is my gulp extract:
gulp.task('docs2', shell.task([
    'jsdoc '+
    '-c xDoc/conf.json ' +                                      // config file
    '-t node_modules/angular-jsdoc/angular-template ' +         // template
    '-d doc3 ' +                                                // out directory
    './README.md ' +                                            // Readme file
    '-r tmp/scripts'                                            // source code directories (space deliminated)
]));

Within my tmp/scripts folder there are for instance a Directives folder and a Services folder etc etc.
Here is a quick output of an example source page:

:

Patch has been implemented and released as version 1.1.6

Hi Allen.  That definitely solved the issue.  Thank you!!! I will mark this issue as closed.


Fix broken links to documentation

Thanks for your pull request. Is this your first contribution to a Snowplow open source project? Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).
:memo: Please visit https://github.com/snowplow/snowplow/wiki/CLA to learn more and sign.
Once you've signed, please reply here (e.g. I signed it!) and we'll verify. Thanks.

I signed it

Confirmed! @marcin-kozinski has signed the Contributor License Agreement. Thanks so much.

👍 Thanks @marcin-kozinski for the correction!


Any plans for PRequesting in detectron2 repo?

Any plans for PRequesting in detectron2 repo?

Thank you for your interest. I don't have such a plan. But you can just copy-paste the code under projects/CenterNet2 to your detectron2 repo and use it as is.

It wont have the detectron2's open support for converting c2 to onnx/tensorrt. The way this model is implemented, the symbol cant be generated to productionize it. (Detectron2 has the same problem).


Video not being played

I have been trying to play a video but I can't make it work. Everything was installed and linked, I'm able to import the Video package without errors but I'm stuck at a white screen and I really don't know the problem, the debug console doesn't show any errors neither the app creator.
So far this is my code (as you can see is a simple video test):

And my video (background.mp4) is at the root:

hi cande1gut try to bring the video in the generated project (iOS or Android) for me worked.
In every day use I prefer load a remote file.
Hope this help you

Hi, @g0dz1ll4, yeah, I was working like that, I'm able to play the video remotely, but I was trying to do it locally since is a best practice. Thank you.

@g0dz1ll4 I tried what you suggested, to bring the video into the Android generated project but it didn't work. Do I have to change my uri or is it ok like that?

@cande1gut sorry but actually i'm away .. I've checked on iOS and all run well you have tried to put the file in raw  or assets directory on android studio project?

Thank you @g0dz1ll4, I'm now able to load local files by a pull someone did to import files like react native does with the default path.

Great @cande1gut , can you describe the solution that work well for you ? so if some other has the same problem find a detailed solution ? ;)

Sure, I managed to import local files with the help of this pull: #251

So, this didn't work very well for me, but I found that dropping the video into
android/app/src/main/res/raw/ works (at least for android)

hhhmm weird @kvnvelasco, maybe it is where react-native-video module checks for assets by default, but good to know that it works for you :)

I'm also having this issue, playing local files.

Hey @magicspon have you tried what @kvnvelasco suggests? Therefore, if you still have this issue try to implement this, which is a pull request to be able to use local files like react native does by default: React Native asset resolving

#251 merged. Please close this issue to help keep repo clean.

@cande1gut I did try but to no success.  I shall take another look tomorrow. Thanks

@magicspon make sure to pull version 0.9.0 of the NPM module. Brent released it within the last 12 hours.

@jhabdas I've pulled in version 0.9.0 via npm i<EMAIL_ADDRESS>still no joy.

@magicspon It's hard to guess what you're doing, specifically. But the following should help:

Documentation updated this morning for new use of the RN asset system available in 0.9.0: https://github.com/react-native-community/react-native-video#load-files-with-the-rn-asset-system (though the existing approach using source={{uri: 'foo.png'}} remains unchanged).
Example code using 0.9.0 with local mp4 file: https://github.com/jhabdas/lumpen-radio/blob/5279b00da8428fd2a79af426dec60eb0fee06a26/src/components/BackgroundVideo.js using both approaches and develop branch can be pulled and run for example currently.
Due to a known bug use of the RN asset system for MP4s (i.e. using require('../assets/video/foo.mp4') works only for Android currently as I added to the README this morning.
Lastly be sure if you're adding video assets to iOS and not using the asset system (because it won't be possible to for that platform until at least RN 0.32) you've included the asset in the copy bundle step in Xcode target settings Build Phases as shown below:

@jhabdas HUZZZAH! It's alive...  I'm only developing for Android so a few points I can ignore.
But using the new require syntax is working.
Thanks for your help, greatly appreciated

Oh fo sho

I did a straight copy and paste, but no luck either. I see the progress bar indicating that the video is indeed advancing frames, but it's all a black/blank video, no matter what mp4 i insert, remote or local.
I am only testing in xCode/iOS simulator.
Here are my dependencies:
"firebase": "^3.2.1",
"react": "15.3.1",
"react-native": "^0.34.1",
"react-native-elements": "^0.6.0",
"react-native-material-kit": "^0.3.2",
"react-native-orientation": "^1.17.0",
"react-native-vector-icons": "^2.1.0",
"react-native-video": "^0.9.0"


feat: check usage of textures before including

this ensures that if a filter does not use body segmentation, it turns off the occlusion manager. and if it doesn't use face textures, it doesn't do all the stuff necessary for face texturing.
unfortunately, this would only work for new filters made after the plugin has been merged.


Provide UI to easier changing of the running location of an extension

Documentation of the remote.extensionKind setting is minimal despite it being very important. There are many issues involving "Remote-SSH" connections using more remote compute resources than users would like (#151205, #1110, #182903, and StackOverflow e.g.), and perhaps these issues could be resolved with a better settings.json API for this behavior and improved documentation.
Proposed change to settings.json: global "ui" extension configuration
In my case, I would like all extensions that can run on "ui" to do so.
Currently, I inspect ~/.vscode-server/extensions/extensions.json and add all the "identifier"."id" values I can find to "remote.extensionKind" in settings.json with the value ["ui"]. I add them manually. This seems like a bad system to me! I should not have to do this. Perhaps the settings UI can give each extension a checkbox setting for "run locally if possible".
For extensions that cannot run on "ui", I would like an option for them to not run at all and/or to give an error message. Maybe "cannot run in workspace" in the extensions UI.
Extension-specific settings plus a "run all extensions on "ui" setting would suffice.
Proposed improvement to documentation
The default value of the "remote.extensionKind" setting seems to be:
"remote.extensionKind": {
        "pub.name": [
            "ui"
        ]
}

What is this "pub.name"? This does not seem to be documented, but several GitHub issues quote settings.json files containing this value.
Also, my method of inspecting ~/.vscode-server/extensions/extensions.json  is basically something I figured out on my own. If this is the best method for forcing extensions to run on "ui", then please document this procedure. If there's a better way, please document it.
Thank you for the "Remote-SSH" extension! I use it every day and I really appreciate it. I have had to piece together subsets of this extension's functionality in the past and I can't overstate how useful it is to have it all together. Feels like good design to me! Thanks.

@zachsiegel-capsida Can you please explain what is missing for you in current Extensions UI?

Here is the user doc for managing extensions in remote set up - https://code.visualstudio.com/docs/devcontainers/containers#_managing-extensions

Thanks for this response @sandy081. My issue was about the following behavior described this documentation you linked to:

Advanced: Forcing an extension to run locally or remotely
Extensions are typically designed and tested to either run locally or remotely, not both. However, if an extension supports it, you can force it to run in a particular location in your settings.json file.
For example, the setting below will force the Docker extension to run locally and Remote - SSH: Editing Configuration Files extension to run remotely instead of their defaults:
"remote.extensionKind": {
    "ms-azuretools.vscode-docker": [ "ui" ],
    "ms-vscode-remote.remote-ssh-edit": [ "workspace" ]
}

A value of "ui" instead of "workspace" will force the extension to run on the local UI/client side instead. Typically, this should only be used for testing unless otherwise noted in the extension's documentation since it can break extensions. See the section on preferred extension location for details.

My feature request is to provide a single settings option that configures VS Code to use "ui" for all extensions that can use it. Something like "preferredExtensionHost" : "ui".
Trying to figure out which extensions can run in "ui", figuring out their internal names (e.g. "ms-azuretools.vscode-docker" is not immediately obvious when you install the "Docker" extension), and building out this "remote.extensionKind" object is not so straightforward, and unfortunately the stakes are high for many users who are experiencing bloat of their .vscode directory, which can have a severe impact when working remotely on a minimal ec2 instance or Raspberry Pi. Of course users don't have to install extensions when working on these remotes, but in many cases running extensions on "ui" whenever possible will solve this issue and support the extensions that users are using VS Code for in the first place.
Thank you!

Updated the summary of the issue accordingly.

This feature request is now a candidate for our backlog. The community has 60 days to upvote the issue. If it receives 20 upvotes we will move it to our backlog. If not, we will close it. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

We closed this issue because we don't plan to address it in the foreseeable future. If you disagree and feel that this issue is crucial: we are happy to listen and to reconsider.
If you wonder what we are up to, please see our roadmap and issue reporting guidelines.
Thanks for your understanding, and happy coding!

@sandy081 if you don't mind me asking, what's the argument against providing an option to prefer extensions to run on "ui" when possible?

It is a user convenient feature and I do not see many users asking for this. One can achieve this already using the existing setting.


fix: now using datadog-light

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Way to retrieve timestamp as part of transcription?

hi
is this possible?

@luzer This is not currently possible through the Google Speech API. Google are planning to start testing it out soon with a small number of their users. See at the link below for more info on that:
https://groups.google.com/forum/#!topic/cloud-speech-discuss/Mh_c4fOKh_U


[BUG] Performance of numeric exact-match queries can be improved

Describe the bug
I was recently looking at a query that included a clause like
"term" : {
  "test": 0
}

In this case, test was a long field, indexed as a point. Profiling the query (with "profiler":true) to see why it was slow, I found that the query was parsing to PointRangeQuery(test:[0 TO 0]), without wrapping it in an IndexOrDocValuesQuery. In this case, the PointRangeQuery was taking up a lot of the query time.
I rewrote the query as:
"range": {
  "test": {
    "lte":0,
    "gte": 0
  }
}

That one did do the IndexOrDocValuesQuery wrapping, and it was much faster (latency dropped from 1.2 seconds to 350ms).
Expected behavior
I would expect the the query mapping by NumberFieldMapper.NumberType would do the IndexOrDocValuesQuery wrapping automatically.
The implementation of termQuery should probably delegate to rangeQuery in most cases, just applying the range over the target value. For the various integer types, we can keep the optimization that returns a MatchNoDocsQuery if the value is not an integer. (As a bonus, while we're there, we could return  a MatchNoDocsQuery for values outside permitted range of a field, like a negative value for unsigned_long or a number >= 128 for  byte.)
At the same time, the implementation of termsQuery for number fields should just be a BooleanQuery over termQuery when the number of terms is small. (For keyword fields, "small" means 16 or fewer clauses.)

Some initial benchmarks:
|                                        50th percentile latency |              simple_term |     4311.71 |     ms |
|                                        90th percentile latency |              simple_term |     4864.34 |     ms |
|                                        99th percentile latency |              simple_term |     4996.19 |     ms |
|                                       100th percentile latency |              simple_term |     5009.17 |     ms |
|                                   50th percentile service time |              simple_term |      32.621 |     ms |
|                                   90th percentile service time |              simple_term |     40.3655 |     ms |
|                                   99th percentile service time |              simple_term |     45.6012 |     ms |
|                                  100th percentile service time |              simple_term |     45.9752 |     ms |
|                                                     error rate |              simple_term |           0 |      % |

Running the http_logs workload with 1 shard and 3 segments per shard.
Query is a simple term query on status:
{
"query":
{
"term":
{
"status": 200
}
}
}

Query is a simple term query on status:

This isn't going to be enough. That should behave the same as the index query alone.
The key piece of the IndexOrDocValuesQuery optimization is that it lets a "lead iterator" do the driving, while the IoDVQ "lazily" matches on doc values (when appropriate). In this case, you don't have a lead iterator.
Imagine you have a simple "A AND B" BooleanQuery. Lucene obtains iterators for both A and B (via the createWeight() method on the Query then scorerSupplier() on the Weight). If you imagine these are both simple TermQuery instances, then iteration starts with the one with lower doc frequency (since its iterator is going to be more sparse), calling nextDoc(), then calls advance() on the other one to jump to the first value >= the first iterator's current doc. They leap-frog until they're both pointing to the same doc ID, at which point you have a match. Then it calls nextDoc on the more sparse iterator, and things continue
Now, imagine instead that B is an IndexOrDocValuesQuery. The key point is that the IoDVQ's Weight's scorerSupplier() is called with the cost of A (i.e. the cost of A is the leadCost). If the index query's cost is sufficiently close to the cost of A (i.e. within a factor of 8), then we still run the index query. If the index query cost is higher than that (i.e. it matches a lot of stuff), then we return the doc value query's iterator. The DV query's iterator has an "approximate" advance method that will always jump to whatever you ask it to jump to (whether the value matches or not). It's wrapped in a TwoPhaseIterator, which exposes a matches() method that loads the doc value for the target doc and checks if it matches the constraints of the query. So, A's iterator does all of the stepping through doc IDs, B's iterator always just says, "Sure, whatever", and the matches() call loads the DV and confirms a match.
tl;dr: We need a lead iterator for IndexOrDocValuesQuery to run in doc values mode.

Hi @msfroh , will documentation for this feature be required for 2.12? If so, can you please create the doc issue, let me know who will be creating the initial doc PR, and add this dev issue to the unified tracker project? Thanks!

Hi @msfroh , will documentation for this feature be required for 2.12?

No documentation changes will be required. It's just an under-the-hood optimization.


Update MOUNIR_NODE2.toml for testnet 15

[validator.MOUNIR_NODE2]
consensus_public_key = "0038dd0714abe2155d40980bf7a68a71c57b5fb51ae48115a40cc5af9f5541fcb3"
account_public_key = "000366610d10575c6f34c0a1fddf3bbcdabbdb650045d034eb1faaf5b3c18e64d7"
protocol_public_key = "00efe939821115f9f99efe227fad6bffaf34825461e4ba4622260114623b418e9f"
dkg_public_key = "600000000fa9ef92db6382e20d863d3ada7b1702fcd2018129aad40ec39b647b107bf8b1d1ac087f1ec038d174c7774f85af8a1142951a257489249841818388e0ac4750b5058abf5bb54dbc0a5884572bb48f4af56b5bbbf887f1cb35f900035c0eba90"
commission_rate = "0.05"
max_commission_rate_change = "0.01"
net_address = "<IP_ADDRESS>:26656"
tendermint_node_key = "00c098383e0efa0bb0400249f2a2b8fc4e8ffbc10f16239ee870a07696a507c5b7"
email =<EMAIL_ADDRESS>discord = "ramonsami"
elements = ""
telegram = "@ramonsami"
twitter = "@ramonsami"

This PR has been automatically closed because it does not follow the correct procedure for updating the genesis file.
Not a namada-public-testnet-15 file
This PR does not add a single .toml file.


Remove version: BlueMarbleGeographics.GlobalMapper version 24.00.0000

[Automated] It returns code over 400 in all urls
Microsoft Reviewers: Open in CodeFlow

URL: https://www.bluemarblegeo.com/downloads/global-mapper/global-mapper-24_0-x64.exe
Response Error: The remote server returned an error: (404) Not Found.
(Automated message - build 673)

URL: https://www.bluemarblegeo.com/downloads/global-mapper/global-mapper-24_0-x64.exe
Response status code does not indicate success: 404 (Not Found).
(Automated message - build 877)

For moderators
This account is automated by Github Actions and the source code was created by CoolPlayLin. If you have any questions about any pull request, don't hesitate to ping Exorcism0666.

[!warning]
With the new version of Komac, numerous Pull Requests may contain bugs or unwanted elements. If such issues arise, please tag russellbanks so that these bugs can be addressed in future updates.

[!important]
Please carefully review these Pull Request before merging. If it is a Pull Request for removing incorrect content and the URLs are issue, free upon manual checking, please close this Pull Request directly. (It is best to inform Exorcism0666 of the closure.)

良い一日をお過ごしください！

URL: https://www.bluemarblegeo.com/downloads/global-mapper/global-mapper-24_0-x64.exe
Response status code does not indicate success: 404 (Not Found).
(Automated message - build 877)

URL: https://www.bluemarblegeo.com/downloads/global-mapper/global-mapper-24_0-x64.exe
Response status code does not indicate success: 404 (Not Found).
(Automated message - build 877)

URL: https://www.bluemarblegeo.com/downloads/global-mapper/global-mapper-24_0-x64.exe
Response status code does not indicate success: 404 (Not Found).
(Automated message - build 877)


High loss_value due to High confidence_loss

Hi,
I am using this code to train it on a custom dataset ( two classes). I have a very high loss_value, around 30 sometimes 50. After debugging, I found out that this is due to a high confidence loss.
Do you have any idea how to fix this issue?
Thanks!

i think High loss value not importance. Importance is accuracy in test dataset and run in real

are you implement fine tune epoch 50 - 100 (request many memory GPU)?

yeah, I am using the same fine-tuning as in the current code, but I resized the images as well to 320 to fix the issue with the memory GPU and the batch size to 24.

Mine goes to around 16 and stays there. Also with fine tuning. Any ideas?

I have the same problem here, I tried using different learning rate, tried to fine tune and defreez all network but any way the loss is stuck at around 30. If someone found the solution please notify me. or even if you found an other repository that allowed you to train on your data without problem, please let me know

Hi, from my experience I would rather not rely at all on the loss value. I had good mAP while my loss value was around 12. It really depends on the number of objects in one image and the dataset. So the next step will be to evaluate your model using mean average precision or simply precision and recall for a given threshold score and IOU. Before that, you should also have a look to ur labels, maybe you have some wrong labels ( it happened to me). Good luck!

我的损失很高 测试效果也不理想怎么办


PEP-8 style issue detected in ./src/offsetanalysis.py:12:1:

E302 expected 2 blank lines, found 1

Fixed in #124


Store the minimum required context for answers in the answer store

What is the context of this PR?
Only answer_id, group_instance and answer_instance should be needed.
Allows question to be moved between sections without breaking the existing answer store.
How to review
Would need to load previous version somehow (backwards compatible)

I couldn't test starting a survey on master and continuing on this branch as this branch doesn't have the session encryption changes from master.

@ajmaddaford ill rebase it so it has those changes


breaking: Remove localPlayerAuthority

This flag does not do anything useful in anything but NetworkAnimator, and confuses the hell out of everyone.
so NetworkAnimator can keep it's own (much more readable) flag.

:tada: This PR is included in version 5.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


error: cannot use import statement outside a module?

I'm getting an annoying error. Am I doing something obviously wrong? Any pointers would be much appreciated.
$ node ./esbuild.config.js --watch
(node:3948) Warning: To load an ES module, set "type": "module" in the package.json or use the .mjs extension.
(Use `node --trace-warnings ...` to show where the warning was created)
/home/koshy/Documents/www/BensRails/mto/esbuild.config.js:3
import watPlugin from 'esbuild-plugin-wat';
^^^^^^

SyntaxError: Cannot use import statement outside a module
    at Object.compileFunction (node:vm:352:18)
    at wrapSafe (node:internal/modules/cjs/loader:1031:15)
    at Module._compile (node:internal/modules/cjs/loader:1065:27)
    at Object.Module._extensions..js (node:internal/modules/cjs/loader:1153:10)
    at Module.load (node:internal/modules/cjs/loader:981:32)
    at Function.Module._load (node:internal/modules/cjs/loader:822:12)
    at Function.executeUserEntryPoint [as runMain] (node:internal/modules/run_main:81:12)
    at node:internal/main/run_main_module:17:47
error Command failed with exit code 1.

// esbuild.config.js
const path = require('path')
const esbuild = require('esbuild')
import watPlugin from 'esbuild-plugin-wat';

esbuild.build({
  entryPoints: ['./application.js'],
  bundle: true,
  outdir: path.join(process.cwd(), "app/assets/builds"),
  absWorkingDir: path.join(process.cwd(), "app/javascript"),
  watch: process.argv.includes("--watch"),
  sourcemap: true,
  mainFields: ['module', 'main'],
  plugins: [
    watPlugin() // options are documented below
  ],
}).catch(e => (console.error(e), process.exit(1)))

My package.json
{
  "name": "app",
  "private": "true",
  "dependencies": {
    "@hotwired/stimulus": "3.0.1",
    "@hotwired/turbo-rails": "7.1.1",
    "@tailwindcss/forms": "0.5.1",
    "autoprefixer": "10.4.5",
    "esbuild": "0.14.38",
    "esbuild-plugin-wat": "0.2.7",
    "postcss": "8.4.12",
    "tailwindcss": "3.0.24",
    "tailwindcss-stimulus-components": "3.0.4",
    "three": "0.141.0",
    "web-ifc-three": "0.0.117"
  },
  "scripts": {
    "build": "node ./esbuild.config.js",
    "build:css": "tailwindcss -i ./app/assets/stylesheets/application.tailwind.css -o ./app/assets/builds/application.css --minify"
  }
}

first pointer: try to google errors before you open issue. this is an extremely well-known and well-documented issue. https://www.google.com/search?channel=fs&client=ubuntu&q=cannot+use+import+statement+outside+a+module+node
second pointer: actually look at the error message before opening an issue:

Warning: To load an ES module, set "type": "module" in the package.json or use the .mjs extension.

third pointer: set "type: module" in the package.json, or rename files that have an import statements to have an .mjs extension

Hey I appreciate your msg, apologies for the question, and the maintenance burdens etc. I agree to your suggestions, however the following might be of benefit (in my case, a tyro esbuild user):
I had followed the readme to a T, I was expecting smooth sailing.
But, unfortunately, the error msg was not particularly clear (to me): I thought I had already taken care of the module issue by following the esbuild docs with the type module included. I wasn't sure what to change to .mjs. The initial searching was not particularly fruitful - and granted: yes, all questions can be answered by diligent study and Googling, but I thought the answer might be extremely obvious to someone who knows, vs someone who doesn't. I didn't see many issues open, and thought: it's worth a shot.
Still, is it worth a documentation PR?

Hey @benkoshy, thanks for your reply! Gotcha, yeah the JS module schism is hella confusing if you're new. Not sure if it's worth the doc PR but I'd merge it for sure


Graphql CLI part 1

Converts cli commands to using graphql in order to support the new accounts subsection. I... think this is the minimum set of commands required to start using accounts??? (meaning any time a private key is used, we do a graphql query instead of using -privkey-path + rpc).
This PR converts:

get-balance
send-payment
delegate-stake
cancel-transaction
set-staking

For now, I've added a client2 cli section in the dev profile, mostly to be able to test out the commands as we plan the transition.
Since swapping over these commands in-place will probably break all scripts using the cli, I included both versions for now. As soon as possible, we will remove the old versions (that's Part 2!).
PR Guide:

First commit: Some improvements  and reorganizing of the graphql client. Moved the queries into their own file and did better error handling.
Second commit: Add some types to the cli_lib in preparation for writing graphql versions of cli commands. user_command_common is flags that are common to all user_commands. We use this because the abstraction we used previously in client.ml doesn't work as well on the gql interface.
Third commit: Add graphql cli commands exposed under client2.

New gql error messages:
avery@mbp$ coda client2 send-payment -amount 5 -receiver 8QnLXwyVdcZFpUaX6Fk9s9iQPsz4B6VAh3oihHdmrbRC6djFyVhqNeU9Wt5DeTXr16 -sender  8QnLXwyVdcZFpUaX6Fk9s9iQPsz4B6VAh3oihHdmrbRC6djFyVhqNeU9Wt5DeTXr16
❌ Error connecting to daemon. You might need to start it, or specify a custom --rest-port if it's already started.
Error message: (Unix.Unix_error "Connection refused" connect <IP_ADDRESS>:3085)

avery@mbp$ coda client2 send-payment -amount 5 -receiver 8QnLXwyVdcZFpUaX6Fk9s9iQPsz4B6VAh3oihHdmrbRC6djFyVhqNeU9Wt5DeTXr16 -sender 8QnLXwyVdcZFpUaX6Fk9s9iQPsz4B6VAh3oihHdmrbRC6djFyVhqNeU9Wt5DeTXr16
❌ Error: Node is still bootstrapping (in ["sendPayment"])

moving our default rest/ql port to 3085 to match you


feat: Added Background for text visibility

Fixes issue#1661

@apu52  Where can I find that template? I am not able to find it out.

@Archi-shaw the template will be automatically there whenever u will submit a new PR , I think maybe u have deleted it mistakenly , u can check over other pr's in this repo details

@Archi-shaw will you submit a new PR , so that i can close this PR or will edit in this pr

@apu52 Closing this pr and creating new one.


Enable cookie auth/Stay logged in

Hi,
I'd like to add the functionality to use the --user-data-dir option to persist the browser data.
This enables the usage of cookies to skip the authentication as long as the cookie is valid.
As a result, credential refresh should be much smoother.
Let me know your thoughts.
Best regards
Lars

Cosed in favour of #47


Address TODO: when ex_machina > v2.2.0 is released

Resolved by https://github.com/source-academy/cadet/commit/9033f117d520788cf2d7acac4f3fdfca66846aaa


Port embedding

@abhidg Please review the commit 8bd5812cbf0e1c57e2e9de26e9861eec76278a1b that aims to avoid the use of a global.


Comment is being deleted

ID: 1d0dadd2-0591-4e89-8b9c-85c48fd0f060
RuleId: CG0659
Existing comments should be preserved

I forgot turning on the comment process. CG0659 is showing comments now. Do you have more rules with comments? I'd like to test couple of more rules.

27b19da0-4601-490e-bbfe-a0d389d1c46a
513d1590-579f-4909-be51-745a306657af
51868eb6-d402-4259-8734-807c0e56dddf

I am closing this one since this is fixed after I turned on the comment processing.


Upgrade Phoenix to 2.2.1

Thanks!
Changes to a cask
Editing an existing cask

[x] Commit message includes cask’s name (and new version, if applicable).
[x] brew cask audit --download {{cask_file}} is error-free.
[x] brew cask style --fix {{cask_file}} left no offenses.

Thank you. Our Travis builds are having some hiccups today, hence the delay.

@vitorgalvao No worries, thanks for your hard work with this project!


Test appveyor pr

:white_check_mark: Build dcos-cli 1.0.3 completed (commit https://github.com/dcos/dcos-cli/commit/e27d24c770 by @bamarni)


Support Pie chart

The input schema has two columns:

a String compatible column for the category name
a number compatible column for the value

included in release 0.9.0


Include styles with the  component to avoid unnecessary global styles.

When using the library with Next.js, you can't include the styles at a component level just for the pages / components using the slider. You have to import it globally, causing additional unnecessary CSS.
There should be a parameter for the <Slider> component that includes the default slick.css and slick-theme.css, such as:
const settings = {
	dots: true,
	infinite: true,
	speed: 500,
	slidesToShow: 4,
	slidesToScroll: 4,
        styles: true
};

<Slider {...settings}>
     ...
</Slider>

+1, I am also looking for a solution to this

I saw other libraries adding an attribute to the main component, in this case , which passes down class names to sub-components. @akiran
For example:
<Slider className={ styles.slider } dynamicClasses={{ arrows: styles.arrows, slide: styles.slide, ... }}>
...
</Slider>

+1
I think the modern way to install a module, the module takes care of installing other dependencies
It should be automatically installed and the css should be scoped, material UI style, with the possibility to change styles


Error in isogram test

CLASS zcl_isogram DEFINITION PUBLIC.

  PUBLIC SECTION.
    METHODS is_isogram
      IMPORTING
        VALUE(phrase)        TYPE string
      RETURNING
        VALUE(result) TYPE abap_bool.
  PROTECTED SECTION.
  PRIVATE SECTION.
ENDCLASS.

CLASS zcl_isogram IMPLEMENTATION.

  METHOD is_isogram.
    DATA res TYPE c LENGTH 26.
    DO strlen( phrase ) TIMES.
      DATA(pos) = find( val = sy-abcde sub = to_upper( phrase(1) ) ).
      IF pos >= 0.
        IF res+pos(1) = abap_true.
          RETURN.
        ELSE.
          res+pos(1) = abap_true.
        ENDIF.
      ENDIF.
      SHIFT phrase LEFT.
    ENDDO.
    result = abap_true.
  ENDMETHOD.

ENDCLASS.

Works in ABAP, fails in exercism:
ZCL_ISOGRAM: running ltcl_isogram->is_isogram1
ZCL_ISOGRAM: running ltcl_isogram->is_isogram2
Error: ASSERT failed

also https://github.com/exercism/abap/issues/37

reproduced as
DATA phrase TYPE string.
phrase = 'abc'.
WRITE / phrase.
SHIFT phrase LEFT.
WRITE / phrase.

@mbtools try again

is_isogram2 works but still:
ZCL_ISOGRAM: running ltcl_isogram->is_isogram1
Error: ASSERT failed

reproduced as
DATA res TYPE c LENGTH 26.
DATA pos TYPE i.
res+pos(1) = abap_true.
pos = 5.
res+pos(1) = abap_true.
WRITE / res.

@mbtools try again

works


Authentication error

Hi
I've just set up my air conditioners using the mobile app, and have added the integration in HACS.
When adding the integration in HA, I get this error:
"Login ID or password is incorrect,or account is locked"
I can use the app successfully with these credentials. It's not clear how I can verify my credentials separately from the app.
One thing to note is that I've used a "+" character in the email address, I'm not sure if this is causing the issue?
<EMAIL_ADDRESS>Log below
This error originated from a custom integration.
Logger: custom_components.panasonic_cc.config_flow
Source: custom_components/panasonic_cc/pcomfortcloud/session.py:158
Integration: Panasonic Comfort Cloud (documentation)
First occurred: 18:37:22 (4 occurrences)
Last logged: 19:50:23
Unexpected error creating device
Traceback (most recent call last):
File "/config/custom_components/panasonic_cc/config_flow.py", line 60, in _create_device
devices = await self.hass.async_add_executor_job(api.get_devices)
File "/usr/local/lib/python3.10/concurrent/futures/thread.py", line 58, in run
result = self.fn(*self.args, **self.kwargs)
File "/config/custom_components/panasonic_cc/pcomfortcloud/session.py", line 201, in get_devices
self.login()
File "/config/custom_components/panasonic_cc/pcomfortcloud/session.py", line 105, in login
self._create_token()
File "/config/custom_components/panasonic_cc/pcomfortcloud/session.py", line 158, in _create_token
raise ResponseError(response.status_code, response.text)
custom_components.panasonic_cc.pcomfortcloud.session.ResponseError: Invalid response, status code: 401 - Data: {"message":"Login ID or password is incorrect,or account is locked","code":4101}

Please check your credentials and try again. Failing that, you may need to wait 24 hours.

This issue has been inactive for a while, so we’re closing it for now. If you believe this issue is still relevant, please feel free to reopen it or create a new one.


update golang.org/x/net dependency to 0.4.0

this change is being proposed to mitigate this CVE, https://pkg.go.dev/vuln/GO-2022-1144

i produced this by updating the go.mod and then running go mod tidy && go mod vendor && go mod verify

closing this issue as the version was bumped with this commit https://github.com/IBM-Cloud/cloud-provider-ibm/commit/4d5076e3814bab2c7084124ef842d00d0ce31a8f


DietPi support

i had the script working fine on DietPi then I did a pull to the latest Repo and I can't get it to reinstall. I get an unsupported distro error

root@DietPi:~# curl -LSs https://raw.githubusercontent.com/estruyf/unicorn-busy-server/master/install.sh | sudo bash -
The requried files are missing... lets clone everything from git...
Cloning files from git using HTTPS to /root/unicorn-busy-server...
bash: line 120: lsb_release: command not found
Unsupported distribution, please consider submitting a pull request to extend the script

if i try to start server.py manually
root@DietPi:~/unicorn-busy-server# ./start.sh
Traceback (most recent call last):
File "./server.py", line 12, in
from lib.unicorn_wrapper import UnicornWrapper
File "/root/unicorn-busy-server/lib/unicorn_wrapper.py", line 5, in
import spidev
ModuleNotFoundError: No module named 'spidev'

had to run a apt-get install -y python3-spidev.
New error
Traceback (most recent call last):
File "./server.py", line 12, in
from lib.unicorn_wrapper import UnicornWrapper
File "/root/unicorn-busy-server/lib/unicorn_wrapper.py", line 7, in
from unicornhatmini import UnicornHATMini
ModuleNotFoundError: No module named 'unicornhatmini'

installed this next - https://github.com/pimoroni/unicornhatmini-python

still getting Traceback (most recent call last):
File "./server.py", line 12, in
from lib.unicorn_wrapper import UnicornWrapper
File "/root/unicorn-busy-server/lib/unicorn_wrapper.py", line 7, in
from unicornhatmini import UnicornHATMini
ModuleNotFoundError: No module named 'unicornhatmini'

i manually installed gpizero but i still can't get it running

i give up for now! PLease advise as I am using the distro and guide from https://www.eliostruyf.com/diy-building-busy-light-show-microsoft-teams-presence/

fixed it by manually installed the pre-requesites but looks like one of your commands to check versions isn't supported by dietpi

Dietpie doesn't appear to support the lsb_release -si

@ghctim thanks for the feedback. @j-maynard updated the installation scripts, and DietPi seems to not yet supported.
Just in case, I have added the old installation script as a fallback: https://github.com/estruyf/unicorn-busy-server/blob/master/install-fallback.sh
Feel free to try it out.

Hi Elio,
I am building the busy light project (my first project), and after running the "install-fallback.sh" script, i get the following errors:
Setting up python3-dev (3.7.3-1) ...
Processing triggers for libc-bin (2.28-10+rpi1) ...
Could not open requirements file: [Errno 2] No such file or directory: './requirements.txt'
cp: cannot stat 'busylight.service': No such file or directory
Failed to enable unit: Unit file busylight.service does not exist.
Failed to start busylight.service: Unit busylight.service not found.
chmod: cannot access './start.sh': No such file or directory
I am running a Pi Zero W with DietPi v6.31.2
Is there something I am missing?

@spazool what does the service show if you run: systemctl status busylight? You should normally see if it's running or failed.
What you can also try is to run the server with the following command: python3 server.py. Check if this works.

Thanks for putting this great project together @estruyf! There are a few additional issues with DietPi that I ran into beyond what's discussed on this thread:

The default user is dietpi instead of pi so the working directory for the busylight.service file needs to be updated to something like:
/home/dietpi/unicorn-busy-server
Additionally, the install-fallback.sh script sets permissions on the wrong file. The service is trying to run python3 server.py and not sh start.sh so I had to set execution permissions on the server.py file itself:
sudo chmod +x ./server.py

I also ran into a bunch of issues getting the Unicorn pHat but I will delve into that some other time. I have also opened an issue about that here (to track the software problems I ran into):
https://github.com/pimoroni/unicorn-hat/issues/141


<EMAIL_ADDRESS>decompress error

PS C:\Users\lepto> scoop install extras/appflowy
WARN  Purging previous failed installation of appflowy.
ERROR 'appflowy' isn't installed correctly.
Removing older version (0.6.1).
'appflowy' was uninstalled.
Installing 'appflowy' (0.6.1) [64bit] from 'extras' bucket
Loading AppFlowy-0.6.1-windows-x86_64.exe from cache
Checking hash of AppFlowy-0.6.1-windows-x86_64.exe ... ok.
Extracting AppFlowy-0.6.1-windows-x86_64.exe ... ERROR Exit code was 1!
Failed to extract files from C:\Users\lepto\scoop\apps\appflowy\0.6.1\AppFlowy-0.6.1-windows-x86_64.exe.
Log file:
  ~\scoop\apps\appflowy\0.6.1\innounp.log

Please try again or create a new issue by using the following link and paste your console output:
https://github.com/ScoopInstaller/Extras/issues/new?title=appflowy%400.6.1%3a+decompress+error

PS C:\Users\lepto> cat ~\scoop\apps\appflowy\0.6.1\innounp.log
Signature detected: Inno Setup Setup Data (6.3.0)
This is not directly supported, but i'll try to unpack it as version 5602
; Version detected: 6300
Critical error: The setup files are corrupted. Please obtain a new copy of the program.

same here,,

Duplicates #13487.


Optionaly force-delete remaining channel items on close

This fixes #2890

@Element-126 approved this in #2890


[WIP] Add vetting to the creation of accounts

Answers #93.
Sign up of new accounts needs to be given an approved flag by admins in addition to a confirmed flag through the usual sign-up process.
This PR adds a requirement to have the CLASSIFY permission (granted to the User role) in order to see surveys. The User role is no longer applied automatically, but must be granted by an administrator after email confirmation.

Now need to add tests for the following:

Users without User role cannot classify surveys.
Users with User/Administrator role can classify surveys.


Standardise solidity version to 0.8.4

Closes https://github.com/keep-network/coverage-pools/issues/98
For Solidity upgrade to 0.8.4 this PR also upgrades OZ to 4.1.0 to align with Solidity compiler. Some of the compilation errors had to be changed as well.

@pdyraga all the comments related to operators are addressed here: 79af193

Can we remove uint256 from this conversion?

I've tried that and all the tests passed, but... according to the docs:

address(uint) and uint(address) : converting both type-category and width. Replace this by address(uint160(uint)) and uint(uint160(address)) respectively.

I'm just not sure if there can be some weird corner case where having just tbtcDepositToken.exists(uint160(depositAddress)) breaks things.

Can we remove uint256 from this conversion?

I've tried that and all the tests passed, but... according to the docs:

address(uint) and uint(address) : converting both type-category and width. Replace this by address(uint160(uint)) and uint(uint160(address)) respectively.

I'm just not sure if there can be some weird corner case where having just tbtcDepositToken.exists(uint160(depositAddress)) breaks things.

I think it is safe, it's just a matter of whether uint160 -> uint256 conversion happens implicitly or explicitly. We properly updated uint(address) to uint(uint160(address)) per Solidity 8 changelog but the additional uint256 should have nothing to do about Solidity 8. That's fine, it's just a detail, we can leave it as-is.


Is  fromTokenId_; code redundant in _createDerivedTokenId function?

Is  fromTokenId_; code redundant in _createDerivedTokenId function?
Just like the follow code in ERC3525.sol:
function _createDerivedTokenId(uint256 fromTokenId_) internal virtual returns (uint256) {
        fromTokenId_;
        return _createOriginalTokenId();
    }

In some scenarios,  the derived token id needs to be related to the from token id in the split action ( transfer some value to address from Id ).   _createDerivedTokenId function is a reserved extension for this case. So, It is necessary to keep the fromTokenId.


Virtual desktop naming

Description of the new feature / enhancement
For windows 11, when you cycle through virtual desktops using windows + ctrl + arrow, would be awesome if you could have an overlay saying the name of the virtual desktop you are on currently.
Windows virtual desktops is almost perfect but missing that 1 thing!
Scenario when this would be used?
When switching virtual desktops rapidly using win + ctrl + arrow
Supporting information
No response

Very plausible. Also good question for Windows Feedback Hub.

/dup #12246


⚠ Bofrost.be has degraded performance

In 2a2b44f, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1277 ms

Resolved: Bofrost.be performance has improved in 3408888 after 18 minutes.


Fix build config

Motivation and Context
I stumbled upon it so thought I'd fix 🙂

My first time creating a package thanks for the help.
Did you build hatchling??

I wrote it yes

Thats really cool! I met a python legend today :)
I do have to deny your pull request for compliance reasons even tho its only a few lines. I have implemented the changes however. Thanks for pointing them out and correcting them.

No worries, happy to help!


Add light-date

By submitting this pull request, I promise I have read the contribution guidelines twice and ensured my submission follows it. I realize not doing so wastes the maintainers' time that they could have spent making the world better. 🖖
⬆⬆⬆⬆⬆⬆⬆⬆⬆⬆
Light Date is a date formatting module, that works in Node.js and the browser. It is super lightweight (157 bytes minified & gzipped, no dependencies) and fast (see the benchmarks). It also follows Unicode Technical Standard #35.
The repo is about 19 days old, so I'm submitting it as a draft PR - will publish it when it reaches 30 days.

Don't open a Draft / WIP pull request while you work on the guidelines. A pull request should be 100% ready and should adhere to all the guidelines when you open it. - https://github.com/sindresorhus/awesome-nodejs/blob/master/contributing.md

@sindresorhus This rule was added after I opened this PR, so sorry but I really wasn't able to see it 😕


Profiler always requires sending columnProfile when we just need table level data

From https://openmetadata.slack.com/archives/C02B6955S4S/p1656658286663549
If we want just to send table-level data, we still need to pass columnProfile: []:
//This succeeds
"tableProfile": [
  {
    "profileDate": "2021-10-18",
    "rowCount": 1909,
    "columnProfile": []
  }
],

//This fails with NPE on Backend
"tableProfile": [
  {
    "profileDate": "2021-10-18",
    "rowCount": 1909,
  }
],

We could mark the columnProfile as optional

@pmbrull this will be fixed by https://github.com/open-metadata/OpenMetadata/pull/6416 🙂

oooh thank you @TeddyCr (L)


Fix scrollbar / overflow

When viewing my website, there was a horizontal scrollbar caused by (then) unknown overflow.
Current solution
overflow-x: hidden; was used to remove the scrollbar, as can be seen in https://github.com/ellipticcurv3/www/blob/30b927ff4df6e7b52d0a09738e7b03a181878b3a/src/.vuepress/theme/styles/index.styl#L11
Cause
It was actually caused by putting the width of the <html> element to 100vw, as can be seen in https://github.com/ellipticcurv3/www/blob/fcd4ee6435cf6fee2e9df71d3d162133bf9b9a5c/src/.vuepress/theme/styles/global/main.scss#L4
Better solution
Putting the width to 100% will remove the overflow, and then you will no longer need to hide the overflow. Besides that, overflow-x: hidden; causes additional problems. An example is that elements can't have position: sticky; when hiding overflow.

Time required: 5 min +/- 25% (3 – 6 min)

Fixed in 731d766ce255e8c87affa4fee12c638999ba51fa + 2a27a9bbda8de673a68687aef753011b9e849462.
Next time, please don't rebase the changes but use a merge instead. Rebase causes all individual branch commits to be added to develop, which is a bit overkill. Especially when commiting in one-change-at-a-time style, like here.


move import path and image to weaveworks

fix #79
in addition to #82
also, fix #78

@phoban01 I think unshadowing the variable like you suggested did the trick.

@phoban01 I think unshadowing the variable like you suggested did the trick.

I think using a pointer will be the right fix long term here, as that's really the behavior we're after in this case. But it's better to do that refactor in it's own PR I think :smile:


Adding Red Flag Performance Regression in e2e tests.

Currently e2e tests will pass based on overall behavior.  Currently we do track performance but we do not gate the overall tests.   It seems logical that we would want to have some baseline to detect regression and simply raise the red flag under those conditions.
e.g. create a utility function that checks

if overall average falls below 10 pods/sec.
if terminated pods > 2% of total pod count
...

/cc @wojtek-t @fgrzadkowski @rrati @jayunit100 @roberthbailey

@fgrzadkowski already started adding some expectations on performance (although we need to tune them) - see #7505

@fgrzadkowski @wojtek-t
It's been a while since we had our test day and we've noticed a fairly large drop in overall throughput.

@timothysc We have also noticed, but couldn't find the reason. I plan to look more into it today.

@fgrzadkowski just as a note of color, but we used the same technique outlined in https://github.com/GoogleCloudPlatform/kubernetes/pull/7573 to track and uncover the regression.

At this stage in the cycle, I'm more inclined to have a script that runs independent of the e2es and the end rip through and check for regression then I am in adding new ways for the tests to fail.

So we have scripts now that churn on the output from e2e to raise a flag when needed.


Installation v2

Hi Jeroen,
I've been trying to install the logger v2 on Ubuntu 18.0.4.
Using npm install throws errors: Error: EACCES: permission denied, mkdir '/home/koen/node-opcua-logger-2/node_modules/node-opcua-certificate-manager/certificates'
Even a sudo npm install doesn't solve this.
I then tried sudo npm install --unsafe-perm=true --allow-root and that seemed to install ok. However, as soon as I ran 'node logger.js' I got an error: Error: Cannot find module 'nedb'. I resolved this by running  sudo npm install nedb --save.
Next issue, again as soon as I run 'node logger.js', I get the following error
/home/koen/node-opcua-logger-2/writepump.js:18
	this.output = new influx({
	              ^

TypeError: influx is not a constructor
    at new WritePump (/home/koen/node-opcua-logger-2/writepump.js:18:16)
    at Object.<anonymous> (/home/koen/node-opcua-logger-2/logger.js:11:10)
    at Module._compile (internal/modules/cjs/loader.js:778:30)
    at Object.Module._extensions..js (internal/modules/cjs/loader.js:789:10)
    at Module.load (internal/modules/cjs/loader.js:653:32)
    at tryModuleLoad (internal/modules/cjs/loader.js:593:12)
    at Function.Module._load (internal/modules/cjs/loader.js:585:3)
    at Function.Module.runMain (internal/modules/cjs/loader.js:831:12)
    at startup (internal/bootstrap/node.js:283:19)
    at bootstrapNodeJSCore (internal/bootstrap/node.js:622:3)

I then changed line 18 in writepump.js to this.output = new influx.InfluxDB({ and that made the issue go away. Perhaps the syntax for influx changed ?
Then, again when executing node logger.js, I get the following error:
/home/koen/node-opcua-logger-2/readpump.js:20
    self.uaClient.connect(self.uaServerUrl, function(err) {
                  ^

TypeError: self.uaClient.connect is not a function
    at ReadPump.ConnectOPCUA (/home/koen/node-opcua-logger-2/readpump.js:20:19)
    at /home/koen/node-opcua-logger-2/readpump.js:319:22
    at nextTask (/home/koen/node-opcua-logger-2/node_modules/async/dist/async.js:5324:14)
    at Object.waterfall (/home/koen/node-opcua-logger-2/node_modules/async/dist/async.js:5334:5)
    at ReadPump.Run (/home/koen/node-opcua-logger-2/readpump.js:316:11)
    at /home/koen/node-opcua-logger-2/logger.js:19:6
    at /home/koen/node-opcua-logger-2/node_modules/async/dist/async.js:3269:12
    at /home/koen/node-opcua-logger-2/node_modules/async/dist/async.js:74:12
    at next (/home/koen/node-opcua-logger-2/node_modules/async/dist/async.js:3514:9)
    at Object.forever (/home/koen/node-opcua-logger-2/node_modules/async/dist/async.js:3516:5)

This time I gave up. :-/ ... something seems fishy - surely something went wrong with my original install? I shouldn't have to chase errors like this to make it work, right? Or.... Could it have anything to do with this issue, commented on by E Rossignon late 2018 ?
Any suggestions?
Koen

Hi Koen,
yeah sorry, the v2 branch is stil heavily WIP. You should run node ./src/index.js instead of node logger.js. I still need to cleanup some files from the old version, planning on releasing an alpha later this week.

Hello Koen,
v2 is now merged in master, and includes better documentation in the readme.
Could you either try again from source, or download a binary from the release page? Let me know if it works!


Create MongoDB

For the beginning we need a MongoDB to work with our data. It is important to clarify whether we should move to an SQL database later and if so, which one.

One of us should just do it and give feedback when it's done.

Hello,
Can you please assign this issue to me so that I can work on this issue.
Thank you

I have created a DB, the Connections String from Mongo DB looks like this:
mongodb+srv://KaHaRo:@doitcluster1.cuse1ap.mongodb.net/DoIt?retryWrites=true&w=majority
ATTENTION!!! for future projects.
For whatever reason MongoDB Atlas says that the string looks like this
mongodb+srv://KaHaRo:@doitcluster1.cuse1ap.mongodb.net/?retryWrites=true&w=majority here is missing before the ? Of course the DoIt varies depending on the name, for me it was in the Capstone gameapp e.g.
so the full String for IntelliJ should be like this:
MONGO_DB_URI=mongodb+srv://KaHaRo:@doitcluster1.cuse1ap.mongodb.net/DoIt?retryWrites=true&w=majority


Generate ChangeTrackingList and ChangeTrackingDictionary types

Generate ChangeTrackingList and ChangeTrackingDictionary types in the internal folder like this. You can refer the implementation from autorest.csharp: ChangeTrackingListProvider.cs and ChangeTrackingDictionaryProvider.cs.

https://github.com/microsoft/typespec/pull/3378


Adding ifxsensor to documentation index for noetic

I'd like ifxsensor to be indexed and documented on ros.org.

I'd like ifxsensor to be indexed and documented on ros.org.


How to scale facerec_from_webcam_faster?

Like the title says, I would like to know how to scale the webcam facerec program to see multiple faces. Since I can't see the exact code that was used for the gif of multiple faces being recognized, I am unsure of what to do to recognize faces that would be relatively fast.

If you have a relatively small number of known faces, you can just pass them all into the compare_faces() function as an array. The math happening is pretty simple and will take advantage of vector operations on your CPU if available.
If you have lots and lots of known faces, you can look into more complicated approaches like using clustering algorithms, indexed database tables or spatial hashing.

When you say lots, do you mean 100? 1000? I'm looking for something in the 1000 known faces range.

On Jul 3, 2017, at 11:51 AM, Adam Geitgey<EMAIL_ADDRESS>wrote:
If you have a relatively small number of known faces, you can just pass them all into the compare_faces() function as an array. The math happening is pretty simple and will take advantage of vector operations on your CPU if available.
If you have lots and lots of known faces, you can look into more complicated approaches like using clustering algorithms, indexed database tables or spatial hashing.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

I don't have any fixed numbers. I'd just try it and see how fast compare_faces() works for you. If it's too slow, then investigate a more complicated solution. No need to introduce complexity unless the simple solution doesn't work.


Incorrect information

"Dell Inc" should be "Dell Inc."
If you forget the period, the driver injection doesn't work.

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

ID: cb440d06-32d1-c982-7736-0588cb4d8718
Version Independent ID: 37cfa729-ee6d-3af4-dbbd-547797e37397
Content: Build a distributed environment for Windows 10 deployment (Windows 10) - Windows Deployment
Content Source: windows/deployment/deploy-windows-mdt/build-a-distributed-environment-for-windows-10-deployment.md
Product: w10
Technology: windows
GitHub Login: @greg-lindsay
Microsoft Alias: greglin

I tried looking for the text "Dell Inc" on that page, but could not find it. Maybe I did not look carefully enough, but normal search (Ctrl-F) did not find it.
Could you point to where on the page this typo is located?

This is a good catch. The doc isn't this one however, it is deploy-a-windows-10-image-using-mdt.md

Fixed


Animation jitter when only slightly larger than container

I'm getting this animation jitter when the text is only a little longer than the given area:
https://user-images.githubusercontent.com/38788426/218218290-29a37d7f-39f1-4aed-8d1a-d00ac153e307.mov
I tried adding a bounceDelay but that only makes it do the same thing but slower.
Edit: setting bounce={false} does fix it, but if I wanted to let it bounce, how could i prevent it from going this fast?

Hi! Have you tried the duration or bounceSpeed props?

Ah i see, i guess I just expected the bounceDelay to only delay the beginning and not also the end so I was confused. Slowing bounceSpeed (and adding a negative shouldAnimateThreshold) got me to the desired outcome. Thanks!


According to the documentation example, the query highlighting settings will run incorrectly

My  PHP : PHP 7.4.28 (cli)
use :  tag/2.1.1

The error is as follows：
PHP Fatal error:  Uncaught ArgumentCountError: Too few arguments to function MacFJA\RediSearch\Redis\Command\Search::setHighlight(), 1 passed in /Code/redisearch-php/nindex.php on line 58 and exactly 3 expected in /Code/redisearch-php/vendor/macfja/redisearch/src/Redis/Command/Search.php:164
Stack trace:
#0 /Code/redisearch-php/nindex.php(58): MacFJA\RediSearch\Redis\Command\Search->setHighlight(Array)
#1 {main}
  thrown in /Code/redisearch-php/vendor/macfja/redisearch/src/Redis/Command/Search.php on line 164

The interface prototype is as follows
https://github.com/MacFJA/php-redisearch/blob/main/src/Redis/Command/Search.php
/**
     * @param null|array<string> $fields
     *
     * @return $this
     */
    public function setHighlight(?array $fields, ?string $openTag, ?string $closeTag): self
    {
        $this->options['highlight']
            ->setDataOfOption('type', true)
            ->setDataOfOption('fields', $fields)
            ->setTags($openTag, $closeTag)
        ;

        return $this;
    }

Correct execution should be
$search
    ->setIndex('person')
    ->setQuery('Doe')
     ->setHighlight(['lastname'],null,null)
    ->setWithScores();
$results = $client->execute($search);

So that I can execute successfully.

Documentation omissions ?

Not an error in the documentation, it's rather an issue with the code.
The parameter #2 and #3 should be default to null, as the open and close tag are completely optional from the RediSearch documentation

ths


Update Composer dependencies (2019-12-06-00-11)

Visual regression test passed!
View the visual regression test report


Update version in documentation when updating the Flutter SDK

What does this PR do?
When a new version of the SDK is deployed, update the README for the package to have the correct version.
Motivation
Make sure documentation is always up to date with the truth
Review checklist (to be filled by reviewers)

[ ] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[x] Make sure you discussed the feature or bugfix with the maintaining team in an Issue
[ ] Make sure each commit and the PR mention the Issue number (cf the CONTRIBUTING doc)

Codecov Report

Merging #878 (469ad4b) into release/1.12.0 (2dc0d22) will decrease coverage by 0.04%.
The diff coverage is n/a.

@@                Coverage Diff                 @@
##           release/1.12.0     #878      +/-   ##
==================================================
- Coverage           80.22%   80.18%   -0.04%
==================================================
  Files                 254      254
  Lines                8547     8547
  Branches             1463     1463
==================================================
- Hits                 6856     6853       -3
- Misses               1335     1342       +7
+ Partials              356      352       -4

Impacted Files
Coverage Δ

...ain/java/com/datadog/opentracing/PendingTrace.java
51.28% <0.00%> (-7.69%)
:arrow_down:

...rc/main/java/com/datadog/opentracing/DDTracer.java
55.60% <0.00%> (-0.41%)
:arrow_down:

...og/android/rum/internal/anr/ANRDetectorRunnable.kt
85.71% <0.00%> (+3.57%)
:arrow_up:

...android/rum/internal/ndk/DatadogNdkCrashHandler.kt
87.95% <0.00%> (+3.61%)
:arrow_up:


Support Orca's whirpool-cpi crate

Orca is the second largest Solana DEX and concentrated liquidity AMM. They publish an official crate to use their product : https://github.com/orca-so/whirlpool-cpi
Would love to see SOLPG integrate it, as it's the most dev-friendly DEX rn (there's no Raydium DEX) and their functions and structs can get quite messy with no crate importation.
Have a nice sunday.

We should be able to add this, but in Anchor 0.30, you'll be able declare program crates (including CPI client) with the declare_program! macro:
declare_program!(whirlpool);

This will mostly remove the need to import crates separately.

Have a nice sunday.

Thanks, you too!

+1 on the request to integrate the Orca crate. In the meantime, how can I upgrade the anchor version? My solpg seems to be on Anchor 0.29


Added an optional uri property to the DocumentSymbol.

Clients can use this when the URI of the text document
cannot be inferred from the request context.
Issue: https://github.com/Microsoft/language-server-protocol/issues/582
Signed-off-by: Akos Kitta<EMAIL_ADDRESS>

I understand why you are requesting this (because of the type hierarchy request) but IMO this is not the right way to go. DocumentSymbols are as the name implies scoped to a document and we shouldn't start changing this. I would rather opt to have a new type TypeHierarchyItem. This ensures we can evolve them independently. In addition we can make these items lazy to better support large type hierarchies.
Any objection to close this ?

because of the type hierarchy request

Yes :)

I would rather opt to have a new type TypeHierarchyItem.
This ensures we can evolve them independently.

This was the initial idea.

Any objection to close this ?

I do not mind adopting the proposals. What's your take on that, @svenefftinge?

I would rather opt to have a new type TypeHierarchyItem.

IMO the name TypeHierarchyItem is not good since the class could be usefull for cases not related to type hierarchy. E. g. in Eclipse you could list the class symbols + inherited members.

IMO the name TypeHierarchyItem is not good since the class could be usefull for cases not related to type hierarchy. E. g. in Eclipse you could list the class symbols + inherited members.

It should be still possible with the TypeHierarchyItem.
Presumably, you are talking about the Ctrl/Cmd+o functionality in Eclipse. It is your outline: textDocument/documentSymbol.
If you want the inherited members (Ctrl/Cmd+o and Ctrl/Cmd+o again), you can get the supertypes of the current document: textDocument/superTypes, and execute textDocument/documentSymbol for each type.  Finally, you merge the result on the client. Although, I did not check with JDT, I would assume, Eclipse does it in two steps too. @yyoncho, correct me if I've overlooked something.

If you want the inherited members (Ctrl/Cmd+o and Ctrl/Cmd+o again), you can get the supertypes of the current document: textDocument/superTypes, and execute textDocument/documentSymbol for each type. Finally, you merge the result on the client. Although, I did not check with JDT, I would assume, Eclipse does it in two steps too. @yyoncho, correct me if I've overlooked something.

Deciding on whether a symbol is inherited or overridden requires knowledge of language semantics, so I'd say this is something to be provided by the language server and not the client.

The LSP actually tries hard to stay way from this. All the types it declares are bound to a tool action and usually return UI data types. The documentSymbols request would have better be named documentOutline and returned OutlineItems.
For most servers that request doesn't return all symbols defined in a document (for example locals are usually not part of this).
I will close the issue. Please ping if someone disagrees.