feat: add fileKey param to uploadFiles method

Describe the feature you'd like to request
Hey! I hope this issue/feature will help to the Uploadthing's DX and UX!
I was thinking that would be great to add the fileKey param to uploadFiles method in order to avoid call 2 methods to do so (uploadFiles and renameFile). In addition, calling two endpoints delays the upload of the files two times of what I was expecting!!
I wanted to raise a PR but it seems that needs an endpoint to achieve this new feature.
Describe the solution you'd like to see
export type FileEsque =
  | (Blob & { name: string })
  | (UndiciFile & { fileKey?: string });

uploadFiles(file: FileEsque);

Additional information
No response
👨👧👦 Contributing

[ ] 🙋♂ Yes, I'd be down to file a PR implementing this feature!

Not sure I understand, do you want to provide your own fileKey instead of letting uploadthing generate one for you?

Yes, it's just to add a new param that lets me provide my own fileKey, this param could be optional, and if this param it's not provided then uploadthing will generate one for me instead. I saw that the S3 API lets you do so, and I think that it helps a looot

Maybe you're confusing the file's name and the fileKey, the property that's actually important is fileKey, with that you can delete your file or add your own id to do some internal business stuff

I was thinking that would be great to add the fileKey param to uploadFiles method in order to avoid call 2 methods to do so (uploadFiles and renameFile)

From this it seemed like you wanted to be able to set the file key so that you can call renameFile() without waiting for uploadFiles() to give you back the fileKey -- perhaps I am misunderstanding though.
I am trying to understand why you would need to set the file key

yes, you're right!
Well, is as simple as the same reasons of why uploadthing has a renameFile method, but instead it will be faster than waiting to upload the files and then renaming them. It doesn’t make sense to me.
I need to rename the key cause of internal stuff in my project, let's say security or organization (by date, by folder or by pattern). In my case I am presetting a fileKey, then if I am receiving a different fileKey I must save two different keys (ut_key and key) in my db, which is a bit redundant. Other option is wait the double of time cause I need to wait for the files upload to rename them.

The reason why we generate the keys is so that we can guarantee that they will be URL safe and will not have any conflicts.
We expect that if you need to do some other form of organization, you will map the keys that we provide to you to something more meaningful for your application in your own DB (eg by using two different keys (ut_key and key) as you suggested). Our file keys are intended to simply be a unique identifier that you can use to retrieve the file.  The file renaming capability we offer (with the renameFile api) is intended to alter the filename that users will see when they download the file.

well, thinking makes sense to me now!
maybe this feature is not necessary at all


Fixed invalid version string

The version string in the library.properties file had a v prefix which caused the Arduino IDE to show an invalid version warning. This goes away if you stick to the x.y.z version format with no other characters.

Thanks. Updated version to 1.0.3 to play nice with the Arduino Library Manager.


Add an option to share an extension

It would be great if the would be an option in the extension view for sharing an extension.
Share
|- Copy Link
|- ... Other options

@daB0bby May I know what is the usecase you are having under sharing the extension?
Is it to share with the team just to other user or recommend the extension for the workspace?

I colleague of mine is using VSCode too. So I wanted to share an extension with him.
Ich don't want to Google for it to find it in the marketplace. It would be much easier to share it directly from within VSCode.

I, too, would love this feature, as we like to share cool extensions with other team members in our Slack channel.

Since we do not have plans to implement this in near future and also there is not many requests for this feature, closing this as out of scope.


Friwi.me Certificate expired

I think the Certificate of the website expired... not able to download from Ubuntu because of ssh exception

#fixed Thanks for reporting :)


Order confirmation: spelling mistake

Reproduce:

Add random item to cart
Start checkout
Provide valid data at all steps
Submit order to arrive on order confirmation page

Issue: The message to the customer reads "Thank you for purchasing, Your order is complete". The sentence contains a spelling mistake ("Your" shouldn't be capitalized) and is missing punctuation. I recommend the following wording which I find more user-friendly: "Thank you for your purchase, your order is being processed."

resolved during work on #189

Works in poster version 2.1.


musikcube: fix rpath for musikcube

Description of changes

I've filed clangen/musikcube#538, since the latest version of musikcube aborts on Darwin when installed with nixpkgs due to an incorrect/unresolved rpath.
At the time of writing I'm uncertain whether this is an upstream issue with musikcube that was introduced with 0.98.0 or something in nix/nixpkgs has changed.
Putting up this draft PR for review to get a conversation going how and where this issue would be resolved best.
Any suggestions for improvement in this PR are welcome.
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

Would you like to add yourself as a maintainer?

Putting up this draft PR for review to get a conversation going how and where this issue would be resolved best.

I'm sorry I have no knowledge of this platform so I can't comment.

Would you like to add yourself as a maintainer?

Oh? Thank you 😄 To help me decide what are the rights and duties of a nxipkg maintainer?

I'm sorry I have no knowledge of this platform so I can't comment.

I'll see what the conversation on the issue in the upstream repo surfaces. It's good to know that at least a workaround, i.e. the changes in this PR, exists now.

I've marked this PR ready for review @aanderse, as I believe the pragmatic path forward is to merge the changes proposed in this PR. After some input from the upstream maintainer and thinking about it a bit more the workaround here seems to solve the issue good enough for the time being.
If I find a better upstream fix for this I'll propose the changes upstream and here as necessary.

Thanks for merging @aanderse 🙏🙂


Use all in every public module to control which names are exported

Question regarding the expectations here. Do you just want to expose all the current files/modules as is or hide in which file they actually reside.
Ex.
from openfeature import api
from openfeature.client import OpenFeatureClient
from openfeature.provider.in_memory_provider import InMemoryFlag, InMemoryProvider

or this
from openfeature import api
from openfeature import OpenFeatureClient
from openfeature.provider import InMemoryFlag, InMemoryProvider

There is room to keep improving the public module hierarchy. The only thing we should avoid is making multiple releases with breaking changes. If we are going to make changes on this then we should bundle all of them together in a single release so that users only have to migrate once.
My current thinking is that we should use __all__ to control visibility within public modules, to avoid re-exporting imports and such; and we should use underscore-prefixed modules for hiding internal structure, either using _internal modules or splitting files into multiple internal prefixed modules that get imported by a single public one.
For a good reference on this check out how the opentelemetry-python SDK does it.
As for your specific question, I don't have a strong preference either way for the OpenFeatureClient import, but the InMemoryProvider one is probably more idiomatic as openfeature.provider.in_memory. It makes sense to separate providers into multiple modules to avoid parsing/loading unused providers.

What I have said is not established convention, just my own curated best practices from years of working with Python. Feel free to suggest alternatives.

One big issue with the current module hierarchy is that the API is commonly imported as:
from openfeature import api

Or worse:
from openfeature.api import set_provider

Neither of these names (api, set_provider) give any indication as to what they belong to, if you find them in the middle of a long file. A more pythonic approach would be to be able to import them like this:
import openfeature

openfeature.set_provider(provider)

Or, if you want:
import openfeature as of

of.set_provider()

Typically I prefer to expose the most important ones directly at the root level of the package and less frequent used ones through the their sub namespace. But I think your suggestion can't be implemented, because openfeature is a namespace package and therefore you can't set __all__ in the __init__.py. For the sub namespaces it is no problem, but the root won't work sadly.

Yeah, I remember now that's the reason we didn't do it before.
I think a good step forward would be to compile a list of suggestions for improving the public package structure and then do these changes all in one go.

Hey @federicobond, can this be closed now?

Yeah, this is done 😄


loading, declaration and execution phases of modules and cyclic references problem ?

This question is related to TypeScript, ES6, SystemJs and ReflectJs specs, but I would like to know your thoughts about it.
So for a function declarations the sequence of code is not important, but for classes it is:
Example with single module
console.info(fnc()); // this is fine
console.info(new Cls()); //TypeError: Cls is not a constructor
function fnc(){
  return "value"
}
class Cls {
  constructor(){
    //...
  }
}

A more complex example when classes located in different modules and have cross references in declaration phase like:
Example with multiple modules and static references
// one.ts
import {Two} from "./two"
class One {
  static two = new One();
}
// two.ts
import {One} from "./one"
class One {
  static two = new One(); //TypeError: One is not a function
}
//main.ts
import {One} from "./one"
import {Two} from "./two"
console.info(One.two)
console.info(Two.one)

so this is throwing error when trying to create One instace, the expectation is to have One declared at the time of exection .
This can be hacked by using some cached getters, which can be somehow used to solve the main problem in runtime(loader) :
// cached.ts
export function Cached(target,key,desc){
    var initializer = desc.get;
    desc.get = function(){
        return Object.defineProperty(this,key,{
            value:initializer()
        })[key];
    };
    return desc;
}
// one.ts
import {Cached} from "./cached";
import {Two} from "./two";
export class One {
    @Cached
    static get two():Two{
        return new Two()
    }
}
// two.ts
import {Cached} from "./cached";
import {One} from "./one";
export class Two {
    @Cached
    static get one():One{
        return new One();
    }
}
//main.ts
import {One} from "./one"
import {Two} from "./two"
console.info(One.two)
console.info(Two.one)

I have prepared some demo to show hack on this problem, you can see bellow
Expected behavior:
Create and instance of Cls and out put it without errors.
Actual behavior:
Throws a TypeError because of undeclared class
Example of cyclic references problem in decoration phase
The other example is annotators cyclic references problem.
function Model(target){}
function Collection(target){}
function Field(target:any,key:any,desc?:any):any{
    var type:any = Reflect.getMetadata("design:type",target,key);
    document.write(`<pre>${target.constructor.name}.${key} : ${type?type.name:"<span style='color: red'>UNDEFINED ?</span>"}</pre>`);
}

@Model
export class User {
    @Field id:String;
    @Field name:String;
    @Field projects:Projects; // Project Is not defined yet
}

@Collection
export class Users extends Array<User> {}

@Model
export class Project {
    @Field id:String;
    @Field name:String;
    @Field author:User;
    @Field contributors:Users; // Users will not be defined
                               // if we will change definition order
}

@Collection
export class Projects extends Array<Project>{}

I think this is IMPORTANT because Dependecy Injection (java guice), ORM's or Serialization frameworks will use this feature deeply, and user code can contain cross referenced types.
So the main reason for this problem is that declaration, decoration and execution phases of classes mixed in output.
// decoration inside execution block and part of class declaration
__decorate([
    Field,
    __metadata('design:type', Projects)
], User.prototype, "projects", void 0);

Potencial hack for this can be generate closure instead of direct type referance in metadata like:
__decorate([
    Field,
    __metadata('design:type', function(){return Projects})
], User.prototype, "projects", void 0);

which can be adjusted after module execution, but I think this is just a hack, and the right way will be finnaly define
specification about Loading > Declaration > Annotation > Decoration and Execution phases of runtime which will support cyclic references.
Any thoughts ?
PS: AND AGAIN THANKS FOR TYPESCRIPT :)

@casser so I'm having a hard time reading through this post and finding the question :smile: - can you clarify what exactly you're asking here?

I just realize that links was broken, sorry :)
Question: Examples above are bugs or expected behaviors ?
new Class(); //TypeError: Cls is not a constructor
class Class {
  constructor(){
    console.info("Created")
  }
}

Expected behavior:
output: Created

Actual behavior: Demo
output: TypeError: Cls is not a constructor

function Field(target,key,desc?){
  console.info(Reflect.getMetadata("design:type",target,key).name)
}
class One {
  @Field field:Two;
}
class Two {
  @Field field:One;
}

Expected behavior:
output:
Two
One

Actual behavior: Demo
output:
undefined
One

Look At this repo for sources

Ah, gotcha. @rbuckton would know a lot better about the decorators. Though, is #4114 related?
As for using a class before its definition, yes, we should be reporting an error. See #21, #4341, and #5207. Apparently there was a fix out for it, but it was never taken in (#4343). @mhegazy what's going on with that?

As for using a class before its definition, yes, we should be reporting an error. See #21, #4341, and #5207. Apparently there was a fix out for it, but it was never taken in (#4343). @mhegazy what's going on with that?

https://github.com/Microsoft/TypeScript/issues/5207 is the correct issue tracking this change

This is a known issue with the --emitDecoratorMetadata switch, and your proposed solution is exactly what we have discussed as the resolution. Regardless of "use before definition" type errors, using --emitDecoratorMetadata would still emit JavaScript that will result in an error in ES6 unless we wrap the metadata values in a function.

Any solution to this? How can I wrap values of metadata in a functions?
It's very blocking for any project with decorators that have circular references.


Instructions in the tensorcomp branch are broken

Hi, I just read your blog about comparing tensor compilers, and I want try the same benchmark on my cards (GTX 1070 Ti and R9 Nano).
I tried following instructions here, but I am getting 'No such option' errors everywhere, something like:
Error: no such option: --results
Error: no such option: --tc
Can you fix the instructions there?

It's very specific about where you put the options, what exact commands are
you running?
On Mon, May 21, 2018, 7:02 AM masahi<EMAIL_ADDRESS>wrote:

Hi, I just read your blog about comparing tensor compilers, and I want try
the same benchmark on my cards (GTX 1070 Ti and R9 Nano).
I tried following instructions here
https://github.com/plaidml/plaidbench/tree/tensorcomp, but I am getting
'No such option' errors everywhere, something like:
Error: no such option: --results
Error: no such option: --tc
Can you fix the instructions there?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/plaidml/plaidbench/issues/49, or mute the thread
https://github.com/notifications/unsubscribe-auth/ARnZ6SlSbSze6ilmUW3eoEMTNGqvnrb5ks5t0sjtgaJpZM4UHCdA
.

They've been updated, please let me know if you have any other issues and we'll be them fixed post-haster

Ah, I forgot to do python setup.py install in the tensorcomp branch. I installed the master branch earlier.
Thanks, it's working now.


Custom provisioning and deprovisioing

Description
Our BTP Manager requires some custom provisioning and deprovisioning capabilities. That means we will not use default reconciliation implementation from module-manager which just creates yamls files out of chart and applies into k8s.
BTP Manager -  external and internal configuration for btp-operator component

Reasons
We want to have full control over how we install btp-operator component.
AC

Provisioning requirements, activity diagram:

[x] Provisioning:

[x] a) use chart values customization which is right now handled by KEB (see):  BTPOperatorPriorityClass and manager.req_cpu_limit: "10m"
[x] b) if secret doesn't exist then do not install btp-operator, adjust status of the BTPOperator CR
[x] c) only one, first BTPOperator CR could be in READY status. All other instances must be set to ERROR state with proper message. All of them must be reconciler from time to time.
[x] d) update documentation

Deprovisioning requirements, activity diagram:

[x] Deprovisioning:

[x] a) When BTPOperator CR is marked to be deleted we should first delete all SBs then SIs and give max 20 minutes to have it gone. If any of those resources are not deleted inn that time, we remove finalizers on those objects (make sure that secrets created by SBs are removed too). Then when no SB, SIs anymore remove every resource created by BTPManager controller.
[x] b) update documentation

Implementation ideas

We can still make use of module-manager implementation where helm templating is in use. That give us a possibility to not apply resource one by one, but just in one chunk.

PRs related to custom provisioning:

replace ManifestReconciler Reconcile method with own Reconcile
add label for every chart resource during provisioning
add Secret dependency check and Reconcile on events originating from operations on the required Secret

Todo:

manage more than one BtpOperator CR
verify chart installation correctness, BtpOperator CR is set to Ready state even when not all chart resources are in a healthy state
tests.

The following PR introduces:

handling many BtpOperators and limiting the operand reconciliation responsibility to the oldest BtpOperator,
check chart resources readiness after provisioning, set Ready state only when all resources are in Ready state.
https://github.com/kyma-project/btp-manager/pull/24

Todo:

handle Ready state,
tests,
docs.

Handle Ready state:
https://github.com/kyma-project/btp-manager/pull/26
Provisioning tests:
https://github.com/kyma-project/btp-manager/pull/27

Provisioning documentation:
https://github.com/kyma-project/btp-manager/pull/35

@ukff please read my comment:
https://github.com/kyma-project/btp-manager/pull/29#issuecomment-1321212807

All tasks defined in the issue are done. We can close it after #29 merges to the dev branch.


Show ownerEmail when personal details aren't available

@danieldoglas, please review when you get the chance 🙇🏾
Details
$ https://github.com/Expensify/Expensify/issues/211211
Tests
Taken from PR that introduced regression:

On NewDot, create a workspace on one account, let's call it the admin account
Create two new accounts, let's call them employee and formerEmployee
From the admin account, invite both the employee and the formerEmployee account to the workspace
From the admin account, remove the formerEmployee account (can also remove the employee account if you want, it shouldn't matter)
Then log into the employee account on OldDot, go to Settings -> Account -> Merge Accounts and try to merge the formerEmployee account by email. If you get an error about personal payments, then comment out this line.
Log into NewDot as the employee account and verify that you can see one archive workspace chat that explains that the formerEmployee account was merged into the current employee account:

(Important step, this is what this PR fixes)
7. Now make sure the title also is working and doesn't just say (archived):

[ ] Verify that no errors appear in the JS console

PR Review Checklist

Contributor (PR Author) Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[ ] I added steps for local testing in the Tests section
[ ] I added steps for Staging and/or Production testing in the QA steps section
[ ] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[ ] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[ ] iOS / native
[ ] Android / native
[ ] iOS / Safari
[ ] Android / Chrome
[x] MacOS / Chrome
[ ] MacOS / Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[ ] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[ ] I verified that comments were added to code that is not self explanatory
[ ] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[ ] I verified any copy / text shown in the product was added in all src/languages/* files
[ ] I verified any copy / text that was added to the app is correct English and approved by marketing by tagging the marketing team on the original GH to get the correct copy.
[ ] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[ ] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] If a new component is created I verified that:

[ ] A similar component doesn't exist in the codebase
[ ] All props are defined accurately and each prop has a /** comment above it */
[ ] Any functional components have the displayName property
[ ] The file is named correctly
[ ] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[ ] The only data being stored in the state is data necessary for rendering and nothing else
[ ] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[ ] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[ ] All JSX used for rendering exists in the render method
[ ] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[x] If a new CSS style is added I verified that:

[ ] A similar style doesn't already exist
[ ] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG)

[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] I have checked off every checkbox in the PR author checklist, including those that don't apply to this PR.

PR Reviewer Checklist
The Contributor+ will copy/paste it into a new comment and complete it after the author checklist is completed

[ ] I have verified the author checklist is complete (all boxes are checked off).
[ ] I verified the correct issue is linked in the ### Fixed Issues section above
[ ] I verified testing steps are clear and they cover the changes made in this PR

[ ] I verified the steps for local testing are in the Tests section
[ ] I verified the steps for Staging and/or Production testing are in the QA steps section
[ ] I verified the steps cover any possible failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[ ] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)

[ ] I checked that screenshots or videos are included for tests on all platforms
[ ] I verified tests pass on all platforms & I tested again on:

[ ] iOS / native
[ ] Android / native
[ ] iOS / Safari
[ ] Android / Chrome
[ ] MacOS / Chrome
[ ] MacOS / Desktop

[ ] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[ ] I verified proper code patterns were followed (see Reviewing the code)

[ ] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick).
[ ] I verified that comments were added to code that is not self explanatory
[ ] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[ ] I verified any copy / text shown in the product was added in all src/languages/* files
[ ] I verified any copy / text that was added to the app is correct English and approved by marketing by tagging the marketing team on the original GH to get the correct copy.
[ ] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[ ] I verified the JSDocs style guidelines (in STYLE.md) were followed

[ ] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[ ] I verified that this PR follows the guidelines as stated in the Review Guidelines
[ ] I verified other components that can be impacted by these changes have been tested, and I retested again (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar have been tested & I retested again)
[ ] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[ ] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[ ] If a new component is created I verified that:

[ ] A similar component doesn't exist in the codebase
[ ] All props are defined accurately and each prop has a /** comment above it */
[ ] Any functional components have the displayName property
[ ] The file is named correctly
[ ] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[ ] The only data being stored in the state is data necessary for rendering and nothing else
[ ] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[ ] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[ ] All JSX used for rendering exists in the render method
[ ] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[ ] If a new CSS style is added I verified that:

[ ] A similar style doesn't already exist
[ ] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG)

[ ] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[ ] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[ ] I have checked off every checkbox in the PR reviewer checklist, including those that don't apply to this PR.

QA Steps
To test this on production, reach out to @TomatoToaster or @trjExpensify when this has been deployed and we can test it. The same steps can be followed, but it requires having some accounts that are already on the Workspace Chat beta and not the Personal Payments beta which requires some setup.

[x] Verify that no errors appear in the JS console

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to staging by @danieldoglas in version: 1.1.90-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

@TomatoToaster or @trjExpensify based on QA steps its PROD PR. Are we ok to check it off?

Yeah, these workspace chats only exist under a beta. I've been trying to test this myself, but I get an infinite spinner using the domain I have under the policyExpenseChat beta.


Extend pusher config

Work in progress for supporting full Pusher config.
We don't want to break former configs for endpoint that were present in object itself without pusherConfig wrapper, so I introduced depreciation for couple of next releases.
[Regarding #8]

review done 👍


CSS blocking hyperlink

Hyperlinks currently only work in bottom region

Run npm install to get router package @victorwu95

fixed implementation


Compressing a Finetuned llama2 model with lora

Thank you for this amazing work. I was wondering if it was possible to run wanda on a llama2 model fine-tuned with lora? When I gave it a try, I got the following error:
AttributeError: 'LlamaForCausalLM' object has no attribute 'layers'

I think you need to find how to access the layers from LlamaForCausalLM, you need .model.layers instead.
https://github.com/huggingface/transformers/blob/main/src/transformers/models/llama/modeling_llama.py#L1131


docs: refresh cmd docs

What type of PR is this?
/kind documentation

What this PR does / why we need it:
refresh cmd docs
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?

Additional documentation e.g., design docs, usage docs, etc.:

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

21 of 21 (100.0%)  changed or added relevant lines in 4 files are covered.
11 unchanged lines in 3 files lost coverage.
Overall coverage decreased (-0.1%) to 74.948%

Files with Coverage Reduction
New Missed Lines
%

pkg/engine/operation/watch.go
1
85.93%

pkg/engine/operation/parser/parser.go
3
79.27%

pkg/cmd/apply/options.go
7
71.83%

Totals

Change from base Build<PHONE_NUMBER>:
-0.1%

Covered Lines:
6486

Relevant Lines:
8654

💛  - Coveralls


I want to see the file before compilation

Hi! I want to browse the c language files in the implementation. If you don't mind the license, could you please upload the original files to create the compiled .a files under AFBR-S50-API/AFBR-S50/Lib/?
I would like to check the detailed implementation of the measurement algorithm and change the contents of the printed error message.

Dear Waarrk,
thanks for your inquiry. Actually, our API is only available with the core functionalities of the pre-compiled library. The main reason for that is safety-related. The pre-compiled library preserves the ability to use our sensors within laser class l under each possible condition. Another intention of the library is a reduction of complexity for the user. If you want to get further information on your application please get in contact with<EMAIL_ADDRESS>Thank you


extrap_start_time

Variable manipulation support for TF backend, when using extrap_start_time.
Also, when using extrap_start_time, self.states in build() should be extended.

I incorporated similar changes in cc76248. Thanks for the suggestions!


react-cookie cookies.set does not set object

I've been unable to set an object to cookies using below:
  const profileString = JSON.stringify(userProfile);
  cookies.set('userProfile', profileString, { path: '/' });

It works for plain strings, and reading your docs I understood it should work by stringifying the object.
Are there any limits concerning depth of the object?

library not have limit by depth or length serialized object. It's just JSON.stringify at
https://github.com/reactivestack/cookies/blob/v4.0.3/packages/universal-cookie/src/Cookies.ts#L73
and JSON.parse at
https://github.com/reactivestack/cookies/blob/v4.0.3/packages/universal-cookie/src/utils.ts#L44
also https://www.npmjs.com/package/cookie(dependency) does not restrict cookie length
but different browser have different restriction for cookies
In my Google Crome 79.0.3945.79 max cookie size is 4096 bytes, you can check it on http://browsercookielimits.squawky.net/
try for yourself something like:
document.cookie = `example_1=${'example_value_1'.repeat(10)}`
console.log(document.cookie.indexOf('example_1') !== -1 ? 'successfully set' : 'failed set')
// in my main browser it prints "successfully set"
document.cookie = `example_2=${'example_value_2'.repeat(10000)}`
console.log(document.cookie.indexOf('example_2') !== -1 ? 'successfully set' : 'failed set')
// in my main browser it prints "failed set"

Thanks @mogafk. I tried with a plain object and it worked. So the issue seems to be that the object I intended to save in cookies is too big. I will save it in indexeddb instead.


⚠ Samsung has degraded performance

In 01a0b66, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2493 ms

Resolved: Samsung performance has improved in 14dfc7f.


Ensure .gitignore when setup a new boilerplate

Close #323

PR updated

Current coverage is 73.28%

Merging #331 into next will increase coverage by +5.95% as of 839eec1

@@             next    #331   diff @@
=====================================
  Files          42      45     +3
  Stmts         548     670   +122
  Branches        0       0
  Methods
=====================================
+ Hit           369     491   +122
  Partial         0       0
  Missed        179     179

Review entire Coverage Diff as of 839eec1

Powered by Codecov. Updated on successful CI builds.

Going to add commit + release as 0.9.1


Duplicate Filesystem Abstraction

Just a suggestion, there is a great library here https://github.com/spf13/afero that implement elegantly a lot of storage and utils/fs.
You can just use afero fs interface and run from memory, filesystem, and more.

I know afero from spf13 is a very good implementation, actually this should be part (some day) of the standard library (at least the interface)
Our implementation is very similar, but for some methods works different, for example Create method, creates the folders if needed (since in git the folder not exists). Sound like a waste of time, but we prefer minimize the number of dependencies.
A PR is more than welcome adding a kind of AferoFS, to allow the users use file systems already written for afero.
Best regards


Make search suggestions have the same colors as the menu on mobile

 Make the colours the same for search suggestions as for the menu in mobile view


Speed up the face matching process

Hi @justadudewhohacks
As you wrote before. The face match process is computing the every euclidean distance between the descriptor of your photo and the descriptors from your database.
The computation time will increase with the database growing. Is there any methods to speed up the face matching process ?
Thanks

Just tried tensorflow KNN classifier for face matching process, but do not know which is better. Anyone have ideas? Thanks
A part of the training code is as follows
const classifier = knnClassifier.create()

await Promise.all(
      this.props.train.face.map(async face => {
        const image = await faceapi.fetchImage(face)
        const descriptors = await faceapi.computeFaceDescriptor(image)
        const tensor = tf.tensor(descriptors)
        classifier.addExample(tensor, 'Name')
      })
)

A part of the recognizing code is as follows
const results = await faceapi
      .detectAllFaces(
        image,
        new faceapi.TinyFaceDetectorOptions({
          inputSize: environment.tinyInputSize,
          scoreThreshold: environment.tinyThreshold
        })
      )
      .withFaceLandmarks(true)
      .withFaceDescriptors()

 if (results) {
      faceapi.matchDimensions(canvas, image)
      const resizedResults = faceapi.resizeResults(results, image)
      resizedResults.forEach(async ({ detection, descriptor }) => {
        const tensor = tf.tensor(descriptor)
        const predict = await this.classifier.predictClass(tensor, 1)
        const label = predict.label.toString()
        const options = { label }
        const drawBox = new faceapi.draw.DrawBox(detection.box, options)
        drawBox.draw(canvas)
     })
}

Full code can be found here and here.


Conditionally add a tile feature?

Hi
Is there any way to conditionally add a tile feature, based on a template?
I'm using an auto-entities card to render some tiles but I only want to add custom tile features on some of the tiles (I only want to add/show the tile feature if the entity has a specific label, if this helps).
Many thanks

Some of the examples show the use of :host { display: none; } but that adds some extra padding to the card compared to when a tile has no padding:

Left is with features but display: none, right is without features.

A recent change in 2024.8 to how card features was laid out added a gap between features (defined in CSS by --feature-padding). Unfortunately this ends up creating two gaps when a custom feature is hidden. There isn’t a good way to fix it with this project as we can only apply styles on the custom feature level and lower. The custom feature is nested within the shadow-root of hui-card-feature, which is what is actually in the the card features container flexbox and is causing the gap. Setting display: none on this element does remove the gaps, but I'm not sure if there's a good way to apply styles to it outside of dev tools. It may be possible with card-mod.

As you suggest, using card_mod to set display: none; on the hui-card-features element does work, but as with all card_mod hacks, the content still appears on a page refresh and then disappears when card_mod kicks in, which is a bit frustrating.
Would it be possible to implement an option on the service-call entries to enable/disable, that accepts a template instead?
Something like:
features:
  - type: custom:service-call
    entries:
      - type: button
        enabled: "{{template here that returns true or false}}"

Thanks

It wouldn't be possible. As I said above we only have control over the custom feature level and lower, which is nested inside the shadow-root of hui-card-feature. This would be a feature request for the Home Assistant frontend repo.


Add DuckDB Dialect Support

This pull request adds support for the DuckDB SQL dialect to the SQL Formatter library.
Description:

Extends the supported dialects to include DuckDB.
Updates the documentation (README.md and potentially others) to reflect the addition of DuckDB support.
Includes any necessary tests to ensure proper formatting for DuckDB queries.

Benefits:

Users working with DuckDB can now leverage the SQL Formatter library for consistent and readable SQL code.
Enhances the overall library coverage by including a popular in-memory database.

Testing:

A test suite is included to verify accurate formatting for various DuckDB queries.

Please review the changes and provide feedback.

Thanks for the PR. A few quick questions and thoughts:

What's the relationship between DuckDB and PostgreSQL? I saw in DuckDB docs that it uses PostgreSQL parser. But does it actually support all of the PostgreSQL syntax? Like, does it support all these operators that you have listed?
You probably have listed too many keywords. Best to only include only reserved keywords. Otherwise you'll have common field names like id, name, location, type detected as keywords and converted to uppercase when using keywordCase: "upper".
Your data types list seems to be missing some basic stuff like INT, CHAR, ARRAY.

I'm pretty busy this week... not sure how much time I have to properly review this.

For bonus points, you can update the wiki with information about DuckDB. That will also make it easier for me to review this. Otherwise I'll have to go and figure out all of this about DuckDB by myself.

I was about to submit a PR also, I will make some comment in your code.

https://github.com/PMassicotte/sql-formatter/tree/duckdb-dialect

Thanks for the comments above. The errors from the test suite are now down to five:
DuckDBFormatter
    ✕ supports ARRAY[] literals (2 ms)
    ✕ dataTypeCase option does NOT affect ARRAY[] literal case
    ✕ keywordCase option affects ARRAY[] literal case
    ✕ dataTypeCase option affects ARRAY type case (2 ms)
    ✕ supports array slice operator

@nene - I'll look into filling out the wiki 👍

To fix ARRAY[] tests:

add ARRAY to dataTypes and remove it from keywords and functions lists.

To fix the bar[1:] test:

remove BAR from list of functions

I would guess the builtin BAR() function is rarely used. Well... at least I failed to find documentation of it, because it's super hard to google as Postgres docs contain loads of foo and bar in example code. I think it's better for the formatter to also support the more common use case of bar as a name used in example code.
PS. Make sure to run yarn pretty. (Looks like that currently also changes the .pre-commit-hooks.yaml file... you can let Prettier to reformat that, or leave it as-is. Either way is fine.)

The test suite is passing completely now 🎉
I'll collate some notes for the wiki over the next week. Are there any other steps needed before merging?

Thanks. I don't think there's anything else.

Hello,
We're are working on a project using DuckDB and have a few issues with the standard SQL format (->> replaced with - > > for instance).
Do you know when this PR will be merged?
Thanks

Thanks for interest. This thing has indeed been sitting here for a while now. Will need to dig back into this to see if there's any reasons why it hasn't been merged yet. I think one of the main reasons was that it seemed really-really similar to PostgreSQL.
So a question to you @Zank94, if you just configure SQL Formatter to treat your SQL as PostgreSQL, would that solve your problems (e.g. the ->> operator is also present in PostgreSQL).

I see, thank you for the advice I will give it a try :+1:


Unreal features in unity when?

Hi, when will dynamic occluders and probe spacing volumes hit unity plugin?
Rather than volumes i would think it better to tag the navigation mesh with spacing. So you can have multiple navigation meshes with different spacing options.
Thanks

Hi @AndersMalmgren, good to hear from you! Unfortunately, the quick answer to both of your questions is "not anytime soon."
Dynamic occluders: We learned a lot from the beta we released for the Unreal Engine plugin. In particular, we found that when the dynamic occluders work, they can solve lots of common use cases. But we found that there are equally many common scenarios that break the functionality entirely and produce pretty bad results. We've essentially gone back to the drawing board on this one and are prioritizing researching a robust solution over porting the existing less-than-ideal solution to Unity.
Probe spacing volumes: Semi-similar story with these. We found they work pretty well on flat terrain, but don't do what you'd expect on non-flat terrain. We'd like to take a swing at fixing that before porting them over.

Hi @AndersMalmgren, good to hear from you! Unfortunately, the quick answer to both of your questions is "not anytime soon."
Dynamic occluders: We learned a lot from the beta we released for the Unreal Engine plugin. In particular, we found that when the dynamic occluders work, they can solve lots of common use cases. But we found that there are equally many common scenarios that break the functionality entirely and produce pretty bad results. We've essentially gone back to the drawing board on this one and are prioritizing researching a robust solution over porting the existing less-than-ideal solution to Unity.
Probe spacing volumes: Semi-similar story with these. We found they work pretty well on flat terrain, but don't do what you'd expect on non-flat terrain. We'd like to take a swing at fixing that before porting them over.

Thanks for reply. I understand. Better make it good!
I did try solving the spacing myself by making a navnesh were i spaced myself. Like you said it worked well on flat terrain but didn't work well on terrain with little hills and stuff.

I guess the algoritm need to be smart and inject more probes around these hills and unevnes in terrain

Hey @AndersMalmgren good to hear from you; yeah it needs to be smarter. Current algo just considers each Z plane as an independent thing for spacing things out, but on terrains - which are common - really need to think in terms of a real 3D solution.

Hey @AndersMalmgren good to hear from you; yeah it needs to be smarter. Current algo just considers each Z plane as an independent thing for spacing things out, but on terrains - which are common - really need to think in terms of a real 3D solution.

Hi, nikunjragh
Good to hear from you too, and from @MikeChemi too btw, sorry was a bit informal above.
Reason I contacted you all now is because I'm making a new large level and increasing spacing for entire level isnt really a solution because the occlusion data etc will be bad in tight places.  I raycasted the entire terrain and created little islands of a quad mesh each 10 meters I think it was.  Then I had a detailed navmesh around important places with buiildings etc. But this didnt work for more complex places on terrain for example if the player was occluded by a small hill in the terrain. Would be very tedious to manually cut a navmesh in blender containing all hills etc.
Another Idea I have that I haven't tested is to create some sort of heightmap over the terrain using raycasting. Patches that have alot of variation in height will be kept intact so it will get detailed spacing. Places with less dense changes in height will receive the island approach. Havent had time to implement it though. You guys haven't experimented with a similar approach in your spacing algoritm?

We will try to implement a filter to our acoustics navmesh using something like https://stackoverflow.com/questions/22218037/how-to-find-local-maxima-in-image


Add cli for description

Solve #240 for description

Please tell me that there is any problem in test_edit_description.

can you please tell me where should I add these tests.( from test_model.py
or test_cli.py)
On Tue, Jun 20, 2017 at 2:44 PM, Hugo Osvaldo Barrera <
<EMAIL_ADDRESS>wrote:

@hobarrera commented on this pull request.
In todoman/cli.py
https://github.com/pimutils/todoman/pull/259#discussion_r123062013:

@@ -466,6 +468,8 @@ def move(ctx, list, ids):
@cli.command()
@pass_ctx
@click.argument('lists', nargs=-1, callback=_validate_lists_param)
+@click.option('--description',

This flag was used here
https://github.com/pimutils/todoman/pull/259/commits/09fcd2177af47abaabf3ab67973e0918ad73f38b#diff-d7b704f0d282081b24d6d106d9ea528fL656.
By deleting that bit of code, this flag is now unused and ignored.
My initial comment on that related bit of code was that there were no
tests for it (though the code itself seemed fine). Basically, that would be
a test that uses runner with 'list', '--description'.
You can either remove the flag (now unused), or undelete that code and add
tests for it (I'm very much prefer the latter, of course. 🙂 ).
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/pimutils/todoman/pull/259#discussion_r123062013, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AXknkfySDlmU5I95-bV5tf0Pn-kKJeFwks5sGBMZgaJpZM4N9dDs
.

--
Anubha Agrawal,
Sophomore year B.tech,
Computer Science Department
Zakir Hussain College of Engineering & Technology, AMU

can you please tell me that is any other change is required??

Looks like this comment hasn't been addressed yet.


How do i make a union query?

Is there any support for this or do i have to just concat the result of two querys?

Please use the forum for generic usage questions, that are not bugs/feature-requests.
Regarding union, I assume you are looking something equivalent to SQL's OR operator. If so, see lf.op.or() Lovefield operator.


jest-editor-support version bug

Bug
I believe https://github.com/facebook/jest/blob/master/packages/jest-editor-support/src/Settings.js#L63 should be parseInt(version, 10) ?  Right now this is returning NaN on windows.  Version is coming back as
(space)18.0.0(EOL) and it's failing to parse currently.
Also the line comment above I believe is now incorrect.  The jest-editor-support appears to have a dependency on jest@18 because of the change to outputFile argument rename.
@cpojer @orta I'm willing to fix, just want someone to verify what I'm saying actually makes sense :)

Yeah, we should fix this. I'd actually prefer to use jest --version for this and then match against the output of that.

Happy to have that use jest --version - it will need to be split up though, the scope of this class was to grab this config JSON:

Getting the version from it was just a convenient freebie. I initially wondered about using node_modules to find it but we can't guarantee to know where it will be.

We can also just simply expose something that gives you the config object for a given root, right?

yeah, I'm not sure on the history of --debug  but a jest config is basically what I was hoping for here

Are you willing to add this? We actually need this for Nuclide too; cc @kentaromiura
One of the problems here is that some people may actually run Jest with --config=… so I'm unsure what the best way is to detect whether we are in a Jest project or not. Do you have any suggestions for that?

Am I ATM? nah, I'd like to spend some time focusing on Danger over the holidays

The original parsing bug is fixed in #2360
Hrm, I think it should be fine if jest config also takes --config into account, that's how people would run their specs

We cannot really make jest config work because right now, in the jest repo, that will run all tests matching config. Not sure how we'd turn Jest into a command system. Maybe jest --show-config?

This'll be in Jest 20.


BackBufferCopy regression in 3.2.4-rc1

Godot version:

3.2.4-rc1
OS/device including version:

Windows 10
GLES3
NVidia GT 1030 2GB
Issue description:

I expected the icon to look the same in 3.2.3 and 3.2.4-rc1. However, in 3.2.4-rc1 the BackBufferCopy doesn't seem to work, as the PaletteSwap shader uses the original SCREEN_TEXTURE.
Steps to reproduce:
Have any node with a screen space shader, then below it have a BackBufferCopy node, then below that have any other screen space shader. In 3.2.3 they will combine as expected, but in 3.2.4-rc1 the second shader will replace the first.
Minimal reproduction project:

BackBufferCopy_bug.zip

This may be a regression from the unified GLES3/GLES2 batching.
cc @lawnjelly

It is batching regression, it looks pixellated when you switch off batching.
I had a look at the diagnose log:
canvas_begin FRAME 553
items
	joined_item 1 refs
			batch R 0-1 [0 - 153] {255 255 255 255 }
	joined_item 1 refs
			batch R 0-1 [0 - 0] {255 255 255 255 }
	joined_item 1 refs
			batch R 0-1 [0 - 0] {255 255 255 255 }
	joined_item 1 refs
canvas_end
	items reordered: 1

It looks caused by the item reordering, if you set rendering->batching->parameters->item_reordering_lookahead to 0, it no longer occurs. I'll see if I can figure out a way to prevent it happening.

Fixed by #45843.


Using multiple databases when creating an html document with interactive sql examples?

Hey thanks for making this awesome javascript widget to create interactive sql examples.
My question whether it is possible to use multiple databases when creating an html document with interactive sql examples? That is, would the following work,

select
  rank() over w as "rank",
  name, department, salary
from employees
window w as (order by salary desc)
order by "rank", id;

select * from employees limit 10;

<sqlime-db name="employees" path="./employees.sql"></sqlime-db>
<sqlime-examples db="employees" selector="pre.employees" editable></sqlime-examples>

<sqlime-db name="demo" path="./demo.db"></sqlime-db>
<sqlime-examples db="demo" selector="pre.demo" editable></sqlime-examples>

Note: both employees.sql and demo.db are taken from this repo

Sure! You can connect as many databases as you want - just create a separate sqlime-db for each of them.

Thanks for your prompt reply! But I am having multiple issues with creating a static html file with interactive SQL examples. So I would really appreciate your help. I am posting these problems here in this thread.  But if it helps you to track the issues separately I can post separate ones.
[Problem One] Failed to load database from URL
Consider the following reprex,
<!DOCTYPE html>

	<meta charset="utf-8">
	<meta name="viewport" content="width=device-width, initial-scale=1">
	<title>sqlime</title>


Make the program print the schema for each of the tables

Make the program print the schema (contained in schema) for each of the table objects when the user enters select from <table name>.schema or something similiar.

done.


Location POST to 'options.url' fails on iOS 11 but works on Android

Your Environment

Plugin version: "^3.0.0-alpha.12"
Platform: iOS
OS version: 11.2.5
Device manufacturer and model: iPhone 6 - A1549
Cordova version (cordova -v): 7.1.0
Cordova platform version (cordova platform ls):

Installed platforms:
  android 6.4.0
  ios 4.5.4

Plugin configuration options:

{
        desiredAccuracy: window.BackgroundGeolocation.HIGH_ACCURACY,
        stationaryRadius: 50,
        distanceFilter: 50,
        notificationTitle: "Background tracking",
        notificationText: "enabled",
        debug: true,
        startOnBoot: true,
        stopOnTerminate: false,
        locationProvider: window.BackgroundGeolocation.ACTIVITY_PROVIDER,
        interval: 10000,
        fastestInterval: 5000,
        activitiesInterval: 10000,
        url:
          "https://qa2web36/Coyote.AnonymousClient.WebApi/api/v1/locationtracking/123456",
        httpHeaders: {
          "Content-Type": "application/json"
        },
        postTemplate: {
          lat: "@latitude",
          lon: "@longitude"
        }
      }

Link to your project:

Closed source
Context
Same configuration appears to work fine on Android.
On iOS, location changes appear to occur, but the POST itself to the configured url option fails.
Documentation and examples show this url to be "http" not "https".  I tried both, and both fail.
The app uses a self-signed SSL cert but is able to do its own posts to that same server just fine,
so that tells me self-signed SSL is configured fine and should not be an issue.
Expected Behavior
I would expect the POST to succeed, and not to see these lines in Xcode:
2018-01-30 20:27:03.673078-0600 CoyoteGO[516:117801] BgGeo postJSON failed: error: (null)
2018-01-30 20:27:03:672 CoyoteGO[516:117802] BgGeo postJSON failed: error: (null)

It would also be nice to have some more info logged here to hint what the problem is.
Actual Behavior
POST fails.  I don't see a trace of it on my backend server, and see these failure messages in XCode trace:
2018-01-30 20:27:03.673078-0600 CoyoteGO[516:117801] BgGeo postJSON failed: error: (null)
2018-01-30 20:27:03:672 CoyoteGO[516:117802] BgGeo postJSON failed: error: (null)

Possible Fix
Not sure.  I do suspect it may be something around self-signed certificates, but it's only a hunch.
Steps to Reproduce
I don't have anything public setup, but I'm curious if anyone tried iOS 11 with "http" or "https" and self-signed certificates with any luck.
Context
We are investigating if we can use this plugin at Coyote for our CoyoteGO app to track truck drivers and their loads.
Debug logs
When the plugin fails to post the location update, I see the following trace in Xcode:
2018-01-30 20:27:00.796258-0600 CoyoteGO[516:117498] BgGeo postJSON failed: error: (null)
2018-01-30 20:27:00:774 CoyoteGO[516:117498] BgGeo postJSON failed: error: (null)
2018-01-30 20:27:00.825578-0600 CoyoteGO[516:117498] BgGeo postJSON failed: error: (null)
2018-01-30 20:27:00:808 CoyoteGO[516:117802] BgGeo postJSON failed: error: (null)
2018-01-30 20:27:03:336 CoyoteGO[516:115062] ActivityLocationProvider activityTypeChanged: UNKNOWN
2018-01-30 20:27:03.339815-0600 CoyoteGO[516:115062] CDVBackgroundGeolocation #onActivityChanged
2018-01-30 20:27:03.344823-0600 CoyoteGO[516:115062] CDVBackgroundGeolocation #onActivityChanged
2018-01-30 20:27:03:339 CoyoteGO[516:115062] BgGeo #onActivityChanged Activity: confidence=2 type=UNKNOWN
2018-01-30 20:27:03:342 CoyoteGO[516:115062] ActivityLocationProvider activityTypeChanged: STILL
2018-01-30 20:27:03:344 CoyoteGO[516:115062] BgGeo #onActivityChanged Activity: confidence=0 type=STILL
2018-01-30 20:27:03.435299-0600 CoyoteGO[516:115062] CDVBackgroundGeolocation #onLocationChanged
2018-01-30 20:27:03:433 CoyoteGO[516:115062] BgGeo #onLocationChanged Location: id=(null) time=2018-01-31 02:27:03 +0000 lat=42.1021139715259 lon=-88.12584232547145 accu=30 aaccu=4 speed=0.1700000017881393 bear=251.015625 alt=266.3565406799316
2018-01-30 20:27:03.506715-0600 CoyoteGO[516:115062] Task ended
2018-01-30 20:27:03.508276-0600 CoyoteGO[516:115062] Task ended
2018-01-30 20:27:03.673078-0600 CoyoteGO[516:117801] BgGeo postJSON failed: error: (null)
2018-01-30 20:27:03:672 CoyoteGO[516:117802] BgGeo postJSON failed: error: (null)
2018-01-30 20:27:06.982440-0600 CoyoteGO[516:115062] CDVBackgroundGeolocation paused
2018-01-30 20:27:06.983151-0600 CoyoteGO[516:117770] BgGeo #switchMode 0
2018-01-30 20:27:06:982 CoyoteGO[516:115062] BgGeo #switchMode 0
2018-01-30 20:27:08.276824-0600 CoyoteGO[516:115062] CDVBackgroundGeolocation #onActivityChanged
2018-01-30 20:27:08:237 CoyoteGO[516:115062] ActivityLocationProvider activityTypeChanged: UNKNOWN
2018-01-30 20:27:08.282817-0600 CoyoteGO[516:115062] CDVBackgroundGeolocation #onLocationChanged
2018-01-30 20:27:08.286482-0600 CoyoteGO[516:115062] CDVBackgroundGeolocation #onActivityChanged
2018-01-30 20:27:08.344480-0600 CoyoteGO[516:115062] THREAD WARNING: ['BackgroundGeolocation'] took '12.073975' ms. Plugin should use a background thread.
2018-01-30 20:27:08:276 CoyoteGO[516:115062] BgGeo #onActivityChanged Activity: confidence=2 type=UNKNOWN
2018-01-30 20:27:08:282 CoyoteGO[516:115062] BgGeo #onLocationChanged Location: id=(null) time=2018-01-31 02:27:06 +0000 lat=42.10226421714025 lon=-88.12590594411652 accu=10 aaccu=4 speed=0 bear=251.015625 alt=267.1140480041504
2018-01-30 20:27:08.381613-0600 CoyoteGO[516:115062] Task ended
2018-01-30 20:27:08.383269-0600 CoyoteGO[516:115062] Task ended
2018-01-30 20:27:08:285 CoyoteGO[516:115062] ActivityLocationProvider activityTypeChanged: STILL
2018-01-30 20:27:08:286 CoyoteGO[516:115062] BgGeo #onActivityChanged Activity: confidence=0 type=STILL
2018-01-30 20:27:08:398 CoyoteGO[516:117498] BgGeo postJSON failed: error: (null)
2018-01-30 20:27:08.535964-0600 CoyoteGO[516:117802] BgGeo postJSON failed: error: (null)
2018-01-30 20:27:08.544441-0600 CoyoteGO[516:115062] CDVBackgroundGeolocation #onLocationChanged
2018-01-30 20:27:08:487 CoyoteGO[516:115062] BgGeo #onLocationChanged Location: id=(null) time=2018-01-31 02:27:08 +0000

Try https://stackoverflow.com/questions/31254725/transport-security-has-blocked-a-cleartext-http

Is this still an issue?

I have not gotten around to testing it any further, but still plan on doing so in the next 2-3 weeks.  If you can, leave it open and I'll follow up.


Update lifesim_lib.py

Health/Fitness traits

By "Any changes to your Health are more intense", do you mean all positive and negative changes, or just the positive changes?

With muscle memory for fitness, I think only the positive boost, while the negative should come from another trait


chore: check and build if outputDir does not exist before preview

Description
If the build operation has not been performed before the preview, the preview server cannot return the corresponding file
Learn in #10564 , throwing an Error where there was none thrown before, and it breaks plugins using configurePreviewServer to customize.
So if preview, outputDir does not exist. Then it will try to execute vite build  to build, if there is an error during the build process, it will remind the user as follows

What is the purpose of this pull request?

[ ] Bug fix
[ ] New Feature
[ ] Documentation update
[x] 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 PR Title 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).
[ ] Ideally, include relevant tests that fail without this PR but pass with it.

I agree with @bluwy here. This is better implemented in frameworks using Vite. Thanks a lot for checking how we could improve Vite here @chenfan0, but we better leave this one for downstream.


Download images

This PR aims to add the option to download post images (#210). Currently still WIP.

This should now work on Android and iOS and I feel like this is ready now :)

Cool! I’ll take a look 🦆
On Fri, 19 Apr 2019 at 00:16, Sophie Tauchert<EMAIL_ADDRESS>wrote:

This should now work on Android and iOS and I feel like this is ready now
:)
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/OpenbookOrg/openbook-app/pull/253#issuecomment-484707849,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADBFXZJOIC3FQEPKLUPPZCDPRDXNLANCNFSM4HFRQKAQ
.

Hmm getting a Unable to copy temp file. Error: The file “CFNetworkDownload_pMIYDN.tmp” doesn’t exist. when tapping download

URLSession:downloadTask:didFinishDownloadingToURL:
Unable to copy temp file. Error: The file “CFNetworkDownload_1OIRNh.tmp” doesn’t exist.
execute query: UPDATE task SET status=4, progress=-1 WHERE task_id="social.openbook.app.download.background.1555627649.266563.4"
open DB successfully
Query was executed successfully. Affected rows = 1
URLSession:task:didCompleteWithError:
HTTP status code: 200


Outbox: A database per service is the recommended approach ?

I'm extending the poc with Outbox Pattern, now i'm trying to use a centralized database.
There are 2 applications, each application only knows their own messages.
When the App1 reads the Outbox table it tries to publish a message in a unknow namespace for the App1, and the following exception is thown:
[17:21:17 ERR] Error occurred processing the outbox.
System.IO.FileNotFoundException: Could not load file or assembly 'Super.App2.Api, Culture=neutral, PublicKeyToken=null'. O sistema não pode encontrar o arquivo especificado.
File name: 'Super.App2.Api, Culture=neutral, PublicKeyToken=null'
   at System.RuntimeTypeHandle.GetTypeByName(String name, Boolean throwOnError, Boolean ignoreCase, StackCrawlMarkHandle stackMark, ObjectHandleOnStack assemblyLoadContext, ObjectHandleOnStack type, ObjectHandleOnStack keepalive)
   at System.RuntimeTypeHandle.GetTypeByName(String name, Boolean throwOnError, Boolean ignoreCase, StackCrawlMark& stackMark, AssemblyLoadContext assemblyLoadContext)
   at System.RuntimeType.GetType(String typeName, Boolean throwOnError, Boolean ignoreCase, StackCrawlMark& stackMark)
   at System.Type.GetType(String typeName, Boolean throwOnError)
   at Silverback.Util.TypesCache.ResolveType(String typeName, Boolean throwOnError)
   at System.Collections.Concurrent.ConcurrentDictionary`2.GetOrAdd(TKey key, Func`2 valueFactory)
   at Silverback.Util.TypesCache.GetType(String typeName, Boolean throwOnError)
   at Silverback.Messaging.Outbound.TransactionalOutbox.Repositories.DbOutboxReader.<>c.<ReadAsync>b__2_1(OutboxMessage message)
   at System.Linq.Enumerable.SelectListIterator`2.ToList()
   at System.Linq.Enumerable.ToList[TSource](IEnumerable`1 source)
   at Silverback.Messaging.Outbound.TransactionalOutbox.Repositories.DbOutboxReader.ReadAsync(Int32 count)
   at Silverback.Messaging.Outbound.TransactionalOutbox.OutboxWorker.ProcessQueueAsync(IServiceProvider serviceProvider, CancellationToken stoppingToken)
   at Silverback.Messaging.Outbound.TransactionalOutbox.OutboxWorker.ProcessQueueAsync(CancellationToken stoppingToken)

Is that the expected bahaviour ?

It's the expected behavior but you can create a table per each service in the same database.
In the DbContext of each service you can map the DbSet<OutboxMessage> to a different table.
In the upcoming big major release, there will be more explicit and intuitive ways to do this.

@BEagle1984, I'm gonna try that! Thank you for you huge support!


Show list of datasets the user has access to

See #223
On a new page under menu item Datasets > Available datasets; see #229
List all datasets in the database (first column), with in the second column Access: yes/no ("Access" being the column header), and in the third column the present number of signs in the dataset

@ocrasborn First version is now live: /datasets/available/
Datasets can be added via de Django admin. When editing a (new) dataset, user permissions per dataset object can be edited by clicking 'Object permissions' in the top right corner.

Nice!

Not sure how group and user permissions interact here (and I cannot set group permissions without also listing a user, is that a bug?): do dataset user permissions now take precedence over (overall?) group permissions?
After saving 'Object permissions', I see only question marks for missing images; should be green checkmarks and red crosses?

@ocrasborn

Not sure how group and user permissions interact here (and I cannot set group permissions without also listing a user, is that a bug?): do dataset user permissions now take precedence over (overall?) group permissions?

Group and user permissions interact as follows (as far as I can see after some testing): if either one has a permission set, the user has the permission. There is no explicit 'no permission', just 'permission' or blank, which is interpreted as 'no permissions'. Another way to look at it: if a user has a permission set in either the user-permissions or group-permissions, the user is allowed to do the action. Otherwise, the action is blocked.

After saving 'Object permissions', I see only question marks for missing images; should be green checkmarks and red crosses?

This is solved now. (@Woseseltops, it was necessary to copy the img directory of the Guardian package. Same as the admin-img directory. Symbolic link didn't work. Any thoughts?)

The file issue should be solvable with the collectstatic command, whicj
should copy all the static files from apps (including admin) into the
folder specified in settings.
13.9.2017 12.30 "Micha Hulsbosch"<EMAIL_ADDRESS>kirjoitti:
@ocrasborn https://github.com/ocrasborn
Not sure how group and user permissions interact here (and I cannot set
group permissions without also listing a user, is that a bug?): do dataset
user permissions now take precedence over (overall?) group permissions?
Group and user permissions interact as follows (as far as I can see after
some testing): if either one has a permission set, the user has the
permission. There is no explicit 'no permission', just 'permission' or
blank, which is interpreted as 'no permissions'. Another way to look at it:
if a user has a permission set in either the user-permissions or
group-permissions, the user is allowed to do the action. Otherwise, the
action is blocked.
After saving 'Object permissions', I see only question marks for missing
images; should be green checkmarks and red crosses?
This is solved now. (@Woseseltops https://github.com/woseseltops, it was
necessary to copy the img directory of the Guardian package. Same as the
admin-img directory. Symbolic link didn't work. Any thoughts?)
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/Signbank/NGT-signbank/issues/231#issuecomment-329112475,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ANkkYkmZf3xSJCkhIBQqBMIIexwcNkeuks5sh6DSgaJpZM4KngfP
.

Forgot to mention, I get an exception when doing collectstatic (at least on my local instance):
django.core.exceptions.ImproperlyConfigured: You're using the staticfiles app without having set the STATIC_ROOT setting to a filesystem path.

Hence the copying by hand.

@vanlummelhuizen That directory needs to be specified in the settings. It is the directory where the (static) files will be copied to, so that they can be served by the web server.
Hope this helps


Getting 500s from Azure.AI.OpenAI.OpenAIClient.GetEmbeddingsAsync

Describe the bug
I'm trying to chat with the bot locally, however, this doesn't go anywhere and in web API logs I'm seeing a lot of errors that look like below. How do I debug this?
fail: SemanticKernel.Service.CopilotChat.Skills.ChatSkills.ChatSkill[0]
Cannot search collection f375dd31-e341-4ab0-a2c6-dc5087cb3aa6-LongTermMemory
Microsoft.SemanticKernel.AI.AIException: Service error: The service failed to process the request, HTTP status:500
---> Azure.RequestFailedException: Internal server error
Status: 500 (Internal Server Error)
  Content:
  { "statusCode": 500, "message": "Internal server error", "activityId": "19ccf22c-8e3d-4a93-8eb9-028cef52fd3a" }

  Headers:
  x-ms-client-request-id: 29beecdc-e4bc-4929-b5be-28594a1db6e1
  apim-request-id: REDACTED
  Strict-Transport-Security: REDACTED
  X-Content-Type-Options: REDACTED
  x-ms-region: REDACTED
  Date: Tue, 08 Aug 2023 00:13:25 GMT
  Content-Length: 111
  Content-Type: application/json

     at Azure.Core.HttpPipelineExtensions.ProcessMessageAsync(HttpPipeline pipeline, HttpMessage message, RequestContext requestContext, CancellationToken cancellationToken)
     at Azure.AI.OpenAI.OpenAIClient.GetEmbeddingsAsync(String deploymentOrModelName, EmbeddingsOptions embeddingsOptions, CancellationToken cancellationToken)
     at Microsoft.SemanticKernel.Connectors.AI.OpenAI.AzureSdk.ClientBase.RunRequestAsync[T](Func`1 request)
     --- End of inner exception stack trace ---
     at Microsoft.SemanticKernel.Connectors.AI.OpenAI.AzureSdk.ClientBase.RunRequestAsync[T](Func`1 request)
     at Microsoft.SemanticKernel.Connectors.AI.OpenAI.AzureSdk.ClientBase.InternalGetEmbeddingsAsync(IList`1 data, CancellationToken cancellationToken)
     at Microsoft.SemanticKernel.AI.Embeddings.EmbeddingGenerationExtensions.GenerateEmbeddingAsync[TValue,TEmbedding](IEmbeddingGeneration`2 generator, TValue value, CancellationToken cancellationToken)
     at Microsoft.SemanticKernel.Memory.SemanticTextMemory.SearchAsync(String collection, String query, Int32 limit, Double minRelevanceScore, Boolean withEmbeddings, CancellationToken cancellationToken)+MoveNext()

or
fail: SemanticKernel.Service.CopilotChat.Skills.ChatSkills.ChatSkill[0]
Cannot search collection f375dd31-e341-4ab0-a2c6-dc5087cb3aa6-WorkingMemory
Microsoft.SemanticKernel.AI.AIException: Service error: The service failed to process the request, HTTP status:500
---> Azure.RequestFailedException: Internal server error
Status: 500 (Internal Server Error)
  Content:
  { "statusCode": 500, "message": "Internal server error", "activityId": "95f34a4a-c52c-4121-a0e5-d58ab0d6a6e3" }

  Headers:
  x-ms-client-request-id: 8647cf4a-e191-4fa7-8480-b58c37977d0b
  apim-request-id: REDACTED
  Strict-Transport-Security: REDACTED
  X-Content-Type-Options: REDACTED
  x-ms-region: REDACTED
  Date: Tue, 08 Aug 2023 00:12:21 GMT
  Content-Length: 111
  Content-Type: application/json

To Reproduce
Steps to reproduce the behavior:

Completed local devbox setup.
Started the app.
Typing questions and not getting responses reliably.

Expected behavior
Get responses reliably.
Platform

OS: Windows
IDE: VS Code
Language: C#
Source: main branch of the repo.

@wasker Azure OpenAI experienced intermittent outage yesterday.  Can you confirm if you are still experiencing this 500 error?

Yeah, I think it's better today. It's unfortunate that there's no visibility into what's going on, the AOAI doesn't have a way to look up those correlation IDs.

Thank you for the confirmation and feedback.


Policy to check for publicly writeable wiki

Allstar should have a check for whether a repo has configured their wiki to be publicly writeable (and therefore open to defacement, manipulation).

Yes, this would be good, but there is currently not an API for it I believe.

I think you're right - you could script some manual browsing but I think that's going to have to remain unfulfilled unless GitHub exposes an API.

Yeah, some GitHub automation tools resort to browser emulators. Not sure I want to go down that route. We would probably need a robot user to accompany the app, as the app can't auth to the website.


How to select specific gpu devices to be used in sharding, or a specific gpu device when sharding is false?

I would want to select what gpu devices can be used or not.
For example I have 2 gpu devices 0 and 1, and I would want to run only in gpu device 1 and let 0 to handle other operations.

You can use the CUDA_VISIBLE_DEVICES env var to select the GPU you want to run on.
Example:
CUDA_VISIBLE_DEVICES=0 text-generation-launcher --model-id .... # run on GPU-0
CUDA_VISIBLE_DEVICES=0,3 text-generation-launcher --model-id ... --num-shard 2 # shard on GPU-0 and GPU-3

Thank you very much, I missed this env var when looking at the code.

It's not really part of text-generation-inference, it's an env var that is used by all machine learning frameworks.
I will make the documentation clearer.


compiling ahead of time (remove jit)

To spead up simulations, the numba jit compiler is used throughout to speed up functions. Would prefer this was done ahead of time during the build process so users do not have to wait 60 secs every time they import aquacrop.
Numba does now have this functionality so will have to test if it can be used https://numba.readthedocs.io/en/stable/user/pycc.html

I have now created a new branch "nojit" which has removed jit decorators from the code so we can explore other ways of speeding up the code.
I have tried using numba's AOT compling modules (link above) and have has some success compiling a few of the functions in solution.py ahead of time.
As some of the functions read in jitclass objects as arguments, these cannot currently be used with AOT compilation. The jitclass object could potentially be replaced by dictionaries as i think these are compatible with jit AOT.

As a quick fix we could look into compiler caching via numba? I have not pursued this yet but see: https://numba.readthedocs.io/en/stable/developer/caching.html
Or perhaps did anyone looked into this yet?

Nevertheless using JIT for non-JIT of course does not seem ideal.

I have not looked at the cache option yet, ideally we would like to distribute the code already compiled, or potentially compile the code during the setup.py. This second option is less ideal for running in colab as you still have to compile when you pip install aquacrop, however for local use it is better than the current situation.

I'm taking a stab at this (replacing jit with aot and ensuring it doesn't degrade performance). we'll see where I get with it this weekend.

As some of the functions read in jitclass objects as arguments, these cannot currently be used with AOT compilation. The jitclass object could potentially be replaced by dictionaries as i think these are compatible with numba AOT.

I'm going ahead with this approach, it doesn't look like jitclasses will be supported soon, see https://github.com/numba/numba/issues/6648

Documenting the current state of the library in terms of speed:

Task
Speed in seconds

JIT Compile Time
66.75

Tunis Run without JIT
39.67

Tunis Run with JIT
3.26

Tunis Run with AOT
?

@thomasdkelly I'm not sure that python dicts can be used with AOT. These are the supported types https://numba.pydata.org/numba-doc/dev/reference/types.html#numba-types and this doc indicates only these can be used in function signatures: https://numba.pydata.org/numba-doc/dev/user/pycc.html  I think with AOT we are currently limited to ints, floats, and arrays of those types. potentially, the solution is to pass around ordered arrays of parameters instead of jitclasses or dictionaries, though that's probably not super desirable because it makes the code harder to read and track during debugging.
I've created a forum post to get some tips: https://numba.discourse.group/t/whats-the-recommended-way-to-pass-around-large-amounts-of-parameters-that-works-with-aot/1154

Hi Ryan thanks for this info and the pull request. How did you perform the 'without jit' test run? Did you use the nojit branch of the repo?
The tests are a good idea, i think i may add another test that runs the tunis example maybe 10 times to get an average "time per 30 year simulation" metric, as this is the number we want to keep fast as we improve the import/compilation time. (this could also be "time per 1 year simulation").
I have had a good look around the links you provided and through the numba docs/source code and it may be worth (at least for the short-medium term) just trying to pass floats and arrays into the solution.py functions then compiling with numba.cc as you have done in your pull request.
This can be done in two ways:
# old code with jit classess (where crop and soil are jit classes)
def func(crop,profile):
    a=0
    b=0
    for layer in profile.layers:
        a += crop.GCG *profile.dz
        b+=crop.CDC*profile.dzsum
    return a,b

answer = func(crop,profile)

# (1) new code with explicit arguments
def func(crop_CGC,crop_CDC,profile_layers,profile_dz,profile_dzsum):
    a=0
    b=0
    for layer in profile_layers:
        a += crop_GCG *profile_dz
        b+=crop_CDC*profile_dzsum
    return a,b

answer = func(crop.CGC,crop.CDC,profile.layers,profile.dz,profile.dzsum)

# (2) new code with packaged arguments
def func(func_args,profile_layers):
    crop_CGC,crop_CDC,profile_dz,profile_dzsum=func_args
    a=0
    b=0
    for layer in profile_layers:
        a += crop_GCG *profile_dz
        b+=crop_CDC*profile_dzsum
    return a,b

func_args = np.array([crop.CGC,crop.CDC,profile.dz,profile.dzsum])
answer = func(func_args,profile.layers)

This isnt ideal as the code looks messy and could lead to some mistakes with so many functions arguments but it would lead to a simple AOT compilation as everything is ints, floats and arrays. If numba ever incorperates jitclassess or dicts (which isnt beyond possibility since they work with the jit compiler) we can change it back.
The other solution is to use cython which would probably be too big a headache implement at this stage when we have other features we would rather spend time on. Though it isnt something to rule out forever as it does allow you to pre compile code that includes custom classes.

Let me know what you think. Solution 1 is how the code used to look (before this github repo) in early development when i couldnt get jitclasses to work. Its probably a couple days work and just involves:

copying sections of code into a text editor
ctrl+H to replace crop. with crop_ (and similarly for other classes)
adjust the function arguments
adjust function calls (luckily most are just called once in timestep.py)
add the cc.export decorator

#33


back button on store gets "pressed" if another button is actually pressed

visually pretty annoying honestly

maybe fixed?

fixed (all by me)

.


bestitAmazonCron::amazonCall() returns unsupposed value and leads to an exception being thrown

bestitAmazonCron::amazonCall() is called by oxView::executeFunction()  (which is called \OxidEsales\Eshop\Core\Controller\BaseController since version 6.0) which expects a new controller string as return value.
Passing true or false leads to a dispatch exception because there is no class named true/false.
You can trigger that exception when you click on one of the expert links in the oxid eshop backend (select an order which was payed with amazon pay and go to the amazon pay tab).

The unit test could look at the assigned view variables instead of checking the return value and and integration test would have covered that exception case.
Due to missing contribution guidelines (e.g. CLA required?) I did not provide a pull request but I think it is trivial to fix for you.
Side note: This issue is also present in module (v2.5.1) for OXID eShop 4.10.

Thanks for the report, will be fixed with the next version.


New package: SuperLU_Base v0.1.0

Registering package: SuperLU_Base
Repository: https://github.com/Wimmerer/SuperLU_Base.jl
Created by: @Wimmerer
Version: v0.1.0
Commit: bb4fc937006c28731d3bfee7631ec6d9fb311b2a
Reviewed by: @Wimmerer
Reference: https://github.com/Wimmerer/SuperLU_Base.jl/commit/bb4fc937006c28731d3bfee7631ec6d9fb311b2a#commitcomment-118583878
Description: Common definitions for SuperLU solver libraries.

You cannot have underscores in a package name, but SuperLUBase should be fine

Oops, how did I never notice that 😅 [noblock]


Fix ConstructorLike for maps

Fixes #1393

Reviewer checklist

[ ] Test coverage: stack test --coverage
[ ] Public API documentation: stack haddock

it solves my problem. thanks


Unable to start printspooler in powershell

This is regarding the class on 12-21-2019. I am unable to start the process printspooler, but I followed every step that was told. I am sharing the screenshots of the error that I got. Please clarify the mistake that I did.

Hello SrivatsaSrikakolapu,
Can you execute the command without the -force parameter? Before doing it, please ensure the 'print spooler' service is stopped.
Thanks.

Hi anupam,
If you see the screenshot, the spooler's service status is already "stopped". I have tried it without force ,it was not working and so I tried it with force later on. Even then its not working.

Hi SrivatsaSrikakolapu,
To isolate the problem, can you create a new process, with a different name and see if it works.
Thanks.

Hello anupam,
I tried that too, but of no use.

hi
please try to fix manually if its works then look at DSC


Added ControllerIntegrationTest from pdd-backend

Added ControllerIntegrationTest from pdd-backend and split up the bootstrap of the DatabaseTest from the internals, which were moved to a superclass called DatabaseBackedTest

#2028  👈
main

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

Join @nickgraz and the rest of your teammates on  Graphite

Oh, and as for open vs. protected, they're related but somewhat independent.
open means you can override it in a subclass, and protected means it is visible in subclasses but not in unrelated classes. You can have open functions/properties that are public rather than protected, and you can have protected functions/properties that subclasses aren't allowed to override but are still able to access.


There is no source code

we also need a mac port, badly

check the releases or gb page.

well too bad new version is out lol


docker-compose.yml

I made your great tool work on a https://github.com/NginxProxyManager/nginx-proxy-manager driven virtual server. Maybe you'd like to add some of the following to your project:

Copy or clone this repo to the server
Modify Dockerfile to run in server mode. Basically I changed the last line to ENTRYPOINT ["./bore", "server", "--min-port=8080", "--max-port=8080"]
Add a docker-compose.yml and build and start it like docker compose up -d

# docker-compose.yml

version: '3'

services:
  tunnel:
    build: .
    restart: unless-stopped
    ports:
      - '7835:7835'

networks:
  default:
    external: true
    name: proxy

The networks line are specific to the proxy.
The port is required to get the protocol working.
In the proxy I configured like this:

Now with something like bore local 8080 --to tunnel.example.com I can fire it up on the client. HTTPS also works fine if configured in the Nginx Proxy Manager.

Instead of rebuild the image you guys can try to use the command settings, it work perfect for me
version: '3' services:     tunnel:         command: 'server --min-port 3333 --max-port 4444'         image: ekzhang/bore         ports:             - '7835:7835' networks:     default:         external: true         name: proxy

Awesome, this way it is even simpler to use. Thanks!
Just the this line:
command: 'server --min-port 3333 --max-port 4444'

needs to change to something like this:
command: 'server --min-port 8080 --max-port 8080'

in order to get a predictable port for the proxy, that will than be mapped to the public HTTPS port 443.

I am confused a little bit with the network part. What do I need to have for the network?

The networks part is only if you use it with https://github.com/NginxProxyManager/nginx-proxy-manager
See this for details: https://nginxproxymanager.com/advanced-config/#best-practice-use-a-docker-network

https://github.com/ricardoapaes/bore-server
I implemented this solution using nginx, then take a look.


backend.coupling_map.make_symmetric() requires account reload to undo

Information

qiskit-ibm-provider version: latest
Python version:
Operating system:

What is the current behavior?
backend = provider.get_backend('ibm_sherbrooke')
print(backend.coupling_map.is_symmetric)
backend.coupling_map.make_symmetric()
print(backend.coupling_map.is_symmetric)
backend = provider.get_backend('ibm_sherbrooke')
print(backend.coupling_map.is_symmetric)

Shows that backend.coupling_map.make_symmetric() modifies the coupling_map of the backend in the provider in-place, and cannot be undone without reloading the account.
Steps to reproduce the problem
Run above
What is the expected behavior?
The coupling_map for the instance is modified, but I should be able to make a new instance that does not have the same changes to it.
Suggested solutions
Make a copy of the coupling map (and perhaps other items) when grabbing a backend from the provider.

This could be considered an issue with terra. The behavior of backend.coupling_map is inherited from the abstract base class (https://github.com/Qiskit/qiskit-terra/blob/main/qiskit/providers/backend.py#L387-L390) which automatically generates the object  by calling backend.target.build_coupling_map() which builds a coupling map from the Target object. Because depending on the size of the device this repeated cost can be expensive because it has to iterate over all the instructions reported (gate + qubits) on a target the Target caches it's output https://github.com/Qiskit/qiskit-terra/blob/main/qiskit/transpiler/target.py#L911-L920 The provider can override this but any backend will exhibit this behavior.
That being said I'm not sure expecting inplace mutation of a coupling map attribute on a single backend instance would be sticky like that (it'd be the same as mutating a list attribute on an object). However, I think the crux of the issue here is that the provider is caching the backend instances. Because in the OP I would expect initializing a new backend with get_backend() a second time would be a fresh instance and wouldn't be reusing the Target between instances.

I believe this has been resolved, closing


conditional render only if interaction required

Using this library I noticed that if we set
<Turnstile
   ...
   options={{
     appearance: 'interaction-only',
    }}
/>

That the widget immediately vanishes on success.
Widget is invisible if no-interaction is required
Widget becomes visible if interaction required
Widget remains visible after interaction

Hello @sgoodluck, I don't fully understand the issue or request here.
According to the docs, that's how it works when appearance is set to interaction-only; the widget will remain invisible until it requires manual interaction.

Yes, that's intended.
Please see this codesandbox where I'm rendering two widgets; one is created by @marsidev/react-turnstile and the other is a vanilla widget (no libraries). Both have the same behavior.

Roger -- will go upstream to cloudflare as that seems like a bug. Thank you for your work on this package!


[ExpressionParser] Clean up SwiftREPL::CreateInstance

This is a mostly-NFC change which makes SwiftREPL::CreateInstance easier
to read with early returns and references.
The only functional change is a bugfix for the call to
GetScratchTypeSystemForLanguage. If that call fails, we will now
correctly tear down the REPL process and the event handler thread.

@swift-ci test

@swift-ci test

@swift-ci test

@swift-ci test

@swift-ci test

Vedant, feel free to merge this after https://github.com/apple/swift-lldb/pull/340 goes in.
I'll start CI testing in the meanwhile.

@swift-ci please test

I'll make a small update to this PR and kick off testing again, now that we have coverage from https://github.com/apple/swift-lldb/pull/340.

@swift-ci test

@swift-ci test


Refactor markdown attention render

Remove some deadcode
Use 2-word name for CSS class names
Remove "gt-*" rules for sanitizer

The UI doesn't change

(I will propose some following improvements and tests after this gets merged)

Switching tabs between main and this branch:

I think it looks worse. We lose some padding/margins and the icons are out of place.

Switching tabs between main and this branch:

Hmm, it is just a alignment trick, TBH the old gt-vm (vertical-align: middle) doesn't really align for most cases, and the best solution to align svg + text is "flex".
For this case, I think using the "margin" trick could make it good enough 68dc134a61397b7de78f06ed98d962ab4b389dac


num_plates_ent_tank function in ent_tank_dict_test.py not working

The error I get when I do shift+enter on the function:

I'm not sure why this issue is coming up. It should be trivial, though.

@anthonyverghese I think you forgot run from aide_design.play import * so that u was undefined. It's working on my computer