Blocked event loop canary

What is wrong?
This is sort of a bug report and an architecture idea.
First the bug:   Block imports are reported on a 5 second interval and when I watch the elapsed part of the log message it regularly is something like 5.1 seconds and quite often you'll see variance up to even a few extra seconds.  This seems to indicate something is either blocking the event loop or the event loop is simply being overloaded.  I don't know how to tell the difference.
    INFO  2020-07-25 07:56:39,208    BodyChainGapSyncer  blks=1     txs=151    bps=0    tps=1     elapsed=5.0  head=#10528187 feee..eab8  age=8h59m31s
    INFO  2020-07-25 08:11:10,446   FastChainBodySyncer  blks=0     txs=0      bps=0    tps=0     elapsed=5.0  head=#10529063 a24a..8af6  age=6h2m3s
    INFO  2020-07-25 08:11:18,144    BodyChainGapSyncer  blks=6     txs=1100   bps=0    tps=8     elapsed=6.3  head=#10529000 2637..a0af  age=6h14m53s
    INFO  2020-07-25 08:11:23,145    BodyChainGapSyncer  blks=0     txs=0      bps=0    tps=8     elapsed=5.0  head=#10529000 2637..a0af  age=6h14m58s
    INFO  2020-07-25 08:11:29,744    BodyChainGapSyncer  blks=0     txs=0      bps=0    tps=7     elapsed=6.6  head=#10529000 2637..a0af  age=6h15m4s
    INFO  2020-07-25 08:11:35,771    BodyChainGapSyncer  blks=11    txs=2430   bps=0    tps=27    elapsed=6.0  head=#10529011 a8be..397e  age=6h12m29s
    INFO  2020-07-25 08:11:41,881    BodyChainGapSyncer  blks=5     txs=912    bps=0    tps=33    elapsed=6.1  head=#10529016 f5b2..7bdd  age=6h11m47s
    INFO  2020-07-25 08:11:46,881    BodyChainGapSyncer  blks=7     txs=1077   bps=0    tps=42    elapsed=5.0  head=#10529023 4e72..9fef  age=6h10m54s
    INFO  2020-07-25 08:11:52,662    BodyChainGapSyncer  blks=5     txs=722    bps=0    tps=46    elapsed=5.8  head=#10529028 3c82..a747  age=6h10m5s
    INFO  2020-07-25 08:11:59,229    BodyChainGapSyncer  blks=8     txs=1512   bps=0    tps=56    elapsed=6.6  head=#10529036 1fca..6a17  age=6h8m17s
    INFO  2020-07-25 08:12:04,397    BodyChainGapSyncer  blks=10    txs=1495   bps=0    tps=67    elapsed=5.2  head=#10529046 61ce..2f30  age=6h6m36s

Second: We should be able to detect this by simply running a loop with a sleep and measuring expected time vs actual time.
How can it be fixed
In whatever process we have that might end up with this problem, we could implement a simple service that does the following.
async def monitor_event_loop_performance():
    while True:
        timer = Timer()
        await asyncio.sleep(10)
        elapsed = timer.elapsed()
        if elapsed - 10 > 0.1:
            logger.warning("Event loop blocked or overloaded:  variance=%f", elapsed - 10)

Logging output might also benefit from including process id and maybe even a text identifier that we explicitly provide.  This would give us a "canary in the coal mine" to help us identify how often this is happening.  If we can actually address whatever is currently causing this problem, this type of architecture should give us some early warnings when/if we introduce new code which causes the loop to be blocked.
Open Question
Is there any way for us to differentiate between the loop being blocked and the loop being overloaded....  That would be nice.

Is there any way for us to differentiate between the loop being blocked and the loop being overloaded.

I couldn't find any, but if our monitoring task also iterated over all running tasks and reported the top X ones ordered by cumulative cpu time (provided that's even possible), it might help identify those issues

Reporting total number of tasks is probably just nice information to have as we might be able to at least draw some correlation there.

Some quick experimentation showed that my event loop would become congested at around 1300-1400 tasks.
import asyncio
import time

global num_tasks
num_tasks = 0
global blocked
blocked = False

async def canary():
    global blocked
    while True:
        start_at = time.monotonic()
        await asyncio.sleep(1)
        end_at = time.monotonic()
        elapsed = end_at - start_at
        if elapsed > 1.01:
            print(f'BLOCKED({num_tasks}): {elapsed}')
            blocked = True
        else:
            print(f"OK({num_tasks}): {elapsed}")

async def clog():
    while True:
        await asyncio.sleep(0)

async def do_it():
    global num_tasks
    tasks = []
    tasks.append(asyncio.ensure_future(canary()))
    while not blocked:
        num_tasks = len(tasks)
        tasks.append(asyncio.ensure_future(clog()))
        await asyncio.sleep(0.001)
    await asyncio.sleep(5)

if __name__ == '__main__':
    loop = asyncio.get_event_loop()
    loop.run_until_complete(do_it())

we could actually measure this value and emit warnings when the number of tasks exceeds our measured value.  Maybe enable this with a CLI flag so that it isn't always on but it's easy for us to turn it on when we run locally.

Last thing.  This might be worth running in our component processes as well but that gets more complex.

Just found that uvloop has a bunch of debug properties (https://github.com/MagicStack/uvloop/blob/6ef69a797db0afe781b5ddcd74691767f8261a53/uvloop/loop.pxd#L96-L130) but those are only available on a debug build as they probably add significant overhead. Might be worth looking into that, although it wouldn't be exactly trivial to enable and all the attributes are private so our reporter may stop working even with minor version updates

Also, I just realized that if we use a monitoring task that keeps track of how often it gets scheduled it will only issue warnings if the loop is blocked at the time it was supposed to be scheduled, so if we have one rogue task that runs frequently and on every iteration blocks for less than the monitoring task's report interval, we'd never get any warnings.  One (hackish) alternative would be to use a custom event loop that inherits from python's default one and overwrites the _run_once() method, to keep track of how frequently the event loop is being called and possibly other things

Also, I just realized that if we use a monitoring task that keeps track of how often it gets scheduled it will only issue warnings if the loop is blocked at the time it was supposed to be scheduled

I agree here.  We would only see the warning if the blocking task is running at the time that the canary should wake up.

so if we have one rogue task that runs frequently and on every iteration blocks for less than the monitoring task's report interval, we'd never get any warnings.

I don't think this is true.  Suppose that our monitoring task is scheduled to wake up in 0.1 seconds.  If our blocking task takes 0.3 seconds to run, and the loop runs it within 0.2 seconds before the canary task is scheduled to wake up, the canary task will end up being woken up late and would report the deviation.  It's not ideal since we could be in a position where this only occurs occasionally, but I still think this is the behavior we want, especially since the larger the blocking time, the larger the chance that it gets detected by the canary, and the smaller the block, the less important it is to worry about it.

Last thing. This might be worth running in our component processes as well but that gets more complex.

Actually, the event loop running in the main process doesn't do much heavy lifting, right? Isn't it mainly taking care of collecting logs and state updates from the component processes? If that's the case I think what we really need is to run a separate monitoring task on the event loop of each component process

Isn't it mainly taking care of collecting logs and state updates from the component processes?

Yes, that's the correct. Specifically, the logs from the syncing progress that sparked the discussion originate from the syncer / peerpool component. So, I guess it really are the component processes that need to be monitored.

Some quick experimentation showed that my event loop would become congested at around 1300-1400 tasks. This number is probably bogus because I'm syncing both turbogeth and geth on my machine and my CPU load is high.

Mine was the same, on a completely idle machine. I guess it is what a single core (which is all python gets, right?) can handle


Added a new testimonial component

Feature/ #8
Added a new testimonial component and a new variant in button -"blackwhite"
Component picture

It is pending review.
You'll receive an update in 30-45 mins.

Hi @Shivampatil22

Can you please remove the animation?
Can you remove the shadow?
Please use the default button available in src\components\ui\button.tsx

Updated Testimonial Component ****


Suggestion: Add RenovateBot or dependabot

Just a suggestion to add RenovateBot to track new versions of the codegen more easily and keep up to date with it.
If you'll add tests we can even think about connecting this repo to the codegen's Connected Build

Thanks for the suggestion @Urigo, I'll look into that. I didn't know about RenovateBot, but it looks interesting


Added awesome graph embedding papers list.

[https://github.com/benedekrozemberczki/awesome-graph-embedding]
[ The repository is a curated list of papers from top tier machine learning conferences on graph embeddings (graph representation learning). It covers graph kernel functions, factorization, deep learning and statistical fingerprints. ]
PRs:
1431
1452
By submitting this pull request I confirm I've read and complied with the below requirements 🖖
Please read it multiple times. I spent a lot of time on these guidelines and most people miss a lot.
Requirements for your pull request

You have to review at least 2 other open pull requests. Try to prioritize unreviewed PRs, but you can also add more comments to reviewed PRs. Go through the below list when reviewing. This requirement is meant to help make the Awesome project self-sustaining. Comment here which PRs you reviewed. You're expected to put a good effort into this and to be thorough. Look at previous PR reviews for inspiration.
I have read and understood the instructions for creating a list.
This pull request has a descriptive title in the format Add Name of List (Example: Add Swift), not Update readme.md or Add awesome list.
The entry in the Awesome list should:

Include a short description about the project/theme of the list. It should not describe the list itself.Example: - [Fish](…) - User-friendly shell., not - [Fish](…) - Resources for Fish..
Be added at the bottom of the appropriate category.

The list I'm submitting complies with the below requirements.

Requirements for your Awesome list

Has been around for at least 30 days.That means 30 days from either the first real commit or when it was open-sourced. Whatever is most recent.
It's the result of hard work and the best I could possibly produce.
Non-generated Markdown file in a GitHub repo.
Includes a succinct description of the project/theme at the top of the readme. (Example)
The repo should have awesome-list & awesome as GitHub topics. I encourage you to add more relevant topics.
Not a duplicate.
Only has awesome items. Awesome lists are curations of the best, not everything.
Includes a project logo/illustration whenever possible.

Either centered, fullwidth, or placed at the top-right of the readme. (Example)
The image should link to the project website or any relevant website.
The image should be high-DPI. Set it to maximum half the width of the original image.

Entries have a description, unless the title is descriptive enough by itself. It rarely is though.
Includes the Awesome badge.

Should be placed on the right side of the readme heading.

Can be placed centered if the list has a centered graphics header.

Should link back to this list.

Has a Table of Contents section.

Should be named Contents, not Table of Contents.
Should be the first section in the list.
Should only have one level of nested lists, preferably none.

Has an appropriate license.

That means something like CC0, not a code licence like MIT, BSD, Apache, etc.
WTFPL and Unlicense are not acceptable licenses.
If you use a license badge, it should be SVG, not PNG.

Has contribution guidelines.

The file should be named contributing.md. Casing is up to you.

Has consistent formatting and proper spelling/grammar.

The link and description are separated by a dash. Example: - [AVA](…) - JavaScript test runner.
The description starts with an uppercase character and ends with a period.
Consistent and correct naming. For example, Node.js, not NodeJS or node.js.

Doesn't include a Travis badge.You can still use Travis for list linting, but the badge has no value in the readme.

Go to the top and read it again.

This awesome list is missing contribution guidelines. It uses uncommon, but consistent formatting.

So?

I'm not too sure whether the formatting is an issue, but you should definitely add the contribution guidelines.

Thanks for making an Awesome list! 🙌
It looks like you didn't read the guidelines closely enough. I noticed multiple things that are not followed. Try going through the list point for point to ensure you follow it. I spent a lot of time creating the guidelines so I wouldn't have to comment on common mistakes, and rather spend my time improving Awesome.

@sindresorhus
I added a detailed contributing.md.
I have a proper license.
The pull requests on works by others:
1431
1452

Would be useful if the items to refer to a paper had a short summary of what it's about.

I don't see this being done.

I think it would look better if the header image was right-aligned. You can use the syntax

.

This pull request has a descriptive title in the format Add Name of List

Would be useful if the items to refer to a paper had a short summary of what it's about.

I don't see this being done.

I do not think that every paper should have a description. Sometimes the core idea/contribution of graph kernel papers cannot be explained in a short summary.

I do not think that every paper should have a description. Sometimes the core idea/contribution of graph kernel papers cannot be explained in a short summary.

Many of the papers come with an abstract. You can create a short summary from that.

Bump

? :)

Do you intend to address my feedback or should I close this PR?

Dear @sindresorhus ,

With all respect I honestly believe that adding/summarizing the abstract would break the current format of the repository. The paper titles in themselves should be enough informative for someone to get an understanding what the paper is about. Sometimes the author names are enough informative.

At the same time I believe that it is sufficiently high quality to be included.

If you decide not to merge I understand that.

I also created repositories that cover other areas of ML:
https://github.com/benedekrozemberczki/awesome-decision-tree-papers
https://github.com/benedekrozemberczki/awesome-community-detection
https://github.com/benedekrozemberczki/awesome-gradient-boosting-papers
https://github.com/benedekrozemberczki/awesome-fraud-detection-papers
Regards,
Benedek

Agree to disagree, I guess.


Buildalyzer - AnalyzerManger ProjectFile-RequiresNetFramework property throws null exception for .vbproj

Buildalyzer - AnalyzerManger ProjectFile-RequiresNetFramework property throws null exception for .vbproj
But it works for .csproj the exception occurs only for .vbproj.

AnalyzerManager analyzerManager = new AnalyzerManager("C:\Users\...\source\repos\WebSite1\WebSite1.sln");
ProjectAnalyzer projectAnalyzer = analyzerManager.GetProject("D:\vb2\ConsoleApp1\ConsoleApp1.vbproj");
var analyzerResults = projectAnalyzer.Build();

For .csproj it works fine and gets value "RequiresNetFramework" property as false and its building the project as expected. But for .vbproj

projectAnalyzer.Build();

throws null exception while building the project.

Stack Trace:

at Buildalyzer.Construction.ProjectFile.<>c__DisplayClass14_0.<get_RequiresNetFramework>b__2(String i)
at System.Linq.Enumerable.Any[TSource](IEnumerable1 source, Func2 predicate)
at System.Linq.Enumerable.Any[TSource](IEnumerable1 source, Func2 predicate)
at Buildalyzer.Construction.ProjectFile.get_RequiresNetFramework()
at Buildalyzer.Environment.EnvironmentFactory.GetBuildEnvironment(String targetFramework, EnvironmentOptions options)
at Buildalyzer.ProjectAnalyzer.Build(String targetFramework)

Note: I am using Buildalyzer 2.3.0

Able to fix the null exception by the below code change.
 public bool RequiresNetFramework =>
            _projectElement.GetDescendants(ProjectFileNames.Import).Any(x => ImportsThatRequireNetFramework.Any(i => x.GetAttributeValue(ProjectFileNames.Project)**?.**EndsWith(i, StringComparison.OrdinalIgnoreCase) **== true**))
            || _projectElement.GetDescendants(ProjectFileNames.LanguageTargets).Any(x => ImportsThatRequireNetFramework.Any(i => x.Value.EndsWith(i, StringComparison.OrdinalIgnoreCase)))
            || ToolsVersion != null;

I think this is resolved in #207 courtesy of @cslong - at a minimum an AnalyzerResult is now created for Visual Basic projects (and tested via unit test). I'll close this issue since it's working as far as I can tell, but please open a new one if it looks like there are still problems with VB support (though my ability to help will be limited since I don't know the first thing about VB or VB projects).


Golang Houdini microservice

This PR replaces PHP-based Houdini with a Go equivalent.

Right now we don't seem to need this, but keeping the PR around for a little longer.


Script for concurrently running both react and node server

Currently the project has two development servers front-end react server and the back-end node server, and both need to be started individually and also by being on the respective directories. Since concurrently is already been installed as dev  dependency therefore a single script can be made to run both the development servers with the help of concurrently.
Similarly the  package installation for react side can also be done without changing the directory with the help of a script on server package.json.
Please let me know If this can be done @genialkartik, I would like to contribute to this repository.

Thanks, @Nikhil-dev13, to dive down into details of the project.
Well, currently the project is in production-ready state. That means you would only need to run build script in the client (frontend) directly and then just by running 'npm start' the app will go live.
I implemented Package 'concurrently' in dev mode but didn't update the same on GitHub (but may find it in the commit history of the project) in the 'macfolio3.0' branch.
Conclusion: current code in master branch is in production-ready mode (state). will update the code of dev mode in branch 'macfolio3.0'.

I'll create a new issue and will update Readme.md file with proper guidance and documentation. So that You'll and other also will find it easy to contribute to the project.
I'd be pleased with your contribution. 👍🏼

Ok, Thanks for your quick response.
I would love to contribute to this repository.
I am closing this issue.
Thank you once again


Crash on server

Minecraft 1.20.1
java.lang.RuntimeException: Could not execute entrypoint stage 'main' due to errors, provided by 'edenring'!
at net.fabricmc.loader.impl.FabricLoaderImpl.lambda$invokeEntrypoints$2(FabricLoaderImpl.java:388)
at net.fabricmc.loader.impl.util.ExceptionUtil.gatherExceptions(ExceptionUtil.java:33)
at net.fabricmc.loader.impl.FabricLoaderImpl.invokeEntrypoints(FabricLoaderImpl.java:386)
at net.fabricmc.loader.impl.game.minecraft.Hooks.startServer(Hooks.java:63)
at net.minecraft.server.Main.main(Main.java:112)
at net.fabricmc.loader.impl.game.minecraft.MinecraftGameProvider.launch(MinecraftGameProvider.java:470)
at net.fabricmc.loader.impl.launch.knot.Knot.launch(Knot.java:74)
at net.fabricmc.loader.impl.launch.knot.KnotServer.main(KnotServer.java:23)
at net.fabricmc.loader.impl.launch.server.FabricServerLauncher.main(FabricServerLauncher.java:69)
Caused by: java.lang.NoSuchFieldError: CLIENT_CONFIG
at paulevs.edenring.config.Configs.(Configs.java:20)
at paulevs.edenring.registries.EdenItems.(EdenItems.java:13)
at paulevs.edenring.registries.EdenBlocks.(EdenBlocks.java:48)
at paulevs.edenring.EdenRing.onInitialize(EdenRing.java:49)
at net.fabricmc.loader.impl.FabricLoaderImpl.invokeEntrypoints(FabricLoaderImpl.java:384)
... 6 more

The crash occurs on both Eden Ring Reforked 0.6.7-1.20.4 and Eden Ring Reforked - 0.6.5-1.20.1.
Eden Ring Reforked runs perfectly fine on servers with Fabric Loader 0.14.25. But it fails to run on servers with Fabric Loader 0.15.0 and above.
Fabric Loader 0.15.0 fixed environment annotation stripping for fields. It must have to do with the @Environment(EnvType.CLIENT) annotations here.
I think the fix is to remove any @Environment(EnvType.CLIENT), restoring the behaviour where Fabric Loader ignored it. I think the right way to do environment-only mixins is to have two completely separate mixins: one for common, one for client.
See https://github.com/MoriyaShiine/extra-origins/commit/e6d4cd4ab32adc7114e420926e3b358ad65acaf7 for example
If you edit this yourself, you only need to modify the server jar, not the client.

if this issue presists feel free to reopen this issue


Cache digit width for tabular numbers

Fixes #470.
The max digit width is now calculated when the font is loaded, even if that font is never used for tabular numbers.

This looks great. Thank you :)

Seems to have been a decent performance improvement:
Scene Management (Default)
Before:
                        time:   [7.3866 us 7.4272 us 7.4747 us]
After:
                        time:   [4.3817 us 4.4423 us 4.4991 us]

Scene Management (Subsplits Layout)
Before:
                        time:   [26.779 us 26.782 us 26.786 us]
After:
                        time:   [20.934 us 21.212 us 21.506 us]


refactor/improve monitor mode selection

Describe your PR, what does it fix/add?
aims to

always ensure a mode is selected, if requested fails still set one, i might add an error notification
clarify mode selection code
consistent logs

also fixes format selection trying to use INVALID
Is there anything you want to mention? (unchecked code, possible bugs, found problems, breaking compatibility, etc.)
Is it ready for merging, or does it need work?
yes probably
maybe some other people should test too, idk who tho

this is ready for review
just need figure out how to test custom modes and do that

rebased + fixed
tested highrr, highres, requested mode, and custom mode, works for me :tm:
also fallbacks are working

ping me after xmas

@vaxry its after xmas :)


Add to install notes: pyocd & libusb fix for M1 Mac

https://github.com/pyusb/pyusb/issues/355#issuecomment-974726078

Betetr way is this one:


New deviation flag for include the IP config enabled explicitly when …

…create vlan sub interface
With this new deviation flag, the IP config enabled element could be turned on of off depends on vendor and/or platform whatever applicable.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 57.081%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1181

Relevant Lines:
2069

💛  - Coveralls

hey @gforeal
Could you elaborate a bit on how is this compared to the current use of deviations deviations.InterfaceEnabled & deviations.IPv4MissingEnabled?
E.g. changes made in #1175 for RT-3.2 case.


"bring line X past line Y" copies the entire region between X and Y

This is with v0.25.0 of the extension , and talon-side cursorless commit 5f295aaa4150309466186c6f41348043905d7bbe

That is by design. If you just want the two lines should say bring line X and Y
@pokey If you have migrated all the issues to the extension side you should probably disable them here

I believe "bring line X to Y" is what you're after here 😊

bring line X to after Y
Your very welcome :)

Thanks, will try.
BTW, I believe "after" is missing from the docs, it occurs once in an
example, but is not actually explained like "pre", "past", etc. are.
On Thu, Jan 13, 2022, 9:17 AM Andreas Arvidsson @.***>
wrote:

bring line X to after Y
Your very welcome :)
—
Reply to this email directly, view it on GitHub
https://github.com/pokey/cursorless-talon/issues/147#issuecomment-1012177335,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AABV6JYCHRRTQ6AQX7P2TK3UV3NHLANCNFSM5L2D3ZCA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID:
@.***>

If you look in the cheat sheet ie cursorless help you find it under positions.


Deprecation warning when including from Node

Hey there, thanks for providing this package. I'm just getting going trying to use it but the first time I did I get this warning when running the app:
(node:3555) [DEP0128] DeprecationWarning: Invalid 'main' field in '/app/node_modules/simple-scaffold/package.json' of 'dist/scaffold.js'. Please either fix that or report it to the module author
(Use `node --trace-deprecation ...` to show where the warning was created)

Using:
node v16.0.0
Here is the code that's using it:
import SimpleScaffold from 'simple-scaffold';

async function main  {
      const scaffold = new SimpleScaffold({
        name: projectName,
        templates: [join(__dirname, "commands", "new", "templates")],
        output: process.cwd(),
        createSubfolder: true,
        locals: {
          description: 'something or other'
        }
      }).run();
}
main();

Found that I can suppress the warnings by adding this at the top of my app but its not ideal:
process.removeAllListeners('warning');

Hi @jonnysamps, sorry it took so long to see this. I will upload a new release shortly that removes this field from package.json. Luckily this is a quick fix.

Release v0.7.5 should fix this issue, please let me know if it it still occuring.

looking good! Thanks!


Unbound: Alias support

Greetings,
I have read the contributing guidelines and searched existing issues. My feature request is related to a problem I have. PfSense has a feature that is extremely convenient for me and it is keeping me from moving to OpnSense. I want to move to OpnSense. I have a docker server with many images behind a proxy that routes based on hostname. When I add a new image, I add a new DNS record for that image. All images have the same IP address. Changing this IP address is tedious with dozens of A records.
I would like the ability to create multiple A records in Unbound from one DNS override entry, tied to multiple dns name aliases. This would allow one IP address change to affect multiple records, giving the flexibility of a CNAME record in configuration.
This would be implemented in the GUI in the same fashion as the Dnsmasq aliases, but would, on the back-end, create multiple A records instead of CNAME records.
This is implemented in the following PfSense files:
GUI to add Overrides and their aliases - services_unbound_host_edit.php
GUI listing DNS overrides and their aliases, also post page - services_unbound.php
Backend service for creating configuration
The aliases are also stored in config.xml. I would post more but I have not dug into how that system works.
A screenshot of how this is represented in the GUI:

Alternatives I have considered: Doing it all with A records, switching to Dnsmasq dns, SSHing into the box and using vi find & replace in the configuration for the one IP address.

Hi @MarkNBroadhead,
Sorry, maybe I misunderstand you. I'm not sure how the feature works and technically I can't derive it from your description / requirement, but I'll take a look as per our discussion via: https://github.com/opnsense/core/issues/1041
Cheers,
Franco

Looking at this further I see two things:
Aliases are a mere convenience feature to not type the IP multiple times.
Aliases are flushed to the /etc/hosts file so that they work regardless of the firewall using its own resolver.
Can you tell me how this doesn't work on OPNsense in your use case. I want to help, but I don't see the technical issue and maybe we can find the correct setup or add something that makes things work correctly in your end. At the moment, I still don't see the need for this.
Thanks,
Franco

Oh wait so all you need is a way to enter multiple host names here?

I'd be happy to walk you through it in more detail if it would help.

Ah, ok, sure. I didn't even know Dnsmasq host overrides supported this... oO

@MarkNBroadhead work in progress:
# opnsense-patch 8143857

What do you think?

@MarkNBroadhead if there's no feedback it'll likely be merged in 19.1.4.

Have tested, this works exactly as I had hoped. Thank you so much!!!

Yay, no problem. I’ll try to squeeze this into the upcoming 19.1.3 then as the impact is pretty low for existing installs.
Thanks for bearing with me. 😊

Sure thing. One question. I'm not sure how installs go. Will I need to revert these files before I upgrade to 19.1.3 or will the upgrade copy over the files?
Dude, no apologies required. I'm shocked you cranked this out so quickly. You rock!

If the code is in 19.1.3 you won't have to do anything other than upgrade. If it's not in 19.1.3 and you upgrade you lose the functionality but not your settings. In the latter case a patch reapply will suffice to bring it back.


Add options for cleaner excelfile

Hi guerinoni,
I needed an excel file with just the strings and translations:

added two command line switches to disable locations/version info. (not sure if this is the way to go)
enabled tests on windows machine
added debug output for some tests

Added some documentation for the flags and how to build & install for windows.
Added to cmake to also install the required dlls to the correct location.


Android build scripts do not check checksums

The android build scripts should check the checksum of the downloaded tar

fixed with f620a31d57


Add mechanisms for selecting RTL sources based on FPGA family and Quartus versions (#345)

The FPGA family tag may be stored in the platform db and extracted with a new script: afu_platform_info.
rtl_src_config automatically defines the environment variables OPAE_PLATFORM_FPGA_FAMILY and QUARTUS_VERSION_MAJOR when it detects a reference to the tags in the list of sources.

Having this in the release will make it far easier to write AFUs with Qsys components that target different FPGA families.


Team 2 HA Review: Missing Critical Assumption/Hazard

@samm82
Name: Team 2 Review
About: Missing Critical Assumption/Hazard
Summary
The critical assumption does not mention whether the current user device has the necessary equipment or supported software versions to properly run the application.
Alternatively, the application could check for this issue and notify the user to update their software or connect the appropriate supporting device.
Deadline
23/10/2023
Estimated Work Hours
1 hour

The app store or google play store will not allow users to install the application if their device does not meet the specification, so this issue does not need to be considered by us, as we defined in our SRS what platforms we are intending to develop for, and what software versions the users will have.


Min value for minReplicas should be 0

The min value for minReplicas is documented as 1. I believe that this is incorrect and should be zero. The default is also zero. It doesn't make sense to have an invalid default value.
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 58fa8bbe-e0eb-4944-bc60-617aec0c0dde
Version Independent ID: eae53948-3193-a451-5963-3e6cd8ee2972
Content: Scaling in Azure Container Apps
Content Source: articles/container-apps/scale-app.md
Service: container-apps
GitHub Login: @craigshoemaker
Microsoft Alias: cshoe

Thanks for the feedback! I have assigned the issue to the content author to investigate further and update the document as appropriate.

This issue has been resolved.

Thanks for pointing out this issue.  I have fixed the article.
#please-close


Check TreeItem for nullptr before accessing.

Fixes #65584
Bit of a tricky one to reproduce but whenever I did, ti = NULL was always the culprit of the access violation.

Thanks!


Add amazonec2 userdata

Description
Introducing support for amazonec2-userdata flag that is not mentioned on driver readme page, but it is there and useful.
For example, it can be used to introduce swap for the runner instance or any other user command.
Migrations required
NO - this is optional flag and defaults to empty string.
Verification
I have verified on my own runners cluster that user script that I've added works (it creates swap in my case).
module "runner" {
  # source  = "npalm/gitlab-runner/aws"
  # version = "5.5.0"

  # ...

  runners_userdata = templatefile("${path.module}/../../templates/swap.tpl", {
    swap_size = "512"
  })
}

But I have no idea how to auto-generate README.md, I think this flag should be documented. (manually edit?)

@npalm See my comment above. Better you check this as well.

Did you do a quick check without using the new variable runners_userdata to make sure that the agent is still able to start?

Ok, it's working. Had a test environment available and deployed your changes. It is working without runners_userdata.


IndexError: list index out of range

Restoring total_amount parser ./models/parsers/amount/best...
Restoring all ./models/invoicenet/total_amount/best...
invoice-template-us-neat-750px.png 0/0
Filename: invoice-template-us-neat-750px.json
Traceback (most recent call last):
File "predict.py", line 92, in
main()
File "predict.py", line 83, in main
labels[field] = predictions[field][idx]
IndexError: list index out of range
Is this an issue because tesseract wasn't able to read the data? I have attached the image file

What's the exact command you ran?

What's the exact command you ran?


OSRM v5.24.0 support

Greetings @quiqua is there a chance this repo will support the latest OSRM 5.24.0? Currently I am seeing
/usr/src/app/src/lib/osrm.js:9
  return new OSRM(opts);
         ^

TypeError: File is incompatible with this version of OSRM: /data/osrm/uk/britain-and-ireland-latest.osrm.icd prepared with OSRM 5.24.0 but this is v5.22.0 (at include/storage/tar.hpp:204)

I'm happy to raise a PR if you could guide me a little.

Will have to be taken care in osrm backend repository

I have not yet build a newer version of OSRM and ran it with this version.
The README of this project is currently a bit off as it assumed different version of OSRM, e.g. this line:
docker pull osrm/osrm-backend:v5.22.0

and this one:
# extract the network and create the osrm file(s)
docker run -it -v $(pwd)/data:/data osrm/osrm-backend:v5.16.1 osrm-extract -p /opt/car.lua /data/osrm/example-areas/example-areas.osm.pbf

should use the same version number.
Yet as long as there exists no docker container provided by OSRM directly for version 5.23, there is nothing I can do really.
The only advice I can give you is to use the same version of OSRM to preprocess the data and then run the routing engine.

Thank you @quiqua makes sense. Works perfectly fine with 5.23.0 now, too - https://www.npmjs.com/package/osrm


typia.llm.schema<T>() function.

Added typia.llm.schema<T>() function which generates type schema of the LLM function callimg by TypeScript type.
Requested by @ryoppippi

LTGM

For people who see this PR.
This is a power-up version of openai functions in typiautil.
When this PR is merged, I'll deprecate the openai modules in typiauitl

Still typia.llm.application() function is left.
It may be completed at this Sunday.


Recommendation: Change "Go to" Link

The link at the bottom of the "Developer productivity" tab is:

Go to the Azure DevOps Projects documentation

Personally, I would demo Azure DevOps Projects to developers new to Azure DevOps but to get them started, this document makes more sense to me:

Start using Azure DevOps

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: f22dc745-f2dd-406b-2a9b-016775c9b7ed
Version Independent ID: c0d94094-2598-acb4-5d9f-71262cc7feb2
Content: Azure innovation guide: Empower adoption - Microsoft Cloud Adoption Framework for Azure
Content Source: docs/innovate/innovation-guide/ci-cd.md
Service: cloud-adoption-framework
Sub-service: innovate
GitHub Login: @absheik
Microsoft Alias: absheik

Great suggestion! Thank you


[Bug][API] save work group

*For better global communication, please give priority to using English description, thx! *
Please review https://dolphinscheduler.apache.org/en-us/docs/development/issue.html when describe an issue.
Describe the bug
save work group failure ,request 404 not found, cannot find /worker-group/save ,its the api is deleted in version of 1.3.2!
To Reproduce
Steps to reproduce the behavior, for example:

Security/work group manage/create worker group
add new worker group
request 404 not found

Expected behavior
save worker group success!
Screenshots

Which version of Dolphin Scheduler:
-[1.3.2-release]
Requirement or improvement

add save worker group api

The 1.3.2 version does not support the creation of worker groups, and the worker group is read from zk. you can refer to issue #3628

Please clear browser cache and refresh this page to check that you should not see the create worker group button.

The 1.3.2 version does not create to worker groups in web page, and the worker group is read from zk. you can refer to issue #3628

tks，i will read doc carefully!

Please close this issue after problem solved.

Please clear browser cache and refresh this page to check that you should not see the create worker group button.

it doesn't work, i still see the create button!you can try it.

The 1.3.2 version does not create to worker groups in web page, and the worker group is read from zk. you can refer to issue #3628

int the quick start of 1.3.2,why suggest user to create worker group in web page, whether it should be removed in next version!

creating worker groups  through web page that mentioned  in  the issue  #3730


What is the difference between JsOS and other "JavaScript OS" implementations?

How JsOS compares to:

https://node-os.com/
https://github.com/charliesome/jsos
?

Is there any important difference between them and JsOS?

Another core, another architecture.
It's just different OSes


Enable SUBUTAI_DESKTOP provisioning option

Basically, if users set SUBUTAI_DESKTOP to true then the VM is set up like a client with everything.

Successfully created VM with the desktop inside:

fixed on master

Wrote a parameter in vagrant-subutai.yml file
Enabled subutai_desktop
and brought a peer up.
Result: Ended up with mate desktop
Fixed


Support journal links

Can you add support for journal links?

Like Visual Active Effects does

Added in v2.2.1


Update retry-service-specific.md

simple typo fix in comment

@TroyWitthoeft : Thanks for your contribution! The author(s) have been notified to review your proposed change.

The typo fix looks good; thanks.


Update Independent Assessments UI on APPLY and Admin

Hi Andrew, as discussed, I'm writing about the IA demarcation as raised at the last Ops Mgmt Meeting.
There is still a requirement for candidates and panel members to identify which of the two nominated IAs is commenting on the candidate’s work as a judge (Judicial Assessor) and/or as a practitioner (Professional Assessor, who can still be a judge).
Context
Historically on JARS, there was no mechanism to separate the different options (i.e. for Salaried roles candidates were salaried judges and so had to nominate 2 x judicial assessors, but for Fee-paid roles candidates were practitioners and so had to nominate 2 x profession, but some were existing fee-paid judges and so had to nominate 1 x judicial and 1 x profession). So get to around that space was made on the form for all four combinations which led to some candidates nominating 4 assessors instead of two. This led to someone making the fairly sensible decision on the Platform to only allow candidates to insert details for two assessors and to get around the possible combinations to label them as Assessor 1 and Assessor 2.
As raised at Ops Mgmt however it transpires we do still need candidates and panel members to confirm if an Assessor is judicial or professional.
Solution
Therefore what would be ideal is for the IA field on the application form to have a drop box or tick box where the candidate can confirm if their nominated IA is commenting on their work in a judicial or professional capacity, and for that info to then follow through into the panel pack.

https://app.zenhub.com/files/201935567/942a1707-30b2-4c1e-b331-8dd7ee8670a6/download


Run unit tests on Windows as part of CI

Github-hosted runners support Windows Server 2019 (https://docs.github.com/en/free-pro-team@latest/actions/reference/specifications-for-github-hosted-runners). I think we should consider running the Antrea Go unit tests on Windows as part of the Github CI to exercise the Windows platform specific code.

This was taken care of in https://github.com/vmware-tanzu/antrea/pull/1505


Message: Resource is not in the state stackUpdateComplete

Before opening, please confirm:

[X] I have installed the latest version of the Amplify CLI (see above), and confirmed that the issue still persists.
[X] I have searched for duplicate or closed issues.
[X] I have read the guide for submitting bug reports.
[X] I have done my best to include a minimal, self-contained set of instructions for consistently reproducing the issue.
[X] I have removed any sensitive information from my code snippets and submission.

How did you install the Amplify CLI?
yarn
If applicable, what version of Node.js are you using?
16.13.1
Amplify CLI Version
9.2.0
What operating system are you using?
Mac arm
Did you make any manual changes to the cloud resources managed by Amplify? Please describe the changes made.
no manual changes
Amplify Categories
api
Amplify Commands
push
Describe the bug
["Index: 4 State: {"deploy":"waitingForDeployment"} Message: Resource is not in the state stackUpdateComplete"]
Expected behavior
Push my updated schema
Reproduction steps
amplify api push --allow-destructive-graphql-schema-updates
--allow-destructive-graphql-schema-updates is intended
GraphQL schema(s)

# Put schemas below this line
type Settings @model @auth(rules: [{allow: public}]) {
  id: ID!
  currencies: AWSJSON
  categories: [String!]
  status: Int!@default(value: "0")
}
type User @model @auth(rules: [{allow: public}]) {
  id: ID!
  sub: String!
  username: String!
  fullname: String
  email: AWSEmail!
  states: [State] @hasMany
  userStateLikes: [UserStateLike] @hasMany
  notifications: Notification @hasOne
  followedByUser: FollowedByUser @hasOne
  regionsHUB: [RegionHUB] @hasMany
  accounts: [Account] @hasMany
  activities: [Activity] @manyToMany(relationName: "UserActivity")
  createdAt: String!
  sortType: String @index(name: "userByDate", queryField: "userByDate", sortKeyFields: ["createdAt"]) @default(value: "User")
}
type FollowedByUser @model @auth(rules: [{allow: public}]) {
  id: ID!
  follows: [String!]
  user: User @belongsTo
}
type State @model @auth(rules: [{allow: public}]) {
  id: ID!
  cities: [City] @hasMany
  auction: Auction @hasOne
  user: User @belongsTo
  regionHUB: RegionHUB @belongsTo
  sortType: String @index(name: "StateByDate", queryField: "StateByDate", sortKeyFields: ["createdAt"]) @default(value: "State")
  createdAt: String!
}
type UserStateLike @model @auth(rules: [{allow: public}]) {
  id: ID!
  from: String!
  user: User @belongsTo
}
type City @model @auth(rules: [{allow: public}]) {
  id: ID!
  state: State @belongsTo
  regions: [Region] @hasMany
  sortType: String @index(name: "cityByDate", queryField: "cityByDate", sortKeyFields: ["createdAt"]) @default(value: "City")
  createdAt: String!
}
type Region @model @auth(rules: [{allow: public}]) {
  id: ID!
  city: City @belongsTo
  regionsUSD: [RegionUSD] @hasMany
}
type Notification @model @auth(rules: [{allow: public}]) {
  id: ID!
  description: AWSJSON
  user: User @belongsTo
}
type Account @model @auth(rules: [{allow: public}]) {
  id: ID!
  regionsSellUSD: [RegionSellUSD] @hasMany
  user: User @belongsTo
}
type RegionHUB @model @auth(rules: [{allow: public}]) {
  id: ID!
  State: State @hasOne
  auction: Auction @hasOne
  user: User @belongsTo
}
type Auction @model @auth(rules: [{allow: public}]) {
  id: ID!

}

Project Identifier
No response
Log output

# Put your logs below this line

Additional information
No response

Hey @Simoneth :wave: thanks for raising this! In order to address this issue effectively can you provide a few details around the steps taken prior to receiving this error? Also, would you be open to sending us your project identifier after running amplify diagnose --send-report?

I'm getting a similar error on push:
An error occurred during the push operation: / ["Index: 0 State: {\"deploy\":\"waitingForDeployment\"} Message: Resource is not in the state stackUpdateComplete"]
Have sent a report:
Project Identifier: 1d28c406156f9f15abedc2efa88e562a


Please make this a fork of the parent repo - https://github.com/spring-petclinic/spring-petclinic-microservices

Please make this a fork of the parent repo. This way - it can be listed among the available forks for petclinic and directly absorb all the changes happening on the parent repo

Prefer to provide a new copy of the base repo at https://github.com/spring-petclinic/spring-petclinic-microservices instead of forking from it.
Also, @selvasingh how can we make this existing repo a fork of https://github.com/spring-petclinic/spring-petclinic-microservices? Would we need to recreate this Azure-Samples/java-microservices-aca-lab repo?

Yes, a simple recreation will help

Thing is that there is quite some ongoing work on the repo currently ... like people working on forks ... they would have a merge hell.
So basically what are your thoughts on this PR #44? That's a fresh copy.


Typo in default repeat value of VariableComponent class constructor

In the constructor of the Variable Component class, I believe the default value of repeat should be repeat='http://sbols.org/v2#one instead of repeat='http://sbols.org/v2#on'.

Thanks for the additional report. This is covered by #387 which will be merged soon.


Update dockershim removal FAQ with content for after v1.24 is released

https://blog.k8s.io/2022/02/17/dockershim-faq/ explains the removal of dockershim but is framed as happening in the future. Update that article so that it's written as if dockershim was already removed.
Alternative: publish a third blog post, move the alias around, and edit the existing two blog posts to link to the updated updated one.

/retitle Update dockershim blog articles with content for after v1.24 is released

/approve

/lgtm
thanks everyone!


Move broker handling to the operator (out of subctl)

Currently the broker creation is managed from subctl, while it should go into the operator itself (for future coordinated upgrade handling, etc...)

@skitt @mangelajo Maybe we can sync on this and I can help?

@skitt @mangelajo Maybe we can sync on this and I can help?

We’re waiting for the migration to SDK 1 to be finished.

@dfarrell07 can you look at this now that SDK v1 is merged?

@dfarrell07 do you still plan on taking care of this, or would you like me to do it?

@dfarrell07 do you still plan on taking care of this, or would you like me to do it?

@skitt Yeah I got pulled away by various other TODOs, sorry. If you have cycles for it now you're very welcome to take it, I'm sure you'd get it done much better and faster than I could. I'd be happy to review and learn from your implementation.

@dfarrell07 do you still plan on taking care of this, or would you like me to do it?

@skitt Yeah I got pulled away by various other TODOs, sorry. If you have cycles for it now you're very welcome to take it, I'm sure you'd get it done much better and faster than I could. I'd be happy to review and learn from your implementation.

OK, I’ve assigned this to the two of us, I’ll look into it this week.


check [ConnectClientId] connection status

Do you hava a function ,use check  [ConnectClientId]  connect  status ,is connect or disconnect?
And have a funtion ,show all connect infomation ,for example connect status ,group,clientid........?

We are closing this issue because no further action is planned for this issue. If you still have any issues or questions, please log a new issue with any additional details that you have.


Compatibility with WPForms Pro

https://wordpress.org/support/topic/wpforms-conflict-with-wp-stateless/#post-10344055

@alimuzzaman can you update the existing WP Forms Lite compatibility to work for both the Lite and Pro versions?

@ewsopp Yes.


Enable Java development using Eclipse

When you want to contribute as a developer, you can work with any Java development environment. When you are using Eclipse, IntelliJ IDEA, Visual Studio or Xcode, there are additional plugins for Gradle to simplify the generation of the respective project files for each environment:
Eclipse
Generates Eclipse project files for the build that can be opened by the IDE. This set of plugins can also be used to fine tune Buildship’s import process for Gradle builds.
IntelliJ IDEA
Generates IDEA project files for the build that can be opened by the IDE. It can also be used to fine tune IDEA’s import process for Gradle builds.
Visual Studio
Generates Visual Studio solution and project files for build that can be opened by the IDE.
Xcode
Generates Xcode workspace and project files for the build that can be opened by the IDE.
For now we will focus on the Eclipse plugin.

Works for Eclipse now. I had to create resource filters for the embedded jetty so that we don't get so many irrelevant error messages.
Also, I had to exclude duplicate jar files from the local gradle cache:
(it.path.contains('commons-lang-2.6.jar') && it.path.contains('.gradle/caches'))
||
(it.path.contains('commons-logging-1.1.1.jar') && it.path.contains('.gradle/caches'))


[Concurrency][SE-0434] Treat captured local globally-isolated functions as Sendable

Description
No response
Reproduction
func test() {
  @MainActor func f() {} // ⚠
  Task {
    await f() // ⚠
  }
}

Expected behavior
The above should compile under SE-0434, because f is globally-isolated.
Environment
Swift version 6.0-dev (LLVM ceb203a206ed550, Swift a5fdf158dc4292a)
Additional information
No response

Update: the code in this issue already works, but the following (reduced from #75799) doesn't:
func usesFunction(_ block: @escaping @Sendable () -> Void) {}

func test () {
  let closure: @MainActor () -> Void = {}
  @MainActor func f() {}

    usesFunction {
        Task {
            await closure() // ⚠
            await f() // ⚠
        }
    }
}

I guess this is the same issue I'm seeing with this reproduction;
public struct Bar {
    @MainActor
    func bar() { }
}

func append(_ body: @escaping @MainActor () -> Void) { }

append(Bar().bar) // ⚠ Converting non-sendable function value to '@MainActor @Sendable () -> Void' may introduce data races

If I mark Bar: Sendable (or internal so its inferred) then it's fine


agate: finish memtable

Based on
https://github.com/tikv/agatedb/pull/45
https://github.com/tikv/agatedb/pull/46
https://github.com/dgraph-io/badger/blob/45bca18f24ef5cc04701a1e17448ddfce9372da0/memtable.go
Signed-off-by: GanZiheng<EMAIL_ADDRESS>

@skyzh Please don't merge it so hurry, I need time to review  🤣

Next time for sure! I'll wait for 2 reviews before merging next time 🥰


VectorizationPass can fail due to "operand does not dominate this use"

When invoking DAPHNE by bin/daphne --vec scripts/algorithms/lmDS.daph XY=\"test/runtime/local/io/ReadCsv1.csv\" icpt=2 reg=0.0000001 verbose=false (the concrete file does not matter), the system crashes during the VectorizationPass with the following error message:
loc(fused["/daphne/scripts/algorithms/lmDS_.daph":72:19, "/daphne/scripts/algorithms/lmDS_.daph":73:20, "/daphne/scripts/algorithms/lmDS_.daph":73:27, "/daphne/scripts/algorithms/lmDS_.daph":99:6, "/daphne/scripts/algorithms/lmDS_.daph":100:6, "/daphne/scripts/algorithms/lmDS_.daph":58:8]): error: operand #2 does not dominate this use
"builtin.module"() ({
  "func.func"() ({
  ^bb0(%arg0: !daphne.Matrix<2x3xf64>, %arg1: !daphne.Matrix<2x1xf64>, %arg2: si64, %arg3: f64, %arg4: i1):
    ...
    %19 = "daphne.constant"() {value = 2 : si64} : () -> si64
    ...
    %39:6 = "daphne.vectorizedPipeline"(%arg0, %21, %42, ...) ({
      ...
    }, {...}) {...} : (...) -> (...)
    ...
    %42 = "daphne.cast"(%19) : (si64) -> f64
    ...
  }) {function_type = () -> (), sym_name = "main"} : () -> ()
}) : () -> ()
loc("scripts/algorithms/lmDS.daph":0:0): error: module pass error

The operand #2 mentioned in the error message is the SSA value %42, which is obviously created after the daphne.vectorizedPipeline, which is not valid. The SSA value %42 is the result of a cast of the scalar %19 from type si64 to f64.
Somehow, the VectorizationPass messes up the IR in such a case.

The scalars in two affected scripts were changed from si64 to f64 in commit 5572587cc3159f1c029a9523d829ac60486b3447. However, this only avoids the problem, but doesn't fix it. These changes to the scripts should be undone once the bug is fixed, since using integer literals in those places is more intuitive and perfectly valid in DaphneDSL.


Vsts/release 37

Updates for release 1.0.788

Docs Build status updates of commit 84bb128:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

powerbi-ps/MicrosoftPowerBIMgmt.Admin/Get-PowerBIActivityEvents.md
:white_check_mark:Succeeded
View (powerbi-ps)

powerbi-ps/MicrosoftPowerBIMgmt.Admin/MicrosoftPowerBIMgmt.Admin.md
:white_check_mark:Succeeded
View (powerbi-ps)

powerbi-ps/MicrosoftPowerBIMgmt.Reports/MicrosoftPowerBIMgmt.Reports.md
:white_check_mark:Succeeded
View (powerbi-ps)

powerbi-ps/MicrosoftPowerBIMgmt.Reports/Remove-PowerBIReport.md
:white_check_mark:Succeeded
View (powerbi-ps)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


knctl snap packaging

I would like to share some work on packaging knctl as a snap and kindly ask your feedback and thoughts. You can find the source on _https://github.com/ipolyzos/knctl-snap_ while the snaps can be downloaded directly from snapcraft store (currently versions v0.0.10 and v0.1.0 are unploaded).

i see. fair enough. thanks for sharing! this issue will should be findable in this repo and might be useful for some people.


Proofread and reworked README file

I read the entire README file and made some minor edits for clarity and to fix general wording errors.  Nothing major but I believe it's good to go now.

thanks for your contribution, i have manually copied ur code


Output of  "fly aps private"

I used the script w/o success.
It didn't find the IP address.
I changed this part of the script :
 fly ips private | awk '(NR>1){ print $5 }'
and it WORKED!
(changed $3 to $5)

The observer script was completely reworked and now uses the fly status --json command. It requires the jq command to be present in the OS, which is a command line JSON parsing utility.


Add polygon to supported network names

Fixes #307 .

I switched this PR to a draft because my initial fix didn't work.


Do not require enable-api-fields=alpha for spire

Prior to this commit, enabling spire support also required setting "enable-api-fields" to alpha. "enforce-nonfalsifiability" is the only behavioral feature flag that also requires setting "enable-api-fields" to alpha, even though it is not the only behavioral feature flag in an alpha state.
This commit removes the requirement that cluster operators set enable-api-fields to alpha in order to use spire.
In addition to creating a consistent approach among behavioral feature flags, this will allow cluster operators to opt into spire without having to opt into all alpha API fields.
Note: Spire support is not yet functional.
/kind misc
Submitter Checklist
As the author of this PR, please check off the items in this checklist:

[x] Has Docs if any changes are user facing, including updates to minimum requirements e.g. Kubernetes version bumps
[x] Has Tests included if any functionality added or changed
[x] Follows the commit message standard
[x] Meets the Tekton contributor standards (including functionality, content, code)
[x] Has a kind label. You can add one by adding a comment on this PR that contains /kind <type>. Valid types are bug, cleanup, design, documentation, feature, flake, misc, question, tep
n/a Release notes block below has been updated with any user facing changes (API changes, bug fixes, changes requiring upgrade notices or deprecation warnings). See some examples of good release notes.
n/a Release notes contains the string "action required" if the change requires additional action from users switching to the new release

Release Notes
NONE

/retest

/lgtm

LGTM.
cc @jerop for visibility since https://github.com/tektoncd/pipeline/pull/6527/files added requirements of setting "enable-api-fields" to

/hold

This change was introduced in https://github.com/tektoncd/pipeline/pull/5902/files#r1086050834 following a discussion between @bobcatfish and @jagathprakash so it'd be great to hear from them -- this change sgtm
@chuangw6 https://github.com/tektoncd/pipeline/pull/6527/files was only refactoring the implementation, it didn't make any functional changes

Thanks for the additional context! @bobcatfish and @jagathprakash aren't actively working on OSS tekton anymore, and @chuangw6 is taking over spire work and has given a LGTM. I'll reach out to the two of them, but I think it makes the most sense to evaluate this change based on the reasons given in the commit message. It looks like the linked conversation was more about returning an error instead of silently failing when enable-api-fields was not set to alpha, rather than the decision to couple the two flags in the first place.

/lgtm

/lgtm


[Improve][Connector-V2][ElasticSearch] Improve es bulk sink retriable mechanism

These code change as bleow:

Improve the retriable mechanism of es bulk sink
Introduce a method computeRetryWaitTimeMillis to compute the wait time
Fix the attempt times bug within method retryWithException

Purpose of this pull request

Check list

[ ] Code changed are covered with tests, or it does not need tests for reason:
[ ] If any new Jar binary package adding in your PR, please add License Notice according
New License Guide
[ ] If necessary, please update the documentation to describe the new feature. https://github.com/apache/incubator-seatunnel/tree/dev/docs

Please resolve ci problem, thanks.

The other code is fine.


strict mode

Couldn't find a discussion on the topic.
What's the logic behind not enforcing strict mode in standard?

Knowing that Node ^4.0.0 requires strict mode to be able to use the new ES6 let, const keywords, function-in-blocks and classes.

My guess is so that you can check for snippets of a file, but I don't know enough about the project's history to make an authoritative comment.

standard already warns for strict-mode incompatible syntax. ES6 already has strict mode enabled by default. I think that between the two, enforcing the "strict mode" pragma at the top of every file should be left up to the authors.
Also: enforcing the pragma would break a ton of packages using standard.

Right, what @yoshuawuyts said.
standard already assumes that your code is running in an ES6 environment, so you'll warnings for things that are strict-mode incompatible. But enforcing "use strict" is going too far, since it doesn't meaningfully change the way code runs for the vast majority of standard users.
If you need node support, just add "use strict" to your files.


WiFI Stick LSW-5

Hi everyone.
I tried adding my afore af6k-sl inverter without success.
unfortunately the inverter has an LSW-5 wifi stick with serial code 35xxx, no one was able to add it to the home assistant.
Is it possible to replace the key with an LSW-3? Can the inverter communicate with a different stick? Thank you

It may be complicated. LSW-5 has USB, and LSW-3 has a "DB9" interface. You can check difference of sticks here: https://www.solarmanpv.com/products/data-logger/stick-logger/


Kafka offramp stops sending data to Kafka if the send request times out

Problem

It was observed that the Kafka offramp stops sending records to Kafka when the Kafka brokers are responding slow leading to a request time out. It looks like the producer buffer queue  gets filled out quick if Kafka brokers respond slow. As a result the producer completely stops sending any new records .
Steps

Start a Kafka offramp with default value for  queue.buffering.max.messages.
Keep sending records to the offramp with large volume
Inject a netwrok delay to Kafka brokers using any network emulator such as netem

Possible Solution(s)

Currently Tremor is using rdkafka::producer::FutureProducer to send record to Kafka with 1 ms  wait on the producer buffer queue to be freed. It can be changed to either no-wait or use send_result instead of send function . Use of send_result is preferred as it will return error immediately in case of failure in enqueuing.
impl Offramp for Kafka {
    // TODO
    fn on_event(&mut self, codec: &Box<dyn Codec>, _input: String, event: Event) -> Result<()> {
        for value in event.value_iter() {
            let raw = codec.encode(value)?;
            let mut record = FutureRecord::to(&self.topic);
            record = record.payload(&raw);
            //TODO: Key
            if let Some(ref k) = self.key {
                task::spawn(self.producer.se.**send(record.key(k.as_str()), 1)**);
            } else {
                task::spawn(self.producer.**send(record, 1)**);
            };
        }
        Ok(())
    }

Notes
Output of rustup --version:
Output of rustup show:
Output of tremor-server --version:0.8.0
(edit/hg: added code block for readability)

Hi Anup! Good catch :)!
It's perfect timing that you found this, we're currently working on circuit breaker and guaranteed delivery and that will require to revisit the Kafka offramp! Knowing about this issue ahead of getting there wills safe some time :D.
I'll link this to the current PR in the area and look at it as part of the offramp overhaul.
If this issue is time critical let us know and we'll look into options to back port that changes from dev to 0.8.*

Thanks @Licenser . This is critical for us now as we see this issue quite frequently. Please see if the change can be added as a hot fix to 0.8.*.

Heya :D thanks for the quick feedback! If it's critical then something will make it in 0.8 hotfixes one way or the other :), I'll put that on the plate first thing next week to look into and see how quick we can resolve fixed.


ResponseArray not available in this version?

Hi!
I should be able to get an array of objects in just 1 line using request.responseArray(). This method does not seem to exist? Any thoughts?

Hey @CedricVanhaverbeke,
Not sure why you are experiencing difficulties as this method still exists:
public func responseArray<T: BaseMappable>(queue: DispatchQueue? = nil, keyPath: String? = nil, context: MapContext? = nil, completionHandler: @escaping (DataResponse<[T]>) -> Void) -> Self

Can you give an example with this method?

Please see the tests included in this project


requirements: Upgrade Python requirements

prod.txt upgrades

boto3: 1.33.6 → 1.33.10
botocore: 1.33.6 → 1.33.10
django-stubs-ext: 4.2.5 → 4.2.7
django[argon2]: 4.2.7 → 4.2.8
importlib-metadata: 6.9.0 → 7.0.0
yarl: 1.9.3 → 1.9.4

Other upgrades

boto3-stubs[s3,ses,sns,sqs]: 1.33.6 → 1.33.10
botocore-stubs: 1.33.6 → 1.33.10
django-stubs: 4.2.6 → 4.2.7
moto[s3]: 4.2.10 → 4.2.11
platformdirs: 4.0.0 → 4.1.0
ruff: 0.1.6 → 0.1.7
semgrep: 1.51.0 → 1.52.0
types-psycopg2: <IP_ADDRESS> → <IP_ADDRESS>

Removing the backport candidate label as we didn't backport this before releasing 8.0.


Does not support null safety

Does not support null safety you need to remove null safety code from flutter which is not good

Just updated, glad to find this package useful!

Just updated, glad to find this package useful!

Thank you for your quick response . Yeah it was very useful for my project.


Not an issue, but a suggestion.

Hi Mathias! I'm a great fan of what you and Ramin, and are currently doing my master thesis in robotics based on the CfC. Playing around with the ncps package I noticed a pretty extreme improvment in training time using impala-cnn in the atari-tf example (BC). The convolution block I used was pretty simple:
class impalaConvLayer(tf.keras.layers.Layer):
def init(self, filters, kernel_size, strides, padding='valid', use_bias=False):
super(impalaConvLayer, self).init()
self.conv = Conv2D(
filters=filters,
kernel_size=kernel_size,
strides=strides,
padding=padding,
use_bias=use_bias,
kernel_initializer=tf.keras.initializers.VarianceScaling(scale=2.0, mode='fan_out', distribution='truncated_normal')
)
self.bn = BatchNormalization(momentum=0.99, epsilon=0.001)
self.relu = ReLU()
@tf.function
def call(self, inputs):
    x = self.conv(inputs)
    x = self.bn(x)
    x = self.relu(x)
    return x

class ImpalaConvBlock(tf.keras.models.Sequential):
def init(self):
super(ImpalaConvBlock, self).init(layers=[
impalaConvLayer(filters=16, kernel_size=8, strides=4),
impalaConvLayer(filters=32, kernel_size=4, strides=2),
impalaConvLayer(filters=32, kernel_size=3, strides=1),
Flatten(),
Dense(units=256, activation='relu')
])
As I believe training time on weak computers often discourage students I think making the example run faster could be wise. What do you think about using impala-cnn before CfC? Is there something I've overlooked that makes this a bad idea?
Anyways, keep up the good work! What you've accomplished is really inspiring!
Robin

Thanks @R-Liebert for pointing this out.
I have updated the docs highlighting this and changed the TF example


Challenge required?

This is a new one because usually if you see this you sign in that account on the app to fix it but there is nothing. I have tried deleting cache/cookies for account to generate new ones and it still happens even if downloading just 1 live it'll stop randomly and merge the files.

To reproduce this, do pyinstalive -df then try it one more time (any time between the first command) it will give a long error and say challenge required. This even happens on fresh accounts.

try testing branch it uses web api instead and its pretty much a rewrite of the entire script that i never finished https://github.com/dvingerh/PyInstaLive/tree/testing
the config file you use will no longer be valid you have to update the names of most values as stated under Usage in the readme

try testing branch it uses web api instead and its pretty much a rewrite of the entire script that i never finished https://github.com/dvingerh/PyInstaLive/tree/testing

This is what I originally had, only went to master release after this issue happened which also has the same thing

Yeah my mistake seems like another instagram thing which is cringe. If anyone comes across issue wait a day or two for that account before idling with it again.


Error when opening custom item menu

File "D:\Programs\Python35\lib\site-packages\spockbot-0.1.3-py3.5.egg\spockbot\plugins\helpers\inventory.py", line 170, in handle_open_window
KeyError: 'minecraft:container'

PS: Just produced a fix (PR #209)

Thanks for te PR. On what server did you see that inv type?

A minecraft server network with a item menu to join sub-servers.

This bug will be fixed when minecraft-data merges https://github.com/PrismarineJS/minecraft-data/pull/87

Going to call this fixed unless there are further reports


mad MAx

madmax plotter windows makes 2-3 plots then it waits until it is transferred to the hdd and that sarting again can someone help me Please

I start with 20 plots the first 2-3 plots it does without waiting after the 3 plot it waits until it is transferred to the hdd at the beginning everything went well why does this happen to me now

Maybe the queue of transferring is full? increase the speed of transferring and try again.

yeah it waits until previous copy is finished

how can i increase the transfer speed do you have an idea


distributed inference

We need to support distributed inference...
In addition, for now, let's make sure to add a warning when a user runs predict with accelerators > 1:
“warning: you’re running .predict with devices > 1. predictions across multiple {self.accelerator}s is not yet supported.”

# prints like
“warning: you’re running .predict with devices > 1. predictions across multiple GPUs is not yet supported.”

so that user could do: https://github.com/Lightning-AI/lightning/pull/15149#issuecomment-1282360328
cc @justusschock @kaushikb11 @awaelchli @akihironitta @rohitgr7 @Borda @carmocca

cc @Borda @carmocca

In the latest 1.7.1, multi-GPU prediction is still not supported.

is this function exist now (ver 2.1)?


Fixed wrong latitudes in NDP CMORizer

Description
While looking at plots of vegetation carbon content of the dataset NDP, @bettina-gier and I realized that the latitude is shifted by about ~5° (see figure below). After looking the documentation we found that the end value of latitude should be ~83.6° instead of 90°. This PR fixes that.

@remi-kazeroni We also realized that the raw data on Levante is a .tar.gz file, but the CMORizer expects an unzipped file. Thus, using the command esmvaltool data format NDP with the default RAWOBS path failed for me. Using my own directory and downloading the files with esmvaltool data prepare NDP worked.

Checklist
It is the responsibility of the author to make sure the pull request is ready to review. The icons indicate whether the item will be subject to the 🛠 Technical or 🧪 Scientific review.

[x] 🛠 This pull request has a descriptive title
[x] 🛠 Code is written according to the code quality guidelines
[x] 🛠 Documentation is available
[x] 🛠 Tests run successfully
[x] 🛠 The list of authors is up to date
[x] 🛠 Any changed dependencies have been added or removed correctly
[x] 🛠 All checks below this pull request were successful

New or updated data reformatting script

[x] 🛠 Documentation is available
[x] 🛠 The dataset has been added to the CMOR check recipe
[x] 🧪 Numbers and units of the data look physically meaningful

To help with the number of pull requests:

🙏 We kindly ask you to review two other open pull requests in this repository

Awesome, thanks for the quick review @remi-kazeroni !!


Inconsistent array casting

Short arrays aren't enough to loop
[to 10] does what expected and so does [.. for [0 to 10]]
But in contrast...
[.. for [to 10]]  throws

Parse error on line 1: Unexpected 'FROM'

Just FYI: [.. for to 10] now works thanks to #859

[.. for to 10] works, but [.. for [to 10]] still doesn't; shouldn't this remain open?

Lexer is to blame.

I am going to patch this temporarily with a grammar fix.


Can't turn on viewing emails in plain text

Please search to check for an existing issue (including closed issues, for which the fix may not have yet been released) before opening a new issue: https://github.com/k9mail/k-9/issues?q=is%3Aissue
Expected behavior
Tell us what should happen
Actual behavior
Tell us what happens instead
Steps to reproduce

Environment
K-9 Mail version:
Android version:
Account type (IMAP, POP3, WebDAV/Exchange):

See #906


Add user name regex for LDAP authentication

New ConfigKey for set up a regex pattern to be matched by the user names before checking credentials in the LDAP server, avoiding unnecessary request
eg. for  domain\user pattern:
brooklyn.webconsole.security.ldap.user_name_regex=.*\\.*

Doc for this: https://github.com/apache/brooklyn-docs/pull/337


[LG] Fix recursive functions error in analyzer

Fixes https://github.com/microsoft/botbuilder-js/issues/3619
Description
Support analyze recursive functions in AnalyzeTemplate API.

Coverage decreased (-0.05%) to 78.876% when pulling 7d8e2bdd9ec24f95ac4e76ed43715b343fa0f884 on hond/analyzer into faf843af5d1cbec00e1b77384e991e77fa0166bb on main.


incorrect CSS order after build when manualChunk is used

Describe the bug
backgound:
The project needs a third party lib css ( libs/pink.css), so I include the file in the main.js, and make it as a manual chunk with vite.config.js:
// third party lib css
import './libs/pink.css';

import { createApp } from 'vue';
import App from './App.vue';

createApp(App).mount('#app');

  build: {
    rollupOptions: {
      output: {
        manualChunks(id) {
          if (id.includes('libs/pink.css')) {
            return 'vendor-pink';
          }
        },
      },
    },
  },

then I need to override some selector , so I add selector in App.vue ,
// app.vue

The Problem:
In dev mode,  npm run dev and the body background color is "lightbue". And this is what I want.
But after I run build command (npm run build , npm run preview), the page background color is "pink".
Because in vendor-pink.css is injected behide index.css:
// file: dist/index.html
<!DOCTYPE html>

    <meta charset="UTF-8" />
    <link rel="icon" href="/favicon.ico" />
    <meta name="viewport" content="width=device-width, initial-scale=1.0" />
    <title>Vite App</title>

    <link rel="stylesheet" href="/assets/index.94bf859f.css">
    <link rel="stylesheet" href="/assets/vendor-pink.52a267b7.css">

How
As I known, pink.css is treated as a "vendor" asserts, then it should be injected before index.css.
how can i achive this ? like

    <link rel="stylesheet" href="/assets/vendor-pink.52a267b7.css">
    <link rel="stylesheet" href="/assets/index.94bf859f.css">

Reproduction
https://stackblitz.com/edit/vitejs-vite-21apjj?file=dist%2Findex.html&terminal=dev
System Info
❯ npx envinfo --system --npmPackages '{vite,@vitejs/*}' --binaries --browsers
success Install finished in 1.522s

  System:
    OS: Linux undefined
    CPU: (4) x64 Intel(R) Core(TM) i9-9880H CPU @ 2.30GHz
    Memory: 0 Bytes / 0 Bytes
    Shell: Unknown - /bin/jsh
  Binaries:
    Node: 14.16.0 - /usr/local/bin/node
    Yarn: 1.22.10 - /bin/yarn
    npm: 7.17.0 - /bin/npm
  npmPackages:
    @vitejs/plugin-vue: ^2.0.0 => 2.0.1
    vite: ^2.7.2 => 2.7.10

Used Package Manager
npm
Logs
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Make sure this is a Vite issue and not a framework-specific issue. For example, if it's a Vue SFC related bug, it should likely be reported to https://github.com/vuejs/vue-next instead.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

Hey there. I am also facing same issues. Is there a way to solve this?

also want to known how to solve this

I have the same problem, but when I upgrade vite to '2.9.0-beta.2' and modify 'vite.config.ts', it will return to normal
If you still return 'vendor-pink', it will not take effect
https://stackblitz.com/edit/vitejs-vite-1pgemr?file=vite.config.js&terminal=dev

I also had the same problem.
I use the output.manualChunks  and find builded css files inserted to  are in an unexpected order.
Now I have to set cssCodeSplit: false to circumvent this problem for temporary.
Hope this issue will be resolved sooooon!

+1

same issues

There are two lines should be took cared:
1.
build: {       cssCodeSplit: true, }
2. imported external css should be place before your imported main component.
3. manual chunks the imported the external css or scss into a verder.css like:
         manualChunks(id) {             if (id.includes('node_modules/xxx/src') && id.includes('.scss')) {               return 'vendor';             }           }
which will be linked the index.html and check the external and your internal css order should be right.


Database Backup task

Initial cut at adding database backup task. I added an integration test, but still need to create unit tests
If you like the direction I'm going here, I plan to also contribute a database migration task inspired by the method described in https://www.martinfowler.com/articles/evodb.html
Looking forward to your feedback and ideas :)

Looks interesting. Taking a backup was something I was considering doing myself, as had to implement that in one place recently.
I'll have a better look in a next few days when with a bit more sleep under my belt pillow. Will approve
Can you elaborate on your plan for migration task? (Probably creating a separate issue for that would be easier to track)
And thanks for contributing!

Re appveyor:  I think for PRs it does not publish the nuget to myget because the publish key is only available for my builds.
Otherwise good work - thanks very much. I'll clean up some other stuff and will push a release soonish

Published to NuGet.org v1.12.0


1.18.x download

Please give release 1.18.x to download :D

Hey! You can compile the project if you need this, by cloning (downloading) the repository, and going into command prompt/powershell (Windows) or terminal (macOS, Linux) and using cd to go into the location you downloaded the folder to (go into the folder too). You'll then want to run ./gradlew.bat build (Windows) or ./gradlew build, wait for it to build, then get your jarfile from the build/libs folder (it's the one without the -sources.jar suffix


File-encryptor sample: fixed PKCS5 paddings for block aligned contents

This sample does not impact this sample when running with input that's not block fully aligned.
But for other file input, it needs to pad an extra block to make it fully compliant with PKCS5 padding rules

bors r+


[QUESTION] Dual boot source in Jetson AGX Orin

Describe the issue
My question is: How can I configure the files on the two memory devices (eMMC and USB Stick) so that if the USB Stick is not connected to the Jetson during boot, then the system will boot from the internal eMMC memory device.
I'm currently using Ubuntu 20.04 as the host pc and the Jetson AGX Orin devkit. I used the flash_jetson_external_storage.sh algorithm to flash the system onto a USB Stick and currently the system is being loaded from this USB Stick.
I'm new to this booting process and I know the file ".../boot/extlinux/extlinux.conf" but just changing this file doesn't seem to be enough.
If you need more information to answer my question, please let me know
Which version of Ubuntu on the Host machine
Ubuntu version: Ubuntu 20.04 as the host pc
What version of L4T/JetPack
L4T/JetPack version: 35.4
Which Jetson
Jetson: AGX Orin devkit

Also, how do I change the label of this issue from bug to question?

@PHCLenzi - I believe that you should be able to set the boot order in the UEFI setup. USB first, eMMC and so on.
You should be able to change the bug to question in the Labels category in the sidebar.
You should probably ask on the AGX Orin Jetson forum for a better answer for your question if setting the boot order does not work.

Hallo @jetsonhacks. I had imagined that during the execution of the "bootFromExternalStorage/flash_jetson_external_storage.sh" script, which in turn executes the "nvsdkmanager_flash.sh" script, there were target configuration files to alter the boot process once the kernel and filesystem had already been flashed to removable memory. I would like to know which files are changed during the execution of "bootFromExternalStorage/flash_jetson_external_storage.sh" that alter the booting process. Do you know?

nvsdkmanager_flash.sh creates a bootable image on the host and then flashes it to the Jetson. To my knowledge, there isn't any post processing to change target configuration files or alter the boot process once flashed.


pip uninstall does not remove *-nspkg.pth for packages installed in editable mode

Reporting here as this is related to the updated *-nspkg.pth mechanism, although I'm not sure if it is a pip or setuptools issue.
I'm working in a virtualenv with pip (9.0.1) and setuptools (32.3.1) with python 2.7.
I have a package a1 which declares a namespace package.
pip install -e a1/ creates a1-nspkg.pth in site-packages as well as a1.egg-link.
pip uninstall a1 removes a1.egg-link but leaves a1-nspkg.pth behind.
I could find a previous mention of this issue in https://github.com/pypa/setuptools/issues/250#issuecomment-258714212 by @jaraco

I had meant to file this as a ticket with pip. I don't think there's anything that Setuptools can do to signal which files get removed, so I believe it's strictly a pip issue.

@jaraco thanks for the feedback. I filed pip https://github.com/pypa/pip/issues/4213


How to chain two type httprouter.Handler and regular http.HandleFunc

Hi I met a issue to use this router.
I'm trying to add a prometheus midware into our endpoint implementation. But our endpoint are using httprouter. Then when I tried to add this midware into existing code base, I cannot find a good way to integrate both together.

router := httprouter.New()
router.GET("/hello", r.Hello)

func (r configuration) Hello(w http.ResponseWriter, req *http.Request, ps httprouter.Params)

func InstrumentHandlerFunc(name string, handler http.HandlerFunc) http.HandlerFunc {
    counter := prometheus.NewCounterVec(
		do something...
	)

	duration := prometheus.NewHistogramVec(
		do something...
	)
	return promhttp.InstrumentHandlerDuration(duration,
		promhttp.InstrumentHandlerCounter(counter, handler))
}

What I need is I need pass a httprouter.Handle endpoint fucntion pointer into a midware function and generate a new httprouter.Handle. Then I can use this httprouter.Handle to register it into the router. When the requests comes it will call that midware first then call the real endpoint.
Below is what I want to do

func InstrumentHandlerFunc(name string, handler httprouter.Handle) httprouter.Handel {

}

router.Get("/hello", InstrumentHandlerFunc("/hello", r.Hello))

Hey, I answered something similar that may help you in an older issue https://github.com/julienschmidt/httprouter/issues/198

Check out github.com/rileyr/middleware
It really helped with chaining middleware with httprouter

See #198

Actually, let's add something to make this easy and intuitive. This should be a rather common use-case.


Update Android SDK to Android 11

Target SDK version upgraded to 30
Migrated to AndroidX
Deprecated GoogleApiClient replaced with FusedLocationProviderClient

Hello, when is this expected to be merged and released?
Would this close the AndroidX issue described here? https://github.com/SiftScience/sift-android/issues/74 siftscience is currently blocking my current company from dropping jetifier.

@sanogueralorenzo We have a future ticket for that fix which is going to be picked up soon, will update on the timeline shortly

@sanogueralorenzo just a quick update, I was mistaken earlier and this ticket has the merged code to address AndroidX issue


Properly handle titlecase characters in cow_to_uppercase and cow_to_lowercase

Fixes #2

I just added benchmarks for to_lowercase on master and it's pretty unfortunate that even for the best-case scenario of no-op to_lowercase Cow version is now 3x slower than regular one whereas w/o this change it's 2x faster:
To Lowercase/to_lowercase/ax40
                        time:   [146.32 ns 149.49 ns 152.90 ns]
                        change: [-5.7087% -2.7402% +0.0303%] (p = 0.08 > 0.05)
                        No change in performance detected.
Found 9 outliers among 100 measurements (9.00%)
  8 (8.00%) high mild
  1 (1.00%) high severe
To Lowercase/cow_to_lowercase/ax40
                        time:   [442.02 ns 449.97 ns 459.55 ns]
                        change: [+1.0719% +4.2921% +7.7698%] (p = 0.01 < 0.05)
                        Performance has regressed.
Found 11 outliers among 100 measurements (11.00%)
  1 (1.00%) high mild
  10 (10.00%) high severe

I understand it does more work now but perhaps there are some low-hanging optimisations? E.g. now that it has to use internal iterator, maybe it needs to flat_map it into the top iterator instead of just doing .find or something like that.
@thomcc Do you want to look into it?

Worst-case scenario, maybe we can only optimise ASCII lowercase strings and forward everything else to owned version... but if possible to optimise unicode scenario too, that would be better.

How much complexity are you willing to accept?

Yeah I thought about using unic_ucd which has this built-in but I suspect that will blow up the size of crate considerably.
I'd rather keep it as small and simple as possible.
OTOH how large is the list of exceptions that don't fit into the previous !is_uppercase check? I'd be willing to accept build-time complexity if runtime / size cost is not too large.

The size cost should be relatively small. I'll look into it when I have more time.

Cow version is now 3x slower than regular one whereas w/o this change it's 2x faster:

I was wondering about this too, but didn't have a chance to bring it up...
Thom and I work together, and one of the projects we're working on really wants to continue to use cow-utils, so @RReverser, if you're okay with a bit more complexity and paying a bit of size costs (it sounds like you are?), then we'll definitely get you a PR.
It'll be nice to have the best of all three of Speed, Cost, Correctness for once!

Yeah, as I said, as long as it doesn't blow up the size, it should be fine. It's unfortunate that essentially every crate (std, regex, us) has to ship its own version of unicode data though.


BUGFIX: I incidentally removed the feature gate check that enables the UseCertificateRequestBasicConstraints feature

I forgot to update this line in https://github.com/cert-manager/cert-manager/pull/6019.
/kind bug
Release Note
NONE

/retest

@inteon: #6154 failed to apply on top of branch "release-1.12":
Applying: BUGFIX: we incidentally removed the feature gate check that enables the UseCertificateRequestBasicConstraints feature
Using index info to reconstruct a base tree...
M	pkg/controller/certificates/requestmanager/requestmanager_controller.go
Falling back to patching base and 3-way merge...
Auto-merging pkg/controller/certificates/requestmanager/requestmanager_controller.go
CONFLICT (content): Merge conflict in pkg/controller/certificates/requestmanager/requestmanager_controller.go
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
Patch failed at 0001 BUGFIX: we incidentally removed the feature gate check that enables the UseCertificateRequestBasicConstraints feature
When you have resolved this problem, run "git am --continue".
If you prefer to skip this patch, run "git am --skip" instead.
To restore the original branch and stop patching, run "git am --abort".

@SgtCoDFish Actually, this bug has not been released yet.


Derive macro breaks when using where for trait with comma

The following works:
#[derive(Encode, Decode)]
pub struct MyStruct<T: Clone> {
    pub a: A,
    pub b: B<T>,
}

Using a where statement does not work:
#[derive(Encode, Decode)]
pub struct MyStruct<T>
where T: Clone,
{
    pub a: A,
    pub b: B<T>,
}

Error:
expected one of `{`, lifetime, or type, found `,`
The issue is made worse by the Rust formatter in VS Code insisting that there is a comma after the 'where' declaration. If not, then I could just remove the comma, but it is automatically inserted

Should be fixed in https://github.com/bincode-org/virtue/issues/35, thanks for reporting!


precision models can dirty data

add a note about precision models being a potential source of getting data dirty in fiona

Coverage remained the same at 76.489% when pulling ffa12571276b0377875c073b18b21009519da6c2 on kannes:patch-1 into 22c738981a8865547b243a82db35c0d685729570 on Toblerity:master.


Container state seems to be preserved in the engine

While researching a bug with Kafka and node-kafka which is unrelated to docker - I've stumbled upon a problem with docker engine.
After running with Kafka 2.0.0 you cannot go back to use the working Kafka 1.1.0 version without restarting the Doccker Service.
Even removing all container, images and volumes and then doing a docker system prune -a --volumes doesn't help.
The only thing that does is running service docker restart - no need to 'prune' - just restart.
I would expect state to be restored w/o even pruning - just killing the container should be enough.
The complete source code and environment can be found here
The kafka/node-kafka bug can be found here

@srfrnk This tracker is for issues with open source registry implementation and not Docker Engine. Kindly file issues in https://github.com/moby/moby/issues/ for problems with Docker Engine. Thanks!

@manishtomar - done
Would be nice if this was mentioned in the docs.
Currently your website refers to this repo :)


Not working for google's go language when golang file is formatted by gofmt

The indentLine is not working for google's go language.The golang file is formatted by gofmt command, so it contains TABs, and i tried to convert all TABs to spaces, it worked as expected.

This plugin is used for displaying thin vertical lines at each indentation level for code indented with spaces. For code indented with tabs I think there is no need to support it, because you can use :set list lcs=tab:|\ (here is a space).


[routing] Handler registration logging

Provide JavalinConfig.handlerRegistrationLogger to add a callback for
custom logging whenever a new handler is added.
Calling Javalin.enableDevLogging will add the
LogUtil.handlerRegistrationDevLogger for basic default logging as
handlers are added.
This PR addresses - https://github.com/tipsy/javalin/issues/1411

@tipsy Hi, will this be reviewed soon? Doesn't matter if it takes a while to get to it, as long as it's not forgotten! 😄

I haven't forgotten, but I'm a bit busy IRL at the moment. Could you rewrite it to use events and be a part of normal dev logging, as mentioned in the issue?

I think that should do it. I was convinced it couldn't be done that way before but looks like I was wrong (hopefully this is what you were thinking of).

I was thinking a bit simpler, give me a moment.

@tipsy Yep, definitely simpler. Wish I thought of this 👍 .


BUG: scipy.sparse.csr_matrix((data, indices, indptr), [shape=(M, N)]) returns different matrices when instantiated by the same attributes

Describe your issue.
There are two ways to instantiate a csr_matrix :

Defining the variables out of the  csr_matrix() function and instantiating it by passing parameters.
Defining the variables just in the csr_matrix() function.

With the same parameters, the matrices obtained in these two different ways are transposed to each other.
Reproducing Code Example
import numpy as np
import scipy.sparse as ssp

indptr = np.array([0, 2, 4, 6])
indices = np.array([0, 1, 1, 2, 0, 2])
data = np.array([1, 2, 3, 4, 5, 6])
a = ssp.csr_matrix((data, indices, indptr), shape=(3, 3)).toarray()  # 1
b = ssp.csc_matrix((np.array([1, 2, 3, 4, 5, 6]), np.array([0, 1, 1, 2, 0, 2]), np.array([0, 2, 4, 6])), shape=(3, 3)).toarray()  # 2

print(a)
print()
print(b)

Error message
[[1 2 0]
 [0 3 4]
 [5 0 6]]

[[1 0 5]
 [2 3 0]
 [0 4 6]]

SciPy/NumPy/Python version information
1.5.0 1.19.2 sys.version_info(major=3, minor=8, micro=12, releaselevel='final', serial=0)

Your b matrix is using CSC format, where a is using CSR.


Update traces exporter to OpenTelemetry HTTP Trace Exporter

Closes #96
Besides the change in the title, we also change a configuration knob, tracing.url to tracing.endpoint to be more technically correct.
There appear to be many more changes in this PR than are actually there.  This is because Go's dependency manager, go modules, automatically adds to the repository the new dependencies needed (and removes the old ones) for the change from the Jaeger Trace exporter libraries to the OpenTelemetry Trace exporter libraries.
Thus, the go.mod, go.sum, and any vendor/* files can be ignored for this PR, as they are managed automatically by the go tools.
The only files to actually review are:

[ ] cmd/refresh-uids-from-ferry/main.go
[ ] cmd/refresh-uids-from-ferry/main_test.go
[ ] cmd/run-onboarding-managed-tokens/main.go
[ ] cmd/token-push/main.go
[ ] internal/tracing/tracing.go (where the biggest changes are - the above files just use the changes from this file)
[ ] managedTokens.yml

Thanks for the changes. LGTM


refactor: change dev workflow

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


wal: command not found in ~/.xprofile (Ubuntu/Gnome)

Hello,
I'm using Ubuntu/Gnome and to make the colorscheme persist I created the file ~/.xprofile (as it did not exist before) and added the line wal -R. However when I reboot, I get the message that the wal command not being found. Here is where wal is located: /home/ege/miniconda3/envs/minico/bin/wal. So should I invoke using the full path or is this just a workaround?
 If wal is not in your $PATH, then you will have to use the full location as expected.

Why is this issue closed?

Not an issue with pywal.

I understand. Then it would be good to know what the issue is about and how I can fix it.

Install pywal with your package manager.


Suggestion: ID placeholder for AL to get rid of IDs

Just an idea, what if we could remove IDs simply from AL development?

IMHO the IDs are not necessary when you write code in AL except when you define new objects or extend objects.
The NST compiler will check the IDs during publishing

Let's us a NAVautoID as placehloder which will be resolved and replced during publishing and compilation by the NST . This could look like this:
page NAVautoID "MyPage"
{
   // some code
}

table NAVautoID "MyTable"
{
   fields
   {
        field(NAVautoID; MyField; Integer)
        {}
   }
}

tableextension NAVautoID "MyTableExt" extends Item
{
   fields
   {
        field(NAVautoID; MyText; Text[30])
        {}
   }
}

In the application manifest app.json we could add a section / element where we are able to add JSON files which contains the licensed IDs e.g.:
{
     "NAVautoID": {
           "Table": ["50100..50150"],
           "Page": ["50100", "50102", "50200..50300"]
     }
}

Could this work? What do you think?

That sounds like a great idea! Then you would be able to completely save yourself the renumbering in the future. Furthermore, it would also be much easier to expand, and one would not always need to know the free IDs. I hope that Microsoft will take care of this issue in any case.

Unfortunately not. From a compiler perspective it could, but not without major changes to the runtime.
Specifically tables are problematic. For synchronization and upgrade we rely the ids for a specific table to be fixed.
Persisted RecordIds contains the table id, and that will break if you suddenly get a different id for the table.

@esbenk, I believe it could be possible. From outside we don't need the IDs and from inside, NAV can save the used IDs in a special table, for reuse after update by sync/upgrade. The identification of fields and tables can be done by Name and Type.
Another possibility is to add an unique ID e.g. a GUID to fields and objects. Both would not break something like persisted RecordIds, because the used (internally stored) ID is still the same.
The only question is, how to transfer the possible / licensed IDs to the NST? Add them to the app.json?

It will be problematic when I will want to rename existing filed. It will automatically create new one instead rename.

Is the NAVUUID just another Identifier just like the combination of type and ID we have today?
Anyway I still like the idea a lot because that could maybe help build for topics most of us will be struggling with that are not directly related to getting rid of IDs. Just some thoughts:

help identifying a connection between a label/caption in a translation file and its location in the code
help to identify if extension dependencies are actually really used. We could use the unique identifiers proposed to build a "database" of given structures and query if they are used more easily.

probably more...

Good thinking!
As an ISV who maintain a solution from NAV 2009 til AL, I work hard to have the same code in the same objects for easy maintenance.
So I need my AL file to still contain a reference to a numbered object for the c/side versions...

Long term plan. We are at the early namespace phase