Code section turns black font color after Publish

This issue has been moved from a ticket on Developer Community.

Whenever I do a publish from a Blazor Server project the entire Code section of the razor page turns to black font color afterwards. All of the opened razor pages will have its code section turn to black. My only remedy is to close the page and open it back again which is a hassle.

Original Comments
Visual Studio Feedback System on 5/25/2020, 02:57 AM:
We have directed your feedback to the appropriate engineering team for further evaluation. The team will review the feedback and notify you about the next steps.

Original Solutions
(no solutions)

This is fixed in the latest VS2022. Thank you for the bug report!


✨ [REQUEST] BirdNET-Pi Improved support for standalone usage outside of Home Assistant

Checked

[X] If a new addon, I have checked on  Google that such as addon doesn't already exists
[X] If a new feature, I have searched in the discussions & issues that it was not already covered

Which addon?
Birdnet-Pi
Is your feature request related to a problem? Please describe
tl;dr - It took too much tinkering to get BirdNET-Pi running in Docker outside of Home Assistant.
I got the container running on a Docker installation outside of Home Assistant, but there was definitely some tinkering. The tinkering was due to some places where it seems to be tied to Home Assistant's ingress architecture and some other cases of configurations not coming out right when running outside of Home Assistant. There's no documentation indicating this is a supported setup other than https://github.com/mcguirepr89/BirdNET-Pi/issues/211#issuecomment-2081383618. I filed this as a feature request since it is only implied this deployment pattern is a feature, but the description is going to read more like a bug report.
Describe the solution you'd like

Some minimal documentation for the deployment process / configuration. An example compose file with the most critical settings defined and an indication of which ports are expected to be used in this deployment pattern.
nginx and avahi should not throw errors or should otherwise be disabled if not necessary or desired.
Various BirdNet-Pi services should auto-start after boot, if enabled.

Here's some context from my tinkering. I will add more to this issue if I dig deeper.
Additional context

Site HTML

The title is <title>buildkitsandbox</title>. Not sure if this is from the upstream or introduced in the containerization / customization.

nginx

It is unclear if nginx should be the point of entry when outside of Home Assistant. Nonetheless, tokens in the ingress.conf were not replaced properly and nginx fails to start. 91-nginx_ingress.sh seems to test for $ingress_entry but   Example locations follow.

https://github.com/alexbelgium/hassio-addons/blob/a71764f5e21e850047868510aef4f4f8488775d1/birdnet-pi/rootfs/etc/nginx/servers/ingress.conf#L2
https://github.com/alexbelgium/hassio-addons/blob/a71764f5e21e850047868510aef4f4f8488775d1/birdnet-pi/rootfs/etc/nginx/servers/ingress.conf#L37-L44

I solved by disabling nginx and using the Caddy ports directly instead.

avahi

Fails to start. Have not investigated why. Temporarily solved by disabling it as I don't need its functionality.

Services

I've never deployed BirdNET-Pi on a Pi so I don't know if services likst Chart Viewer, Spectrogram Viewer, Live Audio Stream, etc. are intended to start automatically on boot, but from what I can tell, they do not start automatically or if they do, there is an appreciable time delay after boot before they start. I've clicked Enable on all these to get them running as I explored the application but it has always been a manual process after boot.

Thanks for your comment! Indeed my love docker image is currently very much linked to ha. I'll see to make an image that would support both ha and standalone building on your experience


The pf-c-switch renders awkwardly if --pf-global--FontSize--md variable is changed from 1rem

We noticed in the OpenShift console that when changing the --pf-global--FontSize--md value caused  the .pf-c-switch to not render correctly, since it uses that value to calculat  --pf-c-switch--Height.
https://github.com/patternfly/patternfly-next/blob/4a2dd6b099c3b1625dd1a37992cb6f7fbb83a8e3/src/patternfly/components/Switch/switch.scss#L3

:tada: This issue has been resolved in version 2.22.2 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Remove unused field xhrBarrier.passed

https://github.com/DevExpress/testcafe/blob/master/src/client/core/xhr-barrier.js#L23

Also remove the xhrBarrier._removeXhrFromQueue method.
https://github.com/DevExpress/testcafe/blob/master/src/client/core/xhr-barrier.js#L49


我的提问被langchain修改了（langchain change my question）

Issue you'd like to raise.
我通过langchain向openai进行提问，langchain会将我的提问与我的聊天历史记录合并为一个新的问题，然后再向openai进行提问。但是新的提问有时候与我旧的提问相差不小。
比如说我问langchain：你好
langchain提问openai：How can I assist you today?
所以我有的时候会得到一些奇怪的回答。为什么会这样以及我该怎么处理呢？
(
i talk to openai by langchain, but langchain will trans my question to a new question by my question and chat history. sometimes i got strange answer! for example
i talk to langchain: hello
langchian talk to openai: How can I assist you today?
so, i got strange answer semms unrelated to my origin question. why langchain do this,and how can i fix this ?
)

Suggestion:
No response

合并是肯定的，gpt应该会结合上下文回复最后一个问题。

合并是肯定的，gpt应该会结合上下文回复最后一个问题。

为什么要合并生成新的问题呢？ 有的时候生成的新的问题，与我的提问完全不相干，导致对话错乱了。为何不把历史记录原样的传给openai，而是要造一个新的问题

合并是肯定的，gpt应该会结合上下文回复最后一个问题。

为什么要合并生成新的问题呢？ 有的时候生成的新的问题，与我的提问完全不相干，导致对话错乱了。为何不把历史记录原样的传给openai，而是要造一个新的问题

I think the merging of conversations has created new problems because you are using ConversationalRetrievalChain to query vector databases based on question similarity.
example:
human： I want to buy
ai:            What do you want to buy?
human:   A car

If you search using all historical records, it is easy to get incorrect answers.
The results obtained from searching in the vector database will be worse as your search history becomes more and more.
If you use the merge history and generate new issues from it, like this:
new question: I want to buy a car.

You can easily and accurately find similar paragraphs from the vector database.
I also encountered a similar problem once before, but it seems some error from openai and I haven't encountered it again.

合并是肯定的，gpt应该会结合上下文回复最后一个问题。

为什么要合并生成新的问题呢？ 有的时候生成的新的问题，与我的提问完全不相干，导致对话错乱了。为何不把历史记录原样的传给openai，而是要造一个新的问题

I think the merging of conversations has created new problems because you are using ConversationalRetrievalChain to query vector databases based on question similarity.
example:
human:  I want to buy
ai:  what do you want to buy?
human:   a car

If you search using all historical records, it is easy to get incorrect answers.
The results obtained from searching in the vector database will be worse as your search history becomes more and more.
If you use the merge history and generate new issues from it, like this:
new question: I want to buy a car.

You can easily and accurately find similar paragraphs from the vector database.
I also encountered a similar problem once before, but it seems some error from openai and I haven't encountered it again.

为了提高准确率而去改变用户的提问，这个感觉有点奇怪。

每次对话没开始多久就会变得很无法理解

同感，体感确实很不好

已收到，谢谢！


eth scripts: add traps

Please could you explain this one more, @anthony-cros?   Thanks!

the idea is to use bash traps for a cleaner error handling mechanism. maybe overkill for a shell script but well, it wouldn't hurt to have that in place!

What elements of the build process in particular do you think would benefit from trapping?

basic things like my "grep" sanity checks for instance, trying to capture the failure (because the "-e" bash flag will make the program stop upon failure for grep to find a match) and displaying a more palatable error message

Gotcha.


Scan - Request count in expense preview is reduced by one when there is a scanning request

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 1.4.49-0
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail: N/A
Issue reported by: Applause - Internal Team
Action Performed:

Admin and employee are in the same workspace

Go to staging.new.expensify.com
[Admin] Open workspace chat with employee
[Employee] Create a manual request
[Employee] Create a scan request
[Employee] Create another manual request
[Admin] Click on the expense preview
[Admin] Return to main workspace chat via header subtitle

Expected Result:
The expense preview will display "2 requests, 1 scanning" because there are 2 manual requests and 1 hidden request
Actual Result:
The expense preview will display "1 request, 1 scanning" when there are 2 manual requests and 1 hidden request
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence

https://github.com/Expensify/App/assets/78819774/a67fa536-2900-40d5-82a6-46296cd82203
View all open jobs on GitHub

We think that this bug might be related to #wave5
CC @dylanexpensify

@mallenexpensify  FYI I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors

Proposal
Please re-state the problem that we are trying to solve in this issue.
Scan - Request count in expense preview is reduced by one when there is a scanning request
What is the root cause of that problem?
In the ReportPreview action, the reportPreviewAction?.childMoneyRequestCount is used to calculate the numberOfRequests in the getNumberOfMoneyRequests.
The numberOfRequests is one less for the recipient because of the scanning receipt which can be used.
https://github.com/Expensify/App/blob/9302ade9988b1cba3e07b48f474bc35a6deea7fb/src/components/ReportActionItem/ReportPreview.tsx#L119
That is why, in the MoneyRequestPreview, the number of money request actions is on two money requests instead of three.
https://github.com/Expensify/App/blob/9302ade9988b1cba3e07b48f474bc35a6deea7fb/src/components/ReportActionItem/ReportPreview.tsx#L145-L147
What changes do you think we should make in order to solve the problem?

When calculating the numberOfRequests to display, we should account for the number of scanning receipts that the user can create or edit.
Therefore, we will use the numberOfCreatedScanningReceipts to subtract from numberOfScanningReceipts.
Psuedo-code
numberOfScanningReceipts: numberOfRequests - (numberOfScanningReceipts - numberOfCreatedScanningReceipts) - numberOfPendingRequests

Proposal
Please re-state the problem that we are trying to solve in this issue.
Scan - Request count in expense preview is reduced by one when there is a scanning request
What is the root cause of that problem?
In the ReportPreview action, the reportPreviewAction?.childMoneyRequestCount is used to calculate the numberOfRequests in the getNumberOfMoneyRequests.
The numberOfRequests is one less for the recipient because of the scanning receipt which can be used.
https://github.com/Expensify/App/blob/9302ade9988b1cba3e07b48f474bc35a6deea7fb/src/components/ReportActionItem/ReportPreview.tsx#L119
That is why, in the MoneyRequestPreview, the number of money request actions is on two money requests instead of three.
https://github.com/Expensify/App/blob/9302ade9988b1cba3e07b48f474bc35a6deea7fb/src/components/ReportActionItem/ReportPreview.tsx#L145-L147
What changes do you think we should make in order to solve the problem?

When calculating the numberOfRequests to display, we should account for the number of scanning receipts that the user can have not create or edit.
Therefore, we will create a constant number called numberOfScanningReceiptsNotCreated to subtract from numberOfScanningReceipts to get the number of scanned receipts created by the user.
Psuedo-code
numberOfScanningReceipts: numberOfRequests - (numberOfScanningReceipts - numberOfScanningReceiptsNotCreated) - numberOfPendingRequests

I was able to reproduce .  @shubham1206agra , can you also please try then review the proposal above? Thx


Template demo-store improvements

Collecting feedback for demo-store improvements.
### Tasks
- [ ] Add email/contact us form

@tkapitein We purposely left it out for now. We will be adding it int he future together with some docs.

I'd like the following:

Pagination, comments and excerpt for the blog.
Checkout screen that is part of the demo store - not a link back to the Shopify Frontend.

@Nistrul Pagination is something we are actively thinking about as well. For comments and excerpt, what are you looking for there? Would love to understand your use-case as best as possible.


Custom event for processing needed

Hello,
old Android browsers usually have render issues while applying the viewport units hack.
For that reason I like to hide parts and show them once the hack was completed. Can you provide a custom event like viewportUnitsBuggyfillPatched that triggers on window? Maby other events are usefull too?
At the moment I obeserve the #patched-viewport what is kind of bad.

$('#patched-viewport').on('DOMNodeInserted', function () {
    alert('viewportUnitsBuggyfillPatched');
});

Thank you

You're welcome to send a pull request for this. You'd likely want a CustomEvent called viewport-units-buggyfill-style to be emitted in the updateStyles() method.

All right, will put this on my todo list.

Pull request send, feel free to extend it with more :-)

released v0.5.5


support custom commands in simple chat

Close https://github.com/sourcegraph/cody/issues/2149 https://github.com/sourcegraph/cody/issues/2140

Add custom commands to simple chat

user can loop through available commands in their chat input box by entering /

Replace prompt for each commands with command name
Add file link to end of display text if user has selected code in current file
Add input history to simple chat

user can loop through their input history in the chat input box by pressing up and down

Remove History view that we no longer use

https://github.com/sourcegraph/cody/assets/68532117/64898186-9c7b-43d8-8b7b-dec7abdf2713
Test plan

[ ] Run any commands from context menu, sidebar menu, command palette etc
[ ] Panel title is updated accordingly with command name followed by file name
[ ] typing / in chat input box will display the list of slash command
[ ] you can run slash commands from chat input box
[ ] Can run multiple commands in different panels at once
[ ] press up and down in empty chat box will loop through your input history

Seems like this also fixes #2140 :)


Feature/assets

Coverage remained the same at 91.818% when pulling bc977ea1aa06a420383bf4ecf2b6044570e85d7d on feature/assets into 551a3f101c350d2d7780f9db492680dc2bd47917 on develop.


export with an empty folder

i updated to sketch Version 3.5 (25232) and this is what isee:

me too :/

I'm very anxious!!! because I am on holiday.

Oh no! and github does not have a 'on holiday' button :/ enjoy! I will poke
in the code to see if it is something trivial
On Tue, Jan 26, 2016 at 5:22 PM utom<EMAIL_ADDRESS>wrote:

I'm very anxious!!! because I am on holiday.
—
Reply to this email directly or view it on GitHub
https://github.com/utom/sketch-measure/issues/81#issuecomment-175328560.

Is there a solution to this! Need it urgently!

same problem here :/ if anyone know a solution feel free to share :dancers:

Tried to poke in the code but I couldn't figure how to debug...
On Thu, Jan 28, 2016 at 3:01 AM João<EMAIL_ADDRESS>wrote:

same problem here :/ if anyone know a solution feel free to share [image:
:dancers:]
—
Reply to this email directly or view it on GitHub
https://github.com/utom/sketch-measure/issues/81#issuecomment-176124157.

fixed!

Still won't work for me!

@pinakeenaik what's sketch version

Version 3.5.1 (25234)

try download plugin last version

Thanks a tonne!


Angular: Unexpected behavior when updating slides if BrowserAnimationsModule is imported

Swiper Version: 6.4.10.
Platform/Target and Browser Versions: MacOS, Chrome.

What You Did
If BrowserAnimationsModule is imported into the application, dynamically replacing the slides leads to unexpected behavior: seemingly empty slides appear at the end, wrong slide count, etc.
Reproduction:
https://stackblitz.com/edit/angular-swiper-browseranimationsmodule-issue?file=src/app/app.module.ts

Replace the slides by clicking on the "replace slides" button.
Remove the imported BrowserAnimationsModule in app.module.ts and retry to get the expected behavior.

Expected Behavior
Update slides properly regardless of the presence of BrowserAnimationsModule.
Actual Behavior
Empty slides appear at the end (although not in the DOM), the slide count is off, the slides collection in the swiper instance contains more items than expected...

I have this problem now. PR not helped?


Unfork

I suggest unforking from mattetti/i18n, to make it clear that this is the canonical place for the gem (forked from mattetti/i18n can cause confusion about that).
It's pretty simple to do, simply contact github and tell them you'd like to unfork, and explain why.
cc @radar

This has always been the canonical place for it. It's got more stars, and more recent commits.
I'll still get in touch with GH and see about it.

GH says that the request needs to come from @svenfuchs.

friendly ping @svenfuchs

It could be easier to write an email directly to @svenfuchs via<EMAIL_ADDRESS>

I'm already in direct contact with @svenfuchs and do not want to be harassing him during this time. He can attend to it when he wishes.

Given the staleness of this open issue, I'd suggest forking/moving this project so that the current maintainer has control (e.g., @radar/i18n).

What benefit would that have?
On 12 Jul 2018, 06:59 +0930, Will Jordan<EMAIL_ADDRESS>wrote:

Given the staleness of this open issue, I'd suggest forking/moving this project so that the current maintainer has control (e.g., @radar/i18n).
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

What benefit would that have?

The benefit discussed in this issue, for one. Given the project can't be unforked without the (inactive) project owner's participation, the alternative is to migrate the project to the current maintainer's ownership.
Another benefit is that it would reduce confusion regarding the current project owner/maintainer.
There are probably other benefits related to other permissions the project owner uniquely holds that can't be delegated to non-owner users with commit privileges.

Right. I can understand that. I would rather that the project not be linked to a single person's account as that makes it seem like i18n is only maintained -- and can only ever be maintained -- by one person. I would much rather have an organisation account for this.
I'm already paying for one: @rubysherpas. We could move it there. I've already done this with https://github.com/rubysherpas/paranoia, so at least there's a precedent for it. I'll ask @svenfuchs to move it, since he's the only one with access currently.

I'd prefer to use https://github.com/ruby-i18n for this, and I've emailed GitHub about unforking the repo.

GitHub has unforked the repository.
🎉

I'm +1 on moving this to ruby-i18n. Let's do it :+1:

Hi @sandstrom @radar now we're unforked, can we move the repo to ruby-i18n? (which already exists, btw)

I do not have the ability to do that. Only @svenfuchs as the repo's admin has that power.

Thanks. Have pinged Sven via email.


⚠ Bofrost.be has degraded performance

In 2fbb2a3, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1104 ms

Resolved: Bofrost.be performance has improved in a742d5c after 6 minutes.


Cancel Hints

Hi,
Could it be possible to add a keybinding to cancel the hints. I know that escape works, but if you in full screen mode in safari escape also cancels full screen.
Cancelations of hints is wanted because many times a clickable element isn't discovered. In that case you have to cancel the hints and use the mouse/trackpad instead

Having the same problem with Esc and Full screen mode


Oauth2 clientCredentials: client-id/client-secret rendering

Some days ago I discovered Oauth2 PR from @fbacker
(this fork https://github.com/fbacker/RapiDoc/tree/feature/oauth2)
This feature does the useful render and processing of client-id/secret in Authentication block

Today PR is gone :) and in the current release no essential clientCredential flow support added - any plans to do that?
P.S. Thanks for the great project, BTW!

Yes there are plans to support other OAuth flows.
In 7.5.0 release  we provided Authorization Code flow.
currently we are receiving feedbacks on UI/UX, how easy /terrible it is to set it up. will add little bit of documentation on how to set it up
If things go fine
password and client-credential flow will follow soon

Feel free to provide your feedback on Authorization Code flow if you are using it

I've tried to adjust my PR a third time to more match mrin9 requests. The latest PR allows for a lot of different types, not all tested but client credentials and authorization code is tested. Sorry to see that this PR is closed as well leaving 7.5.0 with limited oauth support. I need these features myself and are forced to turn back to the swagger ui project.
If you have the time @pavlovmike please try the PR out. Might help you with what you need. https://github.com/mrin9/RapiDoc/pull/163

@fbacker - I'm already in use with your code, client cred flow works fine.
For now, I have only one comment - maybe it's better to move Auth procedure block to the left - because of asymmetric view, but it's very IMHO

Also missing some kind of errors.
I've made an update that shows errors e.g. 'invalid_client' that is returned from the oauth server. Also changes the button text when making the request so we know that something is happening.

So - we will wait for @mrin9 to make a decision about supporting all the flows in the master branch.
Maybe it's a good idea to make this partially-tested code merged into master under some feature-toggling/optional flag mechanism? Just thoughts about.

Maybe in gui when using not tried oauth flows show a messy/ugly description to provide feedback if it worked or what happend. Then for each type that gets tested / fixed a new release can remove the description for the type.
best would of course someone that could create all different types in an oauth provider and test/develop them.
so a approaches maybe :)

release them all and wait for the community to give feedback
hide the non tested auth flow types and wait until a developer can help out because they need it
release them all with a api toggle to enable untested types

Release 7.5.1 adds client credentials and implicit OAuth flow support
checkout the instructions and example on how to set it up

Instructions
Demo

password flow will follow soon


virtualboxGuest: replace requires/after with bindsTo

Otherwise multi-user.target is not met until 90 seconds of waiting for the device have passed. Looks like ConditionVirtualization is considered too late down the dep graph.

cc @aszlig

LGTM, do you happen to know how to reproduce this in a VM test? Because I'm not sure whether the same also applies for the host services.

Yeah, I actually was initially trying this in a nixos test, I can try submitting a PR later.

You could add this to the existing virtualbox test. Right now it's testing minimal initrd-only guests, so I think it would be nice if we'd build real guest images there.

@aszlig @proger is this still relevant?

ping :)

I suspect this has been fixed in https://github.com/NixOS/nixops/issues/391#issuecomment-223444949, so I'll close this. Feel free to reopen if it's not the same issue.


add Soulmask template

not sure if everything works fine

Looks good but can you validate that it works as intended ?
I do not have the game and do not intend to buy it so it's pretty have for me to validate that it works.

Works for me, and with different Settings. But i have not tested all parameters.


The menu bar at the top disappeared after the update

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

版本: 1.95.3 (user setup)
提交: f1a4fb101478ce6ec82fe9627c43efbf9e98c813
日期: 2024-11-13T14:50:04.152Z
Electron: 32.2.1
ElectronBuildId: 10427718
Chromium: 128.0.6613.186
Node.js: 20.18.0
V8: 12.8.374.38-electron.0
OS: Windows_NT x64 10.0.26100
The menu bar at the top disappeared after the update, replaced by an extra button on the left, how can I revert this change?

What version were you previously on?
Is your window narrow?

I don't  remember, but I always follow the newest version, so I guess it might be 1.95.2, and my window is full-screen

Please run the Customize Layout... command from Command Palette (F1) and review the state of the Menu Bar visibility option.

my window is full-screen

Do you mean you have used the F11 toggle to display VS Code full-screen? In that mode the compact form of the menu bar is always used (aka 'hamburger' menu)

Please run the Customize Layout... command from Command Palette (F1) and review the state of the Menu Bar visibility option.

This works for me, Thanks

That's good. Please close your issue.


Cannot boot v2.26.0+rev1

Hi,
I just tried newly released v2.26.0+rev1 on staging. However the beaglebone isn't able to boot. Uboot keeps printing:
U-Boot SPL 2018.01-geac41e26ac (Nov 23 2018 - 09:38:17)
Trying to boot from MMC1

I did use a dev image, all other complete standard.
Thanks.

Just tried an production image and a couple of different SD-cards, same issue.

@FRVisser this should be fixed now.
We were aware of the issues, staging images never reach production without going through our testing suite.
Feel free to close this once you get a chance to test it

@FRVisser can this be closed now?

Yes, thanks!


Using gomock to mock sigs.k8s.io/controller-runtime/pkg/client If

Description
We want to use gomock everywhere in the Koperator repository and generate mocks through Makefile.
The sigs.k8s.io/controller-runtime/pkg/client Client interface was generated with mockery and it used testify mock.
When interfaces changes because updates we can re-generate mock easily with the "make mock-generate" command.
Type of Change

[x] Refactor

Checklist

[x] All new and existing tests pass

I wasn't sure if there needs to be a call to the Finish method of each new Controller we create.
I saw this saying (just like most examples I find online about gomock):

Each test should create a new Controller and invoke Finish via defer.

And gives these usage examples :
//   func TestFoo(t *testing.T) {
//     ctrl := gomock.NewController(t)
//     defer ctrl.Finish()
//     // ..
//   }
//
//   func TestBar(t *testing.T) {
//     t.Run("Sub-Test-1", st) {
//       ctrl := gomock.NewController(st)
//       defer ctrl.Finish()
//       // ..
//     })
//     t.Run("Sub-Test-2", st) {
//       ctrl := gomock.NewController(st)
//       defer ctrl.Finish()
//       // ..
//     })
//   })

And then the Finish method of a Controller indicates :

It should be invoked for each Controller.

But then the same method later added :

// New in go1.14+, if you are passing a *testing.T into NewController function you no
// longer need to call ctrl.Finish() in your test methods.

So I think we're fine without deferring Finish.


Remittance Reports are an option on Purchase Report Selections but are hardcoded to use standard reports

Hi
As the title says, in the Purchase Report Selections there are Usage options for Vendor Remittance and Vendor Remittance - Posted Entries however when you print the Remittance report from the journal or the Vendor Ledger Entries it is still hardcoded to us the standard reports.
Can the standard code on the Vendor Ledger Entries Page and the Payment Journal Page be modified to use the report in the Report Selections please?
Thanks
Chris

Please clarify what version has this issue? I'm looking into next major release code base and cannot find hardcoding of such reports.

This is fixed for 17.0

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.


How make other postcss plugins affect on tailwind classes

Hello. Here is my nuxt.config:

export default defineNuxtConfig({
  modules: ["@nuxt/content", "@nuxtjs/tailwindcss"],
  postcss: {
    plugins: { "postcss-rtlcss": {} },
  },
});

I have included postcss postcss-rtlcss plugin. it affects on my own defined CSS classes (makes [dir="rtl"] mode for each class), but not on tailwind classes. am I missing something?
Thanks.

Hi, your existing postcss configuration should not be affected (https://github.com/nuxt-modules/tailwindcss/blob/255da816049f62ffbd9ce7bb0d3dd0c750793cb1/src/module.ts#L228) - is there something happening that you don't expect?

@ineshbose, my question is how to make postcss plugins affect tailwind classes. When I use postcss-rtlcss plugin, it generates rtl CSS for each CSS selector. but not works on tailwind classes.
current generated CSS:
.ml-6 {
  margin-left: 1.5rem;
}

But I expect the below, because I used postcss-rtlcss plugin:

[dir="rtl"] .ml-6 {
  margin-right: 1.5rem;
}

I see what you mean!
I think in this case, you're possibly looking for a Tailwind plugin rather than PostCSS plugin. On some digging, there's this discussion - https://github.com/tailwindlabs/tailwindcss/discussions/1492

@alihardan the module configuration should work same as you use Tailwind - if it's not working as desired, please provide a reproduction over Stackblitz and we'll be able to investigate this! :)

@alihardan the module configuration should work same as you use Tailwind - if it's not working as desired, please provide a reproduction over Stackblitz and we'll be able to investigate this! :)

I made a reproduction over Stackblitz that shows the unexpected behavior. As you see in this reproduction, postcss plugin affects my .test class, but not the tailwind ml-4 class.
https://stackblitz.com/edit/nuxt-starter-ckqtyt?file=app.vue

Thanks for the repro 🙂
It seems likely that even though we're using object-syntax for PostCSS plugins instead of array, the order of assignment for object-property is still being followed. So postcss-rtlcss is installed before tailwindcss (as a pcss plugin).
The workaround for this, in your nuxt.config, is:
// https://v3.nuxtjs.org/api/configuration/nuxt.config
export default defineNuxtConfig({
  modules: ['@nuxtjs/tailwindcss'],
  postcss: {
-   plugins: { 'postcss-rtlcss': {} },
+   plugins: {
+      tailwindcss: {},
+      autoprefixer: {},
+      'postcss-rtlcss': {}
+    },
  },
});


feat(cli.js): allow configuring tauri app path on package.json #2752

package.json > tauri > appPath defines the path to the tauri crate and cli.js sets the proper CWD when running cli.rs

What kind of change does this PR introduce? (check at least one)

[ ] Bugfix
[x] Feature
[ ] Docs
[ ] New Binding Issue #___
[ ] Code style update
[ ] Refactor
[ ] Build-related changes
[ ] Other, please describe:

Does this PR introduce a breaking change? (check one)

[ ] Yes. Issue #___
[ ] No

The PR fulfills these requirements:

[x] When resolving a specific issue, it's referenced in the PR's title (e.g. fix: #xxx[,#xxx], where "xxx" is the issue number)
[x] A change file is added if any packages will require a version bump due to this PR per the instructions in the readme.

If adding a new feature, the PR's description includes:

[ ] A convincing reason for adding this feature (to avoid wasting your time, it's best to open a suggestion issue first and wait for approval before working on it)

Other information:

if we are gonna add this, then we should probably add the other way around too.
So package.json has a key to link to tauri.conf.json (the file itself is better than the directory) and the other way around.

if we are gonna add this, then we should probably add the other way around too. So package.json has a key to link to tauri.conf.json (the file itself is better than the directory) and the other way around.

I think the Rust CLI won't even know about Node.js or the frontend application. Currently it only needs this information for tauri info and beforeDev/beforeBuildCommand, and my idea was to allow defining the cwd for the scripts and letting cli.js pass information to cli.rs.

Currently it only needs this information for tauri info and beforeDev/beforeBuildCommand, and my idea was to allow defining the cwd for the scripts and letting cli.js pass information to cli.rs.

yeah you're right, this sounds better.
We also had some reports about some people not able to use the rust cli with a cargo workspace but that's for another PR I guess.

Can you link that issue? Last time I checked it worked fine, maybe it's some special setup.

don't remember exactly, maybe it was this https://github.com/tauri-apps/tauri/pull/2765#issuecomment-962494040 .


Let each model class have their own set of plugins

This fixes a bug where Cibyl crashed when it tried to access a plugin on a model that should have never had it in the first place. With this change, each class that inherits from Model gets its own set of plugins, which means that the Zuul job will remain pluginless and will fail no more.

Before reviewing I want to make sure I understand the change correctly. The problem this fixes is that the two Job classes were sharing the same plugin_attributes and by using the new metaclass we make sure that this does not happen because the dictionaries are created whenever a new class is created, is that correct? If that's right, then I think this is a good solution, because even though it's a bit more complex it makes sure the same problem will not be happening again.
What I don't understand is why the two Job classes sharing the plugin_attributes dict caused a crash.

I'm still unable to run cibyl --jobs
Environment: component_pipeline
INFO     cibyl.orchestrator   Performing query on system osp_zuul
INFO     cibyl.orchestrator   Took 169.96s to query system osp_zuul using source: 'zuul_api' of type: 'zuul'
Traceback (most recent call last):
  File "/home/abregman/.cibyl_venv/bin/cibyl", line 10, in <module>
    sys.exit(main())
  File "/home/abregman/cibyl/cibyl/cli/main.py", line 118, in main
    orchestrator.query_and_publish(arguments["output_style"],
  File "/home/abregman/cibyl/cibyl/orchestrator.py", line 329, in query_and_publish
    self.run_query(system)
  File "/home/abregman/cibyl/cibyl/orchestrator.py", line 293, in run_query
    system.populate(model_instances_dict)
  File "/home/abregman/cibyl/cibyl/models/ci/base/system.py", line 164, in populate
    self.add_toplevel_model(model)
  File "/home/abregman/cibyl/cibyl/models/ci/zuul/system.py", line 71, in add_toplevel_model
    self.tenants[key] = model
  File "/home/abregman/cibyl/cibyl/models/attribute.py", line 86, in __setitem__
    self.value[key] = item
TypeError: list indices must be integers or slices, not str

I was getting that error too yesterday, this is caused by having a tenant list at the system level in the configuration, which is not supported, it should be at the source level

I was getting that error too yesterday, this is caused by having a tenant list at the system level in the configuration, which is not supported, it should be at the source level

Good to know. I would expect it to be handled similarily to what is done in Jenkins:
The following key in "osp_jenkins" system type is not supported: 'tenants'

Check the documentation at https://cibyl.readthedocs.io/en/latest/configuration.html for more information

Again one of those points of making sure we have consistent behaviours across systems and sources

And I can confirm it works for me now. Thanks @cescgina for the 'tenants' comment.
Thank you @jsanemet for fixing this bug.

The reason that the Zuul system does not fail instead of the Jenkins one is that the constructor of the Zuul system accepts a tenant argument, but it's intended to store Tenant models found while querying. While discussing this yesterday with @jsanemet, he suggested verifying the configuration against an schema which should generally solve this kind of bugs, and the task is already in our backlog.

The thing this fixes is that with the previous implementation all models shared the same plugin dictionary. Meaning that if I installed a plugin on the System model, then I would also see them in Build for example. The metaclass intention is to make each type of model have their own plugin dictionary, so Job will have theirs, Build a different... This way there is no longer a clash between the Jenkins and the Zuul job models.


Refactor for nf-core/chipseq release 1.0

Close issue https://github.com/nf-core/chipseq/issues/40

I really think that we should remove the uppmax modules config. It goes against the central pillar of nf-core, that versioned pipelines are reproducible. Not only are the environment modules not stable and reproducible, but we're running a versioned copy of the pipeline with varying software versions, which is super bad.
@chuan-wang - if you want the uppmax modules for testing or a specific reason then it's fine to keep this separately, but I'd really prefer to remove this profile from the pipeline.

Close issue #25

Close issue #36

Still some updates to do before the release, but we can do those in separate PRs. I'm happy with this for now, great work @chuan-wang!

Thanks! /chuan
On Wed, Aug 15, 2018 at 3:50 PM, Phil Ewels<EMAIL_ADDRESS>wrote:

Still some updates to do before the release, but we can do those in
separate PRs. I'm happy with this for now, great work @chuan-wang
https://github.com/chuan-wang!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/nf-core/chipseq/pull/38#issuecomment-413203533, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ARdKNNav6xraH-kp9F1WQPLi4zxXVj5gks5uRCc-gaJpZM4Vz5qn
.


Docs/reusable snippets

Description

Related Issues

Checklist

[ ] Is this code covered by new or existing unit tests or integration tests?
[ ] Did you run make unit-testing and make integration-testing successfully?
[ ] Do new classes, functions, methods and parameters all have docstrings?
[ ] Were existing docstrings updated, if necessary?
[ ] Was external documentation updated, if necessary?

Additional Notes or Comments

Plz remember to clean the outputs

Plz remember to clean the outputs

Thanks! Yes


django.core.exceptions.ImproperlyConfigured exception with the latest 4.0.28 release

Please first make sure you have looked at:

Documentation: https://github.com/mkleehammer/pyodbc/wiki
Other issues

Environment
To diagnose, we usually need to know the following, including version numbers.  On Windows, be
sure to specify 32-bit Python or 64-bit:

Python: 3.7.3
pyodbc: 4.0.28 (latest release)
OS: Windows 10
DB: Microsoft SQL Server 2017
driver: ODBC Driver 17 for SQL Server
Django: 2.2 LTS

Issue
With the above specifications, the latest 4.0.28 release causes the following exception in Django 2.2 LTS version:
django.core.exceptions.ImproperlyConfigured: Error loading pyodbc module: DLL load failed: The specified module could not be found.
I had to downgrade to the previous version of pyodbc 4.0.27 in order to resolve this issue.

Related: #663

@tommckennon
Installing latest "Microsoft Visual C++ Redistributable for Visual Studio 2015, 2017, 2019" shall resolve the issue as confirmed by @gordthompson in https://github.com/mkleehammer/pyodbc/issues/663#issuecomment-569365502.
You may not need to downgrade pyodbc version. Please try and confirm the same!

Closed due to inactivity. Feel free to re-open with current information if necessary.


http3: prepare for HTTP Datagrams support

Commit Message:
This commit prepares Envoy for HTTP Datagrams and the Capsule Protocol (RFC 9297) support. The added CapsuleProtocolHandler is used to decode received HTTP/3 Datagrams into Capsules, and encode Capsules into HTTP/3 Datagrams in the EnvoyQuicClientStream and EnvoyQuicServerStream.
This commit does not enable HTTP Datagrams support in Envoy, so it does not change the behavior of Envoy under any circumstances. A subsequent CL will enable the feature with CONNECT-UDP support.
Additional Description:
Risk Level: Low. The current commit does not change the behavior of Envoy.
Testing: unit tests
Docs Changes: N/A
Release Notes: N/A

/assign @DavidSchinazi
Hi David, can you review the commit in this PR? I left a few questions you could answer in some TODOs.

Exciting!  I'll let @DavidSchinazi do first pas.
Also interested in his and @RyanTheOptimist 's take on if this will eventually be a large enough code path we may want to have a compile-out option for EM binary size

Thanks for pointing it out, Ryan. I will fix the clang-tidy errors as well as the msan/asan ones.

/wait on CI (though you can ping David if you think it's ready for another review pass)

Thanks for the reminder, Alyssa. David will take a look at it soon. I will fix the CI errors in the meantime.

/wait on CI

@jeongseokson GitHub UI/Usability is kinda terrible. But if you reply to comments from the "Files Changed" view instead of the "Conversation" view, you can do "Start Review" in which case all of your comments will be sent in a single email instead of one email per comment. Just FYI

@RyanTheOptimist Ah! I didn't know that. Thanks for the information, Ryan.

@RyanTheOptimist Ah! I didn't know that. Thanks for the information, Ryan.

Sure thing! It's completely undiscoverable. sigh I'm glad we have GitHub but sometimes I find it really annoying.

@alyssawilk Hi Alyssa, I checked this PR passes all CI checks. Added one last commit to change the name of a class, so it should pass all the checks again.
Feel free to take a look at this PR when get a chance. It should not change the behavior of Envoy as I described in the commit message, but please let me know if you have any doubt.

/retest

/wait

/retest

/retest

@mattklein123 you Ok with datagram support 1) being added and 2) being treated like HTTP/3 and getting google-only sign off rather than requiring cross-company LGTMs?

@mattklein123 you Ok with datagram support 1) being added and 2) being treated like HTTP/3 and getting google-only sign off rather than requiring cross-company LGTMs?

Sure SGTM!

/reteest

/retest

/retest


Add info on known- (or thought-)good Office(/Windows?) versions

At this time, only will have been able to test on Win7/Office2010 and Win7/Office2013.

Added to Wiki homepage


Make use of diffHTML's transition handlers to decrease work during updates

Currently when we update the page we remove and re-install every subscribed event handler on every update. However, diffHTML might give us tools to avoid doing this as much. The transition API allows us to install event hooks for when elements are created, replaced, or removed in the DOM by diffHTML, which means we could entirely forego removing event handlers, and instead just make sure we add new event handlers to whichever DOM nodes are created, as appropriately determined by the subscription.
If/when #4 is implemented, this might also be used to speed up DOM traversals to find selected nodes, because diffHTML will hand us the changed element, and we need only traverse that node, rather than always starting at the root of the DOM.

This is obviated by https://github.com/GaloisInc/myxine/pull/17 and is no longer relevant, since we don't remove/replace handlers on every update anymore.


Add pyproject.toml for Custom Node Registry

Hey! My name is HaoHao and I'm from comfy-org! We would love to have you join the Comfy Registry, a public collection of custom nodes which lets authors publish nodes by version and automate testing against existing workflows.
Eventually, the registry will be integrated as a backend for the UI-manager where all nodes will go through a security scan. Nodes that pass these checks will have a verification flag (✅) beside their name on the UI-manager. Feel free to read up more on the registry here
Action Required:

[ ] Go to the registry. Login and create a publisher id (everything after the @ sign on your registry profile).
[ ] Add the publisher id into the pyproject.toml file.
[ ] Merge the separate Github Actions PR, then merge this PR.

If you want to publish the node manually, install the cli by running pip install comfy-cli, then run comfy node publish
Otherwise, if you have any questions, please message me on discord at haohao_81202 or join our server!

hey @wandbrandon, wanted to send a small nudge and ask if you had any questions!

@wandbrandon, wondering if there's any way we can support you on this PR!

Hi! Sorry i haven’t been able to keep up to death with this repo. I don’t think these nodes are complete just yet. Once i make some changes to finalize some of the algos I’ll go ahead and merge what’s needed. Thanks for your patience!

Hi! Sorry i haven’t been able to keep up to date with this repo. I don’t think these nodes are complete just yet. Once i make some changes to finalize some of the algos I’ll go ahead and merge what’s needed. Thanks for your patience!

sweet! you are good!!!


[FreeBSD] Lose all input when tty switch back to sway

As the title says. Probably won't be fixed because of a wlc replacement for sway coming soon, just thought I'd add an issue to share my discovery.

Try out sway for first time. I'm starting it out of tty, mouse works but input doesn't. Switch back and forth between tty and sway doesn't help.
@johalun do you have the reference about when sway will switch from wlc?

mouse works but input doesn't.

wat

I mean, up to now I could only see the mouse pointer moving, any other kind of input has no response, keyboard or mouse clicks.

Also, I'm not on FreeBSD, I'm on a freshly installed ArchLinux.

I had to check /run/user/1000 for the sway socket and export SWAYSOCK=/run/user/1000/sway-ipc.1000.515.sock, otherwise swaymsg always returns "sway socket not detected". Now swaymsg -t get_outputs is returning my HDMI display information correctly, but swaymsg -t get_inputs just returns an empty result.

Had to to that from another tty, since the first is taken by Sway/Wayland.

Ops, really sorry for the noise. Got it working finally \o/. The sole issue was the the use of Mod4 in the default configuration file, it simply isn't working for me yet, changing it to Mod1 (Alt) for now solves it. Also, from termite inside sway get_inputs, etc is working normally.


rework 8bit color encoding

Superseding https://github.com/JuliaImages/ImageInTerminal.jl/pull/62.
Goes with https://github.com/JuliaImages/ImageInTerminal.jl/pull/68.
using ImageInTerminal, OffsetArrays, ImageCore

main() = begin
  wheel = OffsetArrays.centered(fill(ARGB(0, 0, 0, 0), (128, 128)))

  for I ∈ CartesianIndices(wheel)
    x, y = I.I ./ (size(wheel) .÷ 2)
    r = sqrt(x^2 + y^2)
    if r < 1
      h = atand(x, y) + 90
      wheel[I] = RGB(HSV(h, r, 1))
    end
  end

  bar = Gray{N0f8}.(repeat(range(0, stop=1, length=64), 1, 2)')

  for enc ∈ (
    ImageInTerminal.XTermColors.TermColor8bit(),
    ImageInTerminal.XTermColors.TermColor24bit()
  )
    @show enc
    display(enc.(wheel))
    display(enc.(bar))
  end
  return
end

main()

@johnnychen94 could you tell me if you obtain the same images with this PR ?
Your graybar image seems better than mine (https://user-images.githubusercontent.com/8684355/147930881-ef076fb5-3252-4d9b-8f00-11a32fa9648f.png), my gradient is not uniform and I have thin dark lines appearing ...
Output with this PR:
8bit colors

sixel

FWIW, this is what I get in iTerm (macOS) for this branch, it does look like the gray version is getting worse:

I've updated the test script.
Here is the output using the 2 color depths, in a sixel compatible terminal:

Thanks, this means that we should release ImageInTerminal 1.0 instead of 0.5, right ?

1.0 means that API is stable and works. I feel ImageInTerminal is far from stable, e.g., the display, sixel and related are still not well tested.
I personally prefer 0.5, but don't take it too seriously, we have many major release versions to use. Just feel free to make 1.0 release if you like it this way!

Ha, I mismatched semver again, let's make it 0.5. I'll merge this now and we can continue discussions in https://github.com/JuliaImages/ImageInTerminal.jl/pull/68.


Add integration test for saas app property

Need to merge https://github.com/wso2-extensions/identity-inbound-auth-oauth/pull/1298 before merging test case.

@Krishanx92 the build is failing from this PR, can you please check and update the PR?

@Krishanx92 the build is failing from this PR, can you please check and update the PR?

We need to close this since we do not merge framework PR.


Custom patter support for MAC  and IP's

Need to have a way to achieve custom patterns for Mac and Ips using snappi.
I want to generate 10M Ip's using some custom pattern and looks like i can't achieve it using snappi.
Please see the screenshot, I want to have a way the pattern and More options , outlined in yellow and green rectangle

Hi @vkuma82 ,  custom pattern is not available in the models. As an alternative you can generate MACs and IPs with some logic and apply as shown in this example https://github.com/open-traffic-generator/snappi-ixnetwork/blob/main/tests/vxlan/test_vxlan_b2b_scale.py#L203
If you want the support from models, please raise issue in the models repo https://github.com/open-traffic-generator/models/issues


Create Student Page

4/8 Friday Deadline
Please check in with Zoya when done

Create the student page as seen on figma
Communicate with the Create Student & Student Group Modals team to connect the edit student button
Communicate with Zubair for reading & assessment pages and Create Student Group Page for link connections

/cib create-student-page

Branch 90-create-student-page created!


Unable to download Youtube Video and Audio Downloader

Hello,
Mozilla will not allow me to download the Youtube Video and Audio Downloader add-on.  Do you know of a work around for this issue?  It will not even let me open the link to the homepage.

The add-on is unrelated to youtube-dl. Please contact it's authors.


FIX capacity overflow when pack size exceeds int size

If not negative number appear as pack size or total size in TreePackPanel

Thanks for contributing.


I can not add breakpoints in notebooks

Testing #11769

VS Code Insiders, Jupyter pre-release extension
Open the following .ipnyb file
I can not add breakpoints 🐛

There is no glyph margin. I do not know how to add a breakpoint. Sorry.
{
 "cells": [
  {
   "cell_type": "code",
   "execution_count": 4,
   "metadata": {},
   "outputs": [
    {
     "name": "stdout",
     "output_type": "stream",
     "text": [
      "hello\n",
      "hello\n",
      "hello\n"
     ]
    }
   ],
   "source": [
    "print('hello')\n",
    "print('hello')\n",
    "print('hello')\n"
   ]
  },
  {
   "cell_type": "code",
   "execution_count": 2,
   "metadata": {},
   "outputs": [
    {
     "name": "stdout",
     "output_type": "stream",
     "text": [
      "second\n",
      "second\n",
      "second\n"
     ]
    }
   ],
   "source": [
    "print('second')\n",
    "print('second')\n",
    "print('second')\n"
   ]
  }
 ],
 "metadata": {
  "kernelspec": {
   "display_name": "Python 3 (ipykernel)",
   "language": "python",
   "name": "python3"
  },
  "language_info": {
   "codemirror_mode": {
    "name": "ipython",
    "version": 3
   },
   "file_extension": ".py",
   "mimetype": "text/x-python",
   "name": "python",
   "nbconvert_exporter": "python",
   "pygments_lexer": "ipython3",
   "version": "3.10.4"
  },
  "orig_nbformat": 4
 },
 "nbformat": 4,
 "nbformat_minor": 2
}

You'll have to install the Python extension as well. This is tricky but realistically anyone who is doing real work in a Python Jupyter notebook will have that extension installed, since we rely on it for things like environment discovery.


Using < in custom delimiters yields wrong compiled template

Vue.js version
2.1.10
Reproduction Link

https://jsfiddle.net/mwmaaju6/
Steps to reproduce
It seem that Vue parse the template strings improperly.
if you add a space before open delimiter. The rendering is correct.
try inside <p> tag
<p> <% text %> </p>
What is Expected?
correctly rendering
What is actually happening?
improperly rendering, I seem to strip the next tags, in this case

Thanks, looks like a bug due to the delimiter starting with <

Thanks, this issue has been fixed via https://github.com/vuejs/vue/pull/4753. Not released yet.


Testing with Jest, Enzyme & Typescript: Warning: React.createElement: type is invalid

🐛 Bug Report
Writing Jest / Enzyme tests with Typescript using React-Spring returns a warning.
To Reproduce
Clone this repo.
Run yarn.
Run yarn test navigation.test.tsx
The expected warning is as follows:
  console.error node_modules/react/cjs/react.development.js:172
    Warning: React.createElement: type is invalid -- expected a string (for built-in components) or a class/function (for composite components) but got: undefined. You likely forgot to export your component from the file it's defined in, or you might have mixed up default and named imports.

Expected behavior
No warning should be returned. If you run yarn test navigationTwo.test.tsx, no warning is returned. This is a similar component, but instead it does not use React-Spring.
Environment

react-spring v8.0.27
react v16.4.0

@aleclarson was wondering if I could get an update?

Try with v9 beta:
npm i react-spring@next

@aleclarson , I have updated the repo with the latest version of react-spring (^9.0.0-beta.33) and updated importing but am still getting the same error message.

@aleclarson is there any update with this?

I get this error when I run yarn in your repo:
An unexpected error occurred<EMAIL_ADDRESS>Request failed \"401 Unauthorized\"".

@aleclarson , that issue should be resolved now.

@aleclarson is there an update with this?

In jest.config.js, you should be using:
'^react-spring$': '<rootDir>/node_modules/react-spring/web.cjs',

Without the ^ and $, you'll accidentally use that module for every package in the @react-spring package scope.
Your test ran fine when I removed that line entirely, so I don't think you even need it.


test(e2e,playwright): OfflineWarning

Checklist:

[x] I have read and followed the contribution guidelines.
[x] I have read and followed the how to open a pull request guide.
[x] My pull request targets the main branch of freeCodeCamp.
[x] I have tested these changes either locally on my machine, or GitPod.

Ref #51705 OfflineWarning Component

Draft PR as required to help with coding.

Can I work on this?

Can I work on this?

Of course! Idk if I have to give you permissions or something like that in order to contribute to this PR. Any clue?

Hey @moradelboca, I noticed that this PR is still a draft. Is there anything we can help you with to get the PR ready for review?

I don't know what am I doing wrong. I created this PR to see if anyone had any idea.. a mod told me to do that in the ds server, but seems this isn't a good solution.
Any clue?

@Ahmadkashif @moradelboca Is this still open?


Please fix the bug.

compilation error
" msa.cc: In function ‘void msa_main(const char*)’:
msa.cc:289:22: error: ‘chdir’ was not declared in this scope
if (chdir(html_fn)<0)
^
make: *** [mcscanx] Error 1"
"""

Solution
http://ubuntuforums.org/showthread.php?t=2205151
if you are building on 64-bit you may need to add
#include <unistd.h>

to msa.h, dissect_multiple_alignment.h, and detect_collinear_tandem_arrays.h

This could be the problem of your C++ compiler. I suggest to use different
computers to have a try.
On Wed, Jan 7, 2015 at 12:53 AM, Wei Shen<EMAIL_ADDRESS>wrote:

                            compilation error

" msa.cc: In function ‘void msa_main(const char
*)’: msa.cc:289:22: error: ‘chdir’ was not declared in this scope if
(chdir(html_fn)<0) ^ make: *** [mcscanx] Error 1"
Solution: http://ubuntuforums.org/showthread.php?t=2205151
if you are building on 64-bit you may need to add
#include
to msa.h, dissect_multiple_alignment.h, and
detect_collinear_tandem_arrays.h
—
Reply to this email directly or view it on GitHub
https://github.com/wyp1125/MCScanx/issues/4.

--
Yupeng Wang, Ph.D.
Research Fellow
Computational Biology Branch
NCBI, NLM, NIH
Bethesda, MD 20894

Thanks for you reply.
here is my gcc version:
$ gcc -v
Using built-in specs.
COLLECT_GCC=/bin/gcc
COLLECT_LTO_WRAPPER=/usr/libexec/gcc/x86_64-redhat-linux/4.9.2/lto-wrapper
Target: x86_64-redhat-linux
Configured with: ../configure --prefix=/usr --mandir=/usr/share/man
--infodir=/usr/share/info --with-bugurl=http://bugzilla.redhat.com/bugzilla
--enable-bootstrap --enable-shared --enable-threads=posix
--enable-checking=release --enable-multilib --with-system-zlib
--enable-__cxa_atexit --disable-libunwind-exceptions
--enable-gnu-unique-object --enable-linker-build-id
--with-linker-hash-style=gnu
--enable-languages=c,c++,objc,obj-c++,fortran,ada,go,lto --enable-plugin
--enable-initfini-array --disable-libgcj
--with-isl=/builddir/build/BUILD/gcc-4.9.2-20141101/obj-x86_64-redhat-linux/isl-install
--with-cloog=/builddir/build/BUILD/gcc-4.9.2-20141101/obj-x86_64-redhat-linux/cloog-install
--enable-gnu-indirect-function --with-tune=generic --with-arch_32=i686
--build=x86_64-redhat-linux
Thread model: posix
gcc version 4.9.2 20141101 (Red Hat 4.9.2-1) (GCC)
沈伟 (Wei Shen)
Department of Microbiology, Third Military Medical University, China
Email<EMAIL_ADDRESS>Site : http://shenwei.me
2015-01-17 21:35 GMT+08:00 Yupeng Wang<EMAIL_ADDRESS>
This could be the problem of your C++ compiler. I suggest to use different
computers to have a try.
On Wed, Jan 7, 2015 at 12:53 AM, Wei Shen<EMAIL_ADDRESS>wrote:

compilation error
" msa.cc: In function ‘void msa_main(const char
*)’: msa.cc:289:22: error: ‘chdir’ was not declared in this scope if
(chdir(html_fn)<0) ^ make: *** [mcscanx] Error 1"
Solution: http://ubuntuforums.org/showthread.php?t=2205151
if you are building on 64-bit you may need to add
#include
to msa.h, dissect_multiple_alignment.h, and
detect_collinear_tandem_arrays.h
—
Reply to this email directly or view it on GitHub
https://github.com/wyp1125/MCScanx/issues/4.

--
Yupeng Wang, Ph.D.
Research Fellow
Computational Biology Branch
NCBI, NLM, NIH
Bethesda, MD 20894
—
Reply to this email directly or view it on GitHub
https://github.com/wyp1125/MCScanx/issues/4#issuecomment-70367162.

[root@localhost MCScanX]# make
g++ struct.cc mcscan.cc read_data.cc out_utils.cc dagchainer.cc msa.cc permutation.cc -o MCScanX
g++ struct.cc mcscan_h.cc read_homology.cc out_homology.cc dagchainer.cc msa.cc permutation.cc -o MCScanX_h
g++ struct.cc dup_classifier.cc read_data.cc out_utils.cc dagchainer.cc cls.cc permutation.cc -o duplicate_gene_classifier
g++ dissect_multiple_alignment.cc -o downstream_analyses/dissect_multiple_alignment
g++ detect_collinear_tandem_arrays.cc -o downstream_analyses/detect_collinear_tandem_arrays
cd downstream_analyses/ && make
make[1]: Entering directory /home/jhh130910/genome_analysis/MCScanX/downstream_analyses' javac -g Cubic.java make[1]: execvp: javac: Permission denied make[1]: *** [Cubic.class] Error 127 make[1]: Leaving directory /home/jhh130910/genome_analysis/MCScanX/downstream_analyses'
make: *** [mcscanx] Error 2

I have add #include <unistd.h> to  msa.h, dissect_multiple_alignment.h, and detect_collinear_tandem_arrays.h ,but get a new error message, what can I do next step ?

got it ,my java not correct ,solve it ,thanks!!

@jhh130910 I also get new error
detect_collinear_tandem_arrays.cc:286:46: error: ‘getopt’ was not declared in this scope
detect_collinear_tandem_arrays.cc:291:32: error: ‘optarg’ was not declared in this scope
detect_collinear_tandem_arrays.cc:307:17: error: ‘optopt’ was not declared in this scope
do you know how to solve ? thank U

I am not sure if the package on github is the final version. Please
download it from http://chibba.pgml.uga.edu/mcscan2/
On Fri, Jan 8, 2016 at 3:14 AM, officiant<EMAIL_ADDRESS>wrote:

@jhh130910 https://github.com/jhh130910 I also get new error
detect_collinear_tandem_arrays.cc:286:46: error: ‘getopt’ was not declared
in this scope
detect_collinear_tandem_arrays.cc:291:32: error: ‘optarg’ was not declared
in this scope
detect_collinear_tandem_arrays.cc:307:17: error: ‘optopt’ was not declared
in this scope
do you know how to solve ? thank U
—
Reply to this email directly or view it on GitHub
https://github.com/wyp1125/MCScanx/issues/4#issuecomment-169928531.

sorry,I can’t solve it.

在 2016年1月8日，下午4:14，officiant<EMAIL_ADDRESS>写道：
@jhh130910 https://github.com/jhh130910 I also get new error
detect_collinear_tandem_arrays.cc:286:46: error: ‘getopt’ was not declared in this scope
detect_collinear_tandem_arrays.cc:291:32: error: ‘optarg’ was not declared in this scope
detect_collinear_tandem_arrays.cc:307:17: error: ‘optopt’ was not declared in this scope
do you know how to solve ? thank U
—
Reply to this email directly or view it on GitHub https://github.com/wyp1125/MCScanx/issues/4#issuecomment-169928531.

Please download mcscanx from its homepage and recompile. GitHub version is incomplete
Sent from my iPhone

On Jan 9, 2016, at 5:07 AM, jhh130910<EMAIL_ADDRESS>wrote:
sorry,I can’t solve it.

在 2016年1月8日，下午4:14，officiant<EMAIL_ADDRESS>写道：
@jhh130910 https://github.com/jhh130910 I also get new error
detect_collinear_tandem_arrays.cc:286:46: error: ‘getopt’ was not declared in this scope
detect_collinear_tandem_arrays.cc:291:32: error: ‘optarg’ was not declared in this scope
detect_collinear_tandem_arrays.cc:307:17: error: ‘optopt’ was not declared in this scope
do you know how to solve ? thank U
—
Reply to this email directly or view it on GitHub https://github.com/wyp1125/MCScanx/issues/4#issuecomment-169928531.

—
Reply to this email directly or view it on GitHub.

Hello,
The same problem still occurs if you download the software now, but here
https://ubuntuforums.org/showthread.php?t=2205151
the post #6 has the solution.


GAMLSS warnings during marginal fitting

Hello,
While fitting marginal distributions for my data, I am getting a couple warning messages (not errors though) that seem to originate from the GAMLSS fitting.
1: In RS() : Algorithm RS has not yet converged
2: In digamma(y + (1/sigma)) : NaNs produced
3: In digamma(1/sigma) : NaNs produced
I assume these are resulting from poor fitting for certain genes. Should such genes be excluded from modeling? Or can these warnings be disregarded?
Thank you,
Connor

Hi Connor,
I'm sorry for not getting back to you sooner. Yeah, this is caused by the poor fitting of certain genes, usually too many zeros. Usually, this is not that bad - if we have a failure in fitting, our method will automatically change to the GAM model (a simpler model compared to GAMLSS). Therefore, I think these warnings can be disregarded.
Best,
Dongyuan

Thanks for the response!


Not able to use without Column or Row

I hope that I've not overlooked something but it seems like it's not possible to use the animations without creating an extra Column or Row.
Would it be possible to implement the feature to use these animations for a single widget (without creating a Column/Row)?
Besides this the package is awesome - thanks for sharing!

Hi there! Thanks for reaching out.
This package is designed to solve the relative animation issue between multiple widgets. Hence, a column/row is required. For single widget, we can use Flutter's default animation class.


upload of data in pending state

when trying to upload data it remains in a pending state

I am not sure where to start to try and debug this error.
Any pointer would be welcomed.

the logs from imports.log
{"user":"username","state":"pending","tables":0,"imported_tables":0,"failed_tables":0,"error_code":null,"import_timestamp":"2018-10-27T05:03:21.714+00:00","queue_server":"84c4d39651da","database_host":"cartodb-postgis","service_name":"public_url","data_type":"file","is_sync_import":false,"import_time":0.003077,"file_stats":{},"resque_ppid":null,"user_timeout":13981,"error_source":null,"id":"e207d685-26f1-4965-8687-baf7af1de79f","total_size":0,"cartodbfy_time":0.0,"import_throughput":0.0,"cartodbfy_throughtput":null,"cartodbfy_import_ratio":0.0,"extra_options":{},"retrieved_items":0}
I then logged into the rails console
irb(main):001:0> di = DataImport['e207d685-26f1-4965-8687-baf7af1de79f'].log
D, [2018-10-27T05:36:55.457906 #533] DEBUG -- :   Sequel::Postgres::Database (2.2ms)  SELECT * FROM "data_imports" WHERE "id" = 'e207d685-26f1-4965-8687-baf7af1de79f'
D, [2018-10-27T05:36:55.461702 #533] DEBUG -- :   Sequel::Postgres::Database (1.3ms)  SELECT * FROM "logs" WHERE ("id" = 'c004611d-0d99-4a08-97e9-a7d9d7fc5af5') LIMIT 1
=> #<CartoDB::Log @values={:id=>"c004611d-0d99-4a08-97e9-a7d9d7fc5af5", :type=>"import", :user_id=>"48a8e8da-8a53-4bb2-981f-bce8a5f6121b", :created_at=>2018-10-27 05:03:21 +0000, :updated_at=>2018-10-27 05:03:26 +0000, :entries=>"2018-10-27 05:03:26 UTC: Running on server 3f0b560d360f with PID: 1776\n2018-10-27 05:03:26 UTC: new_importer()\n2018-10-27 05:03:26 UTC: Fetching datasource public_url metadata for item id public/uploads/05c04f410f3b9b15f2ae/Eastern_Cape_ASIDI_Projects_(on_2018-01-12_at_17.34.53).carto\n2018-10-27 05:03:26 UTC: File will be downloaded from public/uploads/05c04f410f3b9b15f2ae/Eastern_Cape_ASIDI_Projects_(on_2018-01-12_at_17.34.53).carto\n2018-10-27 05:03:26 UTC: Exception: undefined method []' for nil:NilClass\n2018-10-27 05:03:26 UTC: [\"/var/lib/gems/2.2.0/gems/uuidtools-2.1.5/lib/uuidtools.rb:603:in block (2 levels) in first_mac'", "/var/lib/gems/2.2.0/gems/uuidtools-2.1.5/lib/uuidtools.rb:602:in map'\", \"/var/lib/gems/2.2.0/gems/uuidtools-2.1.5/lib/uuidtools.rb:602:in block in first_mac'", "/var/lib/gems/2.2.0/gems/uuidtools-2.1.5/lib/uuidtools.rb:608:in call'\", \"/var/lib/gems/2.2.0/gems/uuidtools-2.1.5/lib/uuidtools.rb:608:in first_mac'", "/var/lib/gems/2.2.0/gems/uuidtools-2.1.5/lib/uuidtools.rb:634:in mac_address'\", \"/var/lib/gems/2.2.0/gems/uuidtools-2.1.5/lib/uuidtools.rb:242:in block in timestamp_create'", "/var/lib/gems/2.2.0/gems/uuidtools-2.1.5/lib/uuidtools.rb:232:in synchronize'\", \"/var/lib/gems/2.2.0/gems/uuidtools-2.1.5/lib/uuidtools.rb:232:in timestamp_create'", "/cartodb/services/importer/lib/importer/job.rb:13:in initialize'\", \"/cartodb/services/importer/lib/importer/runner.rb:414:in new'", "/cartodb/services/importer/lib/importer/runner.rb:414:in new_job'\", \"/cartodb/services/importer/lib/importer/runner.rb:53:in initialize'", "/cartodb/app/models/data_import.rb:785:in new'\", \"/cartodb/app/models/data_import.rb:785:in new_importer_with_runner'", "/cartodb/app/models/data_import.rb:736:in new_importer'\", \"/cartodb/app/models/data_import.rb:423:in dispatch'", "/cartodb/app/models/data_import.rb:189:in run_import!'\", \"/cartodb/lib/resque/importer_jobs.rb:9:in block in perform'", "/cartodb/lib/resque/base_job.rb:21:in call'\", \"/cartodb/lib/resque/base_job.rb:21:in run_action'", "/cartodb/lib/resque/importer_jobs.rb:9:in perform'\", \"/var/lib/gems/2.2.0/gems/resque-1.25.2/lib/resque/job.rb:240:in block (3 levels) in perform'", "/var/lib/gems/2.2.0/gems/resque-metrics-0.1.1/lib/resque/metrics.rb:353:in around_perform_metrics'\", \"/var/lib/gems/2.2.0/gems/resque-1.25.2/lib/resque/job.rb:239:in block (2 levels) in perform'", "/var/lib/gems/2.2.0/gems/resque-1.25.2/lib/resque/job.rb:247:in call'\", \"/var/lib/gems/2.2.0/gems/resque-1.25.2/lib/resque/job.rb:247:in perform'", "/var/lib/gems/2.2.0/gems/resque-1.25.2/lib/resque/worker.rb:250:in perform'\", \"/var/lib/gems/2.2.0/gems/resque-1.25.2/lib/resque/worker.rb:189:in block in work'", "/var/lib/gems/2.2.0/gems/resque-1.25.2/lib/resque/worker.rb:166:in loop'\", \"/var/lib/gems/2.2.0/gems/resque-1.25.2/lib/resque/worker.rb:166:in work'", "/var/lib/gems/2.2.0/gems/resque-1.25.2/lib/resque/tasks.rb:41:in block (2 levels) in <top (required)>'\", \"/var/lib/gems/2.2.0/gems/rake-<IP_ADDRESS>/lib/rake/task.rb:205:in call'", "/var/lib/gems/2.2.0/gems/rake-<IP_ADDRESS>/lib/rake/task.rb:205:in block in execute'\", \"/var/lib/gems/2.2.0/gems/rake-<IP_ADDRESS>/lib/rake/task.rb:200:in each'", "/var/lib/gems/2.2.0/gems/rake-<IP_ADDRESS>/lib/rake/task.rb:200:in execute'\", \"/var/lib/gems/2.2.0/gems/rake-<IP_ADDRESS>/lib/rake/task.rb:158:in block in invoke_with_call_chain'", "/usr/lib/ruby/2.2.0/monitor.rb:211:in mon_synchronize'\", \"/var/lib/gems/2.2.0/gems/rake-<IP_ADDRESS>/lib/rake/task.rb:151:in invoke_with_call_chain'", "/var/lib/gems/2.2.0/gems/rake-<IP_ADDRESS>/lib/rake/task.rb:144:in invoke'\", \"/var/lib/gems/2.2.0/gems/rake-<IP_ADDRESS>/lib/rake/application.rb:116:in invoke_task'", "/var/lib/gems/2.2.0/gems/rake-<IP_ADDRESS>/lib/rake/application.rb:94:in block (2 levels) in top_level'\", \"/var/lib/gems/2.2.0/gems/rake-<IP_ADDRESS>/lib/rake/application.rb:94:in each'", "/var/lib/gems/2.2.0/gems/rake-<IP_ADDRESS>/lib/rake/application.rb:94:in block in top_level'\", \"/var/lib/gems/2.2.0/gems/rake-<IP_ADDRESS>/lib/rake/application.rb:133:in standard_exception_handling'", "/var/lib/gems/2.2.0/gems/rake-<IP_ADDRESS>/lib/rake/application.rb:88:in top_level'\", \"/var/lib/gems/2.2.0/gems/rake-<IP_ADDRESS>/lib/rake/application.rb:66:in block in run'", "/var/lib/gems/2.2.0/gems/rake-<IP_ADDRESS>/lib/rake/application.rb:133:in standard_exception_handling'\", \"/var/lib/gems/2.2.0/gems/rake-<IP_ADDRESS>/lib/rake/application.rb:63:in run'", "/var/lib/gems/2.2.0/gems/rake-<IP_ADDRESS>/bin/rake:33:in <top (required)>'\", \"/usr/local/bin/rake:23:in load'", "/usr/local/bin/rake:23:in `'"]\n2018-10-27 05:03:26 UTC: ERROR!\n\n"}>

Following a posting to https://groups.google.com/forum/#!topic/cartodb/sEDygx8ufaA
I ran this command - UUIDTools::UUID.timestamp_create
script/rails: No such file or directory - addr
NoMethodError: undefined method []' for nil:NilClass         from /var/lib/gems/2.2.0/gems/uuidtools-2.1.5/lib/uuidtools.rb:603:in block (2 levels) in first_mac'
from /var/lib/gems/2.2.0/gems/uuidtools-2.1.5/lib/uuidtools.rb:602:in map'         from /var/lib/gems/2.2.0/gems/uuidtools-2.1.5/lib/uuidtools.rb:602:in block in first_mac'
from /var/lib/gems/2.2.0/gems/uuidtools-2.1.5/lib/uuidtools.rb:608:in call'         from /var/lib/gems/2.2.0/gems/uuidtools-2.1.5/lib/uuidtools.rb:608:in first_mac'
from /var/lib/gems/2.2.0/gems/uuidtools-2.1.5/lib/uuidtools.rb:634:in mac_address'         from /var/lib/gems/2.2.0/gems/uuidtools-2.1.5/lib/uuidtools.rb:242:in block in timestamp_create'
from /var/lib/gems/2.2.0/gems/uuidtools-2.1.5/lib/uuidtools.rb:232:in synchronize'         from /var/lib/gems/2.2.0/gems/uuidtools-2.1.5/lib/uuidtools.rb:232:in timestamp_create'
from (irb):2
from /var/lib/gems/2.2.0/gems/railties-4.2.10/lib/rails/commands/console.rb:110:in start'         from /var/lib/gems/2.2.0/gems/railties-4.2.10/lib/rails/commands/console.rb:9:in start'
from /var/lib/gems/2.2.0/gems/railties-4.2.10/lib/rails/commands/commands_tasks.rb:68:in console'         from /var/lib/gems/2.2.0/gems/railties-4.2.10/lib/rails/commands/commands_tasks.rb:39:in run_command!'
from /var/lib/gems/2.2.0/gems/railties-4.2.10/lib/rails/commands.rb:17:in <top (required)>'         from script/rails:6:in require'
from script/rails:6:in `'

This can be closed.

Hi, @sindile. Sorry, I guess I missed the notifications for this issue. Do you think this issue warrants any changes to this repo?

Hi @davschne maybe update as per https://github.com/dersteppenwolf/cartodb-docker


[BUG gaia-x] Data not consumed, causing error results

Describe the bug
While querying a gremlin task g.V().limit(1000).filter(out().out()).count(), the results are unstable.
I have implemented the query using Gaia-x apis and the error is reproduced.
To Reproduce
Steps to reproduce the behavior:

Go to the 'gaia-x-bugs' branch, ioto the folder of pegasus/pegasus
run cargo run --release --example graph_queries
An assertion error will be shown, indicating that the results are not expected. This error may be casual, in case that the assertion error is not promted, re-run several times.

Expected behavior
The results should be exactly 1000.
Screenshots

This bug has been fixed, with the test case passed consistently


Feature request Title

Enhancement summary
We are trying to adopt arista AVD version v5.0.0, and finding below validation errors.
ERROR! [MBH-CAMP0S]: 'Validation Error: servers': Invalid type 'dict'. Expected a 'list'.
ERROR! [MBH-CAMP0S]: 'Validation Error: tenants': Invalid type 'dict'. Expected a 'list'.
ERROR! [MBH-CAMP0S]: 'Validation Error: l3_edge.p2p_links_ip_pools': Invalid type 'dict'. Expected a 'list'.
ERROR! [MBH-CAMP0S]: 'Validation Error: local_users': Invalid type 'dict'. Expected a 'list'.
ERROR! [MBH-CAMP0S]: 'Validation Error: node_type_keys': Primary key 'key' is not set on all items as required.
ERROR! [MBH-CAMP0S]: 'Validation Error: port_profiles[15].dot1x': Unexpected key(s) 'eapol' found in dict.

We can find from the release notes Data model changes from “dict-of-dicts” to “list-of-dicts” for only Ethernet interfaces, and you are using the key as "name" for the Ethernet interface name.
ethernet_interfaces:
-  Ethernet1:   # <-- User defined key
+  - name: Ethernet1 # <-- "name" here is called the "primary_key" which must have a unique value across all list elements
    <...>

but we can't find the key name for ip_address and ip_helper attributes, please let us know which key we have to use for ip_address and ip_helper values while converting them to list of lists.
Which component of AVD is impacted
eos_cli_config_gen
Use case example
We tried populating the config using below list of lists format for ip_helpers and ip_address for bgp_peers, but getting error saying ip_helper is not a valid key.
        ip_helpers:
        - ip_helper: <IP_ADDRESS>

      bgp_peers:
      - local_as: 65106
        remote_as: 204260
        description: em-palo-3220-a
        ip_address: <IP_ADDRESS>
        nodes:
        - MBH-BL1A

Describe the solution you would like
The config should access the key names ip_helper and ip_address as detailed below.
        ip_helpers:
        - ip_helper: <IP_ADDRESS>

      bgp_peers:
      - local_as: 65106
        remote_as: 204260
        description: em-palo-3220-a
        ip_address: <IP_ADDRESS>
        nodes:
        - MBH-BL1A

Describe alternatives you have considered
No response
Additional context
No response
Contributing Guide

[X] I agree to follow this project's Code of Conduct

also, please advise how can we resolve the below two errors, if you have any idea?
ERROR! [MBH-CAMP0S]: 'Validation Error: node_type_keys': Primary key 'key' is not set on all items as required.
ERROR! [MBH-CAMP0S]: 'Validation Error: port_profiles[15].dot1x': Unexpected key(s) 'eapol' found in dict.

The primary key is always the first key shown in the documentation for a given list of dicts.
You suggestion for ip_helper looks good to me, and we use the same in our test cases.
            ip_helpers:
              - ip_helper: <IP_ADDRESS>

ERROR! [MBH-CAMP0S]: 'Validation Error: node_type_keys': Primary key 'key' is not set on all items as required.

Without seeing your inputs for node_type_keys I cannot say anything else than the error states. You are missing a key named key on one of the list elements.

ERROR! [MBH-CAMP0S]: 'Validation Error: port_profiles[15].dot1x': Unexpected key(s) 'eapol' found in dict.

The eapol key is not supported under the port-profiles according to the documentation and schema, but it actually works because the underlying code is "lazy". We will be adding support for all the dot1x knobs in AVD 5.1.
Until then you can move the dot1x key under structured_config and it will render correctly.

Thanks for providing the details!
if I move dot1x key under structured_config it is rendering correctly, but it is applied to all the switches. I have to see whether I can restrict this in populating for only campus switches.
also , I have amended the type for node_type_keys from spine to l2spine as mentioned in below link
(https://github.com/aristanetworks/avd/pull/4340), but still seeing below validation errors. Can you please advise on these errors?
ERROR! [MBH-SPINE1]: 'Validation Error: node_type_keys': Primary key 'key' is not set on all items as required.
ERROR! [MBH-SPINE1]: 'Validation Error: type': 'spine' is not one of ['bleaf', 'campus', 'ixleaf', 'l2leaf', 'l3leaf', 'overlay-controller', 'l2spine', 'super-spine']

I have attached the screenshot for the node_type_keys config, what is the Primary key 'key' in node_type_keys the config is expecting?

You need to add key which refers to the key like l3leaf under which you set defaults, nodes etc. The value of type is the value that goes under the type: <type> which may or may not be the same.

The above-mentioned errors have been resolved. Currently, the following error is pending:
I have already set the bgp_as on the hosts, but I am still encountering the error below. Could you please advise if there has been any change in how this value should be set on the hosts?
TASK [arista.avd.eos_designs : Set eos_designs facts] ***************************************************************************************************************************************************************
task path: /home/avd/.ansible/collections/ansible_collections/arista/avd/roles/eos_designs/tasks/main.yml:28
redirecting (type: connection) ansible.builtin.httpapi to ansible.netcommon.httpapi
redirecting (type: httpapi) ansible.builtin.eos to arista.eos.eos
The full traceback is:
Traceback (most recent call last):
File "/home/avd/.ansible/collections/ansible_collections/arista/avd/plugins/action/eos_designs_facts.py", line 196, in render_avd_switch_facts
rendered_facts[host] = {"switch": avd_switch_facts_instances[host]["switch"].render()}
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/home/avd/.local/share/virtualenvs/netbox_extract-zMJz_RXI/lib/python3.12/site-packages/pyavd/_eos_designs/avdfacts.py", line 64, in render
return {key: getattr(self, key) for key in self.keys() if getattr(self, key) is not None}
^^^^^^^^^^^^^^^^^^
File "/usr/lib64/python3.12/functools.py", line 993, in get
val = self.func(instance)
^^^^^^^^^^^^^^^^^^^
File "/home/avd/.local/share/virtualenvs/netbox_extract-zMJz_RXI/lib/python3.12/site-packages/pyavd/_eos_designs/eos_designs_facts/init.py", line 121, in bgp_as
return self.shared_utils.bgp_as
^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/lib64/python3.12/functools.py", line 993, in get
val = self.func(instance)
^^^^^^^^^^^^^^^^^^^
File "/home/avd/.local/share/virtualenvs/netbox_extract-zMJz_RXI/lib/python3.12/site-packages/pyavd/_eos_designs/shared_utils/routing.py", line 99, in bgp_as
bgp_as_range_expanded = range_expand(str(get(self.switch_data_combined, "bgp_as", required=True)))
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/home/avd/.local/share/virtualenvs/netbox_extract-zMJz_RXI/lib/python3.12/site-packages/pyavd/_utils/get.py", line 60, in get
raise AristaAvdMissingVariableError(org_key)
pyavd._errors.AristaAvdMissingVariableError: 'bgp_as' is required but was not found.
fatal: [MBH-CAMP0S]: FAILED! => {
"changed": false,
"msg": "'bgp_as' is required but was not found for host 'MBH-OOB-SW01'."
}

So AVD wants to configure BGP for the switch MBH-OOB-SW01 but it cannot find a BGP AS number defined for this switch. Without knowing more of your inventory, it can be hard to say much more.
Feel free to contact me on<EMAIL_ADDRESS>if you want to share some more details.

Please advise on below two issues while generating the config..
Issue !: Getting below warnings for vlans inside ether_interfaces.
We used to have multiple vlans for ethernet_interfaces, but currently the data model is changed to create only one single access_vlan which is of integer type. Can we consider the first vlan in this case to populate?
Data model:
# Set VLAN when interface is in access mode.
# Warning: This should not be combined with ethernet_interfaces[].mode = access/dot1q-tunnel and ethernet_interface[].vlans.
access_vlan: <int; 1-4094>
Warning message:
Multiple VLANs detected for Ethernet30, using the first VLAN (98)
Issue 2: We are getting 3 errors during schema validation of input vars.
We have below local context for LON_IX_SW switch, but currently the data model is not supporting mac_security inside switchport. Please find the error message provided below for details. Please check and advise.
{
"ethernet_interfaces": [
{
"name": "Ethernet5/1",
"switchport": {
"mac_security": {
"profile": "RoS_DCI"
}
}
}
]
}
Error:
TASK [arista.avd.eos_designs : Set eos_designs facts] *************************************************************************************************************************************************************
ERROR! [LON-OOB-SW01]: 'Validation Error: ixleaf.node_groups[0].nodes[1].structured_config.ethernet_interfaces[0].switchport': Unexpected key(s) 'mac_security' found in dict.
ERROR! [LON-IX-SW]: 'Validation Error: ixleaf.node_groups[0].nodes[1].structured_config.ethernet_interfaces[0].switchport': Unexpected key(s) 'mac_security' found in dict.
ERROR! [LON-IX-SW02]: 'Validation Error: ixleaf.node_groups[0].nodes[1].structured_config.ethernet_interfaces[0].switchport': Unexpected key(s) 'mac_security' found in dict.
fatal: [MBH-CAMP0S]: FAILED! => {"changed": false, "msg": "3 errors found during schema validation of input vars."}

Please refer to the porting guide. The changes for vlans and associated keys are covered in this section: https://avd.arista.com/devel/docs/porting-guides/5.x.x.html#switchport-configuration-and-documentation

To add to @ClausHolbechArista
For question 2, the message states mac_security is not defined under swotchport. Indeed it should not be inside switchport but outside.:
doc: https://avd.arista.com/5.1/roles/eos_cli_config_gen/docs/input-variables.html#ethernet-interfaces. You can see this when hovering over the key in the doc:

so this
{
  "ethernet_interfaces": [
    {
      "name": "Ethernet5/1",
      "switchport": {
        "mac_security": {
          "profile": "RoS_DCI"
         }
      }
    }
  ]
}

should be changed to
{
  "ethernet_interfaces": [
    {
      "name": "Ethernet5/1",
      "switchport": {},
      "mac_security": {
        "profile": "RoS_DCI"
       }
    }
  ]
}


Docker dies: Error initializing network controller: Error creating default "bridge" network: cannot create  network docker0

Device type: TS-4900
OS version: Resin OS 2.0.6+rev1 (dev)
Aug 14 18:00:50 730efa3 systemd[1]: [[0;1;39mdocker.service: Unit entered failed state.[[0m
Aug 14 18:00:50 730efa3 systemd[1]: [[0;1;39mdocker.service: Failed with result 'exit-code'.[[0m
Aug 14 18:00:50 730efa3 systemd[1]: docker.service: Service hold-off time over, scheduling restart.
Aug 14 18:00:50 730efa3 systemd[1]: Stopped Docker Application Container Engine.
Aug 14 18:00:50 730efa3 systemd[1]: Starting Docker Application Container Engine...
Aug 14 18:00:51 730efa3 docker[903]: time="2017-08-14T18:00:51.121029667Z" level=warning msg="/!\\
Aug 14 18:00:51 730efa3 docker[903]: time="2017-08-14T18:00:51.185783334Z" level=info msg="Graph migration to content-addressability took 0.00 seconds"
Aug 14 18:00:51 730efa3 docker[903]: time="2017-08-14T18:00:51.207820001Z" level=info msg="Firewalld running: false"
Aug 14 18:00:51 730efa3 docker[903]: time="2017-08-14T18:00:51.730166001Z" level=info msg="Default bridge (docker0) is assigned with an IP address <IP_ADDRESS>/16. Daemon option --bip can be used to set a preferre
d IP address"
Aug 14 18:00:51 730efa3 docker[903]: time="2017-08-14T18:00:51.931294334Z" level=fatal msg="Error starting daemon: Error initializing network controller: Error creating default \"bridge\" network: cannot create
network docker0 (d4504fc3886a40e9fc1b3b6989b7dd9f90daa02ffe50dc5a69e547990a7ce287): conflicts with network febeb99d1d60938eff1bde7dde65b61e1d872cc8cfe4eba9006a3825790df124 (docker0): networks have same bridge na
me"
Aug 14 18:00:51 730efa3 systemd[1]: [[0;1;39mdocker.service: Main process exited, code=exited, status=1/FAILURE[[0m
Aug 14 18:00:51 730efa3 systemd[1]: [[0;1;31mFailed to start Docker Application Container Engine.[[0m
Aug 14 18:00:52 730efa3 systemd[1]: [[0;1;39mdocker.service: Unit entered failed state.[[0m
Aug 14 18:00:52 730efa3 systemd[1]: [[0;1;39mdocker.service: Failed with result 'exit-code'.[[0m
Aug 14 18:00:52 730efa3 systemd[1]: docker.service: Service hold-off time over, scheduling restart.

seems related to this: https://github.com/moby/moby/issues/17083 with the fix being: sudo rm -rf /var/lib/docker/network/files on the hostOS

I got the device into this state by rebooting it from the dashboard, so I guess docker is not properly cleaning up its networks at shutdown.

The reboot device action reboots the host OS, right? This means that at clean reboot the device couldn't start docker?

@agherzan yeah, basically dockers graceful shutdown is not cleaning up correctly. But this has been fixed in 17.03 as far as I can see.

In resinOS 2.0.7 we updated to Docker 17.03.1. Does that mean we would expect this issue to be fixed @shaunmulligan ?

@willnewton yeah that is my understanding of it currently.

@shaunmulligan is the way to reproduce to reboot from the dashboard a large number of times?

@willnewton yeah as far as I understand, but it might only have happened on the TS4900, I haven't ever seen it on the RPI family

@shaunmulligan is this still reproducible? Was it fixed with balena?

@agherzan don't think we've seen this lately and no reports, thus pointing to this not being an issue anymore. Probably good to close now (and reopen if new info comes in)?

Sounds good.


Synchronous version of the websockets client

Thanks Aymeric for this great library. While the WebSocket client example code works great in async fashion, I can't seem to get the websocket.clients to work in synchronous code.
import websockets
from flask import Flask
from flask_restful import Resource, Api

app = Flask(__name__)
api = Api(app)

class HelloWorld(Resource):
    def get(self):
        ws = websockets.client.connect('ws://localhost:8765')
        ws.send('Hello')
        response = ws.recv()
        return {'hello': response}

api.add_resource(HelloWorld, '/')

if __name__ == '__main__':
    app.run(debug=True)

I am actually getting a TypeError: can't send non-None value to a just-started generator
<IP_ADDRESS> - - [25/Aug/2015 18:01:08] "GET / HTTP/1.1" 500 -
Traceback (most recent call last):
  File "/Users/ye/.envs/py3k-flask/lib/python3.4/site-packages/flask/app.py", line 1836, in __call__
    return self.wsgi_app(environ, start_response)
  File "/Users/ye/.envs/py3k-flask/lib/python3.4/site-packages/flask/app.py", line 1820, in wsgi_app
    response = self.make_response(self.handle_exception(e))
  File "/Users/ye/.envs/py3k-flask/lib/python3.4/site-packages/flask_restful/__init__.py", line 261, in error_router
    return self.handle_error(e)
  File "/Users/ye/.envs/py3k-flask/lib/python3.4/site-packages/flask/app.py", line 1817, in wsgi_app
    response = self.full_dispatch_request()
  File "/Users/ye/.envs/py3k-flask/lib/python3.4/site-packages/flask/app.py", line 1477, in full_dispatch_request
    rv = self.handle_user_exception(e)
  File "/Users/ye/.envs/py3k-flask/lib/python3.4/site-packages/flask_restful/__init__.py", line 261, in error_router
    return self.handle_error(e)
  File "/Users/ye/.envs/py3k-flask/lib/python3.4/site-packages/flask/app.py", line 1475, in full_dispatch_request
    rv = self.dispatch_request()
  File "/Users/ye/.envs/py3k-flask/lib/python3.4/site-packages/flask/app.py", line 1461, in dispatch_request
    return self.view_functions[rule.endpoint](**req.view_args)
  File "/Users/ye/.envs/py3k-flask/lib/python3.4/site-packages/flask_restful/__init__.py", line 430, in wrapper
    resp = resource(*args, **kwargs)
  File "/Users/ye/.envs/py3k-flask/lib/python3.4/site-packages/flask/views.py", line 84, in view
    return self.dispatch_request(*args, **kwargs)
  File "/Users/ye/.envs/py3k-flask/lib/python3.4/site-packages/flask_restful/__init__.py", line 520, in dispatch_request
    resp = meth(*args, **kwargs)
  File "/Users/ye/dev/temp/flask-restful-ws.py", line 11, in get
    ws.send('Hello')
TypeError: can't send non-None value to a just-started generator

So my question is, is it possible to use a synchronous fashion to use websockets.client APIs? Assuming that's a no, how would I integration the websockets client api into a non-async webapp like Flask? I am having a hard time figuring out how to get the event loop embedded into an existing web app. It would be great to another example to use websockets.client APIs, thanks!

Well, the downside of explicit async programming (the model chosen by asyncio) is that it's hard to combine with sync programming...
Technically you can wrap every call to a websockets coroutine in asyncio.get_event_loop().run_until_complete but you'll get synchronous execution, which means no concurrency (and your code will look bad).
If you want to run in the same process as a synchronous web framework such as Flask with decent concurrency, you need an implicit async framework such as gevent.
Searching for "flask gevent websockets" should turn up libraries more suitable to you use case.

@aaugustin thanks for the insight and direction guidance! I came across a very thoughtful blog post about explicit vs. implicit async programming in Python by @ncoghlan, which is very helpful for me to understand better about this topic.
Also, perhaps aiohttp or Flask Async is more suitable in my use case to integrate with websockets?

aiohttp has its own built-in websockets API so that's one option.
If you want to stick with Flask you con try flask-sockets.

:+1:


Vector integer add-with-carry/subtract-with-borrow instructions missing mask as a source

The vadc and vsbc vector instructions are not being decoded correctly. These instructions always enable masking and opcodes where masking is disabled are not supported. To support this in Mavis, the vm field is marked as a fixed field, but Mavis assumes that to mean that masking is disabled.
To test a fix, the opcode 0x403100D7 should print this disassembly (mask register v0 included):
vadc.vvm v1,v2,v3,v0.t

@h0lyalg0rithm here is a Mavis issue you can look it. If you look in https://github.com/sparcians/mavis/blob/main/json/isa_rv64v.json you can see how this instruction is defined:
  {
    "mnemonic": "vadc.vvm",
    "tags" : ["v"],
    "form": "V",
    "stencil": "0x40000057",
    "fixed": [ "vm" ],
    "type": [ "vector" ],
    "v-oper": "all"
  },

The stencil defines which bits in an opcode are always set for a particular instruction like the opcode and funct fields. My understanding of the "fixed" fields list is that these are fields that are also always the same. So for this instruction, the VM field is always set to 0. An opcode with the VM bit set to 1 would not be decoded as this instruction. Normally, the VM field is accessible as a "special field", but when it is marked as fixed it is no longer consider a special field of the instruction and its value cannot be accessed.
I don't totally understand how the fixed and special fields work, so please share anything you learn!


Implement IP deny/allow ACL rules

What type of PR is this?
feat: add Support for Allow/Deny IP Subnets
What this PR does / why we need it:
Usually there needs to be way to deny / allow subnets for single routes / entire gateway.
Which issue(s) this PR fixes:
Fixes #2462
Notes to reviewers
disclaimer: This is my first bigger PR to this repository
Questions

Gateway level ACL definitions works. However, if route do have any ACL rules the route will override the Gateway ACL settings. Like mentioned in https://github.com/envoyproxy/gateway/issues/2462#issuecomment-1906591842 how this should work? Is this behavior ok?
This applies rules only to http, there is RBAC policy for network level as well (but those cannot be applied to route level). It might be needed if envoy gateway is used for tcp stuff? That is perhaps task to another PR?
Not sure does it work out of the box with ipv6, I do not have k8s with ipv6 to verify it.

I have verified this PR manually by following test cases:
route: allow ip - ok (allowed ip works and others not)
route: deny ip - ok (denied ip does not work and others will work)
route: allow big cidr, deny single ip(s) from allowed cidr - ok (big cidr works except denied ips)
route: deny big cidr, allow single ip(s) from denied cidr - not working, but it should not either? (whole big cidr does not work, no matter is there something in allow)

gateway: allow ip - ok (allowed ip works and others not)
gateway: deny ip - ok (denied ip does not work and others will work)
gateway: allow big cidr, deny single ip(s) from allowed cidr - ok (big cidr works except denied ips)
gateway: deny big cidr, allow single ip(s) from denied cidr - not working, but it should not either? (whole big cidr does not work, no matter is there something in allow)

Please submit the API design first, otherwise there may be a lot of waste.

@zirain I cannot find any mentioning about such policy in Contributing or developer guide in this project. What you mean with API design? Do you mean that I should make separate PR about stuff that is going under api/? I tried to look old PRs and all of them have API changes in same PR. Could you give example what I should do here.


All fields as import_id_fields

Hey,
is there an option to declare all fields as import_id_fields instead of defining them all one by one in a list ?

Yes, you can override get_import_id_fields():
class BookResource(resources.ModelResource):
    def get_import_id_fields(self):
        return self._meta.fields

    class Meta:
        model = Book
        fields = ('id', 'name')

I get this error:
import_id_fields = [ TypeError: 'NoneType' object is not iterable

Or maybe you i misspelled my question. I want all of the fields i declared above class Meta: to be used as import_id_fields.

Apparently i miss understood the purpose or function of the import_if_fields.


selecting negatives from present item_idxes

before: sampling negatives from [0, max_item_idx]
now: sampling negatives from items, presented in fit

discuss: protected attribute creation and deletion inside fit, sampling with numpy instead of pytorch.

closes #99


Test against check-layout-delay WPT branch

Description
Test against the check-layout-delay branch of our WPT fork, which introduces a small delay before checking the layout to ensure the polyfill has time to apply changes.
Steps to test/reproduce
I'm seeing new passsing tests for anchor-position-inline-*.html, anchor-position-dynamic-*.html and anchor-position-multicol-001.html. Compare the main branch results with the new results.

@jgerigmeyer not sure what's going on with inline-004. I'm guessing it's a problem with multicol as well?

 Wait for polyfill before running WPT assertions.

@jgerigmeyer not sure what's going on with inline-004. I'm guessing it's a problem with multicol as well?

Yeah, it's a combination of the race condition, a bug determining the anchor element (https://trello.com/c/MKApHxX7), and bugs with multicol layouts.


ISPN-6215 Remote Java client package

Hey!
Please have a look at proposition of Remote Java client: https://issues.jboss.org/browse/ISPN-6215
Thanks!
Sebastian

The javadoc included here is for the entire Infinispan API. It should be trimmed to just the HotRod client and remote query.

We agreed with Tristan to remove Javadocs package from the remote client and implement it using different JIRA: https://issues.jboss.org/browse/ISPN-6217

Pushed, thanks


The Kinectis link is broken

@zhaoyue-zephyrus Hi! Thanks for your sharing. And I found that the link to download Kinetics dataset is broken. For example, https://deepmind.com/documents/65/kinetics_val.zip. Could you update the link? In addition， Kinetic official website has 222,580 training videos，18291 validation videos, but you said that you are using 240436 training videos,19796 testing videos.Could you tell me the difference between the official data with your data? Thank you very much!!

The official list of Kinetics-400 can be obtained here: https://storage.googleapis.com/deepmind-media/Datasets/kinetics400.tar.gz. We will update the links in our repo. Our videos were downloaded when Kinetics-400 released for the first time. As time goes on, some Youtube videos become invalid. Maybe that's why the official Kinetics-400 dataset shrinks.

@kennymckormick any update for the kinetics link or workaround to this?


fail to install Julia 1.9.2

export JULIAUP_SERVER=https://mirrors.tuna.tsinghua.edu.cn/julia-releases
curl -fsSL https://install.julialang.org | sh

xx@ubuntu:/tmp$ juliaup add 1.9.2
Installing Julia 1.9.2+0.x64.linux.gnu
Error: Failed to extract downloaded file from url `https://mirrors.tuna.tsinghua.edu.cn/julia-releases/bin/linux/x64/1.9/julia-1.9.2-linux-x86_64.tar.gz`.

Caused by:
    invalid gzip header

wget https://mirrors.tuna.tsinghua.edu.cn/julia-releases/bin/linux/x64/1.9/julia-1.9.2-linux-x86_64.tar.gz  is ok and can get complete file.
xx@ubuntu:/tmp$ juliaup self update
Checking for self-updates
Juliaup unchanged on channel 'release' - 1.11.15

xx@ubuntu:/tmp$ hostnamectl
   Static hostname: VM-1-8-ubuntu
         Icon name: computer-server
           Chassis: server
        Machine ID: adf296e593574c1cb00ec835cac97797
           Boot ID: 2cf32de54d6a426d94455c25ed4319a2
  Operating System: Ubuntu 18.04.4 LTS
            Kernel: Linux 4.15.0-142-generic
      Architecture: x86-64

I don't really know what is going on, but at the end of the day we don't officially support these mirrors...


Don't block pings on durable views

We use pings to:

exchange view info with the clients
exchange time info with replicas

We also have a rule that no message with a view can be sent until that view is durably stored.
Together, this means we can't synchronize time while the view is not durable.
This commit fixes the issue by noticing that we never use pings for views and timestamps at the same time: views are for clients, timestamps are for other replicas.
Pre-merge checklist
Performance:

[x] Compare zig benchmark on linux before and after this pr.# benchmark results before
1000 batches in 200513 ms
40850 transfers per second
max p100 latency per 8191 transfers = 524ms
...

# benchmark results after
1000 batches in 200402 ms
40872 transfers per second
max p100 latency per 8191 transfers = 541ms

OR

[ ] I am very sure this PR could not affect performance.

Ready for review @sentientwaffle!

bors r+