Debug disables itself on master if you catch workers messages.

When I catch the workers messages I won't receive any debugs from master since debug disables all the loggers running on it.

With worker.on("message")

Without worker.on("message")

Is there a way to fix this? I couldn't find anything on it...

Just found out that I can fix it by doing:
debug.enable(process.env.DEBUG);
in my index.ts
But I feel like this should be done by default?

Just found out that I can fix it by doing:
debug.enable(process.env.DEBUG);
in my index.ts
But I feel like this should be done by default?

Can you post a reproduction case with some code? Code images don't help me much.

Can you post a reproduction case with some code? Code images don't help me much.

Can you post a reproduction case with some code? Code images don't help me much.

Okay now I even have it with and without the worker.on("message")
I made an example for you that you can download and test: https://github.com/Bas950/DebugBug
Just run yarn and then yarn dev, if you un-comment the debug.enable in the index.ts file, it will show you Main and Main:Loop debugs, without it, you will only see debugs from Main:Worker (which ofc run only on the worker).

Can you post a reproduction case with some code? Code images don't help me much.

Okay now I even have it with and without the worker.on("message")
I made an example for you that you can download and test: https://github.com/Bas950/DebugBug
Just run yarn and then yarn dev, if you un-comment the debug.enable in the index.ts file, it will show you Main and Main:Loop debugs, without it, you will only see debugs from Main:Worker (which ofc run only on the worker).

@Qix- Anything on this?

No, not really. Your reproduction case is convoluted and kind of complex. There have been problems with dotenv before, can you manually try setting export DEBUG='*' in your terminal prior to running to see if that fixes the issue first?

Sadly that doesn't fix it :/
But as said before debug.enable(process.env.DEBUG); does fix it...

That's strange. I've never heard of or faced this issue before and that's certainly not a fix that should be necessary.
Can you try to reduce the reproduction case a bit?

not really just whenever this is in the code it breaks

That doesn't really help me debug this issue, though. I can't see how adding an event handler would make any difference.
Unfortunately I don't think this is related to debug in particular and instead is an issue specific to your application. If you happen to reduce the reproduction case down to a minimal case, let me know.
Good luck!


you don't use qtcreator to open pro?

how to use qtcreator to open your project?and the ui is implement by yourself

just open vncroot.pro


[llama] llama CI T3000 refactor

[x] Removed [] or :: in pytest commands.
[x] Referencing entire dirs

post commit: https://github.com/tenstorrent/tt-metal/actions/runs/8803004964
multi-device_ci: https://github.com/tenstorrent/tt-metal/actions/runs/8803068544


Crossbow - L&S II Pawn Storm

This is a milestone to have an object in place based on the railgun in Terasology that allows for the destruction of environmental objects and NPCs.
The goal is to create a smaller scale, shorter, ranged version of railgun so that players can destroy towers and NPCs.

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/743045-crossbow-l-s-ii-pawn-storm?utm_campaign=plugin&utm_content=tracker%2F5279&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F5279&utm_medium=issues&utm_source=github).

Closing in favor of #2570.


Server-sent event in traefik - too many transfer encoding errors

Welcome!

[X] Yes, I've searched similar issues on GitHub and didn't find any.
[X] Yes, I've searched similar issues on the Traefik community forum and didn't find any.

What did you do?
I have an issue with Server-Sent events. I cannot state this is a bug report but I really looked everywhere and even in the community forum nor StackOverflow I could not find any help.
I have a simple sse generated from Django that is served correctly by Nginx and is not working under Traefik. I attach a complete setup working under docker-compose.
In docker, I use:
    labels:
       - "traefik.enable=true"
       - "traefik.http.routers.dj.rule=Host(`$SSE_HOST`)"
       - "traefik.http.routers.dj.entrypoints=web"
       - "traefik.http.routers.dj.service=dj"
       - "traefik.http.services.dj.loadbalancer.server.port=8000"
       #- "traefik.http.routers.dj.middlewares=compress-no-stream"
       #- "traefik.http.middlewares.compress-no-stream.compress=true"
       #- "traefik.http.middlewares.compress-no-stream.compress.excludedcontenttypes=text/event-stream"

while in the same docker-compose setup I define nginx with the following conf that is working:
server {
  listen 80;
  server_name _ ;
  charset utf-8;
  client_max_body_size 20M;

  location / {
    proxy_pass http://django:8000;
    proxy_http_version 1.1;
    proxy_set_header Host $host;
  }
}

What did you see instead?
When visiting the Server-Sent Event page directly via nginx I see messages sent from the server getting to the page.
When proxying via Traefik (w/ or w/o compression) no event gets to the browser.
If Traefik proxies to a Django run from console, I get a "Temporary failure in name resolution" and I cannot understand what it refers to.
What version of Traefik are you using?
I'm using Traefik version 2.5.4
What is your environment & configuration?
version: '3.7'
services:
  nginx:
    image: nginx
    networks:
      - django
    ports:
      - "8081:80"
    volumes:
      - ./logs/nginx:/var/log/nginx
      - ./nginx/default.conf:/etc/nginx/conf.d/default.conf

  redis:
    image: "redis:5"
    command: redis-server
    ports:
      - "6379:6379"
    volumes:
     - ./redis-data:/var/lib/redis
      # - ./redis.conf:/usr/local/etc/redis/redis.conf
    environment:
     - REDIS_REPLICATION_MODE=master
    networks:
      - django

  django:
    image: fiverr-traefik
    build:
      context: .
      dockerfile: ./Dockerfile

    ports:
      - "8080:8000"
    environment:
      ENV: dev
      REDIS_HOST: redis
      REDIS_PORT: 6379
      SECRET_KEY: "any-key-will-do"
      #ALLOWED_HOSTS: '*'
    volumes:
      - ./logs/uwsgi:/var/log/uwsgi
      - ./logs/django:/var/log/django
      - ./media:/code/media
    networks:
      - web
      - django
      - redis
    labels:
       - "traefik.enable=true"
       - "traefik.http.routers.dj.rule=Host(`$DJANGO_HOST`)"
       - "traefik.http.routers.dj.entrypoints=web"
       - "traefik.http.routers.dj.service=dj"
       - "traefik.http.services.dj.loadbalancer.server.port=8000"
       #- "traefik.http.routers.dj.middlewares=compress-no-stream"
       #- "traefik.http.middlewares.compress-no-stream.compress=true"
       #- "traefik.http.middlewares.compress-no-stream.compress.excludedcontenttypes=text/event-stream"
    #command: uwsgi --ini /etc/uwsgi.ini

  traefik:
    image: traefik:v2.5.4
    restart: always
    container_name: traefik
    ports:
      - "9000:80" # <== http
      - "443:443" # <== https
    extra_hosts:
    - "django.sse:<IP_ADDRESS>"# your ip; name set in providers.file's static conf
    command:
    #### These are the CLI commands that will configure Traefik and tell it how to work! ####
      - --api.dashboard=true # <== Enabling the dashboard to view services, middlewares, routers, etc...
      - --api.insecure=true # <== Enabling the dashboard to view services, middlewares, routers, etc...
      - --log.level=INFO # <== Setting the level of the logs from traefik
      - --accessLog.filePath=/logs/access.log
      - --log.filePath=/logs/traefik.log
      - --providers.file.directory=/etc/traefik/traefik.d
      ## Provider Settings - https://docs.traefik.io/providers/docker/#provider-configuration ##
      - --providers.docker=true # <== Enabling docker as the provider for traefik
      - --providers.docker.exposedbydefault=false # <== Don't expose every container to traefik, only expose enabled ones
      - --providers.docker.network=web # <== Operate on the docker network named web
      - --entrypoints.web.address=:80 # <== Defining an entrypoint for port :80 named web
    volumes:
      - ./logs/traefik:/logs
      - ./conf/traefik:/etc/traefik # <== Volume for certs (TLS)
      - /var/run/docker.sock:/var/run/docker.sock # <== Volume for docker admin
    networks:
      - web
      - django
    depends_on:
      - django
    labels:
    #### Labels define the behavior and rules of the traefik proxy for this container ####
      - "traefik.enable=true" # <== Enable traefik on itself to view dashboard and assign subdomain to view it
      - "traefik.http.routers.api.rule=Host(`${TRAEFIK_HOST}`)" # <== Setting the domain for the dashboard
      - "traefik.http.routers.api.service=api@internal" # <== Enabling the api to be a service to access

networks:
  django:
  web:
    external: true
  redis:

Attached is a minimal setup to reproduce the problem w/ README
demo-sse-traefik.zip
If applicable, please paste the log output in DEBUG level
The meaningful part of the logs that I believe can give info is this:
level=debug msg="'500 Internal Server Error' caused by: net/http: HTTP/1.x transport connection broken: too many transfer encodings: [\"chunked\" \"chunked\"]"

But again I cannot find useful info on what is should depend on.
Other logs:
time="2021-12-10T14:46:04Z" level=debug msg="vulcand/oxy/roundrobin/rr: begin ServeHttp on request" Request="{\"Method\":\"GET\",\"URL\":{\"Scheme\":\"\",\"Opaque\":\"\",\"User\":null,\"Host\":\"\",\"Path\":\"/demo/sse/channels/sse-example/\",\"RawPath\":\"\",\"ForceQuery\":false,\"RawQuery\":\"\",\"Fragment\":\"\",\"RawFragment\":\"\"},\"Proto\":\"HTTP/1.1\",\"ProtoMajor\":1,\"ProtoMinor\":1,\"Header\":{\"Accept\":[\"text/event-stream\"],\"Accept-Encoding\":[\"gzip, deflate\"],\"Accept-Language\":[\"it-IT,it;q=0.9,en-US;q=0.8,en;q=0.7\"],\"Cache-Control\":[\"no-cache\"],\"Connection\":[\"keep-alive\"],\"Referer\":[\"http://demo.sse:9000/sse\"],\"User-Agent\":[\"Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/95.0.4638.54 Safari/537.36\"],\"X-Forwarded-Host\":[\"demo.sse:9000\"],\"X-Forwarded-Port\":[\"9000\"],\"X-Forwarded-Proto\":[\"http\"],\"X-Forwarded-Server\":[\"3fbd4799b7e7\"],\"X-Real-Ip\":[\"<IP_ADDRESS>\"]},\"ContentLength\":0,\"TransferEncoding\":null,\"Host\":\"demo.sse:9000\",\"Form\":null,\"PostForm\":null,\"MultipartForm\":null,\"Trailer\":null,\"RemoteAddr\":\"<IP_ADDRESS>:37094\",\"RequestURI\":\"/demo/sse/channels/sse-example/\",\"TLS\":null}"
time="2021-12-10T14:46:04Z" level=debug msg="vulcand/oxy/roundrobin/rr: Forwarding this request to URL" Request="{\"Method\":\"GET\",\"URL\":{\"Scheme\":\"\",\"Opaque\":\"\",\"User\":null,\"Host\":\"\",\"Path\":\"/demo/sse/channels/sse-example/\",\"RawPath\":\"\",\"ForceQuery\":false,\"RawQuery\":\"\",\"Fragment\":\"\",\"RawFragment\":\"\"},\"Proto\":\"HTTP/1.1\",\"ProtoMajor\":1,\"ProtoMinor\":1,\"Header\":{\"Accept\":[\"text/event-stream\"],\"Accept-Encoding\":[\"gzip, deflate\"],\"Accept-Language\":[\"it-IT,it;q=0.9,en-US;q=0.8,en;q=0.7\"],\"Cache-Control\":[\"no-cache\"],\"Connection\":[\"keep-alive\"],\"Referer\":[\"http://demo.sse:9000/sse\"],\"User-Agent\":[\"Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/95.0.4638.54 Safari/537.36\"],\"X-Forwarded-Host\":[\"demo.sse:9000\"],\"X-Forwarded-Port\":[\"9000\"],\"X-Forwarded-Proto\":[\"http\"],\"X-Forwarded-Server\":[\"3fbd4799b7e7\"],\"X-Real-Ip\":[\"<IP_ADDRESS>\"]},\"ContentLength\":0,\"TransferEncoding\":null,\"Host\":\"demo.sse:9000\",\"Form\":null,\"PostForm\":null,\"MultipartForm\":null,\"Trailer\":null,\"RemoteAddr\":\"<IP_ADDRESS>:37094\",\"RequestURI\":\"/demo/sse/channels/sse-example/\",\"TLS\":null}" ForwardURL="http://<IP_ADDRESS>:8000"

Hello @sandroden
This seems to be a known issue caused by a stricter check on the Transfer-Encoding header in Go v1.15 which was then brought in Traefik 2.3 in this commit.
In previous versions there was no strict checking and the defective response would go through unnoticed, which might still be the case for other tools like curl for example.
Since this is an issue on the backend and not Traefik I'm closing it, but let me know if you disagree.


'cargo check' and 'cargo clippy' share message cache

Steps to reproduce

Create a new project with the following content:fn main() {
    0;
}

Runcargo clean
cargo check
cargo clippy

Runcargo clean
cargo clippy
cargo check

Expected result
Step 2 should produce the following output:
> cargo clean
> cargo check
    Checking zzz v0.1.0 (C:\temp\zzz)
    Finished dev [unoptimized + debuginfo] target(s) in 0.22s
> cargo clippy
    Checking zzz v0.1.0 (C:\temp\zzz)
warning: statement with no effect
 --> src\main.rs:2:5
  |
2 |     0;
  |     ^^
  |
  = note: `#[warn(clippy::no_effect)]` on by default
  = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#no_effect

    Finished dev [unoptimized + debuginfo] target(s) in 0.40s

Step 3 should produce the following output:
> cargo clean
> cargo clippy
    Checking zzz v0.1.0 (C:\temp\zzz)
warning: statement with no effect
 --> src\main.rs:2:5
  |
2 |     0;
  |     ^^
  |
  = note: `#[warn(clippy::no_effect)]` on by default
  = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#no_effect

    Finished dev [unoptimized + debuginfo] target(s) in 0.40s
> cargo check
    Checking zzz v0.1.0 (C:\temp\zzz)
    Finished dev [unoptimized + debuginfo] target(s) in 0.22s

Actual result
Step 2 produces the following output
> cargo clean
> cargo check
    Checking zzz v0.1.0 (C:\temp\zzz)
    Finished dev [unoptimized + debuginfo] target(s) in 0.22s
> cargo clippy
    Finished dev [unoptimized + debuginfo] target(s) in 0.06s

Step 3 produces the following output
> cargo clean
> cargo clippy
    Checking zzz v0.1.0 (C:\temp\zzz)
warning: statement with no effect
 --> src\main.rs:2:5
  |
2 |     0;
  |     ^^
  |
  = note: `#[warn(clippy::no_effect)]` on by default
  = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#no_effect

    Finished dev [unoptimized + debuginfo] target(s) in 0.40s
> cargo check
warning: statement with no effect
 --> src\main.rs:2:5
  |
2 |     0;
  |     ^^
  |
  = note: `#[warn(clippy::no_effect)]` on by default
  = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#no_effect

    Finished dev [unoptimized + debuginfo] target(s) in 0.09s

Version info
> cargo --version --verbose
cargo 1.44.0-nightly (6e07d2dfb 2020-03-31)
release: 1.44.0
commit-hash: 6e07d2dfb7fc87b1c9489de41da4dafa239daf03
commit-date: 2020-03-31

> cargo clippy --version
clippy 0.0.212 (23549a8c 2020-03-16)clippy 0.0.212 (7907abea 2020-04-03)

I think this was supposed to be fixed in https://github.com/rust-lang/cargo/issues/7635#issuecomment-561248660 , but it wasn't. Latest nightly already includes that commit, but the cache is still shared.

It still requires an update on the clippy side, which is https://github.com/rust-lang/rust-clippy/pull/5363, and it will be opt-in until it is stabilized.

Ok, thank you.

Meanwhile, here is a workaround.
Put this in your Cargo.toml:
[features]
clippy = []

To use separate message caches run cargo check and cargo clippy --features clippy.


Localisation UI issue in TextField

Unable to change text alignments of Card Number, Expiry Date & Security Code TextField.

I'm using the below ThemeDemo.swift file to create the frame style.
ThemeDemo.swift.zip

Hi @Dev-iOS-Appbirds,
I can see that the current API would enable the customisability of that behaviour. Unfortunately the TextFields do not currently use that property.
We have 2 options for this, clean out the external API or extend functionality.
It would be nice to understand if possible, in your use case, would this be an important behaviour to enable the experience your team wants for your users?

hello @alex-ioja-yang-cko
I can see that the current API would enable the customizability of that behaviour. Unfortunately, TextFields do not currently use that property.

hello @alex-ioja-yang-cko
In the current source code is it possible to change the alignment of TextField or you will have to make any changes in your library for this?

This is what we need to decide with Product & Engineering.
As it stands, changing the text alignment in TextField was not planned to be supported and the existence of that property is a symptom of a different code level behaviour that should be improved.
It may help our Product team to make a decision if there are genuine use cases where that property is actually required and improves experience for our merchants users. As it is, they follow the default OS behaviour (left aligned standard and right aligned when RTL - like when using Arabic).

hello @alex-ioja-yang-cko
Yes if you could please add this feature as we have users who are going to use the application and we want our user experience to improve and as you can see the issue in above screenshot attached. Let me know

hello @alex-ioja-yang-cko
Yes if you could please add this feature as we have users who are going to use the application and we want our user experience to improve and as you can see the issue in the above screenshot attached. Let me know if it is possible that there is anything else i can change at my end to solve this problem?

Hey @Dev-iOS-Appbirds, thanks for raising this feedback!
As Alex mentioned, we have followed standard iOS behaviour for left-aligned for languages like English, French etc., and then right-aligned for Arabic.
However, we will keep this feature in mind for a future release, but I don't have a concrete date right now on when we might release a feature to change text alignment within fields.


Publish ychaos images on each release

Summary
@yahoo/ychaos-dev

Publish Python version specific YChaos images on each release

SD Flow

YChaos Doc changes

Checklist
Checklist (Developer)
Prerequisites

[x] I have read the contribution guidelines

Code Analysis

[ ] Covered by Unittests
[ ] Security warnings ignored (Why?!)

Project related

[ ] Schema changed and is backward compatible.
[ ] Introduces a new sub-command for ychaos CLI

Autogenerated Files

[ ] Auto generated schema

Pre-Merge Checklist

[x] All the build jobs are passing

Checklist (Reviewer 1)

[ ] Reviewed Source Code
[ ] Reviewed Tests (If applicable)
[ ] Reviewed Documentation (If applicable)

Checklist (Reviewer 2)

[ ] Reviewed Source Code
[ ] Reviewed Tests (If applicable)
[ ] Reviewed Documentation (If applicable)

This is a great addition to the project. Thanks @vijaybabu4589


A-Frame: Fix double-load when using sync

Replace sync-system with an <altspace-sync> element that derives from ANode. This allows us to block A-Frame from rendering until node.load() is called, which only happens after the sync utility has reloaded the app with a sync instance id in the URL and has established a connection.

Does A-Frame let you put elements before a-assets? Seems like that is the main thing you want to prevent loading, and I thought it required first-child status.


Purr jump eat the grass rip the couch scratched sunbathe, shed everywhere rip the couch sleep in the sink fluffy fur catnip scratched.
the task, Between the style tags, give the p elements font-size of 16px. Browser and Text zoom should be at 100% is not ticked. and shows:
Error: Can't execute code from a freed script

Note: This page is for only reporting issues related to the functionality, maintenance, or improvement of the site. Questions regarding challenges should be posted in one of the various Help Rooms fcc has to offer. I suggest the Help room or even the forum itself. Thanks!

@cosmonaut98
Thanks for reporting this. But your code is incorrect.
GitHub Issues are for reporting bugs on the website only.
Please use the Help chat room for getting challenge related help.
Happy Coding.


feat: AuthModule

fixes #51
fixes #70
Authentication for Writing-Endpoints of API

feature is using this Passport-Strategy
this Strategy required an adjustment in tsconfig.json: The flag esModuleInterop=true had to be added.
Do you know how to implement e2e testing with cucumber? To implement this the Token in the Environment variables always needs to be same, so I'm not quite sure how to do that.

I used the CombinedGuard in the current version, but I also added the single one for the Discordbot, and for the Github Action although they arent used.
Shall I delete them again until we need them, or do we keep them in case we need them in the future?
What do you think is better?

Wow this is great 💥  Thank you @Cahllagerfeld

Btw I think our file naming convention used so far, is files start with a lowercase, but some of these files seem to start with a capital case, could they be made lowercase to be consistent?

sure 👍
I'll adjust that as well.

fixes #70

I dont know hat to adjust the e2e-Tests as the environmentvariables may change from instance to instacne of the app

I fixed the Unittests for now and the e2e-tests run.
There was an issue with default-imports caused by the esModuleInterop=true-Flag in tsconfig.
I researched it and found a solution which doesnt break the default-imports: "allowSyntheticDefaultImports": true
I did some research about cucumber testing here are my key-takeaways:

We probably need new step-defintions for authenticated requests, as we need to send a token-header
Setting environment variables doesnt seem to be that easy in cucumber

I dont know hat to adjust the e2e-Tests as the environmentvariables may change from instance to instacne of the app

Sorry I did not get back to you fast enough, looks like you solved it 👇

I fixed the Unittests for now and the e2e-tests run.

Thank you! 👍 this is great 💥

Furthermore I did some research about cucumber testing, here are my key-takeaways:
We probably need new step-defintions for authenticated requests, as we need to send a token-header
Setting environment variables doesnt seem to be that easy in cucumber

I can take a look at this, but it might be a few days before I am able to

Do not stress yourself with this.
I'll have look at this again during the day as well!

this patch should apply cleanly to your fork and branch issue-51 with git apply 51.patch, any issues let me know
PS. great work on that PR 💥
diff --git a/package.json b/package.json
index b2b0818..32f4392 100644
--- a/package.json
+++ b/package.json
@@ -17,7 +17,7 @@
     "test:watch": "jest --watch",
     "test:cov": "jest --coverage",
     "test:debug": "node --inspect-brk -r tsconfig-paths/register -r ts-node/register node_modules/.bin/jest --runInBand",
-    "test:e2e": "cucumber-js -p default"
+    "test:e2e": "APPROVED_TOKENS=abc,def cucumber-js -p default"
   },
   "dependencies": {
     "@nestjs/common": "^7.6.13",
diff --git a/src/auth/discordGithub.strategy.ts b/src/auth/discordGithub.strategy.ts
index 8606843..8c18fd8 100644
--- a/src/auth/discordGithub.strategy.ts
+++ b/src/auth/discordGithub.strategy.ts
@@ -15,7 +15,7 @@ export class DiscordGithubStrategy extends PassportStrategy(
   async validate(token: string, done) {
     const approvedTokens: Array<string> = this.config
       .get('APPROVED_TOKENS')
-      .split(' ');
+      .split(',');
     if (!approvedTokens.includes(token)) {
       throw new UnauthorizedException();
     }
diff --git a/test/features/discord.feature b/test/features/discord.feature
index c9a5326..f69046c 100644
--- a/test/features/discord.feature
+++ b/test/features/discord.feature
@@ -6,7 +6,8 @@ Feature: discord module
     And the response should be "[]"

   Scenario: add a new user
-    Given make a POST request to "/discord" with:
+    Given authorisation
+    And make a POST request to "/discord" with:
       | bio      | "This is a GitHub Campus Expert"                                                              |
       | username | "khattakdev"                                                                                |
       | socials  | {"discord":"khattakdev","github":"khattakdev","linkedin":"khattakdev","twitter":"khattakdev"} |
diff --git a/test/step-definitions/requests.ts b/test/step-definitions/requests.ts
index b2acb6d..ac23641 100644
--- a/test/step-definitions/requests.ts
+++ b/test/step-definitions/requests.ts
@@ -18,6 +18,11 @@ export class requests {
     await this.context.app.init();
   }

+  @given(/authorisation/)
+  public async authorisation() {
+    this.context.token = 'abc';
+  }
+
   @given(/make a GET request to "([^"]*)"/)
   public async getRequest(url: string) {
     this.context.response = await request(this.context.app.getHttpServer()).get(
@@ -27,9 +32,13 @@ export class requests {

   @given(/make a POST request to "([^"]*)" with:/)
   public async postRequestWithBody(url: string, table: { rawTable: [] }) {
-    this.context.response = await request(this.context.app.getHttpServer())
-      .post(url)
-      .send(this.context.tableToObject(table));
+    const post = request(this.context.app.getHttpServer()).post(url)
+
+    if (this.context.token) {
+      post.set('token', 'abc');
+    }
+
+    this.context.response = await post.send(this.context.tableToObject(table));

     this.context.preRequest = await request(
       this.context.app.getHttpServer(),
diff --git a/test/support/world.ts b/test/support/world.ts
index 708501e..94b472d 100644
--- a/test/support/world.ts
+++ b/test/support/world.ts
@@ -2,6 +2,7 @@ export default class Context {
   public app;
   public response;
   public preRequest;
+  public token;

   public tableToObject(table) {
     return table.rawTable.reduce((result, current) => {

I just need to copy this to a file with .patch right?
I get an error when doing so.. It says error: corrupt patch at line 85, which basically is the last line of this patch.

Do you have any changes locally? This could cause the patch to fail. I created it from your latest commit you pushed to your issue-51 branch. I just tested the patch against that branch and latest commit with no error, maybe you could share your console?
Here is what I did...
Pasted diff content from above into 51.patch
➜  api-cahl git:(issue-51) vim 51.patch

Check there are no new changes locally and apply the patch
➜  api-cahl git:(issue-51) git apply 51.patch

Now there should be changes locally from the patch
➜  api-cahl git:(issue-51) ✗ git status -s
 M package.json
 M src/auth/discordGithub.strategy.ts
 M test/features/discord.feature
 M test/step-definitions/requests.ts
 M test/support/world.ts

I can record my screen if that helps?

I copied this to 51.patch

then I get this result
$ git apply 51.patch
error: corrupt patch at line 85

I dont have any local changes :(

Oh that is strange, can you take a screenshot of what you have on line 85?

Its the last line of the patch

Hmm it looks the same as mine 🤔 the only difference is I have a blank line at the end, I know Linux and GitHub recommend every file to have a blank line at the end with no spaces, but I never thought it would cause something break.

Thats strange haha with a blank line at the end it works :o

lol oh that's good, I learnt something new 🤓

Unfortunately I run into an issue when running npm run test:e2e
I would not rule out the possibility that this is a problem with the operating system. You're using Mac as far as I know, I'm having windows.
$ npm run test:e2e

><EMAIL_ADDRESS>test:e2e C:\Users\Julian\Documents\GitHub\eddiehub-api
> APPROVED_TOKENS=abc,def cucumber-js -p default

Der Befehl "APPROVED_TOKENS" ist entweder falsch geschrieben oder
konnte nicht gefunden werden.
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>test:e2e: `APPROVED_TOKENS=abc,def cucumber-js -p default`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>test:e2e script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR!      C:\Users\Julian\AppData\Roaming\npm-cache\_logs\2021-04-21T12_00_48_050Z-debug.log

Translated it means sth like the command 'APPROVED_TOKENS' is neither writen wrong or couldnt be found

Oh yes, it will be the way I set the environment variable for my Mac (linux style) with APPROVED_TOKENS=abc,def npm run test:e2e. I don't use windows, but maybe we can use the config file from NestJS for that?

I just tested it and we can use .env. I can send you a new patch file or you can make these changes...

package.json:20 back to what it was before with "test:e2e": "cucumber-js -p default"
.env file to have APPROVED_TOKENS=abc,def

We would have to commit the .env then to the repository not just the .env.example.
Well these arent any secret data, its just important then that you use other tokens when taking it to production

I still think we should not commit the .env file, because the first step for anyone or CI (GitHub Actions) will be to duplicate the .env.template file and make it their own.

Okay this seems legit.
I'll implement it as you said :D
We just have to make sure the tests pass at everyone who develops + the ci needs to pass as well.

yep 👍 I think using the .env with NestJS should work for everyone and we will need to make a small tweak to the GitHub Action to pass on CI

This is really coming together and looking great 👍

If we go with this solution, I'll adjust the e2e tests then.
We should definitly document this Authentication thing for e2e-Tests.
Is there already anything planned regarding documentation of the API, or is Swagger enough?

If we go with this solution, I'll adjust the e2e tests then.

Do you feel/think something is missing or there is a better way to do it?

We should definitly document this Authentication thing for e2e-Tests.
Is there already anything planned regarding documentation of the API, or is Swagger enough?

I do love documentation, for the end points I think Swagger is enough, but I think we could have some more info on the authentication side with some sample requests (we can help with this too 🤓 )

Im very happy with this solution as its plattform agnostic and relatively easy to implement (at least in theory) 😄

I do love documentation, for the end points I think Swagger is enough, but I think we could have some more info on the authentication side with some sample requests (we can help with this too 🤓 )
We could provide a rest.http-File for additional Requests.

With .env-File I managed to get them all passed locally.
Now we just need to adjust the actions and we should be ready for reviewing this PR.

FINALLY it works!

Awesome 🎉
Sorry to be a pain, looking at that npm library, the maintainer has archived the project and recommends using alternatives instead  https://github.com/screendriver/cp-cli#this-github-project-has-been-archived-in-favour-of

Sorry I left a few more comments, then I think we are ready to merge - this PR is amazing 💥

I hope I didnt forget anything

I cant believe it :o most intese PR I've done so far :D
Many Many thank for your support !


fix: 3622 - no autofocus and clean FocusNode management

Impacted files:

edit_new_packagings_component.dart: applied best practices for FocusNode (had to switch to StatefulWidget for that)
Podfile.lock: wtf
simple_input_number_field.dart: removed autofocus: true
simple_input_text_field.dart: removed autofocus: true
simple_input_widget.dart: applied best practices for FocusNode

What

The systematic autofocus: true parameter for each text field made no sense and probably provoked the referenced issue.
Instead, putting autofocus: false fixed the issue.
In the process I applied best practices about FocusNode.

Fixes bug(s)

Fixes: #3622

Codecov Report

Merging #3716 (fdbc291) into develop (70bdceb) will decrease coverage by 0.02%.
The diff coverage is 0.00%.

@@             Coverage Diff             @@
##           develop    #3716      +/-   ##
===========================================
- Coverage    10.41%   10.40%   -0.02%
===========================================
  Files          273      273
  Lines        13795    13815      +20
===========================================
  Hits          1437     1437
- Misses       12358    12378      +20

Impacted Files
Coverage Δ

...b/pages/product/edit_new_packagings_component.dart
0.00% <0.00%> (ø)

...p/lib/pages/product/simple_input_number_field.dart
0.00% <ø> (ø)

...app/lib/pages/product/simple_input_text_field.dart
0.00% <ø> (ø)

...oth_app/lib/pages/product/simple_input_widget.dart
0.00% <0.00%> (ø)

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

Thank you @M123-dev for the review!
Can't be worse, we used to say "each field has the top priority!"...

@monsieurtanuki I just got back from a few days off, thank you so much for looking into this!


Updating to ITK version 4.12.2

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.


Time format

function timePickerCallback(val) {
if (typeof (val) === 'undefined') {
console.log('Time not selected');
} else {
var selectedTime = new Date(val * 1000);
console.log('Selected epoch is : ', val, 'and the time is ', selectedTime.getUTCHours(), ':', selectedTime.getUTCMinutes(), 'in UTC');
}
}
at the code above, return value "val" is in what format? second?

It will return the date object in your local time zone. So you can convert it to local or UTC time zone as per your requirement.


Merging in from branch.

Codecov Report

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

@@           Coverage Diff           @@
##           master      #22   +/-   ##
=======================================
  Coverage   88.72%   88.72%
=======================================
  Files           6        6
  Lines         204      204
  Branches       29       29
=======================================
  Hits          181      181
  Misses         23       23

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 3363ad8...1b8801e. Read the comment docs.


Automate Copyright Year

What does this PR do?
Uses new Date().getFullYear() so there's no need to manually update the copyright year.
Test Plan
Manual.
Related PRs and Issues
#165.
Have you read the Contributing Guidelines on issues?
Yes.

[x] QA:


Question about the combination mode

Thank you for the open-source code. When using the module 'DDFPack', what is the difference between the 'add' and 'mul' combination mode?

"mul" means element-wise multiplication between spatial and channel filters
"add" means element-wise addition between spatial and channel filters


Animation with horizontalBars

Hey, just trying out Chartist and loving it so far! I'm having trouble animating stacked horizontal bars, using examples given (#688). Any pointers on what to change within data.element.animate? Thnx in advance.

Please post your question in #688


[User input needed] Remove (CUDA) compute functionality?

The CUDA builds are currently broken again (or rather the tests fail). The CUDA compute functionality (not the low-level futures integration) has been in maintenance mode for quite some time and is unlikely to see improvements in the future. We've kept it alive in case there are users relying on it. We have two options for the 1.8.0 release:

Fix it again.
Just remove it finally.

The latter seems even more appealing if we consider we are not planning to make further improvements to it.
If any users happen to see this, and you're relying on the CUDA functionality, please let us know which parts of it, and how important it is for your use cases.
I'm marking this for 1.8.0 as we should do one or the other before 1.8.0.

Let's remove it. I'm not aware of anybody using it.


Fix video link

Due to changes in Ch9, video link is broken.
Added deeplink to MP4 file.

@juanandmsft : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#label:"aq-pr-triaged"

This is resolved in a newer doc version #please-close


rolling in Dockerfile

Hi, in Docker/Dockerfile, I find the version used is rolling.
It is true to replace all "rolling" to "humble" in branch humble?

Sounds like the right thing to do! Could you please submit a Pull Request with that?

Fixed with #190


How to feed audio via Wi-Fi or BLE?

I can see that the audio data is from recording or auido path;
How to feed audio via Wi-Fi or BLE?

For bluetooth, we automatically connect to the BT microphone if there is one. That's all that possible right now, since our SDK manages all the audio capture.
If there is a viable use case for feeding raw audio (e.g. apple watch app, or audio coming from a different source), we could expose a method that you would use to feed the audio you captured. If that's of interest, please contact me at<EMAIL_ADDRESS>

Appliances with microphones connect to mobile via WiFi or BLE(not earphones),
and  APP runs in background mode.
The audio comes from a appliance that user speak to, and then command sent by APP.

It’s not possible right now, but it would not be that difficult to expose a method that would allow you to feed raw audio.
Best to send me an email to<EMAIL_ADDRESS>to discuss the details.


Show amounts for base currency to favorited currencies.

Is there a way to show the conversion from x units of your base currency to all the currencies you have added as favorites. It seems the reverse is the default.
So if I specify cur 200 and my base currency is USD I want to see how much is 200 USD in all my other currencies.

Please try the latest version and reply if that's what you're looking for.

Looks good. Is there any way to sort the results?

Check out the Setting & Help Methods section which has an animated gif telling you how do achieve that.


remove printStackTrace()

Pull Request Test Coverage Report for Build 569

1 of 2 (50.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.02%) to 52.49%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

commons/gridgo-utils/src/main/java/io/gridgo/utils/ThreadUtils.java
1
2
50.0%

Totals

Change from base Build 562:
0.02%

Covered Lines:
3373

Relevant Lines:
6426

💛  - Coveralls


KYLIN-4099: Using no blocking RDD unpersist in spark cubing job

See: https://issues.apache.org/jira/browse/KYLIN-4099

Codecov Report

Merging #762 into master will decrease coverage by <.01%.
The diff coverage is 0%.

@@             Coverage Diff             @@
##             master    #762      +/-   ##
===========================================
- Coverage     25.71%   25.7%   -0.01%
  Complexity     6020    6020
===========================================
  Files          1387    1387
  Lines         82783   82783
  Branches      11592   11592
===========================================
- Hits          21284   21283       -1
- Misses        59451   59452       +1
  Partials       2048    2048

Impacted Files
Coverage Δ
Complexity Δ

.../apache/kylin/engine/spark/SparkCubingByLayer.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

.../apache/kylin/cube/cuboid/TreeCuboidScheduler.java
63.84% <0%> (-2.31%)
0% <0%> (ø)

...rg/apache/kylin/cube/inmemcubing/MemDiskStore.java
70.21% <0%> (+0.6%)
7% <0%> (ø)
:arrow_down:

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 d659542...34b8637. Read the comment docs.

Pull Request Test Coverage Report for Build 4752

0 of 2 (0.0%)  changed or added relevant lines in 1 file are covered.
2 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.001%) to 28.187%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

engine-spark/src/main/java/org/apache/kylin/engine/spark/SparkCubingByLayer.java
0
2
0.0%

Files with Coverage Reduction
New Missed Lines
%

core-cube/src/main/java/org/apache/kylin/cube/cuboid/TreeCuboidScheduler.java
2
68.46%

Totals

Change from base Build 4751:
-0.001%

Covered Lines:
23334

Relevant Lines:
82783

💛  - Coveralls


Update Electron

Motivation for this change
https://github.com/electron/electron/releases/tag/v12.2.3
https://github.com/electron/electron/releases/tag/v13.6.2
https://github.com/electron/electron/releases/tag/v15.3.2
https://github.com/electron/electron/releases/tag/v16.0.0
Things done

Built on platform(s)

[x] x86_64-linux

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Fits CONTRIBUTING.md.

backport to stable in https://github.com/NixOS/nixpkgs/pull/146453


Add multi-agent example

Hello,
I'd love to see an example of a multi-agent architecture. Ideally it would have a managing agent communicating with a particular set of specialized agents based on the nature of the question being asked by the user. Each specialized agent would have it's own collection of tools that It could reference, and may have its own model.
Perhaps even the tool itself could be tied to a specific model attached to it. For instance, the specialized agent decides to run a sql tool that generates sql, this tool generates sql using a specialized model, then after the query it goes back to its original model so that it can run a pandas/polars for data transformation.
Flow 1:
Managing agent calls an agent to run a task which is required for another specialized agent to run theirs.
May use "human in the loop"
Example:
User wants to pay for the cheapest flight to Alaska(brrr!), managing agent takes this information and decides to pull in a "data analyst" agent to run the task. The agent analyst queries the cheapest flights and associated information from a database and returns back the info to the managing agent.
The managing agent decides to returns this flight info to the user and ask if the user would like  to pay for a ticket("human in the loop"). The user approves, then the prompt goes back to the managing agent who then decides to use the "payment" agent to buy the ticket.
Flow 2:
A managing calls multiple specialized agents to work in parallel.
Example:
A user want to investigate the current market activity of a stock. The managing agent takes that persons question and calls two agents. The first one investigates financial metrics, we'll call this agent the "stock market analyst".
The second agent will be the  "sentiment analyst" agent, this agent is responsible for understanding the sentiment derived from social media channels.  Once both agents are completed with the task, the information will be synthesized by the managing agent and returned back to the user.
You could even add a "human in the loop" aspect at the end asking the user if they would like to purchase more stock.
Similar to part of  Flow 1's example.
Flow 3:
Not quite sure of a good example here, but it would interesting to have a channel of communication between two specialized agents. Perhaps the sentiment analyst or market analyst comes across information that they would like the other to investigate further.

@samuelcolvin is this the right way to do multi-agent orchestration? This is not working for me.
from pydantic_ai import Agent

agent = Agent(
    "openai:gpt-4o",
    system_prompt="""
    You are a helpful assistant. Use your writer agent to help with a given topic.
    """
)

writer = Agent("openai:gpt-4o", system_prompt="Write about a given topic.")

@agent.tool_plain
def writer_agent(topic: str) -> str:
    """
    Ask writer agent to write a poem about a given topic.
    """
    result = writer.run_sync(f"Write a poem about {topic}.").data
    return result

result = agent.run_sync("Saint bernards")
print(result.data)

@jacobweiss2305 I would need to add the agent as a dependency: https://ai.pydantic.dev/dependencies/#agents-as-dependencies-of-other-agents

Any progress on this by chance?

Please provide an example. For example a storyteller agent and 3 character agents. The storyteller guides the plot and the character agents contribute/discuss/act dynamically according to the scene of the plot.

I wrote this agent that uses two agents as tools. Not sure if this helps.
https://github.com/sidsaha-ai/pydantic_ai_learn/blob/master/code/examples/translation/main.py

Thanks everyone for your patience, I've proposed #541 which I hope documents multi-agent applications pretty well.
Full support for graphs is coming too, see #528 and #539.
I'd love your feedback, particularly on #541 at this time.


chore: trust the Boolean :)

People copy-paste this code a lot and I can't help myself :)

"Trust the Boolean"

Freben, Circa 2020


Add testcase coverage for LogAddCommand / Parser, UG documentation for LogAdd Function

Currently no testcase coverage for both LogAddCommand and LogAddCommandParser exists.
To add UG documentation for LogAddCommand under
### 3.2 Log Management

#### Adding a log: `logAdd`

TODO

Test coverage for LogAddCommand is added, still needs to be done for LogAddCommandParser


resolve race condition on c2 startup

After starting gorutines to setup handlers on RPC calls, called SyncC2 function which triggers C2 Profile startup before goroutine setup handler on RabbitMq queue for it. Due to it Profile isn`t started automatically by Mythic server.

Good find! Thanks for making a PR for this! I'll try to get this tested this week


Fix unexpected ConnectionDirection update

Description

A connection direction can unexpectedly change from Outgoing to Incoming in the scenario shown in this diagram. The diagram shows a specific case in PING/PONG. However, if we generalize this, when an Outgoing connection is established and the other node returns WHOAREYOU, the ConnectionDirection will change to Incoming at the time the new session is established. This could potentially cause some problems. e.g. IpVote doesn't work as IpVote ignores Incoming connections.
This PR modifies the handler to give ConnectionUpdate to the service instead of ConnectionDirection so that we can keep an existing connection direction.
Notes & open questions

I have tested this PR with the PING/PONG simulation (https://github.com/sigp/discv5/pull/184), and confirmed that the Outgoing connection directions are kept.
sequenceDiagram
    participant Node1
    participant Node2

    rect rgb(10, 10, 10)
    Note left of Node1: <Node1's KBucket>
[Node2] direction:Outgoing, state:Connected
    end
    Note over Node1,Node2: Session established

    rect rgb(10, 10, 10)
    Note left of Node1: Node1 changes its IP address
 but doesn't update its ENR.
    end
    Note over Node1: *** Change its IP address ***

    Node1 ->> Node2: PING
    Note over Node2: No session with the new IP exists
    Node2 ->> Node1: WHOAREYOU
    Note over Node1: Establish new session
    rect rgb(0, 255, 0)
    Note left of Node1: The Outgoing direction is kept
 when establishing a new session here.
    end
    rect rgb(0, 255, 0)
    Note left of Node1: <Node1's KBucket>
[Node2] direction:***Outgoing***, state:Connected
    end
    Node1 ->> Node2: Auth message
    Note over Node2: Establish one-time session with Node1's new IP
    Note over Node2: Respond the PING using the one-time session
    Node2 ->> Node1: PONG

Change checklist

[x] Self-review
[x] Documentation updates if relevant
[x] Tests if relevant

@AgeManning @divagant-martian Please have a look when you have time.

@AgeManning @divagant-martian I have tested again with the latest changes, and that was fine. 🙆♂ Please have another look.


Remove 'by: :week' option from code

Coverage increased (+0.05%) to 93.876% when pulling d50f0020dbe3de990e5515719b2b3d902f9d1199 on remove-by-week into 3bb1c25c1aa1903c37de88ee5bc8b0643397f2c3 on master.


update

Please update bumpalo since version less then 3.2.1 is unstable
https://github.com/bytecodealliance/wasmtime/blob/a76639c6fb65b3c8427a9f382443fe3b56fd17d1/cranelift/peepmatic/crates/runtime/Cargo.toml
Links:
https://rust.firosolutions.com/paste/e4Q9jYY3eHoSxVUlpIc2Mmzj9RvhwxZ9SGngoQLR4
https://rustsec.org/advisories/RUSTSEC-2020-0006.html

bumpalo 3.2.1 is semver compatible with the required bumpalo 3.2.0, so just updating the Cargo.lock of the user of Cranelift will be enough to use bumpalo 3.2.1 or later. The Cargo.lock of Wasmtime already specifies bumpalo 3.4.0.

Thanks for the reported, but as @bjorn3 mentioned we should be covered here. Projects using wasmtime will pull in the latest bumpalo and we're already using a fixed version too.

awesome


Scoop won't install

When I try running the command given to install scoop, PowerShell tosses me the following (sorry for bad image quality):

What's up here? I already ran the set-executionpolicy (which, by the way, needs cu to be changed to currentuser or else it gives a fatal error) command.

@ReadyP1 It looks like you're using Powershell 2. If you can, upgrade to Powershell 3.
If not, there is a version of Scoop for Powershell 2, but it's not what you would call stable.

@deevus I tried to download WMF 3, but for some reason, it doesn't download. Trying again doesn't help. Is there some sort of direct download link I can get? I'm running Win7 SP1.

@deevus Never mind that download thing. Got it working.

No worries.


Security and Vulnerability Management of the repo

I would like to understand what is the Security and Vulnerability management on this repository. Is there a policy in place documented? Currently I don't see one here.
Are you looking for community support on an ad-hoc basis to resolve an Gihub security findings and open PR's?
Happy to start the conversation rolling and look for opportunities to contribute to this repository.

Very good point @sparachi! We are adopting Bruno in my business company and for that we would also love to have the security profile in place

Do you have an update on this? I'd like to report a security concern, but I cannot find the correct channel. I e-mailed<EMAIL_ADDRESS>to get this information, but I got no reply there.


Revert Development v1.1.3

Compared pull request to the wrong branch.

Compared pull request to correct branch.


password_expirable: improve documentation

Is it possible to mention somewhere that when you add password_expirable to an application with existing users that one should write a migration  that puts a DateTime value in password_changed_at.
If you don't do this, line 17 will return true:
https://github.com/phatworx/devise_security_extension/blob/master/lib/devise_security_extension/models/password_expirable.rb
When you create a new user the value is added automagically, but old users that have no value are stuck in a redirect loop to the password change page.

thank you for this, was getting an error for a missing password_changed_at field


[Question] Is there any script to compute quantitative results under novel lighting, like Table 2 in the paper?

Hello, I want to know if there is a script to test the trained model under given novel lighting and view point?
As far as I know, this method itself does not relight the input object under novel lighting, so we need to export mesh first, import the mesh and materials into render engines like Blender, import the target lighting into Blender, and render the mesh.
Is there any script to help generating the results under novel lighting and compute the error, so I can produce a Table like Table 2 below in the paper?

I would be very grateful if the authors could provide any help (e.g. their code to generate numbers in Table 2). But it is also completely understandable if the authors think it is improper to release or have no such code.
Thanks very much!

Hi,
For table 2 we used novel flashlight but same environment lighting. To render under novel environment you would need to export obj and textures and import them in blender. We don’t have a script for that but it shouldn’t be too difficult using blenders principled bsdf node.
Best
Ziang

Ok, I got it. Nevertheless thanks for your quick reply!

Hi,
For table 2 we used novel flashlight but same environment lighting. To render under novel environment you would need to export obj and textures and import them in blender. We don’t have a script for that but it shouldn’t be too difficult using blenders principled bsdf node.
Best Ziang


ask_filtered should allow users a skip option if they do not want to sel...

adding a skip option to ask_filtered with a description label before the :limited values

Closing since there was no update.


Don't load test sigs by default

test_aulldiv.pat and test_aullrem.pat.gz  are always loaded
while not really a performance impact this can be avoided

we can embed those within fixtures.py rather than distributing them in the sigs directory


Expose DangerfileGitLabPlugin#mr_closes_issues

I didn't test this, because I don't know how, and hoped that it works, being a small change.
I did successfully test part of it using:
project_id = gitlab.mr_json['project_id']
mr_iid = gitlab.mr_json['iid']
closing_issues = gitlab.api.merge_request_closes_issues(project_id, mr_iid)

failure('This merge request does not refer to an existing issue') if closing_issues.empty?

in my Dangerfile though.

Yeah, cool, this looks reasonable to me 👍🏻

Thanks, this is shipped in 8.4.0!

Awesome, thanks!


Update ansible-matrix.md

Ansible 2.5 is released. Add 2.5 into the sheet.
Remove azure_module playbook. Per feedback, two playbook roles make people confused. Let us introduce azure_preview_module first.
Correct the availability for azure_rm_appgwroutetable_facts.

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

@TomArcher, could you please help review it? Thank you.


Add ATSAMG targets

I'd like to support the SAMG55J19 for my project, so would love to see the microchip/atmel SAMG pack pulled into probe-rs.
I attempted to create a pull request using target-gen, but I ran into an issue about a missing 'cores' field in the yaml description. If I've understood correctly from reading commit logs and issues, it appears there's some work ongoing to support multiple cores, and this involves a change to the yaml schema which hasn't yet found its way into target-gen, and I've stumbled in at a bad time to add a part!
In the mean time, I'm working with a fork of 0.11.0 to which I added the SAMG parts.

it appears there's some work ongoing to support multiple cores, and this involves a change to the yaml schema which hasn't yet found its way into target-gen,

Yes, that's unfortunately correct!  :(
It should be easy to manually fix up the YAML to work, if the chip is singlecore.

add cores with a single core named "main", like this:

https://github.com/probe-rs/probe-rs/blob/bbe3bfc4604a7529f6ff3f7a6cf9b1b02cd6cb4f/probe-rs/targets/SAMD51.yaml#L6-L12

Add cores: [main] to all memory map regions

https://github.com/probe-rs/probe-rs/blob/bbe3bfc4604a7529f6ff3f7a6cf9b1b02cd6cb4f/probe-rs/targets/SAMD51.yaml#L19

Add cores: [main] to all flash algos

https://github.com/probe-rs/probe-rs/blob/bbe3bfc4604a7529f6ff3f7a6cf9b1b02cd6cb4f/probe-rs/targets/SAMD51.yaml#L262

This has been improved as much as possible :)
Closing.


Feature: added 2 tests to Point to Polynomial (#36)

Added tests to two of the uncovered lines (35 and 39):
34: if len({tuple(pair) for pair in coordinates}) != len(coordinates): Here I added a test that gives four cordinates, which makes
len({tuple(pair) for pair in coordinates}) = 4
len(coordinates) = 3
and they are no longer equal
38: if len(set_x) == 1:
If there is only one point then we get a straight line, so we use a single coordinate as input to test this.

LGTM


Deploy the windows artifacts to a staging location

What does this PR do?
Deploy the windows artifacts for cloudfoundry into the staging bucket on triggers
Motivation
Have these builds available like the other artifacts.

I ran this with a test pipeline (from another test branch) and it appears to work as expected. I think this should be ready to merge 😄 Thanks for the reviews all, just need one final approval


Fix the tbi test even harder.

A few things here:

I changed the .swift file to be a .sil file. This one is for Jordan. It
should also make the test not run into any issues due to swift level codegen
changes. So we should have an even more robust test.

I fixed the watchossimulator test by eliminating any dependency of this test
on an SDK. This is a pure compiler/target test. Just like the stdlib is not
necessary, SDKs are not necessary either.

rdar://24370377

@swift-ci test


CI stage is failing

Possibly due to updates in the way Node.js emits the status updates during test execution or the format of the output.
@dosu can you help to fix this?

Duplicate


Deep Storage Similar Stack Limit bug

I noticed that LWM's Deep Storage had a more recent update and in brief testing noiticed this:

Build a LWM Deep Storage Meal Rack
Set Similar Stack Limit to 1 or 2 and it works fine
Set Similar Stack Limit to >3 and it seems to be stuck at 2.

Could be related to how I am scanning across all the cells of the deep storage (in this case 2) and trying to count up the total number of stacks. Something possibly changed in either Rimworld 1.3 or with latest DeepStorage.
This ticket is to check into this and fix the behavior so Similar Stack Limit of say 3 works again for Meal Racks.

Got a great report on the behavior here, might be enough to figure out a patch and add a feature flag!
Lurmey Jan 27 @ 9:36am
Just wanna point out that when using this with LWM's Deep Storage, the similar stack limit setting works a little bit differently. Instead of limiting the amount of individual stacks of items it limits the number of tiles within the building that that item can occupy.

So for instance with a set of tall shelves, which can store 8 stacks of items, you can still have 4 whole stacks of one item in one tile with "similar stack limit" set to 1.

This is actually helpful in some cases, but in a lot of others it would be better to limit the number of stacks rather than the number of tiles said stacks can occupy. Could you look into a way to implement both as intended features? :)

I'm experiencing this same problem on RimWorld 1.3.3387

I'm curious how things will work out with the latest 1.4 update that includes vanilla "deep storage" like shelves and custom UI now. The mod landscape has changed a lot in the past few years!


Rename Insert to InsertRow{s}

For consistency and clarity, the BigQueryClient.Insert methods should be called BigQueryClient.InsertRows - except the overloads that only insert a single row.

(And in BigQueryTable too.)


Change OpenCensus to OpenTelemetry with Linkerd

https://github.com/linkerd/linkerd2/pull/6188
OpenTelemetry Go docs: https://opentelemetry.io/docs/go/getting-started/
Gin demo: https://github.com/open-telemetry/opentelemetry-go-contrib/tree/main/instrumentation/github.com/gin-gonic/gin/otelgin/example
Jaeger demo: https://github.com/open-telemetry/opentelemetry-go/blob/main/example/jaeger/main.go

Partially done by #2857 #2858


feature: allow additional arguments after shorthand syntax

This allows to combine the shorthand syntax with additional arguments:
<EMAIL_ADDRESS>OPTIONS "JSON_BUildTests OFF")

This is much shorter than the longer syntax way of writing:
CPMAddPackage(
  NAME nlohmann_json
  VERSION 3.9.1
  GITHUB_REPOSITORY nlohmann/json
  OPTIONS
    "JSON_BuildTests OFF"
)

Hey, thanks for the PR! In general I like the idea of mixing the short syntax with additional arguments and think this is a good approach in doing so. Could you perhaps add an integration test case for this?
Also I'm not sure but it could be that the cmake-format config might need to be updated for this new syntax.

Yes, there are a few things that need to be updated. I wanted to discuss, if there is any chance on success before investing to much time.
I will look through everything and update this PR

Thanks, the changes look good, just want to make sure that the test actually does what it should. One thing that I notice from the readme changes is that having a mix of single + named arguments looks quite inconsistent and therefore harder to read.
How would you feel if we instead make a compromise by introducing a URI argument that takes the role of the single argument instead?
This would allow us to do
CPMAddPackage(
  URI<EMAIL_ADDRESS>  OPTIONS "JSON_BuildTests OFF"
)

Instead of
<EMAIL_ADDRESS>  OPTIONS "JSON_BuildTests OFF"
)

To me this looks much nicer, even though I know CMake itself likes to use this mix of syntax. Of course, using only the single argument syntax should continue to be valid.

I am not so sure of the URI idea.
Pro:

It does look nice.

Con:

Not easy extendible from shorthand syntax, my main idea was that I can just slap on further 'OPTIONS', with this I also need to add a URI
Harder to document. Currently we have two cases: 1. normal usage, 2. shorthand syntax. With URI we kind of have to split the 'normal usage' into two cases. I don't see how this can be documented in any understandable way. (Assuming we don't delete the 2. shorthand syntax)
Also I think it is harder to implement (not so sure about this)

I like to think about it similar to how target_link_libraries works. First I set a free given parameter, and than I add all the other with PRIVATE,PUBLIC or INTERFACE.

Gentleman, how is this coming along?
Is there anything preventing this from getting merged other than the URI or not URI question?

The question of URI or not URI is up to @TheLartians . Both is fine to me, @TheLartians needs to decide what he would like.
The only big stopper for the URI idea for me is, I think URI is a bad keyword, since it is not a URI?!
Any argument for saying it is a URI or an alternative keyword would be helpful.

Thanks for the alternative implementation! I personally prefer that version a lot and added some comments in the other PR.

droped this PR in favor of #617


Add the option to create redis without running redis-trib

In https://github.com/idealista/redis-role/blob/master/templates/cluster-creator.sh.j2#L41, redis-trib connect to the provided IPs and configures the redis-clusters.
The IPs are provided from the play_hosts, meaning the IPs of all of the hosts in which the play occurs. This means that for each host, redis-trib will run.
This causes deadlock, since each hosts in trying to be the "first" node on the cluster, while it waits for the other clusters to join.
Therefore, when running the role on multiple hosts within one play, redis-trib should run only once for the all of the hosts in the play.

Can be solved by adding serial: 1 to the playbook.


[iOS] Add highlight for searchWord in SearchView

Idea Description

Currently, no UI changes for session titles when user input searchWord. Please reflect searchWord to sessionTitle with highlighted effect.

Reference images and links

https://www.figma.com/file/MbElhCEnjqnuodmvwabh9K/DroidKaigi-2023-App-UI?type=design&node-id=58036-78603&mode=dev

🖐

@fumiyasac Thanks for your comment, please go ahead!

@mhidaka
Thanks. I already send pull request about this issue.
I hope I'll merge it. 😍


Live share extension opens in another VSCode opened instance

VSCode Version: 1.44.2
OS Version: Windows 10 Pro 1903, 18362.752, x64

Steps to Reproduce:

Open two projects in two separate VSCode instances
Install Live Share extension (v1.0.1615)
Click the extension icon
Click "Start collaboration session..."
Observe it opening in the other instance which is confusing if you have the project on the ither screen and you have not noticed that VSCode opened the panel but in a different window.

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

/causedByExtension
This is caused by the LiveShare extension. Please file a new issue for it: https://github.com/MicrosoftDocs/live-share/issues


Question: Any plan to support .NET Core?

Hi @charlesw
Any plan to support .NET Core?

Do you intend for it to work cross-platform? Does the [RuntimeDllImport] approach use any APIs not available on .NET core or on certain platforms?

Yes, it should be cross platform. The main issues are more to do with the
integration into .net that's not available in dotnet standard like asp.net
and system.drawing.
My current thoughts are just to exclude these bits from the dotnet standard
target. Though this could create some issues if used from asp.net core.
On Thu., 16 Nov. 2017, 08:34 Aidan Ryan<EMAIL_ADDRESS>wrote:

Do you intend for it to work cross-platform? Does the [RuntimeDllImport]
approach use any APIs not available on .NET core or on certain platforms?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/charlesw/tesseract/issues/378#issuecomment-344735882,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAPzyKzt0hLoH3Icjtp4UY9HIK66VEccks5s21jogaJpZM4QOvhX
.

Great. What aspects of ASP.NET are you using?
I was able to build and run your lib on Windows with a light refactor of the reflection stuff and used a shim for System.Drawing.Imaging: https://www.nuget.org/packages/CoreCompat.System.Drawing.v2
I'll fork and push my work, could be useful at least as a reference, or I can help out with this if you like.

Yes thanks that would be helpful.
As for asp.net that was used to get around how iis creates a shadow copy of
the managed dlls which makes it difficult to find the unmanaged dlls.
On Thu., 16 Nov. 2017, 15:27 Aidan Ryan<EMAIL_ADDRESS>wrote:

Great. What aspects of ASP.NET are you using?
I was able to build and run your lib on Windows with a light refactor of
the reflection stuff and used a shim for System.Drawing.Imaging:
https://www.nuget.org/packages/CoreCompat.System.Drawing.v2
I'll fork and push my work, could be useful at least as a reference, or I
can help out with this if you like.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/charlesw/tesseract/issues/378#issuecomment-344812334,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAPzyJ55GiWNUolYXg7Q4ZZCFLDz96_zks5s27mYgaJpZM4QOvhX
.

OK, here's my fork with the main lib and unit tests ported to NetCore. Next thing would be to try running it on linux.
https://github.com/ajryan/tesseract/commit/f1cad6f22851c4a29cd262c4dbd89a751857ac30

i am looking for new dll file for asp.net code . because i am using OCR for punjabi language but version 3.2 is not including spaces b/w strings and version 3.4 is including space b/w strings. so please assist me how do i update my dll file to 3.4.

It's already done, check out the develop branch. Also see #298.

Your options are:

If you know it's a tiff you can use Pix.LoadTiffFromMemory
Use another library to load image and then create a pic from that. See
the BitmapCoverter class in source.

On Wed., 16 Jan. 2019, 20:25 AsValeO<EMAIL_ADDRESS>wrote:

Hello!
Please, describe how to get Tesseract.Pix from byte[] in .NET Core.
I've tried to use leptonica.net but had no success.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/charlesw/tesseract/issues/378#issuecomment-454709914,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAPzyGbbVoAXUb0lGyF3q8aqMffhon-Vks5vDvAJgaJpZM4QOvhX
.


[Bug] Fix Japanese translation

Describe the bug
Japanese translation is partially incorrect.
I'm native Japanese speaker, so I can fix the discomfort.
If you want it right, I can fix it and create Pull Request.
If fix the comfort, I think more Japanese people use stylebot because they don't distrust it.
Browser
Translation is browser-independent.
To Reproduce
none.
Expected behavior
none.
Screenshots
none.
Additional context
none.

@cohu-dev thanks, please go ahead and submit the pull request to fix the Japanese translation and I will accept it. Thank you

PR: https://github.com/ankit/stylebot/pull/714

Thanks @cohu-dev - I have merged your changes and I will include them in the next release!

Thanks for merging @ankit !
I hope more Japanese notice useful points of stylebot and use this chrome extension.
Thanks for making stylebot!


WIP: tweak ARG processing

Only expand ARGs in COPY instructions if they've been declared in the current stage (changed).
Add a test to make sure that we expand the TARGET and BUILD family of built-in ARGs in FROM instructions, regardless of whether or not they're declared in the heading.
Add a test to make sure that we only expand the TARGET and BUILD family of built-in ARGs in stages after they've been declared in that stage.
Add a field to Builder which can be used to supply override values for the BUILD and TARGET family of predefined arguments.
Flag it as an error if someone tries to redefine any of the BUILD and TARGET family of predefined arguments in a Dockerfile.
Add some godoc for the various arg-related fields in a Builder.
Use strings.Cut() to simplify parsing of ARG=VALUE pairs.

Okay, I think this does what I want it to do.
/retitle Tweak ARG processing, particularly for BUILD... and TARGET...

LGTM
@flouthoc PTAL


Use Compose Multiplatform

Changes

Update AGP to 7.4.0-alpha08.
Update Kotlin to 1.6.21.
Update Compose to 1.2.0-rc02 and add Compose Multiplatform dependency.
Migrate *.gradle Groovy to KTS in order to use Kotlin DSL in common module.
Rename app module to android.
New module common and desktop. Common contains game logic and custom implementation for Composables for android and desktop.
Add internal visibility modifier to some Composables to make them not accessible from android and desktop modules.

Sorry for not checking things on execution during my previous review. After doing so, I've spotted a few things that we could change here (I summarise them here, but you'll also find the specific comment in the corresponding files):

Do not specify the size of the window in JVM, just let it fit the size of the board as it was done in Windows 3.1.
Do not specify a concrete size in NumberDisplay, as this prevents reusing it with a different size (if we ever need to). Instead, resize this composable through the provided modifier parameter.
Specify a width and height attributes on every display-X.svg file. Otherwise, if the composable image holding this SVG does not specify a size through its modifier, the aspect ratio of the SVG is not kept.

I applied the changes that you have proposed. Thanks, I had some issues when I convert XML drawables to SVG.


Skip test_pos_boolarray if numpy>=1.25

Close https://github.com/cupy/cupy/issues/7884.
Part of https://github.com/cupy/cupy/issues/7616.

/test head

71 Errors!

71 Errors!

What happened? lol

there were 117 previously! so the error amount is reduced now 😂

What's special about the "head" CI? I guess I missed something.

@leofang head CI is a test with the latest supported versions of NumPy and SciPy.
https://ci.preferred.jp/cupy.linux.cuda-head/144947/#L155
We are currently working on support for NumPy 1.25, 1.26 and SciPy 1.11, and this PR has reduced the number of test failures of head CI!


nodeByUri/graphcdn fix

This resolves a steady issue causing graphcdn to email us a generic error occurring across a variety of projects.
In #229 we narrowed down that issue to Wordpress supported but not bubs-next supported pathnames for taxonomies. To solve, this PR adds an additional check inside of getStaticContent which verifies that a pathname is actually a contentNode before we try querying contentNode for that path. The function can tell us whether a URL is a contentNode (which we always render), a termNode (we don't render) or neither (for example, a redirect or an actual 404).
This nice thing about this solve instead of hardrcoding a set of taxonomies which is going to change across different projects, we can actually check the source of truth on project taxonomies (which is Wordpress)
An example of this fix in practice:
In production, a 500 error: https://www.nextgenpolicy.org/category/advisory/
In the preview branch with this fix, a 404 http://nextgenpolicy-by91aal7d-patronage.vercel.app/category/advisory/
TODO:

We still need some exceptions for sitemap, feed.

Correcting myself, it's not our api routes (/sitemap.xml and /feed) work fine. But anything random, like /foo.
I get this in ..slug
TypeError: Cannot read properties of null (reading 'isContentNode')
This error happened while generating the page. Any console logs will be displayed in the terminal window.


Link to HIPCC not working

Problem Description
The link to HIPCC on https://rocm.docs.amd.com/en/develop/reference/library-index.html doesn't work

@samjwu, @david-salinas, can one of you take a look?

https://rocm.docs.amd.com/projects/HIPCC/en/latest/


Run unit and E2E tests on MacOS

What this PR does / why we need it
This pull request adds GitHub workflows to run unit and E2E tests on macOS.
Which issue(s) this PR fixes

Fixes #
Describe testing done for PR
The new GitHub workflow run as check for the unit tests  on macOS.

Release note

Additional information
Special notes for your reviewer

Is it the unit tests or the e2e tests we want to run on macOS? The PR title and description says unit tests but the code change is for E2E

Updated, this PR to execute unit and e2e tests in MacOS.


Make a new release with all of the dependency updates.

node version: 12.14.1
npm version: 6.13.7

Error output
  High            Prototype Pollution

  Package         lodash

  Patched in      >=4.17.12

  Dependency of   node-base64-image

  Path            node-base64-image > lodash

  More info       https://npmjs.com/advisories/1065

More details
The fixes have already been merged on master. We just need to make a release for it on NPM.

Done. Thanks for informing.


feat(hisoracle): move Avg Shift and Period to Keeper

Description
Idea: move Avg params to Keeper, to make it easier to change and control in the runtime, when creating the chain.
Esentially, we can update the params in the Keeper after the app is created, and before the app has started.

Codecov Report

Merging #1887 (9223017) into main (8b0184e) will increase coverage by 0.02%.
The diff coverage is 100.00%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

@@            Coverage Diff             @@
##             main    #1887      +/-   ##
==========================================
+ Coverage   56.42%   56.44%   +0.02%
==========================================
  Files          93       93
  Lines        8849     8854       +5
==========================================
+ Hits         4993     4998       +5
  Misses       3486     3486
  Partials      370      370

Impacted Files
Coverage Δ

x/oracle/keeper/params.go
71.92% <ø> (ø)

x/oracle/keeper/historic_avg.go
84.76% <100.00%> (+0.14%)
:arrow_up:

x/oracle/keeper/keeper.go
85.13% <100.00%> (+0.27%)
:arrow_up:


Massage id

Hi! There is a problem with the fact that telegram returns different message ids for the same message. When a new message is sent to Telegram, the UpdateNewMessage receives a Message object with a single message id. When messages are deleted or edited, messages with different message ids are sent to Update Delete Messages or Update Message Content.
For my app, I need to make sure that when you delete or edit a message, the message id is the same as when you send it, otherwise I won't be able to change the message inside the app. Can you tell me how to solve this problem and what it is related to?

See https://github.com/tdlib/td/issues/511.


t2 function does not work with languages which don't have plural forms

This is a bug in ZF2/i18n component. I already sent a pull request to solve this to zf2 repository: https://github.com/zendframework/zf2/pull/6827
However, concrete5 uses version 2.2.* of ZF2/i18n. So, how can I fix this for concrete5.7?

This issue is still not fixed with current develop brunch. Any problems?

This issue is still not fixed with current develop brunch. Any problems?

Yes... https://github.com/concrete5/concrete5/commit/0ba4139e1e0a3549bdedace1726c8488cd70af9f#diff-355fb5f7d1e7fd447871f4cc12ef773eL3117

This issue is still not fixed with current develop brunch. Any problems?

#2460 should fix the zend translator

@hissy could you confirm that everything is working now?

@mlocati I've confirmed this. It is fixed again. Thanks


Training Time

Hi,
Thank you for your great work! What is the training time on Lyft and Nuscenes with 4 NVIDIA 3090 GPUs? Thanks!

Hi! Thanks for your interest! It runs 8 hours / self-training iteration from our experiments.


Last week when?

Was reading through the bit about Hadoop + LustreFS and noticed this sentence:

... Intel's
noticed this and, in version 2.5 of its Hadoop distribution that it quietly
released last week...

Also, "Intel's" should probably be "Intel" or "Intel's Engineers" (?)

I'm going to restructure the "table" with updated information and rework the classification of technologies according with this article [1].
[1] http://www.dataintensive.info/2015/08/03/bigdata-ea-overview/

Happy to help.  Do you have a reference that details Intel Hadoop being discontinued?

http://www.theinquirer.net/inquirer/news/2336750/intel-dropping-its-own-distribution-of-hadoop-big-data-analytics-in-favour-of-cloudera

Thanks @javiroman !


Code improvements & bug fixes for OP-GAN

There were some bugs that caused the code to crash (both with dependencies in original requirements.txt and with newer package versions from environment.yml). I fixed the ones I encountered (there may be more).
Also, I tried to improve (imo) some aspects of the code. For instance, model parameters are now much more configurable.
Changes:

conda instead of pip
remove need to do cd code, just use OP-GAN as working dir, which is nicer because than you can specify data and output dirs etc as data/ instead of ../data/
prints learnable nb of params per network
fixed intermediate image exports (build_super_images), there were bugs with the batch_size being a list, a hard coded number 8 that gives an error (maybe only when current batch_size is smaller than 8), a parameter to a scikit-image function to indicate there is a channel dimension.
moved a lot of constants and hard-to-interpret dimension names from model.py to configs, so they are easier to understand and more configurable
wraps networks in torch.nn.DataParallel when n_gpu > 1, so torch.nn.data_parallel doesn't have to be repeated everywhere
uses .to(device) instead of .cuda() for more device-agnostic code (works on CPU as well)
uses the CUDA_VISIBLE_DEVICES environment variable, so pytorch can only see the GPUs you want it to use, so it won't appear as using unspecified GPUs with 0MB in nvidia-smi
allows to actually set the random seed manually, in config .yml (wasn't the case before)
fixes image resizing in preprocessing so you can use less than 3 discriminators if you want to (gave an error before)
adds DEBUG and DEBUG_NUM_DATAPOINTS parameters to config, so to debug, you can use a smaller version of the dataset
changes all print statements to proper logging, with a timestamp etc, automatically saves these logs to files
fixes a number of deprecation warnings that were displayed, like a deprecated overload of torch's add_

One of the warnings was the following: UserWarning: Default grid_sample and affine_grid behavior has changed to align_corners=False since 1.3.0. Please specify align_corners=True if the old behavior is desired. See the documentation of grid_sample for details docs. I set align_corners=False in stn in model.py, because this seems better, but since the code was developed with pytorch 1.1, and the default behavior was only changed in pytorch 1.3, align_corners=True is likely to be what you used (it was the default). Do you think we should keep it at align_corners=True?

Thanks a lot for this, appreciated.

Thanks a lot for this, appreciated.


Stream model output to UI

Currently the generated answer from the model only gets sent to the client after it is done generating.
It would drastically improve UX if it could instead stream the answer as it generates it, reducing the impression of latency.
This will require implementing Server-sent events in the API.

This should be done as of a60a59318c6b404964e627aca6fa79527ef5a986. I'm not super happy with the code but it seems to be working, so good enough for now I guess.


Azure AD Connect cloud sync - Is PowerShell needed to enable password hash sync?

Is the Enable password writeback in Azure AD Connect cloud sync section on the Azure AD Connect cloud sync installation instructions necessary? It seems that you can enable password hash sync when you create a new sync policy in the Azure Portal or the Entra admin center, which is much more ergonomic than importing a PowerShell module and signing in to your Global Admin account again.
If this step is necessary, an [!IMPORTANT] alert should probably be added to emphasize the importance of this section.

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

ID: e4692b48-2a40-3a4f-bd2c-5ccc8efc2f62
Version Independent ID: e869ea03-68be-9de8-4219-18a4e5e7005e
Content: Install the Azure AD Connect provisioning agent - Microsoft Entra
Content Source: articles/active-directory/hybrid/cloud-sync/how-to-install.md
Service: active-directory
Sub-service: hybrid
GitHub Login: @billmath
Microsoft Alias: billmath

@KennethStebbins
Thanks for your feedback! We will investigate and update as appropriate.

@KennethStebbins
You are correct that you can enable password hash synchronization when you create a new sync policy in the Azure portal or the Entra admin center. This is a more user-friendly way to enable password hash synchronization, and it is recommended that you use this method if possible.
Thank you for your suggestion about adding an [!IMPORTANT] alert to emphasize the importance of this section. I will pass this feedback along to the appropriate team for consideration.
Please add your ideas in below link, so our production team can review it and update the same.
Ideas · Community (azure.com)

Yes you can do it directly through the portal.  See https://learn.microsoft.com/en-us/azure/active-directory/authentication/tutorial-enable-cloud-sync-sspr-writeback#enable-password-writeback-in-sspr
I have updated the install doc and it should include this in the next day.  Thanks!
#please-close


[QUESTION] [REQUEST] Killfeed

Hi, i've been using Osiris for a couple of months and consider it so far the best free cheat for CSGO.
I would like to ask something, is it possible to make an option that makes my killfeed be only normal shots instead of headshots when using aimbot? i mean if it's possible to "hide" the fact that my shots are only headshots due to aimbot?
I know there is an option in Misc called Preserve Killfeed, but that i think only shows headshots or maybe i'm using it wrong, in that case how do i use it? is possible show normal shots instead of head, when using aimbot?
If this is not how it works can anyone explain? if this option doesn't exist i would like to request if possible to add, that way we can appear as we are noobs, when we actually are using some aim :p

if you want other clients to think you aren't hitting headshots then what you're asking for is probably impossible. if you for some reason want it clientside, then i've seen people call the death notice function from memory.

if you want other clients to think you aren't hitting headshots then what you're asking for is probably impossible. if you for some reason want it clientside, then i've seen people call the death notice function from memory.

No no, what i wanted was to hide the fact that i'm doing only headshots, but it's ok if it's not possible.
Still, what does the Preserve Killfeed option do?

Thanks.


docs: considerations on block storage solutions

This adds to the documentation for storage to document the feature of data-locality highlighting the scenarios where instances of the data could be on the same node. This can add a risk because if the node is destroyed (as per treating nodes like cattle) the cluster can be corrupted. The documentation highlights this and suggests certain possible solutions that can be investigated.

Happy to talk about this. @jsilvela

Made the change and approved.
@gbartolini I don't have auth to merge this, can you?
Oh, on @weiyentan 's commits, the signoff is missing. See the DCO failure report:

Commit sha: [a62fbe9], Author: Wei-Yen Tan, Committer: Gabriele Bartolini; The sign-off is missing.
Commit sha: [45d8fb4], Author: Wei-Yen Tan, Committer: Gabriele Bartolini; The sign-off is missing.
Commit sha: [1b86b2c], Author: Wei-Yen Tan, Committer: Gabriele Bartolini; The sign-off is missing.


Issue installing

Hi
Just tried a fresh install on a new VM (Ubuntu Desktop 16.04.6 64-bit), followed instructions but when I tried running tests_training.py I get the following error:
ModuleNotFoundError: No module named 'tensorflow.keras.layers.experimental.preprocessing'
...
ImportError: Keras requires Tensorflow 2.2 or higher.
Obviously a dependency issue...just not sure which has the wrong version... TF or Keras??
Any suggestions?
Thanks
Tim

Hello!
How are you installing aXeleRate? From pip install aXeleRate or pip install git+https://github.com/AIWintermuteAI/aXeleRate ?
Either way it is really strange... I cannot find any code containing tensorflow.keras.layers.experimental.preprocessing --- and I don't remember adding it, because I am yet to update aXeleRate to tf.keras.

Cheers
Tim

Just checked the versions that the pip install installed:

keras: 2.4.2
tensorflow: 1.15.3

I noticed an error when I try to update keras (InvalidArchiveError("Error with archive /home/.... Message from libarchive was: Failed to crate dir 'lib/python3.7/site-packages/tensorflow/python/profiler"   So I deleted that conda env and created a new one.
This time it used keras 2.4.3 and TF 1.15.3
Same error when trying to run tests_training.com.
Then I tried  conda install "keras<=2.3.2" - (which seemed to do nothing more than update tensorflow-base-2.2.0) but when I do conda list I still see keras: 2.4.3 (very strange)
I tried running tests_training.py again.. and it got a bit further, got error:
python tests_training.py
Using TensorFlow backend.
Traceback (most recent call last):
  File "tests_training.py", line 2, in <module>
    from axelerate import setup_training, setup_inference
  File "/home/tim/Dev/ML/aXeleRate/axelerate/__init__.py", line 1, in <module>
    from .train import setup_training
  File "/home/tim/Dev/ML/aXeleRate/axelerate/train.py", line 19, in <module>
    gpu_options = tf.GPUOptions(per_process_gpu_memory_fraction=0.5)
AttributeError: module 'tensorflow' has no attribute 'GPUOptions'

Any suggestions?
Any idea what is different between your environment and mine?   I specifically created this new environment so I could use the same as you.
Thanks again
Tim.

The issue was in Keras package update. I was under the impression that 2.3.1 is going to be last standalone keras version. But tuned out that I was wrong and 2.4.0 was released a few days ago, which is only compatible with tf 2.0.
I changed the requirements for keras from keras >= 2.3.1 to keras == 2.3.1 in this commit
https://github.com/AIWintermuteAI/aXeleRate/commit/947eef377394b11df0424ab5a60b1eeb2a067c7b
Please try creating a fresh environment and installing again - btw if you use GPU it is better to install tensorflow-gpu package from conda, it will automatically install cuda/cudnn as well if you haven't done that already.
Anyways, check the latest commit and see if it resolves the problem :)

Thanks for this.
Just recreated my conda environment.  keras now with 2.3.1 and TF 1.15.3.
Now get the following:
python tests_training.py
Using TensorFlow backend.
Project folder /home/ubuntu/space safety/classifier is created.
Traceback (most recent call last):
  File "tests_training.py", line 125, in <module>
    model_path = setup_training(config_dict=item)
  File "/home/tim/Dev/ML/aXeleRate/axelerate/train.py", line 161, in setup_training
    os.makedirs(dirname)
  File "/home/tim/miniconda3/envs/ml/lib/python3.7/os.py", line 213, in makedirs
    makedirs(head, exist_ok=exist_ok)
  File "/home/tim/miniconda3/envs/ml/lib/python3.7/os.py", line 213, in makedirs
    makedirs(head, exist_ok=exist_ok)
  File "/home/tim/miniconda3/envs/ml/lib/python3.7/os.py", line 223, in makedirs
    mkdir(name, mode)
PermissionError: [Errno 13] Permission denied: '/home/ubuntu'

Where does /home/ubuntu come from?
Thanks again
Tim

From the config dictionary inside tests_training.py
"saved_folder":   		"/home/ubuntu/space safety/classifier",
https://github.com/AIWintermuteAI/aXeleRate/blob/947eef377394b11df0424ab5a60b1eeb2a067c7b/tests_training.py#L41
https://github.com/AIWintermuteAI/aXeleRate/blob/947eef377394b11df0424ab5a60b1eeb2a067c7b/tests_training.py#L78
https://github.com/AIWintermuteAI/aXeleRate/blob/947eef377394b11df0424ab5a60b1eeb2a067c7b/tests_training.py#L110
I'll correct that in the next release, thank you for noticing this nonsense :)
Just correct them to something like "results/name-for-project-folder" for now.

My pleasure!


Getting the correct 'address' field

Getting the right field (we want the IP address)

Both name and IP work, but i'm going to use address to be consistent.  This is not fixed on the 1.10.2 branch.


Fresh compile seems to flare windows defender

Following the build commands for a server seems to flare up windows defender, defense evasion of sorts. Dunno if this should be a GH issue or not but here

Might scare off potential users

Hi there @Just-a-Unity-Dev, thank you very much for your report! Does this happen everytime you're building the program? It looks like you're on Windows 11. Unfortunately, I can't reproduce this issue locally on Windows 10.
I tried using Microsoft's online malware analysis tool, but none of the results seem to detect any threats, either. These are the determinations after scanning with Microsoft Defender Antivirus (Windows 11).

For now, I'll close this issue as "can't reproduce", but please let me know if this problem still occurs.

I just tried to download Dungeon Club on another Windows device and can now confirm that dungeonclub-windows.zip -> server.exe does in fact get flagged as malware on Windows 10.

Thanks again for writing up the issue, I'll keep it open for further investigation.

Hi @doodlezucc, I can confirm that I get the exact same detection (Trojan:Win64/Malgent!MSR) when downloading the latest release (2023-08-28) on a Windows 10 system. Will try to look further into this.


NuGetPublisher check URI and ignore old NuGet.config file

This change will ignore loading any present NuGet.config file and re-write it with a newly generated one. The feed URI will now be checked and the task will fail if an invalid path is used.

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

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

TTYL, MSBOT;

:shipit:
:+1: :package: :package: :package: :package: :package: :package: :+1:


requestAnimationFrame callback arguments not passed in

In alpha36, a requestAnimationFrame's callback correctly gets the timestamp argument:
http://plnkr.co/edit/6ZkphwlO7hUKpoLrBDJw?p=preview
In alpha37, the rAF callback does not get the timestamp argument:
http://plnkr.co/edit/H9bYRx7gDivc5qxkdVB1?p=preview
This causes javascript animations to fail.
Also close: https://github.com/angular/angular/issues/4179

@justindujardin this is probably caused by https://github.com/angular/zone.js/commit/3d6dc0850b20fd86eef5879f8283de5ee668a668
Do you have time to take a look ?


Devbox VM Updates

Disable scrollback on output (per #85).
Increase lines of scrollback (per #85).
Increase font size in terminal (per #85).
Disable screensaver (per #85).
Update to latest Ubuntu server 16.04 ISO.
Update bash hacking to add Minicond3 instead of Miniconda2 to the path - use planemo to fetch it.
Add symbolic links to Galaxy logs to the Desktop.
Cleanup some codebox junk that wasn't deleted properly previously.

Closing in lieu of #100.


[1] Add Git Credential master integration, fields

https://github.com/Shippable/pm/issues/8649

Verified in RC.


fix: fix freeze timestamp for node add command

Description
This pull request changes the following:

Fixed timestamp setting since SetDate and GetDate will not provide a unix timestamp, but a day number within a month

Related Issues

Closes #326

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket: