change desc into one liner | bible_jv_id dataloader

Change desc into one liner

Hello @Stackpas, thank you for the edit. We have discussed about this, and we decided to go with your initial _DESCRIPTION that uses the automatically-formatted code.
Thanks again for the change. Closing the PR now.


add-category comment-aerer-son-logement-pour-ameliorer-l-air-interieu…

…r.md

:tada: This PR is included in version 1.4.5 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


September 16th, 2024 Community Meeting

Please add a topic in this thread and add a link to the GitHub issue associated with the topic.
Please make sure you give folks enough time to review/discuss the topic offline on GitHub before coming into the meeting
(optional) Paste the image of an animal 😸

Meeting minutes:

Discussed on some new PRs from contributors.
We dont have an automation around moving docs from Build to the Website.
Should we consider a different approach for docs? projects that adopted docsy theme tend to have a single /docs repo, where docs, post, etc is hosted.
Shipwright website needs improvements, @ayushsatyam146 to file an issue on this.
@adambkaplan to double check on the domain(Website), and move it to CNCF.


This could be more of a site for any image changing or operations

ISO converter or type conversion in general would not be hard especially with a backed.

since its a PWA we also could just tell their OS to do some of it for sure.


It's impossible to use command line commands with " "

You can see it tries to run [rg, "fun, run"] which is wrong it should run [rg, fun run].

there is stackoverflow question about it. Maybe we can take parser from there [stackoverflowlink]
(https://stackoverflow.com/questions/34118732/parse-a-command-line-string-into-flags-and-arguments-in-golang)

I believe I have this fixed now in 0.3.2 by adding a dependency "google/shlex".


Add categories to Github discussions

Add additional categories which would be useful to the project

Troubleshooting - for helping people setting up their board
Documentation - for discussion about tutorials, learning materials, Wiki
Development - for research on the development board, promotion of useful tools and projects

I would use the "Open ended discussion" format for these.
Existing categories are very general, but I would keep them for whatever discussion that doesn't fit in the suggested categories.

Thank you for creating the categories.

Thank you for creating the categories.


fix: portfolio empting loading state

Description

Pull Request Type

[ ] :bug: Bug fix (Non-breaking Change: Fixes an issue)
[ ] :hammer_and_wrench: Chore (Non-breaking Change: Doc updates, pkg upgrades, typos, etc..)
[ ] :nail_care: New Feature (Breaking/Non-breaking Change)

Issue (if applicable)

closes #
Risk

Testing

Engineering

Operations

Screenshots (if applicable)

Current dependencies on/for this PR:

develop

PR #5711   👈

PR #5403

This stack of pull requests is managed by Graphite.


Implement grammar support for delete

https://tellurium.readthedocs.io/en/latest/antimony.html#submodel-deletions


Deprecate the version command in favor of the --version flag

https://github.com/spf13/cobra#version-flag

Not worth the effort. Current version command is fine.


Replace task runner

Coverage remained the same at 100.0% when pulling 17405b212bec29529892d24269333bc27b08747d on dev into 0f2281a0fc7c35b5335dce5d8f2e0d2130d2eb06 on master.


Feature Request: Desktop and mobile clients should have options for sounds

I noticed that on the desktop client and the mobile app, that there are basic notification settings for the account, but not the device. Such as whether to play a sound, or not.
I see push notifications on my phone, but no vibration or sound.
I see the icon change on my windows task bar, and a brief toaster popup, but no sound to catch my attention.

I thought this was a bug: I have no sound on Gnome Keybase notifications. I do have sound on my iPhone when I get Keybase chat messages, but none on Gnome desktop  (also, teammates experience same behavior). And I do see Gnome toaster for keybase messages -- just no sound.
I keep the phone nearby so I'll know I got a message, if I miss the toaster.

I also get no sound on linux or mobile


passives 84, 85, 10006

84: increase chain modifier for physical attacks. does not increase cap, just hits it faster.
[0, 3, 84, [100]]
85: increase chain modifier for magical attacks. does not increase cap, just hits it faster.
[0, 3, 85, [100]]
10006, not parsed:
Increase resistance to ATK/DEF/MAG/SPR break (100%) for 3 turns when HP drops below 70% (max 1 time)
[0, 3, 10006, [[95,  96,  97,  98], 100, 1, 70, 0, 3]]
10006, parsed:
should specify 'elemental'? "Increase all resistances by 50% when...
[0, 3, 10006, [[23,  24,  25,  26,  27,  28,  29,  30], 50, 9999, 50, 0, 3]]
thanks for all that you do.

one more active that's not parsed. seems to be new this last week with ibara.
active 1014, used in skills 914409 & 914429.
not sure if it does anything more than control how many times per battle a skill can be used.

Fixed the passives with 358c73ae, for the active I need more info (wiki seems to have problems rn).


Azure identity: Managed identity authentication does not handle rate limits

Package Name: azure-identity
Package Version: 1.4.0
Operating System: Ubuntu 18.04
Python Version:  3.6.6

Describe the bug
azure-identiy is not able to handle ratelimit when multiple processes try to authenticate at the same time using Managed Identities.
To Reproduce
Steps to reproduce the behavior:

Start multiple process (e.g 48).
Initialise the authentication with DefaultAzureCredentials().
Interact via the pytohn SDK with the storage account.

Expected behavior
I'm able to run multiple processes authenticating with the storage account simultaneously.
Additional context
DefaultAzureCredentials() does not perform any authentication when it is instantiated. This operation is done when a client (like azure-blob-storage) perform get_token(). This make the handling of the returned exception quite hard since the Pytohn SDK ignore this exception and pass it to the program. The retry_policy does not seem to work since we are not performing that many requests to the storage accounts.

Credentials authenticate only as needed because we don't want to introduce network failure modes to __init__, and a credential can't know the scopes of access it should request.
I don't know how your application works, but assuming each subprocess performs similar operations, i.e. each needs the same scope of access, you could in principle trigger authentication before forking. Each subprocess would then begin with a credential that already has the necessary access. You could either instantiate a client and perform an operation with it or if you know the scope you need, request a token from the credential:
# probably the scope you would want for Storage; using a client is more robust
credential.get_token("https://storage.azure.com/.default")

Hi @chlowell thanks for your response.
The problem of not handling the exception in the identity module is that the exception will be thrown to the azure-storage-blob  module which does not really understand what to do with it.
We could try to force the authentication before forking the process as you suggested, I think this approach would solve the problem if the token does not require to be renewed. In our use case it would be very possible that these processes will run for more than 24 hours. At that point, I think, we will face the same problem because each process will try to authenticate independently at the same time.
We thought about creating a small proxy server that will cache the response but that seems a bit an overkill for this scope. At the moment we have added random delays before the first data are fetched to avoid hitting the concurrency rate limit imposed by Azure. However, this solution does not seem very elegant.

ManagedIdentityCredential does attempt to handle this by retrying on 429. That doesn't help your application because its instances don't share state after the fork. A token valid for one instance is valid for all but after the fork each must request its own. In practice, I expect that means a minority of instances succeed while the rest exhaust their retries. And tokens do expire, so if your 48 instances start with the same one, they nevertheless request a replacement at roughly the same time, bringing you back to where you started.
I think the application needs some way to share state among its subprocesses. A caching proxy, even if not the best solution, seems in the right genre to me.

Were you able to solve this problem?

Were you able to solve this problem?

We have added a random delay and this seems to have solved the problem. Not ideal but I guess we can live with it for a job that run for multiple hours.

Okay, I'll close this issue then. Thanks for opening it and please open another if you encounter other problems using the SDK.


[Regression] Select doesn't work on Modal in IE11

Detailed description
When I use below versions of carbon components, I can not get select work well in IE11.
The problem is by the first click on the select, the options show up. Then click other area to let the options disappear, after that I can no longer open the options again.
    "carbon-components": "^9.5.0",
    "carbon-components-react": "6.21.4",
    "carbon-icons": "6.3.2",
    "react": "16.4.0",
    "react-dom": "16.4.0",

You can play on code sandbox. Or the URL - https://m5oxnqkvl8.codesandbox.io/.

More information:
Below versions of carbon components work well in IE11.
    "carbon-components": "8.18.0",
    "carbon-components-react": "5.34.0",
    "carbon-icons": "6.3.2",
    "react": "16.2.0",
    "react-dom": "16.2.0",


Create edges for Constrained Lists

Currently, fields with lists of models will render edges for lists. However, constrained lists will not. Here is a basic example:
from pydantic import BaseModel, conlist, Field
from typing import List
import erdantic as erd

class Foo(BaseModel):
  name: str

class Bar(BaseModel):
  foo1: List[Foo]
  foo2: conlist(Foo, unique_items=True)
  foo3: List[Foo] = Field(..., unique_items=True)

diagram = erd.create(Bar)
print(diagram)
"EntityRelationshipDiagram(models=[PydanticModel(Bar), PydanticModel(Foo)], edges=[Edge(source=PydanticModel(Bar), source_field=<PydanticField: 'foo1', List[Foo]>, target=PydanticModel(Foo))])"
diagram.draw("foobar.png")

@jtyoung84 @saskiad Sorry for having left you hanging.
Pydantic V2 is out now and the way that constrained lists are implemented has changed (validation is done with annotations instead of changing the types) in a way that is friendlier to erdantic. I'm not that keen on going back and adding this to an older version of erdantic that supports Pydantic V1.
I'm going to close this issue for now. If something still needs to be addressed, we can reopen.
from pydantic import BaseModel, conlist, Field
from typing import List
import erdantic as erd

class Foo(BaseModel):
    name: str

class Bar(BaseModel):
    foo1: List[Foo]
    foo2: conlist(Foo, min_length=1)
    foo3: List[Foo] = Field(..., min_length=1)

diagram = erd.create(Bar)
print(diagram)
#> EntityRelationshipDiagram(models=[PydanticModel(Bar), PydanticModel(Foo)], edges=[Edge(source=PydanticModel(Bar), source_field=<PydanticField: 'foo1', List[Foo]>, target=PydanticModel(Foo)), Edge(source=PydanticModel(Bar), source_field=<PydanticField: 'foo2', List[Foo]>, target=PydanticModel(Foo)), Edge(source=PydanticModel(Bar), source_field=<PydanticField: 'foo3', List[Foo]>, target=PydanticModel(Foo))])

diagram.draw("foobar.png")
#> None

Created at 2023-07-16 16:56:30 EDT by reprexlite v0.4.3
(It looks like unique_items has been removed in V2, so the above example uses min_length instead. There's some discussion about unique validation in this pydantic issue.)


HMR does not work in modal

What version of Remix are you using?
1.19.3
Are all your remix dependencies & dev-dependencies using the same version?

[X] Yes

Steps to Reproduce

clone https://github.com/telcy/remix-hmr-modal-issue
install dependencies
click "simple dialog" button
change modal text to trigger hmr

Expected Behavior
Modal should stay open and text should get replaced.
Actual Behavior
Modal closes

I'm not familiar with mui-modal-provider but I'm not sure if this is a problem with Remix or not.  Looking at it briefly, useModal is destroying the modal in an effect:
  useEffect(
    () => () => {
      if (!disableAutoDestroy) {
        destroy(id.current);
      }
    },
    [disableAutoDestroy, destroy]
  );

destroy there comes from their own internal useContext(ModalContext), so it seems as if it's identity is not stable.
I tested the useModal({ disableAutoDestroy: true }) option and that keeps the modal open during HMR - but I'm unsure of the impacts of keeping that option enabled in production use-cases.
@pcattori unless you have any specific ideas for where Remix HMR might be causing this I'm thinking this may need to be investigated on the MUI side?

I thought I might add that Vite and Next do not have this issue with the exact same modal setup. Unsure if it is a bug or just the nature how Remix's HMR works.

I have a similar problem with modals, where HMR simply doesn't work (it refreshes the page).
I will work on a repro so you can have a better look, but just by having an OverlayProvider component from 'react-aria' as a parent component for the Outlet in root.tsx, is enough to stop HMR from working. Even if there is no modal using that provider.
After building the repro I will take a look at the implementation of the OverlayProvider myself, since I am still unsure whether that is a problem wiht that library or remix.

Same thing happens for me -- it's most obvious in modals (which are always triggered by a setState call), but really all state gets blown away on any HMR update in my app. For me it was DndProvider from react-dnd, which wraps my whole app. At a glance it seems to be a basic wrapper around a react context, but something about it doesn't like HMR. Sounds like a similar problem with these other providers.

I wrote up some limitations of React Fast Refresh in the docs. Could be that things like DndProvider are class components? 🤷

@pcattori I know at least for react-dnd, it doesn't use classes. I remember seeing something similar in vite, where contexts were breaking HMR. They did something to the react-refresh code to deal with this.

I took a look into mui-modal-provider and it does not use classes either.


Outliner with inline equation does not appear well

When the title is an inline equation, the outliner does not appear well.
This occurs with the newest enhancer release. Please see https://yingda.notion.site/test-902aa1a64b1a4bbdae9ac6981ea9168b for the test page.

Formatting can't be rendered in the outliner. Though unfortunate, this isn't a bug.

The $\lambda$ is rendered somehow, but with an improper formatting.
If possible, can you remove the trailing $\lambda$ 1 and only retain the unrendered code \lambda_1?
This is also what the default table-of-content does.

Perhaps, I'll look into it.

Many thanks!


Failed at the<EMAIL_ADDRESS>postinstall script

I have gone through these following points

[x]   Check latest documentation: https://docs.nativebase.io/
[x]   Check for existing open/closed issues for a possible duplicate before creating a new issue: https://github.com/GeekyAnts/NativeBase/issues
[x]   Use the latest NativeBase release: https://github.com/GeekyAnts/NativeBase/releases
[x]   Check examples from NativeBase KitchenSink https://github.com/GeekyAnts/NativeBase-KitchenSink
[x]   For discussion purpose make use of NativeBase Slack: http://slack.nativebase.io/
[x]   For queries related to theme, check Theme Variables from Docs and live NativeBase Theme Editor http://nativebase.io/customizer/

node, npm, react-native, react and native-base version, expo version if used, xcode version
npm = 6.10.2, react-native=0.59.5, react=16.8.3, Xcode: 10.3, no expo
Expected behaviour
native-base version must be updated to 2.14.1
Actual behaviour
unable to install<EMAIL_ADDRESS>Steps to reproduce
migrate the native-base from 2.12.1 to 2.14.1
Issue Description
I am using react-native 0.59 and native-base 2.12.1, I want to update the native-base to 2.13.4 ( or latest) but when I tried to update with
$npm install<EMAIL_ADDRESS>I get the error

><EMAIL_ADDRESS>postinstall /Users/myuser/Documents/Workspace/ReactNative/app/app/node_modules/native-base
> node addEjectScript.js && opencollective-postinstall

┌────────────────────────────────────────────────────────────────────────────────────────────┐
│ NativeBase 2.0 has been succesfully installed!                                             │
│ Run node node_modules/native-base/ejectTheme.js to copy over theme config and variables. │
│ Head over to the docs for detailed information on how to make changes to the theme.        │
└────────────────────────────────────────────────────────────────────────────────────────────┘
sh: opencollective-postinstall: command not found
npm WARN<EMAIL_ADDRESS>requires a peer of react-art@>= 16.3.2 but none is installed. You must install peer dependencies yourself.
npm WARN<EMAIL_ADDRESS>requires a peer of react-dom@>= 16.3.2 but none is installed. You must install peer dependencies yourself.
npm WARN<EMAIL_ADDRESS>requires a peer of react-native-web@>= 0.7 but none is installed. You must install peer dependencies yourself.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.

npm ERR! file sh
npm ERR! code ELIFECYCLE
npm ERR! errno ENOENT
npm ERR! syscall spawn
npm ERR<EMAIL_ADDRESS>postinstall: node addEjectScript.js && opencollective-postinstall
npm ERR! spawn ENOENT
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>postinstall script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR!      /Users/myuser/.npm/_logs/2019-07-31T04_36_44_052Z-debug.log````

Solved with deleting node_modules/ and $npm install

Its not working with this solution ..... deleted whole project and cloned it again but getting same error

i am getting same issue

i am getting same issue

I am also getting same error


Testing a pull request

Sample snippet contributions
For client library sample snippets, review and follow the snippet contribution guidance

[ ] Follow organization guidelines
[ ] Use real types (avoids var)
[ ] Use Context.Log (avoids Console.WriteLine)
[ ] Sample should clean up resources they create
[ ] Avoid samples that arbitrarily destroy data

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


Some toolbar button icons not displayed

Describe the bug
When setting some custom buttons, the icons aren't displayed.
To Reproduce
let ret = new EasyMDE({
    element: document.getElementById('editor'),
    toolbar: [{
        name: 'save',
        action: () => { },
        className: 'fa-solid fa-floppy-disk',
        title: 'Save file'
    }, {
        name: 'saveas',
        action: () => { },
        className: 'fa-solid fa-floppy-disk-pen',
        title: 'Save file as'
    }]
});

Expected behavior
Both buttons will display their icon.
Screenshots

The button is there and calls the action when clicked, but the icon is not displayed.
Version information

OS: Windows
Browser: Edge 97 (embedded WebView2)
EasyMDE version: 2.16.1

Additional context
I have noticed that buttons which have this problem, don't have a pseudo-::before element in the Elements view:

which is consistent with the described behavior, because FontAwesome icons are inserted via the ::before pseudo-selector.

fa-solid fa-floppy-disk looks like a Fontawesome 5 or 6 icon and is not available in the FontAwesome version the editor uses by default (4.7). The 4.7 equivalent is fa fa-floppy-o.
fa-solid fa-floppy-disk-pen is a Pro icon and can only be used when you load in a Pro version of FontAwesome.

With FontAwesome 6 loaded:


 There is no release build, but you can compile it yourself with gradle.

does project is still alive? does procyon version will be updated?


Make input placeholder configurable

The placeholder "Search as you type... " should be configurable

done and deployed


Markdown text in report comment is handling short mentions and long mentions wrong

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.52-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:
Email or phone of affected tester (no customers<EMAIL_ADDRESS>Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Jakub Szymczak - Software Mansion external agency
Slack conversation:
Action Performed:

Have somebody(eg. jakub.szymczak) in your "proposed mention list" just like in video and this user has to have the same email domain as you do. To add smb to your proposed mention list just write a message to him.
Try to mention him with his whole email address.

Expected Result:
When you want to "long mention" anyone, then the whole email address should be contained in mention body.
Actual Result:
The email domain is not contained in mention body, it's just a plain text
Workaround:
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/88395093/2fbfdef8-9d77-4c4d-a6ee-fb20cada74c6

View all open jobs on GitHub

I think this is potentially a dupe of https://github.com/Expensify/App/issues/38303 - fixing that should fix this one.


1.20.4 Update

Hello there,
It seems like worldguardflag needs updating to 1.20.4, I hope the developer can update his own plugin soon, much appreciated!

Thanks

nvm - My bad lol!


[FEAT]: Confusing and clunky workspace chat verbiage and UX. Don’t cross the streams please.

What would you like to see?
Apologies for the novel here. Stick with it though :) Isn’t as big of an ask as it seems. Thanks

First.
New threads get the title of New Thread. That’s fine by itself. But not when the button to create a new thread goes by the same call sign on the screen. It’s too repetitive, and too close together. One thing is something while another does something… in the same spot. Not everybody takes the time to rename the threads. It would serve folks better to have a pretty timestamp as the initial name of the thread. That way, they can tell a bit more context than New Thread. This would take very little time to tweak. Moment to the rescue. This week, I must have clicked the New Thread button 50 times to open an existing thread only to find out it opened another thread. They multiply like rabbits because the eyes can’t tell them apart.
Second.
The button named New Thread isn’t so bad by itself. It is clunky & redundant verbiage, though, and paired with the plus sign, and being a chat, it is very obvious what it does. In fact, the 2 extra words contribute to a more busy ui. If that is desired, then hey, whatever. It isn’t my product. But after using it a while, honestly, seeing “New Thread” “New Thread” all over the place… it’s a bit tiring to look at and use. I’m sure that is not the feeling you want it to invoke. The New Workspace button’s core color is a perfect starting color for the button to add a new thread. If you’re looking for an alternative or another riff to my suggestion above, maybe wrap the New Thread text up inside a white button with the plus sign (exactly like the New Workspace button - hey theming!) and then extend the workspace “tree” down to the button.
Third.
The new workspace button color is great, but for being such a big button in the top of the screen it is a little “loud” and top-heavy feeling. It makes all the other buttons not as interesting, especially when it precedes them . Could you keep the color but mute it a bit? To be completely honest, I think the best thing to do here is swap the color scheme of the new workspace button with the color scheme of the currently selected workspace. That way a user’s eye is drawn to the thing they are working on instead of distracting them.
Welp, there’s my thoughts after spending a week straight looking at it every day. And hopefully you have some ideas flowing for making things look more polished.
I have a lot more feelings on the ui in general too, if you’re interested. Dropped some suggestions into discord, focused on the document indexing/upload modal.
We are about to deploy a customer chat bot using anythingLLM this summer, so looking forward to using it more and getting set up in k8s.

I completely agree with your points regarding the UI, especially on the need for improvements. The repetition of the "New Thread" title and button is indeed confusing and can make navigation cumbersome. An initial timestamp for new threads would definitely provide more context and reduce unnecessary confusion.
The suggestion to streamline the button labeling and adjust color schemes to draw attention more effectively is spot-on. Simplifying the "New Thread" button while aligning it with existing themes can make the interface cleaner and more user-friendly.
Overall, these enhancements would certainly make the user experience more intuitive and visually appealing. Looking forward to seeing these improvements!

Wow those are pretty slick tools that are self hostable. I’m gonna check those out tonight for the fun of it. Not sure if they have all the features of anything LLM but they are def good to draw inspiration from.
Not really in the exact same space, but have been using plandex recently. Badass, if you like CLI-only tools. Am always curious down much ai devs use to bootstrap their ai tools.


feat: make versions work with id

This change is

/deploy

/deploy

/deploy

/deploy

/deploy


[ADD] Add a Reset Button to Clear All Filter Fields in Navbar

Add a Reset Button to Clear All Filter Fields in Navbar
Description
A new Reset button should be added to the navigation bar to allow users to clear all filter fields in one click. This will improve user experience by providing a quick way to reset filters without manually clearing each field.

Task Details

Frontend Changes:

Modify the Navbar.jsx file located in codebase/frontend/src/components/.
Add a Reset button in the navigation bar layout.
Implement functionality to clear all applied filter fields when the button is clicked.
Ensure the button is styled consistently with the existing design.

Behavior Specifications:

Clicking the Reset button should remove all active filters and restore the default view.
Ensure the button works across different pages where filters are applied.

Testing:

Test the button to ensure it clears all filter fields as expected.
Verify that the reset functionality does not affect other components of the application.

Follow Pull Request Guidelines:

Provide meaningful comments in the code for clarity.
Follow the Pull_request_template.md file when submitting the Pull Request.

Hi, i want to work on this pleae can you assign this to me


Update README.md -- bold the "organization-level"

Adding cla: yes label, as the PR author is a googler at the time of this PR was created.


How ca we add Headers?

Hello, This package is great! But is there any way to pass headers when accessing an image? I have a protected endpoint to avoid unauthorized users accessing images.

Hi @ralppppy,
Currently this option does not exist, nevertheless we've added this issue and fix it quite soon.

Feature added. Please refer<EMAIL_ADDRESS>


Help Generator Api Design

This is still not nailed down pre-1.0, and I'm not totally happy with the developer workflow.
Things like 'ignore required' when --help is invoked, leave EntryPoint in a footgun position, if a developer doesn't realise they should check .HelpRequested after parsing.
0.9.5 is out there, but may yet change again.
A current example is detailed here: https://nick-lucas.github.io/EntryPoint/

Two problems here:

Help execution needs to disable Required flags
Developers need to have control of quitting their application
Disabling flags may be a foot-gun scenario if the program does not exit

Design so far:
Route --help to an abstract Help method in BaseCliArguments, just like BaseCliCommands
This will enforce the implementation of this method by developers, and give them a clear opportunity to exit.
Set flag on CliArguments so the developer can check this later.

Closed by aa2a72e9dc7357ebcdef6daffc9728e3d8e37522


Update docs for CRD subresources for 1.11

Ref:

https://github.com/kubernetes/kubernetes/pull/63598
https://github.com/kubernetes/kubernetes/pull/63533

/hold
until https://github.com/kubernetes/kubernetes/pull/63598 is merged.
/assign sttts
Preview Link - CRD subresources page

/lgtm
/approve

The underlying structure of the docs repository has changed due to the recent
Hugo migration. Specifically, all the files previously under docs/ are now in
content/en/docs/. I have rebased your PR without changing any of your work, so
that your changes are modifying the affected files in the new location, so this
PR is now reviewable again.
If you were using Github's web UI rather than a local Git client, you can stop
reading and continue working in this PR.
If you use the Git command-line client, you will need to make sure your local
branch is up to date with your fork. If you have local commits that have not
been pushed to your fork, it may be complicated to get them back. Do git log
and note their commit hashes for later.
Assuming your your fork's remote is called origin, hard-reset your local
branch to be the same as your fork's branch:
git fetch origin; git reset --hard origin/update-crd-1.11

If you didn't have local commits, you're now ready to keep working on this PR
locally. You can stop reading now.
If you did have local commits, cherry-pick them from oldest to newest. For
each commit that needs to be cherry-picked:
git cherry-pick <HASH>

You will need to resolve conflicts in these cherry-picked commits, which
involves moving your modified files into the correct new location. All the
content in docs/ moved to content/en/docs/, so you can move your local files
accordingly:

Do git add for each of the moved files
Do git mergetool and accept the
deleted rather than modified file
Continue the cherry-pick by doing git cherry-pick --continue

When you've cherry-picked all local commits back onto the local branch, push the
branch (with new commits) to your fork. You should not need to force the push.

Thanks for the review, @mistyhacks!
Updated the PR. PTAL, thanks.
https://github.com/kubernetes/kubernetes/pull/63598 has been merged. Removing the hold.
/hold cancel

latest Netlify deploy (built with latest commit): https://deploy-preview-8519--kubernetes-io-vnext-staging.netlify.com/

Should it be there? If so, can you add it and reference this docs PR in the
appropriate field? Does it need a release note?

Yes, I have reached out to @justaugustus about it. :+1:

Just an update: it has now been updated in the feature tracking spreadsheet.

Thanks!


Homework westeros

Open Source Energy Modelling — Homework

Add input for gas power plant and update technoeconomic data and emissions.
Setup bounds on emissions and tax on GHGs.

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


Use gogo Un/Marshal for version and cve store

Description
Adjust to generated code.
Testing Performed
CI

Current dependencies on/for this PR:

master

PR #3446

PR #3448

PR #3449

PR #3450   👈

This comment was auto-generated by Graphite.

Current dependencies on/for this PR:

master

PR #3446

PR #3448

PR #3449

PR #3450   👈

This comment was auto-generated by Graphite.

/cc @mtodor @misberner @rhybrillou

/retest


s/args/props/g

This example code was failing to run. Are docs pages rebuilt automatically if API.md is updated, or is there another step?

This change is

Reviewed 1 of 1 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


feat(mysql): mysql逻辑恢复关闭slow_log

关联研发信息
dev_branch: feat/dbbackup_slowlog/#7965
target_branch: v1.5.0


[Event Request] Codeunit "Requisition Line - Price" event for AddSources()

Could we please have an event at the end of AddSource() similar to codeunits such as "Purchase Line - Price".
local procedure AddSources()
var
    SourceType: Enum "Price Source Type";
begin
    PriceSourceList.Init();
    PriceSourceList.Add(SourceType::Vendor, RequisitionLine."Vendor No.");

    OnAfterAddSources(RequisitionLine, CurrPriceType, PriceSourceList);
end;

[IntegrationEvent(false, false)]
local procedure OnAfterAddSources(RequisitionLine: Record "Requisition Line"; PriceType: Enum "Price Type"; var PriceSourceList: Codeunit "Price Source List")
begin
end;

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


The access tiers are not localized for one GPV2 storage account

Storage Explorer Version: 1.11.0
Build: 20191105.2
Branch: rel/1.11.0
Platform/OS: Windows 10/ Linux Ubuntu 19.04/macOS High Sierra
Language: Czech/ German/ Spanish/ French/ Hungarian/ Italian/ Japanese/ Korean/ Dutch/ Polish/ Portuguese (Brazil)/ Portuguese (Portugal)/ Russian/ Swedish/ Turkish/ Chinese (Simplified) / Chinese (Traditional)
Architecture: ia32/x64
Regression From: Not a regression
Steps to reproduce:

Launch Storage Explorer.
Open 'Settings' -> Application (Regional Settings) -> Select 'Chinese (simplified)' -> Restart Storage Explorer.
Select one GPV2 storage account -> Right click it -> Click 'Set Default Access Tier...'.
Check the dialog.

Expect Experience:
The access tiers are localized.
Actual Experience:
The access tiers are not localized.

More Info:

Verified this issue in Chinese(zh-CN)/ Chinese(zh-TW)/ Japanese/ Korean.

This issue also reproduces for one regular blob.

This issue doesn't reproduce for one ADLS Gen2 blob.

Localized at 10/20/19.
https://devdiv.visualstudio.com/DevDiv/_git/StorageExplorer?_a=annotate&path=%2Floc%2Floc%2Fzh-Hans%2Fsrc%2Fcomponents%2Fse-blob-extension%2Fresources%2FBlobActions.json.lcl&version=GBmaster

This issue also reproduces in Japanese.

Fix merged into master.

Verified this issue on build 20200216.2. 'Archive' is not translated for one regular blob. So we reopen it.

Ok. I'll need to check back later once that one is translated.

Verified on build 20200226.4.
This issue still reproduces in Swedish. So we open it.

@lukaschal are those translations correct for Swedish? I'm looking at se-blob-extension/resources/sv/BlobActions.json

@MRayermannMSFT Verifying with linguist.

@lukaschal any word from the linguist?

@MRayermannMSFT Localized now.
SRC: Hot
New TGT: Frekvent
SRC: Cool
New TGT: Lågfrekvent

Cool Thanks.

Verified.


UnitTests.Viewer: Work on nulalble annotation, formatting, code style

Description
Follow up: https://github.com/MudBlazor/MudBlazor/pull/10107
Part of https://github.com/MudBlazor/MudBlazor/issues/6535
Checklist

[x] The PR is submitted to the correct branch (dev).
[x] My code follows the code style of this project.
[ ] I've added relevant tests.

Do you have a script that changes
<MudTreeView T="string" @bind-SelectedValue="SelectedValue" Disabled="@Disabled" SelectionMode="SelectionMode.ToggleSelection">

to
<MudTreeView T="string"
             @bind-SelectedValue="SelectedValue"
             Disabled="@Disabled"
             SelectionMode="SelectionMode.ToggleSelection">

?

Do you have a script that changes

Sadly, not.
As far as I can see razor is all about manually checking and doing everything, there is no analyzer to throw warning if you are missing quotes: Parameter=value vs Parameter="value", the dotnet format doesn't even check if lambdas have space x => x etc. Truly disappointing.

Do you have a script that changes

Sadly, not. As far as I can see razor is all about manually checking and doing everything, there is no analyzer to throw warning if you are missing quotes: Parameter=value vs Parameter="value", the dotnet format doesn't even check if lambdas have space x => x etc. Truly disappointing.

😔
One tip I have is that ChatGPT can help. but it's not perfect.


Deprecation of retention_policy in diagnostic settings

╷
│ Warning: Argument is deprecated
│
│   with module.azure_container_apps_hosting.azurerm_monitor_diagnostic_setting.cdn,
│   on .terraform/modules/azure_container_apps_hosting/cdn.tf line 349, in resource "azurerm_monitor_diagnostic_setting" "cdn":
│  349: resource "azurerm_monitor_diagnostic_setting" "cdn" {
│
│ `retention_policy` has been deprecated in favor of `azurerm_storage_management_policy` resource - to learn more https://aka.ms/diagnostic_settings_log_retention
│
│ (and 15 more similar warnings elsewhere)

We need to remove any reference to the retention_policy property for diagnostic settings, and instead switch to using a storage_management_policy resource

https://registry.terraform.io/providers/hashicorp/azurerm/latest/docs/resources/storage_management_policy
https://aka.ms/diagnostic_settings_log_retention

https://github.com/DFE-Digital/terraform-azurerm-container-apps-hosting/pull/267


Added ability to have multiple chats

@mattmezza Please let me know if you have any questions or want to discuss. I updated the the chat and the demo, but not your doc. I can do it if necessary. The old demo works with the new chat, so the changes are backward compatible.

@mattmezza Hi Matteo! It looks like mine and several other pull requests failing checks with error "Error: Resource not accessible by integration". It seems to be only happening right after message "Adding message: Thanks for sending out your first PR! to pull request". Could you take a look? https://github.com/mattmezza/vue-beautiful-chat/pull/194/checks?check_run_id=1239683641

@mattmezza I'm done with my fixes. Please take a look again. As for @a-kriya 's idea, why don't you merge it? It would be easier for contributors to see it and do the same. Right now I don't feel like converting my code to this style within this already large pull request.

@eduardsmetanin It's been merged. Since there are now conflicts with master, you'll have to rebase and resolve them. And using the store to access the necessary props will reduce the size of the diff.

@mattmezza @a-kriya This pull request is rebased and ready to be merged. I tried to use store to pass props, but I ran into 2 issues and rolled back store.state my changes:

A prop is undefined when accessed from a method called from data() and this prop is populated via ...mapState(). Perhaps initial value can be set like this:
const store = { state: Vue.observable({   editMessage: null }),, but I ran into a case when I needed my data (initialized in data()) to depend on a prop, which didn't work because of  undefined value in the prop. As a workaround this prop can be accessed via store.state['my_prop_name']. This caused me a lot of time and frustration. Vue props work well in this case and I think we need a state management system that works just like standard props.
We currently update entire state map on every prop change:
handler(props) {       // TODO: optimize       for (const prop in props) {         store.setState(prop, props[prop])       }     }. The TODO says it needs to be optimized. I don't know how it can be optimized.

Does Vue 3 provide an easier way to manage state? Perhaps we should wait for Vue 3 conversion and then change how we manage state?

A prop is undefined when accessed from a method called from data() and this prop is populated via ...mapState()

Expected behavior: https://jsfiddle.net/os3rxwyd/
data is evaluated before computed: https://forum.vuejs.org/t/computed-properties-in-data/11231/3

We currently update entire state map on every prop change:
The TODO says it needs to be optimized. I don't know how it can be optimized.

Insignificant; I'd intended to remove that comment but missed. The single statement in the loop just updates the references in the state object internally. It does not make any copies, and is fairly equivalent to running a few assignment statements.

Does Vue 3 provide an easier way to manage state? Perhaps we should wait for Vue 3 conversion and then change how we manage state?

There's not much Vue 3 will help us with here. The props are essentially put into a reactive object that is accessed from the components. That's all.

@mattmezza @a-kriya I pushed fixes for @a-kriya suggestions.

@eduardsmetanin Thank you. There's still a lot going on here, and I will need a bit to go through it and test the updates. In the meantime, if you think there are any parts to this diff that will be easier to review with some annotations, please go ahead and leave some comments at those lines.

@mattmezza Could you merge this PR? Are you waiting for something?

Hi guys,
Any news about this PR? I'm finding this feature. @eduardsmetanin if I clone your repo, this is 100% ok?

Hi guys,
Any news about this PR? I'm finding this feature.
@eduardsmetanin if I clone your repo, this is 100% ok?
Doc in your repo is outdated!

@silasrm You can use my clone if you want. I do NOT promise to maintain it though. I haven't updated any docs yet. It would be good to see a will to merge my changes from maintainer @mattmezza before I spend any more time on it.

Hi guys,
Any news about this PR? I'm finding this feature.
@eduardsmetanin if I clone your repo, this is 100% ok?
Doc in your repo is outdated!

@silasrm You can use my clone if you want. I do NOT promise to maintain it though. I haven't updated any docs yet. It would be good to see a will to merge my changes from maintainer @mattmezza before I spend any more time on it.

I'm using after clone and build. Now, I'm trying to integrate to https://github.com/musonza/chat :D

@eduardsmetanin I add support to toolbox on emoji and file message types.
https://github.com/beefreelancer/vue-beautiful-chat/pull/1

@mattmezza Could you please merge this pull request?

@eduardsmetanin Could you maybe solve the conflict so that your pull request can be merged in? :)

@a-kriya Will you merge this pull request if I resolve conflicts? People ask for my changes once in a while, so I think it would be beneficial for the project. I can walk you through my changes via a conference call if you wish.
Attn: @miloandco @oldner

@eduardsmetanin Could you maybe solve the conflict so that your pull request can be merged in? :)


Upgrade dev dependencies

Upgraded dev dependencies with high-severity vulns: webpack and firebase-tools.

Builds and passes all frontend tests so looks ok to merge.  Approved and merging...


Update Set-MailboxJunkEmailConfiguration.md

#9380

Docs Build status updates of commit fe9fcdc:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/exchange-ps/exchange/Set-MailboxJunkEmailConfiguration.md
:white_check_mark:Succeeded
View (exchange-ps)

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

Docs Build status updates of commit fcbdc79:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/exchange-ps/exchange/Set-MailboxJunkEmailConfiguration.md
:white_check_mark:Succeeded
View (exchange-ps)

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


Updating confluence pages fails on v2

After upgrading to version 2.5.0, I'm not able to update existing Confluence pages using python -m sphinx -M confluence . _build.  If I downgrade to version 2.4.0, run the command, and then upgrade to version 2.5.0, it works.  Is there a way to make the latest version work directly without the extra step?
This is the log from the initial failing run on version 2.5.0:
# Platform:         win32; (Windows-11-10.0.22631-SP0)
# Sphinx version:   7.2.6
# Python version:   3.12.0 (CPython)
# Docutils version: 0.20.1
# Jinja2 version:   3.1.3
# Pygments version: 2.17.2

# Last messages:
#
#
#
#   publishing documents... [ 10%]
#   index
#
#
#   publishing documents... [ 20%]
#   readme_link
#

# Loaded extensions:
#   sphinx.ext.mathjax (7.2.6)
#   alabaster (0.7.16)
#   sphinxcontrib.applehelp (1.0.8)
#   sphinxcontrib.devhelp (1.0.6)
#   sphinxcontrib.htmlhelp (2.0.5)
#   sphinxcontrib.serializinghtml (1.1.10)
#   sphinxcontrib.qthelp (1.0.7)
#   sphinx.ext.autodoc.preserve_defaults (7.2.6)
#   sphinx.ext.autodoc.type_comment (7.2.6)
#   sphinx.ext.autodoc.typehints (7.2.6)
#   sphinx.ext.autodoc (7.2.6)
#   sphinx.ext.viewcode (7.2.6)
#   sphinx.ext.napoleon (7.2.6)
#   sphinxcontrib.jquery (4.1)
#   sphinx_rtd_theme (unknown version)
#   sphinx.ext.intersphinx (7.2.6)
#   sphinxcontrib.mermaid (7.2.6)
#   myst_parser (2.0.0)
#   sphinxcontrib.confluencebuilder (2.5.0)

# Traceback:
Traceback (most recent call last):
  File "C:\Program Files\Python312\Lib\site-packages\sphinx\cmd\build.py", line 298, in build_main
    app.build(args.force_all, args.filenames)
  File "C:\Program Files\Python312\Lib\site-packages\sphinx\application.py", line 355, in build
    self.builder.build_update()
  File "C:\Program Files\Python312\Lib\site-packages\sphinx\builders\__init__.py", line 293, in build_update
    self.build(to_build,
  File "C:\Program Files\Python312\Lib\site-packages\sphinx\builders\__init__.py", line 366, in build
    self.finish()
  File "C:\Program Files\Python312\Lib\site-packages\sphinxcontrib\confluencebuilder\builder.py", line 760, in finish
    self.publish_doc(docname, output)
  File "C:\Program Files\Python312\Lib\site-packages\sphinxcontrib\confluencebuilder\builder.py", line 509, in publish_doc
    uploaded_id = self.publisher.store_page(title, data, parent_id)
                  ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "C:\Program Files\Python312\Lib\site-packages\sphinxcontrib\confluencebuilder\publisher.py", line 1093, in store_page
    self._update_page(page, page_name, data, parent_id=parent_id)
  File "C:\Program Files\Python312\Lib\site-packages\sphinxcontrib\confluencebuilder\publisher.py", line 1556, in _update_page
    updated_prop['id'] = orig_entry['id']
                         ~~~~~~~~~~^^^^^^
KeyError: 'id'

Here is my configuration:
(system)
 platform: Windows-11-10.0.22631-SP0
   python: 3.12.0 (tags/v3.12.0:0fb18b0, Oct 2 2023, 13:03:39) [MSC v.1935 64 bit (AMD64)]
   sphinx: 7.2.6
 docutils: 0.20.1
 requests: 2.31.0
  urllib3: 2.1.0
  builder: 2.5.0

(configuration)
confluence_disable_autogen_title: True
confluence_page_hierarchy: True
confluence_parent_page: (set)
confluence_publish: True
confluence_server_pass: (set)
confluence_server_url: https://(removed) (cloud)
confluence_server_user: (set)
confluence_space_key: (set; mixed)

(confluence instance)
 connected: yes
   fetched: yes
   decoded: yes
    parsed: yes
      type: confluence
   version: 1000.0.0-5dc985662b86

@cdcadman, thanks for reporting this. For sure a bug has been introduced to support the new Cloud APIs. I will try to push out a fix/release this weekend to address this.
As for now, a workaround would be to force using the older/original API mode when using a Cloud target:
confluence_api_mode = 'v1'

An update has been made to the implementation which should address this issue, and a v2.5.1 release has been made and published to PyPI. Will go ahead and close this issue, assuming it has resolved this use case. Feel free to comment here if there are still issues (which we can re-open and try to address).


Add FromReflect imls

Objective

Some components don't have FromReflect derive.

Solution

Add it.

Changelog
Added

FromReflect for Visibility, Parent, Children.

Doesn't it assume a default value?

If I understand correctly, It creates a "default" values using FromWorld impl as Reflect do.
I also not sure if having FromReflect for Parent and Children is necessary. Should I remove it then?

Yeah, remove those for now, we can add them later if needed.

Okay, done!


register

@JuliaRegistrator register

Registration pull request created: JuliaRegistries/General/68903
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if the Julia TagBot GitHub Action is installed, or can be done manually through the github interface, or via:
git tag -a v0.1.0 -m "<description of version>" fc90ca9bea211cdbe515e73029e8421d8436f71e
git push origin v0.1.0

@JuliaRegistrator register

Error while trying to register: "Tag with name v0.1.0 already exists and points to a different commit"

@JuliaRegistrator retrigger

Error while trying to register: Action not recognized: retrigger

@JuliaRegistrator register

Error while trying to register: "Tag with name v0.1.1 already exists and points to a different commit"

@JuliaRegistrator register(branch="main")

@JuliaRegistrator register

@JuliaRegistrator register

Error while trying to register: "Tag with name v0.1.1 already exists and points to a different commit"


IOSXRDriver Timed out looking for SSH login password prompt

Describe the bug
When using the IOSXRDriver with ssh-config and ssh-key type authentication  I got TimeoutError: Timed out looking for SSH login password prompt.
Strange is that in debug I can see the scrapli did actually succeed to connect to my router but seems like it doesn't match the returned prompt and failed.  I can also see in my router's log that test user did connect succesfully.
What is also interesting that when I try to connect to another IOS-XR router via username/password (no ssh-config, no ssh-key) then everything works ok.
To Reproduce
Steps to reproduce the behavior:

Your script

from scrapli.driver.core import IOSXRDriver
import logging

logging.basicConfig(level=logging.DEBUG)
logger = logging.getLogger("scrapli")

args = {
    "host": "lab-test-r2n0-tt",
    "auth_strict_key": False,
    "ssh_config_file": True,
    "timeout_socket": 15,
}
conn = IOSXRDriver(**args)
conn.open()

INFO:driver:Opening connection to lab-test-r2n0-tt
INFO:transport:Attempting to authenticate to lab-test-r2n0-tt
INFO:transport:Attempting to open session with the following command: ['ssh', 'lab-test-r2n0-tt', '-p', '22', '-o', 'ConnectTimeout=15', '-o', 'ServerAliveInterval=10', '-o', 'StrictHostKeyChecking=no', '-o', 'UserKnownHostsFile=/dev/null', '-F', '/home/test/.ssh/config']
DEBUG:transport:Session to host lab-test-r2n0-tt spawned
DEBUG:transport:Attempting to authenticate. Read: b'\r\n\r\n'
DEBUG:transport:Attempting to authenticate. Read: b'RP/0/RP0/CPU0:lab-test-r2n0-tt#'
INFO:scrapli:Timed out looking for SSH login password prompt
INFO:scrapli:timeout_exit is True, closing transport
DEBUG:transport:Channel to host lab-test-r2n0-tt closed
CRITICAL:transport:Failed to open connection to host lab-test-r2n0-tt. Do you need to disable `auth_strict_key`?
---------------------------------------------------------------------------
TimeoutError                              Traceback (most recent call last)
<ipython-input-1-884c1bd1799b> in <module>
     12 }
     13 conn = IOSXRDriver(**args)
---> 14 conn.open()

/usr/local/lib/python3.6/dist-packages/scrapli/driver/driver.py in open(self)
    575         """
    576         LOG.info(f"Opening connection to {self._initialization_args['host']}")
--> 577         self.transport.open()
    578         if self.on_open:
    579             self.on_open(self)

/usr/local/lib/python3.6/dist-packages/scrapli/transport/systemssh.py in open(self)
    315
    316         # If public key auth fails or is not configured, open a pty session
--> 317         if not self._open_pty():
    318             msg = f"Authentication to host {self.host} failed"
    319             LOG.critical(msg)

/usr/local/lib/python3.6/dist-packages/scrapli/transport/systemssh.py in _open_pty(self, skip_auth)
    453         if skip_auth:
    454             return True
--> 455         self._pty_authenticate(pty_session=self.session)
    456         if not self._pty_isauthenticated(self.session):
    457             return False

/usr/local/lib/python3.6/dist-packages/scrapli/decorators.py in timeout_wrapper(channel_or_transport, *args, **kwargs)
     75                         session_lock.release()
     76                     close()
---> 77                 raise TimeoutError(message)
     78             return future.result()
     79

TimeoutError: Timed out looking for SSH login password prompt

What you're connecting to (vendor, platform, version)

RP/0/RP0/CPU0:lab-test-r2n0-tt#show version
Thu May 28 21:23:13.760 CEST
Cisco IOS XR Software, Version 6.6.3
Copyright (c) 2013-2019 by Cisco Systems, Inc.

Build Information:
 Built By     : hlo
 Built On     : Fri Dec 13 17:04:06 PST 2019
 Built Host   : iox-ucs-032
 Workspace    : /auto/srcarchive15/prod/6.6.3/asr9k-x64/ws
 Version      : 6.6.3
 Location     : /opt/cisco/XR/packages/
 Label        : 6.6.3-0

cisco ASR9K () processor
System uptime is 6 weeks 2 days 6 hours

Anything else relevant

not sure if relevant but in order to connect to my router I need to use a jumphost which is specified in my ssh config file as follow:

host jumphost
  HostName jumphost
  User admin

Host lab-test-r2n0-tt
    Hostname <IP_ADDRESS>
    ProxyCommand ssh -q -W %h:%p jumphost

# Default parameters
host *
    User cisco
    IdentityFile ~/.ssh/id_rsa
    IdentitiesOnly yes
    CheckHostIP no
    PasswordAuthentication yes
    ConnectTimeout 20
    StrictHostKeyChecking no
    ForwardAgent yes

Expected behavior
Scrapli should be able connect to the IOSXR device with the existing ssh config.
Stack Trace
Copy of your stack trace here, please format it properly using triple back ticks (top left key on US keyboards!)
Screenshots
If applicable, add screenshots to help explain your problem, but do note that formatted text is much preferred over
screenshots!
OS (please complete the following information):

OS: linux mint 19.3
scrapli version: 2020.5.9
version of any optional extras (paramiko|ssh2-python|textfsm, etc.)

Additional context
Add any other context about the problem here.

Hey @dorko20 thanks for the bug report!
On quick look I think what is happening is we dont find a ssh key (even though it obviously exists) so scrapli tries to password auth and fails because of course it never finds the password prompt.
I think my kinda messy custom implementation of ssh config parser is to blame for this -- I suspect (but need to verify) that we find the hostname in the ssh config but since the key is not associated to that host we dont check the * entry.
This is a really good motivator for me to probably re-do a bit of the SystemSSH auth stuff and the ssh config parser, but that probably will take me a bit to get to....
In the short term I think you could set auth_bypass to True in your args dictionary, this would cause scrapli to just skip the whole authentication/verification process. Obviously not a great fix, but I believe it would get you going while I re-work the other bits. Do note that this is only available on the system transport option though.
Could you give that a try and let me know if that works, and if not some more logs would be fantastic!
Thanks!
Carl

Hi Carl,
thanks a lot for you answer, I tried to set the "auth_bypass": True and now it is working:
In [3]: from scrapli.driver.core import IOSXRDriver
   ...: import logging
   ...:
   ...: logging.basicConfig(level=logging.DEBUG)
   ...: logger = logging.getLogger("scrapli")
   ...:
   ...: args = {
   ...:     "host": "lab-test-r2n0-tt",
   ...:     "auth_strict_key": False,
   ...:     "ssh_config_file": True,
   ...:     "timeout_socket": 15,
   ...:     "auth_bypass": True,
   ...: }
   ...: conn = IOSXRDriver(**args)
   ...: conn.open()

INFO:driver:Opening connection to lab-test-r2n0-tt
INFO:transport:Attempting to authenticate to lab-test-r2n0-tt
INFO:transport:`auth_bypass` is True, bypassing authentication
INFO:transport:Attempting to open session with the following command: ['ssh', 'lab-test-r2n0-tt', '-p', '22', '-o', 'ConnectTimeout=15', '-o', 'ServerAliveInterval=10', '-o', 'StrictHostKeyChecking=no', '-o', 'UserKnownHostsFile=/dev/null', '-F', '/home/test/.ssh/config']
DEBUG:transport:Session to host lab-test-r2n0-tt spawned
INFO:driver:Attempting to acquire `privilege_exec` privilege level
DEBUG:channel:Write (sending return character): '\n'
DEBUG:channel:Read: b'\n'
DEBUG:channel:Read: b'\n\n'
DEBUG:channel:Read: b'RP/0/RP0/CPU0:lab-test-r2n0-tt#'
DEBUG:driver:Current privilege level could be `privilege_exec`
DEBUG:driver:Determined current privilege level is one of `['privilege_exec']`
INFO:driver:Acquired requested privilege level `privilege_exec`
INFO:channel:Attempting to send input: terminal length 0; strip_prompt: True
DEBUG:channel:Write: 'terminal length 0'
DEBUG:channel:Read: b'terminal length 0'
INFO:channel:Read: b'terminal length 0'
DEBUG:channel:Write (sending return character): '\n'
DEBUG:channel:Read: b'\n'
DEBUG:channel:Read: b'Fri May 29 07:56:53.539 CEST\n'
DEBUG:channel:Read: b'RP/0/RP0/CPU0:lab-test-r2n0-tt#'
INFO:channel:Read: b'\nFri May 29 07:56:53.539 CEST\nRP/0/RP0/CPU0:lab-test-r2n0-tt#'
INFO:channel:Attempting to send input: terminal width 512; strip_prompt: True
DEBUG:channel:Write: 'terminal width 512'
DEBUG:channel:Read: b'terminal width 512'
INFO:channel:Read: b'terminal width 512'
DEBUG:channel:Write (sending return character): '\n'
DEBUG:channel:Read: b'\n'
DEBUG:channel:Read: b'Fri May 29 07:56:53.888 CEST\n'
DEBUG:channel:Read: b'RP/0/RP0/CPU0:lab-test-r2n0-tt#'
INFO:channel:Read: b'\nFri May 29 07:56:53.888 CEST\nRP/0/RP0/CPU0:lab-test-r2n0-tt#'
INFO:driver:Connection to lab-test-r2n0-tt opened successfully

In the meantime I also tried to set the "auth_private_key" and "auth_username" attributes to the IOSXRDriver but with no luck, without the  "auth_bypass": True , it doens' work.
In [6]: #conn.close()
   ...: #del conn
   ...: from scrapli.driver.core import IOSXRDriver
   ...: import logging
   ...:
   ...: logging.basicConfig(level=logging.DEBUG)
   ...: logger = logging.getLogger("scrapli")
   ...:
   ...: args = {
   ...:     "host": "lab-test-r2n0-tt",
   ...:     "auth_strict_key": False,
   ...:     "ssh_config_file": True,
   ...:     "timeout_socket": 15,
   ...:     "auth_private_key": "/home/test/.ssh/id_rsa",
   ...:     "auth_username": "cisco",
   ...: }
   ...: conn = IOSXRDriver(**args)
   ...: conn.open()
INFO:driver:Opening connection to lab-test-r2n0-tt
INFO:transport:Attempting to authenticate to lab-test-r2n0-tt
INFO:transport:Attempting to open session with the following command: ['ssh', 'lab-test-r2n0-tt', '-p', '22', '-o', 'ConnectTimeout=15', '-o', 'ServerAliveInterval=10', '-i', '/home/test/.ssh/id_rsa', '-l', 'cisco', '-o', 'StrictHostKeyChecking=no', '-o', 'UserKnownHostsFile=/dev/null', '-F', '/home/test/.ssh/config', '-v', '-o', 'BatchMode=yes']
DEBUG:transport:Session to host lab-test-r2n0-tt spawned
INFO:scrapli:Timed out determining if session is authenticated
INFO:scrapli:timeout_exit is True, closing transport
DEBUG:transport:Channel to host lab-test-r2n0-tt closed
CRITICAL:transport:Failed to authenticate to host lab-test-r2n0-tt with private key `/home/test/.ssh/id_rsa`. Unable to continue authentication, missing username, password, or both.
---------------------------------------------------------------------------
ScrapliAuthenticationFailed               Traceback (most recent call last)
<ipython-input-6-539b1f0116ec> in <module>
     16 }
     17 conn = IOSXRDriver(**args)
---> 18 conn.open()

/usr/local/lib/python3.6/dist-packages/scrapli/driver/driver.py in open(self)
    575         """
    576         LOG.info(f"Opening connection to {self._initialization_args['host']}")
--> 577         self.transport.open()
    578         if self.on_open:
    579             self.on_open(self)

/usr/local/lib/python3.6/dist-packages/scrapli/transport/systemssh.py in open(self)
    307                 )
    308                 LOG.critical(msg)
--> 309                 raise ScrapliAuthenticationFailed(msg)
    310             msg = (
    311                 f"Failed to authenticate to host {self.host} with private key "

ScrapliAuthenticationFailed: Failed to authenticate to host lab-test-r2n0-tt with private key `/home/test/.ssh/id_rsa`. Unable to continue authentication, missing username, password, or both.

Thanks for the update! Glad its at least working with auth_bypass. I'll hopefully be able to test and recreate this this weekend or next weekend and if I can recreate it (guessing that should be no problem) will try to get it fixed and pushed out w/ the next pypi release.
I'll leave this open and update it when I have more info, thanks again for the detailed report!

Hello again!
This should now be fixed in master/latest pypi version (2020.06.06).
The fix at least for the first portion of this issue was to add support for the ssh config file parser to "merge" attributes for a host. Given a config file like yours:
host jumphost
  HostName jumphost
  User admin

Host lab-test-r2n0-tt
    Hostname <IP_ADDRESS>
    ProxyCommand ssh -q -W %h:%p jumphost

# Default parameters
host *
    User cisco
    IdentityFile ~/.ssh/id_rsa
    IdentitiesOnly yes
    CheckHostIP no
    PasswordAuthentication yes
    ConnectTimeout 20
    StrictHostKeyChecking no
    ForwardAgent yes

scrapli would previously not associate the IdentityFile from the * entry with the host lab-test-r2n0-tt -- this has now been updated to ensure that all subsequent config sections of the ssh config file are parsed/loaded up for a given host.
Note that scrapli still does not "fully" parse the ssh config file -- in the case of system transport we really just care about finding a key for auth so that we can use the pipes authentication process of the system transport as this is more reliable than the default pty authentication process.
In any case, the scenario you provided should be working with the currently available scrapli version, so please let me know if there are any remaining issues!
I'm still a bit unsure what caused things to fail when you explicitly passed the ssh key to use for auth though. If that is still an issue for whatever reason would you mind opening a new issue and we can figure it out there?
Thanks again for submitting this, I'll go ahead and close it for now, but if its still an issue feel free to reopen!
Carl


Request: Edit entries

Enable action (option+e?) to edit text-based clipboard items.

Allows you to have a reference open while you make a edit to a URL, for example.
Allows you to remove part of a clipping after copy if it wasn't intended
Allows you to strip sensitive parts of a copy if needed
Allows you to add text searchable by maccy to find specific copies relating to things, in lieu of a titling system (ex: can manually throw in "[Xcode]“ in front of "Copied item 12" so that I can later search for "Xcode" and get any Xcode-related clippings)

Allows you to have a reference open while you make a edit to a URL, for example.

Can you elaborate? I don't understand what this means.

Allows you to remove part of a clipping after copy if it wasn't intended
Allows you to strip sensitive parts of a copy if needed

I don't intend to implement this. Maccy is designed to be a simple solution so if you need to edit part of copy, it's recommended to do it in the editor.

Allows you to add text searchable by maccy to find specific copies relating to things, in lieu of a titling system (ex: can manually throw in "[Xcode]“ in front of "Copied item 12" so that I can later search for "Xcode" and get any Xcode-related clippings)

I plan to implement aliases (or editable titles) for pinned items. The work will be tracked in #89.

Makes sense, thanks! #89 resolves


stable-baselines3 v1.6.2 fixed

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[x] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

Closes #10

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@conda-forge-admin, please rerender

This is blocked by the lack of the correct version of the import-metadata package on conda-forge: https://github.com/conda-forge/importlib_metadata-feedstock/issues/114

@cyprienc . Finally working :tada:

Hello @cyprienc, do you think we can merge? As some months passed, I think we can first restart the CI.

@conda-forge-admin, please restart ci

Hello @cyprienc, do you think we can merge? As some months passed, I think we can first restart the CI.

The CI is happy, I think this is ready for review, thanks a lot!

@conda-forge/python Is this looking good enough for a merge?

@YYYasin19 you should be able to merge the PR now, thanks in advance!

Let's rerender since the last commit is already from a while back

@conda-forge-admin, please rerender


Running run_test.py results in the error testKeyError: 'articulation'.

Error executing job with overrides: ['experiment=Ditto_s2m', 'trainer.resume_from_checkpoint=/home/lqc/Ditto/data/Ditto_s2m-18-43-30/checkpoints/last.ckpt']
Traceback (most recent call last):
File "run_test.py", line 34, in main
return test(config)
File "/home/lqc/Ditto/src/test.py", line 89, in test
trainer.test(model=model, datamodule=datamodule)
File "/home/lqc/anaconda3/envs/Ditto/lib/python3.8/site-packages/pytorch_lightning/trainer/trainer.py", line 906, in test
return self._call_and_handle_interrupt(self._test_impl, model, dataloaders, ckpt_path, verbose, datamodule)
File "/home/lqc/anaconda3/envs/Ditto/lib/python3.8/site-packages/pytorch_lightning/trainer/trainer.py", line 682, in _call_and_handle_interrupt
return trainer_fn(*args, **kwargs)
File "/home/lqc/anaconda3/envs/Ditto/lib/python3.8/site-packages/pytorch_lightning/trainer/trainer.py", line 949, in _test_impl
results = self._run(model, ckpt_path=self.tested_ckpt_path)
File "/home/lqc/anaconda3/envs/Ditto/lib/python3.8/site-packages/pytorch_lightning/trainer/trainer.py", line 1195, in _run
self._dispatch()
File "/home/lqc/anaconda3/envs/Ditto/lib/python3.8/site-packages/pytorch_lightning/trainer/trainer.py", line 1270, in _dispatch
self.training_type_plugin.start_evaluating(self)
File "/home/lqc/anaconda3/envs/Ditto/lib/python3.8/site-packages/pytorch_lightning/plugins/training_type/training_type_plugin.py", line 206, in start_evaluating
self._results = trainer.run_stage()
File "/home/lqc/anaconda3/envs/Ditto/lib/python3.8/site-packages/pytorch_lightning/trainer/trainer.py", line 1281, in run_stage
return self._run_evaluate()
File "/home/lqc/anaconda3/envs/Ditto/lib/python3.8/site-packages/pytorch_lightning/trainer/trainer.py", line 1329, in _run_evaluate
eval_loop_results = self._evaluation_loop.run()
File "/home/lqc/anaconda3/envs/Ditto/lib/python3.8/site-packages/pytorch_lightning/loops/base.py", line 151, in run
output = self.on_run_end()
File "/home/lqc/anaconda3/envs/Ditto/lib/python3.8/site-packages/pytorch_lightning/loops/dataloader/evaluation_loop.py", line 130, in on_run_end
self._evaluation_epoch_end(outputs)
File "/home/lqc/anaconda3/envs/Ditto/lib/python3.8/site-packages/pytorch_lightning/loops/dataloader/evaluation_loop.py", line 230, in _evaluation_epoch_end
model.test_epoch_end(outputs)
File "/home/lqc/Ditto/src/models/geo_art_model_v0.py", line 498, in test_epoch_end
for k, v in result["articulation"].items():
KeyError: 'articulation'
Set the environment variable HYDRA_FULL_ERROR=1 for a complete stack trace.

Issue has solved. Possible causes:
In training,

half the learning rate when changing the batch size.
trainer max epoch > callbacks: early_stopping:patience.


add logging

https://packboard.atlassian.net/browse/FIR-8571
https://packboard.atlassian.net/wiki/spaces/PACB/pages/2715779103/Logging
Current proposal is to add logging for various operations at INFO level:

Engine - Create/Delete/Start/Stop (start includes intermediate startup statuses, so you can see progress in startup duration)
Database - Create/Delete
Cursor - run Query (includes query text)

Note: any of this logging can be configured/turned off. See the top of the management.ipynb notebook

I'm a little on the fence on logging the queries in INFO, but I realise there's already been discussion on this.

This PR is the time for us to all vote and decide, so please speak your mind!

I believe confirmation that sdk is working (e.g. "engine connected", "deleting database", etc.) fits really well into the INFO level, but the queries themselves are useless 90% of the time unless something is broken, from my experience.
As a next step after WARNING, INFO should provide reassurance that everything is working smoothly (if it is) and dumping queries would just clutter it. Maybe not the whole query itself but some sort of performance metric would be better logged at this level? e.g. INFO: query run successfully in 0.01 s
In either case, I'm not too fussed about this since the default level is WARNING and users should not get unexpected output unless they ask for it.

Maybe not the whole query itself but some sort of performance metric would be better logged at this level? e.g. INFO: query run successfully in 0.01 s

Makes sense, I updated so that DEBUG will show the whole query prior to executing, and INFO will only show the rowcount & runtime


[Home] Fix Homepage template - mock the catalog api

Signed-off-by: Emma Indal<EMAIL_ADDRESS>Hey, I just made a Pull Request!
The change introduced here https://github.com/backstage/backstage/pull/11343 require access to the catalog api and it was mocked for the starredEntities home page component story, but not for the homepage template.

:heavy_check_mark: Checklist

[ ] A changeset describing the change and affected packages. (more info)
[ ] Added or updated documentation
[ ] Tests for new functionality and regression tests for bug fixes
[ ] Screenshots attached (for UI changes)
[ ] All your commits have a Signed-off-by line in the message. (more info)

oops!

@kuangp (I reviewed it 😬 )


cannot use set for lab4?

Summary
I used the entrySet function from ImmutableMap for my KeyableMap's toString. It works fine on my terminal but it seems to keep giving me this message on codecrunch. Any advice?

Code Snippets
Here's my code for your reference:

I think you can consider iterating through the whole map and initialising a boolean variable to true to check if the current entry is the first in the map.
    boolean isFirst = true;

Then u cld do something like this in the for loop
if (isFirst == false) {
output += ", ";
} else {
isFirst = false;
}

You cannot use entry set. You can take at look at the top of the labsheet it demonstrate how to iterate through the map without using entryset.

I also faced this issue - what helped me was that I didn't run through the set and instead just the map in my for loop, aka I did .... : map instead of .... : map.entrySet()

adding onto the comments above, you cannot use entrySet, but i realise that you made use of entrySet().size() in order to prevent the inputation of comma and space for the last entry.
To overcome this problem, you can try using a boolean instead.
so 1) you dont add comma if there's only 1 entry and its the first entry in the map
2) the adding of comma is done before you append entry to output, so the last entry will not have comma behind it
boolean firstEntry = true;
for (Map<String, V> entry: map) {
//adding commas
if (!firstEntry) {
output += ", ";
} else {
firstEntry = false;
}
output += entry.getValue();
}

hi! i faced the same problem as you
what i did was for the parameters of the for loop, instead of writing
for(Map.entry<String, V> entry: map.entrySet())    or
for (String key : map.keySet()) {
i did
for (Map.Entry<String, V> currentMap : map)
where iterates through each entry (key-value pair) in the map instead

iterating directly through the map as shown in the assignment rather than entrySet() works perfectly fine for me!

the issue is in your this.map.entrySet() as this method belongs to the ImmutableMap class which uses java.util.set, instead you can easily loop through this.map by using the code prompt given in Code Crunch.

I think entrySet() is the banned API, your perhaps should take some value from another class

Yeah you can't, the preamble of the CodeCrunch exercise shows you how to iterate if need be. Something to do with Entry & Map in the ImmutableMap class


Add BMF DockerFile for CPU/GPU

can face detect and predict demo work in this docker img?
predict can be verified by: https://github.com/BabitMF/bmf/pull/18

can face detect and predict demo work in this docker img? predict can be verified by: #18

I will verfify later, the PR will also be changed.


Simplify file structure in Grading Criteria Section

Apparently the Grading Criteria Section has separate files for mobile and desktop and most code in both of them is the same. Such thing can be omitted by using Media Queries. Please simplify the file structure by having only 1 jsx file and only 1 css file for this section!

Structure is a bit different between the two versions. It's okay to keep it like that. The only thing I would like is to remove the dots from the classNames in the Desktop version.

@override that^^ I want to have only 1 GradingCriteria.jsx file with 1 main div consistent of 4 other divs, each representing a column of the table. I don't want the tags  to be used. Also, only 1 grading_criteria.css file which will use media queries for desktop-to-mobile responsiveness. Please do not overcomplicate simple code and optimize your code wherever it is possible.


KAFKA-6474: [WIP] Rewrite tests to use new public TopologyTestDriver

Please do not merge yet. This is still work in progress.

Remove ProcessorTopologyTestDriver from TopologyTest
Fix ProcessorTopologyTest
Remove ProcessorTopologyTestDriver and InternalTopologyAccessor
Partially refactored StreamsBuilderTest but missing one test
Refactor KStreamBuilderTest
Refactor AbstractStreamTest
Further cleanup of AbstractStreamTest
Refactor GlobalKTableJoinsTest
Refactor InternalStreamsBuilderTest
Fix circular dependency in build.gradle
Refactor KGroupedStreamImplTest
Partial modifications to KGroupedTableImplTest
Refactor KGroupedTableImplTest
Refactor KStreamBranchTest
Refactor KStreamFilterTest
Refactor KStreamFlatMapTest KStreamFlatMapValuesTest
Refactor KStreamForeachTest
Refactor KStreamGlobalKTableJoinTest
Refactor KStreamGlobalKTableLeftJoinTest
Refactor KStreamImplTest
Refactor KStreamImplTest
Refactor KStreamKStreamJoinTest
Refactor KStreamKStreamLeftJoinTest
Refactor KStreamKTableJoinTest
Refactor KStreamKTableLeftJoinTest
Refactor KStreamMapTest and KStreamMapValuesTest
Refactor KStreamPeekTest and KStreamTransformTest
Refactor KStreamSelectKeyTest
Refactor KStreamTransformValuesTest
Refactor KStreamWindowAggregateTest
Add Depercation anotation to KStreamTestDriver and rollback failing tests in StreamsBuilderTest and KTableAggregateTest
Refactor KTableFilterTest
Refactor KTableForeachTest
Add getter for ProcessorTopology, and simplify tests in StreamsBuilderTest
Refactor KTableImplTest
Remove unused imports
Refactor KTableAggregateTest

\cc @bbejeck @vvcephei

@h314to Thanks a lot for this PR -- it's quite big -- I was wondering if we could break it down into multiple smaller PRs? This would make reviewing simpler.

@mjsax Thank you very much for your thorough review. Sorry for the large PR, but a lot of stuff depends on KStreamTestDriver, which trying to remove. I like your suggestion: I'll move all the straightforward stuff into a separate PR, and take care the remaining stuff subsequent PRs. I'll also be addressing your comments above.

@h314to , Thanks for the PR! One strategy you might consider for sending smaller PRs is not to try and remove the new driver, but just to stop using it one test at a time.
This way, you can have an arbitrary number of PRs where you whittle down the usage or the current driver in favor of TopologyTestDriver and then one last PR where you remove the now-unused classes.

@h314to Are you working on the second PR to extract the "straight forward changes"? Or did I miss that you opened a PR for that already?

In order to keep comment history, which might make further code review simpler, I removed the thorny parts from this PR, addressed your previous comments, and rebased to the current trunk. I hope that is ok. I'll be working on the remaining classes and will submit smaller PR with the remaining stuff when they are done.

There were some conflicts due to recent commits in trunk. I rebased and fixed them.

Most of the tests use the same properties except for the application-id. I'm wondering if we want to add a method StreamsTestUtils where you pass in the application id and get back the properties as configured in the tests.
Or since it looks like all the tests use the same setup and cleanup methods do we want an abstract base test and provide a method to define the application id?

@bbejeck good suggestion. I'll suggest filing a JIRA and address it in a different PR :)

@h314to thanks for your contribution! Please ping us when you have the second PR ready for review.

Thank you all for your helpful reviews! @vvcephei left some nice tips after this was merged (thanks!), so if that is ok I'd like to address his comments in a followup cleanup PR.

Hi Filipe,
Yeah, I think that's a good solution. Sorry I was too slow on my review!
Thanks,
-John
On Fri, Apr 27, 2018 at 7:47 AM, Filipe Agapito<EMAIL_ADDRESS>wrote:

Thank you all for your helpful reviews! @vvcephei
https://github.com/vvcephei left some nice tips after this was merged
(thanks!), so if that is ok I'd like to address his comments in a followup
cleanup PR.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/apache/kafka/pull/4832#issuecomment-384959961, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAy1EwXuf-pSSMrtl-KlrIDwXRqbxyNIks5tsxN-gaJpZM4TJXJ_
.

Ah, oh well, nevermind then. I missed that nuance.
On Fri, Apr 27, 2018 at 10:26 AM, Filipe Agapito<EMAIL_ADDRESS>wrote:

@h314to commented on this pull request.
In streams/src/test/java/org/apache/kafka/streams/kstream/internals/
KStreamKStreamJoinTest.java
https://github.com/apache/kafka/pull/4832#discussion_r184721987:

@@ -327,17 +341,15 @@ public void testWindowing() {
assertEquals(1, copartitionGroups.size());
assertEquals(new HashSet<>(Arrays.asList(topic1, topic2)), copartitionGroups.iterator().next());

   driver.setUp(builder, stateDir);

   driver = new TopologyTestDriver(builder.build(), props, time);

While I was changing it I realised that, although it would be cleaner to
remove time and use advanceTimeMs instead, it is actually easier to use
time because the tests require the time to advance and recede.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/apache/kafka/pull/4832#discussion_r184721987, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAy1Eweeb-r26r6ub5VdmfzpBVF815Ntks5tsziMgaJpZM4TJXJ_
.


L.Control.Draw is undefined with angular

Hi,
I'm writing an angular web application which is making use of Leaflet and Leaflet.draw and I seem to be having issues with L.Control.Draw being undefined. Leaflet is working fine and is drawing the map fine. A snippet from one of my controllers is as follows.
angular
    .module('Yellr')
    .controller('newAssignmentGeofenceCtrl', ['$scope', 'leafletData',
    function ($scope, leafletData) {

        var map = L.map('set-geofence-map', {drawControl: true}).setView([51.505, -0.09], 13);

        L.tileLayer('http://{s}.tile.osm.org/{z}/{x}/{y}.png', {
            attribution:
                '© OpenStreetMap contributors'
        }).addTo(map);

        var drawnItems = new L.FeatureGroup();
        map.addLayer(drawnItems);

        // Initialise the draw control and pass it the FeatureGroup of editable layers
        var drawControl = new L.Control.Draw({ // ERROR: Function is undefined
            edit: {
                featureGroup: drawnItems
            }
        });
        map.addControl(drawControl);
    }]);

It seems like L.Control.Draw is never being created. Have you have people have issues with Leaflet.draw and angular before?

I haven't used angular yet but I think you would need to have an angular leaflet.draw directive. See the leaflet one here: https://github.com/tombatossals/angular-leaflet-directive


Split query and skeleton

Improve loaders in all pages with independent queries and skeletons

https://github.com/liteflow-labs/starter-kit/pull/256
https://github.com/liteflow-labs/starter-kit/pull/265
https://github.com/liteflow-labs/starter-kit/pull/267
https://github.com/liteflow-labs/starter-kit/pull/268
https://github.com/liteflow-labs/starter-kit/pull/269
https://github.com/liteflow-labs/starter-kit/pull/270
https://github.com/liteflow-labs/starter-kit/pull/271
https://github.com/liteflow-labs/starter-kit/pull/272
https://github.com/liteflow-labs/starter-kit/pull/274

@ismailToyran could you have a quick review, you can check all the PRs independently

@EmmanuelDrouin please feel free to have a look UI-wise if some skeleton need to be improved and create issues if that's the case


Display Top 5 Recommendations

takes the user input and passes it in URL to Java Servlet
links the algorithm to the datastore recommendations (rather than hardcoded)
returns a list of recommendations
outputs these by creating HTML elements

UI:

(for reference, the markers can be seen here:)

note: I noticed an error in the logic - when the user filled out the preference form, clicked generate recommendations, and then did it again, the HTML was simply being added to the bottom of the previous "top 5 recommendations". To fix this, I simply rearranged some of the logic.

Updated UI:

This shows only 4 recommendations, as there were only 4 recommendations with category as "restaurants" in the datastore.


Could not create SSL/TLS secure channel

Hi. I've been looking at any number of solutions to this exception, but am still getting it. The error occurs on the GetRequestStream() call in UPSProvider.cs around line 205. Any help is greatly appreciated!

Typical, spend hours looking, post a question and then comes the progress. The SSL/TLS error seems to only happen in Debug. I tried it with a Release configuration and it worked, which tells me there isn't anything fundamentally wrong with certs, or settings. This was just running the sample app, so I'm going to proceed with putting the code in my actual web app and see how it goes. Sorry for posting 5 minutes too soon :-|


difference-of-squares: Rewrite tests to use hspec with fail-fast.

Rewrite tests to use hspec.
Update tests to match x-common
Rewrite tests for an Integral implementation.
Add Integral type signatures to stub solution.

Related to #211.

:+1:


Added non-blocking radio receive

I've added a simple non-blocking receive. The method returns a struct that takes care of checking status and calling cancel_recv(). Now it calls cancel_recv() for every drop. I can add some state to avoid this if needed.
I didn't put much thought into the names, so please feel free to bikeshed.

Thanks much for the contribution!
Sorry about the CI failure, which is new: PR #486 should fix it shortly, after which you'll need to rebase.
Should this be adapted to support the nb crate infrastructure?

If is_done() is called after having received an Ok() or a CRC error, you would get a WouldBlock for ever:
let recv = radio.recv_non_blocking(&mut packet);
recv.is_done(); // -> WouldBlock
recv.is_done(); // -> Ok(crc)
while recv.is_done() == Err(nb::Error::WouldBlock) {} // Would never leave loop

This is not great. I could add some internal state and another error to indicate that we already have received:
enum RecvError {
    CrcFailure(u16),
    AlreadyReceived,
}

Or, maybe subsequent calls to is_done() should just repeatedly return Ok(crc) or Err(nb::Error::Other(crc)).

Is this with nb or without? I've lost track.

This applies to both, with and without nb.
I'm not sure if it matters that much anyway, I was just thinking out loud. Futures suffer from the same issue, and the documentation makes it clear that futures that have returned Ready should not be polled.

I refactored recv() and recv_timeout() to use recv_non_blocking() under the hood. I removed Event::End as it is no longer used. The enum only has one member now, so not sure if it's needed anymore.


MyHeritage and phased kits

Hello,
thank you very much for this tool.
I have available my kit (MyHeritage), brother (Ancestry) and mother (FTDNA). I tried reconstructing missing father's kit using Borland tool. There's no problem with uploading to GEDmatch but I noticed a few problems with MyHeritage.

It is not possible to upload reconstructed father's kit to MH at all. Even when converted using this tool, it ends up with "Invalid kit" error after several hours of processing.
Uploading brother's phased paternal kit ends up with "Invalid kit" error.
I successfully upload my phased paternal kit. But the results are horrible. There are only 171 matches on this phased kit (original kit has 3841 matches) Lots of valid matches is missing. Lots of matches are incorrectly 20-30 cM smaller. I believe it is not problem with phased kit itself but buggy MyHeritage matching algorithm.

Hi BigMuscle85,
Nice to see someone actually uses my code :-)
In my use case I had a DNA kit from my father, and a DNA kit from my mother, as well as a DNA kit of myself.
I don't know how compatible / complete utilizing a reconstructed kit would be.
171 matches does sound awfully low. My ~6000 matches will translate to 1000 matches on my paternal kit and 1000 matches on my maternal kit.
It is quite normal that a significant number of matches drop off, however. As I understood a fair amount of MH matches are false positives.
To see why your result is different I would like to inspect your raw DNA files. Are you able to send them to me via e-mail?
dennis < at > drvink.com


Add wallet json-schema, validate in one test.

A replacement for the would-be protobuf pull request. Depends on this PR: https://github.com/lbryio/types/pull/48/files
The idea is to add the wallet file format to the list of our known formats. This PR will confirm that Wallet.to_dict generates a wallet that conforms to the spec. This way we'll at least have some assurance that the spec is up to date.
I sort of made a guess as to what directory wallet.json would be best to go into.
BTW I added this to Makefile to try to match the protoc stuff, but running make doesn't seem to generate all of the python files that are there. There ends up being files removed from the repository.

Coverage remained the same at 71.727% when pulling 5a8f870bfe2571817505424865895c52cefbb989 on orblivion:json-schema into c3e524cb8b9ce3290672bdf7ee219af1459d5a25 on lbryio:master.


Random Password Generator

Random Password Generator & Keeper
Description :
A website to generate a Random Password and to keep that password with encrypting that using a hashing algorithum.
Like a Combination of two popular website [LastPass](https://www.lastpass.com/) and [PasswordsGenerator](https://passwordsgenerator.net/)

Technologies :

Frontend :

Html, Css, Sass, Vanilla JS, React JS etc.

Backend :

Node JS, Python Flask etc.

Database :

MongoDB

Version Control :

git & github

N.B: bold text means optional like Sass and React JS

Pros/Uses :

To generate a strong & safe password.
To keep that password on the servers with encrypting that password.
To show this project on our Internship.
To prepare Open Source for GSOC.
Best way to learn Version Control.

Environment Setup :

Huroko for online
docker for offline

Random Password Generator & Keeper
Description :
A website to generate a Random Password and to keep that password with encrypting that using a hashing algorithum.
Like a Combination of two popular website [LastPass](https://www.lastpass.com/) and [PasswordsGenerator](https://passwordsgenerator.net/)

Technologies :

Frontend :

Html, Css, Sass, Vanilla JS, React JS etc.

Backend :

Node JS, Python Flask etc.

Database :

MongoDB

Version Control :

git & github

N.B: bold text means optional like Sass and React JS

Pros/Uses :

To generate a strong & safe password.
To keep that password on the servers with encrypting that password.
To show this project on our Internship.
To prepare Open Source for GSOC.
Best way to learn Version Control.

Environment Setup :

Huroko for online
docker for offline

http://passwordsgenerators.net/ working Link


Fix wording in file_source.rs

Fix "For complex samples, the real component must come before the complex imaginary component."

Thanks!


vscode-extension-samples/lsp-sample does not compile

Steps to reproduce:

git clone https://github.com/microsoft/vscode-extension-samples.git
cd vscode-extension-samples/lsp-sample
npm install
npm run compile

Expect Result:
No errors
Program compiles to JavaScript
Actual Result:
Numerous type errors related to jQuery such as:
../../../../node_modules/@types/jquery/JQueryStatic.d.ts:101:102 - error TS2304: Cannot find name 'Document'.

101     <TElement extends HTMLElement = HTMLElement>(html: JQuery.htmlString, ownerDocument_attributes?: Document | JQuery.PlainObject): JQuery<TElement>;

My environment:
Microsoft Windows [Version 6.3.9600]
Node v10.17.0
Note
Running npm ci and then npm run compile produces the same result

A workaround is to manually run tsc in the server and client directories. 185 type errors are reported in the server directory (related to jQuery) but it nevertheless builds.
errors.txt

Looks like it was caused by the existence of a node_modules directory in my user directory and removing it solved this issue. It would appear that tsc was referring to that node_modules directory despite it not be related to the project being compiled.


frontend improvement

Thank you for the pull request, but you shouldn't change the name from "Gitter" to "Twitter". It's a Twitter clone but the name of the project is "Gitter". I cannot merge the PR unless you keep the proper name.


Extend the typography

Hi,
We need to implement some changes in the typography, and therefore we have made our type of control. We use your font-family, font-variant и download font, but we need to change the file kirki\modules\webfonts\class-kirki-fonts-google.php so that in the generate_google_font ($ args) function the condition also should work for our type of control.
At the moment this condition looks like:
if ( isset( $args['type'] ) && 'kirki-typography' === $args['type'] ) {
Looking through the developments of your new version, we have investigated that there is an opportunity to change it that way:
if ( ( isset( $args['type'] ) && 'kirki-typography' === $args['type'] ) || ( isset( $args['choices'] ) && isset( $args['choices']['parent_type'] ) && 'kirki-typography' === $args['choices']['parent_type'] ) ) {
Or to make a filter:
$field_types = apply_filters( 'kirki_typography_field_types', array( 'kirki-typography' ) ); if ( isset( $args['type'] ) && in_array( $args['type'], $field_types, true ) ) {
Could you please add one of this variants to your next update for version 3?
Thank you!

Hello there @JuliaArr
Could you please elaborate? Where would that filter go?

Thanks for your reply!
This filter should be added instead of the condition if ( isset( $args['type'] ) && 'kirki-typography' === $args['type'] ) { in file kirki\modules\webfonts\class-kirki-fonts-google.php
The matter is that at the moment this condition works only with 'kirki-typography' field type. And we need to make it work with custom field types.
Thank you!

Any update for this issue?
Thanks in advance

@JuliaArr a new kirki_generate_google_font filter in the dev branch (v4.0-dev)


Detach completed instances, #617

In order to support simple recurrence models and also keep long running tasks small we detach completed instances at the beginning of a series into separate task instances.

Looks like a couple of comments are outdated. They have to be fixed before merging this.


How would I add a content script?

I have a need to manipulate the DOM in my extension and need a content script, but not sure how that fits in this structure

@Dottenpixel  I had facing same issue as you.
But Solution is here.
add
content.pug.
content.js
and Reference this  .js file in to .pug file same as popup.js .
Also don't forgot to mention content.js file in to menifest.dev.js -> entry point.
If you are not mention right path then you will not get the .html file.
All you need to just make content.js same way as popup.js.

We used chrome.tabs.executeScript instead of defined content script because we can't define the content script in development mode. See also: chrome/extension/background/inject.js.


Firebase signs in with old credentials from Apple Sign

[REQUIRED] Step 1: Describe your environment

Xcode version: 12.0.1
Firebase SDK version: 6.26.0
Firebase Component: Auth
Component version: 6.5.3
Installation method: CocoaPods

[REQUIRED] Step 2: Describe the problem

First I sign in using Apple Sign In with hide my email option, an account is created in Firebase using an anonymized email address (which I call A) and everything goes fine.

Then I sign out and close the app and remove the Apple Sign In from:
Settings ->  Account [Apple ID, iCloud, iTunes & App Store] -> Password & Security -> Apps Using Apple ID -> [Select the App] -> Stop Using Apple ID

Open the app again and do Apple Sign In, I am greeted with a fresh sign in process, This time I choose "Don't hide my email option". Therefore apple provider Firebase a new Token with my real email which I call R.

Apple creates the correct credentials, including the correct email address (R) and first name last name. Resulting JWT token  includes the correct email (R) as well. However, firebase still signs in to the first user (with private email address A)
My expectation is for Firebase to create a new account with this new email address (R).
Firebase, must be caching the credentials or the tokens somewhere on the device (since app uninstall/reinstall does not change anything). Problem is the cache is most probably keyed only by providerId so that it does not notice the difference between the first and second credentials.
Note that, I can still sign in with Google or plain Email address/Password between these sign ins and everything works fine, but that does not change anything with regards to the mentioned problem.
If I remove the first account (with private email A) from Firebase dashboard, then repeat the sign in, Firebase is now forced to recreate the account and creates the account with the correct email address (R)...

On more digging, this looks like the intended behavior, since both tokens share the same subject.
I am closing the issue myself.


Fix ImportC square example

See<EMAIL_ADDRESS>

oh don't make the function name ugly

I thought sqr would be a nice short name, it's how I call it in my code. It's sometimes used for square root though, so for clarity I'll take up your suggestion.


[v4] SetInternal component size regression?

Hello. I have not delved deep into the source code for this project, however, I noticed what appears to be a regression in the 4.0.0 release. Prior to 4.0.0, reference type components were (what I assume to be) correctly sized as sizeof(IntPtr) when calling SetInternal in Entity.cs.
        // Old (working)
        private ref Entity SetInternal<T>(ulong id, ref T component)
        {
            Ecs.Assert(Type<T>.GetSize() != 0,
                "Zero-sized types can't be used as components. Use .Add() to add them as tags instead.");

            bool isRef = RuntimeHelpers.IsReferenceOrContainsReferences<T>();
            int size = isRef ? sizeof(IntPtr) : sizeof(T); // Here

            fixed (void* data = &component)
            {
                IntPtr ptr = default;
                ecs_set_id(World, Id, id, (IntPtr)size, isRef ? Managed.AllocGcHandle(&ptr, ref component) : data);
                return ref this;
            }
        }

After 4.0.0, they are incorrectly sized as (IntPtr)sizeof(T). This seems to cause issues with struct components that contain reference types inside them. In my case, I was beginning a defer, then calling entity.Set(someValue) where someValue was a struct containing a class reference and a bool. It resulted in an internal exception from flecs due to a type size mismatch.
            // Current
            if (RuntimeHelpers.IsReferenceOrContainsReferences<T>())
            {
                Managed.AllocGcHandle(component, out GCHandle handle);
                ecs_set_id(World, Id, id, (IntPtr)sizeof(T), &handle);
            }
            else
            {
                ecs_set_id(World, Id, id, (IntPtr)sizeof(T), component);
            }

In my project, changing the current code to this fixed any issues I was having:
            // Current (revised)
            if (RuntimeHelpers.IsReferenceOrContainsReferences<T>())
            {
                Managed.AllocGcHandle(component, out GCHandle handle);
                ecs_set_id(World, Id, id, (IntPtr)sizeof(IntPtr), &handle); // Simply changed sizeof(T) to sizeof(IntPtr)
            }
            else
            {
                ecs_set_id(World, Id, id, (IntPtr)sizeof(T), component);
            }

Yep you're right, it should be sizeof(IntPtr). This will be fixed in the next NuGet release.