Migrate Kadane's Algorithm

Create Kadane's Algorithm as header file.
Create unit test file.
Delete old Kadane's .cpp file.
Update README.

[x] Read the contribution guidelines
[x] Added unit tests (or unit tests have already been added)
[x] Updated the Contents

@faheel Done.


Add funtionality to allow providing open API parameters without using attributes

related issue : #220
This PR adds some ability to provide additional open api parameters without needing to override or inherit multiple classes. The idea was to allow an easily customisable place that would fit in with the existing architecture while also keeping the change small enough since this is a package that is still in preview.
I have not added any unit tests at this stage in case there are changes required to the code but i will add unit tests to cover the changes.
Overview:

Add 3 properties to IOpenApiConfigurationOptions to allow providing the additional parameters
The existing open api attributes defined in this library are used in 3 newly added interfaces so the existing IDocumentHelper methods can be used without much refactoring

    public interface IOpenApiConfigurationOptions
    {
        ...
        /// <inheritdoc cref="IAdditionalOpenApiParameter"/>
        IAdditionalOpenApiParameter AdditionalOpenApiParameters { get; set; }

        /// <inheritdoc cref="IAdditionalOpenApiRequestBody"/>
        IAdditionalOpenApiRequestBody AdditionalOpenApiRequestBody { get; set; }

        /// <inheritdoc cref="IAdditionalOpenApiResponse"/>
        IAdditionalOpenApiResponse AdditionalOpenApiResponse { get; set; }
        ...
     }

@xsneebsx I should probably hold this PR until my idea gets more concrete. But let's keep this discussion open.

This is really big changes. Could you please elaborate what these changes are for?

The additional changes was around refactoring/consoldiating the DocumentHelperExtensions since the only real difference between them was the function attribute it was using to filter down the azure functions and the rest of the file content were the same. Similar to the DocumentHelper classes which had duplicated code. The reason for refactoring this was so i could test the proposed change to the configuration object in 1 place instead of testing it over 2 places. This change has meant the DocumentHelper class is now generic so that it can take in the specific attribute (either FunctionNameAttribute or FunctionAttribute)
I can revert those changes if it's considered to be too big and i can leave the change with only the additional properties added to the IOpenApiConfigurationOptions and the unit tests for each of the previous DocumentHelper classes.
The changes so far would be the extent of the change, i don't have any other changes to make on this PR outside of concerns or changes from code review.


feat(MeasureTheory): weaken assumptions from SigmaFinite to SFinite

Weaken TC assumptions of many lemmas from SigmaFinite to SFinite.
Drop some MeasurableSet assumptions.

bors delegate+

bors merge

bors r-

Please merge again when the build succeeds

@mattrobball Add you can see from the logs, this is not the PR that caused the build to fail
bors merge

Sorry.


[iOS] AchievementRepository dependency not provided

Describe the bug
When I start the application, I get the following error.
Uncaught Kotlin exception: org.koin.core.error.NoBeanDefFoundException: No definition found for
class:'io.github.droidkaigi.confsched2023. AchievementRepository' q:''. Check your definitions!

iOSMain > io.github.droidkaigi.confsched2023.data.DataModule.kt does not contain AchievementRepository dependency.
To Reproduce
Steps to reproduce the behavior:

Start up application
The error happen

Expected behavior
The application should start up properly.
Screenshots

Thank you for your report! It must be resolved by this PR 🙆♂ https://github.com/DroidKaigi/conference-app-2023/pull/1141


Fixes travis ci security provider error

Build throws java.security.ProviderException, with root cause: no such provider: SunEC when fetching npm from registry.
May be due to openjdk 7 not having the proper security provider.
Try a potential workaround suggested in https://bugs.launchpad.net/ubuntu/+source/openjdk-6/+bug/1006776

Codecov Report

Merging #4163 into master will decrease coverage by 0.04%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #4163      +/-   ##
==========================================
- Coverage   41.15%   41.11%   -0.05%
==========================================
  Files        2354     2354
  Lines       74215    74215
  Branches    10047    10047
==========================================
- Hits        30544    30512      -32
- Misses      40858    40886      +28
- Partials     2813     2817       +4

Impacted Files
Coverage Δ

...vercorp/pinpoint/rpc/packet/ServerClosePacket.java
52% <0%> (-16%)
:arrow_down:

...rp/pinpoint/rpc/control/ControlMessageDecoder.java
64.28% <0%> (-14.29%)
:arrow_down:

...r/response/DefaultResponseTimeMetricCollector.java
85.71% <0%> (-14.29%)
:arrow_down:

.../pinpoint/rpc/client/PinpointClientHandshaker.java
60% <0%> (-6.43%)
:arrow_down:

...rp/pinpoint/rpc/server/PinpointServerAcceptor.java
85.13% <0%> (-4.73%)
:arrow_down:

...java/com/navercorp/pinpoint/rpc/DefaultFuture.java
72.5% <0%> (-1.67%)
:arrow_down:

...pinpoint/profiler/monitor/DeadlockMonitorTask.java
18.6% <0%> (-1.17%)
:arrow_down:

...cluster/zookeeper/ZookeeperClusterDataManager.java
65.3% <0%> (+0.68%)
:arrow_up:

Continue to review full report at Codecov.

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


panic: close of closed channel

The scene of the panic caused by a specific scenario is as follows:
1.start master and slave.
2.Spawning 100 clients at the rate 1 clients/s...
3.click stop before clients less than 100.
4.click new test, repeat 2...
5.slave panic
The following is the stack：
2021/04/15 06:02:43 Spawning 100 clients at the rate 1 clients/s...
2021/04/15 06:02:46 Recv stop message from master, all the goroutines are stopped
panic: close of closed channel
goroutine 37 [running]:
github.com/myzhan/boomer.(*runner).stop(0xc000308000)
/armyant/vendor/github.com/myzhan/boomer/runner.go:218 +0x71
github.com/myzhan/boomer.(*slaveRunner).onMessage(0xc000308000, 0xc000137a10)
/armyant/vendor/github.com/myzhan/boomer/runner.go:378 +0x587
github.com/myzhan/boomer.(*slaveRunner).startListener.func1(0xc000308000)
/armyant/vendor/github.com/myzhan/boomer/runner.go:410 +0x4c
created by github.com/myzhan/boomer.(*slaveRunner).startListener
/armyant/vendor/github.com/myzhan/boomer/runner.go:406 +0x3f

Fixed


Resource defines not applied for VC201x projects (at all!)

The current code, namely vs2010_vcxproj.lua doesn't use resdefines at all. I just noticed this by accident. I'll definitely write a Lua-only workaround for this, but I still need to set up myself to contribute back changes from my (still Mercurial based) repo.
Unfortunately the function preprocessor is a local one, which makes the planned workaround more cumbersome than it would otherwise be. Likely another case where io.capture() is called for.

    local orig_premake_vs2010_vcxproj = premake.vs2010_vcxproj
    premake.vs2010_vcxproj = function(prj)
        -- The whole stunt below is necessary in order to modify the resource_compile()
        -- output. Given it's a local function we have to go through hoops.
        local orig_p = _G._p
        local besilent = false
        -- We patch the global _p() function
        _G._p = function(indent, msg, ...)
            -- Look for indent values of 2
            if indent == 2 and msg ~= nil then
                -- ... with msg value of <ResourceCompile>
                if msg == "<ResourceCompile>" then
                    local cfg = getLocal(3, "e") -- with LuaSrcDiet
                    if cfg == nil then
                        cfg = getLocal(3, "cfg") -- without LuaSrcDiet
                    end
                    assert(type(cfg) == "table" and cfg["resdefines"] ~= nil)
                    orig_p(indent, msg, ...) -- spit the original line out
                    local indent = indent + 1
                    if #cfg.defines > 0 or #cfg.resdefines then
                        local defines = table.join(cfg.defines, cfg.resdefines)
                        orig_p(indent,'<PreprocessorDefinitions>%s;%%(PreprocessorDefinitions)</PreprocessorDefinitions>'
                            ,premake.esc(table.concat(premake.esc(defines), ";")))
                    else
                        orig_p(indent,'<PreprocessorDefinitions></PreprocessorDefinitions>')
                    end
                    if #cfg.includedirs > 0 or #cfg.resincludedirs > 0 then
                        local dirs = table.join(cfg.includedirs, cfg.resincludedirs)
                        orig_p(indent,'<AdditionalIncludeDirectories>%s;%%(AdditionalIncludeDirectories)</AdditionalIncludeDirectories>'
                                ,premake.esc(path.translate(table.concat(dirs, ";"), '\\')))
                    end
                    besilent = true
                end
                -- ... or msg value of <ResourceCompile>
                if msg == "</ResourceCompile>" then
                    besilent = false
                    -- fall through
                end
            end
            if not besilent then -- should we be silent?
                orig_p(indent, msg, ...)
            end
        end
        -- do whatever else is called for ...
        _G._p = orig_p -- restore in any case
    end

Other solutions are obviously possible, notably io.capture(), but I have noticed that this method seems to be lighter on resources, despite using introspection features.
We're looking for the name "e" first, because LuaSrcDiet shortens the variable name "cfg" to that. If we fail, we try again with the original name. After that we assert that it's non-nil and contains the key resdefines ... voila, problem solved.

If you submit this as a PR (as a direct modification of the core core, rather than a patch) I'll be happy to merge it.

Hey @starkos , of course this above solution makes no sense as PR. Because it should be fixed in Premake4 so that above workaround isn't necessary anymore (simply by patching vs2010_vcxproj.lua). What I'm trying with these workarounds is to patch the behavior of Premake4 with what I have, without having to recompile the binary all the time. And also in absence of some real patch management it's kinda hard to keep track. Premake would need something like in Vim where you can check which patches or features are available and execute something based on that condition. In the past I've done this by keeping some of the new symbols global so I could look out for them in my scripts and keep my scripts compatible across a wide range of Premake4 patch levels. No idea if you have something like this patch management planned for Premake5, but it'd definitely make sense to have this.
I'll try to submit the patches I have ready in my fork until the end of the year. No promises, though.
Until then I am keeping that cookbook article in the Wiki up-to-date. I hope that this will be valuable to others running into similar issues.


Docs on using special SSH keypair with ACS/Kubernetes

We already had a section in the Appendices about how to generate an SSH key pair for use with AWS virtual machines. I refactored that section out into a new stand-alone page in the Appendices, because...
In the page about how to deploy a Kubernetes cluster on Azure, I added a link to the new page about generating an SSH key pair, and I showed how to deploy a cluster so that all the VMs in the cluster have the just-generated SSH public key.
I also removed the --location option from the az acs create command because it's not necessary: the cluster location is inherited from the resource group, which is already specified.
I also added a short section on how to ssh to the "master" node.

Codecov Report

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

@@          Coverage Diff           @@
##           master   #1223   +/-   ##
======================================
  Coverage    96.7%   96.7%
======================================
  Files          49      49
  Lines        2306    2306
======================================
  Hits         2230    2230
  Misses         76      76

Continue to review full report at Codecov.

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


dispatchify to return the promise from dispatch

I'm currently writing tests to check actions are called and the state gets updated properly. In my assertion steps I need to wait for the state to be changed once the actions have been raised. I was using the dispatchify function to reduce dependencies in my view models, but dispatchify doesn't return a promise so am unable to use async/await. And thus can't gaurantee the state has been updated at the point of my assertion steps.
I can work around this, by calling dispatch directly on store and using async/await.
I am unable to do this with dispatchify. It seems simple enough to return the promise on the dispatchify function?
Thanks for your time

You're right that was just an oversight, will add this with the next release

@MatthewSwarbrick should be fixed with the latest patch release

Closing this for know. If you still experience any issues we can reopen it


Remove now unused landuse kind mapping transform

Connects to https://github.com/mapzen/vector-datasource/issues/580
@zerebubuth could you review please?

:+1:


Allow to add custom tags

Closes #18

Not what I originally had in mind, but I think it works, and as a bonus, I think it actually closes #13 as well. I need to play around with this a bit later today before giving it the full "that'll work!", but initial impressions are that it will indeed work as needed.


wdio-allure-reporter is showing the results of only 8 feature files

Hi All / @BorisOsipov
I assume this issue is related to wdio-allure-reporter so raising it in here. I've gone through many sites for a solution before raising this issue and since there is no solution identified, ended up requesting the suggestions/help here.
Issue: I have executed total 11 feature files through config.js file and with max instance=3. Although all
11 feature files are executed (can see in the terminal log), Allure results xml's for the feature files mmore than 8 are NOT getting generated and hence the Allure HTML report is NOT showing up results for all 11 feature files instead showing for 8. I've played around with options (disableWebdriverStepsReporting: true,useCucumberStepReporter: true,disableWebdriverScreenshotsReporting: false) but NO luck.
Please can someone shed light in resolving this issue as it is blocking the test reporting? Many thanks in advance.

I have no ideas what is going wrong from your issue description.
Could you please attach zip archive of your allure-results folder and
terminal log with resutls of execution your 11 feature files ?
If you can't share it with everyone you may send link with archive to me
privately in me gitter
https://gitter.im/borisosipov
пт, 15 июн. 2018 г. в 23:48, PavanGurramQA<EMAIL_ADDRESS>
Hi All / @BorisOsipov https://github.com/BorisOsipov
I assume this issue is related to wdio-allure-reporter so raising it as an
issue in here. I've gone through many sites for a solution before raising
this issue and since there is no solution identified and ended up
requesting the suggestions/help here.
Issue: I have executed total 11 feature files through config.js file and
with max instance=3. Although all
11 feature files are executed (can see in the terminal log), Allure
results xml's for the feature files mmore than 8 are NOT getting generated
and hence the Allure HTML report is NOT showing up results for all 11
feature files instead showing for 8. I've played around with options
(disableWebdriverStepsReporting: true,useCucumberStepReporter:
true,disableWebdriverScreenshotsReporting: false) but NO luck.
Please can someone shed light in resolving this issue as it is blocking
the test reporting? Many thanks in advance.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/webdriverio/wdio-allure-reporter/issues/124, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AGNmZDXFo94pD02l5n2nu78ICKHJzkMjks5t9B2xgaJpZM4UqIZg
.

--
--
Best Regards,
Boris Osipov

Hi @BorisOsipov
Thanks for your prompt response again. I've shared the folders in gitter, please have a look and let me know your view.

Missed suites didn't executed success. They are crashed with error
F...ERROR: a handler errored, process exiting: Formatter::handleAfterScenario: Cannot read property 'ELEMENT' of null
chrome
V at C:\node_modules\cucumber\lib\runtime\event_broadcaster.js:78:21
at next (native)
at undefined.tryCatcher (C:\node_modules\bluebird\js\release\util.js:16:23)

I guess it is the main reason why these specs are missing in the report.


add fuzzy search on task

add /api/task/status route
add environment, project, status search option on api/task/ route

need to resolve confilicts


ntr: protein localization to prospore septin ring

modeled on GO:1902935; to use to define phenotypes for PMID:20123972
[Term]
id: GO:new
name: protein localization to mating projection septin ring
namespace: biological_process
def: "A process in which a protein is transported to, or maintained in, a location within the prospore septin ring." [PMID:20123972]
synonym: "protein localisation in prospore septin ring" EXACT []
synonym: "protein localisation to prospore septin ring" EXACT []
synonym: "protein localization in prospore septin ring" EXACT []
synonym: "protein localization to forespore septin ring" EXACT []
is_a: GO:1902935 ! protein localization to septin ring
the destination CC is GO:0032169

EDITED to change to prospore ring

is this a fundamentally distinct process from other protein localization?  can't/shouldn't the destination be specified in the annotation_extension?  surely we don't want protein localization terms from every possible cellular place that a protein could go.  am i missing something?

As in many other cases, it would be exactly semantically equivalent to do a GO annotation with extensions. I've requested a precomposed term mainly because there is already a LOT of precedent for "protein localization to $CC" terms in the ontology (and secondarily because what I actually want to do is point a phenotype logical definition to a GO term, and it's a bit easier to use a precomposed GO term for that).

@mah11  - there is certainly precedent for having these terms in the ontology, however, judging from the number of IMP annotations associates with them, I worry a bit that they are often used to capture phenotypes rather than distinct mechanisms for localizing gene products.
In this particular case, would you also use the term for GO annotation or just for the phenotype term's logical def?

I have been wondering about this too.
@mah11 could we precompose these terms using GO but localization + component?
This way we do not instantiate terms in GO that would never be used for GO annotation, and this would prevent curators using these terms to describe phenotypes.
A lot of localization is "autonomous", but we see "localization phenotypes" that we want to capture (localization dependencies are an important first step in reconstructing pathways). I agree that we don't really want  GO terms for localization to every component (probably even the ones which are not only 'phenotypes' longer term, but especially these). However,  we DO need a way to capture specific localizations in the context of the process that the act in.
General thoughts from discussions with @pgaudet - (mainly from the perspective of the  transcription refactoring and kinetochore assembly processes):
Cellular localization in GO is broadly 2  things
i) transport
ii) recruitment
(sometimes with elements of both, transport system moves things somewhere, recruitment factor maintains localization)
Here we are concerned with 'recruitment'  which is really a molecular function, but historically we have captured it in biological process with
"protein localization to x"
Obviously "recruitment to chromatin" is "localization" but this isn't really the process we are trying to capture. More often it's assembly (i.e of the replisome, transcription initiation complex, heterochromatin formation etc). Clearly, localization happens but there are better ways to capture this. For example, the localisation to the kinetochore as part of “kinetochore assembly” would be:
MF “molecular adaptor” XXXX part_of kinetochore assembly
MF “molecular adaptor” XXXX part_of chromatin assembly
Even better, if there was a specific "molecular adaptor" term for "recruitment". This term could have a MF-BP link to "localization", so the localization would implicitly be captured "behind the scenes".
To do this  we need a more explicit "adaptor" type MF term to describe "recruitment"
eg. MF “recruitment adaptor” part_of  BP "kinetochore assembly"
captures the context in a way that "protein localization" insert_relationship "kinetochore"  doesn't.
We could then retire terms like
"protein localization to kinetochore involved in kinetochore assembly"
and possibly even
"protein localization to kinetochore"
because we could still do
“recruitment adaptor” XXXX part_of  "process localization is required for"


Failing test of waypoint results in 'Cannot read property 'length' of null' error

Waypoint link: http://freecodecamp.com/challenges/waypoint-nest-an-anchor-element-within-a-paragraph
If you fail the second test of the waypoint, that is the text inside the anchor element is not 'cat photos', the test throws an Cannot read property 'length' of null error.

If the test is passed, then it behaves as expected.

@awesomeaniruddh a quick fix of this is removing .length > 0 from line 1140.

Yep, that should fix it


Margin/padding/etc won't merge if you use a variable

Bug report
Describe the bug
When using margin-left/right/top/bottom, they will merge if the value is the same, unless you use a variable.
To Reproduce
https://codesandbox.io/s/admiring-water-kd0u1?file=/src/App.tsx

Create a theme and describe a variable
Create a component from it
Create a padding and a margin for all sides (left, right, top, bottom), one using a theme token, and the other not
See error

Expected behavior
If all the values are the same using a token, they should be merged
Screenshots

System information

OS: Windows 10
Version of Stitches: 1.1.0
Version of Node.js: v14.15.4

Perhaps I am confused, but if you explicitly set all of them, I would expect that they should not be merged. It sounds like unfortunate behavior by the browser’s older CSSOM technology, which happens to merge the values when they are not variables (because they can’t necessarily guess the value of the variables, and because it was probably a bad idea to have ever collapsed them to begin with).

@jonathantneal could you wait before closing, geez, that's why I left react. This behaviour does not happen with any other lib I tried, styled-components does not merge in any circunstance. And since I'm building every variant dinamically I needed that behaviour to work properly. So if this was not supposed to merge under any circunstance, that's still a bug to be fixed.

@darklight9811, we can definitely still chat. Let me show you something.
Consider the following code. This is pure "vanilla" browser JavaScript. Mmmm. No libraries.
This is what happens when we run it in Safari:
const { sheet } = document.head.appendChild(document.createElement('style'));

sheet.insertRule(`body {
	margin-top: 50px;
	margin-right: 50px;
	margin-bottom: 50px;
	margin-left: 50px;
}`, 0);

console.log(sheet.cssRules[0].cssText) // body { margin: 50px; }

sheet.insertRule(`body {
	margin-top: var(--50px);
	margin-right: var(--50px);
	margin-bottom: var(--50px);
	margin-left: var(--50px);
}`, 1);

console.log(sheet.cssRules[1].cssText) // body { margin: var(--50px); }

Did you see the same thing? If you did, you use Safari.
Chrome and Firefox will report the first console.log the same way. They will merge the margin declaration, which is what you expect. However, they will return something different than Safari for that second console.log. This is what they return:
console.log(sheet.cssRules[1].cssText) // body { margin-top: var(--50px); margin-right: var(--50px); margin-bottom: var(--50px); margin-left: var(--50px); }

You have happened upon an example where Safari is the only consistent browser. 😅
Personally, I wish the browsers wouldn’t collapse anything in either example. If I send them all 4, they should keep it that way. But that’s my own opinion.
Your bug report is that you expect things to work consistently, and that puts me in the uncomfortable position of telling you that the inconsistency is coming from Chrome and Firefox.
This isn’t coming from Stitches. We send the browser all 4 properties in both instances, whether it’s thru a pre-rendered <style> tag or a dynamic write to the CSSOM. It’s the browser that reports something different back to you.
I would agree this seems like a bug to be fixed. I’m going to send this to some friendly Chrome peeps.


[AIRFLOW-1611] Customize logging config

Dear Airflow maintainers,
The current way of loading a custom logging config using the logging_config_path does not work:
https://codecov.io/gh/Fokko/incubator-airflow/src/master/airflow/settings.py#L180
Therefore I would like suggest to change this so we can import a logger configuration that is defined in Python using a dict. The logging_config_path property in the config file can be used to point to a custom logging configuration which will be imported.
Change the configuration of the logging to make use of the python logging and make the configuration easy configurable. Some of the settings which are now not needed anymore since they can easily be implemented in the Python config dict itself.
Please accept this PR. I understand that it will not be reviewed until I have checked off all the steps below!
JIRA

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

https://issues.apache.org/jira/browse/AIRFLOW-1611

Tests

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

Commits

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

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

Codecov Report

Merging #2631 into master will decrease coverage by 0.26%.
The diff coverage is 97.29%.

@@            Coverage Diff             @@
##           master    #2631      +/-   ##
==========================================
- Coverage   71.71%   71.45%   -0.27%
==========================================
  Files         151      152       +1
  Lines       11736    11734       -2
==========================================
- Hits         8417     8385      -32
- Misses       3319     3349      +30

Impacted Files
Coverage Δ

...config_templates/default_airflow_local_settings.py
100% <ø> (ø)

airflow/settings.py
92.53% <100%> (+3.4%)
:arrow_up:

airflow/logging_config.py
100% <100%> (ø)

airflow/www/app.py
96.47% <100%> (ø)
:arrow_up:

airflow/jobs.py
77.7% <50%> (+0.09%)
:arrow_up:

airflow/utils/log/s3_task_handler.py
0% <0%> (-26.87%)
:arrow_down:

airflow/utils/log/gcs_task_handler.py
0% <0%> (-23.81%)
:arrow_down:

Continue to review full report at Codecov.

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

I'm not sure that we have the correct upgrade path from 1.8.x here -- i.e. if I am currently using S3 remote logging and I upgrade to this 1.9 it will silently stop working.
I do like the idea of the change though.

Took a while. I've extended the tests with a more complicated path etc.airflow.config.custom_airflow_local_settings.LOGGING_CONFIG. This caused the tests to fail because the temporary path that I was creating required those __init__.py files for Python 2.7.

my thinking on this was that logging_config_path could point to a json file that could be json.loaded, and the resulting dict could be passed to logging.config.dictConfig. Its a bit annoying and not entirely necessary requiring an importable object on the python path (supporting logging_config_path=/path/to/my/config.json would be nice), but maybe this can be a follow up issue.

@fokko almost :GTM. Can you make clear in updating.md (if then wise) what one needs to do when currently using the S3 / GCS log? Ie. what do I need to do to make it work on 1.9.0. Then we can merge.

I've updated the UPDATING.md, please let me know if it is clear. My technical writing is not one of my qualities. Also rebased against master.

One minor point to add to the updating docs, and I think it should be called out explicitly in UPDATING.md that to use S3Log or GCSLogs you will need a custom logging config -- otherwise people would miss it unless they checked the comment in the default template which they might not.

@ashb Thanks, I've made it more explicit.

@ashb Can you do a suggestion?


Search and Scrape Media Confirmation Dialog Scales to Full Monitor Width

Version: 2.9.13
Build: 2018-07-23 18:42
OS: Windows 10 10.0
JDK: 1.8.0_191 x86 Oracle Corporation
What is the actual behaviour?
When scraping metadata/images using the force best match, the dialog boxes are nearly full monitor width.  On a 2560x1080 display it appears to be resized to about 10 pixels on either side of the monitor edge.
It doesn't matter if hte parent TMM window is scaled beforehand to about 800x600 beforehand.
What is the expected behaviour?
I wouls suggest that the scrape metadata window would be located centre aligned with the parent window, perhaps 1280x1024?
Steps to reproduce:
Open TMM 2.9.13.
Click scraper (force best match option).  window opens full width.
Click scrape (scrape selected movies) window resizes ok
Additional
Have you attached the logfile from the day it happened?
No logs, I can do screenshots if required.

tmm remembers the window settings..
just delete the file data/tmm.prop in your tmm folder and everything should be resetted

Confirmed.
Thank you.


Remove GradientTexture2D

Merged in Godot 3.x now, see godotengine/godot#54824, therefore mission accomplished. 🙂

Note that godotengine/godot#54824 introduces a regression in relation to Goost features, waiting until godotengine/godot#56603 is merged before removing the class in Goost.

Lets merge the PR I guess, even if this means introducing a regression with Light2D (I no longer have maintainer privileges in Godot to fix the bug promptly, unfortunately). Besides, it's not currently possible to build Goost on latest Godot 3.x branch (duplicate symbols caused by merging the same feature in Godot).
But hopefully godotengine/godot#56603 gets eventually merged, so we won't have to patch the Godot source for this.


mutter: Enabling rt-scheduler fails

Describe the bug
It fails with the Failed to set RT scheduler: Cannot invoke method; proxy is for the well-known name org.freedesktop.RealtimeKit1 without an owner, and proxy was constructed with the G_DBUS_PROXY_FLAGS_DO_NOT_AUTO_START flag message. I have rtkit enabled because of pipewire.
Steps To Reproduce
Steps to reproduce the behavior:

Enable rtkit
Enable the rt-scheduler experimental feature of gnome-shell

xserver.desktopManager.gnome.extraGSettingsOverridePackages = [ pkgs.gnome.mutter ];
				extraGSettingsOverrides = ''
					[org.gnome.mutter]
					experimental-features=['rt-scheduler']
				'';
			};

Get the error message upon reboot.

Expected behavior
It works.
Notify maintainers
@jtojnar @bobby285271 @maxeaubrey (no idea how to ping the rest)
Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
 - system: `"x86_64-linux"`
 - host os: `Linux 6.2.2-zen1, NixOS, 23.05 (Stoat), 23.05pre460365.3c5319ad3aa`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.13.3`
 - channels(root): `"nixos"`
 - channels(l911p): `""`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixos`

How did you enable rtkit? Are you sure you have security.rtkit.enable  = true; in your configuration?
On Mon, Mar 6 2023 at 20:26:27 -08:00:00, Luis O.
@.***> wrote:

Steps to reproduce the behavior:
Enable rtkit
Message ID: @.***>

@L911P I've added this to my flake.nix
+                  services.xserver.desktopManager.gnome.extraGSettingsOverridePackages = [ pkgs.gnome.mutter ];
+                  services.xserver.desktopManager.gnome.extraGSettingsOverrides = ''
+                                                       [org.gnome.mutter]
+                                                       experimental-features=['rt-scheduler']
+                                               '';

and it looks like its working

you can have a look my base configuration
https://github.com/kirillrdy/dotfiles/blob/master/flake.nix
maybe this will help you to investigate this further

can you do "journalctl --boot=0 | grep scheduler" and see if you get the same message as i do on the title

can you do "journalctl --boot=0 | grep scheduler" and see if you get the same message as i do on the title

yes you right, i get same error
Mar 10 06:19:07 osaka .gnome-shell-wr[1851]: Enabling experimental feature 'rt-scheduler'
Mar 10 06:19:07 osaka .gnome-shell-wr[1851]: Failed to set RT scheduler: Cannot invoke method; proxy is for the well-known name org.freedesktop.RealtimeKit1 without an owner, and proxy was constructed with the G_DBUS_PROXY_FLAGS_DO_NOT_AUTO_START flag


Added support for FontVariant & TextDecorationLine

Hi @eneskarpuz
I just updated the Readme file :)
Thanks for your quick answer. I am new in open-source projects, please forgive if I did any mistake

Hi @eneskarpuz
I just updated the Readme file :)
Thanks for your quick answer. I am new in open-source projects, please forgive if I did any mistake

Please review and merge the update 😃

Please review and merge the update 😃

@eneskarpuz please review my pr and consider merging it, thanks :)


Json.Parse Telegram getUpdate

I haven't parsed in Json. need to parse following. Apps-script code will be highly appreciated.
https://api.telegram.org/bot462465654:AAGVcamJZhXjlwCe2R3qAGHsMXTcNtY2hGY/getupdates

@BlackZero969 you probably posted in wrong repo ) Sorry, I'm closing this.


Package Request: Nody-Greeter

Project description
describe the project a little
Nody-Greeter is a highly customizable LightDM greeter that can be themed with HTML, CSS, and JavaScript. There are several user-created themes made to work with the greeter. If I can include a theme in this issue, it would be https://github.com/JezerM/Aether (GPL v2). Please let me know if I need to make a separate request for the theme.
Metadata

homepage URL: https://github.com/JezerM/nody-greeter
source URL: https://github.com/JezerM/nody-greeter
license: GPL v3
platforms: Linux

Since this has gotten virtually no attention in months, and I've switched away from using graphical display managers, I'm closing.

I'm currently working on this. Can we reopen this issue, as it was never truly completed?


gitignore: add .venv

@GuillaumeCisco Can you update your PR as suggested by Clément ?
We will be able to merge it quickly as it is a minor modification ! 🚀

This is @AurelienGasser PR, I've just transferred it to the new repo.

Okay, sorry for the notification :)
Waiting for Aurelien !

Updated along Clement note !


Chime Setup on Cloud 9

Hello everyone!
I am currently trying to deploy this app on cloud 9 - I have managed to create the stack as well as deploy it successfully and have been getting the URL of the deployed app on cloud formation stack - However its throwing the error as follows ;
{"message":"Missing Authentication Token"}
Can someone guide me on this as I've been stuck on this for quite a few days  and would really love to have this working on my cloud9 server - The deployment guide isn't as good as I thought it would be.

Hi @misterharshil ,
Thanks for reporting the issue. I have removed your URL from the issue as you should not share your deployment URL to open source here.
The "Missing Authentication Token" can be a general error from API Gateway:
https://aws.amazon.com/premiumsupport/knowledge-center/api-gateway-authentication-token-errors/
Which demo you are having issue with?

Hi @devalevenkatesh  Thank you for your response!
I have tried the solution that you showed; It doesn't fix it - I am having issues with this https://github.com/aws-samples/amazon-chime-sdk/tree/main/apps/chat
I am following the deploying part as well as the creating stack mentioned in the solution.


Fix service update --env-add issue

- What I did
This fix tries to address the issue in #25404 where updating environmental variable in service update --env-add will not work.
The issue is because --env-add will only append the env, not update if the same env already exist.
- How I did it
This fix tracks the env variable with a map and update if the variable is the same.
- How to verify it
An integration test has been added.
- Description for the changelog
- A picture of a cute animal (not mandatory but encouraged)
This fix fixes #25404.
Signed-off-by: Yong Tang<EMAIL_ADDRESS>

Seems like it's been a punctual error:
03:57:26 ---> Making bundle: validate-dco (in bundles/1.13.0-dev/validate-dco)
03:57:38 fatal: unable to access 'https://github.com/docker/docker.git/': Couldn't resolve host 'github.com'
Can you trigger the build again manually?

Ping @dnephin @vieux!

LGTM

retest this please

LGTM


secret weight tweaks + nukie name fix

About the PR
fixed nukie names being some dumb shit
tweaked some secret weights
Why / Balance
nukie, rev and survival too high
made fun gamemodes more common

POV Heretic mains when they figured out why its so unlikely to become one

POV Heretic mains when they figured out why its so unlikely to become one

i didn't even touch heretic chance


Update go-iptables library version

From 0.4.1 to 0.4.5.
In version 0.4.1, no error is returned by go-iptables when running
iptables --version or parsing its ouput fails (during
initialization). This leads to the library not being able to correctly
detect whether the iptables version supports --wait, which ultimately
can lead to a deadlock for the Antrea agent.
See https://github.com/coreos/go-iptables/pull/69.
By updating the go-iptables version, we ensure that any such error will
be returned to Antrea, logged, and cause the Antrea agent to fail and
eventually restart.
It is unclear what can cause iptables version detection to fail but
because of the added logging, we will have a better shot at getting to
the root cause if it happens in production again.
Fixes #871

/test-all

But can it fix/close #871 which seems not caused by iptables failure?

@tnqn #871 is actually caused by an iptables failure, in the sense that we try to acquire the iptables lock twice, which causes the agent to deadlock and initialization never completes (which means that we never start the apiserver and cannot reply to readiness probles).

/test-conformance


Exception: Two classes were generated with the same name

Bug description
Code generation fails with: Exception: Two classes were generated with the same name
This occurs because a node has two different children with the same exact type.
In the example query below, friends and notfriends have the same type friendConnection
The generated class name for both is:
UserFriends$Query$Node$User$FriendsConnection$User
Specs
Artemis version: 5.1.0

build.yaml:
artemis:
        options:
          schema_mapping:
            - schema: graphql/km-schema.graphql
              queries_glob: lib/**.graphql
              output: lib/graphql/gen/graphql_api.dart
              # Naming options discussed here: https://github.com/comigor/artemis/pull/96
              naming_scheme: pathedWithFields
              type_name_field: __typename

GraphQL query:
node(id: $id) {
        ... on User {
            id
            __typename
            friends(first: 10) {
                nodes {
                    id
                    name
                }
            }
            notFriends(first: 10) {
                nodes {
                    id
                    name
                }
            }
       }
}

Hello, @MattGson!
We're currently focusing the development on beta branch (prerelease versions). Could you try to use the library with the latest beta version and report back, please?

I'm getting a similar issue today. Here is our type: likedList and profiles have the same type [FullProfile!]
We are using artemis: ^7.11.0-beta.1
type MatchProfileList {
    groupInfo: GroupInfo
    likedList: [FullProfile!]
    myInterests: MyInterests
    newGroup: Boolean
    profiles: [FullProfile!]
    totalFirstChance: Int
    totalLikedMe: Int
    totalSecondChance: Int
}


File reader

The purpose of this PR is

[ ] Add the feature that bots can work with files

translated_for (3).txt

translated_test.txt
test.txt


Ender IO and chests

If you have an Ender IO next to a chest and you open the chest through the Ender IO, even after closing the chest remains open. Using EnderIO 1.7.10-<IP_ADDRESS>1 and latest Forge.
https://www.youtube.com/watch?v=L9O3O1NooBY&feature=youtu.be

Duplicate of #15 ...


Enables building EFTools using Visual Studio 2017

Fixes all the issues I have encountered so far with running BuildEFTools.cmd /t:build on a Developer Command Prompt for VS 2017 RC.3. After this I still get a couple of warnings when building EntityFramework.PowerShell.csproj (but this could be unrelated):

C:\Program Files (x86)\Microsoft Visual Studio\2017\Enterprise\MSBuild\15.0\bin\Microsoft.Common.CurrentVersion.targets(2493,5): warning MSB3283: Cannot find wrapper assembly for type library "EnvDTE
". Verify that (1) the COM component is registered correctly and (2) your target platform is the same as the bitness of the COM component. For example, if the COM component is 32-bit, your target pla
tform must not be 64-bit. [C:\Users\dbveg\Documents\GitHub\EntityFramework6\src\EntityFramework.PowerShell\EntityFramework.PowerShell.csproj]
C:\Program Files (x86)\Microsoft Visual Studio\2017\Enterprise\MSBuild\15.0\bin\Microsoft.Common.CurrentVersion.targets(2493,5): warning MSB3283: Cannot find wrapper assembly for type library "VSLang
Proj". Verify that (1) the COM component is registered correctly and (2) your target platform is the same as the bitness of the COM component. For example, if the COM component is 32-bit, your target
platform must not be 64-bit. [C:\Users\dbveg\Documents\GitHub\EntityFramework6\src\EntityFramework.PowerShell\EntityFramework.PowerShell.csproj]

Superseded by https://github.com/aspnet/EntityFramework6/pull/200.


Fix regression in AbsentUfsPathCache

Cherry-pick  committed PR #12956 into target branch branch-2.3
Add a time stamp to indicate last refreshed time on the absent cache
entries.
Use absent cache to avoid UFS accesses when possible in the metadata
sync and load Metadata processes.
Added more comprehensive integration tests that ensures the number of
accesses to the UFS through SleepingUnderFileSystem and unit tests for
the absent cache.
fixes #12833 because now instead of checking /a, /a/b, /a/b/c for non
existent directories, we will only access the UFS once for /a because of
absent cache. We know /a/b, /a/b/c can not exist on the UFS.
Co-authored-by: David Zhu<EMAIL_ADDRESS>pr-link: Alluxio/alluxio#12956
change-id: cid-1c836c95416214ca0256a0d62feafe78d7cf8489

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-PR-Builder/11458/

alluxio-bot, merge this please


youtube-music: 1.19.0 -> 1.20.0

Changelog: https://github.com/th-ch/youtube-music/blob/master/changelog.md#v1200
Closes #232615
Description of changes

Things done

Built on platform(s)

[x] 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/)
23.05 Release Notes (or backporting 22.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

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 232951 run on x86_64-linux 1

1 package built:

youtube-music


DocsArchive-<IP_ADDRESS>Auto archive content

Please don't merge this PR until target archive repo PR https://github.com/OPS-E2E-PPE/docs-archive-test-target_d1d1a13e-f7a7-420b-bfa4-b80a81415da8/pull/1 is merged into live branch.
Auto archive content to https://github.com/opstest2/docs-archive-test-target_d1d1a13e-f7a7-420b-bfa4-b80a81415da8.git

Docs Build status updates of commit cd04299:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error-UserAccountBuildPermissionInsufficientError] Owner v-alji no longer has Read permission of the repository https://github.com/opstest2/docs-archive-test-source. It might be due to recent GitHub changes on SSO. Please ask the owner v-alji to logon to Docs Portal https://ops.microsoft.com once again. If it is not the case, please ask anyone of Admin permission of the repo to assign a new owner via Docs Portal https://ops.microsoft.com

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


[I/O Site] Mobile: when initially loading the landing page, the carousel layout is not mobile optimized

We should better optimize the responsiveness of the carousel on load.
Steps to reproduce

load the I/O home page on a mobile view
see it takes the carousel 1 - 2 secs to snap to the responsive size.

Screen shot

@jpvajda JIRA ticket CXPUI-38 is created and team will check on this. When i tried to reproduce screen loaded fine, will try to reach out if incase requires any help. for time being this PR can be closed.


Help synthesize Prop and boolean expressions

Prerequisites

[x] Put an X between the brackets on this line if you have done all of the following:

Checked that your issue isn't already filed.
Reduced the issue to a self-contained, reproducible test case.

Description
It is surprising that something that looks so simple doesn't work.  I think folks from other
programming languages would be very surprised by this, can we fix it?
#eval (5 > 2) == true

Similarly this also fails:
#eval (5 > 2) = true

saying
failed to synthesize
  Decidable ((5 > 2) = (true = true))

Steps to Reproduce
see above
Expected behavior: [What you expect to happen]
true

Actual behavior: [What actually happens]
failed to synthesize instance
  BEq Prop

Reproduces how often: [What percentage of the time does it reproduce?]
Versions
You can get this information from copy and pasting the output of lean --version,
please include the OS and what version of the OS you're running.
Additional Information
Any additional information, configuration or data that might be necessary to reproduce the issue.

@lovettchris

#eval (5 > 2) == true

This does not work because Lean is inferring Prop for both sides (as the 5 < 2 : Prop comes first) and there is not (and theoretically cannot be) a BEq instance for  Prop. However, the inverse, #eval true == (5 < 2) , works fine (as Lean then infers Bool for both sides which solves the problem).

#eval (5 > 2) = true

This one fails because Lean also lacks Decidable instance for Prop equality. As before, the inverse #eval true = (5 > 2) works. Here, though, such an instance could be made like so:
instance [d : Decidable (p ↔ q)] : Decidable (p = q) :=
  match d with
  | isTrue h => isTrue (propext h)
  | isFalse h => isFalse fun heq => h (heq ▸ Iff.rfl)

#eval (5 > 2) = true -- now produces `true` as desired

However, I suspect such an instance is missing because it has to rely on propext, which is an axiom.

thanks for the explanation, obviously I still have a lot to learn, but from what I've seen so far, I worry when a language has made itself so powerful that the simple things turn out being very complicated, and in some cases not even possible.  Perhaps Lean needs to strive a little harder to make the simple things simple and the complex things possible.  Right now it seems Lean is making the complex thing simple at the cost of some simple things are impossible.  This right balance takes true genius to discover sometimes...

@lovettchris the key Lean oddity here is simply that </> and <=/>= are propositional operations, not Boolean operations in Lean.
It is important to remember that while Lean is aiming to eventually become a general-purpose language, the theorem proving aspect is still key. Thus, it is more designed for people with a functional programming, mathematical logic, and/or type theory background who also want to do general-programming rather than the other way around (at least at the moment).  In essence, Lean's current design is to attract exactly the kind of people who want certain conventionally complex things made simple even at the cost of potentially make some simple things more complex.
However, a lot of this is just due to the fact that having an easy onboarding for the general-purpose side (with a rich stdlib and clean errors, etc.) is not the priority right now. Currently, afaik, the main priority is to get Lean in shape for Mathport (as mathlib is the core of Lean's current userbase) and develop the components necessary (not just desirable) for CS applications. Thus, many things (like this) which could be made more convenient, but is still currently serviceable, don't end up the focus.
Also, having a few oddities here and there is quite common for a programming language. For example in Python 0 == False and isinstance(False, int) which makes it very difficult to properly distinguish between False and 0 in a program whereas it is much easier to do so in many other OO interpreted languages (e.g., JS, Ruby, etc.).

also, I am surprised I hadn't thought to ask of this until now, but @lovettchris what is the use case for this? In some ways the fact that #eval x == true/#eval x = true does not work is actually a blessing in disguise, as comparing a Boolean-like value to true is almost always an error / bad code smell (e.g., the kind of thing linters would yell at you for).

However, I suspect such an instance is missing because it has to rely on propext, which is an axiom.

@tydeu This is an oversight. We should add this instance. Thanks for catching the problem. Could you please submit a PR for it?
For the #eval (5 > 2) == true, we can write a custom elaborator for it that does something special for the Prop case. I think the Prop vs Bool issue is an obstacle for many new users. The custom elaborator is a bit ad-hoc, but I think it is defensible for this recurrent issue.
Note that, the == notation already uses the special binrel% auxiliary elaborator that makes sure that examples such as
variable (x : Nat) (y : Int)

#check x == y
#check y == x

work.

@leodemoura

@tydeu This is an oversight. We should add this instance. Thanks for catching the problem. Could you please submit a PR for it?

certainly, will do


Respect meta.json order

I'm trying to set up a custom order for my docs files; I thought order is important but looks a different order is considered.
In that case, I want to set up "Welcome" as the first entry.
Any tips for doing that?

There's a typo "Templates": "templates" in your meta.json, which should be "templates": "Templates". If a route is undefined in the config it will show up at the very top :D

🤦 thanks @shuding!


[BUG] Async queries should immediately cancelled on button click

What is the bug?
We see cases when user clicks on the cancel button, the async query loader is still spinning.
How can one reproduce the bug?
Steps to reproduce the behavior:

Create an async query on workbench
Cancel any query in running or failed state.

What is the expected behavior?
The cancel button should immediately stop the queries.
What is your host/environment?

OS: 2.11
Version 2.11
Plugins Workbench

@ps48 are we still tracking 2.12 on this?


KeyError: 'layer1.0.bn1.running_mean'

I am using resnet50-ibn-a to train and got this error. My config is as follow:
MODEL:
DEVICE: cuda
DEVICE_ID: 3,4
IF_LABELSMOOTH: on
IF_WITH_CENTER: yes
LAST_STRIDE: 1
METRIC_LOSS_TYPE: triplet_center
NAME: resnet50_ibn_a
NECK: bnneck
PRETRAIN_CHOICE: imagenet
PRETRAIN_PATH: /data/wwl/Pretrain/resnet50_ibn_a/resnet50-19c8e357.pth
OUTPUT_DIR: /data/wwl/log/BOT
The path is correct, pretrained model is the same with resnet50.

PRETRAIN_PATH: /data/wwl/Pretrain/resnet50_ibn_a/resnet50-19c8e357.pth


window.top & window.parent

title says it all. listings for:
window.top
https://developer.mozilla.org/en-US/docs/Web/API/Window/top
window.parent
https://developer.mozilla.org/en-US/docs/Web/API/Window/parent
Thanks for making this awesome project.

These have been around since ancient time browsers, FYI.

I figured that they have been around for a very long time, just was not sure if nested windows were supported pre-IE8.  I saw that window.location has a similar description of "supported in effectively all browsers": http://caniuse.com/#search=window.location

Available at https://caniuse.com/#feat=mdn-api_window_top and https://caniuse.com/#feat=mdn-api_window_parent


snip not work

1, seems somewhere did not set for snip compl item:
item: {'label': 'for', 'data': {'type': 2}, 'kind': 15}

e.g: no insertTextFormat is 2 found.

perhaps should be using 'prefix' as its label, or following patch should be required.

diff --git a/server/snippets.json b/server/snippets.json
index f7ea08b..aa3bdab 100644
--- a/server/snippets.json
+++ b/server/snippets.json
@@ -39,13 +39,13 @@
         "description": "! operator",
         "prefix": "not",
         "body": [
             "!${1:expression}"
         ]
     },
-    "compare to": {
+    "compare-to": {
         "description": "comparison operator",
         "prefix": "compare-to",
         "body": [
             "${1:expression} ${2|<,>,<=,>=,==,!=|} ${3:expression}"
         ]
     },
@@ -68,13 +68,13 @@
         "prefix": "if",
         "body": [
             "if (${1:condition})",
             "\t$0"
         ]
     },
-    "if else": {
+    "if-else": {
         "description": "if else operator",
         "prefix": "if-else",
         "body": [
             "if (${1:condition})",
             "\t${2:print}",
             "else",
@@ -87,13 +87,13 @@
         "prefix": "while",
         "body": [
             "while (${1:condition})",
             "\t$0"
         ]
     },
-    "do while": {
+    "do-while": {
         "description": "do while operator",
         "prefix": "do-while",
         "body": [
             "do",
             "\t$0",
             "while (${1:condition})"

Are you sure your editor supports snippets as part of LSP?

yes. bashls had done the same.
--
shane.xb.qian

What editor version do u use? Maybe I can try reproduce bug.

yea, i tried to debug with following, but seems difficult to make it be right though i got the snip items if so.
diff --git a/server/src/completion.ts b/server/src/completion.ts
index 1c53179..eabfd22 100644
--- a/server/src/completion.ts
+++ b/server/src/completion.ts
@@ -73,15 +73,19 @@ export function initCompletionList(docs: Documentation, snippets: Snippets): voi
       data: { type: DataEntryType.Documentation, jsonPath: `patterns.${key}` },
     })),
   )

   predefinedCompletionListLight.push(
     ...Object.entries(snippets).map(([title, info]) => ({
-      label: title,
+      label: info.prefix,
       kind: CompletionItemKind.Snippet,
       data: { type: DataEntryType.Snippet },
+
+      detail: info.description,
+      insertText: info.body.join('\n'),
+      insertTextFormat: InsertTextFormat.Snippet,
     })),
   )
 }

 export function getPredefinedCompletionItems(): AWKCompletionItem[] {
   return predefinedCompletionListLight


[Bug]: .npmrc registry is not sanitized

What happened?
If your .npmrc has the following
registry = 'https://url.com'

this rule will take ' and not remove the strings. This will cause installs to fail.
Version
Development (host) and target OS/architectures: Ventura 13.2
Output of bazel --version: 6.0.0
Version of the Aspect rules, or other relevant rules from your
WORKSPACE or MODULE.bazel file:
Language(s) and/or frameworks involved:
How to reproduce
add a registry to .npmrc that is equal to a string

Any other information?
Maybe surface a validation message. Took me a while to find this. Had to bazel fetch the target to find the failure
Fund our work

[ ] Sponsor our open source work by donating a bug bounty

Example of the error log
 Bad URL: 'https://url.com/'/ansi-regex/-/ansi-regex-5.0.1.tgz

@jbedard Did you recently make a similar change to be tolerant of double quotes "?

Yeah, I guess this is the exact same but single quotes... 👍


Update README.md to mention how to use import

GitHub issue https://github.com/googleapis/cloud-trace-nodejs/issues/1398 indicates that a specific incantation is required when using "import" with the trace-agent. This PR updates the README to mention this.
Fixes #1398  🦕

Looks like I might need to do some tricky things to update README.md. I'll look into that, but in the meantime a review of the text edit for clarity and correctness would be appreciated.
@davidbielik

Abandoning this PR, I created a separate one tied to the issue.


[CAD-2060] Adding a SMASH executable alongside db-sync.

https://jira.iohk.io/browse/CAD-2060
The added executable that allows SMASH to be run alongside db-sync.
Synced up to Mary on mainnet.
Two things still missing:

re-fetching of the unavailable metadata, but that can be added easily after we fix the release of smash
fetching of the unique tickers (it was added recently), but we need to create a new release of smash and deploy that

Disabled the Github Cabal building only dependencies, because of an issue I run into - https://github.com/input-output-hk/cardano-db-sync/pull/551/files#diff-c09cc8dc973415b1221308559b96942a1c4b872e690bf360fc22d9b42822d519R140

The biggest problem here is that smash currently uses a separate database to the one used by db-sync. That means that it is not possible to write a query that joins tables maintained by smash and db-sync .
There is a stack overflow question about this and there does not seem to be an easy solution, other than putting everything in the same database.

The biggest problem here is that smash currently uses a separate database to the one used by db-sync. That means that it is not possible to write a query that joins tables maintained by smash and db-sync .
There is a stack overflow question about this and there does not seem to be an easy solution, other than putting everything in the same database.

We are talking about different schemas here. So I don't see an issue with that one.
That query is quite simple - https://stackoverflow.com/questions/1463849/querying-across-schemas-in-postgres

Closing this.


Unsupported expression pattern in destructuring.

I have this in my code, which works:
[this.data] = this.convertedData.CxF
But flow gives me the error in the title.

I can confirm that this is a bug per the spec. MemberExpressions satisfy IsValidSimpleAssignmentTarget which is the relevant criterion for accepting a LeftHandSideExpression as a valid DestructuringAssignmentTarget.
In Flow, destructuring.ml seems to have special handling for Identifiers in this position but not for the other handful of valid DestructuringAssignmentTargets.


[QUESTION] Set retention to 0 hours and play video

I'm trying to set a stream with a retention of 0 and play the live video in the aws console, but I'm getting an error about the retention being 0.

This should be possible according to https://docs.aws.amazon.com/kinesisvideostreams/latest/dg/API_CreateStream.html :
When the DataRetentionInHours value is 0,
consumers can still consume the fragments that remain in the service host buffer,
which has a retention time limit of 5 minutes and a retention memory limit of 200 MB.
Fragments are removed from the buffer when either limit is reached.

The stream is successfully created and the retention is correctly set to 0. The stream is not set in offline mode in the producer library (as that returns a clear error).
The reason behind this setting is to minimise the cost of the BytesHr metric.

Hi @lucabergamini,
Thanks for bringing this to our attention. We will be working on updating the documentation for this.
GetMedia is the only API where you can consume media from a stream with 0 retention. The AWS console uses the DASH, which requires retention > 0.
Note that if you are looking to view live media via peer-to-peer streaming (e.g. from a security camera on your phone), we'd recommend checking out Kinesis Video Streams with WebRTC to see if it fits your use case.

Hi @sirknightj
Thanks for the clarification on this. I still have a question though. We are using HLS in our case but to play videos in the browser we don't go through GetMedia but through GetHLSStreamingSessionURL. Does that mean we can't use 0 retention in this case?

Correct. HLS playback requires retention > 0.

Got it, thanks a lot!


adding new updated pacakge for fixes related to generate.ps1

Can this package be deleted from the repo yet?


[Fatal Error] testng-1.0.dtd:1:3: The markup declarations contained or pointed to by the document type declaration must be well-formed

Getting following error while parsing the testng.xml file. I am using the latest testng version: 6.14.3.
[Fatal Error] testng-1.0.dtd:1:3: The markup declarations contained or pointed to by the document type declaration must be well-formed.
org.xml.sax.SAXParseException; systemId: http://testng.org/testng-1.0.dtd; lineNumber: 1; columnNumber: 3; The markup declarations contained or pointed to by the document type declaration must be well-formed.
at com.sun.org.apache.xerces.internal.parsers.DOMParser.parse(DOMParser.java:257)
at com.sun.org.apache.xerces.internal.jaxp.DocumentBuilderImpl.parse(DocumentBuilderImpl.java:339)
at javax.xml.parsers.DocumentBuilder.parse(DocumentBuilder.java:205)

@sanoj-s - The latest released version of TestNG is 7.0.0 please retry using that and if this is still a problem, please include your suite xml file contents as well.

Hi,
I a facing the same issue and i am using the latest TestNG version 7.0.0 ;
I noticed that the issue is happening when trying to parse the following .xml file
"

"
here is the code where it is failing:
File fXmlFile = new File("testcase.xml");
            DocumentBuilderFactory dbFactory = DocumentBuilderFactory.newInstance();
            DocumentBuilder dBuilder = dbFactory.newDocumentBuilder();
            Document doc = dBuilder.parse(fXmlFile);// **<<<< it is failing here**

I fixed the issue by completely removing all installed JDKs and fresh install the latest one, following the following URL https://www.java.com/en/download/help/mac_uninstall_java.xml

This issue has been fixed via commit https://github.com/cbeust/testng/commit/f895124a0b46de1dc2f09c8f24503f3712694a21 and should be available in the upcoming version of TestNG


Issue with a custom SelectWidget used within an Array field

Prerequisites

[V] I have read the documentation;
[ ] In the case of a bug report, I understand that providing a SSCCE example is tremendously useful to the maintainers.

Description
I implemented a custom SelectWidget which allows the user to search with a search bar inside the options (useful when you have a lot of options).
This custom widget isn't working when used inside an array with various instances of it. Its functionalities are working (options search, options selection and propagation to formData via props.onChange callback), but up/down buttons are messing up its state and delete button always deletes the last item of the array (not the item where I actually click the X).
Steps to Reproduce

Implement a custom SelectWidget.
Replace default SelectWidgets with this.
Those widgets should be used by a schema which has an array of them. Place some widgets in various array items and try to play with up/down/delete buttons.

Expected behavior
Array up/down buttons and delete functionality should work correctly.
The issue seems to be linked to the "key" property that React uses for re-renders.
       const widgets = {             SelectWidget: this.searchableInput         }
   searchableInput = (props) => {         return <SearchableInput key={props.id} {...props}  i18n={this.props.i18n}/>     }
class SearchableInput extends React.Component {     constructor(props) {         super(props)         this.state = {             fields: {                 selected: props.value,                 search: '',                 data: props.options.enumOptions.map((field) => (field.value))             },  ...       }     }
If I set it to a random value "key = {Math.random()}" array functionalities are working but it messes up with the state of my custom components (since the Form is recreating them from props, then the state get reset).
Setting the key to props.id isn't working either.
Actual behavior
Array up/down button mess up with my custom widget, delete button always deletes the last element of the array.
Version
"react-jsonschema-form": "^1.0.5"

@Muntaner can you please provide a reproducible example on codepen or jsfiddle?


Add Cariro4

@marwanzayed-coder Regrettably, a lot of Egypt-based schools' email domains were misused recently. Please apply for free educational license with an official document which proves that you are a student. You can do it using "OFFICIAL DOCUMENT" tab at https://www.jetbrains.com/shop/eform/students.
I have to close the request without merging.
Thank you for your understanding!


Use rustdoc::broken_intra_doc_links

Building the docs emits warning:
warning: lint broken_intra_doc_links has been renamed to rustdoc::broken_intra_doc_links
Use rustdoc::broken_intra_doc_links as suggested.

Does it work on Rust 1.29.0 or 1.36.0?

Does it work on Rust 1.29.0 or 1.36.0?

ooo, my bad - I did not check this.
I can't build master with 1.29.0, do you know what am I doing wrong?
cargo +1.29.0-x86_64-unknown-linux-gnu check

I believe just cargo +1.29.0 check should work if you have that version installed and you use rustup. There's also a PR to bump MSRV to 1.36 (#635) so depending on whether it works with 1.29 this can go before or after it

cargo +1.29.0 check

Cool, thanks. This PR does not work with v1.36.0 - closing it. Sorry for the noise.


Retain cycle in SFBAudioPlayerNode prevents deallocation

There is a retain cycle in SFBAudioPlayerNode that prevents the object from being deallocated when it's removed from the audio processing graph.
At least one cause is the use of self in the render block but there are likely other culprits as well.

Fixed in #297


Chapter 1, Custom Styles

<style>
  {`
    a, a:focus {
      font-weight: 400;
      color: #1565C0;
      text-decoration: none;
      outline: none
    }
</style>

The provided code snippet is missing the trailing backtick before the closing brace :)

thanks @whitefire0 . Can you point out exactly where the back tick is missing?
In the code, this <style> element includes additional styles that are closed by a final back tick. See here:  https://github.com/builderbook/builderbook/blob/457b1c37464e2311bddb75d30a8e06c89a653448/book/1-end/pages/_document.js#L64

@whitefire0 can you confirm if my last comment answers your issue? Thanks

@klyburke I should have been more specific in my OP.
The missing back tick is in the book itself, at this location:
http://prntscr.com/kvtfdu

Thanks @whitefire0.
I clarified this in the book by adding a line to indicate that there is additional code before the closing <style> tag.
<style>
  {`
    a, a:focus {
      font-weight: 400;
      color: #1565C0;
      text-decoration: none;
      outline: none
    }
  // some code
</style>

We would not put the backtick yet, as the final bracket in the code snippet corresponds to the one right after a:focus, not the very first bracket after <style>.


'ImageEx' does not contain a definition for 'ImageFailed'

This has changed in Version 1.5.0 - looks to have been as a result of commit: https://github.com/wasteam/waslibs/commit/49ab82ef17179bf5b1ad30209872fe8662f81ceb
If you set the Source of an ImageEx control to a Uri that doesn't exist, you don't get an exception, but nothing is displayed.
How can you use an ImageFailed function with the ImageEx control?

The ImageFailed event has been included in DEV branch (commit). It will be merged in our next release.

Closing this as no longer relevant (using the Windows Community Toolkit now).