During handling of the above exception, another exception occurred:

Do you know how to fix this error?
I was hoping to download 50k images but only 7000 images got downloaded.
[!!]Indexing page: 56

Traceback (most recent call last):
  File "/home/mona/anaconda3/lib/python3.7/urllib/request.py", line 1319, in do_open
    encode_chunked=req.has_header('Transfer-encoding'))
  File "/home/mona/anaconda3/lib/python3.7/http/client.py", line 1252, in request
    self._send_request(method, url, body, headers, encode_chunked)
  File "/home/mona/anaconda3/lib/python3.7/http/client.py", line 1298, in _send_request
    self.endheaders(body, encode_chunked=encode_chunked)
  File "/home/mona/anaconda3/lib/python3.7/http/client.py", line 1247, in endheaders
    self._send_output(message_body, encode_chunked=encode_chunked)
  File "/home/mona/anaconda3/lib/python3.7/http/client.py", line 1026, in _send_output
    self.send(msg)
  File "/home/mona/anaconda3/lib/python3.7/http/client.py", line 966, in send
    self.connect()
  File "/home/mona/anaconda3/lib/python3.7/http/client.py", line 1414, in connect
    super().connect()
  File "/home/mona/anaconda3/lib/python3.7/http/client.py", line 938, in connect
    (self.host,self.port), self.timeout, self.source_address)
  File "/home/mona/anaconda3/lib/python3.7/socket.py", line 707, in create_connection
    for res in getaddrinfo(host, port, 0, SOCK_STREAM):
  File "/home/mona/anaconda3/lib/python3.7/socket.py", line 752, in getaddrinfo
    for res in _socket.getaddrinfo(host, port, family, type, proto, flags):
socket.gaierror: [Errno -3] Temporary failure in name resolution

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "download_images_from_bing.py", line 2, in <module>
    downloader.download('gun', limit=50000, adult_filter_off=True, force_replace=False)
  File "/home/mona/anaconda3/lib/python3.7/site-packages/bing_image_downloader/downloader.py", line 34, in download
    Bing().bing(query, limit, adult)
  File "/home/mona/anaconda3/lib/python3.7/site-packages/bing_image_downloader/bing.py", line 63, in bing
    response = urllib.request.urlopen(request)
  File "/home/mona/anaconda3/lib/python3.7/urllib/request.py", line 222, in urlopen
    return opener.open(url, data, timeout)
  File "/home/mona/anaconda3/lib/python3.7/urllib/request.py", line 525, in open
    response = self._open(req, data)
  File "/home/mona/anaconda3/lib/python3.7/urllib/request.py", line 543, in _open
    '_open', req)
  File "/home/mona/anaconda3/lib/python3.7/urllib/request.py", line 503, in _call_chain
    result = func(*args)
  File "/home/mona/anaconda3/lib/python3.7/urllib/request.py", line 1362, in https_open
    context=self._context, check_hostname=self._check_hostname)
  File "/home/mona/anaconda3/lib/python3.7/urllib/request.py", line 1321, in do_open
    raise URLError(err)
urllib.error.URLError: <urlopen error [Errno -3] Temporary failure in name resolution>

from bing_image_downloader import downloader
downloader.download('cat', limit=50000, adult_filter_off=True, force_replace=False)

It is more like network issue, However I will recheck.


ISBNdb has paywall

In order to get an API KEY, you need to pay a monthly fee even for the basic plan. It has a free trial period of 7 days
https://isbndb.com/isbn-database

Oh you're right. I will remove it.
Thanks for letting us know @Lansaaa ;)


use prerenderer

Hey Thx for this nice example, but i git an error on prerenderer.
the comand i use
npm run build:prerender && npm run serve:prerender

The error
Users/pascalbrewing/htdocs/universal-nest/dist/prerender.js:112618
'function' === typeof document.createEvent ? function CustomEvent (type, params) {
                      ^

ReferenceError: document is not defined
    at Object.eventmap (/htdocs/universal-nest/dist/prerender.js:112618:23)
    at __webpack_require__ (/htdocs/universal-nest/dist/prerender.js:20:30)
    at Object.NativeCustomEvent (/htdocs/universal-nest/dist/prerender.js:112490:19)
    at __webpack_require__ (/htdocs/universal-nest/dist/prerender.js:20:30)
    at Object.<anonymous> (/htdocs/universal-nest/dist/prerender.js:111775:17)
    at __webpack_require__ (//htdocs/universal-nest/dist/prerender.js:20:30)
    at Module.<anonymous> (/htdocs/universal-nest/dist/prerender.js:111017:65)
    at __webpack_require__ (/htdocs/universal-nest/dist/prerender.js:20:30)
    at Object.ng2-dragula (/htdocs/universal-nest/dist/prerender.js:105256:18)
    at __webpack_require__ (/htdocs/universal-nest/dist/prerender.js:104518:30)
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>generate:prerender: `cd dist && node prerender`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>generate:prerender script.
npm ERR! This is probably not a problem with npm. There is likely additional logging

i didnt change anything in the code.
infos :
node -v            v8.11.3
npm -v              6.4.1

ok what i can see is a change in package-lock.josn
before :
 "@nestjs/ng-universal": {
      "version": "0.0.1",
      "resolved"<EMAIL_ADDRESS>      "integrity": "sha512-/2igQ7+SpfY2GfjxIbU54M4N5UtMCcs71JyBMr2A72mi8MPBYRrXBz/X1iuKQKU1MX3mjA1KsPJU9z6b7BgRZA=="

after
# "version": "0.1.0",
      "resolved"<EMAIL_ADDRESS>      "integrity": "sha512-WGg5+im39/++y/4NkKj3OIeMDqK//4AQ9HBh5xGmocqCKXRNSuY2AujXw5aU4k4RPh9ewhNLcuC/STqhFvy/ww=="

Very weird. I just pulled the repo again and everything works correctly for me

@DrMabuse23 have you solved ?

@sxlwar yes same on me i pulled again everything works

i dont know something wrong with me but I have the same problem


User-provided game piece poses for 3D field

Add object types to set game piece poses. Possible use cases:

Show game pieces within the robot.
Visualize game piece localization.
Animate ball trajectories.

https://discord.com/channels/176186766946992128/528555967827148801/1151636188067811450

Game pieces are now supported in the 3D field definition. The next release will use a new set of field models that support this feature.


Github feed example shows "function" as valid key for Feed

Github feed example shows this as valid:
git_webhook:
  version: 1.0
  license: Apache-2.0
  feeds:
    webhook_feed:
      version: 1.0
      function: github/webhook.js
      runtime: nodejs@6

however, the key "function" is not supported in the actual spec.  Perhaps this is an attempt at some shortcut that we have not discussed/vetted properly?
The only way to ref a function is indirectly, using an "action" (i.e. the name of an action also declared in the manifest file.

Note: we really need to develop Feed into being a first-class citizen of the manifest, that is, we need to:

Support "operation" declarations along with the Input Parms.
Support actual Credential datatype
future: work to document error codes ???

and encourage not having ALL parameters available for ALL operations (wasteful/inaccurate).


cargo-semver-checks: 0.30.0 -> 0.31.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/obi1kenobi/cargo-semver-checks/releases.
meta.description for cargo-semver-checks is: A tool to scan your Rust crate for semver violations
meta.homepage for cargo-semver-checks is: https://github.com/obi1kenobi/cargo-semver-checks
meta.changelog for cargo-semver-checks is: https://github.com/obi1kenobi/cargo-semver-checks/releases/tag/v0.31.0
Updates performed

Rust version update

To inspect upstream changes

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.31.0 with grep in /nix/store/w2mc4rav1knfghpmyqb31xb3alqpgk0k-cargo-semver-checks-0.31.0
found 0.31.0 in filename of file in /nix/store/w2mc4rav1knfghpmyqb31xb3alqpgk0k-cargo-semver-checks-0.31.0

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

cargo-semver-checks

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/w2mc4rav1knfghpmyqb31xb3alqpgk0k-cargo-semver-checks-0.31.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A cargo-semver-checks https://github.com/r-ryantm/nixpkgs/archive/e4285fcc4d991d20ff609cec1e9bf01fc86ae7ef.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/e4285fcc4d991d20ff609cec1e9bf01fc86ae7ef#cargo-semver-checks

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/w2mc4rav1knfghpmyqb31xb3alqpgk0k-cargo-semver-checks-0.31.0
ls -la /nix/store/w2mc4rav1knfghpmyqb31xb3alqpgk0k-cargo-semver-checks-0.31.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review --extra-nixpkgs-config '{ allowInsecurePredicate = x: true; }' run on x86_64-linux 1

1 package built:

cargo-semver-checks

Maintainer pings
cc @figsoda @matthiasbeyer for testing.

Result of nixpkgs-review pr 306063 run on x86_64-linux 1

1 package built:

cargo-semver-checks


Importance of exactly 2 spaces (versus tabs or >2 spaces) in Markdown

Please see this related issue as to why it's important use exactly 2 spaces in markdown files:
https://github.com/donejs/developer-relations/issues/25

ESLint doesn't interact with markdown files, so I don't think it should cause any issues here.

I'm all on board the 2 spaces train personally. And while ESLint on it's own doesn't interact with markdown files. I think it is important for projects to start including https://github.com/eslint/eslint-plugin-markdown in order to make sure their examples are properly linted.


Bug in module resolution

Bug report

What is the current behavior?
After upgrade to Webpack@5, in development mode with HMR, module resolution fails for indirect dependencies via webpack-bundled libraries.
If the current behavior is a bug, please provide the steps to reproduce.
Here is the repo with minimal reproduction of the issue: https://github.com/birdofpreyru/webpack5-bug1

Clone it, and build the mock library
$ git clone https://github.com/birdofpreyru/webpack5-bug1.git
$ cd webpack5-bug1/lib
$ npm install
$ npm run build
$ npm pack

The mock library just imports lodash library and prints its version (can be any other import and use of NPM module - that's does not matter).

Go to the host code, and run it (the bundle is served using HtmlWebpackPlugin and webpack-dev-server), using the mock library from (1):
$ cd ../host
$ npm install
$ npm install ../lib/lib-1.0.0.tgz
$ npm start

In browser go to localhost:8080, and look into the console:
Uncaught TypeError: Cannot read property 'call' of undefined
at __webpack_require__ (main.js:2)
at eval (index.js:1)
at Object../index.js (main.js:2)
at __nested_webpack_require_562843__ (main.js:2)
at main.js:2
at factory (main.js:2)
at Object.823 (main.js:2)
at __webpack_require__ (main.js:2)
at main.js:2
at main.js:2

it fails inside the webpack require function during request for lodash at the line
__webpack_modules__[moduleId].call(module.exports, module, module.exports, __webpack_require__);

It looks like that __webpack_modules__ is the map of modules which are direct dependencies of host code, while lodash module lives inside __webpack_modules__ map bundled into the mock library. The same error happens if lodash is declared as external dependency of the mock library.
At the same time it all works fine if everything is bundled for production and served without dev server.

What is the expected behavior?
Module resolution must not fail. In this specific example lodash should be resolved, and you should see in the console debug messages printed by the library and by the host code:
LODASH: 4.17.20
Hello World!

Other relevant information:
webpack version:  5.1.3
Node.js version: 12.19.0
Operating System: Ubuntu 20.04.1
Additional tools: none

Could you try to disable devtool: false when building the lib?

Tested. Indeed, with devtool: false it works as expected, the same with any option value not containing eval; with devtool: 'eval', or any other eval-*** value, it fails as described. Sure, I just checked a few values from the full set, but it looks like the problem is caused by wrapping modules in eval(..) and source map variations do not matter.
@sokra

Yep this was reported before, but I don't find the issue.
It makes sense to fix that, but in general avoid bundling libraries to be bundled again. Double bundling is inefficient. A library should be released as individual files not as bundle.

Thanks for the advice @sokra ! Is it also true for double bundling when everything is bundled in production / minimized mode, thus without eval() wrappings? I was relying on that to speed-up compilation of a large codebase: split a less frequently updated code, reused across severall apps, into a library bundled by Webpack, then any host code dependent on such library had to do a way less work to bunle in that pre-bundled library, saving a noticeable amount of time.

I can reproduce the bug even with devtool: false.
My project does work as expected with Webpack 4, but fails on Webpack 5.

@daniele-orlando Please create reproducible test repo

Please create a new issue

Yep this was reported before, but I don't find the issue.
It makes sense to fix that, but in general avoid bundling libraries to be bundled again. Double bundling is inefficient. A library should be released as individual files not as bundle.

https://github.com/webpack/webpack/issues/11277

Yes, duplicate of https://github.com/webpack/webpack/issues/11277, let's close


Allow raw PDF objects to be retrieved in display layer.

There should be a way to retrieve the raw PDF objects in a PDF file using PDF.js. This is both useful for inspecting PDF as well as being able to edit PDF files using incremental updates outside of PDF.js (see an example below; my further goal is to be able to add annotations to PDF files).
In previous versions of PDF.js, this was possible by using PDFDocument.xref.fetch() directly as done by pdf.js.utils. PDF.js versions using a Web Worker no longer exposed an API for this.
A demo demonstrating the new API can be found at:
http://ebenda.org/2014/pdf.js.utils/browser/ (updated pdf.js.utils including the possibility to extract raw PDF streams (e. g. click downloadRaw on a DCTDecode/JPEG image) and generate PNG files out of some FlateDecode images (only results in a valid PNG file if a Predictor is present), source at https://github.com/rgieschke/pdf.js.utils ).
http://ebenda.org/2014/pdf-update/build/generic/web/viewer.html (viewer including the possibility to delete the first page of a PDF file, source at https://github.com/rgieschke/pdf.js/blob/pdf-update/web/update.coffee ).

Has anyone implemented the requested changes from comments and pushed?

Closing in favor of #6266, which is exactly this PR but with the comments addressed.


updateSchema error

Can't seem to be able to write simple query with my schema.
When running upadateSchema I always get Error: Ticket.id field type must be Output Type but got: [object Object].
I tried mimicking the one on the tutorial but to no avail
var queryType = new GraphQLObjectType({
  name: 'Query',
  fields: () => ({
    node: nodeField,
    ticket: {
      type: ticketType,
      resolve: () => getTicket(),
    }
  })
});

I must be missing something.
Thanks

Thanks for the question! We are trying to keep questions in the same place. Would you mind copy-and-pasting this question into the Relay issue tracker? https://github.com/facebook/relay


Key ID(kid) missing

With a commit from last month jitsi requires the kid parameter to be set which is currently not sent:
https://github.com/jitsi/jitsi-meet/commit/3a8bd852b277788925bba5f184e59e605f239cf3
   const { kid } = header;
    // if Key ID is missing, we return the error immediately without further validations.

2021-04-20T13:09:17.594Z [features/base/conference] JWT parsing error:
Array [ "- Key ID(kid) missing" ]
Logger.js:154:22
2021-04-20T13:09:17.614Z [JitsiMeetJS.js] :  UnhandledError: connection.passwordRequired Script: null Line: null Column: null StackTrace:  n/a Logger.js:154:22
Uncaught (in promise) connection.passwordRequired

Hi,
i think this is an issue with the PR you mentioned. This project uses a shared secret to sign the JWT.
From the jitsi docs:

Alternately the token may be signed by a private key and authorized via public keyserver using RS256 tokens. In this mode, the 'kid' header of the JWT must be set to the name of the public key. The backend server must be configured to fetch and confirm keys from a pre-configured public keyserver.

So the kid parameter is only required (and makes sense) if the JWT was signed using a private key. Then the kid param would point to the public key which will be used by prosody to verify the signature.
I will open an issue on the jitsi repo to clarify this issue, thanks for the report 👍

The kid warning was fixed in jitsi with https://github.com/jitsi/jitsi-meet/pull/9146.
However the author mentioned that the kid error is not breaking, it did not prevent the auth flow.
So the error might be caused by an actual issue in JWT.
Test it with the current jitsi version and open a new issue if the error still persists.


For CONSOLE Backend Builds: RequestTimeout: Your socket connection to the server was not read from or written to within the timeout period. Idle connections will be closed.

Before opening, please confirm:

[X] I have checked to see if my question is addressed in the FAQ.
[X] I have searched for duplicate or closed issues.
[X] I have read the guide for submitting bug reports.
[X] I have done my best to include a minimal, self-contained set of instructions for consistently reproducing the issue.

App Id
dcaffs0m7jnrr
Region
us-east-2
Amplify Hosting feature
Backend builds, Build settings, Monorepo
Describe the bug
Application: dcaffs0m7jnrr
Branch: led-links
BuildId: 5
2022-07-29T05:40:03.063Z [WARNING]: - Uploading files...
2022-07-29T05:45:30.737Z [WARNING]: ✖ An error occurred when pushing the resources to the cloud
2022-07-29T05:45:30.760Z [WARNING]: ✖ There was an error initializing your environment.
2022-07-29T05:45:30.767Z [INFO]: RequestTimeout: Your socket connection to the server was not read from or written to within the timeout period. Idle connections will be closed.
                                     at Request.extractError (/snapshot/repo/build/node_modules/aws-sdk/lib/services/s3.js:711:35)

Most reports I've seen of this issue happen locally (not in console):
https://github.com/aws-amplify/amplify-cli/issues/4113
https://github.com/aws-amplify/amplify-cli/issues/6248
https://github.com/aws-amplify/amplify-cli/issues/8379
https://github.com/aws-amplify/amplify-cli/issues/9019
https://github.com/aws-amplify/amplify-cli/issues/2121
https://github.com/aws-amplify/amplify-cli/issues/501
https://github.com/aws-amplify/amplify-cli/issues/462
Expected behavior
Build successfully, like my local amplify push.
Reproduction steps
Click 'Redeploy this version'
Build Settings
amplifyPush --simple

Additional information
I guess my local machine is faster than the build image, so I don't often have this issue locally.
Also, my graphql schema is huge.
Is there a way to get a beefier build image or any other workaround for this issue?

Hi @clownbaby apologies for the delay. I think the only workaround you can try is to push changes to your backend resources via the CLI and then skip the backend build during the CI/CD process. You can add environment variable AMPLIFY_SKIP_BACKEND_BUILD in the console and set it to true. This way, your backend is updated and the build for the frontend shouldn't fail since the backend build will be skipped.

Or you can try to create a manually deployed app instead.

OK when will this bug be fixed?  Is there a document where progress on this bug can be tracked?


fix: print error when test record file doesn't exist

Fixes
#1231
Changes
F3D now errors out when the record file doesn't exist.

The CI jobs are failing though. It needs to be fixed first.

Should I create a temp file in parent directory and check if it succeeds? or is there a better way

wait what happened..

for the inexistent path, just pick pick an invalid dir in the testing dir.
For the path without permissions, you can create a file without write permission in the testing dir and try writing to it.

@mwestphal The tests are failing in Windows and Linux GitHub runners. For some reason changing directory permission is not working as expected. On Mac however the permissions are set correctly.

I tried locally on my linux and this is working fine, strange.

maybe try putting no permissions at all ?

I tried locally on my linux and this is working fine, strange.

The issue is only happening inside github runners. specifically windows and linux.
In both of these platform cmake fails to set appropriate permission. The test expects a non writable directory.
I have tried running raw chmod command and it fails too.
If i remove this test then coverage test will fail

Let's remove this test. Coverage reduction is ok when justified

https://github.com/f3d-app/f3d/blob/b40fba3e10401e34816926818a29e47571926777/library/src/interactor_impl.cxx#L684-L689
This todo can be removed now?


KVM HyperV Enlightment for Improved Windows Server 2008+ Performance

Description
Windows has support for several paravirt features that it will use when running on Hyper-V, Microsoft's hypervisor. These features are called enlightenments. Many of the features are similar to paravirt functionality that exists with Linux on KVM (virtio, kvmclock, PV EOI, etc.)
Nowadays QEMU/KVM can also enable support for several Hyper-V enlightenments. When enabled, Windows VMs running on KVM will use many of the same paravirt optimizations they would use when running on Hyper-V.
A number of years ago, a PR was introduced that added a good portion of the code to enable this feature set, but it was never completed. This PR enables the existing features. The previous patch set detailed in https://github.com/apache/cloudstack/pull/1013 also included the tests.
By selecting Windows PV, the enlightenment additions will be applied to the libvirt configuration. This is support on Windows Server 2008 and beyond, so all currently supported versions of Windows Server.
In our testing, we've seen benchmark improvements of around 20-25% running on Centos 7 hosts and it is also supported on Centos/RHEL 6.5 and later. Testing on Ubuntu would be appreciated.
Types of changes

[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] New feature (non-breaking change which adds functionality)
[ ] Bug fix (non-breaking change which fixes an issue)
[X] Enhancement (improves an existing feature and functionality)
[ ] Cleanup (Code refactoring and cleanup, that may add test cases)

How Has This Been Tested?
We've tested this in a lab and QA environment. I'll post some screenshots related to the libvirt xml changes and also provide some performance metric information over the next 24 hours.
Checklist:

[x] I have read the CONTRIBUTING document.
[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
##Testing:
[ ] I have added tests to cover my changes. (not required as the previous PR included the tests)
[ ] All relevant new and existing integration tests have passed.
 I'll keep you posted as I make progress.

Packaging result: ✔centos6 ✔centos7 ✔debian. JID-2326

@blueorangutan test

@borisstoyanov a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

Trillian test result (tid-3051)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 25545 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr2870-t3051-kvm-centos7.zip
Intermittent failure detected: /marvin/tests/smoke/test_multipleips_per_nic.py
Intermittent failure detected: /marvin/tests/smoke/test_privategw_acl.py
Intermittent failure detected: /marvin/tests/smoke/test_vpc_redundant.py
Intermittent failure detected: /marvin/tests/smoke/test_hostha_kvm.py
Smoke tests completed. 66 look OK, 3 have error(s)
Only failed tests results shown below:

Test
Result
Time (s)
Test File

test_nic_secondaryip_add_remove
Error
32.63
test_multipleips_per_nic.py

test_03_vpc_privategw_restart_vpc_cleanup
Failure
1143.81
test_privategw_acl.py

test_04_rvpc_network_garbage_collector_nics
Failure
497.31
test_vpc_redundant.py

Ok, thanks for the feedback guys. I'll refactor this when I have some time later this week.

libvirt XML when enabled -

libvirt XML when disabled -

@kiwiflyer can you see travis error and fix the checkstyle issue.

@kiwiflyer still the same error, please check your changes:
[ERROR] Failed to execute goal org.apache.maven.plugins:maven-checkstyle-plugin:3.0.0:check (cloudstack-checkstyle) on project cloud-plugin-hypervisor-kvm: Failed during checkstyle execution: There is 1 error reported by Checkstyle 8.7 with cloud-style.xml ruleset. -> [Help 1]

@andrijapanic I'm open to changing it to use all Windows OS types. My mine reason for picking one was some backwards compatibility. At least in the KVM world, I'm unsure why we continue to maintain tons of different OS types/names, as running any OS without virtio is extremely painful in terms of performance.

I agree on VirtIO stuff absolutely. It's just that, even with all FAQ and so on, still end-user sometimes continue to use specific a Windows OS type instead of Windows PV, so it makes sense to solve all Windows OS types with this nice patch.
As for future support of OS types inside ACS, that is a good point, but doesn't warrant not applying patch for all of Win OS types, in my opinion.


Text is not legible

The text in the filter dropdown is not legible due to the visibility of text in the cards behind it. I've attached the screenshots:

I can fix this issue, please assign it to me

I'll soon list some more issues to work on and I will also add the required tags and docs as per hacktoberfest guidelines.


Error on installing poolParty

Installing poolParty throws this error at the final step:
Fetching Stride's code... Done

Where do you want to install your stride and cosmovisor binaries? [default: /root/go/bin] y

Building Stride... Done

============================================================================================

Almost there! You'll also need cosmosvisor which will enable automatic upgrades.

Installing now!
This one might take a few minutes...
mv: cannot stat '/root/.stride/cosmovisor/cosmovisor': No such file or directory

The folder name in the init script was wrong. fixed it.


wasm-build improvements

While building a contract, there should be 2 wasm files produced

regular wasm contract with optimized symbols for [_call, setTempRes0]
constructor wasm contract, where _create function is extended to return contract code and renamed to _call, then symbols are optimized again for [_call, setTempRes0]

Currently wasm-build works by looking 2 functions inside 1 wasm.
The variant described here might be one of the options for wasm-build to pack (imagine crate where 2 binaries  - one for constructor, one for code)


[backport] Windows: Fix histogram test failures

Backport of #4777

[automatic post] pfnCI, test this please.

Jenkins CI test (for commit b5fd4011b95b9f3f6a06dda166338ee0ae44537b, target branch v8) succeeded!

LGTM.


fix: correct quiet CURL_VERBOSITY

fixes #900
Co-authored-by: rasa<EMAIL_ADDRESS>

Yes, but the -q has been used correctly for the intended purpose to avoid
issues from reading local configs. It probably got lost from the --quiet
case when that was implemented because -s is quiet in curl, but without -q
in both cases there can be issues.
On Tue, 24 Oct 2023, 17:28 Ross Smith II, @.***> wrote:

@.**** commented on this pull request.
In deb-get
https://github.com/wimpysworld/deb-get/pull/917#discussion_r1370495576:

@@ -656,7 +656,7 @@ function update_repos() {
local WGET_VERBOSITY="-q"

 if [[ " $* " == *' --quiet '* ]]  ; then

    CURL_VERBOSITY="-Ss"

    CURL_VERBOSITY="-Ss -q"

The -q option doesn't quiet curl, -s does.
The -q option tells curl to not read the user's .curlrc file. So, imho,
that option should be added to all the curl calls (or even better, passed
via an environment variable because some users may want curl to read it's
config, so export CURL_OPTS='-q')
—
Reply to this email directly, view it on GitHub
https://github.com/wimpysworld/deb-get/pull/917#pullrequestreview-1695369845,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACDDWY2VNRTBGUZSMWVS3GTYA7UDDAVCNFSM6AAAAAA6M3VXVCVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMYTMOJVGM3DSOBUGU
.
You are receiving this because you authored the thread.Message ID:
@.***>


Mollie payments options do not show at checkout in prestashop paying step

Expected behavior

Actual behavior

Steps to reproduce the behavior

Everything is configuert correctly and api are connected succesfully, but still mollie is showing a error code in log information dashboard with adminmollie error ?
Environment

PrestaShop version: <IP_ADDRESS>
Mollie module version: 4.2.4
PHP version: 7.2

Hi @abdel23421,
Thank you for your message!
 We have an article regarding your situation that I believe will resolve your issue! Please find it here - https://github.com/mollie/PrestaShop/wiki/Troubleshooting#payment-method-does-not-appear-in-front-office
Let me know if it's helpful or not!
--
Best regards,
Invertus Support team

Hi @abdel23421 ,
Did you managed to get your payment methods to show or not?
Let us know!
--
Best regards,
Invertus Support team

Yes, it worked. Thank you for that. But some other problem accurate with the speed of the website when loading. It takes a long time to load mollie, after did a scan i saw that the javascript used for this module, Is old and is upgradeable. If it 's possible could the javascript get an update so it will be faster when loading.

Hey @abdel23421 ,
Thank you for the feedback!
I will definitely check what are the possibilities of that happening, inform Mollie team, and we will see what can be done.
I am marking this issue as resolved, if you'll have any other issues, let us know by raising a new issue!
--
Best regards,
Invertus Support team

@abdel23421 ,
Could you also clarify the name of your website? And maybe clarify a bit further where exactly is the js getting blocked and slows the website down?
--
Best regards,
Invertus Support team


add description field to VPC peering connections

the purpose for VPC peerings is not always obvious. this change
introduces optional description fields to allow documentation
of VPC peering connections.
Ref: https://issues.redhat.com/browse/APPSRE-4428
Signed-off-by: Gerd Oberlechner<EMAIL_ADDRESS>

LGTM


UI makes it too easy to create a function with an invalid EventHub connection string

Steps:

New EventHubTrigger C# function
[Novice used hoping for some kind of good default]: just press Create

Result: you end up with a badly broken connection in function.json
"connection": "<AppSetting name pointing to your EventHub receiving connection string>",
This then causes nasty pop ups: Microsoft.ServiceBus: The argument Endpoints is null or empty.  Parameter name: Endpoints.
We should not allow the user to click Create without having properly picked an connection.

Related to https://github.com/Azure/azure-webjobs-sdk-templates/pull/124. We've fixed that bad default - not sure what additional work we have above the required property validation the portal is already doing ("connection" properties are required).

That change might be good enough. Right now it's one of the top errors. I'll close and we'll monitor if it still comes up.


Feature Request: Immutable [List|Set|Map]

I use collections (list, map, set) a lot. One of the pains of .NET is the legacy around collection interfaces: no support for immutable.
The best one gets is IList.IsReadOnly, which is too little to late.
My humble request:
  ImmutableList<E>
  ImmutableMap<K, V>
  ImmutableSet<E>
The common pattern, in .NET (and it seems in Dart) is to expose the existing types and then throw exceptions in mutate operations (add, remove, clear, etc)--at least from looking at ImmutableList<E>.
This creates clutter in documentation and exceptions at runtime that could easily be resolved by the type system.
I want to expose lists, maps, and sets via properties without having to wrap everything in a wrapper and confuse my user.
Is myObject.items.clear() supported or not?
I want to accept lists, maps, and sets via method/constructor/factory paramaters with the explict promise that I intend to inspect, but not mutate the input.
Scala has an amazing approach here by having a whole namespace of immutable collection interfaces (and classes).
http://www.scala-lang.org/docu/files/collections-api/collections_1.html
While I’d love to have in-the-box immutable collections in Dart, I’d settle for common interfaces that serve as superclasses for the mutable versions.
While I fully support pushing off non-essential features for later, I must plead a bit here.
If v1 ships without immutable collection support, it will be prohibitive to introduce this concept later.
Help us ship better code.

why was this issue merged into #236?
that issue is entitled "Immutable List/Map".  notice how the word "Set" isn't in there.
in order to mark this issue as a dup of that one, you either have to expand the scope of
that issue to include Sets (at which point it would have to be re-opened)
or you would have to un-dup this one.
at any rate, dart still doesn't have an immutable/unmodifiable Set with an interface similar to that for List and Map.  e.g. dart does have UnmodifiableListView and UnmodifiableMapView, but no UnmodifiableSetView.  yes, i know that there is built_collection, but i said "...with an interface similar to..." and such functionality should be equally core as UnmodifiableListView.
another example of the irregular treatment of Set:
List and Map both have an unmodifiable constructor, but Set doesn't.
for reference, #11256 also requests an unmodifiable Set and it was closed as fixed, but i can't figure out why since the request in the title doesn't seem to be resolved.


Add Macchina branding/theming

Right now the default GitBook theme is used.  While this reads well, it would be nice to have more connection with the Macchina brand to make it look more official.

Fixed by #13.  Thanks @ndiesslin!


fix: support for block devices

Currently it's not possible to edit devices (e.g. /dev/sda1) since a new file is created and then moved to its destination.

This is an intended implementation. Currently the bed command continues to read the original file contents, so the writing command must create a new file. Try deleting single byte at the head to see what happens on writing. Also the bed command sometimes deals with a large file contents and may crash while writing, so the original file may corrupt without atomic write.

Ah I see, yeah, if an edit can result in a different file size that's an issue (I only changed some bytes thus I didn't notice). But if the file size remains the same you could only seek and write the modified bytes which for large files or devices would be faster anyway. So how about this: if the file size remains the same, only write the changed bytes (so we need to keep track of it) otherwise you the write and rename approach. Would you see a problem with that?

Yeah I tend to agree, it will require quite some effort to implement this. Meanwhile I found hecate which only needed minor patching since it already uses mmap under the hood to edit device partitions (which has issues with undo of course but is good enough for my use-case). Closing this merge request since it needs more clarification upfront. If someone else wants to take this up, feel free, I won't have time for this in the foreseeable future.

I have an attempt to fix the issue with the following patch. Can you test it?
diff --git a/window/manager.go b/window/manager.go
index 4e3866c..965bcc6 100644
--- a/window/manager.go
+++ b/window/manager.go
@@ -553,6 +553,24 @@ func (m *Manager) writeFile(r *event.Range, name string) (string, int64, error)
 		window.name = filepath.Base(name)
 		window.mu.Unlock()
 	}
+	if fi, err := os.Lstat(name); err == nil && fi.Mode()&os.ModeType == os.ModeDevice {
+		f, err := os.OpenFile(name, os.O_RDWR, 0)
+		if err != nil {
+			return name, 0, err
+		}
+		defer f.Close()
+		if err := f.Truncate(0); err != nil {
+			return name, 0, err
+		}
+		n, err := window.writeTo(r, f)
+		if err != nil {
+			return name, 0, err
+		}
+		if window.filename == name {
+			window.savedChangedTick = window.changedTick
+		}
+		return name, n, nil
+	}
 	tmpf, err := os.OpenFile(
 		name+"-"+strconv.FormatUint(rand.Uint64(), 16),
 		os.O_RDWR|os.O_CREATE|os.O_EXCL, m.filePerm(name),

Hi,
your patch doesn't work since f.Truncate(0) doesn't work for devices (see man(2) ftruncate):
truncate /dev/loop0p1: invalid argument

Just omit the Truncate and check that the file size doesn't change (think about device partitions: you must not modify
their bounds!).
Kind Regards,
Michael


Russian translation

ru_ru.json

Thanks, I'll add this file to the project.

For the next updates, feel free to create a pull request to add the language to the project by yourself.


Object.activate is deprecated.

Got this from Atom deprecation warning:
AtomLinter v0.X.Y API has been deprecated. Please refer to the Linter docs to update and the latest API: https://github.com/AtomLinter/Linter/wiki/Migrating-to-the-new-API
Object.activate (/home/jborg/.atom/packages/Linter/lib/main.coffee:57:8)
Package.activateNow (/usr/share/atom/resources/app.asar/src/package.js:240:19)
<unknown> (/usr/share/atom/resources/app.asar/src/package.js:221:30)
Package.measure (/usr/share/atom/resources/app.asar/src/package.js:165:15)
Package.activate (/usr/share/atom/resources/app.asar/src/package.js:213:14)
PackageManager.activatePackage (/usr/share/atom/resources/app.asar/src/package-manager.js:452:21)

@badray would you wanna take this one or do you want someone else to do it?

@steelbrain Every pull requests are welcome :+1: So if someone have time to do it I would be welcome.

I think I'm gonna take this one

@steelbrain Have fun! :)

I'm caught up in some other work, if you wanna take it, please do :)

+1

:+1:

+1

Same error but different rows in source files:
Object.activate - C:\Users\jkw\.atom\packages\linter\lib\main.coffee:51:8 Package.activateNow - C:\Users\jkw\AppData\Local\atom\app-1.0.3\resources\app.asar\src\package.js:243:19 <unknown> - C:\Users\jkw\AppData\Local\atom\app-1.0.3\resources\app.asar\src\package.js:224:30 Package.measure - C:\Users\jkw\AppData\Local\atom\app-1.0.3\resources\app.asar\src\package.js:167:15 Package.activate - C:\Users\jkw\AppData\Local\atom\app-1.0.3\resources\app.asar\src\package.js:216:14 PackageManager.activatePackage - C:\Users\jkw\AppData\Local\atom\app-1.0.3\resources\app.asar\src\package-manager.js:453:21

Got this after todays update:
AtomLinter legacy API has been removed. Please refer to the Linter docs to update and the latest API: https://github.com/atom-community/linter/wiki/Migrating-to-the-new-API
Report Issue
Object.activate - C:\Users\jkw.atom\packages\linter\lib\main.coffee:52:6
Package.activateNow - C:\Users\jkw\AppData\Local\atom\app-1.0.3\resources\app.asar\src\package.js:243:19
 - C:\Users\jkw\AppData\Local\atom\app-1.0.3\resources\app.asar\src\package.js:224:30
Package.measure - C:\Users\jkw\AppData\Local\atom\app-1.0.3\resources\app.asar\src\package.js:167:15
Package.activate - C:\Users\jkw\AppData\Local\atom\app-1.0.3\resources\app.asar\src\package.js:216:14
PackageManager.activatePackage - C:\Users\jkw\AppData\Local\atom\app-1.0.3\resources\app.asar\src\package-manager.js:453:21

This will be completely broken until https://github.com/AtomLinter/linter-flake8/pull/56 is reviewed/merged.


Reorg providers

Separate open source and commercial providers to reduce clutter

@dotnet-policy-service agree

I'm not sure the current list is "cluttered", it just doesn't separate free/OSS from commercial providers... Any particular reason you think this should be done?

I'll discuss this with the team and we'll see what people think.

If we decide to merge, I think the "Open Source" and "Commercial" headings need change. These two are not mutually exclusive.


[Event Request] codeunit 816 "Purch. Post Invoice" - RunOnCalculateVATAmountsAfterVatCalculationType

Describe the request
Hello, can you please add a new Event RunOnCalculateVATAmountsAfterVatCalculationType in codeunit 816 "Purch. Post Invoice"?
    local procedure CalculateVATAmounts(PurchHeader: Record "Purchase Header"; var InvoicePostingBuffer: Record "Invoice Posting Buffer")
    var
.
.
.
                            GLSetup.Get();
                            if GLSetup."Additional Reporting Currency" <> '' then
                                InvoicePostingBuffer."VAT Amount (ACY)" :=
                                    CurrExchRate.ExchangeAmtLCYToFCY(
                                        PurchHeader."Posting Date", GLSetup."Additional Reporting Currency",
                                        InvoicePostingBuffer."VAT Amount", 0);
                            InvoicePostingBuffer.Modify();
                        end;
                    else
                        PurchPostInvoiceEvents.RunOnCalculateVATAmountsAfterVatCalculationType(TempInvoicePostingBuffer, PurchHeader, VATPostingSetup);
                end;
            until InvoicePostingBuffer.Next() = 0;
    end;

codeunit 826 "Purch. Post Invoice Events"
.
.
.
    procedure RunOnCalculateVATAmountsAfterVatCalculationType(var TempInvoicePostingBuffer: Record "Invoice Posting Buffer" temporary; var PurchHeader: Record "Purchase Header"; VatPostingSetup: Record "VAT Posting Setup")
    begin
        OnCalculateVATAmountsAfterVatCalculationType(TempInvoicePostingBuffer, PurchHeader, VatPostingSetup);
    end;

    [IntegrationEvent(false, false)]
    local procedure OnCalculateVATAmountsAfterVatCalculationType(var TempInvoicePostingBuffer: Record "Invoice Posting Buffer" temporary; var PurchHeader: Record "Purchase Header"; VatPostingSetup: Record "VAT Posting Setup")
    begin
    end;

Additional context
We have an additional vat case.
Internal work item: AB#550246

Availability update: We will publish a fix for this issue in the next update for release 25.
Build ID to track: 26491.


Attempt to ensure metric names are always unique

Only unique metric names will appear in New Relic, so text is added to the metric name to ensure that all reported metrics have unique names.

Could you give me some more information on the scenario where you ended up with duplicate metric names?

I'm selecting data about a process, where there were two instances of that process running on the system. I am selecting data about both processes, but New Relic was treating that data as though it was the same process, where I want them to be recorded and reported separately.
The entry plugin.json is similar to the following: {"provider": "PerfProc", "category":"Process", "instance":"executable", "counter":"ThreadCount", "unit": "threads"}

That makes sense. Let me look at the counter object and see if it's possible to derive a unique name directly from the object instead of just adding an incrementing number.


Access forbidden,

Unhandled Rejection (Error): failed to get balance of account Forbidden:  {"jsonrpc":"2.0","error":{"code": 403, "message":"Access forbidden, contact your app developer or<EMAIL_ADDRESS>

Hi Guys anyone getting this issue ?

Yes. localhost3000 launches in browser after 'yarn start' cmd, shows 'Connect' button, but as soon as I connect my wallet, the page reloads with the same error you posted. Tried removing candy machine mint and starting git clone over, cleared temp files, cookies, restarted computer... same problem. Was not doing this a couple hours ago. HELP!

yes i belive something is really broken with candy machine mint
@levicook can you help please ASAP .. something wrong  , i belive its with candymachines

all rcp calls are falling with below error @levicook
03 Forbidden:  {"jsonrpc":"2.0","error":{"code": 403, "message":"Access forbidden, contact your app developer or<EMAIL_ADDRESS>

Hi @levicook, this is what I get when connecting my wallet. Thanks!

I am getting the same thing. Anyone having any luck?

The script is abusing an endpoint that is dedicated for explorer.solana.com.

The script is abusing an endpoint that is dedicated for explorer.solana.com.

Sorry, Im new to all this. Do you know of a way to fix it? Or what I can look into to fix it?

The script is abusing an endpoint that is dedicated for explorer.solana.com.

Thank you. I'll get my own rpc host and update here it solves the issue.

Im running exactly same problem.

Looks like there is an "uncaught promise error"?  This is what I am getting from my developer tools console.
connection.ts:2172 Uncaught (in promise) Error: failed to get balance of account 3QySa22MnWJTP1HbgF6hacQ8PXJeFwkF2ZNTZTMFPE1r: Error: 403 Forbidden:  {"jsonrpc":"2.0","error":{"code": 403, "message":"Access forbidden, contact your app developer or<EMAIL_ADDRESS>"id": "f7f93027-ee56-4265-a504-4546316c3deb" }
at connection.ts:2172

Im running exactly same problem.

Okay so I just talked to a dev from another project as well as emailed<EMAIL_ADDRESS>Looks like they have blocked using https://explorer-api.mainnet-beta.solana.com as the REACT_APP_SOLANA_RPC_HOST. If you go to https://www.quicknode.com/. Create an account and then create a node for SOLANA it will give you a URL. Put that URL in the REACT_APP_SOLANA_RPC_HOST value.
So instead of:
REACT_APP_SOLANA_RPC_HOST=https://explorer-api.mainnet-beta.solana.com
It will be:
REACT_APP_SOLANA_RPC_HOST="whatever url they give you from quick node"
This fixed the issue for me.

Looks like there is an "uncaught promise error"? This is what I am getting from my developer tools console.
connection.ts:2172 Uncaught (in promise) Error: failed to get balance of account 3QySa22MnWJTP1HbgF6hacQ8PXJeFwkF2ZNTZTMFPE1r: Error: 403 Forbidden: {"jsonrpc":"2.0","error":{"code": 403, "message":"Access forbidden, contact your app developer or<EMAIL_ADDRESS>"id": "f7f93027-ee56-4265-a504-4546316c3deb" }
at connection.ts:2172

My fix above should fix this as well. It all stems from the same thing. Hope this helps :)

https://solana-api.projectserum.com/ can be used as well.

Reference

this solved my issues

Can someone replace the endpoint in the code with a PR?

I put in the url from quicknode.com and tried the projectserum one, but I still get the same error


Added New Hacks

I added in a global cheat section. Added in add max tokes. spam open market boxes. and fixed some of the old decipricated cheats.

All of these cheats are working and won't get you banned.


Fix the transcribeFile function to remove 30 seconds limit

Adjust the recorder to record for 60 seconds and fix the native transcribeFile function to support transcribing files of any size.
I've tested with files that are 60 seconds long.

Ketcap ne


Add a refresh query parameter to the loans feed (PP-1368)

Description
Aspen doesn't need us to go out and refresh our loans feed every time they make a request. They are going to add this query parameter to the loans feed, so we add an additional check to support it.
JIRA: PP-1368
Motivation and Context
Reduce the load of generating the loans feed when we don't actually need to refresh it.
How Has This Been Tested?

Running unit tests

Checklist

[X] I have updated the documentation accordingly.
[X] All new and existing tests passed.


Removed require 'recursive_openstruct' from tests

Theres no need for these after #5955 was merged
fyi @himdel @simon3z @zakiva

@zeari Unless I am mistaken, the explicit gem specification in #5955 does NOT mean that you can stop requiring the gem as in this PR.

@zeari Unless I am mistaken, the explicit gem specification in #5955 does NOT mean that you can stop requiring the gem as in this PR.

Oddly enough, this worked on my station.
Closing.


[Bug]: Unnecessary variabl ShowLogo in Email Accounts Page

Describe the issue
As we have fixed it for:
https://github.com/microsoft/BCApps/pull/663
Variable Show Logo on Email Accounts page have no function:

It's always set to true.
We could do cleanup and remove it, behavior would stay same.
Expected behavior
/
Steps to reproduce
/
Additional context
No response
I will provide a fix for a bug

[x] I will provide a fix for a bug

Sure, let's do that.


Update version of scanner to 20.3.2

Updated version of scanner to 20.3.2

@timovandeput: This one will be followed by another one upgrading the scanner further to <IP_ADDRESS>, but we need these to have the docker images available so we can compare the scanner from 20.2.2 to 20.3.2 to <IP_ADDRESS> (if we want)


Deserialize test fails in chrome

I thought reflectable was supposed to handle the dart2js case however when I run pub run test test/deserializer/deserializer_test.dart -p chrome, it fails
00:28 +0 -1: deserialize: simple
  UnimplementedError
  dart:_internal                                                                          JsLibraryMirror.libraryDependencies
  package:reflectable/src/reflectable_mirror_based.dart 2269:14                           ReflectableImpl._directlySupportedClasses
  package:reflectable/src/reflectable_mirror_based.dart 2355:36                           ReflectableImpl._supportedClasses.<fn>
  dart:_internal                                                                          JsLinkedHashMap.2.H.computeSignature.V.func
  /tmp/dart_test_0eGINL/test_Dk1OSF/deserializer_test.dart.browser_test.dart.js.map 9:53  Object.J.putIfAbsent
  package:reflectable/src/reflectable_mirror_based.dart 2354:12                           ReflectableImpl._supportedClasses
  package:reflectable/src/reflectable_mirror_based.dart 65:10                             wrapClassMirrorIfSupported
  package:reflectable/src/reflectable_mirror_based.dart 2215:14                           ReflectableImpl.reflectType

Is dson supposed to work for browser application?

Yes, let me check what is the problem.

I was actually asking the same question for the serializer package (i am evaluating both) and it seems the tests have to be ran using pub serve (was not really aware of this)
$ pub serve test --port 8999
$ pub run test test/deserializer/deserializer_test.dart -p chrome --pub-serve=8999

and it works! Might be worth mentionning this somewhere though. thanks

sorry, I will add it to the documentation. If you can please add a pull request


adding personal Database files for each Instagram accounts

Please fix travis ci errors and I'll merge your pull request, good job!

I think, for better compatibility, by default need to set "whitelist.txt", "blacklist.txt", etc.

I think, for better backward compatibility, by default need to set "whitelist.txt", "blacklist.txt", etc.

ok i'll try for both


chore:addressing of simple deprecations surrounding Settings and log

⚠⚠⚠ Since we do not accept all types of pull requests and do not want to waste your time. Please be sure that you have read pull request rules:
https://github.com/louislam/uptime-kuma/blob/master/CONTRIBUTING.md#can-i-create-a-pull-request-for-uptime-kuma
Tick the checkbox if you understand [x]:

[x] I have read and understand the pull request rules.

Description
Looking at our files, I found that two functions can be easily migrated:

functions that use settings now use the Settings-class
functions that use log now use the newer logging mechanism (usages of console.XYZ are folded to another PR)

Type of change
Please delete any options that are not relevant.

Other

Checklist

[x] My code follows the style guidelines of this project
[x] I ran ESLint and other linters for modified files
[x] I have performed a self-review of my own code and tested it
[x] I have commented my code, particularly in hard-to-understand areas (including JSDoc for methods)
[x] My changes generates no new warnings

Screenshots (if any)
Please do not use any external image service. Instead, just paste in or drag and drop the image here, and it will be uploaded automatically.

Closing in favor of #5118


Investigate monitoring consolidation - we have dashboard, Pingdom, AppInsights, status page…

Investigate monitoring consolidation - we have dashboard, Pingdom, AppInsights, status page…
Perhaps consolidate in AppInsights?

Issue moved to NuGet/Engineering #199 via ZenHub


limited craftingmonitor updatespeed depending on tasksize fixes #2132

numbers may need some tweaking
currently set to:
-update on change for 0-1000
-update every tick for 1000-10000
-wait for an additional tick up to 20 for every 10000 lookups
The actually expensive part of the calculation is summing up the outputs of all the recipe outputs in the queue. It might be possible to multithread this part for better performance.
I've not looked into other potential fixes that modify the current crafting procedure.

Wait on this one if the autocrafting changes I want to make work this is no necessary.

I'm closing this in favor of your autocrafting PRs


Not an issue, just looking for an example

Would you happen to have an example on how I can use progressbar to track the progress of a subprocess.Popen?
Thanks!

It would depend entirely on what the subprocess does. Are you reading lines of text from the subprocess? If so, it could look something like this:
import subprocess
import progressbar

# Replace this with your command
args = ['ls', '-l']

bar = progressbar.ProgressBar(redirect_stdout=True)
process = subprocess.Popen(args, stdout=subprocess.PIPE)
for line in bar(process.stdout):
    print('Got line from command: %s' % line.strip())

hmm.. i dont think thats quite what I am after. All that code is doing is printing out 'Got line from command ls output' .. i need something print a progress bar on a script or tool as it runs, but I may need to leverage multiprocessing to achieve that...

Generally multiprocessing works quite well for purposes such as these. A simple example:
import progressbar
import multiprocessing

def some_function(i):
    return i + 1

items = list(range(100))  # list() to make sure this code sample also works on Python 3
bar = progressbar.ProgressBar(max_value=len(items), redirect_stdout=True)

for i in pool.imap_unordered(some_function, items):
    print i


[BUG] The automatically generated curl command is missing the \

Describe the bug
After quickly deploying the service via Docker, using the Translator template and testing with the automatically generated curl, I found that the \·
Screenshots

Setup

OS: Debian 11
Browser chrome

Additional context
I saw a similar problem mentioned in the history of the problem, but this good line has not been completely fixed

if you resync fork, does the issue still occurs?
should be resolved with https://github.com/FlowiseAI/Flowise/pull/145

Update is pushed to new release. Let me know if you still see the issue @VernHe

If the docker image has been updated, then theoretically this problem should no longer occur, and I've seen the source code, and it has been fixed. But I did start it as docker-compose directly in the docker directory of the source according to the README documentation.
I will try to reproduce it again. @HenryHengZJ


Hi,I have a python3.9 ctypes question

<EMAIL_ADDRESS>appreciate this project very much,and I have a little question to ask you.It about python3.9 ctypes,I can build libcxx on my centos8,but can't build libcxxabi and I haven't found a resolution.I know that question I should ask in python3.9 fourm,but I am really a novice,maybe you can help me.
Error trackback:
Traceback (most recent call last):
  File "/home/yunyuyuan/dev/news/main.py", line 4, in <module>
    Tg()
  File "/home/yunyuyuan/dev/news/src/modules/tg/tg.py", line 18, in __init__
    self.client = Client(
  File "/home/yunyuyuan/.local/lib/python3.9/site-packages/aiotdlib/client.py", line 283, in __init__
    self.__tdjson = TDJson(library_path=library_path, verbosity=tdlib_verbosity)
  File "/home/yunyuyuan/.local/lib/python3.9/site-packages/aiotdlib/tdjson.py", line 82, in __init__
    self.initialize(library_path, verbosity)
  File "/home/yunyuyuan/.local/lib/python3.9/site-packages/aiotdlib/tdjson.py", line 125, in initialize
    self.inject_library(library_path)
  File "/home/yunyuyuan/.local/lib/python3.9/site-packages/aiotdlib/tdjson.py", line 92, in inject_library
    self.__tdjson = CDLL(library_path)
  File "/usr/lib64/python3.9/ctypes/__init__.py", line 374, in __init__
    self._handle = _dlopen(self._name, mode)
OSError: libc++abi.so.1: cannot open shared object file: No such file or directory

Already resolved


Add CRD fields to know when was the last time the Volume got mounted/unmounted

What should the feature do:
Expose fields into the CRD LogicalVolume to know when was the last time that the PersistentVolume was mounted/unmounted
Everyone is already doing it like for instance OpenStack Cinder Volume, GKE Disk Volume...
So I was thinking to implement it the google way:
"lastAttachTimestamp": string,
"lastDetachTimestamp": string,

What is use case behind this feature:
Know when we last used a specific LogicalVolume so that we can clean up clusters volumes in accordance with the company retention policy and teams management of their data.
PS: If this feature request is accepted, i will code it and open a PR on this repository.

IMHO, it is better to use annotation for these timestamps since these are not necessary for TopoLVM to work. LogicalVolume CR is a resource that only topolvm-controller cares about, so annotation to PVC might be more appropriate.
PVCs can be mounted by multiple Pods at the same time, so even if the lastDetachTimestamp is later than the lastAttachTimestamp, some pod may mounted the PVC. It seems undesirable behavior to me.

Okay i see so is it better for you if we just annotate a field last_updated to the PVCs created by TopoLVM ?

TopoLVM does not create PVCs.
To track mount/unmount time of PVCs, one possible solution is to make topolvm leave Kubernetes event when the PVC is mounted or unmounted. I think this is better than annotation.
Another idea is to use metrics: some of kubelet's PVC metrics (e.g. kubelet_volume_stats_capacity_bytes) are exported only when the PVC is mounted by at least one Pod. This can be used to determine when it was last mounted.

Okay I understand your point of view and I just wanted to say that Kubernetes events are meant for audit purposes and not as some sort of history.
As for the metrics idea, it would mean that we would have to be dependent on two solutions: TopoLVM and the prometheus stack.
I really think that adding a simple annotation to LogicalVolumes CRs like OpenStack and Google does for their volumes/disks management should be the way to go.

I just wanted to say that Kubernetes events are meant for audit purposes and not as some sort of history.

I've never heard of that. Could you give me a link to your source of information?

I really think that adding a simple annotation to LogicalVolumes CRs like OpenStack and Google does for their volumes/disks management should be the way to go.

I'm not familiar with OpenStack or GCE, so I don't know how far I can refer to them. TopoLVM is software for Kuberenetes, so I would like to follow the practices at Kuberentes. I think for now using Kubernetes events or Prometheus metrics is more convenient approach.


Doesn't work on windows 11

I'm unable to run the app on Windows 11. The only thing that shows up is a grey rectangle. Reinstallation doesn't help.

Simillar problem on Windows11.
In my case, Pomotroid works but after 30 minutes or so, the app hangs and pressing "Tab" key stops working on the OS level globally. Killing hanging-up Pomotroid app fixes this. I don't know why at all.


plugin: OverlayPlugin-related cleanup

Replace custom ILogger with OverlayPlugin ILogger
Fix calls to deprecated OverlayPlugin methods (closes #3957)
Remove pointless OverlayAddonMain.cs
The vast majority of this PR is the ILogger change.
Tested changes in-game on 6.11/intl/English.
Is there some sort of styling guide for cs files for the project, by the way? The majority seem to be 2-space indented, but occasionally there's a 4-space indented file...

Is there some sort of styling guide for cs files for the project, by the way? The majority seem to be 2-space indented, but occasionally there's a 4-space indented file...

There's not.  I think I tend to 2-space indent everything, and I never got around to adding C# linting that would format stuff.  So, it's mostly "keep code consistent with what's around it" right now.


alienfan-cli setfans Issue

Help for the setfans option is as follows:
setfans=[,][,mode]  Set fans boost level (0..100 - in percent) with selected mode
Thus there are THREE options, with the first being mandatory and the second two optional. I run the following command, which I HOPE (see below on undertainties with documentation) will set fan 3 (my 4th fan) to a manual speed of 100% of it's maxrpm:
alienfan-cli setfans=100,3,0
I get the following response:
AlienFan-CLI v8.5.1
Supported hardware (4800) detected, 4 fans, 2 sensors, 7 power states.
setfans: Incorrect arguments count (AT LEAST 4 NEEDED)
note, the emphasis is mine
Per the documentation therer are only THREE listed options. What am I missing/doing wrong - or is this a bug?
ADDITIONALLY: I am uncertain about what the first two options are supposed to represent. I assume that one of the two is the BOOST level and the other is the optional ID of the fan you want boosted. But I have no idea which. I am also uncertain what will happen if NO mode is provided. I suggest the following help change:
setfans=[,][,mode]  Set fans boost level (0..100 - in percent), for all fans(??) - or the optionally provided fanN, for all modes(??) - or the optionally provided mode.
As an aside, I was trying to do this because AlienFan Control has yet again decided to completely stop responding to any temperature changes and is leaving all fan speeds at a low and unchanging value despite the cpu/gpu and entire computer temperatures continuing to rise unchecked. And this time it happened while under normal operations, not while coming out from a Sleep state (which seemed to be the only cause of this the multiple other times it has happened to me). I had 8.5.0 when I first noted this, so after a reboot (which failed to correct the broken state of fan control) I updated to the latest 8.5.1 build. No change after that installation. I note that I did not restart the computer after the install, and will try that now and report back.
I really hope you can come up with a cause for this and build a box around it. Or failing that at least come up with a way to detect this issue (which is trivial by simply tracking temps and fan speeds over time). Once detection is made figure out how to either return the system to a functional fan control with AFC active, or with AFC disabled.

You need at least 4 boosts (per fan) and (optionally) mode. So only 4 or 5 arguments acceptable.
About other - it's a strange bug we discuss before. Check curve window - does it have red dot?


Usage tracking

We ought to consider some form of usage tracking for this repo.  The basic version we get under Insights > Traffic only gives us the last 2 weeks and isn't that informative.
More detailed usage statistics would give us a clearer picture of how many unique users we have and how this is changing over time.  This would help justify/rationalise future investments in time etc. Adding a DOI #122 will help with this to a certain extent, but tracking git interactions is still important.

@hsteptoe - do you have a link to any more info on how we could do this? Thanks.

Turns out github don't make this very easy...
It looks like the most straightforward way might be to make sure that we always attached an 'asset' with our releases that way total downloads of the assets associated to a release are tracked, but https://github.community/t/counting-number-of-downloads-of-a-release/359 suggests we have to manually add an asset, not use the auto generated ones... I think
https://repo-analytics.github.io/ https://github.com/repo-analytics/repo-analytics.github.io  seems like it could be a solution, but possible requires giving away too many permissions.


Support CSS Color Module Level 4 colors

VSCode currently only shows the color picker for old style rga and rgba values, i.e.:

Chrome DevTools uses CSS Color Module Level 4 by default now, so currently we have to manually convert colors copied from there to see what they are in VSCode, as the picker doesn't show:

So I'm requesting support for this new syntax which become the default in the future. Thanks!

duplicate of https://github.com/microsoft/vscode/issues/43111

duplicate of https://github.com/microsoft/vscode/issues/43111


Add option to optimize control-flow stage with N-Grams Distribution Distance

Currently, the first optimization stage (structure, although we're gonna start calling it control-flow) is always optimized with the Control-Flow Log Similarity metric (CFLS), which is highly time-consuming.
Refactor it so it works in the same way as the calendar optimization (a set of metrics that can be chosen through the configuration). And add two options:

Control-Flow Log Distance: the same as current one, but transformed to a distance (instead of a similarity metric), so we don't need to invert it (1 - similarity). Details on how to call it here.
N-Grams Distribution Distance: way more efficient, compute the n-grams of the event logs and compare their frequency. Details on how to call it here.

WARNING! After updating the log_similarity_metrics dependency to the 0.8.0 version, the name of the other metrics' functions are different. Update call (see updated names here)
WARNING 2! All the metrics that we are computing now are distances, so we don't need to invert their value to pass it as the loss to the fmin function. Both optimizations should receive a distance when evaluating an iteration, and pass it (without changing its value) as loss.

[ ] Refactor control-flow optimization stage to work in the same way as calendar optimization stage (choosing the evaluation metric from the configuration file).
[ ] Supports both NGRAM_DISTANCE and CFLD for the control-flow optimization stage.
[ ] The variable used to store any current metric value is called distance, we are not inverting its value when passing it as loss as a result of the optimization iteration.

Done in small-refactors branch, (@iharsuvorau) close it when you review and merge that branch.


Fixes

takes care of several outstanding issues.
@patrickarlt this provides and documents a style for input validation errors and successes:

Is that clearer to you?

@paulcpederson Looks great :shipit:


[LOG4J2-3359] Adds Log4j 1.x specific syslog layout

Log4j 1.x always used the BSD syslog layout, but allowed for an internal layout to specify the MESSAGE format.
This PR adds a variant of the Log4j 2.x SyslogLayout, that emulates Log4j 1.x behavior.

@rgoers
Ping: Does this look good to you?

I added a third parameter to DatePatternConverter, so that I can use it with a fixed English locale. I am not documenting it in the manual, because for now it does not work with the FixedDateFormat.
By replacing FastDateFormat from the initial commit with DatePatternConverter I want to fix two problems: the generation of garbage, even if it is not needed and the possibility to cache the formatted date.


In is_running(), list CLI commands generically

Aside from being a solution which would not fail when we add another CLI command, it also allows commands from loaded plugins to be recognized. We need this especially for allowing CLI to do things outside of one account only.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 68.475%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
7017

Relevant Lines:
9714

💛  - Coveralls

Adding a test would be good, actually.


We need a better way to transfer large objects from S3 to Azure

In its current implementation, the S3toAzureTransfer class opens an InputStream from an S3StreamStore, then hands that stream straight into an AzureStreamStore. Depending on how the Azure writeBlob() API is sucking in that stream, it can pull down too much at once and overwhelm the replicator – and then the replicator crashes with an Out-of-Memory exception.
We could up the memory in the task, but that might only push the problem out – the max memory for a Fargate task is 30GB, and we have objects many times that.
We don't have this issue in the S3 replicators because we hand off the heavy lifting to the S3 CopyObject API (via the TransferManager class). The replicator never sees the bytes it's replicating.
We can sorta approximate that in Azure, using the Put Block from URL API: https://docs.microsoft.com/en-us/rest/api/storageservices/put-block-from-url
Unfortunately, that API isn't supported by the Azurite container we use for testing. "Following features or REST APIs are NOT supported or limited supported in this release (will support more features per customers feedback in future releases)" https://github.com/Azure/Azurite/blob/1bf8b593289e8e5edb8febeb5dbc8bcdc68ce6f9/README.md
Since we have no other way to replicate large objects, I think we have to accept this.

Trying with the latest commit and Put Block from URL, I get the following exception:

07:24:25.127 [scala-execution-context-global-46] WARN  u.a.w.p.a.b.s.a.AzurePrefixTransfer - Error copying s3://wellcomecollection-storage-staging/digitised/b30496160/v2/data/objects/b30496160_0003.mxf to azure://wellcomecollection-storage-staging-replica-netherlands/digitised/b30496160/v2: TransferDestinationFailure(s3://wellcomecollection-storage-staging/digitised/b30496160/v2/data/objects/b30496160_0003.mxf,azure://wellcomecollection-storage-staging-replica-netherlands/digitised/b30496160/v2/data/objects/b30496160_0003.mxf,com.azure.storage.blob.models.BlobStorageException: Status code 400, "InvalidBlobOrBlockThe specified blob or block content is invalid.


Lazyloading controllers - Not working

So i'm trying to add lazy loading to my project and it outputting errors
(i'm doing this because the final app.js in build is too large and takes a long time to load)
here is the changes I've made to the project
https://github.com/YasserDRIF/quill/commit/2a3bd10b6ad061bdcddceb13c2b33732346fb8d0
and i'm getting this errors when i go to the dashboard state for example

any idea on how fix this ?

@ocombe !!


doc: add badge of the docker hub image

Signed-off-by: rick<EMAIL_ADDRESS>

hi @Reintjuu below image should work well. I have tested it via the postgres helm chart. You can just replace the image.
docker.io/linuxsuren/pgvector:master@sha256:ac32e076a4bb8b4f47a4f36f34109f32a58e4de333568756123773e702f52861

I created a release. So, nothing will change in that version. https://github.com/LinuxSuRen/pgvector-docker/releases/tag/v0.0.1


Shared data management using tanstack query

https://github.com/kiranojhanp/fullstack-typescript-turborepo-starter/pull/14


Deleting orphan entries from my index

I am running 7.12.0 of ElasticSearch and 2.9.0 of Norconex. I have some web pages in my index that are no longer there; they result in a 404 error. The problem is that my 404 entry pages in the index are not being deleted. I decided to run a full crawl by deleting my crawlstore to fix this problem. As I understand it, the crawlstore determines whether a page should be marked for deletion. But because the crawlstore was created by a full crawl, it does not know about these 404 entries, and as a result never marks them for deletion. What am I missing, and how do I remedy this? Any help would be greatly appreciated.

You are correct that if you wipe out the crawl store, it loses track of whether any URL previously existed or not and can't send a deletion request.  It is the same as doing a brand new crawl.
With version 2.x you'll have to use workarounds (avoiding custom coding).  Obviously cleaning your index would do it but it is rather drastic.  Another approach when doing full crawls (i.e., wiping crawl store), is to also index a date field holding the current date (see CurrentDateTagger).  You can also define a made up constant field to do the same, that you update when doing full crawls (see ConstantTagger.  Then you can rely on this new date/constant field to delete from your index anything older or not having your new constant.
Version 3 introduced the ability to send deletion requests for any triggered crawler events (REJECTED_NOTFOUND in your case) regardless of the crawl store.  When you'll be ready to upgrade, have a look at DeleteRejectedEventListener.

Thank you Pierre for you quick response and insightful wisdom. It is very much appreciated!

Did you mean "Pascal"? 😉

Sorry Pascal.


Allow rocksdb.DB instances to be manually closed

While delete rocks_ptr is a deterministic operation in C++, the Python analogue del rocks_handle is not – disposal and finalization of the Rocks database are entirely dependent on the Python garbage collector (q.v. the python-rocksdb tests themselves, which call gc.collect() after del rocks_handle to attempt to force the destructor to run).
This change exposes a method to trigger the destruction of the underlying Rocks database pointer (deterministic!) through the Python Rocks handle; existing code will not need to be changed, as the Python object destructor (non-deterministic!) will now call this method.

@fish2000
could you please solve the conflict:)

@twmht Hi, I posted a new PR #40, sans any merge conflicts (that is to say, it’s based on HEAD after your ColumnFamilyOptions changes).


Added reference to the navigator global object

Added a reference to the navigator global object.

Hello @ducksoupdev! I have updated my repository for this PR with missing TS type.


:bug: Remove message from technology-usage rules so that they don't appear as issues, only as tags

Fixes https://github.com/konveyor/rulesets/issues/55

@jmle that's a terrible news. Information messages are very very useful. I can explain you why offline if you want


[feature]: Add Redis support for Java SDK

Is there an existing feature request for this?

[X] I have searched the existing issues

Summary
Java SDK does not have Redis support, so Redis Database cannot be mocked . This support needs to be added as many projects use Redis as a Database.
Why should this be worked on?
Adding support to various Databases will improve Keploy's mocking feature .

Closes with https://github.com/keploy/java-sdk/pull/119


Incorrect Website Link in Repository Description

📚 Documentation Improvement
I've noticed that the website link in the repository description for SuperGradients (www.supergradients.com) incorrectly points to https://www.nvidia.com/en-us/ instead of the actual SuperGradients or Deci AI website.
This may confuse users looking for official SuperGradients documentation or information. Could you please update the website link to the correct destination?
Thank you for maintaining this excellent library!

I am facing the same issue as well. Unable to visit the docs of super-gradients.

Nvidia bought Deci Ai so I think for now it will point to the Nvidia website I guess but that is pretty annoying tbh.

Was there someone who was able to preserve deci ai docs?

Is there any info what will happen with this library under Nvidia? From what I've seen they stopped at least for now maintaining this library since purchase?

@Jozefov fyi documentation is saved inside of the repository, althought I have to admit it is not as pretty as website


Support for tabstops in code completions?

Hi.
Firstly, is there a forum for asking the type of question I'll be asking here? If so, and if anyone can direct me to the appropriate forum, I promise not to raise an issue to ask a question in the future.
That said, I wish to ask if it is possible to create completion items where the completion text contains zero or more tabstops? Ace Editor is excellent at this kind of thing. I figure it must be in the spec somewhere however I haven't been able to locate it. I suppose snippets are analogous to completion items with tabstops, but can I supply snippets via lsp?

Snippet support is on its way with the 3.0 version of the library. See https://github.com/Microsoft/language-server-protocol/blob/master/protocol.md#typedstring


serialports monitor ignores parameters (pySerial < 3)

Hi,
The options for the serial monitor are ignored when using pySerial < 3. (I use 2.7). The following command should open a serial monitor with baudrate 115200:
platformio serialports monitor -b 115200

But it opens as 9600, which is the default value.

It seems to be something with the parsing of the options in serial.tools.miniterm. But I can't figure out what exactly.

Hi @MrSnowflake,
This issue is linked with the latest Click 6.0. I will fix it as soon as possible.
Temporary solution:
pip uninstall click
pip install "click<6"

Also, don't forget about :star: ( https://github.com/platformio/platformio/stargazers ) for @valeros :blush:

I've just opened related issue https://github.com/mitsuhiko/click/issues/473
If this isn't Click's bug, I'll drop Click version to "click<6".

Fixed in 2.4.0


Add DUPLICATE_INTERNAL_STATE flag

Resolves #3393

Well, internal properties are still not duplicated, soo...

So I updated the original issue by converting reproduction project from Godot (1.x?) to Godot 4. I can confirm the issue is still the same.
Also to clarify: the issue is about usage of Object.duplicate() method and the fact that variables of returned object (even simple types) are different from the object from which it has been duplicated


some bugs when testing different aggregators on different types

You have to provide the following information whenever possible.

Describe what's wrong
When I added a test to test different aggregators on different types (GlutenClickHouseSyntheticDataSuite on branch: binmahone/test_null_values , this test was originally supposed to cover CH-87). I found:

[ ] some normal types are not supported to aggregate, e.g. "float"/"byte"/short"/"string"
[ ] avg(int) and avg(bigint) can result wrong result (return NaN when input has two rows: Int.Max and Int.Min)
[ ] something went wrong when input has null values

The first one is not very urgent, but the second and third seems to be fatal.
Does it reproduce on recent release?
The list of releases
Enable crash reporting

If possible, change "enabled" to true in "send_crash_reports" section in config.xml:

<send_crash_reports>
        <!-- Changing <enabled> to true allows sending crash reports to -->
        <!-- the ClickHouse core developers team via Sentry https://sentry.io -->
        <enabled>false</enabled>

How to reproduce

Which ClickHouse server version to use
Which interface to use, if matters
Non-default settings, if any
CREATE TABLE statements for all tables involved
Sample data for all these tables, use clickhouse-obfuscator if necessary
Queries to run that lead to unexpected result

Expected behavior

A clear and concise description of what you expected to happen.

Error message and/or stacktrace

If applicable, add screenshots to help explain your problem.

Additional context

Add any other context about the problem here.

avg(bigint) avg(int)结果不正确是因为在聚合的merge阶段，subtrait plan没有正确的表示出CH的中间结果类型，以bigint为例，预期为AggregateFunction(avg, Nullable(Int64)),而实际是AggregateFunction(avg, Nullable(Float64))

当boolean列存在null时，进行聚合计算报错
done https://github.com/Kyligence/ClickHouse/pull/99

Use #123 to trace task one, close this issue.


Create Project for User

Hi,
this time without .gitignore.
Thanks, regards.

Sorry to be a pain but it's still in the PR so will show at latest commit. Can you rebase with only the relevant commit please!

hey,  im not find how fork project in API, its possible?

Hi @vadikk006! No this has not been merged so the functionality is not currently in the library. I will add the changes from this PR into the repository over the weekend so it will be available Monday.

See #94


Feature text with background color

I don't found how to create background box for ol.Feature text style.
Is it possible? It would be great if ol3 has such feature.

You can set a  stroke on your ol.style.Text to create a halo. A rectangular box is not possible.

The easiest to add a rectangular background is using an image. Creating a data-uri for an svg image is quite easy. Just do data:image/svg+xml;utf8," + encodeURIComponent("<svg>...</svg>") with a rectangle in the svg definition.

I found this issue in 2017 and would like to elaborate. Why it is not possible? Seems like quite basic feature..

With the new text renderer, an image is created for the text anyway. That image could be given a stroke and fill. For ol.style.Text, there could be new backgroundFill, backgroundStroke and ´padding´ properties. Anyone interested in creating a pull request?

sounds like a nice improvement, @ahocevar.

I have a problem, I have set the backgroundFill but for some reason it doesn't work. I'm using openlayers 4.4.2
Here is the code:
return new ol.style.Style({
        image: new ol.style.Icon({
            anchor: [0.5, 0.5],
            opacity: 1,
            scale: 1,
            src: 'public-images/map/stena_port_unselected.png'
        }),
        text: new ol.style.Text({
            offsetY: 20,
            text: portName,
            placement: 'point',
            textAlign: 'center',
            fill: new ol.style.Fill({color: 'rgb(255, 169, 163)'}),
            backgroundFill: new ol.style.Fill({color: 'black'}),
            padding: [3, 5, 3, 5]
        })
    });

Can you please help me? Am I doing something wrong?

this code is not in 4.4.2, please try the latest release 4.6.2

is there anyway yet implemented to have a radius setting to curve the background box?

No, there is not.


Weird bugs when using VCR w/ fog for S3 stuff

I'm using VCR to test some s3 code that uses fog.  The first time, I get a 404 from S3.  When I run the test again (using the VCR cassette) it has a weird error:
XML declaration allowed only at the start of the document
     # /Users/myron/.rvm/gems/ruby-1.9.2-p290@silo/gems/nokogiri-1.5.0/lib/nokogiri/xml/sax/push_parser.rb:47:in `native_write'
     # /Users/myron/.rvm/gems/ruby-1.9.2-p290@silo/gems/nokogiri-1.5.0/lib/nokogiri/xml/sax/push_parser.rb:47:in `write'
     # /Users/myron/.rvm/gems/ruby-1.9.2-p290@silo/gems/fog-1.1.2/lib/fog/core/connection.rb:21:in `block in request'
     # /Users/myron/.rvm/gems/ruby-1.9.2-p290@silo/gems/excon-0.9.4/lib/excon/connection.rb:275:in `block in invoke_stub'
     # /Users/myron/.rvm/gems/ruby-1.9.2-p290@silo/gems/excon-0.9.4/lib/excon/connection.rb:240:in `each'
     # /Users/myron/.rvm/gems/ruby-1.9.2-p290@silo/gems/excon-0.9.4/lib/excon/connection.rb:240:in `invoke_stub'
     # /Users/myron/.rvm/gems/ruby-1.9.2-p290@silo/gems/excon-0.9.4/lib/excon/connection.rb:143:in `request_kernel'
     # /Users/myron/.rvm/gems/ruby-1.9.2-p290@silo/gems/excon-0.9.4/lib/excon/connection.rb:89:in `request'
     # /Users/myron/.rvm/gems/ruby-1.9.2-p290@silo/gems/fog-1.1.2/lib/fog/core/connection.rb:26:in `request'
     # /Users/myron/.rvm/gems/ruby-1.9.2-p290@silo/gems/fog-1.1.2/lib/fog/aws/storage.rb:381:in `request'
     # /Users/myron/.rvm/gems/ruby-1.9.2-p290@silo/gems/fog-1.1.2/lib/fog/aws/requests/storage/get_bucket.rb:55:in `get_bucket'
     # /Users/myron/.rvm/gems/ruby-1.9.2-p290@silo/gems/fog-1.1.2/lib/fog/aws/models/storage/directories.rb:24:in `get'

It looks like there is a bit of a bug in excon stubbing.
When a real request is made and fails the :expects status code check, it does not invoke the streaming block:
https://github.com/geemus/excon/blob/v0.9.4/lib/excon/response.rb#L36-39
But there is no such check in for a stubbed response:
https://github.com/geemus/excon/blob/v0.9.4/lib/excon/connection.rb#L270-279

Done:
https://github.com/myronmarston/vcr/commit/5d2e3c8211a23683cbbf8b13f3ad371a57c77026
https://github.com/myronmarston/vcr/commit/99f601c1308d8ae16c53d6dd0e458181cea68b6a
https://github.com/geemus/excon/commit/c9a084cc69cccbab5bbd656786b30e09219f66a4


match container-name field in security policy

Bug Report
General Information
In containerized un-orchestrated mode, currently we are checking only the Spec.Selector.MatchLabels field for the container name. But we should instead check for some specific field under MatchLabels (kubearmor.io/container.name) to match the container name.
Current Behavior
Suppose a sample policy contains:
  selector:
    matchLabels:
      abcd: nginx

then also KubeArmor will match nginx as the container name.
Expected Behavior
Only match container name with kubearmor.io/container.name field value
  selector:
    matchLabels:
      kubearmor.io/container.name: nginx

Ref: https://github.com/kubearmor/KubeArmor/blob/main/KubeArmor/core/kubeUpdate.go#L1286

Hi
I want to work on this issue. Could you please assign it to me, it would be beneficial for my academics too.
Thanks

Hello @HariVamsiK glad to see your interest.
Sorry but this issue is already addressed in #886.
There are many other open issues, you might want to take a look at them. We would be more than happy to help.

@Ankurk99 since this has been handled in #886, let's close it?


Documentation and Setup-Notes Revisited

Please check if setup, development, and deployment is up to date, uniform, and completely documented. We'll check the boxes in the Meetings.

[ ] wiki home page
[ ] tira-host local dev
[ ] tira-host test deploy
[ ] tira-host live deploy
[x] tira-application local dev
[x] tira-application test deploy
[x] tira-application live deploy
[ ] disraptor local dev
[x] disraptor test deploy
[x] disraptor live deploy
[ ] tira-protocol local dev
[ ] tira-protocol test deploy
[ ] tira-protocol live deploy
[ ] create VM images is stale

Documentation Example:
How to develop tira-application:

[Check out git:tira-io/tira]
Update the config in tira-application/config/
[compile protobuf]

development setup

make setup
create a tira user.
give tira user permission rw permissions on tira-model
make run-dev or run-wsgi

test deployment

docker-build-tira-application
docker-run-tira-application

live deployment

docker-publish-tira-appplication
k8s-deploy-tira-application

partial local dev deployment

E.g. local tira-application+discourse and production tira-hosts

I have added a new page in the wiki with hopefully easy instructions on how to set up tira locally for development:
https://github.com/tira-io/tira/wiki/Getting-Started-as-a-TIRA-Developer
To make this as easy as possible, I have left out everything that I did not do (e.g. nearly all optional things). This is the reason for it to be a separate page for now, while the old "Host" and "application" wiki pages still remain. We have to see if we want to integrate these optional points or leave them out for the sake of simplicity.
The instructions are still to be tested on a clean ubuntu!


Tags and Genre are the same

Describe the bug
Just noticed this on a freshly downloaded manga,

and checked the comicinfo to see this
  <Genre>Martial Arts,Revenge,Male Protagonist,Swordplay,Tragedy,Full Color,Time Skip,Anti-Hero,Shounen,Wuxia</Genre>
  <Tags>Martial Arts,Revenge,Male Protagonist,Swordplay,Tragedy,Full Color,Time Skip,Anti-Hero,Shounen,Wuxia</Tags>
  <Notes>Downloaded with Mangal. https://github.com/metafates/mangal</Notes>

I did download it through Kaizoku but I don't think Kaizoku interferes with that
Reproduction steps
No response
Expected behavior
No response
Additional context
No response
Operating System
Linux

Hey,
You're right, yes, it's a bug. Thank you!


Remove redundant public modifiers

Description
Removes 4 warnings relating to redundant public modifiers.
Motivation and Context

0 tolerance for warnings can help prevent Broken Windows Syndrome
How Has This Been Tested?
I entirely relied on the compiler for this one.
Checklist:

[x] I have submitted a CLA form
[x] If applicable, I have updated the documentation accordingly.
[x] If applicable, I have added tests to cover my changes.

I'm not sure why the CLA assistant has stopped firing for this repo, but I can see that you have previously signed the CLA, so we're good there.

Tagged in 3.0.11


Move COMMIT into a variable so it could be overridden if desired

This is so that when the source directory is not a git directory (as is the case with building from a tarball), it becomes possible to manually set the commit hash during build like so:
make VERSION=0.69.420 COMMIT=1337deadbeef cli-linux-server

Currently, as there is no way to override the commit manually, it's not possible to build a working binary from a tarball using make.
If you try anyway (trying is the first step to failure), since git rev-parse --short HEAD will fail silently, the binary will be built with commit = "", which results in a runtime panic:
panic: runtime error: slice bounds out of range [:7] with length 0

Thanks


Support llvm distribution for windows

subj

The main problem with this issue is that I do not have access to a Windows machine to test. And it looks like it will be a fair bit of work to test everything in a non-Unix setting.
I am afraid I will have to rely on the community for this feature implementation.

I started taking a look at this (filed #13 in the process), but there's a problematic issue: LLVM for Windows is released only as a installable package, not a simple archive - or at least I didn't find any on the LLVM website.

Yes, I am not sure how to deal with this either. Can we install the package to a given directory path? Even if is a system default path, I don't see particular harm in installing a specific version of LLVM if the install does not overwrite installs of other versions of LLVM.

Apparently the llvm windows distribution is just a self extracting archive.
If the download_and_extract method would support nsis formats, it could be possible.

We don't have to rely on download_and_extract. We can always execute the self-extracting archive in the repository rule.

I have developed a different prototype patch series that works for Windows (I didn't notice the one developed by @zaucy until just now).  It gets around needing to run the self-extracting executable, by relying on a re-packaged compressed archive instead of shelling out to a separate 7zip install.  For my demonstration purposes, I just attached it as a release artifact to a github release.
https://github.com/mjbots/bazel-toolchain/commits/windows
The patch sequence isn't suited for submission yet, but if the general approach seemed reasonable, I would be happy to repackage it.
I also have a semi-independent ability to build libc++ from source and use that with binaries instead of the msvc standard library.  That let me run coroutines on Windows with clang 10.0.  It is less optimal, since I haven't figured out bazel toolchain transitions well enough to make it seamless yet.  If you use that feature, every cc_binary has to manually depend upon the libc++ library.

If people are still interested in this, I think we can get in a hacky thing in and then make it more proper over time. I don't use a Windows machine myself, so if anyone wants to put in effort, I can give them write access and let them manage the Windows feature.

llvm v18.1.1 is released https://github.com/llvm/llvm-project/releases/tag/llvmorg-18.1.1 and clang+llvm-18.1.1-x86_64-pc-windows-msvc.tar.xz is officially supported. Is it possible to take a look at Windows support for now?


NPM package contains 12.8MB codeclimate-reporter binary

I assume this isn't meant to be in the final package tgz! :)

Ouch!
I'm really sorry about that :scream: I'm releasing a patch immediatly.

:tada: This issue has been resolved in version 3.1.1 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

The issue exists since version 2.2.1<EMAIL_ADDRESS>I'm deprecating old versions  (2.2.1 - 3.1.0) with a warning:

npm package tarball contains useless codeclimate-reporter binary, please update to version 3.1.1. See https://github.com/adriengibrat/ts-custom-error/issues/32

Thanks a lot!


Add possibility to add arbitrary metrics

A user should be able to add any metric as query. The panel should then visualize the metrics, which are provided by the user.

Idea: The queries are named based on a predefined naming convention.

Based on todays meeting, we agreed on focusing on the following points:

Many metrics (e.g. cpu metrics) are collected with three different queries with different groupings.
We could consolidate these metrics by only using one, which includes the group by clause (container, pod, namespace)
The panel visualization fails, when the basic queries:

kube_pod_container_info
count_kube_deployment_labels
some of the count values are important as well.

We should try to establish a naming convention, from which we can derive the name, the grouping as well as the type of the metric:


Xcode version problem

I have created an new project with Xcode(version 10.2) and add CocoaHTTPServer to it,it not work on device.Then i used an old project (created 4 years before) ,it work.

The problem was solved by rebooting the Xcode

thank you for the issue.maybe you will help a lot of people who saw these .
I hope that you can solve my problem. cause I met the same.


All users can login admin dashboard and can change all configs and can delete all other users.

All users can login admin dashboard and can change all configs and can delete all other users except the admin.Is this a question?
Even I add role which just contain Read operate. Which applied to normal users.  BUT it is not worked.

@pozen see docs: https://casdoor.org/docs/basic/core-concepts#how-does-casdoor-manage-itself
and given that you are causing uncomfortable and bias among the community with your title and text, this issue will be deleted shortly.


Fix clunky user interface for editing and saving page names

The UI is currently weird for editing page names. After clicking edit, the context menu button switches downwards, making it confusing for users.
In addition - Perhaps the 'stop editing' (which is really a 'save') should be executed automatically when a user clicks outside of the editor box, and goes back to dashboard editing.

Fixed


Fix unmount warning in React when unmounting component

Issues:

When developers trying to unmount this component without stopping animation. React throws error:
Warning: Can't call setState (or forceUpdate) on an unmounted component. This is a no-op, but it indicates a memory leak in your application. To fix, cancel all subscriptions and asynchronous tasks in the componentWillUnmount method.
Fixed:
Before removing this component. I stopped animation so the component can unmount safely

I am having the same warning! it is nice if merged

Hi @joelmora ,
Thanks for reaching out. I'm gonna test it again to make sure it works.
Will let you know asap.


M00-LAB00: QUICK_DESCRIPTION

Contact Details
<EMAIL_ADDRESS>What happened?
Module 8 - lab 1 : 00
Task: 2
Step: 5
"lookback" is not defined
Should like in Task 1, step 16, line 1, have a definition like "let lookback = 2d"
Lab
Lab 08 Exercise 01 Perform Threat Hunting in Microsoft Sentinel
Relevant screenshots
paste here 😉
Do you want to help us? 👏

[ ] Create a Pull Requestt by following this project's Contribution Guide

Good catch. Surprised no one caught that before. That will be fixed today.

Update will be live later today.


[FEATURE] Testimonial section

Describe the solution you'd like
A testimonial section on the homepage would be beneficial for users to see who recommends nest.land. Note: The users we feature here would not be random users; it would feature people prominent for something.
Additional context
@maximousblk suggested something similar to this:

I'll plan to work on this when I get the opportunity, unless somebody else would like to do this.

If you don't have time, I'll work on it just let me know

It may be this weekend when I get to it because I'm working this week. If you think you can get it done before then, feel free.

Implemented in #169.


ci: test Node.js 6, 8, 10 and 11

Test only Node.js 6, 8, 10 (LTS) and stable (11).
https://github.com/nodejs/Release/blob/master/README.md

For dropping v4/v5, see #873
v10 has a bug right now (#856) (help would be appreciated)
We probably won't drop v7/v9
We can't use npm install, we have our own script (see #896)
We chose c++ as a language a long time ago, and I can't remember why (so I can't tell you if the reason is still relevant). I think it was just to get finer-grained control over our CI steps

We probably won't drop v7/v9

But why? They reached their EOL.

We can't use npm install, we have our own script (see #896)

This produces different results in CI and production (lockfiles are not working there).

We chose c++ as a language a long time ago, and I can't remember why (so I can't tell you if the reason is still relevant). I think it was just to get finer-grained control over our CI steps

But it does not seem to be used.
I'll check and update the CI script.

Not sure I quite understand. Are you objecting to the use of lockfiles in general, or objecting to our install wrapper?

The first, this is why npm ci is a bit dangerous (works in our CI - but not in production).

Probably, or use npm install on CI.
At least CI often produces a different dependency tree than the installed package in production as it is often a child dependency of another dependency and below the root lockfile it is only locked to the SemVer range in the package.json files (or better: pinned there to a specific version).
Not yet sure what the best solution for everyone is. It's a bit inconsistent across platforms (dev, prod, CI), tools (yarn, npm version) and repository settings (a lockfile might be useful for contributors and CI to build reproducible builds but many big devs like Sindre and others do not use lockfiles for such reasons afaik).
Also see https://github.com/sindresorhus/ama/issues/479