Yosys synthesis error

Hi,
While running synthesis with Yosys, it is throwing error from the file "ibex_cs_registers.v"

Executing Verilog-2005 frontend: syn_out/ibex_25_01_2021_15_08_46/generated/ibex_cs_registers.v
syn_out/ibex_25_01_2021_15_08_46/generated/ibex_cs_registers.v:538: ERROR: Non-constant function call in constant expression.
Is there any way to fix this

As usual, can you give the exact command that you're running? I know that @GregAC has been doing some Yosys-based synthesis recently, so I'd expect this to work.

As usual, can you give the exact command that you're running? I know that @GregAC has been doing some Yosys-based synthesis recently, so I'd expect this to work.

There is a small issue with synthesis on current master which I fix here: https://github.com/lowRISC/ibex/pull/1255 however I don't think this is the same problem.
With the fix from that PR I can successfully run synthesis on my machine. What versions of sv2v and yosys are you running? yosys v0.9 should work and I am currently using sv2v version: v0.0.5-18-ga170536

There is a small issue with synthesis on current master which I fix here: https://github.com/lowRISC/ibex/pull/1255 however I don't think this is the same problem.
With the fix from that PR I can successfully run synthesis on my machine. What versions of sv2v and yosys are you running? yosys v0.9 should work and I am currently using sv2v version: v0.0.5-18-ga170536

@GregAC
I am using Yosys version Yosys 0.9+3858 (git sha1 410ea422, clang 3.4.2 -fPIC -Os).
How can I check sv2v version? I suspect the error is from sv2v

@GregAC
I am using Yosys version Yosys 0.9+3858 (git sha1 410ea422, clang 3.4.2 -fPIC -Os).
How can I check sv2v version? I suspect the error is from sv2v

@GregAC
I am using yosys 0.9 and sv2v v0.0.6-9-gb28a3ca?
Can this be the cause?

@GregAC
I am using yosys 0.9 and sv2v v0.0.6-9-gb28a3ca?
Can this be the cause?

It seems likely it's an sv2v issue (or potentially a yosys issue, sv2v emitting correct verilog that yosys isn't accepting). I would suggest trying another version of sv2v, e.g. the 0.0.5 tag https://github.com/zachjs/sv2v/releases/tag/v0.0.5

It seems likely it's an sv2v issue (or potentially a yosys issue, sv2v emitting correct verilog that yosys isn't accepting). I would suggest trying another version of sv2v, e.g. the 0.0.5 tag https://github.com/zachjs/sv2v/releases/tag/v0.0.5

@GregAC
I cloed the specific tag v0.0.5 version of sv2v from git hub. But still the same error is coming.
The commands I used for sv2v  make is as below:
  git clone https://github.com/zachjs/sv2v.git
 cd sv2v
 git checkout tags/v0.0.5
 make

Any hint on this, after this the version of sv2v is showing as sv2v v0.0.5, will it be a yosys issue? But we both are using same version of yosys right? In the below image you can see 10 th one is throwing error.

@GregAC
I cloed the specific tag v0.0.5 version of sv2v from git hub. But still the same error is coming.
The commands I used for sv2v  make is as below:
  git clone https://github.com/zachjs/sv2v.git
 cd sv2v
 git checkout tags/v0.0.5
 make

Any hint on this, after this the version of sv2v is showing as sv2v v0.0.5, will it be a yosys issue? But we both are using same version of yosys right? In the below image you can see 10 th one is throwing error.

My Yosys full version is,
Yosys 0.9+3858 (git sha1 410ea422, clang 3.4.2 -fPIC -Os)

My Yosys full version is,
Yosys 0.9+3858 (git sha1 410ea422, clang 3.4.2 -fPIC -Os)

Looks like this is caused by the Yosys version. I've just reproduced what you're seeing and will have a proper look at what's going on later today.

Looks like this is caused by the Yosys version. I've just reproduced what you're seeing and will have a proper look at what's going on later today.

@rswarbrick
Thanks for the update. Basically what I understood is that Yosys added a new syntheisable construct in their latest update which is causing the issue.

@rswarbrick
Thanks for the update. Basically what I understood is that Yosys added a new syntheisable construct in their latest update which is causing the issue.

Bisecting Yosys builds points at commit 2085d9a55 (committed on 1st Jan 2021) being the first version that fails. I'll try to understand what's going on and then follow up on the Yosys bug tracker.

Bisecting Yosys builds points at commit 2085d9a55 (committed on 1st Jan 2021) being the first version that fails. I'll try to understand what's going on and then follow up on the Yosys bug tracker.

Right, looks like the Yosys guys are working on a fix. I don't think there's anything else to do here (the next Yosys release will probably work again).

Right, looks like the Yosys guys are working on a fix. I don't think there's anything else to do here (the next Yosys release will probably work again).


[Awake] localization

Summary of the Pull Request
Adding resource file. Replacing hard coded strings by localized strings.

PR Checklist

[x] Closes: #11456
[ ] Communication: I've discussed this with core contributors already. If work hasn't been agreed, this work might be rejected
[ ] Tests: Added/updated and all pass
[ ] Localization: All end user facing strings can be localized
[ ] Dev docs: Added/updated
[ ] New binaries: Added on the required places

[ ] JSON for signing for new binaries
[ ] WXS for installer for new binaries and localization folder
[ ] YML for CI pipeline for new test projects
[ ] YML for signed pipeline

[ ] Documentation updated: If checked, please file a pull request on our docs repo and link it here: #xxx

Detailed Description of the Pull Request / Additional comments

Validation Steps Performed
Tested locally

/azp run

Thank you for opening the PR, @donlaci! This is great to see.
Curious - any reason why the shared resource file (where we use strings for the UI) can't be used here too? Feels like there is already one location where everything sits, do we really need a separate Awake-specific resource?

Thank you for opening the PR, @donlaci! This is great to see.
Curious - any reason why the shared resource file (where we use strings for the UI) can't be used here too? Feels like there is already one location where everything sits, do we really need a separate Awake-specific resource?

Hi @dend.
You are refering to the resource file of the settings app, I suppose. It is better to have a separate tiny resource file for the Awake application, that way the app is slightly quicker, easier. Thanks!


Multiple Projects in a Single Seed

It appears that gulp-typescript requires (as of v2.13.6) a new TS Project file created for each running build task, as opposed to the intent which is for the TS Project file to be reused to reduce compilation time ( https://github.com/ivogabe/gulp-typescript/issues/307#issuecomment-226237437 ). For most users, that means the current setup of angular2-seed is perfectly fine, as they're only building and rebuilding a single application. Unfortunately, if you are building and rebuilding a complex suite of applications with a single seed (or an Angular2 library and a demo app using the library), this becomes an issue very fast.
I found a workaround by disabling gulp-cached in the build steps and moving the TS Project creation into the task itself ( https://github.com/ivogabe/gulp-typescript/issues/342#issuecomment-235044823 ).
There's a few things to unpack here:

For the angular2-seed, is there any intention of having "out of the box" support for multiple projects in a single seed? I feel like this is a non-trivial decision, but I think having support for multiple projects in a single seed is a very valuable and useful feature.
By utilizing gulp-cached in the TypeScript compilation task for caching TS Project, we're triggering an error in which gulp-typescript can't recompile changes to the watched files, thus making incremental builds problematic. Note: I have seen reports of this problem affecting non-complex seed project builds as well, but they're seem less likely to occur. I love gulp-cached but I think it would be valuable to identify somewhere to the users of the angular2-seed what files get cached and why.
Is there any value in adding some sort of configuration value like TSPROJECT_CACHING = argv['tsprojcache'] || true; where we can allow the user at runtime or in their project config to override the use of TSProject Caching but leaving in place the current caching for existing seed users? I'd be happy to put together a pull request to implement this if people think it would be valuable.

Author of gulp-typescript here. I would advise to not use gulp-cached with gulp-typescript. TypeScript already has incremental compilation, which is supported by gulp-typescript too. gulp-cached can actually prevent this incremental compilation, and it can be dangerous in certain situations. The emit of an unchanged file can change when some other file does change, and type checking of files depends on other files too. So, I'd advise to call createProject outside of the compilation tasks (create one for each task) and drop gulp-cached for TypeScript compilation. If this is too slow, you can use isolatedModules to get very fast and safe (incremental) compilation, though I'm not sure whether that works with Angular and decorator metadata.

@ivogabe: Thanks for the input!

Hey @mko! This issue aims to solve similar problem https://github.com/mgechev/angular2-seed/issues/913.
@d3viant0ne and @NathanWalker were working on it but experienced some issues with the SystemJS configuration. AFAIK @NathanWalker builds multiple projects for multiple platforms in his advanced seed.
It makes sense to drop gulp-cached as @ivogabe mentioned. Here are other strategies for speeding the build up https://github.com/mgechev/angular2-seed/wiki/Speeding-the-build-up.

Closing it due to lack of activity.


DateTime issues on Heroku

I've encountered a really strange issue with my installation of Rails Admin on Heroku and can't seem to figure it out– wanted to see if anyone had any tips/smoking guns that I should be focusing on. In particular, whenever I change a date with the provided DateTime forms from Rails Admin and submit the form, it removes a space between the Date and Time, like this: "ends_at"=>"July 17, 201523:55" (note the removal of a space between 2015 and 23:55) and gives an argument out of range error.
If I don't change any datetime fields and make changes on the other (non datetime) fields on my model, the form submits fine and there is no issue. In fact, the date is submitted correctly formatted with the form as: "ends_at"=>"July 17, 2015 23:55". It is a very strange bug and does not happen locally in my environment.

Updated gems and this was resolved. Must have been a conflict with another dependency.

Looks like this is still occurring actually. Reopening in case anyone else is dealing with the same issue.

DateTime handling was reworked by https://github.com/sferik/rails_admin/commit/01e8d5fc8ec94e68af6fdbd80759a751cd83f74a, this should not happen now.


Hilt issue / Build Exception

The Hilt Android Gradle plugin is applied but no com.google.dagger:hilt-android dependency was found.
Apparently its reported before ..tried to fix it but so far no luck.
https://github.com/google/dagger/issues/2661

Any luck?

none.. Tried diff versions.. Applied
const val hiltCoreBase = "com.google.dagger:dagger:${Versions.hilt}"
const val hiltCoreCompiler = "com.google.dagger:dagger-compiler:${Versions.hilt}"

Still no luck. This project uses Plugin DSL..no idea if thats the issue..

@ali-digitify comment this line id(Plugins.DAGGER_HILT) in data module's build.gradle once & sync once. Its should build successful. then Uncomment it & sync again.
Also dont forget to have pex_api_access_key="2xxxxxxxxxxxxxxxxxxxxxxx"


Bubble up 404 from SamePage API

When calling the /query POST API with an invalid notebook target, the SamePage API returns a 404. This is correct, except the /query route in this API then returns a 500 up to the frontend. We should forward that error back up to the client so that we could display the correct error message

@ellipses-dev


‼❎ More menu always open (Version <IP_ADDRESS>2 is broken)

Hi,
I have been using refined-github for a long time on Firefox (beta update channel) without problems. A few days ago (I think) I noticed that the GitHub UI had changed. This turned out to be because RG has stopped modifying it (and the plain GH looks "weird'). For example the CI status has vanished.

I am not sure what changed but it seems like basic features that should be there have stopped working. There are errors in my JS console when I visit this page:

I think this is the source of the problem, but I am not sure how to further  debug it. If I hover over the "injectGlobalHook.js" link in the console it links me to moz-extension://c22f98a5-2aa6-4346-9f98-f4bc11b4aa78/build/injectGlobalHook.js I am guessing this is RG but if you know how to match the UUID to an installed extension that would be great so I can confirm it.
I am running Firefox 73.0 (64-bit) from the beta updates channel on OSX 10.14.6 (18G103). I think the only other noteworthy extension I have installed is Privacy Badger. On version <IP_ADDRESS>2 of refined github.
What else should I look at to try and debug what happened/changed?

You're right, it's not working for me anymore either :(
I'm using Firefox 73.0 (64 bit) on Ubuntu 18.04

Doesn't work on Chrome either, but I'm not seeing any errors in console.

Doesn't work on Chrome either, but I'm not seeing any errors in console.

I confirm the same has happened to me.
Using Chrome 80.0.3987.106 on win10.
I had RGH 20.2.3 . It was working fine
(I had a personal token in its settings and the "Show the features enabled on each page in the console" checkbox was enabled).
Today I clicked "Update" in chrome://extensions
and RGH got updated to <IP_ADDRESS>2 .
From that point it stopped working,
Additionally, in its settings the personal token no longer exists
and the Show the features enabled on each page in the console checkbox is now unchecked.

Options screenshot

I have Firefox 73.0 on macOS and RG 20.2.14 and most feature work:

None of the errors in any of those consoles are related to Refined GitHub.
Have you tried turning it off and on again?

I have Firefox 73.0 on macOS and RG 20.2.14 and most features work.

I also have no problem with RG in Firefox 73.0 on win 10.
For me, RG is not working only in Chrome, and, disabling and re-enabling RG in chrome://extensions doesn't help.

I know what happened: <IP_ADDRESS>2 is a broken version, I stopped it in Firefox but it was published in Chrome just today. Also you might have downloaded that version in Firefox while it was available

Fix ready
Update your extension as soon as your store shows 20.2.18

For me it works again, thx for the quick reaction & fix! :)

After upgrading to 20.2.18 on Firefox everything works again. Thanks for the super quick fix.
Closing as it is resolved.

I see some discrepancy in the shipped CSS in Firefox vs. Chrome extensions. Below style is only present in the Firefox extension. Is this related here?
.rgh-collaborator {
  border: 1px solid #c0d3eb;
}

@silverwind chrome store is still on <IP_ADDRESS>2, so that's what you're possibly seeing.

Not sure, that version is 5 days old while the feature should've been there since many months.
Source file here.

Yeah, after closer investigation, it's the broken version which seems to come from a very old codebase.

Reopening for visibility

Same problem and console errors with :

Windows 10 x64 1803
Chrome  78.0.3904.87 (64 bits)
Extension version <IP_ADDRESS>2
GitHub Enterprise Server 2.17.6

Is there a provided package somewhere so we can sideload it in chrome without waiting the official publishing on chrome webstore ?
I didn't find this kind of package on releases page

Is there a provided package somewhere so we can sideload the last working version 20.2.19 in chrome without waiting the official publishing on chrome webstore ?
I didn't find this kind of package on releases page

I've built one for myself using the latest code. If you would like to use it, here it is.
If you do not trust this, you can build one easily, just follow the contributing guide (but replace npm run watch with npm run build).

@s-kocher just go to the github action of the last commit and download the artifact

Thanks @kidonng & @yakov116 I tested both of our solution are working fine :)
I never used github action artifact result, thanks for the information !

I added more info in https://github.com/sindresorhus/refined-github/issues/2796#issuecomment-587428669

Has it been verified that the new version has been pushed to the Chrome web store? I'm surprised it'd take them this long to update.

@Enrico2 don't worry. It gets even more surprising and interesting how i'm getting pretty old version and even already not existing or renamed features while I just installed that version before 2-3 days when started to report tons of other bugs :D
I'm waiting for new chrome store version too :)

Status: https://twitter.com/DotProto/status/1230746697883508737

Update your extensions! A new version has finally been published by the Chrome extensions team :tada:

Don’t forget to restore more-tab extension, if you previously disabled it


DynamoDBChatMessageHistory not working with initializeAgentExecutorWithOptions

I am getting the following error:
"output values have 2 keys, you must specify an output key or pass only 1 key as output"
Code:
  const memory = new BufferMemory({
    chatHistory: new DynamoDBChatMessageHistory({
      tableName: "SessionTable",
      partitionKey: "SessionId",
      sessionId: "1233444",
      config: {
        region: "us-west-2",
      },
    }),
    memoryKey: "chat_history",
    returnMessages: true,
  });

  const model = new ChatOpenAI({
    modelName: "gpt-3.5-turbo-0613",
    temperature: 0,
  });

...
  const executor = await initializeAgentExecutorWithOptions(tools, model, {
    agentArgs: {
      prefix: `(Prefix text here).`,
    },
    agentType: "openai-functions",
    returnIntermediateSteps: true,
    verbose: true,
    memory,
  });

  const result = await executor.call({ input: receivedMessage });

I have tried specifying outputKey: 'text' as well when creating the BufferMemory object, this didn't seem to make a difference though.

Removing returnIntermediateSteps: true, should fix it, or try setting outputKey: "output" which I believe is the default in this case.
Going forwards, we'll be moving docs to show a more compositional and clear approach and deprecate this method. Please reopen if the above doesn't fix the issue.

That worked. Thanks!


broker should count rxQuota on both qos1 and qos2

Please fill out the sections below to help us address your issue.
Version of VolantMQ (or SHA)
master
What issue did you see?
currently, volantmq add rxQuota only reduce when got qos2, but it should count qos1 too.

Make sense, PR?

Actually no. rxQuota is for incoming messages. QoS1 is at-least-once delivery mode thus we don't track how many we received as server responds with PUBACK immediately (even if it not delivered to publisher due to network failure new publish can come after recover)


Information regarding AUTOMATIC MATCHING BETWEEN ORDERS

I copy the message from the MakerChat :
It's becoming a very common thing now to see newcomers getting confused by the lack of order matching. And this is our fault, this should be very explicit because there's not a single centralized exchange which works this way. I suggest a warning msg below the buy/sell boxes and above the "buy orders" and "sell order" boxes.
Suggestions for this message below the buy/sell boxes:

THERE'S NO AUTOMATIC MATCHING BETWEEN ORDERS!
To create a new entry on the order book, simply use the boxes above. But if you want to fill existing orders, you have to click on each desired order, one at a time.
There should be a little x at the right top corner of the message box, so the user can dismiss it (hide it) after having read it.

I will create a few different approaches so we can decide together how we get the user informed regarding the missing automatic matching between orders!

Version 1

Version 2

I prefer version 1

under get started 3) there is a typo: befor -> before.
There also need to be an additional sentence, something like "You're automatically prompted to do this if your allowance isn't already set"

btw lets get rid of the numbers for allowance and just make it a binary approval mechanism that sets the allowance to 999999999 behind the scenes.

I disagree to setting allowance to some arbitrary high number under the
hood for convenience sake. please keep security in mind at all time; I have
heard a couple times of traders being saved from making incorrect trades
due to the allowance mechanism.
On Jan 24, 2017 04:27, "Rune23"<EMAIL_ADDRESS>wrote:

btw lets get rid of the numbers for allowance and just make it a binary
approval mechanism that sets the allowance to 999999999 behind the scenes.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/makerdao/maker-market/issues/128#issuecomment-274691496,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAl5n2evu5siyAQP_5eZ9D-V8byaY2Ylks5rVW-UgaJpZM4LqAWU
.

Version I is updated:
@Rune23

I made your suggested sentence a little bit shorter so it can fits in one line. Do you think this is enough ? Do you have maybe a better idea?
Regarding the allowance thing. I think we should discuss this topic with the whole team and should be moving it to the next design iteration. Right now we should clean most of the issues we already have and make the UX as perfect as possible.

@pimato What happens if you close one of the banners? How the other would show?

if you close the left one, the right one should be expand (and so on) example:

I have a better version of number 3: The first time you make a transaction you will be prompted to set your allowance

UPDATED :

Okay I have a few points :

Warning signs should be aligned to the widgets below.
the number_icon should have the same padding to the left as to the right
You still have a opacity on the number_icon and the dots, please remove it.

This is how it should look like:

I uploaded you new darker crosses. There is a default(normal) edition and a pressed version. Would love to see both integrated.
Archiv.zip

@pimato changes were applied in the repository. When this is uploaded please check what you think about the alignment. The structure is a bit complicated and tried to align it the best as possible. Let me know if looks fine.

small issue when I am cloing the right warning :

@pimato for some reason in your screen the first li fits in one line. The bullet was prepared for two. Will make some tests.

@pimate just made a fix. Let's see if you still see the issue when we upload this.

Yes, sadly I can still see it.

fixed


Login failed

Receive error in iobroker log:
bmw.0 2024-11-18 16:27:53.105	error	undefined
bmw.0 2024-11-18 16:27:53.105	error	Start relogin in 5min
bmw.0 2024-11-18 16:27:53.104	error	Please check username and password or too many logins in 5 minutes
bmw.0 2024-11-18 16:27:53.104	error	{"error":"invalid_client","error_description":"Client authentication failed (e.g., login failure, unknown client, no client authentication included or unsupported authentication method)"}
bmw.0 2024-11-18 16:27:53.104	error	AxiosError: Request failed with status code 401
bmw.0 2024-11-18 16:27:53.103	error	Login failed
Expected behavior
Login was successful since yesterday
Versions:

Adapter version: 2.8.2, 2.8.1 tested
JS-Controller version: 5.0.17
Node version: v18.19.0
Operating system: Debian GNU/Linux 11 (bullseye)

Please test 2.8.3

Thank you a lot - issue solved!


Updating READMEs for Alderaan

@christianbrb We have the section Try Out the Raiden Demo dApp with instructions on how to use the dApp on testnets. It is short but to the point and works well in the readme. I think we should keep it as it is and complement with a similar one but for mainnet? What do you think?

Also, since we are mentioning the dApp and SDK in the main README it makes sense to add a section about the CLI/API also in the main README?

@christianbrb We have the section Try Out the Raiden Demo dApp with instructions on how to use the dApp on testnets. It is short but to the point and works well in the readme. I think we should keep it as it is and complement with a similar one but for mainnet? What do you think?

I would keep this section. Topics for updating the readmes for mainnet can be put here #1785

Also, since we are mentioning the dApp and SDK in the main README it makes sense to add a section about the CLI/API also in the main README?

Yes, please :)


Fix some mathematical presentation to compatible with github markdown

Convert some mathematical symbols to compatible with github markdown. For example:

Superscript $[l]$ denotes a quantity associated with the $l^{th}$ layer.

Example: $a^{[L]}$ is the $L^{th}$ layer activation. $W^{[L]}$ and $b^{[L]}$ are the $L^{th}$ layer parameters.

Will become:

Superscript [l] denotes a quantity associated with the l.th layer.

Example: a[L] is the L.th layer activation. W[L] and b[L] are the L.th (last) layer parameters.

Thank you so much! great updates
Feel free to make more.


Doubts about the routing lifecycle hooks

Reproduction
https://jsfiddle.net/puL79c2f/3/
Steps to reproduce the bug

Click Home link, the consol print beforeEach  and afterEach
Click Home link again, the console print afterEach only

Expected behavior
Second times click Home link, the console should print nothing，because nothing router change
Actual behavior
Nothing router change，but afterEach hook running
Additional information
No response

Check navigation failures on docs. That’s why afterEach triggers


[BENCH-1051] Support the rename of terra cli to vwb

Code-wise, this all looks very good. I have a few questions:

Is there a Jira ticket associated with this, notably which includes sign-off from the Product team on the naming (I know we did a round of voting some time back)?
Are there any dependencies on other code or docs that this needs to be coordinated with? I'm wondering whether we should soft link both ~/.terra -> ~/.vwb as well as the executable itself such that dependent code (including some solutions getting started material) can transition

There are many docs that would need to be changed (in addition to the examples), so I think we need to alias/soft-link.
(Having not looked at the code changes yet)

https://verily.atlassian.net/browse/BENCH-1051
Soft-linking .terra to .vwb sounds good to me.


There is a problem with dart2js usage

I have a mobile App that I developed with flutter. Now I wanted to do a cross-platform desktop application, and I tried flutter, but it was too immature. Now I'm doing it with electron. I want to reuse dart's code, so I want to convert dart to JavaScript or TypeScript. I only find dart2js, and then I convert dart to JavaScript using the following command in the official web documentation
dart2js -O1 -o test.js test.dart

test.dart

Generated JavaScript file test.js

It's just a few lines of dart code, and it generates this much JavaScript code, which is not what I wanted.

Sorry to hear that you found the flutter ecosystem immature for your purpose. It would be great to get some feedback about your experience to make sure we understand what you found lacking or if there are parts of its documentation that could be improved. I'm no expert on the flutter side of things, but I can find out someone to cc here who can direct your messages, or please feel free to reach out directly to them.
About the emitted code. Dart has very different semantics than JavaScript, so any Dart to JS compiler will generate code that has some extra code to correctly model our language. This means that the generated code sometimes is not as idiomatic JavaScript as what you'd write by hand. For example, Dart Lists are implemented with native JS Arrays, but they don't have the same operations or errors. This initial tax diminishes as the apps get bigger.
We have two compilers that emit JS. dart2js is our whole-program optimizing compiler meant to generate optimized code when assembling large apps. The optimization it has help delete unnecessary code (for some apps this can be as high as 50% of the original code), perform local optimizations and enable Dart to run fast on a JS engine.  Even when disabling most optimizations, it operates as a whole-program compiler and it is not really meant to be used to export code to JS.
There is a separate development compiler called dartdevc or DDC. That compiler does work more modularly and has the ability to generate ES6 modules for each library (unless your libraries form a cycle, in which case you may have one module for an entire strongest connected component of your import graph). The code generated by this compiler might be easier to work with if you are trying to export your app and start something with it. Again, I don't expect it to be as idiomatic as if you wrote it by hand because of the extra Dart semantics, but it may be a good starting point if you are doing a migration afterwards.
The easiest way to access the dartdevc compiler is to use the webdev commandline tool. This tool takes care of splitting apps into modules and invoking the compiler one module at a time. While invoking dartdevc by hand is possible, it gets complicated when you start tracking dependencies between modules and you need to feed the compiler data about what each dependency does. For that reason, I strongly recommend using the webdev tool, and extract the code it generates.
Hope you find these pointers useful! Thanks again for reaching out and for trying out flutter. And please do reach out to the team to see if they can assist on your flutter use case!

It seems I originally forgot to mark the issue as closed. Feel free to reach out if you have additional questions.


Feature request: Use Plotly instead of matplotlib

Is your feature request related to a problem?
Using matplotlib is very limited when it comes to graphical representation.
Describe the feature you'd like
The idea is to replace matplotlib module with plotly
Additional context
With plotly:

With matplotlib:

I can take care of the migration to Plotly.

Hi @samatix,
The plotly charts do indeed look awesome, but I don't think plotly should be part of core PyPortfolioOpt.
Ultimately I want PyPortfolioOpt to be focussed primarily on the portfolio optimisation itself – data ingestion, backtesting, plotting, portfolio analysis etc is not truly within the scope of the library.
The existing plotting functionality is meant to be a "quick" solution for non-technical users to get off the ground, but my belief is that any more advanced plotting should be done by the user, using a library of their choice. Also, while I agree that plotly produces superior charts, matplotlib is the "industry standard" (at least in finance), so people will be able to interact with the plots.
All this said, if you are willing to share code, I'd love to add this in one of the tutorial notebooks as an example of advanced plotting (will link/attribute in the docs).
Cheers,
Robert

Fully agreed @robertmartin8, I'll share the code in the cook book section.

@samatix did you ever get the plotly in to a notbook for demo?


feat: support controller as etcd server

Type of change:

[ ] Bugfix
[x] New feature provided
[ ] Improve performance
[ ] Backport patches
[ ] Documentation
[ ] Refactor
[ ] Chore
[ ] CI/CD or Tests

What this PR does / why we need it:

ref:

#610

Pre-submission checklist:

[ ] Did you explain what problem does this PR solve? Or what new features have been added?
[ ] Have you added corresponding test cases?
[ ] Have you modified the corresponding document?
[ ] Is this PR backward compatible? If it is not backward compatible, please discuss on the mailing list first

Codecov Report

Merging #1803 (bf4c617) into master (98ff8e5) will decrease coverage by 2.85%.
 Consider uploading reports for the commit c21e944 to get more accurate results

@@            Coverage Diff             @@
##           master    #1803      +/-   ##
==========================================
- Coverage   39.30%   36.46%   -2.85%
==========================================
  Files          91       91
  Lines        8073     8598     +525
==========================================
- Hits         3173     3135      -38
- Misses       4494     5051     +557
- Partials      406      412       +6

Impacted Files
Coverage Δ

pkg/apisix/consumer.go
26.06% <0.00%> (-11.35%)
:arrow_down:

pkg/apisix/global_rule.go
26.06% <0.00%> (-11.35%)
:arrow_down:

pkg/apisix/plugin_metadata.go
2.51% <0.00%> (-2.14%)
:arrow_down:

pkg/apisix/pluginconfig.go
26.84% <0.00%> (-11.22%)
:arrow_down:

pkg/apisix/route.go
24.87% <0.00%> (-14.98%)
:arrow_down:

pkg/apisix/ssl.go
27.86% <0.00%> (-10.48%)
:arrow_down:

pkg/apisix/stream_route.go
26.42% <0.00%> (-10.81%)
:arrow_down:

pkg/apisix/upstream.go
32.53% <0.00%> (-17.84%)
:arrow_down:

...kg/providers/apisix/translation/apisix_consumer.go
65.21% <0.00%> (-1.95%)
:arrow_down:

pkg/providers/apisix/translation/apisix_route.go
31.14% <0.00%> (ø)

... and 6 more

... and 1 file with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

I want to put this to v1.8.
So we won't do a detailed review of this PR before the release of v1.7.

In a high availability architecture, a simple hang is meaningless for non-leader nodes, they still need to watch k8s resources continue to work, and are not allowed to hold write permissions until they become elected (write k8s resources, write apisix)

In a high availability architecture, a simple hang is meaningless for non-leader nodes, they still need to watch k8s resources continue to work, and are not allowed to hold write permissions until they become elected (write k8s resources, write apisix)

Great design, providing a clearer control plane and data plane architecture. However, I have a few questions:

Does the current architecture support high availability ?
Can a non-leader APISIX ingress controller be used as a viable ETCD server?  How does APISIX avoid connecting to an unavailable ETCD server?
Some extreme edge cases may not be handled, such as when an APISIX ingress controller restarts before K8S CR resources such as APISIXRoute are synchronized to the corresponding ETCD server.  Will the ETCD server come online prematurely and provide service?  APISIX may obtain empty router data at this time, leading to failure.

In a high availability architecture, a simple hang is meaningless for non-leader nodes, they still need to watch k8s resources continue to work, and are not allowed to hold write permissions until they become elected (write k8s resources, write apisix)

Great design, providing a clearer control plane and data plane architecture. However, I have a few questions:

Does the current architecture support high availability ?
Can a non-leader APISIX ingress controller be used as a viable ETCD server?  How does APISIX avoid connecting to an unavailable ETCD server?
Some extreme edge cases may not be handled, such as when an APISIX ingress controller restarts before K8S CR resources such as APISIXRoute are synchronized to the corresponding ETCD server.  Will the ETCD server come online prematurely and provide service?  APISIX may obtain empty router data at this time, leading to failure.

Supported and implemented.
The non-leader node can still work, but it does not have write permission, but APISIX can still read data from it
In the current stage, DP and CP will run in one Pod, and if one of them is abnormal, the Pod will be restarted, and this problem does not exist. This problem only exists in the architecture where DP and CP are separated, which depends on the implementation of etcd-adapter, which can actually be solved, just ensure that etcd revision is always incremented, then APISIX will still read from DP when the DP plane is available Get the data.

Minor typo. Does this feature means the adpater is using btree in-mem?

Yes.

Due to the fact that this feature has already been mostly completed and v1.7 has not yet been released, I think it's possible to move this feature to version v1.7.

Wait for CI.
Then we can merge this one.


Tooltip closes whenever parent rerenders

Detailed description

Describe in detail the issue you're having.
Whenever the tooltip is open, if parent rerenders the tooltip closes.

It looks like the tooltip has a default property of 'open' set to false. Whenever it receives new props (as part of parent rerender), the tooltip sets state 'open' to the value of prop.open (which is always false). This causes the tooltip to close.

Is this a feature request (new component, new icon), a bug, or a general issue?
Bug

Is this issue related to a specific component?
Tooltip

What did you expect to happen? What happened instead? What would you like to see changed?
Expected tooltip to stay open

What browser are you working in?
Chrome & Safari macOS X

What version of the Carbon Design System are you using?
5.36.0

Steps to reproduce the issue
Codesandbox

Hover on tooltip
Wait 3 seconds, it will then close

Hi @nicthemighty! 👋 Thanks so much for making this issue. Definitely agreed this is an issue with the component, so went ahead and labelled it as a bug. Will try and get it prioritized to one our sprints as a bug fix task, but will also accept any/all PRs that help to address this!


Doesnt work since 1.20.6 (purpur)

[13:18:50] [Server thread/INFO]: [AutoServerUpdater] Enabling AutoServerUpdater v1.0.1
[13:18:50] [Server thread/ERROR]: [AutoServerUpdater] SERVER SOFTWARE NOT SUPPORTED BY AUTOSERVERUPDATER
[13:18:50] [Server thread/INFO]: [AutoServerUpdater] Latest Build installed!
[13:18:50] [Server thread/INFO]: [AutoServerUpdater] Disabling AutoServerUpdater v1.0.1

Hmm Purpur must have changed their server brand will take a look later

1 thing to note here is that java 21 (at least the temurin dist) on windows seems to no longer allow writing to jar files while loaded by an application. Since i assume you are running this in your dev env (no sane human should auto update their production server) and that is most likely on windows, if you are affected by this either look into trying another java distro or (what i would recommend) using an external application for setting up your test server such as my MinecraftServerCLI


Refused to apply style from 'https://vroumtest.000webhostapp.com/admin/voyager-assets?path=css%2Fapp.css' because its MIME type ('text/html') is not a supported stylesheet MIME type, and strict MIME checking is enabled

Version information

Laravel: v6
Voyager: v1.3.0
PHP: 7.2
Database: [Mysql] [version] (e.g. MySQL 5.0.12)

Description

Everything is working fine on localhost. but after deployment on production webserver. I am getting the error.
Steps To Reproduce
Steps to reproduce the behavior:

Go to 'install voyager on localhost'
Then 'deploy on production server' (in my case 000.webhostapp.com)
And 'try to login'
You will see the CSS does not apply. you can observe from the screenshot on the left you have the actual output on the right you have the error shown by the console. and in the console, you can see the error (...because its mime type...etc..)

Expected behavior
I was expecting everything to display just like on the localhost. But the CSS does not apply
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Everything works perfectly on localhost

You'll need to provide some more informations:

How the content looks like for that file?
How the response headers look like?
What are the other errors?

Also please always provide full versions like 1.3.0

You'll need to provide some more informations:

How the content looks like for that file?
How the response headers look like?
What are the other errors?

Also please always provide full versions like 1.3.0

I provided more details. Thanks

I provided more details. Thanks

Not the informations I asked, you need to check in Network tab.

I provided more details. Thanks

Not the informations I asked, you need to check in Network tab.

My network tab is empty.
My Response Header:
content-encoding: gzip
content-type: text/html; charset=UTF-8
date: Tue, 17 Dec 2019 08:18:54 GMT
server: awex
status: 200
x-content-type-options: nosniff
x-request-id: 4d3a06e3275735d67cbc57c6fadd2a20
x-xss-protection: 1; mode=block
My Request Header:
:authority: vroumtest.000webhostapp.com
:method: GET
:path: /admin/voyager-assets?path=css%2Fapp.css
:scheme: https
accept: text/css,/;q=0.1
accept-encoding: gzip, deflate, br
accept-language: en-US,en;q=0.9,fr;q=0.8
cookie: _ga=GA1.2.1246130429.1571720651; _gid=GA1.2.939893480.1576529622
referer: https://vroumtest.000webhostapp.com/admin/login
sec-fetch-mode: no-cors
sec-fetch-site: same-origin
user-agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/78.0.3904.108 Safari/537.36
Query string parameter:
path: css/app.css

It's quite unlikely this problem is related to Voyager, more probably some problem with configuration or some strange server setup.
Said that if you want to solve it you'll need to be more proactive debugging this.
You only provided informations about one point out of three asked.

problem solve.it was due to some blank line i left in my helpers class


esp32 exFAT SD not initialising

I have an esp32 using hardware SPI with a tft display and the sd card sharing the SPI.
It is working fine with a FAT32 sd card - the SPI frequency is set to 16Mhz for the sd card and the display. It still works fine using the new library.
The problem arises when I use a new SanDisk Extreme Pro SD card (64Gb).
I have got as far as begin() working but when it tries to open a file it fails.
I have run bench.ino and it fails at the same point (opening the file) though it ran quite happily in the same hardware setup with a fat32 card.   The card is freshly formatted using the proper formatter.
I have
SD_FAT_TYPE 3
My chip select is fine on pin 5 for other cards.  SD_CS_PIN 5
I have SPI_CLOCK SD_SCK_MHZ(16)
I have forced the use of #define SD_CONFIG SdSpiConfig(SD_CS_PIN, SHARED_SPI, SPI_CLOCK)
The output of bench is:-
Type is exFAT
Card size: 63.86 GB (GB = 1E9 bytes)
Manufacturer ID: 0X3
OEM ID: SD
Product: SR64G
Version: 8.0
Serial number: 0X78524428
Manufacturing date: 1/2014
FILE_SIZE_MB = 5
BUF_SIZE = 512 bytes
Starting write test, please wait.
write speed and latency
speed,max,min,avg
KB/Sec,usec,usec,usec
error: preAllocate failed
SdError: 0XC,0X7F
Thanks for your efforts.
I am a bit new to mucking around with SD cards so forgive me if I am being foolish.
Glyn
p..s. perhaps I should mention that I get a compiler warning -
#warning FreeStack is not defined for this system.
but I got that with the old library and it did not seem to make any difference.  I commented out the include to get rid of the warning back then.  I don't know what it does and since things were working I have ignored it.

FreeStack not defined is OK.  It is not supported for ESP. On other systems it returns unused stack space.
I can't help with your problem.  I don't use ESP32 any more.

I am quite happy to do some investigation myself.  I am quite well equipped electronically (if need be) and I am good enough at software to have altered the Utouch library to work with esp32.  Could you perhaps briefly explain the preAllocate failed error or point me at some documentation?
At the moment I do not know where to start to approach this error.

 I can't help.
You could try editing SdFatConfig.h and select standard library driver like this:
#define SPI_DRIVER_SELECT 1
It will be slower than the custom driver.  I will probably drop the custom driver since it is a pain to maintain.
I do have an ESP-VROOM-32 board and tried it with a 64 GB Samsung PRO+ SD.
I selected 25 MHz SPI speed since the ESP32 driver has a bug and tries speeds higher than it can support.
#define SD_FAT_TYPE 3
Here is the bench result:

Use a freshly formatted SD for best performance.

Type any character to start
Type is exFAT
Card size: 64.02 GB (GB = 1E9 bytes)

Manufacturer ID: 0X1B
OEM ID: SM
Product: 00000
Version: 1.0
Serial number: 0X2C06AFA0
Manufacturing date: 9/2015

FILE_SIZE_MB = 5
BUF_SIZE = 512 bytes
Starting write test, please wait.

write speed and latency
speed,max,min,avg
KB/Sec,usec,usec,usec
2226.04,251,228,228
2225.05,251,228,228

Starting read test, please wait.

read speed and latency
speed,max,min,avg
KB/Sec,usec,usec,usec
2130.24,248,238,239
2131.15,248,238,239

I tried shared SPI.  Here is the result.
Type any character to start
Type is exFAT
Card size: 64.02 GB (GB = 1E9 bytes)

Manufacturer ID: 0X1B
OEM ID: SM
Product: 00000
Version: 1.0
Serial number: 0X2C06AFA0
Manufacturing date: 9/2015

FILE_SIZE_MB = 5
BUF_SIZE = 512 bytes
Starting write test, please wait.

write speed and latency
speed,max,min,avg
KB/Sec,usec,usec,usec
435.70,19854,284,1174
435.47,21840,284,1174

Starting read test, please wait.

read speed and latency
speed,max,min,avg
KB/Sec,usec,usec,usec
838.45,2708,471,609
839.44,1999,470,608

Hi I wanna try exfat with esp32 to access some times like last modified last access etc. But last on some forums I checked esp32 doesn't support exfat so will it work

There are conflicts with the esp32 system.  It's best to use the esp32 SD library.  I don't want to fight changes in esp32.
You can try SdFat but expect conflicts.


Github Actions not caching _site

I'm running into an odd issue where every time my site deploys it has to re-generate all of the webp image files, because it can't find them in the destination directory (despite being in the same folder with the originals). The code runs fine locally, but that's because the _site folder is still there. But with this plugin that _site data disappears after it's pushed.
Edit: I should add that I had to fork the action and edit it to run with ubuntu-latest and bash in order to get the thing to run the webp plugin. If there's a way to update it to support the plugin using the current setup. https://github.com/datapolitical/jekyll-deploy-action
So I'm just sort of trying to figure out how to make the two work together correctly, and any thoughts you might have would be very welcome.
https://github.com/sverrirs/jekyll-webp/blob/master/lib/jekyll-webp/webpGenerator.rb

Hi @datapolitical
Thanks for your reporting, I will work to solve this problem.
Thanks and regards

Hi @datapolitical
Thanks for your reporting, I will work to solve this problem.
Thanks and regards

Thanks!

Thanks!

Turns out this is not working and my site build has ballooned to over 16 minutes because of the responsive image plugin I'm using.
The action runs, the cache hit happens, but nothing is being stored in the cache.
I use this:
# Use GitHub Actions' cache to cache site on servers
  - uses: actions/cache@v2
    with:
      path: _site
      key: ${{ runner.os }}-site-${{ hashFiles('**/index.html') }}
      restore-keys: |
        ${{ runner.os }}-site-

from here: https://github.com/datapolitical/chrisfnicholson.github.io/blob/main/.github/workflows/build-jekyll.yml
But the response is:
Received 22 of 22 (100.0%), 0.0 MBs/sec
Cache Size: ~0 MB (22 B).
I assume there's something up with my path but it's exactly what was suggested above.

Hi @datapolitical
I saw that you're not using this jekyll-deploy-action from here https://github.com/datapolitical/chrisfnicholson.github.io/blob/481414b45213dafaf635ba4b980ffa0050050966/.github/workflows/build-jekyll.yml#L54, so I think you cache a wrong site ouput folder.
Thanks and regards

It's the same action I just updated it to run in Ubuntu and added a few extra dependencies so my site would build properly. Shouldn't change the build folder.
https://github.com/jeffreytse/jekyll-deploy-action/compare/master...datapolitical:master

Could you change your action to<EMAIL_ADDRESS>and try again? Maybe it settle your issue abou the _site folder has not been cached. : )

Didn't build because of all the dependencies I load early in the script conflict with what your code expects, but I need them to run imagemagick

Hi @datapolitical , my deploy action has added imagemagick package to the system enviroment, so I don't know why it has the conflicts, could you provide me the action log to analysis?

This is the issue I'm talking about. It was a lot of work to make it compatible.
https://github.com/sverrirs/jekyll-webp/issues/15#issuecomment-739579629

Hi @datapolitical , don't worry, I will try to help you to settle down this issue, please be patient. : )

Hi @datapolitical
Thanks for the patience and currently this issue has been addressed and fixed.

You don't need to use a fork's action and I have test with the action jeffreytse/jekyll-deploy-action@master and it has no any dependencies issue. (See here and the website)
The output folder is not the _site and you should cache  build folder as follow code snippet.

      # Use GitHub Actions' cache to cache site on servers
      - uses: actions/cache@v2
        with:
          path: build
          key: ${{ runner.os }}-build-${{ hashFiles('**/index.html') }}
          restore-keys: |
            ${{ runner.os }}-build-

Thanks and regards

When I use your action I get "￼ /usr/bin/env: ‘ruby2.7’: No such file or directory" on line 89 https://github.com/datapolitical/chrisfnicholson.github.io/runs/2161384500?check_suite_focus=true

I switched it back to my fork and now caching works great! Thanks so much for your help.￼ Happy to debug the issue if you want but very glad this works, it took nearly 10 minutes off my build time.

Hi @datapolitical,
Currently I'm researching the issue "/usr/bin/env: ‘ruby2.7’: No such file or directory", could you test with the action jeffreytse/jekyll-deploy-action@master again and tell me the result?
Thanks and regards

Hi @datapolitical, could you invite me a collaborator as follow instructive, so that I can re-run the action by myself conveniently, it will be fast to settle this issue for me.

Didn't work. Here's the error:
￼

@datapolitical According to the latest result, it seems to a issue that used a wrong bundle cache built for ubuntu but my action's Dockerfile is using Alphine Linux (super lightweight).

If you want to keep debugging, I updated the development version of the action so that the the action will run any time there's an issue added to the repository.
And yes, your action's Dockerfile uses Alpine Linux. If you wan't I'll try editing mine to remove the ubuntu stuff to see if yours can build the site properly, but at least when I tried a couple months ago, a couple of the plugins needed ubuntu.

Hi @datapolitical, thanks for your help to dig this issue, a couple of the plugins's dependencies such as libmagick has been added,  and your whole site has been built without any problems in my action currently, it's another issue, so it's better to settle this cache issue and improve the compatibility.

Hi @datapolitical  I just comment the issue in your repository but never trigger the action

Try opening a new issue, that worked for me.

I think the comment_created event should be better, I am afraid of too many issues 😢

OK now it runs if you make a comment￼.

 Still waiting for it to finish generating all the files but figured I'd give you a heads up.

Actually it looks like you fixed it because the old workflow you just ran also worked.

Hi @datapolitical
Many thanks for your help and the cache compatibility issue has been addressed and fixed (#12), now you can just use the jeffreytse/jekyll-deploy-action@master without any other issues.
Thanks and regards

@datapolitical Could you change back to the previous cache key? So that I can check if it does work.

Trying it now live

Hi @datapolitical,  I have seen the action cleaned up the incompatible gem cache, so this issue has been settled down perfectly.

Also, If you wouldn't mind creating a release, I prefer not to target my code at master because things can change.￼

@datapolitical, I will publish a release in this weekend, any other new bugs I will fix in this week.

It's broken again

Nevermind, I know what happened. it was a different plugin


[App Config] Update raised errors in app config command module

Related command
az appoconfig
Description
This change is being added to updated the raised errors to more specific errors in azure.cli.core.azclierror instead of the base CLIError class.
Testing Guide

History Notes

[App Config] az appconfig: Update raised errors in app config command module

This checklist is used to make sure that common guidelines for a pull request are followed.

[X] The PR title and description has followed the guideline in Submitting Pull Requests.

[X] I adhere to the Command Guidelines.

[X] I adhere to the Error Handling Guidelines.

App Config

@avanigupta @zhoxing-ms  Could you please take a look at this? If this is fine, then a follow-up PR will be submitted for the other files in this repo.

/azp run


DefaultAzureCredential failure shared across different instances

Describe the bug
Use DefaultAzureCredential to connect to two difference resources running on Azure VM. Configure access policy such that access to resource1 fails and access to resource2 succeed.
var credential = new DefaultAzureCredential();
credential.GetToken(new[] { "resource1" }); // Throws exception as expected.
credential.GetToken(new[] { "resource2" }); // Throws the same exception as `resource1`, unexpected.

// Creating two instances of DefaultAzureCredential has the same problem
var credential1 = new DefaultAzureCredential();
var credential2 = new DefaultAzureCredential();
credential1.GetToken(new[] { "resource1" }); // Throws exception as expected.
credential2.GetToken(new[] { "resource2" }); // Throws the same exception as `resource1`, unexpected.

Expected behavior
credential2.GetToken(new[] { "resource2" }) should succeed even though access to resource1 fails.
Actual behavior (include Exception or Stack Trace)
Failure of resource1 causes resource2 to fail.
To Reproduce

When constructing a DefaultAzureCredential without any parameter, the TokenCredential sources is using a shared global static instance;
When ImdsManagedIdentitySource is used, previous authentication failures are cached;

Environment:

Name and version of the Library package used: [Azure.Identity 1.4.0]
Hosting platform or OS and .NET runtime version (Azure VM)

Thank you for your feedback.  Tagging and routing to the team members best able to assist.

Bump the priority of this bug. We have an LSI today caused by this bug. One of our non-mandatory, non-critical downstream API failed to retrieve AAD token, it causes all the other mandatory and critical APIs to fail. We need a robust implementation of DefaultAzureCredential that does not leak failure of one instance to the other.
cc @superyyrrzz @fenxu

Hi @yufeih - I'd recommend switching to using the ChainedTokenCredential, which can be created to contain only the credentials you intend to use and does not cache the chosen provider statically.

No we'll stick with DefaultAzureCredential to benefit from the simplicity and default fallback chain.  We have used new DefaultAzureCredential(new DefaultAzureCredentialOptions()) as a workaround to force a new ImdsManagedIdentitySource instance.
The real problem IMO is in ImdsManagedIdentitySource around 400 handling. It assumes that if one resource or scope returns 400, all the other resources or scopes are also 400. In our case, we used the same DefaultAzureCredential instance (which internally creates the same ImdsManagedIdentitySource instance) to access both an email service and a storage service. When there are some RBAC misconfiguration that causes email service to 400, access to storage service raises exception that shows error message about the email service, which is both incorrect and misleading.
Add @schaabs as the behavior seems to be introduced in https://github.com/Azure/azure-sdk-for-net/pull/9025

@yufeih I'm glad you have a workaround. We plan to do some work in the near future to improve how ImdsManagedIdentitySource handles this scenario.

related to #20617

I'd like to throw my +1 in for a SDK-based fix for this that doesn't require a workaround. I've spent at least a day or two tracking down an issue to discover that the root cause is the same thing as mentioned here.
The specific issue I was coming across is as follows.

.NET 5 web api project using entity framework core
EF is connected to Azure SQL using a managed identity token, retrieved with var token = await new DefaultAzureCredential().GetTokenAsync(tokenRequestContext, cancellationToken);
The project is deployed to AKS (azure kubernetes) and has a startup probe that checks whether the SQL connection can be successfully made
Since the startup probe has a 1-second timeout, the request from kubernetes to the container is cancelled, signalling the cancellation token
This results in the following message being logged from the SDK - [Informational] Azure-Identity: IMDS endpoint is not available. Endpoint: http://<IP_ADDRESS>/metadata/identity/oauth2/token. Error: System.OperationCanceledException (0x8013153b): The operation was canceled.
Since we're using the DefaultAzureCredential, this effectively caches the IMDS failure

I'm going to work around it by using new DefaultAzureCredential(new DefaultAzureCredentialOptions()) as mentioned by @yufeih but it would be great if this wasn't necessary.

The next release of Azure.Identity should address this issue. See #23728


update to 0.27

Description
Updates to use noir version 0.27.0
Problem*

Resolves
Summary*

Additional Context

PR Checklist*

[x] I have tested the changes locally.
[ ] I have formatted the changes with Prettier and/or cargo fmt on default settings.

Not sure why the build preview is failing. Maybe something related to a failed testnet deployment @signorecello ?

So the issue here is that we can't gitignore the verifier contract file, because we can't easily run nargo in the netlify deployment process. Since we're moving that logic to bb, maybe this would be easier to do.
For now, better to not ignore the contract so it builds and deploys correctly

Merging yey!


Should enable validate fluentd ssl ca and hostname for test function

Server version: master
Step to reproduce:

config fluentd logging target with ssl config
ssl config input server.cert not ca.cert
click test function, it validate success
but fluentd throw error with the ssl certificate config
the test function should not return validate success when ssl use server.cert

Version: master 2.2 (2/28/19)
Confirmed fix.

Use the wrong hostname, click TEST, verify failure and prompt error message tls handshake michelia.cnrancher.com:24224 failed: x509: certificate is valid for michelia.cnrancher.com, not michelia.cnrancher.com1
If the ca certificate is wrong, it will not pass the verification.


queue for block ordering and execution

we need to execute blocks in order, without any gaps. however, this is not guaranteed by the current implementation.
we should implement some sort of queue that ensures blocks are executed in the correct sequence, and a block cannot be executed without its parent being the current head.

Related: https://github.com/astriaorg/astria/issues/13


calendar fails on render march 2021

Do you want to request a feature or report a bug?

Report a bug

What's the current behavior?

The calendar fail on render March 2021

Day 13 is duplicate and all consecutive days are in the wrong position, see the picture below. And then look at the April 3 to see how it render out of the calendar area

What's the expected behavior?

I be checking other months, and I figure out that all March for every Years fail on the render, inclusive some time the day of the week fails render too, see in the below picture

resolved


cleanup

made all doubles in example use double literals
made slider valueIndicatorTextStyle visible
convert rotation to double and name it appropriate.
made all doubles in lat_lon_grid_plugin.dart use double literals.
named all values named val more usefully
removed .toString() after .toStringAsFixed()
don't trigger rebuild while building aka. when the first build didn't finish yet

Will review ...


deCONZ Light Groups not showing up as Home Assistant entities

Home Assistant release with the issue:
0.93.1
Last working Home Assistant release (if known):
Unknown
Operating environment (Hass.io/Docker/Windows/etc.):
Docker on Ubuntu 18.04 LTS
Component/platform:
https://www.home-assistant.io/components/deconz/
Description of problem:
deCONZ Light Groups are not showing up as light entities in Home Assistant
Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant):
N/A (there is no valid configuration for deCONZ as far as I can tell)
Traceback (if applicable):
Additional information:
I have lots of lights, switches and sensors that show up with no problem.  I also tried adding a new scene in Phoscon and the Phoscon scene showed up as a scene entity in Home Assistant as expected.
I turned on debug logging and I can see that pydeconz is picking up the light groups and detecting changes in them so the issue appears to be in the deCONZ component.
After doing a little digging in the code I found a config option in the deCONZ component called "allow_deconz_groups".  If this value is not set then it returns true however there is a constant called DEFAULT_ALLOW_DECONZ_GROUPS which is set to false (https://github.com/home-assistant/home-assistant/blob/b50afec5f11e07352f40f9bdbbaf5bff7a6b11ff/homeassistant/components/deconz/const.py).
I tried setting allow_deconz_groups  in my configuration.yaml but this threw an error (either I set it wrong or it's not expecting this config option).
I checked in my core.config_entries file and under deCONZ I see the following
"allow_deconz_groups": false
So it appears that the default is to not allow deCONZ groups and there is no way to override this or change it.  There's also no way to know that your groups are being found but ignored due to this setting, nor is this setting documented anywhere.
It's possible this is presented as part of the initial deCONZ integration setup and I forgot about it!

Thanks for reporting, though this is not a bug and change requests are not supposed to be filed here.
Previously this was configurable during the onboarding flow. But it is now migrated to the options flow (which is not yet supported by the frontend). You can manually change the config_entry file to true if you want them.
Default is actually true, regardless of what that CONST says, it should be removed


Irish (not ready to merge)

There's a lot left to do before I'd want to merge this, but I'm a bit stumped about this:
(p/parse :ga$core "aon")
=> ()
(p/play :ga$core "aon")
  ---   2 | temperature | number as temp            | P = -2.5903 | numbers, 1-10
  ---   1 | number    | numbers, 1-10             | P = 0.0000 |
  aon

Any suggestions about what's going wrong?

Can you try (p/parse :ga$core "aon" [:number])?

Same thing:
myproject.core=> (p/parse :ga$core "aon" [:number])
()

Sorry, I had '(i?)' where I meant to have '(?i)'

Sorry for the long delay. I tried it today but I get the following errors. It may be due to recent merges in master. Could you please have a look?

duckling.core=> (load!)
"not found" "ten cents"
"not found" "$10,000"
"not found" "10K$"
"not found" "$10k"
"not found" "USD1.23"
"not found" "2 dollar and 23 cents"
"not found" "two dollar 23 cents"
"not found" "two dollar and 23"
"not found" "EUR29.99"
"not found" "nine pounds"
"not found" "GBP3.01"
"not found" "GBP 3.01"
"not found" "3.0 km"
"not found" "37 céimeanna ceinteagrád"
"not found" "-10°"
"not found" "- 10 céimeanna"
{:pt$core (:phone-number :number :distance :volume :temperature :time :url :email :timezone :unit :amount-of-money :leven-unit :quantity :ordinal :unit-of-duration :cycle :duration), :ko$core (:number), :ru$core (:number :ordinal), :ja$core (:number :temperature :ordinal), :fr$core (:phone-number :number :time :distance :volume :temperature :url :email :timezone :unit :amount-of-money :leven-unit :leven-product :quantity :ordinal :cycle :unit-of-duration :duration), :es$core (:phone-number :number :distance :volume :temperature :time :url :email :timezone :unit :amount-of-money :cycle :unit-of-duration :duration :ordinal), :da$core (:number :time :timezone :ordinal :cycle :unit-of-duration :duration), :zh$core (:number :temperature :time :ordinal :cycle :unit-of-duration :duration), :ar$core (:number :ordinal), :ga$core (:unit :number :amount-of-money :distance :volume :temperature :time), :it$core (:number :time :ordinal :cycle :unit-of-duration :duration), :de$core (:number :time :ordinal :cycle :unit-of-duration :duration :timezone), :uk$core (:number :ordinal), :et$core (:number :ordinal), :en$core (:phone-number :number :distance :volume :temperature :time :url :email :timezone :unit :amount-of-money :leven-unit :quantity :leven-product :ordinal :cycle :unit-of-duration :duration)}
duckling.core=> (run)
IllegalArgumentException Don't know how to create ISeq from: java.lang.Character  clojure.lang.RT.seqFrom (RT.java:542)
:pt$core: 382 examples, 0 failed.
:ko$core: 85 examples, 0 failed.
:ru$core: 84 examples, 0 failed.
:ja$core: 55 examples, 0 failed.
:fr$core: 486 examples, 0 failed.
:es$core: 295 examples, 0 failed.
:da$core: 354 examples, 0 failed.
:zh$core: 329 examples, 0 failed.
:ar$core: 31 examples, 0 failed.
0 FAIL "ten cents"
Expected null
1 FAIL "$10,000"
Expected D

@ar7hur all but one fixed (that one is disabled until I can figure out what's going wrong), no errors now

ok, all fixed

Thanks!


No colors in Magit

There are no colors in Magit unlike 'articlestudio/nord'
gbprod/nord.vim

articlestudio/nord.vim

Interesting, I'll take a look soon.

When you say Magit, you talk about https://github.com/TimUntersberger/neogit ?

No I'm currently using this one: https://github.com/jreybert/vimagit
But hey! the plugin which you shared looks super awesome and does the work of both fugitive + magit for me :)
Thanks a lot for sharing! I'll use this one for now :D

Should be fixed now, can you confirm ?

Yes It's working now :)
Thanks @gbprod


Introduce dialog support

[ ] Show a confirm dialog for dangerous operations
[ ] Show a custom dialog for more interaction possibilities

Dialogs are not supported in VSCode.


owlvit is not supported

Feature request
The conversion is supported in transfomers[onnx], but not yet supported in optimum.
Motivation
convert open world vocabulary to onnx model for faster inference.
Your contribution
If there is a guideline on how to do it, I think I can help

I see here that there was an issue with the export of aten:broadcast_to: https://github.com/huggingface/optimum/blob/8d97c6806d8b1bb97625f2387f747822b5fee68e/optimum/exporters/onnx/model_configs.py#L772
But it works with transformers.onnx, not sure what the difference is here in Optimum. Any idea @michaelbenayoun @fxmarty ?

Sorry, but one more thing..
How can I extend this so I can convert not only OwlViTModel, but OwlViTForObjectDetection too?

I tried to convert OwlViTForObjectDetection using the unmaintained transformers.onnx module
it seems like it works for OwlVitModel, but it doesnt work for OwlViTForObjectDetection.
It throws this error Exporting the operator 'aten::broadcast_to' to ONNX opset version 14 is not supported.
Shoud I just wait until pytorch support this?

Sorry, but one more thing.. How can I extend this so I can convert not only OwlViTModel, but OwlViTForObjectDetection too?

Here is the guide to add support for new architectures: https://huggingface.co/docs/optimum/exporters/onnx/usage_guides/contribute
So, if you want to add support for a new task, you'll need to register this task here.

I tried to convert OwlViTForObjectDetection using the unmaintained transformers.onnx module it seems like it works for OwlVitModel, but it doesnt work for OwlViTForObjectDetection.
It throws this error Exporting the operator 'aten::broadcast_to' to ONNX opset version 14 is not supported.
Shoud I just wait until pytorch support this?

Okay, so I guess that was the error I mentioned in my first message.
If you want to add support for this task, two possible ways:

On Transformers' side, you can investigate if it is possible to replace the two calls to torch.broadcast_to here and there. It is probably possible to find another way of doing the same thing that is compatible with ONNX export.
On PyTorch's side, it is possible to add support for new ATen ops following this guide but you'll work at a lower level than Transformers here. I also saw that you opened a new issue in the PyTorch repo, maybe someone will submit a PR.

@regisss
using pytorch nightly, now it can support the broadcast_to operation, I have some question on this task thing
I see that we can specify task in onnxconfig, such as
CLIPOnnxConfig(config, task="zero-shot-image-classification")
# or
CLIPOnnxConfig(config, task="feature-extraction")

but both this settings output the same results, is it intended?
logits_per_image, image_embeds, text_embeds, logits_per_text
I made a pull request on https://github.com/huggingface/optimum/pull/1067, but for now I am using torch nightly

Good news that it is now supported in PyTorch! That way, you can work on your PR and we will merge it when the next release of PyTorch is out :slightly_smiling_face:
Regarding your question, it is indeed expected since in Transformers these two tasks are mapped to CLIPModel (see here for feature extraction and there for zero-shot classification). Logits should enable to perform zero-shot classification and embeddings to perform feature extraction.

Hello there! Is zero-shot object detection supported by this PR? I've been trying to convert the OwlViT model (for object detection) to ONNX without success. I see there is no ORTModelFor_______ for object detection. I have also tried converting using transformers.onnx without success. Any tips

Hi @Pedrohgv , yes zero-shot object detection should be supported for owlvit. For example optimum-cli export onnx --model google/owlvit-base-patch32 --task zero-shot-object-detection owlvit_onnx should work.

@fxmarty Thank you for the reply. I successfully converted the model, but couldn't get it to run a sample. My code:
checkpoint = "google/owlvit-base-patch32" processor = AutoProcessor.from_pretrained(checkpoint) np_inputs = processor(text=text_queries, images=image, return_tensors="np") session = ort.InferenceSession(PROJECT_FOLDER + "owlvit_onnx/model.onnx") out =session.run(['logits', 'pred_boxes', 'text_embeds', 'image_embeds'], np_inputs)
This is throwing the error:
RuntimeException: [ONNXRuntimeError] : 6 : RUNTIME_EXCEPTION : Non-zero status code returned while running Reshape node. Name:'/Reshape_3' Status Message: /Users/runner/work/1/s/onnxruntime/core/providers/cpu/tensor/reshape_helper.h:41 onnxruntime::ReshapeHelper::ReshapeHelper(const onnxruntime::TensorShape &, onnxruntime::TensorShapeVector &, bool) gsl::narrow_cast(input_shape.Size()) == size was false. The input tensor cannot be reshaped to the requested shape. Input shape:{9,16}, requested shape:{2,4,16}
Now it seems to be related to some input being wrong, but I cannot get what is wrong. The pre-processing step is the same as for the HF model, only difference being instead of returning "pt" tensors I'm returning "np" so it can work with ONNX. Here are my input shapes:
input_ids: (9, 16)
attention_mask: (9, 16)
pixel_values: (1, 3, 768, 768)
Thanks in advance!

@Pedrohgv could you open an issue with a reproducible export + code?

Great, created this issue.
Thanks!


bug: Pagination Not Working

Describe the bug
Pagination Not Working
To Reproduce
Steps to reproduce the behavior:

Goto Home Page
Try to access another page using pagination

Expected behavior
Next page Buttons Dont Work
Screenshots or GIFs
If applicable, add screenshots to help explain your problem.
Info (please complete the following information):

Browser [e.g. chrome, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

Hey @ashdotenv are you refering to the below bug which is on https://projects.100xdevs.com or something else? Would be great if you can describe it more.


[Talk] Creating Secure OTPs: Best Practices

Title of the talk
Creating Secure OTPs: Best Practices
Description
I will be presenting on "Creating Secure OTPs: Best Practices." Initially, I'll discuss the importance of OTPs (One-Time Passwords) and highlight different types, such as time-based and HMAC-based OTPs. Additionally, I'll explain why relying solely on random number generators for OTP generation isn't ideal due to potential security threats.
Next, I'll delve into the cryptographic approach used to generate OTPs, emphasizing its role in enhancing security. I'll also introduce the PyOTP Python library, which simplifies the OTP generation process. A live demonstration using PyOTP will illustrate its functionality.
Following the demonstration, I'll open the floor to audience questions, encouraging a deeper understanding of the topic. In conclusion, I'll stress the significance of adhering to secure OTP generation practices to effectively mitigate security risks.
What format do you have in mind for your talk?
Talk
Table of contents

Introduction
Security Concerns with Random Number Generator OTPs
Cryptographic Approach to OTP Generation
Overview of Different OTP Types
Time-Based OTPs
HMAC-Based OTPs
Introduction to PyOTP Python Library
Features and Functionality of PyOTP
Live Demo: Generating OTPs with PyOTP
Audience QnA

What domain would you say your talk falls under?
Core Python
Duration in minutes (including Q&A)
20
Prerequisites
Basic knowledge of Python.
Speaker bio
I am a Systems Engineer at TCS with a passion for community engagement and open source contributions. I have worked as a full stack developer with Python based frameworks such as Django, Flask etc. and all major JavaScript frameworks. Throughout my career, I've attended, organized, and spoken at numerous community events, sharing insights and experiences with fellow enthusiasts. As an active open source contributor and maintainer, I strive to foster collaboration and innovation within the developer community.
LinkedIn: https://www.linkedin.com/in/sayaksaha10/
Twitter: https://twitter.com/10_sayak
The talk/workshop speaker agrees to

[X] Share the slides, code snippets and other material used during the talk

[X] If the talk is recorded, you grant the permission to release
the video on BangPyper's YouTube
channel
under CC-BY-4.0
license

[X] Not do any hiring pitches during the talk and follow the Code
of
Conduct

Hey @sayakongit Do you want to present this in this month meetup on May 18th?

Hey @sayakongit Do you want to present this in this month meetup on May 18th?

Hey @anistark, yes sure.

Hi @anistark, the meet-up is happening on 18th?

@sayakongit Yes

@sayakongit We had to cancel May meetup but will you be able to present in June meetup?

Hi @anistark, yes I will be available on 22nd of June.

@sayakongit There's a slight scheduling conflict.
Are you available on 15th?

@anistark It would be difficult to manage on 15th. If possible, I can join the following month.

@sayakongit July meetup is on 13th. Will that work for you?

@sayakongit July meetup is on 13th. Will that work for you?

Yes, I'll be available on 13th.

@sayakongit Can you please share your headshot here. It is required for the meetup poster.

@sayakongit Can you please share your headshot here. It is required for the meetup poster.

@sayakongit gentle reminder on this.

@sayakongit is it ok to use your linkedin pic?

Hi @anistark, yes please! You can go ahead with the LinkedIn picture.


Uncaught TypeError: undefined is not a function

[Enter steps to reproduce below:]

Open a Ruby on Rails model

Atom Version: 0.200.0
System: Mac OS X 10.9.5
Thrown From: rsense package, v0.6.0
Stack Trace
Uncaught TypeError: undefined is not a function
At /Applications/Atom.app/Contents/Resources/app.asar/node_modules/q/q.js:128

TypeError: undefined is not a function
    at /Users/girts/.atom/packages/rsense/lib/rsense.coffee:54:32
    at atom-workspace.<anonymous> (/Applications/Atom.app/Contents/Resources/app.asar/src/workspace-view.js:97:22)
    at atom-workspace.handler (/Applications/Atom.app/Contents/Resources/app.asar/src/space-pen-extensions.js:110:34)
    at atom-workspace.jQuery.event.dispatch (/Applications/Atom.app/Contents/Resources/app.asar/node_modules/space-pen/vendor/jquery.js:4681:9)
    at atom-workspace.elemData.handle (/Applications/Atom.app/Contents/Resources/app.asar/node_modules/space-pen/vendor/jquery.js:4359:46)
    at CommandRegistry.module.exports.CommandRegistry.handleCommandEvent (/Applications/Atom.app/Contents/Resources/app.asar/src/command-registry.js:238:29)
    at CommandRegistry.handleCommandEvent (/Applications/Atom.app/Contents/Resources/app.asar/src/command-registry.js:3:61)
    at CommandRegistry.module.exports.CommandRegistry.dispatch (/Applications/Atom.app/Contents/Resources/app.asar/src/command-registry.js:153:19)
    at TextEditorView.jQuery.fn.trigger (/Applications/Atom.app/Contents/Resources/app.asar/src/space-pen-extensions.js:64:23)
    at TextEditorView.module.exports.TextEditorView.afterAttach (/Applications/Atom.app/Contents/Resources/app.asar/src/text-editor-view.js:178:19)

Commands
     -0:04.6.0 fuzzy-finder:toggle-file-finder (atom-text-editor.editor.is-focused)
     -0:02.0 core:confirm (atom-text-editor.editor.mini.is-focused)

Config
{
  "core": {}
}

Installed Packages
# User
highlight-selected, v0.9.3
language-csharp, v0.5.0
rsense, v0.6.0
ruby-block, v0.3.3
ruby-slim, v0.2.0

# Dev
No dev packages

Any progress on this ?


Indonesia translation

Description
This Pull Request add Indonesia translation to the app. It uses base English file and was ensured to be as understandable as possible by a native Indonesian speaker.
Linked Issues
None
Additional context
The Indonesian language support make updates to the following files:

public/locales/es/common.json
Add Indonesian translation to common.json.
src/components/SwitchLang.tsx
Add Indonesian to the language switcher.
next-i18next.config.js
Add Indonesian to the locales line.

Thanks! I'll review this a bit later :)

Thank you!


Ignore unschedulable nodes for eviction decisions

This commit makes descheduler ignore the unschedulable nodes from
being considered for taking eviction decisions

Can one of the admins verify this patch?

@containscafeine  You need to fix the test case and also please look at the new flag node-selector https://github.com/kubernetes-incubator/descheduler/pull/33, through which you can ignore certain nodes based on labels.
Also, there is a plan in upstream community to replace UnSchedulable field with unschedulable  taint and in descheduler we plan to have taint based descheduling.

@ravisantoshgudimetla ack, fixed the failing test.
Does it make sense that till the time spec.unschedulable is removed, we use it in descheduler?

@containscafeine

Does it make sense that till the time spec.unschedulable is removed, we use it in descheduler?

A readynode in descheduler's context means we have a node on which descheduler could run. Even if the node is unschedulable, it could have some pods running(unless it has been drained) which can cause it to be over highUtilization threshold. In this  case, descheduler should run against it.
IOW, descheduler is just a pod evictor. For eviction, we don't need to check if the node is unschedulable.
If you don't want node to be considered by descheduler, you should label the node and ask it to be excluded by descheduler.

A readynode in descheduler's context means we have a node on which descheduler could run. Even if the node is unschedulable, it could have some pods running(unless it has been drained) which can cause it to be over highUtilization threshold. In this case, descheduler should run against it.

Yes that was the intention to consider unschedulable but ready nodes. However, for LowNodeUtilization strategy, if unschedulable nodes are under utilized, we must ignore them as pods wont be scheduled on them.  IOW unschedulable nodes must only be considered for eviction if they are above target utilization but we must not consider unschedulable nodes if they are under low utilization threshold. If it is not the case, we should fix the LowNodeUtilization strategy.

Ok after reading #42, I understood @containscafeine wants to do here but I believe the change in code should be in utilization strategy file. I will send a PR soon.

Yay, thanks @ravisantoshgudimetla. Closing in favor of #45


[AIRFLOW-2835] Remove python-selinux

This package is not used and it sometimes breaks the CI because it is not available. Therefore it makes sense to just remove it :-)
Example failed builds on the master branch:
https://travis-ci.org/apache/incubator-airflow/jobs/410483664
https://travis-ci.org/apache/incubator-airflow/jobs/410483665
https://travis-ci.org/apache/incubator-airflow/jobs/410484305
Make sure you have checked all steps below.
JIRA

[x] My PR addresses the following Airflow JIRA issues and references them in the PR title. For example, "[AIRFLOW-2835] My Airflow PR"

https://issues.apache.org/jira/browse/AIRFLOW-XXX
In case you are fixing a typo in the documentation you can prepend your commit with [AIRFLOW-2835], code changes always need a JIRA issue.

Description

[x] Here are some details about my PR, including screenshots of any UI changes:

Tests

[x] My PR adds the following unit tests OR does not need testing for this extremely good reason:

Commits

[x] My commits all reference JIRA issues in their subject lines, and I have squashed multiple commits if they address the same issue. In addition, my commits follow the guidelines from "How to write a good git commit message":

Subject is separated from body by a blank line
Subject is limited to 50 characters
Subject does not end with a period
Subject uses the imperative mood ("add", not "adding")
Body wraps at 72 characters
Body explains "what" and "why", not "how"

Documentation

[x] In case of new functionality, my PR adds documentation that describes how to use it.

When adding new operators/hooks/sensors, the autoclass documentation generation needs to be added.

Code Quality

[x] Passes git diff upstream/master -u -- "*.py" | flake8 --diff

Codecov Report

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

@@           Coverage Diff           @@
##           master    #3673   +/-   ##
=======================================
  Coverage   77.51%   77.51%
=======================================
  Files         205      205
  Lines       15751    15751
=======================================
  Hits        12210    12210
  Misses       3541     3541

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ed97204...ed2a781. Read the comment docs.


Development flow issue.

Hi, dear contributors. I want to add some new matchers such as Shoulda. But specs aren't work correctly. Its passed only on mongoid 7 without errors.
A list of errors I encountered:

Rubocop not working with ruby 2.6.* and higher. Need to update gem version and yml file, i think.

Mongoid version 8 required ruby 2.6 and higher. And Specs are crashed. With unexpected errors.

Specs working only on mongoid-7.gemspec. What i am doing wrong?

Thanks!

You're not doing anything wrong! We haven't had any work on this library for over a year, Travis-CI went MIA, so we need to do some work here. Please contribute a replacement for CI using GHA and we can fix all this, you can inspire yourself from other mongoid projects like https://github.com/mongoid/mongoid-grid_fs which have upgraded.

@dblock Hello Daniel. I was open PR with GHA changes. I need help how to prepare to merge PR #244.
Any help or comment in completing this PR would be appreciated.

Closed by #244.


Provide ability to use GIT egui

I have a situation where I would like to use a Table widget.  This is not built into the released version of egui right now, but seems as though a recent merge to master has provided some basic functionality.  Is there a way (without forking bevy_egui) that I can get bevy_egui to use the GIT version of egui?  Right now it seems as though it is hard coded.
See discussion at https://github.com/emilk/egui/issues/296 and https://github.com/emilk/egui/pull/963 for more details.
Possibly adding a feature flag for a specific git branch of egui would be nice.

Hi @justacec! I haven't looked into the most recent Egui changes in git yet, but there's a chance that the current version of bevy_egui is compatible with it. To make your project use the git version, you can put the following into your Cargo.toml:
egui = { git = "https://github.com/emilk/egui.git" }

Let me know if that works!

Humm.  Looks like it did not work:
bash-3.2$ cargo build
    Updating git repository `https://github.com/emilk/egui.git`
error: failed to select a version for `egui`.
    ... required by package `bevy_egui v0.13.0`
    ... which satisfies dependency `bevy_egui = "^0.13"` (locked to 0.13.0) of package `FirmwareClient v0.1.0 (/Users/justaceclutter/Desktop/Topics/RocketTracker/FirmwareClient)`
versions that meet the requirements `^0.17` (locked to 0.17.0) are: 0.17.0

the package `bevy_egui` depends on `egui`, with features: `convert_bytemuck` but `egui` does not have these features.

Seems that the egui author refactored the crates features and changed convert_bytemuck to just bytemuck.

I tried to make a quick upgrade to the git version but stumbled upon some other breaking changes and new features that aren't quite trivial to implement. Unfortunately, I don't have enough time to work on that at the moment. But I'll try my best to release a new version of bevy_egui as soon as the new version of Egui lands on crates.io.

:). Thanks for looking into it.  It's ok.  I have things working as of now.  But not with GIT bevy.
No need to change up your flow for me.  :)

Ok, I figured I won't include PaintCallback support in the release, as I have no idea how to make it work with Bevy. Here's the branch supporting the new version: https://github.com/mvlabat/bevy_egui/pull/99

Just got done making the adjustments in my Cargo.toml file:
bevy_egui = { git = "https://github.com/mvlabat/bevy_egui.git", branch = "egui-0.18"}

[patch.crates-io]
egui = { git = "https://github.com/emilk/egui.git" }

Worked like a charm.  Now to go play with the newer table stuff in egui.
Closing this issue.  Thanks again!


Whitespace blocks have different background color than the default background color on dark themes

Issue Type: Bug

Set the editor theme to a dark theme different than the default one (Nord, Material Theme Palenight, Night Owl tested).
Open a text file
Notice how the white space blocks (not between words, but empty blocks of whitespace, at the end of lines for example) have a different background color than the background of words. When selecting a line (thus making the line highlight appear), the line highlight intersects the different background color area and makes that block dissapear.

VS Code version: Code 1.47.0 (d5e9aa0227e057a60c82568bf31c04730dc15dcd, 2020-07-09T08:30:34.302Z)
OS version: Linux x64 5.4.0-40-generic

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-7700HQ CPU @ 2.80GHz (8 x 3559)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: disabled_offprotected_video_decode: unavailable_offrasterization: disabled_softwareskia_renderer: disabled_off_okvideo_decode: unavailable_offviz_display_compositor: enabled_onviz_hit_test_surface_layer: disabled_off_okwebgl: enabledwebgl2: enabled

Load (avg)
1, 1, 1

Memory (System)
15.52GB (7.39GB free)

Process Argv
--no-sandbox --unity-launch

Screen Reader
no

VM
0%

DESKTOP_SESSION
ubuntu

XDG_CURRENT_DESKTOP
Unity

XDG_SESSION_DESKTOP
ubuntu

XDG_SESSION_TYPE
x11

Extensions (19)

Extension
Author (truncated)
Version

gitlens
eam
10.2.2

vsc-material-theme
Equ
32.8.0

vsc-material-theme-icons
equ
1.1.4

code-runner
for
0.11.0

latex-workshop
Jam
8.11.1

csharp
ms-
1.22.1

python
ms-
2020.6.91350

cpptools
ms-
0.28.3

wordcount
ms-
0.1.0

java
red
0.64.1

code-spell-checker
str
1.9.0

latex-utilities
tec
0.3.7

vscodeintellicode
Vis
1.2.9

vscode-java-debug
vsc
0.26.0

vscode-java-dependency
vsc
0.10.2

vscode-java-pack
vsc
0.9.1

vscode-java-test
vsc
0.23.0

vscode-maven
vsc
0.22.0

markdown-all-in-one
yzh
3.1.0

(2 theme extensions excluded)

![Screenshot from 2020-07-12 13-30-11](https://user-images.githubusercontent.com/49650267/87245205-d8f4db80-c443-11ea-9872-3de2854ce9a2.png)

I could reproduce the issue on both the .tar.gz version and the .deb version, I do not seem to be able to reproduce it on the snap version. This has been persisting for a few months. The default dark theme does not seem to be affected, but that can also be because the background is simply dark and a difference is harder to see. Thanks for the great work and hope this gets resolved soon!

/duplicate of #93977 or #85452
See https://github.com/microsoft/vscode/issues/93977#issuecomment-614459402 or https://github.com/microsoft/vscode/issues/85452#issuecomment-558334562 for current workarounds.


Horizontal result display

Is there any method to display the selected items horizontally?
selectedValueWidgetFn always return vertical items display.

Hello @denniswhchung ,
Yes, there is a way. You can use the selectedAggregateWidgetFn parameter as in the following example:
https://github.com/lcuis/search_choices/blob/48d3fb3071e8d3ede7b9749dc140ce5ff7cd0a70/example/lib/main.dart#L909
SearchChoices.multiple(
        items: items,
        selectedItems: selectedItemsMultiDialogWithCountAndWrap,
        hint: "Select items",
        searchHint: "Select items",
        onChanged: (value) {
          setState(() {
            selectedItemsMultiDialogWithCountAndWrap = value;
          });
        },
        isExpanded: true,
        selectedValueWidgetFn: (item) {
          return (Container(
            margin: const EdgeInsets.all(15.0),
            padding: const EdgeInsets.all(3.0),
            decoration:
                BoxDecoration(border: Border.all(color: Colors.blueAccent)),
            child: Text(
              item,
              overflow: TextOverflow.ellipsis,
            ),
          ));
        },
        selectedAggregateWidgetFn: (List<Widget> list) {
          return (Column(children: [
            Text("${list.length} items selected"),
            Wrap(children: list),
          ]));
        },
      )

The result of this example can be seen under "Multi dialog with count and wrap" (last item as I write those lines) in the web demo site:
https://searchchoices.jod.li/

I believe that you can use another solution for horizontal display than wrap through this selectedAggregateWidgetFn parameter.

Closing for inactivity. Feel free to reopen.


DST settings

DST settings support :)

Isn't this a duplicate of #144?
What kind of support do you expect? Switching DST on and off is trivial to do at the user level by simply adjusting the time by ±1h:
static inline void startDST() {
    rtc.adjust(rtc.now() + TimeSpan(3600));
}

static inline void stopDST() {
    rtc.adjust(rtc.now() - TimeSpan(3600));
}

It is then up to you to provide an appropriate UI for the end user to enable/disable DST. You may want to save the current DST status in NVRAM or EEPROM in order to show it on that UI.
Automatic handling of DST is trickier. If this is what you want, I suggest you keep the RTC in UTC, and convert to local time when needed:
DateTime localTime = utcToLocal(rtc.now());

This is how DST is handled in Unixish systems.
If you only care about the current EU rules, you may adapt this function I once submitted to the avr-libc maintainers:
int eu_dst(const time_t * timer, int32_t * z)
{
    uint32_t t = *timer;
    if ((uint8_t)(t >> 24) >= 194) t -=<PHONE_NUMBER>U;
    t = (t + 655513200) / 604800 * 28;
    if ((uint16_t)(t % 1461) < 856) return 3600;
    else return 0;
}

Keep in mind that avr-libc uses the 2000.0 epoch. You will have to adapt that to the 1970.0 epoch used by DateTime::unixtime(). There is already a macro in the library for the offset: SECONDS_FROM_1970_TO_2000.

Duplicate of #144, this is better left to the user or a separate library.


if/else closures with return type removes curly brackets

Describe the bug
if/else in closure with return type removes required curly brackets. This doesn't seem to be too uncommon to me but I didn't find any issues for this bug. Sorry if this is already known, if so please close this issue and refer to the original.
To Reproduce
fn main() {
    let s: String = "ABAABBAA".chars().map(|c| -> char {
        if c == 'A' {
            '0'
        } else {
            '1'
        }
    }).collect();

    println!("{}", s);
}

After running rustfmt on this code, it becomes
fn main() {
    let s: String = "ABAABBAA"
        .chars()
        .map(|c| -> char if c == 'A' { '0' } else { '1' })
        .collect();

    println!("{}", s);
}

This is no longer possible to compile and gives the following error:
error: expected `{`, found keyword `if`
 --> src/main.rs:4:26
  |
4 |         .map(|c| -> char if c == 'A' { '0' } else { '1' })
  |                          ^^------------------------------
  |                          |
  |                          expected `{`
  |                          help: try placing this code inside a block: `{ if c == 'A' { '0' } else { '1' } }`

error: aborting due to previous error

The following example will not break the code:
fn main() {
    let s: String = "ABAABBAA"
        .chars()
        .map(|c| -> char {
            if c == 'A' {
                return '0';
            }

            '1'
        })
        .collect();

    println!("{}", s);
}

Expected behavior
The code to be formatted, but still working
Meta

rustfmt version: rustfmt 1.4.24-stable (eb894d53 2020-11-05)
From where did you install rustfmt?: rustup
How do you run rustfmt: rustftm <filename> or via Rust vim plugin

Backported in v1.4.30


[FIX ]Update CITATION and add Louis

@LouClement does it look ok for you? Are the details correct?


Sorted semesters in descending order in create-review.tsx

Semesters now sort in descending order in Create Review per beta tester feedback:

Pulled down branch and features LGTM, thank you!