Avoid frozen build when git or .git/+hg are unavailable

This came up in IRC earlier. An unofficial package maintainer for Archlinux had his/her build freezing at ocaml -I scripts -w -3 unix.cma scripts/gen_build_id.ml _build/hack/utils/get_build_id.gen.c. The build is from a tarball, so the .git/ directory is unavailable.
The user didn't have Mercurial installed. After the git case under gen_build_id.ml failed, a read under the hg case from stdout/stderr would not stop blocking. It looks like create_process doesn't recognize exit 127 when hg isn't found, so an input_line call doesn't break. This PR substitutes the open_process_full function which recognizes that the command doesn't exist and interrupts the input_line call. I suspect that the same problem arises in environments without git installed regardless of whether hg is installed.

one constraint here is that it needs to work on Windows. any idea if Unix.open_process_full works there?
cc @gabelevi

No freakin' idea. The Unix module is heavily annotated with whether each member works on Windows, and there's no exception listed from Windows, so I assumed. I don't have a Windows box to test on--on Linux I tested permutations:
(.git/ exists | .git/ doesn't exist) X (hg exists | hg doesn't exist)
(I didn't check (git exists | git doesn't exist), which I probably should have).

@gabelevi can you patch this and find out?

One option that should be Windows-safe is to look for the .git and .hg folders before calling those commands. I'd definitely merge that without actually testing on Windows.

Kinda--I think it would need to check existence of the executables instead of the directories.
From the ocaml sources, Windows would need a COMSPEC environment variable.  That still may be insufficient though.
Let me try polling for program existence without any input reads as a preliminary step. If that doesn't work, I'll just use Sys.unix. When I get around to it, that is....

i think this works around it: 72759d80a884a6d870cd98792952cb7b0ba66dbb
thanks for figuring out the cause!


add .gitignore

@Liyuhui-12 Hi! Thanks for your great work!
Could please you review this PR? It's helpful to have a .gitignore.
Btw, do you have any plan on refactoring the current code base? I'm trying to integrate EAGLE into vLLM and I'd love to contribute to this project.

Thanks a lot for your PR and for showing interest in our project! We're definitely working on improving the code and really appreciate your offer to help, especially with integrating EAGLE into vLLM - that sounds like a great idea.
We're excited to see what you come up with and happy to have you on board!


Incorrect evaluation in directories with spaces in name

Describe the bug
I am new to this software and am testing it to see if it will work for me (I need to install Python libraries unavailable in nixpkgs and I understand it is better practice to use virtual environments anyway). I was going absolutely insane trying to work out what was wrong for a while, but I managed to figure it out after removing some red herrings that the error trace incorrectly returned as having broken.
As the title implies, running devenv shell does not work correctly in a directory with spaces in its name, but works fine in one without.
To reproduce
Run devenv init
Use this simple devenv.nix:
#devenv.nix
{
  pkgs,
  lib,
  config,
  inputs,
  ...
}:

{
  languages.python = {
    enable = true;
    venv.enable = true;
    venv.requirements = lib.readFile ./requirement.txt;
  };

  enterShell = ''
    python -c "import control as ct" && echo "No errors!"
  '';
  cachix.enable = false;
}

Use this requirement.txt:
#requirement.txt
control==0.10.0

Then run devenv shell.
In a folder with no spaces in the name, it will run normally and return
No errors!
But in one with spaces in its name, it will return with an error (in this example, the folder is /home/ejb/Documents/Uni/Masters/Control and Estimation Theory, and each word is erroneously evaluated as a function):
• Building shell ...
error: builder for '/nix/store/mk1x85rpx2fi2rkvbp1ixcnv62kn4zfp-python3-3.11.8-env.drv' failed with exit code 1;
       last 5 log lines:
       > created 4 symlinks in user environment
       > <stdin>: In function 'main':
       > <stdin>:56:6: error: #error makeCWrapper: Unknown argument and
       > <stdin>:57:6: error: #error makeCWrapper: Unknown argument Estimation
       > <stdin>:58:6: error: #error makeCWrapper: Unknown argument Theory/Lab2/.devenv/profile/lib:/nix/store/7v7c61ikm7cbd3kbimk9qr8h7pdmyccn-manylinux2014/lib:/nix/store/c2yb135iv4maadia5f760b3xhbh6jh61-gcc-13.2.0-lib/lib
       For full logs, run 'nix log /nix/store/mk1x85rpx2fi2rkvbp1ixcnv62kn4zfp-python3-3.11.8-env.drv'.
error: 1 dependencies of derivation '/nix/store/427kbgfs1ivm08227jbjhn1y8xm1yx6r-devenv-profile.drv' failed to build
error: 1 dependencies of derivation '/nix/store/zlnikh8fg8jrindcm1y45cw15jrh54mz-init-venv.sh.drv' failed to build
error:
       error: 3 dependencies of derivation '/nix/store/2vjcng7fzpnskmi7c5wsf6bakr926cq6-devenv-shell-env.drv' failed to build

✖ Command produced the following output:

✔ Building shell in 1.4s.
Error:   × Command `/nix/store/w6idni5fc3692nbpl3dnyq1i50ai3b7r-nix-2.21-devenv/bin/nix --show-trace --extra-experimental-features nix-command --extra-
  │ experimental-features flakes --option warn-dirty false --option eval-cache false --keep-going --max-jobs 10 print-dev-env --profile /home/
  │ ejb/Documents/Uni/Masters/Control and Estimation Theory/Lab2/.devenv/gc/shell` failed with with exit code 1

Version
Output of devenv version:
devenv 1.0.7 (x86_64-linux)

Duplicate of https://github.com/cachix/devenv/issues/1107.

Why is this issue closed as completed when the other issue is stale and abandoned? Is there a timeline for getting this problem fixed, or should I find some other software like how the other person had to?

We already have a tracking issue for this.
It's not stale or abandoned. We don't have an immediate fix for this because the issue is with Nix, not devenv.


Problem starting this in docker

When starting this on amazon docker the npm start fails.
The exact error is as follows
><EMAIL_ADDRESS>start /app
> node ./bin/www

/usr/local/bin/node[16]: ../src/pipe_wrap.cc:168:static void node::PipeWrap::Connect(const v8::FunctionCallbackInfo<v8::Value>&): Assertion `args[1]->IsString()' failed.
 1: node::Abort() [node]
 2: node::Assert(char const* const (*) [4]) [node]
 3: node::PipeWrap::Connect(v8::FunctionCallbackInfo<v8::Value> const&) [node]
 4: v8::internal::FunctionCallbackArguments::Call(void (*)(v8::FunctionCallbackInfo<v8::Value> const&)) [node]
 5: 0xb46eac [node]
 6: v8::internal::Builtin_HandleApiCall(int, v8::internal::Object**, v8::internal::Isolate*) [node]
 7: 0xbd3e6f043a7
Aborted

npm info lifecycle<EMAIL_ADDRESS>Failed to exec start script
npm ERR! Linux 4.4.41-36.55.amzn1.x86_64
npm ERR! argv "/usr/local/bin/node" "/usr/local/bin/npm" "start"
npm ERR! node v7.7.3
npm ERR! npm  v4.1.2
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>start: `node ./bin/www`
npm ERR! Exit status 134
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>start script 'node ./bin/www'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the x.x.x package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     node ./bin/www
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs x.x.x
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls x.x.x
npm ERR! There is likely additional logging output above.

npm ERR! Please include the following file with any support request:
npm ERR!     /app/npm-debug.log`

As you see it's throwing a c++ error of node itself.
Node used is 7.7.3
Npm is 4.1.2
Latest express
This happens on npm start.
package.json looks like this:
'"dependencies": {
    "aws-sdk": "^2.40.0",
    "body-parser": "~1.13.2",
    "cookie-parser": "~1.3.5",
    "cors": "^2.8.3",
    "debug": "~2.2.0",
    "ejs": "~2.5.6",
    "express": "~4.13.1",
    "express-jwt": "^5.1.0",
    "jsonwebtoken": "^7.3.0",
    "morgan": "~1.6.1",
    "multer": "^1.3.0",
    "multer-s3": "^2.7.0",
    "redis": "^2.7.1",
    "serve-favicon": "~2.3.0",
    "validator": "^7.0.0"
  },'

It does fail on start. It's passing env variables which i do have a few, through enviroment variables on aws. I'm getting the enviroment variables inside the app with process.env.ENV_VARIABLE.
For some reason it breaks and i can't figure out as it's only giving me that error above, nothing more.
Please help.

Hi @danncoba defiantly very weird. Not sure what would case the Node.js runtime to assert like that in Express. Is it possible you can provide the code that causes the issue so I can poke at it?

Closing due to non-response.


Add support for the resolve option

Hi,
recent versions of css-modules-require-hook changed  the resolving mechanism on the same semver range, and as a consequence most of our unit test broke...
Apparently the resolve option was introduced to be able to address this, but is not yet supported/passed by this plugin ?

Hello, sorry for late response, I don't much free time these days. Maybe I will get some free time on weekend but no promises. PR is welcome :)


[Bug]: Placeholder displying without the proper error message

Preflight Checklist

[X] I could not find a solution in the documentation, the existing issues or discussions
[ ] I have joined the ZITADEL chat

Environment
ZITADEL Cloud
Describe the bug
When the added Company name is not unique, the error message only displaying the placeholder as form.organizationName.notUnique. Please refer the below image.

To reproduce
Go to sign up and add the company name with a existing one
Screenshots
No response
Expected behavior
Proper error message
Version
No response
Operating System
No response
Relevant Configuration
No response
Additional Context
No response

Thanks for your report. I already fixed it, and it will be deployed to production in the next time.


Telegram support

Before merging we need to create this entry in the wiki page: https://github.com/agonyforge/arbitrader/pull/341/files#diff-9ed7902517cecde48e205a8428a5e450aa6485ba19e23a6779edf26a2febefc7R187

Before merging we need to create this entry in the wiki page: https://github.com/agonyforge/arbitrader/pull/341/files#diff-9ed7902517cecde48e205a8428a5e450aa6485ba19e23a6779edf26a2febefc7R187

I think you guys have access to edit the wiki now. If not, let me know and I can do it.


Fixed crash when glyph cannot be loaded (#15)

Fixes #15.

Thank you!
I still try to get it to work as it should and if nothing works, then it will be a reasonable fix.


[Marker.io] Position of nav arrows is not correct in showrooms section

Steps to reproduce:

go to showrooms section
notice nav arrow icons
compare with figma

Actual result: nav arrow icons are not at right place.
Expected result: they should be between the 2 boxes. In the gap present between 2 boxes

Reported by: Vinod<EMAIL_ADDRESS>- Contact via Marker.io
Source URL: https://proav-stg.webflow.io/
Issue details: Open in Marker.io
Device typemobileBrowserSafari 17.6Screen Size375 x 812OSiOS 17.6.1Viewport Size375 x 710Pixel Ratio@3x

🤖 Marker.io bot - This issue was marked as ✅ Resolved in Marker.io


specify license for toki

Hello @taylorchu
First of all thank you for making this project. It's been one of our dependencies for carbon-relay-ng for quite some time and we've never had any issues with it!
We just found out that this project has no license file, and by default, unlicensed code is considered "all rights reserved" and not open source. We were wondering if you'd mind adding a license you consider appropriate for it so that we can continue to use the project. If that's not something you want to do, we can find an alternative solution. Please let us know when you have a moment and thank you again.

Hello @taylorchu! Just checking in to see if you had a chance to look at this

I will upload the MIT license today

Thank you, much appreciated!


Parsing failure of JSON with reserved keywords

Initial situation
There is a people counter sensor, that sends data via MQTT on C8Y. The MQTT looks like:
{
    "device_info": {
        "device_mac": "24:E1:24:F9:28:00",
        "device_name": "People Counter",
        "device_sn": "6757D50072250018",
        "firmware_version": "V_[<IP_ADDRESS>](https://<IP_ADDRESS>/)",
        "hardware_version": "V1.3",
        "ip_address": "[<IP_ADDRESS>](https://<IP_ADDRESS>/)",
        "running_time": 169576
    },
    "line_periodic_data": [
        {
            "children_in": 0,
            "children_out": 0,
            "empty_cart_in": 0,
            "empty_cart_out": 0,
            "full_cart_in": 0,
            "full_cart_out": 0,
            "group_in": 0,
            "group_out": 0,
            "in": 122,
            "line": 1,
            "line_name": "Line1",
            "line_uuid": "fd077b55-8717-449d-85b2-986d2e9deba1",
            "no_full_cart_in": 0,
            "no_full_cart_out": 0,
            "out": 0,
            "staff_in": 0,
            "staff_out": 0
        }
    ],
    "line_total_data": [
        {
            "capacity_counted": -163,
            "children_in_counted": 0,
            "children_out_counted": 29,
            "empty_cart_in_counted": 0,
            "empty_cart_out_counted": 0,
            "full_cart_in_counted": 1,
            "full_cart_out_counted": 2,
            "group_in_counted": 10,
            "group_out_counted": 173,
            "in_counted": 10,
            "line": 1,
            "line_name": "Line1",
            "line_uuid": "fd077b55-8717-449d-85b2-986d2e9deba1",
            "no_full_cart_in_counted": 0,
            "no_full_cart_out_counted": 0,
            "out_counted": 173,
            "staff_in_counted": 0,
            "staff_out_counted": 0
        }
    ],
    "time_info": {
        "dst_status": true,
        "enable_dst": true,
        "end_time": "2024-08-22T14:56:00+02:00",
        "start_time": "2024-08-22T14:55:00+02:00",
        "time_zone": "UTC+1:00 Central European Time (CET)"
    }
}

pay attention to the field with the nameìn.
The dynamic-mapper version is 4.4.0 (latest)
Error
While mapping of the JSON, the error occurs if we try to map the fild ìn`:
at com.api.jsonata4java.expressions.Expressions.parse(expressions.java:113)
 at dynamic.mapping.processor.inbound.JSONProcessorInbound.extractFromSource(jsonprocessorinbound.java:108)
 at dynamic.mapping.processor.inbound.AsynchronousDispatcherInbound$MappingInboundTask.lambda$call$1(asynchronousdispatcherinbound.java:203)
 at java.base/java.util.ArrayList.forEach(ArrayList.java:1541)
 at dynamic.mapping.processor.inbound.AsynchronousDispatcherInbound$[MappingInboundTask.call](https://mappinginboundtask.call/)(asynchronousdispatcherinbound.java:137)
 at dynamic.mapping.processor.inbound.AsynchronousDispatcherInbound$[MappingInboundTask.call](https://mappinginboundtask.call/)(asynchronousdispatcherinbound.java:100)
 at java.base/java.util.concurrent.FutureTask.run(FutureTask.java:264)
 at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1128)
 at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:628)
 at java.base/java.lang.Thread.run(Thread.java:829)

2024-08-22 13:14:04.173  WARN 12 --- [ol-1-thread-202] d.m.p.inbound.JSONProcessorInbound       : Tenant <tenant> - Substitution line_periodic_data[0].in not in message payload. Check your mapping counter/+
2024-08-22 13:14:04.197  INFO 12 --- [ol-1-thread-202] dynamic.mapping.core.C8YAgent            : Tenant <tenant> - New measurement posted: {"self":"<url>","time":"2024-08-22T13:14:04.173Z","source":{"self":"<url>","id":"<id>"},"id":"<id>","type":"c8y_PeopleCounter","c8y_PeopleCounter":{"children_in":{"unit":"","value":0},"children_out":{"unit":"","value":0},"adults_out":{"unit":"","value":0},"group_in":{"unit":"","value":0},"group_out":{"unit":"","value":0}}}

Fix
On the MQTT Broker (EMQX) we fixed it renaming of the field from ìntopeople_in`. The rule engine of EMQX was utilized:
FOREACH
    jq('.line_periodic_data[] |= (.people_in = .in | del(.in))', payload) as modified_payload
DO
    modified_payload.device_info,
    modified_payload.line_periodic_data,
    modified_payload.line_total_data,
    modified_payload.time_info

This looks like an issue with the JSONata4Java library.
I will open a ticket in this project.

Raise issue in the JSONata4Java  project.

We documented this limitation.


Add a movies recommender

Hey, I wanted to add a movies recommender script I have, it gives a list of similar movies based on what the user inputs.
Kindly assign this issue to me

Sure, you can work on this. Please follow the proper folder structure while making your PRs :)

Issue resolved with #21


[syncthing] feat: bump syncthing version and add maintainer

Description of the change
Bump syncthing to v1.17.0
Benefits
Update to current release version.
Possible drawbacks
None.
Applicable issues

N/A
Additional information

Checklist

[x] Chart version bumped in Chart.yaml according to semver.
[x] Title of the PR starts with chart name (e.g. [home-assistant])
[x] Variables are documented in the README.md (this can be done with using our helm-docs wrapper ./hack/gen-helm-docs.sh stable <chart>)

@bjw-s fixed the versioning! Based the PR on the last update which was more significant due to the common chart version change.
Let me know if there are additional changes that you see needing to be made!


Update aad scubagear code to align to revised baseline

🗣 Description
Rearrange, add, & delete aad policy code to match updated baseline. Rearrange, add, & delete aad unit tests to match aad code.
💭 Motivation and context
Necessary to match new aad baseline policies & format (policy ID ex "2.1" -> "MS.AAD.1.1v1")
Closing #395
🧪 Testing

Unit Tests

✅ Pre-approval checklist

[x] This PR has an informative and human-readable title.
[x] Changes are limited to a single goal - eschew scope creep!
[x] All future TODOs are captured in issues, which are referenced
in code comments.
[x] All relevant type-of-change labels have been added.
[x] I have read the CONTRIBUTING document.
[x] These code changes follow cisagov code standards.
[x] All relevant repo and/or project documentation has been updated
to reflect the changes in this PR.
[x] Tests have been added and/or modified to cover the changes in this PR.
[x] All new and existing tests pass.

✅ Pre-merge checklist

[ ] Revert dependencies to default branches.
[ ] Finalize version.

✅ Post-merge checklist

[ ] Create a release.

I tested the tool on my laptop and a virtual machine against the G5/E5 tenants and I'm consistently getting the error below:

This is a problem that occurred when we merged in the baseline changes because of the change in the implementation format. See #405


rpm package git-lfs-1.1.0 source code.

Hello,
I am trying to rpm package git-lfs-1.1.0 using a custom SPEC file .  I get the following error during the build. I don’t know anything about go. Could you please help ?
Executing(%build): /bin/sh -e /var/tmp/rpm-tmp.F57yah

umask 022
cd /home/rpmuser/rpmbuild/BUILD
cd git-lfs-1.1.0
GOARCH=amd64
++ pwd
GOPATH=/home/rpmuser/rpmbuild/BUILD/git-lfs-1.1.0
./script/bootstrap
Using go1.4.2

_/home/rpmuser/rpmbuild/BUILD/git-lfs-1.1.0
./git-lfs.go:23: cannot use lfs.ClearTempObjects (type func() error) as type func() in argument to once.Do
./git-lfs.go:36: cannot use lfs.ClearTempObjects (type func() error) as type func() in argument to once.Do
2016/02/24 09:10:31 exit status 2
exit status 1
error: Bad exit status from /var/tmp/rpm-tmp.F57yah (%build)
Thanks

What version of Go are you compiling with? It's possible it may require Go 1.4 or even 1.5.

Using go1.4.2

Try with Go 1.5, it is the documented minimum.


Added Poppins font to the editor and added scroll to the font selection menu

Added poppins font to the editor and added scroll to the font selection menu.
Before

(There was no poppins font and the font selection menu was way bigger)
After

(There is poppins font now and font selection menu is small and compact)
This PR will close #194

@smaranjitghose Please have a look at this new PR. It will close #194


changing personId to roomId

Proposed changes:

changed personId to roomId on input channel since all of the output channel functions use roomId

Status (please check what you already did):

[ ] added some tests for the functionality
[ ] updated the documentation
[ ] updated the changelog (please check changelog for instructions)
[ ] reformat files using black (please check Readme for instructions)

 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.You have signed the CLA already but the status is still pending? Let us recheck it.

Thanks for submitting a pull request 🚀 @TyDunn will take a look at it as soon as possible ✨

Very much needed fix, thanks!
Although what I fail to understand is why the latest package - 1.10.2 - which released few days back doesn't have this fix/code present.
Please let me know what I may be missing, thanks!

@isaac-philip This was merged into master which means it will be part of the next minor (1.11.0). Would you want to quickly backport the changes? You'd need to branch of 1.10.x, apply the changes from the PR, and then create a PR against 1.10.x.

Thanks @wochinge for the suggestion.
I guess it would make sense to backport it since 1.1.0 is way far off.
Let me get to it.
Have a great day!


The penultimate character is the symbol and the last word is not displayed

I use textPainter to measure and draw it on the canvas，So I'm sure there will be a missing character in this case

Hi @leetomlee123, Thanks for filing the issue, please share a minimal and complete reproducible code along with the output of flutter doctor -v with expected and actual behavior.
Thanks.


Pipeline variable substitution

Is there an option to substitute the pipeline variables into the service configuration file?

I am not sure I understand the question. Can you please elaborate?

Closed by mistake, sorry about that!

It's about the variables you define here:

In a IIS web App deploy for example, those variables are substituted into the web.config file:

I like to do the same for a windows service and substitute those variables on deploy into the app.config file

I see, that is a msdeploy.exe feature that is wrapped by the IIS web app deploy. Since Windows Service Manager does not use msdeploy.exe it isn't possible to do a xml variable substitution. However there is an extension called replace tokens in the market place for replacing tokens. It's what I use to transform my configurations are release time.

Closing this issue as resolved, if you are still having issue please let me know.


scripts/hydra start-genesis not working in ubuntu

on running this scripts/hydra start-genesis from https://docs.constellationnetwork.io/sdk/guides/quick-start
its not working in linux
FAILED - RETRYING: [localhost]: (Global L0) Check if GL0 is Ready (5 retries left).
FAILED - RETRYING: [localhost]: (Global L0) Check if GL0 is Ready (4 retries left).
FAILED - RETRYING: [localhost]: (Global L0) Check if GL0 is Ready (3 retries left).
FAILED - RETRYING: [localhost]: (Global L0) Check if GL0 is Ready (2 retries left).
FAILED - RETRYING: [localhost]: (Global L0) Check if GL0 is Ready (1 retries left).
fatal: [localhost]: FAILED! => {"attempts": 120, "changed": false, "content": "", "elapsed": 1, "msg": "Status code was -1 and not [200]: Connection failure: [Errno 104] Connection reset by peer", "redirected": false, "status": -1, "url": "http://localhost:9000/node/info"}
every dependencies is stalled and updated accordingly

Problem resolved, just switch off your device and then try again.


Setting TLVs in Proxy Protocol V2 header

Title: Setting TLVs in Proxy Protocol V2 header
Description:
Proxy Protocol V2 allows application specific TLVs in the header (https://www.haproxy.org/download/2.1/doc/proxy-protocol.txt). Currently,  the listener filter for proxy protocol has mechanism to read those TLVs but the upstream transport socket for PPV2 does not allow setting your own TLVs.
It would be helpful because proxy protocol can then carry addition information such as instance id, cluster name etc.

No one is working on this that I know of.

Is anyone working on this? This has come up for us, and it seems fairly low-hanging for cluster name (not so easy for request metadata).

HAproxy can generate unique ID TLV, it would be great if Envoy can do that too.

btw, is it possible to manipulate TLVs in Lua?

We have some downstream needs to be able to do this and I've been hacking on a branch these last couple of weeks that allows you to inject custom TLVs in the upstream PP2 transport socket. I'll plan on opening a PR early this week for that branch.


For dropdown, the validation for choices is too strict if id-fn and label-fn are provided

Currently, a single-dropdown requires the choices to be a vector-of-maps. But if a user specifies the id-fn and label-fn, then strictly speaking, each entry in that vector doesn't need to be a map.

This just hit me as well. I was hoping to be able to do a DataScript query and pass a vector of vectors from it as options, with first and second as id-fn and label-fn respectively, but got no love. Not a huge deal to have to munge into maps, but would be nice not to have to :-)


Add FireFox as second browser

add firefox into the main script for bootstrappping
add fiefox to 1.7.sh

fixed


<FIRAnalytics/ERROR> Data loss

I've configured my app according to the getting started guide, but I'm having some errors when using Firebase Analytics. When I log an Event, I get the following error:
<FIRAnalytics/ERROR> Data loss. Unable to update daily event counts. Name, error: screen, Error Domain=com.google.gmp.measurement.ErrorDomain Code=3 "Database operation "execute statement" failed with sqlite code 8 and error message "attempt to write a readonly database"" UserInfo={FIRAMonitorContext=execute statement,8,,, NSLocalizedDescription=Database operation "execute statement" failed with sqlite code 8 and error message "attempt to write a readonly database"}
<FIRAnalytics/ERROR> Transaction failed

Anyone know what the issue could be?

As stated in the error message "attempt to write a readonly database", your device may not allow writing to files. Can you check whether your device is set to read-only or whether it has run out of disk space?

How does my device not allow writing to files?? This is on iPhone, and the device is not out of disk space. In fact, the problem happens on multiple devices, so I'm thinking it's either a problem with the library itself, or how the project is setup, but I followed the setup instructions exactly.

It was a guess since we could not reproduce the errors. Sometimes some devices may not allow writing to files for some reasons. It is interesting that you brought up the information that the problem happens on multiple devices. May I ask for more information such as OS versions, device types, and SDK version? Is it 100% reproducible?
Can you please turn on debug mode by using the argument flag -FIRAnalyticsDebugEnabled (for more information, go here) and give us more log messages to debug? There can be something wrong before that leads to such problem.

I also just had this happen but wasn't able to reproduce it.  I think Xcode had the app in a weird state (after I quit it, another instance of my app launched).  Not sure this is a firebase issue.

same problem described here: https://github.com/firebase/quickstart-ios/issues/34

Agree with the dupe, please track in #34


ImageProvider

import 'dart:ui' as ui;
static Future<ui.Image> loadImageByProvider(ImageProvider provider, BuildContext context) async {
Completer<ui.Image> completer = Completer<ui.Image>(); //完成的回调
late ImageStreamListener listener;
ImageStream stream = provider.resolve(createLocalImageConfiguration(context)); //获取图片流
print("获取图片流");
listener = ImageStreamListener((ImageInfo frame, bool sync) {
  //监听
  final ui.Image image = frame.image;
  print("获取ImageInfo");
  completer.complete(image); //完成
  stream.removeListener(listener); //移除监听
});
stream.addListener(listener); //添加监听
print("获取添加监听");
return completer.future; //返回

}
dubug

flutter run --release

release environment ImageStreamListener  No  callback

@mkui233
When opening an issue, please try to provide all or some of the information indicated in the template so that we can address this issue properly.

Issue Template
If you are looking for support, please check out our documentation
or consider asking a question on Stack Overflow:
- https://flutter.dev/
- https://api.flutter.dev/
- https://stackoverflow.com/questions/tagged/flutter?sort=frequent

If you have found a bug or if our documentation doesn't have an answer
to what you're looking for, then fill our the template below. Please read
our guide to filing a bug first: https://flutter.dev/docs/resources/bug-reports

Steps to Reproduce

Please tell us exactly how to reproduce the problem you are running into.

Please attach a small application (ideally just one main.dart file) that
reproduces the problem. You could use https://gist.github.com/ for this.

If the problem is with your application's rendering, then please attach
a screenshot and explain what the problem is.

1. ...
2. ...
3. ...

Please tell us which target platform(s) the problem occurs (Android / iOS / Web / macOS / Linux / Windows)
Which target OS version, for Web, browser, is the test system running?
Does the problem occur on emulator/simulator as well as on physical devices?

Target Platform:
Target OS version/browser:
Devices:

Run your application with `flutter run --verbose` and attach all the
log output below between the lines with the backticks. If there is an
exception, please see if the error message includes enough information
to explain how to solve the issue.

Run `flutter analyze` and attach any output of that command below.
If there are any analysis errors, try resolving them before filing this issue.

Finally, paste the output of running `flutter doctor -v` here.

To be more precise, please providne the information indicated below:

A completed and minimal reproducible code sample
The output of flutter doctor -v
Reproduction steps, Expected results, Actual results
Which environment did this issue happen in? (Android/iOS/Web/MacOS/Windows/Linux)

Also, please provide the information in text format instead of a screenshot for better readability. Thank you!

sorry  The problem has been solved, and finally found: AndroidManifest.xml (dubug) is written () but AndroidManifest.xml is not written ()

@mkui233 Close the issue since it's solved from your side. Thanks!


Adding Support for Dlib’s 5-point facial landmark detector

Hello Adrian,

I have added support for Dlib’s 5-point facial landmark detector with small changes in Face Aligner .

Results:

100% Verified and Working. Even better results with Dlib’s 5-point facial landmark detector in Face Alignment :) . (See resulting Screenshots below)

Dlib’s 68-point facial landmark detector Output:

Dlib’s 5-point facial landmark detector Output:

Excellent! Thank you for the contribution @abhiTronix! I'll want to test this on my own machine and from there I can merge it in.

This is an example of a great PR, @abhiTronix. I wish all PRs were this good! You documented your updates and included a working example. I have now merged in the PR.

Thanks a lot @jrosebr1, I recently developed an java android application out of this PR.PiciT: Powerful A.I Cropping Tool Thanks for your awesome work :)


Failures in our RC.2 release test runs need to be resolved

Test run results
OS                           Pass Pending Skipped Fail Total
--                           ---- ------- ------- ---- -----
CentOS Linux 7 (Core)        6049     343    1226   17  7635
Debian GNU/Linux 8 (jessie)  6066     342    1226    1  7635
Debian GNU/Linux 9 (stretch) 6066     342    1226    1  7635
openSUSE Leap 42.3           6066     341    1226    2  7635
Red Hat Enterprise Linux     6048     342    1226   19  7635
Ubuntu 14.04.5 LTS           6066     342    1226    1  7635
Ubuntu 16.04.3 LTS           6066     342    1226    1  7635
Ubuntu 17.04                 6066     342    1226    1  7635

Test failure analysis

CentOS 7 and RHEL have the trouble with SSL protocol and Invoke-WebRequest/Invoke-RestMethod.
Redhat is still failing the test for powershell exiting without hanging - this should be looked at more carefully in Jan
OpenSUSE is failing an invoke-item test because 2 ping native commands are found, and the test should be improved to handle multiple binaries being found.
The help tests should be improved to know when a browser is possible.

CentOS 7 and RHEL have the trouble with SSL protocol and Invoke-WebRequest/Invoke-RestMethod

Every release these things come up. In the CentOS Test environment using just the base installed libcurl? Or is the libcurl+openssl+gssapi combo added before the test? I need to get a CentOS 7 vm up and look at this sometime, but we should probably have similar guidance for this OS as we do for macOS.

@markekraus I think we were using the base installed libcurl and it should be the same cause of the macOS issue. /cc @JamesWTruher


Missing Dashboard Navigation Documentation

This page is missing documentation regarding navigation to a dashboard. It should include the following documentation of the API.
----------------------------------------------------------------
dashboard
The object contains the following attributes:

pageType: String. Specify "dashboard".
dashboardId: (Optional) String. The ID of the dashboard to load. If you don't specify it, navigates to the default dashboard.

----------------------------------------------------------------

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

ID: 0c917fb9-84c8-2676-9c6a-f33f03ba5da7
Version Independent ID: 057dd7ac-8383-1650-ceb5-cbea3995180c
Content: navigateTo (Client API reference) in model-driven apps - Power Apps
Content Source: powerapps-docs/developer/model-driven-apps/clientapi/reference/Xrm-Navigation/navigateTo.md
Service: powerapps
GitHub Login: @Nkrb
Microsoft Alias: nabuthuk

@clurdish Thanks for the feedback. We have updated the docs and pushed to live. Closing the issue.  If you need further help, please reopen the issue or open a new issue.


chore(ci): force travis to print logs after driver provider timeout

Travis does not do the after_script step if before_script fails,
so wait_for_browser_provider.sh was not printing out logs.
Force it to print them manually.

Merged to master and 1.3.x


Screen shot feature does not render IconImage

Here is a screenshot using the right-click save screenshot feature:

This is what was on my screen when I used the feature:

Here is the code
input.onGesture(Gesture.Shake, () => {
    basic.clearScreen()
    radio.sendString("DUCK")
})
radio.onDataPacketReceived(({receivedString}) => {
    basic.showIcon(IconNames.Duck)
})
radio.setGroup(5)

Please can you help me find my lost duck??!!

A Blocks / JavaScript code editor for the micro:bit.
pxt.microbit.org version: 0.9.77
Microsoft MakeCode version: 0.12.25
C++ runtime version: v2.0.0-rc8

Did you try /beta?
Get Outlook for iOShttps://aka.ms/o0ukef

From: David Whale<EMAIL_ADDRESS>Sent: Tuesday, April 25, 2017 2:10:32 AM
To: Microsoft/pxt
Cc: Subscribed
Subject: Re: [Microsoft/pxt] Screen shot feature does not render IconImage (#1970)
A Blocks / JavaScript code editor for the micro:bit.
pxt.microbit.org version: 0.9.77
Microsoft MakeCode version: 0.12.25
C++ runtime version: v2.0.0-rc8
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoft%2Fpxt%2Fissues%2F1970%23issuecomment-296968315&data=02|01|jhalleux%40microsoft.com|22317f7171784f47c7cd08d48bbaee84|72f988bf86f141af91ab2d7cd011db47|1|0|636287082370336659&sdata=KA9feQi2jrHJE0fVkYbTVyTHEVFRpxmEz%2FqSLOAcg0I%3D&reserved=0, or mute the threadhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAD-4Kf-Chodzei_91St30B18MByYfATwks5rzbiIgaJpZM4NHMxS&data=02|01|jhalleux%40microsoft.com|22317f7171784f47c7cd08d48bbaee84|72f988bf86f141af91ab2d7cd011db47|1|0|636287082370336659&sdata=sP5YKIN4TAFE4cGJTZvnbtD38rFDNfrrpQUPG5jzy5Y%3D&reserved=0.

Fixed in Beta

Confirmed, fixed in this build of beta. Thanks.
pxt.microbit.org version: 0.11.12
Microsoft MakeCode version: 0.12.55
C++ runtime version: v2.0.0-rc9


[e2e]: generic workflow_dispatch main default slsa3

Repo: https://github.com/slsa-framework/example-package/tree/main
Run: https://github.com/slsa-framework/example-package/actions/runs/11511165988
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.generic.workflow_dispatch.main.default.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.generic.workflow_dispatch.main.default.slsa3.yml
Trigger: workflow_dispatch
Branch: main
Date: Fri Oct 25 03:14:33 UTC 2024

Repo: https://github.com/slsa-framework/example-package/tree/main
Run: https://github.com/slsa-framework/example-package/actions/runs/11528386356
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.generic.workflow_dispatch.main.default.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.generic.workflow_dispatch.main.default.slsa3.yml
Trigger: workflow_dispatch
Branch: main
Date: Sat Oct 26 03:11:06 UTC 2024
Tests are passing now. Closing this issue.


[FLINK-33128] Add converter.open() method call on TestValuesRuntimeFunctions

What is the purpose of the change
Make sure complex type such as Array<Row<>> get converted property in TestValues connector when doing lookup joins.
Brief change log

Added a call to concerter.open() on TestValuesRuntimeFunctions (sync and sync)

Verifying this change
This change added tests and can be verified as follows:

Added integration tests for lookup join with Array<Row<>> on lookup table.

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (yes / no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (yes / no)
The serializers: (yes / no / don't know)
The runtime per-record code paths (performance sensitive): (yes / no / don't know)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: (yes / no / don't know)
The S3 file system connector: (yes / no / don't know)

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

@flinkbot run azure

@flinkbot run azure

@flinkbot run azure


Implementing .NET 8 Identity

My intention is to use this project as a means of testing the new changes to .NET Identity in version 8, now that RC1 has been released (on 12th Sep 2023).
Dan Roth's blog post announcing preview 7, lists the changes to Identity (among many other things).
David Fowler has a sample project on GitHub.
And Andrew Lock has a walkthrough of creating a Web Api with the new Identity API endpoints.

In a follow-up article, Andrew Lock has also pointed out some significant problems.
After trying this out myself, I can see what he means. There are some major issues, specifically a lack of flexibility and customisation. Hopefully they will be addressed by the .NET team over time, but at the moment I may need to look at alternatives.

With RC2 there is a project template that shows how to implement security in .NET 8. It looks messy and complicated.
And I really, really don't like the new Blazor app templates where everything with a .razor extension is thrown under the 'Component' folder.

Here's the Microsoft guide to securing a Blazor WASM app:
https://learn.microsoft.com/en-gb/aspnet/core/blazor/security/webassembly/standalone-with-identity?preserve-view=true&view=aspnetcore-8.0

I watched the demo Jeremy Likness and Stephen Halter did at .NET Conf, but they weren't given enough time and had to race through it.
There doesn't seem to be any detailed documentation yet either, which leaves parts of the code largely unexplained.

It's probably still worth looking at just as an exercise. I can move some of the files and classes across from the .NET 8 Identity project template and then integrate them.
This monolithic version of FotoStorio isn't intended as a turnkey production solution after all.
For the more complex microservice version I'd probably take another look at something like OpenIddict instead of this though.

Closing this as it will be incorporated into the general .NET 8 update.


Did you try to become the maintainer of the origin repository?

Hi,
I am glad this Feliz.MaterialUI project is alive again.
However for me it was big luck that I found this fork. It would be much better if the changes would be merged back in https://github.com/Shmew/Feliz.MaterialUI
Additionally the current situation that there is no package that can be easily installed is quite frustrating. See https://github.com/ArtemyB/Feliz.MaterialUI/issues/3
Since https://github.com/Shmew/Feliz.MaterialUI is currently inactive, and since I find it confusing when the URL of the repo changes each time the owner changes, I would suggest that somebody creates a github company to host the Felitz.MaterialUI repository.
Since @Shmew is not maintaining the project currently, maybe you can ask him if you can take over the repository. Even if you do not plan to contribute further, it would be good to have somebody that could accept or reject occasional merge requests.
@Linschlager - mentioned because he did the Material UI 5 upgrade for this repository.

Hello,
(OMG, no notifications again... Ok, got it, turns out that by default the notifications are disabled)
I haven't tried become the maintainer. However @Shmew seems like not responding at all (to any issues, PRs, etc. in all of his repos), therefore becoming a maintainer could be challenging (at least if he is the only one authorized to do such things).


Remove pkg level state

The branch got messed up during a rebase and Github automatically closed the previous PR (#87). All comments from the previous PR have been addressed.
The primary purpose of this PR is to strip package level state from the service. Global state contributes to application complexity that is considered the cause for many bugs. Package state is 1 notch behind global state.
Package state has been replaced with explicit dependency declarations in functions and, consequently, injected dependencies. This lays a foundation for further refactoring to bring about cleaner code that's easier to work with.
With the state removal some additional fixes and restructuring was done; specifics:

all http handlers are now high-order functions returning an http.Handler that leverages the injected dependencies.
metrics gathered by polling the health function of clients has been isolated in the metrics package and is launched as a distinct goroutine.
a series of bugs in the http health check handler have been fixed. Namely continuing on to write payloads after http.StatusInternalServerError codes are written.
removal of redundant Content-Type headers as the setter was called after WriteHeader() meaning it has no effect.
changed the version end-point to return a known JSON structure in contrast to the previous implementation that, presumably, relied on build time information being injected; the structure used is modeled after the health check endpoint and uses git_rev as the JSON key.
changed http-server package to http and grpc-server package to grpc; these changes have been isolated on their own commit for ease of reviewing.

Metric package state is untouched given it'll likely require a more thought out refactor than simply removing the state and injecting a dependency.
Similarly, I've purposely avoided writing additional tests as I'd like to do a hoslitic rework of unit and e2e testing.
The only behavioral change is that the /_packet/version endpoint payload is now structured and may be different to existing expectations.

The only behavioral change is that the /_packet/version endpoint payload is now structured and may be different to existing expectations.

@mmlb is this potentially a problem?
I wonder if a new endpoint would be preferable for this change.

The only behavioral change is that the /_packet/version endpoint payload is now structured and may be different to existing expectations.

@mmlb is this potentially a problem?
I wonder if a new endpoint would be preferable for this change.

As part of 0.7 id like to see these deprecated anyway, removed in 0.8. The endpoint retrieved its data from a variable that didn't have any definition other than indicating it was JSON. The variable was unset suggesting it was set during build time so I felt comfortable removing that instead returning a defined payload.
The fact this is v0 and that this data had no structure to it, it felt appropriate to strip it out.


Pronouns are added on newly loaded messages on Discord

This is kinda hard to explain but i'll do my best.
When you're opening a conversation, the pronouns are not there.
They're being added only when you're scrolling through unloaded messages or sending new ones.
I hope you understood what I mean and that you'll be able to fix this 👍

CNR, mind sharing a recording of what you're trying to report, as well as sharing your environment?

CNR, mind sharing a recording of what you're trying to report, as well as sharing your environment?

Sorry, I fotgot to check GitHub this week.
Sure, I'll do it when I'll be back to my home.

Sorry, I fotgot to check GitHub this week.
Sure, I'll do it when I'll be back to my home.

Uhh, yeah, I did not really think about that in the past two weeks.
Is it okay for you tho?

Uhh, yeah, I did not really think about that in the past two weeks.
Is it okay for you tho?

I tried to address this issue in b0183b743993ae746e0f6ba02c658452b0bb2fdd, however since I couldn't reproduce locally I don't know if this change will be enough.
Version 0.3.0 of the extension (which includes the fix) is currently pending review in the Chrome Web Store, which is a process that can either be done in 30 seconds or a few days. Keep an eye on the extension see when you get the update, and let me know if this still occurs.

I tried to address this issue in b0183b743993ae746e0f6ba02c658452b0bb2fdd, however since I couldn't reproduce locally I don't know if this change will be enough.
Version 0.3.0 of the extension (which includes the fix) is currently pending review in the Chrome Web Store, which is a process that can either be done in 30 seconds or a few days. Keep an eye on the extension see when you get the update, and let me know if this still occurs.


Error when I want to loop

Hi,
I am starting my first project using autonomous vehicles, and I am trying to use your package for my first simulations. I have a defined and repetitive path (two straight lines and two curves) by its UTM coordinates, and I want the vehicle to follow that path repeatedly.
In the "simulation_demo.launch.py" file, I have established the parameter "want_loop" as True. When the first lap is have finished, the simulation stops and the following error appears:
[goal_pose_creator-3]   File "/home/usuario/nav_ws/install/gps_nav/lib/python3.10/site-packages/gps_nav/goal_pose_creator.py", line 79, in vehicle_pose_callback
[goal_pose_creator-3]     ans = uf_nav.get_look_ahead_point_v2(
[goal_pose_creator-3]   File "/home/usuario/nav_ws/install/gps_nav/lib/python3.10/site-packages/gps_nav/uf_support/route_support.py", line 529, in get_look_ahead_point_v2
[goal_pose_creator-3]     while(dist_remaining > route_segments[look_ahead_seg_num].pt_info[index,4] and
[goal_pose_creator-3] IndexError: index 1 is out of bounds for axis 0 with size 1
If the parameter want_loop is False, there is not any problem.
Could you help me resolving this issue?
Thanks for all.

I have conclude that the problem is not related to loop. There is a point of the pose list where the simulation fails and the terminal shows the error I posted two weeks ago. Why a specific point can trigger that error?
Thanks again.

I have found the error. The path contained two equal points and the vehicle could not advance.
Thanks.


Medium.com links parsed as mail links

E.g.:
Errors in new-docs/content/any/project/performance/n_plus_one.md
↯ medium.com/@bretdoucette/n-1-queries-and-guides-avoid-them-a12f02345be5 (Invalid mail address: medium.com/@bretdoucette/n-1-queries-and-guides-avoid-them-a12f02345be5)

Presumably this is a bit too eager:
https://github.com/lycheeverse/lychee/blob/cefe38ee25d433dd3b7c61a96ccacbec66aa70a8/src/uri.rs#L59-L61

Presumably this is a bit too eager

Exactly. Any idea on how to fix that check?
One idea would be to try and parse the part before the @ as a URL. If that succeeds then assume that the entire link is a URL.
Similar to this:
https://play.rust-lang.org/?version=stable&mode=debug&edition=2018&gist=aff61feeddf28ae7084cda1015f2ef08
Although in this case the link does not begin with a protocol, so it counts as a relative URL without a base, which cannot be parsed by the url crate. (You can check by removing the https:// in the example above.)
@joesan perhaps you have a better idea?

In the meantime you could use --exclude-mail in case you don't want to check mail addresses anyway. 😅

Although in this case the link does not begin with a protocol, so it counts as a relative URL without a base, which cannot be parsed by the url crate. (You can check by removing the https:// in the example above.)

The link actually does begin with https:// — GitHub Actions UI helpfully (🙄) removes the protocol when formatting URLs.

Exactly. Any idea on how to fix that check?
One idea would be to try and parse the part before the @ as a URL. If that succeeds then assume that the entire link is a URL.

If you only care about URLs that include a protocol, then that might work. But if example.com should be treated as a URL, there might be issues since<EMAIL_ADDRESS>is a valid email address.
It looks like that url crate parses mailto: links just fine, so I would probably try something like:

Try parsing the input string with Url::parse (without manually stripping out mailto:).
If (1) fails, try parsing it as an email.
If (1) succeeds and the parsed scheme is "mailto", try parsing the parsed path as an email, same as (2).
If (1) succeeds and the parsed scheme is "http"/"https", great, we have a URL.
If we haven't hit one of the previous cases, fail? Maybe there are other cases to care about — brain's a bit dead at the moment. 😄

Would using a Regex to check for E-Mail solve the problem? I mean using Regex to distinguish between E-Mail and URL. So if I can determine using Regex that the given link is an E-Mail, then I'm done. What do you say @mre

I was able to use Regex to differentiate E-Mail from URL, but now it panics saying that it cannot find a URL base:
thread 'extract::test::test_markdown_email_like_link' panicked at 'Expected valid Website URI: RelativeUrlWithoutBase', src/test_utils.rs:40:34

I was able to use Regex to differentiate E-Mail from URL, but now it panics saying that it cannot find a URL base:
used this as test url: medium.com/@bretdoucette/n-1-queries-and-guides-avoid-them-a12f02345be5
thread 'extract::test::test_markdown_email_like_link' panicked at 'Expected valid Website URI: RelativeUrlWithoutBase', src/test_utils.rs:40:34

That's an error from the url crate:

Although in this case the link does not begin with a protocol, so it counts as a relative URL without a base, which cannot be parsed by the url crate. (You can check by removing the https:// in the example above.)

The link actually does begin with https:// — GitHub Actions UI helpfully (🙄) removes the protocol when formatting URLs.

Do we prepend the URL with https:// or http:// or both and run it through?

Seems there is a PR for the Linkify library that is used here: https://github.com/robinst/linkify/issues/7

Right! Fixing that should fix the problem. The PR is not actively worked on but if you find the time to fix the open issues documented in the issue description then a lot of people would be greatful. 😉

I can try that, but for this case here as a quick win, I will submit a PR that checks for E-Mail using Regex so that the cases without a scheme are not treated as E-Mail. Is that ok?

Partial fix to not treat Medium links as E-Mail: https://github.com/lycheeverse/lychee/pull/164

This also causes false-negatives for CDN's, eg.:
<EMAIL_ADDRESS>(Invalid mail address<EMAIL_ADDRESS>

I'm seeing issues with this too. Pretty much all medium.com links parse as email addresses. Let me see if I can provide a fix :smile:

The PR fixing this properly (hopefully! :crossed_fingers:) is up now.

In the meantime you could use --exclude-mail in case you don't want to check mail addresses anyway. 😅

Hi @mre , I have attempted to use --exclude-mail           Exclude all mail addresses from checking  since am not interested in including email, but still it catches the medium articles as emails:
example of the link that was still caught and reported :
↯<EMAIL_ADDRESS>(Invalid mail address<EMAIL_ADDRESS>

Thanks. Haven't looked at it yet, but my suspicion is that it's because we do the parsing before the rejection of mail addresses in general.

example of the link that was still caught and reported :
↯<EMAIL_ADDRESS>(Invalid mail address<EMAIL_ADDRESS>
The issue should be fixed by #177.

Thanks. Haven't looked at it yet, but my suspicion is that it's because we do the parsing before the rejection of mail addresses in general.

#177 makes it so that links are only parsed as emails if they fail parsing as an URL.

This was fixed by @pawroman, who delivered an amazing PR. Should now work in master and lychee-action@master. 🥳


Compute frequency and norm weights of edge links

Compute both the frequency and norm of the edge weights, summing the totals first and not farming it out to the frequency distribution which must make multiple computations of N.

Fixed in 1.3!


use version number instead of release in .travis.yml

release will change over time, but your REQUIRE file says this
package supports julia 0.3 so it should continue to be tested

Coverage remained the same at 100.0% when pulling 52f20610a3fc7700005d2d02f46cdf4eec39d079 on tkelman:travisver into 0b4a720172a149dc6aaf0f422d7d8ed1ab725897 on rawrgrr:master.


Preload the data node with a small dataset

When the data node is launch, check if the dataset "example-dataset" exists. If not, create it:

create "example-dataset"
create "example-fhir-store"
create "example-annotation-store"
push Patient and Note objects for 2 patients
push Date, Person Name and Physical Address Annotation for the clinical notes pushed

These data can be taken from the 2014 i2b2 training dataset
EDIT: The motivation is that this will simplify the tutorials and demonstrations.

There are other resources that will show how to push datasets and other resources to the data node. Below is a couple of repositories where this information will be made available:

https://github.com/nlpsandbox/i2b2-phi-dataset
https://github.com/nlpsandbox/nlpsandbox-client


bazel: Shift deprecate_* tools to bazel and cleanup requirements

Signed-off-by: Ryan Northey<EMAIL_ADDRESS>Commit Message: bazel: Shift deprecate_* tools to bazel and cleanup requirements
Additional Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Deprecated:]
[Optional API Considerations:]

requires repo_path #17293

What's the new plan for eventually doing config clean up in envoy and should we remove or replace that script?

not sure, i was just going through any scripts that didnt use bazel, and moving them. Mostly that means the lesser used ones - some of which may no longer be required
@htuch ?

/wait for GOVERNANCE/docs update

/wait

The fatal-by-default for deprecated should continue to be a part of the process. Minor versioning has its own tooling that @adisuissa has built out, which adds in additional annotations on when things are removed in Envoy and there is the 1 year cycle around this. But, this doesn't change the fact that we want to signal early to the community that deprecated features are going away, so +1 to keeping this process around.

We are still going to have a deprecation period, where deprecated features are still usable, but marked as fatal-by-default. This will serve as a final-warning and encourage upgrade before removing the support.

Gotcha.  FWIW the script was removed from our release process a while back due to uncertainty around the longer term plan.  If we plan on using it it'd be good to make sure it's still WAI and add it back.

ive updated the GOVERNANCE doc
looking through these scripts, if we are going to keep them, we probably want to update them at some point, but this PR at least brings them into the bazel workflow and reduces dep noise

Gotcha. FWIW the script was removed from our release process a while back due to uncertainty around the longer term plan. If we plan on using it it'd be good to make sure it's still WAI and add it back.

Ack.
Thanks @phlax for the cleanup.

/assign-from @envoyproxy/dependency-shepherds


dySOU: preparation: modified module

I added modules for dynamic SOU.
in /core/testdata.py,
    def dynamic_multiband(...):
        ...
        lc = gen_dynamic_lightcurve(...)
        ...

in /core/lc.py,
    def gen_dynamic_lightcurve(...):
        ...
        rotA = rotating_map(...)
        ...

in /core/map.py,
    def rotating_map(...):

Also, I added unit tests. Please confirm them.

All of the tests worked. Thanks


Unable to Logout

Describe the bug
See issue title.
To Reproduce
Steps to reproduce the behavior:

Go to https://orange-island-0901fe000-59.eastasia.azurestaticapps.net/
Log in with an Azure AD account
Go to https://orange-island-0901fe000-59.eastasia.azurestaticapps.net/.auth/logout
User is still logged in

Expected behavior
User should be logged out
Additional context

The logout request returns the index page, based on Chrome Devtools -> Network.

Looks like the service worker might be messing things up here. I haven't tried this but I think you might need to exclude server-side paths: https://docs.microsoft.com/en-us/aspnet/core/blazor/progressive-web-app?view=aspnetcore-5.0&tabs=visual-studio#support-server-rendered-pages

Thanks for pointing me into the right direction Anthony! :)


Docs Issue - No such file or directory editorconfig

Issue in the docs. Need to install editorconfig before this plugin can be used. Using quelpa + quelpa-use-package on standard emacs and I got the error
 No such file or directory editorconfig

Fixed by using
M-x package-install RET editorconfig RET

I have already declared editorconfig as a dependency. Shouldn't it be resolved and installed automatically?
https://github.com/zerolfx/copilot.el/blob/a68f1ce94addc45984ba5bb1972bfba95c8b7f96/copilot.el#L11

That's weird. It didn't pick it up. I used quelpa for installing it and followed the docs to a T.
Using NixOS. Not sure if relevant.

Sorry, I'm not familiar with the workings of Emacs package management.
After looking at the readmes of other packages, it seems necessary to include a list of (uncommon) dependencies in the readme file. I will add this information at a later time.

Dude, you wrote the copilot plugin. You don't need to apologise. Great work btw.


Adding Syslog Workbook

Summary
Container insights is adding the capability to collect syslog from AKS Linux node. This out-of-box workbook will help customers view syslogs easily.

PR Checklist

[x] Explain your changes, so people looking at the PR know what and why, the code changes are the how.

If adding or updating templates:

[x] post a screenshot of templates and/or gallery changes
[x] ensure your template has a corresponding gallery entry in the gallery folder
[x] If you are adding a new template, add your team and template/gallery file(s) to the CODEOWNERS file. CODEOWNERS entries should be teams, not individuals
[x] ensure all steps have meaningful names
[x] ensure all parameters and grid columns have display names set so they can be localized
[x] ensure that parameters id values are unique or they will fail PR validation (parameter ids are used for localization)
[x] ensure that steps names are unique or they will fail PR validation (step names are used for localization)
[x] grep /subscription/ and ensure that your parameters don't have any hardcoded resourceIds or they will fail PR validation
[x] remove fallbackResourceIds and fromTemplateId fields from your template workbook or they will fail PR validation

#sign-off


Create a simple login and sign up screen (responsive)

may I work on this?

plz assign me this issue sir?

Hi there,  I have made a pull request can you please have a look.

It looks really amazing

Thanks. :)


Install automation test

This PR has following changes.

Jenkins pipeline file config
Installation of native tar and its validation
Installation of RPM pkg and its validation
Poller metric validations for pollers

CI integration : http://<IP_ADDRESS>:8080/blue/organizations/jenkins/harvest2_0%2Fsmoke/detail/smoke/4/pipeline

merged via PR #380


Add cloud message receiving capability

FCM web got released 3 days ago: https://firebase.google.com/docs/cloud-messaging/send-message

http://techstory.in/17102016-google-announces-firebase-cloud-messaging-web/
http://www.eweek.com/cloud/google-adds-web-support-for-firebase-cloud-messaging.html

Turns out that this requires the development of an own XMPP server which is used to initiate the message pushes besides the FCM server. I'm still exploring if I can shortcut the use of such server by utilizing message topics.

https://stackoverflow.com/questions/40199616/is-it-possible-to-broadcast-messages-with-fcm-without-having-an-own-xmpp-server


[doc] Expand documentation of @require_deps decorator

Fixes #1382.

Codecov Report

Merging #1401 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1401   +/-   ##
=======================================
  Coverage   91.77%   91.77%
=======================================
  Files          83       83
  Lines       12708    12708
=======================================
  Hits        11663    11663
  Misses       1045     1045

Impacted Files
Coverage Δ

reframe/core/decorators.py
96.90% <ø> (ø)

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 becd396...39f5460. Read the comment docs.


basic instructions for deploying locally?

I tried
ModuleNotFoundError: No module named 'icecream'
(textgen) [root@pve0 DocOwl1.5]# pip install icecream
Collecting icecream
  Using cached icecream-2.1.3-py2.py3-none-any.whl.metadata (1.4 kB)
Requirement already satisfied: colorama>=0.3.9 in /data/miniconda3/envs/textgen/lib/python3.10/site-packages (from icecream) (0.4.6)
Requirement already satisfied: pygments>=2.2.0 in /data/miniconda3/envs/textgen/lib/python3.10/site-packages (from icecream) (2.17.2)
Requirement already satisfied: executing>=0.3.1 in /data/miniconda3/envs/textgen/lib/python3.10/site-packages (from icecream) (2.0.1)
Requirement already satisfied: asttokens>=2.0.1 in /data/miniconda3/envs/textgen/lib/python3.10/site-packages (from icecream) (2.4.1)
Requirement already satisfied: six>=1.12.0 in /data/miniconda3/envs/textgen/lib/python3.10/site-packages (from asttokens>=2.0.1->icecream) (1.16.0)
Using cached icecream-2.1.3-py2.py3-none-any.whl (8.4 kB)
Installing collected packages: icecream
Successfully installed icecream-2.1.3
WARNING: Running pip as the 'root' user can result in broken permissions and conflicting behaviour with the system package manager. It is recommended to use a virtual environment instead: https://pip.pypa.io/warnings/venv
(textgen) [root@pve0 DocOwl1.5]# python app.py
2024-04-13 16:08:59 | ERROR | stderr | Traceback (most recent call last):
2024-04-13 16:08:59 | ERROR | stderr |   File "/data/mplug-docowl/DocOwl1.5/app.py", line 23, in <module>
2024-04-13 16:08:59 | ERROR | stderr |     no_change_btn = gr.Button.update()
2024-04-13 16:08:59 | ERROR | stderr | AttributeError: type object 'Button' has no attribute 'update'

Hi, @thistleknot , the local demo code should be run with gradio==3.27.0, if you must use gradio==4.26.0, you can refer to our code on HuggingFace space by git clone https://huggingface.co/spaces/mPLUG/DocOwl . We have updated these instructions on README.md of DocOwl1.5


new file:   Scripts/roundtrip_test.py

What does this implement/fix? Explain your changes.
A very simple script to test that we have
rdkit installed and working for Python.
If success, the installed rdkit version is printed out.
Upon failure, if any, we will have an assert failing.
Any other comments?
Useful to anyone who has to install rdkit from sources often.

I don't really see the usefulness of this script. Anything you write that calls it could achieve the same effect (make sure RDKit is installed correctly) by doing: python -c 'from rdkit import Chem'

And does the one liner also checks that rdkit was built inchi support?

python -c 'from rdkit import Chem;print(Chem.MolToInchi)

Ok, fine. I'll put this one-liner somewhere just for myself then.
Also, it still doesn't say the rdkit version that is being used.


run out of heap space when compiling kernel bias_op.cc

Please make sure that this is a build/installation issue. As per our GitHub Policy, we only address code/doc bugs, performance issues, feature requests and build/installation issues on GitHub. tag:build_template
System information

OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Windows 10
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device: NA
TensorFlow installed from (source or binary): source
TensorFlow version: 2.3.1
Python version: 2.7
Installed using virtualenv? pip? conda?:
Bazel version (if compiling from source): 3.7.0
GCC/Compiler version (if compiling from source): NA
CUDA/cuDNN version: NA
GPU model and memory: NA / 16GB

Describe the problem
compile bias_op.cc
fatal error C1002: compiler is out of heap space in pass 2
how to adjust the compiler options?
please help!
Provide the exact sequence of commands / steps that you executed before running into the problem
Any other info / logs
Include any logs or source code that would be helpful to diagnose the problem. If including tracebacks, please include the full traceback. Large logs and files should be attached.

sorry . the complete error message is
unsupported\Eigen\CXX11\src\Tensor\TensorBroadcasting.h(653) : fatal error C1002: compiler is out of heap space in pass 2

@oldoldman
Please, see tested build configuration from here. Can you try with python 3.5-3.8 and verify once. Thanks!

@oldoldman
Please, see tested build configuration from here. Can you try with python 3.5-3.8 and verify once. Thanks!

passed after changed compiler to clang. thanks!


mopidy-youtube: enable tests

Motivation for this change

Enable tests
Add homepage
Add missing inputs
Add pythonImportsCheck

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

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

1 package built:

mopidy-youtube


Useless developer advice.

Regarding the following content:

If you do choose to take a dependency on values found in the ParentContext, you should take steps to ensure that your code is resilient to adapt to potential changes. You should test the logic regularly to verify that the conditions you depend on remain in effect over time.

So, let me see if I get this sequence correct:

Dynamics CRM/365/for Sales/Customer Engagement; the ParentContext was perfectly serviceable through every version of the product from 4.0 to 9.0
Microsoft changes the interface implementation with GrpcSandboxPluginExecutionContext in the new cloud architecture.
Microsoft never officially declares ParentContext as deprecated
Microsoft updates the documentation to require me to write a full-body, Turing Complete A.I. to reverse-engineer what they changed or will change, on-the-fly, whenever previously valid assumptions are invalid

So, tell me?  What "code" should I be writing that is "resilient" to an unclassified, unbounded set of "changes"?  Asking me to regularly test that Microsoft hasn't silently, implicitly, and dramatically altered a decade of strongly-founded assumptions aside, what is the purpose of this statement?
Answer:  To place blame on me for not discovering that an undocumented change and an undeclared obsolescence has impacted my ISV solution, after it's already in the hands of my customers.  I submit it contains no substance, because it's both unachievable without proving P=NP and disingenuously prescribing Application Lifecycle Management practices in neither the correct context nor tone.

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

ID: 4922340f-b68f-187f-8eab-6c9c065847b8
Version Independent ID: f683b1c6-b4ef-d0f0-4bc0-f42b854a7d13
Content: Understand the execution context (Microsoft Dataverse) - Power Apps
Content Source: powerapps-docs/developer/data-platform/understand-the-data-context.md
Service: powerapps
GitHub Login: @JimDaly
Microsoft Alias: pehecke

Thank you for your feedback.
It is true that ParentContext has existed since 4.0 and we know many developers made assumptions about the order in which business logic was applied.
Several years ago, when the various components of the CRM solution were split out into separate offerings the owners of those solutions decided to refactor their business logic. This resulted in changes in the nature or order of events in the ParentContext.
But the order wasn't guaranteed or documented. When we started to hear reports of people experiencing these kinds of logic errors, there was little that could be done. The owners of the modules had shipped their solutions and would not refactor them to the way used-to-work based on a guarantee that was never made. Anyone (whether MS or other) who builds their logic has the right to refactor how it works.
Prior to this, we were not aware that this would be a problem. When this started happening, we added this guidance. ParentContext is not deprecated. It still contains data that may be useful. The intention of this guidance is to urge people not to take a critical dependency on it and find other ways. Ideally, a plug-in should be written to handle any way that the message might be called.
If people have to take a dependency, and there are some cases where it is expected, we saw that they could make their code more resilient by looping through the items in the ParentContext to detect if it occurred anywhere higher in the pipeline without assuming it would be at a specific position. But a better strategy would be to include add a shared variable at a critical point so that any logic occurring later could detect what should happen.
I'm sorry that this has caused problems for you and others. I think we could provide more specific guidance about what we mean by resiliency, but that generally requires a very specific example.
I would take that as an actionable item for the documentation. If you have a specific example you want to share, I'd love to hear it.

Thank you for your feedback.
It is true that ParentContext has existed since 4.0 and we know many developers made assumptions about the order in which business logic was applied.
Several years ago, when the various components of the CRM solution were split out into separate offerings the owners of those solutions decided to refactor their business logic. This resulted in changes in the nature or order of events in the ParentContext.
But the order wasn't guaranteed or documented. When we started to hear reports of people experiencing these kinds of logic errors, there was little that could be done. The owners of the modules had shipped their solutions and would not refactor them to the way used-to-work based on a guarantee that was never made. Anyone (whether MS or other) who builds their logic has the right to refactor how it works.
Prior to this, we were not aware that this would be a problem. When this started happening, we added this guidance. ParentContext is not deprecated. It still contains data that may be useful. The intention of this guidance is to urge people not to take a critical dependency on it and find other ways. Ideally, a plug-in should be written to handle any way that the message might be called.
If people have to take a dependency, and there are some cases where it is expected, we saw that they could make their code more resilient by looping through the items in the ParentContext to detect if it occurred anywhere higher in the pipeline without assuming it would be at a specific position. But a better strategy would be to include add a shared variable at a critical point so that any logic occurring later could detect what should happen.
I'm sorry that this has caused problems for you and others. I think we could provide more specific guidance about what we mean by resiliency, but that generally requires a very specific example.
I would take that as an actionable item for the documentation. If you have a specific example you want to share, I'd love to hear it.


savi deps update fails when run for the first time - succeeds the second time

Also see: https://github.com/mneumann-savi/MQTT/actions/runs/4187381765/jobs/7257136099
> savi deps update
Downloading new library versions from GitHub...
Downloaded ByteStream v0.20230131.0
Downloaded TCP v0.20230215.0
Downloaded IO v0.20221003.0
Downloaded OSError v0.20220325.0
Downloaded IPAddress v0.20230131.0
Downloaded Logger v0.20230131.0
Downloaded Time v0.20230131.0
Downloaded Timer v0.20220321.0
Downloaded Spec v0.20220928.0
Downloaded Map v0.20220304.0

Compilation Errors:

---

Failed to clone version v0.20230131.0 of this dependency:
from ./manifest.savi:40:
  :dependency Time v0
              ^~~~

- please ensure this location is correct, or try again later:
  from ./manifest.savi:41:
    :from "github:savi-lang/Time"
          ^~~~~~~~~~~~~~~~~~~~~~~

---

Failed to clone version v0.20230131.0 of this dependency:
from ./manifest.savi:56:
  :transitive dependency Time v0
                         ^~~~

- please ensure this location is correct, or try again later:
  from ./manifest.savi:57:
    :from "github:savi-lang/Time"
          ^~~~~~~~~~~~~~~~~~~~~~~
---

Failed to clone version v0.20220321.0 of this dependency:
from ./manifest.savi:59:
  :transitive dependency Timer v0
                         ^~~~~

- please ensure this location is correct, or try again later:
  from ./manifest.savi:60:
    :from "github:savi-lang/Timer"
          ^~~~~~~~~~~~~~~~~~~~~~~~

When savi deps update is run, the first time it fails. The second time, it succeeds downloading the remaining repos.

Savi is just running git clone --quiet --depth 1 --branch $VERSION_NAME $URL, and this is the message you see when it fails.
Perhaps we need to add some retry logic to have it try a few more times before declaring failure?

@jemc it fails locally too, not just in github actions.
IIRC, I was once changing related code to limit git clones to run max 2 in parallel.  I guess that's even too much. Maybe just clone them in sequence and retry if it fails...

Trying with
> savi --version
savi version: v0.20230324.0
llvm version: 15.0.6

and getting the same problem:
> savi deps update
Downloading new library versions from GitHub...
Downloaded ByteStream v0.20230223.0
Downloaded TCP v0.20230215.2
Downloaded IO v0.20230215.1
Downloaded OSError v0.20220325.0
Downloaded IPAddress v0.20230131.0
Downloaded Logger v0.20230131.0
Downloaded Time v0.20230131.0
Downloaded Timer v0.20220321.0
Downloaded Spec v0.20220928.0
Downloaded Map v0.20220304.0

Compilation Errors:

---

Failed to clone version v0.20230131.0 of this dependency:
from ./manifest.savi:40:
  :dependency Time v0
              ^~~~

- please ensure this location is correct, or try again later:
  from ./manifest.savi:41:
    :from "github:savi-lang/Time"
          ^~~~~~~~~~~~~~~~~~~~~~~

A second run of savi deps update works.


Upgrade printrun to version 1.3 (Feb 2015 release)

This upgrades printrun to version 1.3 (third OS X release).

Thanks.


Add unionWith

It's seems like it could be useful. Right now I use a couple of newtypes with different semigroup instances for this

We merged https://github.com/purescript/purescript-foreign-object/pull/19 so we could implement the Semigroup instance with append = unionWith (<>) in this pull request.
Tests should pass with a rebase once https://github.com/purescript/purescript-foreign-object/pull/22 is merged.


Please provide sample for .NET Core 3.1

Now where .NET Core 2.2 is out of support, it would be great if the  .NET Core sample could be updated to the latest LTS version (3.1).

@vijayma @vijaymatest1
Hi, could you provide 3.1 sample ?
Thank you for your amazing work!

I've modified the sample to the LTS version (pending pull request)

Thank you, I'll update my code based on your best practices


Accomodate mixed (keyboard+gamepad) input

increasingly PC users are presented options for combining gamepad and mouse/keyboard input, termed "mixed" or "simultaneous" input. limiting "player" slots to either/or means either thrashing between modes when hotswapping is enabled, or being locked out of one or the other input source altogether.
https://www.pcgamingwiki.com/wiki/List_of_games_that_support_simultaneous_input
common devices that leverage mixed input include gamepads (Steam Controller and thus by extension any gamepad leveraging Steam Input,  and numerous Android gamepads that map selective buttons to keyboard), gameboards (Logitech G13, Wooting devices, CoolerMaster's Aimpad enabled MK850 and ControlPad, Belkin's N50 line, Saitek PZ31A, Razer's Orbweaver and Tartarus lines), gamemice (ROG Chakram, Swiftpoint Z, Lexip Pu94), and burgeoning hybrid ergo devices like the Azeron and 3dRudder
generally the consensus is "indicate device type as gamepad, but do not lock-out mouse or keyboard input". i know this is a pain especially for assigning devices per player, but if mouse/keyboard pairing is acceptable perhaps all three in tandem isn't entirely unreasonable ;-)

A mixed input source can be supported, sure, and when player input is set up for this then hotswapping will be disabled. This will require the developer to explicitly set a player's source to INPUT_SOURCE.MIXED e.g. via a menu option.

As discussed this is a single-player-only feature, and in the larger scheme of decoupling players from devicesa player need only claim one source, share one source (eg. two players on keyboard or sharing one pad), or claim all sources (mixed input.

Implemented


Difference between "sumup-android-api" and "sumup-android-sdk"?

When i visit both repositories, i just see, more or less, the same thing: a hello-world demo application for android.
So what's the difference between that both repositories.
Thank you very much for your feedback.

sdk my be the newer version, right? and the api is obsolete...?
sdk current release version:
3.3.2
api current release version:
1.4.0
Is the correctly?

Are there any other differences??

-> https://github.com/sumup/sumup-android-sdk/issues/131


Misc fixes

typos
update links that redirect
add links and backreferences
change notebook -> lab reference in jupyter startup cmd on Hello Charts

Finally got this stuff in. Sorry for the delay. I had overlooked the PR. Just stumbled on it while roaming around.


Use pedvide ADC library for Teensy 3.1

Try to use this library for better ADC control and dual scope mode.
https://forum.pjrc.com/threads/25532-ADC-library-update-now-with-support-for-Teensy-3-1

This is now default in the "teensy" branch, works for Teensy 3.1+ and Teensy LC.
The master continues to use "standard" arduino code for Teensy.

Dual scope mode should be a separate issue.


[5.7] Fix validate method

Issue fix #25736, #25750.
941c8c7 cause when Illuminate\Foundation\Http\FormRequest::validated is called. validated again.
I add contracts method. I think there is no choice to fix this problem.
I apologize...

@sisve
Sure. But there is no choice to fix this problem.

Regarding choice; there's nothing that forces us to choose to target 5.7. We could just target 5.8 and choose to fix the issue in the next release. Not every bug and problem needs to be fixed in the current release, specially if they have the potential to break existing applications.


Import and Exporting of color names

How is this done, exactly? I'm seeing an import button, but not sure of what kind of input is accepted.

"Spec Export" can export the colors.json file


test-branch/execute-job/wrong-commit-sha-on-pr/9b6c4eca-774b-4023-aca3-a8062a1daedb

PR for testing 31466b833351f0f370cbb30235787859e0d2e5f7

/canonical/self-hosted-runners/run-workflows f34a8280e9b03c136cbcfcbde2993867bd48438e


Doesn’t work for Firefox

I love this workflow and use it multiple times a day with Chrome. I’m considering switching to Firefox and was wondering if it is possible to make it work with it. Tried my hand at it, but didn’t succeed unfortunately.

I am an user of Vivaldi browser. I just sent a PR: Support selection text in Vivaldi by heleeos · Pull Request #6 · fallroot/copy-url-for-alfred to make this workflow works for Vivaldi browser. I think you can do this for Firefox.

@khalidl I'm sorry for late reply. This workflow was made by using Open Scripting Architecture that many application supports in macos. Unfortunately Firefox does not supply this feature yet so I can't get the url and title of current page.


jit: finally block is not executed after try/catch throws exception

The Error/stack.js test fails because all the "finally" blocks are not executed (as of git master bbf46243132). I can reduce the issue to just this code
try {
    try {
        throw 42;
    } finally {
        print("In finally");
    }
}
catch (e) {
    print("In catch", e.description);
}

If the test is run with -forceNative we get:
localhost:~/git/ChakraCore%/home/xan/git/ChakraCore/BuildLinux/Test/ch -forceNative foo.js
In catch undefined
Without:
localhost:~/git/ChakraCore%/home/xan/git/ChakraCore/BuildLinux/Test/ch foo.js
In finally
In catch undefined
The tests expect the latter behavior.

Just tried this in an older branch and also latest master on Ubuntu, no repro...

This is a test build, compile target amd64, built with clang 3.9.1 on Fedora 25. I can easily try an older compiler just in case, not sure what else might be relevant.

Compiling with clang 3.8.1 fixes the bug. It also fixes the other failures I was seeing. I'll try to figure out whether this is a bug in clang or something a newer clang has uncovered in Chakra if that seems OK.

I also reproduced with clang 3.9 on Ubuntu. Thanks for reporting, and yes please help investigate!

I don't recall this issue while I was testing ArchLinux / Ubuntu with clang 3.9 couple of weeks ago

Could you please try this one with the latest master? #1964 disables tail call optimization only for noinline functions + clang 3.9 build fix is also merged in.

Same results as of commit 526fbe4fb39. I tried to test with clang 4.0.0 (master) but Chakra does not compile.

I have bisected this and not surprisingly the "bad" commit is the one enabling JIT on Linux. Still as I said above it remains to be seen whether there's a bug in there or in newer clangs.

@xanlpz Thanks for pushing on this. I can also confirm the bug with Clang 3.9

BTW; I don't think this is a Clang bug.

@obastemur @xanlpz i diffed the baseline against CI job's output. In my CI job I was missing the last overflow error which relies on a deeply nested object to trigger stack overflow. Maybe, the count (200000) we are using just doesn't cut it (overflow) anymore. Though, I'm not sure we are talking about the same issue here.
  var count = 200000; // Keep this unrealistic number as we do (osx)
                        // and do not limit stack memory to a particular capacity

One more thing; This is not a Clang 3.9 only issue; CI -> https://ci.dot.net/job/Microsoft_ChakraCore/job/master/job/osx_osx_debug_static/
Looks like we have been hitting this issue randomly.

Just for the future reference. What I have shared above on CI and what's happening with Clang 3.9 Linux are not the same. Interesting part is that we have the problem on the same file.
It is important to remember that this file pushes memory limits a bit.

@xanlpz I saw https://github.com/Microsoft/ChakraCore/blob/master/lib/Backend/NativeCodeGenerator.cpp#L1060-L1065 might be responsible.

The bit that could be somehow compiler dependent in that code snippet is the call to __register_frame actually. I think on Linux you are just supposed to call the glibc implementation, but perhaps there is some difference in how this is handled between clang 3.8 and 3.9 and we end up doing something subtly different.

@xanlpz Thanks for all the details and follow ups. This should be fixed on master.


impl(bigtable): modern AsyncRowReader

Most of the work for #9174

This change is

It is really hard to get a sense of what the tests do

All the tests have the same structure, so I added comments for the first one. I also wrote in sentences the expectations we are looking for, for the tests that are less obvious. I hope that is what you mean.

or how they changed.

They hardly look alike. I think mainly due to using a MockStream vs. running everything on a FakeCompletionQueueImpl.

The coverage looks good, but that is not a perfect way to assess if tests are good enough. Any thoughts on that?

I looked through the code again. I added two tests for code that we were not exercising. I think what we have is good enough.


After installing libfec, liquid-dsp cannot find libfec

installed libfec and then I configured liquid-dsp, in the output of ./configure I am getting these lines about libfec
checking fec.h usability... no
checking fec.h presence... no
checking for fec.h... no
/usr/local/include/ in this path I am able to find fec.h file after instaling libfec.
Please help me with this issue.
thanks.

Hi @sanjeev1994
Can you search config.log in liquid-dsp/ directory for "fec.h" and post any nearby lines here? It will show the test it's doing first "checking for fec.h usability" and then the test it tries and then the result. If you can paste the entries for fec.h, I think I can help you diagnose what's wrong


feat: port CategoryTheory.Triangulated.Rotate

[x] depends on: #2466
[x] depends on: #3038
[x] depends on: #3039
[x] depends on: #3046

This is backported in https://github.com/leanprover-community/mathlib/pull/18670

This PR/issue depends on:

leanprover-community/mathlib4#2466
leanprover-community/mathlib4#3038
leanprover-community/mathlib4#3039
leanprover-community/mathlib4#3046
By Dependent Issues (🤖). Happy coding!

This PR/issue depends on:

leanprover-community/mathlib4#2466
leanprover-community/mathlib4#3038
leanprover-community/mathlib4#3039
leanprover-community/mathlib4#3046
By Dependent Issues (🤖). Happy coding!

bors merge


WIP: Disconnected event

See discussion about the "todo" in Slack.

Changes Unknown when pulling e59a644b4f3db85b12ce2e791127e958b735126e on ceilican:DisconnectedEvent into ** on ScorexFoundation:master**.


Updated README: how to enable IMAP & auth "less secure apps" in Gmail

I'm really thankful for this package and am using it in a project of mine. Two things held me up from getting a much quicker start than I did: realizing that I had to set up IMAP in Gmail and authorize "less secure apps" to access my account.
In this PR, I've added some commentary on how to do that, so that other beginner programmers like me can tackle these obstacles a bit more easily.

Ok, thanks for contributing.


enhance: saga-effect take should be able to take multi actions;

https://github.com/dvajs/dva/issues/1261

Coverage decreased (-0.9%) to 69.604% when pulling ae4653dcbbcae442943dad4bf3127f226e2c2d84 on sjy:saga_take into e114cfb739cd830640e631d080b7b3990ac7ded1 on dvajs:master.

Coverage decreased (-0.9%) to 69.604% when pulling ae4653dcbbcae442943dad4bf3127f226e2c2d84 on sjy:saga_take into e114cfb739cd830640e631d080b7b3990ac7ded1 on dvajs:master.

加一个额外的 take with array 的用例吧。

@sorrycc  加了新的ut，revert对已有ut的修改


Feature request: listen for program exit

Would it be possible to make the program exit (and reset to normal acceleration) on a simple keypress (maybe q), instead of having to terminate it with CTRL-C ? Granted it's very rare, but I encountered on two occasions that the sim rate did not get reset to 1 after terminating the process that way. I'd think that if it would explicitly listen for an exit command the sim rate reset could be guaranteed...

Did you have this issue with version 0.7? It’s supposed to be doing exactly this.

Ah apologies, I didn't know there was already a new release out (I only saw the commit for the nav_mode_guarded   which hasn't been merged yet). Cool, so that's exactly what I was asking for, thanks!
On another note, I wanted to build the curses branch to try it out but I get
Unable to find "env\Lib\site-packages\SimConnect\SimConnect.dll" when adding binary and data files.

Is there a new build step that needs to be done? The old v0.6 always built fine for me...

I think the problem is in simrate_control.spec. I'm using a virtual environment locally and I think I made an undocumented assumption about where SimConnect.dll can be found. I'll look into making that more robust.
binaries=[(SPECPATH + "/env/Lib/site-packages/SimConnect/SimConnect.dll", './SimConnect/')],

Glad it's working well for you!
I think I've resolved the SimConnect.dll search issue, so I'm going to go ahead and close this issue.


Header included more than once breaks method names.

ClassA.h defines ClassA and includes ClassY.h
ClassB.h defines ClassB and includes ClassY.h
ClassA, ClassB, and ClassY are all marked for export.
The C# class generated for ClassY ends up appending '1' to all its method names. Eg, ClassY::Init() in C++ becomes ClassY.Init1() in C#.
Note that ClassY.h is protected by header guards, since it needs to be included in more than one other header file. I assume this situation is not dealt with well by CppSharp and/or may be related to the problem.

This is still a problem almost a year later.  This causes all sorts of confusion when using CppSharp.

@ddobrev I created the 3 headers in the same folder as the CSharp test project files. But, I could not generate their C# files. I also tried to add them to premake4.lua in the CSharp folder by changing
SetupTestCSharp("CSharp")
to
SetupTestCSharp("CSharp", {"ClassA", "ClassB", "ClassY"})
but it didn't work.
How do I add them to any test to generate the output c# files?

Where exactly are your new headers located? If you place them in CppSharp/tests/CSharp, they should appear as soon as you have regenerated your C++# solution.

Hey guys (cc @jamesford42 @tomspilman)
We'd like to get this fixed but @genuinelucifer is having trouble reproducing the issue.
Can you look at https://github.com/mono/CppSharp/pull/620/files and see what we're missing?

@tritao - Maybe using #pragma once instead of header guards?

I can't see it making any difference, it should be equivalent, but give it a shot @genuinelucifer.

Tried, didn't work. @tritao @tomspilman

@genuinelucifer - Maybe it is in how we normally include the files in the generation process:
<EMAIL_ADDRESS>");
<EMAIL_ADDRESS>");
            options.Headers.Add(@"ClassY.h");

Umm. In tests the exact same thing is done via GeneratorTest.cs
So, probably won't make much difference. I'll to test anyway.

@tomspilman could you perhaps gradually decrease your real code to a test case?

@ddobrev - Will do...  should have a test case for you in an hour or so.

@tomspilman thank you very much. @genuinelucifer could you please try with @tomspilman 's test case when it's been completed?

Hello @tomspilman . Have you had any success with the test case?

Sorry... got sucked into another problem.  I'll get you a test case tonight.

@tomspilman there apparently are some specifics in your code which cause the error. I am afraid we cannot proceed with this issue without a test case. So feel free to let us know when you have it.


Layers of niftis don't sync

Hi,
when using layers of nifti-files in cornerstone I get very mixed results:

scrolls only one series
displays the wrong series

what is the right way to use layers with cornerstone-nifti-loader provided volumes? Any example greatly appreciated
Thank you

me too!
do you slove it?


Update feedstock for v2.7.0

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[x] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

@conda-forge-admin, please rerender

@conda-forge-admin, please rerender

@conda-forge-admin, please rerender

@conda-forge-admin, please rerender

@conda-forge-admin, please rerender

@conda-forge-admin, please rerender

@conda-forge-admin, please rerender

conda smithy recipe-lint .

@conda-forge-admin, please rerender

@conda-forge-admin, please rerender

@conda-forge-admin, please rerender

@conda-forge-admin, please rerender

Skipping linux for v2.7.0 until we can figure out vc issues


Facebook: Add support for param "quote"

The Facebook sharer supports an optional "quote" param.
Example: https://www.facebook.com/sharer/sharer.php?u=https%3A%2F%2Fgithub.com%2Fnostadt%2Fphp-share-url%2F&quote=Share on social networks
AC:

Param "quote" is supported and optional.

https://stackoverflow.com/questions/20956229/has-facebook-sharer-php-changed-to-no-longer-accept-detailed-parameters


[Suggestion] Send to chat (like a normal compendium roll)

I've set up two 1d10000 tables of random effects, and wanted to use tabletags to essentially roll a 1d20000. Foundry in general seems to struggle with the size of these tables if you open the GUI by accident, so breaking up into multiple tables and using tabletags sounded like a great idea.
Unfortunately, it looks like tabletags only sends the results to the place the macro was executed, so you don't get the nice chat card or see die rolls, etc.

Tabletags does return a string of the results that you can push as a chat message. Does this answer that question?
Also, as long as you don't roll in silent mode, there should be a table roll visible in chat. For instance, when I roll the below macro, I get two chat messages. Maybe I am not understanding something here...?
`let result = await TableTagsAPI.TableTagRoller(["Dungeon Dressing"]);
ChatMessage.create({
user: game.user._id,
speaker: ChatMessage.getSpeaker({token: actor}),
content: result
});`


Editor: browser finds only textual matches visible in window

If I use the browser's search functionality (Ctrl/Cmd+F), only those matches are found which are visible (located in the editor section that is currently visible in the browser window).

This is due to CodeMirror's flywheel implementation, which enables it to render very large files without taking a severe performance hit. There is a find addon for codemirror that addresses this problem. We'd be happy to receive a PR from any interested users incorporating this, otherwise we'll return to it ourselves after we've improved the reliability of the editor.

I'd like to take a stab at this!

@bookwormgirl910 Are you still working on this, or can I take it up?

@christopherdumas Sorry for leaving this! I haven't had time to work on it, so yes, please feel free!

Okay! I got a few projects taken away from me on the rust compiler, so I didn't want to do that to anyone else. (:

Aw that's very nice of you, I appreciate it! ☺ Have fun and good luck!


2.0 iOS Framework

I’ve refactored the project to support frameworks (the project is now simply an iOS framework), along with Carthage.
A few notes:
CocoaPods support is tanked, but should be pretty easy to regain.
Deployment target is now 8.0, the lowest supported by iOS dynamic frameworks.
I’ve also converted RBQFetchedResultsController (will send a separate PR) and have tested that both work using Carthage.

Woops, I had incorrectly thought you were also behind RBQFetchedResultsController.

There is a subtlety in the case of RBQFetchedResultsController, though (you can check out the PR on that project) so they may not accept the PR right away.
May affect you accepting this PR too...

I am the same guy behind both. RBQFRC was just built for my company Roobiq hence why it lives there.

Great, that should make resolving all the hiccups easier.

I've updated this to use the latest RBQFetchedResultsController.
There is one minor backwards-compatibility issue: the project/module is now called RealmMapViewFramework rather than RealmMapView.
It's because of a name-conflict issue / possible Swift bug. Check out the comments on the dde08a0 commit.
CocoaPods support is still needed, I'd appreciate if you could handle that end because I've never done that. Shouldn't be any different from usual.
Let me know if I can help somehow.

I am confused on why Cocoapods is broken... can you explain why the previous setup with Cocoapods won't work with your changes?

Closed in exchange for: https://github.com/bigfish24/ABFRealmMapView/commit/4cc389f9d4e5a5545e0071f04547e8f17eeb2dd2