As a user of the API I would like to be able to fetch all episodes from a podcast

Currently when consuming the episodes API, only a max of 40 episodes are returned if the max parameter is omitted.
one can provide a very large number to max (i.e. 10000), but it would also be good if a user of the API could provide the value 0 to the parameter max, and that would mean an unlimited number of episodes.
An alternative would be a better support for pagination:

the response would return a field with the total count of episodes
another parameter should be introduced as being the starting index

I've also notice if I set the max to higher than 100, the count is limited to the most recent 100 episodes. Maybe I'm doing something wrong, but how would I return the 1300+ episodes of "No Agenda" if I wanted them?

We only have what’s in the feed (and some of what has been recently). No Agenda doesn’t keep all of the episodes in the feed so we don’t have them all.
For the max value, I still need to go back and change that. Thanks for bumping this.

There seems to be a hard cap of 1000 episodes that is returned, even when the RSS feed contains all the episodes.
Entrepreneurs on Fire (id: 389136) for example has close to 2600 episodes in the RSS feed but the API responds with a max of 1000 episodes.
I tried playing with the since parameter, but as episodes are sorted by reverse chron I can't get to all over them.

Instead of increasing the limit a better solution would probably be some sort of paging or range request. Let me think on it.

Thanks!
Agreed, pagination or other way mechanism would be great.
A short term fix might be to have a "before" parameter as well as the "since" (so get x number of episodes before this date), I can loop around that to get all the episodes too.

Ok, this is on the to-do list.  I have some documentation to take care of on the namespace repository and then I'll circle back to this.

Are there further updates on this? I'd like to be able to paginate through a podcast's episodes, but the since parameter seems to be unhelpful, since the results are always returned in reverse chronological order.

Pagination work has been kicked down the road multiple times for multiple reasons, most of which come down to not having enough time to do it all.  We bumped the max episode count returned way up to compensate for the lack.  I'm assuming in your instance you are wanting to show small windows of episodes in an app that might be too constrained to pull the entire list at once?

I'm curious if this is being worked on yet. I just noticed today that getting more than the latest 1000 eps seems to be impossible. If there is a lot of work to do to enable pagination could you add a 'get all episodes' bool to requests like 'get podcast details by feed id' or bump up the max number to 2000 or 3000? that probably would cover almost all podcasts

This seems more or less essential for building a view that allows user to browse all episodes of a particular feed. Some kind of pagination is necessary.

Any update on this?


fix: Update Docker build to automatically adapt Torch version

Description

Build issue encountered in Docker arising from mismatched versions in WORKSPACE and Docker folder
Update Dockerfile build process to adapt Torch version from requirements.txt
Remove references to Torch version in Docker WORKSPACE file; instead use local Torch version

Type of change

Bug fix (non-breaking change which fixes an issue)

Checklist:

[ x ] My code follows the style guidelines of this project (You can use the linters)
[ x ] I have performed a self-review of my own code
[ x ] I have commented my code, particularly in hard-to-understand areas and hacks
[ x ] I have made corresponding changes to the documentation
[ x ] I have added tests to verify my fix or my feature
[ x ] New and existing unit tests pass locally with my changes
[ x ] I have added the relevant labels to my PR in so that relevant reviewers are notified

22.04 base upgrade and tensorrt/cuddn build args to be addressed in Issue #1782


feat: Validate ISO timestamps with offsets

I've added a bunch of tests with both valid and invalid timestamps and updated the JSDoc to list all the allowed formats.
The regex is relatively permissive as it allows for any number in the offset. If something stricter is required, I can work on it, but I'm afraid it would explode in complexity. For example, a 00:00 offset is valid only if preceded by a + and not by -; the maximum offset should be ±12:00, etc.
Closes #495.

Thank you for creating this PR. Since I am currently busy rewriting the library based on #463, it may take some time before I release the next version. So I don't recommend to wait and use the regex function instead or add a custom isoTimestamp action to your own codebase.

Sure, no problem! 😄

Would you like me to port this changes as part of #502?
This would technically be a breaking change, as inputs that would previously throw an error may now pass, so bundling it with other breaking changes might be appropriate.

Feel free to contribute to #502, but don't go crazy, because I don't want to spend too much time on new features. I want to get the next version out as soon as possible. Until v1, breaking changes are ok.

Thank you! Merged in #530.


Admin culture picker

The admin culture picker is not displaying anymore.

This is because you don't have a zone in your Layout corresponding to Header :
@Display(Model.Header)
and admin culture is injected there :
_workContext.Layout.Header.Add(Shape.AdminCultureSelector(SupportedCultures: supportedCultures));
There is no Footer zone either, in which we could put the OrchardVersion

To have a dropdownlist with the culture, the AdminCultureSelector.cshtml view needs to be restored in TheAdmin theme.

Or another way could be to make a picker rendered with flags of the corresponding countries.

I like having a list. It could be using a Bootstrap dropdown button and each culture could be a button item with an icon. We could switch the current culture flag on the button content.

Here is an example.

Fixed in https://github.com/OrchardCMS/Orchard/commit/7c8ab76ec80d3a9e4252af688d07d8859215bbba

Can be closed when you validate the display is correct.

after enabling the module and adding cultures I can see the culture picker.
Though I don't see any icons in the culture picker.

nano-content css fixed


Question: What is a low celling exactly

how tall shoud my house be if i have the elytrian origin ?

Your interiors should be at least 4 blocks tall.
https://origins.readthedocs.io/en/latest/misc/base_contents/origins/elytrian/

If you have less than two blocks space above you...

thanks


[SR-316] Add Thread Sanitizer Support to Swift

Previous ID
SR-316

Radar
None

Original Reporter
@AnnaZaks

Type
New Feature

Status
Closed

Resolution
Done

Additional Detail from JIRA

Votes
1

Component/s
Compiler

Labels
New Feature

Assignee
None

Priority
Medium

md5: f3761869c1e411173567839b180aa347

Issue Description:
Thread Sanitizer (TSan) is a compile time instrumentation + runtime monitoring technique for detecting threading issues. TSan's compile time instrumentation is done as an llvm transformation pass, and the runtime library is bundled with clang. We would need to teach the Swift driver/front end about TSan and add support for issue suppression, similarly to what clang does.

#2076, I suppose.


Sort features alphabetically

Good for now, I like to have some kind of featuring that allows us to push "hot" stuff on top.


Setting slider to scroll only when it reaches the right most slide

In this fiddle https://jsfiddle.net/craigkg/p9xntqdo/1/ I'd like for the scroll affect to only activate once .slick-current is the right most displayed slide rather than .slick-current always being the left most displayed slide. When the right arrow key is pressed, I want the slides to remain stationary while .slick-current moves to the next item, thus requiring the right arrow to be pressed 4 times before the scrolling reveals slide5. Is this possible?

this doesn't make much sense from a user's point of view...
why would they click "next" 4 times, with no actual effect ?

It is part of a navigational menu system. Each slide contains a different menu set that can be navigated up and down when that slide is the .slider-current slide, so using the right arrow goes to the next menu set to the right making it the new .slider-current. When the edge is reached, we want the slider to move one at a time to reveal the additional menu sets. Setting slidesToScroll: 4 makes .slider-current skip over three menus making only every 4th menu accessible. There is no "clicking" in this system as it has to be completely arrow key accessible.

@craigkg Seems to me you could accomplish that with slidesToScroll: 4, custom prev/next elements with an onClick counter triggering the slickNext and slickPrev menthos, and a little custom jquery to advance active states of the menu items. Hope that helps

Thanks for trying, but that won't work unfortunately. There are no prev/next elements. There is no mouse. There is no "click" so to speak. This is for a system that has 5 button navigation: four arrow directions and a select button. Each time the right arrow is pressed the next horizontal slide is selected. Within each horizontal slide is a category header and a vertical slider with the top slide/link selected. Up and down arrows navigate the vertical menu. When the user gets to the item they want, they hit the middle/fifth button, so there is no way to "click" any other element like prev and next. This is sort of like a grid system, but with each column's vertical navigation being independent of the others.

Interesting. Might that I'm not totally following but I think you can still accomplish what you want with custom jquery and methods, in a way similar to what I described, just swapping out the dummy next/prev with arrows assigning their events to the respective method. Good luck

@leggomuhgreggo has given you the mental-tools needed to get this done :)
Anyway, unfortunately we have a lot of feature requests which are useful across a wide range of use-cases. When you seem to have a completely unique system .. it's time to consider you need to write the code yourself :) -- hacking an existing thing to suit your very unusual use-case is not a good way to go about software development.
Best of luck, Simon

@craigkg wanted the same effect and this codepen https://codepen.io/dolce/pen/QarQde made my day


target_fixture='basket' doesn't seem to be working when introducing parametrize.

In Chapter 4 - Parametrizing Steps I seem to be having issues incorporating target_fixture='basket' to work alongside the changes.
Unsure if this is an issue, or because I am a total newbie and not quite grasped things all that well as yet.
I did have this working in the Chapter 3 Tutorials.
I am essentially stuck at 4 Mins 36 Seconds where you run the tests for the first time, are you able to confirm if this is an issue, or i am doing things incorrectly in the @given step.
Some variations I have tried:
@given(parsers.cfparse('the basket has "{initial:Number}" cucumbers', extra_types=dict(Number=int), target_fixture='basket'))  def basket(initial):     return CucumberBasket(initial_count=initial)
@given(parsers.cfparse('the basket has "{initial:Number}" cucumbers', target_fixture='basket' extra_types=dict(Number=int)))  def basket(initial):     return CucumberBasket(initial_count=initial)
After watching further into the video things change a bit, but the same doesn't seemingly work either
@given(parsers.cfparse('the basket has "{initial:Number}" cucumbers', target_fixture='basket', extra_types=EXTRA_TYPES)) def basket(initial):     return CucumberBasket(initial_count=initial)
@given(parsers.cfparse('the basket has "{initial:Number}" cucumbers', extra_types=EXTRA_TYPES, target_fixture='basket')) def basket(initial):     return CucumberBasket(initial_count=initial)

Hey @Robbie1478! I'm glad to see that you're trying things.
I think target_fixture is a parameter for the step attribute @given and not for the parser call. Try something like this:
@given(parsers.cfparse('the basket has "{initial:Number}" cucumbers', extra_types=dict(Number=int)), target_fixture='basket')
def basket(initial):
    return CucumberBasket(initial_count=initial)

@given should have two arguments: the parsers.cfparse call and the target_fixture value. Notice the placement of parentheses and commas.
Please give this a try and let me know if it works. If it does, then please close the issue. Thanks!

Hey @AndyLPK247 - thank you so much for the response 👍
Would seem I tried everything but your suggestion as I was trying to put everything within the existing brackets.
That worked a treat - thanks!
Would you have any issue with my looking to submit PR for this into your Chapter 4 Branch (I could potentially update Chapter 3 to include the target_fixutre too?
Of course - equally happy not to if your intention is for people to try figure it out for themselve first.

I'll take care of it.

@Robbie1478, everything should be updated and fixed now. I made some other improvements along the way.

Thank you @AndyLPK247 - that was a quick turn around.


[ZEPPELIN-5737] try to avoid deadlock

What is this PR for?
This pull request is to prevent a deadlock which occurs in connection with a fast cancel in the LazyOpenInterpreter.
This PR should also correct the closing of interpreters which are not LazyOpenInterpreter.
What type of PR is it?

Bug Fix

What is the Jira issue?

https://issues.apache.org/jira/browse/ZEPPELIN-5737

How should this be tested?

CI
manual

Questions:

Does the licenses files need to update? No
Is there breaking changes for older versions? No
Does this needs documentation? No

With manual tests, which previously created the deadlock, I can no longer create the deadlock with these changes. But maybe there are other things that don't work anymore because of the changes.

If no other comments are received, I will merge this pull request with the master on Thursday.


undefined reference to `arm_compute::graph::Graph::Graph()'

I built the library for Android, statically linked libarm_compute_core-static.a, libarm_compute-static.a and libam_compute_test_framework.a and still end up with the undefined reference to graph. All functions from previous release work well.

Graph symbols are in arm_compute_graph.
What are you building? Are you using the graph API? (You shouldn't need to link against it otherwise)

I am building the library as it is explained here: https://arm-software.github.io/ComputeLibrary/v17.09/index.xhtml#S3_3_1_library. I am trying to use graph API, but my application can't find graph symbols. I put using namespace arm_compute::graph; and using namespace arm_compute::graph_utils;, but when I try to do this: Graph graph;, I end up with the undefined reference.

There is a problem with the documentation: the graph is in a separate library


treewide: rename name to pname&version

Motivation for this change
#103997
Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.05 Release Notes (or backporting 21.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

@Stunkymonkey please target staging. Otherwise LGTM.

should i extract the single dependency that is causing the massive rebuilds? and make a separate PR? or simply rebasing is enough?


how to publis a android library to bintray  without .aar file

Hello bintray‘s developers:
I want to publish my android library to the maven repository of bintray,but the library has no resources just use some android API,so aar file is useless(Obsessive compulsive disorder ? ),I tried many times and I failed many times. can you provide an example gradle file in project? thank you very much.

@ikidou ,
Would you like an example that publishes a jar instead of an aar?
If so, did you try to modify the packaging inside the build.gradle in the android-maven-example from aar to jar?

Of course, I do the first step is to jar to jar, but to no avail.
I know the gradle-bintray-plugin will be upload files in

${projcet}/build/libs/*
${projcet}/build/poms/pom-default.xml
${projcet}/build/outputs/aar/${projcet}-release.aar . （Unless you delete it）

if tried again ,if replace the 'aar' to 'jar' , the 'packaging' node vaule of build/poms/pom-default.xml always 'aar'.
a part of my build.gradle
apply plugin: 'com.jfrog.bintray'
apply plugin: 'com.github.dcendents.android-maven'

install {
    repositories.mavenInstaller {
        pom.project {
            name ARTIFACTID
            description PROJCET_DESCRIPTION
            url HOME_URL
            inceptionYear '2015'
            packaging 'jar' // if the value is 'jar',the 'packaging' node can't find in pom-default.xml file,aar also uploaded
            groupId GROUPID
            artifactId ARTIFACTID
            licenses {
                license {
                    name 'The Apache Software License, Version 2.0'
                    url 'http://www.apache.org/licenses/LICENSE-2.0.txt'
                    distribution 'repo'
                }
            }
            scm {
                connection HOME_URL
                url GIT_URL
            }
            developers {
                developer {
                    name DEVELOPER_NAME
                    id DEVELOPER_ID
                    email DEVELOPER_EMAIL
                }
            }
        }
    }
}
Properties properties = new Properties()
properties.load(project.rootProject.file('local.properties').newDataInputStream())
bintray {
    user = properties.getProperty("bintrayUser")
    key = properties.getProperty("bintrayApiKey")
    configurations = ['archives']
    pkg {
        repo = 'maven'
        name = ARTIFACTID
        userOrg = user
        licenses = ['Apache-2.0']
        vcsUrl = GIT_URL
        websiteUrl = HOME_URL
        issueTrackerUrl = ISSUE_TRACKER_URL
    }
}
// doc/source/jar  ,if no this part ,only aar and pom file will be published
android.libraryVariants.all { variant ->
    def name = variant.buildType.name
    if (name.equals(com.android.builder.core.BuilderConstants.DEBUG)) {
        return;
    }
    def task = project.tasks.create "jar${name.capitalize()}", Jar
    task.dependsOn variant.javaCompile
    task.from variant.javaCompile.destinationDir
    artifacts.add('archives', task);
}

task sourcesJar(type: Jar) {
    from android.sourceSets.main.java.srcDirs
    classifier = 'sources'
}

task javadoc(type: Javadoc) {
    source = android.sourceSets.main.java.srcDirs
    classpath = files(project.android.getBootClasspath())
}

task javadocJar(type: Jar, dependsOn: javadoc) {
    classifier = 'javadoc'
    from javadoc.destinationDir
}
artifacts {
    archives javadocJar
    archives sourcesJar
}


aim through walls

Anyone know how make this?

what

what

just create your autowall and check damage with tracers o_o


Projects/VC2019: Convert CRLF to LF

These files should use LF like other source files.

Those are not source files but project files used by visual studio, no matter its internal format is XML
I tried Visual Studio 2019 and it accept flawless the projects. So, there is no issue here to fix.


Adapted Codeowners file

Signed-off-by: Johannes<EMAIL_ADDRESS>

Closing this as it was superseded by #221


Include more sources (RarBG, 1337x)

I think that the Pirate Bay is not that good in terms of quality. Plus it also has issues with malware. In my opinion both RarBG and 1337x are way better in these aspects. Could they be considered?

Flixerr already uses these sources alongside the Pirate Bay.


Creating a VirtualChassis with a master device changes the master device's vc_position

Environment

Python version: 3.6
Nautobot version: 1.0.0

Steps to Reproduce

Via the REST API or the ORM (vc_position is not directly settable via the Device create/edit web UI), create a Device and set its vc_position attribute to any value other than null or 1.
Via the REST API or the ORM (master is not directly settable via the VirtualChassis create web UI), create a new VirtualChassis and assign the aforementioned Device as the master of this VC.
Inspect the Device and/or VirtualChassis and observe that the master device now shows its position within the VirtualChassis as 1, regardless of what it was initially set to.

Expected Behavior
Device vc_position, if set to any non-null value, should be preserved when assigning this device as the master of a VirtualChassis.

Observed Behavior
Device vc_position is silently and unconditionally changed to 1.
This impacts data imports via the nautobot-netbox-importer plugin -- in the case where the VC master occupies any non-1 vc_position, and some other device occupies vc_position 1, a database integrity error is reported due to two Devices attempting to occupy the same vc_position on the same VirtualChassis. https://github.com/nautobot/nautobot-plugin-netbox-importer/pull/43 works around this, but I believe that this should be changed/fixed in Nautobot itself as well.
See also https://github.com/netbox-community/netbox/issues/2087; this appears to be a regression.

Code reference: https://github.com/nautobot/nautobot/blob/develop/nautobot/dcim/signals.py#L92

If you don't mind, I would like to take a stab at this on Monday.

Okay, I didn't wait till Monday. It ended up being a fairly straightforward fix.  I hope the test is in a good place.


fix(authz): Move logs containing subject mappings to trace level

resolves #1503

Thanks for this, @elizabethhealy. Is there capacity to check if this is really a warning/error case or something we should bypass entirely?
https://github.com/opentdf/platform/pull/1635/files#diff-a1a2a7e3fed8bf6e543f348e2130da8b24f6ceb39cddc98f0aabc1f17906f818R570-R573


Fix GetOrCreateSCM

Fixes #955

Codecov Report

Merging #961 (126c563) into master (6109200) will not change coverage.
The diff coverage is 0.00%.

@@           Coverage Diff           @@
##           master     #961   +/-   ##
=======================================
  Coverage   32.04%   32.04%
=======================================
  Files          94       94
  Lines        8871     8871
=======================================
  Hits         2843     2843
  Misses       5725     5725
  Partials      303      303

Flag
Coverage Δ

unittests
32.04% <0.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

scm/helper.go
30.76% <ø> (ø)

scm/scm_manager.go
9.61% <0.00%> (ø)

web/hooks/push.go
17.55% <0.00%> (ø)

web/quickfeed_service.go
34.22% <0.00%> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


403 on app_dev.php

I cant access to symfony.dev/app_dev.php, i get 403.
I'm on ubuntu and need to run docker with sudo (sudo docker-compose up) or i have:
"ERROR: Couldn't connect to Docker daemon at http+docker://localunixsocket - is it running?
If it's at a non-standard location, specify the URL with the DOCKER_HOST environment variable."
Unfortunately cache in prod env make me crazy
Can you help me?

You have having 2 different issues. One related to your docker host, and one to your symfony.
You cannot access app_dev.php because symfony has a security built-in. Open the app_dev.php and you should see some code which will block direct access from anything not localhost. Your pc (host) is not the localhost for the docker container. Which means you have to add your host ip-address, or just remove the security check.
Your other issue about the docker daemon is that your user is not in the docker group. Just add your user to the docker group and restart your shell (http://askubuntu.com/questions/477551/how-can-i-use-docker-without-sudo)
sudo usermod -aG docker $USER

Hi @leroy0211
yeah i made some digging and now i can access to app_dev.php ;)  At the moment i just set app_dev.php like app.php.
I didn't know about the user in the docker group. I will set it.
Thankyou!

Great!
Make sure you keep the next code in place! Otherwise you will load your prod configuration instead of the dev configuration.
$kernel = new AppKernel('dev', true);

@manuelDistefano one last thing, don't commit/push your app_dev.php in production 😉

Yeah i excluded app_dev.php from staging/production repository, i will use it only in my local env.
Tnx!


Enabling VirtIO installation is broken

The generator currently outputs the following XML when enabling virtio installation:
<RunSynchronousCommand wcm:action="add">
  <Order>24</Order>
  <Path>powershell.exe -NoProfile -Command "Get-Content -LiteralPath '%TEMP%\VirtIoGuestTools.ps1' -Raw | Invoke-Expression;"</Path>

<File path="%TEMP%\VirtIoGuestTools.ps1">
  <!-- etc. -->
</File>

On Windows startup a cmd.exe window pops open and asks for an Option "Path" (or similar, this is from the top of my head). The script itself is not run. I believe this is due to a problem with the extraction script, which doesn't properly handle the %TEMP% expansion. Manually rewriting the path to C:\Windows\Setup\Scripts\VirtIoGuestTools.ps1 fixes the problem.
Thanks for this great project!

Sorry, cannot reproduce. The extraction script uses the [System.Environment]::ExpandEnvironmentVariables method to expand environment variables. I don't have a Proxmox environment available right now, but using your exact autounattend.xml file on a VMware Windows 10 machine still executes the VirtIoGuestTools.ps1 script, which outputs  a single line to the %TEMP%\VirtIoGuestTools.log file, as expected:

VirtIO Guest Tools image (virtio-win-*.iso) is not attached to this VM.

Note that the logfile's absolute path is C:\Windows\Temp\VirtIoGuestTools.log, not C:\Users\Admin\AppData\Local\Temp, and that you need administrative permissions to read it. Does the logfile file exist on your machine?
I admit that using the %TEMP% variable might unnecessarily complicate things while having little benefit, so I will consider replacing it with a constant C:\Windows\Temp or C:\Windows\Setup\Scripts path.

I have finally moved all script and log files from C:\Windows\Temp to C:\Windows\Setup\Scripts. This places all relevant files used by the unattended process in a single location, hopefully making debugging easier.
Note that installers run during Windows Setup might still output their logs to C:\Windows\Temp (in the same way they output their logs to a user's %TEMP% directory when run later by that user).


Fix For Messaging Icon

Summary
This will fix the missing delete icon
https://github.com/syuilo/misskey/issues/5932

Thank you!


fix the cache interface documentation

The previous comments were out of date, and refer to variables that
have since been renamed, and to older return types.

Applied a correction to the Get comment (how to correctly recognise when the item was not found).

thanks!


Fixed #35899 -- Allowed multiline template tags.

Trac ticket number
ticket-35899
Branch description
Option to support newlines inside {% %} in templates. As discussed https://forum.djangoproject.com/t/allow-newlines-inside-tags/36040/23
Checklist

[x] This PR targets the main branch.
[ ] The commit message is written in past tense, mentions the ticket number, and ends with a period.
[x] I have checked the "Has patch" ticket flag in the Trac system.
[x] I have added or updated relevant tests.
[ ] I have added or updated relevant docs, including release notes if applicable.
[ ] I have attached screenshots in both light and dark modes for any UI changes.

Nice addition.
I think we need to think hard about how to minimise the backwards compatibility issue. A template option that will flip default is going to lead into a hard break… — there seemed to be various ideas on the forum thread to explore. 🤔
I suspect unmatched {% &co are rare... — Ideally we'd opt folks into the new behaviour by default, as we've done with other changes like the move to ZoneInfo, with a fallback and an appropriate warning if we can for the outlying cases (an optional check maybe?). (Making the newness opt-in seems backwards.)

A template option that will flip default is going to lead into a hard break…

Yea. I ruled that out for that reason too. This PR does not do that.

I suspect unmatched {% &co are rare... — Ideally we'd opt folks into the new behaviour by default, as we've done with other changes like the move to ZoneInfo, with a fallback and an appropriate warning if we can for the outlying cases (an optional check maybe?).

This is what this PR does no? Maybe I misunderstand what you mean.

(Making the newness opt-in seems backwards.)

This I don't understand. Opting in to the new behavior is the simplest solution for backwards compatibility, and for a gradual and controlled upgrade path for existing code. This is the method used by Django before. startapp is still default opting in apps to migrations 10 years after migrations were introduced, as an example.

I thought about the idea for a tool to check for problems with this. Can it be as simple as running the old and the new regex and counting number of hits, and doing that for every template?

@boxed That sounds like it could work. (Better than my grep half-idea...)
My query is about the UX of any check we add to wrap that. Should it run by default, and then I just silence it if I'm happy to embrace multiline tags? (Likely, I think, otherwise folks will not run it, and get caught out, but it's a small tweak for the majority nonetheless.)
Not sure if others have better ideas for a smooth transition?

We could run it as a check on startup in the runserver, but it would be best if we only do this once and not on every startup. Not sure what a reasonable way to do that is.

Well, folks can silence system checks, which would be OK, I guess. (If we can't think of something better in the meantime) --- I'd begin prototyping that, then we can move it if needs be on review.

@boxed Hi! Could you please fix the test errors and lint issues, so we provide an initial review?
Once that's done, please update the ticket flags accordingly. Thank you!

@tim-mccurrach has an excellent point that the change of this regex hits not just {% %} but also {# $} and {{ }}. I think this is actually a very good thing, but it's also another strong argument to make this op-in. @carltongibson are you convinced by this argument or do you still think it's worth the risk?

If the implementation is risky, we shouldn't merge it.
If it's not risky then having multiline tags is a good feature that most (you guess the nines) users will benefit from, and it seems that making it the default is the right way in that case.
Mine is just one view. I've said what I think several times now. I don't really want to just keep restating it. 🤷

You haven't said anything about merging or not merging, and that's not what is being discussed. Furthermore there's been some new information. Normally when new information comes to light, people need to rethink their opinion, so I don't see why you're prickly about me asking again when the situation is changed.

You haven't said anything about merging or not merging, and that's not what is being discussed. Furthermore there's been some new information. Normally when new information comes to light, people need to rethink their opinion, so I don't see why you're prickly about me asking again when the situation is changed.

@boxed I find this comment both disturbing and inappropriate, and I believe it does not align with the expectations set out in the Django Code of Conduct. It's important that we all maintain a respectful and constructive tone, at all times, even if you feel frustrated or unanswered.
If this is the first time you're being reminded about this, please consider this a formal warning. If not, I kindly ask you to take a moment to review the Django Code of Conduct and always ensure that your comments are friendly, considerate, and respectful.

I have updated the template for new projects, and added docs. I think that's what was left for completing this ticket.

I have not written anything in the release notes as I am unsure where to add this. 5.2? 5.1.4?

Ping. There is only a question about where to write into release notes left before merge I think.


UncertainArbDiscDataset missing from master branch

@kevinmilner,
The master branch won't compile because org.opensha.commons.data.function.UncertainArbDiscDataset is missing.  Could you restore it?
Thanks.

@kevinmilner
Sorry, it's actually opensha-oaf that fails to compile because it contains references to UncertainArbDiscDataset.  Still, is it OK to restore UncertainArbDiscDataset?

Hi @mbarall, that class was moved when #53 was merged in. I committed a change to opensha-oaf at that time to point to the new location. Pull the opensha-oaf project to get it and things should compile:
https://github.com/opensha/opensha-oaf/commit/55cfb901e6a54e43c3e82f0a0c6d848ba2e7b8dc
Let me know if you run into any issues!

Hi @kevinmilner
That fixed the problem.  Thanks.


Add job duration time metrics

What does this PR do?
Added the execution time of job executed on kubernetes as a metric
Motivation
By acquiring the job execution time, it can be used in various use cases such as execution time monitoring and alert settings.
Additional Notes
Generating metrics with the following calculations
kube_job_status_completion_time - kube_job_status_start_time = execution_time
Calculated on integration for the following two reasons

Since kube_job_status_completion_time and kube_job_status_start_time are unixtime, I decided that they would not work on the monitor as they are.
If calculated on the graph, only kube_job_status_start_time will be sent if the job is running, so it will be negative values when calculated.

Review checklist (to be filled by reviewers)

[ ] PR title must be written as a CHANGELOG entry (see why)
[ ] Files changes must correspond to the primary purpose of the PR as described in the title (small unrelated changes should have their own PR)
[ ] PR must have changelog/ and integration/ labels attached
[ ] Feature or bugfix must have tests
[ ] Git history must be clean
[ ] If PR adds a configuration option, it must be added to the configuration file.

Thank you for the many comments! I'm not very familiar with Python, so it's very helpful

The naming changed because the official used Duration instead of Execute.
https://github.com/kubernetes/kubernetes/blob/master/pkg/kubectl/describe/versioned/describe.go#L2079-L2080
NAME                                              COMPLETIONS   DURATION   AGE
example-cronjob-1560267000                       0/1           19d        19d
example-cronjob-1561735800                       1/1           15s        2d15h
example-cronjob-1561822200                       1/1           22m        39h
example-cronjob-1561908600                       1/1           17m        15h

Hi @int-tt
I think it would make more sense to create a counter returning the current execution time instead of sending only the duration when a job is "complete".
Thanks to this approach, it is possible to create an alert on the execution_duration event if the job is not yet finished.
Then when the job is finished, the execution_duration value will persist until the job is deleted.
Let me know what to you think about it?

Hi @int-tt, Thank you for your contribution and participation in the Datadog community.  Did you hav any thoughts on @clamoriniere 's feedback?

Hi @int-tt, I'm closing this PR for now since it's quite stale now and has conflicting files. If you still want to get your changes merged, you can reopen this PR and fix the merge conflicts.


Edit descriptions in modal

Scope
Currently, when a description or a relationship needs editing, RPG Manager shows the RpgManagerData codeblock for the users to edit the text
Current Problems
As the codeblocks must be valid YAML, inviting users a direct edit in the data poses the risk for the YAML to be incorrectly edited.
Proposed Solution
Create a modal form that allows users to edit the content of a description, saving it in the RpgManagerData codeblock on behalf of the user. This would allow a data sanitisation before writing the file, hence eliminating the risk of YAML errors.
Challenges
Editing the codeblock directly gives users access to the note autocompletion. This autocompletion is embedded in the Obsidian Editor. Unfortunately the editor cannot be created in third-party elements such as a modal form.
To overcome this issue, without eliminating the required auto-completion, there is the need to implement the auto-completion in a different way, mimicking the behaviour of the main Obsidian Editor

Additional information
Obsidian uses a div called suggestion-container at the bottom of the Editor body. There is the possibility to mimic its functionality to reuse the styling component, triggered by some specific events.


Compilation error with swr 0.5.0

I'm submitting a ...
[x] bug report
[ ] feature request
[ ] question about the decisions made in the repository
[ ] question about how to use this project

Summary

Upgrading swr to 0.5.0 causes a TypeScript error in the files generated by graphql-codegen-plugin-typescript-swr. ConfigInterface and keyInterface have also been marked deprecated.

Other information (e.g. detailed explanation, stack traces, related issues, suggestions how to fix, links for us to have context, eg. StackOverflow, personal fork, etc.)

Could you try to update swr to 1.0.x? If you still get error, please create new issue. Thank you.


Added GPT in Chat bot

Issue No. #424

Description
Hello @SahityaRoy ,
I have added Chat Bot using API of Chat GPT v3.5
It is working smoothly and answering all the questions.
Instructions -

Visit to this website - https://rapidapi.com/Glavier/api/chatgpt53
Subscribe the API // important step without this API will not work, also this API is free of cost.
See the right side of the page :-

Copy the API key and paste in the key in the chatbot/script.js

Video -
Screencast from 27-06-23 10:31:02 PM IST.webm

Checklist

[x] My code adheres to the established style guidelines of the project.
[x] I have included comments in areas that may be difficult to understand.
[x] My changes have not introduced any new warnings.
[x] I have conducted a self-review of my code.

@SahityaRoy, Kindly enter the API Key 👍🏻

@SahityaRoy  He is not a part of GSSoC 23 and he is non of the character of this program. He  is just a spammer who wants to loose my ranking.

@Killer2-O same goes to you Killer2OP

I know everything what's going on in GSSoC23 don't worry. I will work in silence now. I don't want any controversy with any one. Bcz I know, how are you.


Get Media by userid

Instead of fetching media via username, Is it possible via userid?
$instagram->getMedias('username', 10);

Right now, it's not possible. But you can try to create your custom query through graphQL, something like this:
$response = Request::get(Endpoints::getGraphQlUrl('YOUR_QUERY_ID', ['id' => 'USER_ID']));
if ($response->code !== 200) {
    throw new InstagramException('Response code is ' . $response->code . '. Body: ' . static::getErrorBody($response->body) . ' Something went wrong. Please report issue.');
}
$jsonData = json_decode($response->raw_body, true);

Yes, you could for example
$account = $instagram->getAccountById('3');
and then
$instagram->getMediasByUsername($account->getUsername())


Kitchen Sink - video does not play on the preview (web)

https://studio.ensembleui.com/app/e24402cb-75e2-404c-866c-29e6c3dd7992/screen/fce92bbb-af8e-403d-bf2d-c10926cc89a0
works fine on iOS

Fixed


feat: Add service key to HTTP Server (#679)

Add service key to http server.

Please also pick this fix
https://github.com/edgexfoundry/go-mod-bootstrap/pull/716


flake8 adherence

The codebase currently doesn't include formatting or styling standards, do you mind if we use flake8? In general it looks like you have stuck to 120 characters max and the usual flake8 standards, it is just never formally declared.
I can create a new PR for this:

[ ] add .flake8 config
[ ] fix all .flake8 errors or add noqa where sensical
[ ] add flake8 testing to .travis.yml builds
[ ] update documentation about using flake8
[ ] add flake8 to requirements documentation
[ ] integrate with coveralls or codecov and add a badge to README

Personally I use a lot of inspections in PyCharm to adhere to coding style, however, making it explicit does not seem like a bad idea (actually, a pretty good idea).

My personal plugins, in case it helps:
flake8
flake8-docstrings
flake8-import-order
flake8-tidy-imports
pep8-naming

My setup.cfg includes:
[flake8]
import-order-style = smarkets

Coala might be a good fit https://github.com/coala/coala


Method "init_by_array" is never used.

Hey!
I've noticed that init_by_array method is not used at all.
What's the purpose of keeping that method with such a complex logic?

There is also few other not used methods related to the Mersenne Twister. I believe removing them will reduce the size of package for a little.

Good call!
The Mersenne Twister was just copy/pasted in from another open source project hence its unused vestigial bits like that.

Yeah, I've got it. I'm just saying that unused parts may be deleted to reduce the size of the package.
I leave this for your consideration.
While the issue is not really an issue you may close it.


Error code: 509, Bandwidth transfer

Bug description
I am trying to download a large area in San Jose: https://www.openhistoricalmap.org/#map=15/37.3343/-121.8866. The website has a limit on the allowed download size. https://github.com/OpenHistoricalMap/ohm-deploy/blob/main/images/web/config/settings.yml#L27
# The maximum area you're allowed to request, in square degrees
max_request_area: 0.25

0.25 square degrees is approximately 3,080.25 square kilometers or 1,188.74 square miles.
The area I am trying to download is within the allowed size, but it seems there is so much data that I am reaching the maximum bandwidth transfer in San jose.
Repro Steps
•	Use JOSM to download the data.
•	Select an area around San Jose.

FYI, in case you want to download a large area, we can use Overpass API, but this might not be recommended for editing. There may be changes in the area that have not yet been applied in Overpass.

This was also reported in Slack a few weeks ago. It’s even worse in iD, which basically stops working: openstreetmap/iD#10299.
Personally, I never edit at zoom level 15 or 16 or even 17, especially in a dense area, so I wouldn’t mind reducing the allowable download area. But I understand why some users would attempt to, since most areas aren’t so dense and we still have some very large features to map.

max_request_area: 0.25

That's the wrong paramter to avoid HTTP 509. max_request_area only controls the maximum size of the area you can download. What you experience here is rate limiting on CGImap side, that's the amount of data in MB you can download in a given amount of time.
You can bump the values there if needed:
Relevant parameters:

"ratelimit" : average number of bytes/s to allow each client
"moderator-ratelimit" : average number of bytes/s to allow each moderator
"maxdebt": maximum debt (in Mb) to allow each client before rate limiting
"moderator-maxdebt": maximum debt (in Mb) to allow each moderator before rate limiting

I have increased the values for cgimap, and I am now able to download large amounts of data without a rate limit. The new settings are as follows:
# Average number of bytes/s to allow each client, 0.5 MB per second
export CGIMAP_RATELIMIT="524288"
export CGIMAP_MAXDEBT="2048"

# Maximum debt in MB to allow each client before rate limiting
export CGIMAP_MODERATOR_RATELIMIT="1048576"
export CGIMAP_MODERATOR_MAXDEBT="4096"

I have downloaded large area around San Jose.

currently this is in staging.

This is been already deployed in production! clossing the ticket


get rid of get_global_connection

Link the connection object to the TaskManager,
Make the TaskManager extensible (inheritance or, preferably, composition) so that someone could implement their own method of connecting to the DB (maybe from Django), with a default implementation that would take no argument and read from env vars, like currently
Standardize whether functions receive the task manager, the connection or the cursor as parameter. Find the right one and use it consistently where applicable.
Use consistent connection options and cursor classes.

I am willing to give it a shot :-)

Nice ! No idea why, but I can't assign it to you. Feel free to do it :)

I can't assign it to myself either :-(


Rule to detect duplicate function definitions.

We have the rule PIE794 which detects duplicate class field definitions. I think it would be nice to have a rule which detects duplicate function definitions:
# Error
class Foo:

    def bar():
        print("hello world")

    def bar():
        print("hello ruff")

This is clearly a mistake and would be great if it was caught by ruff.

Have you tried F811? It flags the second declaration of bar https://play.ruff.rs/f029b8b6-26bd-49f9-9da2-ac2b22685b0b

For function in file scope the duplicated function is found by both pylint E0102 and mypy no-redef, example:
    """banana"""

def banana():
    """banana"""

And in the class method duplication case:
lass BananaTree:
   """BananTree"""

   def banana(self):
       """banana"""

   def banana(self):
       """banana"""

Shows these errors:
pylint: [function-redefined] method already defined line 4 [E0102]
pyflakes: redefinition of unused 'banana' from line 4

According to https://github.com/astral-sh/ruff/issues/970 Pylint rule E0102 is covered by F811 that @MichaReiser mentioned.
That info is missing but should be added to documentation at https://docs.astral.sh/ruff/rules/#pylint-pl

F811 is however called redefined-while-unused and only triggers when the first definition has not been called. This means that F811 is easily missed for exampled in this example:
def banana():
    """banana"""

def banana():
    """banana"""

Give no errors from pyflakes at least. Since the first definition is used, it doesn't report the redefinition as an error. I'm not sure what ruff does in this case.

I see why this isn't being picked up for us, we have private variables, which it doesn't work for
https://play.ruff.rs/ee3c213f-21d7-4540-a052-9acbc6ac4d6f

Why should the second definition be flagged here?
def banana():
    """banana"""

banan()

def banana():
    """banana"""

Yeah, that's due to dummy-variable-rgx. I wonder if that shouldn't be applied to functions. I'm not really sure. It's possible that users would find that really surprising.

Why should the second definition be flagged here?
def banana():
    """banana"""

banana()

def banana():
    """banana"""

This seems valid to me.

I didn't state any opinion about the rule, but E0102 does trigger on this code.
Opinion: Sure this could be a valid case, but it's probably quite rare to redefine functions like this. I can see programmers prefer to have a warning for this. If one doesn't want the warning it can be turned off.


Add additional property config builder methods

The API and implementation to configure resource properties has not yet reached the targeted quality. There are too many boilerplate classes, inconsistencies and usage burdens.
Necessary key changes:

Instead of passing a simple (e.g. boolean) parameter through multiple methods and documenting its purpose/background every time, use an enum instead and centralize the corresponding documentation inside it.
Remove the boilerplate *BehaviorFactory classes and use static methods in their corresponding Behavior classes instead to create the factory instances.
Simplify the usage of configured behaviors; instead of distinguishing between behaviors coupled to a property (only executed if that property is present in the request) simply always execute all behaviors. Each behavior instance can decide by itself if it should execute its behavior depending on the given request data. This solves the problem of property specific fallback behaviors for creation requests.
Rename/deprecate/add methods for better API consistency.

addCreationBehavior() still needs to be adjusted to take a behavior factory that applies constructor and post-constructor behavior. Other methods can be deprecated then.


feat: generate Docker image SBOM

Summary
Use the docker sbom command to generate a CycloneDX
format BOM.
Optionally support logging into an AWS ECR first so that
the image can be pulled.
Related

cds-snc/platform-sre-security-support#94

Do we want to deprecate access keys and instead have teams setup OIDC to this repo that has read access ?

I'm game to set that up - be a good exercise to get that rolled out across the projects


CSS Fix for dcc.DatePickerSingle()

Hi,
the current CSS downloaded from this repo (at least FLATLY, but I'm tempted to believe this applies to all templates) is not compatible to the structure of the aforementioned component and applies a wrong styling, most notably, font-size and width that happen hardcoded inline (I guess due to how the component itself is built?).
Here's also the link to a similar discussion in the Dash forum.
I fixed it overriding these parameters on my app, and managed to make it look in line with the rest of the styling. Especially, font size and width, that are now based on root size and parent container respectively.
It's probably not a clean solution, I apologize in advance, but I'm pretty new to development so most stuff is still a bit shady to me, but it should be clear enough to anyone reading where the mistake is and how to fix it properly.
/* Set the current outer border to 0 because it's messed up,
and put the FLATLY properties inside the DateInput_input class
which works well instead. */
.SingleDatePickerInput {
  border: 0px;
}

/* Fix box width of datepicker to be 100% of parent container*/
.DateInput, .DateInput_1 {
  width: 100%; /* replace with whatever length you prefer */
}

/* Fix the font size and other stuff of datepicker*/
.DateInput_input, .DateInput_input_1 {
  /* my personal stuff */
  text-align: center;
  line-height: calc(1.5em + 0.75rem);
  /* default flatly stuff */
  height: calc(1.5em + 0.75rem + 2px);
  font-size: 0.9375rem;
  font-weight: 400;
  color: #7b8a8b;
  background-color: #fff;
  background-clip: padding-box;
  border: 1px solid #ced4da;
  border-radius: 0.25rem;
  transition: border-color 0.15s ease-in-out, box-shadow 0.15s ease-in-out;
}

Hey @wtfzambo
Thanks for raising this. Yeah, the structure of DatePicker* components changed in Dash v1 (I think it was v1, it was around that time anyway), and I never got around to updating this repo. I had actually done some work on new stylesheets (which are sitting on the css-updates branch on the dash-bootstrap-components repo) but I never pushed them here.
I just opened #8 to address this, can you try downloading the rebuilt CSS from that branch? Here's flatly specifically and let me know if that works for you? I did some testing at this end, but not extensive stuff.
From memory, the changes I made to the DatePicker* components were pretty ready to go, but I got a bit lost trying to restyle the Dropdown component and then life got in the way. Anyway, let me know how you get on with my updates.

So, I tested the new CSS. The border and the font size are fixed on my side, font color is coming from I'm not sure where but is not aligned with the dropdown color, for example, however width is still hard coded and doesn't resize. Screen for reference:

As you can see, the box that contains the date goes well over the width of its parent container. This is a property that I fixed in the class .DateInput, .DateInput_1 (either is good), by setting width: 100%.

Thanks for trying it out. Can you share some code and I'll take a closer look?
Also, are you liking to other stylesheets in addition to the one downloaded from this repo?

@wtfzambo I had a chance to try out your CSS (thanks for including it btw). It works pretty well for DatePickerSingle and Flatly, but causes problems with DatePickerRange and other themes, so it's a little hard to incorporate more generally.
The motivation for having a fixed width is really just that that's what dash-core-components does, and my instinct is to generally try and preserve behaviour as much as possible so that people know what to expect. I'll think some more about how to make things more flexible though.

Yes, of course the CSS I created is tailored for the component I'm using, I didn't test it at all on Range.
Actually, I'm using two single datepickers AS a daterange because I wasn't able to customize some behaviour with range, that I could instead do with single pickers.
As for what stylesheets, I'm just using FLATLY downloaded from this repo and a small override.css that I used to fix the datepicker aspect and a couple of irrelevant aspects (colors and some margin on container), plus the styling for the shadowed box, that's it. It's quite basic.
About the width, it just made more sense to me to have it dynamic, given that I'm using mostly DBC and every component there has dynamic size. Also, for smaller screens, the layout I made would turn out ugly with a fixed width, which is unacceptable to me.
If I get some time, which is probably never, I'll try to come up with some CSS styling that addresses both single and range picker.

About the width, it just made more sense to me to have it dynamic, given that I'm using mostly DBC and every component there has dynamic size.

That's a very fair point, perhaps I should priorities being consistent with Bootstrap rather than consistent with Dash components. I'll tinker around a bit more and see what I come up with.

PS: what code did you want me to share? The CSS or the layout?

I was thinking the layout itself, but I think I have a good idea now of what the issue is and what you're trying to do, so don't worry about it if you're short on time.

Made some changes, not tested extensively yet, but I think I've got the behaviour you want + sensible behaviour for DatePickerRange

Feel free to point me towards the updated file when you think it's ready so I can test them on my app.

I've been pushing everything to the dash-v1-updates branch. I think the latest updates should accomplish a lot of what you want. DatePickerRange is reasonably well supported too, there appears to be a problem with placement of the fang for the second input if the date picker is too wide, but that's a subtle enough problem that I'll leave it for now.
Going to add some updates for Dropdown too before merging.

I've made some improvements to the styles for dcc.Dropdown too and merged to master. Please do try it out and let me know if you have further issues.

Yep, I guess it's perfect now. Width updates, text is centered and font size is inline with the other elements. I'm not sure about what you changed in the dropdown menu instead.
Anyway, here's the screenshot with the updated style. The only thing I would change, but that's my personal preference, is the font color, to be the same as the one within the dropdown (which is lighter w.r.t this one).

Great, glad to hear it!
Good catch on the font colors, you have a sharp eye 👀. I will take a look.

Actually if you check my CSS it's already there.

I just pushed some changes (PR #10) that should address this, it's in master already. Check it out and let me know.

Hi, sorry for the long hiatus. I got stuck into different work stuff and had 0 time in the last month to go back to my dashboard, I'm really sorry.
So, I managed to test the new .css now. It seems that the color of the text in datepicker doesn't change tho:

The color of the text here is #484848, which doesn't appear anywhere in my code nor the CSS, so I guess it takes the browser's default. I see that in the new merge you added the color to .DateInput > input::placeholder. In my specific case the date you see isn't a placeholder, but an actual selected date.
I fixed it by adding the color property also to the .DateInput_input class, right below it (row 9728).
I hope this is somewhat helpful in any way.

Hey @wtfzambo
No worries for the delay, I know how easily other things can come up and get in the way 🙂
Thanks for testing it out. I will take a look into the font colors, I thought I had accounted for that but clearly not.

Hi there, I am pretty new to dash and wondering if there was a fix/workaround the DatePickerSingle input font color?

Apologies for very slow response, for various reasons I didn't have much time for this project for the last several months.
In the testing I've been doing the date picker font colour is being set correctly as far as I can tell. Since this issue is quite old and the original issue that it was opened for has been resolved, I'm going to close it, but please do open up a new issue regarding date picker font colours if you're continuing to have issues. I should have a bit more time to look into it in more detail now.

Apologies for very slow response, for various reasons I didn't have much time for this project for the last several months.
In the testing I've been doing the date picker font colour is being set correctly as far as I can tell. Since this issue is quite old and the original issue that it was opened for has been resolved, I'm going to close it, but please do open up a new issue regarding date picker font colours if you're continuing to have issues. I should have a bit more time to look into it in more detail now.


Commented brackets are colored when writing JS in HTML

Example below:
// Array.prototype.sort()
/* const oldestToYoungest = inventors.sort(function(personA, personB)) */

While writing Javascript inside of an HTML script tag, brackets are still colored in comments. Both // and /**/ comments are affected. This does not happen in Javascript files.

that's not only case where it shouldn't colorize, it also colorizes when brackets, parentesis and curly braces are inside of a string and that literally breaks the entire color matching of a file
example:

same problem but writing JS in php files

Same issue:

I have this problem and it confusing because you don't know if the code is broken or not. Any type of commented out brackets should not be colourised.

same problem in html, hope it can be solved.

I also have this problem with PHP


this plugin is didn't work

i can't make it work

Can you provide a usage scenario or some error screen capture？Thx

Dose not work for "vuepress": "^2.0.0-beta.60"

@sweihub You can try to use vuepress-plugin-mermaid-wrapper

solved the issue with PR#5

@sweihub You can try to use vuepress-plugin-mermaid-wrapper

The memaid-wrapper not working with "vuepress": "^2.0.0-beta.60" as well, thanks anyway, I created a blank VuePress 2 project, it failes to load the plugin.

solved the issue with PR#5

Thanks, did you push to the npm repository, I still cannot get it to work for VuePress 2

"vuepress-plugin-mermaid-next": "^0.1.14"
"vuepress": "^2.0.0-beta.60"

Uncaught SyntaxError: The requested module '/@fs/home/swei/docs2/node_modules/moment-mini/moment.min.js?v=51b21672' does not provide an export named 'default' (at mermaid.core.mjs?v=51b21672:7:8)


docker client times out pulling large fs layer from S3-backed registry

At Altiscale, we have been running a standalone docker registry backed by S3.  The registry works fine with STORAGE_REDIRECT=true.  We're tightening up registry security and bandwidth usage by setting STORAGE_REDIRECT=false, but our docker clients now time out when fetching large fs layers.
Here's the scenario:

On the docker client, pull the image with the large fs layer...

[docker_client_host]$ docker pull registry reghost.altiscale.com/container_name
Pulling repository reghost.altiscale.com/container_name
921a3438ca3d: Pulling image (latest) from reghost.altiscale.com/container_name
921a3438ca3d: Pulling image (latest) from reghost.altiscale.com/container_name, endpoint: https://reghost.altiscale.com/v1/
921a3438ca3d: Pulling metadata
921a3438ca3d: Pulling fs layer

The docker registry receives a request for the layer.
This code in depends/docker-registry-core/docker_registry/core/boto.py:Base.stream_read decides to fetch the layer using the ParallelKey class.

        if not bytes_range and key.size > 1024 * 1024:
            # Use the parallel key only if the key size is > 1MB
            # And if bytes_range is not enabled (since ParallelKey is already
            # using bytes range)
            key = ParallelKey(key)

This code in depends/docker-registry-core/docker_registry/core/boto.py:ParallelKey.__init__ opens a temporary file:

self._tmpfile = tempfile.NamedTemporaryFile(mode='rb')

In 5 (CONCURRENCY) threads, this code in depends/docker-registry-core/docker_registry/core/boto.py:ParallelKey._fetch_part starts transferring 1/5 of the object at the corresponding offset in the temporary file:

with open(fname, 'wb') as f:
            f.seek(min_cur)
            brange = 'bytes={0}-{1}'.format(min_cur, max_cur)
            boto_key.get_contents_to_file(f, headers={'Range': brange})
            boto_key.close()

The fs layer object in S3 is so large that the docker client times out while these 5 threads are running, and retries.
The above process repeats: opens a second temporary file, spawns 5 threads, and the 5 new threads start transferring the object into the second temporary file.  At this point, 10 threads are running, transferring into two different temporary files.
Eventually one of the first 5 threads completes the call to boto_key.get_contents_to_file and calls ParallelKey._refresh_max_completed_byte, which emits the message buffering complete at 20.0% of the total transfer; now serving straight from the tempfile.  At this point, the client has already timed out on its original request, so the initial fetch of the S3 object into the temporary file was not useful work.
This process repeats: each time the threads complete transferring the object, the docker client has already timed out.
After two tries, the docker client emits this error after all of its retries fail:

921a3438ca3d: Error pulling image (latest) from reghost.altiscale.com/container_name, Get https://reghost.altiscale.com/v1/images/921a3438ca3d58320b54f1117a992b5ce2c2f7305ddb354250364a19fb3d37e6/layer: read tcp ip_address_of_reghost:443: i/o timeout

I have confirmed this same behavior using curl directly on the docker registry host.  This curl command demonstrates the same behavior, except that curl doesn't time out so that the initial download eventually works.  When running the curl command, curl does not download the first byte of the file until the first of the 5 docker registry threads calls ParallelKey._refresh_max_completed_byte.  Once the first thread completes, curl starts downloading the layer.
[reghost.altiscale.com]$ curl -v -O http://localhost:5000/v1/images/921a3438ca3d58320b54f1117a992b5ce2c2f7305ddb354250364a19fb3d37e6/layer

This curl command uses the Range header to disable ParallelKey.  This header makes the if not bytes_range conditional in the code above false, so that ParallelKey does not run.  In this case, curl starts downloading bytes immediately.
[reghost.altiscale.com]$  curl -v -O -H 'Range: bytes=0-' http://localhost:5000/v1/images/921a3438ca3d58320b54f1117a992b5ce2c2f7305ddb354250364a19fb3d37e6/layer

Here are some potential solutions for the problem that I would be happy to contribute as code:

Modify boto.py:ParallelKey._fetch_part so that it limits the size of any individual transfer,  and therefore calls ParallelKey._refresh_max_completed_byte faster.  This change would be pretty easy, but each thread would still work on its own segment of the file.  The result is that the streaming transfer of the client would be rate limited by single thread transfer performance.
Modify boto.py:ParallelKey so that each thread picks up a new range of bytes to transfer as close to the beginning of the object as possible, but not overlapping with the ranges being transferred by other threads.  This change would be more substantial (and harder to get right), but the all of the threads would contribute to the rate that the client would be able to transfer.
Modify the docker-registry configuration code to allow the operator to disable ParallelKey entirely, or if an S3 object is over a configured size.
Remove the ParallelKey code from boto.py.
Provide the ability to configure the docker client with a longer timeout.  This approach would eliminate the need to modify docker-registry code, but does not improve performance for the client by allowing it to do a streaming transfer in parallel with the docker registry's transfer from S3.
Provide the ability for the docker client to disable ParallelKey by passing the Range HTTP header to the docker registry.  This approach would work with the current docker registry code, but seems like it's too much of a hack because it depends on the internal implementation of boto.py:Base.stream_read.

Any thoughts on the best way to proceed?
...or maybe I missed some obvious way to fix this issue?  Please let me know if there is some obvious way to get a standalone registry working with STORAGE_REDIRECT=false and large fs layers stored as S3 objects!

Ok, that sounds bad enough.
Given the effort is now on https://github.com/docker/distribution (new golang implementation), little effort is going to be put in the python-registry here.
My suggestion is either remove the parallel key code entirely, or better, make it configurable (and disable it by default).
If you are willing to PR this, that would be awesome...

Thanks for the quick response, Olivier!  I also noticed issues with tmp files not being cleaned up, and saw that others had already reported similar problems (like issue #954).  It seems like the best thing to do at this point is to simplify the code by removing the ParallelKey class, rather than tracking down the corner cases that lead to orphaned tmpfiles.  I'll take care of removing the code and submit a pull request.

Here's a draft of the code, which passes flake8 and seems to work on my test instance:
https://github.com/docker/docker-registry/compare/master...Altiscale:dc_OPS-5907
I'll do a bit more testing, and then submit the pull request.

Fantastic.
Give me a ping as soon as you have a PR, I'll make sure it's reviewed quickly.

I'm moving my performance test results into this document...
https://docs.google.com/spreadsheets/d/1VbdtRL8w8eAf0sKse1qk7vLmd-J10tIiyeUqCTjSfL4/edit?usp=sharing
tl;dr: The pull request fixes the problem and doesn't change performance otherwise.  As expected, STORAGE_REDIRECT provides better bandwidth and therefore better latency for multiple simultaneous clients.  For access by a single client, performance mostly depends on the networking between the client and the registry, rather than the code running on the registry.

Thanks a lot for your work on this!


CSS load issue

Hello,
Every other time I can't load the CSS of the pages and I get this error in the console :
http://<IP_ADDRESS>/new.css net::ERR_CONNECTION_RESET
I have to refresh the page to get the display back.
I use this ESP32: https://www.amazon.fr/gp/product/B08BTLYSTM
I use firmware 6.0
Thank you

Hi and sorry for the late answer. I don't have an idea really, it works for me. Does the "inter.css" load?

P.S.: I'm running on wired ethernet. I'll try switching to WiFi and see if I can reproduce the issue.

I confirm the issue. I'm on wired ethernet with my notebook, but I don't think that how you connect to the network should influence the functionality, you can have more latency, but it should be the same, functionality-wise.
Also, refreshing doesn't help much, it seems random: if I refresh 4-5 times, eventually CSS pulls up, but only one time and in between I see connection reset or the page without CSS.

I mean I'm on wired ethernet with my ESP, using a W5500 module.

Oh sorry, I assumed you had a wireless module like me.

Hi,
I've switched to Wifi for testing, but couldn't reproduce the issue. I'd say it must be something on your end, although it's strange it's always this one file. Can you try with a different browser / computer ?

I tried with Chrome and Edge. Now I installed Brave on another computer:

I just installed Firefox: it always picks up CSS correctly. So it's something with chrome-based browsers. I don't know if Brave is chrome-based, will check it up. In any case, it should work on all browsers.

I tried with both Firefox and Chrome and had no issues. Maybe do a clean install of chrome?

I'm also having the same issue on a fresh install of 6.8.

Same issue here with nuki_hub 7.2 an Chrome 108.0.5359.71 under Linux Ubuntu.

Something's not right with the webpage, here's what Edge (chrome based) is reporting:

The only browser I don't have problems with is Firefox, on ALL chrome-based browsers there's a compatibility issue.

I'm not a web developer, if someone wants to fix the HTML go ahead :D. The page displays fine except for some users where the CSS for some reason doesn't load. I have no explanation for that, it's static content that's delivered if requested. What's even stranger is that inter.css loads, but new.css doesn't - it's pretty much the same code.
The web page is still usable - it's a simple table based page that is just looks a bit nicer with the CSS.

The web page is still usable - it's a simple table based page that is just looks a bit nicer with the CSS.

No it isn't. It takes a lot of time to load, and sometimes it throws errors (RESET, timeout, etc.). I installed Firefox just to use Nuki Hub. :)
I'm not a dev, but I'll check if there are some validation tools online that maybe will optimize the CSS code.

WebCfgServerConstants.h lines 6 and 10. You'll find the URL where I copied it from in the comments above.

P.S.: Still no issue for me, no matter which browser I use.

These things drive me crazy. Many of us have issues with that...on Chrome. How is it possible you have no problems...:)

Maybe a different OS/version?

Maybe a different OS/version?

We found the problem...and fixed it. Great teamwork on Discord. Join us. :)

Please check release 7.3

7.3 works for me now. Thanks :-)

Same here many thanks!

ok I'll close the issue then


Error while installing tox-pipenv

Hi,
I'm trying to install tox-pipenv with pipenv, however I have the following error:
lib/python3.6/site-packages/pipenv/resolver.py", line 71, in main
    sources=project.pipfile_sources,
AttributeError: 'Project' object has no attribute 'pipfile_sources'

I must confess I'm not sure it's related to tox-pipenv itself or to pipenv but pipenv install works with other packages.
Environment
pipenv --version
pipenv, version 2018.05.18

Step to reproduces
pip install -U --user pipenv
mkdir tmp && cd tmp
pipenv install tox-pipenv

looks like a versioning problem. I'll repro

confirmed this is an issue when using a newer version of pipenv, will fix and patch

Thanks!

Released in 1.5.0 yesterday


flag for surge_increment_resource is missing

The how to page mentions this field when setting up a resource to track the wild magic surge incremental counter. However I do not see that option

Hi, the entry may be at the top in the Attribute Bars section. Can you check there? Also you may need to cast a spell first for it to appear. If thats true I wil add a bug fix in later to get it added on start up

I just checked again and yeah I found it. You can close this bug


Not filtering large transactions properly

Seeing Rejecting block with at least one too-big transaction on berkeley

This is meant to keep too-big txns out of the txn pool: https://github.com/MinaProtocol/mina/blob/develop/src/lib/network_pool/transaction_pool.ml#L1015

This was determined to be a version mismatch issue.


when i use npm run build  find  not have chunk Names  in  webpack 4.X

when i input npm run build then i find

and i  want to become

yeah  i  just update webpack  i not know  how to do    i have search  a lot  and test  please  help us  thanks

this is my webpack.config.js

It looks like you just deleted our lovely crafted issue template. It was there for good reasons. Please help us solving your issue by answering the questions asked in this template. I'm closing this. Please either update the issue with the template and reopen, or open a new issue.

You can find more info about filename https://webpack.js.org/configuration/output/#output-filename

i  have  watch  but  no use sir


Clean challenge via hook bug?

i'm using the default HOOK_CHAIN="no" as i'm calling the deploy_challenge and clean_challenge function seperatel and in my case i'm using CHALLENGETYPE="http-01".
i notice that the function "clean_challenge" is not being called in such scenario.
i think that after these 2 lines:
      # Clean challenge token using non-chained hook
      [[ "${CHALLENGETYPE}" = "dns-01" ]] && [[ -n "${HOOK}" ]] && [[ "${HOOK_CHAIN}" != "yes" ]] && "${HOOK}" "clean_challenge" ${deploy_args[${idx}]}

some like this line should appear
      [[ "${CHALLENGETYPE}" = "http-01" ]] && [[ -n "${HOOK}" ]] && [[ "${HOOK_CHAIN}" != "yes" ]] && "${HOOK}" "clean_challenge" ${deploy_args[${idx}]}
is there a problem or am I missing something?

I recently restructured the loop because it left challenges in place after an invalid validation and I mistakenly added a check for dns-01 validations for the hook-call. Removed that check and everything should be fine now.


[Feature] After clicking the picture, enter the preview, the picture cannot be selected

Version information (版本信息)

Device: All devices.
OS: iOS
Package Version: Til newest
Flutter Version: Til newest

Is your feature request related to a problem?  (功能是否与已有问题关联)
No
Describe the solution you'd like (描述你期望的解决方案)
After clicking the picture, enter the preview view, the picture cannot be selected
Describe alternatives you've considered (其他解决方案或疑虑)
After clicking the picture, enter the preview view, the picture cannot be selected
Additional context (附加内容)
nil,I will figure it out,and make pull request later.

https://github.com/fluttercandies/flutter_wechat_assets_picker/issues/28


idea for multiple data sources?

Is there a solution for complex event processing with multiple data sources?

@illsion Can you please elaborate your requirement? It is not clear for me. Are you want to join with datasources when performing event processing. Can you please give more info about the use case?

@mohanvive
String streams = "" +
"define stream QingHeStream (symbol string, price float, volume long); " +
            "@Store(type=\"rdbms\", jdbc.url=\"" + url + "\", " + "username=\"" + user + "\", password=\"" + password + "\", jdbc.driver.name=\"" + driverClassName + "\", field.length=\"symbol:100\")\n" +
            "@PrimaryKey(\"symbol\")" +
            "@Index(\"volume\")" +
            "define table QingHeTable (symbol string, price float, volume long); " +

            "@Store(type=\"rdbms\", jdbc.url=\"" + url2 + "\", " + "username=\"" + user2 + "\", password=\"" + password2 + "\", jdbc.driver.name=\"" + driverClassName2 + "\", field.length=\"symbol:100\")\n" +
            "@PrimaryKey(\"symbol\")" +
            "@Index(\"volume\")" +
            "define table QingHeTable2 (symbol string, price float, volume long); ";

    String query1 = "" +
            "@info(name = 'query1') " +
            "from QingHeStream\n" +
            "insert into QingHeTable ;" +

            "@info(name = 'query2') " +
            "from QingHeStream\n" +
            "insert into QingHeTable2 ;";

io.siddhi.core.SiddhiAppRuntime siddhiAppRuntime = siddhiManager.createSiddhiAppRuntime(streams + query1);
io.siddhi.core.stream.input.InputHandler qingHeStream = siddhiAppRuntime.getInputHandler("QingHeStream");
siddhiAppRuntime.start();
qingHeStream.send(new Object[]{"CHANGE1", 55.6f, 100L});
qingHeStream.send(new Object[]{"CHANGE2", 75.6f, 100L});
siddhiAppRuntime.shutdown();
Running use cases create the same table and insert the same data in different data sources at the same time.
Manually delete the data in one table, run the use case again, and the insertion of another table will be abnormal, but the table that deleted the data before will insert as usual.
Goal: Roll back all data when an exception occurs; that is, distributed transactions that support multiple data sources

Hi @illsion,
At the moment, Siddhi does not support distributed transactions through Siddhi.
As a workaround, we can achieve your requirement through siddhi-store-rdbms CUD extension.
CUD extension provides the capability to perform SQL CUD (INSERT, UPDATE, DELETE) queries on WSO2 datasources. Therefore, you can write your query as a single statement, so that if part of query fails, the whole query would fail. Please note that, this function is only available when running Siddhi with WSO2 SP.
Please find a sample Siddhi app below.

@App:name("store-rdbms-atomic-trans")
@App:description('Receive events via simulator and persist the received data in the store.')
define stream insertSweetProductionStream (name string, amount double);
from insertSweetProductionStream#rdbms:cud('EAF_DEV_DB', "INSERT ALL INTO tableA (tid, name, amount) values (s_tid, s_name, s_amount) INTO tableB (tid, name, amount) values (s_tid, s_name, s_amount) SELECT s_tid, s_name, s_amount FROM (SELECT 1 s_tid, ? s_name, ? s_amount FROM dual )", name, amount)
select *
insert into TempStream;

In order to test the sample, perform the below steps.

Create the below tables
create table tableA(tid VARCHAR2(64), name VARCHAR2(64), amount VARCHAR2(64));
create table tableB(tid VARCHAR2(64), name VARCHAR2(64), amount VARCHAR2(64));

Send the below event to insertSweetProductionStream.
name:data5, amount:5.443435

Alter tableB adding tid as a Primary Key.

Resend an event
name:data2, amount:4.504343

When tableB insert fails with unique constraint violation,  tableA insert would also fail.

We've also opened an issue on Supporting distributed transactions under #1303 so that we can track it for our roadmap.

Closing since provided a workaround. Improvement will be handled separately through #1303.


Cloaked fire laser commando team

Gets around the cloak debuff by setting people on fire. Could be part of laser squad (#33).

Could be like missile miners, but way more annoying.


Show identityId of pod identity in kubectl get triggerauthentication -o wide

Proposal
Show identityId of pod identity in kubectl get triggerauthentication -o wide to simplify investigation.
It should not specifically be a new field but can be azure-workload (foo) and azure-workload (default)/azure-workload when no identity ID was specified
Use-Case
No response
Is this a feature you are interested in implementing yourself?
No
Anything else?
No response

Currently, that value is generated from .spec:

But we could introduce an internal status field with the information. The problem here IMHO will be that the real use case isn't azure-workload (foo), it is azure-workload (c27bf924-db20-4972-a528-b96ecb878895).
I'm not totally sure if this will be useful. we can't provide other info more than the given GUID

The problem here IMHO will be that the real use case isn't azure-workload (foo), it is azure-workload (c27bf924-db20-4972-a528-b96ecb878895).

That is the same actually, I was just lazy and used foo as example for app ID :)

I'm not totally sure if this will be useful. we can't provide other info more than the given GUID

That is exactly what is needed to be able to see if the App ID was correctly configured without having to open the YAML which I had to do due to https://github.com/kedacore/keda/issues/4528#issuecomment-1537508517

Is the GUID useful? I mean, just having 2 states could solve it? My doubts are related with the GUID itself, maybe something like azure-workload (defalt) and azure-workload (provided) or something similar could be more useful, not having 36 characters in the output for seeing the GUID

Yes it is useful, because then you see what is configured. Provided is not very useful as you still have to check the details while the GUID shows exactly what you need imo

I think that we can do it if you believe that it's useful. I'm not sure if getting the ID is useful but it's doable IMO

I think it does because it clearly shows the used ID, but I don't want to push this. @kedacore/keda-maintainers thoughts?

I think it does because it clearly shows the used ID, but I don't want to push this. @kedacore/keda-maintainers thoughts?

Bump @kedacore/keda-maintainers

@JorTurFer @zroubalik Any thoughts?

I think that this can be useful. Maybe we can update the status with the used ID and just expose it as response of the command

I am okay with that

@SpiritZhou Can you work on this when you have some time?


Expand beatmap and beatmapset databases to include more data from osu!

Currently in testing we treat our ID as if it was the osu ID, we should separate these out as to not confuse the client.

Title changed so I can include more data in this ticket


Remove single performance scenario baseline

This reverts https://github.com/gradle/gradle/pull/21095

@bot-gradle test APT

OK, I've already triggered the following builds for you:

AllPerformanceTestsReadyForRelease build

@bot-gradle test APT

OK, I've already triggered the following builds for you:

AllPerformanceTestsReadyForRelease build

@bot-gradle test and merge

Your PR is queued. See the queue page for details.

Important: SWEAT wallet backup code ❗Do not delete or share ❗


rails-api: undefined local variable or method `form_authenticity_token'

Hi, firstable I want to thank you for this work I think is great.
I just begin to build my API,  using the rail-api gem and when I come to login function y jut got the error:
undefined local variable or method `form_authenticity_token' for #<UserSessionsController
I al ready include the sorcery´s reference to the ActionController::API as it´s explained
class ApplicationController < ActionController::API
  include Sorcery::Controller
end

What else do you thing I´m doing wrong here´s my source
https://github.com/manolo07/oneTest
Can you help?

@manolo07 it seems that Sorcery currently does not work with Rails-API. The reason is that always when user is logged in we reset the CSRF token, and the method that does it, form_authenticity_token is not present in Rails-API.
I'll check if it's safe to remove usage of this method, or if its not, how to make it work with Rails-API.

Ok, we definitely need to keep this method there. I'll try to make it work with Rails-API, but I'm not sure when it'll be possible (I hope until end of the year). For now, you can just define this method in your controller:
def form_authenticity_token; end

It will prevent error from being raised. However I expect you might get more issues in the future, since Sorcery wasn't meant to be used with JSON APIs, and I haven't managed to adapt it to this kind of applications yet.

Thanks!, very much

+1


Custom em in reader

Although after last changes it is possible to save audits in custom storage, it was not possible to show it in administration panel. This pull request is changing it.

I totally agree, it was my first thought to store this property in AuditConfiguration, but CodeClimate has rejected that change because of to many arguments in a constructor - do you know how to ignore/change this requirement?

Ok, go ahead with these changes, i’ll take care of CodeClimate ;)


BIOFORMATS-5.1-merge-docs-autogen builds Bio-Formats documentation

Staged at https://www.openmicroscopy.org/site/support/contributing-staging/ci-docs.html#term-bioformats-5-1-merge-docs-autogen.
--no-rebase for contributing docs

Thanks for spotting that!


Dev

This pull request includes several important changes to the pystrukts library, mainly focusing on adding new data structures, updating documentation, and improving the CI configuration.
New Data Structures:

pystrukts/graph/adjacency_matrix.py: Added the AdjacencyMatrix class for graph representation using an adjacency matrix.
pystrukts/graph/graph.py: Introduced an abstract Graph class for graph representations.
pystrukts/tree/trie.py: Implemented a Trie class and TrieNode class for prefix tree data structures.
pystrukts/union_find.py: Added the UnionFind class for the Union-Find Disjoint Set data structure.

Documentation and Readme Updates:

README.md: Updated the status of several data structures to reflect their implementation.

CI Configuration:

.github/actions/setup/action.yml: Updated the setup-python action to use version 5.
.github/workflows/lint.yml: Modified the pylint command to run on the entire project and ignore the tests directory.

Codebase Organization:

Added __init__.py files in various directories to include necessary imports and skip pylint checks. [1] [2] [3] [4]

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:
:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.

Files with missing lines
Coverage Δ

pystrukts/__init__.py
100.00% <100.00%> (ø)

pystrukts/graph/__init__.py
100.00% <100.00%> (ø)

pystrukts/graph/adjacency_matrix.py
100.00% <100.00%> (ø)

pystrukts/graph/graph.py
100.00% <100.00%> (ø)

pystrukts/list/__init__.py
100.00% <100.00%> (ø)

pystrukts/list/doubly_linked_list.py
100.00% <100.00%> (ø)

pystrukts/list/singly_linked_list.py
100.00% <100.00%> (ø)

pystrukts/max_heap.py
100.00% <100.00%> (ø)

pystrukts/tree/__init__.py
100.00% <100.00%> (ø)

pystrukts/tree/trie.py
100.00% <100.00%> (ø)

... and 7 more


Set shim OOM scores to +1 containerd daemon score

This changes the shim's OOM score from a static max killable of -999 to
be +1 of the containerd daemon's score.  This should allow the shim's to
be killed first in an OOM condition but leave the daemon alone for a bit
to help cleanup and manage the containers during this situation.
Signed-off-by: Michael Crosby<EMAIL_ADDRESS>

Travis passed; https://travis-ci.org/containerd/containerd/builds/551367554?utm_source=github_status&utm_medium=notification
2 LGTMs; merging


is there any way to get the ids only, I mean not return the objects,  when I use intersects or within command

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

just like remove the object part

It is possible to change the output format. Just set it as IDS so it will only return object IDs. You can refer to the documentation below.
https://tile38.com/commands/intersects#output-formats

thanks a lot, it worked. I have read this documentation before, but I just did not  get it.

It is possible to change the output format. Just set it as IDS so it will only return object IDs. You can refer to the documentation below.
https://tile38.com/commands/intersects#output-formats

thanks a lot, it worked. I have read this documentation before, but I just did not get it.


Query() vs QueryWithTimeSequenceInformation()

Hello, I am new to your project. And I have some questions. What are their difference?
I extracted chunks from original mp3 file. (song_00_30 means same ffmpeg's args: -ss 0 -t 30) And run test against each other. But results are not good as expected. Why these results are very different? For example MatchedFingerprints 21 vs 1800+ ?
 The number of fingerprints in the result set should not be a concern. What matters is the matched result track not the fingerprint counts.

Fixed in 3.0.0. Its now available on NuGet 3.0.0-beta.

Closing due to inactivity.


Question: Licensing and Fabulous project templates

First of all, thanks for the amazing framework. This is the most promising UI framework I have seen.
Question / Discussion
To my question:
The getting started guide recommends using the project templates e.g. via
dotnet new fabulous-xf-app -n MyApplication
However, in the generated file it includes a license header, and I don't quite understand what that implies when using the templates:
// Copyright Fabulous contributors. See LICENSE.md for license.
So what I understand is that the templates are licensed under the Apache 2.0 License, and thus I need to follow Apache 2.0 in my code-base if I use the templates. So for example, using the templates I need to comply to Apache 2.0 in the my own code base, or no? (E.g. keeping the license header in my code-base) Doesn't this limit the use-case scenarios of the templates?

Hi @TitusLabienus. Thanks for your interest in Fabulous!
I'm no expert on the licensing subject but the way I understand it, Apache 2.0 only requires you to mention the copyright/license of Fabulous somewhere for your users to see. You don't need to have the license header in your own code files.
Usually, I've seen it done in the application About page. Like "Copyright of the app - Copyright lib1 - Copyright Fabulous - etc".
The license header of the templates is only here as a reminder to check the license of Fabulous if you intend to use it.


Unit Testing Framework + Liquid Stake Tests

Summary

Added unit testing framework
Migrate the epochs module to use this framework
Added tests for the Liquid Stake flow.
I've tried my best to describe everything below, but it's probably a little hard to follow just from reading so feel free to reach out if you'd prefer a walk though!

Testing Framework

app/test_setup.go defines a function that initializes a test blockchain instance
app/apptesting/test_helpers.go defines an object AppTestHelper that extends a go testing package called suite and manages the app (App) and context (Ctx) as attributes

It also has a setup function that calls the chain init function (from test_setup.go) and creates sample accounts
It also has testing helper functions (e.g. CreateSampleAccounts, FundAccount, etc.)

For each module, a keeper_test.go file should define a KeeperTestSuite object that extends the AppTestHelper and contains module-specific setup functions (e.g. initializing the message server)

It should be a part of a new package called keeper_test
The purpose of extending the suite class is to group tests together, such that they share common setup, teardown functionality and allow them to run in parallel (although it's still unclear whether tests usually run in parallel by default?)
The keeper_test.go file must also have a function (shown below), that registers all the suite tests with the default go testing context

func TestKeeperTestSuite(t *testing.T) {
   suite.Run(t, new(KeeperTestSuite))
}

Finally, a new file can be created for each message/flow we want to test, (e.g. msg_server_liquid_stake_test.go)

This file will extend the KeeperTestSuite which will give it access to everything it needs including the keepers (suite.App.{KeeperName}) and the message server (suite.msgServer)
The combination of these allows you to call any message directly and test any account balance or state changes

Setting up a new Test

Create the new test file in the module's keeper (e.g. msg_server_liquid_stake_test.go)
The file should be a part of the keeper_test package and each test function must extend the KeeperTestSuite
Create a setup function for the group of tests. This function should be called before every test. This will initialize the state and balances for the group of tests (see SetupLiquidStake). I'd recommend returning a struct from this setup function that provides information about the initial state that can be used in each test function (e.g. initial account balances). The struct should also provide an example of a successful message, given the initial state setup.
Define one or more functions to test a successful flow. For these tests, call the tested functionality and then compare the actual output state to the state that was expected. To compare values use, suite.Require().Equals(expected, actual, description)
Define functions to test out errors. For these, simply alter either the message or the state before calling the tested function, and then confirm the returned error lines up with what's expected using suite.Require().ErrorEquals(...)
Finally you can run the test! I'd recommend running directly from VSCode as you're debugging and moving to the command line at the end. In order for the play button to register in vscode, you'll have to import suite to that file

import _ "github.com/stretchr/testify/suite"

Unanswered Questions

How well this framework will work with IBC transactions
Whether we'll need to iterate blocks

I assume no, because all that would do is finalize the state; but, with this setup, the state is finalized immediately
If we find that we do need to iterate blocks, I'm confident it can be done (osmosis has an example of this)

Remaining Work

Tests only work from VSCode, atm. Working through it now

Future Work

I'm sure things could be organized more gracefully. But I would prefer to be redundant at the start, and then over time we can start combining functionality once we write more tests and it becomes clear what logic is test-specific and what can be generalized. I moved the functions that were obviously generalizable to test_helpers.go
Definitely not a priority - but we should migrate all the old keeper tests to this framework at some point, and eventually remove the testutil directory.

It should be fairly straightforward (just need to extend the KeeperTestSuite and change the keeper and ctx references to come from the suite attributes.

Test plan
make test-unit

*Note: Some tests were already broken before starting this branch

One other nit comment: we should prob pull the definitions of Account, State and TestCase out of the liquid stake test to a more general file in keeper_test bc they'll be reused across tests (e.g. in RedeemStake's test)

One other nit comment: we should prob pull the definitions of Account, State structs out of the liquid stake test to a more general file in keeper_test bc they'll be reused across tests (e.g. in RedeemStake's test)

Yeah I very much agree. But I this is one of those areas where I generally want to start off a little more redundant at the start and wait to see how much overlap we actually have before we start optimizing for organization. For instance, the fields in the state struct will differ depending on the test (e.g. you might not be testing the deposit records). And the Account struct might want to check balances from other coins.


marginal_icdf for >2-dimensional distributions

I'm submitting a ...

[ ] bug report
[x] feature request
[ ] code improvement request

Expected behavior
MultivariateDistribution's marginal_icdf should work for >2-dimensional distributions too.

Has been implemented, see https://github.com/virocon-organization/virocon/blob/master/virocon/jointmodels.py#L524


Plugin is incompatible with 14.1

git flow version
1.8.0 (AVH Edition)
OSX 10.10.2
Intellij IDEA 14.1 EAP (140.2285.5)
Thanks

Same problem here with intellij 14.1 released

It's not working for me either, with IntelliJ 14.1

@Phoboss it seems to be fixed in the v0.4.2: http://plugins.jetbrains.com/plugin/download?pr=&updateId=19210

Great! It works now :)
Thanks


async returns and Promises have different prototypes in initial template

Version
3.10.0
Reproduction link
https://github.com/abbradar/promise-example/
Environment info
Environment Info:

  System:
    OS: Linux 5.2 NixOS 19.09.git.6bd7279 (Loris) 19.09.git.6bd7279 (Loris)
    CPU: (8) x64 Intel(R) Core(TM) i7-8650U CPU @ 1.90GHz
  Binaries:
    Node: 10.16.0 - /usr/bin/node
    Yarn: Not Found
    npm: 6.9.0 - /usr/bin/npm
  Browsers:
    Chrome: Not Found
    Firefox: 68.0.1
  npmPackages:
    @vue/cli: ^3.10.0 => 3.10.0
    @vue/cli-shared-utils:  3.10.0
    @vue/cli-ui:  3.10.0
    @vue/cli-ui-addon-webpack:  3.10.0
    @vue/cli-ui-addon-widgets:  3.10.0
    typescript:  3.5.3
    vue-cli-plugin-apollo:  0.20.0
    vue-jscodeshift-adapter:  2.0.2
    vue-sfc-descriptor-to-string:  1.0.0
    vue-template-compiler:  2.6.10
  npmGlobalPackages:
    @vue/cli: Not Found

Steps to reproduce

Run npm run serve;
Open website in a browser;
Check console log; observe false in a sanity check.

What is expected?
Only one Promise implementation is used for Promise and for transpiling async functions.
What is actually happening?
Two different Promise implementations can be seen - native one and a polyfilled one.

The project is freshly generated with vue create. I fixed this by moving to fast-async with this babel.config.js:
module.exports = {
  presets: [
    ['@vue/app', {
      "exclude": ["babel-plugin-transform-async-to-generator", "babel-plugin-transform-regenerator"]
    }]
  ],
  "plugins": [
    ["module:fast-async", {
      "spec": true,
      "useRuntimeModule": false
    }]
  ]
}

Given that this is the default behaviour of the polyfills / tranforms done by babel's preset-env, what's the drawback?
Specifically because that other plugin that you want to use isn't "official", and hasn't seen any commits since Oct 2018, which of course may simply be an indicator that it's stable - or that it's kind of unmaintained.

I didn't try it with raw preset-env, should I report this issue to them instead?
To be clear my idea isn't to use fast-async by default, that's just my way out of the problem. I'm not sure how to fix this problem in a better way but having two different Promise implementations silently used in a project feels very wrong.
On August 21, 2019 9:53:25 AM GMT+02:00, "Thorsten Lünborg"<EMAIL_ADDRESS>wrote:

Given that this is the default behaviour of the polyfills / tranforms
done by babel's preset-env, what's the drawback?
Specifically because that other plugin that you want to use isn't
"official", and hasn't seen any commits since Oct 2018, which of course
may simply be an indicator that it's stable - or that it's kind of
unmaintained.
--
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub:
https://github.com/vuejs/vue-cli/issues/4456#issuecomment-523341624

--
Nikolay.

Seems no longer reproducible?