Questions about re-implementation on other datasets

Hi!
Thank you for your transparent work! Could you provide the files of args/xxx_publish.csv for other datasets, such as imagenet-r, imagenet-a and vtab?

Seconded. Maybe I can provide more details of where I am stuck (only some ones are not reproducing currently, maybe I have some param wrong somewhere here?). Largely, the following files worked for me:
imagenetr_publish.csv
ID,dataset,shuffle,init_cls,increment,model_name,convnet_type,device,seed,batch_size,tuned_epoch,body_lr,head_lr,weight_decay,min_lr,use_RP,M
0,imagenetr,True,200,200,joint_linear,pretrained_vit_b16_224_in21k,0,1993,128,30,0.0,0.01,0.0005,0.0,False,-1
1,imagenetr,True,200,200,joint_full,pretrained_vit_b16_224_in21k,0,1993,128,30,0.0001,0.01,0.0005,0.0,False,-1
2,imagenetr,True,20,20,ncm,pretrained_vit_b16_224_in21k,0,1993,128,0,0.0,0.0,0.0005,0.0,False,-1
3,imagenetr,True,20,20,ncm,pretrained_vit_b16_224_in21k,0,1993,128,0,0.0,0.0,0.0005,0.0,True,0
4,imagenetr,True,20,20,ncm,pretrained_vit_b16_224_in21k,0,1993,128,0,0.0,0.0,0.0005,0.0,True,10000
5,imagenetr,True,20,20,adapter,pretrained_vit_b16_224_in21k_adapter,0,1993,48,20,0.01,0.01,0.0005,0.0,False,-1
6,imagenetr,True,20,20,adapter,pretrained_vit_b16_224_in21k_adapter,0,1993,48,20,0.01,0.01,0.0005,0.0,True,0
7,imagenetr,True,20,20,adapter,pretrained_vit_b16_224_in21k_adapter,0,1993,48,20,0.01,0.01,0.0005,0.0,True,10000

imageneta_publish.csv
ID,dataset,shuffle,init_cls,increment,model_name,convnet_type,device,seed,batch_size,tuned_epoch,body_lr,head_lr,weight_decay,min_lr,use_RP,M
0,imageneta,True,200,200,joint_linear,pretrained_vit_b16_224_in21k,0,1993,128,30,0.0,0.01,0.0005,0.0,False,-1
1,imageneta,True,200,200,joint_full,pretrained_vit_b16_224_in21k,0,1993,128,30,0.0001,0.01,0.0005,0.0,False,-1
2,imageneta,True,20,20,ncm,pretrained_vit_b16_224_in21k,0,1993,128,0,0.0,0.0,0.0005,0.0,False,-1
3,imageneta,True,20,20,ncm,pretrained_vit_b16_224_in21k,0,1993,128,0,0.0,0.0,0.0005,0.0,True,0
4,imageneta,True,20,20,ncm,pretrained_vit_b16_224_in21k,0,1993,128,0,0.0,0.0,0.0005,0.0,True,10000
5,imageneta,True,20,20,adapter,pretrained_vit_b16_224_in21k_adapter,0,1993,48,20,0.01,0.01,0.0005,0.0,False,-1
6,imageneta,True,20,20,adapter,pretrained_vit_b16_224_in21k_adapter,0,1993,48,20,0.01,0.01,0.0005,0.0,True,0
7,imageneta,True,20,20,adapter,pretrained_vit_b16_224_in21k_adapter,0,1993,48,20,0.01,0.01,0.0005,0.0,True,10000

cub_publish.csv
ID,dataset,shuffle,init_cls,increment,model_name,convnet_type,device,seed,batch_size,tuned_epoch,body_lr,head_lr,weight_decay,min_lr,use_RP,M
0,cub,True,200,200,joint_linear,pretrained_vit_b16_224_in21k,0,1993,128,30,0.0,0.01,0.0005,0.0,False,-1
1,cub,True,200,200,joint_full,pretrained_vit_b16_224_in21k,0,1993,128,30,0.0001,0.01,0.0005,0.0,False,-1
2,cub,True,20,20,ncm,pretrained_vit_b16_224_in21k,0,1993,128,0,0.0,0.0,0.0005,0.0,False,-1
3,cub,True,20,20,ncm,pretrained_vit_b16_224_in21k,0,1993,128,0,0.0,0.0,0.0005,0.0,True,0
4,cub,True,20,20,ncm,pretrained_vit_b16_224_in21k,0,1993,128,0,0.0,0.0,0.0005,0.0,True,10000
5,cub,True,20,20,adapter,pretrained_vit_b16_224_in21k_adapter,0,1993,48,20,0.01,0.01,0.0005,0.0,False,-1
6,cub,True,20,20,adapter,pretrained_vit_b16_224_in21k_adapter,0,1993,48,20,0.01,0.01,0.0005,0.0,True,0
7,cub,True,20,20,adapter,pretrained_vit_b16_224_in21k_adapter,0,1993,48,20,0.01,0.01,0.0005,0.0,True,10000

omnibenchmark_publish.csv
ID,dataset,shuffle,init_cls,increment,model_name,convnet_type,device,seed,batch_size,tuned_epoch,body_lr,head_lr,weight_decay,min_lr,use_RP,M
0,omnibenchmark,True,300,300,joint_linear,pretrained_vit_b16_224_in21k,0,1993,128,30,0.0,0.01,0.0005,0.0,False,-1
1,omnibenchmark,True,300,300,joint_full,pretrained_vit_b16_224_in21k,0,1993,128,30,0.0001,0.01,0.0005,0.0,False,-1
2,omnibenchmark,True,30,30,ncm,pretrained_vit_b16_224_in21k,0,1993,128,0,0.0,0.0,0.0005,0.0,False,-1
3,omnibenchmark,True,30,30,ncm,pretrained_vit_b16_224_in21k,0,1993,128,0,0.0,0.0,0.0005,0.0,True,0
4,omnibenchmark,True,30,30,ncm,pretrained_vit_b16_224_in21k,0,1993,128,0,0.0,0.0,0.0005,0.0,True,10000
5,omnibenchmark,True,30,30,adapter,pretrained_vit_b16_224_in21k_adapter,0,1993,48,20,0.01,0.01,0.0005,0.0,False,-1
6,omnibenchmark,True,30,30,adapter,pretrained_vit_b16_224_in21k_adapter,0,1993,48,20,0.01,0.01,0.0005,0.0,True,0
7,omnibenchmark,True,30,30,adapter,pretrained_vit_b16_224_in21k_adapter,0,1993,48,20,0.01,0.01,0.0005,0.0,True,10000

vtab_publish.csv
ID,dataset,shuffle,init_cls,increment,model_name,convnet_type,device,seed,batch_size,tuned_epoch,body_lr,head_lr,weight_decay,min_lr,use_RP,M
0,vtab,True,50,50,joint_linear,pretrained_vit_b16_224_in21k,0,1993,128,30,0.0,0.01,0.0005,0.0,False,-1
1,vtab,True,50,50,joint_full,pretrained_vit_b16_224_in21k,0,1993,128,30,0.0001,0.01,0.0005,0.0,False,-1
2,vtab,True,10,10,ncm,pretrained_vit_b16_224_in21k,0,1993,128,0,0.0,0.0,0.0005,0.0,False,-1
3,vtab,True,10,10,ncm,pretrained_vit_b16_224_in21k,0,1993,128,0,0.0,0.0,0.0005,0.0,True,0
4,vtab,True,10,10,ncm,pretrained_vit_b16_224_in21k,0,1993,128,0,0.0,0.0,0.0005,0.0,True,10000
5,vtab,True,10,10,adapter,pretrained_vit_b16_224_in21k_adapter,0,1993,48,20,0.01,0.01,0.0005,0.0,False,-1
6,vtab,True,10,10,adapter,pretrained_vit_b16_224_in21k_adapter,0,1993,48,20,0.01,0.01,0.0005,0.0,True,0
7,vtab,True,10,10,adapter,pretrained_vit_b16_224_in21k_adapter,0,1993,48,20,0.01,0.01,0.0005,0.0,True,10000

Accuracies Reproduced for RanPAC (-i 7)
ImageNetR- 77.03
ImageNetA- 52.83 (this didn't reproduce -- I tried a bunch of hparams but probably got something wrong?)
CUB- 90.13
VTAB- 92.72
Omnibenchmark- 79.73
Accuracies Reproduced for NCM (-i 2)
ImageNetR- 54.18 (this didn't reproduce -- I tried a bunch of hparams but probably got something wrong?)
ImageNetA- 48.55
CUB- 86.75
VTAB- 85.88
Omnibenchmark- 73.3
Accuracies Reproduced for RANPAC without RP and without PETL (-i 3)
ImageNetR- 64.47 (this didn't reproduce -- I tried a bunch of hparams but probably got something wrong?)
ImageNetA- 50.46 (this didn't reproduce -- I tried a bunch of hparams but probably got something wrong?)
CUB- 85.87
VTAB- 92.0
Omnibenchmark- 73.06
Accuracies Reproduced for RANPAC without PETL (-i 4)
ImageNetR- 68.32 (this didn't reproduce -- I tried a bunch of hparams but probably got something wrong?)

CUB- 89.94
VTAB- 92.88
Omnibenchmark- 78.25

Hi  @LinglanZhao  and @drimpossible - thank you both for your interest. The repository is now updated with args for 10 datasets. They also now have additional args for ResNet and CLIP results from the Appendix.

All the args have been run using python main_tables.py -d <dataset>
This create csv files in paper_tables/ for all args in args/<dataset>_publish/csv
I haven't checked all cases listed by @drimpossible but where I did check, the difference was in the choice of PETL method.
Note that some experiments have greater variability from run to run than others. E.g. ssf in Phase 1

Hi! Reopening this as I ran into problems with the adapter models -- configs for training them seem to not be available.
Could you provide a fix for this? I am trying to reproduce results from this work.


Make ConfigurationLifecycleFeature lazy

ConfigurationLifecycleFeature is initialized early as part of a static initialization path for framework components.  Since it injects ConfigurationProvider it can trigger early initialization of framework components (such as archaius) that are also dependent on static initialization for early instantiation.  This PR changes the behavior such that ConfigurationProvider is lazily instantiated when it is actually used.

lotm


ChatBot Sample Scene flow stuck on "Loading..."

Description
When Play Mode begins in the Unity Editor and after the LLM Server is started, the following two NullReferenceExceptions are thrown and the chat bubble in the scene is stuck on "Loading..."
NRE 1, which does not repeat:

NullReferenceException: Object reference not set to an instance of an object
LLMUnitySamples.InputBubble.FixCaretSorting (UnityEngine.UI.InputField inputField) (at Assets/Samples/LLMUnity/1.0.1/ChatBot/Bubble.cs:183)
LLMUnitySamples.InputBubble..ctor (UnityEngine.Transform parent, LLMUnitySamples.BubbleUI ui, System.String name, System.String message, System.Int32 lineHeight) (at Assets/Samples/LLMUnity/1.0.1/ChatBot/Bubble.cs:144)
LLMUnitySamples.ChatBot.Start () (at Assets/Samples/LLMUnity/1.0.1/ChatBot/ChatBot.cs:49)

NRE 2, which repeats forever:

NullReferenceException: Object reference not set to an instance of an object
LLMUnitySamples.ChatBot.Update () (at Assets/Samples/LLMUnity/1.0.1/ChatBot/ChatBot.cs:136)

To Reproduce

Load a model
Enter Play mode in the Unity Editor

Expected Behavior

NREs do not appear
User is able to interact with the scene

Screenshots/Video

Configuration

Windows 10
Unity 2021.3.34 LTS
LLMUnity v1.0.1
Mixtral-8x7B-Instruct-v0.1-LimaRP-ZLoss-GGUF

Notes
Thanks for creating this project. Great stuff!

Thank you for submitting the issue :clap: !
It probably has to do with the Unity version, I have only tested from the latest 2022 LTS and 2023.
I'm downloading 2021.3.34 LTS and will include a fix in v1.0.2 ASAP!

@kvarher version 2021 works now in v1.0.2!

perfect :)!


TIM-108 feat(auth): add loading state to login form

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#53  👈
#52
#51
#50
#49
#48
#47
#46
#45
#44
#43
#42
#41
#40
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @xdjeremy and the rest of your teammates on  Graphite


Tests not detected with ASP.NET 5 Class Library

Tried with Visual Studio test runner and the Console runner. The former simply does not detect any tests. If there's logs I don't know where they are. Within VS 2015 CTP 5, I am able to create a .NET 4.5 class library, create tests and Fixie detects and runs them as you'd expect. Likewise, the Console runner works with .NET 4.5 assemblies.
I also thought Fixie might work if I created .NET 4.5 outputs from the ASP.NET 5 Class Library project, but this also does not work.
With the Console runner, there is an assembly binding error:
Fatal Error: System.IO.FileNotFoundException: Could not load file or assembly 'Fixie, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null' or one of its dependencies. The system cannot find the file specified. File name: 'Fixie, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null'
Turning on the assembly bind failure logging gives additional, confusing information (at least to me):
=== Pre-bind state information ===
LOG: DisplayName = Fixie, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null
(Fully-specified)
LOG: Appbase = file:///C:/somePath/Visual Studio 2015/Projects/Sample.Web/artifacts/bin/Tests/Debug/net45
LOG: Initial PrivatePath = NULL
Calling assembly : (Unknown).
LOG: This bind starts in default load context.
LOG: Found application configuration file (c:\projectPath\packages\Fixie.<IP_ADDRESS>\lib\net45\Fixie.Console.exe.Config).
LOG: Using application configuration file: c:\projectPath\packages\Fixie.<IP_ADDRESS>\lib\net45\Fixie.Console.exe.Config
LOG: Using host configuration file:
LOG: Using machine configuration file from C:\Windows\Microsoft.NET\Framework\v4.0.30319\config\machine.config.
LOG: Policy not being applied to reference at this time (private, custom, partial, or location-based assembly bind).
LOG: Attempting download of new URL file:///C:/somePath/Visual Studio 2015/Projects/Sample.Web/artifacts/bin/Tests/Debug/net45/Fixie.DLL.
LOG: Attempting download of new URL file:///C:/somePath/Visual Studio 2015/Projects/Sample.Web/artifacts/bin/Tests/Debug/net45/Fixie/Fixie.DLL.
LOG: Attempting download of new URL file:///C:/somePath/Visual Studio 2015/Projects/Sample.Web/artifacts/bin/Tests/Debug/net45/Fixie.EXE.
LOG: Attempting download of new URL file:///C:/somePath/Visual Studio 2015/Projects/Sample.Web/artifacts/bin/Tests/Debug/net45/Fixie/Fixie.EXE.

@richardmatsu and I already discussed this over twitter, but if anyone else is reading this, support for ASP.NET 5 is planned but not implemented yet.  Since ASP.NET 5 is a dramatic overhaul and is currently a moving target, we can't release something useful or stable for it yet.  The xUnit team is facing a similar challenge, and at the time of this writing they cannot proceed until more is released by the ASP.NET team.  See https://twitter.com/xunit/status/567775550433796096

Looks like progress has been made. Interested to see the same here. ;)

@plioi I can look into this if you're interested.

I'm going to close this as a duplicate of #145. ASP.NET Core support is really .NET Core support, and a great deal has changed there since this issue was first reported.


Getting the size of Sneakers using Objectron

I need to figure out how to calculate the size of Sneakers (length/width), as mentioned in the Objectron documentation. Is there any way to get these output directly or I have to write any specific calculators to extract the relevant bounding box coordinates and then need to perform scaling to get correct measurement? Any kind of help would be appreciated.
Happy coding :)

We compute the scale of the box (from the 3D vertices) in the AnnotationToModelMatrices calculator, but it is not directly exported to the user level via any API (only the orientation and position are). You can export it by adding new output streams to that calculator and subsequently to your graph.
Since there are some interest, we might expose the size via a public API in the future.

Thanks for great explanation. I would like to know more about this like, how I can access that output stream at android end by using AAR. Do I need to create separate proto buff or it is possible to access with existing one. One more thing, How I can access other proto buff present inside the Objectron calculators using AAR only. Any code snippet would be helpful for me.

Hi @ReqWrecker, We don't have any direct code. you can try creating Android AAr for objectron from here https://google.github.io/mediapipe/getting_started/android_archive_library.html#mediapipe-android-archive


Improving the powered by image

the image was looking old and traditional
I designed a new image that looks new something
hope you like it

Thank you for taking the time to contribute! This is sincerely appreciated. 😇
(Powered by GitMate.io)


@Authorized in version 0.7.2 stilll return 404.

happen:
   use Koa2 **async** controller with **async** authorizationChecker, still return a 404 in **@Authorized( )** decorator.

version:
   typescript: 2.4.2
   node: 7.6.0
   koa: 2.3.0
   routing-controllers: 0.7.2
   reflect0metadata: 0.1.10

code:
function sleep( time ) {
          return new Promise(( ok ,noOK ) => {
                  setTimeout(( ) => {
                        console.log('ok')
                        ok( );
                   }, time );
            })
}

@JsonController('/users')
export class UserCtrl {

  @Get('/')
  @Authorized( )
  async getAll(  ) {
        await sleep( 1000 );
        return "hahah" // serve return a 404 instead of ‘hahah’
  }
}

const app = new Koa( );
useKoaServer( app, {
  defaults: {
    paramOptions: {
      required: true
    }
  },
  authorizationChecker,
  routePrefix: '/api',
  controllers: [ UserCtrl ]
});
app.listen( 4000 );

async function authorizationChecker( action: Action, roles: string[]) {
  console.log('i am authorizationChecker')
  await sleep( 1000 );
  return true;
}

Again - cannot reproduce:
import { JsonController, Get, Authorized, useKoaServer, Action } from "routing-controllers";
import * as Koa from "koa";

function sleep(time: number) {
    return new Promise((ok, noOK) => {
        setTimeout(() => {
            console.log("ok");
            ok();
        }, time);
    });
}

@JsonController("/users")
export class UserCtrl {
    @Get("/")
    @Authorized()
    async getAll() {
        await sleep(1000);
        return "hahah"; // serve return a 404 instead of ‘hahah’
    }
}

async function authorizationChecker(action: Action, roles: string[]) {
    console.log("i am authorizationChecker");
    await sleep(1000);
    return true;
}

const app = new Koa();
useKoaServer(app, {
    defaults: {
        paramOptions: {
            required: true,
        },
    },
    authorizationChecker,
    routePrefix: "/api",
    controllers: [UserCtrl],
});
app.listen(4000, () => console.log("listening 4000"));

package-lock.json:
"routing-controllers": {
      "version": "0.7.2",
      "resolved": "https://registry.npmjs.org/routing-controllers/-/routing-controllers-0.7.2.tgz",
      "integrity": "sha1-rOEUUBNJPPFCFPsTDsTwiGa8zts=",

i got the same issue, controller method execute properly without authorizationChecker, when using authorizationChecker,i got the 404 .

@martine777
Please post minimal code sample to make me able to reproduce the issue.
Minimal means like the sample above, not a half of your app 😉

@JsonController('/crud')
export class CrudController {
//CrudController.ts
@Get("/:modelName")
// @Authorized(PermissionClaims.createUser)
async listAll( @Param("modelName") modelName: string, @Ctx() ctx) {
     const temp3 = await new Promise((resolve, reject)=>{
         setTimeout(function() {
             resolve("aaaa");
         }, 1000);
     });
     return {status: 'ok', data: temp3};
}

Code sample is a "copy-paste and run", not the fragment of your code that force me to write by myself the bootstrap and the authChecker...

@JsonController('/crud')
export class CrudController {
@Get("/:modelName")
 @Authorized()
async listAll( @Param("modelName") modelName: string, @Ctx() ctx) {
     const temp3 = await new Promise((resolve, reject)=>{
         setTimeout(function() {
             resolve("aaaa");
         }, 1000);
     });
     return {status: 'ok', data: temp3};
}

}

const app = createKoaServer({
routePrefix: "/api",
defaultErrorHandler: false,
classTransformer: true,
authorizationChecker: async (action: Action, claims: string[]) => {
console.log('claims...', claims);
const verifyJwt = jwt(appOptions.jwt).unless({
path:
["/api/noUse", /^/api/auth/]
});
    if (1 == 1) {return Promise.resolve(true) };

},
controllers: [__dirname + "/ctrl/*.js"],
middlewares: [CustomErrorHandler]

});

Middleware({ type: "before" })
export class CustomErrorHandler implements KoaMiddlewareInterface {
constructor(private logger: Logger) {}

async use(context: any, next: (err?: any) => Promise<any>): Promise<any> {
    try {
        const start = Date.now();
        await next();
        const ms = Date.now() - start;
        console.log('ms is ', ms);
        context.set("Response-Time22", `${ms}ms`);
    } catch (error) {
        console.log('error is222 ', error);
        context.status = error.httpCode || error.status || 500;
        if (Array.isArray(error.errors) && error.errors.every((element: ValidationError) => element instanceof ValidationError)) {// lym ���������֤ģ�Ͷ����sanitizer��ܣ���ÿ�����Խ��м��
            let errorMessage = null;
            error.errors.forEach((element: ValidationError) => {
                Object.keys(element.constraints).forEach((type) => {
                    errorMessage = element.constraints[type];
                    return false;
                });
                return false;
            });
            context.body = { message: errorMessage };
        } else {
            context.body = { message: error.message };
        }
        if (context.status === 500) {
            this.logger.error(error.message, error);
        } else {
            this.logger.error(error.message);
        }
    }

}

}

sorry, wait a miniute

i delete the import lines, the code example is all above

const verifyJwt = jwt(appOptions.jwt).unless is no used, please delete it.

Ok, looks like the problem is with new versions of koa/koa-router/sth.
I created a new project folder with fresh install of modules and I can reproduce the issue.
However inside our test suite it works ok, there's no 404.

i am shamed of my poor english ,execuse me:)

Ok, it's more weird - I've bumped the koa deps and still no error in tests.
Here is the reproduce code to clone and npm install:
https://gist.github.com/19majkel94/cbdc281c35bd24f01f219346f5189a8c
@NoNameProvided
I've pushed enhanced test for this case - f506b182af07e9d3881e7f24fa8ee04b16d582a6.
Can you figure out why this stopped working on newly installed projects?

Hey all, I can confirm this bug. I tried it with a freshly created project using the code and steps @19majkel94 posted. Here is my package.json. I'm running on Node v8.5.0 on a Lubuntu 16.04.3 machine with ES2017 as my typescript target.
{
  "name": "test",
  "version": "1.0.0",
  "description": "",
  "main": "index.js",
  "scripts": {
    "test": "echo \"Error: no test specified\" && exit 1"
  },
  "keywords": [],
  "author": "",
  "license": "ISC",
  "devDependencies": {
    "@types/node": "^8.0.29",
    "ts-node": "^3.3.0",
    "typescript": "^2.5.2"
  },
  "dependencies": {
    "@types/koa": "^2.0.39",
    "@types/koa-bodyparser": "^3.0.25",
    "@types/koa-router": "^7.0.23",
    "koa": "^2.3.0",
    "koa-bodyparser": "^4.2.0",
    "koa-router": "^7.2.1",
    "npx": "^9.6.0",
    "routing-controllers": "^0.7.2"
  }
}

Sorry guys, looks like it's fixed and waiting for release 0.7.3.

the issue is not totally settled, i uninstall routing-controller, and install the new "fixed" routing-controllers,when i use both authorizationChecker and middleware,and in controller use promise and setTimeout, it return 404 again!

i uninstall routing-controller, and install the new "fixed" routing-controllers

It's not published yet, do you installed from the master branch?

I'll publish it today. Guys please don't forget (and users to update) CHANGELOG as well. New version is here, but changelog is empty

i install 0.7.3 version, it works correctly, no 404 emerges ever , thx!

Change log is empty because AFAIK they were only bugfixes. Changelog is for more meaningful changes like breaking changes, right? Only custom driver is a new change but useful for 3 peoples in the world 😝
@pleerock
We have a project board to keep track of merged and released PR:
https://github.com/pleerock/routing-controllers/projects/3

Changelog is for more meaningful change

No, actually if should be for bugfixes as well and every change we made, thats why it's called a changelog 😄

Yeah but we can't left versions empty in there. If its only bugs then we need at least write here "fixed bugs #x #y #z"

@NoNameProvided @pleerock
Done in https://github.com/pleerock/routing-controllers/commit/7e5fb34c2cfedbc2f33b6cc3f76818938490505e thanks to the project board 😄

@19majkel94 @pleerock Thx guys! Nice to meet Routing-controllers!   : )


[Feature Request] Types that can be converted between types that implement Serialize/Deserialize should implement Serialize and Deserialize

Various types in Reqwest can in theory implement Serde's Serialize and Deserialize traits but don't. This can get quite annoying when trying to let users configure various parts of HTTP requests via a JSON file. It requires many uses of #[serde(with = "...")] which gets N times more complicated when having Option<T>, HashMap<K, V>, HashSet<T>, etc.
A non-exhaustive list of Reqwest types that could implement Serialize and/or Deserialize pretty easily:

HeaderMap via converting to and from HashMap
HeaderName via as_str and FromStr
HeaderValue via as_str and FromStr
redirect::Policy, due to current implementation details, can only really implement Deserialize. If it kept track of if it was created via Policy::none/Policy::limited it could implement Serialize too
Method via as_str and FromStr
StatusCode via as_u16, FromStr, and TryFrom<u16>
Cookie via the wrapped type
cookie::Jar via the wrapped type
dns::Name via as_str and FromStr

I'll see if I can do this myself, but since I'm not used to the internals of Reqwest or pull requests it might be a bit messy

I don't currently want to implement Serialize or Deserialize on the types. I realize most of the impls would be straightforward, but I don't want to maintain that the format would stay the same.
I think an application wanting to allowing users to provide some of those as config files should define an app-specific type and format.

If the implementation only uses public APIs then there's no maintenance to do that doesn't arise from those already existing public APIs breaking


Incorrect Library search paths in Example Framework Paths

2 Issues:
1.) Estimate library path is ../EstimoteSDK, which is no longer valid now that the examples are nested in an estimote directory. It should be ../../EstimoteSDK
2.) Hardcoded user path

Fixed, let's have some :cake: to celebrate! (:


napalm-prcurve requires manager level permissions

The current implementation automatically calls the enable command on the prompt after the connection has been sucessfully established.  This requires the given ssh key to have the manager permission level though. I would rather prefer to use the principle of least privileges on the netbox user and have the netbox ssh key only on the operator permission level. At this level the enable command can't be used though.
Is there a specific need for the enable command to be executed at the start of the connection? I would otherwise recommend to either patch it out in a PR or hide it behind a configuration to explicitly require it instead of assuming the enable command to be available.

Hmm. If you only use napalm for monitoring - which napalm-procurve works well for, enable mode is not needed.
If you use napalm for configuration management - which napalm-procurve sucks at due to limitations of the ProcurveOS - enable mode is needed.
As napalm is often used for cfgmgmt, enable mode has been used by default...
But I am sure one could put that behind a flag or so...

As napalm is often used for cfgmgmt, enable mode has been used by default...

While napalm can also be used for configuring a given switch, as far as I can tell the functionality provided by this library so far is all about retrieving information from supported switches. Not configuring them. So I can't see the point in entering the enable mode by default. The enable mode can always be entered at a later point in time if there is need for it.

https://github.com/ixs/napalm-procurve/commit/a622f90ac6b2dd305e1e4613e4d84ddeef27966a reworks the connection method which means it has gotten much easier to use.
As a side benefit, there's now a new optional_arg called force_no_enable that skips the enable step when set to true.
That should take care of your issue I believe.
I'll probably tag a 0.6.1 release in the next days or so that should encompass that change. If you wanna test it out beforehand to see if the above commit works for your use-case, that'd be splendid.

This should be resolved in 0.7.0.
Closing the ticket. If you find that the reworked connection handler is posing problems for you, please feel free to reopen the ticket.


ensure values with spaces are quoted

Options like bounding_opts or domain can have multiple parameters or values separated by spaces. If those strings are not quoted then only the first part is used.
This will add a munge block to those options to ensure the value is quoted if it contains spaces. It will also keep the quotes in the value read from the disk if it contains spaces in order for this to work.
Since changing things like this will restart the network interfaces if configured like this I tried to keep the impact as small as possible so it only changes values where absolutely necessary.

This PR is superseded by the changes in https://github.com/crayfishx/network_config/pull/37


fix: next externals

This is needed since Next.js requires the React to be the same instance in every page. Otherwise, React would be injected individually in every page.
We're using the equivalent regex that the default configuration uses: zeit/next.js

You may release it! 🎉


Bot leaves channel and doesn't come back

[Download] Started: https://www.youtube.com/watch?v=VLRZFLKEXS4
[Download] Complete: https://www.youtube.com/watch?v=VLRZFLKEXS4

Connected!  Musicbot v1.9.5_2

Bot:<PHONE_NUMBER>32996311/Music Bot#6509
Owner:<PHONE_NUMBER>10882912/Anon#3645

Server List:
 - Anonymous

Bound to text channels:
 - Anonymous/music

Autojoining voice chanels:
 - Anonymous/Music

Options:
  Command prefix: !
  Default volume: 50%
  Skip threshold: 100 votes or 50%
  Now Playing @mentions: Disabled
  Auto-Summon: Disabled
  Auto-Playlist: Disabled
  Auto-Pause: Enabled
  Delete Messages: Disabled
  Debug Mode: Disabled
  Downloaded songs will be saved

Attempting to autojoin Music in Anonymous

[Command]<PHONE_NUMBER>28030208/!!panch (!queue)
[Command]<PHONE_NUMBER>28030208/!!panch (!play https://soundcloud.com/itskoe/the-farewell)

The bot does not join the channel, using commands still works but obviously no music is heard. Using !summon makes the bot come back in the voice-channel but the audio can not be heard.

Same problem

Same here, I have to close the entire bots cmd down and open it back up to start my music again, a simple %restart doesn't work for me.

restarting works for me, but wondering why it leaves in the first place

Try updating the bot if this issue is still occurring.


v-charts中的extend属性与vue中的extend冲突报错

Summary 简述
Method "extend" has already been defined as a prop.
Avoid mutating a prop directly since the value will be overwritten whenever the parent component re-renders. Instead, use a data or computed property based on the prop's value. Prop being mutated: "extend"
Expect 期望结果
Reproduce 重现示例

What is the solution


fix(TDI-39582): tSalesforceOutput set invalid dates

What is the current behavior? (You can also link to an open issue here)
Compare to old framework, salesforce lose RawOffset and DSTSavings when write date to salesforce with timezone.
What is the new behavior?
Keep same behavior with old framework.
Please check if the PR fulfills these requirements

[x] The commit message follows Talend standard
[x] Tests for the changes have been added (for bug fixes / features)
[x] Docs have been added / updated (for bug fixes / features) ?
[x] The code coverage on new code >75%
[x] The new code does not introduce new technical issues (sonar / eslint)

What kind of change does this PR introduce?

[x] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build / CI related changes
[ ] Other... Please describe:

Does this PR introduce a breaking change?

[ ] Yes
[x] No

If this PR contains a breaking change, please describe the impact and migration path for existing applications: ...
Other information:
For issue : https://jira.talendforge.org/browse/TDI-39582

Pull request test result: success :star:
Sonar results


New PR to test SignCLA at 2019-06-25 14:14:02.911092256 -0700 PDT m=+6.449640121

Body of PR

unsigned-fail-fork-staging: test succeeded


quantization/add smooth quant for torch backend

Type of Change
feature or bug fix or documentation or validation or others
API changed or not
Description
detail description
JIRA ticket: xxx
Expected Behavior & Potential Risk
the expected behavior that triggered by this PR
How has this PR been tested?
how to reproduce the test (including hardware information)
Dependency Change?
any library dependency introduced or removed

@ftian1 please review commit 48e2a83b4bbbe2a7b2282e8062890bef06f86494 for algo trigger


Accordion using Collapse.show

We have been implementing bootstrap 4 collapse.show property on an accordion.  There is an issue with accordion staying open, unable to utilize the trigger to collapse the accordion.
We currently have the collapse.show property on the accordion that we want defaulted open, when the page loads we are unable to close the accordion.  We did have this working with Bootstrap 4 a few days ago, are there any changes that would have caused this issue?
Thank you!

Depends on what version of the code you have. Please include more details, and ideally a live demo. We've made changes since Alpha 6, but without more info, all I can say is to go look through the history of the relevant JS or docs files.

Bug reports must include a live demo of the problem. Per our contributing guidelines, please create a reduced test case via JS Bin and report back with your link, Bootstrap version, and specific browser and OS details.
This is a saved reply.

I ran into similar issue (collapse.js v3.1.1-3.3.7) wherein the transitioning flag would not be set back to 0 leading to following behavior based on whether element is first shown or hidden:

element is hidden by default, show doesn't trigger complete and therefore hide would not be triggered in next toggle.
element is shown by default, hide doesn't trigger complete and therefore show would not be triggered in next toggle.

Following Fix worked for me:
Replace bsTransitionEnd in following code with $.support.transition.end
    this.$element
      [dimension](0)
      .one('bsTransitionEnd', $.proxy(complete, this))
      .emulateTransitionEnd(Collapse.TRANSITION_DURATION)

Hey there!
We're automatically closing this issue since the original poster (or another commenter) hasn't yet responded to the question or request made to them 14 days ago. We therefore assume that the user has lost interest or resolved the problem on their own. Closed issues that remain inactive for a long period may get automatically locked.
Don't worry though; if this is in error, let us know with a comment and we'll be happy to reopen the issue.
Thanks!
(Please note that this is an automated comment.)


How to use this model as oVAD

First, let me thank you for this useful code and important research.
By the way, is this code designed to apply to wVAD? In the paper it says the same model can be applied to oVADs. it would be great if you could tell me how to use it as an oVAD. Thanks a lot.

Same question.


Issue #138: Fix instability in test_vxlan_ecmp sonic-mgmt test

why
Addressing issue #138:
test_vxlan_ecmp sonic-mgmt test failed occasionally. This is because vl_api_bfd_udp_session_event_t_handler calls set_reply_status. The callback is for unsolicited event (bfd state update). set_reply_status should not be called for unsolicited event and it can mess up other requests. In our case, it is vxlan tunnel create request.
In the long run, we should make request/response more robust by setting request_message_id in the context of request to match its response.
What this PR does
remove set_reply_status from vl_api_bfd_udp_session_event_t_handler
Also fixed bridge domain ID resource leak. BD ID is not put back to the resource pool when vxlan decap is deleted.

Hey, we have PR https://github.com/sonic-net/sonic-sairedis/pull/1473 that have move src from sonic-platform-vpp to virtual switch in sairedis but its not compiling yet, since we dont have pipeline setup in this repo to produce deb packages and i alreayd saw taht you are adding new changes here which get out of sync + @lguohan


Please help me..

How to download these files into my 'lang' directory.. Chrome download them as html
Also where 'lang' dir. must be put ? just on tesseract main directory ?
Thanks in advance..

@zdenop Please close.


Add support for AWS session tokens

Not tested, attempts to provide an elegant alternative to https://github.com/crohr/rpm-s3/pull/16

And moving to iterative org


Oscillations

Description
Create the markdown file for Oscillations as per the reference content.
Lesson PDF (Tamil and English)
10. Oscillations.pdf
10. அலைவுகள்.pdf

assign to @prajanlal


[fud2,tb] Implement testbench integration

I'm not good at fud2 stuff but I think I'm on the right track

Ideally I'd like to move the logic in the tb tool into fud2, but I don't really see that being possible.

Also closes #2162

A TODO thing would be to take advantage of the fact that the verilog incoming is calyx and write a new tester that allows you to interface with calyx components preserving the invariants (in addition to just being able to use cocotb and verilator as currently)

Waiting on #2193

fud2 tests failing because of I hack I added before multi-input ops were a thing. @jku20, I assume they are now?

For some reason, it seems all my commits became unverified?

Now they're verified again...

@ethanuppal what's the status of this PR and what would it take to get it merged?

@rachitnigam I've been working with @EclecticGriffin on using cider2 programmatically; once I have the adder component fuzz test working (under tools/tb/examples/calyx/), I'll fix all the issues so that it can be merged.

@rachitnigam it works!! (for main components due to a limitation in cider right now where you can't run anything else; a hack involves just modifying the Context by setting the entrypoint string

This is blocked on there being no usable support for relative paths in procedural macros.

There are two options:

We figure out a different approach for the testbench that does not rely on the macro magic but is still useful for users writing complex testbenches, or
We require users to provide a shim of the component they want to test and generate the ports based on that

I also have a broader question: there are currently no users of this harness. Is there any buy-in from the various teams to use this? That might better inform which use cases actually matter.

Marking this as stale. @ethanuppal feel free to remove and add thoughts if on how we can get this merged without the nightly features.

I'm going to remove stale since we found a way to get relative paths working.

@EclecticGriffin when you get a chance, could you take a look at why the interpreter tests are failing? Is it the fix I made to subtraction (which is fuzz-tested to be correct in an example I wrote through calyx FFI!) breaking it due to subtraction underflow?

This tool is out of date with respect to the way values work in Cider currently. We replaced the janky Value implementation with baa::BitVecValue. You'll need to merge in the mainline changes but that should fix this


Pico proxy as nodelet

Tested local, awaiting robot test

Tested in the lab

platform submodule update to working rootfs

Tested on the robot

I agree with Katy that we should try to get rid of the global variable. If it is crashing we should try to understand why. Another option is to make it a static local variable so it won't be reinitialized.

Actually I was just initializing it wrong, fixed it xD


Docs: SDK developer orientation

While SDF is focusing efforts on Rust, some other teams are interested in building fr other toolchains. In the absence of a more complete spec, we should at least give some pointers as to how one would approach developing a non-rust sdk.

@graydon I picked this up missing that you were assigned to the task already. Hopefully you weren't also working on it. I assume not because the status of the task was not in progress. Let me know if you had also written anything and I'm happy to spend time incorporating / merging them. What I wrote is in #113.


Missing LICENSE File

I know you added in the README.md that You are welcome to use or adapt this code for your own content. but no LICENSE file.
If there is a LICENSE file it would explain in more detail on what you can do with your source code.

The other code is licensed under MIT, this one isnt. That sadly makes it technically unable to use it legally.


[move_to_map_pokemon_fail] Failure! Map data was not valid

Expected Behavior
The bot should snipe using Pokemongo-Map. Map is running on same machine without problems and sniping have worked before.
Actual Behavior
[MoveToMapPokemon] [INFO] [move_to_map_pokemon_fail] Failure! Map data was not valid
Your FULL config.json (remove your username, password, gmapkey and any other private info)
{
"auth_service": "",
"username": "",
"password": "",
"location": "",
"gmapkey": "",
"tasks": [
{
"type": "HandleSoftBan"
},
{
"type": "CollectLevelUpReward"
},
{
"type": "IncubateEggs",
"config": {
"longer_eggs_first": true
}
},
{
"type": "TransferPokemon"
},
{
"type": "PokemonOptimizer",
"config": {
"// the 'transfer' parameter activate or deactivate the transfer of pokemons": {},
"// at false, no pokemon is going to be transfered, ever": {},
"// at false, you will still get the log information of what the optimizer": {},
"// would have transfered if the parameter was true": {},
"transfer": true,
"// the 'evolve' parameter activate or deactivate the evolution of pokemons": {},
"// at false, no pokemon is going to be evolved, ever": {},
"// at false, you will still get the log information of what the": {},
"// optimizer would have evolved if the parameter was true": {},
"evolve": true,
"// the 'use_candies_for_xp' parameter let you choose if you want the optimizer": {},
"// to use your candies to evolve low quality pokemons in order to maximize your xp": {},
"// at false, the optimizer will still use candies to evolve your best Pokemons": {},
"use_candies_for_xp": false,
"// the 'use_lucky_egg' parameter let you choose if you want the optimizer": {},
"// to use a lucky egg right before evolving Pokemons. At false; the optimizer": {},
"// is free to evolve Pokemons even if you do not have any lucky egg.": {},
"use_lucky_egg": false,
"// the 'evolve_only_with_lucky_egg' parameter let you choose if you want the optimizer": {},
"// to only Evolve Pokemons when a lucky egg is available": {},
"evolve_only_with_lucky_egg": false,
"// the 'minimum_evolve_for_lucky_egg' parameter let you define the minimum": {},
"// number of Pokemons that must evolve before using a lucky egg": {},
"// If that number is not reached, and evolve_only_with_lucky_egg is true, evolution will be skipped": {},
"// If that number is not reached, and evolve_only_with_lucky_egg is false,": {},
"// evolution will be performed without using a lucky egg": {},
"minimum_evolve_for_lucky_egg": 90,
"// the 'keep' parameter let you define what pokemons you consider are the 'best'. These Pokemons": {},
"// will be keep and evolved. Note that Pokemons are evaluated inside their whole family": {},
"// Multiple way of ranking can be defined. Following configuration let you keep the best iv,": {},
"// the best ncp and the best cp": {},
"keep": [
{
"// Following setting let you keep the best iv of the family": {},
"// the 'top' parameter allow you to define how many Pokemons you want to keep": {},
"// at the top of your ranking. If several Pokemons get the same score, they are": {},
"// considered equal. Thus, top=1 might result in keeping more than 1 Pokemon.": {},
"top": 1,
"// the 'evolve' parameter let you choose if you want to evolve the Pokemons you keep": {},
"evolve": true,
"// the 'sort' parameter define how you want to rank your pokemons": {},
"// Critera are sorted fro, the most important to the least important.": {},
"// Available criteria are:": {},
"// 'iv' = individual value": {},
"// 'ivcp' = iv weigted so that for equal iv, attack > defense > stamina": {},
"// 'cp' = combat power (can be increased with candies)": {},
"// 'cp_exact' = combar power (not rounded)": {},
"// 'ncp' (normalized cp) or 'cp_percent' = ratio cp / max_cp": {},
"// iv_attack = attach component of iv": {},
"// iv_defense = defense component of iv": {},
"// iv_stamina = stamina component of iv": {},
"// dps = raw dps based on the moves of the pokemon": {},
"// dps_attack = average dps when attacking": {},
"// dps_defense = average dps when defending": {},
"// Note that the more criteria you add to this list, the less likely Pokemons": {},
"// will be equals": {},
"sort": ["iv"]
},
{
"// Following setting let you keep keep the best normalized cp of the family": {},
"// That is the Pokemon with higher CP once fully evolved": {},
"top": 1,
"evolve": true,
"sort": ["ncp"]
},
{
"// Following setting let you keep keep the best cp of the family.": {},
"// But will not evolve it further (in favor of the best ncp)": {},
"top": 1,
"evolve": false,
"sort": ["cp"]
}
]
}
},
{
"type": "RecycleItems",
"config": {
"item_filter": {
"1": { "keep" : 20 },
"2": { "keep" : 30 },
"3": { "keep" : 60 },
"101": { "keep" : 0 },
"102": { "keep" : 0 },
"103": { "keep" : 15 },
"104": { "keep" : 25 },
"201": { "keep" : 10 },
"202": { "keep" : 15 },
"701": { "keep" : 50 },
"801": { "keep" : 50 }
}
}
},
{
"type": "CatchPokemon",
"config": {
"catch_visible_pokemon": true,
"catch_lured_pokemon": true,
"min_ultraball_to_keep": 5,
"catch_throw_parameters": {
"excellent_rate": 0.1,
"great_rate": 0.5,
"nice_rate": 0.3,
"normal_rate": 0.1,
"spin_success_rate" : 0.6
},
"catch_simulation": {
"flee_count": 3,
"flee_duration": 2,
"catch_wait_min": 2,
"catch_wait_max": 6,
"berry_wait_min": 2,
"berry_wait_max": 3,
"changeball_wait_min": 2,
"changeball_wait_max": 3
}
}
},
{
"type": "SpinFort",
"config": {
"spin_wait_min": 2,
"spin_wait_max": 3
}
},
{
"type": "MoveToMapPokemon",
"config": {
"address": "http://<IP_ADDRESS>:5000/",
"max_distance": 500,
"min_ball": 50,
"prioritize_vips": true,
"snipe": true,
"snipe_high_prio_only": true,
"snipe_high_prio_threshold": 400,
"update_map": false,
"mode": "priority",
"map_path": "raw_data",
"catch": {
"==========Legendaries==========": 0,
"Aerodactyl": 1000,
"Snorlax": 1000,
"Articuno": 1000,
"Zapdos": 1000,
"Moltres": 1000,
"Dratini": 1000,
"Dragonair": 1000,
"Dragonite": 1000,
"Mewtwo": 1000,
"Mew": 1000,
      "==========Region Locked==========": 0,
      "Farfetch'd": 1000,
      "Kangaskhan": 1000,
      "Mr. Mime": 1000,
      "Tauros": 1000,

      "==========Very Rare==========": 0,
      "Lapras": 900,
      "Electabuzz": 900,
      "Magmar": 900,
      "Ditto": 900,

      "==========Starters==========": 0,
      "Bulbasaur": 400,
      "Ivysaur": 600,
      "Venusaur": 1000,

      "Charmander": 400,
      "Charmeleon": 600,
      "Charizard": 1000,

      "Squirtle": 400,
      "Wartortle": 600,
      "Blastoise": 1000,

      "Pikachu": 600,
      "Raichu": 1000,

      "==========Semi Rare==========": 0,
      "Porygon": 200,
      "Scyther": 200,

      "==========Uncommon==========": 0,

      "Omastar": 500,

      "Dewgong": 500,

      "Muk": 500,

      "Cloyster": 500,

      "Haunter": 500,
      "Gengar": 1000,

      "Hypno": 600,

      "Vulpix": 200,
      "Ninetales": 600,

      "Wigglytuff": 500,

      "Dugtrio": 500,

      "Vaporeon": 800,
      "Jolteon": 800,
      "Flareon": 800,

      "Kabutops": 500,

      "Gyarados": 800,

      "Rapidash": 500,

      "Slowbro": 500,

      "Magnemite": 250,
      "Magneton": 500,

      "Kingler": 500,

      "Electrode": 500,

      "Exeggcutor": 500,

      "Hitmonlee": 400,

      "Hitmonchan": 400,

      "Lickitung": 500,

      "Weezing": 500,

      "Rhydon": 500,

      "Chansey": 800,

      "Tangela": 300,

      "Seadra": 600,

      "Seaking": 500,

      "Starmie": 800,

      "==========T1 Evolvers==========": 0,
      "Butterfree": 500,

      "Beedrill": 500,

      "Pidgeot": 300,

      "==========T2 Evolvers==========": 0,
      "Oddish": 100,
      "Gloom": 200,
      "Vileplume": 600,

      "Poliwhirl": 400,
      "Poliwrath": 800,

      "Abra": 300,
      "Kadabra": 600,
      "Alakazam": 800,

      "Machoke": 400,
      "Machamp": 800,

      "Weepinbell": 400,
      "Victreebel": 800
    }
  }
},
{
  "type": "MoveToFort",
  "config": {
    "lure_attraction": true,
    "lure_max_distance": 2000
  }
}

],
"map_object_cache_time": 5,
"forts": {
"avoid_circles": true,
"max_circle_size": 50
},
"websocket_server": false,
"walk_min": 13,
"walk_max": 23,
"action_wait_min": 1,
"action_wait_max": 5,
"debug": false,
"test": false,
"health_record": false,
"location_cache": true,
"distance_unit": "km",
"reconnecting_timeout": 1,
"catch_randomize_reticle_factor": 1.0,
"catch_randomize_spin_factor": 1.0,
"catch_throw_parameters": {
"excellent_rate": 0.7,
"great_rate": 0.5,
"nice_rate": 0.3,
"normal_rate": 0.1,
"spin_success_rate" : 0.8
},
"catch": {
"any": {"catch_above_cp": 0, "catch_above_iv": 0, "logic": "or" },
"// Pokemons with example": { "always_catch": true },
"// Gets filtered with release parameters": {},

"// Legendary pokemons (Goes under S-Tier)": {},
"Lapras": { "always_catch": true },
"Moltres": { "always_catch": true },
"Zapdos": { "always_catch": true },
"Articuno": { "always_catch": true },

"// S-Tier pokemons (if pokemon can be evolved into tier, list the representative)": {},
"Mewtwo": { "always_catch": true },
"Dragonite": { "always_catch": true },
"Snorlax": { "always_catch": true },
"// Mew evolves to Mewtwo": {},
"Mew": { "always_catch": true },
"Arcanine": { "always_catch": true },
"Vaporeon": { "always_catch": true },
"Gyarados": { "always_catch": true },
"Exeggutor": { "always_catch": true },
"Muk": { "always_catch": true },
"Weezing": { "always_catch": true },
"Flareon": { "always_catch": true },

"// Growlithe evolves to Arcanine": {},
"Growlithe": { "always_catch": true },
"// Dragonair evolves to Dragonite": {},
"Dragonair": { "always_catch": true },
"// Grimer evolves to Muk": {},
"Grimer": { "always_catch": true },

"// Magikarp evolves to Gyarados": {},
"Magikarp": { "always_catch": true },
"// Exeggcute evolves to Exeggutor": {},
"Exeggcute": { "always_catch": true },
"// Eevee evolves to many versions, like Vaporeon, Flareon": {},
"Eevee": { "always_catch": true },

"// A-Tier pokemons": {},
"Slowbro": { "always_catch": true },
"Victreebel": { "always_catch": true },
"Machamp": { "always_catch": true },
"Poliwrath": { "always_catch": true },
"Clefable": { "always_catch": true },
"Nidoking": { "always_catch": true },
"Venusaur": { "always_catch": true },
"Charizard": { "always_catch": true },
"Golduck": { "always_catch": true },
"Nidoqueen": { "always_catch": true },
"Vileplume": { "always_catch": true },
"Blastoise": { "always_catch": true },
"Omastar":  { "always_catch": true },
"Aerodactyl": { "always_catch": true },
"Golem": { "always_catch": true },
"Wigglytuff": { "always_catch": true },
"Dewgong": { "always_catch": true },
"Ninetales": { "always_catch": true },
"Magmar": { "always_catch": true },
"Kabutops": { "always_catch": true },
"Electabuzz": { "always_catch": true },
"Starmie": { "always_catch": true },
"Jolteon": { "always_catch": true },
"Rapidash": { "always_catch": true },
"Pinsir": { "always_catch": true },
"Scyther": { "always_catch": true },
"Tentacruel": { "always_catch": true },
"Gengar": { "always_catch": true },
"Hypno": { "always_catch": true },
"Pidgeot": { "always_catch": true },
"Rhydon": { "always_catch": true },
"Seaking": { "always_catch": true },
"Kangaskhan": { "always_catch": true }

},
"release": {
"any": {"keep_best_cp":1, "keep_best_iv":1 },
"Eevee": {"keep_best_iv":2 },
"Dratini": {"keep_best_cp":3, "keep_best_iv":2}

},
"vips" : {
"Any pokemon put here directly force to use Berry & Best Ball to capture, to secure the capture rate!": {},
"any": {"catch_above_cp": 1200, "catch_above_iv": 0.9, "logic": "or" },
"Lapras": {},
"Moltres": {},
"Zapdos": {},
"Articuno": {},
"// S-Tier pokemons (if pokemon can be evolved into tier, list the representative)": {},
"Mewtwo": {},
"Dragonite": {},
"Snorlax": {},
"// Mew evolves to Mewtwo": {},
"Mew": {},
"Arcanine": {},
"Vaporeon": {},
"Gyarados": {},
"Exeggutor": {},
"Muk": {},
"Weezing": {},
"Flareon": {}

}
}
Output when issue occurred
no error - bot is running but just not sniping
Steps to Reproduce

Other Information
OS: ubuntu 14.04 LTS
Branch:  DEV
Git Commit:
Python Version:  2.7.6

the data provided by the server you have set in config is not valid, not a bot problem, it is a problem of your map source... seems like it is not local either - tried reaching it and it times out

its local and i can reach it just fine here. im using the local IP of the box because i haven't set localhost up on it

Got it to work
the "address": "http://<IP_ADDRESS>:5000/",
should not be ended with a /
the correct is
"address": "http://<IP_ADDRESS>:5000",


[DO_NOT_MERGE] Try to reproduce StateStoreSuite.maintenance failure

Test build #54825 has started for PR 12145 at commit 1aa9d57.

Test build #54840 has started for PR 12145 at commit bad6e28.

Test build #54840 has finished for PR 12145 at commit bad6e28.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/54840/
Test PASSed.

Test build #54825 has finished for PR 12145 at commit 1aa9d57.

This patch fails from timeout after a configured wait of `250m`.
This patch merges cleanly.
This patch adds no public classes.

Jenkins retest this please

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/54825/
Test FAILed.

Test build #54844 has started for PR 12145 at commit bad6e28.

Test build #54844 has finished for PR 12145 at commit bad6e28.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/54844/
Test PASSed.

Jenkins retest this please

Test build #54846 has started for PR 12145 at commit bad6e28.

Test build #54847 has started for PR 12145 at commit 73e34e7.

Test build #54847 has finished for PR 12145 at commit 73e34e7.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/54847/
Test PASSed.

Test build #54848 has started for PR 12145 at commit 6f69c84.

Test build #54848 has finished for PR 12145 at commit 6f69c84.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/54848/
Test PASSed.

Test build #54846 has finished for PR 12145 at commit bad6e28.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/54846/
Test PASSed.

Merged build finished. Test PASSed.


Add missing OpenAPI response declarations

Several endpoints were missing the necessary annotations to include their HTTP 2xx
response payloads in the generated OpenAPI schema. Add the missing annotations and
add a test that scans all the controller methods and ensures they either
explicitly declare a 2xx response or don't declare any responses at all (in which
case OpenAPI includes HTTP 200 by default).

Current dependencies on/for this PR:

main

PR #859   👈

This comment was auto-generated by Graphite.


fix(dev): Prevent stripping query params from CSS in HMR

Description
During HMR, this change prevents query params from being stripped when
reloading the CSS.
Additional context
None
What is the purpose of this pull request?

[x] Bug fix
[ ] New Feature
[ ] Documentation update
[ ] Other

Before submitting the PR, please make sure you do the following

[x] Read the Contributing Guidelines.
[x] Read the Pull Request Guidelines and follow the Commit Convention.
[x] Check that there isn't already a PR that solves the problem the same way to avoid creating a duplicate.
[x] Provide a description in this PR that addresses what the PR is solving, or reference the issue that it solves (e.g. fixes #123).
[x] Ideally, include relevant tests that fail without this PR but pass with it.

Looks like a couple of other tests need updating, fixing now.

This is ready for review.

Looks good @matthewp. I pushed a change as it was easier to avoid going through the GitHub suggest dance in this one. I updated the test so we don't need to add a new client message vite:afterUpdate:css. If this event is important to you, it would be good to create a new feature request or PR to discuss.

@patak-dev nope, I only created that event for the test. Your way works too, thanks!


Update Add-PnPNavigationNode.md

As per issue https://github.com/pnp/powershell/issues/901, verified in v 1.7.0 that it works, and added now the information to the document.
Although, I think that it would be better to remove URL parameter for labels altogether.

Docs Build status updates of commit 34ddfe1:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

sharepoint/sharepoint-ps/sharepoint-pnp/Add-PnPNavigationNode.md
:white_check_mark:Succeeded
View (sharepoint-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

@get-itips

Hello @wonderplayer
how are you?
Even though the PnP documentation articles are on this repository, I need to kindly redirect you to open PnP cmdlets issues/pull requests here, in that repository, where a member of the PnP community will be able to correct the article/review your PR with your feedback.
The documentation for PnP cmdlets is then synced from that repo into this one.
Thank you!

Oh, sorry. I actually thought that I'm updating stuff there 😄


Assignment-Logical model design

What changes are you trying to make? (e.g. Adding or removing code, refactoring existing code, adding reports)
Added answers and image files of ERD for a small book store.
What did you learn from the changes you have made?
Learned how to create an ERD
Was there another approach you were thinking about making? If so, what approach(es) were you thinking of?
no
Were there any challenges? If so, what issue(s) did you face? How did you overcome it?
yes. googled and used GPT for clarifying doubts.
How were these changes tested?
by drawing them in lucidchart and testing in VS code
A reference to a related issue in your repository (if applicable)
Checklist

[x] I can confirm that my changes are working as intended

Assignment marked. Your all HW 1,2,3,4,5,6 are missing.

Hey Sidra Bushra,
Sorry for missing the HW. Will complete them and update you.
On Fri, Oct 4, 2024 at 9:01 PM Sidra Bushra @.***>
wrote:

Assignment marked. Your all HW 1,2,3,4,5,6 are missing.
—
Reply to this email directly, view it on GitHub
https://github.com/jeffey97/sql/pull/1#issuecomment-2394839386, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/A2KRG5JKVJKMN7LX442R66TZZ4255AVCNFSM6AAAAABPIZEJ76VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGOJUHAZTSMZYGY
.
You are receiving this because you authored the thread.Message ID:
@.***>


Android HomeScreen Shortcut issue with Flutter

Hi,
As you know, Flutter provides a mechanism for Dart to communicate with Native. You can call native code through dart, or you can call Dart code in native.
There is such a situation, I do not know if you have encountered: Add a desktop shortcut on the Android platform, click the desktop shortcut to call the code logic of dart, if the application is already running, there is no problem at this time. But if you kill the application and then click on the desktop shortcut, it will not work in this case. (From the log, when I click the shortcut, class of native Application has called to create, but not with flutter engine)
How can I implement native and dart communication in this situation?
Thanks,
Andy

the code logic of dart, if the application is already running, there is no problem at this time. But if you kill the application and then click on the desktop shortcut, it will not work in this case.

Can you give us a reproduction case?

the code logic of dart, if the application is already running, there is no problem at this time. But if you kill the application and then click on the desktop shortcut, it will not work in this case.

Can you give us a reproduction case?

Start a flutter application.
Add a shortcut from Android native, when clicking the shortcut, it will call dart logic in flutter.
However, if you kill the application and clicking the shortcut again, flutter application will not startup. You need write all the logic again from native to handle.


fix: RSA removed, according to #164

signing RSA algs removed, since we MUST support EC, according to https://github.com/italia/eudi-wallet-it-docs/issues/164

I suggest to add, as a note, the support of this alghoritms from Android 6+ and iOS 13.0+, like mentioned in the following official pages, that we can add also as bibliography, in particular for Android info here and for iOS here.

I suggest to add, as a note, the support of this alghoritms from Android 6+ and iOS 13.0+, like mentioned in the following official pages, that we can add also as bibliography, in particular for Android info here and for iOS here.

So from the refrence that you provided it seems that on the mobile devices, there is no support for the brainpool curve.

I suggest to add, as a note, the support of this alghoritms from Android 6+ and iOS 13.0+, like mentioned in the following official pages, that we can add also as bibliography, in particular for Android info here and for iOS here.

So from the refrence that you provided it seems that on the mobile devices, there is no support for the brainpool curve.

You're right, we need external library to manage brainpool curve on mobile. In Android are implemented with BouncyCastle and in iOS with Swift extension wrapper.


Change black arrows to white

White arrows had been added to UI style guide

@IlyaVi Yes. I got the arrows from the UI Kit in the guide.


textDocument/implementation on a protocol requirement returns locations at which conformance to that protocol is stated instead of their implementations

Consider doing an implementation request at 1⃣ in the following.
protocol MyProto {
  func 1⃣myFunc()
}

class ClassA: MyProto {
  func 2⃣myFunc() {}
}
class ClassB: MyProto {
  func 3⃣myFunc() {}
}
class 4⃣ClassBSubA: ClassB {}
class 5⃣ClassBSubB: ClassB {}

class RetroactiveConformanceClassWithMyFuncInClassDecl {
  func 6⃣myFunc() { }
}
extension 7⃣RetroactiveConformanceClassWithMyFuncInClassDecl: MyProto {}

class RetroactiveConformanceClassWithMyFuncInExtension {}
extension RetroactiveConformanceClassWithMyFuncInExtension: MyProto {
  func 8⃣myFunc() { }
}

Currently, we report 2, 3, 4, 5, 7, 8
We should not be reporting locations 4, 5 and 7 because they don't actually contain myFunc. We should, however, be reporting location 6.

Synced to Apple’s issue tracker as rdar://132935019


cancel animations before updating value for min-shell animation

Why
There seems to be a performance regression after updating Reanimated to 3.11.0. We were able to track this down to performing updates on mode.value while also running a withTiming animation on the same value. For example, if we were to remove this code:
      mode.value = withTiming(v ? 1 : 0, {
        duration: 400,
        easing: Easing.bezier(0.25, 0.1, 0.25, 1),
      })

we would see the smoothness of the scroll be fixed.
How
Any time that we update mode - whether it is to manually update the value or to animate the value - we should cancel an existing animation that for that value.
Test Plan
Build this branch with yarn android --device --variant release and see if the performance is fixed. From my testing on a device, it does appear to be better. There doesn't appear to be any regression in the animation itself either.

(the main fix that did it for me was https://github.com/bluesky-social/social-app/pull/4391)


Improve SA1131 to also consider methods as constants

The following code triggers SA1131 (Use readable conditions) :
// Totally contrived example code :-) Only here to illustrate the current behaviour.
class TestClass
{
    private readonly System.Action myAction;

    public TestClass(bool stuff)
    {
        myAction = stuff ? DoSomething : DoSomethingElse;
    }

    public bool MyCondition => myAction == DoSomething; // SA1131 here!

    private void DoSomething()
    {
    }

    private void DoSomethingElse()
    {
    }
}

The code fix changes the condition like this:
public bool MyCondition => DoSomething == myAction;
DoSomething is in effect a constant in that condition, so I think that SA1131 should not be triggered.
Unusual case, but might be worth considering if the change is easy?
Tested with 1.2.0-beta.507.

But what the... No, it doesn't. But it did. I think. :-)
Let me try to de-simplify this again. I'll be back.

Ah, it triggers if the field is static readonly. The example needs some changes, but this triggers for example:
class TestClass
{
    private static readonly System.Action myAction;

    static TestClass()
    {
        var someMoreElaborateCondition = true;
        myAction = someMoreElaborateCondition ? DoSomething : DoSomethingElse;
    }

    public bool MyCondition => myAction == DoSomething; // SA1131 here!

    private static void DoSomething()
    {
    }

    private static void DoSomethingElse()
    {
    }
}


The content is not splitted accordingly in to columns

Hello,
I'm investigating TLV as a possible solution to quickly parse Tableau Server Logs.
It's seems very useful with handy options existing out-of-the-box.
I've noticed that the column "value" also contains the time info; if I also put the check to display the time column the rows for the column are empty so the data is not splitted accordingly (please see the attached image).

How can I solve this?
Thanks,
Bogdan

Hi Bodgan,
I am glad to hear that you like our little tool
The Tableau Log Viewer is primarily built to view Tableau's custom log format. It is written for example by Tableau Desktop, Hyper and a few other parts of the Tableau stack.
The log file you are opening looks to me like a log file written by Apache or some similar webserver. Unfortunately, we don't fully support loading this file format yet.
Personally, I hope we will add support for other file formats, such as Apache logs. Right now, we are currently not actively working on it. But pull requests are always welcome, of course :)
Cheers,
Adrian

Hi Adrian,
Thank you for your answer, you're right the log file is for httpd.
I will use TLV like so; if this kind of file will be supported in the future it will be appreciated.
Thanks,
Bogdan


[Bug]: Even numbered variables in a graphql query show highlighting issues on multiple dynamic bindings

Is there an existing issue for this?

[X] I have searched the existing issues

Description
Every even numbered variable has a highlighting issue which is visible as red coloured colon :

Steps To Reproduce

Create a Graphql query
Add query a { repository(a: "{{Api2.data}}", b: "{{Api2.data}}", c: "{{Api3.data}}", s: "{{Api5.data}}") } in the body
Observe the instructed error

Public Sample App
No response
Version
Cloud

@Nikhil-Nandagopal @AmanAgarwal041  i try to reproduce this issue i didnt find any error can you please check this once


make TARGET_HTTP_MODE and PROXY_HTTP_MODE configurable from command line

When I tried running this locally with odoh-server, it failed because it sends requests with https and my odoh-server was running with plain http. It'd be nice to be able to set the scheme (http/https) for the target and proxy without having to change commands/common.go and rebuild.

This would be a good thing to add, yeah. @kimbo, do you want to throw up a PR? If not, I'll take it.

I'm good either way, though I may not get to it for a bit. Here are a few thoughts I had:
--proxy and --target could accept the full URL including the scheme (and the path), and then default to https and /dns-query (or whatever the default path is for the given command) when it's ommitted. Personally I'm used to curl's default of http when you don't specify the scheme, but in his case I think https is a fine default. (lmk if you have a different opinion about that)
Examples values/test cases for --proxy or --target flag for odoh command:

localhost:8080 would send query to https://localhost:8080/dns-query
localhost:8080/ would be https://localhost:8080/
http://localhost:8080 would be http://localhost:8080/dns-query
http://localhost:8080/my-custom-path would be unchanged.

The above would apply to all the commands (doh, odoh, and odoconfig-fetch) with whatever default path they each have.
Not sure how the odoh config URL would look in an odoh lookup with a custom url (and their config isn't in a DNS record). Probably would need to be an additional flag for that.
Anyway, there are my thoughts. Not trying to make this change bigger than it has to be, just thinking out loud.

--proxy and --target could accept the full URL including the scheme (and the path), and then default to https and /dns-query (or whatever the default path is for the given command) when it's ommitted. Personally I'm used to curl's default of http when you don't specify the scheme, but in his case I think https is a fine default. (lmk if you have a different opinion about that)

That's a good suggestion. Defaulting to https would be my preference here.

Not sure how the odoh config URL would look in an odoh lookup with a custom url (and their config isn't in a DNS record). Probably would need to be an additional flag for that.

The config is under a .well-known, which is always at the path root (https://tools.ietf.org/html/rfc5785). If a custom URL was uploaded, I'd imagine the code would just construct the .well-known as  + "/.well-known/odohconfigs".

The config is under a .well-known, which is always at the path root (https://tools.ietf.org/html/rfc5785). If a custom URL was uploaded, I'd imagine the code would just construct the .well-known as + "/.well-known/odohconfigs"

That makes sense. I was imagining a scenario where someone had their odoh config stored under a different path, but maybe it'd be better not to support that 🤷♂

That makes sense. I was imagining a scenario where someone had their odoh config stored under a different path, but maybe it'd be better not to support that 🤷♂

We could always add that in the future. But yeah, for now, it seems like YAGNI.


ETA for langdetect for ES 5.3.2

Hi JPrante,
Do we have ETA on langdetect for ES version 5.3.2? Would it be ready by first weeks of may ?
Thank you

I have just released langdetect plugin version <IP_ADDRESS> for Elasticsearch 5.3.2.


GSD Request

--- GSD JSON ---
{
  "vendor_name": "Linux",
  "product_name": "Kernel",
  "product_version": "versions from v3.19 to before v5.15.11",
  "vulnerability_type": "unspecified",
  "affected_component": "unspecified",
  "attack_vector": "unspecified",
  "impact": "unspecified",
  "credit": "",
  "references": [
    "https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=02219b3abca59fca81711bfe7ee78df7abad97ce",
    "https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=42d08e97b196479f593499e887a9ab81446a34b9"
  ],
  "extended_references": [
    {
      "type": "commit",
      "value": "02219b3abca59fca81711bfe7ee78df7abad97ce",
      "note": "introduced"
    },
    {
      "type": "commit",
      "value": "42d08e97b196479f593499e887a9ab81446a34b9",
      "note": "fixed"
    }
  ],
  "reporter": "joshbressers",
  "reporter_id": 1692786,
  "notes": "",
  "description": "mac80211: track only QoS data frames for admission control\n\nThis is an automated ID intended to aid in discovery of potential security vulnerabilities. The actual impact and attack plausibility have not yet been proven.\nThis ID is fixed in Linux Kernel version v5.15.11 by commit 42d08e97b196479f593499e887a9ab81446a34b9, it was introduced in version v3.19 by commit 02219b3abca59fca81711bfe7ee78df7abad97ce. For more details please see the references link."
}
--- GSD JSON ---

This issue has been assigned GSD-2021-1002735


Add fertilizer role to some CHEBI terms

Could you please add has_role fertilizer to:

[ ]  CHEBI:23414 copper (II) sulfate

[ ] CHEBI:63051 diammonium hydrogen phosphate

[ ] CHEBI:64736 magnesium nitrate

[ ] CHEBI:32599 magnesium sulfate

[ ] CHEBI:62982 ammonium dihydrogen phosphate

[ ] CHEBI:63036 potassium dihydrogen phosphate

[ ] CHEBI:35176 zinc sulfate

[ ] CHEBI:31344 calcium oxide

[ ] CHEBI:3312 calcium chloride

[ ] CHEBI:48838 gypsum

[ ] CHEBI:31794 magnesium oxide

[ ] CHEBI:32599 magnesium sulfate

[ ] CHEBI:26836 sulfuric acid

[ ] CHEBI:131526 potassium carbonate

@marieALaporte

Now done. Thanks

Hi, i have removed the 'has role fertilizer' term from sulfuric acid (CHEBI:26836). I have my doubts, sulfuric acid is used in the production of fertilizers but is not a fertilizer itself. If you can provide any evidence where it says that sulfuric acid is actually a fertilizer, let me know and i will add the term. Thanks.

Hi, thank you for your rapid action! Indeed, I was not able to find evidence that sulfuric acid is used as fertilizer. It seems that some people used it on their field, but there is no solid evidence of its fertilizer role.


Make it so that we don't have to re-create external PRs

Currently CI won't run for PRs created by outside contributors, since CI embeds certain credentials which we want to protect.  To merge a PR from an outside contributor, we currently need to literally recreate the PR, which is a very manual and error-prone process and it might discourage outside contributions.

Duplicate of #1935


Traefik api access not logged

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 working access logs showing all requests to endpoints, but requests to the Traefik dashboard are omitted
What did you see instead?
requests to the Traefik dashboard are not logged
What version of Traefik are you using?
Version:      3.0.0-rc3
Codename:     beaufort
Go version:   go1.22.1
Built:        2024-03-13T15:49:38Z
OS/Arch:      linux/arm64
What is your environment & configuration?
accessLog:
  filePath: /var/log/traefik/access.log
  format: json
log:
  # level: INFO
  level: DEBUG

Add more configuration information here.
If applicable, please paste the log output in DEBUG level
No response

Hi @Morriz,
Thanks for your issue.
Since Traefik v3.0 RC internal ressources observability  have changed. Please the migration guide from v2 to v3.
You have to add addInternals to you access log section: cf https://doc.traefik.io/traefik/v3.0/observability/access-logs/#addinternals
I'm closing this issue. Feel free to reopen it if needed

but that flag would flood the logs with pings? Really?

Please reopen this handle access to the dashboard the same as other routes


No matching key. session topic doesn't exist

After connecting a dApp to a wallet pinging fails and calling any rtegistered method throws "No matching key. session topic doesn't exist" in the dapp. Im used https://react-app.walletconnect.com/ to test this.
Steps to reproduce:
-Start example wallet of this package

connect wallet with https://react-app.walletconnect.com/
try any method

This was still working last tuesday (09.05.2023)
might be related to this: https://github.com/WalletConnect/walletconnect-monorepo/issues/2379
Im still posting the issue as  im not sure if its a problem from this package, as https://react-wallet.walletconnect.com/walletconnect works with https://react-app.walletconnect.com/

I did some more testing and it seems that sessions dont get added properly for some reason. From my understanding after web3Wallet.approveSession is called, the session should be added to the web3Wallet. but when i call web3Wallet!.sessions.getAll() i get an empty list, despite just having approved the session. Does the issue lie here, or am i misunderstanding something?

Looking into it now.

I did some more testing and it seems that sessions dont get added properly for some reason. From my understanding after web3Wallet.approveSession is called, the session should be added to the web3Wallet. but when i call web3Wallet!.sessions.getAll() i get an empty list, despite just having approved the session. Does the issue lie here, or am i misunderstanding something?

Your issue is similar to mine: https://github.com/WalletConnect/WalletConnectFlutterV2/issues/79 After I approve a session in the wallet, I can find it in the dApp, but not in the wallet.

I can confirm that tests I had in my repo that worked 5 days ago are now functioning strangely.
Some update must have been pushed through on the relay server. I'm trying to resolve it now.


ARROW-3267: [Python] Create empty table from schema

I wonder whether classmethods would provide better usability, have you considered e.g.:

Schema.as_table()
Table.from_schema(schema)
Table.empty(schema)
?

I was tempted to create Schema.empty_table()

That feels better :)

@kszucs updated the name.

Merging on green


Revert AppArmor skip tests

These tests are to guarantee protection from malicious containers, enforced by AppArmor. These tests should be working and should not be skipped.
I have run these tests on Ubuntu, running them several times, and have not experienced any failures.

Please sign your commits following these rules:
https://github.com/docker/docker/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "fixaa" git@github.com:ewindisch/docker.git somewhere
$ cd somewhere
$ git rebase -i HEAD~2
editor opens
change each 'pick' to 'edit'
save the file and quit
$ git commit --amend -s --no-edit
$ git rebase --continue # and repeat the amend for each commit
$ git push -f

Ammending updates the existing PR. You DO NOT need to open a new one.

Still works locally. It's not clear why AppArmor isn't (seemingly) working on CI.

@ewindisch Is this ok to look at? Can we close if not?

ping @ewindisch @jfrazelle what should we do with this?

Carried in #18046


IImageOperations should contain the image portion of MiscOperations

In order to better accommodate the public docker api docs grouping our API function with how docker documents them is idea. This means that IMiscellaneousOperations contains a few image functions that need to be moved.
CommitContainerChangesAsync
GetImageAsTarballAsync
GetImagesAsTarballAsync
LoadImageFromTarballAsync
LoadImageFromTarballAsync
BuildImageFromDockerfileAsync

How do you feel about this work item? I can implement this if no one else has taken it. Just a few notes:
GetImageAsTarballAsync should be ExportImageAsync
GetImagesAsTarballAsync should be ExportImageAsync (overloaded to taken a list of images)
LoadImageFromTarballAsync is already implemented

I like all of the suggestions. SaveImageAsync and SaveImagesAsync and remove LoadImageFromTarballAsync since we already have LoadImageAsync
Thanks!

This has been fixed.


Improving the UI of navbar in mobile/tablet screen size

In small screen size, when menu icon appears on scrolling there appears a navbar/header which blocks the vision and affect user experience. It should be more transparent or can be removed.

Fixed #107 Pull Request


TestModel<>.getShortestPathPlans() getSimplePathPlans() can recurse indefinitely

Description
Given a circular state machine path through a transient state node in a test model, when getShortestPathPlans() or getSimplePathPlans() is called, a Maximum call stack size exceeded exception is thrown.
Example state machine:
interface ToggleContext {
  count: number;
}

type ToggleEvent = {
  type: "TOGGLE";
};

const machine = createMachine<ToggleContext, ToggleEvent>({
  id: "machine",
  initial: "inactive",
  context: {
    count: 0
  },
  states: {
    inactive: {
      on: { TOGGLE: "toggling" }
    },
    toggling: {
      onEntry: assign((context) => {
        return { count: context.count + 1 };
      }),
      always: [
        {
          target: "inactive",
          cond: (context) => context.count % 2 === 0
        },
        {
          target: "active",
          cond: (context) => context.count % 2 === 1
        }
      ]
    },
    active: {
      on: { TOGGLE: "toggling" }
    }
  }
});

Example test setup:
// Edit your service(s) here
const service = interpret(machine).onTransition((state) => {
  console.log(state.value);
});

const model = createModel(machine).withEvents({
  TOGGLE: {
    exec: async (page) => {
      service.send("TOGGLE");
    }
  }
});

Expected Result
Infinitely recursive paths are not traversed more than once, or the number of recursions is limited by some specified maximum with a sensible default, or there is a specifiable maximum depth with a sensible default.
Actual Result
Maximum call stack size exceeded exception is thrown
Reproduction
https://codesandbox.io/s/trusting-framework-1irmc?file=/src/index.ts
To reproduce the issue, you must un-comment the last two lines. I've commented them out because they trigger infinite recursion which can cause your browser tab to freeze/crash.
Additional context
None.

I'm working on the next version of @xstate/test, and this would look like:
// FUTURE API
const plans = model.getShortestPlans({ traversalLimit: 100 });
// throws "Traversal limit exceeded" if traversal in excess of 100 iterations


MLMultiarray+Image.swift file gives error

Hi Matthijs, I copied this file into my Xcode but line item 220 gives error. Do you have any idea where it comes from?
Xcode Error: Referencing operator function '-' on 'FloatingPoint' requires that 'T' conform to 'FloatingPoint'
Line item 220:
let scaled = (value - min) * T(255) / (max - min)

My mistake. I duplicated some lines

My mistake. I duplicated some lines


Added time zone management module

This module aims to provide methods to correctly calculate the offset between a datetime in a time zone from UTC and the offset between time zones. Other than the functions in the Type Helper codeunit in the base app, this module takes daylight saving time into account when applicable.
Additionally, the module provides indicators on whether a time zone supports daylight saving time or whether a given datetime falls within the daylight saving time period for a given time zone.

@PeterConijn your pull request looks very promising.
It would be really useful in my opinion to provide a procedure that uses the current time zone to calculate the offset.
Additionally, have you thought about adding a wrapper for GetSystemTimeZones to give access to the available time zone ids?
(https://learn.microsoft.com/en-us/previous-versions/windows/embedded/ms912391(v=winembedded.11)?redirectedfrom=MSDN)
If I read the documentation correctly these names must not be always the same.
https://learn.microsoft.com/de-de/dotnet/api/system.timezoneinfo.getsystemtimezones?view=net-7.0
Now waiting for someone of microsoft.

It would be really useful in my opinion to provide a procedure that uses the current time zone to calculate the offset.

That is a good idea. I will look into it and add it to the PR

Additionally, have you thought about adding a wrapper for GetSystemTimeZones to give access to the available time zone ids?

A list of time zones is already handled by the Time Zone Selection module in the System App so it would be superfluous.

@PeterConijn thanks for clarification. I must have missed this System Feature.
I guess you have to include your permissionset in on the system application basic permissonset

@pri-kise, as Peter indicated: Once this PR is merged, please remember to obsolete/move TypeHelper functions to the new module. Old code:
https://github.com/StefanMaron/MSDyn365BC.Code.History/blob/bcbdcfc434259c1bfc3708b55d62752c5b5fca26/BaseApp/Source/Base Application/TypeHelper.Codeunit.al#L328
When you search the Base app for further occurrences of "TimeZone", you will find even more procedures to obsolete, such as https://github.com/StefanMaron/MSDyn365BC.Code.History/blob/1c682b0af15e55f729253392b821e64244712e68/BaseApp/Source/Base Application/TransformationRule.Table.al#L378 (provided such function exists in the new module; I did not check).

The automated builds seem to fail on the object nos (I used the default 50100-50149 range). Maybe a noobie question, but is this something Microsoft will do or is that something expected of me? If the latter, I would like to know which object numbers I can use.

The best way is to wait for someone of microsoft and update the id's for the Tests and the new Objects afterwards.

@JesperSchulz could you please assign someone to this pull request?
To give feedback if this module can be accepted and to provide some new Ids, that can be used for this new module?

I was hoping for other members of the community to review this code too ;-) I know this particular repository has a history of "partner develops, Microsoft reviews", but in the long run that won't scale. This PR is something that many partners over time have requested, and I was hoping to see some involvement from them.
Oh yeah, and it's year-end, which always means "go-time" for all Business Central developers - that may also play into, why we haven't looked at this PR yet ourselves ;-) But I promise to do so today/tomorrow! The ID problem is an easy one to fix - that I can do for sure. Stay tuned!

@JesperSchulz do you have some Ids for the Test Codeunits too?

132979 seems to be available, both for the PS and test :-)

@PeterConijn, if you grant me permissions to your repo, I can push the new IDs :-) Alternatively, you can update them yourself. I've reserved 8720 - 8724 for you.

Thanks @JesperSchulz. I have done both. You have contributor access to my fork and I have updated the object numbers in the app, test, and test permissions.

Great! Now let's see if this builds, and then it's time for code review 🥳

Build failed. Here's the error: AL0185 Codeunit 'Library - Test Initialize' is missing
Please make build pass before we continue code review.

Thanks for the reminder!
Fixed the bug. Build succeeded.

Alright, that should do it. I made a few changes to the module locally (e.g. removed permissions and used inherent permissions instead). This module will ship with v. 22 in April!

Alright, that should do it. I made a few changes to the module locally (e.g. removed permissions and used inherent permissions instead). This module will ship with v. 22 in April!

That's great news! Thank you very much for your collaboration in this module.

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 52216.

Merging the PR, even though it means the changes will get reverted with 21.4, as the module doesn't release until 22.0. But don't worry! The module will be back 🥳


Add jenkins script to update dev tpv config, job conf and local tool conf file

It would save time to have linting and a jenkins job equivalent to what is done for production.

closed by https://github.com/usegalaxy-au/infrastructure/pull/709


chap 11, SELU

When I tried to run the Example of SELU, I got the following error:
Traceback (most recent call last):
  File "/home/work/PycharmProjects/Tensorflow_Tutorials/Hands-on_Machine_Learning/chap11_example_1.py", line 225, in <module>
    hidden1 = tf.layers.dense(X, n_hidden1, activation=selu, name="hidden1")
  File "/home/.local/lib/python3.5/site-packages/tensorflow/python/layers/core.py", line 184, in dense
    return layer.apply(inputs)
  File "/home/.local/lib/python3.5/site-packages/tensorflow/python/keras/engine/base_layer.py", line 828, in apply
    return self.__call__(inputs, *args, **kwargs)
  File "/home/.local/lib/python3.5/site-packages/tensorflow/python/layers/base.py", line 364, in __call__
    outputs = super(Layer, self).__call__(inputs, *args, **kwargs)
  File "/home/.local/lib/python3.5/site-packages/tensorflow/python/keras/engine/base_layer.py", line 769, in __call__
    outputs = self.call(inputs, *args, **kwargs)
  File "/home/.local/lib/python3.5/site-packages/tensorflow/python/keras/layers/core.py", line 951, in call
    return self.activation(outputs)  # pylint: disable=not-callable
  File "/home/work/PycharmProjects/Tensorflow_Tutorials/Hands-on_Machine_Learning/chap11_example_1.py", line 185, in selu
    return scale * elu(z, alpha)
  File "/home/work/PycharmProjects/Tensorflow_Tutorials/Hands-on_Machine_Learning/chap11_example_1.py", line 161, in elu
    return np.where(z < 0, alpha * (np.exp(z) - 1), z)
AttributeError: 'Tensor' object has no attribute 'exp'

And the error results from the following part:
# ELU
def elu(z, alpha=1):
    return np.where(z < 0, alpha * (np.exp(z) - 1), z)

# SELU
def selu(z,
         scale=1.0507009873554804934193349852946,
         alpha=1.6732632423543772848170429916717):
    return scale * elu(z, alpha)

I found that usually the z of elu(z, alpha=1 is np.ndarray, but sometimes z is a <class 'tensorflow.python.framework.ops.Tensor'>
@ageron Could you please give me some hint when you are convenient?

Hi @ylqi007 ,
Thanks for your question. The first error says that the Tensor object has no attribute exp, and since it happens on a line that only uses exp() in np.exp(z), this seems to indicate that in your case, np is a Tensor instead of being the NumPy module.  Perhaps you assigned a Tensor to np at one point? Can you please check that? It should be the NumPy module:
>>> np
<module 'numpy' from '[...]/python3.6/site-packages/numpy/__init__.py'>

If it's not, check your code, it should never modify np.  You can restart the Jupyter module and ensure that np never gets modified after import numpy as np.
Moreover, in the notebook, I first define elu() and selu() in NumPy, in order to plot a graph, but then I redefine selu() to have a TensorFlow version.  Perhaps you went through these cells, and then came back to earlier cells, which might not work since earlier cells expect the NumPy version, not the TensorFlow version.
In short: ensure that np is never modified after ìmport numpy as np`, and make sure you execute the notebook cells in order.
Hope this helps,
Aurélien


a != a

Hi, thanks for this plugin!
I'm confused about the expected behavior of these two lines of code:
https://github.com/roboticsgroup/roboticsgroup_gazebo_plugins/blob/master/src/mimic_joint_plugin.cpp#L171-L172
In which situation a should not be equal to a?

In which situation a should not be equal to a?

This is a quick way of checking for NaN values, see here...

Oh, I see! Thanks

Oh, I see! Thanks

No problem! Closing this issue! Let us know if you have any other question/issue.


feat: add built-in support for concurrency limiting

Inspired by the fork-ts-checker-webpack-plugin-limiter plugin - thanks @vicary. This implementation uses a global pool to limit plugin concurrency. No configuration needed from the user.
✅ Closes: #441

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

@piotr-oles how to use this option in webpack.config.js can you show me any example?

To change pool size (the default one is number of CPUs), you can set ForkTsCheckerWebpackPlugin.issuesPool.size = x and ForkTsCheckerWebpackPlugin.dependenciesPool.size = x. This is shared static variable, you can't set it in plugin constructor. So an example would be:
// webpack.config.js
const ForkTsCheckerWebpackPlugin = require('fork-ts-checker-webpack-plugin');

ForkTsCheckerWebpackPlugin.issuesPool.size = 10;
ForkTsCheckerWebpackPlugin.dependenciesPool.size = 10;

module.exports = {
  // ...
  plugins: [
    new ForkTsCheckerWebpackPlugin()
  ]
}