Fix key_file config

Actually it might be a bug of laravel, a GOOGLE_CLOUD_PRIVATE_KEY is having many line breaks in single string, the private key works when hard-coded in config file, but when reading private key from .env file, laravel replaced the \n to \n, and ended at 'openssl_sign(): supplied key param cannot be coerced into a private key' when uploading files.

+1 ! We hit this exact same bug(?) and your PR fixes it for us. Thanks!!!

first google app engine do not support .env file to you are expected to move all your environment variable to a file called app.yaml eg:
runtime: php72
env_variables:
Put production environment variables here.
APP_KEY: YOUR_APP_KEY
APP_STORAGE: /tmp
VIEW_COMPILED_PATH: /tmp
SESSION_DRIVER: cookie

first google app engine do not support .env file to you are expected to move all your environment variable to a file called app.yaml eg:
runtime: php72
env_variables:
Put production environment variables here.
APP_KEY: YOUR_APP_KEY
APP_STORAGE: /tmp
VIEW_COMPILED_PATH: /tmp
SESSION_DRIVER: cookie

I am not using google app engine. Your comment does not apply to this PR or to the issue in question.

Cool! Works with updating this PR. Thanks.

It worked. Thank you a lot.


I can't use my firebase-key-file

Hi everyone.
I'm on a linux Linux Mint 21.1 x86_64, with Docker version 26.0.0, build 2ae903e. I've installed a self-hosted version on ntfy with auth enabled.
I have a problem with my firebase-key-file. I've generated it with FCM, I download the file and I've mount the /etc/ntfy folder with docker.
The folder is correctly mount.
I tried to use the ENV var NTFY_FIREBASE_KEY_FILE or a server.yml for use my FCM json but the logs output is always this:

project ID is required to access Firebase Cloud Messaging client

This is the content of my file, named vvfy-XXXX.json, where XXX is the rest of project id inside the file:
{
  "project_info": {
    "project_number": "AAAAAAA",
    "project_id": "vvfy-XXXX",
    "storage_bucket": "vvfy-XXXX.appspot.com"
  },
  "client": [
    {
      "client_info": {
        "mobilesdk_app_id": "APP_ID_VALUE",
        "android_client_info": {
          "package_name": "com.vvfy"
        }
      },
      "oauth_client": [],
      "api_key": [
        {
          "current_key": ""
        }
      ],
      "services": {
        "appinvite_service": {
          "other_platform_oauth_client": []
        }
      }
    }
  ],
  "configuration_version": "1"
}

Can anyone help my?

Resolved!
The file was wrong.
Under Firebase site, I created a new private key into Project settings > Service accounts
The downloaded file need to be linked into container under NTFY_FIREBASE_KEY_FILE env, or in server.yml
After this i downloaded again the google-service.json. This file is necessary for build the apk.
I think this can be useful from someone else, because I didnt undestend the ntfy guide for FCM.

Thank you for sharing how you got it working!


Base shabad view

Design a shabad viewer and mock it with dummy data to test test scrolling. This is related to #49 -> 1

Currently what @saihaj is working on, after discussion of the new designs


Improvements from team 'Peacekeepers'

add names to tanks
add install instructions
fix whitespace mafia
change color of tank names to pink
use wsad for controls
remove wrong rotation based on mouse input

+1

Awesome! Whitespace aside these are some cool changes haha


Analysis of a source file without changing it?

Issue
Say I wanted to analyze a source file and not have the Aider make a change to it, but just give me the insights from the analysis.
"tell me about OWASPs you might see"
"let me know if this recipe has metric and imperial weights and measures"
"does this book chapter flow - give me some feedback"

Does it, (a) have a mechanism equivalent to --write-insights-to path/to/someTextFile.txt
Or should I (b) get it to leave insightful comments in the source - perhaps as comments with # or // in the leftmost position ensuring source still functions the same. Comments I can find later with find-in-files or silversearcher-ag
Or (c) ask it to do a HTTP-POST to some other endpoint to collect the feedback.
Version and model info
No response

Thanks for trying aider and filing this issue.
You could just say something like "Don't modify my source files, but write me a doc owasp.md that tells me about OWASPs you might see"?

You could also run aider with --dry-run to prevent it from actually editing any files.

With a tiny bit of creativity, that'll do, yes, sir:
#!/bin/bash
base_dir="."
ag --markdown -l "cheesecake" | while read -r file; do
        aider --no-auto-commits --dry-run --message "If a line has imperial and metric measures in it, change the entire line to 'AIDER-INSIGHT: \$FILENAME:\$LINENUM: mixed measures encountered' where FILENAME is the name of this file and LINENUM is the current line number" "$file"
done

Then pipe that through grep matching on AIDER-INSIGHT
I now have an AI boosted but waaaay slower version of silversearcher-ag

I'm going to close this issue for now, but feel free to add a comment here and I will re-open or file a new issue any time.

You may want to look at the scripting docs as well.
https://aider.chat/docs/scripting.html

Good solution. Thx


Swift 3.0 branch cmake overlay dependencies

This patch explicitly lists all overlay dependencies in the cmake files and adds a script to update them thus making the build more correct.
Resolves: rdar://problem/28501090
Risk: Low, only touches cmake files and a utils/ script that does not run automatically

@swift-ci Please test

@zisko Please review.

Looks good to me, I'm going to run a clean build of this on a fast machine to verify.

@swift-ci Please test

@swift-ci Please test

@swift-ci Please test

@swift-ci Please test

Build failed
Jenkins build - Swift Test OS X Platform
Git Commit - f5da06d6831860ad9c95e1fab7591043cf7effd5
Test requested by - @erg

@swift-ci Please test OS X platform


Reduce image size

misc/snapshotter/Dockerfile uses a ubuntu base image, and the size is larger for downloading, it could be optimized by using alpine.

fixed by https://github.com/containerd/nydus-snapshotter/pull/283


Feat/add cruntime for machine sdk 17 use tar stream

Attempt to use tar-stream instead of nanotar.

When you build the cruntime package , you,d need to relabel :devel as :0.1.0
docker image tag cartesi/cruntime:devel cartesi/cruntime:0.1.0
Em seg., 6 de mai. de 2024, 20:10, Danilo Tuler @.***>
escreveu:

@.**** commented on this pull request.
how do I test this?
it complains about cartesi/cruntime:0.1.0. How do I build it?
In apps/cli/src/commands/build.ts
https://github.com/cartesi/cli/pull/17#discussion_r1591441664:

@@ -419,29 +362,60 @@ Update your application Dockerfile using one of the templates at https://github.
appGnutarPath,
);

       // prepare OCI Bundle

       const extract = tar.extract();

       const pack = tar.pack();

       const appGnuTarStream = fs.createReadStream(appGnutarPath);

       const appOCIBundleStream = fs.createWriteStream(appOCIBundlePath);

       const ociConfigJSON = JSON.stringify(createConfig(imageInfo));

       // add config.json

       pack.entry({ name: "config.json" }, ociConfigJSON, function (err) {

           if (err) throw err;

           console.log("ERROR config.json");

           pack.finalize();

this looks out of place. because you will add more entries (for each
gnuTar entry)
—
Reply to this email directly, view it on GitHub
https://github.com/cartesi/cli/pull/17#pullrequestreview-2041486979, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAAC4UGJHPY6WQPDTXDFDLZA7IRHAVCNFSM6AAAAABHJO4D3OVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDANBRGQ4DMOJXHE
.
You are receiving this because you authored the thread.Message ID:
@.***>

how do I test this? it complains about cartesi/cruntime:0.1.0. How do I build it?

I recommend testing the draft branch, to see the cruntime working, and this PR is an improvemente so we don't create a --flash-driver just for the config.json.


(review): Add review for A never ending love story

Closes  #34 🥞 ❤

Hi @lucasmdrs 👋
I would love to have your review on this review 😄
Thank you! #gopancakes


Spatial data returned as binary

Hey, first of all thanks for the data api dialect module.
I wanted to use it to connect to an AWS Aurora instance and it looks good except for spatial data (geometry point). It gets returned as type buffer object. I'm running a mysql server locally using the default mysql2 module to connect which returns a correct geometry point object with coordinates instead of a binary buffer.
I defined my field like this:
location: {
    type: Sequelize.GEOMETRY('POINT'),
  },

Any idea what the reason is and how I can fix it? Looks like it's not a Sequelize issue in general because with mysql2 it's working. Really appreciate any help, thanks!

All right, I looked into the Sequelize library code and as far as I can see reason for not converting buffer into GeoJSON is missing type casting which usually is done by mysql2 dialect. But because we swapped it by mysql-data-api dialect this one is handling the type casting which doesn't work. I'm in version 5 and in lib/dialects/mysql/connection-manager.js of Sequelize library we assign a typeCast property (function) which later gets handled by mysql2 but not by mysql-data-api. So the only way to solve this is using a custom getter and setter. Luckily we can simply copy the one mysql2 would get passed here as typeCast property which is this:
static parse(value) {
      value = value.buffer();
      // Empty buffer, MySQL doesn't support POINT EMPTY
      // check, https://dev.mysql.com/worklog/task/?id=2381
      if (!value || value.length === 0) {
        return null;
      }
      // For some reason, discard the first 4 bytes
      value = value.slice(4);
      return wkx.Geometry.parse(value).toGeoJSON();
    }


Original pointer network [DON'T MERGE]

This is just to backup the original one. Code is mostly taken from https://github.com/ArneBinder/pie-document-level/pull/26

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

Comparison is base (8522938) 96.36% compared to head (89874d0) 86.48%.

Files
Patch %
Lines

...models/components/pointer_network/modeling_bart.py
62.74%
231 Missing :warning:

src/pie_modules/models/pointer_network_model.py
80.95%
68 Missing :warning:

...odules/models/components/pointer_network/losses.py
53.43%
61 Missing :warning:

...les/models/components/pointer_network/interface.py
52.63%
36 Missing :warning:

...modules/models/components/pointer_network/utils.py
60.43%
36 Missing :warning:

...les/models/components/pointer_network/generator.py
87.50%
31 Missing :warning:

...dules/pointer_network_taskmodule_for_end2end_re.py
94.45%
20 Missing :warning:

...ponents/pointer_network/bart_as_pointer_network.py
91.50%
13 Missing :warning:

.../models/components/pointer_network/pointer_head.py
93.10%
6 Missing :warning:

...pie_modules/taskmodules/pointer_network/metrics.py
95.41%
6 Missing :warning:

... and 4 more

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #23      +/-   ##
==========================================
- Coverage   96.36%   86.48%   -9.88%
==========================================
  Files          23       38      +15
  Lines        1731     4330    +2599
==========================================
+ Hits         1668     3745    +2077
- Misses         63      585     +522

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

Since #10 is merged, this is not required anymore.


Let HttpUrlConnectionBackend allow custom encodings

During checking backends for supporting custom encodings found out that we have special logic in HttpUrlConnectionBackend which prevents passing custom encoded bodies back to the client.
For consistency with other backends (every of them just let unsupported encoding pass through) here is the fix for HttpUrlConnectionBackend

Before submitting pull request:

[ ] Check if the project compiles by running sbt compile
[ ] Verify docs compilation by running sbt compileDocs
[ ] Check if tests pass by running sbt test
[ ] Format code by running sbt scalafmt

Thanks!
However, the mechanism is different in other backends. They allow specifying a customEncodingHandler: EncodingHandler (you can try searching for this), which specifies how to deal with encodings.
By default, all backends throw new UnsupportedEncodingException(s"Unsupported encoding: $ce")
Maybe you could try making HttpURLConnectionBackend follow suit and support an EncodingHandler parameter as well?

@adamw got your point, let me change that for HttpURLConnectionBackend

@adamw I've added test for custom encoding for HttpURLConnectionBackend, can you please help to implement rest of the tests for other backends

Released in 2.2.3, thanks!


Fixed Sidebar not scrollable. #5333

Fixed the Sidebar not scrollable issue due to the height of the sidebar.

Hey @gaearon , look into this PR.


已经解决了，把工件和机器数量调小就行

          已经解决了，把工件和机器数量调小就行

Originally posted by @Xiaomingge666 in https://github.com/Lei-Kun/End-to-end-DRL-for-FJSP/issues/2#issuecomment-2022467896

          已经解决了，把工件和机器数量调小就行

Originally posted by @Xiaomingge666 in #2 (comment)


Pviii - Book content grammatical errors

I've noticed a couple of grammatical errors in the paragraph under "Create The Project Directories". They're in the second sentence:
"...an src directory..." should be "...a src directory..." (incorrect indefinite article)
"...for unit tests tests:" should be "...for unit tests:" (too many tests)

Fixed, thank you!


changed syntax of dblink_connect

included 'dbname=' into the call, since it didn't run otherwise

Given the dblink_connect params:
https://www.postgresql.org/docs/12/contrib-dblink-connect.html
The db param could vary dramatically from user to user. I need to call it like this, since I'm targeting an instance on a different port:
SELECT parallelsql
(	'hostaddr=<IP_ADDRESS> port=5430 dbname=raster_benchmark',				--database
	'buffer_data',		    --table
	'feat.id',			            --variable to partition by processes
  '$STATEMENT',			--the statement to executed in parallel
	'statistic_mp',			--result table, has to be created first
	'feat',				--table alias used for split column
	8,				    --number of cores
	'1=1',				--replace string in the query
	50000	 			    --block size
);

Leaving it generic, but adding documentation about the connection string is probably a better idea-


⚠ ClamAV has degraded performance

In 4e19fde, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 65 ms

Resolved: ClamAV performance has improved in de89bd6.


Should we test all packages?

Currently in PR we only test modified packages to reduce the testing time. However, there are cases were a package can break if another package is modified. For example if we modify a function in common, we could break all packages that use that function, that are not tested if they are not modified.
Different approaches to handle this better:

Test in all PR all packages. This take some time, but ensure everything works.
Test all packages only in PRs that modify common. For most PR the testing time is still small, but we ensure that modifying common doesn't break any other package. This only solve the problem for common, but not other dependencies
Test only modified packages (current). Prioritize small testing time. If something breaks we will realize in the daily run.

@vm-packages  opinions?

In addition, if a package requires a dependency that hasn't been modify, the test suite fails:

So we need to at least test the dependencies. Ideas how to do this?

This happened for the notepadplusplus package rename and our daily run did not alert us. Is that a separate issue though?
For the general case here I think testing all packages is too much. Can we just test all packages by dependency (based on id/name)?

I vote for the third option:

Test only modified packages (current). Prioritize small testing time. If something breaks we will realize in the daily run.

Agreed, let's close this and reopen, if needed later.


Uncaught TypeError: Cannot read property 'query' of null

I have no idea why I get the exceptions "Uncaught TypeError: Cannot read property 'query' of null" when I click on x to clear Select2 values.
Here is my KnockoutJS v2.2.0 custom binding to Select2  v4.0.1: http://codepen.io/hipertracker/pen/mVLojK

Well, I found a workaround. My snippet has been updated

How did you solved this issue?

Also interested in this using jQuery..


[PyTorch] Fix detection of 3 in 3hd/h3d layouts

Description
This PR fixes how we detect the 3 in 3hd/h3d layouts when generating the output tensor shape. This only impacts the _qkvpacked functions in te.pytorch.cpp_extensions and only when users directly call those middle-layer APIs.
Fixes #1182.
Type of change

[ ] Documentation change (change only to the documentation, either a fix or a new content)
[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Infra/Build change
[ ] Code refractor

Changes
Please list the changes introduced in this PR:

Changed the way we detect 3 in qkvpacked layouts from relying on the value to relying on the location.

Checklist:

[x] I have read and followed the contributing guidelines
[x] The functionality is complete
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes

/te-ci pytorch

/te-ci pytorch


fix: make grid columns span correctly

What does this pull request change?
the grid positioning is referencing the grid lines, not the columns. Therefor I increment the end by one to reference the "ending line" of the column
Why is this pull request needed?
Issues related to this change
closes #638

That didn't quite turn out how I wanted...

Slightly tweaked an example, still not 100%, but I think the grid works as intended now, so the examples shouldn't be blocking the fix for other use-cases.


Can you tell me how to use many models in once post?

like face-detection-retail,face-reidentification-retail,landmarks-regression-retail,l want to use it in once post ,so l want to ust custom bankgrand.

You can serve multiple models by running model server with configuration file:
https://github.com/IntelAI/OpenVINO-model-server/blob/master/docs/docker_container.md#starting-docker-container-with-a-configuration-file
With single request you can run inference on a single model. If you want to run let's say, the same image on few models, you have to make separate requests for each model.

Would be nice to have this feature. Would prevent a lot of round trips when running multiple models such as face detection then emotion detection.

@17702513221 @chr15stevens This functionality has been added to the latest release 2021.1. Now you could create any pipelines of models for a single request. You can check the model ensemble example.


x/tools/cmd/godoc: line numbers are copied when highlighting text

I thought I fixed this in go/tools@0a14ca4e16cec5dfac9208167b18539631488a10, but copying source code with the latest tip also copies the line numbers, in Chromium on my Mac. Maybe a regression, or maybe a different CSS change caused this to break.

Github does some hacks with CSS ::before to avoid line number copies, I think I will do the same.

(Does anyone own godoc currently? If not I can own it I guess)
On Fri, Apr 21, 2017 at 19:15 GopherBot<EMAIL_ADDRESS>wrote:

CL https://golang.org/cl/41418 mentions this issue.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/golang/go/issues/20077#issuecomment-296340424, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAOSI9Mm37wcAW8W1Hjd5UWRo1QEV2Klks5ryWLSgaJpZM4NE77j
.

--
--
Kevin Burke
925.271.7005 | kev.inburke.com


Adjusted error handling to prevent unintentional drop(s)

See #1
Map error
Unfortunately, use of elegant map_err() was not possible anymore, as the lambda was capturing self. I tried the following, but may there is still a way?
self.pin_enable.set_high().map_err(|e| (EnablePinError(e), self))?;

I was undecided whether if let or mach is the better replacement. In the end, I took if let because it was a bit shorter.
Debug trait
Debug trait was needed for unit tests. I added it just for tests, as for production it is not guaranteed that external pins have Debug trait implemented. Do you confirm?
Constructor
The consuming issue also covers the constructor, though the footprint is probably not as heavy. First approach is, to release the pins in the tuple.
Alternatively, another non-initializing constructor would also be a possible.
Unit tests
I did not see the need to extend the unit tests, as the compiler/type system should actually prevent errors.
Looking forward to your feedback.

Looks good to me! It's more complex now, but solves the problem. Hope it works for your project :)


sort events by newest first

Makes it easier to know which events are the ones to care about and also helps when there are a large number of events, where the latest events may be outside of the visible part of the screen
Acceptance Criteria

When I go to any events tab it should be sorted by timestamp in descending order by default.
Sorting by timestamp should also be a secondary sort if we are sorting by another column.  E.g. if I am sorting by name, I would expect rows with the same name to be in descending order via timestamp.
This should be for all event tabs on detail pages such as sources, kustomizations, and helm releases.

@opudrovs FYI, trying on 0.9.3 I'm struggling to get sort by timestamp to alter the display of the data at all
https://user-images.githubusercontent.com/19860021/186162329-a90fdce1-f2b6-4039-b961-8b452cb8105c.mov

@sympatheticmoose sorting event by timestamp or reason was broken, I discovered it when I started working on the current issue. So, I fixed sorting in #2642 first.

Awesome 😄 thanks @opudrovs

@sympatheticmoose np, thanks for reporting it.

@sympatheticmoose @JamWils the Events table will by sorted by timestamp in reverse order (the earliest events on top) by default, this is clear.
Just to confirm the expected behavior: if I switch to sorting the Events table by another column value (for example, reason) and then switch back to sorting it by timestamp by clicking on the timestamp column header, the default sort direction for the timestamp column should be reverse (earliest events first) too after the switch (not only when sorting initially), correct? Because I will need to make some adjustments to the initial sort direction for this column only.

Suggestion: negate the sort value.
We're not displaying it as "time" (where a more recent value is bigger), we're displaying it as "age" (where a more recent value is smaller) - so sorting "1 second ago" before "1 hour ago" is the forwards order, not the reverse order.

@ozamosi interesting suggestion, do you mean to change the it from sortValue: (e: Event) => e.timestamp, to smth. like sortValue: (e: Event) => <now> - e.timestamp,?

Exactly - or just 0 instead of <now>, because the present has an unfortunate habit of not standing still.

OK, thanks, will do it! This simplifies things.


Escape the % in the tmux select-pane -t command to prevent process substitution by fish

Re #148

Let me know if you want me to squash commits before merging.

:+1:
I prefer it squashes, yes.
But we can do so from the GitHub UI.  Feel free to force-push what you like, of course.
Please ping me - giving also time for @christoomey to chime in, but I think this is good to fly.

:fireworks:


Added Cortex debug support from VSCode devcontainer

This PR enables target debugging from within a VS Code devcontainer.
It connects the Docker container to an underlying instance of OpenOCD to allow reflashing and source-level debugging for the target board.
Tested on macOS and Ubuntu

Thank you!


Framework agreement for exchanging certificate data (data sovereignty)

Closing issue to enable archiving of repository bpdm-certificate-management.


Fixes #838 (Incorrect stack traces) and abstracts error type creation

This change should provide a proper stack trace when errors are thrown (#838).
There was also some inconsistent, duplicated code between the error types. I abstracted it into a single function to create error types so it can be documented, tested, and maintained consistently. (It is not currently directly tested, but has full coverage through use cases).

Coverage decreased (-0.01%) to 92.382% when pulling 931cb1ffcb97eb8ea758cee2edde4797f3a3d1d4 on andyfleming:fix-838 into ae5ca814038c282622a79739d409713c2b08ceca on Netflix:master.

@steveorsomethin Thanks for the feedback. I've opened a fresh PR, #908, approaching this how we discussed above.
(Closing this one.)


[CocoaPods] LICENSE

The LICENSE section in Podspec is missing.
Unable to read the license file `LICENSE` for the spec `WeaverDI (0.12.0)`

Resolved in the last release. Thanks for filing.


Check vloc when using FieldDelta

FieldDelta currently does not check if two fields have different vertical staggerings.   MAPL should not support changes to staggerings until a realistic use case, so a check should be added to trap violations for now.

@tclune I don't think I remember any PRs about this, so I'll unstale for now.

Adding @pchakraborty as he'll undoubtedly be involved if we need to act on tihs.


issue121: Add more examples showing participant compensation behaviour

#121

@rdebusscher I correct the case for the @Path annotation. All comments are now resolved so I will now merge this PR.


Appstudio update test-custom-default-branch-dsay

Pipelines as Code configuration proposal

Pipelines as Code CI/test-custom-default-branch-dsay-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
8 seconds
init

✅ Succeeded
22 seconds
clone-repository

✅ Succeeded
43 seconds
build-container

✅ Succeeded
38 seconds
inspect-image

✅ Succeeded
34 seconds
deprecated-base-image-check

✅ Succeeded
5 minutes
clair-scan

✅ Succeeded
2 minutes
clamav-scan

✅ Succeeded
17 seconds
sbom-json-check

✅ Succeeded
42 seconds
optional-label-check

✅ Succeeded
38 seconds
label-check

✅ Succeeded
11 seconds
show-summary

✅ Succeeded
8 seconds
show-sbom


Add support for new IDE

With the imminent release of the new IDE can the extension be updated to work with it?
Particularly important as I can remember reading somewhere that we'll lose the option to use the existing one next year.

I'll take some time to look at the new IDE, but sadly I can't promise anything because this extension is totally build on many hacky things used in the previous IDE, one idea is to switch some internal RPC calls to the Google apps script API (which is the backend of clasp), but I don't know whether I can get enough time to do that.

I've not worked on a chrome extension before, but if you were willing to provide a little hand-holding I'd be willing to help with the upgrade??

Yes, it would be greatly appreciated!
@leonhartX do you have access to the new IDE or not yet?

I am willing to help with it.

Thanks, I have a partial PR created years ago to try to use the GAS API, if someone would like to take a look it will be very appreciate
#107
if the API part is fine, then we just need to modify some UI part to match the CSS of the new IDE(though I haven't tried it yet)

Just to add here some more info on the new IDE: https://www.benlcollins.com/apps-script/google-apps-script-ide/ and that the new IDE looks like it will be rolling out to everyone in the next 2 weeks.
+1 would love to see this extension supported in the new editor!

I also logged a feature request on Google's side to add this support natively to the new IDE: https://issuetracker.google.com/issues/176124900 star it or comment on it if you want to raise awareness

hey guys, with #107 and #130, I think it can works with the new IDE now, the main changes are

need to login google to use scripts API
since it moves to scripts API, we can get rid of some known issues like error with same function name and filename
add support to run in multi tabs

I'll take some more time to check and then publish it to the web store, but if someone wants to try it (and please help to find some bugs), please check out the code, and use Load unpacked in chrome extension settings to install it (need to remove the one installed from web store first)

hey guys, with #107 and #130, I think it can works with the new IDE now, the main changes are

need to login google to use scripts API
since it moves to scripts API, we can get rid of some known issues like error with same function name and filename
add support to run in multi tabs

I'll take some more time to check and then publish it to the web store, but if someone wants to try it (and please help to find some bugs), please check out the code, and use Load unpacked in chrome extension settings to install it (need to remove the one installed from web store first)

submitted 5.0.0 to chrome web store, waiting for review now

submitted 5.0.0 to chrome web store, waiting for review now

5.0.0 released

5.0.0 released

Thank you @leonhartX! And happy new year!

Thank you @leonhartX! And happy new year!

Thank you! This is fantastic news - an essential piece of kit and one more reason I don't have to jump back to the legacy editor.

Thank you! This is fantastic news - an essential piece of kit and one more reason I don't have to jump back to the legacy editor.

Sorry guys, I've noticed that since the OAuth app is still under review, google has a limitation to only allow 100 users give permission to the app, so you may see errors say it can not grant permission 🤦
I don't want to rollback the released one so please wait until the review completed or install it from source code to use v4.0.3 with the legacy editor, sorry for the inconvenience

Sorry guys, I've noticed that since the OAuth app is still under review, google has a limitation to only allow 100 users give permission to the app, so you may see errors say it can not grant permission 🤦
I don't want to rollback the released one so please wait until the review completed or install it from source code to use v4.0.3 with the legacy editor, sorry for the inconvenience


Depend on 4.1.0 version of RMQ connection

Thanks for contributing to Logstash! If you haven't already signed our CLA, here's a handy link: https://www.elastic.co/contributor-agreement/

LGTM


Siad -M, --modules flag

Specify modules for siad to load using the -M or --modules flag. Modules are specified by the first letter of the name. E.g. the default modules loaded are consensus (c), gateway (g), host (h), miner (m), renter (r), transaction pool (t), and wallet (w). This is equivalent to siad --modules cghmrtw.
On startup a status message is printed as each module is loaded. E.g.
$ siad
Loading...
(1/7) Loading gateway...
(2/7) Loading consensus...
(3/7) Loading transaction pool...
(4/7) Loading wallet...
(5/7) Loading miner...
(6/7) Loading host...
(7/7) Loading renter...
Finished loading in<PHONE_NUMBER>45 seconds

only blocking issue is the test for duplicates.


[19.03 backport] Improve readability of Windows connect error

backport of https://github.com/moby/moby/pull/39588
addresses DESKTOP-1286
Improve the readability of the connection error displayed to the user on
Windows when running docker commands fails by checking if the client is
privileged. If so then display the actual error wrapped in a generic
error "This error may indicate that the docker daemon is not running."
If not that display the actual error wrapped in a more specific error:
"In the default daemon configuration on Windows, the docker client must
be run with elevated privileges to connect."

all green; merging


Fixed the link for spec.

This PR fixed the link for spec under The Ingress resource in the Ingress docs.
Here is the spec for the API convention of spec not specific to Ingress.
Part of the PR: #43110

cc @sftim

/retitle Fixed a link to spec (Ingress API)

BTW, if we had an entry for “spec” in our glossary, we could use a glossary tooltip - and that would be even better.

rn, there is no entry for spec in our glossary, if it seems good we can add this.
Open to suggestions.

@sftim @tengqm
your thoughts on this.

/close


Extend dns config with filter

feature.network.dns config has to be extended with an optional local/remote filter, following feature.network.outgoing.filter pattern. The motivation for this is to catch cases where resolving name/address remotely is pointless (and prone to failures), because the resolved will be used for local outgoing connection.

In the referenced example you've put ports but not sure how that applies to DNS

I guess the goal here is to skip DNS resolution for host section of outgoing requests matching the filter. Maybe we can expand the outgoing filter instead e.g.
{
    "agent":
    {
        "ephemeral": true
    },
    "feature":
    {
        "network":
        {
            "incoming":
            {
                "ignore_localhost": true
            },
            "outgoing":
            {
                "ignore_localhost": true,
                "filter":
                {
                    "local":
                    [
                        { "pattern": ":9000", "dns": false },
                        { "pattern": ":9003", "dns": false },
                    ]
                }
            }
        }
    }
}

In the referenced example you've put ports but not sure how that applies to DNS

We've observed calls like getaddrinfo("<IP_ADDRESS>", "9000", ...) and the arguments for name resolution are a (name/address, port) pair. I thought the simplest way to do filtering is just copying what we have in the outgoing filter.

Maybe we can expand the outgoing filter instead

I'm not sure if we want to bind these two settings. If we keep them separate, we'll get an additional free feature from this change - general dns filter. Plus, outgoing filters allow for specifying protocol (udp/tcp), which is not known at the point of name resolution. @aviramha wdyt?

Yes, the separate dns has the benefit of a broader use case, but can be a bit confusing at the same time. But I think with good documentation this can be clarified. So in dns filter, we want to support this pattern: [name|address|subnet/mask]:[port] and for cases that the port is specified, the filter only applies on outgoing requests targeting the specified port. Is it correct?

I think the split is better having dns/outgoing their own.

for cases that the port is specified, the filter only applies on outgoing requests targeting the specified port. Is it correct?

The filter will be checked before any outgoing requests. The DNS query specifies both name/address and the port. So, for example, filter <IP_ADDRESS> would match queries for both <IP_ADDRESS>:80 and <IP_ADDRESS>:22, while filter <IP_ADDRESS>:80 would match only query <IP_ADDRESS>:80. If the local/remote filter does match, outgoing requests/connections to the resolved address will be done locally/remotely

@aghajani Thank you for helping us design and add this improvement!

for cases that the port is specified, the filter only applies on outgoing requests targeting the specified port. Is it correct?

The filter will be checked before any outgoing requests. The DNS query specifies both name/address and the port. So, for example, filter <IP_ADDRESS> would match queries for both <IP_ADDRESS>:80 and <IP_ADDRESS>:22, while filter <IP_ADDRESS>:80 would match only query <IP_ADDRESS>:80. If the local/remote filter does match, outgoing requests/connections to the resolved address will be done locally/remotely

@Razz4780 but we still going to support :9000 filter for dns as well, right? Meaning dns resolution step will be local for any request targeting that port no matter what the IP or host be. This is important in our case since we don't know the IP before the container starts, of course we can do some tricks here too but a flexible pattern will help a lot.  @aviramha I am happy to help, you have good community here.

@aghajani Ah yes, of course ^^


Max length for URLs

Motivation

Currently, our links are the page title or a collection of the page titles in the breadcrumbs. That leads to very long links, especially when they are not in Latin letters. This doesn't look very nice when you link to a page, and if you share a link, it might get cut off by certain apps. This leads to a couple of Sentry issues when the cutting happens in a way that the formatting breaks. We've also discussed this in the App repo.
Proposed Solution

I can think of a couple of solutions but I'm sure other people can think of plenty more:

We can have a hard limit on how many characters each part of a link has, and cut off everything after. This could lead to some pages having the same URL :(
We can force the user to create a short URL. This is probably annoying for everyone involved during implementation but would probably work after.
We can limit URLs to UTF characters, e.g. by using the German / English version of a page with the different language parameter. That might not shorten all URLs sufficiently though.
We can use only the short URLs but I assume people don't like them.

Alternatives

It's also not that common a problem. But it is annoying if someone shares an interesting link with you, presumably because they think that will help you with a specific problem you have, and then the link just errors out.
User Story

As an Integreat app user I want to be able to paste links to any communication app I want so that other people can follow the link that I shared and open the correct page.
Additional Context

First step for this issue: Do some research and present a few possible solutions

My first association: The snippet in the title on sentry is ?(weilheim-schongau/ru/%D0%BF%D0%BE%D0%BB%D0%B5%D1%82-%D1%83%D0%B1%D0%B5%D0%B6%D0%B8%D1%89%D0%B5/%D0%BB%D1%8C%D0%B3%D0%BE%D1%82%D1%8B-%D0%BF%D0%BE-%D0%BF%D1%80%D0%B5%D0%B4%D0%BE%D1%81%D1%82%D0%B0%D...
This suggests to me that the problem mainly arises from non-ascii characters being used in the slug, which also becomes the URL, where they get URL-encoded meaning that every non-ascii character becomes three ascii characters.
My gut feeling is that we should either just

change the way the slug is generated, restricting it to ascii characters (I know that for Japanese characters there exists a romanized way of spelling, maybe there are similar methods for most other languages as well…?)
or leave the slugs as they are, but translate them to some more compact ascii-only form for URLs


Nidoran(Pokemon) update

Nidoran is already present in the website. Please add some unique pokemon.


Tasks with input files marked 'skip when empty' should delete outputs only when it is known to be safe to do so

Currently, the outputs for these tasks are deleted when all source is removed, regardless of whether this is the desired outcome or not. Instead, we should delete these files only when we know this is what the user wants, consistent with whatever we come up with for cleaning up stale files more generally.
Expected Behavior

Current Behavior

Context

Steps to Reproduce (for bugs)

Your Environment

Build scan URL:

From #1012
Expected Behavior

When using the exclude method and actually all files from sourceDir are excluded, the destination dir should remain unchanged
Current Behavior

When using the exclude method and actually all files from sourceDir are excluded,
Then ALL the ecluded files that are already present in the DestinationDir are DELETED
Steps to Reproduce (for bugs)
e.g. in my file system:

sourceDir with several (and only! )  *.sql files
destDir  with the same files copied from sourceDir

then executing the following task
task testBBLCopy (type: Copy) {
	from sourceDir
	//exclude { details -> true }
	exclude '**/*.sql'
	into destDir
	includeEmptyDirs = false
}

Your Environment
Mac OS X ElCapitan 10.11.6
Gradle Distribution: Gradle wrapper from target build
Gradle Version: 3.2
Java Home: /Library/Java/JavaVirtualMachines/jdk1.8.0_102.jdk/Contents/Home
JVM Arguments: None
Program Arguments: None
Gradle Tasks: testBBLCopy

@ danreb25 Could you elaborate why that is a problem for you? Do you have other copy tasks copying into the same destination directory? Where does your destination directory reside? Inside the build directory?
Doesn't executing the copy task after a clean have the same end result as deleting all the files in the destination directory when there is nothing to copy (i.e. the destination directory is empty)?

Hi wolfs sorry for the delay,
*why that is a problem for you?
I expect the  Copy task to copy files from sourceDir to destinationDir and not to "SYNC"
my destinationDir with the sourceDir. I would use the Syn Task for this.

Action on fileSystem
GradleTask runned

Origin Dir
Destination dir

initial state
null
-->
[tex1.txt;tex2.txt;tex3.txt]
:white_check_mark:[text4.txt]

null
myCopyTask
-->
[tex1.txt;tex2.txt;tex3.txt]
:white_check_mark:[text4.txt;tex1.txt;tex2.txt;tex3.txt]

rm from OriginDir tex1.txt;tex2.txt
null
-->
[tex3.txt]
:white_check_mark:[text4.txt;tex1.txt;tex2.txt;tex3.txt]

null
myCopyTask
-->
[tex3.txt]
:white_check_mark:[text4.txt;tex1.txt;tex2.txt;tex3.txt]

null
myCopyTask (rerunned)
-->
[tex3.txt]
:white_check_mark:[text4.txt;tex1.txt;tex2.txt;tex3.txt]

rm from OriginDir tex3.txt
null
-->
[-]
:white_check_mark: [text4.txt;tex1.txt;tex2.txt;tex3.txt]

null
myCopyTask
-->
[-]
:red_circle: [text4.txt]

in breaf when there is nothing in the filtered files in OriginDir all the previous copied files (and only those copied sith the gradle task) are deleted.
With Gradle 3.1 when running the step 3 the task was considered up-to-date and the file wasn't deleted in the destinationDir
The problem exists since gradle 3.2> (tested also on 3.3)
*Do you have other copy tasks copying into the same destination directory?
yes
*Where does your destination directory reside?
Outside the build directory
*Doesn't executing the copy task after a clean have the same end result as deleting all the files in the destination directory when there is nothing to copy (i.e. the destination directory is empty)?
I still need to test this one more in depth

I'm still seeing this behavior in Gradle 4.8.1

@rsshilli Where are you copying to? Somewhere in the build directory?

Yes.  We copy in the results from a previous test into build/test/report as part of the CICD system, and then run another test which copies source code into a number of directories such as build/test/features (but nothing into the report directory) and voila, the report directory is destroyed.


../lib/src/widgets/input_toolbar/input_toolbar.dart(49,32): error GD65BB2B6: The getter 'View' isn't defined for the class 'InputToolbarState'.

../lib/src/widgets/input_toolbar/input_toolbar.dart(49,32): error GD65BB2B6: The getter 'View' isn't defined for the class 'InputToolbarState'.

Can you provide more information, what version you're using, and the code to reproduce your issue

I did some research and it's because you didn't upgrade to Flutter 3.10
You can either upgrade your flutter version or use the flutter-3.7 branch that I just created in case you aren't ready to upgrade yet


Implement pagination with an iterator-like API

This adds two properties and one method to the API:

canFetchMore, true iff the last response was paginated and more pages can be fetched.
fetchMore(), call to generate a request for the next page, or for the first page if nor previous request has been made.
allResults, an array containing results from all fetched pages.

Will rebase after #3 has been merged

I got this error.

Note: You have to run my use-comment-element on frontend as you have merged the one in code-highlighter. And master now complains about a duplicate comment.html

The access token seems to be missing when generateRequest() is called. I'll look into it. Probably some issue with the order of data binding events

I cannot reproduce the problem, does it still error after a bower install / update?

Should be fixed now!


Block nvm unalias default

I did:
nvm unalias default
Deleted alias default

IMO it shouldn't be possible to do nvm unalias default

Why not? default is an alias like everything else, and it's a valid use case to not want anything to be defaulted (and it happens to be the way nvm starts out as well)

I guess it comes down to the personal opinion.
For me the default alias has a special meaning and therefore shouldn't be deletable or at least a notice/warning should be displayed if a default alias has been deleted.

I could certainly print a slightly different message when you unalias "default", but unaliasing is such a rare corner case that I'm not sure I see the hazard.

I guess its story time:
I had a node version aliased as default. Then I needed to change it but checked the current one beforehand using nvm current. I installed a new version then set it as the current one with the wrong alias nvm alias current 5.1.0. The next time I started a shell I was wondering why some stuff didn't work and did a nvm ls to see the current and default aliases. Already confused I did nvm unalias default. After some time I noticed all the bad things I have done and though: "damn, I could really have used some kind of warning before getting rid of the default alias"
Well, since such problems seem to be limited to potato-level intelligent or overworked developers I guess you don't need to bother.
TL;DR: I confused current and default

That does make some sense. However, you can just nvm alias default whatever to restore it, so it's not a huge deal.
What I think I will do, though, is make sure the unalias message includes the thing the alias used to point to.

I guess we have a deal here :+1:
Thanks for the great work on nvm. It saved my sanity more than once.


chore: apply pre-commit

Description

Tests performed

Not applicable.
Effects on system behavior

Not applicable.
Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.

[x] I've confirmed the contribution guidelines.
[x] The PR follows the pull request guidelines.

In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.

[ ] The PR follows the pull request guidelines.

Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.

[ ] There are no open discussions or they are tracked via tickets.

After all checkboxes are checked, anyone who has write access can merge the PR.

Pre-commit CI is still failing from Python codes, but we can ignore it for now.


Subgroups

Hey, I've transferred from GroupManager to PermissionsEx and one feature in GroupManager that I used was Subgroups.
Subgroups was basically like inherit. Players can be in a group but inherit the permissions of another group. I'm not sure how to do this with PermissionsEx; I tried the /pex user Loonyy group add Helper but that either seems to put me in the Helper group or add the group to me but give me the Helper prefix, it's a little unreliable.
Any help?

See this example first.
If Stormbow joined the server and was promoted to the group named "Member", and "Member" inherits permissions from the group named "Guest" (which Stormbow was in originally as a new person on the server; lines 29 and 30), Stormbow gains all of the permissions of both groups.  Stormbow wouldn't need to actually be a member of both groups.
Is that was you're asking about?

No, but thank you for the reply.
I already have group inherits set up correctly. I'm wanting player-group inherits.
I need player group inherits because if I hire a player as Moderator, and they donate for the highest rank (Emperor) with lots of permissions in that Emperor group, I want their main rank to continue to be Moderator but I want that player to inherit the Emperor rank so they still have access to Emperor commands and permissions. Does that make sense?

Basically;
A player can be in multiple groups at once but only one specific groups' prefix shows up.

That's accomplished with group add and weight values.  See the example I linked above.

"Loonyy" would start in the "Guest" group of that example.  If you wanted them to also be an Admin, you'd type /pex user Loonyy group add Admin and they'd now be a member of 2 groups.

I've just setup all the ranks and weights for each group and the wrong prefix for groups on users are showing up still. I am using the latest version of PEX but I'm on Spigot 1.11

Copy your full permissions.yml file to Pastebin for me to look at.

Here you go:
http://pastebin.com/VBRSc1EN

Which user is getting the wrong prefix?  Which prefix should they be getting?

Every group I've tried it with, it doesn't work.
I've mainly been doing it with my staff ranks and donation ranks, here's some examples:
Moderator and Emperor.
Architect and Defender.
Helper and Knight.

Also, when I use the PEX command in game to check the weight and rank, it's all listed as normal.

Many, many problems in that file that I can see so far:
• Groups should not have the same rank values.
• Groups should not have the same weight values.
• All of your option: sections are not indented properly.
• All of your permissions: sections are indented too far.
• All of your weight values are backwards.  Owner should be 1 and whichever single group has default: true should be the highest number, typically 1000.

Brb while I /suicide because I now have to fix all this

Thanks for the support :)

You're welcome.  Let me know— after you've gotten it all fixed —if you run into other problems.

(You can still reply here.  It's just not on the 'open' tickets list now.)

I fixed the issues in the config;
The issue I can see right now is that Emperor is overriding all other groups, meaning if I add someone the Moderator group and Emperor, the Emperor prefix will show.
Here's my Permissions file:
http://pastebin.com/tdgVAgES

In the server.properties file on the server, is online-mode set to true?

Yes it is.

Are there any error messages at all in the startup log?

Nope, no errors.

Show me the full file, including all of the user information.  Upload it to Dropbox or something if it's too long to paste.

Nevermind.  I see the problem.
  weight: ‘500'

is not encapsulated in apostrophes.
weight: '500'

Sigh. Longest sigh of my life. /suicide x10.
Thanks for all the helper, mind answering one more question?
Is there a way to easily add someone a prefix from two groups without doing it manually?
So I can just give someone the prefix from builder and helper rank so it would look like:
[Builder] [Helper] Loonyy

That, unfortunately, would be entirely up to whatever chat management plugin you use.
Personally, I've always stayed away from double and long prefixes displayed in-game because they clutter up chat too much, so I'm not 100% sure on how to do it.  I know others have done it, but I don't know how and haven't done it myself.

My [Prefix/Suffix Guide]9https://dev.bukkit.org/bukkit-plugins/permissionsex/forum/69856-basic-prefix-suffix-tutorial/) only details setting up one prefix.

Yeah me too, it's only for double staff members (Helper and Builder) - although I might use suffix instead.
Thanks for all your help :)

You're welcome. :)  Have fun.


Improve Readme file

Improve the readme file

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Beinan Wang seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Remove extra whitespace from heredoc_end

Hello!
Please see https://github.com/nvim-treesitter/nvim-treesitter/issues/4333 for reference, the heredoc_end rule includes whitespace---this PR removes that whitespace so the heredoc_end is just the identifier.
Checklist:

[x] All tests pass in CI
[x] There are sufficient tests for the new fix/feature
[x] Grammar rules have not been renamed unless absolutely necessary (x rules renamed)
[x] The conflicts section hasn't grown too much (x new conflicts)
[x] The parser size hasn't grown too much (master: STATE_COUNT, PR: STATE_COUNT) (check the value of STATE_COUNT in src/parser.c)

Thanks


Allow checking for user's roles in ManagerInterface::userHasPermission()

Q
A

Is bugfix?
❌

New feature?
✔

Breaks BC?
❌

The changes have been synced with main branch.


SSI- Add signature to document field for update/revoke to registry

SSI needs to send the document along with the signature (signed did doc) (proof section) so it can send the update/revoke call to registry (document request field)

knox go/gomono has this as part of revocation to registry mgmt- need to copy over on the rust side.
will need to add revocation/update functionality to Rust SSI SDK as well to be on par on functionality


Using pydantic custom encoders

Attempt at fixing #18
Not sure this is something you want though
Could use more tests

Awesome! Thank you! :tada:


Feature suggestion to insert static text of weather when calling template plugins

I want to include weather information as static text in my diary, which was created using a template plugin.
The plugin currently uses dynamic syntax like "<% tp.date.now("YYYY-MM-DD", 0)%>" to display the current date, but it doesn't support dynamic weather data.
Therefore, I'm wondering if it would be possible to introduce a new syntax that allows me to add static weather text to the generated template.

That ability already exists, just add the following to your templates for static weather...
You can place the following strings in your templates and when creating a new document using the template, they will automatically be replaced with the corresponding weather strings.

%weather1% - Inserts weather string format One
%weather2% - Inserts weather string format Two
%weather3% - Inserts weather string format Three
%weather4% - Inserts weather string format Four

You can customize the weather string formats in the plugins settings.
If want dynamic weather in your templates that is a bit more involved. Have a look at the  file EXAMPLE.md for details.

Thank you so much for your reply. Please forgive my carelessness for not reading the document carefully. This is an excellent plugin, and I have already recommended it on my blog

Thank you very much, have a great day.


tests-ci : kernel: timer: starve test No Console Output(Timeout)

**Describe the bug**
starve test is No Console Output(Timeout) on zephyr-v3.2.0-1326-ge2cd634185c6 on mimxrt1170_evk_cm7

see logs for details

**To Reproduce**
1.
```
scripts/twister --device-testing --device-serial /dev/ttyACM0 -p mimxrt1170_evk_cm7      --sub-test kernel.timer
```
2. See error

**Expected behavior**
test pass

**Impact**

**Logs and console output**

```
*** Booting Zephyr OS build zephyr-v3.2.0-1326-ge2cd634185c6 ***

Running TESTSUITE starve_fn
START - test_starve
Cycle clock runs at 32768 Hz
There are 8 cycles per tick (4096 Hz ticks)
[     0.022] 1: still running, would pass at 3600 s
[    60.012] 601: still running, would pass at 3600 s
```

**Environment (please complete the following information):**
 - OS: (e.g. Linux )
 - Toolchain (e.g Zephyr SDK)
 - Commit SHA or Version used: zephyr-v3.2.0-1326-ge2cd634185c6

@dleach02 , @mmahadevan108


KYLIN-3991 Provide more config options for Flink cube engine

JIRA ref : https://issues.apache.org/jira/browse/KYLIN-3991

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Hi @shaofengshi I opened a PR to fix the parallelism issue. It seems the Travis has an error. So if you have time, pls review this PR. Thanks.


Updating Gradle plugin version and dependencies.

Hi @piotrek1543, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

In order for us to evaluate and accept your PR, we ask that you sign a contribution license agreement. It's all electronic and will take just minutes. I promise there's no faxing. https://cla.microsoft.com.

TTYL, MSBOT;

@piotrek1543, Thanks for signing the contribution license agreement so quickly! Actual humans will now validate the agreement and then evaluate the PR.
Thanks, MSBOT;

Thank you for your contribution! Have you thought about creating a new game mode?

@JuliaNik I rather thinking about updgrading existing and adding polish language support (some guys on Google play complains about missing this feature). Cheers


Also add the task list as a label when creating an issue on github

Issue opened with imdone.io from code comment on 9f751f1
BACKLOG  Also add the task list as a label when creating an issue on github +waffle id:10

https://github.com/imdone/imdone-atom/blob/9f751f14f022fed15d90b360e4f50f43250dcb82/lib/plugins/plugin-view.coffee#L100-L105

DOING  Also add the task list as a label when creating an issue on github +waffle id:10 gh:242

https://github.com/imdone/imdone-atom/blob/44b9c7651d472506ec8c8c66ccf49260b34949e0/lib/plugins/plugin-view.coffee#L100-L105

APPROVED

DOING  Also add the task list as a label when creating an issue on github!!! +waffle id:10 gh:242

https://github.com/imdone/imdone-atom/blob/37b70ac8c5cd7c3c1aa9e85712323edc29854b90/lib/plugins/plugin-view.coffee#L100-L105

Issue closed with imdone.io by removing a comment.

https://github.com/imdone/imdone-atom/blob/b7251b7ccf00e69c484ae85d9668078cedb7152e/lib/plugins/plugin-view.coffee#L100-L105


Recommit blocks

This PR:

Introduces the notion of recommitting, which is asserting that if a block was valid in the past, it's always going to be valid if part of canon. We can avoid revalidating, and if there is a chain of recommittable blocks, then we can skip intermediate hashing by using past-calculated ledger digests.
Introduces an optimization for recommitting large chains of blocks.

The motivation behind this PR is to resolve an observed phenomenon when syncing at ~400k blocks, where there are 2 valid chains seen based on block 43k competing. This PR should help eliminate that fork by allowing peers to not be stuck recommitting for hours each time they sync a few blocks from one of these forks, and it overtakes the other.
Has been running locally for almost a week, been keeping at canon tip much more reliably.

This should fix https://github.com/AleoHQ/snarkOS/issues/1189.


[BUG] When shared_database crashes, it does not get added to the results YAML file

Describe the bug

This might be the same behaviour with any other test that crashes. But I can only confirm it for shared_database for now.

When shared_database crashes, it is not added to the results YAML file. Screenshot below.

To reproduce

Install the sample CNF at sample-cnfs/sample_coredns cnf
Run the workload tests ./cnf-testsuite workload

Expected behaviour

Check if this error can be handled/rescued with a human readable error message.
If the shared_database test crashes or fails, then the test should still be added to the results YAML file.

The cause of the crash was fixed as a part of another ticket (#1342). This ticket can be of lower priority.

@HashNuke is this issue still relevant for CNTI?

Fixed by #2045


[Bug]: Don`t have permissions for start routing deamon tun2socks on Debian

Application
Outline Client
Describe the bug
Couldnt start routing daemon tun2socks, dont have permissions.
Steps to reproduce
Debian 12, KDE 5.27 Wayland

Download APP from Ouline get started
chmod u+x  ~/Outline-Client.AppImage
Start from ~/Outline-Client.AppImage

What did you expect to happen?
I expected a window to appear for entering the Administrator password. After a warning window indicating the need for administrator rights.
What actually happened?
Apart from a window warning about a possible request for Administrator rights, nothing started.
Outline Version
1.14.0 (480310)
What operation system are you using?
Linux
Operating System Version
Debian 12
Screenshots and Videos
$ ./Outline-Client.AppImage
Outline is  #starting
loading web app from file:///tmp/.mount_OutlinRtt6ud/resources/app.asar/client/www/index_electron.html?appName=Outline
Could not load active tunnel:  [Error: ENOENT: no such file or directory, open '/home/debian/.config/Outline/connection_store'] {
  errno: -2,
  code: 'ENOENT',
  syscall: 'open',
  path: '/home/debian/.config/Outline/connection_store'
}
Checking for update
Update for version 1.14.0 is not available (latest version: 1.14.0, downgrade is disallowed).
[16736:1124/135108.009679:ERROR:gl_surface_presentation_helper.cc(260)] GetVSyncParametersIfAvailable() failed for 1 times!
[16736:1124/135125.717753:ERROR:gl_surface_presentation_helper.cc(260)] GetVSyncParametersIfAvailable() failed for 2 times!
[16736:1124/135136.285176:ERROR:gl_surface_presentation_helper.cc(260)] GetVSyncParametersIfAvailable() failed for 3 times!
connecting to mpsrv Outline (7XXXXX95-XXXX-XXXX-XXXX-dbXXXXXX1d17)...
[tun2socks] - checking connectivity ...
[tun2socks - EXIT]: Exited with code 0
UDP support: true
starting routing daemon
[tun2socks] - starting to route network traffic ...
Routing daemon socket setup failed Error: connect ENOENT /var/run/outline_controller
    at PipeConnectWrap.afterConnect [as oncomplete] (node:net:1157:16) {
  errno: -2,
  code: 'ENOENT',
  syscall: 'connect',
  address: '/var/run/outline_controller'
}
could not connect: Error: {"code":"ERR_ROUTING_SERVICE_NOT_RUNNING","message":"routing daemon is not running","cause":{"code":"ERR_INTERNAL_ERROR","message":"Error: connect ENOENT /var/run/outline_controller"}}
    at Socket.r (/tmp/.mount_OutlinRtt6ud/resources/app.asar/output/client/electron/index.js:2:678675)
    at Object.onceWrapper (node:events:646:26)
    at Socket.emit (node:events:526:28)
    at Socket.emit (node:domain:475:12)
    at emitErrorNT (node:internal/streams/destroy:157:8)
    at emitErrorCloseNT (node:internal/streams/destroy:122:3)
    at process.processTicksAndRejections (node:internal/process/task_queues:83:21)
Error occurred in handler for 'outline-ipc-start-proxying': Error: {"code":"ERR_ROUTING_SERVICE_NOT_RUNNING","message":"routing daemon is not running","cause":{"code":"ERR_INTERNAL_ERROR","message":"Error: connect ENOENT /var/run/outline_controller"}}
    at Socket.r (/tmp/.mount_OutlinRtt6ud/resources/app.asar/output/client/electron/index.js:2:678675)
    at Object.onceWrapper (node:events:646:26)
    at Socket.emit (node:events:526:28)
    at Socket.emit (node:domain:475:12)
    at emitErrorNT (node:internal/streams/destroy:157:8)
    at emitErrorCloseNT (node:internal/streams/destroy:122:3)
    at process.processTicksAndRejections (node:internal/process/task_queues:83:21)
[tun2socks - EXIT]: Exited with code 1
[tun2socks] - terminated due to: a [Error]: {"code":"ERR_SYSTEM_VPN_SETUP_FAILURE","message":"failed to open TUN device","cause":{"code":"ERR_INTERNAL_ERROR","message":"ioctl: operation not permitted"}}

    at t.ChildProcessHelper.b (/tmp/.mount_OutlinRtt6ud/resources/app.asar/output/client/electron/index.js:2:674688)
    at ChildProcess.emit (node:events:526:28)
    at ChildProcess.emit (node:domain:475:12)
    at ChildProcess._handle.onexit (node:internal/child_process:291:12) {
  exitCode: 1
}
could not stop tun2socks: {"code":"ERR_SYSTEM_VPN_SETUP_FAILURE","message":"failed to open TUN device","cause":{"code":"ERR_INTERNAL_ERROR","message":"ioctl: operation not permitted"}}

disconnected from 7XXXXXX5-XXXX-XXXX-XXXX-dbXXXXXX1d17
installing outline routing service...
copying service installation files to /tmp/kCMgp2
all service installation files copied to /tmp/kCMgp2 successfully
trying to run command as root:  trap "/usr/bin/chattr -R -i /tmp/kCMgp2" EXIT; /usr/bin/chattr -R +i /tmp/kCMgp2 && /usr/bin/echo "e06XXXXXXXXXXXXXXXXXXXe2e7de0de1e46386eed713d7141f15d9e4XX53c1cd  /tmp/kCMgp2/install_linux_service.sh" | /usr/bin/shasum -a 256 -c && /usr/bin/echo "0a2XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXebb4f4057e06620b7fdC4a  /tmp/kCMgp2/OutlineProxyController" | /usr/bin/shasum -a 256 -c && /usr/bin/echo "8bfaXXXXXXXXXXXXXXXXXXXXXXXXXXXXbf8c2d365a2a7f3b6c0X32c2aXc915a2  /tmp/kCMgp2/outline_proxy_controller.service" | /usr/bin/shasum -a 256 -c && "/tmp/kCMgp2/install_linux_service.sh" "debian"
undefined
undefined
command is running as root but failed:  Error: Unable to find pkexec or kdesudo.
    at e (/tmp/.mount_OutlinRtt6ud/resources/app.asar/output/client/electron/index.js:2:642401)
    at /tmp/.mount_OutlinRtt6ud/resources/app.asar/output/client/electron/index.js:2:642554
    at FSReqCallback.oncomplete (node:fs:198:21)
^C^C

To fix the problem you need to install pkexec or kdesudo.
I choose apt install pkexec and i got a window to enter the root password


feat(kafka): enforced 'string' format for kafka trace headers and removed 'binary' support

Kafka header migration phase-2

Always send Kafka trace correlation headers in  format 'string'

Remove the ability to configure the header format

Remove the code associated with sending headers in format 'binary' or sending 'both' formats.

Tasks

[x] Remove ability to configure the header format
[x] Remove unwanted tests
[x] PR for updating test suite https://github.ibm.com/instana/tracer-test-suite/pull/96
[x] Create a PR to add a deprecation warning https://github.com/instana/nodejs/pull/1311
[x]  Manual testing

Post Release

[ ] Update migration table in public doc see kafka-migration phase-2
[ ] Merge test suite PR

refs https://jsw.ibm.com/browse/INSTA-809
This PR should merge against V4 branch.

How about we separate the tasks in https://jsw.ibm.com/browse/INSTA-786?

Undone
Done

@aryamohanan Please change the target branch to v4!


feat: As a user i want to get decoded attestation data via the subgraph

What does this PR do?
Decodes the attestationData field of an newly created Attestation.
Related ticket
Fixes #226
Type of change

[ ] Chore
[ ] Bug fix
[X] New feature
[ ] Documentation update

Check list

[ ] Unit tests for any smart contract change
[ ] Contracts and functions are documented

Codecov Report

Merging #229 (e59b4b6) into dev (5a06646) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##              dev     #229   +/-   ##
=======================================
  Coverage   93.88%   93.88%
=======================================
  Files          11       11
  Lines         180      180
  Branches       41       41
=======================================
  Hits          169      169
  Misses          7        7
  Partials        4        4

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


Error: "document is not defined" when using fastboot

I'm getting the following error when trying to authentication on my app using fastboot.
Error while processing route: index document is not defined ReferenceError: document is not defined
    at Class.<anonymous> (/Users/abhishek/cb/codingblocks.online.projectx/tmp/broccoli_merge_trees-output_path-bb7UHGD0.tmp/assets/addon-tree-output/ember-cookies/services/cookies.js:62:1)
    at ComputedPropertyPrototype.get (/Users/abhishek/cb/codingblocks.online.projectx/tmp/broccoli_merge_trees-output_path-bb7UHGD0.tmp/assets/ember-metal.js:3407:1)
    at get (/Users/abhishek/cb/codingblocks.online.projectx/tmp/broccoli_merge_trees-output_path-bb7UHGD0.tmp/assets/ember-metal.js:2780:1)
    at _getPath (/Users/abhishek/cb/codingblocks.online.projectx/tmp/broccoli_merge_trees-output_path-bb7UHGD0.tmp/assets/ember-metal.js:2804:1)
    at get (/Users/abhishek/cb/codingblocks.online.projectx/tmp/broccoli_merge_trees-output_path-bb7UHGD0.tmp/assets/ember-metal.js:2787:1)
    at Class.get (/Users/abhishek/cb/codingblocks.online.projectx/tmp/broccoli_merge_trees-output_path-bb7UHGD0.tmp/assets/ember-runtime/mixins/observable.js:38:1)
    at Class.<anonymous> (/Users/abhishek/cb/codingblocks.online.projectx/tmp/broccoli_merge_trees-output_path-bb7UHGD0.tmp/assets/addon-tree-output/ember-cookies/services/cookies.js:66:1)
    at ComputedPropertyPrototype.get (/Users/abhishek/cb/codingblocks.online.projectx/tmp/broccoli_merge_trees-output_path-bb7UHGD0.tmp/assets/ember-metal.js:3394:1)
    at get (/Users/abhishek/cb/codingblocks.online.projectx/tmp/broccoli_merge_trees-output_path-bb7UHGD0.tmp/assets/ember-metal.js:2780:1)
    at Class.get (/Users/abhishek/cb/codingblocks.online.projectx/tmp/broccoli_merge_trees-output_path-bb7UHGD0.tmp/assets/ember-runtime/mixins/observable.js:38:1)
    at Class.read (/Users/abhishek/cb/codingblocks.online.projectx/tmp/broccoli_merge_trees-output_path-bb7UHGD0.tmp/assets/addon-tree-output/ember-cookies/services/cookies.js:110:1)
    at Class._read (/Users/abhishek/cb/codingblocks.online.projectx/tmp/broccoli_merge_trees-output_path-bb7UHGD0.tmp/assets/addon-tree-output/ember-simple-auth/session-stores/cookie.js:220:1)
    at Class.restore (/Users/abhishek/cb/codingblocks.online.projectx/tmp/broccoli_merge_trees-output_path-bb7UHGD0.tmp/assets/addon-tree-output/ember-simple-auth/session-stores/cookie.js:199:1)
    at Class._callStoreAsync (/Users/abhishek/cb/codingblocks.online.projectx/tmp/broccoli_merge_trees-output_path-bb7UHGD0.tmp/assets/addon-tree-output/ember-simple-auth/internal-session.js:124:1)
    at Class.restore (/Users/abhishek/cb/codingblocks.online.projectx/tmp/broccoli_merge_trees-output_path-bb7UHGD0.tmp/assets/addon-tree-output/ember-simple-auth/internal-session.js:88:1)
    at Class.beforeModel (/Users/abhishek/cb/codingblocks.online.projectx/tmp/broccoli_merge_trees-output_path-bb7UHGD0.tmp/assets/addon-tree-output/ember-simple-auth/initializers/setup-session-restoration.js:22:1)
    at applyHook (/Users/abhishek/cb/codingblocks.online.projectx/tmp/broccoli_merge_trees-output_path-bb7UHGD0.tmp/assets/router.js:210:1)
    at UnresolvedHandlerInfoByParam.runSharedModelHook (/Users/abhishek/cb/codingblocks.online.projectx/tmp/broccoli_merge_trees-output_path-bb7UHGD0.tmp/assets/router.js:665:1)
    at UnresolvedHandlerInfoByParam.runBeforeModelHook (/Users/abhishek/cb/codingblocks.online.projectx/tmp/broccoli_merge_trees-output_path-bb7UHGD0.tmp/assets/router.js:639:1)
    at tryCatcher (/Users/abhishek/cb/codingblocks.online.projectx/tmp/broccoli_merge_trees-output_path-bb7UHGD0.tmp/assets/rsvp.js:215:1)
    at invokeCallback (/Users/abhishek/cb/codingblocks.online.projectx/tmp/broccoli_merge_trees-output_path-bb7UHGD0.tmp/assets/rsvp.js:393:1)
    at publish (/Users/abhishek/cb/codingblocks.online.projectx/tmp/broccoli_merge_trees-output_path-bb7UHGD0.tmp/assets/rsvp.js:379:1)
    at /Users/abhishek/cb/codingblocks.online.projectx/tmp/broccoli_merge_trees-output_path-bb7UHGD0.tmp/assets/ember-testing/ext/rsvp.js:14:1
    at invokeWithOnError (/Users/abhishek/cb/codingblocks.online.projectx/tmp/broccoli_merge_trees-output_path-bb7UHGD0.tmp/assets/backburner.js:214:1)
    at Queue.flush (/Users/abhishek/cb/codingblocks.online.projectx/tmp/broccoli_merge_trees-output_path-bb7UHGD0.tmp/assets/backburner.js:125:1)
    at DeferredActionQueues.flush (/Users/abhishek/cb/codingblocks.online.projectx/tmp/broccoli_merge_trees-output_path-bb7UHGD0.tmp/assets/backburner.js:278:1)
    at Backburner.end (/Users/abhishek/cb/codingblocks.online.projectx/tmp/broccoli_merge_trees-output_path-bb7UHGD0.tmp/assets/backburner.js:410:1)
    at Backburner._runExpiredTimers (/Users/abhishek/cb/codingblocks.online.projectx/tmp/broccoli_merge_trees-output_path-bb7UHGD0.tmp/assets/backburner.js:831:1)
    at Timeout.Backburner._boundRunExpiredTimers (/Users/abhishek/cb/codingblocks.online.projectx/tmp/broccoli_merge_trees-output_path-bb7UHGD0.tmp/assets/backburner.js:368:1)
    at Timeout.wrapped [as _onTimeout] (/Users/abhishek/cb/codingblocks.online.projectx/tmp/broccoli_merge_trees-output_path-bb7UHGD0.tmp/assets/bower_components/raven-js/dist/raven.js:351:1)
    at ontimeout (timers.js:475:11)
    at tryOnTimeout (timers.js:310:5)
    at Timer.listOnTimeout (timers.js:270:5)

I am not sure how to go about debugging this. Or Even if this belongs to ember-simple-auth-token or ember-simple-auth.

The stack trace suggests that ember-cookies tries to read the cookies from document.cookies vs. the request headers which seems strange as this should be true in FastBoot. Can you check service:fastboot is available on the owner in your application?

@marcoow I tried placing a breakpoint, it seems that this.get('_fastBoot').get('isFastBoot') is true, however this.get('_fastBoot.isFastBoot') is undefined.
Any clue as to why this might happen?

Ugh, that seems surprising. What versions of Ember/Ember CLI/ESA/Fastboot are you using?

closing due to no feedback


Update node local dns to v1.22.5

How to categorize this PR?

/area networking
/kind enhancement
What this PR does / why we need it:
Update node local dns to v1.22.5
Which issue(s) this PR fixes:
None.
Special notes for your reviewer:
Release note:

Update node local dns to v1.22.5

/invite @DockToFuture

/assign @DockToFuture


Update README.md to reflect project transition

This should be merged upon TOC decision to Archive the project.

Approved


REST API - Repeater Fields Search "AND" Issue

When using the API rest service to search for elements through the metadata we have a problem to filter when we have multiple metadata groups/repeater values.
For example, if we want to filter those metadata that have as metadata lomes/classification/purpose/value = "education level" and lomes/classification/taxonPath/taxon/id = 1 if we put the query where /xml/lomes/classification/purpose/value IS 'discipline' AND /xml/lomes/classification/taxonPath/taxon/id IS 1 gets us all those who meet that condition even if the taxon is in another classification.
For further context please see internal issue number zendesk #101356
(Internal Ayesa reference number 2471)

@NextEdServices please be aware that most of the community do not have access to zendesk issues (including myself) so please make sure you include ALL details in this github issue.


docs: add Rsdoctor documents

Summary
chore(docs): add Rsdoctor documents.

Checklist

[ ] Tests updated (or not required).
[x] Documentation updated (or not required).

Too many mistakes, please check the documents yourself 😂

Too many mistakes, please check the documents yourself 😂

@chenjiahan  Sorry for so many mistakes, i have fix some mistakes, pls review this again. Thanks~

👍


When publishing as a tool, readme inclusions are resolved from publish dir

Error looks like:
warning NG001: Failed to resolve include: ../../readme.md#content. File not found at expected location /home/runner/work/.../bin/Debug/net6.0/publish/../../readme.md.

This is no longer the case in latest stable


Should --keep-coastlines be kept?

Should the former --keep-coastlines be kept or was it right to remove it, since it breaks often?
If not storing the coastlines, should a replacement be included?

[...] since it breaks often [...]

What breaks often? The coastlines in OSM or the --keep-coastlines option? (I know about the former, but your phrasing makes it sound like the latter breaks.)

The coastlines in OSM break often.
IMO, this has been decided as "do not include coastlines directly" through issues geometalab/conversion-service#97 and geometalab/conversion-service#99


JsonIgnoreCondition.WhenWritingNull is not work as my expect when Serialize a JsonDocument

Description
I Serialize a JsonDocument with setting JsonIgnoreCondition.WhenWritingNull ，but it not work,
SDK: net 6
OS: MacOS
Reproduction Steps
This is my Demo Code
// See https://aka.ms/new-console-template for more information

using System.Text.Json;
using System.Text.Json.Serialization;

Person person = new Person();

var personJsonStr = System.Text.Json.JsonSerializer.Serialize(person);

Console.WriteLine(personJsonStr);

var jsonDocument = System.Text.Json.JsonSerializer.Deserialize<JsonDocument>(personJsonStr);

JsonSerializerOptions options = new JsonSerializerOptions()
{
    DefaultIgnoreCondition = JsonIgnoreCondition.WhenWritingNull
};

var jsonStr = JsonSerializer.Serialize(jsonDocument, options);

Console.WriteLine(jsonStr);

public class Person
{
    public int Age { get; set; }

    public Address Address { get; set; }
}

public class Address
{
    public string Location { get; set; }
}

Expected behavior
jsonStr does not contain null value,
Actual behavior
jsonStr =>  "{"Age":0,"Address":null}"
Regression?
No response
Known Workarounds
No response
Configuration
No response
Other information
No response

Related to https://github.com/dotnet/runtime/issues/54184

ok，I understood

Re-opening so we can look into writing docs describing this behavior.

@layomia

If JsonSerializer features are needed, please use a strongly typed target for serialization and deserialization.

To be honest, I don't know why I'm having trouble figuring this out, but for my sake, could you please elaborate or provide a one-line example?

@reinux
They're saying that you should use the type and not JsonDocument when you deserialize.
Instead of:
var jsonDocument = JsonSerializer.Deserialize<JsonDocument>(str);

Do:
var person = JsonSerializer.Deserialize<Person>(str);

For cases where you don't have the type at runtime
I haven't tested this but pretty sure if you pass the options to the Serialize method it will work:
var options =  new JsonSerializerOptions
{
    DefaultIgnoreCondition = JsonIgnoreCondition.WhenWritingNull
};
var jsonDocument = JsonSerializer.Deserialize<JsonDocument>(new object(), options);

Ahh got it, thanks!


In 2.5.1 you can disable queries to 4bytes by adding this to your hardhat config
{
  tracer: {
    use4bytesDirectory: false
  }
}

thanks,bro


Sidebar sometimes disappears and gets stuck in faulty state

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Repro at https://github.com/KATT/mailing-run-issue
Steps to reproduce the behavior:

 git clone git@github.com:KATT/mailing-run-issue.git
 cd mailing-run-issue
 yarn
 npx mailing

Click Welcome -> Preview email
Reload page

Expected behavior
Should look the same.
Actual behavior

Sidebar disappears
Gets stuck in faulty state even if you clear the URL

Workaround: clear the URL except for the /preview part
Video
https://user-images.githubusercontent.com/459267/230070695-4a94812d-43da-43bb-bada-e2d7a471d5dc.mp4
Additional context:
Might be because of the space here: https://github.com/KATT/mailing-run-issue/blob/aa8fb7744a0c8e13a28198355906c9aae636dbcb/emails/previews/Welcome.tsx#L4

@psugihara This issue can be closed. Fix was merged here: https://github.com/sofn-xyz/mailing/pull/482


*: switch to using turf for gometry bounds

Fixes https://github.com/NWACus/avy/issues/724

Thanks @kevindice


ImageData constructor has incorrect parameter (not optional where it should be)

TypeScript Version:  3.6.2

Search Terms:
ImageData
Code

const data: ImageData = new ImageData(screenBuffer, CANVAS_WIDTH);

Argument of type 'Uint8ClampedArray' is not assignable to parameter of type 'number'.  TS2345

    365 |
    366 |         }
  > 367 |         const data: ImageData = new ImageData(screenBuffer, CANVAS_WIDTH);
        |

also, simplified:
const screenBufferSize = 4;

const screenBuffer: Uint8ClampedArray = new Uint8ClampedArray(screenBufferSize);

const data: ImageData = new ImageData(screenBuffer, 1);

Expected behavior:
MDN says that the height parameter is optional if the first parameter is a Uint8ClampedArray. Typescript should choose the new(array: Uint8ClampedArray, width: number, height: number): ImageData; overload, but the third param should be optional.
Actual behavior:
Typescript is choosing the new(width: number, height: number): ImageData; overload.
Playground Link:
Where's the share button?
Related Issues:

Issue #5338
Issue #4220
Issue #949

Where's the share button?

The URL bar automatically updates as you type code, with the new Playground.

Note: This has never worked.


Fix broken Crown Copyright link

The link to the National Archives page describing Crown Copyright is broken on the readme. Update it to point here:
https://www.nationalarchives.gov.uk/information-management/re-using-public-sector-information/uk-government-licensing-framework/crown-copyright/

Hello @t11947 , I'd like to take this up. Kindly assign it to me.

Hello @t11947 , I'd like to take this up. Kindly assign it to me.

Sorry we never got back to you, would you still like to be assigned?

Yeah, I'm still up for it. Thanks


feat: add a generic start API

Description
Could be merged roughly as is, but resource detection should be shared between the signals.
I currently documented start to enable all stable signals. That means stabilizing signals is a breaking change. Are we OK with that?. Alternative would be to:

Enable nothing by default or
Only ever enable tracing by default - similar to right now, but more explicit.

I would not want to add another option to toggle that behavior.
Type of change
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Internal change (a change which is not visible to the package consumers)
[x] Documentation change or requires a documentation update

How Has This Been Tested?

[ ] Tested manually
[ ] Added automated tests

Checklist:

[x] Unit tests have been added/updated
[x] Documentation has been updated
[x] Change file has been generated (npm run change:new)
[ ] Delete this branch (after the PR is merged)

Codecov Report

Merging #474 (fee5fa4) into main (d8a13ba) will increase coverage by 0.10%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #474      +/-   ##
==========================================
+ Coverage   76.33%   76.43%   +0.10%
==========================================
  Files          21       21
  Lines         676      679       +3
  Branches      157      157
==========================================
+ Hits          516      519       +3
  Misses        160      160

Impacted Files
Coverage Δ

src/metrics/index.ts
92.22% <ø> (ø)

src/instrument.ts
77.77% <100.00%> (ø)

src/profiling/index.ts
19.11% <100.00%> (+2.45%)
:arrow_up:

src/tracing.ts
98.00% <100.00%> (+0.04%)
:arrow_up:

Continue to review full report at Codecov.

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

@seemk, what about the question in the OP :point_up:


about_zulip: Indicate the copy icon to be clickable.

Added a minor commit which indicates the about_zulip copy icon to be a clickable element by setting a CSS property of cursor having value pointer.
Screenshot

Merged after changing the selector to i.fa-copy to be more specific, thanks @aryanshridhar!


Handover partner field issue

[main]

test code
nta/models
nta/api

Coverage decreased (-0.001%) to 99.299% when pulling 41b4721068555de48092d416942e3021e5309000 on handover_partner_field_issue into 56a6d0b03d1b0ab6ea73554431892dac4394c07d on develop.

0.001% coverage decreasing is meaningless. Decide to merge branch


Remove stretch from the distribution as it is no longer supported.

Signed-off-by: Chris Lalancette<EMAIL_ADDRESS>

This needs to go along with https://github.com/ros-infrastructure/ros_buildfarm_config/pull/188


Minima @ 05/06/2021 22:49:48 [146.3 MB] : NEW BALANCE : [{"tokenid":"0x00","token":"Minima","total":"1000000000","decimals":"44","confirmed":"1826.5","unconfirmed":"0","mempool":"0","sendable":"1826.5"},{"tokenid":"0x68C4A8A40B365FFA54AA5F0E2D5920D371E58F54DE6FDC54C364EC3ABCD8CCB0E58F17A9DDB905F1C592F4D00BDA1C0AF488A4716F7B6499CC7F3F123636F203","token":"Bloomberg","description":"","icon":"https://i.postimg.cc/mrR6FMdT/Bloomberg-Terminal-Review.jpg","proof":"","total":"100","decimals":"0","script":"RETURN TRUE","coinid":"0x227A3EACCDC5C550AA5137E78AD820F1E91E482E5E6B06BBD1BD9F4C4B9C30C2F02BACC2CBE0D4B40236E0DDB3643E3F9FB7DE140AF9AAB788FF3EBECD9A8693","totalamount":"0.000000000000000000000000000000000000000001","scale":"44","confirmed":"42","unconfirmed":"0","mempool":"0","sendable":"42"},{"tokenid":"0x21D3088DA0D8B26FBF2E07773CDEADFB45DE1D4F478C9DC135A2DBFB6A6B38C76E26944EFE53E13DC98510EF6B3B0141B3797639CCB068790267B3EA46D09CE1","token":"Cupid complaining to Venus","description":"Lucas Cranach the Elder. ( 1526 )Cupid, the god of erotic love, is complaining to his mother, Venus, the goddess of love: he has been stung by bees after stealing a honeycomb. Venus directs her attention towards the viewer instead.","icon":"https://images2.imgbox.com/89/ea/lcu4dLpD_o.jpg","proof":"http://eurobuddha.com/cupid.txt","total":"1526","decimals":"0","script":"RETURN TRUE","coinid":"0x973167A9ED49BFCFBDB8E4A5A109A3143C9A4CAE00370A01C7373EE9584DDC82261BC141BC747F5F138AA86A8B5EA27EDEE9561DE03026FA556D3F5FD65C98A2","totalamount":"0.00000000000000000000000000000000000000001526","scale":"44","confirmed":"1","unconfirmed":"0","mempool":"0","sendable":"1"},{"tokenid":"0x5F61094947E4BDF46E21DE4DB524A8EE7F40CE89CA6579E7EC5BB72283DC9D2FABECD4D8ADBE19D87E817E07D8524D3864AF05FFD44B71144481DB7DD961CCA6","token":"From Russia with Love Full Movie","description":"From Russia with Love Full Movie Gif. Double Topped","icon":"http://eurobuddha.com/DoubleTopWithLove.gif","proof":"http://eurobuddha.com/rwl.txt","total":"100","decimals":"0","script":"RETURN TRUE","coinid":"0x394299CEF42CD834D55B87DFD58603281B2784229589FEA7230E836CD538919D0DAEE12BFB1C2610C3B2E1E873F76D74AF4D5707DA174C6E8FDC9A773185E6EF","totalamount":"0.000000000000000000000000000000000000000001","scale":"44","confirmed":"1","unconfirmed":"0","mempool":"0","sendable":"1"},{"tokenid":"0xD2248482C19986869332DF4E922E2B4A70AF0482A56DDC003085BA8F60025F2955725181814818EA98A23EE991AA6BE508E7C8791BDBB7B7364E28A618E8AD63","token":"Grandma","description":"Grandma for minima","icon":"http://img-fotki.yandex.ru/get/15546/226582521.9/0_ef1c3_98f1a5e2_orig.png","proof":"","total":"100","decimals":"0","script":"RETURN TRUE","coinid":"0x06F8227C5357AB813C5EDC34DB8315AD2758233F680BFAE2FCB3465D58D7D4F8C728C1883A38930D832EF95DB8A33518287C114160CE6D2BFC3B5A3A63B6FBBD","totalamount":"0.000000000000000000000000000000000000000001","scale":"44","confirmed":"1","unconfirmed":"0","mempool":"0","sendable":"1"},{"tokenid":"0x2D91DF11D8A19F44DD3046A77FC44EB3B705EABEE57F986F9A17EE94363A56858B0901E951076CD31BFA6A5B8DA87300C7566DAF1E7195627F539BF4C61DE47C","token":"Incentive Cash","description":"The Minima Incentive Cash Token is a reward token for participants in the Minima Incentive Program. Tokens will be awarded for a range of designated activities, and will be redeemable at mainnet on a 1:1 valuation with the Minima Coin.","icon":"https://raw.githubusercontent.com/minima-global/incentivecash/main/images/icon.svg","proof":"https://incentive.minima.global/incentiveCash.txt","total":"300000000","decimals":"8","script":"RETURN TRUE","coinid":"0x11BAE84EB1BE88AE6640FE2E656B327A3D338C58E33CB6E927C9898B0C070B258ED2172AD7BDD910E36FAA7F7F50405CF23A94BCEF5E866CC49D1830D87579DB","totalamount":"0.0000000000000000000000000003","scale":"36","confirmed":"28","unconfirmed":"0","mempool":"0","sendable":"0"},{"tokenid":"0xF6C9AF7D1D570CC8271B3A19BC409649BC29148ECB49790B1DD24B60B3FF5A28824DAFFAC511A5D7F4EF80BA49BF3FB03FED52681183936916347D03A16C9D94","token":"JazzBomb","description":"will it break the system","icon":"https://images2.imgbox.com/75/d6/9dtg06iZ_o.jpg","proof":"","total":"222","decimals":"8","script":"RETURN TRUE","coinid":"0xF100C9ED839B61480658E7AEC159117757DE2A894E976342A81516B34BBDE20686C59BBEE7F178BFD07598B81501FFABAF0D0B205382BA637F13929035036FBC","totalamount":"0.000000000000000000000000000000000222","scale":"36","confirmed":"190","unconfirmed":"0","mempool":"0","sendable":"190"},{"tokenid":"0x3E5CAD4150D0A4AC1CAD90D1447B251F1FC66B46991F456DBCEBAD7702D18174C78862399E25B49913ADDE8D9AFD358E58207E59005CEFB5897DE22EE9930C01","token":"Magic Carpet","description":"","icon":"https://images2.imgbox.com/46/69/4bYSmPrf_o.gif","proof":"","total":"100","decimals":"0","script":"RETURN TRUE","coinid":"0x1AE28232B1E0CA24BA7165896260275EC2600F59C6BA8028C30FE2C2FD04AD78A13690667521A6BE6D57E0585EDC741EDAC04AAADF1457EF844E4EF64A1C87CF","totalamount":"0.000000000000000000000000000000000000000001","scale":"44","confirmed":"79","unconfirmed":"0","mempool":"0","sendable":"79"},{"tokenid":"0xAD59C94294806B8163354F07719C9F591994F2977B410346E3637984BB126B09CD85E5B397D983C139FAA9C908C2D5E8BBE6FB5A7F2CB7F38F1D919E201071B5","token":"Saint Michael Triumphs over the Devil","description":"Bartolomé Bermejo 1468. A saint clad in shining armour raises his sword to strike a hideous demon beneath his feet. This is the Archangel Michael fighting the devil, as described in the Book of Revelation.","icon":"https://images2.imgbox.com/99/da/5IRxOvKR_o.jpg","proof":"http://eurobuddha.com/devil.txt","total":"1468","decimals":"0","script":"RETURN TRUE","coinid":"0x54E1785123B6653B3D10D6BDCFF32E9553E4AF2C9DF295A5CAB6F7A546C5CB9EA2B2B2C2EB125DBF489FE58922741616A39CF7621884307179BFAC83EAA0C737","totalamount":"0.00000000000000000000000000000000000000001468","scale":"44","confirmed":"1","unconfirmed":"0","mempool":"0","sendable":"1"},{"tokenid":"0xC5CAE12E0B56680ED21556F8CBDF9A628E3596BADC390FB829787F317BC3BCBC103E4EEC3C82513B38FE91EF871DEA160EBE26A7F6E37F0B7FDD6D55F348F7FD","token":"Susanna at her Bath","description":"Hayez. The virtuous Susannah bathes in her garden, she is approached by two corrupt elders who, lusting after her, threaten to accuse her of adultery if she does not give into them. She refuses and is falsely accused by them, but her innocence is proved.","icon":"https://images2.imgbox.com/01/92/QMJ7aeqZ_o.jpg","proof":"http://eurobuddha.com/bath.txt","total":"1850","decimals":"0","script":"RETURN TRUE","coinid":"0x8168740B255042DE5E82FC6DF47DAE468459636DAA07AC434253CF6C7EF0971E4C85B5826E884FD259412E467E97D626CEC22A98F9CCF6BAB3FBBA74AFAF56B7","totalamount":"0.0000000000000000000000000000000000000000185","scale":"44","confirmed":"1","unconfirmed":"0","mempool":"0","sendable":"1"},{"tokenid":"0xF8F8EA6D15D6F4BF4F7B3B760A4ABFE79C1BCDAD93E061CD747D8C4D942A1C49F62DB072A171AF669808A124CAB1BF941E23567B8B9553253D81FBFF822188F7","token":"eurobuddha","description":"eurobuddha","icon":"https://images2.imgbox.com/d5/e8/UVLX3CHu_o.gif","proof":"http://eurobuddha.com/eurobuddha.txt","total":"6969","decimals":"0","script":"RETURN TRUE","coinid":"0xCF799A35150DAB4E10B9E328FCF58AD27B19F6D09ACB447271235818C9C72C3AF9B35DB3D98E9C44917B00765DDB1769F7A68458AC1452FFE58697ECE1394E01","totalamount":"0.00000000000000000000000000000000000000006969","scale":"44","confirmed":"94","unconfirmed":"0","mempool":"0","sendable":"94"}]
Minima @ 05/06/2021 22:50:21 [101.9 MB] : ServiceID 0xD1303E501D90579C069475DB04E85A17AF832CB3 > BLOCK: timemilli1622929821966
Minima @ 05/06/2021 22:50:30 [121.3 MB] : ServiceID 0xD1303E501D90579C069475DB04E85A17AF832CB3 > BLOCK: timemilli1622929830607
Minima @ 05/06/2021 22:51:01 [102.1 MB] : ServiceID 0xD1303E501D90579C069475DB04E85A17AF832CB3 > BLOCK: timemilli1622929861019
Minima @ 05/06/2021 22:51:03 [114.9 MB] : ServiceID 0xD1303E501D90579C069475DB04E85A17AF832CB3 > BLOCK: timemilli1622929863140
Minima @ 05/06/2021 22:51:12 [100.4 MB] : ServiceID 0xD1303E501D90579C069475DB04E85A17AF832CB3 > BLOCK: timemilli1622929873011
Minima @ 05/06/2021 22:51:13 [101.1 MB] : ServiceID 0xD1303E501D90579C069475DB04E85A17AF832CB3 > BLOCK: timemilli1622929873061
Minima @ 05/06/2021 22:51:31 [97.9 MB] : ServiceID 0xD1303E501D90579C069475DB04E85A17AF832CB3 > BLOCK: timemilli1622929891053
Minima @ 05/06/2021 22:51:39 [122.1 MB] : ServiceID 0xD1303E501D90579C069475DB04E85A17AF832CB3 > BLOCK: timemilli1622929897966
Minima @ 05/06/2021 22:51:59 [116.2 MB] : ServiceID 0xD1303E501D90579C069475DB04E85A17AF832CB3 > BLOCK: timemilli1622929920002
Minima @ 05/06/2021 22:52:24 [101.5 MB] : ServiceID 0xD1303E501D90579C069475DB04E85A17AF832CB3 > BLOCK: timemilli1622929944278
Minima @ 05/06/2021 22:52:41 [121.1 MB] : ServiceID 0xD1303E501D90579C069475DB04E85A17AF832CB3 > BLOCK: timemilli1622929961384
Minima @ 05/06/2021 22:52:45 [107.3 MB] : ServiceID 0xD1303E501D90579C069475DB04E85A17AF832CB3 > BLOCK: timemilli1622929963960
Minima @ 05/06/2021 22:52:52 [108.2 MB] : ServiceID 0xD1303E501D90579C069475DB04E85A17AF832CB3 > BLOCK: timemilli1622929973092
Minima @ 05/06/2021 22:53:05 [109.0 MB] : ServiceID 0xD1303E501D90579C069475DB04E85A17AF832CB3 > BLOCK: timemilli1622929985281
Minima @ 05/06/2021 22:53:28 [101.6 MB] : ServiceID 0xD1303E501D90579C069475DB04E85A17AF832CB3 > BLOCK: timemilli1622930008124
Minima @ 05/06/2021 22:53:40 [115.4 MB] : ServiceID 0xD1303E501D90579C069475DB04E85A17AF832CB3 > BLOCK: timemilli1622930020908
Minima @ 05/06/2021 22:53:45 [119.6 MB] : ServiceID 0xD1303E501D90579C069475DB04E85A17AF832CB3 > BLOCK: timemilli1622930022643
Minima @ 05/06/2021 22:54:07 [124.5 MB] : ServiceID 0xD1303E501D90579C069475DB04E85A17AF832CB3 > BLOCK: timemilli1622930047372
Minima @ 05/06/2021 22:54:08 [102.2 MB] : ServiceID 0xD1303E501D90579C069475DB04E85A17AF832CB3 > BLOCK: timemilli1622930049160
Minima @ 05/06/2021 22:54:17 [103.8 MB] : ServiceID 0xD1303E501D90579C069475DB04E85A17AF832CB3 > BLOCK: timemilli1622930057450
Minima @ 05/06/2021 22:54:27 [123.9 MB] : ServiceID 0xD1303E501D90579C069475DB04E85A17AF832CB3 > BLOCK: timemilli1622930067811
Minima @ 05/06/2021 22:54:54 [123.3 MB] : TRY DISCONNECT SERVICE
Minima @ 05/06/2021 22:55:01 [124.8 MB] : ServiceID 0xD1303E501D90579C069475DB04E85A17AF832CB3 > BLOCK: timemilli1622930101776
Minima @ 05/06/2021 22:55:04 [103.5 MB] : ServiceID 0xD1303E501D90579C069475DB04E85A17AF832CB3 > BLOCK: timemilli1622930103428

This is going to be fixed in the next hard fork.  Token is invalid.

Hi, I'm getting this issue now as well. Exactly as @MiniJaz described.
Steps to reproduce:
Hard reset your minima node
Generate new address
Get tokens sent to you
Token that results in null: 0x7B73253CCAA879D19B63DDDABE454279E3E79787EECBC250142C247FEB4603F4C234A7527E524264DDB6C2A8D18DE7C35EAF913E61694274BF3FE86D34E9F949
Token that shows up okay:
0x6460D88F8E90D5630D6891CE852B9A099EB0D915B80B5F5249AF36402172096DA95CE2CAE9DEBB0066D150E246401753DA0CD2BBD02C21F64AAB28F2C85EC039


Fix off by factor 1024 in Mesos disk allocation (resolves #854)

resolves #854

Great job. This is a very valuable test and I'm embarassed that we didn't think of it earlier.
I think I want this test in toil/test/batchSystems/batchSystemTest.py. Either in a separate class or preferably in AbstractBatchSystemTest. I think at least SingleMachine and Mesos should support this test. GridEngine, Slurm and Parasol are probably too sluggish for the current sleep time. But you could factor the sleep time out into a class attribute that those three override to like 30-60 seconds.

I see. You already have this for Single Machine and Mesos. I still think integrating it with the other batch system tests is worth a shot.

@hannes-ucsc

Jenkins, test this please

@hannes-ucsc

See #861 for follow up.


feat: add MongoDB support

This will add MongoDB support.
You can test this by changing the following files:
dejavu.cnf.SAMPLE
{
    "database": {
        "database": "dejavu"
    },
    "database_type": "mongodb"
}

docker-compose.yaml
version: '3'
services:
  db:
    build:
      context: ./docker/postgres
    environment:
      - POSTGRES_DB=dejavu
      - POSTGRES_USER=postgres
      - POSTGRES_PASSWORD=password
    networks:
      - db_network
  mongodb:
    image: mongo
    ports:
        - "27020:27017"
    networks:
      - db_network
  python:
    build:
      context: ./docker/python
    volumes:
      - .:/code
    depends_on:
      - db
    networks:
      - db_network
networks:
  db_network:

Then you can run docker-compose up -d
and then run docker-compose run python /bin/bash
You'll also need to make something like example_docker_postgres.py to start the fingerprinting process and populate the database.
Then you can use python dejavu.py -r file ... to recognize a file.

@worldveil please check this out.

Hi @ATheCoder, what does the performance look like on MongoDB compared to either Postgre or Mysql?

Hi @ATheCoder, what does the performance look like on MongoDB compared to either Postgre or Mysql?

+1, Has anyone tested the performance?


remove required config.yaml file for the check/print of rules

fix #452

/check-dco

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: Issif
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [Issif]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment