Weird issues

Values such as light_diameter and time won't change, after setting it it will go back to original for some reasons :/

There is a different reason for both of those variables.
For time, it is a matter of who owns the variable, the client or the server. When Cube World starts, it creates both a client and a server, then connect to itself (even in singleplayer mode). Because of this, you end up with two cube::World instances in memory. One at game->world, and the other at game->host.host_world. The host constantly sends it's time for the clients to use, so to change the time, you need to set game->host.host_world.time_info.time.
As for the light_diameter, the skill/stat values are constantly recalculated every game frame based on the equipment/artifacts you have, so it's not really feasible to change it unless you hook the code that is calculating it.

How hard would it be to hook the code that's calculating it ?

I think that depends on your experience using a disassembler and writing x86_64 assembly code. You would first need to find the code that is writing to that variable, then insert a JMP statement to jump to your assembly stub that would handle setting up the stack and calling your C function.  An example of this would be ChrisMiuchiz's modloader code : https://github.com/ChrisMiuchiz/Cube-World-Mod-Launcher/tree/master/CubeModLoader


optimize config_flow by adding options

adds another config step where user can decide which switches should be exposed

[x] locally tested to work

closes issue #61

I am not entirely sure if we decided to use Fritz or FRITZ. Now its FRITZ everywhere in the config_flow

Thanks @AaronDavidSchneider!
I see this being useful for a couple (although only few) people. But I'm not quite sure whether this justifies cluttering the config flow.
It's quite common for integration to create entities for everything it supports.
TBH the reasoning in the mentioned issue is quite shallow: On larger HA setups, people tend to whitelists what to expose to other cloud services (because otherwise they're useless anyways with dozens of bad named entities), if people are annoyed by too many entities in their UI they probably create their own Lovelace config anyways.
Besides, entities can be disabled via the Integrations UI (not sure whether we automatically support this?).
We're creating the stuff asynchronous, so the startup shouldn't be affected too much. Obviously it takes resources to update the entities, but it's probably not worth mentioning.
So, the question I want to rise is: Does it really make sense to add complexity to the code and the configuration because a few people want some entities to not be created (because they don't want to use the concepts already present in HA (disabling entities, whitelisting entities for external exposure)).

Thanks for your comment @mammuth!
This PR doesn't clutter the config flow too much! Its at most one simple additional form.
If you don't use device profiles the config flow skips the last step.

I would say its worth it.  Its at most one simple additional form.

True, I expected it to be way worse 🙈


Made default labels in plot_diagrams correspond to the number of diagrams

Currently the default labels in plot_diagrams only go from H_0 to H_8, which throws an error if there are more than 9 diagrams and you want to use "plot_only" for any higher indices. I've adjusted so that the default labels now account for the number of diagrams in the list of diagrams.

Neither did I @sauln , but it turns out that the directed flag complex of brain networks has some high dimensional homology groups!
I bumped the version and added a little note now.

Thank you!


TagBot trigger issue

This issue is used to trigger TagBot; feel free to unsubscribe.
If you haven't already, you should update your TagBot.yml to include issue comment triggers.
Please see this post on Discourse for instructions and more details.
If you'd like for me to do this for you, comment TagBot fix on this issue.
I'll open a PR within a few hours, please be patient!

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/65646

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/65851

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/65892

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/65900

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/65937

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/65945

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/67193

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/98117


Send welcome email when creating a user

This commit adds:

SMTP config to specify an SMTP server and account to use to send emails. No defaults are provided.

SMTP_SERVER_HOST
SMTP_SERVER_PORT
SMTP_USER
SMTP_PASSWORD

A function to send emails using the given SMTP settings
A welcome email will be sent out using the given HTML template (when SMTP settings are provided)

Example of email:

Nice. My only feedback is to maybe add a link in the email that the user clicks to confirm that they are authenticated with that email account, like with confirming emails through other sites I've used? Also you could make their new email address a link to their provide inbox. thanks

These sound like good features, but can be handled in a future PR.

This is nice, but it's in the wrong repo ;)


A question

I've been grappling with this issue of scheduling my app scheduler to run periodically once every day. However, I have tried many options - AlarmManager, GCMTaskService etc to no avail. Either these behave very unpredictably or don't work at all. Then I tried the recommended JobScheduler but it seems to have a problem that it will only kick off periodic jobs if the app is in the foreground. The periodic schedule doesn't fire if the app is relegated to the background or exited. I chanced upon your library today. Does it have the same limitation? Is there a recommendation to achieve what I want?

Version 1.2.0 will make this pretty easy: https://github.com/evernote/android-job/wiki/Version-1.2.0-and-Android-O#add-helper-class-to-run-jobs-daily

Thanks for the quick reply. I tried a sample program made available here https://github.com/rkpattanaik/AndroidJobDemo but it also did NOT work when the app was in the background or closed normally with a back button.
Does this DailyJob solve this issue? Will it work even when the app is normally closed and not in the foreground?

The problem in your sample is that you schedule the periodic job each time the onCreate() method of your activity is called. That cancels the old job and schedules a new one (with setUpdateCurrent(true). So your job never really has a chance to run.
Since this is not an issue in the library, I'm closing this issue here.

I had a specific query but closing this issue didn't help clarify. A few observations/comments on your last comment:
a. This sample code was NOT written by me. It was written by some blogger who advocated your library. :-)
b. I incorporated DailyJob exactly as you had indicated in your answer above. I set it up to run between 10am and 4pm. It didn't fire the job anytime during this period. Deliberately, I exited my app normally (using the log off button in my app), so I suspect the same issue exists where the scheduled jobs ONLY fire if the app is in the foreground and NOT if the app goes to the background or is exited normally without a force close.
I had a small piece of code to show the status of existing job requests by calling JobManager.getAllJobRequestsForTag and it indeed showed my job was scheduled but it never fired.
Do you mind explaining why it didn't work when the app was exited?

This is an issue tracker. Its purpose is to report defects in the library. Usually I try to help when questions come up and they're challenging, but in this case it's a very basic questions and there are better platforms like Stackoverflow, where you can ask for help.
What you're describing isn't helpful at all. If you think that you found a bug in the DailyJob class, then please share the code with a description how to reproduce the bug. I can't say what's wrong in your case with such a vague explanation. The code you've shared simply implements it wrong and cannot work.

ok thanks. I'll figure it out myself. I already mentioned the issue and I'm surprised you say my explanation is vague. Your DailyJob doesn't get triggered if the app is moved to the background. This is true with all job schedulers I've tried and yours is no exception. I thought you've addressed this issue somehow but looks like the issue persists. You can try the code yourself.

Which code? The sample above doesn't use the DailyJob class and contains bugs which I explained. That's why I said that your explanation is vague. Without a sample I must assume that you're doing something wrong like in the sample above. For us the DailyJob class is working fine in production with millions of daily users.
I say it again: If you found bug, then please create a sample for me, where it's easy to reproduce the bug so that I can fix it. Just describing what you did isn't enough.

ok, let me send you my sample code in some time...

Attached is my simple test application. I don't see any notification even if the app is open. Am I doing anything wrong? I have also attached the logcat output. Don't see any errors either.
src.zip
logcat.txt

Now here's an updated test application to test whether the periodic job works fine or not. As you see here in the log cat output, onRunJob was called twice and the corresponding notification was issued - once at 9:08 and again at 9:19 as long as my app was in the foreground. After 9:19, I relegated the app to the background by clicking the Home button on my Asus phone and using the phone app for calling. After this event, onRunJob has never been invoked and my notification has never been updated with a newer run. . This is exactly what I was complaining about earlier.
src.zip
logcat.txt

In your first app the problem is that you create the JobManager not in your application class.
Your second app looks better, but the log looks also fine to me.
09-02 09:19:35.541 6459-30625/com.finance.testapplication D/JobProxy21: Schedule periodic (flex support) jobInfo success, request{id=1, tag=SchedulerService, transient=false}, start 00:10:00, end 00:15:00, flex 00:05:00
09-02 09:21:05.959 6459-6459/com.finance.testapplication W/IInputConnectionWrapper: getExtractedText on inactive InputConnection
09-02 09:21:05.995 6459-6459/com.finance.testapplication W/IInputConnectionWrapper: getTextBeforeCursor on inactive InputConnection
09-02 09:21:05.997 6459-6459/com.finance.testapplication W/IInputConnectionWrapper: getSelectedText on inactive InputConnection
09-02 09:21:05.998 6459-6459/com.finance.testapplication W/IInputConnectionWrapper: getTextAfterCursor on inactive InputConnection
09-02 09:22:06.764 6459-6459/com.finance.testapplication W/IInputConnectionWrapper: getExtractedText on inactive InputConnection
09-02 09:22:06.765 6459-6459/com.finance.testapplication W/IInputConnectionWrapper: getTextBeforeCursor on inactive InputConnection
09-02 09:22:06.766 6459-6459/com.finance.testapplication W/IInputConnectionWrapper: getSelectedText on inactive InputConnection
09-02 09:22:06.770 6459-6459/com.finance.testapplication W/IInputConnectionWrapper: getTextAfterCursor on inactive InputConnection
09-02 09:29:45.917 6459-6469/com.finance.testapplication W/art: Suspending all threads took: 10.037ms

The job was scheduled 10 minutes before the log ends and there are 5 minutes left to run the job. Also with such a short interval it can happen that the job doesn't run every 15 minutes. Please keep in mind how Doze and other power saving modes work. I'm referring to this slide from a talk I gave: https://speakerdeck.com/vrallev/doo-z-z-z-z-z-e?slide=76
So far everything works expected.

Your job will never run every 15 minutes on newer Android devices. That's expected and a good thing.


Add documentation example on how to pass build arguments into the Groovy script

Context: the Lockable Resources plugin includes an option to specify the needed resource by one of: exact resource name, matching a label, or evaluating a Groovy script for each configured resource that would return true if this resource is okay for this job. Then one of the "okay" resources (assuming "Number of resources to request" setting is 1) is reserved while the build runs.
My use-case, now solved by code in PR #72, is that we have a test farm, where usually any of the systems can be used for a test (so label-matching is in place), except when developers want to run the job against some particular system - and protect it from being used by any other job during this time. The build job (or actually a Multiphase job calling several others) has build arguments, including specification of the environment to use for the test (a "CONTROLLER" in examples below).
For the test, I used a build of the plugin with PR above integrated (made by mvn package), and in the job I selected "Meta Data / This build requires lockable resources", further selected "Groovy Expression" and entered the script below. The "Resources" and "Label" fields were left empty, and the "Number of resources to request" was set to 1 (disregard the warning that Given amount 1 is greater than amount of resources: 0.), and "Reserved resources variable name" is LOCKED_CONTROLLER.
Note the println lines below end up in jenkins.log, and can be pretty noisy, so comment them away when your job definition works well :)
// We need an available testbed resource marked with label "rc-validation-farm"
// If the user specified a particular testbed name in CONTROLLER var, require that one instead (regardless of labels)

/*
println "Inspecting the resource to lock for requested CONTROLLER='" +
    CONTROLLER + "' (looking at resourceName='" + resourceName +
    "' resourceDescription='" + resourceDescription + "' resourceLabels='"
    + resourceLabels + "')"
*/

if ( CONTROLLER.startsWith("LOCK_LABEL:") ) {
    def LOCK_LABEL = (CONTROLLER =~ /^LOCK_LABEL:(.*?)$/ )[0][1];
/*
    println "Looking for LOCK_LABEL='" + LOCK_LABEL + "' among '" +
        resourceLabels + "' for '" + resourceName + "' (" + resourceDescription + ")"
*/
    if (resourceLabels.contains( LOCK_LABEL )) {
//        println "ACCEPTED '" + resourceName + "'"
        return true;
    }
} else {
//    println "Looking for 'rc:" + CONTROLLER + "' in the name '" + resourceName + "'"
    if (resourceName == ("rc:"+CONTROLLER) ) {
//        println "ACCEPTED '" + resourceName + "'"
        return true;
    }
}

// println "Resource '" + resourceName + "' is not suitable for this job"
return false; // Tested resource is not appropriate for this build

The numerous corresponding lockable resource definitions in Manage Jenkins define a name (like rc:controller1) and labels (like rc-validation-farm rc-model-3), and sometimes comments about nuances of the controller model.
The job build parameter CONTROLLER is a predefined Global Choice Parameter, which includes the names like controller1 that our devs can pick, and a default value of LOCK_LABEL:rc-validation-farm.
The script above was made generic enough to ease copy-pasting, so it reacts to the build arguments starting with LOCK_LABEL: and picks the rest of the string as the label to look for in resources; otherwise it would look for the requested resource name (with the prefix rc:).
As you could guess, there is a second side to the medal: the default CONTROLLER build argument is not usable as a definitive host name, so that build argument has to be replaced with the actually picked value :)
For this we use the EnvInject plugin, so in the job I further selected "Prepare an environment for the run" and entered an "Evaluated Groovy script" with:
def map = [:]

/*
println "Requested build arg CONTROLLER=='" + CONTROLLER +
    "'; the locked resource == '" + LOCKED_CONTROLLER + "'";
*/

if ("LOCK_LABEL:rc-validation-farm".equals(CONTROLLER)) {
    def CTLNAME = ( (LOCKED_CONTROLLER =~ /^rc:(.*?)$/)[0][1] )
    println "Extracted CONTROLLER:='" + CTLNAME + "'"
    map << [ CONTROLLER : CTLNAME ]
}

return map

(the script actually evaluates optional overrides for a few other variables as well, hence this structure and not a simpler one). Note that println here ends up in the build job's log.
Finally note, that for sub-jobs called from this one, you should also specify "Predefined parameters" including CONTROLLER=${CONTROLLER} so the one mapped above is in place for those jobs that would access it, and not the default token. The "Build on same node" and "Current build parameters" are useful for such cases as well :)

UPDATE: The pull request has been merged for a while now. Over the past couple of years I've referred a number of IRC discussions to this issue. I believe it has thus become a good basis for a documentation chapter about using this trick, so I rephrased the issue name to have it only closed after adding published documentation rather than keeping it a needle of know-how lost in the haystack.


make positive there is always a safeguard against duplicate adds

The tricky thing here is that what constitutes a duplicate may vary. If this is not done well nontermination will occur

In an undirected graph, edges are reversible for example

This is covered by save. Additionally, a directed graph will be more fun

As a last note - it should be made clear somehow not to save data that you don't want to circulate further


invoke integration-test fails unless invoke start is run first

Environment

Python version:
Nautobot version: next @ 934b6bf2d0607c7c4b04eddd86d4458bbe6e33ea

Steps to Reproduce

Ensure that no Nautobot Docker containers are running (invoke stop)
Run invoke integration-test

Expected Behavior
Integration tests to run successfully.
Observed Behavior
All Selenium tests fail with errors like:
selenium.common.exceptions.WebDriverException: Message: Reached error page: about:neterror?e=dnsNotFound&u=http%3A//nautobot%3A58679/login/&c=UTF-8&d=We%20can%E2%80%99t%20connect%20to%20the%20server%20at%20nautobot.

The issue appears to be that the nautobot_nautobot_1 container is not automatically started (although the nautobot_selenium_1, nautobot_redis_1, and nautobot_postgres_1 containers are started); instead the integration tests are run from a temporary container nautobot_nautobot_run, which is not DNS accessible under the name nautobot.
Running invoke start beforehand so as to ensure that the primary Nautobot container is explicitly started appears to be a workaround for now. (Note that running unit tests via invoke unittest does NOT require this workaround as the Selenium container isn't involved in those tests.)

@glennmatthews is this still a problem?

I believe so, yes.


proposal: strings: add First/Last function

I encountered many times the following scenario:
func First(s string, n int) string {
    if len(s) <= n {
        return s
    }
    return s[:n]
}

This simple function try to get at most N-bytes from a string.   Likewise, for at most N-bytes "tail" from a string:
func Last(s string, n int) string {
    if len(s) <= n {
        return s
    }
    return s[len(s)-n:]
}

These functions will many some programs easier to read, like the TrimPrefix/TrimSuffix functions.

Left ， Right ？

Name of the function is not important, it could be Prefix(), Suffix(), or LeftMost(), RightMost()

For prior discussion, see #53510

@dsnet I don't think my proposal has same intention as #53510.  The key difference is, in #53510, @ianlancetaylor asked:

Should First and Last panic for an empty slice?

The answer is "yes, in the same way that s[0] panics for an empty slice.".    But for this proposal, the key point is that I want to get the first (last) N bytes, if there is that much, or more, if the string is shorter, just give me all -- do NOT panic.   This has exactly the same rational as "strings.TrimPrefix()".
So, comparing to #53510, this is not merely a syntactic sugar, it is a shorthand for, what I shink, a common logic.

I want to get the first (last) N bytes

Why bytes? Bytes are efficient as index positions into strings, usually determined via Index* functions, but they are not suitable for counting. The concept of 'count' doesn't have an obvious meaning in Unicode strings because there are multiple layers to consider.

I'm afraid @gophun's objection is fatal. Anything you do (count bytes, count runes, count grapheme clusters) will be appropriate for some users and inappropriate for others. It does occur to me though that for counting bytes, you can now use min to make a one liner: s := mystr[:min(N, len(mystr)-1)].


Create barrington220.org

@saakethk02 There are a number of issues with your requests #2371 and #2372:

Files must have .txt extension instead of .org extension (they are already in org subdirectory, thus our system recognizes that the domains are subdomains of .org).
Files have not to be empty, they have to have a name of your school in the first line
Which email domain the school students use - barrington220.org or bsd220.org? Do you need to add both domains? If yes, why?
Thank you!

The students use bsd220.org, I will go ahead and remove this.

@saakethk02 I am closing this request without merging as per your request. The request for bsd220.org domain has been merged. Thank you!


Metric Tag Count for Points/Histograms/Spans

New metric in the proxy to find out how many point/histogram/span tags user is emitting

@basilisk487 , please merge if you approve :) Thanks!


New Tracing System

What?
The new tracing system, based on the trace-dispatcher and cardano-tracer components is a complete rewrite of the node tracing system.
Why?
The new tracing system provides the following benefits:

Maintainability: the legacy code is very difficult to reason about -- so features are harder to implement and bugs are harder to find.
Performance: the performance characterstics of the legacy tracing are hard to reason about.  The new system differs in following ways:

based on an arrow-based contravariant abstraction, which allows for explicit separation and sharing of stateful computations -- enhancing our performance reasoning abilities.
generally, has a much cleaner implementation that is easier to reason about.
performs better (uses less memory and spends less CPU), and without performance hacks like TraceXXX: false cut-off flags.

Configuration simplicity:

there is just a single, namespace-based layer of configuration -- instead of two conflicting layers in the legacy system (all the TraceXXX options that control the performance-cutoff hack).
the RTView component is now part of the cardano-tracer component, so is very easy to enable.

Flexibility:

the configuration is dynamic, because there are no static performance cut-off flags.

Documentation:

the system is self-documenting -- therefore enabling automated generation of a central index of all tracers.  This is very important because there are hundreds of different traces emitted by the node.

Reduces the intra-node code complexity and overheads:

we move out all unnecessary trace handling complexity out of the node into a separate cardano-tracer component.

Enables the next-generation of RTView.
Provides a rich state API -- DataPoints now allow querying the node for structured state, in a metrics-like fashion.  This is an enabler for richer and simpler node integrations -- for components such as Daedalus and DB Sync.

Definition of done

The new tracing system is the default, and is released to the SPO community.
The distributed testnet benchmarks use the new tracing system.

Closing this. Please reopen if this is still relevant.


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-source-version-target/pull/213 is merged into live branch.
Auto archive content to https://github.com/opstest2/docs-archive-test-source-version-target.git

Docs Build status updates of commit 69d13d1:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

.openpublishing.redirection.json
:white_check_mark:Succeeded

versioningArchiveSourceFolder/TOC.yml
:white_check_mark:Succeeded
View (sql-server-2014 || azuresqldb-current || azuresqldb-mi-current || azure-sqldw-latest || >= aps-pdw-2016 || >= sql-server-2016 || >= sql-server-linux-2017 || sql-server-previousversions || sqlallproducts-allversions)

versioningArchiveSourceFolder/docfx.json
:white_check_mark:Succeeded

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


Add Google Cloud Platform WAF bypass with 8KB packet

https://kloudle.com/blog/piercing-the-cloud-armor-the-8kb-bypass-in-google-cloud-platform-waf

Hi, thanks for linking this. I added a new section here: https://github.com/0xInfection/Awesome-WAF/blob/master/README.md#abuse-waf-limit-on-http-responses


Need direct access to Elastic Search

Using ElasticSearch for document-level summarization/extraction is more efficient than compiling paragraphs from Weaviate.

#111


"Packages installation failed" Error on Windows Server 2019 and Windows 11

Package version
4.0.0
Describe the bug
Unable to install japa on Windows Server 2019 and Windows 11 following the Installation and Usage guide in the documentation.
Executing npm init japa@latest in the project root and following the prompts results in the following error after the bin folder is created.
> Select the tests reporter · @japa/spec-reporter
> Select the assertion library · @japa/expect
> Select additional plugins · No items were selected
> Select the project type · TypeScript ESM
> Want us to create a sample test? (y/N) · false

CREATE: bin/test.ts
CREATE: bin/japa_types.ts

× Packages installation failed

The above error happens regardless of what options are chosen during the installation prompt.
Reproduction repo
No response

No idea for Windows Server. However I am on Windows 11 and it works perfectly
The problem may comes from here: https://github.com/japa/create-japa/blob/develop/index.ts#L352
The install function comes from the mrm-core library and its definitions is here : https://github.com/sapegin/mrm/blob/master/packages/mrm-core/src/npm.js#L36
If you feel it, you can try to debug it and see where it crashes.
For us I think it's almost impossible to debug without being able to get a hand on your environment and thus be able to reproduce the error

Thank you!
I tried to do some debugging by cloning the create-japa repository and running through the installer, but it executed without a hitch.
npm init japa@latest executes as expected on my Mac and Linux machines as well.
Strangely enough, I was able to successfully run the installer on Windows Server 2019 on an existing project. It still fails under an NX monorepo, however. No success on Windows 11 on my end either.
Seems the issue is solely with my Windows environments. Since it runs as exected through the cloned repo I haven't been able to debug it. I have tried searching but I haven't been able to find a way to debug an npm init command. I'm not sure what steps I should take to continue debugging on my end. Do you have any suggestions on how to proceed?
Thanks again for the assistance.

Closing, since not actionable


sections support

Following along with several issues elsewhere on github, and would like to use trace-mapping as a (transitive) dependency in ts-node.
https://github.com/terser/terser/pull/1164#issuecomment-1068354694
Am I understanding correctly, that this library is missing support for sections?  Can this issue be used for tracking progress on that?

It's implemented via AnyMap. I hadn't documented it because I considered it to be a niche API, but I'd be happy to accept a doc PR if you want to contribute.

I see, does this mean consumers of the API must take special care to leverage it?  Or can a consumer use the API as they otherwise would?  I'm admittedly unfamiliar with sections and what it means.  I naively assume that consumers of this API don't necessarily need to care whether or not a sourcemap uses sections.

I naively assume that consumers of this API don't necessarily need to care whether or not a sourcemap uses sections.

Correct. AnyMap can take any form of sourcemap, and will return a regular TraceMap instance. Eg, if a regular sourcemap is passed to AnyMap, it immediately returns with a new TraceMap, and if a sectioned sourcemap is passed, it converts from the section offsets into a regular sourcemap and returns that as a new TraceMap.

Nice, thanks for the explanation.  I hope to have time to try it soon.


Add CodeQL security scans

Motivation
Follow up to issue https://github.com/open-telemetry/oteps/issues/144
CodeQL is GitHub's static analysis engine which scans repos for security vulnerabilities. As the project grows and we near GA it might be useful to have a workflow which checks for security vulnerabilities with every PR so we can ensure every incremental change is following best development practices. Also passing basic security checks will also make sure that there aren't any glaring issues for our users.
Changes

This PR adds CodeQL security checks to the repo
After every run the workflow uploads the results to GitHub. Details on the run and security alerts will show up in the security tab of this repo.

Workflow Triggers

daily cron job at 1:30am
workflow_dispatch (in case maintainers want to trigger a security check manually)

cc- @alolita

Please update the branch or make the PR editable by maintainers.

Please update the branch or make the PR editable by maintainers.


operator: add replaces specification

When preparing https://github.com/operator-framework/community-operators/pull/2450, 'replaces: pmem-csi-operator.v0.7.0' had to be added manually.
We should automate that or at least document it.

Fixed by https://github.com/intel/pmem-csi/pull/792


Please publish this extension on open-vsx.org

Re-open of #75
Publish
guide

Sorry but I am not a big fan of these unofficial third party sites


Javascript, Android, IPhone, OSX, etc... not present in platform

I don't know is this is a regression or a temporary bug but when launching scons on the latest fork of godot nor of the Javascript, Android, IPhone or OSX platforms are present in the platform option(only windows, server, x11 and haiku).
I don't know if I misreaded something on the doc or anything else but it kinda seems weird that those doesn't show up b\c imo OSX and Android are kind of a big deal, so I just opened up the issue.

What platform are you on?  Did they show up before, and a recent commit broke things?

I'm on x11, Ubuntu-based distro. I actually just forked up and it didn't showed up, didn't checked older commits.

It's normal that they don't show up, they only do if you can actually build them. But you likely have no build environment setup for javascript, android, iphone or OSX, so they're not listed. It's not a regression, it has always been like that :)

Well the environnement for osx was setted up(using osx cross)before relaunching scons when I actually made this issue so...

Then probably not setup properly for the buildsystem. Check: https://github.com/godotengine/godot/blob/master/platform/osx/detect.py#L12
For osxcross you need to define a specific environment variable it seems.

Forget that, it was setted up correctly, just had to relaunch the terminal ^^"


Feature/validators external endpoint

Fixes | Closes | Resolves #

Anything marked as optional that you didn't need to fill in, please remove it from the PR description. Choose one of the keywords above to refer to the issue this PR solves, followed by the issue number (e.g Fixes # 666). If there is no issue, remove the line. Remove this note after reading.

Changes:

Types of changes
What types of changes does your code introduce?
Put an x in the boxes that apply

[ ] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation Update
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] Other (please describe):

Testing
Requires testing

[ ] Yes
[ ] No

In case you checked yes, did you write tests?

[ ] Yes
[ ] No

Comments about testing , should you have some (optional)
Further comments (optional)
If this is a relatively large or complex change, kick off the discussion by explaining why you chose the solution you did and what alternatives you considered, etc...

Codecov Report
Base: 96.68% // Head: 89.39% // Decreases project coverage by -7.28% :warning:

Coverage data is based on head (3398ebe) compared to base (6721d7d).
Patch coverage: 5.40% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #13      +/-   ##
==========================================
- Coverage   96.68%   89.39%   -7.29%
==========================================
  Files          11       11
  Lines         482      481       -1
==========================================
- Hits          466      430      -36
- Misses         13       45      +32
- Partials        3        6       +3

Flag
Coverage Δ

integrationtests
?

unittests
89.39% <5.40%> (-7.00%)
:arrow_down:

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

Impacted Files
Coverage Δ

pkg/eth2/init.go
49.33% <0.00%> (-43.35%)
:arrow_down:

pkg/eth2/networking/subscribe.go
100.00% <ø> (ø)

pkg/eth2/eth2.go
94.85% <100.00%> (+0.11%)
:arrow_up:

pkg/eth2/networking/beacon_api.go
94.44% <0.00%> (-1.21%)
:arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


links Get started and Take the Tutorial have been relocated in mobile…

According to the issue (RUS), the design has been changed  on mobile version as on this version long text (for example in oher languages) is not fully depicted. If there is a need to change styles I'll correct it.
AFTER

BEFORE

@alexkrolick please review this PR.

@StepanNaryshkov take control and checked this PR, it's okay. Sorry for delay!

no problem, if you need help in ui, please ping me


:gift: Deterministic exit code for zap's Fatal for sharedmain

Changes

:gift: Deterministic exit code for zap's Fatal for sharedmain

/kind enhancement
Fixes #2495

/assign @knative/technical-oversight-committee

/assign @dprotaso
/assign @rhuss
/assign @julz
/assign @vagababov
/assign @vaikas
/assign @knative/knative-release-leads
/assign @nader-ziada
/assign @ikvmw

zap's godoc says the Fatal exit code is 1
See: https://pkg.go.dev/go.uber.org/zap#Logger.Fatal
Are you seeing something different?

I'm not particularly sold on this change - as the exit code is sorta random depending on the log message.
Plus a range of exit codes as meant to reserved:
https://man7.org/linux/man-pages/man1/exit.1p.html#RATIONALE

Where do we stand with this PR ? Seems to be dormant since quite some time.

Going to close this out


exporting public API

Hello!
I noticed that there are a couple of functions that are listed in the Public API documentation but that are not exported by StochasticAD.jl. Moreover, I think it would be useful if they were exported. These functions are

value;
delta.

Perhaps there is a good reason for not exporting them, but in case this is an oversight perhaps they can be added to the exports. Thanks!

Hi! Thanks for making the issue. In general, I think there's a practice in Julia to only export symbols that are unlikely to clash with other packages. I felt like value and delta were too generic to export, but they are indeed still public API. (On the other hand, there may be more specific identifiers that are not exported due to my oversight; feel free to open a new issue if you spot any of these.)
Julia 1.11 is going to get a public keyword to use in exactly this situation: once that's available, I'll definitely mark value and delta as public!

That makes sense, thanks!


Spp 7308 aboutthislibrary

Should really be page_body, not page_boby
It may also be worth looking at making the template generic rather than specific for 'About' - if you pass through which aside nav to call, and 'help_centre' for the current_path value you could make the template generic for any page with content and a right hand column
there are 2 major difference between 'about' & 'help category' which blocks the same template will have been used in. for example- 'help category' template uses the component 'onsSectionNavigation' but 'about' template doesn't need that, title used for 'relatedcontent' varies for both templates. So we need to pass large number of parameters to render_template to reuse the same template. I think, it is not nice in terms of readability, maintainability. so leave as it- 'about' & 'help category' use separate template.

created a new branch to cover this change.

created new branch to this change.


[WIP] Returns the newly created object

Discussion in #21

I may have solved this using a similar but simpler take on it... I'll push up a separate PR tonight so we can both take a look.

@jonataswalker check out #24 and let me know any thoughts.

Closing in favor of #24 which is merged and pending release. Thanks again for your efforts here!


Question: localstorage in a Angular2 Service

I am trying to get the local storage to persist some auth token info, and it is occurring in a service.
Strangly enough it does not seem to want to persist.  If I redirect to the auth middleware (IdentityServer3) and then redirect back to the original site, the data I persisted is not there anymore.  Is there something special I need to do to get localstorage to work for a service?

Did you add the LocalStorageModule at the root of your application?
And did you set the storageType config option?
If you put a breakpoint in your code after it stores the auth token, before it redirects off somewhere else, can you see your value stored in localStorage correctly?


Replace validate-commit-msg with commitlint

Only breaking change I'm aware of is that commit lint doesn't support the helpMessage configuration to tell the user to run npm run commit when it fails.
Fixes #188

Coverage remained the same at 100.0% when pulling 50277e5a45abd5f0b07a518068a618575591f15d on mattmazzola:commitlint into cd2ce6c05084caf75c63d912e4f04e1dbce5e6e6 on alexjoverm:master.

Oops, forgot about this one. Finally updated.


Use internal ROM interface for inflector

With changes merged from rom-rb/rom#158, rom now provides an internal API to access an inflector implementation.  This change updates rom-sql to use that API and removes any dependency on the inflecto gem.

thanks!


Add a few fields to MRI class

in MRIScan add:

[ ] rare_factor (title= "RARE factor). int
[ ] vc_orientation can use coordinates.Rotation3DTransform
[ ] vc_position can use coordinates.Translation3DTransform
[ ] vc_subject_position (prone/supine ... need to investigate)
[ ] resolution

I will update as some of these will use classes in the Tile (or coordinates) file. I will detail asap.

##$VisuSubjectPosition=Head_Supine

@saskiad combine position/orientation into List[CoordinateTransformations] a la the other models?


Fix intermittently failing run_playbook test

Depends on #358.
With the introduction of ordered shutdown in PR 358, an intermittent race condition arose with the run_playbook test due to Ruleset 2 bailing before it received the set_fact from ruleset 1. PR 358 adds ordered shutdown, but if there are no pending items in the action queue then the ruleset will end immediately. In this test, once the time specified under shutdown_after in ruleset 1 has expired then a shutdown command will be issued to all rulesets, and since ruleset 2 does not have any pending items it will end immediately, sometimes before the set_fact from ruleset 1 has had a chance to run. Increasing shutdown_after is an option, but this increases the test time unnecessarily.
This change moves the condition that performs the set_fact so that it occurs at the beginning of the run, so we can be sure that ruleset 2 will not end before the set_fact and therefore avoid the race condition. We may run into a race condition with other arch's, such as ppc, in which case we would need to increase SOURCE_SHUTDOWN_AFTER for that particular arch.

Tests fail intermittently due to the post_events and set_facts not always finishing before the source plugin ends. This will be resolved once 358 is merged.

@mkanoor @Alex-Izquierdo pls review now that 358 has been merged.

Failing due to missing dynaconf package from test requirements file. Will be fixed once #381 is merged.


[Bug] Using SparkSQL insert Paimon, the fields do not match

Search before asking

[X] I searched in the issues and found nothing similar.

Paimon version
0.9
Compute Engine
Spark 3.5
Minimal reproduce step
We have some problems when upgrading the paimon version from 0.8 to 0.9.
for example:
CREATE TABLE T (
  id BIGINT,
  a BIGINT,
  b BIGINT,
  c STRING,
  d STRING)
 TBLPROPERTIES (
  'bucket' = '1',
  'primary-key' = 'id'
)

What doesn't meet your expectations?
Problem 1
If I submit the following sql：
INSERT INTO T(id, d, c, b, a)
SELECT CAST(1 AS BIGINT) AS id, 'd' AS d, 'c' AS c, CAST(3 AS BIGINT) AS b, CAST(2 AS BIGINT) AS a

with Paimon-0.9: throw java.lang.ClassCastException: java.lang.String cannot be cast to java.lang.Long
with Paimon-0.8: sql executed successfully, the data in table is (1, 2, 3, c, d)

I guess the fields do not match.
I changed the sql to the following, select item is the same as table schema：
INSERT INTO T
SELECT CAST(2 AS BIGINT) AS id, CAST(2 AS BIGINT) AS a, CAST(3 AS BIGINT) AS b, 'c' AS c, 'd' AS d

with Paimon-0.9: sql executed successfully, the data in table is (1, 2, 3, c, d), (2, 2, 3, c, d)
Problem 2
In addition, we also found that implicit conversion is also invalid:
SET spark.sql.storeAssignmentPolicy=LEGACY;
INSERT INTO T
SELECT 3 AS id, 2 AS a, 3 AS b, 'c' AS c, 'd' AS d

with Paimon-0.9: throw java.lang.ClassCastException: java.lang.Int cannot be cast to java.lang.Long
with Paimon-0.8: sql executed successfully

Anything else?
No response
Are you willing to submit a PR?

[ ] I'm willing to submit a PR!

@luowanghaoyun Can you try the latest 1.0-snaoshot, these problems should have been fixed

@Zouxxyy I have tried the latest version(paimon-spark-3.5-1.0-20241218.002704-69.jar). The problems still exist. Can you check again?

Here my test on the master code

Here my test on the master code

+1 in my test，master is OK

@Zouxxyy @xuzifu666 Thank you for your response.
This is a misunderstanding. The default Spark configuration we use internally is with Iceberg Spark Extensions (which seems to have a conflict with Paimon), leading to the aforementioned erroneous behavior. I have switched to Paimon Spark Extensions, and the issue no longer exists in either version 0.9 or master.


issue #779 #1150 partitionize bulkexport JavaBatch Job & fix CWWKY0041W warnings

Summary of the changes:
(1) Partitionized bulkdata export jobs - system, patient and group exports.
(2) Dropped single COS file multi-parts uploading for the whole export job.
(3) Added multi-parts uploading for each partition.
(4) Added server configurations to control the file size or resource number of each COS file.
- fhirServer/bulkdata/cosFileMaxResources
- fhirServer/bulkdata/cosFileMaxSize
(5) Added simple metrics for the exporting jobs.
(6) Fixed CWWKY0041W warnings of bulkdata both export and import jobs.
(7) Refactoring of the bulkdata export codes.

   symbol: class Dependent
[ERROR] /home/runner/work/FHIR/FHIR/fhir-bulkimportexport-webapp/src/main/java/com/ibm/fhir/bulkexport/system/ExportPartitionCollector.java:[14,32] package javax.enterprise.context does not exist
[ERROR] /home/runner/work/FHIR/FHIR/fhir-bulkimportexport-webapp/src/main/java/com/ibm/fhir/bulkexport/system/ExportPartitionCollector.java:[20,2] cannot find symbol

Needs a dependency like
 <dependency>
            <groupId>javax</groupId>
            <artifactId>javaee-api</artifactId>
            <scope>provided</scope>
        </dependency>

note there is no version as it should come from the parent.


🐛 FIX: Pad with Unicode Display Width

Fixes: #16

This change is fairly small and a long-standing issue, so I'll go ahead and squash and if there are any concerns I would be happy to fix-forward


tests: tmt runs all tests in one vm

We are going to need to split up the tests. And to do that we are going to need to figure out fmf/tmt more.
Just to track things, I am testing out this:
From 891b23bcb44fa7e003588553b7fb243a33d2e0b4 Mon Sep 17 00:00:00 2001
From: Colin Walters<EMAIL_ADDRESS>Date: Fri, 26 Jul 2024 08:48:39 -0400
Subject: [PATCH] wip

Signed-off-by: Colin Walters<EMAIL_ADDRESS>---
 plans/integration-run.fmf                      | 11 +++--------
 tests/basic-readonly/main.fmf                  |  2 ++
 tests/basic-readonly/test.sh                   |  4 ++++
 tests/booted/{ => readonly}/001-test-status.nu |  0
 tests/booted/{ => readonly}/basic.py           |  0
 tests/image-pushpull-upgrade/main.fmf          |  2 ++
 tests/image-pushpull-upgrade/test.sh           |  2 ++
 7 files changed, 13 insertions(+), 8 deletions(-)
 create mode 100644 tests/basic-readonly/main.fmf
 create mode 100755 tests/basic-readonly/test.sh
 rename tests/booted/{ => readonly}/001-test-status.nu (100%)
 rename tests/booted/{ => readonly}/basic.py (100%)
 create mode 100644 tests/image-pushpull-upgrade/main.fmf
 create mode 100755 tests/image-pushpull-upgrade/test.sh

diff --git a/plans/integration-run.fmf b/plans/integration-run.fmf
index 1190fcd..132babe 100644
--- a/plans/integration-run.fmf
+++ b/plans/integration-run.fmf
@@ -6,13 +6,8 @@ provision:
   image: file://./target/testvm/disk.qcow2
   disk: 20
 summary: Execute booted tests
+discover:
+  how: fmf
 execute:
   how: tmt
-  # There's currently two dynamic test frameworks; python and nushell.
-  # python is well known and understood. nushell is less well known, but
-  # is quite nice for running subprocesses and the like while making
-  # it easy to parse JSON etc.
-  script: |
-    set -xeu
-    pytest tests/booted/*.py
-    ls tests/booted/*-test-*.nu |sort -n | while read t; do nu $t; done
+
diff --git a/tests/basic-readonly/main.fmf b/tests/basic-readonly/main.fmf
new file mode 100644
index 0000000..aa0b710
--- /dev/null
+++ b/tests/basic-readonly/main.fmf
@@ -0,0 +1,2 @@
+summary: Concise summary describing what the test does
+test: ./test.sh
diff --git a/tests/basic-readonly/test.sh b/tests/basic-readonly/test.sh
new file mode 100755
index 0000000..585e584
--- /dev/null
+++ b/tests/basic-readonly/test.sh
@@ -0,0 +1,4 @@
+#!/bin/sh -eux
+
+pytest tests/booted/readonly/*.py
+ls tests/booted/readonly/*-test-*.nu |sort -n | while read t; do nu $t; done
diff --git a/tests/booted/001-test-status.nu b/tests/booted/readonly/001-test-status.nu
similarity index 100%
rename from tests/booted/001-test-status.nu
rename to tests/booted/readonly/001-test-status.nu
diff --git a/tests/booted/basic.py b/tests/booted/readonly/basic.py
similarity index 100%
rename from tests/booted/basic.py
rename to tests/booted/readonly/basic.py
diff --git a/tests/image-pushpull-upgrade/main.fmf b/tests/image-pushpull-upgrade/main.fmf
new file mode 100644
index 0000000..aa0b710
--- /dev/null
+++ b/tests/image-pushpull-upgrade/main.fmf
@@ -0,0 +1,2 @@
+summary: Concise summary describing what the test does
+test: ./test.sh
diff --git a/tests/image-pushpull-upgrade/test.sh b/tests/image-pushpull-upgrade/test.sh
new file mode 100755
index 0000000..c8bfe44
--- /dev/null
+++ b/tests/image-pushpull-upgrade/test.sh
@@ -0,0 +1,2 @@
+#!/bin/sh -eux
+exec tests/booted/002-test-image-pushpull-upgrade.nu
--
2.44.0

OK https://github.com/teemtee/tmt/pull/3037 is hard blocking my local testing...

I am so lost in tmt...trying to figure out: if I want to run each test in a distinct VM, does that require a distinct plan for each one? Is that what https://tmt.readthedocs.io/en/stable/spec/plans.html#isolate is trying to help?

I asked on a chat channel, the answer is:

for now, the answer is indeed yes, you need to spread tests across multiple plans, or have multiple guests in one plan, one for each discover phase with corresponding where
tmt does not know yet how to isolate tests or how to automatically turn a set of tests into multiple plans on its own


all routes defined in a single file similar to the Slim Framework

is there any way to make all routes defined in a single file similar to the Slim Framework and the Laravel Framework from PHP
file= "Routes.php" or "index.php"

 require 'vendor/autoload.php';
    use \Slim\App;
    $app = new App();
    $app->get('/books', function ($request, $response, $args) {
        BookController::GetAll($request, $response, $args)
    });
    $app->get('/books/{id}', function($request, $response, $args){
         BookController::GetItem($request, $response, $args)
    });
    $app->post('/books/{id}', function($request, $response, $args){
         BookController::SaveItem($request, $response, $args)
    });
    $app->run();

file=BookController.php

class BookController
{
    public static function getAll(Request $request, Response $response)
    {
        try {

            $parametros = $request->getParsedBody();
            $draw = $parametros['draw'];
            $limit = $parametros['length'];
            $offset = $parametros['start'];
            $search = '%' . $parametros['search'] . '%';

            $query = DBLayer::Connect()->table(Book::TABLE_NAME)
                ->where(function ($query) use ($request, $search) {
                    $query->orWhere(Book::KEY_ID, DBLayer::OPERATOR_ILIKE, $search);
                    $query->orWhere(Book::SISTEMA, DBLayer::OPERATOR_ILIKE, $search);
                });

            $totalRecords = $query->count();
            if ($parametros) {
                $data = $query->orderBy(Book::KEY_ID, DBLayer::ORDER_DIRE_ASC)
                    ->take($limit)->offset($offset)->get();
            } else {
                $data = $query->orderBy(Book::KEY_ID, DBLayer::ORDER_DIRE_ASC)
                    ->get();
            }

            $result['draw'] = $draw;
            $result['recordsFiltered'] = $totalRecords;
            $result['recordsTotal'] = $totalRecords;
            $result['data'] = $data;
        } catch (Exception $e) {
            return $response->withStatus(StatusCode::BAD_REQUEST)->withJson((['message' => StatusMessage::MENSAGEM_ERRO_PADRAO, 'exception' => $e->getMessage(), 'line' => $e->getLine(), 'file' => $e->getFile()]));
        }
        return $response->withStatus(StatusCode::SUCCESS)->withJson($result);
    }
}

something similar to this example
file= "Routes.cs" or "Start.cs"

               var routes = new Routes();
                routes.get("/books", async (request, response, args) => {
                      BookController.GetAll(equest, response, args)
                });
                routes.get("/books/{id}", async (request, response, args) => {
                    BookController.GetItem(equest, response, arg)
                });
                routes.post("/books/{id}", async (request, response, args) => {
                    BookController.SaveItem(request, response, args)
                });
                routes.run();

file=BookController.cs

class BookController
{
 public static getAll(Request request, Response response)
    {
        try {

            var parametros = request.getParsedBody();
            var draw = parametros['draw'];
            var limit = parametros['length'];
            var offset = parametros['start'];
            var search = '%' . parametros['search'] . '%';

            query = DBLayer.Connect().Table(Book::TABLE_NAME)
                .Where(function (query) use (request, string search) {
                    query.orWhere(Book::KEY_ID, DBLayer::OPERATOR_ILIKE, search);
                    query.orWhere(Book::NAME, DBLayer::OPERATOR_ILIKE, search);
                });

            var totalRecords = query.count();
            if (parametros) {
                data = $query->orderBy(BugReport::KEY_ID, DBLayer::ORDER_DIRE_ASC)
                    .take($limit).offset(offset).get();
            } else {
                data = query.orderBy(BugReport::KEY_ID, DBLayer::ORDER_DIRE_ASC)
                    .get();
            }

            var result['draw'] = draw;
            result['recordsFiltered'] = totalRecords;
            result['recordsTotal'] = totalRecords;
            result['data'] = data;
        } catch (Exception e) {
            return response.withStatus(StatusCode::BAD_REQUEST)
                .withJson((['message' => StatusMessage::MENSAGEM_ERRO_PADRAO, 'exception' => e.getMessage(), 'line' => e.getLine(), 'file' => e.getFile()]));
        }
        return response
            .withStatus(StatusCode::SUCCESS)
            .withJson($result);
    }
}

You sure can. Depending on exactly what you want, you can do what's in the docs here:
https://docs.microsoft.com/en-us/aspnet/core/fundamentals/routing?view=aspnetcore-2.2#use-routing-middleware
And in particular, using the extension methods defined here:
https://docs.microsoft.com/en-us/dotnet/api/microsoft.aspnetcore.routing.requestdelegateroutebuilderextensions?view=aspnetcore-2.2

I am a full time PHP programmer in my work, and I thought I could migrate my projects and suggest ASP .NET Core as a viable solution in my company, but for this I have to write a translator so that I can automate the process I need a very similar way in C # to define routes and make access to database with Query Builder similar to the laravel Eloquent, I usually only do Desktop applications in C # so I do not have much knowledge on WEB applications in C #, could you give me an example code that does what I need.

Hi @insinfo , for general questions about how to use ASP.NET Core, to get a faster response we suggest posting your questions to StackOverflow using the asp.net-core tag and possibly also applicable PHP tags (such as something about Laravel or Eloquent)).


Piece wise beadlist

Updated include guard
Removed unused include
Made includes relative where appropriate
Added const to method

@junghans @JensWehner can one of you approve?


Trying to exclude a specific user agent (Web Site Monitor)

I was reading how you had added the ability to add a list of browsers in a text file, and was trying to use this as a way to omit our uptime requests from the log parsing.
We use NodePing, so i created a file called browsers.txt, using your demo file and added a line to the top:
NodePing                                          Crawler
I was using the following command in bash:
bash-4.4$ goaccess /cygdrive/e/temp/goaxess/*.log --log-format='%d %t.%^ ~h{," } %e %v %^ %m %U %q %s %^ %^ %L "%u" %R %^' --date-format=%Y-%m-%d --time-format=%T --http-protocol=no -o /cygdrive/e/temp/ws_all_report_20190919.html --real-time-html --browsers-file=/cygdrive/e/temp/browsers.txt --ignore-crawlers

It still has the same mount of requests over all with or without the last two switches in the command line (I think this is intended to show the total number of requests without filtering the browsers file)
I am just looking for confirmation that this is the way to execute this type of result
Let me know what you think

Please try setting it as Crawlers instead of Crawler. Also make sure it is tab delimited e.g.,
NodePing\t\tCrawlers

that is,
NodePing		Crawlers

I added the tab delimiter in between the name and the type instead of spaces.
I also see that it removes the top requested file based on the browsers.txt file.
This is what i was looking for
Thanks!


ci: bump juju 3.1 -> 3.5 in track/2.1

Part of canonical/bundle-kubeflow#859

The problem with the CI is that mysql-k8s 8.0/edge requires juju <3.5.0 or ==3.5.2, which hasn't been released. This CI will be blocked until we do either of:
a) Use mysql-k8s 8.0/stable - this is problematic because they will soon promote from 8.0/edge, which will leave us with the same issue
b) Pin juju to a different version, it can be 3.4.3

I 'd go with a) since I 'd expect taking this requirement into account before promoting edge -> stable, since it's a breaking change in a way.

https://github.com/canonical/mlflow-operator/pull/246 should help fix the CI. Lets merge that before this one.

Closing this PR as it will be replaced by #241


Excluding ood.flag in all folders for Umbraco gitignore

Reasons for making this change:
The ood.flag file is generated by Umbraco to indicate it's code-first models need regenerating.
This file is automatically created in the root of the folder where it has been designated Umbraco should generate code-first models. It should never be committed to source control.
By default Umbraco stores these files in /App_Data/Models/*, however this is configurable via the Umbraco.ModelsBuilder.ModelsDirectory app setting. In most cases developers will modify this path as files in App_Data are not compiled by default by Visual Studio.
Additionally, the upcoming Umbraco V9 release moves the default location of these files to /umbraco/models/*.
This change adjusts the ignore rule to apply broadly regardless of where the odd.flag file lives.

Looks good to me - thanks!


Label alignment not centred when setting a requisition on the label

When setting a fixed width for a label using SetRequisition, the label will not be centered, but will be left justified.
Brew code snippet from Label.cpp:
22 	if( !label->GetLineWrap() ) {
23 		// Calculate alignment when word wrap is disabled.
24 		sf::Vector2f avail_space( label->GetAllocation().width - label->GetRequisition().x, label->GetAllocation().height - label->GetRequisition().y );
25 		sf::Vector2f position( avail_space.x * label->GetAlignment().x, avail_space.y * label->GetAlignment().y );
26
27 		vis_label.setPosition( position.x, position.y );
28 	}

This issue is I believe is line 24 where the GetRequisition.x is the same size as GetAllocation.width, as your available space will be 0.  It doesn't matter what Alignment you assign, it will always be 0 if this is the case (Left Justified).
Sample code snippet, I have modified the  alignment  of 'File' to be the far right of the label for demonstration.
		m_boSingleFileHeader = sfg::Box::Create(sfg::Box::Orientation::HORIZONTAL, 0.0f);
		m_lSingleName = sfg::Label::Create("File");
		m_lSingleName->SetRequisition(sf::Vector2f(633.0f, 0.0f));
		m_lSingleName->SetAlignment(sf::Vector2f(1.0f, 0.5f));
		m_lSingleDate = sfg::Label::Create("Date");
		m_lSingleDate->SetRequisition(sf::Vector2f(180.0f, 0.0f));
		m_lSingleDate->SetAlignment(sf::Vector2f(0.1f, 0.5f));
		m_boSingleFileHeader->Pack(m_lSingleName, false, false);
		m_boSingleFileHeader->Pack(m_lSingleDate, false, false);

Please see the position of 'File' in the image below, it should be at the far right hand side of the 633.0f label, but instead it is at the far left.

I'll do some playing around to see if I can fix it myself.

I have a fix for the alignment, though I've bundled it in with the ability to set text justification in the Misc class.  I have also added the Misc class via inheritance to the button and togglebutton classes.  This provides left, centre and right justification.  If you are interested in this at all, let me know and I'll send a pull request so you can review the code.

Send a pull request. :wink:
I'll review it and provide feedback. We tend to like individual pull requests for each feature, but if you don't feel like splitting them up, I can edit your commits myself.


Ability to work without companion

The extension should be able to function without a companion in some way. Possibly resizing the parent window?

Companion now works reliably - no need for this.


🎮 Game Request
This game will be a 2 player tic-tac-toe game using Javascript and designed on bootstrap.
Point down the features

Multi player access
Low memory usage
Simple and Fun

Select program in which you are contributing
GSSoC23
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINE of this project.

Hey @trishanu-init !
Thank you for raising an issue 💗
You can self assign the issue by commenting /assign in comment 😀
Make sure you follow CODE OF CONDUCT and CONTRIBUTING GUIDELINES 🚀
Don’t Forget to ⭐ our GameZone🎮
Make sure you join our Discord🕹

Hey @trishanu-init!
We are already having a similar game request in #289 👀
Make sure you come up with a cool unique idea 😀
Waiting for your new game idea 💗.

Hey @trishanu-init ! Thank you so much for your raising the issue💗
It’s all yours, you can come anytime again and make some contributions! 🚀
Alone, we can do little, but together we can do so much! 😇


Insufficient warnings when non-numeric float_value entered for data

When the data field is set to a string, e.g., "asdf", and numeric Data is selected using the radio buttons, the message will not be loaded properly and the error is that data requires one of {value, bytes_value, url}. It would be better to describe that the input was non-numeric but a numeric field was entered, however, this must occur on the JS side not in Python validations

Hi @connorcoley , could you describe how you replicated this issue in the interface? Was this encountered while searching, or elsewhere in the application?

This was in the graphical editor when contributing data. It's a rather weird edge case.

Starting from a completely empty new reaction, add a component to input 1 and add a feature of type text with data/value "text_data". Saving this reaction and reloading the page shows that the message was saved properly.

Starting from a completely empty new reaction, add a component to input 1 and add a feature of type number with data/value "5". Saving this reaction and reloading the page shows that the message was saved properly.

Starting from a completely empty new reaction, add a component to input 1 and add a feature of type number with data/value "text_data". Saving this reaction and reloading the page shows that the message was not saved properly, and it will revert to a completely empty message (because no other fields are defined; if other fields were defined, only this 'feature' would be removed)

It's been a while, but IIRC the issue for this silent failure is that the attempted parsing of the data field to a number happens in JS and isn't passed through to Python during validations


add scale14 to global params

Description
To resolve #36
Status

[ ] Ready to go

Codecov Report

Merging #37 (02447df) into main (b7d8856) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main      #37   +/-   ##
=======================================
  Coverage   78.62%   78.62%
=======================================
  Files           5        5
  Lines         276      276
=======================================
  Hits          217      217
  Misses         59       59

Flag
Coverage Δ

unittests
78.62% <100.00%> (ø)

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

Impacted Files
Coverage Δ

smirnoff_plugins/handlers/nonbonded.py
94.26% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@pavankum thanks for looking into this, I was a little lost as to why this fixed the issue as the scale14 parameter is already added to the global parameter list explicitly in the creation of the custom bond force which holds the 1-4 interactions here. It seems like the root cause of this is in forcebalance when it creates a copy of the system to evaluate the perturbed parameters it only looks for global values in the CustomNonbondedForce but if it also checked the CustomBondForce it would copy both.

I think it's probably easier to fix this here but I have suggested the fix upstream to forcebalance here, feel free to do the fits using this branch @pavankum  as it will take a while to get a new forcebalance version.

Ahh okay, thank you @jthorton for fixing it in FB. Thank you for the review @mattwthompson, since this is fixed in FB I think we need not merge this in master.

Do these patches conflict with each other? All things being equal I'd rather have it here since we have more control over the release cycle and it's easier to pin (well, add a lower constraint) this package than ForceBalance.

Yeah, I think we would be double counting the contributions if both patches are applied. I can still work from this branch and it's not a blocker for me. I would leave it to you and @jthorton to decide what's the best way forward.

I agree it would be easier to fix here and this will fix our pathway of using forcebalance with systems made via the toolkit but will leave it broken for anyone using custom forces via OpenMM xmls, although I don't think that is very common at all! So feel free to merge and Ill leave my FB PR for now and see what happens.

Sounds good to me. Let me know if you want this in a release (0.0.3), otherwise I assume you and @pavankum can keep running experiments from the main branch as useful


Add tap to dismiss keyboard in Notification ViewController

I have added tap to dismiss keyboard in the keyboard. Right now you have to drag to dismiss it. 😄

What does this do when tapping on a notification to view its content?
I haven’t run into this being an issue yet. UX bug or preference?
Sent with GitHawk

I don't think I understand the

What does this do when tapping on a notification to view its content?

I will not call it a bug. It's more a UX enhancement I think. Because it is possible to dismiss the keyboard by dragging the "No new notification view". Not it is possible to tap anywhere to dismiss the keyboard 😄

@kimdv I mean, what happens when there is content in the collection view? I'm assuming that the collection view absorbs the tap and this event doesn't fire? Just making sure this doesn't interfere w/ any other interactions.

@rnystrom I pushed a fix for that. So now it should not cancel other touches 😄
Sent with GitHawk

I can make a video with the use case ?
Sent with GitHawk

Probably ok w/out the video. I def see what you mean. Let's hold for a minute tho b/c of all the refactoring that just happened w/ NotificationsViewController. Even more incoming.


ci-automation: Generate digests for artifacts

This PR adds generation of DIGESTS files and their armored ASCII file counterparts for the built artifacts (images, container tarballs). The only artifacts for which the DIGESTS files are not created are Gentoo packages in the "push packages" job. Not sure if it's worthwhile, because the Packages file already serves as a kinda-sorta-DIGESTS-file, and it's signed by us.
How to use
Run the build (or use the CI build from below), see if the generated artifacts have DIGESTS and asc files next to them for sdk, packages, image and vms jobs.
Testing done
CI: http://jenkins.infra.kinvolk.io:8080/job/container/job/sdk/129/cldsv/

See the output:

images amd64: http://bincache.flatcar-linux.net/images/amd64/9876.0.0+krnowak-digests/
images arm64: http://bincache.flatcar-linux.net/images/arm64/9876.0.0+krnowak-digests/
containers: http://bincache.flatcar-linux.net/containers/9876.0.0-krnowak-digests/
sdk: http://bincache.flatcar-linux.net/sdk/amd64/9876.0.0+krnowak-digests/

Digests for SDK are most likely generated by catalyst. Maybe we should disable it to have digests generated by one routine. But this can happen when we disable the old pipeline.

Right now the script creates digests of almost all the files (so not only images, but also package listings, READMEs, scripts, JSON files and so on). Not sure if this is a problem, though.

Thanks, looks good already! A minor problem I see is that the filenames in the DIGEST file have a path prefix now which they shouldn't have (e.g., there is images/latest/flatcar-container.tar.gz)

I think it's ok to skip creating the summary digest files as they don't follow a clear scheme

Thanks, looks good already! A minor problem I see is that the filenames in the DIGEST file have a path prefix now which they shouldn't have (e.g., there is images/latest/flatcar-container.tar.gz)

Well spotted, thanks. I think I only checked DIGESTS for containers, and those look good. :P

Updated, I have kicked off a rebuild of a vms job (http://jenkins.infra.kinvolk.io:8080/job/container/job/vms/454/) - this should overwrite some of the DIGESTS files with a fixed versions.

Updated, I have kicked off a rebuild of a vms job (http://jenkins.infra.kinvolk.io:8080/job/container/job/vms/454/) - this should overwrite some of the DIGESTS files with a fixed versions.

Scratch that. First, I forgot to push my branch. Second, vms job pick up scripts repo from a tag, instead of my branch, so it will recreate the same bad DIGESTS files.

Started another SDK build - http://jenkins.infra.kinvolk.io:8080/job/container/job/sdk/132/

Updated, I have kicked off a rebuild of a vms job (http://jenkins.infra.kinvolk.io:8080/job/container/job/vms/454/) - this should overwrite some of the DIGESTS files with a fixed versions.

Scratch that. First, I forgot to push my branch. Second, vms job pick up scripts repo from a tag, instead of my branch, so it will recreate the same bad DIGESTS files.

You can update the tag with ./retag-for-jenkins TAG from your branch.

Backported down to 3033


Similar to "An error was reported in next.js project #153"

Describe the bug
Same issue stated in #153
To Reproduce
Steps to reproduce the behavior:
Visit my repo at https://github.com/josephfernandez/titansea/tree/wallet.  Start app using npm run dev then notice localhost:3000/mint shows identical error.
Expected behavior
See "select wallet modal" appear.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: Windows 10
Browser: Chrome
Version 100.0.4896.75 (Official Build) (64-bit)

Additional context
Notice the _app.js file - it attempts to do dynamic import as suggested in #153 importing context/ClientWalletProvider.js

Please try using the provided nextjs-starter project. #153 is closed because the underlying issue has been resolved. You may also want to check https://github.com/solana-labs/wallet-adapter/issues/238#issuecomment-1004557715.


I have an error while insttalling aihwkit

I posted this issue https://github.com/IBM/aihwkit/issues/505#issue-1706344167
now, I have cuda 11.8 installed, I tried again, it gave the following error, I have pytorch 2.0.1 and python 3.10

Hi @patrick-chen-98, Can you post a little more of the error output ? This does not tell anything. Do you actually have a GPU in your system?

Thank you for replying. I have an RTX4080 and an A2000 on my two PC. I will post more error output soon

Hi @patrick-chen-98 , I had similar issues too and I failed to install it on windows, I used these exact steps on the notebook and it worked for me on Google Colab with GPU.
aihwkit_installation_notebook.zip


Accept both str and Path types for functions in databricks/sdk/service/files.py

Changes
Currently, the functions in databricks/sdk/service/files.py accept only str for arguments that are paths. This can lead to confusing behavior for end-users when passing in a Path object.
Tests
I parametrized the unit tests.

[x] make test run locally
[x] make fmt applied
[ ] relevant integration tests applied

Hi @fpgmaas, thanks for contributing this change to the SDK! Unfortunately, the code you're modifying is autogenerated and would be overwritten in the next SDK release if merged. Implementing this would require a change in our codegen tool itself. Can I ask you to make a feature request instead?

Thanks for your response. I overlooked the comment # Code generated from OpenAPI specs by Databricks SDK Generator. DO NOT EDIT. at the top, my bad! I opened the issue https://github.com/databricks/databricks-sdk-py/issues/762.

Can users just call str(...) themselves before using these methods?

Yes, that will definitely work. However, the error that is thrown is not very clear which makes it a bit annoying to debug. So for a better user experience I think allowing Path objects to the path argument would be a nice improvement.


[bug] fix home page button style pollute other button's style

https://github.com/hlxsites/zemax/pull/99


parser fixes and improvements

some leftovers:

uop folding works but probably doesn't match expectations
would be nice to write actual strict scanners to have this very fast™, now the performance is smashed by parsing the whitespace&identifier&integer letters one by one (e.g., each real 123.345 also generates matches for 6 integer sub-suffixes, and consecutively 2 extra unnecessary suffix reals even with lexing phase on)

best!

PS. re the scanners, basically all you'd need to do is to convert all token parsers to stuff that always eats the whole tokens, like the regex ones you have there already. In your case that mostly means "eradicate Parser.satisfy"
As a side case, you have to make sure that the keywords are always matching the whole token, for example if you have:

a rule for identifiers
a rule for parsing cos

then cosechar could parse as "keyword cos" followed by "identifier echar".
To fix that, you need a lookahead in the cos scanner that says something like "this is not followed by anything identifier-ish. A common way to implement that is:
:keyword => P.seq(:cos, P.not_followed_by(:id))
...but you can also do this at the regex level.


Change DataSource Connection Pool from Druid to HiKariCP

Discussed in https://github.com/apache/dolphinscheduler/discussions/6448

Originally posted by dailidong October  5, 2021
As I found that the performance of HiKariCP Connection Pool is better than Druid , I did a small test between these two components(Informal testing), and my test program are as follows:
import org.apache.dolphinscheduler.common.enums.CommandType;
import org.apache.dolphinscheduler.common.thread.ThreadUtils;
import org.apache.dolphinscheduler.dao.datasource.ConnectionFactory;
import org.apache.dolphinscheduler.dao.entity.Command;
import org.apache.dolphinscheduler.dao.mapper.CommandMapper;
import org.slf4j.Logger;
import org.slf4j.LoggerFactory;

import java.util.Date;
import java.util.concurrent.CountDownLatch;
import java.util.concurrent.ThreadPoolExecutor;

/**
 * DataSourceTest
 *
 */
public class DataSourceTest {

    private static final Logger logger = LoggerFactory.getLogger(DataSourceTest.class);

    /**
     * datasource test
     * @param args args
     */
    public static void main(String[] args) throws InterruptedException {

        int threadNumber = 8;
        int totalTests = 10;
        ThreadPoolExecutor insertThreadPool = (ThreadPoolExecutor) ThreadUtils.newDaemonFixedThreadExecutor("insert-test-thread", threadNumber);

        Date s = new Date();

        for(int j = 0; j < totalTests; j++) {
            Date start = new Date();
            int commandCount = 1000;

            CountDownLatch latch = new CountDownLatch(commandCount);
            CommandMapper commandMapper = ConnectionFactory.getInstance().getMapper(CommandMapper.class);

            for (int i = 0; i < commandCount; i++) {
                Command command = new Command();
                command.setId(i*j);
                command.setProcessDefinitionCode(i*j);
                command.setCommandType(CommandType.START_PROCESS);
                command.setCommandParam("{\\\"ProcessInstanceId\\\":2}");
                InsertThread insertThread = new InsertThread(commandMapper, command, latch);
                insertThreadPool.execute(insertThread);
            }
            latch.await();
            Date end = new Date();
            logger.info(String.format("insert cost %d ms: thread : %d ,count: %d ", end.getTime() - start.getTime(), threadNumber, commandCount));
        }

        Date e = new Date();
        logger.info(String.format("total cost %d ms, avg cost %s ms, count: %d ", e.getTime() - s.getTime(),String.valueOf((e.getTime() - s.getTime())/(totalTests * 1.0)), totalTests));

    }

    private static class InsertThread implements Runnable {

        private CommandMapper commandMapper;

        private Command command;

        private CountDownLatch latch;

        public InsertThread(CommandMapper commandMapper, Command command,CountDownLatch latch){
            this.commandMapper = commandMapper;
            this.command = command;
            this.latch = latch;
        }

        @Override
        public void run() {
            Date start = new Date();
            this.commandMapper.insert(command);
            latch.countDown();
        }
    }

}

I test many times, the results like this:
Druid datasource
[INFO] 2021-10-05 17:48:31.782 org.apache.dolphinscheduler.dao.upgrade.DataSourceTest:[70] - insert cost 3424 ms: thread : 8 ,count: 1000
[INFO] 2021-10-05 17:48:32.070 org.apache.dolphinscheduler.dao.upgrade.DataSourceTest:[70] - insert cost 288 ms: thread : 8 ,count: 1000
[INFO] 2021-10-05 17:48:32.299 org.apache.dolphinscheduler.dao.upgrade.DataSourceTest:[70] - insert cost 229 ms: thread : 8 ,count: 1000
[INFO] 2021-10-05 17:48:32.499 org.apache.dolphinscheduler.dao.upgrade.DataSourceTest:[70] - insert cost 200 ms: thread : 8 ,count: 1000
[INFO] 2021-10-05 17:48:32.707 org.apache.dolphinscheduler.dao.upgrade.DataSourceTest:[70] - insert cost 208 ms: thread : 8 ,count: 1000
[INFO] 2021-10-05 17:48:32.959 org.apache.dolphinscheduler.dao.upgrade.DataSourceTest:[70] - insert cost 251 ms: thread : 8 ,count: 1000
[INFO] 2021-10-05 17:48:33.160 org.apache.dolphinscheduler.dao.upgrade.DataSourceTest:[70] - insert cost 200 ms: thread : 8 ,count: 1000
[INFO] 2021-10-05 17:48:33.377 org.apache.dolphinscheduler.dao.upgrade.DataSourceTest:[70] - insert cost 217 ms: thread : 8 ,count: 1000
[INFO] 2021-10-05 17:48:33.631 org.apache.dolphinscheduler.dao.upgrade.DataSourceTest:[70] - insert cost 254 ms: thread : 8 ,count: 1000
[INFO] 2021-10-05 17:48:33.883 org.apache.dolphinscheduler.dao.upgrade.DataSourceTest:[70] - insert cost 252 ms: thread : 8 ,count: 1000
[INFO] 2021-10-05 17:48:33.883 org.apache.dolphinscheduler.dao.upgrade.DataSourceTest:[74] - total cost 5526 ms, avg cost 552.6 ms, count: 10

Hikari DataSource
[INFO] 2021-10-05 19:25:22.536 org.apache.dolphinscheduler.dao.upgrade.DataSourceTest:[70] - insert cost 2906 ms: thread : 8 ,count: 1000
[INFO] 2021-10-05 19:25:22.720 org.apache.dolphinscheduler.dao.upgrade.DataSourceTest:[70] - insert cost 184 ms: thread : 8 ,count: 1000
[INFO] 2021-10-05 19:25:22.944 org.apache.dolphinscheduler.dao.upgrade.DataSourceTest:[70] - insert cost 224 ms: thread : 8 ,count: 1000
[INFO] 2021-10-05 19:25:23.085 org.apache.dolphinscheduler.dao.upgrade.DataSourceTest:[70] - insert cost 141 ms: thread : 8 ,count: 1000
[INFO] 2021-10-05 19:25:23.237 org.apache.dolphinscheduler.dao.upgrade.DataSourceTest:[70] - insert cost 152 ms: thread : 8 ,count: 1000
[INFO] 2021-10-05 19:25:23.372 org.apache.dolphinscheduler.dao.upgrade.DataSourceTest:[70] - insert cost 135 ms: thread : 8 ,count: 1000
[INFO] 2021-10-05 19:25:23.479 org.apache.dolphinscheduler.dao.upgrade.DataSourceTest:[70] - insert cost 107 ms: thread : 8 ,count: 1000
[INFO] 2021-10-05 19:25:23.608 org.apache.dolphinscheduler.dao.upgrade.DataSourceTest:[70] - insert cost 128 ms: thread : 8 ,count: 1000
[INFO] 2021-10-05 19:25:23.709 org.apache.dolphinscheduler.dao.upgrade.DataSourceTest:[70] - insert cost 101 ms: thread : 8 ,count: 1000
[INFO] 2021-10-05 19:25:23.815 org.apache.dolphinscheduler.dao.upgrade.DataSourceTest:[70] - insert cost 105 ms: thread : 8 ,count: 1000
[INFO] 2021-10-05 19:25:23.815 org.apache.dolphinscheduler.dao.upgrade.DataSourceTest:[74] - total cost 4186 ms, avg cost 418.6 ms, count: 10

In the process of inserting data to the command table, we can see that HiKariCP DataSource is indeed better than Druid.
besides, HiKariCP is officially supported by Spring Boot 2+ and has better compatibility with Spring Boot
so I suggest we use HiKariCP instead of Druid,  Do you have any suggestions?

#6828 has done this feature, I will close this issue


Fix docs build

Summary
Change Type
Mark the type of change your PR introduces:

[x] Bug fix (resolves an issue without affecting existing features)
[ ] Feature (adds new functionality without breaking changes)
[ ] Breaking change (may impact existing functionalities or require documentation updates)
[ ] Documentation (updates or additions to documentation)
[ ] Refactoring or test improvements (no bug fixes or new functionality)

Testing
Review Checklist:

[x] Reviewed my own code for quality and clarity.
[ ] Added comments to complex or tricky code sections.
[ ] Updated any affected documentation.
[ ] Included tests that validate the fix or feature.
[ ] Checked that related changes are merged.

@JAORMX Any idea why your previous PR broke the formatting?

not yet


Lost some last frames while streaming video to RTMP using FFmpeg

!!! Before submitting a new bug report, please ensure you have searched for any existing bugs. Duplicate issues or
questions that are overly simple or already addressed in the documentation will be removed without any
response.
Describe the bug
Lost some last frames while pushing video to RTMP using FFmpeg. Only when the next video streaming begins will the last few frames of the previous video be streaming out.
Attempts to resolve the issue by disabling GOP cache, using -fflags nobuffer, -flags low_delay, and -flvflags no_duration_filesize parameters proved ineffective.
Version
5.x, 6.x, v5.0.60.
To Reproduce
Steps to reproduce the behavior:

ffmpeg -re -i 1.mp4 -c copy -f flv rtmp://<IP_ADDRESS>/live/123456, will lost some last frames of 1.mp4
ffmpeg -re -i 2.mp4 -c copy -f flv rtmp://<IP_ADDRESS>/live/123456, will lost some last frames of 2.mp4
When playing rtmp://<IP_ADDRESS>/live/123456 with VLC, you will notice that the last few frames of each video are missing.

Expected behavior
Push the entire video stream, ensuring all frames are transmitted completely.
Screenshots
None
Additional context
None

SRS will send RTMP packet in bundle.
https://github.com/ossrs/srs/blob/7951bf3bd63a33ec5bfddfba39f9d6e2d476e381/trunk/src/app/srs_app_rtmp_conn.cpp#L845-L849
So, when the publish stream stopped, there are maybe still some packet cached there. So I call consumer->wakeup() when stream publish stopped, to make sure all packet send to player. But the player, ffplay, still seems missed a few frame. After I force closed the srs by ctrl-c. The last frames will play.
So, I checked the whether the TCP socket cached the last few frame without send them. It seems so. So I added the log to ffmpeg.
https://github.com/FFmpeg/FFmpeg/blob/4fc43e45f9e7ee50d8d244bcdea15b37b5614b9b/libavformat/tcp.c#L286-L300
The conclusion is that the packet all sent to ffplay. So unless I closed the TCP socket in SRS, The ffplay always cache the last few frame.


[P2P] After peer connection show the remote peer offline.

For replay this bug you need:
Test case

Go to SS and login under any account.
Go to peer registration and register peer.

Result: you can observe that remote peer offline. If you will go to Environments -> Advance mode you can observe remote peer there but without rh.

karaflogs2016-04-03 10_27_58.txt

According to logs, remote peer was inaccessible within specific timeout


plzzz help load more at start called two times

` public void onLoadMore() {
Log.d("p", "onLoadMore()");
reqExp();
loading = true;
}`

@Override     public boolean isLoading() {         Log.d("p", "isLoading()");         return loading;     }
in my response from network
 mUserLst = UserParse.Data(mResponse);                         mAdopter.add(mUserLst);                         loading = false;
my problem is that when activity start on load more called two times. i dont understand
 Paginate.with(mRv,this)                 .setLoadingTriggerThreshold(2)                 .addLoadingListItem(true)                 .setLoadingListItemCreator(new CustomLoadingListItemCreator())                 .build();

@kashifandroiddev can you assert that the value in isLoading() call is false both times ?

i dell the project because your reply is very late .....
do me a favoour
send ma code template that in working well.....
thanks


Use TS Strict Mode

Strengthen many types and fix some errors

if you're going to make 50+ commits to make this few changes i will squash these when i merge

if you're going to make 50+ commits to make this few changes i will squash these when i merge

I will rebase on my end

I will rebase on my end

I lost some structure and it's not worth recovering it so just squash on accept

I lost some structure and it's not worth recovering it so just squash on accept

I don't know what's supposed to happen in StyleMenu.tsx.
Other than that, everything works

I don't know what's supposed to happen in StyleMenu.tsx.
Other than that, everything works

Please resolve this:
https://github.com/cjquines/qboard/blob/b7b029adadeae05addc618366d1ce9a35ffa46dc/src/components/Stylebar.tsx#L47-L48
I didn't assert as non-null because I didn't know whether that was actually true

Please resolve this:
https://github.com/cjquines/qboard/blob/b7b029adadeae05addc618366d1ce9a35ffa46dc/src/components/Stylebar.tsx#L47-L48
I didn't assert as non-null because I didn't know whether that was actually true

All concerns have been resolved, this is ready

i assume you want this after #106?

That would probably make it easier to resolve conflicts (this will be squashed), so yes.

this has a lot of painful merge conflicts so i dont want to look at this yet


(MODULES-8752) Release prep for 2.0.0

Codecov Report

Merging #18 into release will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           release      #18   +/-   ##
========================================
  Coverage    89.89%   89.89%
========================================
  Files            6        6
  Lines          188      188
========================================
  Hits           169      169
  Misses          19       19

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 f8f0411...f3ae6aa. Read the comment docs.


Always return to https://www.mywebsite.com/signin-oidc after logined

I'm working under asp.net core 1.0 (.net framework 4.6.2 ). If this is my first time to login in the identity server, it returns always to https://www.mywebsite.com/signin-oidc page instead of https://www.mywebsite.com/
Here is the code in startup.cs
services.AddIdentity<ApplicationUser, IdentityRole>().AddEntityFrameworkStores<ApplicationDbContext>()               .AddDefaultTokenProviders();             //.AddUserClaimsPrincipalFactory();             services.AddMvc()                     .AddViewLocalization(Microsoft.AspNetCore.Mvc.Razor.LanguageViewLocationExpanderFormat.Suffix)                     .AddDataAnnotationsLocalization();             services.AddTransient<IEmailSender, AuthMessageSender>();             services.AddTransient<ISmsSender, AuthMessageSender>();             services.AddOptions(Configuration);             services.AddApplicationInsightsLoggerProvider();             services.AddIdentityServer()                     .SetTemporarySigningCredential()                     .AddInMemoryStores()                     .AddInMemoryScopes(Config.GetScopes())                     .AddInMemoryClients(                         Config.GetClients(Configuration["AppSettings:ClientMvcWebappRedirectUri"],                                           Configuration["AppSettings:ClientMvcWebappPostLogoutRedirectUri"],                                           Configuration["AppSettings:ClientReactDriverRedirectUri"],                                           Configuration["AppSettings:ClientReactDriverPostLogoutRedirectUri"]))                     .AddAspNetIdentity<ApplicationUser>();
Here is my config file
`return new List
{
// OpenID Connect hybrid flow and client credentials client (MVC)
new Client
{
ClientId = "client-mvc-webapp",
ClientName = "client-mvc-webapp",
AllowedGrantTypes = GrantTypes.HybridAndClientCredentials,
RequireConsent = false,
ClientSecrets =
{
new Secret("secret".Sha256())
},
RedirectUris = { redirectUri },
PostLogoutRedirectUris = { postLogoutRedirectUri },
                AllowedScopes =
                {
                    StandardScopes.OpenId.Name,
                    StandardScopes.Profile.Name,
                    StandardScopes.OfflineAccess.Name,
                    StandardScopes.Roles.Name,
                    "api1"
                }
            },
            new Client
            {
                ClientId = "client-mvc-reactapp",
                ClientName = "client-mvc-reactapp",
                AllowedGrantTypes = GrantTypes.HybridAndClientCredentials,
                RequireConsent = false,
                ClientSecrets =
                {
                    new Secret("secret".Sha256())
                },
                RedirectUris = { redirectUriReactDriver },
                PostLogoutRedirectUris = { postLogoutRedirectUriReactDriver },

                AllowedScopes =
                {
                    StandardScopes.OpenId.Name,
                    StandardScopes.Profile.Name,
                    StandardScopes.OfflineAccess.Name,
                    StandardScopes.Roles.Name,
                    "api1"
                }
            }
        };`

Everthing works perfect except the above one
Thanks

This is perfectly normal behaviour, your client's openid middleware expects that an external OpenId Connect provider returns with a request to /signin-oidc.
Than the client middleware normally redirects to your app, / by default I gueuss.

@khelben it sometimes redirects to /signin-oidc  and not to / .

sometimes it works, sometimes not. sometimes it returns to / by default, sometimes it returns to /signin-oidc, sometimes it returns a error page.
you can try with my link   www.tudingbus.com     if you click on the right-top button, you may encounter the problem....

it is because of my prod website address...with or without www...

those are two different websites.

hi @tudinger,
I got into the same problem as yours with my azure web app, please let me know how did you resolve this problem.
Thank you

Than the client middleware normally redirects to your app, / by default I gueuss

@khelben , do you know which one?
I've added UseCookieAuthentication and UseOpenIdConnectAuthentication as suggested in https://identityserver4.readthedocs.io/en/release/quickstarts/3_interactive_login.html#creating-an-mvc-client, but after Login it ends with
http://localhost:14453/signin-oidc -page not found

@michael-freidgeim-webjet
You can download the samples here https://github.com/IdentityServer/IdentityServer4.Samples

http://xxxxx:8080/signin-oidc -page not found .... I have the same problem but with IIS and when i run it localhost it works fine. any idea?


Bundle error when adding gem

Hey there,
I have this error message on the console when I add the
gem "refile", require: "refile/rails"
gem "refile-mini_magick"
in my gemfile.
I bundle and thats what I get.
Any idea ?

Bundler could not find compatible versions for gem "rack":
  In Gemfile:
    rails (~> 5.1.4) x86-mingw32 was resolved to 5.1.4, which depends on
      actionpack (= 5.1.4) x86-mingw32 was resolved to 5.1.4, which depends on
        rack (~> 2.0) x86-mingw32

    rails (~> 5.1.4) x86-mingw32 was resolved to 5.1.4, which depends on
      actionpack (= 5.1.4) x86-mingw32 was resolved to 5.1.4, which depends on
        rack (~> 2.0) x86-mingw32

    refile-mini_magick x86-mingw32 was resolved to 0.2.0, which depends on
      refile (~> 0.5) x86-mingw32 was resolved to 0.6.2, which depends on
        sinatra (~> 1.4.5) x86-mingw32 was resolved to 1.4.6, which depends on
          rack (~> 1.4) x86-mingw32

I tried to manually add the gem for rack with a version above 2.0. I got the same issue.
Thanks

I got the same issue for rails 5.
When I removed refile-mini_magick from gemfile, it installed version 0.2.5 for refile.
But if I hardcode refile to version 0.6.2 which is the current latest, it gives me the compatibility error-
Bundler could not find compatible versions for gem "sinatra":
  In snapshot (Gemfile.lock):
    sinatra (= 2.0.0)

  In Gemfile:
    sinatra

    refile (~> 0.6.2) was resolved to 0.6.2, which depends on
      sinatra (~> 1.4.5)

Running `bundle update` will rebuild your snapshot from scratch, using only
the gems in your Gemfile, which may resolve the conflict.

I am not getting how can version 0.2.5 be compatible with sinatra 2.0.0 & version 0.6.2 is not?

Solved the error-
You need to point to the master branch of both gems.
Following worked for me:
gem 'refile', require: 'refile/rails', github: 'refile/refile', branch: 'master'
gem 'refile-mini_magick', github: 'refile/refile-mini_magick', branch: 'master'

As per many other issues related to the same problem, pointing to the master branch does the job. Hope we'll get a new release soon.


"references" page in documentation not rendered correctly

The footer appears on the side:

I have the same problem in scirpy after adapting the template, so this is not an isolated issue of the instance repo.

The HTML is actually broken:

(the red underlines mean mismatching tags)
Since it happens on the reference page, I assume that means that the bibtex plugin manages to emit broken HTML, but due to the complexity of MyST, rST, Sphinx plugins, and so on, it could be anything really.

Also the encoding is broken: “BÃ¼ttner”
Seeing mojibake in 2023 makes me real nostalgic, we’re (supposed to be) like 20 years beyond this problem.

The XML looks good, the problem must be conversion to HTML or some post processing:

        <title>References</title>
        <container ids="id1">
            <citation docname="references" ids="id2">
                VBH+23
                <paragraph>Isaac Virshup, Danila Bredikhin, Lukas Heumos, Giovanni Palla, Gregor Sturm, Adam Gayoso, Ilia Kats, Mikaela Koutrouli, Philipp Angerer, Volker Bergen, Pierre Boyeau, Maren Büttner, Gokcen Eraslan, David Fischer, Max Frank, Justin Hong, Michal Klein, Marius Lange, Romain Lopez, Mohammad Lotfollahi, Malte D. Luecken, Fidel Ramirez, Jeffrey Regier, Sergei Rybakov, Anna C. Schaar, Valeh Valiollah Pour Amiri, Philipp Weiler, Galen Xing, Bonnie Berger, Dana Pe'er, Aviv Regev, Sarah A. Teichmann, Francesca Finotello, F. Alexander Wolf, Nir Yosef, Oliver Stegle, and Fabian J. Theis and. The scverse project provides a computational ecosystem for single-cell omics data analysis. <emphasis>Nature Biotechnology</emphasis>, apr 2023. URL: <reference refuri="https://doi.org/10.1038%2Fs41587-023-01733-8">https://doi.org/10.1038%2Fs41587-023-01733-8</reference>, <reference refuri="https://doi.org/10.1038/s41587-023-01733-8">doi:10.1038/s41587-023-01733-8</reference>.</paragraph>
            </citation>
        </container>


SWDT - Random Fatal exception:4 ESP8266 nodeMCU

When I compile sketch a get error like this:
In file included from d:\Moje Dokumenty\Arduino\libraries\ESP_Mail_Client\src\ESP_Mail_Client.cpp:42:
d:\Moje Dokumenty\Arduino\libraries\ESP_Mail_Client\src\ESP_Mail_SMTP.h: In member function 'void ESP_Mail_Client::getResponseStatus(const char*, esp_mail_smtp_status_code, int, esp_mail_smtp_response_status_t&)':
d:\Moje Dokumenty\Arduino\libraries\ESP_Mail_Client\src\ESP_Mail_SMTP.h:2962:25: warning: comparison of integer expressions of different signedness: 'size_t' {aka 'unsigned int'} and 'int' [-Wsign-compare]
2962 | if (strlen(buf) > codeLength && (buf[codeLength] == ' ' || buf[codeLength] == '-') && textLength > 0)
| ^~
d:\Moje Dokumenty\Arduino\libraries\ESP_Mail_Client\src\ESP_Mail_SMTP.h:2983:26: warning: comparison of integer expressions of different signedness: 'int' and 'size_t' {aka 'unsigned int'} [-Wsign-compare]
2983 | while (i < strlen(buf) && buf[i] != ' ')
| ^~~~~~~~~
d:\Moje Dokumenty\Arduino\libraries\ESP_Mail_Client\src\ESP_Mail_SMTP.h:2987:24: warning: comparison of integer expressions of different signedness: 'int' and 'size_t' {aka 'unsigned int'} [-Wsign-compare]
2987 | i = (i < strlen(buf) - 1) ? i + 1 : codeLength + 1;
| ^~~~~~~~~~~~~~
Is it posiible to generate pm_send_nullfunc error(SWDT - Random Fatal exception:4) by those errors?

That is just compiler warning.

The error in compilation error you asked is from your code.

I use it. Thank You. Always when I see warnings I think: If  is it important?

The warning is just about comparison between signed integer (int) and unsigned integer (size_t) which is not important and not affected to library operation as the value stores in the signed integer variable is actually positive.

OK. Thanks for explanation.


isfinite lookup

If you want to test that a lower or upper bound is infinite you should use PostgreSQL's lower_inf and upper_inf functions. There is an open ticket about adding them to Django. You can ask Peter J. Farrell (ticket owner) and reassign ticket to yourself if he don't mind.