Apotheosis crashed at launch

I tried launching but then crashed when launching, I dont have any incompatible mods installed https://paste.gg/p/anonymous/21294c659bda42919b826b7521f45a59

That log appears to be truncated, you'll need to provide the full log for assistance.

here is the full debug log https://paste.gg/p/anonymous/967bd56c529142e7a050e84d3729692d

Hm, there's still not an error in that whole log, so I'm not sure what's going on.
Technically speaking there are more incompatible mods than the ones that I know of.
You may want to see if the issue is fixed with 5.1.1, you might be having the same problem as #514

crashed at launch again, sent me back to launch tab https://paste.gg/p/anonymous/8f52b81ae8de4a8088c4ad0f9b060690

Yeah there's still not a crash in that report; you probably have a mod conflict that is arising from some sort of ASM/Mixin level issue.  The only way to track that down is to remove mods until it stops, and then source the conflicting mod.

could it be some enchantment conflicts or anvil conflicting or certain category

i do also have some mods that changes mob spawners, but i have no enhanced spawner

Identified to be an incompatibility with https://www.curseforge.com/minecraft/mc-mods/xpisnoobject


[WIP] Added Async executor for SecurityChecks

Refactored SecurityCheckExecutor class
Motivation
Current code allows only synchronous security check execution.
Description
Added a way to execute the checks asynchronously
Progress

[x] Add AsyncSecurityCheckExecutor
[x] Refactor current SecurityCheckExecutor
[x] Update current unit tests
[x] Add unit tests

@wei-lee Yes, but I think it is better to give the user the chance to change the executor as well. So, if you agree, I could use the AppExecutor as default, unless the user specifies a custom executor.
WDYT?

@ziccardi can you think any reason why they may want to change the executor at all? Also I think if that is the case, that should be configured for the whole lib (thus it should be the job of the core module). I don't think each individual module should allow that.

@wei-lee Sorry I read too fast. You want to get rid of the AsyncExecutor class.
Well, I think sync executor and async executor are two different objects, so I prefer to model them on two different classes, both inheriting from the same base class.

@wei-lee there are three main reasons I think it could be useful:

You can never know what is going to happen in the future
You can never know what the user will need
The user could implement a custom SecurityCheck that needs very long time, so he wants to use a custom executor to avoid blocking other checks.

I think sync executor and async executor are two different objects

I'd agree here, that decoupling them does make sense, instead of smacking new methods on different concerns to the orignal clazz


Get image size after marking component as mounted

Image size was not set to state on initial render. Fixed it by adjusting call order.

UP! Merge it, please!

Hello, any update on this? I found this fix necessary to render images with custom sizes.
I had to fork this repo until this gets fixed.

@systemride We are in the process of handling a lot of pending issues. I will get merged at one point.


Add typings section to package.json

enable support for typings lookup for npm package

When user has header file and installs npm package with --save, then the header file is duplicated and it occurs the duplicated identifier error.
typings install typescript-stl
npm install --save typescript-stl

You don't need to do typings install typescript-stl with this PR.

Need not to use the typings, it's right if only TypeScript-STL is being used.  However, if there's a library referencing my TypeScript-STL and I enrolled the typescript-stl.d.ts into npm package, then the duplicatation problem must occurs.
Most of TypeScript header files are start with referencing its dependencies. If such dependent relationship exists, then enrolling header file into npm package must cause the critical duplication problem.
/// <reference path="../react/react.d.ts" />
/// <reference path="../jquery/jquery.d.ts" />
/// <reference path="../typescript-stl/typescript-stl.d.ts" />

That triple comment style is also depreciated. You can remove all /// <reference path...  lines, remove typings/ and do:
npm install<EMAIL_ADDRESS>npm install typescript-stl
npm install @types/jquery
npm install @types/react
This is the way typescript has gone... see: https://blogs.msdn.microsoft.com/typescript/2016/06/15/the-future-of-declaration-files/

Amazing. Thanks for giving me such important news

The reference style inclusion is no longer used, and was replaced with the more standards compliant ES6 module import/export notion which was added in TS version 1.6 I think. I think most TS libs are moving towards self describing modules as then you dont need to do lots of hoop jumping.

PR #2 is merged.
Definition of the latest TypeScript-STL is provided only in TSD yet, but @types also will be provided soon. Thanks for your contribution.

Now, using @type is possible.
npm install --save @types/typescript-stl


cleanup sys.meta_path between tests to fix core failures (#4692)

Pull request opened by the merge tool on behalf of #4692

Merged into branches iso5 in f594a4b9a1668d4c0eb423be10e6399c0ab2e346


Module is not loaded with ng cli v11.1.1

After upgrading to @angular/cli to version 11.1.1 this module is not loaded due to the backslash presents in package.json for main and typings
     "main": "src\\index.js",     "schematics": "./collection.json",   "builders": "./builders.json",   "dependencies": {     "@nrwl/workspace": "^10.3.0",     "ngx-build-plus": "^10.0.0",     "cross-spawn": "^7.0.3",     "json5": "^2.1.3"   },   "typings": "src\\index.d.ts",   "peerDependencies": {} }
Changing to
"main": "src/index.js",   "schematics": "./collection.json",   "builders": "./builders.json",   "dependencies": {     "@nrwl/workspace": "^10.3.0",     "ngx-build-plus": "^10.0.0",     "cross-spawn": "^7.0.3",     "json5": "^2.1.3"   },   "typings": "src/index.d.ts",   "peerDependencies": {} }
makes the module loadable again.
If agree, I can file a pull request for this fix.

The issue arise only on macos and linux machines, not in window machines.

@lincetto can you create a pull request? This is a breaking change also for me

Unfortunately no, because this repo seems not aligned to the nom package published here
https://www.npmjs.com/package/@angular-architects/module-federation
Indeed the package version is 1.2.3, here in the repo the source code refers to version 0.0.0, dependencies are not aligned with the one included in package.json inside nom package, etc..
@manfredsteyer is the source code available in order to patch it?

For a quick fix in my pipeline, I'm using the following bash commands
Mac:
sed -i '' 's/src\\\\index.js/src\/index.js/g'<EMAIL_ADDRESS>Linux:
sed -i 's/src\\\\index.js/src\/index.js/g'<EMAIL_ADDRESS>


Form inputs are not reset

<input ... v-model="someInput" ...>

data: function () {
  return {
    someInput: ''
  }
}

Enter some value and run later this.someInput = ''.
On application refresh, someInput will still have entered value.

Fixed with 1.4.2


gradle release v3.4.0

To be merged after the release is complete.

Released:  https://plugins.gradle.org/plugin/com.google.cloud.tools.jib


Generic struct equality comparison

If all struct fields are known, then deep equality can be derived.

How would this work for discriminated unions?


double fire event change if click on label

hi, i write this code and when i click on label text event change is fire two times. if i click on butto or i try to swipe it's work correctly
in chrome work ok, but in FF37 don't work correctly
version of
jquery 1.11.1
bootstrap 3.2.0
bootstrap-switch 3.3.2
how i can resolve it?
<input id="SwitchImageActive" type="checkbox" value="ImageActive" name="ImageActive" data-off-color="danger" data-on-color="success" data-label-width="auto" data-label-text="Image enabled">

$(function(){
    $("#SwitchImageActive").bootstrapSwitch();
        $("#SwitchImageActive").bootstrapSwitch().on('switchChange.bootstrapSwitch', function (event, state) {
        alert( $(this).prop("checked") );
        //alert( $(this).val() );
        //alert(event);
        // alert(state);
        });
});

I'm having this same issue on the Galaxy S3 browser (Android 4.3). It happens on http://www.bootstrap-switch.org/ as well.

@valix85 if i rewrite your code like following
$(function(){
  $("#SwitchImageActive").bootstrapSwitch({
    onSwitchChange: function (event, state) {
      alert($(this).prop("checked"));
    }
  });
});

would it still trigger the problem? can you verify?

i try your code but problem persist, always double alert ONLY if i click in label space, if i click on color button(red or green area) doesn't fire double time.
if i try in chrome it's work correctly but if i try in firefox the event is fired two times, before and after cheange

Having the exact same issue. Clicking on the ON/OFF text will fire the switchChange.bootstrapSwitch event correctly. Although, if I click on the "label" portion of the switch, the event will fire twice. For example, if the switch is on the ON position (true), and you click the label, the event will fire twice, first returning a state of TRUE, and then FALSE. Otherwise, if you click on the ON/OFF text, it will just return a state of FALSE. Having the problem with any version of Firefox and mobile devices.
This is a HUGE problem if you are relying on the state value when the event is fired, not to mention how inefficient it is. If you are using the switch to update a value in a database, it will perform the update twice -- first with the incorrect value.
Any ideas would be greatly appreciated. Was going to use bootstrap switch on a large extranet project, but will cause major issues with the problem described above.

Looks like the issue is related to the "mouseleave" event under the bootrstapSwitch _labelHandlers. When the label portion of the switch is clicked, it triggers the "mouseup" event AS WELL as the "mouseleave" event. Problem is, the "mouseleave" event calls the mouseup event, thus creating a type of loop that causes the switchChange.bootstrapSwitch event to fire twice. Anyways, here's my quick and solution to the problem (until a permanent fix is implemented):
    var BootstrapSwitch;
    BootstrapSwitch = (function() {
      function BootstrapSwitch(element, options) {
        if (options == null) {
          options = {};
        }
        this.$element = $(element);
        this.clckevt = false; // <---- ADD THIS HERE!!!!!
        this.options = $.extend({}, $.fn.bootstrapSwitch.defaults, {
          state: this.$element.is(":checked"),
          size: this.$element.data("size"),

Under _labelHandlers,  change "mouseup" to custom "switcher"
"mouseup.bootstrapSwitch touchend.bootstrapSwitch": (function(_this) { ... } //Change this
"switcher.bootstrapSwitch touchend.bootstrapSwitch": (function(_this) { ... } //To This

Add "click" to _labelHandlers:
          "click.bootstrapSwitch": (function(_this) { //Added for bug fix
            return function(e) {
              _this.clckevt = true;
              return _this.$label.trigger("switcher.bootstrapSwitch");
            };
          })(this),

Edit "mouseleave" under _labelHandlers:
          "mouseleave.bootstrapSwitch": (function(_this) {
            return function(e) {
              if(!_this.clckevt){ //Added for bug fix.
                return _this.$label.trigger("switcher.bootstrapSwitch");
              }
              _this.clckevt = false;
            };
          })(this)

The above changes creates a "click" event which prevent the "mouseleave" event from firing when the "label" is simply just clicked. The "clckevt" variable resets itself, allowing the slider to function normally when the label is not being clicked.  Hopes this helps anybody having the same issue I was having!!!

Thanks!

@WhistlerHusky this reply might be a bit late, but I am afraid this fix, if applied, has not completely happened.
I have been able to fix this problem by modifying one single line. The project owner might want to evaluate to update the changes. If I have time I will prepare a proper pull request.

@tk421 do you have a solution in the meanwhile? I am working also with a database and actual devices. Having the problem, that the device is switching ON and OFF by itself and never stops. My workaround at the moment is to limit the accepted inputs with a time delay. So the second input will be disregarded. Does anyone have a propper solution?
Thanks!


Implement DRONE_RPC_SKIP_VERIFY for drone-agent and drone-controller

Background
The DRONE_RPC_SKIP_VERIFY is described in the https://docker-runner.docs.drone.io/installation/reference/drone-rpc-skip-verify/ for docker runner. But it has not been implemented in the source code.
This env variable can be useful when the drone server is deployed behind the proxy with dynamically generated self-signed certificates. And adding the certs to the trusted roots of the drone-agent (docker runner) is a complicated operation overhead.
How the changes have been tested
All the unit test cases passed.
Setup the local testing environment

Clone the repo

git clone https://github.com/mdluo/drone
cd drone

Generate self-signed certificate (refer to https://letsencrypt.org/docs/certificates-for-localhost/):

openssl req -x509 -out tmp/localhost.crt -keyout tmp/localhost.key \
  -newkey rsa:2048 -nodes -sha256 \
  -subj '/CN=localhost' -extensions EXT -config <( \
   printf "[dn]\nCN=localhost\n[req]\ndistinguished_name = dn\n[EXT]\nsubjectAltName=DNS:localhost\nkeyUsage=digitalSignature\nextendedKeyUsage=serverAuth")

Prepare nginx config (tmp/nginx.conf).

server {
  listen 443 ssl;
  ssl_certificate /etc/nginx/conf.d/localhost.crt;
  ssl_certificate_key /etc/nginx/conf.d/localhost.key;
  location / {
    proxy_pass http://drone-server;
  }
}

Create docker-compose.yml

version: '3'
services:
  nginx:
    image: nginx
    volumes:
      - ./tmp:/etc/nginx/conf.d
    ports:
      - 443:443
  drone-server:
    image: drone/drone
    environment:
      - DRONE_AGENTS_ENABLED=true
      - DRONE_GITHUB_SERVER=https://github.com
      - DRONE_GITHUB_CLIENT_ID=<github-client-id>
      - DRONE_GITHUB_CLIENT_SECRET=<github-client-secret>
      - DRONE_SERVER_HOST=drone-server
      - DRONE_SERVER_PROTO=http
      - DRONE_RPC_SECRET=<rpc-secret>
  drone-agent:
    image: golang:1.13.0
    environment:
      - DRONE_LOGS_DEBUG=true
      - DRONE_LOGS_TRACE=true
      - DRONE_SERVER_HOST=nginx
      - DRONE_SERVER_PROTO=https
      - DRONE_RPC_SECRET=<rpc-secret>
      - DRONE_RPC_SKIP_VERIFY=false
    working_dir: /var/drone
    command: go run cmd/drone-agent/main.go
    volumes:
      - /var/run/docker.sock:/var/run/docker.sock
      - .:/var/drone

Start with docker-compose up

For now the DRONE_RPC_SKIP_VERIFY is set to false, so I'm getting error messages:
drone-agent_1   | {"level":"debug","msg":"successfully pinged the docker daemon","time":"2019-10-06T04:59:27Z"}
drone-agent_1   | {"arch":"amd64","level":"debug","machine":"13bdcd2bcdde","msg":"runner: polling queue","os":"linux","time":"2019-10-06T04:59:27Z"}
drone-agent_1   | 2019/10/06 04:59:27 [DEBUG] POST https://nginx/rpc/v1/request
drone-agent_1   | {"arch":"amd64","level":"debug","machine":"13bdcd2bcdde","msg":"runner: polling queue","os":"linux","time":"2019-10-06T04:59:27Z"}
drone-agent_1   | 2019/10/06 04:59:27 [DEBUG] POST https://nginx/rpc/v1/request
drone-agent_1   | 2019/10/06 04:59:27 [ERR] POST https://nginx/rpc/v1/request request failed: Post https://nginx/rpc/v1/request: x509: certificate is valid for localhost, not nginx
drone-agent_1   | 2019/10/06 04:59:27 [DEBUG] POST https://nginx/rpc/v1/request: retrying in 1s (30 left)
drone-agent_1   | 2019/10/06 04:59:27 [ERR] POST https://nginx/rpc/v1/request request failed: Post https://nginx/rpc/v1/request: x509: certificate is valid for localhost, not nginx

Change the DRONE_RPC_SKIP_VERIFY to true and restart

drone-agent_1   | {"level":"debug","msg":"successfully pinged the docker daemon","time":"2019-10-06T05:00:29Z"}
drone-agent_1   | {"arch":"amd64","level":"debug","machine":"4ca57c88486d","msg":"runner: polling queue","os":"linux","time":"2019-10-06T05:00:29Z"}
drone-agent_1   | 2019/10/06 05:00:29 [DEBUG] POST https://nginx/rpc/v1/request
drone-agent_1   | {"arch":"amd64","level":"debug","machine":"4ca57c88486d","msg":"runner: polling queue","os":"linux","time":"2019-10-06T05:00:29Z"}
drone-agent_1   | 2019/10/06 05:00:29 [DEBUG] POST https://nginx/rpc/v1/request
drone-agent_1   | 2019/10/06 05:00:59 [DEBUG] POST https://nginx/rpc/v1/request (status: 524): retrying in 1s (30 left)
nginx_1         | <IP_ADDRESS> - - [06/Oct/2019:05:00:59 +0000] "POST /rpc/v1/request HTTP/1.1" 524 25 "-" "Go-http-client/1.1" "-"
nginx_1         | <IP_ADDRESS> - - [06/Oct/2019:05:00:59 +0000] "POST /rpc/v1/request HTTP/1.1" 524 25 "-" "Go-http-client/1.1" "-"
drone-agent_1   | 2019/10/06 05:00:59 [DEBUG] POST https://nginx/rpc/v1/request (status: 524): retrying in 1s (30 left)
nginx_1         | <IP_ADDRESS> - - [06/Oct/2019:05:01:29 +0000] "POST /rpc/v1/request HTTP/1.1" 499 0 "-" "Go-http-client/1.1" "-"
drone-agent_1   | 2019/10/06 05:01:29 [ERR] POST https://nginx/rpc/v1/request request failed: Post https://nginx/rpc/v1/request: context deadline exceeded
drone-agent_1   | {"arch":"amd64","level":"debug","machine":"4ca57c88486d","msg":"runner: polling queue","os":"linux","time":"2019-10-06T05:01:29Z"}
drone-agent_1   | 2019/10/06 05:01:29 [DEBUG] POST https://nginx/rpc/v1/request
drone-agent_1   | 2019/10/06 05:01:29 [ERR] POST https://nginx/rpc/v1/request request failed: Post https://nginx/rpc/v1/request: context deadline exceeded
drone-agent_1   | {"arch":"amd64","level":"debug","machine":"4ca57c88486d","msg":"runner: polling queue","os":"linux","time":"2019-10-06T05:01:29Z"}
drone-agent_1   | 2019/10/06 05:01:29 [DEBUG] POST https://nginx/rpc/v1/request

Now the drone-agent connect to the drone-server successfully.
Tested with GitHub pull request pipeline
Use ngrok as GitHub webhook tunnel
ngrok http https://localhost

If the DRONE_RPC_SKIP_VERIFY is set to true, the pipeline can be finished successfully.

If the DRONE_RPC_SKIP_VERIFY is set to false, the pipeline gets stuck at the first step and can never move forward.

Notice the console output of the second build, the drone-agent is not even connected to the drone server, so there should be no output at all.
Looks like it's syncing from the first build, it might be a bug.

 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.

@tboerger Thanks for the review.
I agree that it's not complicated, maybe I need to adjust my description. On the one hand, it's already described in the documentation but not implemented, on the other hand, it can be helpful for those who want to deploy drone agent within the same private network (which drone server was deployed) but don't have access to get or set self-signed cert of the drone server.
I did another experiment, seems I have to create another docker image on top of drone/agent in order to apply the self-signed cert. Which is not as straightforward as just running the original drone/agent image with one extra env variable.
Dockerfile
FROM drone/agent
RUN apk add -U --no-cache ca-certificates
COPY ./tmp/localhost.crt /etc/ssl/certs/
RUN update-ca-certificates

docker-compose.yml
version: '3'
services:
  nginx:
    image: nginx
    logging:
      driver: none
    volumes:
      - ./tmp:/etc/nginx/conf.d
    ports:
      - 443:443
  drone-server:
    image: drone/drone
    environment:
      - DRONE_AGENTS_ENABLED=true
      - DRONE_GITHUB_SERVER=https://github.com
      - DRONE_GITHUB_CLIENT_ID=<github-client-id>
      - DRONE_GITHUB_CLIENT_SECRET=<github-client-secret>
      - DRONE_SERVER_HOST=drone-server
      - DRONE_SERVER_PROTO=http
      - DRONE_RPC_SECRET=<rpc-secret>
  drone-agent:
    build: .
    environment:
      - DRONE_LOGS_DEBUG=true
      - DRONE_LOGS_TRACE=true
      - DRONE_SERVER_HOST=nginx
      - DRONE_SERVER_PROTO=https
      - DRONE_RPC_SECRET=<rpc-secret>
    volumes:
      - /var/run/docker.sock:/var/run/docker.sock

Back when I used self signed certificates I have simply mounted a ca-certificates.crt file from the host system via volume to the agents, that's working really well.
But from the usability perspective a separate flag is easier. But especially if the scm is running on a self signed certificate it still gets more complicated as the clone step doesn't provide any skip verify flag, beside that it got to be configured for every pipeline.
For that use case it's much better to provide a ca-certificates.crt file that includes the custom root certificate.

Yes, I found the solution, just exporting the root CA and mount it under /etc/ssl/certs (no need to replace ca-certificates.crt)

I really appreciate the pull request, however, we actually moved the Docker runner (agent) and Kubernetes runner (controller) to their own separate repositories and skip verify is implemented [1][2]. The agent and controller are now deprecated and are being removed.
[1] https://github.com/drone-runners/drone-runner-kube/blob/master/command/daemon/daemon.go#L68
[2] https://github.com/drone-runners/drone-runner-docker/blob/master/command/daemon/daemon.go#L69


Fixup transaction dates with February 30

Hi,
I've encountered a very strange problem with statements of my bank. Some transactions were dated on February 30.
As it turns out, German banks use a variant of the 30/360 interest rate calculation system (see here if interested, where each month has exactly 30 days. This can lead to transactions that are dated on February 30, which are not accepted by many date handling software, such as Python's datetime module.
I've added a statement pre-processor which sets the date of transactions with an original date of February 30 to the last day of February (February 28 or February 29 depending on leap years).
I was not sure in which directory new test statements should be placed, so I created a new one. There are already three directories (betterplace, cmxl and jejik) with test statements, but these seem to be based on other mt940 libraries.

Hi Sebastian,
The test directories are simply tests copied from other parsers so I could be sure that all code is tested as well as possible. The local tests are simply outside of that.
Thanks for the patch, looks really great!


Repeating Absences

Feature Request:

Allow the creation of Repeating Absences on a Weekly / Bi Weekly Frequency
With a Repeat Until end date.

Any updates on if you think this would be possible?


workaround dune -fPIC issue

see https://github.com/ocaml/dune/issues/5809
I only saw the problem with ocaml 4, not ocaml 5 which ocannl requires anyway, so maybe something was fixed upstream, but technically someone could use this library independently with ocaml 4, idk if you want to merge this, just for posterity

Thank you for your interest! There's nothing in these bindings that requires novel OCaml features, so it will definitely help relax the requirements! I'll cycle back to ocaml-cudajit once ocaml-gccjit is released on opam repo.


⚠ Bofrost.de has degraded performance

In 62aaa62, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1330 ms

Resolved: Bofrost.de performance has improved in 4f928fb.


The g.dart file generated by command isn't correct when result class is nullable

Describe the bug
The g.dart file generated by command isn't correct when result class is nullable
To Reproduce
Steps to reproduce the behavior:

Set class type to nullable. (bool?)
Run command flutter pub run build_runner build

Expected behavior

Set class type to nullable. (bool?)
Run command flutter pub run build_runner build
Change all types to nullable. (bool?)

Screenshots

When I changed the class to nullable, but the g.dart file generated by command isn't correct

Only the first circle in this screenshot is correct, the others should be the same. (bool?)

If not, it will be wrong on line 896, because the json data is null

Here is the result data

Additional context
When I set all circles in pic2 to bool?,  my code will be correct, could you please check the generate command, thanks a lot.

When will this be fixed ?

Any plans to address this?

When will this be fixed ?


why write this instead of using ccxt or https://github.com/JKorf/CryptoExchange.Net ?

https://github.com/JKorf/CryptoExchange.Net

Hi @llvm-x86, I don't use C# :)

Hi @llvm-x86, I don't use C# :)

Sorry let me speak honestly for you.
hugomorg: "Hi @llvm-x86 sorry I'm a retarded communist."


winsparkle website link is wrong

in the readme file the website link to winsparkle it should be .org not .com
https://winsparkle.org/

Updated


When Can I Get the Dataset?

Dear authors, thank you for your sharing. I have submitted the licensing agreement. When will you send me the complete dataset?

Hi@ErinWang2023 , thank you for your interest in OphNet! We have successfully received your application. We are currently developing OphNet V2, which will feature more accurate and diverse labels, a larger volume of videos for the upcoming challenge, and will provide more baseline results. Unfortunately, to prevent data leaks, we have to delay the data release plan. We have recorded your information and will contact you once the final data preparation is completed. Please stay tuned to OphNet and the upcoming challenge.

@ErinWang2023 Thank you for your interest in OphNet. The dataset is now open-sourced; please refer to Dataset Preparation. If you have any other questions, feel free to leave a message.

谢谢你哦！！

Could you give me the Baidu Netdisk link, please?  I can't download from the huggingface, it always stop.

@ErinWang2023 Hi, I’ve updated the BaiduNetdisk download link. Additionally, HF-Mirror is a great alternative.

谢谢你哦！！


Support streaming calls from DocService

It seems like we are filtering non-unary calls intentionally in UnframedGrpcService.
https://github.com/line/armeria/blob/de80c40588b9d3c4fb1d1791a2e798de0295f013/grpc/src/main/java/com/linecorp/armeria/server/grpc/UnframedGrpcService.java#L114-L118
I want to be able to run streaming calls from DocService for simple debugging tasks without worrying about performance. According to my tests, streaming input calls are partially working but only a single payload is processed. For streaming output calls, response hangs.
Let me know what you think? We can probably include a header for DocService to bypass this limitation without changing the actual behavior for production use 🙂

It sounds useful. But that said, I'm unsure what serialization format should be used to send or serve JSON streaming messages. JSON Text sequences?

It sounds useful. But that said, I'm unsure what serialization format should be used to send or serve JSON streaming messages. JSON Text sequences?

Not sure if it is user-friendly to use text sequences. Concatenated JSON also looks favorable to me.
For doc service, I don't think we need the full streaming capability tho. We can just prepend requests to a JSON array from DocService and fetch all responses via an array back.


What about docs?

[ ] #7
[ ] #8
[ ] #9
[ ] Decide if we want to go with something more complex

Hey there, I wonder what we should do for documentation on this project. Any thoughts?

[ ] Create a docs folder
[ ] Create a new index.md document
[ ] Turn on GitHub Pages
[ ] Decide if we want to go with something more complex

Sample comment text


pub crashes or freezes when using dependency_overrides (Dart 1.7-dev.4.5)

This issue was originally filed by<EMAIL_ADDRESS>
I'm trying to upgrade my applications to Dart 1.7, but I'm having some problems when using the dependency_overrides option with the latest sdk.
To better illustrate these problems, I've created a github repository that simulates my development enviroment (It's just a bunch of folders with pubspec.yaml files):
https://github.com/luizmineo/dart-pub-errors
When running pub get in error1/test, it shows the following message:
Changed 1 dependency!
The null object does not have a method 'map'.
NoSuchMethodError: method not found: 'map'
Receiver: null
Arguments: [Closure: (dynamic) => dynamic]
This is an unexpected error. Please run
    pub --trace 'get'
and include the results in a bug report on http://dartbug.com/new.
Running the pub --trace 'get' command produces the attached error.
The second error is more complicated. If you run pub get in error2/test, it won't crash, but will hang for about 20 minutes, and exits with the follwing message:
Resolving dependencies... (13:27.3s)
Connection closed before full header was received

Attachment:
error_log.txt (7.84 KB)

This issue has been moved to dart-lang/pub#1163.


added 11ty-gallery

Hey! creating a pr for an 11ty starter I've made.

😍
<meta name="generator" content="Eleventy v1.0.2">


The --until flag does not seem to work

Thank you very much for this script.
I ran the script a few days to gather data about bitcoin but this doesn't yield any results.
The query i use is:
python3 Exporter.py --querysearch "bitcoin" --since 2016-03-31 --until 2016-4-1
When i remove the --until flag it works perfectly but it starts to gather data from today on (if i could reverse this i could stop the script by hand if i got the data i wanted but it starts today and works its way down)
Is there anything i'm missing or is it a bug?
Cheers
Edit:
I saw a lot of closed issues about this problem, any idea if it's a periodic problem?

Hi woutster,
Did you try with "--until 2016-04-01"?
I'm running it that way with python 2.7 and it works perfectly.
Hope it helps!

I'm having the same issue. I tried what @jtmendez suggested but I got an error when put the quotes. My command is like this:
python3 Exporter.py --querysearch "alckmin" --since 2014-07-06 --until 2018-10-05 --output "/mypath/tweets.csv" --maxtweets 1000
The error I got is UnboundLocalError: local variable 'outputFile' referenced before assignment

@aiquis
As mentioned, you probably won't get support for Python 3 / got3 related problems.
I'm using 2.7.13 and it works with the query below:
python Exporter.py --querysearch "Bitcoin" --since 2017-12-10 --until 2017-12-11 --toptweets --output "output_1210-to-1211.csv"


FancyZones 0.14.0 forces File > Open dialog (e.g., Notepad) to app's zone size

Environment
Windows build number: Windows 10.0.18363.476.
PowerToys version: 0.14.0
PowerToy module for which you are reporting the bug (if applicable): FancyZones

Steps to reproduce

File-Open

Open Notepad.exe (or another application that uses a File-Open dialog).
Use Shift+Drag to move Notepad.exe into a zone.
Select File > Open.

File-Save

Open Notepad.exe.
Use Shift+Drag to move Notepad.exe into a zone.
Select File > Save As....

Expected behavior

File-Open
Notepad's File-Open dialog should be the same size as the previous time it was opened.
File-Save
Notepad's File-Save dialog should be rendered correctly (so that the filenames are visible).
Actual behavior

File-Open
Notepad's File-Open dialog is the same size as the Notepad window and its display seems to have been prevented from rendering.
File-Save
Notepad's File-Save dialog's display seems to have been prevented from rendering.
Screenshots

File-Open

File-Save

Hi @skst
thank you for reporting the issue.
Do you have the Move newly created windows to their last known zone option ON?

Oops! I didn't check for new options. Yes, that option is selected. And, yes, when I disable that option, the File-Open and File-Save dialogs are created and displayed as expected.
These are the settings when the problem shows itself.

OK thank you for confirming, I was able to reproduce it.
That option has been really problematic with so many apps, we will try to fix it once and for all.
Thanks.

@enricogior  does this happen with all applications and modal windows like save / open?

Was also able to reproduce, and issues went away after changing the above settings.
In addition to file open and save, I also experienced rendering issues in outlook when changing folders after snapping to a fancy zone. The app was doing a strange re-draw initially upon snapping, whereby the text would cycle through multiple scales before settling on the correct one and rendering correctly. When a different folder is then selected, much of the text would disappear as with the the open/save dialog. This too has gone away when I disabled the above setting.

@yuyoyuppe https://github.com/microsoft/PowerToys/issues/840 looks like a duplicate of this

So as https://github.com/microsoft/PowerToys/issues/507

Reassigned, since @bzoz has already some idea how to fix it

@skst @JusticeForBeyonce could you try our new DLL and see if the problem persists. Here are the instructions and the download link.

@skst @JusticeForBeyonce could you try our new DLL and see if the problem persists. Here are the instructions and the download link.

Works perfectly now even with the "Move newly created windows to their last known zone" option enabled.

Looks good! I tested File-Open and File-SaveAs on Notepad and a couple other apps I had the problem with. Thanks for the quick turnaround!

The new release is available https://github.com/microsoft/PowerToys/releases/tag/v0.14.1


[Feature](inverted index) add inverted index reader memory size into searcher cache

Proposed changes
Issue Number: close #xxx

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall


Optimize memory usage

Under certain write workloads, when there's a lot of commits with a lot of small value insertions, the memory used by the index overlay can be blown by a factor of 10 or more when compared to the actual data inserted.  There's a good example in  #93. This is because the overlay keeps the whole index page in memory, even though only a single entry in the page may be modified.
I can think of two ways to fix that:

Don't keep the whole page, but rather only modified entries.

Investigate existing mechanisms in linux to control page flushing for memory mapped files. If it is possible to control when exactly individual pages are flushed to disk, we don't need the overlay in the first place.

In any case, this is only worth fixing if the solution does not introduce any performance hits.

Hey! memmap2 crate which is used supports flushing memory regions - memmap2::MmapMut::flush_range.
Could you please point me to overlays which won't be needed after making use of that?

flush_range won't be much of use here. What we actually need is something that prevents flushing until explicitly
requested.
This is how the database guarantees consistency at the moment, simplified.

Each commit results in a set of pages that need to be updated in memory-mapped files.
These pages are copied to heap-allocted memory overlay and modifications are performed there.
The database writes the pages to a write-ahead log and flushes it to disk.
The database then reads the log and applies changes to the memory map.
The heap-allocated memory overlay is released.

We can't modify the pages in the memory-mapped regions initially because we don't know when the kernel decides to flush them to the disk. We need to make sure that the modified pages can't be partially persisted before the WAL is flushed. This guarantees that upon recovery any partial-written page sets will be fixed by replaying the WAL.
So here's how it could work ideally:

Each commit results in a set of pages that need to be updated in memory-mapped files.
Pages that are about to be modified are locked in memory, so that they are not flushed to disk by the kernel.
These pages are modified in the memory-mapped address space directly.
The database writes the pages to a write-ahead log and flushes it to disk.
The database then unlocks the pages and allows the kernel to write them to the final locatioin.

Page locking may be achievable with mlock, although the documentation is not 100% clear if locking will also prevent memory-mapped IO. So this requires some research and experimentation.

Thanks for the digest. Makes sense.
mlock(2) states that:

mlock() locks pages in the address range starting at addr and
continuing for len bytes.  All pages that contain a part of the
specified address range are guaranteed to be resident in RAM when
the call returns successfully; the pages are guaranteed to stay
in RAM until later unlocked.

munlock() unlocks pages in the address range starting at addr and
continuing for len bytes.  After this call, all pages that
contain a part of the specified memory range can be moved to
external swap space again by the kernel.

I think that means that there is a guarantee for that pages won't be flushed to disk untill munlock is called.
But I don't know if VirtualLock is an equivalent to mlock.

@arkpar What do you think of that?

Indeed, this should be the case. man page also has this section:

Memory locking has two main applications: real-time algorithms
and high-security data processing.  ...  Cryptographic security software
often handles critical bytes like passwords or secret keys as
data structures.  As a result of paging, these secrets could be
transferred onto a persistent swap store medium, where they might
be accessible to the enemy long after the security software has
erased the secrets in RAM and terminated.

So locked pages should not be copied to disk.
VirtualLock has the same guarantee:

Pages that a process has locked remain in physical memory until the process unlocks them or terminates. These pages are guaranteed not to be written to the pagefile while they are locked.

There are a few issues that still need clarification

Locked page limit. Default limits on the number of locked pages seems to be rather small. It should be possible to raise the limit on startup though with setrlimit or SetProcessWorkingSetSize but it is not clear what would be the maximum for a typical Linux/Mac/Windows system.
Page size alignment. Various systems may be configured for different page sizes. Database index currently operates on 512-byte pages. If the kernel is configured to use huge page sizes, this may become an issue. Although I imagine systems that use that option usually have lots of RAM and are fine with using more of it.


Merge CMake Rework

Adds improved cmake functionality, along with one-line install init.sh and updated documentation. Furthermore fixed dependency of libjluna_c_adapter and jluna.jl location.

This is a working branch and not master. Please wait until I request a review before commenting, a lot of the things you commented on were temporary fixes

I'm not sure why I can't assign you on github, but feel free to comment now @friendlyanon

merged, c.f. #15 #16

c.f. #15 #16


821_Log.final.out

Sample of STAR Log.final.out from v2.7.1a (with the Chimeric junction option set)

Thanks... I got sidetracked and was going to create a brief readme...
The only details I can give is that I ran STAR with --chimSegmentMin 20  so it has output the CHIMERIC section.  I can't give much more away about the data source.
Best regards
Oliver


Guide on Migrating Applications to Kubernetes

This is a...

[x] Feature Request
[ ] Bug Report

Problem:
People are trying to migrate their applications to Kubernetes and would like a guide on how to do it.
Proposed Solution:
There are several people interested in writing such a guide and it was suggested that should should live in the Kubernetes documentation (e.g., here). This issue is to provide a starting point to discuss and work out any logistics.
Page to Update:
I'm not sure where this multi-page guide should go. Any pointers?

Are there any suggestions on how one would start writing a multi-page guide to initially land it?

@mattfarina 👋 I'm glad to see a proposal for this! The best way to propose a migration guide is to open a pull request. Be sure to follow the docs contributor guidelines.
Based on what you're describing, I'd propose adding a new header under Setup.
Concentrate at first just on getting content up for discussion and review. Don't worry too much about failing build tests; we can address those once the content is solid.
I look forward to seeing a PR soon!

@mattfarina You'd mentioned a desire to see this localized (translated) to at least one other language as well - can you share a few more details of what you were after/hoping to achieve and wanted to create? Hopefully the tutorials template will cover most of what you want, but it hasn't been exercised heavily since it's inception, so we might need to tweak or twiddle it...

@heckj That's real feedback about the tutorials template. Honestly, I think it belongs under Setup; I'm just concerned about providing enough structure to help a writer succeed.

Thanks for all the feedback, folks.  What I was initially envisioning doesn't fit in the format for the k8s docs. Going to see what we can put into the format you have.

@mattfarina would still very much like to know what you had evisioned. i suspect you’ve spoken about it to a few people, but i haven’t caught it and would like to know - and possibly help.

/reopen
I don't think this needs to be closed just yet. We're going to see what we can do here.

Apologies - didn't mean to close this, was a mistake while I was looking at issues in a mobile browser


Wrong MatTextField text positioning when using "FullWidth"

Describe the bug
The position of the text inside a MatTextField changed from 2.4.3 -> 2.6.0 when using FullWidth="true"
To Reproduce

Use MatBlazor package 2.6.0
Create TextFields in MatBlazor with FullWidth="true"
Run project

Blazorfiddle link
Since this is an issue the depends on the version, I am not sure if a blazorfiddle link is helpful - here is a link anyway.

https://blazorfiddle.com/s/mxst4sw4

Expected behavior
I think the positioning in 2.4.3 is correct.
Screenshots

2.4.3
2.6.0

I've been able to reproduce this in one of my projects. The positioning is correct in 2.4.3, while 2.6.0 causes the same issue in my project.

The full width demo seems to be correct running v2.6.0: https://www.matblazor.com/TextField
When you moved to the latest MatBlazor, did you notice that the CSS has been separated from the JS? Now you have to include both:

I did include <script src="_content/MatBlazor/dist/matBlazor.js"></script> in 2.6.
However, I think that was the way it was included in 2.4.3 as well.

@FelixDamrau It looks like my comment was truncated. In addition to the .js script, you also now have to include the css separately:
<link href="_content/MatBlazor/dist/matBlazor.css" rel="stylesheet" />

I did include both files, the js script and the css file.
I uploaded a (somewhat) minimal blazor project that shows the error.
See https://github.com/FelixDamrau/matblazor-textfield
Repro steps

Run the linked project

Enter sth in the text field -> wrong position

clean the project directory (I used clean clean -xfd)

change version of matblazor to 2.4.3 in csproj ProjectReference

rerun

Enter sth in the text field -> correct position

Thank you for the project file. It greatly helps for debugging.
If I comment out bootstrap.min.css and app.css, then the text field displays correctly. So there is obviously some conflicting CSS being loaded. Since the point of MatBlazor is to implement the Material Design guidelines, I think loading Bootstrap would be at odds with that goal. I would start with removing Bootstrap. I would also load your own CSS after the MatBlazor CSS in case you want to override any default styles.

Thank you for your reply and time. <3
I can confirm. The text positioning is correct if I remove the bootstrap css.
The MS Blazor template uses bootstrap by default. If matblazor is/remains incompatible with bootstrap -- which sounds quite reasonable -- the removal of the bootstrap.css should be in the documentation.
I used the grid layout of bootstrap. I guess I will use the material design grid layout.
I will close this issue/bug, since it is not a bug.


IDE compatibility for plugin expectations

Please watch the video. Thanks.
refs:
https://www.jetbrains.com/help/phpstorm/ide-advanced-metadata.html
https://github.com/bmewburn/vscode-intelephense#intelephense
https://user-images.githubusercontent.com/13007665/125004888-29a51180-e063-11eb-89e6-4375f0340000.mov

@olivernybroe Is this a good idea?

@nunomaduro So I got a bit of mixed feelings about this.
This is an approach that works, however it requires the plugins to have IDE specific code in them.
I have something close to this working in phpstorm, however this config works for both phpstorm and vscode 🤔
PHPSTORM_META files are great as they are natively supported by phpstorm and they are trying to push them more and more.
This however doesn't give the full support we want as doing the following will not work
$collection = collect(['a', 'b', 'c']);
expect($collection)->toHaveCount(3)
        ->and($collection)->toBeCollection();

I think I fixed it. :/
https://user-images.githubusercontent.com/13007665/125052121-4c0f4d00-e0ac-11eb-970f-e23771329f57.mov

@mertasan Don't know why, but it's not working for me 😢

Is your PhpStorm the latest version? @olivernybroe
Maybe a few more people should test this out. 🤔
@nunomaduro Can you test this?

@olivernybroe This would be fixed by adding the extends support to your plugin right? Do you have an ETA on this?

https://github.com/pestphp/pest-intellij/pull/135
I got a PR open for it and got pretty far with it. However before making the last stuff needed to finish it, I wanna hear with Adel if there are any big performance issues with my approach...
However one advantage this solution suggested here has, is that it works for both phpstorm and vscode though.
But to answer your question, no ETA yet unfortunately, if Adel approves the way to do it, I will build the rest of it and then it needs a small testing period 👍

Wow! this is awesome. @olivernybroe
@nunomaduro It would be great to support this with the pest-intellij plugin. But this works instantly for anyone who hasn't installed the pest-intellij plugin. People who are new to using the Pest packages will have a very positive effect on their first impression. (and as Oliver said it works for VSCode too.)

@mertasan I think the biggest problem I have with this is adding a new class which is actually just a type hint class, it doesn't really exist 🙂
Next is that it is more stuff to remember to maintain for the them and something that if we do should be done on all packages.
However it is much easier for a new user 🙂


Catch DetachedInstanceError from DebugToolbar

Occasionally, the DebugToobar attempts to __repr__ a SQLAlchemy instance that has become detached from it's session. Instead, catch the exception and return a different representation when it's detached and we can't get the attributes anymore.

Dunno what's up with codecov again. This is at 100% per https://codecov.io/gh/pypa/warehouse/pull/3027 so I'm going to merge it.


Question about Permutation Invariant Pose Feature Extractor

Thanks for your great work!
But I wonder why the model works on target and source mesh with a different vertex order. The feature F_pose extracted from the Encoder seems to be different on the same pose mesh with different vertex order. Also, F_pose is not spatially aligned with the vertex of the identity mesh. Does Pose1(order 1) + Identity1 and Pose1(order 2) + Identity1 outputs nearly the same result? Any insights on why it works on different vertex order inputs?

Hi @07hyx06,
This is a very good question.
Firstly, Pose1(order 1) + Identity1 and Pose1(order 2) + Identity1 outputs nearly the same result no matter you use original or maxpool.
For original model, we think it is the Instance Norm in each chanel exchang some information.
For maxpool, it is a global feature.
Thank you for your question.

Thanks for your reply!
So there are no explicit modules to keep the results invariant to vertex permutation, it can still reach good results even though the pose and identity features are not spatially aligned?

Hi @07hyx06
Through the experiments, it can reach good results using the architecture in our paper directly.


Overlaping of informations with 2.15

@JULDU74 Thanks for your feedback. Is that a mobile browser ?

@qinfengling yes it is but now it's fine, probably just an explorer refreshing matter. Thanks

great.  the current UI framework is work well with some mobile browser. we only test it with Chrome on PC.


💡 [BACK TO TOP  ] -

Idea Contribution

[X] I have read all the feature request issues.
[X] I'm interested in working on this issue
[X] I'm part of GSSOC organization

Explain feature request
Adding a back to top button on the bottom of the home page ,In an effort to improve user experience on our website.
Explain your solution
No response
Any alternative approaches/features
No response
Additional Context
No response

This issue is duplicate


missing find() definition on Database class

When pip install couchdb on python3.5 package comes without find() method.

That's right; currently, there's no released version containing find() yet. I'll see if I can make a release soon.

Okey. Thanks.

I'm hoping to get my hands on this too.
@djc is there something preventing a release? I'm happy to lend a hand

@pts-sergiomartins just my lack of time. I've released 1.2 now, thanks for the ping.

Thanks very much ^_^


bug: negative number displays as 0

i have bug report, negative number displays as 0, is there any soluation for this?

Hi @mercer08. Great find - I'll definitely have this addressed within the next couple of days. I'll keep this issue thread once the package has been updated.

Hi @mercer08. If you would kindly bump your package version to 1.7.7 the issue should now be resolved. Please let me know if things are working as expected and/or you see any other issues in the meantime! 😄

Closing this out as the issue is resolved + no further comments on it 👍


feat : Implemented Allowed Provers function

[x] issue #2
[x] follows contribution guide
[ ] code change includes tests
[ ] breaking change

Implemented the Allowed Provers function as described in the issue

Thank you!


Drawer is open everytime

If I close the app and remove it from recents then open it again drawer is open again as if its the first time running the app is this desired functionality

Because you didn't closed the drawer and opened. It follows the android guidelines that say the drawer should be always opened when the user opens an app until he closes and opens it again.


PHP: No docblock for method with short array syntax

If you write a method in a PHP class like the following, DocBlockr refuses to create a docblock after adding the /**:
public function test(array $foo = []) {}

Everything works fine with the following variant thus apparently the new short array syntax introduced with PHP 5.4 is not yet supported:
public function test(array $foo = array()) {}

Thanks a lot. :-)


Dev bundle changes for release 0.13.2

Description of changes:
Bump up dev versions for release 0.13.2
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Holding to confirm if we need this change for the patch.


Stampede config for input key

Make sure you are not opening a duplicate.

[X] I have searched the issue tracker and did not find an issue describing my bug.

Description of your suggestion.
Hey there, I love the stampede enchantment but I find hitting the sprint button a little cumbersome in combat like that (I use the default sprint key). Personally, I was hoping you would add an option to change it to the attack key, at least in my brain I perceive it more as an attack then movement, and you can't attack with a shield up anyways. It might be beneficial to just add it's own keybind to whatever you want it to be though, so others can pick if they want a different key for it. Let me know what you think!

I hardcoded it intentionally since if you use a shield in real life you can push whatever is in your way by sprinting forward.
You could code an addon to make it a separate key binding but currently I have no plans to do so.


Bug fix: Merge process.env into Patcher's env vars

There's a bug where the custom $HOME does not work in GitHub's runners, but works locally with act.
Merging process.env into the environment variable maps, it's possible to get the correct $HOME variable, and allow env be defined in the action's yaml.

Thanks for reviewing!


companion notebook to previous video

companion to video about web services financial research

@simonrisman - well done!   👍


Linux UK layout: Ctrl + Apostrophe also toggles terminal

VSCode Version: Code - Insiders 1.11.0-insider (6b9548065ad3e31fca4f0e91cf882505fe8c30cf, 2017-03-30T06:18:34.366Z)
OS Version: Linux x64 3.13.0-108-generic
Extensions: none

Verifying #12682
Steps to Reproduce:

US keyboard, UK layout (if that is valid at all)
Ctrl + Backtick correctly toggles terminal
Ctrl + Apostrophe also toggles terminal, but I don't see that keybinding defined

Adding to March for initial investigation

I can reproduce that Ctrl+[Quote] is toggling the terminal... But it doesn't appear to be our code (i.e. keybindingService). In other words we are ignoring the keydown event :|..
I believe it is Electron's confused menus.

I confirmed it is Electron's confused menus by removing the action from the menu:

Ctrl+[Quote] no longer toggles terminal (which is the expected behaviour)

The resolution is to not give any OEM key codes to Electron under non US keyboard layouts.


[Android] Fallback for textAlign: justify

Fixes #5408 as per @ide and @vjeux suggestions here https://github.com/facebook/react-native/issues/529#issuecomment-107328799
Could've been probably done in a single if clause, but this is more explicit and leaves potential place for future implementation (if we ever decide to do so)

@facebook-github-bot shipit


Prepare for 0.0.1 release

In BzlMod, we have a top level module named buildbuddy_toolchain
(singular), which contains the buildbuddy extension.
The extension will then be used to create the buildbuddy_toolchains
(plural) repository, which contains our toolchain and platform
defintions. Not sure yet how to name this nicely.
Add automatic toolchains and exec platforms registration in the
MODULE.bazel of the buildbuddy_toolchain module. This means that any
repository which use our module would immediately get the relevant
toolchains and exec platforms available to them.
Fix documentations and comments in README.md and bzlmod examples.

Marking this as a draft for now because we gonna need to iterate on this.


Request to remove LinkUpdateNotificationCausesRequestForLinkEndPoints

While implementing callbacks for preapproved links I noticed a considerable amount of requests pinging back to forth between ALT and OKM.
When a new link is added to ALT:

ALT ---> OKM : regard-update-link  (request body is linkUuid)
OKM ---> ALT : list-end-points-of-link (request body is linkUuid, response body are link endpoints)
OKM ---> ALT : update-operation-key

When OKM does scheduleKeyRotation, the process is similar:

OKM ---> ALT : list-link-uuids (response body are linkUuids)
for each linkUuid: OKM ---> ALT : list-end-points-of-link
for each linkUuid: OKM  ---> ALT : update-operation-key

This request is about removing additional step from OKM to ALT by allowing ALT to send link endpoints in request body of regard-update-link (same for list-link-uuids).
Further optimization can be achieved by allowing regard-update-link to consume an array of such links.
Note, that each call for list-end-points-of-link  has to retrieve this information from Elasticsearch in multiple steps, first the link is retrieved based on linkUuid, second, the control construct for EACH link input is retrieved too. (Meaning a link with one input and 7 outputs has to go to ES 9 times).
Note as well, that each forwarding also triggers a forwarding to EATL and a query to ES to record-service-request.
Allowing a link to be sent to OKM instead of just uuid will significantly reduce unnecessary traffic between applications and to Elasticsearch.
Some numbers (current state):
There are 95 preapproved links. These links are saved to Elasticsearch and after that, the forwardings are triggered.
Total number of forwardings sent for each link: 6  (1x regard-update-link, 1x list-endpoints-of-link, 1x update-operation-key + 3x record-service-request)
Total number of forwardings = 570
Number of ES queries sent for each link: 6 at least (1x add-link,  2x list-endpoints-of-link, 3x record-service-request)
Total number of ES queries = 570+
Removing forwarding to list-end-poins-of-link:
Total number of forwardings sent for each link: 4  (1x regard-update-link, 1x update-operation-key + 2x record-service-request)
Total number of forwardings = 380
Number of ES queries sent for each link: 3 (1x add-link,  2x record-service-request)
Total number of ES queries = 285
Sending an array of links:
Total number of forwardings = 192 (1x regard-update-link + 95x update-operation-key + 96x record-service-request)
Total number of ES queries = 95  (95x add new link + 96x record-service-request)

ALT is now sending link endpoints in request body of OKM://regard-update-link, too.
It seems that one or two other changes have been considered but not clearly worked out.
ALT://list-link-uuids not only responding the LinkUuids, but also the Link-end-points might be considered.
But this seems to be a quite extensive dataset, which is then processed over several minutes.
Perhaps, it would be more effective to consider not to sent EaTL://record-service-request when ALL://update-operation-key has been activated.


ignore certain criteria?

How can we totally ignore (or auto-resolve) certain risks, eg. Sign-in from unfamiliar locations?

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

ID: 198f0e69-af2a-8c82-041a-c5ca4126f02f
Version Independent ID: 4be57788-f1c2-d6d4-b94a-a59350a618a7
Content: Azure Active Directory risk events
Content Source: articles/active-directory/reports-monitoring/concept-risk-events.md
Service: active-directory
Sub-service: report-monitor
GitHub Login: @MarkusVi
Microsoft Alias: markvi

@skgitti Currently you can only resolve all the issues for a user or resolve events manually at a time. You cannot ignore a certain criteria completely.
You can find out more about the available options here.

That isn't really usable with thousands of students activating unfamiliar locations 24x7, cannot resolve those per user - will remain unresolved and mess up all reports.

@skgitti I understand your concern here. I would suggest providing your feedback as a feature request here explaining your scenario. This is monitored by the product team and would help them to consider adding this a feature.

feedback sent:
https://feedback.azure.com/forums/169401-azure-active-directory/suggestions/37527133-need-option-to-ignore-or-resolve-all-in-identity-p

@skgitti Thanks for taking the time to provide the feedback. We appreciate it.
I will go ahead and close this issue. Feel free to tag me if you have any further questions regarding this.


https://developer.mozilla.org/en-US/docs/Learn/HTML/Introduction_to_HTML/Creating_hyperlinks : Spelling Mistake

URL(s)
https://developer.mozilla.org/en-US/docs/Learn/HTML/Introduction_to_HTML/Creating_hyperlinks
Request type

[x] Correction or update

Details

In https://developer.mozilla.org/en-US/docs/Learn/HTML/Introduction_to_HTML/Creating_hyperlinks: 'commonly' is not spelled correctly in the 'Specifying details' section as can be seen from the screenshot below.

Fixed; thanks for reporting.


Open Graph image tag on category should change

Hi!
I just saw that we are putting block1.png in categories (well, everywhere in the store except on product details).
I think, in the category details view, we should put the category cover image if set. Otherwise... Block1 or the logo? I am not sure if we should whether put block1 or the logo everywhere as default.
Also, that would be quite a nice feature to allow to customize the page first image.

I think {% if ... %}...{% else %}{{ block.super }}{% endif %} would work the best (least number of places to edit when customizing).


Problem: do muffins count?

There is a rather excellent and easy muffin recipe provided by @sallain 's grandmother but I'm concerned that muffins might be seen as scope creep?

Muffins are like cupcakes which are cakes so...It should be fine :octocat:

Muffins count but only if there are no raisins or bran.  Raisins + bran = yech

IMO the spirit of cakematica is what's really important here - if you declare it a treat, then I think it should meet the criteria and be considered for inclusion. Bran is probably a sensible place to draw the line, but raisins are a divisive topic - I think we should permit them as long as the overall baked good tends toward sweet rather than healthful.


Microsoft Tenant page for refresh

#74
Just a quick copy-paste job. I still make sure that it works. Unless there are some places to poke.

Just tested this and somehow it wasn't working for adding. Let me fix this.

@khanghugo I just rebased this one with the web button here so it’s all up to date. I’ll do a review while you work out what you can on the phone numbers PR 🤘

Just saw this comment https://github.com/jambonz/jambonz-webapp/pull/64#issuecomment-1221096346.
Let me have that in these things right away.

Just saw this comment #64 (comment).
Let me have that in these things right away.

Don’t worry about this I already fixed it when I found it lol

@khanghugo I pushed a quick commit to fix an issue on this one regarding accounts. For parity this view is supposed to look at all accounts, not the specific service provider accounts. So that change resolves the need for the redirect even and now it functions the same as the old webapp. So as far as I can tell this one is done now 👍
https://github.com/jambonz/jambonz-webapp/pull/86/commits/f1b117d10623fe6edd97eb24081b5701f2031f91

Nice one. I will be standby for the next assignment then.

@khanghugo I pushed a quick commit to fix an issue on this one regarding accounts. For parity this view is supposed to look at all accounts, not the specific service provider accounts. So that change resolves the need for the redirect even and now it functions the same as the old webapp. So as far as I can tell this one is done now 👍
f1b117d

In looking into this more I'm not actually sure this makes sense. Maybe @davehorton can help clarify if this is working properly or not? We have exact parity with the old webapp on this branch now (with one key difference), so this is how it works and then maybe we can understand if this is correct or not.
How this works currently:
The MS Teams Tenants nav item is not displayed to the user unless the first service provider in the list of service providers fetched from the backend API has the ms_teams_fqdn field filled out. This turns out to be flaky (code here for reference) in retrospect because I can have the ms_teams_fqdn for a service provider that is not zero in the Array index and never see the MS Teams Tenants nav item—which seems like a bug 🤔. The difference of how this works on the refresh webapp is we show/hide the MS Teams Tenants nav item based on whether the currently selected service provider has the ms_teams_fqdn field set.
My question is:
should the MS Teams Tenants be restricting the Accounts and Applications selectors to those of the current service provider only, or should they show all Accounts and Applications across all service providers which is how this feature currently operates.


null or [] in jsonExporter

 "rows": {
    "left": [
      {
          "BoolArrayTest": {
            "Column": {
              "name": "BoolArrayTest",
              "ordinal_position": 16,
              "type": 14,
              "not_null": false,
              "auto_increment": false
            },
            "Value": null
          },
          "BoolTest": {
            "Column": {
              "name": "BoolTest",
              "ordinal_position": 8,
              "type": 7,
              "not_null": false,
              "auto_increment": false
            },
            "Value": null
          },
        }
      }
    ],
    "right": null
  }

In this case, "right": [] is more appropriate because "left" is a list.

this problem is difficult, I think the indication is correct.
when correcting an indication, I think we need to modify ColumnTypeBytesArray 's value etc.

when using json, you should be able to accept either null or empty array

If accept means parsing, imposing type validation on tamate users is not a good idea. [] and null are treated differently in most of languages. For example, for, foreach and any other iteration can iterate [] but not null. If Left and Right are always list, users can immediately use parsed Left and Light.

all right.
correct it.

create PR
https://github.com/Mitu217/tamate/pull/130

Please try it with the current master branch


Updated TrustedHosts commands

Changed non-standard quote marks in command lines, added quote marks around file names in GetThroughput examples to maintain consistency, corrected typo of "Hyp-erV_ProfiledData" in GetThroughput command line.

@SteveBurkettNZ : Thanks for your contribution! The author, @nsoneji, has been notified to review your proposed change.

#sign-off


Fix figure zooming out in some cases

In some cases, pressing a figure image makes it zoom out. This pull requests sets the minimal scale of the zoom to 1.
Gif of the issue:

The _vendor folder doesn't contain Wowchemy files, it contains third-party libraries. Bugs in vendor assets would need to be fixed upstream in their respective repositories and then we can update their dependencies here. Feel free to submit a PR to update the dependency in package.json instead.


Add pagination

Pagination needs to be added to a couple of models.
https://blog.drewolson.org/pagination-with-phoenix-ecto/
https://github.com/drewolson/scrivener

Not for now.


[SEARCH PERFORMANCE] Only Store UNIQUE keywords in product_keyword_dictionary

Description
I encountered this problem in Shopware 6.1.5 in the Shopware\Core\Content\Product\SearchKeyword\ProductSearchKeywordIndexer but I just took a quick look at the Shopware\Core\Content\Product\DataAbstractionLayer\SearchKeywordUpdater and the problem seems to be exactly the same.
You are storing the same keyword into this table over and over again, without checking if it's already there.
private function insertDictionary(array $dictionary): void
    {
        $queue = new MultiInsertQueryQueue($this->connection, 50, false, true);
        foreach ($dictionary as $insert) {
            $queue->addInsert(ProductKeywordDictionaryDefinition::ENTITY_NAME, $insert);
        }

        // try batch insert
        try {
            $queue->execute();
        } catch (\Exception $e) {
            // catch deadlock exception and retry with single insert
            $query = new RetryableQuery(
                $this->connection->prepare('INSERT IGNORE INTO `product_keyword_dictionary` (`id`, `language_id`, `keyword`) VALUES (:id, :language_id, :keyword)')
            );

            foreach ($dictionary as $insert) {
                $query->execute($insert);
            }
        }
    }

This is a BIG problem. The same keyword (a brand name) for example can come up in every product, so a single run of the indexer can insert the same keyword hundreds of times to the dictionary.
Why is this bad? Let's consider a normal search by the user. In the Shopware\Core\Content\Product\SearchKeyword\ProductSearchTermInterpreter the following happens:

It transforms the search term into slops, to find similar sounding keywords from the dictionary
Now the slops are un against the dictionary. If the user's search term is similar enough, our keywords (that have been entered hundreds of times) come back.
Now the search term is split into its words and the put together in all possible orders of these words (no problem here)
Now the matches are scored. And here, we really get a problem. We calculate the levenstein distance of ALL our matches (step 2) against ALL of our combined search tokens (step 3). This algorithm is O(m*n). And it calculates the distance for EVERY match, even if these matches contain the exact same keyword.

To illustrate this problem a little bit more: For one of our customers, a product search (the product name was entered) resulted in 27.000 calculations of the levenstein distance. After we made sure, that only unique keywords ($matches = array_unique($matches)) are being calculated against the tokens, these calculations dropped to 44!
Of course simply running array_unique against the matches before scoring the matches is not the proper way of fixing the issue. It would be match smarter to make sure, that each keyword is only stored ONCE for every language in the product_keyword_dictionary table.

This should be fixed in 6.2.0. Can you test it there again?


[AIRFLOW-941] Use defined parameters for psycopg2

This works around https://github.com/psycopg/psycopg2/issues/517 .
Please accept this PR that addresses the following issues:

https://issues.apache.org/jira/browse/AIRFLOW-941

Testing Done:

Unit tests pass for Postgres

See: https://travis-ci.org/bolkedebruin/airflow/builds/207717186

the code looks good to me.  what's the deal with the 2.7 failures? looking in travis I can confirm that it's not that config read_file issue that was fixed with this revert.
but the travis output is so long I can't really figure out what failed.

LGTM barring nit above and once Travis passes (or if failures are unrelated)

Py3 passes (it should), py2 failures are ldap3 related as mentioned on the mailing list. Will add todo.

Done. merging.


UI edits for FloatingPanel

Swift #6015
Here are different screenshots of the floating panel being used:

Thanks @dfeinzimer. I just pushed one last commit where I updated the doc.


Fix #1

Named Incorrectly :)
UnDenyPlayerSubCommand.php is supposed to be UnDenySubCommand.php

Renamed the file name.


Sphinx: Add title to each doc page

It may be useful for Google SEO to index the pages
Before:

After:

@vinicinbgs hey! can you re-raise the PR with the new structure? we've restructured the repo for better maintainability.


Setting may not be sized adequately for a production environment...

With default values in $ORACLE_HOME/ords/conf/defaults.xml
I get the following errors:
Jan 10, 2016 6:45:10 PM oracle.dbtools.common.config.db.ConfigurationValues log
WARNING: *** jdbc.MaxLimit in configuration apex is using a value of 10, this setting may not be sized adequately for a production environment ***
Jan 10, 2016 6:45:10 PM oracle.dbtools.common.config.db.ConfigurationValues log
WARNING: *** jdbc.InitialLimit in configuration apex is using a value of 3, this setting may not be sized adequately for a production environment ***
By doubling the value 10 to 20 and 3 to 6 all warning have been eliminated.
I recommend you add the new values to the default file.
thanks
Gaspar

@GasparYYC Can this be stored in https://github.com/OraOpenSource/OXAR/blob/master/ords/ords_params.properties


How to "allow force discharging" is supposed to work?

What is the issue? (required)
I can't figure how the option "Allow force discharging" is supposed to work.
I thought I would have to define a low level battery target to work on battery, while plugged-in, until this level is reached, to then charge the battery. But obviously, it doens't work this way. It's works this way with the terminal command only.
Any help?
The terminal command "battery discharge XX" work well though.
What exactly did you do to produce the issue? (required)
Steps to reproduce the behavior:

Set the "Allow force discharging" option

Expected behavior (required)
I don't know what to expect exactly, it's just not clear enough.
Screenshots (optional)
On this 1st screenshot, one sees the option is activated, but the mac is still shown as "plugged in".

Error logs
Additional context (optional)
"battery discharge 30" output:
11/06/24-16:54:22 - Discharging to 30% from 80%
11/06/24-16:54:22 - 🔽🪫 Enabling battery discharging
Error: SMCWriteKey() = e00002bc
11/06/24-16:54:22 - Battery at 80% (target 30%)

Logs:
🖥	Battery GUI logs:

{"percentage":"80","remaining":"unknown","charging":false,"discharging":false,"maintain_percentage":"80","battery_state":"80% (unknown remaining)","daemon_state":"smc charging disabled"}
Generate app menu percentage: 80 (discharge allowed, limited on)
Found image: /Applications/battery.app/Contents/Resources/battery-active-80-Template.png
Display percentage 80 based on 80
Refreshing interface update timer
Get active logo for 80
Limiter status message:
11/06/24-16:50:10 - Battery at 80% (attached; remaining), 12.144V, smc charging disabled
11/06/24-16:50:10 - Your battery is currently being maintained at 80%

Force discharge setting: boolean true
Executing PATH=/opt/homebrew/bin:/opt/homebrew/sbin:/usr/local/bin:/usr/bin:/bin:/usr/sbin:/sbin battery status_csv
Resetting interface timer speed
Battery status:
{"percentage":"80","remaining":"unknown","charging":false,"discharging":false,"maintain_percentage":"80","battery_state":"80% (unknown remaining)","daemon_state":"smc charging disabled"}
Setting interface refresh speed to 10 minutes
Executing PATH=/opt/homebrew/bin:/opt/homebrew/sbin:/usr/local/bin:/usr/bin:/bin:/usr/sbin:/sbin battery status_csv
Refreshing tray icon...
Battery status:
{"percentage":"80","remaining":"unknown","charging":false,"discharging":false,"maintain_percentage":"80","battery_state":"80% (unknown remaining)","daemon_state":"smc charging disabled"}
Executing PATH=/opt/homebrew/bin:/opt/homebrew/sbin:/usr/local/bin:/usr/bin:/bin:/usr/sbin:/sbin battery status
Force discharge setting: boolean true
Display percentage 80 based on 80
Get active logo for 80
Resetting interface timer speed
Refreshing interface update timer
Executing PATH=/opt/homebrew/bin:/opt/homebrew/sbin:/usr/local/bin:/usr/bin:/bin:/usr/sbin:/sbin battery status_csv
Found image: /Applications/battery.app/Contents/Resources/battery-active-80-Template.png
Generate app menu percentage: 80 (discharge allowed, limited on)
Limiter status message:
11/06/24-16:50:41 - Battery at 80% (attached; remaining), 12.145V, smc charging disabled
11/06/24-16:50:41 - Your battery is currently being maintained at 80%

Even the terminal discharge command doesn't really work as expected.
user@mac ~ % battery discharge 30
11/06/24-17:48:39 - Discharging to 30% from 80%
11/06/24-17:48:39 - 🔽🪫 Enabling battery discharging
Error: SMCWriteKey() = e00002bc
11/06/24-17:48:39 - Battery at 80% (target 30%)
11/06/24-17:49:39 - Battery at 80% (target 30%)
11/06/24-17:50:39 - Battery at 80% (target 30%)
11/06/24-17:51:40 - Battery at 80% (target 30%)
11/06/24-17:52:40 - Battery at 80% (target 30%)
11/06/24-17:53:40 - Battery at 80% (target 30%)
11/06/24-17:54:40 - Battery at 80% (target 30%)
11/06/24-17:55:40 - Battery at 80% (target 30%)
11/06/24-17:56:40 - Battery at 80% (target 30%)
11/06/24-17:57:40 - Battery at 80% (target 30%)
11/06/24-17:58:40 - Battery at 80% (target 30%)
11/06/24-17:59:40 - Battery at 79% (target 30%)
11/06/24-18:00:40 - Battery at 79% (target 30%)
11/06/24-18:01:40 - Battery at 79% (target 30%)
11/06/24-18:02:40 - Battery at 79% (target 30%)
11/06/24-18:03:40 - Battery at 79% (target 30%)
11/06/24-18:04:40 - Battery at 79% (target 30%)
11/06/24-18:05:40 - Battery at 79% (target 30%)
11/06/24-18:06:41 - Battery at 80% (target 30%)

As one can sees, the battery starts to get drained, and then, it's charged again to the 80% limits.

Got a new mac. This one doesn't have the SMC error, but the discharge behavior is the same. Once at 79%, it charges again.


Adds Metadata describer

Checklist:

[x] Did you sign the Contributor License Agreement?
[x] Is the linter reporting 0 errors? 0 Errors for the code in this PR
[ ] Did you comply with our styleguide?
[ ] Is there adequate test coverage for your new code?
[x] Does the CI pipeline pass? For what this PR touches
[x] Did you update the documentation?
[ ] Have you done everything you can to make sure that errors that can occur are compile-time errors, and if they have to be runtime do you have adequate tests and documentation around those runtime errors? For more details.

Codecov Report

:exclamation: No coverage uploaded for pull request base (data-driven@1d430cb). Click here to learn what that means.
The diff coverage is n/a.

@@              Coverage Diff               @@
##             data-driven     #155   +/-   ##
==============================================
  Coverage               ?   89.68%
==============================================
  Files                  ?       95
  Lines                  ?     2502
  Branches               ?        0
==============================================
  Hits                   ?     2244
  Misses                 ?      258
  Partials               ?        0

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 1d430cb...182c2ee. Read the comment docs.

I think this is probably good enough to start updating this to Ready for Review.  I'm out for most if not all of next week (and tomorrow) so if we want to make more progress on this, feel free to take it and keep working on it without me.

Looks like the linter is only failing due to two TODOs that I feel are appropriate to stay until data-driven merges into main, so I'm approving & merging.


Use Guidance framework for more efficient agent communications

Use Guidance framework for agent communications
https://github.com/guidance-ai/guidance
See their specific agent example conversation.
This would allow each agent's output to be structured and controlled accordingly to their roles.
This YouTube video explains it well :
https://youtu.be/69bH4IHZivs?si=RkdCcQRARFT84bof

Not under consideration for now.

No offence but, I think you should consider. The output can be structured without the power/bits of the LLM models. This can be a huge, since I'm getting hard time to run the metagpt with a simple task. The output structure is wrong.
@j-loquat, if I invest to do this, can you help me to test it?

@geekan, is there any particular reason for not to consider it? Or is it complicated business decision for anything that we cannot see now?


ODK-X Docs: Improve Documentation on Designing a Form

Description
As described in the Building Application Section, in the Updating framework.xlsx section. When adding another new form to an existing framework.xlsx file, which is after testForm.xlsx must have been created in the appropriate directory. Adding ''?' + opendatakit.getHashString('testForm') in the url column of the survey worksheet of the framework.xlsx prevents you from opening the form and gives unexpected behaviour.
Steps to Reproduce

Navigate to Updating framework.xlsx
Follow the instructions to add a new form to an existing framework.xlsx file

Actual Behaviour
After saving the changes made above, clicking on the follow link button does not trigger the Survey to open as shown.
,
Proposed Fix
Changing the ''?' + opendatakit.getHashString('testForm')  in the documentation to ''?' + odkSurvey.getHashString('testForm') in the url column of the survey worksheet of the framework.xlsx will give the expected behaviour and will trigger the Survey form to open as shown:
,
,

Yes I will @wbrunette


FIX: construct Montage out of channels with position

fixes #3957

this probably requires an additional test so I'll add one tomorrow.

Let me know when you are ready. I have some files locally to test things

I think we usually do (0., 0., 0.) to mean "invalid position" (even though technically it could be a real position). np.nan is probably also okay though

Ah, ok, if you are using 0, 0, 0 as no position then the solution is easier - I'll change it.

Hm, I changed to a super-easy solution (locally) when everything is put into montage - all channels, but some with zeros for all x, y and z. But I am getting a weird error when testing locally:
ValueError: None of the sensors defined in the montage were found in the info structure. Check the channel names.

which wouldn't be strange if not for the fact that Info is:
<Info | 15 non-empty fields
    bads : 'list | 0 items
    ch_names : 'list | FP1, Fz
...

and Montage:
<Montage | user_defined - 2 channels: FP1, Fz ...>

so the channels should agree...
Full traceback below:
Traceback (most recent call last):
  File "test_eeglab.py", line 156, in <module>
    run_tests_if_main()
  File "c:\proj\src\mne-python\mne\utils.py", line 2199, in run_tests_if_main
    val()
  File "C:\Anaconda3\lib\site-packages\numpy\testing\decorators.py", line 147, in skipper_func
    return f(*args, **kwargs)
  File "C:\Anaconda3\lib\site-packages\numpy\testing\decorators.py", line 147, in skipper_func
    return f(*args, **kwargs)
  File "test_eeglab.py", line 85, in test_io_set
    epochs = read_epochs_eeglab(epochs_fname)
  File "c:\proj\src\mne-python\mne\io\eeglab\eeglab.py", line 249, in read_epochs_eeglab
    uint16_codec=uint16_codec)
  File "<string>", line 2, in __init__
  File "c:\proj\src\mne-python\mne\utils.py", line 709, in verbose
    return function(*args, **kwargs)
  File "c:\proj\src\mne-python\mne\io\eeglab\eeglab.py", line 536, in __init__
    info = _get_info(eeg, montage, eog=eog)
  File "c:\proj\src\mne-python\mne\io\eeglab\eeglab.py", line 101, in _get_info
    update_ch_names=update_ch_names)
  File "c:\proj\src\mne-python\mne\io\base.py", line 2340, in _check_update_montage
    _set_montage(info, montage, update_ch_names=update_ch_names)
  File "c:\proj\src\mne-python\mne\channels\montage.py", line 735, in _set_montage
    raise ValueError('None of the sensors defined in the montage were '
ValueError: None of the sensors defined in the montage were found in the info structure. Check the channel names.

Anyway - the current solution (the one that is present here) passes locally and works for me. The only issue is that if some eeglab chanels do not have defined position it gives a warning (from _apply_montage) that not all channels were found in Montage. For me it is ok - but this may break some tests that count warnings. Is such warning ok for you?

@jasmainak @Eric89GXL
CIs are green, you can take a look.

This seems okay to me. Tests that break should probably be made more robust anyway, e.g. by looking for warnings of interest

Fix looks reasonable but maybe a test with different settings of montage would be nice. Because it's been a while looking at this code for me. And the code path is not trivially simple for me to verify. At least it doesn't break any of the data files that I tried locally

@jasmainak
You're right - I didn't consider scenarios with channel positions in set file and montage as a separate file. What should be done if some channels have positions in set but not in montage file? Should such positions be ignored when montage file is passed? That's the easiest solution - so I think I'll add that.

@Eric89GXL - how can I abort two previous CIs so that they run only for the last commit? Or if I can't - if you could do that, it would save us a few hours :)

how can I abort two previous CIs so that they run only for the last commit? Or if I can't - if you could do that, it would save us a few hours :)

Canceled.

Thanks @jaeilepp!

okay LGTM +1 for merge when CIs are happy!

First Travis job kind of fails (although it runs to completion and all relevant tests pass) due to mayavi problems:
Test CombineMarkersModel Traits Model ... Exception occurred in traits notification handler for object: <mayavi.tools.mlab_scene_model.MlabSceneModel object at 0x7ffaf1197650>, trait: do_render, old value: <undefined>, new value: True
Traceback (most recent call last):
  File "/home/travis/miniconda/envs/test/lib/python2.7/site-packages/traits/trait_notifiers.py", line 519, in _dispatch_change_event
    self.dispatch( handler, *args )
  File "/home/travis/miniconda/envs/test/lib/python2.7/site-packages/traits/trait_notifiers.py", line 482, in dispatch
    handler( *args )
  File "/home/travis/miniconda/envs/test/lib/python2.7/site-packages/tvtk/pyface/ui/qt4/scene.py", line 348, in render
    self._vtk_control.Render()
  File "/home/travis/miniconda/envs/test/lib/python2.7/site-packages/tvtk/pyface/ui/qt4/QVTKRenderWindowInteractor.py", line 498, in Render
    self.update()
RuntimeError: wrapped C/C++ object of type _VTKRenderWindowInteractor has been deleted
Exception occurred in traits notification handler for object: <mayavi.tools.mlab_scene_model.MlabSceneModel object at 0x7ffaf1197650>, trait: do_render, old value: <undefined>, new value: True
Traceback (most recent call last):
  File "/home/travis/miniconda/envs/test/lib/python2.7/site-packages/traits/trait_notifiers.py", line 519, in _dispatch_change_event
    self.dispatch( handler, *args )
  File "/home/travis/miniconda/envs/test/lib/python2.7/site-packages/traits/trait_notifiers.py", line 482, in dispatch
    handler( *args )
  File "/home/travis/miniconda/envs/test/lib/python2.7/site-packages/tvtk/pyface/ui/qt4/scene.py", line 348, in render
    self._vtk_control.Render()
  File "/home/travis/miniconda/envs/test/lib/python2.7/site-packages/tvtk/pyface/ui/qt4/QVTKRenderWindowInteractor.py", line 498, in Render
    self.update()
( ... )

and some other unrelated problems.

Cool, more mayavi-related CI problems :(

@Eric89GXL
Actually it's a bit more fun :) - same/similar errors are present in another job that finishes green. Both perform all tests ok (without failures according to summary at the end) - the only major differences I see is that one is marked as green and other as red... And there is another red (but not failing!) job that does not have these errors. So mayavi is not the reason the builds are red - neither any of the tests... Maybe you can spot the differences - it's a fun game to play ;)

But I see some new flake errors that I'll fix - so this is not yet ready.

@Eric89GXL @jasmainak
Ok, it's ready on my end - the builds are marked as failing, but reason for this is not clear to me (as I mentioned in more details above) - all tests pas....

Travis complains

@agramfort
See my previous comments - other PRs have the same problem: tests pass but Travis complains.

Travis is sane again and all builds except one came back green (link). The one that is failing was because URLError: <urlopen error [Errno 101] Network is unreachable> during test for downloading file over http.

LGTM, merged. Thanks @mmagnuski !

Thanks @jasmainak and @Eric89GXL! :rocket:


fix display results on text screen

Fix how results from pings are stored and output them out into text screen widget properly formatted.

Sucessfullyed merged issue branch into dev branch


[Combobox] check props before blindly passing down

What package is this for?

[x] React
[ ] Angular

Describe the bug
Combobox is passing props down that are not valid props for html elements:

To Reproduce
Steps to reproduce the behavior:

Go to any Combobox implementation
Pass down invalid props such as anyTouched
See that a prop warning is thrown in the console

Expected behavior
Only pass valid props down to the html elements
Environment/versions:

OS: MacOS
Browser:  chrome
carbon-addons-iot-react version: 2.146.0-next.64

Specific timeline issues / requests
Do you want this work within a specific time period? Is it related to an
upcoming release?
None

If it for some reason isn't possible to be completely explicit about which props we are passing down we can use filterValidAttributes in packages/react/src/utils/componentUtilityFunctions.js to only pass down valid HTML attributes.


#132: correct typo

can't see that the VS2017 failure reported above has anything to do with my change.

Thanks!


Portfolio -> index.html -> anchor icons rotate 360 deg when hovered

Feature Request
Is your feature request related to a problem? Please describe.
When the anchor icons are hovered, the icons will rotate to 360 deg with a transition of 1.45 s, If you want the transition timing to be less please comment I will change it.
Can I work on this issue?

So you are going to change transition time ?

So you are going to change transition time ?

No, if you want to change I can change, but the main change that I have proposed is : transformation of social icons when hovered, they will rotate 360 degrees.


how to use Component in razor pages

I like.net 2.0 the new features of the core razor pages, but how the razor pages using MVC Component feature

Are you asking about View components? If so, you use them in the same way as in an MVC application. You can call await Component.InvokeAsync(...) or you can use the view component tag helper: https://docs.microsoft.com/en-gb/aspnet/core/mvc/views/view-components#invoking-a-view-component-as-a-tag-helper.

Indeed, we need a bit more info. View components should work exactly the same in Razor Pages as they do in MVC Razor views.

Exactly, I just change the content of the original MVC to use razor pages to deal with.I prefer the razor pages to develop this way

Ok then it doesn't sound like there's an issue here.
If you are still having an issue, please log a new issue with details about what you tried, what didn't work, and what version you are using.


Override name tag on autoscaling group

Hi there,
Thanks for your work on this module.
I'm curious why 'Name' tag is defined here on the auto scaling group resource.  With this logic in place it's not possible to override this tag as it is elsewhere due to use of merge over concat.
Would you accept a PR to merge the ASG tags in a way that allows for overriding Name?

Hello,
Yes feel free to propose something and I'll review it.
Thanks,


Install Chakra UI

Notion Ticket Link

Install Chakra UI

Implementation Description

The actual installation changes are mostly in the frontend package.json and yarn.lock. The rest of the changes were just small fixes to accommodate the updated packages.

Steps to Test

Delete your frontend/node_modules folder. In the frontend directory (cd frontend), run yarn install to get a fresh copy of the packages. This won't affect what Docker gives you, but it'll make the local package errors go away.
After running docker compose down, rebuild the containers with docker compose up --build.
Expected: app compiles with no errors*.

* The login page will probably show "Script error" (and you'll see something about missing the client_id parameter in the console). Don't worry about it, the "Remove Google Auth for Login" ticket will make the error go away eventually.

Screenshots (optional)
Here's what the Chakra UI button looks like:

If you want to also plop a component somewhere to try it out, check out the docs: https://chakra-ui.com/docs/components
Checklist

[x] My PR name is descriptive and in imperative tense
[x] My commit messages are descriptive and in imperative tense. My commits are atomic and trivial commits are squashed or fixup'd into non-trivial commits
[x] I have run the appropriate linter(s)
[x] I have requested a review from the PL, as well as other devs who have background knowledge on this PR or who will be building on top of this PR

Linting & Formatting
Mac
docker exec -it CWC-backend /bin/bash -c "yarn fix"
docker exec -it CWC-frontend /bin/bash -c "yarn fix"

Windows
docker exec -it CWC-backend bash -c "yarn fix"
docker exec -it CWC-frontend bash -c "yarn fix"

Followed the steps, had no issues compiling on my end


Comparison to PyCUDA

It seems many people are interested in the differences between PyCUDA and CuPy. It should be beneficial for them to add a note on them to the official document (maybe adding FAQ section is good?) and possibly a blog post somewhere.

As pointed in https://github.com/cupy/cupy/issues/2450#issuecomment-530677886, when #2450 is closed I believe PyCUDA users can easily migrate to CuPy.


Include arm binaries ruby python

Codecov Report
Base: 75.07% // Head: 74.05% // Decreases project coverage by -1.01% :warning:

Coverage data is based on head (4ae237e) compared to base (bbbc25b).
Patch coverage: 47.05% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #573      +/-   ##
==========================================
- Coverage   75.07%   74.05%   -1.02%
==========================================
  Files          21       21
  Lines         702      717      +15
==========================================
+ Hits          527      531       +4
- Misses        175      186      +11

Impacted Files
Coverage Δ

python/trinsicokapi/wrapper.py
82.44% <47.05%> (-7.22%)
:arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


messages: fix race in test

I saw this race a couple of times because the Purge threads would
race with SetChannel(nil). Instead, I've just created a buffered
channel, which won't cause things to get blocked.

LGTM


(sourcetree) Use MSI installer

Description
Use new enterprise version of Sourcetree which comes as an MSI
Motivation and Context
The current Squirrel based installer is not really suitable for a deployment with Chocolatey as it installs in the user directory. Using the MSI fixes these issues.
Fixes #1001
How Has this Been Tested?
Tested locally
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[x] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Migrated package (a package has been migrated from another repository)

Checklist:

[x] My code follows the code style of this repository.
[x] My change requires a change to documentation (this usually means the notes in the description of a package).
[ ] I have updated the documentation accordingly (this usually means the notes in the description of a package).
[x] All files are up to date with the latest Contributing Guidelines
[ ] The added/modified package passed install/uninstall in the chocolatey test environment.
[x] The changes only affect a single package (not including meta package).

Some questions by myself:
The MSI installs side by side with the Squirrel based installer used in current versions of this package. Since using the Squirrel based installer no longer makes sense I don't think the MSI deserves it's own package, but instead the existing package should be updated. But currently there is no proper upgrade path from previous versions to this version. Should we try to uninstall existing Sourcetree installations in this package? What if a user really want to have both installed side by side?
Settings also want be migrated from existing installations. Should we try to do this migration or is this out of scope for a Chocolatey package?
I also should update the description in the nuspec to reflect the above points based on the direction we want to take.

:x: Package verification failed, please review the Appveyor Logs and the provided Artifacts before requesting a human reviewer to take a look.

:x: Package verification failed, please review the Appveyor Logs and the provided Artifacts before requesting a human reviewer to take a look.

:x: Package verification failed, please review the Appveyor Logs and the provided Artifacts before requesting a human reviewer to take a look.

@pascalberger could you rebase towards the latest of master? Currently there are merge conflicts.
I'll review the PR after that have been done.

@AdmiringWorm @gep13 I've now added a check if there's already an existing installation of Sourcetree installed using the Squirrel installer on the machine. If so it asks the user if it should be uninstalled. Settings will not be migrated to the new installation.
This is ready for a review from my side

:x: Package verification failed, please review the Appveyor Logs and the provided Artifacts before requesting a human reviewer to take a look.

You can detect previous existing package and throw with adequate message for a user. User should devise migration strategy if thats needed.

@majkinetor Currently the package checks for existing Squirrel installation and asks user if he wants to uninstall it. Does this fit with what you asked? Or do you prefer to only print out a note that another SourceTree installation is present and new package will be installed side-by-side?

@majkinetor I removed the uninstall code and just output a warning that standard version of Sourcetree won't be uninstalled. Also added an additional note to the package description

:x: Package verification failed, please review the Appveyor Logs and the provided Artifacts before requesting a human reviewer to take a look.

Is there anything else you need to do here ?
It would be good to have some proof that this works before merging. Could you Test-Package locally or inside VM and give output here (or something equivalent).

Nvm merging this, I thought AV buid failed for different reasons.

Hm... I see now that you didn't throw here. Please throw on older version found. Nobody will see warnings once they do cup all but they will see errors as they are listed at the end of all updates. Nobody expects this - installing the same named software and getting something parallel to existing one.
We have similar situation with VSCode x64 install, and package didn't hendled it, but during installl VSCode would complain, which was singnal to all users that intervention is required.

@majkinetor I don't think throwing will lead to the best user experience. This forces the user to manually uninstall his prior Sourcetree installation before he can install the new version. By this he loses all settings. By only warning he at least can check the settings in the old version and apply them in the new version before uninstalling the old version.

Well... ok...


There are issue with Dockerfile

Description of problem
Thanks all contributors, this project help me quickly removes labels from docker images.
but I meet some problem when I building Dockerfile:
Error installing fpm
The last version of public_suffix (< 6.0, >= 2.0.2) to support your Ruby & RubyGems was 4.0.7.
Try installing it with gem install public_suffix -v 4.0.7 and then running the current command again.
Steps to reproduce
sudo docker build -f Dockerfile -t docker-image-labeler .
docker-image-labeler version
master
Output of failing command
No response

This works for me now.