File differs from already included file only in casing: correct casing but relative path

Error TS1149: File name 'C:/Project/frontend/scripts/State.ts' differs from already included file name '../frontend/scripts/State.ts' only in casing.

I've triple checked the casing in our references and the actual files have the correct casing as well. As far as I can tell, this is solely because the relative path uses incorrect casing, or perhaps it's just because of the relative path itself?
It compiles just fine on Mac and Linux, but throws this error on Windows.

This looks like a bug, but we'll need some way to reproduce it. Do you have a zip file or repo or description of the files we could use?

As it turns out, the issue was with a third party module, tsify. Our project was using 1.0.1, and forceConsistentCasingInFileNames wasn't supported until 4.0.0. Closed.

I had same issue with one of my component import. I component directory name was historical with same "h" I was trying to import it with capital "H". It should be "
import { GraphDataComponent } from "./historical/graph-data/graph-data.component";" instead of "import { GraphDataComponent } from "./Historical/graph-data/graph-data.component" in app.moudule.ts.

Thanks @aaybhangu!

Hi, I am still facing this issue with - error TS1149: File name 'src/models/headers/userRole.ts' differs from already included file name 'src/models/headers/UserRole.ts' only in casing.
I have tried multiple ways to fix it , but no luck . I am using windows.

I am also facing the same issue.
For now, "solved" by setting:
"forceConsistentCasingInFileNames": false,
in tsconfig.json

I stumbled across this exact same error recently.
After doing some googling I found this. The last reply caught my eye. So I simply closed the folder is was working in in Visual Studio Code and re-opened it. After everything loaded up, no errors and I didn't have to mess with the tsconfig.json file.
hope this helps

I am getting this bug after renaming a file from the  camelCase to uppercase all first letters

If you are using vsocde, you can press 'command' + 'p' and type 'typescript restart ts server'

In my case, I used to create a file named Logger but changed to logger later. One file which imported this always shows this error message but actually in the terminal or other files, won't show this message.
So I use VSCode Reload Window feature to reload the VSCode, and the error message disappears. So I think maybe there is a cache somewhere in VSCode for this error and didn't clear? You can have a try if you face this problem after making sure the file name is correct.

I have the same issue, and just discovered that for some reason, in some ts file it is called using a capitalize letter, then I renamed it for lowercase and it works.
You can try also removing this model ts file and creating again with angular-CLI.

Changing case results in exception on updateOpen command. Note how file is opened with different case and closed with original casing in same command.
Info 69   [10:30:18.128] request:
    {"seq":5,"type":"request","command":"updateOpen","arguments":{"changedFiles":[],"closedFiles":["c:/temp/est/Logger.ts"],"openFiles":[{"file":"c:/temp/est/logger.ts","fileContent":"export class logger {\r\n    \r\n}","scriptKindName":"TS","projectRootPath":"c:\\temp\\est"}]}}
Err 70    [10:30:18.153] Exception on executing command {"seq":5,"type":"request","command":"updateOpen","arguments":{"changedFiles":[],"closedFiles":["c:/temp/est/Logger.ts"],"openFiles":[{"file":"c:/temp/est/logger.ts","fileContent":"export class logger {\r\n    \r\n}","scriptKindName":"TS","projectRootPath":"c:\\temp\\est"}]}}:

    Debug Failure. False expression: Script should not exist and not be open already

    Error: Debug Failure. False expression: Script should not exist and not be open already
        at ProjectService.applyChangesInOpenFiles (c:\Typescript\built\local\tsserver.js:138090:34)
        at Session.handlers.ts.createMapFromTemplate._a.(anonymous function) (c:\Typescript\built\local\tsserver.js:138972:46)
        at c:\Typescript\built\local\tsserver.js:140630:88
        at IOSession.Session.executeWithRequestId (c:\Typescript\built\local\tsserver.js:140621:28)
        at IOSession.Session.executeCommand (c:\Typescript\built\local\tsserver.js:140630:33)
        at IOSession.Session.onMessage (c:\Typescript\built\local\tsserver.js:140653:35)
        at Interface.<anonymous> (c:\Typescript\built\local\tsserver.js:141968:27)
        at Interface.emit (events.js:182:13)
        at Interface._onLine (readline.js:290:10)
        at Interface._normalWrite (readline.js:433:12)
        at Socket.ondata (readline.js:149:10)
        at Socket.emit (events.js:182:13)
        at addChunk (_stream_readable.js:283:12)
        at readableAddChunk (_stream_readable.js:264:11)
        at Socket.Readable.push (_stream_readable.js:219:10)
        at Pipe.onStreamRead [as onread] (internal/stream_base_commons.js:94:17)

This issue seems to be more involved than just checking root file names. When reusing program there is another test case that fails which is simple program reuse test:
it("forceConsistentCasingInFileNames works when renaming file with different casing", () => {
            const loggerFile: File = {
                path: `${projectRoot}/logger.ts`,
                content: `export class logger { }`
            };
            const anotherFile: File = {
                path: `${projectRoot}/another.ts`,
                content: `import { logger } from "./logger"; new logger();`
            };
            const tsconfig: File = {
                path: `${projectRoot}/tsconfig.json`,
                content: JSON.stringify({
                    compilerOptions: { forceConsistentCasingInFileNames: true }
                })
            };

            const host = createWatchedSystem([loggerFile, anotherFile, tsconfig, libFile, tsconfig]);
            createWatchOfConfigFile(tsconfig.path, host);
            checkOutputErrorsInitial(host, emptyArray);
            host.writeFile(anotherFile.path, anotherFile.content.replace("./logger", "./Logger"));
            host.runQueuedTimeoutCallbacks();
            checkOutputErrorsIncremental(host, [
                createCompilerDiagnostic(Diagnostics.File_name_0_differs_from_already_included_file_name_1_only_in_casing, loggerFile.path, `${projectRoot}/Logger.ts`),
            ]); // Currently the errors are not reported in watch mode but will be reported if program is created from scratch.
        });

Yep, I'm having this same issue and none of the previously-stated fixes/workarounds do anything to help.
In my case, the message is:
error TS1149: File name '/mnt/c/Users/<username>/Documents/adobe-scripts/InDesign/Create Downloadable
(2020a)/Illustrator/2015.3/index.d.ts' differs from already included file name '/mnt/c/Users/<username>/Documents/adobe-scripts/InDesign/Create Downloadable (2020a)/illustrator/2015.3/index.d.ts' only in casing.

@SturmB

Illustrator

and

illustrator

I had a similar problem, but I can't reproduce it anymore. It seems like a VSCode bug: a file with uppercase name is displayed as a lowercase name in some cases, including the source control menu.

I've noticed that, sometimes, someone might change a folder's name from 'foo' to 'Foo' and when you do a git pull it either doesn't update the folder name, or someone forgot to push the change. In that case all your code references are correct, nothing in your code is wrong, and often times the files in that folder can be found just fine; it'll just make TS freak out.
So find the folder that's the root cause. Change the name of the folder.

Git has a configuration setting that tells it whether to be case sensitive or insensitive: core.ignorecase. To tell Git to be case-senstive, simply set this setting to false

See more:
https://stackoverflow.com/questions/17683458/how-do-i-commit-case-sensitive-only-filename-changes-in-git/17688308#17688308

what did it for me is undoing all renames, shutting down the server, renaming all the files I want to rename and restart the server. Must be some sort of caching issue.

Reloading window in VSCode fixed the issue. I changed a React component to camelCase and came across the issue. It does seem to be some sort of caching issue in VSCode.

I just came across this; my componentA.tsx imported ../store/someStore but this import resulted in this error, suggesting the filename was SomeStore.ts (even though the filesystem and VSCode show a someStore.ts at that location.

File name '/mypath/store/someStore.ts' differs from already included file name '/mypath/store/SomeStore.ts' only in casing. ts(1149)

Turns out some other file (../store/index.ts) had an incorrect import (./SomeStore) but did not raise any error. After fixing that import and restarting VSCode, componentA.tsx no longer raises this error.
Reverting the fix to index.ts and restarting VSCode results in a spurious error on componentA.tsx again.

One thing I did to help my situation was to stop using WSL. I went back to my original custom setup which leverages Cygwin and I haven't had this issue since.

I am also facing the same issue.
For now, "solved" by setting:
"forceConsistentCasingInFileNames": false,
in tsconfig.json

Thanks, it helps

After doing some googling I found this. The last reply caught my eye. So I simply closed the folder is was working in in Visual Studio Code and re-opened it. After everything loaded up, no errors and I didn't have to mess with the tsconfig.json file.

This really worked

I had the issue, I fixed it by setting "forceConsistentCasingInFileNames": false
in tsconfig.json then to true again.

I fix it only reboot VSCode.

I have similar problem in WebStorm after changing file names, I'm quite sure it's some IDE cache.
If clearing cache/rebooting IDE don't help, remove folder, clone repo again and do clean install.

I've found this on a mac:
Scenario

file named ComponentA.ts
componentB.ts has import ComponentA from './ComponentA';
componentC.ts also has import ComponentA from './ComponentA';
change ComponentA.ts file name to componentA.ts
update componentC.ts to import ComponentA from './componentA;'
forget to update the import for componentB.ts so it still has the old capitalized import ComponentA from './ComponentA;'

Error (roughly):
 File name '/componentA.ts' differs from already included file name  '/ComponentA.ts'  only in casing. ts(1149)

Reason:
Case insensitivity on Mac is resolving the remnant import import ComponentA from './componentA;' in componentB.ts first, then 'caching/registering' this import path.
Then the correct import in componentC.ts is erroring even though it is correct.
How to Remedy

find all imports of componentA.ts / ComponentA.ts
check case sensitivity and correct.

In my case, the erroring file componentC.ts was actually correct and I had to fix componentB.ts, even though the message was indicating an issue with componentC.ts.

I had to rename my file and gulp build. Then, I renamed back to the old original name and it builded fine.

Because of weird cache that doesn't cover case of changing name of vue file and thus complaining Vetur, I had to reload VS Code (Ctrl + Shirt + P -> Reload Window) as @uniquexiaobai advised.

Restarting the Typescript-Server will suffice. VSCode restart is not necessary to my knowledge.
On OSX: CMD + Shift + P followed by Restart TypeScript Server.

With webstorm You need to invalide and clear the cache :
file > invalidate Caches/ restart

The shortest and best solution I found is.
Just remove that word from that causing casing problems.
e.g.
ListsDrawerOfContent and ListsDrawerofContent
I removed of from the file name. And the compilers compiled it correctly.
Then later I renamed by my desired case ListsDrawerOfContent.
Worked like a charm.
Also, keep in mind the git.
https://stackoverflow.com/questions/17683458/how-do-i-commit-case-sensitive-only-filename-changes-in-git

I had the same problem, i just restarted the vs code and then it works.

I had the same problem. VSCode restart  and  setting "forceConsistentCasingInFileNames": false can't help me.
error info:(just differ in disk symbol)
File name 'D:/mycode/devmono2/packages/server-sdk/index.ts' differs from already included file name 'd:/mycode/devmono2/packages/server-sdk/index.ts' only in casing.Vetur(1149)
why?

Solved!
Mine was a simple problem.
Everything was fine locally on mac, but i was getting this error on my Jenkins server.  The problem was simply that locally the filename was 'quote.ts' but within my remote git repo it was 'Quote.ts'.
How the problem came about
So the filename was originally 'Quote.ts' which i pushed up. I then changed to 'quote.ts', but git did not see this as a change (I believe macs are case insensitive) - and therefore the case change was not reflected in remote repo.
And so when Jenkins pipeline ran, it was pulling down 'Quote.ts' when it was being referenced as '../../quote.ts' - which results in the error.
How i solved it

Drink coffee
Change filename in remote repo manually to be lowercase
git pull locally
Run pipeline

Boom Bam! Hope this helps someone

Had a similar issue that generated around 100 of these errors.
Didn't read them all to closely as they all seemed to have the same root problem. Had recently switched to running node in WSL2 that I took for granted was the cause of the issue. After 2 hours without success I actually went through the whole list and found one example where there was an actual bug.
import { FooInterface } from '../Foo'; // should be '../foo'
import { BarService } from '../Bar'; // should be '../bar'
I changed it to lowercase, reloaded vscode, rebuilt it and all the errors went away.
What I think happend is that ts tried to import the incorrect path ('Foo' instead of 'foo') and then cached the result and tried to use the incorrect cached path when processing the rest of the code.
Hopefully this helps someone out there!

i just restart my ide then the error is gone

just make a same object in component, and start your ng, after running successfully, then remove object and import your link,
then its should be work,
In my project it worked.

this just happened to me, it's an ongoing issue that hasn't been solved. It happens whenever I change the casing of a directory name and try to commit the change. Git doesn't recognize change in casing for a directory name.

+1......but WHYYYYYYYY

I just faced the problem. It turned out that when I tried to import it using vscode, VSCode used the previous file name and directory name. So I modified it and after running tsc it didn't show any error in the terminal, but I was still getting that read squiggly line in VSCode. So I restarted the Typescript server in vscode (Typescript: Restart TS Server) and it went away. Hope it helps someone.

@ninjavang you helped me! Had the same issue happening with my gitlab ci/cd pipeline. Turns out when i went to the remote branch and checked the file's path, there were two versions of the file, each in their case sensitive version.
I deleted the version unwanted on the remote, copied the file locally, pulled the remote down, refreshed my VS Code, and re-added the file. It worked!
Thanks for the help!

So tried all the restart etc without success... Turns out the issue is that if another file in the project references the old casing then every other file which attempts to use the new (correct) casing will fail.   So ensure that the file's casing has been changed in every file. That is so a bug.

At version 4.1.3 still reproduced.

At version 4.1.3 still reproduced.

Experienced this yesterday when renaming files and methods. WebStorm got lost and was displaying this even after I had fixed everything. A good old cmd+q and restart of WebStorm fixed it. 🤷

Experienced this yesterday when renaming files and methods. WebStorm got lost and was displaying this even after I had fixed everything. A good old cmd+q and restart of WebStorm fixed it. 🤷

If the name was link.ts and you want to save it as Link.ts, on macOS and In vscode go to File -> Save As... and choose Link.ts.

If the name was link.ts and you want to save it as Link.ts, on macOS and In vscode go to File -> Save As... and choose Link.ts.

https://github.com/microsoft/TypeScript/issues/42270#issuecomment-770185974

webstorm encountered this problem  .   Restart TS Server not working

git rm -r --cached .  then reboot VSCode, it works~

The following command fixed things up for me:
git clean -fdx

ctrl+shift+p: restart eslint server

The mentions of this being Windows-only might be related to the drive letter, as I just had this issue with this message:
Already included file name<EMAIL_ADDRESS>differs from file name<EMAIL_ADDRESS>only in casing.
  The file is in the program because:
    Imported via 'webpack-dev-server' from file 'c:/Users/Kelvin/Documents/Workspaces/NPM/vscode-sshfs/webview/webpack.config.js' with packageId '<EMAIL_ADDRESS>    Root file specified for compilation
    Root file specified for compilation ts(1261)

For comparsion:
<EMAIL_ADDRESS><EMAIL_ADDRESS>
Weirdly enough, in my case the cause was that I had the mentioned index.d.ts open in VS Code which actually used C:/, while the in-code version (including other dependencies) use c:/ as path. Just a weird IDE issue for me, but perhaps worth mentioning.

Just reload the window.
shift + comman(macOS)/alt (windows) + P
reload window

This is turning into a support thread but this is not a support forum.
If you have a reproducible specific defect, please file a new issue following the template.
If you have a configuration problem, please use an appropriate support forum such as StackOverflow or Discord.


Dependency check

Check dependencies and update if possible.

Dependabot can be introduced to tackle dependency management.

closed with https://github.com/volkanto/yaba/pull/46


portfolio - withdraw error

flow:

go to portfolio
try to withdraw RSR token
see this error

token transfers are disabled on the token level.
opening a different issue to handle this.


Two phase init was not passing smartContractGW to receipt store

Which meant no WS receipts were being delivered.
This was caused by new initialization order in #227
Found through e2e in FireFly under https://github.com/hyperledger/firefly/pull/1033

Codecov Report

Merging #228 (4b2666f) into main (0e6f5b0) will increase coverage by 0.00%.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #228   +/-   ##
=======================================
  Coverage   96.95%   96.95%
=======================================
  Files          70       70
  Lines        7728     7730    +2
=======================================
+ Hits         7493     7495    +2
  Misses        184      184
  Partials       51       51

Impacted Files
Coverage Δ

ethconnect/internal/ws/wsserver.go
100.00% <0.00%> (ø)

ethconnect/internal/rest/restgateway.go
94.41% <0.00%> (ø)

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


Add a mix sql_fmt.format task

Similar to mix format but for SQL queries 😄 .

The code is mostly "stolen" from mix format. We already use pgformatter in our CI pipelines, we could replace it with this.
If you are ok with adding the mix task, I will add tests and CLI flags for the supported formatting options.

That would be a really cool use case (and one that I didn't think about)! Definitely would be interested in merging this one when you button it up.
I have also used pgformatter in the past as well.....but it is much slower given that it is written in perl and also chokes on very large queries.

@akoutmos ready

This hit me just as I was reviewing the code. How about instead of copying in the mix format mix task we instead make this a mix format plugin? https://hexdocs.pm/mix/main/Mix.Tasks.Format.html#module-plugins
That way we lean on the existing mix tooling and --check-formatted will work just the same. That and we can also create a ~SQL """...""" sigil for formatting SQL statements which would be quite handy in migrations. I always hate manually formatting SQL statements in migrations and a sigil would tie in nicely with the mix format plugin.
Thoughts?

I like it, new PR coming soon. My only concern with the ~SQL sigil is that it will make :sql_fmt a required prod dependency as well, I planned to have it only as :dev, but this is minor.

Closing in favour of #3


Show Dropdown in objective-c

I tried but there error while using show method with dropdown object in objective-c.
how can i call show method for display dropdown in objective-c?

Hello @sagar-vaholiya,
Thanks for using DropDown!
Unfortunately, DropDown was not made to be compatible with Objective-C. The reason the show method cannot be converted from Swift to Objective-C is that the method returns a tuple and tuples are a Swift only feature:
public func show() -> (canBeDisplayed: Bool, offscreenHeight: CGFloat?)

Nonetheless, it seems like the show method is the only problem for Objective-C compatibility. I could do something like:
@objc(show)
public func objc_show() {
	show()
}

So you could have a show method in Objective-C calling the show Swift method. The only problem is that you will miss the 2 parameters canBeDisplayed and offscreenHeight. But I could return a NSDictionary with two NSNumbers representing these values.
What do you think? :)

Thank you for your complement.
Yes, it's working.
But, what about other functionality such as "selectionAction"?
dropDown.selectionAction = { [unowned self] (index, item) in 			self.actionButton.setTitle(item, forState: .Normal) 		}
i think, you have to make complete Objective-C version for it.

The converted Objective-C code I have does contain the selectionAction:
@property (nonatomic, copy) void (^ __nullable selectionAction)(NSInteger, NSString * __nonnull);

And so I can use it like this:
DropDown *dropDown = [[DropDown alloc] init];
dropDown.selectionAction = ^(NSInteger, NSString * _Nonnull) {
	//code
};

You can't use this line of code above without getting an error?

yes, above code generates an error.

Ok, strange. So you'll probably have to change all the typealias because should not be convertible to Objective-C, thus the variable of type of a typealias is not convertible to Objective-C.
So for the typealias
public typealias Closure = () -> Void

Just search Closure in the code and replace by () -> Void.
Same for
public typealias SelectionClosure = (Index, String) -> Void
public typealias ConfigurationClosure = (Index, String) -> String

Then build your project, you should now have the dropDown.selectionAction available in Objective-C, along with cancelAction, ...

I didn't have any trouble with the Closure syntax bridging to Objective-C. Perhaps Xcode 7.3 can handle backwards-bridging the typealiases now?


No gps output on zmq

Reopens #5
I do receive gps info in log files, is this what you mean with fix?
In that case yes, network-listener get gps data (I see it in the logfile) but I never get it on the zmq or on netlink socket.

network-listener 0.1.0-20151203x133227 (and a reboot fixed the issue)


recipe-modifier added

What does this PR do?
This tool can modify recipes to be vegetarian, non-vegetarian, or even tailored for a healthy diet. Enter your recipe, select a modification, and get instant suggestions!
Related PRs and Issues
#625

(Direct link to related PR or issue, if relevant)

Checklist

[x] I have read read the Contribution Guide
[x] If the PR changes or adds functionality, I have updated the relevant documentation

@quest-bot loot #625

@NathanTarbert can you plz review this pr asap as the event is going on and i am losing my confidence due to hanging prs .

Amazing job @anshuman-rai-27!


No overlay with Drawer

I'm trying to use the Drawer component, it works, but without the Overlay.
I'm using it in a very simple component.
class AppMenu extends React.Component {
  constructor(props) {
    super(props);

    this.state = {
      isOpen: true,
    };

    this.handleCloseDrawer = this.handleCloseDrawer.bind(this);
  }

  handleCloseDrawer() {
    this.setState({
      isOpen: false,
    });
  }

  render() {
    const { isOpen } = this.state;

    return (
      <div>
        <Drawer
          anchor="left"
          open={this.state.isOpen}
          onClose={this.handleCloseDrawer}
          variant="persistent"
        >
          <div
            tabIndex={0}
            role="button"
            onClick={this.handleCloseDrawer}
            onKeyDown={this.handleCloseDrawer}
          >
            <MenuItem>Menu Item</MenuItem>
          </div>
        </Drawer>
      </div>
    );
  }
}

[ x ] I have searched the issues of this repository and believe that this is not a duplicate.

Expected Behavior

The Drawer, when open, should render a div tag to display an overlay which can be clicked to close the Drawer.
Current Behavior
The menu is created, without an overlay, so I can't close the menu by clicking outside of it.

Steps to Reproduce (for bugs)
Very simple usage: https://codesandbox.io/s/n51myz595l
Context

I'm trying to have a Drawer like the demos.
Your Environment
The bug are here and on my Mac High Sierra with Google chome: https://codesandbox.io/s/n51myz595l

You are using variant="persistent", which doesn't have an overlay because it is persistent. If you want a temporary drawer with an overlay that can close, use variant="temporary" (the default value for variant).

You're not seeing an overlay because you're using a persistent draw. Try removing the variant prop.

Thank you guys, very sorry about that.

Oops, we both replied at the same time. Jinx!

We also have a backdrop component. It's not documented yet but it's public:
https://github.com/mui-org/material-ui/blob/b338fa3c285b54fa2d3d322bd1b4ab0de61f163d/src/index.js#L48

You're not seeing an overlay because you're using a persistent draw. Try removing the variant prop.

This was how I put a backdrop behind a persistent drawer eventually, thank you!


[BUG] The cancel inverses compilation pass seems to mutate the original circuit(?)

My description might not be technically accurate, however I find the following behaviour unintuitive:

from catalyst.debug import get_compilation_stage
from catalyst.passes import cancel_inverses

dev = qml.device("lightning.qubit", wires=1)

@qml.qnode(dev)
def circuit(x: float):
    qml.RX(x, wires=0)
    qml.Hadamard(wires=0)
    qml.Hadamard(wires=0)
    return qml.expval(qml.PauliZ(0))

@qjit(keep_intermediate=True)
def workflow(x):
    optimized_circuit = cancel_inverses(circuit)
    return circuit(x), optimized_circuit(x)

Output of the compilation pass

>>> print(get_compilation_stage(workflow, "QuantumCompilationPass"))
module @workflow {
  func.func public @jit_workflow(%arg0: tensor<f64>) -> (tensor<f64>, tensor<f64>) attributes {llvm.emit_c_interface} {
    %0 = call @circuit(%arg0) : (tensor<f64>) -> tensor<f64>
    %1 = call @circuit(%arg0) : (tensor<f64>) -> tensor<f64>
    return %0, %1 : tensor<f64>, tensor<f64>
  }
  func.func private @circuit(%arg0: tensor<f64>) -> tensor<f64> attributes {diff_method = "parameter-shift", llvm.linkage = #llvm.linkage<internal>, qnode} {
    quantum.device["/Users/josh/miniconda3/lib/python3.10/site-packages/catalyst/utils/../lib/librtd_lightning.dylib", "LightningSimulator", "{'shots': 0, 'mcmc': False, 'num_burnin': 0, 'kernel_name': None}"]
    %0 = quantum.alloc( 1) : !quantum.reg
    %1 = quantum.extract %0[ 0] : !quantum.reg -> !quantum.bit
    %extracted = tensor.extract %arg0[] : tensor<f64>
    %out_qubits = quantum.custom "RX"(%extracted) %1 : !quantum.bit
    %2 = quantum.namedobs %out_qubits[ PauliZ] : !quantum.obs
    %3 = quantum.expval %2 : f64
    %from_elements = tensor.from_elements %3 : tensor<f64>
    %4 = quantum.insert %0[ 0], %out_qubits : !quantum.reg, !quantum.bit
    quantum.dealloc %4 : !quantum.reg
    quantum.device_release
    return %from_elements : tensor<f64>
  }
  func.func @setup() {
    quantum.init
    return
  }
  func.func @teardown() {
    quantum.finalize
    return
  }
}

It can be seen looking at the output of the compilation pass, that the qjit function is simply returning optimized_circuit(x), optimized_circuit(x) (e.g., there is only one circuit function present after compilation, and it has the hadamard gates cancelled after execution).

I would say that this is the intended behavior, as compiler transformations, like those in clang, are supposed to alter the input function in-place (i.e. without keeping a copy of the original).
If we do want to keep the original, implementation wise it might not be straightforward but it should still be possible.

I now think this is an easy fix since qnodes have a built-in __copy__ method.


Update port to Release 221118-e58fee0fb

Hi.
Is it possible to update the port to latest release 221118-e58fee0fb?
https://github.com/photoprism/photoprism/releases/tag/221118-e58fee0fb
Thank you very much

done @ https://github.com/huo-ju/photoprism-freebsd-port/commit/1bf2a4041e7ef695a6db0e21cf30272c2f6c1584

@dml33 @psa -- are you still maintaining the binaries repo? Is it possible to post a binary for the latest version(s) of PhotoPrism such as https://github.com/photoprism/photoprism/releases/tag/230719-73fa7bbe8? Also, is this able to be compiled under Freebsd 13.2 release?

Is it possible to post a binary for the latest version(s) of PhotoPrism such as https://github.com/photoprism/photoprism/releases/tag/230719-73fa7bbe8?

That is already the case: https://github.com/huo-ju/photoprism-freebsd-port/pull/59

That is already the case: #59

Thank you @tinsukE . Would you please confirm the FreeBSD release you are using for this (12.4 or 13.2) and version of libtensorflow you use?

I compile libtensorflow1 and photoprism myself and run, all on 13.2 jails.
For libtensorflow1  (latest 1.15.5_2) I use the port linked on the README.md: https://github.com/huo-ju/photoprism-freebsd-port#dependencies

Hi.
Sorry for the delay.
I have just uploaded the g20230719 release for FreeBSD 12.3:
https://github.com/psa/photoprism-freebsd-port/releases
Currently, I am still using FreeBSD 12.3. I want to upgrade my NAS to 13.x soon, but I cannot find enough free time to do it. I am sure it will not be as smooth as it would be 👯 As soon as I do it, I will go on publishing the binaries for the FreeBSD 13.x branch.
Regards

I'm building binary package for FreeBSD 13.2/amd64 for myself, if anyone wants I just uploaded it on my fork release page.

I'm building binary package for FreeBSD 13.2/amd64 for myself, if anyone wants I just uploaded it on my fork release page.

were you able (and if yes, how - poudriere, ports, what dev stack tools installed) to build libtensorflow1 under 13.2? I cannot get past an error when trying in a 13.2 jail:
ERROR: /usr/ports/science/libtensorflow1/work-default/tensorflow-1.15.5/tensorflow/core/BUILD:2724:1: ProtoCompile tensorflow/core/framework/allocation_description.pb.h failed (Illegal instruction): protoc failed: error executing command bazel-out/host/bin/external/com_google_protobuf/protoc '--cpp_out=bazel-out/host/bin' -I. -I. -Iexternal/com_google_protobuf/src -Ibazel-out/host/bin/external/com_google_protobuf/src ... (remaining 3 argument(s) skipped)
ERROR: /usr/ports/science/libtensorflow1/work-default/tensorflow-1.15.5/tensorflow/core/BUILD:2747:1 ProtoCompile tensorflow/core/framework/allocation_description.pb.h failed (Illegal instruction): protoc failed: error executing command bazel-out/host/bin/external/com_google_protobuf/protoc '--cpp_out=bazel-out/host/bin' -I. -I. -Iexternal/com_google_protobuf/src -Ibazel-out/host/bin/external/com_google_protobuf/src ... (remaining 3 argument(s) skipped)
INFO: Elapsed time: 0.907s, Critical Path: 0.03s
INFO: 0 processes.
FAILED: Build did NOT complete successfully
*** Error code 1

I don't remember having problems compiling it, but it was a while ago. It's possible that it no longer compiles with more recent dependencies. I will upload my package for that too.

Thank you @lapo-luchini @dml33 and @tinsukE for your updates/binaries. Was able to install photoprism 2023-07-19 package compiled by @lapo-luchini for FreeBSD 13.2-release and libtensorflow1 from @psa repo (https://github.com/psa/libtensorflow1-freebsd-port/releases/tag/1.15.5-pre-release-0). Will report back if there are any functionality issues once I test the install.

I don't remember having problems compiling it, but it was a while ago. It's possible that it no longer compiles with more recent dependencies. I will upload my package for that too.

@lapo-luchini -- would you please upload to your fork release page the binary for the photoprism latest release 230923 (for FreeBSD 13.2/amd64)?
Still unable to compile this (not sure why, maybe it has to do with the jail on top of Truenas and not clean FreeBSD).

binary for the photoprism latest release 230923

Done.


[headerType] Newline on the end for slashstar_style

If you see examples in java, kotlin, etc., they mostly have newline after the copyright statement. See https://github.com/mathieucarbou/license-maven-plugin/pull/186#issuecomment-686104011
It might be because Intellij applies header file in that way and also because in this way this would be easier to divide copyright and the code.
Since customizing the headerDefinition regarding ending newline makes a quite long configuration having to reconfigure all other values, making it default might be better ƒor this plugin to be used more common.

I agree with you, but this change will break the current behaviour for existing users so this is not acceptable the way it is implemented right now.
Also, what you say might also be true for other header styles, not just java.
So we would need to find a more global solution for that, which is backward compatible also.


[ c-documentation ] Adding a warning that DoD Certificates page is out-of-date

It's out of date! We'll update it eventually but for now this is the
quickest fix.

hey @transcom/truss-platform-team. This documentation on this page is old, and I'm adding an admonition to address that it's out of date. Could I get a review on this? Thanks!


Redraw map after width change by div

Hi, I have a map in a bootstrap div:
<div class="col-xs-10 col-xs-offset-1" id="kartencol">
          <leaflet defaults="defaults" center="center" paths="meinePfade" bounds="bounds"     decorations="decorations" width="100%" height="480px" markers="markers"></leaflet>
</div>

Then I use jquery and css animation to change the width of the col div and put a form next to it.
$('#kartencol').toggleClass('col-xs-10 col-xs-7')
$('#formcol').toggleClass('col-xs-0 col-xs-3')

It works great, but when I now do something with the map like setting bounds, angular still thinks I have the old map width.
How do I do a redraw or anything else with the map, to let angular know about the new width?
Best
Joerg

This issue was moved to angular-ui/ui-leaflet#32

I'm going to rework&redesign angular-leaflet-directive to be compatible with Leaflet v1.0. It will mantain almost all its functionality, and will be compatible with the current features of the directive, but I must start from a fresh point, so I'm going to close this issue. If you think it must be worked with the new version, please reopen it.


No registration exists matching provided key

authorization failed after 1m0s: getting authorization failed: 403 urn:acme:error:unauthorized: No registration exists matching provided key

Sorry, accidentally submitted before finishing, I'm editing the issue now with the full details.

I deleted the kube-lego-account secret and the lego pod. The new pod then created a new account secret and everything worked.
I'll close this issue but it may be something to look into more to find out why this would happen and a way to not have it require this manual deletion?

Just trapped into exactly the same issue. kube-lego was installed via helm


Set and get options after menubar is created

Good to have when you need to set some option after the app is created. For instance when you have some logic that depends on electrons screen module which can't run before the app is ready.

Released in 2.1.0


Fix for 1507

Need to skip all non handled syntax kinds early to avoid looking them up in a dictionary and thus causing a key not found exception.
Fixes #1507

Current coverage is 79.44%

Merging #1740 into master will decrease coverage by -0.05% as of 620cb55

@@            master   #1740   diff @@
======================================
  Files          546     546
  Stmts        32741   32749     +8
  Branches      9070    9074     +4
  Methods
======================================
- Hit          26028   26018    -10
- Partial       5269    5271     +2
- Missed        1444    1460    +16

Review entire Coverage Diff as of 620cb55

Powered by Codecov. Updated on successful CI builds.

:+1: Looks good to me

This is basically ready to merge. I verified that the code fix you provided does fail with an AD0001 when the corresponding change to the analyzer is omitted.
:bulb: I highly recommend adding the email address you used for these commits to your GitHub profile. It will not be shared, but it will allow you to associate these commits with your username here. Currently if you view commit 40fe029 the author says Ralf Beckers but you can't click the name to jump to your profile.

addressed all feedback.

:+1:


PlayMode splited in Repeat and Shuffle

PlayMode splited in Repeat and Shuffle
Description
Sonos saves the PlayMode in one ENUM with 6 states, but actually it's Shuffle Yes/No and Repeat Off/RepeatOne/RepeatAll. The library now has thse splitted into two. And you're able to set either and the PlayMode is computed automatically.
Your checklist for this pull request
🚨 Please review the guidelines for contributing to this repository.

[x] Make sure you are requesting to pull a topic/feature/bugfix branch (right side). Don't request your master!
[x] Make sure you are making a pull request against the beta branch (left side). Also you should start your branch off svrooij/node-sonos-ts/beta.
[x] Check the commit's or even all commits' message styles matches our requested structure.
[x] Check your code additions will fail neither code linting checks nor unit test.
[x] Check you add tests for added code.

💔 Thank you!

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

30 of 31 (96.77%)  changed or added relevant lines in 4 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.2%) to 81.587%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/helpers/playmode-helper.ts
10
11
90.91%

Totals

Change from base Build<PHONE_NUMBER>:
0.2%

Covered Lines:
1684

Relevant Lines:
2019

💛  - Coveralls


Add Data Table and REST Book to recommendations in .vscode/extensions.json

see: https://github.com/RandomFractals/vscode-data-table/blob/main/.vscode/extensions.json

see: https://github.com/RandomFractals/vscode-leaflet/blob/main/.vscode/extensions.json


Code completion

I'm able to type and run the example code but didn't see any pop-ups. How is this feature supposed to behave?

Could you try Ctrl-. key combination for completion?

works perfectly thanks!


remove SkipVerify field on secret table.

as title.
Signed-off-by: Bo-Yi Wu<EMAIL_ADDRESS>

removing form the struct is problematic for the database mapper because the field exists in the database, and the mapper expects a corresponding field in the struct. Removing results in warnings from the mapper every time the query is run:
$ go test -v

2017/05/05 11:17:31 meddler.Targets: column [secret_skip_verify] not found in struct
2017/05/05 11:17:31 meddler.WriteTargets: column [secret_skip_verify] not found in struct

In the past I've deprecated fields by doing something like this:
-Conceal bool     `json:"-"               meddler:"secret_conceal"`
+XConceal bool     `json:"-"               meddler:"secret_conceal"`

And what about just writing a migration to drop this column?

@tboerger sqlite doesn't support dropping columns. The process for removing a column is quite complex since you need to copy the data to a temp table, drop and re-create the original table and indexes without the column, copy the data back to the revised table, and delete the temp table.
The migration would need to look something like this:
-- create the temp table
CREATE TABLE tmp_secrets (
 secret_id        INTEGER PRIMARY KEY AUTOINCREMENT
,secret_repo_id   INTEGER
,secret_name      TEXT
,secret_value     TEXT
,secret_images    TEXT
,secret_events    TEXT
,secret_skip_verify  BOOLEAN
,secret_conceal      BOOLEAN
);

-- copy data to the temp table
INSERT INTO tmp_secrets SELECT * FROM secrets;

-- drop the original index and table
DROP INDEX ix_secrets_repo;
DROP TABLE secrets;

-- re-create the table without the column
CREATE TABLE secrets (
 secret_id        INTEGER PRIMARY KEY AUTOINCREMENT
,secret_repo_id   INTEGER
,secret_name      TEXT
,secret_value     TEXT
,secret_images    TEXT
,secret_events    TEXT
, secret_conceal  BOOLEAN
,UNIQUE(secret_name, secret_repo_id)
);

-- re-create the index
CREATE INDEX ix_secrets_repo  ON secrets  (secret_repo_id);

-- copy the data back from the temp table
INSERT INTO secrets
SELECT
 secret_id
,secret_repo_id
,secret_name
,secret_value
,secret_images
,secret_events
,secret_conceal
FROM tmp_secrets;

-- drop the temp table
DROP TABLE tmp_secrets;

For the record I am not against such scripts. I just usually choose to leave the column in place because it is easier and less risky. I certainly support such efforts as long as they are accompanied by heavy testing to minimize risk during upgrades.


⚠ Fortnite has degraded performance

In 97ec086, Fortnite (https://fortnite.com) experienced degraded performance:

HTTP code: 200
Response time: 1715 ms

Resolved: Fortnite performance has improved in ccd0916.


redoc-cli: ReferenceError: URL is not defined

Hi there!
There seems to be an issue with redoc-cli ATM.
Noticed it with one of our docs, but can be reproduced using the demo/swagger.yaml file from this repository:
$> yarn add redoc-cli
$> ./node_modules/.bin/redoc-cli bundle -o './redoc-test' demo/swagger.yaml
[ReDoc Compatibility mode]: Converting OpenAPI 2.0 to OpenAPI 3.0
Prerendering docs
ReferenceError: URL is not defined
    at removeQueryString (/tmp/node_modules/redoc/bundles/redoc.lib.js:6940:15)
    at /tmp/node_modules/redoc/bundles/redoc.lib.js:11055:137
    at Array.map (<anonymous>)
    at Object.children (/tmp/node_modules/redoc/bundles/redoc.lib.js:11051:104)
    at ReactDOMServerRenderer.render (/tmp/node_modules/react-dom/cjs/react-dom-server.node.development.js:3412:55)
    at ReactDOMServerRenderer.read (/tmp/node_modules/react-dom/cjs/react-dom-server.node.development.js:3161:29)
    at Object.renderToString (/tmp/node_modules/react-dom/cjs/react-dom-server.node.development.js:3646:27)
    at /tmp/node_modules/redoc-cli/index.js:189:29
    at Generator.next (<anonymous>)
    at fulfilled (/tmp/node_modules/redoc-cli/index.js:5:58)

$> nodejs --version
v8.10.0
$> yarn --version
1.13.0

it was working 2 days ago... now i got the same error...

Could you try the same with node 10.x?

i got a similar error:
npx -p node@10  redoc-cli bundle --output deploy/api/index.html deploy/swagger.yaml
[ReDoc Compatibility mode]: Converting OpenAPI 2.0 to OpenAPI 3.0 Prerendering docs TypeError [ERR_INVALID_URL]: Invalid URL:      at onParseError (internal/url.js:241:17)     at new URL (internal/url.js:319:5)     at removeQueryString (/usr/local/lib/node_modules/redoc-cli/node_modules/redoc/bundles/redoc.lib.js:6940:15)     at /usr/local/lib/node_modules/redoc-cli/node_modules/redoc/bundles/redoc.lib.js:11055:137     at Array.map (<anonymous>)     at Object.children (/usr/local/lib/node_modules/redoc-cli/node_modules/redoc/bundles/redoc.lib.js:11051:104)     at ReactDOMServerRenderer.render (/usr/local/lib/node_modules/redoc-cli/node_modules/react-dom/cjs/react-dom-server.node.development.js:3412:55)     at ReactDOMServerRenderer.read (/usr/local/lib/node_modules/redoc-cli/node_modules/react-dom/cjs/react-dom-server.node.development.js:3161:29)     at Object.renderToString (/usr/local/lib/node_modules/redoc-cli/node_modules/react-dom/cjs/react-dom-server.node.development.js:3646:27)     at /usr/local/lib/node_modules/redoc-cli/index.js:189:29

Thanks. Working on a fix!

Downgrading to v2.0.0-rc.4 fixes the issue.

rc.7 is coming with the fix

Should be fixed now. @srats could you please confirm.
Thanks in advance!

Still having the same issue.
From yarn.lock file:
<EMAIL_ADDRESS>  version "2.0.0-rc.7"
  resolved "https://registry.yarnpkg.com/redoc/-/redoc-2.0.0-rc.7.tgz#ef27c64ea9fbc0aa5b1135dc85d59b64eef87396"
  integrity sha512-+2OPuEuUDzwt/R6ZXpBuCMy7aCo0k2pQKfOHq6eG+84QO9+y0ASr91WGmaHUtsvDaItsyrfmhW35NjXIbps2ug==

./node_modules/.bin/redoc-cli bundle -o './redoc-test' ~/Downloads/swagger.yaml
[ReDoc Compatibility mode]: Converting OpenAPI 2.0 to OpenAPI 3.0
Prerendering docs
ReferenceError: URL is not defined
    at parseURL (/tmp/node_modules/redoc/bundles/redoc.lib.js:6950:9)
    at removeQueryString (/tmp/node_modules/redoc/bundles/redoc.lib.js:6940:15)
    at /tmp/node_modules/redoc/bundles/redoc.lib.js:11064:137
    at Array.map (<anonymous>)
    at Object.children (/tmp/node_modules/redoc/bundles/redoc.lib.js:11060:104)
    at ReactDOMServerRenderer.render (/tmp/node_modules/react-dom/cjs/react-dom-server.node.development.js:3412:55)
    at ReactDOMServerRenderer.read (/tmp/node_modules/react-dom/cjs/react-dom-server.node.development.js:3161:29)
    at Object.renderToString (/tmp/node_modules/react-dom/cjs/react-dom-server.node.development.js:3646:27)
    at /tmp/node_modules/redoc-cli/index.js:189:29
    at Generator.next (<anonymous>)

🤦♂ my mistake...
rc.8 is coming

It works with Nodejs 10 now, BTW.

@srats could you try it now with rc.8?

  version "2.0.0-rc.8"
  resolved "https://registry.yarnpkg.com/redoc/-/redoc-2.0.0-rc.8.tgz#46fb40900a04f2438cabd973181860323e1ee7de"
  integrity sha512-DpPa/rAUoSDYWGk5k/i0KUoRPrj62WKmbffHmS+c7zZGmFjtzVMpUzwrxrs3Dmf+u1t4YGw5wuhyq2p7qSlhsQ==

Works fine with nodejs 8, thanks :)

Why package number on npmjs is still rc7? https://www.npmjs.com/package/redoc
rc8 marked as «next» :thinking:

node version 10.8.0
redoc version 2.0.0-rc.8
redoc-cli version 0.8.3
[ReDoc Compatibility mode]: Converting OpenAPI 2.0 to OpenAPI 3.0
TypeError [ERR_INVALID_URL]: Invalid URL:
at onParseError (internal/url.js:237:17)
at parse (internal/url.js:246:3)
at new URL (internal/url.js:321:5)
at parseURL (/usr/local/lib/node_modules/redoc-cli/node_modules/redoc/bundles/redoc.lib.js:6950:16)
at removeQueryString (/usr/local/lib/node_modules/redoc-cli/node_modules/redoc/bundles/redoc.lib.js:6940:15)
at /usr/local/lib/node_modules/redoc-cli/node_modules/redoc/bundles/redoc.lib.js:11064:137
at Array.map ()
at Object.children (/usr/local/lib/node_modules/redoc-cli/node_modules/redoc/bundles/redoc.lib.js:11060:104)
at ReactDOMServerRenderer.render (/usr/local/lib/node_modules/redoc-cli/node_modules/react-dom/cjs/react-dom-server.node.development.js:3412:55)
at ReactDOMServerRenderer.read (/usr/local/lib/node_modules/redoc-cli/node_modules/react-dom/cjs/react-dom-server.node.development.js:3161:29)

Why package number on npmjs is still rc7
Fixed to rc.8

got the same error...
node version v10.15.3
redoc-cli version 0.8.3
[ReDoc Compatibility mode]: Converting OpenAPI 2.0 to OpenAPI 3.0
Prerendering docs
TypeError [ERR_INVALID_URL]: Invalid URL: /v1
at onParseError (internal/url.js:241:17)
at new URL (internal/url.js:319:5)
at parseURL (C:\Users\A11180221040052\AppData\Roaming\npm\node_modules\redoc-cli\node_modules\redoc\bundles\redoc.lib.js:6950:16)
at removeQueryString (C:\Users\A11180221040052\AppData\Roaming\npm\node_modules\redoc-cli\node_modules\redoc\bundles\redoc.lib.js:6940:15)
at C:\Users\A11180221040052\AppData\Roaming\npm\node_modules\redoc-cli\node_modules\redoc\bundles\redoc.lib.js:11064:137
at Array.map ()
at Object.children (C:\Users\A11180221040052\AppData\Roaming\npm\node_modules\redoc-cli\node_modules\redoc\bundles\redoc.lib.js:11060:104)
at ReactDOMServerRenderer.render (C:\Users\A11180221040052\AppData\Roaming\npm\node_modules\redoc-cli\node_modules\react-dom\cjs\react-dom-server.node.development.js:3412:55)
at ReactDOMServerRenderer.read (C:\Users\A11180221040052\AppData\Roaming\npm\node_modules\redoc-cli\node_modules\react-dom\cjs\react-dom-server.node.development.js:3161:29)
at Object.renderToString (C:\Users\A11180221040052\AppData\Roaming\npm\node_modules\redoc-cli\node_modules\react-dom\cjs\react-dom-server.node.development.js:3646:27)

Does it work now?

@kevintsengtw, @zhangwbgogogo could you share your spec servers if you use OpenAPI v3 or host, basePath and schemes if using OpenAPI 2

@RomanGotsiy
I have same issue and have no servers field in spec (openapi v3)

@RomanGotsiy Just ran

redoc-cli bundle swagger.yaml"

Should be fixed now with rc.8-1

I also bumped redoc-cli version.
Could you guys verify it is working now? Thanks

This seems to be working correctly for me now. Thanks @RomanGotsiy!

@pohmelie @zhangwbgogogo @kevintsengtw ?

it's working now! thank you @RomanGotsiy!

🎉

@RomanGotsiy
Works fine now, thank you!

@RomanGotsiy
Works fine, thank you!

it's working now! thanks

@RomanGotsiy
<EMAIL_ADDRESS><EMAIL_ADDRESS>Works fine now, thanks


Display time option

Hello,

I am happy with your product, overall!

This is not an issue but request for Enhancement.

When I open a log file, Time column display computer's local time rather than
original time the log was created. With different time zones and different
products, It become an issue to match up all the times.

Is there any way to configure OtrosLogViewer to display time when the log was
created, instead(rather than converting to local time) ? Forgive me if it can
already be done.

I use build from 2012-09-13

Regards
Prashanth

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 3 Feb 2015 at 2:11

Can be useful in case of misaligned time in multiple machines
I'm happy too with your product


Using a template with single quote fails

The follow code
uriTemplate, err := template.New("{+baseurl}/reports/getActiveUserDetail(period='{period}')")

fails with the error
errors.errorString{s:"uritemplate:72:invalid literals"}

according to

If the literal character is allowed anywhere in the URI syntax (unreserved / reserved / pct-encoded ), then it is copied directly to the result string.

https://datatracker.ietf.org/doc/html/rfc6570#section-3.1
single quote being a reserved literal in the URI spec, it should be copied as is (and therefore accepted).
After testing the sample template with a dotnet equivalent library and a java equivalent library parses the template properly.
Removing the quotes from the template also results in a successful parsing.
I believe an entry should be added here, but before I submitted any pull request I wanted to get a confirmation on the analysis.
https://github.com/yosida95/uritemplate/blob/fa7d6df706b972aab27d83eb70e9f7df6696cba5/parse.go#L42

CC @yosida95

Sorry for the late reply. I think the current implementation works fine as per the URI Template spec.
The syntax of the URI template is defined in $2 as follows. Please note single quotation marks are explicitly excluded.

 URI-Template  = *( literals / expression )
 (snip)
 literals      =  %x21 / %x23-24 / %x26 / %x28-3B / %x3D / %x3F-5B
               /  %x5D / %x5F / %x61-7A / %x7E / ucschar / iprivate
               /  pct-encoded
                    ; any Unicode character except: CTL, SP,
                    ;  DQUOTE, "'", "%" (aside from pct-encoded),
                    ;  "<", ">", "\", "^", "`", "{", "|", "}"

-- https://datatracker.ietf.org/doc/html/rfc6570#section-2

As single quotation marks are not allowed in a literal part, I think raising an error while parsing is the correct behavior.
I understood the quote from §3 you mentioned above like "If the character of a literal part is contained in unreserved, reserved or pct-encoded then copy it as-is, otherwise do pct-encoding".
I'm not a native English speaker, my understanding may be wrong. Please feel free to point out any misunderstandings.

Thanks for the reply!
From the paragraph right above this notation

The characters outside of expressions in a URI Template string are
intended to be copied literally to the URI reference if the character
is allowed in a URI...

' is a character that's allowed in a URI without needing to be encoded.
It does appear the spec contradicts itself right here, as well as here https://datatracker.ietf.org/doc/html/rfc6570#section-3.1
Since 2 parts of the spec indicate that single quotes are allowed, and only one says they are not, and since other libraries accept the single quote, I'd argue we should align on that behaviour and accept the single quote in this library as well.
Lastly you can also validate with this tool
https://www.utilities-online.info/uritemplate
{+baseurl}/reports/getActiveUserDetail(period='{period}')

{
  "baseurl": "https://localhost",
  "period" : "2D"
}

Thank you for the extensive reply.
We could probably percent encode the single quotes but I'm not sure this is the right thing to do.
We've opened an issue on the canonical test suite for the spec to request a clarification from the spec authors (and a test case to be added). Let's see what they say!
Additionally, the python library which has been developed by the authors of the spec, includes the reserved range from 3986 to be passed through. And that range differs from that library.
I'd avoid introducing another method in this library for custom behaviour as it's going to be detrimental for these reasons:

Augmented maintenance workload for you (more code, and custom behaviour)
Requires the library users to have a deep understanding of the spec, which makes the whole point of this lib moot : not having to.

Hi @yosida95
We've had a conclusive answer from the specification owners and they agreed this is a mistake in the spec.
I'm in the process of getting an errata through.
I've also submitted a PR to the canonical test cases
And lastly I've submitted a PR here to address the implementation, see #15.
Thanks for you help and please review it at your earliest convenience!

Thank you and your colleages for the clarification work! I'll review your PR soon.


Event advertisement IP config.

Sorry, I apparently missed this piece of the puzzle from #444.

Coverage decreased (-1.2%) to 58.968% when pulling 320aa2c3fa64b432044818984a233c3fa6151f9f on lwis:event_advertisement_ip_config into 2f4b57764fcf96715cb245406b87391af268924d on SoCo:master.

Coverage decreased (-0.03%) to 60.096% when pulling 30b662996ac63883e8194c231ff5849f5ef9bd45 on lwis:event_advertisement_ip_config into 2f4b57764fcf96715cb245406b87391af268924d on SoCo:master.

@lawrenceakka thoughts?

Sorry - haven't had time to test or think about this yet.

I'm not sure why you would ever want a different listener address (which is the address on which Soco listens for events) and the advertisement address (which is, as far as I can see, the address which Soco tells the Sonos devices it is using to listen for events)!

Does it actually work, especially if you different values for the two IP address config variables. Just looking at the code, it doesn't look to me like it will.

Also, it looks like you are failing some of the travis checks due to extra blank lines!

@lawrenceakka use case;
I'm in a Docker container, when it tries to work out it's IP it will get the container IP, which will ultimately send the container IP to the Sonos devices (which is only reachable locally on the machine).
This does work, I'll do some more testing and fix the failing builds. Thanks for getting back to me.

Should event_listener_ip default to event_advertisement_ip (so that if you have set the first, the second by default is the same)?

@lawrenceakka not exactly, as containers cannot bind to the host IP, however can listen on it (by proxy).

Coverage decreased (-0.03%) to 60.096% when pulling 0efdf9e6d88d058ffeae08f3a3c7722acc02dbb0 on lwis:event_advertisement_ip_config into 2f4b57764fcf96715cb245406b87391af268924d on SoCo:master.

Coverage decreased (-0.03%) to 60.096% when pulling 05090b81f938b8b1fbf8cc13fde17adb4ee9a153 on lwis:event_advertisement_ip_config into 2f4b57764fcf96715cb245406b87391af268924d on SoCo:master.

Coverage decreased (-0.3%) to 59.874% when pulling 96423050f54566daddbc0615d92f55c826d2032a on lwis:event_advertisement_ip_config into 2f4b57764fcf96715cb245406b87391af268924d on SoCo:master.

Coverage decreased (-0.3%) to 59.874% when pulling ee4b20eec6c48bf0de46401d925bc6e5df10a67c on lwis:event_advertisement_ip_config into 2f4b57764fcf96715cb245406b87391af268924d on SoCo:master.

Coverage decreased (-0.3%) to 59.874% when pulling b66dfc8129fc55136fa362fadd3e29e222309920 on lwis:event_advertisement_ip_config into 2f4b57764fcf96715cb245406b87391af268924d on SoCo:master.

Coverage decreased (-0.3%) to 59.874% when pulling b66dfc8129fc55136fa362fadd3e29e222309920 on lwis:event_advertisement_ip_config into 2f4b57764fcf96715cb245406b87391af268924d on SoCo:master.

Coverage decreased (-0.3%) to 59.874% when pulling 1244b7320e89f924299a748b1f4c2a3e0302cf2a on lwis:event_advertisement_ip_config into 2f4b57764fcf96715cb245406b87391af268924d on SoCo:master.

Coverage decreased (-0.3%) to 59.874% when pulling b66dfc8129fc55136fa362fadd3e29e222309920 on lwis:event_advertisement_ip_config into 2f4b57764fcf96715cb245406b87391af268924d on SoCo:master.

Coverage decreased (-0.3%) to 59.874% when pulling b66dfc8129fc55136fa362fadd3e29e222309920 on lwis:event_advertisement_ip_config into 2f4b57764fcf96715cb245406b87391af268924d on SoCo:master.

Coverage decreased (-0.3%) to 59.874% when pulling b66dfc8129fc55136fa362fadd3e29e222309920 on lwis:event_advertisement_ip_config into 2f4b57764fcf96715cb245406b87391af268924d on SoCo:master.

Can we resurrect this PR? I'm running into the same issue as @lwis with a docker container on a bridged network that can't use events because bind IP != advertise IP.

Unfortunately, we can't reopen this since the @lwis' branch does not exist anymore.
You can create your own branch, do the changes and submit a new PR.

@ghcs27, I have just created PR https://github.com/SoCo/SoCo/pull/604 for this


Update ORCv1.md: Fix documentation about bitset in bloom filter section

The corresponding code is https://github.com/apache/orc/blob/master/c%2B%2B/src/BloomFilter.cc#L47.
The code is correct, but the documentation is not. The code is to use index % 64 which has the same effect as index & 0x3f.
  constexpr uint64_t BITS_OF_LONG = 64;
  constexpr uint8_t  SHIFT_6_BITS = 6;
  void BitSet::set(uint64_t index) {
    mData[index >> SHIFT_6_BITS] |= (1ULL << (index % BITS_OF_LONG));
  }

The fixes on documentation are

There is no need to use >>>. We have already flip combinedHash if it's negative and position  sign bit will be always zero.
To set bit we have to mask position with 0x3f to keep the least 6 bits.

Thank you, @dirtysalt . Welcome to the Apache ORC community.

Let me revert this.

@dirtysalt . I reverted this because the original claim of this PR is wrong, The code is correct, but the documentation is not.. The document is consistent with the Java code. It's my bad to miss that. Sorry about the back and forth.

Oh. I never know that Java bit shift << is sort of like a circular bit shift.  That's to say 1L << 129 equals 1L << (129 % 64). So that explains why 1L << position works.
If I do that in C++, the compiler will give a warning, and the result is undefined.
/// c++ code
int main() {
    unsigned long long x = 0;
    x |= (1ULL << 129);
    printf("%llu\n", x);
    return 0;
}

/// shell execution
mac :: ~/Public » g++ test.cpp
test.cpp:14:16: warning: shift count >= width of type [-Wshift-count-overflow]
    x |= (1ULL << 129);
               ^  ~~~
1 warning generated.
mac :: ~/Public » ./a.out
4526632784
mac :: ~/Public » ./a.out
4345032528

But I do suggest modifying documentation as 1L << (position % 64) which is less confused to C++ guys.

@dirtysalt . I understand your reasoning. The document has been there since Apache ORC 1.0 for 6 years. We had better keep it AS-IS to avoid any unnecessary further confusions because we didn't change the logic.

https://github.com/apache/orc/blame/branch-1.0/site/_docs/spec-index.md#L105

@dongjoon-hyun Understood.


Automatically bump version with PyPI release?

Comment:
I just released 0.8.3 via GitHub, and it's now available on PyPI.
@rabernat, are you aware of a way to automatically bump
https://github.com/conda-forge/pangeo-forge-recipes-feedstock/blob/8d5f11800899524c5cdad1ab64c06cdbd3885a16/recipe/meta.yaml#L2
when we do this?
Or do we need to manually make a PR to this repo each time?

Oh, I was just a little too quick in opening this issue, looks like #4 takes care of this.


Layer links do not work

Links to view the layer on the NAvigator includes the domain twice. For example:
https://mitre-attack.github.io/attack-navigator/beta/enterprise/enterprise/#layerURL=

This is because the navigator instances in the config already specifies the domain URL, but the code still appends the domain suffix to the URL in the navigator macro:
https://github.com/mitre-attack/attack-website/blob/develop/attack-theme/templates/macros/navigator.html#L21
Removing the {{layer.domain}} suffix should fix the issue.
See also this commit for a manual patch fixing the built content: https://github.com/mitre-attack/attack-website/commit/36d72f6014aaa6f2635f879c6f942571fd4aa8f2

Reopening so that this fix can be added to master and sub-techniques versions

fixed in v1.3.1


Settings dialog not responding

This issue is probably resolved by https://github.com/ihiroky/live-comment/issues/42


New Unity Mediation (Beta)!

Hey, I'm using unity ads on my Flutter apps. Can you please provide me with SDK installation for Flutter?
Currently, I'm using this Fluter package on my all projects https://pub.dev/packages/unity_ads_plugin
Please provide an updated version...

does it support  https://pub.dev/packages/unity_ads_plugin  new unity mediation? I wannt try but I'm not sure about that.

unity_ads_plugin does not support new Unity Mediation yet

you can use it: https://github.com/minhdev2017/flutter_unity_mediation
note: not work with firebase_store, I must to replace it by firebase_database


SLDS IconPrefix does not match FontAwesome IconPrefix

Provide a general summary of the issue here
Icon prefix definition is different than in @fortawesome package. Fortawesome v6.6.0. Throws error in Icon.register function

🤔 Expected Behavior?
no errors using any version of FontAwesome
😯 Current Behavior
Throws error in Icon.register function
💁 Possible Solution
match the icon prefixes to the new list of prefixes from fontAwesome
🖥 Steps to Reproduce
Update the FontAwesome package to version 6.6.0 and use an icon somewhere
What browsers are you seeing the problem on? (only for bugs in code)
Other
If other, please specify.
No response
What operating system are you using?
irrelevant
🧢 Your product/team
SLDS
🎨 Your Theme(s)
Sanoma Learning

This PR should contain a solution already: https://github.com/sl-design-system/components/pull/1414


Plans for advanced LaTeX support?

Hi there!
I am also looking to a good solution to write Markdown + LaTeX (preferably, with custom macros and environments) on static blog generator, for now, I have been using a mixture of pandoc and MathJaX.
But I believe your idea could lead to something a lot more cleaner, do you think it'd be feasible [1] to add support for:

\def, \newcommand, and so on.
environments
local packages (amsthm, mathtools, etc…)

Thank you for your time and your project, that's awesome!
[1]: Meaning that anyone could contribute a PR to do this.

I love the idea. And yes, complete LaTeX support is definitely a thing I will be aiming for down the road.
The main hurdle for this, as I see it, is that as of this moment mistletoe only supports LaTeX elements when rendering to LaTeX. A current working example is LaTeX's inline math elements.
If I understand your proposal correctly, you seem to want LaTeX elements to be rendered into HTML. I also have this goal in mind, but doing so would involve some important design changes, of which I am not sure of the implications. Not saying that this can't be done -- I will probably open a branch and see what I can do -- but it certainly involves some careful planning.
Some random thoughts:

We should probably look into what mobiledoc has been doing, esp. with cards (rich block-level elements) and atoms (rick span-level elements).
This is probably the biggest design question, as it offers a unified interface for future custom elements.

mistletoe should learn to read from its own AST, so that we can store documents as JSON and retrieve it later (this one should be easy).
This is relevant because we need to figure out a way to store custom elements, e.g. LaTeX environments, macros and such.

We probably need some javascript to render LaTeX elements at some point, so the question is to design an API for javascript robust enough that makes the lives of future adventurers easier.
The easiest way to do this is probably just embed the necessary javascript into rendered HTML. Maybe we can get that working before coming up with a cleaner solution.

Once we somewhat figure out the previous points, I think we'll get a better idea of how we can work towards supporting the features you mentioned. I'll keep this issue open to see what others think of it.

Meaning that anyone could contribute a PR to do this

... by all means, please hack around, give a feature example and open a PR.
Thanks for this!

Well it's actually easier than I expected. Take a look at the mathjax module under plugins: it should provide some rudimentary support for MathJax's syntax.
If you want math enclosed in parenthesis or brackets, you need to escape it twice: \\(1 + 1 = 2\\). Similarly with \def or macros:


fix node-gyp rebuild failing with error: flock: ./Release/linker.lock: B...

here is my change to fix the flock error.  I also added the LD_LIBRARY_PATH export here.

Hi, thanks for the request and apologies for the delay. Could you send two different pull requests (one for the build path and another for the rabbit config)?

Now I see it is here https://github.com/ging/licode/pull/212 :)


Repo sync for protected CLA branch

The pull request is created from main638040475175734497sync_temp to main to fix git push error for protected CLA branch

Learn Build status updates of commit 32f50e2:
:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel


Custom dashboards

When I've shown our Prometheus functionality to people at conferences, they have asked about custom dashboards. Specifically, they've asked about:

using grafana
being able to provide their own data sources for grafana

The last case is from someone who is already running prometheus & grafana, but has their own extra proprietary data source that they use to add extra information to their graphs.

This ticket doesn't belong here: Cortex is the backend, frontends should be handled elsewhere.


86b1u14wh

Changes has been done as per the review . Now, mentor able to register students and it will be stored in DB.

Conflicts resolved


feat: serve aliases

Closes #873

I think it would make more sense to actually resolve those names. Having Trunk.toml carry that information, wouldn't make much sense on different machines I guess.
Something like this: https://github.com/hickory-dns/hickory-dns/blob/main/crates/resolver/README.md

I think it would make more sense to actually resolve those names. Having Trunk.toml carry that information, wouldn't make much sense on different machines I guess.
Something like this: https://github.com/hickory-dns/hickory-dns/blob/main/crates/resolver/README.md

My intention here is only to add an additional print statement to easily Ctrl+Click, as seen in the screenshot. DNS validation could also, in my opinion, be a future PR.
But to continue the conversation, i'm not sure how you want to use that. It doesn't seem able to resolve my /etc/host override?
10:40 simbleau on main ~ cat /etc/hosts

<IP_ADDRESS>       localhost
<IP_ADDRESS> broadcasthost
::1             localhost
<IP_ADDRESS>   localhost.loopystudios.com

Using hickory-dns:
10:40 simbleau on main ~ resolve localhost.loopystudios.com

Querying for localhost.loopystudios.com. A from udp:<IP_ADDRESS>:53, tcp:<IP_ADDRESS>:53, udp:<IP_ADDRESS>:53, tcp:<IP_ADDRESS>:53, udp:[2001:4860:4860::8888]:53, tcp:[2001:4860:4860::8888]:53, udp:[2001:4860:4860::8844]:53, tcp:[2001:4860:4860::8844]:53
NoRecordsFound for query localhost.loopystudios.com. IN A
        loopystudios.com. 1800 IN SOA chad.ns.cloudflare.com. dns.cloudflare.com.<PHONE_NUMBER> 10000 2400<PHONE_NUMBER>

I was expecting to see a record that reports localhost.loopystudios.com as <IP_ADDRESS>

IIRC you need to use: https://github.com/hickory-dns/hickory-dns/blob/main/crates/resolver/src/system_conf/mod.rs

Resolver::from_system_conf()

Are you interested in completing that as future work? It's not something I'm particularly interested in, just looking to log out the alias.

Thanks!!


minio: 2021-05-11T23-27-41Z -> 2021-05-16T05-32-34Z

Motivation for this change

fix license change
set version variables with ldflags
Bump minio to 2021-05-16T05-32-34Z

Build and NixOS test passed
λ ./result/bin/minio -v
minio version RELEASE.2021-05-16T05-32-34Z
λ ./result/bin/minio --help
NAME:
  minio - High Performance Object Storage

DESCRIPTION:
  Build high performance data infrastructure for machine learning, analytics and application data workloads with MinIO

USAGE:
  minio [FLAGS] COMMAND [ARGS...]

COMMANDS:
  server   start object storage server
  gateway  start object storage gateway

FLAGS:
  --certs-dir value, -S value  path to certs directory (default: "/home/jk/.minio/certs")
  --quiet                      disable startup information
  --anonymous                  hide sensitive information from logging
  --json                       output server logs and startup information in json format
  --help, -h                   show help
  --version, -v                print the version

VERSION:
  RELEASE.2021-05-16T05-32-34Z

Things done

[X] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[X] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[X] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[X] Ensured that relevant documentation is up to date
[X] Fits CONTRIBUTING.md.

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

1 package built:

minio

Also as a note src.rev is not a commit sha, it's just whatever: https://github.com/NixOS/nixpkgs/blob/3fbdcefe1d133f9a1d8a870bbd79eacb6ca6dc05/pkgs/servers/minio/default.nix#L23 is set to. But that's what it was set to before these changes so I've kept it as it was


Swap Calculation Function to support custom logic when resolving a swap trade

Allows client to inject more complicated logic for converting SwapTrade to ResolvedSwapTrade.
Default behaviour is unchanged.

No longer required


Get rid of converting selector to css, stop sending JS commands to get attributes

Change list

Some methods which haven't defined by W3C spec convert selector from them to css selector

In this PR, I've changed to use id, name, class name and tag name directly.

GetAttribute calls JS-command instead of calling DriverCommand.GetElementAttribute in W3C spec. In this PR, I've changed to call the command directly.

related selenium client code: https://github.com/SeleniumHQ/selenium/blob/25d2e542d39768816a83931a4f43e4a006c9e78d/dotnet/src/webdriver/Remote/RemoteWebElement.cs

closes https://github.com/appium/appium-dotnet-driver/issues/266 and https://github.com/appium/appium-dotnet-driver/issues/265
Types of changes
What types of changes are you proposing/introducing to .NET client?
Put an x in the boxes that apply

[x] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Details
Please provide more details about changes if it is necessary. If there are new features you can provide code samples which show the way they
work and possible use cases. Also you can create gists with pasted C# code samples or put them here using markdown.
About markdown please read Mastering markdown and Writing on GitHub

@KazuCocoa I had a branch for this. Will push this one at some point today

@KazuCocoa Think this should be it. I'll reformat the test project and add the tests for these in another PR.

thanks!

Thanks a lot for tackling this!


chore: update dependencies

Summary
Update dependencies
Ticket Link
NONE

@enahum I have no permissions to add reviewers to this PR

I'll publish it shortly


Error deploying open AI: Specified capacity of account deployment is bigger than available capacity for UsageName Tokens Per Minute (Thousands) ext-Davinci-003

Please provide us with the following information:

This issue is for a: (mark with an x)
- [X ] bug report -> please search issues before submitting
- [ ] feature request
- [ ] documentation issue or request
- [ ] regression (a behavior that used to work and stopped in a new release)

Minimal steps to reproduce
I ran azd up after initializing the project, however the openai deployment failed and I got the error message described. The other components deployed without errors. I'm not sure exactly what's causing the error so I'm not sure how to reproduce it.
Any log messages given by the failure
The specified capacity '60' of account deployment is bigger than available capacity '0' for UsageName 'Tokens Per Minute (thousands) - Text-Davinci-003'.
Expected/desired behavior
Expected behavior is for openai to be deployed without errors.
OS and Version?
Windows 10 Enterprise (OS Build 19044.2965)
azd version?
azd version 1.0.1 (commit e0cd1aca716fa5d08704beade7dcc734fe68f5f1)
Mention any other details that might be useful

I ran into this as well. It turns out that Azure OpenAI changed their quota system last week, and hasn't yet updated the Bicep schema (the infrastructure-as-code language used for this repo and others) to reflect the new quota system. For now, I've been manually resetting the quota of my deployments before each "azd up" by going to the Azure OpenAI studio, selecting the Quotas tab, and sliding the TPM to 1K for each of them.

same here https://github.com/Azure-Samples/azure-search-openai-demo/issues/307

I'm completely new to Azure.
How do you manually reset the quota of your deployment, when you havent deployed it?
In my Azure OpenAI studio, there is no deployment visible, before and after deploying this demo with "azd up".
I only see the deployment in portal.azure.com.
As I already mentioned, I'm new to Azure / Azure OpenAI and I really don't understand where to look.
I can set an environment-name for the deployment, which creates a ressource-group, but that's all I understand.
I can also see the quota TPMs per region and subscription, which currently sits at 120/120.
Since I have no deployment running in East US and it still sits at 120/120, I wonder how I can determine which deployment currently takes up all quotas?
For West Europe I can see the deployments, taking up the TPMs, but for East US I can't.
Can it be that my account doesn't have sufficent permissions? My company created it for tests.

I'm completely new to Azure.
How do you manually reset the quota of your deployment, when you havent deployed it? In my Azure OpenAI studio, there is no deployment visible, before and after deploying this demo with "azd up".
I only see the deployment in portal.azure.com.
As I already mentioned, I'm new to Azure / Azure OpenAI and I really don't understand where to look. I can set an environment-name for the deployment, which creates a ressource-group, but that's all I understand. I can also see the quota TPMs per region and subscription, which currently sits at 120/120. Since I have no deployment running in East US and it still sits at 120/120, I wonder how I can determine which deployment currently takes up all quotas? For West Europe I can see the deployments, taking up the TPMs, but for East US I can't. Can it be that my account doesn't have sufficent permissions? My company created it for tests.

@fakoe, temporary solution is to go to Azure AI studio, Deployments, and for each Edit deployment, Advanced, set "Tokens per Minute Rate Limit" from 120 to e.g. 2. Then deploy your accelerator with "azd up". Also open the project it in VSCode, it will suggest you cool extensions

@fakoe, temporary solution is to go to Azure AI studio, Deployments, and for each Edit deployment, Advanced, set "Tokens per Minute Rate Limit" from 120 to e.g. 2. Then deploy your accelerator with "azd up". Also open the project it in VSCode, it will suggest you cool extensions

I understand where to set it, but the problem is, that I don't see the deployment.
I followed all installation steps in this demo and that's it. They don't tell you how to setup a deployment that you can see in Azure AI studio. I got an account from my company, where they told me, that it has a ChatGPT subscription.
I deployed a demo without problems last week.
Now, I wanted to create another and I run into the described error.
If I lookup the currently available deployments in Azure AI studio, I only see a deployment, that's not mine, nor I ever referenced or used it.
The demo references using existing sources. I never did that, I just ran azd up, selected the subscription and the correct region and that's it.
How can I configure or edit a deployment, if I haven't created it with azd up?
Do I have to create a new deployment for, let's say text-davinci-003 (which seems to be the default) and explicitly reference it through AZURE_OPENAI_CHATGPT_DEPLOYMENT or AZURE_OPENAI_GPT_DEPLOYMENT with azd up?
Which makes me wonder how the demo has worked without that so far (until last week).
To my understanding, and all I can track from that demo, when I run azd up is the general deployment of all sources. Only the openapi source fails there with the described error from above. But when I want to check this ressource, it says it's not even created. It just gives me that error message and that's it. I don't know how I should give that particular source TPM, when it's not even created due to insufficient TPM :/

It looks like https://github.com/Azure-Samples/azure-search-openai-demo/pull/322 might address this, I'll be testing it out shortly.

Just merged #322 which addresses this. Closing this, but feel free to reopen if some scenario comes up where the change is not effective.

This Problem Persists
I Deployed this about 24 hours ago and it was working with your previous deployment files, but now for some reason it wont work, and I still get the same error with your latest changes.
This what I ran into this morning:

I pulled your latest build, and this was the error in the build process:

Very similiar error.
I'm Deploying to US East, but I'm closest to US West, I don't know if if that matters.
I would also like to see a bicep deployment of this at the target scope of a resource group. Often larger companies don't actually deploy things at the subscription level, and reserve those permissions for administration. It would make a lot more sense for these to target a specific resource group (Although I'm aware support for the azure developer CLI at a resource group scope is in alpha).
Targeting a subscription level deployment, limits the sqift implementation of this example to individuals, small startups, and very high level system administrators at larger companies.

open ai service is "soft deleted" if you have one or a few you've deleted you may need to purge them to release quota

I'm facing the same issue with terraform azurerm provider.
Latest update is 8 days ago, and the doc doesn't reference the new attribute "capacity" that #322 added to bicep
Any chance to get an update to azurerm provider for people using terraform ?

open ai service is "soft deleted" if you have one or a few you've deleted you may need to purge them to release quota

Purge solved it for me. tnx

Thanks for tip. That worked!

Hi,
Thank you for your response. Purge worked.
kr

From: Orion Sehn @.>
Sent: Thursday, June 22, 2023 5:47 PM
To: Azure-Samples/azure-search-openai-demo @.>
Cc: Gregor Cvek @.>; Comment @.>
Subject: Re: [Azure-Samples/azure-search-openai-demo] Error deploying open AI: Specified capacity of account deployment is bigger than available capacity for UsageName Tokens Per Minute (Thousands) ext-Davinci-003 (Issue #313)
Caution: This email is from an EXTERNAL SENDER and may be malicious. Do NOT Click any links or Open any attachments if you were not expecting them.
open ai service is "soft deleted" if you have one or a few you've deleted you may need to purge them to release quota
Thank you very much, this resolved my issue as well. Annoying that the error message was exactly the same as the issue resolved in this thread.
—
Reply to this email directly, view it on GitHubhttps://github.com/Azure-Samples/azure-search-openai-demo/issues/313#issuecomment-1602881414, or unsubscribehttps://github.com/notifications/unsubscribe-auth/BAXKQ5MRPFPSVVEYZZV5D7LXMRSHTANCNFSM6AAAAAAZGXN2EQ.
You are receiving this because you commented.Message ID: @.***>

the issue is still here. But purging fixes the issue. How to purge , see this : https://learn.microsoft.com/en-us/azure/cognitive-services/manage-resources?tabs=azure-portal

THIS is what I searched for!
If you stop the deployment, while its running, or if you have a successful deployment and then re-deploy and something goes wrong, I couldn't find the previously built ressources and with "azd down --purge" it just was able to capture the current deployment and not the previous one.
Therefore the old deployments, even if they weren't active, blocked the available quotas.
To me, the problem with the quotas doesn't seem to be a bug of the demo, but a general annoyance of how Azure and Cognitive-Search keep allocating resources to soft-deleted deployments.
It's my first time using Azure and the Cognitive-Service and I just didn't have the knowledge to understand what was happening.
Thanks for the clearification!

Purging doesn't fix the issue at all. It frees some quota if you have soft deleted resources, which in turn makes deployment possible because you suddenly have quota available.
This issue is about being able to define a "capacity" when using bicep, which was implemented a few days ago and allows bicep users to deploy openAI service with a specific capacity instead of using the "default Microsoft policy" defining quotas. It has very little to do with soft deleted openAI services.

well, for me, using the fix was not enough.

Purging doesn't fix the issue at all. It frees some quota if you have soft deleted resources, which in turn makes deployment possible because you suddenly have quota available.
This issue is about being able to define a "capacity" when using bicep, which was implemented a few days ago and allows bicep users to deploy openAI service with a specific capacity instead of using the "default Microsoft policy" defining quotas. It has very little to do with soft deleted openAI services.

Well, if soft-deleted resources take the maximum amount of quotas, you still won't be able to re-deploy the demo, even with definable tpm...

Reducing quota actually helped, thanks a lot.

im having this same error:
Deployment Error Details:
InvalidTemplateDeployment: The template deployment 'openai' is not valid according to the validation procedure. The tracking id is '14df4aaf-343b-402a-a57f-ff468cfffea1'. See inner errors for details.
InsufficientQuota: The specified capacity '1' of account deployment is bigger than available capacity '0' for UsageName 'Tokens Per Minute (thousands) - Text-Davinci-003'.
There are no openai services to purge!

@pablocastro @pamelafox
It seems #322 doesn't resolve the issue. I did run a few minutes ago and received below error.
ERROR: deployment failed: failing invoking action 'provision', error deploying infrastructure: deploying to subscription:
Deployment Error Details:
InvalidTemplateDeployment: The template deployment 'openai' is not valid according to the validation procedure. The tracking id
is .......'. See inner errors for details.
InsufficientQuota: The specified capacity '1' of account deployment is bigger than available capacity '0' for UsageName 'Tokens
Per Minute (thousands) - Text-Davinci-003'.

azd up updates the capacity to 120k, so regardless if updating the deployment quota to 1k prior to azd up, deployment fails with error - InsufficientQuota: The specified capacity '120' of account deployment is bigger than available capacity '99' for UsageName 'Tokens Per Minute (thousands) - GPT-35-Turbo'.

Yea .. I am getting the same issue. There was nothing to purge using the Cog Serv Anomaly Detector.
I have been having issues since the suite of deployment models was greatly reduced within the past week or so. I've not been successful at implementing any of the 3 new ones.  GPT-35-Turbo, GPT-35-Turbo-16k, and Text-Embedding-Ada-002.
{
  "code": "InvalidTemplateDeployment",
  "message": "The template deployment 'openai' is not valid according to the validation procedure. The tracking id is '3d1c4208-e1f1-4c96-98cd-8722a488465d'. See inner errors for details.",
  "details": [
    {
      "code": "InsufficientQuota",
      "message": "The specified capacity '120' of account deployment is bigger than available capacity '118' for UsageName 'Tokens Per Minute (thousands) - GPT-35-Turbo'."
    }
  ]
}

@TroyHostetter Did you update your Bicep to change the deployment to 30? That message indicates its still the default of 120, I think. Here's what it looks like now:
https://github.com/Azure-Samples/azure-search-openai-demo/blob/0cc155b2c8ce788be7b0916f392d8f531a71f6d9/infra/main.bicep#L145
You could diff your main.bicep with the current main.bicep using a tool like diffchecker.com

I can confirm that the update @pamelafox mentioned did resolve that particular issue, and it does deploy correctly for me.
Somewhat unrelated but is there any chance we'll get a deployment pipeline at a resource-group target scope instead of the subscription level? Many large companies don't give out subscription level permissions across the company, and a resource-group targetScope would help get the deployed faster. (I know this azd feature is in alpha).

@OrionSehn Please subscribe to this issue in the azd repo, as that seems to be what you're looking for: https://github.com/Azure/azure-dev/issues/337 Thanks for the feedback!

@TroyHostetter Did you update your Bicep to change the deployment to 30? That message indicates its still the default of 120, I think. Here's what it looks like now:
https://github.com/Azure-Samples/azure-search-openai-demo/blob/0cc155b2c8ce788be7b0916f392d8f531a71f6d9/infra/main.bicep#L145
You could diff your main.bicep with the current main.bicep using a tool like diffchecker.com

That did the trick, with a few other "diff" changes. Thanks for the help! Last question, is there a place for general questions about the openai-demo repo?

@pamelafox , It works for me as well after making changes in main and cognitiveservices bicep. my changes are way ahead, will do a merge this weekend

I ran into this as well. It turns out that Azure OpenAI changed their quota system last week, and hasn't yet updated the Bicep schema (the infrastructure-as-code language used for this repo and others) to reflect the new quota system. For now, I've been manually resetting the quota of my deployments before each "azd up" by going to the Azure OpenAI studio, selecting the Quotas tab, and sliding the TPM to 1K for each of them.

I have faced the same issue and the deployment worked when the token rates were manually set to 1k from the portal side. Doesnt this increase the deployment time drastically ? Is there any other way to improve the token rate ? Like using a number higher than 1k that would be faster and wont cause issues? Is there any way of changing this parameter in the bicep file of the repo. I tried changing the param chatGptDeploymentCapacity int = 30 to 10, in the main.bicep file and other numbers but it hasnt made any difference.

follow this to resolve the issue

Thanks . I knew this , also sometimes you need to do via cmd as well .
The issue was not resolved by this.
On Wed, 2 Aug 2023 at 10:58 pm, Rajashekhar A @.***>
wrote:

[image: MicrosoftTeams-image]
https://user-images.githubusercontent.com/29478647/257819867-423e8526-ed89-483a-9842-9f51437bec53.png
follow this to resolve the issue
—
Reply to this email directly, view it on GitHub
https://github.com/Azure-Samples/azure-search-openai-demo/issues/313#issuecomment-1662167558,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AZUBBTPNSK44OLMVNBX6IVTXTJFIFANCNFSM6AAAAAAZGXN2EQ
.
You are receiving this because you commented.Message ID:
@.***>

If anyone is still experiencing this and think its an error, here is a related issue: https://github.com/Azure/bicep-types-az/issues/1660

Purging Azure OpenAI resources fixes the issue for me ;)


Updates-style-guide

Signed-off-by: natebower<EMAIL_ADDRESS>Description
Updates style guide
Checklist

[ ] By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license and subject to the Developers Certificate of Origin.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@kolchfa-aws Can you give me a sanity check and let me know if you think there's anything else I should add?


Firebase Authentication : Unlink user account

After lookng at all the current open issues and since this is not implemented I am raising this issue for the firebase_auth plugin: https://github.com/flutter/plugins/tree/master/packages/firebase_auth
The functionality exists as part of firebase:
https://firebase.google.com/docs/auth/android/account-linking#unlink-an-auth-provider-from-a-user-account
Raised against the plugin: firebase_auth up to 0.5.8

@aqwert
This issue has been moved to https://github.com/FirebaseExtended/flutterfire/issues/834. Any further collaboration will be done there.


Question about the shape of prototype

Hi! In the code the prototype is initialized as torch.zeros(num_classes, num_queries, 1, 256), and I don't know why there is an extra shape of num_queries. I think the prototype should be set for each class, so it should be torch.zeros(num_classes, 1, 256), but I may make some misunderstanding. Could you explain it? Thanks a lot!

Each query from the same class shares the same prototype. We simply repeat them.


FTW:Log Suggestion - Suggestion for saving old log problem.

This PR is an addition for comment.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: zzy-supplient sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.


Add single client fixture

This PR introduces a new fixture called client that returns a single client instance rather than multiple of them. This solves the issue of many of the tests using clients but only requiring one client and needing to use the same for block to remedy for it.
Tests that use one client now use this fixture and have had the associated for statements removed. clients has been renamed to clis to prevent confusion, and the tests still using this fixture have been modified accordingly. Some tests require a host_id despite using one client, and since client doesn't return a host_id, those tests will still be using clis.
This PR resolves #75.

Don't rename clients to clis. Just keep it as clients. clients cannot be iterated so it should be easy enough to catch the error usages. cli also means command line-interface, so don't want to get confused with that.
Otherwise LGTM.

That's quick!
Merged, thanks!


Extension upgrade manual for agents

1. Abstract
Time to time we use to release a new version of our Chrome Extension.
Such a release may include new features, fix patches or both.
Releases with new features increase the mayor number of the version (1.x, 2.x, 3.x, etc.)
Releases with fix patches increses the minor number of the version (x.1, x.2,  x.3, etc.).
2. Removing Current Extension from Your Browser

3. Finding New Releases

Access your scraper dashboard here:
https://connectionsphere.com/scraper/dashboard?service=scraper

Find the latest release in the "Downloads" section.

4. Installing New Release in Your Browser

Done.
---> https://connectionsphere.com/seminars/scraper/upgrading


feat(theme): support 'raw' theme that doesn't include base tokens

Purpose
Enables efficiently including several themes by only emitting base tokens once.
When only 'day' and 'night' are used the difference is negligible.
However it scales much better for apps that require multiple themes.
The feature is opt-in, the default is still emitting all tokens.
Approach
Refactor Sass theme mixins by combining {theme} and {theme}-class.
When 'class' is specified as an argument, it will put all tokens inside the theme's appropriate class selector.
Otherwise it will use & which is essentially the same as not nesting inside a selector.
Then add the option of being 'raw'.
When 'raw' is specified as an argument, base tokens are not included in the output of the mixin.
Testing

Built styles (CSS) locally, output remains the same.
Tested 'raw' locally, does not output base tokens, only theme contextual/alias variables.

Risks
BREAKING CHANGE: {theme}-class mixins removed in favour of {theme}('class')
See README.md changes for more details.

I believe scope here should be core instead of theme as per documentation?

I believe scope here should be core instead of theme as per documentation?

Only if we choose to use package names as scope, like in @commitlint/config-lerna-scopes.
I don't mind doing that, even if not using lerna-scopes, so I'll amend the commit.


PIP was not working

Pip was not working from VSCode

Ref : https://stackoverflow.com/questions/37220055/pip-fatal-error-in-launcher-unable-to-create-process-using


Provide TextTerminal.printf

The example shows the output with String concatenation
TextTerminal terminal = textIO.getTextTerminal();
terminal.println("\nUser " + user + " is " + age + " years old, " +
"was born in " + month + " and has the password " + password + ".");
It would be easier to read if there is a printf method
TextTerminal terminal = textIO.getTextTerminal();
terminal.printf(
"\nUser %s is %s years old, was born in %s and has the password %s.",
user, age, month, password
);

Implemented in 1.6.0.


"Peer-to-peer (P2P) networking" page of Cardano Docs has out of date information

In the "Peer-to-peer (P2P) networking" page of the Cardano docs it states, "Note: The P2P deployment is currently in testing stages. The team is focused on the delivery of final features for SPOs and the preparation of QuickCheck simulation tests. The full P2P deployment will happen later in 2021. This documentation describes the P2P functionality for information purposes only ahead of this release." I believe this is out of date but do not wan't to create a pull request because this is not my area of expertise.

I'm updating this issue because I found new information. On the "Cardano entropy" page it states, "The Cardano blockchain has transitioned from a federated system run only by 8 nodes in July 2020, to its present state where block production is fully decentralized. The final part of this transition happened on March 31st, 2021 (D=0 Day), a major milestone in Cardano's journey. As of that date, 100% of block production is the responsibility of stake pool operators (SPOs)."

Thanks for raising this issue @amichae2 !
We will update the P2P entry. Currently, the P2P testnet is being actively tested by SPOs to be launched on mainnet soon. This work is ongoing in line with preparation for the Vasil hard fork.
As per the D parameter, this is a bit different topic. D=0, and this means that block production is fully decentralized. No single IOG node signs or validates transactions - this is in the hands of the community.
P2P on the other hand allows nodes in the network to communicate with each other directly, and establish connections without relying on the topology file, or the connection with the IO relay node. This is not about the D parameter, but about enhanced and automated connections.
Hope this clarifies some details.
Kind regards


Allow setting of command and args on elastalert deployment

What this PR does / why we need it: Allowing for custom commands and args will give more flexibility to the helm chart. :smile:
I hope to use it with https://github.com/jertel/elastalert-docker/pull/1 but it should be pretty generically useful.
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
Special notes for your reviewer:

@rjkernick bumped minor version, rebased on master
thanks for feedback :smile:

/lgtm

@rjkernick I rebased on master to remove a conflict in Chart.yaml

/assign @prydonius
@prydonius the CI robot suggested you :smile:

cc @jertel @pickledrick

/lgtm

@pickledrick thoughts?

@unguiculus @rjkernick @jertel
Is there someone else I should tag on this?

Not that I'm aware of. The original chart author, pickledrick, hasn't chimed in on the past several PRs.

/lgtm


Listener to check if ad loaded or failed

Do we have a call back or listener for ad loaded and ad failed? Please post the snippet if it is there i the library.

@Sahana312  Uh...I guess there's no any straightforward way for now. BUT you can try to do something like this
adapterWrapper = new AdmobExpressRecyclerAdapterWrapper(this, testDevicesIds){

            @Override
            protected void addAdViewToWrapper(@NonNull ViewGroup wrapper, @NotNull NativeExpressAdView ad) {
                super.addAdViewToWrapper(wrapper, ad);
                AdListener adListener = ad.getAdListener ();
                // and see https://developers.google.com/android/reference/com/google/android/gms/ads/AdListener to get what you can do with :)
            }
};